Skip to content

Commit 4303a08

Browse files
authored
Merge pull request #1 from IslasGECI/hot_fix/parse_dates
⚠️✅ Remove warning when parsing dates While working on *bycatch_thesis* issue #735, an error occurred when running this recipe and its equivalents. The `get_trips()` function failed due to a missing column. Initially, this was suspected to be caused by an update to `seabird.tracking` in `construct_bl_table()`, but both functions actually use `get_trips()`. The `construct_bl_table()` function requires raw data with a `time` column, while the `time_gmt` column is specific to the seabird tracking database. The recipe `data/processed/trips_geographic_points.csv` is redundant since `bird_life_format_gps_albatross.csv` already includes trip labels. Relevant commit: ♻️ Refactor removing a date-format warning from `lubridate::parse_date_time()`.
2 parents 79b7d92 + 9e4584a commit 4303a08

File tree

2 files changed

+11
-10
lines changed

2 files changed

+11
-10
lines changed

CHANGELOG.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,13 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
66
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
77

88
## [Unreleased]
9+
### Added
10+
- `formatDT = "ymd_HMS"` in order to remove parse date time warning.
11+
### Fixed
12+
13+
### Changed
14+
15+
### Removed
916

1017
## [0.3.0] - 2025-06-25
1118

@@ -19,13 +26,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
1926

2027
## [0.1.0] - 2024-10-03
2128

22-
### Added
23-
24-
### Fixed
25-
26-
### Changed
27-
28-
### Removed
2929

3030
[unreleased]: https://github.com/IslasGECI/bycatch/compare/v0.1.0...HEAD
3131
[0.1.0]: https://github.com/IslasGECI/bycatch/releases/tag/v0.1.0

R/track_example.R

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,12 @@ get_summary_of_trips <- function(gps_data, config_content) {
88
get_trips <- function(data, config_content) {
99
dataGroup <- track2KBA::formatFields(
1010
dataGroup = data,
11-
fieldID = "track_id",
11+
fieldID = "track_id",
1212
fieldDate = "date_gmt",
1313
fieldTime = "time",
14-
fieldLon = "longitude",
15-
fieldLat = "latitude"
14+
fieldLon = "longitude",
15+
fieldLat = "latitude",
16+
formatDT = "ymd_HMS"
1617
)
1718
colony <- config_content$colony
1819
trips <- track2KBA::tripSplit(

0 commit comments

Comments
 (0)