Skip to content
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.

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

WIP: PyTorch-backed forward simulation #389

Closed
wants to merge 3 commits into from
Closed

Conversation

rileyjmurray
Copy link
Collaborator

PR Status

Jan 17: Just creating this to share WIP. Nothing useful as of yet.

@rileyjmurray rileyjmurray changed the base branch from master to develop January 17, 2024 14:17
@rileyjmurray rileyjmurray requested review from a team as code owners January 17, 2024 14:17
@sserita
Copy link
Contributor

sserita commented Jan 17, 2024

If this is WIP, can we keep it as a draft? While in draft mode, it does not assign reviewers/auto-request reviews from code owners, so that seems like the best workflow for not overloading email inboxes while something is not ready for review yet.

@rileyjmurray rileyjmurray marked this pull request as draft January 17, 2024 22:59
@rileyjmurray
Copy link
Collaborator Author

@sserita, marked! Sorry. I'm not used to the formal draft feature.

Can you remove the reviewers who were already assigned? GitHub says it'll keep pinging them whenever I update this. If you don't see how do that then I can close and open a new PR.

@rileyjmurray
Copy link
Collaborator Author

Superseded by #390.

@sserita
Copy link
Contributor

sserita commented Jan 18, 2024

No worries! Reviewers can turn off the notifications themselves and/or remove themselves, but I think the new issue is cleaner - that way, I'll make sure to get the emails whenever you turn it into not draft mode :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants