Skip to content

StateT #30

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
vasilmkd wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

StateT #30

vasilmkd wants to merge 2 commits into from

Conversation

@vasilmkd
Copy link
Member

@vasilmkd vasilmkd commented Nov 5, 2020

@djspiewak I'm having some trouble implementing an ApplicativeThread instance for StateT, particularly start. If you could provide some insight, that would be amazing. Thanks.

@vasilmkd vasilmkd marked this pull request as draft November 5, 2020 01:02
@vasilmkd
Copy link
Member Author

vasilmkd commented Nov 5, 2020 •
edited
Loading

And now I saw the comment on the ApplicativeThread companion object. 😆

@vasilmkd
Copy link
Member Author

vasilmkd commented Nov 5, 2020 •
edited
Loading

It basically becomes impossible to fork an MVar action, if it switches to a StateT representation, as evidenced by the failing test.

@djspiewak
Copy link
Member

djspiewak commented Nov 7, 2020

It basically becomes impossible to fork an MVar action, if it switches to a StateT representation, as evidenced by the failing test

This actually surprises me. I'll take a look at it! I would have expected fork would still work with MVar.

@vasilmkd
Copy link
Member Author

vasilmkd commented Nov 7, 2020

It could just be my own ignorance. 😆

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vasilmkd @djspiewak