larsgeb / hmclab

BSD 3-Clause "New" or "Revised" License
23 stars 0 forks source link

Salvus' license message to stdout breaks VSCode test discovery #14

Open larsgeb opened 4 years ago

larsgeb commented 4 years ago

When Salvus is imported during the import of hmc_tomography, a message is displayed showing the status of the license, e.g. here: https://github.com/larsgeb/hmc-tomography/blob/21d2bb322b7da31457b3f0ae211972db3f211391/hmc_tomography/Distributions/__init__.py#L31 VSCode's way of discovering tests is to run Pytest's discover and parse its output. This gets mixed with the Salvus output and subsequently fails with a JSON error.

This is also a bad design choice of VSCode. Other people encounter these issues too: https://github.com/microsoft/vscode-python/issues/11729.