|
| 1 | +# Jan 14, 2021 |
| 2 | + |
| 3 | +## Attendees |
| 4 | + |
| 5 | +- Richard Gibson (RGN) |
| 6 | +- Shane F. Carr (SFC) |
| 7 | +- Younies Mahmoud (YMD) |
| 8 | +- Cam Tenny (CJT) |
| 9 | +- Philip Chimento (PFC) |
| 10 | +- Ujjwal Sharma (USA) |
| 11 | +- Justin Grant (JGT) |
| 12 | +- Leo Balter (LEO) |
| 13 | +- Philipp Dunkel (PDL) |
| 14 | +- Manish Goregaokar (MG) |
| 15 | +- Daniel Ehrenberg (DE) |
| 16 | + |
| 17 | +## Agenda |
| 18 | + |
| 19 | +### Stage 3 status |
| 20 | +- PFC: Currently the open questions are the IETF draft, the question of whether DurationFormat stage 3 is a blocker, and the issues that came up during the calendar research that are included on the agenda below. |
| 21 | + |
| 22 | +### Do Temporal Stage 3 and DurationFormat Stage 3 depend on each other? |
| 23 | +- USA: What do others think, should Temporal Stage 3 be blocked on DurationFormat Stage 3? Given the agenda deadline of tomorrow. |
| 24 | +- JGT: If they could be done in parallel, that would be great. |
| 25 | +- LEO: If the sentiment in the group is to advance to Stage 3, even if there are some open questions, it should go on the agenda, but please make sure to include slides with the open questions such as this one. |
| 26 | +- USA: Even if we are dealing with larger questions for DurationFormat, such as "should there be a DurationFormat object or part of RelativeTimeFormat"? |
| 27 | +- PFC: That might be an objection for Stage 3 of DurationFormat, but not Temporal. |
| 28 | +- USA: I agree it doesn't make sense to do DurationFormat before Temporal, but does it make sense to do Temporal before DurationFormat? |
| 29 | +- SFC: Temporal is one of the most expensive proposals to implement that will have been passed by TC39 in a while. Implementations won't have gotten around to implementing `Duration.toLocaleString()` by the time DurationFormat proceeds. |
| 30 | + |
| 31 | +### IETF draft |
| 32 | +- USA: The agreed-upon format for annotations is `[namespace-key-value]`. One-letter namespaces are reserved for BCP 47, so Unicode Consortium gets the namespace "u". If everyone in this call is happy with it, my intention is to finish the draft this week and submit it for adoption. |
| 33 | +- RGN: High-level question, where do you expect the registry of extensions to live? IANA, or Unicode Consortium? |
| 34 | +- USA: I'm copying the process described in BCP 47 for this. They are managed by IANA. |
| 35 | +- RGN: That's the norm for IETF. Since there were other groups involved I wasn't sure, but this is fine. I think it's the best possible outcome. |
| 36 | +- USA: This also allows registering extensions without amending the RFC. |
| 37 | +- RGN: What's the best way to give feedback on the draft? |
| 38 | +- USA: GitHub issues or email. |
| 39 | +- PFC: Do we need to register our calendar key, or is it already registered because it gets the existing `u-ca-` one from BCP 47? |
| 40 | +- USA: Possibly, but given its prior existence I think it's pretty safe. |
| 41 | + |
| 42 | +### DurationFormat issues; `Duration.toLocaleString()` |
| 43 | +- Should we format absolute duration? e.g. “3 days” or “4 days and 5 hours” |
| 44 | +- JGT: If we adopt RelativeTimeFormat, then the default is already decided for us. What's the argument in favour of a separate object? |
| 45 | +- USA: The argument is that it feels weird to have an option to RelativeTimeFormat to format a duration as absolute. |
| 46 | +- YMD: RelativeTimeFormat for relative, DurationFormat for absolute. |
| 47 | +- SFC: It's good to have the output of `toLocaleString()` represent the same concept as the output of `toString()`, which is absolute, disregarding the sign for the moment. The second issue is that RelativeTimeFormat currently does not support formatting of mixed units. That would require an additional proposal, and neither CLDR or ICU currently support it. This feature has been requested, but that's a 2022 time scale feature. |
| 48 | +- JGT: That's the same problem as DurationFormat. |
| 49 | +- SFC: It's a completely different code path. |
| 50 | +- YMD: ??? |
| 51 | +- JGT: If you send a negative duration into a localized method, it should not just drop the sign. |
| 52 | +- SFC: Negative durations should not determine the decision. We should optimize for the case of positive durations, and then decide how to handle a negative sign. |
| 53 | +- JGT: Do you think that the localized output for `+PT2H` and `-PT2H` should be the same? |
| 54 | +- SFC: I don't have a strong feeling either way. |
| 55 | +- JGT: That question drives me to think that the default output should be relative. |
| 56 | +- SFC: Why could it not be the absolute output with a minus sign? |
| 57 | +- YMD: I would prefer to agree whether we need the DurationFormat or not, and then decide what to do with negative durations. |
| 58 | +- JGT: I disagree. Given that the three formats exist, spreading them across two separate objects seems wrong. |
| 59 | + |
| 60 | +### Make reference year / day required in MonthDay / YearMonth [#1240](https://github.com/tc39/proposal-temporal/issues/1240) and MonthDay.equals() [#1239](https://github.com/tc39/proposal-temporal/issues/1239) |
| 61 | +- PFC: I think we should close these two given the stage that we're in. |
| 62 | +- JGT: Should the calendar canonicalize the reference year? |
| 63 | +- SFC: You only get these weird cases if you use the constructor. That's just an unfortunate corner of the API and I'm not sure there's a way around it. |
| 64 | +- JGT: How does the calendar choose a reference year given a month and day in `from()`? |
| 65 | +- SFC: First year after 1970? |
| 66 | +- JGT: Many calendars have a 60-year cycle. |
| 67 | +- SFC: Then a different standard, as long as it is consistent. It's OK if the reference year is in the future. |
| 68 | +- PFC: I think this is out of scope for Temporal. The calendar is responsible for picking the reference year. |
| 69 | +- (Some confusion about the term "canonicalize") |
| 70 | +- SFC: In the ISO calendar, if you pass a PlainDate to `PlainYearMonth.from()`, it does discard the year and picks 1972, correct? |
| 71 | +- PFC: Correct. |
| 72 | +- We agree to keep the status quo where the calendar is always responsible for choosing the reference year (except in the type's constructor where it is given exactly), and `from()` always ignores a passed-in `year` field. |
| 73 | + |
| 74 | +### Move validation of fields and options objects into calendar implementations [#1229](https://github.com/tc39/proposal-temporal/issues/1229) / [#1235](https://github.com/tc39/proposal-temporal/issues/1235) / [#1253](https://github.com/tc39/proposal-temporal/issues/1253) |
| 75 | +- PFC: I think that this does meet the bar for changes at this point, so can we agree to go ahead with these? |
| 76 | +- USA: Agreed, if you are using a non-ISO calendar then validation can mean something very different. |
| 77 | +- JGT: One question I had was that we currently access custom fields after the built-in fields, do we have to do that? |
| 78 | +- PFC: If we go ahead with this, then we no longer have to specify it except for the ISO calendar. |
| 79 | +- SFC: Very much agreed to not do pre-processing on the options bag. |
| 80 | + |
| 81 | +### Month / monthCode / monthIndex [#1203](https://github.com/tc39/proposal-temporal/issues/1203) |
| 82 | +- JGT: I have five points that we can discuss before deciding on the names. |
| 83 | + |
| 84 | +1. iCalendar string representation of months |
| 85 | + - SFC: I'm not comfortable making this decision without Manish here. The other thing not covered by this is Hindu calendar months that may be combined. |
| 86 | + - RGN: Are you saying that the spec does not accommodate this? |
| 87 | + - JGT: Correct, you can end up with a month that's month 3 and month 4 combined, and that's not accommodated by the "L" syntax. |
| 88 | + - RGN: It doesn't specify exactly how, but it does accommodate it. If you merge months 3 and 4, there are multiple ways to represent that; skipping "3" or "4", for one. It's underspecified, but not impossible to accommodate. |
| 89 | + - JGT: We can safely say that there's no specification for the Hindu calendar. |
| 90 | + - USA: The Indian national calendar is the one that's supported by ICU. |
| 91 | + - JGT: All the ICU-supported calendars are fine. |
| 92 | + - SFC: Maybe JGT, MG, and I can meet to work this out. |
| 93 | + - JGT: Sounds good. My standpoint is that the name of the month should not matter for the identifier. |
| 94 | + - SFC: I understand that point of view. The goals of having an identifier for use in formatting and having equality semantics are mutually exclusive. |
| 95 | + |
| 96 | +2. We need both a month code and a month index property |
| 97 | + - SFC: You've definitely presented use cases for a monotonically increasing month index property. I don't feel strongly about it, but I'm OK to have it. |
| 98 | + |
| 99 | +3. Either the month code or the month index should be able to be used in `from()` and `with()`. |
| 100 | + - RGN: Is the idea that they'd be passed with the same property name? Or would there be no ambiguity because they would use different property names? |
| 101 | + - JGT: The latter. |
| 102 | + - RGN: So by the naming, both would be equally "dis-privileged" or one of them privileged by virtue of having the name `"month"`? |
| 103 | + - JGT: That's the bikeshed. |
| 104 | + - PDL: What do you imagine the month code for the ISO calendar to be? |
| 105 | + - JGT: The string versions of the integers. |
| 106 | + - PDL: Can we have disambiguation depending on the type? |
| 107 | + - ??? |
| 108 | + |
| 109 | +### `eraYear` [#1231](https://github.com/tc39/proposal-temporal/issues/1231) |
| 110 | +We decide to have an overflow meeting to close out these questions. Since the editors have not reviewed the spec for the January meeting, and SFC is not comfortable proceeding without resolving these questions, we will attempt to go to Stage 3 in the March meeting. We will need to make it extra clear that the spec is ready for review. PDL, DE, PFC stress the need of not changing spec text from our side, and only accepting changes that originate from delegate reviews. |
0 commit comments