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.

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

Remove Dimension::Undefined #188

Closed
wants to merge 478 commits into from
Closed

Conversation

Weibye
Copy link
Collaborator

@Weibye Weibye commented Jun 23, 2022

Objective

Fixes #114

Replaces #160 and #148

Context

The new Resolve-traits really helped clean this up :)

Feedback wanted

  • I'm not yet sure why the simple_child test are failing

@Weibye
Copy link
Collaborator Author

Weibye commented Jun 23, 2022

I suspect the current failures comes from Measure functions assuming Size<Dimension> and not Size<Option<Dimension>> so that is what I will investigate next

@Weibye Weibye changed the title Remove Dimension::Undefined v3 Remove Dimension::Undefined Jun 24, 2022
@Weibye
Copy link
Collaborator Author

Weibye commented Jun 24, 2022

The generated files are still showing in the review. Is #149 not working as intended?

Weibye and others added 14 commits June 26, 2022 20:07
* Cleanup `new_with_children()` uses

* Format

* Update scripts/gentest/src/main.rs

Co-authored-by: Alice Cecile <[email protected]>
Updates the requirements on [rstest](https://github.com/la10736/rstest) to permit the latest version.
- [Release notes](https://github.com/la10736/rstest/releases)
- [Changelog](https://github.com/la10736/rstest/blob/master/CHANGELOG.md)
- [Commits](la10736/rstest@0.14.0...0.15.0)

---
updated-dependencies:
- dependency-name: rstest
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Forest unit tests

* Removing old Forest compute_layout uses

* Forest unit tests additions and refactoring

* Code style adjustments

* Removing Forest node_measure_eq warning

Co-authored-by: Alice Cecile <[email protected]>
…oxusLabs#196)

* feat: use slab instead of bespoke arena

* fix: all but one test

* chore: rip out forest and merge it into taffy

* fix: final tests

* polish: fix up some warnings

* chore: clean up forest and data fails

* chore: make clippy happier

* fix: compilation for no-std targets

* chore: clean up some more clippy stuff and renable more tests

* chore: remove outdated indexmap

* fix: update benches to not unwrap new elements

* fix: revert back to errors everywhere

* fix: revert complex bench too

* chore: remove unused import

* fix: removing parents

* chore: clean up and remove references to forest

* chore: remove final reference to forest

* chore: update release log

* Update error name in RELEASES

* Add single parent fix to RELEASES

* Remove extra whitespace

* chore: add warning to mark dirty function

Co-authored-by: Alice Cecile <[email protected]>
@Weibye
Copy link
Collaborator Author

Weibye commented Jul 14, 2022

🤔 Tests fails on height calculation but all width calculations are correct. Trying to dig into why that is but progress is slow (lots of new things to learn)

@alice-i-cecile
Copy link
Collaborator

Going to close this out: this will be a pain to rebase. Anyone can feel free to remake this though!

@Weibye
Copy link
Collaborator Author

Weibye commented Nov 19, 2022

Yeah, it was a worthwhile attempt but I couldn't get it across the finish line due to some nuances in the algorithm I could not fully figure out. Now with taffy undergoing some great improvements I think this could be easier to attempt again after #238 and #246 should it still prove useful to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change A change that breaks our public interface code quality Make the code cleaner or prettier.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider removing Dimension::Undefined