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

chore: minor dirtree modifications #4

Open
AngryMaciek opened this issue Sep 16, 2021 · 1 comment
Open

chore: minor dirtree modifications #4

AngryMaciek opened this issue Sep 16, 2021 · 1 comment

Comments

@AngryMaciek
Copy link

I would suggest a few changes to the repo structure to conform with what is usually available:

  • create a top lvl dir resources and put go_annotations inside
  • create a top lvl dir test
    • add a simple bash script under scripts; the script would download the h5ad files from zenodo automatically and put them under test/data.
    • is theta_annotation a resource or an example input? It should be put either under resources or under test. Should this go together with the go annotations(?)
  • DeepCycle.py is a script so it should go under scripts too.
  • a standard name for a conda env is environment.yml (instead of DeepCycle_env.yml). Then you can just run conda env create and the package manager automatically looks for environment.yml file. The env will still retain the name specified within the YAML file.
@AngryMaciek
Copy link
Author

Also: people generally include their colorful GH shields on top of the README file, they are very attractive ;)
I would suggest to move the zenodo example data shield to the top and later in the text just use a link with a URL. Also, it would be nice to upload the repo itself to zenodo! Nothing wrong with having two zenodo shields on top.

Repository owner deleted a comment Feb 15, 2024
Repository owner deleted a comment Feb 15, 2024
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

No branches or pull requests

1 participant