emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
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

  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).