Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wikifyのようにmarkdownを出力したい #69

Open
tossy310 opened this issue May 4, 2020 · 8 comments
Open

wikifyのようにmarkdownを出力したい #69

tossy310 opened this issue May 4, 2020 · 8 comments
Milestone

Comments

@tossy310
Copy link
Member

tossy310 commented May 4, 2020

早稲田の一部の人で、僕がforkした https://github.com/tossy310/rime/tree/markdownify を使って、wikify_full と同内容のものをmarkdown形式出力したものを使っています。

需要があれば、上の実装をクリーンアップしたものを PR に出したいと思うのですが、いかがでしょうか。

GitHub 上で作問作業をする場合、htmlifyの出力をgithub pagesから見ることも可能ですが、markdownのほうが特別な追加作業なしで見れるので便利でした。
(一方、markdownの表現力が弱いため、表が多少見にくくなりがちです。)

@hiroshi-cl
Copy link
Member

すでにパッチが存在しているのであれば初めからプルリクとして出していただくと良いと思います
プラグインの形になっていれば既存機能を壊すものでもないのでぱぱっとマージしてしまって良さそうです

tossy310@651c2f2 👀

@tossy310
Copy link
Member Author

tossy310 commented May 4, 2020

わかりました

hiroshi-cl added a commit that referenced this issue May 5, 2020
@tossy310
Copy link
Member Author

tossy310 commented May 7, 2020

このissueについて、既存の wikify_full, htmlify_full とと合わせて実装をクリーンアップしようと思っています。そこで、2点あるのですが、

(1) テンプレートエンジン(Jinjaなど)を使うとコードが綺麗になりそうなのですが、 rime では外部ライブラリはあまり依存しないようにしているのでしょうか?

(2) そもそも not さんが今やっていらっしゃるという、内部の作り直し(?) との作業の衝突が少し気になっています。plugin周りの計画などありますでしょうか?

@not522
Copy link
Member

not522 commented May 7, 2020

Rimeを設計から見直す計画について、JAG内部の議論で閉じがちでほとんど外部から見えない状態になっているので、ちゃんとissueを立てますね。

@not522 not522 mentioned this issue May 7, 2020
@not522
Copy link
Member

not522 commented May 9, 2020

  1. テンプレートエンジン

導入しましょう。問題文機能にも使えそうですし、良いと思います。

  1. plugin

masterブランチに修正を加える形で作業してもらうのが良いと思います。自分が書き直しているコードは #70 より前の時点でブランチを切っているので、どのみち修正してから取り込む必要があります。

@not522
Copy link
Member

not522 commented May 9, 2020

コマンドの仕様も変えちゃって良いかなと思っていて、
rime wikify_full -> rime summarize --pukiwiki
rime htmlify_full -> rime summarize --html
rime markdownify_full -> rime summarize --markdown
というのはどうでしょう。
rime wikifyはたぶん使ってる人が少ないので廃止してしまっても良いと思います。

@tossy310
Copy link
Member Author

tossy310 commented May 9, 2020

テンプレートエンジン

Jinja を使って実装中です。(WIP: https://github.com/tossy310/rime/tree/useTemplate
Jinja は現状 python2.7.x と Python 3.5+ に対応なので、そこまで大きな問題にはならないかと思っていますが、 #71 でのコメントのように python2 系のサポートを残すということでしたら、外部依存があると将来的に少しやりにくくなるかも...? とも思います。

コマンドの仕様

wikify は不要で wikify_full だけで十分と同じく思っていました。
コマンドをまとめるのも良いと思います。ちなみに細かいですが、オプションを指定しなかった場合の挙動は何が良いでしょうか...?
CLIに何か表示するだけでも良いですし、デフォルトを決め打ちしても良いですし、インタラクティブに聞いても良いと思います... (wikifyはJAGのみの利用だと思うのでデフォルトをwikifyにするのはあまり惹かれないです)

@not522
Copy link
Member

not522 commented May 9, 2020

テンプレートエンジン

Python2はこれ以上の更新が行われないはずなので、現在サポートのあるライブラリであれば問題ないと思います。もし今後サポートが打ち切られるとしても、バージョン指定でインストールすれば良いと思います。

コマンドの仕様

指定がない場合はエラーにしてCLIにメッセージを出すので良いと思います。(ちなみに、少なくとも北大はPukiWikiを使っています。)

@not522 not522 added this to the v3 milestone May 9, 2020
not522 added a commit that referenced this issue Jun 28, 2020
Internal cleanup of summarize commands #69
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants