ProfileReport: output not displaying correctly in JupyterLab2.0

Report not displaying in JupyterLab output cell
Here are the contents of the [IN] and [OUT] cells
Cell [IN]
import pandas_profiling as pf
pf.ProfileReport(ts)

Cell[Out]
Tab(children=(HTML(value='
\n
\n   …
Report generated with pandas-profiling.

Version information:

Windows: 10
Python version: 3.7.6
Environment:  Conda + JupyterLab
JupyterLab:  2.0.0
Pandas: 1.0.1
pandas_profiling: 2.4.0

You probably need to enable the widgets:
jupyter nbextension enable --py widgetsnbextension
https://ipywidgets.readthedocs.io/en/latest/user_install.html

@sbrugman  It says the conda install enables it! But I ran the command anyway.
>jupyter nbextension enable --py widgetsnbextension
Enabling notebook extension jupyter-js-widgets/extension...
      - Validating: ok

Did not make a difference to the output in JupyterLab!

FYI
I haven't got around to debugging this 'ipywidgets' issue yet!
My workaround is to write to file.
It actually kinda suits my workflow. And since I'm not blocked by this issue I'm not focusing on getting this fixed with any urgency.

You probably need to enable the widgets:
jupyter nbextension enable --py widgetsnbextension
https://ipywidgets.readthedocs.io/en/latest/user_install.html

Although I would agree this isn't the responsibility of PP, but would be great to have such tips in the Docs somewhere so we can always check if we have the necessary/minimum installation in place.

I currently have ipywidgets 7.5 in my conda env. I see there is a release candidiate out for 8.0 which is intended to be fully on par with the jlab2 changes.
I will revisit this issue when there is  a stable ipywidgets 8.0


Add DNS-SD API

Proposal: Add a DNS-SD discovery and announcement API

Summary
Windows 10 added support for DNS-SD a while ago, but its not really usable in most apps
as there is no API to discover and announce services that is usable for Win32
apps.

Rationale

Windows 10 already supports DNS-SD discovery
Having an API available for Win32 would make it easier to use the OS-included support for it instead of having to rely on 3rd party libraries

Scope

Capability
Priority

This proposal will allow developers to discover services using DNS-SD
Must

This proposal will allow developers to announce services using DNS-SD
Must

This proposal will allow developers to discovery/announce services using other means of autodiscovery
Could

Important Notes
Proposing an API seems a bit premature here, as there are decisions to make for this:
Should the API be specific for DNS-SD? Should it allow handling other discovery mechanisms (at the
cost of ease of use/simplicity of the API)? How to abstract the ability to set/read additional service metadata
(via TXT record) in the API?
Open Questions
The main issue would be how to handle lack of DNS-SD on older Windows 10 versions that are supported by Project Reunion.

Check out DnsServiceBrowse and DnsServiceRegister for use with at least Windows 10.  Those are supposed to work in Win32 apps, and if they don't we'd like to hear details (and sample code!) about it.
What else would you want support-wise?

Hi, thanks a lot, I had no idea this API exists! (For whatever reason its nearly impossible to find, I was only able to find the WinRT APIs for this, would help to maybe have a link to this API on the DNS-SD WinRT API docs)

Hi, @ePirat, FYI there are now links to the Win32 API topics from the Windows.Networking.ServiceDiscovery.Dnssd namespace topic, and from the type topics within there (for example, DnssdRegistrationResult).
Thanks!
Steve

Thanks! After looking at the API, I am wondering about the version in DNS_SERVICE_BROWSE_REQUEST, the documentation mentions DNS_QUERY_REQUEST_VERSION2 for it but it seems its not actually present in the SDK headers? Additionally the documentation makes no mention about the exact Windows 10 version since when the DNS_QUERY_REQUEST_VERSION2 would be available or not, so I am wondering whats the proper way to check if version 1 or 2 can be used?
I suspect I can always assume at least DNS_QUERY_REQUEST_VERSION1 is usable when the DnsServiceBrowse function exists?

Ok - an update that makes the VERSION2 tag available in the SDK is coming.  Internal folks, track http://task.ms/30284268 ... for now, you can manually add it to your pch.h or any other common spot, and remove it when a new SDK is released:
#define DNS_QUERY_REQUEST_VERSION2  0x2

You're right that there's no easy way to determine whether VERSION2 is available. A little source archaeology suggests that it's available on at least Windows 10's initial release.  Our API design guidelines have evolved since then so you can detect these "versioned structs" more easily.

After further playing around with the API and working on a very simple prototype, it turns out the API is incredibly useless unfortunately as it is so low level that using it is really really hard.
Additionally it seems to just not work reliably at all. With a started discovery session, whenever a new device is announced during the running session, the callback is never called for the added device. Interestingly, the callback is called once the device is removed again during an active session (callback called with a result with TTL of 0, which according to how I understood the related RFC means removal of the device). But the device was never added in the first place.
Given it is so hard to use the API and that it does not even work properly, it does not seem sufficient at all to get Apps to adopt native DNS-SD on Windows. Using something like Apples Bonjour Service + DNS-SD API is far far simpler and actually works, contrary to the DnsServiceBrowse API, from what I can tell.
Of course it could be I am just using the API wrong, given how low level it is. But then again, the documentation does an incredibly poor job detailing how it is supposed to be used.

Does anyone know a good Python library to implement this on windows (10)?

@ePirat I remember I found your comment above before I implemented my plugin. It was a bit disheartening, but I decided to see for myself.
Since I have used the DNS-SD APIs on macOS / iOS, Android and this one here, I think I can safely say I agree about the "low-level" aspect of the API. It feels incredibly old school and there is a lot of guesswork involved - especially for developers that don't deal with low-level Windows APIs every day.
On the other hand, I disagree about the reliability - once I figured out the details, it has been working well for me. The project has automatic tests for service registration, service unregistration, start discovery and stop discovery and they pass every time.
This class in my plugin handles all the Windows DNS-SD API calls and may be of help for anybody looking for a usage example in the future.

@sebastianhaberey Thanks for posting this, I will have another look at this. The specific case that did not work reliably for me was when discovering _services._dns-sd._udp.local to get all service types, but maybe I made a mistake somewhere in the logic of that code…


fix some typos.

fix some typos.

LGTM

LGTM
This only touches source code comments. Merging despite CI flakiness.


[tcgc] don't move server description onto endpoints

fixes #1212

All changed packages have been documented.

:white_check_mark: @azure-tools/typespec-autorest-canonical
:white_check_mark: @azure-tools/typespec-autorest
:white_check_mark: @azure-tools/typespec-client-generator-core

Show changes
@azure-tools/typespec-client-generator-core - breaking ✏

refactor tcgc build-in types, please refer pr's description for details and migration guides

@azure-tools/typespec-client-generator-core - feature ✏

Support model format of @multipartBody

@azure-tools/typespec-client-generator-core - fix ✏

have @clientName work for operation groups as well

@azure-tools/typespec-client-generator-core - fix ✏

Fix @clientName conflict validation that operation defined within an interface

@azure-tools/typespec-client-generator-core - feature ✏

add example types support

@azure-tools/typespec-autorest - internal ✏

create tcgc context change to async

@azure-tools/typespec-autorest-canonical - internal ✏

adopt tcgc breaking

@azure-tools/typespec-client-generator-core - feature ✏

add xml usage and change enumvalue arg representation in generic decorators

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


peakfitcube tool fails with index error

Generated Stokes QU cubes from ASKAP data using WSClean. Ran rmsynth_3d (which completes), then rmtools_peakfitcube, which fell over with an index error. Output log for both tasks and traceback are below.
(possum-mamba-env) canders@rsaa-074284 rmsythesis_working % rmsynth3d ./pbeam20.1-Q-mosaiccrrg-cube.fits ./pbeam20.1-U-mosaiccrrg-cube.fits ./freqs.txt -l 150 -f -v
Reading ./pbeam20.1-Q-mosaiccrrg-cube.fits ...
done.
Dimensions of the input cube are:  NAXIS1 = 6000  NAXIS2 = 6000  NAXIS3 = 16
Dimensions of the input array are:  (16, 6000, 6000)
Reading ./pbeam20.1-U-mosaiccrrg-cube.fits ...
done.
Dimensions of the input cube are:  NAXIS1 = 6000  NAXIS2 = 6000  NAXIS3 = 16
Dimensions of the input array are:  (16, 6000, 6000)
PhiArr = -149.55 to 149.55 by 11.50 (27 chans).
Weight type is 'uniform'.
Running RM-synthesis by channel.
[========================================] 100%
Calculating 1D RMSF and replicating along X & Y axes.

RM-synthesis completed in 145.62 seconds.
Saving the dirty FDF, RMSF and ancillary FITS files.
./FDF_dirty.fits
WARNING: VerifyWarning: Card is too long, comment will be truncated. [astropy.io.fits.card]
./RMSF.fits
./FDF_maxPI.fits
./FDF_peakRM.fits

(possum-mamba-env) canders@rsaa-074284 rmsythesis_working % rmtools_peakfitcube -v ./FDF_dirty.fits ./freqs.txt ./cena_inner_
0%|                                                                                | 27/36000000 [00:00<48:22, 12401.03it/s]
Traceback (most recent call last):
File "/Users/canders/miniconda3/envs/possum-mamba-env/bin/rmtools_peakfitcube", line 33, in
sys.exit(load_entry_point('RM-Tools', 'console_scripts', 'rmtools_peakfitcube')())
File "/Applications/RM-Tools-master/RMtools_3D/RMpeakfit_3D.py", line 380, in main
map_dict=pixelwise_peak_fitting(FDF, phiArr_radm2, fwhmRMSF,lambdaSqArr_m2,
File "/Applications/RM-Tools-master/RMtools_3D/RMpeakfit_3D.py", line 103, in pixelwise_peak_fitting
fwhmRMSF_pix=fwhmRMSF[xarr[i],yarr[i]]
IndexError: index 27 is out of bounds for axis 0 with size 27

I think I know what's going on. Try running rmsynth3d without the -f flag? That flag is partially deprecated and the resulting output format isn't properly supported by most of the other tools (including peakfitcube). I should probably either commit fully to deprecating that keyword, or supporting it fully in all tools, but so far my laziness has led to this problem.
Let me know if that doesn't help.
Cheers,
Cameron

Ah --- right. Thanks!
But without that flag, rmsynth_3d splits the FDF into multiple components. How do you feed the real, imag FDF components into peakfitcube (which asks for the positional argument "FDF_file.fits  An FDF cube from RMsynth3D or RMclean3D")? Presumably it needs these to e.g. calculate angle maps etc.

From: Cameron-Van-Eck @.>
Sent: Thursday, July 13, 2023 10:43 PM
To: CIRADA-Tools/RM-Tools @.>
Cc: Craig Anderson @.>; Author @.>
Subject: Re: [CIRADA-Tools/RM-Tools] peakfitcube tool fails with index error (Issue #88)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
I think I know what's going on. Try running rmsynth3d without the -f flag? That flag is partially deprecated and the resulting output format isn't properly supported by most of the other tools (including peakfitcube). I should probably either commit fully to deprecating that keyword, or supporting it fully in all tools, but so far my laziness has led to this problem.
Let me know if that doesn't help.
Cheers,
Cameron
—
Reply to this email directly, view it on GitHubhttps://github.com/CIRADA-Tools/RM-Tools/issues/88#issuecomment-1634178584, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKEOHW2OETD3N53GZ6CK3OTXP7UQFANCNFSM6AAAAAA2I3GYVI.
You are receiving this because you authored the thread.Message ID: @.***>

I've set up the tools so that you can feed in one the filename of the components, and it will automatically search for the other(s) with the same filenames. Maybe not the most robust (or obvious) solution, but it will work if you directly feed the rmsynth3d output into peakfitcube.
I'm going to make another pass through all the documentation sometime soon(TM), to hopefully clean up some of these ambiguities.

Perfect -- thanks!

From: Cameron-Van-Eck @.>
Sent: Thursday, July 13, 2023 10:51 PM
To: CIRADA-Tools/RM-Tools @.>
Cc: Craig Anderson @.>; Author @.>
Subject: Re: [CIRADA-Tools/RM-Tools] peakfitcube tool fails with index error (Issue #88)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
I've set up the tools so that you can feed in one the filename of the components, and it will automatically search for the other(s) with the same filenames. Maybe not the most robust (or obvious) solution, but it will work if you directly feed the rmsynth3d output into peakfitcube.
I'm going to make another pass through all the documentation sometime soon(TM), to hopefully clean up some of these ambiguities.
—
Reply to this email directly, view it on GitHubhttps://github.com/CIRADA-Tools/RM-Tools/issues/88#issuecomment-1634190424, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKEOHWYE33VEWU5DFCSXWBLXP7VOZANCNFSM6AAAAAA2I3GYVI.
You are receiving this because you authored the thread.Message ID: @.***>


Collection isn't working

Hello.
The collection isn't working at all. I've tried to add path to posts in the collection array, not working. But the jigsaw console do notice the _posts folder because it would throw me an error if it isn't there, but it just won't render any item in the folder. What am I did wrong?

The picture note:

The left editor is the 1-helloworld.md
The right editor is the config.php

I can't see any issues with your code (although the file 1-helloworld.md is set to extend the master template, so your post template won't be used), though it's possible there's some issue relating to running on Windows. Do you get any error message when trying to build the site?
If you upload a zip of your site, or link to a repo, I can test it out more fully.

There aren't any error, it just won't render the output.
For the repo: https://github.com/imam/imamtech-jigsaw

Your code builds fine for me when I run it on my mac, but something is causing the collection build to fail silently when I test it on Windows. I'm not certain yet of the cause.

Actually, scratch that, it's working fine for me on Windows. At least, on Windows running the Ubuntu bash, which it looks like you are running as well? (In my earlier test, I had mistakenly cleared out config.php so it had appeared to not work.) So I'm a bit stuck, since I can't reproduce the error when building from your repo.

@imam @damiani Just hit the same problem, everything works fine but it does not generate the output files, the data is parsed because I can loop through the collection as $posts.
Do you manage to solve this?

I wasn't able to reproduce the issue. Could you post a repo of your code, as well as some details about your setup? Running on Windows, using the Ubuntu bash shell, @imam's code built fine for me. I'd love to give yours a try to see if I can recreate the problem.

@damiani Thanks for such a quick answer, this is a link to my project repo, I'm using Windows 7, git bash, jigsaw installed globally and I think that's all.

Can confirm it doesn't work using Powershell with the provided jigsaw.bat
I had to run jigsaw through my VM running debian to build successfully

I believe I have fixed this bug with the pull request #126

I have confirmed that https://github.com/tightenco/jigsaw/pull/126 fixes the issue.

hello
In regards to this, I am using Windows 10, git bash and it does not generates the output files...

@adrianR84 Did you apply the fix from #126, and it still didn't work on Windows?

Yes, I did not work. I have tried both, $parts = preg_split('/[\\/\\\]+/', $this->relativePath()); and $parts = explode(DIRECTORY_SEPARATOR, $this->relativePath()); .... This code was added to .\vendor\tightenco\jigsaw\src\File\InputFile.php file and after I tried generating the build once again with ./vendor/bin/jigsaw build command.  Am I doing something wrong ?
Thank you.

@adrianR84 you also need the changes to the relativePath method of InputFile. Grab the latest changes from the below link and replace the entire InputFile with it. Also you need to make sure u are running the  ./vendor/bin/jigsaw build command rather than the global jigsaw build . Please let me know your results
https://raw.githubusercontent.com/garhbod/jigsaw/efd36fd0fa7144ed1cd19528db57de2b3f035dd8/src/File/InputFile.php

@cmosguy can you please test again with the latest changes mentioned in my previous post

Hello
Considering the file that you have given to me, it is working perfectly. Thank you for your help and quick reply.

@damiani @adrianR84 I have tested the updated and it is working just fine now.  I have tested on my windows 10 system.
Thanks for the updates!

Excellent, thanks for testing!

Addressed by https://github.com/tightenco/jigsaw/pull/126


Better heuristic for instance relevance

Currently we add an ALLINSTANCES import to every module we generate. We then gather imports of all relevant instances in this module. Roughly an instance is relevant if it is for a type that is mentioned by any slice transitively used by the top slice we want to compile (i.e. main).

Whenever we compile a different top level slice we have to recompile everything because the ALLINSTANCES module has changed. We would like to add all relevant instances to each slice directly.

We could be more clever in deciding which instances are relevant by using a different criterium: the instance class and the instance type have to be in mentioned or there is no way the instance is relevant.

Change number 2 probably leads to a massive reduction of the number of slices we have to compile. It is probably easier to implement if we have already implemented change number 1 because then we could do it in DeclarationSlices and not in SliceCompiler.

To every slice we add a list of relevant instances. As of 98297e77ecf7cdaa10166828e0044c007ff4062b relevant means that the slice transitively mentions both the class and the type of an instance. This indeed lead to a massive reduction of declarations to be compiled.


Add undertow and jetty profile

Since spring boot supports another embedded web servers, I added undertow and jetty profiles.

Thanks!


[hotfix] Do not use nulls in status for savepoint trigger and cluster info

Seems like there is a problem with the fabric8 patch logic and it doesn't allow us to set status fields to "null" and thus remove them.
This meant that savepoint trigger information would not be cleared correctly.
I will still investigate to see if there is a potential simpler workaround.
The problem seems to be caused by
https://github.com/fabric8io/kubernetes-client/blob/master/kubernetes-client-api/src/main/java/io/fabric8/kubernetes/client/utils/Serialization.java#L53
https://github.com/fabric8io/kubernetes-client/blob/master/kubernetes-client-api/src/main/java/io/fabric8/kubernetes/client/utils/serialization/UnmatchedFieldTypeModule.java

@wangyang0918  do you have any alternative idea maybe? :)

Does it caused by it used the PatchType#JSON_MERGE which will delete the key when it's value is null ?

@Aitozi it should delete the key when the value is null that is correct. But it doesn’t seem to work as it drops the null values before making the request to k8s.
you can verify by calling: Serialization.asJson(resource)

You mean it didn't keep the null value, so the patch for the field is skipped right ? @gyfora

Exactly, the path request sent to k8s should contain the null values so merge would remove those keys. Unfortunately fabric8 applies some custom plugin to the objectmapper that forces it to drop nulls (with a simple objectmapper they would be there).
So the request sent to k8s will not contian nulls so it is impossible to remove keys from the status (which is what we want when we set something to null)

Thanks, I think I get what you mean, I tried a way to custom the serializer may be can pass the null
    @Test
    public void test() {
        var application = TestUtils.buildApplicationCluster();

        Serialization.jsonMapper().registerModule(new TestModule());
        System.out.println(Serialization.asJson(application));
    }

    public class TestModule extends SimpleModule {

        public TestModule() {
            super("CustomTestModule", VersionUtil.versionFor(SavepointInfo.class));
            addSerializer(SavepointInfo.class, new SavepointInfoSerializer());
        }
    }

public class SavepointInfoSerializer extends JsonSerializer<SavepointInfo> {

    @Override
    public void serialize(
            SavepointInfo savepointInfo,
            JsonGenerator jsonGenerator,
            SerializerProvider serializerProvider)
            throws IOException {
        jsonGenerator.writeStartObject();
        if (savepointInfo.getTriggerId() == null) {
            jsonGenerator.writeStringField("triggerId", null);
        }
    }
}

generated:
"savepointInfo":{
       "triggerId":null
 }

@Aitozi that would mean we need to write custom serializers instead of simply avoiding nulls (and we also rely on the implementation of the fabric8 serialization logic which makes it more unsafe)
I think avoiding nulls is simpler and feels less hacky.

Yes, I think your solution looks good to me :) , custom serializer is a burden. +1 from my side

BTW, I have to admin that setting the value to null in json/yaml is not a best practice.

+1 It looked scary for the first read. Happy to see an elegant solution for this.


noaa-apt: 1.3.1 -> 1.4.0

Description of changes
Update to version 1.4.0

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg build noaa-apt noaa-apt.passthru.tests

Result of nixpkgs-review pr 230317 run on x86_64-darwin 1

1 package built:

noaa-apt


How should code respond on non-existing terms?

Sometimes, the user inputs a list of GO-terms that contain unknown GO-terms (e.g. terms that might have recently changed). How should our application react to this? Do we ignore those terms? Do we print an error / warning message?

covered in #9


Use of Key Grants

Great little utility! We are looking to utilize it but we need to constrain each service to specific encryption context. In your documentation you state the following:
"They are also useful for constraining access to a given credstash stored credential by using KMS Key Policy conditions and KMS Grant conditions. Doing so allows you to, for example, make sure that your database servers and web-servers can read the web-server DB user password but your database servers can not read your web-servers TLS/SSL certificate's private key."
Could you please elaborate or perhaps include an example of how one can use Key Grants to accomplish above?
Thanks,
Alex

Here's a short blurb I wrote up that may be helpful:
KMS encryption and decryption operations allow you to pass an “encryption context”. This is additional data (represented as key-value pairs) that, if supplied during encryption, must also be supplied during decryption.
KMS key grants permit access to KMS keys by AWS IAM users or roles. You can specify operations (e.g. Decrypt) and mandate an encryption context.
These two features combined let us restrict access to secrets by services.
When you write a secret, you could supply an environment string (e.g. production or development) and a service name. Then, you could use these values to create the following encryption context and include it in the encryption operation:
“environment:production,service:service-name”
These are included in the encryption context for the secret, and are thus tied to the secret.You must provide these exact same context variables (environment and service) when decrypting the secret.
You would then create a key grant for the production service user could specify that they can only use the KMS credstash key for decryption if they send the “environment:production,service:service-name” encryption context:
aws kms create-grant --key-id $STEALTH_KEY --grantee-principal "arn:aws:iam::0000000000:role/service-name" --constraints "EncryptionContextEquals={environment=production,service=service-name}" --operations Decrypt
This way, the service user/role could only get access to the secrets they are intended to get access to. They couldn’t decrypt the secrets for a different service, or a different environment, because they wouldn’t have access to decrypting with that encryption context.

Thanks for writing this up. I copied it to the credstash wiki (https://github.com/fugue/credstash/wiki/Using-Key-Grants-and-Encryption-Contexts-to-restrict-access) for future reference.
I'm going to close this issue for now, but feel free to reopen if ^ didnt answer your issue.


Fix dragging in color area and sync value

Description
Fixed bug when dragging handle in color area, where handle jumped to left position reported in https://github.com/umbraco/Umbraco.UI/issues/463
https://github.com/umbraco/Umbraco.UI/assets/2919859/7f2f925a-bf25-405a-8d46-0db142877639
With changes in this PR:
https://github.com/umbraco/Umbraco.UI/assets/2919859/4ffc61e0-af84-4542-95a1-9c10b178e8c8
Types of changes

[x] 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)
[ ] Chore (minor updates related to the tooling or maintenance of the repository, does not impact compiled assets)

Motivation and context

How to test?
Screenshots (if appropriate)
Checklist

[ ] If my change requires a change to the documentation, I have updated the documentation in this pull request.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

@iOvergaard could you have a look at this one? :)


Zombie reaping not occurring as expected

I'm working on an issue, where I've got a test harness for reliable reproduction (and it's fairly simple), and I think I've tracked the source of the issue down to containerd. In any version after 17.09.1, the test fails. I think this is because of a change introduced in containerd 1.0.0. The test is essentially this:

create a container called bad_parent (we're using ubuntu:xenial)
docker run --init -itd --name bad_parent bad_parent /bin/bash
run docker exec bad_parent date on a loop to show that the container is responsive
create a zombie pid

#!/usr/bin/env python
import os
import subprocess
import time

pid = os.fork()
if pid == 0:  # child
    # time.sleep(100)
    pid2 = os.fork()
    if pid2 != 0:  # parent
        while True:
            print('The zombie pid will be: {}'.format(pid2))
            time.sleep(30)
else:  # parent
    # os.waitpid(pid, 0)
    subprocess.check_call(('ps', 'xawuf'))
    time.sleep(5)

observe that the container stops responding

I've tried the latest containerd (compiled from git source). I've tried 18.09, but haven't been able to build rpms for 18.10 (build errors) from master. Any thoughts on where to go with this?
Here are the log files (in debug) for the issue: https://gist.github.com/cwpenhale/2eb23ffcb2abd2b4fccc509a50f2fd0a
Testing with the following host, virtualized in VirtualBox:
[root@localhost docker_exec_hang_issue]# uname -a
Linux localhost.localdomain 5.1.5-1.el7.elrepo.x86_64 #1 SMP Sat May 25 16:10:51 EDT 2019 x86_64 x86_64 x86_64 GNU/Linux
[root@localhost docker_exec_hang_issue]# cat /etc/redhat-release
CentOS Linux release 7.6.1810 (Core)

Was advised by @cpuguy83 that this is the wrong project by way of Slack. Closing issue.


Can't get an app running

I'm trying the following application and it stucks, however I can launch it normally from windows explorer.
images.zip

Hi there, your issue has been closed due to not correctly following the issue template. Please read the documentation about filing issues and open a new issue which does follow the issue template. Do not reply to this issue with more information or with a correctly formatted issue, as that will only lead to confusion.
I try to address all github issues and fix bugs quickly, but badly formatted issues or poorly described bug reports especially are a significant drain on project resources. Taking a few minutes to file a good issue makes a huge difference and will get it addressed faster - everybody wins!
In github when you first create a new issue the form is pre-populated with a template which you should read over and follow. As linked above there is documentation going into more detail about what makes a good issue.


Fix for issue #334 where, when using grouping, you cannot Select All …

Fix for issue #334 where, when using grouping, you cannot Select All / Unselect All for the filtered results.

Also fix issue #348. Set groupCachedItems appropriately so when using grouping and clearing a search filter, items are repopulated correctly.


Train Xtransformer using PubMedBert for 10 epochs

The idea here is that more steps might be needed to converge to Mesh. One hint towards that is the fact that the best threshold was 0.001 so some probabilities had not converged to high enough numbers.
This yielded 0.587 which is not much better.

I am closing because this is done


fix project filter condition, ad project null check for web_url

Latest master didn't work for me, the path filter always returned true.
Also there was an error when initially a project was null.

Thank you!


[#31] Fix for filtering of lowPassData

Fix for low pass filter to work on the emg buffer

Getting movement states. Added a control for aggregate movement based on emg data collection rate


Code review

STATUS: REQUIRED CHANGES :recycle:
Hi @ahadb802  :wave:
Your project page is looking fantastic so far :star_struck: !
However, there is still some room for improvement, that can help you have better code and make your project professional :sunglasses:

Highlights :sparkles:

Great work using html and css files

Congrats on your work so far! :clap:
Required Changes ♻

Add linters to file
add professional document to readme file

Optional suggestions :ballot_box_with_check:
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can improve the quality of your code :100:
Happy coding and keep it up! :raised_hands:
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @ahadb802  in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

_As described in the Code reviews limits policyyou have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form

STATUS: REQUIRED CHANGES :recycle:
Hi @ahadb802  :wave:
Your project page is looking fantastic so far :star_struck: !
However, there is still some room for improvement, that can help you have better code and make your project professional :sunglasses:

Highlights :sparkles:

Great work using html and css files

Congrats on your work so far! :clap:
Required Changes ♻

Add linters to file
add professional document to readme file

Optional suggestions :ballot_box_with_check:
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can improve the quality of your code :100:
Happy coding and keep it up! :raised_hands:
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @ahadb802  in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

_As described in the Code reviews limits policyyou have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form

STATUS: REQUIRED CHANGES :recycle:
Hi @ahadb802  :wave:
Your project page is looking fantastic so far :star_struck: !
However, there is still some room for improvement, that can help you have better code and make your project professional :sunglasses:

Highlights :sparkles:

Great work using html and css files

Congrats on your work so far! :clap:
Required Changes ♻

Add linters to file
add professional document to readme file

Optional suggestions :ballot_box_with_check:
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can improve the quality of your code :100:
Happy coding and keep it up! :raised_hands:
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @ahadb802  in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

_As described in the Code reviews limits policyyou have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form

STATUS: REQUIRED CHANGES :recycle:
Hi @ahadb802  :wave:
Your project page is looking fantastic so far :star_struck: !
However, there is still some room for improvement, that can help you have better code and make your project professional :sunglasses:

Highlights :sparkles:

Great work using html and css files

Congrats on your work so far! :clap:
Required Changes ♻

Add linters to file
add professional document to readme file

Optional suggestions :ballot_box_with_check:
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can improve the quality of your code :100:
Happy coding and keep it up! :raised_hands:
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @ahadb802  in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

_As described in the Code reviews limits policyyou have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form


DOC/TST: Use r2 as a metric

Fixes #196

Codecov Report

Merging #197 (1a00ab7) into master (56399a9) will decrease coverage by 0.47%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #197      +/-   ##
==========================================
- Coverage   99.55%   99.08%   -0.48%
==========================================
  Files           6        5       -1
  Lines         671      653      -18
==========================================
- Hits          668      647      -21
- Misses          3        6       +3

Impacted Files
Coverage Δ

scikeras/wrappers.py
99.42% <ø> (-0.01%)
:arrow_down:

scikeras/_saving_utils.py
96.20% <0.00%> (-3.80%)
:arrow_down:

scikeras/__init__.py

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 56399a9...1a00ab7. Read the comment docs.


List could return duplicated files

The cause is not addressed, could be listDirNext or the iterator.

https://github.com/beyondstorage/go-service-fs/blob/f15f7b2996fdd953eb0dc5d8dedbffc54e76b395/readdir_unix.go#L72-L90
The buf length could be calculated incorrectly. We need to use the returning n as the buf length.


There is some issue in the footer section.

Spelling of technology is wrong.

@ANSHIKA-26
Can you assign me this issue, I would like to work on it.

Kindly assign me this issue under the gssoc'24-ext, hacktoberfest'24 and hacktoberfest-accepted.

hey @singh-shreya27 , this issue is too small for a PR to be raised , anyone working on footer UI can fix this , kindly raise meaningful issues , otherwise hacktoberfest might mark the issue as spam


Scenes in Editor

Add functionality to handle scenes in Editor.

Very, very basic scene editor done.


please consider a distro independent binary distribution

A big fat tar to extract somewhere in /opt/ would be quite useful for people using distributions other than rpm/deb based ones.

Thanks for reaching out.
I will check for feasibility of this request.
I will update once I hear from the team. Thank you.

I am trying to package ROCm for Debian and this looks to me like the opposite of what I want.
Having a big tarball in /opt/rocm is exactly how AMD currently favors distributing its stack.
It is coupled with the fact that they quite setup their entire build system on that assumption.
Those two elements make for a very hard time packaging it properly into distributions while respecting the Filesystem Hierarchy Standard, which is common to an overwhelming majority of distros.

Installing the whole ROCm in 'untar' fashion is a necessary feature for HPCs which generally don't rely on distro package manager for user facing software packages. Right now I have to manually unpack all RPMs and then move files to a desired location other than /opt/rocm. I'd like to have an untar route like CUDAtoolkit installer offers.
ROCm needs to serve both HPC and Workstations. So need to keep consolidating "distro packages" and "untar" routes.

We will have both routes for a while and this is not a bad thing:

the CPackDeb cmake generator building the AMD nightly binary deb/rpm packages for people looking for cutting-edge.
distro official integration

I think that currently, the untar route is in a better shape than the 'distro package' one, mirroring the HPC focus, compared to general consumer market support from AMD.
I think that AMD should completely ditch the CPackDeb infrastructure which pollutes the whole codebase, and instead setup dedicated per-distro CI/CD, nightly build farms.

@ye-luo Hello, I am having discussions around the build system and some cmake recommendations for Debian, you could chime in to make sure your HPC perspective is taken into account as well!
https://lists.debian.org/debian-ai/2022/01/msg00103.html

@Maxzor CC me via email (Click my github profile and then homepage).

we do have support for hipGraphAddChildGraphNode,
https://github.com/ROCm/HIP/blob/develop/include/hip/hip_runtime_api.h
here is the header and also inside hip code the functionality is supported.

Hi @esistgut, apologies for the late response. There is currently work in progress for a self contained installer that does not require a package manager like rpm or deb, and will be coming soon.
@Maxzor With regards to having distro official integration, the distro packages are mainly handled by the the specific distributions themselves and not AMD so it is recommended that users follow the instructions in our docs (which pull the packages from https://repo.radeon.com) to install ROCm.
@Maxzor @ye-luo With regards to your point on using the standard paths for installing software on linux (/usr/bin, etc.), this is being addressed and will hopefully be fixed on rpm at least in an upcoming release.
I am keeping this issue open right now to track the progress for (1) and (3). Also, if the above comments don't address installation for you specific use case, feel free provide additional feedback so we can look into it.


[BUG] Do not delete CAPI cluster resources when finalizers are removed from provisioning cluster

Rancher Server Setup

Rancher version: v2.7.3

Describe the bug

When finalizers are removed from a provisioning cluster object, there is a race condition between adding the finalizers back, and the generating framework deleting the generated rkecluster, rkecontrolplane, cluster.cluster.x-k8s.io and machinedeployments.
To Reproduce

Reproduction is not reliable, but have the same reproduction steps as https://github.com/rancher/terraform-provider-rancher2/issues/993
Result
Cluster resources are undesriably removed.
Expected Result

Generating handler waits until finalizers are added.
Screenshots

Additional context

Test Environment:
Rancher version: v2.7-head 0bcf068
Rancher cluster type: HA
Docker version: 20.10
Downstream cluster type: rke2 aws node driver cluster using terraform

Testing:
Tested this issue with the following steps:

Provision RKE2 cluster with terraform
modify additional_manifest for RKE2 cluster
apply change

Result
As mentioned in ticket the race condition is hard to reproduce and I was not able to hit the issue but the expected workflow has not been affected and terraform was able to update the cluster successfully.


Wrap up missing in notebook 5

I will have a look at it tomorrow.

Notebook 5 (HMMs) has a wrap up...

In this notebook, we have learned how to use a hidden Markov state model (HMM) and how they differ from an MSM. In detail, we have used

pyemma.msm.timescales_hmsm() function to obtain an implied timescale object for HMMs,
pyemma.msm.estimate_hidden_markov_model() to estimate a regular HMM,
pyemma.msm.bayesian_hidden_markov_model() to estimate a Bayesian HMM, and
the metastable_assignments attribute of an HMM object to access the metastable membership of discrete states.

... but notebook 6 has not. Thanks for taking care!

Solved via #35.


Export config-loader (+other lib files?) to a new aio-lib-app module

motivation see https://github.com/adobe/generator-aio-app/issues/157#issuecomment-891682315

https://github.com/purplecabbage/aio-cli-lib-app-config
https://git.corp.adobe.com/OpenSourceAdvisoryBoard/opensource_submission_process/issues/1092

merged


docs: add customer.io brand header to README

👋  Hey there @customerio/mobile! This PR updates the README for our Android SDK to include our brand logo and mission similar to how we have on language specific integration libraries like https://github.com/customerio/customerio-ruby#readme
Let me know if there's any questions or suggestions to better fit your repository's style!

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

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

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


Fix git-resource Dockerfile build to run non-interactively

The oci-build step of the git-resource pipeline runs indefinitely because the following step prompts for interactive input:
#7 18.54 Setting up libpam-systemd:amd64 (249.11-0ubuntu3.9) ...
#7 18.67 debconf: unable to initialize frontend: Dialog
#7 18.67 debconf: (TERM is not set, so the dialog frontend is not usable.)
#7 18.67 debconf: falling back to frontend: Readline
#7 18.76 PAM configuration
#7 18.76 -----------------
#7 18.76
#7 18.76 One or more of the files /etc/pam.d/common-{auth,account,password,session} have
#7 18.76 been locally modified.  Please indicate whether these local changes should be
#7 18.76 overridden using the system-provided configuration.  If you decline this
#7 18.76 option, you will need to manage your system's authentication configuration by
#7 18.76 hand.
#7 18.76
#7 18.76 Override local changes to /etc/pam.d/common-*? [yes/no]

Example failure.
Note that this change started when we switched to the ubuntu-hardened base image; it's possible that usg fix changed one of the files that this error refers to ("One or more of the files /etc/pam.d/common-{auth,account,password,session}").
A few (but probably not all) possible fixes in order of preference:

Figure out how to make the existing approach run without interaction. Maybe we can pass an environment variable to the docker build? Maybe, but less likely, we can pass a flag to buildkit in the oci-image step?
Submit a contribution upstream to make this step always work without interaction.
Fork the repo like we forked slack-notification-resource and change the relevant instruction in the dockerfile to run non-interactively.

The cflinuxfs4 Dockerfile has some apt tricks that might be useful. Note the DEBIAN_FRONTEND=noninteractive.

Fixed by adding DEBIAN_FRONTEND=noninteractive as an ARG to the Dockerfiles which use the base_image


0.1.9: pytest warnings

I'm packaging your module as an rpm package so I'm using the typical PEP517 based build, install and test cycle used on building packages from non-root account.

python3 -sBm build -w --no-isolation
because I'm calling build with --no-isolation I'm using during all processes only locally installed modules
install .whl file in </install/prefix>
run pytest with $PYTHONPATH pointing to sitearch and sitelib inside </install/prefix>
build is performed in env which is cut off from access to the public network (pytest is executed with -m "not network")

Here is pytest output:

+ PYTHONPATH=/home/tkloczko/rpmbuild/BUILDROOT/python-wilderness-0.1.9-2.fc35.x86_64/usr/lib64/python3.8/site-packages:/home/tkloczko/rpmbuild/BUILDROOT/python-wilderness-0.1.9-2.fc35.x86_64/usr/lib/python3.8/site-packages
+ /usr/bin/pytest -ra -m 'not network'
==================================================================================== test session starts ====================================================================================
platform linux -- Python 3.8.16, pytest-7.2.1, pluggy-1.0.0
rootdir: /home/tkloczko/rpmbuild/BUILD/wilderness-0.1.9
collected 14 items

tests/test_application.py ........                                                                                [ 57%]
tests/test_formatter.py ....                                                                                [ 85%]
tests/test_group.py ..                                                                                [100%]

===================================================================================== warnings summary ======================================================================================
wilderness/tester.py:23
  /home/tkloczko/rpmbuild/BUILD/wilderness-0.1.9/wilderness/tester.py:23: PytestCollectionWarning: cannot collect test class 'Tester' because it has a __init__ constructor (from: tests/test_application.py)
    class Tester:

tests/test_application.py:12
  /home/tkloczko/rpmbuild/BUILD/wilderness-0.1.9/tests/test_application.py:12: PytestCollectionWarning: cannot collect test class 'TestApp' because it has a __init__ constructor (from: tests/test_application.py)
    class TestApp(Application):

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
============================================================================== 14 passed, 2 warnings in 0.06s ===============================================================================

Here is list of installed modules in build env

Package                       Version
----------------------------- -----------------
alabaster                     0.7.12
appdirs                       1.4.4
attrs                         22.2.0
Babel                         2.11.0
build                         0.9.0
charset-normalizer            3.0.1
contourpy                     1.0.6
cssselect                     1.1.0
cycler                        0.11.0
distro                        1.8.0
docutils                      0.19
exceptiongroup                1.0.0
extras                        1.0.0
fixtures                      4.0.0
fonttools                     4.38.0
gpg                           1.18.0-unknown
idna                          3.4
imagesize                     1.4.1
importlib-metadata            5.1.0
iniconfig                     1.1.1
Jinja2                        3.1.2
kiwisolver                    1.4.4
libcomps                      0.1.19
lxml                          4.9.1
MarkupSafe                    2.1.1
matplotlib                    3.6.3
numpy                         1.24.1
olefile                       0.46
packaging                     21.3
pbr                           5.9.0
pep517                        0.13.0
Pillow                        9.4.0
pip                           22.3.1
pluggy                        1.0.0
Pygments                      2.14.0
PyGObject                     3.42.2
pyparsing                     3.0.9
pytest                        7.2.1
python-dateutil               2.8.2
pytz                          2022.4
requests                      2.28.1
rpm                           4.17.0
scour                         0.38.2
setuptools                    65.6.3
six                           1.16.0
snowballstemmer               2.2.0
Sphinx                        5.3.0
sphinxcontrib-applehelp       1.0.2.dev20221204
sphinxcontrib-devhelp         1.0.2.dev20221204
sphinxcontrib-htmlhelp        2.0.0
sphinxcontrib-jsmath          1.0.1.dev20221204
sphinxcontrib-qthelp          1.0.3.dev20221204
sphinxcontrib-serializinghtml 1.1.5
testtools                     2.5.0
tomli                         2.0.1
urllib3                       1.26.12
wheel                         0.38.4
zipp                          3.11.0

Thanks for reporting this @kloczek! What exactly is the issue here? Pytest throws a warning but this seems to be an internal warning within Pytest: https://docs.pytest.org/en/stable/how-to/capture-warnings.html#internal-pytest-warnings

That warning is abput content of the tests/test_application.py


Huggers cannot hug without assistance

Huggers are unable to hug hosts unless a drone puts the hugger on.

Not a bug, you can't hug from 5 miles away; you need to be right next to them, the same way it is on cm13
The range is very short
Unless you mean it's something else
https://github.com/CM-14/CM-14/assets/122584947/6faccb3d-582a-4179-8ba3-8e91e0f893af
https://github.com/CM-14/CM-14/assets/122584947/b778dce1-53f5-42db-a86a-165f4b30e39f

Clarification, huggers cannot hug people in crit no matter the range, requiring a drone to put them on instead.

Tested locally and I think the confusion is that you can't drag yourself to hug, only clicking works.
This is fixed now in https://github.com/CM-14/CM-14/commit/8ee24441bf718718ada8a7ae156826f6970b9dbd


Lambda nodejs22.x fails

Error
Failed to launch the browser process!
/tmp/chromium: error while loading shared libraries: libnspr4.so: cannot open shared object file: No such file or directory
Environment

@sparticuz/chromium Version: "~130.0.0" = "130.0.0"
puppeteer-core Version: "~23.7.0" = "23.7.1"
Node.js Version: 22.11.0
Lambda / GCF Runtime: nodejs22.x

I will investigate more later, and will retry with the latest @sparticuz/chromium & puppeteer-core combo.

I will most likely need to add in support for node 22 in the 'is this a lambda environment' function

same bug with "playwright-core", I can reproduce on a Vercel Serverless environment

Do you have any documentation about Vercel serverless functions being moved to node 22? Most likely the issue is that it's not being correctly detected as a lambda environment. If you can find environment variables that would let me detect the environment we can get that added but I don't believe that is related to node 22, as the node 22 stable environments on lambda have only been out a few days

which environment variables do you need exactly ? there are a lot ...
I can quickly see :

AWS_EXECUTION_ENV: 'AWS_Lambda_nodejs20.x'  //     AWS_EXECUTION_ENV: 'AWS_Lambda_nodejs22.x',
LAMBDA_RUNTIME_DIR: '/var/runtime',   //     LAMBDA_RUNTIME_DIR: '/var/runtime',

I adjusted the functions https://github.com/Sparticuz/chromium/pull/319

I tried using it with the adjusted function but I am running into
Error: The input directory "/var/task/functions/bin" does not exist.
    at Function.executablePath (/node_modules/@sparticuz/chromium/build/index.js:235:19)

Maybe this has been adjusted in node22, I will further investigate.

I hardcoded the path for extraction which worked but now the next issue arises
ERROR	[ERROR]:  Error: spawn ETXTBSY
    at ChildProcess.spawn (node:internal/child_process:420:11)
    at Object.spawn (node:child_process:759:9)
    at new Process (/node_modules/@puppeteer/browsers/src/launch.ts:292:41)
    at launch (/node_modules/@puppeteer/browsers/src/launch.ts:173:10)
    at tre.launch (/node_modules/puppeteer-core/src/node/BrowserLauncher.ts:142:28)
    at generatePdf ...
  errno: -26,
  code: 'ETXTBSY',
  syscall: 'spawn'
}

Maybe node22 in lambda environments is not so straighforward or I am missing something (which I hope I do 😄). Ill let it rest for now.

I did some more investigation, seems like I just had a bug before. I can now successfullt read a website.
I had to hardcode the paths (in my case to the lambda layer) though. Could be that this changed in node22 to how it was before. Let me know if you want me to take a closer look or if you will take it from here 😊

in my case i got the following error
2024-11-25T09:19:19.984Z	7e4b98d1-f6fe-4b8f-a4cc-ea980b566d9c	ERROR	Invoke Error 	{
    "errorType": "LT",
    "errorMessage": "Target page, context or browser has been closed\nBrowser logs:\n\n<launching> /tmp/chromium --disable-field-trial-config --disable-background-networking --disable-background-timer-throttling --disable-backgrounding-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-background-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=ImprovedCookieControls,LazyFrameLoading,GlobalMediaControls,DestroyProfileOnBrowserClose,MediaRouter,DialMediaRouteProvider,AcceptCHFrame,AutoExpandDetailsElement,CertificateTransparencyComponentUpdater,AvoidUnnecessaryBeforeUnloadCheckSync,Translate,HttpsUpgrades,PaintHolding,PlzDedicatedWorker --allow-pre-commit-input --disable-hang-monitor --disable-ipc-flooding-protection --disable-popup-blocking --disable-prompt-on-repost --disable-renderer-backgrounding --force-color-profile=srgb --metrics-recording-only --no-first-run --enable-automation --password-store=basic --use-mock-keychain --no-service-autorun --export-tagged-pdf --disable-search-engine-choice-screen --unsafely-disable-devtools-self-xss-warnings --headless=old --hide-scrollbars --mute-audio --blink-settings=primaryHoverType=2,availableHoverTypes=2,primaryPointerType=4,availablePointerTypes=4 --no-sandbox --allow-pre-commit-input --disable-background-networking --disable-background-timer-throttling --disable-backgrounding-occluded-windows --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-background-pages --disable-component-update --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-hang-monitor --disable-ipc-flooding-protection --disable-popup-blocking --disable-prompt-on-repost --disable-renderer-backgrounding --disable-sync --enable-automation --enable-blink-features=IdleDetection --export-tagged-pdf --force-color-profile=srgb --metrics-recording-only --no-first-run --password-store=basic --use-mock-keychain --disable-domain-reliability --disable-print-preview --disable-speech-api --disk-cache-size=33554432 --mute-audio --no-default-browser-check --no-pings --single-process --font-render-hinting=none --disable-features=Translate,BackForwardCache,AcceptCHFrame,MediaRouter,OptimizationHints,AudioServiceOutOfProcess,IsolateOrigins,site-per-process --enable-features=NetworkServiceInProcess2,SharedArrayBuffer --hide-scrollbars --ignore-gpu-blocklist --in-process-gpu --window-size=1920,1080 --use-gl=angle --use-angle=swiftshader --allow-running-insecure-content --disable-setuid-sandbox --disable-site-isolation-trials --disable-web-security --no-sandbox --no-zygote --headless --user-data-dir=/tmp/playwright_chromiumdev_profile-iNShsX --remote-debugging-pipe --no-startup-window\n<launched> pid=14\n[pid=14][err] /tmp/chromium: error while loading shared libraries: libnspr4.so: cannot open shared object file: No such file or directory\n[pid=14] <process did exit: exitCode=127, signal=null>\n[pid=14] starting temporary directories cleanup\nCall log:\n  - <launching> /tmp/chromium --disable-field-trial-config --disable-background-networking --disable-background-timer-throttling --disable-backgrounding-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-background-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=ImprovedCookieControls,LazyFrameLoading,GlobalMediaControls,DestroyProfileOnBrowserClose,MediaRouter,DialMediaRouteProvider,AcceptCHFrame,AutoExpandDetailsElement,CertificateTransparencyComponentUpdater,AvoidUnnecessaryBeforeUnloadCheckSync,Translate,HttpsUpgrades,PaintHolding,PlzDedicatedWorker --allow-pre-commit-input --disable-hang-monitor --disable-ipc-flooding-protection --disable-popup-blocking --disable-prompt-on-repost --disable-renderer-backgrounding --force-color-profile=srgb --metrics-recording-only --no-first-run --enable-automation --password-store=basic --use-mock-keychain --no-service-autorun --export-tagged-pdf --disable-search-engine-choice-screen --unsafely-disable-devtools-self-xss-warnings --headless=old --hide-scrollbars --mute-audio --blink-settings=primaryHoverType=2,availableHoverTypes=2,primaryPointerType=4,availablePointerTypes=4 --no-sandbox --allow-pre-commit-input --disable-background-networking --disable-background-timer-throttling --disable-backgrounding-occluded-windows --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-background-pages --disable-component-update --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-hang-monitor --disable-ipc-flooding-protection --disable-popup-blocking --disable-prompt-on-repost --disable-renderer-backgrounding --disable-sync --enable-automation --enable-blink-features=IdleDetection --export-tagged-pdf --force-color-profile=srgb --metrics-recording-only --no-first-run --password-store=basic --use-mock-keychain --disable-domain-reliability --disable-print-preview --disable-speech-api --disk-cache-size=33554432 --mute-audio --no-default-browser-check --no-pings --single-process --font-render-hinting=none --disable-features=Translate,BackForwardCache,AcceptCHFrame,MediaRouter,OptimizationHints,AudioServiceOutOfProcess,IsolateOrigins,site-per-process --enable-features=NetworkServiceInProcess2,SharedArrayBuffer --hide-scrollbars --ignore-gpu-blocklist --in-process-gpu --window-size=1920,1080 --use-gl=angle --use-angle=swiftshader --allow-running-insecure-content --disable-setuid-sandbox --disable-site-isolation-trials --disable-web-security --no-sandbox --no-zygote --headless --user-data-dir=/tmp/playwright_chromiumdev_profile-iNShsX --remote-debugging-pipe --no-startup-window\n  - <launched> pid=14\n  - [pid=14][err] /tmp/chromium: error while loading shared libraries: libnspr4.so: cannot open shared object file: No such file or directory\n  - [pid=14] <process did exit: exitCode=127, signal=null>\n  - [pid=14] starting temporary directories cleanup\n",
    "name": "LT",
    "stack": [
        ""
    ]
}

closed in #320


[#30083] Add processing time to prism.

Add a ProcessingTime queue to Prism's element manager, that can be appropriately controlled by the TestStream notion of time.
Basic design is that the queue doesn't contain the elements, but instead manages which stage needs to be notified when to inject queue elements into processing, and in particular, syncing it with existing watermark processing, which decides when elements are ready to be processed anyway. This allows continuing to have each stage maintain all the state for the stage, instead of splitting it between the element manager and the stages.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Codecov Report
Attention: Patch coverage is 82.55814% with 15 lines in your changes are missing coverage. Please review.

Project coverage is 38.55%. Comparing base (8dcbf96) to head (275e307).
Report is 6 commits behind head on master.

:exclamation: Current head 275e307 differs from pull request most recent head f434d46. Consider uploading reports for the commit f434d46 to get more accurate results

Files
Patch %
Lines

...am/runners/prism/internal/engine/elementmanager.go
26.66%
10 Missing and 1 partial :warning:

sdks/go/pkg/beam/runners/prism/internal/stage.go
85.18%
2 Missing and 2 partials :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #30492       +/-   ##
===========================================
+ Coverage        0   38.55%   +38.55%
===========================================
  Files           0      699      +699
  Lines           0   102439   +102439
===========================================
+ Hits            0    39497    +39497
- Misses          0    61307    +61307
- Partials        0     1635     +1635

Flag
Coverage Δ

go
54.37% <82.55%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

R: @damondouglas

LGTM pending the latest check re-run passes; nothing really stood out for me accept for the couple of comments.

The only failure was the Dataflow run of the ProcessingTime_Bounded test, which I had not filtered out. Dataflow only properly handles processing time in Streaming pipelines,and that test pipeline executes as "batch".


Scroll view is not reset y axis offset when dragging to an area outside of the scroll view area

It works fine when the dragging finishes inside the area of image slideshow

Closed since it looks like I have incorrect layout.


Disable failing windows tests issue #231

Signed-off-by: Jorge Perez<EMAIL_ADDRESS>Temporary remedy for #231. This is a continuation of the issue described in #106 IIUC.
Summary
Disabled known to fail tests, in order to get CI green checks to clean the whole stack.
Checklist

[x] Signed all commits for DCO
[x] Added tests -> Actually disables tests on windows
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.
PR jobs should be green, except windows with protobuf warnings.

There's one more failing test: UNIT_ign_src_TEST

I don't see those in our buildfarm tracked jobs, I'll re-run the check to see if it's a new regression, a flaky one, or somehow only related to PR jobs.
Original build: https://build.osrfoundation.org/job/ign_fuel-tools-pr-win/239/
Manual re-run: https://build.osrfoundation.org/job/ign_fuel-tools-pr-win/242/

@ros-pull-request-builder retest this please

No test regressions on the PR jobs, merging, thanks for the review!


Days Off - Not working for Individuals added to their team as part of a group alias

Our developers are all added to Teams in our project as part of a group alias
My account was added individually, as were a few others.
Only those people who were added individually can use "Add Days Off" as individuals.
For all others when selecting "Add Days Off", the dialog prompts them to schedule for their entire group - as individuals the OK button remains grayed out.
ie. we can't use this feature for our entire Development org in Capacity Planning - and we need it badly.
Is there any plan to address this functionality in the future?
The current configuration doesn't adversely affect us in ANY other Azure Devops extensions or core functionality in our environment.

Please note, that we also tested to see if the user had to be a Team Administrator to add Days Off
I attempted (as an Administrator) to Add Days Off for individual members of our groups. The OK button was grayed out each time, but I could Add Days Off for people added individually who were not administrators.

We are experiencing the same issue.  This is a significant hindrance to being able to manage team members (Roles) in just one place (Active Directory groups).  As is, with each addition or departure from the team, we have to update the AD group and put them in the ADO dev team.

Hi, work Rest client doesn't support users added a group alias. So even if I show users the actual call to create the day off would fail.

We're closing old issues so we can get back to actively maintaining the extension. Feel free to reopen if this is still an issue for you.


SessionLockLostError error handling on streaming receiver with sessions enabled

This PR has changes to surface SessionLockLostError on using streaming receiver with sessions enabled.
Updated the tests as well.
Theory behind this PR:

About why addCredit() needs to be invoked in the finally, it looks like the underlying flow()/addCredit() call eventually executes this code https://github.com/amqp/rhea/blob/3ef54378e67bd63be6f057d9b74132ae18d0abac/lib/connection.js#L661 which is blocking until the link has been settled (which occurs when the delay within OnMessage ends) and hence the error event when emitted by service bus does not get received by the link.
Invoking addCredit before executing onMessage handler does not surface the error event, while invoking after it does surface the error. So it seems like the polling phase of eventloop is likely obstructed by the spin wait - this seems like an unreliable yet consistent issue and may be a bug to be fixed in rhea.
Since nothing stops message from getting completed other than receiver being closed, the message gets completed as receiver's still open.

Testing done: All tests pass in renewLockSessions.spec.ts
Refer to https://github.com/Azure/azure-sdk-for-js/issues/1051#issuecomment-460853986, https://github.com/Azure/azure-sdk-for-js/issues/1051#issuecomment-461264529 and https://github.com/Azure/azure-sdk-for-js/issues/1051#issuecomment-461287666 for more context.

Closing in favor of https://github.com/Azure/azure-sdk-for-js/pull/1128


V5: Add Key-Manipulation Functions

This adds the functionality to manipulate V5 header's keys (add/delete/change).

[x] Use the dexios key xx subcommand
[x] Change Key functionality
[ ] Key Add functionality
[ ] Key Remove functionality
[ ] Update change key functionality to use the features added in #140
[ ] Give header restore the functionality to detect if there's empty space at the start of the file or not - this was discussed in #140. If no empty space is present, prompt the user if they're sure, and concatenate the header+encrypted data, and write it to the output. I'm not too sure I like this solution though.
[ ] Add GA tests for all of the above

@pleshevskiy I don't think Update change key functionality to use the features added in https://github.com/brxken128/dexios/pull/140 is doable.
decrypt_master_key() doesn't return an index/the ciphertext/anything that matched, so I'm not able to replace it for key change, or use it for key delete. I could use it for key add though.
We'd need to return the index/matching ciphertext/anything to identify the keyslot we'd like to change or modify.

each version of the headers is different) it is possible that this utility is not suitable for you) Don't worry) we'll get to make the core look nice and convenient, too)

each version of the headers is different) it is possible that this utility is not suitable for you) Don't worry) we'll get to make the core look nice and convenient, too)

Damn I am excited for that!
On another note, would you like collaborator access to this repo, and your name under the authors? I believe you've done more than enough - all that I ask is that changes are still done via PR, and that someone else has checked over them first - I'll stick to these rules too. Let me know and I'll send you the invite if so :)

On another note, would you like collaborator access to this repo, and your name under the authors? I believe you've done more than enough - all that I ask is that changes are still done via PR, and that someone else has checked over them first - I'll stick to these rules too. Let me know and I'll send you the invite if so :)

I would be very happy to work together on such a great project) I always make all changes through MR. Even in my small projects)

I would be very happy to work together on such a great project) I always make all changes through MR. Even in my small projects)

I've invited you! I try but sometimes I forget, but I'll adhere to those rules in future.

Great! Now I can resolve conversation!

@pleshevskiy How do we feel about exposing the vec_to_arr() function in the core?
I feel like we could get a lot more use out of it, even if we just define it in key.rs or make a new module called utils.rs.

I originally added the change to allow environment variable specification of new keys, but I feel as though this will be awfully confusing to the user (with no way to differentiate between the two).
I reverted it, and then just decided to go with a second keyfile for these tests. It's more than sufficient, and easier to manage.

How do we feel about exposing the vec_to_arr() function in the core?

I wanted to find out why the protected structure didn't like it.

But we can remove Protected from this function and convert only vec to an array. Thus, we can use it as follows:
let vector = vec![];
let protected = Protected::new(vec_to_arr(vector));

I updated what I could to the newly-exposed vec_to_arr(), although the master key decryption with the index could really do with being standardised. I'll take a proper look into it tomorrow - I don't think it'll be too hard.

I plan to get #163 done, and maybe a decrypt_master_key variant that returns the index as well as the master key, so that this code can be a lot neater than it currently is.
The functionality is there and the tests pass, and my manual testing seems fine also.

So I found a bug within the domain/decrypt area, and fixed it there as well as in the header restore utility. With this, I gave it an error type and fixed an incorrect error that I found.
I think this PR is good to go, but I'd be happy to make any changes.

Looks good)


Fix path to requirements.txt

@akmalabbasov  I'm pretty sure this is not necessary; can you provide repro steps that produce the error this fixes?

@johndmulhausen you're right, sorry by bad :)


Add NormHead for Baichuan-2 for 7B/13B model

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.同润 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Maintainability: Clear Code and Clean Code

I'm not sure if this could be D&I or Risk or Value (or all of the above).
Maintainability:

If a stranger can modify your code and fix a bug in less than an hour, it is maintainable.

Test it. Ask someone who is outside of the project to take a look at your code and tell you how clear it is. Not how beautiful your classes and code constructs are—that’s what makes it clean. Instead, ask someone to fix a bug in just 30 minutes and see how they react. You will realize how clear the code is and whether it speaks the language a stranger can understand.

Full reference: https://www.yegor256.com/2018/09/12/clear-code.html

Posted on D&I workgroup: chaoss/wg-diversity-inclusion#74

Closing here as posted to D&I


Don't switch to Debug Console when using integrated terminal

This isn't really an issue, more of a feature request. Currently, if the console in a launch task is set to integratedTerminal, the window switches over to the build task information in the debug console (I have to manually switch windows to the terminal). This gets annoying after a while, and I found this issue when searching for why this behavior occurs. Turns out it can be controlled by an option "internalConsoleOptions": "neverOpen". While users could be expected to write this on their own, is it possible you could implement it in the code itself? Kind of like how vscode-node-debug did it?

I think the ideal solution would be for VS Code to make the terminal and debug console separate tabs so that both can be visible at once. The C# extension actually goes out of its way to show the debug console because it is the only way we have to show messages. We used to actually stick this in the templated launch.json, but we got feedback that we had two many options in the default template, so we removed it. So I feel like unfortunately there isn't a good answer here.


[Feature request] Built-in autoCruising options

I came across egjs-view360, and it looks very promising! In addition to #381 the only missing feature is built-in auto-rotate (autoCruise) options. You already have an example custom-control.html using custom functions, but it's not optimal.

Users should be able to drag/swipe also view is auto-rotating.
Auto-rotate should pause on user-interaction and be able to resume after user inactivity timeout.
It would be useful if this functionality was built-in, autoRotate autoRotateIdleDelay.

Thanks. I just wanted to post some feedback as I was researching several Panorama viewers, and egjs-view360 looks like one of the most promising allround solutions ⭐

Hello @mjau-mjau! Thanks for the suggestion!
I'm sure we have to add that feature. We're preparing for the next major version of this component so I think it will be added then.
I'm working on another project at the moment so it might take a while though.
Take a look at this another component of our team, it has an option named autoplay with exact same feature.
I think if the feature is made in View360, it might look just like this :)

https://naver.github.io/egjs-view3d/docs/options/control/autoplay

Take a look at this another component of our team, it has an option named autoplay with exact same feature. I think if the feature is made in View360, it might look just like this :)

https://naver.github.io/egjs-view3d/docs/options/control/autoplay

👍🏼 Indeed. Thanks.

autoplay is added in the new major version (v4), please try that!

https://naver.github.io/egjs-view360/docs/options/miscellaneous/autoplay


add codustry to  INTHEWILD.md

according to https://github.com/PyThaiNLP/pythainlp/discussions/514

Thank you

Thank you


Improve reporting tutorial based on 2019-10-04 feedback

During the 2–4 October 2019 workshop, we identified some issues with the westeros_reporting.ipynb tutorial:

~For the moment, this tutorial requires installing the master branch of both ixmp and the message_ix; the tutorial should provide or refer to instructions on how to do this.~ as of 2021-05-24, no longer necessary.
~messageix-dl does not download the tutorials from the master branch in some cases.~ fixed in #256.
The [reporting] extra dependencies (i.e. pyam-iamc) must also be installed in order to run the tutorial.
Some typos.

