-
Notifications
You must be signed in to change notification settings - Fork 509
Release 4.2.0 #3167
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
Draft
andrew-platt
wants to merge
459
commits into
main
Choose a base branch
from
dev
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+14,986
−6,584
Draft
Release 4.2.0 #3167
Changes from all commits
Commits
Show all changes
459 commits
Select commit
Hold shift + click to select a range
3a674ff
Merge remote-tracking branch 'OpenFAST/dev' into wavetank3
andrew-platt 7b014eb
ADI-cbind: remove comment about Re-Init
andrew-platt be17f97
Point to updated r-test
abhineet-gupta 195d8a9
Fix small bug with r-test and modify OpenFAST_io to support HubIner_T…
abhineet-gupta 16e41c6
nwtclib/version unit tests: fix CU usage from commit 6361417b3
andrew-platt cd90e0c
Registry: add basic ISO_C_BINDING types
andrew-platt 4c1301e
Registry
andrew-platt 3b5aee0
Update modules/openfast-registry/src/registry.hpp
andrew-platt c4391d4
Registry: revise the iso_c_binding logic per @deslaughter's recommend…
andrew-platt 5e851bd
Registry: typo in registry.hpp
andrew-platt a4df9c0
Merge pull request #2987 from andrew-platt/f/registry_isocbinding
andrew-platt 36695eb
Add documentation for 2-bladed hub ineria input
abhineet-gupta 76d4450
Merge remote-tracking branch 'OpenFAST/dev' into wavetank3
andrew-platt a50d0c3
ADI cbind: typo in registry
andrew-platt 465dc3a
wavetank: split out IO and types files
andrew-platt 9c25b6b
Wavetank: switch to FileInfo parsing for file read
andrew-platt db45f45
wavetank: improve error handling in python driver
andrew-platt 229f25b
wavetank: error handling in python example code
andrew-platt f8eb51e
wavetank: rearrange a bunch of code and delete some things
andrew-platt 25137ad
HD c-bind: declare added mass routine public, notes on init position
andrew-platt 4fc16a9
SS c-bind: linting (indentation not matching)
andrew-platt 25d5f64
SS c-bind: GetFluid* routines
andrew-platt 378f333
Address comments on PR #2981 for hub inertia for 2-bladed turbines
abhineet-gupta 19f91c6
SS wavefield: change WaveField_GetNodeWaveVel to WaveField_GetNodeWav…
andrew-platt c9ab843
seastate: update python lib, revise pointer passing
andrew-platt 8baa0c9
ss c-bind: more debuglevel outputs and tidying
andrew-platt cbf7b78
ss c-bind: more linting/tidying/compacting. Add GetWaveNorm routine
andrew-platt 244a7f7
ss c-bind: split out PreInit from Init
andrew-platt 63cc3bc
making sure to close file handle in turbsim reader
gbarter f348285
Correct descriptions of hubiner_teeter and Hubf2Iner
abhineet-gupta f190e1a
Merge pull request #2994 from gbarter/ofio_fix
andrew-platt cfd4f04
ss c-bind: remove density from the SeaSt_C_GetFluidVelAccDens routine
andrew-platt 41914b1
ss c-bind python: interface updates to match c-bind updates
andrew-platt 61a2ab1
ss c-bind/python: add vtk inputs to pre-init (not working yet)
andrew-platt 69ab53f
ss c-bind: add regression test
andrew-platt 2903aa3
ss c-bind: add vtk outputs
andrew-platt fff4aba
ss c-bind: add MemClear routine
andrew-platt 24af040
ss c-bind: update notes on a few routines
andrew-platt 6d510fc
ss c-bind python: change restype to None
andrew-platt 781f61d
ss c-bind: remove WaveElevSeriesFlag_C, WrWvKinMod_C from init
andrew-platt 716f5f4
ss c-bind: incorrect type, failed to compile in double
andrew-platt 4cdc47a
ss python: revert change to interface_abcy.py, add suggestions from @…
andrew-platt 9f347e7
GH: update the PR template: include AI section, r-test update
andrew-platt c69f059
gh workflow: add ss python interface test directory
andrew-platt 49fc00b
Merge pull request #2998 from andrew-platt/f/PR_template_update
andrew-platt bf1e67a
ss python: reduce precision in regression test results
andrew-platt 0e929ae
SS: set default r in GetNodeWaveNormal to 5 mm
andrew-platt ffe6283
Merge pull request #2997 from andrew-platt/f/c-bind_updates_SH
andrew-platt 7c38ef8
Added a set of generic uniform grid interpolation tools
luwang00 fa8d33c
prepping for py314, as is conda, and making requirements less restric…
gbarter b8af079
fix versioning typo on ruamel_yaml. Curren version is 0.15.80, no 0.18
gbarter 877e6c9
continuing to simplify
gbarter c5c2e20
closing beamdyn blade file
gbarter b0a3061
reverting changes
gbarter f39b9d1
Merge pull request #3001 from gbarter/ofio_fix
andrew-platt 495c588
Update r-test pointer for teeter inertia PR
andrew-platt c8e722e
Merge remote-tracking branch 'OpenFAST/dev' into hubiner_2bladed
andrew-platt e9cc4ac
Bug fix in the new grid interpolation SetIndex subroutine
luwang00 02f3f66
Adopted the new grid interpolation module in SeaState, HydroDyn, and …
luwang00 9899bc3
Merge pull request #2981 from abhineet-gupta/hubiner_2bladed
andrew-platt 5db120c
Updated the method for computing the free-surface normal
luwang00 1d0ee6c
Cleanup
luwang00 ebd04c1
Linearization: Fix pointer to dUdy block for SD inputs to MAP outputs
bjonkman 436a4ad
Merge pull request #3009 from bjonkman/b/MAP-SD_linear
andrew-platt 20dd14f
Merge dev branch
luwang00 d6e5794
fix rtd copyright year, add link to rtd when using
mayankchetan 90ae7c1
Update r-test pointer
luwang00 31474e2
Update r-test pointer
luwang00 1184401
TurbSim: add OMP directives on Coeffs2TimeSeries
andrew-platt 5f9d7c5
TurbSim: add OMP on CalcFourierCoeffs_IEC
andrew-platt 1637f4a
TurbSim: OpenMP notes and limits on nested in Coh2H
andrew-platt cd17f89
TurbSim: correction to OMP directive
andrew-platt 448f1a5
TurbSim: simplify looping in EyeCoh2H
andrew-platt c4b0026
[BugFix] FF awae: set v_plane=0 when center of wake plane leaves domain
andrew-platt 85d96f2
FF awae: add velocity kick to planes exiting domain
andrew-platt ccf66c4
FF: align initial wake grid with the blades
andrew-platt 35339ad
FF awae: used wrong mesh on previous commit
andrew-platt 1774f60
Merge pull request #3031 from andrew-platt/b/FF_initMeshMapError
andrew-platt f878b33
MD: add parameter for MD_MaxNCoef (for coefficient tables)
andrew-platt 82f485a
MD: add parameters for MD_MaxBdAtch MD_MaxPtAtch MD_MaxFailLines
andrew-platt 9ffbf21
MD: MD_MaxBdAtch and MD_MaxPtAtch to 100
andrew-platt e9e1a69
MD: add error checks on *_Add* routines
andrew-platt 3175977
Merge remote-tracking branch 'OpenFAST/dev' into wavetank3
andrew-platt 055be89
wavetank: update interfaces after merging dev changes
andrew-platt 55ada5b
ss c-bind: pass inputfilename outrootname as fixed length char array …
andrew-platt c2c4edb
wavetank: major revisions
andrew-platt ebd50b3
wavetank: add option to send all screen to file
andrew-platt d6b6979
wavetank: add 'static' linking of libs
andrew-platt 9fb9f81
wavetank: add NacYaw to input file
andrew-platt 82fb812
adi c-bind: output filenames in ShowPassed data
andrew-platt 0dd71f9
ss c-bind: revise vtk directory handling (was broken)
andrew-platt d68fa7c
wavetank: module input files read from input file instead of passed
andrew-platt c454340
wavetank: remove input files from interface WaveTank.vi
andrew-platt d2d5e8b
wavetank: return VTKdir from init
andrew-platt 4f318d9
wavetank: update WaveTank.vi example
andrew-platt 0824380
Merge pull request #3034 from andrew-platt/f/MD_increase_attach_points
andrew-platt f9c18e7
Merge pull request #3015 from mayankchetan/docFix
andrew-platt d85b5d1
wavetank: revert to standard tdms library (offshore group)
andrew-platt 64c4ede
wavetank: CalcStep routine, dbgouts file
andrew-platt 7d86049
MD: segfault if MD_End called twice from c-bind
andrew-platt b0ffb4d
wavetank: add output file from c-bind with all output channels
andrew-platt af3e21b
wavetank: add placeholder for scaling at interface
andrew-platt 608ae34
wavetank: add rough start of struct model
andrew-platt 7ff22cf
wavetank: vtk output partially assembled.
andrew-platt aea2856
wavetank: fixes to compile in double prec
andrew-platt 73eb4f9
SeaState: add ability to time shift the wavefield
andrew-platt e75ac45
wavetank: add buoy for wave elev, input for WaveTimeShift, ADI_Cbind …
andrew-platt 1359e3b
wavetank: add meshes, update SS for WaveTimeShift
andrew-platt 8783e49
wavetank: add more meshes, add Froude scaling routines (not working)
andrew-platt 6a39af1
wavetank: remove TipRad -- unused
andrew-platt 7d5ba52
wavetank: add blade root motion meshes, cleanup
andrew-platt 3bd6dba
wavetank: all calls and mappings complete. forces returned
andrew-platt e9fade8
wavetank: missing MD forces/moments
andrew-platt 6daccea
wavetank: don't transpose DCM to ADI
andrew-platt 31ad352
FASTlib: fix typo in error message
andrew-platt db517c1
Merge pull request #3064 from andrew-platt/b/ErrMsg_Issue3057
andrew-platt 1832fc9
wavetank: remove scaling option (not complete)
andrew-platt c96e3c5
Merge remote-tracking branch 'OpenFAST/dev' into wavetank3
andrew-platt 79e942e
pyOpenFAST: minor changes to interface_abc
andrew-platt 92ca19d
adi c-bind: move vtk from _Init to _PreInit
andrew-platt f2676cb
ifw c-bind: update interface mem handling, new routines
andrew-platt a55c8bc
adi c-bind: add pointer get/set FlowField
andrew-platt 4513b10
ADI c-bind: remove redundant loop on setting rotor motion
andrew-platt 82104bd
ADI c-bind: allow setting external IfW instance with pointer to data …
andrew-platt 7de30a2
adi: fix bug in previous commit
andrew-platt c650b7a
ifw c-bind: comment out IfW_C_SetFlowFieldPointer
andrew-platt 8c9c297
ifw c-bind: reneable set pointer routine
andrew-platt 9928df3
ss c-bind: add routine to get min/max elev
andrew-platt 9126941
HD: bug fixes for Morison members
luwang00 dd1d56a
Fixed a typo in HD user docs
luwang00 6f4ac93
Merge pull request #3076 from luwang00/b/HDFlippedMember
andrew-platt ea0e9d3
Merge remote-tracking branch 'origin/wavetank3' into f/c-bind_updates…
andrew-platt e275c5e
ADI c-bind: bug introduced in merge
andrew-platt ea016c5
nwtc_c_bind: specify integer type on errstat
andrew-platt 33f1c4d
ss c-bind: initialize returned vals from init (in case of early abort)
andrew-platt 91a0110
SS c-bind: add routines to get env vars
andrew-platt 054d478
HD c-bind: correct error message about multiple nodes
andrew-platt b759d11
Update r-test pointer after r-test merge #168
andrew-platt 0f4a7ee
wavetank: fix type issue so double prec compiles
andrew-platt d527a23
Merge remote-tracking branch 'origin/wavetank3' into f/c-bind_updates…
andrew-platt cc473e8
TempQuickFix: disable wavetank since it won't compile
andrew-platt 5291ed3
Remove the _OPENMP ifdefs and let OpenMP set the number of threads. U…
deslaughter 5858941
Change main CMakeLists.txt file to prefer sequential version of MKL w…
deslaughter 667208f
Merge remote-tracking branch 'OpenFAST/dev' into f/GridInterp
andrew-platt e204943
docs: update info on VS versions and IFX
andrew-platt 3831599
GridInterp: update VS projects
andrew-platt 3d19110
Merge pull request #3083 from andrew-platt/f/Docs_WinVS_install
andrew-platt 7ce0149
Update GridInterp based on review comments
luwang00 1f74491
Merge pull request #3007 from luwang00/f/GridInterp
andrew-platt 1d2580a
ifw flowfield: add box corners to out of bounds message on 4D
andrew-platt 1850c1a
Correction to the index math for box upper bound
andrew-platt 5aa1e87
Merge pull request #3099 from andrew-platt/f/FF_HR_bounds_box_corners
andrew-platt d905352
FF WAT: change default WAT_Scalebox to True
andrew-platt 24556d9
FF WAT: update docs on WAT_Scalebox
andrew-platt 14ad361
Merge pull request #3020 from andrew-platt/f/TurbSim_OpenMP
andrew-platt 83ee12d
Docs: updates to StC docs
andrew-platt d04e201
Docs: correction to previous commits
andrew-platt 7664528
SeaSt c-bind: replace numTimeSteps with time_max
andrew-platt 30eb71d
Merge remote-tracking branch 'OpenFAST/dev' into wavetank3
andrew-platt b706b8e
ED: comment in registry was never updated
andrew-platt 7935e21
SeaSt_WaveField: remove unused params from registry
andrew-platt 92c80cc
SeaSt c-bind: fix WaveTimeShift after `dev` merge
andrew-platt d0fb325
cmake: objects for AD, MD, SS c-bind shared/static lib builds
andrew-platt 224e772
seast c-bind: fix issue in reg test
andrew-platt ed1eb8d
HD: improve error handling in CalcOutput
andrew-platt 9ca10b3
Add regression test case for MHK RM1 scaled tank size
andrew-platt 117eb52
Merge pull request #3109 from andrew-platt/StC_theory_eqs
andrew-platt d741577
Add error message when using BeamDyn with AeroMap
deslaughter 4922d57
Check that FAST_InitializeAll is successful in FAST_InitializeSteadyS…
deslaughter f90d379
Merge pull request #3111 from deslaughter/aeromap_bd_err
deslaughter 245e0c5
move wavetank example python to regression
andrew-platt 0bf37cf
functional wavetank test case
andrew-platt 9b6f6dd
wavetank: fix debug level issue
andrew-platt 3a79346
wavetank: updates for GH actions
andrew-platt f305048
wavetank regression: rearrange for GH action
andrew-platt 44171ae
wavetank regression: rearrange again for GH actions
andrew-platt 2d3cd87
FF: WAT_dXdYdZ from high res grid if same for all turbines
andrew-platt fb7fec7
FF WAT_DxDyDz: fix typo in 2d3cd877e
andrew-platt 6656ae9
FF: add WAT info to summary file, fix segfault
andrew-platt 7cd603a
FF WAT: fix seg fault in summary file
andrew-platt e731411
FF WAT: update first set of reg test results
andrew-platt afec862
FF WAT: update second set of reg test results
andrew-platt 071fb06
typo found by ai review
andrew-platt d02ee8c
Merge remote-tracking branch 'OpenFAST/dev' into b/FF_WAT_scalebox
andrew-platt 7f5b3a1
Merge pull request #3104 from andrew-platt/b/FF_WAT_scalebox
andrew-platt 12be0fb
Merge remote-tracking branch 'OpenFAST/dev' into wavetank3
andrew-platt 01ad8b5
wavetank: add brief documentation
andrew-platt 380a6e9
Merge remote-tracking branch 'origin/wavetank3' into f/c-bind_updates…
andrew-platt 7740c88
Merge pull request #3065 from andrew-platt/wavetank3
andrew-platt 2c2a46e
Merge remote-tracking branch 'OpenFAST/dev' into f/c-bind_updates_ADI
andrew-platt 00cf2a5
fix when elastodyn tower file path is absolute
4518300
update wavetank interface after merge
andrew-platt a709863
Initial plan
Copilot e345faf
Fix get_elevMinMax function signature and docstring
Copilot 5945d8f
Merge pull request #28 from andrew-platt/copilot/sub-pr-27
andrew-platt c3caf84
update py_SS reg test after merge
andrew-platt d4627d5
HD c-bind: add clarifying comment after co-pilot confusion
andrew-platt 676f495
ADI c-bind: remove stale comment
andrew-platt 40d4f91
SS c-bind: correct name of subroutine
andrew-platt b0ef954
ADI logic update
andrew-platt 9463e62
ADI: fix bug in commit b0ef95402
andrew-platt 883c72e
Merge pull request #3108 from andrew-platt/f/c-bind_updates_ADI
andrew-platt 7a980f5
Merge pull request #3130 from louis-papillon/bugfix_elastodyn_tower_a…
andrew-platt 1a15d0a
AeroAcoustics: fix element size calculation
bjonkman 4f98b47
AeroAcoustics BUG FIX
bjonkman ecb52cc
Merge pull request #3134 from bjonkman/b/AA_angle
andrew-platt c77c55c
HD: Fixed a bug where only the potential-flow wave excitation of the …
luwang00 6a26078
FF: update AWAE plane velocity outside bounds
andrew-platt 502b51c
Merge pull request #3141 from luwang00/b/HD_PtfmRefztRot
andrew-platt 102b757
Merge remote-tracking branch 'OpenFAST/dev' into b/FF_planeDomainExit
andrew-platt cd38609
AWAE: include z component in wake drift out of bounds
andrew-platt df2bb8f
AWAE: Update comment
andrew-platt 36c8acf
Merge pull request #3025 from andrew-platt/b/FF_planeDomainExit
andrew-platt 1364d80
docs: Remove references to `pfunit` from docs (no longer used)
andrew-platt 0fb1548
WD: Ct_avg was not set when polar wake used giving RtCtAvgT
andrew-platt 34cb3ee
FF reg test: Update ModAmb3 case after RtCtAvgT bugfix
andrew-platt 72176dd
Merge pull request #3144 from andrew-platt/b/docs_pfunit_references
andrew-platt 5e30ef1
Merge pull request #3145 from andrew-platt/b/FF_RtCtAvgT
andrew-platt 6a57521
Reenable use of MAP when writing checkpoint files
deslaughter b02534c
Merge pull request #3146 from deslaughter/reenable-map-chkpt
andrew-platt 391d428
NWTC Library: update error message with IOSTAT return value
bjonkman d2be8f1
Merge pull request #3148 from bjonkman/b/NWTCLib_errMsg
andrew-platt 693c107
AeroAcoustics cleanup + added driver code
bjonkman 4d0b696
Update AA reg test results
andrew-platt 0c90a24
Merge remote-tracking branch 'OpenFAST/dev' into b/AA_elementSize
andrew-platt f9f1986
Merge pull request #3133 from bjonkman/b/AA_elementSize
andrew-platt 2480d90
Merge remote-tracking branch 'OpenFAST/dev' into b/AeroAcoustics_cleanup
andrew-platt 64d0e14
fixes access to the 2nd column (GenTrq_TLU) in the DLL_NumTrq related…
LKa505 f6fe2c5
Linear Trim Solution: update range calculation when NLinTimes=1
bjonkman 82276ef
Merge pull request #3156 from bjonkman/b/Linear
deslaughter 4768172
StC: bugfix in DLL applied forces
andrew-platt 4fac4c3
Merge pull request #3153 from LKa505/b/FAST_reader_GenTrq_TLU
andrew-platt 63a91ef
StC: add placeholders for moments applied by DLL
andrew-platt 88fce61
StC: add moments for DLL prescribed forces/moments
andrew-platt 7795154
StC: address comments from GH copilot and minor format fixes
andrew-platt aa2fd1a
Merge pull request #3158 from andrew-platt/b/StC_Ctrl_Frc
andrew-platt ea60d47
Add in yaw friction cases
andrew-platt 58a1254
testing: add MinimalExample case
andrew-platt c8f4707
Try to get simulink to build
deslaughter ad3db13
Try ifx compilter from intel-fortran-essentials
deslaughter 7a5db9d
Merge pull request #3159 from andrew-platt/b/YFriction_regTests
deslaughter 69cab72
Merge pull request #3162 from deslaughter/fix-simulink-build-win
andrew-platt cd53c27
Merge pull request #3149 from bjonkman/b/AeroAcoustics_cleanup
andrew-platt f18d025
AA driver: add driver compile to cmake
andrew-platt e1ab2fb
AA driver: bug in previous commit
andrew-platt 4ba5c85
AA driver: update simulink build with aeroacousticslib
andrew-platt c44d3f4
Merge pull request #3163 from andrew-platt/f/aa_driver_cmake
andrew-platt 68e3826
Update r-test pointer after r-test PR#175
andrew-platt 7ff0bc8
project description update NREL --> NLR
andrew-platt d07739b
Merge pull request #3166 from andrew-platt/b/nrel_name_change
andrew-platt File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.