From: Axel Kielhorn <org-mode@axelkielhorn.de>
To: Org-Mode Mailing List <emacs-orgmode@gnu.org>
Subject: Re: Updating column view dynamic block does not work with {est+}
Date: Thu, 20 May 2021 21:10:42 +0200 [thread overview]
Message-ID: <07BFE527-63BA-4B2E-8891-D668F8C4D93F@axelkielhorn.de> (raw)
In-Reply-To: <877djt1e4b.fsf@nicolasgoaziou.fr>
> Am 20.05.2021 um 19:58 schrieb Nicolas Goaziou <mail@nicolasgoaziou.fr>:
>
> Org Duration is strict about what it is fed with (which is good). Effort
> property expects a duration as value. But "3-8" is not a valid duration.
> However, "3" is a valid duration; it means 3 minutes.
The problem is that effort can either be a duration and in that case the strict duration library ist fine.
Or it can be a range (of days).
3:30 is fine when : is used to add the times
3.0 - 4.0 is a range estimate when est+ is used
3:00 - 4:00 is only correct by chance, 3:30 - 4:30 will lead to the same result since est+ does not handle durations.
Splitting it into 2 properties (effort and effort_range) is even worse since it will be inconsistent after a few edits.
Some functions only expect 1 value for the effort, which is the correct value?
What happens when I : add a range?
The dynamic block code juggles with conversion to minutes, which is wrong when a range is involved.
It gets even more interesting when on task has 3-4 (implicit) days, while another has 8:00 (implicit) hours.
(Are 8 h one work day, or are 24 h one calendar day?)
> Maybe Effort property should simply accept a duration or a duration
> range.
That’s what I first thought it would do, since a duration is a time (8:00 for 8h).
The question is how to resolve ambiguity?
1.0 is one day
1:00 is one hour
1 is one minute, really? yes, that is the default for the duration library. But it used to mean one day???
Maybe a new est: function to work with durations and the old est+ function to work with numbers
(which could mean days, but it could mean ms as well)? And a warning about inconsistent units.
What happens when I use a range in a clock table?
The more I think about that, the more questions I have.
To be honest, I’m not even sure I will use the est+ function in real life.
(I discovered it when preparing some training material and thought that was interesting.)
I’ll probably just : add the hours and add 10% safety margin on top.
But since the feature is advertised it would be great if it works.
Would I be sorry if you deprecate this feature? Probably not.
Will anyone notice? Probably not, it has been broken for 4 years.
Greetings
Axel
next prev parent reply other threads:[~2021-05-20 19:11 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-16 5:51 Updating column view dynamic block does not work with {est+} Axel Kielhorn
2021-05-20 17:12 ` Axel Kielhorn
2021-05-20 17:58 ` Nicolas Goaziou
2021-05-20 19:10 ` Axel Kielhorn [this message]
2021-05-20 20:15 ` Nicolas Goaziou
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.orgmode.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=07BFE527-63BA-4B2E-8891-D668F8C4D93F@axelkielhorn.de \
--to=org-mode@axelkielhorn.de \
--cc=emacs-orgmode@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).