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

Fix CI tests and PyPlot failures #66

Merged
merged 9 commits into from
Jul 3, 2024
Merged

Fix CI tests and PyPlot failures #66

merged 9 commits into from
Jul 3, 2024

Conversation

askprash
Copy link
Member

@askprash askprash commented Jul 2, 2024

CI seems to be giving issues repeatedly. Pinning matplotlib==3.8 worked for the docs earlier but somehow doesn't seem to work for this anymore.

I've tried to set the environment variables etc without much success so I think this might be the opportunity to explore using PythonCall and PythonPlot instead that seem to work better according to a variety of discourse posts. It seems like it will also allow us to manage the python dependencies with a nice CondaPkg.toml file which might be what we need.

  • Replace PyCall and PyPlot with PythonCall and PythonPlot
  • Fix any syntax changes
  • Setup CondaPkg.toml
  • Fix the CI.yml

@askprash askprash changed the title Update CI.yml Fix CI tests and PyPlot failures Jul 2, 2024
@askprash askprash force-pushed the askprash-patch-1 branch 2 times, most recently from d435216 to d4c0209 Compare July 3, 2024 01:11
Pinning matplotlib version like #52
PyCall to PythonCall and PyPlot to PythonPlot

Replace PyCall with PythonCall

Update to pythonplot and pythoncall

Update subplots with `subplot_mosaic`

fix array of arrays

add pythonplot to TASOPT
Update CI.yml
@askprash askprash merged commit ec64327 into main Jul 3, 2024
3 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.

1 participant