Skip to content

Conversation

@ewgrace
Copy link

@ewgrace ewgrace commented Aug 15, 2025

Before submitting a pull request (PR), please read the contributing guide.

Please fill out as much of this template as you can, but if you have any problems or questions, just leave a comment and we will help out :)

Description

What is this PR

  • Bug fix
  • Addition of a new feature
  • Other

Why is this PR needed?

What does this PR do?

For the OWS Hackday, building a function to compute thigmotaxis index for behavioural analysis. This is creating an example on the website.

References

Solves #649

How has this PR been tested?

Please explain how any new code has been tested, and how you have ensured that no existing functionality has changed.

Is this a breaking change?

If this PR breaks any existing functionality, please explain how and why.

Does this PR require an update to the documentation?

If any features have changed, or have been added. Please explain how the
documentation has been updated.

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
Copy link
Member

Hi @ewgrace, @agholivand, @elikram101 👋🏼

Thanks for working on this during the hackday, and sorry for the slow follow-up. I had quite a backlog after Open Software Week but things are settling again.

Your work made me realise this example is hard to showcase properly without an easy way to draw ROIs in the GUI. Without that, we need too much code to define ROIs programmatically, which distracts from the main goal: demonstrating how to compute a thigmotaxis index.

So I’ve decided to prioritise #378 (ROI-drawing functionality) first. I plan to tackle it in November, and once it’s released and documented, I’ll ping you here to see if you’d like to revisit this example, entirely depending on your availability, of course.

Happy to hear any thoughts or questions in the meantime.

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