Time
S
Nick
Message
06:28 poikilotherm joined #dataverse
08:42 poikilotherm joined #dataverse
10:10 pdurbin
poikilotherm: thanks again for all the pull requests!
10:11 poikilotherm
Morning pdurbin :-)
10:14 pdurbin
Morning. I'm going to be busy all day at the second all day event for The Open Source Software Health Index Project: https://projects.iq.harvard.edu/osshealthindex
10:14 pdurbin
But it doesn't start for a few hours, so if anyone needs anything, please let me know.
10:15 poikilotherm
I'm working on tests for #5991
10:15 poikilotherm
We need to take off with this
10:16 pdurbin
Oh! That reminds me.
10:18 pdurbin
poikilotherm: if you click Log In at https://data.cimmyt.org what do you notice under "Other Options"? A new option! Microsoft! Please see https://github.com/IQSS/dataverse/issues/6155 (and I can show you their fork).
10:18 poikilotherm
Yeah, I read that
10:18 poikilotherm
From a technical perspective, this is not a big deal
10:18 poikilotherm
ScribeJava supports Microsoft and many many others
10:21 poikilotherm
I want more than just another provider... ;-)
10:24 pdurbin
Sure, of course. I just thought you might be excited that we're going to put the auth code through QA again. I thought maybe you could code review Alejandra's pull request (once it's made). And we should think about how to make the docs better. At the very least "OAuth Login: ORCID, GitHub, Google" in the sidebar is getting a little long. Maybe we just shorten it to OAuth. :)
10:30 poikilotherm
Ah pdurbin speaking about docs:
10:30 poikilotherm
I wonder if you guys would be ok with adding PlantUML support in Sphin
10:30 poikilotherm
Sphinx
10:30 poikilotherm
I thought about adding a sequence diagram for the OAuth flow
10:33 pdurbin
I'm already using PlantUML here, for example: http://guides.dataverse.org/en/4.16/admin/make-data-count.html#architecture
10:33 poikilotherm
Yeah
10:33 poikilotherm
I meant using PlantUML as a plugin
10:33 poikilotherm
"extension"
10:33 pdurbin
Oh. Sounds fine. Separate pull request, please.
10:33 poikilotherm
Obviously
10:34 pdurbin
:)
10:36 poikilotherm
Would you support adding architecture docs in the dev guide?
10:37 pdurbin
poikilotherm: hmm, do you mean moving stuff like https://github.com/IQSS/dataverse/blob/v4.16/doc/Architecture/auth.md (plantuml diagrams at the bottom) to the dev guide? Yes!
10:37 poikilotherm
That would be a start
10:38 poikilotherm
Would you move this to dev guide or create a separate arch guide?
10:38 pdurbin
Dev guide but I think the dev guide needs more hierarchy. Folders.
10:38 poikilotherm
Yeah
10:38 poikilotherm
Definitly
10:38 poikilotherm
Sublevels
10:39 pdurbin
Yeah, here's an example of sublevels: http://guides.dataverse.org/en/4.16/user/data-exploration/index.html
10:39 pdurbin
or closer to home (in the dev guide itself): http://guides.dataverse.org/en/4.16/developers/unf/index.html
10:39 poikilotherm
:-)
11:02 pdurbin
poikilotherm: you know about the shib and oath page in the dev guide, right? It could use some love (and diagrams): http://guides.dataverse.org/en/4.16/developers/remote-users.html
11:54 donsizemore joined #dataverse
12:27 pdurbin
donsizemore: morning. I'm in a workshop all day for https://projects.iq.harvard.edu/osshealthindex . Should be fun but please don't expect much response from me today. Half an hour before I head down if you need anything.
12:28 donsizemore
@pdurbin we're good so far as i know?
12:28 pdurbin
xarthisius: thanks for resolving merge conflicts in https://github.com/jupyter/repo2docker/pull/739 !! I just left a comment in that issue about WIP vs MRG.
12:28 donsizemore
@pdurbin i'm hoping for a quiet day and may even get to poke at jenkins.dataverse jobs some today
12:28 pdurbin
donsizemore: if you're good, I'm good. :)
12:28 donsizemore
@pdurbin i'd be better with an afternoon happy hour
12:29 pdurbin
heh
12:29 pdurbin
You're reminding me of the "coffee breaks" at Dataverse Community Meetings: https://twitter.com/philipdurbin/status/1142148725525090307 :)
12:30 poikilotherm
Cheers!
12:30 pdurbin
poikilotherm: I see you hacking away on your auth branch. Great!
12:30 poikilotherm
On my way to get things testable
12:31 poikilotherm
Injecting FacesContext
12:31 pdurbin
Perfect. It's testing week here at IQSS. 3 weeks actually. The focus of the current sprint.
12:31 poikilotherm
He.
12:31 poikilotherm
Things would be better not being stuck on old platforms...
12:32 poikilotherm
But who am I telling this... ;-)
12:32 pdurbin
How stuck are we? What if we write a new UI on top of APIs? Does the API test suite pass with Dataverse running on Payara?
12:33 poikilotherm
I had this tought a few times... But this is a frick'n HUGE thing to do
12:33 pdurbin
Well, we'd start with the UI that provides the most value, right?
12:33 poikilotherm
thought
12:34 poikilotherm
Dunno. Maybe. Sounds like a 5.x thing
12:34 poikilotherm
It might be a very good idea to actually decouple the repo and the UI
12:35 pdurbin
Yes! That's my point exactly.
12:35 pdurbin
dataverse vs dataverse-ui
12:36 pdurbin
You could even install them on separate servers.
12:36 poikilotherm
Buhuhuhuhuhu
12:36 poikilotherm
T_T
12:36 poikilotherm
Oh how I hate this
12:37 poikilotherm
Could you guys please start using migrations properly? I know there is an issue for this already, but hey - it would be so cool to use Flyway as designed
12:37 poikilotherm
ERROR: column "id" of relation "setting" already exists
12:37 poikilotherm
And more like this
12:37 poikilotherm
We should really STOP using JPA to generate the tables
12:37 poikilotherm
It will always break Flyway doing its job
12:37 pdurbin
poikilotherm: I just requested your review of https://github.com/IQSS/dataverse/pull/6166
12:43 poikilotherm
Just left a very sad comment and review
12:47 pdurbin
lots of tears, I see
12:47 pdurbin
a vale of tears
12:49 poikilotherm
Do you want me to cherry-pick and test your PR?
12:52 pdurbin
Well, one way you could test it is by spinning up https://github.com/IQSS/dataverse/tree/5028-dataset-explore-btn because I already merged that branch/commit into that external tools branch. And you could play with external tools :)
12:53 pdurbin
But you don't support arbitrary branches, do you? That's what https://github.com/IQSS/dataverse-kubernetes/issues/64 is about.
12:53 poikilotherm
I do have some local solution for this which I am testing
12:53 poikilotherm
If I feel this is usable for others, I will push it
12:54 poikilotherm
Obviously I'll need some time writing docs on how to use it
12:54 pdurbin
Oh! Great!
12:54 poikilotherm
Currently using skaffold
12:54 poikilotherm
In combination with kind
12:55 poikilotherm
Lots of stuff to write down
12:55 poikilotherm
And kind of far away from cloud native ;-)
12:55 poikilotherm
Ok, it deployed successfully now
12:56 poikilotherm
Maybe this is a good time to write down things...
12:57 pdurbin
You're using my branch? Yes! Please write it down! :)
12:57 poikilotherm
Not using your branch. Merged it into my #5991 branch and deployed that ;-)
13:02 pdurbin
ok
13:03 poikilotherm
Hmm that sounds I killed your enthusiasm
13:23 pdurbin
poikilotherm: YOU CANNOT KILL MY ENTHUSIASM! :)
13:24 poikilotherm
Phew...
13:35 pdurbin
many have tried ;)
13:36 poikilotherm
When you get bored in your meeting you might risk playing with https://github.com/IQSS/dataverse-kubernetes/tree/64-use-develop-branch later when I push the interesting things ;-)
14:20 xarthisius
pdurbin: what's wrong with the following query: https://dataverse.harvard.edu/api/search?q=filePersistentId:%22doi:10.7910/DVN/6ZXAGT/3YRRYJ%22
14:20 xarthisius
I can't figure out the proper q syntax
14:23 xarthisius
filePersistentId is not a search keyword? How do I search for file knowing it's persistent_id
14:50 donsizemore
@xarthisius do you just want https://dataverse.harvard.edu/api/search?q=persistentId: "doi:10.7910/DVN/6ZXAGT/3YRRYJ"
14:59 xarthisius
donsizemore: {"status":"OK","data":{"q":"persistentId:\"doi:10.7910/DVN/6ZXAGT/3YRRYJ\"","total_count":54
14:59 xarthisius
nope, I want something that gives total_count: 1 :)
15:17 poikilotherm
pdurbin: I didn't get it all in, but need to quit for today. Skaffold usage and kind/k3s is still missing.
15:17 poikilotherm
But you can already build docker images from branches. Docs included :-D
15:18 poikilotherm
https://github.com/IQSS/dataverse-kubernetes/blob/64-use-develop-branch/docs/images.md
15:18 poikilotherm
Have a nice weekend everyone
20:53 icarito[m] joined #dataverse
21:07 pdurbin
xarthisius: I'm pretty sure the file you're searching for isn't indexed. This one works: https://dataverse.harvard.edu/api/search?q=filePersistentId:%22doi:10.7910/DVN/TJCLKP/3VSTKY%22 (for me at least)
21:08 pdurbin
But I gotta run. Let's talk next week. Have a great weekend, all!
21:08 pdurbin left #dataverse
21:42 rigelk joined #dataverse
