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

Update CellLineMetaData #334

Merged
merged 27 commits into from
Aug 8, 2023
Merged

Update CellLineMetaData #334

merged 27 commits into from
Aug 8, 2023

Conversation

wxicu
Copy link
Collaborator

@wxicu wxicu commented Aug 7, 2023

PR Checklist

  • Referenced issue is linked
  • If you've fixed a bug or added code that should be tested, add tests!
  • Documentation in docs is updated

Description of changes

Technical details

Additional context

Copy link
Member

@Zethson Zethson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of the "overview of available stuff" and "what it contains" etc are things that I think we should document in the docstring and not necessarily always print out.

For the general design of everything please also have a look at https://github.com/laminlabs/bionty

pertpy/tools/_metadata/_cell_line.py Outdated Show resolved Hide resolved
pertpy/tools/_metadata/_cell_line.py Outdated Show resolved Hide resolved
pertpy/tools/_metadata/_look_up.py Outdated Show resolved Hide resolved
pertpy/tools/_metadata/_look_up.py Show resolved Hide resolved
@wxicu
Copy link
Collaborator Author

wxicu commented Aug 7, 2023

This PR should also be related to #266

@Zethson Zethson merged commit bbdf99b into scverse:development Aug 8, 2023
7 of 9 checks passed
Zethson added a commit that referenced this pull request Aug 21, 2023
* RTD config

Signed-off-by: zethson <[email protected]>

* RTD config

Signed-off-by: zethson <[email protected]>

* RTD config (#333)

* RTD config

Signed-off-by: zethson <[email protected]>

* RTD config

Signed-off-by: zethson <[email protected]>

* RTD config

Signed-off-by: zethson <[email protected]>

* RTD config

Signed-off-by: zethson <[email protected]>

---------

Signed-off-by: zethson <[email protected]>

* Update CellLineMetaData (#334)

* update cell line annotation

* add LookUp object

* pass nox test

* fix test

* keep downloaded files

* improve error message and lookup initialize

* add annotate_from_gdsc

* add metadata source in the docstring

* fix typo

* update test script and speed up downloading metadata

* add remotezip dependency

* parametrize Pytest fixture

* fix import error

* poetry update

Signed-off-by: zethson <[email protected]>

* Fix fixture

Signed-off-by: zethson <[email protected]>

* fix test error

* Refactor

Signed-off-by: zethson <[email protected]>

* Refactor

Signed-off-by: zethson <[email protected]>

* Refactor

Signed-off-by: zethson <[email protected]>

* improve LookUp object

* add openpyxl

Signed-off-by: zethson <[email protected]>

* update lookup

* replace dict with namedtuple

---------

Signed-off-by: zethson <[email protected]>
Co-authored-by: Lukas Heumos <[email protected]>

* Remove whitespace in bulk RNA expression (#337)

* remove whitespace in bulk RNA annotation

* reformat

* add details of random whitespace issue

* [DEPENDABOT]: Bump pypa/gh-action-pypi-publish from 1.8.8 to 1.8.10 (#342)

Bumps [pypa/gh-action-pypi-publish](https://github.com/pypa/gh-action-pypi-publish) from 1.8.8 to 1.8.10.
- [Release notes](https://github.com/pypa/gh-action-pypi-publish/releases)
- [Commits](pypa/gh-action-pypi-publish@v1.8.8...v1.8.10)

---
updated-dependencies:
- dependency-name: pypa/gh-action-pypi-publish
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* [DEPENDABOT]: Bump mypy from 1.4.1 to 1.5.0 (#341)

Bumps [mypy](https://github.com/python/mypy) from 1.4.1 to 1.5.0.
- [Commits](python/mypy@v1.4.1...v1.5.0)

---
updated-dependencies:
- dependency-name: mypy
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* [DEPENDABOT]: Bump coverage from 7.2.7 to 7.3.0 (#340)

Bumps [coverage](https://github.com/nedbat/coveragepy) from 7.2.7 to 7.3.0.
- [Release notes](https://github.com/nedbat/coveragepy/releases)
- [Changelog](https://github.com/nedbat/coveragepy/blob/master/CHANGES.rst)
- [Commits](nedbat/coveragepy@7.2.7...7.3.0)

---
updated-dependencies:
- dependency-name: coverage
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Python 3.11 support

Signed-off-by: zethson <[email protected]>

* palette support for da_neighborhoods (#343)

* palette support for da_neighborhoods

Signed-off-by: zethson <[email protected]>

* Remove custom css

Signed-off-by: zethson <[email protected]>

---------

Signed-off-by: zethson <[email protected]>

* Fix #284

Signed-off-by: zethson <[email protected]>

* Switch to scverse cookiecutter (#349)

* Remove cookietemple

Signed-off-by: zethson <[email protected]>

* pre-commit

Signed-off-by: zethson <[email protected]>

* pyproject toml

Signed-off-by: zethson <[email protected]>

* pre-commit

Signed-off-by: zethson <[email protected]>

* Add new wfs

Signed-off-by: zethson <[email protected]>

* RTD config

Signed-off-by: zethson <[email protected]>

* references

Signed-off-by: zethson <[email protected]>

---------

Signed-off-by: zethson <[email protected]>

* RTD config

Signed-off-by: zethson <[email protected]>

* sphinx-design

Signed-off-by: zethson <[email protected]>

---------

Signed-off-by: zethson <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Xichen Wu <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

2 participants