June 29, 2022, 2:08pm
I have a simply dictionary, to get all the values I have to use Dictionary.Components and then use the values output. The Dictionary.Values node doesn’t work at all.
It isn’t really a problem, since the workaround works just fine, but I thought I would share.
The warning states that it’s an issue with rank reduction. This is because your values are lists. The node is meant to return a single value (
var) but is getting a list (
June 29, 2022, 3:32pm
I see, so this node only supports one dictionary as input.
It will take multiple dictionaries if they have singleton values. It just wont handle dictionaries whose values are lists. Look at your values in the
Dictionary.Components node. The values are sublists.
I think this was just fixed:
10:58PM - 21 Jun 22 UTC
Fixes a case like this that would fail. It now works as expected.
This introduces an API break but I think it's a necessary fix. ~~Alternatively, we can add a new property with a different but similar name to `Values` and deprecate `Values`.~~
- [x] The codebase is in a better state after this PR
- [ ] Is documented according to the [standards](https://github.com/DynamoDS/Dynamo/wiki/Coding-Standards)
- [ ] The level of testing this PR includes is appropriate
- [ ] User facing strings, if any, are extracted into `*.resx` files
- [ ] All tests pass using the self-service CI.
- [ ] Snapshot of UI changes, if any.
- [ ] Changes to the API follow [Semantic Versioning](https://github.com/DynamoDS/Dynamo/wiki/Dynamo-Versions) and are documented in the [API Changes](https://github.com/DynamoDS/Dynamo/wiki/API-Changes) document.
- [ ] This PR modifies some build requirements and the readme is updated
### Release Notes
(FILL ME IN) Brief description of the fix / enhancement. **Mandatory section**
@Amoursol what do you think?
June 29, 2022, 3:58pm
Oh yeah I see! In this example there is just 1 value per key since that makes testing go a lot faster, normally there could be multiple values per key, therefore the sublists.