vitessce / vitessce-python

Python API and Jupyter widget for Vitessce
https://python-docs.vitessce.io
MIT License
40 stars 7 forks source link

Separate dev dependencies from actual dependencies #376

Closed flying-sheep closed 1 month ago

flying-sheep commented 1 month ago

black should almost certainly not be in project.dependencies.

keller-mark commented 1 month ago

It is a bit unusual but that one is correct. See https://github.com/vitessce/vitessce-python/blob/main/vitessce/config.py#L1687

keller-mark commented 1 month ago

This is used to construct Jupyter notebooks that contain Python code. The use case is to go from JSON dict to the equivalent Python code block that uses the VitessceConfig API that will generate that same Vitessce configuration

keller-mark commented 1 month ago

But would be open to making it optional and imported in the function scope if that is preferred

flying-sheep commented 1 month ago

I see, interesting! Sorry for assuming here.