We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Masks are currently used somewhat ad-hoc here and there, in particular:
GraphSampler
GraphTransformerGFN
Instead, it would be good for GraphActionCategorical to do all the accounting of masks and mask-related code.
GraphActionCategorical
The text was updated successfully, but these errors were encountered:
This has been addressed in #129 🎉
Sorry, something went wrong.
No branches or pull requests
Masks are currently used somewhat ad-hoc here and there, in particular:
GraphSampler
explicitly uses them, but it should be abstracted awayGraphTransformerGFN
and derivatives end up applying masks themselves, but this feels like the "wrong" placeInstead, it would be good for
GraphActionCategorical
to do all the accounting of masks and mask-related code.The text was updated successfully, but these errors were encountered: