Skip to content

Added data_transfer tool#4

Merged
Andrey170170 merged 9 commits intomainfrom
data_transfer
May 22, 2025
Merged

Added data_transfer tool#4
Andrey170170 merged 9 commits intomainfrom
data_transfer

Conversation

@Andrey170170
Copy link
Copy Markdown
Collaborator

@Andrey170170 Andrey170170 commented Feb 16, 2025

Tool to transfer data from downloader format to Tree Of Life format

Things to do before it is ready for merge:

  • Test with data
  • Clean up classes
  • Write README.md

Revised the project description, added programming language classifiers, and enhanced optional dependencies with 'ruff'. Introduced new keywords and added a script entry point for better usability.
Added detailed docstrings to classes and methods for the Data Transfer module, improving clarity of filtering, scheduling, and execution workflows. Expanded the README with comprehensive descriptions of input, output, and process flow.
@Andrey170170 Andrey170170 marked this pull request as ready for review May 12, 2025 08:27
@Andrey170170
Copy link
Copy Markdown
Collaborator Author

Can't be merged for now - it needs a different tools_worker.slurm to run, need to define how to write it first for user

Andrey170170 and others added 3 commits May 20, 2025 20:29
Updated the README to include documentation for the new `tools_worker.slurm` script.
Copy link
Copy Markdown
Member

@egrace479 egrace479 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just quick formatting fix and it's good to go

Comment thread src/TreeOfLife_toolbox/data_transfer/README.md Outdated
Andrey170170 and others added 2 commits May 22, 2025 16:20
Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>
@Andrey170170 Andrey170170 requested a review from egrace479 May 22, 2025 21:38
@Andrey170170 Andrey170170 merged commit 4ef2cd5 into main May 22, 2025
@Andrey170170 Andrey170170 deleted the data_transfer branch May 22, 2025 21:55
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.

2 participants