Skip to content

[Draft] Replace level_map and deprecate reference_point parameter#92

Draft
cfs-data wants to merge 5 commits intomainfrom
replace-map-level
Draft

[Draft] Replace level_map and deprecate reference_point parameter#92
cfs-data wants to merge 5 commits intomainfrom
replace-map-level

Conversation

@cfs-data
Copy link
Collaborator

@cfs-data cfs-data commented Jan 21, 2026

The intention for now is not to merge this PR, but to use it for testing. If no problems are encountered over time, this could be considered a replacement of the current implementation.

Changes:

  1. Our implementation of level_map() is replaced by a built-in function from surfalize
  2. It seems reference_point has no influence on end-to-end tests so this parameter has been removed in this branch.

@cfs-data cfs-data changed the title Replace level_map with built-in function [Draft] Replace level_map with built-in function Jan 21, 2026
@cfs-data cfs-data marked this pull request as draft January 21, 2026 13:43
@cfs-data cfs-data changed the title [Draft] Replace level_map with built-in function [Draft] Replace level_map with built-in function and depecrate reference_point parameter Jan 21, 2026
@cfs-data cfs-data changed the title [Draft] Replace level_map with built-in function and depecrate reference_point parameter [Draft] Replace level_map and depecrate reference_point parameter Jan 21, 2026
@cfs-data cfs-data changed the title [Draft] Replace level_map and depecrate reference_point parameter [Draft] Replace level_map and deprecrate reference_point parameter Jan 21, 2026
@cfs-data cfs-data changed the title [Draft] Replace level_map and deprecrate reference_point parameter [Draft] Replace level_map and deprecate reference_point parameter Jan 21, 2026
@snregales
Copy link
Collaborator

@Raytesnel will PR impact us?

@Raytesnel
Copy link
Collaborator

@Raytesnel will PR impact us?

ja

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Diff Coverage

Diff: origin/main..HEAD, staged and unstaged changes

  • packages/scratch-core/src/conversion/leveling/core.py (100%)
  • packages/scratch-core/src/conversion/leveling/data_types.py (100%)
  • packages/scratch-core/src/conversion/preprocess_impression/preprocess_impression.py (100%)

Summary

  • Total: 18 lines
  • Missing: 0 lines
  • Coverage: 100%

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Code Coverage

Package Line Rate Branch Rate Health
. 96% 88%
comparators 100% 100%
container_models 99% 100%
conversion 97% 86%
conversion.export 100% 100%
conversion.filter 92% 83%
conversion.leveling 100% 100%
conversion.plots 98% 85%
conversion.preprocess_impression 99% 91%
conversion.preprocess_striation 89% 58%
extractors 98% 75%
parsers 98% 80%
parsers.patches 89% 60%
preprocessors 95% 75%
processors 100% 100%
renders 98% 50%
utils 91% 75%
Summary 96% (1831 / 1904) 81% (202 / 248)

Minimum allowed line rate is 50%

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