-
-
Notifications
You must be signed in to change notification settings - Fork 87
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
Misc fixes for core from the cogent lab development branch and removal of all lab stuff. #1391
Conversation
…bad aspect of WGSL..
… and does the assignment for writable vars.
…fset in tensor.Shape to support encoding the strides in the first NumDims dimensions of a tensor.
… space b/c everything is generated.
… functions. also return *Float64 directly as that makes more sense -- goal wraps in generic tensor automatically but ops should return the most specific value possible to facilitate further opts.
… nogui; fixes for tensor, table io to not use core.Filename; finally don't have gui mode activated for goal command.
…e colors such files with distincitve blue-grey color. Fixes #1238 Also finally fixed dark / light mode switching for texteditor highlighting -- uses distinct generated theme colors instead of changing the original color, which remains intact.
…ead only; does NOT set the buffer read only status based on styling (only effect of buffer is undo -- need that for output buffers that don't need to be recording undos).
…e and ideal targets.
…differences from integer values.
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.
Fix CI! Also is this new sticky nested hovering on trees intentional?
sticky hover on tree definitely caused by switch from Actual -> Alloc in the bbox computation. trying to fix -- will pick up again in am. don't hold up merging for that -- can definitely will do a separate PR on that! |
…t to have the same name as generated files
For future reference, we are bailing on Alloc to Actual change here since it causes the tree sticky hovering, so that will just happen in #1389. |
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.
Finally all good!
verified that everything in lab is working with this branch.
*From @kkoreilly: this is built on #1229, except with goal, tensor, plot, etc removed given that they are now in Cogent Lab (that removal is the main cause of the large number of line deletions).
See https://github.com/emer/etensor for a copy of the old tensor and plot functionality.*