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

Update worked examples #1

Merged
merged 27 commits into from
Nov 21, 2024
Merged

Update worked examples #1

merged 27 commits into from
Nov 21, 2024

Conversation

rwood-97
Copy link
Collaborator

@rwood-97 rwood-97 commented Nov 8, 2024

This PR:

  • Streamlines which worked examples we give to include:
    • Standard classification pipeline
    • Context classification pipeline
    • Text spotting with DPText-DETR and DeepSolo
    • Plant phenotype classification example
    • Mnist classification example
  • Updates docker build, we don't pre-download all the maps in the docker build as we probably will intend this for use by general users vs workshops
  • Adds a scheduled (1st of each month) github workflow to check worked examples are running fine with most up to date MapReader code

@rwood-97
Copy link
Collaborator Author

rwood-97 commented Nov 14, 2024

@kallewesterling could you check this still runs on your laptop and also potentially update the docker hub image after we have merged.

also sorry re. the number of files changes in this PR, it looks like loads but I think its not picking up renaming properly I fixed this

@kallewesterling
Copy link
Contributor

Oh, this looks great @rwood-97. I can try to run it while attending the meeting tonight to see if it works :)

Switch to main branch in requirements.txt
Copy link
Contributor

@kallewesterling kallewesterling left a comment

Choose a reason for hiding this comment

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

LGTM

@rwood-97 rwood-97 merged commit 6468386 into main Nov 21, 2024
1 check passed
@rwood-97 rwood-97 deleted the dev branch November 21, 2024 14:45
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