automl / automl/mighty_project_template

Exemplary run scripts fail

Open
#1 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Hi there,

while doing the review for JOSS (https://github.com/openjournals/joss-reviews/issues/10439) I checked out this template repository and tried to run some of the runscripts. I'm not exactly sure what the expectation management here is, so I will consider that they should run after a fresh clone and `make install` - otherwise it's something to document.

The two HPO scripts cannot run due to bad paths (`mighty_domain_randomization`).
The multiple seeds script fails on a missing dependency:

```bash
(mighty_project_template) janik@Mac mighty_project_template % ./runscripts/run_multiple_seeds.sh
Traceback (most recent call last):
File "/Users/janik/source/mighty_project_template/mighty_project/run_mighty.py", line 8, in
import minigrid # noqa: F401
^^^^^^^^^^^^^^^
ModuleNotFoundError: No module named 'minigrid'
```

I don't think this is a blocking issue for the JOSS review, but since it's linked in PyPi I stumbled over it 🙂

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with make install and run the two HPO scripts plus runscripts/run_multiple_seeds.sh. Check the HPO scripts' mighty_domain_randomization paths and the missing minigrid dependency reported from mighty_project/run_mighty.py. Done means the scripts run after a fresh clone and make install, or their prerequisites and limitations are documented.

Written by the indexing model from the issue text.

Assessment

Tech stack
python, shell
Domain
tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.