transforms: Invert

Hi!
I had written a transform that inverts grayscale images from a custom dataset for my own use.
Since I believe this might be useful to others I wanted to share this extended version which accepts PIL images of modes "L", "LA", "RGB" and "RGBA".
I have further written a unit test which is included in the PR.
A simple demo is available here.

@sebastianberns looks good, but could you please provide more background on benefits of invert applied on RGB images ?

In any case, where the information of interest in a pixel image is placed on a white background this function would be helpful for data preprocessing. We want a black background since a convolution by default adds zeros all around as padding.
While this is obvious for grayscale (MNIST) this similarly holds for any number of channels. (Thinking about this now, the invert function could be rewritten to simply invert all channels but the alpha channel.)
Imagine fashion-MNIST in color, or image renders of Shapenet (although here it might be trivial to place an object on black instead of white background).
That being said, I personally use the invert function rather to manipulate the background than the foreground.

@sebastianberns sorry for delayed answer.  I agree that this could make sense on gray-level objects and where background is homogenious and white. I still have a doubt on inversion of RGB images. Take as example your demo, where a frog is inverted. In such cases network will see green, brownish frogs  and as well as blue and magenta ones and probably it will learn only geometric features rather than color features to distiguish frogs.

Please, take as much time as you need for reviews and replies.
Also, please understand this PR as my way of sharing what I thought useful. Ultimately it shall be the maintainers’ decision to add any feature or not. I’m not familiar with the corresponding policies.
My demo is an illustration of the feature’s capability, not an exposition of its purpose.
Your doubts about the inversion of color images with bleed (that is to say a photo that covers the whole image map with no uniform margin) are reasonable, of course, because inverting in such a case is not necessary, and not the use case I am drawing.
Since you somehow accept this feature’s usefulness for grayscale images unquestioned, let me insist that inversion is useful because of a uniform (usually white) background, independent of the number of channels (1, L, RGB, LAB, …).
Even if the image of the frog that you reference was black and white still it would not make a difference inverting it. But image the same frog sitting on a white background instead of the ground of a forest (similarly to the data in fashion-MNIST and Shapenet I have mentioned). Then, I argue, you’d want to invert it.
If you were to reject color inversion I can offer to rewrite to only allow single-channel images.

In the meantime, I have set up a separate repository to save this custom transform for future reference.
https://github.com/sebastianberns/torchvision.transforms.invert

I'm not convinced about this transformation.
Indeed, in RGB images, inverting the colors changes the color distribution and makes a pre-trained CNN completely fail.
If the main use-case is to invert white color background into black, this would be better as a user layer for a particular application, but I'm not 100% sure that it's general enough to be merged in torchvision.
Thanks for the PR though!


Unhandled rejection for tests that starts with about:blank page

If you specify unhandled rejections handler in node:
process.on('unhandledRejection', function(reason){
    console.log(reason.message);
});

You will get Protocol "about:" not supported. Expected "https:". For tests that utilizes about:blank page. This is happens because we missing return in special pages branch in Hammerhead: https://github.com/DevExpress/testcafe-hammerhead/blob/master/src/request-pipeline/index.js#L25, so we still try to perform request for such pages.
@miherlosev @churkin Please fill a corresponding issue in hammerhead repo.

I will create issue

@miherlosev Please, cross-reference them

Fixed in hammerhead, waiting for release


vinyl-fs to 2.0

We need to get vinyl-fs to 2.0 before gulp 4 is released.  There are still a number of things to complete and I made a list over at https://github.com/wearefractal/vinyl-fs/issues/92
If you can help on any of those issues, please do!

This has been completed.  However, I think we will be releasing a vinyl-fs 3.0 before the gulp 4.0 release.  I will open an issue if we need any help on that.


Feat: Improving the UI of Doctors/Therapist Profiles section.

Describe the feature
Currently when we visit the Doctors/Therapist Profiles section the two buttons view profile and schedule Appointment appear at the bottom of the photos of the doctors card instead I want to add the feature that instead of appearing in the bottom they appear on the photos of the doctors whenever the user hover on them, this provides a more dynamic and realistic feel and overall good user experience.
So Initially there will be no profile and schedule appointment button but as soon as the user hovers on the pic of the doctor, the photo will zoom a little providing a dynamic motion and profile and schedule Appointment button will Popup.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'23 contributor
[X] I want to work on this issue

@MonalikaPatnaik can you please review this asap.

The concept looks good but you need to improve color scheme of button if you are planning them on the image

What color Scheme do you Suggest ? maybe we can make the button more transparent and the buttons will not be permanently present on the image they will only appear when the user will hover on the image.


AttributeError while installing CVAT

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

I followed the official CVAT installation guide - first cloned the git repository, installed latest docker-compose via pip (pip install docker-compose --upgrade), then wanted to start via:
$ docker-compose up -d
Traceback (most recent call last):
  File "/usr/local/bin/docker-compose", line 5, in <module>
    from compose.cli.main import main
  File "/usr/local/lib/python3.10/dist-packages/compose/cli/main.py", line 19, in <module>
    from ..config import ConfigurationError
  File "/usr/local/lib/python3.10/dist-packages/compose/config/__init__.py", line 3, in <module>
    from .config import ConfigurationError
  File "/usr/local/lib/python3.10/dist-packages/compose/config/config.py", line 48, in <module>
    from .validation import match_named_volumes
  File "/usr/local/lib/python3.10/dist-packages/compose/config/validation.py", line 8, in <module>
    from jsonschema import Draft4Validator
  File "/usr/lib/python3/dist-packages/jsonschema/__init__.py", line 11, in <module>
    from jsonschema.exceptions import (
  File "/usr/lib/python3/dist-packages/jsonschema/exceptions.py", line 152, in <module>
    @attr.s(hash=True)
AttributeError: module 'attr' has no attribute 's'

Any ideas what could be the problem here?
 Please try to reinstall docker-compose. Be sure that you can run it. It is definitely not a problem with CVAT.

@nmanovic I am trying to install CVAT on an ARM machine.
Could that be the issue - do you support that architecture?


[bug] URI Links open a new instance of Dogecoin QT rather than existing (running) instance

Bug Report
Current behavior:

[Windows] After 1.14.8 update, URI links launch a new instance of QT rather than opening the existing instance.
Edit: This appears to persist in 1.14.7 across multiple tests on different machines
Expected behavior

URI (deep) links should find an existing instance of dogecoin-qt.exe to open URI path.
Steps to reproduce:

Open 1.14.8 dogecoin-qt.exe
Click any URI link on Chrome:
dogecoin:DSh2kvyQ8Jkvu9cCf2kuQKm7D98cR9NtG4
Windows opens new instance of QT and outputs error (see below)

Screenshots

Dogecoin Core version

1.14.8
Machine specs

OS: Windows 11 Home
CPU: Ryzen 7 3750H
RAM: 32
Disk size: 1.5 TB
Disk Type (HD/SDD): SSD

Extra information

Edit: Windows RegEdit HKCR URI open command @ Computer\HKEY_CLASSES_ROOT\dogecoin\shell\open\command points to the QT instance that I have open "D:\Dogecoin\dogecoin-qt.exe" "%1")

Doing some research online, most people sharing URI multi-instance problems indicate that as long as the URI is properly registered w/ Windows Registry > Shell > Command the "as designed" result should be opening new instances each link click, and that the solution is to apply Single Application "shared memory" to allocate resources from URI Link clicks to the existing running app.
Adding links to online references for @chromatic:
QT "Single Instance Best Practices" stack q&a:
https://stackoverflow.com/questions/5006547/qt-best-practice-for-a-single-instance-app-protection
C++ / QT "Force Application to open only one exe" stack q&a:
https://stackoverflow.com/questions/4087235/how-to-force-my-application-to-open-one-exe-only-qt-linux
Note: I think 1.14.8 QT already enforces a single instance hence shutting down on URI links
C# "Single Instance URI handling sample" blog:
https://www.brad-smith.info/blog/archives/842

Found the issue:
paymentserver snip generates an ipcServerName string to reference on URI file handling:
`static QString ipcServerName()
{
QString name("DogecoinQt");
// Append a simple hash of the datadir
// Note that GetDataDir(true) returns a different path
// for -testnet versus main net
QString ddir(GUIUtil::boostPathToQString(GetDataDir(true)));
name.append(QString::number(qHash(ddir, IPC_SOCKET_HASH)));

return name;

}`
The string generates a server name hash unique to the datadir for the instance.  If a user installs multiple versions with shared directories, QT is unable to find the ipc server and opens a new instance via the URI registration in Computer\HKEY_LOCAL_MACHINE\SOFTWARE\Classes\dogecoin\shell\open\command

I think the problem is logic:

Init -> ipcSendCommandLine()
Socket -> ipcServerName()
ipcServerName() -> rand INT IPC_SOCKET_HASH
IF ipcSendCommandLine() = False -> initializeServer()
initializeServer() -> create server

Step 4 will always result in "False" because each URI click generates a new ipcServerName() via rand INT appending and can never connect to the listen server..
Did this not fail before because, prior to QT5, there were ifdefs preventing the rand INT IPC_SOCKET_HASH from being used until 1.14.8?

I think the problem is logic:

Init -> ipcSendCommandLine()
Socket -> ipcServerName()
ipcServerName() -> rand INT IPC_SOCKET_HASH
IF ipcSendCommandLine() = False -> initializeServer()
initializeServer() -> create server

Step 4 will always result in "False" because each URI click generates a new ipcServerName() via rand INT appending and can never connect to the listen server..
Did this not fail before because, prior to QT5, there were ifdefs preventing the rand INT IPC_SOCKET_HASH from being used until 1.14.8?

you should review this pr bruh. https://github.com/dogecoin/dogecoin/pull/3466

@xanimo are you asking me to review the PR because the issue is contained in that PR and should be resolved with the changes in it,
or are you telling me that this appears to be a real issue that affects the PR and I should review it in that context?

@xanimo are you asking me to review the PR because the issue is contained in that PR and should be resolved with the changes in it, or are you telling me that this appears to be a real issue that affects the PR and I should review it in that context?

i'm saying your input would be appreciated on that pr if you happen to have the time to review. that's it <3


CSS code needs creating to make UI better

header and main, questions, etc have some style added. But need to add style to past results and clear results buttons and to list.

css updated. could use better design, but good enough.


Use namespace varible for package folder instead of hardcoding it

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Can one of the admins verify this patch?

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

@fearthecowboy can you merge?  the tools choke on the large number of commits :(


Module Account generation rule for Collection, TaxSource Account

Summary
Problem Definition
The Module Account generation method used to create an address with a simple hash of String Byte for old 20-byte lengths, and the 32-length Module Account generation option extended in adr-028 requires the module name to be received together.
Therefore, it is necessary to define rules for Tax.TaxSourceAccountName, Tax.CollectionAccountName verification and the Module account generation.
Proposal
If / is not included, create and validate with old 20-byte rules, if included, The 32-byte generation rule is applied in accordance with the format of {moduleName}/{moduleAccountName}.

Bullet Points

[ ]

References

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

The decision on issue https://github.com/tendermint/tax/issues/15 has led this issue to not proceed.


Respect sampled bit in probability sampler

Fixes #787

/cc @Flarna

I think we should respect the trace flag not only in the true case.
Maybe move this check to the tracer and call the sampler only for root spans?

I think we should respect the trace flag not only in the true case.
Maybe move this check to the tracer and call the sampler only for root spans?

I don't want to move the logic to the tracer because you could theoretically want to implement a sampler that does not respect trace flags (perhaps you want to use some tracestate entry instead).

Looks good. I agree that keeping it out of tracer is the better choice.
The checks in processors seem to be redundant as unsampled spans are not forwarded to processors anyway.

@Flarna I removed the isSampled check from the span processors. I looked through the spec and don't see anywhere it says the span processor should be checking. I verified that you are correct, unsampled spans are indeed not sent to the span processor. I think whoever implemented the span processors was probably just being safe.
/cc @mayurkale22 WDYT about this decision?

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@fc287d3). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #798   +/-   ##
=========================================
  Coverage          ?   92.91%
=========================================
  Files             ?      244
  Lines             ?    10669
  Branches          ?     1048
=========================================
  Hits              ?     9913
  Misses            ?      756
  Partials          ?        0

Impacted Files
Coverage Δ

packages/opentelemetry-api/src/metrics/Metric.ts
100% <ø> (ø)

...y-api/test/noop-implementations/noop-meter.test.ts
100% <ø> (ø)

packages/opentelemetry-metrics/src/Metric.ts
95.12% <ø> (ø)

Continue to review full report at Codecov.

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


FlixScans: fix search

closes  #986
Checklist:

[ ] Updated extVersionCode value in build.gradle for individual extensions
[x] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[ ] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio
[ ] Have removed web_hi_res_512.png when adding a new extension

@AwkwardPeak7 thank you very much ❤


[CI Problem] Flaky Test: tests/python/driver/tvmc/test_compiler.py::test_compile_tflite_module_with_external_codegen_cmsisnn

Branch/PR Failing
Several instances of CI flakiness on the same unittest: tests/python/driver/tvmc/test_compiler.py::test_compile_tflite_module_with_external_codegen_cmsisnn
Jenkins Link
Some examples:

https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/2070/pipeline
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/2069/pipeline
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/2068/pipeline
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-9486/2/pipeline
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-9488/2/pipeline

Flakiness
Yes

Hi @junrushao1994,
Can you assign this to me? I have no idea why it would continue to flip and it might just be old branches intersecting so needs monitoring some.

I suspect driver_api.cc:439 is picking out the host module non-deterministically since a) can end up with two kDLCPU targets for the given and the 'default' used for shapes etc and b) map iteration order is non-det.

Thank you @Mousius @mbs-octoml for acting quickly on this :-)


feat(backend): Signing a JWT respects 'iat' claim

Description
Calling the signJwt function overrides the 'iat' (issued at) claim of the payload.
import { signJwt } from '@clerk/backend/jwt';

const key = 'the-private-key';
const payload = {
  sub: 'subject',
  // provide the issued_at claim
  iat: 123,
}
const { data } = await signJwt(payload, key, {
  algorithm: 'RS256',
  header: {
    alg: 'RS256',
    typ: 'JWT',
  }
});
data.iat // => returns the current time instead of 123.

This PR fixes the problem.
When calling the signJwt function to generate a JWT the value of the 'iat' claim in the payload will be respected, falling back to the current time if the claim is not present in the payload.

Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[x] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Is there any scenario where we want to have a IAT older then when the token was signed?

Please allow me to reverse the question a bit. Is there any reason why the caller should not be able to specify the 'iat' claim?
I expect that it's possible to generate tokens with whatever value we want for the 'iat' claim.
The ability to set the "issued at" claim could also prevent clock discrepancies between nodes of a distributed system, by guaranteeing that all chronological values are generated from a trusted/central time source.

Yep this makes 100% sense. I guess ensuring that no invalid IAT is added should be part of the code generating the JWT in the first place and not when signing it!

Clerk's SDK (in particular the Handshake and Interstitial mechanism) depend on the iat claim using the same clock as the __client_uat cookie.  What is motivating the change?

What is motivating the change?

@colinclerk I'd like to reuse the clerk/backend package for generating JWTs at Edge. I would expect that the caller should set the 'iat' claim based on their own "clock". Our FAPI currently sets 'iat' to a value with its own clock for example.

Clerk's SDK (in particular the Handshake and Interstitial mechanism) depend on the iat claim using the same clock as the __client_uat cookie.

I see. So the Clerk SDK cannot accept a clock and relies on Date on each platform that the code is executed, right? If I understand correctly you are suggesting that callers should not set the 'iat' claim on their own.
I find that kind to be kind of backwards.
In any case this change allows you to either pass the 'iat' claim or default to Date.now() which should preserve the existing behavior of the library.

After discussing with @colinclerk and @nikosdouvlis we decided to change the internal implementation of signJwt.
Closing this PR until then.


chore: rename lib/types -> lib/types-proto

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#986  👈
#974
#977
#965
#964
#963
#962
#957
#956
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MasterPtato and the rest of your teammates on  Graphite


Update the 'game over' message

update the 'game over' message so people know how to play again.

comment1


Ingress controller should reside in installation namespace

Current installation templates put ingress proxies in the application namespace (cc @andraxylia ).
I recommend we put all ingress and egress proxies into the installation namespace, as they are managed by istio and handle cluster-wide ingresses, rather than having independent identical proxies in each application namespace.
@rshriram @linsun @costinm @louiscryan

cc @dcberg

This incorrect, the ingress and egress controller are in the istio-system namespace.

Just the actual ingress resource is in the app namespace, since it is controlled by the developer.
Please check the design doc for cluster-wide, and close this issue:
https://docs.google.com/document/d/1Of0SUxNLzVQyBO9xx2fEKsN4vwyp72csbWd-8xjeLS0/edit

So the services run in istio-system, but resources must be in app namespace, for security reasons.
We need to minimize users crossing namespace boundaries - ideally no service account in user namespace should have permissions on the istio-system namespace.

Please fix the code:
https://github.com/istio/pilot/blob/master/test/integration/infra.go#L216

So this issue is about changing the pilot e2e tests, I see. Sure, it can be done differently. There were issues with running pilot e2e tests in a shared cluster, so this was a way to test cluster-wide without causing interference among various concurrent runs.
The actual templates in istio/istio are like this, and ingress is in the installation ns:
https://github.com/istio/istio/blob/master/install/kubernetes/istio-cluster-wide.yaml#L568
I am adding cluster-wide e2e tests in istio/istio, which need to be run using the isolated cluster infrastructure that only became available yesterday.


Upgrade wiped out custom mods in ~/Library/Application Support/Karabiner/private.xml

I upgraded recently and lost my mods in private.xml. I would expect upgrade to not do that. Thanks.

Karabiner does not do that.
Is your private.xml not a regular file such as link or in network filesystem?

It's a regular file. I believe it did do that because after I upgraded I
lost my custom mods in private.xml. I don't know how else it might have
happened. What was in there was contents of samples.xml.
Thanks
Mike
On 9/2/15 12:19 AM, Takayama Fumihiko wrote:

Karabiner does not do that.
Is your private.xml not a regular file such as link or in network
filesystem?
—
Reply to this email directly or view it on GitHub
https://github.com/tekezo/Karabiner/issues/473#issuecomment-136939535.

Karabiner will create private.xml by the following contents if private.xml does not exist at launch.
<?xml version="1.0"?>
<root>
</root>

If your private.xml is same as above, I guess your private.xml have been lost by disk error or other errors and Karabiner recreated it at launch.
Please consider making an external xml file and include it in private.xml.
https://pqrs.org/osx/karabiner/xml.html.en#dropbox
If your private.xml is not above, like samples.xml,
https://github.com/tekezo/Karabiner/blob/version_10.8.0/src/core/server/Resources/include/checkbox/samples.xml
I guess Time Machine or other backup softwares reverted your changes.
Please confirm your system.

I have a backup system and I was able to retrieve my private.xml from
backup without a problem. As soon as I upgraded Karabiner I lost my
custom functionality and began investigating. As soon as I restored my
private.xml from backup and did a reload xml, I regained my custom
functionality.
I just wanted to share this info I thought you'd appreciate it.
Thanks
Mike
On 9/2/15 12:38 AM, Takayama Fumihiko wrote:

Karabiner will create private.xml by the following contents if
private.xml does not exist at launch.

If your private.xml is same as above, I guess your private.xml have
been lost by disk error or other errors and Karabiner recreated it at
launch.
Please consider making an external xml file and include it in private.xml.
https://pqrs.org/osx/karabiner/xml.html.en#dropbox
If your private.xml is not above, like samples.xml,
https://github.com/tekezo/Karabiner/blob/version_10.8.0/src/core/server/Resources/include/checkbox/samples.xml
I guess Time Machine or other backup softwares reverted your changes.
Please confirm your system.
—
Reply to this email directly or view it on GitHub
https://github.com/tekezo/Karabiner/issues/473#issuecomment-136941707.

Thank you for information.
I'm glad that you backed up your private.xml and restore it properly.
If you faced same error again, let me know the contents of private.xml that has been reset.
And I recommend you to verify your disk by Disk Utility.
(If your private.xml has been vanished by disk error, you should replace the disk.)
http://support.apple.com/kb/PH5864


Issues with deployment in failed state

Connect uffizzitest docker hub account
Create preview from compose file uffizzi preview create compose.yml. File contains a mistake: The image from compose file does not exist in the docker hub.

services:
  webhooks_test_app:
    image: uffizzitest/aaaa-webhooks-test-app:latest
    environment:
      POSTGRES_USER: postgres
      POSTGRES_PASSWORD: postgres

x-uffizzi-ingress:
  service: webhooks_test_app
  port: 80

x-uffizzi-continuous-preview:
  delete_preview_after: 1h
  delete_preview_when_image_tag_is_updated: true
  deploy_preview_when_image_tag_is_created: true

Result 1. Deployment failed
Error
Deployment Not Found

Expected Result:  Show message Deployment Failed

Run uffizzi preview update deployment-NUMBER test-compose/compose.ym

Result 2. Deployment cannot be updated.
Error:
undefined method `kind' for nil:NilClass

Expected result: We need to update the failed deployment and if it fails again - show Deployment failed message

To reproduce the error:

Connect to uffizzi_cli
Create preview from compose file bundle exec uffizzi preview create compose.yml

Example file:
services:
  webhooks_test_app:
    image: uffizzitest/aaaa-webhooks-test-app:latest
    environment:
      POSTGRES_USER: postgres
      POSTGRES_PASSWORD: postgres

x-uffizzi-ingress:
  service: webhooks_test_app
  port: 80

x-uffizzi-continuous-preview:
  delete_preview_after: 1h
  delete_preview_when_image_tag_is_updated: true
  deploy_preview_when_image_tag_is_created: true

Because the image in the compose.yml is invalid, this will return the message Deployment failed

TESTED on prod:
Issures was created:
https://github.com/UffizziCloud/uffizzi_app/issues/244
https://github.com/UffizziCloud/uffizzi_app/issues/243
https://github.com/UffizziCloud/uffizzi_platform/issues/240


fix: update kosmosctl to support new version

What type of PR is this?
/kind bug

What does this PR do?

Update kosmosctl to support new and historical versions:

Provide --version parameter to adapt to the installation history or update version of kosmos;
Differentiate modules, eliminate unnecessary resource creation of independent modules, and make parameters such as enable-link available;
Fix the exception of creating pods on virtual nodes during check, and add existing labels to the agent role node;
Supplement the missing crds when creating the scheduler module;
Fix some other known errors.

Which issue(s) does this PR fix?

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

/ok-to-test


Use .distinct() by default for streams

title says it all

I strongly agree !!
You could add an indistinct() method if you want it to do a call for every db update regardless of "real" changes or not.

BTW: Are we sure that the distinct function will catch if there are similarities. I have situations now, in not too complex datasets that the distinct comparison "fails", so I need to provide with "manual" comparing of the datasets to be sure of the outcome. If drift could somehow provide the comparators of the datasets, it would have been very convenient.

I may post a suggestion tha there should be produced comparators on more complex queries...

If you enables the override_hash_and_equals_in_result_sets option, it will be even better...
See #3326


[RTMP] Add configuration to turn GOP cache on or off

Opening the GOP causes the player to open the video immediately, but with a higher delay.
Closing the GOP caching may cause the player to wait the video, but with a smaller latency.

supported in v0.5.0


Corrected e_shnum parsing from first section header.

Hello!
I found an issue with a very large ELF in “the wild” that contained many sections, it did not load correctly since the e_shnum was not properly being parsed from section_header0.sh_size, as the conditional for doing so appears to be incorrectly implemented:
if (e_shnum >= Short.toUnsignedInt(ElfSectionHeaderConstants.SHN_LORESERVE)) {
        e_shnum = readExtendedSectionHeaderCount(); // use extended stored section header count
}

Per the ELF (5) man page, when the number of section headers within an ELF file is greater than SHN_LORESERVE then the e_shnum value will be zero, not that e_shnum itself will be greater than SHN_LORESERVE. Full excerpt is below. It’s a quick fix, I left the current conditional in there in case other toolchains do in fact make the same interpretation a I don’t think it should hurt, though you may wish to consider removing the old one or combining the conditionals if you think this is best.
       e_shnum
              This member holds the number of entries in the section
              header table.  Thus the product of e_shentsize and e_shnum
              gives the section header table's size in bytes.  If a file
              has no section header table, e_shnum holds the value of
              zero.

              If the number of entries in the section header table is
              larger than or equal to SHN_LORESERVE (0xff00), e_shnum
              holds the value zero and the real number of entries in the
              section header table is held in the sh_size member of the
              initial entry in section header table.  Otherwise, the
              sh_size member of the initial entry in the section header
              table holds the value zero.

Applying this fix did allow the binary to load correctly :+1: .
Unfortunately the original binary cannot be shared for testing, though I've attached a contrived hello world example that's been modified "by hand" in case that is helpful. ( I just swapped the e_shnum and sec[0].size values).
hellow_world_patched.zip

I do not believe this handles the general case and is likely specific to Linux and lld.  There are many cases where no sections are defined within an ELF image in which case e_shnum will be zero.  Our original implementation of extended section count was done and completed in support of core dump files.  I will need to investigate further.  Could you please indicate the nature/source of your original binary where noted the issue.

The original binary that had this issue is an embedded RTOS image. Is that enough information?

I have put in a ticket to handle this issue internally to ensure it handles the general case (e.g., images where sections have been stripped or are absent).  I am also taking it further and ensure we handle e_shstrndx and the possible use if SHT_SYMTAB_SHNDX sections.  These may also be relevant when a very large number of sections exist.  Our internal ticket number is GP-2334 which should hopefully merge later this week.

The relevant changes have been merged into the master branch.


Point multiplication by negative factor is not correct

Simple verification with negative factor fails like shown:
#!/usr/bin/env python

from fastecdsa.curve import P256
from fastecdsa.point import Point

a = -1 * P256.G
b = (-1 % P256.q) * P256.G

assert(a == b)

As stated in StackExchange, the identity xP = (x+q)P should hold even for negative values.
I think that if all multiplications should be done modulus curve's q (but I don't know much about EC, so the fix could be broader).

Thanks for catching that, you're correct that negatives should simply by multiplied by the element in Zq that they are congruent to. Fixed in commit  e38578e, will be in version 1.4.3.
In [1]: from fastecdsa.curve import P256

In [2]: from fastecdsa.point import Point

In [3]: a = -1 * P256.G

In [4]: b = (-1 % P256.q) * P256.G

In [5]: assert(a == b)

In [6]:


Run only for development?

Is it possible to only run this in development mode?

This module is currently not opinionated about that and it's up to you to disable it for production. I've been thinking about automating this, but not sure the best way of doing it, because as far as I know there's no convention of what's debug and release builds with Electron apps. How do you define "development mode"?

I see.. As far as I know from my limited experience with Electron there doesn't seem to be an official way to do it. I'm currently doing NODE_ENV=dev electron . to start the app and using process.env.NODE_ENV to check for the variable value.
Will be closing this issue now as my problem is resolved!

I'd like to keep it open as I want to look into this at some point.

Fixed by https://github.com/sindresorhus/electron-debug/commit/4039271607d2c72a696a6f895406616435743d8a. Available in 0.6.0.


Refactor Webhook + Add X-Hub-Signature

close #16115
fixes #7788
This PR removes multiple unneeded fields from the HookTask struct and adds the two headers X-Hub-Signature and X-Hub-Signature-256.
"Breaking" change (or fix?): The webhook history shows the real called url and not the url registered in the webhook (deliver.go@129).

Whilst we're here we should stop adding the secret to the payload. it's supposed to be secret.

Ref #11755
@techknowlogick does this now break drone integration?

Codecov Report

Merging #16176 (d4bdaa7) into main (5f2ef17) will increase coverage by 0.03%.
The diff coverage is 48.78%.

@@            Coverage Diff             @@
##             main   #16176      +/-   ##
==========================================
+ Coverage   45.46%   45.50%   +0.03%
==========================================
  Files         705      705
  Lines       83154    83124      -30
==========================================
+ Hits        37808    37825      +17
+ Misses      39280    39233      -47
  Partials     6066     6066

Impacted Files
Coverage Δ

models/webhook.go
71.69% <ø> (ø)

modules/structs/hook.go
5.88% <ø> (-1.37%)
:arrow_down:

routers/api/v1/utils/hook.go
0.00% <0.00%> (ø)

routers/web/repo/webhook.go
1.12% <0.00%> (ø)

services/webhook/dingtalk.go
76.59% <ø> (+0.80%)
:arrow_up:

services/webhook/discord.go
70.00% <ø> (+0.53%)
:arrow_up:

services/webhook/feishu.go
85.91% <ø> (+1.19%)
:arrow_up:

services/webhook/msteams.go
90.30% <ø> (+0.45%)
:arrow_up:

services/webhook/slack.go
83.22% <ø> (+0.55%)
:arrow_up:

services/webhook/telegram.go
77.77% <ø> (+0.85%)
:arrow_up:

... and 11 more

Continue to review full report at Codecov.

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


manual disable of sample strategies

Don't know if this is intended or not...but when I manually disable either of the sample strategies the orders are cancelled but the position is not closed.
Otherwise... Great Work...thank you SO MUCH

Thats controlled by the NinjaTrader platform -  you could add a section on Terminate to try close positions etc


Fix hidden command usage

Kind
Bugfix
Description
Fix regression introduced by #49.
Now the hidden commands will be runnable.

Coverage increased (+0.06%) to 90.103% when pulling e02e0b3556e38b67b03e3c83e7dacfd3bd80de46 on nmengin:hotfix/allow-using-hidden-commands into cd1d1cd595f21a1b9ccedd4670fcf95b55d139f3 on containous:master.


Remove Fast Render Itegration

Currently, we depends on fast-render for dealing with client side data availability.
So, we need to implement a similar data environment which is available on the server for react to render initially.
In this case, we may need to re-use a lot of fast-render goodies. Try to use them.
If this is the simply re-implementing fast-render, why don't we fix the bug of using fast-render
(Which is something we won't get the data)

May be we need to reconsider this since we need to support upcoming SubsManager's FR integration.


Feature/capps2/proebbox

Summary

This PR is a new feature.

It does the following:

Fixes #1210
Adds the ability to read in a subset of a ProE mesh.  The user supplies a lambda function to accept or reject tetrahedra.  The PR also adds convenience functions to build a lambda from a user-specified bounding box.

To Do

[ ] Make sure it works for a distributed reader
[ ] Add documentation

The only thing I don't feel 100% comfortable with is the use of std::vector<bool>. But, if we're not doing anything too fancy with it, it should be OK.

@rhornung67 -- Would our dynamic bitset be preferable?
See: https://github.com/LLNL/axom/blob/develop/src/axom/slam/BitSet.hpp

If we have a dynamic bitset, that may be better. std::vector may be OK here depending on how it's used. However, it has been a sore spot in the standard for years.

@kennyweiss , is the issue of std::vector<bool> vs slam::BitSet a style or doctrinal issue, in that we should "eat our own dogfood?" Or, is there any technical advantage to using slam::BitSet?

@agcapps maybe paranoia on my part. Many use cases work fine with std::vector. However, for technical reasons and choices made by the C++ standards committee, it doesn't follow the same rules as other STL containers.  As long as it works for what you are using it for and it remains limited in scope (i.e., not requiring massive changes if something goes wrong in the future), I say we go with what you have.   https://softwareengineering.stackexchange.com/questions/328112/how-did-stdvectorbool-come-about

@kennyweiss , @rhornung67 , is the issue of std::vector<bool> vs slam::BitSet a style or doctrinal issue, in that we should "eat our own dogfood?" Or, is there any technical advantage to using slam::BitSet?

@agcapps -- There are several issues w/ vector<bool> related to semantics and performance.
See, e.g.: https://stackoverflow.com/questions/16738579/is-using-a-vector-of-boolean-values-slower-than-a-dynamic-bitset
My suggestion was keying off @rhornung67's discomfort and the fact that we have an available replacement.
If you decide to replace it and compare the performance, I'd be interested in seeing the results.

All, I've added some Sphinx docs.  Please cast your eyes over the changes.  Thanks!


Cannot read property 'forEach' of undefined in Google Analytics Graphs (dashboard)

Laravel Version: 5.5.28
Voyager Version: 1.0.12
PHP Version: 7.1.3
Database Driver & Version: SQLite3 & 3.15.1

Description:
After setting up the Google Analytics Client ID, two graphs are not loading due to the following error:
Uncaught (in promise) TypeError: Cannot read property 'forEach' of undefined
    at admin:657
    at <anonymous>

At line 657 this is what can be found:
response.rows.forEach(function (row, i) { // line 657
    data.push({
        label: row[0],
        value: +row[1],
        color: colors[i]
    });
});

The very same error also occurs at line 685.
If you look inside index.blade.php, the code that throws the errors appears to be at lines 361 and 389 instead of 657 and 685 as reported in the browser.
Steps To Reproduce:

add the Google Analytics Client ID to the Voyager setting('admin.google_analytics_client_id')
go to dashboard, do an Empty cache and Hard Reload and check the console output

Any clue how to solve this?

That implies that the response has no rows.  Can you attempt to debug the response?  I'm curious to see what it actually is.

Sure thing @fletch3555.
Here is output of console.log(response)  at line 361 from index.blade.php:
{
  "kind": "analytics#gaData",
  "id": "https://www.googleapis.com/analytics/v3/data/ga?ids=ga:{{REMOVED}}&dimensions=ga:browser&metrics=ga:pageviews&sort=-ga:pageviews&start-date=7daysAgo&end-date=yesterday&max-results=5",
  "query": {
    "start-date": "7daysAgo",
    "end-date": "yesterday",
    "ids": "ga:{{REMOVED}}",
    "dimensions": "ga:browser",
    "metrics": [
      "ga:pageviews"
    ],
    "sort": [
      "-ga:pageviews"
    ],
    "start-index": 1,
    "max-results": 5
  },
  "itemsPerPage": 5,
  "totalResults": 0,
  "selfLink": "https://www.googleapis.com/analytics/v3/data/ga?ids=ga:{{REMOVED}}&dimensions=ga:browser&metrics=ga:pageviews&sort=-ga:pageviews&start-date=7daysAgo&end-date=yesterday&max-results=5",
  "profileInfo": {
    "profileId": "{{REMOVED}}",
    "accountId": "{{REMOVED}}",
    "webPropertyId": "UA-{{REMOVED}}-1",
    "internalWebPropertyId": "{{REMOVED}}",
    "profileName": "All Web Site Data",
    "tableId": "ga:{{REMOVED}}"
  },
  "containsSampledData": false,
  "columnHeaders": [
    {
      "name": "ga:browser",
      "columnType": "DIMENSION",
      "dataType": "STRING"
    },
    {
      "name": "ga:pageviews",
      "columnType": "METRIC",
      "dataType": "INTEGER"
    }
  ],
  "totalsForAllResults": {
    "ga:pageviews": "0"
  }
}

And here it is for line 389:
{
  "kind": "analytics#gaData",
  "id": "https://www.googleapis.com/analytics/v3/data/ga?ids=ga:{{REMOVED}}&dimensions=ga:country&metrics=ga:sessions&sort=-ga:sessions&start-date=7daysAgo&end-date=yesterday&max-results=5",
  "query": {
    "start-date": "7daysAgo",
    "end-date": "yesterday",
    "ids": "ga:{{REMOVED}}",
    "dimensions": "ga:country",
    "metrics": [
      "ga:sessions"
    ],
    "sort": [
      "-ga:sessions"
    ],
    "start-index": 1,
    "max-results": 5
  },
  "itemsPerPage": 5,
  "totalResults": 0,
  "selfLink": "https://www.googleapis.com/analytics/v3/data/ga?ids=ga:{{REMOVED}}&dimensions=ga:country&metrics=ga:sessions&sort=-ga:sessions&start-date=7daysAgo&end-date=yesterday&max-results=5",
  "profileInfo": {
    "profileId": "{{REMOVED}}",
    "accountId": "{{REMOVED}}",
    "webPropertyId": "UA-{{REMOVED}}-1",
    "internalWebPropertyId": "{{REMOVED}}",
    "profileName": "All Web Site Data",
    "tableId": "ga:{{REMOVED}}"
  },
  "containsSampledData": false,
  "columnHeaders": [
    {
      "name": "ga:country",
      "columnType": "DIMENSION",
      "dataType": "STRING"
    },
    {
      "name": "ga:sessions",
      "columnType": "METRIC",
      "dataType": "INTEGER"
    }
  ],
  "totalsForAllResults": {
    "ga:sessions": "0"
  }
}

Does this help?

At the moment, I don't see how this ever would've worked.  Perhaps Google changed their API or something...

It appears so, the rows property is nowhere to be found. Or, could it because I am trying it on localhost and perhaps there is not enough data?

If I remember correctly, google analytics doesn't work on localhost.  It needs an actual domain in the request, since that domain needs to map to one you have on your GA account.  Can you try that and see if it's still an issue?

I think it does work because I am able to see active users, most visited pages and some other data both in the Voyager dashboard and the GA dashboard in real time.
I will try what you suggested and report back tomorrow morning. I am actually very curious to see if that is the issue.

@fletch3555, I checked again the browser console--on http://localhost--and the error is gone. Now the response object does contain a rows property.
It appears that GA works fine on http://localhost, but because there was no data, the response object omitted the rows property. Would it be a good idea to add a check for rows on these responses before attempting to access them?

sometimes rows exists, and sometimes dont. I  think you were right response only contain rows if containing data

i meet the same issue , i think it's better to check rows property before push data.


TypeError: '<' not supported between instances of 'list' and 'tuple'.

I git pull http directly and not git fork any more...
So, I do NOT provide pull request for now.
A bug:
TypeError: '<' not supported between instances of 'list' and 'tuple'.

File third_party/gpus/find_cuda_config.py
Line 447:
    if cuda_version.split(".") < (10, 1):

to
    ver = cuda_version.split(".")
    ver1 = int(ver[0])
    ver2 = int(ver[1])
    if (ver1, ver2) < (10, 1):

BTW, I'm using CUDA 10.1
Cheers
Pei

@jiapei100 Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

@jiapei100
File third_party/gpus/find_cuda_config.py
if cuda_version.split(".") < (10, 1):
modified to
if tuple(int(v) for v in cuda_version.split(".")) < (10, 1):

@jiapei100
File third_party/gpus/find_cuda_config.py
if cuda_version.split(".") < (10, 1):
modified to
if tuple(int(v) for v in cuda_version.split(".")) < (10, 1):

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Added version detection

Troubleshoot repeated switchover on windows

Would be great to have it fallback to the engine in package.json if no .nvmrc file found


Expose idTokenClaims on AccountInfo

Since AccountEntity objects and ID tokens are cached separately, when building AccountInfo objects we will need to look up the ID token for the given account, decode it, and attach it to the returned object. Should work for browser and Node since all the changes are done in common.

When is this PR going to be completed?

Coverage decreased (-1.2%) to 80.297% when pulling e533115b19aa2263e0b0f4ef3a8d5e213b615e8b on browser-idtokenclaims-accountinfo into 50f8dfb1e6b375bb4baf0cc59b7f4e34c3d24146 on dev.

When is this PR going to be completed?

Looking to have it merged soon and included in<EMAIL_ADDRESS>


add plot recipes

defining convert_arguments for ODESolution.
adding a plot recipe orbit.
adding a plot recipe monitor
adding some tests and a pertinent CI.

The docstring and documentation about the argument vars are still not perfect, more modifications will be appended to this PR.

I have modified some documentations directly. I noticed that you use 4-spaces indentations; this is fine as long as it's consistent. In TestParticle.jl source files I use 3-spaces indentations. Just a note, no need to change.

I just ran the monitor recipe on my laptop. It looks super cool! It would be great if we have an animation in the documentation besides the figures!


New "neu-crystalgrape" theme

Themes for Issue #36

Please use yarn for installing dependencies instead of npm.


Add content about Lambda Expressions and Functional Programming in C++

Select the Day of Content
Day 21
What information is missing?
Day 21: Lambda Expressions and Functional Programming in C++
Introduction:
Lambda expressions provide a way to create anonymous functions (i.e., functions without a name) in C++. They are particularly useful for short snippets of code that are passed to algorithms or used as callbacks. Functional programming, which emphasizes immutability and first-class functions, can be effectively implemented in C++ using lambda expressions and the standard library's functional utilities.
Topics Covered:

Introduction to Lambda Expressions
Syntax of Lambda Expressions
Capturing Variables
Using Lambdas with Standard Algorithms
Lambdas as Callbacks
Practical Examples and Exercises

1. Introduction to Lambda Expressions:
A lambda expression in C++ allows you to define an inline function directly in the code where it is used. This is especially useful for short operations, such as those passed to algorithms.
2. Syntax of Lambda Expressions:
The basic syntax of a lambda expression is:
[capture](parameters) -> return_type { body }

Capture: Specifies which variables are captured from the surrounding scope.
Parameters: The parameters passed to the lambda, similar to a regular function.
Return type: (Optional) The return type of the lambda. It can often be inferred by the compiler.
Body: The code that gets executed when the lambda is called.

Example:
auto add = [](int a, int b) -> int {
    return a + b;
};
std::cout << "Sum: " << add(2, 3) << std::endl; // Output: Sum: 5

3. Capturing Variables:
Lambda expressions can capture variables from their enclosing scope. There are several ways to capture variables:

Capture by value: [x, y] captures x and y by value.
Capture by reference: [&x, &y] captures x and y by reference.
Capture all by value: [=] captures all variables by value.
Capture all by reference: [&] captures all variables by reference.
Mixed capture: [=, &x] captures all variables by value except x, which is captured by reference.

Example:
int x = 10;
int y = 20;

auto printSum = [x, y]() {
    std::cout << "Sum: " << x + y << std::endl;
};

printSum(); // Output: Sum: 30

4. Using Lambdas with Standard Algorithms:
Lambdas are particularly useful with the Standard Template Library (STL) algorithms. Here’s an example using std::for_each:
#include <algorithm>
#include <vector>
#include <iostream>

int main() {
    std::vector<int> numbers = {1, 2, 3, 4, 5};
    std::for_each(numbers.begin(), numbers.end(), [](int n) {
        std::cout << n << " ";
    });
    // Output: 1 2 3 4 5
    return 0;
}

5. Lambdas as Callbacks:
Lambdas can be used as callbacks for event handling or asynchronous operations. Here’s an example with a simple callback mechanism:
#include <iostream>
#include <functional>

void performOperation(const std::function<void(int)>& callback) {
    int result = 42; // Imagine this is the result of some operation
    callback(result);
}

int main() {
    performOperation([](int result) {
        std::cout << "Operation result: " << result << std::endl;
    });
    // Output: Operation result: 42
    return 0;
}

6. Practical Examples and Exercises:
Example 1: Sorting a Vector with a Lambda
#include <algorithm>
#include <vector>
#include <iostream>

int main() {
    std::vector<int> numbers = {5, 2, 9, 1, 5, 6};

    std::sort(numbers.begin(), numbers.end(), [](int a, int b) {
        return a < b;
    });

    for (int n : numbers) {
        std::cout << n << " ";
    }
    // Output: 1 2 5 5 6 9
    return 0;
}

Example 2: Filtering Elements with std::copy_if and a Lambda
#include <algorithm>
#include <vector>
#include <iostream>

int main() {
    std::vector<int> numbers = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10};
    std::vector<int> evenNumbers;

    std::copy_if(numbers.begin(), numbers.end(), std::back_inserter(evenNumbers), [](int n) {
        return n % 2 == 0;
    });

    for (int n : evenNumbers) {
        std::cout << n << " ";
    }
    // Output: 2 4 6 8 10
    return 0;
}

Exercises:

Write a lambda function to calculate the factorial of a number.
Use a lambda to transform a vector of integers by squaring each element.
Create a lambda that captures a local variable by reference and modifies it within the lambda.

please assign this issue to me and put the suitable labels


Update scalatest to 3.2.16

About this PR
📦 Updates org.scalatest:scalatest from 3.2.14 to 3.2.16
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (3.2.14).
You might want to review and update them manually.
soteria.json

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalatest", artifactId = "scalatest" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalatest", artifactId = "scalatest" }
}]

labels: test-library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #264.


Add support for @apollo/client

Fixes #12

@helfer Finally, we got green build! Let me know what you think, and of course feel free to make any extra changes :)

Hey @helfer! Any news about this??

@helfer could you please merge ?

Since no one merged it, if you want to use it right away... -> https://www.npmjs.com/package/omerman-apollo-link-debounce
published it under omerman-apollo-link-debounce.

Thanks a lot for doing this @1v4nx, and sorry I didn't get around to merging it sooner!

Just published 3.0.0 with the changes from this PR 🎉


[Feature-Request] Multi-cam image stitching

Start with the why:
Because it's simples/cheaper to use multiple camera sensors (eg. using our Modular Camera design) that are at an angle compared to using a high-res high-fov camera sensor. Also for a full 360deg view you need multiple camera sensors, and then stitch the image together.
Move to the what:
Create a demo for multi-camera stitching. We would first need multi-cam extrinsics calibration script in order to then develop image stitching.
Move to the how:
Computer vision. Maybe look into solution by kornia?

Any progress on the stitching part?

Hi @tinito , we haven't yet created a demo of this, as there wasn't much interest.


Set install_method for iis to be powershell - Relevant only for chef client 13

Description
Changes to the windows cookbook v3.0.0 has caused windows_feature to install features using dism as the default method. However, not all machines have the feature enabled by default on startup. The install then "silently fails" and continues on past the feature installation, and bugs out when trying to start the Service W3SVC. This PR fixes this problem by installing IIS using the powershell install method, which will install IIS, and "enable" any auxiliary IIS features - thereby allowing for a default dism install for the rest. Note that this is only an issue for chef client 13, as in chef client 14, AFAIK the default windows_feature install method is powershell, not dism.
Issues Resolved
https://github.com/chef-cookbooks/iis/issues/281
https://github.com/chef-cookbooks/iis/issues/280
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Looks like the test timed out! Can someone help me figure out why?

I've done quite a bit of modernization to the cookbook on master that I think better presents the root cause of what you're running into. We now require the windows cookbook 4.1, which is a complete rewrite of how features are handled and matches the code that went into Chef 14. That code will fail when it should and also handles passing in arrays of features which we're now using to speed up runs. You really don't want to use powershell by default since that didn't ship with a good chunk of the Windows release. I'm going to cut a new major release today. I'm closing this PR out now, but do let me know if you're still running into issues.


Add API for custom LSP requests

Implement a function ale#lsp_linter#SendRequest that allows to send
custom LSP requests to an enabled LSP linter.
Resolves #2474

Thanks for the review comments! I didn't realise I could re-use the logic of StartLSP.  If I am not overlooking anything, now it should be better.

Cheers! :beers:


add dependency

for https://github.com/Azure/azure-rest-api-specs/pull/25827

@msyyc, what error were you seeing before pinning this version?

@msyyc, what error were you seeing before pinning this version?

The file is used to generated Mgmt SDK from typespec. While some service like (e.g. servicenetworking) has dependency about typespec-providerhub: https://github.com/Azure/azure-rest-api-specs/blob/79d4b87b6632452bf83711814faa5b71e4761544/specification/servicenetworking/ServiceNetworking.Management/main.tsp#L4

We should not need to add this providerhub package here if there is a good reason please let me know and we can reopen.


[FLINK-21462][sql client] Use configuration to store the option and value in Sql client

What is the purpose of the change
In this PR, we use Configuration to store all the key value in the sql client rather than Envrionment. In this way, we can also set the options from the runtime when using SET command.
To compassionate with origin Environment, it also introduces a util named ConfigurationUtils that is able to convert ExecutionEntry to Configuration.
Brief change log

Deprecate Environment(It's only used when using YAML file to start the client)

Verifying this change
This change added tests and can be verified as follows:

Added compatibility util test ConfigurationUtilsTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Add @deprecated Javadoc for every deperacted files.

The PR is mainly designed for compatibility between ExecutionEntry and Configuration. However, it does introduce some incompatibility. I think the incompatibility is from 2 parts.

Some entries is not supported, e.g. execution.time-characteristic
The default value for option and entry may be different, e.g. the default value for execution.restart-strategy.attempts is Integer.MAX_VALUE but the default value for restart-strategy.fixed-delay.attempts is 1.

The reason why I decide to not supprot execution.time-characteristic :

The value of execution.time-characteristic is different from StreamPipelineOptions.TIME_CHARACTERISTIC
TimeCharacteristic is also a deprecated class.

The main reason to drive me to the conclusion that ignore the incompatibility is users will set the value for restart strategy option in general in production environment...
If we really needs completely compatibility, I think it needs more work to be done...
Hope my feedback can reduce some comments...

Thanks for @wuchong detailed guidance. =-=
Cheers!


Update core SSH keys

Fixes https://github.com/RHsyseng/cluster-relocation-operator/issues/8
Workflow:

this one is fairly simple, it just creates a MachineConfig for the new SSH authorized keys. The new keys get appended to whatever is there currently (the existing key isn't deleted)
because the MachineConfig is owned by the CR, when the CR is deleted, the MachineConfig is deleted as well

@danielchg this one is ready for review


Added subtext to various items in Bank image

Description:

Tiny, small, average, large & huge lamps to display xp value on bank image
the uncharged sanguinesti staff and Scythe of vitur will have subtext displaying so.

Changes:

added a new section for XP Lamps adding the xp value as subtext for each lamp.
added uncharged to the uncharged versions of sanguinesti staff & Scythe of vitur.
added JMod to the Scythe of vitur (JMod) to easily tell it apart from the rest.

Other checks:

[ ] I have tested all my changes thoroughly.

I am new at this so sorry for the multiple PR, but once I figured some of the commits I had made weren't going where they should have been I didn't want to risk it so I made a new PR but with all the same changes, put in the right place.

added uncharged to the uncharged versions of sanguinesti staff & Scythe of vitur.

This could also be merged to osb as a useful addition. Meaning it could be removed from this pr and a new pr on master branch (osb) then the commit would be merged over leaving only bso specific things in this pr.

added uncharged to the uncharged versions of sanguinesti staff & Scythe of vitur.

This could also be merged to osb as a useful addition. Meaning it could be removed from this pr and a new pr on master branch (osb) then the commit would be merged over leaving only bso specific things in this pr.

I could totally do this... you're talking about the uncharged stuff right

@TastyPumPum ty SO very much for what you did! for real

@themrrobert can you look at this & merge it if all checks out?

Implemented differently here: https://github.com/oldschoolgg/oldschoolbot/commit/b17089dd7784cfe1c91b01b6109afe1222d030b3

Implemented differently here: https://github.com/oldschoolgg/oldschoolbot/commit/b17089dd7784cfe1c91b01b6109afe1222d030b3

No way you were able to do that all within one line of code that's awesome


puppet: Only install ksplice uptrack if there is an access key.

Merged, thanks @alexmv!  I suppose I can imagine having a world where we have an else clause to print a warning, so that one will notice when running puppet if a machine is misconfigured without an access key?


I'm using the C# example and nothing shows up

`InitializeComponent();
		var myCarousel = new CarouselViewControl();
		myCarousel.ItemsSource = new ObservableCollection<int> { 1, 2, 3, 4, 5 }; // ADD/REMOVE PAGES FROM CAROUSEL ADDING/REMOVING ELEMENTS FROM THE COLLECTION
		//myCarousel.ItemTemplate = new MyTemplateSelector(); //
        myCarousel.ItemTemplate = new DataTemplate (typeof(MyView));
		myCarousel.Position = 0; //default
		myCarousel.InterPageSpacing = 10;
		myCarousel.Orientation = CarouselViewOrientation.Horizontal;

        Stack.Children.Add(myCarousel);`

I did do the Init on both projects.

Don't know whats the problem you are facing.

@mmcarson I had the same issue.  But after 2 days I finally got a demo working.  I have no idea what exactly it was that was causing it not to work but at least you can look at my demo and maybe it will help you.  You can find it HERE

Thanks so much @helzgate for the example! I got it working and added images 😄


task: fix issue where a dataset could not be downloaded

Description

Apparently when opening some Dataset asset they might not download properly because of the hash added by keycloak.js.
This is a second attempt to resolve this issue.

@chris-dickson and @bigglesandginger I added you for awarness of this issue with keycloak.

Have you considered adding this in the router?
router.afterEach(() => {
  window.location.hash = '';
});

Have you considered adding this in the router?
router.afterEach(() => {
  window.location.hash = '';
});

This is a smart idea!
Do you mind adding this to your PR, if it is necessary, as keycloack.js should do it automatically, and I will close mine now that is irrelevant.


boards: arm: mimxrt1024_evk: enable flexcan1

Enable FlexCAN1 on the NXP i.MX RT1024 Evaluation Kit.
Signed-off-by: Henrik Brix Andersen<EMAIL_ADDRESS>

Thank you for the contribution, could you kindly share what tests were run to verify the driver.

Thank you for the contribution, could you kindly share what tests were run to verify the driver.

I have ran all the tests tagged with can (actually, all the tests selected for this board) using twister on the hardware without issues.


CMake-Tools does not (re)configure on CMakeLists.txt change/save

CMake-Tools does not (re)configure on CMakeLists.txt change/save.
When the project gets successfully configured either by manual invocation or when configured for the first time, then any saved change of related CMakeLists.txt does not trigger automatic reconfiguration of the project.
How to reproduce

Setup simple project with single CMakeLists.txt and single main.cpp. Fill both files with minimal contents to build dummy TestA.exe
Lets have a VSCode with CMake-Tools installed
Open the project in VSCode
After successful configuration change the CMakeLists.txt e.g. by adding another executable target TestB.exe and save the file

Expected:
The CMake-Tools should automatically reconfigure the project and adjust the CMake-Tools tree/view accordingly (in this case by adding TestB executable target).
Apparent Behavior:
The CMake-Tools does not automatically reconfigure the project.
CMake Tools Log
Does not log any issues.
Developer Tools Log
Does not log any issues.
Platform and Versions

Operating System: Windows 10 , Ubuntu 18.04
CMake Version: 3.10.2
VSCode Version: 1.30.2
CMake Tools Extension Version: 1.1.3
Compiler/Toolchain: Visual C++ 2017, Clang 6.x

Other Notes/Information

Before I studied the source code I would bet the behavior I described in this bug was working ok
I tested my projects (one where I noticed the issue, the other minimal case described in this bug) with the pure CMake in server mode (cmake -E server --debug) and the server notifies properly about changes in CMakeLists.txt
When I studied the source code, it looks the functionality was never there ! :-). The place which should trigger (by my opinion) the configure is here

+1
workaround is select folder, right click, remove folder from workspace, then add folder.

Fixed for CMake Tools upcoming 1.4 release.
Closing as duplicate of https://github.com/microsoft/vscode-cmake-tools/issues/1187.

Hello, I have the opposite problem. VSCode re-configures cmake always I save something new in a CMakaLeist.txt. This is very annoying mainly for big projects where you spend more time on cmake configurations.
How could I disable this behaviour? I want VSCode re-configure only when I need to.
Thank you.


realvnc-vnc-viewer: 6.22.515 -> 7.1.0

Description of changes
update realvnc-vnc-viewer version from 6.22.515 -> 7.1.0
adding build Inputs rely on https://github.com/NixOS/nixpkgs/issues/225963
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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

[x] 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/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Please squash the first and the third commit.


chore(DIST-144): Update Travis npm release config

Docs for semantic release say it should be configurable via release prop in package.json but it does not work. Passing the argument to the command directly seems to work.

How about introducing a .releaserc file for keeping the semantic-release configuration in it? It would result in a cleaner package.json

@prilutskiy I can try that. However for some reason the config in package.json did not work 🤷♂

For some reason the semantic-release is not reading the config from a file (neither package.json nor .releaserc), it needs to be passed directly via command line option.

@mathio it's a pity to hear that. Seems like they have a bug, because I'm pretty sure I used to get such setup working :)
Nevertheless - approved.


dialouge is bugged

Describe your bug here.
so when i use my dialouge in game its just ramdom portaits its so annoying so can someone help me
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

You got your xml wrong


Donation widget - form base aesthetics

What:
Complete aesthetics for all forms within the donation wizard
Why:
This is related to the effort to complete the donation widget (machine + wizard)
How:

Create Wizard components and compose them within each of the steps of the wizard
Add styled-components and twin.macro as a way to reduce the noise and improve maintainability of each wizard piece

At this point the only thing I render is..

I will continue to work on this branch on all forms but the idea of this draft is to communicate progress and keep the team within the loop of the decisions made.

I have added more aesthetics matters, a11y improvements, possibility to go back to edit the amount, required fields using HTML, configuration of Jest and dynamic title. A bit on how it looks at here

As I mentioned on the video further PRs will include:

Donation type controller switch
Enhancements for inputs (one PR per each component input enhancement)
Actual server integration to make the donation

LGMT, I'm just concerned about using styled-components

LGMT, I just find odd using styled-components

I'm going to merge as seems that the only topic pending to discuss is the one over the styled-components usage. Repercussions of this decision seems to be on the performance side and this can be addressed further as an optimisation request if we found necessary.

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


[NEW CONTENT TYPE REQUEST] JPEG2000

What type of file would you like magika to detect?
JPEG2000 (aka JP2 and JPF)
What software can create/open these files?

See the JPEG.org site's list.
IrfanView with a plugin.
XnView MP with a plugin
OS X Preview app

Where can these files be found?
Examples:

Most digital libraries
Internet Archive books, e.g. https://archive.org/download/2018Fundamentals.ofPython/2018_fundamentals.of-python_jp2.zip
https://github.com/openpreserve/jpylyzer-test-files

If possible, please provide a specification for this file type.

https://jpeg.org/jpeg2000/workplan.html

Additional context
Currently misidentifies as MP4:
(env) dev:~/test_magika$ magika /data/Pictures/MBL_Album_3_pg_19b_1894_ten_scientists_inc._Lillie_Crane.jpf --json
[
    {
        "path": "/data/Pictures/MBL_Album_3_pg_19b_1894_ten_scientists_inc._Lillie_Crane.jpf",
        "dl": {
            "ct_label": "mp4",
            "score": 0.9982088804244995,
            "group": "video",
            "mime_type": "video/mp4",
            "magic": "ISO Media",
            "description": "MP4 media file"
        },
        "output": {
            "ct_label": "mp4",
            "score": 0.9982088804244995,
            "group": "video",
            "mime_type": "video/mp4",
            "magic": "ISO Media",
            "description": "MP4 media file"
        }
    }
]

File command is correct:
(env) dev:~/test_magika$ file /data/Pictures/MBL_Album_3_pg_19b_1894_ten_scientists_inc._Lillie_Crane.jpf
/data/Pictures/MBL_Album_3_pg_19b_1894_ten_scientists_inc._Lillie_Crane.jpf: JPEG 2000 Part 1 (JP2)

Thanks for reporting! Marked to be imported to our KB and closing it for now (see #152 for context). Thanks!


Getting CUDA error: no CUDA-capable device is detected for TeraChem energy calculation

ChemCloud version: 0.8.2
input.toml:
calctype = "energy"

[extras]

[files]

[keywords]

[model]
method = "B3LYP"
basis = "6-31g"

[molecule]
symbols = [ "H", "H",]
geometry = [ [ 0.0, 0.0, 0.0,], [ 1.3889487024901424, 0.0, 0.0,],]
charge = 0
multiplicity = 1
connectivity = []

[molecule.extras]

[molecule.identifiers.extras]

Calculation setup:
from chemcloud import CCClient
from qcio import Molecule, ProgramInput
from ase.units import Bohr

client = CCClient()
mol = Molecule(symbols=['H', 'H'], geometry=[0., 0., 0., 0.735/Bohr, 0., 0.])
inp = ProgramInput(
    molecule=mol,
    model={"method": "B3LYP", "basis": "6-31g"},
    calctype="energy",
    keywords={},
)
future_result = client.compute("terachem", inp)
output = future_result.get()

Truncated TeraChem log from output.stdout

| Startfile from command line: tc.in

|       ***********************************************************
|       *                 TeraChem v1.9-2023.09-dev               *
|       *                   Development Version                   *
|       *           Chemistry at the Speed of Graphics!           *
|       ***********************************************************
|       * This program may only be used in connection with        *
|       * a valid license from PetaChem, LLC. Use of this program *
|       * or results thereof indicates acceptance of all terms    *
|       * and conditions stated in the license and that a valid   *
|       * license agreement between the user and PetaChem, LLC    *
|       * exists. PetaChem, LLC does not warrant the correctness  *
|       * of results or their suitability for any purpose.        *
|       * Please email bugs, suggestions, and comments to         *
|       *<EMAIL_ADDRESS>                    *
|       *                                                         *
|       ***********************************************************

|       ***********************************************************
|       *  Compiled by root         Fri Sep  8 02:45:55 UTC 2023  *
|       *  Supported architecture SMs: 52 61 70 80 89             *
|       *  Build cuda_11.8.r11.8/compiler.31833905_0              *
|       *  Git Version: 48f1c4b24460eb111f0d752b6dea2ab9d1eb956f  *
|       ***********************************************************

| Job started   Thu Oct 12 04:36:10 2023
| On ee4a8d8c6618 (available memory: 336130 MB)

| ######################################### RUNTIME INFO ##########################################
| terachem  tc.in

|  NVRM version: NVIDIA UNIX x86_64 Kernel Module  525.89.02  Wed Feb  1 23:23:25 UTC 2023
|  GCC version:  gcc version 11.4.0 (Ubuntu 11.4.0-1ubuntu1~22.04)

[lib information truncated]

| Checking Amazon Cloud instance ID...   Not found
| Cannot find license.dat file /opt/terachem/license.dat

| TRYING THE NETWORK LICENSE...
| Connecting to license server 'x.x.x.x' port 'x'...
| Connected!
| Checking your license...

| **************************************************************
|  Greetings, Martinez Group! You have x licenses in total
|  IN USE: x
|  AVAILABLE: x
| **************************************************************

-----------------------------------
Processed Input file:
                      run energy
              coordinates geometry.xyz
                   charge 0
                 spinmult 1
                   method b3lyp
                    basis 6-31g
-----------------------------------
Jobname: geometry
Scratch data will be generated
Scratch directory: ./scr.geometry
| Random number seed:<PHONE_NUMBER>

XYZ coordinates geometry.xyz
Molden File Output: ./scr.geometry/geometry.molden
Using basis set: 6-31g
dmrgstart not found
Spin multiplicity: 1
DIIS will use up to 10 vectors.
Condition number limit for the DIIS overlap matrix is            0
WF convergence threshold: 3.00e-05
Using DIIS algorithm to converge WF
Maximum number of SCF iterations: 100
Incremental fock with rebuild every 8 iterations
Will switch to conventional Fock if diffuse functions are detected
X-matrix tolerance: 1.00e-04
PRECISION: DYNAMIC
TeraChem will select linear algebra engine
DFT Functional requested: b3lyp
Method: B3LYP
  Hartree-Fock exact exchange:          0.20
  Slater exchange functional:           0.80
  Becke 1988 exchange functional:       0.72
  Lee-Yang-Parr correlation functional: 0.81
  VWN(I) correlation functional:        0.19
Wavefunction: RESTRICTED
DFT grid type: 1
Using dynamic DFT grids.
Initial guess generated by maximum overlap

********************************************
***** SINGLE POINT ENERGY CALCULATIONS *****
********************************************
Starting TensorBox...
| TeraChem will select linear algebra engine
CUDA error: no CUDA-capable device is detected, file tensorbox/src/tensorbox.cpp, line 35

 Job terminated: Thu Oct 12 04:36:11 2023

Thanks for the write-up. Fixed!
The issue was I'd poached a few GPUs for TeraChem from my debug node, but someone put python processes on underneath them and so TeraChem couldn't access the GPU. Dropped those workers and everything works now. Thanks!

Great! Thanks for the quick fix!


Check failed: tree[nid].is_leaf()

Hi, when I trained xgboost model, after few iteration, I got this error:
[11:12:47] src/tree/./updater_basemaker-inl.h:272: Check failed: tree[nid].is_leaf
()
terminate called after throwing an instance of 'dmlc::Error'
what(): [11:12:47] src/tree/./updater_basemaker-inl.h:272: Check failed: tree[nid].is_leaf()
Any clue about why this might happened?

@EvanzzzZ Can you post a script that reproduces the issue?

I don't have a script right now but I am running binary:logistic at scale.


Permission error with version v1.0.0rc1

Hello,
I installed netbox with the version 0.9.x of this playbook.
I would like to switch to new version (v1.0.0rc1) of this playbook (to install new pip dependencies) without upgrading netbox.
When I execute the playbook again, I have this error:
failed: [xxxxxx] (item=dns) => changed=false
  ansible_loop_var: item
  attempts: 2
  cmd:
  - /opt/netbox/current/venv-py3/bin/pip3
  - install
  - dns
  item: dns
  msg: |-
    stdout: Collecting dns

    :stderr: The directory '/opt/netbox/.cache/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
    The directory '/opt/netbox/.cache/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
      Could not find a version that satisfies the requirement dns (from versions: )
    No matching distribution found for dns

However:
root@netbox:~# ll /opt/netbox/.cache
ls: cannot access '/opt/netbox/.cache': No such file or directory

Thanks.

what's the task name for this failure?
can you print the ownership of /opt/netbox?
are you also having this issue when running with the version of the role you originally used?

Sorry No matching distribution found for dns, I made a mistake in the python module.
The good name is dnspython and not dns.
So the new playbook has been deployed successfully.
However:
root@netbox-app-dc1-01:/opt/netbox/current# venv-py3/bin/pip list | grep dns
dnslib

Gotcha. Good to hear.


The get variable command fails on one variable only

Traceback (most recent call last):
File "E:\Táda\Plocha\Systém Táďos\Stažené soubory\Comment-Filter-Bypass-main\Comment-Filter-Bypass-main\comment.py", line 29, in
username = Encoding.decode(scratch3.get_var(pid, "pw"))
File "C:\Users\TAĎAS\AppData\Local\Programs\Python\Python37-32\lib\site-packages\scratchattach_encoder.py", line 119, in decode
letter = letters[int(f"{inp[i2]}{inp[(i2)+1]}")]
ValueError: invalid literal for int() with base 10: 'No'

scratch3.get_var(pid, "pw") seems to return None. Are you sure you specified the correct project id, and does the "pw" cloud variable actually exist? Has it been set since it was created?

scratch3.get_var(pid, "pw") seems to return None. Are you sure you specified the correct project id, and does the "pw" cloud variable actually exist? Has it been set since it was created?

yes


Any plan to support UITesting in the future?

Description
Looks that Swift Testing does not support UITest at the moment. Any plans to support UITest in the future?
Certainly, there are some differences in testing APIs between unit testing and UI testing, But supporting UITest would help maintain a unified and friendly testing experience and test ecosystem.
Expected behavior
Use  Swift testing in UITest
Actual behavior
Can not import Testing in UITest Target

Steps to reproduce
No response
swift-testing version/commit hash
No response
Swift & OS version (output of swift --version && uname -a)
No response

Thanks for bringing this up, @HollyFoxtel!
Swift Testing itself is primarily concerned with defining tests, running them, and getting their results. The exactly nature of those tests, i.e. the steps they perform, Swift Testing is agnostic about.
On Apple platforms, XCTest provides UI automation APIs like XCUIApplication for automating apps in its (closed-source) framework included in Xcode. Currently, those APIs do not behave correctly when used in Swift Testing tests, so in Xcode 16 Beta, the build system shows the error you mentioned when attempting to import Testing in any UI test target. To allow using XCTest's UI automation features within Swift Testing tests, work will be needed in Apple's proprietary XCTest framework. Since that project is separate, I encourage you to file a Feedback with Apple requesting this functionality.
Beyond that, it is also likely that additional supporting functionality will need to be added here, in Swift Testing, to allow UI testing systems to work properly. For example, there may need to be additional controls over parallelization or result reporting. These kinds of enhancements need to be explored and defined, but they would potentially be relevant/applicable to any UI automation system, not just XCTest UI automation. In general, we do want Swift Testing to be capable of hosting UI automation style tests, although this is not something which we have the capacity to focus on for its initial release aligned with Swift 6. We will keep this GitHub issue open to track the aspects of this work which directly affect Swift Testing.

Tracked internally at Apple with rdar://95593221.


Feature Request: dapr-pubsub configuration binding via Azure Managed Identities

Is your feature request related to a problem? Please describe.
We want to use Azure Service Bus via pubsub dapr component over Managed Identities. As of today, the situation is not possible as only the connection string is an option. References:

https://docs.dapr.io/reference/components-reference/supported-pubsub/setup-azure-servicebus/#azure-active-directory-aad-authentication
https://docs.microsoft.com/en-us/azure/container-apps/managed-identity?tabs=arm%2Cdotnet#limitations

Describe the solution you'd like.
Assuming:

you have a managed identity
you grant the proper RBAC grant in service bus for the managed identity ( doh :) )

Ideally, the following would be perfect:

you create a dapr component for service bus pubsub in Azure Environment
as dapr-metadata you provide the namespace name
optionally, as dapr-metadata you provide a client id (appId in AAD) of the managed identity (if you have more and we have more, 1 user-assigned for container app, 1 user assigned for service bus interaction - shared with multiple container apps)
then you deploy a container app which uses the above and it works.

Describe alternatives you've considered.
Passing an example Service Bus connection string works like a charm. No issue here.
Additional context.
When we tested this "kind-of" solution, we've got the error in Log Analytics, in ContainerAppConsoleLogs_CL table:
time="2022-07-14T15:24:12.981104983Z" level=warning msg="error initializing pub sub pubsub.azure.servicebus/v1: MSI not available" app_id=autodeploy02 instance=autodeploy02--439-99d9854d4-rwjng scope=dapr.runtime type=log ver=1.7.3-msft-1
time="2022-07-14T15:24:17.982122772Z" level=fatal msg="process component triggers error: MSI not available" app_id=autodeploy02 instance=autodeploy02--439-99d9854d4-rwjng scope=dapr.runtime type=log ver=1.7.3-msft-1

Highly recommend creating a user-assigned identity; annoying you have to reference the client id everywhere, but it solves a 'chicken and egg'  issue when creating a new environment from scripts/bicep where your ACA app cant pull images from ACR because the system-assigned managed identity hasnt been created/assigned AcrPull yet.

Yeah I agree @onionhammer... I think it makes most sense to use user assigned at least for dapr specifically. @cwe1ss I too ran into the problem when I didn't include the azureClientID. I wasn't initially aware this was a requirement. Instead of creating a new component for every app, I would consider using a shared user assigned identity for apps that need similar access policies on certain resources. I will still go back and talk to the team to see if there is a way we can try to optimize this experience

It would be great if any of the following could be done :)

An update to the docs so it is clear that azureClientId is required. For instance here it's explicitly listed as optional: https://docs.dapr.io/developing-applications/integrations/azure/authenticating-azure/
Also make it clear that the identity that is configured for the Component should be the assigned to the App as well. Or what would be even better if the azureClientId would indeed be optional and that it picks a system assigned identity when it's available, or a user assigned identity if there is only one.

If the above isn't possible, it would be great if we could assign an identity to the entire Container App Environment, which could be set as a node identity (while having the App identity itself set as pod identity). That way we could fix multiple problems:

We can create an identity (user or system assigned) and assign that one AcrPull rights so there is no chicken/egg problem anymore if we would want to use system assigned identities for Apps, because all nodes can pull from the Acr.
This identity could maybe also be used for Dapr components (if so configured), so we can create a single CAE with a bunch of Dapr components that just have the proper rights. Teams could then just deploy their Apps and use the Components without having to know which identities to assign to their app.

Another option to let apps just have a single identity is to create Dapr components per app, but that feels wrong to me. I don't want to have to create app specific things in an environment that is shared over multiple apps. It would also cause the amount of components to explode, not sure if there is a limit.


InventoryCloseEvent is called when opening a inventory

The event InventoryCloseEvent is fired when I open an inventory :
When I try to close my inventory with the 1.8.9 client version it's working but if the event is also fired if a player open an inventory with the 1.9.4 version.
I tried with 1.8.9 & 1.9.4 versions.
ViaVersion 0.9.9.
Can you fix this bug ?
Thanks.

Can't reproduce this bug with only ViaVersion and ProtocolLib.
Please follow the issue template (https://github.com/MylesIsCool/ViaVersion/blob/master/ISSUE_TEMPLATE.md) and give us steps how to reproduce it.
Thanks

/viaversion dump output :
-> http://hastebin.com/lubekugoko.json
Version: git-Spigot-e4d4710-e1ebe52 (MC: 1.8.8) (1.8.8-R0.1-SNAPSHOT)
-> No error in the console or else

What are the drawbacks of this issue?
Are we able to use a certain plugin and see something?

This was removing an inventory that I was trying to use.

@MylesIsCool I used a plugin that is opening an inventory when player right click at an NPC.

@Blamo27 do you have a link for us to that plugin, or a way to reproduce it?

Click on this link to download a small plugin that I created to reproduce the same bug :
=> https://file.io/AyJ9Vb
You need Citizens 2.0 to run this plugin.
And after :
-> Create an NPC
-> /trait test
-> RightClick on the NPC
You should see a message when you Open / Close the inventory.

{
success: false,
error: 404,
message: "Not Found"
}

The file does not exist.

Yes sorry, https://dropfile.to/fsGEa8U
I changed this.

The InventoryCloseEvent works as expected. The problem is that NPCRightClickEvent gets activated twice, that'll close the previous inventory.

Oh ..  But how to fix this problem ?

@Blamo27 Working on a small patch now

Thanks !

Should be fixed using the latest devbuild :smile:


Improper version management

My understanding is: The semantic hub mirrors the models that are governed in sldt-semantic-models. However, on  https://portal.int.demo.catena-x.net/semantichub/, there are two released versions of MaterialForRecycling. sldt-semantic-models has no v1.1.0 - still it appears in the semantic hub (see screenshot below)

Thanks @arnoweiss for the report. @bs-jokri do you have an idea?

Hi @arnoweiss ,
We have synchronized the sldt-semantic-models and semantic-hub application.
MaterialForRecycling is showing only version 1.1.0 now

Please let us know if there are any open issues.

Thanks for the fix!


[5.7-04252022] Update Hero gradient color

Rationale: Update the gradient colors and the dark overlay for Hero based on newest spec
Risk: Low
Risk Detail: only CSS change.
Reward: High
Reward Details: Better contrast between Hero image color and the background color.
Original PR: https://github.com/apple/swift-docc-render/pull/213
Issue: rdar://91450837
Code Reviewed By: @mportiz08
Testing Details: Manually test in browser and verify.

@swift-ci test


Visual Studio Code plugin that autocompletes filenames can't work

VSCode Version: Code 1.6.0 (e52fb0bc87e6f5c8f144e172639891d8d8c9aa55, 2016-10-10T18:37:40.189Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions: christian-kohler.path-intellisense-1.0.2

Visual Studio Code plugin that autocompletes filenames can't work。
all path autocompletes can not work

I belive this functionality is coming from chrisitan kohler paht-intellisense extension which you have installed. Can you please file this issue in the repository of that particular extension.
Thank you

but any extension that path autocomplate in html can't work , it is't one extension. i think it is your bug

ping @aeschli to assess


tests: boot: mcuboot_recovery_retention: Fix configuration issue

Fixes an issue with the tests whereby the RAM memory configuration files were not being used, and could not be passed to the mcuboot image.

@fabiobaltieri could you also merge this please?


Add support for hnsw+bq

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (310bd06) 95.49% compared to head (3d1fd1b) 95.42%.
Report is 16 commits behind head on main.

Files
Patch %
Lines

integration/test_collection_config.py
85.71%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #900      +/-   ##
==========================================
- Coverage   95.49%   95.42%   -0.07%
==========================================
  Files         167      167
  Lines       17198    17221      +23
==========================================
+ Hits        16423    16433      +10
- Misses        775      788      +13

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fixed drawing the caret at eol when at eol is not visible

Fixed the caret being draw at eol when its not visible.
closes #27631

Thanks!


How to implement the connection persistance with BLE device

Prerequisites
Please answer the following questions for yourself before submitting an issue. YOU MAY DELETE THE PREREQUISITES SECTION.

[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed
[x] I'm sure that question is related to the library itself and not Bluetooth Low Energy or Classic in general. If that so, please post your question on StackOverflow or on our Gitter channel.

Question
We need to implement connection persistance with BLE Device.
Description

First of all we are calling the startDeviceScan method to scan the device.
After scanning successfully, we are calling connectToDevice method to initiate the connection with BLE Device.
We are storing the deviceId in AsyncStorage after successful connection established.

Use Case
We have to implement connection persistance with the previous connected BLE Device without scanning. To implement this we are calling connectToDevice and passing the previous connected BLE DeviceId. On some devices it is working fine but some devices it is not working fine because deviceId of ble device is being changed what we found. Any idea on this ?

Hi,
Your approach on storing the deviceId is not the one to consider for several points :

New Ble specification  use dynamic MAC adresses
Android gives the MAC address for deviceId, but iOS gives an UUID
Scanning before each connection does not cost so much time

So I would advise to scan before connecting to the device.
Also, there is 2 subjects:

Reconnect to a known device after closing the app.
This is done by storing the ble name of the device, scan for it and reconnect.

Maintain connection in the app with screen navigation
For this, what I do is to check if the device is connecting in the lib everytime I want to use it.
If it is the case you can get the device object from manager.connectedDevices().
You might have to maintain the connection state (connecting, connected, disconnected) in the global state of your app.

Chears

I think what you want to do would require a device bound but we don't currently support this feature