feat (new-widget): country selector

Should we change the country name to be bold Black instead of Green? Right now it looks like a link to me

@julianpoma Agree with Mark. Without thinking I clicked on the country and nothing happened.
Also, can you center this 3 elements? in mobile it's more obvious:


[ios] Upgrade folly to 2<IP_ADDRESS> and boost to 1.76.0

Summary

[ios] upgrade folly to 2<IP_ADDRESS> which aligned to android.
folly compile setting from c++14 -> c++17: this folly requires c++17 for std::unordered_map::insert_or_assign.
boost 1.63.0 -> 1.76.0:  the old boost does not support c++17.
deprecating react-native-community/boost-for-react-native: by cocoapods installer, we could download the official target.

Changelog
[iOS] [Changed] - Upgrade folly to 2<IP_ADDRESS> and boost to 1.76.0
Test Plan
CI passed

looks like it failed internally. please let me know anything i could help.

Yup. I have almost everything working, but we have some internal codegen builds that are failing. I'll ping the owner again today to help take a look.

@yungsters awesome! thanks for your update.


Added github key property

A quick patch to implement the automatic configuration and deployment of a Github secret key, in the event Github is needed for in Hookshot.

!bump minor

Jenkins has merged this PR into master and has automatically performed a minor-level version bump to v1.1.0.  Have a nice day!

!bump minor

You forgot to add ~ at the end FWIW.


Use proper jQuery objects instead of strings for better security (second try)

A rebase of https://github.com/vedmack/yadcf/pull/630

@m417z please go over the diff between the versions, it looks like you omitted onchange / onkeyup / etc'

They've not been omitted, they've been moved to be placed as properties. For example:
https://i.imgur.com/PMLFJuz.png

Very impressive! Thanks a lot @m417z


Check docker return code before running commands

Resolves #3673
What is being addressed

Add check for docker return code before changing permissions
[x] Update CHANGELOG.md if needed

/test-force-approve
Passed: https://github.com/microsoft/AzureTRE/actions/runs/5867301767


feat: Eliminated the use of temporary keys for tests with kubectl in privatelink-access pattern

Description
The privatelink-access pattern documentation had steps which involved creating temporary set of AWS credentials for the use of testing PrivateLink connectivity. This approach is manual and copy pasting credentials, however short-lived isn't ideal.
With the new set of changes introduced in this PR:

The EKS cluster would initially come up with its API endpoint as public
This would ensure that the Client EC2 instance's attached role is added as system:masters in aws-auth configmap
Which would allow the Client EC2 to run kubectl commands as an Admin against the EKS cluster in the Client VPC without the need for any credentials
The documentation is also corrected to:

Include the right steps to deploy the pattern
Include steps to take the cluster API endpoint private after Terraform finishes
Include cleaner steps to test the pattern
Add additional step of take the cluster API endpoint public before tearing down the resources

How was this change tested?

[ ] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[ ] Yes, I ran pre-commit run -a with this PR

Got additional unplanned files in the commit, closing this for now.


Added support for Dual and Quad SPI modes and updating FT81x driver to use it

This resolves issue #201 by updating the base SPI display driver code to support Dual and Quad SPI modes as well as updating the FT81x drivers to use those modes. New kconfig options where added to control SPI modes, pins, etc... These changes should have minimal to no impact on the other existing drivers as using DIO/QIO modes are opt-in and the code path for not using it remains the same.
Note that I also normalized to spaces in kconfig as using mixed tabs can cause ESP-IDF to freak out in some cases. This might make the kconfig look like it has changed more than it really has.
This update doubles and quadruples the refresh rate for the FT81x when using Dual or Quad SPI modes.

Hi,
Sorry for the late reply. I won't be able to review this until next weekend, hope you can wait for it.

I did a careful review and fixed up the other drivers but since they are not using the new DIO/QIO features, they shouldn't even notice. The extra parameter (dummy bits) would only change behavior if they also passed in the new flag to activate them.


Big refactor

There are a lot of changes here, but I think it improves a lot of things.
Thoughts?

Thanks so much! I took a look at everything. Looks very good! I'll go ahead and merge it.


Explicitly encourage and teach Python 3

https://pythonclock.org/ says Python 2.7 will retire in 1 Month, 10 Days, 4 Hours, 56 Minutes and 49 Seconds
More to the point: this is one of the best tutorials on getting a Python development environment set up I've ever seen, but I want people to get going with Python 3 so they can use my Datasette project!
I imagine updating it to Python 3 is not an insignificant amount of work, due to the need to re-record the installation videos. But I'm optimistically filing a bug report anyway!

The plan is rewrite the class with Python 3 and Altair for NICAR 2020!

Awesome!


Dependabot requires private repo auth when it is unnecessary

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
pip
Package manager version
poetry 1.7.x
Language version
python 3.11
Manifest location and content before the Dependabot update
No response
dependabot.yml content
No response
Updated dependency
No response
What you expected to see, versus what you actually saw
When using poetry, dependabot cannot open PRs for vulnerable packages identified in poetry.lock because it is trying to install private packages that are not needed to update the poetry.lock file.
For example, in GitHub, an error like this will be seen on the Dependabot vulnerability page.
Creating virtualenv my-app-3Y8T20qu-py3.11 in /home/dependabot/.cache/pypoetry/virtualenvs
Updating dependencies
Resolving dependencies...
Source (my_private_pypi): Authorization error accessing <redacted>

Package [my-private-dependency] not found.

However, locally I can do all that is needed to update the dependencies using the following pattern:
poetry update <vulnearble-dep> --lock
Here, the --lock will constrain the work to only updating the lock file, not needing to install all the other dependencies.
Native package manager behavior
% poetry update <vulnearble-dep> --lock
Updating dependencies
Resolving dependencies... (1.9s)

Writing lock file

Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

To clarify some thing here, this is probably due to Dependabot trying to install deps before doing the update. As I see it currently, dependabot is using --lock on the actual update command. But there is no need to actually install packages beforehand, or create a venv at all to do the update. But Dependabot is obviously using a venv.
e.g.
% pip freeze
% poetry update sqlparse --lock --no-interaction
Updating dependencies
Resolving dependencies... (1.9s)

Writing lock file

Hi @mbrancato I'll try to get some eyes on this. Do you have an example that you could share that reproduces this, or the job ID of a job that shows this behavior? The job ID should be printed out in the job logs.

related:

https://github.com/dependabot/dependabot-core/issues/8554
https://github.com/dependabot/dependabot-core/issues/8020
https://github.com/dependabot/dependabot-core/issues/7940
https://github.com/dependabot/dependabot-core/issues/7440

Hi @Nishnha - here is a job ID: 827058885
I was able to do the same work that Dependabot should try to do without any authorization by simply relying on updating the public depndencies.


Tool 'builtin_brush.Draw' not found for space 'VIEW_3D'

Daily build 3.3 Alpha with latest master.
Every time opening Selection pie menu to select Object this error appears:
Tool 'builtin_brush.Draw' not found for space 'VIEW_3D'

The issue appears to be related to the grease pencil in the selection pie file. Unfortunately, I do not code, but I did notice that the grease pencil section of that file was edited 4 months ago. Someone with knowledge of python may be able to fix that.

@skarmanami I found that if I created a new modeling tab in the workspaces and deleting the original, that the issue does not continue over to the new copy. I then renamed and moved the new tab. I then saved the file as the startup default under the file menu


Add support for branch rulesets

Repo-policy-compliance only supports checking classic branch protection, but users can also define branch rulesets: https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/managing-rulesets/about-rulesets. The application should support the latter.

@cbartz is this issue still relevant?

Hi, yes it is, https://warthogs.atlassian.net/browse/ISD-2134 is the internal ticket.


ssl authentication error

Describe the bug
ssl authentication error
To Reproduce
Steps to reproduce the behavior:
docker-compose.yml
...
      environment:
      - ARGILLA_ELASTICSEARCH_CA_PATH=/ssl/server.crt
      - ARGILLA_ELASTICSEARCH=https://elasticsearch:9200/
      - ARGILLA_ELASTICSEARCH_SSL_VERIFY=True
...

Screenshots

Environment (please complete the following information):

OS [e.g. iOS]:  linux
Browser [e.g. chrome, safari]: chrome
Argilla Version [e.g. 1.0.0]: 1.2.1
ElasticSearch Version [e.g. 7.10.2]: 7.7.0
Docker Image (optional) [e.g. argilla:v1.0.0]: argilla:v1.2.1

Additional context
Current Workaround
->
modify "argilla/server/daos/backend/client_adapters/factory.py" file
factory.py
from argilla.server.settings import settings

  def _fetch_cluster_version_info(cls, hosts: str) -> Tuple[str, str]:
      try:
          if settings.elasticsearch_ssl_verify:
              response = httpx.get(hosts, verify=settings.elasticsearch_ca_path)
          else:
              response = httpx.get(hosts)

          data = response.json()

          version_info = data["version"]
          version: str = version_info["number"]
          distribution: str = version_info.get("distribution", "elasticsearch")

          return version, distribution
      except Exception as error:
          raise GenericSearchError(error)

@Joonyeong97 thanks for reporting this 🦾


Acetaminophen issues on 2023may18 NodeNorm and NameRes

We have three different acetaminophen issues:

