fbopt ci failed after merged master into fbopt, with pr commandline interface entry point

[smilesun]

https://github.com/marrlab/DomainLab/actions/runs/7583966552/job/20656686984#step:7:786
with commit
aa1d5cd

no module named tensorboard

the previous commit has no such a problem.

can it be the pyproject.yaml file? https://github.com/marrlab/DomainLab/pull/740/files

[smilesun]

@agisga , will we get inconsistency now for the requirements.txt which is required by pypi and the pyproject.toml??

2319105

[smilesun]

now Creating virtualenv domainlab-okngg5wM-py3.10 in /home/runner/.cache/pypoetry/virtualenvs
Installing dependencies from lock file
Warning: poetry.lock is not consistent with pyproject.toml. You may be getting improper dependencies. Run poetry lock [--no-update] to fix it.

Because domainlab depends on tensorboard (^2.14.0) which doesn't match any versions, version solving failed.
Error: Process completed with exit code 1.

https://github.com/marrlab/DomainLab/actions/runs/7584309962/job/20657710847#step:6:132

[smilesun]

this commit revert the way how pytest is run

04a6f25

[agisga]

@agisga , will we get inconsistency now for the requirements.txt which is required by pypi and the pyproject.toml??

2319105

requirements.txt was generated from pyproject.toml before. Did you rerun https://github.com/marrlab/DomainLab/blob/master/scripts/sh_gen_requirements.sh ?

[agisga]

tensorboard

It's better to run poetry add tensorboard instead of editing pyproject.toml by hand. Then it should find a compatible version.
You may also need to run poetry lock --no-update if you still get the warning about poetry.lock.

[smilesun]

@agisga , do you have time to take a look maybe? I tried several commits but still could not resovle it.

[agisga]

@agisga , do you have time to take a look maybe? I tried several commits but still could not resovle it.

I can take a look at it tomorrow.

[smilesun]

thanks! that will be great!

[agisga]

thanks! that will be great!

The PR seems to address the issue, unless I'm misunderstanding something: #765

[smilesun]

thanks! that will be great!

The PR seems to address the issue, unless I'm misunderstanding something: #765

thanks! How did you find out the version fo tensorboard should be the one you changed to in the pyproject.toml?

486d81a