Dmitry Dygalo

Results 492 comments of Dmitry Dygalo

Hi @Manishearth Thank you for opening this! Indeed, the current implementation is somewhat sketchy and very poorly documented. My thinking is: - `Pin` keeps the addresses of documents stable &...

This would be useful indeed! Here are a few things to think about: - Behavior for recursive references. I have a use case when it will be useful to inline...

FWIW, `referencing` passes most of [the official referencing test suite](https://github.com/python-jsonschema/referencing-suite/tree/main) (except URI normalization) + `jsonschema` which uses `referencing` passes [the official JSON Schema Test Suite](https://github.com/json-schema-org/JSON-Schema-Test-Suite) except for [1 test case](https://bowtie.report/#/implementations/python-jsonschema)...

@lucab Thank you for taking a deep look into this! The presence of registry, ctx, etc, is surely an awful hack. In this PR, the static references are resolved at...

Reworked the recursive references recently, so the memory consumption issue is solved. I'll continue to make more improvements to the compiler later on, separately.

Hi! Right now there are no feasible way to debug it from the outside :( However, I suspect that the issue could come from the “pattern” keyword when it is...

@j7an and what about "pattern"? This is the main reason why certain schemas are skipped in the test corpus in this repo. From your description, it is my main assumption...

@j7an Thanks! Yes, the number of parameters is indeed the root cause. There was a PR that made this generation work on demand, but I've closed it as there were...

Thank you for opening this issue and for so many details! Next week, I plan to collect some ideas and corner cases for `enum` / `anyOf` / `oneOf` keywords (some...

In 0.31.0, `jsonschema` now provides [context](https://docs.rs/jsonschema/latest/jsonschema/error/enum.ValidationErrorKind.html#variant.OneOfNotValid.field.context) for failed `oneOf` keywords. There is currently no context for cases where multiple subschemas match, but I'll add it in the next release (and...