Incorporate all node weights in additive blending

Objective
In the existing implementation, additive blending effectively treats the node with least index specially by basically forcing its weight to be 1.0 regardless of what its computed weight would be (based on the weights in the AnimationGraph and AnimationPlayer).
Arguably this makes some amount of sense, because the "base" animation is often one which was not authored to be used additively, meaning that its sampled values are interpreted absolutely rather than as deltas. However, this also leads to strange behavior with respect to animation masks: if the "base" animation is masked out on some target, then the next node is treated as the "base" animation, despite the fact that it would normally be interpreted additively, and the weight of that animation is thrown away as a result.
This is all kind of weird and revolves around special treatment (if the behavior is even really intentional in the first place). From a mathematical standpoint, there is nothing special about how the "base" animation must be treated other than having a weight of 1.0 under an Add node, which is something that the user can do without relying on some bizarre corner-case behavior of the animation system — this is the only present situation under which weights are discarded.
This PR changes this behavior so that the weight of every node is incorporated. In other words, for an animation graph that looks like this:
┌───────────────┐
│Base clip      ┼──┐
│      0.5      │  │
└───────────────┘  │
┌───────────────┐  │  ┌───────────────┐     ┌────┐
│Additive clip 1┼──┼─►┤Additive blend ┼────►│Root│
│      0.1      │  │  │      1.0      │     └────┘
└───────────────┘  │  └───────────────┘
┌───────────────┐  │
│Additive clip 2┼──┘
│      0.2      │
└───────────────┘

Previously, the result would have been
base_clip + 0.1 * additive_clip_1 + 0.2 * additive_clip_2

whereas now it would be
0.5 * base_clip + 0.1 * additive_clip_1 + 0.2 * additive_clip_2

and in the scenario where base_clip is masked out:
additive_clip_1 + 0.2 * additive_clip_2

vs.
0.1 * additive_clip_1 + 0.2 * additive_clip_2

Solution
For background, the way that the additive blending procedure works is something like this:

During graph traversal, the node values and weights of the children are pushed onto the evaluator stack. The traversal order guarantees that the item with least node index will be on top.
Once we reach the Add node itself, we start popping off the stack and into the evaluator's blend_register, which is an accumulator holding up to one weight-value pair:

If the blend_register is empty, it is filled using data from the top of the stack.
Otherwise, the blend_register is combined with data popped from the stack and updated.

In the example above, the additive blending steps would look like this (with the pre-existing implementation):

The blend_register is empty, so we pop (base_clip, 0.5) from the top of the stack and put it in. Now the value of the blend_register is (base_clip, 0.5).
The blend_register is non-empty: we pop (additive_clip_1, 0.1) from the top of the stack and combine it additively with the value in the blend_register, forming (base_clip + 0.1 * additive_clip_1, 0.6) in the blend_register (the carried weight value goes unused).
The blend_register is non-empty: we pop (additive_clip_2, 0.2) from the top of the stack and combine it additively with the value in the blend_register, forming (base_clip + 0.1 * additive_clip_1 + 0.2 * additive_clip_2, 0.8) in the blend_register.

The solution in this PR changes step 1: the base_clip is multiplied by its weight as it is added to the blend_register in the first place, yielding 0.5 * base_clip + 0.1 * additive_clip_1 + 0.2 * additive_clip_2 as the final result.
Note for reviewers
It might be tempting to look at the code, which contains a segment that looks like this:
if additive {
    current_value = A::blend(
        [
            BlendInput {
                weight: 1.0, // <--
                value: current_value,
                additive: true,
            },
            BlendInput {
                weight: weight_to_blend,
                value: value_to_blend,
                additive: true,
            },
        ]
        .into_iter(),
    );
}

and conclude that the explicit value of 1.0 is responsible for overwriting the weight of the base animation. This is incorrect.
Rather, this additive blend has to be written this way because it is multiplying the existing value in the blend register by 1 (i.e. not doing anything) before adding the next value to it. Changing this to another quantity (e.g. the existing weight) would cause the value in the blend register to be spuriously multiplied down.
Testing
Tested on animation_masks example. Checked morph_weights example as well.
Migration Guide
I will write a migration guide later if this change is not included in 0.15.

@pcwalton @james7132 @mockersf I'm inclined to include this in 0.15, to avoid shipping weird behavior.


[Store]Error when generating an access token

Description:
Error when trying to generate the access token from UI
Suggested Labels:
Bug
Suggested Assignees:
N/A
Affected Product Version:
APIM 3.0.0 Beta
OS, DB, other environment details and versions:
macOS 10.15 (19A583)
Error log
Please refer this error log

Faced the same issue on APIM 3.0.0-RC3 as well.

Cannot reproduce this issue in the latest pack. Please reopen if you encounter this issue again.


Day46: 'Kth Smallest Element in a BST' solution

Kth Smallest Element in a BST
https://leetcode.com/problems/kth-smallest-element-in-a-bst/
Given the root of a binary search tree, and an integer k, return the kth smallest value (1-indexed) of all the values of the nodes in the tree.

Example 1:
Input: root = [3,1,4,null,2], k = 1
Output: 1
Example 2:
Input: root = [5,3,6,2,4,null,null,1], k = 3
Output: 3


Reload pocketbase on JavaScript code changes

correctly handle JavaScript hooks and migrations in development environment

@Bewalticus , I believe pocketbase already restarts when pb_hooks/*.pb.{j|t}s files change. Is that not true? That's been my experience.

@jkdoshi, thank you for the advice. I did not recognize this behavior yet but thought the files were missing in the restart configuration somehow.
Will close the PR.


Create wrong3.php.inc

should be the main one

@WinterSilence Thank you :+1:
@samsonasik Could you look into fix for this?

I will try.

@WinterSilence I cherry-picked your commit at PR https://github.com/symplify/symplify/pull/3046


Update Prometheus Operator in kube-prometheus to v0.29.0

Just realized per https://github.com/coreos/prometheus-operator/issues/1757#issuecomment-411034013  that kube-prometheus is not versioned.


about @ in this line

https://github.com/AIgen/df-posthoc-calibration/blob/109da93c1487cb38ee51fcac47088cdd29854999/assessment.py#L187
Hello, when i read this code, its very nice and give me a great help.  And i have a question, in this line,  whats mean about @.
Thanks!

If I am not mistaken, that indicates a dot product in numpy ;)


parent added

@CodiumAI-Agent /review

PR Reviewer Guide 🔍
Here are some key observations to aid the review process:

⏱ Estimated effort to review: 3 🔵🔵🔵⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review
Navigation LogicThe navigation logic in the 'handleNext' function has been modified to include a condition for navigating to the 'Parent' component. Ensure that this logic aligns with the intended user flow and that the condition is robust enough to handle various user inputs and states.
Form ValidationThe form submission in the 'Parent' component does not include any validation logic beyond the 'required' attribute in HTML. Consider implementing more comprehensive validation to handle different types of input and to provide user feedback before submitting the form.
Inline StylesThe 'Landingpage' component uses extensive inline styles. This approach can make the code harder to maintain and less performant than using CSS classes. Consider refactoring these inline styles into a dedicated CSS file or using CSS modules for better performance and maintainability.


refactor: fund test migrate, closes #4187

Try out this version of the Leather — download extension builds.

See title

hm, very weird, fund tests are passing locally

What's the providers.json? These come from the wallet-config right? 🤔

What's the providers.json? These come from the wallet-config right? 🤔

Yes, you're right, sorry. I meant that the activeFiatProviders come from wallet-config.json but I think somtimes it doesn't get loaded correctly and then the tests fail as a result

hmm, feel like last commit fixed it


Fix: fix preview for rich text

Draft text bodies weren't showing when use Preview was turned on as the condition was the wrong way round

Do we need "fix: fix"? 🤔


get color from mtl file

How to obtain colors of vertices from the MTL file associated with the OBJ file?

.obj spec does not support vertex color.
PR for vertex color through extension is always welcome!
https://blender.stackexchange.com/questions/31997/how-can-i-get-vertex-painted-obj-files-to-import-into-blender

I see, thank you.
Does OBJLoader offer support for finding color through the texture maps specified in the MTL file?

Does OBJLoader offer support for finding color through the texture maps specified in the MTL file?

This is not a job of obj loader.
You can write your own tool/application to do that. Writing example code and sending a PR are always welcome!

You are better to study 3D graphics programming firstly, e.g. with
http://graphicscodex.com/

Hello!
It seems that tinyobjloader now supports vertex color loading, however when trying to upload the Cornell Box (from your reppo, colors show up on blender) in my program, all vertex colors returned by tinyobj are set at (1,1,1). My code for importing the file is basically a copy-paste of your example file, so I have trouble understanding why I have these colors. Is this the expected behavior?

Please attach minimal & reproducable .obj/.mtl


Thumbnails sometimes do not honour image rotation and appear sideways

not sure why, maybe there are multiple ways to implement rotation and libraries only support some?
or maybe phones always save photos the right way up which i why this is not a more prevalent issue?

Some phones save the photo the way the sensor is oriented, and then use a JPEG field to indicate it should be displayed rotated. That might be what's causing this?

yea that is what that last commit was supposed to add, but seems to not work for all jpeg files
yea standards etc
my next idea is to add support for reading exif thumbs

further debugging to figure out why it is only sometimes working: it depends on which JPEG decoder actually gets used.

if it decodes using com.sun.imageio.plugins.jpeg.JPEGImageReader (first choise) then exif rotation is ignored since it hard-codes Dimension/ImageOrientation="normal.
if it fails to decode with the JVM provided decoder[1], it falls back to com.twelvemonkeys.imageio.plugins.jpeg.JPEGImageReader which does populated ImageOrientation with the actual value.

tested with JVM 11, not checked if later JVMs have an updated JPEGImageReader.
note i set this decoder ordering specifically cos the jvm-internal one seems to be quite a bit faster and i did not realise it was incomplete like this.
[1] eg fails with javax.imageio.IIOException: JFIF APP0 must be first marker after SOI

nope, still hard-codes no rotation: https://github.com/openjdk/jdk/blob/12028000db2ef3b1c784af119c495aa3ef9590cf/src/java.desktop/share/classes/com/sun/imageio/plugins/jpeg/JPEGMetadata.java#L969

next idea was to read thumbs from headers to avoid needing to regenerate and hope those were orientated correctly... except that the com.sun decoder fails to read those also.  it seems in only supports JFIF where as the twelvemonkeys also supports JFXX and Exif, and seems most thumbs are in Exif...
sigh.  this is coming down to a speed vs completeness trade off...
i could set it to always use twelvemonkeys decoder and the problem would go away, but it would be much slower.
i should add metrics to measure how much slower...
a possible mitigation is to make use of thumbs in headers, though these are inconsistent in resolution so would need to come up with some selection criteria like if its close to 200x200 then use as is, if much bigger then shrink it down.
though either way the thumb would get recompressed since there is no way to read the compressed thumb bytes directly.

added a metric: 948ec4ae830e9ced12d0f39c051821210e688627
possibly unsurprisingly, performance very much varies depending on the mix of files being processed
without thinking i through all my phone phones from this year at it and it was Slower when using what i had thought of as the "optimised" path because the "try with the faster decoder then fallback" actually takes longer if it Always falls back
i should have seen that coming really
maybe just delete this "optimisation" and stop over thinking this?
https://github.com/haku/mediatoad/blob/948ec4ae830e9ced12d0f39c051821210e688627/src/main/java/com/vaguehope/dlnatoad/util/ImageResizer.java#L53-L69

TODO for another day: read exif thumbs if present and dimensions are usable.
possible logic:
s = max(width, height)

if thumb with 100 < s < 300 px: use as is
if all thumbs have s < 100 px: full generate
else: resize largest thumb to 200x200 px

or lazier logic would be always use embedded thumb no matter what size it is, not sure about failure modes
i have vague memories of photoshop or similar sometimes embedding huge thumbs in tiny images etc
i guess i should somehow survey what is typical for embedded thumbs...


Support Material Web Components

Hello 👋
One of the key reasons developers turn to building websites with Kotlin is to create WebViews for their Android apps, enabling them to make certain parts of the app dynamic. However, many developers lack experience with HTML styling. Additionally, frameworks like Bootstrap don’t align closely with Android’s component design. Supporting Material Web, in my opinion, can significantly bridge this gap and address these challenges effectively.

It is on my todo list for some time, but there is a problem with Material libs. The official Material3 library is not complete and is not maintained anymore. If you read this thread, you will find a lot of people are angry and disappointed by Google decision. There are some alternatives, but they are not perfect. I'm thinking about BeerCSS library, but I'm also closely watching ambitious compose-material3-html project (unfortunately it is still far from complete).

It is on my todo list for some time, but there is a problem with Material libs. The official Material3 library is not complete and is not maintained anymore. If you read this thread, you will find a lot of people are angry and disappointed by Google decision. There are some alternatives, but they are not perfect. I'm thinking about BeerCSS library, but I'm also closely watching ambitious compose-material3-html project (unfortunately it is still far from complete).

Alright, so based on what you mentioned, having Material Components would either be a long process, or we’d have to reinvent the wheel or rely on unofficial projects created by others.

Alright, so based on what you mentioned, having Material Components would either be a long process, or we’d have to reinvent the wheel or rely on unofficial projects created by others.

I'm afraid that's true. But relying on community projects isn't a bad thing - Kilua already uses many open source projects for various components.


Add NAKKKIM

Add NAKKKIM to UA domains
https://en.wikipedia.org/wiki/National_Academy_of_Government_Managerial_Staff_of_Culture_and_Arts
https://nakkkim.edu.ua/
@philipto
@nataershova

@t-tiana Please provide us with an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course. I did not find any course at the website of the Academy that might include some software development tasks.
Thank you!

@t-tiana You might miss my previous comment. Please react.

@philipto You are right and the Academy does not offer a matching course, but the It department would be happy to get the license to develop a modern website and the IT infrastructure for the academy. We are planning to develop an internal software product for the art expertise center. https://nakkkim.edu.ua/instituti/instituti-institut-praktichnoji-kulturologiji-ta-art-menedzhmentu/tsentr-ekspertizi-kulturnikh-tsinnostej

@philipto You are right and the Academy does not offer a matching course, but the IT department would be happy to get the license to develop a modern website and the IT infrastructure for the Academy. We are planning to develop an internal software product for the Art Expertise Center. https://nakkkim.edu.ua/instituti/instituti-institut-praktichnoji-kulturologiji-ta-art-menedzhmentu/tsentr-ekspertizi-kulturnikh-tsinnostej

@t-tiana Thank you for the explanation! Educational licenses are for educational purpose only. Please send an email to<EMAIL_ADDRESS>and discuss the matter with them. You will at least get an academic discount, and they probably will be able to offer more to support you.
I have to close the request.
Thank you for your work and using JetBrains tools!


Fix small typo in ssh-cert-bastion.md

This adds a missing space, the sentence as a whole could be worded differently but adding this space should be good for now 🙂

Howdy @frusznyak! Thanks for the typo fix. I'm going to go ahead and merge in since it doesn't require any additional feedback on your end.


update-review-documentation

update-review-date

Sorry Tony @AntonyBishop - I made a mistake with my link - would it be possible to review again please?


[Usersnap] Different Voting Power display on top of screen and in card

Add a comment: Maybe my interpretation/lack of understanding, but confused as to why voting power is different in the top right wallet balance style field, vs the balance available for delegation in the bottom left. Apologies if I am not understanding (highly likely)
Screen size: 1576 x 1268
Browser: Chrome 122 (Linux)
Referer: SanchoNet Govtool
Open #37 in Usersnap

This is still pertinent as a user issue. It's very confusing to see two different 'voting power' displays. Which one is correct?
I think we need to re-think how we display Voting Power, i.e. we should display the 'as-it-is-now' figure, possibly along with the figure at epoch boundaries. It doesn't make sense to say delegate 100k ADA to yourself, and have a zero voting power showing at the top of the screen. Especially if when you vote for something, you ostensibly apply the entire 100k to the Vote.

@m-i-k-e On which environment do you observe and experience this? Looking at dev and staging envs, the top right button is not displayed anymore at all. What am I missing?


Added basic action error handling

no issue

Stores external action name and error in new @action property available in handlebar template
Inside of the .hbs template can be used as<EMAIL_ADDRESS>to check for errors and<EMAIL_ADDRESS>to check for action names

can be reopened once this is a priority again


kind cluster: context deadline exceeded

Describe the bug
I am running k8s kind cluster on macos arm64. When running:
go test -v -tags kubernetes -run TestKubernetesHelloWorldExample
It do apply the pods, but when testing for http://<IP_ADDRESS>:5000 it runs into problems, see terminal output below.
Do you use client-go to access http://<IP_ADDRESS>:5000 or do I need to create an ingress to access from the host?
Nice to have

[ ] Terminal output
TestKubernetesHelloWorldExample 2024-04-16T19:26:01+02:00 retry.go:103: HTTP GET to URL http://<IP_ADDRESS>:5000 returned an error: Get "http://<IP_ADDRESS>:5000": context deadline exceeded (Client.Timeout exceeded while awaiting headers). Sleeping for 3s and will try again.

Versions
Terratest version: v0.46.13
MacOS
go version
go version go1.21.7 darwin/arm64
Same On Windows 11 WSL
go version go1.21.3 linux/amd64
Additional context
Add any other context about the problem here.

Hi,
to access Kubernetes is used go client, it may be that the request reaches the cluster but hangs for some reason(API problem, network policies) - I would say to try to use kubectl cli to check if the cluster is reachable and can create pods.

Hi @denis256
Thanks, I can create pods and use kubectl for usual stuff. I can also see:
kubectl get pods
NAME                                     READY   STATUS    RESTARTS       AGE
hello-world-deployment-944cbd46d-2jpzz   1/1     Running   0              18h

I can also run my own client-go up against the cluster.

Hm, it may be a problem with accessing exposed services - it is responding when accessing through services?

That is the problem, I think. The service is running in a kind cluster, using docker on macos. I can see the services, but unable to "curl <IP_ADDRESS>:5000" to the service from host machine:
kubectl get svc
NAME                  TYPE           CLUSTER-IP      EXTERNAL-IP   PORT(S)          AGE
hello-world-service   LoadBalancer   <IP_ADDRESS>     <IP_ADDRESS>    5000:31173/TCP   25h
kubernetes            ClusterIP      <IP_ADDRESS>       <none>        443/TCP          27d

curl <IP_ADDRESS>:5000
curl: (7) Failed to connect to <IP_ADDRESS> port 5000 after 8 ms: Couldn't connect to server

This is known issue running on non linux environment, but using client-go should be able to to access the services from within the same network, I think?

On macOS and Windows, docker does not expose the docker network to the host. Because of this limitation, containers (including kind nodes) are only reachable from the host via port-forwards

https://kind.sigs.k8s.io/docs/user/loadbalancer/

I wish I could say any it is not working, but it does.

Maybe a suggestion could be to use https://github.com/curl/curl-docker

Note: Running on Apple MBP M1

I was getting the same error but managed to figure it out after looking closer. It's actually an issue with the image, training/webapp:latest. execing into the pod, there were no Flask processes running and when trying to manually run it, I received Segmentation fault (core dumped). Possible reasons? Trying to pull the image yields:
"Suggest the author of docker.io/training/webapp:latest to upgrade the image to the OCI Format or Docker Image manifest v2, schema 2."
Changing the image and corresponding port (in the test, too) will get the test to pass.


⚠ Bofrost.de has degraded performance

In ce6d911, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1628 ms

Resolved: Bofrost.de performance has improved in ccae527 after 7 minutes.


Release: v0.7.5

cc @lvwerra

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


(Bug) Utility Belt compatibility with ReservedItemSlot

Hello!
I have encountered a bug with the compatibility between the two mods. If you have items in the reserved slots when you equip the utility belt, the items get moved to the new slots added by the belt, while the HUD still shows them in the reserved slots.
While I am not familiar with Lethal Company modding, I took a look at the code for the belt, and the problem might be in the function where the extra slots get added. I think the items might go to the wrong index. (Again, it's a guess, I'm probably wrong)
https://github.com/EvaisaDev/LethalThings/blob/7ed9e8fcd77105686a3efe655714a507123dabc2/LethalThings/MonoBehaviours/PouchyBelt.cs#L333
I love your work with the mod, so I hope I can help even just a little bit, since mod compatibility was explicitly mentioned.

Do you know how I could contact the ReservedItemSlot dev? There is no GitHub linked to the mod, and I couldn't find a post for it in the modding Discord server.
P.S.:
I also noticed that if you have everything in the right place (first equipping the belt, then the reserved items) and drop the utility belt, the reserved items also get dropped.

A lot of updates came out in the last few days, both to the mods and to the game, and somewhere among those updates all bugs mentioned here have been resolved. As far as I could see, the two mods now work flawlessly together.


Adding public API test coverage for Aspire.Hosting.Python

Initial issues #2649
Issues Check List: Validate arguments of public methods
Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run

/azp run

/azp run

@Zombach thanks once again for all your work on some of these PRs. Appreciate your efforts.


Dark mode

When I switch between dark and light mode, there is a 500 HTTP error and the CSS stop working (Android - Chrome)

Fixed in 80c35a61384eb45d38806ce77aa41f05b2f0622c

It works perfect by now, good job !

(the dark mode stay better according ton me 😅 but now it works !)


The "I navigated to the (.+) page" page navigation step should use the PageNavigationAttribute Url value and not UrlTemplate since no parameters were specified

Hi.
The "I navigated to the (.+) page" page navigation step should use the PageNavigationAttribute Url value and not UrlTemplate since no parameters were specified.
For example, given the page:
[PageNavigation("/Account/Login", UrlTemplate = "/Account/Login?ReturnUrl={ReturnUrl}")]
public class LoginPage
{
}

When the following step is executed
Given I navigated to the Login page

Then I expected the step to navigate to the Url "/Account/Login".
Instead, it navigates to the Url "/Account/Login?ReturnUrl={ReturnUrl}" which is unexpected.
Thank you.

I think you might have the template wrong here, it should be /Account/Login?ReturnUrl={0} and then a parameter in the given step. Are you trying to specify the return URL dynamically or do something else?

I'm basically trying to re-use a page for the following steps that appear in different scenarios:

Given I navigated to the Login page

Given I navigated to the Login page with parameters
    | ReturnUrl             |
    | /WebApplication/Users |

In the case where the page is defined as
[PageNavigation("/Account/Login", UrlTemplate = "/Account/Login?ReturnUrl={ReturnUrl}")]
public class LoginPage
{
}

Then the step from scenario 2 works as expected. The UrlTemplate is correct as SpecBind uses RegEx.Replace and matches the parameter keys with regex group names to replace values.
Another example of the UrlTemplate can be found in https://github.com/dpiessens/specbind/wiki/Page-Navigation-Binding

A sample attribute value for the URL (http://mysite.com/products/1) would be "/products/{Id}".

The step from scenario 1 also uses the path specified in the UrlTemplate but without replacements so I get a 404 error after logging into my web application because the web application tries to redirect to user to the literal url "{ReturnUrl}" which doesn't exist.
Currently I have to work-around this by creating two similar classes:
    [PageNavigation("/Account/Login")]
    public class LoginPage : BasePage
    {
    }

    [PageNavigation("/Account/Login", UrlTemplate = "/Account/Login?ReturnUrl={ReturnUrl}")]
    public class LoginWithReturnUrlPage : LoginPage
    {
    }

And use the following steps instead
1.
Given I navigated to the Login page

Given I navigated to the Login with return url page with parameters
    | ReturnUrl             |
    | /WebApplication/Users |


Combined contribute_to_qiskit.rst with contributing.md

Summary
This PR copies over relevant information in contribute_to_qiskit.rst into the CONTRIBUTING.md, meaning we can safely remove https://qiskit.org/documentation/contributing_to_qiskit.html when the time comes.
This PR also updates CONTRIBUTING.md to no longer use the term "terra"
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
9 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.002%) to 86.954%

Files with Coverage Reduction
New Missed Lines
%

qiskit/pulse/library/waveform.py
3
93.75%

crates/qasm2/src/lex.rs
6
91.92%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
74358

Relevant Lines:
85514

💛  - Coveralls

Bump @Qiskit/terra-core

Apologies - I misremembered: they were your concerns, not Eric's.

It's only 1 week until we get the redirects going and I highly doubt we'll need to do any docs deployments at this later stage, so I'm happy to take that risk and deploy this now :)


Use built-in network task metrics from __NSCFLocalSessionTask

When possible, the internal __NSCFLocalSessionTask task metrics should be used for network samples so that data is as precise as possible.

Before this is implemented, need to find all cases where the delegate is causing this (e.g. #41)


Preview build

Preview build 17718? do you mean 17713?

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

ID: ce750351-c450-13d2-db04-3df88a4f4a9c
Version Independent ID: de9bf9a8-c50f-52ef-6d92-e0ad69d4450e
Content: New Microsoft Edge Group Policies and MDM settings
Content Source: browsers/edge/new-policies.md
Product: edge
GitHub Login: @shortpatti
Microsoft Alias: pashort

Hi @spallison, the preview build number has been updated to 17713+and will appear in the documentation later this afternoon.

+1


Phonegap: Can not find script file  "C:\res\windows\zip.js"

When I try and enter

phonegap build android
into the cmd I just get back the error 'Can not find script file "C:\res\windows\zip.js"'
Any help is appreciated
Thanks

Same issue here - Tried on Win7 and Win8.1, my issue is over on the node build repo:
https://github.com/phonegap/node-phonegap-build/issues/61

Same issue here on Windows 8.1.

Same issue here on Windows 8.1 !!!

Moving to the Backlog. I'll try to reproduce this issue this week!

I could not reproduce this on Windows 8.1 using<EMAIL_ADDRESS>Without the Android SDK installed I get this, which provides a clear error message:
C:\Users\labuser\src>phonegap create test

You have been opted out of telemetry. To change this, run: cordova telemetry on.

Creating a new cordova project.

Using cordova-fetch for phonegap-template-hello-world

C:\Users\labuser\src>cd test

C:\Users\labuser\src\test>phonegap build android
[phonegap] executing 'cordova platform add --save android' ...
[phonegap] completed 'cordova platform add --save android'
[phonegap] executing 'cordova build android ' ...
Error: Failed to find 'ANDROID_HOME' environment variable. Try setting setting i
t manually.
Failed to find 'android' command in your 'PATH'. Try update your 'PATH' to inclu
de path to valid SDK directory.

With the Android SDK installed and the PATH and ANDROID_HOME environment variables set up properly, I get this (note I had to specifically install a newer cordova-android to support the latest Android SDK):
C:\Users\labuser\src>phonegap create test

You have been opted out of telemetry. To change this, run: cordova telemetry on.

Creating a new cordova project.

Using cordova-fetch for phonegap-template-hello-world

C:\Users\labuser\src>cd test

C:\Users\labuser\src\test>phonegap platform add<EMAIL_ADDRESS>Adding android project...

Creating Cordova project for the Android platform:

        Path: platforms\android

        Package: com.phonegap.helloworld

        Name: helloworld

        Activity: MainActivity

        Android target: android-25

Subproject Path: CordovaLib
Android project created with<EMAIL_ADDRESS>
Installing "cordova-plugin-battery-status" for android

Installing "cordova-plugin-camera" for android

Installing "cordova-plugin-compat" for android

Installing "cordova-plugin-console" for android
Installing "cordova-plugin-contacts" for android

Installing "cordova-plugin-device" for android

Installing "cordova-plugin-device-motion" for android

Installing "cordova-plugin-device-orientation" for android

Installing "cordova-plugin-dialogs" for android

Installing "cordova-plugin-file" for android

Dependent plugin "cordova-plugin-compat" already installed on android.

The Android Persistent storage location now defaults to "Internal". Please check
 this plugin's README to see if your application needs any changes in its config
.xml.

If this is a new application no changes are required.

If this is an update to an existing application that did not specify an "Android
PersistentFileLocation" you may need to add:

      "<preference name="AndroidPersistentFileLocation" value="Compatibility" />
"
to config.xml in order for the application to find previously stored files.

Installing "cordova-plugin-file-transfer" for android

Dependent plugin "cordova-plugin-file" already installed on android.

Installing "cordova-plugin-geolocation" for android

Installing "cordova-plugin-globalization" for android

Installing "cordova-plugin-inappbrowser" for android

Installing "cordova-plugin-media" for android

Dependent plugin "cordova-plugin-file" already installed on android.

Installing "cordova-plugin-media-capture" for android

Dependent plugin "cordova-plugin-file" already installed on android.

Installing "cordova-plugin-network-information" for android

Installing "cordova-plugin-splashscreen" for android

Installing "cordova-plugin-statusbar" for android

Installing "cordova-plugin-vibration" for android

Installing "cordova-plugin-whitelist" for android

               This plugin is only applicable for versions of cordova-android gr
eater than 4.0. If you have a previous platform version, you do *not* need this
plugin since the whitelist will be built in.

C:\Users\labuser\src\test>phonegap build android
[phonegap] executing 'cordova build android ' ...
[phonegap] completed 'cordova build android --no-telemetry'

Closing this issue. If someone is still experiencing this issue, please feel free to re-open and provide more details, like node/npm version, phonegap-cli version, state of Android SDK install, whether or not you have the ANDROID_HOME environment variable set correctly, and any and all command line output, please.


Event for failed artist search in the consignments flow

Consignments business team wants to know who users are attempting to submit artworks for outside of our P1/P2 list; this event will provide a way to query for those search bar inputs. Example of where this happens on web:

:rocket: PR was released in v2.17.0 :rocket:


Copying files from RHEL8 VM to Windows Hyper-V host works, but host to VM does not

After running the script on a newly installed RHEL 8.6 VM under Hyper-V on Windows 10/11 and using PowerShell to enable ESM on the VM, Hyper-V Enhanced Session mode is active and working as expected for:

Resolution selection & scaling
Drive mapping of C: from host
Copy/paste of text either direction between host & VM
Copy-VMFile powershell command from host to VM works
Right-click copy of a file from VM and right-click paste that file into host

However when I try to right-click copy a file from the Windows host and right-click paste the file into the RHEL8.6 VM, "paste" is greyed out in the context menu and it will not work.
I'm curious if anyone else is experiencing the same issue or able to reproduce it.  Thanks!
I had Hyper-V Enhanced Session Mode with copy/paste of files either direction working on RHEL7.9, haven't been able to make this one specific part work on RHEL8 so far.

I just tested a clean install of CentOS 8 Stream fully updated & ran script + instructions to enable Hyper-V Enhanced Session Mode there.... got the exact same results.  All ESM features work normally except Copy a file from the Windows host and attempting to Paste into a folder in the CentOS8 VM, the "paste" option is greyed out just like on RHEL8.


Virtual Simulator (real-time "mocks" accessible from UI)

From Issue-20221007-1 (this was also raised recently internally):

Please do consider a virtual device for development. Would be much appreciated for development outside laboratory environment.

I would start with a virtual WP-785X-ILP.
I haven't thought this through, but I'm tentatively picturing a "Virtual Spectrometer" widget, which might offer a "Model" pull-down (WP-785X-F13-ILP).  It would be cool if it also included a "Sample" pull-down (cyclohexane, polystyrene, xenon, argon etc) which provided a basic simulation of what the spectra would look like with that excitation, spectral range, FWHM, integration time, and noise characteristics.
Doing a "proper job" on this would take awhile — this has a lot of interesting corners.
The real nest of snakes would be if we allowed "expert mode" to specify an arbitrary spectral range, excitation etc.  It would be very easy to provide a simulation of a virtual spectrometer that isn't actually possible to make, at least with available detectors, COTS optics etc.  Let's leave that on the backburner for now, and start by allowing selection of a few standard models we know well, and then cautiously grow out from there.

What are your thoughts on exposing the current test specs through the GUI using the new factory page? I'm thinking a button on that page to add the test spec and it presents a file dialog and the user selects the right folder. It connects and then there is a remove button as well. This means, to start, only one virtual spectrometer at a time can be connected. There can be some defaults that are shipped with Enlighten and placed in the EnlightenSpectra folder.

I think it would be a good start to let us test the functionality internally.  Ultimately, when the feature seems to be working well and doing what we think users would like, then I might add a "✅ Virtual Spectrometer" to the current "Spectrometer" widget (below "Lock," "Hide Others" etc), which if checked exposes a comboBox with all the installed virtual spectrometers.


Wrong requirement versions

numpy fixed 1.7.1 causes:
RuntimeError: module compiled against API version 9 but this version of numpy is 7
Using django-chartit causes:
'chartit' is not a valid tag library: ImportError raised loading chartit.templatetags.chartit: cannot import name simplejson. This is a known issue of django-chartit, which has been updated for 1.5 year.

The requirement versions are running properly on test/development boxes.  Are you running in a clean virtualenv with no site-packages enabled?

I have site-packages and it failed on my mac.

https://github.com/daaaarcy/pytrader/commit/26c33f0edfc69fc38b90ae68750f56cbb5635eff

Thanks for the additional information, to confirm, your virtualenv is clean and not using the system-wide site packages?  OSX tends to be a bit weirder with the way it handles packages, so ensuring that your local virtualenv is clean is the best option usually.

PR #58

Hi @Snipa22 , I'm not using vitualenv, so i guess there's a chance i'm using system-wide site packages. What's the best approach?

@daaaarcy You'll want to get your virtualenv package, if you've got pip available, do pip install virtualenv, then you can generate a new virtualenv.
A quick guide appears to be available at: https://hackercodex.com/guide/python-development-environment-on-mac-osx/ for usage of virtualenvs on OSX.
I'm unsure if your system will have all of the requirements for numpy/scipy/etc as they tend to require a number of system packages for C extensions, and I'm not sure how you'd aquire them for OSX.  Unfortunately, I don't have an OSX based machine available to provide information on the best way to build the app on OSX at this time.  You may want to consider developing/running in a docker container, as this will also allow you to cleanly run postgres/etc.

@Snipa22 I'll give it a go. Thanks.

Let us know where you land darcy.  need to decide what to do with https://github.com/owocki/pytrader/pull/58

Still getting the same error about numpy using virtualenv:

RuntimeError: module compiled against API version 9 but this version of numpy is 7
Traceback (most recent call last):
File "manage.py", line 10, in
execute_from_command_line(sys.argv)
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/django/core/management/init.py", line 338, in execute_from_command_line
utility.execute()
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/django/core/management/init.py", line 312, in execute
django.setup()
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/django/init.py", line 18, in setup
apps.populate(settings.INSTALLED_APPS)
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/django/apps/registry.py", line 108, in populate
app_config.import_models(all_models)
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/django/apps/config.py", line 198, in import_models
self.models_module = import_module(models_module_name)
File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/importlib/init.py", line 37, in import_module
import(name)
File "/Users/darcy/Projects/pytrader/history/models.py", line 4, in
from pybrain.datasets import SupervisedDataSet
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/pybrain/init.py", line 1, in
from structure.init import *
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/pybrain/structure/init.py", line 2, in
from modules.init import *
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/pybrain/structure/modules/init.py", line 2, in
from gate import GateLayer, DoubleGateLayer, MultiplicationLayer, SwitchLayer
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/pybrain/structure/modules/gate.py", line 11, in
from pybrain.tools.functions import sigmoid, sigmoidPrime
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/pybrain/tools/functions.py", line 4, in
from scipy.linalg import inv, det, svd
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/scipy/linalg/init.py", line 157, in
from .misc import *
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/scipy/linalg/misc.py", line 5, in
from . import blas
File "/Users/darcy/Virtualenvs/pytrader/lib/python2.7/site-packages/scipy/linalg/blas.py", line 131, in
from scipy.linalg import _fblas
ImportError: numpy.core.multiarray failed to import

I also need to add "pytz" to the requirements file and use django_chartit2 in order to run the project. Maybe it's because of Mac.

Strangely enough, I am on a mac also, but seem to be fine with numpy==1.8.0rc1.

Is anyone else having issues on this?

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.1 ETH (69.21 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.1 ETH (69.21 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.1 ETH (69.21 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.2 ETH (138.42 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.1 ETH (69.21 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.1 ETH (69.21 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.1 ETH (69.21 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.1 ETH (69.21 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.1 ETH (69.21 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.1 ETH (69.21 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer


Fix for CUDA stuff not compiling due to trying to use the deprecated tensor.Memory.Pointer function

The Problem
So when I try to run with build tag cuda, it won't compile. It looks like the interface Memory in package gorgonia.tensor is supposed to declare a function Pointer (or at least this package expects it to). When I added that function to the interface it broke many things.
After further investigation, it looks like the Pointer method was actually removed from the tensor package. So I guess this might actually be the correct way to fix the problem.
My Fix
I have just added a function that converts a Value into an unsafe.Pointer. This may be a bad way to do this, but it seems to work correctly for me.
My package versions
gorgonia v0.9.17
tensor v0.9.24
cu v0.9.4
Tests
I ran go test in the gorgonia directory and it passed. I think this problem might have slipped through the automated tests if they are not testing stuff with the cuda build tag. Maby this should be added?

Hmm.. I'm quite sure the tensor package hasn't removed Pointer... as I'm still using it on a daily basis, with CUDA. Let me check

nope you  are right... I will merge this once it passes the tests

I think you might need to approve the workflow as I am a first time contributor. On my end it still says waiting for approval to run checks.

@chewxy Please may you approve the workflow so it can perform tests

sorry... fixed


svg title used as document title after refreshing sprig component

Question
Hi Ben.
When using an inline svg within a sprig component, the title of the document gets overwritten by the svg´s title. We are using craft´s svg function and setting a title there. https://craftcms.com/docs/3.x/dev/functions.html#svg
It seems like htmx is looking for title in selectAndSwap and picks the title from the svg as document title.
Thanks for your advice
Additional context
you can reproduce this by using a svg with title in a sprig component like this
            {{svg("icons/ic_bookmark", "Bookmark"|t)}}

Can you double check what version of htmx is being loaded into the page?

Hi Ben. Yes, you are right. htmx 1.6.0 and sprig 1.9.0 fixes this.
I thought I was up to date as no updates were showing, but have rechecked this morning I was actually on the dev/pagination branch we tested. I have update to 1.9.3 and all working as expected.
Thank you.


Add support for quick switcher

It would be great to be able to type in natural language dates in the quick switcher to navigate to a Daily Note. At the moment we can only navigate to today's note using a hotkey or use the calendar plugin to manually find and click on the date. If nldates supported the quick switcher, we could just hit cmd+o and type "next wednesday" to open next Wednesday's daily note.

I would LOVE this feature

I'd love to see this as well. If not through the quick switcher, I don't mind having a separate command palette just to jump to a relative date quickly.


readme: add missing LockOSThread() to example

Hi @gonutz, there is LockOSThread() under the sdl package's init function. I believe it has the same effect, if I understood the docs correctly.

Oh yes you are right, I am closing this now. I added this to my DLL port as well.


Support Wear OS by Google

What do you think about Wear OS by Google?
|  |  |  |

@linsui, do we have to update the F-Droid setup for this?
The code for the Wear OS companion app will be in a separate folder wear.

I guess the scanner will complain about non-free deps in the wear folder. So you need to add it to rm: and remove it from settings.gradle.

I guess the scanner will complain about non-free deps in the wear folder. So you need to add it to rm: and remove it from settings.gradle.

I just opened a MR on GitLab: https://gitlab.com/fdroid/fdroiddata/-/merge_requests/14316.
I also addressed https://gitlab.com/fdroid/fdroiddata/-/issues/2128# which I did not see until today. Sorry for the late reply.

@karimapps will develop this feature. 👍

https://gitlab.com/fdroid/fdroiddata/-/merge_requests/14397


can`t find CallManager  class,why?what should i do?

Have you tried with the latest version (android-23) ?

yes

Please follow the instruction as described here. But if the error still happen, attach the detail, i.e. Android Studio version, etc.

Have the problem resolved?

Seems that this issue is no longer active. I will close it.


[Draft] NYX Eiger2 Flyer initial commit

Still not sure why the history switches between the commits, I may just not restore the old flyer in order to preserve the history and manually copy/add it back in.

I have to re-post this PR to see if it fixes the missing history.


Invite to org

Name: Abhishek Morla
Why would you like to join the organization?
hey, hope you doing great. I would like to join this organization to enhance my work experience in the openSource project.
 This will show our profile org on your profile to others:
Follow This:

After you join the organisation, search your name in the list of people. 🔍
Follow this below: ⬇

🎉🎊Welcome to Jaidev Studio🎉🎊!!

Why not contribute to...
Unique-Github-Profiles!

I will close this issue for now, but if you have any questions, feel free to open this again, or open a new issue. Thanks.


IGNITE-12666 Provide cluster performance profiling tool

Cluster profiling tool based on performance logging to profiling files. See https://github.com/apache/ignite/pull/7693
Run command to build the tool: mvn clean install
Release archive will be be under the directory: modules/profiling-ext/target/release-package-ignite-profiling.zip
To collect statistics in runtime and to build the performance report follow:

Start profiling. See IgniteProfilingMBean
Collect workload statistics.
Stop profiling. See IgniteProfilingMBean
Collect profiling files from all nodes under an empty directory.
Run script profiling/build-report.sh path_to_files to build the performance report.

The report contains statistics:

cache operations: get|getAll|put|putAll|remove|removeAll|getAndPut|getAndRemove|lock|invoke|invokeAll
transaction commit, rollback; histogram of transaction durations
SQL, SCAN queries, top of slowest queries
Tasks and jobs. Top of slowest tasks.

@NSAmelchev There is conflicts in this PR, can you please resolve the conflict and then go ahead and merge the changes.

@NSAmelchev Can we modify build process so it will download css, js resources in the build time?
I think we shouldn't store css or js resources in Ignite repository.


Rails

Rails always look curved and the other rail types are always powered

I will have to actually add some block state things to fix this. It isnt fixible in 1.3 at the minute.

Fixed in next release. Is fixed in the discord bot.


Implement default settings for database connections

Define and implement a unified set of settings (along with their defaults) for connecting to MySQL and Postgres databases. datajoint-core should currently only use default settings, but setting these settings programmatically and reading them from a file should also be supported. All settings should be appropriately mapped to SQLx for creating a database connection.

Closed by merge request on 10/1/2021 #12


Let border-height be configurable

The border height of the table is currently fixed at 1px: https://github.com/facebook/fixed-data-table/blob/master/src/FixedDataTable.react.js#L39
var BORDER_HEIGHT = 1;

I'd like to integrate a table with no border, but the content area is then 2px shorter than it should be because FixedDataTable assumes 2px of its height are consumed by its top and bottom borders.

I would recommend for this using a transparent border, this way you keep the width but it looks like its not there. Our default layout theme does this. In the future we are looking to make this stuff configurable via inline styles but its currently assumed in a lot of places

I'm having this issue as well. Using a transparent border doesn't fix the problem as it still takes up space.

did this get changed?


Moving the installation of golangci-lint to Dockerfile

Sending the PR to just change the Dockerfile as without this is merged the unit tests will never pass for the Makefile change in https://github.com/openshift/hive/pull/528
Fixes part of  https://jira.coreos.com/browse/CO-539
Signed-off-by: Lalatendu Mohanty<EMAIL_ADDRESS>

@staebler @joelddiaz need the lgtm again.


Generated commit messages should use present tense

When editing a file in Prose, the commit message starts with something like "Updated", "Created", or "Deleted". However, it is recommended to use present tense in commit messages, and git itself follows this when generating commits (such as merge commits). I think commit history would look more consistent if Prose used present tense to start commit messages.

👍 Happy to accept a PR on this.

I created a PR for this, updated the content of the language key and also renamed them so that it matches.
I assume the translations itself will be fixed via transifex.
I am not clear if I should also manually update the key names of all the translations!?


Return Promise.reject for action path of services when catching an er…

This commit resolves the issue with patch action. The action does not return a Promise.reject on catch.

Thanks!  Published as<EMAIL_ADDRESS>


Update template index to run "Hello world!"

The current implementation doesn't seem to run the function that was exported from Rust, main_js().
At first, I thought this was intentional, but the current docs which are out of date and say that a run() function should be here.
I'll add a link to my change to the docs here shortly.

The pull request for the documentation change is wasm-pack Issue #899.


Use logger module throughout our server code

Now that #93 is in, we have lib/logger.server.ts available to all of our server code.  I'd like to see us use it to add proper debug, info, warn, error, style logs.
We should prefer to use debug for most things, and only bother with info when absolutely necessary (e.g., in production, we don't need to log normal behaviour).  We should use warn for cases where we handle the error, and leave error for actual errors that we want to know about in production.

@humphd, Since we are no longer using pino, is it safe to close this issue?

No, I still want this, just using the new logger vs. pino.  I've updated the title.

We're doing this all over now.


Disclosure component hydration missmatch

Wrapping it in ClientOnly fixes it but will be better if the generated id matches and the component is rendered server side.

<Disclosure v-slot="{ open }">
    <DisclosureButton
      class="row text-center w-full flex items-center m-0 py-2"
    >
    Button
      <div class="col-md-1 justify-end flex">
        <ChevronUpIcon
          :class="!open ? 'rotate-180 transform' : ''"
          class="h-7 w-7 text-sky-800 self-center flex"
        />
      </div>
    </DisclosureButton>
      <DisclosurePanel
        class="text-gray-500 py-2 text-center"
      >
      Expanded content
      </DisclosurePanel>
  </Disclosure>

This is an upstream issue. Please see https://github.com/tailwindlabs/headlessui/issues/2913.

@P4sca1  https://github.com/tailwindlabs/headlessui/issues/2913#issuecomment-1924681165 could this fix by @reinink work?


Moved maintenance in its own folder.

Scope & Purpose
Moved all Maintenance files into their own folder.

[ ] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[x] :hammer: Refactoring/simplification
[ ] :book: CHANGELOG entry made

Backports:

[x] No backports required
[ ] Backports required for: (Please specify versions and link PRs)

Testing & Verification
(Please pick either of the following options)

[x] This change is a trivial rework / code cleanup without any test coverage.
[ ] The behavior in this PR was manually tested
[ ] This change is already covered by existing tests, such as (please describe tests).
[ ] This PR adds tests that were used to verify all changes:

[ ] Added new C++ Unit tests
[ ] Added new integration tests (e.g. in shell_server / shell_server_aql)
[ ] Added new resilience tests (only if the feature is impacted by failovers)

[ ] There are tests in an external testing repository:
[ ] I ensured this code runs with ASan / TSan or other static verification tools

https://jenkins.arangodb.biz/job/arangodb-matrix-pr/14847/

https://jenkins.arangodb.biz/job/arangodb-matrix-pr/14848/

https://jenkins.arangodb.biz/job/arangodb-matrix-pr/14851/


fix: deployer

Codecov Report

Merging #13 (1b8329d) into develop (e1f12f7) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 1b8329d differs from pull request most recent head 7f9c078. Consider uploading reports for the commit 7f9c078 to get more accurate results

@@            Coverage Diff             @@
##             develop      #13   +/-   ##
==========================================
  Coverage      56.52%   56.52%
  Complexity        19       19
==========================================
  Files             10       10
  Lines             46       46
==========================================
  Hits              26       26
  Misses            20       20


add constant

What problem does this PR solve?
Issue Number: close pingcap/tidb#19907
What is changed and how it works?
add two constant for function SUM and AVG
Check List
Tests

No code

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


NSInvalidArgumentException, reason: GUL_UIApplicationDelegateImpl

Environment
package.json:

{
"nativescript": {
"id": "ar.waisoft.sampayo",
"tns-android": {
"version": "6.0.0"
},
"tns-ios": {
"version": "6.0.1"
}
},
"description": "NativeScript Application",
"license": "SEE LICENSE IN ",
"repository": "",
"dependencies": {
"@angular/animations": "~8.0.0",
"@angular/common": "~8.0.0",
"@angular/compiler": "~8.0.0",
"@angular/core": "~8.0.0",
"@angular/forms": "~8.0.0",
"@angular/http": "~8.0.0-beta.10",
"@angular/platform-browser": "~8.0.0",
"@angular/platform-browser-dynamic": "~8.0.0",
"@angular/router": "~8.0.0",
"nativescript-angular": "~8.0.0",
"nativescript-block-ui": "^1.0.0",
"nativescript-fancyalert": "^3.0.8",
"nativescript-localstorage": "^2.0.0",
"nativescript-mercadopago-px": "^1.1.5",
"nativescript-plugin-firebase": "^9.0.4",
"nativescript-theme-core": "^1.0.6",
"nativescript-toast": "^2.0.0",
"nativescript-ui-listview": "^7.0.1",
"reflect-metadata": "~0.1.12",
"rxjs": "~6.5.0",
"tns-core-modules": "~6.0.0",
"zone.js": "~0.9.1"
},
"devDependencies": {
"@angular/compiler-cli": "~8.0.0",
"@ngtools/webpack": "~8.0.0",
"nativescript-dev-webpack": "~1.0.0",
"node-sass": "^4.12.0",
"typescript": "~3.4.0"
},
"gitHead": "f28dbc60d74dd2cef4b645afd8fdd63bbb12c73e",
"readme": "NativeScript Application"
}

CLI: 6.0.0
Cross-platform modules:
iOS Runtime: YES
Android Runtime (if applicable): NO
Plugin(s): nativescript-mercadopago-px

Describe the bug
When I run the nativescript-mercadopago-px plugin in the example everything works fine, but when I try to use it in my application, the plugin was run but it ends up closing the app
-[GUL_UIApplicationDelegateImpl-390FB4A5-51F3-4449-9CF3-3A3557105BE1 window]: unrecognized selector sent to instance 0x2828f0700 *** JavaScript call stack: ( 0   UIApplicationMain@[native code] 1   _start@file:///node_modules/tns-core-modules/application/application.js:277:0 2   run@file:///node_modules/tns-core-modules/application/application.js:305:0 3   bootstrapNativeScriptApp@file:///node_modules/nativescript-angular/platform-common.js:205:0 4   bootstrapApp@file:///node_modules/nativescript-angular/platform-common.js:106:0 5   bootstrapModule@file:///node_modules/nativescript-angular/platform-common.js:90:0 6   ./main.ts@file:///app/bundle.js:536:116 7   __webpack_require__@file:///src/webpack/bootstrap:74:0 8   checkDeferredModules@file:///src/webpack/bootstrap:43:0 9   webpackJsonpCallback@file:///src/webpack/bootstrap:30:0 10  anonymous@file:///app/bundle.js:2:61 11  evaluate@[native code] 12  moduleEvaluation@:1:11 13  promiseReactionJob@:1:11 ) *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[GUL_UIApplicationDelegateImpl-390FB4A5-51F3-4449-9CF3-3A3557105BE1 window]: unrecognized selector sent to instance 0x2828f0700' *** First throw call stack: (0x205dc298c 0x204f9b9f8 0x205cdf1c8 0x2326f2220 0x205dc7ed0 0x205dc9e6c 0x1010fb1a4 0x1010fb334 0x205d33a28 0x205d339f4 0x205d32ee8 0x205d32b94 0x205cac474 0x205d32644 0x20671b6f4 0x2324d6378 0x2324d8e44 0x232b5225c 0x232b52708 0x2324d893c 0x2324df784 0x2324d7fac 0x232551280 0x23254a804 0x2326f5a00 0x2326f16e4 0x232b4887c 0x2329ad434 0x1023f6980 0x1023f7a54 0x232122a20 0x232123384 0x232086244 0x23207b640 0x232145f24 0x2321561f4 0x232b51d1c 0x232b28a74 0x232b29048 0x232b290e8 0x20a3113c8 0x2058017d4 0x2057af008 0x205d5432c 0x205d4f264 0x205d4e7c0 0x207f4f79c 0x2326c6c38 0x101f1c044 0x101f19780 0x101f1927c 0x1013bcf54 0x101f18488 0x101f16804 0x101f16804 0x101f16804 0x101f16804 0x101f16804 0x101f16804 0x101f16804 0x101f16804 0x101f16764 0x101f16804 0x101f022ac 0x1<\M-b\M^@\M-&>
Expected behavior
https://github.com/NeriCodec/nativescript-mercadopago-px/tree/master/demo-angular

@NeriCodec Can you please provide more information about how we can reproduce the issue. It looks like the crash is related to the code in your AppDelegate implementation, which seems to lack the window property.

Hello @mbektchiev
Yes, I have the code here where it fails https://drive.google.com/open?id=1YX_Rzw9Yt_i7KiWt6mQeUmFhF6VoAOz2

I've reproduced the crash and found out the issue is caused by having both Firebase and MercadoPago plugins:

The Firebase plugin adds an app delegate here
One of MercadoPago's CocoaPod dependencies has this method in its platforms/ios/Pods/MLUI/LibraryComponents/MLSnackBar/classes/MLSnackbar.m:

- (UIWindow *)window
{
	return [[[UIApplication sharedApplication] delegate] window];
}

It unconditionally calls the delegate's window method, which is @optional and should be called only if the delegate implements it. In this case, it doesn't and hence the call throws the exception you're reporting.
As a workaround, you can either implement the window method and I recommend that you request the authors of the library to fix their code.

@mbektchiev Thank you very much, but how could I do this in my plugin?
As a workaround, you can either implement the window method and I recommend that you request the authors of the library to fix their code.

I meant in the app. I don't recommend that you modify the app delegate in your plugin. You can advise any users of the plugin who are facing the issue to do this in their apps.

Since they should add the delegate in their mobile applications, it would be helpful to find out

@mbektchiev some clue ?

Please refer to this app.js file. you will need to implement the following delegate in order to secure the window.
var application = require("tns-core-modules/application");
var firebase = require("nativescript-plugin-firebase");
firebase.init({}).then((instance) => {
console.log("[] Firebase was successfully initialised");}, (error) => {
console.log("[] we've an initialization error: " + error);});
var appDelegate = UIResponder.extend({
get window() {
return this._window;
},
set window(aWindow) {
this._window = aWindow;
}
}, {
protocols: [UIApplicationDelegate]
});
application.ios.delegate = appDelegate;
application.run({ moduleName: "app-root" });

@svpasumarthy thanks man, this solution work fine !!!....


[Major update] Add multilane link; Separate Analyzer class from uxsim.py

Wow big update! Looking forward to using the multilane link!

I have updated PyPI too. I believe it is functional and properly integrated with OSMImporter.
And thanks for the shortest path optimization. I will look at the test results later, but unfortunately my spring break is over, so it may take some time. I hope I can make quick comments tomorrow.


Add com.growthbook/experiment_viewed/jsonschema/1-0-0

Added GrowthBook experiment_viewed event tracking schema
Fixes #1422

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

I signed it!

Confirmed! @Auz has signed the Contributor License Agreement. Thanks so much.

Thanks @Auz - the JSON path file is no longer needed (thankfully) so you can just PR the schema in this instance.
Are additionalProperties required here? If so what do you expect these to generally be or if this is a fixed set can we set this to false?

Thanks @Auz - the JSON path file is no longer needed (thankfully) so you can just PR the schema in this instance.
Are additionalProperties required here? If so what do you expect these to generally be or if this is a fixed set can we set this to false?

They are not required. In Growthbook, when we query to get the list of users that are exposed to an experiment, any additional columns can be returned to be used as dimensional data. These are typically things like 'browser' or 'country', which might be readily available at the time of exposure, but harder or more expensive to query when making the reports if it requires a lot of joins. I wanted to make it possible to set additional fields, but if this is not how Snowplow does things, let me know.

If there are more custom things it may make sense to have a part of the schema that defines an array of key value pairs (additionalProperties is tricky as we aim to infer types / column names from the schema itself). Things like browser, country, device information etc are captured automatically as part of the out of the box data (either directly or via enrichments) so hopefully shouldn't be required.


[Update]: smplayer's persistent data will be in dedicated subdirectory

Prerequisites

[X] I have written a descriptive issue title.
[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.
[X] I have verified that I am using the latest version of Scoop and corresponding bucket.

Package Name
smplayer
Expected/Current Behaviour
the details of current manifest of smplayer are listed here:

https://github.com/smplayer-dev/smplayer/issues/934#issuecomment-2014860249

the details of behaviour after update is that, all these persistent configs .ini or .m3u8 files are moved under config/ directory

https://github.com/smplayer-dev/smplayer/issues/934#issuecomment-2070073965

Steps to Reproduce
> scoop install smplayer

> # https://superuser.com/q/318197/a#1056648
> # See the scattered persistent files in the install dir of smplayer
> dir <scoop-apps-dir>/smplayer/current/ | where {$_.extension -in ".ini", ".m3u8"}

> # See that 'config' subdir is missing
> test-path <scoop-apps-dir>/smplayer/current/config

Possible Solution
change the manifest to reflect that.
the problem is that the existing persistent files will have to be moved in the config/ dir too
other than that, there's not much to it.
for a trial of this fresh behaviour at dev builds, i created this manifest for myself:
{
    "$schema": "https://raw.githubusercontent.com/ScoopInstaller/Scoop/master/schema.json",
    "description": "A video player forked from MPV. It can also play Youtube videos, search and download subtitles, and includes other features like a thumbnail generator and audio and video filters.",
    "homepage": "https://www.smplayer.info/",
    "license": "GPL-2.0-or-later",
    "extract_dir": "smplayer-portable",
    "shortcuts": [
        [
            "smplayer.exe",
            "SMPlayer"
        ]
    ],
    "persist": [
        "screenshots",
        "config"
    ],
    "checkver": {
        "url": "https://api.github.com/repos/smplayer-dev/smplayer/releases",
        "jsonpath": "$[0].assets[*].name",
        "regex": "smplayer-portable-([\\d.]+)-win32-qt(?<qtversion>[\\d.]+)\\.7z"
    },
    "autoupdate": {
        "architecture": {
            "64bit": {
                "url": "https://github.com/smplayer-dev/smplayer/releases/download/v$version/smplayer-portable-$version-x64.7z"
            },
            "32bit": {
                "url": "https://github.com/smplayer-dev/smplayer/releases/download/v$version/smplayer-portable-$version-win32-qt$matchQtversion.7z"
            }
        }
    },
    "version": "<IP_ADDRESS>66",
    "architecture": {
        "64bit": {
            "url": "https://github.com/smplayer-dev/smplayer/releases/download/v<IP_ADDRESS>66/smplayer-portable-<IP_ADDRESS>66-x64.7z",
            "hash": "1c0c88a507e8e7aeb4b432e3c1e4a303f4da753b2fba2121394a7cfcf057b4fa"
        },
        "32bit": {
            "url": "https://github.com/smplayer-dev/smplayer/releases/download/v<IP_ADDRESS>66/smplayer-portable-<IP_ADDRESS>66-win32-qt5.6.7z",
            "hash": "faa113ea8b2f26088be5f19bc60092a06e348d11388a7c6c0ce7f1543037b7c6"
        }
    }
}

Scoop and Buckets Version
> scoop --version
Current Scoop version:
105e4161 (HEAD -> master, tag: v0.4.0, origin/master, origin/HEAD) chore(release): Bump to version 0.4.0 (#5424)

'main' bucket:
623969617 (HEAD -> master, origin/master, origin/HEAD<EMAIL_ADDRESS>Fix hash (Closes #5735)

'extras' bucket:
a0a043ccb (HEAD -> master, origin/master, origin/HEAD) youtrack: Update to version 2024.1.28928

'nerd-fonts' bucket:
6d01d9ec (HEAD -> master, origin/master, origin/HEAD) SarasaGothic: Update to version 1.0.10

'nonportable' bucket:
2fd31611 (HEAD -> master, origin/master, origin/HEAD) autohotkey-np: Update to version 2.0.13

'java' bucket:
04ec327c (HEAD -> master, origin/master, origin/HEAD) temurin22-jre: Update to version 22.0.1-8

'dorado' bucket:
23176ec1 (HEAD -> master, origin/master, origin/HEAD) swift-nightly: Update to version 20240421.3

'svah' bucket:
fatal: not a git repository (or any of the parent directories): .git

Scoop Config
> scoop config

root_path    : D:\UserFiles\scoop
global_path  : D:\ProgramFiles\scoop_global
last_update  : 2024-04-22T20:21:36.8486836+05:30
alias        : @{cat-dum=scoop-cat-dum}
debug        : True
scoop_branch : master
scoop_repo   : https://github.com/ScoopInstaller/Scoop

PowerShell Version
> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.19041.4170
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.4170
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Additional Softwares

smplayer

it is released now
https://www.smplayer.info/en/changes
> scoop status
WARN  Scoop bucket(s) out of date. Run 'scoop update' to get the latest changes.

Name                       Installed Version    Latest Version         Missing
                                                                       Dependen
                                                                       cies
----                       -----------------    --------------         --------
smplayer                   <IP_ADDRESS>            <IP_ADDRESS>


Too many dependencies for basic crash analytics

I would like to use AppCenter for crash analytics, but I count about 10 dependencies including NewtonSoft.Json and SQLLite when adding the NuGet package Microsoft.AppCenter.Analytics. I have a significant investment in eliminating dependencies (and startup time) in my application, and this large dependency overhead unfortunately makes AppCenter unusable for me.

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

ID: 06ece9ab-9965-2ddf-34b6-44f0f61fe599
Version Independent ID: 16261b98-a85e-060f-ccf4-3a24f5f14c4c
Content: Get Started with UWP - Visual Studio App Center | Microsoft Docs
Content Source: docs/sdk/getting-started/uwp.md
Service: vs-appcenter
GitHub Login: @elamalani
Microsoft Alias: emalani

@dtaylorus Do you have a question about the docs?  If you have feedback about the product or need help with the product, please use the chat option in App Center (the blue chat icon in the lower-right corner of most App Center pages).


fix(client): let requests read env vars

This fix lets the requests library read configuration from environment
variables.
This is intended to fix a problem I had with being unable to
authenticate with deis when using a self-signed cert from a debian
system. By default, requests does not trust the system cert bundle on
debian systems and requires either verify=/path/to/bundle (as a config
value of requests.Session) or REQUESTS_CA_BUNDLE=/path/to/bundle. But,
in order for that environment variable to get read,
requests.Session.trust_env must be True.

Thanks for the contribution! Please ensure your commits follow our style guide. This code will be tested once a Deis maintainer reviews it.

From discussion in the IRC channel, @mboersma didn't recall the original reasoning for this line. It's possible it was due to weird proxy interactions. It doesn't seem necessary anymore, whatever the original reasoning was.

Code LGTM.

I'm following the instructions in http://docs.deis.io/en/latest/contributing/testing/. Will report back when complete.

Ha, that change predates me, accoring to git blame!
><> git blame deis.py | grep trust_env
40bf6f64 client/deis/client.py (Matt Boersma         2013-07-22 13:04:46 -0600   94)         self.trust_env = False
><> git show 40bf6f64
commit 40bf6f64ddb2d9dbe7acbd4fd528c8087d9bbca0
Author: Matt Boersma<EMAIL_ADDRESS>Date:   Mon Jul 22 13:04:46 2013 -0600

    Merged three projects into one.

[...]

Code LGTM.

Jenkins, add to whitelist.

I haven't been able to get the tests running locally, yet. Can anybody tell if the jenkins failures are related to this change?

@sbuss It doesn't seem to be releated, it looks to me like a timeout because a app took more time to come up than expected.

By the way, here's the effect of this change (as it pertains to my use case):
On master:

(venv)sbuss@scratch:/vagrant/deis/client$ python deis.py login https://deis.deis.counsyl.com

https://deis.deis.counsyl.com does not appear to be a valid Deis controller.
Make sure that the Controller URI is correct and the server is running.

Specific Error: [Errno bad handshake] [('SSL routines', 'SSL3_GET_SERVER_CERTIFICATE', 'certificate verify failed')]
(venv)sbuss@scratch:/vagrant/deis/client$ env REQUESTS_CA_BUNDLE=/etc/ssl/certs/ca-certificates.crt python deis.py login https://deis.deis.counsyl.com

https://deis.deis.counsyl.com does not appear to be a valid Deis controller.
Make sure that the Controller URI is correct and the server is running.

Specific Error: [Errno bad handshake] [('SSL routines', 'SSL3_GET_SERVER_CERTIFICATE', 'certificate verify failed')]

With this patch:
(venv)counsyl@scratch:/vagrant/deis/client$ python deis.py login https://deis.deis.counsyl.com

https://deis.deis.counsyl.com does not appear to be a valid Deis controller.
Make sure that the Controller URI is correct and the server is running.

Specific Error: [Errno bad handshake] [('SSL routines', 'SSL3_GET_SERVER_CERTIFICATE', 'certificate verify failed')]
(venv)counsyl@scratch:/vagrant/deis/client$ env REQUESTS_CA_BUNDLE=/etc/ssl/certs/ca-certificates.crt python deis.py login https://deis.deis.counsyl.com
username: sbuss
password:
Logged in as sbuss

Code LGTM.


youtube-dlg stops downloading from lynda.com. After many tries the same message appears (Error). In the log there is the message i get   [11/13/18 11:14:20] ERROR: Unable to extract signin form; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

youtube-dlg stops downloading from lynda.com. After many tries the same message appears (Error). In the log there is the message i get
[11/13/18 11:14:20] ERROR: Unable to extract signin form; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
PS : i made the update many times
Thank you for helping me
Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

WARNING
All invalid issues will be rejected!!

Before going further

If your problem is a bug with youtube-dl or a request for new site support please report it here

Make sure you are using the latest youtube-dl-gui version (Click the Settings icon and then About to view the current version)

Make sure you are using the latest youtube-dl version (Click the Settings icon and then Update to update to the latest youtube-dl version)

Make sure you searched the bugtracker for similar issues including closed ones

Make sure to read the FAQs file

[ ] I think my problem is NOT with youtube-dl
[ ] I've verified and i assure that I'm running youtube-dl-gui 0.4
[ ] I assure that i am using the latest version of youtube-dl
[ ] Searched bugtracker
[ ] I've read the FAQs file

What is the purpose of your issue?

[ ] Bug report
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Please remove any sections between (---) if they are not related to your issue

Bug report
If the problem occurs when downloading a URL please provide the full verbose output as follows:

Restart youtube-dl-gui
Go to Options > Extra tab
Enable Debug youtube-dl
Go to Options > Advanced tab and Clear your log content
Try to download the URL
Copy the whole log content and insert it between the ``` part below

delete me and insert your log content here

What operating system do you use ?
List of actions to perform to reproduce the problem:

..
..
..

What is the expected behaviour ?
What happens instead ?

Feature request (request for a new functionality)
Please make sure that the requested feature is NOT already in the TODO list

[ ] I've verified and i assure that my requested feature is NOT in the TODO list

What operating system do you use ?

https://github.com/rg3/youtube-dl/issues/18158


Panic

That can be a problem of trampoline

try use other base address for smp.
see
https://github.com/nifanfa/MOOS/blob/350f406a42b0a5805415df960a7a557250684021/Kernel/SMP.cs#L10
and
https://github.com/nifanfa/MOOS/blob/350f406a42b0a5805415df960a7a557250684021/Tools/Trampoline.asm#L1

this address should be lower than 1mb. however it is hard to verify which address is available to use. 0x50000 is free on my PC

@AppleInternal


Add a lightweight logger frontend

Rationale
Currently logging in SFML is done by simply piping strings through sf::err, but most logging frameworks are message oriented and thus it's quite cumbersome to integrate the SFML log with anything else than another std::ostream. Furthermore it's hard to prevent mixing message parts in a multithreaded enviroment. So I made up this approach which serves the following design goals:

end user simplicity (as simple as sf::err)
message orientation, with metadata based filtering (channel, importance)
simple log backend integration
extensibility, i.e. SFML addons can log messages using their own channels utilizing the same backend
performance; if logging is turned off there should be little overhead caused by dropping messages

Usage
The public interface is completely documented --- html version (you'll probably get a certificate error, because the certificate is issued by cacert.org, so simply ignore it or install the root certificate).
A minimal example would be:
#include <SFML/System/Log.hpp>
int main( )
{
    int toBeOutputted = 4;
    sfLogSysWarn() << "Your log message formatted by a std::ostream " << toBeOutputted;
    return 0;
}

This should output the following to std::cerr (if SFML system was compiled with _DEBUG):
SFML SYSTEM WARNING: Your log message formatted by a std::ostream 4

In order to redirect the log output or format it differently you would do something like this:
#include <SFML/System/Log.hpp>
#include <fstream>

void logEntryProcessor(void *, sf::Log::Channel channel, sf::Log::Level level, const std::string &message)
{
    static std::ofstream backend("cerr.txt", std::ios::out | std::ios::trunc);
    backend << "${TIME} SFML" << channel.toString() << " " << level.toString() << " " message << std::endl;
}

int main( )
{
    sf::Log::Manager::setCustomLogEntryProcessor(&logEntryProcessor,  NULL);
    int toBeOutputted = 4;
    sfLogSysWarn() << "Your log message formatted by a std::ostream " << toBeOutputted;
    return 0;
}

Questions
What do you think of this?
I would like to completely replace sf::err() with the appropriate sfLogXXXXXX() throughout the library and deprecate the former one, so everyone can profit from this, would this be accepted?
Best wishes
Henrik Gaßmann
Edit: I just want to add that replacing the usage of sf::err() with the appropriate sfLogXXXXXX() doesn't necessarily has to be a breaking change, because we could pipe all the messages from the default log processor through sf::err(), so that existing implementations which redirect sf::err() will continue to work until sf::err() gets completely removed.

More flexible logging is something we wanted to discuss already for quite a while, I have now created a forum thread.
I think we should be clear about the general requirements and design before discussing concrete implementations, so I encourage you to participate in that thread. Let's focus on a high-level requirement standpoint first and look at the implementation later :)

While we appreciate the work, I don't see this being considered any time soon and if so, most likely with a different interface.
Since there's no point in keeping things open indefinitely, I'm going to reject the PR and close it. It might still be useful as a reference if the topic comes up again.


Add guidance to explain our support role

What
Add guidance to the Design System website to:

tell users what we can and cannot support
link to other resources that might help users

Why
Some users think we look after other design systems across government. For example, users on Zendesk asking about the Home Office Design System.

I'm an Associate Developer. I'd like to ask if there's a way that the home office design system styles and components can be implemented in a create-react-app environment?

Who needs to work on this
Content designer/technical writer, anyone involved in support
Who needs to review this
Anyone on the team can review
Done when

[ ] location found for update
[ ] update drafted
[ ] update reviewed and approved
[ ] update published

This issue covers recommendation #7 in @SaraC19's support analysis slides.


Create bucket before provisioning access logs on the ALB

On three occasions when using this module the S3 bucket was either created after the ALB, or wasn't ready by the time the ALB was trying to set up access logs and I received errors like this:
aws_alb.main: Failure configuring ALB attributes: ValidationError: S3Bucket: bucket_name does not exist
	status code: 400, request id: xxxxx-xxxxxx-xxxxxx-xxxxxx-xxxxxx

If the option to have the module create the bucket is selected, this ensures the bucket is created before trying to tell the ALB to send logs there.
I tested this with both create_log_bucket set to true and false and it worked both ways. Additionally I ran the kitchen test and here is the tail end of the output (it was succesful)
       Destroy complete! Resources: 24 destroyed.
$$$$$$ Running command `terraform workspace select default`
       Switched to workspace "default".
$$$$$$ Running command `terraform workspace delete kitchen-terraform-default-aws`
       Deleted workspace "kitchen-terraform-default-aws"!
       Finished destroying <default-aws> (1m8.48s).
       Finished testing <default-aws> (4m10.89s).
-----> Kitchen is finished. (4m11.48s)

When I was installing the pre-reqs to do the kitchen testing I actually had to change the kitchen-terraform gem from branch ncs-alane-3.0.0 (doesn't exist anymore?) to the SHA of a recent commit in that repo, so that it looked like the following (I did not commit this change):
diff --git a/Gemfile b/Gemfile
index 19a2c89..5eaf21f 100644
--- a/Gemfile
+++ b/Gemfile
@@ -9,5 +9,5 @@ end
 gem(
   "kitchen-terraform",
   git: "https://github.com/newcontext-oss/kitchen-terraform",
-  branch: "ncs-alane-3.0.0"
+  branch: "ed1885e77442998781abed5fb26a16dd64310514"
 )```

Hey @zxjinn. Thanks for chiming in with the PR. Your changes seem reasonable to me and I'd like to add to it.
I didn't know the branch would be deleted (looks like that happened today - apologies, will keep this in mind going forward) but the 3.0.0 gem has been published to rubygems as of today. So, if you could remove the extra bits in the Gemfile here and move kitchen-terraform back to the source block, that should resolve the gem issue.
Also, addressing the bucket creation dependency, an explicit depends_on = ["aws_s3_bucket.log_bucket"] added to the ALB resource would likely clear that issue. It'd be worth testing how that behaves with the log bucket created and not.
Did I miss anything?

To be honest I've never used depends_on when I knew the resource count was going to be 0, turns out it works! And, it's much less complicated to understand what's going on.
I updated the Gemfile to point to at least version 3.0.0 of that gem.

One thing I did notice is that if I do not want logging on at all, which is the case for a lot of non production environments, it essentially doesn't allow you to do that
aws_alb.main: Failure configuring ALB attributes: ValidationError: The value of 'access_logs.s3.bucket' cannot be empty
	status code: 400, request id: xxxxx-xxxxxx-xxxxxx-xxxxxx-xxxxxx

I was thinking of modifying the file to look something like this. Though, I realize that all the outputs would need to be modified along with anything else that directly depends on the alb resource. I just don't know of a good way to exclude an access_logs block. Do you?
resource "aws_alb" "main" {
  count = "${var.log_bucket_name != "" ? 1 : 0}"
  name            = "${var.alb_name}"
  subnets         = ["${var.subnets}"]
  security_groups = ["${var.alb_security_groups}"]
  internal        = "${var.alb_is_internal}"
  tags            = "${merge(var.tags, map("Name", format("%s", var.alb_name)))}"

  access_logs {
    bucket  = "${var.log_bucket_name}"
    prefix  = "${var.log_location_prefix}"
    enabled = "${var.enable_logging}"
  }

  depends_on = ["aws_s3_bucket.log_bucket"]
}

resource "aws_alb" "main-nobucket" {
  count = "${var.log_bucket_name == "" ? 1 : 0}"
  name            = "${var.alb_name}"
  subnets         = ["${var.subnets}"]
  security_groups = ["${var.alb_security_groups}"]
  internal        = "${var.alb_is_internal}"
  tags            = "${merge(var.tags, map("Name", format("%s", var.alb_name)))}"

  access_logs {
    bucket  = "${var.log_bucket_name}"
    prefix  = "${var.log_location_prefix}"
    enabled = "${var.enable_logging}"
  }

  depends_on = ["aws_s3_bucket.log_bucket"]
}

The block that controls logging was specifically added to address the problem of logging being a hard requirement earlier in the project's life. I had designed a hacky solution similar to what you're proposing but decided not to promote it, especially in light of the enabled setting (added in the past 3 months, iirc). I think it'd be worth proving out that the current approach of enabling and disabling logs doesn't work through tests before pursuing what you've pitched further.
The explicit dependency, gem fix, and doc update are valuable. Let's separate out the logging issue to a different PR/issue. Thanks again on the contribution! I'll add an issue for proving out the disabled logging option in test.

I should have been more explicit about what I was suggesting in the second half of my previous comment. I completely agree with the idea of separating the logging issue to a separate PR/issue. I was just getting the discussion started since it was something I was trying to do, and wasn't able to successfully enable. I agree logging working should be the default if it turns out to be a binary switch, and with our non production environments the cost of logging is so low it's almost not worth mentioning, so no harm in having it enabled.
I've subscribed to issue 23 so I can monitor the progress on that front. Thanks for accepting my PR!

And thanks again for the PR and feedback. Super appreciated, @zxjinn ! 🍰


The solution button text color is kinda difficult to read

I think color in dark theme is also ugly, what color should we use?

How about these?

:orz:
Interesting idea, can you push the code so I can preview it?


Enhance the comment-attaching algorithm

This algorithm currently tests if the comment "belongs" to an AST tree if the range of the AST covers the comment. This end up giving the following result:
The comment of
a  //x

is attached to the ExpressionStatement, but the comment of
a; //x

is attached to the FunctionBody.

I think you mean ExpressionStatement, not FunctionBody?

I'm not sure which one should be attached to.

But the SpiderMonkey AST doesn't have FunctionBody. You've created an issue on esprima.

Ah, I see... It's the body of the program which is an Array...
I think It should be Expression and ExpressionStatement.

I shall close this now for the issue number bump.


chore(deps): update dependencies

update dependencies
remove husky, use .githooks

📝 https://github.com/JS-DevTools/karma-config does not support webpack 5


Prioritise typed overloads in commands

Right now our command system has an issue with multiple overloads:
[Command("a")]
public async Task OverloadedCommand(IUserMessage message, int argument)
[Command("a")]
public async Task OverloadCommand(IUserMessage message, string argument)

Since these overloads do not get prioritised based on what arguments were provided, the overload with the string parameter may get invoked instead of the overload with the integer parameter, but never the other way around.
One solution for this, as foxbot suggested, would be to add a 'Priority' attribute, to prioritise commands manually. Another solution would be to use the data provided by the type readers to choose the best overload to invoke.

Basically I'm thinking:

A user can specify a [Priority] manually
If lacking the attribute, the lib implicitly rates an overload with a string parameter the lowest priority (if that exists), which could at least take care of the simplest case (which is the example given)

Also:

What should happen if one overload has a [Priority] set and another doesn't?
What should happen if multiple overloads have the same [Priority] value set?

I'm thinking a mix of both would probably be best. We could defer to the typereader data in the case of a stalemate between priorities. In the case of no priority I'd say having a default '0' would be best.

Considering Typed over String first should be done, regardless, though. Though I definitely like what has been discussed, for the Priority attribute

How high is this on the priority list? Any progress being made? Quite annoying.

Right now I'm not sure how to implement the consideration for typed properties over untyped properties but implementing a PriorityAttribute should be fairly simple.


Update dependency versions

To resolve the dependabot alerts:

A developer build has not yet been created for this branch. Click here to go ahead and create the build...

CI Build Link:
http://sou-jenkins2.swinfra.net:8080/job/FileAnalysisSuite/job/FileAnalysisSuite~adaptersdk-schema~update-dependencies~CI


multiple_data_strategy.py throwing exception: your connection is rejected

Hey there,
I'm encountering this issue while running the unmodified multiple_data_strategy.py sample in paper trading mode. I have no other scripts running on the account. This is the exception:
Starting Portfolio Value: 100000.0
Exception in thread Thread-7:
Traceback (most recent call last):
File "/usr/lib/python3.7/threading.py", line 917, in _bootstrap_inner
self.run()
File "/usr/lib/python3.7/threading.py", line 865, in run
self._target(*self._args, **self._kwargs)
File "/home/pi/.local/lib/python3.7/site-packages/alpaca_backtrader_api/alpacastore.py", line 483, in _t_streaming_prices
streamer.run()
File "/home/pi/.local/lib/python3.7/site-packages/alpaca_backtrader_api/alpacastore.py", line 132, in run
self.conn.run(channels)
File "/home/pi/.local/lib/python3.7/site-packages/alpaca_trade_api/stream2.py", line 275, in run
loop.run_until_complete(self.subscribe(initial_channels))
File "/usr/lib/python3.7/asyncio/base_events.py", line 584, in run_until_complete
return future.result()
File "/home/pi/.local/lib/python3.7/site-packages/alpaca_trade_api/stream2.py", line 250, in subscribe
await self.data_ws.subscribe(data_channels)
File "/home/pi/.local/lib/python3.7/site-packages/alpaca_trade_api/stream2.py", line 104, in subscribe
await self._ensure_ws()
File "/home/pi/.local/lib/python3.7/site-packages/alpaca_trade_api/stream2.py", line 88, in _ensure_ws
await self._connect()
File "/home/pi/.local/lib/python3.7/site-packages/alpaca_trade_api/stream2.py", line 52, in _connect
raise Exception(f"Error while connecting to {self._endpoint}:"
Exception: Error while connecting to wss://data.alpaca.markets/stream:your connection is rejected while another connection is open under the same account

Hi, so working with multiple data is currently an issue.
there's a branch solving this issue but it's not yet perfect.
you could install it and work with it to run this example code.
install it like so:
pip install -U git+https://github.com/alpacahq/alpaca-backtrader-api@shared_websocket_streamer

Are you guys talking about running it on multiple tickers?

yes

Thanks, this got backtesting up and running.


chore: updating dependabot to support GHA, cargo and docker packages

Hi @WadeBarnes , this is ready for review now.

@WadeBarnes — what do you want to do about the failing test?  Looks unrelated obviously. Go ahead with the merge?

Yes, you can go ahead with the merge.  I added a ticket to look at the build.


test_no_replication_across_clusters unstable

Replication test test_no_replication_across_clusters unstable: failed 4 times in a row, blocking release CI: https://github.com/canonical/mysql-k8s-operator/actions/runs/4667631230/jobs/8265947160
Marked as unstable in #201

Just for the history: those PR merged and released after re-triggering.
I think, I know the reason of instability for those test:
We start 2 DB clusters there: mysql and "anothermysql" each of them eats 50% RAM, causing OOM which triggers issues due to know Workload container restart and lack of pebble socket connection:
  File "/var/lib/juju/agents/unit-another-mysql-1/charm/venv/ops/pebble.py", line 252, in connect     self.sock.connect(self.socket_path) FileNotFoundError: [Errno 2] No such file or directory
See: https://github.com/canonical/operator/issues/844#issuecomment-1488268566
And the fix is planned here: https://github.com/canonical/mysql-k8s-operator/issues/192 we must lock (with contsraints) RAM usage of each mysql and anothermysql to 1GB and not 50% RAM.

I am really curious, what is using ~6-8G there during the tests?
We are not importing any data into database in those tests, so both DBs are empty. Sure, innodb_buffer_pool_size=50% RAM, but MySQL should not use all of them, it is a maximum allowed value.
Any ideas why MySQL eats 6 GB RAM? Also I can't explain why RAM usage is not decreased on workload charm restart (gap on the picture?).
@carlcsaposs-canonical @delgod


MAINT: versioneer

Coverage decreased (-15.7%) to 77.246% when pulling 69d33d1338733d15d3fdf6057f62dd4d5d2c2c40 on thermokarst:versioneer into ad67afa24748e516366cf6943620a7ef233166ea on qiime2:master.

Coverage decreased (-15.7%) to 77.246% when pulling bb2e016e25ded476afdeb2b4715077bfb54e380f on thermokarst:versioneer into ad67afa24748e516366cf6943620a7ef233166ea on qiime2:master.

Coverage remained the same at 92.931% when pulling bbf9c60d3592b1166fd6e23256bd648dc8fa9db6 on thermokarst:versioneer into ad67afa24748e516366cf6943620a7ef233166ea on qiime2:master.

Coverage remained the same at 92.931% when pulling ad162ef68e059809ebab3c0ae10141222566ec71 on thermokarst:versioneer into ad67afa24748e516366cf6943620a7ef233166ea on qiime2:master.

Saving for later. BRB.