oss-gate / workshop

OSSの開発に未参加または参加したことはあるけどまだ自信がない人を後押しするワークショップ用のリポジトリー
124 stars 547 forks source link

OSS Gate Workshop: Tokyo: 2024-09-21: 3-yuma-3: FlatGeobuf: Work log #1877

Closed 3-yuma-3 closed 6 days ago

3-yuma-3 commented 6 days ago

This is a work log of a "OSS Gate workshop". "OSS Gate workshop" is an activity to increase OSS developers. Here's been discussed in Japanese. Thanks.

作業ログ作成時の説明

以下のテンプレートを埋めてタイトルに設定します。埋め方例はスクロールすると見えてきます。

OSS Gate Workshop: ${LOCATION}: ${YEAR}-${MONTH}-${DAY}: ${ACCOUNT_NAME}: ${OSS_NAME}: Work log

タイトル例↓:

OSS Gate Workshop: Tokyo: 2017-01-16: kou: Rabbit: Work log

OSS Gateワークショップ関連情報

3-yuma-3 commented 6 days ago

https://flatgeobuf.org/

geojsonに独自のpropertyを生やしていると、JavaScriptではserialize, deserializeできるけど、Javaだとできなかった(半年~1年前頃)ので、直したい

3-yuma-3 commented 6 days ago

今もJavaだとserialize, deserializeできないのかを調べる・試してみる

過去に調べたissue https://github.com/3-yuma-3/practice_flatgeobuf?tab=readme-ov-file#%E9%96%A2%E4%BF%82%E3%81%97%E3%81%A6%E3%81%9D%E3%81%86%E3%81%AAissue

今回それっぽいと思った別のissue https://github.com/flatgeobuf/flatgeobuf/issues/32 https://github.com/flatgeobuf/flatgeobuf/issues/36 https://github.com/flatgeobuf/flatgeobuf/issues/37

3-yuma-3 commented 6 days ago

flatgeobufのライセンス

https://github.com/flatgeobuf/flatgeobuf/blob/1d3647cec277aee63494a9f3b1958c3cb58674f5/LICENSE

opensource.org に書いてあるのでOSSという理解であってそう

https://opensource.org/licenses?ls=BSD-2-Clause

3-yuma-3 commented 6 days ago

2024-09-21 時点 1d3647cec277aee63494a9f3b1958c3cb58674f5 で同じように動かないかどうかを試す

3-yuma-3 commented 6 days ago

JavaのAPI仕様がないので、generateするようにfeedbackするのもあり

3-yuma-3 commented 6 days ago

JavaScriptでdeserializeするexampleはあるけど、serializeするexampleはないので、作ってあげるといいのか

3-yuma-3 commented 6 days ago

contribution.mdがリポジトリにない

https://github.com/flatgeobuf/flatgeobuf/tree/1d3647cec277aee63494a9f3b1958c3cb58674f5

issueのtabには以下のような文章がある

👋Want to contribute to flatgeobuf/flatgeobuf? If you have a bug or an idea, browse the open issues before opening a new one. You can also take a look at the Open Source Guide.

https://opensource.guide/ja/

他にissueを漁ってみたけど、テンプレはなさそう

3-yuma-3 commented 6 days ago

issueを上げる文章を一部日本語で考える

提案の形にしてみる プロジェクトとしてこうあったほうがいろんな人が助かるんじゃないか

title

serialize.js example should be there

body

deserializeのサンプルはあるけど、serializeのサンプルはなかった。ぐらいシンプルでいいかも

Thank you for deesrialize.js, it's very helpful.

There is deserialize.js in example, but no serialize.js so serialize.js should be there.

import { geojson } from 'flatgeobuf'
import { readFileSync, writeFileSync }  from 'fs'

const originalGeojson = JSON.parse(readFileSync('./no_properties_1.json'))
// console.log(JSON.stringify(originalGeojson, undefined, 1))
const serialized = geojson.serialize(originalGeojson)
writeFileSync('./no_properties_1.fgb', serialized)
const bufferJs = readFileSync('./no_properties_1.fgb')
const deserialized = geojson.deserialize(new Uint8Array(bufferJs))
console.log(JSON.stringify(deserialized, undefined, 1))
3-yuma-3 commented 6 days ago

これぐらいの差分であればいきなりPR作っても良さそうとのこと。 issueにコードを貼ると、それも所有権の問題を気にされる可能性もあるので、 issueにコードを貼るときはコードブロック内にライセンスの表記とかするらしい。

3-yuma-3 commented 6 days ago

次はJavaのexampleを作りたい

youkidearitai commented 6 days ago

Might be a helpful example but I think I need to make it more clear that the current ts/js impl. doesn't make spatial index

What should I do?みたいに食い下がっても良さそうですね

github-actions[bot] commented 6 days ago

おつかれさまでした!

ワークショップの終了にともないissueを閉じますが、このまま作業メモとして使っても構いません :ok_hand:

ワークショップの感想を集めています!

ブログなどに書かれた際は、このページへリンクの追加をお願いします :pray:

またの参加をお待ちしています!