Skip to content

Conversation

hmzh-khn
Copy link

@hmzh-khn hmzh-khn commented Mar 12, 2024

The visualizer currently assumes that assets like in ../assets, which prevents us from running this code from anywhere but the root directory of this repo. We generalize this by adding a CLI argument with a convenient default.

Since the datasets have visualizer_params folders, we do the same and add an argument for that file.

Replaces #8.

hmzh-khn and others added 3 commits March 11, 2024 11:26
…ory.

For convenience, and due to the structure of the various datasets, I make it easier for a user to specify where the dataset is located without also having to specify the location of the visualizer params.
Default visualizer parameters to living under provided dataset directory
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant