🐛 BUG: "r2" added incorrectly to help messages when using invalid r2 subcommands

What version of Wrangler are you using?
0.0.27
What operating system are you using?
WSL (ubuntu 20.04, ext4 filesystem)
Describe the Bug
npx wrangler@beta r2 asdf outputs the following help message:
Commands:
  wrangler r2 init [name]       📥 Create a wrangler.toml configuration file
  wrangler r2 dev [script]      👂 Start a local server for developing your worker
  wrangler r2 publish [script]  🆙 Publish your Worker to Cloudflare.
  wrangler r2 tail [name]       🦚 Starts a log tailing session for a published Worker.
  wrangler r2 secret            🤫 Generate a secret that can be referenced in the worker script
  wrangler r2 kv:namespace      🗂  Interact with your Workers KV Namespaces
  wrangler r2 kv:key            🔑 Individually manage Workers KV key-value pairs
  wrangler r2 kv:bulk           💪 Interact with multiple Workers KV key-value pairs at once
  wrangler r2 pages             ⚡ Configure Cloudflare Pages
  wrangler r2 r2                📦 Interact with an R2 store
  wrangler r2 login             🔓 Login to Cloudflare
  wrangler r2 logout            🚪 Logout from Cloudflare
  wrangler r2 whoami            🕵  Retrieve your user info and test your auth config

Flags:
  -c, --config      Path to .toml configuration file  [string]
  -h, --help        Show help  [boolean]
  -v, --version     Show version number  [boolean]
      --legacy-env  Use legacy environments  [boolean]

Unknown argument: asdf

"asdf" can be replaced with any invalid subcommand of "r2".
"r2" should not be added as a prefix for all of these commands in the help message

lol, great catch.

oh that's interesting

Found the section of [code] (https://github.com/cloudflare/wrangler2/blob/main/packages/wrangler/src/index.tsx#L2415-L2419) responsible since it is a unknown argument it gets caught at in a top-level .fail() yargs function that rethrows the error. Likely the solution should be creating a .fail() helper function similar to the .command(subHelp) that can be used with all the commands which will have the context of the subCommand makeCLI.argsv which is lost at the top level


Docker: Error: Unable to connect to Jellystat Backend

I get this error with a fresh install. This is what I see in the logs:
[1] [HPM] Error occurred while proxying request <IP_ADDRESS>:3000/auth/isConfigured to http://<IP_ADDRESS>:3003/ [ECONNREFUSED] (https://nodejs.org/api/errors.html#errors_common_system_errors)
[1] [HPM] Error occurred while proxying request <IP_ADDRESS>:3000/auth/isConfigured to http://<IP_ADDRESS>:3003/ [ECONNREFUSED] (https://nodejs.org/api/errors.html#errors_common_system_errors)
[1] [HPM] Error occurred while proxying request <IP_ADDRESS>:3000/api/getconfig to http://<IP_ADDRESS>:3003/ [ECONNREFUSED] (https://nodejs.org/api/errors.html#errors_common_system_errors)
[1] [HPM] Error occurred while proxying request <IP_ADDRESS>:3000/api/getconfig to http://<IP_ADDRESS>:3003/ [ECONNREFUSED] (https://nodejs.org/api/errors.html#errors_common_system_errors)
Here is the compose file I am using:
version: '3.3'
services:
jellystat-db:
container_name: jellystat-db
image: postgres
environment:
- PUID=$PUID
- PGID=$PGID
- TZ=$TZ
- POSTGRES_USER=$POSTGRESUSER
- POSTGRES_PASSWORD=$POSTGRESPWD
volumes:
- $DOCKER_PATH/jellystat/postgres-data:/var/lib/postgresql/data # Mounting the volume
jellystat:
container_name: jellystat
image: cyfershepard/jellystat:latest
environment:
- PUID=$PUID
- PGID=$PGID
- TZ=$TZ
- POSTGRES_USER=$POSTGRESUSER
- POSTGRES_PASSWORD=$POSTGRESPWD
- POSTGRES_IP=jellystat-db
- POSTGRES_PORT=5432
- JWT_SECRET=$JWTSECRET
ports:
- "3000:3000" #Server Port
- "3004:3004" #websocket port
volumes:
- $DOCKER_PATH/jellystat/backup-data:/app/backend/backup-data # Mounting the volume
depends_on:
- jellystat-db
networks:
default:

After some tinkering with the compose file it looks like it is working now

After some tinkering with the compose file it looks like it is working now

Can you share what changes you made? I have the same problem. Thanks.

Solved also for me. For anyone interested, I set the port with 3003:3000 and inserted the external ip for the database instead of the name of the container.


Escape special chars in JS (like quotes)

Hi,
I think it could be great to escape specials chars of $fullText when including JS global CRAFT_ENVIRONMENT_LABELin EnvironmentLabelService.php.
I had an JS error which affect backend of CraftCMS after writing something like (in french) :

Vous n'êtes pas sur le site de production

craft()->templates->includeJs("window.CRAFT_ENVIRONMENT_LABEL = '" . $fullText . "';");
Kind regards,
Denis

@justdoweb-be thanks for reporting this.
Yay character encoding :)
I'm actually planning on removing the JS global in the next major release. Is it something you're actively using or just that it happens to throw an error?

No, I don’t use the JS global. It just throw an blocking error.

On 10 Feb 2017, at 11:54, Tom Davies<EMAIL_ADDRESS>wrote:
@justdoweb-be https://github.com/justdoweb-be thanks for reporting this.
Yay character encoding :)
I'm actually planning on removing the JS global in the next major release https://github.com/madebykind/craft.labelenvironment/pull/6. Is it something you're actively using or just that it happens to throw an error?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/madebykind/craft.labelenvironment/issues/7#issuecomment-278915063, or mute the thread https://github.com/notifications/unsubscribe-auth/AA1pH1jYFIv2KM-7CLiA5QebY0Wu6Ojxks5rbEHqgaJpZM4Lwip5.

@justdoweb-be should be fixed in the 2.1.1 release I just pushed - just wrapped the label string in `json_encode.


Rule "no-unused-variable" does not work anymore

Bug Report

TSLint version: 5.0.0
TypeScript version: 2.2.1
Running TSLint via:  CLI / VSCode

TypeScript code being linted
import * as Net from 'net';

let foo;

with tslint.json configuration:
{
  "rules": {
    "no-unused-variable": [
      true
    ]
  }
}

Actual behavior
No error reported
Expected behavior
Report errors

Ah,,, I see. It now requires --type-check. I didn't turn it on because of another bug.

Oh wait, I think a warning is still missing for that if it's not a bug?

This was one of the breaking changes in 5.0.0. What kind of warning are you looking for? Enabled rules that require the type checker but are not invoked with --type-check do produce a warning right now.

@adidahiya Ah sorry, I just missed that.


Remove-SPOWebPart Documentation

Hi
The documentation for Remove-SPOWebPart is a little confusing there are 15 pairs of "Syntax" and "Parameters".  I suspect there should only be one.
Here are some examples of the syntax for the MD file :-1:
Remove-SPOWebPart -Title "Table of Contents" -ServerRelativePageUrl /Pages/Chickpeas.aspx
Remove-SPOWebPart -Identity "8a23981f-5d27-4904-b199-45a582101801" -ServerRelativePageUrl /Pages/Pulses.aspx
Regards
Nigel

Hi @NPrice99  - Looks like this was caused by the latest Commit for the Oct Release
@jansenbe @erwinvanhunen - this is happened by the looks of things on almost all the cmdlet helps from that commit

Thanks for bringing this to my attention. I regenerated the documentation now with pr #89


Add a Makefile to build llbuild using CMake

This provides a convenient way to build and test llbuild using CMake.
Since it is possible to build llbuild using multiple buildsystems, we
can extend the Makefile as needed.

(put another way, this feels very specific to a particular workflow... extending it to support other workflows feels like it would be tantamount to just reinventing the commands themselves). Is there a reason why this needs to be part of the project versus just a script of your own?

The idea was to provide a convenient way for developers to build and test llbuild on desktops, similar to SwiftPM's Utilities/bootstrap script or Swift's utils/build-script. I don't have strong arguments for keeping it in llbuild (other than convenience).


Initialize a custom formly select type field using service

Hello,
I 've created a custom select type component using an api to fill the dropdown list.
There is a stackblitz example for this.
Stackblitz
The problem is that I want to initialize the select field  with a value from another api service  (like setting a default value).In fact,  I m using this code with groups where there is no constructor to call a service so I cant solve it. Any help?
https://stackblitz.com/edit/ngx-formly-demo?file=src%2Fapp%2Fforms%2Fgroups%2Faddress.group.ts

I ve changed it a bit. Can you please check out this stackblitz example? I can't set an initial value for the select type component
https://stackblitz.com/edit/angular-formly-fed-select-dgdkgh?file=app/app.component.ts

you don't need to create a custom type for such case, options allows passing an observable value see https://stackblitz.com/edit/angular-yz51jm

I am using different groups to define fields. like this example https://stackblitz.com/edit/ngx-formly-demo?file=src%2Fapp%2Fforms%2Fgroups%2Faddress.group.ts  How could I pass the Observable if there isn't any constuctor? I posted it also at stackoverflow and they suggest a solution with  compareWith https://stackoverflow.com/questions/58602212/how-to-set-initial-value-to-select-formly-field-in-angular/58602584#58602584.

pass the defaultOptions throughout group component where you can inject services: https://stackblitz.com/edit/ngx-formly-demo-ncfxap


WIP: #190 JSON export

Now there are 2 buttons over each Content field of the Messages table.

If you hover over any of them there will be a popup hint which tells the user what this button does:

The first button will copy the Content to the clipboard and the second will trigger a file download (.json if the content is an object or JSON.parse doesn't throw an exception; .txt otherwise.)

I have no idea how to test the file download yet. Still working on it.

Okay, after some research I've got an impression that the file-downloading part of the functionality is pretty much untestable. I didn't find a way to trigger the file download inside of the testing environment.
It is possible to dig into the specific implementation and test it, but this seems wrong to me. In this way, we treat the component as a white box, exposing all of its guts, tying our tests to the specific implementation. Looks like a violation of a bunch of principles and good practises.
@workshur @whotake what do you think?

@GneyHabub Does it throw any errors when are you trying to test it?
I think we can split view and download logic and test them separately:

Test view that it calls saveFile with proper arguments
Test saveFile itself. And there a few options on how to do it:
2.1 Split function into modules and test important logic. We care about file name and content more than pretty much standard downloading function.
2.2 Mock document and check setAttribute to content
2.3 Combine two methods above


New rule addition for vodaphonedsl.it

/^net(-[12]?[0-9]{1,2}){4}.cust.vodafonedsl.it$/     REJECT  Generic - Please relay via ISP (vodaphonedsl.it)

Added to develop branch of main file for testing.


There is no search history

Can't easily redo previous search, which is very annoying and decreases productivity.

VSCode Version: 1.0.0
OS Version: OS X 10.11.4

Steps to Reproduce:

Press Shift+Cmd+F
Search for something
Search for another thing
Press UP key (or Cmd+Z or whatever)

Expected: Show previous search term, like all other editors
Result: Nothing happens
VS Code:
![VS Code Search Example]
(https://s31.postimg.org/6th4ds8hn/search-code.gif)
Atom:
![Atom Search Example]
(https://s31.postimg.org/4q6p648or/search-atom.gif)

@kieferrm heard you are taking over search :-)

I really liked this feature in Atom and would use it regularly.

I would love to give this feature a shot. Any chance you'd accept a contribution on this one?

@Urik I have already started on this and planning to push it today.. Sorry for that.

That's so great to hear Sandy, I've been lusting over that feature since I started working with VSCode!
I'd love to see these possible features, thanks.

Implemented following support:
Search input in Search view
Find input in Find widget

alt+up : Shows the previous term in history
alt+down : Shows the next term in history

Customize key board short cuts

search.history.showPrevious : alt+up
search.history.showNext : alt+down
find.history.showPrevious : alt+up
find.history.showNext : alt+down

this work well but i feel its kinda hidden behind the alt+. is there any way to have it just be up/down on the search fields?

@hipstersmoothie You can customize the key board short cuts overriding the commands and keys mentioned in comment

Thats great, still, the defaults should be intuitive and natural. Not sure that the use of ALT satisfies that. Is there any downside to not using ALT? These are single line queries (absent the use of \n), anyway, so I would think up and down do not have a meaning in these contexts.

@gandhis1 Up and Down are already used to navigate between the input boxes (Search, Replace, Files include, Files exclude). Using Alt goes with other key board short cuts for actions - Toggle match case, reg ex, whole word.
I am open for the suggestions to change the default one to more better ones.
Thanks for the feedback

@sandy081 navigation between fields is actually expected by tab, shift+tab. So I don't see any reason to not just use arrows for search/replace history in those fields.

Thanks for the feedback. Filed the following issue to change it
https://github.com/Microsoft/vscode/issues/12645


JSON - Router

You can put the router in json and make a structure that can be used, it will be more organized for the user.

Example:
[
{
method: "get",
url: "/xxx",
function: "xxx"
}
]

Can we separate the routers?
Sample:
./router/admin.js
./router/account.js


Completely broke after beta4 upgrade

I had the following code and it was working just fine before the latest upgrade.
<%= turbo_frame_tag "videos__frame", src: videos_path %>

It does not dispatch the request with the proper Accept header as you can see on the following image. What's even more strange is that the header Turbo-Frame was sent.

https://github.com/hotwired/turbo/pull/146 clarifies everything.


AppVeyor fix x64 bit build(s)

https://ci.appveyor.com/project/ovr/owl/build/216-5.6.8/job/4019l6jy2mskt3kb
https://ci.appveyor.com/project/ovr/owl/build/216-5.6.8

@steffengy
Can you help with it?
Thanks

Are you sure you want to use x64 builds for php 5.X?
"x86_64 Builds
The x64 builds of PHP for Windows should be considered experimental, and do not yet provide 64-bit integer or large file support. Please see this post for work ongoing to improve these builds."
http://windows.php.net/
http://marc.info/?l=php-internals&m=137002754604365&w=2

@ovr
It could work if you replace '"%VS110COMNTOOLS%\VsDevCmd" %platform%'
with '"C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcvarsall" %platform%'


No matching audio stream!

I get the error: "No matching audio stream!" when trying to save with the preset "mp3" this video https://www.youtube.com/watch?v=RKpjqddcuy0
Any idea why?

Yes it works, it just few video where I get this issue. Maybe 10% of all video I try to process, I can give you other example if you want

I think I need to add some more debugging functionality to really see what the issue might be. I would think that there is something going on with those videos, not necessarily this library. But I could be wrong! Under the covers I use ytdl-core, which does the actual parsing of various media types and downloading. I will try to take a look at this sometime this week.
(In the meantime if mp3 versions are critical try downloading the whole video and then use another program to extract just the audio (ffmpeg would be perfect)

Ok keep me updated and I will try the full video + ffmpeg process

Same problem here. I tried about a dozen videos from https://www.youtube.com/channel/UCEGJQkfFh4wk2HYtPLInnPA/videos and had that error with each of them :(

Hmm, there should be a common ground when it comes to the raw data coming back. We can try adding more logging when it fails and see if we can determine anything.

Failed for this video too:
https://www.youtube.com/watch?v=VfnYmGSE1aY
Why does this happen?

I was having a similar issue, but with video streams. All video formats were being filtered out because the format.type was undefined. This issue was related to the raw field changing from type to mimeType. This was addressed in https://github.com/jimbuck/pully-core/commit/1b2d4275c564235b18d356815f82d6facc068b64 when https://github.com/jimbuck/pully-core/blame/master/src/index.ts#L42 was changed. Updating to the latest pully-core resolved the issue for me. I had to do it the dirty way by reaching into node_modules/pully and installing the latest version manually. It looks like pully-core will be upgraded in the next release due to this commit https://github.com/jimbuck/pully/commit/c491a50ebc019edb5395bc8fab10f3fa999fed60

@nakleiderer You are absolutely correct. A few things changed in the underlying APIs so I am working through those now. Hopefully I'll get pully updated and released soon.

I found out that in analyzer.ts (or \node_modules\pully\dist\lib\analyzer.js if you installed from npm), in the _createVideoFilter and _createAudioFilter functions, you need to change every format.type to format.raw.mimeType.


fix: [FTL-8773] add COR-32 error

Please, update changelogs and versions for common-libs too

Please, update changelogs and versions for common-libs too

done


Add ability to see links between devices (controller -> responder,  controller <-> responder)

Create a visual representation of the controller/responder relationship between two devices. Allow this relationship to be editable similar to how a scene is editable.

Visual representation of links would be a very useful feature.


Dependent hardware daemons

Some hardware, notably OPAs and the (in progress) SDC delay cause underlying daemons to move position.
This issue (following a phone conversation with @untzag) summarizes our strategy for dealing with this:

Create a yaq trait has-dependent-daemons, which introduces and standardizes a yaq message get_dependent_yaq_params (names flexible), which returns a host:port string for each dependent yaq daemon.
yaqc_bluesky will surface this info (as yet details of implementation are subject to change.)

Ideally the name of the data key does not change between scans which directly interface with the underlying daemon and scans which interface via the controlling daemon... a more ophyd like parent/child approach may involve namespacing these data keys, a more naive approach may involve double defining keys, which would be less than ideal.)
Alternative to in yaqc_bluesky would be to do so at the plan level, which may solve the namespacing problems, but may be less transparent/harder to know what is going on.

This means that with minimal yaq implementation (adding a trait and defining a single method that returns info already available to the daemon) and no additional human written configuration (or happi for that matter) we can describe these relationships and keep data files relatively minimal (only record data at each scan point for motors that move, even if bluesky doesn't know they are moving itself), without missing out on valuable info that may not be what was "intended" to be the useful axes.
Potential implementation:

perhaps in a stage step communicate with the bluesky objects, which will cause them to be in the plan, I think? have to look at this a bit.

https://github.com/bluesky/ophyd/blob/dd4b3e389a0202ecacce39fc3965d703c616b0d4/ophyd/device.py#L1233
For reference, Ophyd handles this via "Components" which are just tacked on to the read/read_configuration (and describe counterparts) transparently.
I don't think it should be a real problem to double define (they should all be done moving anyway... and this really only makes sense for motors) should be pretty easy, all things considered, though may wish to build in caching to Device (perhaps as simple as @lru_cache on Device) to avoid having many copies of identical clients floating around... though that could lead to problems when you want to regenerate the client...

Closed by #72


View Schedule According to Speakers

[x]  Make speakers.html available in Zip.
[x]  Add anchor on top of page for Speaker's page
[x]  Write Markup for Speaker page
[x]  Write SASS for Speaker page

@mariobehling  @championswimmer  All solved here #372

Done :+1:


JavaScriptExecutor doesn't seem to have an 'executeScript' feature

I'm trying to access my own banking details (not other peoples!) by clicking on the 'Continue' button of:
https://onlinebanking.nationwide.co.uk/AccessManagement/IdentifyCustomer/IdentifyCustomer
but without success. I have tried a normal 'click()' on the button, but nothing happens no matter how long I wait.
I have already successfully used Selenium in a .Net program to overcome this problem, and whilst the Selenium version (in C#) also seems to ignore a 'click()', when I use:
JavaScriptExecutor.ExecuteScript("arguments[0].click();", targetButton);
it does have the required effect, i.e. the button 'clicks' and I transfer to the next html page so all is good.
Using a similar approach with 'htmlunit-android' however, there doesn't seem to be an extension 'executeScript' that I can use with 'JavaScriptExcutor'.  Have I missed something?  Is it there under another guise??
Here's the Kotlin snippet (sorry its not Java, but I have to use Kotlin because Java doesn't support co-routines):
import org.htmlunit.javascript.background.JavaScriptExecutor
import org.htmlunit.WebClient
import org.htmlunit.html.HtmlButton
import org.htmlunit.html.HtmlPage

private lateinit var mainScreen : HtmlPage

var webClient = WebClient()     // Create the webClient
var myURL = "https://onlinebanking.nationwide.co.uk/AccessManagement/IdentifyCustomer/IdentifyCustomer"

var mainPage = webClient.getPage(myURL) as HtmlPage        // Retrieve the target page

var jsexecutor: JavaScriptExecutor = webClient as JavaScriptExecutor    // Declare a JS executor

var buttonsList = mainPage.getElementsByTagName("button") as List<HtmlButton> // Find all buttons
for (targetButton in buttonsList )   // Loop round the buttons
{
        if (targetButton.textContent == "Continue") // This is the one I'm looking for
        {
                jsexecutor.executeScript("arguments[0].click();", targetButton) // <= 'executeScript'!! isn't there!
         }
}

Thanks in advance for any assistance

Had a first look - opening the page generates a bunch of js errors. Will have a look.

Yes. Its full of errors, I agree. But the Selenium version manages to overcome them (somehow). Its the 'executeScript' feature that is stopping me. But thanks for looking, much appreciated

the first bunch of (simple fixes) are done - now i have to have a look at the tricky ones :-)

step one done, i can load the start page without getting js errors, next step is filling the form, clicking and get the error msg

I am waiting with bated breath!  =:-]

Found the next problem, a bunch of objects are not working when used from a worker. This will require some time because i like to write a bunch of tests for this.
So please continue breathing for a while


Publish to a public repository

If other projects (like IJava) are going to depend on this repository then this really needs to be published somewhere to allow for dependencies on specific versions and avoid the need to compile this project before use.

A request was submitted to sonatype for open source repository hosting. See the ticket OSSRH-37889


Mentioning Checker Framework's license

Issue #, if available:
Description of changes:
Suggested text to highlight that Checker Framework's license is GPL w/ CPE.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This required a little rebase.  The rebase was too tricky to do in the github ui, as the lines are so long.  So I added your text in a different commit (5ce0a6a)


Review current impl and tickets to ensure they are update to date with latest TUF spec

Significant changes were implemented in this PR along with added clarity.
https://github.com/theupdateframework/tuf/pull/440/files?w=1

This implementation is up to date with https://github.com/theupdateframework/tuf/blob/9025b90bac0ee9bcd3398b39f952ca0363372e67/docs/tuf-spec.txt, not including delegations.


[Hub Generated] Review request for Microsoft.MachineLearning to add version 2017-01-01

If you are a MSFT employee you can view your work branch via this link.
Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python
Nothing to generate for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby
Nothing to generate for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Can one of the admins verify this patch?

Automation for azure-sdk-for-node
Nothing to generate for azure-sdk-for-node

Automation for azure-sdk-for-go
Nothing to generate for azure-sdk-for-go

@ravbhatnagar,  @xinzouMicrosoft  sent suppression approval request, please see email with subject "Violations Suppression requested for PR (Provide PR #)", could you take a look?

@ravbhatnagarhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fravbhatnagar&data=02|01|xinzou%40microsoft.com|b2707459caf246979a5008d69ccb2cfd|72f988bf86f141af91ab2d7cd011db47|1|0|636868793787850914&sdata=xYMBtgAgve7yYxS%2Bp4QcLsB7iLt7gUJdxEjN%2BPYjqFc%3D&reserved=0,could you take a look at my PR when you have one chance? I submitted it one week ago.
Thanks in advance!
Regards,
Xin
From: Anu Thomas Chandy<EMAIL_ADDRESS>Sent: 2019年2月27日 23:50
To: Azure/azure-rest-api-specs<EMAIL_ADDRESS>Cc: Xin Zou (AI PLATFORM<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-rest-api-specs] [Hub Generated] Review request for Microsoft.MachineLearning to add version 2017-01-01 (#5241)
@ravbhatnagarhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fravbhatnagar&data=02|01|xinzou%40microsoft.com|b2707459caf246979a5008d69ccb2cfd|72f988bf86f141af91ab2d7cd011db47|1|0|636868793787850914&sdata=xYMBtgAgve7yYxS%2Bp4QcLsB7iLt7gUJdxEjN%2BPYjqFc%3D&reserved=0, @xinzouMicrosofthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FxinzouMicrosoft&data=02|01|xinzou%40microsoft.com|b2707459caf246979a5008d69ccb2cfd|72f988bf86f141af91ab2d7cd011db47|1|0|636868793787850914&sdata=ccqPjwdfOhF%2BQzZaYFS5Ap%2FOWYkrzT4EPMI%2B8bfhoSY%3D&reserved=0 sent suppression approval request, please see email with subject "Violations Suppression requested for PR (Provide PR #)", could you take a look?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F5241%23issuecomment-467915216&data=02|01|xinzou%40microsoft.com|b2707459caf246979a5008d69ccb2cfd|72f988bf86f141af91ab2d7cd011db47|1|0|636868793787860907&sdata=yIlEf%2FEizuvmgDL6ZMDYmetQUSJp7KbRP1MYpZgGd%2Fw%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAthQUD_avZFuO1BlMBjne-Xl8pcwVpJQks5vRqkQgaJpZM4bGquw&data=02|01|xinzou%40microsoft.com|b2707459caf246979a5008d69ccb2cfd|72f988bf86f141af91ab2d7cd011db47|1|0|636868793787860907&sdata=wo%2BOzmCiCv7bhTyCzJCqTNmSIUHFmITCvqd28nJ2ezY%3D&reserved=0.

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/5467
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/5467/commits/72de60ddcaad458705af65fc59d4267aa740533f


Temporarily introduce an error in a system test to make it fail in Charges CD

Description
This PR sounds a bit wierd, but the idea is to test that the system test running after deployment to U-001 will guard further deployments if it fails.
So this PR makes a small correction to a system test, that will make it fail when running in Charges CD.
And will be followed by another PR that rectifies it again.
References

#951

Just got info on a better way to do this, so only one PR is needed. Closing this.


How can you map the common crawl source back to metadata?

The common crawl data entries have a source like this:
"source":"cc/2023-06/en_head_0000.json.gz/line401859"
What's the right way to map that back to metadata where the entry came from?  In particular I'd like the original url and timestamp it was downloaded.  Is that possible?  Most of the metadata seems to be in terms of the WARC format, not the WET format I believe was used by cc_net to process the data.
Thanks,
Craig

Hi Craig, this is a great point! I saw the discussion on discord -- we are working on a feature that allows to store more metadata for the commoncrawl slice (see issue #39 ).

To answer your question here for completeness: currently, we cannot map an individual record back to the original url / perplexity score etc., as this data is not contained in the current version of the dataset.


nixos.programs.neovim: fix runtime

Description of changes

Fix the handling of the runtime option so resulting files and directories end up where neovim will look for them.
When this module was first introduced, it processed the runtime option in a way that nested the resulting files and directories under an etc directory.
https://github.com/NixOS/nixpkgs/pull/98506/files#diff-685092dbb1852fbf30857fe3505d25dc471dc79d0f31c466523b5f5822b68127R11-R21
That implementation relied on nixos/modules/system/etc/make-etc.sh, a script that was later removed.
https://github.com/NixOS/nixpkgs/pull/131102/commits/eb7120dc79966d5ed168321fd213de38de13a2b1
The implementation was updated to use linkFarm, which changed the behavior slightly, in that the configured files and directories are no longer automatically nested under an etc directory. https://github.com/NixOS/nixpkgs/commit/307b1253a797f9ab9f5da38a470099945ea7ce04
But the module still configures neovim's runtimepath in a way that assumes the old nesting behavior.
https://github.com/NixOS/nixpkgs/blob/04f574a1c0fde90b51bf68198e2297ca4e7cccf4/nixos/modules/programs/neovim.nix#L173
Restore the original behavior, nesting runtime files and directories under an etc directory.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Sorry for intruducing this regression, I'm not a neovim user myself which is why I couldn't properly test it.
I hope this PR improves my mistakes ;)


wip: Canvas Pen Pressure

What type of PR is this? (check all applicable)

[x] Feature

Have you discussed this change with the InvokeAI team?

[x] Yes

Description

Adds Initial Pen Pressure Support To Canvas.

-- Lots left to do.

@blessedcoolant - Should we put this on the backburner (i.e., not on roadmap for next release) until we can figure out how to get Size to change on drag? It sounds like this involves a major rewrite of the input

Yeah. I haven't had the chance to work on it after that day with the whole SSD burning down issue. I'll get back on it soon and hope for a eureka moment that solves it. But remains a draft until then. Hopefully next release.


AKSettings.enableRouteChangeNotifications removed from macOS API

I've been using AudioKit 4.9.5 with AKSettings.enableRouteChangeNotifications set to true (which I realize is the default). Upon updating to 4.10.1, I got a compile-time error saying that AKSettings.enableRouteChangeNotifications does not exist.
Looking into the source code of AKSettings, it appears that this has been wrapped in an #if that only creates it for iOS.
I feel that this might be a bug, since route change handling is certainly not only an iOS concept. At any rate, I would like to be able to control route change handling on macOS, and I'm not sure what the expected behavior is without this setting.
Thank you!

I am under the impression that macOS does not do the same kind of route change notifications as iOS.

It looks like this is in beta for the upcoming macOS release, so I believe it is set up right to have it not on macOS as of yet: https://developer.apple.com/documentation/avfoundation/avaudiosession/1616493-routechangenotification


drivers: modem: bg9x: Prevent compiling unused configs

Currently the default Kconfig values from bg9x modem are being mistakenly compiled into the autoconf.h due to a missing if guard in the Kconfig.quectel-bg9x. This pr patches that.
Fixes #38535

Im getting BG9X related configs in my autoconf.h

@cfriedt @bwasim

@ycsin - can you please create an issue for this PR? The sooner the better, as the release team is triaging issues today.

@ycsin - can you please create an issue for this PR? The sooner the better, as the release team is triaging issues today.

Done, sorry for the delay!


Fix sensor multi poke issue release0.7

fix: xcom state logic

fix: sensor poke count based start event notification

closing this PR as we will not be releasing v0.8


Ads don't get blocked on myonlineradio.hu

List the website(s) you're having issues:
'https://myonlineradio.hu/'
What happens?
Advertise
List Subscriptions you're using:
Default uBO
Your settings
OS/version: Android 13
Browser/version: uBO Latest
Adblock Extension/version: 1.46.0
Other details:
Original issue: https://github.com/easylist/easylist/issues/14872

Hi @d4f5409d I don't see any ads with hufilter and easylist. Could you provide some screenshots of the problem, and your enabled filters?

Maybe my issue is outdated since it's been 5 months, but I'll try later

The only ad servers the site connects to are head-clickfusion.com (without hufilter, this is ehat causes the ads) and doubleclick.net, both are blocked by hufilter. Otherwise, you can turn adguard (ads) on in ubo, or add ||head-clickfusion.com^$third-party to your custom filters. These should fix the problem.
Do let us know if it fixes the problem.

Looks like my issue was really outdated, because it looks like now it's resolved. However, self sponsored ads still appear on the site. (https://myonlineradio.hu/public/img/efi/hu/img_300_600.gif and https://myonlineradio.hu/public/img/efi/hu/img_300_250.gif)

Thank you


Dockerize Application and Redis

Application and Redis needs Docker containers for more deterministic deployment

https://github.com/HackUCF/OnboardLite/pull/26


chore(ip 2.0.1): ip version upgrade

Details
ip version upgrade to 2.0.1 for Accessibility Insights for Action
Motivation
Upgrade the ip version to latest 2.0.1 in Accessibility Insights
for Action.
https://dev.azure.com/mseng/1ES/_componentGovernance/1010/alert/201464?typeId=286939
Context

Pull request checklist

[X] Addresses an existing issue: Fixes #0000
[X] Added relevant unit test for your changes. (yarn test)
[X] Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
[ ] Ran precheckin (yarn precheckin)

@rohittarpara shouldn't this have a change in package.json as well?

@rohittarpara shouldn't this have a change in package.json as well?

@madalynrose No, because In the package.json file, i could not find the ip package.

@rohittarpara if that's the case, it's being managed by a different package and this change will be overwritten by any yarn install calls. If we need a particular version for security, we should add a yarn resolution for it.

@madalynrose i have added "<EMAIL_ADDRESS>"^2.0.1"" in package.json file, can you please verify.


Webhook URL fails (too long)

My actions before raising this issue

[X] Read/searched the docs
[X] Searched past issues

Error: Request failed with status code 400. {"target_url":["Ensure this field has no more than 200 characters."]}.
Steps to Reproduce (for bugs)

Create a Teams webhook, the url is quite large... more than 200 chars

Possible Solution

Maybe is a DB field with this size??
Your Environment

cvat release v2.3.0

We don't have reasons for have limit in 200 characters, sent PRs with increasing this value


Fix action bar background not being moved

The action bar text is moved, but not the background. The fix is for renderActionBarStart to mixin at InGameHud.drawTextBackground rather than at TextRenderer.draw.

Thanks for the fix!


Taxonomies: Add History Support

Feature Description
A user should be able to undo/redo the addition and removal of taxonomy terms from a story. This will match the behavior of the Gutenberg editor.
After implementation: enable the should not have new history changes when switching tabs without changes test.
Alternatives Considered

Additional Context

@littlemilkstudio @samwhale thanks for making this ticket, can you add a point estimate to it?

According to https://github.com/google/web-stories-wp/pull/9048#discussion_r711614156, this ticket should fix the issue where opening the blank editor already creates a history entry. Is that correct? Because it.s quite the annoying regression, so should be fixed swiftly.

@swissspidy ahh, I'll do a quick fix for that issue that you posted above, then work on proper history integration after (which might be pretty tough with the existing setup)


svelte-migrate breaks on local variable called state

Describe the bug
Hi,
I have a Svelte 4 file containing a variable called state. When migrating to Svelte 5, this breaks as there is now the variable as well as the $state rune...
I think svelte-migrate should detect this and throw an error...
Reproduction
I don't know how to do repros for svelte-migrate
Logs
No response
System Info
System:
    OS: Linux 6.5 Ubuntu 22.04.5 LTS 22.04.5 LTS (Jammy Jellyfish)
    CPU: (12) x64 12th Gen Intel(R) Core(TM) i7-1265U
    Memory: 5.90 GB / 15.22 GB
    Container: Yes
    Shell: 5.8.1 - /usr/bin/zsh
  Binaries:
    Node: 22.11.0 - /usr/bin/node
    Yarn: 1.22.21 - /usr/bin/yarn
    npm: 10.9.0 - /usr/bin/npm
    pnpm: 8.15.8 - /usr/bin/pnpm
  npmPackages:
    @sveltejs/adapter-static: ^3.0.6 => 3.0.6
    @sveltejs/kit: ^2.8.0 => 2.8.0
    svelte: ^5.0.0 => 5.1.12
    vite: ^5.4.10 => 5.4.10

Severity
annoyance
Additional Information
Probably holds for other rune names.

You can still have a variable named state
https://svelte.dev/playground/419dd3d00d3741828923bfa8f50562c1?version=5.1.12
try to migrate this and see that it still works.
You can do this to reproduce for a migration error: paste the code in the playground and share the svelte 4 code (we can then click migrate to verify)

You can have a variable named state, but only if you don't use a state rune in the same file (which was the result of my migration):
https://svelte.dev/playground/419dd3d00d3741828923bfa8f50562c1?version=5.1.12#H4sIAAAAAAAAA22OwQ6CMAxAf2U2JmhiJF5xknjzH8TDgCpLxrZsBTWEf7cgiRcvO7z32nUAq1qEDFr9CIpQROIXdnDXBiNk1wHo7adgAsyX_Oz9PvZo5rZUEf_xyllCS7wGZKyC9pQXtiCD9P1GnETydMHUyZH5Yhw1GNis52SzZSXT37SV2vqORKltnfXKdHga5nIUKXvZHPILGuPEQlcyZVRYPofwRZBR6HC8jR-aQYyW-QAAAA==

Gotcha...will fix and by fix mean add a migration task in this case (this should probably be moved to the svelte repo)


[BUG] hyperf/json-rpc的依赖问题

/opt/www # uname -a && php -v && composer info | grep hyperf && php --ri swoole
Linux 0ad8ccb12661 4.9.184-linuxkit #1 SMP Tue Jul 2 22:58:16 UTC 2019 x86_64 Linux
PHP 7.4.3 (cli) (built: Mar 20 2020 07:14:36) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
hyperf/cache                         v1.1.30  A cache component for hyperf.
hyperf/command                       v1.1.26  Command for hyperf
hyperf/config                        v1.1.26  An independent component that provides configuration container.
hyperf/constants                     v1.1.26  A constants component for hyperf.
hyperf/contract                      v1.1.26  The contracts of Hyperf.
hyperf/database                      v1.1.30  A flexible database library.
hyperf/db-connection                 v1.1.26  A hyperf db connection handler for hyperf/database.
hyperf/devtool                       v1.1.30  A Devtool for Hyperf.
hyperf/di                            v1.1.28  A DI for Hyperf.
hyperf/dispatcher                    v1.1.26  A HTTP Server for Hyperf.
hyperf/event                         v1.1.26  an event manager that implements PSR-14.
hyperf/exception-handler             v1.1.26  Exception handler for hyperf
hyperf/framework                     v1.1.27  A coroutine framework that focuses on hyperspeed and flexible, specifically use for build microse...
hyperf/guzzle                        v1.1.26  Swoole coroutine handler for guzzle
hyperf/http-message                  v1.1.26  microservice framework base on swoole
hyperf/http-server                   v1.1.29  A HTTP Server for Hyperf.
hyperf/json-rpc                      v1.1.30  A JSON RPC component for Hyperf RPC Server or Client.
hyperf/load-balancer                 v1.1.26  A load balancer library for Hyperf.
hyperf/logger                        v1.1.26  A logger component for hyperf.
hyperf/memory                        v1.1.26  An independent component that use to operate and manage memory.
hyperf/model-cache                   v1.1.28  A model cache component for hyperf.
hyperf/model-listener                v1.1.26  A model listener for Hyperf.
hyperf/paginator                     v1.1.26  A paginator component for hyperf.
hyperf/pool                          v1.1.26  An independent universal connection pool component.
hyperf/process                       v1.1.26  A process component for hyperf.
hyperf/redis                         v1.1.26  A redis component for hyperf.
hyperf/rpc                           v1.1.26  A rpc basic library for Hyperf.
hyperf/rpc-client                    v1.1.28  An abstract rpc server component for Hyperf.
hyperf/server                        v1.1.26  A base server library for Hyperf.
hyperf/testing                       v1.1.28  Testing for hyperf
hyperf/translation                   v1.1.26  An independent translation component, forked by illuminate/translation.
hyperf/utils                         v1.1.27  A tools package that could help developer solved the problem quickly.
hyperf/validation                    v1.1.28  hyperf validation

swoole

Swoole => enabled
Author => Swoole Team<EMAIL_ADDRESS>Version => 4.5.0
Built => Apr 26 2020 07:19:39
coroutine => enabled
epoll => enabled
eventfd => enabled
signalfd => enabled
spinlock => enabled
rwlock => enabled
openssl => OpenSSL 1.1.1g  21 Apr 2020
http2 => enabled
pcre => enabled
zlib => 1.2.11
mutex_timedlock => enabled
pthread_barrier => enabled
mysqlnd => enabled
async_redis => enabled

Directive => Local Value => Master Value
swoole.enable_coroutine => On => On
swoole.enable_library => On => On
swoole.enable_preemptive_scheduler => Off => Off
swoole.display_errors => On => On
swoole.use_shortname => Off => Off
swoole.unixsock_buffer_size => 8388608 => 8388608

composer require hyperf/rpc-client

composer require hyperf/json-rpc

/opt/www # composer require hyperf/json-rpc
Using version ^1.1 for hyperf/json-rpc
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 1 install, 0 updates, 0 removals
  - Installing hyperf/json-rpc (v1.1.30): Loading from cache
hyperf/json-rpc suggests installing hyperf/rpc-server (Reqiured to use JSON RPC in hyperf rpc server.)
Writing lock file
Generating autoload files
composer/package-versions-deprecated: Generating version class...
composer/package-versions-deprecated: ...done generating version class
> init-proxy.sh
../../
Runtime cleared
Scanning app ...
Scan app completed, took<PHONE_NUMBER>4614 milliseconds.
Scanning vendor ...

Fatal error: Uncaught Error: Class 'Hyperf\RpcServer\Server' not found in /opt/www/vendor/hyperf/json-rpc/src/TcpServer.php:34
Stack trace:
#0 /opt/www/vendor/composer/ClassLoader.php(444): include()
#1 /opt/www/vendor/composer/ClassLoader.php(322): Composer\Autoload\includeFile()
#2 [internal function]: Composer\Autoload\ClassLoader->loadClass()
#3 [internal function]: spl_autoload_call()
...

Description:
我只想弄一个jsonrpc服务的客户端，所以安装了上面命令中的两个composer包(rpc-client & hyperf/json-rpc)
从上面的错误中，可以看出hyperf/json-rpc依赖了hyperf/rpc-server。
而实际按文档上来讲，rpc客户端只需要安装rpc-client和json-rpc就好了。
⚠当前先安装rpc-server解决
Steps To Reproduce:

在 2.0 中已经修正这个问题了


Added pi400 to supported devices

Change-type: patch
Signed-off-by: Phil Wilson<EMAIL_ADDRESS>

@balena-ci I self-certify!

@balena-ci I self-certify!


No service type definitions generated when using --with-typescript

Thank you for adding the --with-typescript flag in PR #125.
When using --with-typescript it seems no service type definitions are generated.
In the ts-protoc-gen docs it's mentioned to add service=true before the OUT_DIR, like so:
--ts_out="service=true:${OUT_DIR}"

See https://github.com/improbable-eng/ts-protoc-gen#generating-grpc-service-stubs-for-use-with-grpc-web for more info.
@esilkensen as you worked on this recently do you care to extend your current implementation with this service=true option?
Thanks in advance!

Sure, yeah I will take a look!

I wonder if it's worth making this configurable somehow?
The ts-protoc-gen README notes that

Both js and d.ts service files will be generated regardless of whether there are service definitions in the proto files.

So it generates empty _service.js and _service.d.ts files for each .proto file that doesn't have a service definition... (which makes me think, is there a reason those files are generated, or could ts-protoc-gen itself be updated to suppress those...)

I've used another lib in the past (https://github.com/agreatfool/grpc_tools_node_protoc_ts) and that one does only generate d.ts service files if there are service definitions found.
Not sure about ts-protoc-gen.

Oh, interesting -- and refreshing my memory -- so I think ts-protoc-gen with service=true generates JS and corresponding TypeScript declarations for grpc-web service implementations, but not for the normal service definition generated by protoc.
I wonder if ideally ts-protoc-gen could be updated to generate TypeScript declarations for protoc-generated service implementations regardless of whether service=true...

Oh, thanks - I think I understand more now. So it is the grpc_node_plugin that generates the gRPC JS, which is separate from the normal JS output.
It looks like https://github.com/agreatfool/grpc_tools_node_protoc_ts was forked as https://github.com/niklaus0823/protoc-gen-grpc-ts - maybe that could be leveraged to generate TypeScript declarations for gRPC services in this project instead of / in addition to ts-protoc-gen...?

I think we could switch to protoc-gen-grpc-ts - but there may be one issue that would be good to resolve first:
That project generates enum definitions in .d.ts files, which is the only difference as far as I can tell between what it generates and ts-protoc-gen (except for it also generating the gRPC service implementation types).
But enum definitions are special in TypeScript and have their own implementation and shouldn't go in a .d.ts file (https://lukasbehal.com/2017-05-22-enums-in-declaration-files/)
I think I'll look into opening a PR / Issue for discussion with protoc-gen-grpc-ts and then circle back here!

https://github.com/esilkensen/docker-protoc/tree/feature/node-grpc-ts-support - branch that would swap out the plugins, and potentially all it would need is a version bump if this enum issue is indeed a problem that could be resolved with a PR in that other repo.

Actually, looking through ts-protoc-gen's history it seems like there have been attempts to add grpc-node typings to that project which just haven't made it all the way in yet -- e.g. https://github.com/improbable-eng/ts-protoc-gen/pull/75 -- maybe the best solution is to try and pickup that work since that project is more popular and better supported with test cases, etc.

ts-protoc-gen@next now has support for generating types for both grpc-web and grpc-node services via the service=grpc-web (previously service=true) and service=grpc-node options.
I'm hoping that makes it into ts-protoc-gen@latest soon, and figured then I would open a PR here that bumps the version and adds the service=grpc-node option.

Great! 👍

@mhamrah, Thanks for merging #141! I think this issue can be closed. I confirmed that with the latest docker image, the new .d.ts files for gRPC services are generated.
@bartdeslagmulder, let me know if you get a chance to try it out and run into any issues.


Add credo config file

Make it easier to configure how credo is run.
Example: CI uses --strict, so in the config I turned strict on by default to avoid running mix credo locally and thinking all is well before pushing.
Also added custom config for Credo.Check.Design.AliasUsage so we could remove some magic credo comments.
i.e.
[
  if_called_more_often_than: 1,
  excluded_lastnames: defaults ++ ~w[Adapter Utils]
]

or could just disable the alias check altogether?
Could potentially do similar for Credo.Check.Refactor.CyclomaticComplexity.

Yeah! This is great! I've always hated how credo exclusions just kinda end up everywhere. Consolidating it makes a lot of sense.
I got rid of all the in-file credo overrides:

Cyclomatic complexity now checks globally at 15 (it's a sensible check, just a bit too sensitive)
I disabled alias checks entirely; I have a soft policy of not using aliases in public code to make it more explicit and require less ramp-up speed to read, so disabling this check wholesale makes sense in this project

Have a look and if you're happy with these changes we can merge this straightaway!


feat: add auto bump version ops template

based on this comment from slack
https://kubefirst.slack.com/archives/C03U34WJ7FW/p1703867034753859
i though to have have such automated mechanism, would be a game changer for the ops template
that's why i came with a quick implementaiton of renovate
this bot will create PR for all argocd APP:kustomization and terraform .
i make a the default setting for the AUTOMERGE( but can be easily change in the JSON file , we can discuss about it if you want) (all config options are here
for me you can allow ALL to an AUTOmerge (in FF mode ) (fastforward merge lover here) .
because you provide a template on a one time based . so no risk to break anything on update in customer side. ( but maybe i'm wrong about it)
meanwhile,
you have to enable install the app trhough the github portal => https://github.com/apps/renovate
then i advise you to make firsly a silent config, like this you can check what is achieve by renovate.
then allow only repository where a config file is present.
more info about renovate here
edit: you can check created MR on my fork https://github.com/DrummyFloyd/gitops-template/pulls?q=is%3Apr+is%3Aclosed
to check how it's looked

Thanks for this PR @DrummyFloyd . Will check with the engineering team what auto updating these dependencies would mean for kubefirst.

Thank for the review , do not hesitate if you want more info !


Dispatch when offline?

Hi, what will happen with the data when the ios-user is offline at dispatch time? Will the data be available and send the next time the user is online? Or is it lost?

I believe the sdk library keeps track of the data and sends it when available. Please refer to the sdk documentation to confirm.


Please update babel dependencies.

What problem does this feature solve?
The official babel 7.0.0 has been released,  but the version used by @vue/babel-preset-app is 'beta.47'.
Thanks for your efforts to create such a great framework.
What does the proposed API look like?
Update babel dependencies' version from '7.0.0-beta.47' to '7.0.0'.

@LinusBorg there is a PR #2377

@634750802 Shouldn't "babel-core" deps also be changed to "@babel/core"?

@corbinu  Which package.json file? In<EMAIL_ADDRESS>there is no "babel-core".

The main "babel-core" dep is under version 7.0.0-bridge. I think this may be a feature.

Yes as in it was the "bridge" until 7.0.0 was released ... @babel/core replaces it.

I'm not sure about what the bridge used for. Does it affect the babel compiler? I found another '@babel/core' dep in '@vue/cli-plugin-babel' which I think is the dep that babel used in fact.

# yarn.lock
<EMAIL_ADDRESS>  version "23.5.0"
  resolved "https://registry.yarnpkg.com/jest-config/-/jest-config-23.5.0.tgz#3770fba03f7507ee15f3b8867c742e48f31a9773"
  dependencies:
    babel-core "^6.0.0"
    babel-jest "^23.4.2"
    chalk "^2.0.1"
    glob "^7.1.1"
    jest-environment-jsdom "^23.4.0"
    jest-environment-node "^23.4.0"
    jest-get-type "^22.1.0"
    jest-jasmine2 "^23.5.0"
    jest-regex-util "^23.3.0"
    jest-resolve "^23.5.0"
    jest-util "^23.4.0"
    jest-validate "^23.5.0"
    micromatch "^2.3.11"
    pretty-format "^23.5.0"

Maybe it is used for jest, which depends on babel 6. Should I upgrade jest dep in this issue or create a new one?

https://github.com/facebook/jest/tree/master/packages/babel-jest

Needs help, It's beyond my ability.

jest needs<EMAIL_ADDRESS>to work with<EMAIL_ADDRESS>I believe the reason is that jest still imports babel core as babel-core, so the "bridge" is meant to force @babel/core to be imported.

Jest itself is ok, but we need this PR to be merged:
https://github.com/airbnb/babel-plugin-dynamic-import-node/pull/64
As this plugin is still using babel 6

@LinusBorg seems the v2.2.0 release support both babel 6 and 7 already,  according to this comment https://github.com/airbnb/babel-plugin-dynamic-import-node/pull/64#issuecomment-428011905

is this fixed? can somebody told me what to do?
currently i have
dependencies: { "@babel/core": "^7.1.2", }
and
devDependencies: { "babel-core": "7.0.0-bridge.0",     "babel-jest": "^23.0.1", }
and a lot more @babel plugin. but now each time i do npm isntall it says @babel/{plugins} requires a peer of<EMAIL_ADDRESS>but none is installed.
i am confused, should i downgrade it to @babel/core 7.0.0-beta.47 ? or can i do something?

@rgalaxy You need to upgrade those babel plugins.

@vikr01 hi vikr thanks for the reply, i already try to update babel plugins, but it still show the same, for instance
i upgrade @babel/preset-env but it still show the message<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
i dont understand, is it because there is a hidden plugin somewhere? and what's weird lot of the dependencies is not even ont package.json devdependencies. is it bcs vue cli 3 isntall it somewhere? even without showing on package.json?

@vikr01 It's a dependency of @vue/cli-plugin-babel. This is gonna be fixed in the next release.
Maybe the errors are due to the "@babel/core": "^7.1.2" in the package.json suppressed dependency hoisting so for now you can remove this field to see if it works.

@sodatea hi sodat, thanks for the reply, when you say  'This is gonna be fixed in the next release', it means i need to update my vue cli version and then reinstall? or will it update all the project inside and able to fix also the problems?
and i tried to remove all the @babel/ from my package.json, remove package-lock.json and run npm install, and.... guess what is still gaves me the errors..
and i can see this line<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
i am so ultra mega ultimately confused, i tried to create another vue-cli project and it works normally (without all the @babel/ thing)

@rgalaxy Are you able to create a repo that reproduces the issue? Just with all babel dependencies you had installed.

i am not sure, can you suggest me where i can do this?

I'm experiencing the same problem. @vikr01, if you are wanting an example you could use this package.json.
{
  "name": "test",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "serve": "vue-cli-service serve",
    "build": "vue-cli-service build",
    "lint": "vue-cli-service lint",
    "test:e2e": "vue-cli-service test:e2e",
    "test:unit": "vue-cli-service test:unit"
  },
  "dependencies": {
    "register-service-worker": "^1.0.0",
    "vue": "^2.5.17",
    "vue-class-component": "^6.0.0",
    "vue-property-decorator": "^7.0.0",
    "vue-router": "^3.0.1",
    "vuetify": "^1.3.0",
    "vuex": "^3.0.1"
  },
  "devDependencies": {
    "@cypress/webpack-preprocessor": "^3.0.0",
    "@types/jest": "^23.1.4",
    "@vue/cli-plugin-babel": "^3.0.5",
    "@vue/cli-plugin-e2e-cypress": "^3.0.5",
    "@vue/cli-plugin-eslint": "^3.0.5",
    "@vue/cli-plugin-pwa": "^3.0.5",
    "@vue/cli-plugin-typescript": "^3.0.5",
    "@vue/cli-plugin-unit-jest": "^3.0.5",
    "@vue/cli-service": "^3.0.5",
    "@vue/eslint-config-prettier": "^3.0.5",
    "@vue/eslint-config-typescript": "^3.0.5",
    "@vue/test-utils": "^1.0.0-beta.20",
    "babel-core": "7.0.0-bridge.0",
    "node-sass": "^4.9.0",
    "sass-loader": "^7.0.1",
    "stylus": "^0.54.5",
    "stylus-loader": "^3.0.1",
    "ts-jest": "^23.0.0",
    "typescript": "^3.0.0",
    "vue-cli-plugin-vuetify": "^0.4.4",
    "vue-template-compiler": "^2.5.17",
    "vuetify-loader": "^1.0.5"
  }
}

Well, in most cases you shouldn't ignore this; the issue arises from babel having so many package versions (babel 6, babel 7; etc.)
Here is a reference: https://nuget.qite.be/feeds/NPM/babel-core/7.0.0-bridge.0
solution for me was: sudo npm install --save-dev<EMAIL_ADDRESS>


ConnectionDetails has turned into options as well, so needs re-naming

Re-name ConnectionDetails class to ConnectionOptions.

This options class may be used for other operations in the future, so the name might change again.

Fixed, just forgot to reference it in the PR.


Update Android library version constant to spokestack-android-11.3.0

in gatsby-config.js:
ANDROID_VERSION: 'spokestack-android-11.3.0',

ff8747632185cbda38e4497f6660c3ee4cfea508


Shorten readme anchor text

Both for inbound and outbound the links are overflowing in the README.

In the Ruby library I just changed the text in the link to be different than that in the header.

Can we please do the same thing here?

Due to how  ReST handles linking to headings that wont work unless i change the section headings.

@danielknell maybe convert it back to markdown then?

@danielknell had a quick look at RST, pretty sure you can just do this:
`Get fax list <#get-outbound-fax-list>`

