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

FN2 CheckIt attempt #498

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

FN2 CheckIt attempt #498

wants to merge 5 commits into from

Conversation

AbbyANoble
Copy link
Contributor

@AbbyANoble AbbyANoble commented Dec 18, 2024

I tried to make the second part model what I mention in #495, but I don't know if it is correct.

Closes #495

Copy link

🚀 Preview available 🚀

https://fa883b86.tbil.pages.dev

@AbbyANoble
Copy link
Contributor Author

Is there a way to make it all one line, listing all the x's together, instead of two lines like shown below?

Screenshot 2024-12-18 at 1 45 03 PM

Copy link

🚀 Preview available 🚀

https://7c68ac99.tbil.pages.dev

@AbbyANoble
Copy link
Contributor Author

"simple" fix. Gee, thanks Drew. haha. It does leave a trailing comma though.

Screenshot 2024-12-18 at 2 25 24 PM

@siwelwerd
Copy link
Contributor

"simple" fix. Gee, thanks Drew. haha. It does leave a trailing comma though.

Yeah, that's why it is simple. I just moved the loop so that it was only looping over the x-values, instead of the whole phrase. Will have to do something less simple to get it to look nicer (probably on the generator side)

Copy link

🚀 Preview available 🚀

https://6b3ba697.tbil.pages.dev

Copy link

🚀 Preview available 🚀

https://7dd0cb1c.tbil.pages.dev

@siwelwerd
Copy link
Contributor

I think this now hits everything you asked for in #495 @AbbyANoble , take a look and let me know.

@siwelwerd siwelwerd marked this pull request as ready for review December 18, 2024 20:05
@StevenClontz StevenClontz added the under-review This PR has been assigned a reviewer label Dec 18, 2024
Copy link

🚀 Preview available 🚀

https://b6e6a64c.tbil.pages.dev

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
precalculus under-review This PR has been assigned a reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FN2 CheckIt tweak
3 participants