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

Updated C2D script #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Updated C2D script #1

wants to merge 5 commits into from

Conversation

nklomp
Copy link

@nklomp nklomp commented Sep 22, 2023

Hi Simon,

These are updates to the Docker image and script to get everything working. It has been a bit of a struggle. It turned out to be some resource constraints on the C2D environment which caused issues. These have been addresses yesterday by our partner Delta Dao

I updated the Marketplace with the latest version and you can run and see (previous) outputs at https://www.energyshr.nl/asset/did:op:51e21e623bdca2bf72bf7b47e1b9681d5a622ace03ac0f352213eb3540a44950.

Main changes:

  • Use mimetype detection because the dataset filename will be 0 in the C2D environment, so you cannot rely on file extensions
  • Removed the RUN entry from the Dockerfile and adjusted the local run script to behave in the same way as the C2D environment (so it provides the entrypoint)
  • Removed some nice things you did in the Dockerfile, because at this moment the C2D environment wasn't working with these. I for sure will get in contact with Delta Dao about these

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.

1 participant