-
Notifications
You must be signed in to change notification settings - Fork 12
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
TRex loader #400
base: main
Are you sure you want to change the base?
TRex loader #400
Conversation
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #400 +/- ##
==========================================
- Coverage 99.80% 99.71% -0.10%
==========================================
Files 14 15 +1
Lines 1025 1050 +25
==========================================
+ Hits 1023 1047 +24
- Misses 2 3 +1 ☔ View full report in Codecov by Sentry. |
Hi @albiangela, and welcome to Thank you for getting in touch and for opening this PR. Support for
|
I forgot to mention that @roaldarbol has already implemented a TRex loader into his |
It might be worth following this chat I'm having with @mooch443 about data export formats for TRex v2 - input will be appreciated (at least on my end) as always. :-) @albiangela out of curiosity, do you have a feel for what the split is of R and Python users across MPI Animal Behaviour/Collective Behaviour for the final analysis of behaviour? Feel free to open a discussion over at animovement to not clutter this PR though. :P |
Hi all, I’m also glad to see a general interest in adding collective behavior functions. I’m interested in collaborating to expand on this, possibly with contributions from some colleagues at our institute. Happy to be in touch with you, @roaldarbol! I often refer R users to Angela |
Sounds good @albiangela, don't hesitate to ping me whenever you need my input. |
@albiangela Happy to be in touch too!
Good to know thanks! The exact split doesn't matter, just wanted to know whether animovement would be filling a niche with you guys - happy to hear that is the case! 😃 |
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
Hi!
My name is Angela Albi and I am a postdoc at MPI of Animal Behavior. I just had a chat with @vigji who introduced me to your repository! I am very excited to start contributing as well, as from our department we do a lot of behavioral analysis - in particular collective behavior.
I work closely with @mooch443 who developed TRex - a tracking software you might have heard of. We are close to releasing a new version and worked extensively on making it more user friendly, and are starting to give workshops to some institutions in Germany. I am just now starting to follow your criteria to write a loader function to add TRex to the list of softwares to load data from, and in the future I am interested in talking to you about possibly adding more collective behavior functions, also together with @jacobdavidson who is a Postdoc in Berlin, who works on large tracking datasets of honeybees
As for the loader function, TRex outputs an .npz (or .csv) file in the format of for example
And because it’s often used for multi-animal tracking, I normally get the list of files inside a folder (using glob) and loop through the files to append them and convert to a pandas dataframe. This time, I will work to adapt this logic to eventually have tracking data to give as input to your
from_numpy
function.Would you prefer me to write a more general function that loads multiple files from a folder, which I can call from the trex loading function, if that could in some way be beneficial to other multiple files data loading?
In any case I will test functions locally now, and I look forward to discussing more to possibly contribute with these new features to the package.
Best,
Angela
Why is this PR needed?
What does this PR do?
References
Please reference any existing issues/PRs that relate to this PR.
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: