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

解パッチがファイル出力offでログにも表示されなくなる & ファイル出力オプションの挙動が不自然 #801

Closed
ty-v1 opened this issue Nov 30, 2020 · 4 comments · Fixed by #807

Comments

@ty-v1
Copy link
Contributor

ty-v1 commented Nov 30, 2020

noNeedOutputfalseに設定すると,修正パッチをファイルにも標準出力にも出力しない.

https://github.com/kusumotolab/kGenProg/blob/master/src/main/java/jp/kusumotolab/kgenprog/output/Exporters.java#L28-L36

@shinsuke-mat shinsuke-mat changed the title パッチ出力の挙動が気持ち悪い 解パッチのファイル出力オフでログにも表示されなくなる Dec 2, 2020
@shinsuke-mat
Copy link
Member

shinsuke-mat commented Dec 2, 2020

現状はこう.
noNeedOutput = 1 でコンソールにすらパッチが出力されなくなる.

noNeedOutput historyRecord where to output?
0 0 consl+patch
1 0
0 1 consl+patch+json
1 1 json

コンソール出力は常に必要なはず.
こうあるべき.

noNeedOutput historyRecord where to output?
0 0 consl+patch
1 0 consl
0 1 consl+patch+json
1 1 consl+json

noNeedOutput と historyRecord は前者のみが非定型なので解釈しにくい.
より改善するとこう.両方非定型にする.

noPatchOutput noHistoryRecord where to output?
0 0 consl+patch+json
1 0 consl+json
0 1 consl+patch
1 1 consl

デフォルトは noPatchOutput = 1, noHistoryRecord = 1 でよさそう

@shinsuke-mat
Copy link
Member

shinsuke-mat commented Dec 2, 2020

両方肯定系の方がより自然か.
コードも読みやすいし.

patchOutput historyRecord where to output?
0 0 consl
1 0 consl+patch
0 1 consl+json
1 1 consl+patch+json

デフォルトは同様に patchOutput = 0, historyRecord = 0

@RyokoIzuta
Copy link
Contributor

patchOutputがfileの出力を管理していて,historyRecordがjsonの管理をしているのであっていますか?
両方否定形にする前後でwhere to output内の内容が入れ替わっているので

@shinsuke-mat
Copy link
Member

shinsuke-mat commented Dec 2, 2020

patchOutputがfileの出力を管理していて,historyRecordがjsonの管理をしているのであっていますか?

厳密には

  • patchoutput → パッチdiffのファイル出力
  • historyRecord → 歴史jsonのファイル出力

内容が入れ替わっているので

just a typo
直しといた

shinsuke-mat added a commit that referenced this issue Dec 3, 2020
@shinsuke-mat shinsuke-mat changed the title 解パッチのファイル出力オフでログにも表示されなくなる 解パッチがファイル出力offでログにも表示されなくなる & ファイル出力オプションの挙動が不自然 Dec 3, 2020
@hrtwt hrtwt closed this as completed in #807 Dec 8, 2020
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

Successfully merging a pull request may close this issue.

3 participants