Menubar: Incorrect trigger interaction on menu close

Describe the bug
When closing the menu there are two cases where the menu will close visually but stay in a state as if the menu is open.

Clicking on the trigger will continue to open menus when hovering over other triggers.
Closing the menu with Escape will stop keyboard navigation and highlight other triggers, without opening the menu.

To Reproduce
Steps to reproduce the behavior:
First case;

Click on a trigger to open its menu.
Click on the trigger again to close the menu.
Hover other triggers; it will open their menus.

Second case;

Click on a trigger to open its menu.
Press Escape to close the menu.
Hover other triggers; Will highlight items on hover, but keyboard navigation and interaction doesn't work.

Expected behavior
For the first case, after closing the menu on the trigger, other triggers should not open when hovered.
For the second case, it should close the menu, and keep keyboard navigation functioning.
Screenshots
First case

Second case

Desktop (please complete the following information):

OS: macOS 10.14
Browser Brave
Version 1.57.64 (Chromium 116.0.5845.188)

Additional context
See Radix UI Menubar as an example of expected behavior.

Thanks again!


[#1674] Add basic DOCKER_ENV logging for run_demo

signed-off-by: Thomas Diesler<EMAIL_ADDRESS>

Codecov Report

Merging #1675 (fd0e51f) into main (3774e85) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1675   +/-   ##
=======================================
  Coverage   95.50%   95.50%
=======================================
  Files         528      528
  Lines       32766    32766
=======================================
  Hits        31293    31293
  Misses       1473     1473


(DISCUSSION) Plan for .NET 5 support?

Is there a plan for .NET 5 support?
If this has been discussed for this library, what does the timeline look?

Hello @Reapism ,
The library already supports .NET 5 and probably every upcoming framework release.
The library support .NET Standard 2.1 which is compatible with .NET 5.
Let me know if that answers correctly to your question.
Best Regards,
Jon

Hi @JonathanMagnan
That is awesome! When I was looking at the dependencies on the nugget manager, I didnt see .NET 5.
Today, I learned. Thank you!


Long port numbers should not use commas

(From TestFlight)
Port numbers over three digits long are displaying with thousand comma separators.  See attached image

Fixed


ForNpgsqlHasColumnType missing from version 2.0.0

ForNpgsqlHasColumnType() on PropertyBuilder is missing when I updated to 2.0.0. I can't see a suitable alternative in intellisense. Is this a bug?

This is a general change in EF Core 2.0.0, not related to Npgsql: see https://github.com/aspnet/EntityFrameworkCore/pull/8600 and https://github.com/aspnet/EntityFrameworkCore/issues/7166 for background and the new way of doing things.


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

merged now


Random crash on FontContext::lock() on app exit.

Note: Noticed on deviceFonts-android and text-color branch. Try to repro this on master.
Both the feature branches are modifying any of the code path responsible to call FontContext::lock.
Also, if reproduced update with the stacktrace.

On a first look it seems the order of destructor calls for TileManager and FontContext is implicitly given by the order they appear in tangram.cpp - So the global FontContext could be destroyed while TileManager is still waiting for it's workers to shut down.
http://stackoverflow.com/questions/469597/destruction-order-of-static-objects-in-c

I just removed m_ftContext in deviceFonts-android feature branch, because now its not needed there. I have not been able to recreate it, but its random :|.

Any related crashes since the merge on this issue?

Nothing I am aware of. We can mark it resolved for the time being.
On Sep 10, 2015 2:58 PM, "Karim Naaji"<EMAIL_ADDRESS>wrote:

Any related crashes since the merge on this issue?
—
Reply to this email directly or view it on GitHub
https://github.com/tangrams/tangram-es/issues/252#issuecomment-139344950
.

Ok, there is other issues but it doesn't seem to be related to this specific one.


Version pinning for plugins

Hi, is there a way to pin specific plugin versions?
Thanks

Not yet. Already tracking this future feature here:
https://github.com/mattmc3/antidote/issues/1
For now, if this is something that's absolutely essential, you could maintain a fork/branch of the commit you want as a workaround.


export Hebrew or Russian text from Django admin to CSV file

Hi all,
I use  "django-import-export/django-import-export" in my project and when I try to export Hebrew or Russian text from Django admin to CSV file I receive unknown text (For example:
"יום העצמאות" from Django will be represented in CSV as "׳™׳•׳ ׳”׳¢׳¦׳ž׳ ׳•׳×"). Import work well. I checked to_encoding and from_encoding it is 'utf-8' format. Phython 3.5.2, Djando 1.10.1.
Thank you,
Alexander

@AlexandrKO I use the same configuration and I have no problem with it. I use openoffice for edit csv-file.


Try training with the VGG Face dataset

Links to the images can be found at
http://www.robots.ox.ac.uk/~vgg/data/vgg_face/
A script to download the images can be found at
https://github.com/davidsandberg/facenet/blob/master/facenet/src/download_vgg_face_dataset.py

Train with the MSCeleb dataset instead.


react-native template typescript doesn't copy the template directory

When creating a new react-native project with typescript as the template, it creates a template directory inside the project, but doesn't move the contents of that directory, in the root. So we would have a react-native directory without typescript, and then template directory inside it.
React Native version:
System:
OS: Windows 10
CPU: (8) x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
Memory: 7.13 GB / 15.89 GB
Binaries:
    Node: 10.16.0 - C:\Program Files\nodejs\node.EXE
    npm: 6.9.0 - C:\Program Files\nodejs\npm.CMD
SDKs:
    Android SDK:
    API Levels: 23, 24, 25, 26, 27, 28, 29
    Build Tools: 28.0.3, 29.0.0
    System Images: android-28 \| Google Play Intel x86 Atom
IDEs:
    Android Studio: Version  <IP_ADDRESS> AI-182.5<IP_ADDRESS>99772
npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.2 => 0.61.2
--

Steps To Reproduce

react-native init PROJECT_NAME --template typescript

Describe what you expected to happen:
Expected to have a project with typescript template. Should probably move the contents of template directory to the root directory and then do a npm install to install the new dependencies.
screenshot:

Seeing this also.

You're using the wrong command for creating new project with the template. Please see https://github.com/react-native-community/react-native-template-typescript/blob/master/README.md#note-on-the-legacy-clip and if you have more issues open an issue there.

@radko93 thanks
So this should probably be changed:
https://facebook.github.io/react-native/docs/0.60/typescript

@ipinlnd there's a PR but it's not merged


Support for MacOS X Desktop application

As I need MixPanel capabilities in a Desktop application, I did modify the iOS code to be able to compile and execute under MacOSX. The first test are positive but review is certainly needed to explore all MixPanel features. Following points have to be noted

distinctID is the serial number of the desktop machine and is taken using IOKit method
depending if the application is sandboxed or not the archive location may differ
All UIApplication related notifications have been "transferred" to their NSApplication couterparts
Currently the code considers background handling, but it may be more suited to not stop the flush while in the background. This part may need more work
This is a first version of this support. As code changes start to be "big" , I am asking for a pull request and ad a review process before going further, in order to get feedback as soon as possible in the process.
thanks

See #12


Add Pull Request creators

Currently working on branch feature/add-prs. I found an issue with depaginate, which now needs to be updated in each dep. Fun times.

Done! Ping @gr2m


exec alias

Alias for shell command #15 .
I used ctx.forward(), since an approach as presented in https://stackoverflow.com/questions/46641928/python-click-multiple-command-names did not work.

I'm not sure whats the issue with "SonarCloud". I just added a spinner when creating a project. For me it works.

Strange. You may disable sonarcloud for this line with #NOSONAR


chore(release): v0.10.6 🎉

:robot: I have created a release beep boop
0.10.6 (2023-02-10)
Bug Fixes

pkg: emit declarations in dist (79215a0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/declapract/releases/tag/v0.10.6 :sunflower:


Chore/update team permissions

Fills out some initial ideas for teams for code review delegation.

I've been meaning to ask, is there a peribolos command line validator?

I've been meaning to ask, is there a peribolos command line validator?

It looks like Knative has a python script.


"defaultLangAnalyzer": "ambar_de" doesn't work

Hi there,
first of all thank you for all the work that was put in here! This looks truly epic!
The installation was ok (some minor hiccups) but now I have a larger problem:
Changing the Language Analyzer to German somehow does not work...
I tried:
ambar_de
ambar-de
ambar_deu
After every change I restarted the service but it still says that the Language Analyzer would be English (WEB GUI -> Account).
Console output:
Docker version 17.07.0-ce, build 8784753
docker-compose version 1.16.1, build 6d1ac219
Stopping Ambar...
f6f2a25b953b
Crawlers containers removed
f6da9687efae
Pipeline containers removed
Stopping ambar_frontend_1     ... done
Stopping ambar_webapi_1       ... done
Stopping ambar_db_1           ... done
Stopping ambar_proxy_1        ... done
Stopping ambar_es_1           ... done
Stopping ambar_rabbit_1       ... done
Stopping ambar_webapi-cache_1 ... done
Removing ambar_frontend_1     ... done
Removing ambar_webapi_1       ... done
Removing ambar_db_1           ... done
Removing ambar_proxy_1        ... done
Removing ambar_es_1           ... done
Removing ambar_rabbit_1       ... done
Removing ambar_webapi-cache_1 ... done
Removing network ambar_internal_network
Ambar is stopped
vm.max_map_count = 262144
net.ipv4.ip_local_port_range = 15000 61000
net.ipv4.tcp_fin_timeout = 30
net.core.somaxconn = 1024
net.core.netdev_max_backlog = 2000
net.ipv4.tcp_max_syn_backlog = 2048
Creating network "ambar_internal_network" with the default driver
Creating ambar_rabbit_1 ...
Creating ambar_webapi-cache_1 ...
Creating ambar_proxy_1 ...
Creating ambar_db_1 ...
Creating ambar_rabbit_1
Creating ambar_es_1 ...
Creating ambar_webapi-cache_1
Creating ambar_proxy_1
Creating ambar_db_1
Creating ambar_es_1 ... done
Creating ambar_webapi_1 ...
Creating ambar_webapi_1 ... done
Creating ambar_frontend_1 ...
Creating ambar_frontend_1 ... done
Waiting for Ambar to start...
Ambar is on http://<IP_ADDRESS>:80

Hi!
After changing default lang. analyzer in your config.json you have to reset Ambar to apply changes: sudo ./ambar.py reset - this command will recreate the index with specified lang. analyzer.

Are you sure?
I just performed this command and Ambar is completely gone...

Yes, this command recreates your index.

So I reinstalled Ambar? Because I could not start it anymore...
I then changed the language to "ambar_de" before starting it the first time and it worked!
Thx!


Wrong keyboard shortcut

OS
macOS 14.0
LinearMouse
v0.1.9-beta.1
Describe the bug
I wanted to map button 3 and 4 as Control + leftArrow and Control + rightArrow, but instead it's mapped as Command + leftArrow and Command + rightArrow
To reproduce
No response
Expected behavior
No response
Anything else?
No response

Apologies for the delayed response. However, I haven't reproduced this problem.
Just to clarify, are you referring to whether LinearMouse has incorrectly recorded the keyboard shortcuts, or if it has recorded the correct shortcuts but is simulating the wrong ones?
By any chance, have you swapped the control and command keys in the System Settings or through any other keyboard utilities?

LinearMouse ha correctly recorded the keyboard shortcuts, but is simulating the wrong ones. I recorded Control + leftArrow and Control + rightArrow but is simulating Command + leftArrow and Command + rightArrow. I didn't change the default setup of my keyboard, and if i use the shortcuts from the keyboard, they works just right

Could you please reproduce this issue and upload the logs exported in General settings here?
BTW, is your keyboard layout QWERTY?

Sure. i have an italian qwerty
LinearMouse.log

Thank you! I can confirm this issue and will fix it soon.
In the meantime, you may select "Move Left a Space" and "Move Right a Space" actions if they fulfill your requirements.

Thank you! i'm so dumb i didn't notice there were the option without a keyboard shortcut but i'm glad i could found a bug that can be fixed for someone else. Thank for your work!


Use Java 8 Instant for time mapping

We should support mapping to both Instant and LocalDate/LocalDateTime. This might be a bit complicated since the mapper must be able to handle the case when the times of the database is zone-specific and the java world is zone-less. One way of supporting this could be to store the database timezone as a member in the TypeMapper class and use it when converting back to the database.

Proposed mappings at the end of the article http://www.oracle.com/technetwork/articles/java/jf14-date-time-2125367.html


Added support for context when reporting statuses

Build ouptut:
# Executing command line:
/usr/lib/jvm/java-7-oracle/bin/java
-Xms2048M
-Xmx2048M
-Xss6M
-XX:MaxPermSize=512M
-jar
/home/travis/.sbt/launchers/0.13.0-RC5/sbt-launch.jar
warn
"set javaHome in ThisBuild := Some(file("/usr/lib/jvm/java-7-oracle"))"
info
++TRAVIS_SCALA_VERSION
test

Any idea who and how is injecting the warn and the info parameters?


Migrate Billing related components to tailwind/shadcn - Part 1

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR primarily refactors the onboarding and billing components, improving the UI structure and enhancing the modal functionalities. It replaces the OnboardingTitle with TitleAndDescription, streamlining the code and updating various components for better usability and consistency.
Detailed summary

Replaced <OnboardingTitle> with <TitleAndDescription> in several components.
Updated OnboardingModal to remove redundant onClose props.
Refactored billing alert components to use new UI elements.
Enhanced the layout of forms and alerts for better UX.
Improved state management for modal visibility.
Added new utility functions for account and billing data stubs.
Updated import statements for consistency and clarity.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#4879  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MananTank and the rest of your teammates on  Graphite


Reordered images for restage prompt

Prompt to restage app comes after the application connection is created. This PR reorders this images and makes slight changes to the instructions to follow the actual user path.

giving @scottdangelo and @justinmccoy a chance to verify, too.


[Event Request] Codeunit 7307 "Whse.-Activity-Register" - procedure RegisterWhseItemTrkgLine - OnRegisterWhseItemTrkgLineOnAfterGetWhseItemTrkgSetup

Please add an Event OnRegisterWhseItemTrkgLineOnAfterGetWhseItemTrkgSetup in RegisterWhseItemTrkgLine in Codeunit 7307 "Whse.-Activity-Register"
local procedure RegisterWhseItemTrkgLine(WhseActivLine2: Record "Warehouse Activity Line")
    var
        ProdOrderComp: Record "Prod. Order Component";
        AssemblyLine: Record "Assembly Line";
        JobPlanningLineRec: Record "Job Planning Line";
        WhseShptLine: Record "Warehouse Shipment Line";
        QtyToRegisterBase: Decimal;
        DueDate: Date;
        NextEntryNo: Integer;
        WhseDocType2: Enum "Warehouse Activity Document Type";
        NeedRegisterWhseItemTrkgLine: Boolean;
        IsHandled: Boolean; <-- NEW VAR
    begin
        with WhseActivLine2 do begin
            if (("Whse. Document Type" in
                 ["Whse. Document Type"::Shipment, "Whse. Document Type"::"Internal Pick",
                  "Whse. Document Type"::Production, "Whse. Document Type"::Assembly, "Whse. Document Type"::"Internal Put-away", "Whse. Document Type"::Job]) and
                ("Action Type" <> "Action Type"::Take) and ("Breakbulk No." = 0)) or
               (("Whse. Document Type" = "Whse. Document Type"::Receipt) and ("Action Type" <> "Action Type"::Place))
            then
                NeedRegisterWhseItemTrkgLine := true;

            if ("Activity Type" = "Activity Type"::"Invt. Movement") and ("Action Type" <> "Action Type"::Take) and
               ("Source Document" in ["Source Document"::"Prod. Consumption", "Source Document"::"Assembly Consumption", "Source Document"::"Job Usage"])
            then
                NeedRegisterWhseItemTrkgLine := true;

            if not NeedRegisterWhseItemTrkgLine then
                exit;
        end;

        if not ItemTrackingMgt.GetWhseItemTrkgSetup(WhseActivLine2."Item No.") then
            exit;

        IsHandled: false;
        OnRegisterWhseItemTrkgLineOnAfterGetWhseItemTrkgSetup(WhseActivLine2, DummyItemTrackingSetup, IsHandled); <-- NEW EVENT
        IF IsHandled then begin
            exit;
        end;

        QtyToRegisterBase := InitTempTrackingSpecification(WhseActivLine2, TempTrackingSpecification);

        ....

       end;
    end;

    [IntegrationEvent(false, false)]
    local procedure OnRegisterWhseItemTrkgLineOnAfterGetWhseItemTrkgSetup(WhseActivLine: Record "Warehouse Activity Line"; var WhseItemTrackingSetup: Record "Item Tracking Setup"; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 42323.


[NTR] restrictive pulmonary disease

Preferred term label:
restrictive pulmonary disease
Synonyms
restrictive lung disease
NCIT:C91762 (exact synonym)
Definition (free text, please give PubMed ID, if applicable, in format PMID:#######)
Decreased lung volume and inadequate ventilation due to parenchymal lung disorders (e.g., interstitial pulmonary fibrosis) or extrapulmonary disorders (e.g., scoliosis). Patients present with shortness of breath and cough. NCIT:C91762
Parent term (use OLS, or your favorite ontology browser)
lung disease MONDO:0005275
Children terms (if applicable) Should any existing terms that should be moved underneath this new proposed term?
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here

Hello @Vineyardintern
I'm Sabrina, a new curator who recently joined the Mondo team.
We will address your issue as soon as possible. Thank you for your patience.
If this is a high priority, please let me know and we can prioritize it. Thank you!

@nicolevasilevsky
I added the NCIT term as definition dbxref, as an an exact synonym and also as a dbxref
In terms of synonyms and dbxrefs there are a few more to add please see below and the diff and let me know if this is the correct way to do it

NCIT:C91762
https://www.ebi.ac.uk/ols/ontologies/ncit/terms?iri=http%3A%2F%2Fpurl.obolibrary.org%2Fobo%2FNCIT_C91762
Added NCIT:C91762 to the definition as a dbxref, to the exact synonym term as a dbxref and also as a separate annotation dbxref with source MONDO:equivalentTo

In the OLS link above there is an NCIT dbxref to UMLS:C0085581.
Added UMLS:C0085581 as an annotation dbxref with source MONDO:equivalentTo. I was a bit uncertain if in this instance I should also add source NCIT:C91762 but I did because that's where I found the cross reference to UMLS.

SNOMED SCTID:36485005
https://www.ebi.ac.uk/ols/ontologies/snomed/terms?iri=http%3A%2F%2Fsnomed.info%2Fid%2F36485005&viewMode=All&siblings=false
Added SCTID:36485005 to the exact synonym term as a dbxref and also as a separate annotation dbxref with source MONDO:equivalentTo

In OLS the SNOMED term above has a dbxref to ICD10CM:J98.4 Other disorders of lung.
https://www.icd10data.com/ICD10CM/Codes/J00-J99/J96-J99/J98-/J98.4#:~:text=2021 ICD-10-CM Diagnosis,4%3A Other disorders of lung
Restrictive lung disease is shown under Approximate Synonyms
I am not really sure how to represent this given that it is not an exact synonym but it falls under an ICD grouping term that contains this disease. For now I 've added ICD10CM:J98.4 as a separate annotation dbxref with source SCTID:36485005

@nicolevasilevsky @matentzn
The PR failed QC https://github.com/monarch-initiative/mondo/pull/3721/checks?check_run_id=3732031973


fixed day count

How Many Days Until Halloween? PR Request Template
Please note: Has this feature already been added? Sometimes, duplicate pull requests happen. It's worth checking the pull requests and issue page to see if the change you are requesting has already been made.
Descriptive name.
Your pull request should have a descriptive name.
Type of Change was Made?
What type of change does your code introduce? After creating the PR, tick the checkboxes that apply.

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Description of the Change Being Made.
It's helpful to outline what changes were made to which files so that I have an idea of what will be involved in reviewing the code and—hopefully—merging it into the codebase.
Issue Number
If your pull request is related to a specific issue, please included it in your description and or pull request name. It helps to keep changes linked. Any issues that are referenced in pull requests become part of the discussion history of the issue.
Potential Performance Issues
Does the PR have a potential impact on performance on the codebase? If so, to what degree and why does the PR warrant the performance hit?
Tests/Checks
What tests were conducted to ensure the PR functions have no impact on previous functionalities of the code base?
New Dependencies
Have new dependencies been introduced? Please list them with links to documentation and add installation steps to the README.

Hello. Looks like this issue was completed on a previous pull request (I'm trying to go in order, oldest to newest). Feel free to make another addition or correction


Grabbing other tags

It would be handy if Sveld could grab any other present @tags in the JSDoc comment block and add them to the props exported per entity found.  We would like to use things like @see to point to external docs and also use things like custom @childOf and @parentOf to help show relationships.
FYI we are using Sveld through vite-plugin-sveld on a svelte kit project and we don't see any extra tags come through when adding them (just in case the Sveld CLI behaves differently.

Ahh! PR'd for Matt https://github.com/mattjennings/vite-plugin-sveld/pull/2 and local testing shows it coming through.
Thank you!


Deploying/Consuming AzureML webservices with 2+ arguments

What is the right way to consume the AzureML Deployed Webservice when deploying a model with 2+ arguments?

I have deployed a model using 2 arguments on the function run.
def run(batch_string, model_id):
    #some code here

I've tried like that:
import json
test_samples = json.dumps({"batch_string": "This is a string", "model_id":42})
test_sample = bytes(test_samples,encoding = 'utf8')
aks_service.run(test_sample)

But I've received the following error code:
---------------------------------------------------------------------------
WebserviceException                       Traceback (most recent call last)
<ipython-input-107-8f1ec51edd66> in <module>()
----> 1 aks_service.run(test_sample)

~/anaconda2/envs/aml36/lib/python3.6/site-packages/azureml/core/webservice/aks.py in run(self, input_data)
    330                                       'Response Code: {}\n'
    331                                       'Headers: {}\n'
--> 332                                       'Content: {}'.format(resp.status_code, resp.headers, resp.content))
    333
    334     def update(self, image=None, autoscale_enabled=None, autoscale_min_replicas=None, autoscale_max_replicas=None,

WebserviceException: Received bad response from service:
Response Code: 502
Headers: {'content-length': '56', 'cache-control': 'max-age=0, private, must-revalidate', 'x-ms-request-id': '03c2ceae-1581-46f2-8745-46e3810bdd9f', 'connection': 'keep-alive', 'content-type': 'text/html; charset=utf-8', 'date': 'Tue, 09 Oct 2018 14:24:43 GMT', 'server': 'nginx/1.10.3 (Ubuntu)', 'x-ms-run-function-failed': 'True'}
Content: b"run() missing 1 required positional argument: 'model_id'"

I've also tried to consume this model via cURL, but received an equivalent error:
curl -X POST -H "Content-Type:application/json" -H "Authorization:Bearer <my-auth-key>" --data "{\"batch_string\": \"This is another string\", \"model_id\": 21}" <my-scoring-url>
Return:
run() missing 1 required positional argument: 'model_id'

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: cc87eb58-5bdf-15d3-1a52-ca9d0706726f
Version Independent ID: 460fb6ba-ca28-2505-b4d9-1dec3972d968
Content: Deploy models to Kubernetes from Azure Machine Learning service
Content Source: articles/machine-learning/service/how-to-deploy-to-aks.md
Service: machine-learning
GitHub Login: @raymondlaghaeian
Microsoft Alias: raymondl

@nansravn Thanks for the feedback! We are currently investigating and will update you shortly.

@nansravn The input into the run(input_data) function is a json paylod which can have multiple nodes which mean you can include multiple arguments inside it. For example:
Input_data = { “batch_string": "this is a string", "model_id": 123 }
You would then parse the json inside the run function and access each node's value.

Ok @raymondlaghaeian, I'll adapt my code to have only 1 argument which is a JSON with multiple keys inside it.
So, based on your answer, nowadays the only type of data accepted as an argument are JSONs (application/json)? Is it possible to send binary data to the webservice (application/octet-stream)? (for example if I want to make an Object Detection Webservice, sending binary images without any base64 or ASCII encoding)

@nansravn We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.

@YutongTie-MSFT - please assign to @Blackmist thank you.

@nansravn Support for binary data is coming soon-ish. I'm waiting on an example that demonstrates handling binary data, which I'll use to update this doc. The example is held up currently as it relies on some back-end bits that aren't ready yet. I'll respond here once the example is ready and documentation updated.

@Blackmist, this already helps me a lot!
I'll test it and prepare an example code to publish on my github, so we can have a deployment example reference.

@nansravn Glad it helps! One disclaimer, which will be in the docs, is that his uses things from the azureml.contrib namespace. Things in here are going to change fairly often as improvements are made to the product. So any code you write against these objects might need to be updated in the next SDK release. Once the code/feature is "finished" it will move out of contrib and into another part of the SDK namespace.

@Blackmist I'm supporting deployment in which we are testing this preview feature using azureml-core[databricks] version 1.0.2 (inside a Databricks Workspace).
Basically I'm trying to receive an image as bytes for a deployment of a Computer Vision ML model.
from azureml.contrib.services.aml_request  import rawhttp
from azureml.contrib.services.aml_response import AMLResponse

from PIL import Image
from io import BytesIO
import json

init():
   pass

@rawhttp
def run(request):
   if request.method == 'POST':
      reqBody = request.get_data(False)
      myImage = Image.open(BytesIO(reqBody))
      return json.dumps(myImage.size) #Example, returning the size of the image
   else:
        return AMLResponse("bad request", 500)

If I want to load a PIL.Image from request.get_data(), should I use io.BytesIO() as in this documentation?
Theretically, am I doing something wrong in the example above? I'm basically trying to load an image from bytes and then return the image size as a json.

@Blackmist The code above worked well, thanks a lot for your contribution! I've made a production deployment of a TensorFlow Object Detection model using the same logic of the above code.
In the future I'll update my code, when this feature leaves the azureml.contrib namespace.


[QUERY] Not Sure If Customized TokenCredential in ArmClient Should Meet Multi Tenant Requirement

Library name and version
Azure.ResourceManager 1.9.0
Query/Question
When creating a new ArmClient, we passed our own class as the TokenCredential implementation instead of using the ones provided by Azure.Identity. Is it possible that in the future, the trend will be to use TokenCredentials which provided by Azure to catch up with the latest security requirements? Additionally, is it possible that there will be a KPI check for ArmClient's TokenCredential parameter, and that the customized implementation class will need to meet various latest security requirements?
We have this question because we noticed that the ClientSecretCredential code handles multi-tenant scenarios using AdditionallyAllowedTenantIds, as described in this document. Our own implementation of TokenCredential uses a first party app to obtain tokens in the GetToken method. However, we only pass the tenant ID in the constructor and do not pass any other tenant from the GetToken method. The class and tenant are a 1:1 relationship, so we did not set AdditionallyAllowedTenantIds. Since the first-party app itself is multi-tenant app, we are not sure if this approach has any security risks.
The code is like:
public class ArmTokenCredential : TokenCredential
{
    private IAppTokenProvider appTokenProvider; // This uses MSAL and first party app to get token

    public ArmTokenCredential(IAppTokenProvider appTokenProvider, string tenantId, AzureEnvironment azureEnvironment)
    {
        this.appTokenProvider = appTokenProvider;
        this.TenantId = tenantId;
        this.AzureEnvironment = azureEnvironment;
        this.ResourceUrl = azureEnvironment.ArmEnvironment.Audience;
    }

    internal AzureEnvironment AzureEnvironment { get; }

    internal string TenantId { get; }

    internal string ResourceUrl { get; }

    public override AccessToken GetToken(TokenRequestContext requestContext, CancellationToken cancellationToken)
    {
        var authenticationResult = this.appTokenProvider.GetAuthenticationResultByTenantIdAsync(this.AzureEnvironment, this.TenantId, this.ResourceUrl).GetAwaiter().GetResult();
        return new AccessToken(authenticationResult.AccessToken, authenticationResult.ExpiresOn);
    }

    public async override ValueTask<AccessToken> GetTokenAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
    {
        var authenticationResult = await this.appTokenProvider.GetAuthenticationResultByTenantIdAsync(this.AzureEnvironment, this.TenantId, this.ResourceUrl);
        return new AccessToken(authenticationResult.AccessToken, authenticationResult.ExpiresOn);
    }
}

Also, we would like to ask why ArmClient and BearerTokenAuthenticationPolicy code do not pass the tenant ID when creating a new TokenRequestContext. In my understanding, ArmClient should have an owner tenant, all resources is under that tenant. So why not pass this tenant directly into the TokenRequestContext constructor? This way, we can directly use the TokenRequestContext's tenantID in the GetToken method. Like:
public async override ValueTask<AccessToken> GetTokenAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
{
    var authenticationResult = await this.appTokenProvider.GetAuthenticationResultByTenantIdAsync(this.AzureEnvironment, **requestContext.TenantId**, this.ResourceUrl);
    return new AccessToken(authenticationResult.AccessToken, authenticationResult.ExpiresOn);
}

Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @SuyliuMS - Here is a blog post describing the purpose of AdditionallyAllowedTenantIds and related security risks https://devblogs.microsoft.com/azure-sdk/guidance-for-multi-tenant-applications-using-the-azure-identity-libraries/

Also, we would like to ask why ArmClient and BearerTokenAuthenticationPolicy code do not pass the tenant ID when creating a new TokenRequestContext. In my understanding, ArmClient should have an owner tenant, all resources is under that tenant. So why not pass this tenant directly into the TokenRequestContext constructor?

Typically the tenantId is supplied by the credential, or in some cases, by a challenge response in the case where the requested resource supports tenant discovery and the target resource's tenant is not known ahead of time (example).

Hi @SuyliuMS - Here is a blog post describing the purpose of AdditionallyAllowedTenantIds and related security risks https://devblogs.microsoft.com/azure-sdk/guidance-for-multi-tenant-applications-using-the-azure-identity-libraries/

Also, we would like to ask why ArmClient and BearerTokenAuthenticationPolicy code do not pass the tenant ID when creating a new TokenRequestContext. In my understanding, ArmClient should have an owner tenant, all resources is under that tenant. So why not pass this tenant directly into the TokenRequestContext constructor?

Typically the tenantId is supplied by the credential, or in some cases, by a challenge response in the case where the requested resource supports tenant discovery and the target resource's tenant is not known ahead of time (example).

Thanks @christothes .
Wondering is there any list that what azure resources support tenant discovery? Does ARM support it? It sounds like if the resource doesn't support tenant discovery, then there is no point to set the AdditionallyAllowedTenantIds as the tenant is always supplied by the credential?

I'm not aware of a list, but Storage and Key Vault are two I am aware of.

It sounds like if the resource doesn't support tenant discovery, then there is no point to set the AdditionallyAllowedTenantIds as the tenant is always supplied by the credential?

That's correct.


Add note to rsct doc for memory limit

Signed-off-by: mjulie<EMAIL_ADDRESS>

@juliemathew, what does large memory configuration mean? It would be great if you could share your experiences with us, because our largest worker are configured with ~200GB memory.

@juliemathew, what does large memory configuration mean? It would be great if you could share your experiences with us, because our largest worker are configured with ~200GB memory.

@neikei
We deployed an OCP 4.8 cluster with large config, in order to execute scale tests. One of the worker nodes is configured with 2 TB RAM, 64 cores.
powervm-rmc pod crashed on this worker node with the following error message.
On modifying the memory limit from 1 Gi to 2Gi, the pod started  running fine.
Containers:
  powervm-rmc:
    Container ID:   cri-o://de9e9175049665f87e054c3a64cf873a34790cc9f8be829e470507ba2896b2d3
    Image:          quay.io/powercloud/rsct-ppc64le:latest
    Image ID:       quay.io/powercloud/rsct-ppc64le@sha256:2bd63f8e18ed1642e36897eeef7992c307552c1107dff215a3f1817873e88a1f
    Ports:          657/TCP, 657/UDP
    Host Ports:     657/TCP, 657/UDP
    State:          Waiting
      Reason:       CrashLoopBackOff
    Last State:     Terminated
      Reason:       OOMKilled
      Exit Code:    137


fix(solid): safeguard sharedConfig.load in Suspense

Should the sharedConfig.load() call be safeguarded in Suspense? I see it safeguarded in other places. I am trying to use Suspense in an Astro app and line 143 throws an error because load is undefined.

I see. Astro I believe only does Solid's synchronous rendering currently which doesn't do Suspense on the server so it doesn't need load. I assumed it didn't need the hydration script whereas all my examples that aren't Astro have it. So I didn't notice this. Thanks.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 89.005%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/solid/src/render/Suspense.ts
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
1242

Relevant Lines:
1325

💛  - Coveralls


Re-group imports

This re-groups imports for OMIS modules to make them conform to the cryptography style.
Issue number: #770

@elcct I haven't touched the search/investment ones, they are the only ones to fix before upgrading.

Codecov Report

Merging #841 into develop will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           develop     #841   +/-   ##
========================================
  Coverage    96.67%   96.67%
========================================
  Files          159      159
  Lines         4657     4657
  Branches       452      452
========================================
  Hits          4502     4502
  Misses          96       96
  Partials        59       59

Impacted Files
Coverage Δ

datahub/omis/order/serializers.py
100% <ø> (ø)
:arrow_up:

datahub/omis/notification/client.py
100% <ø> (ø)
:arrow_up:

datahub/omis/notification/signal_receivers.py
88.57% <ø> (ø)
:arrow_up:

datahub/dbmaintenance/management/base.py
100% <ø> (ø)
:arrow_up:

datahub/omis/order/admin.py
94.78% <ø> (ø)
:arrow_up:

datahub/omis/invoice/managers.py
100% <ø> (ø)
:arrow_up:

datahub/omis/order/signal_receivers.py
100% <ø> (ø)
:arrow_up:

datahub/omis/quote/views.py
100% <ø> (ø)
:arrow_up:

datahub/omis/order/views.py
100% <ø> (ø)
:arrow_up:

...ance/management/commands/update_omis_uk_regions.py
94.73% <ø> (ø)
:arrow_up:

... and 10 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd65aea...55c52d9. Read the comment docs.


Drupal content search for non-US EN admin not returning correct results

BUG
Current Behavior
Overview: There is no way to get to translated campaigns -- either through keyword search or filtering. The following two issues outlines both:
Issue 1
This translated campaign exists: https://www.dosomething.org/mx/voluntario/mirando-con-las-estrellas?source=campaigns. When you're in Drupal and searching for "estrellas", it doesn't show up in the results:

We think this is happening because Drupal is searching within the source language's node. In this case, this campaign was initially published in US EN, so the node title is in US EN: "Staring with the Stars," So "estrella" wouldn't be picked up.
Issue 2
When searching in Drupal's Content section, if I filter by Content Type --> Campaign, Language --> Spanish, Mexico, Status --> Any, you don't see all translated campaigns for that language.  Right now, you can only see 11 campaigns, but there's more translated campaigns than this, including this campaign: https://www.dosomething.org/mx/voluntario/mirando-con-las-estrellas?source=campaigns.
Desired Behavior
As a non-US admin, when I search Drupal for a translated page, it should show up.
For keyword search, I think most admins search for the keyword instances in the title, so expanding Drupal's search to look in both the node title of the original campaign and the translated campaign title would be ideal.
For the filtering, it would be ideal if when you filter by "Language," it pulls back all translated campaigns in that language.
Since we're working within the constraints of Drupal, I understand if the above are not doable. If they aren't doable, would love to understand more about what's going on with the search and if there are potentially other solutions we can relay to our global partners.
Note: We spoke with our partner in Mexico and he said that he most commonly searches by "Spanish, Mexico" and that seems sufficient for him. Not sure if this could be the easiest fix or if it's all complicated.
Steps to Replicate

Use TorGuard and set as non-US (Mexico)
Login w/ non-US login  (Mexico admin)
Search Content
Search Spanish, Mexico
This isn't a complete list

Relevant Screenshots
The screenshot below is filtered for Spanish, Mexico language, and it only shows 11 campaigns (there are more than this, like https://www.dosomething.org/mx/voluntario/limpieza-de-calles-por-la-tortuga-marina -- which is not appearing on the list because it's a translation, the source language isn't Spanish):

@mirie Can you confirm that this is still an issue on DS production? It seems to work completely fine locally which uses a Staging DB, but Staging itself is down right now so I can't test there, and I don't have an admin account on production.

Discussed over a meeting today - both of us can't seem to replicate this using Staging data, however, I do see the issue on Production. Re-assigned to myself to investigate further with a different dataset.


Admit list argument in ag.find

Since https://github.com/cloudhead/neovim-fuzzy/commit/46f908aedef6af039c5134056ad008fe7aae1cbc find expects a list instead of a string and only rg.find was updated. This PR adds the missing change to ag.find.
I don't know anything about VimL but it Works on My Machine™

Great, good enough for me!


Add retry wrapper to codecov command in CI

To prevent fail job if sending report to codecov is failed for some reason (usually timeout). It's happening too frequently on AppVeyor CI.

Thanks @blueyed!


bitcoin miners

have our reddit and discord friends fund the servers

lovely

Ze install da mod? MINE EET


Switch to strict math mode for less compilation.

The default lessc compilation attempts to calculate CSS calc(...) statements, but makes a strange choice: calc(100% - 25px) ends up being compiled down to calc(75%) which is almost always incorrect.
In our current CSS, we only have two uses of calc, but because it is valid CSS and because it solves a real problem we have, I'm proposing that we use the strict math mode compilation flag for Less. This way, CSS authors will not have to do anything special to get their CSS to work as expected.
The inspector content rendering has a bug now that we've switched to less without strict math:

After turning on the flag, it renders correctly:

cc: @ellisonbg

Fully agree we want strict math mode, merging!


Quick action -> custom icon issue

Hi, I am facing an issue while setting up custom icon in iOS. Please let me know if anyone having the idea how to fix this .
My code:
QuickActions.setShortcutItems([
{
type: "Invoices",
title: "Invoices",
icon: Platform.OS == "ios" ? "Invoices" : "invoices",
userInfo: {
url: "",
},
},
I have created Images.xcassets folder and added Invoices imageset  but still no luck.

Try deleting the application and reinstalling from your device

I also do have this problem, even after clean install. Would be nice, if there would exist some example repo that would use custome icons in dynamically set quick actions.

I now noticed even stranger behaviour. It actually copies the shape of the image I want to use but makes it all black. So my circle image is just black circle, and my credit card icon is just black rectangle. And when I put a non-existent name, it is small black dot.

@Pranay2019 so I finally figured it out. For ios are acceptable only svg (in assets you need to add Symbol Image set) or png that is single color on transparent background. So basically the ios would use just your shape and provide the color that matches other quick actions


Encryption at Rest

Encryption at rest .. Just wondering if this is a feature ?

This isn't supported right now. Do you mean with something like SQLCipher?


Null is not an object (evaluating 'this.imageUri.ios') when imageUri is get passed null

Which platform(s) does your issue occur on?

iOS simulator

Please, provide the following version numbers that your issue occurs with:

CLI:  5.0.2
Cross-platform modules: 5.1.0
Runtime(s): 5.1.0
Plugin(s):
"nativescript-image-cache-it": "^4.1.0",

Please, tell us how to recreate the issue in as much detail as possible.
When imageUri get passed null value, application crashes with this exception:
file:///app/tns_modules/nativescript-image-cache-it/image-cache-it.js:60:68: JS ERROR TypeError: null is not an object (evaluating 'this.imageUri.ios')
*** JavaScript call stack:

Great work @triniwiz


remove cluster name and remove unecessary subprojects

[X] styleCheck passes
[X] unit tests pass
[X] integration tests pass
[X] manual tests pass
[X] API docs updated

Remove references to the deleted subprojects
Delete the unecessary subproject
Remove use of clusterName from the configs
Remove use of clusterName from the spandex es client
Remove use of clusterName from all tests

This build looks good.


KAFKA-8098: fix the flaky test by disabling the auto commit to avoid member rejoining

In the test, we first test removing 1 member from group, and then test removing the other 2 members from group, and it failed sometimes at the 2nd member number assert. After investigation, I found it's because we enabled auto commit for the consumers(default setting), and the removed consumer offset commit will get the UNKNOWN_MEMBER_ID error, which will then make the member rejoin. (check ConsumerCoordinator#OffsetCommitResponseHandler) So, that's why after the 2nd members removing, the members will sometimes be not empty.
I set the consumer config to disable the auto commit to fix this issue. Thanks.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@feyman2016 @huxihx , since you have experience in this test before, could you review this small PR? Thanks.

hi @feyman2016  @huxihx , could you help review this small PR? Thanks.

@feyman2016  @huxihx @abbccdda , since you have experience in this test before, could you help review this PR? Thanks.

@abbccdda , could you please review this PR? Thanks.

@abbccdda @omkreddy , could you review this PR? Thanks.

@abbccdda @omkreddy @feyman2016 @huxihx , could you help review this PR? Thanks.

ok to test

@omkreddy , I didn't see the Jenkins build results. Should we run again? Thanks.

merge the latest trunk to have auto-build

@omkreddy , any comment for this PR? Thanks.

@omkreddy , could you review this PR? Thanks.


failed to get symbol 'rmw_init_options_init'

Describe the bug
The standalone 1.3 release for humble/ubuntu 22.04 fails to get a symbol when running the Ros2cs.Init() function.
To Reproduce
Steps to reproduce the behavior:

Start with a clean Ubuntu 22.04 install
Do not install ROS2 (not sure if this is relevant)
Create a new Unity project with editor version 2022.3.7
Install libspdlog-dev using apt-get
Download and unzip https://github.com/RobotecAI/ros2-for-unity/releases/download/1.3.0/Ros2ForUnity_humble_standalone_ubuntu22.zip
Move the unzipped folder under Assets in the Unity project.
Add the ROS2UnityComponent script to a game object in the scene.
Press play
See error

The full error is:
RuntimeError: failed to get symbol 'rmw_init_options_init' due to Environment variable 'AMENT_PREFIX_PATH' is not set or empty, at ./src/functions.cpp:171, at ./src/rcl/init_options.c:75

Screenshots

Desktop (please complete the following information):

OS: [Ubuntu22.04]
ros2 distro [humble]
ros2-for-unity version [1.3.0]
ros2 environment setup [single pc]

I installed ROS2 locally on the same computer with the error, and it disappeared.
It would be really nice to have this working on computers without needing ROS2 installed.
We pretty much exclusively use ROS2 in docker at my company and don't tend to have a local ROS2 install.

Hi, I believe you were missing libtinyxml2-dev as well.
This should be fixed now (currently on develop ros2cs branch).


refactor(bootstrap): Refactor all services to use latest go-mod-bootstrap

Latest go-mod-bootstrap has new refactored Configuration and Registry clients
as well as new common command-line parsing.
close #2316

Codecov Report

Merging #2317 into master will increase coverage by 0.22%.
The diff coverage is 0%.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e6e0a20...ff4bb4b. Read the comment docs.

@michaelestrin , @tsconn23 , @jim-wang-intel , @tingyuz. This PR is now ready for official review.

I am running consul locally, and added the program argument -cp="consul.http://localhost:8500" which seems to be correct since it was successfully loaded. I might be doing something wrong.

Which service? I see in the log that it loaded config from http://localhost:8500. Does the service have [Registry] section in Consul?

@lenny-intel , I was running core-command and I specified the registry flag. Here are my program arguments:
-cp="consul.http://localhost:8500" -r --confdir=/home/anthony/GoLandProjects/edgex-go/cmd/core-command/res
I also, updated the TOML to not include the registry section to ensure the data was being read from the command line arguments.

I also, updated the TOML to not include the registry section to ensure the data was being read from the command line arguments.
@AnthonyMBonafide , that is the issue. -r specifies to use the Registry, but expects to get the Registry provider information from the registry section.

With the split to have distinct Config provider and Registry provider, only the Config Provider information is provider on command-line. Once the config is loaded the Registry info is available, similar to other clients in the config. Currently both are configured to use Consul. In the future that could change.

@tingyuz have you reviewed and ensured changes to the various security-related start up services don't introduce any issues?
@jim-wang-intel have your review comments been adequately addressed (and if so are you willing to resolve your review or re-review to move out of "change requested" status)?
@AnthonyMBonafide have your review concerns been adequately addressed (and if so are you willing to resolve your review or re-review to move out of "change requested" status)?
@lenny-intel I'm intentionally waiting to do my review until the above three reviewers have completed their respective reviews and any necessary follow-on work is completed.

@AnthonyMBonafide have your review concerns been adequately addressed (and if so are you willing to resolve your review or re-review to move out of "change requested" status)?

Yes, my concerns have been addressed.

@AnthonyMBonafide , that is the issue. -r specifies to use the Registry, but expects to get the Registry provider information from the registry section.
With the split to have distinct Config provider and Registry provider, only the Config Provider information is provider on command-line. Once the config is loaded the Registry info is available, similar to other clients in the config. Currently both are configured to use Consul. In the future that could change.

Got it, sorry for the misunderstanding.

BTW, I completed running Blackbox tests for both secure and non-sure mode. Only one failure which is an existing known failure with Command Device API.

@jim-wang-intel have your review comments been adequately addressed (and if so are you willing to resolve your review or re-review to move out of "change requested" status)?

See my question and comment here: https://github.com/edgexfoundry/edgex-go/pull/2317#discussion_r376134971

@lenny-intel
Please resolve conflicts and ping me when new code is pushed.  I'll then undertake a final review.


Typo fix

Fix ProcessDefinitionNotFoundError error message double space

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.283%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
736

Relevant Lines:
789

💛  - Coveralls


Remove obsolete CLI flags - eth1

Describe the bug
Eth1 deposit contract & chain id are now used from IBeaconParams, but there are CLI flags still hanging.
Expected behavior
Remove the obselete CLI flags

Which options specifically are still hanging?

@wemeetagain bump

deposit contract & chain id


Support/backup restore tasks

Pull Request Test Coverage Report for Build 47

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 64.916%

Totals

Change from base Build 45:
0.0%

Covered Lines:
692

Relevant Lines:
1066

💛  - Coveralls


Fall back to hashing the zip entry when ABI extraction fails during fingerprinting

Fixes #20398

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

OK, I've already triggered the following builds for you:

PullRequestFeedback build

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.


fix: The false result value has the wrong type

Description
Whenever you define your service_integrations and set create_role && attach_policies_for_integrations to false, you expect to have an empty map to skip the appropriate blocks and resources to be created within the module.
Therefore Terraform will throw an exception for not accepting different map structure within the same condition. The first one with your key/values you are providing and the second one is an empty map.
When executing terraform plan the below error appears:
│   on .terraform/modules/step_function/main.tf line 123, in resource "aws_iam_policy_attachment" "service":
│  85:   for_each = local.create_role && var.attach_policies_for_integrations ? try(tomap(var.service_integrations), var.service_integrations) : tomap({})
│
│ The false result value has the wrong type: map has no element for required attribute "lambda".

Used variables in order to reproduce it.

module "step_function" {
  . . .

  create_role = false
  attach_policies_for_integrations = false

  service_integrations = {

    lambda = {
      lambda = [. . . .]
    }

    xray = {
      xray = true
    }
}

. . .

}

Motivation and Context
This change keeps the same logic behind the conditions and resources creation, it's simply a different way of using values and condition with the for_each.
With this solution we create our map whenever we have values, and it skips the block when the condition is not met.
Breaking Changes
There no really breaking changes

Remove stale label

Fixed in #30.
Thank you very much!


Major updates

Release notes:

Adds JSON downloads (requested in issues #14 and #60)
Adds sticky headers for tables
Implements locale aware timestamp for the "Last updated" time
Creates a Cookies page with details of cookies and their lifetime
Increases the Cookies preference cookie lifetime to 365 days
Addresses failed download requests for the manifest.json file (issue #43)
Changes minimum of circle radii (map blobs) to zero (partially addresses issue #30)
Corrects the time-shift between data being generated and data being made available (incorporates the ETL-generated data source)
Incorporates analytics for browser-triggered download events (when user has consented)
Removes unused dependencies
Adds description for UTLA abbreviation (using tooltips)
Fixes the issue that highlighted the incorrect page in the navigation

 Make historic data accessible by the public

 Bug: Cookie banner shows up everytime

 Cookies preferences page and different banner text

 Base circle radius scales at 0 cases and 0 pixels

 Not showing date that it was updated

 Incorporate the new data source with updated timestamp + locale timestamp

 download files as JSON

 manifest.json is missing on website

 Count CSV and JSON downloads through Analytics


Add note that the docker provider is supported

Also ordered providers in README alphabetically to be user friendly.
The ip address field structure is the same as it is for the VMWare provider.

Great, thank you!


Bump vm2 from 3.9.16 to 3.9.17 in /Tasks/AppCenterDistributeV3

Task name: AppCenterDistributeV3
Description: Update vm2 to fix CVE-2023-30547
Documentation changes required: N
Added unit tests: N
Attached related issue: N
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

@lucen-ms please run the tests

/azp run

@lucen-ms please merge it


Ubuntu 18.04 gpg dirmngr IPC connect call failed

Your Windows build number:  10.0.17134.81

What you're doing and what's happening:

Building MonoDevelop for Windows looks like a real hassle and there are no binaries available.
I decided to give MonoDevelop for Ubuntu binaries a try on WSL.
I installed a clean Ubuntu 18.04 from the Microsoft Store.
To get it all caught up I executed:
sudo apt-mark hold ebtables # for the current upgrade bug
sudo apt-get update
sudo apt-get upgrade -y

and then I picked up with the MonoDevelop IDE instructions:
sudo apt install apt-transport-https dirmngr
sudo apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv-keys 3FA7E0328081BFF6A14DA29AA6A19B38D3D831EF

Then I run into an issue.
I receive the following error message:
$ sudo apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv-keys 3FA7E0328081BFF6A14DA29AA6A19B38D3D831EF
[sudo] password for hayden:
Executing: /tmp/apt-key-gpghome.BG01sRSmD5/gpg.1.sh --keyserver hkp://keyserver.ubuntu.com:80 --recv-keys 3FA7E0328081BFF6A14DA29AA6A19B38D3D831EF
gpg: connecting dirmngr at '/tmp/apt-key-gpghome.BG01sRSmD5/S.dirmngr' failed: IPC connect call failed
gpg: keyserver receive failed: No dirmngr

What's wrong / what should be happening instead:

gpg isn't able to call dirmngr
What I have attempted:

Running dirmngr in another bash instance
Making sure gpg-agent is running in the background
This. https://stackoverflow.com/questions/49496667/gpg-key-generation-canceled and https://stackoverflow.com/questions/46673717/gpg-cant-connect-to-the-agent-ipc-connect-call-failed

Strace of the failing command.

Note: I have also tried building MonoDevelop from source on Ubuntu in WSL, but ran into this issue.

3424  bind(3, {sa_family=AF_UNIX, sun_path="/tmp/apt-key-gpghome.wZVVfa2T2R/S.dirmngr"}, 43) = -1 ENOENT (No such file or directory)

The process you strace keeps looking for "/tmp/apt-key-gpghome.wZVVfa2T2R/S.dirmngr" - and does not find it.
That this then ends up as
3424  socket(AF_UNIX, SOCK_STREAM, 0)   = 3
3424  stat("/tmp/apt-key-gpghome.wZVVfa2T2R/S.dirmngr", 0x7fffe2242bd0) = -1 ENOENT (No such file or directory)
3424  bind(3, {sa_family=AF_UNIX, sun_path="/tmp/apt-key-gpghome.wZVVfa2T2R/S.dirmngr"}, 43) = -1 ENOENT (No such file or directory)

does not seem to be overly surprising.
This begs the question: What do you actually see in sudo ls -la /tmp/apt-key-gpghome* at the time of failure?

It's Bugging Since The First Bionics in wsl. But the temporary to pass this problems is using apt-key add instead adv options through wget or curl :
curl -sL "http://keyserver.ubuntu.com/pks/lookup?op=get&search=0xA6A19B38D3D831EF" | sudo apt-key add

This workaround works. Thank you. I had a completely block when I was trying to think of other ways to install the key.
I created an example script here. https://github.com/sirredbeard/monodeveloponwsl/edit/master/install_via_apt.sh
I also reported to mono develop that they should list installing on WSL as an option for installing MonoDevelop on Windows here.
https://github.com/mono/md-website/issues/84

I hate to look a 'closed' gift horse in the mouth, but this one merits some further attention because it will keep coming up until we have a work-around on the books for the dirmngr fail proper. The problem is unrelated to Mono or MonoDevelop, natch. MariaDB (key 0xF1656F24C74CD1D8) fails the same, as will every apt-key howto anyone has ever posted on the Interwebs.
The problem that dirmngr (sudo apt install dirnmgr) isn't running (or isn't autolaunched?) and thus there is nothing on the other end for apt-key via gpg to talk to. After around half an hour of investigation, like @sirredbeard, I wasn't able to make dirmngr at home on WSL manually. Maybe someone else will have better luck or suggestions.

I support re-opening for the wider issue with dirmgr.
I think I was able to import the key by curling it in and piping it to apt-key.
Is this only on Ubuntu and only 18.04? If so, should we file a bug report upstream with Canonical?

I think I was able to import the key by curling it in and piping it to apt-key.

Yes that's effectively the work-around until the base cause and a solution to the root problem is articulated.

Is this only on Ubuntu and only 18.04?

That is part of what merits investigation. It wasn't a problem on 16.04 (but you already knew that). Whether it wasn't a problem in 16.04 because (a) an operable dirmngr wasn't previously a requirement or (b) dirmng was working fine and nobody thought anything of it or (c) something changed with gpg or apt-key or dirmngr or something in between that now causes the sequence to fail, I can't tell you. Or at least, not with any authority after looking at it for a short bit.

but sometimes in metapackages with ubuntu-desktop when i was want to install Crystal Language by importing their gpg and it's works through the adv options

I'd like to report that any debian based distro (possibly any distro) utilizing an updated dirmngr before 18.04 was officially released as LTS was recieving this error. The issue was resolved just in time for 18.04 release. So this may be a regression or something Ubuntu did to screw with it.
When I had experienced this error utilizing Debian Buster, I simply reinstalled Debian from the Store after 18.04 was released and the updated package containing the fix was imported when upgrading from Stretch to Buster. As of yesterday evening, I have yet to see this issue re-occur.
The work-around mentioned above failed at the time and did prevent any package (including stock) from successfully upgrading. Even apt update failed.
I would recommend others test a different distro other than Ubuntu 18.04 that uses the same version of dirmngr (or upgrade to it if not part of the base) and see if it still has an issue.
For completeness, I'm running build 17134 with the latest patch.

@Alternate-Egoist
I wanted to do the same but could not duplicate your success.

 I got .... etc etc .... IPC connect call failed

Well, yes. Because Quoth:
The problem is unrelated to Mono or MonoDevelop, natch. MariaDB (key 0xF1656F24C74CD1D8) fails the same, as will every apt-key howto anyone has ever posted on the Interwebs.

same error while install mongodb

Same problem here with MariaDB, if you really need the repository you could "avoid" the key authentication using sudo apt-get --allow-insecure-repositories update
Still it is not a proper solution, not to mention the potential security issues that could arise.

Can try this:

use sudo apt update  then will get something like The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 5523BAEEB01FA116
Got the pubkey 5523BAEEB01FA116
Use curl -sL "http://keyserver.ubuntu.com/pks/lookup?op=get&search=0x5523BAEEB01FA116" | sudo apt-key add Replace 0x5523BAEEB01FA116 with 0x<your pubkey in 2step>
complete, you can run sudo apt udpate no error

@foxliu solution seems to work just fine.
But still I wasn't able to download the public key using curl, so I copied manually the key from the Ubuntu Keyserver link,to a new file in WSL, then loaded it with sudo apt-key add.
For future reference, this is the MariaDB Package Signing Key:
http://keyserver.ubuntu.com/pks/lookup?op=get&search=0xF1656F24C74CD1D8

i have the same error while installing mongodb

For future reference, manual solution is this:
$ curl -sL "https://www.mongodb.org/static/pgp/server-4.0.asc?_ga=2.264892495.1953852568.1531143056-750073170.1531143056" | sudo apt-key add

I took a look at this today and have determined that this is a timing-related issue that is exposed because of a difference in how Windows and Linux handles connection attempts to localhost sockets.  On Linux an attempt to connect to a localhost tcp socket on a port that is not active will return a failure immediately.  On Windows there appears to be a 1 second timeout.  This causes the following sequence to occur.

gpg spawns dirmngr
dirmngr attempt to connect to localhost port 9050 (this is attempted twice).
gpg attempts to connect to a unix socket that dirmngr creates after the localhost socket connection fails.

This one second timeout in step 2 is enough to cause gpg to think that dirmngr is not responding.  There appears to be a retry loop in gpg, but it is not waiting long enough to account for the 1 second connect timeout (the connect is actually attempted twice).
I've reached out to some Windows networking experts to determine if there is a way that this timeout can be avoided.

That's awesome Ben. Thank-you for digging; by total chance I was going to dive this one myself this evening. Motivation being, this is blocking LXC, which uses gpg to fetch keys for the images in the same manner as apt-key 😉.
Being ahead of me, if you've got a suggestion a temporary work-around in the gpg or dirmngr source to make them collectively happy (avoid the timeout) that would be helpful. Even if not, at least I know where to look now, which is very appreciated. If this is going to be a long lead time thing (won't make fall release) maybe I can push a PPA to launchpad.

I haven't looked through the gpg source but it seems to be after it has launched dirmngr and is trying to connect to a unix socket it expects dirmngr to create. There is a retry loop that goes for about 20 iterations.

I took another look this morning and found the bug in gpg.  Their retry loop waiting for dirmngr to come up is passing an invalid argument to the nanosleep system call.  From the nanosleep man page:
The value of the nanoseconds field must be in the range 0 to 999999999.
The retry loop starts with a relatively small sleep and doubles the duration, eventually capping out at one second.  However they pass 1 second as<PHONE_NUMBER> nanoseconds, which returns EINVAL immediately.
This was actually already fixed by this commit: https://github.com/gpg/gnupg/commit/0cfdd3b57d592fb6baa7dafe8fde124e8a6c7798#diff-01af2aa0c9e9ef29097d81dfcf608938

I received a suggestion from the Windows networking team to set a socket option for localhost sockets to disable TCP retry.  With this change the apt-key command succeeds.  @Brian-Perkins has a change in code review now.

Well that turned out well. Might even address some other outlying fails. It was only by chance that dirmngr regressed and exposed this.
I also managed to learn something. 48 hours ago I would have bet a buck that it was glibc that checks for the the billion nanosecond overflow, returning EINVAL to satisfy some posix definition; and the Linux kernel would eat it and just normalise the timespec (a saner behaviour). It doesn't. The kernel checks. Who knew.

@therealkenc - Yeah I suspect this might speed up some random things that are attempting to connect to things things on localhost.

Thank you to @therealkenc and @benhillis.
Has a bug report been opened with GPG re: regression?

@benhillis Ah, I thought it was an upstream regression in gpg, my mistake.

So is there ultimately no fix for this?  Do we have to compile gpg from source to upgrade?

I think I'm having the same issue. I'm trying to install MongoDB and nothing happens when running sudo apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv 9DA31620334BD75D9DCB49F368818C72E52529D4

@foxliu hey I tried your steps
sudo apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv 9DA31620334BD75D9DCB49F368818C72E52529D4
converted to:
curl -sL "http://keyserver.ubuntu.com/pks/lookup?op=get&search=0x9DA31620334BD75D9DCB49F368818C72E52529D4" | sudo apt-key add
did I did this right?? it returned
Ok. then I just proceeded to install MongoDB works just fine.

@hermanomark    OK means the key has been added, you can install the one you need

I'm trying to install MongoDB.  I looked it up all morning.
sudo curl -LO https://www.mongodb.org/static/pgp/server-4.0.asc
sudo apt-key add server-4.0.asc

This same error occurs trying to install ROS via these instructions and I have not been able to figure out how to use curl in this instance.

@buddha314: there is a Q&A on ROS Answers about this (this one).

@benhillis For those of us not on an insider build (corporate network), is there any way to get this fix?

Hello, I arrived to this thread for a similar problem installing MongoDB, but maybe this steps are useful for you too:
(Were ran in Ubuntu 18.04 installed like a subsystem of windows 10)
First download the key manually (This key is for mongodb-org repository)
wget https://www.mongodb.org/static/pgp/server-4.0.asc
Add key manually
sudo apt-key add server-4.0.asc
Remove it
rm server-4.0.asc
Update package list
sudo apt-get update
Install MongoDB-org
sudo apt-get install -y mongodb-org
That's it, I hope to have helped to anyone.
Cheers!

If you need to install something from Microsoft, say like Azure CLI

wget https://packages.microsoft.com/keys/microsoft.asc

Following the guide by @jefraroce helped me

If you need to install something from Microsoft, say like Azure CLI

wget https://packages.microsoft.com/keys/microsoft.asc

Following the guide by @jefraroce helped me

Where is the guide @ppumkin ?
I am stuck on this now - thx!

Use what @jefraroce  has written up just aboive my previous comment. In particular

wget https://www.mongodb.org/static/pgp/server-4.0.asc

would be

wget https://packages.microsoft.com/keys/microsoft.asc

Add no-use-tor to dirmngr.conf.

Hi guys. I had problem installing mongodb on wsl like you. Then I just downloaded "TGZ" package from https://www.mongodb.com/download-center/community?jmp=docs and followed the installation steps from  https://docs.mongodb.com/v3.2/tutorial/install-mongodb-on-linux/ and everything works for me.
Hope this could help you.

안녕 얘들 아. 나는 당신처럼 wsl에 mongodb를 설치하는 데 문제가있었습니다. 그럼 난 그냥에서 "TGZ"패키지를 다운로드 https://www.mongodb.com/download-center/community?jmp=docs 에서 설치 단계 및 다음 https://docs.mongodb.com/v3.2/tutorial/ install-mongodb-on-linux / 모든 것이 저에게 효과적입니다.
희망이 당신을 도울 수 있습니다.

Hi guys. I had problem installing mongodb on wsl like you. Then I just downloaded "TGZ" package from https://www.mongodb.com/download-center/community?jmp=docs and followed the installation steps from  https://docs.mongodb.com/v3.2/tutorial/install-mongodb-on-linux/ and everything works for me.
Hope this could help you.

I can not see the results for that link

@Choco0914 copy and paste the link in the new tab. Seems Gitlab doesn't allow for direct linking

I have a similar problem while installing azure-cli.
sudo apt-key --keyring /etc/apt/trusted.gpg.d/Microsoft.gpg adv \

 --keyserver packages.microsoft.com \
 --recv-keys BC528686B50D79E339D3721CEB3E94ADBE1229CF

Executing: /tmp/apt-key-gpghome.nTgRqyZ9R8/gpg.1.sh --keyserver packages.microsoft.com --recv-keys BC528686B50D79E339D3721CEB3E94ADBE1229CF
gpg: connecting dirmngr at '/tmp/apt-key-gpghome.nTgRqyZ9R8/S.dirmngr' failed: IPC connect call failed
gpg: keyserver receive failed: No dirmngr

@xsqian I had the same problem with Azure CLI, so I stole the command from the SQL Server Linux install (https://docs.microsoft.com/en-us/sql/linux/sql-server-linux-setup-tools?view=sql-server-2017) and got it to work:
curl https://packages.microsoft.com/keys/microsoft.asc | sudo apt-key add -

@lukerogers Yes, thank you so much for sharing the solution, love it!

@therealkenc This is still an issue. Following any instructions that instruct you to run apt-key adv still fail for the same error. Got the error following instructions from https://docs.pritunl.com/docs/ssh-client under the Ubuntu Bionic section.

this

No, this was resolved in 17723.

curl -sL "http://keyserver.ubuntu.com/pks/lookup?op=get&search=0xA6A19B38D3D831EF" | sudo apt-key add

I tried following this https://downloads.mariadb.org/mariadb/repositories/#mirror=rise&distro=Ubuntu&distro_release=bionic--ubuntu_bionic&version=10.4 and I have similar problem.
Solution:
replace this 0xA6A19B38D3D831EF with key found on mariadb page. in my case 0xF1656F24C74CD1D8
curl -sL "http://keyserver.ubuntu.com/pks/lookup?op=get&search=0xA6A19B38D3D831EF" | sudo apt-key add

For those still with the same problem (I had it with MySQL 8 on a fresh Ubuntu 18.04), this solved my problem. In this case, it's Spotify but it works with any other repository.

$ wget -q "https://keyserver.ubuntu.com/pks/lookup?op=get&search=0xA87FF9DF48BF1C90" -O- | gpg --dearmor > spotify-A87FF9DF48BF1C90.gpg $ sudo mv spotify-A87FF9DF48BF1C90.gpg /etc/apt/trusted.gpg.d/

1- download the key
2- pass it to gpg to turn the ASCII into binary (unwrapping)
3- move the new file to apt's keyring.
After that, I was able to do apt update and to install.

Follow the https://docs.mongodb.com/manual/tutorial/install-mongodb-on-ubuntu/
sudo service mongod start (Will not work with WSL)
Step1 : sudo mkdir -p /data/db
Step2 : sudo mongo --dbpath /data/db
This will start the mongod server in WSL without any additional start
Step3 : mongo 👍

This issue still exists in 20.04. The original workaround (calling apt-key add - manually) no longer works. @tsilefy's workaround above does work. Is there any change of getting this resolved for good?

@veloscillator which Windows build? in 20.04 libc6 is broken in WSL1 and shows a very similar error message. You can try  https://github.com/microsoft/WSL/issues/5125#issuecomment-619350931 workaraound.

Both didn't work.

Both didn't work though.

@veloscillator which Windows build? in Ubuntu 20.04 libc6 is broken in WSL1 and shows a very similar error message. You can try #5125 (comment) as workaraound.

This resolved my issue, thank you (I'm on 2004 19041.208).

I ran into this issue trying to install postgres on WSL(1).
with:
MYURL=https://www.postgresql.org/media/keys/ACCC4CF8.asc

both
wget --quiet -O - $MYURL | sudo apt-key add -

and
curl -sL "$MYURL" | sudo apt-key add -

yield the following error:
gpg: can't connect to the agent: IPC connect call failed.

I can probably hold off on installing postgres on this machine for a month until WSL2 is available in a non-Insider build (corporate network).
Can I expect this to be fixed in WSL2 (for ubuntu 20.04)? Thanks for all the work you all have done on this.

I can't even use apt-key add because it also bombs out. So here's a super-manual way to get this done:
curl https://download.mono-project.com/repo/xamarin.gpg | gpg --dearmor > /etc/apt/trusted.gpg.d/mono.gpg

Putting this here in case I'm not the only person for whom this is a mission :/

I have a Pi4 running Linux version 4.4.0-19041-Microsoft
root@DESKTOP-BBF5IEP:~# curl -sL "http://keyserver.ubuntu.com/pks/lookup?op=get&search=0x5523BAEEB01FA116" | sudo apt-key add
Illegal instruction (core dumped)
I can't even apt-get update on either Ubuntu 18 or 20.  At best the dirmngr crashes or trying to add keys results in an Illegal instruction.  Is there anything about running on Cortex-A72's?   This is feeling like PowerPC all over again...

For those still with the same problem (I had it with MySQL 8 on a fresh Ubuntu 18.04), this solved my problem. In this case, it's Spotify but it works with any other repository.

$ wget -q "https://keyserver.ubuntu.com/pks/lookup?op=get&search=0xA87FF9DF48BF1C90" -O- | gpg --dearmor > spotify-A87FF9DF48BF1C90.gpg $ sudo mv spotify-A87FF9DF48BF1C90.gpg /etc/apt/trusted.gpg.d/

1- download the key
2- pass it to gpg to turn the ASCII into binary (unwrapping)
3- move the new file to apt's keyring.
After that, I was able to do apt update and to install.

This worked for me in the lastest WSL ty

Finally latest Windows update brings nanosleep support in Ubuntu 20.04 WSL1.
KB4571756 (OS Build 19041.508)


boolean expression must not be null

see
https://github.com/CaiJingLong/flutter_photo/issues/7

fix in #13


About nnet.utils.vis_utils

Hi, I tried to run code but I have an error with the nnet.utils.vis_utils. I think that is not a package in python. Is that a user-defined module? How can get the access to use it?
Thanks

Hi, maybe you are referring to this file https://github.com/dnlcrl/PyFunt/blob/master/pyfunt/utils/vis_utils.py
Please keep in mind that this is an unmaintained repo, if you find other similar errors you can simply fix them and make a pull request.
Also I strongly suggest you to use instead a deep learning package as tensorflow or pyTorch or one of the many others available, because this repo along with pyfunt and pydatset are for demonstration only.
If you want to train a residual network as in http://arxiv.org/abs/1512.03385, I suggest you to check this repo (where lua + torch are used): https://github.com/gcr/torch-residual-networks.

Thanks! That's what I want.


Amazon IVS Player support

https://docs.aws.amazon.com/ivs/latest/userguide/player-web.html it's similar like hls. But hls is having issue with custom codecs

The issue is unclear so closing it out for now. This may be something that needs to be raised in the hls.js GitHub repo.


Feature/platfowner/new feature

Summary:

More entries for tracker server's /status and /metric APIs
Add prometheus.yml (without ip addresses) and grafana.json for monitoring
Rename node.status -> node.state, BlockchainNodeStatus -> BlockchainNodeState
Rename consensusStatus.status -> consensusState.state in getConsensusStatus() results.

#232

Thanks for the fast review. Now merging..


Add some additional options to make_normalised_raster()

add default resampling method defaults based on comparing resolutions of both PU and input raster.
add manual override using terra default methods
add conditional_expression to allow user to do simple single raster algebrda using defined function, (e.g., function (r) ifel(r > 4, 1, 0) )
add fill_na option (default = TRUE) to fill in NA values to 0 before masking to PU layer
add default output datatype (INT1S or FLT4S) and NAflag when writing to file based on raster number type.

Looks good @scabecks, just some small things: I think it would be good to generalise the structure of functions and use %>% instead of |> (just because I used this in all other functions so far). I only saw this once now in line 96 in make_normalised_raster(). And when writing the roxygen skeleton I think it would also be good to have a generic way of doing it - I don't mind which way, but I saw that in make_normalised_raster() you give the inputs with [] (e.g. raster_in [SpatRaster] ), but not in the other new function you added. I really like the idea of including the conditional expression!

@sandra, the roxygen formatting is just me being lazy and having chatgpt fill in the commenting - and it's never constant. Good idea though - we'll stick with the generic way that exists already.


Add vpc connector

Closes https://github.com/PrefectHQ/prefect-gcp/issues/122
Example

Screenshots

Checklist

[ ] References any related issue by including "Closes #" or "Closes ".

If no issue exists and your change is not a small fix, please create an issue first.

[ ] Includes tests or only affects documentation.
[ ] Passes pre-commit checks.

Run pre-commit install && pre-commit run --all locally for formatting and linting.

[ ] Includes screenshots of documentation updates.

Run mkdocs serve view documentation locally.

[ ] Summarizes PR's changes in CHANGELOG.md

Thank you so much!


test: native asm2arm

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Binary operation for ArrayView

I'm confusing errors arising around +.
let dist = Range::new(0, 1);
let a = Array::<f64, _>::random((5, 5));
let x0 = Array::<f64, _>::random(5);
let xs: Vec<_> = a.axis_iter(Axis(0)).map(|x| x + &x0);

I expect that x + &x0 produces new array since document says

&A @ &A which produces a new Array

But it produces an error:
error[E0369]: binary operation `+` cannot be applied to type `ndarray::ArrayBase<ndarray::ViewRepr<&f64>, usize>`

After struggling, I find it works when I rewrite the last line as following:
let xs: Vec<_> = a.axis_iter(Axis(0)).map(|x| x.to_owned() + &x0);

I hope the first my code works. Is there any technical problem to implement Add for ArrayView?

x in x + &x0 is an array view, which is an "A", not an "&A". So &x + &x0 should work.

Oh...
I now understand my mistake and what document says.
The above error message means ArrayView cannot be consumed. It is hard to read :<
Thanks for your help!

It's a bit unfortunate that one needs to pass &ArrayView in many places where just passing a view by value should be enough. It's partly due to missing support for custom view types in Rust. Feel free to report any issues you find with ndarray.


Adding delays between requests

In a situation like this one, I needed to add some delay between http requests to prevent 503s:
from page in Enumerable.Range(0, 15)
select Http.Get(new Uri($"https://aa.com/bb?p={page}/"))
.Html()
// ...

My first, naive approach has been:
from page in Enumerable.Range(0, 15)
let _ = new System.Threading.ManualResetEvent(false).WaitOne(200)
select Http.Get(new Uri($"https://aa.com/bb?p={page}/"))
.Html()
// ...

Is that the approach you would follow, @atifaziz ?
Is there a built-in way of doing such a thing?
If there isn't, would it be possible to add an extension that allows doing it?

Is there a built-in way of doing such a thing?

No because it would be redundant with Rx that already has a built-in way with Observable.Interval. Here's an example showing how you can add delays between requests:
var q =
    from n in Observable.Range(1, 15)
                        .Zip(Observable.Interval(TimeSpan.FromSeconds(0.5)), (p, _) => p)
    from f in Http.Get(new Uri($"http://example.com/?p={n}"))
                     .Html()
    select new
    {
        Time = DateTime.Now,
        PageNumber = n,
        Url = f.RequestUrl.OriginalString,
        Title = f.Content.QuerySelector("html > head > title")?.NormalInnerText,
     // Intro = f.Content.QuerySelector("h1 ~ p")?.NormalInnerText
    };

foreach (var e in q.ToEnumerable())
    Console.WriteLine(e);

The idea is to zip an observation of a range of page numbers with another that makes observation at a given interval.
Sample output:
{ Time = 08/05/2020 10:19:35, PageNumber = 1, Url = http://example.com/?p=1, Title = Example Domain }
{ Time = 08/05/2020 10:19:36, PageNumber = 2, Url = http://example.com/?p=2, Title = Example Domain }
{ Time = 08/05/2020 10:19:37, PageNumber = 3, Url = http://example.com/?p=3, Title = Example Domain }
{ Time = 08/05/2020 10:19:38, PageNumber = 4, Url = http://example.com/?p=4, Title = Example Domain }
{ Time = 08/05/2020 10:19:39, PageNumber = 5, Url = http://example.com/?p=5, Title = Example Domain }
{ Time = 08/05/2020 10:19:40, PageNumber = 6, Url = http://example.com/?p=6, Title = Example Domain }
{ Time = 08/05/2020 10:19:41, PageNumber = 7, Url = http://example.com/?p=7, Title = Example Domain }
{ Time = 08/05/2020 10:19:42, PageNumber = 8, Url = http://example.com/?p=8, Title = Example Domain }
{ Time = 08/05/2020 10:19:43, PageNumber = 9, Url = http://example.com/?p=9, Title = Example Domain }
{ Time = 08/05/2020 10:19:44, PageNumber = 10, Url = http://example.com/?p=10, Title = Example Domain }
{ Time = 08/05/2020 10:19:45, PageNumber = 11, Url = http://example.com/?p=11, Title = Example Domain }
{ Time = 08/05/2020 10:19:46, PageNumber = 12, Url = http://example.com/?p=12, Title = Example Domain }
{ Time = 08/05/2020 10:19:47, PageNumber = 13, Url = http://example.com/?p=13, Title = Example Domain }
{ Time = 08/05/2020 10:19:48, PageNumber = 14, Url = http://example.com/?p=14, Title = Example Domain }
{ Time = 08/05/2020 10:19:49, PageNumber = 15, Url = http://example.com/?p=15, Title = Example Domain }

If you're specifically dealing with numbers then knowing that Observable.Interval notifies integers (starting with zero) at each tick, you can simplify the code to just:
var q =
    from i in Observable.Interval(TimeSpan.FromSeconds(1))
                        .Take(15)
    select i + 1 into n
    from f in Http.Get(new Uri($"http://example.com/?p={n}"))
                     .Html()
    select new
    {
        Time = DateTime.Now,
        PageNumber = n,
        Url = f.RequestUrl.OriginalString,
        Title = f.Content.QuerySelector("html > head > title")?.NormalInnerText,
     // Intro = f.Content.QuerySelector("h1 ~ p")?.NormalInnerText
    };

I am closing this assuming it answers your question.


Footer in submission template

Hi,
The conference submission template has a footer that includes the year of the last conference. I think that it should be changed from "Submitted to AutoML Conference 2022" to "2023".
Best,
Anna

I believe you may have an old version of the template:
https://github.com/automl-conf/LatexTemplate/blob/15c9fa31fe43830f32c396406fb80fda620fab53/automl.sty#L13


list index out of range

Hi there,
I have installed the repo and have tried running the following command:
python3 tools/mc_demo_yolov7.py --weights pretrained/yolov7-d6.pt --source ~/Videos/TrainStation.mp4 --fuse-score --agnostic-nms --with-reid
This has left me with the following error:
Traceback (most recent call last):
File "/home/simon/Documents/Tracking/BoT-SORT/yolov7/utils/google_utils.py", line 27, in attempt_download
assets = [x['name'] for x in response['assets']]  # release assets
KeyError: 'assets'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "tools/mc_demo_yolov7.py", line 254, in
detect()
File "tools/mc_demo_yolov7.py", line 55, in detect
model = attempt_load(weights, map_location=device)  # load FP32 model
File "/home/simon/Documents/Tracking/BoT-SORT/yolov7/models/experimental.py", line 241, in attempt_load
attempt_download(w)
File "/home/simon/Documents/Tracking/BoT-SORT/yolov7/utils/google_utils.py", line 31, in attempt_download
tag = subprocess.check_output('git tag', shell=True).decode().split()[-1]
Please can you advise, am I doing something wrong?
Thanks,
Simon

i have same problem in experiment with yolov7...
Does anyone know how to solve it

This issue should be relevant
https://github.com/WongKinYiu/yolov7/issues/940


Allow Overriding Sanctum Actions via app/Api/StarterKits/Sanctum

I would like to request the ability to easily override the default Sanctum actions using a custom path within the application, specifically app/Api/StarterKits/Sanctum. This would provide developers with the flexibility to customize authentication flows without modifying vendor files or extensively altering route definitions.

Thank you @vstpots this is a great suggestion
Please provide an example and I will get this implemented in one of the next patch releases before 1.2.
I assume you would like to add some additional configuration.
Alternatively feel free to start a PR and we can collaborate on this one 🤠
Meanwhile just released v1.1

You can already override any of these invokable actions, so I assume all you need is the ability to change only the uri? ideally we should keep the namespace the same for example for register this is api.v1.sanctum.register then if I understand your request you would like to customise the uri path from api/v1/sanctum/register to api/v1/signup for example?


RTL touchups

Description
While the site still functions under RTL language direction, it looks extremely unpolished and will give a bad impression (in addition to the frustrating partial translations). What follows is a list of the individual places that need to be touched up to get the site "presentable" for RTL languages.
The following descriptions are all what it should be for RTL languages. For each fix, ensure that it does not cause a regression to the LTR languages.

[x] #426

The navigation items should be aligned with the left side of the screen
The dropdown items should not have a large leading gap
Should the openverse logo be on the opposite side of the openverse brand text? cc @panchovm

[x] #424

The search input and button should have the correct corners rounded
The "I want something I can..." checkboxes should have their spacing fixed to match the LTR language spacing

[x] #431

The table has a double border on the left side
The table is missing the border between the Source and Domain column
The spacing of the "Suggest a new Source" button with the text and the icon is a little off compared to the LTR version (the left side of the button has too much padding)
Trailing slashes for all domains listed should be removed as they get automatically put at the front of the domain in RTL languages (https://asia.si.edu/ gets visually transformed by the browser into /https://asia.si.edu)

[x] #428

The search bar needs to have its button and input corners correctly rounded as well as its padding adjusted for RTL
The "safer browsing" flag icon needs to have its padding adjusted to match the LTR language padding
The scroll to top button should be moved to the opposite corner
License icons and their corresponding text need to have their separation padding adjusted for RTL

[x] #435

"Go to image's website" button needs to have its icon padding adjusted
The attribution box and tabs need to have their borders adjusted to be on the correct side for RTL languages
The plain text attribution also needs to have its quotation marks corrected as currently they're being moved around by the browser into strange places
The license icon needs to have its padding adjusted against the associated text
The information table needs to have its padding adjusted as the "dimensions" text overlaps the listed dimensions
The leading edge of the information/reuse table needs to be fixed for RTL languages to align with the rest of the text in the result information

[ ] Single result page for Audio

The audio tags need to be adjusted for RTL languages, their borders are all over the place

[ ] Audio search result

The audio result license icon spacing compared to the track length needs to be adjusted, see screenshots and description in the comments.

Reproduction
The easiest way to reproduce is to set your language to an RTL language (like Arabic). I'm not sure what the best way to do this is, but the way I did it was to edit to i18n_redirected cookie value to be ar. I don't know if this cookie is always present, it might exist only because I tested the WordPress.org Openverse theme translations at one point. If that is the case, then follow the instructions here to set up the theme on a local WordPress install and set your language via wp-admin. Alternatively, manually edit the dir attribute on each page to be rtl. The attribute will be reset to ltr on each page load, so this is a very tedious option for testing as you'll have to edit the attribute every time you reload the page or navigate to a new one.
Because we will inevitably have partial translations down the line as well (for example when new strings are introduced or existing strings are updated), the site should look passable as RTL even with English strings, so for now the partial translations work in our favor as we fix these issues.
Expectation

After fixing the issues above, the site should look polished in both LTR and RTL language directions.
For most of these the best solution might be to convert the underlying markup to use Tailwind CSS with the RTL ready start/end based styles.
Environment

Device:
OS:
Browser:
Version:
Other info:

Resolution

[ ] 🙋 I would be interested in resolving this bug.

Based on a conversation with someone more expert in internationalization some of the text heavy pages will just have their direction attribute temporarily set to ltr until they become more fully translated. We can put off some work this way too.

@WordPress/openverse-frontend Could someone create new issues for the remaining RTL items?


Add optional "layer-top" class to force an elements stack order above all other WebUI components.

Completed.


Better naming for Crowdin workflows

Let all GitHub Actions workflow definitions for Crowdin…

have a filename starting with crowdin-…
Have a name starting with Crowdin »

Fixed conflicts and branch name

Looking good now!