Update Blog style

The blog still uses the old CSS.
Pleas change it to use the new styles. It's probably easiest to copy all the styles into the blog repo.
You'll have to get jekyll up and running, which is a cool tool.
Any questiosn, ask me!

Branch and changes have been made.

Looks great.
However, one thing that won't work:
<link href="http://www.kittysplit.com/css/app.css?vsn=129D7C9" rel="stylesheet" type="text/css">
The ?vsn=129D7C9 is a cache-busting measure, and will change the next time Kittysplit is deployed. I'm going to remove it completely, should work then

Yeah sure.

On 31 May 2015, at 6:42 pm, Caspar<EMAIL_ADDRESS>wrote:
Looks great.
However, one thing that won't work:

The ?vsn=129D7C9 is a cache-busting measure, and will change the next time Kittysplit is deployed. I'm going to remove it completely, should work then
—
Reply to this email directly or view it on GitHub https://github.com/caspii/kittysplit-blog/issues/3#issuecomment-107222215.

One more thing: could you make the font-weight lighter to look more in line with http://www.kittysplit.com/help?

Done. Had to load the Google font seeing the @import() didn't deliver it
through the stylesheet.
On Sun, May 31, 2015 at 7:29 PM, Caspar<EMAIL_ADDRESS>wrote:

One more thing: could you make the font-weight lighter to look more in
line with http://www.kittysplit.com/help?
—
Reply to this email directly or view it on GitHub
https://github.com/caspii/kittysplit-blog/issues/3#issuecomment-107226089
.

Cool, thanks. You shouldn't be using @import anyway, see here:
http://stackoverflow.com/questions/7199364/import-vs-link
On Mon, Jun 1, 2015 at 8:35 AM, Matthew Henderson<EMAIL_ADDRESS>wrote:

Done. Had to load the Google font seeing the @import() didn't deliver it
through the stylesheet.
On Sun, May 31, 2015 at 7:29 PM, Caspar<EMAIL_ADDRESS>wrote:

One more thing: could you make the font-weight lighter to look more in
line with http://www.kittysplit.com/help?
—
Reply to this email directly or view it on GitHub
<
https://github.com/caspii/kittysplit-blog/issues/3#issuecomment-107226089>

.

—
Reply to this email directly or view it on GitHub
https://github.com/caspii/kittysplit-blog/issues/3#issuecomment-107322327
.

--
Caspar +491635561322 Berlin, Germany

Good to know. You can be the one to tell the boys ;-)
On Mon, Jun 1, 2015 at 8:56 AM, Caspar<EMAIL_ADDRESS>wrote:

Cool, thanks. You shouldn't be using @import anyway, see here:
http://stackoverflow.com/questions/7199364/import-vs-link
On Mon, Jun 1, 2015 at 8:35 AM, Matthew Henderson <
<EMAIL_ADDRESS>wrote:

Done. Had to load the Google font seeing the @import() didn't deliver it
through the stylesheet.
On Sun, May 31, 2015 at 7:29 PM, Caspar<EMAIL_ADDRESS>wrote:

One more thing: could you make the font-weight lighter to look more in
line with http://www.kittysplit.com/help?
—
Reply to this email directly or view it on GitHub
<

https://github.com/caspii/kittysplit-blog/issues/3#issuecomment-107226089>

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/caspii/kittysplit-blog/issues/3#issuecomment-107322327>
.

--
Caspar +491635561322 Berlin, Germany
—
Reply to this email directly or view it on GitHub
https://github.com/caspii/kittysplit-blog/issues/3#issuecomment-107329525
.

Don't need to! The blog is still hosted on Google App Engine along with the
old Kittysplit.
On Mon, Jun 1, 2015 at 8:59 AM, Matthew Henderson<EMAIL_ADDRESS>wrote:

Good to know. You can be the one to tell the boys ;-)
On Mon, Jun 1, 2015 at 8:56 AM, Caspar<EMAIL_ADDRESS>wrote:

Cool, thanks. You shouldn't be using @import anyway, see here:
http://stackoverflow.com/questions/7199364/import-vs-link
On Mon, Jun 1, 2015 at 8:35 AM, Matthew Henderson <
<EMAIL_ADDRESS>wrote:

Done. Had to load the Google font seeing the @import() didn't deliver
it
through the stylesheet.
On Sun, May 31, 2015 at 7:29 PM, Caspar<EMAIL_ADDRESS>wrote:

One more thing: could you make the font-weight lighter to look more
in
line with http://www.kittysplit.com/help?
—
Reply to this email directly or view it on GitHub
<

https://github.com/caspii/kittysplit-blog/issues/3#issuecomment-107226089>

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/caspii/kittysplit-blog/issues/3#issuecomment-107322327>

.

--
Caspar +491635561322 Berlin, Germany
—
Reply to this email directly or view it on GitHub
<
https://github.com/caspii/kittysplit-blog/issues/3#issuecomment-107329525>

.

—
Reply to this email directly or view it on GitHub
https://github.com/caspii/kittysplit-blog/issues/3#issuecomment-107330355
.

--
Caspar +491635561322 Berlin, Germany

Finally deployed this.


feat(optimizer): support agg group by simplify rule

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Resolve https://github.com/risingwavelabs/risingwave/issues/9578
Use first_value to replace redundant group by key, so that first_value could be purged by the following column pruning optimization.

Example:
before:
select count(*) from sbtest1 group by id, k, c;

use functional dependencies  (id) ==> (k, c)

after:
select count(*) from sbtest1 group by id;

Checklist

[ ] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Furthermore, will we support that?
CREATE TABLE t(v int, k int primary key);
SELECT v, k from t group by k;
ERROR:  QueryError: Invalid input syntax: column must appear in the GROUP BY clause or be used in an aggregate function

To be compatible with PostgreSQL, I think we don't need to support it, although other databases like MySQL support this feature.

Maybe we need a any_value (exists in DuckDB while its semantic is first non-null value) or arbitrary_value instead of first_value. Currently streaming impl of first_value requires an order and thus a state table...

Use first_value to replace redundant group by key, so that first_value could be purged by the following column pruning optimization.

How does column pruning purge the first_value. Seems like it is still present in some of the output plans?

If those columns haven't been used, they could be pruned, however, if they are still being used, they will present.

Maybe we need a any_value (exists in DuckDB while its semantic is first non-null value) or arbitrary_value instead of first_value. Currently streaming impl of first_value requires an order and thus a state table...

Yes, but for each group key, this rule can ensure that there is always one row in the first_value state table corresponding to it.

Maybe we need a any_value (exists in DuckDB while its semantic is first non-null value) or arbitrary_value instead of first_value. Currently streaming impl of first_value requires an order and thus a state table...

Yes, but for each group key, this rule can ensure that there is always one row in the first_value state table corresponding to it.

Aren't there #input rows of each group rows of the same value?

Yes, but for each group key, this rule can ensure that there is always one row in the first_value state table corresponding to it.

Exactly not :hot_face: there are multiple records in each group
CREATE TABLE User{
    id int Primary key,
    name varchar,
}

CREATE TABLE Events {
    user_id int,
}

SELECT user_id, name, count(*) FROM User
Join Event On Events.user_id = User.id
Group by user_id, name;


Need to find logfiles.

Where do SfB 2015 on-prem drop logfiles?

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

ID: 888fcfa7-e18a-f60d-734c-a38293e2baf0
Version Independent ID: c85ef7a6-9323-5828-4544-c284f821c242
Content: Centralized Logging Service in Skype for Business 2015
Content Source: Skype/SfbServer/management-tools/centralized-logging-service/centralized-logging-service.md
Product: skype-for-business-itpro
GitHub Login: @jambirk
Microsoft Alias: jambirk

Thank you for this feedback and for contributing to the wellness of the content! This is a good idea and we have added it into our backlog for triage and prioritization.

@tseward Is investigating this.

Hi, thanks for your feedback.
You can find that information in attribute ETLFileFolder that is showed in Get-CsCLSConfiguration, cmdlet described in the referenced article.
Thank you


Fix 2 issues

Kick ppl from Under The Bridge and make bot say whole sentences.
Resolves #26
Resolves #17

Resolves #26

@szyjah Also please use Guild.afk_channel to get current afk channer. Just use a check if after channel is guild's afk channel

Apllied suggested change.
Regarding #17 : I revorted wrong commit. Upsie.


Clone user with permissions

Would that be possible? to clone user and just change email name and password for new user but preserve all permissions?

Thanks for the request @akurzawa.
I can see how this would be useful but I'm not sure how it can be intuitively be shown as an option in the interface. You can't exactly have a button that says "Clone user with permissions but not email, name or password". I think the use-case may be minimal for the extra UI and process it would require.
I'll leave this open though to judge if this is a pain-point for others.

Thanks for the request but I'm going to proceed and close this off since there's been little demand for this feature since the original request.


fix: error TS2339: Property 'VISUALIZE' does not exist on type 'typeof TargetLanguage'

We are using the @internal directive to instruct typescript compiler to drop some statements during compilation.
This may cause the compiler to emit invalid declarations if we are not careful.
To guarantee that our declarations are valid, we do an additional run of tsc on the output.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

💚 All backports created successfully

Status
Branch
Result

✅
maintenance/v5.2

✅
maintenance/v5.3

Questions ?
Please refer to the Backport tool documentation and see the Github Action logs for details


Update mods for latest version.

Atm I only know of 2 mods that require an update to be compatible with the latest stable version of the game, Miju Games renamed/changed a few internals that's breaking compatability.
Currently known broken mods:
Recycle Veggies - Causes recycler to not recycle.
Recipe Import and Export - Won't export recipes due to missing internal field.

Disable Build Constraints - dosen't activated

That one is a keybind toggle, I know it still works.
On Sun., Oct. 30, 2022, 2:40 p.m. zeonler, @.***> wrote:

Disable Build Constraints - dosen't activated
—
Reply to this email directly, view it on GitHub
https://github.com/TysonCodes/PlanetCrafterPlugins/issues/2#issuecomment-1296323827,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGFQE6M6KLA5BOMJW7SID3WF2XIHANCNFSM6AAAAAARGQFZ2Y
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ore extractor also does not work any more. Breaks also the water collector and the water filter .


Allow Minting from Mintlist

Feature overview:
https://www.loom.com/share/22e0fe0f7f264529a719046d01d0d8a2
Extras:

The nft card component used both on the mintlist and collection page has been extracted into the NftCard component. cc @benborgers

QA:
You can use the following mintlist to test on devnet: https://nftoken-git-l-5182-allow-minting-from-mintlist.luma-dev.com/mintlist/heT9rmaXNQNttfwe5vPn6xymgEJg33cMDnNDGF6RuRo?network=devnet

@vpontis

What do you think about putting them both in their own pages?

I agree that would make a lot of sense. Shall we do this in a separate PR though?

@vpontis

What do you think about putting them both in their own pages?

I agree that would make a lot of sense. Shall we do this in a separate PR though?

Sure! I already approved this so feel free to merge it in


[doc] update docs/reference/commandline/network_inspect.md

The following sentences (introduced in v1.11 via #21160) were misleading for Swarm mode services (>= v1.12)

For networks backed by multi-host network driver, such as Overlay,
this command also shows the container endpoints in other hosts in the
cluster. These endpoints are represented as "ep-{endpoint-id}" in the output.

Update #24684
Update #24186
Signed-off-by: Akihiro Suda<EMAIL_ADDRESS>

cc @mavenugo

cc @mstanleyjones

updated PR
diff --git a/docs/reference/commandline/network_inspect.md b/docs/reference/comm
andline/network_inspect.md
index 943f329..85d4045 100644
--- a/docs/reference/commandline/network_inspect.md
+++ b/docs/reference/commandline/network_inspect.md
@@ -39,8 +39,8 @@ The `network inspect` command shows the containers, by id, in
its
 results. For networks backed by multi-host network driver, such as Overlay,
 this command also shows the container endpoints in other hosts in the
 cluster. These endpoints are represented as "ep-{endpoint-id}" in the output.
-However, for Swarm mode services, only the endpoints that are local to the node
-are shown.
+However, for swarm-scoped networks, only the endpoints that are local to the
+node are shown.

 You can specify an alternate format to execute a given
 template for each result. Go's


java.net.BindException: Address already in use: connect on HTTP Proxies

Hi all,
I'm encountering a bind error whenever I use a single rotating HTTP Proxies with a high amount of threads (800).
The error comes in at about 10-16k requests to the same proxy, which is interesting because as far as I can tell, HTTP Proxy sockets are constructed the same way as normal no proxy sockets.
I'm using SmartProxy as a proxy solution right now and it happens under them, but should happen with any provider or a Squid server.
The full Stacktrace is:
at java.base/java.net.PlainSocketImpl.waitForConnect(Native Method)
	at java.base/java.net.PlainSocketImpl.socketConnect(PlainSocketImpl.java:107)
	at java.base/java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:399)
	at java.base/java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:242)
	at java.base/java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:224)
	at java.base/java.net.SocksSocketImpl.connect(SocksSocketImpl.java:403)
	at java.base/java.net.Socket.connect(Socket.java:591)
	at okhttp3.internal.platform.Platform.connectSocket(Platform.kt:126)
	at okhttp3.internal.connection.RealConnection.connectSocket(RealConnection.kt:268)
	at okhttp3.internal.connection.RealConnection.connectTunnel(RealConnection.kt:234)
	at okhttp3.internal.connection.RealConnection.connect(RealConnection.kt:170)
	at okhttp3.internal.connection.ExchangeFinder.findConnection(ExchangeFinder.kt:236)
	at okhttp3.internal.connection.ExchangeFinder.findHealthyConnection(ExchangeFinder.kt:109)
	at okhttp3.internal.connection.ExchangeFinder.find(ExchangeFinder.kt:77)
	at okhttp3.internal.connection.Transmitter.newExchange$okhttp(Transmitter.kt:162)
	at okhttp3.internal.connection.ConnectInterceptor.intercept(ConnectInterceptor.kt:35)
	at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:112)
	at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:87)
	at okhttp3.internal.cache.CacheInterceptor.intercept(CacheInterceptor.kt:82)
	at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:112)
	at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:87)
	at okhttp3.internal.http.BridgeInterceptor.intercept(BridgeInterceptor.kt:84)
	at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:112)
	at okhttp3.internal.http.RetryAndFollowUpInterceptor.intercept(RetryAndFollowUpInterceptor.kt:71)
	at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:112)
	at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:87)
	at io.kekr.core.base.Kekr$launchCheck$1$invokeSuspend$$inlined$-addInterceptor$1.intercept(Interceptor.kt:82)
	at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:112)
	at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:87)
	at okhttp3.RealCall.getResponseWithInterceptorChain(RealCall.kt:184)
	at okhttp3.RealCall.execute(RealCall.kt:66)
	at io.kekr.core.connection.OkHttpHelperKt$await$2$1.invokeSuspend(OkHttpHelper.kt:56)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
	at kotlinx.coroutines.DispatchedTask.run(Dispatched.kt:238)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)

Using OkHttp 4.0.1 but also happens on 3.14.0
Thanks!

Update:
I found the exact error. SmartProxy uses a session based system, so every Proxy Authenticator instance has to be different. This makes every Address distinct, so OkHttp cannot pool connections and we wind up with this error. I was using Proxy Authenticator as a backup authentication mechanism if for whatever reason my Interceptor failed. Removing that fixes the issue, had to look at okhttp code first.
I would document this somewhere as it could be irritating for someone in my same position.

This is a better question for stackoverflow.
For now, I guess limit concurrency.  But it seems like this problem exists outside of OkHttp, just by usage.

@mcjp78 which docs did you read prior to encountering this? We wanna put documentation where it'll be useful!

I read the Connection Pool documentation, and then had to take a peek at the Address.kt class and the .equals() method to find it wouldn't pool.
Not really sure where the best place to put this info is.
I think this is fixable by allowing connections to the same server but different authenticators to pool, not sure if this is totally possible but I'm assuming it is.

Oh and limiting concurrency won't fix the issue. Windows can't close and reuse TCP connections for two minutes. You would be able to execute 5-7k requests but then I suspect it would fail. I tested this and it seemed accurate

Thinking about it for a sec, simply removing the Authenticator equality check in Address#equals should do the trick.

I don't think we should remove it from Address#equals because that already isn't the complete story.  equals is used for the quick path, but we do a slower second path to look for connection coalescing opportunities.
If you look at both Address#equalsNonHost and RealConnection#isEligible, you can see the logic.
But it isn't clear to me that OkHttp can know that proxy auth changing doesn't taint the connection.  So I think OkHttp is doing the correct thing here.
Just looking into it for smartproxy, I don't follow why you want a different Authenticator?  Can't you use a single Authenticator, but make the logic inside the Authenticator conditional?  potentially split out whatever is different and put it in a payload of the Request tag?

They have a session control system, so adding -session to the proxy username changes the exit ip. I can devise a workaround for it with a conditional authenticator, but it gets very messy very quickly. The only thing that changes in that is the Proxy-Authentication header, which when I'm using https i have to control with the Authenticator.

I don't think we should remove it from Address#equals because that already isn't the complete story. equals is used for the quick path, but we do a slower second path to look for connection coalescing opportunities.
If you look at both Address#equalsNonHost and RealConnection#isEligible, you can see the logic.
But it isn't clear to me that OkHttp can know that proxy auth changing doesn't taint the connection. So I think OkHttp is doing the correct thing here.
Just looking into it for smartproxy, I don't follow why you want a different Authenticator? Can't you use a single Authenticator, but make the logic inside the Authenticator conditional? potentially split out whatever is different and put it in a payload of the Request tag?

isEligible requires .equalsNonHost to be true, which will never happen because the proxy authenticator's don't equal. No coalescing, no pooling, but nothing differs beyond that.

I’d argue that proxy auth being different could be significant. OkHttp can’t know it’s not.

It for sure is significant, but how is it significant for connection pooling? I don't see how it modifies anything with regards to that.

Is the proxy send you to different origin servers based on the with credentials.

Is the proxy send you to different origin servers based on the with credentials.

The proxy sends via different IPs based on that but no not different origins. I can see how that may be an issue though, pooling wouldn't work right.
In that case, maybe provide a way to force it to pool?

I don't think this feature request fits our security goals. The downsides of sharing authenticators are many; the upsides are few!


Use a USB Webcam?

rpi-deep-pantilt version: 1.0.0 (2019-12-01)
Python version: 3.7
TensorFlow version: 2.2
Operating System: Raspberry OS

Description
Is it possible to use a regular USB Webcam? What changes do I have to make to make it run?
Additional question: When I run a coral model on object detection I get a max 8 FPS. Would that be faster using a Pi Cam?
Regards,
Armin

Hey @arminf82! I haven't tried a USB web cam. The performance is definitely going to depend on the camera's frame rate, USB 2 vs 3, and how efficiently the camera's driver can serialize data.
The PiCamera uses a MIPI serial interface, which is specifically designed for high-speed image data transfer. My gut feeling is that it'll have a higher bitrate on the Pi than most USB cameras. If I were trying to compare the two, I'd write a tiny program that benchmarks how fast each method can read/write to a ring buffer.
https://www.mipi.org/specifications/csi-2
You could also use cprofile to see if the inference loop is waiting on buffer being available to read.
Here's the line to replace if you implement a USB image stream. Replace this .read() method with one that takes the most recent frame from your buffer, then zeroes the buffer.
https://github.com/leigh-johnson/rpi-deep-pantilt/blob/master/rpi_deep_pantilt/detect/camera.py#L93

Closing for now, so it's easier for me to see issues in need of triage at a glance. Good luck with this!

I bought a PiCam v2. I had two different USB Cams and both of them were much slower.  Thank you.


Add a shortcut key to execute queries

It would be great if hitting something like Control + Enter or F5 would execute a query.

Oh, looks like F5 actually does this.  Would be helpful to add this information to the "Execute" tooltip.


iOS Version 6.0.1 Trezor Sending 0 XTZ to KT address when Inputting to tz address

I had an older version of TexBox, and my transactions were not showing.
So I updated to the 6.0.1 Hotfix version, and reloaded a Trezor wallet from scratch. The transactions are now showing, that's good.
But, I go to send a transaction from a KT address to a tz address:

And notice that the trezor screen is still sending it back to the KT address, not showing the tz address, and is also showing sending 0 XTZ (with 0.05 fee).

Followed by error:

My KT funds are still shown as "delegated". Maybe I need to "undelegate" them first, I'm not sure, but when I go to the Delegate tab, it says "Delegation has changed with the Babylon upgrade - you can now delegate directly from your tz* address. If you want to change your delegate, please transfer your funds to your tz* address and delegate from there. We will re-add delegation for KT1 addresses in the future."

Bump, so what's goin on here, Trezor is showing "Send 0 XTZ", when I'm trying to send 10.  What to do.

Should I try running this on Chrome, or Windows, or some other platform? Maybe I'll have better luck? Thanks


[proposal] google/go-containerregistry as replacement for regclient library to use cosign oci package

Is your feature request related to a problem? Please describe.
Fetching artifacts from container images is currently done by accessing the registry with regclient. Accessing the SBOM and attestation references requires manually constructing the registry references with string interpolation.
Describe the solution you'd like
Cosign, one of the pioneers of storing security artifacts inside OCI registries, offers a library to access SBOM, signature, and attestation references by using their oci package, which tightly integrates with google/go-containerregistry.
Therefore, using google/go-containerregistry as the main library to access OCI content on registries should benefit in the long run when integrating with the sigstore landscape.
Describe alternatives you've considered
An alternative library to use would be oras-go, which would not fix the problem described above.
Additional context
google/go-containerregistry is spread much wider and adopted by more projects than the oras-go or regclient library. It is used as a direct dependency by 411 repositories.
deps.dev shows that in the newest version, regclient is adopted by 4 repositories (including guac), while oras-go has 18 direct adoptions.
Open for discussion
This issue should be used to discuss this change moving to google/go-containerregistry and the impact on this project. It came up from a discussion in #1449.

cosgin has too many dependencies https://deps.dev/go/github.com%2Fsigstore%2Fcosign/v1.13.1/dependencies and must be factored into the decision making.
That is one of the reasons https://github.com/sigstore/sigstore-go was built.

cosgin has too many dependencies https://deps.dev/go/github.com%2Fsigstore%2Fcosign/v1.13.1/dependencies and must be factored into the decision making.
That is one of the reasons https://github.com/sigstore/sigstore-go was built.

sigstore-go seems to be more for verifying the signatures/attestations instead of fetching them from container images at the moment. I'm not sure what the roadmap looks like for the library.

Hi @hown3d, thanks for creating the issue. It sounds like the intent behind this change is to move towards "integrating with the sigstore landscape". If GUAC were to collect signatures from other sources such as Notary, should we tightly couple to the OCI package for sigstore?

FYI @sudo-bmitch who may also have input here

Accessing the SBOM and attestation references requires manually constructing the registry references with string interpolation.

If I understand the concern correctly, storing metadata using the cosign tags is considered a temporary state until OCI is able to GA the subject/referrers work. Once that's done, sigstore has indicated they intend to transition to using the subject/referrers method to store their metadata (it's already supported with experimental flags), same at notation has been doing, along with various other projects.
regclient isn't going to add all of the signing libraries of cosign to replicate its functionality (as mentioned, that adds a lot of dependencies). So if verification of the signatures is needed, I'd understand if you wanted to go that direction.

It would be good to move to the current verifier to start using https://github.com/sigstore/sigstore-go as mentioned by @naveensrinivasan.


Simplified STDIO

What are you trying to accomplish with this PR?
This PR fixes #25 among other things, simply by replacing the custom logging (hooked to the child process' stdout) with the stdio: inherit option flag in child_process.spawn.
From the Node.js docs (link):

'inherit' - Pass through the corresponding stdio stream to/from the parent process

This is beneficial for a few reasons:

Output is formatted the same as Theme Kit's (more consistent = better?)
ANSI colours aren't lost when piping child process output
Fewer lines of code to maintain and test
More faithful to the Node.js API's intended purpose

Some Notes

This PR shouldn't be merged in until Theme Kit 0.8.1 gets released and integrated into this package because of an issue with error logging introduced in the 0.8.0 Theme Kit binary. See Shopify/themekit#562.

I removed a unit test that spied on the now-removed custom logging entirely. It was kind of a bad unit test anyways (relied on testing the binary's output), so I'll be opening a separate PR that properly tests the contents of install.js. (See #28)

Checklist
For contributors:

[x] I have updated the documentation in the README file to reflect these changes, if applicable.

For maintainers:

[ ] I have :tophat:'d these changes.
[ ] I have bumped the package.json version in a separate PR, if applicable.

Considerations:

Changes may require updates to the Slate docs or the @shopify/slate-tools package.

@t-kelly Theme Kit has been updated! I opened a separate PR #29 to update the wrapper's TK version.


Introduce byte counting to the transport API

Stats are published on the 'recv_trailing_metadata' edge, which is the indicator to the channel pipeline that the call is finished.
Still a little buggy: grpc headers are not accounted as framing bytes on the outbound path.
No tests yet. For discussion only.

I think all comments have been addressed. PTAL.

+1 let me know if you want me to merge this.

I saw some failures here:
https://grpc-testing.appspot.com/job/gRPC_pull_requests/7424/
Python seems to be more flaky than what we usually see, do you mind taking another look (and rerunning tests if things seem alright).


Initial setup issue

I've followed the instructions, but I keep getting this error:

Argument of type 'PrismaClient<{ errorFormat: "minimal"; }, never, false>' is not assignable to parameter of type 'IPrisma'.
The types of 'session.create' are incompatible between these types.
Type '(args: SelectSubset<T, SessionCreateArgs>) => CheckSelect<T, Prisma__SessionClient, Prisma__SessionClient<...>>' is not assignable to type '(args: ICreateArgs) => Promise'.
Types of parameters 'args' and 'args' are incompatible.
Type 'ICreateArgs' is not assignable to type '{ select?: SessionSelect; data: (Without<SessionCreateInput, SessionUncheckedCreateInput> & SessionUncheckedCreateInput) | (Without<...> & SessionCreateInput); }'.
Types of property 'data' are incompatible.
Type 'ICreatePrismaSession' is not assignable to type '(Without<SessionCreateInput, SessionUncheckedCreateInput> & SessionUncheckedCreateInput) | (Without<...> & SessionCreateInput)'.
Type 'ICreatePrismaSession' is not assignable to type 'Without<SessionUncheckedCreateInput, SessionCreateInput> & SessionCreateInput'.
Type 'ICreatePrismaSession' is not assignable to type 'Without<SessionUncheckedCreateInput, SessionCreateInput>'.
Types of property 'id' are incompatible.
Type 'string' is not assignable to type 'never'.
My page imports look like this:
import expressSession from 'express-session';
import { PrismaClient } from '@prisma/client';
const prisma = new PrismaClient({ errorFormat: 'minimal' });
import { PrismaSessionStore } from '@quixo3/prisma-session-store';

Hi @mjknight50,
I've just tried the following minimal server, which I think incorporates the same imports and app.use expression from your notes above:
import express from 'express';
import expressSession from 'express-session';
import { PrismaClient } from '@prisma/client';
const prisma = new PrismaClient({ errorFormat: 'minimal' });
import { PrismaSessionStore } from '@quixo3/prisma-session-store';

// rest of the code remains same
const app = express();

app.use(
  expressSession({
    cookie: {
      maxAge: 7 * 24 * 60 * 60 * 100,
    },
    secret: 'a santa at nasa',
    store: new PrismaSessionStore(
        prisma,
        {
          checkPeriod: 2* 60 * 1000,
          dbRecordIdIsSessionId: true,
          dbRecordIdFunction: undefined,
        }
    ),
  })
)

const PORT = 8000;
app.get('/', (req, res) => res.send('Express + TypeScript Server'));
app.listen(PORT, () => {
  console.log(`⚡[server]: Server is running at https://localhost:${PORT}`);
});

This seems to work for me. At least, I'm not encountering any warning / error re: the prisma parameter.
If you use this code sample, do you still encounter the same issue?

Hi @kleydon !
Thanks for taking the time to help me out...
Yes, I get that same error copying your sample code.
I am on:
"prisma": "^2.24.1",
"@prisma/client": "^2.24.1",
"@quixo3/prisma-session-store": "^3.0.1",
"express": "^4.17.1",
"express-session": "^1.17.2",
Here is my tsconfig in case that helps too:
{
"compilerOptions": {
"declaration": false,
"noUnusedLocals": true,
"emitDecoratorMetadata": true,
"experimentalDecorators": true,
"lib": ["es2020", "dom"],
"removeComments": false,
"sourceMap": true,
"module": "commonjs",
"esModuleInterop": true,
"moduleResolution": "node",
"outDir": "../dist/",
"target": "es2020",
"allowJs": false,
"typeRoots": ["../node_modules/@types"],
"baseUrl": "."
}
}

@mjknight50 hmmm...
The module versions I've used for testing on my end are slightly different, but I'd be surprised if they account for the problem.
Here are all the files for my test case (below). You'll still need to run npm/yarn install, for all the node modules. Perhaps you can use this as a base, and progressively, slowly add in changes from your test, until the problem reveals itself? I'd love to know what's going on, if you do find something amiss.
test-server.zip
Another approach might be to create a project/branch that reproduces what you are seeing, so that we're all looking at the same thing.
Best,
-K

@kleydon
Using your sample code, without changing anything, I get a slightly different error:
{
	"resource": "/home/knight/Desktop/test-server/test-server/index.ts",
	"owner": "typescript",
	"code": "2345",
	"severity": 8,
	"message": "Argument of type 'PrismaClient<{ errorFormat: \"minimal\"; }, never, false>' is not assignable to parameter of type 'IPrisma'.\n  Property 'session' is missing in type 'PrismaClient<{ errorFormat: \"minimal\"; }, never, false>' but required in type 'IPrisma'.",
	"source": "ts",
	"startLineNumber": 18,
	"startColumn": 9,
	"endLineNumber": 18,
	"endColumn": 15,
	"relatedInformation": [
		{
			"startLineNumber": 44,
			"startColumn": 5,
			"endLineNumber": 44,
			"endColumn": 12,
			"message": "'session' is declared here.",
			"resource": "/home/knight/Desktop/test-server/test-server/node_modules/<EMAIL_ADDRESS>		}
	]
}

Here is some more info:
❯ npm -v
7.13.0
❯ node -v
v14.15.1
I tried with both yarn and npm because sometimes we've seen differences there.

@mjknight50 Not sure what's going on there.
Two thoughts:

Would it be possible for you to create a branch that includes everything needed to try to reproduce the problem?
Curious what happens when you try to run the above code sample within a docker environment, further isolating it from your local machine environment.

-K

I'm  also running into this error...

@sayerrrr, care to post a reproduction branch, to help diagnose?

@kleydon Hopefully this helps..
❯ npm -v
7.13.0
❯ node -v
v14.15.1
reproduction.zip

@mjknight50 This is happening because you did not run prisma generate after editing schema.prisma and the @prisma/client types does not include Session data model.
First add Session data model to schema.prisma (Codes are in README.md)
Then run prisma generate or npx prisma generate and you will be good to go.
Don't forget to close the issue if your problem got fixed.

@erfanasbari, thanks for your intuition.
@mjknight50, @sayerrrr: Could this explain what you are experiencing?
If not, let me know, and I'll test out the reproduction.zip file above.
-K

@erfanasbari Thanks for the help and idea, but that isn't it.  Same error message as originally reported.

@mjknight50 , @sayerrrr ,
When I...

Download reproduction.zip, posted above
Decompress
Enter the reproduction directory
Run npm install
Open reproduction/server/app.ts in VS Code locally (i.e not "reaching into" any docker container)
Look at line 48: store: new PrismaSessionStore(prisma, {

The word "prisma" does not, in my environment, have the squiggly underline depicted in the image at the top of this thread. So it seems that, so far, I'm not able to reproduce the problem you are seeing with this set of files.
In case its useful, here is my environment:
Mac OS 10.14
VS Code v1.57.0
npm v.6.14.7
node v14.9.0
Could it be that you have globally-installed npm packages (for prisma, or other dependencies) that are over-riding the local ones? (I don't know if this is even possible, but it was the only new thought I had...)
If the steps I've listed above don't seem like the right ones to do, in attempting to reproduce this issue, please let this forum know.
-K

I've spent all day today chasing this and I can't find the pattern.  I'm going to move on to something else but I wish you the best of luck with what I think is a nice idea.

Fair enough. Sorry it seems to have wasted a bit of your time, but perhaps its the sort of thing that becomes clear (or disappears) when one stop fixating on it? Not sure; hopefully time will tell.
At any rate, good luck with your project!
-K

On Jun 21, 2021, at 11:05 AM, Matthew Knight @.***> wrote:

I've spent all day today chasing this and I can't find the pattern. I'm going to move on to something else but I wish you the best of luck with what I think is a nice idea.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

fwiw I had a smilar error, caused by adjusting the session model to
id Int @id instead of id String @id

Thanks for the tip, @rikschennink.

Ran into this issue myself.
After changing my npx prisma generate output from the default one everything started to work.

this fixed it for me

@tobiasostman - thanks for posting this.
Not sure why an explicit, custom output location for the prisma client would make the difference - but glad it has!
(Maybe the symptoms described above are general; maybe they happen whenever the prisma client fails to successfully generate? Just a theory.)

I encounter the same error first time. The code at the README.md didn't work.
I guess the problem might be that instead of giving prisma from @prisma/client to PrismaSessionStore, we might want to create an instance of new PrismaClient() from @prisma/client.
Which means instead of
import expressSession from 'express-session';
import { PrismaSessionStore } from '@quixo3/prisma-session-store';
import { prisma } from '@prisma/client';

try
import expressSession from 'express-session';
import { PrismaSessionStore } from '@quixo3/prisma-session-store';
import { PrismaClient } from '@prisma/client';

and instead of
app.use(
  expressSession({
    cookie: {
     maxAge: 7 * 24 * 60 * 60 * 1000 // ms
    },
    secret: 'a santa at nasa',
    resave: true,
    saveUninitialized: true,
    store: new PrismaSessionStore(
      prisma,
      {
        checkPeriod: 2 * 60 * 1000,  //ms
        dbRecordIdIsSessionId: true,
        dbRecordIdFunction: undefined,
      }
    )
  })
);

try
app.use(
    expressSession({
      cookie: {
        maxAge: 7 * 24 * 60 * 60 * 1000, // ms
      },
      secret: 'a santa at nasa',
      resave: true,
      saveUninitialized: true,
      store: new PrismaSessionStore(new PrismaClient(), {
        checkPeriod: 2 * 60 * 1000, //ms
        dbRecordIdIsSessionId: true,
        dbRecordIdFunction: undefined,
      }),
    }),
  );

Hope it works for you !

Hi @alaeddine119,
Thanks for the feedback.
The approach you recommend - passing a new PrismaClient() into the store - seems like a good one, but it remains puzzling to me why passing in an existing prismaClient instance (e.g. prisma from the read-me) would cause problems... I don't understand this.
If anyone else is encountering the same behavior, please let us know.

Same issue. Passing in a new PrismaClient() instance works :shrug:
Using the imported client gives
Type 'typeof import(".../node_modules/.prisma/client/index").prisma' is not assignable to type 'Record<"session", { create(args: ICreateArgs): Promise<IPrismaSession>; delete(args: IDeleteArgs): Promise<IPrismaSession>; deleteMany(args?: unknown): Promise<...>; findMany(args?: IFindManyArgs | undefined): Promise<...>; findUnique(args: IFindUniqueArgs): Promise<...>; update(args: IUpdateArgs): Promise<...>; }>'

Typescript 4.6.3

Thanks for the help @mjknight50 / @alaeddine119 / @hbriese / @rikschennink ; finally fixed the docs on importing PrismaClient, in PR #86 (of prisma-session-store)


Compiling Benchmarks Folder returns undefined functions, libraries

Currently trying to compile benchmarks on a A53 board, the highkey. Was able to compile the OpenBLAS library:
make TARGET=ARMV8
and install, after adding the target line to this level as well:
make TARGET=ARMV8 PREFIX=/opt/Openblas install
However, making for the benchmarks folder doesn't seem to work. The result is below. A similar error is very apparent in the forums, but this seems to occur most often when OpenBLAS is being firstly compiled, due to not being able to detect the architecture, and is most easily resolved by adding the TARGET= argument. This doesn't seem to work in the benchmark file, though. The result is below. Also, references to an odd library as well, near the bottom, libopenblas_unknownp-r0.2.19.dev.a.
linaro@linaro-alip:~/P3/opt_libs/OpenBLAS/benchmark$ make
getarch_2nd.c: In function ‘main’:
getarch_2nd.c:12:35: error: ‘SGEMM_DEFAULT_UNROLL_M’ undeclared (first use in this function)
printf("SGEMM_UNROLL_M=%d\n", SGEMM_DEFAULT_UNROLL_M);
^
getarch_2nd.c:12:35: note: each undeclared identifier is reported only once for each function it appears in
getarch_2nd.c:13:35: error: ‘SGEMM_DEFAULT_UNROLL_N’ undeclared (first use in this function)
printf("SGEMM_UNROLL_N=%d\n", SGEMM_DEFAULT_UNROLL_N);
^
getarch_2nd.c:14:35: error: ‘DGEMM_DEFAULT_UNROLL_M’ undeclared (first use in this function)
printf("DGEMM_UNROLL_M=%d\n", DGEMM_DEFAULT_UNROLL_M);
^
getarch_2nd.c:15:35: error: ‘DGEMM_DEFAULT_UNROLL_N’ undeclared (first use in this function)
printf("DGEMM_UNROLL_N=%d\n", DGEMM_DEFAULT_UNROLL_N);
^
getarch_2nd.c:19:35: error: ‘CGEMM_DEFAULT_UNROLL_M’ undeclared (first use in this function)
printf("CGEMM_UNROLL_M=%d\n", CGEMM_DEFAULT_UNROLL_M);
^
getarch_2nd.c:20:35: error: ‘CGEMM_DEFAULT_UNROLL_N’ undeclared (first use in this function)
printf("CGEMM_UNROLL_N=%d\n", CGEMM_DEFAULT_UNROLL_N);
^
getarch_2nd.c:21:35: error: ‘ZGEMM_DEFAULT_UNROLL_M’ undeclared (first use in this function)
printf("ZGEMM_UNROLL_M=%d\n", ZGEMM_DEFAULT_UNROLL_M);
^
getarch_2nd.c:22:35: error: ‘ZGEMM_DEFAULT_UNROLL_N’ undeclared (first use in this function)
printf("ZGEMM_UNROLL_N=%d\n", ZGEMM_DEFAULT_UNROLL_N);
^
make: *** [getarch_2nd] Error 1
make: *** No rule to make target '../libopenblas_unknownp-r0.2.19.dev.a', needed by 'slinpack.goto'. Stop.

You have to use the same arguments to benchmark make which you used for the main library build. In this case it will be "make TARGET=ARMV8"

Don't know how I missed that. Thanks

Please close the issue if the problem is solved.
Thanks


Replace underscore dependency with lodash custom build

Did you consider to replace underscore with a custom Lo-Dash build in order to reduce the size of your component?

Custom Lo-Dash builds
Lo-Dash functions as npm packages

Thanks for posting this suggestion! We have been considering ways to remove the dependency on underscore but haven't made too much progress on that yet. Lo-Dash seems like it could fit the bill nicely.
The other idea that we are kicking around is to see if we can get by with using only the new ES6 features instead of using a library. It would be ideal to have Griddle depend only on React but we'll see how that works out. Any thoughts on this approach?

I think ES6 does not help you here. Replacing underscore with a Lo-Dash build that only contains the methods you are using would be very simple. You just have to make a custom build and replace var _ = require('underscore') with var _ = require('lodash'). It's the same API.

One thing to note is that Griddle's webpack is currently configured to externalize underscore so the user could choose whether they want to use underscore or lodash by including the CDN in their app.  Webpack more or less makes it so require('underscore') == window._.  No code changes necessary and reap the benefits of lodash.
If you're looking to squeeze those kb then the you could build your own lodash build and package it with your app.
This little bash script produces a ~6kb lodash subset and it seems to work fine against the Griddle docs
# depends on: npm install -g lodash-cli

# Search through all source producing comma delimited lodash functions
usedFuncs=$(
    grep -ohPr '_\.[a-zA-Z0-9]+' scripts |
    cut -d. -f2 |
    sort -u |
    tr "\\n" ",")

lodash include=${usedFuncs}

Perhaps this could eventually be part of the build process of Griddle in some shape but I think this sort of tuning should be left up to the user. What if I'm using lodash in my app and there's overlap? I may end up having some lodash funcs included twice in my bundle.

@scvnc That is a very slick script and thanks for trying that against the Griddle docs!
@moklick You are dead-on that by default ES6 doesn't serve as a replacement to underscore/lodash :+1: . I should have worded what I meant a little more clearly -- I am looking into what it would take to replace the underscore methods with functions written specifically for those tasks. It seems that using ES6 would make this easier, however, it's not something I want to approach too quickly. Both underscore and lodash are pretty solid.
I think using lodash is a great suggestion and that may make its way into a future version :+1: . I will go ahead and close this for now. Please follow up if the suggestion @scvnc suggested doesn't work out for you.


SAK-29205 Don't allow to change the gradebook item's title when some assignment sends grades to it

There are some other JIRAS related with gb item name instead of id. For instance SAK-23141. Is it time to change this behavior and solve all the related issues?

Yes is it possible to make the change to send the data by id instead of assignment title?

Yes, it is possible to fix this jira changing the code in Assignment tool instead of gradebook in the sense Steve and Matthew said. I was reviewing how I can made it but It is not as simple because there will be many lines to change. I don't know if we at UPV will have the resources to fix this.

I see, it's an expensive to resolve using ID's, and this PR would resolve some problems in short term. So do we go ahead and add this verification to avoid problems to the LMS administrators? :+1:

I think the PR isn't necessary yet because of SAK-29396

I believe the gb_gradable_object_t.external_id stores the assignment references (eg. /assignment/a/ba4a0d97-4a47-4131-a15f-43fd23ed9f88/1c9e60b8-32db-445d-a049-b23e311ffada). If this is the case, we might be able to use this to pull the original assignment more efficiently than grabbing all the assignments in the site.

I don't feel this PR has much usefulness in Sakai 11.

Gradebook is deprecated for GradebookNG and this doesn't do anything for Gradebook NG
This only deals with Assignments, other tools that link in (like Forums, possibly Lessons and Samigo) aren't handled
Realistically we'd want to allow title changes, but this would involve linking tools by id instead of title. This change would be better and captured on the subtasks under https://jira.sakaiproject.org/browse/SAK-30710

Closing due to inactivity. Please send a new PR if feature/fix is still desired.


maxConcurentCallsPerSession replacement when upgrading to Service Bus SDK v7?

Query/Question
What is the equivalent of maxConcurrentCallsPerSession for the new Azure Service Bus SDK? I've tried using maxConcurrentCalls, but it seems like it is not working. Thanks!
Why is this not a Bug or a feature Request?
It might be a feature request after my question get's answered.
Setup (please complete the following information if applicable):

OS: [e.g. iOS]
IDE: [e.g. IntelliJ]
Library/Libraries: com.azure:azure-messaging-servicebus:7.13.0

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ x ] Query Added
[ x ] Setup information Added

@ZejiaJiang could you please follow up with @danilobanjac on this question?

Hi @danilobanjac ,
we do use maxConcurrentCalls in latest SDK(which is V7), this parameter is used to receive message from all session like its naming. So the behavior is different from the lagacy SDK. I'd like to know what's your scenario and help you to use SDK.

Hi @ZejiaJiang,
I'm trying to achieve the following: I want to receive and process sessions concurrently, but the messages within the session are to be sequential. I've tried different combinations and could not come up with the solution. Therefore I've introduced locking, which I would like to avoid.

Hi @danilobanjac ,
It seems like a very basic scenario, could you provide some code snippet? And how many sessions do you have?

This is the session processor I've been using:
new ServiceBusClientBuilder().connectionString(                 azureServiceBusConfig.getConnectionString()             )             // Give priority to the library's retry mechanism             .retryOptions(new AmqpRetryOptions().setMaxRetries(0))             .sessionProcessor()             .disableAutoComplete()             .maxConcurrentCalls(1)             .maxConcurrentSessions(10)             .maxAutoLockRenewDuration(                 Duration.ofSeconds(30)             )             .prefetchCount(0)             .receiveMode(ServiceBusReceiveMode.PEEK_LOCK)             .topicName(azureServiceBusConfig.getAlertsTopicName())             .subscriptionName(azureServiceBusConfig.getAlertsTopicSubscriptionName())             .processMessage(alertMessageProcessor::processMessage)             .processError(alertMessageProcessor::processError)             .buildProcessorClient();

I've tried also different combinations for maxConcurrentCalls  and maxConcurrentSessions  but I always get messages within the session also in a concurrent mode. This is causing a race condition in my code.

Thank you @danilobanjac , I got your point, you mean that even under a same session, the message are not sequential, thats cause a race condition in your code.
I'll try the code. BTW, I see that you set 100 sessions in the code, do you have 100 sessions? just want to ensure this to repro this issue.

Thank you. I'll try to repro the sequential issue in one session first.

Hi @danilobanjac ,
After testing and discussing with my coworker,  the session processor process message with maxConcurrentSessions  > 1 has ordering issue is a known issue. My coworker did this before, and found that code need to be changed a lot and add more error handlings. Thus, we haven't fixed this yet. I'm trying to fix this base on my coworker's effort. It need some time to refactor related code and test new code.
I'm sorry for this issue and here is a workaround, you can create a ServiceBusSessionReceiverAsyncClient for each sessionId and you need to manage all receiver clients by yourself. or use ServiceBusProcessorClient with maxConcurrentSessions  is 1.

Thanks for your effort @ZejiaJiang .
I've the same problem as @danilobanjac . I use the sessions to have a FIFO Subscription, but if I set the maxConcurrentCalls > 1, my messages listener gets called by multiple threads for the same session. I must set the maxConcurrentCalls=1 and maxConcurrentSessions=1 but it is very inefficient since I may have a lot of sessions at the same time.

This is a known issue. This is part of our work in: https://github.com/Azure/azure-sdk-for-java/issues/33706

We are also stuck on this issue.
We want to concurrently process multiple sessions. But within each session, process message one after the other (we dont care whether sequence is maintained or not).
In its absence, the processor class is not useful in case of sessions.

Reaching out on this :)

Seems to be fixed in the azure-messaging-servicebus 7.15.0 release.
See https://learn.microsoft.com/en-us/azure/developer/java/sdk/troubleshooting-messaging-service-bus-overview#upgrade-to-715x

Yes, this is addressed in the 7.15.x version, closing.


Add bank icon in header in Plaid flow

Details

This PR adds the bank icon on the Plaid bank account selection page when the bank icon is available. When it isn't, we default to the generic bank icon.
Fixed Issues

$ https://github.com/Expensify/Expensify/issues/166782
Tests

From an account that has a workspace, but no cards set up:

Click on profile image
Click on workspace name to go to workspace settings
Click Expensify Card
Click Get Started
Click Log into your bank
Click Continue
Choose Chase
Enter user_good and pass_good for credentials
Click Continue
Land on account selection page and make sure Chase logo is showing before the text and size/alignment looks right
Click back arrow
Click Log into your bank again
userClick Continue
Choose Wells Fargo
Enter user_good and pass_good for credentials
Click Continue
Land on account selection page and make sure default bank icon is showing before the text and size/alignment looks right

QA Steps

From an account that has a workspace, but no cards set up:

Click on profile image
Click on workspace name to go to workspace settings
Click Expensify Card
Click Get Started
Click Log into your bank
Click Continue
Choose Chase
Enter user_good and pass_good for credentials
Click Continue
Land on account selection page and make sure Chase logo is showing before the text and size/alignment looks right
Click back arrow
Click Log into your bank again
userClick Continue
Choose Wells Fargo
Enter user_good and pass_good for credentials
Click Continue
Land on account selection page and make sure default bank icon is showing before the text and size/alignment looks right

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

cc @shawnborton just double-checking that all the styles look good.

Looks good to me!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by @chiragsalian in version: 1.1.6-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Predict Upcoming

Everything works up to the point of Predicting Upcoming
Date parsing error regardless of the date selected

Hello,
Thanks for contacting me. It will be fixed in the next update! For now, you can manually predict matches. There is an option for that!
Best Regards
-V.K.
From: @.>
Sent: Tuesday, December 13, 2022 2:14 PM
To: @.>
Cc: @.***>
Subject: [kochlisGit/ProphitBet-Soccer-Bets-Predictor] Predict Upcoming (Issue #3)
Everything works up to the point of Predicting Upcoming
Date parsing error regardless of the date selected
—
Reply to this email directly, view it on GitHubhttps://github.com/kochlisGit/ProphitBet-Soccer-Bets-Predictor/issues/3, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALUJURCTEO4ZDKNZNIUBCY3WNBSBHANCNFSM6AAAAAAS5EJJYM.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

following


Color helpers

Hi I want to propose following color helpers:
type
  RGB = tuple
    R: int
    G: int
    B: int

proc `&`*(x: RGB): cstring =
  &x.R & cstring"," & &x.G & cstring"," & &x.B

proc rgb*(rgb: RGB): cstring =
  cstring"rgb(" & &rgb & cstring")"

proc rgba*(rgb: RGB, alfa: float): cstring =
  cstring"rgba(" & &rgb & cstring"," & &alfa & cstring")"

Usage:

const WhiteColor* = (R: 255, G: 255, B: 255)

tdiv(style=style(StyleAttr.color, rgba(WhiteColor, 0.5)):
  text "Test"

Would be better to have constraints on R, G, B to be not out of the range [0;255]


Follow-up for fixing IDE/SourceKit tests after changes for [SE-0111]

What's in this pull request?
Fixes for IDE/SourceKit tests addressing changes related to [SE-0111].
Resolved bug number: (SR-)
rdar://27642873

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test

@swift-ci smoke test OS X platform

@benlangmuir please review.

@swift-ci smoke test OS X platform

@swift-ci Please test

@swift-ci smoke test OS X platform

@swift-ci Please test

LGTM

Linux test had a build timeout (30 mins timeout for some reason, probably a mistake), merging.


Upgrade typescript to 2.1.4

ERROR in ./tests.webpack.js
Module not found: Error: Can't resolve './tests' in '/home/travis/build/phovea/phovea_bundle_lib'
 @ ./tests.webpack.js 12:14-61
ERROR in [default] /home/travis/build/phovea/phovea_bundle_lib/node_modules/phovea_core/src/range.ts:1283:40
Argument of type 'Range1D[]' is not assignable to parameter of type 'Range1DGroup[]'.
  Type 'Range1D' is not assignable to type 'Range1DGroup'.
    Property 'name' is missing in type 'Range1D'.

and others


Add acorn image history command

Add an acorn image history <image> command that prints an ordered set of updates that have been made to a given acorn image over its lifespan. This command is analogous to git log for an acorn image reference.
(more details tbd)

Hey @njhale , Can I work on this issue?

Hi @imrajdas,
As written, I think this issue is a bit underspecified to start just yet; i.e. it needs design work and a clear picture of what the UX for the command should look like.
That being said, if you have any ideas design/UX-wise please feel free to post them here.


resolved priority bug

The priority wasn't changing, after editing a task's priority.
Also validation to priority is also added. If someone doesn't changes the priority, then error message displays.
Screenshots:

By the way check PR #296 ... their is visible UI bug.

align the tasks also then send the pr

align the tasks also then send the pr

the tasks are aligned now. Check the new pr

Thanks for your Valuable contribution.
Don't forget to star the repo.


import package auto suggestion not working

Issue Type: Performance Issue
import package auto suggestion not working
VS Code version: Code 1.57.0 (Universal) (b4c1bd0a9b03c749ea011b06c6d2676c8091a70c, 2021-06-09T17:22:31.215Z)
OS version: Darwin arm64 20.5.0
Restricted Mode: No

related #126020

Please share an example and steps to reproduce
/needsMoreInfo


Access to other models?

Hi, can you provide other models for ex. the one with CTC instead of attn.

OK, it takes about 2days.

Hello,
I just uploaded 2 CTC models (1,2) at here

None_VGG_None_CTC
None_VGG_BiLSTM_CTC
and we are training 2 models (3,4) as well.
None_ResNet_None_CTC
TPS_ResNet_BiLSTM_CTC
3,4 will be also uploaded soon.

Best

I just uploaded 3,4 models :)
3. None_ResNet_None_CTC
4. TPS_ResNet_BiLSTM_CTC


Initialise Data Analysis

Through detailed examination of the testing methodologies, tools, and frameworks employed across these repositories, this branch will work towards understanding common patterns and identifying areas for improvement or standardisation. The insights gained will potentially lead to the development of strategies that can be applied across various projects, enhancing their testing processes and overall software quality.

Add src/DS/metrics_extraction.py script

Introduced a read_and_parse_file utility function for handling file reading and AST parsing, reducing code duplication and improving error handling.
Enhanced logging across the script:

Added detailed logging for each step in the file analysis process, including DEBUG, INFO, WARNING, and ERROR levels.

The script loads an existing DataFrame if available, allowing it to continue from where it left off.
Improved the use of dictionary unpacking (**) for merging analysis results into the final DataFrame, ensuring clean and maintainable code.
Refactored the script to introduce batch processing for saving results, reducing the frequency of I/O operations and improving overall performance. Results are now accumulated in memory and written to the output CSV file in batches (default: 100 files per batch).
Added argparse for command-line argument parsing, allowing users to specify input and output file paths.

This is related to the issue #84


Updating dependecies

Description
Updating PL, torchmetrics and pytest to the newest version
How to Test/Run?
pytest

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@45b1a4e). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev     #136   +/-   ##
======================================
  Coverage       ?   78.75%
======================================
  Files          ?       59
  Lines          ?     3144
  Branches       ?        0
======================================
  Hits           ?     2476
  Misses         ?      668
  Partials       ?        0

Flag
Coverage Δ

cpu
78.75% <0.00%> (?)

pytest
78.75% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Carousel utilizes buttons

Buttons are a Discord feature that replaces buttons in a much neater way. It does not require the old and slow reaction system.
Preview:

looks good!


'External' bundle no longer works if the URL is local

Andrew,
After upgrading to 0.9.2 I have that the use of local external references no longer works due to the check for IsUrl() in src/Cassette/Scripts/ExternalScriptBundle.cs (ln 52).
We have a JS file that is generated on the fly from the server which we want to manage as if it was external. Prior to upgrading I was able to add an external module using a local path (e.g. './myFIle.js') which worked fine.
Can this be reverted to allow relative URLs again.
Thanks,
Adam

I'll look into removing the constructor constraint requiring a URL. I don't think it adversely affects other code - just need to check...
It would interesting to know more about your use case as well. Are you able to tell me more about the kind of JS you're generating? I'm guessing you've got a controller action returning the JS?

Yeah that bis correct. The JS we are generating contains definitions of commands (in the CQRS sense) that have been developed on the server, to include things such as validation details that can be automatically applied on the client-side.
There is probably a better way to do it (e.g. use T4 templates, post-build step or something similar) to get compile-time output but for now the simple case of using Razor / MVC has been working well.

It might be possible to implement Cassette's IAsset interface in a class that provides your generated script. This asset could then be contained within a ScriptBundle and fit into the rest of Cassette's bundle referencing and rendering system. You'd gain all the minification and caching benefits of Cassette.
I'm planning an article discussing interesting extensions to Cassette. This kind of dynamic asset generation will be a great use case to demonstrate.
However, the simplest solution is often the best. So it's probably best stick with what you've got for now. I'll get Cassette updated to allow local URLs.

For now at least we are happy with just using an external bundle, although of course we do want to find a better way of doing this going forward.
Never even thought about doing direct integration with Cassette as you described to be honest, I was going to investigate compile-time methods of generating the JS (as above, T4 or similar) as we definitively want the other benefits such as minification and caching.
The thing that will need consideration if you do go down the route of creating an article based on this use case would be the best way to integrate a templating solution, don't want to be creating JavaScript all in code. Would be interested in seeing what comes out of this, as it means I'll be able to do a little less thinking when it comes to migrating our current implementation to something better ;)

Good point - concatenating script strings together is awful!
It should be simple enough to call into the MVC view engine to render a view that generates the script content. I'll keep you posted :)

v0.9.4 is in now nuget


Queue Length Measurement Support

What Has Been Done

[x] Updated README doc
[x] Implemented QueueLenghtMonitoringThread class with queue length monitoring support!
[x] Added an docker-compose.yaml that has been used to test the scenarios during development.
[x] Test coverage for the new class/metrics.
[x] Test with Celery 3 + PromTools 0.5.0
[x] Test with Celery 4 + PromTools 0.5.0 (A downgrade for python3.6 on docker image was needed because celery 4 + routes/queues are not supported with 3.7).
[x] Test with Celery 3 + PromTools 0.3.0
[x] Test with Celery 4 + PromTools 0.3.0

Improvements

The solution has been tested with both redis and rabbitmq and works flawlessly.

Questions

Should we initialize queue lengths in setup_metrics?

Caveats

There is no support for automatic discovery of the queues, you need to specify it by using and environment var or arg;
When a queue doesn't exists and you try to read it, an amqp.ChannelError is returned even if you're using an redis or a memory backend  (Kombu things ...) o.O  Because of that, if an exception has been thrown during the read of the queue's length, its value will become 0 in the metrics endpoint and a logging message will be presented in the logs.

Relates to: https://github.com/zerok/celery-prometheus-exporter/issues/41

Looks like prometheus-client 0.3.0 would suffice some headaches. @zerok Do you have any good ideas on how to change the tests to pass on both 0.3.0 and 0.5.0 versions?

Awesome! Thank you very much! I won't be able to give it a try this weekend but hopefully the next. Regarding the various test-environments: I'm currently contemplating dropping support for prometheus-client < 0.5.0. Would this be OK for you?

Hi, thanks for all your effort.
Is there really the need to support python 3.7 at this time?
imho postponing py3.7 support/testing would be feasible, as the celery library itself does not support it until 4.3 is released.
We are really looking forward to this feature, so if there is anything I could help with let me know.

Awesome! Thank you very much! I won't be able to give it a try this weekend but hopefully the next. Regarding the various test-environments: I'm currently contemplating dropping support for prometheus-client < 0.5.0. Would this be OK for you?

For now, the tests are passing even with prometheus 0.3.0. Personally I think that its seems to be the case of best compabiliy effort, not guarantee.

We are also waiting for this specific metric :)

Thank you 🙂 I will remove the py37 build from Travis and tox for now.


Don't die if no go.sum file exists

What type of PR is this?
/kind bug
What this PR does / why we need it:
In golang codebases, bom will die when no go.sum file exists.
This should not happen, it only means that the project is not pulling external modules.
This commit fixes the go package generation to return an empty deps list when encountering such a repo.
Thanks to @strongjz for reporting!
Signed-off-by: Adolfo García Veytia (Puerco<EMAIL_ADDRESS>Which issue(s) this PR fixes:
None
Special notes for your reviewer:
Does this PR introduce a user-facing change?
When indexing golang repos, `bom` would throw a fatal error if no `go.sum` file was found. Now it returns an empty dependency list and generates the SBOM from the repository correctly.

/lgtm
thanks!

/assign strongjz


duckduckgo_search Tool Implementation ✅

#289 | duckduckgo_search Tool Implementation ✅
This is currently working with griptape!
I find ddg to be a really amazing tool for LLMs, better than most of the search scraping tool out there.

Closing in favor of https://github.com/griptape-ai/griptape/issues/383


External variable not accessible in params.libsonnet

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
> ks show foo -V x=bar

ERROR find objects: evaluating params.libsonnet: RUNTIME ERROR: Undefined external variable: x
        params.libsonnet:12:26-41       builtin function <extVar>
        During manifestation

What you expected to happen:
I expect std.extVar("x") to work in params.libsonnet just like it does in any component.jsonnet.  External variables in this case are being used to interpolate values.
How to reproduce it (as minimally and precisely as possible):
Stick this in any component params.libsonnet:
{
  components: {
    api: {
      image: "gcr.io/" + std.extVar("x")
    },
  },
}

and run ks show foo -V x=bar
Anything else we need to know?:
Environment:
ksonnet version: 0.12.0
jsonnet version: v0.11.2
client-go version: kubernetes-1.10.4

Note, using local x = std.extVar("x"); in an user-defined file.libsonnet works too, so the special case appears to be params.libsonnet so far.

Duplicate of https://github.com/ksonnet/ksonnet/issues/561?


Design Slight UI screens on Paper

Will get it done today.
Will do them on my diary and upload screenshots on the repo.
Will do it for both Mobile screens and Desktop screens

Added the UI screens for Admin and User.
It's very rough design.
Might have to redesign again.


Data not populating

@jlopp reported:
"Looks like your data stopped populating? https://dashboard.bitcoinops.org/d/TDhboADmk/bitcoin-optech-dashboards?orgId=1"

It is populating with data, just too slowly to catch up with tip.
We need to move the dashboard from AWS to a local VM once our cluster is fully set up in the new office. Until then, I should redirect dashboard.bitcoinops.org to a warning saying that the site is down until further notice.

The dashboard is based on the the getblockstats RPC, right?  That RPC received a massive speedup in https://github.com/bitcoin/bitcoin/pull/14802 (over 100x in my tests), so it might be worth rebasing our code on to Bitcoin Core master.  (That said, I'm unfamilar with the workings of either the original RPC or our modified version, so I don't know if rebasing would be particularly difficult or if it wouldn't result in any speed up for our cases.)

See https://github.com/bitcoinops/bitcoin/pull/3

@0xB10C thanks!  That's the second time today I had someone point me to an existing issue or PR for something I was commenting on.  Rookie mistake on my part not searching for previous discussions, I'm embarrassed.  :-|

Rookie mistake on my part not searching for previous discussions

My fault for not referencing the rebase in this issue.
It's going through testing now, so hopefully we'll get it deployed and the dashboard back online in the next week.

Most of the data seems to be loading correctly now! :tada:
However the statistics for new dust ouputs and the mempool statistics page both seem to be not working as intended.

@0xB10C Thanks for pointing this out. Ive fixed this and data mempool data moving forward should be logging correctly.

Just looked at the dashboard today and it's looking great!  Thank you everyone who helped fix this.


screen reader issue since 1.89.0 with open file CTRL+p

Does this issue occur when all extensions are disabled?: Yes/No
Yes

VS Code Version: 1.89.0
OS Version: Windows 10

Steps to Reproduce:
I'm using NVDA as screen reader, before 1.89.0 this worked:

Opening the "search file by name or symbol" with CTRL+p
If I type path where only 1 file exists, the file name isn't "announced", cannot do up/down arrow as there's only one result.
In fact, the first file isn't announce for result with more than one results.

To be clear, image we have the following file/directory tree
|- dir1/
|--- abc.go
|- main.go
Doing CTRL+p and than typing dir1/ it displayed the abc.go file but screen reader says nothing.
For result with multiple file, the first file of the list isn't "sent" to screen reader, doing down arrow and the second file in list is "said" by scree reader.
If I may this is a huge problem.
Thanks

Looking at NVDA's log, I see this:
d
1 Results
Search files by name (append : to go to line or @ to go to symbol)  list
abc.go dir1, file results  1 of 1
i
r
slash

with your example:

So we _do_ announce the name, but the second you type another character that gets lost. In 1.88 it would re-say the file.

The change that caused this regression is here:
https://github.com/microsoft/vscode/pull/208913
cc @joaomoreno

and locally, in main, I reverted that commit and the issue went away.
It looks like we were depending on that insert in order to trigger the a11y event to read things out.
Repro:

I used NVDA on Windows
have a workspace with the following structure:

dir/

abc.go

main.go

Open the file picker (ctrl+P)
type, slowly, d I r.
:bug: file abc.go only gets read out on the d and not the other typed characters

The tree seems to be doing the right thing for performance sake: it is not modifying the DOM. Before #208913, the list row was thrown away and rendered again for no good reason. That had the nice side effect of A11Y being read as we desired.
Trying to find a way to kick NVDA to still read it, even though nothing has changed.

I think I am still reproing the original issue:

I made the folder structure from https://github.com/microsoft/vscode/issues/211976#issuecomment-2093805691
ctrl+p
Type d, then before it can announce the name of the file, type i
abc.go is never announced.

I didn't have screen reader accessibility mode turned on.

This issue is back in version 1.93.1 might have been the version prior to that as well. /cc @alexr00

I'll reopen for now and let @joaomoreno decide next steps.


Undo pam pkgconfig change

Only the very latest version of PAM (as of https://github.com/linux-pam/linux-pam/commit/b4f0e2e1f7a18169938eedeff8180e0c75473bb5) ships pkgconfig (.pc) files and so building modern OpenRC isn't going to work on e.g. Ubuntu LTS.
I think we may need to revert bfa634493d48c828330fb2a66418873c3ccf2cb9.

doesn't seem like reverting is necessary.  just add find_library fallback logic when pkg-config isn't found.


Overflow on successful wallet claim

Message is too long and overflowing modal

Second transaction is processed again. VM returns minus .5 taada. This should be prevented.

Would permanently printing the transaction id onto the page with a copy button help with support issues? Remove the modal altogether?

I have put temporary fix on the tx ID. It will still be shown, but wont overflow the box
we need a better mechanism to get the tx status. currently it needs the wallet to check for the tx. It is better to check the unique address directly for everything (check if fee is received, check if tokens are sent)


The layout of the 'Add Entity' dialog is unfriendly when pulling down the scrollbar

Storage Explorer Version: 1.25.0-dev
Build Number: 20220731.1
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Monterey 12.4 (Apple M1 Pro)
Architecture: ia32/x64
How Found: Ad-hoc testing
Regression From: Previous release (1.24.3)
Steps to Reproduce

Expand one storage account -> Tables.
Create a table -> Add one entity with many custom properties.
Click 'Add' -> The 'Add Entity' dialog displays.
Pull down the scroll bar.
Check whether the layout is friendly.

Expected Experience
The layout is friendly.

Actual Experience
The layout is unfriendly.

Additional Context

This issue also reproduces on 'Edit Entity' dialog.
This issue also reproduces when clicking 'Add Property'.

Verified this issue on the build rel/20220802.4. Fixed.


[mscgen mode] documentation fixes

adds MscGen to the 'Language modes' index
merges the demo pages of the  MscGen dialects/ derived languages into mode/mscgen/index.html (took C-like as an example for this)
in that same file: changes title from CodeMirror: Oz mode => CodeMirror: MscGen mode
makes samples a little more uniform

Thanks! Merged as c0e878a...fb113ab


feat: add validation to request / response interactions + adjust scoring appropiately

Overview
Adds a request response validator to the request response module that will make sure the data
returned from the reqresp request is the data that has been requested.
In the Transaction case, this means verifying the proof to ensure that the first nullifier is actually correct.
fixes: https://github.com/AztecProtocol/aztec-packages/issues/8483
fixes: https://github.com/AztecProtocol/aztec-packages/issues/8485

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#8641  👈
#8633
master

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

Join @Maddiaa0 and the rest of your teammates on  Graphite


Feature request: Need a way to report NSFW content

Found a video that is NSFW, but is not marked so, and there is no way in the UX to report is as NSFW - current reporting only allows for rights infringement.
Video in question -> lbry://@oscopelabs/thelovewitch-trailer.
I currently don't have the NSFW box checked, but this is the only thing I see, so I'm just guessing it isn't marked as such.
It's not super NSFW, but borderline PG-13/R for almost nudity :D

LBRY App version: 0.13.0
Operating system: Windows 10

Right now, NSFW flag is control by the person publishing the file. No one but the uploader can change this setting for that particular claim.
On the 6/22 call, something similar to this was discussed, but the question is how to do it correctly in a decentralized way.

What about the mechanism that's used to blacklist content that infringes copyright? Seems like that could be used as well.

Right now that has to be done manually (with human intervention) via the report process. Also not a trial task to do in a decentralized way via app.

The real solution to this is community tagging.
I'd like to avoid an interim solution, but if we do it would be centralized and a lbry-app based solution.


Using Jenkins variables to create the release version.

I am trying to create a release version using Jenkin variable $BUILD_NUMBER. I enter the variable in the plugin field it does not pull the value from Jenkins. How can I get this to work?

This issue is still happening when using Jenkins Pipelines. Any idea when it will be fixed?
Using the example from the docs
e.g.
octopusPushPackage overwriteMode: 'FailIfExists', packagePaths: './artifacts/OctoPetShop.1.1.${BUILD_NUMBER}.zip', serverId: 'octopus-server', spaceId: 'Spaces-1', toolId: 'octocli'
The variable ${BUILD_NUMBER} is not substituted and causes an error.
It works fine for the Push Package Info example though

Hi @codingbadger - suspect you may have already sorted things out at this point, but I came across a similar issue with my interpolated string not resolving as expected.  After taking a look at: https://www.jenkins.io/doc/book/pipeline/jenkinsfile/#string-interpolation, it became clear that I had to use double quotes instead of single quotes.
Perhaps this was your issue as well?

try this :

This issue is still happening when using Jenkins Pipelines. Any idea when it will be fixed?
Using the example from the docs e.g.
octopusPushPackage overwriteMode: 'FailIfExists', packagePaths: './artifacts/OctoPetShop.1.1.${BUILD_NUMBER}.zip', serverId: 'octopus-server', spaceId: 'Spaces-1', toolId: 'octocli'
The variable ${BUILD_NUMBER} is not substituted and causes an error.
It works fine for the Push Package Info example though

try this ${env.BUILD_NUMBER} :
octopusPushPackage overwriteMode: 'FailIfExists', packagePaths: './artifacts/OctoPetShop.1.1.${env.BUILD_NUMBER}.zip', serverId: 'octopus-server', spaceId: 'Spaces-1', toolId: 'octocli'


Updated home.css and education.css

Please mention the respective issue, your solution and a short clip/snapshot of the changes in the PR description before sending in for review.

closed due to inactivity


In memory reservoir sampling

1st PR of In memory reservoir sampling

Work in progress PR. I am testing with perf-tools and looking into ways to optimize this.
FYI @tblaette

closing, this has become unmergable and is currently not on the critical path.
Avinash, please retain the branch for posterity, we might want to revisit this in the future.


[2.0] bundle update refuses to allow versions that are (seemingly) compatible

While trying to update Rails from 5.1.4 to 5.2.0.rc2, I got this error:
Bundler could not find compatible versions for gem "actionpack":
  In snapshot (Gemfile.lock):
    actionpack (>= 5.1.5)

  In Gemfile:
    rails (= 5.2.0.rc2) was resolved to 5.2.0.rc2, which depends on
      actionpack (= 5.2.0.rc2)

    devise (~> 4.4) was resolved to 4.4.3, which depends on
      railties (< 6.0, >= 4.1.0) was resolved to 5.1.5, which depends on
        actionpack (= 5.1.5)

Note that the conflict doesn't make sense... railties is already unlocked because rails depends on it, and the version requirement from devise allows 5.2.0.rc2.
To try to fix it, I tried also explicitly unlocking devise, and then every single other gem that also depended on railties. Eventually, that produced this error:
Bundler could not find compatible versions for gem "activerecord":
  In snapshot (Gemfile.lock):
    activerecord (>= 5.1.5)

  In Gemfile:
    rails (= 5.2.0.rc2) was resolved to 5.2.0.rc2, which depends on
      activerecord (= 5.2.0.rc2)

    tiddle was resolved to 1.2.0, which depends on
      activerecord (>= 4.2.0)

That error makes even less sense to me. The conflicting requirement is completely open-ended and doesn't seem like a conflict at all. activerecord (= 5.2.0.rc2) ought to be compatible with activerecord (>= 4.2.0) Why isn't this allowed?
Here's the output from bundle env:
## Environment

Bundler       2.0.0.dev
  Platforms   ruby, x86_64-darwin-17
Ruby          2.5.0p0 (2017-12-25 revision 61468) [x86_64-darwin17]
  Full Path   /opt/rubies/2.5.0/bin/ruby
  Config Dir  /opt/rubies/2.5.0/etc
RubyGems      2.7.3
  Gem Home    /Users/andre/.gem/ruby/2.5.0
  Gem Path    /Users/andre/.gem/ruby/2.5.0:/opt/rubies/2.5.0/lib/ruby/gems/2.5.0
  User Path   /Users/andre/.gem/ruby/2.5.0
  Bin Dir     /Users/andre/.gem/ruby/2.5.0/bin
Tools
  Git         2.11.0
  RVM         not installed
  rbenv       not installed
  chruby      0.3.9

## Bundler Build Metadata

Built At          2018-03-24
Git SHA           0fc3fc1c3
Released Version  false

## Bundler settings

gem.test
  Set for the current user (/Users/andre/.bundle/config): "rspec"
gem.mit
  Set for the current user (/Users/andre/.bundle/config): true
gem.coc
  Set for the current user (/Users/andre/.bundle/config): true
gem.foobar
  Set for the current user (/Users/andre/.bundle/config): "../foobar"
local.foobar
  Set for the current user (/Users/andre/.bundle/config): "/Users/andre/Downloads/foobar"
path
  Set for your local app (/Users/andre/src/ketchupapp/ketchup/.bundle/config): ".bundle"
jobs
  Set via BUNDLE_JOBS: 8

To reproduce this issue for yourself, check out Bundler, run rake install, and then:
$ git clone https://github.com/ketchupapp/ketchup-web.git
$ cd ketchup-web
$ git checkout bundler/rails-upgrade-error
$ bash repro.sh

On suggestion from @segiddins, I ran bundle config only_update_to_newer_versions false, and tried bundle update rails again. It worked this time. But the diff doesn't show any gems whose versions went down: https://gist.github.com/indirect/2d4234c90af642021713e7d7bdc4a694.

I was having the same issue as @indirect.  bundle config only_update_to_newer_versions false did not change anything for me.  Downgrading Bundler from 1.16.1 to 1.15.4 resulted in completely different version compatibility errors that actually made sense and, once resolved, allowed the update to proceed.

Spent some time looking into this, the crux of the issue is that stuff that requires rails is being activated before the rails dependency, and I think the required version of rails being a pre-release is mucking everything else up

aka with the following patch, we very quickly get to the correct error (cc @greysteil ):
diff --git a/lib/bundler/resolver.rb b/lib/bundler/resolver.rb
index d7911c1de..e7c36ee17 100644
--- a/lib/bundler/resolver.rb
+++ b/lib/bundler/resolver.rb
@@ -207,7 +207,7 @@ module Bundler
         name = name_for(dependency)
         vertex = activated.vertex_named(name)
         [
-          @base_dg.vertex_named(name) ? 0 : 1,
+          # @base_dg.vertex_named(name) ? 0 : 1,
           vertex.payload ? 0 : 1,
           vertex.root? ? 0 : 1,
           amount_constrained(dependency),

=>
Bundler could not find compatible versions for gem "rails":
  In Gemfile:
    rails (= 5.2.0.rc2)

    devise-pwned_password (~> 0.1.4) was resolved to 0.1.4, which depends on
      rails (~> 5.1.2)

I am having this same issue now. Is there any update?


feat(content): Explain how to fix common errors

This PR:

Creates a new "Troubleshoot" section in the Railway docs
Add a "Fixing Common Errors" section underneath it
Add instructions for how to fix "Application Error: This application failed to respond" (i.e. wrong host/port)

Motivation
In the Discord community, there are a lot of help requests on post-deployment issues. They're commonly user errors unrelated to Railway, and a subset of them are Railway errors unrelated to the platform - for those issues, the problem & solution is always the same.
I believe that a section dedicated to troubleshooting will streamline how Railway provides support. It eases the burden of community helpers & the Railway team by allowing them to link out to highly-specific help sections in the docs without "manually" responding individually to each help request, and it makes the specific errors pop up in search engines.
More generally, this new section will hold content for troubleshooting common errors users may encounter. Apart from the newly-added page ("Fixing Common Errors"), we can add tech-specific pages into this section that explains common gotchas (such as logs not appearing in Python because they're buffered & not flushed to stdout):
Troubleshoot
  |> Fixing Common Errors
  |> Python
  |> Node
  |> ...

Rendered Changes
Sidebar

Content (Troubleshoot -> Fixing Common Errors)

[resolves log-282]


[BUG] MSF FIX error

Any help?
Android 10 with RR rom

clone it again before that remove ruby from bin & lib


Parse RSA CRT parameters from PKCS1 private keys

Currently they are ignored in the serialized key and then regenerated from P/Q/D. But that exposes key loading to a side channel attack on the modular inversion and GCD bignum functions when QP is computed. [And probably similar issues for DP/DQ during the division step but no attack there has been published yet.]
Also this reduces computational overhead of loading RSA private keys from memory which will be nice for us in roche.
Backport of https://github.com/ARMmbed/mbed-crypto/pull/352

@jethrogb ping

bors r+

bors r-

bors r+


Can't play this video on Facebook

I'm using the latest git with latest Chrome on Firefox 42 and I can't play this video on Facebook. Click on the following link:
https://www.facebook.com/AutoMoto.it/videos/836939869714027/
Anyone experienced the same issue? If you need more info, just ask.

Plays fine. However with hardware decoder enabled, occasionally visual glitches occur.

Facebook changed player. It now uses HTML5 by default, and falls back to Flash on failure.

You're right. So i'm closing this since it's not related to Flash.


Add ability to set the default project visibility

Add the ability change the visibility in projects by default because sometimes you don't want to instantly share your current project :)

If you don't want to be asked every time you can change the default in the default settings for new projects
https://www.jetbrains.com/help/idea/configure-project-settings.html#new-default-settings

(From #faq on discord)


Make examples work in Internet Explorer

Fixes #10416.

Looks like you might need to change the “fixes” issue number.

Thanks @tschaub, you are absolutely right.


adb pair should be a String

Hi, I have found a bug where the pair code starts with 0, as shown below:

if you parse the string "091102" as a u32 let code = ""091102".parse().unwrap(); the number will be incorrect
Hi, looking in adb source code: pairing_connection_test, pairing_server.cpp, pairing_connection.cpp
I found that the password or the pair code should be a vector of 8 bytes (Vec)
// pairing_connection_test.cpp at line 161
TEST_F(AdbWifiPairingConnectionTest, ClientCreation) {
    // All parameters bad
    auto client = PairingClient::create({}, client_info_, {}, {}, "");
    EXPECT_EQ(nullptr, client);
    // Bad password
    client = PairingClient::create({}, client_info_, {0x01}, {0x01}, "<IP_ADDRESS>");
     ...
    // Valid params
    client = PairingClient::create({0x01}, client_info_, {0x01}, {0x01}, "<IP_ADDRESS>");
    EXPECT_NE(nullptr, client);
}
TEST_F(AdbWifiPairingConnectionTest, SmokeValidPairing) {
    std::vector<uint8_t> pswd{0x01, 0x03, 0x05, 0x07};
    initPairing(pswd, pswd);
   .....

// pairing_server.cpp   at line 397
 std::unique_ptr<PairingServer> PairingServer::create(const Data& pswd, const PeerInfo& peer_info,
                                                     const Data& cert, const Data& priv_key,
                                                     int port) {
    if (pswd.empty() || cert.empty() || priv_key.empty() || port <= 0) {
        return nullptr;
 adb_pairing_auth_test   }
....

// pairing_connection.cpp at line 57
struct PairingConnectionCtx {
  public:
    using Data = std::vector<uint8_t>; // <-- Data is defined as a vector of u8
    using ResultCallback = pairing_result_cb;
    enum class Role {
        Client,
        Server,
    };

I tried to implement as Vec, but it's challenging with many edge cases. A Rust String works nicely instead :)

Thanks for pointing this issue ! Can you add your fix in a PR ;) ?

Closed and merged in 1.0.6


Automated cherry pick of #2293: baremetal: raid disk inode inconsistent due to mkfs.ext4 largefile option

Cherry pick of #2293 on release/2.10.0.
#2293: baremetal: raid disk inode inconsistent due to mkfs.ext4 largefile option

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@06f7589). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #2296   +/-   ##
================================================
  Coverage                  ?   7.56%
================================================
  Files                     ?     410
  Lines                     ?   63620
  Branches                  ?       0
================================================
  Hits                      ?    4814
  Misses                    ?   58396
  Partials                  ?     410

Flag
Coverage Δ

#aFlag
7.56% <0%> (?)

Impacted Files
Coverage Δ

pkg/baremetal/utils/disktool/disktool.go
0% <0%> (ø)

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 06f7589...17fae7f. Read the comment docs.

/lgtm
/approve


useEffectOnce update/fix

Making sure useEffectOnce actually runs only once. Particularly preventing React 18 from rendering twice in development mode.

Take a look at https://dev.to/iamyoki/comment/1pfc9
function useEffectOnce(effect) {
  const effectFn = useRef(effect)
  const destroyFn = useRef()
  const effectCalled = useRef(false)
  const rendered = useRef(false)
  const [, refresh] = useState(0)

  if (effectCalled.current) {
    rendered.current = true
  }

  useEffect(() => {
    if (!effectCalled.current) {
      destroyFn.current = effectFn.current()
      effectCalled.current = true
    }

    refresh(1)

    return () => {
      if (rendered.current === false) return
      if (destroyFn.current) destroyFn.current()
    }
  }, [])
}

This seems to be designed to skip a render, but it's not clear why.


docs: add small note about submitFocusError

I noticed that the autofocus on error doesn't work when using "custom" components like mentioned in the documentation (submitFocusError). However, forwarding the ref makes this work. I thought it was worth mentioning
const Input = React.forwardRef((props, ref) => {
  return <input {...props} ref={ref}></input>;
});

thank you @devrnt ❤


Reset / revert need to be added in some way

Need to followup on Derek's research on what kinds of reverting/resetting users need to do.
Need to develop a prototype design for

what features go in
how to design them in an interface

Need to run those designs by some users
Then need to implement.

Added via 9249a80ac5aaa024db38174a673b531ef3dbd929


Add support for sun2

Untested, because I still have to learn how to compile these JavaScript things, but this should theoretically work with https://github.com/pnbruckner/ha-sun2 (which is a lot easier to install, especially on HCAS).
Could you please take a look at this?

Thanks for the PR. I really appreciate your work, but I was in a middle of implementing this on my own.


Page not found

I've deployed this app on heroku to test and the inside links are returning not found.
http://qulture360.herokuapp.com/survey/create
Is there anything i need to do?
Thanks

Thanks for trying out feedback360.
Before deploying the app, please complete the following 2 steps:

Rename production_example.conf.php to production.conf.php and update the properties inside it.
After you've installed Composer (https://getcomposer.org/doc/00-intro.md), from the project's root director run: composer update

I've updated the ReadMe file with the same.

I did everything and the app is running http://qulture360.herokuapp.com but the internal pages are returning not found. Is there any config needed on the php?

Solved! My mistake. For some reason the .htacess was out. Fixed! Thanks!! :)


Unable to enter Station ID ST-00101xxx in configuration.

Hi
Trying to update to 1.0.9 and re-configuring (for some reason), I'm unable to enter my station Id (ST-00101xxx).  The entry for Station ID will not accept "ST-" and will not accept a "0" as the first character.

This is not your Station ID. Please see the Configuration Section in the README file on how to obtain your Station ID.

No feedback for several weeks. Assume solved.


why export function set interface is error

TypeScript Version:  2.7.5-dev.2018.3.14

Search Terms:
interface
Code

var glob = require( "glob" );

interface ConfigData {
    include: number[],
    exclude: number[]
}

function getIncludefiles ( config: ConfigData ): any {

    let { include, exclude }: { include: number[], exclude: number[] } = config;
    let includeCheckFiles: number[] = [];
    let excludeCheckFiles: number[] = [];

    // 包含文件
    if ( include && include.length != 0 ) {
        for ( let i = 0; i < include.length; i++ ) {
            let files: number[] = glob.sync( include[ i ], { cwd: process.cwd() } );
            includeCheckFiles = includeCheckFiles.concat( files );
        }
    }
    //排除文件
    if ( exclude && exclude.length != 0 ) {
        for ( let i = 0; i < exclude.length; i++ ) {
            let files: number[] = glob.sync( exclude[ i ], { cwd: process.cwd() } );
            excludeCheckFiles = excludeCheckFiles.concat( files );
        }
    }
    //最终需要检测的文件
    if ( excludeCheckFiles && excludeCheckFiles.length > 0 ) {
        for ( let i = 0; i < excludeCheckFiles.length; i++ ) {
            let index: number = includeCheckFiles.indexOf( excludeCheckFiles[ i ] );
            if ( index != -1 ) {
                includeCheckFiles.splice( index, 1 );
            }
        }
    }

    return includeCheckFiles;
}

export = getIncludefiles;

Expected behavior:
no error
Actual behavior:
error TS4078: Parameter 'config' of exported function has or is using private name 'ConfigData'.
How should I set it up?

First, if possible it would be better to use a default export over export = (export default function getIncludefiles(..) { ... }). Then you can add export in front of the interface (export interface ConfigData { ... }) and they can be imported separately with import getIncludefiles, { ConfigData } from "./module-path";.
If you really want an export = you have to put the type in a namespace merged with the function:
namespace getIncludefiles {
    export interface ConfigData {
        include: number[],
        exclude: number[]
    }
}

function getIncludefiles ( config: getIncludefiles.ConfigData ): any { ... }
export = getIncludefiles;

Or you could write the type inline: config: { include: number[], exclude: number[] }

I did exactly what you said, but I didn't have any hints in the editor.
before:

after:

Am I still doing something wrong?

Those are both valid hints -- neither is wrong.

@andy-ms Thank you very much for your patience.