-
Notifications
You must be signed in to change notification settings - Fork 0
Cpt conditioning continued #5
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
Open
bramvandeneijnden
wants to merge
63
commits into
main
Choose a base branch
from
CPT_conditioning_continued
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.
Open
Changes from all commits
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
8015988
added first version of conditioned random field generator
bramvandeneijnden dfe8826
corrections, clean-up, efficiency and commenting
bramvandeneijnden 7e2ddfa
Added tutorial:
bramvandeneijnden b9324d8
Extension of tutorial to 3D conditioned random fields
bramvandeneijnden 6b54e89
added sklearn to the requirements
bramvandeneijnden 78643d3
deleted: random_fields/generate_conditioned_field.py
bramvandeneijnden 5fadc51
modified: setup.cfg
bramvandeneijnden 04170dd
modified: setup.cfg (sklearn --> scikit-learn)
bramvandeneijnden 3b9df07
Update docs/installation.rst
bramvandeneijnden 72e157b
Update random_fields/generate_field.py
bramvandeneijnden 0712869
Changes for merge `Conditional random fields`
bramvandeneijnden b60343d
changes accounting for review comments:
bramvandeneijnden f76e0f0
modified: requirements.txt
bramvandeneijnden fadddba
Introduction of geostatistical interpretation of CPTs
bramvandeneijnden ecfd3e4
TODO: update dependencies...
bramvandeneijnden 837f49b
added function to export conditioning data for visualisation
bramvandeneijnden 0ccabe9
modified: random_fields/geostatistical_cpt_interpretation.py
bramvandeneijnden 29e0d37
Several minor changes and fixes
bramvandeneijnden ae546bb
added dependencies scikit-learn and d-geolib-plus
bramvandeneijnden 5bc2477
minor fix
bramvandeneijnden 59eebfc
set numpy version from ==1.24 to >= 1.24
bramvandeneijnden 5a1b431
updated docstrings and cleanup
bramvandeneijnden 13f246a
scipy requirements weakened
bramvandeneijnden be3acd2
Update geostatistical_cpt_interpretation.py
bramvandeneijnden d1a77a0
updated authors.rst
brunozc ec7bc72
Correct GitHub username casing in documentation configuration
brunozc bb76a5c
Update dependency versions in requirements and setup configuration
brunozc 6ac10e0
Add configuration files for flake8, mypy, and pre-commit; update CPTt…
brunozc 72105fc
updated documentation
brunozc 860e7ee
style fixed
brunozc 98f6940
WIP
brunozc 21da906
Sort file list in CPT directory and update default seed value for geo…
brunozc 47d5c2b
Update dependencies and enhance documentation for CPT-based random fi…
brunozc da38c12
Fix package naming for d-geolib-plus in requirements files
brunozc 93dd70c
Add unit tests for conditional 2D random field generation using GEF a…
brunozc cc45031
Downgrade scipy version to 1.13.1 in requirements files
brunozc f19560f
Refactor code for improved readability and maintainability in random …
brunozc c94a6d3
fixed test for linux
brunozc ea2eb6a
Fix file path for 3D random field test on Windows and remove skip marker
brunozc 6d04569
Add Read the Docs configuration file for documentation build
brunozc 6439e69
Enhance plotting functions to support conditional points and customiz…
brunozc 268d135
fixed style
brunozc e224365
Update 3D random field test to use pickle format and change output fi…
brunozc 290abe0
Rename "STEM team" to "Random Fields team" in documentation and corre…
brunozc 60b7837
Update tutorial to reflect 2D/3D conditioned Random Field on CPT data
brunozc ecc4d98
Update tutorial heading to use "and" for clarity in 2D and 3D conditi…
brunozc 2833dbc
fixed bug in test
brunozc c4a686f
Update output file format in 3D random field test from PNG to EPS
brunozc 50b54ca
Add tests for ElasticityFieldsFromCpt and RandomFieldProperties
brunozc 963cf01
Refactor assertions in 3D conditioned random field test to use np.tes…
brunozc e7247be
Increase precision of assertions in 3D conditioned random field test …
brunozc caa7cd6
Update GitHub Actions workflow to allow manual triggering and broaden…
brunozc 8d5e1d3
Fix formatting in tutorial_4.rst by adding missing newlines for bette…
brunozc 69fe1a3
Improve mathematical notation in tutorial_4.rst for clarity and consi…
brunozc 1f6152a
Refactor ElasticityFieldsFromCpt class to improve variable naming and…
brunozc 580569b
Update docs/tutorial_4.rst
brunozc 1753666
Update docs/conf.py
brunozc 829600e
Update docs/tutorial_4.rst
brunozc 1461715
Update random_fields/generate_field.py
brunozc cc0faad
Update random_fields/generate_field.py
brunozc 9d1dc3d
Enhance calibration method to accept three boolean indices and add va…
brunozc 83cbce7
Merge branch 'main' into CPT_conditioning_continued
brunozc 33b2815
update: refine dependencies and remove redundant entries in setup.cfg
brunozc 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| [flake8] | ||
| ignore = F403, F405, W503, W504 | ||
| max-line-length = 120 | ||
| per-file-ignores = __init__.py:F401 |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,4 +6,5 @@ pytest_cache/* | |
| build/* | ||
| random_fields.egg-info/* | ||
| .coverage | ||
| .tox/* | ||
| .tox/* | ||
| docs/build/* | ||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| repos: | ||
| - repo: local | ||
| hooks: | ||
| - id: yapf | ||
| name: yapf | ||
| language: python | ||
| entry: yapf | ||
| args: [-i] | ||
| types: [python] | ||
| - id: mypy | ||
| name: mypy | ||
| language: python | ||
| entry: mypy | ||
| files: "^random_fields/" | ||
| args: [ --config-file, "pyproject.toml" ] | ||
| types: [python] | ||
| - id: flake8 | ||
| name: flake8 | ||
| language: python | ||
| entry: flake8 | ||
| files: "^random_fields/" |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| # Read the Docs configuration file | ||
| # See https://docs.readthedocs.io/en/stable/config-file/v2.html for details | ||
|
|
||
| # Required | ||
| version: 2 | ||
|
|
||
| # Set the OS, Python version, and other tools you might need | ||
| build: | ||
| os: ubuntu-22.04 | ||
| tools: | ||
| python: "3.11" | ||
| jobs: | ||
| pre_build: | ||
| - sphinx-apidoc -o docs ./random_fields | ||
| - pip install . | ||
|
|
||
| # Build documentation in the "docs/" directory with Sphinx | ||
| sphinx: | ||
| configuration: docs/conf.py | ||
|
|
||
| # Optionally, but recommended, | ||
| # declare the Python requirements required to build your documentation | ||
| # See https://docs.readthedocs.io/en/stable/guides/reproducible-builds.html | ||
| python: | ||
| install: | ||
| - requirements: docs/requirements.txt | ||
|
|
||
|
|
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,12 @@ | ||
| [style] | ||
| based_on_style = pep8 | ||
| SPLIT_ALL_TOP_LEVEL_COMMA_SEPARATED_VALUES = False | ||
| dedent_closing_brackets = False | ||
| EACH_DICT_ENTRY_ON_SEPARATE_LINE = True | ||
| FORCE_MULTILINE_DICT = False | ||
| coalesce_brackets = True | ||
| column_limit = 120 | ||
| split_arguments_when_comma_terminated = True | ||
| SPLIT_BEFORE_FIRST_ARGUMENT = False | ||
| #SPLIT_PENALTY_FOR_ADDED_LINE_SPLIT = 1 | ||
| ALLOW_SPLIT_BEFORE_DICT_VALUE = False |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| # Minimal makefile for Sphinx documentation | ||
| # | ||
|
|
||
| # You can set these variables from the command line, and also | ||
| # from the environment for the first two. | ||
| SPHINXOPTS ?= | ||
| SPHINXBUILD ?= sphinx-build | ||
| SOURCEDIR = . | ||
| BUILDDIR = _build | ||
|
|
||
| # Put it first so that "make" without argument is like "make help". | ||
| help: | ||
| @$(SPHINXBUILD) -M help "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O) | ||
|
|
||
| .PHONY: help Makefile | ||
|
|
||
| # Catch-all target: route all unknown targets to Sphinx using the new | ||
| # "make mode" option. $(O) is meant as a shortcut for $(SPHINXOPTS). | ||
| %: Makefile | ||
| @$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O) |
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
File renamed without changes
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
File renamed without changes
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| .. _authors: | ||
|
|
||
| Random Fields team | ||
| ================== | ||
|
|
||
| Maintainers | ||
| ----------- | ||
| - Bruno Zuada Coelho <bruno.zuadacoelho@deltares.nl> `@brunozc <https://github.com/brunozc>`_ | ||
| - Aron Noordam <aron.noordam@deltares.nl> `@aronnoordam <https://github.com/aronnoordam>`_ | ||
|
|
||
| Developers | ||
| ---------- | ||
| - Bram van den Eijnden <bram.vandneeijnden@deltares.nl> `@bramvandeneijnden <https://github.com/bramvandeneijnden>`_ | ||
| - Davide Moretti <davide.moretti@tno.nl> `@morettid <https://github.com/morettid>`_ |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,60 @@ | ||
| # Configuration file for the Sphinx documentation builder. | ||
| # | ||
| # For the full list of built-in configuration values, see the documentation: | ||
| # https://www.sphinx-doc.org/en/master/usage/configuration.html | ||
|
|
||
| # -- Project information ----------------------------------------------------- | ||
| # https://www.sphinx-doc.org/en/master/usage/configuration.html#project-information | ||
|
|
||
| import random_fields | ||
|
|
||
| project = 'RandomFields' | ||
| copyright = '2024, STEM team' | ||
| author = 'STEM team' | ||
| version = random_fields.__version__ | ||
|
|
||
| # -- General configuration --------------------------------------------------- | ||
| # https://www.sphinx-doc.org/en/master/usage/configuration.html#general-configuration | ||
|
|
||
| extensions = ['sphinx.ext.autodoc', 'sphinx_rtd_theme', 'sphinx.ext.intersphinx'] | ||
|
|
||
| templates_path = ['_templates'] | ||
| exclude_patterns = ['_build', 'Thumbs.db', '.DS_Store'] | ||
|
|
||
| # to add __init__ documentation to the build | ||
| autoclass_content = 'both' | ||
|
|
||
| # -- Options for HTML output ------------------------------------------------- | ||
| # https://www.sphinx-doc.org/en/master/usage/configuration.html#options-for-html-output | ||
|
|
||
| html_theme = 'sphinx_rtd_theme' | ||
| html_static_path = ['_static'] | ||
|
|
||
| # # Configure the RTD theme options | ||
| html_theme_options = { | ||
| 'logo_only': False, | ||
| 'display_version': True, | ||
| 'prev_next_buttons_location': 'bottom', | ||
| 'style_external_links': True, # Toc options | ||
| 'collapse_navigation': False, | ||
| 'sticky_navigation': True, | ||
| 'navigation_depth': 2, | ||
| 'includehidden': True, | ||
| 'titles_only': True, | ||
| } | ||
|
|
||
| # If true, "Created using Sphinx" is shown in the HTML footer. Default is True. | ||
| html_show_sphinx = False | ||
| # If true, "(C) Copyright ..." is shown in the HTML footer. Default is True. | ||
| html_show_copyright = True | ||
| # If true shows the source link to the rst code | ||
| html_show_sourcelink = False | ||
|
|
||
| # Link to the github | ||
| html_context = { | ||
| "display_github": True, | ||
| "github_user": "StemVibrations", | ||
| "github_repo": "RandomFields", | ||
| "github_version": "main", | ||
| "conf_py_path": "/docs/", | ||
| } | ||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,111 @@ | ||
| Code contribution | ||
| ================= | ||
|
|
||
| Steps for submitting your code | ||
| ------------------------------ | ||
|
|
||
| When contributing code follow this checklist: | ||
|
|
||
| #. Fork the repository on GitHub. | ||
| #. Create an issue with the desired feature or bug fix. | ||
| #. Make your modifications or additions in a feature branch. | ||
| #. Make changes and commit your changes using a descriptive commit message. | ||
| #. Provide tests for your changes, and ensure they all pass. | ||
| #. Provide documentation for your changes, in accordance with the style of the rest of the project (see :ref:`style_guide`). | ||
| #. Create a pull request to RandomFields main branch. The STEM team will review and discuss your Pull Request with you. | ||
|
|
||
| For any questions, please get in contact with one of the members of :doc:`authors`. | ||
|
|
||
|
|
||
| .. _style_guide: | ||
|
|
||
| Code style guide | ||
| ---------------- | ||
| The additional features should follow the style of the RandomFields project. | ||
|
|
||
| We follow the PEP 8 style guide for Python code, with our custom modifications as defined in the | ||
| `Yapf file <../../.style.yapf>`_ and the `flake8 file <../../.flake8>`_. These files can be ran manually by using the | ||
| following command from the root directory of the project: | ||
|
|
||
| .. code-block:: | ||
|
|
||
| pre-commit run --all-files | ||
|
|
||
|
|
||
| The class or function name should be clear and descriptive of the functionality it provides. | ||
|
|
||
| There should be a docstring at the beginning of the class or function describing its purpose and usage. | ||
| The docstring should be in the form of a triple-quoted string. | ||
|
|
||
| The class or function must have a type annotation. | ||
| The class should specify the attributes and inheritance. | ||
| The function should specify the arguments, exceptions and returns (in this order). | ||
| The return type annotation should be in the form of a comment after the closing parenthesis of the arguments. | ||
|
|
||
| Please, avoid inheritance, and favour composition when writing your code. | ||
|
|
||
| An example of a class: | ||
|
|
||
| .. code-block:: | ||
|
|
||
| class ClassName(object): | ||
| """ | ||
| Work in progress: docstring style to be specified | ||
|
|
||
| Inheritance: | ||
| -:class:`object` | ||
|
|
||
| Attributes: | ||
| - <attribute_name_1> (<attribute type>): <attribute description>,shape (n,m) | ||
| - <attribute_name_2> (<attribute type>): <attribute description>,shape (m,) | ||
|
|
||
|
|
||
| """ | ||
| def __init__(self,<args>,...): | ||
| """ | ||
| Constructor of the RandomFields class | ||
|
|
||
| Args: | ||
| - <arg>: (<type>) <description>, shape (n,m,), default | ||
| """ | ||
| self.attribute_name_1: <attribute_type> = <value> | ||
| self.attribute_name_2: <attribute_type> = <value> | ||
|
|
||
|
|
||
| An example of a function: | ||
|
|
||
| .. code-block:: | ||
|
|
||
| def generate(self, nodes: npt.NDArray[np.float64]) -> None: | ||
| """ | ||
| Generate random field | ||
|
|
||
| Args: | ||
| - nodes (ndarray): The nodes of the random field. shape (:,`self.n_dim`) | ||
|
|
||
| Raises: | ||
| - ValueError: if dimensions of `nodes` do not match dimensions of the model | ||
|
|
||
| Returns: | ||
|
|
||
| """ | ||
|
|
||
| # check dimensions of nodes agrees with dimensions of model | ||
| if nodes.shape[1] != self.n_dim: | ||
| raise ValueError(f'Dimensions of nodes: {nodes.shape[1]} do not match dimensions of model: {self.n_dim}') | ||
|
|
||
| # scale of fluctuation | ||
| scale_fluctuation = np.ones(self.n_dim) * self.vertical_scale_fluctuation | ||
|
|
||
| # apply the anisotropy to the other dimensions | ||
| mask = np.arange(len(scale_fluctuation)) != self.v_dim | ||
| scale_fluctuation[mask] = scale_fluctuation[mask] * self.anisotropy | ||
|
|
||
| model = self.random_field_model(dim = self.n_dim, | ||
| var = self.variance, | ||
| len_scale = scale_fluctuation, | ||
| angles = self.angle) | ||
| self.random_field = gs.SRF(model, | ||
| mean = self.mean, | ||
| seed = self.seed) | ||
| self.random_field(nodes.T) |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,60 @@ | ||
| RandomFields | ||
| ==== | ||
| Python package to generate random fields in 2D or 3D on unstructured grids. | ||
|
|
||
|
|
||
| User guide | ||
| ========== | ||
|
|
||
| Background information | ||
| ...................... | ||
| The random fields are generated using `gstools <https://geostat-framework.readthedocs.io/en/latest/>`_. | ||
|
|
||
| Currently, the following covariance models are available: | ||
|
|
||
| * Gaussian | ||
| * Exponential | ||
| * Matern | ||
| * Linear (not for conditioned random fields) | ||
|
|
||
| Additional functionalities are used to generate conditioned random fields. | ||
|
|
||
|
|
||
| Tutorials | ||
| ......... | ||
|
|
||
| * :ref:`tutorial1` | ||
| 1D unconditioned and conditioned random field generation. | ||
|
|
||
| * :ref:`tutorial2` | ||
| 2D unconditioned and conditioned random field simulation. | ||
|
|
||
| * :ref:`tutorial3` | ||
| 3D unconditioned and conditioned random field simulation. | ||
|
|
||
| * :ref:`tutorial4` | ||
| Non-normal Gaussian fields and conditioning based on CPT data. | ||
|
|
||
|
|
||
| How to contribute | ||
| ================= | ||
| If you want to contribute to RandomFields please follow the steps defined in :doc:`contributions`. | ||
|
|
||
|
|
||
| Random Fields team | ||
| ================== | ||
| The Random Field package is part of the STEM is a research programme that results from a | ||
| collaboration between the following partners: | ||
|
|
||
| * `ProRail <https://www.prorail.nl>`_ | ||
| * `Deltares <https://www.deltares.nl>`_ | ||
| * `TNO <https://www.tno.nl>`_ | ||
| * `TU Delft <https://www.tudelft.nl>`_ | ||
|
|
||
| The :doc:`authors`. | ||
|
|
||
|
|
||
| Package documentation | ||
| ===================== | ||
|
|
||
| The :doc:`random_fields` documentation. |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| RandomFields installation | ||
| ================= | ||
|
|
||
| .. _python_random_fields: | ||
|
|
||
| RandomFields Python package | ||
| ------------------- | ||
|
|
||
|
|
||
| To be copied from STEM |
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.