Add an option to disable imagick in NextCloud

There is some discussion about the use of imagemagick in NextCloud due to past CVEs. Admins may want to disable it unless they are certain that users won't upload untrusted files (which can easily happen by accident if folder sync is active).
NextCloud is only passing certain file types to imagemagick but it is doing so based on the file extension. Therefore, an attacker can pass any file type to imagemagick. I have tested this with a PNM image (not supported by NextCloud) that I have renamed to tux.heif. tux.pnm doesn't get a preview image but tux.heif does, i.e. NextCloud thinks it is HEIF and passes it to ImageMagick, which processes it as PNM.
There are several ways to disable imagick:

Disable enable_previews in the NextCloud config. This will not only disable imagick but also previews for PNG and JPEG (which do not use imagick).
Set enabledPreviewProvidersin the NextCloud config. This may need manual adjustment after a NextCloud update.
Disable the imagick extension for PHP. This also disables it for NextCloud plugins.

Option 1 and 2 can already be used by an admin if the drawbacks are acceptable. However, I think 3 is the best option in most cases. This commit has added options that disable the other PHP extensions that used to be added by default. I think we should do the same for imagick.
If imagick is disabled, NextCloud will automatically disable the preview generators that need it. It will also show a warning about that (which is subject to debate in the issue linked above) and there may be some implications for the themes app (which I don't use). Therefore, I think we should keep imagick enabled by default and leave the decision to the admin.
We could also add a suitable policy for ImageMagick and thereby disable any file formats or features that are considered insecure. I don't think I could write such a policy and be sure that it is secure (or even "secure enough"). In addition, we will have to keep it up-to-date with newer versions and CVEs. That is why I recommend that we only add a simple off switch.
The above information is based on NextCloud 20. I will wait until this commit and NextCloud 21 have landed in nixos-unstable (and on my server) and then test again and create a PR if there is any interest in having this upstream.

Ouch, decisions based purely on a file ext is a behavior that sounds pretty broken to me :/
Optiobally disabling imagemagick and adjusting the config accordingly sounds reasonable to me, so feel free to file a PR. You can ping me for a review or if you need assistance with modifying the module :)

Ouch, decisions based purely on a file ext is a behavior that sounds pretty broken to me :/

It's ok for what they do with it. It won't do double duty as a mitigation but I guess that was never intended.

Optiobally disabling imagemagick and adjusting the config accordingly sounds reasonable to me, so feel free to file a PR. You can ping me for a review or if you need assistance with modifying the module :)

Thanks. Any suggestions on a name for the new option? Simply services.nextcloud.imagemagick ?
Should I add any tests for that? The existing tests only test three different configurations so adding one just for imagick seems like overkill. It should be quite easy though. If we want to test that previews actually aren't generated... Well, that's a different story. We would have to automate the web interface, for that. I think the existing tests are only testing the service, CLI and WebDAV.

Thanks. Any suggestions on a name for the new option? Simply services.nextcloud.imagemagick ?

As imagemagick enabled will remain the default, I'd suggest services.nextcloud.disableImagemagick.

The existing tests only test three different configurations so adding one just for imagick seems like overkill. It should be quite easy though

Yes, adding a fourth VM test just for imagemagick is indeed an overkill IMHO since those tend to be fairly expensive. I'd suggest to write a testcase in nextcloud/basic.nix, other integrations such as davfs are also tested there.

Well, that's a different story. We would have to automate the web interface, for that. I think the existing tests are only testing the service, CLI and WebDAV.

Right that could be a non-trivial thing to solve. Maybe check if there's a REST API or a way to implement this with the CLI of nextcloud-client. But I don't think it'd be worth spending too much time on this if there's no reasonable way to test :)

I think we still want to test the default option in basic.nix so we will need a second configuration that disables imagick. We don't have to start that VM if we only want to check whether imagick is in its closure. As there isn't any test helper for that, we probably don't want to do that though ;-)
see https://github.com/NixOS/nixpkgs/pull/115372/commits/d83a43a396c7ec5ac2cd633262648306e25c6789
Unfortunately, qemu on ARM big.little is too unreliable for the automatic tests because qemu crashes when started on one of the little cores (i.e. 4 out of 6 times). I have done some interactive testing of the above commit but not an automated run. I will have to find some system that is not ARM, not high-security, not already virtualized...

Qemu/KVM does work if a kernel build or stress-ng is hogging the big cores :)
@Ma27 https://github.com/NixOS/nixpkgs/pull/115372 is ready for review.


Updating security & id portions

Fixes #18 and removes the concept of an id for now.
The security aspects will be expanded in the future via #31.

Do we want to add the guidance on doing Subject Access reviews ?

I am thinking about leaving that for RC2 or beyond, since different use cases (e.g. using an Operator, or Tekton Pipeline, or a dev directly using SBO, or someone using odo, etc) would probably need different guidance.
Right now the spec is just suggesting the metadata declared by the bindable service for the role needed, and leaving the implementation details open.  Would be interesting to start prototyping this in SBO, which would likely improve our understanding of the flow.
Perhaps we can revisit as part of https://github.com/application-stacks/service-binding-specification/issues/31?

Sounds good!


Update docs about caches, specifically hazelcast

Documenting https://github.com/tumblr/collins/pull/367
@tumblr/systems

Fixed all the spelling mistakes that @defect pointed out.

+1

LGTM


logcatd_android.cc: Fix for repeating the same message.

Can we make the test stricter to catch this bug? Maybe we can clear the logcat buffer and whitelist the PID of our test before starting logcat, then we can check all the messages.
https://github.com/commaai/openpilot/blob/master/selfdrive/logcatd/tests/test_logcatd_android.py

the better(probably best) solution is to use blocking mode like logcat :
https://android.googlesource.com/platform/system/core/+/refs/heads/android10-dev/logcat/logcat.cpp#1592
and catch  signal to exit
https://android.googlesource.com/platform/system/core/+/refs/heads/android10-dev/logcat/logcat_main.cpp#25


DOC: Improve the documentation clarity for NormalizeImage.

Expand upon the existing caveats to clarify the intended
output values of the image.

Should it be a warning or error if the output type is an integer?
Should the default output type default to the real pixel type?
This seems more than just a documentation issue to me.

@blowekamp I wanted to keep the scope of this PR small.  If you want to make mandated errors/warnings, I would be supportive.
This is backwards compatible.  What you propose is not, although I agree it is probably good.


Time tag considered as custom element

Hello,
It seems the <time> tag is considered as a custom element by VueJS.
I keep getting the following warning on 1.0.0 alpha and beta:
[Vue warn]: Unknown custom element: <time> - did you register the component correctly?

Vue is using element.toString() to check if the browser returns [object HTMLUnknownElement]. Looks like this is a bug in Chrome where valid HTML5 elements <time>, <data>, <rb> and <rtc> return unknown instead of [object HTMLElement].
I have filed a bug for Chrome, you can ignore this warning for now. Also the warning will not show up if you use the production build.

So chrome team considers it a WontFix at the moment: https://code.google.com/p/chromium/issues/detail?id=540526
I've added an extra rule to exclude the affected elements from the check: 13e0335

Oh, this is a Chrome issue, my bad.
Thanks for the patch and the heads-up about the production build, didn't notice it :+1:

This very same thing happens with the details tag as of vue 1.0.24

Same thing happening with <details> and <summary> tags on Firefox.

@Gangwolf should be addressed in 63dbc1ca10d1663cb379ca7bd255d0ae3e836673

tag h7 return same problem in firefox using versions 2.2.4

Just got this in Chrome for <navigation>, is it still an issue?

no, it shouldn't, if you still have the issue you should open an issue with a repro so we can take a look

<navigation> is an HTML5 element?

You're right @LinusBorg , I thought he was getting an error with time 😅

Sorry, the HTML5 tag is . Clearly I've spent too much time writing JS and I'm now forgetting HTML. On 13 Jun 2017 15:27, Eduardo San Martin Morote<EMAIL_ADDRESS>wrote:no, it shouldn't, if you still have the issue you should open an issue with a repro so we can take a look
—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or mute the thread.


Listen to Siemens PLC

Is there a way to keep listening to Siemens PLC and trigger an event when data is received?
What I want is, instead of running a ReadArea function every 100ms, I want to keep listening to Siemens PLC and wait for data that he send.

I think what you are searching for is the Partner model, but this is not yet implemented in node-snap7.

@mathiask88 thank you, is there any plans to implement it, and when?

Yes there are plans to implement the partner but i can't tell you any date because this is a free time project. I recently added the server functionality but it is not yet released on npm.
There should be no risk to poll the PLC because u can set the communication ressources used by the PLC cpu in TIA. I got a project where I poll the PLC nonstop and there are no issues.

ok @mathiask88, thank you.

Old thread already, but giving here my experiences with polling S7-400 and S7-1500 series. Latest project I have been working with we had 4 clients with node-snap7 connected to our S7-1500. One is "plc-poller" ticking with 500ms cycle time, reading some DB and parsing around 3500 signals out of it. Other three are running other tasks including reading and writing to plc with 1000ms cycle time.
So far I have seen no issues with the S7 CPU or our node processes lagging or failing reads / writes, it has been robust. The plc-poller process actually spends most of its cpu cycles parsing and handling the signals from the Buffer and doing other stuff with it.

Cool, I'm glad to hear that node-snap7 performs well for you. Being robust is very important for this type of software and I'm happy that it is.
I will try to implement the partner as well in the coming months. And the server needs some small fixes and testing. I also got a prebuild feature near completion so that windows, mac and linux will download prebuilt binaries instead of compiling.

Cool, I'm glad to hear that node-snap7 performs well for you. Being robust is very important for this type of software and I'm happy that it is.
I will try to implement the partner as well in the coming months. And the server needs some small fixes and testing. I also got a prebuild feature near completion so that windows, mac and linux will download prebuilt binaries instead of compiling.

Do you support reading and writing of DB blocks for s7-1200?Test finds read and write DB block failed
#36700160 - CLI : function refused by CPU (Unknown error)

@SherryHolmes do you have put/get enabled on the 1200?

Please have a look at the compatibility table mentioned in the About section


Improve CountSpec testing

It uses the actual types like CountUnit rather than common denominator strings
The apply method uses default values to shorten the specification
The text representation of CountAttachment includes both "value" and value.d for comparison

The "Improve CountSpec testing" changes should help.  I'm not sure how the previous changes were merged, so there's a PR for everything.

@kwalcock, thanks for working on this! I have just merged and adjusted the rest of the tests to the new format. Many of the group tests seem to work. We will go through them more thoroughly in the next few days to see if the testing works as intended.


Create documentation for switching models in CO2.js

Once https://github.com/thegreenwebfoundation/co2.js/pull/96 gets merged, users will have an easier way to change to using the Sustainable Web Design model in CO2.js
import CO2 from "./co2.js";
co2 = new CO2({ model: 'swd' });

This needs to be documented, as there is currently no material in the CO2.js docs on how to change models.

I've merged this change in now.
I'll make a new npm release this morning to demonstrate the different methods in an observable notebook.
This might be useful to refer to in the docs, it it's also something that I think can be done independently of this issue here.

This has been added in #10


feat(interactions): add the app_permissions field

Summary
This uses the newly added app_permissions field for the permissions property in Interactions. Closes #720.
Checklist

[ ] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.
[ ] I have run task pyright and fixed the relevant issues.

[ ] This PR fixes an issue.
[x] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

So it looks like permissions is permissions for the user and app_permissions is permissions for the bot, so permissions probably shouldn't be touched at all.

Oh I thought permissions was for the bot
My bad

What happened to the colon after Permissions?


HTTP REST API

This PR aims to implement as much of #415 as possible. The API specification is written using the OpenAPI specification for REST APIs. This specification was chosen because it allows custom tools to parse the API spec into auto-generated client implementations, which may prove useful. Right now there are endpoints specified for basic operations and their return codes. Discussion welcome!

It would be nice to see an example of the full JSON schema for at least one pair of request/response objects - whichever is simplest (maybe the one best specified in the existing Irmin code) is fine.  Some short descriptions would be nice too, even though they might seem superfluous.
The TODO on watches is attention-catching.  From what I understand about REST and watches, a design which allows the user to register a webhook for watches, rather than receiving notifications directly on the connection which asked for the watch, is an idiomatic way to solve this problem (and I think would be significantly easier to tackle in this design/implementation, although that shouldn't necessarily drive the decision).  @talex5 and @samoht probably have something more intelligent to say about this.

Also, thanks for this PR!  :)

Thanks for the PR!
I agree with Mindy's comments :-)
About the watch API: as identified early this is one of the hard one (with transactions). In the previous API, we used a long-lived HTTP connection where the server will send to the client update events: I still think that the simplest way to implement this, although having webhooks would definitely be nice as well -- but more complex as the client needs a public endpoint for the server to reach it. So I would just keep the long-lived HTTP connection for now on.

Per the commit description, I've added the requested schema and descriptions. I will do the watch endpoints tomorrow.

About the watch endpoints: Per the discussion above I've opted for a streaming endpoint rather than a webhook because webhooks require the client to also be able to receive HTTP requests, which I'm not sure is reasonable here. I'm still not 100% sure how to specify that the endpoint is a stream in the OpenAPI spec, so there are still 2 lines in the spec marked TODO.

The low-level watch API exposes a GET endpoint, and return an array of objects:  the trick is to never close that array and have non-blocking clients.

I am closing this in favour of the upcoming GraphQL API. Thanks anyway for your efforts on the topic @dudelson

No problem! I regret leaving this in such a sorry state, but I look forward to seeing what you do with graphql.


If topology name is big, listing page gets misaligned

After going to topology editor and back to listings page

Fixed in commit: 0e58a957c0b651a1cf6f4a75c165d3bc76b8b6f4


[Bug]: x-axes inconsistent on Stage 3 output plots

FAQ check

[X] Yes, I checked the FAQ and my question has not been addressed.

Instrument
MIRI (Stages 1-3)
What happened?
The x-axes are inconsistent between the 2D Background Subtracted Frame( e.g. fig3301_file0_int00000_ImageAndBackground.png)  plots and the 1D Spectrum plots (e.g. fig3302_file0_int00000_Spectrum.png). It seems like the 1D spectrum plots have the Detector Pixel Position flipped.
Error traceback output
No response
What operating system are you using?
No response
What version of Python are you running?
No response
What Python packages do you have installed?
No response
Code of Conduct

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

Ah indeed, the x-axis for Fig 3301 is labelled backwards for MIRI observations only, and for Fig 3302 it would probably be best to invert the x-axis so that short wavelengths are on the left. I'll aim to fix this minor bug in a week or so

Alright, looking more carefully it looks like even more plots are messed up for MIRI. Specifically, figs 3301, 3303?, 3308, and 3506 have the detector pixels labelled backwards, and fig 3302 should be flipped so that wavelength increases toward the right and not pixel position. It's also not immediately clear to me if the column numbering in figs 3501 are in the correct for MIRI. While I'm at it, fig 3304 appears not to be using bbox_inches='tight' or something since parts of the plot are getting cut off in the saved image.
I'll work on this today and hopefully submit a PR.

PR #600 resolved the issues with figures 3301, 3302, 3308, and the bug with 3304 doesn't seem to occur anymore. That only leaves figs 3303 and 3506 which aren't wrong but just use different units (relative pixel position rather than detector pixel position). This seems fine to me, so I'm going to close this issue now


Office 365 AddIn with Groups Authorization Sample

Office 365 AddIn with Groups Authorization Sample

Thanks Luis for this. There's two asks we would however have related on this.

You will need to sign the electronic contribution agreement, which has been sent to your email or you can access that using https://cla.microsoft.com
Second request relates on the naming of the sample. We do not actually use terminology of add-in related on the Office 365 API based solutions. That's only for SharePoint and Office. Better prefix for this one would be AzureAD so that it's grouped under similar samples (one exists already under samples before this one as well). This will help to categorize samples accordingly.

I'm sorry for the inconvenience related on the name change and we certainly will work on the guidance related on the right naming of the samples.

ok fixed, do I need to create another pull request?

If you've fixed code in your Office365AddinWithGroupsAuthorization branch it will be automatically picked up, no need to issue a new PR. In reality a PR is a compare between your branch and the PnP dev branch at the moment it's being processed.

@levalencia, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Thanks for signing the CLA Luis. Will have a look at the PR and process it


small optimization in strlen

Since you are negating BC and then decrementing, this is equivalent to just inverting BC.
It's just a small optimization, but I thought I'd share.

Thanks!


[Logging] Start Adding Logging to Code

I should decide how to log errors that occur

Transitioned to Jira


ci: add a updated ci with linter check

Issue
Closes #8
Chnages

Add a Go Linter check for incoming PR to analyse source code for errors, bugs, and other issues.

I am experiencing the above errors for pull request from contributors. Can you solve this error along with the linter part?

I will fix that. I see you delete the workflows folder

Hey, @vigneshs-dev
I have fixed the issue. Approve the workflow run in the PR and test.

Hey, @vigneshs-dev
I have fixed the issue. Approve the workflow run in the PR and test.

Hey, @vigneshs-dev
I have fixed the issue. Approve the workflow run in the PR and test.

Thankyou for your contribution. Keep contributing.

Hey,
You merged the PR even though the CI was falling. The  golangci/golangci-lint-action@v6 was falling and everything was working fine because you changed the project structure and it's not able to locate the go file.


installing in docker

hi, while copying the docker compose yaml file and starting up the container lot of errors are coming, its not deploying in doocker could you guide how to istall in docker

I ran a quick test and the Dockerfile should work fine. You can make some simple adjustments to the Docker-compose file to get it running. However, I’m no longer maintaining this project. It still uses .NET SDK 7.0, and I don't plan to upgrade it.


Add sign method

Note: (Have to add publish_to: None until we publish newest version of polkadart_keyring.)
Have to provide SignatureType to encode() method.
Now possible to do:
final extrinsic = Extrinsic(
   signer: publicKey,
   method: encodedCall,
   signature: hexSignature,
   eraPeriod: 64,
   blockNumber: blockNumber,
   nonce: 0,
   tip: 0,
 ).encode(api.registry, SignatureType.sr25519).sign(keyPair);

or
final extrinsic = Extrinsic(
   signer: publicKey,
   method: encodedCall,
   signature: hexSignature,
   eraPeriod: 64,
   blockNumber: blockNumber,
   nonce: 0,
   tip: 0,
 ).encodeAndSign(api.registry, SignatureType.sr25519, keyPair);

@leonardocustodio

A i'm merging without reviewing as the code is going to appear again in the other pr, that would just "duplicate" the review. We can see everything just a single time in there.


[WIP e2e test] fix(backup): detach the volume when the status is synced

ref: longhorn/longhorn#6124
Detach the volume only when the status and the backup is synced

Have tested in local environment for 30 times, the issue didn't happen
Local testing
# backup controller attach the volume
time="2023-06-16T03:07:15Z" level=info msg="===[DEBUG backup]=== Attaching the volume" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:15Z" level=info msg="[DEBUG] ticketID: backup-controller-backup-5e40535b844741b1 already exists"
time="2023-06-16T03:07:15Z" level=info msg="===[DEBUG before checkmonitor]===" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:15Z" level=info msg="[Debug] Checking Attachment Ticket Satisfied attachmentTicket.Generation 0, attachmentTicketStatus.Generation 0"
time="2023-06-16T03:07:15Z" level=info msg="[Debug] Checking Attachment Ticket Satisfied attachmentTicketStatus.Satisfied 0%!(EXTRA int64=0)"
time="2023-06-16T03:07:15Z" level=info msg="[Debug] Checking Attachment Ticket Satisfied is Satisfied: true"

# backup controller syncmonitor sync the backup
time="2023-06-16T03:07:18Z" level=info msg="===[DEBUG before syncWithMonitor]===" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:18Z" level=info msg="===[DEBUG after syncWithMonitor]=== backup.Status.State " backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:18Z" level=info msg="===[DEBUG backup]=== Attaching the volume" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:18Z" level=info msg="[DEBUG] ticketID: backup-controller-backup-5e40535b844741b1 already exists"
time="2023-06-16T03:07:18Z" level=info msg="===[DEBUG before checkmonitor]===" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:18Z" level=info msg="===[DEBUG before syncWithMonitor]===" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:18Z" level=info msg="===[DEBUG after syncWithMonitor]=== backup.Status.State " backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:20Z" level=info msg="===[DEBUG backup]=== Attaching the volume" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:20Z" level=info msg="[DEBUG] ticketID: backup-controller-backup-5e40535b844741b1 already exists"
time="2023-06-16T03:07:20Z" level=info msg="===[DEBUG before checkmonitor]===" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:20Z" level=info msg="===[DEBUG before syncWithMonitor]===" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1

# backup synced completed
time="2023-06-16T03:07:20Z" level=info msg="===[DEBUG after syncWithMonitor]=== backup.Status.State Completed" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:20Z" level=info msg="===[DEBUG]=== backup disableBackupMonitor backup-5e40535b844741b1" controller=longhorn-backup node=kworker1

# When it failed to update the backup status, it won't detach the volume now
time="2023-06-16T03:07:20Z" level=debug msg="Requeue backup-5e40535b844741b1 due to conflict" backup=backup-5e40535b844741b1 controller=longhorn-backup error="Operation cannot be fulfilled on backups.longhorn.io \"backup-5e40535b844741b1\": the object has been modified; please apply your changes to the latest version and try again" node=kworker1

# backup controller re-do the sync
time="2023-06-16T03:07:20Z" level=info msg="===[DEBUG backup]=== Attaching the volume" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:20Z" level=info msg="[DEBUG] ticketID: backup-controller-backup-5e40535b844741b1 already exists"
time="2023-06-16T03:07:20Z" level=info msg="===[DEBUG before checkmonitor]===" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1

# the generation of the volumeattachment ticket is still 0, because it is not detached
time="2023-06-16T03:07:20Z" level=info msg="[Debug] Checking Attachment Ticket Satisfied attachmentTicket.Generation 0, attachmentTicketStatus.Generation 0"
time="2023-06-16T03:07:20Z" level=info msg="[Debug] Checking Attachment Ticket Satisfied attachmentTicketStatus.Satisfied 0%!(EXTRA int64=0)"
time="2023-06-16T03:07:20Z" level=info msg="[Debug] Checking Attachment Ticket Satisfied is Satisfied: true"

# sync again
time="2023-06-16T03:07:22Z" level=info msg="===[DEBUG before syncWithMonitor]===" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:22Z" level=info msg="===[DEBUG after syncWithMonitor]=== backup.Status.State " backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:24Z" level=info msg="===[DEBUG backup]=== Attaching the volume" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:24Z" level=info msg="[DEBUG] ticketID: backup-controller-backup-5e40535b844741b1 already exists"
time="2023-06-16T03:07:24Z" level=info msg="===[DEBUG before checkmonitor]===" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:24Z" level=info msg="===[DEBUG before syncWithMonitor]===" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1

# backup synced completed
time="2023-06-16T03:07:24Z" level=info msg="===[DEBUG after syncWithMonitor]=== backup.Status.State Completed" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:24Z" level=info msg="===[DEBUG]=== backup disableBackupMonitor backup-5e40535b844741b1" controller=longhorn-backup node=kworker1

# backup controller detach the volume only when it is synced
time="2023-06-16T03:07:24Z" level=info msg="===[DEBUG backup controller]=== Update Backup Status successfully and the backup is synced, Deleting attachment" backup=backup-5e40535b844741b1 controller=longhorn-backup node=kworker1
time="2023-06-16T03:07:24Z" level=info msg="Deleting instance test-e-19bc50a9"

@mergify backport v1.5.x


FireEdge Sunstone is not filtering by labels

Description
When using the label functionality to filter VMs in FireEdge Sunstone, the datatable shows a message saying that There is no data available
To Reproduce
Assing a label to a VM
Try to filter by label
Expected behavior
VMs with the selected label to be displayed
Details

Affected Component: FireEdge Sunstone
Hypervisor: [e.g. KVM]
Version: 6.4.2, development

Progress Status

[ ] Code committed
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

Code: https://github.com/OpenNebula/one-ee/pull/2394
Docs: https://github.com/OpenNebula/docs/pull/2440


added whenever_variables as a configuration variable

I recently needed to set an additional variable via a cap task and found that the only way to do so was to override whenever_update_flags like so:
set :whenever_update_flags, defer { "--update-crontab #{fetch :whenever_identifier} --set 'environment=#{fetch :whenever_environment}&log_path=#{shared_path}'" }

This is a minor backwards compatible change that allows you to set the environment + variables independently:
set :whenever_variables, defer { "'environment=#{fetch :whenever_environment}&log_path=#{shared_path}'" }

Any thoughts on merging this in?  Thanks.

merged!


Show vector init update

I think I did the correct incantation to squash the merge commit.  Is that even English?

The Windows CI failed on windows here is the CDash report: https://open.cdash.org/viewBuildError.php?buildid=6528345

Dammit!  I spent all day yesterday trying to build it on Windows (my little Surface).


Support for DATABASE_URL environment variable

In addition to specifying database credentials in the [[providers]] section of the config TOML, DATABASE_URL should be read (as a URI), allowing configuration files to be freely hosted and shared without concern for leaking secrets.
E.g., postgis://tegola:<redacted>@<redacted>:5432/osm (and/or postgresql for compatibility with libpq).

Just to clarify, you're suggesting we read in an environment variable into a "template variable" defined in the config? I like this idea as it could support multiple database URIs when the config is working with several data providers.

Yes, I think we're talking about the same thing as long as only 1 set of database credentials is in use.
The goal is to remove the database credentials from the config and have them dynamically added by the server.

This is related to #77

And it looks like a dupe of #95


Extend matchers

I would like to propose a few additions and refactors (with your approval, I can begin working on them):

Use the expected and actual terminology in assertion messages and method signatures.
Allow method chaining, expect([1,2,3]).to.contain(3).and_not_to.contain(7)
Add additional expectations:

to_be_list()
to_have_length()
to_have_attribute()
to_have_attributes()
to_almost_equal()
to_be_instanceof()
to_be_bool()
to_be_numeric() # Strict
to_be_callable()
to_be_int()
to_be_float()
to_be_str()
to_be_none()
to_startwith()
to_endwith()
to_match() # regex

Add higher-order expectations using expect([1,2,3]).each.to_be_numeric(3).

All, 1., 2. and 4. sound great, I have nothing to add there.
On 3rd point I have just few clarifying questions:

to_have_attributes() is going to take a list of names of expected attributes as arguments?
What is to_almost_equal() meaning to match? Numeric similarity with some sort of acceptable delta?

Your changes are very welcome!

Regarding to_have_attributes, I imaging the following overloaded signatures:
def to_have_attribute(self, key: str):
    ...

def to_have_attribute(self, key: str, value: Any):
    ...

def to_have_attributes(self, *keys: str):
    ...

An explanation of to_almost_equal from the Python's built-in unittest package.

Both sound great! Looking forward to the PR(s)!

@petereon Before I get started, one more question, is there a reason why you split alias and negated test into separate methods?
Example:

test_to_contain_subfolder
test_to_contain_subfolder_failing
test_to_not_contain_subfolder
test_to_not_contain_subfolder_failing

@bdsoha Yes, it's for legacy reasons, there have previously been separate implementations for negated matchers. Aliases don't necessarily make sense, I agree. As it is it would presumably make more sense to test just the __init__ method that it negates correctly.

@petereon I have an OOP proposition.
Is there an alternative location were we can discuss the idea or do you want to continue here?


Data Collection

Tasks:
Conduct interviews with selected AI, IoT, and blockchain experts.
Distribute surveys to targeted IoT developers and companies.
Gather technical data and requirements for the simulation models

Tasks:
Conduct interviews with selected AI, IoT, and blockchain experts.
Distribute surveys to targeted IoT developers and companies.
Gather technical data and requirements for the simulation models

to fix your trouble check this fix, i see it in another issue,
https://github.com/Jostino/RimWorld-it/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://github.com/Jostino/RimWorld-it/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


Fill in mapping for OML/ORM

Backlog Task
Following the OML/ORM mapping work completed by RS there remains a number of fields that unmapped.

PID-2 - Missing because PID-2 is "Backwards Compatible" field in 2.5.1

PID.19 and PID.20 - are deprecated in the HL7v2.7 and NIST HL7v2.5.1 specs. Further, the HAPI v2.7 model has set both # fields to NULLDT.

ORC-7 - Missing because ORC-7 is "Backwards Compatible" field in 2.5.1

OBR-27- Missing because OBR-27 is "Backwards Compatible" field in 2.5.1

SPM-35 - Missing because SPM-35 does not exist in 2.5.1

Completion Criteria

[ ] Above fields can be mapped into FHIR and back out to HL7 for OML
[ ] Above fields can be mapped into FHIR and back out to HL7 for OML

Tasks

[ ] What tasks need to be done to fulfill the completion criteria?

Other Notes

Any other notes to help clarify this task for the team

ORC-7 not available in HL7 v2.7. May well be an issue with the other fields.


Support query string for the search plugin

For example, https://docsify.js.org/#/?s=navbar would open the website and automatically input "navbar" in the search bar (if the search plugin is enabled).
This would be helpful when a user asks you something that is written in the docs, but you do not bother to find the exact link for him.

Good idea


Is it possible to play a video using this?

I was messing around with this and was wondering if it would be possible to play a video using this by converting the video to base64 or something like that with a decoder in scratch and get at least 1fps? Even if it has to buffer the video into a list or something like that.

With a resolution of 50x50, at a FPS of 1, 50 seconds video would take 16 minutes to transmit.

yikes

With a lowwer resolution, base64 encoding and pixel compressing, you can very maybe get 5 min.

I heard someone got a browser working years ago, do you know if that actually show'd the page or if it was just basic html text?

I think it only showed the basic html text.

That makes sense, I want to try to recreate it but only allow whitelisted sites/have content filtering.

One of the issues with sending images to Scratch is that almost all of them are corrupted. I mean, they get randomly cut off or with a weird offset in some rows.

I really wish the ST would just implement an extension for this

Yeah

In Scratch it's not possible, but in turbowarp it is

Well, it is. You can send a 1 FPS video, like a slideshow.

Well, it is. You can send a 1 FPS video, like a slideshow.

If it was a 1hr video, that would be 3600 frames (60 secs * 60 mins)


Fixed missing import statement

Missing import statement is causing Elan QC step to fail on large BAM files, the bug occurs when:

A BAM file is stupidly huge (14G in this instance) -> this causes hashing the file to take longer than 5 mins.
This leads to the block starting on line 284 to execute which then promptly falls over because the math python module is not imported.
Elan QC fails with a message about  there being a "space in fasta header" which is a red herring.

I'm unable to test this change, happy to do so if you give me some guidance.

This leads to the block starting on line 284 to execute which then promptly falls over because the math python module is not imported.

lol
Nice edge case. I suspect we'll probably want to remove this partial hashing entirely (for CLIMB-COVID at least) as the file system is fast enough to hash these files and it seems more ideal for the file hashes in Majora to not be partially calculated. This will need an update and full roll-out of Ocarina, so in the meantime I've updated the errorStrategy for Elan's play_ocarina step to ignore errors that do not go away after an initial retry which will stop Elan full-stopping on this problem.

Elan QC fails with a message about there being a "space in fasta header" which is a red herring.

This is unrelated to the issue here, that warning is emitted from the previous ocarina command that processes the FASTA (recall play_ocarina calls ocarina a few times for various things). Nextflow will have shown this as part of the job failure output as it summarises the stdout of failed jobs if available. The real error (as I suspect you found) is in the .command.err for the play_ocarina job (which would be my go-to for weird errors in future).


add python formatting proposal

Changes introduced with this PR
Adding a proposal to standardize on one python code formatting solution.

By contributing to this repository, I agree to the contribution guidelines.

Seems fine to me

@janosdebugs, see https://github.com/arcalot/arcalot-round-table/pull/18, I did not push it to that branch since it was @dustinblack's proposal.


found some more files with "dos" line endings

IMPORTANT: Please don´t commit to master, we will test your changes first at the beta branch!
1. Tell us something about your changes.
2. If this changes affect the .conf file. Please provide the changed section
3. Reference a issue (optional)
Note: For further discussions use our forum: forum.hyperion-project.org

Ah now i get it. If someone makes a PR and whole files will be rewritten and you see no code changes. This will be a often a convert?!


double-bit ECC failures that occur during load from ICCM do not set cptra_hw_error_fatal::iccm_ecc_unc

To reproduce:
Corrupt a word in ICCM, and try to load it with the lw instruction.
The only symptoms that occur when this happens are an NMI with generic MCAUSE 0xf000_0001 (NMI_CAUSE_DBUS_NON_BLOCKING_LOAD_ERROR):
mcause: 0xf0000001,
mscause: 0x0,
mstatus: 0x1800,
mtval: 0x0,
cptra_hw_error_fatal: 0x0
cptra_hw_error_enc: 0x0
cptra_hw_error_non_fatal: 0x0
error_internal_intr_r: 0x0

If this happens in the field, it will difficult to root-cause it, as there doesn't seem to be any indication to the uC an ECC error occurred.

Digging into VeeR logic here. Looks like the iccm ecc double error fires only on IFU requests to ICCM. When LSU does a DMA to the ICCM it just flags it as a bus error (same with DMA to DCCM as far as I can tell).
Found this in the PRM:

That makes sense because the data retrieval is not an IFU operation. This would be similar to operation on the bus where there is a parity error (I understand that our bus doesnt support parity errors :-)).

@bharatpillilli I'm fine with the fault being a generic fault, but we need some mechanism to tell that ECC was the cause of the fault. When this happens, I would like to see the hardware set the iccm_ecc_unc bit in cptra_hw_error_fatal.

This error could be communicated to the SoC by writing to the cptra_fw_error_fatal register, which will also trigger the cptra_error_fatal output pin. Firmware should probably be doing this for any NMI it encounters already.
I understand that an NMI may be unrecoverable and firmware might be unable to set the register - but this is the best option with current hardware.

The firmware is already setting the cptra_fw_error_fatal register, but the outside world doesn't know the reason for the failure.


Package upload fails when client uses HTTP/2 with Istio Ingress Gateway

NOTE: Logging this here to provide visibility into it as a "Known Issue".  It is possible that a future Istio upgrade might fix this or that an alternative Ingress controller won't have this issue.
Issue
This issue is a result of our investigation into CAPI Story #174918523: Package upload fails intermittently.
When using curl to upload a package, you will frequently receive 503 errors back from the API. The cf-api-server/nginx logs will include errors like this:
cf-api-server-749798466-w6tmt > nginx | 2020/09/21 21:37:09 [info] 10#0: *74674 started uploading file "dora.zip" to "/tmp/uploads/0000000011" (field "bits", content type "application/octet-stream"), client: <IP_ADDRESS>, server: , request: "POST /v3/packages/d4d5232d-ed0f-47b7-bd1f-0fc5c968b8ad/upload HTTP/1.1", host: "api.nat.k8s.capi.land"
cf-api-server-749798466-w6tmt > nginx | 2020/09/21 21:38:10 [alert] 10#0: *74674 aborted uploading file "dora.zip" to "/tmp/uploads/0000000011", dest file removed, client: <IP_ADDRESS>, server: , request: "POST /v3/packages/d4d5232d-ed0f-47b7-bd1f-0fc5c968b8ad/upload HTTP/1.1", host: "api.nat.k8s.capi.land"

The cf CLI does not appear to have this issue.
Context
We did quite a bit of debugging in #174918523. Things we've found:

curl gets its connection upgraded to use HTTP/2 by the Ingress Gateway Envoy
This does not happen for the cf CLI for some reason. It continues to use HTTP/1.1.
The nginx we have configured and sidecar Envoy do not use HTTP/2. The Ingress Gateway uses HTTP/2 with the client, but converts it to HTTP/1.1 when delivering the request to the Pod.
The entire package file never makes it to the Sidecar Envoy/nginx and nginx will eventually timeout, resulting in a 408.
The Sidecar Envoy returns this as a 503 which makes its way to the user.

Steps to Reproduce
Deploy cf-for-k8s and use curl to upload some "app source" (I used this Shakespeare txt file):
Example:
curl -k https://api.tim.k8s.capi.land/v3/packages/some-package-guid/upload -X PUT -f -F<EMAIL_ADDRESS>-v -H "Authorization: $(cf oauth-token)"