you can, but both github and pypi appear to disagree on how to generate the final html, so doing that for one will break the  other.

@danielknell you mean to say PyPi doesn't like:
`Get fax list <#get-outbound-fax-list>`

?

github wont, it prefixes the titles, pypi doesn't.

@danielknell so by all means we should be fine to use that?

we can give it a go.


Error occured when generating corruptions

Hi, thank you for your excellent paper and code release. I tried to generate ModelNet40-C but when I run generate_c.py, an error occured.
Traceback (most recent call last):
  File "data/generate_c.py", line 401, in <module>
    save_data(data[:,index,:], cor, sev)
  File "data/generate_c.py", line 360, in save_data
    new_data.append(MAP[corruption](data[i],severity))
TypeError: occlusion() takes 1 positional argument but 2 were given

I saw you comment some lines in generate_c.py, then can you give me some guides for 'occlusion' and 'lidar' corruptions?
# if cor in ['occlusion', 'lidar','original']:
        #     continue

And, if I want to use this dataset for benchmark, should I download ModelNet40-C directly using ./download.sh modelnet40_c rather than generate myself?
Thank you!

Hi Nahyu,
Thank you so much for pointing out this bug, we will fix it soon. It would be simpler for you to directly download the ModelNet40-C using our script.
Best,
Jiachen


the loss of my training and testing has been declining. I tested it with epoch 85th, but the effect is not so good. How many times does this epoch have to be better?

Do you mean 85 batches? To train 85 epochs, it will take too much time.
I trained two epochs to generate sample output as here.
For example,
python train.py --num_epochs=2

please let me know if you have run training for 2 epochs on all the gigaword data 3.8M rows or just 50K sample rows


Fix go-grpc generate.sh script

To update:

the proto/ directory change,
the breaking change of the go 1.17 & code generator:

https://github.com/golang/protobuf/issues/1070
https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/issues/902
https://github.com/grpc-ecosystem/grpc-gateway/issues/2263

Generate ok.
However, it might has problems after deploy to the repo because we have introduced the 3rd parties protobuf definitions.
Need to be confirmed after the code published.

Ci seems to have no problem. The new proto has some incompatible modifications. need to modify the go–wechaty code

I have commented out the below CI code today because it keeps failing:
https://github.com/wechaty/grpc/blob/208ca779d470d796fc6ab2319a78e1e58d899be5/.github/workflows/go.yml#L37-L39
Could you please have a look at it?
Thank you very much!


Reduce product tests' memory footprint

Might help with https://github.com/prestosql/presto/issues/683

Before: https://travis-ci.com/prestosql/presto/builds/109897460
After: https://travis-ci.com/prestosql/presto/builds/109897501

ready for review, travis failure unrelated. I pushed empty commt to rerun tests without obliterating "history".
The change is basically only that i reduce Xmx for master node in multinode  setup.
I preserved Xmx in multinode-{tls,kerberos} setups because there coordinator acts as a worker there. But we don't run heavy queries there, so somehow it doesn't fail (yet).
@sopel39 ?


fix: husky plugin compatibility

husky v9 only required knip to stop follow the git hook path and always check in .husky – something that happens to work great for husky v8 as well.
The other logic in #500 causes issues rather than helps, as gitHookPathsInV9 will be equal to gitHooksPathInV8 in a traditional setup.
I simply reverted the plugin to the implementation prior to #500 and then added the false to getGitHookPaths('.husky', false) and that keeps all tests passing – both Husky v8 and Husky v9 – and made it find the dependencies used in my husky projects, both for v9 and v8 husky projects.
(Side note: Husky v9 says its backwards compatible with Husky v8 and that one shouldn't have to do anything when upgrading for it to work, this small change aligns with that)
Does this look okay to you @risu729?

First of all,

I think this PR should work at all and is a better solution for supporting v9.
I made it to follow the git config because the original implementation of the husky plugin followed it.
I just followed it since I was not sure about older versions of husky, so I had a concern about what if core.hooksPath was configurable in the past.
However, husky started using core.hooksPath in v5 (source), and seems it didn't have any configuration of the hooks directory name (which is .husky/).
So, I believe this PR doesn't cause any issues.

@risu729 Makes sense 🙂

Thanks @voxpelli! Sometimes less is literally more. Truly appreciate the efforts + team work here.

:rocket: This pull request is included in v5.0.4. See Release 5.0.4 for release notes.


Duplicate Commento divs in Vue SPA

I am trying to add Commento++ to a SPA built in Vue. On my site the main page has a list of links to subpages. Each subpage has a commento-box div and runs a function almost identical to that from the README on the component's onMounted hook (runs when the DOM tree has been rendered). By adding two console.log to this function, I have tested that the commento.main() or commento.reInit() is only run once when switching to a new subpage, as expected. The main page does not run any Commento code.
This works, but sometimes, especially in a particular scenario, a subpage will have some of the Commento elements duplicated. It's always the same elements being duplicated, see below.

The issue happens a bit inconsistently, but the best way to trigger it (happens maybe 80% of the time) is to:

Go to main page.
Refresh page.
Go to subpage. (Usually no duplication.)
Go to main page (via a link from subpage, so there is no page reload).
Go to subpage. (Duplication happens approx 80% of the time.)

When the issue has occurred once, moving to other subpages or back to the same subpage usually does not trigger the issue again until the page is reloaded while on the main page. (But it has triggered a couple of times in this scenario too, I believe.)
I had the same issue when trying to add the hosted Commento (not ++) widget.
Any suggestions on what might cause this or how it could be resolved would be appreciated!

I probably know too little about Vue and/or Commento to figure out why this happens, but moving the code to its own Vue component seems to mitigate this issue.
So in summary for anyone else wanting to use this with Vue (Vue 3, composition API), I simply created the following component as Commento.vue (remember to replace my.domain.com with your commento domain):

<template>

</template>

and then use this component as such:
<Commento :page-id="some-id-variable"></Commento>


Output all differences between the compared derivations

Before this change, if the set of input names to a derivation changed,
then the set of matching-names inputs was not compared.
This fixes the issue, allowing to use nix-diff as a tool for system
administration: there, you have to know all the differences between the
two system derivations, not only until an input set has a mismatching
name (eg. a linux-kernel upgrade should not hide an nginx upgrade).
Fixes #5.

For ease of reviewing, the changes here are:

Change the if leftNames /= rightNames then A else B in when (leftNames /= rightNames) A >>= B to not shortcut (which means the indentation change that makes the diff hard to read)
Remove 4 trailing spaces I noticed when doing this diff

Awesome!  Thanks for doing this :)

And thanks for accepting it!  :)


Not able to install package via npm

Hi,
Trying to run simply npm i @javascript-obfuscator/escodegen fails with 404 not found. Is the package removed by some one?
thanks

Can confirm.

Also confirming

That's strange.
Maybe some NPM issues?
The package itself is here https://www.npmjs.com/package/@javascript-obfuscator/escodegen

What npm view @javascript-obfuscator/escodegen show for you?

For me npm i @javascript-obfuscator/escodegen works
So i believe it was something on the NPM side

https://status.npmjs.org/ shows that there were some issues with install packages

It works for me now. Must've been npm. Thanks!

Yes working now. Thank you every one.
Closing.


Update get_customer_verify_beneficial_ownership_from_body/1 to match resp

The Dwolla endpoint at /customers/{id} returns a few _links; one of those is used to populate the Dwolla.Customer struct field verify_beneficial_ownership.
The response I see when hitting this endpoint is (irrelevant portions omitted):
{
	"_links": {
		"verify-beneficial-owners": {
			"href": "https://api-sandbox.dwolla.com/customers/135ceaf3-7774-4f56-a602-ca0f8ea65af9/beneficial-owners",
			"type": "application/vnd.dwolla.v1.hal+json",
			"resource-type": "beneficial-owner"
		},
		"certify-beneficial-ownership": {
			"href": "https://api-sandbox.dwolla.com/customers/135ceaf3-7774-4f56-a602-ca0f8ea65af9/beneficial-ownership",
			"type": "application/vnd.dwolla.v1.hal+json",
			"resource-type": "beneficial-ownership"
		},
                ...
	},
        ...
}

I updated the first get_customer_verify_beneficial_ownership_from_body/1 function clause to account for this.
Please let me know if I can help with tests, or whether you can update the test factory to properly account for this scenario -- it looks like it uses real data from a sandbox account?
Thanks!

Apologies for the approval, I got confused and thought this PR was in our repo...


test(STONEINTG-385): update unit tests to be more clear about errors

This is outlined more thoroughly in Jira, but basically using Expect(a == b).To(BeTrue()) isn't idiomatic and hides important information in the case that tests fail.  This MR fixes statements like that throughout our unit tests.

1: UC1 Description should start with lowercase
My commitment to good grammar is killing me here

The Definition of Done document says that changes should be deployed to stage.  I assume that doesn't apply here since only unit tests were change?

The Definition of Done document says that changes should be deployed to stage. I assume that doesn't apply here since only unit tests were change?

Yes, it should be fine to consider this as done after the PR is merged.

do we want to fix the conflicts? or is that not needed?

@dheerajodha I fixed them.  They were just a couple of lines from #146.  Waiting for tests to pass before merging.


the warning and strange number of flops

Thanks for your work and I want to implement it to see my new model's performance. This is the first tool I have seen so far that could count some special operation in the forward process but not limited in just nn.Module.
I installed this tool and tried to count a simple linear model's flops. But the results seem strange with some warning. Would you please check what's going on? Thanks.
This is code.
import torch
import torch.nn as nn
from torch_flops import TorchFLOPsByFX

device = torch.device('cuda:3')

model = WindowAttention(dim=dim, window_size=(8, 8), num_heads=1).to(device)

dim = 192

model = nn.Linear(dim, dim*3)

x = torch.randn([1, 64, 64, dim])

print("=" * 10, "model", "=" * 10)

flops_counter = TorchFLOPsByFX(model)

flops_counter.propagate(x)

total_flops = flops_counter.print_total_flops(show=True)

And this is output.
❯ python flopsTest.py
========== model ==========
/home/guest/anaconda3/envs/Compressor_3_10/lib/python3.10/site-packages/torch/overrides.py:110: UserWarning: 'has_cuda' is deprecated, please use 'torch.backends.cuda.is_built()'
  torch.has_cuda,
/home/guest/anaconda3/envs/Compressor_3_10/lib/python3.10/site-packages/torch/overrides.py:111: UserWarning: 'has_cudnn' is deprecated, please use 'torch.backends.cudnn.is_available()'
  torch.has_cudnn,
/home/guest/anaconda3/envs/Compressor_3_10/lib/python3.10/site-packages/torch/overrides.py:117: UserWarning: 'has_mps' is deprecated, please use 'torch.backends.mps.is_built()'
  torch.has_mps,
/home/guest/anaconda3/envs/Compressor_3_10/lib/python3.10/site-packages/torch/overrides.py:118: UserWarning: 'has_mkldnn' is deprecated, please use 'torch.backends.mkldnn.is_available()'
  torch.has_mkldnn,
total_flops =   0 (1 operations are ignored or not recognized)

My environment:
❯ conda list pytorch
# packages in environment at /home/guest/anaconda3/envs/Compressor_3_10:
#
# Name                    Version                   Build  Channel
pytorch                   2.1.0           py3.10_cuda11.8_cudnn8.7.0_0    pytorch
pytorch-cuda              11.8                 h7e8668a_5    pytorch
pytorch-mutex             1.0                        cuda    pytorch

The reason is that directly wrapping nn.Linear to a graph module will use the function of linear (i.e. torch.nn.functional.linear) rather than the module of Linear (i.e. nn.Linear). The function linear was not considered in the old version of torch_flops. This issue can be addressed in two ways:

Change from: model = nn.Linear(dim, dim*3) to: model = nn.Sequential(nn.Linear(dim, dim*3))
I have updated the operations in torch_flops. You can update the library to version 0.1.3 by running the following command: pip install torch_flops --upgrade -i https://pypi.org/simple. After updating, please try running your script again. I have tested your code, and it runs successfully.

Just ignore the warnings. They do not matter.

Thanks, I will try it later. Thanks for your hard working.

It works, thanks.


The AwaitEventuallyConsistentResult should be implemented in ADO.NET, to not waste CPU

Does not look like a big deal


Upgrade otel to 1.28.0

cc @trask
autoconfigure module is stable in 1.28.0

API change check
API changes are not detected in this pull request.


[Prototype] Force shutdown routine to be run

Hacky version of (hopefully) sound continuations (#814). These added constraints should force the prover to set the jump_to_shutdown_routine flag correctly, e.g.:

It should not be set while still in the bootloader
After the bootloader, it should be set exactly once
The shutdown routine should have enough time to actually finish

This is only a hacky implementation, because we could not have added these constraints already. For example:

The PC update rule is replaced
Accesses the instr_jump_to_bootloader_input flag to find out if the bootloader has finished
Accesses the pc_update intermediate polynomial to make sure that the final PC is exposed correctly in the bootloader inputs.

To test:
PILCOM=$(pwd)/pilcom/ cargo test --all --all-features --profile pr-tests -- --ignored --exact test_many_chunks_memory

@georgwiese just curious: can https://github.com/powdr-labs/powdr/pull/1221 be used for the pc stuff here?


ERROR help!

http://imgur.com/a/FHqy7

Update agario-client which can be installed via (npm install agario-client).
ID : https://github.com/davidmann4/ogar-feeder-bot/issues/324


Correction for raw affiliation Compagnie Pulso (France)

Correction needed for raw affiliation Compagnie Pulso (France)
raw_affiliation_name: Compagnie Pulso (France)
new_rors:
previous_rors: 03mwypp50
works_examples: W4393301663
contact: 9d983fc81ca284ec9ba6fcb4c0e8438a:3a511ed210ac8f9cfbe7c5cc8026bb @ recherche.gouv.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


PD Toolkit: Be able to render advanced Pure Data types of nodes with SVG

Number2, VSlider/HSlider, VRadio/HRadio, VUMeter, Canvas, Graph, Array

improve Bangs and Toggles


Upsteam20240911

About the PR

Why / Balance

Technical details

Media

Requirements

[ ] I have read and am following the Pull Request and Changelog Guidelines.
[ ] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

wrong repository

big wizden doesnt want you to know what ss14's upstream is...


[Question] get_dataset failed

Question
when I run
env = gym.make('maze2d-umaze-v1')
env.reset()
env.step(env.action_space.sample())
dataset = env.get_dataset()
print(dataset['observations'])

something wrong happened:

urllib.error.HTTPError: HTTP Error 404: Not Found

I have tried to download dataset directly from  link [link], however I don't have permission to access /datasets/offline_rl/maze2d/maze2d-umaze-sparse-v1.hdf5 on this server.
How can i fix this problem?

import gym import d4rl  env_name = 'halfcheetah-medium-v2' env = gym.make(env_name) dataset = d4rl.qlearning_dataset(env)
I have the same issue with the above code

import gym import d4rl env_name = "halfcheetah-medium-v2" env = gym.make(env_name) dataset = d4rl.qlearning_dataset(env)
I have the same issue with the above code

The problem may be caused by an internet connection issue. Downloading the data manually and placing it under the corresponding path will solve it.

Hello friends, I had a similar problem and this solved the issue for me:
Install the stable version of D4RL from here: https://github.com/Farama-Foundation/D4RL/tree/master
Have a nice day!

Hello friends, I had a similar problem and this solved the issue for me:
Install the stable version of D4RL from here: https://github.com/Farama-Foundation/D4RL/tree/master
Have a nice day!

Hello friends, I had a similar problem and this solved the issue for me:
Install the stable version of D4RL from here: https://github.com/Farama-Foundation/D4RL/tree/master
Alternatively, datasets can be downloaded from these links (in my case the links were different, starting with gs): https://github.com/Farama-Foundation/D4RL/blob/master/d4rl/infos.py
Have a nice day!


[Infra] Cocoapods support #258

Description

Podspec file
File duplicates removed
*Imports.swift files for @_exported imports

Resolves #258 (issue)
How Has This Been Tested?
Cocoapods integration can be tested on v1.0.5-rc tag
pod 'WalletConnectSwiftV2', :git => 'https://github.com/WalletConnect/WalletConnectSwiftV2.git', :tag => 'v1.0.5-rc'

Due Dilligence

[ ] Breaking change
[x] Requires a documentation update

it's great !

some file miss, just like Sign and Auth Pakcage

some file miss, just like Sign and Auth Package

Good catch, fixed. There was wrong source paths.
Released new tag:
pod 'WalletConnectSwiftV2', :git => 'https://github.com/WalletConnect/WalletConnectSwiftV2.git', :tag => 'v1.0.5-rc3'

Regarding Auth. Auth is available buy downloading subspec.
pod 'WalletConnectSwiftV2/WalletConnectAuth', :git => 'https://github.com/WalletConnect/WalletConnectSwiftV2.git', :tag => 'v1.0.5-rc3'

the demo can not use,by cocoapods version

the demo can not use,by cocoapods version. Error Tips 'No such module 'WalletConnectSign''

maybe you can import walletConnectv2  replace 'WalletConnectSign'


feat(widget-image): enable images ordering

closes #2701
Summary
Hello! First of all, big thanks to the maintainers and contributors for this massive achievement !
To the best of my knowledge, there is currently no way to re-order images when using the widget-image (when combined with the Cloudinary media extension for instance to enable multiple selection). I see this issue has already been raised here, and I'm surprised it didn't get a lot of votes, as it seems to me that it's quite an important feature for all websites that display some sorts of photo galleries. Personnally, this is unfortunately kind of a showstopper for some projects, typically when building static websites for artists or photographers: these folks appreciate (rightfully) to be able to polish the ordering of their image galleries in the CMS directly). I guess I'm not the only one blocked by this, so hopefully improving this can even help to increase Netlify CMS adoption for people encountering the same issue :innocent:
So I did this by using the react-sortable-hoc package already used in the widget-list. Disclaimer: as it's my very first time touching any React code (Vue dev here), I was kind of in unkown territories, so don't hesitate to challenge this little PR, I'm open to any guidance. This is more like a basic proposition to bring something to the table.

Test plan
It's immediately testable in the image field at the bottom of the Kitchen Sink when the config's media_library is switched to Cloudinary. The test suites passed all green. Not sure if anything else is needed ?
Question for reviewers
I have sometimes this little warning in the console, but that I'm having a hard time debugging since my knoweldge of React is pretty limited.. I also notice the same warning for some other components in the Kitchen Sink, on master, so I don't know if it has something to do with the dev-test environment or if it's something that can be easily fixed. Any guidance on how to solve this is welcome :)

A picture of a cute animal (not mandatory but encouraged)

This is amazing @Francois-Stevenart  Sorry for the very late review.
I just made a small change to use pointer only if the image is sortable.

Great, thank you !
And good addition indeed for the cursor :)

This is amazing @Francois-Stevenart  Sorry for the very late review.
I just made a small change to use pointer only if the image is sortable.

Great, thank you !
And good addition indeed for the cursor :)


fix(terms): fix removing terms from schema field & add cypress tests to cover these flows

Introduced in https://github.com/linkedin/datahub/pull/3921
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

Just re-triggered CI on accident, but this just passed an earlier run w/ the same code;


lang: [nl] Add Dutch Languages

Added the Dutch languages

Hello, thanks for the PR.
We have failure in PHPCSFixer, can you run the following command to fix the error?
composer cs-fix tests/Language/AbstractTranslationTestCase

Sorry, I will try again today to update the previous one as well, but unfortunately I can't at the moment verify the other ones.
Probably due to ignorance as this is my first time on github to help with another project

See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/workflow.md#gpg-signing-old-commits

It seems your PR branch is a bit out of dated.
Try first:
$ git fetch upstream
$ git rebase upstream/develop

Sorry, but that didn't help the output stays the same.
The repo is in sync i'm sure.
With the time from yesterday I'm trying now an hour off 3 with no results.
I don't understand why so small thing is so hard to get right.
I want to help with the dutch translation but not giving up right now so do you have some other solutions.
Or is deleting the pull request the only solution and start again with a new one?

the output stays the same.

Do you mean the same error message?
If so, please delete the branch in your local repository, and fetch the branch in GitHub again,
and try.
I got your PR branch and tried and got:
$ git fetch upstream
$ git rebase upstream/develop
Successfully rebased and updated refs/heads/snelledre/develop.

I think you will got the same message.
Your branch has a merge commit, so git rebase will remove it.

I don't understand why so small thing is so hard to get right.

It seems you are a very beginner of Git.
Git is very difficult tool, so it is difficult to understand.
And if you don't understand Git concepts, you cannot use it easily.
Starting again new PR might be easier than solving your current issue.
If you create a new PR, I recommend you create a new branch for the PR,
not using develop branch.

@snelledre Starting anything is a bit difficult, if you are not familiar with it, it becomes more difficult.
Your situation in this PR has existed for all of us one day, like me confused, ambiguous and :weight_lifting:
You are not the first person in this situation and you will not be the last :snake:
The conflict and difficulties of these PR can make you interested in learning new topics such as git, github cli ,phpunit test and... think positive 🦾
As @kenjis' said before, if you don't succeed, just create a new PR.

If you don't know how to use git, I recommend you try https://learngitbranching.js.org/


Implement version security checks

As a IDEasy user, I want to get security warnings if I am using outdated software with critical known CVEs so that I can keep my software secure.
This is the devonfw-ide story 1106 to be implemented for IDEasy.
ATTENTION: There is a specialty for git that is not typically managed by IDEasy (what might change see #47). For this also have a look at the old PR implementing this story in devonfw-ide.

Currently it seems, only VersionRange entries are allowed inside the security file. Should we also allow a VersionIdentifier? Should there also be a commandlet that lets you add versions to the security file?

I've added a first batch of security files in this PR: https://github.com/devonfw/ide-urls/pull/15

After discussing this issue we have to answer following questions.

Should we use CVE's to check internally if a tool version is to be considered as good or bad (based on the severity of its CVE's)?
Should the severity threshold be adjustable by the user of IDEAsy?
Are there different CVE tools which can check at runtime if a tool version is not safe?
What about identically CVE's in different versions of the same tool?
What about modified CVE descriptions, how to handle the diffs and where?

Should we use these CVE's to decide internally if a tool version is to be considered as good or bad (based on the severity of its CVE's)?

Definetly, yes.

Should the severity threshold be adjustable by the user of IDEasy?

There are general two options:

We use the standard OWASP tool directly inside IDEasy to check for CVEs. This will imply that before every tool download the entire CVE database has to be updated what takes time and slows down the process. However, this will give lots of flexibility (user can then configure the severity threshold and this question is to be answered with "yes") and most recent CVEs and most accurate results.
We have an internal process that does the CVE checks in our UpdateURLs workflow every night. Here we use a fixed severity threshold that is rather high to avoid annoying warnings to every user. The end user only deals with security.json files that we add to ide-urls repository and works independent of OWASP tooling. Then the answer is "no".

