greg/mentat

Fork 0

No way to bind aggregate to variable in query #61

New issue

Open

opened 2020-08-06 16:29:08 +00:00 by gburd · 7 comments

gburd commented

2020-08-06 16:29:08 +00:00

(Migrated from github.com)

The datomic docs have an example query of the form:

[:find ?year (median ?namelen) (avg ?namelen) (stddev ?namelen)
 :with ?track
 :where [?track :track/name ?name]
        [(count ?name) ?namelen]
        [?medium :medium/tracks ?track]
        [?release :release/media ?medium]
        [?release :release/year ?year]]

Our query syntax doesn't allow the [(count ?name) ?namelen] clause.

Between this and https://github.com/mozilla/mentat/issues/647 I'm not sure we can implement a top-sites like query directly. (That might not be true, and could just a reflection of my ignorance, though!)

Original by @thomcc

The [datomic docs](https://docs.datomic.com/on-prem/query.html) have an example query of the form: ```edn [:find ?year (median ?namelen) (avg ?namelen) (stddev ?namelen) :with ?track :where [?track :track/name ?name] [(count ?name) ?namelen] [?medium :medium/tracks ?track] [?release :release/media ?medium] [?release :release/year ?year]] ``` Our query syntax doesn't allow the `[(count ?name) ?namelen]` clause. Between this and https://github.com/mozilla/mentat/issues/647 I'm not sure we can implement a top-sites like query directly. (That might not be true, and could just a reflection of my ignorance, though!) Original by @thomcc

gburd commented

2020-08-06 16:29:10 +00:00

(Migrated from github.com)

Possibly a duplicate of #750?

Original by @thomcc

Possibly a duplicate of #750? Original by @thomcc

gburd commented

2020-08-06 16:29:11 +00:00

(Migrated from github.com)

Yep, dupe of that.

Original by @rnewman

Yep, dupe of that. Original by @rnewman

thomcc commented

2020-08-06 16:33:21 +00:00

(Migrated from github.com)

Hey, it's very cool that someone's picking mentat up... But if possible could you not @ me unless you have a question or something.

Feel free if you do have a question, but all the "original by @thomcc" comments are blowing up my notifications...

Hey, it's very cool that someone's picking `mentat` up... But if possible could you not @ me unless you have a question or something. Feel free if you do have a question, but all the "original by @thomcc" comments are blowing up my notifications...

gburd commented

2020-08-06 17:35:15 +00:00

(Migrated from github.com)

Apologies for the noise.

I've been trying to migrate the various non-code assets (PRs, issues, etc.)
to a new repo (this one) and the script I used ping'ed everyone as it tried
to recreate the assets. Sorry! I've done most of the migration, which was
mostly successful, so I think that'll stop happening now. Please let
others on the team know as well.

best, and thanks for the amazing work.

-greg

On Thu, Aug 6, 2020 at 12:33 PM Thom Chiovoloni notifications@github.com
wrote:

Hey, it's very cool that someone's picking mentat up... But if possible
could you not @ me unless you have a question or something.

Feel free if you do have a question, but all the "original by @thomcc
https://github.com/thomcc" comments are blowing up my notifications...

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/qpdb/mentat/issues/61#issuecomment-670038251, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABWEHJW6Z5X6KDQ244FK3R7LLOBANCNFSM4PWYLBWQ
.

Apologies for the noise. I've been trying to migrate the various non-code assets (PRs, issues, etc.) to a new repo (this one) and the script I used ping'ed everyone as it tried to recreate the assets. Sorry! I've done most of the migration, which was mostly successful, so I think that'll stop happening now. Please let others on the team know as well. best, and thanks for the amazing work. -greg On Thu, Aug 6, 2020 at 12:33 PM Thom Chiovoloni <notifications@github.com> wrote: > Hey, it's very cool that someone's picking mentat up... But if possible > could you not @ me unless you have a question or something. > > Feel free if you do have a question, but all the "original by @thomcc > <https://github.com/thomcc>" comments are blowing up my notifications... > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <https://github.com/qpdb/mentat/issues/61#issuecomment-670038251>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/AAABWEHJW6Z5X6KDQ244FK3R7LLOBANCNFSM4PWYLBWQ> > . >

thomcc commented

2020-08-06 17:59:22 +00:00

(Migrated from github.com)

Again, feel free to ping me if it's relevant. In particular, I'm one of the maintainers of rusqlite now, and would be happy to do what I can to make your lives easier.

No worries! Again, I'm thrilled someone's picking this up. Mentat's a really cool piece of tech, and it was sad to see it be languish just because it didn't fit with our needs in time for a fairly strict deadline. Again, feel free to ping me if it's relevant. In particular, I'm one of the maintainers of rusqlite now, and would be happy to do what I can to make your lives easier.

gburd commented

2020-08-06 18:40:05 +00:00

(Migrated from github.com)

Let's connect via email, mine is 4 letters of first name at second name dot
me. I've got some questions about rustqlite.

-greg

On Thu, Aug 6, 2020 at 1:59 PM Thom Chiovoloni notifications@github.com
wrote:

No worries! Again, I'm thrilled someone's picking this up. Mentat's a
really cool piece of tech, and it was sad to see it be languish just
because it didn't fit with our needs in time for a fairly strict deadline.

Again, feel free to ping me if it's relevant. In particular, I'm one of
the maintainers of rusqlite now, and would be happy to do what I can to
make your lives easier.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/qpdb/mentat/issues/61#issuecomment-670086433, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABWEGCSDUCQ4SMPD6O653R7LVQRANCNFSM4PWYLBWQ
.

Let's connect via email, mine is 4 letters of first name at second name dot me. I've got some questions about rustqlite. -greg On Thu, Aug 6, 2020 at 1:59 PM Thom Chiovoloni <notifications@github.com> wrote: > No worries! Again, I'm thrilled someone's picking this up. Mentat's a > really cool piece of tech, and it was sad to see it be languish just > because it didn't fit with our needs in time for a fairly strict deadline. > > Again, feel free to ping me if it's relevant. In particular, I'm one of > the maintainers of rusqlite now, and would be happy to do what I can to > make your lives easier. > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <https://github.com/qpdb/mentat/issues/61#issuecomment-670086433>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/AAABWEGCSDUCQ4SMPD6O653R7LVQRANCNFSM4PWYLBWQ> > . >

thomcc commented

2020-08-06 19:06:19 +00:00

(Migrated from github.com)

I sent you an email (I'm chiovolonit@gmail), but might have gotten the address wrong. I'm just getting off my lunch break (and probably wont respond for a few hours, but it's exciting to see this project come back to life.