You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@age.apache.org by Josh Innis <jo...@gmail.com> on 2022/01/03 21:08:14 UTC
Re: [VOTE] Release Apache AGE Viewer (incubating) 1.0.0-rc0
Hi Felix,
I believe that dependencies that are not bundled in the release are not
required. Which would be all the dependencies listed in the two
package.json files.
https://infra.apache.org/licensing-howto.html#bundled-vs-non-bundled
For agensGraphRepository.js, the project AgensGraph is distributed under
the Apache 2.0 license
https://github.com/bitnine-oss/agensgraph/blob/master/LICENSE. No bundled
bits are included in the release. A scan of the United Statement Patent and
Trademark database yields no relevant results.
We should remove the reference to bitnine/agviewer and either remove the
section or construct a docker image under the apache name.
On Thu, Dec 30, 2021 at 7:29 PM Felix Cheung <fe...@hotmail.com>
wrote:
> lots of dependencies - are these licenses reviewed and handled in the
> LICENSE file?
>
> https://github.com/apache/incubator-age-viewer/blob/main/backend/package.json
>
> https://github.com/apache/incubator-age-viewer/blob/main/frontend/package.jso\
>
> this requires a component - again could be a LICENSE check, if it is
> required for execution (at runtime) we should check license of that too.
> Also what is the trademark on the “AgensGraph” name?
>
> https://github.com/apache/incubator-age-viewer/blob/main/backend/src/models/agensgraph/agensGraphRepository.js
> '@bitnine-oss/ag-driver');
>
> The
> https://github.com/apache/incubator-age-viewer/blob/main/README.md
> references to bitnine/agviewer
> ________________________________
> From: Eya Badal <ey...@apache.org>
> Sent: Thursday, December 30, 2021 6:44:11 AM
> To: dev@age.apache.org <de...@age.apache.org>
> Subject: Re: [VOTE] Release Apache AGE Viewer (incubating) 1.0.0-rc0
>
> Looks good to me
> +1
>
> I checked the following:
>
> - Signature and hash - check.
> - DISCLAIMER - check.
> - LICENSE and NOTICE - check.
> - No unexpected binary files - check.
> - incubating in name - check.
>
> Best,
> Eya
>
> On 2021/12/23 00:58:26 Alex Kwak wrote:
> > Dear Apache AGE Community,
> >
> > This is an official vote for the Apache AGE Viewer release v1.0.0-rc0
> that we have been working toward it.
> >
> > To learn more about Apache AGE, please see http://age.apache.org/
> >
> > Functionalities included and addressed in this release:
> > - Graph visualization for AGE.
> > - Extends edge and vertex point by point.
> >
> >
> > The vote is open now and until December 25th at 5:00 PM PST and passes
> if a majority +1 votes are cast, with a minimum of 3 +1 votes.
> >
> > [ ] +1 Release this package as Apache AGE 0.6.0
> > [ ] 0 I won't get in the way
> > [ ] -1 Do not release this package because ...
> >
> >
> > The git tag to be discussed and voted upon
> > https://github.com/apache/incubator-age-viewer/releases/tag/v1.0.0
> >
> > The git commit hash:
> > commit 2c55b2302875449565674aa039517734cf338031
> >
> > The release files, including signatures, digests, etc. can be found at:
> >
> https://dist.apache.org/repos/dist/dev/incubator/age/viewer/apache-age-viewer-1.0.0-incubating-rc0/
> >
> > The SHA512 Checksum for these artifacts is:
> >
> 8fa45254bba973678243a78321869aaf88c746b55168d4401931c5d6e87192c80d581d97e927c8b87098aaacba8d6f77bf1ca225e864c49117f5cb1887c699d0
> >
> > Release artifacts are signed with the following key:
> > https://downloads.apache.org/incubator/age/KEYS
> >
> > The fingerprint of key to sign release artifacts:
> > 0E7F 408D 8C6A 1952 329C B379 D471 FDCE 5F5C 5B82
> >
> > For more information about the contents of this release, see:
> > https://github.com/apache/incubator-age-viewer/releases/tag/v1.0.0
> >
> >
> >
> > Best regards,
> > Alex Kwak
> >
>
Re: [VOTE] Release Apache AGE Viewer (incubating) 1.0.0-rc0
Posted by Josh Innis <jo...@apache.org>.
Correction: AgensGraph is trademarked.
https://patft.uspto.gov/netacgi/nph-Parser?Sect1=PTO2&Sect2=HITOFF&p=1&u=%2Fnetahtml%2FPTO%2Fsearch-bool.html&r=1&f=G&l=50&co1=AND&d=PTXT&s1=AgensGraph&OS=AgensGraph&RS=AgensGraph
On 2022/01/03 21:08:14 Josh Innis wrote:
> Hi Felix,
>
> I believe that dependencies that are not bundled in the release are not
> required. Which would be all the dependencies listed in the two
> package.json files.
>
> https://infra.apache.org/licensing-howto.html#bundled-vs-non-bundled
>
> For agensGraphRepository.js, the project AgensGraph is distributed under
> the Apache 2.0 license
> https://github.com/bitnine-oss/agensgraph/blob/master/LICENSE. No bundled
> bits are included in the release. A scan of the United Statement Patent and
> Trademark database yields no relevant results.
>
> We should remove the reference to bitnine/agviewer and either remove the
> section or construct a docker image under the apache name.
>
>
>
>
> On Thu, Dec 30, 2021 at 7:29 PM Felix Cheung <fe...@hotmail.com>
> wrote:
>
> > lots of dependencies - are these licenses reviewed and handled in the
> > LICENSE file?
> >
> > https://github.com/apache/incubator-age-viewer/blob/main/backend/package.json
> >
> > https://github.com/apache/incubator-age-viewer/blob/main/frontend/package.jso\
> >
> > this requires a component - again could be a LICENSE check, if it is
> > required for execution (at runtime) we should check license of that too.
> > Also what is the trademark on the “AgensGraph” name?
> >
> > https://github.com/apache/incubator-age-viewer/blob/main/backend/src/models/agensgraph/agensGraphRepository.js
> > '@bitnine-oss/ag-driver');
> >
> > The
> > https://github.com/apache/incubator-age-viewer/blob/main/README.md
> > references to bitnine/agviewer
> > ________________________________
> > From: Eya Badal <ey...@apache.org>
> > Sent: Thursday, December 30, 2021 6:44:11 AM
> > To: dev@age.apache.org <de...@age.apache.org>
> > Subject: Re: [VOTE] Release Apache AGE Viewer (incubating) 1.0.0-rc0
> >
> > Looks good to me
> > +1
> >
> > I checked the following:
> >
> > - Signature and hash - check.
> > - DISCLAIMER - check.
> > - LICENSE and NOTICE - check.
> > - No unexpected binary files - check.
> > - incubating in name - check.
> >
> > Best,
> > Eya
> >
> > On 2021/12/23 00:58:26 Alex Kwak wrote:
> > > Dear Apache AGE Community,
> > >
> > > This is an official vote for the Apache AGE Viewer release v1.0.0-rc0
> > that we have been working toward it.
> > >
> > > To learn more about Apache AGE, please see http://age.apache.org/
> > >
> > > Functionalities included and addressed in this release:
> > > - Graph visualization for AGE.
> > > - Extends edge and vertex point by point.
> > >
> > >
> > > The vote is open now and until December 25th at 5:00 PM PST and passes
> > if a majority +1 votes are cast, with a minimum of 3 +1 votes.
> > >
> > > [ ] +1 Release this package as Apache AGE 0.6.0
> > > [ ] 0 I won't get in the way
> > > [ ] -1 Do not release this package because ...
> > >
> > >
> > > The git tag to be discussed and voted upon
> > > https://github.com/apache/incubator-age-viewer/releases/tag/v1.0.0
> > >
> > > The git commit hash:
> > > commit 2c55b2302875449565674aa039517734cf338031
> > >
> > > The release files, including signatures, digests, etc. can be found at:
> > >
> > https://dist.apache.org/repos/dist/dev/incubator/age/viewer/apache-age-viewer-1.0.0-incubating-rc0/
> > >
> > > The SHA512 Checksum for these artifacts is:
> > >
> > 8fa45254bba973678243a78321869aaf88c746b55168d4401931c5d6e87192c80d581d97e927c8b87098aaacba8d6f77bf1ca225e864c49117f5cb1887c699d0
> > >
> > > Release artifacts are signed with the following key:
> > > https://downloads.apache.org/incubator/age/KEYS
> > >
> > > The fingerprint of key to sign release artifacts:
> > > 0E7F 408D 8C6A 1952 329C B379 D471 FDCE 5F5C 5B82
> > >
> > > For more information about the contents of this release, see:
> > > https://github.com/apache/incubator-age-viewer/releases/tag/v1.0.0
> > >
> > >
> > >
> > > Best regards,
> > > Alex Kwak
> > >
> >
>
Re: [VOTE] Release Apache AGE Viewer (incubating) 1.0.0-rc0
Posted by Josh Innis <jo...@gmail.com>.
The northwind.jsx is an example on how to use AGE Viewer with AgensGraph.
Currently, AGE Viewer is compatible with both AgensGraph and Apache AGE, so
the file does not involve AGE, but it is helpful in how to use AGE Viewer
in general.
SIL dictates the License must be included
https://tldrlegal.com/license/open-font-license-(ofl)-explained#fulltext.
We must include the SIL license in the header.
The Cytoscape lib must be noted as such in the LICENSE.
References to Bitnine need to be removed, so the logo needs to change, the
same is true for the lines in the About sidebar.
On Mon, Jan 3, 2022 at 4:50 PM Nicholas Sorrell <ni...@cint.io> wrote:
> I have a few questions:
>
>
> * Should this file be included in the repo? It seems to be specific to
> Agensgraph.
> https://github.com/apache/incubator-age-viewer/blob/main/frontend/src/documents/tutorial/northwind.jsx
> * The SVGs referenced in this file (
> https://github.com/apache/incubator-age-viewer/blame/main/frontend/src/static/icons/font/fontello.svg)
> seem to possibly be under the SIL license? I'm not entirely sure how
> fontello packages/licenses.
> * This Cytoscape lib seems to be bundled and under a different
> license:
> https://github.com/apache/incubator-age-viewer/blob/main/frontend/src/lib/cytoscape-cxtmenu-bitnine/LICENSE
> - should it be noted as such?
> * Are there any possible issues from using the Bitnine logos for AGE
> Viewer in the frontend/public folder? Here is an example:
> https://github.com/apache/incubator-age-viewer/blob/main/frontend/public/safari-pinned-tab.svg
> * Should the following references to "AGViewer" be changed to "AGE
> Viewer" or "Apache AGE Viewer"?
> *
> https://github.com/apache/incubator-age-viewer/blob/839ce463543e8472fcbfd30784c4d46a441be1d6/frontend/public/site.webmanifest#L2
> *
> https://github.com/apache/incubator-age-viewer/blob/839ce463543e8472fcbfd30784c4d46a441be1d6/frontend/public/site.webmanifest#L3
> *
> https://github.com/apache/incubator-age-viewer/blob/839ce463543e8472fcbfd30784c4d46a441be1d6/frontend/public/index.html#L29
> *
> https://github.com/apache/incubator-age-viewer/blob/839ce463543e8472fcbfd30784c4d46a441be1d6/frontend/public/index.html#L38
> *
> https://github.com/apache/incubator-age-viewer/blob/main/frontend/src/components/sidebar/presentations/SidebarAbout.jsx
> lines 27, 44, 49
> * SidebarAbout.jsx -
> https://github.com/apache/incubator-age-viewer/blob/main/frontend/src/components/sidebar/presentations/SidebarAbout.jsx
> * Should line 31 say "Made by Bitnine"?
> * Should line 71 reference the Apache repo instead of the Bitnine
> repo?
>
> --
> Nick Sorrell
> 513.288.1084
> https://cint.io
>
>
> ________________________________
> From: Josh Innis <jo...@gmail.com>
> Sent: Monday, January 3, 2022 4:08 PM
> To: dev@age.apache.org <de...@age.apache.org>
> Subject: Re: [VOTE] Release Apache AGE Viewer (incubating) 1.0.0-rc0
>
> Hi Felix,
>
> I believe that dependencies that are not bundled in the release are not
> required. Which would be all the dependencies listed in the two
> package.json files.
>
> https://infra.apache.org/licensing-howto.html#bundled-vs-non-bundled
>
> For agensGraphRepository.js, the project AgensGraph is distributed under
> the Apache 2.0 license
> https://github.com/bitnine-oss/agensgraph/blob/master/LICENSE. No bundled
> bits are included in the release. A scan of the United Statement Patent and
> Trademark database yields no relevant results.
>
> We should remove the reference to bitnine/agviewer and either remove the
> section or construct a docker image under the apache name.
>
>
>
>
> On Thu, Dec 30, 2021 at 7:29 PM Felix Cheung <fe...@hotmail.com>
> wrote:
>
> > lots of dependencies - are these licenses reviewed and handled in the
> > LICENSE file?
> >
> >
> https://github.com/apache/incubator-age-viewer/blob/main/backend/package.json
> >
> >
> https://github.com/apache/incubator-age-viewer/blob/main/frontend/package.jso\
> >
> > this requires a component - again could be a LICENSE check, if it is
> > required for execution (at runtime) we should check license of that too.
> > Also what is the trademark on the “AgensGraph” name?
> >
> >
> https://github.com/apache/incubator-age-viewer/blob/main/backend/src/models/agensgraph/agensGraphRepository.js
> > '@bitnine-oss/ag-driver');
> >
> > The
> > https://github.com/apache/incubator-age-viewer/blob/main/README.md
> > references to bitnine/agviewer
> > ________________________________
> > From: Eya Badal <ey...@apache.org>
> > Sent: Thursday, December 30, 2021 6:44:11 AM
> > To: dev@age.apache.org <de...@age.apache.org>
> > Subject: Re: [VOTE] Release Apache AGE Viewer (incubating) 1.0.0-rc0
> >
> > Looks good to me
> > +1
> >
> > I checked the following:
> >
> > - Signature and hash - check.
> > - DISCLAIMER - check.
> > - LICENSE and NOTICE - check.
> > - No unexpected binary files - check.
> > - incubating in name - check.
> >
> > Best,
> > Eya
> >
> > On 2021/12/23 00:58:26 Alex Kwak wrote:
> > > Dear Apache AGE Community,
> > >
> > > This is an official vote for the Apache AGE Viewer release v1.0.0-rc0
> > that we have been working toward it.
> > >
> > > To learn more about Apache AGE, please see http://age.apache.org/
> > >
> > > Functionalities included and addressed in this release:
> > > - Graph visualization for AGE.
> > > - Extends edge and vertex point by point.
> > >
> > >
> > > The vote is open now and until December 25th at 5:00 PM PST and passes
> > if a majority +1 votes are cast, with a minimum of 3 +1 votes.
> > >
> > > [ ] +1 Release this package as Apache AGE 0.6.0
> > > [ ] 0 I won't get in the way
> > > [ ] -1 Do not release this package because ...
> > >
> > >
> > > The git tag to be discussed and voted upon
> > > https://github.com/apache/incubator-age-viewer/releases/tag/v1.0.0
> > >
> > > The git commit hash:
> > > commit 2c55b2302875449565674aa039517734cf338031
> > >
> > > The release files, including signatures, digests, etc. can be found at:
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/age/viewer/apache-age-viewer-1.0.0-incubating-rc0/
> > >
> > > The SHA512 Checksum for these artifacts is:
> > >
> >
> 8fa45254bba973678243a78321869aaf88c746b55168d4401931c5d6e87192c80d581d97e927c8b87098aaacba8d6f77bf1ca225e864c49117f5cb1887c699d0
> > >
> > > Release artifacts are signed with the following key:
> > > https://downloads.apache.org/incubator/age/KEYS
> > >
> > > The fingerprint of key to sign release artifacts:
> > > 0E7F 408D 8C6A 1952 329C B379 D471 FDCE 5F5C 5B82
> > >
> > > For more information about the contents of this release, see:
> > > https://github.com/apache/incubator-age-viewer/releases/tag/v1.0.0
> > >
> > >
> > >
> > > Best regards,
> > > Alex Kwak
> > >
> >
>
Re: [VOTE] Release Apache AGE Viewer (incubating) 1.0.0-rc0
Posted by Nicholas Sorrell <ni...@cint.io>.
I have a few questions:
* Should this file be included in the repo? It seems to be specific to Agensgraph. https://github.com/apache/incubator-age-viewer/blob/main/frontend/src/documents/tutorial/northwind.jsx
* The SVGs referenced in this file (https://github.com/apache/incubator-age-viewer/blame/main/frontend/src/static/icons/font/fontello.svg) seem to possibly be under the SIL license? I'm not entirely sure how fontello packages/licenses.
* This Cytoscape lib seems to be bundled and under a different license: https://github.com/apache/incubator-age-viewer/blob/main/frontend/src/lib/cytoscape-cxtmenu-bitnine/LICENSE - should it be noted as such?
* Are there any possible issues from using the Bitnine logos for AGE Viewer in the frontend/public folder? Here is an example: https://github.com/apache/incubator-age-viewer/blob/main/frontend/public/safari-pinned-tab.svg
* Should the following references to "AGViewer" be changed to "AGE Viewer" or "Apache AGE Viewer"?
* https://github.com/apache/incubator-age-viewer/blob/839ce463543e8472fcbfd30784c4d46a441be1d6/frontend/public/site.webmanifest#L2
* https://github.com/apache/incubator-age-viewer/blob/839ce463543e8472fcbfd30784c4d46a441be1d6/frontend/public/site.webmanifest#L3
* https://github.com/apache/incubator-age-viewer/blob/839ce463543e8472fcbfd30784c4d46a441be1d6/frontend/public/index.html#L29
* https://github.com/apache/incubator-age-viewer/blob/839ce463543e8472fcbfd30784c4d46a441be1d6/frontend/public/index.html#L38
* https://github.com/apache/incubator-age-viewer/blob/main/frontend/src/components/sidebar/presentations/SidebarAbout.jsx lines 27, 44, 49
* SidebarAbout.jsx - https://github.com/apache/incubator-age-viewer/blob/main/frontend/src/components/sidebar/presentations/SidebarAbout.jsx
* Should line 31 say "Made by Bitnine"?
* Should line 71 reference the Apache repo instead of the Bitnine repo?
--
Nick Sorrell
513.288.1084
https://cint.io
________________________________
From: Josh Innis <jo...@gmail.com>
Sent: Monday, January 3, 2022 4:08 PM
To: dev@age.apache.org <de...@age.apache.org>
Subject: Re: [VOTE] Release Apache AGE Viewer (incubating) 1.0.0-rc0
Hi Felix,
I believe that dependencies that are not bundled in the release are not
required. Which would be all the dependencies listed in the two
package.json files.
https://infra.apache.org/licensing-howto.html#bundled-vs-non-bundled
For agensGraphRepository.js, the project AgensGraph is distributed under
the Apache 2.0 license
https://github.com/bitnine-oss/agensgraph/blob/master/LICENSE. No bundled
bits are included in the release. A scan of the United Statement Patent and
Trademark database yields no relevant results.
We should remove the reference to bitnine/agviewer and either remove the
section or construct a docker image under the apache name.
On Thu, Dec 30, 2021 at 7:29 PM Felix Cheung <fe...@hotmail.com>
wrote:
> lots of dependencies - are these licenses reviewed and handled in the
> LICENSE file?
>
> https://github.com/apache/incubator-age-viewer/blob/main/backend/package.json
>
> https://github.com/apache/incubator-age-viewer/blob/main/frontend/package.jso\
>
> this requires a component - again could be a LICENSE check, if it is
> required for execution (at runtime) we should check license of that too.
> Also what is the trademark on the “AgensGraph” name?
>
> https://github.com/apache/incubator-age-viewer/blob/main/backend/src/models/agensgraph/agensGraphRepository.js
> '@bitnine-oss/ag-driver');
>
> The
> https://github.com/apache/incubator-age-viewer/blob/main/README.md
> references to bitnine/agviewer
> ________________________________
> From: Eya Badal <ey...@apache.org>
> Sent: Thursday, December 30, 2021 6:44:11 AM
> To: dev@age.apache.org <de...@age.apache.org>
> Subject: Re: [VOTE] Release Apache AGE Viewer (incubating) 1.0.0-rc0
>
> Looks good to me
> +1
>
> I checked the following:
>
> - Signature and hash - check.
> - DISCLAIMER - check.
> - LICENSE and NOTICE - check.
> - No unexpected binary files - check.
> - incubating in name - check.
>
> Best,
> Eya
>
> On 2021/12/23 00:58:26 Alex Kwak wrote:
> > Dear Apache AGE Community,
> >
> > This is an official vote for the Apache AGE Viewer release v1.0.0-rc0
> that we have been working toward it.
> >
> > To learn more about Apache AGE, please see http://age.apache.org/
> >
> > Functionalities included and addressed in this release:
> > - Graph visualization for AGE.
> > - Extends edge and vertex point by point.
> >
> >
> > The vote is open now and until December 25th at 5:00 PM PST and passes
> if a majority +1 votes are cast, with a minimum of 3 +1 votes.
> >
> > [ ] +1 Release this package as Apache AGE 0.6.0
> > [ ] 0 I won't get in the way
> > [ ] -1 Do not release this package because ...
> >
> >
> > The git tag to be discussed and voted upon
> > https://github.com/apache/incubator-age-viewer/releases/tag/v1.0.0
> >
> > The git commit hash:
> > commit 2c55b2302875449565674aa039517734cf338031
> >
> > The release files, including signatures, digests, etc. can be found at:
> >
> https://dist.apache.org/repos/dist/dev/incubator/age/viewer/apache-age-viewer-1.0.0-incubating-rc0/
> >
> > The SHA512 Checksum for these artifacts is:
> >
> 8fa45254bba973678243a78321869aaf88c746b55168d4401931c5d6e87192c80d581d97e927c8b87098aaacba8d6f77bf1ca225e864c49117f5cb1887c699d0
> >
> > Release artifacts are signed with the following key:
> > https://downloads.apache.org/incubator/age/KEYS
> >
> > The fingerprint of key to sign release artifacts:
> > 0E7F 408D 8C6A 1952 329C B379 D471 FDCE 5F5C 5B82
> >
> > For more information about the contents of this release, see:
> > https://github.com/apache/incubator-age-viewer/releases/tag/v1.0.0
> >
> >
> >
> > Best regards,
> > Alex Kwak
> >
>