abundance_per_year_compare() have 1 week offset

[ScientiaFelis]

There seem to be a week offset in the data or presentation of weeks in figure.
See issue #23 for figures

[ScientiaFelis]

This could be because excel and R count weeknr and start of week differently.

• check what week 2022-01-02 or 2022-04-10 is (sunday) in week() and isoweek()
• check how weeks count in R, week() and isoweek()

[ScientiaFelis]

isoweek() starts the week on Monday and follow ISO8601 standard.

week just count the first seven days from 1 of januari as the first week and so on.

@larspett should we change the week setting from isoweek()?

[larspett]

what do we currently use? isoweek? What does that give for April 1 2022 and September 30 2022? 13 & 39? Same for same dates in 2011?

[ScientiaFelis]

We use isoweek

2022-04-01 = W 13
2022-09-30 = W 39

2011-04-01 = W 13
2011-09-30 = W 39

[larspett]

Let’s keep that then 1 nov. 2023 kl. 19:30 skrev Georg ***@***.***>:  We use isoweek 2022-04-01 = W 13 2022-09-30 = W 39 2011-04-01 = W 13 2011-09-30 = W 39 — Reply to this email directly, view it on GitHub<#27 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AEVQXZGGRIP3MILZWATFAA3YCKIKVAVCNFSM6AAAAAA6YANKBSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBZGQ2TCMJSHE>. You are receiving this because you were assigned.Message ID: ***@***.***>

[larspett]

looks fine