Allow to select the source projectId when using secret

I would like to load easily secrets from more than 1 project. Today, only the default project, or the project set in the properties can be easily used by the SecretManagerTemplate bean
Describe the solution you'd like
The capability to create and read secrets from default project (like today) but also to have the capability to set a projectId for reading the secrets in another project
This implies changes into SecretManagerTemplate and into the bootstrap of secret. On the bootstrap, a solution can be to set the project id in the same way as the versions are defined today.
spring.cloud.gcp.secretmanager.projectId.<secretId>
Open to discuss about the best implementation.
Describe alternatives you've considered
N/A
Additional context
N/A

Thanks for your PR.

On the bootstrap, a solution can be to set the project id in the same way as the versions are defined today.
spring.cloud.gcp.secretmanager.projectId.<secretId>

Yeah I like this idea a lot; would be in favor of letting users specify a custom project id.

I created a new PR for bootstrap: #2289
Integration test relevance to discuss. TODOs in code where updates are required.
Open to discuss.

Regarding property source loading, how would you handle the case where if two projects had 2 secrets with the same secret ID?

Hey I took a look at the PR, my bad; so my comments about name collisions are incorrect; this is like specifying the project-id similar to specifying a version over the default.
Could you describe your use-case a bit more? Just want to learn more about it; right now I guess my only concern is that the feature increases the complexity of the settings, and then also there might be unexpected interactions with if a user also sets the version override setting for the specific secret as well.

In my context, and for cost reason, we have 1 shared project for test and dev environment with 1 Cloud SQL MySQL and 1 Cloud SQL PostGreSQL database. We chose to store the passwords in the secret manager of this shared project.
Then, the developer and the other projects can use this database by using the password stored in SecretManager, in addition of other secret stored in their project (like API Keys).
But, you have right. If I have the "password" secret name in my default project and in the shared project, how can I specify both ?
The best way is to wrap the projectId and the secretId in the same key, something like projectId/secretId. If projectId/ is missing, we use default project. Like this, it's compatible with the existing.
But I never saw / in properties name. And, I you said, it's become to be quite complex to define the properties and to understand them.
The other solution is to do nothing. The bootstrap configuration work only for secret inside the current project. If you want to load secret from other project, use template (with code example for helping the user to achieve this at the start phase).

Hmm I see, yeah it makes sense to support this use-case. Let me circle back with our team to discuss more how to best do this; there are some design things I think we need to work out.
I see what you're saying with the projectId/secretId key; we can let users load secrets from other projects and then load them into the environment with property key:
.
But we have a couple other settings like choosing the secret version and the secret-name-prefix where I feel users might get confused how these interact with that.
Thanks for your PR again; I will put your bootstrap properties PR on hold for now until we figure it out.

Great. Let me know what is the preferred design. No problem for working on it on my side.

Hey so I have some ideas for supporting this, I think I will need to rework it quite a bit though. I will implement this in the next few days if my team agrees.
In order to support loading secrets from multiple projects, my ideas are to:

Currently the code will load all the secrets in your project as properties eagerly; however I think we need to move to a lazy model of loading secrets if we want to support allowing loading secrets from multiple projects.

I think we shall invent some kind of "fully-qualified-name" syntax to let you load secrets from multiple projects. It might be of the form:
{SECRET_NAME_PREFIX}{PROJECT_ID}.{SECRET_ID}.{VERSION_NAME}

So you can specify secrets like: @Value("${secret-prefix.my-gcp-project.my-secret.latest}")

Probably with this, we will remove our existing way of specifying the version of a secret to load in the properties.

Hey.

I don't know what imply the lazy load (in code) but if you can load secret not at start up but later, it could help cold start duration (and Spring is really not good in it!). But it's a nice to have.
However, I don't catch what is the relation between the multi-project secret and the lazy loading.

You are right for the fully qualified name. But the lazy convenience to omit the current project and the latest version is pleasant. I don't know if it's possible to mix the 2 capabilities.  Is something like this is possible?
// Your definition
@Value("${secret-prefix.my-gcp-project.my-secret.latest}")

// On current project and latest version, omit the values my-gcp-project and latest
@Value("${secret-prefix..my-secret.}")

If so, and if it enforces the Spring best practice, I upvote!

And yes, this evolution isn't backward compatible.

Thanks for your feedback Guillaume. We recently had a discussion with the Secret Manager team that gave us some guidance as well. Based on what you said and their feedback I think we may converge on the following solution:
So we are planning for users to be able to specify secrets in their application.properties like this:
application.properties
# Long form
spring.property=${secret-prefix:projects/<project-id>/secrets/<secret-id>/versions/<version-id>}

# Long form - "latest" version
spring.property=${secret-prefix:projects/<project-id>/secrets/<secret-id>}

# Short form
spring.property=${secret-prefix:<project-id>/<secret-id>/<version-id>}

# Short form - "latest" version
spring.property=${secret-prefix:<project-id>/<secret-id>}

# Shortest form - "latest" version and default project.
spring.property=${secret-prefix:<secret-id>}

Following the model of Secret Manager on Github Actions:
https://github.com/GoogleCloudPlatform/github-actions/tree/master/get-secretmanager-secrets#inputs
This at least is our current idea; may not be exactly what we ultimately produce but hopefully it gives you a sense.
To get this working, we are also semi-blocked on an in issue in Spring-cloud commons here: https://github.com/spring-cloud/spring-cloud-commons/issues/724

Great feedback!
Is it possible to "reserve" the project_id "default" for specifying the current project? The idea is the following:
# Short form
spring.property=${secret-prefix:default/<secret-id>/<version-id>}

Like this, you can specify the version-id without locking the project-id name.
Only the account that owns the default project will be limited with this feature
Another solution is to reserve the 2 keywords to the secret-id and version-id on the current project:
# Short form - "default" project
spring.property=${secret-prefix:<secret-id>/<version-id>}

From my usage, I use about 90% of time the project of the current project and more exceptionally the external projects. But, it's only my usage!

I see, good point. Yeah, since I also foresee most people working within the default project, it makes sense to adopt one of your two suggestions. Will make a note of this.

Hey @guillaumeblaquiere, we recently submitted #2302 which should offer an easier way of specifying secrets across projects. It introduces the new syntax as we discussed, full docs here: https://github.com/spring-cloud/spring-cloud-gcp/blob/a81143bbd26db0ae7760e8d9aa3147565232f034/docs/src/main/asciidoc/secretmanager.adoc#secret-manager-property-source
And should be available in the 1.2.3.BUILD-SNAPSHOT if you want to try it out.
Going to close but let me know if there's any remaining issues.


example put, trying to replicate issue 893

Changes Unknown when pulling 91f3d74d85089969e854e7aefdb6017918a43220 on carlo-rtr:example_put into ** on dropwizard:master**.

The original author of issue #893 reused the Response object which doesn't work with JAX-RS (as the underlying OutputStream can only be consumed once).


ci broken

https://github.com/docker/distribution/pull/2401 -> https://circleci.com/gh/docker/distribution/5350?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
https://github.com/docker/distribution/pull/2402 -> https://circleci.com/gh/docker/distribution/5352?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

Thanks for the report.


generate_temp_url for OpenStack and other providers

Currently, only AWS has a generate_temp_url implementation. Both GCE and Azure have similar capabilities for generating temp_urls, so that should eventually come along.
OpenStack however, is somewhat different.
http://torgomatic.us/blog/2013/05/08/an-introduction-to-tempurl-in-openstack-swift/
Basically, to use temp_urls in OpenStack, you must first generate a secret key that is associated with your OpenStack account. Without this secret key, one cannot generate a temp url.
This however, means that each user must first associate a temp url with their account before they can start generating urls. Now, we could try to auto-generate this secret key, but the problem is that, you can only have one secret key associated with your account, so each time you associate a new secret key, all previous temp_urls for that account are invalidated. So if two different apps use two different keys, they risk cancelling each other out.
We could ask that users manually provide their secret key when initializing the provider, but that means that they must always provide that same secret key to every app that uses cloudbridge, something that is again rather unnatural. So the only portable implementation that comes to mind is:

Use a hash of the user's password as the secret key
CloudBridge will auto associate that hash with the user's account automatically each time you generate a temp_url.
If two different cloudbridge apps generate temp urls, it will work, because the hashes will be the same.
If a different application, associates its own secret with the account, all previous temp urls will be invalidated. But cloudbridge will re-associate a new hash, so it's ok for cloudbridge, but not so much for that other app.
We could also provide a facility to provide the secret key as a provider setting. If it's not provided, we can auto generate as above.

@afgane @VJalili Sound ok? Any security holes or problems that you can foresee with this approach?

Looks like I've made a mistake. I forgot that a swift account corresponds to a project and not a user account. That invalidates that previous proposal and makes that issue far more difficult

So this seems to have stemmed from a misunderstanding on my part. It turns out that the "secret key" can actually be retrieved at will.
http://chianingwang.blogspot.com/2015/09/how-to-generate-tempurl-for-swift.html
I guess this solves the problem. We can simply check for the existence of a secret key, and if it doesn't exist, generate a new one.

I guess it would be better to not deal with secret (and/or access) key at cloudbridge level, so cloudbridge can rely on access and secret key provided to it by the user.


How can I leave a reference for my hoster

Subject: How can I leave a reference for my hoster
Description:
can not find any buton to write a review
Results:
Send me a link or tell me the step to wirte a review for my hoster.
Backend version: develop-fef052b8
Frontend version: develop-fef052b8
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Page: https://couchers.org/preview
User: yizhuo (7954)

We do send links.

Dupe of #5160


"preventClicks" causes unclickable links on slides in Chrome

Check that this is really a bug

[X] I confirm

Reproduction link
https://crm.itdesign.de/
Bug description
We encountered that the preventClicks functionality causes unclickable links on slides. This bug occured only in Chrome, not f.e. in Firefox. That is why we think it is a bug. It appeared mostly on Windows devices and couldn't be reproduced in Ubuntu Chrome for example.
We propose that the issue will be either fixed or the preventClicks / preventClicksPropagation parameters are set to false by default since this can break things and I think it should be a nice-to-have feature but not the default behaviour.
Please note: We are currently setting the preventClicks parameters to false so the Reproduction link won't be showing the issue for long from now on. But it's easy to reproduce I guess.
Expected Behavior
Click on a link opens the target URL.
Actual Behavior
Nothing is happening. It seems the link behaviour is prevented by JavaScript (e.preventDefault).
Swiper version
9.0.5
Platform/Target and Browser Versions
Windows / Chrome 110
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

I can't see such issue, all works correctly in Chrome for me. Plus, the link you have provided doesn't illustrate the issue


Correcting date in the cert creation example

@grumpydev-net : Thanks for your contribution to the Azure documentation! The author, @adhurwit, has been notified to review your proposed change.

#sign-off

@lleonard-msft: I'm sorry - only the author of this article, @adhurwit, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


HDDS-5949. EC: Create reusable buffer pool shared by all EC input and output streams

What changes were proposed in this pull request?
When reading and writing EC, we often have to allocate chunk sized Bytebuffers. It would be good if we could reuse them across multiple input / output instances to save freeing and re-allocating memory.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5949
How was this patch tested?
Existing tests

Thanks @sodonnel for working on this. Latest changes looks good to me. Pending CI !

Pushing a new commit to fix the conflicts.

Looks like there is some issue in conflict merging: We need to remove this line: https://github.com/apache/ozone/blob/e9e0b1071633982f6b2cb5169dc2d15928e11e47/hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/io/ECKeyOutputStream.java#L263
We have removed this line as part of HDDS-6132 and test added in that JIRA is failing here as that line is still present.
Please check that change once and that should get green CI.


Fix to allow specifying restler_multipart_formdata directly in the grammar

Using the following in the dictionary and grammar, with this change, restler_multipart_formdata can be used to send a file as the body of the corresponding
POST request:
    # Remove this next Content-Type line
    #primitives.restler_static_string("Content-Type: application/json\r\n"),
    primitives.restler_refreshable_authentication_token("authentication_token_tag"),
    primitives.restler_static_string("\r\n"), # Make sure this is here
    primitives.restler_multipart_formdata("file1"),
    primitives.restler_static_string("\r\n"),

Dictionary:
  "restler_multipart_formdata": {
    "file1": [
      {
        "content-disposition": "name=\"file1\"; filename=\"test0.txt\"",
        "datastream": "c:\\temp\\restler\\test0.txt"
      },
      {
          "content-disposition": "name=\"file2\"; filename=\"test1.txt\"",
          "datastream": "c:\\temp\\restler\\test1.txt"
      }
    ]
  }

I have the save 2nd issue as @ijard-akshay


Create EIA entity tables

As a part of the transform process, once we have all of the dataframes for the data tables created and cleaned, we can create the entity tables by compiling all instances of each entity. The entities are (at least) plants, generators, boilers, and utilities. These tables can be used to compile information about these entities that can be pulled out of the more data-like tables (this especially needs to be done for 860).

I think moving those entity-like pieces of info into the entity tables early on totally makes sense.  I almost turned the prime mover codes / descriptions into a list of descriptive codes like with many of the other little glue tables, and I think that's the right thing to do, but I wasn't in a place to be mucking with the transform step at that point, so I only did the simplest re-org of those tables.

I'm glad we're on the same page... because I've mostly already finished with this!
On making this work for the outputs, I'm thinking it might be nice to either a) consolidate all of the EIA output functions into one module, or b) create another EIA module. I'm leaning towards to former bc I've removed so much of the content from many of the 860 tables in this harvesting process, including several full tables.

I've come across a minor issue with the location in the plants_entity_eia table: the (lon, lat) for Clayville in New Jersery (plant ID 58235) is  (-39.424444, 39.424444). The actual location is (-75.0201, 39.4245). There could be other plants with issues like this; I only found this one because the point doesn't have a timezone in timezonefinder.
I'm not sure what the easiest way would be to add tests for the location. Maybe verifying that the lat/lon is in the state we expect?

You could certainly check each plant to see if they are in the state in question (sample code). The next step would be assigning a timezone based on the state/county, which would require a pre-populated list of timezones for each state/county.
At that point, do we just assume that state/county info for each power plant is correct and merge the timezone in using the pre-populated list?

@karldw That seems like maybe a data entry error on the part of the utility.  Is it like that for all of the years, or just one?  @cmgosnell is close to finished with the normalization of EIA 860, and I think her process ensures uniform values in those fields...

@zaneselvans, definitely a data entry error. Looking through the excel sheets, it seems like it was reported erroneously in 2012 and correctly after that.
@cmgosnell, I think this is a small bug in the consistency normalization. Here's some demo code:
import pandas as pd
import pudl
# Testing note: this df needs at least 4 rows for the 70% consistency cutoff to matter
df = pd.DataFrame({
    "plant_id_eia": [1, 1, 1, 1],
    "report_date": [2011, 2012, 2013, 2014],
    "latitude": [2, -1, 2, 2]
})
clean_df = df.head(0)
plants_df = pd.DataFrame({"plant_id_eia": [1]})
pudl.transform.eia._lat_long(df, clean_df, plants_df, "latitude")
# This one works like it should:
#>    plant_id_eia  latitude latitude_consistent  occurances  report_date
#> 0             1         2                True         4.0         2011

# But this one doesn't, only changing the order of latitude:
df = pd.DataFrame({
    "plant_id_eia": [1, 1, 1, 1],
    "report_date": [2011, 2012, 2013, 2014],
    "latitude": [-1, 2, 2, 2]
})
pudl.transform.eia._lat_long(df, clean_df, plants_df, "latitude")
#>    plant_id_eia  latitude latitude_consistent  occurances  report_date
#> 0             1        -1                   1         4.0         2011

And then because python views 1 == True as true, latitude_consistent gets treated as a true value and -1 is then treated as the correct, consistent latitude.
@cmgosnell and @zaneselvans, I can turn the code above into a test if you'd like. Should it go in etl_tests.py or somewhere else?
@gschivley, I made a mapping of states to timezones as part of my PR, but I'm not certain that was a good idea, since many states have multiple timezones. (A few counties do too.) I think it's worth getting the latitude/longitude right for a bunch of applications, not just timezones, and if we have lat/long, using timezonefinder avoids these edge cases. Thanks for the demo code!


Test WindowEvaluatorFactory on WindowedValues in multiple windows

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

LGTM. Will merge and rebase #308 so I can use this as a reference for adjusting the runner.


Crash when placing Crawdad on Crawdad ghost

Mod source
Factorio Mod Portal
Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[ ] Graphical
[X] Crash
[ ] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
Game completely crashes when placing a Crawdad on a Crawdad ghost (by hand, not bots).
Steps to reproduce

Place Crawdad ghost (I did it by placing a normal Crawdad down, pipette tool picking up a ghost and placing that somewhere else, then picking up the Crawdad again).
Pick up Crawdad from inventory and place it on the ghost

Additional context
Currently playing without WiFi (using 4G to make bug report). Can/will send crash report file later when I have internet to my PC again
Log file
No response

factorio-current.log
Same Issue - Here's the crash report


Ocelot supports web sockets.  What functionality is missing?

Ocelot supports web sockets.  What functionality is missing?

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

ID: 10ce8eaa-b3ff-9b69-fea1-131eed36b9ec
Version Independent ID: 771d9f72-b506-79ef-5398-030c1705ee9a
Content: Implementing API Gateways with Ocelot
Content Source: docs/architecture/microservices/multi-container-microservice-net-applications/implement-api-gateways-with-ocelot.md
Product: dotnet-architecture
Technology: microservices
GitHub Login: @nishanil
Microsoft Alias: nanil

Hey @bit7 , thank you for reaching out and reporting this.
As Ocelot is no longer maintained, we moved away from using it and we are currently using Envoy proxy for the redirections. Will be updating the content of docs for API Gateways accordingly.
I am closing this issue as of now, feel free to track the updation of docs in a similar issue https://github.com/dotnet/docs/issues/27445


fix the repository link

Hi there!  It looks like this package was moved to this repo before 0.1.3 was released.  This PR updates package.json so that at least the next release will have a link to the proper repo.  I've also filed https://github.com/paulja/memory-streams-js/pull/20 to mark the old repo as unmaintained.
All of this will help anyone finding this package to find its source and determine whether or not it is malicious.

Hi there.  I'm afraid you're mistaken.  This repo is a fork that we created because we wanted to make a slight alteration to the library after we had adopted it for our project.  I'm afraid we're not in communication with @paulja and have no access to the relevant npm account to publish updates.
The repository that this is forked from should continue to be considered the official repo for the npm package.

Hey @mike-lang I'm no longer working in the Node space, and have not updated this project (or any other Node) project for many years. I would gladly pass this project over to you if you wish to maintain it?

Thanks for the response @paulja.  I'll have to check with my boss to see if we here at thirdiron can take over public responsibility for the library.  It'll be a couple of days before we can sort that out.  I'll let you know

Thanks, both -- in the end, it'd be great to be able to find the revision from which 0.1.3 was released be easily found by someone using the package.


Block third-party cookies issue

Hi, Thanks for this awesome package.
Expected Behavior
Shopify app work properly even when third party coockies are block .https://prnt.sc/t7f9jw
Current Behavior
App not working properly, app is going for login page continously.
Failure Logs
It appear in chrome latest 83.0.4103.116 version only

Package Version: 11.4
Laravel Version: 7.0
PHP Version: v7.3.13
OS: Linux Mint 19.2

'secure' => env('SESSION_SECURE_COOKIE', true),
'same_site' => 'none',
I also set above configurations, still it not working.

This package does not have support to ask for the test cookie. Its been on my radar but something I havent had time to dig into as it requires a lot of flow rework and honestly browsers are becoming a pain to constantly support for.
I am attempting to begin some code next week to look into this but unfourtunately this work will take a nice while.. closing for now until I can add support for this.


8.8.0 regression when using custom locale

Describe the bug
I am not using Next.js i18n feature because it doesn't work for my case. So I pass the locale in getStaticProps like that:
const locale = sanitizeLanguage(context.params.uri[0]);

return {
  props: {
    ...(await serverSideTranslations(locale)),
    locale,
    ....
  },
}

It was working fine before because next-i18next is using the initial language parameter passed to serverSideTranslations to initialize i18next. But now, you are using the locale from the router (const { locale } = props.router) which is undefined in my case, because I don't use the official i18n config of Next.js.
Occurs in next-i18next version
This commit cause the issue: https://github.com/isaachinman/next-i18next/commit/1f981771427c2142c65d3fd914857482c16a596c, specifically in the src/appWithTranslation.tsx file.
Steps to reproduce
Call serverSideTranslations with a language without using Next.js i18n implementation.
Expected behaviour
It should work as before, initializing i18next with the right locale without having to use Next.js i18n implementation.
OS (please complete the following information)

Device: Windows 10 Pro
Browser: Chrome 93.0.4577.82

next-i18next relies on the NextJs i18n functionality. Unfortunately your use case is not supported. I would suggest you pin to a previous version, and raise your use case to the NextJs team so that it might be supported first-class.

We have similar issue. We have implemented Next.js i18n but initialLocale in serverSideTranslations function does not work in 8.8.0 version. We need to change language by initialLocale value as in 8.7.0 for each page.
Can you resolve this?


Dubbo-2013

What is the purpose of the change
Let user can config the threadpool of consumer !
Brief changelog
add a tag "threadpool" to dubbo:consumer in dubbo.xsd
add a property "threadpool" to ConsumerConfig class
modify Client code to get the threadpool by user's config

Would you please submit a patch for this PR according to the review comment now? I would suggest that we add this enhancement to V2.6.3,  this version is discussing in the community and is about to release soon.

Codecov Report

Merging #2114 into master will increase coverage by <.01%.
The diff coverage is 52.94%.

@@             Coverage Diff             @@
##             master   #2114      +/-   ##
===========================================
+ Coverage     54.19%   54.2%   +<.01%
- Complexity     5083    5086       +3
===========================================
  Files           559     559
  Lines         24908   24922      +14
  Branches       4438    4442       +4
===========================================
+ Hits          13499   13508       +9
- Misses         9377    9379       +2
- Partials       2032    2035       +3

Impacted Files
Coverage Δ
Complexity Δ

...src/main/java/org/apache/dubbo/common/Version.java
48.23% <52.94%> (-1.07%)
15 <2> (+2)

...apache/dubbo/rpc/protocol/dubbo/FutureAdapter.java
58.06% <0%> (-6.46%)
3% <0%> (ø)

...he/dubbo/remoting/transport/netty/NettyClient.java
81.35% <0%> (+8.47%)
13% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Migrate custom-checks tests to JUnit 5

Migrate custom-checks tests to JUnit 5.
See #4450

@3flex did you migrate the whole test package manually? Did you use some kind of transformation script to ease the migration? I think for some cases this might work.

did you migrate the whole test package manually?

Yes, manually. A scripted approach would be great. I have tried a series of regex find/replace rules for the various parts which will definitely help (migrating specs like it(...) to @Test fun ...() is simple) but there are other parts that are more difficult. context and describe can't be easily converted to Nested JUnit tests because the context & describe descriptions are typically sentences which can't be used as class names without triggering class naming detekt rules (though that could be turned off for test sources).

I just created a script that ports nearly all the tests of detekt-core (I didn't try any other module):
#!/usr/bin/env bash

replace_all() {
  find detekt-core/src/test/ -name "*Spec.kt" -type f -print0 | xargs -0 gsed -i -E "$1"
}

replace_all '/ (it|test|context|describe)\(.*\)/s/[.:`>]/_/g'
replace_all 's/(object|class) (.*) : Spek\(\{/class \2 {/g'
replace_all 's/^class (.*)Spec/internal class \1Spec/g'
replace_all 's/class (.*)Spec/class \1Test/g'
replace_all 's/^\}\)$/}/g'
replace_all 's/describe\("(.*)"\)/@Nested inner class `\1`/g'
replace_all 's/context\("(.*)"\)/@Nested inner class `\1`/g'
replace_all 's/it\("(.*)"\)/@Test fun `\1`()/g'
replace_all 's/test\("(.*)"\)/@Test fun `\1`()/g'
replace_all 's/beforeEachTest \{/@BeforeEach fun setUp() {/g'
replace_all 's/afterEachTest \{/@AfterEach fun tearDown() {/g'
replace_all '/^package /a \\nimport org.junit.jupiter.api.Nested\nimport org.junit.jupiter.api.Test\nimport org.junit.jupiter.api.BeforeEach\nimport org.junit.jupiter.api.AfterEach'
replace_all '/^import org.spekframework.spek2/d'

I know that it's far from optimal but it does the work and it's kind of easy to work with.
The biggest problem to port are the memoized. I used the "Replace structurally..." option that intellij give us using this two patterns:
val $val$ by memoized {
    $code$
}

private val $val$ = $code$

I couldn't find a way to make it with a script so these part need some "clicks".
And the last thing to note here is @TestInstance(Lifecycle.PER_CLASS). I don't like the idea to add that annotation to al our tests. It's way easy to forget about it. I found this that we could pass this argument when running our tests -Djunit.jupiter.testinstance.lifecycle.default=per_class and that will make all the tests PER_CLASS by default. And, if we want to use PER_METHOD we can do it using @TestInstance(Lifecycle.PER_METHOD).

I just created a script that ports nearly all the tests of detekt-core

Great job. I ran it on detekt-rules-empty and it seemed to do a good enough job to reduce the monkey work to a minimum.
I can definitely help transforming tests to junit. Should we split up by submodule? I would hate to see duplicate work on this.

I found that we could pass this argument when running our tests -Djunit.jupiter.testinstance.lifecycle.default=per_class and that will make all the tests PER_CLASS by default. And, if we want to use PER_METHOD we can do it using @TestInstance(Lifecycle.PER_METHOD). But I'm don't know how to configure it with gradle.

I agree, we should use PER_CLASS as the default. In build-logic/src/main/kotlin/module.gradle.kts we can add this to the task config.
tasks.withType<Test>().configureEach {
    useJUnitPlatform()
    systemProperty("junit.jupiter.testinstance.lifecycle.default","per_class")
    // ...
}

Thanks all - to close this one out I will:

Switch to the extension provided by marschwar (but I'll skip the subclass, I think the extension is easy enough to use on its own)
Apply the system property junit.jupiter.testinstance.lifecycle.default=per_class

Anything else? Any concerns?
Once that's done and this is merged @BraisGabin could adjust his script. Then we can setup a GitHub project to track the work to migrate each module, so as marschwar pointed out we can avoid duplication of effort.


The char spacing options does not work when text is selected

Won't fix. This is how it work in the original version.


feat: gpt-4o supports reading images

Supported image reading with the GPT-4o model.

Hey, I've glanced through the code — this is a great implementation. I'm currently going through a refactor of how Messages work internally to make multi-modal a bit cleaner. This would fare better if it sat on top of that implementation, so I'll try my best to get that PR wrapped up (holiday season, so availability is limited). I'll ping you when I get that PR out!


GPL-nnn Support Heron Lysed TubeRack purposes from SS

User story
As SRA for CGaP (Laura) I want to use Extraction Lims to accept the plate purpose Heron Lysed TubeRack for tube racks as an input from Sequencescape.
Who the primary contacts are for this story
Laura (CGaP)
Acceptance criteria
To be considered successful the solution must allow:
The user can scan the barcode from a Heron Lysed TubeRack rack
The rack is imported into Extraction lims
Additional context
As today 20th May, this story affects the plate purpose created at story GPL-478.

Tasks

[ ] Depending on result from GPL-476 (precondition) create a new TubeRack plate purpose in Sequencescape named Heron Lysed TubeRack with size 96.
[ ] Add a test to check everything behaves as expected in Extraction.


Feature (idempotency): write end-to-end tests for the utility

Description of the feature request
Problem statement
Before a Powertools utility can be considered production ready it needs to have e2e tests implemented and running every time a new feature or release is added.
Summary of the feature
This action item tracks the activities and changes needed to integrate the new utility in the e2e tests, which high level and at the time of writing consist of:

Write the e2e tests cases and corresponding functions
Optionally implement any helper needed
Add the new utility in the workflow that runs the tests  (related to #1097)

Code examples
N/A
Benefits for you and the wider AWS community
Confidence that the utility has been tested in a real AWS Lambda execution environment.
Describe alternatives you've considered
N/A
Additional context
All utilities have extensive end to end test cases that aim at testing the different features and usages in a real AWS environment with real resources, permissions, and runtimes.
Related issues, RFCs
#447

Waiting for the implementation to be completed first.


Allow manual padding disable

As specified by RFC4648, the padding symbol is not mandatory.
I am currently using your module and I need to disable the padding, in order to be coherent among other languages.
Sources:
https://godoc.org/encoding/base64#pkg-variables
https://github.com/golang/go/blob/244c98fe01b2af844f6f41b7123a17b7b56f1c2a/src/encoding/base64/base64.go#L105

Sounds reasonable 👍
Any ideas for option name? Maybe { disablePadding: true }, or just { padding: false }... or { stripPadding: true } 🤔

At first, when I created this PR I was thinking about { padding: false }, but I prefer padding to be the variable that you already use. So, I was thinking on something more similar to what is being called in GO, such as { rawEncoding: true }. What do you think?
I can create the PR later or tomorrow with this 😄

How about something like this?
-module.exports = function base32Encode (buffer, variant) {
+module.exports = function base32Encode (buffer, variant, options) {
+  options = options || {}
+
-  var alphabet, padding
+  var alphabet, defaultPadding

   switch (variant) {
     case 'RFC3548':
     case 'RFC4648':
       alphabet = RFC4648
-      padding = true
+      defaultPadding = true
       break
     case 'RFC4648-HEX':
       alphabet = RFC4648_HEX
-      padding = true
+      defaultPadding = true
       break
     case 'Crockford':
       alphabet = CROCKFORD
-      padding = false
+      defaultPadding = false
       break
     default:
       throw new Error('Unknown base32 variant: ' + variant)
   }

+  var padding = (options.padding === undefined ? defaultPadding : options.padding)
   var length = buffer.byteLength
   var view = new Uint8Array(buffer)

Would love a PR 🙌

SGTM! I will try to do it later


Training code

Hi
Really glad to see this is available: I've been keen to give it a go since the paper appeared on arxiv. I've had some success previously training useful models with PhysNet (as yet unpublished work) and this looks even better.
I cloned this repository and checked I could get the "example.py" code to run, and it did, so it seems I have a python environment that can cope with SpookyNet.
Perhaps I am missing something but I don't see, at the moment, any code here relating to training, eg definition of the loss function, loading the training data, or optimising the parameters with AMSGrad. In PhysNet you had a train.py which was straightforward to use.
Is that something you'll be adding at some point? Quite difficult to work out how to use this without it.
Thanks
Nick Hine
University of Warwick

@OUnke just wondering if you have any plans in regard to releasing training code?

Hi Nick,
I recently went through the same process to get this training on a custom network.  If you look at example.py, the author shows how to encode your molecules to generate energies etc (this is mostly in spookynet_calculator.py, which you don't need to directly use but can learn from) .  Training is basically the same thing.. you want to put your molecules in a batch via this encoding, generate the energies, and then compare those energies vs your known values to compute a loss function.  Here's some rough code to give you an idea.  It's pruned from a more complicated project of mine so it probably has a bunch of errors, and obviously you want to do things like store the current state so you can pause training etc, but that's the same for all of these projects.  There are also a bunch of SpookyNet specific options if you want to incorporate charges or dipoles into your loss function-- the paper explains this.  I'm pretty impressed with this model so far, at least compared to others I've experimented with such as ANI and PhysNet (which is the same person but this code is more modern).   Hopefully it helps.  Good luck.
class SpookyBatch:
    device = torch.device('cuda')

    def __init__(self):
        self.N = 0
        self.Z = []
        self.R = []
        self.E = []
        self.idx_i = []
        self.idx_j = []
        self.batch_seg = []

    def toTensor(self):
        self.Z = torch.tensor(self.Z,dtype=torch.int64,device=SpookyBatch.device)
        self.R = torch.tensor(self.R,dtype=torch.float32,device=SpookyBatch.device,requires_grad=True)
        self.Q = torch.zeros(self.N,dtype=torch.float32,device=SpookyBatch.device) # not using this so could just pass the same tensor around
        self.S = torch.zeros(self.N,dtype=torch.float32,device=SpookyBatch.device) # ditto
        self.E = torch.tensor(self.E,dtype=torch.float32,device=SpookyBatch.device)
        self.idx_i = torch.tensor(self.idx_i,dtype=torch.int64,device=SpookyBatch.device)
        self.idx_j = torch.tensor(self.idx_j,dtype=torch.int64,device=SpookyBatch.device)
        self.batch_seg = torch.tensor(self.batch_seg,dtype=torch.int64,device=SpookyBatch.device) # int64 required for "index tensors"
        return self

def load_batches():
    batches = []
    batch = None
    NM = 100 # how many mols we put in each batch
    nm = 0 # how many in current batch
    for  m in my_mols: # some structure which stores your loaded mol data
        if nm == 0:
            na = 0 # num total atoms in this batch
            batch = SpookyBatch() # stores the data in a format we can pass to SpookyNet

         batch.N += 1
         batch.Z.extend(m.species)
         batch.R.extend(m.coords)
         batch.E.append(m.energy) # target energy
         cur_idx_i,cur_idx_j = get_idx(tm.coords) # see below but also look at SpookyNetCalculator for more options
         cur_idx_i += na
         cur_idx_j += na
         batch.idx_i.extend(cur_idx_i)
         batch.idx_j.extend(cur_idx_j)
         batch.batch_seg.extend([nm]*len(m.species))
         na += len(m.species)

         if nm >= NM:
             batches.append(batch.toTensor()) # or you could convert to a tensor during training, depends on how much memory you have
             nm = 0

    if batch:
        batches.append(batch.toTensor()
    return batches

# taken from SpookyNetCalculator
def get_idx(R):
    N = len(R)
    idx = torch.arange(N,dtype=torch.int64)
    idx_i = idx.view(-1, 1).expand(-1, N).reshape(-1)
    idx_j = idx.view(1, -1).expand(N, -1).reshape(-1)
    # exclude self-interactions
    nidx_i = idx_i[idx_i != idx_j]
    nidx_j = idx_j[idx_i != idx_j]
    return nidx_i.numpy(),nidx_j.numpy() # kind of dumb converting to numpy when we use torch later, but it fits our model

def train():
    NUM_EPOCHES = 1000
    BEST_POINT = 'best.pt'

    model = SpookyNet()
    model.train()

    optimizer = torch.optim.Adam(model.parameters(),lr=START_LR,amsgrad=True)
    scheduler = torch.optim.lr_scheduler.ReduceLROnPlateau(optimizer, factor=0.5, patience=25, threshold=0)

    training = load_batches(my_training_mols)
    validation = load_batches(my_validation_mols)
    mse_sum = torch.nn.MSELoss(reduction='sum')

    for epoch in range(NUM_EPOCHES):
        random.shuffle(training)

        for batch in tqdm.tqdm(
            training,
            total=len(training),
            desc="Epoch {}".format(scheduler.last_epoch)
        ):

            N = batch.N
            res = model.energy(Z=batch.Z,Q=batch.Q,S=batch.S,R=batch.R,idx_i=batch.idx_i,idx_j=batch.idx_j,batch_seg=batch.batch_seg,num_batch=N)
            E = res[0]

            loss = mse_sum(E, batch.E)/N

            optimizer.zero_grad()
            loss.backward()
            optimizer.step()

        rmse = compute_rmse(validation,model)
        if scheduler.is_better(validation_rmse, scheduler.best):
            model.save(BEST_POINT)

        scheduler.step(rmse)
        print('Epoch: {} / LR: {} / RMSE: {:.3f} / Best: {:.3f}'.format(scheduler.last_epoch,learning_rate,rmse,scheduler.best))

def compute_rmse(batches,model):
    mse_sum = torch.nn.MSELoss(reduction='sum')
    total_mse = 0.0
    count = 0
    model.eval()
    for batch in batches:
        N = batch.N
        res = model.energy(Z=batch.Z,Q=batch.Q,S=batch.S,R=batch.R,idx_i=batch.idx_i,idx_j=batch.idx_j,batch_seg=batch.batch_seg,num_batch=N)
        E = res[0]

        # sum over pairings
        total_mse += mse_sum(E, batch.E).item()
        count += N

    model.train()
    return ev2kj*math.sqrt(total_mse / count)

Hi shubbey,
Thank you for sharing your customized training code! I really appreciate it. Your code is very straightforward.
I have a few questions.
I know the code is an example code for training the data so, some features are missing but I just want to be confirmed a few very basic things:
{my_mols}: pass the extended xyz file which contains n many molecules. If that is so, I should add a few lines in which column(s) are coordination, species, energy, etc.
Is this correct?
Thank you
DGKang234

Hi shubbey, @shubbey
Thank you for sharing your customized training code!
Do you know that how this code can train the energy, force and dipole simultaneously (just like the eq28 in the author's paper). I guess that we can define a loss function including energy, force and dipole. However, I am not sure that during the training process, the electrostatic energy can be delete using the charge and vdw information automatically if I only just define the loss function with energy, force and dipole.

Hi @zhengcheng233,
Unfortunately I haven't played with training multiple parameters simultaneously so I can't really give you good advice here.  In my experience with these things, you usually can just define a generalized loss function as long as you put them on the same scale.  I guess you'll have to experiment to see what kind of results you get.  Sorry I can't be of more help.


bugfix/fix table in documentation for langchain-community

Description: This PR aims to fix the documentation for langchain-commnuity.

Issue: The table In this page : https://python.langchain.com/docs/integrations/llms/ is built based on old module.  The proposed fix is to modify the import from langchain-commnuity.

Dependencies: N/A

Twitter handle: N/A

Thanks @mingkang111 !


Set up CI with Azure Pipelines

This PR was apparently made automatically when Azure pipelines were set up for the repo (see #3). We should remove the dummy steps and add some actual CI in here before merging (again see #3).

Replaced by #13


Add Kotlin style guide

I would love to see your code style for Kotlin as a configuration file for IntelliJ

https://developer.android.com/kotlin/style-guide

refresh it

It's rated more than 1 year but why not for Kotlin?

How it goes on?

How it goes on?

up it. any update ?

thanks, fixed in commit 54fe3035bedc2a19896311c9ef804983e40530e0 now


:bookmark: Chore: missing changeset

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

#1018  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @roninjin10 and the rest of your teammates on  Graphite


Create project-sources archive during full build

cc @baldimir

@mbiarnes or @dgutierr could you please take a quick look?

@dgutierr thanks. Since you are the gatekeeper of the dashbuilder repo, please merge the PRs when you approve them.


Game broken

When attempting to access this at https://githubschool.github.io/github-games-beardofedu, I am getting a 404. This could be caused by a couple things:

GitHub pages needs to be enabled on master. You can fix this in the repository settings.
the index.html file is incorrectly named inde.html. We will fix this together in class.

Can you please fix the first bullet @beardofedu?

Fixed in 9482ddd


xADUser invalid $PSBoundParameter index for the $UserName parameter

Creating a new user will fail with:
A parameter cannot be found that matches parameter name 'Name'.
Index operation failed; the array index evaluated to null.

The issue is that $PSBoundParameter is declared for $Name, instead of $UserName, hence the failed index operation.
Changing the two occurrence of:
$PSBoundParameters['Name'] = $Name;

to:
$PSBoundParameters['UserName'] = $UserName;

in DSCResources\MSFT_xADUser\MSFT_xADUser.psm1 resolves the issue.
Tested in Azure using PowerShell DSC VM extension.
Hope this helps.

Hi @tamas7 - thanks for reporting this. I'll get it fixed. Is this on the dev branch? If so, I'm not sure how it got passed all the unit tests... :confounded:

@tamas7 I've fixed this, added test coverage and submitted PR #59. I've actually removed that assignment as the UserName PSBoundParameter will always be there as it's mandatory!
@TravisEz13 @KarolKaczmarek Can we make sure this is merged before the next release is pushed to the PSGallery as the updated xADUser resource won't work without it?!

@iainbrighton Thanks for the PR, tested it in Azure, works!


Deprecate most ezy.neuron.Neuron methods.

Current coverage is 100.00%

Merging #299 into master will not affect coverage as of f727d09

@@            master    #299   diff @@
======================================
  Files           41      41
  Stmts         1832    1854    +22
  Branches         0       0
  Methods
======================================
+ Hit           1832    1854    +22
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of f727d09

Powered by Codecov. Updated on successful CI builds.

Looks fine w/ me

Nice!


Support 3-dimensional iterators in from_array

This change allows 3-dimensional iterators to be used in the first argument to png.from_array.
For example, previously to draw a 4x4 black and red chequered pattern using a list, you would have to do this:
>>> a = [[0, 0, 0, 255, 0, 0], [255, 0, 0, 0, 0, 0]]
>>> png.from_array(a, 'RGB').save('check.png')

With this change, if you wish, you are able to instead add a third dimension, separating the pixels in the rows like so:
>>> a = [[[0, 0, 0], [255, 0, 0]], [[255, 0, 0], [0, 0, 0]]]
>>> png.from_array(a, 'RGB').save('check.png')

Cute! Will merge this once there is a test (which I will write, at some point).

This worked for me. Any chance of merging it and getting it into a release?

yes. :)


Backport all commit at master to maintenance including pr140

Backports of all changes done at master branch upto including #140 .

Test PASSed.


NPE in Roster.getPresencesInternal(BareJid)

Please refer this link:
https://github.com/igniterealtime/Smack/pull/307/commits/8da9f6bc6a97a68d924c6e70fc45258fcb2b4fb0
Can you merge this fix?
We are facing this NullPointer exception when we switch between Wi-Fi and Mobile Data network.

Hello,
I updated this module to make it compatible with smack 4.4.1 which i believe have fix for this, please give it a try.


Feature Request: X Premium Analytics

Feature Request: X Premium Analytics
Hey! I would like to request the feature to fetch analytic data for the account. This might be a big feature so I will (if needed) be able to help you implement this in core and the API. I've already figured out most of the API endpoints needed to fetch the data.
The analytic internal API is divided in 3 sections

Overview -> Overview of all the accounts analytics
(User) Audience -> Information about the audience of the account
Content -> Analytic per tweet

Overview
URL:
https://x.com/i/api/graphql/NlJ6RM-hgHxt-iu9cPQz7A/overviewDataUserQuery
Payload:
{
    "rest_id": "715786798056779776",
    "requested_metrics": [
        "Engagements",
        "Impressions",
        "ProfileVisits",
        "Follows",
        "VideoViews",
        "Replies",
        "Likes",
        "Retweets",
        "MediaViews",
        "Bookmark",
        "Share"
    ],
    "to_time": "2024-09-15T00:00:00.000Z",
    "from_time": "2024-09-01T00:00:00.000Z",
    "granularity": "Daily"
}

Property
Type
Description
Example Value

rest_id
String
Rest ID
715786798056779776

requested metric
String[]
n/a
"Engagements"

to_time
Timestamp
End date timestamp to request analytic data from
2024-09-15T00:00:00.000Z

from_time
Timestamp
Start date timestamp to request analytic data from
2024-09-1T00:00:00.000Z

granularity
String
Determines the level of detail for the analytic report. If set to "Daily", it will return daily reports of analytic numbers between from_date and to_date. Other possible values could be "Weekly" or "Monthly" for different levels of aggregation.
"Daily

Response:
{
    "data": {
        "result": {
            "result": {
                "__typename": "User",
                "organic_metrics_time_series": [
                  // ...
                ],
                "id": "{{  User:{ID} but decoded as base64  }}"
            },
            "id": "VXNlclJlc3VsdHM6NzE1Nzg2Nzk4MDU2Nzc5Nzc2"
        }
    }
}

organic_metrics_time_series response object
[
    "metric_values": [
        {
            "metric_value": "{{ VALUE:INT }}",
            "metric_type": "{{ TYPE:STRING }}"
        },
    ],
    "timestamp": {
        "iso8601_time": "2024-09-01T00:00:00Z"
    }
]

You still have the other 2 pages left, However I don't think they supply information that someone needs daily,weekly or monthly so for now we could skip that one and focus on this. LMK If you need more information or want help.

Thanks for the information. Unfortunately, I don't have X premium to test it out or implement it and it seems X doesn't have any trial for X premium which might have helped me implement this feature. Still, I'll be keeping this issue open if someone else with premium is willing to work on this.

I'll probaly be able to take a look into it.

Started working on this in both Core and API.

@Timtendo12 Sorry for the late reply.
Once you add the request config and the associated types in rettiwt-core, ping me so I'll make a new alpha release of rettiwt-core which we can then use to make the same changes on rettiwt-api.

@Timtendo12 Sorry for the late reply.
Once you add the request config and the associated types in rettiwt-core, ping me so I'll make a new alpha release of rettiwt-core which we can then use to make the same changes on rettiwt-api.

Don't worry about it! :)
I’ve submitted a PR for the core. Since this is my first time working on this project, I’d appreciate any critical feedback.
https://github.com/Rishikant181/Rettiwt-Core/pull/185

@Timtendo12 Rettiwt-Core v4.4.0-alpha.0 has been released with the required request configuration. The request config can be accessed by the user.analytics endpoint of rettiwt-core. You may use that alpha package to develop the feature in Rettiwt-API.
Ping me if you need an explanation on how to go about implementing those changes OR I can implement those by going in blind and ask you to test it :P


[Request] Nintendo Switch port (Homebrew)

Hello,
I don't know if this is possible but any chance to have a portage of this xcloud client to nintendo switch as homebrew ? I hope so...i have a switch OLED and android cannot be yet installed on it : Chiaki & Moonlight have switch versions, i'm dreaming  about a xcloud & geforcenow homebrew.
Thanks.

With the current codebase i dont think it is possible to port it to the Nintendo Switch. Would love to try to develop an alternative client however i do not have a device to develop on.

Hey! Now you can install ubuntu on switch oled and use linux version.
https://gbatemp.net/threads/l4t-ubuntu-a-fully-featured-linux-on-your-switch.537301/


Reverse proxy - real ip not shown.

Home Assistant release with the issue:
0.73.0
Last working Home Assistant release (if known):
0.73.0
Operating environment (Hass.io/Docker/Windows/etc.):
Docker
Component/platform:
requirements.txt
Description of problem:
Hass doesn't see the real ip behind the reverse proxy (Traefik).

Request content
GET /dev-info HTTP/1.1
Host: <IP_ADDRESS>:8033
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.99 Safari/537.36
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8
Accept-Encoding: gzip, deflate, br
Accept-Language: en,nl-NL;q=0.9,nl;q=0.8
Cache-Control: max-age=0
Cookie: __cfduid=XXXXXXXXX
Upgrade-Insecure-Requests: 1
X-Forwarded-For: <IP_ADDRESS>
X-Forwarded-Host: <IP_ADDRESS>:8033
X-Forwarded-Port: 8033
X-Forwarded-Proto: https
X-Forwarded-Server: c3524884eb18
X-Real-Ip: <IP_ADDRESS>

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

   use_x_forwarded_for: True
   trusted_proxies:
    - <IP_ADDRESS>
    - <IP_ADDRESS>
    - <IP_ADDRESS>/16

Traceback (if applicable):

Additional information:

samehere.

Got the same issue. I have tried adding RemoteIPHeader X-Forwarded-For and ProxyPreserveHost On in my apache config but it doesn't make any difference.


disable if DISABLE_RERUN is true

Merge after #42, #43, #44.
See https://github.com/SCG82/wdio-rerun-service/pull/3 to view diff.

closing in lieu of #48


Skip verbose logs when logger names start with our exporter package name

Fix https://github.com/Azure/azure-sdk-for-java/issues/40546

API change check
API changes are not detected in this pull request.


Request for Spark! Tech Resources for [CISS Immigration Raids]

Project Name
CISS Immigration Raids
Project Type
Data Science / Machine Learning
Team Members + Emails
"Xie, Handi"<EMAIL_ADDRESS>"Zhang, Yuhao"<EMAIL_ADDRESS>"Pierre-Louis, Farrah"<EMAIL_ADDRESS>"Daylor, Kaya"<EMAIL_ADDRESS>"Erozer, Can"<EMAIL_ADDRESS>Detailed List of Resources Needed
Access to OpenAI API.
We plan to test our prototype with GPT-4o-turbo.
Description of Resource Usage
It's only for experimental usage, not for production-level usage.
Course Deadlines (if applicable)
The sooner the better, we have already pivoted a few times.

@PalmPalm7 I added the entire team using the emails you provided, please accept the invite.

@PalmPalm7 I added the entire team using the emails you provided, please accept the invite.

Thank you Michelle!


store.combine() is mutating emitted state object

Which @ngneat/elf-* package(s) are the source of the bug?
store
Is this a regression?
No
Description
Using store.combine() mutates the current object when emitting changes rather generating a new object reference causing downstream rxjs operators not to operator as expected.
The behavior is exhibited when using any operator that is maintaining references to emitted values such as distinctUntilChanged or pairwise operators.
Please provide a link to a minimal reproduction of the bug
https://stackblitz.com/edit/fvvzkm-842hjx?file=index.ts
Please provide the exception or error you saw
The previous and current object value are the same as the object is mutated.

Please provide the environment you discovered this bug in
No response
Anything else?
Using store.combine() should spread it's emitted value thus avoiding mutations.
Do you want to create a pull request?
Yes

Fixed in #462 ?


Handle Java updates & multiple versions

Oracle wasn't releasing minor Java updates for EV3, so we were stuck with one version. This has changed - openjdk-ev3 can be rebuilt at any time.

Maybe the installer could do periodic auto-update?
Or we could create a repository for the packages and download them via APT.

The idea about Oracle stuff is previous to @JakubVanek age! :)
Now, the idea with Installer project is the automation of everything related with the project in an easy way. The first step is the JVM to run the programs. Later other debian packages like OpenCV for example.


Insiders binaries are inside 1.2.1 stable release

Both 32 and 64 bit stable packages for 1.2.1 are suspiciously big:

Looking inside one, I found an insiders binary:

Does not seem to have happened to 1.3.0

No issue then because 1.3.0 is fine? I ran gulp vscode-linux-x64 and doesn't produce 2 binaries also.


[Feature request]: When used as a server for WSGI and ASGI, can be running with Gunicorn

Uvicorn provides a worker to be used together with gunicorn：

Gunicorn is a mature, fully featured server and process manager.
Uvicorn includes a Gunicorn worker class allowing you to run ASGI applications, with all of Uvicorn's performance benefits, while also giving you Gunicorn's fully-featured process management.
This allows you to increase or decrease the number of worker processes on the fly, restart worker processes gracefully, or perform server upgrades without downtime.

Can socketify.py provide a worker to work with gunicorn?

socketify wants to replace gunicorn, we will add all gunicorn features to socketify CLI https://docs.socketify.dev/cli.html
I will keep it open because probably is faster to create a gunicorn worker than implementing all features.


Array indices can only have 63 bits

The problem
As pointed out by @lizmat here
Suggestions
Please mention it in the subscripts

Perhaps mention Array::Sparse and/or that one can use hashes with integer keys?


[Tabs] Fix calculating tab indicator position

See disussion at the end of #8379.
Two questions:

Is it okay to test internals (ie. valueToIndex)?
Should I squash my failing test?

@ljani It's perfect 👌


Graph Explorer Preview version does not render image on me/photo/$value

Use the issues in this repository to open issues about the Graph Explorer application. Questions about the Microsoft Graph should be directed to StackOverflow. Use the [microsoftgraph] tag on StackOverflow.
Expected behavior
Should render the photo as in the original version of Microsoft Graph Explorer.
Actual behavior
Empty json value is returned instead:  {}
Steps to reproduce the behavior
Perform a get request to https://graph.microsoft.com/v1.0/me/photo/$value  -- in production version, an image is rendered.  In the preview version, just the empty json is returned.
HTTP traffic capture

@RayGHeld this is now resolved here: https://developer.microsoft.com/en-us/graph/graph-explorer


Issues in the examples

Hi, thank you very much for this amazing library
There's an issue in the examples and probably in the code too
let palette = spectral.palette('#00357B', '#F0F0F0', `9);
let palette = spectral.palette('rgb(240, 240, 240)', 'rgb(240, 240, 240)', 9);
let palette = spectral.palette('[240, 240, 240]', '[240, 240, 240]', 9, spectral.RGB);

I guess it should be:
let palette = spectral.palette('#00357B', '#F0F0F0', `9);
let palette = spectral.palette('rgb(0, 53, 123)', 'rgb(240, 240, 240)', 9);
let palette = spectral.palette([0, 53, 123], [240, 240, 240], 9, spectral.RGB);

The result doesn't seem to be correct either.
palette = spectral.palette('#00357B', '#F0F0F0', 6);
console.log('PALETTE 1 ', palette)  // ['#00357b', '#01509c', '#288aca', '#67bee3', '#b1e2ee', '#f0f0f0']

--> palette = spectral.palette('rgb(0,53,123)', 'rgb(240,240,240)', 6);
--> console.log('PALETTE 2 ', palette)  // ['#NaNNaNNaN', '#NaNNaNNaN', '#NaNNaNNaN', '#NaNNaNNaN', '#NaNNaNNaN', '#NaNNaNNaN']

palette = spectral.palette([0, 53, 123], [240, 240, 240], 6);
console.log('PALETTE 3 ', palette)  // ['#00357b', '#01509c', '#288aca', '#67bee3', '#b1e2ee', '#f0f0f0']

Is it a issue in the example or a bug in mix() function in the for loop for ( let i = 0; i < SIZE; i++ ) ... ?
 The ‘rgb()’ probably isn’t parsed as it should. I will look at this as soon as I can.

Hi! I've updated the library to v2.0.1, the color string parsing is fixed in this release.
I've also fixed the errors in the readme.
Thank you for pointing out the errors.
I will close this issue.
If there is still a problem or you found a new problem please open an new issue.

Hi! I've updated the library to v2.0.1, the color string parsing is fixed in this release.
I've also fixed the errors in the readme.
Thank you for pointing out the errors.
I will close this issue.
If there is still a problem or you found a new problem please open an new issue.

Hi. You're welcome. It's working as expected.
Cheers
rjs


Using CORS_MODEL with custom headers

Hi,
I am using django-cors-headers==2.0.0 with Django 1.10.
Things seem to work as expected except whenever CORS_MODEL = myapp.MyCorsModel is added in settings with some custom headers included in  CORS_ALLOW_HEADERS , the custom headers are not included in returned responses.
Adding CORS_ORIGIN_ALLOW_ALL = True or adding the required origins in CORS_ORIGIN_WHITELIST custom headers are returned but  with CORS_MODEL this is not supposed to be required.
I think this issue is caused by the current  middleware logic, when the CORS_MODEL is used response is returned before adding custom headers https://github.com/ottoyiu/django-cors-headers/blob/master/corsheaders/middleware.py#L118-L126

Fixed by https://github.com/ottoyiu/django-cors-headers/pull/208

Thanks for updating issue


Default Health Endpoints

Per #1279 the initial func.yaml should ideally include only those values which deviate from static defaults.
The healthEndpoints are currently written out, despite being defaults:
> cat func.yaml
...
healthEndpoints:
  liveness: /health/liveness
  readiness: /health/readiness
...

These are of course the defaults, and are already defined as DefaultReadinessEndpont and DefaultLivenessEndpoint.
These constants should be used when the function struct's .HealthEndponts member is at zero value, rather than populated with these values when created.  Also, the member should be set to omitempty when serialized out as .yaml.
Caviats
There is a good case to be made that a Function struct should be populated with these static defaults upon instantiation, rather than requiring all the users of that struct "downstream" from said instantiation to check for empty values and then use the default if not set.  The serialized form would then include all these defaults, and one becomes required to use the constructor NewFunction() rather than the struct literal Function{}.  There are benefits to both, but considering we are serializing the function to disk as func.yaml, minimizing unnecessary member initialization gains cleanliness that will be seen by end users.

/kind enhancement

/kind good-first-issue

/remove-lifecycle stale

/assign @lance


I can´t install DRAWNOW package on anaconda environment

Problem Description
I need to install drawnow package.
I tried to use Anaconda Navigator and the package don't exist.
Then, I tried to use Anaconda console:

conda install drawone

PackageNotFoundError: Packages missing in current channels:

drawnow

We have searched for the packages in the following channels:

https://repo.continuum.io/pkgs/main/win-64
https://repo.continuum.io/pkgs/main/noarch
https://repo.continuum.io/pkgs/free/win-64
https://repo.continuum.io/pkgs/free/noarch
https://repo.continuum.io/pkgs/r/win-64
https://repo.continuum.io/pkgs/r/noarch
https://repo.continuum.io/pkgs/pro/win-64
https://repo.continuum.io/pkgs/pro/noarch
https://repo.continuum.io/pkgs/msys2/win-64
https://repo.continuum.io/pkgs/msys2/noarch

pip install drawnow
Exception:
Traceback (most recent call last):
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\basecommand.py", line 215
, in main
status = self.run(options, args)
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\commands\install.py", lin
e 335, in run
wb.build(autobuilding=True)
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\wheel.py", line 749, in b
uild
self.requirement_set.prepare_files(self.finder)
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\req\req_set.py", line 380
, in prepare_files
ignore_dependencies=self.ignore_dependencies))
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\req\req_set.py", line 554
, in _prepare_file
require_hashes
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\req\req_install.py", line
278, in populate_link
self.link = finder.find_requirement(self, upgrade)
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\index.py", line 465, in f
ind_requirement
all_candidates = self.find_all_candidates(req.name)
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\index.py", line 423, in f
ind_all_candidates
for page in self._get_pages(url_locations, project_name):
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\index.py", line 568, in _
get_pages
page = self.get_page(location)
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\index.py", line 683, in _
get_page
return HTMLPage.get_page(link, session=self.session)
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\index.py", line 811, in g
et_page
inst = cls(resp.content, resp.url, resp.headers)
File "C:\ProgramData\Anaconda2\lib\site-packages\pip\index.py", line 731, in _
init
namespaceHTMLElements=False,
TypeError: parse() got an unexpected keyword argument 'transport_encoding'

I don't know what to do. It's package is very important for me because I need it to migrate from MATLAB.
I am using Windows platform, Windows 7.

Sorry, we can't help you with this. Please open an issue in the drawnow repo to ask for help there:
https://github.com/stsievert/python-drawnow/issues


AssemblyLoadContext-related task errors in 16.5

Tracking issue:

https://github.com/dotnet/sdk/issues/10878
https://github.com/SpecFlowOSS/SpecFlow/issues/1912
GitTools/GitVersion#2063

Figured out the SpecFlow case: https://github.com/SpecFlowOSS/SpecFlow/issues/1912#issuecomment-604462903.
In short, this is an expected behavior of Assembly.LoadFile that was sort of worked around before because MSBuild could find dependencies in AssemblyLoadContext.Default.

@rainersigwald It turns out it isn't solved by coverlet.msbuild 2.8.1. This PR is blocked in updating to .NET SDK 3.1.201 because of this same break, and updating coverlet didn't fix it. So I'm still blocked.
https://github.com/AArnott/Nerdbank.Streams/pull/180/files

@AArnott sorry it took me a while to get to this, but I don't see anything that looks like an MSBuild task failure in that PR. Can you point me to what you think is failing?

Quite right. I forgot in that PR that updating to coverlet.msbuild 2.8.1 also changes the filenames of the coverage files generated, so there's more to adapting to the task loader change than just updating the package reference.

Closing: all of the known active-development tasks that had problems have been fixed in new releases.

For gitversion, recent exceptions customers have been facing have been the below: Adding a comment to help with customer searches.
Error Exception in initialization: System.IO.FileNotFoundException: Could not load file or assembly ‘System.ValueTuple, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51’ or one of its dependencies. The system cannot find the file specified.


Add embedded ruby template

I added the *.erb file type, which was missing.

ERB (Embedded RuBy) is a feature of Ruby that enables you to conveniently generate any kind of text, in any quantity, from templates.

A file that contains an ERB template may have any name, but it is the convention that the name of file should end with the .erb extension.

Thank you!


Strip the url fragment for outgoing requests

Cherry-picked to v9.0.x branch.
Cherry-picked to v9.1.x branch.

If we want this for 8.x can we get a backport?


DOCS: Need a detailed log-path filter development chapter

As an SC4S community member I need detailed directions I may follow to successfully design, build and test a new log-path filter so that I may ingest data sources not yet supported
This section should contain the following at the least...
Detailed breakdown of log-path filter functionality via walk through of an existing log-path
LP design guidance
Steps to build example
Details on testing new filter
Steps and best practices for contribution of filter to community
know gotchas

Hi, did this happen?

+1

Done


Dark mode toggle

I love this tool! A dark mode toggle would be a great feature. Colors look so different when layered on black/white, so being able to toggle the background color helps see the colors properly.

I can't believe I didn't think of this before. That's such a useful and quite important feature.
Thank you very much for the suggestion @fatesallow!

@pabueco Oh good! I'm glad I mentioned it. Thanks for making this. It's super helpful!


[Bug]:always to check

Version of the software
0.2.4
Operating system
windows11
Browser / Native
brave
What happened?
All the accounts go for to check , no failed , no hits

Relevant LoliCode if needed
No response

This is an issue with your config in particular, not with OB2 itself. Please enable "bot log" in the RuriLib settings and then check the bot log of one of those hits, it will have information on why it ended with status NONE instead of FAIL / SUCCESS etc.


can ump_shared.dll be open sourced?

Very good, can ump_shared.dll be open sourced?I want to try the plugin function on android.

https://github.com/wongfei/mediapipe/tree/unreal/mediapipe/unreal


feat(includes.mk): add Go 1.5 vendor support

closes #4476

d'oh:
package github.com/deis/deis/builder/tests (test)
	imports github.com/deis/deis/tests/dockercli
	imports github.com/docker/docker/api/client
	imports github.com/docker/docker/api
	imports github.com/docker/docker/engine
	imports github.com/docker/docker/utils
	imports github.com/docker/docker/pkg/archive
	imports github.com/docker/docker/vendor/src/code.google.com/p/go/src/pkg/archive/tar: must be imported as src/code.google.com/p/go/src/pkg/archive/tar

Sounds like http://google-opensource.blogspot.ca/2015/03/farewell-to-google-code.html finally happened. I recall reading something on Hacker News this morning about this.

@bacongobbler yup, it happened :(

See my comment on #4874 re: switching to glide.  imo, we need to do that for LTS.  If not, then we wouldn't need this change either.

closing this for the reasons stated above as well as some failures, as I cannot seem to migrate Godep over to the Go 1.5 vendor experiment due to #4878.


[SPARK-20038] [SQL]: FileFormatWriter.ExecuteWriteTask.releaseResources() implementations to be re-entrant

What changes were proposed in this pull request?
have theFileFormatWriter.ExecuteWriteTask.releaseResources() implementations  set currentWriter=null in a finally clause. This guarantees that if the first call to currentWriter() throws an exception, the second releaseResources() call made during the task cancel process will not trigger a second attempt to close the stream.
How was this patch tested?
Tricky. I've been fixing the underlying cause when I saw the problem (HADOOP-14204), but SPARK-10109 shows I'm not the first to have seen this. I can't replicate it locally any more, my code no longer being broken.
code review, however, should be straightforward

Test build #74903 has started for PR 17364 at commit 725740b.

Note that as the exception handler tries to close resources before calling committer.abortTask(taskAttemptContext), without this patch a failing releaseResources() means that  abortTask() isn't invoked, though hopefully abortJob will handle tasks too

Test build #74903 has finished for PR 17364 at commit 725740b.

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/74903/
Test PASSed.

@steveloughran, maybe this is strictly not related with the problem specified in JIRA but do you know if we should do the same thing to SparkHadoopMapReduceWriter? I remember I had to fix the resource related problem identically with FileFormatWriter before later.

I haven't reviewed that bit of code: make it a separate JIRA and assign to me. This one I came across in the HADOOP-2.8.0 RC3 testing; the underlying fix there is going in, but the spark code still should be made more resilient to failure here. It's always those failure modes which get you —and working with S3, that close() can be where the PUT is initiated, so P(fail) > 0.
Part of [HADOOP-13786] (https://issues.apache.org/jira/browse/HADOOP-13786) is MAPREDUCE-6823: making it straighforward to define a different implementation of the FileOutputFormat committer. That should make it easier to do some fault injection in the commit processes, especially all those bits that violate the state machine entirely. I'll see what I can do about breaking things :)

Created SPARK-20045. I think there's room to improve resilience in the abort code, primarily to ensure that the underlying failure cause doesn't get lost. The codepath there is fairly complex  and I'm not going to point at a snippet and say "here". Some faulting mock  committer would probably be the actual first step: show the problems, then fix.

Note that as the exception handler tries to close resources before calling committer.abortTask(taskAttemptContext), without this patch a failing releaseResources() means that abortTask() isn't invoked, though hopefully abortJob will handle tasks too

it doesn't look that way to me -- the abortTask is in a finally, so it should get called no matter what.
You could add a very targeted regression test -- create the ExecuteWriteTask with mock OutputFormats, have those mocks throw an exception on close.  Then make sure two calls to releaseResources work correctly.  Your change is obviously correct, but would help avoid future regressions.
in any case, lgtm

looking some more, yes,  as tryWithSafeFinallyAndFailureCallbacks wraps task commit, it guarantees that the original cause doesn't get lost. The abortJob code isn't so well guarded, and looks like a failure there my hide a previous one (like a commitJob failure).

I don't have a time/plans to do the test here, as it's a fairly complex piece of test setup for what a review should show isn't doing anything other than guarantee the outcome pf currentWriter==null out of all sequential execution paths

@squito Is this ready to go in? Like I warned, I'm not going to add tests for this, not on its own

merged to master
sorry I forgot to take look at this for a while @steveloughran, thanks for the reminder

thanks.


Modify default Ask breadcrumb logic

This PR modifies the logic by which the breadcrumb is determined for an Answer page. The previous logic was:

If there is a primary portal topic, use the primary portal topic;
If no topics have been selected, use /ask-cfpb;
If one or more topics have been selected but no primary topic, use the first topic selected from the checklist

This PR changes the third item to

If one or more topics have been selected but no primary topic, use /ask-cfpb.

Changes

Changes to get_context function within answer_page.py that determine how the breadcrumb is selected

How to test this PR

localhost
navigate to any existing Answer page
modify options within "Money Topic portal tags" to test all variations of the logic

Screenshots
Page editor and live-preview view of "Ask CFPB" as the breadcrumb when no primary topic is selected for a page with two topics checked

Checklist

[ ] PR has an informative and human-readable title

PR titles are used to generate the change log in releases; good ones make that easier to scan.
Consider prefixing, e.g., "Mega Menu: fix layout bug", or "Docs: Update Docker installation instructions".

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code follows the standards laid out in the CFPB development guidelines
[ ] Future todos are captured in comments and/or tickets
[ ] Project documentation has been updated, potentially one or more of:

This repo’s docs (edit the files in the /docs folder) – for basic, close-to-the-code docs on working with this repo
CFGOV/platform wiki on GHE – for internal CFPB developer guidance
CFPB/hubcap wiki on GHE – for internal CFPB design and content guidance

I added a few unit tests to better document the expected logic. Works as expected!

@chosak looks like https://github.com/cfpb/consumerfinance.gov/pull/7804/commits/e6bd981f39905cd3a8cb4082b9b4113fef104ef2 had two duplicate test_portal_page_primary_topic blocks. I removed one in https://github.com/cfpb/consumerfinance.gov/pull/7804/commits/88c375c0c4a32c3a3d0440eadddd607358c919ae
Is that ok?


Fix copy bug

Closes #2232

Rerunning gcc build because coverage uploading failed.

General comment, we should cancel CI runs that are redundant. I cancelled run #6875 for you.


update aws sdk: permit to work with s3 compliant storage with sigv4

AWS GO SDK update to be able to store images on openio-sds, swift3 (s3 compliants) storages.

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "feat-update-aws-sdk-go" git@github.com:hightoxicity/distribution.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@hightoxicity Make sure to format your commit message correctly. It looks like the subject line and sign off have been switched around some how.


Add ONNX export support for PatchTST

What does this PR do?
This PR adds ONNX export support for PatchTST models, as well as fixes other models that contain the torch.unfold operator, which is not supported by torch.onnx.export. This is done by applying a set of default patching specs.

Fixes https://github.com/huggingface/transformers.js/issues/1047
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you make sure to update the documentation with your changes?
[x] Did you write any new necessary tests?

Who can review?
@michaelbenayoun, @echarlaix

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


App is not working with uber &left

Hello brother'
Your app work very good  .but The big fat issue is . Its not working with some apps at all like uber driver and left and bolt driver  these apps direct open the camera and seem to ignore giving me a choice  .
Do u have any idea what might be the issue  ?

Same issue, some app block it. Have you tried with android 10 ?

Same issue, some app block it. Have you tried with android 10 ?

Yes i have s9 pluse with android 10 . I brougth it just for this  xd


Add Scoop command

I added the command that can be used to install kubectx and kubens using Scoop

@ahmetb  Is everything good now ?


Update lib

Project URL
**
Category

Description

Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a clear README in English

          1 Warning

:warning:
Found 1 link issue, a project collaborator will take care of these, thanks :)

Link issue by awesome_bot

Line
Status
Link

1662
301
https://github.com/VirgilSecurity/virgil-sdk-x redirects tohttps://github.com/VirgilSecurity/sdk-x

  Generated by :no_entry_sign: Danger

@fauzisho We cannot merge your PR yet until you write some tests for your lib. Feel free to open PR once it is ready.
Please check our CONTRIBUTION GUIDELINES


Fixed local XSD lookup

Pull Request Type

[X] Bug fix (non-breaking change which fixes an issue.)
[ ] Enhancement/New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
There is an issue in the XSD lookup logic, where it always pulls the XSD file over the network if the primary ResourceAccessor fails to find it and the "fallback" one does.
When we'd get to the fallback code, the && SECURE_PARSING) would be false and make the if block fail regardless of whether it found resources or not.
The standard Liquibase integrations (CLI, Maven, etc.) set up the primary resource accessor so we never hit the problem code path, but custom setups could. The integration tests, for example, did hit the problem code.

Notes and test results:
Things to be aware of

With my internet broken, the integration tests failed without this fix and pass with it in
Added "autocandidate" since it's not something reachable from the CLI, maven, etc.
It's an isolated fix to fallback logic

Things to worry about

Nothing


False report about unused navigation UI library

Gradle version
6.0.1
Android Gradle Plugin (AGP) version
3.5.3
Describe the bug
Some of our modules are using the navigation component library. It generates code from xml that describes navigation logic.
To Reproduce
Steps to reproduce the behavior:

Add androidx.navigation:navigation-ui-ktx:<latest version>
Create some navigation xml
Run the advice for the project

Expected behavior
Do not report false positive.
Additional context
Add any other context about the problem here.

I'm really sorry for the wrong ticket. The UI ktx dependency was not used but there was class used from the transitive dependency.


Symptom that may indicate a problem.

When refreshing a page (F5) where a Tiwbstabcontrol is present the activetab is reset to the default one (the tab that was set at design time or set in the oncreate event of the form).
If a modal dialog is opened (for example using webapplication.ShowMessage or a Tframe) then the active tab won't be changed anymore doing a page refresh.

That is the designed behavior.
Selected page is reported to served side on any async call.
If you need to keep your selected page always in sync with the server you can simple set the OnAsyncChange event of the tab with an empty code:
procedure TIWForm2.IWTabControl21AsyncChange(Sender: TObject;
  EventParams: TStringList);
begin
  //
end;

This will force to submit any page change to the server.
Note that this will impact on performance on slow connections.
This is why is not enabled by default.
Regards


Add "--log_metadata_and_status" interop client flag (and a test)

See internal bug b/203714502

This is ready for initial review now. See internal bug b/203714502 for context.

@gnossen, before we merge this, can you please do an internal cherry-pick of this PR to double check everything is fine?
I ran a cherry-pick before and it seemed that they're might be and ODR violation between the internal version of interop_client and the updates from this PR.
FTR my cherry-pick is cl/410486644 and the failure is sponge/1cacc4d3-14ed-4c05-8868-12d725339af1
(I'm currently OOO).

I have resolved the ODR violation in a CITC client. After the last couple of OSS tests here finish, I will cherry-pick this in.

@gnossen, fyi, lately the ObjC iOS test has been getting stuck.

Yes. At any rate, after some digging, it looks like there shouldn't be any dependency relationship between this PR and the ObjC iOS basic tests, so going ahead and merging now.


Delivery Service APIs: State

[ ] get /api/$version/deliveryservices/:id/state

this endpoint is addressed in #2232


Analyze current status of Kogito straight-through processing

Kogito straight-through processing was late in development in Red Hat. We need to find out what is the current status, gaps etc.
Acceptance criteria:

Document describing the current status exists.
Issues for the identified gaps are created in kie-roadmap or kie-issues repository.

BAPL from Red Hat JIRA is here: https://issues.redhat.com/browse/BAPL-1944

Kogito straight-through processes status.xlsx

Very basic status of the issues listed in the Epic linked in the previous comment:
Kogito STP status.xlsx

Collecting information in Kogito STP - current state


Update permissions are not being processed

Will be fixed in next release.
Update your logic/declare_logic.py to include Grant.process_updates(logic_row=logic_row):
    def handle_all(logic_row: LogicRow):  # OPTIMISTIC LOCKING, [TIME / DATE STAMPING]
        """
        This is generic - executed for all classes.

        Invokes optimistic locking.

        You can optionally do time and date stamping here, as shown below.

        Args:
            logic_row (LogicRow): from LogicBank - old/new row, state
        """
        if logic_row.is_updated() and logic_row.old_row is not None and logic_row.nest_level == 0:
            opt_locking.opt_lock_patch(logic_row=logic_row)
        enable_creation_stamping = False  # CreatedOn time stamping
        if enable_creation_stamping:
            row = logic_row.row
            if logic_row.ins_upd_dlt == "ins" and hasattr(row, "CreatedOn"):
                row.CreatedOn = datetime.datetime.now()
                logic_row.log("early_row_event_all_classes - handle_all sets 'Created_on"'')
            Grant.process_updates(logic_row=logic_row)

Also, you need to add the import:
from security.system.authorization import Grant

fixed in 9.05.06


Add missing timestamper cert to the newer catalog signing APIs

This fixes Issue #4060
Code to obtain the file signature time stamp cert was missing from the newer catalog signing APIs, with the result that Get-AuthenticodeSignature would not include the time stamp cert in the Signature object.
Fix is to refactor and use the older API code for obtaining this.
I didn't add a test for this because no catalog signing tests are in GitHub yet and porting them will be a separate work item.

I created issue #4087 to document the lack of tests

@mirichmo Is there anything preventing this from being merged?  I also need to port this to internal branches.  Thanks!


Web IDE in Firefox

Lastest Firefox: 41.0.1

The selection is not visible. (I can see it in Chrome.)
The Dark button changes the syntax highlighting but not the background.
Also drag'n'drop of a line works... if I drop after the first char of the target line (or on an empty line). Nearer of the gutter, I get a cursor blinking between pointer and "not drop", and the drop is ignored.
Actually, if I double-click a word and drag it, the cursor might be stable or blinking, depending on the drop position (no particular pattern, a same position can blink, and if I go back to it, it can be stable). If I drop when stable, it works: word is moved (copied with Ctrl). If I drop when blinking... Firefox attempts to go to www..com!

Generic remark (also applies to Chrome): the IDE should detect the user navigates away from the page, and if the document is dirty, ask for confirmation.

The selection is not visible. (I can see it in Chrome.)
The Dark button changes the syntax highlighting but not the background.

I can't reproduce either of these problems in Firefox on Mac OS.

Nor do I observe anything weird with drag+drop.

Sorry, I should have mentioned I see this on Windows 7 Professional.
I just upgraded to v.41.0.2, with same behavior.

Just tried with 41.0.1 on Windows 7 Home Premium and things work just fine.
Can't it be some plugin you installed maybe?

I get different results (Firefox 43.0a2 (2015-10-14), Arch Linux):

The selection is visible, but in Dark mode pretty bright, making the text almost unreadable.
Dark changes syntax highlighting (though the changed highlighting for Markdown renders the [] part of links almost unreadable) and background, but not the rest of the page (which still has some bright elements).
Drag and drop doesn’t work at all. The selection color changes from blue-ish to grey-ish, but the cursor remains the same, and nothing happens.

Generic remark (also applies to Chrome): the IDE should detect the user navigates away from the page, and if the document is dirty, ask for confirmation.

+1. That would be an beforeunload event handler.

Oops! I totally forgot I had a Stylish style to fix dark theme code snippets on Ceylon-lang.org, which applies here too...
Deactivating it restores the selection and the dark theme.
I still have issues with drag'n'drop, I will check if deactivating all the extensions fixes that.
Sorry for the noise.

Closing.


Error database when attempting to install

Hi, i get this error went i try to install the Mod.
DATA BASE ERROR : Data truncated for column 'is_solved' at row 11

		ALTER TABLE smf_topics
		CHANGE COLUMN `is_solved` `is_solved` tinyint(3) unsigned NOT NULL default '0'

Could you provide info about your setup?
In any case, try changing the column type to smallint.

Now is working, the problem was that column "is_solved" was already existent in my data base with some data saved in it and looks like is used be another mod (I'm not 100% sure, I'll check this when I have more free time.).
I solved the problem by changing the mod to use the name "iss_solved" and "cans_solve" because i don't want to brake anything else.
For now I noted this and in the future I will know if there are new updates to keep the same name.


Upgrade npm dependencies, notably nan to 1.6.2

This allows v8-debug to build under io.js and Node 0.12. However, v8-debug does not run under io.js! There is a problem where processDebugRequest is not getting called and the test hangs. This isn't a regression and this commit is in the right direction to getting v8-debug and node-inspector working on io.js -- it needs more investigation though.
The tests all pass under Node 0.10.36.

Thanks. I merged this.
Unfortunately this doesn't resolve main current problem


use encodeURIComponent instead of encodeURI for query parameters

Right now, the normal string gets encoded using encodeURI, which might need some correction to use encodeURIComponent.
For ex: name=alice&age=20 works fine
whereas, name=alice&bruno&age=20 fails since the first & is part of the first query parameter.

I only see encodeURIComponent in superagent source. Can you give more information about the bug?


Adjust frame dropping frame time calculation

... so it calculates from the beginning of the last frame to the beg of the current one. It's more exact when processing out of our task takes significant time.

I've discovered something new and I still need to do more testing.
It still doesn't work well on my mac's builtin display (120Hz). I'm going to make it smarter and detect the frame rate instead of having a hardcoded one.


Decrease Message Space and Some CSS Refactoring

Decrease space between messages.
Relate bottom padding of chat body and height of chatroom footer with variables.
Move legacy .box-footer styles to newer #chatroom-footer css.

should i delete any fall backs related to adminLTE.css?


replaced container div with react fragment

This change allows you to use ColorPicker in a form without having to fight the default generic div container styling.
React Fragments

I just noticed my auto-linter went wild with "s and ;s.  Let me know if you want me to modify the PR to better match the previous style.

@LoicMahieu not a problem! I made the changes to revert back to the original code style. Thank you!

<EMAIL_ADDRESS>
Thanks!


remove snapshot test

Snapshot tests were a worthwhile experiment, but I believe they have not provided enough net value to justify keeping them around. This removes the last remaining one.

@zarsky-broad (cc: @panentheos) I hope we did a bit better. I hope we proved that we can have high reliability without these. I'd love to see an addendum to our Reliability Strategy doc that discusses this (noting the timeframe we had them in place).


Add 9anime or Animepahe

Hi, I love this app,
4anime is missing many animes and either 9anime or animepahe would solve that issue if u would like

please open all issues related to extensions in jmir1/tachiyomi-extensions


Live demo doesn't work

When I click the live demo I get this:
"This site can’t be reachedtypescript-node-starter.azurewebsites.net’s server IP address could not be found.
DNS_PROBE_FINISHED_NXDOMAIN"

This is the same issue from Oct 24: https://github.com/microsoft/TypeScript-Node-Starter/issues/243

It's down again :(

It is still down.

Related PR: #302

It's still not working.

Over 9 months to fix a url in a README doesn't inspire a lot of confidence. Can anyone fix this issue before it's first birthday?

We're having this issue over like a year and a half now, so why can't just a maintainer push this project to a service like Heroku? It could be done under half an hour and would end all this mess.

Over 9 months to fix a url in a README doesn't inspire a lot of confidence. Can anyone fix this issue before it's first birthday?

Uhm, we already hit its first birthday. We're approaching its second birthday. 😂

Almost forgot to wish this issue a happy birthday 🎂

@orta  That's the fix, remove the links? MS owns github now... isn't it possible to get it running on a GhPages branch and provide even more value to those cloning this starter repo?

@orta  That's the fix, remove the links? MS owns github now... isn't it possible to get it running on a GhPages branch and provide even more value to those cloning this starter repo?

The issue is that GitHub Pages can only serve static pages.

How about Azure?

@orta  That's the fix, remove the links? MS owns github now... isn't it possible to get it running on a GhPages branch and provide even more value to those cloning this starter repo?

The issue is that GitHub Pages can only serve static pages.

One option is to add a mongo-dump step to create a JSON file and allow it to be served static? If the problem is that it's a kitchen sink starter kit that can neither be reliably demonstrated, nor reliably hosted by the creators' own cloud service - maybe it requires some non-trivial redesign.