We started with the latter approach but already added too many CVEs with that and then questioned the entire approach.
To be honest it does not seem an easy decision what would be the best way to go.
I still see pros and cons in both approaches.
However, after revisiting everything, I tend for the first option. We can make it configurable and therefore potentially could even allow the security check to be switched off saving performance (wait times for CVE DB).
If we go for it, we could still integrate OWASP tooling directly into IDEasy as maven dependency increasing our code base or we could do it as a local or global tool. Then we would first install that owasp-dependency-check tool and from that point on, use it and just ask it if «tool» + «edition» + «version» is fine or there are CVEs that should prevent the installation or at least require user-confirmation. The tighter integration would allow direct processing of the found CVEs, threshold filtering, and more control of what we display to the end-user and what questions to ask or options to give. However, if the CVE-DB format changes or the library itself is buggy or has security flaws, we are responsible and need to implement changes and deploy updated releases while if we call the tool as something external (black-box), we have less such responsibility.

Are there different CVE tools which can check at runtime if a tool version is not safe?

Yes, but most probably we want to stay with the OWASP one that is already integrated and open-source.
There are also other tools with advanced features that require a commercial license.
I distinguish two totally different aspects:

Analyze IDEasy and the tools it installs for a project
Analyze the product that some project is building and that project team may happen to use IDEasy.

In the 2. case, we should give more freedom if we want to support the security analysis of a git code repository with the help of IDEasy. There we could integrate such tools and also support commerical ones. However, this is a totally different story. See e.g. https://github.com/devonfw/ide/issues/1278[ide#1278 (snyk)]

What about identically CVE's in different versions of the same tool?

With the approach of the security.json we more or less wanted to define versions/-ranges that are "bad" (that is what devonfw-ide supported with security files and what I planned to reimplement in IDEasy).
Only for information and curiosity of the user, we wanted to give further details.
However, with PR #190 we took various rounds how the structure of security.json should be.
Current state is that somehow there are entries for version ranges but those only refer to a specific CVE (but since it is only a String it could also be multiple CVEs or we could redesign to a JSON array).
With all the complexity that arose in PR #190 and related aspects, I started questioning the entire approach.
Should we instead integrate OWASP dependency check directly (option 1) instead of creating our own dependency.json format managed in ide-urls (option 2 and original design)?

What about modified CVE descriptions, how to handle the diffs and where?

Exactly a good question that is a big pro for option 1 and con of option 2.
As I stated above, I therefore tend to go for option 1.
My suggestion would be to do a first PoC to see if that can be done rather easily and works well without blocking the user with hours of CVE-DB downloads, etc.


MQTT -> SmartThings Suddenly Stopped Working

Has something in either the smartthings code, or the mqtt bridge code changed to cause MQTT messages to SmartThings to stop working? I had this going for months without any issues, just last week I noticed none of my mqtt based temperature sensors were working. I see all the messages from smartthings to MQTT working fine, but nothing going back. Have been using a raspberry pi 3, also tried wiping the SD card and starting from scratch, same result every time. Any ideas?

I'm having the same problem.  No errors in Smartthings, but nothing posting on the bridge on my raspberry pi.  Started on the evening of 4/3/2017.

I uninstalled and re-installed and it is now working.  Sorry for the false alarm.

I've been running it since I published this module, and it hasn't failed for me yet.  One thing I noticed is if the SmartThings hub reboots and is assigned a new IP then connections will fail.
Should SmartThings get a new IP, you need to re-save the Smart App.
I recommend giving both the Raspberry Pi and the SmartThings Hub a static IP from your router.

I've got this problem.
I added a new switch and that caused me to turn off the power, and thus to the smartthings hub.
When I powered on, nothing is coming in to mqtt even.
I checked the settings and the IP, etc., is right in the IDE.
I can send mqtt from my PC and the lights go on/off.  I'm just not getting data from smartthings at all.
I see messages in the IDE log after changing the slider value for the ceiling fan in the iPhone smartthings app:
9:22:24 PM: debug Forwarding device event to bridge: {"path":"/push","body":{"name":"Ceiling Fan","value":"91","type":"level"}}
7ccd047f-9131-491b-ace4-8dc95e2367d3  9:22:24 PM: debug Sending '{"path":"/push","body":{"name":"Ceiling Fan","value":"91","type":"level"}}' to device
86db73ca-ac45-4800-b11c-27ee32ecba4c  9:22:07 PM: debug Forwarding device event to bridge: {"path":"/push","body":{"name":"Ceiling Fan","value":"53","type":"level"}}
7ccd047f-9131-491b-ace4-8dc95e2367d3  9:22:07 PM: debug Sending '{"path":"/push","body":{"name":"Ceiling Fan","value":"53","type":"level"}}' to device

I don't see anything from my mqtt server, though.
I stopped the bridge, removed the logs, state.json, etc.,, and restarted it.
The only file of interest is the events.log which looks like this:
 tail -f events.log
2017-07-02T04:31:04.006Z - info: Starting SmartThings MQTT Bridge - v1.3.3
2017-07-02T04:31:04.012Z - info: Loading configuration
2017-07-02T04:31:04.026Z - info: Loading previous state
2017-07-02T04:31:04.028Z - info: No previous state found, continuing
2017-07-02T04:31:04.028Z - info: Perfoming configuration migration
2017-07-02T04:31:04.034Z - info: Saving current state
2017-07-02T04:31:04.039Z - info: Connecting to MQTT at mqtt://<IP_ADDRESS>
2017-07-02T04:31:04.081Z - info: Configuring autosave
2017-07-02T04:31:04.083Z - info: Configuring API
2017-07-02T04:31:04.213Z - info: Listening at http://localhost:8080

Nothing more, even if I use the smartthings app or alexa to turn on/off things.
I checked in the app that there is an app with all my devices checked so the bridge should work.
I checked in the IDE and the settings have the right IP, MAC address, and port.
Next I'm going to try and kill the device and all that's in the IDE and redo it.
Maybe something in the README about how the hub's IP changing affects things?
Thanks for this.
It's making it possible for me to roll my own HA setup.

@mschwartz Interesting, it shows no previous state?  That indicates it has nothing to listen on.  Go into the SmartThings UI on your phone, edit the MQTT bridge app configuration, and just save it.  It should send a notification to resubscribe.

It shows previous state, but that state never changed.
I did switch from mosca to mosquitto, but everything worked.
I also switched from using my router to assign IP -> mac address to everything on my network to dnsmasq.  In doing so, I definitely renumbered things.  I use a netmask of <IP_ADDRESS>, which allows me to have 192.168.0.* for my computers, 192.168.1.* for my mobile devices, 192.168.3.* for home automation (echo, nest, smartthings hub, etc.), 192.168.4.* for my home entertainment devices (TV, apple TV, tivo, etc.).
Everything was working fine until I turned the power off at the circuit breaker to install a new smart switch.  When I turned the breaker on agian, the smartthings hub almost certainly got its new assigned IP.
I was able to get it working again.  I removed everything in the IDE and reinstalled it all and carefully did the device configuration.
If you want to see screen shots of my HA software, let me know.
I could post them here.

I think I am also having this issue but I have just done a fresh install, when something happens in Smart Things I can see the message in the bridge log and the state updates in Home Assistant but nothing works the other way, I turn "Plug" off, I see the message in the bridge log but nothing happens after that

I'm running into the same issue. I just installed hass.io and installed the smartthings mqtt bridge add-on. My bridge gets updates from smartthings and home assistant updates accordingly but when I use a switch in home assistant smartthings doesn't receive it. The bridge seems to see it but the smartthings hub doesn't?

So I managed to fix my issue - for me, I was running the bridge as a docker container and I used the MAC from the docker container but it turns out you have to use the MAC address of the physical host, after I changed that and re-added the app in Smart Things two way traffic started working

I think that was it for me too. I just switched to Hass.io which puts home assistant in a docker and I was using the docker MAC address too. Thanks for your help!!

If he docker container is exposing a port to the network, you clearly/obviously need to use the host's MAC address.
For example, the container exposes port 8080 for HTTP server/access, you'll be able to hit the host that's running the container, port 8080 and get the WWW server.
If the docker container is using docker network between containers only, then the MAC of the container matters (but I've not seen where anyone should care about the container's MAC anyhow...)

@mschwartz  I don't think its fair to say "Clearly/Obviously" as testified by the fact that other people have made this mistake too, If you are new to Docker as I was, having a container in bridge mode and seeing it have a MAC address I believed that was the MAC to use, it was only trial and error that identified my mistake.

I left an explanation for the next people to have the issue so they understand why.  It should help with future additions to your set up as you expand it.

Yeah that makes sense @mschwartz. Thanks for the technical explanation. I think what messed me up really was that I was using my router to find the MAC address and it was only showing me the docker MAC. I had to find the pi MAC address somewhere else.

If you don't mind some more technical information...
I turned off dhcp in my router and set up dnsmasq on one of the computers on my LAN.  The dnsmasq server does both DHCP and DNS.  Since doing this and having DHCP/dnsmasq set DNS server for everything on my LAN to my dnsmasq server, my web browsing and other network activity is much much faster.  There's quite a lag, in general, if you're doing DNS lookup over the internet all the time.  It's also a lot cleaner than editing your /etc/hosts file on all your computers to have symbolic name to IP address lookup for your LAN.
I have three tips that may help someone.
First, I set my NETMASK to <IP_ADDRESS>.  Most people probably just use up to 254 IP addresses  for their LAN.  This NETMASK allows you to use any IP address in the 192.168.. range.  That is, 192.168.0., 192.168.1., 192.168.2.*, and so on.  You now have 65536 IP addresses to assign as you see fit.
Second, I set up dnsmasq to assign a static IP to absolutely every device that's normally in my home.  That includes my TVs, my harmony hubs, the TiVo DVRs and set top boxes, my phones and tablets, my SmartThings hub...  EVERY device!
Third, in order to have a lot of flexibility to adding devices of a particular type (home theater, mobile devices, etc.), I assign a block of 255 IPs to the categories:

192.168.0.* - my router/gateway, my time capsule, etc.
192.168.1.* - my computers and laptops
192.168.2.* - my mobile devices (phones, tablets)
192.168.3.* - my home automation devices (amazon echo, ring doorbell, smartthings hub, thermostat, fitbit scale, etc.)
192.168.4.* - my home entertainment devices (smart tvs, TiVo defices, harmony remotes, roku, apple tv, audo receivers, etc.)

Hello,
After like...48 hours of configurations, i'm very new to HA, finally, i was able to get it working ST -> HA but not the other way around, in my IDE the MAC there is the PIs, im not using docker...
if i change a light state its reflected on HA but not the other way around...
any help is appreciated.

anyone?


🐞 [BUG] -

Issue Verification

[X] I have verified that this is a new bug report.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Describe the bug
In this website , there is a blank about section.
Expected behavior
I can add about section for this website so Please assign this issue to me under GSSoC'24.
Steps to reproduce
No response
Additional Context
No response

Duplicate issue


⚠ Jenkins Plugins Artifactory has degraded performance

In 3c89002, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1092 ms

Resolved: Jenkins Plugins Artifactory performance has improved in d39b747 after 12 minutes.