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

Misc fixes for core from the cogent lab development branch and removal of all lab stuff. #1391

Merged
merged 448 commits into from
Dec 23, 2024

Conversation

rcoreilly
Copy link
Member

@rcoreilly rcoreilly commented Dec 22, 2024

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.*

…fset in tensor.Shape to support encoding the strides in the first NumDims dimensions of a tensor.
… 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).
Copy link
Member

@kkoreilly kkoreilly left a 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?

@rcoreilly
Copy link
Member Author

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!

@kkoreilly
Copy link
Member

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.

Copy link
Member

@kkoreilly kkoreilly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finally all good!

@kkoreilly kkoreilly merged commit d797b66 into main Dec 23, 2024
1 check passed
@kkoreilly kkoreilly deleted the miscfix branch December 23, 2024 20:53
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

Successfully merging this pull request may close these issues.

2 participants