[x] Searching for "acetaminophen" on NameRes (http://name-resolution-sri-dev.apps.renci.org/lookup?string=acetaminophen&offset=0&limit=10) returns many combinations of acetaminophen with other drugs instead of just acetaminophen. The closest actual match is "acetaminophen [EPC]" (UMLS:C2917659), which isn't the correct UMLS identifier: UMLS:C0000970 ("acetaminophen").
[x] Although 2023may18 does include UMLS:C0000970 ("acetaminophen") in NodeNorm (https://nodenormalization-dev.apps.renci.org/1.3/get_normalized_nodes?curie=UMLS%3AC0000970&conflate=true), this CURIE isn't available in NameRes at http://name-resolution-sri-dev.apps.renci.org/reverse_lookup.
[x] Acetaminophen is typed as a SmallMolecule instead of a Drug.

We still have a bunch of acetaminophen mixtures we need to deal with, but "Acetaminophen" is now the top result (https://name-resolution-sri.renci.org/lookup?string=acetaminophen&offset=0&limit=10). These can be removed by filtering out RXCUI identifiers (https://name-resolution-sri.renci.org/lookup?string=acetaminophen&offset=0&limit=10&exclude_prefixes=RXCUI).

UMLS:C0000970 is now being normalized to CHEBI:46195 "Acetaminophen", even with drug-chemical conflation turned off (https://nodenormalization-sri.renci.org/1.5/get_normalized_nodes?curie=UMLS%3AC0000970&conflate=true&drug_chemical_conflate=false&description=false&individual_types=false).

We have decided that biolink:Drug should only be used for individual drug formulations, so biolink:SmallMolecule is the correct type for acetaminophen (https://github.com/TranslatorSRI/Babel/issues/348).

Closing.


dynamically grab latest GKE cluster version for Jenkins

Currently, GKE's default master Kubernetes version is 1.5.6, but the catalog needs >=1.6.0. However, the patch version keeps changing, and there's not way to specify "at least 1.6" when creating a cluster. Until the default is >=1.6.0, manually grab the latest GKE master version from the list of valid versions, and use that for Jenkins.

@MHBauer I'd be completely shocked if they removed support for 1.6+ at this point, but I'll add in an extra check just to be 100% confident.

When can expect to see a v1.7 release pop up? I think that might break the revised script.

Yup, that'll break it, but at that point the workaround shouldn't be necessary. Didn't want to put in anything super heavy duty.

Not for a while. Juneish iirc
On Thu, Apr 27, 2017 at 5:51 PM Michael Kibbe<EMAIL_ADDRESS>wrote:

Yup, that'll break it, but at that point the workaround shouldn't be
necessary. Didn't want to put in anything super heavy duty.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/service-catalog/pull/765#issuecomment-297849198,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWXmDKAldu1Sfelciukh2cbECwpv5zdks5r0Q3KgaJpZM4NKnTl
.


Public IP address in type origin instead private IP

Hi,
I have implemented ratelimit in spring cloud zuul but when i apply type as origin it takes private IP address of the client and gets save in the rate table. Is there any way to get the public IP address of the client ?
Thanks ! Quick response appreciated.

Hi @CasperOm, is your application running behind a proxy?

Hi @CasperOm, is your application running behind a proxy?

Hi @marcosbarbero No its not running behind any proxy. Only below 4 annotations are used in Zuul.
@EnableResourceServer
@EnableZuulProxy
@EnableEurekaClient
@SpringBootApplication

Hi @marcosbarbero No its not running behind any proxy. Only below 4 annotations are used in Zuul.
@EnableResourceServer
@EnableZuulProxy
@EnableEurekaClient
@SpringBootApplication

How's your deployment setup? Where's the application running? How is the request being performed? Is the client in a different network? Is there a load-balancer or any other thing between the client and the API Gateway?

How's your deployment setup? Where's the application running? How is the request being performed? Is the client in a different network? Is there a load-balancer or any other thing between the client and the API Gateway?

Zuul is been running on my local with which i'm expecting the IP to come in is <IP_ADDRESS> but I'm not receiving this ip instead it coming as 0:0:0:0:0:0:0:1.
Both client and service are on same network. I'm hitting the zuul from the postman. There is no externally configured load-balancer between the client and the API Gateway.
And yes i'm also using eureka.

Zuul is been running on my local with which i'm expecting the IP to come in is <IP_ADDRESS> but I'm not receiving this ip instead it coming as 0:0:0:0:0:0:0:1.
Both client and service are on same network. I'm hitting the zuul from the postman. There is no externally configured load-balancer between the client and the API Gateway.
And yes i'm also using eureka.

So, you are using Eureka to discover the gateway by name and perform the request?

So, you are using Eureka to discover the gateway by name and perform the request?

Yes you are correct, but the ratelimiter is implemented in Zuul itself. When i tried debugging it i'm not able to get public IP.

Yes you are correct, but the ratelimiter is implemented in Zuul itself. When i tried debugging it i'm not able to get public IP.

This behavior is explained at Spring Cloud's documentation. I believe this may solve your issue.

This behavior is explained at Spring Cloud's documentation. I believe this may solve your issue.

Thanks for the suggestion, but it didnt worked. I used inetutils.preferredNetworks: 127.0 and also inetutils.useOnlySiteLocalInterfaces: true but still the same issue.

Thanks for the suggestion, but it didnt worked. I used inetutils.preferredNetworks: 127.0 and also inetutils.useOnlySiteLocalInterfaces: true but still the same issue.

If you can share a sample repo reproducing the error, I can take a look.

If you can share a sample repo reproducing the error, I can take a look.

Sure will share in some time.

Sure will share in some time.

@marcosbarbero Sorry for the delay in sharing the sample repo.
https://github.com/CasperOm/zuul-rate-limiter.git This is the repo which consists sample code for the issue mentioned above.
Run as spring boot application and i have used H2 database where you will be able to see the origin as 0:0:0:0:0:0:0:1 instead of <IP_ADDRESS>. Please have a look in this and if we can find a solution for this it will be very helpful for me.
Thanks, If you are not able to access the repo let me know.

@marcosbarbero Sorry for the delay in sharing the sample repo.
https://github.com/CasperOm/zuul-rate-limiter.git This is the repo which consists sample code for the issue mentioned above.
Run as spring boot application and i have used H2 database where you will be able to see the origin as 0:0:0:0:0:0:0:1 instead of <IP_ADDRESS>. Please have a look in this and if we can find a solution for this it will be very helpful for me.
Thanks, If you are not able to access the repo let me know.

@CasperOm maybe the repo is private? I can't see it

@CasperOm maybe the repo is private? I can't see it

@marcosbarbero Please check now if you can see it?

@marcosbarbero Please check now if you can see it?

Now I can see it, will check on it later today.

Now I can see it, will check on it later today.

Sure Thanks

Sure Thanks

@CasperOm unfortunately, there's something wrong with the sample project.
2021-01-19 10:47:16.699  WARN 8190 --- [           main] ationConfigEmbeddedWebApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'entityManagerFactory' defined in class path resource [org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaAutoConfiguration.class]: Invocation of init method failed; nested exception is java.lang.NoClassDefFoundError: javax/xml/bind/JAXBException
2021-01-19 10:47:16.700  INFO 8190 --- [           main] o.apache.catalina.core.StandardService   : Stopping service [Tomcat]
2021-01-19 10:47:16.720  INFO 8190 --- [           main] utoConfigurationReportLoggingInitializer :

Error starting ApplicationContext. To display the auto-configuration report re-run your application with 'debug' enabled.
2021-01-19 10:47:16.729 ERROR 8190 --- [           main] o.s.boot.SpringApplication               : Application startup failed

org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'entityManagerFactory' defined in class path resource [org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaAutoConfiguration.class]: Invocation of init method failed; nested exception is java.lang.NoClassDefFoundError: javax/xml/bind/JAXBException
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1628) ~[spring-beans-4.3.14.RELEASE.jar!/:4.3.14.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:555) ~[spring-beans-4.3.14.RELEASE.jar!/:4.3.14.RELEASE]
	```

@CasperOm unfortunately, there's something wrong with the sample project.
2021-01-19 10:47:16.699  WARN 8190 --- [           main] ationConfigEmbeddedWebApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'entityManagerFactory' defined in class path resource [org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaAutoConfiguration.class]: Invocation of init method failed; nested exception is java.lang.NoClassDefFoundError: javax/xml/bind/JAXBException
2021-01-19 10:47:16.700  INFO 8190 --- [           main] o.apache.catalina.core.StandardService   : Stopping service [Tomcat]
2021-01-19 10:47:16.720  INFO 8190 --- [           main] utoConfigurationReportLoggingInitializer :

Error starting ApplicationContext. To display the auto-configuration report re-run your application with 'debug' enabled.
2021-01-19 10:47:16.729 ERROR 8190 --- [           main] o.s.boot.SpringApplication               : Application startup failed

org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'entityManagerFactory' defined in class path resource [org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaAutoConfiguration.class]: Invocation of init method failed; nested exception is java.lang.NoClassDefFoundError: javax/xml/bind/JAXBException
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1628) ~[spring-beans-4.3.14.RELEASE.jar!/:4.3.14.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:555) ~[spring-beans-4.3.14.RELEASE.jar!/:4.3.14.RELEASE]
	```

@marcosbarbero I tried again in my STS workspace, it is started successfully on port 8762. Could you please try doing mvn clean install and check.

@marcosbarbero I tried again in my STS workspace, it is started successfully on port 8762. Could you please try doing mvn clean install and check.

I tried in every possible way, unfortunately, it simply just not run

I tried in every possible way, unfortunately, it simply just not run

@marcosbarbero Okay let me try implementing it other than JPA and share the updated code. Just wanted to confirm that you are getting JPA relation error only right? no other errors?

@marcosbarbero Okay let me try implementing it other than JPA and share the updated code. Just wanted to confirm that you are getting JPA relation error only right? no other errors?

@marcosbarbero Could you pls try spring-zuul-rate-limiting project from this https://github.com/CasperOm/zuul-rate-limiter.git repo and try if you run this project in your STS.

@marcosbarbero Could you pls try spring-zuul-rate-limiting project from this https://github.com/CasperOm/zuul-rate-limiter.git repo and try if you run this project in your STS.

@marcosbarbero Pls use this project. It doesn't have any H2 database. You can see ip which is assigned in response headers, Project name is spring-zuul-rate-limiting from https://github.com/CasperOm/zuul-rate-limiter.git
Reference : https://www.baeldung.com/spring-cloud-zuul-rate-limit

@marcosbarbero Pls use this project. It doesn't have any H2 database. You can see ip which is assigned in response headers, Project name is spring-zuul-rate-limiting from https://github.com/CasperOm/zuul-rate-limiter.git
Reference : https://www.baeldung.com/spring-cloud-zuul-rate-limit

Unfortunately, this project doesn't run either.
spring-zuul-rate-limiting git:(main) ✗ mvn clean package
[INFO] Scanning for projects...
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[FATAL] Non-resolvable parent POM for com.baeldung.spring.cloud:spring-zuul-rate-limiting:0.0.1-SNAPSHOT: Could not find artifact com.baeldung.spring.cloud:spring-cloud-zuul:pom:0.0.1-SNAPSHOT and 'parent.relativePath' points at wrong local POM @ line 10, column 13
 @
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project com.baeldung.spring.cloud:spring-zuul-rate-limiting:0.0.1-SNAPSHOT (/Users/marcosbarbero/dev/github/zuul-rate-limiter/spring-zuul-rate-limiting/pom.xml) has 1 error
[ERROR]     Non-resolvable parent POM for com.baeldung.spring.cloud:spring-zuul-rate-limiting:0.0.1-SNAPSHOT: Could not find artifact com.baeldung.spring.cloud:spring-cloud-zuul:pom:0.0.1-SNAPSHOT and 'parent.relativePath' points at wrong local POM @ line 10, column 13 -> [Help 2]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
[ERROR] [Help 2] http://cwiki.apache.org/confluence/display/MAVEN/UnresolvableModelException

Unfortunately, this project doesn't run either.
spring-zuul-rate-limiting git:(main) ✗ mvn clean package
[INFO] Scanning for projects...
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[FATAL] Non-resolvable parent POM for com.baeldung.spring.cloud:spring-zuul-rate-limiting:0.0.1-SNAPSHOT: Could not find artifact com.baeldung.spring.cloud:spring-cloud-zuul:pom:0.0.1-SNAPSHOT and 'parent.relativePath' points at wrong local POM @ line 10, column 13
 @
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project com.baeldung.spring.cloud:spring-zuul-rate-limiting:0.0.1-SNAPSHOT (/Users/marcosbarbero/dev/github/zuul-rate-limiter/spring-zuul-rate-limiting/pom.xml) has 1 error
[ERROR]     Non-resolvable parent POM for com.baeldung.spring.cloud:spring-zuul-rate-limiting:0.0.1-SNAPSHOT: Could not find artifact com.baeldung.spring.cloud:spring-cloud-zuul:pom:0.0.1-SNAPSHOT and 'parent.relativePath' points at wrong local POM @ line 10, column 13 -> [Help 2]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
[ERROR] [Help 2] http://cwiki.apache.org/confluence/display/MAVEN/UnresolvableModelException


add SSSE3 code to cornerScore()

I improve the function "cornerScore()".
I eliminate the wasted "_mm_loadu_si128()" instruction for using ssse3 instruction.

@K-Shinotsuka, thanks for the contribution! do you have some numbers that's the speedup?

@vpisarev

thanks for the contribution! do you have some numbers that's the speedup?

Yes.
For example, ./opencv/modules/imgproc/src/canny.cpp.

It is possible to replace existing code with "_mm_abs_epi16()" in L370-L386 and L761-L777.
It is possible to replace existing code with "_mm_madd_epi16()" in L404-L420 and L795-L811.

ok, I see, but did you measure the improvements?

@vpisarev

@K-Shinotsuka, thanks for the contribution! do you have some numbers that's the speedup brings?

I'm sorry.
I did not understand your sentense.
I understand that you want to know how many cycles the function "cornerScore()" is improved.
I'm sorry but I didn't measure the improvements.
I have linux machine only.
Are you sure that I measure this code at linux machine?

@vpisarev
I measured the function "cornerScore()".
Please observe the following.
Function,                           Average Clocks,  Faster Rate (compared with sse2)
cornerScore<8>   sse2,      62.00,                -
cornerScore<8>   ssse3,	   59.75,                3.76%
cornerScore<12> sse2,    121.00,                -
cornerScore<12> ssse3,  117.75,                2.76%
cornerScore<16> sse2,    137.00,                -
cornerScore<16> ssse3,  133.50,                2.62%
Measurement Conditions
OS                                : Ubuntu 16.04
Compiler                       : g++ (Ubuntu 5.3.1-13ubuntu3) 5.3.1 20160330
CPU                              : Intel(R) Core(TM)2 Duo CPU     E8500  @ 3.16GHz
Compiler Option           : -msse2 -mssse3 -O3
Measurement Function: rdtsc()
Number of Loop           : 10000
Note:
Average Clocks is derived "(sum of clocks of function call) / (Number of Loop)".
I measure the clocks at linux machine only.

@vpisarev
Sorry, there are some mistakes in my test code for measurement.
My ssse3 code is fast compared to existing sse2 code because cache worked.
So I correct my test code and measure my code and existing sse2 code again.
The speed of my ssse3 code and existing sse2 code are same.
So I will close this pull request.
Next time later, I will be careful to write test code for measurement.
I apologize for the inconvenience.


:seedling: Bump BMO to v0.8.0-rc.0

As we accidentally published BMO v0.8.0-rc.0 instead of v0.7.0-rc.0, we decided to skip release-0.7 and jump directly to release-0.8.

/cherry-pick release-1.8


Support doubles

Update: #232 has added the ability to build Terrain3D with double precision.
To get it working you need to:

Build Godot with scons precision=double
Regenerate godot-cpp bindings with this new executable
Build Terrain3D (which includes godot-cpp) with scons precision=double custom_api_file=YOUR_CUSTOM_FILE

@zuriel13 reported the demo is working fine using Godot 4.2.1 and Terrain3D 0.9.1-dev.
Using a single precision engine, the terrain and shader look fine when moving the camera even 100k units away from the origin. This may break down at extremes though and bears testing.
Shaders do not support double precision, however the engine does include some double emulation. Clayjohn wrote an article describing how to emulate double precision in shaders for positional elements. He writes that the camera and model transform matrices needed to be emulated, and that is now done automatically in the engine.
https://godotengine.org/article/emulating-double-precision-gpu-render-large-worlds/
It's possible that other elements of the shader will need to emulate double precision, but no issues are apparent, yet.
Related to regenerating bindings: https://github.com/godotengine/godot-cpp/issues/1386

Include a build flag that supports doubles in both the shader and C++ where relevant. Probably only locations. This is done in godot source:
//math_defs.h
/**
 * The "Real" type is an abstract type used for real numbers, such as 1.5,
 * in contrast to integer numbers. Precision can be controlled with the
 * presence or absence of the REAL_T_IS_DOUBLE define.
 */
#ifdef REAL_T_IS_DOUBLE
typedef double real_t;
#else
typedef float real_t;
#endif

// Elsewhere
#ifdef REAL_T_IS_DOUBLE
	if (p_feature == "double") {
		return true;
	}
#else
	if (p_feature == "single") {
		return true;
	}
#endif // REAL_T_IS_DOUBLE

10k out

1M out, corruption and the mesh jitters all over the place

Double support has been tested and is reported working fine. Updated instructions in the first post.


Update node version used in CI

We are currently using Node 16, which has end of life on 2023-09-11
https://nodejs.org/en/blog/announcements/nodejs16-eol

I'm not able to test the GH actions workflows or the docker image, but Codespaces builds fine and the web still works (which also uses Docker) - so in that regard it looks good.


In which class characters recognized?

I've traced the code several times looking for the class which the characters recognized, but I can't found it.
where does the characters recognize?

Please use tesseract user forum for asking support.


Add a GET endpoint to retrieve change requests from the DNB service

Description of change
This is an expansion of the company-change-request endpoint, which can now retrieve change requests from the DNB service, and will be used to pass them to the frontend. In order to do this I have made a new serializer/utility function as the validity requirements are different for GET requests in this case.
For more context see this PR in the DNB service repo.

Checklist

[x] If this is a releasable change, has a news fragment been added?

Explanation
A news fragment is required for any releasable change (i.e. code that runs in or affects production) so that a corresponding changelog entry is added when releasing.
Check changelog/README.md for instructions.

[x] Has this branch been rebased on top of the current develop branch?

Explanation
The branch should not be stale or have conflicts at the time reviews are requested.

[x] Is the CircleCI build passing?

General points

Other things to check

Make sure fixtures/test_data.yaml is maintained when updating models
Consider the admin site when making changes to models
Use select-/prefetch-related field lists in views and search apps, and update them when fields are added
Make sure the README is updated e.g. when adding new environment variables

See docs/CONTRIBUTING.md for more guidelines.

I'll leave the review to Ali and Bredan, but congrats on getting this up Christopher! Something to really be proud of!! 🥳 🎉


AFK players don't get kicked and count toward the popcap limit.

The server can already detect afk players- it does it for admins consistently. Players marked as afk for extended periods of time should be automatically removed from the server to avoid taking up slots.
This is also an easy bandaid fix for #9092

@moonheart08 I know players intentionally don't get kicked because the idea is they're allowed to come back.
Should we remove them from limit counter but keep them in the round? Maybe remove from cap after x minutes and kick after y?
Pinging you 'cuz you seem most knowledgeable about it.

Been fixed twice, presumably.


Incorrect results while using a where clause(a string condition) against a field that has boolean values

Example:
I have a measurement called "m1" with field called "fault" that is a boolean. There are two points, one with fault = true, and another with fault = false.
Now, when I run the query : select * from m1 where fault = 'something'
it returns those rows where the value for "fault" is false. Shouldn't it return 0 rows as there are no rows where the value of "fault" is "something" ?

Comparing a boolean field with a non-boolean value is currently undefined. You are mixing types, so there is ambiguity here. What are you trying to do?

Strings evaluate to 0 in a Boolean context, which is equivalent to False. It's not ideal behavior but it is expected behavior. Since the type of a field is fixed only per-shard, the query cannot throw a parser error at runtime.

This works correctly in 0.13.


feat: add nested entity filters

Depends on https://github.com/checkpoint-labs/checkpoint/pull/246
This PR adds support for filtering entities by their subentities, for example following query becomes now possible:
{
  spaces (where: { metadata_: { name_contains: "Shadow DAO"}}) {
    id
    metadata {
      id
      name
      executors
      executors_types
    }
    voting_delay
  }
}

Test plan
Test against https://github.com/snapshot-labs/sx-api/pull/181

Awesome, this is a great feature. I tried and it seem to work well, I have just these 2 issues:
When I try to query the sub field "id" it will fail with this error:

I've tested on TheGraph a similar query and it works (they keep "id" filter in the sub entity "where").
Also when I try to query a sub entity using a required field as filter it fail. Here is my schema:
...
type User {
  id: String!
  name: String
  about: Text
  discussion_count: Int! # required field
  vote_count: Int! # required field
}
...

Here is the response I get:

Without this feature I imagine we wouldn't be able to do things like search on proposals on SX-UI, now I understand why you were looking on adding metadata directly in the Space entity level. It's great that you take that different approach, it will make it easier to integrate on UI as we can keep same format with the subgraph APIs.


dvdisaster: 0.79.9 -> 0.79.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/dvdisaster/versions.
meta.description for dvdisaster is: "Data loss/scratch/aging protection for CD/DVD media"
meta.homepage for dvdisaster is: "https://dvdisaster.jcea.es/"
meta.changelog for dvdisaster is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.79.10 with grep in /nix/store/xm0x4wc0z8jkqwpnag25n3vwcp6y9xgk-dvdisaster-0.79.10
found 0.79.10 in filename of file in /nix/store/xm0x4wc0z8jkqwpnag25n3vwcp6y9xgk-dvdisaster-0.79.10

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
dvdisaster
dvdstyler

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xm0x4wc0z8jkqwpnag25n3vwcp6y9xgk-dvdisaster-0.79.10 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xm0x4wc0z8jkqwpnag25n3vwcp6y9xgk-dvdisaster-0.79.10
ls -la /nix/store/xm0x4wc0z8jkqwpnag25n3vwcp6y9xgk-dvdisaster-0.79.10/bin

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

2 packages built:

dvdisaster
dvdstyler

Maintainer pings

Result of nixpkgs-review pr 184797 run on x86_64-linux 1

2 packages built:

dvdisaster
dvdstyler


Username/password is missing from required gmail schema

Username/password is missing from required gmail schema and they are required

Closed by #203


⚠ SolarOscp has degraded performance

In a2401f8, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 398 ms

Resolved: SolarOscp performance has improved in c25f9a3 after 12 minutes.


added recommeded extenstion.json

What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Use cases and why

One use case

Another use case

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Wasn't aware of this feature, great addition, thanks!


Build Error - Unable to clone jsoncpp repo, all others seem to be working

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary):  Source from master branch
TensorFlow version (use command below):
Python version: 3.6.2
Bazel version (if compiling from source):  cmake 3.9.4
CUDA/cuDNN version: CUDA 9.0/cuDNN 7
GPU model and memory:  GTX 1080
Exact command to reproduce:  MSBuild /filelogger /m:4 /p:Configuration=Release tf_python_build_pip_package.vcxproj

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
I have been trying to build the TensorFlow pip package.  For some reason, while I have verified that the build is able to clone the other git repos, the jsoncpp one seems to be killing my build.
Source code / logs
    42>CustomBuild:          Creating directories for 'jsoncpp'
    42>CustomBuild:          Performing download step (git clone) for 'jsoncpp'
    42>CustomBuild:          fatal: could not create work tree dir 'jsoncpp': Permission denied
    42>CustomBuild:          fatal: could not create work tree dir 'jsoncpp': Permission denied
    42>CustomBuild:          fatal: could not create work tree dir 'jsoncpp': Permission denied
    42>CustomBuild:          -- Had to git clone more than once:                    3 times.
    42>CustomBuild:          CMake Error at C:/Users/Bryce/Documents/Programming/os_clones/tensorflow/tensorflow/contrib/cmake/build/jsoncpp/tmp/jsoncpp-gitclone.cmake:66 (message):            Failed to clone repository:            'https://github.com/open-source-parsers/jsoncpp.git'
And later on,
C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\Common7\IDE\VC\VCTargets\Microsoft.CppCommon.targets(171,5): error MSB6006: "cmd.exe" exited with code 1. [C:\Users\Bryce\Documents\Programming\os_clones\tensorflow\tensorflow\contrib\cmake\build\jsoncpp.vcxproj]
msbuild.log

This question is better asked on  StackOverflow since it seems like a problem with your environment. There is also a larger community that reads questions there. Thanks!


Custom modeling llama with sharded tgi

This PR fixes a bug, where we don't use custom modeling_llama.py when TGI is used on more than 1 card. User needs to add path to the directory containing modeling_llama.py to PYTHONPATH before using TGI in sharded mode.

Rejected


Dev-tools minor update with: Ruby 3.2 (including bundler), Kitchen-terraform 7, and Alpine 3.18

Let's first complete https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/issues/1633


Support for OSGI bundle classifier

Thanks for this great tool it is invaluable in looking at a JBOSS Fuse upgrade project.
Currently what I am facing is a problem with the general use of bundles and jars where the type is generally not defined. The plugin is identifying 2 nodes a bundle and a jar. I tried to exclude or include only what I need but had no luck. Appreciate any advice.
Thanks
Ben

Hi,
Can you give me an example or do you have probably a project on GitHub to try?  Generally, the plugin considers the type of an artifact (but it does not show up in the graph). But with includes and excludes you can filter by type, e.g. *:*:jar:*.

I have the same problem. It happens for bundles built using maven-bundle-plugin.
I solved it using <mergeScopes>true</mergeScopes>.

@madmanu Yes, the scopes of dependencies can be merged into one node. However, the type of a dependency can not (classifiers can not either).
@bfesq Does mergeScopes solve your problem or is it really the type. Here you can find an example of mergeScopes (in the comments at the end): https://github.com/ferstl/depgraph-maven-plugin/issues/37

Hi I tried merge scopes and didnt work, the type of jar or bundle still
came up as jar is assumed when type not declared I guess. I have simply
done some manual edits to adjust the diagram. Will try and find an example
to reproduce.
On 4 July 2017 at 17:33, Stefan Ferstl<EMAIL_ADDRESS>wrote:

@madmanu https://github.com/madmanu Yes, the scopes of dependencies can
be merged into one node. However, the type of a dependency can not
(classifiers can not either).
@bfesq https://github.com/bfesq Does mergeScopes solve your problem or
is it really the type. Here you can find an example of mergeScopes (in
the comments at the end): #37
https://github.com/ferstl/depgraph-maven-plugin/issues/37
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ferstl/depgraph-maven-plugin/issues/38#issuecomment-312916076,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD9_wUuqh3fP2G9pkAaVApssus9B_ovTks5sKmlJgaJpZM4N3H5Z
.

@bfesq Ok, an example would be helpful. Have you tried filtering by type as I suggested in the second comment of this issue?

@ferstl You are right, Bundles that differ only from a jar by their type are not merged.
However, I cannot filter them out because some dependency arrows start from the jar, and some others start from the bundle.
We may have both bundle and jar because of the way the Felix Maven plugin handles the bundle packaging.
Here is my minimal POM:
<project>
	<modelVersion>4.0.0</modelVersion>

	<groupId>x</groupId>
	<artifactId>y</artifactId>
	<version>z</version>
	<packaging>bundle</packaging>

	<dependencies>
...
	</dependencies>

	<build>
		<plugins>
			<plugin>
				<groupId>org.apache.felix</groupId>
				<artifactId>maven-bundle-plugin</artifactId>
				<extensions>true</extensions>
				<configuration>
...
				</configuration>
			</plugin>
		</plugins>
	</build>

</project>

I think introducing a mergeTypes option in the plugin would help. This option might also be set to true by default.

Version 3.0.0 is available on Maven Central

Thanks!


Temporarily enter command mode from insert mode with ctrl+o

As I gather, this plugin „unbinds self from the type event“. Which I like in general… except: What would it take to allow ctrl+o to temporarily enter command mode from insert mode? It seems to work in principle with other mappings such as ctrl+r.

This may work by just enabling <C-o> in package.json while being in insert mode (will route the request to neovim and it will trigger command line mode event which will be handled by extension). Not sure about consequences though

For now, I added this to my keybindings.json. Seems to work so far.
{
    "key": "ctrl+o",
    "command": "vscode-neovim.send",
    "args": "<c-o>",
    "when": "editorTextFocus && neovim.mode == insert"
}

Great stuff! 👍

It doesn't work as expected.
Reproduce

open test.txt
press "i" and input "test"
press "ctrl+o" and "2h"
the cursor stays on the position behind "test".

The cursor should lie between "t" and "e".

@wrvsrx I suggest you open a separate bug with a more detailed title. ctrl+o works fine in most cases by my observation.

@wrvsrx I suggest you open a separate bug with a more detailed title. ctrl+o works fine in most cases by my observation.

@trkoch i believe it still won't see insert mode changes, so use at your own risk

@trkoch i believe it still won't see insert mode changes, so use at your own risk

@trkoch I've made a PR to fix this

@QAston 😎 I'll give it a try. Hope @asvetliakov finds time to review.


Feature/signup

"As a user, I should be presented with the option to apply to the mentor or mentee program". Initially, the signup page directed users to the mentee application form and gave no option to apply to become a mentor. This feature presents users with both options so that they are able to navigate to the correct form whether they want to become a mentee or a mentor.
*** Loom video link: https://www.loom.com/share/7c91f4963edd45d5aa7144d05303ad98 ***
Type of change

Updated signup.js component

Created cards that presented a user with the option to sign up for the mentor or mentee program

directed users to the correct application form

created additional signup.css file to make the cards follow the wireframe that was created for this component

[x] Bug fix (non-breaking change which fixes an issue)

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have removed unnecessary comments/console logs from my code
[x] I have made corresponding changes to the documentation if necessary (optional)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

Good job, the code is clear and easy to read. Good use of antd.


fix url metadata

The title wasn't displaying correctly on LinkedIn, probably because og:url was always the same and wasn't dynamic with the page
https://www.linkedin.com/post-inspector/inspect/https:%2F%2Fexpressjs.com%2Fen%2Fblog%2Fposts

LGTM
https://www.linkedin.com/post-inspector/inspect/https:%2F%2Fdeploy-preview-1665--expressjscom-preview.netlify.app%2F2024%2F10%2F15%2Fv5-release

we could have an image, author in post

Closed in favor of #1666


How to display the images which are the ouput of predict_generator() using matplotlib

Please make sure that the boxes below are checked before you submit your issue. If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.
Thank you!

[ ] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/keras-team/keras.git --upgrade --no-deps

[ ] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[ ] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

This is the code and i trained a Unet model using keras generators.The generator has 20 images in it.I need to display all predictions.
y_pred=model.predict_generator(test_generator,verbose=1)
y_pred.shape
imgs=np.resize(y_pred,(256,256))
plt.imshow(imgs,cmap='gray')

plt.show()

@RamananThiru Moving this issue to closed status as there has been no recent activity, in case you still face the error please create a new issue.Thanks!


Conda manager: confirmation popup

From contrebasse on 2015-01-23T04:03:31Z
Spyder Version:  2.4.0dev (4711+: rb5a5ef2bb4d5 )
Python Version:  3.4.2
Qt Version    :  4.8.6, PyQt4 (API v2) 4.11.3 on Linux
pyflakes >=0.6.0:  0.7.3 (OK)
pep8 >=0.6      :  1.5.7 (OK)
IPython >=1.0   :  2.3.1 (OK)
zmq >=2.1.11    :  14.4.1 (OK)
pygments >=1.6  :  2.0.1 (OK)
pandas >=0.13.1 :  0.15.2 (OK)
sphinx >=0.6.6  :  1.2.3 (OK)
psutil >=0.3    :  2.1.3 (OK)
jedi >=0.8.1    :  0.8.1-final0 (OK)
rope >=0.9.2    :  0.9.4-1 (OK)
matplotlib >=1.0:  1.4.2 (OK)
sympy >=0.7.0   :  0.7.6 (OK)
pylint >=0.25   :  1.4.0 (OK)
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Right now the confirmation popup is divided in 4 parts, similar to conda command line interface. Most of the time the same package appear in multiple sections. I think it doesn't work well with a gui, I would expect a table indicating for each package what action will be performed (uninstall, unlink, download...).
Please provide any additional information below
.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=2133

From<EMAIL_ADDRESS>on 2015-01-23T01:57:38Z
Agreed, good idea :), will put it on my todo list

From ccordoba12 on 2015-01-30T14:13:00Z
Labels: MS-v2.4 Cat-Miscelleneous

Working on this from next week!

Relocated in https://github.com/spyder-ide/conda-manager/issues/1


import latest v2.7.0 argo version in go project

Summary
How can I import latest argo in go project?
Motivation
Why do you need to know this, any examples or use cases you could include?
when I want import argo v2.7.0 in my project
go get<EMAIL_ADDRESS>
But I got error
not found<EMAIL_ADDRESS>invalid version: module contains a go.mod file, so major version must be compatible: should be v0 or v1, not v2

How to fix it?

We had a PR to fix this, #1375 ?
Can you do go get github.com/argoproj/argo?

We had a PR to fix this, #1375 ?
Can you do go get github.com/argoproj/argo?

go get github.com/argoproj/argo

But got
github.com/argoproj/argo v2.5.2+incompatible

not v2.7.0

Referring to  https://github.com/golang/go/issues/35732.
Use
go get github.com/argo/proj/argo@master

It will get latest argo version.
Referring to https://golang.org/cmd/go/#hdr-Module_compatibility_and_semantic_versioning
It should be
module github.com/argoproj/argo/v2

not https://github.com/argoproj/argo/blob/master/go.mod#L1

Would you like to submit a PR to fix this ?

@tcolgate FYI

Yes, my previous PR fixes this, but it's a big change and is very invasive (also has implications for future v3 updates, and people should probably discuss if the related kube CRD resource version should be taken into consideration).
Realistically I'm not going to get to work on this any time soon, but the PR may be enough.

It does't look possible to to migrate to Gomodules using v2 because you must either use "branch" or "subdirectory":
https://stackoverflow.com/questions/53344471/taking-a-repository-to-v2
Both these solutions have downsides that mean we could not use them without making changes that are too time consuming and risky.
Marking as "won't fix".

@alexec Since the bug is not going to be fixed then the supported installation method becomes using argo@stable or argo@release-<version>? It's not as ideal but it's better than using argo@master. Is this something we could add to the README for users that have argo as a dependency?
Users could add the version number as a comment in the go.mod file to keep track of what version they are on as a workaround. That is likely what my company will end up doing for our projects since v0.0.0-20200528233840-0fff4b21c21c is not useful.
$ go get github.com/argoproj/argo@stable
go: github.com/argoproj/argo stable => v0.0.0-20200528233840-0fff4b21c21c
go: downloading github.com/argoproj/argo v0.0.0-20200528233840-0fff4b21c21c

or
$ go get<EMAIL_ADDRESS>go: github.com/argoproj/argo release-2.8 => v0.0.0-20200528233840-0fff4b21c21c

I'd like to +1 that @blkperl, just spent a lot of time trying to figure out how to get the argo downloaded properly without a bunch of errors popping up (it kept fetching the 2.5 incompatible releases)

I think we should document this.

#3369

hey I know I'm running here very late but I think the problem that you guys ran into was that if you're opting into modules at a version later than major v0 or v1, you must release this as a new major version of the library.
See https://github.com/gofrs/uuid/issues/61

https://donatstudios.com/Go-v2-Modules

Ok.  We cannot support Gomodules because awful.
Instead, get the specific tag related:
revision=$(git ls-remote --tags https://github.com/argoproj/argo refs/tags/v2.11.1 | grep "$rx" | sort -d | tail -n1)
go get github.com/argoproj/argo@$revesion

Patch version? E.g. v2.11.1?
rx=^v2\.11\.1$

Minor version? E.g. v2.11
rx=^v2\.11\.[0-9]*$

Major version. E.g. v2
rx=^v2\.[0-9]\.[0-9]*$


docs: update README.md

alingning -> aligning

Thanks!

Thanks for the contribution @eltociear! I must write a guide for contributions :-)


Chrome extension, getAuthenticationUrl & Invalid redirect_uri

Hi!
I am using "dropbox": "2.5.3" on node v7.8.0, chrome version 58
I have whitelisted my chrome extension URL in app administration, but when executing getAuthenticationUrl i always get Error 400 (Invalid redirect_uri)
In previous version of API i used chrome_oauth_receiver but that is probably out of date and i am unable to find any documentation for extension support. Thanks a lot!

Hi @chren-de-large , this SDK wasn't built with explicit support for running in Chrome extensions, but we'll be happy to take a look at this issue to see how we can help.
First, can you share the URL you're getting from getAuthenticationUrl that isn't working for you? Thanks in advance!

Hi! Yes, exactly - those url is heading to those error 400

On 22 May 2017, at 21:27, Greg K<EMAIL_ADDRESS>wrote:
Hi @chren-de-large , this SDK wasn't built with explicit support for running in Chrome extensions, but we'll be happy to take a look at this issue to see how we can help.
First, can you share the URL you're getting from getAuthenticationUrl that isn't working for you? Thanks in advance!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Can you post the actual URL value you're getting back from getAuthenticationUrl though? Thanks!

https://www.dropbox.com/oauth2/authorize?response_type=token&client_id=s340kh3l0vla1nv&redirect_uri=chrome-extension://imloifkgjagghnncjkhggdhalmcnfklk

https://www.dropbox.com/oauth2/authorize?response_type=token&client_id=k1qq2saf035rn7c&redirect_uri=chrome-extension://imloifkgjagghnncjkhggdhalmcnfklk

I'm currently getting the following, different error message for this URL:

Implicit grant disallowed for this app ("response_type" can't be "token").

That indicates that you've set "Allow implicit grant" to "Disallow" for that app. You can switch that back via the App Console.
Please give that a try and let me know if you're still able to reproduce the redirect_uri issue.

Ou, yes - thats helps, i screw up the config there.
Now, i got error on redirect back due to Chrome security policy, i have to somehow attach that redirect to web_accessible_resources in manifest ... will explore more and write back
Denying load of chrome-extension://imloifkgjagghnncjkhggdhalmcnfklk/#access_token=kpUWLsNM_…bearer&uid=484049575&account_id=dbid%3AAADngg4RY5thIBCOjThwAvFQZNC1PuWb96c. Resources must be listed in the web_accessible_resources manifest key in order to be loaded by pages outside the extension.

ok, thanks for help :) would be great to have some chrome extension example for the future, but i understand that you have other priorities.


Open player detail on hover

Is it possible an option to let the window keep current with the current target/hover player? that way I can have on sight the player notes or other info useful to have without each time have to lookout a separate list, or option/lookout player on the plugin.

Not sure this would work well. The user can already be easily opened with the context menu.


visualize jaxpr graph?

i was wondering if there are any tools to visualize jaxprs.
Certainly, one can visualize the entire HLO graph using jax.xla_computation. But that tends to produce overly complex graphs, especially when jitting.
It would be nice to be able to work on jaxprs and visualize jaxpr dependency graphs. Does jax have any tools in this regard (besides the toy example in https://gist.github.com/mattjj/a60e0991455965ae960a8d2dcddc3407)? Ideally, the tool would also allow annotation of functions or arrays.
My motivating example is that I am going through a fairly complex custom likelihood function on time series observations, that involves many lu_solves and eigh operations and the HLO graph is immense.

This is likely out of scope for the time being. We don't consider the Jaxpr IR an official public part of JAX's API. The example you link is a good place to start for putting something bespoke together, and hopefully you could adapt that to your needs (over time, it would have to adapt to changes in Jaxpr).


ERC-20: Add token price display for BCS

block.cc Link: https://mifengcha.com/coin/bcachain
Token Symbol: BCS
Token Contract Address: 0x31274db8b609Df99E5988ee527071643b5160Fc3
Etherscan Link: https://etherscan.io/token/0x31274db8b609df99e5988ee527071643b5160fc3

Done

Thank you. @sfsy520


chore: test to make sure nothing is logged during n3reasoner calls

closes #249

:tada: This PR is included in version 4.18.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v4.18.0

Your semantic-release bot :package::rocket:


Provide custom warning when misplaced file matches a missing file except by file extension

As Makin reports:

the double error is confusing, so maybe if there is a matching filename situation you can just say "need to define filetype or change the file"

I'm not sure the exact right syntax, but for cases like this, it should be filtered out of both missing and misplaced (when logging) and put under a separate "Expected a different file extension for these files, replace them or specify a custom file extension" list. Or something similarly descriptive!

Done in cabae7f32a08c8d1e9ce9609b0325cfe81135b60.

1 expected image file is missing from media:
 - (missing) album-art/broken-radio/lost-transmission.jpg
2 image files, present in media, weren't expected:
 - (misplaced) All files under artist-avatar
 - (misplaced) album-art/broken-radio/lost-transmission.png
Of these, 1 has an unexpected file extension:
 - (expected .jpg) album-art/broken-radio/lost-transmission.png
To handle unexpected file extensions:
 * Source and replace with the correct file, or
 * Add "Cover Art File Extension" field (or similar)
   to the respective document in YAML data files.


Slow HTTP Response

First of all, great job, it works like a charm!
I found out that if I send any code (e.g. "http://hostname.local:80/msg?code=E0E040BF:SAMSUNG:32") the IR Blaster seems to be busy until the http-Response (the HTML) is delivered to the client (e.g. the Browser). Maybe it will be possible to add a get/JSON parameter not to return the full HTML-Response, only the HTTP-Status. It could look like:
http://hostname.local:80/msg?code=E0E040BF:SAMSUNG:32&simpleresponse=1
If the IR code was sent correct, just return HTTP-Status 200 (no HTML-Body). If not, return HTTP-Status 400 with an error message.

Great idea, just pushed an update to include this and updated the readme
Add &simple=1 to achieve this feature


Story: publishing a package to the registry

Story link
This is a story issue to track the overall progress towards achieving a user story.
Task issues will be track here soon.

Cleaning up old issues


Pytests now failing in new typer-cli branch

~/coding/audio_program_generator on  typer-cli ⌚ 8:02:07
$ pytest
ImportError while loading conftest '/Users/jwr003/coding/audio_program_generator/tests/conftest.py'.
tests/conftest.py:10: in <module>
    import audio_program_generator.apg as apg
audio_program_generator/apg.py:14: in <module>
    from audio_program_generator import cli
audio_program_generator/cli.py:10: in <module>
    from audio_program_generator.apg import AudioProgramGenerator
E   ImportError: cannot import name 'AudioProgramGenerator' from partially initialized module 'audio_program_generator.apg' (most likely due to a circular import) (/Users/jwr003/coding/audio_program_generator/audio_program_generator/apg.py)

Resolved.


OOM

OM when allocating tensor with shape[100,256,28,28] and type float on /job:localhost/replica:0/task:0/device:GPU:0 by allocator GPU_0_bfc
Traceback (most recent call last):
File "keras_maskrcnn/bin/train.py", line 314, in
main()
File "keras_maskrcnn/bin/train.py", line 310, in main
max_queue_size=1,
File "/home/devinda/libs/keras-maskrcnn/.env/local/lib/python2.7/site-packages/keras/legacy/interfaces.py", line 91, in wrapper
return func(*args, **kwargs)
File "/home/devinda/libs/keras-maskrcnn/.env/local/lib/python2.7/site-packages/keras/engine/training.py", line 1418, in fit_generator
initial_epoch=initial_epoch)
File "/home/devinda/libs/keras-maskrcnn/.env/local/lib/python2.7/site-packages/keras/engine/training_generator.py", line 217, in fit_generator
class_weight=class_weight)
File "/home/devinda/libs/keras-maskrcnn/.env/local/lib/python2.7/site-packages/keras/engine/training.py", line 1217, in train_on_batch
outputs = self.train_function(ins)
File "/home/devinda/libs/keras-maskrcnn/.env/local/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py", line 2715, in call
return self._call(inputs)
File "/home/devinda/libs/keras-maskrcnn/.env/local/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py", line 2675, in _call
fetched = self._callable_fn(*array_vals)
File "/home/devinda/libs/keras-maskrcnn/.env/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1399, in call
run_metadata_ptr)
File "/home/devinda/libs/keras-maskrcnn/.env/local/lib/python2.7/site-packages/tensorflow/python/framework/errors_impl.py", line 526, in exit
c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.ResourceExhaustedError: OOM when allocating tensor with shape[100,256,28,28] and type float on /job:localhost/replica:0/task:0/device:GPU:0 by allocator GPU_0_bfc
[[{{node training/Adam/gradients/mask_submodel/roi_mask_features/convolution_grad/Conv2DBackpropFilter-0-TransposeNHWCToNCHW-LayoutOptimizer}} = Transpose[T=DT_FLOAT, Tperm=DT_INT32, _device="/job:localhost/replica:0/task:0/device:GPU:0"](mask_submodel/roi_mask_upsample/resize_images/ResizeNearestNeighbor, PermConstNHWCToNCHW-LayoutOptimizer)]]
Hint: If you want to see a list of allocated tensors when OOM happens, add report_tensor_allocations_upon_oom to RunOptions for current allocation info.

Two questions, what GPU are you using, and are you sure you're on the latest version of maskrcnn? There was a change not so long ago that greatly reduces the memory requirements.

GeForce 940M, is this not enough?
I will get an update

It only has 2Gb of memory, which isn't enough. The default settings on my system uses 8531Mb when training COCO.

it has 4GB, aslo i'm not training COCO but my own dataset, still conditions same? need 8.5 GB?

I got an update and its working, Thanks


The ListeningConditions experiment is broken on production

Important update: This bug probably has been fixed already but not yet deployed.
=====
Describe the bug
The ListeningConditions experiment cannot fetch the next round.
To Reproduce
Steps to reproduce the behavior:

Go to https://app.amsterdammusiclab.nl/lisc
Navigate through the consent
See error

Expected behavior
The ListeningConditions experiment is loaded successfully without errors
Screenshots

Desktop (please complete the following information):

OS: macOS
Browser Safari

Additional context

Sentry error is probably this one prepare_result() missing 1 required positional argument: 'session'

I have deployed this and verified it solved the issue!


Add editorconfig file

This file contains rules that editors can use for code analysis. (such as vscode)
Copied straight from osu since we want to maintain the same conventions.

@frenzibyte are you happy with this? I can’t test from my phone. :P

Will check once I'm back home


Update README.md with NVT + CERT + SCAP database feeds

According to this blog, (have confirmed, it works) I think it would be safe to add the following code snippet for those who want to update all the three databases
    greenbone-nvt-sync
    greenbone-certdata-sync
    greenbone-scapdata-sync
    openvasmd --update --verbose --progress

    /etc/init.d/openvas-manager restart
    /etc/init.d/openvas-scanner restart

@naman This should now be fixed via https://github.com/mikesplain/openvas-docker/pull/170
Thanks for the tips!


Issues with async/await in handlers

When playing with the library, I found two issues when using asynchronous code in my handlers. I'm using it with koa, but I think the problem would be the same for every framework.
First, the calls to the handlers are not awaited, meaning I can't do async work in the handler.
There is a possible fix here.
Second, its a really small one but since we only have one instance of the validation function, the validation errors can be lost in the case a async call is made in the error handler.
For example, is the following handler is registered as validationFail, c.validation.errors in the body can contains a different error.
module.exports = async (c, req) => {
  await logToElasticsearch(c.validation.errors);
  req.response.status = 400;
  req.response.body = c.validation.errors;
};

This is obviously easy to fix like this:
module.exports = async (c, req) => {
  const errors = c.validation.errors;
  await logToElasticsearch(errors);
  req.response.status = 400;
  req.response.body = errors;
};

That being said I made a small patch for this aswell.
Feel free to reuse any of this, and/or close the issue.

Great work!
As far as I know, when returning an async function (i.e. a function that retuns a promise) I'm pretty sure you shouldn't need to await. Could you give me a working example where this is not the case? I haven't run into this issue with my async handlers yet.
For the second problem, I think this upcoming change will actually fix the issue since we are no longer returning the validator function in the context object, but a similar validation result object with just a shallow copy of the errors (I think). Can you verify?
https://github.com/anttiviljami/openapi-backend/pull/8/files#diff-7d4dec20edd9d5dea9fe6a05f70cf757R158

I mean I don't really see any problem with adding the awaits, but I'm just really curious to see why async work can't be performed.
Not adding the await just seems better in my eyes so users have an option to do non-async handlers as well.
export type Handler = (context?: Context, ...args: any[]) => any | Promise<any>;

You're actually right for the first one, I feel kind of dumb now :'(.
I added the await at the same time on api.handleRequest and inside handleRequest not realizing the latter was actually useless...
I'm going to try the PR for the second issue now.

Yes I confirm that the second issue is fixed when running the code inside the PR.
Thanks for checking.

Great! Closing this issue now and publishing changes from #6. :)


Adapter name for SFT trainer

Hi, I appreciate that the PPOTrainer is enabled to assign ''adapter_name'' to trainable adapters, would you please also enable SFTTrainer with this function?

Hi @para-zhou
Thanks for the issue ! adapter_name is used on our AutoModelForxxx classes that are internal to TRL. In order to switch between adapters in SFTTrainer you should first create your PeftModelForxxx and call model.set_adapter(adapter_name)


Support for 1.24

Describe the feature
Sorry for the format - the README suggests to post questions in issues, but the issues require a template to be used. If there's a more suitable place for discussion on these topics, please direct me to it - thanks!
Three questions:
Is support for EKS 1.24 coming to blueprints? 1.23 is EoL in 7 months time, and CDK will soon be supporting 1.25. There's already a very short time-span of support for EKS/k8s in general, and having blueprints support so far behind is making me wary of basing anything on it.
How about an example of how to set up VPC peering on the resulting VPC?
What is the best practice for upgrading a blueprints cluster from one k8s version to the next?
Use Case
...
Proposed Solution
No response
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request

CDK version used
2.60
EKS Blueprints Version
1.5.4
Node.js Version
v16.17.0
Environment details (OS name and version, etc.)
Darwin Kernel Version 22.2.0

@k-dahl it is the right place for such discussions.
Support for 1.24 is already available in the 1.5.4 now, with this change. You should be able to create a cluster passing your version to the builder:
import { KubernetesVersion } from 'aws-cdk-lib/aws-eks';
...
   blueprints.EksBlueprint.builder()
            .version(KubernetesVersion.V1_24)
            .addOns(...)

We will add support for 1.25 with the upcoming 1.6 release. In general, if a version of the target EKS version is not yet in the CDK, you can still pass it as KubernetesVersion.of("1.25") for example, and use the kubectl layer for the previous version which should be compatible for most cases.
Cluster upgrades (in-place) simply requires passing the new version to the control plane. This won't handle the kubelet upgrade though, which will require passing correct amiReleaseVersion to the node group as desribed here.
VPC peering setup may be a good pattern. Potentially, you can achieve it by registering a custom VPC resource provider that handles peering set up for the new VPC. This, however, would be a good pattern for us to add to the patterns repo. I can keep this issue to track it.

Thanks for the insights @shapirov103. Unfortunately I have not been able to get 1.25 working.
If I use KubernetesVersion.of("1.25") I receive this warning:
[Warning at /custom-eks-cluster/eks-blueprint/eks-blueprint] You created a cluster with Kubernetes Version 1.25 without specifying the kubectlLayer property. This may cause failures as the kubectl version provided with aws-cdk-lib is 1.20, which is only guaranteed to be compatible with Kubernetes versions 1.19-1.21. Please provide a kubectlLayer from @aws-cdk/lambda-layer-kubectl-v25.

If I try and use a generic provider and specify the 1.24 KubeCtlLayer I get this:
ERROR [main   printFatalErrorUponExit] Error: KubectlV24Layer at 'custom-eks-cluster/kubectllayer-v24' should be created in the scope of a Stack, but no Stack found

I can't find any example online of Generic Cluster Provider being used with KubectlV24Layer so I am not sure if this is just something I have configured incorrectly or a bug.

@k-dahl thank you for looking into this.
There was a fix to supply the 1.24 layer for any version above 1.24 here which must have failed. This will be addressed and included in the next maintenance release.

@k-dahl please try with version 1.6.0 of Blueprints. I ran the tests for 1.25 and the only issue I discovered was kube-proxy addon was incompatible and required version upgrade to v1.24.10-eksbuild.2. You can specify the add-on version when you create them.
The rest of the upgrades are here: https://github.com/aws-quickstart/cdk-eks-blueprints/pull/623
However, the cluster deployed fine with no code changes (only config for a couple of addons).

1.24, 1.25, 1.26 are all supported, however testing was performed on 1.24 and 1.25


Update to QPois and added unit test

Cache transition points for both upper and lower quantiles, issue #226. Added unit test for QPois: is_outlier.

Please remove unused print functions and implement more precise unit tests (using Poisson ppf function in R to get correct values), thanks!

Please remove unused print functions and implement more precise unit tests (using Poisson ppf function in R to get correct values), thanks!


Example powersystem.py doesn't work at all

Steps to reproduce

pip install nevergrad
git clone git@github.com:facebookresearch/nevergrad.git
cd nevergrad/examples
python powersystem.py
using anaconda python 3.9 on Linux Mint 20.3.

Observed Results

What happened?  This could be a description, log output, etc.

ana39/lib/python3.9/site-packages/nevergrad/optimization/base.py:146: InefficientSettingsWarning: num_workers = 10 > 1 is suboptimal when run sequentially
warnings.warn(msg, e)
Traceback (most recent call last):
File "/home/xxx/sich/examples/powersystem.py", line 43, in
optimizer.minimize(power_system_loss)
File "/home/xxx/ana39/lib/python3.9/site-packages/nevergrad/optimization/base.py", line 641, in minimize
result = job.result()
File "/home/xxx/ana39/lib/python3.9/site-packages/nevergrad/optimization/utils.py", line 137, in result
self._result = self.func(*self.args, **self.kwargs)
File "/home/xxx/ana39/lib/python3.9/site-packages/nevergrad/functions/base.py", line 119, in call
return self.function(*args, **kwargs)
File "/home/xxx/ana39/lib/python3.9/site-packages/nevergrad/functions/powersystems/core.py", line 129, in _simulate_power_system
agent.set_parameters(array)
File "/home/xxx/ana39/lib/python3.9/site-packages/nevergrad/functions/powersystems/core.py", line 37, in set_parameters
raise ValueError(f"length = {weights.size} instead of {self.dimension}: {weights}.")
ValueError: length = 7560 instead of 1260: [-0.73698052  1.41931025  1.34390607 ... -0.79129847 -0.77101758
-0.6903909 ].
Expected Results

What did you expect to happen?

Expected "optimizer.minimize(power_system_loss)" to perform some optimization and
"power_system_loss.make_plots" to show some plots.
As it is the one and only nevergrad example the observed result is a bit disappointing.

`import nevergrad as ng
from nevergrad.functions.powersystems.core import PowerSystem
budget = 3500
width = 6
depth = 6
num_dams = 6
year_to_day_ratio = 0.5
back_to_normal = 0.5
num_thermal_plants = 6
constant_to_year_ratio = 4.0
power_system_loss = PowerSystem(
num_dams=num_dams,
depth=depth,
width=width,
year_to_day_ratio=year_to_day_ratio,
back_to_normal=back_to_normal,
num_thermal_plants=num_thermal_plants,
constant_to_year_ratio=constant_to_year_ratio,
)
added line:
param = int(power_system_loss.dimension/power_system_loss.num_dams)
changed 'parametrization=power_system_loss.dimension' in the optimizer parameters:
optimizer = ng.optimizers.SplitOptimizer(
parametrization=param, budget=budget, num_workers=1
)
optimizer.minimize(power_system_loss)
power_system_loss(optimizer.provide_recommendation().value)
power_system_loss.make_plots(
f"ps_{num_dams}dams_{depth}_{width}_ytdr{year_to_day_ratio}_btn{back_to_normal}"
f"_num_thermal_plants{num_thermal_plants}_ctyr{constant_to_year_ratio}_budget{budget}.png"
)`


[SPARK-19945][SQL]add test suite for SessionCatalog with HiveExternalCatalog

What changes were proposed in this pull request?
Currently SessionCatalogSuite is only for InMemoryCatalog, there is no suite for HiveExternalCatalog.
And there are some ddl function is not proper to test in ExternalCatalogSuite, because some logic are not full implement in ExternalCatalog, these ddl functions are full implement in SessionCatalog(e.g. merge the same logic from ExternalCatalog up to SessionCatalog ).
It is better to test it in SessionCatalogSuite for this situation.
So we should add a test suite for SessionCatalog with HiveExternalCatalog
The main change is that in SessionCatalogSuite add two functions:
withSessionCatalog and withSessionCatalogAndExternal
And replace the code like  val catalog = new SessionCatalog(newBasicCatalog) with above two functions
How was this patch tested?
add HiveExternalSessionCatalogSuite

Test build #74501 has started for PR 17287 at commit 4468446.

Test build #74501 has finished for PR 17287 at commit 4468446.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class InMemorySessionCatalogSuite extends SessionCatalogSuite
abstract class SessionCatalogSuite extends PlanTest
class HiveExternalSessionCatalogSuite extends SessionCatalogSuite

Merged build finished. Test FAILed.

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

Test build #74502 has started for PR 17287 at commit 9ee9fa7.

Test build #74504 has started for PR 17287 at commit d25466c.

Test build #74502 has finished for PR 17287 at commit 9ee9fa7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #74504 has finished for PR 17287 at commit d25466c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #74512 has started for PR 17287 at commit c895a30.

Test build #74514 has started for PR 17287 at commit aaaedf9.

Test build #74512 has finished for PR 17287 at commit c895a30.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #74514 has finished for PR 17287 at commit aaaedf9.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #74521 has started for PR 17287 at commit bc8af7e.

Test build #74521 has finished for PR 17287 at commit bc8af7e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class HiveExternalCatalogSuite extends ExternalCatalogSuite with BeforeAndAfterAll

Merged build finished. Test FAILed.

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

Test build #74531 has started for PR 17287 at commit 47c1797.

Test build #74532 has started for PR 17287 at commit c360936.

Test build #74532 has finished for PR 17287 at commit c360936.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #74531 has finished for PR 17287 at commit 47c1797.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class HiveExternalCatalogSuite extends ExternalCatalogSuite
class HiveExternalSessionCatalogSuite extends SessionCatalogSuite with TestHiveSingleton

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

Merged build finished. Test PASSed.

Test build #74596 has started for PR 17287 at commit 35c561f.

Test build #74596 has finished for PR 17287 at commit 35c561f.

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

Merged build finished. Test PASSed.

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

@gatorsmile @cloud-fan I have fix some code view, could you help to continue reivew this?

Test build #74637 has started for PR 17287 at commit 80df8c7.

Test build #74638 has started for PR 17287 at commit 4214379.

Test build #74642 has started for PR 17287 at commit 25da5f6.

Test build #74643 has started for PR 17287 at commit d82e8ed.

Test build #74637 has finished for PR 17287 at commit 80df8c7.

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

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

Merged build finished. Test PASSed.

Test build #74638 has finished for PR 17287 at commit 4214379.

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

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

Merged build finished. Test PASSed.

Test build #74642 has finished for PR 17287 at commit 25da5f6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #74647 has started for PR 17287 at commit d82e8ed.

Test build #74647 has finished for PR 17287 at commit d82e8ed.

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

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

Merged build finished. Test PASSed.

LGTM

Thanks! Merging to master.

a late LGTM :)

I addressed these comments in the PR https://github.com/apache/spark/pull/17354


[api] Update the carbonmark-api GROQ query to sanity to grab the array of image urls

@Najada please check with @Atmosfearful if any questions.

@jabby09 all clear

hi @Najada what is the status of this one?
are we ready for the front end work for this epic to begin?

https://github.com/KlimaDAO/klimadao/issues/1093#issuecomment-1545723668


Please remove version 1.3 (?)

Hello,
We have recently updated our dependency on rdf-urdna to version 1.3. This update was resulting in issues and my conclusion was that there's a bug in this version.
However, as I was looking to report this issue here, I've noticed that the tag for version 1.3 (https://github.com/setl/rdf-urdna/tree/1.3) is presently not on the master branch, and this raised some suspicions. Was the intention to roll the release of this version back by resetting the master branch? If so, please remove the tag and make sure that the jar is unpublished from Maven, because those currently still exist.
If version 1.3 in its current state is actually supposed to exist, then I'll report said bug in a separate issue. Please let me know.
Thanks!

Hello, thanks for confirming.
I've dug a little deeper and the issue is on our end, actually. We had an exclusion on titanium-json-ld, but version 1.3 of rdf-urdna apparently requires titanium 1.4.0 to function correctly. So never mind about that bug report! ;)


Start using gopkg.in or something similar

I've been making a lot of breaking changes. It would be nice for anyone who actually wants to use this to have a more stable base.

I'm going to close this for now because there isn't anything I (currently) plan on adding which would make this necessary


Conflicting statements regarding which MSBuild/VS versions are supported by 8.0 SDKs

Describe the issue or suggestion
From what I could find there are at least three places that state the minimum MSBuild/VS versions supported by each feature band of the .NET SDK.

The dedicated documentation page
The .NET download page inlcudes "Visual Studio support" for each SDK
One of the .NET 8 breaking changes announcement announces changes to the rules.

All of these places disagree on what is compatible and what isn't. Below are the compatible MSBuild/VS version according to each.

SDK Version
Documentation page
Download page (as of release 8.0.8)
Breaking changes

8.0.1xx
17.7
17.8
17.7

8.0.2xx
17.8
N/A
17.8

8.0.3xx
17.8
17.10
17.8

8.0.4xx
17.8
17.11
17.9

After "falling in the trap" of believing the documentation page and getting a really poor error message from VS 17.8 with SDK 8.0.400, I believe that the information on the breaking changes page is the one that is correct. but I don't really know.
These version support tables should be unified to contain the correct information in all places.

@marcpopMSFT Can you comment on this discrepancy?

The download page is listing the version of VS that we ship in. It might be worth changing the wording there to make that more clear. This matches the second column in the documentation page that was linked.
The documentation page captures our intent. However, NuGet took an upgraded dependency that forced us to upgrade our minimum VS version to 17.9 in 8.0.4xx. I didn't update the documentation page but rather got NuGet to fix that upgrade and we're in the process of resolving this so that the November release will again allow for 17.8. We'll need to fix the breaking changes page at that time.
Note that even though we allow 8.0.4xx to be used with 17.8, we cannot guarantee that everything will work. The CodeStyle analyzer for example will not work in that configuration unless you set an additional property BuildWithNetFrameworkHostedCompiler set to true which is not well known. That property is on by default in net9 fwiw.
https://github.com/dotnet/sdk/pull/43995 will fix the min version

@marcpopMSFT Regarding the download page: I was looking at these two pieces of information:

The Included in value seems pretty clear to me as to what it means. Are you saying that Visual Studio support is also supposed to be the version it is included in? If so, what is the reason for having what seems to be the same information twice?

Ahh, the Included in section doesn't exist for the 8.0.10 release and I didn't expand the page. I think those basically mean the same thing though I'd need someone more familiar with the intent on the download page to comment. @gewarren do you know why Included in isn't there for 8.0.10 and do you know what the intent was for that versus the VS support node?

Sorry for the delay on this. Tagging @mairaw for the downloads page question here.

Tagging @rbhanda since this data is driven from the releases.json files on dotnet/core. Rahul, did we miss any VS tagging for the latest runtime release? I'd imagine that one of those versions is included in VS.

https://github.com/dotnet/core/pull/9558 to fix this issue

I think there's still an outstanding question about the difference between Included in and Visual Studio support.


Cydarm: New App - v1.0.0

Please ensure your pull request (PR) adheres to the following guidelines:

Please refer to our contributing documentation for any questions on submitting a pull request, link: Contribution Guide

Pull Request Checklist
Please check if your PR fulfills the following requirements:

[ ] Testing of all the changes has been performed (for bug fixes / features)
[ ] The manual_readme_content.md has been reviewed and added / updated if needed (for bug fixes / features)
[X] Use the following format for the PR description: <App Name>: <PR Type> - <PR Description>
[X] Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.
[ ] Verify all checks are passing.
[X] Do NOT use the next branch of the forked repo. Create separate feature branch for raising the PR.
[ ] Do NOT submit updates to dependencies unless it fixes an issue.

Pull Request Type
Please check the type of change your PR introduces:

[X] New App
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation
[ ] Other (please describe):

Security Considerations (REQUIRED)

[ ] If you are exposing any endpoints using a REST handler,
please document them in the manual_readme_content.md.
[X] If this is a new connector or you are adding new actions

[X] Please document in the manual_readme_content.md all methods (eg, OAuth) used to authenticate
with the service that the connector is integrating with.
[ ] If any actions are unable to run on SOAR Cloud, please document this in the manual_readme_content.md.

[ ] Are you introducing any new cryptography modules? If yes, please elaborate their purpose:
[ ] Are you are accessing the file system? If yes, please verify that you are only accessing paths returned through
the Vault API.
[ ] Are you are marking code to be ignored by Semgrep with nosemgrep?
If yes, please provide justification in an additional comment next to the ignored code.

Release Notes (REQUIRED)
First release for Cydarm app (version 1.0.0)
Includes actions:

test connectivity
get case
get case quick search
get cases filtered
create action instance data
create case data comment
create case
update case
update case history
create case playbook
get case playbook
get case playbooks
get playbook action
create playbook action
create playbook
add action to playbook
get acl
get user
add watcher to case
add member to case
add case tag
delete case tag

What is the current behavior? (OPTIONAL)

Describe the current behavior that you are modifying.

What is the new behavior? (OPTIONAL)

Describe the behavior or changes that are being added by this PR.

Other information (OPTIONAL)

Any other information that is important to this PR such as screenshots of how the component looks before and after the change.

Pay close attention to (OPTIONAL)

Any specific code change or test case points which must be addressed/reviewed at the time of GA release.

Screenshots (if relevant)

Thanks for contributing!

Hi team,
Need help with following:

Add my app's ID to https://cd.splunkdev.com/phantom/app_release/-/blob/main/data/repo_name_to_appid.json
Add following action names to https://cd.splunkdev.com/phantom/app_release/-/blob/main/data/approved_verbose_action_names.json:

get case quick search
create action instance data
create case data comment
add action to playbook
add watcher to case
add member to case

Hi team, Need help with following:

Add my app's ID to https://cd.splunkdev.com/phantom/app_release/-/blob/main/data/repo_name_to_appid.json

~2. Add following action names to: https://cd.splunkdev.com/phantom/app_release/-/blob/main/data/approved_verbose_action_names.json:~

get case quick search
create action instance data
create case data comment
add action to playbook
add watcher to case
add member to case

I've updated the action names to be max 3 words.

@bliew-splunk
Due to some of the dependencies of dev-requirement.txt prodsec-scans pipeline was failing
so, I have renamed the file to ignore the file
We have 2 more options to cater

remove the dependencies (i.e. certifi, urllib3, pyclean) that are causing the issue
remove the file

We have 2 more options to cater

Thanks @mishalp-crest - I've deleted the file since it's not really necessary

@bliew-splunk, Do we need .sh files for our app?
If not, can we remove it?

@bliew-splunk, Do we need .sh files for our app? If not, can we remove it?

The .sh files are only required for development. Are they causing an issue right now? I'm happy to remove them if needed.

@bliew-splunk, Do we need .sh files for our app? If not, can we remove it?

The .sh files are only required for development. Are they causing an issue right now? I'm happy to remove them if needed.

Not causing any issue  but as it is not related to app. let's remove it to make the app folder clutter free


Add isEditingFile to fork-source-link-same-view

LINKED ISSUES:
https://github.com/sindresorhus/refined-github/pull/3230#issuecomment-671308585

TEST URLS:
https://github.com/yakov116/refined-github/edit/upstream/package.json

SCREENSHOT:
None

Sorry about

Send it just to blob?
Originally posted by @yakov116 in https://github.com/sindresorhus/refined-github/pull/3230#issuecomment-671309723

I made a mistake and thought that doesFileExist looks at route. (For the second time!)

For the second time!

you can't get fooled again

Commit and PR titles should not describe the code, but what they do