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

[BUG] my CNNSearchSpace implementation had to change in order for distributed training to work. #190

Open
lovettchris opened this issue Mar 10, 2023 · 1 comment
Labels
bug Something that is not working

Comments

@lovettchris
Copy link
Member

Describe the bug
I had the quick start CNNSearchSpace to start with, but when I started using it in an AsyncModelEvaluator it started crashing because in that case the mutate and crossover methods are called with ArchaiModel objects that have an uninitialize arch model. All they have is an archid.

To Reproduce

See cnn_search_space.py and compare this with the one in the quickstart notebook.

Expected behavior

We need to change the EvolutionarySearchSpace api so that it encourages the creation of a search space that works in both sync and async modes.

@lovettchris lovettchris added the bug Something that is not working label Mar 10, 2023
@asdafa3
Copy link

asdafa3 commented Apr 16, 2024

How did you solve it? The linked file does not exist anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that is not working
Projects
None yet
Development

No branches or pull requests

2 participants