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

CI code that produces the confusion matrix from episode 2 on 3 operating systems #237

Closed
wants to merge 10 commits into from

Conversation

psteinb
Copy link
Collaborator

@psteinb psteinb commented Apr 25, 2022

This closes #233

- setup github actions CI script
- run script on ubuntu
@psteinb psteinb self-assigned this Apr 25, 2022
@psteinb psteinb marked this pull request as ready for review April 25, 2022 16:46
@psteinb
Copy link
Collaborator Author

psteinb commented Apr 25, 2022

This PR pulls in the cce app into the repo. I needed to fix a small bug in it and thought then I could include it straight away.

@psteinb
Copy link
Collaborator Author

psteinb commented Apr 25, 2022

Open question for me: should I try to find a way to either:

  • create CI artifacts (the confusion matrices in a .txt file)
  • or try to commit back the confusion matrix into the repo in any way.

Feedback welcome.

@dafnevk
Copy link
Collaborator

dafnevk commented Jul 8, 2022

This PR pulls in the cce app into the repo. I needed to fix a small bug in it and thought then I could include it straight away.

Maybe this is fine for now, the cce tool does not have an actual release and is not very actively maintained. However, we may develop and maintain it further in the esciencecenter-digital-skills organization, then we don't want to maintain a copy.

@dafnevk
Copy link
Collaborator

dafnevk commented Jul 8, 2022

Would your idea be to include the output of this action directly into the lesson material? Then I guess we need to

  1. add a (hidden) line of code into the episode that writes a .png file
  2. Have the CI commit the image back

Of course we can only include the output for one run one one OS in the final lesson.

@svenvanderburg svenvanderburg changed the base branch from gh-pages to main July 19, 2023 08:57
@svenvanderburg
Copy link
Collaborator

Closing this PR as it is stale for over a year now.

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.

Create CI job to reproduce confusion matrix
3 participants