Please leave a comment with any additional issues.

After installing ixmp and subsequently message_ix and pyam (see https://github.com/IAMconsortium/pyam/issues/259) from source, I ran into the following issue with westeros_reporting.pynb:
AttributeError                            Traceback (most recent call last)
<ipython-input-12-20b6f6717002> in <module>
      1 # Create a reporter from the existing Scenario
----> 2 rep = Reporter.from_scenario(scen)
      3
      4 # Reporter uses the Python pint to handle units. '-', used in the Westeros
      5 # tutorial, is not a defined SI unit. We tell the Reporter to replace it with

~\AppData\Local\Continuum\anaconda3\lib\site-packages\message_ix\reporting\__init__.py in from_scenario(cls, scenario, **kwargs)
    144             key += ':pyam'
    145             try:
--> 146                 rep.as_pyam(qty, year_dim, key, collapse=collapse_cb)
    147             except KeyError as e:
    148                 log.info(f'{e.args[0]} not in scenario → not adding {key}')

~\AppData\Local\Continuum\anaconda3\lib\site-packages\message_ix\reporting\__init__.py in as_pyam(self, quantities, year_time_dim, key, drop, collapse)
    195         if isinstance(quantities, (str, Key)):
    196             quantities = [quantities]
--> 197         quantities = self.check_keys(*quantities)
    198
    199         keys = []

AttributeError: 'Reporter' object has no attribute 'check_keys'```

Near the end of the westeros_reporting.ipynb tutorial, the plotnine package is imported, however this does not automatically get installed with Anaconda, hence the user needs to first install it before being able to run the last plotting part.
So I would suggest to add a line about doing pip install plotnine

Postponed to v3.3 (or later).

Postponed to v3.3 (or later).

Postponed 'til after 3.3.

Postponed, as no time to complete this week.


Redirect events do not work

Describe the bug
Redirect events do not work when clicking on a link anywhere in the document.
Expected behavior
Option to disable this setting. Or remove this behavior altogether.

@kolia-zamnius
I'm afraid I can't understand your request.

Did you mean that you cannot click any links in your document?
How exactly does it have any relationship with the calendar?
Can you show any example code to reproduce the problem? Any sandboxes like JSFiddle, Codesandbox, Glitch would be fine.


Use tokio-util DelayQueue to flush streams on timeout

Creates a DelayHandler construct to manage timeouts used for flushing streams on a timely basis
Why?
At times a connected application may die off, after having pushed not enough data elements to fill a Stream's designated max_buffer_size, leading to it's data being lost. By using a timeout we intend to ensure the application's data will eventually be flushed by a global flush_timeout, the time period for which is set optionally with the flush_period field in the Config or will be defaulted to 10s.
Trials Performed

initial data is received on a stream: timeout is created and gets triggered

16:23:47 [DEBUG] (1) uplink::collector::tcpjson: Received line = "{ \"stream\": \"can\", \"sequence\": 1, \"timestamp\": 12345, \"data\": 100 }"
16:23:47 [WARN ] (1) uplink::base: Sequence number anomaly! [1, 1]
16:23:48 [INFO ] (19) uplink::base::mqtt: Incoming = PubAck(PubAck { pkid: 44 })
16:23:56 [DEBUG] (19) uplink::base::mqtt: Outgoing = Publish(45)
16:23:56 [INFO ] (19) uplink::base::mqtt: Incoming = PubAck(PubAck { pkid: 45 })
16:23:57 [INFO ] (1) uplink::base: Flushing stream name: can, topic: /tenants/xxx/devices/xxx/events/can/jsonarray

multiple data packets are received before timeout, without flushing the stream: timeout is created once, but reset multiple times, triggering exactly after flush_period duration from last data being received.

16:24:29 [DEBUG] (1) uplink::collector::tcpjson: Received line = "{ \"stream\": \"can\", \"sequence\": 1, \"timestamp\": 12345, \"data\": 100 }"
16:24:29 [WARN ] (1) uplink::base: Sequence number anomaly! [1, 1]
16:24:29 [DEBUG] (1) uplink::collector::tcpjson: Received line = "{ \"stream\": \"can\", \"sequence\": 1, \"timestamp\": 12345, \"data\": 100 }"
16:24:29 [WARN ] (1) uplink::base: Sequence number anomaly! [1, 1]
16:24:36 [DEBUG] (6) uplink::base::mqtt: Outgoing = Publish(30)
16:24:38 [INFO ] (6) uplink::base::mqtt: Incoming = PubAck(PubAck { pkid: 30 })
16:24:39 [INFO ] (1) uplink::base: Flushing stream name: can, topic: /tenants/xxx/devices/xxx/events/can/jsonarray

stream gets flushed before timeout: stream's timeout doesn't trigger as it is removed.

16:25:24 [DEBUG] (1) uplink::collector::tcpjson: Received line = "{ \"stream\": \"can\", \"sequence\": 1, \"timestamp\": 12345, \"data\": 100 }"
16:25:23 [INFO ] (22) uplink::base::mqtt: Incoming = PubAck(PubAck { pkid: 5 })
16:25:25 [DEBUG] (22) uplink::base::mqtt: Outgoing = Publish(6)
16:25:26 [DEBUG] (1) uplink::collector::tcpjson: Received line = "{ \"stream\": \"can\", \"sequence\": 1, \"timestamp\": 12345, \"data\": 100 }"
16:25:26 [WARN ] (1) uplink::base: Sequence number anomaly! [1, 1]
16:25:26 [INFO ] (1) uplink::base: Flushing stream name: can, topic: /tenants/xxx/devices/xxx/events/can/jsonarray
16:25:29 [INFO ] (22) uplink::base::mqtt: Incoming = PubAck(PubAck { pkid: 6 })
16:25:39 [DEBUG] (22) uplink::base::mqtt: Outgoing = Publish(7)
16:25:40 [INFO ] (22) uplink::base::mqtt: Incoming = PubAck(PubAck { pkid: 7 })
16:25:49 [DEBUG] (22) uplink::base::mqtt: Outgoing = Publish(8)
16:25:50 [INFO ] (22) uplink::base::mqtt: Incoming = PubAck(PubAck { pkid: 8 })

data is received on stream with custom timeout set in config: the stream's config is as follows

[streams.can]
topic = "/tenants/{tenant_id}/devices/{device_id}/events/can/jsonarray"
buf_size = 2
flush_period = 5

It is observed that the stream is flushed as expected, 5secs after data is received.
17:18:13 [DEBUG] (1) uplink::collector::tcpjson: Received line = "{ \"stream\": \"can\", \"sequence\": 1, \"timestamp\": 12345, \"data\": 100 }"
17:18:17 [DEBUG] (8) uplink::base::mqtt: Outgoing = Publish(27)
17:18:17 [DEBUG] (8) uplink::base::mqtt: Outgoing = PingReq
17:18:18 [INFO ] (8) uplink::base::mqtt: Incoming = PubAck(PubAck { pkid: 27 })
17:18:18 [INFO ] (1) uplink::base: Flushing stream name: can, topic: /tenants/xxx/devices/xxx/events/can/jsonarray

@de-sh Can you list out all the tests that we've done here?


[KYUUBI #5611][AUTHZ] Authz support path privilege for SaveIntoDataSourceCommand

Why are the changes needed?
To close #5611
Authz support path privilege for SaveIntoDataSourceCommand
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Was this patch authored or co-authored using generative AI tooling?
No

ping @yaooqinn

Thank you @AngersZhuuuu merged to master


Special handling of error when executables are not found

The error case when executables can't be found will be commonly encountered by users.
We should perhaps consider adding a special error message in this situation.

The error message which is shown in this case is:
node: Service.start: child_process failed: Error: spawn cardano-node ENOENT

I think that explains things well enough.


What token is required to use the RTM client?

From various tutorials, it seems that the current bot setup supported by slack is to create an app and a bot, each of which have a token. The SlackRtmClient, and I've tried both my bot and app token and neither seem to work.
I end up with
Exception in thread "main" slack.api.ApiError: not_allowed_token_type

I think there are several types allowed.
At one place I am using a Custom Integrations | Bot | API Token. Its value starts with xoxb-. But from what I remember, this has been deprecated in Slack.

The docs for https://api.slack.com/methods/rtm.connect say name two options:

User tokens
Legacy bot tokens
bot

So the Legacy bot token is probably my usage. Then the other answer is "User token" which I have little experience with.

Would you be open to adding a socket mode client to this library?
https://api.slack.com/start/planning/choosing
I could look into adding this. Without going into any detail, the docs say Socket Mode is preferred to RTM

That'd be a great contribution. Feel free to proceed. Please consult if you want to add new non trivial dependences.

Thanks. My understanding is the RTM client starts a websocket connection with slack. If so, the SocketMode client would do something very similar.


Is pacal calling convention supported?

This is related to #485. The Win16 API function calls should use __pascal calling convention for the argument order to be right (left to right). I didn't find anything useful in the code regarding this issue. Is it possible to define a new convention with arguments being pushed left to right? Maybe there's a parameter for a cspec definition that reverses the argument order? So far I couldn't find an easy way to do that.

Rightclick -> Edit Function Signature allows you to customize the calling conventions.
You can specify where parameters go, including the stack.
This can be done left to right and right to left, just set the correct stack locations.

Yeah, I know about this but it's not very convenient. I'd prefer the order to be automatic if possible.

This is a flaw in the Java frontend only, not the back end as pointed out.  The decompiler just expects the stack locations to be correct.  __pascal is __stdcall with a reverse order.  It should be added but also the library definitions need to have knowledge about it for it to be practically useful.

The backend does not really care much about the order.  Its only a display issue.  So the XML would be identical for stdcall.  Underneath the hood the stack offsets are fixed and merely needs to be displayed backwards.  The decompiler does not need to know about it whatsoever.
So exactly how to fix this is less clear - some extra metadata like displayorder="ltr/rtl" could be added again only for the frontend to display it differently.  But perhaps extra metadata in files generally intended only for sleigh/decompiler which will not be used by them is best integrated elsewhere.

This is still outstanding and shouldn't be closed.

So I'm hitting this issue again today. Is there anything on latest Ghidra that can help to make this work? I.e. is the backend now capable of dealing with this and it's just a matter of changing the XML to add the right calling convention?


NavigationViewItem content clipping regression in 2.4.x

Describe the bug
NavigationViewItem clips its internal contents in 2.4.x much more than 2.3.x. This makes it very difficult to render custom elements in NavigationViewItem alongside standard NavigationViewItem icons. The problem is visible when the NavigationViewItem is in collapsed mode.
The following are screenshots of the NavigationView's PaneFooter:
2.3.2:

2.4.2:

Steps to reproduce the bug

Steps to reproduce the behavior: use this XAML as a NavigationView footer, and compare results between 2.3.x and 2.4.x:
            <ui:NavigationView.PaneFooter>
                <ui:NavigationViewItem>
                    <StackPanel Orientation="Horizontal" Margin="-6,0,0,0">
                        <ui:PersonPicture PreferSmallImage="True" Height="24">
                            <ui:PersonPicture.ProfilePicture>
                                <BitmapImage UriSource="some source" />
                            </ui:PersonPicture.ProfilePicture>
                        </ui:PersonPicture>
                        <TextBlock Margin="6,0,0,0" VerticalAlignment="Center" Text="some text" />
                    </StackPanel>
                </ui:NavigationViewItem>
            </ui:NavigationView.PaneFooter>

Expected behavior
Clipping behavior in 2.3.x seems much more sane.
Screenshots
See above
Version Info
Comparing 2.3.200213001 and 2.4.2.

Windows 10 version
Saw the problem?

Insider Build (19041)
Yes

Device form factor
Saw the problem?

Desktop
Yes

@mfeingol is this the same as seen in this issue ? Does it repro with items only in the pane footer and no other items ?

@ranjeshj: not sure if it's the same issue.
It repros with with items under NavigationView.MenuItems as well as items under NavigationView.PaneFooter.

The root issue is that the we now enforce a margin for the Content presenter, which results in 16px to the right of the content to allow the Expand chevron for hierarchical NavigationViewItems to always be completely visible and not block the NavigationViewItems content.
The question would be: How do we enforce that the expand/collapse chevron has enough space while also not clipping the content of the NavViewItems in Compact.

As the expand/collapse chevron is not shown in compact mode with pane closed, we can utilize the work done in PR https://github.com/microsoft/microsoft-ui-xaml/pull/2342 to restore the ContentPresenter margin to its value seen in WinUI 2.3 (and below)


[blocked on Django 1.9 upgrade] App url update

quite unsure about some patterns, @timabbott looking for your feedback for further changes!

Automated message from Dropbox CLA bot
@auvipy, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

Automated message from Dropbox CLA bot
@auvipy, thanks for signing the CLA!

It looks like this doesn't work -- if you just visit localhost:9991 you'll get a traceback with this branch.

@auvipy is it possible this syntax isn't supported until Django 1.9?  We are on Django 1.8.

yeah should work until 1.9

OK, I guess we can pick this up once Zulip is on Django 1.9.  I've updated the PR title to make that clear.

We did this as part of the Django 1.10 migration.


[tiered storage] Provide LedgerOffloaderFactory for creating offloaders

Motivation
In order to use NAR for packaging offloaders, we need a factory interface for creating offloaders.
Changes

Provide a ledger offloader factory interface for creating offloaders.
Move implemention specific settings to implementation package to be respecting to offloader factory interface

This change is based on #2391 . commit 3322b06 is the change to be reviewed in this PR.

#2391 is merged. so the whole PR is ready for review


Improvement when Intelellij starts the service glassfish.jersey does not work

Alluxio Version:
What version of Alluxio are you using?
2.10.x
Describe the bug
I started the AlluxioMaster and AlluxioWorker services in intellij and wanted to be able to access ports 19999 and 30000 normally in my browser. However, an accident occurred here, prompting some of the following error messages:
三月 28, 2023 8:48:51 下午 org.glassfish.jersey.message.internal.WriterInterceptorExecutor$TerminalWriterInterceptor aroundWriteTo 严重: MessageBodyWriter not found for media type=application/json, type=class alluxio.wire.WorkerWebUIInit, genericType=class alluxio.wire.WorkerWebUIInit.
Browser effects:

This seems to be related to 'glassfish.jersey'.
To Reproduce
Configure a reasonable environment in Intellij, then start the AlluxioMaster and AlluxioWorker services.
Expected behavior
We should see the correct data in the browser.
Urgency
Describe the impact and urgency of the bug.
Are you planning to fix it
Please indicate if you are already working on a PR.
Additional context
Add any other context about the problem here.

Here's a way to fix the problem above.
'
Gson gsonCover = new GsonBuilder().create();
'
return Response.status(Response.Status.ACCEPTED).entity(gsonCover.toJson(webUIInit)).build();
I'm not sure if it's all json format returns, compatibility is involved here.
@jiacheliu3  @LuQQiu  @maobaolong , do you have some different opinions?

Here are some examples of tests.
http link:
http://localhost:30000/api/v1/worker/webui_init
Results returned:
{"mDebug":false,"mWebFileInfoEnabled":true,"mSecurityAuthorizationPermissionEnabled":true,"mMasterHostname":"<IP_ADDRESS>","mMasterPort":19999,"mRefreshInterval":15000}

Thanks for raising the request.

Thanks @ChunxuTang .


"registerProtocolHandler"    MIME type and protocol handler registration  This API allows ...

"registerProtocolHandler"
http://dev.w3.org/html5/spec/timers.html#custom-handlers
MIME type and protocol handler registration
This API allows sites to register themselves as handlers for certain schemes.

This issue was imported from Google Moderator
Moderator votes: +6
Issue added by jhuckaby on 2012-01-16
To vote this issue up or down, simply include +1 or -1 in your comment.

Already added at http://caniuse.com/#feat=registerprotocolhandler


[Bug]: How can I run server code on adminJS?

Contact Details
No response
What happened?
I am trying to upload and delete some data to Cloudinary using its SDK. Since that requires to be run on the server side, how can I do it with adminJS since it cannot function on the client side of the browser?
I tried using it in one of the custom components but couldn't as it needed to be run on the server side.
Bug prevalence
2
AdminJS dependencies version
"adminjs": "^6.6.5",
What browsers do you see the problem on?
No response
Relevant log output
No response
Relevant code that's giving you issues
No response

Got it, thank you.


fix:fix 'kpm add' would redownload the dependences each time.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

fix #49

2. What is the scope of this PR (e.g. component or file name):

kpm add
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[x] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

Adjusted the path of calculating dependency sum so that the same dependency will not be downloaded multiple times.
4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 15 (46.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.09%) to 40.867%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/package/package.go
7
9
77.78%

pkg/cmd/cmd_add.go
0
6
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.09%

Covered Lines:
575

Relevant Lines:
1407

💛  - Coveralls


WebSocket connection to 'ws://localhost:8734/' failed:

When trying to run the web workspace, I got this error message. I already enabled in and out tcp connection from that port, and updated deprecated libs. However, WebSocket still cannot connect.

thanks @Amorim33
can you share the fully stacktrace?


Signup UI

Description
There is no Figma File for signup page.
Domain
UI/UX
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of Smart Home App.

@Lakhankumawat Can you please assign this task to me ? I will create UI according to Login Screen or update Figma file.

you don't have access to update figma file , instead you create your design in a new figma file show me design and i will accept it if it LGTM

any updates ?

I would like to work on this issue

Hey I am a flutter developer, I would like to contribute on this issue .