feat: sort rows in EDA - Githubissues

Safe-DS / DSL

Statically checked Data Science programs.

https://dsl.safeds.com

MIT License

14 stars 0 forks source link

feat: sort rows in EDA #1183

Closed SmiteDeluxe closed 4 months ago

SmiteDeluxe commented 4 months ago

Closes #1151

Summary of Changes

History has overrideId prop on entries now and RunnerAPI only keeps last entry of each overrideId, allowing overriding sorts, voiding of sorts and in general execution of only the relevant needed entries once history is fully implemented
Thus sorting working incl. visual feedback & loading screen if > 500ms load & preserving of relevant state on Table update & removing of sort by clicking active sort again
All of pastEntries execution in correct order, with only relevant entries & correct passing on of placeholders

github-actions[bot] commented 4 months ago

🦙 MegaLinter status: ✅ SUCCESS

Descriptor	Linter	Files	Fixed	Errors	Elapsed time
✅ CSS	stylelint	1	0	0	1.61s
✅ JSON	prettier	5	0	0	2.42s
✅ REPOSITORY	git_diff	yes		no	0.11s
✅ TSX	eslint	4	0	0	7.84s
✅ TYPESCRIPT	eslint	4	0	0	7.1s
✅ TYPESCRIPT	prettier	4	0	0	1.26s

See detailed report in MegaLinter reports _Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff_

_MegaLinter is graciously provided by _

SmiteDeluxe commented 4 months ago

This ideally is merged after #1181 as it includes commits from there by having merged the hide columns branch into this branch at the start.

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.79%. Comparing base (e79de17) to head (54a1bf9). Report is 37 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #1183 +/- ## ======================================= Coverage 99.79% 99.79% ======================================= Files 110 110 Lines 19243 19243 Branches 4109 4109 ======================================= Hits 19203 19203 Misses 40 40 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

lars-reimann commented 4 months ago

:tada: This PR is included in version 0.17.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: