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

\magを使わないようにするrealオプションを追加 #3

Merged
merged 5 commits into from Jul 10, 2016

Conversation

Projects
None yet
5 participants
@abenori

abenori commented Jul 7, 2016

bxjsarticleやltjsarticleと同じようにして\magを使わないようにするrealオプションを追加してみました.また,magオプションで従来のように\magを使うようにしてみています.

@aminophen

This comment has been minimized.

Show comment
Hide comment
@aminophen

aminophen Jul 7, 2016

Member

いいですね。機能していそうな気がします。

ただ、mag というクラスオプション名は geometry を誤作動させそうなので

! Package keyval Error: No value specified for mag.

もっと別なオプション名が良いと思います。real もなにか未知のものと被るかもしれないです。

Member

aminophen commented Jul 7, 2016

いいですね。機能していそうな気がします。

ただ、mag というクラスオプション名は geometry を誤作動させそうなので

! Package keyval Error: No value specified for mag.

もっと別なオプション名が良いと思います。real もなにか未知のものと被るかもしれないです。

@okumuralab

This comment has been minimized.

Show comment
Hide comment
@okumuralab

okumuralab Jul 7, 2016

Contributor

プルリクありがとうございます。オプション名が問題ですね。あとokumuraとなっているところはもうtexjporgとかでいいと思っています。ゆるい合意のもとでどんどん改変していく方針でどうでしょう?

Contributor

okumuralab commented Jul 7, 2016

プルリクありがとうございます。オプション名が問題ですね。あとokumuraとなっているところはもうtexjporgとかでいいと思っています。ゆるい合意のもとでどんどん改変していく方針でどうでしょう?

@aminophen

This comment has been minimized.

Show comment
Hide comment
@aminophen

aminophen Jul 7, 2016

Member

オプション名、なにがいいですか? とりあえず

あとokumuraとなっているところはもうtexjporgとかでいいと思っています。

#2 対応中の aminophen/master に abenori さんの変更をとりこんでそっちで変えました。

Member

aminophen commented Jul 7, 2016

オプション名、なにがいいですか? とりあえず

あとokumuraとなっているところはもうtexjporgとかでいいと思っています。

#2 対応中の aminophen/master に abenori さんの変更をとりこんでそっちで変えました。

@zr-tex8r

This comment has been minimized.

Show comment
Hide comment
@zr-tex8r

zr-tex8r Jul 8, 2016

Contributor

生の「mm」や「cm」が残っているのが気ニナル……。

jsclasses.dtx 1738行目

で、bxjsナントカにも「1cm」が1ヶ所だけ残っているのがもっと気ニナル……。(うわぁ)

Contributor

zr-tex8r commented Jul 8, 2016

生の「mm」や「cm」が残っているのが気ニナル……。

jsclasses.dtx 1738行目

で、bxjsナントカにも「1cm」が1ヶ所だけ残っているのがもっと気ニナル……。(うわぁ)

@okumuralab

This comment has been minimized.

Show comment
Hide comment
@okumuralab

okumuralab Jul 8, 2016

Contributor

うわぁ。で思い出しましたが \topskip は1.38zw にしてその分を別のところで調整するのも TODO でした。

Contributor

okumuralab commented Jul 8, 2016

うわぁ。で思い出しましたが \topskip は1.38zw にしてその分を別のところで調整するのも TODO でした。

@abenori

This comment has been minimized.

Show comment
Hide comment
@abenori

abenori Jul 8, 2016

オプション名「real」はLuaTeX-jaのltjsarticleからとったものです.これ自身はbxjsarticleのmagstyle=realからだったので,\magを使う方をmagstyle=magにあわせてmagにしたのですが,これは確かにまずそうですね.

生の「mm」や「cm」が残っているのが気ニナル……。

うん,これは確かに気になる……

abenori commented Jul 8, 2016

オプション名「real」はLuaTeX-jaのltjsarticleからとったものです.これ自身はbxjsarticleのmagstyle=realからだったので,\magを使う方をmagstyle=magにあわせてmagにしたのですが,これは確かにまずそうですね.

生の「mm」や「cm」が残っているのが気ニナル……。

うん,これは確かに気になる……

@zr-tex8r

This comment has been minimized.

Show comment
Hide comment
@zr-tex8r

zr-tex8r Jul 8, 2016

Contributor

某bxjsナントカの 1cm は後から(v1.0で)加わったもので、その時に考慮が足りなかったらしい 😞

ちなみに、bxjsナントカはUIにおいては可能な限りjsclassesを踏襲しようとします。
だからmagstyle関連のオプションで素敵な名前が付けば、
bxjsナントカもそれに合わせることになる、のでしょう。

Contributor

zr-tex8r commented Jul 8, 2016

某bxjsナントカの 1cm は後から(v1.0で)加わったもので、その時に考慮が足りなかったらしい 😞

ちなみに、bxjsナントカはUIにおいては可能な限りjsclassesを踏襲しようとします。
だからmagstyle関連のオプションで素敵な名前が付けば、
bxjsナントカもそれに合わせることになる、のでしょう。

@aminophen

This comment has been minimized.

Show comment
Hide comment
@aminophen

aminophen Jul 8, 2016

Member

ltjs のオプション名をちゃんとみていなかったので存じませんでした。が、real というオプション名単独では何を意味するのかわかりにくいと思ってしまいます。mag がマズそうなついでに、こちらも素敵な名前を付けてよいのではないでしょうか。

…といっても全然素敵な名前を思いつかないので適当に書きます。

たとえば bxjsナントカ や jsナントカ の現在の default が「magstyle=mag」なわけですが、この shortened form を「magstyle」にできないでしょうか。「magstyle=real」は \mag しないのだから shortened form にすると「nomag」とかが考えられます。

Member

aminophen commented Jul 8, 2016

ltjs のオプション名をちゃんとみていなかったので存じませんでした。が、real というオプション名単独では何を意味するのかわかりにくいと思ってしまいます。mag がマズそうなついでに、こちらも素敵な名前を付けてよいのではないでしょうか。

…といっても全然素敵な名前を思いつかないので適当に書きます。

たとえば bxjsナントカ や jsナントカ の現在の default が「magstyle=mag」なわけですが、この shortened form を「magstyle」にできないでしょうか。「magstyle=real」は \mag しないのだから shortened form にすると「nomag」とかが考えられます。

@kuroky49

This comment has been minimized.

Show comment
Hide comment
@kuroky49

kuroky49 Jul 8, 2016

ユーザに見える表面的な違いは,\mag を使うと,trueなしの絶対単位(たとえばcm) がtrue付きの絶対単位(たとえばtruecm)と異なる,というところにあると思います.(注.「絶対単位」の「絶対」は,zwなどのまわりの要素の大きさで変化しないという意味で用いました.)
 その観点で言うと,unit=absolute/relative みたいなものが,表現としては合っているでしょうか?

kuroky49 commented Jul 8, 2016

ユーザに見える表面的な違いは,\mag を使うと,trueなしの絶対単位(たとえばcm) がtrue付きの絶対単位(たとえばtruecm)と異なる,というところにあると思います.(注.「絶対単位」の「絶対」は,zwなどのまわりの要素の大きさで変化しないという意味で用いました.)
 その観点で言うと,unit=absolute/relative みたいなものが,表現としては合っているでしょうか?

@zr-tex8r

This comment has been minimized.

Show comment
Hide comment
@zr-tex8r

zr-tex8r Jul 8, 2016

Contributor

magstyleを実装した当時、丸一日考えていい名前が思いつかなかったので、仮の名雨として「最終的な正解にならなさそうなもの(=mag/real/xreal)」を敢えて選んだ、とかいうアレな経緯があったりする。

  • magstyle=magnativemag
  • magstyle=realcomputedmag
  • magstyle=xrealcomputedmag*

コレでどうかな?

※もちろん、これらのオプションが「上級者向け」であることを前提とします。

Contributor

zr-tex8r commented Jul 8, 2016

magstyleを実装した当時、丸一日考えていい名前が思いつかなかったので、仮の名雨として「最終的な正解にならなさそうなもの(=mag/real/xreal)」を敢えて選んだ、とかいうアレな経緯があったりする。

  • magstyle=magnativemag
  • magstyle=realcomputedmag
  • magstyle=xrealcomputedmag*

コレでどうかな?

※もちろん、これらのオプションが「上級者向け」であることを前提とします。

@zr-tex8r

This comment has been minimized.

Show comment
Hide comment
@zr-tex8r

zr-tex8r Jul 8, 2016

Contributor

アッ、jsclasses のオプションに keyval を持ち込もうとする「勇気のある者」が現れたぞ!

Contributor

zr-tex8r commented Jul 8, 2016

アッ、jsclasses のオプションに keyval を持ち込もうとする「勇気のある者」が現れたぞ!

@abenori

This comment has been minimized.

Show comment
Hide comment
@abenori

abenori Jul 8, 2016

生の「mm」や「cm」が残っているのが気ニナル……。

うん,これは確かに気になる……

直した感じな気分.

abenori commented Jul 8, 2016

生の「mm」や「cm」が残っているのが気ニナル……。

うん,これは確かに気になる……

直した感じな気分.

@okumuralab

This comment has been minimized.

Show comment
Hide comment
@okumuralab

okumuralab Jul 8, 2016

Contributor

\addtolength\fullwidth{-36mm} とか,いくつかあるみたいですね。
あんまり直しちゃうと,過去のものが再現できなくなるので,ちょっと怖い気もします。

Contributor

okumuralab commented Jul 8, 2016

\addtolength\fullwidth{-36mm} とか,いくつかあるみたいですね。
あんまり直しちゃうと,過去のものが再現できなくなるので,ちょっと怖い気もします。

@abenori

This comment has been minimized.

Show comment
Hide comment
@abenori

abenori Jul 8, 2016

あんまり直しちゃうと,過去のものが再現できなくなるので,ちょっと怖い気もします。

とりあえず寸法っぽいのは機械的に全部スケーリングさせた値におきかえてしましましたが,どうでしょうか?(realを指定しなければ挙動は変わらないようにしているつもりではあります.)

abenori commented Jul 8, 2016

あんまり直しちゃうと,過去のものが再現できなくなるので,ちょっと怖い気もします。

とりあえず寸法っぽいのは機械的に全部スケーリングさせた値におきかえてしましましたが,どうでしょうか?(realを指定しなければ挙動は変わらないようにしているつもりではあります.)

@okumuralab

This comment has been minimized.

Show comment
Hide comment
@okumuralab

okumuralab Jul 8, 2016

Contributor

(realを指定しなければ挙動は変わらないようにしているつもりではあります.)

あぁそういう仕様でしたか。すみません,ちゃんと見てませんでした。

Contributor

okumuralab commented Jul 8, 2016

(realを指定しなければ挙動は変わらないようにしているつもりではあります.)

あぁそういう仕様でしたか。すみません,ちゃんと見てませんでした。

@aminophen

This comment has been minimized.

Show comment
Hide comment
@aminophen

aminophen Jul 9, 2016

Member

オプション名

上級者向けなのか一般向けなのか? が決め手な気がします。私は mag の副作用を気にする一般ユーザも割といると思っていて、TikZ と相性が悪いとかだと普通に mag やめさせたい状況も起きると思います。

  • magstyle=mag → nativemag
  • magstyle=real → computedmag
  • magstyle=xreal → computedmag*

確かに実装に即していて、ZR さんは上級者向けと仰いますが、私はそんなに認識としては一般からかけ離れているとは思わないです。だからこれでも一般向けでよいかも。

一方

ユーザに見える表面的な違いは,\mag を使うと,trueなしの絶対単位(たとえばcm) がtrue付きの絶対単位(たとえばtruecm)と異なる,というところにあると思います.

の観点もよいと思います。unit=absolute/relative だと keyval が必要になってしまうので、これを短縮したものを考えようかと思って

  • magstyle=mag → magunit
  • magstyle=real → absunit (または absolute)
  • magstyle=xreal → absunit* (または absolute*)

うーん…(追記:単なる absolute だと textpos.sty と被ってしまうらしい。)

Member

aminophen commented Jul 9, 2016

オプション名

上級者向けなのか一般向けなのか? が決め手な気がします。私は mag の副作用を気にする一般ユーザも割といると思っていて、TikZ と相性が悪いとかだと普通に mag やめさせたい状況も起きると思います。

  • magstyle=mag → nativemag
  • magstyle=real → computedmag
  • magstyle=xreal → computedmag*

確かに実装に即していて、ZR さんは上級者向けと仰いますが、私はそんなに認識としては一般からかけ離れているとは思わないです。だからこれでも一般向けでよいかも。

一方

ユーザに見える表面的な違いは,\mag を使うと,trueなしの絶対単位(たとえばcm) がtrue付きの絶対単位(たとえばtruecm)と異なる,というところにあると思います.

の観点もよいと思います。unit=absolute/relative だと keyval が必要になってしまうので、これを短縮したものを考えようかと思って

  • magstyle=mag → magunit
  • magstyle=real → absunit (または absolute)
  • magstyle=xreal → absunit* (または absolute*)

うーん…(追記:単なる absolute だと textpos.sty と被ってしまうらしい。)

@aminophen

This comment has been minimized.

Show comment
Hide comment
@aminophen

aminophen Jul 9, 2016

Member

少し話題がそれますが:

\mag を使った版面拡大・縮小が行われ、かつ jsclasses のクラスオプションで用紙サイズ指定するとき、geometry には truedimen オプションを渡すのがベストという話を思い出したのですが

\PassOptionsToPackage{truedimen}{geometry}

をちりばめることはできないでしょうか? \PassOptionsToPackage はパッケージを呼ばない場合は何も起こらなくて、実際に呼び出すときだけ効力をもつのでいいんじゃないかと思っています。

Member

aminophen commented Jul 9, 2016

少し話題がそれますが:

\mag を使った版面拡大・縮小が行われ、かつ jsclasses のクラスオプションで用紙サイズ指定するとき、geometry には truedimen オプションを渡すのがベストという話を思い出したのですが

\PassOptionsToPackage{truedimen}{geometry}

をちりばめることはできないでしょうか? \PassOptionsToPackage はパッケージを呼ばない場合は何も起こらなくて、実際に呼び出すときだけ効力をもつのでいいんじゃないかと思っています。

@abenori

This comment has been minimized.

Show comment
Hide comment
@abenori

abenori Jul 9, 2016

(realを指定しなければ挙動は変わらないようにしているつもりではあります.)

あぁそういう仕様でしたか。すみません,ちゃんと見てませんでした。

jsclasses内に現れている**ptという寸法を\js@mptに変更しました.その上で,

  • realが指定されたら,\magの発行をやめ\js@mptを適当な値に設定する
  • magが指定されたら,\magを発行し\js@mpt=1ptとする(つまり従来と同じ動作)

としています(bxjsclsの真似,オプション名は上で議論されている通りで,変更されるでしょうが).ちなみにデフォルトはmag指定と同じです.(個人的にはreal指定と同じにしたいかも…….)

abenori commented Jul 9, 2016

(realを指定しなければ挙動は変わらないようにしているつもりではあります.)

あぁそういう仕様でしたか。すみません,ちゃんと見てませんでした。

jsclasses内に現れている**ptという寸法を\js@mptに変更しました.その上で,

  • realが指定されたら,\magの発行をやめ\js@mptを適当な値に設定する
  • magが指定されたら,\magを発行し\js@mpt=1ptとする(つまり従来と同じ動作)

としています(bxjsclsの真似,オプション名は上で議論されている通りで,変更されるでしょうが).ちなみにデフォルトはmag指定と同じです.(個人的にはreal指定と同じにしたいかも…….)

@zr-tex8r

This comment has been minimized.

Show comment
Hide comment
@zr-tex8r

zr-tex8r Jul 9, 2016

Contributor

keyval をクラスオプションに適用するのって結構難しいんですよね……。自分は bxjsナントカ で3回くらいバグを出した 😞

なので keyval は使わないことにすると、今まで出てきたの案は3つ。新たに2つ加えました。

  1. magstyle / nomag / nomag*
  2. nativemag / computedmag / computedmag*
  3. magunit / absunit / absunit*
  4. magunit / realunit / realunit*
  5. usemag / nomag / nomag*

3の“absolute”というのが自分にはピンと来なかったので、代わりに従来の“real”を持ってきたのが4です。

「magstyleの動作内容」を理解するのは一般のユーザには難しいと思います。一方で「magstyleを用意した動機」は簡単で、要するに「mag がトラブルを起こすのでヤダ」なわけです。「mag を使うか使わないか」と捉えたのが5です。

Contributor

zr-tex8r commented Jul 9, 2016

keyval をクラスオプションに適用するのって結構難しいんですよね……。自分は bxjsナントカ で3回くらいバグを出した 😞

なので keyval は使わないことにすると、今まで出てきたの案は3つ。新たに2つ加えました。

  1. magstyle / nomag / nomag*
  2. nativemag / computedmag / computedmag*
  3. magunit / absunit / absunit*
  4. magunit / realunit / realunit*
  5. usemag / nomag / nomag*

3の“absolute”というのが自分にはピンと来なかったので、代わりに従来の“real”を持ってきたのが4です。

「magstyleの動作内容」を理解するのは一般のユーザには難しいと思います。一方で「magstyleを用意した動機」は簡単で、要するに「mag がトラブルを起こすのでヤダ」なわけです。「mag を使うか使わないか」と捉えたのが5です。

@zr-tex8r

This comment has been minimized.

Show comment
Hide comment
@zr-tex8r

zr-tex8r Jul 9, 2016

Contributor

結局 5 がいいような気がしてきた。

Contributor

zr-tex8r commented Jul 9, 2016

結局 5 がいいような気がしてきた。

@aminophen

This comment has been minimized.

Show comment
Hide comment
@aminophen

aminophen Jul 10, 2016

Member

結局 5 がいいような気がしてきた。

もう決まりでいい気がします(笑)

Member

aminophen commented Jul 10, 2016

結局 5 がいいような気がしてきた。

