xarm5 and xarm6 have links with invalid inertias

We have ported this repository to use Ignition Gazebo instead of Gazebo Classic. When using Gazebo Classic, warnings arise indicating that the inertias of some links are not realistic but, as there is no critical error, the code can be executed. However, Ignition Gazebo arises some errors indicating that the inertias of some links are invalid, making it impossible to spawn xarm5 and xarm6. In the case of xarm6, which is the robot at our disposal, the results obtained are as follows:

Inertia files
Links with invalid inertia

xarm6_type9_HT_BR2.yaml (lite6)
-

xarm6_type6_HT_BR2.yaml (default)
Link 3

xarm6_type6_HT_BR.yaml
Link 3

xarm6_type6_HT2_LD.yaml
Link 3

xarm6_type8_HT_BR2.yaml
Link 2 and Link 3

xarm6_type11_HT_LD.yaml
Link 2 and Link 3

We would like to know the reason for this problem and, if necessary, a revision of the inertia parameters. Thank you.

I'm also bumping into this issue. Same situation. Files don't work as provided. No clue about which inertia files should be used for xArm 6.  This leads to a bad development experience. Having things tested and working in ignition is key since classic (Gazebo) is deprecated.
@vimior, @penglongxiang, @eborghi10 can you please advise on this topic? What are the right inertial files for the xArm 6 to be used together with Ignition Gazebo?

I have the same issue. Can you please take a look at the inertia parameters? Thank you in advance!

Hi all, sorry about the inconvenience. The issue may be related to the negative product of inertia convention used by URDF, while our parameters were originally calculated in positive convention. This incompatibility would not get reported until the latest ROS and Gazebo versions. As suggested by the MathWorks doc, all the previous Ixy, Ixz and Iyz values have to get negated. We managed to do it automatically in scripts, please update the repository and test on Ignition Gazebo.

Thanks for the prompt reply @penglongxiang.
@mgraoikerlan, can you test this out on our behalf please and report back?

Thanks for the quick reply @penglongxiang . Unfortunately, I have tried with the updated inertias and I still have the same problem.

Thanks for the quick reply @penglongxiang . Unfortunately, I have tried with the updated inertias and I still have the same problem.

So that we know exactly what we're referring to @mgraoikerlan, can you point @penglongxiang to the branch with the changes you tested?

Thanks for the quick reply @penglongxiang . Unfortunately, I have tried with the updated inertias and I still have the same problem.

So that we know exactly what we're referring to @mgraoikerlan, can you point @penglongxiang to the branch with the changes you tested?

Sure @vmayoral. I tested the branch humble.

Hi @mgraoikerlan, the inertia values were calculated by CAD software and they seem normal to me. We did notice there were outputs like "the link is static or has unrealistic inertia so the equivalent inertia box will not be shown" in rviz or classic gazebo before, however after the sign reverse this time, these warnings are gone, that's why we thought the issue was solved and the cause was found.
Since we have no experience of using Ignition Gazebo, we might need more time to look into it. Meanwhile, is it possible for you to give us some instructions and sample scripts to reproduce this issue (docs can be sent by email)? Or could you please send us the exact output of the error message for us to locate in third-party source code? More information could be helpful, thanks.

I retract my previous comment, it seems that the changes had not been fully implemented. The problem has indeed been fixed by negativising the inertia. Soyy for the confusion and thank you very much for your help.

Thanks for the support @penglongxiang. Thanks everyone for the quick reaction.

@vmayoral No problem, you also helped us found an issue we did not notice.


Edit readme.md

add examples
add wallet creation
change model name to meta-llama/Llama-3.1-8B-instruct
change example code chainId to mainnet(1)

LGTM, Thanks!


Navigation menu / drawer opens the wrong pages

Canvas example in the Tabris.js app for iOS (Downloaded from iTunes)
iPhone 5s 8.3
When you open the canvas example and switch through pages at one point it starts to open the wrong pages. You just need to open different pages a few time and suddenly every link opens the same or a other page than its supposed to.  Same error occurs in the Chart.js example. This possible could be an error in every Menu.

Install Tabris.js app from iTunes
Open it -> Examples -> Canvas or Chart.js
Open Navigation / Drawer
Swtich through pages a few times
Now you should be able to see the error with the wrong pages

I also made a example myself which you can find in my GITHUB repositories.
https://github.com/bakopf/myApp. The same error in the Menu occurs there.

Thanks for the report. We knew about this issue and have already fixed it. You will get it with the next release soon.


Refactor code order of navigateToNativeApp

Issue

close https://github.com/DroidKaigi/conference-app-2024/issues/672

Overview (Required)
Changed code order of navigateToNativeApp to avoid potential confusion. 👌🏼

Looks great! Thank you for your contribution!


Extend downloader plugin for values files

Hi,
Currently HELM support  downloader plugins only for charts. We have layered values, which are coming from various sources (local, GIT, nexus, etc.), so we need similar functionality for values files. This feature will allow us to use whatever source that we want without adding additional prolog command to download the file (only modify the location).
I suggest to allow plugin.yaml configuration like:
valuesDownloaders:
- command: "bin/mydownloader"
  protocols:
  - "myprotocol"
  - "myprotocols"

Then command like:
helm install release chart.tgz -f myprotocol://uri
will passthrough bin/mydownloader and use it standard output as values file.

@AssafKatz3 we're still working out some details on values files for Helm 3, but this is likely not something we will add in current Helm (2.14+)
In the meantime you may wish to write a plugin that either wraps "helm install" or fetches values file from your desired source prior to the install

@jdolitsky Not good for us but understandable :disappointed:

I'm going to close this ticket due to inactivity, but please re-open if this still needs to be addressed. Thanks!

@bacongobbler I changed a job since I had opened this issue, but it only added more use cases for it (downloading of values.yaml from S3 for example). Moreever, meanwhile more wrappers where created. Therefore this request is still valid :-)

Are you planning to implement this? It’s been a year since this ticket was opened.

I don't have the skill to write this code.


Fix smoke tests to run on arm64

Before this change, running smoke tests on arm64 using:
./build.sh
./build.sh --run-smoke-test

fails with an error that looks like this:
.dotnet/sdk/5.0.100-preview.4.20258.7/MSBuild.dll /bl:source-build-test.binlog /clp:v=m /flp:v=diag /t:RunSmokeTest build.proj
ls: cannot access 'artifacts/x64/Release/dotnet-sdk-5.0*': No such file or directory

This is effectively a port of https://github.com/dotnet/source-build/pull/1652 to the 'master' branch.

macOS failure is known issue: https://github.com/dotnet/source-build/issues/1641.
@crummel @dseefeld can one of you review + merge? (Want to let you to make sure it doesn't set back what you're working on in some way.)


Add reanalyze repository to test_wild

Hey @nkrkv,
during #67 I try to add reanalyze repository to the test, and it had several errors in it
So it could be interesting to add it to the parser
Thanks for maintained this treesitter grammar :)
Cheers
(I don't have time to take a look at it for now, but will when I will have more time)

Yep, reanalyze would be nice to have among the “wild” repos. Thank you!


Xamarin.iOS

Will this same approach using Firebase also work with Xamarin.iOS? Try to avoid using APNs.

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

ID: 2a03ab5f-3dda-f469-33e0-9504c90d468c
Version Independent ID: 4f8e4309-cd06-f09f-00f0-d978b3d16777
Content: Push notifications to Xamarin.Android apps using Azure Notification Hubs
Content Source: articles/notification-hubs/xamarin-notification-hubs-push-notifications-android-gcm.md
Service: notification-hubs
GitHub Login: @jwargo
Microsoft Alias: jowargo

@trichardson01-kellogg, Thanks for the feedback! We are taking a look into this and will get back to you soon.

@trichardson01-kellogg I'm not sure what you're asking. Are you asking if you can use FCM to send notifications to iOS devices?

@jwargo  That is correct. Can FCM be used to send notification to iOS devices. All the tutorials I am finding references using APN.

@jwargo If I already have FCM configured in Azure Notification Hub, do I also need to following the APNS steps at https://docs.microsoft.com/en-us/azure/notification-hubs/xamarin-notification-hubs-ios-push-notification-apns-get-started#connect-your-app-to-the-notification-hub

@trichardson01-kellogg As I understand it you can use FCM to send notifications to iOS devices, but FCM then uses APNS to send it notifications the iOS devices. So you're adding an unneeded, unnecessary level of complexity to the process. Just use Notification Hubs to send notifications to Android users through FCM and iOS users through APNS. That's what ANH takes care of for you.

@trichardson01-kellogg, Just checking in to see if you have had a chance to see the previous response from John. We will now proceed to close this thread. If there are further questions regarding this matter, please tag us in your reply. We will gladly continue the discussion and will reopen the issue.


Update Composer dependencies (2023-11-08-00-15)

Visual regression test passed!
View the visual regression test report


Hover message improvement

Add more detailed information into hover doc (for imports, variables).
I think hover feature is now large enough to have its own file.
Though it might seems to have to much information in hover message, please bear with it for now. This can make symbol registration debugging easier, an feature switch can be added later if needed.

Spot a bug in structures/ropes/ropes.v. Rope uses []rune for storing content, while everywhere else is indexing into content with byte index.
This cause problem when source code contains characters which takes more than one byte, OR when source code uses encoding other than UTF-8.
Gonna fix it here, since it's vital to other things in this PR.

Module name hover doc now has proper test case.

Excellent!


Reduce maintenance burden by letting setup-go track our Go versions

actions/setup-go supports "stable" and "oldstable" as automatic aliases for the latest release and the previous to latest release, which was our intent with two of these three hard-coded versions.  The third is hard-coded to match our go.mod, which actions/setup-go also supports reading from, so we can be completely DRY here.
See https://github.com/opencontainers/image-spec/pull/1225#discussion_r1882995860

go.mod: https://github.com/opencontainers/image-spec/actions/runs/12306398144/job/34348030870?pr=1229#step:3:27
go version go1.18.10 linux/amd64

oldstable: https://github.com/opencontainers/image-spec/actions/runs/12306398144/job/34348031172?pr=1229#step:3:21
go version go1.22.10 linux/amd64

stable: https://github.com/opencontainers/image-spec/actions/runs/12306398144/job/34348031420?pr=1229#step:3:21
go version go1.23.4 linux/amd64


Patch for Iso channel column number error

The issue is described here.
https://github.com/AllenNeuralDynamics/aind-behavior-blog/issues/735
Briefly, large fraction of Iso channel data.csv files have 9 columns instead of 6 (timestamp, ROI0, 1, 2, 3, Blank). Practically, this is not an issue as what happens is having three additional columns with all 0. As real data should be with values >255, the erroneous columns are easy to be eliminated.

This issue has been caused by an bug in the following Bonsai code (will fix this shortly). The node should be Activity.val0 instead of Activity.

As this is confusing (why 9 columns? why different from other two channels?), we should take care of this before nwb packaging.
Also, some pilot data acquired in 2023Late have only 4columns—Timestamp, ROI0, ROI1, Blank. This should be handled by a patch too.

Plan to fix early January.

This is fixed on the experimenter rig.
When we will be processing older data, we will fix the pipeline side as well.

Will need to eliminate some 0-filled data.


KDE Menu Favourites - Entries not updated when there's an updated to the programs

A really annoying thing about Nixos is that in KDE you have a favourites menu where you can put the  favourite appications in so you can access the quickly.
However on upgrades, they all break. They are still there but not working anymore. So everytime I need to delete them again and readd them.

Which version of KDE are you using?

KDE 4

(triage) sorry for the silence, is this still a problem?

At least KDE5 doesn't seem to have this problem anymore. KDE4 had this.

We can probably close it as ”moving on” then. cc @ttuegel @sjau


update application to 0.0.6

UpdateBot pushed helm dependency: application to: 0.0.6

UpdateBot commands:
updatebot push-version --kind helm application 0.0.6

/test this
/approve


Improve support for online COG

Based on the example datasets, it seems an XML file is needed to render COG hosted online. Any guidelines on how you created the XML file? It would be awesome if a tile client can be created the same way for both local and online COG files.
Ideally, something like this:
url = 'https://opendata.digitalglobe.com/events/california-fire-2020/pre-event/2018-02-16/pine-gulch-fire20/1030010076004E00.tif'
tile_client = TileClient(url)

I used to rely on titiler to render online COGs. I am considering switching to localtileserver to support both local and online COGs.
https://leafmap.org/notebooks/03_cog_stac/
import leafmap
m = leafmap.Map()
url = 'https://opendata.digitalglobe.com/events/california-fire-2020/pre-event/2018-02-16/pine-gulch-fire20/1030010076004E00.tif'
m.add_cog_layer(url)
m

Aha, so you're bringing up something that I wanted to make clear with the name. localtileserver is intended to be used only serve tiles form raster files that exist on your filesystem. If you want a tile server that can handle accessing remote files, you have a few options:

We create a new remotetileserver package or submodule here
You can use Resonant GeoData (which my team and I at Kitware have put a TON of work into handling tile serving from cloud-hosted data). See https://github.com/ResonantGeoData/ResonantGeoData/pull/603 for a demonstration.
You can use GDAL's Virtual Storage Interface since we're using large_image which is simply wrapping GDAL for data access.

In your particular case, this should work because your raster is a pre-tiled Cloud Optimized GeoTiff. I have this working locally with minimal changes to localtileserver I will push the change to a PR for you to test soon

Resonant GeoData seems awesome. I need to look into it. It would be great if you can make localtileserver work for pre-tiled COG. It might be too much work to create and maintain another package like remotetileserver.

It might be too much work to create and maintain another package like remotetileserver.

Well that is what Resonant GeoData is.... 😉


Improve Data Deletion from Postgres Database

We should provide guidance in our documentation to detail either how to manually delete the relevant information from each table or alternatively we should modify our schema to implement cascading deletion.
Currently have a jupyter notebook for this purpose which if tidied up could be utilised as part of the process depending on the method we decide to go for.

This has already been implemented in v3.2.0, right?

No this feature has not been included as of yet.

Ah, ok. My mistake.

Implemented in https://github.com/ARM-software/workload-automation/pull/1121


Base Texture Resolution Issue (0.5 when should be 1)

After Updating from 4.1.0, to 4.2.3, the textures are all double the size they should be when the device pixel ratio is 1. This still works fine when it is 2. When creating a new PIXI.Texture, if I log the resolution in either the constructor or in BaseTexture.prototype.update
-- DPR1 --
console.log(mySprite.texture.baseTexture.resolution) = 0.5
-- DPR2 --
console.log(mySprite.texture.baseTexture.resolution = 2
After much digging around in the BaseTexture, BaseRenderTexture and RenderTexture (constructors or update prototype), I am unable to find the source of this. If I am to console.log the resolution set in the BaseTexture, it shows as 1. I can't find where the resolution is being overriden, and I have tried overriding it myself by setting (mySprite.texture.baseTexture.resolution - 1) however this has no effect at all.
I even tried adding the<EMAIL_ADDRESS>to my atlas's in an attempt to override this. Just as a note, this works fine for every version prior to 4.2.X.

I'm struggling to remember how/where I discovered something similar, but I did and the issue was due to #3297.  Looking in my generated sprite sheets json, I strangely found (can't remember how I generated it or what tool I used) something like this;
"meta": {
    "image": "myspritesheet.png",
    "size": {"w":972,"h":1780},
    "scale": "0.5"
}

I'm not sure how the 0.5 got there in scale but switching it to "1" magically solved this problem for me, as it wasn't until #3297 that PIXI was referencing that field.

Found it - it was a codepen for this issue that broke (was pointing at latest pixi) so to fix it I did this.
Prior to the fix the left side images were displayed in double size.

You are right, we found the issue much later last night. I don't know why PIXI is parsing the scale object in the texture packer json file, but I think it probably shouldn't be. Now I have to make sure I manually update all my textures after exporting them :(
Thanks for taking the time to find the issue! Do we leave this as open for the dev's to fix or close this?

Considering @GoodBoyDigital set it, I'm assuming he knows why its like that, so I'll close.

This was changed recently to respect the spritesheet's internal scale. Previous this was ignored and the only way to specify the resolution/scale was with the name of the file<EMAIL_ADDRESS>There was a need to support both scenarios.
I think one way we could support this situation (filename is set but also has a defined non-1 scale value), Would be to return null with getResolutionOfUrl if no value is available on the file suffix. This would allow us to prioritize the filename resolution over the internal resolution.
What do you think @staff0rd?

Doesn't affect me - I don't use the scale normally, it was TexturePacker that set it for a demo spritesheet, and I don't otherwise use TexturePacker.  Perhaps @megmut has some thoughts.

When you export 3 sizes, you start with x3, then x2, then x1.. but the images you place in there are always at x3. So @3x would be scale = 1, @2x would be scale 0.6333 and @1 would be 0.3333. This doesn't make sense to me. Using the Texture Packer scale to determine the resolution for the baseTexture only works if you are using @1x scale.
I like the functionality to read of getting the resolution from the texture packer JSON file, however as it completely breaks a game if everything doesn't go according to plan, could we pass an option to read resolution from the JSON file, or better yet.. for us to pass our own resolution and  that gives the ability to read and interpret the JSON file up to the developer?

What I'm suggesting is to always prioritize the filename resolution (e.g. @2x). If that isn't set at all, fallback to the scale set in the JSON. Otherwise, resolution of 1.
The current implementation reads the filename but defaults to 1 and then overrides with the JSON scale if it's defined and not 1.

Okay, that sounds good! Thanks for the support guys, my first bug submit here 😄

@megmut thanks for helping to make Pixi better. I'll put together a PR for this.

@megmut Here's the build with that fix. Would appreciate it if you could verify that it works as expected: http://pixijs.download/fix-spritesheet-resolution/pixi.js

I can't import your code directly as my project doesn't run from an already bundled pixi.js. I'm importing the classes on build with webpack and typescript. If you like, you could just give me the source of the class you changed and I can update my local build, fix the typings if need be and let you know :)

Thanks, that'd be helpful.


QIR: function names should consistently use underscore instead of period

E.g. https://github.com/microsoft/qsharp-language/blob/main/Specifications/QIR/Quantum-Runtime.md is still using periods:
quantum.rt.qubit_allocate should be __quantum__rt__qubit_allocate

@alan-geller Do you mind updating that?


[nrf noup] Bluetoooth: Add experimental for non-qualified options

Zephyr controller does not yet have a qualification for this release.
Bluetooth Mesh does not yet have a qualification for this release.
Manifest PR: https://github.com/nrfconnect/sdk-nrf/pull/3370
Signed-off-by: Joakim Andersson<EMAIL_ADDRESS>

I need to change the GitHub Actions settings on this repo, and it may mess up your status reports.   Can we merge this quickly?


Is it intended that not all links open in the in-app browser?

Avoid duplicates

[X] Question is not already answered in the FAQ
[X] Question has not already been asked in another issue

Your Question

Question: The FAQ link (safari_corona_website - link to code) on the main page opens in the in-app browser.
Links in f.e. the "Terms of Use" are all opened in the devices standard browser.

Is this the intended behavior, if yes, why?

@Ein-Tim We will try to get an answer.

@dsarkar
Thanks, although it's a bit weird that some links open via in-app and others don't, I'll close this issue now.


Charts api

added charts api. issue #5.

@arundhaj Thanks for the PR! But we are currently not supporting the Charts API on M2X client libraries because it is undergoing big changes. Take a look at the "Class Naming" section of the CLIENT-CONTRIBUTIONS guidelines.
Having said that, I will be closing this PR and issue #5, thank you and sorry about that


Frappe-Client.INSERT : : No JSON Object could be decoded

Code :
import json
from frappeclient import FrappeClient
data = json.loads(open("user.json").read())
client = FrappeClient("http://localhost:5000", "xxxxxxx", "xxxxxxxx")
notes = [
{
"doctype": "Employee",
"eid" : "242",
"name" : "niraj1",
"dob" : "05-05-1993",
"doj" : "01-08-1993",
"dept" : "R&D",
"manager" : "raghu"
}]
print(client.insert(notes[0]))
Error:
Traceback (most recent call last):
File "client.py", line 32, in
print(client.insert(notes[0]))
File "/home/niraj/workspace/frappe-client/frappeclient/frappeclient.py", line 53, in insert
return self.post_process(res)
File "/home/niraj/workspace/frappe-client/frappeclient/frappeclient.py", line 203, in post_process
rjson = response.json()
File "/usr/local/lib/python2.7/dist-packages/requests/models.py", line 896, in json
return complexjson.loads(self.text, **kwargs)
File "/usr/lib/python2.7/json/init.py", line 339, in loads
return _default_decoder.decode(s)
File "/usr/lib/python2.7/json/decoder.py", line 364, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python2.7/json/decoder.py", line 382, in raw_decode
raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded
Can anyone help me here I am really banging my head in this seems to be a silly issue?
-Niraj

Hey @nirajKpanda
Did you figure this one out?

This is my code. Not sure whats wrong here
from frappeclient import FrappeClient
import pyodbc

def set_uom():

    https = FrappeClient("https://me.com", "api", "Me1234")

    doc = https.insert({
            'doctype': 'Item Group',
            'parent': 'Foods',
            'name': 'new-group'
        })

set_uom()

It's working now.
Sorry for the long pending issue, should have closed it.
Sent from Outlook Mobilehttps://aka.ms/blhgte

From: Adam Dawoodjee<EMAIL_ADDRESS>Sent: Monday, April 20, 2020 6:24:18 PM
To: frappe/frappe-client<EMAIL_ADDRESS>Cc: Niraj Kumar Panda<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [frappe/frappe-client] Frappe-Client.INSERT : : No JSON Object could be decoded (#23)
This is my code. Not sure whats wrong here
from frappeclient import FrappeClient
import pyodbc
def set_uom():
https = FrappeClient("https://me.com", "api", "Me1234")

doc = https.insert({
        'doctype': 'Item Group',
        'parent': 'Foods',
        'name': 'new-group'
    })

set_uom()
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/frappe/frappe-client/issues/23#issuecomment-616533369, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACXIA4OIUVNZIXQOVE52ZRLRNRAXVANCNFSM4FQNQJGQ.

It's working now. Sorry for the long pending issue, should have closed it. Sent from Outlook Mobilehttps://aka.ms/blhgte

I'm also trying to solve this problem. What was your solution?

I stopped working on this project about 2 years back.
I don't have a code trace for reference. Sorry I could not be of help in this.
Hope wider Frappe community will help you in this.
Thanks,
Niraj Kumar Panda

From: Adam Dawoodjee<EMAIL_ADDRESS>Sent: Monday, April 20, 2020 6:42 PM
To: frappe/frappe-client<EMAIL_ADDRESS>Cc: Niraj Kumar Panda<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [frappe/frappe-client] Frappe-Client.INSERT : : No JSON Object could be decoded (#23)
It's working now. Sorry for the long pending issue, should have closed it. Sent from Outlook Mobilehttps://aka.ms/blhgte
I'm also trying to solve this problem. What was your solution?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/frappe/frappe-client/issues/23#issuecomment-616543761, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACXIA4MOANYXSFEFP33HJ3TRNRC45ANCNFSM4FQNQJGQ.

The issue was a wrong field name. Item group name variable is item_group_name not name.
This resolved my issue.
Thanks for your time Niraj. I advise we close the issue now.


Argument labels decoded in the wrong order for a contract interaction transaction.

The "dataDecoded" for the transaction has the parameter 0 "receiver" and parameter 1 "tokenAddress", however in the contract code the parameter 0 is "_tokenAddr" and parameter 1 is "_to". i.e. the order of labels is switched.

This is a collision based on the method id. Fixing this would have performance impact and therefore we will not do so right now.


Get Projects and Get Users endpoints have different name for querying

Observed/problematic behavior
Consistency between q for projects and query for users
Expected behavior
either query in both urls or q
Steps to reproduce
https://hangar.papermc.io/api/v1/users?query=D&limit=2&offset=0
https://hangar.papermc.io/api/v1/projects?limit=10&offset=0&prioritizeExactMatch=true&q=a
View the difference in the URL query param name
Other
No response

query is the proper name I assume as all other endpoints use it


How to get fundamental data for stocks in other exchanges ?

I am not able to fetch any fundamental data for stocks other than NASDAQ. Is this a known limitation ?
An example:
I want to fetch the fundamental data for INDIGO trading in NSE or BSE.
Steps 1:

Search for the symbol:

https://www.alphavantage.co/query?function=SYMBOL_SEARCH&keywords=indigo&apikey=[apiKey]

{
    "bestMatches": [
        {
            "1. symbol": "INDIGO.BSE",
            "2. name": "InterGlobe Aviation Ltd",
            "3. type": "Equity",
            "4. region": "India/Bombay",
            "5. marketOpen": "09:15",
            "6. marketClose": "15:30",
            "7. timezone": "UTC+5.5",
            "8. currency": "INR",
            "9. matchScore": "0.8000"
        }
   ]
}

Get overview of the company

https://www.alphavantage.co/query?function=OVERVIEW&symbol=INDIGO.BSE&apikey=YDUI9TGWSVE7ZPWV

This gives an empty json

Try to get the fundamental data

        data = {
            "function":"BALANCE_SHEET",
            "symbol":"INDIGO.BSE",
            "outputsize":"full",
            "datatype":"json",
            "apikey": [apikey]
        }
        return requests.get("https://www.alphavantage.co/query", data).json()

This gives empty json.

[2023-07] Temporarily closing this issue due to inactivity.


Add experimental Span Suggesters

This PR adds three new experimental suggester functions for the spancat component and a spaCy project showcasing how to use them in a config.cfg file.
Subtree Suggester:

Uses annotations from the Tagger and Parser to suggests subtrees of individual tokens

Chunk Suggester:

Uses annotations from the Tagger and Parser to suggest noun_chunks

Sentence Suggester:

Uses sentence boundaries to suggest sentences

These suggesters also come with the ngram functionality which allows users to set a list of sizes for suggesting individual ngrams
The spaCy project covers:

How to source components from existing models
How to use frozen_components & annotating_components
How to use custom suggester functions registered in the registry

The general plan is to initially release things like this under spacy-experimental for easier testing, and then move features to the core library or other places once they seem stable and generally useful. Not everything will make the cut and the APIs might change a bit, but I think these particular suggesters have a very high chance of being moved to the core library soon.

I think that these span suggesters would be simpler and faster without the ngrams thrown in, and it would make sense to have a more general way of combining arbitrary suggesters/suggestions than what's proposed here. (In particular sentences + ngrams doesn't make a lot of sense to me?)

I think, having the ngram functionality as a baseline will be useful for most use-cases. For example, the sentences + ngrams could be used for the healthsea dataset. I also like that it's optional to use them, when leaving the sizes list empty, the suggester will skip the whole code block that handles the ngram suggesting.

It is a problem that it's hard to specify a variable number of things (each with their own configs) to combine in a config block, which is something that's also come up with augmenters.
If we're going to do this, I think it would be better to have a utility function that merges suggestions in a more general, efficient way. The ops.to_numpy() for each row in the array is going to be particularly slow in the current version and appending everything row-by-row vs. working with the existing numpy array looks like it may also slow things down. (In a case where you already know you want it as numpy, I think you could just pass NumpyOps as the ops instead to the suggester and then you could skip all the conversions?)
I decided to sketch out what a suggestions merger could look like (could still use some typing and testing, particularly on GPU; note that there's no cupy.unique so you have to convert to numpy):
from typing import List, Optional, Iterable, cast
import numpy
from thinc.api import get_current_ops, Ops
from thinc.types import Ragged, Ints1d
from spacy.tokens import Doc
from spacy.util import registry
from spacy.pipeline.spancat import Suggester

@registry.misc("experimental.ngram_sentence_suggester.v1")
def build_ngram_sentence_suggester(sizes: List[int]) -> Suggester:
    """Suggest ngrams and sentences. Requires sentence boundaries"""

    ngram_suggester = registry.misc.get("spacy.ngram_suggester.v1")(sizes)
    def ngram_sentence_suggester(
        docs: Iterable[Doc], *, ops: Optional[Ops] = None
    ) -> Ragged:
        ngram_suggestions = ngram_suggester(docs, ops=ops)
        sentence_suggestions = sentence_suggester(docs, ops=ops)
        return merge_suggestions([ngram_suggestions, sentence_suggestions], ops=ops)

    return sentence_suggester

def sentence_suggester(docs: Iterable[Doc], *, ops: Optional[Ops] = None) -> Ragged:
    if ops is None:
        ops = get_current_ops()
    spans = []
    lengths = []

    for doc in docs:
        sents = list(doc.sents)
        spans.extend((sent.start, sent.end) for sent in sents)
        lengths.append(len(sents))

    lengths_array = cast(Ints1d, ops.asarray(lengths, dtype="i"))
    if len(spans) > 0:
        output = Ragged(ops.asarray(spans, dtype="i"), lengths_array)
    else:
        output = Ragged(ops.xp.zeros((0, 0), dtype="i"), lengths_array)

    return output

def merge_suggestions(suggestions: List[Ragged], ops: Optional[Ops] = None) -> Ragged:
    if ops is None:
        ops = get_current_ops()

    spans = []
    lengths = []

    if len(suggestions) == 0:
        lengths_array = cast(Ints1d, ops.asarray(lengths, dtype="i"))
        return Ragged(ops.xp.zeros((0, 0), dtype="i"), lengths_array)

    len_docs = len(suggestions[0])
    assert all(len_docs == len(x) for x in suggestions)

    for i in range(len_docs):
        combined = ops.xp.vstack([s[i].data for s in suggestions])
        uniqued = numpy.unique(ops.to_numpy(combined), axis=0)
        spans.append(ops.asarray(uniqued))
        lengths.append(uniqued.shape[0])

    lengths_array = cast(Ints1d, ops.asarray(lengths, dtype="i"))
    if len(spans) > 0:
        output = Ragged(ops.xp.vstack(spans), lengths_array)
    else:
        output = Ragged(ops.xp.zeros((0, 0), dtype="i"), lengths_array)

    return output

And I wouldn't be surprised if there's a faster way to manage the per-doc unique step, but this has reached the end of my numpy knowledge.

Wow, that's awesome! Thanks adriane! 🥳
I've implemented the merge_suggesters function and also added suggester registries that don't use the ngram at all, if the user does not want to:
  spacy-experimental.subtree_suggester.v1
  spacy-experimental.ngram_subtree_suggester.v1
  spacy-experimental.chunk_suggester.v1
  spacy-experimental.ngram_chunk_suggester.v1
  spacy-experimental.sentence_suggester.v1
  spacy-experimental.ngram_sentence_suggester.v1

Can you remove the formatting changes to azure-pipelines.yml?

Can you add documentation for these to the top-level README?

Can you add sample configs for one of the types of suggesters that also explains how to use sizes for the ngram versions?

Not to keep nosing into y'all's business, but once this merges would consumer (my) feedback on trying a model using this be warranted/appreciated?

@Lolologist: Feedback is welcome, that's pretty much what spacy-experimental is there for! We want it to be easy for users to try things out so we can improve them before they're added to the core library.

Added example snippets to the readme with 1f52f1e 🎉


[Mesheryctl] Installation script fails

Description
Mesheryctl installation script fails with the following error:
$ curl -L https://git.io/meshery | bash -
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  3156  100  3156    0     0   4401      0 --:--:-- --:--:-- --:--:--  4401
Downloading mesheryctl-v0.5.1 for Linux...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100     9  100     9    0     0     49      0 --:--:-- --:--:-- --:--:--    49
Extracting mesheryctl-v0.5.1...
Archive:  /root/meshery.zip
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of /root/meshery.zip or
        /root/meshery.zip.zip, and cannot find /root/meshery.zip.ZIP, period.
Installing mesheryctl in /usr/local/bin.
mv: cannot stat '/root/mesheryctl': No such file or directory
mv: cannot stat '/root/mesheryctl': No such file or directory
Cannot install mesheryctl. Check permissions of root for /usr/local/bin.

Environment:

Meshery Version: v0.5.1
Kubernetes System Type: docker-desktop
Host OS:
Browser:

I tried this locally on a Linux m/c.
Extracting mesheryctl-v0.5.1...
+ unzip -o /root/mesh/test/meshery.zip
Archive:  /root/mesh/test/meshery.zip
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of /root/mesh/test/meshery.zip or
        /root/mesh/test/meshery.zip.zip, and cannot find /root/mesh/test/meshery.zip.ZIP, period.
+ install_dir=
+ '[' -z '' ']'
+ install_dir=/usr/local/bin
+ printf '\nInstalling mesheryctl in /usr/local/bin.\n'

Installing mesheryctl in /usr/local/bin.
++ whoami
+ WHOAMI=root
+ mv /root/mesh/test/mesheryctl /usr/local/bin/mesheryctl
mv: cannot stat '/root/mesh/test/mesheryctl': No such file or directory
+ sudo mv /root/mesh/test/mesheryctl /usr/local/bin/mesheryctl

unzip command is failing on Linux OS.
I do not think its due to the recently introduced "-o" option to unzip, as the unzip command fails even without -o option.
# unzip meshery.zip
Archive:  meshery.zip
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of meshery.zip or
        meshery.zip.zip, and cannot find meshery.zip.ZIP, period.

Checked whether the meshery.zip file is correct or not.
+ URL=https://github.com/layer5io/meshery/releases/download/v0.5.1/mesheryctl_0.5.1_Linux_x86_64.zip
+ printf '\nDownloading %s for %s...\n\n' mesheryctl-v0.5.1 Linux

Downloading mesheryctl-v0.5.1 for Linux...

+ curl -L https://github.com/layer5io/meshery/releases/download/v0.5.1/mesheryctl_0.5.1_Linux_x86_64.zip -o /root/mesh/test/meshery.zip
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100     9  100     9    0     0     18      0 --:--:-- --:--:-- --:--:--    18
+ printf '\nExtracting %s...\n' mesheryctl-v0.5.1

# file meshery.zip
meshery.zip: ASCII text, with no line terminators
# cat meshery.zip
Not Found

The issue is with https://github.com/layer5io/meshery/releases/download/v0.5.1/ path. It throws 404.

@hexxdump Are you pushing the fix for this tonight?

I see that the new github path for release v0.5.1 has only the source.zip file - https://github.com/layer5io/meshery/archive/refs/tags/v0.5.1.zip
But not the platform specific zip files. This could be an issue with how the https://github.com/layer5io/meshery/releases/ page is updated.
NAME="mesheryctl-${MESHERY_VERSION}"
URL="https://github.com/layer5io/meshery/releases/download/${MESHERY_VERSION}/mesheryctl_${MESHERY_VERSION:1}${OSEXT}${OSARCHITECTURE}.zip"
The fix should be to copy the OS and architecture specific zip files under https://github.com/layer5io/meshery/releases/download/v0.5.1/ path

On Ubuntu 20.04.2 Focal
hwpplayer1@masscollabs:~$ curl -L https://git.io/meshery | bash -
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0
100  3156  100  3156    0     0   1605      0  0:00:01  0:00:01 --:--:--  4745

Downloading mesheryctl-v0.5.1 for Linux...

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100     9  100     9    0     0     11      0 --:--:-- --:--:-- --:--:--    11

Extracting mesheryctl-v0.5.1...
Archive:  /home/hwpplayer1/meshery.zip
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of /home/hwpplayer1/meshery.zip or
        /home/hwpplayer1/meshery.zip.zip, and cannot find /home/hwpplayer1/meshery.zip.ZIP, period.

Installing mesheryctl in /usr/local/bin.
mv: cannot stat '/home/hwpplayer1/mesheryctl': No such file or directory
[sudo] password for hwpplayer1:
mv: cannot stat '/home/hwpplayer1/mesheryctl': No such file or directory
Cannot install mesheryctl. Check permissions of hwpplayer1 for /usr/local/bin.
hwpplayer1@masscollabs:~$


[iam-role-for-service-accounts-eks] Cluster ID isn't used to constrain built in policies

Is your request related to a new offering from AWS?
No
Is your request related to a problem? Please describe.
This module doesn't constrain the built in service policies by the cluster ID/name as is documented in the service deployment documentation. Two services (CA & Karpenter) do have cluster ID inputs but as they're lists this suggests that a shared role is appropriate, which for the services with the highest blast radius should be strongly advised against.
Describe the solution you'd like.
I'd like to see a cluster ID input for built in policies where it's relevant. I'd also like to see cluster_autoscaler_cluster_ids and karpenter_controller_cluster_ids changed to be for a single ID. For all other non-idiomatic usage the user should be providing custom policies.
Describe alternatives you've considered.
I already have this implemented myself, but I was looking to standardise.
Additional context
n/a

Karpenter was updated with #209


[core-tracing] Downgrade to "^0.18.0" of opentelemetry/api for compatibility

It appears moving to 1.0.0-rc.0 was premature as the remainder of the opentelemetry ecosystem have not yet moved to it. It looks like it's prepped to happen at any time, but relying on that timing means we can't get a consistent set of tracing packages within our own stack, which isn't a good idea.
There is an version matrix here - 0.18.x is the most modern version that is listed as compatible with the latest core-version. So we're a bit ahead and should go back.
As they sort out some versioning/compatibility issues between 1.0-rc.0 and the rest of the opentelemetry stack we will:

Downgrade core-tracing's opentelemetry/api to ^0.18.0 (from 1.0.0-rc.0)
Upgrade everyone to core-tracing-preview.12 (which only needs that downgrade, no code changes required)
Update centralized rollup config to accommodate the version range.
Bump cache version to 5 (for now).

NOTE: @xirzec, I am fairly certain we're going to run into the "Two incompatible versions of @azure/core-tracing have been loaded." - this happened last time in the samples and I erroneously bumped GLOBAL_TRACER_SYMBOL up in addition to GLOBAL_TRACER_VERSION` to fix it. This happened last time because various packages were pulling stuff in a combination from either npm or from the local tree and thus getting two different core-tracing versions.
I believe the only way to fix this is time the releases of the packages so we get 1) core-tracing, then 2) core* and 3) identity out. Then no matter where the packages are pulling their dependencies from they'll get the same core-tracing package.
So:

Release core-tracing
Release core-http, core-rest-pipeline and core-client (I believe)
Release identity hotfix since all packages are using the GA identity and not the in-tree version of identity. We have this branch: https://github.com/azure/azure-sdk-for-js/tree/hotfix/identity_1.3.0. It's not yet released, but @sadasant will be working on that this week. @sadasant, we'll need to update that branch to refer to<EMAIL_ADDRESS>after step #1 above.
Update the entire tree to<EMAIL_ADDRESS>

@sadasant : this PR will probably require us to cancel https://dev.azure.com/azure-sdk/internal/_build/results?buildId=821590&view=results and restart it after updating to this release version of core-tracing.

@hectorhdzg (this is the downgrade PR we were talking about. Thank you for notifying me!)

/azp run js - keyvault-secrets - tests

/azp run js - keyvault-admin - tests

/azp run js - keyvault-certificates - tests

/azp run js - keyvault-keys - tests

/azp run js - service-bus - tests

/azp run js - storage-blob - tests

/azp run js - communication-chat - tests

/azp run js - storage-file-datalake - tests

/azp run js - mixedreality-authentication - tests

So if I understand it correctly, we moved too optimistically to 1.0rc, and no actual OT implementations support it yet. I am a little surprised that we didn't catch this issue during testing of the 1.0rc PR or before releasing the new core-tracing package.
Since previews get pinned and this is fairly disruptive to re-release all of core, I have to ask, is it worth doing this?
OpenTelemetry won't work for some period of time, but presumably they will eventually release 1.0 compatible libraries and we will start working again. @ramya-rao-a what do you think?

@xirzec , @richardpark-msft I have no urgency in releasing this 1.3.0 hotfix for Identity. I'm thinking we can push this to be released on May. I don't want to rush something that alters so many packages.

I am assuming this is also related to https://github.com/open-telemetry/opentelemetry-js/issues/2055 which in turn does not affect us since we do manual propagation.

OpenTelemetry won't work for some period of time, but presumably they will eventually release 1.0 compatible libraries and we will start working again. @ramya-rao-a what do you think?

I doubt it is the case the OT does not work at all at the moment when the code from the master branch is used.
I'd like to get a confirmation that what is in the master branch i.e. core-tracing v11 meets our tracing needs.
@sadasant, @maorleger Can either of you check this for Key Vault and @jeremymeng, can you check this for storage?

And @xirzec, can you confirm if the endless loop being discussed in https://github.com/open-telemetry/opentelemetry-js/issues/2055 affects us or not?

And @xirzec, can you confirm if the endless loop being discussed in open-telemetry/opentelemetry-js#2055 affects us or not?

The loop issue discussed is happening inside an exporter, which perhaps @hectorhdzg is interested in for the monitor exporter, but isn't related to us producing spans.
My hypothesis is we aren't affected, but I'm trying to figure out a good repro/test case to validate our current packages on.

@xirzec , @richardpark-msft I have no urgency in releasing this 1.3.0 hotfix for Identity. I'm thinking we can push this to be released on May. I don't want to rush something that alters so many packages.

@sadasant , you'll still want to release the 1.3.0 hotfix - the original purpose of that release was to make it so everyone could finally reference the same core-tracing package (ie, this is independent of this particular PR). You should just go with the<EMAIL_ADDRESS>rather than relying on this PR to go out. (in which case you can also just use the release pipeline we had).

So if I understand it correctly, we moved too optimistically to 1.0rc, and no actual OT implementations support it yet. I am a little surprised that we didn't catch this issue during testing of the 1.0rc PR or before releasing the new core-tracing package.

I did run a basic test but my "test" must be unrealistic, although it sounds like this issue isn't necessarily easy to hit.

Since previews get pinned and this is fairly disruptive to re-release all of core, I have to ask, is it worth doing this?

I'd be perfectly fine is the answer was 'no'. This was a proactive PR and I didn't have a real chance to see if we could ignore this problem or not. :)

OpenTelemetry won't work for some period of time, but presumably they will eventually release 1.0 compatible libraries and we will start working again. @ramya-rao-a what do you think?

Just chiming in that I'm totally fine with this. It's a bit unclear how the authors were planning on patching this, but they already have PRs out that are trying to move to the opentelemetry/pi 1.0.0-rc.0 version for their existing packages.

@xirzec the infinite loop is actually triggered by the exporter because the actual http calls to Azure Monitor will be generating Spans as well by @opentelemetry/instrumentation-http, but the issue is with different global contexts, SpanProcessors are responsible to set this context before calling the export method https://github.com/open-telemetry/opentelemetry-js/blob/main/packages/opentelemetry-tracing/src/export/BatchSpanProcessor.ts#L147, you must be able to repo using @opentelemetry/instrumentation-http, Azure SDK using OpenTelemetry and any exporter sending the data through Http

@richardpark-msft packages using core-tracing 1.0.0-preview.11 seem to work fine together. Is that correct? Would merging this PR break packages currently depending 1.0.0-preview.11?

I did some testing with @azure/storage-blob from master and the following:
    "@azure/core-tracing": "1.0.0-preview.11",
    "@opentelemetry/core": "^0.18.2",
    "@opentelemetry/exporter-zipkin": "^0.18.2",
    "@opentelemetry/node": "^0.18.2",
    "@opentelemetry/tracing": "^0.18.2",

It produced the correct spans as expected through zipkin without any trouble. Not sure if 0.18.2 magically fixed everything, but it seems to be working fine now, so I'm in favor of closing this PR.

I did some testing with the following gist: https://gist.github.com/sadasant/47c84fb78a091ee37b62d462c3b59333
It shows that nothing breaks and that the output makes sense.
My test worked with: core-tracing ^1.0.0-preview.11, identity 2.0.0-beta.1 (with core-tracing 1.0.0-preview.11) and keyvault-certificates (also 1.0.0-preview.11).
When I switch to Identity 1.3.0, with core-tracing 1.0.0-preview.12, the test fails (I'm guessing it means that keyvault-certificate would need to upgrade to 1.0.0-preview.12).

I tested mixing current master of storage-blob and latest released version of storage-queue in one app (without using tracing). It worked as I expected. They depend on different versions of core-tracing which have the opentelemetry versions pinned

Posting my findings as well:
    "@azure/core-tracing": "^1.0.0-preview.11",
    "@azure/identity": "^1.2.5",
    "@azure/keyvault-keys": "file:../../../keyvault-keys/azure-keyvault-keys-4.2.0-beta.5.tgz",
    "@azure/keyvault-secrets": "file:../../azure-keyvault-secrets-4.2.0-beta.4.tgz",
    "@opentelemetry/api": "^1.0.0-rc.0",
    "@opentelemetry/exporter-zipkin": "^0.18.2",
    "@opentelemetry/node": "^0.18.2",
    "@opentelemetry/tracing": "^0.18.2",
    "@types/node": "^14.14.37",
    "dotenv": "^8.2.0",
    "typescript": "^4.2.3

I kept keyvault-secrets pinned to master but tried keyvault-keys latest, next, dev, and master to check and no errors without tracing. Tracing seemed to be working well for me with the above deps.

Thanks @xirzec, @sadasant, @maorleger and @jeremymeng!
With the above findings we conclude that we do not need to downgrade opentelemetry/api and can go ahead with the changes in our master branch


feat: add validation & error handling ux improvements

Description
The PR adds some validation and improved messaging. I just added an error message for missing quote in the case of no other error. We could refactor to get the actual error but more often than not missing quote errors are because missing rates so I've just added a message to reflect this
Other changes
Increased the refetch interval of the swap quote by two seconds.
Add debouncing to validation logic to prevent excessive SDK calls and heavy computations on each keystroke
Tested
Swapped and tried to trigger errors on the preview deployment
Related issues

Fixes #156 #157

Checklist before requesting a review

[x] I have performed a self-review of my own code
[ ] The PR title follows the [conventions](https://www.notion.so/Git-Branching-and-Commit-Message-Conventions-

@nvtaveras Here are some issue found during testing:

Doesn't handle a case when no wallet connected and put a large number (e.g from the bug ticket:<PHONE_NUMBER>223123234925). I think it should not change state to the grey continue button and after to the red with connect wallet text. One of them would be nice:
Uploading Screen Recording 2024-12-04 at 12.50.04.mov…
It looks like validation with the grey continue button state after making some actions works fine BUT, I can't get my loadded balances for tokens after wallet connection - do you have this opportunity on vercel link?

https://github.com/user-attachments/assets/ddb6db49-91c2-4c61-b102-9fe7000c4e69

Still can proceed with a larger amount than balance (reproduced with CELO that equals 20 with<PHONE_NUMBER>22312323492566622277023 amount).
It breaks the page on the Confirm Swap state

Bug-fix list related to this ticket:

[ ] Doesn't handle a case when no wallet is connected and puts a large number (e.g., from the bug ticket:<PHONE_NUMBER>223123234925)


Fix badge : compatibility score

Add previous-version: 5.72.1
Add new version: 5.73.0

What kind of change does this PR introduce?
I fixed the broken compatibility score badge.

Did you add tests for your changes?
I read the dependabot documentation.
In order to fix the existing broken badge, I applied dependabot's new method.
https://api.dependabot.com/badges/compatibility_score?dependency-name=${name}&package-manager={package-manger}&previous-version=${previous}.1&new-version=${newversion}

name: webpack
package-manger: npm_and_yarn
pervious: 5.72.1 (latest old version)
newversion: 5.73.0 (latest version)

Does this PR introduce a breaking change?
Nope

What needs to be documented once your changes are merged?
Nope

Thanks


v2-20.3.00

[v3.7.0-rc] - eSignature API v2-20.3.00 - 09/24/2020
Changed

Added support for version v2-20.3.00 of the DocuSign eSignature API.
Updated the SDK release version

Fixed

A bug with ApiClient SetBasePath which did not update basePath for RestClient (DCM-4276).

AppVeyor build and test pass.


Mail not working, how can I solve it?

EDIT by @nledez
So right now I'm using Kitsu from a docker container with good success, the only problem so far is that the mailing options don't work, so users cannot recover their password or receive notifications.
How can I solve this?
[ ..... cut part .... ]
Now... back on track... how can I solve the mail not working?
Thanks!
Not related to issue (moved by @nledez - may put this in a README pull request):
BTW I think it could be a good idea to mention this to other users, to have persistent data, but at the same time have everything in a container that is a folder in the host computer I'm using this command to create a named volume that points to a folder in the host system:
docker volume create --driver local -o o=bind -o type=none -o device="/Volumes/MacServer/Docker/kitsu" kitsu-storage
And the command I use to run the container is this one:
docker run --init -d -p 80:80 --name kitsu_bone -v kitsu-storage:/var/lib/postgresql -v kitsu-storage:/opt/zou/previews cgwire/cgwire
After that I use this to identify the container:
docker ps
And then I use this command to make the container auto-start always:
docker update --restart=always co1n2t345678
Being the big number the container ID I got with the previous command.
This way I'm able to access the actual data because it's inside an actual folder of my system.
Maybe you want to add this process to the possible process for other users, because I think it makes it easier to use the docker image as a production ready version :)

For future reference:
Using Docker Compose and this image:
https://gitlab.com/mathbou/docker-cgwire
I've been able to correctly configure the email and not it's correctly working, thanks! :)


Fix for November 4th Rust update

This is contingent upon a new OnSamSiteTargetScan(SamSite samSite) hook being added.


ctrl-c not working on 'Bash on Ubuntu for windows'

Ctrl-c does not produce the expected results in bash. It doesn't kill a process and it doesn't properly get transmitted to the underlying shell, e.g., in emacs Ctrl-X Ctrl-C should close emacs, but instead of Ctrl-c, emacs receives only c.
It seems that when pressing Ctrl-c the shell receives only c (it prints c when pressing Ctrl-C in bash).
My build number is 15007.

Duplicate of #1569.


Update renovatebot/github-action action to 34.21.4

This PR contains the following updates:

Package
Type
Update
Change

renovatebot/github-action
action
patch
v34.21.3 -> v34.21.4

Release Notes

renovatebot/github-action
v34.21.4
Compare Source
See the the changelog for changes in all releases.
34.21.4 (2022-11-09)
Bug Fixes

deps: update renovate/renovate docker tag to v34.21.4 (7024e5c)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

@balena-ci I self-certify!


cherry-pick manpages fix

cherry-pick of https://github.com/docker/docker/commit/1e83886dd52f92e8ea738b33ec6ec6bd992c1263 (https://github.com/docker/docker/pull/24950)

LGTM 👼

updated to use the individual commits, not the merge-commit

LGTM


Use static interfaces for Ambient declarations in lib.d.ts

Ported from : https://typescript.codeplex.com/workitem/1085
Some core ambient vars already do this when the only interface is the static one e.g. JSON : https://github.com/Microsoft/TypeScript/blob/master/src/lib/core.d.ts#L916-L959
Would be nice to do the name for others like String, Date perhaps calling them StringStatic and DateStatic. It is need for definitions for quite a few JavaScript libraries.

It looks like this is closed.

StringConstructor be implemented as part of a class or does it contain special functions that can't be implemented

@series0ne  Can't be implemented by a class. But stuff like this can be provided by a function : http://stackoverflow.com/questions/25340601/typescript-hybrid-type-implementation
:+1: For closing. Haven't had any issues lately :rose:

This looks like fixed, but how do I go about adding a custom static property on an inbuilt class?
The concrete example would be to add type info to be able to use this:
Element.ALLOW_KEYBOARD_INPUT
While I've decided not to use this any more (only worked in safari, seemingly doesn't even work there any more), it's a good example because Element doesn't have this defined in lib.d.ts.

Hey there. I know this issue is long closed now, but don't know if I should open a new issue for that or maybe I'm just missing something.
I wanted to extend CustomEvent constructor, to add a custom interface for detail when given specific event name (need to do that for addEventListener too). For example:
interface CustomEvent {
  new(typeArg: 'my-event', eventInitDict?: CustomEventInit & { detail: MyEventDetail }): CustomEvent;
}
interface GlobalEventHandlersEventMap {
  "my-event": Event & { detail: MyEventDetail };
}

Is there a chance to achieve that? As far as I can see CustomEvent is declared as var:
declare var CustomEvent: {
    prototype: CustomEvent;
    new(typeArg: string, eventInitDict?: CustomEventInit): CustomEvent;
};

So far I have played around but couldn't make it to work :(.


Merge pull request #1 from mdhiggins/master

Update to current master

What happened here?

wrong button. Was trying to update my fork to current master


Complete the Star Trek theme

Fill out the words array with 75 unique Star Trek terms

Just updated the words collection for the Star Trek theme


device: Optimize device_get_binding search efficiency

Optimize 'device_get_binding' function search so that when a matching device is found, it returns NULL if it is not ready.
Another solution.
	STRUCT_SECTION_FOREACH(device, dev) {
		if (dev->name == name ||strcmp(name, dev->name) == 0) {
			return z_impl_device_is_ready(dev) ? dev : NULL;
		}
	}

But. This simplifies it to a single loop and preserves the original requirement (I'm not sure if this solution will be unusual in large-scale deployments)

I kind of prefer the other solution mentioned above, but I think there should be additional parenthesis because this is a core part of the OS and should abide by certain MISRA rules.
STRUCT_SECTION_FOREACH(device, dev) {
		if ((dev->name == name) || (strcmp(name, dev->name) == 0)) {
			return z_impl_device_is_ready(dev) ? dev : NULL;
		}
	}

Another thought, and maybe I'm mistaken, but IIRC Zephyr rarely ever does an actual string comparison on the device name anymore. I think it would only be needed for devices that are not included in Devicetree, and potentially for LLEXT modules.
It's hard to say how much of an optimization this is as well without having exact numbers. I also suspect that since most devices are initialized in a predetermined order, we might be optimizing for the cold path.

I kind of prefer the other solution mentioned above, but I think there should be additional parenthesis because this is a core part of the OS and should abide by certain MISRA rules.
STRUCT_SECTION_FOREACH(device, dev) {
		if ((dev->name == name) || (strcmp(name, dev->name) == 0)) {
			return z_impl_device_is_ready(dev) ? dev : NULL;
		}
	}

+1

Once merged, this would close #41665

It's hard to say how much of an optimization this is as well without having exact numbers. I also suspect that since most devices are initialized in a predetermined order, we might be optimizing for the cold path.

The key point of optimization is that if the device is not ready, but it already exists, it will run the second loop after the first loop exits, and repeat the search and then return. After optimization, it is obvious that if it is not ready, it returns NULL and no more search(because it is very likely that it is still not ready in the second loop, which makes no sense. Maybe the speed may not be improved, in the extreme case, that is, in the second round of search, it is still not prepared and runs the second search in full.)

we might be optimizing for the cold path.

You are right, the patch includes cold path optimization :)


Test smartcar_old.ino with the headset before merge

This is an old file which does requrie continuous input to run, therefore matching the older version which is confirmed to work with the headset. The latest features like turning on the spot and input mapping as well as formatting has been added.

I don't know why the tests failed, or what they mean. But, the file compiled correctly in the arduino ide.

I don't know why the tests failed, or what they mean. But, the file compiled correctly in the arduino ide.

You should fix the project structure and place arduino sketches into separate folders that are named appropriately.
Also, remove any files/sketches you are not using.

I don't know why the tests failed, or what they mean. But, the file compiled correctly in the arduino ide.

Errors has been located and fixed in the latest commit.


undefined symbol: g_unix_get_passwd_entry : Error when quantized tf_yolov4_coco_416_416_60.3G_2.5

Hi all,
I'm trying to quantize tf_yolov4_coco_416_416_60.3G_2.5 with Vitis-AI 2.5 on vitis-ai-cpu container.
I did it following to README.md.
But I got errors as below.
"""""""""""""""""""""""""""""""""
(vitis-ai-tensorflow) Vitis-AI /workspace/tf_yolov4_coco_416_416_60.3G_2.5/code/quantize > bash ./quantize_416.sh
Traceback (most recent call last):
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/lib/python3.6/site-packages/tensorflow_core/contrib/decent_q/python/decent_q.py", line 260, in _parse_input_fn
module = import(input_fn_str.rsplit('.', 1)[0], fromlist=True)
File "./input_fn_416.py", line 6, in
import cv2
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/lib/python3.6/site-packages/cv2/init.py", line 181, in
bootstrap()
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/lib/python3.6/site-packages/cv2/init.py", line 153, in bootstrap
native_module = importlib.import_module("cv2")
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/lib/python3.6/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ImportError: /opt/vitis_ai/conda/envs/vitis-ai-tensorflow/bin/../lib/libgio-2.0.so.0: undefined symbol: g_unix_get_passwd_entry
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/bin/vai_q_tensorflow", line 11, in
sys.exit(run_main())
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/lib/python3.6/site-packages/tensorflow_core/contrib/decent_q/python/decent_q.py", line 1255, in run_main
app.run(main=my_main, argv=[sys.argv[0]] + unparsed)
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/lib/python3.6/site-packages/tensorflow_core/python/platform/app.py", line 40, in run
_run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/lib/python3.6/site-packages/absl/app.py", line 312, in run
_run_main(main, args)
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/lib/python3.6/site-packages/absl/app.py", line 258, in _run_main
sys.exit(main(argv))
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/lib/python3.6/site-packages/tensorflow_core/contrib/decent_q/python/decent_q.py", line 1254, in
my_main = lambda unused_args: main(unused_args, FLAGS)
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/lib/python3.6/site-packages/tensorflow_core/contrib/decent_q/python/decent_q.py", line 821, in main
input_fn = _parse_input_fn(flags.input_fn)
File "/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/lib/python3.6/site-packages/tensorflow_core/contrib/decent_q/python/decent_q.py", line 263, in _parse_input_fn
raise ValueError('Fail to import input_fn, error: ', e)
ValueError: ('Fail to import input_fn, error: ', ImportError('/opt/vitis_ai/conda/envs/vitis-ai-tensorflow/bin/../lib/libgio-2.0.so.0: undefined symbol: g_unix_get_passwd_entry',))
"""""""""""""""""""""""""""""""""
My PC's OS is Ubuntu 20.04LTS.
I installed glib (pip install glib) but it didn't make any effect on it.
How can I solve it?
Any help would be appreciated.
Thank you,

Hi @Satoski-H ,

What is the version of glib?
I have not reproduced the error. The steps to reproduce the issue are as follows:

conda activate vitis-ai-tensorflow
pip install opencv-python==<IP_ADDRESS>
pip install imgaug
bash quantize_416.sh

I hope this helps! Let me know if you have any further questions.

Hi @zhenzhen-AMD
Thank you for your great help!
The way you showed me worked fine.
The version of glib I installed was 1.0.0.
But I didn't export it into my environment so this time there was no glib installed.
Then it seemed that  my probrem was caused by the version of opencv-python, right?
Kind regards.

Hi @Satoski-H ,
I can't determine the specific reason for the error.
However, according to the error message you provided, it seems the error may be due to a missing glib.
In addition, this project also depends on the opencv-python and imgaug.
Best Regard.

Hi @zhenzhen-AMD
Thank you for your reply.
My error message appeared to say there was no glib though, it was okay without installing it.
Anyway thank you again for all of your kindness.
Kind regards.

You are welcome.


Added Rings implementation documentation

@yradsmikham this is a really good start to documenting what we need - I'll work with you on getting some of the knowledge filled in here.

@yradsmikham I've been thinking about how we can incorporate this into the high level GitOps pipeline description we have to maintain some continuity in the documentation.
Perhaps we can talk about a ring pipeline augmentation in a more abstract manner if we aren't providing/maintaining a Rings Operator application. I'm thinking we leave it at diagrams.
Additionally we should point out the impetus of why you might consider this pipeline augmentation up front. That way folks can immediately know if the situation applies to them.
If I'm understanding correctly, a developer would consider this pipeline augmentation when the following are true:

You want to practice Bedrock GitOps
You want a specific application/microservice rollout behavior (e.g. similar to blue/green or canary)
You don't want to use a service mesh
You have an application that will serve to replicate service behaviors ( BYO Ring Operator application)

Thoughts? @evanlouie @bnookala @samiyaakhtar

Closing this PR because it appears that the Ring Operator implementation using Traefik (v2.0) is currently blocked. We cannot open source a Ring Operator if there is not a working ingress operator that is compatible with it.


feat(Protocol-designer, components, step-generation): multi tiprack support

Closes RAUT-507
Overview
🔈  This is blocked until post-7.0 PD release 🔈
This PR introduces multi tiprack support in PD. During file creation, you can select up to 3 tipracks per pipette and edit them in the edit pipettes area. On the timeline, the compound command creators related to moving liquid (mix, consolidate, distribute, and transfer) now have a tiprack dropdown to specify which tip you want to use. If you don't select a tip, an error creator will show up telling you a tip is missing
Test Plan

Changelog

allow for multi tipracks in CreateFileWizard and PipetteFields, update tests
create TiprackField component for a dropdown in the liquid compound command creator forms
create a new arg TipRack
update pipetteTiprackAssignments type from string to string[] and add that change to the migration
update affected tests

Review requests
Is my approach in supporting multi tipracks correct? I changed the pipetteTiprackAssignments key value from a string to a string[]. This change shoudn't affect anything outside of PD since the pipetteTiprackAssignments key is only part of designerApplication in the schema.
Risk assessment
low

Closing but keeping the branch open for when multi tiprack support is implemented in the future


[10.x] Allow to specify protocols in URL validation

This PR makes it possible to specify which protocols are allowed in URL validation.
This can be useful in two situations:

To use exotic protocols that are not in the hardcoded protocols list without having to modify the list directly in Laravel
To authorize only some specific protocols (e.g. only https to force secure links)

There is no breaking changes, as the default behavior (when no parameters are used) is the same as current: only the protocols in the existing list are allowed.
Example usage:
$v = Validator::make($data, [
    'url' => 'sometimes|url', // current behaviour
    'link' => 'sometimes|url:http,https', // only http and https
    'secure_link' => 'sometimes|url:https', // only https
]);

Just combine url and starts_with validation rules.

Just combine url and starts_with validation rules.

@taylorotwell Thank you, it work in the second case, but, in the first case it’s not a solution as the rule will always fail if the protocol isn't in the list (just realized I added a test for this use case, but forgot it in my code example, sorry!)
In my case the valid minecraft:// protocol is failing, so there is a need for a  new PR each time a new protocol is required ?
Updated example usage:
$v = Validator::make($data, [
    'url' => 'sometimes|url', // current behaviour
    'link' => 'sometimes|url:http,https', // only http and https
    'game_join_url' => 'sometimes|url:minecraft,steam', // also support more protocols such as 'minecraft://'
]);

Thanks again


[primitives-fixes] xilinx: handle RAM128X1S's scalar address port; fix RAM256X1S INIT assignment

Great work! Many thanks!


[Package Request] Just

Just
Manifest:
Id: casey.just
Publisher: casey
Name: just
Version: 0.8.1
License: Creative Commons Legal Code
InstallerType: zip
Installers:
  - Arch: x64
    Url: https://github.com/casey/just/releases/download/v0.8.1/just-v0.8.1-x86_64-pc-windows-msvc.zip
    Sha256: 22E2D14EDBF76DDA21358C61392A7CC4041B20D60847F1D89FB4B5EDB665D2EF
ManifestVersion: 0.0.1

Blocked by: https://github.com/microsoft/winget-cli/issues/140


Performance : TypeScript 2.3 language service proxy

Bug Report

TSLint version: last
TypeScript version: 2.3
Running TSLint via: VSCode

Actual behavior
tslint uses its own TypeScript process, impacting performance.
Expected behavior
With TypeScript 2.3 language service proxy, already shipped in nightlys, tslint can use the main TypeScript process, with just this line in tsconfig.json :
{
  "compilerOptions": {
    "plugins": [{ "name": "tslint" }]
  }
}

There are already :

a try : https://github.com/angelozerr/tslint-language-service
instructions to do TypeScript plugins : https://github.com/RyanCavanaugh/sample-ts-plugin
a working example : @angular/language-service

Isn't angelozerr/tslint-language-service already doing this?

Closing, as a new VS Code extension is coming for this.


Rename, Upgrade and Improve Template

Description
Total overhaul after the fork:

update to golang 1.16
make the http template work with my internal firebase dependencies
improve final image size - dont copy source into final image
sync http and middleware
use distroless:nonroot instead of alpine
reduce number of layers during build

How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?
Checklist:
I have:

[x] updated the documentation and/or roadmap (if required)
[ ] read the CONTRIBUTION guide
[ ] signed-off my commits with git commit -s
[ ] added unit tests

I'm a bit confused about this PR? Did you mean to submit it to your own fork?

Jop 🙇, I'm sorry. I normally work on my self hosted Gitlab instance and don't create PR from forks 😬.


Run this

Well im new user with node js, how can I run this client on node js? Im have installed node js and python.
I get this:
D:\mumble\web>node index.js
module.js:338
    throw err;
          ^
Error: Cannot find module 'bindings'
    at Function.Module._resolveFilename (module.js:336:15)
    at Function.Module._load (module.js:278:25)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (D:\mumble\web\node_modules\jitterbuffer\index.js:2:20
)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)

I recommend you read a tutorial about node.js first
This is not a client but a library which could be used to created bots on top of it
You need to install the dependencies. npm install in the projects root direcrory should help you. Try npm install --python=python2 if you experience problems with node-gyp and python 3.
You do not want to run index.js, try one of the examples instead.

Also you'll need to sort out the build chain for node-jitterbuffer, node-celt and node-opus. I don't think those have Windows build configs at the moment.

Windows support done! So now it can be used there. The easiest way to confirm it on Windows is probably running the following commands:
git clone https://github.com/Rantanen/node-mumble
cd node-mumble
npm install
set MUMBLE_URL=your.mumble.address
node examples/sin.js

Although for actual usage you'll want to create your Node application and then do npm install mumble.

Very nice.

I'm going to close this for now as based on this there's nothing we can do.
If @robotrono still has problems running this in Windows, re-open it.


opendp-logger wrapper util and test

After talking with Mike, it was good to surface that opendp-logger hasn't kept up with features in opendp, but we shouldn't plan to use it in the POC. Closing without merging.


Improve the error when an invalid named argument is used

Description:
When an invalid named argument is used, atm the error is something like undefined defaultable parameter 'i'.
But now that we can use named arguments with positional params too, IMO the error should just be undefined parameter 'i'.
Steps to reproduce:
public function main() {
    // error: .::xyz.bal:4:9: undefined defaultable parameter 'i'
    // error: .::xyz.bal:4:5: missing required parameter 's' in call to 'foo'()
    foo(i = 1);

    // error: .::xyz.bal:7:9: undefined defaultable parameter 'i'
    bar(i = 1);
}

function foo(string s) {

}

function bar(string s = "hello") {

}

Affected Versions:
slp2

Hi @MaryamZi, How are you doing?
Is this issue is till open to work on, can you please let me know so that I can work on this issue!

@Suraj-kumar00, I'm doing well, hope you are too.
Yes, you can work on this issue.
To help you get started, here are some resources that may help you get started with Ballerina:

Understanding Ballerina Platform:
Learn what Ballerina is all about: Ballerina Platform

How to Contribute:
Read our contributing guidelines to understand how you can contribute effectively: Contribution Guide

Learn Ballerina:
If you're new to Ballerina or want to enhance your skills, our official website offers a wealth of learning resources: Learn Ballerina

Get Help and Connect:
Join our Discord community to chat with fellow Ballerina enthusiasts and get assistance: Ballerina Discord

Feel free to ask questions, propose ideas, or report issues. Together, we can make Ballerina even better!
Happy coding! 🎉

Hi @MaryamZi thanks for assigning this issue to me, can I know like how much time I have to work on this issue? And should I create a draft pull request for the conversation regarding this issue or do that here only?

Hi @MaryamZi thanks for assigning this issue to me, can I know like how much time I have to work on this issue? And should I create a draft pull request for the conversation regarding this issue or do that here only?

Anything that works for you would be okay.

The error message is defined in compiler.properties, where this needs to be updated.
Since the key is already undefined.parameter we wouldn't in any changes there or to DiagnosticErrorCode.java.
The errors are logged via BLangDiagnosticLog.java which you can debug to identify the flow as required.

The error message is defined in compiler.properties, where this needs to be updated.
Since the key is already undefined.parameter we wouldn't in any changes there or to DiagnosticErrorCode.java.
The errors are logged via BLangDiagnosticLog.java which you can debug to identify the flow as required.

Hi @MaryamZi, How are you doing?
So I was hoping you could help me understand this issue, as of now I understand that I have to the correct parameter names when calling the functions, the named argument i, which is not a valid parameter name for either foo or bar.
So should I do this
foo(s = "example");
bar(s = "world");
Did I get it wrong or should I do this?

So I was hoping you could help me understand this issue, as of now I understand that I have to the correct parameter names when calling the functions, the named argument i, which is not a valid parameter name for either foo or bar. So should I do this
foo(s = "example");
bar(s = "world");
Did I get it wrong or should I do this?

The requirement is to stop saying
undefined defaultable parameter 'i'

and say
undefined parameter 'i'

for code like the examples from the original description. So basically, remove the term "defaultable". Hope this is clear.

And also the code that you have provided above I never found them in the location that you provide me below of that code like in the three locations...

Sorry, this is not clear. Do you mean you cannot find the files when you check the code out locally?

So I was hoping you could help me understand this issue, as of now I understand that I have to the correct parameter names when calling the functions, the named argument i, which is not a valid parameter name for either foo or bar. So should I do this
foo(s = "example");
bar(s = "world");
Did I get it wrong or should I do this?

The requirement is to stop saying
undefined defaultable parameter 'i'

and say
undefined parameter 'i'

for code like the examples from the original description. So basically, remove the term "defaultable". Hope this is clear.

And also the code that you have provided above I never found them in the location that you provide me below of that code like in the three locations...

Sorry, this is not clear. Do you mean you cannot find the files when you check the code out locally?

Yeah the three file location you have provided above I haven't found there!!!

yes let me check and i'll inform you...

Hello @MaryamZi, How are you doing?
Actually, I'm facing a lot of errors. I don't know why. Can you please help me with this?

Hello @MaryamZi, How are you doing?
Can you please tell me about the above error I'm getting?

Can you please share what exactly the errors are?
Are you trying to add tests? Just updating the existing tests (e.g., in jballerina-unit-tests) should be enough.

Can you please share what exactly the errors are?
Are you trying to add tests? Just updating the existing tests (e.g., in jballerina-unit-tests) should be enough.

As I shared in the screenshot above you can see the errors in the folders itself and in the right side of the screenshot you can see what the errors are?

Can you please share what exactly the errors are?
Are you trying to add tests? Just updating the existing tests (e.g., in jballerina-unit-tests) should be enough.

As I shared in the screenshot above you can see the errors in the folders itself and in the right side of the screenshot you can see what the errors are?

Can only see the red line and not an actual error? Is it just an unresolved modue error?
Anyway, are you seeing this after following the guide for building from source?

https://github.com/ballerina-platform/ballerina-lang/blob/master/CONTRIBUTING.md
https://github.com/ballerina-platform/ballerina-distribution/blob/master/docs/build-ballerina-from-source.md#build-only-the-ballerina-runtime-with-the-tools

Okay let me try this

@Suraj-kumar00 Are you still working on this issue?

Hi @keizer619, actually I'm working on this but facing some errors I'll try to solve this by week and after that i'll let you know.

Hello @MaryamZi, how are you doing?
Pardon me for not be active here form few days but here I'm and working on it.
So I get this while fetching the updated from the main branch and I never changed anything here but it's showing untracked files i don't know why? can you please help me in this?

Hello @MaryamZi, how are you doing? Pardon me for not be active here form few days but here I'm and working on it. So I get this while fetching the updated from the main branch and I never changed anything here but it's showing untracked files i don't know why? can you please help me in this?

Is this after doing a build? You do a git clean to remove unnecessary untracked files too.

@Suraj-kumar00 Are you working on this issue?

Hi, @keizer619 Can I work on this issue ?

Hi, @keizer619 Can I work on this issue ?

Hello @Aviral0702
I have assigned the issue to your name. You can start working on this issue.
To help you get started, here are some resources that may help you get started with Ballerina:
Understanding Ballerina Platform:
Learn what Ballerina is all about: Ballerina Platform
How to Contribute:
Read our contributing guidelines to understand how you can contribute effectively: Contribution Guide
Learn Ballerina:
If you're new to Ballerina or want to enhance your skills, our official website offers a wealth of learning resources: Learn Ballerina
Get Help and Connect:
Join our Discord community to chat with fellow Ballerina enthusiasts and get assistance: Ballerina Discord
Feel free to ask questions, propose ideas, or report issues. Together, we can make Ballerina even better!
Happy coding! 🎉

Hi @gimantha @Aviral0702 @keizer619, apologies was working on this and sent a PR. If there are anymore changes, do let me know and maybe we can work on it together.

@Aviral0702 Have you already started working on this issue?

Hi @gimantha @Aviral0702 @keizer619, apologies was working on this and sent a PR. If there are anymore changes, do let me know and maybe we can work on it together.

@areebniyas Please make sure that in future, you pick  issues which are not assigned to anyone as we cannot split the number of credits between multiple assignees. Let's keep your pull request open for the moment. Thank you for understanding!


Message.yml grammatical fixes

Made the not-synced message make a little more sense.
Made the request-reply message make more sense too.

Thanks for the fixes! Appreciate it.


fix: avoid breaking the sourcemap

As the current state breaks the source map, we need to support source map as described in #3.
I'm not really familiar with this, but I think we can start by "not breaking them".
Obviously It's not perfect, but we can move on later on with supporting more.
What do you think @cometkim

i think the real solution should use https://github.com/Rich-Harris/magic-string 🤔


Latest Sitemaps and NotFoundHandler dont work together in .Net6

Related to https://github.com/Geta/geta-optimizely-sitemaps/issues/60 and https://github.com/Geta/geta-notfoundhandler/issues/52
If we try to use Geta.Optimizely.Sitemaps 3.0.0 and Geta.NotFoundHandler 5.0.0 in Net6 - it won't build
To reproduce (assuming Net6 installed):
dotnet new web
dotnet add package EPiServer.CMS
dotnet add package Geta.Optimizely.Sitemaps
dotnet add package Geta.NotFoundHandler.Admin
dotnet add package Geta.NotFoundHandler.Optimizely
dotnet build

error during build:
error : Two assets found targeting the same path with incompatible asset kinds:  [C:\work\temp\geta6\geta6.csproj]
error : '<..>\packages\geta.optimizely.sitemaps\3.0.0\contentFiles\any\net6.0\wwwroot\css\dashboard.css' with kind 'All' [C:\work\temp\geta6\geta6.csproj]
error : '<..>\packages\geta.notfoundhandler.admin\5.0.0\contentFiles\any\net6.0\wwwroot\css\dashboard.css' with kind 'All' [C:\work\temp\geta6\geta6.csproj]
error : for path 'css/dashboard.css' [C:\work\temp\geta6\geta6.csproj]

@lanorkin How does your csproj look like?

@lanorkin I could reproduce it. You should not install both - Geta.NotFoundHandler.Admin and Geta.NotFoundHandler.Optimizely. Install just Geta.NotFoundHandler.Optimizely which will include Geta.NotFoundHandler.Admin as a dependency.
I will check how to fix this. But it seems related to this: https://github.com/dotnet/aspnetcore/issues/39606 and it will be fixed only in .NET 7.

Thanks @marisks by doing this I cannot control specific lib versions; but seems it is working as a work around, so closing it.


Adding environment variable to systemd unit file

Modified parliament systemd file to load environment variables and asded $OPTIONS to command line.
This will allow users to specify startup options, like --dashboardOnly, without having to change the systemd unit file

Fixed the name on parliament service description (was wise)

Is it standard to have 2 EnvironmentFile lines?  Should we just do all the systemd files at once with this change?

For the 2 env. files, I can't say if it's standard but I've seen it in many other places.
It covers both rpm and debian based systems.
And for other systemd unit files, I can submit the changes for them too if you want.


[WIP] update controller-gen to v0.6.2

https://github.com/skai-x/elastic-jupyter-operator/issues/22

@mkkb473 @gaocegege Is it ready for merging to master branch?


ameba and spectator should be dev deps

Instead of being dependencies, ameba and spectator should be deps used just for the dev of this shard

Nevermind they are I was looking at the wrong logs


ng add @angular/bazel: build fail on peer dependencies

🐞 Bug report
Command (mark with an x)

- [ ] new
- [x] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?
No.
Description

Make sure you have a dependency that has a transitive peer dependency, like bootstrap, whose popper.js is a peer dependency of.
Enable bazel with ng add @angular/bazel
Run ng build

🔥 Exception or Error

$ ng build
...
INFO: Call stack for the definition of repository 'npm' which is a npm_install (rule definition at C:/users/medeb8/_bazel_medeb8/ccghtaa2/external/build_bazel_rules_nodejs/internal/npm_install/npm_install.bzl:294:15):
 - C:/users/medeb8/_bazel_medeb8/ccghtaa2/external/build_bazel_rules_nodejs/defs.bzl:77:5
 - C:/users/medeb8/git/Project/solutions/Company/Company.Project.client/WORKSPACE:64:1
ERROR: An error occurred during the fetch of repository 'npm':
   Traceback (most recent call last):
        File "C:/users/medeb8/_bazel_medeb8/ccghtaa2/external/build_bazel_rules_nodejs/internal/npm_install/npm_install.bzl", line 292
                _create_build_files(repository_ctx, "npm_install", node, r...)
        File "C:/users/medeb8/_bazel_medeb8/ccghtaa2/external/build_bazel_rules_nodejs/internal/npm_install/npm_install.bzl", line 146, in _create_build_files
                fail(("generate_build_file.js failed:...)))
generate_build_file.js failed:
STDOUT:

STDERR:
Could not find peer dependency 'popper.js' of 'bootstrap'
ERROR: error loading package '': Encountered error while reading extension file 'install_bazel_dependencies.bzl': no such package '@npm//': Traceback (most recent call last):
        File "C:/users/medeb8/_bazel_medeb8/ccghtaa2/external/build_bazel_rules_nodejs/internal/npm_install/npm_install.bzl", line 292
                _create_build_files(repository_ctx, "npm_install", node, r...)
        File "C:/users/medeb8/_bazel_medeb8/ccghtaa2/external/build_bazel_rules_nodejs/internal/npm_install/npm_install.bzl", line 146, in _create_build_files
                fail(("generate_build_file.js failed:...)))
generate_build_file.js failed:
STDOUT:

STDERR:
Could not find peer dependency 'popper.js' of 'bootstrap'
ERROR: error loading package '': Encountered error while reading extension file 'install_bazel_dependencies.bzl': no such package '@npm//': Traceback (most recent call last):
        File "C:/users/medeb8/_bazel_medeb8/ccghtaa2/external/build_bazel_rules_nodejs/internal/npm_install/npm_install.bzl", line 292
                _create_build_files(repository_ctx, "npm_install", node, r...)
        File "C:/users/medeb8/_bazel_medeb8/ccghtaa2/external/build_bazel_rules_nodejs/internal/npm_install/npm_install.bzl", line 146, in _create_build_files
                fail(("generate_build_file.js failed:...)))
generate_build_file.js failed:
STDOUT:

STDERR:
Could not find peer dependency 'popper.js' of 'bootstrap'
INFO: Elapsed time: 465.835s
INFO: 0 processes.

Installing the peer dependency seems to "solve" the problem and the build passes this point, build standard ng build doesn't require that, so I thought it would be worth reporting.

peerDepedencies should always be installed, that being said on standard npm install or yarn install these are warnings instead of errors.
I am not sure if these should be handled the same under Bazel.
//cc @alexeagle & @gregmagolan


Modularize using Browserify and Modernize with Babel

Hey, taking your comments from https://github.com/glowbox/maptasticjs/pull/4, maybe something like this would work? It would still allow people to use the single minified file if they want, but also will allow people using this in larger applications to use it more easily with a packager.
Mainly adds a browserify and babel grunt task, as well as moving from build to dist. Still a work in progress but wanted to see if this was an acceptable answer to your comments. No worries if not, I'll use my fork for my weird needs!

I'm going to close this, as it doesn't seem like there's much interest, but I have made a few good improvements on my fork if anyone becomes interested. Main changes so far: ES6 codebase, bundled + published to NPM with Rollup as UMD package, dropped the dependency on numeric.


Parsing error for configuration provided by kubernetes

Do you want to request a feature or report a bug?
Bug
What did you do?
Configured Traefik as ingress controller for kubernetes.
Enabled forward authentication with annotations in ingress objects.
Ingresses point to external services:type: ExternalName.
    traefik.ingress.kubernetes.io/auth-type: forward
    traefik.ingress.kubernetes.io/auth-url: https://kubew00:6443
    traefik.ingress.kubernetes.io/auth-tls-secret: tls-auth
    traefik.ingress.kubernetes.io/rule-type: "PathPrefixStrip"

What did you expect to see?
Mutual TLS authentication between Traefik and remote backends. I'm using Traefik as API GW for multiple remote API servers. I do not want to pass client certificates (used for frontends) to the backends. I'm expecting Traefik to use certificates stored into kubernetes secrets to autenticate against remote API servers.
What did you see instead?
Traefik fails parsing the configuration above provided by kubernetes.
Also, it claims there are no secrets in the current namespace.
Output of traefik version: (What version of Traefik are you using?)
traefik_linux-amd<IP_ADDRESS>rc3

What is your environment & configuration (arguments, toml, provider, platform, ...)?
# (paste your configuration here)

If applicable, please paste the log output in DEBUG level (--logLevel=DEBUG switch)
time="2018-08-21T13:40:40+02:00" level=error msg="Failed to retrieve auth configuration for ingress kube-system/caldera: failed to load auth secret: secret \"kube-system\"/\"tls-auth\" does not exist"
time="2018-08-21T13:40:40+02:00" level=debug msg="Configuration received from provider kubernetes: {\"backends\":{\"/clusters/caldera/\":{\"loadBalancer\":{\"method\":\"wrr\"}}}}"
time="2018-08-21T13:40:40+02:00" level=debug msg="Received Kubernetes event kind *v1.Secret"
time="2018-08-21T13:40:40+02:00" level=error msg="Near line 51 (last key parsed 'frontends.clusters/caldera/.auth.forward.tls.cert'): strings cannot contain newlines"

Closed by #3804.


Add support for custom copy command

Closes #43

Up ?

Looks good, thank you for contributing (and sorry for the wait)


[dot11] use SetTruncated() for too short packet

Captured dot11 packets are sometimes broken and too short by nature. That caused slice-bounds-out-of-range runtime error very often, and triggered builtin recovery, which cost counted.

Thanks!


feat: node log command also download config.txt and settings.txt

Description
This pull request changes the following:

Minor change to also download config.txt and settings.txt
Update setting.txt to align with latest on hedera repo
update how to use name space in calling getNodeLogs()

Related Issues

Closes #336

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Heroku blocked the app

Isn't there any way to use the app without heroku knowing it .

No way. This is their infra, they know everything.


Dash uber rides demo

Issue for app: #13
App pull request

[ ] This is a new app
[x] I am improving an existing app (redesigns/code "makeovers")

About

Playground deployment URL (new version): https://dash-playground.plotly.host/dash-uber-rides-demo/
Current gallery app URL: https://dash-gallery.plotly.host/dash-uber-rides/

Workflow

[x] I have created a branch in the appropriate monorepo, and the
elements necessary for successful deployment are in place.
[x] If the app is a redesigned and/or restyled version of an
existing gallery app, I've summarized the changes requested in the
appropriate Streambed issue and confirm that they have been applied.
[x] If the app is on the Dash Gallery portal, I have added a link to
the GitHub repository for the source code in the portal description.
[ ] If the app is a reimplementation of a Python gallery app for the
DashR gallery, the app in this PR mimics, as closely as possible,
the style and functionality of the existing app.

The pre-review review
I have addressed all of the following questions:

[x] Does everything in my code serve some purpose? (I have removed
any dead and/or irrelevant code.)
[x] Does everything in my code have a clear purpose? (My code is
readable and, where it isn't, it has been commented appropriately.)]
[x] Am I reinventing the wheel? (I have used appropriate packages to
lessen the volume of code that needs to be maintained.)

Please make the text a little bigger, and have it align with the dropdowns:

The undo/redo buttons also need some work:

💃

@ronniebugia Some small design changes:

Please fix the margins at the top and the left to match the ones in the mockup:

You can also change the width of the border for the dropdowns to 1px
Please remove the margins on the map and increase the margins on the dropdowns/logo/etc. in mobile view, like in the mockup:

If it's not too difficult, also please try to put the two dropdowns next to one another! (If this ends up taking a long time, don't worry about it.)
In the meantime I'll take a look at the playground deployment failure. Please re-request a review when these changes have been implemented!

Looks great! Thank you for fixing all of those :)
Just one more thing -- can you also set a max-height on the rightmost dropdown (the one with all the times)? I did something similar for Dash Bio clustergram here: https://github.com/plotly/dash-bio/blob/bfb9d23c4b5fea14ced7c5f809d8410a8220b223/assets/clustergram-style.css#L126

I would maybe make the max height approximately a couple of rows tall. I also can't really see the scrollbar. Once those are fixed then it should be good to go! :)

In the mobile layout, might be better to put these one on top of another, since the heights won't always match:

The dropdown options for the first dropdown are also sometimes too long and they get clipped by the narrower width of the mobile layout's dropdown.

💃 good job!


Italic failed when followed by Chinese punctuation.

https://seleniumhq.github.io/docs/site/zh-cn/

The source code is:
Selenium 的核心是 _[WebDriver]({{< ref "/webdriver/_index.md" >}})_，这是一个编写指令集的接口，可以在许多浏览器中互换运行。
这里有一个最简单的说明：

compare to en:
At the core of Selenium is _[WebDriver]({{< ref "/webdriver/_index.md" >}})_,
an interface to write instruction sets that can be run interchangeably in many
browsers. Here is one of the simplest instructions you can make:

I think the issue is caused by ，.

Thank you for reporting this @imba-tjd, this will be fixed by #339