[NetAppFiles] Anf 13322 net app files api version 2021 06 01

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[x] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@audunn, there are a couple of breaking changes detected in the API version, could you follow the breaking change review process to get them approved?

@audunn, there are a couple of breaking changes detected in the API version, could you follow the breaking change review process to get them approved?

@lirenhe I beloved at lest some of them to be false positives. For example DefaultValueChanged on Pool "serviceLevel" is that I saw it was the same enum used several times in the spec and changed to use one common.  The default has indeed not changed see below the first is the 2021-06-01 and then 2021-02-01 both have default of  "default": "Premium"
        "serviceLevel": {
          "title": "serviceLevel",
          "type": "string",
          "$ref": "#/definitions/serviceLevel",
          "example": "Ultra",
          "default": "Premium"
        },

        "serviceLevel": {
          "title": "serviceLevel",
          "type": "string",
          "description": "The service level of the file system",
          "enum": [
            "Standard",
            "Premium",
            "Ultra"
          ],
          "x-ms-enum": {
            "name": "ServiceLevel",
            "modelAsString": true,
            "values": [
              {
                "value": "Standard",
                "description": "Standard service level"
              },
              {
                "value": "Premium",
                "description": "Premium service level"
              },
              {
                "value": "Ultra",
                "description": "Ultra service level"
              }
            ]
          },
          "example": "Ultra",
          "default": "Premium"
        },

The other changes are needed, yearlyBackupsToKeep in BackupPolicies is not supported by the service and should not have been inthere in the first place. Using it will cause error for users.
We are also correcting an omission by adding readonly tag on properties volumesAssigned volumeBackups that are indeed read only properties that the user cannot change directly.
@lirenhe Now that you have a bit of context can you please let be know if we need to go through the breaking changes process for those changes.

@audunn, thanks for providing the context but based on our policy, it is still required for the breaking change review as the review board is responsible for the approval.  Could you provide the above justification to them as well?
@jianyexi, could you help with the one for 'Default Value changed'?

@audunn  In autorest, the default value next to the "$ref" will be ignored , it's not supported to add default value in your way.
see https://github.com/Azure/autorest/blob/main/docs/openapi/howto/$ref-siblings.md

Thank you @lirenhe and @jianyexi  this has fixed the issue with the default on ServiceLevel.  Leaving the Backup Policies properties,
Note that the Backup feature is in private preview still.

Good to see

Thank you @lirenhe and @jianyexi looks like this has fixed the issue with the default on ServiceLevel. Leaving the Backup Policies properties,
Note that the Backup feature (including backup policies) is in private preview still.

As the breaking change comes from the latest stable version, I added the label of 'BreakingChangeReviewRequired'. Please help to follow the process which is required for this PR. Thanks

Thank you @lirenhe and @jianyexi looks like this has fixed the issue with the default on ServiceLevel. Leaving the Backup Policies properties,
Note that the Backup feature (including backup policies) is in private preview still.

As the breaking change comes from the latest stable version, I added the label of 'BreakingChangeReviewRequired'. Please help to follow the process which is required for this PR. Thanks

We have started the process  https://msazure.visualstudio.com/One/_workitems/edit/10295481

😂

👍

🚀

🎉

😒

😂

🙏

❤

👍

👍

👍

We are still waiting on breaking change review

😂
IHSAN KAPAHI

On 10-Aug-2021, at 2:08 PM, Auðunn @.***> wrote:

We are still waiting on breaking change review
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

NewApiVersionRequired reason:

This is a new api-version NewApiVersionRequired reason


[BUG] Dependency conflicts when running in Colab

When I run the colab demo, I get this error in the first cell:
Cloning into 'ConvLab-3'...
remote: Enumerating objects: 1275, done.
remote: Counting objects: 100% (1275/1275), done.
remote: Compressing objects: 100% (998/998), done.
remote: Total 1275 (delta 321), reused 837 (delta 230), pack-reused 0
Receiving objects: 100% (1275/1275), 438.42 MiB | 19.93 MiB/s, done.
Resolving deltas: 100% (321/321), done.
Updating files: 100% (1153/1153), done.
Looking in indexes: https://pypi.org/simple, https://us-python.pkg.dev/colab-wheels/public/simple/
Obtaining file:///content/ConvLab-3
  Preparing metadata (setup.py) ... done
Collecting joblib>=1.2.0
  Downloading joblib-1.2.0-py3-none-any.whl (297 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 298.0/298.0 kB 6.2 MB/s eta 0:00:00
Collecting pillow>=9.3.0
  Downloading Pillow-9.5.0-cp39-cp39-manylinux_2_28_x86_64.whl (3.4 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 3.4/3.4 MB 52.0 MB/s eta 0:00:00
Requirement already satisfied: protobuf>=3.20.2 in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (3.20.3)
Requirement already satisfied: oauthlib>=3.2.1 in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (3.2.2)
Collecting accelerate
  Downloading accelerate-0.18.0-py3-none-any.whl (215 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 215.3/215.3 kB 19.8 MB/s eta 0:00:00
Collecting rouge-score
  Downloading rouge_score-0.1.2.tar.gz (17 kB)
  Preparing metadata (setup.py) ... done
Collecting sacrebleu
  Downloading sacrebleu-2.3.1-py3-none-any.whl (118 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 118.9/118.9 kB 6.3 MB/s eta 0:00:00
Collecting tensorboardX
  Downloading tensorboardX-2.6-py2.py3-none-any.whl (114 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 114.5/114.5 kB 8.5 MB/s eta 0:00:00
Collecting boto3
  Downloading boto3-1.26.110-py3-none-any.whl (135 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 135.6/135.6 kB 4.5 MB/s eta 0:00:00
Requirement already satisfied: matplotlib in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (3.7.1)
Requirement already satisfied: seaborn in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (0.12.2)
Requirement already satisfied: tabulate in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (0.8.10)
Collecting python-Levenshtein
  Downloading python_Levenshtein-0.20.9-py3-none-any.whl (9.4 kB)
Requirement already satisfied: requests in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (2.27.1)
Requirement already satisfied: numpy in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (1.22.4)
Requirement already satisfied: nltk in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (3.8.1)
Requirement already satisfied: scipy in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (1.10.1)
Requirement already satisfied: tensorboard in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (2.12.1)
Collecting torch<=1.13,>=1.10.1
  Downloading torch-1.13.0-cp39-cp39-manylinux1_x86_64.whl (890.2 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 890.2/890.2 MB 1.5 MB/s eta 0:00:00
Collecting transformers<=4.24.0,>=4.17.0
  Downloading transformers-4.24.0-py3-none-any.whl (5.5 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 5.5/5.5 MB 82.6 MB/s eta 0:00:00
Collecting sentence-transformers>=2.2.2
  Downloading sentence-transformers-2.2.2.tar.gz (85 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 86.0/86.0 kB 7.4 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
Collecting datasets>=2.0
  Downloading datasets-2.11.0-py3-none-any.whl (468 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 468.7/468.7 kB 35.7 MB/s eta 0:00:00
Collecting seqeval
  Downloading seqeval-1.2.2.tar.gz (43 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 43.6/43.6 kB 4.4 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
Requirement already satisfied: spacy in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (3.5.1)
Collecting simplejson
  Downloading simplejson-3.19.1-cp39-cp39-manylinux_2_5_x86_64.manylinux1_x86_64.manylinux_2_17_x86_64.manylinux2014_x86_64.whl (137 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 137.4/137.4 kB 13.5 MB/s eta 0:00:00
Collecting unidecode
  Downloading Unidecode-1.3.6-py3-none-any.whl (235 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 235.9/235.9 kB 19.8 MB/s eta 0:00:00
Requirement already satisfied: jieba in /usr/local/lib/python3.9/dist-packages (from convlab==3.0.1.dev20230411) (0.42.1)
Collecting embeddings
  Downloading embeddings-0.0.8-py3-none-any.whl (12 kB)
Collecting visdom
  Downloading visdom-0.2.4.tar.gz (1.4 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.4/1.4 MB 36.8 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
Collecting quadprog
  Downloading quadprog-0.1.11-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (453 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 453.4/453.4 kB 32.9 MB/s eta 0:00:00
Collecting fuzzywuzzy
  Downloading fuzzywuzzy-0.18.0-py2.py3-none-any.whl (18 kB)
Collecting json_lines
  Downloading json_lines-0.5.0-py2.py3-none-any.whl (6.8 kB)
Collecting gtts
  Downloading gTTS-2.3.1-py3-none-any.whl (28 kB)
Collecting deepspeech
  Downloading deepspeech-0.9.3-cp39-cp39-manylinux1_x86_64.whl (9.2 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 9.2/9.2 MB 82.4 MB/s eta 0:00:00
Collecting pydub
  Downloading pydub-0.25.1-py2.py3-none-any.whl (32 kB)
Collecting responses<0.19
  Downloading responses-0.18.0-py3-none-any.whl (38 kB)
Requirement already satisfied: pandas in /usr/local/lib/python3.9/dist-packages (from datasets>=2.0->convlab==3.0.1.dev20230411) (1.4.4)
Requirement already satisfied: packaging in /usr/local/lib/python3.9/dist-packages (from datasets>=2.0->convlab==3.0.1.dev20230411) (23.0)
Requirement already satisfied: pyarrow>=8.0.0 in /usr/local/lib/python3.9/dist-packages (from datasets>=2.0->convlab==3.0.1.dev20230411) (9.0.0)
Collecting huggingface-hub<1.0.0,>=0.11.0
  Downloading huggingface_hub-0.13.4-py3-none-any.whl (200 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 200.1/200.1 kB 18.7 MB/s eta 0:00:00
Collecting aiohttp
  Downloading aiohttp-3.8.4-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (1.0 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.0/1.0 MB 54.3 MB/s eta 0:00:00
Requirement already satisfied: pyyaml>=5.1 in /usr/local/lib/python3.9/dist-packages (from datasets>=2.0->convlab==3.0.1.dev20230411) (6.0)
Requirement already satisfied: tqdm>=4.62.1 in /usr/local/lib/python3.9/dist-packages (from datasets>=2.0->convlab==3.0.1.dev20230411) (4.65.0)
Collecting xxhash
  Downloading xxhash-3.2.0-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (212 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 212.2/212.2 kB 18.6 MB/s eta 0:00:00
Collecting multiprocess
  Downloading multiprocess-0.70.14-py39-none-any.whl (132 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 132.9/132.9 kB 11.5 MB/s eta 0:00:00
Requirement already satisfied: fsspec[http]>=2021.11.1 in /usr/local/lib/python3.9/dist-packages (from datasets>=2.0->convlab==3.0.1.dev20230411) (2023.3.0)
Collecting dill<0.3.7,>=0.3.0
  Downloading dill-0.3.6-py3-none-any.whl (110 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 110.5/110.5 kB 12.4 MB/s eta 0:00:00
Requirement already satisfied: charset-normalizer~=2.0.0 in /usr/local/lib/python3.9/dist-packages (from requests->convlab==3.0.1.dev20230411) (2.0.12)
Requirement already satisfied: certifi>=2017.4.17 in /usr/local/lib/python3.9/dist-packages (from requests->convlab==3.0.1.dev20230411) (2022.12.7)
Requirement already satisfied: idna<4,>=2.5 in /usr/local/lib/python3.9/dist-packages (from requests->convlab==3.0.1.dev20230411) (3.4)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in /usr/local/lib/python3.9/dist-packages (from requests->convlab==3.0.1.dev20230411) (1.26.15)
Requirement already satisfied: torchvision in /usr/local/lib/python3.9/dist-packages (from sentence-transformers>=2.2.2->convlab==3.0.1.dev20230411) (0.15.1+cu118)
Requirement already satisfied: scikit-learn in /usr/local/lib/python3.9/dist-packages (from sentence-transformers>=2.2.2->convlab==3.0.1.dev20230411) (1.2.2)
Collecting sentencepiece
  Downloading sentencepiece-0.1.97-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (1.3 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.3/1.3 MB 63.0 MB/s eta 0:00:00
Requirement already satisfied: typing-extensions in /usr/local/lib/python3.9/dist-packages (from torch<=1.13,>=1.10.1->convlab==3.0.1.dev20230411) (4.5.0)
Collecting nvidia-cudnn-cu11==<IP_ADDRESS>
  Downloading nvidia_cudnn_cu11-<IP_ADDRESS>-2-py3-none-manylinux1_x86_64.whl (557.1 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 557.1/557.1 MB 2.2 MB/s eta 0:00:00
Collecting nvidia-cuda-nvrtc-cu11==11.7.99
  Downloading nvidia_cuda_nvrtc_cu11-11.7.99-2-py3-none-manylinux1_x86_64.whl (21.0 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 21.0/21.0 MB 59.1 MB/s eta 0:00:00
Collecting nvidia-cuda-runtime-cu11==11.7.99
  Downloading nvidia_cuda_runtime_cu11-11.7.99-py3-none-manylinux1_x86_64.whl (849 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 849.3/849.3 kB 52.2 MB/s eta 0:00:00
Collecting nvidia-cublas-cu11==<IP_ADDRESS>
  Downloading nvidia_cublas_cu11-<IP_ADDRESS>-py3-none-manylinux1_x86_64.whl (317.1 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 317.1/317.1 MB 2.5 MB/s eta 0:00:00
Requirement already satisfied: wheel in /usr/local/lib/python3.9/dist-packages (from nvidia-cublas-cu11==<IP_ADDRESS>->torch<=1.13,>=1.10.1->convlab==3.0.1.dev20230411) (0.40.0)
Requirement already satisfied: setuptools in /usr/local/lib/python3.9/dist-packages (from nvidia-cublas-cu11==<IP_ADDRESS>->torch<=1.13,>=1.10.1->convlab==3.0.1.dev20230411) (67.6.1)
Requirement already satisfied: regex!=2019.12.17 in /usr/local/lib/python3.9/dist-packages (from transformers<=4.24.0,>=4.17.0->convlab==3.0.1.dev20230411) (2022.10.31)
Collecting tokenizers!=0.11.3,<0.14,>=0.11.1
  Downloading tokenizers-0.13.3-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (7.8 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 7.8/7.8 MB 82.9 MB/s eta 0:00:00
Requirement already satisfied: filelock in /usr/local/lib/python3.9/dist-packages (from transformers<=4.24.0,>=4.17.0->convlab==3.0.1.dev20230411) (3.10.7)
Requirement already satisfied: psutil in /usr/local/lib/python3.9/dist-packages (from accelerate->convlab==3.0.1.dev20230411) (5.9.4)
Collecting botocore<1.30.0,>=1.29.110
  Downloading botocore-1.29.110-py3-none-any.whl (10.6 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 10.6/10.6 MB 84.3 MB/s eta 0:00:00
Collecting s3transfer<0.7.0,>=0.6.0
  Downloading s3transfer-0.6.0-py3-none-any.whl (79 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 79.6/79.6 kB 6.5 MB/s eta 0:00:00
Collecting jmespath<2.0.0,>=0.7.1
  Downloading jmespath-1.0.1-py3-none-any.whl (20 kB)
Requirement already satisfied: click<8.2,>=7.1 in /usr/local/lib/python3.9/dist-packages (from gtts->convlab==3.0.1.dev20230411) (8.1.3)
Requirement already satisfied: six in /usr/local/lib/python3.9/dist-packages (from json_lines->convlab==3.0.1.dev20230411) (1.16.0)
Requirement already satisfied: python-dateutil>=2.7 in /usr/local/lib/python3.9/dist-packages (from matplotlib->convlab==3.0.1.dev20230411) (2.8.2)
Requirement already satisfied: fonttools>=4.22.0 in /usr/local/lib/python3.9/dist-packages (from matplotlib->convlab==3.0.1.dev20230411) (4.39.3)
Requirement already satisfied: pyparsing>=2.3.1 in /usr/local/lib/python3.9/dist-packages (from matplotlib->convlab==3.0.1.dev20230411) (3.0.9)
Requirement already satisfied: kiwisolver>=1.0.1 in /usr/local/lib/python3.9/dist-packages (from matplotlib->convlab==3.0.1.dev20230411) (1.4.4)
Requirement already satisfied: cycler>=0.10 in /usr/local/lib/python3.9/dist-packages (from matplotlib->convlab==3.0.1.dev20230411) (0.11.0)
Requirement already satisfied: importlib-resources>=3.2.0 in /usr/local/lib/python3.9/dist-packages (from matplotlib->convlab==3.0.1.dev20230411) (5.12.0)
Requirement already satisfied: contourpy>=1.0.1 in /usr/local/lib/python3.9/dist-packages (from matplotlib->convlab==3.0.1.dev20230411) (1.0.7)
Collecting Levenshtein==0.20.9
  Downloading Levenshtein-0.20.9-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (175 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 175.5/175.5 kB 12.5 MB/s eta 0:00:00
Collecting rapidfuzz<3.0.0,>=2.3.0
  Downloading rapidfuzz-2.15.1-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (3.0 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 3.0/3.0 MB 65.9 MB/s eta 0:00:00
Requirement already satisfied: absl-py in /usr/local/lib/python3.9/dist-packages (from rouge-score->convlab==3.0.1.dev20230411) (1.4.0)
Collecting portalocker
  Downloading portalocker-2.7.0-py2.py3-none-any.whl (15 kB)
Collecting colorama
  Downloading colorama-0.4.6-py2.py3-none-any.whl (25 kB)
Requirement already satisfied: lxml in /usr/local/lib/python3.9/dist-packages (from sacrebleu->convlab==3.0.1.dev20230411) (4.9.2)
Requirement already satisfied: wasabi<1.2.0,>=0.9.1 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (1.1.1)
Requirement already satisfied: jinja2 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (3.1.2)
Requirement already satisfied: langcodes<4.0.0,>=3.2.0 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (3.3.0)
Requirement already satisfied: spacy-legacy<3.1.0,>=3.0.11 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (3.0.12)
Requirement already satisfied: typer<0.8.0,>=0.3.0 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (0.7.0)
Requirement already satisfied: thinc<8.2.0,>=8.1.8 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (8.1.9)
Requirement already satisfied: smart-open<7.0.0,>=5.2.1 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (6.3.0)
Requirement already satisfied: preshed<3.1.0,>=3.0.2 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (3.0.8)
Requirement already satisfied: pathy>=0.10.0 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (0.10.1)
Requirement already satisfied: pydantic!=1.8,!=1.8.1,<1.11.0,>=1.7.4 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (1.10.7)
Requirement already satisfied: spacy-loggers<2.0.0,>=1.0.0 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (1.0.4)
Requirement already satisfied: cymem<2.1.0,>=2.0.2 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (2.0.7)
Requirement already satisfied: catalogue<2.1.0,>=2.0.6 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (2.0.8)
Requirement already satisfied: murmurhash<1.1.0,>=0.28.0 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (1.0.9)
Requirement already satisfied: srsly<3.0.0,>=2.4.3 in /usr/local/lib/python3.9/dist-packages (from spacy->convlab==3.0.1.dev20230411) (2.4.6)
Requirement already satisfied: google-auth<3,>=1.6.3 in /usr/local/lib/python3.9/dist-packages (from tensorboard->convlab==3.0.1.dev20230411) (2.17.1)
Requirement already satisfied: werkzeug>=1.0.1 in /usr/local/lib/python3.9/dist-packages (from tensorboard->convlab==3.0.1.dev20230411) (2.2.3)
Requirement already satisfied: tensorboard-plugin-wit>=1.6.0 in /usr/local/lib/python3.9/dist-packages (from tensorboard->convlab==3.0.1.dev20230411) (1.8.1)
Requirement already satisfied: google-auth-oauthlib<1.1,>=0.5 in /usr/local/lib/python3.9/dist-packages (from tensorboard->convlab==3.0.1.dev20230411) (1.0.0)
Requirement already satisfied: tensorboard-data-server<0.8.0,>=0.7.0 in /usr/local/lib/python3.9/dist-packages (from tensorboard->convlab==3.0.1.dev20230411) (0.7.0)
Requirement already satisfied: grpcio>=1.48.2 in /usr/local/lib/python3.9/dist-packages (from tensorboard->convlab==3.0.1.dev20230411) (1.53.0)
Requirement already satisfied: markdown>=2.6.8 in /usr/local/lib/python3.9/dist-packages (from tensorboard->convlab==3.0.1.dev20230411) (3.4.3)
Requirement already satisfied: tornado in /usr/local/lib/python3.9/dist-packages (from visdom->convlab==3.0.1.dev20230411) (6.2)
Collecting jsonpatch
  Downloading jsonpatch-1.32-py2.py3-none-any.whl (12 kB)
Requirement already satisfied: websocket-client in /usr/local/lib/python3.9/dist-packages (from visdom->convlab==3.0.1.dev20230411) (1.5.1)
Requirement already satisfied: networkx in /usr/local/lib/python3.9/dist-packages (from visdom->convlab==3.0.1.dev20230411) (3.0)
Collecting async-timeout<5.0,>=4.0.0a3
  Downloading async_timeout-4.0.2-py3-none-any.whl (5.8 kB)
Collecting frozenlist>=1.1.1
  Downloading frozenlist-1.3.3-cp39-cp39-manylinux_2_5_x86_64.manylinux1_x86_64.manylinux_2_17_x86_64.manylinux2014_x86_64.whl (158 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 158.8/158.8 kB 14.4 MB/s eta 0:00:00
Collecting aiosignal>=1.1.2
  Downloading aiosignal-1.3.1-py3-none-any.whl (7.6 kB)
Collecting multidict<7.0,>=4.5
  Downloading multidict-6.0.4-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (114 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 114.2/114.2 kB 12.8 MB/s eta 0:00:00
Requirement already satisfied: attrs>=17.3.0 in /usr/local/lib/python3.9/dist-packages (from aiohttp->datasets>=2.0->convlab==3.0.1.dev20230411) (22.2.0)
Collecting yarl<2.0,>=1.0
  Downloading yarl-1.8.2-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (264 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 264.6/264.6 kB 20.7 MB/s eta 0:00:00
Requirement already satisfied: cachetools<6.0,>=2.0.0 in /usr/local/lib/python3.9/dist-packages (from google-auth<3,>=1.6.3->tensorboard->convlab==3.0.1.dev20230411) (5.3.0)
Requirement already satisfied: pyasn1-modules>=0.2.1 in /usr/local/lib/python3.9/dist-packages (from google-auth<3,>=1.6.3->tensorboard->convlab==3.0.1.dev20230411) (0.2.8)
Requirement already satisfied: rsa<5,>=3.1.4 in /usr/local/lib/python3.9/dist-packages (from google-auth<3,>=1.6.3->tensorboard->convlab==3.0.1.dev20230411) (4.9)
Requirement already satisfied: requests-oauthlib>=0.7.0 in /usr/local/lib/python3.9/dist-packages (from google-auth-oauthlib<1.1,>=0.5->tensorboard->convlab==3.0.1.dev20230411) (1.3.1)
Requirement already satisfied: zipp>=3.1.0 in /usr/local/lib/python3.9/dist-packages (from importlib-resources>=3.2.0->matplotlib->convlab==3.0.1.dev20230411) (3.15.0)
Requirement already satisfied: importlib-metadata>=4.4 in /usr/local/lib/python3.9/dist-packages (from markdown>=2.6.8->tensorboard->convlab==3.0.1.dev20230411) (6.1.0)
Requirement already satisfied: pytz>=2020.1 in /usr/local/lib/python3.9/dist-packages (from pandas->datasets>=2.0->convlab==3.0.1.dev20230411) (2022.7.1)
Requirement already satisfied: threadpoolctl>=2.0.0 in /usr/local/lib/python3.9/dist-packages (from scikit-learn->sentence-transformers>=2.2.2->convlab==3.0.1.dev20230411) (3.1.0)
Requirement already satisfied: confection<1.0.0,>=0.0.1 in /usr/local/lib/python3.9/dist-packages (from thinc<8.2.0,>=8.1.8->spacy->convlab==3.0.1.dev20230411) (0.0.4)
Requirement already satisfied: blis<0.8.0,>=0.7.8 in /usr/local/lib/python3.9/dist-packages (from thinc<8.2.0,>=8.1.8->spacy->convlab==3.0.1.dev20230411) (0.7.9)
Requirement already satisfied: MarkupSafe>=2.1.1 in /usr/local/lib/python3.9/dist-packages (from werkzeug>=1.0.1->tensorboard->convlab==3.0.1.dev20230411) (2.1.2)
Collecting jsonpointer>=1.9
  Downloading jsonpointer-2.3-py2.py3-none-any.whl (7.8 kB)
Collecting torchvision
  Downloading torchvision-0.15.1-cp39-cp39-manylinux1_x86_64.whl (6.0 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 6.0/6.0 MB 68.4 MB/s eta 0:00:00
  Downloading torchvision-0.14.1-cp39-cp39-manylinux1_x86_64.whl (24.2 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 24.2/24.2 MB 39.9 MB/s eta 0:00:00
  Downloading torchvision-0.14.0-cp39-cp39-manylinux1_x86_64.whl (24.3 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 24.3/24.3 MB 15.1 MB/s eta 0:00:00
Requirement already satisfied: pyasn1<0.5.0,>=0.4.6 in /usr/local/lib/python3.9/dist-packages (from pyasn1-modules>=0.2.1->google-auth<3,>=1.6.3->tensorboard->convlab==3.0.1.dev20230411) (0.4.8)
Building wheels for collected packages: sentence-transformers, rouge-score, seqeval, visdom
  Building wheel for sentence-transformers (setup.py) ... done
  Created wheel for sentence-transformers: filename=sentence_transformers-2.2.2-py3-none-any.whl size=125942 sha256=89d5170d8740793618d416a7e38f6764267deb1f162a4cce002f5aa630c1a92a
  Stored in directory: /root/.cache/pip/wheels/71/67/06/162a3760c40d74dd40bc855d527008d26341c2b0ecf3e8e11f
  Building wheel for rouge-score (setup.py) ... done
  Created wheel for rouge-score: filename=rouge_score-0.1.2-py3-none-any.whl size=24954 sha256=e9dc960b1f2edd43ca05fcee0ea6f18fa8a7269f198a55c4f92a5ed06e046a57
  Stored in directory: /root/.cache/pip/wheels/9b/3d/39/09558097d3119ca0a4d462df68f22c6f3c1b345ac63a09b86e
  Building wheel for seqeval (setup.py) ... done
  Created wheel for seqeval: filename=seqeval-1.2.2-py3-none-any.whl size=16180 sha256=ffbd9afcf332b427c5143fe19ff34f281ea4b8cbdf56b0ca2cb1fd6b71348798
  Stored in directory: /root/.cache/pip/wheels/e2/a5/92/2c80d1928733611c2747a9820e1324a6835524d9411510c142
  Building wheel for visdom (setup.py) ... done
  Created wheel for visdom: filename=visdom-0.2.4-py3-none-any.whl size=1408216 sha256=d2062f3a4b943481110ad6f0da2b0843e94c520bea3a5dcf3a96a0e8ea74b7cb
  Stored in directory: /root/.cache/pip/wheels/58/9e/14/30f7cc4dafdd4d602fb00ca33c6edd1424fc0f5df10a02e060
Successfully built sentence-transformers rouge-score seqeval visdom
Installing collected packages: tokenizers, sentencepiece, pydub, fuzzywuzzy, xxhash, unidecode, tensorboardX, simplejson, rapidfuzz, quadprog, portalocker, pillow, nvidia-cuda-runtime-cu11, nvidia-cuda-nvrtc-cu11, nvidia-cublas-cu11, multidict, jsonpointer, json_lines, joblib, jmespath, frozenlist, dill, deepspeech, colorama, async-timeout, yarl, sacrebleu, responses, nvidia-cudnn-cu11, multiprocess, Levenshtein, jsonpatch, huggingface-hub, gtts, embeddings, botocore, aiosignal, visdom, transformers, torch, seqeval, s3transfer, rouge-score, python-Levenshtein, aiohttp, torchvision, boto3, accelerate, sentence-transformers, datasets, convlab
  Attempting uninstall: pillow
    Found existing installation: Pillow 8.4.0
    Uninstalling Pillow-8.4.0:
      Successfully uninstalled Pillow-8.4.0
  Attempting uninstall: joblib
    Found existing installation: joblib 1.1.1
    Uninstalling joblib-1.1.1:
      Successfully uninstalled joblib-1.1.1
  Attempting uninstall: torch
    Found existing installation: torch 2.0.0+cu118
    Uninstalling torch-2.0.0+cu118:
      Successfully uninstalled torch-2.0.0+cu118
  Attempting uninstall: torchvision
    Found existing installation: torchvision 0.15.1+cu118
    Uninstalling torchvision-0.15.1+cu118:
      Successfully uninstalled torchvision-0.15.1+cu118
  Running setup.py develop for convlab
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
torchtext 0.15.1 requires torch==2.0.0, but you have torch 1.13.0 which is incompatible.
torchdata 0.6.0 requires torch==2.0.0, but you have torch 1.13.0 which is incompatible.
torchaudio 2.0.1+cu118 requires torch==2.0.0, but you have torch 1.13.0 which is incompatible.
pandas-profiling 3.2.0 requires joblib~=1.1.0, but you have joblib 1.2.0 which is incompatible.
Successfully installed Levenshtein-0.20.9 accelerate-0.18.0 aiohttp-3.8.4 aiosignal-1.3.1 async-timeout-4.0.2 boto3-1.26.110 botocore-1.29.110 colorama-0.4.6 convlab-3.0.1.dev20230411 datasets-2.11.0 deepspeech-0.9.3 dill-0.3.6 embeddings-0.0.8 frozenlist-1.3.3 fuzzywuzzy-0.18.0 gtts-2.3.1 huggingface-hub-0.13.4 jmespath-1.0.1 joblib-1.2.0 json_lines-0.5.0 jsonpatch-1.32 jsonpointer-2.3 multidict-6.0.4 multiprocess-0.70.14 nvidia-cublas-cu11-<IP_ADDRESS> nvidia-cuda-nvrtc-cu11-11.7.99 nvidia-cuda-runtime-cu11-11.7.99 nvidia-cudnn-cu11-<IP_ADDRESS> pillow-9.5.0 portalocker-2.7.0 pydub-0.25.1 python-Levenshtein-0.20.9 quadprog-0.1.11 rapidfuzz-2.15.1 responses-0.18.0 rouge-score-0.1.2 s3transfer-0.6.0 sacrebleu-2.3.1 sentence-transformers-2.2.2 sentencepiece-0.1.97 seqeval-1.2.2 simplejson-3.19.1 tensorboardX-2.6 tokenizers-0.13.3 torch-1.13.0 torchvision-0.14.0 transformers-4.24.0 unidecode-1.3.6 visdom-0.2.4 xxhash-3.2.0 yarl-1.8.2

It seems that google colab originally installed torchtext, torchdata, and torchaudio along with torch==2.0.0. However, ConvLab does not require these packages, so it should be ok to ignore the warning. We require joblib>=1.2.0 because this alarm.

The version 3.2.0 of pandas-profiling is almost a year old. Is there a way to update it to something more recent, so that it requires a more recent joblib?

I think it is installed in colab as default. If you want to change it you can just install a newer version. However, ConvLab does not require this package so I think it's fine to ignore it.

Can we add instructions to uninstall the conflicting packages to the notebook so that it works in Colab?

Sure, you can uninstall the package in Colab

Would you please show me and the other users by changing the notebook?

I am not sure what you want. Do you mean that I change the tutorial notebook so that it will not report python package conflict in colab?

Yes. I do not know how to implement the change and I feel like the purpose of the notebooks is that potential users in general do not need to know, either. To cater to them, the notebook should be made executable by uninstalling the conflicting packages.

I see. Thanks for your advice! I will try to make a change.

Wow, that is interesting, but restarting the kernel feels like a hack. I would strongly prefer a more proper solution to present to potential users.

It's not a hack. It's a reminder. When you start the jupyter notebook (just like you run a python program), the change of its environment will not take effect until you restart the kernel (re-run the python program). The above code is copied from HuggingFace to auto restart kernel in case users forget to do so.

It's not a hack. It's a reminder. When you start the jupyter notebook (just like you run a python program), the change of its environment will not take effect until you restart the kernel (re-run the python program). The above code is copied from HuggingFace to auto restart kernel in case users forget to do so.


codemeta creation

create a basic codemeta.json file and get rid of codemetapy dependency, enable usage in windows

@apirogov I forgot to rename after deleting old files, updated it now

I tried it in my Windows PC too, it worked without problems, as it did in testing pipeline.

resolves #63  #48  #49


More options for showing country flags

Right now I went a little overboard and put flags EVERYWHERE. People should be able to select just playing games or forum or something like that for their flags.

I think we're fine for now. Closing this until someone complains.


add an option for not adding the repo for IBM service and productivity tools for Power

So it is possible to install the IBM service and productivity tools for Power from a local source.

solved by pull request #115.


fromProviderState on path doesn't work in pact-jvm-consumer-groovy

I used fromProviderState on path parameter, but it doesn't work:
withAttributes(method: 'get', path: fromProviderState('/shoppingCart/v2.0/shoppingCart/${shoppingcartId}', '/shoppingCart/v2.0/shoppingCart/ShoppingCart_05540051-1155-4557-8080-008a802200aa'),
query: ([mask: 'returnStockInformation,returnNote', includeExpiredShoppingCart: 'false', excludePrices: 'true']))
I see that the contract is generated wrongly. The generator for the path is created on the response instead of on the request. When I modify the contract manually, it's working.

Version 4.0.7 has been released with this change


Add retrieveIssuesToFollow api and refactor issuesRetrieve related methods

Add retrieveIssuesToFollow api and refactor issuesRetrieve related methods
#842

👍


[Fix #8971] Fix a false alarm for # rubocop:disable Lint/EmptyBlock

Fixes #8971.
This PR fixes the following false alarm for # rubocop:disable Lint/EmptyBlock inline comment with Lint/RedundantCopDisableDirective.
% cat example.rb
# frozen_string_literal: true

assert_equal nil, combinator {}.call # rubocop:disable Lint/EmptyBlock

% bundle exec rubocop example.rb
Inspecting 1 file
W

Offenses:

example.rb:3:38: W: Lint/RedundantCopDisableDirective: Unnecessary
disabling of Lint/EmptyBlock.
assert_equal nil, combinator {}.call # rubocop:disable Lint/EmptyBlock
                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

1 file inspected, 1 offense detected, 1 offense auto-correctabl

Lint/EmptyBlock is AllowComments: true by default. Lint/RedundantCopDisableDirective was occurring because
# rubocop:disable Lint/EmptyBlock inline comment was treated as an allowed comment by Lint/EmptyBlock.
This PR does not treat # rubocop:disable as a comment allowed by AllowComment.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Thanks!


Debugging tool for simplification

We can run the prover itself incrementally, by running one rewrite step at a time.
We should architect the new simplifier to be able to do something similar. This will make it much easier for both developers and users to debug the backend/their semantics.

@goodlyrottenapple and @jberthold please add any ideas you have to this issue.

(Summarising points I remember from brainstorming during stand-up:)

The simplifier iterations are currently not observable , and it is hard to observe the simplifier alone at all.
As a debugging tool we could think about logging the term during simplifier iterations (where applicable) to a file, similar to the recent change #3284  to log rewrite rule applications to a yaml file.
A stand-alone tool could consume such a simplificaiton log to display a diff-like view of a pretty-printed term
Both logs (rewriting and simplification) should probably write a combined yaml log file, in order for the tool to identify the execution steps.


karma.error

After incorrectly setting up karma-systemjs I was receiving this error

TypeError: karma.error is not a function

When I run the following typeof window.__karma__.error I get 'undefined'.
The call to karma.error is being used here and here.
Is there any reason for this? Can we just dump the error as is?

It's done that way because errors need to be fed into Karma so that they can in turn be fed into the reporters.
Otherwise they won't get picked up on things like JUnit reporters used by CI servers like Bamboo.
I could dump the errors out using console.log(), but it might get a bit noisy.
Better option might be to check that window.__karma__ is defined first and fail fast if it's not.

Hey guys. This is currently happening in my set up. I have
frameworks: ['systemjs', 'jspm', 'jasmine'],
plugins:[
  "karma-jspm",
  'karma-jasmine',
  'karma-chrome-launcher',
  "karma-systemjs"
],

What's the missing config to be able to call karma.error? It's undefined if I open the browser and try to debug on it. I tried looking at the karma source code but couldn't find that method defined anywhere.

First thing I see is you're using both karma-jspm and karma-systemjs.
You don't need both. If you're using JSPM I recommend going with karma-jspm.

The problem identified by diestrin, and perhaps by whitneyit, is actually a karma bug. In debug mode, the karma debug.html page load creates a limited karma object which does not include an error function. I forked karma and added a debug.html error function which corrects the immediate error and allows karma to print the error to the console. I filed a karma issue for this bug.


Handle sigint to close sockets

Closing with solution: use reusable addr to allow new sockets to bind, except when there is an active listening socket


Gettin parent name of menu/menu_item

I am trying to to create a menu where the renderer will also display the parent item name on the current item. For example, think of this structure:
Main Menu
- Item 1
-- Item A
-- Item B
- Item 2
- Item 3

So, when browsing the menu, you will get something like this:
Line 1: Main Menu
Line 2: Item1

Or this:
Line 1: Item 1
Line 2: Item A

This is my current (and early) implementation of a renderer:
class Renderer : public MenuComponentRenderer {
public:
    void render(Menu const& menu) const {
        oled.clearDisplay();
        menu.get_current_component()->render(*this);
    }

    void render_menu_item(MenuItem const& menu_item) const {
        oled.setTextSize(1);
        oled.setTextColor(WHITE);
        oled.setCursor(0,0);
        oled.println("Menu Item");
        oled.setTextSize(2);
        oled.setTextColor(WHITE);
        oled.setCursor(0,17);
        oled.println(menu_item.get_name());
        oled.display();
    }

    void render_back_menu_item(BackMenuItem const& menu_item) const {}

    void render_numeric_menu_item(NumericMenuItem const& menu_item) const {}

    void render_menu(Menu const& menu) const {
        oled.setTextSize(1);
        oled.setTextColor(WHITE);
        oled.setCursor(0,0);
        oled.println("Render menu");
        oled.setTextSize(2);
        oled.setTextColor(WHITE);
        oled.setCursor(0,17);
        oled.print(menu.get_name());
        oled.display();
    }
};
Renderer menu_render;

I can see there are methods like get_parent(), but I am not sure of how to implement them. Can you please point me to the right direction? I am quite new to arduino programming...

Given your menu structure above, when the current menu is the root "Main Menu", the first line displays "Main Menu" and the second line allows you to scroll through the components in that menu. Is that what you mean?
When you call MenuSystem::display it will call Renderer::render with the current menu (in this case "Main Menu"). If you first display the menu's name in that render method before rendering it's current component, that should do the trick:
class Renderer : public MenuComponentRenderer {
public:
    void render(Menu const& menu) const {
        oled.clearDisplay();
        // code to output menu.get_name() to line 1 here
        menu.get_current_component()->render(*this); // <-- this will trigger rendering for the current item
    }

    // Code omitted. Each menu component render function should output to line 2
};

I hope that makes sense. If you have any questions (or it doesn't work) please let me know.

 I am using a declaration like below for the menu, and the menu.get_name() comes up empty when I navigate the level 0 (Item 1,2, etc)
Here is a sample of my menu definitions:
MenuSystem ms(menu_render);
Menu mm("Main Menu");
MenuItem mi_i1("Item 1", &on_menu_item1);
MenuItem mi_i2("Item 2", &on_menu_item2);
Menu mi_i3("Item 3");
MenuItem mi_i3_a("Item A", &on_menu_i3a);
MenuItem mi_i3_b("Item B", &on_menu_i3b);

The MenuSystem class creates the top-level root menu for you, and it's name is an empty string, so internally your menu structure above looks like the following:
* "" // this is created automatically
  * main menu
    * Item 1
    * Item 2
    * Item 3
      * Item A
      * Item B

This assumes you're using MenuSystem::get_root_menu().add_xxx to connect your MenuComponent instances to MenuSystem.
In previous version of this library the client would create the root manually and pass it in the MenuSystem constructor. I changed it to create it automatically because it was never displayed (or so I thought) and therefore felt awkward to create.
So that's the why. The next thing is how do we solve your problem. Great question! :smile:
I still think it's a good idea for MenuSystem to create the root Menu. The problem is that it assumes it won't be displayed and sets its name to an empty string. I think the best solution is to add an optional char* root_name parameter to the MenuSystem constructor for setting the name of the root Menu.
I'm open to other suggestions if you have them.

Well, one way could be to see add back the ability to set the root menu, along with a name, like v2.1. If the user has defined one, use it. Otherwise, create the root menu automatically, like in v3.0.
For now, I just modified the render method to check if the menu name is empty and display a pre-defined string if it is. Not too hacky, and is the simplest solution I could find, without modifying the library...

I prefer the current implementation where the root is created for you. I'll add an extra, optional parameter for setting the name.

Hi, I am facing the same issue, was an update posted with the option to setting the root name?
Thank you!


Must have ability to checkpoint and continue on execution completion, by default

On success, or failure, of batch processing, we need to "checkpoint" and continue.
Follow the same behaviour as current Event Hubs trigger.

Unlike EventHubs, Kafka provides checkpoint saving internally (saved as a compact topic). The idea is to use this but save it depending on how settings have been defined, right?

I think this makes sense.  The timing of the checkpoint is a bit important though.  We want to have "at least once" delivery, so as long as the Kafka internal checkpointing allows us to "checkpoint" a batch after processing that should work great.

Yes, we call commit topic/partition/offset. Current implementation does that after batch is consumed, but I need to add more customisation and retries.

So it checkpoints as soon as batch is read from Kafka, before it is processed?

@fbeltrao let's chat about what "more customisation" we need in this first milestone. We can always come back and make it more customisable in future.

My proposal:

For triggers with batch (KafkaEventData[]) we commit after successfully triggering
For single event triggers we commit every X times

Batch sounds good
For single, I am not sure what you mean by every X times.
If I read 1 event from Kafka, process it do we not checkpoint then?
That means if another event later somewhere fails then this one we've already processed successfully will be replayed?

Correct. Now that I think about, maybe is better to do the way you described, saving after each message, giving the most accurate solution out of the box.
Performance is bad, but anything less than trivial should be using batch anyway.

And to clarify the behavior of Event Hubs is:

Retrieve in batch (regardless of if single message function or batch message function)
2a. If they have written their function to process individual messages

Do a Task.WhenAll() and parallel-y invoke a function for each message in the batch.  So if you got a batch of 30 messages, you would have 30 function invocations tasks kicked off and potentially processing in parallel.
2b. If they have written their function to process a batch (recommended and default)
Pass the entire batch into a single execution

Wait for batch to process (either all the concurrent executions or the single batch execution)
Checkpoint and retrieve the next batch

Implemented as above


More general approach for theming

As far as i am aware theme extensions are only available via material library. Im interested in the design choice of this api. It feels not very clean imo. Why is there no extensions property in CupertinoTheme? For example to theme a cupertino based app which does not use any material widgets its either necessary to use a custom InheritedWidget or the Theme widget from material which has material related properties like different themes of material components?
Proposal:
Move Theme/ThemeData to the widgets library and remove the material depending component themes but still keep the mechanism to register extensions.
Material, Cupertino, Fluent or custom library can register their component themes, colors etc via a theme extension when creating their respective MaterialApp, CupertinoApp, FluentApp, ... instead of giving them a specialized theme (CupertinoTheme, Theme, etc) with hardcoded references to component themes.
This leads to:
Theme beeing part of widgets library where it semantically belongs.
Material, Cupertino, ... and Custom Themes can be achived via registering their component themes, colors, textstyles etc as an extension
In widget there is only the need to call Theme.of(context).extension<Foo>(); (for custom) and for example Theme.of(context).extension<DividerTheme>() for (material divider)

@jtdLab
Check this issue and see if it resonates with your proposal.


Bug when resizing grouped shapes

There is a curious bug when resizing grouped shapes : some items are moving on the side whereas they shouldn't.
Normal size :

When resizing :

It's more likely related to linear element resizing issues, and is being tracked here: https://github.com/excalidraw/excalidraw/issues/4001. I didn't have an opportunity to dive into this yet :(.


Added getting urls from clipboard

I thought instead of copying the urls to a spesific place, just copying them to the clipboard and getting the url from there would be a good idea.

I think this is cool, but running the script and blindly downloading the contents in the clipboard is a security concern.

Thanks for your attention! First, it's really security concern and second, it's not really needful and comfortable for me


Incorrect state root in transition test block

In minimal/altair/transition/core/pyspec_tests/transition_with_finality/blocks_0.ssz_snappy the state root is 0x0000000000000000000000000000000000000000000000000000000000000000 so Teku rejects the block as invalid.
blocks_1.ssz_snappy appears to have a non-zero state root so not sure what happened to this first block.  Same problem occurs in a few other transition tests as well for both mainnet and minimal config:

transition_with_no_attestations_until_after_fork
transition_with_random_half_participation
transition_with_random_three_quarters_participation

Interestingly a couple of the tests seem to have the first block right but then a zero state root at later blocks.  Like transition_with_no_attestations_until_after_fork which fails to process blocks_16.ssz_snappy (in minimal - blocks_64.ssz_snappy in mainnet).

Thanks for this! I also can confirm this abnormality while testing Prysm

ah, i have misused some of the helper code in the pyspec testsuites so the state root was not being set correctly -- thanks for raising this, i'll have a fix up soon

the fix for this should be in #2443. i was able to verify all the blocks for the minimal setting were well-formed, had signatures and state roots. i was not able to verify the mainnet setting as i'm lacking up-to-date code for the latest spec version but they should follow the minimal outcome on this issue. please let me know if you run into any other errors!


Point client-side logs at /xoplatform/logger, add x-app-name header

Changes

Point client-side logs at /xoplatform/logger, a new endpoint for client-side logs.
Add x-app-name header to identify these logs as "checkoutjs" logs.

Let's wait to merge this till next week

Not a pressing issue, but FYI master's failing. https://travis-ci.org/paypal/paypal-checkout/builds/221969574
/cc @bluepnume

@mstuart You saw nothing.


fix: Comet should not fail on negative limit parameter

Which issue does this PR close?

Closes #287.
Rationale for this change

What changes are included in this PR?

How are these changes tested?

cc @sunchao @andygrove

Merged. Thanks.


@googlemaps/markerclusterer requires map to have a mapId props [Bug]

Description
I am trying to use clustering markers in a styled map.
In useMap documentation https://visgl.github.io/react-google-maps/docs/api-reference/hooks/use-map it seems like the Map component has a prop id which indicates the id of the map. However, in your example with clustering https://github.com/visgl/react-google-maps/blob/main/examples/marker-clustering/src/app.tsx#L30 the map's id is passed through mapId props.
If I use mapId style is not applied, markers appear correctly.

If I use id then My map is blocked with a warning , markers appear correctly.

After I close this popup, everything is good but on every zoom in or out that changest the cluster appearance the pop up is back again.
Error from console is: _googleMapsCallback__:334 The map is initialized without a valid Map ID, which will prevent use of Advanced Markers.

Steps to Reproduce

Use code from https://github.com/visgl/react-google-maps/tree/main/examples/marker-clustering/src
Try to apply a custom style

Or

Use code from https://github.com/visgl/react-google-maps/tree/main/examples/marker-clustering/src
Change mapId prop to id within  component
Try to apply a custom style

Environment

Library version:
Google maps version: weekly
Browser and Version:
OS:

Logs
No response

Now I have observed the warning that says that I need to style my map via cloud console, so I think that the problem is solved.
https://developers.google.com/maps/documentation/cloud-customization#cloud_tooling


CosmosDB insertion bugfix

Description
This pull request addresses a bug with Azure CosmosDB vector store implementation. Previously, there was an additional metadata field "timeStamp" that included the time when the record was inserted. This caused JSON encoding errors because this field is a datetime object. This PR removes this field, bringing these in line with other integrations.
Fixes #17287
New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[ ] Yes
[x] No

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[ ] Yes
[x] No

Type of Change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Your pull-request will likely not be merged unless it is covered by some form of impactful unit testing.

[ ] I added new unit tests to cover this change
[x] I believe this change is already covered by existing unit tests

Suggested Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

@JeffreyEnglish can you bump the versions of those two packages? Then they will publish on merge.
I'm hoping there isn't a structural reason for why azure might need those fields, but going to have to assume you tested this change 🙏🏻

I've pushed a new change with version increments.
The field is not required, I checked using my local branch and the code I am working on migrating to use CosmosDB. After the change the records insert as expected. CosmosDB (at least CosmosDB No SQL) has a native timeStamp metadata field "_ts" generated in it's own so the extra metadata is redundant.


[oap-native-sql] Add open permission parameter

What changes were proposed in this pull request?
Add open permission parameter
How was this patch tested?
manually tested

@zhouyuan , I remembered you sent out an same fix before? can you verify this PR and merge it?

#1252

Somehow this issue only occurs on Ubuntu 20.04
gcc 9.3 on fedora works fine


Fix Windows make scripts

go get no longer install the required dependencies in the included Windows make scripts.
Replaced all go get references with go install.

Hey there @SansDK, thank you for your interest in contributing to deej.
As per the project's contribution guidelines, I'm currently not accepting PR submissions for deej's main fork. This change is already included in the work on v0.10.0 and will be pushed with the rest of that version when it releases.
Cheers!


Add replace option on unzipfile

Check for and add replace options on unzipfile lines.

Issue closed by @Bucky1192 , thanks.


feat(cd)!: major change to deploy, adhere to QSOS, use HELM , move away from DC(Deprecated Already.)

Thanks for the PR!
Deployments, as required, will be available below:

Frontend available

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

@DerekRoberts @tobiasmllr whenever you guys get time, please review this, once merged , there might be some issues, I will fix them as they come up.


Nicer styles for

Fixes #1224
It seems #677 never actually got merged into master

Hi @jessetan, is there anything else needed for this to go in?

I've put this on the 1.1 milestone

A pleasure merging this PR @Blendify - looking forward to saying hi some day :wave:


Update data-lake-storage-best-practices.md

Fix typo

Docs Build status updates of commit 6dc3fc1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/storage/blobs/data-lake-storage-best-practices.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


How to install macOS 10.13 from scratch?

Hi!
How to install (step by step) macOS 10.13 from scratch, using   macOS 10.13 (config only).utm?
Thanks

Instructions are now in the README.


Runtime assets are not installed/loaded

Description of problem

What did you do?

I've installed basic sensu-backend with sensu-puppet with client on the same machine.

What happened?

I see in the dashboard, that the keepalive events are working correctly but the cpu-check not. I see the error sh: 1: check-cpu.rb: not found.

What did you expect to happen?

The runtime assets should be downloaded in the agent's cache.

How can someone reproduce the problem?

Use the following manifest file:
node "........" {
  class { 'sensu':
    password => '~~~~~~',
    use_ssl => false,
  }
  include sensu::backend
  sensu_check { 'check-cpu':
    ensure        => 'present',
    command       => 'check-cpu.rb -w 75 -c 90',
    interval      => 60,
    subscriptions => ['linux'],
    runtime_assets => ['sensu-plugins-cpu-checks','sensu-ruby-runtime'],
  }
  class { 'sensu::agent':
    subscriptions => ['linux'],
  }
}

Command used and debugging output

What does your puppet config look like (including any hiera config)

See the manifests file above

Is this a masterless or master based puppet setup?

masterless
Platform and version information

Your OS: Ubuntu 18.04.5 LTS, running on a Vagarnt virtual machine.
Your Ruby version: ruby is not installed
Your version of Puppet: 6.25.1 open source
Your version of Sensu: 6.6.3-5588
Your version of this module: 5.8.0

Anything else to add that you think will be helpful?
Note, that if I add the assets manually, it works:
sensuctl asset add sensu-plugins/sensu-plugins-cpu-checks -r sensu-plugins-cpu-checks
sensuctl asset add sensu/sensu-ruby-runtime -r sensu-ruby-runtime

So basically the question is, how can I achieve equivalent functionality as with these 2 sensuctl commands?

The solution: the bonsai plugins has to be defined, so the correct manifests file:
node ".........." {
  class { 'sensu':
    password => '~~~~~~~~~~',
    use_ssl => false,
  }
  include sensu::backend
  sensu_check { 'check-cpu':
    ensure        => 'present',
    command       => 'check-cpu.rb -w 75 -c 90',
    interval      => 60,
    subscriptions => ['linux'],
    runtime_assets => ['sensu-plugins-cpu-checks','sensu-ruby-runtime'],
  }
  sensu_bonsai_asset { 'sensu-plugins/sensu-plugins-cpu-checks':
    rename => 'sensu-plugins-cpu-checks',
    ensure => 'present',
  }
  sensu_bonsai_asset { 'sensu/sensu-ruby-runtime':
    rename => 'sensu-ruby-runtime',
    ensure => 'present',
  }
  class { 'sensu::agent':
    subscriptions => ['linux'],
  }
}


Improve logging and retries for a gcloud command

This switches from exec.Command(...) to the helper retryCmd(...), which handles retrying and streaming stdout/stderr.
This is to help debug the ongoing Restart test problems.

@roberthbailey / @ixdy may have context but this isn't terribly complicated

it's good to confirm at least that the PR Jenkins merges against head...
@k8s-bot test this again please

@ixdy thanks for looking. I addressed your comment and went ahead and switched the rest of the file to use this, resolving several TODOs. PTAL!

@mbforbes - can you please rebase (or close if this is no longer needed)?

I'll rebase soon. If we still want this hopefully we can, uh, get it in soonish

LGTM with rebase.

@mbforbes Please rebase this PR.

Rebased and updated description.

still LGTM


Add default_server to nginx development configuration

This PR adds a default_server to our Nginx development configuration, conf.d/cantusdb.conf.development. In doing so, I think it fixes #1331 (we already have default_servers specified in cantusdb.conf.staging and cantusdb.conf.production)
This PR also removes commented-out lines from cantusdb.conf.development and cantusdb.conf.staging - I had initially thought these would be helpful, but cantusdb.conf.production has been updated and this change has not been added to the comments in the two other files, and I was finding that the extra commented lines make things difficult to read anyways.

This is not really what I meant in the comment that sparked this issue...I should probably have been more specific in the issue.
(I approve getting rid of the commented lines, so if you'd still like to do that, all good!)

Adding default_server where you have in the development nginx.conf file makes the default explicit, but that server was already the default one by virtue of being the first (and only, for that matter) server block.
What prompted the issue was noticing that a request with an incorrect host (or with no host) to the production server on port 80 is getting a 301 redirect to https://cantusdatabase.org and a request with an incorrect host (or with no host) to the production server on port 443 is getting passed to the django container (gunicorn). And mutatis mutandis with staging. So this doesn't affect that (as you say, it doesn't touch the actual configurations on staging or production).

I guess the upshot is that I approve if we de-link this from the issue b/c it otherwise looks fine.

so I'm confused - we do have a default server specified in cantusdb.conf.production: https://github.com/DDMAL/CantusDB/blob/259a1af7508d91303a3d6c1dcf2ca4485a6fcf24/config/nginx/conf.d/cantusdb.conf.production#L34
It's also in the analogous place in cantusdb.conf.staging. This was the only place I could find where we weren't specifying a default_server.
You've referred several times to nginx.conf - is this shorthand for cantusdb.conf, or am I missing something?

You've referred several times to nginx.conf - is this shorthand for cantusdb.conf, or am I missing something?

Yes, sorry...I mean the nginx config file, cantusdb.conf.

we do have a default server specified in cantusdb.conf.production

Sorry, I think my language in the original comment was probably misleading. It's not so much that we need to specify a default server (as you note, we have either done that explicitly in both our .production and .staging configurations in the case of our 443 port, or, in the case of our 80 port, we only have the single server block so it is the default by, well, default. It's that we need to specify a better default server block (we need to make a new server block that we will set as default) so that the case that I mention in the original comment and above (requests with bad hosts being accepted) don't happen.

Thanks to this issue I've had a look at our own configuration for RISM Online. Here's what I've come up with:
    # A default server block to weed out all the direct
    # IP address and non-valid hostnames.
    server {
        access_log off;
        server_name _;

        listen 80 default_server;
        listen [::]:80 default_server;

        listen 443 default_server;
        listen [::]:443 default_server;
        ssl_reject_handshake on;

        # Any request that does not match a defined server name
        # will have its connection closed immediately.
        return 444;
    }

A bit more context: You can set a Host header manually, while still targeting the IP address of your website. With this you can do "host name spoofing" where you try to bypass name-based virtual hosts. Sometimes, if the webserver is not configured correctly, you could have a "private" admin tool (phpmyadmin) being the default virtual host if no actual hostname matches, just by virtue of it being the first configuration loaded.
By setting an unnamed server name (server_name _) you're catching all requests for virtual hosts other than the ones configured. This then returns a 444 response, which immediately closes the connection.
See: https://nginx.org/en/docs/http/request_processing.html#how_to_prevent_undefined_server_names
A few extra bits: Since you don't want to do anything with the request, there's no need to do any SSL stuff, so you can also have it listen to 443, and just immediately reject any SSL attempts, instead of configuring a certificate to only then reject it.
Also, there's not really a need to log any of it (at least, it's not particularly interesting to me) so I just turned off access log entries for this whole server block.

to summarize, just to make sure I'm understanding correctly (this summary assumes we're talking about the Production server):

We have a default_server specified, but it currently handles any traffic sent to domains other than cantusdatabase.org, www.cantusdatabase.org and mass.cantusdatabase.org.
However, that server block should really only handle traffic to cantusdatabase.org.
We should remove all the instances of default_server that are currently in cantusdb.conf, and create a new server block to serve as our default_server which will handle all requests to non-valid hostnames

This new block will listen on both port 80 and port 443, and will reject any requests that come to it (either via a 444 code or via ssl_reject_handshake)

Question: I see, @ahankinson, that you've included both listen 80 and listen [::]:80. A little bit of research suggests that the latter of these allows our server to handle IPv6 traffic in addition to IPv4 traffic. Should we be adding an analogous line with [::]: elsewhere in our configuration wherever we have listens?

The default_server designation indicates which server block handles an incoming request when the incoming request does not match any server_name. The nginx documentation has a useful page that gives more details. If no server block has default_server, the first block in the configuration file (for the given port) is the server that handles the request.

We have a default_server specified, but it currently handles any traffic sent to domains other than cantusdatabase.org, www.cantusdatabase.org and mass.cantusdatabase.org.

Technically, in our current configuration, there are two server blocks that can be thought of as a "default": the first server (the only one listening on port 80) and the last server (the one listening on port 443 with the default_server designation). Any request coming in on port 80 will currently be handled by the first server (because it is the first server block in our configuration listening on port 80 and there is no other server block listening on port 80 that has been designated as the default with default_server). Any request coming in on 443 that is not to www.cantusdatabase.org or mass.cantusdatabase.org will be handled by the server listening to 443 with the default_server designation. The default server, whether designated by default_server or by its position in the configuration file, will handle all requests on the given port that don't match a server_name. Sorry if this is nit-picking, but I figure it may be helpful.

However, that server block should really only handle traffic to cantusdatabase.org.

Correct, with the addition that is should only handle ssl traffic to cantusdatabase.org. We are saying we only want to really return 200 responses to https requests to cantusdatabase.org. If you request on port 80 and/or with a host alias (mass. or www.) we will redirect you, but anything else you come with, we are not going to respond.

I don't know if this is helpful? I've pasted two commands and their results below.
The first is a cURL request to the cantus database server, using the IP address, but spoofing the Host header to something nonsensical. You can see that the server actually responds with a 301 to https://cantusdatabase.org.
However, this isn't defined behaviour. The fact that it caught this isn't really by design -- it just caught the port 80 handler for http://cantusdatabase.org/ and returned that. It definitely didn't catch it by matching the server_name.  If your order of hostname config file parsing changes, it could actually land on another host handler that might not be as useful.
By setting a "catch-all" default server, you can actually route all non-matching hostnames to a dedicated "go away" handler. I've demonstrated it with the same request to the RISM Online server, and you can see that it simply closes the connection.
Cantus Database
❯ curl -vvv -H "Host: smooflydoo.org" http://<IP_ADDRESS>/
*   Trying <IP_ADDRESS>:80...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 80
> GET / HTTP/1.1
> Host: smooflydoo.org
> User-Agent: curl/8.4.0
> Accept: */*
>
< HTTP/1.1 301 Moved Permanently
< Server: nginx
< Date: Tue, 05 Mar 2024 15:37:08 GMT
< Content-Type: text/html
< Content-Length: 162
< Connection: keep-alive
< Location: https://cantusdatabase.org/
<

<title>301 Moved Permanently</title>

301 Moved Permanently

nginx

* Connection #0 to host <IP_ADDRESS> left intact

RISM Online
❯ curl -vvv -H "Host: smooflydoo.org" http://<IP_ADDRESS>/
*   Trying <IP_ADDRESS>:80...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 80
> GET / HTTP/1.1
> Host: smooflydoo.org
> User-Agent: curl/8.4.0
> Accept: */*
>
* Empty reply from server
* Closing connection
curl: (52) Empty reply from server

PS: If you look at your log files you'll also now see a request for the site "smooflydoo.org".

should we be adding an analogous line with [::]: elsewhere in our configuration wherever we have a listen?

No, because your server doesn't route IPv6. If you check the instances in the Arbutus OpenStack you'll see the difference between the servers that have IPv6, and those that don't.
So you can remove that bit from your config.


Pagination not working after deleting symbols

Version
Version: v0.0.90 (48865a0)
Latest Version:
v0.0.90
Description
After deleting symbols from the list of coins to trade, pagination does not work any more.
To Reproduce

Have at least 13 symbols in your list.
Remove one symbol from your list.
You will not be able to switch to the page containing your last symbol.

Expected Behaviours
One should still be able to cycle through the page of coins displayed, in order to be able to delete them.

Hello @Rayn0r
I understand that.
We have to count the number of the symbols from trailing-trade-cache instead of the global configuration.
Another possible solution is, we can remove the data from trailing-trade-cache so it will be reinitiated after applying the new monitoring symbols.
By doing one of these solutions, the pagination behavior will work as expected.
I will submit a PR to solve this when I have time.


Capturing a URL - unable to replay in ReplayWeb.page

Hi,
I'm trying to test the basic scenario of capturing a URL, and save it as a WARC file.
The code below kind of works, the file is created (see gist),  but when I try to open the WARC file in ReplayWeb.page it says "Archived Page Not Found"
Any ideas how to troubleshoot this?
Also, a quick follow-up question, is there a way to automatically fetch all the assets used by the page? (css, js, etc) My end goal is to fully capture a code-based artwork (like this one one), including network requests initiated by the piece.
My code:
const { WARCRecord, WARCSerializer } = require('warcio');
const fs = require('fs');

async function captureWebPage(url, warcFilename) {

    const { default: fetch } = await import('node-fetch');

    const warcStream = fs.createWriteStream(warcFilename);

    const response = await fetch(url);

    const warcInfo = await WARCRecord.createWARCInfo(
        {
            filename: warcFilename,
            warcVersion: 'WARC/1.1',
        },
        {
            software: 'warcio.js in node',
            description: `WARC file containing capture of ${url}`,
        }
    );

    const serializedWARCInfo = await WARCSerializer.serialize(warcInfo);
    warcStream.write(serializedWARCInfo);

    const warcRecord = await WARCRecord.create(
        {
            url,
            date: new Date().toISOString(),
            type: 'response',
            warcVersion: 'WARC/1.1',
            httpHeaders: response.headers.raw(),
        },
        response.body
    );

    const serializedRecord = await WARCSerializer.serialize(warcRecord);
    warcStream.write(serializedRecord);

    warcStream.end();

    console.log(`Captured WARC for ${url} and saved to ${warcFilename}`);
}

captureWebPage('https://arkivo.art', 'arkivo.warc');

You are missing the trailing slash, so its not a full URL, if you do:
captureWebPage('https://arkivo.art/', 'arkivo.warc');

It will work. (Note that copying gist will not work because it has \r stripped out).

Also, a quick follow-up question, is there a way to automatically fetch all the assets used by the page? (css, js, etc) My end goal is to fully capture a code-based artwork (like this one one), including network requests initiated by the piece.

This library is designed to be low-level tool for writing WARC files. To do what you want, you need to capture through the browser. We have several tools that do this:

ArchiveWeb.page - a browser extension which you can turn on while browsing, and you can then download the data as WARC or WACZ.
Browsertrix Crawler - an automated crawler for automatically crawling pages through the browser as a CLI tool.

It worked, thanks!
I need to do this programatically, from a node app, but I will investigate Browsertrix, thanks!

Also added a fix in #77 that will add a trailing slash if it's missing. Closing this as the original issue is answered.


Big File Not Download From /opt/traccar/modern From V5.8

Dear Anton,
Big file is not download from V5.8
Please see the screenshot: i have put a 1GB file in /opt/traccar/modern and current traccar V5.10

and when i hit http://<IP_ADDRESS>/backup.zip in my browser it show 500 - Server Error

Please note that upto V5.7 any kind of file download perfectly and has no problems.
Lastly my use case is: i have download mysqldump backup file from other vps for any kind of failure in my server.
Thanks

I don't think we can classify this as a bug. Also, you shouldn't need to do it this way.


feat: provide support for yarn v4

@antongolub In fact I observed the same problem with yarn 3.2.1, where the yarn.lock file is temporary downgraded from metadata v6 to v5 and two git urls do not have the double quotes around them. Apparently yarn itself is able to repair that automatically after but for some time I can see git client reporting the file as being modified, causing some confusions.
Any change we can address this?

We're moving our (QIWI) monorepos to yarn v4, so yarn-audit-fix will be upgraded soon


Error on Install: node-gyp rebuild

Getting an error when trying to install into a new project.
$ npm install --save-dev gulp-responsive

><EMAIL_ADDRESS>install /PROJECT_PATH/node_modules/gulp-responsive/node_modules/sharp
> node-gyp rebuild

Package vips was not found in the pkg-config search path.
Perhaps you should add the directory containing `vips.pc'
to the PKG_CONFIG_PATH environment variable
No package 'vips' found
gyp: Call to 'PKG_CONFIG_PATH="/usr/lib/pkgconfig:/usr/local/Library/ENV/pkgconfig/10.10:$PKG_CONFIG_PATH:/usr/local/lib/pkgconfig:/usr/lib/pkgconfig" pkg-config --libs vips' returned exit status 1. while trying to load binding.gyp
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:357:16)
gyp ERR! stack     at ChildProcess.emit (events.js:110:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:1074:12)
gyp ERR! System Darwin 14.5.0
gyp ERR! command "node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /PROJECT_PATH/node_modules/gulp-responsive/node_modules/sharp
gyp ERR! node -v v0.12.7
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
npm ERR! Darwin 14.5.0
npm ERR! argv "node" "/usr/local/bin/npm" "install" "--save-dev" "gulp-responsive"
npm ERR! node v0.12.7
npm ERR! npm  v2.14.2
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the sharp package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls sharp
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /PROJECT_PATH/npm-debug.log

Currently using node v0.12.7 and npm 2.14.2 on Mac 10.10.5 Yosemite.

Same problem io.js 1.6.4, npm 2.7.5 on Windows

Hi @jhough10 @rvboris please follow installation documentation of sharp library http://sharp.dimens.io/en/stable/install/
Notify me if it does not solve your problem.

@mahnunchik works like a champ now.
If using a Mac I ran the following command brew install homebrew/science/vips --with-webp --with-graphicsmagick and it resolved the issue with being able to install gulp-responsive.

@jhough10 great! :fireworks:

:+1:


User credentials request is not visible from commandline

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
I'd prefer not to paste my company's Bitbucket repo URL here, but basically use a pypy-dependencies entry with some { git = "https://repo.mycompany.com" } spec that will require username + password authentication and then
pixi install

For example, this is my (redacted) pyproject.toml:
[build-system]
requires = ["pdm-backend"]
build-backend = "pdm.backend"

[project]
name = "my_project"
version = "0.0.1"
authors = [
  { name="Cars Chandler"<EMAIL_ADDRESS>},
]
description = "N/A"
readme = "README.md"
requires-python = ">=3.12"
classifiers = [
    "Programming Language :: Python :: 3",
    "Operating System :: OS Independent",
]

[project.urls]
Homepage = "https://repo.mycompanyname.com/users/myusername/repos/my_project"

[tool.pixi.project]
channels = ["conda-forge"]
platforms = ["linux-64"]

[tool.pixi.pypi-dependencies]
my_project = { path = ".", editable = true }
my_lib = { git = "https://repo.mycompanyname.com/users/myusername/repos/my_lib.git" }

[tool.pixi.dependencies]
python = ">=3.12"
dash = "*"
dill = "*"
netcdf4 = "*"
numpy = "*"
pandas = "*"
pandas-stubs = "*"
plotly = "*"
pyarrow = "*" # required for upcoming pandas 3.0
python-kaleido = "*" # for plotly figure image exports
scipy = "*"
xarray = "*"

[tool.pixi.feature.dev.dependencies]
ipython = "*"
icecream = "*"
matplotlib = "*"

[tool.pixi.environments]
dev = ["dev"]

Issue description
My company has some custom SSL certificate for their Bitbucket server that I have to use and it typically prompts me at the terminal with a Username: followed by a Password: input field whenever I connect to it via git. The loading bar graphics of pixi (they look awesome by the way) prevent me from ever seeing these fields. They are still there, and when I enter my username and password blindly, the authentication does succeed, but it would be nice if there were a way to pause those bars while the authentication requests show. I'm aware that pixi auth is intended to make this a non-issue, and while it isn't working for me, I think it would still be nice to show the credentials requests to users who either aren't aware of pixi auth, aren't able to use it successfully, or just prefer not to.
Expected behavior
Loading bars show, then are removed once a credential request is printed to stdout, then they resume after credentials are collected.

We currently don't expect interaction with the installer/solver during pixi install so we would have to support that. Put it on the backlog. @wolfv your generally good at these issue, could you drop your thoughts on the matter?

That's certainly a fair assumption and it's not a common use case at all so I understand. If only my company would set up SSH on their Bitbucket 🤧.

@carschandler do you have some way of reproducing the behavior with a publicly accessible endpoint? That would help :)
Do I need to manually add a SSL certificate in some way that it asks for a password/

@wolfv I've been thinking of the best way to go about this... I will try to find a public example that presents this behavior. Apologies I didn't have it in the original post!

@wolfv okay, I found a simple way. Just use a private GitHub repo (doesn't actually have to be a Python package) and use the https link to it. You can git clone this link to see what the desired behavior would show, and if you use this snippet in a pixi.toml, then it will show what I'm talking about.
[pypi-dependencies]
my_library = { git = "https://github.com/user/my_library.git" }

Everything will go fine, and eventually it will hang on the cloning https://github.com/user/my_library.git step since the credential request has actually been printed, but was just instantly covered by the loading bars (I think). You can type in your username blindly, hit enter, and paste in a personal access token, and you'll authenticate successfully.


Integrate the contribution information into the documentation site

@react-learner is now working on it with his work on the documentation contribution documentation :)

I think it's done?

@react-learner, I think it's done?%

If so, please check the appropriate box at https://github.com/meilisearch/devrel/issues/17 after closing this issue.

not super self-explanatory, but a lot of work was done to improve contributions within the DevRel team in the past months.
We're not aware of any repo missing guidelines for contributors.


Consistently use HLO_StaticShapeTensor

Some ops in the StableHLO dialect have "load-bearing" result shapes, meaning that these shapes cannot be inferred from op's operands, attributes and regions.
For these ops, the current design philosophy involves one of the two approaches:

The shape is specified statically in the result type, e.g. see
ReshapeOp.
The shape is passed dynamically as an operand (or can be computed
indirectly from shape-related operands), e.g. see DynamicReshapeOp.

In the future (#8), we want to have one unified approach in these situations, but we're not there yet, and this PR is about an incremental improvement to the state of the art.
One important piece of the design philosophy for 1) is that these ops need to be restricted to produce HLO_StaticShapeTensor, otherwise it is unclear what their semantics is.
%1 = "stablehlo.reshape"(%0) : (tensor<2x4xf32>) -> tensor<?x?xf32>
For example, the code snippet above doesn't make sense because we need an actual shape to execute the op - either passed at compile time or passed at runtime - and neither is happening here.
When working on #622, I discovered that there are some ops which fall into 1) but don't have their result types restricted to be static: InfeedOp, IotaOp, RecvOp and RngBitGeneratorOp.
This means that StableHLO producers can currently produce these ops with dynamic result types, which would be unsound. This PR fixes this issue by changing the result types to HLO_StaticShapeTensor or equivalent.

Good questions! broadcast_in_dim already has let results = (outs HLO_StaticShapeTensor), so it's already handled. custom_call will be handled in #851.


Fix detection of SLE11 SP3 Update channels (bsc#938478)

maintenance team just switched to a different tag, which breaks
the existing check. Adjust.

+1
Took a look the repositories. The new tags seem to work for me.

@dguitarbite Already successfully tested the patch  myself as well

@dirkmueller Yes, Just mentioned the reason for +1. I think we need another +1 before merging.

its a backport, one +1 should be enough :-)


About inverse trigonometric functions

I think the differential formulas of the inverse trigonometric functions are incorrect. I'm not familiar with F#, but Calculus.fs needs to be modified as follows:
    let rec differentiate symbol = function
        ...
        | Function (Asin, x) -> (1Q / sqrt(1Q - pow x 2Q)) * (differentiate symbol x)
        | Function (Acos, x) -> (-1Q / sqrt(1Q - pow x 2Q)) * (differentiate symbol x)
        | Function (Atan, x) -> (1Q / (1Q + pow x 2Q)) * (differentiate symbol x)

It seems like you are right. Maybe you want to send PR with fix and tests?


Required Workflow Support

Github introduced Org Required Workflow that enables a Github Action to be defined in one place and applied across applicable repos in the entire org.
Here is the link for the  API documentation for the same:
https://docs.github.com/en/rest/actions/required-workflows?apiVersion=2022-11-28
Being able to call these API's from this repo would enable progress on the below issue in Github Terraform provider:
https://github.com/integrations/terraform-provider-github/issues/1475

Thank you, @avivek - do you want to provide a PR to resolve this issue, or would you like me to open this up to other contributors to this repo?

Hi @gmlewis,
Thanks for getting this merged
Is there a release planned??
We can use this in the terraform github provider once released.

Yes. I can work on putting together a release shortly.

@avivek - this should now be available in: https://github.com/google/go-github/releases/tag/v51.0.0


[Help] How to use Import inside worker

I'm trying to use import within the web worker in the rendering process, but it doesn't do anything. I didn't print the test console. If I remove the import it works.
myWorkerScript:

import { BasicPitch, addPitchBendsToNoteEvents, noteFramesToTime, outputToNotesPoly } from '@spotify/basic-pitch';

console.log('test');

self.addEventListener('message', async (event) => {
  const message = event.data;
  const midiNotes = await getMidNotes(message.decodedSong);

  self.postMessage({
    status: 'complete',
    data: midiNotes
  });
});

Is there a way for it to work?

Could you please offer the minimum demo repo?


adding ability to annotate collections with @XmlJavaTypeAdapter

@Josephblt as you asked i added @XmlJavaTypeAdapter support for collections, please check and let me know if it works for you.

Thanks, @treblereel. I'll test it right away.


[Feature request] Implement scrolling for an arbitrary window

Basically title. Currently, as far as I can tell, only supported for current window.
Why: to scroll popup/preview/qf/diagnostics windows without having to move the cursor there. Especially annoying if the hover() documentation does not fit in the popup.
I initially tried to do something like vim.fn.win_execute(winid, "lua require'nescroll'.scroll(...)"), but that only scrolls the window for one line and then keeps scrolling the current one. From the looks of it, implementing this would require refactoring some globals into per-winid tables and calling vim.fn.* (such as winline()) inside win_execute context - which is a handful, but possible.
I'm just moving from Vim to Neovim and learning Lua, but I would be happy to take a crack at it in a PR. However, if the feature is not needed or impossible for some reason I can't see, please let me know.

Made a rough draft that mostly works, though not tested with all possible configs and has a bug with gg: https://github.com/karb94/neoscroll.nvim/compare/master...alfaix:alfaix/arbitrary-winid
Discovered a couple of bugs:

with so_scope = wo, scrolloff can be -1, which causes very annoying off-by-1 errors (e.g., zt causes the current line to disappear)
I think currently the timer is always on and does nothing most of the time, but still takes resources. Called timer:stop() in stop_scrolling()
closing window while scrolling sometimes causes errors, sometimes keeps scrolling the new current window. In the new implementation it still scrolls the current window (though maybe it's worth patching that out), but if an error occurs it doesn't get spammed a billion times over.

Please let me know if a PR would be ok

I also think you should be able to apply scroll() to any window. As you said this requires quite a bit of refactoring and no one was really asking for it so I focused on other features. Right now my priority is to build tests so that I don't miss any bugs like some of the ones you mentioned moving forward. Once I set the tests and fix all bugs (including the ones you mentioned) I will be happy to help with/accept a PR on this.
Regarding the bugs you've identified, can you clarify what is wrong with the timer? stop_scrolling() already calls timer:stop() as far as I can tell.
Thank you for all your help.

This is now implemented with the winid key in the opts argument of all scrolling functions. Closing.


plugins/conform: init + test

Adds support for the conform-nvim plugin.
I could not test this due to conform-nvim not being in nixpkgs-unstable yet.
I'll amend further changes.

@evccyr You may now rebase on the main branch. The flake.lock has been updated and conform.nvim is now available.

You include the file as conform-nvim.nix in the plugins/default.nix file, while you renamed the file to conform.nix

You include the file as conform-nvim.nix in the plugins/default.nix file, while you renamed the file to conform.nix

I'm just struggling with the logLevel for now. How should I do that?


During project/unit put, add staging records to delete any child tables that no longer are attached to the primary table

@alexandradumitru22 is this still an issue?

@MichaelTaylor3D solved


Re-use existing branches

Closes #95

As part of simple due diligence I pulled the branch and ran the tests locally. For me I see issues with test/internals/github-integration.js.
I am looking further.

For me I see issues with test/internals/github-integration.js.

These tests need a valid GITHUB_TOKEN exposed. They get skipped if there is no token at all - maybe you had one for some other GH instance?

@ghinks as these tests are passing for me locally, and also in CI, I'll assume that all is fine and merge this - ping me if you want to dig deeper to see what's going in there?

:tada: This PR is included in version 0.9.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Find method can return slice of structs or pointers

Describe the feature
Some libraries accept slice of structs and some use slice of pointers, but find method can only generate slice of pointers.
hopefully the return value can be changed at configuration or execution stage.
Motivation

Related Issues

@ddman #261

What was the resolution on this ?
Is there any way i can set a configuration for it to not generate only a slice of pointers for the return type ?


Toast is not always showing in RN0.59.9

Toast was working fine. However, recently I upgraded to RN0.59.9 and it does not show every time. Sometimes it works fine and sometimes it does not. I am aware of that if the debugger is on, toast may not work as expected. But in release mode, it does not work perfectly also

I'm having this issue too using RN 0.59.10. Sometimes it delays a lot to show and stay for waaay less than the duration before disappearing.

I have been experiencing the same thing. My toast waits a while, and then flashes in and out.
Although, it works in other places. The behaviour has become very inconsistent.

I met the same issue, I use Toast.show in a useEffect but it delayed too long that drives me crazy...
  useEffect(() => {
    console.log("Hey I'm in useEffect!");
    Toast.show({
      text: 'Wrong password!',
      buttonText: 'Okay',
      duration: 3000,
      type: 'warning',
    });
  }, [errors]);

Looks like it's because my laptop and my android phone are not at the same time. I manually adjusted the time of my android phone, the delay seems shortened...

Hi @mjm918 , please provide minimal code reproduction on snack for the same.
Thanks

I created this issue a year ago. I didn't receive any help back then so decided to stop using it. So sorry I don't have any code for this


Be able to import and use structure maps that have been defined in separate files

Is your feature request related to a problem? Please describe.
Our program requires the creation of a 'newly diagnosed client' care plan from separate points in our app. For example, when a patient is first registered, they are assigned a the care plan. Similarly, when potential patient has an HIV test that comes back positive, they should be assigned this same care plan. It's frustrating that I can't just import the same care plan and use it. In addition, for maintainability's sake, it will be good to define this care plan once and import it when needed.
Describe the solution you'd like
I would like to import the structure map as shown here and use it's groups.
Describe alternatives you've considered
The only alternative is duplicating the code in separate files.
Additional context
N/A
Would you like to work on the issue?
Not sure but your help would be greatly appreciated

@f-odhiambo Kindly take a look

The fhir StructureMapUtilities loads the imported map from IWorkerContext. The SDK ResourceMapper currently uses SimpleWorkerContext for worker support. The worker allows caching resources including questionnaires, structuremaps etc. We can try calling public void cacheResource(Resource r) on SimpleWorkerContext passed to application and cache all structure-maps which inturn would be used internally when a group is referenced from any other strcuture-map
Check flow from ResourceMapper.extractByStructureMap, StructureMapUtilities.findMatchingMaps, SimpleWorkerContext.listTransforms, BaseWorkerContext.getTransform, BaseWorkerContext.cacheResource

I'll check the flow and respond accordingly

I'll check the flow and respond accordingly

@KhumboLihonga , are these changes needed based on the workaround provided by @maimoonak ?

@sevenreup could you check this out?


Incompatibility: FontFixer

Opening the portable terminal is fine, until you go to open the store, then the console is spammed with:
[Warning: Unity Log] Unable to add the requested character to font asset [IBM 3270]'s atlas texture. Please make the texture [IBM 3270 Atlas] readable.
The spam bogs down the game to 1 FPS unless you close out of the store
AdvancedCompany v1.1.1
FontFixer v1.0.0

FontFixer shouldnt replace my fonts or add all necessary characters for my terminal to work. I will add it to the incompability list.


Release Google.Cloud.OsConfig.V1 version 2.2.0

Changes in this release:
No API surface changes; just dependency updates.

Created release for Google.Cloud.OsConfig.V1-2.2.0


Make job queue working outside of browser

Task linked: SALEOR-1259 When no browser is present (window === undefined), SDK attempts to restore jobs from localStorage


refactor: make compatible with grammY

My apologies. I accidentally submitted this PR to your repository. Please feel free to disregard it.


GoLang based examples

Can I  golang based examples in this repo. Can you list the workflow.

yes
you can contribute, there is no restriction on language all the contributions are welcome

Just post the example in the assigned folder and if no folder is of type please create a folder named Other

@kshitizsaini113 great thanks

Will look forward towards your contributions.


sending api_key through http header with basic authentication

Adding SampleCode.java: A very basic java application that connects to a running web service and computes a simple addition. I added to make testing java client easier.

@wangping25 your suggestions are addressed