-
Notifications
You must be signed in to change notification settings - Fork 4
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
Fix JET tests #72
Comments
@BatyLeo the only JET test failing is linked to the generic imitation loss, care to take a look? |
When I remove the |
Well then, let's do it and close this issue? |
There are some other errors, one fixed by #84, and one caused by the extension for |
What does the FrankWolfe error look like? |
Methods are not defined (because they are only defined in the extension) |
I thought loading DifferentiableFrankWolfe before applying JET tests would mean the extension is also loaded |
It does not seem to be the case inside the |
Since I figured out how to use
JET.report_package
properly, it throws a few errors for interface functions that are not implemented. For now the test is skipped, but we need to reactivate it.The text was updated successfully, but these errors were encountered: