From: Carsten Dominik <carsten.dominik@gmail.com>
To: Mikael Fornius <mfo@abc.se>
Cc: emacs-orgmode@gnu.org
Subject: Re: [PATCH] Add min/max/mean age operators to column view.
Date: Sun, 1 Nov 2009 18:24:41 +0100 [thread overview]
Message-ID: <18B989B4-CC11-4597-BF16-8A0A2DFFD5C4@gmail.com> (raw)
In-Reply-To: <87k4ybel4q.fsf@eee.in>
Hi Mikael, thanks for doing the testing!
James, do you want to integrate Mikaels changes? If yes, can you
please do so
and let me know when and from where I can pull?
Also, if one of you would like to go through the trouble to adapt the
XEmacs
version, that would be very nice.
Thanks!
- Carsten
P.S. What needs to be added to the manual?
On Oct 31, 2009, at 2:41 PM, Mikael Fornius wrote:
>
> Thanks for a very good explanation, now I understand and I have tested
> it successfully. Looks like it works perfectly well to me.
>
> The interactive function org-columns-new also works as expected.
>
> This is a very good addition and I think it can be useful in many
> different areas. It also opens for more advanced calculations in
> colview
> mode due to the extension of `org-columns-compile-map'.
>
> Personally I prefer using colview to tables for temporary views of
> dynamic and alive data (like my running exercise diary) and want to
> use
> it more then I do now.
>
> I have modified the patch with one missing parenthesis and a more
> extensive documentation string to the important variable
> `org-columns-format-compile-map' which I wrote most for my own
> recapitulation of the subject and understanding your patch.
>
> Also org-columns-new had a reference to free variable `funcs' and I
> changed it to `fun' (member of functions argument list) which I
> believe
> is the right one?, you have to check me on this.
>
> Two other references, probably older, to free variables I have also
> tried to fix by let binding them.
>
> Feel free to use it if you want. (Attached patch to this messages.
> Difference is against your misc-new-features branch pulled today.)
>
>
>
> As an ending parenthesis commenting column view in general:
>
> I see three small problems with org-colview that I (or someone else
> interested) should fix some day:
>
> 1. A annoying problem is not being able to have multiple columns
> summarizing the same property but with different operators.
>
> #+COLUMNS: %TIMESTAMP_IA{@min} %TIMESTAMP_IA{@max}
>
> This will result only in min calculations for both columns.
>
> I remember that I have looked at this before and think it is non
> trivial to fix because of the internal way data is collected and
> calculated. Almost a rewrite of this part is necessary.
>
> 2. Being able to use Emacs calc HMS forms
>
> (info "(Calc) HMS forms")
>
> for calculating and presenting time as I can to in the org-mode
> spreadsheet. I use this time format to manipulate my exercise data
> now days. Maybe this be added easily with the new `calc' slot in the
> `org-columns-compile-map' structure.
>
> 3. More intelligent handling of property values like automatic
> detection
> of times, dates or numbers to simplify the definitions and have only
> one 'max' summary option instead of now three (max, :max, @max).
>
>
> Cheers!
>
> <diff>
> (And don't forget to patch the bastard colview-xemacs ;-), could be
> made
> effectively with a ediff manual merging I think.)
>
> --
> Mikael Fornius
>
- Carsten
next prev parent reply other threads:[~2009-11-01 17:24 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-26 0:04 [PATCH] Add min/max/mean age operators to column view James TD Smith
2009-10-28 17:01 ` Carsten Dominik
2009-10-28 19:36 ` James TD Smith
2009-10-29 11:12 ` Mikael Fornius
2009-10-29 13:17 ` Mikael Fornius
2009-10-29 20:10 ` Mikael Fornius
2009-10-30 3:36 ` James TD Smith
2009-10-29 21:15 ` James TD Smith
2009-10-31 13:41 ` Mikael Fornius
2009-11-01 17:24 ` Carsten Dominik [this message]
2009-11-01 18:59 ` James TD Smith
2009-12-30 11:32 ` Manish
2009-12-30 14:01 ` James TD Smith
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=18B989B4-CC11-4597-BF16-8A0A2DFFD5C4@gmail.com \
--to=carsten.dominik@gmail.com \
--cc=emacs-orgmode@gnu.org \
--cc=mfo@abc.se \
/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).