ユーザーがパラメーターを設定できるように修正

KeiTa4446 commented 3 weeks ago

Close #5

Description（変更内容）

CsmapParams をユーザーが設定できるように修正した。
ウィンドウの見た目が悪いが，これは別途修正する。

...

Manual Testing（手動テスト）

sample.zip のデータを使用して，デフォルトパラメーターのままCS立体図を作成してください。
上記と同じデータを使用して，パラメーターを変更してCS立体図を作成してください。
デフォルトのパラメーターを使用した時とパラメーターを変更した時でCS立体図の見た目が変われば問題ありません。
q: パラメーター設定画面を見やすくするために必要な修正があればアドバイスいただきたいです。

sample.zip

Summary by CodeRabbit

新機能
- UIから設定可能なパラメータ（gf_size、gf_sigma、curvature_size、height_scale、slope_scale、curvature_scale）を追加しました。
- 入力パス変数をself.ui.mQgsFileWidget_input.filePath()に変更しました。
UI改善
- ウィジェットのレイアウトとプロパティを大幅に変更し、視覚的なプレゼンテーションと機能性を向上させました。

coderabbitai[bot] commented 3 weeks ago

Walkthrough

この変更は、dem_to_csmap.pyファイルのconvert_dem_to_csmapメソッドにおいて、CsmapParamsの初期化に使用するパラメータの設定をUIの値を使って行うように更新されたものです。また、入力パス変数がself.ui.mQgsFileWidget_input.filePath()を使用するように変更されました。さらに、dem_to_csmap.uiファイルにおいて、UIのレイアウトとプロパティの大幅な変更が行われ、視覚的なプレゼンテーションと機能性が向上しました。

Changes

ファイル	変更内容
`dem_to_csmap.py`	`convert_dem_to_csmap`メソッドが更新され、`CsmapParams`の初期化にUIの値を使用するようになり、入力パス変数が`self.ui.mQgsFileWidget_input.filePath()`に変更されました。
`dem_to_csmap.ui`	UIファイルのレイアウトとプロパティに大幅な変更が加えられ、`QLabel`、`QgsFileWidget`、`QCheckBox`、`QPushButton`、`QGroupBox`、`QSpinBox`、`QDoubleSpinBox`などの新しいウィジェットが追加されました。

Poem

変更の風が吹くよ、コードの中に新たな光。パラメータがUIで踊り、入力パスも新しい道。 🐇✨ UIの顔も輝きを増し、みんなの作業が楽しくなる。

Tips

### Chat There are 3 ways to chat with [CodeRabbit](https://coderabbit.ai): - Review comments: Directly reply to a review comment made by CodeRabbit. Example: - `I pushed a fix in commit .` - `Generate unit testing code for this file.` - `Open a follow-up GitHub issue for this discussion.` - Files and specific lines of code (under the "Files changed" tab): Tag `@coderabbitai` in a new review comment at the desired location with your query. Examples: - `@coderabbitai generate unit testing code for this file.` - `@coderabbitai modularize this function.` - PR comments: Tag `@coderabbitai` in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples: - `@coderabbitai generate interesting stats about this repository and render them as a table.` - `@coderabbitai show all the console.log statements in this repository.` - `@coderabbitai read src/utils.ts and generate unit testing code.` - `@coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.` - `@coderabbitai help me debug CodeRabbit configuration file.` Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. ### CodeRabbit Commands (invoked as PR comments) - `@coderabbitai pause` to pause the reviews on a PR. - `@coderabbitai resume` to resume the paused reviews. - `@coderabbitai review` to trigger an incremental review. This is useful when automatic reviews are disabled for the repository. - `@coderabbitai full review` to do a full review from scratch and review all the files again. - `@coderabbitai summary` to regenerate the summary of the PR. - `@coderabbitai resolve` resolve all the CodeRabbit review comments. - `@coderabbitai configuration` to show the current CodeRabbit configuration for the repository. - `@coderabbitai help` to get help. Additionally, you can add `@coderabbitai ignore` anywhere in the PR description to prevent this PR from being reviewed. ### CodeRabbit Configration File (`.coderabbit.yaml`) - You can programmatically configure CodeRabbit by adding a `.coderabbit.yaml` file to the root of your repository. - Please see the [configuration documentation](https://docs.coderabbit.ai/guides/configure-coderabbit) for more information. - If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: `# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json` ### Documentation and Community - Visit our [Documentation](https://coderabbit.ai/docs) for detailed information on how to use CodeRabbit. - Join our [Discord Community](https://discord.com/invite/GsXnASn26c) to get help, request features, and share feedback. - Follow us on [X/Twitter](https://twitter.com/coderabbitai) for updates and announcements.

geogra-geogra commented 3 weeks ago

動作確認しましたLGTM @Kanahiro

もしパラメータが数字ではなくバーがよいとかなった場合はまた別でissue立てると良いと思います。

デザインについては、ボタンがひきのばされていてみづらい気もします。機会あるタイミングで、横幅を狭くしてみるのも良いかと思います

Kanahiro commented 3 weeks ago

あと、max_workersは変更させない？

KeiTa4446 commented 3 weeks ago

Close #27

@geogra-geogra ボタンを修正しました。見た目の確認をお願いします。

@Kanahiro max_workersとchunk_sizeをオプションで設定できるようにしました。 chunk_sizeは256ずつ上下するように設定しました。また，デフォルトは1024に戻しました。

geogra-geogra commented 3 weeks ago

デザイン良いと思います！このタスクについては、あとはお二人にお任せします

KeiTa4446 commented 3 weeks ago

@Kanahiro 一旦，入力データを複数選択できないように，元に戻しました！

Kanahiro commented 3 weeks ago

@KeiTa4446 もしかすると、max_workersの部分がなおっていないかもです？

KeiTa4446 commented 3 weeks ago

@Kanahiro 修正しました。手動テストで早くなったことが実感できたので，直っていると思います。

MIERUNE / csmap-qgis-plugin