[NEW ICON] PowerShell

Skill Name
PowerShell
Why?
Similar to Bash, PowerShell is simple, yet powerful scripting language from Microsoft primarily for automation of tedious and repetitive tasks to run, build, and test systems and servers via the .NET ecosystem.
Reference Image

Added in 9e1ef97!


deleting a comment generates a popup that you have a new comment

Joshua writes:
deleting a comment generates a popup that you have a new comment, but I think that'd be one case we want to ignore if it is easy?

@jdanish Currently as of 10/10/2024 on http://rapt.inquirium.org:3000/#/ deleting a comment does not seem to generate a new status update.

Now I cannot reproduce it either. Closing and will re-open if I figure it out. Sorry!

I'm also having trouble reproducing the Evidence Link comment appearing in the wrong place, but I'm pretty sure I saw it earlier too.  There may have been an error condition in there somewhere, or it's also possible there's an async issue.


What is it exactly?

I wrote this on a Show HN about this, but was (correctly) called out for not opening an issue about that:

The information on /login should probably be on / so I can read somewhere what "Stateless authentication microservice" actually means.
Also maybe a sentence why this is neat and should be used...

Someone else wrote:

"Stateless authentication microservice" has only one possible meaning to me. Did you want to ask something more specific, or do you just not know what those (common) words mean?
reply

to which I then replied

The me it means... nothing specific.
It tells me that is has something to do with login, "without state" (whatever this means in this context) and that it is somehow not a "big service" but very "micro", so probably independent. Stateless and microservice can mean so many things.

So please take this as a bit of feedback: Not everyone really and fully understand the current description.

Sorry for not getting the answer you would like to have. It is definitely a legitimate question to ask! I'll add an "About/Why should I use/nedd this" section in the readme or on the homepage/demo.

After read some more comments and also understanding the demo at login-with.com I think something along "this takes care of social login and delivers authenticated users with JWT back to your application without storing login information itself" would have helped me.

Can this be deployed as AWS Lambda functions?

@danesparza AWS Lambda is my next target. Due to work in #27 this should be possible with something like https://www.npmjs.com/package/lambda-express or https://github.com/awslabs/aws-serverless-express
Could perhaps work


Question/Feature

Is there a way to limit how neotest searches for tests?
Currently I'm working with a massive mono-repo and neotest tries to grab every test it can find which slows my computer to a halt.
It would be great if I could set a set of root files that neotest could search upwards for lazily in order to find the current working directory, in my case this could be ["main.go", "README.md"] etc.
Thanks!

There are several options related to test discovery. You can disable discovery completely so only open buffers are shown. You can also filter directories to search using a function. See :help neotest.Config.discovery for both
neotest-go looks for go.mod and go.sum already to find the root directory https://github.com/nvim-neotest/neotest-go/blob/d29d20d912aca81a07c50022d880cc66f0d26542/lua/neotest-go/init.lua#L23 but currently it searches from the current directory and searches upwards, it doesn't search sub-directories. You could assign your own root discovery function to the adapter if you wanted
local go_adapter = require("neotest-go")
go_adapter.root = function (cwd)
  ...
end
require("neotest").setup({
  adapters = {go_adapter},
})

Thank you @rcarriga this is really helpful I'll see if I can implement something to suit my needs - I'll close this as well


refacto(assistants builder): move parents state deeper

Moves the parents-related logic to DataSourceResourceSelector to simplify as recommended @spolu

I think it's easier to understand like this  @spolu

\o/


Add Alchemy to default providers

We are using ethers.js default provider:

This creates a FallbackProvider backed by multiple backends (INFURA and Etherscan).

Add Alchemy too using FallbackProvider
Note: Check if events filter (used by DecentralandUtils) will have a good response time.

Another alternative maybe to pursue in parallel is to lobby ethers.js to add them to the default provider list

Another alternative maybe to pursue in parallel is to lobby ethers.js to add them to the default provider list

https://github.com/ethers-io/ethers.js/issues/445#issuecomment-483760073


Cherry-pick key PRs from unity to support LLaMA/Vicuna optimization

This includes:

https://github.com/apache/tvm/pull/15118
https://github.com/apache/tvm/pull/15028
https://github.com/apache/tvm/pull/15030
https://github.com/apache/tvm/pull/15029

cc @sunggg

we will do a rebase soon, so no cherry pick is needed

the rebase is now completed


N4BiasFieldCorrection error

Hi, I tried a lot to reproduce your results, but it seems there is a problem in my machine.
In the case that n4-bias-correction is True, I get the following error:
Loading scans...
-> Applyling bias correction...
Traceback (most recent call last):
  File "n4_bias_correction.py", line 24, in <module>
    n4.run()
  File "/home/azim/.local/lib/python2.7/site-packages/nipype/interfaces/base.py", line 1043, in run
    runtime = self._run_wrapper(runtime)
  File "/home/azim/.local/lib/python2.7/site-packages/nipype/interfaces/base.py", line 1660, in _run_wrapper
    runtime = self._run_interface(runtime)
  File "/home/azim_se/.local/lib/python2.7/site-packages/nipype/interfaces/base.py", line 1686, in _run_interface
    (self.cmd.split()[0], runtime.hostname))
IOError: command 'N4BiasFieldCorrection' could not be found on host othrys2.dhcp.af.op.dlr.de
Interface N4BiasFieldCorrection failed to run.
Traceback (most recent call last):
  File "n4_bias_correction.py", line 24, in <module>
    n4.run()
  File "/home/azim/.local/lib/python2.7/site-packages/nipype/interfaces/base.py", line 1043, in run
    runtime = self._run_wrapper(runtime)
  File "/home/azim/.local/lib/python2.7/site-packages/nipype/interfaces/base.py", line 1660, in _run_wrapper
    runtime = self._run_interface(runtime)
  File "/home/azim/.local/lib/python2.7/site-packages/nipype/interfaces/base.py", line 1686, in _run_interface
    (self.cmd.split()[0], runtime.hostname))
IOError: command 'N4BiasFieldCorrection' could not be found on host tehran.mpi.inf-mpg.de
Interface N4BiasFieldCorrection failed to run.
Traceback (most recent call last):
  File "brain_pipeline.py", line 167, in <module>
    save_patient_slices(patients, 'reg')
  File "brain_pipeline.py", line 139, in save_patient_slices
    a = BrainPipeline(path)
  File "brain_pipeline.py", line 25, in __init__
    self.slices_by_mode, n = self.read_scans()
  File "brain_pipeline.py", line 49, in read_scans
    scans = [flair[0], t1_n4[0], t1_n4[1], t2[0], gt[0]]
IndexError: list index out of range

I would do appreciate if you could help me figure this problem out.

@smajida you need to install N4BiasFieldCorrection from ANTs.

@ellisdg
thanks it solved the problem, but still I am having the following error. Do you think you could take a quick look at it as well?
https://github.com/naldeborgh7575/brain_segmentation/issues/5

Hi.   why i just can run this N4BiasFieldCorrection from ANTs in windows win10 x64, i have install this and add this to the enviroment editor... or i need a linux to solve this problem?
the error is the same as this question......

I'm a newcomer in this field,but i have the same problem with you.I've already downloaded ANTs that the version is ANTs-1.9.x-win32. And I have tried to install  N4BiasFieldCorrection, but it didn't succeed. In fact ,i don't know how to install N4BiasFieldCorrection from ANTs. I'd like to ask which version of ANTs you have downloaded, how do you install  N4BiasFieldCorrection, and how i should do concretely to solve the problem successfully：IOError: command 'N4BiasFieldCorrection' could not be found on host MS-20170816ZZTH.DHCP HOST. I sincerely hope that you can help me to solve the problem. Thanks.

How did you get rid of the error?

I added the bin folder to my Path variable. Didn't solve the problem on WIN 10, though. Any suggestions ?

I just started using linux. Windows subsystem on Linux. Works well.


feat: Adds docs dynamic label mapping for OS

Please wait with merge until this feature is merged to dev


Getting error while looking for window location

Expected behavior
It should not fail
Actual behavior
It failing with error
✘ Error: No node with given id found, run .trace for more info.
Steps to reproduce

launch repl
openBrowser()
evaluate(()=>{return window.location.toString()})

Versions
node : v11.13.0
taiko : d591dcfae8416d6e9b35fc9dc2f9d6d9ac7d86de

This issue has been verified and found fixed with the coomit: b3207ffb094e44f98b22febddf02e84c258a638f

Working on this.

@shrirambalakrishnan @luciferankon  What is the use case of running an evaulate on blank page.
Below code snippet works fine.
const { openBrowser, goto, evaluate, closeBrowser } = require('taiko');
(async () => {
    try {
        await openBrowser();
        await goto('google.com');
        await evaluate(()=>{return window.location.toString()});
    } catch (error) {
        console.error(error);
    } finally {
        await closeBrowser();
    }
})();

Result:https://www.google.com/?gws_rd=ssl

@saikrishna321
It's a failing edge case. I can't think of any use case though.
evaluate(()=>{return window.location.toString()}) fails only when executed immediately after openBrowser().
It works after explicitly navigating to goto("about:blank") by returning
Result:about:blank

Instead of the API failing my thought would just throw a warning to the users if evaluate() before goto()

Since we can't think of any use case for this, my suggestion will be to raise exception if user tries to evaluate() in the blank page.
Please let me know your thoughts on this.

The url on the location bar on the chromium instance after openBrowser is about:blank so ideally it should return Result:about:blank

Working on 1.0.16
Version: 1.0.16 (Chromium: 85.0.4168.0)
Type .api for help and .exit to quit

> openBrowser()
 ✔ Browser opened
> evaluate(()=>{return window.location.toString()})
'about:blank'
 ✔ Evaluated given script. Result:about:blank
>


Broken when running with jsfiddle

http://stackoverflow.com/questions/20107490/confirm-that-batarang-is-broken
Detailed with images that are included in above link

This is because we don't currently support iframes (and JSFiddle displays the result of your code in an iframe).
Dupe of #14


chore: Golang example

Progress towards: #1350

This looks good. Could we change the directory to go instead of Go. The R directory is capitalized to make it more recognizable (since R files end with .R).

for sure


64 bit race on timer counter in cavs_timer

The CAVS hardware has a 64 bit timer counter (it's the same device as the HD Audio wall clock timer).  But the code to read it looks like this:
static uint64_t count(void)
{
	return shim_regs->walclk;
}

When compiled on Xtensa, this produces two sequential 32 bit reads of the value.  And that's wrong, because the hardware continues to run.  It's possible for the two reads to straddle a rollover of the low word and increment of the high word.
This needs to be fixed to be safe.  The simplest scheme is to read the high word twice, compare the two values, and repeat the process if they don't match.

Discovered via conversation with @lgirdwood  and @lyakh who had tripped over the same bug in upstream (non-Zephyr) SOF code.  We have exactly the same messup.

Discovered via conversation with @lgirdwood  and @lyakh who had tripped over the same bug in upstream (non-Zephyr) SOF code.  We have exactly the same messup.

@andyross fell free to copy and paste from SOF once it merged and passes QA and validation.

@andyross fell free to copy and paste from SOF once it merged and passes QA and validation.


use new class smart-select-value to set selected option in Smart Select

Smart Select automatically sets div.item-after to be the selected option. But there is another use case missing that is to set selected option based on a value. This change is to implement the missing use case.
Adding a value to div.item-after and also add a new class smart-select-value to it will set the select option which matches the value to be selected.

Merged, thanks


MAINT: interpolate: no need to pass dtype around if it can be inferred

Follow the "Type checking specializations" section of http://docs.cython.org/src/userguide/fusedtypes.html
and stop passing dtype around.

This looks OK to me so I've merged it.  I'm not sure what version of Cython this requires, but it looks like the scipy cython requirement will be bumped soon anyway https://github.com/scipy/scipy/pull/4753.


[PE-D][Tester A] Unclear documentation of find command

find command can't search by phone number or email address, but this is not specified in the command instruction both inside the app and in the user guide, which may cause quite a lot confusion to users. The user guide says "only name is searched", but the examples in the app use courses as search keyword and didn't mention which keyword can be used to search and which cannot.
Suggestion:
Update user guide and error message inside the app.

Labels: type.DocumentationBug severity.High
original: Rachelcoll/ped#11

Update UG/DG: Find Command


cephfs: Add support to create RWX PVC from ROX PVC

This PR blocks the creation of ROX clone from ROX volume as there is no use case currently. Add support to create an RWX clone from the ROX clone, in ceph No subvolume clone is created when the ROX clone is created from a snapshot just an internal ref counter is added. This PR allows creating an RWX clone from an ROX clone which allows users to create an RW copy of PVC where cephcsi will identify the snapshot created for the ROX volume and create a subvolume from the CephFS snapshot.

/test ci/centos/mini-e2e/k8s-1.28

/test ci/centos/mini-e2e/k8s-1.28

/test ci/centos/mini-e2e/k8s-1.28

/test ci/centos/mini-e2e/k8s-1.28

Finally CI passes 👍🏻

@Mergifyio queue


Update app-templates.md

Replaced broken "adaptive card" link
Fixes: #1777

Docs Build status updates of commit 060cc89:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/samples/app-templates.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Hi @Jarbas. Thank you.
The relative path was missing the file extension — .md:
/task-modules-and-cards/cards/cards-reference#adaptive-card
should be
/task-modules-and-cards/cards/cards-reference.md#adaptive-card
I updated the link while I was working in the app templates file.
Thanks again!

Thanks @laujan for your explanation and sorry for missing this part!


Can net framework or standard2.0 support be added

As shown in the question,
many industrial software still remain in the net framework

Hey @XtremlyRed , I'm not willing to support legacy. But you can fork and downgrade version. You will need to change some syntax sugar and don't use Span (maybe something else that I don't remember)


imports that conflict with dart: libs should give warnings

In file conflictsWithDart.dart:
library conflictsWithDart;
class Duration {
  static var x = 100;
}
In file testDartConflicts.dart:
library testDartConflicts;
import 'conflictsWithDart.dart';
main() {
  print("Hail Caesar ${Duration.x}");
}
This should give a warning about the conflict between Duration from dart:core and the one defined in conflictsWithDart, but it does not.

What do we do when the warning says this?
WARNING: Element 'List' from SDK library 'core.dart' is implicitly hidden by 'core.dart'.


bug with loop option in playlist

I use https://github.com/JeyKeu/mep-feature-playlist for creting playlist and i found bug in file mediaelement-and-player.js (probably in other core files) it's line 2574 and code t.media.pause(); - make pause after each track is end of playing, and react to play in code if (t.options.loop) { t.media.play(); but it's mistake because if a want to play next track automaticaly in playlist and select "loop" option to false(because i realy don't want to play all my long playlist again) i can't write extend function to bind "ended" event correctly for my. My decision is just remove line 2574. and behaviour becomes normal.
Sorry for bad english.

@ghost Closing old issues. If you'd like to reopen this, let us know. Thanks!


Add a login page for upload and news editing

Add a login page that is only triggered if someone tries to navigate to the upload pages or the news editing pages. Typically this means wrapping the routing in an authorisation wrapper.
Acceptance Criteria

[ ] If the user is not logged in and tries to go to the upload pages then they are redirected to the login page, upon successful login then they are directed back to the upload page.
[ ] If the user is logged in and goes to the upload page then it will let them through.
[ ] If the user isn't logged in and goes to the news editing page then they are redirected to the login page, upon successful login they are directed back to the news editing page.
[ ] If the user is logged in and goes to the news editing page then it will let them through.

Superseded by other stories


fix(headless): Add export for category recommendation strategies

In this PR I forgot to add the exports.
Don't know why there is so many commits, all but the last one should already be in master.

For the extra commits, did you sync your branch with the latest changes on master?

That was the issue, I can still see the commits but at least it removed all the noise from the files. Thank you.


Fix go vet faults

Fixes a few composite literal uses unkeyed fields errors that occurred when I ran make vet locally; for some reason, make vet has still been passing in CI builds.

LGTM

LGTM

let L-G-T-M work

let L-G-T-M work


feat(sdk/elixir): apply +ignore to sdkSourceDir

Make Elixir SDK consistent with PHP (in #8369), the include patterns is now moved to +ignore directive.

👋 @wingyplus
We know that the job cancellations have been super disruptive. We are planning on changing all jobs to use on-demand instances, so that they don't get cancelled. @jedevc is looking into it.


Height-based mob spawning configurability

Certain spawning limitations designed for vanilla worlds may turn out completely nonsensical in certain CustomCubic presets, such as ocelot and squid spawning, and ashens and dwellers from Quark. It may be a good idea to make these customisable somehow to better fit customised world generation

Biome-based generation could also potentially be configured. I think all of this functionality might be better off as a CC addon mod in itself like CustomCubic is, which could maybe include other world features such as sky and cloud colour.

Sounds a lot like Vertical Spawn Control (https://www.curseforge.com/minecraft/mc-mods/vertical-spawn-control), and possibly the sky color stuff in Gravity Zones (https://www.curseforge.com/minecraft/mc-mods/gravity-zones).

At the very least, I do believe some mobs with height-based spawning conditions (such as squids) should - by default - ignore said limitations in CC, if they aren't a necessity (why are squids height-based anyway?). Currently, I'm using InControl to spawn squids, which performs the job adequately, but the dependency of another mod is suboptimal.


Jesse: Watch Link[\W_]*+Building

Jesse requests the watch of the watch_keyword Link[\W_]*+Building. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Link[\W_]*+Building has been seen in 103 true positives, 15 false positives, and 0 NAAs.


[MOBILE][CREATE PROPOSAL] "Description" field has a scrollbar although the field is empty

Tested on on mobile device Pixel 5. Android 12. Google Chrome
STR:

Open DAO
Click on Create proposal button

Expected result
"Description" field has no scrollbar
Actual result
"Description" field has a scrollbar

Verified on dev. Fixed


Feature Request: Build in some priority

Is it possible to give a deferrable load priority?

No, this is not possible.
You can achieve this by making different optimization calls during the day and providing different functioning times for your deferrable loads. You can do this using automations. I don't that we should mess with this in the code but if you have any proposition I'm open to discuss.


lua51Packages.luaevent: 0.4.3 -> 0.4.4

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 0.4.4 with grep in /nix/store/p0130dpjhicsqkx4dxhi707893lgq8bi-lua5.1-luaevent-0.4.4
directory tree listing: https://gist.github.com/41f797478b7c735e48a957dd7c2166ec

cc @k0ral for review

Failure on x86_64-darwin (full log)
Attempted: lua51Packages.luaevent
Partial log (click to expand)
      _buffer_event_push in buffer_event.o
      _event_buffer_read in event_buffer.o
      _event_buffer_write in event_buffer.o
      _getSocketFd in luaevent.o
      _luaevent_addevent in luaevent.o
ld: symbol(s) not found for architecture x86_64
clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [Makefile:25: all] Error 1
builder for '/nix/store/bwscn4zaf754aggydkly4ddcjh4bbcks-lua5.1-luaevent-0.4.4.drv' failed with exit code 2
error: build of '/nix/store/bwscn4zaf754aggydkly4ddcjh4bbcks-lua5.1-luaevent-0.4.4.drv' failed

Success on aarch64-linux (full log)
Attempted: lua51Packages.luaevent
Partial log (click to expand)
mkdir -p /nix/store/zrmbrhj2f1wpkxk73xjml5cnnml0kb1g-lua5.1-luaevent-0.4.4/lib/lua/5.1/luaevent/
install core.so /nix/store/zrmbrhj2f1wpkxk73xjml5cnnml0kb1g-lua5.1-luaevent-0.4.4/lib/lua/5.1/luaevent/core.so
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/zrmbrhj2f1wpkxk73xjml5cnnml0kb1g-lua5.1-luaevent-0.4.4
shrinking /nix/store/zrmbrhj2f1wpkxk73xjml5cnnml0kb1g-lua5.1-luaevent-0.4.4/lib/lua/5.1/luaevent/core.so
strip is /nix/store/3zq400fri5dv7d30lpxlqm2v9y1iis6j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/zrmbrhj2f1wpkxk73xjml5cnnml0kb1g-lua5.1-luaevent-0.4.4/lib
patching script interpreter paths in /nix/store/zrmbrhj2f1wpkxk73xjml5cnnml0kb1g-lua5.1-luaevent-0.4.4
checking for references to /build in /nix/store/zrmbrhj2f1wpkxk73xjml5cnnml0kb1g-lua5.1-luaevent-0.4.4...
/nix/store/zrmbrhj2f1wpkxk73xjml5cnnml0kb1g-lua5.1-luaevent-0.4.4

Success on x86_64-linux (full log)
Attempted: lua51Packages.luaevent
Partial log (click to expand)
mkdir -p /nix/store/p0130dpjhicsqkx4dxhi707893lgq8bi-lua5.1-luaevent-0.4.4/lib/lua/5.1/luaevent/
install core.so /nix/store/p0130dpjhicsqkx4dxhi707893lgq8bi-lua5.1-luaevent-0.4.4/lib/lua/5.1/luaevent/core.so
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/p0130dpjhicsqkx4dxhi707893lgq8bi-lua5.1-luaevent-0.4.4
shrinking /nix/store/p0130dpjhicsqkx4dxhi707893lgq8bi-lua5.1-luaevent-0.4.4/lib/lua/5.1/luaevent/core.so
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/p0130dpjhicsqkx4dxhi707893lgq8bi-lua5.1-luaevent-0.4.4/lib
patching script interpreter paths in /nix/store/p0130dpjhicsqkx4dxhi707893lgq8bi-lua5.1-luaevent-0.4.4
checking for references to /build in /nix/store/p0130dpjhicsqkx4dxhi707893lgq8bi-lua5.1-luaevent-0.4.4...
/nix/store/p0130dpjhicsqkx4dxhi707893lgq8bi-lua5.1-luaevent-0.4.4


[Feature Request] Implement type defined easy access query parameters.

Current Behavior

Originally from discord
I'm talking about the type definition, and the controller receiving a predictable variable, exactly like the example above.
Everything in Robyn happens through a request dictionary, just like flask. meaning you don't know the keys, don't know their types, and don't have autocomplete
# This is Robyn
@app.post("/jsonify/:id") # the param ID
async def json(request): # you get a "request" dictionary, without any types
    print(request["params"]["id"]) # you access the param through the dict key, you don't validate the key type
    return jsonify({"hello": "world"})

# This is FastAPI
@app.post("/jsonify/{id}") # the param ID
async def json(id: int): # you excplitly tell the param and it's type (auto completion and type safety)
    print(id) # you already have the ID in hands, (auto complete)
    return jsonify({"hello": "world"})

Desired Behavior

Add type defined query params.
Screenshots / Mockups

Alternatives

Step 1: Splitting the request object into **kwargs

This seems pretty interesting. Mind if I try my hands on this, @sansyrox?

Interesting, it seems I might have to tweak the Rust code for the first time.

@guilefoylegaurav , yep yep. Lmk if you need any help :D

Hey @sansyrox while I have been working on this issue since quite some time, I haven't quite arrived at a solution yet. While I shall be working on this to dig out a solution, let make this issue open to all (when someone is assigned an issue, others presume that the assignee shall get the issue solved and thus don't think about taking the issue up) - in the interest of updating the codebase quickly and efficiently

No worries, @guilefoylegaurav . 😄
I have a decent understanding regarding ot this issue. If anyone picks this up, feel free to tag me here again. 😄

Is it?
It would be great if we can discuss the approach then.
I have been trying to experiment with calling the handler with the route params directly, but have been running into some confusing issues.
Also, the obsessed coder in me isn't willing to let go of the issue without a fight

Haha, that's great to hear 😊
Can you text me your exact code approach either here or on discord and the we can work from there??

I was bored and decided to try this one out, and I have managed to not only convert the request object to **kwargs, but also perform rudimentary type checking (to see if the request body has argument names and types matching that of the function body) and did some investigation on how to extend the dependency injection system to FastAPI levels (with or without Pydantic). However, I am stuck on the part where we are calling the handler function; for some reason the handler.call is not accepting the **kwargs? Everything compiles, so it's not an issue of conversion into a PyDict (for **kwargs usage)

Nevermind, I fixed it. Will create a PR that minimizes breaking changes and go back to working on my original issue. Msgspec's JSON validation does have interesting applications with query parameter validation though?

Thank you @kliwongan 😄 . Apologies for not reverting earlier; things were a bit hectic at my day job, and couldn't revert earlier.

Msgspec's JSON validation does have interesting applications with query parameter validation though?

Interesting in a good way or interesting in a bad way?

Interesting in a good way. We can not only use msgspec for JSON serialization/deserialization, but also use the schema features to validate the request body against the function signature using their dynamic struct creation feature https://jcristharif.com/msgspec/structs.html#runtime-definition


Deploy-Sql-SecurityAlertPolicies - Doesn't Allow E-Mail Addresses To Be Provided As An Input

Describe the bug
Deploy-Sql-SecurityAlertPolicies - Doesn't Allow E-Mail Addresses To Be Provided As An Input. It currently has a hard-coded value. (check line 9456 in policies.json)
Screenshot

Proposed Fix
Needs to be changed to an array input parameter and also added to the initiative definition of Deploy-Sql-Security as an input parameter.

The additional missing settings regarding storage are also a little odd given we configure these in the Deploy-Sql-VulnerabilityAssessments Policy Definition.
Given we don't have this Policy Definition linked to a Policy Assignment (either directly or via the Deploy-Sql-Security Policy Initiative referencing it), is this and its related policies still needed? What was the reason for removing them?
Any insights please @krnese and @uday31in? Were these removed as not production-ready, or just deemed no longer necessary? I see we have similar built-in policies, but these only cover the AuditIfNotExists effect.
I've also found in testing that the Deploy-Sql-Tde Policy Definition generates the following error during remediation, so this will also require modification of the policyRule to exclude these databases:
Cannot encrypt a system database. Database encryption operations cannot be performed for 'master', 'model', 'tempdb', 'msdb', or 'resource' databases.

Trigger ADO Sync 1

Trigger ADO Sync 2


[io2d] add new library to ports

This adds the 'io2d' reference implementation, as found (currently) at https://github.com/cpp-io2d/P0267_RefImpl , to vcpkg.
This has been tested on Windows.  Mac support is still being worked on (with some dependent-libraries, namely GraphicsMagick, not yet fully-working in vcpkg).

Thanks!


Dashboards are not imported when using k8s deployment

When I use k8s deployment to run the integration then dashboards are not being installed.

Hi @mfranczak  the issue has been addressed with latest PR merge. Dashboards will be installed with K8S deployment if ReadConfiguration / WriteConfiguration scopes are granted to DT token.


Add ASCII-art for cards in hand

https://github.com/Ostronkex/100-Days-of-Code-The-Complete-Python-Pro-Bootcamp-Solutions/blob/365516a4a0ec5929afa7ba8e7377a437fb054776/Day11/Blackjack.py#L164
Add ASCII-art as a placeholder for the cards in hand instead of numbers for a prettier experience.

I started on this but didn't know how I'd solve the counting part
I could change the cards_values list to unique values such as H2, H3 etc and grab that index and input that index into a print statement that prints the corresponding ASCII-card. However this means there is no value to count the total with. How do I assign a value to either the ASCII art or the H2, H3, H4 etc?
I received a suggestion to remove the letter from H2, H3 by using .remove[0] leaving just the value. For J D (Queen) K A, I would make a dictionary storing the letters and values as key:value pairs. Calling these values when counting made my head spin so I gave up for now. Will return to this in the future if a simpler solution presents itself.


Resizing the window causes the title text to mis-align

If you resize the window the white-on-color text will not align with the colored-on-white text.  Changing the source will re-align the two sets of title text.   I found this problem on Chrome and Firefox.

@IrosTheBeggar Good catch. Thanks for reporting!


mattermost: 5.9.0 -> 5.15.0

Motivation for this change
Software update
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@jslight90 , is this still WIP?

@jslight90 , is this still WIP?

No, I forgot to update it.  We have about 50 users and have been using 5.15.0 for about 2 months without any issues.

It sounds like this can be merged.

This version did not work for me: https://github.com/NixOS/nixpkgs/issues/89194#issuecomment-636238242


Use env APP_NAME variable for syslog identifier

Instead of hardcoded syslog identifier "laravel" I propose to use environment variable APP_NAME.
In case when you have a few Laravel projects on the same machine that log to syslog all of them will write to log with same identifier "laravel" thats not very transparent which message belongs to which project

@akaroot I feel like we should add a fallback of 'laravel'.
$log->useSyslog($this->app->make('config')->get('app.name', 'laravel'), $this->logLevel());

While unlikely that this config key is missing, we don't want an empty value if it is missing.

@browner12 useSyslog method already  has a  fallback for null name
 public function useSyslog($name = 'laravel', $level = 'debug', $facility = LOG_USER)

@akaroot that value will be overriden if config('app.name') is null, but inside config/app.php there is already a fallback for the app name.

Would it make sense to change this to use a hyphen seperated string? When using spaces, it looks kinda weird in Papertrail. Not sure about other services.
Eg. 'My Cool App' would be: my-cool-app, similar to other services (systemd-logind etc)
Papertrail also uses some logic to filter or group by name, which fails when using spaces..


Automatically generated version on build

I think we can learn Golang's version management.
On master, all the version numbers will be gave on build and the version number is the commit hash. Since the hash is too long, we can use the first 7 characters.  i.e. tip+2b445c7
On release branch, all the version number will be gave on build and the version number is branch name plus git hash. i.e. 1.0+2b445c7
For a tag version i.e. 1.0.0, the version number could be static and we can give it a special number not a git hash.
So that, when a user report an issue, we can know the question is from which version quickly.

In our project we use git describe --tag and build with
$ go build -ldflags "-w -s -X main.Version=${VERSION}"

+1 for using git describe (maybe git describe --always in case
tags are not present). NOTE: at the moment, tags are not present...

It seems it's a good idea

So we should create a make.bash & make.bat instead a Makefile? @tboerger

We can accomplish that with the Makefile as well, I would just add the version part to it.

Makefile ask people to install make system first. Many gophers come from Ruby and Python not C or C++. And since build Gitea is so easy that it's no need a Makefile. Or we can move it to 1.1.0?

+1 to move it to 1.1.0
BTW, we still need version embedded in a .go file for "go get"
to end up with a proper version printed in pages, no ?

Go get should simply have 0.0.0+master

But most gophers also got build-essentials installed which also includes make

On Mon, Nov 07, 2016 at 06:31:56AM -0800, Thomas Boerger wrote:

Go get should simply have 0.0.0+master

How about 1.0.0-dev, and incremented after release ?

Because we should always provide valid version numbers.

is 1.0.0-dev invalid ?

Yes

According to which specification is 1.0.0-dev invalid ?
Should maybe be 1.0.0+dev or something similar ?

go build on branch master: 1.0+dev
go build on branch v1.0: 1.0.0+dev
go build on tag v1.0.0: 1.0.0
make on branch master: 1.0+
make on branch v1.0: 1.0.0+
make on tag v1.0.0: 1.0.0

@lunny that means never having the commit-hash !
I think commit-hash should always be there except for builds on tag

These are my 2 cents, because we should always keep valid semver versions:
go build on branch master: 1.0.0+dev, 1.1.0+dev, 1.2.0+dev
go build on branch v1.0: 1.0.0+dev, 1.0.1+dev, 1.0.2+dev
go build on tag v1.0.0: 1.0.0+631c18a
make on branch master: 1.0.0+631c18a, 1.1.0+631c18b, 1.2.0+631c18c
make on branch v1.0: 1.0.0+631c18a, 1.0.1+631c18b, 1.0.2+631c18c
make on tag v1.0.0: 1.0.0+631c18a
These are also valid versions for most packages like rpm or deb.

On Mon, Nov 07, 2016 at 07:26:56AM -0800, Thomas Boerger wrote:

These are my 2 cents, because we should always keep valid semver versions:
go build on branch master: 1.0.0+dev, 1.1.0+dev, 1.2.0+dev
go build on branch v1.0: 1.0.0+dev, 1.0.1+dev, 1.0.2+dev
go build on tag v1.0.0: 1.0.0+631c18a
make on branch master: 1.0.0+631c18a, 1.1.0+631c18b, 1.2.0+631c18c
make on branch v1.0: 1.0.0+631c18a, 1.0.1+631c18b, 1.0.2+631c18c
make on tag v1.0.0: 1.0.0+631c18a
These are also valid versions for most packages like rpm or deb.

+1, it works for me.

go build on tag v1.0.0: 1.0.0+631c18a
make on tag v1.0.0: 1.0.0+631c18a
I think only 1.0.0 is ok for tags.

@lunny that'd be even better but I'm afraid go build cannot possibly include the hash w/out changing a version file on each and every commit. make instead would extract it dynamically


Indexing custom fields from a sitemap

Hi,
We have created a sitemap with custom fields. We would like to understand how to have the custom fields being added in the indexed documents.
Thank you

I cannot think of an out-of-the-box way to do so with custom fields, but I can think of a workaround if you know your Java.
You could crawl your sitemap as a start URL, and write your own ILinkExtractor.  The link extractor produces links with some predefined metadata fields that will be stored with the document.  You could highjack one of those fields to store your own metadata in it (e.g. the "text" attribute of the produced Link objects.   Then you would use one of the manipulation options in the Importer module to split your custom metadata values into their own fields.
Not the most straightforward for sure.   We can make this a feature request too.

Thank you, Pascal for your response.
A new feature request would be a better solution.

+1 to this feature request.


Homebridge does not receive the room IDs

Hi,
i just setup my roborock S5 to run with HomeKit via Homebridge.
Everything seems to work except the "get room ID".
[15.4.2024, 10:28:46] Homebridge v1.7.0 (HAP v0.11.1) (Homebridge 0054) is running on port XXXX. [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] STA getDevice | Connected to: XXXXXXXXXXXX [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] STA getDevice | Model: roborock.vacuum.s5 [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] STA getDevice | State: charging [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] STA getDevice | FanSpeed: 101 [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] STA getDevice | BatteryLevel: 100 [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] MON changedCharging | ChargingState is now true [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] changedCharging | Charging is active [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] changedCharging | Robot was docked. [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] MON changedSpeed | FanSpeed is now 101% [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] WAR changedSpeed | Speed was changed to 101%, this speed is not supported [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] getSerialNumber | Serial Number is XXXXXX [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] STA getDevice | Serial Number: XXXXXX **[15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] INF getRoomMap | Map is**  [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] getFirmware | FirmwareVersion is 3.5.8_002034 [15.4.2024, 10:28:46] [RoboRock] [Model=roborock.vacuum.s5] STA getDevice | Firmware Version: 3.5.8_002034 [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCleaning | Cleaning is false [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareSensors | Sensors dirtyTime is 39428 seconds / 36.51%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareFilter | Filter workTime is 166137 seconds / 30.77%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareSideBrush | SideBrush workTime is 166137 seconds / 23.07%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareMainBrush | MainBrush workTime is 166137 seconds / 15.38%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareSensors | Sensors dirtyTime is 39428 seconds / 36.51%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareFilter | Filter workTime is 166137 seconds / 30.77%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareSideBrush | SideBrush workTime is 166137 seconds / 23.07%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareMainBrush | MainBrush workTime is 166137 seconds / 15.38%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCharging | Charging is true (Status is charging) [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getDocked | Robot Docked is true (Status is charging) [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareSensors | Sensors dirtyTime is 39428 seconds / 36.51%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareSensors | Sensors dirtyTime is 39428 seconds / 36.51%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareFilter | Filter workTime is 166137 seconds / 30.77%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareFilter | Filter workTime is 166137 seconds / 30.77%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareSideBrush | SideBrush workTime is 166137 seconds / 23.07%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareSideBrush | SideBrush workTime is 166137 seconds / 23.07%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareMainBrush | MainBrush workTime is 166137 seconds / 15.38%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getCareMainBrush | MainBrush workTime is 166137 seconds / 15.38%. [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getBatteryLow | BatteryLevel is 100% [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getBattery | BatteryLevel is 100% [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getPauseState | Pause possible is false [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getSpeed | Fanspeed is 101 over miIO. Converting to HomeKit [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getSpeed | FanSpeed is 101 over miIO "Quiet" > HomeKit speed 40% [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getFirmware | FirmwareVersion is 3.5.8_002034 [15.4.2024, 10:28:47] [RoboRock] [Model=roborock.vacuum.s5] getSerialNumber | Serial Number is R0018S81012969 [15.4.2024, 10:28:51] [RoboRock] [Model=roborock.vacuum.s5] getFirmware | FirmwareVersion is 3.5.8_002034 [15.4.2024, 10:29:20] [RoboRock] [Model=roborock.vacuum.s5] getCleaning | Cleaning is false [15.4.2024, 10:29:20] [RoboRock] [Model=roborock.vacuum.s5] getSpeed | Fanspeed is 101 over miIO. Converting to HomeKit [15.4.2024, 10:29:20] [RoboRock] [Model=roborock.vacuum.s5] getSpeed | FanSpeed is 101 over miIO "Quiet" > HomeKit speed 40%

@gothma, since you implemented this feature. Would you mind helping out?

@vanGeek Thanks for providing the log. It seems to me that you haven't set autoroom to a list of room names, since according to the log the fully automatic (getRoomMap) not the semi automatic (getRoomList) method is used, which doesn't work with the S5.
Can you ensure you followed the semi-automatic instructions?
If it doesn't work can you provide a copy of the config without secrets?


Tooltip appears too far from the hovering element

[x] The issue is present in the latest release.
[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
tooltip is very far from the place where i am hovering

Expected Behavior 🤔
there could be a prop called displacement which tell how far the tooltip needs to be present.

Steps to Reproduce 🕹
when i click the tooltip is appearing which is very far and not able to identify which one i hovered.

https://codesandbox.io/s/i5f8h

Context 🔦

Your Environment 🌎

`npx @material-ui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @material-ui/envinfo` goes here.

This is done on purpose. If the distance was smaller, the tooltip would be covered by the cursor.

If you still want to change this, you can apply margin: 0; to the tooltipPlacementBottom rule.
const useStyles = makeStyles({
  tooltipBottom: {
    margin: 0
  }
});

<Tooltip
  classes={{
    tooltipPlacementBottom: classes.tooltipBottom,
  }}
>

See the demo.

Perfectly answered by @michal-perlakowski. Closing since this is expected behavior that can be customized.


[SPARK-17738] [SQL] fix ARRAY/MAP in columnar cache

What changes were proposed in this pull request?
The actualSize() of array and map is different from the actual size, the header is Int, rather than Long.
How was this patch tested?
The flaky test should be fixed.

cc @cloud-fan @kiszk

Test build #66136 has started for PR 15305 at commit 8fca8c8.

Test build #66136 has finished for PR 15305 at commit 8fca8c8.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66136/
Test FAILed.

retest this please

LGTM

Test build #66139 has started for PR 15305 at commit 8fca8c8.

Test build #66139 has finished for PR 15305 at commit 8fca8c8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66139/
Test PASSed.

Merged build finished. Test PASSed.

Merging this into master.

I am sorry for my mistake, and thank you very much for quick catch.


Question

On June 29, 2022, I wrote a list of most of the bugs, flaws, and suggestions about new mobs' abilities. Today is February 22, 2023. Have you corrected the mistakes and added some of the suggestions, or have you abandoned the mod?

I did actually fix some of the issues you've noted in your previous post (like the ability for striders to walk on lava), though this is only present in the newest build of the game currently available on curseforge.
As for your second question, the mod is neither abandoned nor in active development, but rather something inbetween.
I hope I could answer your questions with this post :+1:


OP_REQUIRES failed at conv_ops.cc:1106 : Not found: No algorithm worked!

Hi All,
I am trying out "digit-recognition-kaggle-competition" in charmed kubeflow using RTX3060 on Ubuntu 22.04
https://github.com/kubeflow/examples/tree/master/digit-recognition-kaggle-competition
It is failing with the below error. Any ideas what I am doing wrong here ?
history = model.fit(np.array(X_train), np.array(y_train),
                    validation_split=.1, batch_size=int(BATCH_SIZE), epochs=int(EPOCHS))

2023-06-14 21:19:07.654236: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:116] None of the MLIR optimization passes are enabled (registered 2)
2023-06-14 21:19:07.654500: I tensorflow/core/platform/profile_utils/cpu_utils.cc:112] CPU Frequency:<PHONE_NUMBER> Hz
Epoch 1/2
2023-06-14 21:19:07.897998: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcublas.so.11
2023-06-14 21:19:08.249471: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcublasLt.so.11
2023-06-14 21:19:08.251540: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudnn.so.8
2023-06-14 21:19:08.955292: W tensorflow/core/framework/op_kernel.cc:1763] OP_REQUIRES failed at conv_ops.cc:1106 : Not found: No algorithm worked!

Same model using CPU works fine instead of GPU works just fine

Hi @ACodingfreak, could your issue be the same as https://github.com/tensorflow/tensorflow/issues/45044? It sounds like it.
Since this looks more like an issue related to the ML framework and your setup than to the charms, I am closing this issue, but feel free to file a new one if you have more questions related to Charmed Kubeflow.


Swapped out linear algebra GW loss term for FFT approach.

Updated GW loss to use the FFT approach.  One important distinction is that we now calculate the air phase and amplitude yearly within the loss function instead of beforehand in the data prep.

just adding a note to this conversation that Simon and I are working on debugging this. He can get it to work locally and on TG. I am getting errors on both pangeo and TG.

seems to be a cpu / gpu issue. the fft approach fails on the gpu's and runs correctly on cpu's


bump hops to <IP_ADDRESS>-snapshot

Closing this. Changes have been included in: https://github.com/logicalclocks/hive/pull/7


[Bug][Manager] Create group workflow fail

What happened

What you expected to happen
Not fail
How to reproduce
Create any group
Environment
No response
InLong version
master
InLong Component
InLong Manager
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I will deal with this problem.


Resolve binary path for firejail arg

Hey,
when using orjail with firejail enabled like this I get following error on NixOS:
$ sudo orjail -f chromium
Error: no suitable chromium executable found

On NixOS the command chromium has the following path:
$ which chromium
/home/onny/.nix-profile/bin/chromium

But this is a symlink to a different path
$ readlink -f /home/onny/.nix-profile/bin/chromium
/nix/store/vbvrcmz9rp6vz8dbylki19irkask5gn7-chromium-97.0.4692.99/bin/chromium

If I supply the absolute path, starting chromium with orjail and firejail enabled works.
This PR uses both commands to resolve the absolute path and uses it as argument for the firejail command.

Maybe this should also get fixed upstream in firejail?

I'm closing this since this is a NixOS specific issue and I guess it is better to fix it in the firejail package

no :) Don't close this! Let's work on it. Maybe we can check if $0 is a link. And only in that case, we call readlink. Or maybe we can check if which returns an error, and we call readlink only if it succeeds.

Thank you for your help and willingness to fix this issue :D If opened an other issue because I guess this is an upstream issue with NixOS packaging https://github.com/NixOS/nixpkgs/issues/170784
This would fix the issue both for firejail and orjail on NixOS :)


Facebook auth fails if using the same email as an existing Google user

Login doesn't finish if using facebook auth with google auth.
Loading indicator doesn't appear in authViewController.

Can you share a project that reproduces this issue?

@morganchen12
Thank you for your reply.
I created sample project. Please check this repository.
If sign in with a facebook account with the same address as Gmail, login doesn't be completed.
https://github.com/akifumi/FUISample675

Hi @akifumi , I don't understand "using facebook auth with google auth". Could you please detail the steps to reproduce?

Hi, I think I'm having the same problem. It doesn't happen always, for me it works, but every time my workmate tries to login with his account, the app gets stuck on the Auth Picker View Controller like in the image:

He has the Facebook app installed, if it makes any difference. For me it works on my phone with or without FB app.
Thanks for the help.

@jmmartinezdev are there any errors in console when this happens?

@morganchen12 no I don't get anything on the console. It looks like a background operation is frozen, though the UI is still responsive and login can be cancelled.

Can you reproduce this by installing the Facebook app on your phone (or fix the issue by uninstalling the Facebook app from your coworker's phone)?

Closing due to inactivity, but will re-open if there are any updates.

This situation occurs if FUIEmailAuth is not added.

Yup, including FUIEmailAuth resolved it for me.

I'm having this same problem, 100% repeatable.
If I create an account with Google, I can't then sign in with Facebook using the same email address.  And it's not specific to iOS either - I also can't do it in the web or Android version of FirebaseUI.
There are weird combinations, though.  Like on the web, if I sign in with Apple and Google, then Facebook sometimes works to join the party.
And if I sign in with Facebook first, it always works.
Anyway, I'll keep this to iOS-specific for now.  At first I was stuck with the spinner blocking the app entirely as described earlier in this thread.  Then I added FUIEmailAuth as suggested, and now the spinner goes away, but instead I get a popup that says "This type of account isn't supported by this app", which makes little sense at all given that this type of account works fine in most cases.
So what's going on here?


Restore old webview provider

Restore old webview provider so ds/cusom_editor can be merged into master.

Bug fix with run all cells
Removed error message from telemetry, as it can contain PII

Separate PRs to:

[x] Restore tests

[x] Double check all
[x] Save As
[x] Save
[x] Hot exit
[x] Undo/redo
[x] Keyboard shortcuts (z)

[x] Opening ipynb in notebook
[x] Open all ipynb in notebook editor after extension activates
[x] Restore undo/redo or similar (what ever has been removed)
[x] Restore support for saving (save and save as)
[x] Restore hot exit (saving when existing VSC and restoring after opening VSC)
[x] Restore other features that have been removed/disabled as a result of using the new VSC custom editor (i.e. restore old webview approach functionality)

Note: Custom editor has been disabled in this PR.
Once merged into master, I'll bring them back as a subsequent PR.

Codecov Report

:exclamation: No coverage uploaded for pull request base (ds/custom_editor@ca10f93). Click here to learn what that means.
The diff coverage is 36.53%.

@@                 Coverage Diff                 @@
##             ds/custom_editor   #10239   +/-   ##
===================================================
  Coverage                    ?    60.8%
===================================================
  Files                       ?      567
  Lines                       ?    30603
  Branches                    ?     4365
===================================================
  Hits                        ?    18607
  Misses                      ?    11048
  Partials                    ?      948

Impacted Files
Coverage Δ

src/client/telemetry/index.ts
87.38% <ø> (ø)

src/client/datascience/types.ts
100% <ø> (ø)

...ient/datascience/interactive-ipynb/nativeEditor.ts
11.06% <0%> (ø)

...c/client/common/application/customEditorService.ts
33.33% <20%> (ø)

...ascience/interactive-ipynb/nativeEditorProvider.ts
53.37% <47.22%> (ø)

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 ca10f93...4871f39. Read the comment docs.

@rchiodo @IanMatthewHuff Please have another look

This will be merged into ds/custom_editor.
Then i'll merge ds/custom_editor into master
Finally, enable custom editor in ds/custom_editor


feat: add configs in setup

See https://github.com/mikesmithgh/kitty-scrollback.nvim/issues/12#issuecomment-1729696275

Thanks, that's interesting, I didn't realize that the plugin uses a separate config file. Are there any plans to go with the conventional require("kitty-scrollback").setup(...) configuration method? That'd make configuration a lot easier.

I originally used separate config files because the kitten (written in python) starts Nvim and passes kitty specific info as arguments. I did some experimenting and think I could get the conventional setup method to work and agree it would be cleaner.
require('kitty-scrollback').setup({
  default = function(kitty_data)
    return { ... default config here ... }
  end,
  another = function(kitty_data)
    return { ... another config here ... }
  end,
  ...
})

Then the kitten would be called like kitty.conf
map ctrl+shift+h kitty_scrollback_nvim --config default
map f1 kitty_scrollback_nvim --config another

That way you could centralize all your configs in the setup and then specify the particular config by key. @musjj Does that sound like a good approach to you or do you have any other ideas?

Yes, it would be nice if there's an option to configure this too!

Nice, I'll add an issue for that

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[3849] Strip trailing leading whitespace from address fields

Context
We strip surrounding spaces from some of the trainee fields when the user enters them.
Changes proposed in this pull request

Expand the list to include address fields and other user entered data and strings that we may import from HESA, DTTP etc
Add the stripping to Degree as well while we're at it
Fix a flakey spec

Guidance to review

Enter a trainee address with extra spaces. They shouldn't be persisted to the database.

Important business

~Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?~
~Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?~

 [M] Strip trailling/leading whitespace from address fields


[Auto-generated] Update dependencies

Update dependencies
Automatically created PR from ci/dependency-updates.
For more information see the "Dependabot updates" workflow.
To-do

[ ] Check that the diff is sensible, and that tests and builds pass with the new dependency versions.
[ ] Make sure that the PR is squash merged, with a sensible commit message.

Codecov Report

Merging #57 (dcbee9a) into master (b0444d3) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #57   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          210       210
=========================================
  Hits           210       210

Flag
Coverage Δ

linux
100.00% <ø> (ø)

windows
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


add certificate_chain attribute to aws_server_certificate

Add Certificate chain attribute to the aws_server_certificate resource.

This build seems to be failing with no changes, is the build server OK? It appears to be complaining of a missing credentials file.
/home/travis/build/chef/chef-provisioning-aws/lib/chef/provisioning/aws_driver/credentials.rb:20:in `default': No credentials loaded!  Do you have a ~/.aws/config?

@philbrookes We enabled integration tests on Travis but it uses encrypted Travis env variables, and these are only ran on branches in the original org (IE, chef).
Will you re-add your changes?  I can manually run the tests and validate it.

Merged as https://github.com/chef/chef-provisioning-aws/commit/ede1ddc6bfbb6e2e76696090a795e661e6bf499c - thanks for the contribution @philbrookes !


Organize gas payment calculation and create sustainable business model

Description

Currently, only gas payment deduction is implemented
This should check other fees (swap, bridge) and make it sustainable

I think we can deduct

bridge fee
swap fee
actual gas fee

This is the original cost user has to pay, so we are not taking the user's additional money.
But we are doing batch bridge and swap, so the gas fee is much more efficient, so we can make small running money on it.

I'd like to have a staking solution in paymaster.
Users can provide liquidity on the paymaster contract and then get the wrapped token.

I added minimum task to implement #34

I think the future roadmap would be the paymaster decentralization.
In this model, If it gets large liquidity, it gets more efficient, so staking model could be nice.
For the time being, close this issue and try to complete development.


test_build_info_metric is flaky

A recent failure in main: https://github.com/neondatabase/neon/actions/runs/3564596050/jobs/5989355902
main seems failing:passing about 50:50 at the moment, each failure is unique in its flakiness.
____________________________ test_build_info_metric ____________________________
[gw0] linux -- Python 3.9.2 /github/home/.cache/pypoetry/virtualenvs/neon-_pxWMzVK-py3.9/bin/python
subprocess.CalledProcessError: Command '['/tmp/neon/bin/neon_local', 'pageserver', 'start']' returned non-zero exit status 1.

The above exception was the direct cause of the following exception:
test_runner/regress/test_build_info_metric.py:7: in test_build_info_metric
    env = neon_env_builder.init_start()
test_runner/fixtures/neon_fixtures.py:623: in init_start
    self.start()
test_runner/fixtures/neon_fixtures.py:619: in start
    self.env.start()
test_runner/fixtures/neon_fixtures.py:916: in start
    self.pageserver.start()
test_runner/fixtures/neon_fixtures.py:1778: in start
    self.env.neon_cli.pageserver_start(overrides=overrides, extra_env_vars=extra_env_vars)
test_runner/fixtures/neon_fixtures.py:1583: in pageserver_start
    return self.raw_cli(start_args, extra_env_vars=extra_env_vars)
test_runner/fixtures/neon_fixtures.py:1356: in raw_cli
    raise Exception(msg) from subprocess.CalledProcessError(
E   Exception:             Run ['/tmp/neon/bin/neon_local', 'pageserver', 'start'] failed:
E                 stdout: Starting pageserver at 'localhost:15030' in '/tmp/test_output/test_build_info_metric/repo'.....
E   pageserver has not started yet, continuing to wait.....
E
E                 stderr: pageserver start failed: pageserver did not start in 10 seconds
------------------------------ Captured log setup ------------------------------
2022-11-28 12:45:50.907 INFO [neon_fixtures.py:2706] get_test_output_dir is /tmp/test_output/test_build_info_metric
2022-11-28 12:45:50.908 INFO [neon_fixtures.py:2727] test_output_dir is /tmp/test_output/test_build_info_metric
2022-11-28 12:45:50.909 INFO [neon_fixtures.py:2130] starting proxy with command line ::: /tmp/neon/bin/proxy --http <IP_ADDRESS>:15027 --proxy <IP_ADDRESS>:15028 --mgmt <IP_ADDRESS>:15029 --auth-backend link --uri http://dummy-uri
------------------------------ Captured log call -------------------------------
2022-11-28 12:45:51.743 INFO [neon_fixtures.py:910] Config:
default_tenant_id = '5af0573e7b8ba44dde7f3a4c65822a96'

[etcd_broker]
broker_endpoints = ['http://<IP_ADDRESS>:15001/']
etcd_binary_path = '/usr/local/bin/etcd'

[pageserver]
id=1
listen_pg_addr = 'localhost:15030'
listen_http_addr = 'localhost:15031'
auth_type = 'Trust'

[[safekeepers]]
id = 1
pg_port = 15032
http_port = 15033
sync = false
2022-11-28 12:45:51.744 INFO [neon_fixtures.py:1319] Running command "/tmp/neon/bin/neon_local init --config=/tmp/tmpqpy0kl0x --pg-version 14"
2022-11-28 12:45:51.744 INFO [neon_fixtures.py:1320] Running in "/tmp/test_output/test_build_info_metric/repo"
2022-11-28 12:45:52.891 INFO [neon_fixtures.py:1347] Run success: Starting pageserver at 'localhost:15030' in '/tmp/test_output/test_build_info_metric/repo'.
pageserver started, pid: 4785
Successfully initialized timeline e573cded774a95c80cbc39e5223b0554
Stopped pageserver 1 process with pid 4785

2022-11-28 12:45:52.891 INFO [neon_fixtures.py:1319] Running command "/tmp/neon/bin/neon_local pageserver start"
2022-11-28 12:45:52.891 INFO [neon_fixtures.py:1320] Running in "/tmp/test_output/test_build_info_metric/repo"
2022-11-28 12:46:02.967 INFO [neon_fixtures.py:1355]             Run ['/tmp/neon/bin/neon_local', 'pageserver', 'start'] failed:
              stdout: Starting pageserver at 'localhost:15030' in '/tmp/test_output/test_build_info_metric/repo'.....
pageserver has not started yet, continuing to wait.....

              stderr: pageserver start failed: pageserver did not start in 10 seconds
---------------------------- Captured log teardown -----------------------------
2022-11-28 12:46:03.321 INFO [neon_fixtures.py:777] Cleaning up all storage and compute nodes
2022-11-28 12:46:03.321 INFO [neon_fixtures.py:2487] Stopping safekeeper 1
2022-11-28 12:46:03.321 INFO [neon_fixtures.py:1319] Running command "/tmp/neon/bin/neon_local safekeeper stop 1 -m immediate"
2022-11-28 12:46:03.322 INFO [neon_fixtures.py:1320] Running in "/tmp/test_output/test_build_info_metric/repo"
2022-11-28 12:46:03.341 INFO [neon_fixtures.py:1347] Run success: safekeeper 1 is already stopped: no pid file "/tmp/test_output/test_build_info_metric/repo/safekeepers/sk1/safekeeper.pid" is present

2022-11-28 12:46:03.342 INFO [neon_fixtures.py:722] no remote storage was set up, skipping cleanup

This is  fixed this


Handle access_denied from Hello, update README

When a user decides to not proceed with hellō, the user is redirected back to the configured callback URL with the following:
- http://localhost:3000/callback#error=access_denied&error_description=&error_uri=
- https://next-with-hello-leewc.vercel.app/#error=access_denied&error_description=&error_uri=

This PR checks if the hash contains and error, redirects to the backend and returns a 403 if it is denied.
An alternative behavior would be to redirect back to / and have a message bubble saying Hello OAuth flow cancelled.
Docs:

https://www.hello.dev/documentation/errors.html#request-errors

Testing:

Click Login
Continue with Hello
Select an identity provider and then click 'cancel' on the OAuth dialog.
Redirection happens.

Code can be played around here: http://next-with-hello-leewc.vercel.app/ (or copy paste the URL above to simulate the behavior).
I cloned the repo using Vercel (which unfortunately makes the repo a standalone/not a Github fork, so I had to test and then cherry-pick the commit over).

Thank you!


implement anchoring-by-default

This is the successor to #31

[ ] onyo ls should not list .anchor files
[x] onyo tree should not show .anchor files
[ ] onyo new should not create folders by default (see #57)
[ ] onyo mv should not create folders by default (see #57)

onyo mv and onyo new will stop creating folders in an incomming PR, that also removes onyo anchor and onyo unanchor.
onyo ls is not yet implemented.
I made a comment in #54 so it is remembered when it will be implemented, so after the above mentioned PR I will close this issue, since all tasks are either done or the information put in a more specific issue.

This is done and the PR merged, so I close this.


Updated tinyweb for new uasyncio?

Damien has merged the new uasyncio into mainline for ESP32 (and other ports) so I was hoping that we could get a new "compatible" version of tinyweb that uses it.
From what I understand (been told by Jim Mussared) tinyweb is using some internal asyncio APIs that wont carry over to the newer uasyncio. I'm not sure how hard it will be to move it, but the plan is to have the new uasyncio ship in 1.13 (frozen), so things are going to likely break when that happens.
I'm happy to be a bleeding edge tester on this as I'm keeping the project I am working on the latest mainline right now, so I'm happy to break things on my side to help get it done.
Cheers!
Seon

Hey, thanks for heads up!
I'll try to update it this weekend.. )

Wicked, thanks @belyalov !

Hello.
I'm here to test it just push the beta version :-)
Best regards
MvincM

Hey @belyalov, any progress on looking into this? No pressure, just curious.
I see more uasyncio improvements have got into master, so it's get some love from Damien atm, so would be a good time to get any tinyweb/uasyncio issues addressed if needed.
Cheers,
Seon

Hey,
No progress so far - but I'm going to give it a try today.. :)

Made some progress.
It seems working well, however, I hit few issues:

unix ports does not have uasyncio enabled by default, so unable to run unittests at this time.
new uasyncio does not currently have serve_forever() so some workarounds needed from app side

Anyway it is now looks better and really close to python3.x:
import tinyweb
import uasyncio

# Create web server application
app = tinyweb.webserver(host='<IP_ADDRESS>', port=8081)

# Index page
@app.route('/')
async def index(request, response):
    # Start HTTP response with content-type text/html
    await response.start_html()
    # Send actual HTML page
    await response.send('
Hello, world! (table)
\n')

async def main():
    await app.start()

if __name__ == '__main__':
    uasyncio.run(main())
    uasyncio.get_event_loop().run_forever()

Hopefully will get back in few days.. :)
Micropython issue: extmod/uasyncio: add serve_forever()
WIP is on new_uasync_io branch.

Excellent progress, thanks @belyalov !

reader.readexactly() isn't implemented, yet

Hey @belyalov - I guess this isn't going to happen? I've been in a bit of a holding pattern waiting for this, as I need to move to a newer MP version for other fixes/features, and without this being compatible with the new asyncio, I'll have to remove it and use microdot instead, which would suck as I have the features using tinyweb fully implemented in my project already.

@belyalov are there any plans to support the new uasyncio? Much like @UnexpectedMaker my v1.12 firmware has asyncio v3.0 baked in, and I would love to be able to use tinyweb with it.
Thanks for the awesome library :-)


Add more info for the stride value in vector-strided-segment-load/store.

In https://github.com/riscv/riscv-v-spec/issues/172, there is no restriction for the stride value. This pr try to add a note for that.

Merged with minor modifications in 9688e01837c512466ddd73ac9f79147a1fe1b2d8.  Thank you.


[communication]-[sms] Fix opt out remove action

Packages impacted by this PR
communication-sms
Issues associated with this PR
Describe the problem that is addressed by this PR
Fix the error in optOut.remove method
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

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

/azp run js - pullrequest


debug: create type formatter for hstring.RenderedString

For instance in .Title of image render hooks
Hugo Version: 0.124.1

closing all "create type formatter" issues for now.


Improve upload CSV frontend

Before:

After:

Improvements:

Better separation between the different option descriptors
Bulleted list sorted alphabetically is easier to read and review


Cluster reset

We need to implement cluster reset.  neon-desktop and neon-cli are already calling this but the code currently throws a NotImplementedException.

Tasks:

[ ] neon-desktop prompts with: type permanently reset
[ ] Delete and recreate default namespace
[ ] Delete all non-standard namespaces
[ ] Delete and recreate ClusterRegistry CRs from cluster definition
[ ] Reset Harbor
[ ] Reset Minio
[ ] Reset CRI-O on nodes (remove non-standard images)
[ ] Reset Dex
[ ] Reset Grafana

Delete namespace isn't working how I expected:

default namespace cannot be deleted, so we'll need to explicitly remove its contents
just deleting everything in a namespace doesn't always work.  I've seen posts about finalizers causing problems and I think that deployment (and perhaps) other controllers may be sabotaging things by recreating deleted pods.

Namespace deletion is working well now.  I'm in the process of working on resetting CRI-O now.

I've implemented the critical features and have added some todos to the code for the remaining ones and I'm going to move this issue back to the Release Backlog pipeline.

Moving this to the product backlog.  This implementation is good enough for now.


Examples cannot be empty

state Active is {   handler ActiveOrganization is {     on command UpdateParent is {       example Success is {???}     }   } }
gives the following error:
[error] Error: organization.riddl(55:21): Thens in Example 'Success' must not be empty:                     example Success is {???}                     ^

Fixed previous.y


Update syntax-for-issue-forms.md

Added warning to the label attribute description that the label must already be present on the repository in order for this to be automatically suggested on the resulting issue created from the template

Why:
Closes:
github/docs-content/issues/10319

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

👋 @jpritchard Thanks so much for opening a PR! I'll get this triaged for review :zap:


Ignored Meross Matter devices discovered again and again

Dear krahabb,
I am not sure if this is "in your powers" or a matter of hass. A matter device MSS315 (Smart Plug Mini) is constantly being discovered as a new Meross device although I ignored it multiple times. That device is coupled via Matter and I do not want it in Meross LAN.
Could yu please have a look into the matter? What information do you need to investigate on that?
Best regards
Sven

Thanks for fixing!

I hope it'll work since I couldn't test a real dhcp scenario so I've just tried test it with some emulated messages.


Update EasyRTCClient to Foundry VTT 0.7.9

Which class should be updated?
EasyRTCClient
For which version should the class be updated?
Foundry VTT 0.7.9

Taking this

Fixed by #291


Collect errors when deserializing untagged enums

This also adds a verbose-debug feature that enables the new behavior.
Previously:

data did not match any variant of untagged enum ReferenceOr at line 6 column 4

Now, with verbose-debug feature enabled:

data did not match any variant of untagged enum ReferenceOr

attempted to deserialize Reference but failed with: missing field $ref
attempted to deserialize Item but failed with: invalid value: string "lol", expected expected format \dXX at line 6 column 4

cf. #773

See this more as a draft that just happens to help me debug stuff locally :)
Bikeshedding of feature name and error format very welcome.

Just used this to debug some nested enums that had a typo. Very helpful! Thanks!

@dtolnay are you interested in merging this? I'm not sure what Serde's approach to verbose debugging is/whether there is any precedence for this.

What would be holding us back from merging this in? Seems fantastic.

I cannot get this branch to work. Compiling throws a lot of errors like:
error[E0277]: the trait bound `serde_yaml::Mapping: serde::Deserialize<'_>` is not satisfied
  --> src/config.rs:48:2
   |
48 |     #[serde(default)]
   |     ^ the trait `serde::Deserialize<'_>` is not implemented for `serde_yaml::Mapping`
   |
   = note: required by `serde::de::SeqAccess::next_element`

how to use it along with serde_json cause I can override serde version but not serde_json version of serde

You can use [patch] in your Cargo.toml. Check out this section of the rust book: https://doc.rust-lang.org/cargo/reference/overriding-dependencies.html

On Aug 9, 2020, at 4:09 AM, Antoine PLASKOWSKI<EMAIL_ADDRESS>wrote:

how to use it along with serde_json cause I can override serde version but not serde_json version of serde
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

You can use [patch] in your Cargo.toml. Check out this section of the rust book: https://doc.rust-lang.org/cargo/reference/overriding-dependencies.html
…
On Aug 9, 2020, at 4:09 AM, Antoine PLASKOWSKI @.***> wrote:  how to use it along with serde_json cause I can override serde version but not serde_json version of serde — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

This would work cause there would be two version of serde, and so as @piegamesde reported this cause Deserialize to not be implement for serde_json version of serde.

Thanks for doing this work. Out of curiosity, is this behind a flag because the message is too long or because collecting the debugging information comes at a performance cost (or something else)?

Just had a use for this again because the error messages serde gives me when reading in a config file are not helpful to users.
@dtolnay I'd really appreciate if you could have a look at this :)

+1 for adding this or more extensive error messages - some failures are very hard to debug.

Can any of the maintainers take a look at this? Debugging deserialization problems into enums is basically impossible without this.

I know it has been asked already but I think maintainers should seriously consider checking this. It looks like a small change and it will be of huge help to all of us using untagged enums in our projects.

Is there any reason why this is not being considered? This would be so helpful for actually finding errors in YAML files.

I really need something like this. Unfortunately I was not able to use the git version in the PR repo because it's a bit outdated. My use-case: I'm querying an API that either returns an error JSON or the value requested. The API is apparently unstable because after some time, my code no longer works. I'm trying to work out what's not matching in the JSON, but it's really difficult because it's a huge chunk and it almost fits. Is it a type not matching? Is it a field that became optional? I have no way of knowing without looking at each field in my struct and trying to manually match that against the JSON I got.
My workaround is likely going to be: identify an error response without deserializing (i.e. dumb string matching) and then deserializing only the concrete types and use serde_path_to_error to gain some insight.

@dtolnay ?

@dtolnay there seems to be a fair amount of interest for this issue. We'd all appreciate if you let us know what your concerns are, and if you see a path for it to get merged.

For anyone who encounters this issue today, you need a branch where the serde version matches your other dependencies.
I took a similar commit (that uses no compiler feature flag) from this branch: https://github.com/jonasbb/serde/tree/dev and cherry picked it on top of master.
Result:
https://github.com/kurtbuilds/serde is version 1.0.144 of serde, and it will give you better error messages. Use it with these lines in your Cargo.toml
[patch.crates-io]
serde_derive = { git = "https://github.com/kurtbuilds/serde" }

Hopefully this is mainlined soon.

Serde has such a fast release cadence that the repository that @kurtbuilds kindly provided is already out of date. If anyone wants this patch, they practically need to keep their own automatically-updated fork with the patch (using tools such as the wei/pull GitHub Action), or else bend over backwards to pin to a Serde version and make sure that nothing ends up pulling a slightly different one, including dependencies of dependencies. Both approaches are inconvenient and contribute to technical debt in downstream applications.
What has been blocking this improvement for more than three years now? It's a really, really welcome change for those that use Serde to parse user-provided configuration.

@dtolnay Did you have any thoughts on this approach? When trying to build user facing tools/services this would be a big help.
If you have thoughts on a different approach that's cool too. It'd be great to get some collaboration going on it.

https://github.com/serde-rs/serde/pull/2403 introduced new merge conflicts for this PR, which I've just fixed in my fork (if anyone's using my fork and notices any problem due to that, please file an issue on the fork). If conflicts are keeping this PR from being merged, I'd be happy to contribute and submit my updated patches upstream.

When will this PR be merged? This is really useful! There is so little misinformation now!

One way to substitute this is with serde_path_to_error, for me, that has completely satisfied what this pr did.

Thank you @dtolnay for your clarification on the state of this PR - I appreciate being decisive and explicit with PRs as important for certain users as this one has been for years.

When trying to use serde-untagged I also ran into issues with not being able to roll the parser back thus not being able to easily deserialize an enum containing multiple structs. However I guess an alternate implementation could potentially either fork/copy over the Content API or buffer to some other intermediate struct such as serde_json::Value - it does however require re-implementing a fair bit of code for each struct just to get error messages that are decent.

Adding to some of the points exposed by @33KK:

Forking serde and serde_derive is obviously complicated since that would also require patching the entire dependency chain to use that fixed version. That is on top of maintaining the fork, keeping it up to date and resolving any conflicts.

I've been doing that maintenance work for a while now, so other people don't have to. Maintaining that fork has been straightforward so far, barring the time when precompiled binaries where introduced, but they ended up being reverted for a plethora of reasons. That does not solve the problem that patching serde and serde_derive is only viable for end applications, but fortunately those are the most affected by these cryptic error messages.

[...] I think this feature is pretty much basic and a must-have, since currently it's basically impossible to tell what went wrong in an untagged enum, especially when it's the top level type or has a lot of children, then the entire error is completely worthless.

While I agree with this, I recently stumbled upon and documented the little-known expecting container attribute, which allows setting a custom fixed error message when deserializing untagged enums. By design, such a message lacks any situational information, unlike the changes proposed by this PR, but applications can write it so to be user-friendly enough for most circumstances, and supersede this PR for the applicable use cases.

It's a shame this is still an issue. Most of the issues with our configuration that is reported by users is caused by enums/unions swallowing the actual inner error, and we have no way to present that to the user.
We also use expecting but that also doesn't really work well.
For example, say we have a field that accepts a string ID, or a vec of string IDs. Very simple enum, as we can do something like expecting = "expected a string or list of strings". But say we add custom parsing rules for an ID type that whitelists the allowed characters. When the user enters an invalid character, that parse error is not bubbled up, and they have no idea why their ID/config isn't working.


1.5.6

Make sure https://github.com/Uniswap/universal-router-sdk/pull/138 is merged first for clean git history

closing in favor of unforked https://github.com/Uniswap/universal-router-sdk/pull/141


ipfs: 0.12.1 -> 0.12.2

Description of changes
https://github.com/ipfs/go-ipfs/releases/tag/v0.12.2
"This patch release fixes a security issue wherein traversing some malformed DAGs can cause the node to panic."
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

I'll need to backport this. A patch release was made for 0.11.x.


switching accounts doesn't refresh feed

if logout and then login to a different account the home feed doesn't refresh, need to do a pulldown refresh

specifically, firefish.social


Aquire debugger package only once when using devcontainers

Hi,
I'm using devcontainers and each time a container gets rebuilt, the extension acquires the platform specific debugger package again, although it is already installed under ~/.vscode-server/extensions/.
On that note, the prebuilt packages don't work in alpine containers, so the extension is downloading the wrong binary again and again for nothing.
thx Michi

The extension you install from VSC Marketplace is just a stub that determines which platform it runs on and then downloads the full package.  By rebuilding the container you are losing the downloaded data.
You can install the appropriate full package directly from CodeLLDB releases and make it a part of the container image.

You were right, my bind mount was no longer working.


Release note version update

There is no 12.9.2. It's 12.10

No DCO needed and my readme change isn't breaking travis

@marcparadise Is this correction true? I thought this is still an outstanding bug?


StreamReader Failed to open the input io.BytesIO

🐛 Describe the bug
StreamReader Failed to open the input io.BytesIO which is save by torchaudio.save
import io
import torchaudio
from torchaudio.io import StreamReader

wav_file = "demo.wav"
streamer = StreamReader(wav_file)  # works fine

with open(wav_file, 'rb') as f:
    streamer = StreamReader(f)  # works fine

waveform, sample_rate = torchaudio.load(wav_file)
buffer_ = io.BytesIO()
torchaudio.save(buffer_, waveform, sample_rate, format="wav")
streamer = StreamReader(buffer_) # error

Traceback (most recent call last):
  File "/home/jackie/code/temp.py", line 14, in <module>
    streamer = StreamReader(buffer_)
  File "/home/jackie/anaconda3/lib/python3.9/site-packages/torchaudio/io/_stream_reader.py", line 351, in __init__
    self._be = torchaudio._torchaudio_ffmpeg.StreamReaderFileObj(src, format, option, buffer_size)
RuntimeError: Failed to open the input "<_io.BytesIO object at 0x7fd22a59ff90>" (Invalid data found when processing input).

Versions
PyTorch version: 1.12.1+cu102
Is debug build: False
CUDA used to build PyTorch: 10.2
ROCM used to build PyTorch: N/A

OS: Ubuntu 20.04 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
Clang version: Could not collect
CMake version: version 3.16.3
Libc version: glibc-2.31

Python version: 3.9.12 (main, Apr  5 2022, 06:56:58)  [GCC 7.5.0] (64-bit runtime)
Python platform: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.31
Is CUDA available: False
CUDA runtime version: No CUDA
GPU models and configuration: No CUDA
Nvidia driver version: No CUDA
cuDNN version: No CUDA
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

Versions of relevant libraries:
[pip3] mypy-extensions==0.4.3
[pip3] numpy==1.21.6
[pip3] numpydoc==1.2
[pip3] torch==1.12.1
[pip3] torchaudio==0.12.1
[pip3] torchvision==0.13.1
[conda] blas                      1.0                         mkl
[conda] mkl                       2021.4.0           h06a4308_640
[conda] mkl-service               2.4.0            py39h7f8727e_0
[conda] mkl_fft                   1.3.1            py39hd3c417c_0
[conda] mkl_random                1.2.2            py39h51133e4_0
[conda] numpy                     1.21.6                   pypi_0    pypi
[conda] numpydoc                  1.2                pyhd3eb1b0_0
[conda] torch                     1.12.1                   pypi_0    pypi
[conda] torchaudio                0.12.1                   pypi_0    pypi
[conda] torchvision               0.13.1                   pypi_0    pypi

Hi
The buffer needs to be rewound before reading it again.
buffer_.seek(0)

thx, my mistake

When a source object is passed to StreamReader, the StreamReader parses the metadata, so it needs at minimum header data to be available. How it parses the metadata depend on the media format. In your case, you are passing format='f32le', which is headless, so in theory passing empty BytesIO could work, but it is also possible that the parsing mechanism still attempts to read some data. (this is the detail abstracted away even in FFmpeg codebase, so I cannot say things for sure.)
You can investigate how StreamReader is using the file-like object by wrapping the instance with dummy class which logs the call to read and seek.
class Wrapper:
    def __init__(self, buffer):
        self.buffer = buffer

    def read(self, n):
        print(f"read: {n} bytes")
        return self.buffer.read(n)

    def seek(self, offset, whence):
        print(f"seek: {offset}, {whence}")
        return self.buffer.seek(offset, whence)

If read and seek is happening in the constructor of StreamReader, then there are some data buffered so that BytesIO buffer don't get to EOF.

yes, read and seek is happening in the constructor of StreamReader
read: 4096 bytes
read: 4096 bytes
read: 4096 bytes
read: 4096 bytes
seek: -1, 2
seek: 0, 0
seek: -1, 2
seek: 0, 0
seek: -1, 2
seek: 0, 0

it looks like we can't add more data to BytesIO after initialize StreamReader
import io
import torchaudio
from torchaudio.io import StreamReader

wav_file = "demo.wav" # 16khz audio
waveform, sample_rate = torchaudio.load(wav_file)
buffer_ = io.BytesIO()

num = 8000
buffer_.write(waveform[:num].numpy().tobytes())
buffer_.seek(0)
streamer = StreamReader(
    buffer_,
    format='f32le',
    option={"sample_rate": "16000"},
)
position = buffer_tell() # which is 8000, since constructor of StreamReader do `read` and `seak`
buffer_.write(waveform[8000:].numpy().tobytes())

streamer.add_audio_stream(
    frames_per_chunk=2560,
    filter_desc=f"aresample=8000,aformat=sample_fmts=fltp",
)

for multi_streams_chunk in streamer.stream():
    print(multi_streams_chunk[0].shape)

still, RuntimeError: Failed to process a packet. (Invalid argument).

The issue with the approach is that StreamReader detects EOF during the construction. Since the plain io.BytesIO object has seek method, StreamReader uses it to detect the EOF and mark it. You write to buffer object and extend the content after that, but StreamReader has already seen the original EOF, any data found after that is illegal.
To feed data interactively, you need to write custom file object class with read method, and fill the buffer in the read method as necessary, and make sure that read method does not return an empty byte string until you wan to finish processing.

For example; this will generate infinite noise.
Similarly you should perform buffer re-filling in read method.
Also note the lack of seek method, so that the behavior is true streaming fashion. No peeking ahead, no rewinding.
See https://pytorch.org/audio/stable/tutorials/streaming_api_tutorial.html#file-like-objects
import torch
from torchaudio.io import StreamReader

class InfiniteNoise:
    def __init__(self):
        pass

    def read(self, n):
        print(f"read: {n} bytes")
        return torch.randn((n // 4, )).numpy().tobytes()

buf = InfiniteNoise()

streamer = StreamReader(
    buf,
    format='f32le',
    option={"sample_rate": "16000"},
)

streamer.add_audio_stream(
    frames_per_chunk=2560,
    filter_desc="aresample=8000,aformat=sample_fmts=fltp",
)

for i, (chunk, ) in enumerate(streamer.stream()):
    print(i, chunk.shape)

    if i > 5:
        break


UI- Launch catalog: Select the catalog of cluster/project level, catalog name is not updated

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Add cluster/project level catalog
Launch catalog, select the catalog of cluster/project level

Result:
Catalog name is not updated

Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
rancher/rancher:master
Installation option (single install/HA):

Version -2.2 master 12/07
Verified fixed


Different rotation matrix applied for res_vb and res_vt?

Hi there,
In the function of: box_estimator.solve_x_y_z_theta_from_kpt, it seems res_vb and res_vt use different rotation matrices.
res_vb = y/(z -np.sin(theta)*bottom_w + np.cos(theta)*bottom_l) - bottom_v
res_vt = (y - h)/(z +np.sin(theta)*bottom_w -np.cos(theta)*bottom_l) - top_v

Since they both rotate around y-axis, why different rotation matrices are applied?
Thanks for your time!

Hi there,
The rotation matrix is the same, while the 3D coordinates of vb and vt are different. So the equations are different.


Transaction log button appears repeatedly

Hi,
My transaction button bar looks weird: the button appears repeatedly. Somebody have a tip?

Can't reproduce.
Did you notice when extra button appears?

Since I update to version 4.0.

Try to reset workspace UI (https://github.com/serge-rider/dbeaver/wiki/Reset-workspace)

It's works!
Thanks!

Great

So, DBeaver crashed and started to duplicate again.

Emm, is any information about crash? Logs?

It crashed when I was using another program. I was switch back to DBeaver, I figured that it have be stopped.

I've been able to reproduce this.
And I'll add extra config fix for this toolbar.


[java.sql.SQLException: Out of sort memory, consider increasing server sort buffer size

Affected module
User creation
Describe the bug
An exception with message [java.sql.SQLException: Out of sort memory, consider increasing server sort buffer size [statement:“/* UserDAO.listAfter */ SELECT ue.json FROM user_entity ue LEFT JOIN entity_relationship er on ue.id = er.toId LEFT JOIN team_entity te on te.id = er.fromId and er.relation = :relation WHERE ue.deleted = FALSE AND (JSON_EXTRACT(ue.json, ‘$.isBot’) IS NULL OR JSON_EXTRACT(ue.json, ‘$.isBot’) = FALSE ) AND (:team IS NULL OR te.name = :team) AND ue.name > :after GROUP BY ue.name, ue.json ORDER BY ue.name LIMIT :limit”, arguments:{positional:{4:null,5:16,6:,7:10}, named:{limit:16,team:null,after:,relation:10}, finder:[]}]] was thrown while processing request.
To Reproduce
Screenshots or steps to reproduce
Expected behavior
A clear and concise description of what you expected to happen.
Version:

OS: [e.g. iOS]
Python version:
OpenMetadata version: [e.g. 0.8]
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.

follow the docs here https://docs.open-metadata.org/v1.1.0/deployment/upgrade#update-sortbuffersize-mysql-or-workmem-postgres


avoid unnecessary application restarts.

Issue
If multiple relation data fields like, for example, username, password and endpoints are changed (even at the same time in the provides side/database side), the requires side libary will trigger multiple events at once  (like database created and endpoints changed).
This, in some application charms, can cause multiple restarts of the application, because some applications would need to restart in both situations
Solution

Don't trigger endpoints changed event if a database created event was trigerred in the same relation changed event.
Don't trigger read only endpoints changed event if a endpoints changed or a database created event was trigerred in the same relation changed event.

Testing

The existing unit test related to emitting events was updated to ensure that multiple event are not triggered.
It was also manually tested the new unit test code with the previous library code (it correctly fails).

Release Notes

Change the requires library to avoid triggering multiple events at once

Test implemented.