eval_baseline.py: read_meta undefined in test dataloader

Open Beginner friendly
#153 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
86/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Quiet
Tech stack
python
Domain
machine-learning

Research direction

Start in moge/test/dataloader.py at _load_instance and compare its metadata loading with moge/train/dataloader.py and moge/utils/io.py. Run moge/scripts/eval_baseline.py with configs/eval/benchmarks/diode.json and baselines/da_v2.py; done means the NameError is gone and the evaluation proceeds without the environment's pipeline package shadowing MoGe's import.

Written by the indexing model from the issue text.

Description

Running moge/scripts/eval_baseline.py with configs/eval/benchmarks/diode.json and baselines/da_v2.py fails:

File "moge/test/dataloader.py", line 96, in _load_instance
    meta = read_meta(Path(path, 'meta.json'))
NameError: name 'read_meta' is not defined

moge/train/dataloader.py uses read_json for the same path; moge/utils/io.py defines read_json but not read_meta.

Suggested fix: read_meta → read_json in moge/test/dataloader.py.

Note: a PyPI package named pipeline in the same venv can shadow MoGe's import pipeline in the test dataloader.

Dominant language
JavaScript
Stars
3k
Forks
234
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from microsoft/MoGe

All issues in microsoft/MoGe

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.