ORNL/GridKit

Bug: JSON parser infers data type from input parameter value

Ouverte

#227 ouverte le 10 sept. 2025

Voir sur GitHubGuide contributeur
 (8 commentaires) (0 réaction) (1 personne assignée)C++ (10 forks)auto 404
bughelp wanted

Métriques du dépôt

Stars
 (23 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Description how to reproduce the bug

JSON parser infers parameter data type from its value. This among other things will cause that floating point parameter is interpreted as integer if it is entered without decimal point in the input file. Retrieving this parameter as a floating point type will then cause an exception to be throw:

terminate called after throwing an instance of 'std::bad_variant_access'
Abort trap: 6

Possible solutions are

  • to interpret every parameter as the floating point number
  • have separate enums for real, integer and boolean parameters.

@superwhiskers

GridKit™ version

develop @ b191e6619bfd7d70a47e24b41fac238e3de1b9f3

System and environment details

Observed on MacOS 15.6.1 with gcc 13.3.0 and clang 16.0.6.

Additional information

No response

Guide contributeur

Direction de recherche
Reproduisez le bogue en créant un fichier JSON avec une valeur flottante sans point décimal. Ensuite, tracez le code de l'analyseur pour trouver où le type est déduit. Envisagez de mettre en œuvre l'une des solutions suggérées : traiter tous les nombres comme des flottants ou ajouter des énumérations séparées pour réel, entier et booléen.
Stack technique
cpp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75