nschloe / stressberry

Stress tests for the Raspberry Pi :sweat_smile:
GNU General Public License v3.0
327 stars 28 forks source link

convert to matplotx from dufte, fixes #118 #119

Closed graysky2 closed 2 years ago

graysky2 commented 2 years ago

I am not python-savvy by any means but I think this does the job. Build environment for me was having python-matplotx substituting in for python-dufte (all other depds/makedeps on board).

test/test_cli.py . [ 50%] test/test_stressberry.py . [100%]

====================================================== 2 passed in 27.55s ======================================================

nschloe commented 2 years ago
nschloe commented 2 years ago

dufte needs to be removed from setup.py, too. Can you also bump the Python requirements to 3.7 in the same file and in .github/workflow/ci.yml?

graysky2 commented 2 years ago

Sure, see latest

nschloe commented 2 years ago

remove Programming Language :: Python :: 3.6

shortbloke commented 2 years ago

This PR doesn't completely replace the changes I proposed in #111 as that also added back capabilities to:

Is the "fixes nschloe#111" incorrect and should be relating to a different issue?

graysky2 commented 2 years ago

@shortbloke -

Is the "fixes #111" incorrect and should be relating to a different issue?

That is in reference to https://github.com/nschloe/stressberry/issues/118 ... typo now corrected

EDIT: I always thought Issues were just referenced by <number sign/number> whereas pull requests were referenced by <PR/number sign/number> but perhaps I am wrong.

nschloe commented 2 years ago

Thanks!