RouteGroup::merge adds slash to the route prefix, when parent RouteGroup prefix is empty / not set.

Laravel Version: 9.27.0
PHP Version: 8.1.9
Database Driver & Version: X

This is a follow up issue to my old one, where i miss understood the problem.
Description:
Route prefix is returned with slash at the start when parent RouteGroup prefix is empty.
For exmaple:
Route::middleware("auth")->group(function () { // parent RouteGroup prefix is empty
    Route::view("/", "page.test");

    Route::prefix("dashboard")->group(function () { // we add another group with its own prefix
        Route::view("/settings", "page.test");
    });
});

Expected behavior:
When user visits */dashboard/settings the Route->current()->getPrefix() should return dashboard
Actual behavior:
The route prefix is returned with slash at the start -> /dashboard
This only happens when parent RouteGroup prefix is empty
For example this code would work as expected:
Route::prefix("admin")->group(function () { // parent RouteGroup prefix is set to admin
    Route::view("/", "page.test");

    Route::prefix("dashboard")->group(function () { // we add another group with its own prefix
        Route::view("/", "page.test");
    });
});

When user visits */admin/dashboard the Route->current()->getPrefix() returns admin/dashboard as expected.
The main problem is that route like this wouldn't work too.
Route::group(["prefix" => "dashboard"], function () {
    Route::view('/', "page.test");
});

When user visits */dashboard the Route->current()->getPrefix() returns /dashboard
This is because all routes inside web.php are loaded using group with empty prefix in RouteServiceProvider boot method.
Route::middleware('web')
                ->group(base_path('routes/web.php')); // RouteGroup prefix is empty

Source of the problem
If i understood the problem correctly i think the issue is in the RouteGroup::formatPrefix method.
https://github.com/laravel/framework/blob/7fa4beb386a139cabb730a7e3b9571ba8ea2a181/src/Illuminate/Routing/RouteGroup.php#L64-L73
Here the $old variable  is set to empty string, when the parent  RouteGroup prefix is empty / not set.
And now when the old prefix is empty and new prefix is set.
This is what happens in the code on line 69  -> "" . "/" . "newPrefix"
There is slash added to the start of the new prefix.
I also managed to write a failing test for this issue.
$old = []; // leave array empty, this is what happens when loading routes in RouteServiceProvider
$this->assertEquals(['prefix' => 'foo', 'namespace' => null, 'where' => []], RouteGroup::merge(['prefix' => 'foo'], $old)); // add prefix to route group

Test output
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    'prefix' => 'foo'
+    'prefix' => '/foo'
     'namespace' => null
     'where' => Array ()
 )

My solution
I managed to fix this issue by wrapping the return from formatPrefix in trim.
Reference:
https://github.com/laravel/framework/blob/7fa4beb386a139cabb730a7e3b9571ba8ea2a181/src/Illuminate/Routing/RouteGroup.php#L64-L73
My fix:
return trim(isset($new['prefix']) ? trim($old, '/').'/'.trim($new['prefix'], '/') : $old, '/');

This should guarantee the prefix is always returned without slash at the start as expected.
I also tried running all the tests in the Routing folder after this fix and all of them have passed.
If my understanding of the problem is correct i would be open to submit a PR with fix and some additional test.

Feel free to send in a PR but keep in mind that none of the existing behavior for non-cached routes can change.

I just wanna leave this comment in case others have the same issue as I:
this change introduced a breaking change. before, the function $request->route()->getPrefix() returned the prefix with "/" at the beginning. Now this slash is removed.
While the new behavior is cleaner and better, it broke my application and required some refactoring.

@ruedigerm-playster did this break something for you in non-cached route behavior or with cached routes?

I think non-cached behavior. (at least I'm not using any caching explicitly)
I use prefix like
Route::prefix('my-api')->group(function () {
Old behavior: $request->route()->getPrefix() returned the prefix as "/my-api"
New behavior: $request->route()->getPrefix() returned the prefix as "my-api"

@JirakLu we're going to roll back your PR. Seems it did break something while routes aren't cached. We cannot change the behavior for non-cached routes on patch releases.


px4 console ctrl+c doesn't work directly

Ctrl-C -> Exiting... [Enter] error
Exiting...

pxh> Shutting down
/cygdrive/c/PX4/home/Firmware/Tools/sitl_run.sh: Zeile 150: kill: (8676) - No such process
ninja: build stopped: interrupted by user.
make: *** [Makefile:170: posix_sitl_default] Fehler 2

https://github.com/PX4/Firmware/issues/11027 has more explanation for the same issue.


Specifying dependencies that need to be present on developer's machine

It's not immediately clear that you need to have those components installed. Adding info to readme.

Thanks, good idea.
I'm not completely happy with the way this reads, with the install instructions before the dependencies required for installing. Perhaps rename your heading to "Build dependencies" and refer to them from the install instructions? Do you want to have a go at this change?

Yeah, no problem, can do.

How about now

Super, thanks!


Spelling fix: 'comma seperated'=>'comma-separated'

Changed  "comma seperated" to "comma-separated" in docker-options/commands and related docs.

:+1:


Strategy Documentation - Exponential Moving Average

@ta-oliver @Cal231995 I've initialised the creation of documentation for Exponential moving average strategy.
@Cal231995 please provide images and relevant mathematical formulation along with the required content to be added in documentation.

@Cal231995 I've added a pdf documentation of this strategy here. Let me know your thoughts. I need to add the mathematical formulation of this strategy in section equation. Can you provide that?

@Cal231995 I've added a pdf documentation of this strategy here. Let me know your thoughts. I need to add the mathematical formulation of this strategy in section equation. Can you provide that?

Here is the formula for Exponential Moving Average

@Cal231995 I've added a pdf documentation of this strategy here. Let me know your thoughts. I need to add the mathematical formulation of this strategy in section equation. Can you provide that?

Here is the formula for Exponential Moving Average

you can find the source here. thanks
https://www.investopedia.com/terms/e/ema.asp

I think we should add the definition of Ema first then introduce the trading strategy next.
Table of contents
1.) defintion of Ema (includes formula)
Then
2.)trading strat description
3.)rule parametrs

Defintion of ema

I think we should add the definition of Ema first then introduce the trading strategy next.
Table of contents
1.) defintion of Ema (includes formula)
Then
2.)trading strat description
3.)rule parametrs
Defintion of ema

@ta-oliver any thoughts or recommendations ? Thanks

definition of ema

Trading strat image

@Cal231995 can you also provide the default smoothing factor.

I think we should add the definition of Ema first then introduce the trading strategy next.
Table of contents
1.) defintion of Ema (includes formula)
Then
2.)trading strat description
3.)rule parametrs
Defintion of ema

@ta-oliver any thoughts or recommendations ? Thanks

@Cal231995 , @fhashim  - let's follow the same format as the other PDFs for consistency to avoid having to recreate any of the existing rules. We can change format later if we really want to, but will need to be for all PDFs.

@Cal231995 can you also provide the default smoothing factor.

This should hopefully be in the associated Python code?

@Cal231995 can you also provide the default smoothing factor.
https://www.investopedia.com/terms/e/ema.asp

@Cal231995 please review the updated documentation. I've added the equation for this rule and I've updated the rule parameters.
Please verify if the default values are correct in rule parameters. thanks.
@Cal231995 for the changes that you've requested I am not sure what to do as changes to tex file ChangeRegression & ChangeRelationship has nothing to do with this branch and documentation of this rule. If you want to view changes that I've made to existing docs you can review them in this PR.

@Cal231995 please review the updated documentation. I've added the equation for this rule and I've updated the rule parameters.
Please verify if the default values are correct in rule parameters. thanks.
@Cal231995 for the changes that you've requested I am not sure what to do as changes to tex file ChangeRegression & ChangeRelationship has nothing to do with this branch and documentation of this rule. If you want to view changes that I've made to existing docs you can review them in this PR.

@fhashim I've only requested for the addition of mathematical formula. I think it's okay now. just need to confirm the lenght we used as deafult. I know it's 50 not 14

@bi-kash need your help for confirmation. thanks

@bi-kash need your help for confirmation. thanks

@Cal231995 - the default value is in the function argument of the added function. @fhashim can help show you where this is in allocations.py
If you need a second opinion on the Python, @NikolaR01 can assist.

@Cal231995 as per allocations.py currently it is set to 1. @bi-kash Can you also confirm this? thanks.

@fhashim made the necessary update for default values. Please refer to this:
https://github.com/ta-oliver/infertrade/pull/161/commits/715b5e0fbb9dfa2498d08b0d69d07cf39602fa7e

@bi-kash thanks for updating.
@Cal231995 I've updated the doc with default window size. Please review the doc. If there are any changes required please suggest.

@bi-kash thanks for updating.
@Cal231995 I've updated the doc with default window size. Please review the doc. If there are any changes required please suggest.

The last file is ok. also, window size is now 50. this is good to go.

this is good to go.

N.B. you are a reviewer, Cal, so you can approve the PR and merge if all is good.

Exponential Moving Average (EMA) is a type of moving average similar to simple moving average, but it reduce the lag by applying more weight to recent prices. The weighting applied varies depending on the number of periods use in the moving average. Traders used this more to get a clearer view of the most recent price change of an asset.
@fhashim you can used this definition of ema. I composed this myself. Thank you


Underlines and links

Is your feature request related to a problem? Please describe.
We have a few (known) places where the universal lack of underlines in links is causing confusion.  The first is in the footer text -- if the site is configured with a link that link has no indication it is clickable.  For instance in this screenshot "Giving" is a link but you would never know:

Is your feature request related to an existing functionality? Please describe.
There are existing links across Hypha that appear the same as non-link text; this applies to all of them.
Describe the solution you'd like
A lot of links in Hypha would be more legible with an underline.  For instance clickable elements in the submissions table, and even in-page navigation such as "Show All".
Might we consider making underline the default for all links and explicitly removing the underline for .button based links (e.g. the log out button)
Describe alternatives you've considered
Color alone doesn't do the trick since (A) not everybody can see color and also (B) the blue is used in other places on non-clickable text.
We could instead apply underline just to identified concern points, but I'm worried that will create inconsistency.
Priority

Medium (slowing you down)

Affected roles

Staff
Applicants
Reviewers
Partners
General public

Ideal deadline
We'll make this change locally on our own timeline, so no major urgency.

The original designer was allergic to underlining I believe :-). As a developer I rather like them, from an accessibility stand point they are good as well.


Change default applications

Hi :D i would like to change default applications to launch files from PCManFM or CLI with open command. for example i would like to open all of my videos with mpv player instead of MPlayer. how can i do that ?

No idea what "open" is, but for me it is working using right-click on a file, "Open With" > "Open With" > selecting the tool and checking the box to keep this as a default. Isn't that what you aim for?

@boredland
i meant xdg-open, for example , open myVideo.mp4 will open it with MPlayer. i found what you said and it works, but there should be a config for managing this, what if i remove PCManFM? how do i set to open a file type with a specific program then ?

So you want a dedicated tool to configure xdg mime, am I getting this right? In the past I always used the file manager for that, so I have no particular ideas...

@boredland that's exactly what i'm looking for, thanks! :D

this would be a gui editor for this: https://aur.archlinux.org/packages/mime-editor


Separate table/model for friendship blocking records

friendship table contains too much information, it would be nice if some of them are decoupled.
I think it'd be a good idea to make a separate table/model for records for friendship block/unblock, with blocker_id and blockee_id.

If we have a blocker_id we don't need a blockee_id. We could also just add a "blocked" and "bocked_by" status; according to whom blocked the relation.

You're right. We don't need both blocker_id and blockee_id. Plus, they sound funny. Maybe the new table blocked_friendships could have friendship_id foreign key and blocker_id, and blocker_type to make the polymorphic association complete.

I have been thinking about it and it seems more logical to me to keep the same convention you used for the friendship request. That way we keep the table cleaner. I think we should add some validations too

What convention do you mean? Do you mean that we keep the current structure?

Just like for a request we have "requested" and "pending", we can do the same for blocked records: for example we can have "blocked" and "blocked_by"

That's a reasonable approach. We could add statuses like was_blocked and did_block. I think this is not an urgent feature because current setup works just fine. This is something to consider in the future improvement. Let's sit on it and wait for more input from the others.

By the way your PR was very helpful. Would you like to be a collaborator for this gem?

Sure:) It would be fun to keep collaborating in this project


⚠ Samsung has degraded performance

In b0cb0bc, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2474 ms

Resolved: Samsung performance has improved in 5315a96.


Cargo install failed

I also faced the same issue while installing icx-proxy locally. You may need to install pkg-config and openssl to get rid of these errors:
brew install pkg-config
brew install openssl

Had the same issue, found the solution here.


 So there are python-base:alpine and python-base:debian (== python-base:latest), for example.
debian-base and alpine-base images become base:debian/base:latest and base:alpine. I thiiiink the only thing using these images is the FreeSWITCH image.
The only change to the pypy/python/python3 images is the addition of :debian tags (before those were just :latest).

:+1:


[FEA] Sync tools supported ops with plugin

Is your feature request related to a problem? Please describe.
As of Jul 16, 2024:
**supportedDataSource.csv (FROM TOOLS TO PLUGIN)**
**supportedExecs.csv (FROM TOOLS TO PLUGIN)**
Row is removed: MapInArrowExec, S, None, Input/Output, S, S, S, S, S, S, S, S, PS, S, NS, NS, NS, NS, PS, NS, PS, NS, NS, NS
**supportedExprs.csv (FROM TOOLS TO PLUGIN)**
Row is added: HiveHash, TNEW,  , None, project, input, S, S, S, S, S, S, S, S, PS, S, NS, S, NS, NS, NS, NS, NS, NS, NS, NS
Row is added: HiveHash, TNEW,  , None, project, result, NA, NA, NA, S, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NS, NS
Row is removed: EphemeralSubstring, S, substr; substring, None, project, str, NA, NA, NA, NA, NA, NA, NA, NA, NA, S, NA, NA, NS, NA, NA, NA, NA, NA, NS, NS
Row is removed: EphemeralSubstring, S, substr; substring, None, project, pos, NA, NA, NA, S, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NS, NS
Row is removed: EphemeralSubstring, S, substr; substring, None, project, len, NA, NA, NA, S, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA, NS, NS
Row is removed: EphemeralSubstring, S, substr; substring, None, project, result, NA, NA, NA, NA, NA, NA, NA, NA, NA, S, NA, NA, NS, NA, NA, NA, NA, NA, NS, NS

HiveHash needs to be handled
We need to fix parse_url https://github.com/NVIDIA/spark-rapids-tools/issues/1190 . Also, it will be interesting why did we miss the changes of the parse_url in the plugin side.

Also, it will be interesting why did we miss the changes of the parse_url in the plugin side.

I checked the supported files in tools and plugin. We didn't miss this. The supportedExprs.csv file was not updated in plugin side.


Create api for DB

Insert new business
Get questions
Insert answers
Get resources
Get aggregate responses

by topic
by question
by business sector or type

More?

done


Can't open files Error: open out/[...] no such file or directory

Buona sera!
I'm trying to encrypt a file and it isn't working, even after building and using the binary, am I doing something wrong here?
pedro@Pedros-MacBook-Air stegosecrets % go run cmd/stego/stego.go encrypt --file ./README.md -p 5 -t 3
Encrypting './README.md'
Error: open out/./README.md.key: no such file or directory

Ciao! Yes, the command is correct but it's assuming to have an existing out directory where to store the files.
This is a bug, we should at least create the folder if it doesn't exists.

What do you think about using something like this function to create the folder?
https://github.com/enrichman/stegosecrets/pull/20/files#diff-09669fc96c318226e77e7d95f799a0b3e1f8bbf99f4fbee778b1237a58373e9fR58-R63

Fixed with #22


How to install it?

How to install it?

Oh actually this is a private project. I had issues fetching it on a remote device thats why i made it public. I will make th public release soon. You can email me<EMAIL_ADDRESS>


Review Bundle Export Functionality

Provide a clearer option for users to select all tags available in an AOI for the PBF file format during the export set up process. At the moment this is available through selecting the POSM Bundle on the Summary tab.

@bdon we need to change the wording for 'Bundle for POSM' as previously discussed. Many of our users do no know what POSM is and are missing the opportunity to use its functionality to export all available data within their AOI in the OSM .pbf file format. This should perhaps be re-worded to 'Export all OSM data - this will be provided for the AOI as a .pbf'

@mataharimhairi "Bundle for POSM" previously created "unfiltered" PBF exports but with the new code all PBFs are unfiltered. So this may not be necessary at all.

The "Bundle for POSM" and "Unfiltered" are now two separate checkboxes.

@bdon can we change the text to 'Unfiltered files - all OSM data for PBF, MWM, OBF and IMG formats'


added chunk_every function

chunk_every
By passing a $size, $step and whether to $discardIncompleteChunk you have full control over the chunks created.
chunk_every([1, 2, 3], 2); // [[1, 2], [2, 3], [3]]
chunk_every([1, 2, 3], 2, 1, true); // [[1, 2], [2, 3]]

Inspired by Elixir

Closing because of inactivity


Swap retry & timeout examples

These two examples are permuted:
https://github.com/slok/goresilience/blob/master/examples/retry/retry.go
https://github.com/slok/goresilience/blob/master/examples/timeout/timeout.go

Hi @mdouchement
I have mistaken on a rename of the examples or something :/
Many thanks and sorry for the error!


can't extract prompt object but only remove background

I have install all need packages and have not error in console, but no matter what prompt i add, it only remove background. The tile division BG Removers and cascadePSP I set to disable.

I am having this same issue.  It seems to be ignoring the prompt entirely.  I tried all 3 models but there was no difference.  I'm running on windows 10 and theres no output from the automatic1111 window.  Is there any log or something i could post that could help debug this?

@dkojevnikov thank you for the response, it was not clear to me from the model card that post processing was required.  with a post processor enabled changing the prompt does change what it removes, but it does not work very well as you say.  I was only able to get it to extract the object I wanted once, the rest of the time it either removed the entire background, none of it, or only the foreground

I'll fix it as soon as I can.


Release process and the master branch

Hi,
I'm trying to understand the release process of leveldb.
should the master branch only contain safe to use and vetted code or is it not recommended to use the master branch and only use released versions?
Is there any vetting/checking/testing that happens on releases that makes it "safer" than master?
Thanks,
Elichai.

Hi @elichai:
master should be treated as "releasable" for every commit. We run all tests on all platforms for every commit to master before landing a change. Releases get no additional testing - at least not in this project. Other downstream projects, e.g. Chromium, run tests that have the potential of uncovering bugs in leveldb, but it's been a while since that happened.

Closing as answered. Reopen if you have any follow-on questions.


Updated SPIFFS to 0.3.6

As @pellepl just updated SPIFFS to 0.3.6 I saw it fitting to update the submodule in EOR. I currently  have no HW to test on though.

Don't fret Johan, it's backwards compatible. Hope you get back to a working
adapter soon 😉

I have verified. Both examples(spiffs, posix_fs) work ok.

Thanks @sheinz!


Single step does not work on centos 7 w/ 32bit python

From @jhsterling on October 24, 2017 17:18
Environment data
VS Code version:  1.17.2
Python Extension version: 0.7.0
Python Version:   2.7.13 :: Anaconda, Inc.
OS and version:    CentOS Linux release 7.1.1503 (Core)
Actual behavior
When debugging if F10 or single step is pressed the debug session is lost
Expected  behavior
single step
Steps to reproduce:

Start a debug session with stop on entry == true
press F10

Pressing break causes nothing to occur
Pressing stop ends the debug session
All others are grayed out
works with 64 bit python

Logs
Output from Python output panel

Output from Console window (Help->Developer Tools menu)

Copied from original issue: DonJayamanne/pythonVSCode#1338

works with 64 bit python

If this is the case, then under what config doesn't it work. I assume it doesn't work in 32bit python.

From @jhsterling on October 26, 2017 0:39
Right it does not work in 32bit python on Centos 7.  I will try it on Ubuntu tonight.

From @jhsterling on October 26, 2017 1:14
Same behavior on Ubuntu.  Using a debugger (pdb i assume) via ipython works correctly.

From @ancillaryCode on November 7, 2017 18:22
I get the same issue on Debian 32 bit. When I run strace on the Python process, it's waiting on a futex.
$ strace -p 3906
strace: Process 3906 attached
futex(0xc5df40, FUTEX_WAIT_BITSET_PRIVATE|FUTEX_CLOCK_REALTIME, 1, NULL, 0xffffffff

I have the same issue on Ubuntu 32bit.
VS Code Version 1.20.0
Commit c63189deaa8e620f650cc28792b8f5f3363f2c5b
Date 2018-02-07T17:11:23.959Z
Shell 1.7.9
Renderer 58.0.3029.110
Node 7.9.0
Architecture ia32
Python Extension Version: 2018.1 (01 Feb 2018)

@jhsterling @goranpetrovikj
Please could you try our experimental debugger. You can find further information on this here:
https://blogs.msdn.microsoft.com/pythonengineering/2018/03/08/python-in-visual-studio-code-feb-2018-release/

@DonJayamanne The experimental debugger works!

Closing as the experimental debugger resolves this issue and the experimental debugger is the default.


IPv6 AAAA records don't connect, while raw [::] addresses do

What works: coap://[fd01::dead:beef]/abc/def, what doesn't work: coap://lamp01/abc/def, where lamp01 resolves to fd01::dead:beef on a AAAA record.
I had to manually add
reqOpts.agent = coap.globalAgentIPv6;

to function _makeRequest(msg) for the DNS record to resolve.
I fear this is a limitation in the coap library though. If so, I'll direct my issue there. However I think it would imply a major interface redesign there, and maybe the node-red interface should expose a workaround -- somehow.

Have you been able to further investigate this issue, @rubdos? I updated the package's dependencies, maybe this solved the problem. If not, let me know :)

Hey @JKRhb, we couldn't have this lab session at the uni this year because of COVID... We did remote Zigbee stuff instead. I hope to have it back next year though, which would be November 2021.
@thielemans, do you know by any chance whether the dependency bump would have helped here?

Hey @JKRhb, we couldn't have this lab session at the uni this year because of COVID... We did remote Zigbee stuff instead. I hope to have it back next year though, which would be November 2021.

Fingers crossed! :)
Regarding the issue, I noticed that there have been similar problems in other packages that have been using the default udp4 agent. Similar to the addition in #17, I would add a checkbox in the request node that lets you choose between the two types of agent. If the checkbox is ticked (which should be the default, I guess) then the line
reqOpts.agent = coap.globalAgentIPv6;

will be included. Do you think this could be a working solution?

Do you think this could be a working solution?

I'm not 100% sure; if the globalAgentIPV6 has no A-record fall back, then that change will probably break the plugin for all the V4 users out there that use DNS.
Besides, this issue should probably be resolved at https://github.com/mcollina/node-coap and not here, now I think about it. DNS and IP is a lower layer than what this package should provide, but I'll have to dig deeper in the source code for that.
By the way (re #17): binding on [::1] doesn't guarantee a hybrid/multi protocol socket. I think that binding on [::] yields a v4/v6 hybrid socket on some Linux systems, but that's distro-dependent iirc.
Thanks for coming back to this, sorry that I'm currently not very helpful!

Thanks for coming back to this, sorry that I'm currently not very helpful!

No worries, thank you for your helpful feedback and for raising this issue in the first place! I will try to investigate this problem further in the next days, especially when it comes to the potential breaking change the switch to globalAgentIPV6 might be. However, I've seen a similar problem in other packages where a distinction between V4 and V6 URIs was added to ensure continued support for the former. Maybe this could be a solution (or rather: a workaround) for this package as well.

By the way (re #17): binding on [::1] doesn't guarantee a hybrid/multi protocol socket. I think that binding on [::] yields a v4/v6 hybrid socket on some Linux systems, but that's distro-dependent iirc.

You are right – maybe it would be more accurate to label the option as "Use UDP6 socket" and turn it off by default?

I just opened #25 as a potential fix/workaround.

Hmm, I just noticed that a distinction between IPv4 and IPv6 hostnames is already happening in node-coap (see here, making the changes in #25 unnecessary. Therefore, this issue should actually be resolved there if it still persists, I guess.

The COVID situation finally allows us to give this lab again on-campus, which means that I'm again bouncing on this.
So, it appears that the way node-coap exposes the API, i.e., via an Agent that creates a socket before sending a packet, this issue needs to be worked around. node-coap should provide an API that only creates the UDP socket after resolving the DNS name. The reason for the API being exposed as it currently stands, is because node-coap seemingly wants to (incorrectly, IMO) unify a listening socket and a sending socket under the Agent abstraction. I have filed https://github.com/mcollina/node-coap/issues/320 to discuss this. I find this extra painful, because NodeJS is a very dynamic thingy where I would expect that connecting to a udp://-style socket with hostname already has all these things built-in.
I suggest we merge #25;, I can confirm after tomorrow 3pm CEST whether that patch is sufficient as workaround.

Thank you for the update on this issue and filing mcollina/node-coap#320! I updated the workaround in #25 and hope that works for now. Otherwise I totally agree with you that this should be handled on a lower level and I am looking forward to the discussion over at node-coap!


The queue does not accept NIL value.

 Instead of store elements of buffer directly to the queue directly, I just store indices of the buffer. One issue I have with the queue is, push() and try_push() do not accept a zero value which is the first index of the buffer.
I want to know if there is any way to make the queue accept a zero value.
Thanks

Queues for atomic elements require one element value to be reserved for NIL. By default it uses a value initialized element (0 for fundamental types). You can specify a different value for NIL as the template argument. E.g. if your element type is unsigned, you can specify static_cast<unsigned>(-1) as NIL value, as long as your indexes are from a limited range [0, N], where N < static_cast<unsigned>(-1).
Alternatively, the queue versions for non-atomic elements do not require reserving an element value.

Your advise is really helpful. Thank you.
Actually I have just tried your lib some days ago. I did not know the NIL value can be customized.
By the way here, I have a question does not relate to your lib, but I think you may give me an idea.
So far I often use two queues to do the producer-consumer problem, one plays as a pool and the other plays as a product queue. The code is like below:
  `constexpr` auto SIZE = 10;
atomic_queue::AtomicQueue<int, SIZE, -1> qu_producer, indices_pool;
struct BigRecord {
	int data;
};
BigRecord sharedBuffer[SIZE]{};

for (auto i = 0; i < SIZE; ++i)
	indices_pool.push(i);

std::thread producer([&] {
	while (bAppRunning) {
		int item {};
		if (indices_pool.try_pop(item)) {
                            // fill data
			sharedBuffer[item].data= {rand()};
			fmtLog::trace("producer: {}", sharedBuffer[item].data);
			qu_producer.push(item);
		}
	}
	});

std::thread consumer([&] {
	while (bAppRunning) {
		int item {};
		if (qu_producer.try_pop(item)) {
			fmtLog::trace("consumed data: {}", sharedBuffer[item].data);
			indices_pool.try_push(item);
		}
	}
	});

`
The struct BigRecord here is just an example, in fact it may contain more complex data items. My question is
How can I avoid using the additional indices_pool but still able to use the queue as a "atomic queue" with cheap "push" (without copying big data)?.
Thanks.

If you only have 1 producer and 1 consumer thread, then a single-producer-single-consumer ring-buffer with a separate commit functions is the most efficient solution for your problem. It is wait-free and 0-copy. E.g.:
#include <atomic>
#include <thread>

class ProducerConsumerRange
{
protected:
    std::atomic<unsigned> free_beg_{0};
    std::atomic<unsigned> pend_beg_{0};

public:
    struct Range {
        unsigned begin, end;
        bool empty() const noexcept { return begin == end; }
        unsigned size() const noexcept { return end - begin; }
    };

    void produced(unsigned len) noexcept { free_beg_.fetch_add(len, std::memory_order_release); }
    void consumed(unsigned len) noexcept { pend_beg_.fetch_add(len, std::memory_order_relaxed); }

    Range range() const noexcept {
        return {pend_beg_.load(std::memory_order_relaxed), free_beg_.load(std::memory_order_relaxed)};
    }
};

struct BigRecord { int data; };

class ProducerConsumerQueue {
    ProducerConsumerRange range_;

    static int constexpr SIZE = 1024;
    BigRecord buffer_[SIZE];

public:
    BigRecord* back() noexcept {
        auto r = range_.range();
        return (r.size() < SIZE) ? (buffer_ + (r.end % SIZE)) : 0;
    }

    void push_back() { range_.produced(1); }

    BigRecord* front() noexcept {
        auto r = range_.range();
        return r.size()  ? (buffer_ + (r.begin % SIZE)) : 0;
    }

    void pop_front() { range_.consumed(1); }
};

int main() {
    ProducerConsumerQueue q;
    constexpr int N = 32;

    std::thread producer([&] {
       for(int i = N; i--;) {
            if(BigRecord* r = q.back()) {
                r->data = i;
                q.push_back();
            }
        }
    });

    std::thread consumer([&] {
        for(bool stop = false; !stop;) {
            if(BigRecord* r = q.front()) {
                stop = !r->data;
                q.pop_front();
            }
        }
    });

    producer.join();
    consumer.join();
}

Thanks for your detailed answer.
Your example seems to be working, but my queue, however, needs one producer but allows multiple consumers to access concurrently. Regardless of that, there are still some issues with your solution.The first is your queue has no mechanism to maintain the emptiness or fullness of the queue. Second, since the producer and consumer operate in parallel without checking fullness and emptiness, at one point, two ones may overlap the head/end of each other.
Additionally, I have just detected one issue with your atomic_queue, I will post it on another thread to avoid confusion for other people.

In the future, you may like provide full details of your problem and code samples that corresponds to your use case. Not everyone can read your mind.


recipe for target 'webcam_face_pose_ex' failed - I'm on Ubuntu 16.04 64 bit

//usr/lib/x86_64-linux-gnu/libgstreamer-1.0.so.0: undefined reference to `g_type_check_instance_is_fundamentally_a'
//usr/lib/x86_64-linux-gnu/libgstreamer-1.0.so.0: undefined reference to `g_type_class_adjust_private_offset'
collect2: error: ld returned 1 exit status
CMakeFiles/webcam_face_pose_ex.dir/build.make:122: recipe for target 'webcam_face_pose_ex' failed
make[2]: *** [webcam_face_pose_ex] Error 1
CMakeFiles/Makefile2:3471: recipe for target 'CMakeFiles/webcam_face_pose_ex.dir/all' failed
make[1]: *** [CMakeFiles/webcam_face_pose_ex.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2
I couldn't find a solution for this. Everything else compiles, it just breaks down at 98%.

Those errors aren't part of dlib.  You must have installed OpenCV wrong or one of OpenCV's dependencies is messed up, maybe part of libgstreamer.

I compiled that example separately from the examples directory and it worked. Any idea why?

No idea.  What folder you are in when you compile shouldn't matter at all.


TTL command returns true instead of time

Hi,
redis.ttl(key)

returns true, while it should return the time to expiration according to the redis documentation.
Is there a way to get the remaining time ?
Thanks.

All Redis commands in Node.js are asynchronous which means you'll have to catch the result in a callback:
redis.ttl(key, function(error, ttl){
   console.log('TTL is', ttl)
});

Hi,
Yes, sorry, I just figured it out a few minutes after posting this issue. That is why I closed it immediately.
Thanks for your answer anyway.


Improve developer experience

The first time the project is built, opening the application in the browser till look like this:

By tracking the ClojureScript files directly this is avoided, and it also boosts speed slightly because wait time for transpiling is entirely avoided.
I've also removed the now deprecated TAILWIND_MODE (since Tailwind version 3.0.9).

I've created an issue to draw attention to the PR.

This is still an issue as far as I know.

Thanks for the PR. I updated deps and also included some ideas from this PR. If we would change the content in tailwind.config.js to only "./src/main/**/*.cljs" this would possibly break applications that use shorthand hiccup syntax for classes :div.p-4 that is why the cljs-runtime is included there as it already has strings instead.
Thanks again and sorry that it took so long.


[caat-2697] Add DontCares to undriven ports

[caat-2697] Add DontCares to undriven ports

@davidbiancolin can this get review please

Closing, this has been superseded by PR #9


How to exclude images auto backup in Android?

I am trying to exclude cached images to be excluded from android auto backup feature? according to the documentation
we can define    <exclude domain="file"  path="string" /> but I am not sure what path this should be?  for database simply I used, <exclude domain="database"  path="mydatabase.db" />  becasuse database is installed on a specific location looks like. how about images? do you have any check or experience with that?

Auto Backup excludes files in directories returned by getCacheDir(), getCodeCacheDir(), or getNoBackupFilesDir(). The files saved in these locations are only needed temporarily, or are intentionally excluded from backup operations.

FFImageLoading uses cache folders by default (it is configurable though) - those folders aren't usually included into any backups.


[Compatibility] CompactUI Mod

Red Warnings still appear with CompactUI installed.

Opposite issue for Compact UI #5

Fixed as of Compact UI v3.0.1

Thanks for the report. I see it's already been fixed, so I'll close this issue.


Identity Management Service

This will be a very useful feature. Looking forward to this.
Please add any description or ideas or expected flow/outcomes on this issue.
Thank you

@brtrvn Please share your ideas on this, so that we can also comment on those. I hope this is for tenant users, and not saas system users.
@PoeppingT FYI

Is the idea from the below workshop included in this issue?
https://catalog.us-east-1.prod.workshops.aws/workshops/eba791b1-742c-4899-88bb-71ff70f2eedc/en-US
The admin UI with two identities will be super useful - the System User Service (users of the control plane) and the Identity Service (for tenants - to onboard tenants with initial admin user)
@brtrvn @PoeppingT


Union type safety

Description
graphqlgen doesn't enforce type safety for union types.
Steps to reproduce
Schema:
union Availability = EstimatedAvailability | RemainingAvailability
type EstimatedAvailability {
    timestamp: Int!
}
type RemainingAvailability {
    remainingItems: Int!
}
type Offer {
    availability: Availability!
}

Model classes:
export type Availability = EstimatedAvailability | RemainingAvailability
export interface EstimatedAvailability {
	timestamp: number
}
export interface RemainingAvailability {
	remainingItems: number
}
export interface Offer {
	availability: Availability
}

Expected results
export namespace OfferResolvers {

  export type AvailabilityResolver = (
    parent: Offer,
    args: {},
    ctx: Context,
    info: GraphQLResolveInfo
  ) => Availability | Promise<Availability>;

  export interface Type {
    availability: (
      parent: Offer,
      args: {},
      ctx: Context,
      info: GraphQLResolveInfo
    ) => Availability | Promise<Availability>;
  }
}

Actual results
export namespace OfferResolvers {

  export type AvailabilityResolver = (
    parent: Offer,
    args: {},
    ctx: Context,
    info: GraphQLResolveInfo
  ) => {} | Promise<{}>;

  export interface Type {
    availability: (
      parent: Offer,
      args: {},
      ctx: Context,
      info: GraphQLResolveInfo
    ) => {} | Promise<{}>;
  }
}

Versions

graphqlgen: 0.4.0
OS name and version: Node 8.11.2, NPM 6.4.1, Windows 10, 64 Bit

This is related to https://github.com/prisma/graphqlgen/pull/149


Code Quality: Reorganze namespaces

Description
Reorganize namespaces. For more information, see Details of Changes
Motivation and Context
I found that there're a lot of namespaces that need to be reorganized and there're namespaces that do not need to be separated and I merged them into the same.
Details of Changes

Previous
New
Rationale

Files.App.Controllers.IJson
Remove
Unused

Files.App.ValueConverters
Files.App.Converters
Same

Files.App.AppModels
Files.App.DataModels
Same

Files.App.CommandLine
Files.Backend.CommandLine
Can move to Backend

UserControls/Pane/PreviewPane
UserControls/PreviewPane
No need to be another folder

UserControls' ViewModels(Files.App.ViewModels.*)
Files.App.ViewModels.UserControls
Organized

Validation

[x] Built and ran the app
[ ] Tested the changes for accessibility

Ready to review :)

@onein528 could you look into the conflicts?

@d2dyno1 are you okay with these changes?


fix: send a dlog proof for h1 wrt h2

Prove that the order of h1 and h2 are equal for the Zksetup. The previous proof only proved that h2 is in the subgroup <h1>. This bug was reported in this blog post for another implementation.
Update the max message sizes to accommodate the new proof.

There are 1 or 2 dangling comments but I might not be available before we need to merge, so I'll just approve and leave it to you to merge at your discretion. Worst case you can merge as-is and we can open new issues for any comments that demand more work.

Sure, I'll merge this, and let's discuss if we can improve the design more next week.


write_protected set to true because get_active_comp_upgrade returns F, which shouldn't be a possible value?

I've been trying to find out why all the (not manually set to write-protected) repos on the sytem i'm working on are being shown as write protected and show up as such every time i start abapgit again despite changing it in local settings.
As far as i can tell

the setting of write_protected to true happens in METHOD zif_abapgit_persist_repo~list.
because the case where write_protected is false and zcl_abapgit_factory=>get_environment( )->is_repo_object_changes_allowed( ) = abap_false happens
because in is_system_changes_allowed lv_is_upgrade is set to true for ??? reasons
sy-subrc after calling UPG_GET_ACTIVE_COMP_UPGRADE is 0
ls_upginfo-putstatus is F, which is not listed in the possible values (value range of UVERS-PUTSTATUS) at all

does anyone know what the F stands for / how to proceed here?

thanks! that helped


Bug: Furnace is not correctly loaded from URL

Reproduce by changing the furnace from electric to steel/stone then reloading webpage. The furnace icon will be a black box (because of line 156). The steel/stone furnace will not be used in any calculations (because of line 145).
Fix:
settings.js, line 144
furnace = settings.furnace should be furnace = {name: settings.furnace}
or (but I prefer the one above)
the two furnace.names on lines 145 & 156 could be just furnace

Fixed in 8330172f5f244d9248824a7f586b4c310f95aec8.


Check localisation params

Generated XML has
                    <PartText name="Minutes_dc23" alpha="255" height="203" pivotX="0.5" pivotY="0.5" width="189"
                        x="214" y="7">
                        <Localization calendar="GREGORIAN" />

but timeZone is required in WffDsl
                            partText(height = 203, width = 189, x = 214, y = 7) {
                                localization(calendar = "GREGORIAN", timeZone = "SYNC_TO_DEVICE")

Do you think SYNC_TO_DEVICE should be the default value?


Some "esotic" operation

I am just courious if It could be tested at some point some operation that It Is not trivial on how it could be expressed in numpy like distance tranform or connected components.
I don't know if it can have any interesting implications in this
exploration.

That's not a bad intermediate test case, but will require a fair bit of basics to be right (but I don't see any fundamental issues).
My next step is to model the numpy ufunc abstraction in the numpy MLIR dialect. There are a few ways to do it, but I'm thinking of having a ufunc_call op which references a module level ufunc (which could be of a few different types) that holds the metadata about the operation.
Since the ufunc is such a core abstraction within numpy, and since it covers so many of the fundamental operations, it makes sense to me to spend some time to get it modeled isomorphic to how numpy reasons about it (at least modeling a proper subset of cases).
Beyond that, for your example, we would need:

Indexing/slicing ops
Some 'constructor ops' (zeros, etc)
where
dot

I'm trying to model it as isomorphic to numpy as is reasonable, since that is part of the MLIR philosophy and makes the tracer/compiler really-really thin, leaving the hard work to the compiler infra, which (in theory) is where we have the best tooling.
I'm going to try to get a first draft of the numpy dialect ~this week. I've been thinking on the design of it, but the limited time I've had to actually work on it has been swallowed up by setup/build issues (which I think I'm past now).

Thanks.
Just to share with you something more is that you know custom ops are going to be very problematic to maintain.
If you take a look at https://github.com/tensorflow/addons/issues/1752#issuecomment-621322109 that is related only to activations but it is quite common to have these kind of issues on more "esotic custom operations" operators that could frequently land be proposed for Addons and other SIGs repo (as the barrier is lower than TF core).

FYI - we've created an official repo for this now: https://github.com/google/mlir-npcomp
It won't let me transfer issues across orgs. Would you mind re-filing the issue there so you preserve authorship?
Regarding custom ops - I agree with the need, and I think the issue on the TF side comes down to the mechanism they use for this being very fragile and non-scalable. There are a lot of patterns to employ here that will be both more usable and full featured.


Safe MSSQL Queries

Hi,
Please assist/suggest.
Following sqlsserver.go we are working on a custom query plugin.
Here, I am passing db name into the query as a variable from telegraf.conf.
Example:
query3 := SELECT count(*) AS 'VALUE' FROM {{.db_Alert}}.[dbo].[Alerts]
I want to make sure it is safe and not vulnerable.
Thanks

Since you are generating dynamic SQL for the database name, you can't use a parameter to ensure against SQL injection. Easiest is to:

Put the value inside []s. That lets users use pretty much any character other than []s themselves. Validate the value to ensure it does not contain []s.

Or (more difficult and possibly code error-prone):

If you don't require that kind of flexibility in your db names, you could limit them to characters that fit the Rules for Regular Identifiers.

Thank you @David-Engel for your suggestions.
Yes i thought about this as prepared statements will not allow us to manage variables from .conf.
I was also thinking to sanitize variables.
I guess these are the only possible solutions to pass variables from the telegraf configuration!


Conflict with Chrome Remote Desktop connection

I have a WPF application that sends video stream for screen sharing to a Web application over WebRTC connection, and i am using MixedReality.WebRTC C# on the WPF side. This is working fine, but the problem occurs when I run the apps on different PCs and before making the web-rtc connection using my apps I connect both PCs via Chrome Remote Desktop connection, and when the apps try to make their web-rtc connection, WPF app crashes right after receiving SDP offer.
From event viewer i can see that the faulting module is 'mrwebrtc.DLL', Here is the complete event details:
Faulting application name: Nylytics.exe, version: <IP_ADDRESS>, time stamp: 0xe62ec6ba
Faulting module name: mrwebrtc.DLL, version: <IP_ADDRESS>, time stamp: 0x5f293f90
Exception code: 0xc0000409
Fault offset: 0x00000000005ba01d
Faulting process id: 0x1a74
Faulting application start time: 0x01d690ada54fd0df
Faulting application path: C:\Users\Hashim\AppData\Local\nylytics\app-2.6.0\Nylytics.exe
Faulting module path: C:\Users\Hashim\AppData\Local\nylytics\app-2.6.0\mrwebrtc.DLL
Report Id: c6fe29b3-e546-4a90-a19a-5be18f3b33f2
Faulting package full name:
Faulting package-relative application ID:
Expected behavior
WPF app shouldn't crash
Environment

MixedReality-WebRTC C# 2.0.0
Platform: [Windows Desktop]
Architecture: [x64 Windows 10]

Can you attach a debugger and get the crash callstack and/or debug logs?

This seems to have been fixed in the newer version, can't recreate this on MixedReality-WebRTC 2.0.1


Create a page for creating the form-specific intro screen (to replace single field)

~Note: I believe our decision on this will be to stick to the single question as it is. I just wanted to get this documented because I can see it being brought up. If by no one else, then by me in the future. We can close this as soon as that's confirmed.~
[Decision: Make the developer-specific intro screen its own weaver screen to be more explicit about its purpose. Ask for title and description.]
Right now from what I can see, we have a field that asks the developer for a sentence that will be included in the intro screen. Since the intro screen is a necessary part of the interview, should expand this to just let the developer create their intro screen? We could keep the helpful language of the field.
Pros:

There will be context when the developer answers the prompt question.

Cons:

More time spent in the wizard.
Possibly a significant pause if the developer stops to try to research what the wording and instructions need to be.
Possible frustration when more experienced devs can't utilize more advanced features of docassemble, like terms.

Agreed this is good to document.
My understanding is that the first intro screen is organizational branding. It may not serve the user as well, but a lot of our forms will be linked directly without the user starting on courtformsonline.org. It is a way for us to establish a consistent brand and explain what the site is, as well as ensure users accept the terms of use. Most sites do use this kind of branding. Ours is a lot lighter than the branding on LawHelpInteractive, for example.
There could be a better way to serve those goals, but I think we probably don't want that to be customized more than including the form's name at the moment.
The author has the full creative control over the second intro screen, and that's the only one we really want customized. We do need a few variations on the intro screen for different kinds of forms. I started to add that flexibility so letters get a different intro screen than court forms already.

Yeah, the second one is the one I'm referring to. Feel free to edit the title (and/or something else) to make it more clear which form this discussion is about.

If I understand correctly, we don't want to give developers full control over the second intro screen, correct?

I'm confused--isn't the second screen already fully customizable?

Closed by #281.


Flink k8s v2

What changes were proposed in this pull request
Issue Number: close #xxx

Brief change log

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (yes / no)

Need to continue to improve, you can not review


[QUERY] DefaultAzureCredentials picks up new environment variables with delay

Query/Question
We have some self hosted agents on which we have setup Azure_Client_Id, Azure_TenantId and Azure_client_secrets as env variables. We recently rotated the secret for the azure app and updated it in the build agent. However since then, the build started failing on some agents with error "ClientSecretCredential authentication failed: Original exception: AADSTS7000215: Invalid client secret is provided. Trace ID: XX Correlation ID: XX".
On some build agents, we did not have the issue. After about 4-5 hours, the build started working indicating that new credentials had been picked up.
Can you please point us to the reason why there was a delay in new credentials being picked up?
Environment:

Name and version of the Library package used: [e.g. Azure.Storage.Blobs 12.2.0]
AzureIdentity 1.3
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): [e.g. Azure AppService or Windows 10 .NET Framework 4.8]

.NET Core SDK (reflecting any global.json):
Version:   3.1.402
Commit:    9b5de826fd
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.18363
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.1.402\
Host (useful for support):
Version: 3.1.8
Commit:  9c1330dedd
.NET Core SDKs installed:
2.1.810 [C:\Program Files\dotnet\sdk]
2.2.402 [C:\Program Files\dotnet\sdk]
3.1.402 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.22 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.22 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.22 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.8 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.8 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

IDE and version : [e.g. Visual Studio 16.3]
VS v 16.0.30503.244

Error Message:
** threw exception:
Azure.Identity.AuthenticationFailedException: ClientSecretCredential authentication failed: A configuration issue is preventing authentication - check the error message from the server for details. You can modify the configuration in the application registration portal. See https://aka.ms/msal-net-invalid-client for details.  Original exception: AADSTS7000215: Invalid client secret is provided.
Trace ID: XXX
Correlation ID: XX
Timestamp: 2021-07-22 00:29:30Z ---> Microsoft.Identity.Client.MsalServiceException: A configuration issue is preventing authentication - check the error message from the server for details. You can modify the configuration in the application registration portal. See https://aka.ms/msal-net-invalid-client for details.  Original exception: AADSTS7000215: Invalid client secret is provided.
Trace ID: XXX
Correlation ID: XX
Timestamp: 2021-07-22 00:29:30Z
Stack Trace:
Microsoft.Identity.Client.Internal.Requests.RequestBase.HandleTokenRefreshError(MsalServiceException e, MsalAccessTokenCacheItem cachedAccessTokenItem)
Microsoft.Identity.Client.Internal.Requests.ClientCredentialRequest.d__2.MoveNext()
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
Microsoft.Identity.Client.Internal.Requests.RequestBase.d__13.MoveNext()
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
Microsoft.Identity.Client.ApiConfig.Executors.ConfidentialClientExecutor.d__3.MoveNext()
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
Azure.Identity.AbstractAcquireTokenParameterBuilderExtensions.d__01.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Identity.MsalConfidentialClient.<AcquireTokenForClientAsync>d__7.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Identity.ClientSecretCredential.<GetTokenAsync>d__16.MoveNext() Azure.Identity.CredentialDiagnosticScope.FailWrapAndThrow(Exception ex) Azure.Identity.ClientSecretCredential.<GetTokenAsync>d__16.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Identity.EnvironmentCredential.<GetTokenImplAsync>d__11.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() Azure.Identity.CredentialDiagnosticScope.FailWrapAndThrow(Exception ex) Azure.Identity.EnvironmentCredential.<GetTokenImplAsync>d__11.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Identity.EnvironmentCredential.<GetTokenAsync>d__10.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() Azure.Identity.DefaultAzureCredential.<GetTokenFromSourcesAsync>d__14.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Identity.DefaultAzureCredential.<GetTokenImplAsync>d__12.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() Azure.Identity.CredentialDiagnosticScope.FailWrapAndThrow(Exception ex) Azure.Identity.DefaultAzureCredential.<GetTokenImplAsync>d__12.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Identity.DefaultAzureCredential.<GetTokenAsync>d__11.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Security.KeyVault.ChallengeBasedAuthenticationPolicy.<AuthenticateRequestAsync>d__9.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Security.KeyVault.ChallengeBasedAuthenticationPolicy.<ProcessCoreAsync>d__8.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() Azure.Core.Pipeline.RetryPolicy.<ProcessAsync>d__11.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() Azure.Core.Pipeline.RetryPolicy.<ProcessAsync>d__11.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Core.Pipeline.HttpPipelineSynchronousPolicy.<ProcessAsync>d__1.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Core.Pipeline.HttpPipelineSynchronousPolicy.<ProcessAsync>d__1.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Core.Pipeline.HttpPipelineSynchronousPolicy.<ProcessAsync>d__1.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Core.Pipeline.HttpPipeline.<SendRequestAsync>d__10.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Security.KeyVault.KeyVaultPipeline.<SendRequestAsync>d__27.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Security.KeyVault.KeyVaultPipeline.<GetPageAsync>d__171.MoveNext()
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
Azure.Core.PageResponseEnumerator.FuncAsyncPageable1.<AsPages>d__2.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Threading.Tasks.Sources.ManualResetValueTaskSourceCore1.GetResult(Int16 token)
Azure.AsyncPageable1.<GetAsyncEnumerator>d__6.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() Azure.AsyncPageable1.d__6.MoveNext()
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
System.Threading.Tasks.Sources.ManualResetValueTaskSourceCore1.GetResult(Int16 token) Azure.Extensions.AspNetCore.Configuration.Secrets.AzureKeyVaultConfigurationProvider.<LoadAsync>d__10.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() Azure.Extensions.AspNetCore.Configuration.Secrets.AzureKeyVaultConfigurationProvider.<LoadAsync>d__10.MoveNext() System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) Azure.Extensions.AspNetCore.Configuration.Secrets.AzureKeyVaultConfigurationProvider.Load() Microsoft.Extensions.Configuration.ConfigurationRoot..ctor(IList1 providers)
Microsoft.Extensions.Configuration.ConfigurationBuilder.Build()
..Startup..ctor(IHostingEnvironment env) in E:\adoagent_work\3\s*\services*\service\src\Startup.cs: line 82
..UnitTests.StartupTests.ConfigureServicesTest() in E:\adoagent_work\3\s*\services*\service**: line 56

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

Hi @shubmittal - Did you observe any behavior that would indicate that the environment variables were updated on the host but not being picked up by DefaultAzureCredential? It's possible that either the environment itself was delayed in being updated, or that the environment was updated but your application process hadn't restarted so that the new environment was loaded. Either way, this doesn't sound like an issue with the DefaultAzureCredential itself.

/unresolve

@christothes :  The stack trace showed that the point at which it was failing was DefaultAzureCredentials.  I had restarted the hosts too, so I am assuming new process would have been started.

It makes sense that the credential is on the stack, but it sounds like DefaultAzureCredential is a victim of the environment. Do you have a repro of this behavior that demonstrates that the environment variable is correct and visible to the application, but DefaultAzureCredential is getting a different value?

The issue happened only on 3 agents in a pool of 5 and since it is not happening now, I am unable to get the exact repeatable steps to reproduce the issue.

Thanks @shubmittal - Please let us know if you are able to reproduce the issue again.


core[patch], langchain[patch]: LangSmith tracing improvements for OutputFixingParser

Problem: when using OutputFixingParser, the tracing with LangSmith doesn't work properly. This is because:

BaseLLMOutputParser doesn't pass on callbacks to parseResult
OutputFixingParser uses LLMChain, whose .call function for some reason doesn't pass along callbacks. When I create the following runnable myself and pass it in as retryChain in my PR, it works

retryChain: NAIVE_FIX_PROMPT
        .pipe(fields.extractLLM ?? fields.llm)
        .pipe(new StringOutputParser()) as any

Before (LangSmith shows multiple traces)

After (LangSmith trace shows the parsing traces as part of the parent trace)

This is awesome, thank you!


test(BDD): add inital BDD for node cordoning

Describe the initial behaviours of the node cordoning feature.

bors merge


Add permissions to enable CodeBuild to upload Docker images to ECR

What

Add ECR permissions to aws_iam_policy_document

Why

Enable CodeBuild to upload Docker images to Amazon ECR

References

http://docs.aws.amazon.com/codebuild/latest/userguide/sample-docker.html

Never use aws_iam_policy_attachment with roles on AWS as it causes unpredictable behavior unless you know exactly what you're doing. See hashicorp/terraform#13381 (comment)

Should we create a new PR for this (to change aws_iam_policy_attachment to aws_iam_role_policy_attachment?
It seems unrelated to this PR.
If we do, can we approve this PR?


Function not found error

➜  mxnet-lambda git:(master) ✗ aws lambda update-function-code --function-name mxnet-lambda --zip-file fileb://lambda_function.zip

An error occurred (ResourceNotFoundException) when calling the UpdateFunctionCode operation: Function not found: arn:aws:lambda:us-east-1:111111111111:function:mxnet-lambda

I had the same error

Can you make sure to specify the region correctly?  @davidneudorfer is your default region different than the one where you create the function in the first place?

For orhers that may encounter this, I found the Qualifier dictionary parameter in the Boto3 Lambda call to be hosing me up.
"Qualifier (string) -- Specify a version or alias to invoke a published version of the function."


Out of date info -

The Office Security and Compliance center has been deprecated, please update the notes about the Global reader role to describe any limitations in the new Microsoft Purview Center

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

ID: 0767aac4-fcd9-eaac-2c30-8f17d8eadd92
Version Independent ID: a380a828-03f9-e037-3a62-e4165fda7cb6
Content: Azure AD built-in roles - Azure Active Directory
Content Source: articles/active-directory/roles/permissions-reference.md
Service: active-directory
Sub-service: roles
GitHub Login: @rolyon
Microsoft Alias: rolyon

@DeanGross
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Hi @DeanGross
Thanks for your feedback. We've sent a message to the team to review.

Hi @DeanGross
We've made some updates to the Global Reader role description.
https://docs.microsoft.com/en-us/azure/active-directory/roles/permissions-reference#global-reader
Let us know if you have additional feedback.
thanks

#please-close


switch to typed-builder

I tested this in one repo and it worked fine. As this touches basically everything it should be checked more carefully I guess.
closes #46
BREAKING CHANGES:

FooBarBuilder::default() → FooBar::builder()
build().unwrap() → build()

@EdJoPaTo wow. nice work. I'll test it tomorrow first thing in the morning.
but looks good!


Re-land static-executable-args.lnk changes

The original change didn't care about the case where SWIFT_BUILD_STATIC_STDLIB=YES but no static-executable-args.lnk is necessary (like on Darwin). It broke freestanding build https://ci.swift.org/job/oss-swift-test-stdlib-with-toolchain-minimal/4880/ with the following error, so the change was reverted by https://github.com/apple/swift/pull/66811
CMake Error at stdlib/public/runtime/CMakeLists.txt:133 (add_dependencies):
  add_dependencies called with incorrect number of arguments

This is the take 2 to re-land the change with a fix for the case.

@swift-ci Please test

preset=stdlib_S_standalone_minimal_macho_x86_64,build,test
@swift-ci please test with toolchain and preset

@swift-ci build toolchain

@swift-ci test windows

@swift-ci please build toolchain Windows platform


README is wrong

Instructions in the readme are broken.

submodule does not init
tests don't work
I believe we need specific versions of Node and npm

$ cd jaeger-client-node
$ git submodule init update
error: pathspec 'update' did not match any file(s) known to git.

so fix it by
$ git submodule init
$ git submodule update

# now try running tests
$ npm test
...
sh: flow: command not found
sh: eslint: command not found
...
npm WARN Local package.json exists, but node_modules missing, did you mean to install?

I also experienced this. My workout was running all commands as root:
$ sudo su
# ...

I think this was resolved in 7804b0ef6c8c0cec15e8238292d954718438b09f


[Feature Request] Ability to skip pages when generating the graph

Wanted to add in a .md page tag where we can tag pages to be skipped when it generates the nodes for the graph.
The markdown tag in file would look like this or similar:
'''
ZOLA_GRAPH_SKIP: Y
'''

Then add something in the Config where you could turn this on/off.
ZOLA_GRAPH_SKIP = "y"

Similarly - could add a tag to add pages to the Navigation bar at the top of the theme.

Just adding another thought. Since the graph is so good looking and fun, I think adding a separate page with just the graph would be cool. Then just add a navbar/footer link in.

This would be ideal for me too, if possible, as I tend to use single list notes to link to many other notes, which complicates and slows down the graph a lot.

Is there a solve for this? I'm currently using daily journal pages as activity logs (updated & new files) and they create an awfully big mess in the graph since they link to the pages directly. Would like to be able to exclude them from the graph while still publishing them.


Release azure-cli-cognitiveservices 0.1.0

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Started releasing azure-cli-cognitiveservices 0.1.0...

Release of 'azure-cli-cognitiveservices' with version '0.1.0' successful. View at https://pypi.python.org/pypi/azure-cli-cognitiveservices/0.1.0.

GitHub release created. https://github.com/Azure/azure-cli/releases.

Codecov Report

Merging #3201 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3201   +/-   ##
=======================================
  Coverage   70.35%   70.35%
=======================================
  Files         381      381
  Lines       24777    24777
  Branches     3785     3785
=======================================
  Hits        17431    17431
  Misses       6237     6237
  Partials     1109     1109

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 39a8fcd...09a270d. Read the comment docs.


Specify derive feature for serde

ra_project_model build will fail when no dependencies are using this feature. Reproduced by creating a crate depending on ra_batch.

bors r+
Thanks!


Update react-router-dom version to 6.4.3

@ppchart Upgrading the version of the dependent library may cause other problems, which needs to be handled internally by us

@robinv8 This PR should not be merged until dependencies are upgraded
https://github.com/answerdev/answer/pull/17

revert #60
Let's discuss internally

@ppchart You mentioned a PR that limits pnpm

https://github.com/answerdev/answer/pull/62
Submitted a PR that only supports pnpm. If we tell users that only pnpm is supported, the problem of the wrong version of react-router-dom after installing dependencies on npm can be ignored

提交了一个只支持 pnpm 的 PR，如果我们告诉用户只支持使用 pnpm，针对 npm 安装依赖后 react-router-dom 版本错误的问题就可以被忽略了

Thanks for the contribution


Yamaha AW2816 - Issues when formatting device

Info

Which version of Pi are you using: Zero W
Which github revision of software: 23.4.1 (Pre-built image)
Which board version: akuker PiSCSI Zero 1.1
Which computer is the PiSCSI connected to: Yamaha AW2816 (Digital mixing console)

Describe the issue
Hello,
I'm having an issue using the PiSCSI on a Yamaha AW2816.
The emaulated drive is successfully recognized by the console, but it needs formatting by the console before it's usable.
When excecuting the formatting, the communication seems fine for a few seconds (based on the indicator LEDs), then it halts without completing and the console is stuck on "formatting".
I've gathered some logs at the error phase, which are showing the following:
May 23 16:09:49 piscsi PISCSI[355]: [2023-05-23 16:09:40.605] [trace] (ID 0) - Command phase
May 23 16:09:49 piscsi PISCSI[355]: [72B blob data]
May 23 16:09:49 piscsi PISCSI[355]: [2023-05-23 16:09:40.606] [debug] (ID 0) - Error status: Sense Key $5, ASC $20
May 23 16:09:49 piscsi PISCSI[355]: [2023-05-23 16:09:40.606] [trace] (ID 0) - Error (to status phase)

I'm attaching the full log for reference.
piscsi-aw2816-0523.txt
Is it possible that this issue is related to #1133 ?
Let me know if you need more tests, I will be happy to help!
Thank you for the great work
Daniele

According to the Sense Key and ASC there is an issue with the SCSI command sent by the device. The command is usually also logged, but where the relevant logging output should be your logfile says "[72B blob data]". This sounds as if there are binary data instead of normal logfile text data.
Regular log output for the respective piece of code (SCSI Command phase) looks like this:
[2023-05-23 19:13:28.444] [trace] (ID 1) - Command phase
[2023-05-23 19:13:28.444] [debug] (ID 1) - Controller is executing Read6/GetMessage10, CDB $0803fdde0400
[2023-05-23 19:13:28.444] [debug] (ID:LUN 1:0) - Device is executing Read6/GetMessage10 ($08)
[2023-05-23 19:13:28.444] [trace] (ID:LUN 1:0) - READ/WRITE/VERIFY/SEEK, start block: $0003fdde, blocks: 4

I cannot tell why, but your piscsi process does not appear to generate proper log data for some parts of the logfile. I have never seen something like that before, but I am afraid without the missing logfile data it is not possible to find out why the SCSI command was considered illegal.

The piscsi logs are always in text format, i.e. there is no way/need to force them from binary into text. If there are binary data in your log this is because something not (only) related to SCSI formatting is not working as expected. As you can see from my logfile example a regular logfile entry for SCSI commands is plain text and lists the SCSI command block with the command bytes. This information is missing in your log, and as long as this is the case it is not possible to address any SCSI format issue. The binary data block further analysis.
Just a general note: SCSI commands (and also the SCSI format command) do not have to do anything with filesystems. Filesystems and SCSI are completely independent things. This is the same with IDE or ATA: They are also not aware of filesystems and essentially only know how to read and write data from/to a drive. The nature of these data depend on the OS, and only the OS knows about filesystems.

Thank you for the in-depth explanation! I know a bit of C++, I'll try to look at the logfile problem

The relevant logging code is in scsi_controller.cpp, the method is ScsiController::Execute, right at the beginning:
	stringstream s;
	s << "Controller is executing " << command_mapping.find(GetOpcode())->second.second << ", CDB $"
			<< setfill('0') << hex;
	for (int i = 0; i < BUS::GetCommandByteCount(static_cast<uint8_t>(GetOpcode())); i++) {
		s << setw(2) << GetCmd(i);
	}
	logger.Debug(s.str());

Something with these lines of code does not work as expected on your system.

@procionefotone Any news on this? I'm afraid without your feedback keeping this ticket open does not make much sense.

Closing this ticket because without feedback it cannot be worked on.


Error handling 'checkPlatformOverride'

I start my application with F5, and the application runs normal, but I get some errors that make the hot reaload and hotRestart not work, any solution?
Flutter 1.22.5 version actually
Stable channel
Error handling 'checkPlatformOverride' custom request: method not available: ext.flutter.platformOverride
Error handling 'checkBrightnessOverride' custom request: method not available: ext.flutter.brightnessOverride
Error handling 'checkIsWidgetCreationTracked' custom request: method not available: ext.flutter.inspector.isWidgetCreationTracked

@flutteresbolivia
please provide your complete flutter doctor -v and your complete flutter run -v

@flutteresbolivia
please provide your complete flutter doctor -v and your complete flutter run -v

Hi @flutteresbolivia
Please provide what was asked above along with the steps to reproduce the issue.

Hi @flutteresbolivia
Please provide what was asked above along with the steps to reproduce the issue.

Hi, I faced same issue while started using rate_my_app plugin.
when i did below code for never ask for rating. then after reload this issue occur, it does not crash app but only warning. I have checked this on android device in debug mode
await rateMyApp.callEvent(RateMyAppEventType.noButtonPressed);
Navigator.pop<RateMyAppDialogButton>(context, RateMyAppDialogButton.no);

Hi, I faced same issue while started using rate_my_app plugin.
when i did below code for never ask for rating. then after reload this issue occur, it does not crash app but only warning. I have checked this on android device in debug mode
await rateMyApp.callEvent(RateMyAppEventType.noButtonPressed);
Navigator.pop<RateMyAppDialogButton>(context, RateMyAppDialogButton.no);

@uc-dve
if you are experiencing the issue with the 3rd party plugin rate_my_app
you may want to open an issue in the dedicated github
otherwise could you provide a minimal reproducible code sample without 3rd party packages

@uc-dve
if you are experiencing the issue with the 3rd party plugin rate_my_app
you may want to open an issue in the dedicated github
otherwise could you provide a minimal reproducible code sample without 3rd party packages

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

Console warnings on hot reload and restart:
Error handling 'checkBrightnessOverride' custom request: method not available: ext.flutter.brightnessOverride
Error handling 'checkIsWidgetCreationTracked' custom request: method not available: ext.flutter.inspector.isWidgetCreationTracked
Restarted application in 11.487ms.
Error handling 'serviceExtension' custom request: [Sentinel type: Sentinel, kind: Collected, valueAsString: <collected>] from ext.flutter.inspector.setPubRootDirectories()
Error handling 'checkPlatformOverride' custom request: method not available: ext.flutter.platformOverride
Error handling 'checkBrightnessOverride' custom request: method not available: ext.flutter.brightnessOverride
Error handling 'checkIsWidgetCreationTracked' custom request: method not available: ext.flutter.inspector.isWidgetCreationTracked

Flutter doctor:
[√] Flutter (Channel stable, 1.22.6, on Microsoft Windows [Versione 10.0.19041.804], locale it-IT)
• Flutter version 1.22.6 at C:\src\flutter
• Framework revision 9b2d32b605 (4 weeks ago), 2021-01-22 14:36:39 -0800
• Engine revision 2f0af37152
• Dart version 2.10.5
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at C:\Users\Thetis\AppData\Local\Android\sdk
• Platform android-30, build-tools 30.0.2
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
X Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.
[!] Android Studio (version 4.1.0)
• Android Studio at C:\Program Files\Android\Android Studio
X Flutter plugin not installed; this adds Flutter specific functionality.
X Dart plugin not installed; this adds Dart specific functionality.
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
[√] VS Code (version 1.53.2)
• VS Code at C:\Users\Thetis\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.19.0

@0biWanKenobi

Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

I have told you what happens: those warnings appear in console on app hot reload and/or restart. I have also appended the verbose output from flutter doctor. ATM I don't have any logs to provide, I had already stopped the app when I stumbled on this github issue. If it happens again, I'll try to provide logs too. I thought contributing something was better than nothing.

@0biWanKenobi
was closed as not reproducible and it isn't currently worked on,
the partial logs you provided are appreciated, but don't make the issue workable,

If it happens again, I'll try to provide logs too.

thank you in advance,
doing so, please open a new issue your flutter doctor -v, your complete  flutter run -v
and a minimal reproducible code sample without 3rd party plugins

I am still facing this issue, my code is as:
import 'package:flutter/material.dart';
import 'package:contacts_service/contacts_service.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  List<Contact> contacts;

  @override
  void initState() {
    super.initState();
    fetchContacts();
  }

  fetchContacts() async {
    // Get all contacts on device
    await ContactsService.getContacts().then((value) {
      contacts = value;
      setState(() {});
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Test contact service'),
      ),
      body: Container(
        child: ListView.builder(
            itemCount: contacts.length,
            itemBuilder: (context, index) {
              return ListTile(
                title: Text(contacts[index].displayName),
              );
            }),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Getting error in console:
Error handling 'checkPlatformOverride' custom request: ext.flutter.platformOverride: (-32000) Bad state: The client closed with pending request "ext.flutter.platformOverride".
Error handling 'checkBrightnessOverride' custom request: ext.flutter.brightnessOverride: (-32000) Bad state: The client closed with pending request "ext.flutter.brightnessOverride".
Error handling 'serviceExtension' custom request: ext.flutter.inspector.setPubRootDirectories: (-32000) Bad state: The client closed with pending request "ext.flutter.inspector.setPubRootDirectories".
Error handling 'checkIsWidgetCreationTracked' custom request: ext.flutter.inspector.isWidgetCreationTracked: (-32000) Bad state: The client closed with pending request "ext.flutter.inspector.isWidgetCreationTracked".
Please check and suggest, what is the issue and how can we fix it. Thanks a lot.

@kamleshwebtech
please open a new issue providing your flutter doctor -v, your complete  flutter run -v
and a minimal reproducible code sample without 3rd party plugins like contacts_service

please provide your complete flutter doctor -v and your complete flutter run -v. by this command solve my problems.


fix: the date of Amplify Meetup 02 in Japan

Issue #, if available:
N/A
Description of changes:
Fixed the date of Amplify Meetup 02 in Japan.
This is due to https://github.com/aws-amplify/community/issues/17
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

cc @dabit3

@dabit3 thank you!

Thank you :)


Edit Assets and Investments

Assets and Investments are editable, but their tpyes and subtypes, among other fields, are not editable. They need to all be editable.

[x] Assets
[x] Investments

We must refactor using this: https://github.com/powmedia/backbone-forms

I love the editing menu, it works great. @allthesignals


V.02- Request to merge

Coverage decreased (-2.5%) to 88.57% when pulling f7b8cc0ac411358b3af3b074a41d4507dce22f47 on V.02-yeejfe into da604047441c0e39d7e1dcdfc9b7774e42125abb on V0.2.

Coverage decreased (-7.0%) to 84.064% when pulling fedf29864ef5915dd077d8e9bee5f4003926e8b9 on V.02-yeejfe into da604047441c0e39d7e1dcdfc9b7774e42125abb on V0.2.

Coverage decreased (-2.5%) to 88.498% when pulling 021abf17c873fd828ee2f5da1ac0b72a4cb0eb5b on V.02-yeejfe into da604047441c0e39d7e1dcdfc9b7774e42125abb on V0.2.


⚡ TailwindCSS config

Summary
This PR significantly optimizes the build and run time for development by refining the content configuration in tailwind.config.js.
Changes

Modified content configuration in tailwind.config.js to include a more specific set of directories and file types.

Details
The old content configuration was:
content: ['./src/**/*.{js,ts,jsx,tsx,vue,svelte}', './**/*.{liquid,json}'],

This has been updated to:
content: ['./{assets,config,layout,locales,sections,snippets,src,templates}/**/*.{js,ts,jsx,tsx,vue,svelte,liquid,json}'],

Performance Improvements
The previous configuration had a significant performance drawback as it was scanning through unnecessary folders like node_modules, leading to longer build times.
For example, with the old configuration, it took more than 20 seconds to build and run a development CLI. With the new configuration, the time is reduced to approximately 1 second. This is a substantial improvement, making the development process much more efficient.
Why This is Necessary
This modification ensures that Tailwind CSS properly purges unused styles from a more targeted range of files and directories. This not only makes the final CSS output more optimized but also dramatically speeds up the development process.

Correct me if I'm wrong but does the config folder need to be included?

Don't know who will, but imagine if a config contains a class as a value or something else
Just to cover all cases (even weird ones) :)


Add add_graph_documents support for FalkorDBGraph

Adding add_graph_documents support for FalkorDBGraph and extending the Neo4JGraph api so it can support cypher.py

It a superset of a previous PR: pull/11064
@hwchase17, @baskaryan can you plesae review it?

thanks @gkorland!


Skip 10sec function

The app is good and stable . But when i click the skip 10 seconds button the response is so delay. Please improve that if possible.

Marked for the update .
Thank you bro.


Wrong colors in color_list from scipy.cluster.hierarchy.dendrogram

# Init
import pandas as pd
import numpy as np
import matplotlib.pyplot as plt
import seaborn as sns; sns.set()

# Load data
from sklearn.datasets import load_diabetes

# Clustering
from scipy.cluster.hierarchy import dendrogram, fcluster, leaves_list
from scipy.spatial import distance
from fastcluster import linkage # You can use SciPy one too

%matplotlib inline

# Dataset
A_data = load_diabetes().data
DF_diabetes = pd.DataFrame(A_data, columns = ["attr_%d" % j for j in range(A_data.shape[1])])

# Absolute value of correlation matrix, then subtract from 1 for disimilarity
DF_dism = 1 - np.abs(DF_diabetes.corr())

# Compute average linkage
A_dist = distance.squareform(DF_dism.as_matrix())
Z = linkage(A_dist,method="average")

The colors specified by the color_list key don't sync up with the leaves.
Can future versions please allow the link_color_func to support a dictionary object that has {key = leaf_label: value=color} ? It seems to be the most intuitive way for the user to specify colors.

I just realized that dcoord and icoord have a smaller size than ivl and leaves ?
icoord	9
ivl	10
color_list	9
leaves	10
dcoord	9

Here's my answer on stackoverflow:

icoord, dcoord and color_list describe the links, not the leaves.  icoord and dcoord give the coordinates of the "arches" (i.e. upside-down U or J shapes) for each link in a plot, and color_list is the color of those arches.  In a full plot, the length of icoord, etc., will be one less than the length of ivl, as you have observed.
Don't try to line up the ivl list with the icoord, dcoord and color_list lists. They are associated with different things.

You said "The colors specified by the color_list key don't sync up with the leaves."  That's because those are the colors of the links, not the leaves.

You also made an enhancement request: "Can future versions please allow the link_color_func to support a dictionary object that has {key = leaf_label: value=color}"  If my previous comment made sense, you'll see that this request doesn't make sense, because link_color_func is for coloring the links, not the leaves.  dendrogram doesn't color the leaves.  In a full plot, the leaves are simply the labels on the x axis.
I'm closing the issue, but feel free to add more comments here.  If I've misunderstood what you want, we can reopen the issue.

Thanks for you answer.  It makes more sense now.  Is there a tutorial available for  custom coloration of the links if one has a dictionary of leaves to colors? I'm not sure how to traverse the dendrogram function output to reconstruct the dendrogram with custom colors.

I don't know of any tutorials like that.  I also don't fully understand what you want to accomplish.  The lines draw by dendrogram (which I called "arches" above) are the graphical representation of a link, and each link is associated with at least two leaves.  What method do you propose to map from your leaf color dictionary to link colors?   For example, suppose the dendrogram results in a link between two leaves that are assigned different colors.  What color should be used to draw the link?

Sorry, I think I was using the terminology loosely.  Once I compute the dendrogram, I usescipy.cluster.hierarchy.fcluster to find the clusters after cutting the tree at a certain height.  After I have those clusters, I make a dictionary that assigns a specific color to each of the clusters (and all of their leaves).  I want to be able to color my clusters (w/ leaves and links below distance threshold) with the new colors I assigned.  My reason for this was to have non-redundant colors for samples with lots of clusters.  Very similar to the results from the color_threshold but with my more flexibility and scalable.


rtmpdump: unrecognized option `--jtv'

Hi I installed rtmpdump using : git://git.ffmpeg.org/rtmpdump
But I launch
livestream http://twitch.tv/followgrubby 720p

I got this error :
rtmpdump: unrecognized option `--jtv'

You'll have to update the version of rtmpdump that you're using.  I ran into this issue as well the other day.  What I did was checkout the latest version of rtmpdump from it's git repo and compile and install that.  The problem went away.

You were totally right I did a bad mistake with homebrew. Issue closed.


Conditional build feature

Hi, I'm thinking about switch from jenkins to phpci, because it has too much features that I don't need. But one think I'm missing and it's conditional build or build parameters.
My use case is that I want CI to manage deploy of applications using deployer. So I created plugin to run deployer but I want to rollback if there is problem or pass different options for build.
Is there way how to pass frontend specified options to plugin, in best case when I click on build.

@zajca you are right, but you can add parameters (eg branch/commit hash) if you invoke the build using REST API, so I suppose you can easily create a bash script (or create a UI ) to specify parameter for the build (but it also depends on what parameter you need to specify).

In jenkins now I have "Choice parameter", it's just array of string and name of the variable. And in build step I have bash switch like this:
case "$Deployer" in
"deploy-prod1")
    dep deploy prod1;
    ;;

"rollback-prod1")
    dep rollback prod1;
    ;;

"deploy-test")
    dep deploy test;
    ;;

"rollback-test")
    dep rollback test;
    ;;

*)
    echo 'ERR';
    ;;
esac

And when I trigger build in UI, I can choose which option I want and it's passed to bash as variable.


update to dotnet 6 and c# 10

Are any changes on the NWNX plugin side needed?

Probably shouldn't happen until EOL in May next year, as this will break any project referencing it. https://dotnet.microsoft.com/platform/support/policy/dotnet-core

Is there still a benefit of updating the sdk to 6.x, but keeping <TargetFramework> at net5.0?

Are any changes on the NWNX plugin side needed?

I think the host APIs should be identical, though I do want to clean up the plugin at some point: https://docs.microsoft.com/en-us/dotnet/core/tutorials/netcore-hosting

I think this is ready for review now, I should have marked this as a draft PR before now, my apologies.


Updated README, added license file and accessibility report

they look super nice! thank you for editing that!

Again, since my previous one wasn't merged, my recent commits got added to this one. But I fixed the Person and all associated entities and interfaces so that it follows the approach exemplified by Room and all of its associated entities and interfaces. Now we shouldn't have to worry about any violations of the ISP or LSP with methods that are unnecessarily implemented!

looks good! meets all reqs so I'm merging it


docs: fix emphasis in source.rst

Hi. I was going over the docs for lib and noticed wrong markup for emphasis. Thought I'd drop this quick fix.
Thanks for the work by the way.

Thanks!


Chore/cherry pick translations

Release Note
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 9337a5e0daa5e40f3c99ff890cd7c2f65a42b21b


Syntax: constructor wirh all

thanks! But no idea ... please provide example.

sorry, hit return by mistake. Issue updated.

Hi @kylefleming (= the master of syntax highlighting). Any ideas/hints or fixes for us? Sorry to bother! All the Best, Benedikt


#695 Do not open a browser if Configuration.reopenBrowserOnFail is …

…false and user has not set webdriver manually
Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Checklist

[x] Checkstyle and unit tests pass locally with my changes by running gradle check chrome htmlunit command
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

@asolntsev Do you take into account such changes will break tests for users who accidentally still use reopenBrowserOnFail=true?

@BorisOsipov Why? reopenBrowserOnFail is true by default. Nothing will change in this case.

I've misprinted. If someone accidentally have been setted reopenBrowserOnFail=false.

@BorisOsipov Yes, right, in this case the behaviour will change. The browser will not start. It's ok for me. The test will fail with a clear error message:
java.lang.IllegalStateException: No webdriver is bound to current thread: 1, and cannot create a new webdriver because Configuration.reopenBrowserOnFail=false


(tweak) Please consider adding a license

I am thinking of using the Babble 3D models for an assembly animation. I wanted to confirm this was suitable with you for our use case.

Sounds cool to me! Go for it.
I'll go ahead and add the MIT License unless someone has an alternate suggestion.

added https://github.com/Physics-Dude/Phys-Index-EyetrackVR-HW/blob/main/LICENSE


Complete The guide us page

hey thanks for assigning! I just want to know one thing can you please explain how your guide book page shiuld look like, so i will work according to it

Can you please contact me on Discord?

yeah

AMANJAIMAN#3931

I think you cant complete it


gulp error

I have no idea at the moment where is the problem with GULP .
I just tried to install terria on Ubuntu 16.04 node 6, and I get the following error:
npm ERR! Linux 4.4.0-21-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "run" "gulp"
npm ERR! node v6.11.0
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>gulp: gulp
npm ERR! Exit status 137
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>gulp script 'gulp'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the terriajs-map package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs terriajs-map
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls terriajs-map
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/vladimir/TerriaMap/npm-debug.log

Did you run npm install in the TerriaMap directory first? If so, it could be worth removing your node_modules directory and running it again.  You can also try npm install -g gulp and then run gulp directly instead of npm run gulp.

Ok removing node_modules worked. Thanks a lot