-
Notifications
You must be signed in to change notification settings - Fork 544
Add constructCell to CLI #1078
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
Add constructCell to CLI #1078
Conversation
Yup, updated. |
|
Here's a thought: What about making the resolution parameter optional? We can infer the res from the length of the digits. I'm raising it here because I'm considering doing something like that for the Python bindings. Another solution would be to not have a resolution parameter at all, and just read from the length of the digits. That's not so different from what, say, |
I would be fine with optional, as long as an explicit mismatch results in an error. It can be useful to have it automatically inferred when using it manually, but including the resolution inside of a batch scripting job so that you can find corrupt input data. |
Makes sense, I like this approach. Updated. |
ajfriend
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. Thanks!
constructCellto CLI.