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

Nested case objects: no default value #12

Open
mratsim opened this issue Dec 12, 2019 · 0 comments
Open

Nested case objects: no default value #12

mratsim opened this issue Dec 12, 2019 · 0 comments

Comments

@mratsim
Copy link

mratsim commented Dec 12, 2019

Companion to #10 and #11 same test case as #10.
When setting the default value of a string nested into 2 level of case objects to "attestation", the default value is the empty string.

image

mratsim added a commit to status-im/nimbus-eth2 that referenced this issue Dec 20, 2019
* nbench PoC

* Remove the yaml files from the example scenarios

* update README with current status

* Add an alternative implementation that uses defer

* Forgot to add the old proc body

* slots-processing

* allow benching state_transition failures

* Add Attestations processing (workaround confutils bug:
- status-im/nim-confutils#10
- status-im/nim-confutils#11
- status-im/nim-confutils#12

* Add CLI command in the readme

* Filter report and add notes about CPU cycles

* Report averages

* Add debugecho style time/cycle print

* Report when we skip BLS and state root verification

* Update to 0.9.3

* Generalize scenario parsing

* Support all block processing scenarios

* parallel bench runner PoC

* gitBetter load issues reporting (the load issues were invalid signature and expected to fail)
zah pushed a commit to status-im/nimbus-eth2 that referenced this issue Jan 6, 2020
* nbench PoC

* Remove the yaml files from the example scenarios

* update README with current status

* Add an alternative implementation that uses defer

* Forgot to add the old proc body

* slots-processing

* allow benching state_transition failures

* Add Attestations processing (workaround confutils bug:
- status-im/nim-confutils#10
- status-im/nim-confutils#11
- status-im/nim-confutils#12

* Add CLI command in the readme

* Filter report and add notes about CPU cycles

* Report averages

* Add debugecho style time/cycle print

* Report when we skip BLS and state root verification

* Update to 0.9.3

* Generalize scenario parsing

* Support all block processing scenarios

* parallel bench runner PoC

* gitBetter load issues reporting (the load issues were invalid signature and expected to fail)
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

No branches or pull requests

1 participant