feature:出力先で名前をつけて保存

KeiTa4446 commented 1 month ago

Description（変更内容）

-これまでは出力先フォルダを指定するだけだったが，出力したい場所で名前をつけて保存し，その結果がレイヤに追加されるようにした。 ...

Manual Testing（手動テスト）

QGISでプラグインを実行してください。
"出力レイヤ"で保存先を指定して，名前をつけて保存できるか確認してください。
保存した名前.tifがレイヤに追加されたら成功です。

Summary by CodeRabbit

新機能
- 入力および出力データのファイルフィルタを追加しました。
- 出力パスの処理を改善しました。
- 変換とキャンセル操作のボタンクリックイベントの処理を調整しました。
スタイル
- QLabelウィジェットのテキストを「出力フォルダ」から「出力レイヤ」に変更しました。
- QgsFileWidgetの名前を「mQgsFileWidget_」から「mQgsFileWidget_output」に変更しました。

coderabbitai[bot] commented 1 month ago

Walkthrough

この変更では、dem_to_csmap.pyスクリプトのファイルハンドリングロジックとユーザーインターフェース要素に関する調整が行われました。特に、入力および出力データのファイルフィルター、ストレージモード設定、および出力パスの処理に関する変更が含まれています。また、変換およびキャンセル操作のボタンクリックイベント処理も調整されました。

Changes

ファイル	変更内容
`dem_to_csmap.py`	ファイルフィルター、ストレージモード設定、出力パス処理、およびボタンクリックイベント処理の変更。
`dem_to_csmap.ui`	QLabelウィジェットのテキストとQgsFileWidgetの名前、および矩形の幅の変更。

Sequence Diagram(s) (Beta)

sequenceDiagram
    participant User
    participant UI
    participant Script

    User->>UI: 入力ファイル選択
    UI->>Script: ファイルパス送信
    User->>UI: 出力レイヤ選択
    UI->>Script: 出力パス送信
    User->>UI: 変換ボタン押下
    UI->>Script: 変換処理開始
    Script->>UI: 変換完了通知
    UI->>User: 完了メッセージ表示

Poem

変換ツールが新しくなり、
ファイル選択も簡単に、
出力レイヤも設定して、
ボタン一つで変換完了、
うさぎも喜ぶ便利さよ！
🐇✨

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.

KeiTa4446 commented 1 month ago

issue #13 とは無関係の修正です。こちらは今後対応していく予定です。

geogra-geogra commented 1 month ago

中身についてはファイル名(例：csmap)、または拡張子含めたファイル名(例：csmap.tif)のいずれも指定したファイル名のtifファイルが作成され、レイヤに表示されることを確認しました。今回の変更点について、山本からの指摘事項はありません。 @Kanahiro さんお願いします。

以下、コンフリクトおきそうなのであとで対応していただきたい点です。ボタンについてになります。現状、runではなくcancelがデフォルトになってしまっていているので、uiファイルの46行目以降のitemにデフォルトを加えることで、回避すると良いと思います。

Designerを用いて以下画面を編集して、下のコードのように変わっているかの確認を行うのがよいかもです。

加えるべきコード

       <property name="default">
        <bool>true</bool>
       </property>

runボタン司る部分の全体

     <item>
      <widget class="QPushButton" name="pushButton_run">
       <property name="text">
        <string>Run</string>
       </property>
       <property name="default">
        <bool>true</bool>
       </property>
      </widget>
     </item>

MIERUNE / csmap-qgis-plugin