[WIP] Compact Filter protocol messages

Hello, this is a work in progress for light client protocol messages. Still a WIP because of lack of proper tests. I would like to give credit to @lontivero for opening #401, getting the ball rolling on the issue, and giving me a solid place to start.
There has been a few changes regarding filter types, for now the only supported filter is "Basic". Therefore I have commented out the conditional checks for "Extended" filter type, also commented out the enum value.
For the payloads I have included all the fields included in bip157. I included the "length" fields which are technically not necessary to save in the object, as far as I am concerned.
I have overrode ToString() for testing purposes.
I appreciate all comments and feed back given.
The end goal is to eventually create a compact filter light client, similar to neutrino on top of Nbitcoin.

Btw, are there already any serving full nodes on the mainnet?

The only node on mainnet that I am aware of is the BTCD node that I myself am running.

@MsBarber Is it merged to btcd already? Last time I checked it wasn't.
In fact I'm looking at Roasbeef's btcd fork: https://github.com/Roasbeef/btcd is "This branch is 137 commits ahead btcsuite:master." Of course this may be something else.

@nopara73 As far as I am concerned development has moved over to https://github.com/btcsuite/btcd
Also this implementation of the reference client does support compact filters, https://github.com/btcsuite/btcd/blob/2a560b2036bee5e3679ec2133eb6520b2f195213/config.go#L156 you can see the option to disable it here.
I have a node running and have been sending cf related messages and getting appropriate responses.
If you would like to connect to my node and give it a spin I would not mind sharing the IP, its mainnet.
The BTCD node does take a good amount of time to sync if you chose to spin it up.

There is also several warning to fix.

Can you rebase and fix the build?

@NicolasDorier Hey Nick, Appveyor seems to still be failing, however travis is passing.
The error from appveyor makes no sense as the filter type is clearly being found by travis.

We use to say: the SQL is not broken. Take a closer look at the console log:
NBitcoin -> C:\projects\nbitcoin\NBitcoin\bin\Release\net452\NBitcoin.dll
  NBitcoin -> C:\projects\nbitcoin\NBitcoin\bin\Release\net461\NBitcoin.dll
Protocol\Payloads\CompactFilterCheckPointPayload.cs(53,10): error CS0246: The type or namespace name 'FilterType' could not be found (are you missing a using directive or an assembly reference?) [C:\projects\nbitcoin\NBitcoin\NBitcoin.csproj]
Protocol\Payloads\CompactFilterCheckPointPayload.cs(21,41): error CS0246: The type or namespace name 'FilterType' could not be found (are you missing a using directive or an assembly reference?) [C:\projects\nbitcoin\NBitcoin\NBitcoin.csproj]
  NBitcoin -> C:\projects\nbitcoin\NBitcoin\bin\Release\netstandard1.3\NBitcoin.dll
  NBitcoin -> C:\projects\nbitcoin\NBitcoin\bin\Release\netcoreapp2.1\NBitcoin.dll
  NBitcoin -> C:\projects\nbitcoin\NBitcoin\bin\Release\netstandard2.0\NBitcoin.dll

It compiles okay for all the targets except for net461. That's why it pass green in travis. Next, remove all the targets except net461 and build again, it has to fail.

Just to make sure I cloned the repo following the instructions shown in the console log:
git clone -q --depth=1 https://github.com/MetacoSA/NBitcoin.git c:\projects\nbitcoin
git fetch -q origin +refs/pull/548/merge:
git checkout -qf FETCH_HEAD

Then built the project with:
dotnet build

I expected to fail because I work on Linux and don't have .NET Framework. It failed and gave me the same error:
/usr/share/dotnet/sdk/2.1.500/Microsoft.Common.CurrentVersion.targets(1179,5): error MSB3644: The reference assemblies for framework ".NETFramework,Version=v4.6.1" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend. [/home/lontivero/GitHub/nb/NBitcoin.Tests/NBitcoin.Tests.csproj]
/usr/share/dotnet/sdk/2.1.500/Microsoft.Common.CurrentVersion.targets(1179,5): error MSB3644: The reference assemblies for framework ".NETFramework,Version=v4.6.1" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend. [/home/lontivero/GitHub/nb/NBitcoin.Altcoins/NBitcoin.Altcoins.csproj]
/usr/share/dotnet/sdk/2.1.500/Microsoft.Common.CurrentVersion.targets(1179,5): error MSB3644: The reference assemblies for framework ".NETFramework,Version=v4.6.1" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend. [/home/lontivero/GitHub/nb/NBitcoin/NBitcoin.csproj]
/usr/share/dotnet/sdk/2.1.500/Microsoft.Common.CurrentVersion.targets(1179,5): error MSB3644: The reference assemblies for framework ".NETFramework,Version=v4.5.2" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend. [/home/lontivero/GitHub/nb/NBitcoin.TestFramework/NBitcoin.TestFramework.csproj]
/usr/share/dotnet/sdk/2.1.500/Microsoft.Common.CurrentVersion.targets(1179,5): error MSB3644: The reference assemblies for framework ".NETFramework,Version=v4.5.2" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend. [/home/lontivero/GitHub/nb/NBitcoin/NBitcoin.csproj]
Protocol/Payloads/CompactFilterCheckPointPayload.cs(53,10): error CS0246: The type or namespace name 'FilterType' could not be found (are you missing a using directive or an assembly reference?) [/home/lontivero/GitHub/nb/NBitcoin/NBitcoin.csproj]
Protocol/Payloads/CompactFilterCheckPointPayload.cs(21,41): error CS0246: The type or namespace name 'FilterType' could not be found (are you missing a using directive or an assembly reference?) [/home/lontivero/GitHub/nb/NBitcoin/NBitcoin.csproj]
/usr/share/dotnet/sdk/2.1.500/Microsoft.Common.CurrentVersion.targets(1179,5): error MSB3644: The reference assemblies for framework ".NETFramework,Version=v4.5.2" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend. [/home/lontivero/GitHub/nb/NBitcoin.Altcoins/NBitcoin.Altcoins.csproj]
    0 Warning
    8 Error(s)

Ohhh it is simply that some files has the #if !NOSOCKET and other don't!!! so, yes, the FilterType was not defined.

@lontivero

Ohhh it is simply that some files has the #if !NOSOCKET and others don't!!! so, yes, the FilterType was not defined.

Lol, yes I also noticed this not too long ago and have applied the fix. Thank you so much for the help!

Can you rebase?

TravisCI was unable to located dotnet-sdk-2.1?
E: Unable to locate package dotnet-sdk-2.1 E: Couldn't find any package by glob 'dotnet-sdk-2.1' E: Couldn't find any package by regex 'dotnet-sdk-2.1' The command "sudo apt-get install -qq dotnet-sdk-2.1" failed and exited with 100 during .

Retrying

Can you add some tests? Is there some test vector in the BIP so we can try serializing and deserializing?


Add DNS record config

Bump
On Fri, Dec 17, 2021, 8:24 PM github-actions[bot] @.***>
wrote:

This issue is now marked as stale because it hasn't seen activity for a
while. Add a comment or it will be closed soon.
—
Reply to this email directly, view it on GitHub
https://github.com/RenovoSolutions/cdk-library-renovo-instance-service/issues/8#issuecomment-997117649,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADDGQ6UDYWHOB5RL6VSXXZDURPPDFANCNFSM5GHNFUYQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were assigned.Message ID:
<RenovoSolutions/cdk-library-renovo-instance-service/issues/8/997117649@
github.com>


Better solution to modifying API requests in dev and production?

Right now, whenever I pull I have to manually change the frontend API url to localhost:3000/api/week from /api/week. I'm musing, would it be feasible to run the webpack dev server through the existing backend? There's a way to integrate it, and that way, it wouldn't require any changes--that'd be run in unison.
I may not be explaining this clearly enough but it's just an idea

Maybe this?

Moving to Firebase obviates this, but using webpack-dev-server's proxy feature would solve it anyways.


Improve handling of defaultStatementTimeout #503

For details, refer to #503 .
Please review.

mistake..


use new versions for debian and graalvm java8

Your images are truly useful, thank you for the hard work. Here's a PR that updates the debian version to the latest one and uses the graalvm 20.0.0 for java8

I haven't updated the oraclelinux one as I'm not very knowledgeable about that

Thanks for the contribution. And many thanks for the kind words, highly appreciated!


Should this repo be archived using the GitHub interface

Readme states the repo is archived. Do you want to archive the repo in the GitHub UI?

Good point, @jni @melissawm afaik the napari-plugin-template is ready to go right, so we should be able to archive this repo without any impact?

I think we were waiting for #191, but maybe it's not needed...

If we archive it can it no longer be used by cookiecutter?

it can be used, but read-only = no pushes (afaik), so we could no longer fix with your suggestion:

I think the easiest way to address it would be to have the pre_gen_project.py hook raise an error that points to the napari plugin template?

yes I think that's right

I think the easiest way to address it would be to have the pre_gen_project.py hook raise an error that points to the napari plugin template?

Perhaps use the pre_prompt.py hook since it runs before pre_gen_project.py and can bail before the cookie-cutter prompting.

Opened #195 to address!

#195 is merged, I'm going to close this and archive. Thanks for the prod @willingc!


Finding detox is the right tool for our mobile automation

I am working in QA, our dev developed native ios code through swift and Android uses Kotlin.  Is detox supports this? or will detox only support react native app?

Detox doesn't properly support native / brownfield apps on Android, I'm afraid.
iOS might be in better shape in that sense but we can't guarantee that it would stay that way.

Thanks. do you think we can implement detox for ios now?  do you think Wix may drop support for this?

I wish I could have guarantees, but I do not, at least not at the moment.


[Trivial] inconsistency in runtime arguments *

Not sure how much this matters, but the capitalized name was different with the argument name. Given that neither spellings are actually real words I chose the one I thought looked more appealing.

Before merging should check that this isn't set on staging or if it is fix the env var there too.

I can't see anywhere in staging that this is used yet in any of our configuration files. Seems like its fine to change and won't actually require anything for next release.
I am now considering removing all the env = CAPS_STUFF everywhere right here, but this also shouldn't require any action for deployment.

Given that this PR has received a lot of approvals and will probably be forgotten. I will leave the changes as is and merge whats here. Will follow up with another PR for removal of env = CAPS_STUFF as proposed here.


unsafe repositories multiple repo warning text needs tuning

Testing #167413
The warning text confused me (was it talking about the parent folder?) and is only partially pluralized:

Should the warning say "...potentially unsafe as the folders are owned....
Could the warnings be shorten to:
"The git repositories in the following folders are potentially unsafe as the folders are owned by someone other than the current user".
"The git repository in the following folder is potentially unsafe as the folder is owned by someone other than the current user".

@gregvanl, thank you very much for the improved message.

Looks like the warning text and UI has moved on from the proposed suggestion. New messaging looks good

@gregvanl, yes, we have tweaked the messaging a little bit. Thanks for your help!


[DNM] test negative

Please provide a description of this PR:

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
b9d9a888237127ce07249765ec5a03c935df5da7
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
f42f1354da588c51337ef4de63a512ac09e4a42b
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
f42f1354da588c51337ef4de63a512ac09e4a42b
link
true
/test lint

integ-distroless_istio
f42f1354da588c51337ef4de63a512ac09e4a42b
link
true
/test integ-distroless

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
f42f1354da588c51337ef4de63a512ac09e4a42b
link
true
/test integ-distroless

lint_istio
a64d98cf478601fe024d69e31ac7213316844acb
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
a64d98cf478601fe024d69e31ac7213316844acb
link
true
/test lint

integ-cni_istio
a64d98cf478601fe024d69e31ac7213316844acb
link
true
/test integ-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio
a64d98cf478601fe024d69e31ac7213316844acb
link
true
/test integ-cni

integ-telemetry_istio
187268a455a89acbf4f78a00d40f36e771ac40a6
link
true
/test integ-telemetry

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio
a64d98cf478601fe024d69e31ac7213316844acb
link
true
/test integ-cni

integ-telemetry_istio
187268a455a89acbf4f78a00d40f36e771ac40a6
link
true
/test integ-telemetry

integ-telemetry-discovery_istio
187268a455a89acbf4f78a00d40f36e771ac40a6
link
true
/test integ-telemetry-discovery

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio
a64d98cf478601fe024d69e31ac7213316844acb
link
true
/test integ-cni

integ-telemetry_istio
187268a455a89acbf4f78a00d40f36e771ac40a6
link
true
/test integ-telemetry

integ-telemetry-discovery_istio
187268a455a89acbf4f78a00d40f36e771ac40a6
link
true
/test integ-telemetry-discovery

integ-telemetry-istiodremote_istio
187268a455a89acbf4f78a00d40f36e771ac40a6
link
true
/test integ-telemetry-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry_istio
187268a455a89acbf4f78a00d40f36e771ac40a6
link
true
/test integ-telemetry

integ-telemetry-discovery_istio
187268a455a89acbf4f78a00d40f36e771ac40a6
link
true
/test integ-telemetry-discovery

integ-telemetry-istiodremote_istio
187268a455a89acbf4f78a00d40f36e771ac40a6
link
true
/test integ-telemetry-istiodremote

integ-telemetry-mc_istio
187268a455a89acbf4f78a00d40f36e771ac40a6
link
true
/test integ-telemetry-mc

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.


Bug: Change password of a user without auth mechanism set

Describe your changes :
When basic auth is enabled and the user is created through the API without a password, the stored auth mechanism is null. This can cause a null pointer exception when changing the user password for the first time.
Type of change :

[x] Bug fix

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers
Ingestion: @open-metadata/ingestion

Should we disallow user creation without a password instead?

Should we disallow user creation without a password instead?

@TeddyCr when we create a user when the SSO is enabled, the password is not required.


Automatic Release process fails when bumping version

Reqnroll Version
n/a
Which test runner are you using?
MSTest
Test Runner Version Number
n/a
.NET Implementation
.NET 8.0
Test Execution Method
Visual Studio Test Explorer
Content of reqnroll.json configuration file
No response
Issue Description
see https://github.com/reqnroll/Reqnroll/actions/runs/11722083417
error: GH013: Repository rule violations found for refs/heads/main.
Steps to Reproduce
run & deploy a new release
Link to Repro Project
No response

Fixed. The Deploy key was not added to the branch protection rule bypass list.


Datagrid uses wrong trackBy and breaks selection

Describe the bug
We have a dynamic datagrid with a configuration source for the columns. Because of this, we use a ngFor-loop for the clr-dg-column and utilize a trackBy function to improve performance.
We experience the issue that if the grid is initialized with selected items, the first selected item is not selected but the last item of the list. By debugging i found out, that the selection feature is using the Items.trackBy function and that this one is the one from the column-loop instead of the row-loop. This happens as there is ClrDatagridItemsTrackBy<T> which sets the trackBy function of the Items Object to the trackBy of any ngForTrackBy that is found in the grid.
I got things working by removing every trackBy in the whole grid, but this isn't really a good solution for this problem.
How to reproduce
I added a minimal example here: https://stackblitz.com/edit/clarity-light-theme-v2-bzpjmm

You can see in the Component, that the first and second item are selected programatically, but the selection switches to the second and third (last) item.
In the console you can see that the trackBy function of ClrDatagrid.items.trackBy is the column-trackBy
If you remove the trackBy of the column-loop, selection works again

Expected behavior
The selection feature should only use the trackBy function of *clrDgItems or ngForOf of the clr-dg-row component and not any other unconstrained ngForOf within the grid.
Versions
App

Angular: 8
Node: 10.16
Clarity: 2.3.0

I think the issue here is that the selector is generic for ClrDatagridItemsTrackBy: '[ngForTrackBy]' so its picking up the last trackBy to be rendered in the datagrid. We should be able to either make the selector more specific, or figure out how to filter it someway.

Hi, I’m just following up on the issue raised, is there any update on a fix?
Thanks

Hi,
do you have a workaround for this problem?
I am using  in our datagrid and have *ngFor for the buttons in it. After I set a trackBy function on the  *ngFor  for the action bar buttons a weird  selection on a last row appeared in my grid. I investigated and saw that the problem is the same as  the one described in this issue.
Do you have a plan to fix this issue?

@rootix The team looked into the scenario you provided and a little bit further. We came into the conclusion trackBy set on anything else than the clr-dg-row generation loop is just not needed.
We encourage you to use only trackBy on the row generation loop e.g. <clr-dg-row *ngFor="let item in items; trackBy: trackBy"></clr-dg-row> or <clr-dg-row *clrDgItems="let item in items; trackBy: trackBy"></clr-dg-row>.

Hi @bbogdanov
could you provide me with a link to the documentation on how to use the trackBy function? What's the method signature?
I've looked through all 21 datagrid demos, but trackBy seems not to be mentioned anywhere. Maybe it is missing from the docs?


Expose SonarQube API and helpers

Signed-off-by: Magnus Persson<EMAIL_ADDRESS>Hey, I just made a Pull Request!
We are in need of creating some UI elements that consume sonarqube metrics. This PR intends to export the API/client so that we don't have to reimplement it for our instance. I've made a best effort of adding exports and annotations for the api report, but I'm not sure that it should be marked as public. It might even be that we dont intend on exposing the API just yet. But I needed to start somewhere :)
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

@dhenneke @scthi @Neemys any feedback on this? Is there a reason why we shouldn't expose the API?

We find that the provided SonarQubeCard is much too visual and thus had to resort to copying the plugin to our instance. This gave us access to some necessary hooks and types:
...
const { entity } = useEntity();
const sonarQubeApi = useApi(sonarQubeApiRef);

const projectTitle = useProjectKey(entity);
...

And with that we created our own CompactSonarQubeCard:

In the future we will look at Tech Insights and creating facts from sonarqube metrics. Facts are materialized on the backend so being able to import and use SonarQubeClient would be awesome.

@magnusp - I'm curious how you are intending to use these types? Any chance you can provide an abbreviated example?

We find that the provided SonarQubeCard is much too visual and thus had to resort to copying the plugin to our instance. This gave us access to some necessary hooks and types:
...
const { entity } = useEntity();
const sonarQubeApi = useApi(sonarQubeApiRef);

const projectTitle = useProjectKey(entity);
...

And with that we created our own CompactSonarQubeCard:

In the future we will look at Tech Insights and creating facts from sonarqube metrics. Facts are materialized on the backend so being able to import and use SonarQubeClient would be awesome.

Thanks for the context. I can't see a huge harm in exporting these, but I will point out a couple alternative approaches:

Provide your textual view as an alternative view in the open source version.
Consider moving the public types, etc to a @backstage/plugin-sonarqube-react package.

That said, the plugin is pre v1, so any future breaking changes to these types should be expected. Probably best for @backstage/maintainers to chime in.

I see this is still a draft and some conflicts crept in. Gentle ping here, how are you feeling about the PR?

Since Im not seeing anything stoppning the export of these types I will go
over the changes again and keeping it as a draft as of now. Introducing a
-react package seems to make sense so I will explore that route.
Den mån 14 nov. 2022 16:15Fredrik Adelöw @.***> skrev:

I see this is still a draft and some conflicts crept in. Gentle ping here,
how are you feeling about the PR?
—
Reply to this email directly, view it on GitHub
https://github.com/backstage/backstage/pull/14314#issuecomment-1313913250,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK5RBI62DIEQE2CLVRDPALWIJJQXANCNFSM6AAAAAARNYUQUE
.
You are receiving this because you were mentioned.Message ID:
@.***>

So I separated parts of plugin-sonarqube to plugin-sonarqube-react with the expectation that we should now be able to use useProjectInfo and useApi(sonarQubeApiRef) externally. I'm not feeling very confident in what types got moved, nor the annotations for @alpha. I used a package.json from a neighbouring package, and did my best on filling in the dependencies. If everything looks good to the parties involved so far I'm ready to move out of the draft.

I see that @manusant recently did some work on the packages. Got any input on these changes?

@magnusp A gentle ping here!

Sure, looks good. Maybe the changeset could list the types that were moved. A very defensive thing to do is to keep exporting them from the original package too, but marked as deprecated. Like here for example. That way you don't have to do the breakage right away

If I'm understanding this correctly the types were never exported outside of @backstage/plugin-sonarqube and I believe that the API report reaffirms that. That being said we could more specific on what was previous internal to @backstage/plugin-sonarqube and henceforth public via @backstage/plugin-sonarqube and @backstage/plugin-sonarqube-react.


Docs website is not accessible

Problem
The page, which the link in the README.md file refers to as the documatation, is not working:

https://vue-concurrency.netlify.app/

How to reproduce

Visit this link: https://vue-concurrency.netlify.app/
Or try to go to the page via the README.md file

Solution proposal
I don't know - I would be happy if the docs-page would work! :)

Huh, thanks for reporting @WaldemarEnns ! It's working for me but I'm investigating. Any info about your OS / Browser would be appreciated.

@MartinMalinda
Browser and OS:
Google Chrome | 83.0.4103.116 (Official Build) (64-Bit)
OS | macOS Version 10.15.4 (Build 19E287)

Screenshot:

It could be some kind of SSL issue with Netlify I suppose. I reported it further and I hope I'll get some answers. Thanks a lot for the report.
Meanwhile I guess some kind of proxy could help... or cloning this repo and running npm install && npm run docs:dev :)

@MartinMalinda Your'e welcome! Thanks for the great repo! :)
I have cloned the repo and started the docs in dev mode already 👍

Apparently it was a temporary issue an it should be fixed by now:
https://community.netlify.com/t/site-not-accessible-for-certain-users-http-vue-concurrency-netlify-app/18264/3

Thanks for the information!
Currently it is still not available for me, I will check it again in the evening 👍

Huh, on the community forums they say it should be fine, but I'll keep this open just in case someone else experiences this.

Closing. Let's hope it was some rare Netlify glitch that won't happen again.


Use Less Memory

        result_matrix[:, feature_id_intersection_cas_indices] = input_matrix.tocsc()[
            :, feature_id_intersection_adata_indices
        ]

Was causing a memory blow up, with lots of dense matrices.  To fix this, use a COO matrix format and edit the columns.

Looks like Fedor already pulled this into the CAS upstream: https://github.com/cellarium-ai/cellarium-cas/pull/34


Set the exported experiment's status to inactive

This PR fixes the https://github.com/CarnegieLearningWeb/UpGrade/issues/644
It seems to work fine but I'm not sure if this is the proper way to do it.

Closing this due to Steve's comment: https://github.com/CarnegieLearningWeb/UpGrade/issues/644#issuecomment-1337973390


Dynamic fields sometimes don't render when initially switching to a new group

Describe the bug
When creating a multi-group form, dynamic fields don't always populate correctly when switching to a new group until you press a keyboard input, which triggers an update. This causes dynamic descriptions to be completely blank.
To Reproduce
Tested using huh 0.6.0 as well as the current main branch (c9b2c9cdb7b6c76d636bfdf1d5609818a28779ed) with this reproduction script:
package main

import (
        "github.com/charmbracelet/huh"
)

func main() {
        path := "initial path"

        form := huh.NewForm(
                huh.NewGroup(
                        // Note: behavior is different depending on whether this is an Input or FilePicker; try changing this
                        // to huh.NewInput() as well.
                        huh.NewFilePicker().
                                Title("Select a file").
                                Value(&path),
                ),

                huh.NewGroup(
                        huh.NewNote().
                                DescriptionFunc(
                                        func() string {
                                                return "Selected path: " + path
                                        },
                                        &path,
                                ).
                                Title("Summary"),

                        huh.NewConfirm().
                                Title("Confirm?"),
                ),
        )

        if err := form.Run(); err != nil {
                panic(err)
        }
}

To reproduce:

Create a main.go file with the above contents.
go mod init example
go get<EMAIL_ADDRESS>go run .
You should now be on the first group with a file picker. Pick any file and press enter.
You should now be on the second group with an empty summary note.
Press any key (e.g. arrow key) and observe that the empty summary note suddenly re-renders with the expected description.

Expected behavior
I expect the "summary" note in the second group to display the dynamic description as soon as the user navigates to that group, like this:

Observed behavior
The "summary" note is completely empty:

Once I press any key on the keyboard, it re-renders and now looks like the first screenshot above, as expected.
Desktop (please complete the following information):

OS: macOS using iTerm

Potential patch?
I was able to resolve the issue by making this patch to trigger updating fields when switching to a new group:
diff --git a/group.go b/group.go
index d890751..bb2aae2 100644
--- a/group.go
+++ b/group.go
@@ -190,6 +190,8 @@ func PrevField() tea.Msg {
 func (g *Group) Init() tea.Cmd {
        var cmds []tea.Cmd

+       cmds = append(cmds, func() tea.Msg { return updateFieldMsg{} })
+
        if g.selector.Selected().Skip() {
                if g.selector.OnLast() {
                        cmds = append(cmds, g.prevField()...)

This appears to fix the issue for me and I don't notice any side effects, but it does break a test and I don't fully understand the architecture of this project so it may not be a correct fix.
Additional context
The behavior seems to vary depending on the type of input. If you replace NewFilePicker() with NewInput() in the minimal example, the second group now initially renders empty but then quickly re-renders with the dynamic description. This is different from the FilePicker which never re-renders until you press a key.
Screen recording showing the minimal reproduction with NewInput() so you can see the flash of missing dynamic fields:
https://github.com/user-attachments/assets/d7ae294f-393c-4e89-9769-5207f51c92e0

would love to see this change PR'd and merged!


How to get html text with font?

I'm using [self getHTML] of ZSSRichTextEditor to get html string. Its works great. But now at point, I need to set font for the same text and to get html text belonging to that particular font. Is it possible? Any luck?

I can't think of a way that you could do this without some additional features being added.

I'm not getting you.
Currently, you have set some font for the textview. But when I get the text out of it, its not containing font inside that html? And when I use, setHTML: method to set html text back, it also works well. So this is something strange. Any thoughts on this?

@nnhubbard, please check this Stackoverflow link for more details: http://stackoverflow.com/questions/33758040/how-to-format-html-string-with-font-for-zssrichtexteditor

The editor is styled using the following:
* {outline: 0px solid transparent; -webkit-tap-highlight-color: rgba(0,0,0,0); -webkit-touch-callout: none;}
html, body {padding:0; margin:0; font-family:Arial, Helvetica, sans-serif;}
body {padding:10px; overflow-y: scroll; -webkit-overflow-scrolling: touch; height:100%;}

This is the same thing that and browser does, is set some default styling for the content.

Can you just write down a code to export html text to a label with the same font as in ZSSRichTextEditor?

Just set the UILabel font to Arial. That is all you would have to do.

Ok thanks, @nnhubbard but what if I am using some other custom fonts?

ZSSRichTextEditor wasn't necessarily designed to do what you are wanting. You will have to come up with your own solution for this.
You could also use an NSAttributedString with your UILabel and then put the HTML into that.

@nnhubbard, yes currently I'm doing that thing only.
I'm converting html string to attributed string. But it doesn't contains font. Thus, its showing me some other font then in ZSSRichTextEditor. Any thoughts on this?

You would have to show me an example project so I can know what you mean.


Unable to select items in components menus

They keep collapsing when I try to hover over them, because they're overlapping the components below:

Yep, in fact I should make them clickable, to be ready for touch devices.

It's visible in the animation above, but I wanted to reiterate that the z-index of the drop-down menu needs to be adjusted such that the vertical dots aren't on top. (Personally, I would opt for icons w/ tooltips instead of the drop-down.)

Removed the contextmenu in https://github.com/aframevr/aframe-editor/pull/189


Short solution needed: "How to go to end of file" (vim)

Please help us write most modern and shortest code solution for this issue:
How to go to end of file (technology: vim)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to go to end of file


More complex navbar menu with dropdowns

Like what vuepress supports
##Acceptance

[ ] Menu dropdown in Desktop view
[ ] Menu dropdown in Mobile view

##Tasks

[x] Look at what vuepress supports
[ ] Menu dropdown in Desktop view
[ ] Menu dropdown in Mobile view

##Clarity

Should multi level dropdown be supported ?
onHover or onClick?

idea for single level implementation

and for multi level

@PhilippeduPreez good start 👏
My suggestion for analysis is to prep this in a hackmd and link that - as easier to edit and comment on and then we can copy across.
I would focus first on a job story then on thinking a bit technically how to do it and then finalize in terms of acceptance and tasks.
Aside: small formatting issues in description - markdown for Acceptance and Tasks heading is not working.

@rufuspollock Desktop view thus far...
Please let know if you prefer different styling (background hover) etc. Otherwise I use the same styling as the top level nav and will implement the same for mobile view.
Nested dropdowns are a bit more complicated, so please let know if we stick to one level for now or whether we should add nested functionality.

Mobile view thus far

@PhilippeduPreez just to sum up our today's conversation:
Looks great so far! But let's try switching from on-click to on-hover for opening the dropdown (in the navbar) so that the menu item can be a clickable item itself (in our case this could be e.g. docs page, with its guides listed in the dropdown). In the sidebar, it should be on-click, since, as you correctly noticed, there is no hover-over on mobile.
Also, you can already use it for our Docs page, so we have a live example on the Flowershow website :smiley:

Another issue is that we wouldn't be able to click on such an item in the sidebar, so we're going with your solution with just a small change - on-click to on-hover for navbar menu items. Once this is finished I'll just rename our docs page to "What is Flowershow", remove links to guides and I'll use all these pages as dropdown items to "Docs" navbar item.

FIXED. See commit here

I think there is some UI issues atm with how this works:

Width of sub-items is constrained to width of main item which is problem if text is longer (see image)
No indication that subitems exist e.g. no marker. cf current life itself website with the carets or how headlessui does it https://headlessui.com/react/menu
Hard to separate items (maybe need separator) - may be a consequence of first point

Before we do more on this though we should look at whether we want dropdown navbar at all IMO - see https://github.com/flowershow/flowershow/discussions/256


add --volume support for hyperctl

use bind mount to close #341
Now support: hyperctl run -v /host-src:/container-dest -t
Test log:
http://pastebin.com/Wr1mZwEa

@laijs @gnawux please help check the new version. Thanks

@laijs Thanks for your comments. and I have refined the codes. please help check

could you squash the commits into one please.

I would like to git rebase. but don't know what's the fast/correct way? Can you advise?


Update README.md shipped in bundle generator

https://github.com/massdriver-cloud/azure-cosmosdb-mongo/pull/3#discussion_r925656547

Lets get the readme up to date w/ some contribution / developer info and interpolate just the name and description at the top. We should also include links to docs.massdriver.cloud as that will be ready soon. Fine to 404 for now


not smooth

Captured images looks wonderful, but when running on genuine android device, its fluency makes me sad.  Hope you can optimize it, and make it a great repository.

Hello. Thanks for this issue. Its all about android technics and methods. So I has one method bringToFront() and i'll do the best currently. Hope someone provide to me more improved ways to reach this effect on all devices.

I mean fast switching pages is not consistent. BringtoFront should only affect the efficiency in the dragging part, not in the front-view-taking-over part. Later if i can help to improve this, i'll share the key point to u.

Thanks.

Hello, i forked this repository, and i have applied a new pull-request just now. I modified the touch event intercept principle by using a detector, this issue Quick swipes easy open browser is solved, and the front-view-switch section looks quite consistent and smooth. If there would help, u can merge the code.


Adding CosmWasm's Malaga-420 testnet

I would suggest we call this cosmwasmtestnet since this is what it is for. It just so happens that the token is Malaga (Great name btw!)
@msteiner96 I would suggest one of two paths:

[ ] update chain_name to cosmwasmtestnet
[ ] update pretty_name to CosmWasm Testnet
[ ] update token name in asset list to Malaga

or, if you prefer to use Malaga Testnet, rename the folder and do similar for Malaga instead of CosmWasm

Done :)

one final change in assetlist.json - change chain_name to cosmwasmtestnet and we're good!


Error: IMAP authorization error. Make sure login and password are set correctly

Hi
I have 2-step verification enabled so created the app password for the SMS Backup+.
But whenever I am trying to backup it giving me IMAP authorization error.
Please help to resolve and make it working.

Able to get it working after seeing log in detail. found that IMAP was disabled


ceylon/HDD motor encoder

Hi again!
i'm trying to follow your tutorial on using hdd motor as an encoder. however, i ran in to a couple of problems: according to schematic, it says to use pins 7, 6 and 1 on both inputs (comparing hdd pins 1 and 3/ 2 and 3) is it a mistake, because in text you mention that you needed 2 comparators and lm339 has 4.
Then the second question: in tutorial, you connected comparator output to pins 1 and 2(interrupt pins), but in wiki of opendeck, encoders are connected to 2/3, 4/5 and 6/7.
adding to that, you showed where you made changes to code. is it needed to change and upload code to arduino, when doing this?
thanks, Samu

I'm not sure what do you mean with 'mistake'. Could you elaborate? Three outputs from HHD are needed, however, one of those outputs is reference which is being used for both inputs.
That post is really old anyways and currently, OpenDeck doesn't support such fast encoders. I could implement this again on Arduino boards however sometime soon.

Okay, by mistake i meant this:

which says that 2 comparators are used out of 4. but:

shows that only pins 6,7 and 1 are used i.e. only one comparator out of 4.
then, about the pins, i misread. sorry.
But i understand that these are not supported at the moment, thanks for replying!
ps. if it isn't big trouble, hdd motors as encoders would be awesome

shows that only pins 6,7 and 1 are used i.e. only one comparator out of 4.
Ah, I see now - yeah it's an mistake then. I've actually used pins 4, 5, 6 and 7 on LM339 as inputs. As I've said - that's really outdated. I'll see what I can do about implementing that kind of setup again.


Update connector-dep.yml with OpenFaaS image

The current manifest in yaml/kubernetes/connector-dep.yml still points to my personal repo. I can easily file a fix for this but saw some inconsistencies with the tags used to build the image.
The repo currently lists tags 0.2.0 and 0.2.1 but Travis tags for the Docker images have a distinct rc-X tag. Not sure which one I should use to file a PR or just use :latest (which doesn't exist yet in Travis though)?

It is now using 0.4.0


Update 5th Edition OGL by Roll20.html

Changes / Comments

When encumbrance changes, update speed and set disadvantage appropriately

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[x ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ x] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
It assumes that the existing character's speed attribute data is correct when the script is added. It then adds a "max" value for speed, set to this value. Also these changes only apply if the Variant encumbrance rules are selected.
[ x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Greetings @geoidesic
Thanks for fixing this. :) I will need to put it through our QA process for testing. This PR may stay open for longer than typical.

Cassie

Greetings @geoidesic
Speed
The speed update appears to be working except for one important scenario. If you use the variant rules and are encumbered the speed is updated as expected. However if you then toggle off the variant rules it needs set speed back to its original value.
One potential solution would be setting speed_max silently before reducing it. Then if variant rule is toggle off set speed back to the max.
Disadvantage
The encumbrance penalty for Disadvantage only applies to the three physical skills. Adding this will turn it on for all rolls. This will cause confusion for some users. At this time we want to avoid this level of automation.
Thanks,
Cassie

@clevett I'll leave that to the Roll20 devs. I'm happy with the change for my own purposes and I offer my code here as an aid to those who may benefit from it or wish to use it as a starting point.
Speed
The speed_max is being set silently already, so the only change required as per your comments would be to copy speed_max to speed when Variant rules are toggled off. I don't know how to check for the toggle status.
Disadvantage
Fair enough. Personally I think having disadvantage set across the board is a valuable reminder and the DM can always resolve any confusion. That's how I'll be running it in my campaigns anyhow.

@geoidesic
Thanks for the code. :) Its good to know regarding the max speed. I can look at our schedule and see about getting that added in.

Closing this request for now. It has been added to our backlog of feature requests.

Any news on when this might be implemented?


Multiple Select + Validator Error

When using data-validate="required" for a multiple select.
the Validate function does not work and says val.trim is not a function.

Thanks, fixed in 4.2.39


Could not find any Service Instance for the selected Space message

Could not find any Service Instance for the selected Space message is given in Eclipse for ABAP cloud project. API endpoint is https://api.cf.eu10.hana.ondemand.com and region is Europe (Frankfurt). Is there any available fix for this?
Trying to do this tutorial. There is no available information on creating service instance.

@no-glue You need a SAP Cloud Platform ABAP Environment user + password to create a ABAP Cloud Project.


The description for require a reset to complete the SET_CERTIFICATE request in the specification is not clear

The description for require a reset to complete the SET_CERTIFICATE request in the specification is not clear.
It doesn't mention --- After the Responder has processed the reset:
if the Requester sends the same SET_CERTIFICATE that was sent before the reset, what to do then?
if the Requester sends the different SET_CERTIFICATE request than the request that was sent before the reset, what to do then?

Refer to paragraph 675 in DSP0274_1.2.1.
The device might require a reset to complete the SET_CERTIFICATE request, potentially so that the device can generate AliasCert certificates using lower firmware layers. If the device requires a reset to complete the SET_CERTIFICATE request, then the device shall respond with an ErrorCode=ResetRequired response.

I think more description for require a reset to complete the SET_CERTIFICATE request should be added in the specification  DSP0274_1.2.1.
May be just like the paragraph 666 in DSP0274_1.2.1 for requires a reset to complete the GET_CSR request.
If the device requires a reset to complete the GET_CSR request, then the device shall respond with an ErrorCode=ResetRequired response. After the Responder has processed the reset, the Requester sends the same GET_CSR request that was sent before the reset, which signals to the Responder to send the CSR response. If the Requester sends a different GET_CSR request than the request that was sent before the reset, then the Responder shall treat it as a new request or reject the different GET_CSR request with an ErrorCode=UnexpectedRequest response. On receipt of a new GET_CSR request, the Responder can discard any existing CSR responses.

Unlike GET_CSR, the Requester doesn't have to do anything if a Responder requires a reset to complete SET_CERTIFICATE. If, after a reset, the Requester sends another SET_CERTIFICATE the Responder processes it, regardless if it's the same certificate or a different certificate.

If, after a reset, the Requester sends another SET_CERTIFICATE the Responder processes it, regardless if it's the same certificate or a different certificate.

Is that in SPDM spec? I am surprised on that.

The ResetRequired was put in to support DICE. If the leaf certificate is generated by the bootrom then, after a SET_CERTIFICATE, the reset will generate a new leaf certificate. The Requester doesn't have to do anything.

So, for the Scenario--the device requires a reset to complete the SET_CERTIFICATE request:

Requester send the SET_CERTIFICATE request to the device.
the device respond with an ErrorCode=ResetRequired response, due to require a reset to complete the SET_CERTIFICATE request.
the device processed the reset and completed the SET_CERTIFICATE request.
Requester send another SET_CERTIFICATE request(the same or a different SET_CERTIFICATE request) to the device.
the device respond with an ErrorCode=ResetRequired response, due to require a reset to complete the SET_CERTIFICATE request.
the device processed the reset and completed the SET_CERTIFICATE request.
repeat the step-4 ~ 6
... ...

in the above Scenario, It looks like:
a. The device will always respond with an ErrorCode=ResetRequired response.
In other words, the Requester will always receive an ErrorCode=ResetRequired response, the Requester will never receive a Successful SET_CERTIFICATE_RSP response message.
b. Then the requester how to know whether the device completed the SET_CERTIFICATE request Successfully?

Then the requester how to know whether the device completed the SET_CERTIFICATE request Successfully?

Requester can issue a GET_CERTIFICATE and compare that the portion it sent to the Responder is the same that's returned from the Responder.

Sending 2nd SET_CERTIFICATE will override the previous CERTIFICATE.

In current code, libspdm_get_response_set_certificate will call libspdm_write_certificate_to_nvm.
    /* set certificate to NV*/
    result = libspdm_write_certificate_to_nvm(slot_id, cert_chain,
                                              cert_chain_size);

In the libspdm_write_certificate_to_nvm, the slot_id is related to the file_name.
file_name[strlen(file_name) - 1] = (char)(slot_id+'0');
When requester send 2nd SET_CERTIFICATE for the same slot_id, the responder will override  the previous CERTIFICATE.

please add unit test.


externalPackages doesn't work in monorepos

External Packages Fail in Monorepos Due to Dependency Hoisting - Major DX Issue
Description
There's a significant issue when using Convex's externalPackages for node envs feature within monorepos. The current implementation doesn't account for monorepo package management, where dependencies are typically hoisted to the root. This affects ALL external packages, with Sharp being a prime example in my case.
Problem Scope
This is NOT just a Sharp-specific issue - it affects any external package used with Convex in a monorepo setup.
The externalPackages feature expects packages in the local node_modules, and since monorepos hoist the dependecies to the root, convex is unable to find them.
Current Behavior
When using ANY external package in a monorepo:

Dependencies are hoisted to root node_modules (standard monorepo behavior)
Convex can't locate the package even when it's in root package.json
Deployment fails due to missing dependencies
Adding to root package.json is insufficient - must also add to the Convex app's package.json

Example with Sharp
monorepo/
  ├─ node_modules/ (where packages actually live)
  ├─ apps/
  │  ├─ convex-app/
  │  │  ├─ node_modules/ (where Convex expects packages)
  │  │  ├─ package.json (needs duplicate declaration)

Current Workaround (Hacky)

Create manual node_modules inside the Convex app directory
Physically copy package files from root node_modules
Add duplicate package entries in both root and app package.json
Configure as externalPackage

Suggested Solutions

Short-term: Document this limitation clearly in the externalPackages docs
Medium-term: Add monorepo detection and automatic package location resolution
Long-term: Redesign the externalPackages feature to work with hoisted dependencies

Environment

Package Managers Affected: pnpm, yarn, npm, bun (all with workspaces)
Monorepo Tools: Turborepo, Nx, others
Convex Version: ^1.14.4

@IAlphaOmegaI thanks for submitting, sorry this slipped through the cracks for a month. If you have a paid Convex plan feel free to email<EMAIL_ADDRESS>if you're blocked on something like this in the future.
What's your status with this currently, how's the workaround going?
There are several different ways monorepos can be set up, we use Rush which doesn't use a shared package.json at the root, but yeah the root style is more common.
Sounds like the convex CLI needs to implement the same package resolution behavior as Node.js, walking up directories to the first ancestor with a node_modules directory with this directory.

@IAlphaOmegaI thanks for submitting, sorry this slipped through the cracks for a month. If you have a paid Convex plan feel free to email<EMAIL_ADDRESS>if you're blocked on something like this in the future.
What's your status with this currently, how's the workaround going?
There are several different ways monorepos can be set up, we use Rush which doesn't use a shared package.json at the root, but yeah the root style is more common.
Sounds like the convex CLI needs to implement the same package resolution behavior as Node.js, walking up directories to the first ancestor with a node_modules directory with this directory.

The workaround works great honestly, it's not that hard to maintain and it does the job well. Since I don't use a big number of external dependencies, it is not hard for me to cut and paste from monorepo node_modules to apps/provider (where my convex app is) node_modules.


Rely on more reliable invitation_created_at

Use more reliable method for determining of an invitation was created.

Closing. This is a preventative PR. If the problem actually appears, we can just re-open this PR.


Clean resources

Merge #97 first!

Velocyto is not working, but even so a lot of components are fixed by merging this branch :)


Definition request: devextreme 15.1

DevExtreme has updated to v15.1 and adds some nice new functionality.  Could we please get an updated d.ts for this?

Hello @daveshapiro,
At the moment, you can get TypeScript definitions for DevExtreme 15.1 from here.

Beauty!  Thank you very much, @Seteh!


Drag and drop not working on touchscreen devices

It seems that we can't drag and drop the items on the detail page on touchscreen devices. When I try drag an item, it would scroll the page instead.
https://github.com/RasyaJusticio/JscioTasks/assets/125113883/2c4630df-ec9a-48ef-8e7a-3f6057f1b592

I might need to use a third-party drag & drop library that is reliable instead of my own to fix this.


LLMOps recommends a project（fastgpt）

https://github.com/labring/FastGPT/
FastGPT is a knowledge-based question answering system built on the LLM. It offers out-of-the-box data processing and model invocation capabilities. Moreover, it allows for workflow orchestration through Flow visualization, thereby enabling complex question and answer scenarios!

you could file a PR for it, thanks


Container created on network mode host instead of managed

Rancher versions:
rancher/server or rancher/rancher: v1.6.15
rancher/agent or rancher/rancher-agent:
Docker version: (docker version,docker info preferred)
Docker version 18.03.1-ce, build 9ee9f40
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
ubuntu
Hi, I have 3 containers deployed on Rancher on network mode Managed so they receive an IP address from Rancher. I am launching an Hyperledger command ( composer network start ) from a different host, this command connect to one of my container and it try to create a new container (4th). My issue is that this last container is created with network mode host so it cannot communicate with the 3 manually created before.
The container who receive the connection has an env variable named CORE_VM_DOCKER_HOSTCONFIG_NETWORKMODE, this env var is read by my process but apparently considered as a network name, not as a mode ....
On the docker side, if a try a "docker network ls", I see that the 3 first container are all linked to the same network named "none" ....
Can somebody help ?
Thanks

Container with host networking can reach containers on the overlay network by IP, but that's not very dynamic and requires retrieving the correct IP but can be done. If you want a container launched via Docker to join the managed network, you can add a label to it as described here:https://rancher.com/docs/rancher/v1.6/en/rancher-services/networking/#containers-created-with-the-docker-cli
For any containers launched through the Docker CLI, an extra label --label io.rancher.container.network=true can be used to launch the container into the managed network.

Additional info on the same issue : https://github.com/hyperledger/composer/issues/4129
Thanks to superseb, I will investigate this and come back to you.

@superseb  The point is that I am not using Docker CLI (not directly) I am launching an Hyperledger composer command so I don't see yet how to pass this --label option ....

I tried the same thing with all container on network mode "host", but not working because containers can communicate with each other .... Any idea around ?

Thanks for your report!
With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.
If you think we should keep this issue open, please let me know.


Kaboom examples - button  example text broken

Steps to reproduce:
got to https://kaboomjs.com/examples#button
View the rendered result- >

Expect to see:  Text on the Buttons.
Actually see:  Text is obscured by black rectangles.
If you comment out the rect() call, the text reappears:
add([
			text(txt),
			pos(p),
			//rect(100, 100),
			origin("center"),
			color(0, 0, 0),
		]);

Is this intentional or a bug?

Yes this looks like a bug that's introduced in the middle of some graphics refactor, I'll investigate

(fixed)


DBZ-3364 add additional metadata to fields for API spec generator for the Debezium UI

This is the first part of splitting up PR #2273 into smaller parts, and continue from that PoC PR.
part of https://issues.redhat.com/browse/DBZ-3364

@jpechane, can you take a look too.  Note this change by itself doesn't provide any value yet, it's rather the prerequisite for the API spec generator that will be created in a next step, which will create a JSON Schema definition of all configuration options, which then are exposed from KC to the DBZ UI backend.

@gunnarmorling A question related to numbering added.

@rk3rn3r, about validating the order, I'm fine with doing this in the generator. In that light, can we merge this one then?

@gunnarmorling I was thinking about this yesterday. I am leaning towards adding unit tests that validate duplicates per connector. so we realize on every PR when a connector has duplicate positions. So we keep the production code slim but have validation (earlier than only when executing the generator).

That's fine for me too. Whatever floats the boat. Let's just try and get this done for next week's release. Thanks :)

Great! I'm close to having a junit test that can give output like that:
[ERROR] Failures:
[ERROR]   FieldTest>AbstractFieldTest.shouldNotAllowDuplicateGroupAssignment:50 Duplicate field group assingnments found: "signal.data.collection" uses an occupied position in group "ADVANCED". Position no. 19 is already taken by field: "source.struct.version

@rk3rn3r, could you rebase this PR, addressing the merge conflict? Other than that, is it good to go from your PoV?

Applied; thx a lot, @rk3rn3r! Looking forward to the follow-ups and the usage for the UI 🚀 .


refactor(ES6): WatchTestCases.test.js

What kind of change does this PR introduce?
refactor, ES5 => ES6

Did you add tests for your changes?
All existing tests pass

If relevant, link to documentation update:
N/A

Summary
Refactor WatchTestCases.test.js to ES6

Does this PR introduce a breaking change?
No

Thanks


A bizarrerie url will return null by method getType().

If you have an issue with a specific extension or type
Locate the definition for your extension/type in the db.json file in the mime-db project.  Does it look right?

[x] No. File a mime-db issue.
[ ] Yes: Go ahead and submit your issue/PR here and I'll look into it.

Example
mime.getType('https://s3.cn-north-1.amazonaws.com.cn/xiaoju-material/public/5d0231f2b847d96f9cdfdae1_1587716279293_null') // return null

This url will return null by getType().
Do you have some good idea about process this kind of url?

This module is not designed to extract type information from a URL endpoint (e.g. by fetching the URL and inspecting headers).  It is strictly for inferring type from a file name (specifically, a file extension).
Most URLs will provide a Content-Type header which will be made available by whatever API module you use to make the request.  If you need to know the type prior to fetching the URL, you can make a HEAD request.  E.g. request.head(), axios.head().  Or for a one-off occurance you can do this using curl from the command line:
$ curl -I "https://s3.cn-north-1.amazonaws.com.cn/xiaoju-material/public/5d0231f2b847d96f9cdfdae1_1587716279293_null"
HTTP/1.1 200 OK
x-amz-id-2: uvMK24y/0EYSCIzAegqN14LFflHYJJ9pI70IWGd/WiBd7dDlbwoDIeE38b8rSSYfu0Rf18gNb8c=
x-amz-request-id: 44B2F7E2B31F51AA
Date: Fri, 24 Apr 2020 16:50:20 GMT
Last-Modified: Fri, 24 Apr 2020 08:18:00 GMT
ETag: "fdac0d8bfcc3ea1bda5337d0802260e4"
Accept-Ranges: bytes
Content-Type: image/png
Content-Length: 11576
Server: AmazonS3

Thanks for your professional answer.


Keyboard manager sometimes doesn't work

Microsoft PowerToys version
0.76.1
Installation method
PowerToys auto-update, Microsoft Store
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce

Assign CTRL + Q to Alt + F4
Open some specific programs such as whatsapp, steam
After openning these, This shortcut doesn't work

✔ Expected Behavior
Specified shortcut should be working even if some apps are opened.
❌ Actual Behavior
Shortcut doesn't work after this apps are opened.
Other Software
No response

I apologize for getting into the wrong topic, but perhaps you can tell me where and how to make a request to add to this Keyboard Manager the ability to reassign one key in a particular application. Specifically, I would like it so that when I press the Enter key in Discord, Discord would accept the command to move the line (Shift + Enter) rather than send a message.

This also is an issue for me.
On the bug report files I found the error "Improper JSON format for shortcut remaps":

[2023-12-12 08:26:39.274972] [p-13420] [t-13424] [info] keyboard-manager logger is initialized
[2023-12-12 08:26:39.325444] [p-13420] [t-13424] [trace] SettingsHelper::LoadSettings()
[2023-12-12 08:26:39.402071] [p-13420] [t-13424] [error] Improper JSON format for shortcut remaps. Skip to next remap type
[2023-12-12 08:26:39.402235] [p-13420] [t-13424] [error] Improper JSON format for single key to text remaps. Skip to next remap type
[2023-12-12 08:26:39.917450] [p-13420] [t-13424] [trace] SettingsHelper::LoadSettings()
[2023-12-12 08:26:39.918140] [p-13420] [t-13424] [error] Improper JSON format for shortcut remaps. Skip to next remap type
[2023-12-12 08:26:39.918462] [p-13420] [t-13424] [error] Improper JSON format for single key to text remaps. Skip to next remap type
[2023-12-12 09:33:45.016280] [p-13420] [t-13560] [trace] PowerToys runner exited.
[2023-12-12 09:33:45.016511] [p-13420] [t-13560] [trace] Exiting KeyboardManager engine
[2023-12-12 09:33:58.404552] [p-20100] [t-18756] [info] keyboard-manager logger is initialized
[2023-12-12 09:33:58.460091] [p-20100] [t-18756] [trace] SettingsHelper::LoadSettings()
[2023-12-12 09:33:58.462507] [p-20100] [t-18756] [error] Improper JSON format for shortcut remaps. Skip to next remap type
[2023-12-12 09:33:58.462620] [p-20100] [t-18756] [error] Improper JSON format for single key to text remaps. Skip to next remap type
[2023-12-12 09:33:58.970382] [p-20100] [t-18756] [trace] SettingsHelper::LoadSettings()
[2023-12-12 09:33:58.972002] [p-20100] [t-18756] [error] Improper JSON format for shortcut remaps. Skip to next remap type
[2023-12-12 09:33:58.972108] [p-20100] [t-18756] [error] Improper JSON format for single key to text remaps. Skip to next remap type

I understand this is the referenced invalid JSON (three remappings):
{"remapKeys":{"inProcess":[]},"remapShortcuts":{"global":[{"originalKeys":"164;112","newRemapKeys":"91;162;37"},{"originalKeys":"164;113","newRemapKeys":"91;162;39"},{"originalKeys":"164;115","newRemapKeys":"135"}],"appSpecific":[]}}

I found a temporal workaround. Locking / unlocking (Win+L) the screen fixes the issue. Until the next time.

This is still a problem for me, although the locking/unlocking workaround works.


0.1.33: Subzero temperatures broken on DHT22

Using the example dht22_test on Arduino Uno and Arduino 1.8.15, I get this when I hang the sensor out the window:
DHT22, 	OK,	57.7,	0.5,	4836
DHT22, 	OK,	57.9,	0.4,	4888
DHT22, 	OK,	58.1,	0.4,	4932
DHT22, 	OK,	58.3,	0.3,	4980
DHT22, 	OK,	58.6,	0.3,	5024
DHT22, 	OK,	58.7,	0.2,	5024
DHT22, 	OK,	59.0,	0.2,	4932
DHT22, 	OK,	59.2,	0.1,	4888
DHT22, 	OK,	59.4,	0.1,	4932
TOT	OK	CRC	TO	CON	ACK_L	ACK_H	UNK
140	140	0	0	0	0	0	0
DHT22, 	OK,	59.5,	0.0,	4928
DHT22, 	OK,	59.8,	0.0,	4876
DHT22, 	OK,	60.0,	-3276.7,	5072
DHT22, 	OK,	60.2,	-3276.7,	5120
DHT22, 	OK,	60.3,	-3276.6,	5212
DHT22, 	OK,	60.6,	-3276.6,	5072
DHT22, 	OK,	60.9,	-3276.6,	5024
DHT22, 	OK,	61.1,	-3276.5,	5072
DHT22, 	OK,	61.3,	-3276.5,	5116
DHT22, 	OK,	61.4,	-3276.4,	5072
DHT22, 	OK,	61.7,	-3276.4,	5172
DHT22, 	OK,	62.0,	-3276.4,	5072
DHT22, 	OK,	62.2,	-3276.3,	5212
DHT22, 	OK,	62.3,	-3276.2,	5304
DHT22, 	OK,	62.6,	-3276.2,	5176
DHT22, 	OK,	62.8,	-3276.2,	5172
DHT22, 	OK,	63.0,	-3276.1,	5352
DHT22, 	OK,	63.2,	-3276.1,	4976
DHT22, 	OK,	63.5,	-3276.1,	5076
DHT22, 	OK,	63.7,	-3276.1,	5116

Hi @jeskata
Thanks for posting this issue.
Please use the DHTNew  library if possible as it has many fixes.
It has also a fix for his subzero one.
If you definitely need this AVR-optimized version let me know, then I will fix it asap.
Rob

@jeskata
I have made a branch - https://github.com/RobTillaart/DHTlib/tree/develop
Can you test it?
Thanks,

Yes, it seems to fix the issue.

Then I will merge it into master,
thank you for testing!

Released 0.1.34
If there are no more questions you may close the issue.


hashcat-dictionary-tasker Pause() cannot send error

There is no return that includes an error.

Pause never has an error. If it cannot pause the job, then by convention
you set the Error variable and quit. I documented that somewhere.
On Apr 6, 2015 9:47 PM, "Lucas Morris"<EMAIL_ADDRESS>wrote:

There is no return that includes an error.
—
Reply to this email directly or view it on GitHub
https://github.com/jmmcatee/cracklord/issues/33.


Transaction specification in benchmark file could be better

Currently we define the transaction details (number of, or duration of) by:
{
  label: 'TestLabel0',
  txNumber: 60,
  rateControl: { },
  workload: {
    module: 'fabcar/createCar.js'
  }
}

it would be better to have:
{
  label: 'TestLabel0',
  transactionLoad : {
    type: duration | number,
    magnitude: number
  },
  rateControl: { },
  workload: {
    module: 'fabcar/createCar.js'
  }
}

or as part of the workload definition itself
{
  label: 'TestLabel0',
  rateControl: { },
  workload: {
    module: 'fabcar/createCar.js'
    transaction: {
      type: duration | number,
      magnitude: number
  }
}

This would be of benefit if we ever moved into typings, since it is a more logical layout

I absolutely support this! I like the middle version, it shows nicely that the concepts are orthogonal 👍


[6.x] Removed static return type PHPDoc from first()

first() can never return static, so it shouldn't be listed as a possible return type.
For example, Model::where('id', 1)->first() is only going to return Model or null.
Per #22854 it can return a generic object.
static was added in #20254, when the PHPDoc for the function was changed from mixed to union return types, though there isn't any reason given why static was included. I can't see in documentation anywhere where it should be returning a Builder object either. If there in fact is a way that this could occur then this is moot, but I don't think there is.
Including static generates incorrect suggestions in IDEs and implies from reading PHPDoc return types that we don't have to worry about.

Well, wouldn't User::first() return a User, not a Model? That's what the static return type means. Removing the return type means the type hinting will no longer suggest methods in the User class.

@sisve good call. Sent in the reverting pr here: https://github.com/laravel/framework/pull/29892

Thanks @sisve and @driesvints !

It was a deep drive to determine you're right, in some cases removing static does disrupt autocomplete.
So first() does not exist in Model or any class that User extends, it only exists in Illuminate\Database\Query\Builder and Illuminate\Database\Eloquent\Builder. This is why the static typehint is suggesting Builder as a return type. This would lead me to believe that it wouldn't be possible for it to suggest User as a return type.

User::first() passes through __callStatic to __call which runs first() on a new instance of the Eloquent Builder.
However, if you use laravel-ide-helper, it copies the docblock into a faux class it uses for typehinting, where first() then does in fact exist in a User class, where static can then typehint back to it. So it depends on the IDE tools and the tricks they're using.
There is still then an open issue, though, of the opposite case. If you're not using that helper, static is saying that an Eloquent Builder object is returned, not a specific Model.


[AKS] az aks create/update: Add support for KEDA workload auto-scaler

Related command

Description

[AKS] az aks create/update: Add support for KEDA workload auto-scaler
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Live test failed due to some error on rp side.

AKS

@v-shenoy In fact, it is not necessary to add yaml files for the @live_only tests.


Desert Eagle does not work as a weapon and nitrous causes the passenger to die.

Deagle does not equip. Also sometimes the drug dealer sells nothing

I am not able to reproduce this issue with the deagle. It equips fine.
Nitrous issue is likely specific to the car. In which car did you experience this?
The drug dealer is always either selling or buying something. Sometimes he's only buying, and if you don't have the item he's buying, it doesn't show up in the window.

Please close.


Investigate the SLO history

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Could we see the history of SLO reports, in some fashion.
To be investigated ...
Probably, add a command like slo history
maybe add some flags to select a service ? a slo ?
maybe add a number of reports to view/retrieve ?
maybe add a time window to get the history between to dates, date&time ?
Describe alternatives you've considered
Visualisation, rendering of the history has to be investigated...
do we display the history directly in the terminal ? as an html page ?
could we have a chart for the history (https://github.com/go-echarts/go-echarts)
Additional context
https://github.com/gonum/plot/
https://github.com/buger/goterm/
https://github.com/wcharczuk/go-chart

WIP https://github.com/reliablyhq/cli/tree/slo-history


GPL-3.0 identifier

https://spdx.org/licenses/GPL-3.0-only.html
https://spdx.org/licenses/GPL-3.0-or-later.html
https://opensource.org/licenses/GPL-3.0
https://www.gnu.org/licenses/gpl-3.0-standalone.html

This license was released: 29 June 2007. This license identifier refers to the choice to use the code under GPL-3.0-only, as distinguished from use of code under GPL-3.0-or-later (i.e., GPL-3.0 or some later version). The license notice (as seen in the Standard License Header field below) states which of these applies to the code in the file. The example in the exhibit to the license shows the license notice for the "or later" approach.

The only difference is in the Standard License Header, which is not included in license. So for most apps not state the Standard License Header, we don't know which one is used. Why don't use GPL-3.0 for them ? Or we should regard them as GPL-3.0-only ?

Most of the licenses is GPL-3.0-or-later, except for those explictly said GPL v3 only. You can check the license file attached, and look for or later.

If the Program specifies that a certain numbered version of the GNU General Public License "or any later version" applies to it, you have the option of following the terms and conditions either of that numbered version or of any later version published by the Free Software Foundation.

This program is free software: you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version.

I can only find two place. The first one is an assumption. The second one is a template. -only or -or-later need to be state in the Standard License Header. For example, Shadows-libev states -or-later in readme.

No, look for or later in license file attached with software, or for GitHub hosted one, check license section.
The standard GPLv3 license file has or later statement, so GPL-3.0-or-later.

I can't find such a statement... :confused:

Just example: https://github.com/franko/gsl-shell/blob/master/LICENSE#L640

It is after the END OF TERMS AND CONDITIONS and in How to Apply These Terms to Your New Programs section. As is says before

To do so, attach the following notices to the program.

I don't think it is a statement.

See L572 in section 14, if not explicitly stated, later version could be applied by default.

Each version is given a distinguishing version number.  If the
Program specifies that a certain numbered version of the GNU General
Public License "or any later version" applies to it, you have the
option of following the terms and conditions either of that numbered
version or of any later version published by the Free Software
Foundation.  If the Program does not specify a version number of the
GNU General Public License, you may choose any version ever published
by the Free Software Foundation.

If I'm reading it right, if the licensor specifies a version of the license but no explicit "or later" statement, the software is under GPLv3 only, not GPLv3 or later.

If the Program specifies that a certain numbered version of the GNU General Public License "or any later version" applies to it, you have the option of following the terms and conditions either of that numbered version or of any later version published by the Free Software Foundation.

I do not understand this clearly until now... In my opinion it means when the GNU General Public License "or any later version" applies to it , you can choose other version. As I'm not good at English, this realy make me confused...


Linaf/bedrockdoc

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: katarzyna-koltun-mx:x: mx-awsdevteamYou have signed the CLA already but the status is still pending? Let us recheck it.

https://github.com/mendix/docs/pull/7398 created to resolve the file conflict. I moved all the changes there.

Closing.


Works only version 2.0.0 not latest version on Phonegap build

I'm submitting a ...  (check one with "x")

[ ] question
[x] any problem or bug report
[ ] feature request

If you choose 'problem or bug report', please select OS:  (check one with "x")

[x] Android
[ ] iOS

cordova information:  (run $> cordova plugin list)
insert the output from the command here

If you use @ionic-native/google-maps, please tell the package.json (only @ionic-native/core and @ionic-native/google-maps are fine mostly)
Current behavior:
With 2.0.0 version everything works fine. Latest versions not working as well on phonegap build.
Map loads correctly but cant do anything with touch (rotate maps, touch on poi, ect)
Expected behavior:
Latest versions should work like the 2.0.0
Screen capture or video record:

Related code, data or error log (please format your code or data):
The person who share your project files on Github (or other git repository) is in faster lane than other people.
Please share your project files on Github or others(Bitbucket, Gitlabs...etc).
If you don't want to share your project files, please create a demo project, then share it.

Screen captures, and/or native logs(such as Logcat, xcode logs) are appreciate.

Giving much information, you are waiting time is less.
Thank you for your cooperation.

Please share your project files on github.

ping

I'm using Vuejs with Quasar Framework and phonegap build.
Is enough that i change this line in my config.xml
<plugin name="cordova-plugin-googlemaps" spec="2.0.0">
with a bigger spec ex.
<plugin name="cordova-plugin-googlemaps" spec="2.2.2">
the map freezes and i cant do anything with touch.
I will try to share the code.

In order to reproduce your issue, please share your project files. Otherwise, please debug by yourself

Here the code: google maps project
the component that i use is "test/src/components/GeoGoogleMapsCordova.vue"
When i use this code with the version 2.0.0 in my config.xml everything works.

Thank you for sharing, but it seems you don't include the config.xml file

Share the config.xml also!

Thank you for sharing. Could you tell me how to build and run your project? I'm not familiar with Quasar Framework

Sure,
after install quasar in the root project just give quasar build it will create a "dist" folder with compiled files inside.
Those files must be included in 'www' phonegap folder with this structure:

www (inside the compiled files)
config.xml

zip everything and load on phonegap build site.

Ok, I finally figured out the reason.
First of all, the v2.0.0 and v2.2.2 are completely different softwares.
So comparing each others are non-sense.
In the v2.2.2, the touch recognition complies with the CSS rules perfectly(I believe so), such as z-index and  overflow rules.
In your app, the parent element of the map div has overflow: hidden, and the element height is 0.

(you can click the images)
That's why the map div(<div id="map_canvas">) can not receive the touch event, thus, you can not touch the map.

Thanks man! Was my code fault.

You are welcome.
By the way, I appreciate if you support for this project.
This is not mandatory asking, but I appreciate if you donate some tip because I spend my private time.
This is $5 donation link at paypal


Getting  error, API down?

I have it on my bot in Discord but I haven't used it, let alone, touched the command for cleverbot. Now I just get  as a output.

The cleverbot code must be structured differently now (always wrapped in a prepare() statement).  See the usage here: https://github.com/fojas/cleverbot-node

I just checked and that's actually how I have it, excluding the tweaks to post stuff in my chat.
var cleverbot = require('cleverbot-node'),
    talkbot = new cleverbot

exports.talk = {
    talk(bot, msg, suffix) {
        cleverbot.prepare(function() {
            var conv = suffix
            talkbot.write(conv, function(response) {
                bot.reply(msg, `${response.message}`)
                console.log(response.message)
            })
        })
    }
}

Which version are you using? The older versions do not work anymore because of changes to cleverbot. The last few updates have just been for keeping up with those changes, so everything before the latest release is effectively broken.


Caching Kokkos build for wheel-builds

Context:
This PR introduces the use of caching for Kokkos' libraries, to decrease wheel-build time.
Description of the Change:
The cache GH action is used for caching the Kokkos libraries.
Benefits:
Faster build times.
Possible Drawbacks:
Related GitHub Issues:

Nice work @AmintorDusko, but it seems like there is a typo in yml file for MacOS: https://github.com/PennyLaneAI/pennylane-lightning/actions/runs/2870428237 Could you fix it?

Nice work @AmintorDusko, but it seems like there is a typo in yml file for MacOS: https://github.com/PennyLaneAI/pennylane-lightning/actions/runs/2870428237 Could you fix it?

Hi @chaeyeunpark, this PR is a WIP. Only the Linux wheel is supposed to work at the moment.
Thanks for the fix for the CMAKE kernels libraries file.

Nice work @AmintorDusko, but it seems like there is a typo in yml file for MacOS: https://github.com/PennyLaneAI/pennylane-lightning/actions/runs/2870428237 Could you fix it?

Hi @chaeyeunpark, this PR is a WIP. Only the Linux wheel is supposed to work at the moment. Thanks for the fix for the CMAKE kernels libraries file.

Aha, I see. Still, speed-ups for Linux wheels seem great!

Nice work @AmintorDusko, but it seems like there is a typo in yml file for MacOS: https://github.com/PennyLaneAI/pennylane-lightning/actions/runs/2870428237 Could you fix it?

Hi @chaeyeunpark, this PR is a WIP. Only the Linux wheel is supposed to work at the moment. Thanks for the fix for the CMAKE kernels libraries file.

Aha, I see. Still, speed-ups for Linux wheels seem great!

Yes! I'm pretty happy with that!

Here follows a rough estimative of increase in workflow performance.
Data was extracted from one of the most recent runs (Python 3.7).

Wheel
timing without caching
timing now

Wheel::Linux::ARM
49
14

Wheel::Linux::PowerPC
56
13

Wheel::Linux::x86_64
10
5

Wheel::MacOS::Intel
10
5

Nice work @AmintorDusko Just a few follow-up comments.
We may also want to backport the Windows fix to a fork of the v0.25.0 branch for the v0.25.1 release candidate.

Are we pushing it to a proper PR or directly to the v0.25.1 rc?

Nice work @AmintorDusko Just a few follow-up comments.
We may also want to backport the Windows fix to a fork of the v0.25.0 branch for the v0.25.1 release candidate.

Are we pushing it to a proper PR or directly to the v0.25.1 rc?

I think since this only affects the CI, and hence the build of the wheels we have, we can probably offshoot this into a new v0.25.1 branch with 0.25.0 as the base. This should also make sure nothing else major was added between the 0.25 and master branches though, otherwise we will need to do a git cherrypick of these specific commits.


[ask for help] how to use mqtt simulator

hi, i apply the mqtt simulator, but how to create a devicelink to use the simulator?
could you give a readme for help please, thank  you very much.

@carolove , we will synchronize the usage of the simulator to the Octopus repo ASAP, and I will ping you later.

@carolove , the simulator is in, you can follow the example under deploy/e2e to have a try.

thank you for reply.


Fixed bug where sys.executable may follow symlinks

Currently, sys.executable calls std::env::current_exe. With current_exe, it is platform defined if it follows symlinks. However, from my testing sys.executable never follows symlinks in CPython. This behavior is important in certain cases, e.g this test only passes if symlinks aren't followed.
This changes sys.executable to not follow symlinks.

I just added those tests.

Thanks for contributing!


Bug#1167 my subscription page search issue

Changes in this Pull Request:

Clear searchPhraseProperty after page unload

Mandatory checks:

[x] Branch is up to date (latest changes were fetched from the upstream branch before creating this PR)
[x] Changes were tested manually
[ ] Java code changes are unit tested
[ ] Webtests are successful

To be checked by the reviewer:

[ ] Clean Java code and unit tested changes
[ ] UI customizablity is preserved: No hard-coded styling and URL references in JSF views

To be checked by the pull request owner:

[ ] .MASTER/job/BES_MASTER_IT/
[ ] .MASTER/job/OSCM_WS_Tests_INTERNAL/
[ ] .MASTER/job/OSCM_WS_Tests_OIDC/
[ ] .MASTER/job/OSCM_Integration_Tests_INTERNAL/
[ ] .MASTER/job/OSCM_Integration_Tests_OIDC/

Cross repository and core changes:

[ ] Interdependent changes have been communicated to the team (if applicable)
[ ] This pull request includes high risk modifications or core changes (e.g API, datamodel, authentication...). Mandatory reviewers: @goebell or @kowalczyka.

OSCM Build References:

<CI node number>: <tag>, example: ci10:goebell

Screenshot (if applicable):

add a screenshot illustrating your change

This change is

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @farmakiG and @KabacinskiPiotr)

oscm-portal/WebContent/WEB-INF/facelets/tags/marketplace/subscriptionFinderBootstrap.xhtml, line 27 at r1 (raw file):

  window.onunload = function() {
   document.getElementById('subListForm:searchField').value = "";
   document.getElementById('subListForm:submitLink').click();

What is the purpose of this inline javascript? Please avoid this if possible.

The goal is to solve the problem:
When you leave the page and go back, only the searched subscriptions are shown, although the search field is empty .
I think that without a thorough reconstruction of the backend I cannot do anything better.

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @farmakiG and @KabacinskiPiotr)

oscm-portal/WebContent/WEB-INF/facelets/tags/marketplace/subscriptionFinderBootstrap.xhtml, line 27 at r1 (raw file):

  window.onunload = function() {
   document.getElementById('subListForm:searchField').value = "";
   document.getElementById('subListForm:submitLink').click();

What is the purpose of this inline javascript? Please avoid this if possible.

The goal is to solve the problem:
When you leave the page and go back, only the searched subscriptions are shown, although the search field is empty .
I think that without a thorough reconstruction of the backend I cannot do anything better.

oscm-portal/WebContent/WEB-INF/facelets/tags/marketplace/subscriptionFinderBootstrap.xhtml, line 27 at r2 (raw file):
Previously, GoebelL (Lorenz Goebel) wrote…
Please move the java script function in AdmUtils

Done.


Return correct rule name while using i18n

Currently, rules translations don't work when i18n is enabled.

@timriley yes lgtm

Do you have any updates with this PR?

Merged! Thanks @jozzi05. I'll get a minor release out later today.

Released as 0.12.2

Great, thanks! ;)


IPNWLGEFPW

aka "ipynb with lightweight gui elements for preprocessing workflow".
Discussed during the meeting this week with @nsteinme @thesamovar @shabnamkadir .
Very related to #212. When, and to what extent, do we want to do these views? Some will be IPython widgets (parameters editor, probably for 0.3.0); some might be matplotlib/vispy WebGL/HTML GUIs (KK2/SD2 monitoring), some might be more full-fledged interactive GUI elements (R-L clustering, obviously KlustaViewa).
We should probably split this into several tasks with several milestones, but we should probably consider what individual modules we'll have. Have we ditched a central "control centre" - NDManager-esque app?

I would recommend to use IPython.widgets for everything and try to not use any manual HTML/JS if possible...

Personally I think the IPython NB version - without a "control panel" - is
just as good as (or better than) having the full blown thing with control
panel. Further GUI elements (like a panel of buttons to run each step, or
some display summarizing the state of all the preprocessing) could probably
be added without too much trouble from there if necessary, anyway, so it
seems like a good idea to skip the control panel altogether for now.
On Fri, May 29, 2015 at 3:20 PM, Cyrille Rossant<EMAIL_ADDRESS>wrote:

I would recommend to use IPython.widgets for everything and try to not use
any manual HTML/JS if possible...
—
Reply to this email directly or view it on GitHub
https://github.com/kwikteam/phy/issues/315#issuecomment-106823987.

OK if Nick finds that it is all fine without a control panel and with a notebook instead, I'm happy to go with this.

I think we can close this for now in favour of the new pure-Python .prm with task-directed dependency graph etc? @rossant

hmm I think that eventually, some users might still like a graphical interface (notebook-based or not), so we can probably keep this open, although with a very low priority!

I think it would still be super useful to have the kinds of "during
preprocessing" visualization that we talked about being part of this, i.e.
see some raw data traces and choose which to exclude, see some detected
spikes and be able to choose different spikedetekt params, perhaps even see
some depiction of the clustering as it goes so you can decide whether it's
done or not. I think that stuff was the core of what we meant here.
On Wed, Sep 30, 2015 at 12:23 PM, Cyrille Rossant<EMAIL_ADDRESS>wrote:

hmm I think that eventually, some users might still like a graphical
interface (notebook-based or not), so we can probably keep this open,
although with a very low priority!
—
Reply to this email directly or view it on GitHub
https://github.com/kwikteam/phy/issues/315#issuecomment-144365702.

@nsteinme I think what you describe is related but slightly different. Having real-time visualization would indeed be useful (and relatively high priority!) and can be done without the GUI stuff this issue is about. In other words, the plan is to let you show custom and interactive visualizations in the notebook by writing code, rather than by being in a dedicated GUI.


Better error handling, deployment params

Disable invasive alert dialog popups
Handle resource manager failures
Clean up process parameter duplication, hardcodes
Support app relocation under a base URL

lgtm @parente

Thanks @ericdill


When I hit a breakpoint vscode opens a new readonly version of the file in Vagrant

VSCode Version: Code 1.18.1 (929bacba01ef658b873545e26034d1a8067445e9, 2017-11-16T18:32:36.023Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

vscode-eslint
dba
1.4.3

debugger-for-chrome
msj
3.5.0

ReactSnippets
xab
1.7.0

Steps to Reproduce:

Use Vagrant (Linux guest on a Windows 10 host)
Debug a project
Hit a breakpoint
The file which hits the breakpoint is open as read only and his path is the vagrant box path /vagrant/myfolder/myfile which is ofcourse not a valid path in the Windows host
The other files (without breakpoints) they have the proper path windows path (C:\dev\myfolder\myfile)
IMPORTANT NOTE: I am using webpack with devtool set to cheap-module-source-map, it might have an impact on this

Reproduces without extensions: Yes

If you are using WSl the you might want to try the "useWSL" flag in your launch config.
See the WSL topic under https://code.visualstudio.com/docs/nodejs/nodejs-debugging#_remote-debugging


Naming convention

A lot of the existing code from "Spire" uses a naming convention where type members are in UpperCamelCase. The new convention is that type declarations are UpperCamelCase while value declarations are in loweCamelCase (with the exception of enum tags for now, which use upper when scoped, and a k prefix when unscoped).
We need to make a pass over the codebase and regularize the convention at some point.
(Consider trying to set up clang-format to help catch issues).

Closing. We have a lot more code using the new convention now, and can clean up other areas as we come to them.


Update HostConfig files to current schema

@riarenas,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:shipit: