entproto: valuescanner support

Opening this early for feedback on uuid changes necessary to integrate it using ValueScanner. We switch to using string to represent UUID rather than bytes, allowing us to generalize the implementation.

Something I am struggling with:
When storing ValueScanner types in the proto, we need to someone decide what type to use. Right now, the type is derived based on the Field type. But if the user specifies a custom GoType then it is stored using the Valuer type (driver.Value). How do we figure out the driver.Value type to set the proto type when generating the protos?

Hey @tarrencev ,
Re providing the proto type annotation, this is possible today:
https://github.com/ent/contrib/blob/e16031b69b23333c597e278f7472040b7ac990eb/entproto/field.go#L48-L60


Is there any way to integrate Protractor with TestRail API?

I am using protractor to automate a Angular website. I need to log the test result in TestRail. Is there any way to do it using protractor?

It is possible to generate test result in JUnitXml & NUnitXml format. Does TestRail have plugin to for junit Or nunit report ?

Please read the contributing guidelines; these types of questions are ill-suited for GitHub. Thanks!


Vulnerability fix for AzureRmWebAppDeploymentV3

Task name:  AzureRmWebAppDeploymentV3
Description: Fixed latest vulnerability.
Documentation changes required: (Y/N)
Added unit tests: (Y/N)
Attached related issue: (Y/N)
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected
[x] node make.js build --task AzureRmWebAppDeploymentV3
[x] node make.js test --task AzureRmWebAppDeploymentV3

/azp run


Minimum Windows version for Windows binary

Fixes #1377

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/26bb79af9430ae43c76a49e995fd1fc444263fc9/

Sorry for not seeing it before merging. I think this text is misleading in that it conflates "the Windows binary" with the source code which is what actually requires Windows 8+.
In other words, the current wording suggests that building from source might remove this requirement, but it would not.

@tamird, "build from source" isn't even presented as an option on Windows, as it is on Mac and Linux. We only present either "download the binary" or "use docker".
If we want to provide instructions for building from source on Windows, I'll need help understanding how the process differs. Feels unnecessary, though, given that Windows support is only experimental...

I suppose it could say "CockroachDB requires Windows 8 or later. Windows
support is experimental and has not been extensively tested by Cockroach
Labs. The binary is provided..."
The current wording doesn't bother me, though.
On Mon, May 8, 2017 at 12:14 PM, jseldess<EMAIL_ADDRESS>wrote:

@tamird https://github.com/tamird, "build from source" isn't even
presented as an option on Windows, as it is on Mac and Linux. We only
present either "download the binary" or "use docker".
If we want to provide instructions for building from source on Windows,
I'll need help understanding how the process differs. Feels unnecessary,
though, given that Windows support is only experimental...
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/cockroachdb/docs/pull/1379#issuecomment-299914181,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA15IGQ9VyvzvujBiUKPEI4jhqPUdL5Lks5r3z99gaJpZM4NTXXp
.

@jseldess I think it is naive to assume that users won't try something just because it isn't presented in the docs (we have seen a variety of platforms and even had folks try ARMv8, after all).
I also appreciate that we don't necessarily want to consider those users when writing our docs, since they're off in uncharted territory, yet I find that the language here is imprecise and that making  it precise could be achieved without making it more pedantic or convoluted.
Would you accept a PR with such rewording? I'd be happy to take the task.

Of course, @tamird. Thanks for offering.


EVEREST-478 Flaky tests

Fix flaky tests
Problem:
EVEREST-478
In the backend repo there are several flaky tests for monitoring and backup-storages
Solution
Add random naming and cleanup

not relevant anymore, fixed by https://github.com/percona/percona-everest-backend/pull/431


Data not update continously(not Loop)

Hi. I tried that code and its working, but the new data not showing after 10 second(because default delay is 10 second) in debug console. Unless I restart the debugging, new data updated in debug console

This issue has been closed because it has not had recent activity. Please reopen if this issue is still important to you and you have additionally information.


suggestion

Do you suggest splitting it into multiple files and adding some test code?

When I was working on this the first time (with a megaraid driver host) I'd just redirected the storcli command output to a text file. The second one was for the drives output. The test blocks are still hanging out in there commented out.
https://github.com/blakehartshorn/storcli-collector/blob/33e306d94ced5818e32a55ed3c2723590d6ca117/storcli-collector.go#L314-L324
https://github.com/blakehartshorn/storcli-collector/blob/33e306d94ced5818e32a55ed3c2723590d6ca117/storcli-collector.go#L348-L355


dependabot: monthly updates of github actions, pip to monthly also

This is one of several PRs opened in the juptyterhub github organization to systematically configure dependabot to update github actions on a monthly basis, for more information see https://github.com/jupyterhub/team-compass/issues/636.

Actually, we have requirements.txt and requirements.in used to freeze requirements.txt using a github workflow already, so I think the pip dependabot bumping should be removed instead.


OpenSSL: use proper factories over direct allocation

While investigating #4650 I noticed that OpenSSL::SSL::Context insecure initializer relied on #allocate. This patch avoids this by using .new factories an a single #initialize method taking a LibSSL::SSLContext pointer, and a boolean argument to set secure defaults, or not (using OpenSSL defaults).

It's OK, but using of new should be documented for newcomers, for example.
Here or anywhere else.

I'm wondering if Context::Server and Context::Client really need to be two separate types. The behaviour is exactly the same, except from the constructors but they could still just be different ones.
Client context has protected method set_cert_verify_callback but it wouldn't hurt if it was accessible on server as well, it's not public anyway. And it's only needed for OpenSSL < 1.0.2 which is out of support.

The original idea between separate types was to actually have type safety for not using a client context in a server socket and vice versa.


Always On Top zones for certain FancyZones

Summary of the new feature/enhancement
When you have applications that are in a FancyZone, like windows terminal/PowerShell you may want this to always show on top of other applications, like say Outlook/Teams/Code etc to reduce the need to switch windows whilst still enabling ability to do more work with a smaller screen estate.
Proposed technical implementation details (optional)
Something like a per zone setting whether that be via Key Shortcuts or set on the zone when they are defined if using custom zones

@kilasuit
thank your for your feedback. Since we were already tacking this feature request, I'm closing this issue in favor of https://github.com/microsoft/PowerToys/issues/1127
Feel free to add your comments there.
Thank you.


mate.mate-themes: 3.22.20 -> 3.22.21

Motivation for this change
Update to version 3.22.21
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-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
[x] Fits CONTRIBUTING.md.

I see there's some icon themes here
ContrastHigh  mate  mate-black

one of whichContrastHigh you generate an icon cache for.
You'll need to set dontDropIconThemeCache = true; so the gtk setup hook doesn't remove it.

Done.

Should we generate icon cache's for the other icon themes as well?

I suppose no, because they are cursors only.


[docs] Adjust React + Material-UI + Firebase for v2.0

[x] I have followed (at least) the PR section of the contributing guide.

Adjust feature list for v2.0 of RMUIF (https://github.com/Phoqe/react-material-ui-firebase/tree/dialog-host)

@Phoqe It's great to see such great improvements 🎉 Well done!

@joshwooding Thank you! :)


Can't open task in new pane

What happened?
I have macOS and when I want to use cmd + click to open a task in a new pane, it doesn't and just ignores it.
I created a new vault to test this: I only installed the dataview plugin and tested. It doesn't work.
DQL
No response
JS
No response
Dataview Version
0.5.41
Obsidian Version
0.15.9
OS
MacOS

Can repro on Mac.

But it don’t work a least 5 people who said me that in Obsidian community in Discord

Oh I meant to say that I do see this error and we'll investigate.

Once my PR merges, this should be fixed.

Once my PR merges, this should be fixed.

When can I try it?

I didn't make a release in my fork. Blacksmithgu should be the one to review this and I expect he'll find time maybe over the weekend or later next week. If that's blocking you, let me know and I'll make a temp release on my fork.

I didn't make a release in my fork. Blacksmithgu should be the one to review this and I expect he'll find time maybe over the weekend or later next week. If that's blocking you, let me know and I'll make a temp release on my fork.

It would be really cool if it could be done temporarily. This is really blocking my work, I have over 300 documents and they are long, task work is just incredibly difficult. Thank you so much.

Will get around to tomorrow

Will get around to tomorrow

Could you update status on that? Will you implement in temporary fork? Cuz it's bug is exist in my obsidian yet:(

Oh my, this slipped my mind. Give me an hour.

Okay so I couldn't create a proper release but you can install "Single plugin based on frozen release tag" via BRAT. Install BRAT first and then use the command I mentioned. Use AB1908/obsidian-dataview as the repo name and 0.5.36a as the tag to install. Let me know if you run into issues.

Okay so I couldn't create a proper release but you can install "Single plugin based on frozen release tag" via BRAT. Install BRAT first and then use the command I mentioned. Use AB1908/obsidian-dataview as the repo name and 0.5.36a as the tag to install. Let me know if you run into issues.

Did I understand correctly that I need to disable the current data view and put your version through BRAT?

Okay so I couldn't create a proper release but you can install "Single plugin based on frozen release tag" via BRAT. Install BRAT first and then use the command I mentioned. Use AB1908/obsidian-dataview as the repo name and 0.5.36a as the tag to install. Let me know if you run into issues.

ooo, it work! Thank you so much! I don't have any questions:)

Merged and to be released as 0.5.42 tomorrow.

Merged and to be released as 0.5.42 tomorrow.

Thank you so much!

Merged and to be released as 0.5.42 tomorrow.

Hey! My problem already exist yet:(

Hey! Problem isn't solving yet( I can't open task in new pane( @blacksmithgu @AB1908

There hasn't been a new release yet, sorry. I can't find time to do this unfortunately. Please continue using my fork if it is critical.

0.5.42 has been released.

Still not working in 0.5.56 for ⌘ + ⌥ + Click on MacOS (Obsidian Version 1.4.12).

Should work for command click

If not, please file a ticket with steps to repro


tests: Increase test coverage of action.py.

tests: Test bots deactivation with owner deactivation. (wasn't sure whether it should go inside test_signup but test_bots seems a better place to me.)

tests: Test invalid "default stream group" name.

(I'm trying to make myself more comfortable in testing, next, I'll be writing some tests with mocking and frontend test especially Casper test :) tell me if there are some important testing tasks)

Yes, its the part of #7809 and other linked issues, I can't take the whole issue as they are too broad so, I used to do them in parts, should I claim the issue? (the problem is that I take the look at that issue In my spare time or when I got tired of working on another issue)

Nice, merged, thanks @shubhamdhama!  I would just do individual sub-issues, and comment when there's some we should check off.


Not all path return a value issue with multiline code.

Godot version
4.02.stable
System information
Linux Ubunto 22.04
Issue description
The following code is invalid :
func get_normaldir( normal:Vector3 ) -> String:
var x:float=abs(normal.x); var y:float=abs(normal.y); var z:float=abs(normal.z);
if(x>y and x>z): return "x"; if(y>x and y>z): return "y"; if(z>x and z>y): return "z"; return ""
But this is fine :
func get_normaldir( normal:Vector3 ) -> String:
var x:float=abs(normal.x); var y:float=abs(normal.y); var z:float=abs(normal.z);
if(x>y and x>z): return "x"; if(y>x and y>z): return "y"; if(z>x and z>y): return "z";
return ""
Steps to reproduce
The following code is invalid :
func get_normaldir( normal:Vector3 ) -> String:
var x:float=abs(normal.x); var y:float=abs(normal.y); var z:float=abs(normal.z);
if(x>y and x>z): return "x"; if(y>x and y>z): return "y"; if(z>x and z>y): return "z"; return ""
But this is fine :
func get_normaldir( normal:Vector3 ) -> String:
var x:float=abs(normal.x); var y:float=abs(normal.y); var z:float=abs(normal.z);
if(x>y and x>z): return "x"; if(y>x and y>z): return "y"; if(z>x and z>y): return "z";
return ""
Minimal reproduction project
The following code is invalid :
func get_normaldir( normal:Vector3 ) -> String:
var x:float=abs(normal.x); var y:float=abs(normal.y); var z:float=abs(normal.z);
if(x>y and x>z): return "x"; if(y>x and y>z): return "y"; if(z>x and z>y): return "z"; return ""
But this is fine :
func get_normaldir( normal:Vector3 ) -> String:
var x:float=abs(normal.x); var y:float=abs(normal.y); var z:float=abs(normal.z);
if(x>y and x>z): return "x"; if(y>x and y>z): return "y"; if(z>x and z>y): return "z";
return ""

The function is bugged by the way.... Its just a example.

Just repeating the same in each section isn't helpful, also please format your code snippets they are hard to read like this, use "gdscript" at the start and "" at the end as separate rows

The reason for this is that you aren't doing what you are thinking you are doing
If statements are indentation based, your code is equivalent to:
func get_normaldir( normal:Vector3 ) -> String:
    var x:float=abs(normal.x); var y:float=abs(normal.y); var z:float=abs(normal.z);
    if(x>y and x>z):
        return "x";
        if(y>x and y>z):
            return "y";
            if(z>x and z>y):
                return "z";
                return ""

not
func get_normaldir( normal:Vector3 ) -> String:
    var x:float=abs(normal.x); var y:float=abs(normal.y); var z:float=abs(normal.z);
    if(x>y and x>z):
        return "x";
    if(y>x and y>z):
        return "y";
    if(z>x and z>y):
        return "z";
    return ""

I guess I didnt put an good example. By the way, the reason why I repeated the code on each section was because my browse glitch and didnt allow me to type on it.
If you have a code like :
if(something): return that else: return this; return somethingelse
You will get an error that its impossible to fix unless you dont use multiline code. You will force do to something like this to fix:
if(something): return that else: return this;
return somethingelse
The issue is with the multiline interpreter of the gdscript or with the error handler for if then else, that is not considering multiline code, I am not sure if the same happens with c#.

Following some more consistent example:
# this is invalid :
func test(a:int) -> int:
	if(a==1): return 0; if(a==0): return 1; return -1

# this is valid but should be the same as above :
func test2(a:int) -> int:
   if(a==1): return 0; if(a==0): return 1;
   return -1

As I understand for the multiline code convention, it should work, so I think its a bug.

The reason for this is that you aren't doing what you are thinking you are doing, not a bug, and if you check code warnings you will see that the code is unreachable, indicating your error
If statements are indentation based, your code is equivalent to:
func get_normaldir( normal:Vector3 ) -> String:
    var x:float=abs(normal.x)
    var y:float=abs(normal.y)
    var z:float=abs(normal.z)
    if(x>y and x>z):
        return "x"
        if(y>x and y>z):
            return "y"
            if(z>x and z>y):
                return "z"
                return ""

not
func get_normaldir( normal:Vector3 ) -> String:
    var x:float=abs(normal.x)
    var y:float=abs(normal.y)
    var z:float=abs(normal.z)
    if(x>y and x>z):
        return "x"
    if(y>x and y>z):
        return "y"
    if(z>x and z>y):
        return "z"
    return ""

Multiple statements on the same line is not recommended

I dont agree... My understand of multiline convention is that when you use an  endstatement the interpreter/compiler should consider same as an new line, eater that or it makes an dangerous mess in the language.

I dont agree

Agree with what? This is how the code works, not an opinion

I dont agree

Agree with what? This is how the code works, not an opinion

Do you know what is the function of the ";" char in the language ?

To end a statement, not to end a block

GDScript is an indentation based language, this is how it works
So to give you an example:
if a:
    if b:
       if c: foo(); bar()

Should this be read as:
if a:
    if b:
       if c:
           foo()
           bar()

or
if a:
    if b:
       if c:
           foo()
       bar()

or
if a:
    if b:
       if c:
           foo()
   bar()

or
if a:
    if b:
       if c:
           foo()
bar()

As I understand for the multiline code convention

What convention, where is this documented?

My issue here is that I don't think:
if x: foo(); bar()

Should be different from
if x:
    foo(); bar

It'd be very confusing
I am sorry if I am being dismissive, but this is a matter of how the language is designed, and to me makes the most sense and follows the principle of least astonishment, the base assumption would be "this doesn't imply that we change indentation"

This was already discussed in godotengine/godot-proposals#3759, see https://github.com/godotengine/godot-proposals/issues/3759#issuecomment-1005974260

Opened a proposal for the discontinuing of this code, feel free to voice your opinion, concern, and reasoning there godotengine/godot-proposals#6924
Otherwise I'd consider this not a bug and suggest further conversation at the proposal


[labs/ssr] Add localName to DOM emulation for SSR

Should this be an RFC?

[X] This is not a substantial change

Which package is this a feature request for?
SSR (@lit-labs/ssr)
Description
When running the SSR script the element's localName and tagName are not added to the element. This has lead to a bug in the ElementInternals polyfill at this line.
This line exists to make sure that the node being referenced by the internals object is a custom element. This might be useful server-side behavior so that elements using AriaMixin via the ElementInternals spec can sprout attributes on their hosts (yet to be tested).
Alternatives and Workarounds
A current workaround for the polyfill has already been implemented, returning an empty object for the internals instance in case no localName is present.

Hi, I would love to try and help if this is open to external contributors.
@justinfagnani, is this just a matter of adding the localName field when initializing the ElementRenderer, or is it deeper than that? https://github.com/lit/lit/blob/main/packages/labs/ssr/src/lib/element-renderer.ts#L49

@willmartian sorry for the delayed response. We're definitely open to external contributors (and would love to erase even the external distinction over time). That's one reason I tagged this as a "Good First Issue".
I think what's needed is to implement a .localName getter on Element and to get the tag name of the element from the custom elements registry in the constructor with new.target. We don't have to worry about upgrades on the server.

after making the required changes how can i test it?


pipeline example eventually fails

If I run the pipeline example, I eventually get
ERROR: An HTTP request took too long to complete. Retry with --verbose to obtain debug information.
If you encounter this issue regularly because of slow network conditions, consider setting COMPOSE_HTTP_TIMEOUT to a higher value (current value: 60).

Do you have an idea of how long you ran it for to obtain this output?

Adam, would it be possible to repeat the experiment without a link node? (with the stream processing function of client2 - streamMap (\st-> reverse st) - moved into the sink so that the same stream processing pipeline is being executed.. but only in 2 containers?). It makes the configuration simpler, and could help us find out if it's the link node that's the problem, Paul

Do you have an idea of how long you ran it for to obtain this output?

I think approximately a minute, corresponding to the default COMPOSE_HTTP_TIMEOUT value, but I'm not certain. Re-running to check.
(I hit this, by the way, before or after PR #12, so it wasn't caused by that)

The issue is the tty: true for the link node. This causes docker compose to expect log messages from the node, and when it doesn't see any, it thinks there's a fault and kills all three nodes.

Adam, would it be possible to repeat the experiment without a link node? (with the stream processing function of client2 - streamMap (\st-> reverse st) - moved into the sink so that the same stream processing pipeline is being executed.. but only in 2 containers?). It makes the configuration simpler, and could help us find out if it's the link node that's the problem, Paul

@PaulWatson9994 I'm assuming these comments are related to the throughput testing notes in PR #12? I'll create a new issue with the information from that PR and respond there.


Make PlantumlDiagram public so that ToPumlString method is accessible

Make PlantumlDiagram public so that its ToPumlString method is accessible from outside of this library - it's useful for creating a puml string when you don't want to save it to a file or export it to a PNG (for example, in my project I want to post the puml string to an external server, so all I need is the string).

Hi @yanjustino, could you take a look at this PR? Thanks!

Thank you for your contribution, @BenRStutzman! I believe that your submission will bring many benefits to library users!

Soon I will release a new version

Great, thanks @yanjustino! I have a couple other ideas for changes that I'll make PRs for in a bit.


Examples do not compile

In the Node.js examples there is a reference to an object 'm' that is not defined, thus my typescript/nodejs compiler will not compile the code.
d.ruleset('test', function() {
// antecedent
whenAll: m.subject == 'World'
// consequent
run: console.log('Hello ' + m.subject)
});
Error:(27, 26) TS2304: Cannot find name 'm'.

If I declared 'm' in scope as
let m;
it works. But otherwise, it's undeclared.


Enable Extensions

Hi,
How do we enable ext. like spellcheck?
When I use "spellcheck" true in the option, I get the following error
misspelled option "spellcheck"
Thanks.

Please do not open a new issue when you could have just commented on the old one. In that case, I have no idea. You will have to look into it more yourself, and possibly submit a pull request if you really need it.


Add "Official" git-prompt.zsh  and my theme.

Add "Official" git-prompt.zsh  and my theme.

I don't think this repo is accepting any themes.

Thanks for your reminding....
谢谢您的提醒。。。


Better java support

Currently KtGui supports Java through Kotlin's native compatibility, but I've made no effort to actually add better Java useability.
Implementing Builder functions that we can chain, where Kotlin would have code blocks, could help.
GuiScreen gui = new GuiScreen(Component.text("Title"), 3)
	.addChild(
		new GuiButton<>(Material.DIRT)
			.named(Component.text("Item")
			.click(ClickType.LEFT, (event) -> event.getPlayer().sendMessage(Component.text("clicked!")))
			.lore(Component.text("Lore line one"))
			.lore(Component.text("Lore line two"))
			.slot(3)
	).addChild(...);

// Will need to add compat for signals, not as pretty as kotlin.
Signal<String> signalExample = gui.createSignal<String>("mattmx");

gui.effect(signalExample, () -> {
	new GuiButton<>(Material.DIAMOND)
		.named(Component.text(signalExample.get()))
		.slot(5)
		.childOf(gui);
});

gui.open((event) -> event.getPlayer().sendMessage(Component.text("opened.")));

gui.open(player);

Features are currently not focused on Java support, but support will be added upon request.


[PROPOSAL]use cached calculate value

Package version, Laravel version
2.1.24
Expected behavior
I would like to have the calculate config field to have 3 option : yes, no and cached.
the Cached would be the same as calling $cell->getOldCalculatedValue() that get the internal excel.
thanks

If PHPExcel doesn't have a method like that, it won't be possible

PHPExcel has a method like that "$cell->getOldCalculatedValue()"

So, use that one then ;)

the purpose is to include it inside your abstraction ...

I prefer not to support that.


Git commit property for the build

Use the --giit-commit flag for nemerosa/ontrack-cli#8

The --commit flag is already available.


Config option to disable Spawners Modifiers

I've tried setting the item to "minecraft:air" but I can apply modifiers with bare hand

Disable the module or set the item to minecraft:barrier


Morning session: Peer-to-peer code review ♻

Hi Tite lunda🖐
I'm Abenezer your morning session partner
Good job so far! 🎉

But there are some issues that you still need to work on but you are almost there!
Highlights
✔Good job overall following the template design ✅
✔Correct GitHub Flow ✅
Something Changes ♻

[ ] Fix margin property

Please try to mimic the figma design for a better layout.
Cheers and Happy coding!👏👏👏

Thanks so much


update to latest service specs

https://github.com/Azure/azure-rest-api-specs/commit/322d0edbc46e10b04a56f3279cecaa8fe4d3b69b

Thanks for the review @heaths. I also run the --all-features test before merging large updates. Looks like I need to fix:
    Checking azure_svc_mixedreality v0.1.0 (/Users/runner/work/azure-sdk-for-rust/azure-sdk-for-rust/services/svc/mixedreality)
error[E0072]: recursive type `InnerError` has infinite size
  --> svc/mixedreality/src/package_0_3_preview_0/models.rs:94:1
   |
94 | pub struct InnerError {
   | ^^^^^^^^^^^^^^^^^^^^^ recursive type has infinite size
...
98 |     pub innererror: Option<InnerError>,
   |                     ------------------ recursive without indirection
   |
help: insert some indirection (e.g., a `Box`, `Rc`, or `&`) to make `InnerError` representable
   |
98 |     pub innererror: Box<Option<InnerError>>,
   |                     ++++                  +

https://github.com/ctaggart/azure-sdk-for-rust/runs/4703057191?check_suite_focus=true

Each batch updates brings in a few properties that need updating. With the latest updates to resolve everything before codegen, automatically finding these may be possible. #225
error[E0072]: recursive type `package_2021_12_preview::models::InnerError` has infinite size
   --> mgmt/dataprotection/src/package_2021_12_preview/models.rs:761:1
    |
761 | pub struct InnerError {
    | ^^^^^^^^^^^^^^^^^^^^^ recursive type has infinite size
...
767 |     pub embedded_inner_error: Option<InnerError>,
    |                               ------------------ recursive without indirection
    |
help: insert some indirection (e.g., a `Box`, `Rc`, or `&`) to make `package_2021_12_preview::models::InnerError` representable
    |
767 |     pub embedded_inner_error: Box<Option<InnerError>>,
    |                               ++++                  +
    ```

error[E0072]: recursive type `package_2021_06_30_preview::models::InnerError` has infinite size
  --> svc/digitaltwins/src/package_2021_06_30_preview/models.rs:69:1
   |
69 | pub struct InnerError {
   | ^^^^^^^^^^^^^^^^^^^^^ recursive type has infinite size
...
73 |     pub innererror: Option<InnerError>,
   |                     ------------------ recursive without indirection
   |
help: insert some indirection (e.g., a `Box`, `Rc`, or `&`) to make `package_2021_06_30_preview::models::InnerError` representable
   |
73 |     pub innererror: Box<Option<InnerError>>,
   |                     ++++                  +

https://github.com/ctaggart/azure-sdk-for-rust/runs/4725927764?check_suite_focus=true


Please update broken link (ID: 1071)

(OpSec) > ( Privacy / Clean Up) > (Hiding from the Internet)
NEW LINK: https://inteltechniques.com/data/workbook.pdf
{
  "id": "<ID>",
  "name": "Hiding from the Internet",
  "type": "url",
  "url": "https://inteltechniques.com/data/workbook.pdf"
},

Fixed. Thanks for the heads up!


zero_to_fp32

The script will need to move the optimizer files out of the "global_step" directory to the root of the checkpoint folder.
eg
cd checkpoint_dir
cp global_step100/*.pt .
otherwise it will not work.

also, just a potential addition, making a single script for exporting the model could be helpful (you could possibly make an sh script that that calls the python scripts).

Thanks! I will mention this in the README.md


Needs More Steve

At least 50% more Steve is needed.

We have enough Steve.


Google OAUTH directing to blank page

It happened on my laptop, but not on Willy's laptop...
It's navigating to /auth/google instead of Google's OAUTH

@JasminBarco is this still an issue? I think we have keys on the production heroku for oauth now

Not sure what this issue is even about! Let's check it out on Thursday? @ximsce

I won't be there on Thursday, but if you can navigate through the production heroku site and sign a campaign then I think this can be resolved.

@ximsce @JasminBarco Looks like this was closed; I'm still having issues on production, seeing the 404 when trying either facebook or google buttons:


Connection issue with Redshift on COPY command

Hi, and thank you a lot for the work here, it already helped me way more than I could have expected.
We are using docker-airflow image to setup workers, and one of the things our ETL does is perform a COPY command to put some processed data from s3 to Redshift.
I adjusted the image to be able to use psql, and I verified that connection works. I can connect to worker container, from there connect to Redshift and run various queries. I also verified that COPY command itself works - I can successfully copy reasonably small amount of data at once.
But the main goal is to be able to copy massive chunk of data using manifest, and normally that could take a few minutes. I noticed that every time I try it, after 10-20 seconds I get:
SSL SYSCALL error: EOF detected
The connection to the server was lost. Attempting reset: Succeeded.

I googled and it seems that this is related to connection timeout. I tried following https://docs.aws.amazon.com/redshift/latest/mgmt/connecting-firewall-guidance.html#change-tcpip-settings and set those settings via docker-compose, and verified that those are in the /sbin/sysctl -a, but this doesn't help either.
If you have any hints of what could be tried - I would be happy to try them out.

Version of docker issue, details: https://stackoverflow.com/questions/48496689/copy-from-s3-to-redshift-in-docker-container-timeout

I upgraded to the latest stable docker version (19.03.1) and I still have the same issue.


Support for request parameters in OIDC request

PSD2

Fixed with https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/773


how to customize map with different curvature?

Curves in the original map are near 90 degrees, which are far from reality. So I want to customize it.

Hi @18605973470,
I'm sorry, I'm afraid this map requires the changes introduced in later versions, I'm not sure it would work with 0.8.2.
Maybe you can try opening the UE4 editor on our last version, exporting the map, and re-importing it in 0.8.2. But there's going to be a bunch of issues, some missing C++ classes and problems with downgrading UE4.19 map to UE4.18.

Hey, Generating customized map in UE4 in CARLA 0.8.2 with lane informations are the tasks I recently working on , I think you should check the picture in my issue #825 ,if that is what you want,I can give you a little advise:
First you have to create static meshes that you want to , these static meshes are the parameters that you can combine to become a road with curvatures. Second add these static mesh in Carla  source files (carlamapgenerator and so on)and also add the lane informations like the the degree of the angles of the road in roadmap file.finally rebuild Carla and you can open ue4 then drag them in editor and add them in carlamapgenerator and here you go you have a road with curvature.
Hope it helps.


Support runnable templates

Similarly to https://github.com/Kentico/cloud-boilerplate-net/issues/1, we'd like to support the new VS tooling and be able to install the site using the dotnet new command.
Ideally, the template should be parameterized with the project guid.
All necessary resources can be found in the related task.

A quick documentation to give you a good head start.
The new templating functionality is super easy for developers. Basically, moving a few files and creating a manifest in an ordinary VS project folder makes it a source of a template.
A high-level set of steps is:

In the project folder, just create a subfolder 'content'.
Move all contents to that 'content' subfolder.
In the project folder (mentioned above), create a 'Template.nuspec' file (described below).
In the 'contents' subfolder, create two subfolders:

A subfolder named according to the project name
A subfolder called '.template.config/'

In the '.template.config/' subfolder, create a 'template.json' file (described below).

Now, a simple example of a 'template.json' file would be the following:
{
  "author": "Me",
  "classifications": [ "Custom Templates" ],
  "name": "My first template",
  "identity": "MyName.MyTemplate.CSharp",
  "shortName": "helloworld",
  "guids": [ "dc46e9be-12b0-43c5-ac94-5c7019d59196" ],
  "sourceName": "MyTemplate"
}

An example of the 'Template.nuspec' file would be:
<?xml version="1.0" encoding="utf-8"?>
<package>
  <metadata>
	<id>KenticoCloud.CloudBoilerplateNet</id>
	<version>1.0.0</version>
	<authors>Kentico Software</authors>
	<description>Kentico Cloud .NET MVC Boilerplate</description>
	<language>en-US</language>
	<projectUrl>https://github.com/Kentico/cloud-boilerplate-net</projectUrl>
	<licenseUrl>https://github.com/Kentico/cloud-boilerplate-net/blob/master/LICENSE</licenseUrl>
	<requireLicenseAcceptance>false</requireLicenseAcceptance>
	<packageTypes>
	  <packageType name="Template" />
	</packageTypes>
  </metadata>
  <files>
	<file src="**" exclude="content\Web-CSharp\**\*;**\bin\**\*;**\obj\**\*;**\*.user;**\*.lock.json" />
  </files>
</package>

Finally, the package gets created with nuget pack Template.nuspec. It can be uploaded to NuGet.org or installed directly from the filesystem.
When the template gets used for creating a new project, the GUIDs get recreated throughout the template so that the uniqueness of the new project is ensured. Moreover, in the Template.json, there can be 'symbols' defined to make the template parametrizable. With that, various connection strings or other configuration can be set at the time of the instantiation of the template.


Update developer guide on sorting and storage component

Codecov Report

Merging #296 (ffadd18) into master (53ba177) will decrease coverage by 0.11%.
The diff coverage is 48.73%.

@@             Coverage Diff              @@
##             master     #296      +/-   ##
============================================
- Coverage     59.51%   59.39%   -0.12%
- Complexity      353      472     +119
============================================
  Files            75       89      +14
  Lines          1319     1825     +506
  Branches        133      233     +100
============================================
+ Hits            785     1084     +299
- Misses          465      663     +198
- Partials         69       78       +9

Impacted Files
Coverage Δ

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

...rc/main/java/seedu/address/logic/LogicManager.java
71.42% <0.00%> (+71.42%)
:arrow_up:

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <ø> (+58.33%)
:arrow_up:

...java/seedu/address/logic/commands/MarkCommand.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/seedu/address/logic/commands/UnmarkCommand.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...seedu/address/logic/commands/sort/SortCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/MarkCommandParser.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../seedu/address/logic/parser/SortCommandParser.java
0.00% <0.00%> (ø)

...eedu/address/logic/parser/UnmarkCommandParser.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 68 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 53ba177...ffadd18. Read the comment docs.


⚠ Processors has degraded performance

In 42063ea, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 117 ms

Resolved: Processors performance has improved in a4e3884.


Use ModelClassName to differentiate kfserver or torchserve imp for v1 protocol

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Use v2 protocolVersion to choose TorchServe implementation is confusing as v2 protocol is not supported yet, added the validation to reject when v2 protocol is specified.
Added the check that when ModelClassName is specified KFS choose the kfserver implementation otherwise use torchserve implementation.

Release note:

/cc @jagadeeshi2i

/cc @jagadeeshi2i

/lgtm

/lgtm


Add custom feeds support

#65

Thank for writing a PR! Feel free to help yourself to contributors.md.


dnt is not ready yet for some new Deno stuff

❯ deno run -A _build.ts 1.0.5
[dnt] Transforming...
[dnt] Running npm install...

added 7 packages, and audited 8 packages in 3s

found 0 vulnerabilities
[dnt] Building project...
[dnt] Type checking ESM...
src/aliases.ts:81:36 - error TS2339: Property 'Command' does not exist on type 'typeof Deno'.

81   const process = new dntShim.Deno.Command(command, {
                                      ~~~~~~~
src/deps.ts:1:23 - error TS2307: Cannot find module '@std/path' or its corresponding type declarations.

1 export * as path from "@std/path";
                        ~~~~~~~~~~~
src/deps.ts:2:45 - error TS2307: Cannot find module '@std/assert' or its corresponding type declarations.

2 export { assertEquals, assertRejects } from "@std/assert";
                                              ~~~~~~~~~~~~~

error: Uncaught (in promise) Error: Had 3 diagnostics.
          throw new Error(`Had ${diagnostics.length} diagnostics.`);
                ^
    at getProgramAndMaybeTypeCheck (https://deno.land/x/dnt@0.40.0/mod.ts:457:17)
    at build (https://deno.land/x/dnt@0.40.0/mod.ts:345:17)
    at eventLoopTick (ext:core/01_core.js:168:7)
    at async file:///home/lukas/code/deno-shell-aliases/_build.ts:12:1

Therefore unable to publish newer versions to NPM.

https://github.com/denoland/node_shims/blob/main/packages/shim-deno/PROGRESS.md


func "test" in deploy.php not working properly

Q
A

Issue Type
Bug, Question

Deployer Version
6.4.3

Local Machine OS
MacOS 10.14.4

Remote Machine OS
N/A

Description
I am trying to figure out how I can test if its the first time running on a server when deploying a TYPO3 instance.
I figured I could check the filesize of "LocalConfiguration.php". If the created symlink filesize is 0, I know I run for the first time.
Steps to reproduce
Content of deploy.php
// Tasks
task('build', function () {
    //Remove {{deploy_path}}/typo3conf if exists, cause thats the wrong folder!
    run('cd {{deploy_path}} && if [ -d typo3conf ]; then rm -rf typo3conf; fi');

    run('cd {{release_path}} && {{php_composer}} install --verbose --prefer-dist --no-progress --no-interaction --no-dev --optimize-autoloader');

    //Is first run?
    run('cd {{release_path}} && cd {{typo3_webroot}}/typo3conf');
    if (TRUE === test('[[ -f LocalConfiguration.php && ! -s LocalConfiguration.php ]]')) {
        run('cd {{release_path}} && cd {{typo3_webroot}}/typo3conf && touch pqtest');
        run('cd {{release_path}} && cd {{typo3_webroot}}/typo3conf && rm LocalConfiguration.php');
        run('cd {{release_path}} && cd {{typo3_webroot}}/typo3conf && rm PackageStates.php');
    } else {
        run('cd {{release_path}} && cd {{typo3_webroot}}/typo3conf && ln -s {{deploy_path}}/shared/typo3conf/LocalConfiguration.php LocalConfiguration.php');
        run('cd {{release_path}} && cd {{typo3_webroot}}/typo3conf && ln -s {{deploy_path}}/shared/typo3conf/PackageStates.php PackageStates.php');
    }
});

The filesize (of a symlink file) is 0, therefore the if condition should match, (And it does match) but the condition won't get properly executed in PHP's world.
➜  typo3conf git:(master) ✗ if [[ -f LocalConfiguration.php && ! -s LocalConfiguration.php ]]; then echo 'true'; fi
true
➜  typo3conf git:(master) ✗ ls -lah
total 0
drwxr-xr-x   5 pascal.querner  staff   160B May 21 11:53 .
drwxr-xr-x   9 pascal.querner  staff   288B May 21 11:53 ..
lrwxr-xr-x   1 pascal.querner  staff    84B May 21 11:53 LocalConfiguration.php -> /Users/pq/projects/xxx/_deploytest/shared/typo3conf/LocalConfiguration.php
lrwxr-xr-x   1 pascal.querner  staff    79B May 21 11:53 PackageStates.php -> /Users/pq/projects/xxx/_deploytest/shared/typo3conf/PackageStates.php
drwxr-xr-x  10 pascal.querner  staff   320B May 21 11:53 ext
➜  typo3conf git:(master) ✗ cat LocalConfiguration.php #empty return

Output log
➤ Executing task build
[localhost] > cd /Users/pq/projects/xxx/_deploytest && if [ -d typo3conf ]; then rm -rf typo3conf; fi
[localhost] > cd /Users/pq/projects/xxx/_deploytest/releases/20190521095837 && composer install --verbose --prefer-dist --no-progress --no-interaction --no-dev --optimize-autoloader
[localhost] < Loading composer repositories with package information
[localhost] < Installing dependencies from lock file
[localhost] < Dependency resolution completed in 0.002 seconds
[localhost] < Analyzed 238 packages to resolve dependencies
[localhost] < Analyzed 668 rules to resolve dependencies
[localhost] < Nothing to install or update
[localhost] < Generating optimized autoload files
[localhost] < Register typo3/console-plugin file in root package autoload definition
[localhost] < Writing TYPO3_ACTIVE_FRAMEWORK_EXTENSIONS environment variable
[localhost] < The following TYPO3 core extensions are marked as active: about, adminpanel, backend, belog, beuser, core, extbase, extensionmanager, felogin, filelist, fluid, fluid_styled_content, form, frontend, impexp, info, install, recordlist, redirects, reports, rte_ckeditor, scheduler, seo, setup, sys_note, t3editor, tstemplate, viewpage
[localhost] < Register typo3/cms-composer-installer file in root package autoload definition
[localhost] < Inserting TYPO3_COMPOSER_MODE constant
[localhost] < Generating class alias map file
[localhost] < Inserting class alias loader into main autoload.php file
[localhost] < Executing TYPO3 installer scripts
[localhost] > cd /Users/pq/projects/xxxx/_deploytest/releases/20190521095837 && cd public/typo3conf
[localhost] > if [[ -f LocalConfiguration.php && ! -s LocalConfiguration.php ]]; then echo 'true'; fi
[localhost] > cd /Users/pq/projects/xxx/_deploytest/releases/20190521095837 && cd public/typo3conf && ln -s /Users/pq/projects/xxx/_deploytest/shared/typo3conf/LocalConfiguration.php LocalConfiguration.php
[localhost] > cd /Users/pq/projects/xxx/_deploytest/releases/20190521095837 && cd public/typo3conf && ln -s /Users/pq/projects/xx/_deploytest/shared/typo3conf/PackageStates.php PackageStates.php
• done on [localhost]

So, what do you think we need to fix?

Well, first I want to find out if I've done anything wrong.

Try test('-f LocalConfiguration.php && ! -s LocalConfiguration.php')

This outputs
[localhost] > if -f LocalConfiguration.php && ! -s LocalConfiguration.php; then echo 'true'; fi
[localhost] < sh: -f: command not found

I found the problem. I am in the wrong workdir.
[localhost] > cd /Users/pq/projects/kreativrausch/_deploytest/releases/20190522082821 && cd public/typo3conf
[localhost] > pwd
[localhost] < /Users/pq/projects/xx/typo3-cms-xx
/Users/pq/projects/xx/typo3-cms-xx/deploy.php:92:
string(56) "/Users/pq/projects/xx/typo3-cms-xx"
[localhost] > if [[ -f LocalConfiguration.php  &&  ! -s LocalConfiguration.php ]]; then echo 'true'; else echo 'false'; fi
[localhost] < false

run('cd {{release_path}} && cd {{typo3_webroot}}/typo3conf');
    $x = run("pwd");
    var_dump($x);
    $x = run("if [[ -f LocalConfiguration.php  &&  ! -s LocalConfiguration.php ]]; then echo 'true'; else echo 'false'; fi");
    var_dump($x);
    die('LINE : ' . __LINE__ . ' METHOD : ' . __METHOD__);

Maybe test() should switch to the dir I've cd'd to ?
Passing in the fullpath works fine of course.

Hmmm, but I think if it cd to dir it will break code for many mode devs.
Now this is solution:
test('[[ -f {{release_path}}/{{typo3_webroot}}/typo3conf/LocalConfiguration.php  &&  ! -s {{release_path}}/{{typo3_webroot}}/typo3conf/LocalConfiguration.php ]]')

Maybe with optional parameter?
On Thu, 23 May 2019, 08:22 Anton Medvedev<EMAIL_ADDRESS>wrote:

Hmmm, but I think if it cd to dir it will break code for many mode devs.
Now this is solution:
test('[[ -f {{release_path}}/{{typo3_webroot}}/typo3conf/LocalConfiguration.php  &&  ! -s {{release_path}}/{{typo3_webroot}}/typo3conf/LocalConfiguration.php ]]')
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/deployphp/deployer/issues/1845?email_source=notifications&email_token=ABBJODHN7LRLM7SJNDS3ORDPWYZYVA5CNFSM4HOJNTUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWBGBFY#issuecomment-495083671,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBJODCD2VHKXIECTCSOOLLPWYZYVANCNFSM4HOJNTUA
.

Fixed in master. Please verify.


added_class_zoom_and_wipe

#i have created the classes wipe and zoom but i am 100 percent sure that its wrong also i didnt called the clases so plzz see my code and tell me where to change and what are the thinks i should do to fix the code it will be of great help to me if u would allow me to work with it so that i can learn

Fixes Issue

Description

Check List (Check all the applicable boxes)

[ ] I understand that my contributions needs to pass the checks.
[ ] If I created new functions / methods, I documented them and add type hints.
[ ] If I modified already existing code, I updated the documentation accordingly.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Thanks for your PR @indra057!
I will review this later next week, but you can already take a look at the different checks and read why they fail: you should be able to pass all checks :-)
An error I see is that your implementation imports from manim, but will fail with code from manimlib.
If you can, you should run pre-commit run --all-files locally to format the files (pip install pre-commit is needed).

Any update @indra057?

sorry sir i dont now how to fix it ...as my test is failing...can u guide me with little help or with some previous pr so that i can look and try to fix issue

Did you have a look at the code from Manim?
For example, this is how they implement an animation based on others: https://docs.manim.community/en/stable/_modules/manim/animation/transform.html

Closing as completed by #285


Count of valid or invalid fields

Hi,
is there any way how to get a number of valid/invalid fields before submitting whole form by submit button? I need to create some progress bar in the sidebar and update it automatically during filling my form. Is there any way how to do it with jQuery Validation? Maybe any asynchronous solution?
Note: I need to get only these numbers, not trigger validation messages.
Thanks for your help.
Martin

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually, especially after several years with no activity. It doesn't mean I'm abandoning the project, just that I'm unable to work through 200+ open issues with the little time I can afford to spend on this project.
To the reporter (or anyone else interested in this issue): If you're still affected by the same issue, please consider opening a new issue, with a testpage that demonstrates the issue with a current version of the plugin. Even better, make an attempt to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!


Implemented MMR search for Redis

Description: Implemented MMR search for Redis. Pretty straightforward, just using the already implemented MMR method on similarity search–fetched docs.
Issue: #10059
Dependencies: None
Twitter handle: @hamza_tahboub

cc @Spartee @tylerhutcherson

Will review tomorrow.

@Spartee had a chance to take a look at this?

It's next up after these slides :)

@nfcampos thanks, made a few changes to fix all of that, including implementing a similarity_search_by_vector function as has been done for some other vector scores, and I double-checked sure that everything still works. Let me know if everything looks all right now!

@Spartee added some integration tests. Let me know if that looks okay.

Hey @Spartee, had a chance to take a look at the changes and tests yet?


Started gvi django project

Just created basic django project.

Oh you so crazy.
Great use of the django-admin command!


called Option::unwrap() on a None value

Problem
use super::*;

#[derive(Props)]
pub struct DropdownMenuProps<'children> {
    dropdown_content: Element<'children>,
    controller_open: Element<'children>,
    controller_closed: Element<'children>,
}

pub fn DropdownMenu<'children>(cx: Scope<'children, DropdownMenuProps<'children>>) -> Element {
    let open = use_state(cx, || false);
    let onclick = |_| open.set(!open.get());

    cx.render(rsx!(
        div { class: "DropdownMenu",
            div { class: "controller-content", onclick: onclick,
                if *open.get() {
                    &cx.props.controller_open
                } else {
                    &cx.props.controller_closed
                }
            }
            div { class: "dropdown-content",
                open.get().then_some(()).and_then(|_| cx.props.dropdown_content.as_ref())
            }
        }
    ))
}

panicked at 'called `Option::unwrap()` on a `None` value', /home/niedzwiedz/.cargo/git/checkouts/dioxus-1e619ce595d3799d/6acec6f/packages/core/src/diff.rs:213:57

above component crashes after I toggle it 3 times.
Steps To Reproduce
dioxus = { git = "https://github.com/DioxusLabs/dioxus", rev = "c97f43146d17abca049f77a616ae6b116d90ca34" }
dioxus-web = { git = "https://github.com/DioxusLabs/dioxus", rev = "c97f43146d17abca049f77a616ae6b116d90ca34" }

discord thread:
https://discord.com/channels/899851952891002890/899851952891002893/1108757735522177147

Could you post the props you pass into the element in the example where it crashes?
I tested this code and could not reproduce the issue:
use dioxus::prelude::*;

fn main() {
    dioxus_web::launch(app);
}

fn app(cx: Scope) -> Element {
    let dynamic = String::from("testing");
    cx.render(rsx!(DropdownMenu {
        dropdown_content: render! {
            div { "Hello, world! {dynamic}" }
        },
        controller_open: render! {
            div { "Close {dynamic}" }
        },
        controller_closed: render! {
            div { "Open {dynamic}" }
        },
    }))
}

#[derive(Props)]
pub struct DropdownMenuProps<'children> {
    dropdown_content: Element<'children>,
    controller_open: Element<'children>,
    controller_closed: Element<'children>,
}

pub fn DropdownMenu<'children>(cx: Scope<'children, DropdownMenuProps<'children>>) -> Element {
    let open = use_state(cx, || false);
    let onclick = |_| open.set(!open.get());

    cx.render(rsx!(
        div { class: "DropdownMenu",
            div { class: "controller-content", onclick: onclick,
                if *open.get() {
                    &cx.props.controller_open
                } else {
                    &cx.props.controller_closed
                }
            }
            div { class: "dropdown-content",
                open.get().then(|| cx.props.dropdown_content.as_ref())
            }
        }
    ))
}

https://github.com/Niedzwiedzw/dioxus-crash-borrow-mut
I created a repo that showcases the exact scenario... I needed to recreate it cause the original contains some SECRET stuff :D but this is the same scenario and it's somewhat minimal @Demonthos

I have a guess what is happening here:

When components are diffed the prop of the new component is taken
If a component is reused between renders, the component has no props the second time it is used which causes the panic

hmm so is it a bug or am I doing something against the rules?

hmm so is it a bug or am I doing something against the rules?

This is an unintended consequence of the rule that you cannot use an element twice in rsx (in your example they end up getting used twice: Once in the first render, and once in the second render). I would consider it a bug.
One possible solution is to wrap the props in Rc to keep the reference alive longer so that if you reuse a prop multiple times, the reference will stay alive until the last use is dropped.

hmmm but am I not returning a closure which creates a new element every time?

by the way, if you scroll down, there is another panic - something about borrowing some value while it is already borrowed. not sure which one causes the other, but the error didn't occur without using shared state right?

hmmm but am I not returning a closure which creates a new element every time?

You do in the parent component (CounterDropdown), but when the child component (DropdownMenu) reruns by itself because of the open hook updating, the elements in the props are the same as the elements used in the initial render with a component that has had it's props .taken

by the way, if you scroll down, there is another panic - something about borrowing some value while it is already borrowed. not sure which one causes the other, but the error didn't occur without using shared state right?

That is pretty normal with rust/WASM panics. It is the same bug

use std::{
    collections::hash_map::DefaultHasher,
    hash::{Hash, Hasher},
};

use dioxus::prelude::*;
use tracing::debug;

fn calculate_hash<T: Hash>(t: &T) -> u64 {
    let mut s = DefaultHasher::new();
    t.hash(&mut s);
    s.finish()
}

/// dioxus has a limitation:  components cannot be reused... this works around that
pub fn frag<'scope, Props, Builder, ScopeProps>(
    cx: Scope<'scope, ScopeProps>,
    props: Props,
    builder: Builder,
) -> Element<'scope>
where
    Props: Hash,
    Builder: Fn(Props) -> LazyNodes<'scope, 'scope>,
{
    let hash = calculate_hash(&props);
    debug!(
        "[{}] -> [{}]: fragment is redrawing",
        std::any::type_name::<ScopeProps>(),
        std::any::type_name::<Props>()
    );

    cx.render(rsx! {
        Fragment {
            key: "{hash}",
            builder(props)
        }
    })
}

so I wrote this as a workaround, could someone please double check me? are there any downsides to doing it this way?


Update using-admin-roles.md

Added the word "the" to the table to make it read easier.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LolaJacobsen PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


wildfly-service.exe and jbosspass wrong with # inside

Hi there,
I am installing wildfy 10.1.0 as service on a win 20012 server.
It is working but I cannot stop the service.
I figured out where the problem is coming from.
When I installe Wildly as a service, I run the following command:

E:\Products\wildfly-10.1.0.Final\bin\service>service.bat install /serviceuser .\JBoss /servicepass my#pass /controller localhost:9991 /jbossuser myuser /jbosspass my#pass

But I can see whe I am trying to stop the service, the command running is:

E:\Products\wildfly-10.1.0.Final\bin\jboss-cli.bat  --controller=localhost:9991 --connect --user=myuser --password=my" "pass --command=:shutdown

Any workaround?
Thanks,
S.

Not experienced in this, but did you try to pass password into quotation marks?
service.bat ... /jbosspass "my#pass"

Yes - It is a bug  - # is used as a delimitator to parse parameters...
On Mon, May 15, 2017 at 11:10 AM, Jan Kalina<EMAIL_ADDRESS>wrote:

Not experienced in this, but did you try to pass password into quotation
marks?
service.bat ... /jbosspass "my#pass"
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wildfly/wildfly-core/issues/1999#issuecomment-301419446,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHNKT4Atv-rkQa8OyOXmE3yXNiqI65koks5r6BZrgaJpZM4LBDud
.

--
-----
Sébastien Cuber

If this is an issue please file an issue at http://issues.jboss.org/browse/WFCORE.


how to change ui framerwork to nextui

You can remove Tailwind and install any UI framework as per their instructions.


Create Youtrack issue for publishing problems with enableGranularSourceSetsMetadata

Gonna do the HMPP changes everywhere as they're fixed by Jetbrains


Grafana Unexpected "}" in operand

BUG REPORT:
Version of Helm and Kubernetes:
kubectl version
Client Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.6", GitCommit:"7fa1c1756d8bc963f1a389f4a6937dc71f08ada2", GitTreeState:"clean", BuildDate:"2017-06-16T18:34:20Z", GoVersion:"go1.7.6", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.4", GitCommit:"d6f433224538d4f9ca2f7ae19b252e6fcb66a3ae", GitTreeState:"clean", BuildDate:"2017-05-19T18:33:17Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}

$ helm version
Client: &version.Version{SemVer:"v2.5.0", GitCommit:"012cb0ac1a1b2f888144ef5a67b8dab6c2d45be6", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.5.0", GitCommit:"012cb0ac1a1b2f888144ef5a67b8dab6c2d45be6", GitTreeState:"clean"}

Which chart:
stable/grafana
What happened:
running helm lint or helm install or helm upgrade results in
[ERROR] templates/: parse error in ... template ... /grafana/templates/job.yaml:41: unexpected "}" in operand
What you expected to happen:
no template error
How to reproduce it (as minimally and precisely as possible):
Download LATEST commit 912f50c7165383834fc4706ffbe162dd7068b9dd and run helm lint. Previously, everything worked, but starting with the new commit (without version change) our deployment pipeline fails.
Anything else we need to know:
Would be nice, if you change chart version numbers when updating the charts and run a helm lint.

just saw the same.

This will be fixed in #1458.


Fixes/mark html as safe 2.7

Backport of #313

Coverage increased (+6.4%) to 68.53% when pulling 9bd72b92a2c60475f4eb8550c3784488a568ea3d on czpython:fixes/mark-html-as-safe-2.7 into 452a98f6abdafc30d980eb044962ebc721234583 on divio:support/2.7.x.

Coverage increased (+6.5%) to 68.595% when pulling f1676b8532ccc8e200be0b64eb791b20b1793ae1 on czpython:fixes/mark-html-as-safe-2.7 into 452a98f6abdafc30d980eb044962ebc721234583 on divio:support/2.7.x.

Left some comments, LGTM otherwise.

Coverage increased (+6.6%) to 68.672% when pulling 2312c8ba6d4768dac497c3fa77409a3c7d8052d0 on czpython:fixes/mark-html-as-safe-2.7 into 452a98f6abdafc30d980eb044962ebc721234583 on divio:support/2.7.x.

LGTM


getQuerydata undefined on route change - NextJs

Hi,
I'm using NextJs to create a chat app. Left hand side are chat rooms and right hand side is the current chat room opened. When a user type a messager I call useMutation() with onmutate where I use optimistic update. So far so good, I cancel the queries, setQueryData, then onsuccess invalidate queries.
However as soon as I change room, call router.push and try sending a mutation.mutate(data) again it fails
It fails because in my onmutate function, setQueryData("mykey", oldData) -> oldData is undefined.
Is there a way to persist the queryData on route change ? Especially because there was no page refresh. So I don't understand why getQueryData is now undefined.

Do you have a reproduction please? Where are you creating your queryClient?

Tricky to reproduce quickly as there are websocket and it's quite a big structure. Will try with a simplified version.
In the meantime I'm creating my query client in _app.js.
In [id].js I fetch chat rooms and current room messages server side. I have 2 components:
ChatMedsages.js and ChatRooms.js
Inside the ChatMessages component I use useInfiniteQuery and useMutation fetching messages from the current room.id. Basically all works great until I call router.push and try mutateAddMessage.mutate(data) again... Optimistic update fails as old data are undefined.

@TkDodo I ran into this issue in my application as well. This only seems to happen when navigating to the same component (re-render, vs. re-mount).
My reproduction shows that the queryCache contains the correct data, but useQuery isn't aware of it after a query key change.
https://codesandbox.io/s/usequery-hydration-bug-fqf6i

I think https://github.com/tannerlinsley/react-query/issues/1978 is related  or a duplicate

@rodymolenaar thanks for the reproduction. I could solve the issue by moving the queryClient creation from the function MyApp to outside of it:
https://codesandbox.io/s/goofy-swirles-xzpir?file=/pages/_app.js
it's the only change I made.
As far as I understand it, it's fine to create a new QueryClient in getServerSideProps, because you only dehydrate it. On the frontend though, the cache must stay stable. You can also see that in my version, the devtools now correctly show all queries, while in your version, it was only showing the post with id 1. I'm viewing the hydration as basically "spreading" the state that we get from the backend over the state we have on the frontend. By keeping the cache stable, we just "add" values to the cache with every page transition. My experience with NextJs is limited (okay - nonexistent to be honest) - apologies if that analogy is incorrect :)

Ah yes, you're right. I've been previously aware of this fact.
However, I think our concern with creating the QueryClient outside of the MyApp function was that it causes the same QueryClient instance to be used for all users. Not sure if this is a legitimate concern, however it did solve our instability issues.
I found
const queryClientRef = React.useRef();

if (!queryClientRef.current) {
  queryClientRef.current = new QueryClient();
}

at https://react-query.tanstack.com/examples/nextjs#_top
So it seems this has been figured out, but it looks like it isn't documented anywhere else.

you can also use state to lazily initialize a queryClient (and potentially create a new one in an effect when the user changes), I've written about that here: https://tkdodo.eu/blog/use-state-for-one-time-initializations
it is also advisable to clear the cache with queryClient.removeQueries when a user logs out to avoid sharing that data.

you can also use state to lazily initialize a queryClient (and potentially create a new one in an effect when the user changes), I've written about that here: https://tkdodo.eu/blog/use-state-for-one-time-initializations
it is also advisable to clear the cache with queryClient.removeQueries when a user logs out to avoid sharing that data.

I'd like to rephrase my previous comment:

However, I think our concern with creating the QueryClient outside of the MyApp function was that it causes the same
QueryClient instance to be used for all users. Not sure if this is a legitimate concern, however it did solve our instability
issues.

The issue we were experiencing probably had to do with SSR. The QueryClient outside of the custom app component would not change for every render on the server and create React hydration issues when the server's QueryClient had different data than the client's. Because of this, we were forced to create the QueryClient inside of the render function.
Seeing the Github issues where unstable QueryClient instances is the most probable cause, it might warrant putting this information somewhere in the docs. Since just lifting the QueryClient out of the custom app component is also probably problematic.

I believe you could also use this (shorter) syntax to initialize once:
const resource = React.useRef(new Resource());

please PR to the docs if you think they can be improved 👍

I believe you could also use this (shorter) syntax to initialize once:

It can, but bear in mind that useRef´ doesn't have a lazy initializer. So the new Resource()will be created each render, and then discarded. Might not matter, but it also might. I preferuseState` for that :)

please PR to the docs if you think they can be improved 👍

I believe you could also use this (shorter) syntax to initialize once:

It can, but bear in mind that useRef doesn't have a lazy initializer. So the new Resource() will be created each render, and then discarded. Might not matter, but it also might. I prefer useState for that :)

Ah check, learned something new :)
I'll try and find this information a good home in the docs, thanks!

please PR to the docs if you think they can be improved 👍

I believe you could also use this (shorter) syntax to initialize once:

It can, but bear in mind that useRef doesn't have a lazy initializer. So the new Resource() will be created each render, and then discarded. Might not matter, but it also might. I prefer useState for that :)

Ah check, learned something new :)
I'll try and find this information a good home in the docs, thanks!

@TkDodo Actually this was already added to the docs in February, I just completely missed it 🤡

Create a new QueryClient instance inside of your app, and on an instance ref. This ensures that data is not shared between different users and requests.

https://github.com/tannerlinsley/react-query/commit/862bb2be3a7c6161b70a2310f4661859adb6d943#diff-f8a7d7140c7558298858c319f0ab8931140c6dd6dbc87598ec4bddc346ad60ad

oh very nice, now I've learned that we need to do it inside the app, but it has to be stay stable :)
So I'll close this issue as we've found the solution I guess

Hi,
I'm using NextJs to create a chat app. Left hand side are chat rooms and right hand side is the current chat room opened. When a user type a messager I call useMutation() with onmutate where I use optimistic update. So far so good, I cancel the queries, setQueryData, then onsuccess invalidate queries.
However as soon as I change room, call router.push and try sending a mutation.mutate(data) again it fails It fails because in my onmutate function, setQueryData("mykey", oldData) -> oldData is undefined.
Is there a way to persist the queryData on route change ? Especially because there was no page refresh. So I don't understand why getQueryData is now undefined.

I'm having the same problem, have you solved it?

Hi,
I'm using NextJs to create a chat app. Left hand side are chat rooms and right hand side is the current chat room opened. When a user type a messager I call useMutation() with onmutate where I use optimistic update. So far so good, I cancel the queries, setQueryData, then onsuccess invalidate queries.
However as soon as I change room, call router.push and try sending a mutation.mutate(data) again it fails It fails because in my onmutate function, setQueryData("mykey", oldData) -> oldData is undefined.
Is there a way to persist the queryData on route change ? Especially because there was no page refresh. So I don't understand why getQueryData is now undefined.

Hi. I'm having the same problem. Have you solved it?


MWPW-142972 | Integrating review comments

Integrating review comments for delay authoring of Start over button. 0-99 will be considered as sec and 99+ will be ms
Resolves: MWPW-142972
Test URLs:

Before: https://main--cc--adobecom.hlx.page/drafts/mathuria/interactiveelems/interactive-workflow4?martech=off
After: https://mwpw-142972--cc--aishwaryamathuria.hlx.live/drafts/mathuria/interactiveelems/interactive-workflow4?martech=off?martech=off

Validation done in below url :
https://mwpw-142972--cc--aishwaryamathuria.hlx.page/drafts/mathuria/interactiveelems/interactive-workflow4


fix input and division errors, add optional vocabulary file generation

Running distance.py under Python 3.6 results in a divide-by-zero error, and an error for the raw_input command, which was renamed input() under PEP 3111 (https://docs.python.org/3/whatsnew/3.0.html). The proposed changes fix these issues.
Because the pre-trained GloVe vector sets don't come with vocabulary files, I also added the option to automatically generate a vocabulary file from the input vectors on-the-fly. The vocabulary file does not get saved out, only used while the script is running.

I merged the raw_input change.  Thanks.
I'm pretty sure the vocab file generation is not working correctly as per my earlier comment.  If you want to fix that, please feel free to do so and resubmit the pull request.


Error while installing jupyter notebook in anaconda

From @npatwa on October 12, 2016 14:10
An unexpected error has occurred, please consider sending the
following traceback to the conda GitHub issue tracker at:
https://github.com/conda/conda/issues

Include the output of the command 'conda info' in your report.
Traceback (most recent call last):
File "C:\Anaconda3\lib\site-packages\conda\cli\main.py", line 208, in args_func
File "C:\Anaconda3\lib\site-packages\conda\cli\common.py", line 612, in inner
File "C:\Anaconda3\lib\site-packages\conda\cli\main_install.py", line 46, in execute
File "C:\Anaconda3\lib\site-packages\conda\cli\install.py", line 413, in install
File "C:\Anaconda3\lib\site-packages\conda\plan.py", line 527, in execute_actions
File "C:\Anaconda3\lib\site-packages\conda\instructions.py", line 148, in execute_instructions
File "C:\Anaconda3\lib\site-packages\conda\instructions.py", line 95, in UNLINK_CMD
File "C:\Anaconda3\lib\site-packages\conda\install.py", line 674, in unlink
File "C:\Anaconda3\lib\site-packages\conda\install.py", line 311, in mk_menus
File "C:\Anaconda3\lib\site-packages\menuinst__init__.py", line 23, in
from .win32 import Menu, ShortCut, dirs
File "C:\Anaconda3\lib\site-packages\menuinst\win32.py", line 13, in
from .knownfolders import get_folder_path, FOLDERID
File "C:\Anaconda3\lib\site-packages\menuinst\knownfolders.py", line 152, in
SHGetKnownFolderPath = windll.shell32.SHGetKnownFolderPath     # [5] [3]
File "C:\Anaconda3\lib\ctypes__init_.py", line 364, in getattr
func = self.getitem(name)
File "C:\Anaconda3\lib\ctypes__init__.py", line 369, in getitem
func = self._FuncPtr((name_or_ordinal, self))
AttributeError: function 'SHGetKnownFolderPath' not found
Copied from original issue: conda/conda#3609

From @silentquasar on February 8, 2017 7:39
Ever come to a solution on this?

From @brombeerberg on March 23, 2017 11:36
Got a very similar error today while installing Anaconda2 4.3.1 and Anaconda2 <IP_ADDRESS> on Windows XP using the zip archive as the Computer is offline:
...
SHGetKnownFolderPath = windll.shell32.SHGetKnownFolderPath # [5] [3]
 File `"C:\Programme\Anaconda2\lib\ctypes\__init__.py",` line 375, in __getattr__
 func = self.__getitem__(name)
 File "C:\Programme\Anaconda2\lib\ctypes\__init__.py", line 380, in __getitem__
 func = self._FuncPtr((name_or_ordinal, self))
 AttributeError: function 'SHGetKnownFolderPath' not found

After ignoring during the installation process I noticed that icons were missing and I couldn't start Spyder.

Any resolution to above?
I got similar issue during installation of 'navigator' for anaconda 2.2.0 in windows xp.


Android module dependency

This PR removes the android dependency for core module and adds a new module for android
Please @juanpedromoreno could you review? Thanks

Minor comments added and even the code coverage has been decreased, but this code and this job deserve a huge LGTM!

Thank you men, I decreased the test coverage 2.7%. It's not a big deal I think. The next PR will add some tests :smile:


Allow Import of Existing Friendly Name to Email Mapping from Tautulli

Use Case: User already has friendly name to Plex user email mapping assigned in Tautulli
Current State: User manually re-enters mapping in site
Proposal: Provide a method for a user to supply Tautulli instance URL and API key to the client, call the API's get_users endpoint to retrieve the Tautulli user list, deliver the necessary payload to the site, and populate the email to friendly name mapping.
Benefits

Simplifies onboarding
Encourages users to mask email addresses
Lowers barrier of entry for users with larger user rolls

Resources
get_users endpoint
http(s)://{{instance_ip}}/{{http_root}}/api/v2?apikey={{api_key}}&cmd=get_users
Example Response
    "response": {
        "result": "success",
        "message": null,
        "data": [
            {
                "row_id": 1,
                "user_id": 31010000,
                "username"<EMAIL_ADDRESS>                "friendly_name": "Sally",
                "thumb": "https://plex.tv/users/643aaa0000aaa0a0/avatar?c=158000000",
                "email"<EMAIL_ADDRESS>                "is_active": 1,
                "is_admin": 0,
                "is_home_user": 0,
                "is_allow_sync": 1,
                "is_restricted": 0,
                "do_notify": 0,
                "keep_history": 1,
                "allow_guest": 0,
                "server_token": "uCaNh4sMyT0k3n",
                "shared_libraries": "4;1;3;2",
                "filter_all": "",
                "filter_movies": "",
                "filter_tv": "",
                "filter_music": "",
                "filter_photos": ""
            },
            {
                "row_id": 2,
                "user_id": 36850000,
                "username"<EMAIL_ADDRESS>                "friendly_name": "Robb",
                "thumb": "https://plex.tv/users/643aaa0000aaa0a1/avatar?c=1258000001",
                "email"<EMAIL_ADDRESS>                "is_active": 1,
                "is_admin": 0,
                "is_home_user": 0,
                "is_allow_sync": 1,
                "is_restricted": 0,
                "do_notify": 0,
                "keep_history": 1,
                "allow_guest": 0,
                "server_token": "uCaNh4sMyT0k3n",
                "shared_libraries": "1;2",
                "filter_all": "",
                "filter_movies": "",
                "filter_tv": "",
                "filter_music": "",
                "filter_photos": ""
            },
            {
                "row_id": 3,
                "user_id": 17000,
                "username": "amnesia",
                "friendly_name": "phutur1st",
                "thumb": "https://plex.tv/users/643aaa0000bfa0a1/avatar?c=1358000001",
                "email"<EMAIL_ADDRESS>                "is_active": 1,
                "is_admin": 1,
                "is_home_user": 1,
                "is_allow_sync": 1,
                "is_restricted": 0,
                "do_notify": 0,
                "keep_history": 1,
                "allow_guest": 0,
                "server_token": "AbRaCaDabRa",
                "shared_libraries": "1;4;2;3;5;999999;6",
                "filter_all": "",
                "filter_movies": "",
                "filter_tv": "",
                "filter_music": "",
                "filter_photos": ""
            },
            {
                "row_id": 4,
                "user_id": 12180001,
                "username": "Ash",
                "friendly_name": "Ash",
                "thumb": "https://plex.tv/users/cd24560a0a0a0a0a/avatar?c=130000000",
                "email": "",
                "is_active": 1,
                "is_admin": 0,
                "is_home_user": 1,
                "is_allow_sync": 1,
                "is_restricted": 1,
                "do_notify": 1,
                "keep_history": 1,
                "allow_guest": 0,
                "server_token": "aBraCaDabRa",
                "shared_libraries": "6;4;1;3;5;2",
                "filter_all": "",
                "filter_movies": "",
                "filter_tv": "",
                "filter_music": "",
                "filter_photos": ""
            }
        ]
    }
}

Thank you for the detailed request. This is closed by this commit: https://github.com/Notifiarr/notifiarr/pull/115/commits/decc061783b2a225435955874f2dd5be3ee88de4
This feature will be included in the next release.


Types inside the renamed file are seen as duplicate

Environment data
dotnet --info output:
 Version:            1.0.0-preview2-003121
 Commit SHA-1 hash:  1e9d529bc5

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.12
 OS Platform: Darwin
 RID:         osx.10.12-x64

VS Code version:
Version 1.7.2 (1.7.2)
7ba55c5860b152d999dda59393ca3ebeb1b5c85f

C# Extension version:
1.5.3

Steps to reproduce
Load a project, rename a file which has a class in it.
Expected  behavior
Should work as it has been before.
Actual behavior
It doesn't unload the type inside the deleted file and see the type inside new file as a duplicate:

This is the same bug in https://github.com/OmniSharp/omnisharp-vscode/issues/785 and https://github.com/OmniSharp/omnisharp-vscode/issues/803

In addition to this one, there is also another bug which is related to file delete. If you delete a file, the members inside that file still seems valid (e.g. shows up on IntelliSense list, etc.). Should I open a separate issue for this?

@tugberkugurlu I'm able to recreate that bug too. I think it's related. The Intellisense seems to not update when any existing files are renamed or deleted; however, it works fine when you're adding new files that didn't previously exist. Hoping this gets fixed, it's been a reported issue since September (https://github.com/OmniSharp/omnisharp-vscode/issues/785).

@tugberkugurlu I'm able to recreate that bug too. I think it's related. The Intellisense seems to not update when any existing files are renamed or deleted; however, it works fine when you're adding new files that didn't previously exist. Hoping this gets fixed, it's been a reported issue since September (https://github.com/OmniSharp/omnisharp-vscode/issues/785).

The problem is that at the moment, upon rename, the Workspace only raises a DocumentAdded event for the renamed doc, but never raises DocumentRemoved for the old one.
If you continue renaming 10 times, you end up with 10 copies of the same document in the workspace.

Yeah, we need to fix that.

OK. We're going to use https://github.com/OmniSharp/omnisharp-vscode/issues/785 to track fixing this issue.


Adding CE2 information to js fetch backends doc

I had a bit of trouble identifying what the artifact name would be for CE2, so I figured this might help others in the future so they don't need to browse https://repo1.maven.org/maven2/com/softwaremill/sttp/client3/ to find the dependency
Before submitting pull request:

[x] Check if the project compiles by running sbt compile
[x] Verify docs compilation by running sbt compileDocs
[x] Check if tests pass by running sbt test
[x] Format code by running sbt scalafmt

Thanks!

Also I think Any effect implementing the cats-effect Concurrent typeclass can be used. is no longer valid - in CE3 it's Async if I remember correctly?


Link to new definitions of pipeline/component from various docs

A number of the Pipelines docs include either a wordy definition of a pipeline/component, or a link to some page or other in an attempt to get around defining a pipeline/component yet again.
We now have handy definitions of pipeline and component in our new concepts docs.
We need to go through the docs, move/merge the long definitions into the new concept docs, and leave all the other docs with a very short definition of each concept plus a link to the relevant concept doc.
This may apply to other concepts too.

You've also added a component specification reference page in #604, and in many of the places where it's relevant to link to the concept docs, it may be useful to link to the specification as well (or instead). In addition, perhaps the component concept page and the component specification page should link to one another.
(Feel free to separate this into a new issue; I was going to file one, but it seemed related.)

/assign


Fix ghosts getting spawned in nullspace

About the PR

When a body is gibbed and destroys organs the brain gets destroyed before the ghost can be spawned, which puts it in nullspace. This fixes it by making ghosting run before the entity is deleted instead of delaying it till later.
Players that are on a map that gets deleted, now spawn as a ghost on another random map.
Fixes #26366
Fixes #22625
Why / Balance

Technical details

Ghosts now get spawned on the same tick that the entity is deleted, so that the coordinates can still be used for calculating spawn position.
Moved ghost spawning to GhostSystem.
Commits are mostly viewable in order.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

fix: Fixed screen freeze in some cases when getting gibbed or butchered.

This will probably fix #26366 and #22625

Related #27124


Make rendering content based on the user easier (auth tag helper)

Make rendering content based on the user context easier.

Render content only when the user is authorized.
Render content only when the user is anonymous
Render content only for a specific user
Render content when a user has a claim.
Render content when a user has a claim with a specific value
Determine the user based on a specific authorization policy

Sameple usages:
Render content only for authenticated users.

Render content only for anonymous users.

Render content only for a specific user.

Render content when a user has a claim.

Render content when a user has a claim and a specific value.

Authorize a user with a specific policy:

I've created a quick and dirty proof of concept below. It doesn't do roles and needs all the bells and whistles.
https://github.com/aspnet/Mvc/blob/javiercn/auth-taghelper/src/Microsoft.AspNetCore.Mvc.TagHelpers/AuthorizationTagHelper.cs

@DamianEdwards @rynowak

A bit like this one I wrote for the workshop we teach: https://github.com/jongalloway/aspnetcore-app-workshop/blob/master/src/FrontEnd/TagHelpers/AuthzTagHelper.cs
It's rough, but it's the same idea.

Meant to say, yes, I absolutely agree, which is why I built that one :smile:

I think is similar idea to a closed issue #5018

Definitely a good idea, but not high enough priority for 2.1. If anyone decides to write up a few of these and ship their own NuGet package, we'd be happy to link to it.

Dupe of #3785


What about grouping the sample per service category ?

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Mention any other details that might be useful
The samples are all ordered by service (e.g. AKS, CosmosDB...) and not by service category (e.g. Compute, Databases...) which then end up following an alphabetical order:
aci/
acr/
adx/
aks/
appservice/
aro/
cosmosdb/
eventhubs/
functions/
group/
keyvault/
mssql/
mysql/
postgresql/
redis/
servicebus/
spring-cloud/
storage/

If you look at something like Azure Charts, the services are ordered per category:
AI + Machine Learning
Analytics
Compute
Databases
Development
Identity + Security
IoT + MR
Integration
Management + Governance
Media + Comms
Migration
Networking
Storage

What about reordering the samples and grouping them by category ? We could end up, either with an extra directory:
analytics/
  adx/
  eventhubs/
compute/
  aci/
  acr/
  aks/
  appservice/
  aro/
  functions/
core/
  group/
databases/
  cosmosdb/
  mssql/
  mysql/
  postgresql/
  redis/
development/
  spring-cloud/
integration/
  servicebus/
security/
  keyvault/
storage/
  storage/

Or with a prefix of the category:
analytics-adx/
analytics-eventhubs/
compute-aci/
compute-acr/
compute-aks/
compute-appservice/
compute-aro/
compute-functions/
core-group/
databases-cosmosdb/
databases-mssql/
databases-mysql/
databases-postgresql/
databases-redis/
development-spring-cloud/
integration-servicebus/
security-keyvault/
storage-storage/

We can even flatten the entire structure:
analytics-adx-create-database/
analytics-adx-create-table/
analytics-adx-create/
analytics-adx-ingest-client/
analytics-eventhubs-create-eventhub/
analytics-eventhubs-create-namespace/
analytics-eventhubs-receive-event/
analytics-eventhubs-send-event
...

I would have no problem with reordering. Just not by what Azure Charts does, but rather by what the Azure Portal does as that is the official source. Note that if you do the reordering you will have to take into account that all the READMEs need to be touched ;)

Yes, it makes sense to take the official Azure Portal organization.
So @mnriem, what do you like the best ?

Extra Sub directories

analytics/
  adx/
    create-database/
    eventhubs/
  eventhubs/

Sub directories

analytics-adx/
    create-database/
    eventhubs/
analytics-eventhubs/

Flat structure

analytics-adx-create-database/
analytics-adx-eventhubs/
analytics-eventhubs-create/

I quite like the flat structure (it's also easier to point at external files/readmes in terms of relative paths) but it can quickly end up having hundreds of directories (one per example)

@agoncal For me it would be 1.


packaging tool

See CPack

CPack is a powerful, easy to use, cross-platform software packaging tool distributed with CMake since version 2.4.2. It uses the generators concept from CMake, to abstract package generation on specific platforms, and it can be used with or without CMake.
Using either a simple configuration file or the CMake module, a complex project can be packaged into an installer.

example:
SET(CPACK_CMAKE_GENERATOR "Unix Makefiles")
SET(CPACK_GENERATOR "STGZ;TGZ;TZ")
SET(CPACK_INSTALL_CMAKE_PROJECTS "/home/andy/vtk/CMake-bin;CMake;ALL;/")
SET(CPACK_NSIS_DISPLAY_NAME "CMake 2.5")
SET(CPACK_OUTPUT_CONFIG_FILE "/home/andy/vtk/CMake-bin/CPackConfig.cmake")
SET(CPACK_PACKAGE_DESCRIPTION_FILE "/home/andy/vtk/CMake/Copyright.txt")
SET(CPACK_PACKAGE_DESCRIPTION_SUMMARY "CMake is a build tool")
SET(CPACK_PACKAGE_EXECUTABLES "ccmake;CMake")
SET(CPACK_PACKAGE_FILE_NAME "cmake-2.5.0-Linux-i686")
SET(CPACK_PACKAGE_INSTALL_DIRECTORY "CMake 2.5")
SET(CPACK_PACKAGE_INSTALL_REGISTRY_KEY "CMake 2.5.0")
SET(CPACK_PACKAGE_NAME "CMake")
SET(CPACK_PACKAGE_VENDOR "Kitware")
SET(CPACK_PACKAGE_VERSION "2.5.0")
SET(CPACK_PACKAGE_VERSION_MAJOR "2")
SET(CPACK_PACKAGE_VERSION_MINOR "5")
SET(CPACK_PACKAGE_VERSION_PATCH "0")
SET(CPACK_RESOURCE_FILE_LICENSE "/home/andy/vtk/CMake/Copyright.txt")
SET(CPACK_RESOURCE_FILE_README "/home/andy/vtk/CMake/Templates/CPack.GenericDescription.txt")
SET(CPACK_RESOURCE_FILE_WELCOME "/home/andy/vtk/CMake/Templates/CPack.GenericWelcome.txt")
SET(CPACK_SOURCE_GENERATOR "TGZ;TZ")
SET(CPACK_SOURCE_OUTPUT_CONFIG_FILE "/home/andy/vtk/CMake-bin/CPackSourceConfig.cmake")
SET(CPACK_SOURCE_PACKAGE_FILE_NAME "cmake-2.5.0")
SET(CPACK_SOURCE_STRIP_FILES "")
SET(CPACK_STRIP_FILES "bin/ccmake;bin/cmake;bin/cpack;bin/ctest")
SET(CPACK_SYSTEM_NAME "Linux-i686")
SET(CPACK_TOPLEVEL_TAG "Linux-i686")

It would be cool to have this kind of tool, maybe by improving the package task :+1:

@uael The package task is only used to generate common cross-platform package for package dependence  management and it's a builtin action.
But the cpack feature will generate various package format files. So I'm going to add a new plugin task to handle this feature. It is totally irrelevant to the package task. for example:
$ xmake pack -k deb
$ xmake pack -k rpm -o ...
$ xmake pack -k other_pkg_type -o ...

We uses xmake package to support it in the future. e.g.  for iOS/MacOS app
xmake package

It will generate .dmg/.ipa installation file. @see https://xmake.io/#/guide/project_examples?id=iosmacos-program


XFA - Get line height from the font

when the CSS line-height property is set to 'normal' then the value depends of the user agent. So use a line height based on the font itself and if for any reasons this value is not available use 1.2 as default.
it's a partial fix for https://bugzilla.mozilla.org/show_bug.cgi?id=1717681.

Almost all the xfa reftests should failed but overall I'd say it's an improvement.
/botio test

There is a regression in firefox-xfa_imm1295e-page2 and firefox-xfa_imm1344e-page2.

/botio test

/botio makeref


TS-5103: replace ua_raw_buffer_reader with ua_buffer_reader

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1275/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1170/ for details.

Yes, this looks like the correct change to me.  We should be checking that there is more data on the ua_buffer_reader.  In this case, there should always be data on the ua_raw_buffer_reader, so we will be incorrectly blind tunneling GET requests without keep alive enabled. (I think).
We only want to flip to tunnel mode if there is data after the already parsed header which would be reflected by the  ua_buffer_reader.


Make TypeChecker's Context and Diags fields private

Remove the remaining external uses of TypeChecker's Context field, and make it private along with Diags.
Also remove TypeChecker fields from a few walkers, flipping things such that instead of storing a TypeChecker and asking for the ASTContext when needed, store an ASTContext, and ask for a TypeChecker when needed.

@swift-ci please smoke test

⛵


Document logging options for Bots

https://github.com/medplum/medplum/pull/2528 introduced the ability to configure bot logging to fit high volume use cases.
The documentation needs to be updated accordingly
Proposed changes in this document: https://www.medplum.com/docs/bots/bots-in-production

Add a section Configuring Bot Logging
Note that when there are thousands of bot invocations per day you may want to tune the log level of bots. Given an example of the kind of situations that may generate high-volume bots (e.g. ADT feeds, high-frequency device messages, etc.)
There are two levers for controlling bot log levels (see this PR)  a. What type of events are logged (error, success etc) known as auditEventTrigger (see here)
b. Where the events are logged
i. Option 1. To DB so you can see it in the app.
Best practice is to always do this to start, for debugging / testing
ii. Option 2. To log only, not visible in Medplum app but logged - better for high volume

I'll work on this

Fixed by #2814


cutting edges

not showing whole image cut the edges adjust by itself

I'm not sure what this is about. Please re-open with more details if still an issue.


Migrate from Material UI to Shadcn UI

Change dependencies.
Port the project from material ui to shadcn ui.

Done


vcpkg-tool: 2023-12-12 -> 2024-01-11

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/microsoft/vcpkg-tool/releases.
meta.description for vcpkg-tool is: Components of microsoft/vcpkg's binary
meta.homepage for vcpkg-tool is: https://github.com/microsoft/vcpkg-tool
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2024-01-11 in filename of file in /nix/store/lawps8073cfvawk0ili3vm09fq0afmvf-vcpkg-tool-2024-01-11

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
vcpkg
vcpkg-tool

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lawps8073cfvawk0ili3vm09fq0afmvf-vcpkg-tool-2024-01-11 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A vcpkg-tool https://github.com/r-ryantm/nixpkgs/archive/20f4c5233d4a7a5d955ac045cc171160fb66dcfa.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/20f4c5233d4a7a5d955ac045cc171160fb66dcfa#vcpkg-tool

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lawps8073cfvawk0ili3vm09fq0afmvf-vcpkg-tool-2024-01-11
ls -la /nix/store/lawps8073cfvawk0ili3vm09fq0afmvf-vcpkg-tool-2024-01-11/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

vcpkg
vcpkg-tool

Maintainer pings
cc @Guekka @gracicot for testing.

Result of nixpkgs-review pr 281764 run on x86_64-darwin 1

2 packages built:

vcpkg
vcpkg-tool


Uniswap v3 staking contract update

[x] replaced param _length in creativeIncentive function by _startTime & _endTime.
[x] updated uniswap staker contract address
[x] Build Connector on Arbitrum
[ ] Deploy and comment Address

Deployed on Polygon:
Name: UNISWAP-V3-STAKER-A
Address: 0x35339B06890B52e72C3A4f29c983e082aa87E873

Deployed on Arbitrum:
Name: UNISWAP-V3-STAKER-A
Address: 0x8dA60dee0815a08d16C066b07814b10722fA9306

Deployed on Mainnet:
Name: UNISWAP-V3-STAKER-A
Address: 0x6ea1fb0bd094b926f37b22fe5846e7a0f8378b42


A minimum height of 200 by default

Just paying some attention to a thing which is probably not that much of a bug.
If I start a waybar with an empty config or a config like this:
{     "height": 15 }
(so, it's basically an empty bar with no modules), it shows up as expected, but outputs a message like this:
Requested height: 30 exceeds the minimum height: 200 required by the modules
Bar configured (width: 1366, height: 200) for output: LVDS-1

So, there is this number 200 showing up. As soon as I add a clock module to my config (others not tested), it starts working normally.
Thus, just a small note.

Should be fixed on master :)


[EventRequest] Add OnUpdateReservationOnBeforeTempTrackingSpecificationInsert in table Warehouse Activity Line

Hello, please consider adding the following event to table 5767 "Warehouse Activity Line"
    [IntegrationEvent(false, false)]
    local procedure OnUpdateReservationOnBeforeTempTrackingSpecificationInsert(var TempTrackingSpecification: Record "Tracking Specification"; WarehouseActivityLine: Record "Warehouse Activity Line")
    begin
    end;

and raise it in the following procedure
    local procedure UpdateReservation(TempWhseActivLine2: Record "Warehouse Activity Line" temporary; Deletion: Boolean)
    var
        TempTrackingSpecification: Record "Tracking Specification" temporary;
    begin
        with TempWhseActivLine2 do begin
            if ("Action Type" <> "Action Type"::Take) and ("Breakbulk No." = 0) and
               ("Whse. Document Type" = "Whse. Document Type"::Shipment)
            then begin
                InitTrackingSpecFromWhseActivLine(TempTrackingSpecification, TempWhseActivLine2);
                TempTrackingSpecification."Qty. to Handle (Base)" := 0;
                TempTrackingSpecification."Entry No." := TempTrackingSpecification."Entry No." + 1;
                TempTrackingSpecification."Creation Date" := Today;
                TempTrackingSpecification."Warranty Date" := "Warranty Date";
                TempTrackingSpecification."Expiration Date" := "Expiration Date";
                TempTrackingSpecification.Correction := true;
                OnUpdateReservationOnBeforeTempTrackingSpecificationInsert(TempTrackingSpecification, TempWhseActivLine2);
                TempTrackingSpecification.Insert();
            end;
          [...]
    end;

Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 42495.