Skip to content

AER-4143 - euroclass warning#368

Merged
JornC merged 6 commits intoaerius:mainfrom
JornC:aer-4143-euroclass-warning
Feb 2, 2026
Merged

AER-4143 - euroclass warning#368
JornC merged 6 commits intoaerius:mainfrom
JornC:aer-4143-euroclass-warning

Conversation

@JornC
Copy link
Member

@JornC JornC commented Jan 28, 2026

This adds support for implementors to supply a list of removed euroclass codes, which, if they are encountered, will be converted from a specific vehicle source (now unsupported) to a custom vehicle source with 0 emission. Previously, these occurrences would result in an error, while now, they result in a warning, and a successfully converted source.

@JornC JornC marked this pull request as draft January 28, 2026 13:01
JornC added 4 commits January 28, 2026 14:41
Call convertRemovedVehicleCodes in ImaerImporter after reading sources
and before validation, enabling the euroclass removed codes feature.

Includes short-circuit optimization when no removed codes are configured,
and integration test to verify the wiring.
@JornC JornC force-pushed the aer-4143-euroclass-warning branch from 09e96cd to bfa3dfe Compare January 28, 2026 13:42
@JornC JornC requested a review from BertScholten January 28, 2026 15:04
@JornC JornC force-pushed the aer-4143-euroclass-warning branch 2 times, most recently from b2bc815 to 262687f Compare January 28, 2026 18:37
Move removed vehicle code handling from post-processing to parsing.
When a removed code is encountered in GML2VehicleUtil, the specific
vehicle is converted to a CustomVehicle with zero emissions and a
warning is added.

- Add RemovedVehicleUtil for converting specific to custom vehicles
- Add checkRemovedCode() to GMLConversionData
- Change getRemovedCodes() to return Map<GMLLegacyCodeType, Set<String>>
- Remove post-processing RemovedVehicleCodeConverter
@JornC
Copy link
Member Author

JornC commented Jan 29, 2026

(blocked by release not relevant here because it's on a later/snapshot version)

@JornC JornC requested a review from BertScholten January 30, 2026 11:36
Copy link
Member

@BertScholten BertScholten left a comment

Choose a reason for hiding this comment

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

LGTM

@JornC JornC marked this pull request as ready for review February 2, 2026 11:07
@JornC JornC merged commit dfd5270 into aerius:main Feb 2, 2026
1 check 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