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

Add clock sun record table #344

Merged
merged 9 commits into from
Dec 9, 2024
Merged

Conversation

damianooldoni
Copy link
Member

This pull request includes several updates to the camtraptor package, primarily focusing on adding new columns to the get_record_table() output (see #341), updating the package version, and adding new tests to ensure the correctness of the new functionality.

Updates to get_record_table() function:

  • get_record_table() now returns four new columns: longitude, latitude, clock (clock time in radians), and solar (solar time in radians).

Package version and dependency updates:

  • Updated the package version from 0.26.0 to 0.27.0 in the DESCRIPTION file.
  • Added activity (for getTime() used for clock column) and overlap (for sunTime() used for solar column) to the Imports section in the DESCRIPTION file.

New helper function:

  • Introduced a new helper function add_coordinates() to add deployment coordinates to observations based on deploymentID.

New tests:

  • Added tests to ensure warnings are returned if some observations have no timestamp, and to check that clock and solar values are always within the range [0, 2*pi].

@damianooldoni damianooldoni self-assigned this Nov 20, 2024
@PietrH
Copy link
Member

PietrH commented Nov 27, 2024

Cool new feature Damiano, well done!

Copy link
Collaborator

@MartijnUH MartijnUH left a comment

Choose a reason for hiding this comment

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

@damianooldoni works like a charm! I did not encounter any issues with setting up a pipeline for analyses of activity patterns (via activity), nor for the modelling of occupancy and/or abundance (via camtrapR and unmarked).

@damianooldoni damianooldoni merged commit 0559ca0 into main Dec 9, 2024
8 checks passed
@damianooldoni damianooldoni deleted the add-clock-sun-record-table branch December 9, 2024 08:21
damianooldoni added a commit that referenced this pull request Dec 24, 2024
Pull changes from camtraptor 0.27, see #344
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