Skip to content

Conversation

@samsimkin
Copy link

Includes adding pointID specific coordinates and mapping some additional fields

Includes adding pointID specific coordinates and mapping some additional fields
@clnsmth
Copy link
Contributor

clnsmth commented May 15, 2025

Hi @samsimkin, thanks for opening this pull request!

To help us understand the context and evaluate these changes properly, could you please create a new issue describing the problem this pull request aims to solve? Having more background information would be really helpful. Thanks a lot!

@sokole
Copy link
Collaborator

sokole commented May 15, 2025

@clnsmth that's a good idea. Also, I'll take a look at the proposed updates before we merge anything in. This should really affect any of the functionality of the package. Just update how the birds are mapped to ecocomDP and fix some issues we had identified.

@clnsmth
Copy link
Contributor

clnsmth commented May 15, 2025

Awesome @sokole, thanks for the heads-up! Good to know this is on your radar.

Sounds like you'll be handling the review and merge of this pull request. Thanks so much!

@samsimkin
Copy link
Author

@clnsmth Yes, I can make a Git issue. And yes, @sokole and I have been discussing this for a while. I didn't think to mention that in my intial pull request.

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