Skip to content

Conversation

@masinter
Copy link
Member

No description provided.

@rmkaplan
Copy link
Contributor

I think BKSYSOBJ should be a separate lispusers package. It will evolve separate from this particular demo.

@masinter
Copy link
Member Author

masinter commented Mar 24, 2023 via email

@Michele31415
Copy link

Michele31415 commented Mar 24, 2023 via email

@masinter
Copy link
Member Author

@Michele31415 I meant in this context, of a TEdit Image Object which, when clicked would BKSYSBUF to a matching Exec. It isn't reliable enough to use it programmatically, but as demo-ware it's useful.

@rmkaplan
Copy link
Contributor

rmkaplan commented Mar 25, 2023 via email

@masinter masinter requested a review from pamoroso March 31, 2023 15:10
@masinter
Copy link
Member Author

Could we make DEMOs for the #1097 screenshots?
people could then run it live and play with it, look at the code, etc.

@masinter masinter mentioned this pull request Apr 23, 2023
@masinter masinter requested a review from rmkaplan November 29, 2023 15:36
@masinter masinter requested a review from phantomics February 21, 2024 19:08
@MattHeffron
Copy link
Contributor

The DEMO-0VERVIEW.TEDIT has a zero instead of letter O in the filename. I.e., it should be DEMO-OVERVIEW.TEDIT.

Copy link
Contributor

Choose a reason for hiding this comment

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

The file name starts with 0 (zero) instead of O (capital "O"), which may not be what you wanted.

@masinter
Copy link
Member Author

it should be TEDIT.INSERT.OBJECT. I think the whole package is "aspirational" -- most of the time was spent practicing the talk and preparing the TEdit files. I really wanted a self-driving demo but didn't have it.

@masinter
Copy link
Member Author

I think there might be a charset issue? The image object is (BKSYSOBJ "(CONS NIL")unbanalced left parenthesis in string quotes. Your screenshot is missing those.

Copy link
Contributor

@pamoroso pamoroso left a comment

Choose a reason for hiding this comment

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

It seems to work as intended and it's a good start.

@masinter
Copy link
Member Author

I have been experimenting with BKSYSOBJ for Loops testing with the idea of possibly automating those tests in the 'test' repo that are labeled 'hand'.

See for example the files in the Loops repo PR Interlisp/loops#18 which uses BKSYSOBJ if you (MAKE-TEDIT-TEST TESTAV1) you will wind up with a TEdit file in which each of the steps of the test script are BKSYSBUF'd into the exec window.

This is another use case, and I think we might be able to do more automatic testing for things as well as demos.

@masinter masinter merged commit 49cb172 into master Feb 24, 2024
@masinter masinter deleted the demo-starting branch March 8, 2024 04: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.

6 participants