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

Issue 352-EV7-suggestion #353

Merged
merged 10 commits into from
Dec 18, 2024
Merged

Issue 352-EV7-suggestion #353

merged 10 commits into from
Dec 18, 2024

Conversation

jkostiuk
Copy link
Contributor

My attempt at addressing #352.

First, I got rid of part (c) in 2.7.3 and changed the MC options in the revised activity. My thinking is that this more focused now around the (in)dependence and spanning/not spanning of the solution set.

Second, I copied this sequence with a more complex example (this time having 4 free variables).

My reasoning is that this example is now meaty enough that we will be forced to consider how we might test that the resulting set is independent. I suspect some teams may defer to the methods of EV4/EV6 and then we probe them about why it's the case that these guys are automatically independent.

Because this example is so meaty, I got rid of the two examples that follow but kept the one where the solution space is trivial.

@jkostiuk
Copy link
Contributor Author

tagging @StevenClontz @siwelwerd @jford1906 for thoughts

Copy link

🚀 Preview available 🚀

https://e006138f.tbil.pages.dev

@StevenClontz
Copy link
Contributor

While we're touching this, can we add <answer>s for the instructor edition?

@StevenClontz
Copy link
Contributor

Last comment for now: maybe in Fact 2.7.5 we should \textcolor these rows to emphasize why RREF of the appropriate matrix will have all pivot columns:

image

Converting this to a draft while we discuss things.

@StevenClontz StevenClontz marked this pull request as draft October 11, 2024 19:32
Copy link

🚀 Preview available 🚀

https://2bcc198a.tbil.pages.dev

@jkostiuk
Copy link
Contributor Author

While we're touching this, can we add <answer>s for the instructor edition?

do we give answers for all parts, or just the MC?

@jkostiuk
Copy link
Contributor Author

Last comment for now: maybe in Fact 2.7.5 we should \textcolor these rows to emphasize why RREF of the appropriate matrix will have all pivot columns:

image

Converting this to a draft while we discuss things.

great idea

Copy link

🚀 Preview available 🚀

https://664d828f.tbil.pages.dev

@StevenClontz
Copy link
Contributor

While we're touching this, can we add <answer>s for the instructor edition?

do we give answers for all parts, or just the MC?

Definitely MCQs, other things can't hurt either, with the ultimate goal of having answers for everything someday (at least enough for an instructor to use).

@jkostiuk
Copy link
Contributor Author

IMG_2023

leaving for the night, but here's the work for the example so we can add the answer and possibly consider restructuring the activity

Copy link

🚀 Preview available 🚀

https://b1ddda25.tbil.pages.dev

@jkostiuk
Copy link
Contributor Author

@StevenClontz I made a few changes and am ready for more input. For some reason, it's not building and there's a conflict to merge...

@siwelwerd
Copy link
Contributor

there's a conflict to merge...

It was the settings.json file, I just discarded those changes so it will merge fine. You had started work on this before we de-tracked the settings.json file is why that popped up.

Copy link

🚀 Preview available 🚀

https://7219860f.tbil.pages.dev

@jkostiuk jkostiuk marked this pull request as ready for review December 16, 2024 18:58
@jkostiuk
Copy link
Contributor Author

@StevenClontz you and I were going back and forth on this a while back and I forgot to mark it ready for review again, so now it's "ready for review"!

Copy link

🚀 Preview available 🚀

https://3d32cbe9.tbil.pages.dev

@jkostiuk
Copy link
Contributor Author

@StevenClontz just made a commit--is that what you mean?2

Copy link

🚀 Preview available 🚀

https://a5fe8fbf.tbil.pages.dev

Comment on lines 224 to 228
<answer>
<p>
D.
</p>
</answer>
Copy link
Contributor

Choose a reason for hiding this comment

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

Similar here.

Comment on lines 230 to 237
<sage language="octave">
<input>
row1=[]
row2=[]
row3=[]
rref([row1;row2;row3])
</input>
</sage>
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we usually just have a blank sage cell when we don't provide the matrix terms.

Copy link

🚀 Preview available 🚀

https://b5112b7f.tbil.pages.dev

@StevenClontz StevenClontz merged commit dffa671 into main Dec 18, 2024
4 checks passed
@siwelwerd siwelwerd mentioned this pull request Dec 18, 2024
siwelwerd added a commit that referenced this pull request Dec 20, 2024
* suggested edits

* adjusted MC options

* Update source/linear-algebra/source/02-EV/07.ptx

Co-authored-by: Steven Clontz <[email protected]>

* accepted a suggestion and fixed some typos

* Update source/linear-algebra/source/02-EV/07.ptx

Co-authored-by: jkostiuk <[email protected]>

* implemented suggestions: added answers, textcolor

* Discard changes to .vscode/settings.json

* reorganize statement/answer

* further reorganizing

---------

Co-authored-by: Steven Clontz <[email protected]>
Co-authored-by: Drew Lewis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants