ARM64 ABI: Inconsistent handling of aggregate structs

Hi folks: I've got an interesting edge-case in the ABI where Clang and MSVC behave differently (this shows up in a Chromium test suite called dawn_end2end_tests). What the tests do is they create an empty D3D12_RESOURCE_DESC and return it on the stack (see a cut-down demonstrator at https://godbolt.org/z/jE8538).
Here, D3D_RESOURCE_DESC is 56 bytes, it's an aggregate type, no user-provided constructors, no private or protected non-static data members, no base classes, and no virtual functions, trivial copy-assignment operator, and it has a trivial destructor, so it should fall into this case:

For types greater than 16 bytes, the caller shall reserve a block of memory of sufficient size and alignment to hold the result. The address of the memory block shall be passed as an additional argument to the function in x8. The callee may modify the result memory block at any point during the execution of the subroutine. The callee isn't required to preserve the value stored in x8.

However, that's not actually what happens, if I disassemble the main() function linked in the demonstrator, space gets allocated and passed in x1.
|main|  PROC
|$LN8|
        str         lr,[sp,#-0x10]!
        sub         sp,sp,#0x40               ; Allocate space to hold the D3D12_RESOURCE_DESC plus a 8-byte class
        add         x1,sp,#8                    ; Increment to get original sp - 56 bytes to hold the D3D_RESOURCE_DESC in X1
        mov         x0,sp                        ; $this pointer is at the base of stack, passed in X0
        bl          |virtual D3D12_RESOURCE_DESC B::GetResource(void) __ptr64|
        ldr         w0,[sp,#8]
        add         sp,sp,#0x40
        ldr         lr,[sp],#0x10
        ret

        ENDP  ; |main|

Clang (the official compiler for Chromium/Edge) follows the current wording of the spec and allocates room in X8 (https://godbolt.org/z/rs7G7a). Are there additional cases covered by MSVC which have not been accounted for in the specification?

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

ID: 8ee4db12-6226-a1e6-6c8c-d2d5d77ae92e
Version Independent ID: 180f6f4e-d5a9-88cd-c297-6cba5b16289e
Content: Overview of ARM64 ABI conventions
Content Source: docs/build/arm64-windows-abi-conventions.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

@richard-townsend-arm
I can't fault your analysis, but it's not reaching the right audience. This sounds more like an issue for the product engineers than for the docs team, at least to start with. This a question for the Visual Studio C++ Developer Community site, which the dev team looks at directly. If it's an ABI violation, they can fix it. If it's a gray area, they can tell us what to add or clarify in the docs. If you'd be kind enough to report it there, and come back here and link your issue, I'd appreciate it.

OK, apologies about that: I've raised it here. Will re-open this issue if I hear back.

Thanks!

@richard-townsend-arm
A docs fix is in the works, and should appear tomorrow.


Consolidate duplicate state id constant

TRANSITION_ID_SEND_TICKET_GRANTING_TICKET duplicated STATE_ID_SEND_TICKET_GRANTING_TICKET.
The pattern for naming CAS webflow constants is to use STATE_ID_ for any state id's. So, consolidate the duplicate constants to that.

just a minor cleanup :)

Coverage increased (+0.02%) to 46.035% when pulling 361230eaec3ca6cf7ab68dbfcc7d91ccc872892f on frett:webflowStateConstant into 334669fc9ff307fe4065891db3ab1bd630e63871 on apereo:master.

Thanks. Excellent work!


docs: Detailed documentation

This (WIP) PR adds detailed documentation to Zeth. The updates aim to provide clearer context and more detailed explanations for developers.
TODO:

[ ] guests

[ ] eth-block

[ ] main.rs

[ ] host

[ ] main.rs

[ ] lib

[ ] block_builder.rs
[ ] consts.rs
[ ] execution.rs
[ ] host

[ ] mpt.rs
[ ] provider

[ ] cached_rpc_provider.rs
[ ] file_provider.rs
[ ] rpc_provider.rs

[ ] provider_db.rs

[ ] mem_db.rs
[x] validation.rs

[x] primitives

[x] access_list.rs
[x] block.rs
[x] ethers.rs
[x] keccak.rs
[x] lib.rs
[x] receipt.rs
[x] revm.rs
[x] signature.rs
[x] transaction.rs
[x] withdrawl.rs
[x] trie

[x] account.rs
[x] mpt.rs

Draft: looking for early feedback - lmk if it missing anything, typos, or is too pedantic.

Closing this and opening a new PR soon simply to avoid conflicts with latest refactor.


Backport "Teach provablyDisjoint about AnyKind" to LTS

Backports #18510 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

Suppressed this PR from LTS backport as it lead to regression in our tests


Can Not Add Third Car

Below, I add my code. I can add two vehicles (vehicle and vehicleE). But when I am trying to add thrid one, its automatically destroying (vehicleA). Can anyone tell me how to solve this?
try:
m = world.get_map()
start_pose = random.choice(m.get_spawn_points())
waypoint = m.get_waypoint(start_pose.location)
    blueprint_library = world.get_blueprint_library()

    try:

        vehicle = world.spawn_actor(
            random.choice(blueprint_library.filter('vehicle.*')),
            start_pose)
        actor_list.append(vehicle)
        vehicle.set_autopilot(True)
        print("1st done")
    except:
        print("1st not done")

    try:
        vehicleE = world.spawn_actor(
            random.choice(blueprint_library.filter('vehicle.*')),
            start_pose)
        print("2nd Done")
    except:
        print("2nd not done")

    try:
        vehicleA = world.spawn_actor(
            random.choice(blueprint_library.filter('vehicle.*')),
            start_pose)
        print("3rd Done")
    except:
        print("3rd not done")

    camera_rgb = world.spawn_actor(
        blueprint_library.find('sensor.camera.rgb'),
        carla.Transform(carla.Location(x=-5.5, z=2.8), carla.Rotation(pitch=-15)),
        attach_to=vehicle)
    actor_list.append(camera_rgb)

Are you sure this works? You are trying to spawn three vehicles on exactly the same location, so you should get a "Spawn Failed" error! Your script works for me when I am using deterministic, non-colliding spawn positions.
Also a general advice: You get a lot more feedback why your code does not work if you print/log the error messages instead of just catching them and doing nothing with them!

@arnabfly has your issue been resolved?


Create a footer blue-3

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

After building the component please raise a PR with a screenshot of the component and add the component in Footer/footer-blue-3.html.
If you need to use any icon please use it from Hero Icons
Good luck.

can you assign this to me please

Go ahead @kingwisdom


Add i18n support and pt_BR translation

@nileshgulia1, @avoinea Could you guys take a look and make a release, please?

@ericof I will test these changes and merge if everything is fine, holiday season caught with us for
the ability to check community contributions.

@ericof and @andreclimaco  tested locally and working fine, merging now.
One small thing that I will need to fix after the merge is the
prettier fix,  try to run before the pull request
make prettier-fix
have a look at the makefile for whatever linters we use,
in general we check stylelint, prettier and eslint.
We will create better documentation in Develop.md with
info about lining and testing, until then we will fix it ourselves.


Add support for defining Caches at startup

It would be very helpful if we can pass in a list of pre-defined caches so that clients are always guaranteed to have them available? This is particularly useful when deploying to AWS Beanstalk in single container mode.

To enable this behaviour, we should make the image process a configured CLI batch file on startup. This provides greater flexibility than simply providing a cache.xml file as it will be possible to configure other resources such as counters and users.

This will require changes to the Infinispan server which are tracked by ISPN-12052

Is it possible to define a new CRD in the infinispan operator to define de caches, so it is the operator the one that consolidates that cache list, same as topic operator for strimzi?

https://github.com/infinispan/infinispan-operator/issues/224

Is it possible to define a new CRD in the infinispan operator to define de caches, so it is the operator the one that consolidates that cache list, same as topic operator for strimzi?

This features will be available in ver. 2.0.0.
Good reference is also the PR https://github.com/infinispan/infinispan-operator/pull/360


Google Pay tokenization returns "No value for signature"

I am integrating with Google Pay and the payment succeeds, after that when I try to tokenize the payment using generateGooglePayToken method, it always returns exception: "No value for signature" although the tokenJsonPayload string format is correct as shown below (simplified) and contains the signature from google pay response.
{
   "type":"google_pay",
   "token_data":{
      "protocolVersion":"ECv1",
      "signature":"MEQCIDg6/kRoR8af1YTSVfkI9UovV2V+jV8YATG2rvL...juAowEl7rvzbPx8g==",
      "signedMessage":"{\"encryptedMessage\":\"32GByKqzr68CsQ2eYUnx1RJMheZAcp5QQ5LZnrrjymztdmeITfeHn4vLF1WhDjG2U9dxFDTrnOdJxjbHsUFsO/r...0+HR4GWH26ipyMgzF/TMrl3RHrYWM+WKAsJnx\",\"ephemeralPublicKey\":\"BH5/P7a009qJClyZm91+l...CIrEwEeL0Tp72/3NpwrEk\\u003d\",\"tag\":\"u2Mcf/ySnRLdM...FSuO4Nax0k8\\u003d\"}"
   }
}

Unfortunately, I couldn't debug the native code. How can this be solved?

@edwardmp Can you support?

@Hassanmohsaid this is an educated guess, but when creating the payment did you set the gateway to checkoutltd and did you set up and created a merchant ID in the Google Pay console?
See https://www.checkout.com/docs/payments/add-payment-methods/google-pay#Step_1:_Integrate_with_Google_Pay

Otherwise I feel the error indicates that maybe your are not passing the payload as JSON to generateGooglePayToken?
For me it looks like this:
FlutterCheckoutPayment.generateGooglePayToken(
          tokenJsonPayload: paymentData["paymentMethodData"]["tokenizationData"]
              ["token"],
        )
      ```

@edwardmp yes, I setup the gateway, checkout key (based on env.) and merchant id from Google Pay console.
I am passing tokenJsonPayload as a string to generateGooglePayToken like this:
"{
   "type":"google_pay",
   "token_data":{
      "protocolVersion":"ECv1",
      "signature":"MEQCIDg6/kRoR8af1YTSVfkI9UovV2V+jV8YATG2rvL...juAowEl7rvzbPx8g==",
      "signedMessage":"{\"encryptedMessage\":\"32GByKqzr68CsQ2eYUnx1RJMheZAcp5QQ5LZnrrjymztdmeITfeHn4vLF1WhDjG2U9dxFDTrnOdJxjbHsUFsO/r...0+HR4GWH26ipyMgzF/TMrl3RHrYWM+WKAsJnx\",\"ephemeralPublicKey\":\"BH5/P7a009qJClyZm91+l...CIrEwEeL0Tp72/3NpwrEk\\u003d\",\"tag\":\"u2Mcf/ySnRLdM...FSuO4Nax0k8\\u003d\"}"
   }
}"

As it requires a string not a map, is this wrong?

@Hassanmohsaid I hope that @edwardmp could help you.
Thanks @edwardmp ❤

@ShadyBoshra2012 yes, it is solved now.
Thank you both!


[TextareaAutosize] Simplify logic and add test

A quick continuation of #37135. I was triggered because I saw 3 PRs that fix bugs without tests on the same component:

https://github.com/mui/material-ui/pull/33238.
https://github.com/mui/material-ui/pull/33253
https://github.com/mui/material-ui/pull/37135. I have tried to add tests on this one, but the problem is that in Karma, the browser gets confused, It triggers ResizeObserver loop completed with undelivered notifications but it's an hallucination https://developer.mozilla.org/en-US/docs/Web/API/ResizeObserver#observation_errors. More details in https://github.com/WICG/resize-observer/issues/38, also faced in https://github.com/mui/mui-x/issues/8733.

I'm also removing these if (inputRef.current) { defensive checks. Sebastian was asking the right question in https://github.com/mui/material-ui/pull/33238#issuecomment-1162696735. As far as I know, the origin is React 17 https://github.com/facebook/react/pull/17925 which doesn't run the effect cleanup synchronously anymore.
@mnajdova could we make the same change in https://github.com/mui/material-ui/pull/33277/files, moving to a layout effect? Thanks

Released in v5.14.11


feat: add gc jobs

We need garbage collector for completed and failed jobs

/ok-to-test

I dont have access to the logs, but u @rast1025 please run make lint and make test locally, and see if there are any issues ...

Also, it would be good to have a dry-run flag like other args.

I dont have access to the logs, but @rast1025 please run make lint and make test locally, and see if there are any issues ...

There are no issues

/lgtm


Upgrade rest-client

Security: redact password in URI from logs (#349 / OSVDB-117461)

Look good.


chore: bond to local interface

What this PR does / why we need it:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Which issue/s this PR fixes

How Has This Been Tested?

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I've read the CONTRIBUTION guide
[ ] I have added CI tests to cover my changes.
[ ] All new and existing tests passed.

/code pr

:robot:
Review for commit e1aeec3ac788c95410c6ca4541cda10bc0a7e889:

function/Dockerfile:

Line 9: Changing the default POD_IP from <IP_ADDRESS> to <IP_ADDRESS> restricts the container to only accept connections from the local host. Ensure this change aligns with the deployment environment and intended accessibility.
Line 20: The addition of --no-install-recommends is a good practice to minimize the image size by avoiding unnecessary packages. Well done.

function/Dockerfile.server:

Line 9: Similar to the Dockerfile change, modifying the POD_IP to <IP_ADDRESS> limits accessibility to localhost only. Confirm this is the desired behaviour for the server container.

function/index.py:

Line 8: Updating the HOST variable to <IP_ADDRESS> aligns with the Dockerfile changes but restricts the application to localhost. This should be reviewed to ensure it meets the application's requirements for network accessibility.

Overall, the changes are clear and mostly pertain to tightening network accessibility. It's crucial to verify that these changes do not conflict with the intended use cases of the application. The update to reduce unnecessary package installations in the Dockerfile is a commendable improvement.
> gpt-4-turbo-preview

:robot:
The changes in this patch primarily focus on binding the application to the local interface (<IP_ADDRESS>) instead of allowing it to be accessible on all interfaces (<IP_ADDRESS>). Additionally, the Dockerfile has been modified to only install the required libraries, improving the build efficiency and security posture of the container. Here are some suggestions and issues flagged by file:
function/Dockerfile

Enhancement: The use of --no-install-recommends in the apt-get install command is a good practice as it minimises the installation of unnecessary packages. However, it's also recommended to clean up the apt cache by adding && apt-get clean && rm -rf /var/lib/apt/lists/* at the end of the RUN command. This reduces the image size by removing temporary files used during the installation process.
Example:
RUN apt-get update && \
    mkdir -p /usr/share/man/man1 /opt/tika/libs && \
    apt-get install --no-install-recommends -yy openjdk-11-jre-headless curl && \
    curl -vSL https://archive.apache.org/dist/tika/${APACHE_TIKA_VERSION}/tika-app-${APACHE_TIKA_VERSION}.jar -o /opt/tika/tika-app.jar && \
    curl -vSL https://repo1.maven.org/maven2/org/apache/tika/tika-core/${APACHE_TIKA_VERSION}/tika-core-${APACHE_TIKA_VERSION}.jar -o /opt/tika/libs/tika-core.jar && \
    curl -vSL https://repo1.maven.org/maven2/org/apache/tika/tika-xmp/${APACHE_TIKA_VERSION}/tika-xmp-${APACHE_TIKA_VERSION}.jar -o /opt/tika/libs/tika-xmp.jar && \
    apt-get clean && \
    rm -rf /var/lib/apt/lists/*

function/Dockerfile.server

Consistency: The change from <IP_ADDRESS> to <IP_ADDRESS> for the POD_IP environment variable is consistent with the change in the other Dockerfile. This is good for maintaining consistency across your Docker configurations.

function/index.py

Potential Issue: Changing the host from <IP_ADDRESS> to <IP_ADDRESS> in a Docker environment might not be desirable in all cases. Binding to <IP_ADDRESS> means the application will only accept connections from within the container itself. If the intention is to allow access from outside the container (e.g., for development purposes or when using Docker's port forwarding), this change could prevent the application from being accessible as intended.
Suggestion: If external access is required, consider providing a way to configure the host environment variable (POD_IP) to allow overriding it back to <IP_ADDRESS> if necessary. This could be documented in the project's README or through comments in the relevant configuration files.

Overall, the changes are sensible and improve the security and efficiency of the Docker setup. However, careful consideration should be given to the implications of binding to <IP_ADDRESS> in containerised environments, ensuring it aligns with the intended use cases of the application.
> gpt-4-turbo-preview

/code pr

:robot:
Review Summary
function/Dockerfile

Change in POD_IP Environment Variable: Changing the default POD_IP from <IP_ADDRESS> to <IP_ADDRESS> is a good security practice as it restricts the container to only accept connections from the local host. However, ensure this change aligns with the deployment environment requirements.

Optimization in apt-get install: The addition of --no-install-recommends is a positive change, reducing the size of the image by avoiding unnecessary packages. This is a best practice for Docker images.

function/Dockerfile.server

Change in POD_IP Environment Variable: Similar to the Dockerfile, changing POD_IP to <IP_ADDRESS> is a good practice for security reasons. Ensure this does not conflict with the intended use case of the server container.

function/index.py

Change in HOST Variable: Setting the HOST variable to <IP_ADDRESS> by default is a secure default for development environments. For production, ensure that the service is still reachable by other services if required.

Suggestions
No additional suggestions for enhancements are identified based on the modified lines. The changes made are generally in line with best practices for security and optimization.
Conclusion
The modifications in this pull request are sensible, focusing on security and optimization. Ensure that the changes, especially the binding to <IP_ADDRESS>, are compatible with the deployment and operational requirements of the application.
> gpt-4-turbo-preview

:robot:
The changes made in this patch are generally sound, focusing on enhancing security by binding services to the local interface and optimising the Docker build process by only installing required libraries. Here are some specific suggestions and issues flagged by file:
function/Dockerfile

Enhancement: The change to use --no-install-recommends with apt-get install is a good practice to minimise the size of the image by avoiding unnecessary packages. However, it's also advisable to clean up the apt cache by adding && apt-get clean && rm -rf /var/lib/apt/lists/* at the end of the RUN command to further reduce the image size.
Suggested change:
RUN apt-get update && \
    mkdir -p /usr/share/man/man1 /opt/tika/libs && \
    apt-get install --no-install-recommends -yy openjdk-11-jre-headless curl && \
    curl -vSL https://archive.apache.org/dist/tika/${APACHE_TIKA_VERSION}/tika-app-${APACHE_TIKA_VERSION}.jar -o /opt/tika/tika-app.jar && \
    curl -vSL https://repo1.maven.org/maven2/org/apache/tika/tika-core/${APACHE_TIKA_VERSION}/tika-core-${APACHE_TIKA_VERSION}.jar -o /opt/tika/libs/tika-core.jar && \
    curl -vSL https://repo1.maven.org/maven2/org/apache/tika/tika-xmp/${APACHE_TIKA_VERSION}/tika-xmp-${APACHE_TIKA_VERSION}.jar -o /opt/tika/libs/tika-xmp.jar && \
    apt-get clean && \
    rm -rf /var/lib/apt/lists/*

function/Dockerfile.server

Consistency: The change from <IP_ADDRESS> to <IP_ADDRESS> for POD_IP is consistent with the change in function/Dockerfile. This is a good practice for services that do not need to be exposed outside of the local machine. No further changes are suggested here.

function/index.py

Security: Changing the default host from <IP_ADDRESS> to <IP_ADDRESS> is a security enhancement, ensuring that the service is not unintentionally exposed to the network. This change is appropriate given the context provided by the other files in the patch.

Code Quality: It might be beneficial to add a small comment explaining why <IP_ADDRESS> is used instead of <IP_ADDRESS>, especially for future maintainers who might not be aware of the security implications. This is a minor suggestion and not strictly necessary.
Suggested addition:
# Bind to localhost to prevent unintentional exposure on the network
HOST = os.environ.get("POD_IP", "<IP_ADDRESS>")

Overall, the patch is well-constructed with a clear focus on improving security and optimising resource usage. The suggestions provided aim to further enhance these aspects without introducing significant changes to the proposed modifications.
> gpt-4-turbo-preview

/code pr

/code pr

:robot:
The changes introduced in this patch are generally well-structured and aim to enhance security and efficiency by binding services to the local interface and reducing unnecessary library installations. Below are some suggestions and issues flagged by file:
function/Dockerfile

Enhancement: The addition of --no-install-recommends and cleanup commands (apt-get clean && rm -rf /var/lib/apt/lists/*) are commendable for reducing the image size and removing unnecessary files. However, it's advisable to combine apt-get update, apt-get install, and cleanup commands in a single RUN instruction to reduce layers and further optimize the image size. For example:

RUN apt-get update && apt-get install --no-install-recommends -yy openjdk-11-jre-headless curl && \
    <installation commands> && \
    apt-get purge -yy curl && apt-get autoremove -yy && apt-get clean && rm -rf /var/lib/apt/lists/* /var/cache/apt/* /tmp/* && \
    pip install --no-cache-dir -r requirements.txt && \
    fc-cache -f

Issue: The fc-cache -f command at the end of the RUN instruction is used to build font cache, but there's no clear indication that custom fonts are being added or that fontconfig is directly used by the application. If not necessary, consider removing it to streamline the build process.

function/Dockerfile.server

Enhancement: Similar to the function/Dockerfile, combining apt-get update, apt-get install, and cleanup commands can optimize the build process. Additionally, specifying --no-install-recommends is a good practice that has been correctly applied here.

Issue: The use of tesseract-ocr-all package installs all available language packs for Tesseract OCR, which might not be necessary and can significantly increase the image size. Evaluate the actual requirements and consider installing only the needed language packs to optimize the image size and build time.

function/index.py

Enhancement: Changing the default host from <IP_ADDRESS> to <IP_ADDRESS> is a security improvement as it restricts access to the local machine. However, ensure that this change aligns with the deployment environment and that services requiring external access are appropriately configured.

Issue: There's no issue with the modification itself, but it's essential to update documentation or configuration files that reference the old default IP address (<IP_ADDRESS>) to avoid confusion or misconfiguration.

Overall, the patch moves in the right direction by enhancing security and optimizing resource usage. The suggested enhancements aim to further refine these improvements.
> gpt-4-turbo-preview

:robot:
Review of PR: chore: bind to local interface and only install required libs
function/Dockerfile

Changing the default POD_IP environment variable from <IP_ADDRESS> to <IP_ADDRESS> is a good security practice as it restricts access to the local interface. This change is correctly applied on line 9.
The modification to use --no-install-recommends with apt-get install on line 20 is a positive change, reducing the size of the image by avoiding unnecessary packages.
Adding apt-get clean and removing /var/lib/apt/lists/* after package installation on line 33 is another good practice for Docker images, ensuring that the image size is kept to a minimum.

function/Dockerfile.server

The change in the default POD_IP environment variable to <IP_ADDRESS> mirrors the security enhancement made in function/Dockerfile. This is correctly applied on line 9.
The use of --no-install-recommends during the apt-get install command and the subsequent cleanup (lines 27-28) are consistent with the improvements made in function/Dockerfile, contributing to a leaner Docker image.

function/index.py

Updating the default HOST value to <IP_ADDRESS> on line 8 aligns with the changes made in the Dockerfiles, ensuring that the application binds to the local interface by default. This is a sensible security measure.

General Comments

All changes are consistent with the stated goals of the patch: improving security by binding to the local interface and optimising the Docker images by only installing required libraries and cleaning up after installations.
The PR is well-structured, and the changes are appropriately focused. Each modification serves a clear purpose and is correctly implemented.
No issues or further enhancements are identified within the scope of the modified lines.

This PR is ready for merging, pending any further testing that may be deemed necessary.
> gpt-4-turbo-preview


Changelog for 12.1.0 (huge release!).

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


Reenable IL Emit DI backend

https://github.com/aspnet/Extensions/pull/570 is fixing the bug we had with code generation.
It has the same runtime performance but is faster and uses less memory to generate methods.
Runtime flag is not available yet.

cc @ajcvickers We're going to enable this again.
@pakrym I'm not sure we should be putting this in preview1. We need to verify that EF works.

I can merge it after preview1 goes out. We didn't have problems with EF and IL Emit before. I'll do a quick test with the music store.

@pakrym @davidfowl Anything that even might be risky should not go in while we're trying to get preview1 builds stabilized, so we should hold of on this.
/cc @muratg @natemcmaster @Eilon

@pakrym you're safe to checkin preview 2 work to this repo. I've only locked the master branch of aspnet/AspNetCore for preview 1 work.


[file-list] Disable or Hide Quick Start Button when unable to start

#60

Why not just display only the files that can be quick started ?

The idea of putting all the files on the same table might not be a good one.

Do you only want to display the training files and the fine-tuning result files? If so, I have a suggestion:
Display only the training files on this page, and link the fine-tuning result files to the job list.
This is because the fine-tuning result files are created as a result of the job.

Resolved in #170.

close #170

Resolved in #170


add support for ICS20 withdrawals to pcli

Rebased; it'd be good if we could get the review comments addressed so we don't miss shipping this.

review comments addressed, with the exception of the unified value field, since it seems like the Value type can't parse unknown denoms into a full denom? the withdrawal type needs a full denom, not just an asset id

Oh, hmm, good catch. Maybe in the future we can make a FromStr for ValueView or something. Anyways, not super important now.


No detection in darknet_ros, while darknet is detecting with the same .weights

Hello !
I trained yolo v3 on custom dataset (within darknet). I can see some detections in darknet, however it fails to detect when I use trained  .weights in darknet_ros.
.cfg, .yaml, .launch are created.
If anyoane have seen such case, please share your solution.

I downloaded again by the following order.
git clone --recursive git@github.com:leggedrobotics/darknet_ros.git
This time it also downloaded the darknet(yolov3). I make it again and the detections shows successfully.
However, when I tried  run yolo(AB), it still shows no bounding boxes.

I successfully ran YOLO(AB) just now. Did you exchange the darknet folder with your own darknet? Maybe the problem is that. YOU can try tou use git clone to download darknet it provides.

Oh, great ! So glad to hear it is working for you :)
I am dealing with ssh problem right now, after I will try your solution.  Before I cloned this repository with http, maybe that is why my model is behaving so.

I have tried ssh, but the result was not so good. Now i am learning Socket.

I'm sorry my pool English made you misunderstand what I mean.
I use   'git clone --recursive git@github.com:leggedrobotics/darknet_ros.git' to download the whole code it provides(include Darknet).  Compile directly and it run successfully.  Don't  replace darknet inside draknet_ros with yours. You don't even need to compile Darknet.

Ura ! It is working :)
Thanks for your help @wildpig888

@akmaralAW  I did the solution above and successfully detected the object. However, the performance are worse than the original darknet (darknet detect demo). Im wondering if you facing or feeling the same problem? tks

Hello @tabowsy,
I am afraid my answer won't be helpful, since I didn't test my model within darknet_ros with real images. In a simulated environement it didn't work so well.  I think this is because my original model was not trained well enough, I used just 700 images.
If you could provide more info on your training part (dataset size, average loss, etc.), I would try to help.

@akmaralAW  I meet same problem. but, i can't solve this problem.
yolov3 on darknet_ros is doing well. but my model does not work.
I train my data and get .weights, .cfg. Then, i created .yaml and .launch.
But mymodel on darknet_ros does not work(no detection, nothing). help....

@SanghyunPark01 I think there's some problem in this repository which need to be fixed. I've tried to use other ros darknet source code and successfully applied my trained weights in it. But it was 5 months ago and I didn't use it for so long, you can search for alternative developments, good luck!

@tabowsy thanks! I will try it


Support checkstyle report output #108. Support for the -r and -f para…

…meters of scss-lint.
This enables to set in options
format: 'Checkstyle',
require: 'scss_lint_reporter_checkstyle'

The names are in compliance with the CLI tool:
Usage: scss-lint [options] [scss-files]
    -f, --format Formatter           Specify how to display lints
    -r, --require path               Require Ruby file

As a side-effect, with my fix you can use all built-in formatters of scss-lint (Config, Files, JSON, XML) as long as you use the reporterOutput option of grunt-scss-lint.

Try setting the options.timeout configuration in Gruntfile.js to a higher value, e.g. 2500. It seems other developers struggle with similar problems.

AppVeyor fails, Travis passes. If I go to AppVeyor history for this project, I can't really find any build that was successful. Is it possible that it is misconfigured?

I've updated the PR. It passes on Travis. On AppVeyor things are more difficult. I had to tell mocha-istanbul to hide the output of mocha + istanbul. AppVeyor gets confused and treats that output as an error. But even with this, AppVeyor dies at the final command, regarding code climate. That one however in the documentation says it only works for the master branch.
In the PR I've done some further changes:

I've updated all gem and node dependencies to the latest, as the master is quite old.
I've set AppVeyor to run also for node 4.4 (Travis picks up the latest anyway)
Unit tests are written in various styles and most of them are integration tests anyway. I grouped them based on the technique that is being used.

At this point, I can't really proceed any further unless @ahmednuaman has a look at the PR and my findings regarding AppVeyor.
Also, I bumped up the version from 0.3.8 to 0.4.0 but perhaps it is worth bumping it up to 1.0.0 since all dependencies are upgraded.

Let me remove this appveyor nonsense.

Can you squash the commits please and hopefully that'll kick off the tests again.

Hm I don't see travis picking it up. I even added a dummy commit in .gitignore

Actually it works: https://travis-ci.org/ahmednuaman/grunt-scss-lint/builds/142527737 I don't know why it's not displayed inline in the PR in GitHub

Can you rebase please?

Of course, should be fine now! Here's the latest build: https://travis-ci.org/ahmednuaman/grunt-scss-lint/builds/142999385

I took out the 'rake' dependency from Gemfile.lock . Latest build: https://travis-ci.org/ahmednuaman/grunt-scss-lint/builds/143670110
It fails with message: "Your Gemfile.lock is corrupt. The following gem is missing from the DEPENDENCIES
section: 'rake'"

I also removed the 'rake' nested dependency underneath scss-lint, same result. Like I said initially, I have not added the 'rake' dependency myself. In Gemfile, it is not present. This is apparently a dependency of scss-lint and it gets written into the generated file Gemfile.lock once one runs 'bundle'.

Thanks for merging!


Analysis page

Changes to add a dashboard page.
This includes

The code for the page itself
The menu item
changes on the gulp.js build

This looks really great to me! Thank you @FabioFuschi-SNK!

This would partly implement #1627.

@Ein-Tim Thanks for the review.

Hi @FabioFuschi-SNK, look fantastic, however, I tested locally (Mac: Safari, Chrome) and found that some links are now broken:

all links to science blog articles on

http://localhost:8000/de/science/
http://localhost:8000/en/science/

Dropdown box (Version of screenshots) on

http://localhost:8000/de/screenshots/
http://localhost:8000/en/screenshots/

@Ein-Tim Since you seem to have tested locally, could you confirm, please? Thanks.
@brifemu Could you also locally build the page, please? Thanks.

Since you seem to have tested locally already, could you confirm, please? Thanks.

I can confirm this issue!

@Ein-Tim thanks. Good night!

First of all, this is amazing new functionality! Well done!
However ...
There are major changes being made by this PR, especially about links, which are unexplained.
I would feel more comfortable if the PR were to be split into separate ones which each had a clear purpose, were understandable on their own, were individually testable with a high confidence of not introducing unwanted side-effects, like breaking pages unrelated to analysis.
Also, for major changes like this which are being developed in a fork, it would be much more convenient for testing to first merge them into a separate branch on the https://github.com/corona-warn-app/cwa-website repository. Otherwise I have tended to clone contributors' forks locally, which takes up additional 3GB disk space each time (especially as the screenshots for each previous version accumulate), requires an additional redundant npm installation and has to be separately updated each time in order to test.
The contribution guidelines suggest squashing commits. This PR, with 102 commits, does not come close to following those best practice guidelines.
The cypress tests to check the functioning of the website have not been developed in parallel to the website development, so for instance there is no testing of the screenshot pages. @dsarkar found manually that this PR breaks the screenshot pages, even though cypress passed all tests. Consideration should be given to adding tests so that website testing can be done at least semi-automatically including all significant new functions.
If extensive new functionality is added by a new contributor from a fork it would be helpful in the PR to mention if the functionality is being developed on behalf of the Open Source Team. There was nothing in this PR to indicate that point and @FabioFuschi-SNK had never previously contributed to this project on GitHub.

@brifemu Could you also locally build the page, please? Thanks.

Tested on Windows 10 (Version 21H1 - OS Build 19043.1237) using Google Chrome Version 94.0.4606.61 (Official Build) (64-bit) with same errors

@MikeMcC399 Thank you very much for checking this PR! FYI - The content (Dashboard) of this PR was approved by RKI.
@FabioFuschi-SNK Could you have a look how to fix the broken links, please?  Thanks.

Hello @dsarkar, the broken links are fixed now. Sorry that i missed them.
Thanks for all other feedback, i  take it, and will make it in future better.

@FabioFuschi-SNK
The access to the science blog and screenshot older versions are now working again. 👍🏻
This PR seems to also resolve:

https://github.com/corona-warn-app/cwa-website/issues/1402 and
https://github.com/corona-warn-app/cwa-website/issues/1168 (warning ignored)

Can you explain the reason behind changing so many hyperlinks which are not related to the analysis page?


[BUG] NameError: name 'resource' is not defined; missing dependencies

Describe the bug
After a clean install and trying to use the CLI, the following error occurs due to missing dependencies:
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Users\-------\.virtualenvs\test_installs-jItMaX31\Scripts\rickle.EXE\__main__.py", line 4, in <module>
  File "C:\Users\-------\.virtualenvs\test_installs-jItMaX31\Lib\site-packages\rickled\cli.py", line 11, in <module>
    from rickled.net import serve_rickle_http, serve_rickle_https
  File "C:\Users\-------\.virtualenvs\test_installs-jItMaX31\Lib\site-packages\rickled\net.py", line 25, in <module>
    class HttpResource(resource.Resource):
                       ^^^^^^^^
NameError: name 'resource' is not defined

To Reproduce
Steps to reproduce the behavior:

install in clean env.
use rickle as cli tool

Expected behavior
No exception..

done


feat: Allow instance store policy to be configured on BR family

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

51 of 51 (100.0%)  changed or added relevant lines in 12 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 83.394%

Totals

Change from base Build<PHONE_NUMBER>4:
0.4%

Covered Lines:
5544

Relevant Lines:
6648

💛  - Coveralls


Wish to support on Apple Silicon.

➜  ~ brew install imageoptim-cli
imageoptim-cli: The x86_64 architecture is required for this software.
Error: imageoptim-cli: An unsatisfied requirement failed this build.

+1

A workaround is to install the Mac App (https://imageoptim.com/mac) and instal CLI with node.


Installed Homebrew formulas are always preferred over other ways to satisfy dependencies

Build#expand_reqs has the following logic:
      elsif req.satisfied? && req.default_formula? && (dep = req.to_dependency).installed?
        deps << dep
        Requirement.prune
      end

This is what we want if the requirement was satisfied by the installed default_formula. It is probably not what we want if satisfy found some other way to satisfy the requirement.

Can you elaborate on why this is a problem and what formulae (if any) it's currently negatively affecting? Thanks!

The way I discovered this was I was trying to test macvim against a pyenv python3 with Homebrew python3 installed. Even though the pyenv python3 was first in PATH, --with-python3 (which triggers a Python3Requirement) ended up building against the Homebrew python3.
We sometimes describe the behavior of the PythonRequirements as "building against the first python in PATH," which is not exactly the observed behavior.
We could wontfix as a documentation bug; I'd prefer that we finally fully-bake an API for adding paths to superenv environments as the result of a satisfy operation.

(Sorry, filed this on a plane, didn't want to forget about it)

Even though the pyenv python3 was first in PATH, --with-python3 (which triggers a Python3Requirement) ended up building against the Homebrew python3.

I see. Makes sense to tweak this (but obviously will need tested with a bunch of other things as this code is prone to 💥).

I'd prefer that we finally fully-bake an API for adding paths to superenv environments as the result of a satisfy operation.

Something beyond Requirements? Feels like that may be overkill.

@tdsmith I think this may work:
diff --git a/Library/Homebrew/build.rb b/Library/Homebrew/build.rb
index 8915e94..22b1dd9 100644
--- a/Library/Homebrew/build.rb
+++ b/Library/Homebrew/build.rb
@@ -49,7 +49,7 @@ class Build
         Requirement.prune
       elsif req.build? && dependent != formula
         Requirement.prune
-      elsif req.satisfied? && req.default_formula? && (dep = req.to_dependency).installed?
+      elsif req.satisfied? && req.default_formula? && req.uses_default? && (dep = req.to_dependency).installed?
         deps << dep
         Requirement.prune
       end
diff --git a/Library/Homebrew/requirement.rb b/Library/Homebrew/requirement.rb
index 7a40d41..f653b00 100644
--- a/Library/Homebrew/requirement.rb
+++ b/Library/Homebrew/requirement.rb
@@ -56,9 +56,17 @@ class Requirement
   def satisfied?
     result = self.class.satisfy.yielder { |p| instance_eval(&p) }
     @satisfied_result = result
+    if default_formula? && Pathname === result
+      installed_prefix = Formula[default_formula].installed_prefix
+      @uses_default = result.realpath.to_s.include? installed_prefix
+    end
     !!result
   end

+  def uses_default?
+    @uses_default
+  end
+
   # Overriding #fatal? is deprecated.
   # Pass a boolean to the fatal DSL method instead.
   def fatal?
diff --git a/Library/Homebrew/requirements/python_requirement.rb b/Library/Homebrew/requirements/python_requirement.rb
index 2800f52..7948f4c 100644
--- a/Library/Homebrew/requirements/python_requirement.rb
+++ b/Library/Homebrew/requirements/python_requirement.rb
@@ -11,7 +11,7 @@ class PythonRequirement < Requirement
     version = python_short_version
     next unless version
     # Always use Python 2.7 for consistency on older versions of OSX.
-    version == Version.create("2.7")
+    version == Version.create("2.7") ? python : false
   end

   env do

We could wontfix as a documentation bug

I'd prefer this, actually. We know building against our default_formula is definitely going to work so I do think it should be preferable if already installed.


Turn on the AJP connector based on a hiera or puppet-variable lookup

I tried to make it similar to the way that proxy is set up, such that you can add additional parameters to the Connector object.

Thank you. Would you consider adding some tests to: spec/classes/jira_config_spec.rb
Its not a requirement, but perhaps also add validate_hash($ajp) to init.pp


chore: add secure base image dockerfile alternative

Hey, I just made a Pull Request!
chore: add secure base image dockerfile alternative
I have created an alternative Dockerfile that reduces vulnerabilities based on wolfi-base. The Dockerfile successfully runs the Backstage application and generated the techdocs locally in the container.
Please refer to https://github.com/backstage/backstage/issues/23323 for more information regarding this PR and the reduction in vulnerability counts.
Closes https://github.com/backstage/backstage/issues/23323
If this alternative is successful, consideration of changing the default Dockerfile would be beneficial to the community as this Dockerfile reduces the vulnerabilities in Backstage by 386 vulnerabilities based on data from trivy.

:heavy_check_mark: Checklist

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

A changeset was not needed

changesets are only needed for changes to packages within packages/ or plugins/ directories

Comparing upstream dockerfile to this dockerfile

This is awesome @karlhaworth! I validated the CVE count of the backend image:
➜  backstage git:(master) ✗ grype example-backend-bookworm:latest  ✔ Vulnerability DB                [updated]    ✔ Loaded image                                                                                example-backend-bookworm:latest  ✔ Parsed image                                                                          sha256:c94454d5aa56e389d4d1ecab16e3008df265847ca60aab541a7cb698233f8f39  ✔ Cataloged contents                                                                           28cd5b2e61e21f11efe1a66f28377b270ae36dc593a22b5f838089367ac58a30    ├── ✔ Packages                        [2,288 packages]      ├── ✔ File digests                    [8,937 files]      ├── ✔ File metadata                   [8,937 locations]      └── ✔ Executables                     [911 executables]    ✔ Scanned for vulnerabilities     [391 vulnerability matches]      ├── by severity: 2 critical, 35 high, 95 medium, 7 low, 210 negligible (42 unknown)    └── by status:   10 fixed, 381 not-fixed, 0 ignored
And it looks like a dramatic reduction!
➜  backstage git:(master) ✗ grype example-backend:latest  ✔ Vulnerability DB                [no update available]    ✔ Loaded image                                                                                example-backend:latest  ✔ Parsed image                                                                          sha256:7499a76de08ba5ddf9a3ac85e7792372be6bde6fbf8928f821c3dc65e992dbf0  ✔ Cataloged contents                                                                           cbf583f1287bffe76f6bdeee7ba0a8435c4a51f661e1caf3d3e038b31e860c02    ├── ✔ Packages                        [2,187 packages]      ├── ✔ File digests                    [9,736 files]      ├── ✔ File metadata                   [9,736 locations]      └── ✔ Executables                     [289 executables]    ✔ Scanned for vulnerabilities     [8 vulnerability matches]      ├── by severity: 1 critical, 1 high, 5 medium, 1 low, 0 negligible    └── by status:   5 fixed, 3 not-fixed, 0 ignored

This is awesome @karlhaworth! I validated the CVE count of the backend image:

Glad to see similar results! Thanks for validating the vulnerabilities.
I'm not sure how many of those 8 vulnerabilities you show were OS level, but in my testing I saw 0 OS level vulns as shown in https://github.com/backstage/backstage/issues/23323. 🤔 Seemed easier to solve the node vulns than it is to solve the OS level vulns in some cases?
Internally we got the count down to 0 through resolutions and other means. It's much easier to handle 8 vulns than it is to handle 391!

I can confirm that these are coming from node:
NAME                   INSTALLED  FIXED-IN  TYPE  VULNERABILITY        SEVERITY
axios                  0.27.2     0.28.0    npm   GHSA-wf5p-g6vw-rhxx  Medium
es5-ext                0.10.62    0.10.63   npm   GHSA-4gmj-3p3h-gm8h  Low
got                    9.6.0      11.8.5    npm   GHSA-pfrx-2q88-qq97  Medium
ip                     2.0.0      2.0.1     npm   GHSA-78xj-cgh5-2h22  Medium
lodash.pick            4.4.0                npm   GHSA-p6mc-m468-83gw  High
monorepo-symlink-test  0.0.0                npm   GHSA-2jcg-qqmg-46q6  Critical
request                2.88.2               npm   GHSA-p8p7-x288-28g6  Medium
tough-cookie           2.5.0      4.1.3     npm   GHSA-72xf-g2v4-qvf3  Medium

In 38c5e7e I changed from node 20 to 18 to better match upstream. Node 20 was tested and needed one more item added to the dockerfile to make it work ENV NODE_OPTIONS="--no-node-snapshot".

I want this open still. Thanks!

Still want to keep this opened. Waiting on reviews.

Keep open! Waiting for reciew


removed dependency of Mongo adapter to Mongoid

I'm not happy with including/requiring Mongoid ORM if I'm on pure mongo.rb database adapter.

@codegourmet Ok, seems reasonable. Thanks!


Do we need internet connectivity always to verify the access key?

Even if the processing is offline for porcupine, it requires an access key to create an instance.
So, ideally, does this mean we need some sort of connectivity always? Or, can we just create an instance once and run the other part of offline processing algo (detection) completely and truly offline?
Working on Windows system and RaspberryPi.

What about devices which remain offline for weeks or months? Will they just suddenly stop working?

Yes,  according to the communication

Given this, calling it "offline" is disingenuous. I can't be sure of online connectivity and I can't have it cease to function, so I'll have to look for another solution.

It's a shame that communication has to be done every few days. Doesn't seem logical to have it set up that way since it's a software for a specific type of application which not always imply that there is a network connection. It should be enough with one -time verification only!


help to newbies which not similar with npm

Hello, i am trying to install and use this package. I am not familiar with npm.
Can you provide step by step instruction how use it on linux pc without node.js and npm.
I am run "npm install plexsystems/rabbit-viz" and script collect over 250mb other modules
npm run start  do nothing
cd node_modules/rabbit-viz/
npm run start  throw errors

<EMAIL_ADDRESS>start /home/noone/waiter2call/node_modules/rabbit-viz
react-scripts start

It looks like you're trying to use TypeScript but do not have typescript installed.
Please install typescript by running npm install typescript.
If you are not trying to use TypeScript, please remove the tsconfig.json file from your package root (and any TypeScript files).
npm ERR! Linux 4.15.0-46-generic
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "run" "start"
npm ERR! node v8.10.0
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: react-scripts start
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'react-scripts start'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the rabbit-viz package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     react-scripts start
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs rabbit-viz
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls rabbit-viz
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/noone/waiter2call/node_modules/rabbit-viz/npm-debug.log
noone@harorah:~/waiter2call/node_modules/rabbit-viz$
can you help me how to use it?

Thanks for reaching out, @AntonSikidin. I agree, we should probably update our documentation to include how to get rabbit-viz up and running from a clean state.
To get this project to work locally, you should just need to do the following:

git clone https://github.com/plexsystems/rabbit-viz.git
(this will download the source to the current working directory, a folder named rabbit-viz should appear)

cd rabbit-viz

npm install
(this will install all of the required modules and dependencies. it might take a little bit, so just wait until the downloads are complete)

npm run start
(this will start up a local copy of the rabbit-viz project. A browser should appear and rabbit-viz should be visible. If for some reason it's not, try navigating to http://localhost:3000 after rabbit-viz has finished the run command)

Alternatively, if you're just looking to use rabbit-viz, this repository hosts a working version at https://plexsystems.github.io/rabbit-viz/

thanks, it works,
another question: where i can generate json from my rabbitmq server to view?

previously i think that  rabbit-viz get data from running server

answer
http://localhost:15672/#/ ->overview ->Export definitions->Download broker definitions

Here's the docs on the RabbitMQ site for exporting definitions: https://www.rabbitmq.com/backup.html#definitions-export
TL;DR: You can retrieve the definitions for your server a couple of ways:

Make a GET request against the management api at /api/definitions
Download it from the management UI overview pane

feature request:
add button to load definition from local server

http://localhost:15672/api/definitions  work fine, but atomaticaly it runs better


sudo cleanup for services status

This should work without having to do anything with sudo:
$ coins services status
[sudo] password for jragle: [sudo] password for jragle: [sudo] password for jragle: [sudo] password for jragle:

you should work without having to do sudo! ;)

HAH!


Events no longer received in 0.11

Hi All,
I just noticed that my app is no longer receiving events with 0.11 - I normally use this to be notified of the "Next Track" when "random" is enabled.
Looking at the log I can see all the subscribe details:
Event listener started
Event listener running on ('<IP_ADDRESS>', 1400)
Subscribed to http://<IP_ADDRESS>:1400/MediaRenderer/AVTransport/Event, sid: uuid:RINCON_000E58C8FC4601400_sub0000000063

I guess that when these notifications are received I would then see:
Event content: ...

In the log, but I never see that.
Is anyone else seeing this?
(Sorry for raising a bug, wasn't really sure the best way to ask the question)
Thanks
Rob

Hi @robwebset
Sorry about the long wait on this. Can you share code and instructions to reproduce, that I will check it here.
Regards Kenneth

Hi @KennethNielsen
Not on my Sonos system at the moment, but it is pretty much like this:
sonosDevice = SoCo("IP-ADDESS")

# Subscribe to receive notifications of changes
sub = sonosDevice.avTransport.subscribe()

keepLooping = True
queueItem = None
while keepLooping:
        try:
            # Get the most recent event received
            while not sub.events.empty():
                try:
                    # Get the next event - but do not block or wait for an event
                    # if there is not already one there
                    queueItem = sub.events.get(False)
                except:
                    log("Sonos: Queue get failed: %s" % traceback.format_exc())

            # Now log the details of an event if there is one there
            if queueItem is not None:
                log("Event details: %s" % queueItem)
        except:
            log("Sonos: Failed to get latest event details: %s" % traceback.format_exc())

	if queueItem is not None:
		keepLooping = False

Thanks
Rob

If I run the example code from the events module on 0.11.1:
from __future__ import print_function
try:
    from queue import Empty
except:  # Py2.7
    from Queue import Empty

import soco
from pprint import pprint
from soco.events import event_listener
# pick a device at random
device = soco.discover().pop()
print (device.player_name)
sub = device.renderingControl.subscribe()
#sub2 = device.avTransport.subscribe()

while True:
    try:
        event = sub.events.get(timeout=0.5)
        pprint (event.variables)
    except Empty:
        pass

    except KeyboardInterrupt:
        sub.unsubscribe()
        sub2.unsubscribe()
        event_listener.stop()
        break

and only listen for rendering control, then yes, I get events.

Hallo @robwebset
I just remembered that you are probably using this for the Kodi plugin, in which case you may not see the exceptions. All the event parsing exceptions happen in their own threads, which mean that they do not make the program stop. So, if you do not see terminal output, but simply wait for events on the queue, it would seem as if none is ever received. So, if this is the case, this issue is probably the same as #276.

Hi @KennethNielsen
Thanks for your assistance - I managed to get a little time today to look more at this - I think the Sonos has actually stopped sending the "Next Track" information in events now (last firmware update) - as the way the playlist is generated has changed - so the next item in the playlist (even if random is enabled) will always be the next track (Which was not the case if random was enabled previously)
Thanks
Rob


Allow user to view site as if it was a point in the past

I'm not sure what this means - just not showing posts/replies after a certain point or what?
Seems like archive.org can handle this OK for us, I'd expect!


[INFRA] Updated project dependencies

Some of the updated dependencies are provided by the runtime (e.g. Spark provides Paruqet). Additional manual checks are needed before this can be merged, as it might cause binary incompatibilities.
The versions in emma-quickstart/pom.xml should be synced, and the emma-archetype artifacts should be updated with
./archetype.sh <path-to-emma-project>

Then run
mvn clean install -DskipTests

and go over the Project Setup steps.

Updated emma-quickstart in sync with emmalanguage/emma-quickstart#1


Using the right domain separator when hashing to BLS12-381 G1

Working on #74 I am running into a problem I am not sure how to address.
G1.hashToCurve uses the G2 domain separator ("BLS_SIG_BLS12381G2_XMD:SHA-256_SSWU_RO_NUL_") instead of the appropriate one for G1. I tried to fix this with the following change:
diff --git a/src/bls12-381.ts b/src/bls12-381.ts
index d4c7250..700b27e 100644
--- a/src/bls12-381.ts
+++ b/src/bls12-381.ts
@@ -1075,7 +1087,10 @@ export const bls12_381: CurveFn<Fp, Fp2, Fp6, Fp12> = bls({
     ),
     a: Fp.ZERO,
     b: _4n,
-    htfDefaults: { ...htfDefaults, m: 1 },
+    htfDefaults: { ...htfDefaults, m: 1,
+                   DST: 'BLS_SIG_BLS12381G1_XMD:SHA-256_SSWU_RO_NUL_',
+                   encodeDST: 'BLS_SIG_BLS12381G1_XMD:SHA-256_SSWU_RO_NUL_',
+                 },

but it does not seem to have any effect.
I can work around this by explicitly setting the domain separator to use when signing or verifying a short signature, but this feels pretty awkward/error prone.
I think this is missed by the existing hash to curve tests since they always use some explicit dst.

I can work around this by explicitly setting the domain separator to use when signing or verifying a short signature, but this feels pretty awkward/error prone.

Why?

G1.hashToCurve uses the G2 domain separator

I don't think there is a such thing as standard G1 or G2 DSTs.

Why?

Mostly because if you don't specify it, you'll use not just a wrong domain separator but even one that is reserved for G2.

I don't think there is a such thing as standard G1 or G2 DSTs.

Maybe not standard per se but this internet draft specifies the DSTs for G1 and G2 https://datatracker.ietf.org/doc/html/draft-irtf-cfrg-bls-signature-04#section-4.2.1 and the current G2 default DST used in this library is exactly the one from that draft.
That same domain separator is also used by https://github.com/ethereum/py_ecc https://github.com/supranational/blst https://github.com/filecoin-project/lotus and those are just the first three that pop up if I search that string. I think despite being just a draft every BLS implementation out there has followed it since there is nothing better wrt standardization.

there is nothing better wrt standardization

true
So, this feels like a miss from my end. If you could fix it, that would be great.

Fix for this is included in #74


Don't resort categories if there are only 2-3 categorical values

Even if dynamic sorting is used, if all 2-3 categories are visible on the screen, resorting does not add any value. The relative difference/change is apparent in few number of items.
Solution: Auto-set no_resort: true in sortingOptions when this case happens.

Done. Static ordering limit is <=4 categories.


Fix for XCode9

Declare selector visible to ObjectiveC
Remove unnecessary '_ in' in the beginning of a block

Hi @ipavlidakis.
Indent of my code is 2space because according to the swift-style-guide
Please be close If you do not have mind to fix.
Thanks PR!


Requesting document update

Hi all - some key information is missing from this documentation.
If the customer (source) has a different currency than the partner (destination), reserved instances will be transferred but then automatically cancelled on the next billing cycle

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

ID: 3673197a-4414-5a4c-b717-3aaa7529c5ed
Version Independent ID: 4656d633-57a0-a8b8-29b9-5f5479c5e48a
Content: Transfer Azure product billing ownership to your Microsoft Partner Agreement (MPA) - Microsoft Cost Management
Content Source: articles/cost-management-billing/manage/mpa-request-ownership.md
Service: cost-management-billing
Sub-service: billing
GitHub Login: @amberbhargava
Microsoft Alias: banders

@bec-smith
Thanks for your feedback! We will investigate and update as appropriate.

@bandersmsft Is this expected behavior? If yes, can you update the document?

@bec-smith Hi there. I've sent email to the subscription transfer PM feature owner and CCed you. When I have actionable info, I'll reply here.

@bec-smith Per the Subscriptions engineering team, this is not something that should be documented.
Accordingly, closing this issue.
#please-close


Nuget Package can't find dll in WinRT Component(C#) referenced by UWP(JS) project.

Hi,
I created a UWP Project based on javascript and added a windows runtime component of C#.
In the Component I added SQLite.Net-PCL nuget package and use it.
In UWP main project, I call the component function. And the FileNotFound exception was thrown.
Same problem here: SO Case.
Currently the workaround is to add the dlls manually.
But still curious to know the root cause of this :)

Sorry I don't control the SQLite.Net-PCL package. This code is released as sqlite-net-pcl


Why does this happen when I switch to another application?

When the rigid body moves or does not access the page for a period of time, it seems that a huge force has been applied to the rigid body
https://github.com/schteppe/cannon.js/assets/70420794/b578d4dd-a8e7-41fa-a342-315c10362fbb
If I keep accessing this page, it's normal
https://github.com/schteppe/cannon.js/assets/70420794/e9fc621d-d648-4280-a918-cdeec573847b
What is the reason for this? Where should I learn relevant knowledge and fix it

I have fixed it，The following code needs to be added
delta = Math.min(delta, 1 / 10);    // min 30 fps


Auto create tables?

In one of the earlier versions of sprinkles, the Migration class had a createTable() method that seemed to make it much easier to manage the create statements. Is there anything similar in the current 1.3.1 version that would prevent me from having to craft my own CREATE statement?

Same question here...


Images have actions with obsolete node

presubmit-roundup
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: technote-space/workflow-conclusion-action@45ce8e0eb155657ab8ccf346ade734257fd196a5. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
As seen on https://github.com/chainguard-images/images/actions/runs/8631273464 please upgrade if possible

https://github.com/technote-space/workflow-conclusion-action says

This repository has been archived by the owner on Nov 16, 2023. It is now read-only.


chore: Carousel Component is island

Carousel is good.
But carousel is island.
Carousel don't work in components because carousel use JavaScript.
So, need to add 'island' tag for developers.

I have used a CSS only carousel as a component (a refactored version of
this with TWIND or PICO CSS)
link to working codepen: A CSS-only Carousel Slider (codepen.io)
https://codepen.io/Schepp/pen/WNbQByE
On May 9, 2023, SleepyShay @.***> wrote:

Carousel is good.
 But carousel is island.
 Carousel don't work in components because carousel use JavaScript.
So, need to add 'island' tag for developers.

You can view, comment on, or merge this pull request online at:
https://github.com/denoland/fresh/pull/1157
Commit Summary

6a850e7
https://github.com/denoland/fresh/pull/1157/commits/6a850e7a9528aa0f0a1b7b47d782b836f1d0ba65
chore: Carousel Component is island

File Changes

(1 file https://github.com/denoland/fresh/pull/1157/files)

M
www/islands/ComponentGallery.tsx
<https://github.com/denoland/fresh/pull/1157/files#diff-
03b1c2e9c80559bd058a0602450a5a7a1b5726a983bf3381f973d0578a4f590d>
(2)

Patch Links:

https://github.com/denoland/fresh/pull/1157.patch
https://github.com/denoland/fresh/pull/1157.diff
—
Reply to this email directly, view it on GitHub
https://github.com/denoland/fresh/pull/1157, or unsubscribe
<https://github.com/notifications/unsubscribe-
auth/ATXWUNTC55MD4VLNZNXSQHTXFJE67ANCNFSM6AAAAAAX3K5MTI>.
You are receiving this because you are subscribed to this
thread.Message ID: @.***>

Oh sry. "Carousel Components" I said is #1096.
The carousel works in  only islands, not components.
But I think your carousel is nice idea.


Remove automatic "vertical" upcasting of union columns.

We've decided to drop the functionality of automatic upcasting the columns.
It's complicated at the moment because of the parallel outputs/fields that we need to use.
We've decided to do this in the future after the refactoring that will merge outputs and fields.
Postgres also asks for excplicit casts when > 2 relations.

Merged & Squashed in union-new branch


opt(WIP) optimize agg and window normalization step 2

refactor aggregate normalization to avoid data amplification before aggregate
move expression normalization and optimization after plan normalization to avoid unexpected expression optimization.
remove useless aggreagte processing in ExtractAndNormalizeWindowExpression

TODO:

push down redundant expression in aggregate functions
refactor normalize repeat rule

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.62 seconds
stream load tsv:          426 seconds loaded<PHONE_NUMBER>9 Bytes, about 167 MB/s
stream load json:         24 seconds loaded<PHONE_NUMBER> Bytes, about 93 MB/s
stream load orc:          59 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230505104644_clickbench_pr_138691.html

run buildall

run buildall

run buildall

run buildall


update site status abandonment clarification

Clarification of the steps made during well abandonment

[ ] Content Review @LizDerrington

[ ] Technical Review @LukeHauck

I approve. The definition is comprehensive and good.

I approve. The definition is comprehensive and good.

Tech good. approved.

Tech good. approved.

I approve. Content is logical and good.


SSBench only creates containers then hangs

Running ssbench with a very simple scenario is resulting in container creation but nothing else.
Output remains as below:
"INFO:Ensuring 1 containers (test02_*) exist; concurrency=10...
INFO:Initializing cluster with stock data (up to 1 concurrent workers)"
Containers are created but no file creations occur.
I have searched the web and openstack forums other have this issue but no answers, just one comment saying read the docs here (which I have).
Please advise.  Here's my scenario.
{
"name": "Small test scenario - Upload to 1 container",
"sizes": [{
"name": "tiny",
"size_min": 4096,
"size_max": 65536
}, {
"name": "small",
"size_min": 100000,
"size_max": 200000
}],
"initial_files": {
"tiny": 10,
"small": 10
},
"operation_count": 10,
"crud_profile": [100, 0, 0, 0],
"user_count": 1,
"container_base": "test02",
"container_count": 1,
"container_concurrency": 10
}

Also should add that create, update delete of files works with swiftclient.

Are you running with --keep-objects?

No but I just tried.  Same result and again the job just appears to hang indefinitely (for days if I allow).

@elligottmc you may have already resolved this or moved on from this issue, but if you're still stuck on this, could you paste the invocation command you were using?

I have the same issue on Fedora 23. Container is created and then nothing happens
./bin/ssbench-master -v run-scenario -f share/ssbench/scenarios/very_small.scenario -S "$STORAGE" -T $TOKEN -u 1 -c 1 -p Policy-0
2016-03-26 22:19:25,206:INFO:SwiftStack Benchmark (ssbench version 0.3.9)
2016-03-26 22:19:25,210:INFO:Starting scenario run for "Small test scenario"
2016-03-26 22:19:25,211:DEBUG:Using token fd2d4900b0284390bdfb6409b61a39bf at one of ['http://controller:8080/v1/AUTH_a1c551bbbf4f4affa940680daaab45c7']
2016-03-26 22:19:25,211:INFO:Ensuring 1 containers (ssbench_*) exist; concurrency=10...
2016-03-26 22:19:25,246:DEBUG:REQ: curl -i http://controller:8080/v1/AUTH_a1c551bbbf4f4affa940680daaab45c7/ssbench_000000_Policy-0 -I -H "X-Auth-Token: fd2d4900b0284390bdfb6409b61a39bf"
2016-03-26 22:19:25,246:DEBUG:RESP STATUS: 204
2016-03-26 22:19:25,246:INFO:Initializing cluster with stock data (up to 1 concurrent workers)
2016-03-26 22:19:25,247:DEBUG:active: 0 concurrency: 1
It gets stuck at
lib/python2.7/site-packages/ssbench/master.py
line 205
self.work_push.send(msgpack.dumps(send_q))
When pyzmq is installed there is an error while building the package. Maybe it is somehow connected
cc -c /tmp/timer_createN80gIj.c -o build/temp.linux-x86_64-2.7/tmp/timer_createN80gIj.o
/tmp/timer_createN80gIj.c:1:1: warning: return type defaults to ‘int’ [-Wimplicit-int]
 main (int argc, char **argv) {
 ^
/tmp/timer_createN80gIj.c: In function ‘main’:
/tmp/timer_createN80gIj.c:2:5: warning: implicit declaration of function ‘timer_create’ [-Wimplicit-function-declaration]
     timer_create();
     ^
cc build/temp.linux-x86_64-2.7/tmp/timer_createN80gIj.o -o build/temp.linux-x86_64-2.7/a.out
build/temp.linux-x86_64-2.7/tmp/timer_createN80gIj.o: In function `main':
timer_createN80gIj.c:(.text+0x15): undefined reference to `timer_create'
collect2: error: ld returned 1 exit status

@MaximBB can you also paste how the workers are started?
I'll try to reproduce this on a Fedora 23 VM.

There is actually issue with workers but I haven't investigated it further yet
[root@qtest ssbench-env]# ./bin/ssbench-master -v run-scenario -f share/ssbench/scenarios/very_small.scenario -S "$STORAGE" -T $TOKEN -u 1 -c 1 -p Policy-0 --workers 1
2016-03-26 23:24:25,972:INFO:SwiftStack Benchmark (ssbench version 0.3.8)
2016-03-26 23:24:25,974:INFO:Spawning local ssbench-worker (logging to /tmp/ssbench-worker-local-0.log) with ssbench-worker --zmq-host <IP_ADDRESS> --zmq-work-port 13579 --zmq-results-port 13580 --concurrency 1 --batch-size 1 -v 0
Traceback (most recent call last):
File "./bin/ssbench-master", line 614, in
args.func(args)
File "./bin/ssbench-master", line 217, in run_scenario
close_fds=True))
File "/usr/lib64/python2.7/subprocess.py", line 710, in init
errread, errwrite)
File "/usr/lib64/python2.7/subprocess.py", line 1335, in _execute_child
raise child_exception
OSError: [Errno 2] No such file or directory

@MaximBB thanks for the quick reply. If the worker doesn't start (as in the above output), the master will hang forever waiting for any worker to connect.
The most likely cause of the above error is related to finding the worker binary in the $PATH. You could work around this in one of two ways:

Start a separate worker process before starting the master:
./bin/ssbench-worker --zmq-host localhost 1
(or a different host for multi-node experiments; the "1" is a worker ID argument. It only needs to be unique between workers)
Add the path to the worker to $PATH:

PATH=<ssbench virtual env path>/bin:$PATH ./bin/ssbench-master -v run-scenario -f share/ssbench/scenarios/very_small.scenario -S "$STORAGE" -T $TOKEN -u 1 -c 1 -p Policy-0 --workers 1

Can you try either (or both) of these and report back? Also, I noticed that you're running this as root. That should not be required. If you run into issues using ssbench as a regular user, please report those as well.

@timuralp thanks a lot!
Both proposed options worked well!
It would be still useful if an error is raised if a worker process is not available. For new users even after reading available documentation it is not obvious

@MaximBB glad that got resolved.
Good suggestion -- I will see what we could improve to make it more obvious that the master is waiting for a worker and a better error message when we cannot find the worker in the PATH.


Netcat listen mode injection leaks processes

when netcat is used with -l AND -k stdout of the child process will close, but as long as stdin is not actively closed, the reading of request will hang forever. this has no functional effect, ebcause due to the -k a new handler is started, but the part parsing the incoming request nevre terminates.
to fix this we should intoriduce a wrapper that forces stdin closed when stdout terminates

the wrapper that closes stdin on stdout close doesnt work because the actual observing wrapper keeps stdout open. maybe we could let these two interact and kill the outer wrapper when the inner wrapper reports the actual process dies


STM32f103 A4963

Hello
I have STM32f103.Can i controll motor speed with spi? What ı need to write to reigsters using spi? For test i need connect allegro 4963 to motor or can i see analog values on osilascope without connecting the motor? Can you help me? I need level shifter for shift 3.3v to 5v ? (Spi)

Motor speed is controlled by PWM, SPI is used to configure the controller parameters.
The speed example code, together with the datasheet, should give you an idea of what registers do. Note that there are a lot of parameters, but most (if not all) are optional and don't need to be changed.
Without a motor, the outputs you will measure will not be relevant, since the controller use feedback from the motor.
You do not need level shifter, SPI and PWM work with 3.3V (and are also 5V tolerant, IIRC)
Let me know if you want a schematic, i probably have that somewhere.

I read the datasheet and i see i can controll motor speed using spi.I m wrong?
If i cant how can i drive it using pwm? what is spi data needed for it? what i need send? and i read i need PWM between 5Hz-1kHz.If you have schematic i can check my hardware desing.
Thanks

The normal way to control speed is by pwm, and i don't see any register that allows to control it via spi, but maybe i miss something. What page of the datasheet are you refering to?
SPI is used to modify the various options of the controller (eg enable/disable, direction, brake, control mode, PI controller parameter, speed range, etc). For example if you want the controller to keep a constant speed independenlty of motor load, you need to select closed-loop speed control, and that can only be done with spi.
brushless_controler_mini.pdf

I see SPI controll on datasheet i will send it when i have time.Currently testing a4963 i dont have problem with hardware.I have one qeustion about controlling motor using pwm
I want basicly controll motor using PWM.How can i do this.Can you write steps to what i need to? Motor speed changed by PWM.I just turn motor without load.
Which spi value required to send  a4963 for controll motor with PWM?
Which parameters i need config?

Looks good, the motor should start.
What happens? The motor doesn't move at all? It moves erratically?
Does the A4963 detect a fault? You can read the diagnostic register via spi for more details (eg a faulty transistor).
Is the power supply sufficiently powerfull? A motor may need a lot of current at startup.

I fixed problem.There is a problem with PWM cable.Its broken :) I replace it but now It moves erratically.I try fix it

Yeah, it will work better with all the wires connected ;-)
For the erratic movement, you can try to play with the startup duty cycle and startup speed parameters.
Note that it will depend on the motor load, so you may want to experiment with the load on the motor.

I can run other motor but unable run my own motor its looks like this.Other motor runs very easyly with any config its simple but unable run my motor its make a realy trouble.I can run it with 30A esc but a4963 cant run it thats stupid :/
My motor same as linked
https://www.banggood.com/Wholesale-XXD-A2212-KV1000-Brushless-Motor-H363-For-RC-Airplane-Quadcopter-p-57432.html?cur_warehouse=CN
You dont need PWM for controll motor speed.You can set motor speed using SPI.You need change Run Register
Like this
0xF0 09  //Motor speed is based on input pwm signal
0xF0 F9 //Full speed without pwm
So with this litle spi value you can change motor speed.
Thanks for helps


Not getting constant types

Seems the constants are produced without their types?

This was fixed in the main project already


Component name label proposal

Here is a proposal of showing in the component's box a little label with the component name while the user hovers the box.

Its as simple as adding this code to the component's code:

{componentName}

When I first discovered this project I really struggled to identify the individual components while I didn't want to have all of these inputs in my codebase.

Hey @baginski11, thanks for the feedback! We’re currently focused on building more components, but we’ll definitely add those changes down the line. P.S. A trick to easily find the component in the project files is copying the component code and search in your editor ;)


macOS: When switching between macOS dark mode and light mode, current article scrolls to top

I was nearly finished with a long article while my OS was in dark mode due to its "Auto" setting; the sun rose, macOS switched to light mode, and the article scrolled to the top.
I've tested this with manual changes to light and dark mode and the same thing happens.
NNW 5.1.2b2 (3014)
macOS 10.15.7 (19H2)

This is fixed and will be in the upcoming 6.0 release.


adjusted syntax in line 72 to not break, since it wanted to substract…

… strings resulting in this error:
test.sh: line 72: "1539849411" - "1539849393" : syntax error: operand expected (error token is ""1539849411" - "1539849393" ")
removing the quotes will handle the variable as integer and not as a string.

Thank you! @CayoM 🎉


Clicking on Extension hub in SPSS warns: Pre-requisites not available

Have SPSS 25 and have installed R 3.3.1.
Downloaded the zip file and unpacked. When I open SPSS -> Extensions -> Extension Hub
I get the above error.
Am I missing a step?
Thank you,
Yara

You need to install this via the Extensions > Extension Hub menu item.

This is what we tried to do, but we get the error: 'Pre-requisites not available'. It's as if SPSS doesn't recognize that the extension was downloaded.

Try downloading the STATS_R33_CONFIGURATION package and then installing it locally using Extensions > Install Local ...

It kepps downloading and installing STAT_R33_CONFIGURATION..  What shoulod I do?

Have you installed the appropriate version of R first?  What version of
Statistics are you using and on what platform?
On Thu, Mar 12, 2020 at 8:30 PM learang<EMAIL_ADDRESS>wrote:

It kepps downloading and installing STAT_R33_CONFIGURATION.. What shoulod
I do?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/IBMPredictiveAnalytics/STATS_R33_CONFIGURATION/issues/2#issuecomment-598518378,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAEWQPU3V6ETKE2K4C5O73RHGLFXANCNFSM4HV3RFNQ
.

--
Jon K Peck
<EMAIL_ADDRESS>


How to specify payload encoding to base64?

I cannot found "payload encoding" in  params of Publising, did you miss it?

See this example.

Thanks very much.
But it is not what i referred to. It is content-encoding, not
payload-encoding. I want the payload stored in base64 format.
It finally solved by just passing bytes to rabbitmq, it is encoded
automatically!
My mistake. Thank u very much.
On Sat, Sep 16, 2017 at 12:42 AM, Michael Klishin<EMAIL_ADDRESS>wrote:

See this example
https://github.com/streadway/amqp/blob/2e25825abdbd7752ff08b270d313b93519a0a232/_examples/simple-producer/producer.go#L88
.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/streadway/amqp/issues/289#issuecomment-329835436, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAjJWi2x16ZWWE3I2UhnJFp2ioPAZhYQks5siqjkgaJpZM4PZMsk
.

@kkzhang RabbitMQ treats message payloads as opaque streams of bytes. You can pass anything you want for message body. The above link demonstrates how to set a message property that will communicate to other apps what the content encoding really is (it's up to the apps to make use of it, client libraries do not perform any automatic serialisation).


MNE Fiducials and Their Usage in mne.coreg.get_mni_fiducials

Problem
II'm trying to use the mne.coreg.get_mni_fiducials function, but it is not clear what the output is. Is the dictionary ordered as LPA, nasion, RPA? If not what is it?
In addition, I was wondering what the best way of attaching these coordinates to a raw is? Would one save the fiducials say as JSON and can then reload and attach during the set_montage step?
Suggested glossary term
It would be great to also see a glossary item explaining how fiducials are used in general (or links) and how it is used in mne-python.

fiducials are necessary to define the head coordinate system. What this
function does it transform to subject head the
fiducials hard coded for fsaverage
clear enough?

@agramfort @adam2392 I think there is already description in the glossary and proper documentation of the function. Is it still an open issue?

@adam2392 https://github.com/adam2392 are the links shared above clear
enough?
On Fri, Mar 19, 2021 at 10:04 AM Valerii @.***> wrote:

@agramfort https://github.com/agramfort @adam2392
https://github.com/adam2392 I think there is already description in the
glossary
https://github.com/vagechirkov/mne-python/blame/main/doc/glossary.rst#L123
and proper documentation of the function
https://github.com/vagechirkov/mne-python/blob/56e9fbe67a554272a71dfd2b9ae96eb7fd58d104/mne/coreg.py#L1224.
Is it still an open issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/7858#issuecomment-802669499,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABHKHEBSWJZX5XZZVCGDTLTEMHPXANCNFSM4NP7TUNA
.

Yes I think this is stale now. Can close


Add installation instructions for Windows

Adds installation instructions for Windows, and cleans up the errors presented by markdownlint.
Closes #169

I've fixed the conflicts that came about due to https://github.com/skevy/graphiql-app/pull/183
@skevy, FYI


AnimatedPlotterly doesn't plot multiple GroundTruths when they start at different timestamps

Title. It appears only paths starting on the first timestamp will be displayed, while others are simply ignored.

Can you provide an example? I did have this issue early on in making the plotter but thought it was ironed out, as shown in tutorials 9 and 10.


If media overlay is playing, audio doesn't stop when app is backgrounded or device is turned off

There is a bug where media overlay audio doesn't stop playing when the app is backgrounded or the device is turned off.
Repro steps:

Launch SDKLaunder-Android
Open a book with a media overlay
Play the media overlay
Background the app or touch the power button to turn off the display

Actual results: audio continues playing
Expected results: audio stops
One potential fix is to call ReadiumSDK.reader.toggleMediaOverlay() if the media overlay is playing in WebViewActivity.onPause(). Seems to work.

The proposed solution sounds good. Please note that the Readium native application "launchers" are minimalistic by design, so there may be a number of other user experience / device integration issues.

Finally created a pull request for this here: https://github.com/readium/SDKLauncher-Android/pull/73


Create aggregated sensor message

Why
We need to find a way to connect the new V4 sensor software to the database used for the V1-3 sensors so that they show up on the public map: https://dashboard.ribbitnetwork.org/
How
As discussed on Discord, there are two possible paths:

The first path is to give the sensors an influxdb token, and just send the data in the legacy format to influxdb directly from the sensor.
The second path is to have a process somewhere that picks up the data we already have in Google Pub/sub, massages it in the legacy format and sends it to influxdb.

The second option is advantageous as it eliminates the need for the sensors to hold the InfluxDB write key, which is a bit of a security risk.
In order to achieve this, the sensor should publish only one message to Golioth's LightDB service that contains the current state of all sensors. This eliminates the need for a complex backend service to aggregate the messages and hold the required state to write a datapoint into influxDB.
Testing
I have tested this on a test golioth fleet and confirmed all is well:

Hey @damz I know you might still be traveling, but will you be able to take a look at this soon?

It's been a few weeks, so I am going to merge this now to get the current v4 sensors hooked up to the map.


call of cardAction not working facebook

Bot Info

SDK Platform: Node.js
SDK Version: 3.14.1
Active Channels: FB
Deployment Environment: Azure Bot Service

Issue Description
call of cardAction not working on facebook and not display the message on messenger
-->the channel of azure has issues that「Action.Value is not a valid url: +8860001111」
I try to use openurl method still not working on facebook and not display the message on messenger
-->the channel of azure has issues that「Action.Value is not a valid url: tel://+8860001111」
Code Example
builder.CardAction.call(session, "+8860001111", "Call"),
builder.CardAction.openUrl(session, "tel://+8860001111", "Call")
Reproduction Steps

Expected Behavior

Actual Results

I have resolved the issue
I try to use the openUrl method with "tel:+886123456789 (I delete //)
but the call method still not support for facebook

Hi areckkimo! Were you trying to call out with the bot or recieve a call?

Closing for lack of response.


Deleting application blogs

Trello card
https://trello.com/c/wcvR07Eg/5018-delete-and-redirect-blog-posts-relating-to-applications
Context
We want to remove old blog posts that are serving duplicate content and better direct users to our applications guidance.
Changes proposed in this pull request
Guidance to review

Looks good @EllieNodder
Should you delete the images and associated alt text currently on /blog/7-things-to-include-in-your-teacher-training-application and /blog/application-tips-from-a-teacher-training-provider? Are they used elsewhere?


Optimization code

What is the purpose of the change
Optimization code
1.Use StringBuilder#append(Char) to improve performance
2.Replace runnable with lambda
3.The original implementation calls the get method twice.Replace it with computeIfAbsent.
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #10158 (c7ae4e5) into 3.0 (89863e0) will decrease coverage by 0.32%.
The diff coverage is 61.40%.

@@             Coverage Diff              @@
##                3.0   #10158      +/-   ##
============================================
- Coverage     65.68%   65.36%   -0.33%
+ Complexity      320      317       -3
============================================
  Files          1224     1224
  Lines         53413    53380      -33
  Branches       8077     8044      -33
============================================
- Hits          35086    34891     -195
- Misses        14508    14654     +146
- Partials       3819     3835      +16

Impacted Files
Coverage Δ

.../rpc/cluster/router/state/RouterGroupingState.java
0.00% <0.00%> (ø)

...ache/dubbo/qos/command/impl/GetRouterSnapshot.java
10.71% <0.00%> (ø)

...he/dubbo/qos/server/handler/QosProcessHandler.java
58.33% <0.00%> (ø)

.../dubbo/remoting/http/jetty/JettyLoggerAdapter.java
73.33% <0.00%> (ø)

.../org/apache/dubbo/test/check/DubboTestChecker.java
34.92% <0.00%> (ø)

...he/dubbo/registry/multicast/MulticastRegistry.java
64.67% <25.00%> (ø)

...va/org/apache/dubbo/common/utils/ExecutorUtil.java
63.79% <44.44%> (ø)

...etadata/report/support/AbstractMetadataReport.java
67.71% <55.55%> (ø)

...apache/dubbo/common/extension/ExtensionLoader.java
78.80% <85.71%> (-1.04%)
:arrow_down:

...e/dubbo/rpc/cluster/router/RouterSnapshotNode.java
81.08% <100.00%> (ø)

... and 84 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 89863e0...c7ae4e5. Read the comment docs.


Motion Sensor Support

I’m excited to install this plugin soon. Would be great if it also supports motion sensors. I have one, so I can test it out if that helps.

I have a pull request for motion sensor support, but it’s outdated. I’ll update it in the next couple of days and can give you instructions to test it out.

Ok, I've updated the pull request.

If you haven't already, install homebridge (see https://www.npmjs.com/package/homebridge#installation).
If you've already installed homebridge-scout, uninstall it: npm uninstall -g homebridge-scout
Check out the pull request: git clone git@github.com:jordanryanmoore/homebridge-scout.git
Change to the new local directory: cd homebridge-scout
Install the plug-in: npm install -g
If you haven't already, configure the plug-in via homebridge's config.json per https://www.npmjs.com/package/homebridge-scout#configuration.
Run homebridge.

Please try the motion sensor and (any other sensors you might have) and report back which ones work or don't. Thanks so much for your help!

I don't know if I did anything wrong but in step 3 I get the following error:
Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

I've updated step 3, so please try again. 🤦♂

The motion sensor is added to HomeKit. As soon as motion is detected, it crashes HomeBridge.
[12/31/2019, 10:26:36 AM] [ScoutAlarm] Device [0a70080064100450d80fa000a0000100] triggered event fired.
[12/31/2019, 10:26:36 AM] ReferenceError: device is not defined
at /home/pi/homebridge-scout/lib/device.js:168:85
at Array.forEach ()
at /home/pi/homebridge-scout/lib/device.js:146:28
at Array.forEach ()
at Device.getCharacteristicValues (/home/pi/homebridge-scout/lib/device.js:145:22)
at Device.onDeviceTrigger (/home/pi/homebridge-scout/lib/device.js:102:23)
at /home/pi/homebridge-scout/lib/scout-platform.js:89:24
at PrivateChannel.module.exports.Dispatcher.emit (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:7237:33)
at PrivateChannel.module.exports.Channel.handleEvent (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:8091:18)
at /home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:10148:29
at ConnectionManager.module.exports.Dispatcher.emit (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:7237:33)
at message (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:8631:23)
at Connection.module.exports.Dispatcher.emit (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:7237:33)
at message (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:7765:27)
at TransportConnection.module.exports.Dispatcher.emit (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:7237:33)
at TransportConnection.module.exports.TransportConnection.onMessage (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:7431:14)
at Client.socket.onmessage (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:7448:19)
at Client.dispatchEvent (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:4066:30)
at Client._receiveMessage (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:3257:10)
at Client. (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:3157:49)
at Client.emit (events.js:215:7)
at Client. (/home/pi/homebridge-scout/node_modules/pusher-js/dist/node/pusher.js:3768:14)
[12/31/2019, 10:26:36 AM] Got SIGTERM, shutting down Homebridge...

I just made a fix to the pull request. Starting after step 4, you should be able to run git pull and continue from there.

I can confirm motion sensor works perfect with this fix.

Awesome! I'll get this cleaned up and merged to master… hopefully in the next day or two.

Released in 0.3.0…


ci: Bump go version to 1.23

This PR bumps the go version for our tests workflow to 1.23.
Most notably, this changes our testing coverage results:
1.22 (https://github.com/gittuf/gittuf/actions/runs/10819478695/job/30017444897):
Run go test -covermode=atomic `go list ./... | grep -v -f .test_ignore.txt`
	github.com/gittuf/gittuf		coverage: 0.0% of statements
ok  	github.com/gittuf/gittuf/internal/attestations	0.267s	coverage: 27.4% of statements
ok  	github.com/gittuf/gittuf/internal/cmd/common	0.037s	coverage: 2.3% of statements
	github.com/gittuf/gittuf/internal/common/set		coverage: 0.0% of statements
	github.com/gittuf/gittuf/internal/common		coverage: 0.0% of statements
ok  	github.com/gittuf/gittuf/internal/display	0.060s	coverage: 15.5% of statements
ok  	github.com/gittuf/gittuf/internal/gitinterface	2.156s	coverage: 50.6% of statements
ok  	github.com/gittuf/gittuf/internal/policy	9.846s	coverage: 67.8% of statements
?   	github.com/gittuf/gittuf/internal/signerverifier/common	[no test files]
	github.com/gittuf/gittuf/internal/signerverifier		coverage: 0.0% of statements
	github.com/gittuf/gittuf/internal/third_party/go-securesystemslib/signerverifier		coverage: 0.0% of statements
ok  	github.com/gittuf/gittuf/internal/repository	15.255s	coverage: 55.7% of statements
ok  	github.com/gittuf/gittuf/internal/rsl	2.521s	coverage: 53.5% of statements
ok  	github.com/gittuf/gittuf/internal/signerverifier/dsse	0.038s	coverage: 9.3% of statements
ok  	github.com/gittuf/gittuf/internal/signerverifier/gpg	0.007s	coverage: 2.6% of statements
ok  	github.com/gittuf/gittuf/internal/signerverifier/ssh	1.323s	coverage: 19.0% of statements
ok  	github.com/gittuf/gittuf/internal/tuf	0.023s	coverage: 30.7% of statements

1.23 (https://github.com/gittuf/gittuf/actions/runs/10945101864/job/30388530890) :
Run go test -covermode=atomic `go list ./... | grep -v -f .test_ignore.txt`
	github.com/gittuf/gittuf		coverage: 0.0% of statements
	github.com/gittuf/gittuf/internal/common		coverage: 0.0% of statements
	github.com/gittuf/gittuf/internal/common/set		coverage: 0.0% of statements
ok  	github.com/gittuf/gittuf/internal/attestations	0.556s	coverage: 59.1% of statements
ok  	github.com/gittuf/gittuf/internal/cmd/common	0.059s	coverage: 14.6% of statements
ok  	github.com/gittuf/gittuf/internal/display	0.147s	coverage: 82.7% of statements
ok  	github.com/gittuf/gittuf/internal/gitinterface	2.636s	coverage: 65.0% of statements
	github.com/gittuf/gittuf/internal/signerverifier		coverage: 0.0% of statements
?   	github.com/gittuf/gittuf/internal/signerverifier/common	[no test files]
	github.com/gittuf/gittuf/internal/third_party/go-securesystemslib/signerverifier		coverage: 0.0% of statements
ok  	github.com/gittuf/gittuf/internal/policy	11.529s	coverage: 78.4% of statements
ok  	github.com/gittuf/gittuf/internal/repository	14.845s	coverage: 63.0% of statements
ok  	github.com/gittuf/gittuf/internal/rsl	2.671s	coverage: 88.2% of statements
ok  	github.com/gittuf/gittuf/internal/signerverifier/dsse	0.035s	coverage: 75.0% of statements
ok  	github.com/gittuf/gittuf/internal/signerverifier/gpg	0.008s	coverage: 85.7% of statements
ok  	github.com/gittuf/gittuf/internal/signerverifier/ssh	1.352s	coverage: 80.3% of statements
ok  	github.com/gittuf/gittuf/internal/tuf	0.024s	coverage: 92.7% of statements

Most notably, this changes our testing coverage results

What am I missing, why is this?

Can you bump other workflows too? I think we should test on the two latest Go versions, as we gear up for #545 and a gittuf API.


Add ability to add shapes like ellipse or rectangle

Would be cool if we could add shapes to our cards as well. In this way a little graph could be made for example (couple of rectangles horizontally stacked). Maybe render it to image on the host application?

Hey @niels9001, cool idea, could you possibly provide some mockups of the cards you would like to create using rectangles/ellipses, so that we can visualize the idea and have a more tangible example?
Thanks!

First thing that comes to mind is stuff for charts like below:

Ideally SVG would be supported: this would allow extreme flexibility in terms of shapes but also stuff like icons!

Thanks niels9001, we're tracking this on our portal and is currently a candidate for 1.3: https://portal.productboard.com/adaptivecards/1-adaptive-cards-portal/c/32-svg-support