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
Newlines in two-argment show #915
Comments
gwater
referenced
this issue
in gwater/Optim.jl
Jan 20, 2022
Re-implement two-argument Base.show() such that it produces an executable string of Optim.Options() as specified in Base docs [1]. Move previous implementation to three-argument Base.show() which should be human-readable [2]. [1] https://docs.julialang.org/en/v1/base/io-network/#Base.show-Tuple{IO,%20Any} [2] https://docs.julialang.org/en/v1/base/io-network/#Base.show-Tuple{IO,%20Any,%20Any} Refs: #915
gwater
referenced
this issue
in gwater/Optim.jl
Jan 20, 2022
gwater
referenced
this issue
in gwater/Optim.jl
Jan 24, 2022
Re-implement two-argument Base.show() such that it produces an executable string of Optim.Options() as specified in Base docs [1]. Move previous implementation to three-argument Base.show() which should be human-readable [2]. [1] https://docs.julialang.org/en/v1/base/io-network/#Base.show-Tuple{IO,%20Any} [2] https://docs.julialang.org/en/v1/base/io-network/#Base.show-Tuple{IO,%20Any,%20Any} Refs: #915
pkofod
pushed a commit
that referenced
this issue
Feb 4, 2022
* fix(test): Types testset * feat(test): Enable testset Types * fix(show)!: produce executable string for Options Re-implement two-argument Base.show() such that it produces an executable string of Optim.Options() as specified in Base docs [1]. Move previous implementation to three-argument Base.show() which should be human-readable [2]. [1] https://docs.julialang.org/en/v1/base/io-network/#Base.show-Tuple{IO,%20Any} [2] https://docs.julialang.org/en/v1/base/io-network/#Base.show-Tuple{IO,%20Any,%20Any} Refs: #915 * fix(test): add coverage for three-argument show
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
These should be avoided in e.g.
Optim.jl/src/types.jl
Lines 134 to 143 in adc5b27
show
docs. Only the three-argument show methods should have newlines.The text was updated successfully, but these errors were encountered: