[WIP][SPARK-18890][CORE](try 3) Move task serialization from the TaskSetManager to the CoarseGrainedSchedulerBackend

What changes were proposed in this pull request?
See https://issues.apache.org/jira/browse/SPARK-18890
In the case of stage has a lot of tasks, this PR can improve the scheduling performance of 15%
The test code:

val rdd = sc.parallelize(0 until 100).repartition(100000)
rdd.localCheckpoint().count()
rdd.sum()
(1 to 10).foreach{ i=>
  val serializeStart = System.currentTimeMillis()
  rdd.sum()
  val serializeFinish = System.currentTimeMillis()
  println(f"Test $i: ${(serializeFinish - serializeStart) / 1000D}%1.2f")
}

and spark-defaults.conf file:
spark.master                                      yarn-client
spark.executor.instances                          20
spark.driver.memory                               64g
spark.executor.memory                             30g
spark.executor.cores                              5
spark.default.parallelism                         100
spark.sql.shuffle.partitions                      100
spark.serializer                                  org.apache.spark.serializer.KryoSerializer
spark.driver.maxResultSize                        0
spark.ui.enabled                                  false
spark.driver.extraJavaOptions                     -XX:+UseG1GC -XX:+UseStringDeduplication -XX:G1HeapRegionSize=16M -XX:MetaspaceSize=512M
spark.executor.extraJavaOptions                   -XX:+UseG1GC -XX:+UseStringDeduplication -XX:G1HeapRegionSize=16M -XX:MetaspaceSize=256M
spark.cleaner.referenceTracking.blocking          true
spark.cleaner.referenceTracking.blocking.shuffle  true

The test results are as follows
The table is out of date, to be updated

SPARK-17931
941b3f9

17.116 s
21.764 s

How was this patch tested?
Existing tests.

Test build #73769 has started for PR 17139 at commit bfa285b.

Test build #73771 has started for PR 17139 at commit af5fc9f.

Test build #73775 has started for PR 17139 at commit a06f8c8.

Test build #73776 has started for PR 17139 at commit 6874d1e.

Test build #73775 has finished for PR 17139 at commit a06f8c8.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class SerializeTask(task: TaskDescription) extends CoarseGrainedClusterMessage
 class SerializeTaskEndpoint(override val rpcEnv: RpcEnv) extends RpcEndpoint with Logging

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73775/
Test FAILed.

Merged build finished. Test FAILed.

Test build #73776 has finished for PR 17139 at commit 6874d1e.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class SerializeTask(task: TaskDescription) extends CoarseGrainedClusterMessage
 class SerializeTaskEndpoint(override val rpcEnv: RpcEnv) extends RpcEndpoint with Logging

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73776/
Test FAILed.

Test build #73769 has finished for PR 17139 at commit bfa285b.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class SerializeTask(task: TaskDescription) extends CoarseGrainedClusterMessage
 class SerializeTaskEndpoint(override val rpcEnv: RpcEnv) extends RpcEndpoint with Logging

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73769/
Test FAILed.

Test build #73771 has finished for PR 17139 at commit af5fc9f.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class SerializeTask(task: TaskDescription) extends CoarseGrainedClusterMessage
 class SerializeTaskEndpoint(override val rpcEnv: RpcEnv) extends RpcEndpoint with Logging

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73771/
Test FAILed.

Test build #73848 has started for PR 17139 at commit 4d9b666.

Test build #73848 has finished for PR 17139 at commit 4d9b666.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class SerializeTask(task: TaskDescription) extends CoarseGrainedClusterMessage
 class SerializeTaskEndpoint(override val rpcEnv: RpcEnv) extends RpcEndpoint with Logging

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73848/
Test PASSed.

ping @kayousterhout  @squito

Why is the time improvement so much larger here than in the other PR?

Added the multi-threaded code for serialization TaskDescription .

Can you also post the time differences for some smaller jobs (e.g., 100 tasks, 1000 tasks, 10K tasks) to get a sense of how this varies with size?

@kayousterhout The test report has been updated.

Test build #74270 has started for PR 17139 at commit 8fbe15c.

Test build #74270 has finished for PR 17139 at commit 8fbe15c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74270/
Test PASSed.

Merged build finished. Test PASSed.

@witgo Are you still working on this?

@jiangxb1987 ,Yes do you have any questions?

Please bring this PR up to date, then maybe some guy can review this. :)

This should probably just be closed

We are closing it due to inactivity. please do reopen if you want to push it forward. Thanks!


Not fully downloading file.

The download gets sent to the server and only downloads about 64kb of information before stopping. I removed everything and reinstalled. Still not working.
Window 10 Pro x64
AriaNg v1.1.6 (d22d61c)
Aria2 1.35.0

please go to https://github.com/aria2/aria2/issues for help


X11: Update mouse position in XI_Motion event (closes #241)

Implements the fix proposed in #241

Thanks!


Expand common_mean_bayes() to include test for differing precision

I just committed a common_mean_bayes() function to ipmag.py that implements the Heslop and Roberts (2018) common mean/reversal test.
I have utilized their code to implement the version of the test that us for populations with common precision (Appendix A; jgrb52994-sup-0003-Data S2.unk).
We should also implement the version with differing precisions (could be a flag in the same function), but this involves import of a special library quadpy which we may not want to make a dependency of ipmag.py. The version of the test is documented in Appendix B and SI materials: jgrb52994-sup-0004-Data S3.unk.

Putting this here as a feature request issue as a reminder to myself to work on this down the line.


Selecting a sprite with big texture on it is very slow

Godot 2.1.3, Godot 3.0 master
Windows 10 64 bits
I just noticed that selecting a sprite having a big texture on it in the viewport freezes the editor for a second or two. On comparison, selecting a smaller sprite is instant, scaled or not. Selecting in the tree doesn't have this issue.
It's unclear to me why this would be so slow?
Repro:

Create a big PNG, like 4000x4000 or above
Assign it on a sprite (notice first freeze as you assign it, but it's because of a load)
Deselect anything
Now click on the sprite in the viewport: notice short freeze

PR #15012 tries to solve this in a different way than #12312
I dont know wich one is better since i sincerely dont understand how #12312 works


Why use a web application policy to grant access in Step 5?

Is there a required reason why this is a recommended way to grant AAD users access to the web application? It feels fairly heavy handed to grant them access (especially full access) via the web app policy rather than individually within the site collections they need to access. I kind of thought these policies were mostly for service accounts and such, am I misunderstanding a requirement for implementing this?

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

ID: f079fa08-284c-3f14-be4a-1f32d5b0e7b9
Version Independent ID: 707cbeee-d311-d430-6119-e9ae8b8aeae5
Content: Using Azure AD for SharePoint Server Authentication
Content Source: Enterprise/using-azure-ad-for-sharepoint-server-authentication.md
Service: o365-solutions
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

@ferringer - hey John, this is really for testing purposes. This article really only serves to educate how to perform and confirm an initial configuration, it does not intend to educate users to the nuances of SharePoint authorization and permission models.
We restricted this article to give specific instructions to get a user started, not to provide comprehensive product documentation on how authorization works in various contexts.
There is a lot of variability in the ways that zones, web applications, and site collections can be provisioned. This article does not propose to clarify all the various combination. Rather, this article shows how to configure a single user to auth against AAD. This topic could cover a book, and nearly has (see Steve Peschka's blog as an example). The web application policy is used in this context because it goes against the STS and is much less complicated than trying to instruct how to restrict access to a list.
There are a ton of additional considerations such as using groups, creating custom claims providers, splitting auth per zone, using different host headers per site collection per zone, configuring site collection permissions, application permissions, list permissions, breaking inheritance, search experience for claims users, user profile property mapping, user profile import for claims users, app service for claims users, workflows for claims users... we could write about this extensively and still not cover all customer use cases. We cannot fit them all here, if you see that we missed any we welcome that feedback.
If there are additional improvements to this article that you feel are needed, please consider forking this repo and creating a pull request for your changes.


Use proper fields for quest complete

While troubleshooting some work around quest completion, I noticed I kept getting a message in my logs stating Unknown quest end reason: 12389213 or similar numbers. After doing some investigation, the third parameter here is actually the number of frames that were used as part of the quest, and the final parameter is the reason parameter.
Happy to walk through this if necessary, also only tested on Linux Proton, but I don't think that would matter for something like this.

Yep, I've been using this update in my personal mod that relies in quest complete/failure

Alright, thank you, looks good


Kay added unix and git notes

Worked with instructor Beth Long on testing out of Unix and Git for Everyone workshop.

Well done!


Comprehensive Guide on Providers and Output Adapters

This PR contains daily documentation updates based on the following suggestion:
Introduce a new documentation page titled 'Understanding Sublayer Providers and Output Adapters' to provide comprehensive details about various AI providers supported by Sublayer (OpenAI, Claude, and Gemini) and different types of output adapters (SingleString, NamedStrings, etc.). This page should explain how to configure Sublayer to use these providers and the use of output adapters in generating structured responses.
description of file changes: Add a new file: docs/concepts/providers_and_adapters.md.
Content: Explain what each provider is used for, how to set them up using configuration options, and how various output adapters in the Sublayer framework help in formatting the output from LLMs. Include code examples and practical use cases of different adapters.

pros: I'm glad it's trying to add more about adapters and providers
cons: why is it trying to lump them together. haha, i think this is more a note about the intelligence/power of the model than anything of significant value to the script.


Invalid 2FA Code, only when tests are ran on Mac Mini server.

Describe the bug
I use this library for 2FA in my automated tests.  The application is stored on our own server (Mac Mini) and uses BrowserStack to run the tests, the 2FA code is always invalid.
Expected behavior
2FA code  to be valid. If I run the them locally or on the cloud via my machine, 2FA always works.
Additional context
I understand this is a very niche issue. A point in the right direction would be much appreciated. Some code below:
const otplib = require('otplib');

const secret = "secret here";

module.exports.generate2FACode = function() {
    return otplib.authenticator.generate(secret);
}

async enterTwoFactorCode () {
        try {
            const element = await this.findE2eElement("twoFactorCodeInput");
            await this.focusOnElement(element);
            await element.clear();
            await element.sendKeys(twoFactorAuth.generate2FACode());
        } catch (error) {
            await this.logFailure();
            assert.fail("Unable to enter Two Factor code.");
        }
    }

@George-BC
Just getting the usual first culprit out of the way:
Is the clock on the mac mini and browserstack showing correct time?
Since it is time based, the generated code will differ.

@yeojz I will look into this and get back to you in next day or so, thanks!

@yeojz The Mac Mini was 3 minutes behind, changing it fixed the issue. Thanks for the help!


[Consistency] Raises section within linalg

Currently, the only function that specifies a Raises section in Extensions / Linear Algebra Functions is matrix_power.
I would assume that we would want to either specify this for all the functions (which would be quite tricky, as it would involve defining when is a spectrum non-singular or when is a matrix SPD), or for none of them.
cc @rgommers @mruberry

I don't know if this is clearly spelled out, but generally when exceptions are not explicitly required by the spec, it means libraries are allowed to raise an exception or to do something else, which may vary depending on how the libraries work. Even so, I don't know if the current spec gives proper allowances for libraries potentially raising exceptions for things like that if they want to.

Raises can simply be removed here I think. The two conditions are:

if the innermost two dimensions of x are not the same size (i.e., form square matrices).
if n is less than zero and a square matrix is not invertible.

Those are also captured by the conditions given in the parameter descriptions, so Raises adds nothing of interest.


Added 'Streaming-Pointcloud' example code

Kinect point cloud merger code
Examples:
Touch Designer
Unity
Three.js

 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.:x: mromeolisten sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Helm repo no longer available!

Hi,
Seems your repo is out of date now, please update how to install today, thanks!
[root@kube-1 hdfs]$ helm repo add incubator https://kubernetes-charts-incubator.storage.googleapis.com
Error: repo "https://kubernetes-charts-incubator.storage.googleapis.com" is no longer available; try "https://charts.helm.sh/incubator" instead
Cheers,
Dan

More info:
[root@kube-1 hdfs]$ helm repo add incubator https://charts.helm.sh/incubator
"incubator" has been added to your repositories
[root@kube-1 hdfs]$ helm dependency build charts/hdfs-k8s
Error: could not find charts/hdfs-k8s: stat charts/hdfs-k8s: no such file or directory

The stable and incubator repositories of the Helm charts have been moved to a new location. You must updated URI in charts.yaml (or requirements.yaml) to point to the new repositories in order to let the Helm dependency resolver find the correct location.
Name	Old Location	New Location
stable	https://kubernetes-charts.storage.googleapis.com	https://charts.helm.sh/stable
incubator	https://kubernetes-charts-incubator.storage.googleapis.com	https://charts.helm.sh/incubator

you should change the repo of hdfs-k8s/requirements.yaml URI.
dependencies:
  - name: zookeeper
    version: "1.3.1"
    repository: https://charts.helm.sh/incubator  # change this
    condition: condition.subchart.zookeeper
    tags:
      - ha
      - kerberos

So since this project is dead, the only way we can use the charts here is to maintain a fork with the changed URLs. Gross...


Refactor Codebase to Improve Code Quality

address parts of DAT-333, https://rapiddweller.atlassian.net/browse/DAT-332


add http://101.ru/radio/channel/43 station

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

station added


[hi] Localize /en/docs/reference/glossary/limitrange.md

Prerequisites:

If this is your first contribution, visit Contribute to K8s docs for information on how to get started.
Review a few https://github.com/kubernetes/website/issues/35600#issuecomment-1214215859 to see which
content you have to change and which you leave unchanged.

Procedure:

Make sure your base, target branch is main and create a working branch called hi-limitrange.md.
Create a file called content/hi/docs/reference/glossary/limitrange.md
Copy the content of
https://github.com/kubernetes/website/blob/main/content/en/docs/reference/glossary/limitrange.md
to the file you created
Localize the content.
Push your change and create a pull request.

Completing this feature issue fixes part of https://github.com/kubernetes/website/issues/35600

Note:- Don't forget to join slack kubernetes-docs-hi channel if facing any problem!

Note:- As we are creating this issues for new contributors to help them. Please follow this guideline that one time get assigned one issue only. After completion that you can take another one. Follow this approach! This issue is for first-time-contributor only

/language hi
/triage accepted

/assign

Ahan! A great approach to helping new contributors. Sorry I missed that point.
/unassign

Hello Team, This is my first issue on kubernetes/website. I would like to work on it
/assign

@chiukapoor Keep working on this issue. Feel free to raise a PR


[Extension Bug] Atomic Data error after completing setup

Steps To Reproduce

Install atomic extension
Press continue on setup
Error :(

ReferenceError: WebSocket is not defined

wr:index.js:9:45398

---
 6: `+I+c.join(`,
 7: `+I)+`
 8: `+i+"}":"{"+c.join(",")+"}",I=i,o}}Wt.exports=function(t,e,r){var n;if(I="",Ie="",typeof r=="number")for(n=0;n<r;n+=1)Ie+=" ";else typeof r=="string"&&(Ie=r);if(ie=e,e&&typeof e!="function"&&(typeof e!="object"||typeof e.length!="number"))throw new Error("JSON.stringify");return $e("",{"":t})}});var Gt=se(rt=>{rt.parse=Ft();rt.stringify=qt()});var Zt=se((Jn,Yt)=>{var Le=typeof JSON!="undefined"?JSON:Gt();Yt.exports=function(t,e){e||(e={}),typeof e=="function"&&(e={cmp:e});var r=e.space||"";typeof r=="number"&&(r=Array(r+1).join(" "));var n=typeof e.cycles=="boolean"?e.cycles:!1,o=e.replacer||function(c,a){return a},s=e.cmp&&function(c){return function(a){return function(l,f){var p={key:l,value:a[l]},m={key:f,value:a[f]};return c(p,m)}}}(e.cmp),i=[];return function c(a,l,f,p){var m=r?`
 9: `+new Array(p+1).join(r):"",b=r?": ":":";if(f&&f.toJSON&&typeof f.toJS (VERY LONG)
10:
11: `+r)}i()},[t]),t.error?t.error.message:n}async function Bn(t,e,r){let n=await r.getProperty(t),o=e.toString();switch(n.datatype){case"https://atomicdata.dev/datatypes/atomicURL":o=`[${e}](${e})`;break;case"https://atomicdata.dev/datatypes/resourceArray":o="",X(e).map(s=>{o=o+`[${s}](${s}),`});break;case"https://atomicdata.dev/datatypes/timestamp":o=ut(e).toLocaleString();break}return`
12:
---

At.setWebSocket:index.js:9:48511
At.setServerUrl:index.js:9:48427
At:index.js:9:45790
Object.<anonymous>:index.js:13:628
Module._compile:loader:1101:14
Object.Module._extensions..js:loader:1153:10
Module.load:loader:981:32
Function.Module._load:loader:822:12
Module.require:loader:1005:19

I think I know how to fix it: conditional websocket support in @tomic/lib

It seems to work just fine locally with yarn dev, didn't see an error.
Two possible reasons:

Errors in production are handled differently then in development
WebSockets aren't available in production, but they are in development

After my PR has been merged, my issue continues to happen. I've waited a couple of hours after the updated version was published, I've uninstalled the extension, but it seems my bug isn't fixed. This could happen because my fix wasn't good, or because I'm somehow still running the older version.
The biggest hurdle for me to fixing this, is that the problem does not happen while running npm run dev. Could the dev version / runtime differ significantly? Could it be that some runtime errors are hidden for me during development? Or could it be that WebSockets is only available in development?
@mathieudutour could you perhaps help me with this?

you could try with npm run build which will build the prod version, but yeah WebSocket isn't a NodeJS global so it's expected that it's not available

@mathieudutour

you could try with npm run build which will build the prod version, but yeah WebSocket isn't a NodeJS global so it's expected that it's not available. Keep in mind that the extensions are running in a NodeJS process and not in a browser

It makes sense that it's not available, I understand it's running in Node, but it's kind of weird that it doesn't crash in npm run dev mode, right? Is WebSocket available in dev mode, or is the error somehow handled and hidden? It's not appearing in the logs, or popping up.
I ran npm run build, which succeeded, but I don't know where the built version is stored (can't find a dist or .js anywhere after running build).

Ah indeed, I found what's happening. WebSocket and window are indeed available in dev mode because react-dev-tools requires them. I'm going to add a warning whenever those global are accessed so that people will know it'll break in prod.
So to fix it, you can do something like this:
const customGlobal = global as any;
customGlobal.WebSocket = require("ws");

and add a dependency on ws.
Also, in order to test with the "prod" environment, you can go to the Raycast preferences > Advanced > Use Node production environment.
Sorry for the confusion

Thanks for the help and explanation! Turning on the Node Production environment setting makes the error appear, which helps me fix the issue.

Let me know when it's ready to go :)

It's ready to go @mathieudutour!
Thanks!

It's working!


random stdlib does not work

The random package is not supported at all, but will likely come in as part of bringing in the std lib. Only _random is and even that might not work, until we seed the custom randomness.


Cannot see animations after first when used on consecutive slides in instructions

If there is a single animation in a set of instructions or (I think) if there are slides of text separating two animations, all is well. When animations occur on consecutive slides, however, only the first is displayed.  If I 'reset animation', I can get a partial display which shows only the remote part of the animation, but the local side is never shown.
In some cases (especially in the final levels), it is not easy to determine what the animation looks like from the textual description alone.
I'm using the chromium browser on linux.
chromium: 129.0.6668.100-1
linux: 6.11.2-arch1-1 #1 SMP PREEMPT_DYNAMIC Fri, 04 Oct 2024 21:51:11 +0000 x86_64 GNU/Linux
I'm not sure what debugging info to provide since the problem isn't a response to commands in the console but appears partway through the instructions for the various levels affected.

Hrmmm that's odd, this might be a chromium or linux issue? I just checked on Google Chrome and its working fine.
Could you try using Chrome on linux and see if that fixes it?


List of custom short-codes used by hugoplate

Is there a list somewhere of custom short-codes used by this template? Is there documentation for that?

Hey, please check the "Shortcode" tab from this page https://gethugothemes.com/hugo-modules

Thanks


esrally: error: --target-hosts and --client-options must define the same keys for multi cluster setups.

Rally version elastic/rally latest version

The command is as shown above. Because the destination elasticsearch uses xpack, I use client-options to prevent a security exception.
    ____        ____
   / __ \____ _/ / /_  __
  / /_/ / __ `/ / / / / /
 / _, _/ /_/ / / / /_/ /
/_/ |_|\__,_/_/_/\__, /
                /____/

[ERROR] Cannot race. 2.

Getting further help:
*********************
* Check the log files in /rally/.rally/logs for errors.
* Read the documentation at https://esrally.readthedocs.io/en/2.6.0/.
usage: esrally [-h] [--version]
               {race,list,info,create-track,generate,compare,download,install,start,stop}
               ...
esrally: error: --target-hosts and --client-options must define the same keys for multi cluster setups.
* Ask a question on the forum at https://discuss.elastic.co/tags/c/elastic-stack/elasticsearch/rally.
* Raise an issue at https://github.com/elastic/rally/issues and include the log files in /rally/.rally/logs.

Even after looking at the official documentation, I can't quite understand... The official documentation only says to use it as above. What should I do?
target-hosts is only one
im use elasticsearch eck!

Thanks very much for your interest in Rally.
This appears to be a user question, and we'd like to direct these kinds of things to the forums. If you can stop by there, we'd appreciate it. This allows us to use GitHub for verified bug reports, feature requests, and pull requests. There's an active community in the forums that should be able to help get an answer to your question. As such, I hope you don't mind that I close this.
However, I think the issue is that your --target-hosts parameter is malformed, and parses to a JSON dictionary somehow? Which means Rally thinks you have multiple clusters. It's not easy to tell as you've hidden the important part. :)

@pquentin
hi thank you so much for replying
However, the target host has only one cluster, not multiple clusters.

Hi @sangkyunkim89 , sorry about the dead link. You can post the question here and tag it with rally so that we can help out. As @pquentin mentioned, please be sure to include the full invocation so we can help out.


Pod Horizontal automatic expansion

apiVersion: autoscaling/v2beta2
kind: HorizontalPodAutoscaler
metadata:
  name: hpa-yhz
  namespace: test
spec:
  minReplicas: 2
  maxReplicas: 20
  metrics:
  - type: Resource
    resource:
       name: cpu
       target:
          type: Utilization
          averageUtilization: 40
  scaleTargetRef:
    apiVersion: apps/v1
    kind: Deployment
    name: php-apache
  behavior:
    scaleDown:
      stabilizationWindowSeconds: 300
      policies:
      - type: Percent
        value: 100
        periodSeconds: 15
    scaleUp:
      stabilizationWindowSeconds: 180
      policies:
      - type: Percent
        value: 100
        periodSeconds: 15

Question:
Stop the pressure test after making the CPU utilization of the Pod unstable through the pressure test. Why does the HPA continue to expand and shrink the number of Pods when the CPU utilization is 0 ?

/kind support


usbhost example won't compile due to cdc_acm / cdc_ecm mixture

The cmakefiles refer to a file named class/cdc/usbh_cdc_ecm.c
https://github.com/bouffalolab/bouffalo_sdk/blob/0f07c3b8ecbed6b5eda90cc794611c6dd9077851/components/usb/cherryusb/CMakeLists.txt#L59
but in the repository only a file named class/cdc/usbh_cdc_acm.c is actually included:
https://github.com/bouffalolab/bouffalo_sdk/tree/master/components/usb/cherryusb/class/cdc
This prevents the usbhost demo from being compilable.

Perhaps ECM should just be disabled in the project config:
https://github.com/bouffalolab/bouffalo_sdk/blob/0f07c3b8ecbed6b5eda90cc794611c6dd9077851/examples/peripherals/usbhost/proj.conf#L4
It's not enabled by default in the demo itself, anyway:
https://github.com/bouffalolab/bouffalo_sdk/blob/0f07c3b8ecbed6b5eda90cc794611c6dd9077851/examples/peripherals/usbhost/usb_host.c#L12

Hi,
Do you got the new release version of CherryUSB running on this SDK?

Yes. But I only tested the host part.

Yes. But I only tested the host part.

I've ported the CherryUSB lib, but seems that interrupt isn't happening. Could you share your port?
I'm facing a strange problem with my composite device(MSC+CDC). Sporadically, in the middle of transfer, the device is enumerated again but without a CPU reset. I will update the lib to test and open a case, but is really a mess because the problem should be with BL616 port. And the Boufallo support is worse and don't care about a single customer issue. Here on GitHub, there's no answers to the majority of issues opened. The SDK has weeks and months without an update. The documents are missing description about USB peripheral.
BL616 is a great device, but the more important thing is a good ecosystem and SDK are missing. I'm really excited to the esp32-p4 release.
Do you get to debug BL616 through JTAG interface?

See https://github.com/harbaum/bouffalo_sdk/
This repo includes an updated cherry USB as a submodule.
Yes, the chip is cool and has a lot of potential and the SDK support is rather disappointing ...
I have not tried to use jtag.


PyUnicode_AsUTF8 ?

This is a "suboptimal" name for the function in Unicode.cs .

The C# function returns a string which is UTF16
The CPython API call returns a pointer to an UTF8 string (C# byte*) so the callers should be also named different

The correct names in the C# API would be

PyUnicode_FromUTF8
PyUnicode_FromUTF8_
PyUnicode_FromUTF8Raw

or "_ByUTF8" as there a parts of the CPython API which want a pointer to an UTF8 buffer ( like PyRun_StringFlags etc. ) and using those would imply calling the "As" name
Besides renaming, unfortunally can using CPythons PyUnicode_FromUTF8 API allocate an UTF8 string in cases where the internal format was UTF16.  This additional representation seems to be cached for the lifetime of the CPython string.
Looking at the CPython sources there might be a way to directly obtain an C# compatible UTF-16 pointer if that is the format used. But as there a much "inlines" there I'm not shure if that is possible from C# code or a C module would be needed for that.
According to https://github.com/python/cpython/blob/main/Include/cpython/unicodeobject.h CPython tries to store the smallest form of representation, which makes sense in cases when memory is of concern. But not in conjunction with C# ...
Enforcing UTF-16 where CPython "decides" how to store in a form that C# can just memcpy by modifying CPython might be worth a try - if modules are using the CPython API correctly nothing should break...

Since 3.7 CPython is evolving to use UTF-8 as the standard for everything, in 3.15 it will become default. The file text reader for open() default to utf-8.
As for the name of the function. Most of the functions in CPython/CPythonAPI are LibraryImport wrappers around the CPython C-API. The names reflect the names of the CPython API's we're calling. PyLong_AsLongLong for example returns a C# long. I don't really want to change the names of these functions as it'll create more confusion as to which C-API they're referring to.
All of these functions are internal to CSnakes, this folder is the internal wrapper around the C-API with some safety and safehandle marshallers.
Using UTF-16 in CPython, whilst possible, is unlikely. We should probably add a test to see what happens when you run CPython in UTF-16 mode and return a string back, but I wouldn't try and optimize this path.

So the C# way would be to define another .dll call with the same name but giving back a byte * ?? Like:
    [LibraryImport(PythonLibraryName, EntryPoint = "PyUnicode_AsUTF8")]
    private static partial byte* PyUnicode_FromUTF8_(PyObject s);

Since 3.7 CPython is evolving to use UTF-8 as the standard for everything, in 3.15 it will become default. The file text reader for open() default to utf-8.**

Thanks for that info. For communicating with the outside world I'm fine with that, but for embedding Python into Windows using UTF-16 internally would be the solution to trade memory for speed.

Using UTF-16 in CPython, whilst possible, is unlikely.

In my current case I'm using SQLAlchemy and want the GUI sorting via LINQ etc. . C# would create for every comparison one or two UTF-16 strings just to throw them away...
Hope that the CPython core team in their wisdom doesn't disable the ability to enforce UTF-16 internally. I read the PEP in a different way - it might be easier to enfore UTF-16 in a customized CPython build.

We should probably add a test to see what happens when you run CPython in UTF-16 mode and return a string back, but I wouldn't try and optimize this path.

I don't know if there are UTF-8 strings which use more bytes due to escaping than UTF-16 - maybe chinese etc. or emojis? But IF such a string exist and is given, the current (3.13) CPython WILL use UTF-16 internally...


Virtual attributes and mass assignment

I can define a virtual attribute in an ActiveRecord model class like
def virtual_attr
  ...
end

def virtual_attr=(val)
  ...
end

It allows me to make calls like model.update(virtual_attr: 'Hi!')
When I try to do the same for a decorated model class it fails in internal _assign_attribute method. I think the reason is that it tries to do send(:virtual_attr=) for the model object and does not find such a method there.
What is the recommended way to create virtual attributes in decorator classes so that I could do decorated_model.update(virtual_attr: 'Hi!')?

Any news on this? I'm having this same problem.

Honestly... isn't this a bit out of scope? What prevents you from accessing the non decorated record directly?

What is the recommended way to create virtual attributes in decorator classes so that I could do decorated_model.update(virtual_attr: 'Hi!')?

I'm not sure if there could be any 🤔
Here, #update is called on an underlying model unless the method is explicitly defined in a decorator (in Darper, it's not). A model does (and should) know nothing about its decorator. Thus, its #update can't get access to decorator's attributes.
Another matter to be considered is separation of concerns. #update is a part of persistence interface, meanwhile decorators stand for representation which is the reverse side of a model. That's why it's hardly possible for that interface will ever be implemented here in Draper.

…Unless the method is explicitly defined in a decorator…

Thus, if you still eager to, you may define #update for your basic decorator, based on the code from Active Record.


GraphQL Playground

Is it possible to also create a graphql playground? Currently the /graphql url does not return anything.

Not sure if you are using Apollo, but the Apollo chrome extension will also intercept requests to mirage. It includes a playground that works quite well.

@fazulk, that's cool. Thanks for sharing! I'll have to check that out some time.

Not sure if you are using Apollo, but the Apollo chrome extension will also intercept requests to mirage. It includes a playground that works quite well.

@AlexandrosVasileiou, I'm having trouble understanding the need for this. Maybe you could provide some reasons why you would like this for Mirage?

Is it possible to also create a graphql playground? Currently the /graphql url does not return anything.


Testing in Another Country

Hello Team,
I am developing an iPhone application with integrating payment method brain-tree venmo. I am facing some issue regarding show venmo icon in drop in. I know the venmo is not available in India. Please let me know any alternative way to check integration or doing test payment in India.
Looking forward to hear from you.
Thanks & Regards
Sham Dhiman

Any update on this? Looking same here...

No @chetan-duke


Add stable actions on useNavigation

This solves: https://github.com/react-navigation/hooks/issues/40

I think this is not the responsibility of react-native-hooks since this changes the semantics of how the navigation object behaves. Refactoring between props.navigation and useNavigation should not change the behaviour or break code. This kind of change should be done in React Navigation core.
Also, we should not hardcode these methods on the navigation object. As you pointed out, methods like openDrawer will now exist on stack navigator which shouldn't be the case. It also means that for navigators that are not handled here will behave differently, for example, tab navigator has a jumpTo method which won't be stable like all other methods here.
A better solution is to send a PR to core to make these methods stable :)

There's an issue already closed on core repo for this:
https://github.com/react-navigation/core/issues/71
This should be here because its too dificult to make that change on core right now and also because v5 it's coming and this package will be deprecated.

Doing more research I found there's too many posible bugs with this implementation.
More info here: https://github.com/facebook/react/issues/16956
Maybe it's just simpler let this as it is and add a warning so users can just add this helper to theirs code base:
function useStableCallback(cb: (...args: any) => any) {
  const ref = useRef(cb);

  useLayoutEffect(() => {
    ref.current = cb;
  }, [cb]);

  return useCallback((...args) => ref.current(...args), [ref]);
};

its too dificult to make that change on core right now

I don't think it's too difficult. The action creators are created in one place it should be possible to use a similar approach by storing these in a cache object instead ref.
It's more work but it's also correct and handles way more cases you can handle here without complicating code.

and also because v5 it's coming and this package will be deprecated

That doesn't mean we should make it inconsistent with the version of React Navigation we use it with.
Anyway, as I already mentioned, there are many issues with this approach.

Hardcoding shouldn't be done because

There can be multiple navigators. e.g. currently your code doesn't handle jumpTo for tab. Custom navigators can have their own methods.
Users can provide their custom action creators. This doesn't handle it.
There are other methods in React Navigation as well. Methods like emit are not covered and hardcoding means any changes/additions will make this code incorrect.

It makes navigation object semantically different. You refactor from useNavigation to props.navigation and now the code breaks. This shouldn't happen.
Methods like getParam and isFirstRouteInParent should not be stable.

Good points, thanks @satya164
I'm closing the PR then

It makes navigation object semantically different. You refactor from useNavigation to props.navigation and now the code breaks. This shouldn't happen.

Agree on this, and also a reason I don't think we should fix it here but in core instead.
Temporarily, I think the best option would be to recommend users to fix this in userland, with a workaround like
function useNavigationRef() {
  const navigation = useNavigation();
  const ref = useRef(navigation);

  useLayoutEffect(() => {
    ref.current = navigation;
  }, [navigation]);

  return ref;
};

const navigationRef = useNavigationRef();
  useEffect(() => {
      if (isAuthenticated) {
          navigationRef.current.navigate("AuthenticatedNavigator")
      } else {
          navigationRef.current.navigate("UnauthenticatedNavigator")
      }
  }, [isAuthenticated,navigationRef])

What do you think?


[BUG] Closing a form should not log errors

I noticed that when I use the built-in OpenForm function Schema.slack.functions.OpenForm in a workflow, errors are logged if the form is closed by the user.
Steps:

User triggers the workflow
A form is displayed
User clicks the "Close" X button in the upper right corner
Slack logs errors (see below)

Expected outcome: Closing a form should not log errors, this is normal user behaviour. It should probably just be a "Info" level log message.
2024-02-21 10:13:13 [error] [Fn010N] (Trace=Tr06KVMVE2AF) Function 'Collect info in a form' (Slack function) failed
	user_closed_form
2024-02-21 10:13:14 [error] [Wf06KFKK0B5G] (Trace=Tr06KVMVE2AF) Workflow step 'Collect info in a form' failed
2024-02-21 10:13:14 [error] [Wf06KFKK0B5G] (Trace=Tr06KVMVE2AF) Workflow 'Post errors for a specific MSN' failed
	user_closed_form
2024-02-21 10:13:14 [error] [Fn06K90WLP5L] (Trace=Tr06KVMVE2AF) Function 'Post errors for a specific MSN' (workflow function) failed
	user_closed_form

Hi @tomas-zijdemans-vipps  thanks for reporting this 💯
I was able to reproduce this with the following steps

slack create test-form-close -t https://github.com/slack-samples/deno-issue-submission
cd test-form-close/
slack run & create the triggers
past the shortcut trigger in slack
run the workflow
When the from opens close it

I get the following error log
2024-02-21 11:38:25 [error] [Fn010N] (Trace=Tr06KYGW3FS7) Function 'Collect info in a form' (Slack function) failed
	user_closed_form
2024-02-21 11:38:25 [error] [Wf06LB9A4DGR] (Trace=Tr06KYGW3FS7) Workflow step 'Collect info in a form' failed
2024-02-21 11:38:25 [error] [Wf06LB9A4DGR] (Trace=Tr06KYGW3FS7) Workflow 'Submit an issue' failed
	user_closed_form
2024-02-21 11:38:25 [error] [Fn06LMD59R24] (Trace=Tr06KYGW3FS7) Function 'Submit an issue' (workflow function) failed
	user_closed_form

I will report this internally find out if these is new behavior

@tomas-zijdemans-vipps - Following up here, we've confirmed that this is in fact new behavior, but I agree with your reasoning that this seems more correctly an "Info" level log message. Working on getting some more detail here from the Backend teams that implemented the change.

@tomas-zijdemans-vipps - Following up as promised; the current behavior helps us distinguish between workflows that are waiting for user action vs aborted by user vs stuck due to unknown issues. I've given the feedback internally that the scenario of closing a form would be more useful as an info-level statement but for now, I don't have much else to report on this, so I'm going to close this issue up!


Render hooks not always called by .RenderString

v0.71.0, the currently latest version.

There are a couple of similar reports about render hooks not called by .RenderString, but I think this is another variant.
In hope to make debugging easier, I boiled down my fairly complex site to about the minimum reproducing the issue: merchants-www-debug.tar.gz (1.1 kiB).
I build with hugo server --gc --renderToDisk, pages are then http://localhost:1313/pages/test.html (works) and http://localhost:1313/ (doesn't work, text "render-link.html called!" and target="_blank" in HTML missing). Visible content of both pages is created by the same template.

Pretty sure this is fixed by https://github.com/gohugoio/hugo/commit/9698b0dab11f52d52145e85ff71311d2f103cb4e
The current (stupid) workaround is to make sure you do {{ $tmp := .Content }} or something before you invoke RenderString.

Yes, indeed fixed. Thank you so much!


Use resolutions from settings for set position of dmd

When I working at my VPinball-System, I'm connected over RemoteDesktop, because it is much easer, then to work directly on the 4k display. But for set position of dmd it is using the current resolution of the desktop environment, which is different from the pinball environment.
Would be really cool, when he could use the resoultion of the pinball environment from the settings, instead of the current resolution of RemoteDesktop.

Use vPin Studio's Remote Client and 4k resolution and settings will work properly. You will not have any issues.  You should not be using Remote Desktop (RD) for this function, or for nearly anything. The only thing I use RD for is when I'm building tables or pup packs.


Wrong @param type for build method?

See Container.php line 719 (master)
It specifies that the method's $concrete variable is of type string, but if you gettype() you get an object.
Also, $concrete can be an instance of  Closure.

Yes, that type should be changed to mixed at least, or preferably a pipe delimited list of the argument types that function accepts.
Officially, ReflectionClass constructor declares its argument as mixed:
http://php.net/manual/en/reflectionclass.construct.php
But does specify string or object as types.
IMO it's more valuable to IDE users to see explicit types they are allowed to pass in, so I think
string|object|closure would be the most useful to IDE users.

string|object|closure would be the most useful to IDE users.

Thanks guys. A PR would be much appreciated whenever you have a moment.


Nonpaged Pool Memory Leak in netio.sys When Using Mirrored Network Mode

Windows Version
Microsoft Windows [Version 10.0.22631.4112]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-2
Distro Version
Arch rolling
Other Software
No response
Repro Steps
.wslconfig
[experimental]
networkingMode=mirrored

Expected Behavior
Nonpaged Pool Memory should be a stable and low value, usually within 1G.
Actual Behavior
Nonpaged Pool Memory will grow by 1~2 Mib per second until all memory is occupied.

This problem does not occur when I set networkingMode=NAT, but only occurs when I change it to mirrored.
Diagnostic Logs
No response

Logs are required for review from WSL team
WSL 团队需要审核日志
If this a feature request, please reply with '/feature'. If this is a question, reply with '/question'.如果这是一个功能请求，请回复“/feature”。如果这是一个问题，请用“/question”回复。 Otherwise please attach logs by following the instructions below, your issue will not be reviewed unless they are added. These logs will help us understand what is going on in your machine.否则，请按照以下说明附加日志，除非添加日志，否则您的问题将不会被审核。这些日志将帮助我们了解您的机器中发生的情况。
How to collect WSL logs
如何收集WSL日志
Download and execute collect-wsl-logs.ps1 in an administrative powershell prompt:下载并在管理 powershell 提示符下执行collect-wsl-logs.ps1 ：
Invoke-WebRequest -UseBasicParsing "https://raw.githubusercontent.com/microsoft/WSL/master/diagnostics/collect-wsl-logs.ps1" -OutFile collect-wsl-logs.ps1
Set-ExecutionPolicy Bypass -Scope Process -Force
.\collect-wsl-logs.ps1

The script will output the path of the log file once done.完成后，脚本将输出日志文件的路径。
If this is a networking issue, please use collect-networking-logs.ps1, following the instructions here如果这是网络问题，请使用collect-networking-logs.ps1 ，按照此处的说明进行操作
Once completed please upload the output files to this Github issue.完成后，请将输出文件上传到此 Github 问题。
Click here for more info on logging单击此处了解有关日志记录的更多信息 If you choose to email these logs instead of attaching to the bug, please send them to<EMAIL_ADDRESS>with the number of the github issue in the subject, and in the message a link to your comment in the github issue and reply with<EMAIL_ADDRESS>，并在主题中注明 github 问题的编号，并在消息中提供指向您在 github 中的评论的链接使用“/emailed-logs”发出和回复。
View similar issues 查看类似问题
Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it!请查看下面的问题，看看它们是否能解决您的问题，如果该问题描述了您的问题，请考虑关闭此问题并点赞另一个问题，以帮助我们优先处理它！
Open similar issues: 打开类似问题：

networkingMode=mirrored doesn't work as expected (#11090),  similarity score: 0.72networkingMode=mirrored 无法按预期工作 (#11090) ，相似度得分：0.72

Closed similar issues: 已关闭类似问题：

Effective MTU in mirrored mode is lower than interface MTU (#11584),  similarity score: 0.74镜像模式下的有效 MTU 低于接口 MTU (#11584) ，相似度得分：0.74
Networking mode does not work in mirrored mode (#11540),  similarity score: 0.72网络模式在镜像模式下不起作用 (#11540) ，相似度得分：0.72
In Mirrored mode incoming packets from remote device on LAN are not received in WSL2 (#10535),  similarity score: 0.71在镜像模式下，WSL2 中不会收到来自 LAN 上远程设备的传入数据包 (#10535) ，相似度得分：0.71
mpirun hangs when networkingMode=mirrored (#11215),  similarity score: 0.71当networkingMode=mirrored时mpirun挂起 (#11215) ，相似度得分：0.71

Note: You can give me feedback by thumbs upping or thumbs downing this comment.注意：您可以通过赞成或反对此评论来向我提供反馈。

WslNetworkingLogs-2024-09-08_22-15-38.zip

same problem

Same problem, hope to restart the issue.


Well done! Now for the final touches.

Well done with the PSR-0 standardization, now all it needs is a composer.json descriptor to add the final touches and it can be installed as a package requirement by other projects.
Keep up the good work!
P.S> Also includes modifications to .gitignore to hide said composer.phar install artifacts. Njoy =)

It probably was a good idea at the time.  :interrobang:
gl blending... =)

Hi
As far as I saw before we began to work on this library, composer was there, classes where loaded with PSR-0.. then the job is done :) and was a good idea at time of writing !
I plan to migrate to PSR-4 (see develop branch).

So let me get this str8...
First they come up with PSR-0 which defined this rigid conforming structure, which at the time not one single project actually implemented already.

The autoloader will iterate over every possible scenario starting with the class name and working back through any sub namespace names and a namespace as possible namespace prefixes to see which one will be the folder. Oh and lets throw in any number of completely undefined, unknown, unspecified and random base directories of any depth into the mix just for kicks.
The only good thing is it doesn't break PSR-0, the new autoloader will only take X(n) longer to find your source files.

PSR-0 or PSR-4, it does not matter a lot (for me). I think at least the library must use one file per class.
I may go back to PSR-0 as the change is not released yet.

You should not be responsible for providing an autoloader, so the choice is solely on what structure to use. I then agree with your conclusion PSR-0 is already implemented and so will have a smoother path to market.


Bridging run adding the fishery age comps before adding the survey?

Just wondering (and I don't think I can see it in #313 but I think we may have discussed at some point) - clearly (Fig 15, p112) shows that adding the 2017 survey index pulls the recent years down, then adding the fishery age comps doesn't make much more of a difference. However, did anyone also try adding the fishery age comps first (without the survey index)?
The text (p51) says the addition of the age comps (fishery and survey) had relatively little additional impact ... indicating the ages were consistent with the model estimates without those data. The age residuals are small for 2017 (Fig 25).
So that somewhat could imply that adding the age data first also has the same effect. If we have time then it may be worth doing such a run (if not done already) - it's been brought up that the survey is bringing the biomass down (and so would it be possible to have a 2018 survey?). But the data seem consistent and so it would be good to know if a non-2017-survey model would give similar results.

Andy, I think that is a good idea.  I have not done that run.  Perhaps Ian did when doing the bridging, but if we haven't it would be worth doing and I can do that one.

I think this is a good idea. You could also add yet another option where the survey ages are added before either fishery ages or survey biomass.
I just posted issue #416 to help us keep straight what additional sensitivities are being run and keep track of the numbering.

attaching a pdf comparing one alternative bridge chronology - survey still mostly driving trend in recent years, but entirely.
CompareAltBridge.pdf

This makes sense. It looks like the fishery ages reduces the 2014 cohort but leaves 2010 alone, so the impact is only on spawning biomass in 2016 and beyond. Adding the survey biomass brings 2010 down as well, which causes an impact on spawning biomass from 2012 onward.

And for completeness a second bridge set where survey ages are introduced first.  Nothing really new here, suggesting survey ages were in line with what was expected given 2017 assessment.
CompareAltBridge2.pdf

Aaron - if it's easy then please remake it in more landscape style (for the talk) - the code in beamer is
  <<bridge.biomass2, fig.height=4.5, fig.width=8, out.width='0.9\\columnwidth'>>=
    oldpar <- par(mar=c(5,4,1,1),no.readonly = TRUE)

though I can't see where \columnwidth is set (or if it's needed). So a similar aspect ratio would be good.

Okay, updated the readme and uploaded those model runs to drive.  Yes, I was just going to ask if you wanted any particular figures uploaded as .eps.  So the second one in first alternative bridge set, sending over email now
...

Okay, just sent you one but let me try an alternative with better aspect ratio

landscape version coming over email now...

Got it thanks. Aspect ratio is perfect.
Sorry, are you able to do the one with adding fishery data first? Makes a bit more sense (because without a survey you wouldn't have the survey ages). Email is fine - once we've settled on one I can add to GitHub.

Sure.  GitHub doesn’t like .eps uploads, but I can put into a figures folder in assessment folder.

On Feb 22, 2018, at 12:03 PM, andrew-edwards<EMAIL_ADDRESS>wrote:
Got it thanks. Aspect ratio is perfect.
Sorry, are you able to do the one with adding fishery data first? Makes a bit more sense (because without a survey you wouldn't have the survey ages). Email is fine - once we've settled on one I can add to GitHub.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Seems to be hit or miss with .eps uploads.
The beamer/SRG/Management/Figures folder has a few .eps files that worked fine, as does doc/main-figures. So far, I think it's only been the maps that got messed up.

About the .eps uploads - it depends which software you used to create them. If you used a Unix-based software on Windows the end-of-lines are \lf which are switched to \crlf by Github. I can't find the related issue now but we can fix this once done.

It makes sense the the software would be the key difference. Thus, it seems that .eps files created in R using cairo_ps() are OK while the maps that Andy converted from PDF using imagemagick or whatever were the issue.

As well as .eps files created in Adobe Pro, they also have that issue.

On Feb 22, 2018, at 12:44 PM, Ian Taylor<EMAIL_ADDRESS>wrote:
It makes sense the the software would be the key difference. Thus, it seems that .eps files created in R using cairo_ps() are OK while the maps that Andy converted from PDF using imagemagick or whatever were the issue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Okay, just committed your eps file Andy to SRG\Assessment\Figures
this is the one I meant to sent first

Thanks, now included. That was worth doing.
If people request to add it to the document we can probably just do it manually.


Update client3:akka-http-backend, ... to 3.7.1

Updates

com.softwaremill.sttp.client3:akka-http-backend
com.softwaremill.sttp.client3:core

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

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.client3" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.softwaremill.sttp.client3" }
}]

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

Superseded by #411.


[fix] gws/schedule: remove selected items (m37)

Coverage decreased (-0.2%) to 79.219% when pulling b6f45346c87776ab5e73e67cfd50aedee3d47c37 on tany:gws-m37 into 3c3e0d43559b24158ab6ee1ce3de8a41b1a8ea56 on shirasagi:master.


Update README

Is your feature request related to a problem? Please describe.
There is a description of when Files was a beta version.
## Vision for Files
In the short term, we plan to focus on crucial functionality to achieve parity with Windows File Explorer. Before leaving beta, we'll address many of these pain points and improve accessibility. Later on, the project will reach a point where we can fully celebrate our core audience by adding sought-after power user features. Stay tuned!

Describe the solution you'd like
Please update the description.
Describe alternatives you've considered
N/A
Additional context
N/A

@yaichenbaum

The vision for Files is to make it the best file manager while keeping it simple and easy to use. Whether it's implementing new features, or pushing the boundaries of the platform, your input will help shape the future of Files.```

@pklion Would you like to work on this issue?

The vision for Files is to make it the best file manager while keeping it simple and easy to use. Whether it's implementing new features, or pushing the boundaries of the platform, your input will help shape the future of Files.```

I think this is a brilliant description, so I'll make the PR.


[Feature Request]: CrateDB Bulkloader

What would you like to happen?
Add a transform which should play the role of bulk loader both with HTTP endpoint and COPY FROM statement
Issue Priority
Priority: 3
Issue Component
Component: Transforms

.take-issue


Spark 2.2.0: pip pyspark doesn't work well in notebook on Windows 10; got Exception: Java gateway process exited before sending the driver its port number

Step1: pip install pyspark
Step 2: jupyter notebook
Step 3: initialize a Spark Context

from pyspark import SparkContext
sc = SparkContext()

But I got the error Exception: Java gateway process exited before sending the driver its port number.
One workaround is to point SPARK_HOME variable to one of the pre-built Spark package directory (e.g., C:\spark-2.2.0-bin-hadoop2.7), and step 3 works fine so a Spark Context is initialized. I looked through the pyspark folder installed by pip and the pre-built package from download and found the file structures are a bit different, which should be the cause for the problem.
Weirdly, step 1-3 seems to work well on Mac OS 10.11.6.

Can one of the admins verify this patch?

Could you close it and resubmit the PR?

@ShuaiW close this please


Works around quirks for k8s watch

Turns out there is a default server timeout of 30 minutes in k8s watch api.  This PR sets the timeout to a large number (7 days).   Even so, we may occasionally see server connection drops.  To make it more robust, also added retry mechanism.
Fixes #512

Pull Request Test Coverage Report for Build 1695

0 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals

Change from base Build 1671:
0.0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls


E2E test for alpha features on Antrea Agent

Instead of running whole suite for every alpha feature, it was discussed
in Issue #893 to run the suite by enabling all alpha features. This PR handles alpha
features on Antrea Agent.
As there is ongoing geneve tunnel issue, Issue #897 with ClusterNetworkPolicy
alpha feature on the controller, I did not club alpha features on agent and
controller. Similarly, the e2e tests of Traceflow feature could be modified,
when both agent and controller alpha features are clubbed together.
This PR is useful to turn on e2e tests for flow exporter feature PR #825.
Modified e2e documentation to run specific tests corresponding to alpha feature locally.

I'd rather wait for #897 to be addressed and merge a PR that is more "consistent" and enables alpha features for both controller and agent.
If you need to test flow export before then, you can add a new job IMO

If I add a separate job for the flow exporter feature, it will add one more extra job and the resulting time in CI, right? Thought this could be a decent transition until #897 is fixed, and follow up change could be on top of this without removing much of code from this change.

Yes it is one more job but maybe you can set it up so only the exporter tests are run to reduce the likelihood of failure. I am hoping that #897 won't take too long to resolve.


Host and placeholders

We should get rid of placeholders CHANGE_ME in jenkinsbox.yml. Those settings should be provided in separate .yml file, e.g. config.yml. Also host IP should be provided in config.yml.

@danylevskyi check ability to include cibox.cfg from ansible.cfg

We should also pass into playbook variable jenkins_server_ip that should be equals to host ip.


f'ed up comment big time, captured a bunch of text somehow

edited comment from today, but screenshot here;

leaving open for now, check again after a week or so to see if this doesn't happen or does


Using multiple backends in single website may cause SystemError

As discussed in slack

I will post an issue regarding There was an error of type SystemError running the code.

math-franklin | ┌ Warning: There was an error of type SystemError running the code.
math-franklin | └ @ Franklin ~/.julia/packages/Franklin/RDM4z/src/eval/run.jl:81

This phenomenon is caused by a large amount of backend use like my website and we can reproduce the error with the following code:

# sample.jl

# code block A e.g. https://terasakisatoshi.github.io/MathSeminar.jl/gallery/gr/
import GR

GR.inline("mov")
x = collect(0:0.01:2*pi)
for i = 1:30
    GR.plot(x, sin.(x .+ 3i / 10.0), size=(400,300))
end

s = GR.show()
println("~~~$(s.s)~~~")

# Note that GR.inline("mov") is still enabled
# end of code block A

# code block B #
using Plots

p = plot(sin)
Plots.savefig(joinpath("save.svg")) # <-- This line will cause System Error ...
# end of code block B

$ julia sample.jl
ERROR: LoadError: SystemError: opening file "/var/folders/lp/9bwxfgs118nb_32vq8psnd_40000gn/T/jl_EZLHkN.svg": No such file or directory

After some trial and error, I found that the solution is to insert GR.inline() right before end of code block A.
As we can see, each block of code works fine on its own, but combining them may have unintended consequences. 😅

Hey sorry could you send a reproducing markdown? I get confused about what exactly you wrote in the markdown. Also a fair bit of this seems unrelated to Franklin but rather to problems with GR or plots.
Also the join path with a single argument will not work ?


iOS text record defaulting to UTF-16 encoding

Hi there,
I've been using nfc_in_flutter on android quite successfully, but when I got around to setting it up on iOS the result wasn't as expected.
I am creating a tag with a single text record, on android it writes as expected (encoding the payload in utf8), however, on iOS it encodes as utf-16.
List records = [
NDEFRecord.text(
payload,
)
];
Has anyone else seen this issue?

Ive managed to get it working by altering NfcInFlutterPlugin.m
altering the if statement at line 391 to the following:
if ([@"T" isEqualToString:recordType]) {
payloadData = [[NSString stringWithFormat:@"%c%@%@", recordLanguageCode.length, recordLanguageCode, recordPayload] dataUsingEncoding:NSUTF8StringEncoding];
tnfValue = NFCTypeNameFormatNFCWellKnown;
}
it seems that the "wellKnownTypeTextPayloadWithString" function call in the original code always returned the record as utf16


Mkdocs

I can't find any mkdocs documentation online and I can't create it myself. The instructions for creating documentation on README.md (mkdocs branch) is still for ReadTheDocs.

@pnrobinson See comment above


fix caching, restructure project a bit

This takes aim at #136. It's not 100% done, not totally happy with it, but it's as much as I could do in the time I have available.
The project structure is simplified a bit: all the source code goes in the src directory (with client, server, service-worker and universal subdirectories). All the generated code goes into a new build folder, with hashed filenames (including blog.json and guide.json), which makes the server's job slightly easier.
All the content (blog, guide, examples) sits in a new content directory.

took a while (problems with now — may have to investigate alternatives, this seems to happen too frequently) but there is now a staged deployment:
https://sveltetechnology-ohjrtgvqfp.now.sh
will give it a bit of a kick, then merge this in if nothing breaks


Ollama hosting component should expose resource commands for model management

Related to an existing integration?
Yes
Existing integration
Ollama hosting
Overview
Following on from this discussion here it would be useful if as part of the Aspire 9 release the Ollama hosting component exposed some resource commands that would allow a user to manage the models in their Ollama instance.
Usage example

[ ] Re-download a model
[ ] Delete a model

Breaking change?
No
Alternatives
Using the ollama API or Open WebUI can achieve this (to a degree)
Additional context
No response
Help us help you
Yes, I'd like to be assigned to work on this item

This might be better waiting for #119 first, as then the commands can be on the resource for the model rather than on the Ollama hosting resource, which I think would make more sense.


Style dont display correctly in Window Server 16

First of all, thank you so much for these amazing libraries! I started using them recently and immediately fell in love. The problem I'm facing is that the CLI looks perfect on my Windows 10 PC, but when I run it on the Windows server where it will be used, the colors and font look bad.  Is there something I need to configure at the Windows Server level?

This looks like a problem in the terminal emulator you're using on windows server.
Have you tried different ones?
In any case, without more context, not much we can do here... would need details, screenshots, etc.
Closing for now.


Explicitly set latest release

enable to set latest release
https://github.blog/changelog/2022-10-21-explicitly-set-the-latest-release/

make_latest is by defaul true but yaba can be extended to set make_latest explicitly.


Improve UX on small screens for the beta version

Goal
Improve UX on mobile, following the feedback we got here https://github.com/bestofjs/bestofjs/issues/176#issuecomment-1685607435

show Best of JS logo
replace the input field that triggers the search palette with a search icon
hide Discord and GitHub icons (TODO: add them to the navigation menu for mobiles)
make "Featured Projects" block take all the available space instead of the hard-coded 300px
make the search palette take the whole height

On small screens, the top bar is now made of the following elements:

The menu button that reveals a meeny sliding from the left
The Best of JS logo
The search icon
The dark/light mode switcher

Screenshots

Improving the search modal
I noticed the layout of the search results is currently broken because of the description that is not correctly truncated.
I fixed that by using a min-width: 0 property.


python311Packages.python-lsp-ruff: 2.1.0 -> 2.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.python-lsp-ruff is: Ruff linting plugin for pylsp
meta.homepage for python311Packages.python-lsp-ruff is: https://github.com/python-lsp/python-lsp-ruff
meta.changelog for python311Packages.python-lsp-ruff is: https://github.com/python-lsp/python-lsp-ruff/releases/tag/v2.2.0
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.2.0 with grep in /nix/store/ald4vixml8lf3g60si6fd4dg8hxl8sxq-python3.11-python-lsp-ruff-2.2.0
found 2.2.0 in filename of file in /nix/store/ald4vixml8lf3g60si6fd4dg8hxl8sxq-python3.11-python-lsp-ruff-2.2.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.python-lsp-ruff

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ald4vixml8lf3g60si6fd4dg8hxl8sxq-python3.11-python-lsp-ruff-2.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.python-lsp-ruff https://github.com/r-ryantm/nixpkgs/archive/b60be8179d26d8212efadc7d9767ef0ce603c606.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b60be8179d26d8212efadc7d9767ef0ce603c606#python311Packages.python-lsp-ruff

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ald4vixml8lf3g60si6fd4dg8hxl8sxq-python3.11-python-lsp-ruff-2.2.0
ls -la /nix/store/ald4vixml8lf3g60si6fd4dg8hxl8sxq-python3.11-python-lsp-ruff-2.2.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

python311Packages.python-lsp-ruff
python311Packages.python-lsp-ruff.dist

Maintainer pings
cc @linsui for testing.


https://openbuildservice.org/support/ mentions "openFATE" which shows "There are currently no products open for new features. "

Issue Description
https://openbuildservice.org/support/ references openFATE for new features but following the link just brings one to the main page of https://features.opensuse.org/ with the popup error "There are currently no products open for new features.".
Expected Result
Probably openFATE is not used anymore but also github issues? I suggest to simply link to the github issues instead or not even ask the user to make the distinction between feature request or bug anymore.
How to Reproduce

Go to https://openbuildservice.org/support/
In section "we better ourselves" click on openFATE
See the fate page loaded with the error message

There is no fate.


Make the file limit configurable

Currently, Linguist has a 100,000 file limit hardcoded in lib/linguist/repository.rb:
    protected
    MAX_TREE_SIZE = 100_000

When the number of files in the repository exceeds that, Linguist gives up and returns an empty result:
    def compute_stats(old_commit_oid, cache = nil)
      return {} if current_tree.count_recursive(MAX_TREE_SIZE) >= MAX_TREE_SIZE

This may be a good limit for GitHub, but I think that the library would be more widely useful if it was possible to change it—for example, via a getter/setter pair that accepted either a number or nil (to disable the check entirely).
A related issue is that when the limit is exceeded, Linguist returns an empty hash, which is the same as for an empty repository, with no indication of what happened. This can be quite confusing—or at least, it confused me the first time it happened, and issues https://github.com/github-linguist/linguist/issues/2915 and https://github.com/github-linguist/linguist/issues/3342 indicate that it has confused other people as well. Perhaps it could be changed to something that communicates the issue clearly, such as return :too_many_files or raise FileLimitExceededError?

#6606

Resolved by https://github.com/github-linguist/linguist/pull/6806 and https://github.com/github-linguist/linguist/pull/6825


Endpointslice backend gives nil when using FQDN for aws rds database

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When creating a service which points to an endpointslice that is addressType: FQDN, the backend is nil, this manifests as a red-herring error message about ipv6.
`
---
apiVersion: v1
kind: Service
metadata:
labels:
endpointslice.kubernetes.io/managed-by: platform
name: rdsmariadb
namespace: test
spec:
ports:
- port: 3306
protocol: TCP
targetPort: 3600
---
apiVersion: discovery.k8s.io/v1
endpoints:

addresses:

eks-cluster-mariadb.(redacted).ap-southeast-2.rds.amazonaws.com
conditions: {}
kind: EndpointSlice
addressType: FQDN
metadata:
labels:
endpointslice.kubernetes.io/managed-by: platform
kubernetes.io/service-name: rdsmariadb
name: rdsmariadb
namespace: test
ports:

name: ""
port: 3600
protocol: TCP
`

Gives this log line from cillum (in debug mode)
level=debug msg="Upserting service" backends="[{ 0  {<nil> {TCP 3600} 0} 0 false false}]" l7LBFrontendPorts="[]" l7LBProxyPort=0 loadBalancerSourceRanges="[]" serviceIP="{<IP_ADDRESS> {TCP 3306} 0}" serviceName=rdsmariadb serviceNamespace=test sessionAffinity=false sessionAffinityTimeout=0 subsys=service svcHealthCheckNodePort=0 svcTrafficPolicy=Cluster svcType=ClusterIP
As opposed to, when addressType: IPV4 is given in the endpointslice, and the IP is used:
level=debug msg="Upserting service" backends="[{ 0  {<IP_ADDRESS> {TCP 3600} 0} 0 false false}]" l7LBFrontendPorts="[]" l7LBProxyPort=0 loadBalancerSourceRanges="[]" serviceIP="{<IP_ADDRESS> {TCP 3306} 0}" serviceName=rdsmariadb serviceNamespace=test sessionAffinity=false sessionAffinityTimeout=0 subsys=service svcHealthCheckNodePort=0 svcTrafficPolicy=Cluster svcType=ClusterIP
This is a problem because in a multi-az rds deployment, the actual endpoint ip can and will change when failover occurs, so the workaround of using the IP isn't possible in most cases.
Cilium Version
1.12.3
Kernel Version
running on eks in aws, but its eks version 1.24
Kubernetes Version
1.24
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

FQDN types are not supported as of yet, see https://github.com/cilium/cilium/blob/e5d8cbc566af95f49cbc2ecd715c0f75f95f693a/pkg/k8s/endpoints_test.go#L1147.

not stale

cc @cilium/sig-lb

not stale.

still not stale.


Added Basic SBT & Tests

Added Basic SBT & Tests

included TODOs as comments for further discussion

@ExtremelySunnyYK looks ok for now - let's reconvene when your finals are done


Remove the CHERI mode enable CSR bits

With the mode bit of PCC always present when legacy mode is available,
we no longer need a separate out-of-band state bit to indicate the
current mode. The CHERI register enable (CRE) bit remains present to
prevent access to CHERI instruction and mseccfg.CME was changed to
behave as a register enable bit for M-mode.
Having the mode bit always present ensures that we can set the execution
mode for trap handlers without affecting the mode of other code.

CC: @sorear
This relates to https://github.com/riscv/riscv-cheri/pull/130, moving all CHERI mode state to PCC and out of xenvcfg.

@tariqkurd-repo : Could you review again? I fixed the bit indexes in xenvcfg diagrams

@tariqkurd-repo @arichardson : This was merged onto an alternate branch instead of main. We need to merge it into main if we are happy with the changes

@tariqkurd-repo @arichardson : This was merged onto an alternate branch instead of main. We need to merge it into main if we are happy with the changes

I think the easiest way will be to recreate this PR and merge it. Will do so shortly.

I will most likely not be able to look at any CHERI-related issues until two weeks from now.


Improved OES: transactions

Improve OES
Add tests for the following
Staking V1

[x] stake
[x] unstake

Staking V2

[x] stake
[x] addstake
[x] unstake

Vesting

[x] redeem

Stake box ids change every day after compound. Cannot test unstake/addstake.

unstake/addstake - https://github.com/ergo-pad/ergopad-oes/commit/a6a91169c21e97fa17480fcc606f902c541d5ec6


[test] Make DGEMM test generic

Fixes UES-401

@jenkins-cscs retry daint

@jenkins-cscs retry none

Codecov Report

Merging #827 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #827   +/-   ##
=======================================
  Coverage   91.81%   91.81%
=======================================
  Files          80       80
  Lines        9982     9982
=======================================
  Hits         9165     9165
  Misses        817      817

Continue to review full report at Codecov.

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


Casting sound gets repeated for each pattern evaluated by a meta-evaluation or macro

While playing with @walksanatora's test pack, I noticed that if I used Hermes on a list or drew a Hexal Macro, I would hear what sounded like a copy of the initial sound for each pattern in the list being executed. For instance, if I used Hermes to execute something 15 patterns long, I would hear Hermes Gambit's sound 15 times + The initial pattern. The same would happen with Macros, except it would be the standard casting zap sound that gets repeated.
Just executing 100 Nullary Reflections was enough that I'm glad I pulled my headset off my ears before trying it😬.
(Edit: Also I hope you're okay with the ping Walks, since your pack was mentioned)

dw I am subscribed to all actions on hexcasting anyways (so I would have known anyways)

questioning if this is a bug. you're executing 15 patterns + the Hermes gambit? so it makes sense to make 15+1 noises?

An audio comparison between 1.19 and 1.20 would be helpful.

Wasn't this fixed c6612c54? Probably should test in-game


Error while launching the admin web interface

Hi,
When I try to run
java -cp "drivers/h2/*;lib/*" ^  -Deasy.jobs.database.config.file=%cd%\conf\database.properties ^  -Dserver.port=9000 ^  org.jeasy.jobs.admin.Application
I got
java.lang.IllegalArgumentException: Could not resolve placeholder 'application.version' in value "${application.version}"
The server works fine but it's impossible to run the admin web interface.
I didn't make any modifications. I'm just trying to run the "demo" that you provide.
Did I miss something ?
BTW, java version is openjdk 1.8.0_171 on a debian 9.
Thanks

Hi,
Thank you for reporting this issue! It took me sometime (and hairs!) to figure out the problem. The issue occurs with version 0.3 but not with version 0.3-SNAPHOT.
The application.version property is defined in the application.properties files of the admin interface but not in the application.properties of the server. Both the admin and the server (along with their dependencies) are deployed in a single folder lib. When you run the startup command of the admin interface with version 0.3-SNAPSHOT, it works fine. But when you do it with version 0.3 it fails because for some reason, the server's jar is loaded before the admin UI jar and hence the application.version property is not found.
Since the issue can arise with future versions, I separated the admin and server libs under lib/admin and lib/server. There will be some duplicate jars in the distribution file (which is what I wanted to avoid by putting both in the same folder) but this will be addressed in another issue.
Son basically what happened is: I tested with version 0.3-SNAPSHOT and it was ok, I released version 0.3 and it does it work.. From now on, I will give a shot with the released version and write a end to end test for the distribution file. Other improvement I am considering is writing startup scripts for windows/unix and creating a docker image for each artifact.
Anyway, thank you for reporting the issue. I released version 0.4 for the matter. Could you please give it a try?
Thanks!

BTW, I couldn't ask you to test with 0.4-SNAPSHOT since the same issue may arise again. I had to release a version. So I'm closing this issue for now since it is tagged for 0.4 and will re-open it if necessary for 0.5.


Support runtime appearance adjustment on macOS

On macOS switching appearance (Light -> Dark or Dark -> Light) when Bitcoin Core is running makes the GUI pretty unusable.
This bug is especially important when a user chose the "Auto" mode to adjust appearance automatically.
This PR fixes Bitcoin Core behavior.
This is an alternative to #268.
This PR does not enable the dark mode for builds that have this feature unavailable now, i.e., native builds with depends.

@goums @jarolrod
Do you mind testing this PR?

ACK 3696ebc873bd71e1a7036200d8f1cf96d262fb9f
Wow! Looks great!
MacOS 10.14.6 (18G8022) Light Theme

MacOS 10.14.6 (18G8022) Dark Theme

@RandyMcMillan

Concerning Accessibility: Using Accessibility Inspector on MacOS...

Accessibility is out of this PR scope, as its only goal is the smooth runtime switching between dark and light appearances (note that the dark appearance is supported since #154).
I'll be happy to review dedicated prs that improves accessibility. Probably, it'd be nice to start from some kind of a design guide.

@jarolrod

Spent all day playing around with this and have some findings.

Many thanks for your extensive testing and reviewing!

Dark mode does not work when compiled through depends natively on macOS 11 and 10.15. It does work on macOS 10.14 (out of scope of this PR)

Sorry, still investigating this weird issue.

There is a weird bug with rows in a table, they have their borders colored in white when in dark mode.

Is this our bug or Qt's one (upstream)? Is this bug of the switching between modes? Or it was present in #154?

Rebased 57373016e6d581e2a0d32b0cfaf8c63d28a9c20b -> c231254a65d390a3350fcef456d57e4a6eca0506 (pr275.01 -> pr275.02) on top of the merged https://github.com/bitcoin/bitcoin/pull/21793.

@promag @jarolrod @Sjors
Do you mind having a (final) look at this PR after the recent rebasing?

This PR does not enable the dark mode for builds that have this feature unavailable now, i.e., native builds with depends.

Is this still true after the latest dependency bumps?

@Sjors

This PR does not enable the dark mode for builds that have this feature unavailable now, i.e., native builds with depends.

Is this still true after the latest dependency bumps?

Good catch! Removed.

tACK c231254a65d390a3350fcef456d57e4a6eca0506 on macOS 11.4
The code changes look sane, but I didn't study them in detail.
This should also be tested on Linux and Windows, since not everything is wrapped in #ifdef Q_OS_MACOS.
I tested with and without depends. I'm able to toggle between dark and light mode in various screens and it looks healthy.
Minor issue, maybe for followup:
When no wallet is loaded, the tab items are grey instead of white (depends build):

ACK c231254a65d390a3350fcef456d57e4a6eca0506
Tested on mac OS Big Sur (11.2.3)
Light/dark mode switching works very smoothly on every windows 👍
Compare to previous version, all icons and labels are re-colorized accordingly to the theme change
Screen capture for those who are not testing on macOS:
https://user-images.githubusercontent.com/5577626/119896943-0a817300-bf40-11eb-8464-a8f11c1e1270.mov

This should also be tested on Linux and Windows, since not everything is wrapped in #ifdef Q_OS_MACOS.

On Linux Mint 20.1 (Qt 5.12.8) with installed qt5ct package I'm able to switch themes for Qt apps during their runtime.
This PR does not add support for such theme switching on Linux though. It ls left for follow ups.

Tested ACK c231254a65d390a3350fcef456d57e4a6eca0506 on macOS Big Sur arm64.


Invalidated Socket Error in Kitura-net:handleWriteHelper in 0.32

We were using Kitura fine at 0.30.  Starting in 0.32, we get constant socket errors.
The issue is at line 118-119 of IncomingSocketHandler.swift
As the socket is being written to over time, writeBufferPosition increases with every write.  Eventually, it becomes equal to writeBuffer.length at which point bufSize: writeBuffer.length - writeBufferPosition becomes bufSize: 0, which causes Socket.write() to throw Socket.SOCKET_ERR_INVALID_BUFFER.
This is blocking the Sandbox from moving to 0.32.

Thanks for detailed debug of the problem. I'll get that fixed right away.

@carlbrown: It'd be easy to change BlueSocket.write() to return a zero (0) if the bufSize is equal to zero (0) but it seems me that that'd be most likely, the wrong thing to do...  I would think that the proper fix would be to check the length before issuing the write...
if writeBuffer.length != 0 {
    do {
        var writeLength = writeBuffer.length - writeBufferPosition
        if writeLength == 0 {
            writeBuffer.length = 0
            writeBufferPosition = 0
        } else {
            let written = try socket.write(from: writeBuffer.bytes + writeBufferPosition, bufSize: writeLength)
            if written != writeBuffer.length {
                writeBufferPosition += written
            } else {
                writeBuffer.length = 0
                writeBufferPosition = 0
            }
       }
    catch {
       ....

Didn't mean to close the issue. 😃

@carlbrown I have created a branch of Kitura-net with a fix. Can you please test it?
To test this branch:

Move up to Kitura 0.33
Clear your Packages directory
Do a swift package fetch
Do a cd Packages/Kitura-net-0.33.x
Run the following commands:

git config remote.origin.fetch "+refs/heads/*:refs/remotes/origin/*"
git fetch origin
git checkout issue_762
cd -

Run swift build
Run your server and test

Much better. Thank you.
Tested (and working) with Swift 3.0-RELEASE on both macOS 10.11.6 and Ubuntu 15.10 (via Vagrant)

Your welcome.


Replace our xout library with Gabi Melman's spdlog

spdlog is especially interesting to elastix because it offers thread support, it allows logging level selection (to be used later), and it is actively maintained.

beautiful work Niels! Perhaps you can show us some logger output, comparing old with new.

Thanks @mstaring With this pull request, the logger output should be just the same as with the old logger (but with added thread-safety, and prepared for future extensions, like log level support). But it's a good idea to double-check that all the output is still there!
My idea is to further adjust the logging in follow-up pull requests, because this pull quest is already quite large!

great to hear, if you can show me two log files, then we can move ahead as you suggest with follow-up PRs.

The result of the logging is partially online, already  😃 That is: the output to standard I/O is shown by the GitHub CI. I did some comparison between the CI logs at https://github.com/SuperElastix/elastix/actions/workflows/ElastixGitHubActions.yml and they look comparable (comparing the Add-spdlog-as-submodule branch to the main branch).
I also compared the "out" directories of a simple registration, after having run both elastix 5.1.0 (release version) and a local build. They look similar (except for changes in the line breaks, from "\r\n" to "\n", the subject of my PR https://github.com/gabime/spdlog/pull/2636), including the "IterationInfo" files. I can show you, of course.

elx-5.1.0-release-out.zip
elx-Debug-spdlog-out.zip

looks great Niels, go ahead!

The pull request now has 14 commits, but I think I'll squash a few of them. I think I would like to just have separate commits for 1. adding the spdlog library, 2. adding the elastix log class, 3. using the new logging, and 4. removing the xout library.


Shaders

I wanted to use this program: https://www.codeandweb.com/spriteilluminator with my Easeljs game but FAQ says the framework has to support shaders.
Does Easeljs use shaders and if not, how tricky would it be to write something that could use and apply shader data?
Thanks!

Currently there is no shader support. The WebGL functionality that exists bring some of the existing functionality into WebGL, but doesn't enable additional WebGL features. Future versions may have this support though.


Coverage info on pull requests and README

I believe that showing information about the coverage about the changes that pull request inflicts is something very useful in order to automatically verify the coverage in a code review.

Agreed.


increase line buffer size

The line buffer size is set to 64 * 1024 * 1024 = 64MiB per default. Japi-Requests with more bytes will fail and cause the error
ERROR: Maximum line size of 67108864 bytes exceeded!

However, for some applications like transferring data to the server increased line sizes would be needed. The line size can be adjusted with redefining CREADLINE_MAX_LINE_SIZE to the desired value.

I temporarily fixed the line size to 1GiB.
However, the line size is depending on the user application and should be somehow adjustable when including the library (With a setter function, with config files, etc.) and not by changing the source software.


Complete python api rewriting

Currently, we can't obtain arguments information from python jobs.
There is initial implementation which possibly can fix that problem(branch)
Also, we need to implement mist-logger for them.
Example:
from mist.decorators import *

@with_args(
    arg('numbers', type_hint=list),
    arg('multiplier', type_hint=int, default=2)
)
@on_spark_context
def simple_context(sc, numbers, multiplier):
    rdd = sc.parallelize(numbers)
    result = rdd.map(lambda s: multiplier * s).collect()
    return {"result": result}

Would like to see a feature to specify the python interpreter as part of the context definition (e.g. PYSPARK_PYTHON, PYSPARK_DRIVER_PYTHON).


Update README.md

Please resolve merge conflicts

it show file has conflict , and how to fix it ?

@fdciabdul merge master into your branch and look through the file to see merge conflicts and fix them manually.


Unable to affect Pmenu colour

The floating diagnostics window has a dark background yielding a pretty confusing view.

With other themes, I can control the background with:
:hi Pmenu guibg=#808080

Giving me something like, with evening theme, for example:

However, with gruvbox-baby, this has no effect.
User error?

Could you try putting this before calling the colorscheme?:
vim.g.gruvbox_baby_highlights = {Pmenu = {bg = "#808080"}}

Thanks for the response. Unfortunately, no effect.

But the colour of the auto-complete window is affected. I don't know how that could be, that the diagnostic window is affect by Pmenu in one them but not gruvbox-baby.

Turns out that diagnostic menu is NormalFloat, but blue, slate, and other built-in themes link Pmenu to that :angry:
Confusing!
Thanks for the theme, and thanks for the support.


add ClaroSmartSwitch

I'm pretty sure this is right but it'd be nice if somebody who has one checked.
Closes #115

@usingapc,
Do you mind checking on the DeviceType of your Claro Smart Switch?  You can just post the contents of the lutron_caseta diagnostics txt file, or have a look through it.  We would like to confirm if the DeviceType it show is "ClaroSmartSwitch"
Thanks.

It appears that the Claro Switches show up as a "DivaSmartSwitch"  Here are
snips of the diagnostics file for each of the new types of switch/dimmer:
Claro Switch:
"21": {
"device_id": "21",
"current_state": 100,
"fan_speed": null,
"zone": "14",
"name": "Kitchen_Pantry Light",
"button_groups": null,
"type": "DivaSmartSwitch",
"model": "DS-5ANS",
"serial": 80857385
},
Diva Dimmer:
"22": {
"device_id": "22",
"current_state": 0,
"fan_speed": null,
"zone": "15",
"name": "Kitchen_Sink Light",
"button_groups": null,
"type": "DivaSmartDimmer",
"model": "DS-6WCL",
"serial": 80822679
}
On Mon, Sep 19, 2022 at 11:25 AM Kevin Addeman @.***>
wrote:

@usingapc https://github.com/usingapc,
Do you mind checking on the DeviceType of your Claro Smart Switch? You can
just post the contents of the lutron_caseta diagnostics txt file, or have a
look through it. We would like to confirm if the DeviceType it show is
"ClaroSmartSwitch"
Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/gurumitts/pylutron-caseta/pull/117#issuecomment-1251321856,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVQ2DNHQUVCWBBM3B5SERTDV7COZ5ANCNFSM6AAAAAAQPVELZ4
.
You are receiving this because you were mentioned.Message ID:
@.***>


add boost::hana

Please create a hunter-packages/hana fork from https://github.com/boostorg/hana

Your standard library for metaprogramming http://boostorg.github.io/hana

Hana is a part of Boost package. Meaning if you want to patch it we should use this repo:

https://github.com/hunter-packages/boost

I need to update code to newer version. Can do it after this merge:

https://github.com/ruslo/hunter/pull/1265


change i18n of 'reference number' depending on docment scheme: https:…

…//github.com/metanorma/metanorma-iso/issues/1096
Metanorma PR checklist

[x] Breaking changes (list related PRs) : https://github.com/metanorma/isodoc/commit/4ce0738943a6f90edd65d9956de05326ffa3e936
[ ] Documentation update required (create task for this)
[ ] External dependency introduced (documentation update need)
[ ] Gem with native library introduced

Merged into https://github.com/metanorma/metanorma-iso/pull/1102


How to revoke access through gapi.auth2.authorize

Hello community.
Is it currently possible to revoke consent through GoogleUser.disconnect() or get the granted scopes through GoogleUser.getGrantedScopes() with just the gapi.auth2.authorize option?
I'm having trouble finding out how to access the GoogleUser in this case.

You can revoke the access making an HTTP request to https://accounts.google.com/o/oauth2/revoke?token={TOKEN}. See doc. Using the client library to revoke token is currently not possible if the tokens were obtained through the authorize method.
You will get the granted scopes in the response to gapi.auth2.authorize when requesting an Access Token (response_type: 'permission').


RNCNetInfo.getCurrentState got 3 arguments, expected 2

Environment

MacOS Catalina 10.15.6
Platforms

Android
Versions

Android: 10
react-native-netinfo:5.9.5
react-native: https://github.com/expo/react-native/archive/sdk-35.0.0.tar.gz
react:16.8.3

Description

I'm getting an error which state RNCNetInfo.getCurrentState got 3 arguments, expected 2  while running the native project using expo start.  We were using NetInfo module from react native previously but since i encountered this bug i wanted to use the latest package . I followed the instructions is expo docs : https://docs.expo.io/versions/latest/sdk/netinfo/ but it did not work .
Things i have tried so far :
• install the RCN netinfo  package using expo install  after deleting node_modules
• Installing the RCN Netinfo package using yarn after deleting node_modules
Reproducible Demo

Old logic:
 import { NetInfo, View, Text } from 'react-native';
   NetInfo.getConnectionInfo().then((connectionInfo) => {
      if (connectionInfo && connectionInfo.type !== 'none') {
        this.props.ConnectionActions.checkStatus();
       }
    });

New logic :
import NetInfo from '@react-native-community/netinfo';
  NetInfo.fetch().then(connectionInfo => {
      if (connectionInfo && connectionInfo.type !== 'none') {
        this.props.ConnectionActions.checkStatus();
      }
    });

As you can see i'm not trying to do anything complex.  All i ever want to do is to fire up some action based on network status

Take this up with Expo. "Expo 35" is not a react-native version.
If you can reproduce this with the example, there is a problem, but there is not a problem, there is something wrong with Expo in conjunction with this module, and it's an Expo thing
This is common when using expo and 3rd party native modules, it's an unfortunate side effect


Bugfix/delay error message doc upload

Trying to jQuery poll docUrl to solve #1151
Not working, don't merge.

Closing.


Visitor EV3

This is a PR for the visitor changes in the EV3 Plugin.
Things needing to be discussed (@rbudde)

[x] Do we need to rethink the Collector structure. Meaning should the interface SC be refactored to a enum and should all properties in it be documentated ? Problem is e.g. in visitPlayFileAction should I register a UsedActor with SC.SOUND or SC.MUSIC?
[x] When adding a internal UsedActor should we use - EMPTY_PORT - or an empty string as port ?
[x] Do we want Collector Tests like the EV3ProgramUsedHardwareCheckTest in ReuseIntegrationAsUnitTest ?  -> Integrate it!
[x] In DifferentialMotorValidatorAndCollectorVisitor#visitTurnAction should the warning for Zero speed also come if there is no duration ?
[x] In DifferentialMotorValidatorAndCollectorVisitor shouldn't be addLeftAndRightMotorToUsedActors() called in every method ?

I need to review this by myself, but I think this is finished.


fix: splitting up open-scd and plugins

closes #1419

The folder google inside plugins can be removed