These are chat archives for inveniosoftware/invenio

8th
Mar 2016
Tibor Simko
@tiborsimko
Mar 08 2016 04:06
@eamonnmag Cool! @jacquerie I also think this may be better addressed server-side, people using other browsers... Anyhow, the future http://inveniosoftware.org/ project page will get some dynamic dashboards of this kind.
Eamonn Maguire
@eamonnmag
Mar 08 2016 07:53
The point of using the chrome extension is that I can dynamically insert content. You can't do this any other way, so it's the only option for now. It may encourage github to offer that availability in the future though.
I've just made an update so it can add tests to badges to only show them when the language is a certain kind etc.
Eamonn Maguire
@eamonnmag
Mar 08 2016 08:39
Might be of interest to some of you. A nice overview of ML techniques and a cheat sheet on when to use them :) https://azure.microsoft.com/en-us/documentation/articles/machine-learning-algorithm-choice/
Johnny Mariéthoz
@jma
Mar 08 2016 10:40
Hello with the master version of dojson I got order = output['__order__'] KeyError: '__order__' error which is not the case with the 1.0.1 version. Any idea?
Jiri Kuncar
@jirikuncar
Mar 08 2016 10:46
@jma it's for keeping the order of fields. You can ignore this error for now.
Btw can you send us more detailed traceback so we can see which rule has failed.
Johnny Mariéthoz
@jma
Mar 08 2016 12:15
I had to copy the order function in my model to be able to convert using my own rules. But it works.
Jiri Kuncar
@jirikuncar
Mar 08 2016 12:16
@jma please check inveniosoftware/dojson#115
Johnny Mariéthoz
@jma
Mar 08 2016 12:25
@jirikuncar thanks a lot!
Jiri Kuncar
@jirikuncar
Mar 08 2016 12:25
@jma thanks to @SamiHiltunen ;)
Chokri Ben Romdhane
@chokribr
Mar 08 2016 13:53
please @kaplun @tiborsimko i need your opinions about to this issue inveniosoftware/invenio#3600 and inveniosoftware/invenio#3605
Ludmila Marian
@ludmilamarian
Mar 08 2016 13:55
@chokribr a quick fix can be: record_owners = print_record(int(sysno), 'tm', \
[CFG_WEBSUBMIT_RECORD_OWNER_EMAIL]).strip()
so int(sysno) instead of sysno
the fix would be in the Send_APP_Mail.py
Chokri Ben Romdhane
@chokribr
Mar 08 2016 14:32
so simple only a cast operation many thanks @ludmilamarian
i wonder if in our case we need to test wether the user have the right to edit or not the record?
Ludmila Marian
@ludmilamarian
Mar 08 2016 15:27
@chokribr the test is there in order for the print_record() function to know if to display or not to the user a link 'edit record'
actually this Send_APP_Mail function doesn't really need to call the print_record (this is some very old code). It can just as simply run get_fieldvalues(sysno, CFG_WEBSUBMIT_RECORD_OWNER_EMAIL) to retrieve the same info
Chokri Ben Romdhane
@chokribr
Mar 08 2016 15:30
totaly agree many thanks @ludmilamarian
Chokri Ben Romdhane
@chokribr
Mar 08 2016 15:36
This message was deleted
Tibor Simko
@tiborsimko
Mar 08 2016 16:59
@chokribr @ludmilamarian If the fix works, can you please send a PR?