Skip to content

Fix Index Error in LASAM #19

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.

Sign up for GitHub

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

Closed
mkarim-rtx wants to merge 3 commits into from
Closed

Fix Index Error in LASAM #19

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e0072fb
Define LOG in submodule instead of relying on generic definition in e…
cmaynard-ngwpc May 12, 2026
7db775c
Fixed wrong variable name to read max soil type and updated default m…
mkarim-rtx May 12, 2026
661f8ba
riverted renaming max_valid_soil_types as it will be taken care by msmw
mkarim-rtx May 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 7 additions & 3 deletions include/Logger.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,17 @@
#ifndef LASAM_LOGGER_HPP
#define LASAM_LOGGER_HPP

#include "ewts/logger.hpp"
#include "ewts/module_constants.hpp"
#include "ewts/logger.hpp"
#include "ewts/log_levels.hpp"

#define LOG(...) ::ewts::GetLogger(::ewts::modules::EWTS_ID_LASAM).Log(__VA_ARGS__)
#define GetLogLevel() ::ewts::GetLogger(::ewts::modules::EWTS_ID_LASAM).GetLogLevel()
#define IsLoggingEnabled() ::ewts::GetLogger(::ewts::modules::EWTS_ID_LASAM).IsLoggingEnabled()

using ewts::EwtsInit;
using ewts::LogLevel;

// Provide the constant in the global namespace:
inline constexpr const char* EWTS_ID_LASAM = ewts::modules::EWTS_ID_LASAM;
inline constexpr const char* LASAM_MODULE_ID = ewts::modules::EWTS_ID_LASAM;

#endif /* LASAM_LOGGER_HPP */
2 changes: 1 addition & 1 deletion include/all.hxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ extern string verbosity;
#define use_bmi_flag FALSE // TODO set to TRUE to run in BMI environment

#define MAX_NUM_SOIL_LAYERS 4
#define MAX_NUM_SOIL_TYPES 15
#define MAX_NUM_SOIL_TYPES 16
#define MAX_SOIL_NAME_CHARS 25
#define MAX_NUM_WETTING_FRONTS 300

Expand Down
4 changes: 2 additions & 2 deletions src/bmi_lgar.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,9 @@ Initialize (std::string config_file)
{
// Initialize the Error, Warning and Trapping System
#ifdef EWTS_HAVE_NGEN_BRIDGE
EwtsInit(EWTS_ID_LASAM, true);
EwtsInit(LASAM_MODULE_ID, true);
#else
EwtsInit(EWTS_ID_LASAM, false);
EwtsInit(LASAM_MODULE_ID, false);
#endif

LOG("Inside BmiLGAR::Initialize \n", LogLevel::INFO);
Expand Down