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

Release 16.3.0 #828

Merged
merged 169 commits into from
Feb 7, 2023
Merged

Release 16.3.0 #828

merged 169 commits into from
Feb 7, 2023

Conversation

dvvanessastoiber
Copy link
Contributor

@dvvanessastoiber dvvanessastoiber commented Feb 7, 2023

Release notes

Release dependencies first

In case of dependent Phovea/TDP repositories follow dependency tree from the top:

🏁 Finish line

  • Inform colleagues and customers about the release
  • Celebrate the new release 🥳

dvzacharycutler and others added 30 commits May 6, 2022 15:46
- Add optional `diffSubtypes` to `IMultiSelectionAdapter`
- Remove `ReprovisynMultiSelectionAdapter`
- Add generics to `MultiSelectionAdapter`
- Comment functions of `MultiSelectionAdapter`
Unecessary convoluted call
- Add and refactor return types `Promise<any>` to `Promise<void>`
- Switch to async/await where possible
- await `removeDynamicColumns` before `addDynamicColumns` -> previously it was not waited for the removal
* Added build-storybook to build script

* Add tdp_core/dist alias to the storybook webpack config

* updated version of storybook (#736)

Co-authored-by: Moritz Heckmann <[email protected]>

Co-authored-by: dvmoritzschoefl <[email protected]>
Co-authored-by: Moritz Heckmann <[email protected]>
Probably a missing type cast in PR #729
Waiting for `this.built` will never resolve/terminate and show an infite loading icon.

Reason: `this.built` is created with `build()` in line 355 and it is only resolved after the `build()` call is complete.

With the changes to the selection adapter the `this.selectionAdapter.selectionChanged` now returns a valid promise. Previously, `context.add()` did not return it's promise (see 919cfb2#diff-f0f07e4a6ac30a813315b152545245d4cf26830e01b701f3e82ff7b5c22363a7R18). Now, we are waiting until all dynamic columns are added. The circle closes when we add the columns which calls `withoutTracking()` which in-turn is waiting for the resolved `this.built` promise.

As far as I can see, we always call `withoutTracking` on a built/resolved ranking view. In the other places (line 377 and 400) we have a dedicated `await this.built;` before triggering changes to dynamic columns.

In my tests adding and removing columns via score and selection adapter worked as before. I also tested reloading the page and checked the result of the session. In all cases it looks the same as before.
dvvanessastoiber and others added 21 commits November 24, 2022 09:38
* fix: remove range id mapping in DB view filter

Closes Caleydo/tdp_bi_bioinfodb#1416

* fix lint
* Cypress 11 and CI integration

* Enable cypress in build.yml

* Delete build_nightly.yml
…andas (#816)

* Upgrade deps and remove numpy, numba, pandas

* Replace BaseHTTPMiddleware with much faster ASGI equalivalent

* Increase anyio threads to 100

* Remove broken etag support

* Improved logging

* Switch to a2wsgi

* Remove a2wsgi again

* Remove greenifier extension

* Bump FastAPI to 0.89.0

* Add pytest in settings.json

* Set SQLAlchemy version to >= 1.4.40 and <= 1.4.46 to allow SQLModel > 0.0.6 (#819)

* Loosen Flask and marshmallow-sqlalchemy versions for superset

* Removed flake8, isort, pep8 in favor of ruff
* upgrade webpack for storybook

* use webpack.config from storybook and override rules

* add HtmlWebpackPlugin

* Fix storybook path aliases

* use latest version of storybook

Co-authored-by: Michael Puehringer <[email protected]>
Co-authored-by: Michael Pühringer <[email protected]>
* enabled custom vis in lineup

* Made idField optional

Co-authored-by: Moritz Heckmann <[email protected]>
Co-authored-by: Michael Puehringer <[email protected]>
…786)

Co-authored-by: Holger Stitz <[email protected]>
Co-authored-by: Michael Pühringer <[email protected]>
Co-authored-by: Michael Puehringer <[email protected]>
@dvvanessastoiber dvvanessastoiber added the release: minor PR merge results in a new minor version label Feb 7, 2023
@dvvanessastoiber dvvanessastoiber requested a review from a team as a code owner February 7, 2023 07:55
Copy link
Member

@thinkh thinkh left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@thinkh thinkh merged commit 20de9d1 into main Feb 7, 2023
@thinkh thinkh deleted the release-16.3.0 branch July 7, 2023 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release: minor PR merge results in a new minor version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants