Skip to content

meetup: description.md doesn't match up with canonical-data #1069

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

Closed
m-dango opened this issue Jan 4, 2018 · 10 comments · Fixed by #2147
Closed

meetup: description.md doesn't match up with canonical-data #1069

m-dango opened this issue Jan 4, 2018 · 10 comments · Fixed by #2147

Comments

@m-dango
Copy link
Member

m-dango commented Jan 4, 2018

According to the exercise description, the expected input should be a description of the date:

you will take a description of a meetup date, and return the actual meetup date.
For example, if given "The first Monday of January 2017", the correct meetup date is 2017/1/2.

It looks like things may have gotten mixed up in PR #1050.

@ErikSchierboom
Copy link
Member

I don't see how things have gotten mixed up. With the merged PR, the input looks like this:

"input": {
  "year": 2017,
  "month": 1,
  "week": "first",
  "dayofweek": "Monday"
}

To me, this matches exactly the example you quote above, which has four parts: "The first Monday of January 2017" (emphasis on different parts).

@m-dango
Copy link
Member Author

m-dango commented Jan 4, 2018

From what I gather from description.md, the key description should be the input.

EDIT: I recall a similar discussion was had previously in PR #1006 (comment)

@ErikSchierboom
Copy link
Member

@mienaikage Aha, but I only tried to conform to the new "input" format, and not update the existing format.

I'm not opposed to doing what was suggested in #1006 and converting the input to a sentence, but that should be a new PR IMHO.

@rpottsoh
Copy link
Member

rpottsoh commented Jan 4, 2018

It appears to me that the canonical data has not agreed with the description for quite some time, I think #1050 was correct as its purpose was not to bring the test data and description into agreement.

The canonical data and/or the description should be updated so they agree with each other (as has been previously discussed). It doesn't matter to me which file changes or if they both change in order to reach common ground.

@ErikSchierboom
Copy link
Member

@mienaikage Would you be interested in creating a PR for this? If so, you might want to also look at #978, which also aims to improve the meetup canonical data by introducing some grouping.

@m-dango
Copy link
Member Author

m-dango commented Jan 4, 2018

I'll create a new PR shortly.

@ErikSchierboom
Copy link
Member

Great!

@m-dango
Copy link
Member Author

m-dango commented Jan 15, 2018

Apologies for the delay, I'm still working on rewriting canonical-data.json.

@petertseng
Copy link
Member

A small request to the person who do this: Decide and explain whether to recommend that implementing language tracks must also use the full string as input (adding the parsing aspect to the required implementation for this exercise) or if instead they might feel free to continue using the structured input (in which case the problem is about turning the structured specification into a date)

@petertseng
Copy link
Member

If the person who makes that decision needs to refer back to history, that person might benefit from starting with exercism/exercism@1e87b00 the original

kytrinyx added a commit that referenced this issue Nov 10, 2022
There has been a lot of confusion around this exercise, especially
for people who don't speak English as their native language.

This attempts to explain very clearly where 'teenth' comes from,
and how it pertains to recurring monthly meetup schedules.

In doing this I have also made an executive decision to _not_
change the structure of the inputs (see #1069).
This is because I don't believe that adding string parsing to
get the pieces of the general description makes this exercise
more interesting or fun.

In order to keep the exercise focused on just the date munging,
this instead changes the description to match more closely how
the inputs are given.

Closes #1786.
Closes #1069.
kytrinyx added a commit that referenced this issue Nov 10, 2022
There has been a lot of confusion around this exercise, especially
for people who don't speak English as their native language.

This attempts to explain very clearly where 'teenth' comes from,
and how it pertains to recurring monthly meetup schedules.

In doing this I have also made an executive decision to _not_
change the structure of the inputs (see #1069).
This is because I don't believe that adding string parsing to
get the pieces of the general description makes this exercise
more interesting or fun.

In order to keep the exercise focused on just the date munging,
this instead changes the description to match more closely how
the inputs are given.

Closes #1786.
Closes #1069.
kytrinyx added a commit that referenced this issue Nov 10, 2022
There has been a lot of confusion around this exercise, especially
for people who don't speak English as their native language.

This attempts to explain very clearly where 'teenth' comes from,
and how it pertains to recurring monthly meetup schedules.

In doing this I have also made an executive decision to _not_
change the structure of the inputs (see #1069).
This is because I don't believe that adding string parsing to
get the pieces of the general description makes this exercise
more interesting or fun.

In order to keep the exercise focused on just the date munging,
this instead changes the description to match more closely how
the inputs are given.

Closes #1786.
Closes #1069.
jiegillet pushed a commit that referenced this issue Nov 18, 2022
* Clarify description of meetup

There has been a lot of confusion around this exercise, especially
for people who don't speak English as their native language.

This attempts to explain very clearly where 'teenth' comes from,
and how it pertains to recurring monthly meetup schedules.

In doing this I have also made an executive decision to _not_
change the structure of the inputs (see #1069).
This is because I don't believe that adding string parsing to
get the pieces of the general description makes this exercise
more interesting or fun.

In order to keep the exercise focused on just the date munging,
this instead changes the description to match more closely how
the inputs are given.

Closes #1786.
Closes #1069.

* Add specific dates to meetup examples

Co-authored-by: Erik Schierboom <[email protected]>

* Tweak meetup description for flow

* Ensure consistent date format in meetup

* Tweak description of meetup

Co-authored-by: Erik Schierboom <[email protected]>
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 a pull request may close this issue.

4 participants