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.

Sign up for GitHub

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

Jump to bottom

Merging upstream ngageoint/csm commits #432

Open
dshean opened this issue Nov 11, 2022 · 2 comments
Open

Merging upstream ngageoint/csm commits #432

dshean opened this issue Nov 11, 2022 · 2 comments

Comments

@dshean
Copy link

dshean commented Nov 11, 2022

Hi all. Yesterday we were discussing some of @oleg-alexandrov's recent work implementing usgscsm support for commercial VHR Earth Observation satellites (e.g., Airbus Pleiades-HR/Neo, Maxar WorldView-1/2/3).

I noticed that following for the csm submodule:

This branch is 41 commits ahead, 15 commits behind ngageoint:master.

I was wondering if there were any plans to merge upstream changes into the usgscsm fork? Thanks!
@acpaquette
Copy link
Collaborator

@dshean This is worth doing at some point, it may also be worth while to port some of our changes back into upstream. There could also be conflicts that need to be resolved (which it looks like there are 😢) I don't have an ETA on it but it's something I can figure out!

@acpaquette
Copy link
Collaborator

@dshean Starting this now, over a year later XD

It looks like the main difference is a CMakeFile added on our end. I should have a PR up shortly with changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dshean @acpaquette