Fixed incorrect data #153

Merged

snedadah merged 2 commits from fix-data into main 2 months ago

snedadah commented 2 months ago

Owner

There is no content yet.

snedadah added 1 commit 2 months ago

93d0a69d78 Fixed incorrect data

snedadah requested review from e26chiu 2 months ago

e26chiu added 1 commit 2 months ago

fbb1bb4bd9 Correct data

e26chiu approved these changes 2 months ago

e26chiu left a comment

Collaborator

Looks like every point in the document was addressed, good job @snedadah ! I changed some of the data points according to the google sheets (inconsistencies, etc.) Everything should be ready to merge

data/post-grad.ts Outdated

													
					    text: "Computer Networking",
				
					    value: 2,
				
					  },
				
					  {

e26chiu commented 2 months ago

Collaborator

I also noticed in the original data, there is a generic Computer Software category which most people opted (65). I didn't include this category in my change because it wasn't telling us anything new. Do we want to still include this category?

snedadah commented 2 months ago

Poster

Owner

No I don't think we should

snedadah merged commit 09978edd0e into main 2 months ago

snedadah deleted branch fix-data 2 months ago

snedadah referenced this issue from a commit 2 months ago

Fixed incorrect data (#153)

Reviewers

e26chiu approved these changes 2 months ago

continuous-integration/drone/push Build is passing

Details

The pull request has been merged as 09978edd0e.

Sign in to join this conversation.

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: www/cs-2022-class-profile#153