-
Notifications
You must be signed in to change notification settings - Fork 278
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
Nested attr supporting keys #631
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Can you look at the failing CI @Isaac-Flath ? Lemme know if you need help fixing it. |
That was fast :) |
Got another CI issue @Isaac-Flath ! |
Argh, another issue. I am going to sleep on this and fix it in the morning. I will get the fastai testing figured out for local tests so I can test these integration tests that are failing before pushing the next attempt. |
@jph00 All tests pass now, but it feels like there should be a better way. I tried looking for a |
@Isaac-Flath i think you want the hasattr() function |
I used it in some of the previous versions that were failing the integration tests, and tried several things using it and kept failing test so got rid of it. But I just added it back and it seems to work on local fastcore and fastai tests, so I must have been doing something different wrong. Anyway - much better. Thank you! |
Thanks @Isaac-Flath . In your other PR that uses this, please update settings.ini to require this version of fastcore , and at-mention when that's ready, and I'll release both. |
Added support for keys in
nested_attr
@jph00