[Diagram Editor] Bug in type inference for unzip
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 45/100
Research direction
Start by reproducing the Unzip cases in the diagram editor: compatible next operations, connection creation, and the connection helper's green or red state. Trace the type-inference and compatibility checks used by these entry points, comparing their results for Unzip. Done means all three surfaces consistently reject incompatible connections and allow compatible ones.
Written by the indexing model from the issue text.
Description
I've noticed inconsistencies when creating Unzip operations.
- Sometimes the Unzip operation shows up in the
Compatible next operationswhen it shouldn't. - Sometimes you're allowed to connect to unzip when you shouldn't
- Sometimes the connection helper shows green for unzip, despite having an incompatibility
- Sometimes the connection helper shows red for unzip but still allows the connection be created
I'm not sure if these things happen for any other operation, I've only tested this out with Unzip after noticing an error occur.
- Dominant language
- Rust
- Stars
- 75
- Forks
- 18
- Avg merge
- 12d 12h
- Merged PRs (30d)
- 3
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from open-rmf/crossflow
-
Diagram editor fails to build on Windows: build.rs can't find pnpm (pnpm.cmd)Possibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Difficulty 4/5 3-5 days Newbie friendliness 52/100
-
Difficulty 3/5 1-2 days Newbie friendliness 55/100
-
Difficulty 5/5 Over a week Newbie friendliness 45/100
-
Add cleanup event to TracedEventKindMay be free again @mxgrey claimed this 158 days ago, and no pull request is open. Open
All issues in open-rmf/crossflow
Similar issues
-
comp-formats
Difficulty 2/5 Half a day Newbie friendliness 64/100
ClickHouse/ClickHouse#131694 ·
Maintainers usually reply within 1 day
-
cli: `execCargoMetadata` decodes stdout per chunk, corrupting non-ASCII paths across pipe boundariesOpen
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Copperpour/Copperpour#500 ·
Maintainers usually reply within 1 day
-
state:triage-needed
Difficulty 1/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day