Notifications

Thanks @GrahamCampbell for previous fixes.
Hope I've understood all your CS stuff ^^

@Crash-- apply the diff from here and rebase please..

@jbrooksuk In your Contributing file (https://github.com/cachethq/Cachet/blob/master/CONTRIBUTING.md) you specify that braces should be on a new line but the patch put it on the same line :(

Sorry @Crash-- that file needs updating to match what StyleCI does.

I think it's good now.

This definitely doesn't require a service now? It should just do nothing if there isn't a row in the Service table.

We still need a service even if the Service table is empty to add the observer on Incident::saved. But if there is nothing in the table, nothing more will happen.
As @GrahamCampbell tells me to launch the new PR against cachethq:notifications you can't see that directly. But if you take this PR and make a diff with master, you'll see the entire diff (it's much more relevant).

Merged this into the wip brach. I'll take another look at this tomorrow.


Revision 3.0.x

Change the puppet module to use the following:

Use ahuffman's changes to use package cloud repos
Use elastic/elasticsearch model to dynamically create gitlab.rb based on hash. Reducing the number of parameters down to a dozen.

https://packages.gitlab.com/gitlab/gitlab-ce/install

Any chance EE will be supported?

Any chance ee will be supported, now that the access scheme has changed?

Yes, just found out that gitlab finally added gitlab-ee packages to package cloud. I don't have an enterprise license, so I need to research how to implement this (is it a serial number, product key) ect..
Do you know how gitlab ee is activated?

The release blog post:
https://about.gitlab.com/2015/05/22/gitlab-7-11-released/
To install the license, vist /admin/license in your GitLab instance as an
admin. Here you can upload your .gitlab-license file, which will instantly
unlock GitLab Enterprise Edition.
Basically, you upload the license you get from gitlab itself into the app.
On Wed, Jun 3, 2015 at 4:41 PM, Spencer Owen<EMAIL_ADDRESS>wrote:

Yes, just found out that gitlab finally added gitlab-ee packages to
package cloud. I don't have an enterprise license, so I need to research
how to implement this (is it a serial number, product key) ect..
Do you know how gitlab ee is activated?
—
Reply to this email directly or view it on GitHub
https://github.com/spuder/puppet-gitlab/issues/157#issuecomment-108609031
.

--
Michael Chmielewski* |* Practice Lead, Release Engineering
T<PHONE_NUMBER>
--
http://www.bullhorn.com
Twitter https://twitter.com/bullhorn | Linkedin
http://www.linkedin.com/company/bullhorn | Facebook
https://www.facebook.com/Bullhorn
http://www.bullhorn.com/engage/

Have you published a branch for this? I'd love to look at it, and maybe even help here and there.

I haven't pushed a branch yet because of the problem with the packagerepo puppet module. Thanks for letting me know about vshn's module. This does look exactly like what I was wanting to do with branch 3.x

I wouldn't cross your fingers about the packagecloud ever being fixed.
I've been using my forked packagecloud branch since 7.10 with my forked
puppet-gitlab repo and it's been working just fine.  Feel free to pull in,
and/or rework anything that I did there as well as the process to install
the newer gitlab packages.
On Wed, Jul 8, 2015 at 11:15 AM, Spencer Owen<EMAIL_ADDRESS>wrote:

I haven't pushed a branch yet because of the problem with the packagerepo
puppet module. Thanks for letting me know about vshn's module. This does
look exactly like what I was wanting to do with branch 3.x
—
Reply to this email directly or view it on GitHub
https://github.com/spuder/puppet-gitlab/issues/157#issuecomment-119620996
.

--
Regards,
Andrew

Module transitioned to here:  https://github.com/vshn/puppet-gitlab


fuse(2.8.1) is not compatible with cluster (2.9.1)

Alluxio Version:
fuse: 2.8.1
master/worker: 2.9.1
Describe the bug

accessing cluster(2.9.1) data through fuse(2.8.1) will cause the amount of cached data generated to be much larger than the actual size of ufs;
accessing cluster(2.9.1) data through fuse(2.9.1) meets expectations;

Expected behavior

compatibility between minor versions;
even if the fuse version does not match, try not to affect the cache of the entire cluster;

Additional context

size of UFS

access data directly through fuse for a period of time, the amount of cached data is much larger than size of UFS; at this time, there is no error in the master/worker log, and the fuse will prompt that the version does not match(Alluxio CLIENT version (2.8.1) does not match Alluxio cluster version (2.9.1)).

continue to request for a while, and find that the cluster starts to release cache space for the latest access.

@LuQQiu do you happen to know the reason?

@Xenorith and also is there a way to collect historical non-compatibility and generate a doc so the user can reference it? Not only as a part of the release doc

we started using the label https://github.com/Alluxio/alluxio/issues?q=label%3Abreaks-compatibility
does that work for this use case?

Good finding, may need to look into more details to figure out why
Will be more careful in the future fuse stable release for these kind of issues


Missing documentation regarding On-Premise SP2016

Category

[x] Question
[ ] Typo
[ ] Bug
[x] Additional article idea

There is a demand to build solution based on SPFx onPrem.
We are focussing on 2016 but in the near future will go to 2019.
Expected or Desired Behavior
I was not able to find a proper documentation/artcile to create a proper sp2016 on-prem solution.
Apparently there is so many limitation in the versions of Node / SPFx / etc... That I suggest to have a dedicated article point to proper steps.
Observed Behavior
After 3 days of testing many differents solution packaging i still cannot load a proper sppkg on my App Catalog... with the simple hello world in SPFx v1.7.0 using node v8.15.1.
It will be also usefull to provide tips and tricks regarding the simple gulp serve issue on NODE_NO_HTTP2...

Hi @psylion ,
Here is a good documentation https://docs.microsoft.com/en-us/sharepoint/dev/spfx/sharepoint-2016-support

@psylion , you can check the article here: SharePoint-2016-support.md
and SharePoint Framework - SPFx Development With SharePoint 2016

@nanddeepn , @Dennis-LZ , thanks for the links, I already look at those content.
Looking also across issues in git, I saw comment regarding now we should use SPFx 1.7.0 for on premise. Definitely, we should have a link pointing to LATEST compliant versions for OnPrem.
The first link video content dated Sept 2017 (2years ago) and use SPFx 1.0.2
The second link provide steps from December 2018 (10months old) and does not specify any versions.
As soon as I moved on, I have make big steps forwards this morning:

after checking App catalog / auth. provider. / etc...
I finally be able to upload valid sppkg onPrem
Now i'm confused:
for onPrem spppkg creation, can we have a requirement list of versions for:
Node
@microsoft/generator-sharepoint
Cuurently using Node v8.15.1 and the following details from my  .yo-rc.json...
"@microsoft/generator-sharepoint": {
"isCreatingSolution": true,
"environment": "onprem",
"version": "1.7.0",
"libraryName": "default-mg-s",
"libraryId": "db31f07a-1026-4e69-b837-984ad66a8bf0",
"packageManager": "npm",
"componentType": "webpart"

When doing simply :
gulp bundle -- ship and gulp package-solution --ship
I got the following error when WebPart are adde into a page:
***ERROR MESSAGE:
***Failed to load component "a14df85e-8dbf-45fc-9c92-8c911283398a" (HelloWorldWebPart).
Original error: ***Failed to load path dependency "a14df85e-8dbf-45fc-9c92-8c911283398a" from component "HelloWorldWebPart" (Error loading https://component-id.invalid/a14df85e-8dbf-45fc-9c92-8c911283398a_0.0.1/HelloWorldWebPartStrings
Unable to load script /defaultmgs-helloworldwebpartstrings_en-us_536e65149b0acf4d52c0043073b9fc59.js).
Original error: {3}
Certainly I need to change somethings like the manifest or other config file...

Please see the docs on setting up your dev environment: https://docs.microsoft.com/en-us/sharepoint/dev/spfx/set-up-your-development-environment
If you're on SP2016, you must have SP 2016 Feature Pack 2 installed as that includes SPFx.
You should install the latest Yeoman generator for SharePoint (currently, v1.9.1). When you create a project, it will prompt you for the environment you are targetting. You'll pick SP 2016. The project created will use the version of SPFx that your env. supports (SPFx v1.0.1).
There is a note on the above-mentioned about troubleshooting... it addresses your question on HTTP2 & HTTPS.
First, verify your project works in the local workbench (run gulp serve). If that works, then code/project part works.
If after deploying you're having trouble, it's likely an issue with the SP 2016 installation & configuration to support hosting apps.

@psylion said:

I have now the Node v1.16.3

Assume you mean Node.js v10.16.3? If you are running Node
@psylion said:

The gulp serve is running perfectly but I had to modify the serve.json like this:
"https": false,
"initialPage": "http://localhost:5432/workbench",
Now I can run local workbench no issues

You should not have to do this... it doesn't sound like you have trusted the dev cert which was a step in the link I shared above.
@psylion said:

But still when gulp bundle --ship and gulp package-solution --ship created the package and loaded into my app catalog (Valid app package = Yes)
I can add the app on the site after having add into the central admin.
The app is available on the site
I can add the webpart to a page
And then the issue

What "issue"... the one about loading the component? If so, did you deploy the JavaScript bundle & manifest to a public location & configure the project to pull from that location? This page in the docs explains deploying to an Azure CDN, but you don't have to use Azure. It just has to be a public location that people can pull the files from when they load the web part in the page. You will need to configure your project so SharePoint knows where to pull the files from as outlined in this section of the same doc.
@psylion said:

How to configure the solution package to not use CDN and just the local sharepoint where the package is deployed?

Not possible... you said you were on SharePoint 2016. You don't have the option to deploy the assets (JS bundle & manifest) with the SharePoint package. that was added to SPFx v1.4.1 and only available in SharePoint Online.
Your only deployment option is the two-step option:

upload & deploy the *.sppkg to an app catalog, then install in a site
deploy the assets needed for the component to run (minimum, the JS bundle & manifest JSON files.

For the deployment of the assets, they can go anywhere as long as the users of the web part have permissions to pull from that location. They can be in an Azure CDN, AWS CDN, GCP CDN, public file share, SharePoint doc library... it doesn't matter. Just has to be HTTPS accessible by everyone using the page.

Diging into other forums I found a description on how to physically modify the write-manifest.json to enter cdnBasePath  a path pointing to the local SiteAssets where I copied the content of temp\deply folder 1 json and 2 js.
I allowed the JSON extension onto the web applications and then after recreating my package I've been able to make it working.
Are these steps mandatory?
The gulp bundle and pakage-solution does not integrate the json and js files?
Can you point me where this is documented?

@andrewconnell I think you just reply to my questions in the same time i wrote my last comment.
Now its start to be clear and under control.
the only remaining issue is the change of the serve.json... I'm running on Win10 and https was not working
even I followed another thread saying I need to delete the "homeuser"/.gcb-serve-data and recreate the gulp trust-dev-cert
Which I did and I had the Popup to approve ther usage of the certificat, but still not able to run in default mode with https true on port 4321

@psylion there are numerous issues, both open & closed, related to HTTPS & the local workbench in this issue list. Please review those and comment back on all the things you've tried. This is not an uncommon issue & entirely dependent on your machine. Usually on Windows it's resolved by untrusting the cert, then re-running the trust-dev-cert task from an administrative command prompt.
Regardless, these are two different issues and you should open a different issue related to that.

Ok thank you
The initial issue is solved, I'll keep an eye on the #2343

I wouldn't watch a closed issue like the one you reference... once an issue is closed, it's considered "done"... closed issues don't get much if any attention.
In fact, me responding to this issue, now closed, is quite out of the norm :)


[translation] translated checklist in sections for PL

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

I translated the starting-a-project checklist into Polish

Thanks again @olsza!


startScanning with serviceUUIDs, linux issue

Hi, I have a device which sends advertisement with content as follows:
"serviceData":[{"uuid":"180a","data":{"type":"Buffer","data":[5]}}]
I'd like to detect devices which contains only '180a' UUID, so start scanning looks like this:
noble.startScanning(['180a'], true, (err) => { ... });
devices are correctly detected on mac but on linux I have to remove serviceUUIDs do detect anything.
BR,
Krzysztof

@rosek86 what peripheral are you scanning for? It doesn't look time your device is advertising the service, only including service data in the advertisement.
Could you please share the output of sudo hcidump -t -x while your app is running? Also, what does OS X show for the advertisement data?

@sandeepmistry it's property design based on Nordic chip, it sends service data in 0x16 data type [1]. Both iOS and noble on OS X detects that advertisements correctly, only noble on Linux doesn't.
Here is code snippet for iOS which I use to detect peripheral:
let serviceUUIDs:[CBUUID] = [CBUUID(string: "180A")]
centralManager.scanForPeripheralsWithServices(serviceUUIDs, options: options)

{"id":"09e90d452ea149e9950cdcc773506317","address":"dd:35:85:28:3d:3b","addressType":"unknown","connectable":true,"advertisement":{"localName":"TST","manufacturerData":{"type":"Buffer","data":[84,68,1,5,33,84,0,65,3,1,0,0,0,19,0,2,0,1]},"serviceData":[{"uuid":"180a","data":{"type":"Buffer","data":[7]}}],"serviceUuids":[]},"rssi":-56,"state":"disconnected"}
[1] https://www.bluetooth.org/en-us/specification/assigned-numbers/generic-access-profile

@rosek86 I've push a change for this to master (https://github.com/sandeepmistry/noble/commit/4a324910d168467c85cd7ab1876c765facb14c70), you can test it out via npm install sandeepmistry/noble before it is released to npm.
Please let me know if there are any issues.

@sandeepmistry I confirm that your fix works well.
Many thanks

@rosek86 great! Thanks for verifying.


Doesn't support browsers missing secondary file/protocol registrations

The application currently only works if the target browser has the following file and protocol registrations:

.htm
.html
http
https

However, Windows only requires a browser to support http and https.

Fixed at c184c67778aa3b662ea9bc18e155fd759b77fefd.


Mongock is not compatible with the native build

Describe the bug
Mongock is a library to help us to handle the data migration with a MongoDb database.
JHipster already uses it with Mongo and we reused the same with Quarkus.
Mongock uses org.reflections that cause an error when we build it with GraalVM to generate a native executable.
To Reproduce
Steps to reproduce the behavior:
Generate a project with MongoDb as a database.
Install GraalVM
openjdk 11.0.10 2021-01-19
OpenJDK Runtime Environment GraalVM CE <IP_ADDRESS> (build 11.0.10+8-jvmci-21.0-b06)
OpenJDK 64-Bit Server VM GraalVM CE <IP_ADDRESS> (build 11.0.10+8-jvmci-21.0-b06, mixed mode, sharing)

Run the following command: ./mvnw clean verify -Pnative -DskipTests
Caused by: com.oracle.graal.pointsto.constraints.UnsupportedFeatureException: Unsupported features in 2 methods
Detailed message:
Error: com.oracle.graal.pointsto.constraints.UnresolvedElementException: Discovered unresolved type during parsing: org.apache.commons.vfs2.VFS. To diagnose the issue you can use the --allow-incomplete-classpath option. The missing type is then reported at run time when it is accessed the first time.
Trace:
	at parsing org.reflections.vfs.Vfs$DefaultUrlTypes$7.createDir(Vfs.java:287)
Call path from entry point to org.reflections.vfs.Vfs$DefaultUrlTypes$7.createDir(URL):
	at org.reflections.vfs.Vfs$DefaultUrlTypes$7.createDir(Vfs.java:287)
	at org.reflections.vfs.Vfs.fromURL(Vfs.java:98)
	at org.reflections.vfs.Vfs.fromURL(Vfs.java:90)
	at org.reflections.Reflections.scan(Reflections.java:236)
	at org.reflections.Reflections$1.run(Reflections.java:199)
	at java.lang.Thread.run(Thread.java:834)
	at com.oracle.svm.core.thread.JavaThreads.threadStartRoutine(JavaThreads.java:519)
	at com.oracle.svm.core.posix.thread.PosixJavaThreads.pthreadStartRoutine(PosixJavaThreads.java:192)
	at com.oracle.svm.core.code.IsolateEnterStub.PosixJavaThreads_pthreadStartRoutine_e1f4a8c0039f8337338252cd8734f63a79b5e3df(generated:0)
Error: com.oracle.graal.pointsto.constraints.UnresolvedElementException: Discovered unresolved type during parsing: org.apache.commons.vfs2.VFS. To diagnose the issue you can use the --allow-incomplete-classpath option. The missing type is then reported at run time when it is accessed the first time.
Trace:
	at parsing org.reflections.vfs.Vfs$DefaultUrlTypes$7.matches(Vfs.java:281)
Call path from entry point to org.reflections.vfs.Vfs$DefaultUrlTypes$7.matches(URL):
	at org.reflections.vfs.Vfs$DefaultUrlTypes$7.matches(Vfs.java:281)
	at org.reflections.vfs.Vfs.fromURL(Vfs.java:97)
	at org.reflections.vfs.Vfs.fromURL(Vfs.java:90)
	at org.reflections.Reflections.scan(Reflections.java:236)
	at org.reflections.Reflections$1.run(Reflections.java:199)
	at java.lang.Thread.run(Thread.java:834)
	at com.oracle.svm.core.thread.JavaThreads.threadStartRoutine(JavaThreads.java:519)
	at com.oracle.svm.core.posix.thread.PosixJavaThreads.pthreadStartRoutine(PosixJavaThreads.java:192)
	at com.oracle.svm.core.code.IsolateEnterStub.PosixJavaThreads_pthreadStartRoutine_e1f4a8c0039f8337338252cd8734f63a79b5e3df(generated:0)

	at com.oracle.graal.pointsto.constraints.UnsupportedFeatures.report(UnsupportedFeatures.java:129)
	at com.oracle.svm.hosted.NativeImageGenerator.runPointsToAnalysis(NativeImageGenerator.java:770)
	... 8 more

Expected behavior
The native build should succeed without errors.
Suggestions
Move from Mongock to Liquibase with Mongo.
Find a way to make Mongock compatible with the native build.
Additional context
Use ./mvnw dependency:tree -Dverbose to check which library is using VFS.
Check this issue: https://github.com/quarkusio/quarkus/issues/9801
cc @danielpetisme

closed by #180

I have just realised about this ticket and it seems that has been somehow sorted out.
However is there anything that we can provide natively from Mongock to help on this or improve.
Just let me know.


Create CoD (Bacalhau) Dev Tools Page for Launchpad Colo v8

Describe the use case
Have a session informing residents about Network Indexer during colo week
Describe the problem
We would like to have reference materials in order to enable residents to quickly learn about CoD, resources available, and try a hands-on tutorial for the Jan 30, 2023 Launchpad
Proposed Solution

[x] Create a new branch and PR for a CoD page in Dev Tools
[x] Add in some baseline content/ summary content about what Cod (Bacalhau) is and how it's tied to PL use a lot of content from here
[ ] Add Iryna to launchpad repo & reach out to get review from her
[ ] Add screenshots, directions, and stickies for the stable diffusion tutorial to the miro for the v8 colo
[ ] Add video & pre-requisite instructions for the tutorial, e.g. install python or Docker container

Resources

Cohort v8 Miro
Build with Bacalhau Video that goes over the tutorial we are doing during colo

PR is here: https://github.com/protocol/launchpad/pull/479/files

Enol still needs to

[ ] Try out & confirm tutorial
[ ] Add dependencies to message for cohort
[ ] Add instructions/ screenshots of tutorial to miro board and add instructions for what to turn in

Done!


Design timeline component

@xx13 check out some items on wrapbootstrap.com, i think that timeline would be a nice feature to have

@xx13 please add the designs to zeplin, nice


en.wiki.bpe.op50000.model

where can I  get the model  of en.wiki.bpe.op50000.model

You can use the en.wiki.bpe.vs50000.model which is available online as it is just an update to the old models.


1kLoC Enhancement: Improve find

@Happytreat

Alright, I will take note of the use of present tense for future commits.
Regarding the time complexity:

By using a hashmap to represent and store a person's details. The put and get operation should be O(1) if keys are even distributed which may not always be the case.

Since the program has fixed number of details about a person, there would not be a need to add different details to an array avoiding the O(n) scenario of adding into an already full array.

So in conclusion the change from String[] to Hashmap may slow the program. Is this true?
Thanks!
@Happytreat

Remember to close this PR after reading it. :)


Added linkedlist concept

@pankaj-bind , i have raised this issue, and @AnshDwivedi03 has created a PR without even being assigned by you.
Please resolve close this PR as its duplicate, and assigned my issue to me #180

@Shariq2003  first of all raising issue is not the big deal, secondly you should work fast and raise ur pr .., you are too slow kindly work on ur ability

@Shariq2003 first of all raising issue is not the big deal, secondly you should work fast and raise ur pr .., you are too slow kindly work on ur ability

What 😂, bro, this is open source in which there are multiple projects, and currently I am working on multiple projects.
Raising a PR without even being assigned by anyone is not an ability. You just found an issue without doing any research on the GitHub repo and just raised a PR for it and called this the so-called Ability.
If this is an ability then sorry I am not able like you.

@Shariq2003 @AnshDwivedi03  Guys I have been working of Linked List, from the beginning and that Linked List folder was created by me, and I am still working on it.
What I have been doing is that, I am creating an issue, and waiting for the admin to assign it to me before creating a pull request, following the contributing guidelines. So I request you to find some other issue or raise your own rather than this one and about Linked List as I am working on it.

@Upendra48 With due respect.., I know many are working on it But the point is who is faster , Creating folder is not too big thing . You raise your pr if admin approve it's good. This is a race and the one who works fast always win

@AnshDwivedi03
Kindly read contribution guidelines,
Thank you

@AnshDwivedi03 Kindly read contribution guidelines, Thank you

@Upendra48 don't waste your time in explaining @AnshDwivedi03 about how to contribute and read contributing.md.
This huy is insane.
Lightning fast.

@Shariq2003 Lol.., you are assigned and haven't raised pr till now.. Knowledge <<<
I raised pr within an hr. Knowledge >>>.
Kids

@AnshDwivedi03

If No assignees means Shariq is assigned then I am blind and fortunately u got the super powers

Do not raise PR until I assign you.


Confused by the documentation for radios 'conditional'

I was trying to use the conditional option on radios.
I saw that the example used
conditional:{
    html:"<p> ...

I didn't need html, just a sentence so I thought I could use text instead of html as you can in other places in Frontend. However, I tried it and it didn't work - there's no text option here.
When I re-read the docs, I misread and thought the 'conditional' option, being a string, could just accept text.
However, I misread it, that doesn't work - the docs actually say:

conditional | string | If true, content provided will be revealed when the item is checked.

But I don't think this is correct? In the examples, conditional is not a string, it's an object with only one property - html
I wonder if this might be better as two options:
conditionalHTML and conditionalText
This would avoid another object, and be more consistent with the use of HTML and text elsewhere?

@joelanman Can you provide a link to that in the docs, as that looks like it needs correcting? It should read:
conditional | boolean | If true, content provided will be revealed when the item is checked.
conditional.html | string | Provide content for the conditional reveal.
``` (which I can see on https://design-system.service.gov.uk/components/checkboxes/)

Good point about being able to provide a `text` option though, that definitely seems to better match what we do elsewhere.

It was radios I was looking at:
https://design-system.service.gov.uk/components/radios/
I would say even boolean is a bit confusing, as the examples don't use a boolean, they use an object

For now we've agreed just to remove the conditional entry from the macro options for radios and checkboxes, which will make it consistent to the way the accordion works and is documented (apart from not supporting conditional.text).

The documentation around this is still a bit off as it still suggests setting conditional with a text value would work, which it definitely doesn't judging by the code below:
https://github.com/alphagov/govuk-frontend/blob/master/src/govuk/components/radios/template.njk#L14-L19
Documentation reference: https://design-system.service.gov.uk/components/radios/

The macro options for radios and checkboxes still display conditional entries. Do we need to remove these entries?

just come across this again in my work - conditional is still listed as a string option

thanks!


Request for Python 3.10 Support for Arm64

tensorflow-macos now support Python3.10 for arm64, so could you please help us to add Python 3.10 support for tensorflow-addons for arm64?

I have a tutorial here that you can build yourself, but you can also download it directly from the releases. I hope to help you. 😊

Thank you for your reply. This is great!  If you are okay, could you add this release to PyPi so that we can install it using pip?

First, I'm glad to help you. At the same time, I'm also trying to build the M1 package of io and decision_forests recently. If it helps you, welcome your star and fork. As for submitting pip I'm sorry, because it's not my repo so I can't do this.

Gotcha. Thank you for helping with this. It is very useful. Since we are moving forward to Python 3.10, it would be very helpful if we can get official support.

TFA 0.17.1 released with support for py3.10 on macos arm64

TFA 0.17.1 released with support for py3.10 on macos arm64

@seanpmorgan That's great news! Thank you for updating this!


feat: add delete button

I think it would be helpful to have a delete button on the issue table. That way, users can easily remove any issues they no longer need or want to track. It could be positioned in the select status like so:

Feel free to test and review

It's probably best to have this feature I agree


Call to undefined method CodeIgniter\Database\MySQLi\Builder::findAll()

Hi.  after installing and configuring the app on a live site all seems ok, but on the admin page after i enter login details i have the following error:
Call to undefined method CodeIgniter\Database\MySQLi\Builder::findAll()
location: VENDORPATH/basic-app/crud/BaseIndexAction.php at line 89

A quick fix for this;
https://github.com/basic-app/crud/blob/master/BaseIndexAction.php#L22
$query = $model;


Customizing doesn't work

I'm on Firefox.
It generates the zip and then nothing happens. Apparently with & without arm(64) selected.

Please share the config of the selected options (click on "Save config" and paste the resulting json in a code block here). Also check Ctrl+J or your downloads folder that it might not just have generated a download in the background.
And what exactly do you mean with "nothing happens"? What did you expect to happen other than generating the zip file?

Ok thanks. It happens to me with every single config I tried. Never occurred to me it could be joml

Looking into the F12 / developer tools always helps when diagnosing web page issues. :)

joml-1.9.15.jar was missing from the build server.
I've uploaded both joml-1.9.15.jar and joml-1.9.15-sources.jar but I'm not sure where I'm supposed to find joml-1.9.15-javadoc.jar...
Pinging @Spasi

If in doubt, then repo.maven.apache.org is always right: https://repo.maven.apache.org/maven2/org/joml/joml/1.9.15/
But I am very sure that the download did work at one point with 1.9.15, as I immediately verified it myself when the PR was deployed.

Thanks, I've updated the javadoc jar as well. Everything should be working now.
I've opened an issue for downloading addons directly from Maven. It's a good idea.
https://github.com/LWJGL/lwjgl3-www/issues/68


[3.x] Classref typo fixes and improvements

Mostly fixes of typos in recent update. Also updated some necessary markups.

Thanks!


LSP Improvements

Go to definition
"Run" button (code lens)
Problems (diagnostics)

It kinda obvious that we need this so will close for now


Add Lainey's new legal update to "Additional Reading"

Issue #886

Thanks for the contribution @okeul !


fix deployment repository test failures on windows

Without this changes certain windows systems would get this failures:
ContentRepositoryTest.testChangeExplodedContent:228 » ExplodedContent WFLYDR00...
ContentRepositoryTest.testExplodeContent:133 » ExplodedContent WFLYDR0018: Err...
ContentRepositoryTest.testExplodeSubContent:155 » ExplodedContent WFLYDR0018: ...
ContentRepositoryTest.testListContents:279 » ExplodedContent WFLYDR0018: Error...
DeletionCollisionTest.testFileLockByReadContent:77 » ExplodedContent WFLYDR002...
DeletionCollisionTest.testFileLockByRemoveContent:100 » ExplodedContent WFLYDR...
PathUtilTest.testReadFile:141 » AccessDenied target\temp\pathutil\unarchive148...

Which is result of "wrongly" setting permissions when creating temporary directory.

@ehsavoie please review

+1


Add the Default option to the LLVM switch case for robustness when the device arch string is invalid.

Add the Default option to the LLVM switch case for robustness if the device arch string is invalid.

Hi, sorry I was on holiday last week. The default case was intentionally left out because the if it's needed it's a sign there is a problem earlier in the pipeline. The PR looks like it's fixing a symptom of a problem, not the problem itself.
I thought that the there would be an assert if none of the cases matched, is that wrong?

Hi, sorry I was on holiday last week. The default case was intentionally left out because the if it's needed it's a sign there is a problem earlier in the pipeline. The PR looks like it's fixing a symptom of a problem, not the problem itself. I thought that the there would be an assert if none of the cases matched, is that wrong?

This device arch attribute has been replaced by the minSubSize. It is no longer an issue here.

Hi, sorry I was on holiday last week. The default case was intentionally left out because the if it's needed it's a sign there is a problem earlier in the pipeline. The PR looks like it's fixing a symptom of a problem, not the problem itself. I thought that the there would be an assert if none of the cases matched, is that wrong?

This device arch attribute has been replaced by the minSubSize. It is no longer an issue here.

should the related code be removed then?


fix: keep the tab order

SUMMARY

When export all tabs to a PDF in report & alert, user noticed that the all tabs export doesn't seem to export the tabs in any specific order (our report with 3 tabs exported with tab 2, then 3, then 1 in the PDF)
reference: https://github.com/apache/superset/pull/30694#issuecomment-2455854080
Context
The exported tabs are now ordered based on the sequence of anchors under the all_tabs field from the tabs endpoint response. This enhancement aligns the export order with the order displayed on the dashboard.
BEFORE PDF

before.pdf
AFTER PDF
after.pdf
TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[X] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

FYI @yousoph

LGTM, but @eschutho might be better poised to actually test this before approving/merging.


What is the corresponding version of prerequisites for cu10.0?

📚 Installation

I noticed that there are cpu, cu92, cu101, cu102, or cu110 for torch-geometric Installation. However, my cuda is 10.0 version, and I tried cu101, and it failed. I am wondering if there is the version  for cu10.0, thankyou for your kind help if there is any reply on my question.
Environment

OS: linux
Python version:  3.6
PyTorch version: 1.7
CUDA/cuDNN version: 10.0
GCC version:
How did you try to install PyTorch Geometric and its extensions (wheel, source):
Any other relevant information:

Checklist

[ ] I followed the installation guide.
[ ] I cannot find my error message in the FAQ.
[ ] I set up CUDA correctly and can compile CUDA code via nvcc.
[ ] I do have multiple CUDA versions on my machine.

Additional context

We follow PyTorch here and provide pre-built versions that match with the one PyTorch is providing (for example, PyTorch dropped cu100 installation in PyTorch 1.6.0). I suggest to install the CUDA version that matches the one of PyTorch:
import torch
print(torch.version.cuda)

Note that you may need to add the miniconda3 cudatoolkit to $LD_LIBRARY_PATH. Mine is located in ~/.miniconda3/lib.

We follow PyTorch here and provide pre-built versions that match with the one PyTorch is providing (for example, PyTorch dropped cu100 installation in PyTorch 1.6.0). I suggest to install the CUDA version that matches the one of PyTorch:
import torch
print(torch.version.cuda)

Note that you may need to add the miniconda3 cudatoolkit to $LD_LIBRARY_PATH. Mine is located in ~/.miniconda3/lib.

Thankyou for your reply, I will try to change my cuda version to match pytorch.


LoadError: Double registration for method

With this configuration I am getting following the error at module loading time:
ulia> using Geant4
[ Info: Precompiling Geant4 [559df036-b7a0-42fd-85df-7d5dd9d70f44]
Warning: Type 8G4String already had a mapped type set as G4StringAllocated using hash<PHONE_NUMBER> and const-ref indicator 0
ERROR: LoadError: Double registration for method (:function, :copy, :Base, 0x9df832de6d2b232b)

Somehow CxxWrap thinks the method or type is defined twice, but I do not see it in the generated code.
Any idea of this this could be? When I tried yesterday it was somehow working.

Hello Pere,
I cannot reproduce the issue,  because it fails before  with the following error:
/home/opt/lib/libG4geometry.so: cannot allocate memory in static TLS block
Philippe.

I guess the problem is a missing compilation/linking flag such as
  CXXFLAGS += -ftls-model=initial-exec -pthread

But the easiest is to use CMake to build the wrapper library instead of a hand-made Makefile. This is what I am using currently CMakeLists.txt

More info. The problem appears when I add G4Box.hh and G4UBox.hh to the list of headers. This explains why I was not getting this error yesterday.
input               = [ "Geant4Wrap.h",
                        "G4RunManager.hh",
                        "G4NistManager.hh",
                        "G4LogicalVolume.hh",
                        "G4PVPlacement.hh",
                        "G4Box.hh",
                        "G4UBox.hh" ]

Hello Pere,
Neither adding the compilation flag or using cmake to build fixed the issue I had. Nevertheless, I've managed to circumvent it by setting the environment variable LD_PRELOAD  as:
LD_PRELOAD=/home/opt/lib/libG4geometry.so:/home/opt/lib/libG4digits_hits.so:/home/opt/lib/libG4processes.so

Coming to your issue, the problem comes from a typedef/alias in G4VSolid.hh:
using G4GeometryType = G4String;

Typedefs are not handled properly by wrapit. We still need to define how to do it. Current code make a wrapper for the typedefs, but CxxWrap does not accept to have more than one wrapper for a type or one of its aliases. So it works only if the type is used uniformly with the same alias.
Two option to solve your issue, until I implement a clean handling of typedefs:

Disabling typedef wrapping by  commenting out the line #define WRAP_TYPEDEFS in src/CodeTree.cpp;
or adding G4GeometryType in your veto file.

After fixing this, I have an error message about a CLHEP class. I expect this can be solved by adding a CLHEP header in the extra_headers but I've not investigated it more.
Philippe.

Thanks for looking into it. I did check the diff of the generated wrapper code between working and not working I didn't realize that the culprit was  the wrapper for G4GeometryType.
What is the logic of generating the wrapper for the typesdefs if at the end is replaced by the actual type. Put it in another way, if a have a method/function returning a  G4GeometryType, it will actually return a G4String for which we have the wrapper and CxxWrap knows about it. So, the typedef should be complete invisible, isn't it?

The function wrapper will actually return a G4GeometryType .
One solution would indeed be to resolve all typedefs to the underlying type. The drawbacks are that, first, the underlying type can be complex, in particular for templated  types (take as example std::string), second, the function signature of the Julia binding will differ from the original one.

I do not think is true. In my case it returns a G4String or subclass of it.
box = G4Box("Box1", 1,2,3)

@test typeof(GetEntityType(box)) == Geant4.G4StringAllocated
@test GetEntityType(box) == "G4Box"

@grasph What is needed for typedefs (at least in my use case) is the following:

Do not generate the wrapper for the typedef since it does not make sense because is an empty type with no methods.  I have achieved this by adding the type to the veto file
Make sure that the real types are selected by adding their header files as input to wrapit
Add in the Julia module the following lines:  const G4RotationMatrix = CLHEP!HepRotation
  const G4ThreeVector = CLHEP!Hep3Vector

  export G4ThreeVector, G4RotationMatrix

I have done this step by hand, but it could be done with wrapit I guess.

With these changes I am able to create G4ThreeVectors and operate on them.
julia> box = G4Box("Box1", 1,2,3)
Geant4.G4BoxAllocated(Ptr{Nothing} @0x00007f9d136199f0)
julia> m1, m2 = G4ThreeVector(), G4ThreeVector()
(Geant4.CLHEP!Hep3VectorAllocated(Ptr{Nothing} @0x00007f9d0d4f6ca0), Geant4.CLHEP!Hep3VectorAllocated(Ptr{Nothing} @0x00007f9d10d27ea0))
julia> BoundingLimits(box, m1, m2)
julia> @test m1 == G4ThreeVector(-1,-2,-3)
Test Passed
julia> @test m2 == G4ThreeVector(1,2,3)
Test Passed
julia> @test m1 + m2 == G4ThreeVector()
Test Passed

OK. Thanks for the feedback. The typedef is useful for the ex002-ROOT example. It's a special case. It's for FILE which is a typedef to a struct which is declared in the system header, but not defined
I agree with you that wrapper to typedef  should not be produced and another approach should be taken for the FILE use case  (or it should be produced only in absence of wrapper for the underlying type).
The Julia aliases are needed for convenience only. You could also use  Geant4.CLHEP!Hep3Vector. Am I correct?
Philippe.

The Julia aliases are needed for convenience only. You could also use  Geant4.CLHEP!Hep3Vector. Am I correct?

Yes, you can use Geant4.CLHEP!Hep3Vector but will diverge from the C++ way of naming and showing the implementation choices.

Workaround is working fine

Some followup:

This commit solves the issue with removes the typedef wrapper generation while solving use cases that depended on it.
Automatic generation of julia type alias to reflect c++  typedef/using statements is put on the TODO list. The list contains items with higher priority, like proper support  of templates and splitting the generated code in smaller source files.

@peremato above commit should fix the issue with CLHEP!HepRotation!IIDENTITY.


Python wrapping and Protocol Decode

[x] add GitHub actions for new features
[x] add sync patch
[x] update README

Closing this as we're going with Glasgow for protocol analysis.


ui settings page

move the setting page to the harvester UI(i.e. similar to the namespace page).
change the user logo to avatar style(please refer to the rancher UI).
update setting's drop style and make it consistent with the current UI design.

close in favor of https://github.com/rancher/harvester-ui/pull/82


Using default behavior instead of elses

I just removed elses because there was no need for them for being there since were just default behaviors

Please also squash your commits down to just one

@fntlnz thanks for the pr but it looks like some of the changes also change behavior.  Could you update this and reopen the pr after the changes are made and commits squashed?  Thanks!


HTML 5 Video Tag Controls Not Working in Full Screen (Android)

Hi everyone,
Am trying to load video in my ionic app using html 5 video tag . It works fine in normal screen but when i switch to full screen mode, the video controls are displayed but unable to click . This happens on Android so far (works fine on browser), I haven’t tested the app on iOS.
This is the simple html code for the video display. Any help would be so appreciated, thank you.
<div class="video-container">
<video src="http://www.sample-videos.com/video/mp4/720/big_buck_bunny_720p_1mb.mp4" controls width="100%" poster="https://previews.123rf.com/images/andreus/andreus1409/andreus140900437/31970710-colored-trash-bins-used-to-recycle-paper-plastic-and-glass-digital-illustration--Stock-Photo.jpg">Thiết Bị Của Bạn Không Hỗ Trợ HTML5 videos.</video>
</div

Are you using tabs, or hamburger side menu, or any other kind of overlay?
I was facing similar issue and video controls were not receiving click event.
setting display:none for tabs and other overlays, when going full screen resolved the issue.

Comment the line contain:strict; in node_modules/ionic-angular/components/app.scss :
.app-root,
.ion-page {
// contain: strict;
}
And recompile;
Cheers!


Clean up files API

The current CUPS files API still returns integer values for success/fail, sometimes 0 for true and -1 for false and sometimes 1 for true and 0 for false. Update to use bool types.

[master b44a87037] Clean up cupsFile API (Issue #15)


WDL parsing check for recursive imports catches cases that are not recursive

See https://github.com/gevro/gatk4-exome-analysis-pipeline-flat
If you try and register it with the new dockstore WDL 1.0 parsing code, it will say there might be recursive imports. I checked the import structure and this is not true, it is a flaw with our code.
Our current WDL parsing code cannot parse these 1.0 files either, so users cannot publish their valid 1.0 workflows.
From
https://discuss.dockstore.org/t/unable-to-get-publish-button-to-become-active/1972
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.7
┆Sprint: Seabright Sprint 17 Raft
┆Issue Number: DOCK-911

@agduncan94 taking a look, let me know if you have code for this already

@denis-yuen I have the code, it is in https://github.com/dockstore/dockstore/tree/feature/2766/recursive-imports
I still have to write tests, was going to use the one from the discourse discussion.

Ah, too late

I can now register https://raw.githubusercontent.com/gevro/gatk4-exome-analysis-pipeline-flat/master/ExomeGermlineSingleSample.wdl


PPAify flask_talisman

My understanding is that current SRCF policy encourages installing Python dependencies from Ubuntu packages.
control-panel depends on flask_talisman which is currently installed on pip.srcf via pip - it would be convenient if this were in a PPA please
(See also #4)

Done: https://launchpad.net/~malcscott/+archive/ubuntu/flask-talisman


kafka support

Kafka is a message broker, https://kafka.apache.org/
it would be great if dbeaver would support basic kafka management

create/modify/delete topic
insert a message into a topic
show messages from topic
delete a message from a topic

Sounds interesting. We don't have messaging/queue datasources yet. This might be the first one.
And we can support KSQL as a kind of SQL dialect.
We could use Java APIs for integration (https://kafka.apache.org/documentation/#api)

About kafka https://habr.com/ru/post/354486/

would be cool to have SASL auth onboard

Kafka is no a relational or document or key-value storage, it doesn't have query language and it is not completely clear how to support it in DBeaver. What kind of UI / data management interface should we use?

this could be helpful https://github.com/mmolimar/ksql-jdbc-driver/blob/master/README.md

@p91paul interesting, thanks!

I am afraid, this driver is for KSQL Engine, which is a separate product and unfortunately can not be included in to DBeaver.

I am afraid, this driver is for KSQL Engine, which is a separate product and unfortunately can not be included in to DBeaver.

Yes, KSQL is a separate product but it is a product that talks to Kafka with SQL-like semantics. In this case, this makes much more sense to include KSQL. In fact, I cannot think of any other way to use DBeaver to access Apache Kafka.

It'll a very useful feature to interact with near-realtime data pipelines. 😃

Please support the schema registry coming with kafka out of the box - What essentiall happens is that the "kafka" is scanned for the right datasets when a query is run. Also a "live" view would be nice.

Looks like something similar already exists, but not free: https://www.cdata.com/kb/tech/kafka-jdbc-dbvr.rst

Just curious, will this be available in both Community Edition and Lite Edition?


feat: Import changes from #1123 for Key Vault

Description
I have migrated changes from 1123 into my own feature branch to close and completes the related tasks to 1123

adds tests for ec and rsa kty key parameters
adds checks for dependencies between keySize and curveName key parameters
Expanded UDT for Secrets
Exapanded UDT for Keys
added support for releasePolicy for key
resolves the conflicts from #1123

Pipeline Reference

Pipeline

Type of Change

[x] Update to CI Environment or utlities (Non-module effecting changes)
[ ] Azure Verified Module updates:

[ ] Bugfix containing backwards compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[x] Breaking changes and I have bumped the MAJOR version in version.json.
[ ] Update to documentation

Checklist

[ ] I'm sure there are no other open Pull Requests for the same update/change (#1123 IS OPEN AND SHOULD BE DISCARDED)
[x] I have run Set-AVMModule locally to generate the supporting module files.
[x] My corresponding pipelines / checks run clean and green without any errors or warnings

@fblix - if you get chance can you approve this if you get time. If we don't have a repsonse by the end of the week then the core team will approve this.


Fix radius calculation for AxisAlignedBoundingBox

Is this really a fix or more code bautification?
Either way, we should :shipit:

Both, as the previously used glm::vec3::length() actually returns the number of components of the vector, i.e. 3. Correct would be glm::length(someVec), or (even better here) glm::distance.

Nice error (which I would somehow blame glm for) :laughing:


Can download the image on Mac using Safari

Can easily download the image on a Mac running macOS Monterey (V 12.0.1) using Safari Browser (V15.1).

Same on iOS 15

lol


[WIP] Add screen

This is in a somewhat workable state, I'm going to merge and then continue with smaller PRs.


URI in Icon Style not validating

Hello,
I am currently running into an issue when comparing the GeoJson schema (http://cmapi.org/versions/v1.2.0/channels/cmapi.appendix.b.js) to my JSON, specifically the iconStyle url. I am not sure how to provide a type of "uri". As of now I am trying to feed it a url of type string (e.g. http://cmapi.org/img/Fotolia_56211233_S_crop.jpg) but the tv4.validate fails. Was wondering if I am missing something or if I am doing something wrong?
Thanks,
Rodney

The schema in the spec should have identified the icon style as string, and format as uri.  As we looked further into this, we decided that for v1.3 we would have style as string and not constrain the format to uri because that would require absolute URIs vice relative paths.  We think that relative paths should be allowed.

Corrected schema in v1.2 to icon style = string.


[GIE-DB] Add GIE DB impl

Add Cypher end-to-end test for HighQPS Engine.
Introduce GIE-DB prod(interactive).

Codecov Report

Merging #3030 (3cdfc68) into main (c1df6f2) will increase coverage by 0.02%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3030      +/-   ##
==========================================
+ Coverage   42.62%   42.65%   +0.02%
==========================================
  Files          99       99
  Lines       10685    10688       +3
==========================================
+ Hits         4555     4559       +4
+ Misses       6130     6129       -1

see 6 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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

We need issue #3061 to be resolved first, before this PR is merged.


Retry callback bug

Hello! I found a bug with a retry callback. The module do only one call of given callback, after that it uses a default callback.
Here is a bugged code:
var shouldRetry = extra.retries > 0 && await extra.retryEvaluator(err);
to fix it we just have to change extra to options:
var shouldRetry = extra.retries > 0 && await options.retryEvaluator(err);

Thank you, I will fix that. :)

Thank you, I will fix that. :)

You are welcome and thank you for useful module!

Sorry but when you release this fix on Pub?

var shouldRetry = extra.retries > 0 && await options.retryEvaluator(err);
where to change this

i have tried but the function api is nhi been
retries

on 404 it only retries only once

I fixed this in my branch by manually applying the PR. Here's how to use it in your project. You can use my patched version by adding this to your pubspec.yaml instead of the existing dio_retry package.
dio_retry:
git:
url: https://github.com/lewcianci/dio_retry.git
path: dio_retry

I also removed the dependency on logger in my fork. I prefer a more generic callback.

@aloisdeniel any idea when you can get this fixed? Would be awesome to have it on prod


Defining Directory Structure

Defined a tentative directory structure.

Yeah, hence the word tentative. I've removed the 'Important Files' folder and added 'Data' folder back. Closing for now.


Fix arlo intilization when no base station available

Description:
Currently arlo component always check if base station is included in PyArlo return object. However, in case user does not allow full control of base station, or only cameras are available, arlo initialization will fail. This diff should fix it.
Related issue (if applicable): #15026
Cases tested:

When both base station and cameras are available, Arlo plugin should work.
When only cameras are available (no base station return), Arlo cameras should still work.
When no cameras or base station are returned, an error message No camera or base station available. was shown in the log.

Hi @zellux,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

CI has passed (https://travis-ci.org/home-assistant/home-assistant/builds/426783190) but status has not been updated on GitHub yet, is it a known issue?


Browser compatibility

Testing the compatibility across browsers (copying code from test.html)
Test: https://kiwicopple.github.io/postgres.run/compatibility.html
Source: https://github.com/kiwicopple/postgres.run/blob/main/compatibility.html

[x] Chrome ✅
[x] Arc  ✅
[x] Safari  ✅
[x] Firefox ❌: TypeError: window.indexedDB.databases is not a function
[x] Mullvad ❌: TypeError: window.indexedDB.databases is not a function
[x] Edge ¯_(ツ)_/¯

Offending line:
https://github.com/electric-sql/pglite/blob/33267d433ced6a828ad7c381a67dfb13e9317101/packages/pglite/src/idbfs.ts#L14
Relevant links

SO: Is there is any function in firefox to get all indexedDB names?

Firefox support added in v0.0.2

nice work here @samwillis


I have the ability to watch any stream without paying btc

I messaged iwantathink on reddit about the issue. I'll be vague here so it isn't abused until it's fixed.

Thanks! It will be dealt with as soon as possible.


[red-knot] @override lint rule

Summary
Lots of TODOs and things to clean up here, but it demonstrates the working lint rule.
There's also some unused code added here; I wrote iter_parent_scopes and associated iterator support and then decided not to actually use it in this PR, but I'm confident we will want it in future so I'm leaving it in.
One thing to note is that the choice to use a trait object for the db in semantic lint context (rather than making it generic over the db) forces exposing everything as query functions on the db interface. Maybe this is a good thing?
Test Plan
➜ cat main.py
from typing import override
from base import B

class C(B):
    @override
    def method(self): pass

➜ cat base.py
class B: pass

➜ cat typing.py
def override(func):
    return func

(We provide our own typing.py since we don't have typeshed vendored or type stub support yet.)
➜ ./target/debug/red_knot main.py
...
1   0.012086s TRACE red_knot Main Loop: Tick
[crates/red_knot/src/main.rs:157:21] diagnostics = [
    "Method C.method is decorated with `typing.override` but does not override any base class method",
]

If we add def method(self): pass to class B in base.py and run red_knot again, there is no lint error.

Thanks for the review!

SemanticLintContext probably must be generic over Db to prevent that we need to leak the internal storage layout on the Database API. We should solve this as part of this PR.

Why does using a trait object db require us to leak the internal storage layout on the database API? I think it led to that in this PR just because I was cutting corners to get things working, but I don't think it requires it. Instead it requires that all data access we need available to lint rules must be implemented as queries on the Db trait interface. These queries don't need to leak implementation details, but it will mean there will be a lot of them.
In some ways it seems like making everything generic over Db has more risk of leaking internals, because it means that the full Db interface is always available instead of just what we expose on the trait.
(I don't have strong opinions here, I'm still working out the pros and cons of different patterns and trying to understand the reasoning for different options.)

My ultimate goal was to prevent that lint rules get access to db and instead force them to go through the context. I can see now how this is challenging with e.g. FunctionTypeId::name taking a db as argument. I still think that not giving lint rules access to the db is desired, but I'm not aware of a good solution right now.

Yes, I think this will be pretty challenging with the interning-based storage approach, because pretty much anything you might want to do requires a db.

This is related to not giving access to db to the semantic lint rules. I think that it's very easy today to mix local ids from different files. Ideally, lint rules would only ever get local ids for the current file or otherwise global ids.

Yeah, I agree we should try to avoid this footgun, though it's not clear to me how to actually make it impossible. (Even a global ID will contain local IDs, and making those inaccessible will make those global IDs quite difficult or impossible to use in the code that actually needs to break them apart to do the queries. Maybe we can address this with crate visibility boundaries, if in future we break lint rules into a separate crate from the db.)

Addressed most review comments. Would like to discuss this more, as the right path isn't clear to me:

SemanticLintContext probably must be generic over Db to prevent that we need to leak the internal storage layout on the Database API. We should solve this as part of this PR.


Missing guide articles on site, while they exist in the repo.

Combine Multiple Reducers or Use Middleware to Handle Asynchronous Actions have stub articles online, but the proper articles are available in the git repo.
Same is also true for other stubs.

They will be available in our next release. We are in the process of making them as a editable wiki. We will update as we progress is made. Thanks for your patience.


Add limelight like hyperfocus to zen mode

For even less destruction in zen mode, it would be great to have only the current code section highlighted. Similar to the limelight plugin in vim. Here is an example:

Any chance we can get a "hyperfocus zen mode" in future versions of VSCode?

This is a fair feature request, however we do not have it on our immediate plan. Thus assigning to backlog and we might add this in the future. Thanks

I've been looking for the same thing for months now. It would be great to have this feature
+1

For anyone still looking for an extension that can do the same thing this might do it for you.

This feature increases productivity, I think this is a must-have


Azure v2.0 issuer compatibility

I'm having a few issues attempting to use Azure as a oidc server.
As documented in #108, discovery validation fails on the issuer and jwt keys can't be downloaded due to CORS, but I think both can be avoided (by manually specifying config, and exposing/proxying keys through our own back-end.)
However, there is one problem I don't have a workaround for. The Microsoft endpoint at https://login.microsoftonline.com/common/v2.0 can be used to authenticate any Azure tenant, but the tokens it provides have the issuer set to https://login.microsoftonline.com/{GUID}/v2.0
I'd be happy to submit a PR to make this work, but should it:

Support a custom issuer validation function/interface?
Have a flag to allow any issuer with the same domain name?
Have a flag to disable issuer validation completely? (Relying on user's of the library to separately validate.)

I think 2 is the simplest (and best) option, but does that make sense to you?
For reference, here are the azure docs: https://docs.microsoft.com/en-us/azure/active-directory/develop/active-directory-v2-protocols-oidc

I'm facing the exact same issue. There definitely should be a way to make the validation more lax. I would also vote for the second option.

Sorry for the late reply. I was busy at some conferences.
I think, you could just set the following flag:
https://manfredsteyer.github.io/angular-oauth2-oidc/angular-oauth2-oidc/docs/classes/AuthConfig.html#strictDiscoveryDocumentValidation
What do you think?

Hi Manfred,
That flag isn't enough to make it work, the processIdToken function fails with a 'wrong issuer' error.
That flag would be useful/necessary for discovery document validation, but Microsoft don't support CORS on their jwt endpoint so it discovery won't work properly anyway.
I've actually decided to use my .Net back-end to authenticate to the various oidc servers, hopefully simplifying the angular side and removing the need for any changes here.
Cheers,
Matt

@manfredsteyer I just tried to put the strictDiscoveryDocumentValidation to false and it doesn't change anything. I still get the following error message:
invalid issuer in discovery document expected: https://login.microsoftonline.com/tfp/test.onmicrosoft.com/B2C_1_SignUp/v2.0 current: https://login.microsoftonline.com/339e6f86-7de6-4a7f-89d7-4bb50b002291/v2.0/

having similar issue on my local machine with or without setting: strictDiscoveryDocumentValidation;

I have just fixed it by suffixing with a slash at the end, is that right?
    **issuer: window.location.origin + '/',**

@asadsahi Great that this works for you now.
@Gimly I think it's a bit more difficult for Azure AD b/c them "violate" the OIDC here. strictDiscoveryDocumentValidation only activates/ deactivates additional checks that are best practices.
The next version that lands in a few days will have a config flag skipIssuerCheck. By setting it to true, this check should go away.
What do you think @Gimly, do you want to contribute a sample that shows how to use Azure AD? Perhaps you already have such a proof of concept.

@manfredsteyer Unfortunately I don't have a proof of concept using this library anymore. I got frustrated about Azure AD not respecting OIDC, so I switched to using their own library directly. It's pretty annoying as their library won't work if I decide to switch to another identity provider, but it works.
If I have a bit of time I'll try to put something together, it shouldn't be that much work since I already did a version for msal.js. But I can't promise it will be fast 😄.


Shopify theme doesn't know the path to cloak-app plugin

@weotch check this out. On the shopify theme it's trying to look in shopify-theme/plugins rather than go out to the package to get the initialize plugin to initially set the customer

Can you send me a link to the repo and branch with shopify-theme/plugins/vuex.coffee?  I wanna make sure I'm looking in the same place.  The first thing I was gonna check was that @cloak-app/customer was a dependency of shopify-theme

https://gitlab.com/dripdrop-hydration/dripdrop-pro/-/blob/main/shopify-theme/plugins/vuex.coffee

I fixed this in #15 with this:
https://github.com/BKWLD/cloak-customer/blob/ecfedf84c20ce7881a3b2bececf0ffd7e7052b25/plugins/initialize.coffee#L4
And this:
https://github.com/BKWLD/cloak-customer/blob/ecfedf84c20ce7881a3b2bececf0ffd7e7052b25/nuxt.js#L43-L45
In other words, I'm passing the path wherever this package ends up to the plugin and using Nuxt's template-compiling behavior to fix the path.  I'm doing the same thing in @cloak-app/boilerplate, btw, which is where I came up with this workaround.

I found a better pattern than ☝ .  Check out
https://github.com/BKWLD/cloak-customer/blob/3aab33fd7741d7e8734672d261ed5f7cd77b005f/nuxt.js#L36-L39
I'm not using the this.addPlugin so our plugin doesn't route through Nuxt module's template system.  Then our imports can be simple again:
https://github.com/BKWLD/cloak-customer/blob/3aab33fd7741d7e8734672d261ed5f7cd77b005f/plugins/initialize.coffee#L3-L4


⚠ AGEG website (prod) has degraded performance

In 1b1bfb1, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2407 ms

Resolved: AGEG website (prod) performance has improved in 0a96846.


C#: Add taint-tracking steps through conversion operator calls

I wonder if we should treat the case where we have the source code of the operator differently? If we have the source code, then we could do regular taint tracking, whereas if we don't have the source code, then we conservatively assume that all operator calls preserve taint.

Good idea. Let's restrict this to library conversions.


STUNNED

Hello rye team (@mitsuhiko et al)
I just started using rye and have been STUNNED at how well it works! It even works well on windows -- windows doesn't even work well on windows.
Really and truly excellent.

Don't think this should be an issue? Should go into discussions instead 🤔


[backporting for 3.3.x] Read properties first! #723

Backport of #723
(cherry picked from commit ba0c9fbccb05b6aacf09f0b0dbfe3e1db671929a)
Conflicts:
src/test/java/org/apache/ibatis/builder/CustomizedSettingsMapperConfig.xml

Thank you, @ikeyat !


Mac Compatibility Issue - "Can't Find Project or Library"

Using version 4.1.2, I keep getting the following error when I run my program on a Mac (I've tried two different systems, one 32 and one 64)
"Compile Error: Can't Find Project or Library"
The error is triggered by these lines (and possibly more, I didn't do an exhaustive review):

Public AsyncRequests as Dictionary
Private web_pConverters as Dictionary
Public function ParseUrlEncoded(Encoded as String) As Dictionary

Any idea what could be causing these errors?
Thank you so much for creating this program, it has saved countless hours at a small non-profit!

@ethanlowens I see you closed this issue. Did you find a workaround? I'm running into a similar issue coming out of the WebHelpers classes, specifically in:
============================================= '
' 3. Url handling
' ============================================= '
''
' Join Url with /
'
' @example
' VB.net ' Debug.Print WebHelpers.JoinUrl("a/", "/b") ' Debug.Print WebHelpers.JoinUrl("a", "b") ' Debug.Print WebHelpers.JoinUrl("a/", "b") ' Debug.Print WebHelpers.JoinUrl("a", "/b") ' -> a/b '
'
' @param {String} LeftSide
' @param {String} RightSide
' @return {String} Joined url
''
Public Function JoinUrl(LeftSide As String, RightSide As String) As String
If Left(RightSide, 1) = "/" Then
RightSide = Right(RightSide, Len(RightSide) - 1)
End If
If Right(LeftSide, 1) = "/" Then
LeftSide = Left(LeftSide, Len(LeftSide) - 1)
End If
If LeftSide <> "" And RightSide <> "" Then
    JoinUrl = LeftSide & "/" & RightSide
Else
    JoinUrl = LeftSide & RightSide
End If

End Function

FYI - when I looked at the references in the VBA editor there was a package was showing as "MISSING" in the list. When I unchecked it, my code ran wihtout errors.


chore(deps): update terraform terraform-ibm-modules/cos/ibm to v8.2.11

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cos/ibm (source)
module
patch
8.2.10 -> 8.2.11

Release Notes

terraform-ibm-modules/terraform-ibm-cos (terraform-ibm-modules/cos/ibm)
v8.2.11
Compare Source
Bug Fixes

fix bug related to missing KMS auth policy (#621) (a395e83)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 7.18.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Remove a deleted quests

There are a case where a quest is deleted when a new day, week or month has started.

155de8573d2be12987d06207e1fafa93651593e0


Windows machines need more than 100GB of disk space

After playbook install, the amount of space used by various directories on the machines are as follows (Checked on one of the new test-ibmcloud-win2012r2 machines. This means there is barely 10GB of disk space for our processes to use and with the new 10GB minimum free disk space check we are hitting this limit. We should ensure that all Windows machines have more than this available - absolute minimum of 150Gb and ideally 250GB+. Build machines will likely require more.

cygwin_packages 2Gb
cygwin64 11.7Gb (11.2Gb in files)
Program Files 4.4GB
Program Files (x86) 36.5GB (36.2GB in files)
Windows 24.7Gb

Total of the above = 79.3Gb excluding other smaller directories on the machine
Breakdown of Program Files (x86) excluding a few directories which are under 1Gb:

LLVM 0.8Gb
Microsoft SDKs 1.3Gb
Microsoft Visual Studio 12.5Gb
Microsoft Visual Studio 9.0 0Gb
Microsoft Visual Studio 10.0 1.8Gb
Microsoft Visual Studio 11.0 0.9Gb
Microsoft Visual Studio 12.0 2.7Gb
Microsoft Visual Studio 14.0 2.2Gb
Windows Kits 11.5Gb
WiX Toolset v3.14 0.1Gb

Request to increase/reprovision new ibmcloud machines which have 100Gb has been sent to @rajdeepsingh1

Sounds like adding a second disk to the system may be the easier option, so I'm happy with that - if we can get an extra disk to be used as the jenkins workspace drive that would meet the needs (subject to other stuff on the machines not expanding too much in the future!)

With the new Cygwin install method, ( #1596 ), it should be substantially lower. Those machines were setup with the cache install. I'm more than happy to remove that and do the new method?
Also, ref: #1594

What is the size saving? But yes certainly happy for that to be done with the four new windows systems we're setting up

After the new installation : cygwin64 is at 1.1G on the new install, cygwin_packages are at 167MB, so a good ~12GB saving

Extra space will hopefully be available later today

@sxa @Willsparker
I've added a 500GB disk to win2012r2-x86-1
This has been assigned as drive G: on the system. The Jenkins workspace is running on a 50G F: disk.
Please review and advise - I believe you would like the F: disk to be on the new 500G since it is the remoteFS: for Jenkins.

Once this system is good to go I can add the additional 500G to the remaining 5 machines.

Hi @rajdeepsingh1 - can you check if that's been applied to one of the AdoptOpenJDK systems as I can't seem to see the extra disk on them.

@sxa @Willsparker
The following machines should now have a 500G drive mounted and ready to go:
bld-ibm-w2012x1.adoptopenjdk.net
bld-ibm-w2012x2.adoptopenjdk.net
tst-ibm-w2012x1.adoptopenjdk.net
tst-ibm-w2012x2.adoptopenjdk.net

@Willsparker Can you verify that t Jenkins user can write to the new drives then we can adjust the Jenkins definitions to point the workspace onto it

@Willsparker @rajdeepsingh1 Good news - I've been able to run a sanity.openjdk run on a test machine and a JDK15/J9 build on the build box without issues. JDK8 failed due to an inconsistency in the JDK7 boot dir on the machine and how it was defined in jenkins which I've now resolved (permanant fix here)  The build process does seem to use C:\workspace explicitly as part of the process but as long as that has enough space for one build (which it does) then that should be ok. We should definitely look at why the bulid fell back to trying to use --with-boot-jdk=C:openjdkjdk-8 as a default when JDK7 wasn't in the expected place and resolve that.
Copying @smlambert as there are new test-ibmcloud-win2012r2 machines in place now to replace test-softlayer-win2012r2 ones in case they show up anywhere any problems

@sxa Awesome, thanks :-)


GLFW Throws platform unsupported on linux

Summary
When i'm trying to create new window i get this Error:
System.PlatformNotSupportedException: GLFW is not supported on this platform.
at Silk.NET.Windowing.Glfw.GlfwPlatform.ThrowUnsupported()
at Silk.NET.Windowing.Glfw.GlfwPlatform.CreateWindow(WindowOptions options)
at Silk.NET.Windowing.Window.Create(WindowOptions options)
i have called Registerplatform for GlfwWindowing and GlfwInput before creating new window
Steps to reproduce

Platform: Desktop - Fedora 37 Workstation (Wayland Windowing System)
Framework Version: .NET 7.0
API: OpenGL
API Version: OpenGL 4.6
1.Call platform registers for GlfwWindow and GlfwInput
2.Create new window
3.Run Window

Not sure if  this is a bug or Silk.NET just simply don't support glfw windowing on linux with wayland windowing system.

You need to install GLFW using your system package manager, which seems to be fedora, so you can use yum, and install glfw with yum glfw

installed and still same issue


Problem with setup on macOS Sierra

I followed the directions on how to install on macOS Sierra and Unreal Editor doesn't seem to recognize the Plugins folder nor attempt to build them.
o New MacBook Pro 2016
o Xcode 6, 7 and 8 are all on the system
o Followed the directions for a blank C++ project and it does compile and run
Any ideas?

can you paste your current project directory structure ? by the way we will start releasing binary packages even for osx soon

All looks good, so basically your project starts without prompting you with the plugin build dialog ?

Correct. And when I look in the log, I remember the first time I tried it said that it was reading UnrealEnginePython.uplugin. Every time I open the project after that, I don't see that message.

I believe Python 3.5 is installed correctly also:
(flask)dhess@Tilsit-3:[~/Desktop]$ ls -al /Library/Frameworks/Python.framework/Versions/3.5/
total 12192
drwxrwxr-x   9 root  admin      306 Dec 15 20:50 .
drwxrwxr-x   3 root  wheel      102 Jun 26 12:49 ..
lrwxr-xr-x   1 root  admin       18 Dec 15 20:50 Headers -> include/python3.5m
-r-xrwxr-x   1 root  admin  6235360 Jun 26 12:48 Python
drwxrwxr-x   5 root  admin      170 Jun 26 12:49 Resources
drwxrwxr-x  21 root  admin      714 Dec 15 20:51 bin
drwxrwxr-x   3 root  admin      102 Jun 26 12:49 include
drwxrwxr-x  14 root  admin      476 Dec 15 20:50 lib
drwxrwxr-x   4 root  admin      136 Jun 26 12:49 share
(flask)dhess@Tilsit-3:[~/Desktop]$

The Epic Games Launcher is giving me Unreal Engine 4.14.1.

Well, just tried it again and this time I see this in the log:
LogPluginManager: Loaded Plugin UnrealEnginePython, From /Users/dhess/Documents/Unreal Projects/MyProject2/Plugins/UnrealEnginePython/UnrealEnginePython.uplugin

But the dialog box never pops up and nothing gets built.

Ok, every time I went to the Plugins panel, it never listed the project level plugin. This time it finally did and I was able to enable it and then restart – and it gave me the dialog to build the plugins.
So I think there was something wrong on my system and it would help to include in the directions the need to go into the Plugins panel and find and enable the plugin at the project level.

Other than the documentation addition to add a step to enable the plugin, I think this was mainly pilot error.


RewardsEstRequest missing height params

Currently in osmosisd,
osmosisd query incentives rewards-estimation
allow to query by block height
--height int        Use a specific height to query state at (this can error if the node is pruning state)
But it's missing in RewardsEstRequest
export interface RewardsEstRequest {     owner: string;     lockIds: Long[];     endEpoch: Long; }

nvm

Hey @tandn37 same issue here did you find a solution ?

Hey @tandn37 same issue here did you find a solution ?

sorry I dont remember about this at all, may be osmosisd version mismatch.


Include ZFS in installer

Needed for ZFS vault, HV=kubevirt

I assume this needs to be backported to 13.4-stable, so I added the stable label.

Ok, once merging I'll submit the cherry-pick back port.

The Eden tests do go through the onboarding process and it's useful to sanity check that. I thought there were tests running the installer image as well, but clearly not with zfs since then we wouldn't have had this regression.

there were tests running the installer image as well
wow, I didn't know... I have at least to find what are the test cases that we run))

All green! Merging!
@andrewd-zededa, waiting for backporting PR(s)


LNK2019: unresolved external symbol Scintilla_RegisterClasses in wWinMain

After adding the missing folder TotalPE2\scintilla\win32 from the downloaded scintilla source, LNK2019 error occurred.

win32 folder in scintilla is not pushed into git repository, can you please add it?
https://scintilla-interest.narkive.com/XUpyrWbT/compiling-and-using-scintilla-as-static-library-in-my-app
"On Windows, Scintilla is normally used as a dynamic
library as a .DLL file. If you want to link Scintilla
directly into your application .EXE or .DLL file, then
the STATIC_BUILD preprocessor symbol should be defined
and Scintilla_RegisterClasses called. STATIC_BUILD
prevents compiling the DllMain function which will
conflict with any DllMain defined in your code.
Scintilla_RegisterClasses takes the HINSTANCE of your
application and ensures that the "Scintilla" window
class is registered. To make sure that the right
pointing arrow cursor used in the margin is displayed
by Scintilla add the scintilla/win32/Margin.cur file
to your application's resources with the ID IDC_MARGIN
which is defined in scintilla/win32/platfromRes.h as
400."
I guess the original win32 folder vcporject setting is not for STATIC_BUILD...

Which externals are missing?

win32 folder in scintilla is not pushed into git repository, can you please add it?
https://scintilla-interest.narkive.com/XUpyrWbT/compiling-and-using-scintilla-as-static-library-in-my-app
"On Windows, Scintilla is normally used as a dynamic
library as a .DLL file. If you want to link Scintilla
directly into your application .EXE or .DLL file, then
the STATIC_BUILD preprocessor symbol should be defined
and Scintilla_RegisterClasses called. STATIC_BUILD
prevents compiling the DllMain function which will
conflict with any DllMain defined in your code.
Scintilla_RegisterClasses takes the HINSTANCE of your
application and ensures that the "Scintilla" window
class is registered. To make sure that the right
pointing arrow cursor used in the margin is displayed
by Scintilla add the scintilla/win32/Margin.cur file
to your application's resources with the ID IDC_MARGIN
which is defined in scintilla/win32/platfromRes.h as
400."
I guess the original win32 folder vcporject setting is not for STATIC_BUILD...

Done.
It seems a rule in .gitignore rejected "Win32"...


Add trait obligations for where clauses when calling functions/methods

E.g. if we call foo<T: Into<u32>>(x), that adds an obligation that x: Into<u32>, etc., which sometimes allows type inference to make further progress.

bors r+


moloch make failed at "http.c:370: error: ‘G_SOURCE_REMOVE’ undeclared"

Make failed in moloch compile process. Have someone any idea on what wrong here.
Error:
Makefile:219: warning: overriding commands for target capture/plugins/Makefile' Makefile:217: warning: ignoring old commands for target capture/plugins/Makefile'
Making all in capture
make[1]: Entering directory /root/SETUP/moloch/capture' gcc -fPIC -O2 -ggdb -Wall -Wextra -D_GNU_SOURCE -c main.c db.c nids.c yara.c http.c config.c parsers.c plugins.c field.c trie.c writers.c writer-inplace.c writer-disk.c writer-null.c \ 	    -I/usr/include/pcap \ 	    -Ithirdparty -pthread -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include   http.c: In function ‘moloch_http_curlm_timer_callback’: http.c:370: error: ‘G_SOURCE_REMOVE’ undeclared (first use in this function) http.c:370: error: (Each undeclared identifier is reported only once http.c:370: error: for each function it appears in.) http.c: In function ‘moloch_http_send’: http.c:545: error: ‘CURLMOPT_MAX_HOST_CONNECTIONS’ undeclared (first use in this function) http.c:558: error: ‘CURLOPT_CLOSESOCKETFUNCTION’ undeclared (first use in this function) http.c:558: warning: type defaults to ‘int’ in declaration of ‘_curl_opt’ http.c:559: error: ‘CURLOPT_CLOSESOCKETDATA’ undeclared (first use in this function) http.c:559: warning: type defaults to ‘int’ in declaration of ‘_curl_opt’ writer-disk.c: In function ‘writer_disk_init’: writer-disk.c:421: warning: implicit declaration of function ‘g_thread_new’ make[1]: *** [all] Error 1 make[1]: Leaving directory /root/SETUP/moloch/capture'
make: *** [all-recursive] Error 1

I would guess your glib2 is too old, it was added in 2.32

Looks like your version of curl ibrary is too old also, it was added in 7.21.7


Remove extraneous curly brace from cc-mode fori

Should be for (1; 2; 3) {...} instead of for (1; 2; 3}) {...}.

Thanks for your help!


Terminal not restoring properly.

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.67.0
OS Version: Linux x64 5.17.5-76051705-generic

Steps to Reproduce:

https://user-images.githubusercontent.com/98533939/167239165-a7c0cc8c-dbc3-4afd-88d2-f99c328a2e4d.mp4

Created a PR https://github.com/microsoft/vscode/pull/149621 with a fix


feat(apps/whale-api): add updatemasternode indexer

What this PR does / why we need it:
Update masternode details in whale when masternode have updated
Which issue(s) does this PR fixes?:
Fixes part of https://github.com/JellyfishSDK/jellyfish/issues/1842
Additional comments?:

docs/node/CATEGORIES/11-masternode.md applied in https://github.com/JellyfishSDK/jellyfish/pull/1921


statusCode

$client = new \GuzzleHttp\Client();
$url = 'https://www.toutiao.com/i655557862281563150/';
try{
$response = $client->get($url);
$a = $response->getBody()->getContents();
var_dump($a);
} catch (\GuzzleHttp\Exception\ClientException $exception) {
$responseBody = $exception->getResponse()->getBody()->getContents();
//$responseBody = $exception->getResponse()->getBody(true);
$code = $exception->getCode();
var_dump($responseBody);
}
The status code returned by the open page is 404, but the status code returned by the code is 200? How do I get the 404 status code?

Guzzle doesn't support HTTP2:
$ curl -i https://www.toutiao.com/i655557862281563150/
HTTP/2 404
server: Tengine
content-type: text/html; charset=utf-8
content-length: 1146
date: Mon, 28 May 2018 16:47:39 GMT
vary: Accept-Encoding
vary: Accept-Encoding
x-tt-timestamp:<PHONE_NUMBER>.893
via: cache37.l2ot7[157,0], cache1.us1[200,0]
timing-allow-origin: *
eagleid: 4266ff0115275260597314323e

$ curl --http1.1 -i https://www.toutiao.com/i655557862281563150/
HTTP/1.1 200 OK
Server: Tengine
Content-Type: text/html; charset=utf-8
Content-Length: 361
Connection: keep-alive
Date: Mon, 28 May 2018 16:48:17 GMT
Vary: Accept-Encoding
Vary: Accept-Encoding
Vary: Accept-Encoding
Vary: Accept-Encoding
X-TT-TIMESTAMP:<PHONE_NUMBER>.426
Via: cache19.l2ot7[170,0], cache1.us1[181,0]
Timing-Allow-Origin: *
EagleId: 4266ff0115275260972842521e

think you

thanks


RELEASE-2.1.x allinone fails with undefined variable follow

trying to spin up an allinone env for testing purposes by following the instructions for the allinone vagrant VM, and getting this error:
...
2016-04-06 09:47:19,505 p=1680 u=nathwill |  TASK: [client | migrate neutron services script] ******************************
2016-04-06 09:47:19,505 p=1680 u=nathwill |  Wednesday 06 April 2016  09:47:19 -0700 (0:00:00.087)       0:02:52.425 *******
2016-04-06 09:47:19,992 p=1680 u=nathwill |  ok: [allinone]
2016-04-06 09:47:19,992 p=1680 u=nathwill |  NOTIFIED: [client | update ca certs] ******************************************
2016-04-06 09:47:19,992 p=1680 u=nathwill |  Wednesday 06 April 2016  09:47:19 -0700 (0:00:00.486)       0:02:52.912 *******
2016-04-06 09:47:20,716 p=1680 u=nathwill |  changed: [allinone]
2016-04-06 09:47:20,717 p=1680 u=nathwill |  PLAY [keystone code and config] ***********************************************
2016-04-06 09:47:20,717 p=1680 u=nathwill |  Wednesday 06 April 2016  09:47:20 -0700 (0:00:00.724)       0:02:53.637 *******
2016-04-06 09:47:20,717 p=1680 u=nathwill |  ===============================================================================
2016-04-06 09:47:20,718 p=1680 u=nathwill |  TASK: [logging-config | logstash-forwarder config directory] ******************
2016-04-06 09:47:20,718 p=1680 u=nathwill |  Wednesday 06 April 2016  09:47:20 -0700 (0:00:00.000)       0:02:53.638 *******
2016-04-06 09:47:21,196 p=1680 u=nathwill |  ok: [allinone]
2016-04-06 09:47:21,197 p=1680 u=nathwill |  TASK: [logging-config | install log template] *********************************
2016-04-06 09:47:21,197 p=1680 u=nathwill |  Wednesday 06 April 2016  09:47:21 -0700 (0:00:00.478)       0:02:54.116 *******
2016-04-06 09:47:23,974 p=1680 u=nathwill |  fatal: [allinone] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'follow'", 'failed': True}
2016-04-06 09:47:23,974 p=1680 u=nathwill |  fatal: [allinone] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'follow'", 'failed': True}
2016-04-06 09:47:23,974 p=1680 u=nathwill |  FATAL: all hosts have already failed -- aborting

any suggestions for how to fix?

I am also getting this problem. ill look into it

awesome, thanks folks!


Help please with landmark coordinates

I have tried your pretrained model for hand tracking on my laptop with tflite.
Using hand_landmark_3d.tflite I was able to get response with 93 float values, but I don't know how to interpret them. If I guess correctly it should be like this [x, y, z, x, y, z, x, y, z, ......]
In another issues I found that instead x, y, there is u, v coordinates. Can someone help me figure out how to map this coordinates back to image. Or share some links where I can learn something how to work with this coordinates.
Here is example of output
[[ 5.73803955e+02  1.13868750e+03 -5.46218395e-01  5.12194763e+02
5.26353577e+02 -1.75876827e+01  3.95716187e+02  3.61917419e+02
-6.36454277e+01  2.47399094e+02  4.43307892e+02 -1.11231621e+02
3.95803040e+02  2.40306549e+02 -1.26765953e+02  3.38830536e+02
-2.76079865e+02  2.41141205e+01  2.83156219e+02 -1.70654221e+02
-6.85159779e+00  2.28728485e+02 -2.76015808e+02  4.23018265e+01
3.32072327e+02 -4.60349854e+02  7.72872162e+01  4.22068481e+02
5.75553589e+01 -5.78513641e+01  6.83203506e+01  2.89094452e+02
-2.13282196e+02  1.12702522e+02  4.30184937e+02 -3.22525116e+02
4.39910492e+02  9.63115768e+01 -3.96457275e+02  4.50602661e+02
3.00958435e+02 -1.23822266e+02  6.66966431e+02  4.33050293e+02
-2.13000671e+02  7.11620422e+02  5.44539368e+02  2.39527969e+02
5.37973145e+02  5.62555664e+02  5.77881592e+02  3.24635773e+02
3.63354736e+02 -2.22293304e+02  5.68385254e+02  6.90362366e+02
-2.63504822e+02  6.28268127e+02  5.53707825e+02 -3.67202225e+01
6.08915894e+02  3.19211914e+02  2.04959793e+02]]

hey did you work in desktop? How did you manage to extract this array of coordinates???

what did you do? @Michael89

Code for extracting coordinates on desktop: https://gist.github.com/eknight7/d4a57504c8f866fc80c0eb2c61ff6b4f.

Code for extracting coordinates on desktop: https://gist.github.com/eknight7/d4a57504c8f866fc80c0eb2c61ff6b4f.

I don't understand what PreProcess/ PostProcess on images/keypoints was performed (i am writing a simple python inference script).
Could you please help me with this?

I don't understand what PreProcess/ PostProcess on images/keypoints was performed (i am writing a simple python inference script).
Could you please help me with this?

Sorry for necrocomment.
I can feel your pain, case mp is not very friendly with newcommers (especially with only scripting languages background).
So if you'd like to get landmark pozition in python, you should do some things:
output landmarks are normalized, and returned in class NormalizedLandmarkList (which is list :) ). So you can access landmark (which is NormalizedLandmark class) by it index, as usual list.
If so, you get something like this:
x: 0.3427850008010864
y: 0.5966814756393433
z: 0.04402058944106102

It is protobuf message with normalized x,y and z coord. As I know there is somewhere methods like .x() or .y() to get corresponding value, but not in python. To access x or y you should use any protobuf parser (the best one is google.protobuf.json_format IMHO). So just parse message to dict (or json, as you wish)(https://googleapis.dev/python/protobuf/latest/google/protobuf/json_format.html) and you get dict with coordinates for landmark.
As I think to get corresponding pixel on image, you can just multiply x on width and y on height of image in pixels, because as I understand normalization is get by [0,1] with input image (but i'm not shure I do not need this, just test it).


Fix some more things found by fuzzing

Leaving this as a draft because I want to collect more potential fixes from further fuzzing... Although currently both of my fuzzing runs are blocked on #1611.

Coverage increased (+0.003%) to 70.968% when pulling 74d90cb7f179b9d212a8c155e27d6e2f55e65f40 on YaLTeR:fix-more-things-2 into cc170c2a17675aae6fdcfd12ecc97eaeba90b974 on xiph:master.

This probably should be revisited now, the tests are still good though.

Ready for review as nothing else was found.


Manage cloud account

We need the cloud account management pages.
As a subscriber or as a service provider (and also as admin) I need to be able to manage the cloud accounts of my organization.
Add, delete, edit, list them
I should be able to have a page where I can list the cloud accounts (maybe in the home page of the user?).

This issue depends on https://github.com/CloudOpting/cloudopting-manager/issues/96

Cloud Account managing is working.
Only missing the selector for cloud account providers.

Can we try to close this asap?


An unsupported batteryPowerLevel: access was provided

I’ve encountered a log entry indicating an unsupported batteryPowerLevel value after connecting to a 2025 Volkswagen ID.4 Pro.
Here is the log message:

2024-11-02T06:21:51+0000:WARNING:addressable:/vehicles/{VIN}/domains/readiness/readinessStatus/connectionState/batteryPowerLevel: An unsupported batteryPowerLevel: access was provided, known values are [comfort, emergency, off, unknown battery power level] please report this as a bug

Version info: vwsfriend 0.24.7 (using WeConnect-python 0.60.5, WeConnect-mqtt 0.49.2)

I've also been having a similar issue - it went away after charging the car but has since returned:
2024-11-04T14:00:30+0000:WARNING:addressable:/vehicles/{VIN}/domains/readiness/readinessStatus/connectionState/batteryPowerLevel: An unsupported batteryPowerLevel: location was provided, known values are [comfort, emergency, off, unknown battery power level] please report this as a bug
Also running VWsFriend 0.24.7

Thank you for reporting, this will be added in the next version


hotfix: THR_MDL_FAC parameter: bump minor version and default to disabled

flight tested in stabilize and acro modes on QAV250

Thanks, sorry for missing that!


Update TopologyGraph.from_openff_topology for new API

PR Summary:
The Topology class in the OpenFF toolkit is undergoing a heavy refactor, including removing some API  points that Foyer uses. The refactor will not be in a release for some months, but we are actively testing downstream packages by installing against a feature branch. The parts of the refactor that are relevant here are unlikely to change significantly, but that's not yet guaranteed.
This PR also makes an attempt to support both APIs.
PR Checklist

[x] Includes appropriate unit test(s)
[ ] Appropriate docstring(s) are added/updated
[x] Code is (approximately) PEP8 compliant (no longer burned on the dev, thanks to pre-commit!)
[ ] Issue(s) raised/addressed?

Hey folks,
The new version of the OpenFF Toolkit has been released, so I'm able to run these tests without pulling in RC/beta builds. I'm not sure why CI isn't running - is it because I'm pushing to a fork?

Tests appear to be fail-fast and failing because we (OpenFF) no longer package on Python 3.7. I opened #513 to possibly mitigate this.

Agree with changing the element arg to symbol, gonna do a quick commit to change the var name

Thanks!


KeyError

Hi, iget the following error;
Traceback (most recent call last):
File "Train_NER.py", line 95, in
model.setDataset(datasets, data)
File "/home/hanah/Documents/Gitstuff/elmo-bilstm-cnn-crf-v02/neuralnets/ELMoBiLSTM.py", line 86, in setDataset
self.idx2Labels[modelName] = {v: k for k, v in self.mappings[labelKey].items()}
KeyError: 'NER_BIO'
I set the parameters like this;
{'columns': {0:'tokens', 4:'NER_BIO'}
and my data looks like this

EP1764360_0095 5566 5567 I-M I-M
pyridin-2-yl)-carbamic EP1764360_0095 5567 5589 I-M I-M
acid EP1764360_0095 5590 5594 I-M I-M
tert EP1764360_0095 5595 5599 I-M I-M

Could you paste your complete definition of the datasets  variable?

0: tokens, 1: document ID, 2: start offset, 3: End offset, 4: BIO encoding, 5: IOBES encoding

I mean the datasets variable in your python script.
See https://github.com/UKPLab/elmo-bilstm-cnn-crf/blob/master/Train_Chunking.py
line 44

datasets = {
'Biosemantics':                                   #Name of the dataset
{'columns': {0:'tokens', 4:'NER_BIO'},    #CoNLL format for the input data. Column 1 contains tokens, column 2 contains NER information using BIO encoding
'label': 'NER_IOBES',                      #Which column we like to predict
'evaluate': True,                        #Should we evaluate on this task? Set true always for single task setups
'commentSymbol': None}

Could it be that the CoNLL file is not split correctly? Check that when the CoNLL file is read, that each token as the columns 'tokens' and 'NER_BIO'.
Maybe individual lines are malformatted, which causes that not enough columns are extracted which then causes the keyerror.

It works fine when I select column 5 for IOBES encoding, but throws error with column 4 for BIO encoding. If the lines were malformatted then it would have not worked with column 5.

Maybe you can send me you train python script and the dataset? The I can have a look<EMAIL_ADDRESS>

Hi @nreimers
I have tried sending email multiple times at the given email, it doesn't work and message is undeliverable.

Then try<EMAIL_ADDRESS>Please try to not send too large files, they might get rejected from the mail server. If your dataset is too large, upload it somewhere and send me the link

sent


Update pandas dependency and pandas-stubs version 1.4.3.

Update the Pandas version and pandas-stubs release version number in pyproject.toml.

[ ] Closes #62
[ ] Tests added: None

I'm going to close this, because #66 takes care of this by not requiring any version of pandas to be installed when you install the distributed version of the stubs.


Remove Device from Topic based on previous categories

This work is base on AGPUSH-1632 ticket.
This is my first PR for UPS server and code might not be up to best standards so I am looking forward to your comments.
Ping @matzew @secondsun

This works most of the time, however if the device is subscribe to topics which UPS doesn't know about then the device will still get older messages.
TO demonstrate :
Use this Android app : https://github.com/secondsun/unified-push-helloworld/tree/unsubscribe_topics
Start up an unpatched UPS instance
Setup your push app and replace the push-config in the android project
Start the android project
After you register select all of the topics from the options menu one at a time (there are three, sorry about the white on white)
You will notice that UPS thinks you are only subscribed to one topic, but you will receive messages for all topics.
Now patch the server and rerun the app.
You will reregister with 0 topics and UPS will unsubscribe you from the one topic it knew about.  You will still be subscribed to tow topics however.
You can work around this by querying the topics from the Google servers and updating based on THAT list and not on the list from the previous installation.
Hope this helps.

Thank you for review this guys.
Nice catch @secondsun, I'll fix it as you suggested - by querying the topics from Google server and then unsubscribing.

Now it queries Instance ID service for subscribed topics and after that unregisteres from those that are no longer relevant.
Do you have suggestions on Exception handling here (instead of just logger.fine(..)) ?

Can we have some unit tests?

I'll add unit tests as soon as possible.


Add a config option to override the prefix in SD chat messages

This PR adds a config option that provides a setting to override the entire prefix for messages posted by SD in chat. This would allow one to add any link as a prefix (not just to the GitHub repo), but a user could also set the prefix to something else as well.

I changed the one to a config option, I’ll change the other ones there to config options as well later today

I've converted the other two options to the config file. I also tested this change here, and it seems to work as expected.

I'm sorry for the delay in merging this. I really need to get better at keeping track of these and actually dealing with them.


spi-conf: Conf support nacos grpc request

Add basic nacos grpc support for spi-conf.

Security Checks:
It's ok to use self-signed tls certification for testing. The older one has expired.


fix table directories creation (#1)

Thank you, I see I missed the addition of this function in Minetest 0.4.13.


Cross Server/SinglePlayer/MP Mailing

loving this mod but kinda needing a feature where you:
1: can send a mail to a server from outside the server
2: optionally id also love to be able to send parcels from singleplayer to the server
been a whole day looking for a way to be able to transfer items from sp to mp seamlelly without any luck , i am aware there are flaws on doing this , but still need a way to do it

these features need another independent server to exchange info, like how e-mails work, which I cannot support.
network security or spam is also a problem.