Skip to content

[uss_qualifier/resource] Add base class for geospatial modifier#1467

Merged
BenjaminPelletier merged 1 commit into
interuss:mainfrom
Orbitalize:1458_geospatial_modifier
Jun 4, 2026
Merged

[uss_qualifier/resource] Add base class for geospatial modifier#1467
BenjaminPelletier merged 1 commit into
interuss:mainfrom
Orbitalize:1458_geospatial_modifier

Conversation

@the-glu

@the-glu the-glu commented May 19, 2026

Copy link
Copy Markdown
Member

Continuing on #1465 for #1458, this PR introduce:

  • GeospatialResource, as suggested to ease implementation of
  • GeospatialModifier, allowing a resource to generate unique 'Geospatial' resources

Include simple unit tests with a TestSquareResource.

@the-glu the-glu changed the title 1458 geospatial modifier [uss_qualifier/resource] Add base class for geospatial modifier May 19, 2026
@BenjaminPelletier

Copy link
Copy Markdown
Member

I think I misread #1465 and would like to adjust it before looking closely at this one. The issue is that I would expect "resource modifier" to mean something that modifies resources, but the more important aspect of ResourceModifier in #1465 is that it presents as the modified resource itself, so it seems like a better name would be "IndexAdjustableResource". That's important because in this PR we have GeospatialModifierResource and with a name like that, I would expect it to handle any geospatial modification we might want to make to resources. Instead, it's actually just one particular way to offset GeospatialResources. With AdjustableResource, I think it would be more obvious that a name like GeospatiallyOffsetResource is probably more descriptive than GeospatialModifierResource. I think this is important because this is an external surface of the product that users will directly see and touch. I'll propose an adjustment PR.

@the-glu

the-glu commented May 20, 2026

Copy link
Copy Markdown
Member Author

I think I misread #1465 and would like to adjust it before looking closely at this one. The issue is that I would expect "resource modifier" to mean something that modifies resources, but the more important aspect of ResourceModifier in #1465 is that it presents as the modified resource itself, so it seems like a better name would be "IndexAdjustableResource".

Ok, to do that (behaving as the modified resource, witch it doesn't do for now) we can just have it expose the underlying type as well + proxy with getattr()/setattr() attributes to the underling resources.

Let me know, I can also do the change :)

@the-glu the-glu force-pushed the 1458_geospatial_modifier branch from 420725a to 5cb5a06 Compare May 27, 2026 15:33
@the-glu

the-glu commented May 27, 2026

Copy link
Copy Markdown
Member Author

@BenjaminPelletier I pre-rebased that one on top of #1474

Not sure about naming for this base class, I will update next ones in the chain when that one seems ok

@BenjaminPelletier

Copy link
Copy Markdown
Member

This branch currently has merge conflicts which makes the diffs hard to interpret; can we merge main into this branch or rebase the changes onto main?

@the-glu the-glu force-pushed the 1458_geospatial_modifier branch from 5cb5a06 to d10f877 Compare June 3, 2026 19:34
@the-glu

the-glu commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

This branch currently has merge conflicts which makes the diffs hard to interpret; can we merge main into this branch or rebase the changes onto main?

Rebased, not sure why it diverged again

Comment thread monitoring/uss_qualifier/resources/modifiers.py Outdated
Comment thread monitoring/uss_qualifier/resources/geospatial.py
Comment thread monitoring/uss_qualifier/resources/modifiers.py Outdated
Comment thread monitoring/uss_qualifier/resources/geospatial.py
@the-glu the-glu force-pushed the 1458_geospatial_modifier branch 4 times, most recently from 8f7b469 to 34b5e74 Compare June 4, 2026 06:48
@the-glu

the-glu commented Jun 4, 2026

Copy link
Copy Markdown
Member Author

@BenjaminPelletier thanks for the review, everything should be fixed :)

Comment thread monitoring/uss_qualifier/resources/geospatial.py Outdated
@the-glu the-glu requested a review from BenjaminPelletier June 4, 2026 06:54
@the-glu the-glu force-pushed the 1458_geospatial_modifier branch from 34b5e74 to 6e9c7e2 Compare June 4, 2026 13:59
@BenjaminPelletier BenjaminPelletier merged commit 77f1619 into interuss:main Jun 4, 2026
23 checks passed
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.

2 participants