-
Notifications
You must be signed in to change notification settings - Fork 25
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
ogma-core
: Make DB2Copilot backend to use JSON parser
#122
Labels
CR:Status:Closed
Admin only: Change request that has been completed
CR:Type:Management
Admin only: Change request for conformance with policies or procedures
Milestone
Comments
ivanperez-keera
added
CR:Status:Initiated
Admin only: Change request that has been initiated
CR:Type:Management
Admin only: Change request for conformance with policies or procedures
labels
Jan 24, 2024
Change Manager: Confirmed that the issue exists. |
ivanperez-keera
added
CR:Status:Confirmed
Admin only: Change request that has been acknowledged by the change manager
and removed
CR:Status:Initiated
Admin only: Change request that has been initiated
labels
Jan 24, 2024
Technical Lead: Confirmed that the issue should be addressed. |
ivanperez-keera
added
CR:Status:Accepted
Admin only: Change request accepted by technical lead
and removed
CR:Status:Confirmed
Admin only: Change request that has been acknowledged by the change manager
labels
Jan 24, 2024
Technical Lead: Issue scheduled for fixing in Ogma 1.3. Fix assigned to: @ivanperez-keera. |
ivanperez-keera
added
CR:Status:Scheduled
Admin only: Change requested scheduled
and removed
CR:Status:Accepted
Admin only: Change request accepted by technical lead
labels
Jan 24, 2024
ivanperez-keera
added
CR:Status:Implementation
Admin only: Change request that is currently being implemented
and removed
CR:Status:Scheduled
Admin only: Change requested scheduled
labels
Jan 24, 2024
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
…. Refs #122. Not all kinds of input files that use the JSON parsing library will provide all information. In some, information is implicit or simply not available, yet we should be able to deal with such files. This commit modifies the JSON spec parsing library so that some fields can be optional. In general, we assume we'll still have the essential fields (requirements, IDs, specs), but not all auxiliary information.
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
A prior commit has modified the JSON Spec parsing library to make some fields in input JSON files optional. This will make the library more versatile. This commit modifies the existing modules that use the JSON parsing library to use the new interface with optional types for some JSON Format spec fields.
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
The JSON parsing functions currently used by ogma-core to parse requirement DBs are very specific to the formats supported. This commit replaces the existing FRET DB parser and converter by one that is based on the new generic JSON-based parsing library. The old module implementing the adhoc converter, as well as the dependency on the ad hoc parser, are removed.
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
The library ogma-language-fret-reqs is no longer needed, since its functions have been generalized in other libraries. Therefore, this library can be completely removed.
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
The JSON parsing functions currently used by ogma-core to parse requirement DBs are very specific to the formats supported. This commit replaces the existing FRET DB parser and converter by one that is based on the new generic JSON-based parsing library. The old module implementing the adhoc converter, as well as the dependency on the ad hoc parser, are removed.
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
The library ogma-language-fret-reqs is no longer needed, since its functions have been generalized in other libraries. Therefore, this library can be completely removed.
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
The JSON parsing functions currently used by ogma-core to parse requirement DBs are very specific to the formats supported. This commit replaces the existing FRET DB parser and converter by one that is based on the new generic JSON-based parsing library. The old module implementing the adhoc converter, as well as the dependency on the ad hoc parser, are removed.
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
The library ogma-language-fret-reqs is no longer needed, since its functions have been generalized in other libraries. Therefore, this library can be completely removed.
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
Implementor: Solution implemented, review requested. |
ivanperez-keera
added
CR:Status:Verification
Admin only: Change request that is currently being verified
and removed
CR:Status:Implementation
Admin only: Change request that is currently being implemented
labels
Jan 24, 2024
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
A prior commit has modified the JSON Spec parsing library to make some fields in input JSON files optional. This will make the library more versatile. This commit modifies the existing modules that use the JSON parsing library to use the new interface with optional types for some JSON Format spec fields. The extensional behavior of these modules does not change, compared to their behavior prior to the modification of the JSON Format Spec definition to make some fields optional.
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
The JSON parsing functions currently used by ogma-core to parse requirement DBs are very specific to the formats supported. This commit replaces the existing FRET DB parser and converter by one that is based on the new generic JSON-based parsing library. The old module implementing the ad hoc converter, as well as the dependency on the ad hoc parser, are removed.
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
The library ogma-language-fret-reqs is no longer needed, since its functions have been generalized in other libraries. Therefore, this library can be completely removed.
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
ivanperez-keera
added a commit
that referenced
this issue
Jan 24, 2024
Change Manager: Verified that:
|
Change Manager: Implementation ready to be merged. |
ivanperez-keera
added
CR:Status:Closed
Admin only: Change request that has been completed
and removed
CR:Status:Verification
Admin only: Change request that is currently being verified
labels
Jan 25, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
CR:Status:Closed
Admin only: Change request that has been completed
CR:Type:Management
Admin only: Change request for conformance with policies or procedures
Description
The requirements-DB-2-Copilot backend uses an adhoc parser, instead of using the generalized JSON parser. Using the latter would simplify the code, as well as make it more versatile.
Type
Additional context
None.
Requester
Method to check presence of bug
Not applicable (not a bug).
Expected result
The specific library that deals with parsing requirement DBs is removed, and we instead rely on the generic JSON parser.
Desired result
The specific library that deals with parsing requirement DBs is removed, and we instead rely on the generic JSON parser.
Proposed solution
Express the logic of parsing the requirements DB file using the JSON parser. Remove the dedicated library for parsing requirement databases, as well as the file in
Language/Trans
.Further notes
None.
The text was updated successfully, but these errors were encountered: