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

WIP: library_upload_dir: improvements #7179

Conversation

bernt-matthias
Copy link
Contributor

@bernt-matthias bernt-matthias commented Jan 7, 2019

major improvement: when using upload_from_galaxy_filesystem the PATH needed to be given absolute and the script generates the complete hierarchy in Galaxy library (i.e. starting from root). Now the script has an additional parameter --root_folder ROOT which allows to upload ROOT/PATH into the library folder PATH.

minor improvements:

  • added --library_name which allows to specify the library by name
  • fixed --folder which now works as documented, ie the root folder of the library is used if not given
  • added preserve dir and tag using files names
  • also started a bit with documentation

Questions:

  • maybe there is also a better way to add filesystem folders to a library folder
  • maybe a mechanism like the one I implemented with --root should be in the API .. then this would also be possible with parsec
  • should this script stay here or is it better to integrate into (e.g.) ephemeris ... ?

@galaxybot galaxybot added this to the 19.05 milestone Jan 7, 2019
@bernt-matthias bernt-matthias force-pushed the topic/improve-library_upload_dir branch from f31024d to f909a31 Compare January 7, 2019 16:07
major improvement: when using upload_from_galaxy_filesystem the PATH needs to be given absolute and the script generates the complete hierarchy in Galaxy library (i.e. starting from root). Now the script has an additional parameter `--root_folderi ROOT` which allows to upload ROOT/PATH into the library folder PATH.

minor improvements:
- added `--library_name` which allows to specify the library by name
- fixed `--folder` which now works as documented, ie the root folder of
  the library is used if not given
- added preserve dir and tag using files names
- also started a bit with documentation
@bernt-matthias bernt-matthias force-pushed the topic/improve-library_upload_dir branch from f909a31 to 54751c1 Compare January 28, 2019 10:23
@jmchilton
Copy link
Member

should this script stay here or is it better to integrate into (e.g.) ephemeris ... ?

This script since it is here for demonstration purposes - I'm not sure it is designed to actually be used. I' rather see enhancements to https://github.com/galaxyproject/ephemeris/blob/master/src/ephemeris/setup_data_libraries.py or a second data library script there. But I'm not going to abandon this so I'll merge the PR.

@jmchilton
Copy link
Member

jmchilton commented Feb 13, 2019

Actually, there are linting problems - will you fix those?

./scripts/api/library_upload_dir.py:25:56: W291 trailing whitespace

@bernt-matthias bernt-matthias changed the title library_upload_dir: improvements WIP: library_upload_dir: improvements Feb 13, 2019
@bernt-matthias
Copy link
Contributor Author

I'm fine with adding the script to ephemeris. I thought that it would fit better there. I will test this a bit more and then PR to ephemeris.

@jmchilton jmchilton modified the milestones: 19.05, 19.09 Apr 16, 2019
@mvdbeek mvdbeek added area/libraries Data libraries and removed triage labels May 27, 2019
@martenson
Copy link
Member

@bernt-matthias how do you see the future of this PR?

@jmchilton jmchilton modified the milestones: 19.09, 20.01 Aug 22, 2019
@mvdbeek mvdbeek modified the milestones: 20.01, 20.05 Dec 17, 2019
@mvdbeek mvdbeek modified the milestones: 20.05, 20.09 Apr 20, 2020
@mvdbeek mvdbeek removed this from the 20.09 milestone Sep 16, 2020
@mvdbeek
Copy link
Member

mvdbeek commented Sep 16, 2020

I've dropped the milestone here, but I also agree with John that this should probably go as an enhancement to ephemeris (if that didn't already happen?).

@nsoranzo nsoranzo marked this pull request as draft September 17, 2022 06:59
@bernt-matthias bernt-matthias deleted the topic/improve-library_upload_dir branch November 22, 2024 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/libraries Data libraries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants