add DEPLOY_PATH support

Building an index-top.html that has a publicPath of DEPLOY_PATH, is required to support releasing with the s3 deploy github action.
This PR sets up that support and fixes places where assets are loaded relative to the index.html so they will instead be loaded relative to the public path.
This PR combined with this other PR:
https://github.com/concord-consortium/collaborative-learning/pull/1455
Should give CLUE the support for the new deployment approach.

[x] with this change stamps in the drawing tool cause new firebase images to be created each time the drawing tool is loaded. This is because the stamp ends up with a absolute URL. And the ImageMap automatically uploads absolute URLs to firebase. However the drawing tools image objects do not store the uploaded url ccimg://fbrtdb..., so each time they request a new entry from the image map the content is uploaded again. We need a solution for this, perhaps identify asset urls better in the ImageMap. Or update the drawing tool so it replaces the absolute URL with the firebase URL.
[x] check tile and document export of tiles with images. This code seems to make assumptions about images with filenames. If the url is absolute perhaps it will break something here.

To address the issue with the stamps, updated all the places that render stamps to use the ImageMap and I removed the pre process snapshot in the stamp object. So now the stamp.url will stay relative. And the urls of the ImageObject's in the drawing tool will have these same relative URLs.
This also changed how legacy URLs are handled in the drawing tool:
Legacy URLs in drawing tool saved state (assets/...) will not be converted. The URLs are still handled, the saved state is just not updated. This approach seems best to move the legacy conversion all into a single place.
The approach of relative URLs means that if a stamp is removed from a future version of the curriculum the stamp will be broken in documents based on the old curriculum.  This is not new behavior it is is how it worked before.
I checked the export of document JSON and it seems to be working fine even when javascript and curriculum is in a different place than the index.html. What it seems to do is assume any image in the document is located in an images folder for the unit path. Despite my worries, it seems this behavior is not affected by changes in the PR.

@tejal-shah I added you as a reviewer so you could see what kinds of code and build system changes are needed to support this new approach to doing releases.  I think CLUE will be our most complicated project to migrate to the new s3 deployment since it allows user to upload images and has json content with relative references to images.
The AP has json content with images but they aren't relative references. Question Interactives allows student to upload attachments but again these are handled as absolute URLs.  And I think customized stamps can be made in the Question Interactives drawing interactives but these are also absolute URLs.
There are some projects that download collections of resources from the project build, those might have some of these complications. I think some of the Geo projects do this.

Thanks @scytacki for including me! I'm wondering if we'll need to make similar changes for codap v3 as well where, not right now, but in the future, there will be .codap, .csv and image file downloads.

@tejal-shah, yes perhaps if there are some standard CODAP documents that are included in the CODAP repository and downloaded from the same place as the CODAP javascript files then they will need a similar treatment.  I'd guess CODAP doesn't use something like the ImageMap that CLUE has, so I'd guess these changes will be more simple.

I'm wondering if we'll need to make similar changes for codap v3 as well where, not right now, but in the future, there will be .codap, .csv and image file downloads.

I don't think so. The issue that CLUE has is that it's storing references to curriculum content and to user-uploaded images in user content. When CODAP imports a document it just reads the data without storing any internal links to assets/resources, which is where the tricky issues come up.


BEEFY: disaster-recovery: losing validator keys

With BEEFY there is the concern of enough validators losing their session/authority keys and thus BEEFY getting stuck.
The probability of this happening is very low, but impact is critical in case of trustless bridges based on BEEFY light-clients.
Example scenarios with probability:

AWS or GCP have a massive, even temporary, outage impacting 1/3 of active validators - if they lose ephemeral session keys during this event, consensus is broken - probability very low,
assuming validators rotate session keys often (which they don't currently, but theoretically could/should), say BEEFY finality stops because of some bug (but GRANDPA continues ok) and it takes multiple sessions until we deploy a fix, it could happen that validators simply rotate their ephemeral keys out of existence (there is no mechanism preventing this rn) - probability low.

Impact

On Polkadot/Kusama (local) chain, this can be fixed through governance (or some other sudo) - impact high, but fixable
In case of a trustless bridge like Polkadot <> Ethereum, where we want the bridge to be completely decentralized and trustless and be controlled/controllable only by Polkadot Governance, there is no fix: if the last active validator set cannot reach consensus, any governance intention/decision/action on Polkadot cannot be bridged over and acted upon on Ethereum; basically the BEEFY light-client on Ethereum side cannot be updated - impact fatal, bridge is dead, funds are lost (locked forever).

Solutions/Ideas
On-chain encrypted backups for session keys (fix on Polkadot side)
Validators use a master key (can be their controller or stash key - which is likely very much not ephemeral, but on some hardware device, or KMS in case of controller keys, or similar) to encrypt the private key part of their ephemeral session keys pair and post the encrypted backup on-chain where it can be recovered.
Cons: "breaking" change, adds complexity to validators
BEEFY light-client ForceChangeAuthorities support (fix on Ethereum side)
BEEFY light-client has fallback ForceChangeAuthorities mechanism to "force" new authorities.
One idea for implementing this is using GRANDPA keys or even controller keys of the authority set active in the last bridged header to sign a Smart Contract upgrade "custom" payload (either setting new BEEFY keys, or upgrading light-client, etc).
Validity/authenticity of the GRANDPA or controller keys signing this "force" change can be proven on ethereum side using Polkadot storage proofs verified against last bridged header state_root.
Cons:

a lot of complexity on light-client side to support this "force" upgrade path,
very expensive (doesn't really matter in a disaster scenario) - verifying Polkadot storage proofs in EVM, verifying sr25519 signatures in EVM - would it even fit an eth block?

Others?
cc @burdges, @vgeddes, @andresilva

Thanks for writing this up! Is definitely a tricky problem to solve.
Some notes from my side:

@seunlanlege developed a Solidity prover for substrate storage proofs here: https://github.com/polytope-labs/solidity-merkle-trees#merkle-patricia-trie. I vaguely recall the gas cost for verifying a proof was around 120,000, which isn't too bad.

Isn't this already a risk with GRANDPA/BABE session keys which would first devastate the network's liveness

Isn't this already a risk with GRANDPA/BABE session keys which would first devastate the network's liveness?

Yes, same risk there, but impact is not as bad as governance can ForceSetChange GRANDPA/BABE authorities.
In case of bridge-scenario, governance cannot act on the Eth side of the bridge while bridge is not functioning.

In case of bridge-scenario, governance cannot act on the Eth side of the bridge while bridge is not functioning.

Contracts generally will have admin keys or some kind of admin functionality, This is entirely for safety reasons which this scenario would fall under.

Contracts generally will have admin keys or some kind of admin functionality, This is entirely for safety reasons which this scenario would fall under.

We are pursuing an architecture where this is not the case: fully trustless with no admin other than chain governance.

We do not various options here but..
It's likely simplest to do a backup encrypted to the controller, or encrypted to the controller plus its stash, or some other combination.  We'd need to support controllers and stashes in all combinations of SR25519, Ed25519, and secp256k1 yes?

It's likely simplest to do a backup encrypted to the controller, or encrypted to the controller plus its stash, or some other combination.

On-chain encrypted backup, or did you have smth else in mind?

We'd need to support controllers and stashes in all combinations of SR25519, Ed25519, and secp256k1 yes?

yes, I don't see why we'd do only a subset

An on-chain backup maybe easier, but we could store off-chain however folks like
Actually I've an amusing idea:
A BEEFY node creates secp256k1 and Ristretto/Ed25519 secret keys, called x and y, with which it computes key exchanges with all other validator's controller keys, which produces 3f+1 = 1000ish random [u8; 32]s.  It treats these as scalars in the F_r field for BLS12-381, creates a Lagrange polynomial f that interpolates the first 3f+1 = 664ish of these as f(1), f(2), etc, and then creates a string M of the f = 333ish bitmasks that make the others fit the polynomial.  It then treats the evaluation z = f(0) as the BLS12-381 secret key.
Now finally the BEEFY node computes the public keys (X,Y1,Y2) for x and y, makes up an ephemeral key pair (u,U) of the key type as the controller public key C, computes the key exchange k = u * C, and forms a ciphertext Z, U, X, Y1, Y2, AEAD_k(M).
What happens if BEEFY keys are lost?  A controller key owner can download decrypt and validate their Z, X, Y1, Y2, M, so they can then ask 2/3 of the other controller key holders to compute key exchanges with whichever of X, Y1, Y2 fits them.   After these are returned then it reinterpolates f and recomputes z = f(0).
Is this better than simply encrypting to the controller?  It ensures a compromised controller cannot decrypt the BEEFY secret key, but one expect controllers would be air gaped, so one being compromised sounds unlikely.  And both controller and stash being compromised sound even more unlikely.  It avoids deployment complexity, but it makes the recovery pretty nasty, so meh.


Make bots split if they have to much mass before they hit the player

Sometimes a fed bot is chasing you, would be nice if the bot would be smart enough to split feed you.

if distance is less than x*radius of master cell (or if coordinates of master-cell are close to the coordinates of the bot), and for i; in each bot's bot.mass is more than 50 mass, bot.split()

it is even easier:
 getDistanceBetweenBalls: function(ball_1, ball_2) {
        return Math.sqrt( Math.pow( ball_1.x - ball_2.x, 2) + Math.pow( ball_2.y - ball_1.y, 2) );
    },

I believe it's better to implement something like https://github.com/davidmann4/agario-feeder-bot/issues/12 and https://github.com/davidmann4/agario-feeder-bot/issues/13 then you will get full control of bots.

yeah I think it is the most common way to do it :)
would need a own userscript + server - currently we piggyback on the minimap userscript:
https://github.com/dimotsai/agar-mini-map

if you have good AI > player controlled
most of the scripts ony provide macro managment capabilities

Well, minimap script works fine with bots as far as I have tested and you have minimap feature :)
Anyway there're some others scripts for bots that could be used, just need adapt server code to work and use some or all of the script features.

done


Error if file is empty

I create new empty file and got alert:
An error has occurred. See error log for more details.
validator cannot be null

please post the stack trace from /path/to/workspace/.metadata/.log when this happens.

Sorry, after reinstall all package I can't repeat this bug


ngRequired

Hi there,
I need to make an element required based on the value of another, so I need to use ng-required, but then ngValidate is not taking in consideration it.
Any solution?

You can use a function to determine whether te required validation should be performed on a field. For example see https://stackoverflow.com/a/23953089/2572602 where they use such a function on the minlength validator.


Problems with new dev experience on fresh Windows install

Describe the bug
I just got a new laptop and decided to be a naïve new developer for a minute. I installed VS Code (and Visual Studio), installed Git for Windows (I wasn't a completely naïve new dev), and read the README file. I then installed Docker Desktop, Skaffold, and Task as recommended. All went well at first, but then I hit the following snags.
When I ran "task setup" for the first time, Windows wouldn't let me run the Powershell script to download sena-3.zip and elawa.zip: default execution policy (Restricted) forbids running .ps1 scripts. I had to read and understand the error message taking me to the Microsoft documentation, and understand that I had to run Set-ExecutionPolicy to set the execution policy to RemoteSigned. Then I was able to run task setup and then task up. (This is where I briefly stopped pretending to be a naïve dev, and skimmed through the documentation looking for the exact syntax since I already understood the concepts. If I'd been a complete newbie, I would probably have taken 1-2 hours to understand what I needed to do here).
The first time I ran task up, it failed because I hadn't realized that the Skaffold file I downloaded was a standalone exe, not an installer. That was simple to solve, though.
When I browsed to localhost and signed in as admin, everything looked good... until I clicked on a project (sena-3). Then I got a bunch of error messages about Mongo connections not being able to be established, and I couldn't load the project. I had to clone the web-languageforge repo and get LF up and running before I could proceed with testing local Lexbox.
Finally, as a long-standing Linux dev I wanted to get my SSH private and public keys copied over to the new laptop so I could clone git@github.com:sillsdev/languageforge-lexbox, but in the character of a naïve dev I would probably not be as familiar with the Windows SSH subsystem as an experienced Linux dev would be.
Based on the mistakes I made while pretending I was a brand-new dev unfamiliar with the codebase, I suggest the following changes to our README:
Changes needed

We should mention the need to run Set-ExecutionPolicy before task setup in our README
We should expand a little on the instructions for putting Skaffold in your PATH
We may want to guide Windows devs through the process of setting up an SSH public/private keypair and uploading it to Git (Linux devs can be presumed familiar with SSH keypairs and will need much less guidance)

It's possible the Git credentials manager is sufficient and git push can be done over HTTPS; I'll have to experiment a little

Lexbox using a local copy of Language Forge is good, but there should be a graceful fallback if Language Forge isn't available at all. Being completely unable to access any project pages in the absence of a running LF is not great DX.

That last one might be extracted to a different issue, as that will involve code changes, whereas the other points are just things to note in the README.

I would suggest that cross-platform Git Credential manager is the recommended way to authenticate with git and that SSH keys auth is less desirable.  I think our README should reflect that.  GCM is easier, especially if you don't have SSH setup.

I created #1092 to track that last bullet point. The rest, I will fix with a PR that updates the README.


closure calls its own deprecated installStyles method

This code: https://github.com/google/closure-library/blob/master/closure/goog/editor/field.js#L2273
goog.editor.Field.prototype.installStyles = function() {
  if (this.cssStyles && this.shouldLoadAsynchronously()) {
    goog.style.installStyles(this.cssStyles, this.getElement());
  }
};

gives us the warning:
/closure/goog/editor/field.js:2270: ERROR - Property installStyles of type goog.style has been deprecated: Use {@link #installSafeStyleSheet} instead.
    goog.style.installStyles(this.cssStyles, this.getElement());

Thanks for the report.
On Sep 9, 2016 6:23 AM, "Nick Santos"<EMAIL_ADDRESS>wrote:

This code: https://github.com/google/closure-library/blob/master/
closure/goog/editor/field.js#L2273
goog.editor.Field.prototype.installStyles = function() {
if (this.cssStyles && this.shouldLoadAsynchronously()) {
goog.style.installStyles(this.cssStyles, this.getElement());
}
};
gives us the warning:
/closure/goog/editor/field.js:2270: ERROR - Property installStyles of type goog.style has been deprecated: Use {@link #installSafeStyleSheet} instead.
goog.style.installStyles(this.cssStyles, this.getElement());
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/closure-library/issues/755, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABMDKqcwmrZyg0Kqdn-m61V2kWbxWLAyks5qoV21gaJpZM4J5Cw7
.


azure-agent: switch back to upstream WALinuxAgent

@Phreedom's fixes were merged upstream.
Upstream is now ahead of @Phreedom's fork.

LGTM


Cron is fired after trigger deletion

Hi,
I was searching for an existing issue, could not find any.
The problem is that we periodically see that Alarms package is firing the triggers that were deleted. Here is an example:
[2017-09-11T00:45:02.501Z] [INFO] [??] [alarmsTrigger] [deleteTrigger] trigger<EMAIL_ADDRESS>successfully deleted from memory

[2017-09-11T00:45:11.357Z] [INFO] [??] [alarmsTrigger] [fireTrigger] Trigger<EMAIL_ADDRESS>was successfully fired

I have quickly scanned the code, could not find any obvious reason. Hopefully you could help to shed light here.
Thats the alarmsFeedTests.scala in blue.
Thanks,
Vadim.

Closed this one, since the problem was related to the test setup.


folding in output panels

It's very helpful to add folding to output panels like in vscode:
this way you can fold the references or diagnostics of certain files and focus on the ones you want.
and it's as simple as adding
    "gutter": True,
    "fold_buttons": True

to OUTPUT_PANEL_SETTINGS.

Yes, but the position of the fold button depends on the update, if theres a new warning in the folded file
(here src\provider_references.jl):
 `´◌ src\provider_formatting.jl:
        1:21  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable JSONRPC
        1:76  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable DocumentFormattingParams
        2:34  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable URI2
        3:14  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable JSONRPC
DocumentFormattingParams

 >◌ src\provider_references.jl: ##

the fold button (>) moves to the added warning:
 `´◌ src\provider_formatting.jl:
        1:21  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable JSONRPC
        1:76  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable DocumentFormattingParams
        2:34  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable URI2
        3:14  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable JSONRPC
DocumentFormattingParams

 ◌ src\provider_references.jl:
 >     158:53  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable LintState ##

and the fold button goes up if a warning is removed:
 `´◌ src\provider_formatting.jl:
        1:21  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable JSONRPC
        1:76  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable DocumentFormattingParams
        2:34  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable URI2
>        3:14  	LanguageServer.LSDiagnostic	warning   	Possible use of undeclared variable JSONRPC
DocumentFormattingParams ##

I just tested this and confirm the fold buttons do not keep a correct position after updating the panel.
Also, the fold buttons do not appear until at least two lines appear under a file.
Given these issues I'm not really enthusiastic to pursue it further, but feel free to revive it if the fixes are simple.

#462


[FEATURE] wRPC MVP Integration

Affected project(s)

[ ] documentation
[ ] examples
[X] wasmCloud host
[x] wasmCloud CLI (wash)
[ ] wasmCloud dashboard UI (washboard)
[x] capability providers
[x] provider bindgen
[x] control interface client
[ ] other / not sure

Is your feature request related to a problem? Please describe.
This is a tracking issue to coordinate our work on wrpc, as we merge the MVP into the host.
Describe the solution you'd like
wrpc usable and merged in the host
Describe alternatives you've considered
N/A
Additional context
No response

We shouldn't forget to add some proper typing for operation that we're passing around which is currently stringly-typed:
See this comment

[ ] Properly type the operation that moves around the host, since we don't have to do backwards compat with Smithy anymore (this should be similar to TargetInterface)

PolicyRequestTarget (and the policy service interaction in general) needs to be re-done to match with what's available in a post-wrpc world:
See this comment

[ ] Change policy service interaction to match what will be available

Also, we'll need to eventually derive async Encode as well, see this comment after some work is done in upstream wrpc_transport.

Just so I don't lose track of it, we're instrumenting traces for custom interfaces but need to make sure that for all of our well-known ones (like HTTP, blobstore, etc) that we're using the wasmcloud_host::wasmbus::wasmcloud_transport::Client to send those invocations. Might need a bit of modification of handlers in order to pass around the context.

One more, I'd love to refactor the Bus trait
    /// Identify the target of component interface invocation
    async fn identify_interface_target(
        &self,
        interface: &CallTargetInterface,
    ) -> anyhow::Result<Option<TargetEntity>>;

to both drop the notion of a TargetEntity, and not return a Result to be clear. We'll immediately fail to invoke the target if the target is None so that should be easily handled.
    /// Identify the target of component interface invocation
    async fn identify_interface_target(
        &self,
        interface: &CallTargetInterface,
    ) -> Option<WrpcInterfaceTarget>;

We can rename, of course.

A note for onlookers when it comes to wRPC support for interfaces, we plan on supporting the following interfaces in wasmCloud:

WASI 0.2.0 (including wasi:http)
Keyvalue (Depending on formalization of wasi:cloud, wasi:keyvalue or similar)
Blobstore (Depending on formalization of wasi:cloud, wasi:blobstore or similar)
Messaging (Depending on formalization of wasi:cloud, wasi:messaging or similar)
Custom interfaces with all data types. See WIT Types for what constitutes as a type, but generally enums, structs, and all simple data types are included. Custom interfaces will not support resources, streams, and asynchronous functions for wasmCloud 1.0, but support for those is possible and will be added in a future release.

For all of the interfaces that depend on formalization of wasi:cloud, we will either offer that functionality using the wasi:cloud interface or a similar interface like wasmcloud:keyvalue in the case that the interface just isn't quite ready yet. Once that interface is available, however, we plan on deprecating our wasmcloud:* interface and polyfilling that to use the wasi:cloud interface, supporting it until 2.0.

Some more things before we can merge feat/wrpc:

[ ] Update all providers to use the stuffed-into-target-config setup for config
[ ] Build messaging-triggered actor to replace HTTP-triggered actor that was critical to most other tests
[ ] Update all provider tests (ex. tests/kv-vault.rs) to use the target config setup

Also related to the topics that wasmbus.rpc should be listening on:

[ ] wasmbus.rpc.<lattice>.<provider>.linkdefs.put is not getting the traffic it should during a link put.

Would be great to have a test for the new policy code

[ ] Add tests for the policy service changes

Adding OTEL to more bindgen'd code would be good:

[ ] Add missing TraceContext to get_wrpc_client()

Some more things:

[ ] re-add integration tests for providers (i.e. files that were removed from tests in https://github.com/wasmCloud/wasmCloud/pull/1633)
[ ] re-add wash call integration test (i.e. tests that were removed in https://github.com/wasmCloud/wasmCloud/pull/1645)

Add a check for an invocation coming in should have had a link set the other way -- i.e. when an actor comes in and connects to a provider, and the provider needs to call some interface on the actor (an example of this is blobstore's chunk-receiver legacy interface)

[ ] Validate invocations coming in to make sure they're from linked providers

is this now done? @vados-cosmonic @brooksmtownsend

@rvolosatovs As far as I know no, the points that still have checkboxes next to them are unfinished, or at the very least need to be re-checked if they were finished during some other work.

Only remaining work here is to validate traces and add a test for wash call.

WIth #1869 , we're done!


Is there a way to increase select-ability size of edges?

Right now, especially when using dashed edge connection lines, they are quite difficult to select with the mouse for deletion. It would be nice to be able to add some padding around the edge connections so they are easier to select with the mouse and can be deleted easier. Is there a solution for this that already exists? Padding prop to the edges perhaps?

I'm also running into this issue.
I guess you could increase the edge width, but it would be nice if there was some way to make the selection a little easier and maintain the narrow lines.
Increasing the width on hover might have this effect.

Was able to do this with css:
path.react-flow__edge-path:hover {
  stroke-width: 4;
}

There might be a better way to do this, but the effect is actually quite nice.

Thanks @ChuckJonas I'll check this out. Appreciate the help

We changed the pointer-events of the edges to visiblestroke. That was a bad idea.. I will look into it.

HI I have the same issue. I round not find the way to increase edge select region.
This a a workaround be not very effective.  Still a bit too narrow for some users.
  stroke-width: 4;
}

Is there any solution for this.

The solution I used was to use a custom edge that rendered 2 copies of the path, one transparent with a large stroke, and one that I wanted to see. Works really well,
jsx
<path style={style} className="react-flow__edge-path-selector" d={path} markerEnd={markerEnd} fillRule="evenodd" />
<path style={style} className="react-flow__edge-path" d={path} markerEnd={markerEnd} fillRule="evenodd" />

css
.react-flow__edge-path-selector:hover + .react-flow__edge-path, .react-flow__edge-path:hover, .react-flow__edge-path-selector:hover  {
    stroke: #555;
    cursor: pointer;
}
.react-flow__edge-path-selector {
    fill: none;
    stroke: transparent;
    stroke-width: 28;
}

The solution I used was to use a custom edge that rendered 2 copies of the path, one transparent with a large stroke, and one that I wanted to see. Works really well,
jsx
<path style={style} className="react-flow__edge-path-selector" d={path} markerEnd={markerEnd} fillRule="evenodd" />
<path style={style} className="react-flow__edge-path" d={path} markerEnd={markerEnd} fillRule="evenodd" />

css
.react-flow__edge-path-selector:hover { cursor: pointer }
.react-flow__edge-path-selector:hover + .react-flow__edge-path, .react-flow__edge-path:hover  {
    stroke: #555;
    cursor: pointer;
}
.react-flow__edge-path-selector {
    fill: none;
    stroke: transparent;
    stroke-width: 28;
}

I implemented this 6 months ago haven't had a single complaint from users.

Hi RickeyWard,
sorry about my question, but where did you create that two new path?

@EmanueleGurini You'll need to create a copy of an existing edge (or make a custom one from scratch) to modify it and set it as a custom edge type. Basic understanding can be pulled from the Edges example https://reactflow.dev/examples/edges/
I apologize I don't have time to produce the full example at the moment. Happy holidays.

@EmanueleGurini You'll need to create a copy of an existing edge (or make a custom one from scratch) to modify it and set it as a custom edge type. Basic understanding can be pulled from the Edges example https://reactflow.dev/examples/edges/
I apologize I don't have time to produce the full example at the moment. Happy holidays.

Amazing: solution works well!
Thank's indeed for your contribution and answer. Happy holidays!

I'm facing this issue today, and there is my solution.
Base on the CustomEdge.js example, add this below the <path> line:
{/* A transparent path to extend the path width*/}
<path style={{ strokeWidth: 40, stroke: "initial" }} d={edgePath} />

This is the preferred solution: https://github.com/wbkd/react-flow/issues/1211#issuecomment-883673705 Thanks @RickeyWard

If anybody else stumbles into this issue, it looks like interactionWidth was added to the edge options which seems to achieve this natively https://github.com/wbkd/react-flow/commit/51e957e5dbafb55578ad432d348c582b73c8b9c9

Good point @benlongo . We added this feature in v11 :)

Does this not work for custom edges?
I send interactionWidth to it, but it doesn't get affected (interaction stays the same)

@AmmarZeer you can use the BaseEdge if you want to use interactionWidth https://reactflow.dev/docs/api/edges/base-edge/

Tried to use BaseEdge with EdgeLabelRenderer and pass interactionWidth -> no lucky. No interaction zone at all.

For those using Custom Edge with BaseEdge and who hate writing CSS. This pretty much gets the job done:
    <BaseEdge
        path={edgePath}
        {...props}
        style={{
          stroke:
            isNodeHovered
              ? "#EF4444"
              : "white",
        }}
      />

      <BaseEdge
        path={edgePath}
        {...props}
        style={{
          strokeWidth: "25px",
          opacity: 0,
        }}
      />


update readme

Codecov Report

Merging #1944 (529c467) into main (73d3a81) will increase coverage by 0.00%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main    #1944   +/-   ##
=======================================
  Coverage   61.07%   61.08%
=======================================
  Files         108      108
  Lines       11459    11459
=======================================
+ Hits         6999     7000    +1
+ Misses       3939     3937    -2
- Partials      521      522    +1

see 5 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Issues with ETCD-backup-Policy

Issues have been reported, policy will be reviewed, tested and updated

@ch-stark I did not have the reported problem with the commands being re-ordered.  I'm not sure yet if the policy functions as expected.  I did have to make some additional edits:

The ClusterRoleBinding points to a service account and the namespace for that service account was needed.
Some of the policy that I copied and pasted ended up missing.  It was the volumes for the job.  Keep an eye out to see if you see that too.

Working on a simpler policy

https://github.com/ch-stark/etcd-backup-policy

I tried getting this working and never made good progress.  I suspect we would need to decide if we can have someone focus on fixing this policy.

Today Christian let me know that the policy linked in comment https://github.com/stolostron/policy-collection/issues/199#issuecomment-1021229256 was a good solution -- tried by someone that gave good feedback.  It does involve a custom image that needs to be created that goes along with the policy.  So need to decide if we want that to be created under stolostron.

The nice solution here is currently Tech Preview: https://access.redhat.com/documentation/en-us/openshift_container_platform/4.14/html/backup_and_restore/control-plane-backup-and-restore#creating-automated-etcd-backups_backup-etcd


Implement dual list selector

Implements the dual list selector.
Features
Props
Currently supported properties:

disabled
pane titles
options
pane statuses
onlistchange event

Currently unsupported properties but planned before merge:

[x] additional classes
[ ] additional custom callbacks for control buttons
[x] tooltip props
[ ] keyboard navigation
[ ] onoptionselect event

Currently unsupported properties which aren't planned for the merge:

id
tree
onoptioncheck event
aria
searchable
filtering
dragging
actions

Other stuff that needs to be done

[ ] I should document this as experimental and implement a feature gate
[ ] Insert a demo into the quickstart

Implementation Details
Similar to how the table is currently implemented, I've defined a new trait DualListSelectorItemRenderer which has a blanket implementation for types implementing Debug + Clone + PartialEq + ToHtml + 'static.
This type is used to render a single entry of a list in each of the two panes (available and chosen) by calling to_html.
Open Questions
1. I see anymap is listed as a dependency for the project but it doesn't seem to be used anywhere, is this what I should be using for passing custom props to the tooltips?
2. Patternfly-react is serving the svg of the arrows in the control column but we are using the fa icon which leads to the icons not being centered properly. Would it be possible to serve the svg instead?
3. Are there any further acceptance criteria for this MR that I'm not aware of?

Awesome!

Additional tooltip props are working. I didn't realize they were just asking for the completely normal TooltipProperties.

oh this is a new impl , that's cool!

The off-center control buttons are due to the width not being fixed to "1em". In patternfly-react all icons have a fixed width of 1em
@ctron Should we update the icons in a separate PR to also behave like this or would you prefer I just force the 1em width in this instance alone?

That's an interesting issue. So far, the icons we are using are font based, not SVG based. I think it would make sense in general to provide some more support for alternatives. Because an "icon" should just be Html. Now that makes it more difficult I guess.]
Not sure there's a better solution to this at the moment. Maybe just ignore the issue for now and create an issue, tracking this and adding the information which you found. That would be very helpful I guess.

I tried forcing the width by using the Icon::with_style but that doesn't have any effect. Only setting the width on the <i> seems to work. So leaving it for now as you said.

Keyboard handling seems out of scope since we're missing the handle_arrow_keys function and it's out of scope for this PR.
I'll open a separate issue for it.

You can now have a look at the demos at https://github.com/aDogCalledSpot/patternfly-yew-quickstart/tree/dual_list_selector


Fix placeholder support

This fix this issue: https://github.com/threepointone/glamor/issues/14#issuecomment-270473786

this looks just fine, thank you very much! I'll merge it in for now, will release through the weekend sometime.

@threepointone sorry to bother you, but do you know when the next version will be released?

Sorry for the delay, I'll pub within the next 2 hours.

sorry for the delay, this is live in 2.20.22

No problem, thanks 😉


Upgrade cobra dependency to v1.0.0

What this PR does / why we need it:
Upgrades cobra dependency to v1.0.0. This required a fair bit of work due to the issues described in https://github.com/kubernetes/kubernetes/pull/92735 ...
Requires #3057
Release note:
NONE

/kind cleanup

flake
/retest

to use a v1 suffix.

v1 is special and doesn't use the prefix, only v2 onwards will 😄


Bugfix/deseng575 Language picker should only appear on engagements with more than one language

Issue #: https://apps.itsm.gov.bc.ca/jira/browse/DESENG-575
Description of changes:

Language picker should only appear on engagements with more than one language
Language picker disappears completely when not viewing an engagement
Allow for the web app to see if there are available translations by language
Disable language switching itself for now until design decisions are made

I've had to kneecap the language switcher functionality until we can make some design decisions regarding what to do about:

Language switching for engagements only vs. whole-site
Sites that have differing languages between static and engagement content
Location of the language switcher (we may move it so it's nested in the engagement rather than in the header)

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

Project coverage is 75.77%. Comparing base (7d064cd) to head (1089a07).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2496      +/-   ##
==========================================
+ Coverage   75.69%   75.77%   +0.07%
==========================================
  Files         590      590
  Lines       21106    21093      -13
  Branches     1530     1525       -5
==========================================
+ Hits        15977    15983       +6
+ Misses       4882     4864      -18
+ Partials      247      246       -1

Flag
Coverage Δ

metapi
87.89% <100.00%> (+0.02%)
:arrow_up:

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

Files
Coverage Δ

...t-api/src/met_api/models/engagement_translation.py
96.82% <100.00%> (+0.33%)
:arrow_up:

...pi/src/met_api/resources/engagement_translation.py
98.91% <100.00%> (+0.14%)
:arrow_up:

...met_api/services/engagement_translation_service.py
88.65% <100.00%> (+0.74%)
:arrow_up:

met-web/src/apiManager/endpoints/index.ts
100.00% <ø> (ø)

met-web/src/components/common/LanguageSelector.tsx
90.00% <ø> (+36.51%)
:arrow_up:

.../src/components/engagement/view/EngagementView.tsx
76.31% <ø> (-0.61%)
:arrow_down:

met-web/src/constants/language.ts
100.00% <ø> (ø)

met-web/src/services/engagementService/index.ts
27.77% <ø> (ø)

LGTM!

Thanks @ratheesh-aot . I'm just finishing fixing the linter/test errors and will merge in a second


Minor fixes to installation instructions

Few small fixes walking through the install info

Remove the specific image tag, as it had problems with parsing components. the "latest" tagged version of the rudr image works, following the tutorial.


Failing tests in case of too old Ghostscript: "Error: File has unbalanced q/Q operators (too many q's)"

I had Ghostscript 9.19 from 2016 installed on my Windows 10 1809, which lead to errors like the following:
qpdf test 375 (convert original file to image) FAILED
cwd: C:/Users/[...]/Win32/Debug/out/qtest/qpdf
command: (cd tif1; gs 2>nul -q -dNOPAUSE -sDEVICE=tiff24nc -sOutputFile=a.tif - < ../large-inline-image.pdf)
 at qpdf.test line 4031.
	main::compare_pdfs("large-inline-image.pdf", "a.pdf") called at qpdf.test line 756
--> BEGIN EXPECTED OUTPUT <--
--> END EXPECTED OUTPUT <--
--> BEGIN ACTUAL OUTPUT <--

   **** Error: File has unbalanced q/Q operators (too many q's)
			   Output may be incorrect.
--> END ACTUAL OUTPUT <--
--> DIFF EXPECTED ACTUAL <--
--- C:/Users/[...]/testtemp.3676/expected	2019-02-14 14:10:06.794747700 +0100
+++ C:/Users/[...]/testtemp.3676/actual	2019-02-14 14:10:06.794747700 +0100
@@ -0,0 +1,3 @@
+
+   **** Error: File has unbalanced q/Q operators (too many q's)
+               Output may be incorrect.
--> END DIFFERENCES <--

Updating to the most current version 9.26 and testing manually on the shell with the mentioned files resolves the issues, not error message is printed and the expected file created. I didn't test any older newer version to know when exactly this has been fixed.
You might want to add some docs mentioning trying a newer version or such in the README for Windows. Feel free to close this otherwise, the issue itself might be enough documentation already.

Thanks for the report. I probably won't do anything with this...I think most people don't run the image comparison tests, and as you point out, it works with the latest ghostscript. The particular file in question is not compliant with the PDF spec, and some other viewers have trouble with it. Some viewers, including Adobe Reader, are able to handle it, and since files like this have been seen in the wild, I wanted to make sure qpdf could handle it.
I agree that this issue is good documentation -- if anyone else encounters it, hopefully they will find this issue, or if not, I will point them to it. Thanks again!


Verified: FlyQ EFB

I verified this app works with FlyQ EFB from Seattle Avionics.

Cool, thanks!

Cool, thanks!


chore: test function delete

What kind of change does this PR introduce?
tests
What is the new behavior?

use codegen client
handle cancellation

Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

16 of 18 (88.89%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+1.02%) to 36.351%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/functions.go
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

internal/debug/postgres.go
3
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
1.02%

Covered Lines:
1795

Relevant Lines:
4938

💛  - Coveralls


diskonaut: 0.3.0 -> 0.9.0

Bump diskonaut from 0.3.0 to 0.9.0

Motivation for this change
Diskonaut now measures file sizes consistently and tests can be run
without modification
discussion: https://github.com/imsnif/diskonaut/issues/50
tracking issue: https://github.com/imsnif/diskonaut/issues/56
implementation: https://github.com/imsnif/diskonaut/pull/66
Things done

remove special handling for checkPhase
tested on F2FS and EXT4

Tested with nix-shell -p nixpkgs-review --run "nixpkgs-review pr 93018" and works fine


TAB issue in text editor

Issue Type: Bug
When text is selected pressing TAB does not indent the selected text. In fact text editor loses focus and you focus different UI elements (buttons etc.)
Steps to reproduce:

add a code file (cpp in my case)
write some code :)
select some text
press TAB and code editor will lose focus
(I can't see what is focused on the first press but by continuing to press you will cycle throug activity bar buttons anf file tabs)

VS Code version: Code 1.26.1 (493869ee8e8a846b0855873886fc79d480d342de, 2018-08-16T18:38:57.434Z)
OS version: Windows_NT x64 10.0.17134
Extensions (8)

Extension
Author (truncated)
Version

code-gnu-global
aus
0.2.2

vscode-tfs
iva
0.6.0

csharpextensions
jch
1.3.0

cpptools
ms-
0.18.1

csharp
ms-
1.15.2

team
ms-
1.136.0

platformio-ide
pla
0.17.5

vscode-arduino
vsc
0.2.20

that is not all of the steps needed to reproduce this issue, it works fine for me with those steps and the same version. I didn't check the add-ons, does it happen without them?

By bad. Somehow I have enabled the 'TAB moves focus' option by mistake.. :/


Extensibility: Context menu has databaseName in profile, but it's not in the profile.options

SQL Operations Studio Version:

Steps to Reproduce:

Create an extension with context menu contributions
Right-click and run some action

Expected:

both profile.databaseName and profile.options.database match
Actual:
profile.options.database is empty

The main issue is when merging the same code in a context menu command and a dashboard task. You use options since it's meant to be the "universal truth" but it doesn't have the value.
Root cause:

I missed that you need to use a ConnectionProfile and update the database value there, rather than cloning as an .toIConnectionProfile and editing that instead. The ConnectionProfile class does write-through to the options but .toIConnectionProfile does not since it's a flat object type.

Closing out older issues.


table scrolls when you're interacting with header on Chrome and IE

Tested on: Chrome 40 / IE 11

Open demo fiddle
Scroll to the bottom of the table
Click and hold your mouse button on any header and move your mouse (i.e. try to highlight header's title)

Weird... I know ;) The problem doesn't exist on Firefox though..

Whoa, that is weird.

Can't reproduce with Chrome 50.  Procrastination FTW!


Ability to add custom expressions, order by is null

Would be great to have the ability to add custom expressions so we can add different things that aren't covered in the library already. For example, I need to run a ORDER BY "fieldName" IS NULL, but there is no such expression available for the fields. Is there another way to do this without running custom SQL?

SQLite 3.30.0 supports NULLS FIRST and NULLS LAST, so this should probably be added instead
https://sqlite.org/lang_select.html


[ENH] Add drop_constant_columns function

PR Description
Please describe the changes proposed in the pull request:

Add drop constant columns function to the functions_list

**This PR resolves #788 **

PR Checklist

Please ensure that you have done the following:

[x ] PR in from a fork off your branch. Do not PR from <your_username>:dev, but rather from <your_username>:<feature-branch_name>.

 Already a PR in process for this.

[x ] Add a line to CHANGELOG.md under the latest version header (i.e. the one that is "on deck") describing the contribution.

Do use some discretion here; if there are multiple PRs that are related, keep them in a single line.

Automatic checks
There will be automatic checks run on the PR. These include:

Building a preview of the docs on Netlify
Automatically linting the code
Making sure the code is documented
Making sure that all tests are passed
Making sure that code coverage doesn't go down.

Relevant Reviewers

Please tag maintainers to review.

@ericmjl
@samukweku

Hopefully this goes through faster :)

Actually, I just realized @samukweku has done a review already. Okie dokes, we'll hit the merge button when the final suite of tests are done!

Actually, I just realized @samukweku has done a review already. Okie dokes, we'll hit the merge button when the final suite of tests are done!

noob question here: Aren't all the tests already run?? I can see all checks(10 successful checks) have passed, is there anything else that runs in the background??

@fireddd nope, nothing else had to run in the bg. As it turns out, GitHub still asked me to trigger the CI just before I posted my comment. I think it's because this was your first official PR to the library. (Your other PRs have triggered the CIs automagically, IIRC.)

We're good here, thanks for the PR, @fireddd! :smile:


Use offsets_for_times for setting consumer offsets to a specific timestamp

Kafka API already provides a method to get offsets for a specific timestamp:
https://kafka-python.readthedocs.io/en/master/apidoc/KafkaConsumer.html
We would like to use this method instead of consuming messages from all partitions, because for big topics, this way should be much faster.

closed by #130


fixing HTML typos

I think you've typed it twice, now it should be ok.

I guess it is ok right now.

@MedUnes thanks!


docker-containers-shim duplication

Hi Team !
I have a problem need help
docker-containers-shim duplication
not showing the container when using ps command docker
do not use docker exec -it container
Server Version: 1.12.0

Output of docker version:
Server Version: 1.12.0

Output of docker info:
Containers: 5
 Running: 5
 Paused: 0
 Stopped: 0
Images: 9
Server Version: 1.12.0
Storage Driver: devicemapper
 Pool Name: docker-thinpool
 Pool Blocksize: 524.3 kB
 Base Device Size: 16.11 GB
 Backing Filesystem: xfs
 Data file:
 Metadata file:
 Data Space Used: 2.948 GB
 Data Space Total: 310 GB
 Data Space Available: 307.1 GB
 Metadata Space Used: 2.179 MB
 Metadata Space Total: 3.263 GB
 Metadata Space Available: 3.261 GB
 Thin Pool Minimum Free Space: 31 GB
 Udev Sync Supported: true
 Deferred Removal Enabled: true
 Deferred Deletion Enabled: false
 Deferred Deleted Device Count: 0
 Library Version: 1.02.107-RHEL7 (2016-06-09)
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: overlay null bridge macvlan host
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.4.16-1.el7.elrepo.x86_64
Operating System: CentOS Linux 7 (Core)
OSType: linux
Architecture: x86_64
CPUs: 24
Total Memory: 31.38 GiB
Name: caas-693.251
ID: 75PQ:KCAJ:KAXA:JNJI:PIRP:JZLT:FRSV:3KWN:GN5F:ZC2N:SGVO:P676
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 46
 Goroutines: 59
 System Time: 2016-08-10T01:38:28.604348442+07:00
 EventsListeners: 0
Registry: https://index.docker.io/v1/
WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled
Labels:
 ip=<IP_ADDRESS>
 type=mix
Insecure Registries:
 <IP_ADDRESS>:5000
 registry-localhost:5000
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
physical
Steps to reproduce the issue:

using docker exec -it container suspend fail to work
not using docker ps & not show list container runing
detect duplicate process docker-containers-shim
kill procces duplicate Back to normal operation , check log not show error

Describe the results you received:
root     30927 33206  0 00:25 ?        00:00:00 docker-containerd-shim 6a69f0fc2490fc66ff2091650c602e7cb45234f7ac4b6902a18b3e6173416954 /var/run/docker/libcontainerd/6a69f0fc2490fc66ff2091650c602e7cb45234f7ac4b6902a18b3e6173416954 docker-runc
root     35017 33206  0 00:36 ?        00:00:00 docker-containerd-shim 6a69f0fc2490fc66ff2091650c602e7cb45234f7ac4b6902a18b3e6173416954 /var/run/docker/libcontainerd/6a69f0fc2490fc66ff2091650c602e7cb45234f7ac4b6902a18b3e6173416954 docker-runc
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):

/cc @mlaventure

@T24-hunter Could you give us the command you used to reproduce the issue? I unfortunately could not derives them from your issue description. Also, if you have the daemon logs it could help.
Thanks!


Uninstallation

The resolution seems to be incorrect for me and I want to uninstall it. How would I do that?

GRUB Customizer will do it:
https://launchpad.net/~danielrichter2007/+archive/ubuntu/grub-customizer
In the customizer, you can change the resolution manually, or other properties about the theme.

http://tipsonubuntu.com/2018/03/11/install-grub-customizer-ubuntu-18-04-lts/


Chart Library Comparison Table

This is the start of a comparison table between multiple charting libraries, as previously discussed, which can be included somewhere on the website in the future.
This list does not have much right now, so please feel free to add anything that you feel is important.

Feature
Chart.js
D3
HighCharts
Chartist

Completely Free
✓
✓

✓

Canvas
✓

SVG

✓
✓
✓

Built-in Charts
✓

✓
✓

8+ Chart Types
✓
✓
✓

Extendable to Custom Charts
✓
✓

Supports Modern Browsers
✓
✓
✓
✓

Extensive Documentation
✓
✓
✓
✓

Open Source
✓
✓
✓
✓

@zachpanz88 I think chartist would be a good thing to have here.

@jakesyl I was thinking about that too. I'll do some research and update the chart.

I think one of the most important thing is types of chart that all they have, so it look like another table.

@Kopleman Updated to include built-in chart types. D3 was excluded because it does not have any built in chart types.

FYI: There's a library called NVD3 which has reusable charts for D3. You could also compare that on the "built in chart types" table.
NVD3

This project is an attempt to build re-usable charts and chart components for d3.js without taking away the power that d3.js gives you. This is a very young collection of components, with the goal of keeping these components very customizable, staying away from your standard cookie cutter solutions.


Bazel build for dialogflow_es fails

The command line is too long:
[966 / 1,320] Compiling google/cloud/dialogflow_es/internal/participants_stub_factory.cc; 7s local, remote-cache ... (32 actions running)
ERROR: T:/[b/ypyiys2y/external/google_cloud_cpp/google/cloud/dialogflow_es/BUILD.bazel:40](https://cs.corp.google.com/piper///depot/google3/b/ypyiys2y/external/google_cloud_cpp/google/cloud/dialogflow_es/BUILD.bazel?l=40):11: Linking external/google_cloud_cpp/google/cloud/dialogflow_es/google_cloud_cpp_dialogflow_es.lib failed: (Exit -1): lib.exe failed: error executing command (from target @google_cloud_cpp//google/cloud/dialogflow_es:google_cloud_cpp_dialogflow_es) C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.33.31629\bin\HostX64\x64\lib.exe /nologo ... (remaining 208 arguments skipped)
Action failed to execute: java.io.IOException: ERROR: src/main/native/windows/process.cc(165): CreateProcessWithExplicitHandles("C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.33.31629\bin\HostX64\x64\lib.exe" /nologo /OUT:bazel-out/x64_windows-fastbuild/bin/external/google_cloud_cpp/google/cloud/dialogflow_es/google_cloud_cpp_dialogflow_es.lib /MACHINE:X64 bazel-out/x64_windows-fastbuild/bin/external/google_cloud_cpp/google/cloud/dialogflow_es/_objs/google_cloud_cpp_dialogflow_es/agents_client.obj bazel-out/x64_windows-fastbuild/bin/external/google_cloud_cpp/google/cloud/dialogflow_(...)): command is longer than CreateProcessW's limit (32767 characters)

https://source.cloud.google.com/results/invocations/5f969e4b-1099-4998-a862-9055239c70fb

https://source.cloud.google.com/results/invocations/c2181f10-b241-448a-817b-fd9f0fb1c88a

We need to wait until the bug is fixed in Bazel.  Marked as external.

This is fixed in https://github.com/bazelbuild/bazel/issues/17068 which got cherry-picked into the 6.1.x series: https://github.com/bazelbuild/bazel/commit/5a7eca2d4cc698eab921929c5f561afe8d039686
We have moved to 6.2.x (and higher). I think it is safe to close this.


Remove useless 'lenient-model-deduplication'

Part of https://github.com/azure/autorest.csharp/issues/1818

API changes have been detected in Azure.ResourceManager.Network. You can review API changes here

API changes have been detected in Azure.ResourceManager. You can review API changes here


printable result of preview

css was modified to have a printable version of the preview, which can be used as a handout...
works in google chrome > print

Thank you! Cool feature.


Remove version: GitHub.GitHubDesktop version 3.1.3

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Publisher has changed their URL structure;


PLBD wait for connection info + message explain that user must check …

…if he has permission when creation file fails

also fix #65


CoD Plutonium launcher doesn't launch with Wayland

When Wine is set to use the native Wayland driver with Winetricks, Plutonium launcher doesn't work.
wl_surface#70: error 0: DRM Syncobj surface object already created for surface 70

Explicit sync related? Tries to create it twice?
Software information
Plutonium Launcher for various old Call of Duty games.
System information

GPU: Intel Arc A750
Driver: Mesa 24.3.2 (Intel Xe kernel driver, instead of i915)
Wine: wine-tkg-git 10.0rc3.r0.gf10d2d04-327 ( TkG Staging NTsync )
DXVK: 2.5.2
VKD3D: 2.14
Lutris: 0.5.18.r95.g9f972ea-1
DE/WM: GNOME 47.2 on Wayland
OS: Arch Linux
Kernel: Linux 6.12.6-arch1-1

Environment Variables
DXVK_CONFIG_FILE=/home/nokia8801/dxvk.conf
MANGOHUD_CONFIGFILE=/home/nokia8801/MangoHud.conf
LD_PRELOAD=/usr/lib64/libgamemode.so
SDL_VIDEODRIVER=wayland

DXVK Configuration File
dxgi.enableHDR = False
dxgi.hideIntelGpu = True
d3d11.maxFeatureLevel = 12_1
d3d9.shaderModel = 3
dxvk.deviceFilter "A750"
d3d11.samplerAnisotropy = 16
d3d9.samplerAnisotropy = 16
dxvk.enableMemoryDefrag = True

Additional Info
System Wine is wine-tkg-git compiled with WoW64, Wayland, Staging and NTsync support. Using Lutris to manage everything. In a newly created prefix just for Plutonium. It's just this application that has this error. Everything else works fine, so far.
It launches with WineD3D. But the performance is trash. So it's not a Wine, WoW64 or NTsync issue. Most likely a DXVK issue when combined with Wayland and explicit sync. Just this app too, every other game I tried works perfectly with native Wayland too.
Lutris Log
lutris.log

I don't see how that could possible be a DXVK issue. It probably works with wined3d because that uses OpenGL instead of vulkan. DXVK doesn't know anything about wayland or drm syncobjects.

It still works with renderer=vulkan set with Winetricks when using WineD3D. So again, most likely a DXVK issue. I asked about this in GNOME Shell matrix channel before making this issue and one dev said it's probably a DXVK issue.

The wined3d Vulkan renderer doesn't support d3d9 if that is what the launcher uses so you are still using OpenGL.

Could be. MangoHud says "DAMAVAND". Never seen that before. I first thought it might be a Mutter issue but dev said:

seems like DXVK is trying to create a sync object twice which afaik is not allowed by the corresponding wayland protocol
perhaps report this issue for dxvk developers

Another dev said:

if the nvidia driver is installed on the system, there was a driver bug which could result in that symptom
if that's not the case, it's indeed likely a DXVK issue
or another entity using Vulkan API

But I don't use NVIDIA, it's an Intel Arc GPU with Mesa drivers. This is the only application that has this error. Everything else works fine.

Could be. MangoHud says "DAMAVAND". Never seen that before.

Ok it should be using Vulkan then yes

Could be. MangoHud says "DAMAVAND". Never seen that before.

Ok it should be using Vulkan then yes

I'm confused. So is it using Vulkan or OpenGL when 'renderer=vulkan`? I've never seen DAMAVAND before, don't know what it means.
Is it not a DXVK issue like Mutter dev said?
With OpenGL, the launcher and the game works, but the performance is trash.
With Vulkan, the launcher works, but the game doesn't launch. I get this error:
Video card or driver doesn't support enough textures for the DirectX 9 code path.

Which is hopefully unrelated. But at least the launcher works. It just doesn't work at all with DXVK.
wl_surface#72: error 0: DRM Syncobj surface object already created for surface 72

seems like DXVK is trying to create a sync object twice which afaik is not allowed by the corresponding wayland protocol

DXVK at least doesn't create any sync objects or use any wayland protocol by itself at all. The only thing DXVK uses is the vulkan and win32 api and win32 vulkan wsi. The sync objects are created either by the driver or wine.

Here is the log from Lutris with WAYLAND_DEBUG=1 variable.
wayland.log
I also found this issue looking around. But it's about NVIDIA, again.
https://gitlab.gnome.org/GNOME/mutter/-/issues/3664
Either this is a DXVK issue or I'm the first person to report this for Intel using Mesa?

I'm confused. So is it using Vulkan or OpenGL when renderer=vulkan? I've never seen DAMAVAND before, don't know what it means.

damavand is the name of wined3d's Vulkan renderer.
In regards to the issue in question i can't personally comment on as i don't have the knowledge.

Like the others have already said: while it's built for Linux, DXVK is essentially a Windows DLL. It doesn't know X11 or Wayland, it puts the image on screen using the Win32 Vulkan APIs. It's Wines job to make that work with X11 or Wayland.

I have no idea why. But I just created a new prefix and instead of Lutris, I just ran the exe from the terminal with all those environment variables prefixed. I also installed dxvk-mingw and ran setup_dxvk install into that new prefix. And everything works. Very weird.
But doing the same from Lutris, I get that sync error.


test(recurring-job/fstrim): test RWO and RWX volume

Which issue(s) this PR fixes:

Issue:

longhorn/longhorn#7768
longhorn/longhorn#7771

What this PR does / why we need it:
Extend test_recurring_job_filesystem_trim with pytest.mark.parametrize fixture to test RWO and RWX volumes.
Special notes for your reviewer:
None
Additional documentation or context
None

Test result: https://ci.longhorn.io/job/private/job/longhorn-tests-regression/6077/

There are many test cases modified, but only test case test_recurring_job_filesystem_trim is run. Maybe another test is required to make sure affected test cases still work.

There are many test cases modified, but only test case test_recurring_job_filesystem_trim is run. Maybe another test is required to make sure affected test cases still work.

The refactor involves assigning names to the optional argument when they are called. There isn't any behavioral change.

Could you share the reason you think this change would make the related cases not work as expected?

Like this one https://github.com/longhorn/longhorn-tests/pull/1689#discussion_r1465338260?

Like this one #1689 (comment)?

It has been addressed. Any other concerns?

@mergify backport v1.6.x v1.5.x


[TASK]: Change readme badges

Task Summary
Add good badges to the readme, like the license type, number of forks, these kinds of things.
I guess this could be a good issue for beginners to contribute to the project.

@bruno-queiroz Can I pick this, WIll create a pull request for same

@romiljain5 seems like you just changed icons not the badges. I'll take up the badge work


ci: make avro schema check optional

closes #405

This change is part of the following stack:

#417 ◀

Change managed by git-spice.


upload fails

error msg from midas:
The connection to the server was interrupted, press the Resume Upload button to resume.
apache error:
PHP Fatal error:  Class 'Api_AppController' not found in /var/www/midas/modules/api/controllers/JsonController.php on line 23, referer: http://xxxx/midas/admin
any ideas? thanks!

Enable the API module.

Thanks! works now. Can you please put this into the installation documentation or make the error message in the upload window more clear?


[Issue] azd provision is provisioning two connections that point to same AOAI resource causing issues

[ ] Make sure you've installed the latest version using instructions in the wiki

Output from azd version
Run azd version and copy and paste the output here:
daily - this is the aistudio integration work that was recently merged.
Describe the bug
azd provision is provisioning two connections that point to same AOAI resource causing issues.
it should NOT be provisioning the Default_AzureOpenAI connection at all.
To Reproduce
azd provision or azd up the starter template for aistudio
Expected behavior
A single aoai-connection should be created as part of provisioning.
Environment
Information on your environment:
* Language name and version
* IDE and version : [e.g. Visual Studio 16.3]
Additional context
This is a //build requirement.

Resolved in AI Studio starter template


Do not lowercase asset file extensions

Objective
Resolves #17064
Solution

Bevy no longer converts asset file extensions to lowercase before trying to resolve an asset loader

Testing

I adapted the custom_asset example (see comment in #17064)
The changes were tested on Linux

As far as I know, Windows has a case-insensitive file system by default, so case-sensitive asset file extensions are probably bad practice in a game. But we should be case-sensitive everywhere or handle asset paths completely case-insensitive.
Before this PR:

asset loader extensions are case-sensitive
asset file names are case-sensitive
asset file extensions are converted to lowercase :zap:

Now everything should be case-sensitive

@alice-i-cecile I don't think the WASM atomics failure is due to my changes

@alice-i-cecile I don't think the WASM atomics failure is due to my changes

Just update your branch. It was already fixed. 🙂


Menu OnTouch is not available and also touch is not working on android device

Bug type
Component
Component name
MudMenu
What happened?
<MudMenuItem<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>             OnTouch="@(() => SetThemeAsync("DARK"))"
             OnClick="@(() => SetThemeAsync("DARK"))"
             Disabled="@hasDarkTheme">Dark Theme UI</MudMenuItem>

Above code is from version 6.21 which is working fine in all devices
Now in version 7.2, there is no OnTouch parameter. And also touch feature is not working on android device. Not yet tested on iPhone
Expected behavior
If there is no OnTouch, OnClick should do the same functionality in all devices.
Reproduction link
N/A
Reproduction steps
Already described
Relevant log output
No response
Version (bug)
7.2
Version (working)
6.21
What browsers are you seeing the problem on?
Other
On which operating systems are you experiencing the issue?
Android
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

also touch feature is not working on android device.

Should work unless you have MudSwipeArea on a page that doing some nasty things via JS https://github.com/MudBlazor/MudBlazor/issues/9288
Workaround: https://github.com/MudBlazor/MudBlazor/issues/9288#issuecomment-2199377053
@danielchalmers

OnClick is now used instead of OnTouch and above comment is another consideration

OnTouch and OnAction have been removed. OnClick handles all use cases for both mouse and touch.  #8492
From: v7.0.0 Migration Guide #8447

Isn't working for me too.
I dont have MudSwipeArea

Please, provide in that case a minimal reproduction project via zip or repository.

I just figured out something.
When a MudSnack is showing.
The Menu Item dont get the click event. Doesnt matter if is android device or not.
https://try.mudblazor.com/snippet/cYmSkLvXrIhMQuCz

When a MudSnack is showing.

Yeah, because MudSnackbarElement is using:
@ontouchstart="InteractionStartHandler"
@ontouchend="InteractionEndHandler"
@ontouchcancel="InteractionEndHandler"

and we've already figured out here for the MudSwipeArea https://github.com/MudBlazor/MudBlazor/pull/9445 that it's messing up the MudMenuItem for some reason.

Right. But in some pages in my App (not all pages) the menu dont work.
I tried to reproduce in a new page, but worked well.
The only difference that i suspect is problem is that i render conditionally items in MudMenu.
<MudMenu FullWidth Label="Opções" AnchorOrigin=Origin.BottomLeft Variant="Variant.Filled" Color="Color.Secondary">
    @if (Alterar.HasDelegate)
    {
        <MudMenuItem<EMAIL_ADDRESS>OnClick="AlterarAsync">
            @* <MudTooltip Arrow Text="Alt + A"> *@
            <MudText Style="margin-left:-25px">Alterar</MudText>
            @* </MudTooltip> *@
        </MudMenuItem>
    }
    @if (Excluir.HasDelegate)
    {
        <MudMenuItem<EMAIL_ADDRESS>IconColor="Color.Error" OnClick="OnExcluir">
            <MudTooltip Arrow Text="Alt + X">
                <MudText Style="margin-left:-25px">
                    Excluir
                </MudText>
            </MudTooltip>
        </MudMenuItem>
    }
    @if (Clonar.HasDelegate)
    {
        <MudMenuItem<EMAIL_ADDRESS>IconColor="Color.Secondary" OnClick="ClonarAsync">
            <MudTooltip Arrow Text="Alt + C">
                <MudText Style="margin-left:-25px">
                    Clonar
                </MudText>
            </MudTooltip>
        </MudMenuItem>
    }
    @if (ItensExtra is not null)
    {
        @ItensExtra  @* <-------------- *@
    }
</MudMenu>

The popover is preventing the touch event default action before the event reaches the menu item. Looking into it further.


Update readme to reflect the current implementaion

The readme seems to be outdated, for example, there is not inspect subcommand anymore.

Sorry, I'm confused by the java parquet-tools.


Add Dark Mode

Can we please add dark mode to super putty?

Yes please! Only downside is if you have custom icons that are also dark - you'll have to adjust them.


Bug: waveOutMessage does not appear to work.

Which crate is this about?
windows
Crate version
0.35.0
Summary
I am trying to query my default output device for all of it's supported configurations.
(Channels, Samplerates, Bitrate, and all of the combinations.)
winapi-rs does not offer a good solution to this, so I hoped that windows-rs would instead.
Currently I am attempting to use the only method I know of, which is explained on this stack overflow.
https://stackoverflow.com/questions/50396224/how-to-get-audio-formats-supported-by-physical-device-winapi-windows
Unfortunately it appears that waveOutMessage is not working as intended or its use is very unclear.
I've tried following this guide.
https://docs.microsoft.com/en-us/windows/win32/api/mmeapi/nf-mmeapi-waveoutmessage
Toolchain version/configuration
Default host: x86_64-pc-windows-msvc
rustup home:  C:\Users\Zageron.rustup
installed toolchains
stable-x86_64-pc-windows-msvc
nightly-x86_64-pc-windows-msvc (default)
installed targets for active toolchain
wasm32-unknown-unknown
x86_64-pc-windows-msvc
active toolchain
nightly-x86_64-pc-windows-msvc (default)
rustc 1.62.0-nightly (60e50fc1c 2022-04-04)
Reproducible example
use windows::Win32::Media::Audio::{waveOutGetNumDevs, waveOutMessage, HWAVEOUT, WAVE_MAPPER};
use windows::Win32::Media::Multimedia::DRVM_MAPPER_PREFERRED_GET;
use windows::Win32::Media::{MMSYSERR_BADDEVICEID, MMSYSERR_INVALPARAM};

let device_id: usize = usize::default();
let device_status: usize = usize::default();
let err = waveOutMessage(
	HWAVEOUT(WAVE_MAPPER.try_into().unwrap()),
	DRVM_MAPPER_PREFERRED_GET,
	device_id,
	device_status,
);

match err {
	MMSYSERR_BADDEVICEID => println!("bad device id"),
	MMSYSERR_INVALPARAM => println!("invalid param"),
	_ => println!("{:?}", err),
}

Crate manifest
windows = { version="0.35.0", features = [ "Win32_Media_KernelStreaming", "Win32_Media_Audio", "Win32_Media_Multimedia" ] }

Expected behavior
Get default device id
Actual behavior
Get MMSYSERR_INVALPARAM
Additional comments
No response

I tried this API from C++ but couldn't get it to work either. I don't think the problem is with the Rust bindings - I think we just don't know how to use it and its certainly not obvious. 😄
Perhaps @jonwis can suggest a simpler API or can point us to a working example. Alternatively, I also see that there is a COM API that looks like it might be easier to use: https://docs.microsoft.com/en-us/windows/win32/coreaudio/mmdevice-api

Further information:
https://docs.microsoft.com/en-us/windows/win32/coreaudio/device-formats

I tried this API from C++ but couldn't get it to work either. I don't think the problem is with the Rust bindings - I think we just don't know how to use it and its certainly not obvious. 😄
Perhaps @jonwis can suggest a simpler API or can point us to a working example. Alternatively, I also see that there is a COM API that looks like it might be easier to use: https://docs.microsoft.com/en-us/windows/win32/coreaudio/mmdevice-api

Thanks for the quick reply and test.
Unfortunately the standard model cannot easily enumerate device capabilities and instead the user must build capability definitions (2ch, 44100, 16b) and pass it to the device and ask if it is supported.
As it is, the examples that I have found so far on windows only ever report the default (user # set) capability configuration, and don't return any others. :(

Paging @aklemets

This appears to be a modern functional example. (Probably?)
https://github.com/audacity/audacity/blob/7da6a532136d862bc5cdc3ee17b236d9dfe601e5/lib-src/portmixer/src/px_win_endpoint.c#L189

I also found this DirectSound example which uses the same "just guess a whole bunch of formats until we get a list".
https://github.com/audacity/audacity/blob/2703c64c072f922a2948aca68df6bda404c8013e/libraries/lib-audio-devices/AudioIOBase.cpp#L368

Why are you trying to use waveOutMessage and DirectSound? These are super old audio APIs from the Windows XP era that don't support any newer audio functionality such as stream categories, offloading, etc.
I would say use it at your own risk...
The replacement for waveOutMessage and related APIs is WASAPI (IAudioClient, etc.)
The replacement for DirectSound is XAudio 2.9.
Please make sure to use the newer APIs unless you still need to support Windows XP or Windows Vista.

Why are you trying to use waveOutMessage and DirectSound? These are super old audio APIs from the Windows XP era that don't support any newer audio functionality such as stream categories, offloading, etc.
I would say use it at your own risk...
The replacement for waveOutMessage and related APIs is WASAPI (IAudioClient, etc.)
The replacement for DirectSound is XAudio 2.9.
Please make sure to use the newer APIs unless you still need to support Windows XP or Windows Vista.

Thanks for all of that information. The APIs are very unclear as to what is modern and what is old so it's difficult to understand what should be used.
I assume there is no modern equivalent to getting all capabilities of a device without just asking over and over.

I'm going to close this issue as this is not a windows-rs issue, but feel free to keep the conversation going.


[Feature Request] Write and WriteBytes should have option to create directory

File.Move already has the option to create target directories. It would be useful to have the same option for File.Write and File.WriteBytes.

Feature created. Pull request can be found here: https://github.com/FrendsPlatform/Frends.File/pull/27


Task5

Issue: #6
used ls and ls -a commands

Hey @official-ashutosh
Thanks for opening this PR 🚀. Mentor will review your pull request soon and till then, keep contributing and stay calm.
Thanks for contributing in OpenCode'24 ✨✨!

Hey @official-ashutosh
Your PR has been merged 🥳🥳 and you have earned 10 points.
Thanks for contributing in OpenCode'24 ✨✨


Formatting with F33

@votca-bot format

@votca-bot changelog: format with clang-11

@junghans you want to upgrade in all repos?

@junghans you want to upgrade in all repos?

Only csg had some changes.
However the format action had a bug in it, too, will need to fix that.


Upgrade/angular10

This PR updates Angular to version 10, and various libraries to be compatible with Angular 10. This closes #31 .
The upgrade has been done following https://update.angular.io/
At a time, only one version has been updated (as recommended by Angular), and they have been separated by commits.
The changes have been tested with Fortinet VPN, as well as the service worker has been tested with https using ngrok.
Please review and merge @yashsriv @dryairship

Please review this so that we can merge it soon @yashsriv


uploading external image

Chloe in RB reports that when she tried to upload an image from outside DPUL, she got a page telling her she was not authorized to access the page.  (She is able to use images from within the system)

Need more context here. Is this still valid?

I believe so, yes.  I don't get the unauthorized message, but images that I tried to upload as recently as last week didn't appear in the system.

@joycebcat Is this different from #271 ?

@escowles No, I believe this is the same problem was #271.  Sorry for the duplication.

@joycebcat No worries, we do it all the time!


Needs testing on Windows

Hi @steffengy, may I please ask you to run go test on windows_{amd64,386}? Thank you in advance.

Updates cznic/sqlite#12.

Running go test in cznic/memory just PASSes.
Anything else you'd like me to try or any output you need?

Running go test in cznic/memory just PASSes.

Good! Thanks for testing.

Anything else you'd like me to try or any output you need?

Thanks, no, that's all I wanted to know before switching CRT to use this package.

Tests no more pass on Windows, reopening. See: https://github.com/cznic/sqlite/issues/12#issuecomment-308130854

Fixed. Confirmed here: https://github.com/cznic/sqlite/issues/12#issuecomment-308143282.


 Look no further, because this is here to take your anime music experience to the next level! Crafted with love using HTML, CSS, and JavaScript, this is the ultimate anime music app that brings the best of anime openings right to your fingertips.
Requirements 🛠

HTML, CSS, and JavaScript: Utilize these core technologies for building the user interface, ensuring responsive design and smooth user interactions.

Responsive Design: The app should adapt to various screen sizes, including desktops, tablets, and mobile devices.

Example:
Song Playlist
Search Functionality
Song Information
User-Friendly UI

Bonuses ✨
Dynamic Playlist Generation
Offline Song Caching
Audio Analysis and Recommendations:

Any number of people can work on a single issue 👨💻👨💻
This issue is open to all.🌍

Project location 👉🏻 Anime-Music-Player/piyush-mahajan/

👨🏻💻 Happy Coding 👩🏻💻

Good Issue.


Support For React Native Windows

Is your feature request related to a problem? Please describe.
Trying to run react-native within as a Widget causes a crash with error
ReactInUWP.exe' has exited with code -1073741819 (0xc0000005) 'Access violation'.
looking at the packager it seems there was no request to the packager.
Describe the solution you'd like
I think (if it's possible) a sample would be more than enough.
Additional context
code -> https://gist.github.com/ShoeBoom/1e2ec26df57c576c5f20e5b0f29800bb

React Native for Windows is a very active project which changes rapidly. We've experimented with it in the past and found it to have some threading issues that prevent it from working correctly in Game Bar. The project has also made an architectural decision to move to WinUI3 which Game Bar cannot currently host. We are working with the WinUI team to add the necessary support to enable its use, but there's currently no ETA.
We have a long term goal of supporting React Native in the future, but as of now we can't recommend it for widget development.


give usefull errors in case of rocksdb init aborting with 'IOError' w…

…hich happens i.e. on cifs mounts
This addresses https://github.com/arangodb/arangodb/issues/2749 to be better understandeable for users whats wrong with their setup.

Now looks like that:
2017-08-31T16:17:36Z [6197] FATAL {startup} unable to initialize RocksDB engine: IO error: directory: Invalid argument; Maybe your filesystem doesn't provide requried features? (Cifs? NFS?)


WIP: First steps for enabling the add API in pyWATTS

Description
closes #28
Related Issues
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos)
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings
[ ] Did you update the CHANGELOG

PR review
Anyone in the community is free to review the PR once the tests have passed.

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones to the PR so it can be classified

TODOs:

[ ] The control flow is not correct

The pipeline.add currently does everything, however it should only collect previous calls and pass them to set_params

[ ] The tests are failing
[ ] Tests for new APIs are missing


Solved issue #136: Fixed embedded link in logo bug [GSSOC'23]

Description
This PR adresses the issue #136.
This PR fixes #
This PR fixes the bug in the events page where the embedded link in the logo was incorrect.
Changes made:
Changes made in .\events\events.html
Testing:
The link is now working by pressing the logo on the events page.
PFA the image which displays the link on the bottom left when hovered over the logo

@akshathar-navada Please let me know if there is any discrepancy. I have tried making the PR according to the code of conduct. Thankyou.

Thank you @rayyan016 for describing the PR in a systematic way. We will soon merge it.
Thanks for contributing to our repo 🤩


New option to output in ModelCIF format instead of PDB format

This should solve #284 . The one missing feature is headers (REMARK and PARENT). Which at this point I'm not sure how they would translate in ModelCIF.
As commented in #284, this also fixes a mistake in the docs for protein.from_pdb_string()

Thanks for the PR!


Change default catalog for dev to dcp2

Replaced by HumanCellAtlas/data-portal#827.

Replaced by HumanCellAtlas/data-portal#827.


[RTM]: use window.history.back instead of url previous

The pr replaces the url()->previous() with window.history.back() from javascript. With that, the back button really works like that it just move only backward.

[x] Move back button on the show-group view as on the user profile detail

Please @vaceksimon take a look on the view show for the user profile detail. I have changed a size of the back button a little bit. At the same time I allowed to show this button for the current profile owner as well. Please check it after me, if I do it right based on that it is your part of code. Thanks in advance.


Can't use Spring security with @FeignClient

When I use Spring security and @FeignClient together:
@FeignClient("base-server")
public interface BaseServerClient {
    @RequestMapping(value = "/find", method = RequestMethod.GET)
    School find(@RequestParam("Id") long Id);
}

And
@Autowired
private BaseServerClient baseServerClient;
...
baseServerClient.find();

It throw
java.lang.NullPointerException: null
	at feign.ribbon.LoadBalancingTarget.apply(LoadBalancingTarget.java:95) ~[feign-ribbon-8.7.1.jar:8.7.1]
	at feign.SynchronousMethodHandler.targetRequest(SynchronousMethodHandler.java:141) ~[feign-core-8.7.1.jar:8.7.1]
	at feign.SynchronousMethodHandler.executeAndDecode(SynchronousMethodHandler.java:83) ~[feign-core-8.7.1.jar:8.7.1]
	at feign.SynchronousMethodHandler.invoke(SynchronousMethodHandler.java:71) ~[feign-core-8.7.1.jar:8.7.1]
	at feign.ReflectiveFeign$FeignInvocationHandler.invoke(ReflectiveFeign.java:94) ~[feign-core-8.7.1.jar:8.7.1]
	at com.sun.proxy.$Proxy121.find(Unknown Source) ~[na:na]

And I found ribbonClient is null
	protected <T> T loadBalance(Feign.Builder builder, Class<T> type, String schemeName) {
		builder.logger(new Slf4jLogger(type)); // TODO: how to have choice here?
		if (this.ribbonClient != null) {
			return builder.client(this.ribbonClient).target(type, schemeName);
		}
		else {
			return builder.target(LoadBalancingTarget.create(type, schemeName));
		}
	}

But when I remove Spring security from my project, It is work.
Why? Is it a bug?

I know why! When you config your security should add an @Order to your WebSecurityConfigurerAdapter

I'm not sure I understand that. Can you provide a simple project to reproduce the problem?

I just fell this problem is about spring-cloud-config.
this is a demo to reproduce the problem https://github.com/zhsyourai/demoFeign

spring-cloud-config knows nothing about feign or spring security, so I don't see the connection. Can you add a README with steps to reproduce? Also is it really necessary to have 3 modules in order to reproduce the problem (a single source tree would be easier)? Also please paste a link to your spring-cloud-netflix issue so we can continue the discussion there. Thanks.


Add liquidlegions formulas.

Do @jiayu-google's most recent changes need internal review before MRC s/o?

@RonPinelliMRC, thanks for raising the concern.
Yes, they do. The internal review is now done and ready for MRC s/o.
Thank you!


fix(core): Add "additionalProvidersData" to "safeUserObject"

Needed for management of social accounts.

LGTM. I've tested this, and verify that the social accounts are now showing up correctly.
Thanks @francisdaigle!

@mleanos @francisdaigle Was this a bug? Can someone elaborate slightly more on why this was needed? I don't have an issue with it, just want to vet it before it gets merged. Thanks!!!

@codydaig I hadn't noticed before, but on the latest master the social accounts profile view wasn't showing the connected accounts. This is because the additionalProvidersData field isn't present on the client-side User object.
In short, it is a bug that this is fixing.

@codydaig: Yep. Came across it while working on our fork. Must have been omitted by mistake. Makes it a fix I would think for the reasons @mleanos cited.

Thanks @mleanos and @francisdaigle for the clarification.


[CSharp] How to mention to channel

I'd like to ask about how to mention to channel or team in Microsoft Teams from Bot.
The following document says that such a type of mention is not supported.
https://docs.microsoft.com/ja-jp/microsoftteams/platform/scenarios/bots-channel-conversations#mentions
“At this time, team and channel mentions are not supported.”
Is it not supported even now?
And do you have any plan to support this?
I'd like to make it possible to notify all of the members in a teams channel from Bot.

@yuimam - Team and channel mentions are still not supported. Documentation for this will be updated once this feature is enabled (ETA March 2018).


implement error trait for pvrecorder

This allows it to be used in a more general way, along with the ? operator e.g.
fn main() -> Result<(), Box<dyn Error>> { todo!() }

Will be pushing packages sometime later, as we have some additional changes we'd like to include


Swap depth issue with addmod and mulmod

Input code:
fn HEY(x, y) -> (z) {
    z = addmod(x, y, 3)
}

Actual output:
#define macro HEY() = takes(2) returns(1) {
    // takes:                      [y, x]
    0x3                         // [y, x, 0x3]
    swap1                       // [y, 0x3, x]
    addmod                      // [z]
    // returns:                    [z]
}

Expected output:
#define macro HEY() = takes(2) returns(1) {
    // takes:                      [y, x]
    0x3                         // [y, x, 0x3]
-   swap1                       // [y, 0x3, x]
+   swap2                       // [y, x, 0x3]
    addmod                      // [z]
    // returns:                    [z]
}

Thanks for reporting, have confirmed and will fix later this week.

@Philogy Crit in balls :sob:

Fixed in df45004, thanks for reporting!


SM Explorer (12/6/2018 ver xx38) is crashing on Find Callers

The latest update to the JKI State machine editor is crashing with Find Callers.  It seems to find the callers - the popup box opens, then immediately closes and SME crashes.  And if there is only one caller, that frame is activated before the crash.  This is happening on two installs, both W10/LV2018 Pro.

Hi @arejaw. Thanks for reporting this. I see what you're saying. A little bit of work done to improve the Find Data Accessors dialog also need to be also done in the Find Callers dialog. I'll fix that and release a new version shortly...

Fixed in JKI State Machine Release 2<IP_ADDRESS>. Available for download and install with VIPM.

Thanks, Jim - it's working great now!


add host network metrics

I need some host network metrics so that I can keep an eye on host level network bottlenecks.
I probably would like to see throughput for each nic and not just an aggregate host throughput which seems useless.
I was trying to figure out what the net.throughput.contention.summation metric actually reports as this might be an easier alternative.

This issue was moved to pryorda/vmware_exporter#12


Launch quick start app fails on Windows 10

I am using Windows 10 bash. Install node and npm through apt-get not standard installation. May be that is why?
root@SERGEY:/mnt/c/Users/Sergey/Sites/electron-quick-start# npm start

><EMAIL_ADDRESS>start /mnt/c/Users/Sergey/Sites/electron-quick-start
> electron .

/mnt/c/Users/Sergey/Sites/electron-quick-start/node_modules/electron/dist/electron: error while loading shared libraries: libgtk-x11-2.0.so.0: cannot open shared object file: No such file or directory

npm ERR! Linux 3.4.0+
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "start"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.10
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>start: `electron .`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'electron .'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the electron-quick-start package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     electron .
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs electron-quick-start
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls electron-quick-start
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /mnt/c/Users/Sergey/Sites/electron-quick-start/npm-debug.log

The error says you are missing libgtk-x11-2.0. Install it.

@Serhioromano @hovancik The issue here is you are trying to launch Electron (a GUI application) through the Linux subsystem for windows.  (Bash on Ubuntu on Windows).
This will not work as Ubuntu is simply running in a headless VM and does not have a "screen" to launch ubuntu onto.  You need to use command prompt to actually launch / develop with Electron.


Stripe module has no attribute create_form

What I'm trying to achieve
I was trying to use Stripe locally, so I've added the env vars and added stripe to the CHECKOUT_PAYMENT_GATEWAYS, but in the check when I choose the payment method and I proceed I got an attribute error module 'saleor.payment.gateways.stripe' has no attribute 'create_form'.
*I've checked in the stripe module but I couldn't find the create_form while it exists for other gateways
Steps to reproduce the problem

Setup Saleor locally using docker
Add Stripe env vars and Add Stripe to the CHECKOUT_PAYMENT_GATEWAYS
Go to the storefront, add an item to the cart and checkout
Choose Stripe as a payment method
Click proceed
The error occurs

What I expected to happen
See a Stripe form, enter my payment info
Screenshots

Hey @taoufik07,
Unfortunately new stripe implementation is not compatible with old saleor storefront and is accessible only through API due to Stripe API significant change (using PaymentIntents). We're not keeping backward compatibility for new code since we prefer to put resources into new SPA storefront.
You could easily roll back stripe gateway to old implementation but please consider it will stop working pretty soon in European Union due to SCA regulation.
You could also write create_form for new stripe implementation on your own, that could additionally be very appreciated contribution 😉

Hey @taoufik07,
Unfortunately new stripe implementation is not compatible with old saleor storefront and is accessible only through API due to Stripe API significant change (using PaymentIntents). We're not keeping backward compatibility for new code since we prefer to put resources into new SPA storefront.
You could easily roll back stripe gateway to old implementation but please consider it will stop working pretty soon in European Union due to SCA regulation.
You could also write create_form for new stripe implementation on your own, that could additionally be very appreciated contribution 😉

Hi@salwator ,
I've meet the same problem"module 'saleor.payment.gateways.stripe' has no attribute 'create_form'"
I just want to achive stripe,but have tried so much time.How to write create_form?
thanks!!!!!!!!

Hi @syusama,
There is an example of create_form in braintree gateway implementation in form of BraintreePaymentForm. Once you can create_form method that returns usable form, your stripe integration with old saleor storefront should work fine.
You can use great stripe docs to learn how to use PaymentIntents on the web.


Fetch the locations associated to a client

As an API client I want to fetch information about CLIENT_LOCATION given a FOREST_CLIENT's client number or its acronym So That part of this information can be displayed for the final user.
Additional Context

Client and location information will be mostly displayed together in our application.
Either creating a specific endpoint for this (e.g. /clientLocations?clientNumber=&acronym=, /client/{clientNumber}/locations) or just returning the location information in the existing client endpoint (/clients) would work for us.

Acceptance Criteria

[ ] Given (Context), When (action carried out), Then (expected outcome)
[ ] Given (Context), When (action carried out), Then (expected outcome)

Definition of Done

[ ] Ready to Demo in Sprint Review
[ ] Does what I have made have appropriate test coverage?
[ ] Documentation and/or scientific documentation exists and can be found
[ ] Peer Reviewed by 2 people on the team
[ ] Manual testing of all PRs in Dev and Prod
[ ] Merged

Sorry for opening three issues with the same content. I don't know how it happened.


[WFLY-13695] Upgrade wildfly-http-client to 1.0.23.Final

I created this PR in case it makes sense to have it upgraded in 20.x, it is a bug fix upgrade.
Jira: https://issues.redhat.com/browse/WFLY-13695
Master PR: https://github.com/wildfly/wildfly/pull/13422
    Release Notes - WildFly EJB HTTP Client - Version 1.0.23.Final

        Bug

[WEJBHTTP-47] -         Ensure the HttpRemoteTransactionPeer captures current AuthenticationContext and SSLContext

    Release Notes - WildFly EJB HTTP Client - Version 1.0.22.Final

        Bug

[WEJBHTTP-45] -         UT000103 thrown when WildflyClientOutputStream size is exactly 1024 bytes

I am going to close this one for now, the WildFly 20.0.1.Final release has already been created - generally there is one release after the X.0.0.Final release.  If that changes for 20.x we can reopen.


/teams/ page does not look nice on mobile

Picture 3: Overflow of GitHub username outside the box outline for Harshit

@zulipbot claim


Scrollbars making everything look unclean

The website is shown with vertical and horizontal scrollbars:

I don't know if this issue has to be fixed within the applet configuration or with my pc / scaling config. Is there any way to hide them?

I'm getting the same issue. Would also be nice if we could resize the window.


PHASE 05.1 - Link to User Stories

My first pass in Wiki, 12/18/23
ChatGPT prompts 12/15/2023

Here's the link, also in the README.md file. https://github.com/mcfuzznstuff/personal-art-gallery-v23.1.0/wiki/User-Stories


Correction for raw affiliation CIRAD, AGAP institute, Avenue Agropolis, BP 5035, 34398 Montpellier, France; University of Montpellier, Avignon University, CIRAD, Institut Agro, Reunion University, Montpellier, France; University of Montpellier, CIRAD-INRAE-Institut Agro, University of Montpellier, Montpellier, France

Correction needed for raw affiliation CIRAD, AGAP institute, Avenue Agropolis, BP 5035, 34398 Montpellier, France; University of Montpellier, Avignon University, CIRAD, Institut Agro, Reunion University, Montpellier, France; University of Montpellier, CIRAD-INRAE-Institut Agro, University of Montpellier, Montpellier, France
raw_affiliation_name: CIRAD, AGAP institute, Avenue Agropolis, BP 5035, 34398 Montpellier, France; University of Montpellier, Avignon University, CIRAD, Institut Agro, Reunion University, Montpellier, France; University of Montpellier, CIRAD-INRAE-Institut Agro, University of Montpellier, Montpellier, France
new_rors: 05kpkpg04;051escj72;03rnk6m14;003vg9w96;00txxwv34;02w4exq36
previous_rors: 05kpkpg04;051escj72;03rnk6m14;003vg9w96;00txxwv34
works_examples: W4306711467
contact: 7eecc850a636e3c6cd13fb228c6c8ff3:612b6e5a7506e9f7412c205395ba1e12634d @ inrae.fr

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


fixed transaction id and envelope id for xml export.

fixed transaction id and envelope id for xml export.

 [DIT] Edit measures XML issue investigation (6)