もう決まりでいい気がします(笑)

@okumuralab

This comment has been minimized.

Show comment
Hide comment
@okumuralab

okumuralab Jul 10, 2016

Contributor

結局 5 がいいような気がしてきた。

Contributor

okumuralab commented Jul 10, 2016

結局 5 がいいような気がしてきた。

@abenori

This comment has been minimized.

Show comment
Hide comment
@abenori

abenori Jul 10, 2016

変えてみました.デフォルトはusemagですが,良いですか?

abenori commented Jul 10, 2016

変えてみました.デフォルトはusemagですが,良いですか?

@okumuralab

This comment has been minimized.

Show comment
Hide comment
@okumuralab

okumuralab Jul 10, 2016

Contributor

Contributor

okumuralab commented Jul 10, 2016

@zr-tex8r

This comment has been minimized.

Show comment
Hide comment
@zr-tex8r

zr-tex8r Jul 10, 2016

Contributor

real対応の変更を早く取り込みたい、のであれば、「どれを既定にするか」の議論は後回しにする(現状維持にする)方がいい、と思います。

Contributor

zr-tex8r commented Jul 10, 2016

real対応の変更を早く取り込みたい、のであれば、「どれを既定にするか」の議論は後回しにする(現状維持にする)方がいい、と思います。

@aminophen

This comment has been minimized.

Show comment
Hide comment
@aminophen

aminophen Jul 10, 2016

Member

「どれを既定にするか」の議論は後回しにする(現状維持にする)

賛成です。「nomag オプションを規定にするかどうか」は影響が大きいので、ここではなく forum で決めたほうがよいと思います。(jsclasses のリポジトリの存在は、まだちゃんと forum で告知してないのでアレの二の舞になったら困るし)

Member

aminophen commented Jul 10, 2016

「どれを既定にするか」の議論は後回しにする(現状維持にする)

賛成です。「nomag オプションを規定にするかどうか」は影響が大きいので、ここではなく forum で決めたほうがよいと思います。(jsclasses のリポジトリの存在は、まだちゃんと forum で告知してないのでアレの二の舞になったら困るし)

@okumuralab

This comment has been minimized.

Show comment
Hide comment
@okumuralab

okumuralab Jul 10, 2016

Contributor

現状維持というのはusemagがデフォルトということですね。なら賛成です(というか挙動が変わるのは避けたい)

Contributor

okumuralab commented Jul 10, 2016

現状維持というのはusemagがデフォルトということですね。なら賛成です(というか挙動が変わるのは避けたい)

@aminophen

This comment has been minimized.

Show comment
Hide comment
@aminophen

aminophen Jul 10, 2016

Member

大勢が決まったようにみえますが、もう merge してよいですか?

Member

aminophen commented Jul 10, 2016

大勢が決まったようにみえますが、もう merge してよいですか?

@abenori

This comment has been minimized.

Show comment
Hide comment
@abenori

abenori Jul 10, 2016

よさげならばマージお願いします.(自分でするのはなんか変な気がするので……ここはやはり奥村先生にしていただくのが筋でしょうか?)

abenori commented Jul 10, 2016

よさげならばマージお願いします.(自分でするのはなんか変な気がするので……ここはやはり奥村先生にしていただくのが筋でしょうか?)

@okumuralab okumuralab merged commit fc12d79 into master Jul 10, 2016

@okumuralab

This comment has been minimized.

Show comment
Hide comment
@okumuralab

okumuralab Jul 10, 2016

Contributor

まーじしますた。

Contributor

okumuralab commented Jul 10, 2016

まーじしますた。

@okumuralab

This comment has been minimized.

Show comment
Hide comment
@okumuralab

okumuralab Jul 10, 2016

Contributor

日付変えるのを忘れてる>自分。あと,s/okumura/texjporg/g でしょうか。

Contributor

okumuralab commented Jul 10, 2016

日付変えるのを忘れてる>自分。あと,s/okumura/texjporg/g でしょうか。

@okumuralab

This comment has been minimized.

Show comment
Hide comment
@okumuralab

okumuralab Jul 10, 2016

Contributor

(クローズしたのに追記)とりあえず日付と若干のコメントを加えましたが,作者名変更とかいろいろは,私の一存ではあれかもしれないので,とりあえず皆さんの議論にお任せ。

Contributor

okumuralab commented Jul 10, 2016

(クローズしたのに追記)とりあえず日付と若干のコメントを加えましたが,作者名変更とかいろいろは,私の一存ではあれかもしれないので,とりあえず皆さんの議論にお任せ。

@abenori abenori deleted the nomag_test branch Jul 11, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment