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

Continuous testing should be mostly green #1106

Closed
samreid opened this issue Mar 17, 2022 · 15 comments
Closed

Continuous testing should be mostly green #1106

samreid opened this issue Mar 17, 2022 · 15 comments

Comments

@samreid
Copy link
Member

samreid commented Mar 17, 2022

As discussed at today's dev meeting, we would like to invest more resources into getting CT to show more green more of the time so there is less trouble and less noise. This will take coordination from many developers in fixing sim-specific code, common code, etc, so it is an epic-level issue. I opened this so it can be triaged and scheduled for 2022 Q2.

@samreid
Copy link
Member Author

samreid commented Jul 5, 2022

For instance, here are the red cells from the last 10 columns:

image

@kathy-phet
Copy link

I want check in on how I should interpret the above ... are the only sim-specific repos that currently need attention just the top 9 rows? Should ignore the 7/4 column? Or is 7/4 more accurate because it had more tests done and I should add those rows? (e.g. light versus dark shades).
buoyancy
center-and-variability
CCK AC
CCK DC
density
fractions-mided-numbers
gravity-and-orbits
number-play

@chrisklus
Copy link

It looks like the latest columns finished tests and has similar results to your list above. I think puppeteer is sometimes causing errors that are showing up repeatedly, like the 12:14 column from below. see the error for RIAW - it was the same for the those last 6.

image

@samreid
Copy link
Member Author

samreid commented Aug 3, 2022

I fixed 3 separate CT issues today.

I pushed a proposed fix for the acid-base-solutions CT error in phetsims/acid-base-solutions#174
I also fixed CT error phetsims/nitroglycerin#13
I also fixed CT error phetsims/circuit-construction-kit-ac-virtual-lab#6

@samreid samreid self-assigned this Aug 10, 2022
@samreid
Copy link
Member Author

samreid commented Aug 27, 2022

samreid added a commit to phetsims/fractions-common that referenced this issue Aug 27, 2022
samreid added a commit to phetsims/griddle that referenced this issue Aug 27, 2022
samreid added a commit to phetsims/ratio-and-proportion that referenced this issue Aug 27, 2022
@samreid
Copy link
Member Author

samreid commented Aug 27, 2022

After the fixes, my local CT has been been pretty green, I'll put on hold and check again in the coming week.

@samreid
Copy link
Member Author

samreid commented Aug 29, 2022

I also fixed phetsims/energy-forms-and-changes#416

@samreid
Copy link
Member Author

samreid commented Sep 1, 2022

Since last week, I have fixed or worked around 10 CT issues. I'll continue monitoring these as I have time.

@samreid
Copy link
Member Author

samreid commented Sep 8, 2022

I kept an eye on this again this week and addressed several issues.

@samreid
Copy link
Member Author

samreid commented Nov 17, 2022

Nov 17, 2022

@samreid samreid removed their assignment Nov 17, 2022
@samreid
Copy link
Member Author

samreid commented Dec 1, 2022

Here are the last 10 columns with any problems:

image

I'll look into the Circuit Construction Kit ones.

@chrisklus
Copy link

From 12/8/22 dev meeting:

Here is the current state:

image

@samreid
Copy link
Member Author

samreid commented Dec 22, 2022

Today's status:

image

@AgustinVallejo AgustinVallejo removed their assignment Jan 6, 2023
@zepumph
Copy link
Member

zepumph commented Apr 28, 2023

Today I made some changes that added some metrics to each column, now we can see how many tests are complete and how many are failing:

image

I hope this helps with this issue. There are ~40 problems on CT right now, but much of that is duplicated, and some of that is because of sparky weirdness that @jonathanolson and I are still sorting out over in https://github.com/phetsims/special-ops/issues/234.

@chrisklus chrisklus removed their assignment Apr 28, 2023
@samreid
Copy link
Member Author

samreid commented Oct 26, 2023

CT was mostly green for a while, closing. (We agreed on this as a developer meeting team). Nice work everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants