Where is the 'correlation_threshold' parameter?

In the previous versions the below command was correct:
profile = pandas_profiling.ProfileReport(data,correlation_threshold=1.0)
In the 2.x version an error is reported:
profile = data.profile_report(correlation_threshold=1.0)
How can I set this parameter?
Thanks in advanced,

You can find the configuration settings here:
https://pandas-profiling.github.io/pandas-profiling/docs/#advanced-usage

Dear Simon,
I read configuration settings before, but there isn't any reference to
correlation parameters. There is anyway to indicate pandas-profiling
doesn't remove any variable?
Regards,
El mié., 24 jul. 2019 a las 11:41, Simon Brugman<EMAIL_ADDRESS>escribió:

You can find the configuration settings here:
https://pandas-profiling.github.io/pandas-profiling/docs/#advanced-usage
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pandas-profiling/pandas-profiling/issues/221?email_source=notifications&email_token=ADRXXNXRNL33Y2SCRUND223QBAPTHA5CNFSM4IGNZR6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2VY64Y#issuecomment-514559859,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADRXXNUO2IAXTUVH2K6VQTDQBAPTHANCNFSM4IGNZR6A
.


fix: some issues related to previous PRs

Have some fixes related to previously merged PRs:

Wait for block - delegation_delay in waitForPeriodFinalization to not slow down syncing
Check peer period in broadcastPreviousRoundNextVotesBundle

prnet cleaned-up successfully


Disable walking/running while performing expressions

Disable walking/running while performing expressions. Also disable expressions if the player is not on ground
related:
https://github.com/webaverse/app/issues/3362
result:
https://user-images.githubusercontent.com/60634884/182276463-7661852c-080c-44e4-8cc7-85b750e88798.mp4

This works fine. We also need to disable expressions while dancing. Should it be done in this PR or a separate pr.
https://user-images.githubusercontent.com/35926530/182678188-25215d56-e22d-4329-915d-9852008cf959.mp4

disable expression while dancing

@gonnavis fixed this in a better way with https://github.com/webaverse/app/pull/3558.
So closing this PR.


Internal Server Error - Incognito Mode did not work.

Background: A manager reported an "Internal Server Error" on Sept 18. She needed to submit a  telework agreement as an employee and approve her staff's telework agreements. She cleared her browser cache, and with her sharing her screen [with me] went into incognito/private mode for both Edge and Chrome. The Error message still persisted.
Recent changes from her: IDIR changed within the last 3 months,  last name changed, Email looks suspicious<EMAIL_ADDRESS>IDIR is: NFRANKS
Note: having a past employee the day before select Incognito mode did work, however this new manager was not able to bypass the error message with me watching. User had contacted the OCIO and said her IDIR was fine.
We need to find out how to troubleshoot this issue and its cause. SSO had recommended to contact the WAM team (web access management - not sure if this is the recommended step going forward).
Private Zenhub Image
Steps to Reproduce:
Actual Behavior:
E[Uploading... NFanks_InternalServerError.png]
xpected Behaviour: Employee should be able to login to the forms portal without an internal server message.
Environment: Prod

We have another user that reported an Internal Server Error on September 25th. Director needs to approve her employee's telework agreements and receives the error after she logins her IDIR credentials.  Please note: Last name has changed within the last month, OCIO indicated there are no duplicate conflicting IDIRS. However, employee has two active emails still open (that she uses).  IDIR: BGILLILA.  Keycloak record indicates she has no attributes listed.  I did not delete any attributes in the past.

[Uploading... B_GillilandError.png]

Thank you @chriscaldwell-psa  for the detailed description. I'll look into it and will get back to you.

For both users, I found error logs in the Keycloak pods indicating a problem:
ERROR: Duplicate key value violates unique constraint "uk_ru8tt6t700s9v50bu18ws5ha6"
  Detail: Key (realm_id, username)=(psa-digital-journey, user_email) already exists.

This means that when these two users logged into our Keycloak Instance, Keycloak identified them as existing users. They had temporary Keycloak user accounts, likely created when they were assigned as managers. The issue was that Keycloak couldn't merge their IDIR information with the existing temporary user account, as both were mapped based on the user's email.
I couldn't pinpoint the exact failure in the Keycloak login flow, as I couldn't find a useful log. However, I believe that if we delete their temporary user accounts and have them log in again, this should resolve the problem. We just need to add their manager group/role after a successful login.

I deleted their profile in Keycloak this morning and indicated to Nicola that she should be able to login and to let me know if she encounters any issues.
I'll update this ticket if she responds with any issues.

I deleted their profile in Keycloak this morning and indicated to Nicola that she should be able to login and to let me know if she encounters any issues.
I'll update this ticket if she responds with any issues.

Just received an update  that the first employee who had the issue no longer is receiving the error message - Removing their profile seems to be consistently working! thanks Iman :)

If we see this error again, we should tell a developer to look at the logs, as this is our server we have all the logs we need to diagnose the issue and recommend a solution.
Try not to put any private info - IDIR, emails as all this is public . Best to connect with developers directly


Update newdeploymenttemp.json

Hi, @pavanskd  - Thank you for the contribution!  This might be better suited as a parameter being provided as for example in this repo: https://github.com/Azure/azure-quickstart-templates/blob/master/quickstarts/microsoft.web/webapp-basic-linux/azuredeploy.json#L13-L18

Thank you for your contribution.  Given the greater than 30 days since last update, I'm going to close the PR but would love to work with you on any additional contributions you'd like to make.


WebsiteDocs-4: Added the English code-conduct.md which was present …

…in Chinese but wasn't present in English and how to access it in 'development.js' file and how to access the Chinese version too

This PR is associated to this main issue

@Segun-Ogundipe @Tianqi-Dotes PTAL.Thx

@Tianqi-Dotes sure thing, thanks for the review, let me get working on those

Hi, please resolve the conflict..

Hi, please resolve the conflict..

I have solved this conflict file because it's the first contribution by @IamMujuziMoses


solved #534 added monthly rent calculator

🚀 Pull Request
Added montly rent calculator
#534 ---
https://github.com/user-attachments/assets/5107bd45-949e-4b2c-b839-033c3502d61b
Related Issue
If this pull request addresses an issue, link to it here (e.g., Closes #123).

Type of Change
What kind of change does this pull request introduce?

[ ] 🐛 Bug fix
[ ] 🚀 New feature
[ ] 📄 Documentation update
[ ] 🛠 Code improvement/refactor
[ ] 🖼 UI/UX update
[ ] Other (please specify)

Screenshots (if applicable)
Add screenshots or screen recordings to demonstrate the new feature or fix.

Checklist
Please ensure the following tasks are complete before submitting the pull request:

[ ] I have tested my changes and ensured they work as expected.
[ ] I have added any necessary documentation or comments.
[ ] I have checked the existing issues and pull requests to avoid duplicates.
[ ] I have starred this repository.
[ ] I have reviewed my code for best practices and readability.
[ ] I have included any relevant tests (if applicable).

Additional Context
Add any other relevant information or context about the pull request here.

@gauravsingh1281 please add level3 sir

@gauravsingh1281 please review this

Hey @KapuluruBhuvaneswariVspdbct  , your changes are not visible, and Netlify has not generated your deploy preview. Please pass all the checks by Netlify.


Added explanation to README.md and some changed some spaces which relieve developer's eyes

Add explanation to the tutorial in README.md

So, um, did it accepted ?

Hi,
Thanks for taking the time to send a PR, but please try to focus on one thing at a time.
"Does the readme need this description paragraph?" is one question. I'm leaning "no".
Random whitespace changes in a bunch of other places is not part of that question, and would not be accepted as a PR on its own.


Environment vars not refreshing even after refreshenv

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
I have Windows Terminal 1.11.2921.0 installed and Powershell 7.2.0 installed.
I have just done a fresh install of NodeJS 16.13.0 and then close all terminal windows.
Open up Powershell 7 (which is set as the default for Win Terminal and try
node -v
it says node is not installed even after I try and force a refreshenv
Expected behavior
Expected output

C:\Users\aaron>node -v
v16.13.0

Actual behavior
node -v
node: The term 'node' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.2.0
PSEdition                      Core
GitCommitId                    7.2.0
OS                             Microsoft Windows 10.0.22000
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
https://user-images.githubusercontent.com/694964/143571362-c5edef5b-569a-4b0c-b277-5c1395e0b015.mp4

If refreshenv is a batch file it’s going to spawn a new cmd process and refresh the env vars in that process. This works for cmd as it runs it in process and thus updates the env vars for cmd. This is not the same for powershell or pwsh as it’s run in a separate process. You would need a powershell script that replicates refreshenv but in powershell itself.

Thanks for the fast response @jborean93
Looking into it refreshenv is a script that is installed with Choco. Your explanation sounds fine for teh refreshenv script, but if I quit Terminal / Powershell, shouldn't it then be loading up the new Env anyway?

You could open GUI for editing of environment variables and check if they was updated.

@iSazonov yup I can open the Windows GUI and check they have definitely updated.
I have been playing around with this and I am loading up Powershell through Windows Terminal and I don't think that program is closing fully.
I am going to link the Windows Terminal Issue I created and update this description here to ask "Is there a way to refresh the env variables in the current session" and if not can we add one?
https://github.com/microsoft/terminal/issues/11829

"Is there a way to refresh the env variables in the current session" and if not can we add one?

Yes you can enumerate the registry keys that store the env vars and use Set-Item "env:\$name" $value to set those updated env vars in the powershell process. Considering refreshenv isn't a builtin Windows thing I would doubt it would be something implemented by PowerShell itself but it can definitely be made by anyone and distributed through the PowerShell gallery.


Profanity Filters

Profanity filters need to be added to the account registration and unban request submissions.

Profanity detection has been added.


Scheduler Crashed and now won't re-create it's self

Rancher Version: 1.2.2
Docker Version: 1.12.5
OS and where are the hosts located? (cloud, bare metal, etc): AWS
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB) Single node, RDS Db
Environment Type: (Cattle/Kubernetes/Swarm/Mesos) Cattle
I went in to check on things this am and noticed that the scheduler agent was throwing these errors:
Activating (Failed to allocate instance [container:1i330653]: Bad instance [container:1i330653] in state [error]: Scheduling failed: No healthy hosts with sufficient resources available)

I don't see anything the logs and all of my hosts are responding as normal.

I have the same issue. 1.2.1

Same issue with this environment :
Rancher	v1.2.1
Cattle	v0.174.8
User Interface	v1.2.39
Rancher Compose	v0.12.1
Also one of our other environments has dropped down with this first alerting message, and we are unable to remount it correctly. Others environments are all ok.
The only differences we see is that upgrades are available in the interface for healthchek, ipsec, scheduler and network-service.
Environments that work well are up to date. Do we need to run update even if "No healthy hosts with sufficient resources available" ?

Closing in favor of https://github.com/rancher/rancher/issues/7482


copr: Add vectorize month_name

Signed-off-by: tw<EMAIL_ADDRESS>PCP #5751

What have you changed?
Add vectorize month_name
What is the type of the changes?

Improvement (a change which is an improvement to an existing feature)

How is the PR tested?

Unit test
Integration test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No

@sre-bot /run-integration-copr-test copr-test=pr/94

PTAL @tw666

@sre-bot /run-integration-copr-test copr-test=pr/94

@sre-bot /run-integration-copr-test copr-test=pr/94

@iosmanthus  @hicqu PTAL

@tw666 please fix DCO

@tw666 please fix DCO

I don’t know how to fix DCO.

@tw666 DCO fails because some commits are not committecd with -s.
Please use git commit -s instead of git commit.

Hi @tw666 , you can sign history commits with following command then commit -f.
git filter-branch -f --msg-filter \
  'git interpret-trailers --if-missing add \
    --trailer "Signed-off-by: $(git config user.name) <$(git config user.email)>"' \
  master..HEAD


Q: how to trigger the use of uniform buffers.

So, this is more of a support question than an issue, sorry about that! If inappropriate, I will delete it.
I would like clspv to generate SPIR-V with the use of uniform buffers for arguments, instead of storage buffers.
Is there some OpenCL syntax that will induce the use of uniform buffers? Or a clspv option?
The result I am trying to achieve, is having the kernel use constant memory which in CUDA speak would be __constant__ prefixed.
So far I tried --pod-ubo which did not help me.
So in short: How to get the src data of the kernel below in constant memory?
#define uint32_t	uint

__kernel
void foo
(
	__global const uint32_t* __restrict__ src,
	__global uint32_t* __restrict__ dst
)
{
	const uint32_t pindex = get_global_id(0);
	uint32_t s = src[pindex];
	dst[pindex] = s ^ 0xffffffff;
}

Thanks!

You can try --constant-args-ubo.
Note that you might also want to have a look at: --max-ubo-size, relaxed-ubo-layout and std430-ubo-layout.
I think it is completely fine to open issue to ask this question. But know that we have a discord server as well (linked in the README), it's up to you to use the one you prefer.

Feel free to reopen if one needs more help on that topic.


Pari/debug memory issue

Have set cuda clean up along with debugging code bug causing optimization error.

Sorry, sent here by mistake


Manually inject sidecar failed while Automatic inject successfully

Bug description
I use openshift 4.3, istio 1.5.2, and i install the istio CNI.
When automatic inject the sidecar, it is successful. But when i manually inject the sidecar ,it is failed, show error logs as:

in new validator: <IP_ADDRESS>
Listening on <IP_ADDRESS>:15001
Listening on <IP_ADDRESS>:15006
Error connecting to <IP_ADDRESS>:15002: dial tcp <IP_ADDRESS>:0-><IP_ADDRESS>:15002: connect: connection refused
Error connecting to <IP_ADDRESS>:15002: dial tcp <IP_ADDRESS>:0-><IP_ADDRESS>:15002: connect: connection refused
Error connecting to <IP_ADDRESS>:15002: dial tcp <IP_ADDRESS>:0-><IP_ADDRESS>:15002: connect: connection refused
Error connecting to <IP_ADDRESS>:15002: dial tcp <IP_ADDRESS>:0-><IP_ADDRESS>:15002: connect: connection refused
Error connecting to <IP_ADDRESS>:15002: dial tcp <IP_ADDRESS>:0-><IP_ADDRESS>:15002: connect: connection refused

I comprae the two kinds of pod spec(through oc describe pod), and found the difference as bellow
Manually inject:

istioctl kube-inject -f sleep.yaml -o sleep-injected.yaml --injectConfigMapName istio-sidecar-injector

oc create -f sleep-injected.yaml

apiVersion: v1
kind: Pod
metadata:
annotations:
k8s.v1.cni.cncf.io/networks-status: |-
[{
"name": "openshift-sdn",
"interface": "eth0",
"ips": [
"<IP_ADDRESS>"
],
"dns": {},
"default-route": [
"<IP_ADDRESS>"
]
}]
openshift.io/scc: anyuid
sidecar.istio.io/interceptionMode: REDIRECT
traffic.sidecar.istio.io/excludeInboundPorts: "15020"
traffic.sidecar.istio.io/includeOutboundIPRanges: '*'

Automatic inject：

apiVersion: v1
kind: Pod
metadata:
annotations:
k8s.v1.cni.cncf.io/networks: istio-cni
k8s.v1.cni.cncf.io/networks-status: |-
[{
"name": "openshift-sdn",
"interface": "eth0",
"ips": [
"<IP_ADDRESS>"
],
"dns": {},
"default-route": [
"<IP_ADDRESS>"
]
},{
"name": "istio-cni",
"dns": {}
}]
openshift.io/scc: anyuid

The only difference is the annotations k8s.v1.cni.cncf.io/networks: istio-cni. When I add the annotation to file sleep-injected.yaml, and manually inject again, it is Successful! So i think the command istioctl kube-inject may lost some configuration when it running.
Expected behavior
Steps to reproduce the bug
Every time when i manually inject sidecar
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
How was Istio installed?
Follow the instruction istio supplied
Environment where bug was observed (cloud vendor, OS, etc)
`# cat /etc/*-release
CentOS Linux release 7.4.1708 (Core)
NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"
CENTOS_MANTISBT_PROJECT="CentOS-7"
CENTOS_MANTISBT_PROJECT_VERSION="7"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="7"
CentOS Linux release 7.4.1708 (Core)
CentOS Linux release 7.4.1708 (Core)
`

The istioctl binary does not support the CNCF networks annotation in istio 1.5. The workaround is adding the k8s.v1.cni.cncf.io/networks: istio-cni annotation to the deployment manually before applying the output of istioctl kube-inject in CNI mode. See https://github.com/istio/istio/issues/24425


100% coverage for contrib.appengine

@dhermes this module and its existing tests are really quite messy, so I'm sorry if these tests seem a bit off.
Towards #212

@dhermes friendly ping on this one.

Finished review. Close to LGTM but no cigar.

Everything has been addressed, should be cigar now.

LGTM
Pretty please file an issue to encapsulate some of the issues / notes you're referring to in the module under test.


swagger-2.0.0-SNAPSHOT: Swagger 2.0 throws NoSuchBeanDefinitionException during startup

swagger-2.0.0-SNAPSHOT
The configuration:

EnableSwagger2
DocumentationType.SWAGGER_2

@SpringBootApplication
@EnableSwagger2
public class Application {

    public static void main(String[] args) {
        SpringApplication.run(Application.class, args);
    }

    @Bean
    public Docket swaggerSpringMvcPlugin() {
      return new Docket(DocumentationType.SWAGGER_2)
              .groupName("business-api")
              .select()
                 //Ignores controllers annotated with @CustomIgnore
                //.apis(not(withClassAnnotation(CustomIgnore.class)) //Selection by RequestHandler
                //.paths(paths()) // and by paths
                .build()
              .apiInfo(apiInfo());
              //.securitySchemes(securitySchemes())
              //.securityContext(securityContext());
    }

    private ApiInfo apiInfo() {
	      ApiInfo apiInfo = new ApiInfo(
	              "My Apps API Title", "My Apps API Description", "1.0",
	              "My Apps API terms of service", "My Apps API Contact Email",
	              "My Apps API Licence Type", "My Apps API License URL"
	        );
	      return apiInfo;
	    }

    //Here is an example where we select any api that matches one of these paths
    /*private Predicate<String> paths() {
      return or(
          regex("/business.*"),
          regex("/some.*"),
          regex("/contacts.*"),
          regex("/pet.*"),
          regex("/springsRestController.*"),
          regex("/test.*"));
    }*/

}

causes Exception during startup:
Caused by: org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type [springfox.documentation.swagger.mappers.ServiceModelToSwaggerMapper] found for dependency: expected at least 1 bean which qualifies as autowire candidate for this dependency. Dependency annotations: {@org.springframework.beans.factory.annotation.Autowired(required=true)}

The class ServiceModelToSwaggerMapper is not mentioned in the transition guide (https://github.com/springfox/springfox/blob/master/transitioning-to-v2.md), should it be configured somewhere?

@jfiala It should automatically be pulled in if you have the @EnableSwagger[2] annotation.

I have set the @EnableSwagger2 annotation (see above), but I get the exception during startup.

Could you look at whats different between the demo application and what you have? Also how are you launching the application

I copied my controllers to the demo, now it is working.
Minor:
http://localhost:8080/swagger2/webjars/springfox-swagger-ui/2.0.0-SNAPSHOT/sdoc.jsp
SwaggerUI has by default the wrong API-URL:
current: http://localhost:8080/swagger2/api-docs
correct: http://localhost:8080/swagger2/v2/api-docs

In fact, including both dependencies to springfox-swagger2 and springfox-swagger was responsible for the exception.
<dependency>
    <groupId>io.springfox</groupId>
    <artifactId>springfox-swagger2</artifactId>
    <version>2.0.0-SNAPSHOT</version>
</dependency>
<!-- for swagger spec 1.2 optionally -->
<dependency>
    <groupId>io.springfox</groupId>
    <artifactId>springfox-swagger</artifactId>
    <version>2.0.0-SNAPSHOT</version>
</dependency>

After I remove one of the both dependency, everything is working fine.

maybe a hint should be added in the docs that they are not usable in parallel.

hmm thats weird because they totally work together in our contract-tests

@dilipkrish I reckon we shouldn't support both spec versions 1.2 and 2.0 in parallel?

I'll test it out in the demo app and see if its a problem

Verified via this commit

Here you can reproduce:
https://github.com/jfiala/swagger-spring-demo/tree/master/user-rest-service-2.0.0
simply check out user-rest-service-2.0.0 and do mvn spring-boot:run and you get
org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type [springfox.documentation.swagger.mappers.ServiceModelToSwaggerMapper]

during startup.

Just for own interest: Why is there no reference to the issue in your commit comment (which would it probably make easier to track commits)?

@jfiala thats because its in a different repo

@jfiala I've not been able to reproduce it using my fork. Here is the relevant commit

I justed committed the same change as you did.
https://github.com/jfiala/swagger-spring-demo/commit/466991dbace6c201beb6c0bdd979afe0a875e87f
In fact, the behaviour changes to:
With Swagger / DocumentationType.SWAGGER_12
Exception in thread "main" java.lang.NullPointerException: null key in entry: null=springfox.documentation.spring.web.plugins.Docket@1b35db6f
With parallel + Swagger2 / DocumentationType.SWAGGER_2 it's still the same:
Caused by: org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type [springfox.documentation.swagger.mappers.ServiceModelToSwaggerMapper] found for dependency: expected at least 1 bean which qualifies as autowire candidate for this dependency. Dependency annotations: {@org.springframework.beans.factory.annotation.Autowired(required=true)}


Erasure coding: Failover of the ErasureCodingManager

The failover of the ErasureCodingManager needs to be properly implemented and tested. Right now, the leading NameNode will automatically takeover and execute the ErasureCodingManager in case a previously existing manager failed. However, running encoding or repair jobs are currently neither recovered nor aborted. This should be fixed by storing information about running jobs so that their state can be recovered.

It should be possible to recover the running encoding job by persisting their JobID. A newly started MapReduceEncodingManager then has to recreate the respective RunningJobs and report their state to the ErasureCodingManager. The Report should be extended to include the information necessary to delete this log entry after success or failure of the job. The ErasureCodingManager then needs to delete these log entries in the same transaction as it is using to update the state.


Feature/harsh yash/login screen

Tracking Info
Resolves #25 #15
Make sure your branch name conforms to: <feature|staging|bugfix|...>/<username>/<3-4 word description separated by dashes>. Otherwise, please rename your branch and create a new PR.
Changes
We finished the login screen view, the user can input their email and password to log in. The XCode project is connected to the Firebase, and the Firebase Authentication services are used. Errors are presented to the user if the login fails.

TODO

Testing
We ran the file on the iOS simulator and made sure all the proper errors and routes show up.

TODO

Confirmation of Change

TODO...

Not sure if it's supposed to show both incorrect password error and account error at the same time, but if that's ok then everything else looks good!

From our discussion during the meeting, there seemed to be couple things that we said we will complete later. Can you make issues for them and leave it unassigned so that we don't loose track? @HarshGurnani @YashRavipati1

Not sure if it's supposed to show both incorrect password error and account error at the same time, but if that's ok then everything else looks good!

I don't think this is desired. What do you think? @HarshGurnani @YashRavipati1


Complex coefficients are converted to floats when entered by hand

This behaviour ist most likely caused by safe_eval() default format float in input_coeffs.py

I had fixed this already ... however, when editing coefficients in fixpoint format, only real-valued coefficients are stored. This is ok for the moment, does anyone want to design complex-valued fixpoint filters?


yield from generate meant for one user reaches another user

Must be gradio bug.

Should be fixed by streaming rotating hash too now, was only predict, not submit, before.


Select options doesn't scroll down on using the arrow down  when using the hook

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
Steps:

Open the select
Use the keyboard arrow down to navigate

Current behavior 😯
The scroll of select option doesn't scroll on using the keyboard navigation
https://github.com/mui/material-ui/assets/19767710/e2213114-06a0-412d-93b5-1d676e342d2d
.
Expected behavior 🤔
The menu list should scroll down using the keyboard arrow navigation
Context 🔦
No response
Your environment 🌎

npx @mui/envinfo
System:
OS: macOS 13.4
Binaries:
Node: 16.19.0 - ~/.nvm/versions/node/v16.19.0/bin/node
Yarn: 1.22.18 - /opt/homebrew/bin/yarn
npm: 8.19.3 - ~/work/cogs.js/node_modules/.bin/npm
Browsers:
Chrome: 114.0.5735.133
Edge: Not Found
Firefox: Not Found
Safari: 16.5
npmPackages:
@emotion/react: 11.10.6 => 11.10.6
@emotion/styled: 11.10.6 => 11.10.6
@mui/base: 5.0.0-beta.4 => 5.0.0-beta.4
@mui/core-downloads-tracker:  5.10.1
@mui/joy: 5.0.0-alpha.41 => 5.0.0-alpha.41
@mui/material: 5.9.2 => 5.9.2
@mui/private-theming:  5.9.1
@mui/styled-engine:  5.10.1
@mui/system:  5.10.1
@mui/types:  7.1.5
@mui/utils:  5.9.1
@types/react:  17.0.40
react: 17.0.2 => 17.0.2
react-dom: 17.0.2 => 17.0.2
styled-components: 4.4.1 => 4.4.1
typescript: 4.6.3 => 4.6.3
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://mui.com/r/issue-template), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve


Pull request for verification of Federal University Dutse

Institution/School Name
Federal University Dutse
Instiution/School Website
FUD
Fud Courses
Email Users
Please place an x between the square brackets if yes

[ x] Students
[x] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[x] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

@bbkrmuhd Pull request merged. Thank you!

Thank you too.


tweak(keyviz): update tidb table map immediately

Codecov Report

Merging #1219 (b0a79a3) into master (e79a650) will decrease coverage by 7.59%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1219      +/-   ##
==========================================
- Coverage   31.66%   24.07%   -7.60%
==========================================
  Files         291       92     -199
  Lines       17115     9314    -7801
  Branches      686        0     -686
==========================================
- Hits         5419     2242    -3177
+ Misses      11465     6932    -4533
+ Partials      231      140      -91

Flag
Coverage Δ

be_integration_test_latest
?

be_integration_test_v4.0.1
?

be_unit_test
24.07% <0.00%> (-0.01%)
:arrow_down:

e2e_test
?

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

Continue to review full report at Codecov.

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


check for Lnz overflow

Fixes #24

Pull Request Test Coverage Report for Build 132

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 127:
0.0%

Covered Lines:
84

Relevant Lines:
84

💛  - Coveralls

It seems that checking for signed overflow as I have done it is not well defined behaviour in C.   I will look for another way.

@bstellato I have rewritten the overflow check to avoid undefined signed int overflow behaviour.
There are two options for this:

Use <limits.h> to extract the maximum value of whatever type is used for QDLDL_INT.   This is the easiest and safest way but requires the new header (but not a library).    QDLDL_INT_TYPE_MAX is defined via CMakeLists.txt in this case.

Alternatively, we could try to compute the maximum value of QDLDL_INT directly without the header.   I have done as an alternative method in the qdldl_types.h.in and commented it out.   It looks complicated b/c its necessary to carefully calculate the bound without actually overflowing (i.e. 2^(N-1)-1 is not safe because 2^(N-1) already overflows).   I believe the commented implementation is safe and would work on any 2's complement based architecture.


passing flags to make when using cmake-ide-compile (-j4)

Hi,
Is there a good way to pass a flag to make, e.g. -j4 (compile with 4 cores)?
Currently I changed the line in cmake-ide--get-compile-command from:
(concat cmake-ide-make-command " -C " dir)
to:
(concat cmake-ide-make-command " -j4 -C " dir)
but I was wondering if there were a better way to do it.
Thanks

Add the -j4 to cmake-ide-flags-c and/or cmake-ide-flags-c++. Or in this specific case of -j4, just use ninja, which is way better than make anyway ;)

@yavitzour , try to use helm-make. This package is nice addition to cmake-ide.
It allows to switch compilation target and specify make -j4 options.

Add the -j4 to cmake-ide-flags-c and/or cmake-ide-flags-c++.
Does this solution still work at the newest version?

(use-package cmake-ide
  :ensure t
  ;;; ...
  :config
  (setq cmake-ide-flags-c++ "-j4"))

-j4 isn't a C++ compiler command-line parameter, it's a make or ninja one. You can set cmake-ide-compile-command to make -j4 however.

I see, so what you suggested above is no longer a valid approach, and we can use cmake-ide-compile-command instead.
That's good, but that overwrites the whole make command, instead of adding the given parameter to make, which is not ideal because then cmake-ide-compile won't be able to build for the missing build path.
For example,
If I call cmake-ide-compile, the default make command goes:
make --no-print-directory -C <your build path>

After manipualting cmake-ide-compile-command, it goes:
;; (setq cmake-ide-compile-command "make -j4")
make -j4
make: *** No targets specified and no makefile found.  Stop.

Can we selectively add some make parameterswithout having to make direct changes tocmake-ide.el`? If not, open to a PR?

I use (setq cmake-ide-make-command "make -j8") which works fine for me. (well I'm not up to date with last version).

I see, so what you suggested above is then no longer a valid approach?

I don't think that was ever a valid approach, I don't know what I was thinking when I wrote that.

That's fine, but that overwrites the whole make command, instead of adding the given parameter to make,

You could overwrite the whole make command yourself as well.

If we can't at the current stage, open to a PR?

I'm open, yes, but I don't know how that would work. I'd suggest just using ninja and forgetting about manually specifying how many threads to use.


Spell Compendium Admin Files

The second help tab (generell_help_modifier.tab)  contains a few lines that overwrite existing lines in help.tab. This is done to support the new spells help topic.

The second help tab (generell_help_modifier.tab) contains a few lines that overwrite existing lines in help.tab. This is done to support the new spells help topic.

Ok that’s an important issue actually.
You have to remember that ToEE also has non english versions. With that in mind, we should override as few original entries as possible, at least in the general tpgamefiles.
You have more leeway in co8infra since that is only added when Co8 is installed, which is inherently english only.
The other thing is that I don't think it's necessary to add the new classes to the spell help files. It'll just be a huge mess, keeping it core-only is fine and the new classes can link to their own spell lists.

Ok, should I completly drop the new spells overview entry in the help then?
These are the new entries I made:

Spells List scrolls down for all casting classes:

Example window for a spell level (where you can select the according spell for details)

Modifier entry:

Clicking on the entries describes the bonus/modifier type (flavor)
To make this work, I had to modify following help entries:

All core assassin and blackgurad spell help entries (e.g. adding TAG_BLACKGUARD_1 as a tag to every core Blackguard level 1 spell)
Modify the blackguard prestige class to support spell lists (you can now jump to the spell list now from the class entry)
Modifiy the assasin prestige class to support the spell lists (you can now jump to the spell list now from the class entry)
Modify Tag_Root (adding Modifier and Spell List Tab)

All actual spell help files for the new spells are stored in a different extender file, they are not influenced by that.
I would like to keep the Modifer entries, I linked to them in a lot of spell help entries, but they can be kept without a root entry if needed.

You can add this stuff to co8infra.
BTW this discussion is relevant for mes files as well.

You can add this stuff to co8infra.
BTW this discussion is relevant for mes files as well.

I am unsure what mes files you a refering to, but I personally would have no problem to move all my stuff to co8 if this prefered? Can also do a new folder that reqiuires co8infra if this would be even better for you?
In the end, I don't think, it makes a lot of sense to use new spells but ignore the co8 stuff anyways? I have never thought about this though, cause I use the co8 stuff since it was released.

You can add this stuff to co8infra.
BTW this discussion is relevant for mes files as well.

I am unsure what mes files you a refering to, but I personally would have no problem to move all my stuff to co8 if this prefered? Can also do a new folder that reqiuires co8infra if this would be even better for you?
In the end, I don't think, it makes a lot of sense to use new spells but ignore the co8 stuff anyways? I have never thought about this though, cause I use the co8 stuff since it was released.

Just to make sure we're on the same page, I'm talking about the co8infra in /tpdatasrc, that is then packed to co8infra.dat.
Re. Mes files, for example the damage.mes file.


RATIS-1166. Link DataStream with LogEntryProto.

See https://issues.apache.org/jira/browse/RATIS-1166

Once a log entry is created for a DataStream, the server should link log entry with the DataStream so that the StateMachine must not delete the data unless the log entry is truncated.

@szetszwo Hi, I have one question. link log entry with the DataStream is aim for we can delete the data when truncate the log entry ?

Yes, one of the goals is to let state machine delete the data when truncating the log entry.
Another goal is to replicate the data with the log entry to other servers (not yet implemented).
The idea is similar to the state machine data stored by StateMachine.DataApi.write(..).  The state machine should manage the stream data the same way.

@szetszwo Got it. Thanks for explanation.


Basic chart examples

Tried to combine and put together rough.js and several basic charts
Hope this link land on your example page
https://beta.observablehq.com/@bumbeishvili/trying-out-rough

Really nice. I will add this to the page.
Btw, you can make the map look a bit more hand drawn by setting the simplification  (0.2-0.5 may be a good value for US map) https://github.com/pshihn/rough/wiki#simplification
Of course, people may not want simplification.

Added to https://github.com/pshihn/rough/wiki/Examples

@pshihn
Nice catch, it looks better after simplifying  I think


cat: /proc/sys/fs/binfmt_misc/qemu-aarch64: No such file or directory

After I run cat /proc/sys/fs/binfmt_misc/qemu-aarch64 refering to this(https://github.com/NVIDIA/nvidia-docker/wiki/NVIDIA-Container-Runtime-on-Jetson#enabling-jetson-containers-on-an-x86-workstation-using-qemu), I get error as : cat: /proc/sys/fs/binfmt_misc/qemu-aarch64: No such file or directory. Please help me to resolve this.
Enviroment:
OS: Ubuntu 18.04
Device: Jetson nano

Hi @Darshcg! This repository is for amd64/x86_64 hosts only (#77). For other host archs, you can use dbhi/qus:

images are provided for each of seven host architectures officially supported by Docker, Inc. or built by official images: amd64, i386, arm64v8, arm32v7, arm32v6, s390x and ppc64le.


Regenerate projects to fix master

https://grpc-testing.appspot.com/job/gRPC_master/17412/

LGTM

Seems that some funny conflict happened between https://github.com/grpc/grpc/pull/6750 and renaming _win32 suffix.


Fix for error: use of undeclared crate or module zingo_sync

When compiling from source per README, I cant compile. Verified on both my pi and linux pc.
Steps to reproduce
 git clone --recurse-submodules https://github.com/zingolabs/zingolib.git
 cd zingolib
 cargo build --release --package zingo-cli

Error

Fix
in Cargo.toml
default = ["zingo-sync"]
zingo-sync = ['dep:zingo-sync']

@Oscar-Pepper I thought that zingo-sync wasn't required for anything yet?


Rreplace facebooks hyperclick with something that's maintained

https://github.com/steelbrain/declarations/issues/2#issuecomment-191951080

Yeah :disappointed:  Also some relevant exchange and links here: https://github.com/facebook/nuclide/issues/229#issuecomment-188757899


Prefab export: invalid Java package name

Describe the bug
if the name package contains a '-' character (eg "boost-endian") then the generated package name will be wrong in AndroidManifest.xml
<manifest xmlns:android="http://schemas.android.com/apk/res/android" package="com.vcpkg.ndk.support.boost-endian" a>
    <uses-sdk android:minSdkVersion="16" android:targetSdkVersion="29" />
</manifest>

https://docs.oracle.com/javase/tutorial/java/package/namingpkgs.html
Environment

OS: Ubuntu 20.04.1 LTS
Compiler: gcc 9.3.0

To Reproduce
Steps to reproduce the behavior:

./vcpkg install boost-endian:arm-android boost-endian:arm64-android boost-endian:x86-android boost-endian:x64-android
./vcpkg export --triplet x64-android boost-endian --prefab --prefab-maven --prefab-debug
add implementation 'com.vcpkg.ndk.support:boost-endian:1.75.0' in app/build.gradle
click build in android studio

Expected behavior
Build success.
Failure logs
> Task :app:processDebugResources FAILED

Execution failed for task ':app:processDebugResources'.
> A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
   > Package 'com.vcpkg.ndk.support.boost-endian' from AndroidManifest.xml is not a valid Java package name as 'boost-endian' is not a valid Java identifier.

Lack of android knowledge, @vicroms can you please take a look?
Thanks.

I am meetting the same issue. but this is an error of vcpkg-tool. There are some errors there and I am trying to fix them.

I don't think doing it manually is the solution to the problem. :)

This issue hasn’t been updated in 3 month, if it is still an issue, please reopen this issue.


WIP testing documentation

Got bored, but maybe its already a bit helpful

This almost reads to me like an e2e test. Wouldn't it be nice to translate this into one somehow? In my last project I really liked using https://www.cypress.io.

Yes, I also thought that automating this would be super nice :)


fix: weather negative temps

The weather plugin crashes on negative temperatures as it is forwarding negative temperature values to drawNumbers(), where we're accessing smallNumbers[numbers.at(i)]. With numbers.at(i) being negative, the resulting instruction (e.g. smallNumbers[-1]) will crash the device.
The actual fix is provided in 22314b2cef3e9e88240fde68d2a15ccaa03642f8, but I also changed the EOL from CRLF to LF, which is why the entire file was rewritten in bf71d1db226d9647777896144e58056a7ee7be1d. Feel free to cherry-pick.

Thank you very much! :) Could you please check the formatting as everything in the file has changed. After that I will merge the fix

I didn't change the formatting. Or do you see any problems? If you prefer I revert the line ending change.

Hmm, really strange. The problem is that no one can see the changes you've made, because every single line has changed. Can you maybe force push it without your newline changes?

That's why I added two separate commits. One with the code change, one with the line ending change. But now it's just the code change. Hope that helps :)

Ah :) Thank you! Merged


Is the Voice-to-Voice better than RVC?

Looking for a better free service than RVC's V2V--how is this V2V service? Thanks!
Will donate if it's good :)
<3

This repo only supports TTS. Closing as the question is out of scope.


[terraform] update the tf_lambda module to remove complexity

Background
The tf_lambda/main.tf file contains resources like function_vpc and function_no_vpc. The only difference between these resources is the presence of a vpc_config.
Desired Change
With the introduction of Terraform 12's null values - we can prevent blocks from getting 'applied' simply by setting them to the value of null. Therefore, we do not need to do janky stuff like count         = var.enabled && local.vpc_enabled ? 1 : 0 to conditionally created resources. We can now have a singular resource aws_lambda_function and have a variable of vpc_config that has a default value of null. This also applies to the aws_lambda_alias resources. The various outputs in outputs.tf will also need to be updated/simplified.
This is being used elsewhere with success: https://github.com/airbnb/streamalert/blob/616c8ff4ada443bb4e1e54f28264b30e22f4622a/terraform/modules/tf_cloudtrail/variables.tf#L54
https://github.com/airbnb/streamalert/blob/616c8ff4ada443bb4e1e54f28264b30e22f4622a/terraform/modules/tf_cloudtrail/main.tf#L95-L96

fixed with #1225


[Ukrainian] Translate /advanced/structuring.md

It seems like #705 wasn't synced, right?

@KnorpelSenf Because these changes affect many files that have not yet been reviewed, it would be difficult to apply these changes to every translation. I decided that we would gradually apply these changes during the review process and, after translation, apply them to the pages that have already been fully translated.


fault during compiling apps in command prompt

sorry for my english, but i have problem with compiling apps for wixel 1337.
please look to my topic in polish forum with that problem
thanks in advance
link:
http://www.elektroda.pl/rtvforum/topic3098109.html#15050102

It says something about an empty source file which means that something
either moved the dexdrip.c file or renamed it (or added a different
extension to it or something)
On Thu, Oct 8, 2015 at 2:18 PM sarunia<EMAIL_ADDRESS>wrote:

sorry for my english, but i have problem with compiling apps for wixel
1337.
please look to my topic in polish forum with that problem
thanks in advance
link:
http://www.elektroda.pl/rtvforum/topic3098109.html#15050102
—
Reply to this email directly or view it on GitHub
https://github.com/StephenBlackWasAlreadyTaken/wixel-xDrip/issues/40.

Please compile files and pack to .rar archive and send to me by email,
My transmitter ID is 6ACTL, maybe my polish windows 7 version something
block, I don`t know,
thank you
W dniu 2015-10-08 o 21:19, Stephen Black pisze:

It says something about an empty source file which means that something
either moved the dexdrip.c file or renamed it (or added a different
extension to it or something)
On Thu, Oct 8, 2015 at 2:18 PM sarunia<EMAIL_ADDRESS>wrote:

sorry for my english, but i have problem with compiling apps for wixel
1337.
please look to my topic in polish forum with that problem
thanks in advance
link:
http://www.elektroda.pl/rtvforum/topic3098109.html#15050102
—
Reply to this email directly or view it on GitHub
https://github.com/StephenBlackWasAlreadyTaken/wixel-xDrip/issues/40.

—
Reply to this email directly or view it on GitHub
https://github.com/StephenBlackWasAlreadyTaken/wixel-xDrip/issues/40#issuecomment-146661187.

Hello Stephen,
can you compiling apps to my wixel module?
My ID transmitter is 6ACTL,
i can`t make this step i command prompt- something is bad
slawek from poland
W dniu 2015-10-08 o 21:19, Stephen Black pisze:

It says something about an empty source file which means that something
either moved the dexdrip.c file or renamed it (or added a different
extension to it or something)
On Thu, Oct 8, 2015 at 2:18 PM sarunia<EMAIL_ADDRESS>wrote:

sorry for my english, but i have problem with compiling apps for wixel
1337.
please look to my topic in polish forum with that problem
thanks in advance
link:
http://www.elektroda.pl/rtvforum/topic3098109.html#15050102
—
Reply to this email directly or view it on GitHub
https://github.com/StephenBlackWasAlreadyTaken/wixel-xDrip/issues/40.

—
Reply to this email directly or view it on GitHub
https://github.com/StephenBlackWasAlreadyTaken/wixel-xDrip/issues/40#issuecomment-146661187.

everything working!!!


Blast shield at Nishina disappears

One of the blast shields on the landing pad disappears when it is being lowered.
Details
When first arriving:

After the animation for lowering the blast shield starts:

It's just this one that is affected. The others are fine.
Game Version
<IP_ADDRESS>
Language
English (EN)
Have you recreated this issue on a new game?
Yes, I can recreate this issue.
Reproduction Steps
Land at Nishina on Freya III (part of the Entangled mission).

I too was able to reproduce this bug.

This was a bit more complicated that I thought it would be.
MQ207CLandingMarkerRef [REFR:0023B8A5] has a linked Ref to [REFR:0023B8A6] (This is the blast shield that is disappearing when lowering).
No matter what blast shield I linked here, the linked shield would disappear. If there was none linked, the shield animations did not work.
I looked at other landing markers and blast shields and found that they were using an enable marker with a keyword; LinkedRef_LandingPadShield_EnableMarker [KYWD:0028B2A0]
I created a new Enable Marker, and linked it using this keyword on 0023B8A5 in addition to the already linked blast shield.
This kept the shield from disappearing, but now many of the shields were not raised when landing (the were missing linked ref). I went through all of them and relinked them in order. The red lines are the links that were in vanilla, the yellow is what they are now.
0023B8A6 links to 0023B8A7 which means when 0023B8A6 starts to move, then 0023B8A7 will begin it's animation.
0023B8AE is the last in the chain so is not linked to anything.

Fix attached
NishinaBlastShieldFix.zip

Updated the ESM attached so that 0023B8A5 has the changes previously made by SFCP (removed incorrect properties CloseSoundMarker and OpenSoundMarker).
NishinaBlastShieldFix.zip

Some of these edits were ITMs, so I've excluded them.


Only record to cypress dashboard on master runs

Didn't realize I'd hit the record max so quickly on the cypress dashboard free plan. (This error also fails the circleci build). Just recording on master should be fine, though could potentially be a pain if I hit a limit off of that, and then master is failing.
For now I'm going to disable dashboard records in #8 and then reenable just for master in june.
We encountered an unexpected error talking to our servers.

There is likely something wrong with the request.

The server's response was:

StatusCodeError: 402

{
  "code": "FREE_PLAN_EXCEEDS_MONTHLY_TESTS",
  "payload": {
    "code": "FREE_PLAN_EXCEEDS_MONTHLY_TESTS",
    "name": "FreePlanExceedsMonthlyTests",
    "subject": "billing",
    "message": "Organization with free plan has exceeded monthly test recordings limit.",
    "used": 522,
    "limit": 500,
    "orgId": "666a3828-59ff-40cd-a842-11722127bb23",
    "buildUrl": "https://dashboard.cypress.io/#/projects/d43yi2/runs/17"
  }
}
error

This was closed in #14


webgl2.0(experimental feature) is now default on Safari15

After updated Safari to 15.0 on Mac,  webgl2.0 is now default and text is not showing on ttyd.
Looks this is a xtermjs issue (https://github.com/xtermjs/xterm.js/issues/3357) but can we fall back to dom rendering if webgl2.0 is not supported?

Looks like xtermjs disabled webgl on safari on Sep 3.  Can we bump xtermjs version to the latest?
https://github.com/microsoft/vscode/commit/71830d5896c8ea2afdf1bb4ee8b6f76c635e4062#diff-8df911bf58ca35baec7e0c02b60c36c1dc33694f7a61089ba69adf888c98f466


Is netcdf4 really mandatory ?

Title says it all :)

indeed good point.
We've implemented netcdf to test exports in io_test.py

I might also add shapely because it only used here:
from shapely.geometry import LinearRing
[...]
            # Determine if boundary is clock-wise
            if len(current_boundary_nodes) > 2:
                boundary_geom = LinearRing(
                    [
                        (self.x[n - 1], self.y[n - 1])
                        for n in current_boundary_nodes[:-1]
                    ]
                )
                ccw = boundary_geom.is_ccw
            else:
                ccw = True  # arbitrary value (order can not be defined!)

in Serafin.py
If we find another implementation we could also get rid of the dependency to shapely

The are other issues with the dependency metadata.
$ curl --silent https://pypi.org/pypi/xarray-selafin/json | jq '.info.requires_dist'
[
  "numpy",
  "pytest",
  "scipy",
  "shapely",
  "xarray",
  "netcdf4",
  "dask; extra == \"dask\"",
  "matplotlib; extra == \"dev\""
]

For instance:

why is pytest getting installed when you install xarray-selafin?
why do we install the dev "extra" for matplotlib
dask doesn't  have an "extra" named dask
There is also #2 which is related.

The pyproject.toml has a few inconsistencies indeed:
[tool.poetry.dependencies]
python = ">=3.9"
numpy = "*"
pytest = "*"
scipy = "*"
shapely = "*"
xarray = "*"
netcdf4 = "*"

[tool.poetry.group.dask.dependencies]
dask = "*"

[tool.poetry.plugins."xarray.backends"]
selafin = "xarray_selafin.xarray_backend:SelafinBackendEntrypoint"

[tool.poetry.group.dev.dependencies]
matplotlib = "*"

I see already a few fixes to address this issues:

[ ] move pytest to [tool.poetry.group.dev.dependencies]
[ ] matplotlib is not used in the package and can be removed
[ ] dask is not used in the package, and there is no intention of implementing it soon, so it can be removed too

Tasks to close this issue:

[ ] Remove netcdf4 from the list of dependencies i.e. removing it from the tests
[ ] Move scipy to the dev group; it's only used in the tests.
[ ] Consider whether we want to keep on using shapely. It adds GEOS as a binary dependency. check if TELEMAC is_ccw method can replace LinearRing is_ccw class method: https://shapely.readthedocs.io/en/stable/reference/shapely.is_ccw.html#shapely.is_ccw I.e. is_ccw(g) & is_valid(g)

this simple function (method found on SO):
def signed_area(vertices, ax):
    # Initialize area
    area = 0.0
    # Calculate the signed area
    for i in range(len(vertices)):
        x1, y1 = vertices[i]
        x2, y2 = vertices[(i + 1) % len(vertices)]
        area += (x2 - x1)*(y2 + y1)
    return area / 2.0

does the job.
I have compared it this way:
import matplotlib.pyplot as plt
from shapely import LinearRing
from shapely.geometry import Polygon

def signed_area(vertices, ax):
    # Initialize area
    area = 0.0
    # Calculate the signed area
    for i in range(len(vertices)):
        x1, y1 = vertices[i]
        x2, y2 = vertices[(i + 1) % len(vertices)]
        ax.plot([x1, x2], [y1, y2])
        ax.plot(*zip(*vertices))
        area += (x2 - x1)*(y2 + y1)
    return area / 2.0

def is_ccw(vertices, ax):
    # A positive signed area indicates a counter-clockwise orientation
    return signed_area(vertices, ax) < 0

# Example usage:
polygon =  [[[173.283, -39.474],[172.829, -39.507],[172.391, -39.607],[171.985, -39.769],[171.625, -39.988],
            [171.326, -40.255],[171.098, -40.562],[170.952, -40.896],[170.894, -41.245],[170.927, -41.597],
            [171.052, -41.936],[171.265, -42.251],[171.560, -42.528],[171.924, -42.756],[172.345, -42.926],
            [172.804, -43.031],[173.283, -43.067],[173.763, -43.031],[174.222, -42.926],[174.643, -42.756],
            [175.007, -42.528],[175.302, -42.251],[175.515, -41.936],[175.640, -41.597],[175.673, -41.245],
            [175.615, -40.896],[175.469, -40.562],[175.241, -40.255],[174.942, -39.988],[174.582, -39.769],
            [174.176, -39.607],[173.738, -39.507],[173.283, -39.474],]]

polygon1 = [[[-122.412, 37.804],[-122.507, 37.778],[-122.501, 37.733],[-122.412, 37.804]],
            [[-122.449, 37.782],[-122.465, 37.779],[-122.454, 37.788],[-122.449, 37.782]],
            [[-122.484, 37.771],[-122.481, 37.761],[-122.494, 37.758],[-122.496, 37.769],
            [-122.484, 37.771]]]

polygon2 = [[(37.803, -122.408), (37.736, -122.491), (37.738, -122.380), (37.787, -122.39)],
            [(37.760, -122.441), (37.772, -122.427), (37.773, -122.404), (37.758, -122.401), (37.745, -122.428)]]

for polys in [polygon, polygon1, polygon2]:
    fig, ax = plt.subplots()
    for poly in polys:
        p = Polygon(poly)
        print(abs(p.area) - abs(signed_area(poly, ax))<1e-14)
        ring = LinearRing(poly)
        assert(ring.is_ccw == is_ccw(poly,ax))
    plt.show()

we get as close as 10e-14 precision to the .area method from shapely.Polygon.
I have tested more complex forms but I think this should do it.


How to with HtmlWebpackPlugin?

my config is the following
const { join } = require('path');
const webpack = require('webpack');
const HtmlWebpackPlugin = require('html-webpack-plugin');

module.exports = {
  context: __dirname,
  entry  : {
    main: './src/app.js'
  },
  output : {
    path         : join(__dirname, 'dist'),
    filename     : '[name].bundle.js',
    chunkFilename: '[id]_chunk.js'
  },
  module : {
    rules: [
      { test: /\.js$/, exclude: /node_modules/, loader: 'babel-loader' },
      { test: /\.css$/, loader: "style-loader!css-loader" },
      { test: /\.(scss|sass)$/, loaders: ['style-loader', 'css-loader', 'sass-loader'] },
      { test: /\.(png|jpg|jpeg|gif)$/, loader: 'url-loader?limit=8192' },
    ]
  },
  plugins: [
    new webpack.ProvidePlugin({m: 'mithril'}),
    new HtmlWebpackPlugin({
      title: 'MyTitle'
    })
  ],
  devServer: {
    historyApiFallback: true
  }
};

my entry point is just rendering a simple component
import m from 'mithril';
import Landing from './landing';

import './styles/theme.scss';

m.render(document.body, m(Landing));

now when Add the static site generator at the end of plugin list always got an "Error: window is not defined" maybe I'm doing something wrong with my config

I'm not sure if this'll help, but I ran into a similar issue and it was simply because I was not specifying the entry point correctly in webpack.config.js - the first argument to the plugin should be the name of your entry point, e.g. your entry point from above is:
entry  : {     main: './src/app.js'   },
therefore in your webpack config you'd want to specify:
new StaticSiteGeneratorPlugin('main', some_array_of_routes, some_other_file),
specifying 'main' is what I had overlooked, hope it helps.

@nealgriffin Now I have a different error

Still have the same error

Understood. I was really interested in this, it took awhile to get the elements correct, but I did end up creating a repository that shows how to use this plugin with Mithril. I believe the root cause of the issues you are having is that mithril is expecting to be run from the browser and this plugin runs it from node's context. I walkthrough the steps I took to get the two to work together in the linked repository.
I'm not sure if it'll solve what you are ultimately trying to do, but my hope is that it is helpful.


Fixed link to swaps in i18n contributor docs

Fixes #564
Changes:
Updated the link to correct url

Looks good to me. Thanks!


HBASE-26208 Supports revoke namespace specified permission

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 37s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 28s
master passed

+1 :green_heart:
spotless
0m 42s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 10s
the patch passed

-0 :warning:
rubocop
0m 11s
The patch generated 8 new + 49 unchanged - 5 fixed = 57 total (was 54)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
spotless
0m 40s
patch has no errors when running spotless:check.

Please add an UT for this new command?

Please add an UT for this new command?

OK，I am ready to add unit tests.

add UT

@Apache9 UT completed

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 30s
master passed

+1 :green_heart:
spotless
0m 44s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 11s
the patch passed

-0 :warning:
rubocop
0m  6s
The patch generated 29 new + 84 unchanged - 5 fixed = 113 total (was 89)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
spotless
0m 41s
patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 13s
The patch does not generate ASF License warnings.

8m 11s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4555/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4555

Optional Tests
dupname asflicense javac spotless rubocop

uname
Linux 4a4a0a568980 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f6861e8d

Default Java
AdoptOpenJDK-1.8.0_282-b08

rubocop
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4555/2/artifact/yetus-general-check/output/diff-patch-rubocop.txt

Max. process+thread count
64 (vs. ulimit of 30000)

modules
C: hbase-shell U: hbase-shell

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4555/2/console

versions
git=2.17.1 maven=3.6.3 rubocop=0.80.0

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 30s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 10s
master passed

+1 :green_heart:
spotless
0m 41s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 11s
the patch passed

-0 :warning:
rubocop
0m  5s
The patch generated 9 new + 84 unchanged - 5 fixed = 93 total (was 89)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
spotless
0m 42s
patch has no errors when running spotless:check.

Are the failed UTs related?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 45s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m  8s
master passed

+1 :green_heart:
spotless
0m 53s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m  0s
the patch passed

-0 :warning:
rubocop
0m  9s
The patch generated 9 new + 84 unchanged - 5 fixed = 93 total (was 89)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
spotless
1m  7s
patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m  9s
The patch does not generate ASF License warnings.

10m  8s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4555/4/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4555

Optional Tests
dupname asflicense javac spotless rubocop

uname
Linux 199e206a4e2c 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 70a2ee1716

Default Java
AdoptOpenJDK-1.8.0_282-b08

rubocop
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4555/4/artifact/yetus-general-check/output/diff-patch-rubocop.txt

Max. process+thread count
64 (vs. ulimit of 30000)

modules
C: hbase-shell U: hbase-shell

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4555/4/console

versions
git=2.17.1 maven=3.6.3 rubocop=0.80.0

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

The UT failures seems related?

The UT is still failing? PTAL?

The UT is still failing? PTAL?

OK, I'll fix it

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/4555 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/4555

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4555/6/console

versions
git=2.25.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  4s
https://github.com/apache/hbase/pull/4555 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/4555

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4555/6/console

versions
git=2.25.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  2s
https://github.com/apache/hbase/pull/4555 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/4555

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4555/6/console

versions
git=2.25.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  2s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 44s
master passed

+1 :green_heart:
spotless
0m 42s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 32s
the patch passed

-0 :warning:
rubocop
0m  7s
The patch generated 36 new + 90 unchanged - 5 fixed = 126 total (was 95)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
spotless
0m 40s
patch has no errors when running spotless:check.

Any updates here?


AKFFTTap on low pass butterworth filter gives 'com.apple.coreaudio.avfaudio', reason: 'error -10868

Hi,
I'm working on a wav file, simulator iPhone 5s iOS 10.3.
(same behavior on real device iPhone 5s iOS 10.2.1)
It works when the player is given to the AKFFTTap,
but it crash when tries to build the AKFFTTap object with the filter.
let file = try! AKAudioFile(readFileName: "fileName.wav",
                                    baseDir: .resources)
player = try! AKAudioPlayer(file: file)
player.looping = true
let filter = AKLowPassButterworthFilter(player)
filter.cutoffFrequency = cutoffFrequency // 150 Hz
AudioKit.output = filter
AudioKit.start()
player.play()
let fft = AKFFTTap(filter) //  CRASH
let fft = AKFFTTap(player) //  OK

Maybe I've missed something as I'm new to AudioKit?
Thx for your good work!

If you want to FFT the filter, it may help to send it through a mixer first.  For some reason taps don't like certain kinds of nodes.  So just do
let mixer = AKMixer(filter)
...
let fft = AKFFTTap(mixer) // Should be OK

The mixer won't be doing anything to the sound, and it is low energy, so it should be a fairly easy modification. Mind you, make sure these variables are local, so in a real project you'll probably want them to be instance variables retained by a class.

Thx for the quick response!
It doesn't crash anymore.


Consider using REST API instead of float CLI

If you have a REST API for MemoryMachine Cloud, it would be nice to use that instead of the float CLI in your custom executor. That way, users don't even have to install the float command to get started (even though it is really easy and they'll probably do it anyway because it's useful). Also, a REST API should simplify your code because you won't have to construct the bash commands.
In this case you would only need to extend Executor rather than AbstractGridExecutor. The K8sExecutor in Nextflow is a good example of an executor that delegates to a REST API.
In any case, this issue is not urgent, and the float CLI should suffice in the meantime.

Thanks for the suggestion.  Considering the effort required, we may not be able to do that in a short term.

Out of curiosity, do you have a REST API that you could use?

Yes, the command line actually talks to the REST API.  But we don't have a lib for java platform.

If it's a REST API then you could just use an HTTP client, like K8sClient in Nextflow. But I understand if that would still be a lot of work to construct the HTTP calls properly.


Place Optimizely::Project in optimizely/project.rb

Summary

Moves Optimizely::Project into the file corresponding to its name.
Removes requires that aren't necessary (don't provide referenced constants within Project)
Adds missing require
Keeps "extraneous" requires in lib/optimizely.rb to make this change essentially a noop from behavior perspective.

Classes and modules should be defined in files that correspond to their name. This ensures that developers will know the right place to look because things will be in their conventional location.

@jasonkarns This looks good, would you mind doing a rebase/remerge if it's not too much trouble and we can approve (assuming our tests pass)

Travis Link: https://travis-ci.com/optimizely/fullstack-sdk-compatibility-suite/builds/139270452

too much drift.


Replicate state

Replicate state across regions.

:tada: This issue has been resolved in version 0.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Color option

I searched the Obsidian forum and Discord and I cannot find any good solution how to change color of individual folder except going into CSS and adding fixed rules for folder name.
Request feature
Expect adding of the icon for folder and file add option to:

chose any color that will be applied to the icon (for individual folder!)
chose if the color will be applied only to icon or also to folder name (color of text)
chose for example the children files to just have color inhirited not the icon, but also add inhirit all (icon + color)

All these features work with individual folders, meaning I can have different folders with differnet color.
Benefit
Your plugin will become one solution for all visual tweaking of the folder and files. And it will be the only one, as there isn't anything like that.

@dxcore35 There is the File Color plugin that does that very well.
This one also colors the icon from this plugin here.
I think it might be unnecessary or even tricky to duplicate that functionality here.

This is now available in v2.9.0.

Amazing!!


[Enhancement] Optimize the message body of the Rust SDK's returned reply message

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request
Optimize the message body of the Rust SDK's returned reply message.
link #4631
Describe the solution you'd like
Optimize the message body of the Rust SDK's returned reply message
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

Hi, I am willing to work on this issue, can u please provide more details? I looked into request_reply_inner method inside grpc_client.rs and AFAIK what i need to do is

Check if it's a subscription reply message
Do additional logging for subscription reply
Add attributes to indicate subscription response

Hi, I am willing to work on this issue, can u please provide more details? I looked into request_reply_inner method inside grpc_client.rs and AFAIK what i need to do is

Check if it's a subscription reply message
Do additional logging for subscription reply
Add attributes to indicate subscription response

@kyooosukedn Sure.
Delete the SubscriptionReply struct, which has already been deleted in the Java SDK. For more details, please refer to PR#4632. For the specific implementation logic, please refer to the Java SDK SubStreamHandler#createReceiver method.
The implementation in the Rust SDK is in the subscribe method in grpc_consumer.rs. Here you can find that only EventMeshMessage is supported, and other modifications will be made later. Therefore, you can simply follow the existing logic.


Unpin leak_tracker.

FYI you'll probably get a conflict when #132350 merges

It's ok. I will refresh it. Thanks.
I want to wait for  https://github.com/dart-lang/leak_tracker/pull/121 anyway.
Converted to draft for now.


Branch synchronization: prerelease/2.5.0_stabilization --> mrtk_development

This is a pull request initiated by an automated process to keep prerelease/2.5.0_stabilization and mrtk_development in sync

@RogPodge Looks like CI might be broken from #8557 :/


Merge keys appear not to be supported

I found this in my own project when attempting to use merge keys ("<<:", https://yaml.org/type/merge.html). Specifically, the key gets parsed literally as a scalar with value "<<". To see this, try the following:
tests/example-deconstructor < examples/anchors.yaml

You will see a node like this in the output:
  - type: SCALAR
    value: "<<"
    implicit: {plain: true, non-plain: false}
    style: plain

Apparently this "Working Draft" did not make it into "Final Draft" YAML 1.1 specification. There are no mentions of "<<" or "merge".

Yes, I noticed that. It seems to be supported in many other tools (e.g. pyyaml and most online YAML validators), thus more of a de-facto than official standard.
If we decide not to support this for now, I think it would be good to at least:

Change or remove the example in the repository, which suggests that it is supported.
Throw an error when someone uses "<<" as a key value, as this YAML won't work as intended (it will appear to succeed but not parse correctly). This would break some users, but such users were already broken.

For a parser, the scalar << is just a scalar node like foo or bar.
Only the composer/constructor resolves scalars and creates booleans, numbers, null, and, if implemented/enabled, merge keys.
At the parsing level nothing special should be done with a scalar <<.
Btw, as you mentioned pyyaml - yes, pyyaml supports merge keys, and even when it's used with libyaml as the backend.

I see. So to clarify, it's intended that C code that wants to use merge keys needs to implement that itself? This seems pretty non-trivial to implement on top of libyaml. Do you have an example of how this work? It seems like a lot of code. Is there a higher-level C library on top of libyaml that I could use to implement without writing a lot of custom code?

A higher level library like pyyaml processes merge keys when it already has constructed a partial data structure.
With a merge key you usually merge an existing alias. This is not trivial by itself if you look at the spec for merge keys, but the given example covers all use cases I believe.
So if you want to add this on top of libyaml, you need to have the previous anchors stored in a mapping-like data structure, and additionally the currently processed mapping has to be constructed already before you can add the data from the merge keys.
You could try out https://github.com/pantoniou/libfyaml, its documentation says it supports merge keys on request.

I don't think it will be too much code... If you are interested I can probably provide an example.

Yes, an example would be helpful.
I'm a little confused about something: anchors seem to be working just fine. For instance, the following parses without issue:
a: &node_a
  a
b: *node_a

b will get the same value as a after parsing with libyaml, with no extra work necessary. I had expected this is how merge keys with <<: would be treated, as they're related to anchors. Are anchors part of parsing but merge keys are not part of parsing?
I guess I'm trying to understand if this is a question of "we haven't implemented this but would like to", or is it "we plan never to implement this, by design."

An alias is part of the syntax. Aliases will be parsed as an alias_event. The value, however, can not be resolved until the data is constructed.
How do you process the YAML? If you just use the parsing event API from libyaml, then you shouldn't get a scalar event, but an alias event.
Or are you using the document API? When constructing the document, an alias will be loaded as a pointer, so you would see the same value.
It's hard to help if we don't know what you are actually doing.

I'm using document parsing. This is the code if you want to look: https://github.com/XevoInc/yobd/blob/master/src/parser.c

@martingkelly  ok, like I said, resolving of booleans and other types does not happen in the document loader, so also merge keys are simple salar nodes like true, false, 23 or something. I also don't see such code in your link.
I'm not saying it would be impossible to implement that in the document loader. But post processing would probably a better idea than putting it directly into the existing loader code.

In my link, I parse directly into my own data structures; I don't do intermediate processing or translation of the YAML. e.g. in https://github.com/XevoInc/yobd/blob/master/src/parser.c#L320, we convert a scalar to a number and stick it into a data structure.
It sounds like this issue can be resolved as "by design", as libyaml is intended to do only parsing and not composing/construction, and alias resolution is considered part of composing/construction. Is this correct?


es6 transpiller and es6 modules

@mehranhatami  es6 are the future. Why not convert definejs to use es6 modules? I have set up a es6 workflow  so could use that as the framework.
I'm using jasmine to run the test cases.
Travis are green and working example found here:
https://github.com/Kflash/ES6-workflow.

@mehranhatami I close this. The developer  seems  dead


Add gapic resource name support

This PR:

Removes resource_name_format.proto
Adds GAPIC configuration for resource names and resource name treatment

This PR does not:

Support Oneofs
Support "invalid" resource name types

Current coverage is 80.41% (diff: 80.00%)

Merging #677 into master will increase coverage by 0.01%

@@             master       #677   diff @@
==========================================
  Files           277        277
  Lines         10547      10610    +63
  Methods           0          0
  Messages          0          0
  Branches       1397       1411    +14
==========================================
+ Hits           8480       8532    +52
  Misses         1705       1705
- Partials        362        373    +11

Powered by Codecov. Last update 579df0b...a110fe5

Updated to test parameter_resource_name_treatment setting, and increase code coverage.
A question regarding generation of samples - if the method-level resource name treatment setting is set to NONE, VALIDATE, or unset, but a resource name has been specified and generated for a field, what behaviour would we like:

Don't use the resource name in the sample (current behaviour)
Use the resource name, and pass it to the flattened method via the toString() method

I like the idea of (2), in order to encourage usage of the resource name types.

LGTM.
I feel like things are getting very complicated. That is a discussion for another day though, I suppose.


Update to 3.5.17 and attempt cross-compile for Linux aarch64

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found some lint.
Here's what I've got...
For recipe/meta.yaml:

❌ In conda-forge.yml: $.provider = {'linux_aarch64': 'linux_64'}.

{'linux_aarch64': 'linux_64'} is not valid under any of the given schemas

Schema
{
  "anyOf": [
    {
      "$ref": "#/$defs/provider"
    },
    {
      "type": "null"
    }
  ]
}

For recipe/meta.yaml:

ℹ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12429679805. Examine the logs at this URL for more detail.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12429900463. Examine the logs at this URL for more detail.


--splitlog help

When we pass this --splitlog argument.. It supposed to split the HTML log file in pieces but I am just getting javascript files with some index attached to it. Isn't it supposed to break HTML file and it should give me separate HTML files? If there is any other way to do it then please tell.

Works as designed and documented:
http://robotframework.org/robotframework/latest/RobotFrameworkUserGuide.html#splitting-logs


Export iconNames array and names type

I updated the build script to include an iconNames array. I really want to do something like this:
import { iconNames } from '@heroicons/react/20/solid';

export const Icon = ({ name }) => {
  const ActualIcon = React.lazy(() => import(`@heroicons/react/20/solid/${name}.js`));

  return (
    <Suspense fallback={null}>
      <ActualIcon />
    </Suspense>
  );
};

Icon.propTypes = {
    name: PropTypes.oneOf(iconNames)
};

I also made iconNamesType so someone could do the same thing with TypeScript.
import { iconNamesType } from '@heroicons/react/20/solid';

export const Icon = ({ name: iconNamesType }) => {
  const ActualIcon = React.lazy(() => import(`@heroicons/react/20/solid/${name}.js`));

  return (
    <Suspense fallback={null}>
      <ActualIcon />
    </Suspense>
  );
};

I'm pretty sure the CJS type is the same format as ESM, but I could be wrong there.
Solves https://github.com/tailwindlabs/heroicons/discussions/813. In my case (not using TS), if I did something like the suggested code from that discussion, the entire library would always be bundled.

Hey!
Thanks for this contribution, but I think for now we're just going to leave this how it is. While the approach you're suggesting would work with code splitting, you'd end up getting a unique request for every icon on the page, and without code splitting you'd end up with all the icons in your bundle.
Also, while not totally the same thing, you can get all the names doing something like this:
import * as Heroicons from '@heroicons/react/20/solid'
type Icons = keyof typeof Heroicons

And worst case you could just hard code a set of icon names into your project.
Hope that makes sense 👍

👍 Fair enough!
I'm not able to use the example you posted though, since I'm not using TS. And I'm not planning on hardcoding icon names, haha.


Improve multitenancy support

This PR improves multitenancy support within Inbox delivery. The introduced change ensures each batch of the delivery is running within a tenant-aware operation.

@armiol PTAL.


Will not work in Bash command when information piped in

To recreate, create the following file prompt-test.js, as a very simple test:
#!/usr/bin/env node
const prompt = require("prompt-sync")({"sigint": true})

for (var i = 0; i < 5; i++) {
  console.log("Before prompt line")
  var result = prompt("This is a test -- enter something >");
  console.log("Result of prompt", result)
}

If run as ./prompt-test.js, it will ask for an input 5 times and print that output as you might expect.
However if you trying running echo hello world | ./prompt-test.js, the prompt itself is caught in a loop and repeats the prompt line and all previous inputs continually.
This might be happening because of my lack of understanding about Bash processes, sub-processes, or piping. Any help appreciated.

This might have something to do with the reliance on process.stdin ... code suggests that's being used when fd is win32. From the little I know, if a TTY readstream can be used instead, for example, it means piped input and keyboard input can be accepted and should work.


Fix bug: getTouchingPropertyName does not return undefined, but it may return the source file node

Fixes #8553.

Hi @andy-ms, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Andy Hanson). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

there are other references in this file that expects getTouchingPropertyName to return undefined or a token. can your update these as well?

:+1:


Improve subversion recommendation

Thanks!
The arch wiki page is very inconsistent with this unfortunately, I tried to fix it as much as possible, but I already suspected that I missed a few.


⚠ St. Lawrence Global Observatory CKAN has degraded performance

In f11912f, St. Lawrence Global Observatory CKAN (https://catalogue.ogsl.ca/en/) experienced degraded performance:

HTTP code: 200
Response time: 7691 ms

Resolved: St. Lawrence Global Observatory CKAN performance has improved in 4fdbe4e after 11 minutes.


More aggressive wrapping

Description of the Change
Wrapping now includes wrap on dash and slash. This is a response to issue #16904, in which the user wanted more "aggressive" wrapping. To implement this change, the isWrapBoundary variable was given character checks for spaces, slashes, and dashes as wrap boundaries.

Alternate Designs
The proposed version was selected because it produced the simplest fix to this problem.

Why Should This Be In Core?
This should be in the core because it has to do with editor functionality.

Benefits
A much more intuitive soft-wrap functionality that doesn't breakup words in the middle if it doesn't have to.

Possible Drawbacks
None

Verification Process
The new soft-wrap was tested with tests such as these:

Applicable Issues

Any idea on why these are failing given the addition of only one line of code?

The failings tests deal explicitly with soft wrapping. I'm guessing you'll want to change the tests since you're changing the soft-wrapping behavior?
Also, in prose, the convention is that dashes should stay at the end of the line. Can you adjust this code so that the dash is not moved to the following line?

@Rezayovan Do you have any questions around this?

Closing this out since we haven't heard back. Let us know if you're interested in picking this back up and we can reopen it.

https://github.com/atom/atom/blob/3c9620dfa54d1cb8e1a5f356d9a5c095b95788e9/src/text-utils.js#L105-L106
should be replaced by
const isWrapBoundary = (previousCharacter, character) =>
  isWordStart(previousCharacter, character) || isCJKCharacter(character) ||
  previousCharacter === '-' || character === '/' || character === ' '

I would have preferred to create a PR myself, I can’t seem to get a dev environment for Atom.

I finally managed to set up a dev environment, and I’m seeing an issue (which was already here): the cursor can’t be after the last character, because it’s the same position than the beginning of the following line, and so “Move to end of line” puts the cursor before the last character of the line visually.


Merge 6.0 back into master

Summary

Ticket Link

Related Pull Requests

Has server changes (please link here)
Has mobile changes (please link here)

Screenshots

Release Note

/release-note-none


[BUG] Extractor started generating the All Api subscription folder In artifacts

Release version
4.1.0
Describe the bug
Previously there is no subscription folder generated in the API artifacts. Not it seems it is generating the subscription folder with All API subscription.
Expected behavior
It should not generate subscription folder with All Api subscription information.
Actual behavior
Subscriptions auto created when the product created in during the deployment. Is it really required to be generate the subscription folder? If yes, what is the linkage of the same.
Reproduction Steps
Run the extractor pipeline for any api and you can see the subscription folder generated in the api artifacts.

@Gabriel123N,
Thanks for the consideration. Actually if I see from the user perspective they are only cared about apis and details belongs to the same api. If something new come up there are new .gitignores needs to done and that is bit of extra work so suggestions is to keep the artificats only related to apis when specific filter given.


Verify FID signature external view

Motivation
This PR adds an external view to the IdRegistry contract such that a fid's signature can be verified. Having a single way to verify a fid signature will make the developer experience easier since the registry has 1271 signature support.
https://t.me/farcasterdevchat/6142 (original telegram thread on this topic)
Change Summary
This PR:

Adds an external view to IdRegistry that verifies a fid signature by checking if the signer owns the fid, and if the signature is correct.
Adds tests to make sure the view works as exptected.

Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review

[x] The PR title adheres to the conventional commits standard
[ ] The PR has been tagged with change type label(s) (i.e. documentation, feature, bugfix, or chore)
[x] The PR's changes adhere to all the requirements in the contribution guidelines
[x] This PR does not require changes to the Farcaster protocol

Additional Context
If this is a relatively large or complex change, provide more details here that will help reviewers.

PR-Codex overview
Detailed summary

Added a new internal function _signMsg in IdRegistryTestSuite.sol to sign a message using a private key and return the signature.
Added a new internal function _verifySigPlain in Signatures.sol to verify a plain signature using a digest, signer address, and signature.
Added a new external view function verifyFidSignature in IdRegistry.sol to verify a signature from a fid owner using the signer address, fid, digest, and signature.
Added new test functions testFuzzVerifyFidSignature and testFuzzCannotVerifyFidSignature in IdRegistry.t.sol to test the verifyFidSignature function.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

I can't change the labels after creating the PR :(

Left some comments on things that should be changed - core implementation is fine, but we can tighten up the tests a bit.

Resolved comments, and cleaned up tests.

Made most changes requested.
Pretty sure adding fid->owner resolution is out of scope for this PR.