Re: ssn and github

Hi Maxime,

I have taken notice of these changes. They are small enough to be discussed quickly in our extended phone conference next week.

Cheers,
Armin

From: Maxime Lefrançois <maxime.lefrancois@emse.fr>
Date: Tuesday, 21 February 2017 at 7:57 am
To: "janowicz@ucsb.edu" <janowicz@ucsb.edu>, Kerry Taylor <kerry.taylor@anu.edu.au>, "Simon.Cox@csiro.au" <Simon.Cox@csiro.au>, "danh.lephuoc@tu-berlin.de" <danh.lephuoc@tu-berlin.de>, Armin Haller <armin.haller@anu.edu.au>
Cc: "public-sdw-wg@w3.org" <public-sdw-wg@w3.org>
Subject: Re: ssn and github


Agreed. These are small changes (just some metadata), so we will have to work on them anyways some day.
I'll do what you suggest for bigger modifications.
Kind regards,
Maxime

Le lun. 20 févr. 2017 20:26, Krzysztof Janowicz <janowicz@ucsb.edu<mailto:janowicz@ucsb.edu>> a écrit :
Hi Maxime,



I guess you are referring to the pull requests I created today.

Not only but also :-).



 - Each of these PR mention: "Before merging this pull request, please discuss options in the Wiki <some link to the wiki>"

Yes, but then we should create the changes in the code and issue a pull request after the discussion and not before, right?



 - I purposely  created one PR = one wiki page, hence we can discuss, vote, modify the related PR branch accordingly, and merge only the changes that have been voted for.

I see. This is why I said that I will ignore those requests for now. I agree that it makes sense to issue small pull requests and do minor updates so that we have something to show during our telcos (in fact, I did so one time as well),  but for larger and more substantial changes, I would propose to go step-by-step.

Cheers,
Krzysztof



On 02/20/2017 11:09 AM, Maxime Lefrançois wrote:
Hi Krysztof, all,

Krysztof, you wrote:
> Yes, but there are many, many new pull requests for features that have not yet
> been discussed in the group, so I will only look at and accept those that reflect
> consensus within the group.

I guess you are referring to the pull requests I created today.

 - Each of these PR mention: "Before merging this pull request, please discuss options in the Wiki <some link to the wiki>"

 - I purposely  created one PR = one wiki page, hence we can discuss, vote, modify the related PR branch accordingly, and merge only the changes that have been voted for.

 - the base branch for each of these PR is "maxime-methodology", which is still not merged with "gh-pages" (see https://github.com/w3c/sdw/pull/554). Merging any of these should then be safe anyways.

Kind regards,
Maxime


Best,
Jano




On 02/19/2017 10:33 PM, Kerry Taylor wrote:
Ok, thks. My PR yesterday  does the ssn.ttl /ssn.owl part  of the fix   https://github.com/w3c/sdw/pull/560 – I will do the rest when I get a chance.

Krysztof, can you please review and merge?

-Kerry



From: Simon.Cox@csiro.au<mailto:Simon.Cox@csiro.au> [mailto:Simon.Cox@csiro.au]
Sent: Monday, 20 February 2017 12:15 PM
To: Kerry Taylor <kerry.taylor@anu.edu.au><mailto:kerry.taylor@anu.edu.au>; janowicz@ucsb.edu<mailto:janowicz@ucsb.edu>; danh.lephuoc@tu-berlin.de<mailto:danh.lephuoc@tu-berlin.de>; Armin Haller <armin.haller@anu.edu.au><mailto:armin.haller@anu.edu.au>
Cc: public-sdw-wg@w3.org<mailto:public-sdw-wg@w3.org>
Subject: RE: ssn and github

Hi Kerry –

We had some issues integrating a pull-request relating to O&M alignment. It was originally issued before the re-name, and we were trying to retain the commit history on the branch I had been working on. The ttl had gotten re-ordered along the way due to me having opened it in TopBraid. We tried to reverse that, but clearly failed.

Would be very helpful if you could reset the ssn artefacts.

Simon



From: Kerry Taylor [mailto:kerry.taylor@anu.edu.au]
Sent: Sunday, 19 February, 2017 15:02
To: janowicz@ucsb.edu<mailto:janowicz@ucsb.edu>; danh.lephuoc@tu-berlin.de<mailto:danh.lephuoc@tu-berlin.de>; Armin Haller <armin.haller@anu.edu.au<mailto:armin.haller@anu.edu.au>>
Cc: public-sdw-wg@w3.org<mailto:public-sdw-wg@w3.org>
Subject: ssn and github

Github appears to have yet another error on it around ssn:

I can see that there is a new file created here https://github.com/w3c/sdw/commit/da109c5b1a8758582caf18c8c797887387123aee#diff-95f609f3731bdeb74f41d82be5f0fadc called ssn.owl.

1) didn’t every one want “ssn.ttl” to be  used instead of “ssn.owl”  (.owl suffix)? Or  have  I misunderstood?

2)  It is out of date--- it appears to be an old copy that does not capture changes to ssn.ttl  in the past few weeks and is inconsistent with ssn.ttl

3) I strongly suggest we keep only one of them on git – either ssn.owl or ssn.ttl but not  both for fear of getting out of sync (as they are right now)


I will remove ssn.owl in my next PR – I hope that is ok. Please let me know ASAP if that should not be done.

There is also something  not right going on with dul-alignment.owl – but I am going to leave that alone for the principle of “first, do no harm.”

--Kerry


Dr Kerry Taylor
Associate Professor (Data Science)
Research School of Computer Science
ANU College of Engineering and Computer Science
Canberra ACT 2601 Australia
+61 2 6125 8560





--

Krzysztof Janowicz



Geography Department, University of California, Santa Barbara

4830 Ellison Hall, Santa Barbara, CA 93106-4060



Email: jano@geog.ucsb.edu<mailto:jano@geog.ucsb.edu>

Webpage: http://geog.ucsb.edu/~jano/<http://geog.ucsb.edu/%7Ejano/>

Semantic Web Journal: http://www.semantic-web-journal.net





--

Krzysztof Janowicz



Geography Department, University of California, Santa Barbara

4830 Ellison Hall, Santa Barbara, CA 93106-4060



Email: jano@geog.ucsb.edu<mailto:jano@geog.ucsb.edu>

Webpage: http://geog.ucsb.edu/~jano/


Semantic Web Journal: http://www.semantic-web-journal.net