Invalid state error returned instead of unauthorized error

SDK Version: 1.10.1
React Platform Version:17.0.0

When getting this redirect that that I set up with rules I get "invalid state" as the error instead of the actual error returned.
The redirected URL:
http:///?error=unauthorized&error_description=Please%20verify%20your%20email%20before%20logging%20in.&state=QnFTa0JoRnJscDZPcklwTWdrbnM3OThMUDh6ODZ2YXlpRHZtUWhzLWhUcw%3D%3D#/
in const { isAuthenticated, error, logout } = useAuth0() the error is invalid state instead of the expected unauthorized.

Hi @weisssean - thanks for raising this
This is expected behaviour, the client checks for the state (to confirm that the login was initiated from the application) before asserting any errors from the AS here
Which makes sense, since if the client can't establish that the login started from the app, then it can't confirm that the messages from the server are for it

So how do I get and show the correct error to the user? scrape it from the URL? seems broken to me.

Are you hitting that URL as part of a login flow? If so, then your login flow is broken, because the SDK cannot pick up the state cookie to verify the callback request.
If you're just hitting that URL out of the context of a login flow, then I don't see why you want to show that message to the user. You're just creating a route that shows the user whatever you put in the error parameter.

OK, my issue was just that I had an error already in the url and I refresh the page. Instead of that error being in the useAuth0 ().error, it returns that "invalid state" error.
Not the expected behavior but I can live with that.
Thank you @adamjmcgrath


Decouple find TopProjectionSegment with select statement

For #4342.

Pull Request Test Coverage Report for Build 9611

18 of 31 (58.06%)  changed or added relevant lines in 14 files are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.0006%) to 57.629%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/core/filler/impl/dml/InsertColumnsFiller.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/sql/statement/dml/InsertStatement.java
1
2
50.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/MySQLDMLVisitor.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-oracle/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/OracleDMLVisitor.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-postgresql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/PostgreSQLDMLVisitor.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-relation/src/main/java/org/apache/shardingsphere/sql/parser/relation/segment/select/pagination/engine/PaginationContextEngine.java
5
6
83.33%

shardingsphere-sql-parser/shardingsphere-sql-parser-sql92/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/SQL92DMLVisitor.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-sqlserver/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/SQLServerDMLVisitor.java
0
1
0.0%

shadow-core/shadow-core-rewrite/src/main/java/org/apache/shardingsphere/shadow/rewrite/token/generator/impl/RemoveShadowColumnTokenGenerator.java
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-orchestration/sharding-orchestration-core/src/main/java/org/apache/shardingsphere/orchestration/internal/util/IpUtils.java
3
76.0%

shadow-core/shadow-core-rewrite/src/main/java/org/apache/shardingsphere/shadow/rewrite/token/generator/impl/RemoveShadowColumnTokenGenerator.java
4
0%

Totals

Change from base Build 872:
0.0006%

Covered Lines:
11086

Relevant Lines:
19237

💛  - Coveralls


AN 576 | Template ID verification for the signed data endpoint

Initially I was adding the templateId as a parameter to the processHttpSignedDataRequest function but thought it would be good to get other opinions first. With the way its currently implemented there is very little need to change any of the methods in index.ts .

In that case, I would suggest renaming the field for HTTP signed data gateway to encodedParameters, so the difference is obvious. I think it would be confusing if one example curl command would contain something like '{"parameters": {"param1": "string", "param2": 5}}' but the other one would be {"parameters": "0x3173737300....000"}.

Yeah that makes sense to me as well, although onchain the encodedParameters are referred to as parameters so maybe its fine?

In that case, I would suggest renaming the field for HTTP signed data gateway to encodedParameters, so the difference is obvious. I think it would be confusing if one example curl command would contain something like '{"parameters": {"param1": "string", "param2": 5}}' but the other one would be {"parameters": "0x3173737300....000"}.

Yeah that makes sense to me as well, although onchain the encodedParameters are referred to as parameters so maybe its fine?

I'd still say we should rename it to convey the distinction.

@bbenligiray You can review this, it should be the final implementation of the sign data gateway.

@amarthadan I think we can go ahead and merge this and burak can take a look at the documentation once that's been updated


Can you use the airbnb specification to constrain the project?

Can you use the airbnb specification to constrain the project?

If you mean their eslint configuration, yes, you just have to modify .eslintrc to point to their configuration; their repo has instructions for setting this up (it's trivial). Once you switch to their config, just run npm run lint -- --fix to automatically apply code style changes. If you mean something else entirely, let me know.


[BUG] -

Briefly describe the bug.
0x1774BebF95c5a0807177fd0Dbf0855b13658de66
How can we reproduce the bug?

Go to '...'
Click on '....'
Scroll down to '....'
See error

What is the expected behaviour?
No response
Can you attach screenshots?
No response
Can you provide your device specific details below?

Device Type - [ex. Desktop or Mobile]
Device OS - [ex. Android, MacOS, Windows, iOS]
OS Version - [ex. iOS 11, Windows 10]
Browser - [ex. chrome, safari]
Browser Version - [ex. 101]
Wallet - [metamask, portis, walletconnect]
Space - [ex. ENS, Gitcoin]
Any additional information we should know -

Please create a ticket at https://discord.snapshot.org and provide more details


Transition email topic is out of date

I just subscribed to the transition check email and got this:

Looks like this needs an update:
https://github.com/alphagov/finder-frontend/blob/7aa6970d8f9e62f2b766a8c449d24d68c2374100/app/controllers/brexit_checker_controller.rb#L57

Thanks for reporting this Tijmen!
We're deploying a fix shortly.

This is now fixed, so closing this issue


Depends on ansiblebit.launchpad-ppa-webupd8 even on Centos

Installing under CentOs gives the following error:
ERROR! the role 'ansiblebit.launchpad-ppa-webupd8' was not found in /home/klo/build-resources/ansible/roles:/etc/ansible/roles:/home/ko/build-resources/ansible/roles:/home/klo/build-resources/ansible

The error appears to have been in '/home/klo/build-resources/ansible/roles/ansiblebit.oracle-java/meta/main.yml': line 42, column 5, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

dependencies:
  - role: ansiblebit.launchpad-ppa-webupd8
    ^ here

$ ansible all -i targethost, -m setup | grep ansible_distribution
        "ansible_distribution": "CentOS",
        "ansible_distribution_major_version": "7",
        "ansible_distribution_release": "Core",
        "ansible_distribution_version": "7.2.1511",

@marsbard which ansible version did you use?
on the meta/main.yml I have defined this
dependencies:
  - role: ansiblebit.launchpad-ppa-webupd8
    when: ansible_distribution | lower == 'debian' or ansible_distribution | lower == 'ubuntu'

and it was working in the past but that may have changed.
if you tell me the version you use I can simulate the problem on my laptop.

@marsbard any input? please see my previous reply.

I'll see if I get a chance to look at it tomorrow when I'm at work (although it's slipped on my priority list right now...)

no answer back.
please reopen whenever you get some extra info.
cheers!

the role dependency was removed so the issue is definitely resolved from version >= 5.


Undefined variable model in add.blade.php view

I am facing an issue that when i clicked on add button for open the add form then error occurred that says
Undefined variable model in add.blade.php view.
I think there is need to set "isset" condition.

Hello
just do this :
php artisan vendor:publish
then go to  C:\xampp\htdocs\your_project\resources\templates
you will find file view.add.tpl.php
edit it and replace with this and you will get problem solved
view.add.tpl.php.zip

Thank you, quickest fix ever.

Thank you.


Build on windows, update Rubocop

When running tests on Windows machine with git config core.autocrlf set to true I got a failing test about content_length.
That was because ::File#size on Windows returns an actual file size (with CRLF line endings) but to_s used to read file in a text mode that converts CRLF into LF. Please read general info about this issue here if you want.
I set core.autocrlf to true in AppVeyor configuration as this setting is recommended by Github and Git book. Also it allowed to reproduce an issue there.
In addition, I updated Rubocop to 0.40.0 as old version didn't support a new Rake version. 0.40.0 is used in httprb/http repository.
cc @ixti, @tarcieri as Tony is a watcher in http repository but not here
Here is a build status in my fork - https://ci.appveyor.com/project/abotalov/form-data-rb/history

Coverage remained the same at 100.0% when pulling 01a38cc2da2a2e10fa4adddd9ba3f26504b48579 on abotalov:build-on-windows into 491c893a771dd29578c94f2ac0f98a7e4ebc01a9 on httprb:master.

Coverage remained the same at 100.0% when pulling 01a38cc2da2a2e10fa4adddd9ba3f26504b48579 on abotalov:build-on-windows into 491c893a771dd29578c94f2ac0f98a7e4ebc01a9 on httprb:master.

Coverage remained the same at 100.0% when pulling 1f487386fe036910e2b791b2e52f5449a2cbf603 on abotalov:build-on-windows into 491c893a771dd29578c94f2ac0f98a7e4ebc01a9 on httprb:master.

Coverage remained the same at 100.0% when pulling 1f487386fe036910e2b791b2e52f5449a2cbf603 on abotalov:build-on-windows into 491c893a771dd29578c94f2ac0f98a7e4ebc01a9 on httprb:master.

Coverage remained the same at 100.0% when pulling 1f487386fe036910e2b791b2e52f5449a2cbf603 on abotalov:build-on-windows into 491c893a771dd29578c94f2ac0f98a7e4ebc01a9 on httprb:master.

Coverage remained the same at 100.0% when pulling 1f487386fe036910e2b791b2e52f5449a2cbf603 on abotalov:build-on-windows into 491c893a771dd29578c94f2ac0f98a7e4ebc01a9 on httprb:master.

Coverage remained the same at 100.0% when pulling 1f487386fe036910e2b791b2e52f5449a2cbf603 on abotalov:build-on-windows into 491c893a771dd29578c94f2ac0f98a7e4ebc01a9 on httprb:master.

Coverage remained the same at 100.0% when pulling 1f487386fe036910e2b791b2e52f5449a2cbf603 on abotalov:build-on-windows into 491c893a771dd29578c94f2ac0f98a7e4ebc01a9 on httprb:master.

@abotalov thanks! Any interest in submitting something similar for the main repo?

My primary goal was to add a new feature to this repository (I haven't done it yet).
I wanted to make build pass on Windows as the first step.
I'll add AppVeyor to a main repository if I would want to do some work there.


Updates to RENAME TABLE page

Fixes #8129.

Updated the RENAME TABLE pages for 19.2, 20.1, and 20.2 to correctly refer to the statement as ALTER TABLE ... RENAME TO (RENAME TABLE is not a statement).
Made some minor formatting changes to 19.2, 20.1, and 20.2 RENAME TABLE pages.
Added a note about the limitation on changing schemas with a table rename statement.

In the future, we should avoid mislabeling statements. Another instance of this is with RENAME DATABASE, which is not a statement (it should be ALTER TABLE ... RENAME TO).

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b06c05915b301c3041077547bba5ae8baa123f2d/
Edited pages:

v19.2/alter-table.md
v19.2/rename-table.md
v20.1/alter-table.md
v20.1/rename-table.md
v20.2/alter-table.md
v20.2/rename-table.md

@lnhsingh TFTR!


revert material3 to 1.0.1 state

Revert only changes in commonMain, androidMain and tests, but keep our changes in skikoMain and desktopMain.

Have you taken changes before merge or from a specific commit from androidx?

Have you taken changes before merge or from a specific commit from androidx?

I used compare with branch - "release"/1.3.1"  where it's known that material3 was based on 1.0.1 version from Jetpack Compose.
For all differences, I simply took the version from our "release/1.3.1" branch.


Frame lag from supposed caching/sorting problem w/Chisels & Bits

After going through the motions apparently there is some sort of compatibility issue with C&B that is causing frame drops while in any Grid interface that has a C&B block. Algorithm uses some fancy words to describe the exact problem here. Just scroll on down!
https://github.com/AlgorithmX2/Chisels-and-Bits/issues/116#issuecomment-237938772

Interesting! Let me see if I can add sorting on demand instead of every frame, must be very doable.

Thanks for the help!


use abp zero Development project has  performance  problem

After develop several MPA webp projects, there are a few performance issues
The  page opens very slowly
The pop-up box loads the page very slowly

Hi,
Thank you for such a feedback. But "page is very slow" is too generic problem. Why do you think it's because of the framework. Do you have metrics that show it?


fix: use proper npm script

Summary by CodeRabbit

Chores

Updated the workflow for checking AdGem SDK updates to use a new command for version retrieval.

:tada: This PR is included in version 1.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Nested form array field value entered at the first got inserted in all object in array.

Description
I'm trying to create a nested form array. but I found out when I enter of words for example "John" in the first input field.. the rest of form array inserted the first letter of "John" that is "J". Is this some kind of bug or my mistake the way I create the form?.
here is the sample code
stackblitz sample

Oh never mind. I just need to do in the array.push(fields) in +page.server.ts


Uses document_type and language to enable switching from docs of the same type

This PR fixes the following bug: https://www.pivotaltracker.com/story/show/152392874
The bug prevented us from switching between ndc documents when a country had multiple.
I've used both the document_type and the language to set the value of the dropdown, and now we can happily switch between multiple NDCs. We just got to wait for them to start having multiple NDCs on the same language.... =D but I think that's not going to happen, because they told us that it wouldn't. =)
TEST
Go to the China full ndc page and switch between both NDCs. =) http://localhost:3000/ndcs/country/CHN/full

Good work! Just added a filter to remove duplicates, thanks!


Makefile: implement running DB tests locally

In the endeavor to implement pruning the customer data after two years (see https://issues.redhat.com/browse/HMS-4244 )
We first implement running the DB test locally by spinning up a container with postgres.
This is as close as possible to the implementation in the github action and enables to develop more DB tests and execute them.
Also changed here: image-builder-db-test is not compiled and run, but just executed as test, as we don't really need the binary

/retest

/retest

/retest


APT error

Hi,
I have an issue with the installer when I add custom apt sources with (public) GPG keys
I get the following error:
onfiguring apt:
Configuring Raspbian repository... OK
Checking Raspbian repository entry... OK
Adding raspberrypi.org GPG key to apt-key... OK
Configuring RaspberryPi repository... OK
Configuring RaspberryPi preference... OK
Copying './jessie-backports.list' to /etc/apt/sources.list.d/... OK
Copying './sury-php.list' to /etc/apt/sources.list.d/... OK
Copying './debian-archive-jessie.gpg' to /etc/apt/trusted.gpg.d/... OK
Copying './debian-archive-wheezy.gpg' to /etc/apt/trusted.gpg.d/... OK
Copying './sury-php.gpg' to /etc/apt/trusted.gpg.d/... OK
Updating package lists... ERROR: 100, trying again (2/3)... ERROR: 100, trying again (3/3)... ERROR: 100, FAILED !
Oh noes, something went wrong!
You have 10 seconds to hit ENTER to get a shell...
Raspberry info:
Revision v1.5.1 (bd078c1)
Built on Wed Jan 25 12:41:03 CET 2017
Running on Raspberry Pi version 3 Model B
also it's not placing files and I have a files. list in /raspberrypi-ua-netinst/config/files/root folder it looks like:
timv:timv           755   /home/timv/.bash_prompt
timv:timv           755   /home/timv/.bashrc
www-data:www-data   755  /var/www/index.html
www-data:www-data   755  /var/www/404.html
www-data:www-data   755  /var/www/503.html

Found the error with apt it missing
apt-transport-https
it is needed even tho I am using http URLS, maybe add this before apt sources etc... are added.

The list file has to be innthe same folder as the sample file. Is it?

Thanks for the apt info!

@TimVNL

Not installing files issue:
The custom files list file has to be in the same folder as the sample file (custom_files.txt). Only the source files have to be in the subfolder root. Is it?
Missing apt-transport-https issue:
I'd be happy, if you could test 8f08a9a. It should be fixed there.

@FooDeas

Fixed the custom files issue wasn't paying attention.
Will test this as soon as I have me new rpi ;)

Also maybe make a guide for exporting keys to put in the apt folder.


Add dribbled past duel type

PR related to https://github.com/PySport/kloppy/issues/257.
We introduced A DRIBBLED_PAST DuelType and added the TACKLE qualifier to duels for Opta & StatsBomb.

@DriesDeprest Done!

Would it be possible to add the TACKLE qualifier also to the Wyscout deserializers such that a "tackle" is at least consistently defined everywhere?
Intuitively, there should be a DRIBBLED_PAST event for each successful dribble. Would it be possible to write a test for this? I am not familiar with the StatsBomb "Dribbled Past" and Opta "Challenge" events and this would convince me that my interpretation of their documentation is correct.

Intuitively, there should be a DRIBBLED_PAST event for each successful dribble. Would it be possible to write a test for this? I am not familiar with the StatsBomb "Dribbled Past" and Opta "Challenge" events and this would convince me that my interpretation of their documentation is correct.

In StatsBomb's event data, the Dribbled Past event is typically paired with a completed Dribble event but can also be paired with an incompleted Dribble event. For example, the data for one of our recent games includes one Dribbled Past event that is paired with an incompleted Dribble event. The overrun flag for the Dribble event is set to true though.

{
  "id": "dd23312c-596e-4f63-b5a0-b75967299e9d",
  "type": {
    "id": 14,
    "name": "Dribble"
  },
  "related_events": [
    "17d2402c-0685-4dba-9e7b-dbeb055d02e8"
  ],
  "dribble": {
    "overrun": true,
    "outcome": {
      "id": 9,
      "name": "Incomplete"
    }
  }
}

{
  "id": "17d2402c-0685-4dba-9e7b-dbeb055d02e8",
  "type": {
    "id": 39,
    "name": "Dribbled Past"
  },
  "related_events": [
    "dd23312c-596e-4f63-b5a0-b75967299e9d"
  ]
}

Would it be possible to add the TACKLE qualifier also to the Wyscout deserializers such that a "tackle" is at least consistently defined everywhere?

Will do this!

How Intuitively, there should be a DRIBBLED_PAST event for each successful dribble. Would it be possible to write a test for this?

How would you implement this?
For StatsBomb we can use the related_event_ids property to check for all successful dribbles whether one of their related events is an event with the DRIBBLED_PAST qualifier
But for Opta & Wyscout we don't have these related event ID's, how would you go about this?

How would you implement this?

count(type  = DRIBBLED_PAST) == count(type = DRIBBLE & outcome = SUCCESS)

@DriesDeprest any update on this one?

Not really, I've had other priorities. I'll let you know if it changes.


What happened to squidgyberries RSPNVPK?

https://r2northstar.readthedocs.io/en/latest/guides/tools/tools.html#rpsnvpk-s
https://github.com/squidgyberries/RSPNVPK Page not found

Good question. Poking @squidgyberries

Note sure where the difference between that version and https://github.com/Taskinoz/RSPNVPK is but we could link to the other version instead ^^


Explain log queries in sig-monitoring

Once we have a demo (see https://github.com/giantswarm/roadmap/issues/2765) we can re-use this and present it in sig-monitoring

@TheoBrigitte should we close this ?

Yes, I ran the Loki demo in sig-monitoring and the recording is out. See https://github.com/giantswarm/roadmap/issues/2765#issuecomment-1785009879


[Bug] Loading SEG on new OHIF viewer gives "Something went wrongfalse"

Describe the Bug
In the new OHIF viewer (3.8/3.9?), I uploaded some of my data and it is only able to show one segmentation. When I try to load the second segmentation, I get a "Something went wrongfalse" error on the screen and "Error: getVolume: volumeId must not be undefined" in the console.
Steps to Reproduce

I first checked the validity of my .dcms (CT + SEG + SEG) in 3D Slicer  (download data here)

I then drag-dropped my .dcms to https://viewer-dev.ohif.org/localbasic.

I was able to view the first segmentation
But when I click on the second segmentation I get the following error

So when I click on "Show details" I just get a black page.
Checking the console, I saw a "Error: getVolume: volumeId must not be undefined" message

The current behavior

It does not show me the second segmentation (no matter in which order I load them into the viewer)

It shows me an error in the OHIF viewer and also in the console.

The expected behavior
It should also load the second segmentation.
OS
Windows 11
Node version
N/A
Browser
Chrome 130.0.6723.117

Seems like they each can load separately, but not together

I can see the issue, i'm fixing it


Send to Unreal - Collision meshes get merged into LODs

Addon: Send to Unreal
Steps to Reproduce the Problem:

Enable "LODs" under Import settings (and disable "Grooms")
Add Mesh > Monkey, named "Suzanne_LOD0"
Add Mesh > UV Sphere, named "Suzanne_LOD1"
Add Mesh > Cube, named "UCX_Suzanne_LOD0" (aside: rather awkwardly, the collision doesn't import correctly if this is named "UCX_Suzanne" without the "_LOD0", but this merge bug does still happen)
Export to Unreal
Observe that LOD1 in Unreal has a visible Cube merged into it (i.e. "UCX_Suzanne_LOD0")

Error Log: n/a
OS: Windows
Addon Version: Send to Unreal 2.4.9
Blender Version: Blender 4.1
Unreal Version: Unreal 5.3.2

I fixed this locally by adding an if lod == 0 guard around the collision code in export.py's export_mesh, which has solved this for my use cases without any negative consequences visible to me.
    # select collision meshes
    if lod == 0:
        asset_name = utilities.get_asset_name(mesh_object.name, properties)
        utilities.select_asset_collisions(asset_name, properties)

Nice catch! Thanks for taking the time to find a solution!


can sparrow be used as an offline signing device?

Sorry if that's a dumb question.
I don't see why an offline sparrow would be any different from a traditional hardware wallet (actually offline sparrow would be much better in a few ways, as I see it).  I see some evidence and comments that it can, and I've tried hard to make it work, but maybe it's not possible, or I'm doing something wrong.
I'd like my offline, never, ever, online, signing device to be sparrow on, say, tails, on an old laptop.  And then export my xpubs via qr or file from the offline sparrow to an online sparrow on another old laptop, build my transactions in the online sparrow, and then pass those transaction via qr or file to the offline sparrow for signing, and then similarly pass the signed transaction back to the online sparrow for broadcasting.
What repeatedly happens, with no ui indication of anything wrong, and nothing obvious to simpleton me in the logs, is:

in online sparrow 1.8.4-1, watch-only, single sig, Native Segwit (P2WPKH), build tx, Finalize Transaction for Signing, Save Transaction, sd card, .psbt file, eject sd card.
in offline sparrow 1.8.4-1, File, Open Transaction, .psbt file from sd card.  Transaction Overview and Details look correct (though seeing the Locktime Block was surprising to this beginner), Signing Wallet looks correct. Click blue Finalize Transaction for Signing, click blue Sign, see Signatures blue bar progress to apparent completion on the right with BIP39 label.  Blue Sign button still blue.  No indication of a problem. Save Transaction, to sd card, .psbt.psbt file, eject sd card.
in online sparrow, Load Transaction, .psbt.psbt file from sd card. Signatures blue bar progresses to far right, with BIP39 label.  I think a Broadcast Transaction button should appear.

What am I missing?  How can I help troubleshoot and test and document?
Thanks in advance, and thanks for an awesome product!

On the offline sparrow, the log ends with
a series of Transaction output value: ..
Found input witness utxo amount...
Found input sighash_type All...
Found input bip32_derivation with master fingerprint...
Found output bip32_derivation with master fingerprint...
Calculated fee at ...

My testing so far has had two different complexities:

One was, my motivation was to RBF a low fee transaction that I broadcasted via other more complicated means (and so I was trying to use this offline-online-sparrow config to simplify that).
Another complexity was my "simpler" test was trying to send to my taproot wallet (because I'm cheap, as you know, and I hear taproot has lower fees).

For the sake of science, I just now made an even simpler test, just sending some sats back to a new address in my same Native Segwit (P2WPKH) wallet, and this time, on the offline sparrow, when I clicked Sign, it changed to Save Final Transaction...  I saved that to a ".txn" file (note not .psbt.psbt file this time), and then loading that .txn file into online sparrow, the Signatures bar went full, with BIP39 label, and a big beautiful blue Broadcast Transaction button appeared.
My cheapness has cost me many hours...let's work together to make those hours not a complete waste...

What you are doing is a well tried and tested approach :)
There is no problem with loading a signed .psbt into Sparrow to broadcast. There's also no problem with this transaction sending to a Taproot address, or being an RBF transaction. What I think is likely here is just that the "online" Sparrow was actually offline, as in the toggle in the lower right was off.
Fwiw I just tested your original scenario and everything worked fine. Happy to reopen this if you find some issue.

I bet tor is the problem.
To be clear, online sparrow had great connectivity to the default public node, quickly and completely finding all of my utxos from all of my different wallets and accounts, had great fast responsive current fee estimations, and like a lazy lifeguard watched my too-low-fee transaction slowly sinking into the deep end of the mempool. :)
Did you test on mainnet, with tor?
Do you think the problem (on tor) was at the transaction-building step, during .psbt creation?  I think it must be, because of the missing "Save Final Transaction" on the offline Sparrow. The .psbt must be missing something subtle.  I think that may be the first bug.
Then the signing sparrow thinks it has enough info to sign the transaction, and so it seems to sign it and move the progress bar to the far right (looks complete and successful), not knowing that something is wrong.  Part of the code thinks it signed the tx, but the next part of the code either disagrees or is not aware, silently.  That could be the second bug (or two).
How can I help?

Do you think the problem (on tor) was at the transaction-building step, during .psbt creation? I think it must be, because of the missing "Save Final Transaction" on the offline Sparrow.

I don't think so, but could you check the log file for the offline Sparrow? Help menu. It may show something that will indicate the cause.

sparrow_redacted.log.gpg.zip
I don't see anything obvious.  Sorry if I over-redacted.

Unfortunately nothing useful I could see in the log file either. If you can find a set of steps to reproduce (perhaps on testnet) that would be useful.

Here's a consolidation.psbt file and testnet trace log with the same behavior.  Remove trailing ".zip" from .gpg filename.
testnet-fail.zip.gpg.zip
Let me know how I can be useful. :)

Thanks. Unfortunately I can't see any problems with the PSBT, and the log file contains no errors. Of course without the seed I can't sign the PSBT to actually see what you're seeing. At what stage in your described process did you save this PSBT - was it on the online Sparrow, or after signing with the offline one?
With my own seed, I tested the scenario again on my side without any issues.

Sorry for this distraction. I hope we discover something helpful.
Testnet_Test_20240409_01-05-37AMUTC.zip.gpg.zip

Thanks for the data. Unfortunately it has brought us no closer to a solution on this though. Here's what I did:

Created an offline only version of the wallet with the seed. This wallet has no history.
Successfully signed testcons.psbt with this wallet. The Save Final Transaction button appears.
Created a watch only version of the wallet with the output descriptor. This wallet was connected and has history.
Created testcons2.psbt with this wallet following the steps you described.
Successfully signed testcons2.psbt with the offline wallet.

If you do the same on your online computer (using the toggle in the lower right to connect/disconnect) are you able to reproduce the problem? I can't think of a reason the clock would cause any issues, but who knows at this point.

Inspired by something about your wording, on my offline sparrow, I closed my testnet wallet, and created a new wallet with the same seed, and it signed the .psbt perfectly, creating a .txn file, which broadcast and confirmed perfectly from my online sparrow.
Do we have a silently corrupt offline wallet?  Both wallets attached.
testnet_wallets.zip.gpg.zip
Let me know how I can help.

Thanks - finally found it. It turns out to be a simple gap limit problem (napkin_0 required a gap limit of 35, not 34 to sign all the inputs).
However, Sparrow should automatically increase the gap limit, but it didn't in this case because it could partially sign the transaction (some of the inputs). This edge case has been fixed in 93893111.
Thanks for your help in finding this! Increasing the gap limit on your original offline wallet should solve the problem.

Great to hear! Thanks for your patience and hard work!
Any logging or UI changes practical to help prevent or troubleshoot the next issue?

Nothing new - log file is best.


Clarify some areas in the docs and update linting options, list tasks

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (a68613b) 55.63% compared to head (a3e681c) 55.63%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #66   +/-   ##
=======================================
  Coverage   55.63%   55.63%
=======================================
  Files           7        7
  Lines         293      293
=======================================
  Hits          163      163
  Misses        130      130

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Chimera Watchdog Repair

Changelist

Last year, when doing board bringup/chimera testing, we disabled the watchdog. Bringing back watchdog broke Chimera, since the chimera version of tasks was not petting watchdog.
This PR removes io_chimera_sleepIfTaskEnabled() in favour of io_chimera_block() and io_chimera_task().

io_chimera_task implements a simple watchdog pinging task.
io_chimera_block operates like io_chimera_sleepIfTaskEnabled, and sleeps the task if chimera is enabled.

Testing Done

[ ] Tested in BMS @DJ90864

Resolved Tickets

I think we need a far deeper fix for this. The watchdog code from before I just raw ported.

changes I think should be here:

refresh_watchdog removed in favour of bringing a reference to the watchdog into the watchdog monitor
refresh HAL watchdog only when there are no timeouts detected
if there are no watchdogs registered, then the watchdogs are vacuously fine, and so we should trigger a refresh
in chimera task, we trigger watchdog monitor on the vacuous case


modify increment from 5 minutes to 12 hours

Fixes #2

Modifies interval to every 12 hours
Also removes redundant npm install near-protocol-rewards, since npx is being used

@elliotBraem thanks so much for these changes and revisions! The original workflow was set to every 5 minutes to ensure the functionality was working correctly between the SDK and dashboard. I agree with you on extending the increment to 12 hours - thank you for catching this!


References!

Have you added references to your documentation? Please let me know which format have you used for your references?

I have just pasted the link of the resources we have used.


Fix missing processes issue

This PR fixes the issue #844
The changes include:

[x] fix libbpf get batch premature return bug to collect all keys in the map
[x] fix libbpf code to use current task to set the right cgroup id
[x] increase the bpf map size to be able to trace more threads
[x] fix get containerID from cgroup path bug of return containerID unknown
[x] fix get containerID intermittently returning system_processes instead of the real containerID
[x] fix containerID bug to return a valid container ID, some system processes were not having the system_processes ID, so that the aggregations were not properly done
[x] reset PIDs list since it changes in each metric collection
[x] update to use a valid PID to start a cgroup manager

example of previous invalid container ID:
I0817 10:39:05.777237       1 resolve_container.go:264] ##### DEBUG: extractPodContainerIDfromPath 2 containerID: /sys/fs/cgroup/system.slice/containerd.service cgroup path: /sys/fs/cgroup/system.slice/containerd.service

@sunya-ch @SamYuan1990 the libbpf building is failing in the integration test.
Do you know how to fix it?

@marceloamaral do you have a updated https://github.com/sustainable-computing-io/kepler/tree/main/bpfassets/libbpf/bpf.o

@rootfs updated!


[ci] Better reporting on invalid JUnits

This makes it so parsing JUnits is not an all-or-nothing proposition. Now failures are put into a side channel and reported but they do not affect other JUnits. This also improves reporting a bit so we can see if the pytest_wrapper.py script has failed or not.
Fixes #11749
cc @Mousius @areusch

we just need to update the junitparser in the qemu image to 2.4.2


Update Hindi README.md

Update the docs/in/README.md to match the latest version of the README.md in #3313

Merged in #3333


Update request: tenacity 1.3-beta2 → 1.3.1

Package name: tenacity
Latest released version: 1.3.1

Current version on the unstable channel: 1.3-beta2
Current version on the stable/release channel: 1.3-beta2

So, wanted to use tenacity because i'm not letting my grudge against muse go, sorry, when i noticed something odd: the version in nixpkgs is older than the latest release.
appears there's no mechanism to alert maintainers of updates for this automatically so here's an issue i guess

[X] Checked the nixpkgs pull requests

Notify maintainers
@IreneKnapp @lheckemann

Note for maintainers: Please tag this issue in your PR.

oh hey yep, that's me! thanks for the heads up, I'll get on that

(and I feel similarly about the whole audacity mess, it's why I packaged tenacity. I think this will be the first time there's been a proper stable tenacity release in nixpkgs, I'm looking forward to it.)


Add array type support for EventPipe to TraceEvent

Companion PR to https://github.com/dotnet/runtime/pull/36242. This PR adds support for array type parsing to EventPipeEventSource.

@noahfalk I apparently don't have the permissions to add you as reviewer

Theres one remaining failure that is only showing up in CI, I believe I know what the issue is and I'm working on it.
@noahfalk you probably want to at least glance again, the changes are big enough I don't consider them just updates

Ok, nevermind. It looks like this is a pre-existing failing test

Overall, this looks good to me as well.  @noahfalk, @davmason are you both ready for merge?

I'm done with my changes and ready to merge


Add "Available Peers" to menu bar

Issue type
[X] Feature request
[ ] Bug report
[ ] Documentation

Environment

Python version:   3.5.3
Peering Manager version:   0.99-dev

Description
Quick access to all available peering sessions across all IXs from Peering Manager menu bar.

This function would be a really cool feature as we try to setup a as redundant peering as possible to the networks we peer with!


chore: release main

:robot: I have created a release beep boop
renderer: 1.0.6
1.0.6 (2023-12-23)
Bug Fixes

update readme (07c7880)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/pavophilip/react-shader-fiber/releases/tag/renderer-v1.0.6 :sunflower:


Import DHCP IP ends up as Active

When attempting to import a DHCP IP the IP's all show up as Active rather than DHCP.  Example of import format below which results in Active.
<IP_ADDRESS>/24,,,DHCP,,,,

Duplicate of #674, which was fixed just earlier today. Will be resolved in v1.7.1.


Instagram phishlet fails to load images

I can successfully capture session cookies and import them from the attacker machine to "hijack" a session, but on the victim side, they see this:

I would like to add that I've set up A records in accordance with the results of "phishlets get-hosts Instagram" which includes things like img.instagram.myDomain.com, i.instagram.myDomain.com, etc. where "myDomain" is the domain I've set up and connected to DigitalOcean.

May I ask if you ever had the problem, that you couldn't accept or deny cookies at the cookies prompt (login screen)? If yes, how did you solve it? Thanks.

I have the same problem, @kgretzky any update on this?
I have also used the latest patch from @charlesbel PR (https://github.com/kgretzky/evilginx2/pull/611) but the problem was still not fixed.
I have also tried to fix the phishlet but with no satisfactory results so far.

I have the same problem, @kgretzky any update on this? I have also used the latest patch from @charlesbel PR (#611) but the problem was still not fixed. I have also tried to fix the phishlet but with no satisfactory results so far.

Have you had any luck so far? My guess is the phishlet is missing one of the domains that Instagram pulls images from but I've had no luck trying to fix it.

@LarmonH I managed to do it, now it loads all the images correctly except for the ones that are round shaped right under the search bar. (But I will make them load as well soonish, I'm getting there).
You are correct about your above guess. It doesn't pull the images from the correct domain.
By the way, I have one more problem, when I enter the credentials another tab opens where I am logged in and the previous one loads continuously (like it is trying to log in), have you managed to fix this one?

@wirespecter I also noticed that. Something must be causing a new tab to be opened upon the user inputting credentials rather than just logging in on the original tab. I haven't had any luck trying to fix it as I've been focussed on the images problem. I would like to take a look at your solution to that by the way whenever's convenient for you.

I will gladly show you the solution for the images if you could help me please with the tab opening bug, I have already spent so much time on this. It seems to happen when a post request is sent to /accounts/login/ajax.

I found some more info about this, it seems that instagram's "link shim" is responsible for the new tab. A script takes the new link and translates it to: "l.instagram.com/URL".

Update: I found a solution for the tab opening as well! It took me only 7 days :(
It gets easy when you start to debug the javascript, thanks everybody for not helping at all! Cheers!! Later world!!

@wirespecter Apologies for not finding the fix before you, I've been very busy with school. I'm sure the developers would appreciate you sharing how you fixed the phishlet (more specifically @charlesbel on his updated phishlets page: https://github.com/charlesbel/Evilginx2-Phishlets). Give it some consideration.


Half Donut

This PR introduces support for a "half donut" chart.

Summary of changes:

Export constants that define half donut angle ranges
Add a new example that cycles through orientations (top, right, bottom, left) and also lets you resize dynamically  (height > width, height < width)
Make the scaling work for all 8 cases (top, right, bottom, left) X (height > width, height < width)

Based on a previous attempt in

https://github.com/f5/unovis/pull/444/files

Current status: distracted by TypeScript linting warning:

I thought it would pick up the change... Maybe I need to build or something?

Got it - just needed to run npm run build:ts

Current status:

Solved left and right cases
First pass of the feature is ready
This will require custom margins to be passed in from the outside in the case that there is a text label
IMO this is acceptable, since a more automatic solution would probably involve additional complexity around measuring text bounding boxes, which may not be worth it

Looks good @curran -- could be the screenshots, but can you confirm that the space between the sections is the same as the other pie charts

The size of the gap in the middle depends on the size of the container.

@krisexa Are you talking about this?

If so, you can just ignore it.

Yeah AOK then

@rokotyan This is ready for re-review.
Summary of changes since last review:

Addressed code feedback
Changed the base of the PR to [1.5.0-exaforce](https://github.com/ExaForce/unovis/tree/1.5.0-exaforce)
Also opened a PR with the same changes in https://github.com/f5/unovis/pull/502

I suppose we can merge this one here and I'll let you take it from there re: package publishing. Cheers!

Investigating how to do this part:

Unovis doesn't squash the commits so we should clean up the commit history to have only two: one for the component changes (Component commitlint prefix), and another one for the dev example (Dev commitlint prefix).

The additional work around rearranging the changes into two commits, one for the component changes and one for the dev example, seems to be fairly involved, and has some downsides to it as well. For example, we'll lose the thread here of the feedback comments interleaved with the commits, as discussed in [StackOverflow: How to squash commits in git after they have been pushed?](https://stackoverflow.com/questions/5667884/how-to-squash-commits-in-git-after-they-have-been-pushed).
Would it be acceptable to squash the commits into just one instead of two? That seems more readily doable with off-the-shelf git commands. For making two separate commits, I'm not sure how to go about that. I would need to somehow "un-stage" all the changes that have already been committed and pushed, stage one file, commit that, stage the other file, then commit that. I'll see what I can do but it doesn't seem so straightforward. If you have any suggestions for the specific commands to use to make this happen, any guidance would be appreciated.
I'm also not sure who this is for... Is there someone who cares deeply about the cleanliness of the Git commit history? Is that a philosophy of the project or something? I don't understand why that matters.
For the long term, it would be great if we could enable commit squashing at the GitHub level, so we don't need to spend additional time thinking about how to clean up the git history. Since we're on a fork, maybe that's possible?
Will continue trying and see what I can do... The instructions here look promising: Reddit: Is there a way to remove all local commits while keeping the changes?.  This is uncharted territory for me and I fear I might mess up the branch.  You might need to plan certain things in advance if you're adding multiple changes to the same files and want to have a separate commit history. In this specific case, it should be rather simple as the change is relatively small.

For the long term, it would be great if we could enable commit squashing at the GitHub level, so we don't need to spend additional time thinking about how to clean up the git history. Since we're on a fork, maybe that's possible?

Since we're planning to raise PRs to the main repo from the same branches, I don't think it'll solve the problem. And I always appreciate having a clean and structured git history — it's helpful when you need to find certain things in it.

@curran From my experience, the easiest way to comb the history is git's interactive rebase, especially f and e commands to squash comments and update their messages.

Thanks for the tip! I started looking into that, but I don't think that would work for this case since we don't want to just squash groups of commits, but rather group files together into commits, aggregating all the changes from the whole thread of commits.
This is what finally worked:
git reset --soft HEAD~9
git reset # Required to un-stage the changes
git add packages/ts/src/components.ts
git add packages/ts/src/components/donut/index.ts
git commit -m "Component | Donut: Half donut support"
git status -s
git add . # stage the new example
git commit -m "Dev | Donut: Half donut example"
git push --force


Official TypeScript badge for READMEs

Search Terms

badge, typescript badge
Suggestion

Official TypeScript badge (just like jest has one in their README)
Use Cases

There's no official badge for projects that use TypeScript.
There's an unofficial repo which provides different options for typescript badges https://github.com/ellerbrock/typescript-badges

Would be interesting if we could get a Bᴜɪʟᴛ Wɪᴛʜ TʏᴘᴇSᴄʀɪᴘᴛ badge of some sort.

@DanielRosenwasser I let my creative side flow free and created this badge for you: https://imgur.com/bLmkPCR
Feel free to use it!

Damn that's good - it looks like it might say "BuiLt with TyreScrirt ❤", which I think kind of symbolizes how everyone sees something different in what they love about TypeScript.

What about something like this? :D

Okay, this is getting extremely competitive. I'm excited to see where the community goes with this.

 or
https://badgen.net

Can we have a special badge if the project enables strict mode? 🤗

Does anyone actually use tsc in loose mode nowadays?  Other than for legacy reasons, of course.  Enabling all the strict flags is literally the first thing I do in any new project now.

Does anyone actually use tsc in loose mode nowadays?

If you're being very nit-picky, TypeScript itself since they don't have the real 💪 strict mode enabled (strictFunctionTypes and strictBindCallApply are missing) ;)

How about a badge that is dynamically generated based upon your tsconfig.json. if it's completely strict then you get the black belt TypeScript badge?

Maybe this could be simple?

[![Strict TypeScript Checked](https://badgen.net/badge/TS/Strict "Strict TypeScript Checked")](https://www.typescriptlang.org)

Maybe this could be simple?

Looks good, only thing it's missing is the 💪

Jokes apart, some projects (including mine) use the convention “types | TypeScript” to signify that types are available. I think this is all that is necessary and we don’t need a badge specifically for projects written in TS.

I'm not really a README badges kinda person (I built my own markdown parser to get rid of them for CocoaPods) especially when it's duplicating GitHub chrome (like forks/stars/language...)
Though I see most of the value here is in showing the npm page and other derived READMEs.
Hopefully we can get npm to show types on their site like yarn does, but for now I think @AnandChowdhary probably has the right angle: showing folks they get types as they are exploring the ecosystem vs highlighting potential code quality improvements being created with TS.
Maybe this as the "official" one:

![Typed with TypeScript](https://flat.badgen.net/badge/icon/Typed?icon=typescript&label&labelColor=blue&color=555555)

Found a badge used in Microsoft own repo microsoft/fast, which is

[![TypeScript](https://img.shields.io/badge/%3C%2F%3E-TypeScript-%230074c1.svg)](http://www.typescriptlang.org/)

![Typescript](https://img.shields.io/badge/TypeScript-007ACC?style=for-the-badge&logo=typescript&logoColor=white)
Source: https://dev.to/envoy_/150-badges-for-github-pnk

@DanielRosenwasser I let my creative side flow free and created this badge for you: https://imgur.com/bLmkPCR
Feel free to use it!

I love that I came here looking for a badge and this is what I found.

Closing this issue as the request was created three years ago, and there are multiple options for a TypeScript badge.

[![Typescript][Typescript]][Typescript-url]
[Typescript]:https://img.shields.io/badge/typescript-ffffff?style=for-the-badge&logo=typescript&logoColor=blue
[Typescript-url]:https://www.typescriptlang.org/

[![Typescript][Typescript]][Typescript-url]
[Typescript]:https://img.shields.io/badge/typescript-ffffff?style=for-the-badge&logo=typescript&logoColor=blue
[Typescript-url]:https://www.typescriptlang.org/


Commenting and annotating

Add functionality for direct web commenting and annotating our jupyter book #9. See here for options. I would vote for utterances since it is required to have another account instead of Github. However, the Hypothesis overlay of annotations is very nice too.

Please modify _config.yml accordingly.

I would allow commenting only under markdown files summarizing plots in the jupyter book. Insert the following in the end
```{raw} html
<script src="https://utteranc.es/client.js"
        repo="VACILT/SSW_project"
        issue-term="pathname"
        label="utterances"
        theme="github-light"
        crossorigin="anonymous"
        async>
</script>```


TechDebt: Centralize Virus Scan

Links to Jira Tickets
n/a
Description of Changes
Move the virus scan step into the app.ts file, so that it doesn't need to be included in every endpoint that handles file uploads.
Testing Notes
Uploading/Downloading files works as before.
Can use this string (paste into a random txt file) to trigger our APIs file upload virus scans: https://en.wikipedia.org/wiki/EICAR_test_file
I've tested a few places we upload files, and they work as expected. Every file upload goes through the same field and should be handled by the same multer middleware anyways, so if one works they should all work.

@mauberti-bc I know we talked about patching the error handling as part of this PR, but on second thought there is no real overlap, so in the interest of small PRs we may as well merge this when its ready. I can just open a separate PR for the frontend changes.


After upgrading to 5.4.7 from 5.4.6, a blank page is rendered

Upgraded from 5.4.6 to 5.4.7.
On iOS, if initialPage is set, the pager view will not render anything (blank page).
It's most likely related to the change in https://github.com/callstack/react-native-pager-view/pull/462/files
Downgrading to 5.4.6 fixes the issue.

Also getting this but only in production builds.


jest.mock needs to be called for mock on system module

🐛 Bug Report
I have a __mocks__ folder setup with files like aws-sdk.js and util.js inside.
In my test file, I correctly don't need to call jest.mock('aws-sdk'), for the code to reference the mock, but I needed to do jest.mock('util') for it to access my mock of that module.
To Reproduce
Steps to reproduce the behavior:

Create a util.js file in __mocks__
Run a test that requires a file that requires util
See that the util being used is NOT the mock.
Add jest.mock('util') to the test file
See that the util being used is the mock

Expected behavior
Expected mock to be used with or without explicit mock-the-file call.
Link to repl or repo (highly encouraged)
https://repl.it/repls/OldEnchantedPublishers
envinfo
  System:
    OS: macOS Mojave 10.14.4
    CPU: (12) x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
    Memory: 64.70 MB / 16.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 8.15.0 - ~/.nvm/versions/node/v8.15.0/bin/node
    Yarn: 1.3.2 - ~/.nvm/versions/node/v8.15.0/bin/yarn
    npm: 6.13.7 - ~/.nvm/versions/node/v8.15.0/bin/npm
  Managers:
    CocoaPods: 1.6.1 - /usr/local/bin/pod
    Homebrew: 2.2.3 - /usr/local/bin/brew
    pip3: 19.1.1 - ~/.pyenv/shims/pip3
    RubyGems: <IP_ADDRESS> - /usr/bin/gem
  Utilities:
    Make: 3.81 - /usr/bin/make
    GCC: 4.2.1 - /usr/bin/gcc
    Git: 2.19.1 - /usr/local/bin/git
    Clang: 1<IP_ADDRESS> - /usr/bin/clang
    Subversion: 1.10.4 - /usr/bin/svn
    FFmpeg: 4.1.2 - /usr/local/bin/ffmpeg
  Servers:
    Apache: 2.4.34 - /usr/sbin/apachectl
  Virtualization:
    Docker: 19.03.5 - /usr/local/bin/docker
  SDKs:
    iOS SDK:
      Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
  IDEs:
    Emacs: 22.1.1 - /usr/bin/emacs
    Nano: 2.0.6 - /usr/bin/nano
    Sublime Text: Build 3211
    Vim: 8.0 - /usr/bin/vim
    Xcode: 11.3.1/11C504 - /usr/bin/xcodebuild
  Languages:
    Bash: 3.2.57 - /bin/bash
    Perl: 5.18.2 - /usr/bin/perl
    PHP: 7.1.23 - /usr/bin/php
    Python: 3.6.5 - /Users/dstein/.pyenv/shims/python
    Python3: 3.6.5 - /Users/dstein/.pyenv/shims/python3
    Ruby: 2.3.7 - /usr/bin/ruby
  Databases:
    MySQL: 10.14 - /usr/local/bin/mysql
    SQLite: 3.24.0 - /usr/bin/sqlite3
  Browsers:
    Chrome: 80.0.3987.116
    Firefox: 72.0.2
    Safari: 12.1

This is expected behavior. From the docs:

Warning: If we want to mock Node's core modules (e.g.: fs or path), then explicitly calling e.g. jest.mock('path') is required, because core Node modules are not mocked by default.

Ugh. Can’t believe I missed giant yellow warning on that page. Sorry!


Inverted bitmask for do_set_motion_velocity in case MAVLINK_MSG_ID_MA…

Inverted bitmask for do_set_motion_velocity in case MAVLINK_MSG_ID_MANUAL_CONTROL
As discussed in #32

QGC Joystick now again works.


HHH-18116 Support slow query logging for stored procedures

https://hibernate.atlassian.net/browse/HHH-18116

Unless I'm missing something, it seems to me that it be better to pass the SQL string to the constructor of ProcedureOutputsImpl and then to the constructor of OutputsImpl, instead of passing it as an argument to the executeStatement() method.

Fair point, corrected.


Publish 1.4.1 to NPM

Its missing, only 1.4.0 is present.

+1

@Rycochet and I just published 1.4.2 which has an additional IE8 fix since. Didn't publish the 1.4.1 tag since the package.json was not correctly set. Hope that's good enough for now.
That said, if you specifically want 1.4.1 then re-open this issue and we'll get that fixed.


Posting Code

Hey @scottkehler hoping you can post your code to this repo so we can see it and start to figure out the best way to deploy it in our architecture.  Let me know if there are any blockers I can help with, with respect to getting what you have posted.  It doesn't have to work or be pretty, just want to get a better understanding of what is going to be required to deploy.

Hi @franTarkenton. We had planned to have the code uploaded by Thursday for you to start working with, but I'll have Paul upload what he has today so you can get started. It is mostly complete but we aren't done testing yet.

hey @scottkehler see you posted the code... thanks that helps a lot


Sonos cast support

OS:
Win 10, 64-bit
Issue Descriptions:
Whereas chromecast audio works (thanks a ton btw!), Sonos speakers do show up as cast option on the Android app of play music, but not on the desktop player. Any chance you're planning to tackle this in foreseeable future? Thanks!
Steps to Reproduce:
Desktop player does not recognize any Sonos speakers as cast option.

@niklasjohannes
https://www.makeuseof.com/tag/stream-computer-audio-output-sonos/
http://www.streamwhatyouhear.com/download/


Show 1px height bars for silent sections of an audio file

Here's how waveforms look when there is a very quiet or silent part. Notice the blank gap:

I'd like for it to look like this for very quiet / silent segments:

How would I go about displaying small bars for silent parts of my audio?

Also see #45

@javaskript what wavesurfer params are you using?

@thijstriemstra The default parameters for the most part. I'm only using WaveSurfer to render the waveform, uploading the result to the server via ajax, pulling the .json file, and drawing it on unique canvases (not using it to play the tracks at all, I'm using jPlayer for that). I'm using a custom renderer, however, to make use of gradients and a half-height reflection. Image: https://i.imgur.com/SDJKG6U.png
Still working on improving my custom renderer, but haven't figured out how to show 1px bars for silenced portions of the waveform. It's mostly a cosmetic issue. And I believe Hark that you suggested would not work in the case of pre-rendered waveforms? I'm having the waveform .json generated via WaveSurfer on the client's end. After the rendering event is complete, I'm uploading the .json to the server.
$.get('/uploads/json/' + trackID + '.json', function(data) {     if (data) {         var peaks = data;         wavesurfer.load('', peaks);     } else {         var peaks = '';         wavesurfer.on('ready', () => {             let pcm = wavesurfer.exportPCM(1024, 10000, true);             // Do something with pcm             $.post('/waveform/json_generate.php', {                 data: pcm,                 songid: trackID             }, function(result) {                 // success                 console.log('Waveform successfully generated and uploaded to the server.');             });             document.getElementById('waveform' + trackID).classList.add('generated-waveform');         });         wavesurfer.load(trackIDURL, peaks);     }     var responsiveWave = wavesurfer.util.debounce(function() {         wavesurfer.drawBuffer();     }, 1);     window.addEventListener('resize', responsiveWave); });

would love a minimum height feature for quiet and silent parts as well

barMinHeight is available to control this case


Exit if PACKCHECK_GITHUB_COMMIT is not specified

In the CI config files, check if PACKCHECK_GITHUB_COMMIT is null and if not then exit, otherwise we may access an unstable URL and the build may not be reproducible.

@harendra-kumar Can you do this for appveyor.yml

[ ] Appveryor (@harendra-kumar)
[ ] GithubCI
[ ] TravisCI
[ ] CircleCI


In Time picker required field issue. When user select time but field is invalid on press mouse key event. On Enter event working fine.

When user select time and time text-input field also is required.
<md2-datepicker name="defaultStartTime"                           placeholder="Start Time"                           formControlName="defaultStartTime"                           openOnFocus="true"                           format="HH:mm"                           type="time"                           mode="portrait"                           container="dialog"                           required                           class="form-control">           </md2-datepicker>

@dharmeshpipariya Thanks. It is working fine


Implement transaction dependencies estimation

Similar to the Rust SDK, the TS SDK should be able to estimate the transaction dependencies (contract input set and variable outputs).
This removes the guesswork of setting the variable outputs and contract input set when performing a call.

I will try having a go at this one @digorithm

After doing some digging around this, I found this function in our codebase: addMissingVariables. It does more or less what the estimate_tx_dependencies function from our Rust SDK does, and it is automatically called by several of TS SDK's functions like sendTransaction, call & simulate. So I think the TS SDK already adds in any missing transaction dependencies for you. I also found this unit test titled 'Automatically adds variableOutputs` which, I believe, tests this behaviour.
Do you think we still need to create another function? I might be missing something so please correct me if I'm wrong 😅

We currently have this solution working by simulating the tx. @digorithm @Dhaiwat10, if is now possible to know missing dependencies ahead of time without the need to call the tx then the work, is to replace the current solution.

Unfortunately, we still have to simulate the transaction to be able to estimate the transaction dependencies.
So, is this feature live or not? I can't seem to find the documentation for it yet!

@digorithm I would say yes, but documentation around it is missing.
Our addMissingVariables function does the same things that the estimate_tx_dependencies in the Rust SDK does.

@digorithm I would say yes the feature is live, but documentation around it is missing.
Our addMissingVariables function does the same things that the estimate_tx_dependencies in the Rust SDK does.

Ah, I see. Then I'll ask of you two things: Can we rename addMissingVariables to estimateTxDependencies? Rationale: We should unify the nomenclature/terminology between the two SDKs. Please let me know what you think. The other thing: let's add this stuff to our docs; this is something users will often look for!

@digorithm

Yes, I agree with the renaming suggestion because of the point you raised, and the fact that the addMissingVariables also adds any missing external contract IDs and not just missing variable outputs.
Will add docs for this!


Issue with PermissionsEx

Description
PermissionsEx wont load on startup.
Detailed Information
[03:34:45 INFO]: PermissionsEx version 1.22.9
Stack trace -> (https://gist.github.com/YoloxPanda/8a41bb56ef52790833dc/)

You're using the wrong version of PermissionsEx for 1.8 servers.

Thank you so much! :)


Add History In 'gdx-setup.jar'

It is not bug but it would be great if you add history recently that we have create last time. For example: We have create first project and fill all field.
Next day, we want to create new project, but we should fill all of them again, besides we only want to change the title or class.
It would be great id you add dropdown that have recent history of it, and just load it when it clicked.
For Android SDK field you could detech ANDROID_HOME first if not found then add default value. It would make a lot easier to generate new project.

Are you serious...?

Yes :)
Did I say something wrong?

Nah, you didn't. I appreciate your idea of improving the user experience.
That being said, think about what saving 90% of the values means. When
creating a new project, you'd still have to change:

the app's name
the destination folder
the package name/main class name

There's no net win, you still have to modify these. I'm not sure how
Dropbox would help. Also, you'd need to provide credentials for that to
work. That's a lot of work on our end, for little to no gain.
Reading ANDROID_HOME should already be done. If not, we should add that :)
On Sep 5, 2015 10:00 AM, "johanessurya"<EMAIL_ADDRESS>wrote:

Yes :)
Did I say something wrong?
—
Reply to this email directly or view it on GitHub
https://github.com/libgdx/libgdx/issues/3396#issuecomment-137925842.

I think by Dropbox he meant a dropdown menu with the recently created projects. It took me a while to figure that out as well.
@johanessurya I didn't want to sound rude, but it really sounds like you will have a hard time actually programming the game code itself if you are too lazy to re-enter a few fields in the setup ui :)

Thank you for quick reply. I'm very appreciate that.

Reading ANDROID_HOME should already be done. If not, we should add that :)

Yes, we should. It's little bit annoying when always copy and paste path on it.

the app's name
the destination folder
the package name/main class name

I mean for beginner who just re-create like me. Hehehehe...
@nooone
Hahaha... probably. I'm really lazy person to type that over and over again.
Yes, I mean dropdown not dropbox. I'm sorry about that.

ANDROID_HOME is already set automatically... https://github.com/libgdx/libgdx/blob/master/extensions/gdx-setup/src/com/badlogic/gdx/setup/GdxSetupUI.java#L524

@nooone WOW! Thanks :)

As stated ANDROID_HOME is already set, and is the only field that should be automatically picked up.  You shouldn't be making projects that often with the same settings.


Add kitty support

With some slight tweaking, (see Step 1 of https://git.envs.net/iacore/kitty-echo-test), kni CLI can be made to have clickable actions.
This is not officially supported by kitty.
In other terminals, the special escape sequence is ignored.

I’m so very happy that you’ve found this project and gotten some use of it. And I’ve very excited about the user experience this feature would provide. However, I do not think that this kitty plugin should be used because at a glance this looks like a very powerful attack vector. I do not think I can in good conscience get behind turning click events into arbitrary shell commands.
If I’ve misread the nature of this feature or if there are mitigating considerations, please feel free to keep this conversation going. For example, if the clickable output of one program can only produce input for that same program, I might be convinced.


Align FederationConfig API with K8s API conventions

Partial fix for #879 .
NOTE: Due to the K8s API conventions doc being ambiguous over the use of pointers for optional fields, I have kept optional fields without a built-in nil value as non-pointers for now. This is subject to change pending a more formal API review.

@marun thanks for the prelim review! Pending CI passing I think this should be ready for review.

@marun I've rebased and pushed 2 new commits to make all fields required and update generated code, which I can subsequently squash. I've also opted to keep the Options.ClusterHealthCheckConfig field a pointer to call by reference into StartController() functions and other functions further down the path.

I think this moves us forward on the beta candidate release, and it looks good to me as progress.
/lgtm


[FLINK-32618][jdbc-driver] Remove flink-core dependency from jdbc module

What is the purpose of the change
Currently the jdbc bundle jar also includes flink-core and sql-client jar. However, the jdbc bundle relies on flink-core is because we use AutoCloseableRegistry in the ExecutorImpl. So we copy the class from the flink-core and remove the dependency.
I think the jdbc-bundle jar is used in the java program, which provides a client to communicate with gateway. Sql-client prodives a terminal client for users to submit their sql easily. So I think it should the sql-client relies on the jdbc, which is different from the current implementation.
Brief change log

Copy the AutoCloseableRegistry from the flink-core to jdbc module
Move the Executor from the sql-client module to jdbc module. The Executor provides a low level API to submit sql to the gateway comparing to the jdbc.
Change the dependency between jdbc and sql-client.

@flinkbot run azure

Hi @zentol. The reason we bundle flink-core is to make the tool easier to use. They can just add the jdbc-bundle dependency in their projects without detailed dependency management.
I think it is difficult for jdbc users to use flink-dist. As the example "FlinkDriverExample" shows, users only need to write java code to communicate with the running remote gateway service without building flink-dist in the local environment.
Your suggestion about the usage of the IOUtils makes me realize it's difficult to remove the flink-core... Many dependencies used by the flink-jdbc-bundle use org.apache.flink.util.Preconditions everywhere, e.g. RestClient. I think it's not possible to remove this directly.

The PR only decrease the bundle jar from 21MB to 18MB because we can not remove the flink-core easily. On the other hand, many users may use the ExecutorImpl to build their own platform. The change in this PR modifies the class package. So it's better to delay the change until we can remove the flink-core.


Webserver fails to start with symlink in global homeadditions

Is there an existing issue for this?

[X] I have searched the existing issues

Run a Diagnostic and Paste Link Here
I had been linking my .gitconfig file using ln -s ~/.gitconfig ~/.ddev/homeadditions/.gitconfig to create a symlink. This adds my up-to-date .gitconfig to my global DDEV homeadditions.
Recently, web containers no longer build and start successfully, and I get this error on every web container:

cp: '/mnt/ddev_config/.homeadditions/./.gitconfig' and '/home/kali/./.gitconfig' are the same file

Gist: https://gist.github.com/mbomb007/d8fa53fac541271ffe53bbcd5ee34e05
Current Behavior
I had been linking my .gitconfig file using ln -s ~/.gitconfig ~/.ddev/homeadditions/.gitconfig (see this DDEV issue comment here), but now it causes web containers to fail to start with this error:

cp: '/mnt/ddev_config/.homeadditions/./.gitconfig' and '/home/kali/./.gitconfig' are the same file

Expected Behavior
I expect to see the symlinked .gitconfig file copied to my Docker containers, allowing my per-user settings to be used within containers.
Steps To Reproduce

Create a .gitconfig in WSL2
Symlink to it in the DDEV global homeadditions: ln -s ~/.gitconfig ~/.ddev/homeadditions/.gitconfig
ddev config
ddev start
Webserver fails to start, showing:

cp: '/mnt/ddev_config/.homeadditions/./.gitconfig' and '/home/kali/./.gitconfig' are the same file

I then verified this is the issue by removing the symlink, removing the container, reconfiguring, and recreating the container. It works fine. But I don't understand why there's an issue copying the file to the container.
Anything else?
Gist: https://gist.github.com/mbomb007/d8fa53fac541271ffe53bbcd5ee34e05
Viewing the logs of the ddev-webserver container that failed to start for tryddevproject shows this at the end:
...
export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"  # This loads nvm
[ -s "$NVM_DIR/bash_completion" ] && \. "$NVM_DIR/bash_completion"  # This loads nvm bash_completion
+ '[' -d /mnt/ddev_config/.homeadditions ']'
+ cp -r /mnt/ddev_config/.homeadditions/. /home/kali/
cp: '/mnt/ddev_config/.homeadditions/./.gitconfig' and '/home/kali/./.gitconfig' are the same file

Hi @mbomb007 - You can't use symlinks in homeadditions... because they won't resolve when they're in the context of the container. You have to copy things in there. See the caveat in https://ddev.readthedocs.io/en/latest/users/extend/in-container-configuration/#in-container-home-directory-and-shell-configuration

Caveats:

Absolute symlinks inside a homeadditions directory won't work because they can't be resolved inside the container.

Couldn't the startup process copy the actual file that is linked instead of the symlink?

It's possible. Copying the target of symlinks can have some unexpected consequences, but I'm open to it!  Want to try a PR?

You did the right thing; existing tests should be OK. But as noted over there, you'll have to push image, I'm happy to help or  can do it for you.


WIP: Vendor score priority

This PR will make trivy-db favour vendors over NVD for severity.
Adresses: https://github.com/aquasecurity/trivy/issues/310

Why has this pull request been closed? Will it be merged at some point?

This was a WIP, we're still working on it here: https://github.com/aquasecurity/trivy-db/compare/vendor-severity-map


Need a guide for Non-validated VPN devices to be validated

As a hardware vendor, looking for a guide how to be validated by Microsoft

@Groovycho - Thank you for your question.
Can you please share with us the URL of the document that you are having issues with?
Please note that we scope issues on this repo to feedback related to the docs. If you have a product question, please post it on Microsoft Q&A or Stack Overflow. Thank you for your understanding.

@Groovycho - Can you please share with us the URL of the document that you are having issues with?

@Groovycho - Since we have not heard back from you we will now proceed to close this thread.
However if you can share additional information regarding the documentation you are referring and circle back on this thread , we will gladly reopen the issue and continue the discussion. Thank you.


VSR: Header.release

Rename Header.version to Header.protocol
To disambiguate from the "release" version.
The difference is:

(Wire) protocol version changes very infrequently.
Release version changes with every (weekly) release, and is independent per state-machine/release-infrastructure.

Add Header.release
Not all header types set a (nonzero) release. Similar to Header.view, there are enough headers that do require a release that we include it in the frame anyway.
Rationale for which messages have release == 0 vs release != 0:

Request.release is the client's release.
Prepare.release and PrepareOk.release are the same as Request.release. This is not necessarily the same as the release that the cluster is running as it commits this op.
Clients inform the cluster of their release.
The cluster informs the clients of their release. (So this can be escalated to a upgrade warning/error if necessary.)

Nonzero Header.release fields are currently set to a placeholder (1).
Ease protocol restriction for grid blocks
Even if we make a breaking protocol change, the legacy grid blocks must be handled indefinitely, since blocks are sent unwrapped over the network.

Prepare.release and PrepareOk.release are the same as Request.release. This is not necessarily the same as the release that the cluster is running as it commits this op.

This is a bit surprising, naively I'd expect prepare to use primary's version. Could you add a code comment explaining why we do it this way? Is this to leave a possibility of branching on the client version in the state machine?

Note that I should have said that Prepare.release and Reply.release match Request.release; PrepareOk.release is zero. This was a typo in the original PR description. (Fixed now though.)

Is this to leave a possibility of branching on the client version in the state machine?

That is the idea, though I'm open to switching it. If we switch it, there is still the possibility of a fallback in the form of conditionally (branched on the client version) rewriting the request body when it is converted to a prepare. (I think that the latter fallback is a little weaker than the former though.)
Another reason for setting Reply.release == Request.release is that it emphasizes that the format of the reply body must be compatible with the client's (potentially older) version, even if the cluster is ahead.
In either case, each replica already knows which release should commit each prepare based on the combination of "what checkpoint is this op within" and "what is the release version at that checkpoint" (where the checkpoint's release version will be modified by committing operation=upgrade messages).


Fast Matrix Exponentiation

Do the checklist before filing the issue:

[x] Do you have Node.js and all the relevant dev-dependencies installed?
[ ] Is this a bug fix?
[x] Is this an enhancement?
[x] Is this a feature request?

NOTE: Provide a clear and concise description of the feature that needs to be added! Or if its a bug, then provide the necessary steps to reproduce it along with screenshots.

Matrix exponentiation is an integral part of competitive programming and can be used to solve many dynamic programming questions. The aim here is to make it a bit easier, by implementing Matrix Exponentiation.
Validations-:

Should be a matrix, can be represented as array of arrays
Should be a square matrix
Power should be positive number

M.matrixExpo( [ [1,2],[3,4] ] , 2 )        // [ [7,10],[15,22] ]

Validations -
M.matrixExpo( [ [1,2],[3,4] ] , -2 )       // Error not positive power
M.matrixExpo( [ [1],[3] ] , 2 )              // Error not square matrix
M.matrixExpo( [ [1,2],[3,4,5] ] , 2)      // Error not a correct matrix

Also, matrix should contain only floats and ints and no strings.
Note- Need to implement fast exponentiation.
Links-
Fast Matrix Expo and Why

I will grab this issue.

@sateslayer you're assigned!


[flutter_tools] remove mocks from file system test

Work towards https://github.com/flutter/flutter/issues/71511

Sorry, added one more that was basically identical

Sorry, added one more that was basically identical


Update npm packages

Signed-off-by: Roberto Raggi<EMAIL_ADDRESS>Thank you for contributing to harp.gl!
Before requesting a pull request, please remember to check the following documents:

contribution guidelines
coding style

If you are adding new functionality we would highly appreciate if you can describe what is the capability you are adding and even better if you can add some examples. Please also remember to add tests for it.
CI Check
Our bots will check whether your PR can be directly integrated into the mainline. We have some internal integration tests running on the background, our bots will inform you of the next steps and someone from our team will take a look and help if needed!
And please do not forget to sign-off your commit! You can read more about DCO here. But, in short, you just need to use git commit -s or append --signoff when you are committing to the repo.
Happy contributing!

Codecov Report

Merging #1559 into master will increase coverage by 0.47%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1559      +/-   ##
==========================================
+ Coverage   60.52%   61.00%   +0.47%
==========================================
  Files         263      270       +7
  Lines       24351    24401      +50
  Branches     5490     5493       +3
==========================================
+ Hits        14739    14886     +147
+ Misses       9612     9515      -97

Impacted Files
Coverage Δ

@here/harp-mapview/lib/DecodedTileHelpers.ts
56.76% <ø> (ø)

@here/harp-omv-datasource/lib/OmvDataSource.ts
68.85% <ø> (ø)

@here/harp-omv-datasource/lib/OmvDecoder.ts
75.49% <ø> (ø)

@here/harp-mapview/lib/ThemeLoader.ts
86.58% <100.00%> (ø)

@here/harp-materials/lib/IconMaterial.ts
44.44% <0.00%> (-55.56%)
:arrow_down:

@here/harp-mapview/lib/CameraMovementDetector.ts
61.97% <0.00%> (-28.17%)
:arrow_down:

@here/harp-map-controls/lib/Utils.ts
46.15% <0.00%> (-20.52%)
:arrow_down:

@here/harp-omv-datasource/lib/OmvTiler.ts
50.00% <0.00%> (-16.67%)
:arrow_down:

@here/harp-utils/lib/Mixins.ts
76.92% <0.00%> (-13.99%)
:arrow_down:

@here/harp-utils/lib/UrlUtils.ts
84.09% <0.00%> (-4.49%)
:arrow_down:

... and 44 more

Continue to review full report at Codecov.

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


Update dependencies and workflows

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.94%. Comparing base (c85a884) to head (522bbb1).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #99   +/-   ##
=======================================
  Coverage   93.94%   93.94%
=======================================
  Files          26       26
  Lines         512      512
  Branches       71       71
=======================================
  Hits          481      481
  Misses         31       31

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


AD Password Change Breaks All Saved Connections

Issue Type: Bug
I had to change my password on my AD account and now all of my saved connections do not work. It keeps asking me to re-authenticate, takes me out to a web page to log in, after logging in it says I'm good to go, but then the whole cycle starts over once back in Azure Data Studio.
Azure Data Studio version: azuredatastudio 1.27.0 (765c7064d46253d88393e0e1ff78874b6fe8fa64, 2021-03-16T21:24:59.168Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.82GB (20.66GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

cms
Mic
0.8.0

#12637 closing for duplicate issue


add get person detail

Could you update the readme with an example :)

@dawoudt
I've created another pull request with the readme changes: #43
Could you please create a new release since the changes from the readme won't be accessible from the current pip package?


Force AAC for surround audio by command

Hi, I've been using video_transcoding for my library, and went to test other_video_transcoding and it's a LOT faster and quality seems the same to me. Thanks for making this all available.
But on video_transcoding I can use a command to force AAC even on 5.1 (--audio-format all=aac together with --audio-width all=surround). On other_video_transcoding those commands don't exist, and I couldn't find a similar one.

@maverickjl07 Thanks for using all of my tools!
And it's your lucky day, because I just implemented a --aac-only option for other-transcode last week which does that very thing. :) However, it does limit AAC surround to 5.1 channels for maximum playback compatibility.
I haven't released it yet and probably won't for a few more days. But you can download the version on GitHub and rename it to try it out.
Please let me know what you think.

Thank you so much! That's exactly it. I did a quick test and it worked!

@maverickjl07 Excellent! Thanks for the quick feedback.
I'll update this issue when it gets officially released.

@maverickjl07 I released 0.9.0 today so gem update other_video_transcoding to get the final version. Thanks again!


Functions in Javascript

Description
Introduction to Functions in Javascript
Domain
Frontend Dev HTML/CSS/JS
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

I want to contribute to this issue : Functions in Javascript
Name-Arya Gupta
Domain-Frontend with HTML CSS
Batch-16
Serial No-3172
Issue Number-3228
Contribution Approaches-What is a function, why do we need functions, function scope, what are predefined functions, arrow functions
Type-Audio

/assign


"AudioPlayer" Can't play asset song in release mode

Checklist

[X] I read the troubleshooting guide before raising this issue
[X] I made sure that the issue I am raising doesn't already exist

Current bug behaviour
It is able to play in debug mode but can not play in release mode
Expected behaviour
It should be able to play in release mode
Steps to reproduce

Execute flutter run on the code sample
...
...

Code sample

Code sample
void main() {
}

Affected platforms
Android
Platform details
No response
AudioPlayers Version
5.2.0
Build mode
release
Audio Files/URLs/Sources
No response
Screenshots
No response
Logs

my relevant logs

Full Logs

my full logs or a link to a gist

Flutter doctor:
Output of: flutter doctor -v

Related issues / more information
No response
Working on PR
no way

I did flutter run --release on Android with our example and it worked perfectly fine. Can you provide some logs and fill the missing parts in the template?


MauiAppBuilder does not contain a definition for AddAudio()...

I created a brand new MAUI Hybrid .NET 8 project. I got only this NuGet package and I only added the using and the AddAudio() line in MauiProgram.cs but I'm receiving the error, "MauiAppBuilder does not contain a definition for AddAudio()...". What am I missing?
``
using Microsoft.Extensions.Logging;
using Plugin.Maui.Audio;
namespace MauiAudioTest1
{
public static class MauiProgram
{
public static MauiApp CreateMauiApp()
{
var builder = MauiApp.CreateBuilder();
builder
.UseMauiApp()
.AddAudio()
.ConfigureFonts(fonts =>
{
fonts.AddFont("OpenSans-Regular.ttf", "OpenSansRegular");
});
        builder.Services.AddMauiBlazorWebView();

#if DEBUG
builder.Services.AddBlazorWebViewDeveloperTools();
builder.Logging.AddDebug();
#endif
        return builder.Build();
    }
}

}

What version are you using?

AddAudio only exists in v3 which is in preview. Perhaps we needed to highlight this in the readme

@bijington , thank you for the clarification. I updated to the latest preview version (was using 2.1.0) and that resolved it. I think adding that to the readme would be helpful as it was not clear to me.


--variable=systemdsystemunitdir: command not found

II am trying to install xrdp-0.9.1 and I get this error when running   ./configure PKG_CONFIG_PATH=/usr/local/opt/openssl/lib/pkgconfig
I have installed "brew install openssl"
./configure: line 12138: --variable=systemdsystemunitdir: command not found
checking for dlopen... yes
checking for OPENSSL... no

--variable=systemdsystemunitdir: command not found is not an error.

Is not an error.


FA v6 for Laravel 8

Description
I'm using Laravel 8, blade-fontawesome v1 and Font Awsome v5. Everything is good!
But... I have a designer asking for an icon from FA6 and I'm embarrassed to tell them how long that will take - first upgrading a major Laravel version!
I could, of course, hack it with an inlined SVG but I'd rather not. Is there any way we can get FA6 support on Laravel 8 with this package?
Possible implementation
I appreciate this would have to be a patch for blade-fontawesome v1 but if it's only a small change it would be much appreciated!
Thank you

Hi @mattgreenfield, I am no longer using Laravel 8, so I did not want to have to support it. The current package (at v2) should be fully compatible with Laravel 8 though, so you're welcome to fork the repository and reference this with a Composer` VCS repository in your project.
You should just be able to add ^8.0 here: https://github.com/owenvoke/blade-fontawesome/blob/eb8fc65b8770e794960eb34166e05f0b8ea19a9a/composer.json#L9

Thank you @owenvoke - this worked a treat.
The only other thing I had to do was upgrade the npm package to v6 in my projects package.json.


Axes disabled in 1.2.0

I updated to the latest version, and it has disabled axes, meaning axes no longer break anything faster than doing it barehanded. And when I found a fairy ring and tried to harvest the mushrooms, barehanded and with shears, they were just destroyed when I broke them and didn't drop anything. I went back to Habitat 1.1.9, and axes work again and I can collect the fairy ring mushrooms. The mod Jade shows me that axes are the preferred tool for breaking the fairy ring mushrooms, so I assume it's part of the same issue. With Habitat updated to 1.2.0, Jade does not show the axe as the preferred tool for any block.

I'll look into it.


tasks: Add end time and/or duration to batch jobs

This information is available with the updated_at field in the executable job object.

#815 is a requirement


Corner Radius isnt rendering correcly on Skia

<Border Padding="50" BorderThickness="1" CornerRadius="1" BorderBrush="Black" VerticalAlignment="Center" HorizontalAlignment="Center">
            <TextBlock Text="Testing" />
</Border>

Image shows avalonia on the left and uwp on the right....
Also corner radius = 2 on UWP / WPF looks better than avalonia... we may have some rounding or calculation issue.

BoxShadows PR has changed the way borders are rendered. You should test this without that PR being merged.


Limit / Page Size too small

Hey filip - thanks so much for this, this API is really helpful.
Unfortunately I noticed that the API has a small page size and does not include all sponsors.
In my case, I have 165 past sponsors and 62 active sponsors:

..but your API (https://ghs.vercel.app/v2/sponsors/mrousavy) unfortunately only includes 40 past sponsors and 14 current sponsors (so 54 in total, whereas it should've been 227 in total).
I've seen your discussion post in the GitHub community forum (https://github.com/orgs/community/discussions/3818) - but I don't think there's an official REST API endpoint for sponsors yet, right?

Hi @mrousavy 👋, and thanks for the report. The v2 of the API simply worked by scraping the https://github.com/sponsors/user/ endpoint directly, and therefore could not by itself interactively load further sponsors. I have tried using GitHub's own pagination endpoint and have implemented a draft of that in #37.
Can you try if the v3 endpoint returns what you expect? I hope it should take into account all of your sponsors now. Note that it will not consider private sponsorships, but I hope we'll be able to have a /v3/sponsors/count endpoint which would take them into account as well.
https://ghs.vercel.app/v3/sponsors/mrousavy

Amazing stuff filip - this works like a charm for me! :) Thank you!


Fix Plantronics Hub

Change type to fix  Plantronics Hub

Also add support for iMazing Profile Editor

Thanks, looks good.


⚠ Bofrost.at has degraded performance

In cee7dd8, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2374 ms

Resolved: Bofrost.at performance has improved in 08794cd after 6 minutes.


publish as a gem

Rubyfmt is a standalone script that only loads the standard library of Ruby, as such it is not packaged as a gem

Even if there are no dependencies, it would still be nice to install rubyfmt with gem install rubyfmt, rather than cloning the repository and manually adding it to your PATH. The other nice thing is you could add rubyfmt to your Gemfile and run it with bundle exec rubyfmt to make sure that everyone is using the same version.

The problem is that distributing rubyfmt as a gem implies a significant performance penalty that does not happen if it's installed in the system path correctly. I feel like it would irresponsibly encourage bad behaviour (slow command line tooling) to vendor it as a gem, as such I'm deliberately not releasing it as gem.
/cc @searls WDYT?

Use gel

Homebrew / apt / whatever package manager could work, right? So we’d still get automatic(ish) updates.

Opened PR at Homebrew: https://github.com/Homebrew/homebrew-core/pull/49775
Opened PR at ArchMac: https://github.com/archmac/packages/pull/12

@penelopezone I feel like it's not because it's distributed and packaged as a gem that it has to be always used with Rubygems enabled.
By having gem install rubyfmt work, it can be used right away with the caveat that it takes a hit on performance. It's possible to output a message upon gem install using spec.post_install_message in the gemspec file to warn about that, and instruct the user to echo "something" > /usr/local/bin/rubyfmt for awesome performance.†
To me this has the benefit of making rubyfmt extremely usable right away, with one's familiar tooling, at the cost of an openly advertised performance hit. Once the user has discovered the usefulness of rubyfmt, one can then take the next step to enhance performance at one's own discretion.
Case in point, the current make install scheme (on v0.2.0 at least) does not support the customary DEST env var and installs in ~/bin right away, which may totally not be what the user wants.
Later down the road this will also have the added bonus that you'll be able to distribute the Rust part as a binary gem.
† I'm wondering if such a script could be included in the gem with a #!/usr/bin/env ruby --disable=gem  shebang, and require_relative with or without path magic, and the user just has to ln -s it into its favorite path location.

I'd just like to pipe up with another use case here which is with code generation. I've got a gem that templates out ruby files and the generated output is valid but hideous (and I don't want to spend the rest of my days fixing all imaginable whitespace issues and whatever else). It would be nice to just be able to do something like this in-process:
require "rubyfmt"

def format(input)
   Rubyfmt::Format.call(input)
end

I'm not sure what the perf penalties are there, but they probably wouldn't be as bad as our current solution, which is to use prettierrb, which in practice means shelling out to node, which then shells out to another Ruby process to parse the source 🙈
(Also prettierrb sometimes just doesn't work for completely impenetrable NPM reasons on some people's laptops ... )

@tragiclifestories rubyfmt is very unix friendly in this manner, so you could do that with https://ruby-doc.org/core-2.6.1/Process.html. Given that rubyfmt is statically linked, it wouldn't have the same issue as the one you're describing with prettier. Rubyfmt is very fast (on the order of <20ms for small files), so this would likely be fine


KYLIN-4799 Create CONTRIBUTING.md

Proposed changes
There are documents for developers on the website. Placing CONTRIBUTING.md in the root of the GitHub repository is defacto-standard but this project doesn't. I create a CONTRIBUTING.md file and make it refer to documents on the official website of Kylin. I expect this makes other developers easy to access to development guide.
I make this PR to the master branch because the CONTRIBUTING.md file should be located in root dir in this repository. I understand the document branch only takes documents for the website. If I understand wrong, please let me know.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Pull Request Test Coverage Report for Build 6513

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.003%) to 28.048%

Files with Coverage Reduction
New Missed Lines
%

stream-core/src/main/java/org/apache/kylin/stream/core/storage/CheckPointStore.java
2
73.74%

Totals

Change from base Build 6502:
0.003%

Covered Lines:
26253

Relevant Lines:
93600

💛  - Coveralls

LGTM, thanks!


Only fire drag event when the user left clicks.

When right clicking to inspect the tab header in chrome the behavior is undesirable.  This fixes it.

thanks, will test and merge shortly

Any update on this MR?  We have some users who have expressed interest in this fix.

Looks great to me: thanks for your patience on this. @WolframHempel is snowed under with his start up and so I'm trying to get back to some of these long-standing PRs.

@mattgodbolt Thanks!  Open source software is great, but keeping it maintained is difficult in Today's world...I appreciate the help getting this merged in as well as all of @WolframHempel's efforts throughout the years!

@noah79 can you take a look at master please: I had to do a bunch of merges and while it seems like it has mostly worked, I can still repro issues by:

dragging using the left button and releasing
right-clicking to get a menu up while on a divider
left clicking to dismiss the menu elsewhere: the left-click seems to be treated as a "drop".

I'm not sure if this was an issue before or as a result of my merge.

@mattgodbolt I checked in a 'fix' to our current fork of GoldenLayout that only allows dragging with left-click.  It's essentially a single line change and would suggest rolling back the original commits and just updating it with this single line fix.

Thanks @nkoterba ; will take a look properly tonight. My first attempt at merging failed: Splitter.js refers to new Map() but I don't see where that is defined - are you using newer Javascript functionality for this? https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map suggests so, and this isn't supported on the browsers GoldenLayout supports. I'll try to work around if so.

@mattgodbolt Yes, it appears we've slipped in some ES6 by accident...we're doing most of our dev in ES6 now so it's easy to forget when working with ES5 compliant libs you can't use those things.
I wouldn't suggest or even try merging our entire fork.  I actually just went thru the process of merging in all the updates you've been making into our fork to see how things 'behave'.
But in terms of not letting users drag tabs with the right-click button, I believe the solution in the commit here https://github.com/ericvandever/golden-layout/commit/b1961252e0cab0457bc7abdafa07ecd2cb89a91b is slightly more elegant than the original proposal for this issue.
By only 'adding' the event handlers on a valid left-button you don't need all the messy checks in the other event handlers if this._timeout is null or not.
So if you desire, I'd suggest just manually editing DragListener.js to use my change above and reverting the other fix above.
NOTE: that some additional things have been added to DragListener.js (I believe to support iframe drag and drop stuff) but it was still a pretty easily manual merge.

Thanks for the comprehensive reply! I'll admit I lazily just tried a complete merge. I'll definitely look at modifying DragListener as you say.

I've done the bare minimum that seems to work for now; further comments (and/or PRs) welcomed if you feel you can improve. Many thanks again!