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

CSE 586 Final Code Additions #1485

Draft
wants to merge 99 commits into
base: main
Choose a base branch
from
Draft

CSE 586 Final Code Additions #1485

wants to merge 99 commits into from

Conversation

therkels
Copy link

@therkels therkels commented Dec 11, 2024

Description

PR outlining the work completed for the CSE 586 Final project. This includes the initial GinS POC, tracing, and some test cases/files we used for developing GinS.

Jayanaka-98 and others added 29 commits November 28, 2024 01:35
I lost track of what I should've been doing and did a bunch of random
stuff that isn't very useful

TODO: there's a weird bug where adding newlines around the PROGRAM_INPUT
assignment can either cause
  - total_operational_time to not be initialized
  - the print("tracer: PROGRAM_INPUT...") line in the tracer to not print
right now, it seems to "work" by reinitializing total_operational_time
again...

Presumably this has something to do with how we skip lines
(frame.f_lineno += 1) in the tracer to avoid evaluating a (potentially
side-effectful) assignment twice.
After all the trouble this caused, we should've just set a flag when we
assign to PROGRAM_INPUT and grabbed the value on the next breakpoint.
…-passing-a-program-input-to-gins

A way over complicated way to mock passing a program input to gins
@CLAassistant
Copy link

CLAassistant commented Dec 11, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 6 committers have signed the CLA.

✅ Jayanaka-98
❌ clin155
❌ therkels
❌ hejohns
❌ jmgrode
❌ Kellen Kanarios


Kellen Kanarios seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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