Does not work with the new Visual Studio editor

Latest updates to Visual Studio (8.1.5 (build 9) for instance) automatically chooses to use the new editor and it does not work with StyleCop. The menu option in the right-click menu no longer appears and running it from Tools -> Run StyleCop shows "Violations: 0" where there should be some.
Reverting to the old editor allows it to work again.

I'll have a look at it but I can't garuantee to get all functionality back the old editor has like showing the warnings properly at the line they occur. Running it from Tools or right click menu of the current files tabulator works fine for me so far, you use the latest version of the addin (<IP_ADDRESS>), if not upgrade please.
Maybe its best to switch to the CodeAnalyzers which make use of roslyn as VS for Mac goes more and more into that direction and I won't make that switch for this StyleCop addin. If you don't know the roslyn based StyleCop Analyzers yet, have a look at it here.
You'll have to create the configuration manually as far as I know as I think it isn't compatible with the old StyleCop configuration file but I guess sooner or later (sooner) I'll give up on this project as it would need a rewrite that I can't do anymore, also the currently used StyleCop version isn't the latest anymore I guess but as that project is also almost dead in favor of the StyleCop.Analyzers project and the roslyn based approach there isn't such a big gap between the version in use here and the latest released StyleCop version.

FYI if you have a standard .Net project and add StyleCop.Analyzers NuGet package you must add the following ItemGroup to your project file manually (at least when using VS for Mac) as for some reason it won't do that automatically:
  <ItemGroup>     <Analyzer Include="packages\StyleCop.Analyzers.1.1.118\analyzers\dotnet\cs\StyleCop.Analyzers.CodeFixes.dll" />     <Analyzer Include="packages\StyleCop.Analyzers.1.1.118\analyzers\dotnet\cs\StyleCop.Analyzers.dll" />   </ItemGroup>
Depending on where the project file is on file system you must also correct the path to the dlls..
After that you can build the project and the analyzer will do its job. The good thing using this analyzer is that it'll also provide code fixes.

Hello again, I tested a bit more and tried to get the StyleCop warnings/errors somehow shown in the new C# editor but wasn't succesful. The old editor does the display of the warnings/errors itself without that I've to do anything else than adding the warnings/errors to error pad, but the new doesn't seem to do that and I don't know how to influence that.
I also get the warnings/errors of StyleCop in error pad with the new editor, only displaying it in the editor doesn't work in the new editor anymore, but that is in my opinion a problem of the new editor. As said above I didn't have to do anything but add the warnings/errors to error pad to get the old editor show these.
I'll close this bug as at least the missing context menu entries are added back with the latest version <IP_ADDRESS> which will also add support for VS for Mac 8.2 (or better it only works with that version).
If you need more please consider to switch to the StyleCop.Analyzers NuGet package (as mentioned above) which is still under development...

I think this will be fine for the time being. Thank you for your work on this!

FYI I tested StyleCop.Analyzers NuGet package again and I think I was wrong, while you have a file open and are editing, it'll show you problems and suggest quick fixes in editor window but not in error pad.
To see problems in error pad you must build the project.
Unfortunately you might still have to do the manual editing of the project file mentioned above.
I tested with a new .Net 4.7 project and there it works without manual editing but adding the NuGet package to the MonoDevelop.StyleCop project requires me to add the above mentioned lines manually for whatever reason but then the analyzer works.


The search does not always take you to the expected place

Problem Description:
When searching for things on the glamorous website, the links sometimes take you to https://glamorous.rocks/api#__next rather than the heading on the page

Suggested Solution:
It looks like these titles on the pages don't actually have ids so whatever is powering the search wouldn't be able to link to the correct place.

Investigate adding ids to all titles in the documentation to allow linking
investigate what needs to happen to let the search know about those ids

Yes, I'd love to fix that also! Thanks for bringing it up.


Add Godot version compatibility information to documentation

Hi!  I recently learned about this tool through Gamefromscratch and it looks really cool.
However, one of the top questions on my mind is, "What version of Godot is this compatible with?"  Since 4.0 was released relatively recently and introduced a lot of breaking changes, and since the 3.x line is still considered the "LTS" version, it seems valuable to call this out prominently in the documentation somewhere.
(I would just submit a PR to add the docs myself but I don't actually know what versions it supports!)

I've updated the docs.  I have no plans to provide a Godot 3 version since there are a lot of breaking changes between the two.  The code would need a lot of rewriting to work on the older version.

Awesome, thanks!


Create pytest to verify that KSA and GSA are set correctly for workload identity manager

In 0.7 we are enabling identity workload management on GKE.
As part of this when we create profiles we want to correctly bind GCP service accounts to KSAs.
kubeflow/kubeflow#3917 will update the profile controller to support this.
We also have code in kfctl to do this for the default namespace.
We should create a pytest that checks that the KSA and GSA for the newly created namespaces are correctly configured for workload identity.
We should then include this in a suitable E2E test for workload identity

@kunmingg Did you add a test for this already?

Since GCP has migrated out of kfctl and use gcp-blueprint instead, close the issue
/close


MM-32351: Add Compliance Subsections

Summary
Adding the sub sections for Compliance section in system console
Ticket Link
https://mattermost.atlassian.net/browse/MM-32351
RelatedPRs
https://github.com/mattermost/mattermost-redux/pull/1377
https://github.com/mattermost/mattermost-server/pull/17023
Screenshots

/update-branch

/update-branch

/update-branch

Tested as a System Admin and System Manager.
System manager was No Access, Read Only, Full Access.
When I had each of those permissions, I got expected behaviour (Not being able to call end point, not seeing the side bar).
Thanks Scott.


mysql failing push images

I lost track of the other issue, but mysql is still causing flakes in the push job:
https://ci.openshift.redhat.com/jenkins/view/Image Verification/job/push_images/18216/console
@mnagy @mfojtik
(if the other issue is still open, feel free to close this one).

@bparees push_images seems to work fine now.

i'm not so sure, but i'll reopen it the next time we rebuild all the images and it fails again :)

i'll bookmark this issue to know the weather on mysql push front :-)
On Fri, Feb 12, 2016 at 3:41 PM, Ben Parees<EMAIL_ADDRESS>wrote:

i'm not so sure, but i'll reopen it the next time we rebuild all the
images and it fails again :)
—
Reply to this email directly or view it on GitHub
https://github.com/openshift/mysql/issues/120#issuecomment-183355843.

--
Michal Fojtik<EMAIL_ADDRESS><EMAIL_ADDRESS>Red Hat OpenShift, Engineering


Tutorial on writing a node with rclrs

This is inspired by a question @AlexKaravaev had a while ago.

@tylerjw feel free to leave comments if you like.

@nnmm awesome work! Thanks for writing this up 🙂

One possible way for avoiding all this Arc<Mutex<_>> stuff would be to make rclrs::Node have a field data: Arc<Mutex<T>>, and make subscriptions callbacks take an additional &mut T argument which is supplied by the node, which would basically play the role of this in member function callbacks in C++. I'm not sure yet it's worth doing, so let me know if you have any comments.


ui: Increase graph left padding for y axis numbers

Closes #25

-12000% should be enough for now and the 100% is now fully shown for errors  too...

needs just a little more for errors imo

needs just a little more for errors imo

Is this screenshot based on this PR already?

Yup, other Y scales are looking much better


Implement captcha handler

It should:

knows how to detect if captcha had happened (unexpected landing to captcha url or existing on the page captcha scope)
knows scope of confirm button and input for enter captcha.
take a picture of captcha (do base64 from picture in evaluate)
has ability to set a promised function, which will determine captcha and return value.

Implement:

does action back (if needed) and continue parsing

This going to be implemented in here https://github.com/redco/goose-captcha-handler

@jifeon can you do this for me?

so, the idea is to have captcha, which will be initialized like that...
captcha(parser) -> this will add captcha action via Parser.addAction to the parser


Opening Camera on Huawei P9 Lite (Android 6.0)

Hi guys,
when I try to use camera, it will not open.
Logcat: E/Camera: Camera new cameraInitNormal:0
I think, it is because deprecated Camera class is no longer maintained on some devices like Huawei P9 Lite.
When I try to use newer android.hardware.camera2 package API, it works.
Example:
        CameraManager cm = (CameraManager) getSystemService(CAMERA_SERVICE);

        try {
            cm.openCamera("0", new CameraDevice.StateCallback() {
                @Override
                public void onOpened(CameraDevice camera) {
                    Log.d("XXX", "opened");
                }

                @Override
                public void onDisconnected(CameraDevice camera) {
                    Log.d("XXX", "disconnected");
                }

                @Override
                public void onError(CameraDevice camera, int error) {
                    Log.d("XXX", "error");
                }
            }, null);
        } catch (SecurityException | CameraAccessException e) {}

Have you resolved this issue?

Not yet.

It works in Huawei CAZ-AL

i also have problem with webrtc on Huawei P10. it will not stream my video. talky.io has problems too...:(

I'm closing this since we are now using the camera2 API. Feel free to open a new issue.


No module named 'Model'

ImportError: No module named 'Model'

@RongchangZhao Sorry for missing essential files.
I have updated the repo.
You can try it again ~ :)

It's ok. Many thanks to you!


Using command that returns large json doesn't work

I've been using this package with success but lately found an issue when I execute a command that will return a large json.
The issue appears to be that node is chunking the std out data, but each chunk is assumed to be a complete command response (or multiple command responses).
I'll attach a sample fix that works for me - essentially assuming that if data doesn't end with a newline we're getting an incomplete chunk of data, so keep appending and then process once we have a data chunk that is on a command response boundary
Sample fix in gonode.js:
// Receive data from go-module
function handleStdout(go, data) {
   go.stdoutData += data;

    // Response may be several command responses separated by new lines
   var dataSplit = go.stdoutData.toString().split("\n");
   if (dataSplit[dataSplit.length-1].length === 0) {
     // Data ended on \n so assume we have a complete command output (or complete set of cmmand outputs),
     // process the full stdout data
     dataSplit.forEach(function(resp) {
       // Discard empty lines
       if(resp.length > 0) {
         // Parse each command response with a event-loop in between to avoid blocking
         process.nextTick(function(){parseResponse(go, resp)});
       }
     });
     // Reset stdoutData
     go.stdoutData = '';
   }
}

yes, that will fix it, another enhancement is to initialize go.stdoutData to an empty string, if not you will have to remove the string "undefined" when parsing the output


UX Research

To create a User Experience Research for Iguana Hive.

[x] Finish my changes
[ ] Push my commits to GitHub
[ ] Open a pull request


Unable to run trivy operator: starting controllers manager: failed to wait for vulnerabilityreport caches to sync: timed out waiting for cache to be synced for Kind *v1alpha1.VulnerabilityReport

What steps did you take and what happened:
$ kubectl logs --selector "app.kubernetes.io/name=trivy-operator" --all-containers=true --since=4h --follow
...
W0409 10:08:26.052351       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.Deployment ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.052476       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.CustomResourceDefinition ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.052533       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.SbomReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.052585       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.Pod ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.052629       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.ClusterRbacAssessmentReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.052675       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.ClusterComplianceReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.052720       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.Node ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.052759       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.ClusterVulnerabilityReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.052801       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.ClusterConfigAuditReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.052846       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.LimitRange ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.052889       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.ClusterRoleBinding ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.052935       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.ClusterRole ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.053019       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.ClusterInfraAssessmentReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.053088       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.ReplicaSet ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.053137       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.ResourceQuota ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.053201       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.NetworkPolicy ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.053250       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.ConfigAuditReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.053571       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.DaemonSet ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.053859       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.StatefulSet ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.054063       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.Role ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.054277       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.ExposedSecretReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.054538       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.VulnerabilityReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.054745       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.RoleBinding ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.054931       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.Job ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.055242       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.ReplicationController ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.055500       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.CronJob ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.055675       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.ConfigMap ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.055869       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.Ingress ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.056202       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.Service ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.056529       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.RbacAssessmentReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
W0409 10:08:26.056777       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.InfraAssessmentReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from succeeding
{"level":"info","ts":"2024-04-09T10:08:26Z","msg":"Stopping and waiting for webhooks"}
{"level":"info","ts":"2024-04-09T10:08:26Z","msg":"Stopping and waiting for HTTP servers"}
{"level":"info","ts":"2024-04-09T10:08:26Z","msg":"shutting down server","kind":"health probe","addr":"[::]:9090"}
{"level":"info","ts":"2024-04-09T10:08:26Z","logger":"controller-runtime.metrics","msg":"Shutting down metrics server with timeout of 1 minute"}
{"level":"info","ts":"2024-04-09T10:08:26Z","msg":"Wait completed, proceeding to shutdown the manager"}
unable to run trivy operator: starting controllers manager: failed to wait for vulnerabilityreport caches to sync: timed out waiting for cache to be synced for Kind *v1alpha1.VulnerabilityReport
[   474.444654s]  INFO ThreadId(01) inbound:server{port=8080}:rescue{client.addr=n.n.n.n:47648}: linkerd_app_core::errors::respond: HTTP/1.1 request failed error=error trying to connect: Connection refused (os error 111) error.sources=[Connection refused (os error 111)]

What did you expect to happen:
No errrors.
Anything else you would like to add:
None.
Environment:

Trivy-Operator version (use trivy-operator version): v0.19.4
Kubernetes version (use kubectl version): v1.29.3
OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc): macOS 14.4.1

This issue is no longer reproducible after #1992 is resolved.


Crash report

came home to find daemon crashed.   Have been testing remote access to home computer, so recently set up port forwarding.
coda_crash_report_2020-02-08_20-49-17.639772.tar.gz

Happened again. Give a shout with any tests you would like me to run.
coda_crash_report_2020-02-09_03-53-34.605138.tar.gz

Closing this issue as it is stale for a long time.


Enhance the srctop, bldtop, data and result functions to check the result

This affects bldtop_dir, bldtop_file, srctop_dir, srctop_file,
data_dir, data_file, result_dir, and result_file.  They are all
enhanced to check that the resulting path really is a directory or a
file.  They only do this if the path exists.
This allows the tests to catch if these functions are used
incorrectly, even on systems where the syntax for directories and
files is the same.

This will stop failing as soon as #16518 has been merged into master

#16532 will be needed as well for this PR to stop failing

This pull request is ready to merge

Merged
2fe2279d1f3bbfa934e432d4f2c3a7e6a6b0f958 Enhance the srctop, bldtop, data and result functions to check the result


Abstract over effect types

Instead of using IO, we can use an F that satisfies the appropriate type classes.

This is stale but could be useful later... closing for now


VMA_ASSERT(index < m_ListsCount);

I am getting this assertion failure on Linux with a AMD GPU, and not with NVIDIA.
The assertion failure is triggered while trying to allocate an image with
VMA_MEMORY_USAGE_AUTO_PREFER_DEVICE
and
VK_IMAGE_USAGE_STORAGE_BIT
The format is VK_FORMAT_R8G8B8A8_UINT.
The GPU is a 6900 XT.
Validation layers are enabled with no validation error reported.
Here is the relevant code:
VkExtent2D extent {........}; // this is arbitrary but always valid extent..
VmaAllocationCreateInfo _allocInfo {.usage = VMA_MEMORY_USAGE_AUTO_PREFER_DEVICE};
VkImageCreateInfo _imageInfo {
            .sType = VK_STRUCTURE_TYPE_IMAGE_CREATE_INFO,
            .flags = 0,
            .imageType = VK_IMAGE_TYPE_2D,
            .format = VK_FORMAT_R8G8B8A8_UINT,
            .extent = VkExtent3D{.width = extent.width, .height = extent.height, .depth = 1},
            .mipLevels = 1,
            .arrayLayers = 1,
            .samples = VK_SAMPLE_COUNT_1_BIT,
            .tiling = VK_IMAGE_TILING_OPTIMAL,
            .usage = VK_IMAGE_USAGE_STORAGE_BIT | VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT,
            .sharingMode = VK_SHARING_MODE_EXCLUSIVE,
            .initialLayout = VK_IMAGE_LAYOUT_UNDEFINED
        };
vmaCreateImage(_device->_allocator, &_imageInfo, &_allocInfo, &_handle, &_allocation, nullptr)

This issue happens both on Windows and Linux, with both the current master branch of VMA and the v3.0.1 tag.
This code has been working fine for years with nvidia gpus.

Do you mean the assert inside function VmaBlockMetadata_TLSF::InsertFreeBlock? If so, is this the only or first allocation you make using VMA? If not, I won't be able to reproduce this issue by just passing the same parameters of the allocated image. This assert relates to TLSF allocator, which means the order and parameters of allocations and deallocations made before it likely influence the result. Additional help in debugging this issue is needed.

Yes that is the assertion I'm referring to.
It is the first image allocation made from a specific dynamic lib, but not the first allocation overall.
My engine allocates some buffers and images using VMA, and then loads some modules dynamically (dlopen/LoadLibrary), and the first vma image allocation from a certain module causes this assertion failure.
I have tried with other flags/usage and the issue persists, so it does have something to do with it being the first (or just after something else, in this case a storage buffer).
It is not a trivial engine/game so sharing source code won't be useful here.. but I can try to help as much as I can by answering questions and/or trying things on my end.

Could it be memory corruption in VMA internal CPU data structures? Or a synchronization issue? VMA should be thread-safe, but you can try to #define VMA_DEBUG_GLOBAL_MUTEX 1 and see if it helps. You may also disable / comment out parts of your project.

I am getting lock-order-inversion in amdvlkpro64.so upon calling vkCreateInstance
Would that mean it's a bug in AMD's drivers?

Yes but that shouldn't cause corrupted memory, only potential deadlocks. Also on Linux you really should be using RADV.

Does RADV support VK_KHR_ray_tracing? (my project requires it)

But anyway the assertion failure also happens on Windows.

Nothing to report with AddressSanitizer.
However, Valgrind reports a Condition jump or move depends on unitialized values.... in amdvlkpro64.so of course....

Does RADV support VK_KHR_ray_tracing? (my project requires it)

I think so, in some version at least. Maybe not the latest stable yet.

However, Valgrind reports a Condition jump or move depends on unitialized values.... in amdvlkpro64.so of course....

You can add --track-origins=yes to check if said value originates in your code.
One of the advantages of RADV is that you can check the driver source to see what's going on.

The uninitialized value comes from libvulkan.so.
But it doesn't seem to be related to the issue at all as it happens before creating the device and the vma allocator.
Also, I have tested with RADV, the lock-order-inversion error is gone when compiled with thread sanitizer, but I still get the original assertion failure.
I have also disabled all external modules in my engine and that assertion still happens just with a different allocation (the next one in line).
The last VMA allocation that worked before the assert is an image with
VMA_MEMORY_USAGE_AUTO
and
VK_IMAGE_USAGE_STORAGE_BIT | VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT | VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT
with format VK_FORMAT_R8G8B8A8_UNORM and extent is the size of the screen (1080p).
Then the allocation that fails is also an image, also with VMA_MEMORY_USAGE_AUTO but the usage also has the VK_IMAGE_USAGE_SAMPLED_BIT and the format is VK_FORMAT_R32G32B32A32_SFLOAT, and the extent is half that size.
Also note that in this last test I did without any module, the vulkan initialization and every allocation runs in the same (main) thread.
Let me know if there is any more information I can give you.

If it helps you debug the issue:

And here's the stack in my own code where I call the vma function to allocate that image.

I confirm that the assertion failure happens on the first call to VmaBlockMetadata_TLSF::InsertFreeBlock.
The other allocations before it don't seem to use TLSF at all.
I have also checked again with an nvidia gpu and it uses some TLSF allocations properly and without ever failing. (RTX 2080 and RTX 3090, windows and linux).

Thank you for showing this debugging session. This is very helpful. Can you please use some Debug (unoptimized) version of the code and check with the debugger what happens with the calculations inside VmaBlockMetadata_TLSF::InsertFreeBlock? When block->size = 32400, VMA_BITSCAN_MSB should return 14 and memClass should be 14 - 7 = 7, while your screenshot shows 42. Is the function VmaBitScanMSB working incorrectly in your environment?

static_cast<uint8_t>(__builtin_clzll(size)) with size 32400 results in the value 14.
But in that code path it's 63 - that 14, which results in 49... maybe this should directly return the value (14) instead?

Yep, that fixed it, no more assertion failure, all allocations work with my suggested fix.

What compiler version are you using? If this was the bug, it is hard to believe that no one stumbled upon it before. This function is used on any GCC or Clang compiler.

-- The C compiler identification is GNU 13.2.1
-- The CXX compiler identification is GNU 13.2.1
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found Python3: /usr/bin/python3.11 (found version "3.11.5") found components: Interpreter
-- Google Mock was not found - tests based on that will not build
-- spirv-tools not linked - illegal SPIRV may be generated for HLSL
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
-- Found Threads: TRUE

Funny story: Now with that "fix", I get the assertion failure with an NVIDIA GPU, but not with AMD...

It seems you compile your program on a different machine with different compiler and other hardware/software environment. I recommend to compile it once and check the same executable on both machines to separate the issue we have here from the GPU you are using.

I also get that same assertion failure on Windows (compiled using MINGW GCC from a Linux machine).
The assertion failure happens both on Windows and Linux, only with AMD gpu.
But with NVIDIA gpu it always works fine on both Windows and Linux.
And now after removing that 63 -, it's the other way around.

Can you please confirm that the same executable compiled once on one machine and copied to the other machine works correctly on one and fails on the other?

I can confirm that. It was the case when I first posted the issue.
I was compiling with GCC 13.2.1 on the Nvidia machine, then running the executable on the AMD machine directly.
It was only yesterday that I decided to re-create that dev env on the AMD machine so I can do quicker iterations, and it gives the same results.

Yes, launched in the same way (tried both ways on both machines)

Can you please test this simple program compiled and launched on both your machines? This should help us debug the issue without connection to VMA or the GPU you have installed.
https://godbolt.org/z/3TrTe6fs4
#include <cstdio>
#include <cstdint>
int main()
{
    unsigned long long size = 32400;
    uint8_t y = static_cast<uint8_t>(__builtin_clzll(size));
    printf("%i\n",(int)y);
}

Compiler Explorer says it prints 49, as expected by the current version of VMA code.

got 49 on the NVIDIA machine and 14 on the AMD machine, compiled individually with same environment as the main project (just an additional target in the cmake config).
I have also copied the one compiled from the NVIDIA machine to the AMD machine, and it also gives 14 there.
I confirm that the result depends on which machine it was executed on, not which machine compiled it.
Additional info about the two machines:
Machine 1: Linux kernel 6.1.44-1-MANJARO with NVIDIA gpu, intel i9-9900k
Machine 2: Linux kernel 5.15.130-1-MANJARO with AMD gpu, intel i7-2600k
both 64-bit of course.

This is a very interesting finding. The difference is likely due to a different version of the software environment (OS, standard C library, kernel). It surely doesn't depend on the GPU installed, although we can still call these machines AMD and NVIDIA just to give them names.
Is there a known bug in the older Linux version related to this function? Maybe it is worth searching for it.

But the interesting thing is I also get that assertion failure on Windows, on that AMD machine (multiboot) compiled from MINGW

I have updated the AMD machine to Linux kernel 6.1, it still gives 14.
I have compiled and tested that same test for windows, on that AMD machine it gives 14.
On another Windows machine with an NVIDIA GPU, it gives 49....
All machines have Intel CPU.
Weirdly, it really looks like it has nothing to do with the OS, but the GPU....

I just swapped the GPU for an old GTX 1060 and I still get 14.
Now let's compare two Windows machines:
Machine 2: Intel i7-2600k, Windows 10 pro 22H2, 8 Gb RAM, GTX 1060. result = 14
Machine 3: Intel i7-11700k, Windows 10 pro 22H2, 32 Gb RAM, RTX 3090. result = 49
There is quite a big gap in the CPU architecture year, maybe that is it...

I have swapped both the SSD and the AMD GPU from Machine 2 to Machine 3.
So, Machine 3 now has the AMD RX 6900 XT and running the exact same Linux OS as the one that had the error.
I am now getting 49 as expected.
We can safely rule out the GPU and the OS.
Which leaves us with CPU, Motherboard and RAM.  I'm guessing it has to do with the CPU, even though they're both Intel...

Thank you for all the testing you are doing, including hardware manipulation. Despite the bug is clearly unrelated to VMA, let's continue discussion about it here, as I am curious about the root cause.
Can you use the debugger to enter inside __builtin_clzll and see the x86 disassembly, to see whether there are some calculations inside that can go wrong, or is it just usage of a processor instruction that gives wrong result straight away?

Could this issue be related?
https://stackoverflow.com/questions/45107728/different-gcc-output-for-builtin-clzll-on-different-optimisation-levels-and-wr

Figured it out after looking at the disassembly (thanks).
I was using the -mlzcnt optimization flag, and turns out that was not supported on the 2600k hence was giving the wrong result.
On the left it's with that flag on, and on the right it's off.
With the flag off it properly gives the value 49.

I will close this issue.


Object reregistering bug

the object sometimes does not properly reset position and is registered several times affecting users score
This bug must be fixed before user tests

Fixed by 581855d


Update google-cloud-pubsub to 1.115.2

Updates com.google.cloud:google-cloud-pubsub from 1.107.0 to 1.115.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-pubsub" } ]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #284.


.table-responsive fails with .d-flex.flex-row ancestor

I'm working with the Blazor template, that is built with Bootstrap 4.5, that comes out of the box with a new Blazor project. I noticed that .table-responsive doesn't work due to:

.d-flex.flex-row in the outermost div, and
the .table-responsive has a div between it and the outmost div

Removing either of those functional pieces fixes the scroll issue, but I can't tell if this is a bug with how the .table-responsive is coded or something else.
Here is a reduced test case: https://www.codeply.com/p/sohA2Q2ojS
Environment

macOS 10.15.5
Safari 13.1.1, Chrome 83

I think the outer div should have col so its width is constrained, responsive-table can't do much about the it's parent element

Left item

@psimyn I see what you're going for there, and it makes sense. I think my actual use case is more complicated than the reduced test case since adding the col or a fixed width to the parent doesn't seem to fix it.
Namely, as the browser width is reduced (and scrollable responsive table width is reduced), there is what looks like a right margin on the body that grows inversely proportional to the shrinking of the responsive table. It is not a margin or anything according to the browser tools, it's just a white column to the right of the body that causes the browser window to scroll.
I'll circle back when I have time to produce another reduced test case.
If you have any other thoughts in the meantime, please share!

Here is a more specific reduced test case, which just uses the overflow-x: auto; that .table-responsive uses.
Note that if you open this link and reduce your browser to the smallest width, the table scrolls but the browser window also still scrolls.
Is it an issue with flex-direction: row;?
https://www.codeply.com/v/njwJ5mDjAn

@mrlife Your latest example has a body scroll because you set a width larger than the window. It allows to scroll the table (which needs a constraint to work). You can work around this by usin max-width: 500px; and min-width: calc(100vw - 100px); —100px being your sidebar width, approximatively.
Your first example sure is interesting but to make this pattern using Bootstrap, you should use .row and .col-* classes for your layout. This would handle flexbox model and widths appropriately and allow your table to work fine.
Since this is not a bug in Bootstrap (but more likely a misunderstanding about tables behaviour in a flex container), I'm closing this as won't fix.

@ffoodd Thanks. Hopefully there will be an official fix for this soon.
Discussion at https://github.com/w3c/csswg-drafts/issues/865


Update README

CSS require is different, at least in Rails 4.

You are right about this. I accidentally used the SCSS syntax of Sass. Can you please rebase the master into your branch before I merge this?


Asynchronous Extraction and Progress

@pixelglow do you have any recommendations/best practices for reading (extracting) a ZZArchive asynchronously?
Obviously I could just use GCD. Just wondering your thoughts.
Note: originally asked in #59

We definitely support extraction on any single thread, single serial dispatch queue or equivalent. That is, all access and extraction happens on the single thread or single serial dispatch queue.
That being said, ZipZap may work with more esoteric models. In particular, individual ZZArchiveEntry objects could be read or extracted in different threads or serial dispatch queues. I haven't vetted the code for this possibility, but since ZZArchiveEntry objects are read-only and independent of each other, it should work. On the other hand, making ZipZap eventually work with huge zip files may end up imposing some dependencies between the entry objects, although I'd still try to keep them independent. Is this extraction model important to you?

It definitely is; especially with progress reporting (like talked about in #59) which is why I originally brought it up there.
The project that we make use of ZipZap for generally only unzips archives > 250 MB. As you may expect, doing so on the main thread synchronously obviously is less than ideal. Currently I have my own class that inherits ZZArchive (it originally was just using extensions in Swift but I liked the inheritance better in the end) and implements the NSProgressReporting Protocol.
Internally it creates a serial queue and works through each of the elements in the entries array. I don't have the code in front of me right now but notionally it looks something like this:
func extract() {
  self.progress.totalUnitCount = entries.count
  let serialQueue = dispatch_queue_create("me.lovelett.namespace", DISPATCH_QUEUE_SERIAL)
  entries.forEach { (entry) in
    dispatch_async(serialQueue) {
       // Create directories as necessary
       // Write data from entry
       // Update the progress
       if (entries.last == entry) {
         // Clean-up and notify the callback that the async task is done
       }
    }
  }
}

I think having a convenience class like this that does both sync and async "extraction" would be helpful. That having been said, I understand it may not make sense for this library itself, I don't know what your intended scope or audience is. At the very least though I think providing a link to a Gist or something on the recipes page may be a good first step.

As you may expect, doing so on the main thread synchronously is less than ideal.

You can extract on a single thread or single serial dispatch queue, it doesn't have to be the main thread.

Internally it creates a serial queue and works through each of the elements in the entries array.

The code iterates in one thread and then extracts on a different single thread/queue. For now, this should work, and after I've finished supporting large archives, this should still work but I can't guarantee it now (I may end reimplementing this iteration lazily for example). The safer alternative is to move the iteration code to the serial queue as well.

Currently I have my own class that inherits ZZArchive ... and implements the NSProgressReporting Protocol.

If you have a granular NSProgressReporting implementation, I'd love to see it in mainline! The idea is we should be able to see progress down to the chunk level e.g. even within the various streaming objects.

I think having a convenience class like this that does both sync and async "extraction" would be helpful. That having been said, I understand it may not make sense for this library itself, I don't know what your intended scope or audience is.

I'm trying to keep ZipZap minimal and composable, a platform others can build on top of. Otherwise, including all combination of possibilities will end up complicating the API and the understanding of the API, and including code that not everyone wants. For example, not every needs to extract to a file, so there's no direct API for that.
That being said, I'd be OK with really common idioms making it to the API.
As for sync vs async, it's a tougher call. I like a lot of callback-based API's e.g. node.js because they don't block the main thread by default. On the other hand, an extract-only async may not be the right level of granularity, for example you might want to do something heavy to the extracted data in that other thread and not have to hand the data back to the calling thread for further processing -- creating a CGImageRef bitmap out of extracted data comes to mind, or immediately writing out the data to a file, or doing something that requires 2 or more entries. Sync API's are simpler to understand, reason about and compose. On the balance of things, I'd keep the API sync.

At the very least though I think providing a link to a Gist or something on the recipes page may be a good first step.

For sure. If you have a suitable example, perhaps by expanding out your snippet while keeping it focussed, I'll include it in the recipes page.

I'll work on distilling the class down the minimal and try and post by Friday.


[Python] Managed Transforms API

Adding Python API for Managed transforms, similar to the Java API.

Successful Dataflow run (2024-06-12_11_40_12-3984761114382040860) with the following code:
import logging
import apache_beam as beam
from apache_beam.options.pipeline_options import PipelineOptions

_LOGGER = logging.getLogger(__name__)
_LOGGER.setLevel(logging.INFO)

options = PipelineOptions([
  "--runner=DataflowRunner",
  "--job_name=managed-iceberg-read-demo4",
  "--project=apache-beam-testing",
  "--temp_location=gs://apache-beam-testing-ahmedabualsaud/tmp",
  "--region=us-central1",
  "--sdk_location=sdks/python/dist/apache_beam-2.57.0.dev0.tar.gz",
  "--sdk_harness_container_image_overrides=.*java.*,gcr.io/apache-beam-testing/beam-sdk/beam_java11_sdk:latest"
])

with beam.Pipeline(options=options) as p:
  p | "Managed Read from ICEBERG" >> beam.managed.Read(
    "iceberg",
    config={
      "table": "testRead.table16bc5d47",
      "catalog_config": {
        "catalog_name": "name",
        "catalog_type": "hadoop",
        "warehouse_location": "gs://temp-storage-for-end-to-end-tests/IcebergIOIT/testRead/08a68b5b-c973-450e-b6dc-fc552a623b35"
      }}) | "Log rows" >> beam.Map(_LOGGER.info)

Successful Dataflow run (2024-07-15_13_54_11-2004515317250011573) with the following code:
import logging
import apache_beam as beam
from apache_beam.options.pipeline_options import PipelineOptions

_LOGGER = logging.getLogger(__name__)
_LOGGER.setLevel(logging.INFO)

options = PipelineOptions([
    "--runner=DataflowRunner",
    "--job_name=managed-iceberg-read-demo",
    "--project=apache-beam-testing",
    "--temp_location=gs://apache-beam-testing-ahmedabualsaud/tmp",
    "--region=us-central1",
    "--sdk_location=sdks/python/dist/apache_beam-2.59.0.dev0.tar.gz",
    "--sdk_harness_container_image_overrides=.*java.*,gcr.io/apache-beam-testing/beam-sdk/beam_java8_sdk:latest"
])

with beam.Pipeline(options=options) as p:
    p | beam.managed.Read(
        "iceberg",
        config={
            "table": "my_database.my_table",
            "catalog_name": "ahmed_catalog",
            "catalog_properties": {
                "catalog-impl": "org.apache.iceberg.hadoop.HadoopCatalog",
                "warehouse": "gs://ahmedabualsaud-apache-beam-testing"
            }}) | "Log rows" >> beam.Map(_LOGGER.info)

LGTM. We can merge after existing comments are addressed.

@chamikaramj the last comment was just addressed. It's been a while though, lmk if you'd like to take another quick look before I merge

Fixes #31830

Looks like we merged this with
beam_PreCommit_RAT (Run RAT PreCommit) failing, and it is still broken at master @ahmedabu98 - could you PTAL? Thanks!

> Task :rat
Files with unapproved licenses:
 - /runner/_work/beam/beam/sdks/python/apache_beam/transforms/managed_iceberg_it_test.py

Ahh my apologies, I didn't pay attention and thought the failure was a flake


Is there a way I could subscribe to 3 different WVD tenants with the app? I use WVD for three different organizations.

Is there a way I could subscribe to 3 different WVD tenants with the app? I use WVD for three different organizations.

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

ID: 524e47b0-1295-7d07-0cbc-86e6daaa1358
Version Independent ID: 1cb3a5d8-4f16-583c-ad85-325f7638a873
Content: Connect to Windows Virtual Desktop Windows 10 or 7 - Azure
Content Source: articles/virtual-desktop/connect-windows-7-and-10.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@ThomasITS can you post a link to the doc you are referring to?

URL is https://docs.microsoft.com/en-us/azure/virtual-desktop/connect-windows-7-and-10

Good lord, please let there be an answer for this.  This Remote Desktop Client has a ways to go before I can install it in a shared environment or use it for multiple subscriptions.
Thanks!

@ThomasITS
Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.

The answer above is not helpful.  As I said, these are three different organizations,  They also have three different Azure AD systems. Is there a way I could subscribe to 3 different WVD tenants (3 different Azure ADs) with the app?

@ThomasITS  - I agree - there needs to be a solution to this before WVD can be considered an enterprise solution.  It's really disappointing that we cannot just use the native windows remote desktop and apps client

@ThomasITS Apologies for the confusion. I checked with internal team on this, and currently this is not possible.
Requesting to provide this product feature feedback in Tech Community Forum here, where the WVD product team is actively checking and responding.


gh-1679 FederatedStore should execute operations using the fully User…

… Context object

Codecov Report

Merging #1685 into develop will decrease coverage by 0.92%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #1685      +/-   ##
=============================================
- Coverage      65.81%   64.89%   -0.93%
  Complexity      1956     1956
=============================================
  Files            803      757      -46
  Lines          25832    24587    -1245
  Branches        3325     3184     -141
=============================================
- Hits           17002    15956    -1046
+ Misses          7493     7337     -156
+ Partials        1337     1294      -43

Impacted Files
Coverage Δ
Complexity Δ

...mulostore/operation/impl/GetElementsWithinSet.java
95.45% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...affer/operation/impl/generate/GenerateObjects.java
88% <0%> (ø)
10% <0%> (ø)
:arrow_down:

...gchq/gaffer/operation/impl/function/Transform.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

.../gchq/gaffer/operation/impl/job/GetJobDetails.java
100% <0%> (ø)
7% <0%> (ø)
:arrow_down:

...k/gov/gchq/gaffer/operation/impl/compare/Sort.java
100% <0%> (ø)
13% <0%> (ø)
:arrow_down:

.../gov/gchq/gaffer/operation/impl/output/ToList.java
100% <0%> (ø)
7% <0%> (ø)
:arrow_down:

...gaffer/operation/impl/add/AddElementsFromFile.java
100% <0%> (ø)
14% <0%> (ø)
:arrow_down:

.../gov/gchq/gaffer/operation/impl/DiscardOutput.java
100% <0%> (ø)
6% <0%> (ø)
:arrow_down:

...ark/operation/javardd/ImportJavaRDDOfElements.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

.../spark/operation/scalardd/GetRDDOfAllElements.java
36.84% <0%> (ø)
3% <0%> (ø)
:arrow_down:

... and 109 more

Continue to review full report at Codecov.

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


Suggestion: replace Azure NetApp Files with Azure Files

As a Microsoft Technical Trainer, I don't see much activity with Azure NetApp Files. Instead, Azure Files seem more common.
To that end, may I suggest that under the table at https://learn.microsoft.com/en-us/azure/architecture/guide/design-principles/managed-services#paas-alternatives-to-iaas-solutions, we replace Azure NetApp Files with Azure Files (https://azure.microsoft.com/en-ca/products/storage/files)?

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

ID: 952ee382-9a9e-0f79-807b-1f7c2b819a42
Version Independent ID: 82f16168-5f05-f8e3-8971-fa9c7cbb828f
Content: Use platform as a service (PaaS) options - Azure Architecture Center
Content Source: docs/guide/design-principles/managed-services.md
Service: architecture-center
Sub-service: guide
GitHub Login: @martinekuan
Microsoft Alias: architectures

Thank you for pointing this out. This has been updated and should be live within one or two business days.


docker-compose error

Hi, @luontola
Unfortunately I've got error while starting the project with following commands docker-compose pull and docker-compose up -d:
ERROR: In file './docker-compose.yml' service 'version' doesn't have any configuration options. All top level keys in your docker-compose.yml must map to a dictionary of configuration options.
How can you resolve it?
Best regards,
@aimkata

Solved by re-installing my docker-compose (See ref. https://docs.docker.com/compose/install/#install-compose)


hostap: add WPS PBC and PIN for AP mode

add WPS PBC and PIN for AP mode.

Hi @jukkar @krish2718 please review

Hi @jukkar  please review

Hi @krish2718 @jukkar @dleach02 please review

Please rebase on main and resolve any merge conflicts.

Hi @krish2718 @jukkar @dleach02 please review

Hi @krish2718 @jukkar @dleach02 please review

Hi @krish2718 @dleach02 fix the conflict, please review again

Hi @krish2718 @jukkar @dleach02 fix the conflict, please review again

Hi @dleach02  please review and approve this

Hi @henrikbrixandersen rebased and conflicts resolved, can you help review again?

temporary hold until #79536 merges

#79536 was merged. Please take another look.

Hi @henrikbrixandersen
David was worried that this PR may cause conflict on https://github.com/zephyrproject-rtos/zephyr/pull/79536, which is also from me and more urgent, so he commented on this.
Now since https://github.com/zephyrproject-rtos/zephyr/pull/79536 was merged, and I rebased and fixed the conflict, I think can go ahead and merge this PR. Thanks.


Charm++ type traits for compile-time querying of types

Original issue: https://charm.cs.illinois.edu/redmine/issues/1864

Spectre defines these here: https://github.com/nilsdeppe/spectre/blob/charm_simplifications/src/Parallel/TypeTraits.hpp

Original date: 2018-04-13 18:35:21

We can probably just take the code from Spectre ad stick it into charm++.h or some header. The only thing we need to be careful of it C++14/17 features used here.

Original date: 2018-04-13 19:52:01

Looks like std::void_t is the only C++17 thing here, so we can just add a configure-time check for it and conditionally compile it.

Original date: 2018-04-13 22:58:50

~~https://charm.cs.illinois.edu/gerrit/#/c/charm/+/3988/~~ https://github.com/UIUC-PPL/charm/commit/e33294a32d06aa1bfcffd5748b3104c7dc7240cc


KAFKA-12380 Executor in Connect's Worker is not shut down when the worker is

The Worker class has an executor field that the public constructor initializes with a new cached thread pool (https://github.com/apache/kafka/blob/02226fa090513882b9229ac834fd493d71ae6d96/connect/runtime/src/main/java/org/apache/kafka/connect/runtime/Worker.java#L127.]).
When the worker is stopped, it does not shutdown this executor. This is normally okay in the Connect runtime and MirrorMaker 2 runtimes, because the worker is stopped only when the JVM is stopped (via the shutdown hook in the herders).
However, we instantiate and stop the herder many times in our integration tests, and this means we're not necessarily shutting down the herder's executor. Normally this won't hurt, as long as all of the runnables that the executor threads run actually do terminate. But it's possible those threads might not terminate in all tests.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

cc @showuon , PTAK, thank you


Header inclusion tweaks, ensuring that libaudit development files are present

Hi Alec, these are a few changes to how you're pulling in system headers.  The docs say that pwd.h requires declarations from sys/types.h, so I added it, and changed "" to <> for files that we expect to find in the system include directories rather than in the docker source tree.

Hi Nalin,
Thanks so much for sending these over to me. I really appreciate it. Regarding the changes to the libaudit bindings, I wonder if those should go to their upstream source as well?
https://fedorahosted.org/audit/browser/trunk/bindings/golang/audit.go

Oh, I didn't realize that this came from somewhere else.  Will do.  Thanks!

No worries Nalin, I can send them up if it makes things easier for you. I just wasn't entirely sure what the process for that would be so I just wanted to check before I merged anything :)


[CCI] chore: cci workflow

Description
[CCI]
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I'm not sure about this. This would be very convenient for sure but we don't let non-maintainers to add labels for security reasons and this will bypass that measure.

I'm not sure about this. This would be very convenient for sure but we don't let non-maintainers add labels for security reasons. And this will bypass that measure.

Totally expected, haha
Closing this


DCOS-12274: Show rejected offers warning in service breadcrumb

This PR displays the rejected offers warning icon & tooltip in the service breadcrumb and moves the corresponding code to a separate component.

retest this please

LGTM after addressing @nLight's comments!

Since I would like to use this functionality also on a new filter (has:rejectedoffers) does it make sense to move the check into a utility file? (In addition to the new component)? Otherwise I will do so in a future PR when I introduce the filter...

run unit tests

@Poltergeist :1st_place_medal:

I addressed the last comment from nLight. @jfurrow

run unit tests


Autoscrolling when adding new items in donations/distributions/purchases

Summary
When creating a new donation/distribution/purchase,  add autoscrolling, so that when you add a new item,  the item is on the screen!
Why
Reduces bank user frustration -- they have to manually scroll every time they add an item now.  Some distributions have over a dozen items.
Details
To see the issue ...
Sign in as<EMAIL_ADDRESS> In the left hand menu, click Donations, then "New Donation".   Add in the basic info (partner, storage location),  then  click"add an item"
When you press the "add an item" button,  the screen should scroll so that the item selection that is added appears on the screen. (if the first one fits on your screen,  do it again, and you'll see)
This is probably a fix in one piece of code that applies across donations/distributions/purchases
Criteria for completion

[ ] The "add an item" behaviour described above works on new distributions, purchases, and donations
[ ] This behaviour also works when editing distributions, purchases, and donations

Bonus Round

[ ] Same thing for requests (it's possible this will happen automatically -- IIRC there is a common element)

I'd like to work on this!


Replace pytest pep8 with black

The linter we use, pytest-pep8, emits pytest warnings. That tool received no updates since 2014.
I will replace it with black, running the tool accross the whole codebase.
I will also introduce a .pre-commit-hook.yaml, so that contributors using pre-commit will run the linter check automatically before each commit.
To be merged after https://github.com/paylogic/halogen/pull/24

Coverage remained the same at 98.513% when pulling d52baf6c9fe1c8fcc845338e7ce9375573615a13 on replace-pytest-pep8-with-black into b01ec8ad99c433d66d916413c10b01c1ef51b38b on fix-warnings.


add pythagoras_theorem.wy

亦称勾股定理

Hi @MerakDipper , thanks for the new example!
However, I think 勾股求弦 already has a (more sophisticated) implementation in the standard library 算經. (https://github.com/LingDong-/wenyan-lang/blob/master/lib/算經.wy#L746)
Is there a particular reason you think it would be necessary to add this version?
Thanks!

@LingDong- my bad, I didn't realize it already existed. I did a search on "勾股" but github didn't return me anything code.
My intention was to add more examples that people with less programming/algorithm knowledge but more literature knowledge （with basic mathematical understanding maybe)  can relate to. Similarly I was about to create an example of 贾宪三角（杨辉三角 ）. Could you let me know if that's also implemented here?
Thanks for replying.

Hi @MerakDipper , I don't think pascal's triangle is in there yet, feel free to add that. In fact, examples in ancient math textbooks are most welcome!
Thanks!

closing per discussion above


Update Octicons

Fixes #65508 and fixes #36053
This updates our version of Octicons, while still having backward compatibility with icons that have been removed in the newer versions. I've created this tool that generates the new icon from the Octicon npm module.
The only minor changes are that organization and person are now filled versions instead of outlines.
Additions

arrow-both
bold
eye-closed
fold-down
fold-up
grabber
italic
kebab-horizontal
kebab-vertical
logo-gist
note
plus-small
project
screen-full
screen-normal
smiley
tasklist
text-size
unverified
verified

Updated

color-mode is persisted
organization-outline persists previous version of organization
person-outline persists previous version of person

Comparison
After installing a few extensions that use octicons in the statusbar (vscode-spotify, live share, github pull request), you can compare the differences.

@misolori just noticed two things:

please keep in mind to release note this properly, I would also think we could create an issue over in vscode-docs to document the valid icons a bit better (maybe with showing the icon on the website?)
run a search for octicon in our repo to see where we use octicons in our core and verify nothing is broken, e.g. it looks like in some places we directly reference an icon through CSS
finally it would be good to run a full build with your changes just to make sure things are packaged up properly. e.g. I noticed some build rules in our gulpfiles related to octicons

@bpasero:

Removing the logo-gist and plus-small is totally fine. I've seen people not wanting the Gist logo in other issues and we already have a regular plus.
I can do a swap of the people and organization icons for live ahre and move the new ones to the appended name
I was already planning on mentioning this in the release notes, I've created https://github.com/Microsoft/vscode-docs/issues/2315 to track this is
I'll also do a final test and double-check the other areas Octicons are being used and create a custom build

👍 Thanks for feedback!

@misolori great, feel free to merge then after you tested the build


team leave post; commandline command

Still in progress (have to write some tests).

The commits 6c2653b4b59b...1c142d32afa4 introduce visual changes on linux. :mag_right: 1 changed

Changed: Tracker-New_user_Scroll_1 (before) (after)

The commits fdb0b21e1021...dd7459497e1f introduce visual changes on linux. :mag_right: 1 changed

Changed: Markdown-emoji (before) (after)

@maxtaco - I added an "aux: JsonPayload" parameter to the posting method, so leaving a team will have {sigs: ..., aux: {permanent: false}} for example.

ok cool, lmk when i should take another look

The commits fdb0b21e1021...2f2b52dc67e6 introduce visual changes on linux. :mag_right: 1 changed

Changed: Markdown-emoji (before) (after)

I think it's fine now. Just working on testing the command line

@maxtaco Sorry, still a bit confused about SharedSecret. Does the leaver sign the leave link?

That's the individual's device's key.  I didn't think it came through that code path.

Okay I think I get it, so I just leave out signingKey and encryptionKey in the case of a leave?

yes!

Now
-doesn't require sharedsecret
-validates against sigchain player
-removing yourself performs a leave
-leaving as admin works (added test verifies this)


Use camel case for component param names

Be consistent with existing components, doc this somewhere later

Super! 👍


[BUG] Setting body after make_case() causes request to be sent without a body

First off: Great library :100:

Checklist

[x] I checked the FAQ section of the documentation
[x] I looked for similar issues in the issue tracker
[x] I am using the latest version of Schemathesis

Describe the bug
The body is not sent if it is manipulated after calling make_case(). Setting body when calling make_case() works.
Related: https://github.com/schemathesis/schemathesis/issues/1094, https://github.com/schemathesis/schemathesis/issues/1188
To Reproduce
import schemathesis
from schemathesis import Case

schema = schemathesis.from_path(
    './petstore.yaml',
    base_url='http://<IP_ADDRESS>:4010',
    validate_schema=True,
)

def test_api():
    # This does not work:
    add_pet: Case = schema.get_operation_by_id('addPet').make_case()
    add_pet.body = {'name': 'ABC'}
    response = add_pet.call_and_validate()
    # This will print None
    print(response.request.body)

    # This works instead:
    add_pet: Case = schema.get_operation_by_id('addPet').make_case(body={'name': 'ABC'})
    response = add_pet.call_and_validate()
    # This will print the request body
    print(response.request.body)

    # Capture stdout
    assert False

🚨 Mandatory 🚨: Steps to reproduce the behavior:

Download OpenAPI specification
Start mock server (e.g. using Prism)
pytest main.py

Please include a minimal API schema causing this issue:
https://github.com/OAI/OpenAPI-Specification/blob/main/examples/v3.0/petstore.yaml
Expected behavior
The body should be sent since the media type is specified in the OpenAPI specification.
Environment
- OS: Linux
- Python version: 3.12
- Schemathesis version: 3.28.1
- Spec version: OpenAPI 3.0.0

@simonhammes
Thank you! I've been busy with other issues, but I hope to get back to this in a week or so. Sorry for the delay.

The fix will be available in the next release


Async uploads are blocking but async downloads are not

You can get by this by running an async upload on a different thread or using

import asyncio
# Make sure you've authenticated
def download(client, drivename, localname):
    loop = asyncio.get_event_loop()
    loop.run_until_complete(client.item(drive='me', path=drivename).download_async(localname))

download(client, "largefile.whatever", "downloadedfile.whatever")

but this is odd behavior that should probably be fixed.

Thank you for reaching out and for your patience. This SDK is being officially deprecated. See #209 for more information


Expose v8_fn_ptr from an op

Some usages in deno depend on remapping OpDecl to change some details, currently following error is returned:
error[E0624]: associated function `v8_fn_ptr` is private
   --> runtime/ops/os/mod.rs:58:43
    |
58  |       v8_fn_ptr: deno_core::op_void_sync::v8_fn_ptr as _,
    |                                           ^^^^^^^^^ private associated function
    |
   ::: /Users/ib/dev/deno_core/core/ops_builtin.rs:107:1
    |
107 | #[op2(core, fast)]
    | ------------------ private associated function defined here

Note that this is actually exposing a footgun -- the ops need to copy the fast function as well as the slow. #26 will fix this but will require a deno_core patch:
   deno_os_worker,
   ops_fn = deno_ops,
   middleware = |op| match op.name {
-    "op_exit" | "op_set_exit_code" => deno_core::OpDecl {
-      v8_fn_ptr: deno_core::op_void_sync::v8_fn_ptr as _,
-      ..op
-    },
+    "op_exit" | "op_set_exit_code" => op.with_implementation_from(&den
o_core::op_void_sync::DECL),
     _ => op,
   },
 );


Default SSL Options don't work in OTP-26+

See https://github.com/erleans/pgo/issues/77
Also, it'd be great if, like say Ruby's pg, we could maybe automatically enable ssl when needed.

gleam_pgo is using pgo under the hood, and I needed to have a correct SSL defaults for the package. In the end, I ended up with a configuration like:
default_ssl_options(Host, Ssl) ->
  case Ssl of
    false -> [];
    true -> [
      {verify, verify_peer},
      {cacerts, public_key:cacerts_get()},
      {server_name_indication, binary_to_list(Host)},
      {customize_hostname_check, [
        {match_fun, public_key:pkix_verify_hostname_match_fun(https)}
      ]}
    ]
  end.

Which solves the settings for SSL. Maybe this should be merged directly to the pgo package? I suppose pgo should also take care of merging the defaults settings with the user settings, or let user overrides the defaults settings?

That looks fab to me!
Though I must admit I'm not very familiar with configuring SSL in Erlang 😁

I'm not sure @tsloughter wanted to push it in pgo. I let him confirm or infirm here 🙂


Character Preview in lobby is incorrect

Description
LobbyCharacterPreviewPanel doesn't account for species and is hardcoded to spawn a MobHumanDummy
Screenshots
Slimeperson but it uses a Human in the preview

LobbyCharacterPreviewPanel

Ah shoot did I miss a file

seems so
https://github.com/space-wizards/space-station-14/blob/512d6a38c3cf0da68ddd17868f12f2f6dd4a316d/Content.Client/Lobby/UI/LobbyCharacterPreviewPanel.cs

Well the solution is easy, literally just checkout LobbyCharacterPreviewPanel from Outer Rim 14, as it's already correct there.

Well the solution is easy, literally just checkout LobbyCharacterPreviewPanel from Outer Rim 14, as it's already correct there.

I have done just that


Params are empty for grouped previews

The params[:lookbook][:display] is empty for grouped previews no matter what and where I try to set.

Thanks for the report @mdrbohlav - I'll try and take a look at this as soon as I can.
In the meantime could you let me know which version of Lookbook you are using to help me track down the issue?

@allmarkedup Thank you, I'll try to debug it as well but I just found this gem so it might take a little more time.
I am using version 0.7.1.

Hi @mdrbohlav - I've just released a beta version (v0.7.2.beta.4) that should address this issue. If you could give it a test and let me know if it fixes things for you that would be great.
It's worth noting that @display params work slightly differently for grouped previews - all of the @display values set on the examples in the group are merged together as they are all rendered in the same layout. So you only really need to set the @display values on one of the examples in the group.
Hope that makes sense and let me know how you get on if you get a chance to try the beta version.

@allmarkedup Merging the params in the group seems logic. Looks good in the beta 4, thank you for the quick response!


Do not give right to my profile if they are already in my contacts WebACL group

It seems this code doesn't work, because there are a lot of users who are given individual rights to view my profile (via acl:agent), even though they are part of my contacts WebACL group:
https://github.com/assemblee-virtuelle/activitypods/blob/master/packages/events/services/registration.js#L53-L73

Done here 8e542982a00234d77cb9b2de384ce23167382380
Eventually we could run a migration to remove the acl:agent, after ensuring that they are all in the contacts WebACL group.


Token2022 (extensions) support

Add support for Token2022 instructions(extensions).
Added:

token2022 program id
ProgramIdToken2022 enum value
switch case to handle token2022 in message and transaction_printer
extension names to the token instruction tag enum
unit tests

@sponomarev Hello, I've made some changes. Hopefully I didn't miss anything :)
I've checked transactions with multiple instructions for old spl token as well as for the Token2022 and the same with simple examples like transfer. On my end it looked fine.
In my opinion the best way to handle this was to unify both token types as the same programId. Otherwise handling transaction_printers was getting a little tricky.
Let me know it there is anything else to change or if you find any other problems.

The extensions are not supported currently. The main purpose of this PR was to properly handle basic operations on token when using Token2022 program. We wanted to make it backward compatible with existing features without the need of blind signing.
This scope was discussed with @jnwng

Hello @tiago18c can you please summarize what are the next steps? Is there anything else from our side that should be done?


Add toml support for Configurate

Currently, dash is using Jackson for TOML (de)serialization, and there are some existing implementations of TOML support for configurate. However, these implementations don't use Jackson, they use TOMLJ instead.
The existing configurate-Jackson can be a great example for assignees.
Create a new repository to do this.

@EvanLuo42 how is it going?

@EvanLuo42 how is it going?

In progress

please tell me your progress

please tell me your progress

I don't know what do you mean by using Jackson to implement. Isn't Jackson only for Json?

By simply check the repository you'll find that there are many dataformat supports in Jackson.

and TOML is one of them, what you need to do is make some patches for TomlMapper and TomlFactory etc, I think

what is going on

月考ing

huh  you are busier than me

Assign received, configurate is the one by SpongePowered?
I haven't heard about Jackson for TOML.

https://github.com/FasterXML/jackson-dataformats-text/tree/2.13/toml

Ctrl+F won't hurt you.

I will do it in several days.

https://github.com/qyl27/Configurate-TOML
A simpleimplementation with some known issues.


Save key isn't active

panic ?

Stop cry. Just find it.

Hmm. I don`t need this button?..
Close issue.


Remove TraitBase + SpecificationsBase

#19
Remove traitbase and specification base from traitgen generated code.
The C++ was practically already done, just needed the include removed. Copied the methods from Specification + Traitbase into the python verbatim.

Also added release note, had forgotten.


Hermes doesn't work on ubuntu18.04

When I try to use hermes binary on ubuntu18.04 it gives me the following error:
./hermes: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by ./hermes)

Version
v0.12.0
Steps to Reproduce
Just try running it on ubuntu18.04 x86_64.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Thanks for the report by the way, we will update the guide to mention the prerequisite on glibc 2.29.

It works if I build it manually.


missing CImg.h in bunnyanim.cpp

Hi there,
Thanks for sharing, I really like the brevity and clarity of the code.
I stumble on a small issue with missing header bunnyanim.cpp.
Can you point me the source please? Or should I skip it?
Thanks

Seems I resloved my problem by downloading the header from link below:
http://cimg.sourceforge.net/download.shtml

Wonder if copying the CImg.h is sufficient or should I also include plugins diirectory?
At this point

triagle example works properly
bunny example is leaking and never complets
bunnyanim complains about wood texture format:

CImg] *** CImgIOException *** [instance(0,0,0,0,(nil),non-shared)] CImg::load(): Failed to recognize format of file 'example/woodgrain.jpg'.
terminate called after throwing an instance of 'cimg_library::CImgIOException'
what():  [instance(0,0,0,0,(nil),non-shared)] CImg::load(): Failed to recognize format of file 'example/woodgrain.jpg'

Hi there

bunny example is leaking and never complets

Try editing the size of the framebuffer.  It's not leaking memory to my knowledge, but by default it generates an absolutely giant framebuffer which takes time to render.   https://github.com/Steve132/uraster/blob/master/example/bunnystatic.cpp#L126  Change that from 640<<5,480<<5 (which is like 20,000 by 15000) to just 640x480
I'm actually gonna change this in the code now.

bunnyanim complains about wood texture format:

2 possibilities: either the way you are installing CImg is wrong (I don't know how to correctly install CImg under windows, you need to read their documentation for that).
The other possibility is that you need to run bunnyanim from the root source directory so it can find example/woodgrain.jpg.  It builds in some other directory so you either need to move it or invoke it from the root source dir.

Thanks, indeed resolution was the problem.


Add option to disable searching if entry is not found

Default gopass show behavior to search for matching entries if entry is not found is great for normal use, but nightmare for scripting.
It would be great to add option to show action which will disable this behavior when needed.
My use case: need to find if entry exists in store.
BTW: I'm willing to submit PR if somebody helps me with #550 :)

Thanks for raising this issue. I understand your need. We had this discussion around gopass find before and change the default. If we the behavior of show is inconsistent we'll fix that as well.

If you redirect the output of gopass show to a file it should not display the search results: https://github.com/justwatchcom/gopass/blob/master/action/show.go#L110
But if this doesn't cover your needs feel free to open an PR.


Try not to take down half a cluster if we dont need to

OMG Yes!

will this make deploys take longer?
if so suggest we do a bit more aggressively

@mbryzek yes, this will make deploys of larger services go longer, but given load on FN - we are at a point where we may not be able to afford half of the experience or shopify or harm clusters going down, for example, and need to do it more deliberately (i.e. only 1 instance at a time)

Another idea would be to introduce a "min healthy instances" config where we explicitly define a number of instances that need to stay up at any given time, including during a deploy. thsi will then inform what we can set in the minhealthy cluster setting in ECS. if not defined in .delta, then will default to the formula in this pr

in the logic for a cluster with say 8 instances- i think we should do at least 2 at a time (maybe 3) vs 1 by 1

why don't we copy the settings from k8s here - will be good to socialize with team and they should just carry over to k8s.

@dasmb @mbryzek Where can i see the relevant k8s settings that define something similar to this?

i don't have off hand but pretty sure it is a config setting on the k8s deployment (https://kubernetes.io/docs/concepts/workloads/controllers/deployment/)


New release 2023-12-20

Read this before submitting the PR

Always create a Draft PR first
Go through sections 1-5 below, fill them in and check all the boxes
Make sure that the branch is updated; if there's an "Update branch" button at the bottom of the PR, rebase or update branch.
When all boxes are checked, information is filled in, and the branch is updated: mark as Ready For Review and tag reviewers (top right)
Once there is a submitted review, implement the suggestions (if reasonable, otherwise discuss) and request an new review.

If there is a field which you are unsure about, enter the edit mode of this description or go to the PR template; There are invisible comments providing descriptions which may be of help.
1. Description / Summary
Add a summary of the changes in this PR and the related issue.
2. Jira task / GitHub issue
Link to the github issue or add the Jira task ID here.
3. Type of change
What type of change(s) does the PR contain?
Check the relevant boxes below. For an explanation of the different sections, enter edit mode of this PR description template.

[ ] New feature

[ ] Breaking: Why / How? Add info here.
[x] Non-breaking

[ ] Bug fix
[ ] Security Alert fix

[x] Package update

[ ] Major version update

[ ] Documentation <!-- Should be checked if the PR adds or updates the CLI documentation -- anything in docs/ directory. -->
[ ] Workflow
[ ] Tests only

4. Additional information

[x] Sprintlog
[x] Blocking PRs: https://github.com/ScilifelabDataCentre/dds_cli/pull/670

[ ] Merged

[x] PR to master branch: If checked, read the release instructions

[x] I have followed steps 1-8.

5. Actions / Scans
Check the boxes when the specified checks have passed.
For information on what the different checks do and how to fix it if they're failing, enter edit mode of this description or go to the PR template.

[x] Black

[x] Pylint

[x] Prettier

[x] Yamllint

[x] Tests

[x] TestPyPI

[x] CodeQL

[x] Trivy

[x] Snyk

does it has to be updated before merging or is it ok?


[Suggestion] Remove VRC and VRCSDK dependencies

This tool will error without the VRChat SDK. This is inconvenient when using it in projects other than for VRChat as most of the functions in this tool are not specific to VRChat and the tool itself is incredibly useful even if it is tailored to VR avatar making.
This would also make pumpkin tools work in ChilloutVR related unity projects without the need to also install VRChats SDK.
Proper ChilloutVR CCK support would also be nice but generalizing the tool would be amazing to do.

I'm currently rewriting everything to work without the vrc sdk and more.
I'll eventually support CVR as well but it's gonna take a while as there's a lot to do.
If you want to see development you can check the dev-updates channel on my discord or check the dev overhaul branch here on github

From: NotAKidoS<EMAIL_ADDRESS>Sent: Thursday, November 12, 2020 2:19:08 AM
To: rurre/PumkinsAvatarTools<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [rurre/PumkinsAvatarTools] [Suggestion] Remove VRC and VRCSDK dependencies (#26)
This tool will error without the VRChat SDK. This is inconvenient when using it in projects other than for VRChat as most of the functions in this tool are not specific to VRChat and the tool itself is incredibly useful even if it is tailored to VR avatar making.
This would also make pumpkin tools work in ChilloutVR related unity projects without the need to also install VRChats SDK.
Proper ChilloutVR CCK support would also be nice but generalizing the tool would be amazing to do.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/rurre/PumkinsAvatarTools/issues/26, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD7RGWKQGC4NJBGVELCC7KTSPMSXZANCNFSM4TSTYACA.

Before that's done, I actually went ahead and removed dependencies in the WIP release of 0.9.2, which you can get on my discord!


[BACKLOG-10815] Updating samples for Pentaho Server

@bmorrise @kcruzada

Build Completed
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.


source_id registration of BCC-ESM1

Here are instructions both for registering your institution and model for CMIP6 and for raising other issues concerning CMIP6 controlled vocabularies. We can update details iteratively, until you begin writing CMIP6 data.
PLEASE DELETE THESE INSTRUCTIONS BEFORE SUBMITTING YOUR REGISTRATION/ISSUE

Registering your institution and model

Before publishing CMIP6 model output on ESGF, you must register information about your institution and  model as described in the registration guidance documents available at https://github.com/WCRP-CMIP/CMIP6_CVs/blob/master/.github/Institution_registration_template.md and https://github.com/WCRP-CMIP/CMIP6_CVs/blob/master/.github/Model_registration_template.md.

Raising other issues

Your issue "title" should begin with the name of the Controlled Vocabulary of interest and also include a terse indication of the issue (e.g., "activity_id -- add 'PMIP'").

label = BCC-ESM1
label_extended = BCC-ESM1
source_id = BCC-ESM1
institution_id = BCC
release_year = 2017
activity_participation = [AerChemMIP]
aerosol:
description = none
nominal_resolution = none
description = BCC_AGCM3 (T42; 128x 64 longitude/latitude; 26 levels; top level 2.19 hPa)
nominal_resolution = 280 km
atmosChem:
description = BCC-AGCM3-Chem
nominal_resolution = 280km
land:
description = BCC_AVIM2
nominal_resolution = 280 km
landIce:
description = none
nominal_resolution = none
ocean:
description = MOM4 ( 1/3 deg in 30S-30N, 1/3-1 deg in 30-60 N/S, and 1 deg in high latitudes; 360 x 232 longitude/latitude; 40 levels)
nominal_resolution = 30-100 km
seaIce:
description = SIS
nominal_resolution = 100 km

@bcccsm we need some additional fields to correctly register your source_id please fill in/correct the entries below:
source_id = BCC-ESM1
label = BCC-ESM 1
label_extended = BCC-ESM 1
institution_id = BCC
release_year = 2017
activity_participation = [CMIP, AerChemMIP]

aerosol:
description = none
nominal_resolution = none
atmos:
description = BCC_AGCM3_LR (T42; 128 x 64 longitude/latitude; 26 levels; top level 2.19 hPa)
nominal_resolution = 250 km !see https://github.com/WCRP-CMIP/CMIP6_CVs/blob/master/CMIP6_nominal_resolution.json
atmosChem:
description = BCC-AGCM3-Chem
nominal_resolution = 250 km
land:
description = BCC_AVIM2
nominal_resolution = 250 km
landIce:
description = none
nominal_resolution = none
ocean:
description = MOM4 (1/3 deg 30S-30N, 1/3-1 deg 30-60 N/S, and 1 deg in high latitudes; 360 x 232 longitude/latitude; 40 levels; top grid cell 0-X m) !Please enter top grid cell, see http://rawgit.com/WCRP-CMIP/CMIP6_CVs/master/src/CMIP6_source_id.html
nominal_resolution = 50 km
ocnBgchem:
description = none
nominal_resolution = none
seaIce:
description = SIS !Is there a version number to record
nominal_resolution = 100 km

@bcccsm would it be possible for the additional information (noted above with the !... leading characters) to be provided so we can finalize and register BCC-ESM1?

The nominal_resolution of BCC_AGCM3-LR is 250 km.
There is no version number for SIS>

@bcccsm we also need the X value for the ; top grid cell 0-X m in the ocean description, for the GFDL-ESM2M model that contributed to cmip5, this value is 0-10m, with values recorded at the vertical resolution center point of 5 m. I assume that the ocean and the sea-ice are on the same grid and therefore the seaIce nominal_resolution is 50 km?
MOM4p1 is normally coupled with SISp2 so if you agree, I will rename SIS to SIS2

@bcccsm I have made the changes, following the assumptions outlined above. Please review CMIP6_source_id.html and let me know if any further changes or corrections are required

The top grid cell of MOM4 used in our model is the same as GFDL-ESM2M. The value is 0-10m, with values recorded at the vertical resolution center point of 5 m.
I agree with your suggestion that SIS is changed to SIS2.

@bcccsm no problem. The values you indicate above are exactly what is found in CMIP6_source_id.html, so no changes are required


QUESTION: Why is the css of my datepicker plugin not loaded?

Hey, fairly new in using this implementation of AdminLTE and Laravel. Sorry if I misuse some terminologies.
Anyways, I've been trying to use the datepicker plugin to try and recreate the theme from the official AdminLTE example.
I have configured the config/adminlte.php file to initialize my datepicker plugin like shown
        [
            'name' => 'Date Picker',
            'active' => true,
            'files' => [
                [
                    'type' => 'css',
                    'asset' => true,
                    'location' => 'vendor/adminlte/bower_components/bootstrap-datepicker/dist/css/bootstrap-datepicker.min.css',
                ],
                [
                    'type' => 'js',
                    'asset' => true,
                    'location' => 'vendor/adminlte/bower_components/bootstrap-datepicker/dist/js/bootstrap-datepicker.min.js',
                ],
            ],
        ],

And I have put this in the @section('js') at the blade file like shown
@section('js')
<script>
    $(document).ready(function () {
      $("#calendar").datepicker();
    });
  </script>
@stop

Putting that in my blade file has at least made my datepicker plugin appeared, however the style of the datepicker plugin wasn't shown correctly, as if the css was not present.
The weirdest solution I currently found was to reference the location of the datepicker plugin again in the @section('js') like shown
@section('js')
  <script src="{{ asset('vendor/adminlte/bower_components/bootstrap-datepicker/dist/js/bootstrap-datepicker.min.js')}}"></script>

  <script>
    $(document).ready(function () {
      $("#calendar").datepicker();
    });
  </script>
@stop

While this works, as using the code snippet above has actually made the appearance of the datepicker plugin identical to the official AdminLTE example, I believe there might be a more elegant solution than this.
What should I do to resolve this issue other than the current solution that I have provided.
Help is appreciated.
Stanley Marvin.

What's the AdminLTE and Laravel version?
Anyways, since the assets location are working when you inject then in your views js section, you are probably having issues in the plugin configuration. I coudn't spot the problem with the code you posted, so double check the plugins array openings/closings...

Thanks for the reply @andcarpi .
I'm currently using jeroennoten/Laravel-AdminLTE/ v1.27.2 and Laravel 5.8
Anyways, I've found the cause of the problem and the solution for it.
The mistake I did to cause the problem was I accidentally initialized the jQuery plugin again in the config/adminlte.php..., which I obviously shouldn't have done as the jQuery plugin was already injected before the other plugins in the master.blade.php file.
The obvious solution was to simply remove the plugin initialization of the jQuery plugin in the config/adminlte.php...
Should have triple checked the plugins configuration..
Thank you again for replying.
Regards,
Stanley Marvin.


Jetbrains Rider compatibility

Hi,
I love the plugin in WebStorm.
Now, however, I'm also using Jetbrains Rider and I was wondering if you could add support for that browser as well?
Thanks!

Yes please, I would love to use it on Rider also.

Hello, I present you this 4 year old issue, #18, which tracks the advancement on the development of the plugin for each Jetbrains product.
Unfortunately, the plugin doesn't work properly in Rider without more development (I don't know what is the problem, but it seems the reformat doesn't work).
I won't have time to implement this anytime soon, but you can submit a PR anytime by following the https://github.com/dubreuia/intellij-plugin-save-actions/blob/master/CONTRIBUTING.md guide.

Duplicates #18


nan_to_num: TypeError: Wrong number of arguments

Thank you for using and contributing to CuPy!
If your issue is a request for support in using CuPy, please post it on Stack Overflow or Google Groups (en / ja).
Developer team members are monitoring questions on these channels.
If it is a bug report, please include the following information:

Conditions (you can just paste the output of python -c 'import cupy; cupy.show_config()')

CuPy version: cupy=9.4.0=py39h499daff_0 through Anaconda and channel conda-forge
OS/Platform: Arch Linux (Linux-5.14.9-arch2-1-x86_64-with-glibc2.33)
CUDA version: cudatoolkit=11.2.72=h2bc3f7f_0 through Anaconda and channel nvidia
cuDNN/NCCL version (if applicable):

cudnn=<IP_ADDRESS>=h86fa8c9_0 from channel conda-forge
nccl=<IP_ADDRESS>=hdc17891_0 from channel conda-forge

Code to reproduce

#! /usr/bin/env python3

# numpy
import numpy
np_a = numpy.array([float("NaN"), float("inf"), -float("inf")])
np_a = numpy.nan_to_num(np_a, True, 123., 456., 789.)
print(np_a)

# cupy
import cupy
cp_a = cupy.array([float("NaN"), float("inf"), -float("inf")])
cp_a = cupy.nan_to_num(cp_a, True, 123., 456., 789.)
print(cp_a)

Error messages, stack traces, or logs

Traceback (most recent call last):
  File "test.py", line 12, in <module>
    cp_a = cupy.nan_to_num(cp_a, True, 123., 456., 789.)
  File "cupy/_core/_kernel.pyx", line 1112, in cupy._core._kernel.ufunc.__call__
TypeError: Wrong number of arguments for 'cupy_nan_to_num'. It must be either 1 or 2 (with outputs), but given 5.

Thanks for reporting! Support nan, posinf, neginf has been added in #5295 which is in v10.0.0a2, so v10.0.0b3 should not have the error. I've tried with cupy/cupy:v10.0.0b3 docker image but unable to reproduce the error.

I've tried with cupy/cupy:v10.0.0b3 docker image but unable to reproduce the error.

This was my fault, should have tried your reproducer as-is :bow:

I opened a PR in #5870 that addresses this issue.


Normalize screws up  font weight

This seems to be an issue with normalize.css. Example below:

The one on the left is a <button> and the one on the right is an <a>
EDIT: This was tested in chrome on Ubuntu 16.04

Fixed in 6ebaaadef1f39e6864beccca5f655996167627e7


Failed to install

Not being able to install Version 1.2.1
--> "Failed to install" message.

Same issue, and same error here. First time installing the plugin but it is failing :(

same problem with 1.2.1

same. I am getting Error: Request failed, status 404

Hi @ollecello @Cammagno @berot3. Version 1.2.1 was currently as a draft. With no main.js file included. I've officially published version 1.2.1 with all the requisite files.
Please let me know if the issue persists. Awaiting your feedback.

Thanks for the update!
As far as your 2nd query goes, It's not really an issue related to my plugin. Why don't you make a post on the Obsidian Forum? You should find people to help you with your problem there.
Closing this issue.


Add ability for marathon provider to set maxconn values, loadbalancer algorithm, and circuit breaker expression

Initial implementation: Force both maxconn labels to be present to trigger behavior.

to address #612

Thanks a lot @jangie :)
LGTM :tada:

@emilevauge I've removed the log statement for now.
I'll be taking a stab at adding similar functionality to the other providers. Is it also your opinion that the maxconn settings should be an all-or-nothing affair? I.e., that without extractorfunc, amount should do nothing, as well as vice versa?

LGTM, but needs rebase :)

Rebased

Rebased again.

ping @containous/traefik


Keystore package does not work in a Node ESM environment

What version of fuels-ts are you using?
latest/ 0.39.0
Steps to Reproduce

Create a Node esm project
Attempt to load a fuels package or util that requires keystore
The execution fails as require is not available

Expected Behavior
This code should also work in esm style environments.
This block needs to support require for legacy support but also somehow do dynamic import for ESM:
https://github.com/FuelLabs/fuels-ts/blob/6c8ce489f2384de7def5daec009b6af20bc38523/packages/keystore/src/universal-crypto.ts#L22-L31
Actual Behavior
Crashes with error message:
keystore expects a standard Web browser or Node environment. Error: Dynamic require of "crypto" is not supported
    at file:///test-fuels/bg/node_modules/.pnpm/@fuel-ts+keystore@0.39.0/node_modules/@fuel-ts/keystore/dist/index.mjs:1:231
    at file:///test-fuels/node_modules/.pnpm/@fuel-ts+keystore@0.39.0/node_modules/@fuel-ts/keystore/src/universal-crypto.ts:25:22

I stumbled upon this issue when debugging all the CJS/ESM shenanigans at:

#827

I left some annotations with a possible fix here:
https://github.com/FuelLabs/fuels-ts/blob/735bc66495ec1a37fd11593b2fad194a11973087/packages/keystore/src/universal-crypto.ts#L28-L38


Add powerups and color changers to the generation process

I close this one and move "color changers" to the powerup list at issue #12


Iosevka overrides leads to NPM errors.

Describe the bug
I'm trying to customize Iosevka currently with:
fonts.fonts = with pkgs; [
    fira-code
    fira-code-symbols
    (iosevka.override {
      set = "custom";
      privateBuildPlan = ''
        [buildPlans.iosevka-custom]
        family = "Iosevka Custom"
        spacing = "normal"
        serifs = "sans"
        no-cv-ss = true

        [buildPlans.iosevka-custom.variants.design]
        capital-a = "curly-serifless"
              '';

    })
    noto-fonts
    noto-fonts-cjk
  ];

But this leads to:
╰─$ sudo nixos-rebuild switch                                                                                ...
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `node utility/ensure-verda-exists && verda -f verdafile.js "--jCmd=32" "ttf::iosevka-custom"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

To Reproduce
Steps to reproduce the behavior:

Use the font configuration above.
nixos-rebuild

Expected behavior
The build should succeed.
Notify maintainers

@cstrahan
@jfrankenau
@ttuegel
@babariviere
@rileyinman
@AluisioASG

Metadata

system: "x86_64-linux"
host os: Linux 5.12.10, NixOS, 21.05.1075.246502ae2d5 (Okapi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.12
channels(root): "nixos-21.05.1075.246502ae2d5, oldpkgs-19.03.173676.d1dff0bcd9f"
channels(ashesh): "home-manager-20.09, nixos-20.09.3975.17b101e29df"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
Iosevka

I wish I could be of more help, but I don't really understand how iosevka is packaged with node/npm.

The names of many options have changed. You will probably need to re-create your custom configuration with the customizer.

The names of many options have changed. You will probably need to re-create your custom configuration with the customizer.

@ttuegel I did create the options via the customizer at the time of reporting the issue, but I'll retry.

I ran into this same issue.
Updating my configuration did the trick! Two of my settings were renamed:

variants.design.d from toothless-corner to toothless-corner-serifless
variants.design.asterisk from low to hex-low (well technically there's just more options for that grapheme now)

I'm extremely annoyed with how frequently the iosevka developers break their build process but I guess there's not really anything we can do about that in NixOS.


Watermelon Extension vs.code

The watermelon extension has had an issue since first installed. It can't create content. I'm logged in with GitHub and have the correct email address. I get repeated messages to sign into the Watermelon extension, and I am, but it persists. I am at a loss, too, cause of my problem. A message to my account on vs. code says to sign in.

Hello!
I will look at this shortly, but first I need to know what version of Watermelon do you have?

Hi, thanks I'm running Watermelon version 2.0.7. Looking at the  Changelog I installed Watermelon about a month ago. I feel I'm missing something here

Will close as stale