Expected result
For a valid package guid I would expect a successful 200 level response. For testing purposes you can use a bogus package guid and expect to get a 404 back from the API after the file is uploaded.
Current result
The curl will output something like this and hang at "We are completely uploaded and fine" for 60 seconds:
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to api.tim.k8s.capi.land (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*   CAfile: /etc/ssl/cert.pem
  CApath: none
* TLSv1.2 (OUT), TLS handshake, Client hello (1):
* TLSv1.2 (IN), TLS handshake, Server hello (2):
* TLSv1.2 (IN), TLS handshake, Certificate (11):
* TLSv1.2 (IN), TLS handshake, Server key exchange (12):
* TLSv1.2 (IN), TLS handshake, Server finished (14):
* TLSv1.2 (OUT), TLS handshake, Client key exchange (16):
* TLSv1.2 (OUT), TLS change cipher, Change cipher spec (1):
* TLSv1.2 (OUT), TLS handshake, Finished (20):
* TLSv1.2 (IN), TLS change cipher, Change cipher spec (1):
* TLSv1.2 (IN), TLS handshake, Finished (20):
* SSL connection using TLSv1.2 / ECDHE-RSA-CHACHA20-POLY1305
* ALPN, server accepted to use h2
* Server certificate:
*  subject: C=USA; O=Cloud Foundry; CN=*.tim.k8s.capi.land
*  start date: Sep 28 17:11:13 2020 GMT
*  expire date: Sep 28 17:11:13 2021 GMT
*  issuer: C=USA; O=Cloud Foundry; CN=ca
*  SSL certificate verify result: unable to get local issuer certificate (20), continuing anyway.
* Using HTTP2, server supports multi-use
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* Using Stream ID: 1 (easy handle 0x7fc23680fe00)
> PUT /v3/packages/some-package-guid/upload HTTP/2
> Host: api.tim.k8s.capi.land
> User-Agent: curl/7.64.1
> Accept: */*
> Authorization: bearer <bearer-token-removed>
> Content-Length: 5458392
> Content-Type: multipart/form-data; boundary=------------------------d0063abea4df3cad
>
* Connection state changed (MAX_CONCURRENT_STREAMS ==<PHONE_NUMBER>)!
* We are completely uploaded and fine

Eventually it will fail with:
* The requested URL returned error: 503
* stopped the pause stream!
* Connection #0 to host api.tim.k8s.capi.land left intact
curl: (22) The requested URL returned error: 503
* Closing connection 0

When we did tcpdumps at various places[0] we found that curl was successfully sending the whole file to the Ingress Gateway Envoy, but it was not making it to the sidecar Envoy. Cloud Controller's nginx was waiting for the complete file (looking for the ------------------------d0063abea4df3cad boundary) and eventually timed out returning a 408. The sidecar Envoy considered this a bad response and bubbled it up as a 503 to the user.

[0] I took the tcpdump from the cf-api-server Pod's nginx container by including tcpdump in its Dockerfile and rebuilding the image. You may also try a tool like ksniff.
For example, this one shows traffic between the Ingress Gateway and the Pod's Sidecar Envoy on eth0:
kubectl exec -n cf-system cf-api-server-6b5895c557-scr95 -c nginx -- tcpdump -i eth0 -w - | wireshark -k -i -

This one shows traffic between the Sidecar Envoy and nginx on lo:
kubectl exec -n cf-system cf-api-server-6b5895c557-scr95 -c nginx -- tcpdump -i lo -w - | wireshark -k -i -

Possible Fix
A mitigation for curl is to force HTTP/1.1 with the --http1.1 flag.
Screenshots

Version Information

curl version:

curl 7.64.1 (x86_64-apple-darwin19.0) libcurl/7.64.1 (SecureTransport) LibreSSL/2.8.3 zlib/1.2.11 nghttp2/1.39.2
Release-Date: 2019-03-27
Protocols: dict file ftp ftps gopher http https imap imaps ldap ldaps pop3 pop3s rtsp smb smbs smtp smtps telnet tftp
Features: AsynchDNS GSS-API HTTP2 HTTPS-proxy IPv6 Kerberos Largefile libz MultiSSL NTLM NTLM_WB SPNEGO SSL UnixSockets

CF API version: 3.89.0
cf-for-k8s SHA: c03c38f03495fc5afc4d108799149419678514c8 (vendir.yml)
Istio version: 1.6.4

closing since cf-for-k8s is no longer under active development


[Enhancement]: the sql editor can scroll to the line that I located when the position is not in current screen

What would you like to be added?
I type too many sqls in the editor and I click check button. I choice an issue to locate the position. Unfortunately, the position I located is not in the screen, It seems like that the location button does not work.

I wish that the sql editor can scroll to the line which I locate when the line is not in the current screen~
Why is this needed?
To improve the user's experience

Fixed In Commit f7de18b7fae938488489828a3ca0fc4614012c40
------
陈朝俊
Wed Dec 13 09:27:46 2023 +0800


Channel override: adjust representation for the removal of bundles

We simplified the TED-Ed channel to not have bundles anymore, just the videos. Remove the bundle specific custom presentation.
https://github.com/endlessm/kolibri-explore-plugin/issues/730

Seems enough, but wouldn't it be necessary to remove the style.scss file as well?

The https://github.com/endlessm/kolibri-explore-plugin/blob/master/packages/template-ui/channel-overrides/ted-ed-lessons/styles.scss still has some settings we want to keep.


Update prospectus with 2019 version

Let's also update prospectus.md to take out the note that the pdf is out of date.
Also if you have the dependencies installed, you can run the website locally to make sure it looks ok (bundle exec jekyll serve).

Updated with recommendations. Also confirmed it looks fine locally


Preact crashes when trying to access children property when rendering

Preact Version: 10-alpha.2
In a project im working on, Preact seems to be crashing when it tries to render because of a undefined error related to something its doing internally. Using TypeScript 3.3.4 with ts-loader, along with preact-cli.
Im trying to wait for data to come through a socket, update the state with the data, and then re-render the page with the new data.
Here's the error that appears in Chrome's console:
debug.js?c91e:84 Uncaught (in promise) TypeError: Cannot read property 'children' of undefined
    at p (debug.js?c91e:84)
    at Object.preact__WEBPACK_IMPORTED_MODULE_1__.options.diff (debug.js?c91e:11)
    at x (index.js?c988:34)
    at b (children.js?b0ec:57)
    at C (index.js?c988:261)
    at x (index.js?c988:152)
    at b (children.js?b0ec:57)
    at C (index.js?c988:261)
    at x (index.js?c988:152)
    at b (children.js?b0ec:57)

The source for the page can be found here: https://github.com/AEnterprise/GearBot/blob/secrets/web/src/components/guildpage.tsx
Any help would be greatly appreciated.

@BlackHoleFox Can you share an example of a backend response? I got the frontend working but I'm unable to start the backend server.

Sure, this is the JSON data that the backend is currently sending up through the socket (Python):
guildPageData = {
            "name": "The Gearbox",
            "owner": "AEnterprise",
            "id":<PHONE_NUMBER>74410241,
            "members": 127,
            "textChannels": 22,
            "voiceChannels": 1,
            "totalChannels": 23,
            "creationDate": "05-10-2017 (538 days ago)",
            "vipFeatures": False,
            "serverIcon": "https://cdn.discordapp.com/icons/365498559174410241/735df5f0db5581592b7744f8fc10701f.webp?size=1024",
            "roles": [
                "Gear Spinners",
                "Gear Creators",
                "Amazing Gears"
            ],
            "serverEmojiCount": 51,
            "memberStatuses": {
                "online": 11,
                "idle": 10,
                "dnd": 10,
                "offline": 96
            }
        }

The issue is caused by passing a random object into the template. Preact tries to render it but fails to do so, because it isn't a valid vnode.

  Member Statuses: {this.state.guildPageStats.memberStatuses}

Where memberStatus equals:
{
  online: 11,
  idle: 10,
  dnd: 10,
  offline: 96
}

The above is not a valid vnode. One solution would be to pass the properties individually:
const status = this.state.guildPageStats.memberStatuses;

  Member Statuses:
  online: {status.online}
  idle: {status.idle}
  dnd: {status.dnd}
  offline: {status.offline}

That resolves the issue.
Note: The html in that component is not quite valid. A  (=table data cell) must include a
 (=table row) element as its children. Per spec it must be the other way round :wink:

foo

  foo

@marvinhagemeister Thanks for the quick response. Good to know in the future that I can't just toss whatever into Preact. With your method, its working fine now. I have also fixed the tables to fall into spec, the pointer is appreciated :)


Still alive?

Would benefit a lot this:
https://github.com/jtransc/gdx-backend-jtransc/issues/4
This project is still alive? In which state it is? Can it compile real code already?

maybe stuck with llvm-2.7.
saw the llvm-3.3 branch, didn't test if it works.
awesome project!
feel sorry for its death.


librime: 1.9.0 -> 1.10.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/librime/versions.
meta.description for librime is: Rime Input Method Engine, the core library
meta.homepage for librime is: https://rime.im/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.10.0 with grep in /nix/store/im727g9is5c1h5hnr7ldardqdrmb6m89-librime-1.10.0
found 1.10.0 in filename of file in /nix/store/im727g9is5c1h5hnr7ldardqdrmb6m89-librime-1.10.0

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

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
brise
emacsPackages.rime
fcitx5-rime
ibus-engines.rime
librime
rime-data

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/im727g9is5c1h5hnr7ldardqdrmb6m89-librime-1.10.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/5ea62d1e61462145aa78db0ce2bac612c207fca2#librime

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/im727g9is5c1h5hnr7ldardqdrmb6m89-librime-1.10.0
ls -la /nix/store/im727g9is5c1h5hnr7ldardqdrmb6m89-librime-1.10.0/bin

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

6 packages built:

brise
emacsPackages.rime
fcitx5-rime
ibus-engines.rime
librime
rime-data

Maintainer pings
cc @Vonfry for testing.

@ofborg eval


IPAM support

Feature Requests
IPAM support
Description of requested functionality
Can cilium support ipam plugin which will be useful if we need to know the IP of the POD before hand.
Example use cases
I am using cilium with direct routing mode. Routes have been created by kube-router. It works fine with the new cilium fix provided in https://github.com/cilium/cilium/pull/4547. Now PODS created on Node-0 gets an IP address from <IP_ADDRESS>/24 and Node-1 gets IP address from <IP_ADDRESS>/24. Kubeadm was started with  --pod-network-cidr=<IP_ADDRESS>/16.
Now, I would like to PODs to have predefined IP from the nodes subnet prefix. For example, POD-1 created on Node-0 to get <IP_ADDRESS> or <IP_ADDRESS> instead of any random IP address from  <IP_ADDRESS>/24.
We work with CRD and define ipam section in this.
apiVersion: "kubernetes.com/v1"
kind: Network
metadata:
name: cilium-conf-network-first
plugin: cilium-cni
args: '[
{
"name": "cilium-conf-network-first",
"type": "cilium-cni",
"mtu": 1500,
"ipam": {
"type": "host-local",
"subnet": "<IP_ADDRESS>/24",
"rangeStart": "<IP_ADDRESS>",
"rangeEnd": "<IP_ADDRESS>"
}
}
]'
POD is created with network name as "cilium-conf-network-first". It seems cilium doesn't support this ipam section and POD got the IP address as <IP_ADDRESS>.
List of sub tasks

[ ] Task 1
[ ] Task 2
[ ] Task 3

@subrnath We don't implement the IPAM configuration yet and rely on the the k8s PodCIDR configuration.
What I don't understand in your example is how it ties an IP to a particular pod. The configuration of the IPAM pool is cluster wide.

In this scenario, we are scheduling the POD to a particular node and that node specific label is put in the nodeselector field of the manifest.
OK. Agree with you, with as the IP allocation from  k8s PodCIDR configuration, it might not be feasible to  have IPAM.


parser: add support for GAME section

For the rationale of this: https://github.com/yifanlu/taiHEN/issues/78
This will allow to shorten config.txt a huge amount when many games are involved.

I've tested this with oclockvita and it works (e.g. combo only works in games, meaning the plugin isn't anywhere it's not supposed to be)

Good job.

So did this give you the ability to exclude certain plugins with homebrew? I'm currently trying to exclude VHS from retroarch but not been successful

You can already do that by specifying halt points, if I'm remembering retroarch's titleid correctly, something like:
*!RETROARCH
*ALL
ur0:tai/plugin.suprx

will exclude all following plugins from loading in retroarch.
What this patch adds is an alternative to *ALL named *GAME that only injects plugins into titles with an ID formed like an officall sony title, e.g. PCS[A-F][0-9][0-9][0-9][0-9][0-9] which can be used as such:
*GAME
ur0:tai/plugin.suprx

The patch was never merged for whatever reason (even though it works perfectly fine and is useful), so if you want it you'll need to compile taihen with this PR manually applied. Otherwise, for your usecase @Durelle, taihen can already exclude a certain title from loading any further plugins. This was mainly intended to simplify config files so you didn't need 100 lines exlcuding system apps and another sixteen or so excluding homebrew before an *ALL section.

Thanks for the contribution. I am poorly placed to decide what is convenience for users in this regard, hence the hesitation with this pull request. I've had few (zero) complaints regarding the problem this code solves.
Can you detail to me the use case for this? An example would be great.

@DaveeFTW this was at a moment when the VSH menu was only working in games and not in system apps.
But we will soon have a better VSH menu.
So you can close this request. Adding *GAME is not useful.

Most plugins are simply better at bailing out on unsupported applications now, so the original use case is somewhat gone.
In retrospect it does raise questions. Should there be a *SYSTEM specifier for NPXS and co? *HOMEBREW?
Or rather, if this sort of complexity is needed, why not just turn the titleid into a regex and be done? It's not like configs would break from that, you could just slap
*PCS[A-I][0-9][0-9][0-9][0-9][0-9]
ur0:tai/plugin.suprx

in the file and be done, and no existing configs would brek.
I'm fine with leaving what happens to this PR to Davee rather than closing it myself. I do think myself that plugins being less crap lately weakens any argument for inclusion, and something like interpreting regular expressions would be far more useful than this, which is kind of a hack imho.
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Most plugins are simply better at bailing out on unsupported applications now, so the original use case is somewhat gone.
In retrospect it does raise questions. Should there be a *SYSTEM specifier for NPXS and co? *HOMEBREW?
Or rather, if this sort of complexity is needed, why not just turn the titleid into a regex and be done? It's not like configs would break from that, you could just slap
*PCS[A-I][0-9][0-9][0-9][0-9][0-9]
ur0:tai/plugin.suprx

in the file and be done, and no existing configs would break.
I'm fine with leaving what happens to this PR to Davee rather than closing it myself. I do think myself that plugins being less crap lately weakens any argument for inclusion, and something like interpreting regular expressions would be far more useful than this, which is kind of a hack imho.


Moving modules directory from outside configurator to inside. Picked …

…the bin files from nfp-integration branch.

Changes are safe to merge.


Create tree-sitter-swift-tags.scm

from: https://github.com/alex-pinkus/tree-sitter-swift/blob/main/queries/tags.scm
Added swift lang support.
I want to make --watch-files work and i'm also going to make PR to grep-ast

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

related to: https://github.com/Aider-AI/grep-ast/compare/main...DmitriiChikovinskii:grep-ast:patch-1
and: https://github.com/Aider-AI/aider/issues/2376

Thanks for your interest in aider and for filing this PR. Unfortunately the tree sitter package we are using doesn't support swift.


Explicitly call out that reuse is not implemented yet.

Resolves #170

@zachnew-msft the paragraph starting "Finally, you can specify that the DataSourceSet should take ownership of the" is also not correct

@zachnew-msft the paragraph starting "Finally, you can specify that the DataSourceSet should take ownership of the" is also not correct

I have a note right after it that it is not fully implemented and false is not supported. The API is there.

@zachnew-msft the paragraph starting "Finally, you can specify that the DataSourceSet should take ownership of the" is also not correct

I have a note right after it that it is not fully implemented and false is not supported. The API is there.

Sorry I referenced the wrong part. The sentences beginning with "Similar to how the DataSourceSet
can take ownership of the PluginSet, the Engine takes ownership of the
DataSourceSet." is incorrect. It references EngineCreateInfo.OwnsDataSource, which does not exist

@zachnew-msft the paragraph starting "Finally, you can specify that the DataSourceSet should take ownership of the" is also not correct

I have a note right after it that it is not fully implemented and false is not supported. The API is there.

Sorry I referenced the wrong part. The sentences beginning with "Similar to how the DataSourceSet can take ownership of the PluginSet, the Engine takes ownership of the DataSourceSet." is incorrect. It references EngineCreateInfo.OwnsDataSource, which does not exist

Thanks! I will fix it.


Fixes #72 - Show error messages

Pull request submitted to https://github.com/liferay/liferay-themes-sdk/pull/118. :octocat: Sent from GH.

There were some merge conflicts, so I've rebased and resent to https://github.com/liferay/liferay-themes-sdk/pull/118


Update InMemoryGetValue sample app

Addresses #1950
:steam_locomotive: Takes sample to 2.0

@guardrex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Cannot debug Azure function with IntelliJ IDEA

I was trying this document to debug Azure function code with IntelliJ IDEA. It can be run the function locally but it doesn't stop at breakpoints. Debugging port seem ok? I can see port 5005 being logged.
Below is complete logs from IntelliJ
Extension bundle specified, skip install extension
Using port : 7071
Azure Functions Core Tools (3.0.2931 Commit hash: d552c6741a37422684f0efab41d541ebad2b2bd2)
Function Runtime Version: 3.0.14492.0
[2020-10-15T07:29:00.019] File 'C:\Program Files\dotnet\dotnet.exe' is not found, 'dotnet' invocation will rely on the PATH environment variable.
[2020-10-15T07:29:01.803] File 'C:\Program Files\dotnet\dotnet.exe' is not found, 'dotnet' invocation will rely on the PATH environment variable.
[2020-10-15T07:29:02.238] File 'C:\Program Files\dotnet\dotnet.exe' is not found, 'dotnet' invocation will rely on the PATH environment variable.
[2020-10-15T07:29:02.387] File 'C:\Program Files\dotnet\dotnet.exe' is not found, 'dotnet' invocation will rely on the PATH environment variable.
[2020-10-15T07:29:02.422] Worker process started and initialized.
[2020-10-15T07:29:02.458] Listening for transport dt_socket at address: 5005
Hosting environment: Production
Content root path: C:\Users\XXXXXX\AppData\Local\Temp\azure-functions9228867168306650650
Now listening on: http://<IP_ADDRESS>:7071
Application started. Press Ctrl+C to shut down.
Functions:
HttpTrigger-Java: [GET,POST] http://localhost:7071/api/HttpTrigger-Java

For detailed output, run func with --verbose flag.
[2020-10-15T07:29:03.049] Executing 'Functions.HttpTrigger-Java' (Reason='This function was programmatically called via the host APIs.', Id=7555bbbd-e074-46ec-9efc-7b41e8e774f5)
[2020-10-15T07:29:03.125] Java HTTP trigger processed a request.
[2020-10-15T07:29:03.132] Function "HttpTrigger-Java" (Id: 7555bbbd-e074-46ec-9efc-7b41e8e774f5) invoked by Java Worker
[2020-10-15T07:29:03.174] Executed 'Functions.HttpTrigger-Java' (Succeeded, Id=7555bbbd-e074-46ec-9efc-7b41e8e774f5, Duration=136ms)
[2020-10-15T07:29:07.474] Host lock lease acquired by instance ID '0000000000000000000000007FC87E52'.
For now I have to use Eclipse remote debugging (run local function with mvn first) as a workaround, but definitely 1-click debugging in IntelliJ  is much more convenient.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ca90dae1-a7bb-b8f5-3a23-75f4ff9c4137
Version Independent ID: aec1302d-df09-ef04-6d31-2b1cb633229f
Content: Create an Azure function with Java and IntelliJ
Content Source: articles/azure-functions/functions-create-maven-intellij.md
Service: azure-functions
GitHub Login: @jeffhollan
Microsoft Alias: jehollan

@EthanNguyen132 Thank you for your feedback! We will review and update as appropriate.

Hello @EthanNguyen132, regarding the File 'C:\Program Files\dotnet\dotnet.exe' is not found, 'dotnet' invocation will rely on the PATH environment variable. . Looks like dotnet is not include in the PATH environment variable. You can follow these steps to set it up.
Regarding the issue with setting up a breakpoint, I think it is related to IntelliJ Platform. You can post your query on their forum. I am now proceeding to close this issue out. Please tag me in your reply if there are any additional concerns.

Please reopen. @ChaitanyaNaykodi-MSFT, I have the same experience, i.e. following the steps in https://docs.microsoft.com/en-us/azure/azure-functions/functions-create-maven-intellij verbatim, I am able to generate a function app project in Intellij and successfully run it locally, but despite what is stated in the docs, breakpoints are never hit. Breakpoints in Intellij work fine, just not with projects generated by the Azure Toolkit for IntelliJ plugin.

C:\Program Files\dotnet\dotnet.exe' is not found, 'dotnet' invocation will rely on the PATH environment variable

I'm running Java function anyway. How is that dotnet has anything to do with the Java debugger :D
I'm able to debug with Eclipse.

Please reopen. @ChaitanyaNaykodi-MSFT, I have the same experience, i.e. following the steps in https://docs.microsoft.com/en-us/azure/azure-functions/functions-create-maven-intellij verbatim, I am able to generate a function app project in Intellij and successfully run it locally, but despite what is stated in the docs, breakpoints are never hit. Breakpoints in Intellij work fine, just not with projects generated by the Azure Toolkit for IntelliJ plugin.

@ChaitanyaNaykodi-MSFT  Did you manage to solve this problem? I am also facing exactly same problem, not able to debug azure function locally?

Hello,
I have the same problem. I can start the function from Intellij, but then the @Bean registered is not found in the applicationContext, and a NullPointerException is thrown.
]  :: Spring Boot ::       (v3.0.0-SNAPSHOT) [2022-09-16T11:25:36.562Z] 2022-09-16T13:25:36.561+02:00  INFO 45549 --- [pool-2-thread-4] o.s.boot.SpringApplication               : Starting application using Java <IP_ADDRESS> on xxx.local with PID 45549 (/private/var/folders/dv/kmf67v1x1ll0gtb6tkg645_nqf_jrr/T/azure-functions4524586258428149717/lib/spring-cloud-function-web-4.0.0-20220914.102052-449.jar started by xxx in /private/var/folders/dv/kmf67v1x1ll0gtb6tkg645_nqf_jrr/T/azure-functions4524586258428149717) [2022-09-16T11:25:36.562Z] 2022-09-16T13:25:36.562+02:00  INFO 45549 --- [pool-2-thread-4] o.s.boot.SpringApplication               : No active profile set, falling back to 1 default profile: "default" [2022-09-16T11:25:37.125Z] 2022-09-16T13:25:37.125+02:00  INFO 45549 --- [pool-2-thread-4] o.s.boot.SpringApplication               : Started application in 0.815 seconds (process running for 14.102) [2022-09-16T11:25:37.162Z] Executed 'Functions.echo' (Failed, Id=0b342d41-5ef7-4685-8de0-c484f5044d95, Duration=1272ms) [2022-09-16T11:25:37.162Z] System.Private.CoreLib: Exception while executing function: Functions.echo. System.Private.CoreLib: Result: Failure [2022-09-16T11:25:37.162Z] Exception: NullPointerException: Cannot invoke "org.springframework.cloud.function.context.catalog.SimpleFunctionRegistry$FunctionInvocationWrapper.apply(Object)" because "function" is null [2022-09-16T11:25:37.162Z] Stack: java.lang.reflect.InvocationTargetException [2022-09-16T11:25:37.162Z] 	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) [2022-09-16T11:25:37.162Z] 	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) [2022-09-16T11:25:37.162Z] 	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) [2022-09-16T11:25:37.162Z] 	at java.base/java.lang.reflect.Method.invoke(Method.java:568) [2022-09-16T11:25:37.162Z] 	at com.microsoft.azure.functions.worker.broker.JavaMethodInvokeInfo.invoke(JavaMethodInvokeInfo.java:22) [2022-09-16T11:25:37.162Z] 	at com.microsoft.azure.functions.worker.broker.EnhancedJavaMethodExecutorImpl.execute(EnhancedJavaMethodExecutorImpl.java:55) [2022-09-16T11:25:37.162Z] 	at com.microsoft.azure.functions.worker.broker.JavaFunctionBroker.invokeMethod(JavaFunctionBroker.java:61) [2022-09-16T11:25:37.162Z] 	at com.microsoft.azure.functions.worker.handler.InvocationRequestHandler.execute(InvocationRequestHandler.java:33) [2022-09-16T11:25:37.162Z] 	at com.microsoft.azure.functions.worker.handler.InvocationRequestHandler.execute(InvocationRequestHandler.java:10) [2022-09-16T11:25:37.162Z] 	at com.microsoft.azure.functions.worker.handler.MessageHandler.handle(MessageHandler.java:45) [2022-09-16T11:25:37.162Z] 	at com.microsoft.azure.functions.worker.JavaWorkerClient$StreamingMessagePeer.lambda$onNext$0(JavaWorkerClient.java:92) [2022-09-16T11:25:37.162Z] 	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539) [2022-09-16T11:25:37.162Z] 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) [2022-09-16T11:25:37.162Z] 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) [2022-09-16T11:25:37.162Z] 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) [2022-09-16T11:25:37.162Z] 	at java.base/java.lang.Thread.run(Thread.java:833) [2022-09-16T11:25:37.162Z] Caused by: java.lang.NullPointerException: Cannot invoke "org.springframework.cloud.function.context.catalog.SimpleFunctionRegistry$FunctionInvocationWrapper.apply(Object)" because "function" is null [2022-09-16T11:25:37.162Z] 	at org.springframework.cloud.function.adapter.azure.FunctionInvoker.handleRequest(FunctionInvoker.java:128) [2022-09-16T11:25:37.162Z] 	at azurefunctions.EchoAzureHandler.execute(EchoAzureHandler.java:26) [2022-09-16T11:25:37.162Z] 	... 16 more


Generate scripts for local and Docker flow

This PR adds _scripts target, which generates two scripts, for local flow (using orfs) and Docker flow (using docker_shell.sh).
To follow https://github.com/The-OpenROAD-Project/bazel-orfs/pull/27#issuecomment-2069770213, scripts are generated with _scripts target, and _make targets depends on _scripts apart from building all required dependencies for the stage.
Moreover, docker_shell can detect whether bazel-orfs directory has to be mounted and automatically adds necessary argument to docker run. Therefore, no additional changes will be required, when bazel-orfs is used with local_path_override.

I renamed targets according to the comment


How to enable Cors for Blazor WebAssembly app created by VS2022, the project does not have app (app.UseCors)

For example,  how do we do
app.UseCors(MyAllowSpecificOrigins);
in a VS2022 WebAssembly app. It does not have Startup.cs and app.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 59bca5db-1196-2a8b-a0d8-4e89b2382288
Version Independent ID: d3f332c4-4c60-039e-0e1e-bc6b11831f87
Content: Enable Cross-Origin Requests (CORS) in ASP.NET Core
Content Source: aspnetcore/security/cors.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello @LimingRenPA ... CORS is enabled for server-based web APIs, not for a client-side Blazor WebAssembly app. You can control the request aspects for the HttpClient of the Blazor WebAssembly app per the guidance at ...
https://learn.microsoft.com/en-us/aspnet/core/blazor/call-web-api?view=aspnetcore-7.0&pivots=webassembly#httpclient-and-httprequestmessage-with-fetch-api-request-options
... but the server app has to issue the correct headers to make CORS work. That's where the UseCors call is made.
Our CORS section that addresses the server CORS requirements in the Blazor docs is at ...
https://learn.microsoft.com/en-us/aspnet/core/blazor/call-web-api?view=aspnetcore-7.0&pivots=webassembly#cross-origin-resource-sharing-cors
... which points out that in the default case a hosted Blazor WebAssembly app's Server project doesn't need special config because the client and server are on the same origin. If they differ in origin, then yes ... the Server project requires CORS config to permit its Client app on a different origin to make requests to it (or any other client for that matter).


Full screen Dialogs?

Is there any way to define a full-screen dialog using this library?
http://www.google.com/design/spec/components/dialogs.html#dialogs-full-screen-dialogs
Thank you!

#241

A full screen dialog is an Activity.


mask loss calculation

https://github.com/dbolya/yolact/blob/6e22d1b1ccbeab4001bc6acd9124f5a679083d21/layers/modules/multibox_loss.py#L524-L525
I'm wondering, since there is already sigmoid activation for the predicted mask, why you still clamp them to 0-1? thanks!

See the discussion in #42. Specifically @andyhahaha's comments.


sf org display uses 15digit org ids

Summary
When using sf org display command it uses 15 digit org ID and in a situation when two orgs exist (e.g. 00D3M00000017eF and 00D3M00000017EF) command seems to use the org that has been created last.
Steps To Reproduce
Cannot reproduce it really unless you happen to generate two scratch orgs (which I did).
Using command it actually shows a mash-up of both orgs (e.g. I know that Instance URL being shown is that of the org that hasn't been deleted yet; username being shown is that of the valid org but signup username is that of the deleted org).
System Information
Tried on both Powershell and GitBash (Windows).
{
  "cliVersion"<EMAIL_ADDRESS>  "architecture": "win32-x64",
  "nodeVersion": "node-v18.18.0",
  "osVersion": "Windows_NT 10.0.22621",
  "shell": "C:\\Program Files\\Git\\usr\\bin\\bash.exe",
  "rootPath": "C:\\Users\\User\\AppData\\Roaming\\npm\\node_modules\\@salesforce\\cli",
  "pluginVersions": [
    "@mshanemc/plugin-streaming 1.1.7 (user)",
    "@mshanemc/sfdx-sosl 1.1.0 (user)",
    "@oclif/plugin-autocomplete 2.3.8 (core)",
    "@oclif/plugin-commands 2.2.26 (core)",
    "@oclif/plugin-help 5.2.19 (core)",
    "@oclif/plugin-not-found 2.4.1 (core)",
    "@oclif/plugin-plugins 3.8.2 (core)",
    "@oclif/plugin-search 0.0.22 (core)",
    "@oclif/plugin-update 3.2.3 (core)",
    "@oclif/plugin-version 1.3.10 (core)",
    "@oclif/plugin-warn-if-update-available 2.1.0 (core)",
    "@oclif/plugin-which 2.2.32 (core)",
    "@salesforce/analytics 1.4.11 (user)",
    "@salesforce/cli 2.11.8 (core)",
    "apex 2.3.15 (core)",
    "auth 2.8.17 (core)",
    "data 2.5.12 (core)",
    "deploy-retrieve 1.18.3 (core)",
    "info 2.6.45 (core)",
    "limits 2.3.35 (core)",
    "login 1.2.31 (core)",
    "marketplace 0.2.2 (core)",
    "org 2.10.8 (core)",
    "packaging 1.26.1 (user)",
    "schema 2.3.28 (core)",
    "settings 1.4.30 (core)",
    "sobject 0.2.7 (core)",
    "source 2.10.36 (core)",
    "telemetry 2.3.4 (core)",
    "templates 55.5.13 (core)",
    "trust 2.6.15 (core)",
    "user 2.3.32 (core)",
    "sfdmu 4.33.4 (user)",
    "shane-sfdx-plugins 4.43.0 (user)"
  ]
}

Additional information

Workaround
Log into DevHub and find the scratch org with the same ID (in my case it was a deleted org already) and delete it from "Scratch Org Infos" tab.

I am happy for this issue to be closed given I cannot reproduce the error anymore - I deleted the org and the other org has expired. I accept that I was not using the latest version of sf cli but clearly the bug was there, so hopefully it has been fixed in later versions. If not - there is a workaround.


Auto rocket jump

the auto rocket jump doesn't work I press mouse 2 it goes normal but in the second it reloads and doesn't fire I press mouse2 and keep pressing it but it doesn't fire and I'm sorry if it works is that I only tested the rocket 2 times

m2 is alt fire to stop rockets reloading in middle

It's because of it running after prediction probably, I don't really care tbh, turn off auto reload if you really want to use the feature, or better yet, learn to rocket jump.


How to build the sensor_msgs::PointCloud2 message required by the plugin?

I've tried to use pcl_conversion to convert pcl::pointcloudpcl::xyznormal into  sensor_msgs::PointCloud2, but the rviz failed with segmentation fault. Could you give a example to show how to build the sensor_msgs::PointCloud2 message required by the plugin? Or how to convert a pcl pointcloud in to  the required sensor_msgs::PointCloud2 message?

As long as the message contains position fields 'x', 'y', 'z' and normal fields 'normal_x', 'normal_y', and 'normal_z', it should work, despite how the message was generated. pcl_conversion should also work.
Can you provide several frames of your message in a rosbag that causes the crash? I'll look into it.

In my code, a .pcd file is loaded and then used for estimating normals. The normals are generated with "pcl::NormalEstimation", and can be visualized properly with pcl viewer. However, the "pointcloud2_normal_rviz_plugin" cannot work. My code is pasted here:

#include <pcl/features/normal_3d.h>
#include <pcl/filters/voxel_grid.h>
#include <pcl/io/pcd_io.h>
#include <pcl/visualization/pcl_visualizer.h>
#include <pcl_conversions/pcl_conversions.h>
#include <ros/console.h>
#include <ros/package.h>
#include <ros/ros.h>
#include <sensor_msgs/PointCloud2.h>

#include <Eigen/Core>
#include <Eigen/Geometry>
#include <iostream>

ros::Publisher sample_points_pub;
std::shared_ptr<pcl::visualization::PCLVisualizer> viewer;

int main(int argc, char **argv) {
  ros::init(argc, argv, "sample_points_publisher");
  ros::NodeHandle nh;

  sample_points_pub =
      nh.advertise<sensor_msgs::PointCloud2>("sample_points", 10000);

  pcl::PointCloud<pcl::PointXYZ> cloud, cloud_filtered;
  pcl::io::loadPCDFile("debris_cut.pcd", cloud);
  pcl::VoxelGrid<pcl::PointXYZ> sor;
  sor.setInputCloud(cloud.makeShared());
  sor.setLeafSize(0.2f, 0.2f, 0.2f);
  sor.filter(cloud_filtered);

  pcl::PointCloud<pcl::Normal> normals;
  pcl::NormalEstimation<pcl::PointXYZ, pcl::Normal> ne;
  pcl::search::KdTree<pcl::PointXYZ>::Ptr tree(
      new pcl::search::KdTree<pcl::PointXYZ>());
  tree->setInputCloud(cloud_filtered.makeShared());
  ne.setInputCloud(cloud_filtered.makeShared());
  ne.setSearchSurface(cloud_filtered.makeShared());
  ne.setRadiusSearch(0.5);
  ne.setSearchMethod(tree);
  ne.compute(normals);
  std::cout << "normals size = " << normals.size() << std::endl;

  pcl::PointCloud<pcl::PointNormal> normal_pcd;
  pcl::concatenateFields(cloud_filtered, normals, normal_pcd);

  viewer = std::shared_ptr<pcl::visualization::PCLVisualizer>(
      new pcl::visualization::PCLVisualizer("Viewer"));
  viewer->setBackgroundColor(0, 0, 0);
  viewer->addPointCloud<pcl::PointNormal>(normal_pcd.makeShared(), "foo");
  viewer->addPointCloudNormals<pcl::PointNormal, pcl::PointNormal>(
      normal_pcd.makeShared(), normal_pcd.makeShared(), 25, 0.15,
      "normals");

  while (ros::ok()) {
    viewer->spinOnce();
    if (normals.size() > 0) {
      sensor_msgs::PointCloud2 pcd_msg;
      pcl::toROSMsg(normal_pcd, pcd_msg);
      pcd_msg.header.frame_id = "map";
      pcd_msg.header.stamp = ros::Time::now();
      sample_points_pub.publish(pcd_msg);
    }
  }
}

I tried your code using one PCD file in the tutorial capture0001.pcd and it works fine. The only modification I made is to change line 26 to
  pcl::io::loadPCDFile(argv[1], cloud);

The result looks like

Have you tried other PCD files? Maybe you can upload debris_cut.pcd somewhere so that I can have a look into that file.

When i use the tutorial capture0001.pcd, it still not works. Note that i run the code on ubuntu18.04 and ros-melodic. When compling the rviz plugin, it failed with errors because <rviz/ogre_helpers/ogre_vector.h and version_check.h> were not found. To solve it, I commented the headers and passed the compilation. However, only an arrow can be shown in rviz. Is there somthing wrong with my environments?

The plugin was developed and tested on Ubuntu 20. I'll take a look.

There is some difference in RViz and PCL between melodic and noetic. Removing those two headers should be a valid workaround, but I do not have a physical machine running Ubuntu 18, and in my VM the plugin consistently crashes with "Cannot create GL vertex buffer" message, which I found could be a OpenGL driver issue. In my tests disabling 3D acceleration in VM settings makes the situation worse.
Since the performance in my VM is extremely inconsistent, I cannot further examine and fix this. You may update your environments and try if the plugin works in your situation under Ubuntu 20.


Protractor configuration file for the frontend

This is a basic config that allows to run the tests. Subsequent pull
requests will use it in Gulp tasks to run protractor tests over
development and production code.

LGTM


non-Clifford CNOT-Dihedral RB

Summary
#256
Details and comments
RB of the CNOT-dihedral group.
code is based on the following paper:
https://arxiv.org/abs/1510.02720
https://www.nature.com/articles/npjqi201612

Still some lint issues to fix

I added an on hold label here to block merging while we have some debug settings in the test job: https://github.com/Qiskit/qiskit-ignis/pull/296/commits/4b9f67ba7f797146f098c3f9e27c41758b99ca77 I'll remove that label when we revert that change

It seems that the error in the tests is in Python 3.7 and not in Python 3.6,
and is related to reading of a pickle file of a table which is in a dict format:
Captured traceback:
~~~~~~~~~~~~~~~~~~~
    b'Traceback (most recent call last):'
    b'  File "d:\\a\\1\\s\\.tox\\py37\\lib\\site-packages\\ddt.py", line 145, in wrapper'
    b'    return func(self, *args, **kwargs)'
    b'  File "D:\\a\\1\\s\\test\\rb\\test_rb.py", line 534, in test_rb'
    b'    **rb_opts_nonclifford)'
    b'  File "D:\\a\\1\\s\\qiskit\\ignis\\verification\\randomized_benchmarking\\circuits.py", line 256, in randomized_benchmarking_seq'
    b'    Elmnts[rb_pattern_index], new_elmnt_gatelist)'
    b'  File "D:\\a\\1\\s\\qiskit\\ignis\\verification\\randomized_benchmarking\\dihedral_utils.py", line 273, in compose_gates'
    b'    G_table = self.load_tables(self._num_qubits)'
    b'  File "D:\\a\\1\\s\\qiskit\\ignis\\verification\\randomized_benchmarking\\dihedral_utils.py", line 159, in load_tables'
    b"    picklefile='cnot_dihedral_%d.pickle' % num_qubits)"
    b'  File "D:\\a\\1\\s\\qiskit\\ignis\\verification\\randomized_benchmarking\\dihedral_utils.py", line 134, in load_dihedral_table'
    b'    return pickle.load(pf)'
    b'EOFError: Ran out of input'
    b''

Any suggestions?

I'm trying to add to the rb test directory the pickled tables of the clifford and non-clifford groups

I'll try to change the format of the 2-qubit non-clifford group table (which is a dict) from pickle to json

It seems that the group table for the 2q non-clifford group are quite complicated (it's a dict where the keys are complicated objects), so it's not possible to save them as json, and saving them as pickle cause problems where there are several Python versions. So, I'm now trying to save them in dill format:
https://pypi.org/project/dill/

Is there a way to install dill as part of the testing environment?

The group table of the CNOT-Dihedral group is a dict whose keys are objects (CNOTDihedral objects).
Since we should save this table to make the group calculations, I save it in pickle format.
To make the format of the table more consistent across python versions I changed the keys of a CNOTDihedral objects to from tuple to string, and the hash to hashlib.md5().
It seems that currently the tests work for windows Python 3.6 and windows Python 3.7, but fail in Travis Python 3.5 (which should be deprecated?).
Any suggestions? Other formats that I use?

I will try to find a better data structure for the group, which may be more easily pickled.

Finally all the tests have passed!
@mtreinish - Thank you very much for your help. Could you please remove your commit 4b9f67b and the "on hold" label?
@dcmckayibm - Could you please review the code?

It seems pretty slow to generate the circuits

In the example notebook I would show how to  get the alpha_z + alpha_R params

It seems pretty slow to generate the circuits

Indeed, it's since the CNOT-Dihedral group table is a more complicated data structure than the Clifford table.
I didn't make too many changes in the original code. I think that you can merge and we can discuss future improvements later.


fix: ApiManagement-Service - Changed loggers to logger

Description

Changing folder name to singular to align with AVM spec: resource module naming
Fixed incorrect description. See failed test here

Pipeline Reference

Pipeline

Type of Change

[ ] Update to CI Environment or utilities (Non-module affecting changes)
[ ] Azure Verified Module updates:

[x] 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.
[ ] Breaking changes and I have bumped the MAJOR version in version.json.
[ ] Update to documentation

Hey @tony-box, could you give this PR a spin some time? 💪

Hey @tony-box, could you give this PR a spin some time? 💪

Looks good, sir! Sorry for the delay. My dev environment is dead and I've been swamped with other work :(
Thank you for the update!

@Azure/avm-module-reviewers-bicep whoops, i accidentally closed this without merging. I reopened and I think it reset the reviews. Can someone please re-review?

Not onyl that, but it also removed you as a reviewer 😄 A feature, I guess.


worldhopper: add Bounty Hunter option to world type filter

Adds a Bounty Hunter filter option and removes Bounty Hunter worlds from the "normal" world type filter.
Resolves #17960

PLEASE merge I need this, so annoying


Add _targetRef and _domRef modifiers

Resolves partially #231. _targetRef is a modifier that can be used by any renderer, while _domRef is an adaptation of that for DOMRenderer. Both are underscored as they are not available in SwiftUI, and also their stability is currently not so well known to us, we may consider changing this API in the future.
Example use:
struct DOMRefDemo: View {
  @State var button: JSObjectRef?

  var body: some View {
    Button("Click me") {
      button?.innerHTML = "This text was set directly through a DOM reference"
    }._domRef($button)
  }
}

I've also fixed all known line length warnings in this PR.

Awesome 👍


✏Fix s3 ipa and dsym docs

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
Fixes s3 documentation typos
Description
Fixes s3 documentation typos
Testing Steps
N/A

@googlebot I signed it!


Support branch name with space in create review step

Fix the issue in apiview pipeline step when branch name has space.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

Change looks fine but what cases allow a space in the branch name?

Change looks fine but what cases allow a space in the branch name?

It's not space. It's & in the branch name.


README needs to have Gemfile and Gemfile.lock

It will be great help for community if you can share Gemfile and Gemfile.lock, so user won't have any issues running this project.
Currently I'm blocked because of https://github.com/burtlo/yard-cucumber/issues/80

I see a Gemfile and Gemfile.lock in the project. I know that the yard-cucumber.gemspec really defines what was currently defined as a dependency of the project. If I'm incorrect and you need something more, please re-open the issue. I'll take a look at #80 right now.


Test resource deletion prevention via WebHook

What: We need a way to prevent deletion of resources when uninstalling packages
Why: To avoid deleting resource in the cloud when/if we need to rollback a given package

[ ] Talk with Matthias @ Upbound about his WebHook solution for preventing deletion
[ ] Tested out suggested strategy
[ ] Present for team

Matthias will prepare a sample solution for this. Talk with him before beginning any work on this issue

deleting CRD causes deletion of resources that depend on the CRD

awaiting Matthias from Upbound to get back with an example

alternative solution that can be investigated kyverno or our own webhook

we will solve this problem by extending compositions to expose the deletePolicy with a sensible default of "Orphan" when we feel its required


openpose json data

Hi,
Would it be possible to also output the openpose json data?
I would sometimes like to adjust the detected pose when it gets something wrong in the openpose editor, but currently I can only estimate and rebuild the pose from the image.
Thanks in advance!

@BurningSpy I just added Openpose JSON in Openpose and DWPose nodes' outputs. Check here.


pugixml: Fix pkg-config file installation and use fetchFromGitHub

Motivation for this change
FileZilla was not building (#56505) and tracked it down to a pugixml change (#56351).
Tested this by building FileZilla.
cc @pSub
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg build filezilla


etcd panic if cluster fetched from remote peers doesn't contain its name

When I try to bring up my either follower with these command arguments
-name Duchess -advertise-client-urls http://<IP_ADDRESS>:4001 -data-dir=/var/lib/etcd/ -initial-advertise-peer-urls http://<IP_ADDRESS>:7001 -initial-cluster-state existing -initial-cluster etcdLeader1=http://<IP_ADDRESS>:7001 -listen-client-urls http://<IP_ADDRESS>:4001,http://<IP_ADDRESS>:4001 -listen-peer-urls http://<IP_ADDRESS>:7001,http://<IP_ADDRESS>:7001 -cors=*

I get the following output:
2015/02/05 18:11:23 etcd: listening for peers on http://<IP_ADDRESS>:7001
2015/02/05 18:11:23 etcd: listening for peers on http://<IP_ADDRESS>:7001
2015/02/05 18:11:23 etcd: listening for client requests on http://<IP_ADDRESS>:4001
2015/02/05 18:11:23 etcd: listening for client requests on http://<IP_ADDRESS>:4001
2015/02/05 18:11:23 etcdserver: name = Duchess
2015/02/05 18:11:23 etcdserver: data dir = /var/lib/etcd/
2015/02/05 18:11:23 etcdserver: snapshot count = 10000
2015/02/05 18:11:23 etcdserver: advertise client URLs = http://<IP_ADDRESS>:4001
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x4dcf5d]

This happens if user starts etcd wrong:

it starts etcd with -initial-cluster-state=existing and -initial-cluster that doesn't contains its own name
etcd checks the cluster info from the remote peer, and it fits the initial-cluster well
etcd tries to get the local info using its name from the cluster info, and it panics

I think this has been fixed by join existing check.

@yichengq I tried and confirmed this is fixed.


Compatibility with file type icons

With file type icons

Screenshot edited for indentation to make sense hierarchically

Keep folder icons hidden, and un-hide & indent files to make sense with icons ?

fixed in #185


Add missing events for admin-related methods

closes #1173

Thanks a lot for your PR! clap I had started some work yesterday already to speed this up but wasn't ready to push. You may want to eyes at #1179 for inspiration or you review mine and we close this.
Thank you for your comment @alpe  and i am fine whichever option you choose. If you wanna close this PR, I will review yours if it need it or I will add tests for this PR If this PR continue.

I will close this PR now in favour of #1179 just to speed up the process for v0.31 since you had no strong opinion about the implementation.
I appreciate your contribution very much and I am sorry for the duplicate work. Please keep adding issues and PRs to improve the project 🌻


[un energy] stat var names

There are a few stat vars where the names could be clarified, e.g.
https://datacommons.org/browser/Annual_Generation_Energy_Bagasse
The name is Annual Generation of Bagasse -- should it be Annual Generation of Energy from Bagasse
Perhaps the description could also be clarified, since it now read as Bagasse is being generated, not energy.
https://datacommons.org/browser/Annual_Consumption_Energy_ElectricityGeneration_Heat_FuelTransformation
The name and description here is also confusing, since it mentions both consumption and generation. Is it referring to Heat generated by Fuel Transformation consumed for Electricity Generation
Sorry these were missed during the reviews..

Thanks for pointing this out.
Prashanth indicated that the population type is dropped when the name is auto generated.
I'll update the scripts to generate names as well.

Tracked in internal bug.


Git code not tagged to correspond with npmjs.com?

This is more of a question than an issue :-)
I see on npmjs.com that the package is being updated with the work here - last publish is 1.3.8 and recently: https://www.npmjs.com/package/rn-update-get-sms-android
However, I don't see any git tags or anything here, so I'm not sure what code is in which version? In the future could you git tag vX.Y.Z and git push --tags for the corresponding commit that you npm publish ? I would appreciate it, as I appreciate all the effort here :-). Thanks

Hey Mike. Thanks for flagging this.
I've been tagging versions through npm and not pushing them using git. I will definitely start doing this from now onwards.
Appreciate the pointers; I've managed to push them now so that's a start. 😃
https://github.com/briankabiro/react-native-get-sms-android/releases

Fantastic - thanks


Calling asList() on 3d MultiDataSet gives list of 2d MultiDataSets

I was trying to split a batch of time series into individual time series data sets, and used asList(). Strange stuff happened and I figured out that the resulting objects had 2D arrays instead of 3D. A bunch of stuff breaks down at this point, like DataSetUtils that doesn't look at masks anymore when tailoring to 2D.
The actual problem appears to be that INDArray.get(...) drops dimensions that have size 1, and this is used in in MultiDataSet::getSubsetForExample(...) for getting 1 example from the batch.
INDArray SUT = Nd4j.create(
    new float[]{1, 2, 3, 4, 5, 6, 7, 8},
    new int[]{2, 2, 2}
);
System.out.println(SUT.rank());

INDArray slice = SUT.get(NDArrayIndex.point(0), NDArrayIndex.all(), NDArrayIndex.all());
System.out.println(slice.rank());

Will output
3
2

I mentioned this on gitter and @AlexDBlack was pretty sure this is a bug, so here.

Thanks. Fixed here: https://github.com/deeplearning4j/nd4j/pull/1458


python312Packages.openai: 1.56.1 -> 1.57.4

Update to latest version. No changes that effect build.
Diff: https://github.com/openai/openai-python/compare/refs/tags/v1.56.1...v1.57.4
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.