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

Milestone 4 Feedback #62

Open
zwarham opened this issue Mar 28, 2023 · 4 comments
Open

Milestone 4 Feedback #62

zwarham opened this issue Mar 28, 2023 · 4 comments

Comments

@zwarham
Copy link

zwarham commented Mar 28, 2023

  • Submission (4)
    o 4/4 mechanics
  • Improving your app (20)
    o 20/20 mechanics
  • Reproducibility – testing (10)
    o 5/5 mechanics
    o 0/5 accuracy
     Tests did not pass in testing.yaml
  • Reproducibility – continuous integration and deployment
    o 5/5 mechanics
    o 5/5 accuracy
  • Tie it all together and deliver a production ready app (50)
    o 19/20 mechanics
     App title in the browser tab is broken
    o 15/20 accuracy
     The widgets refresh twice upon the first category selection (and intermittently after that)
    o 10/10 visualization
  • Reflection
    o 6/6 reasoning
@carolinetang77
Copy link
Collaborator

Hello! I see we lost marks for the tests not passing, but according to the github actions workflow, they all passed. Could you clarify why we were docked marks for this?
image

@zwarham
Copy link
Author

zwarham commented Mar 28, 2023

Hi, this is the information we review for the workflows

image

@carolinetang77
Copy link
Collaborator

The checks did fail in our initial commit of the testing.yaml, but we fixed the issue in later commits that didn't involve changes to the yml file (we had to update the renv lockfile), and the actions currently run without error, as indicated in the github actions history and on the README.

image
image

@zwarham
Copy link
Author

zwarham commented Mar 28, 2023

In that case please create a conversation on Slack with all your group members, myself and Florencia and ask Florencia for a regrade request and explain your reasoning

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

No branches or pull requests

2 participants