Skip to content

Conversation

@niksirbi
Copy link
Member

@niksirbi niksirbi commented Oct 20, 2025

Description

What is this PR

  • Bug fix
  • Addition of a new feature
  • Other

Why is this PR needed?

The landing page for animovement changed to https://animovement.dev/, see #Movement > animovement update on Zulip.

This caused linkcheck to error, because the earlier link is now broken.

What does this PR do?

Updates the link in the README. Note that the link in the docs is autogenerated from the one in the README, so the fix propagates to the docs as well.

References

N/A

How has this PR been tested?

Local docs build and CI docs buid.

Note

CI tests fail for an unrelated reason, and I'm tackling them separately in #685

Is this a breaking change?

No.

Does this PR require an update to the documentation?

No.

Checklist:

  • The code has been tested locally
  • Tests have been added to cover all new functionality
  • The documentation has been updated to reflect any changes
  • The code has been formatted with pre-commit

@sonarqubecloud
Copy link

@niksirbi niksirbi marked this pull request as ready for review October 21, 2025 09:34
@niksirbi niksirbi requested a review from a team October 21, 2025 09:34
Copy link
Contributor

@sfmig sfmig left a comment

Choose a reason for hiding this comment

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

thanks Niko, LGTM

@niksirbi niksirbi merged commit 7a1499d into main Oct 21, 2025
23 of 43 checks passed
@niksirbi niksirbi deleted the fix-animovement-link branch October 21, 2025 10:58
animeshsasan pushed a commit to animeshsasan/movement that referenced this pull request Nov 16, 2025
github-merge-queue bot pushed a commit that referenced this pull request Nov 17, 2025
* Fixing Movement Build step

* Testing movement docs deployment

* Update contributors list

* Update links to SLEAP docs (#681)

* improve resulting displacement vectors (#657)

* cart2pol now always sets phi=0 when rho=0

* Improve displace vectors computation

* deprecate movement.kinematics.compute_displacement
* add movement.kinematics.compute_forward_displacement
* add movement.kinematics.compute_backward_displacement
* adapt examples and docstrings accordingly
* adapt tests to the implementation changes

* update compute_kinematics example

shows the existance of compute_path_length

* Add backward displacement to the example. Small edits to wording and flow

* Fix double ticks

* Deprecation warning after short summary following numpydoc style

https://numpydoc.readthedocs.io/en/latest/format.html#deprecation-warning

* Factor out deprecation test for kinematics as a general test

* Clarify handling of unsigned zeros in cart2pol

* Fix rendering bits

* Fix broken SLEAP link

---------

Co-authored-by: sfmig <[email protected]>

* [pre-commit.ci] pre-commit autoupdate (#682)

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.12.11 → v0.13.3](astral-sh/ruff-pre-commit@v0.12.11...v0.13.3)
- [github.com/pre-commit/mirrors-mypy: v1.17.1 → v1.18.2](pre-commit/mirrors-mypy@v1.17.1...v1.18.2)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* update links to animovement (#684)

* Pin netcdf<1.7.3 to fix failing integration tests (#685)

* read netCDF file with the engine used for saving them.

* Revert "read netCDF file with the engine used for saving them."

This reverts commit c8f382e.

* exclude problematic versions of xarray

* pin xarray version

* try upper-pinning h5py and netcdf4

* upper pin only on linux

* try upper-pinning only netCDF4

* also try without tables pip dependency

* Add documentation to specify Movement require identical keypoints (#150) (#658)

* Add documentation to specify that skeleton must be identical for all individuals

* Fixed tabs displaying

* Fixe tabs displaying

* Update docs/source/user_guide/input_output.md

Co-authored-by: Niko Sirmpilatze <[email protected]>

* remove note from from_file()

* Update movement/io/load_poses.py

Co-authored-by: Niko Sirmpilatze <[email protected]>

---------

Co-authored-by: Niko Sirmpilatze <[email protected]>

* [pre-commit.ci] pre-commit autoupdate (#692)

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.13.3 → v0.14.3](astral-sh/ruff-pre-commit@v0.13.3...v0.14.3)
- [github.com/mgedmin/check-manifest: 0.50 → 0.51](mgedmin/check-manifest@0.50...0.51)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* Update contributors list (#690)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Bump actions/download-artifact from 5 to 6 (#691)

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 5 to 6.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v5...v6)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

* Revert changes to main

* Add function for computing homography transform

* Fix unit tests and improve validation

* Resolve comments and add more tests

* Minor fixes as per comments

* Apply suggestion from @niksirbi

Co-authored-by: Niko Sirmpilatze <[email protected]>

* remove duplicate entry from people

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Niko Sirmpilatze <[email protected]>
Co-authored-by: carlocastoldi <[email protected]>
Co-authored-by: sfmig <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: CeliaLrt <[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.

3 participants