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

Undocumented exceptions #24043

Open
mschauer opened this issue Oct 7, 2017 · 17 comments
Open

Undocumented exceptions #24043

mschauer opened this issue Oct 7, 2017 · 17 comments
Labels
docs This change adds or pertains to documentation good first issue Indicates a good issue for first-time contributors to Julia Hacktoberfest Good for Hacktoberfest participants help wanted Indicates that a maintainer wants help on an issue or pull request

Comments

@mschauer
Copy link
Contributor

mschauer commented Oct 7, 2017

Some days ago I generated a list of Exceptions without inline documentation. Here it is:

 Base.DNSError
 Base.Distributed.BatchProcessingError
 Base.LibGit2.Error.GitError
 Base.LinAlg.ARPACKException
 Base.LinAlg.LAPACKException
 Base.LinAlg.PosDefException
 Base.LinAlg.RankDeficientException
 Base.LinAlg.SingularException
 Base.Pkg.Entry.PkgTestError
 Base.Pkg.PkgError
 Base.Pkg.Resolve.MaxSum.UnsatError
 Base.PrecompilableError
 Base.SimdLoop.SimdError
 Base.SparseArrays.CHOLMOD.CHOLMODException
 Base.SparseArrays.UMFPACK.MatrixIllConditionedException
 Base.Test.FallbackTestSetException
 Base.UVError                                                          
 ~Base.WrappedException~   
 InvalidStateException
 SegmentationFault
 UnicodeError

Some of those are quite prominent, e.g. PosDefException.

@fredrikekre fredrikekre added docs This change adds or pertains to documentation Hacktoberfest Good for Hacktoberfest participants good first issue Indicates a good issue for first-time contributors to Julia help wanted Indicates that a maintainer wants help on an issue or pull request labels Oct 7, 2017
@Vaibhavdixit02
Copy link
Contributor

Hey I would like to work on it can someone help me out on how to move forward?

@mschauer
Copy link
Contributor Author

Nice. Maybe we can collect some docstring proposals here. While we'll happily explain the howto, finding the right words for some of these exceptions needs some knowledge. For example, I myself also do not know what Base.WrappedException is for and could not write a docstring.

@Vaibhavdixit02
Copy link
Contributor

Can you suggest how to move forward, I can't seem to understand that.

@mschauer
Copy link
Contributor Author

Sure. Which Exception(s) would you be comfortable to start with? We'll go through the steps at https://github.com/JuliaLang/julia/blob/master/CONTRIBUTING.md#improving-documentation

@Vaibhavdixit02
Copy link
Contributor

Base.LinAlg.LAPACKException
Base.LinAlg.SingularException

Are two of the errors that I am a little familiar with so I think it would be better to start with them.

@mschauer
Copy link
Contributor Author

Do you want to prepare an initial pull request with reference to this issue so that we have something to base the discussion on?

@Vaibhavdixit02
Copy link
Contributor

Yeah sure.

@mschauer
Copy link
Contributor Author

mschauer commented Oct 18, 2017

A documented exception (example: DimensionMismatch) has

@Bubblemelon
Copy link

Bubblemelon commented Oct 23, 2017

Hi, is anyone working on the other exceptions?

@samoconnor
Copy link
Contributor

#7841

@mschauer
Copy link
Contributor Author

mschauer commented Oct 1, 2018

Update (using (subtypes(Exception)[map(x-> nothing != findfirst("No documentation found.", string(Base.Docs.doc(x))), subtypes(Exception))])

Base.CodePointError
Base.IOError
Base.InvalidCharError
Base.PrecompilableError
Base.SimdLoop.SimdError
CapturedException
Core.Compiler.DimensionMismatch
Core.Compiler.InvalidCodeError
Core.Compiler.KeyError
Distributed.BatchProcessingError
InvalidStateException
LibGit2.Error.GitError
LinearAlgebra.ARPACKException
LinearAlgebra.LAPACKException
LinearAlgebra.PosDefException
LinearAlgebra.RankDeficientException
LinearAlgebra.SingularException
Pkg.Resolve.MaxSum.UnsatError
Pkg.Types.PkgError
Pkg.Types.ResolverError
SegmentationFault
Sockets.DNSError
SuiteSparse.CHOLMOD.CHOLMODException
SuiteSparse.UMFPACK.MatrixIllConditionedException
Test.FallbackTestSetException

@Jutho
Copy link
Contributor

Jutho commented Apr 12, 2019

Not sure why LinearAlgebra.ARPACKException is even still defined? This could probably be removed?

@mschauer
Copy link
Contributor Author

Good find. I opened a corresponding issue at JuliaLinearAlgebra/Arpack.jl/issues/64, which still uses this

@AtsushiSakai
Copy link
Contributor

I added docstring for Sockets.DNSError in #34189

@manish011003
Copy link

manish011003 commented Oct 24, 2023

would you like me to add some more of Exception...

@mschauer
Copy link
Contributor Author

Yeah why not take inventory

@mschauer
Copy link
Contributor Author

Update

Base.CodePointError
Base.IOError
Base.InvalidCharError
Base.JuliaSyntax.ParseError
Base.PaddingError
Base.PrecompilableError
Base.ScheduledAfterSyncException
Base.SimdLoop.SimdError
Base.TOML.ParserError
Base.UnwrapTaskFailedException
CanonicalIndexError
CapturedException
ConcurrencyViolationError
Core.Compiler.CanonicalIndexError
Core.Compiler.DimensionMismatch
Core.Compiler.InvalidCodeError
Core.Compiler.KeyError
FileWatching.Pidfile.PidlockedError
InvalidStateException
LibGit2.Error.GitError
LinearAlgebra.LAPACKException
LinearAlgebra.RankDeficientException
Pkg.Resolve.ResolverError
Pkg.Resolve.UnsatError
Pkg.Types.PkgError
Printf.InvalidFormatStringError
SegmentationFault

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation good first issue Indicates a good issue for first-time contributors to Julia Hacktoberfest Good for Hacktoberfest participants help wanted Indicates that a maintainer wants help on an issue or pull request
Projects
None yet
Development

No branches or pull requests

8 participants