Update phase2.md Add/refactor information about Data Types

frozenfrank commented 3 weeks ago

Overview

This represents a draft proposal for ways we could improve the handling of data types, especially with students coming from a background in interpreted languages in mind.

This expands and improves upon the changes from #29.

Discussion

With the addition of explicit instruction about data types, it becomes reasonable to remove the comments about data types from the actual "Data Format" specification.

This edition provides examples, discussion about sizing consideration, and links to official documentation to help illuminate the importance of attention to data types in general.

This edition still clearly provides the answer about which data type to use along with a discussion about pros and cons of different definitions of the data type. I feel this is reasonable since this "Scientific Computing" is not designed to test student's knowledge of data types, although learning more about them along the way is a plus.

Anticipated Requests

The reviewers may want to suggest or contribute the following adjustments before completing this PR:

Moving this content to a "reading" page where it may be more appropriate
Removing the explicit declaration of the exact data type needed in favor of additional context to equip students to make the decision themselves
Adjusting the verbiage or formatting of the material

Reviewers

@paigeid-orc What do you think about this?

paigeid-orc commented 6 days ago

I don't think we actually want to put all this in phase2.md. We changed the type and I think that's enough on this page. If you really feel strongly about including this information, create a new page for this and link to it from our Resources page.

frozenfrank commented 6 days ago

Thanks for the advice. I’ll give it a stab when I get a chance.

On Thu, Nov 7, 2024 at 8:59 AM paigeid-orc @.***> wrote:

I don't think we actually want to put all this in phase2.md. We changed the type and I think that's enough on this page. If you really feel strongly about including this information, create a new page for this and link to it from our Resources page.

— Reply to this email directly, view it on GitHub https://github.com/BYUHPC/sci-comp-course/pull/31#issuecomment-2462605865, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEXEBOFJFEYNYQ5UKE62GFTZ7OE6NAVCNFSM6AAAAABQQHWL6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRSGYYDKOBWGU . You are receiving this because you authored the thread.Message ID: @.***>

BYUHPC / sci-comp-course