llnl / llnl/SEAMsurrogates

Move simulate_data functions to new module

Open
#6 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
6
Forks
2
Avg merge
20m
Merged PRs (30d)
2

Description

The function `simulate_data` in the gaussian_process_regression module should be moved to a new module since data from the test functions can be used with other models, like neural networks.

There is an almost duplicate function in the sensitivity_analysis module as well. The two `simulate_data` functions can likely be merged.

Contributor guide

No contributing guide indexed for this repository

Research direction

Read the `simulate_data` implementations in the gaussian_process_regression and sensitivity_analysis modules and compare their inputs, outputs, and differences. Identify the shared module boundary and verify that both existing callers can use the merged function while preserving current behavior; run the project’s tests afterward.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
machine-learning
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.