llnl / llnl/conduit

Nestsets association: element / vertex

Open
#535 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
248
Forks
72
Avg merge
1d 16h
Merged PRs (30d)
15

Description

Is there a reason/use case to allow both? I feel like they both specify the same thing, and by allowing both, it increases the processing and validation code needed. For example, I would need to globally validate that all are either vertex or element to make sure things make sense.

Contributor guide

No contributing guide indexed for this repository

Research direction

The issue names no files, tests, or entry points. First clarify whether element and vertex should be mutually exclusive, then trace nestsets validation and processing to identify affected callers. Done requires an agreed behavior and corresponding validation or tests, but the current thread does not define acceptance criteria.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
data, hpc
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.