Skip to content

Conversation

@manugv
Copy link
Collaborator

@manugv manugv commented Nov 12, 2025

I have updated the files and tested to run on python 311. However, the results are different. This can be due to update on S2 data, and scikit-image libraries. Also the files [.py] are synced with the community examples. So the reference_data in benchmark_scenarios/parcel_delineation.json need to be updated.

@manugv
Copy link
Collaborator Author

manugv commented Nov 12, 2025

#264

  • Still reference_data needs to be updated.

@algorithm-services-catalogue
Copy link

algorithm-services-catalogue bot commented Nov 12, 2025

🧹 Catalogue's Preview Site Cleanup

The preview site for this pull request has been cleaned up.

If you need a new preview, please reopen the PR or create a new one.

@soxofaan
Copy link
Contributor

are you planning to produce/upload the reference data yourself

or should we merge this and wait for a run to produce actual data and update the reference links to that?

@manugv
Copy link
Collaborator Author

manugv commented Nov 13, 2025

I would prefer if we merge this and wait for it to produce actual data. I have tested it twice and I did get same results. I didn't know how to proceed so I created a todo list in above comment.

@soxofaan
Copy link
Contributor

I would prefer if we merge this and wait for it to produce actual data. I

👍

you can merge yourself I assume?

@manugv manugv merged commit 2f937ca into main Nov 18, 2025
4 checks passed
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.

4 participants