Add rho training log to the pipeline file

Per the title, this PR actually adds the log of downsampling phase at selector to the selector log, hence further to the pipeline log. The intention here is to see how well the il model is trained with the record_loss_every.

The unit test failed last time last night was due to flakiness, not something really wrong :)


OutPut model always will set the max/minMultiplicity for relations even if you don't set on the attribute

Hi,
When I place a Relation attribute on a non-collection property of a class without initializing the Max/MinMultiplicity, I get these with default value of 0 in the output models. However the maxMultiplicity = 0 is not allowed by the DTDL definition and the minMultiplicity is by default 0 if you don't set it. Due to the issue that in C# attribute properties can not be nullable, I made these changes to not include them when the value is 0 which will fix the problem.
considering this model:
[DigitalTwin(Version=1,DisplayName="Twin with a relationship")]
    public class TwinWithRelationship: TwinBase
    {

        [TwinProperty]
        public int Count { get; set; }

        //[TwinRelationship(Name = "Test Relationship", MaxMultiplicity = 5)] // I have commendted this one
        [TwinRelationship(Name = "Test Relationship")] // this one has no max/min
        public SimpleTwin Relationship { get; set;}
    }

you can see the issue here in this model:
{
      "@id": "dtmi:telstra:twins:test:twinwithrelationship;1",
      "@type": "Interface",
      "@context": "dtmi:dtdl:context;2",
      "displayName": "Twin with a relationship",
      "contents": [
        {
          "@type": "Property",
          "name": "count",
          "schema": "integer"
        },
        {
          "maxMultiplicity": 0,
          "minMultiplicity": 0,
          "target": "dtmi:telstra:twins:test:simpletwin;1",
          "@type": "Relationship",
          "name": "relationship"
        }
      ]
    }

public partial class ModelRelationship : Content
    {
        public static ModelRelationship Create(PropertyInfo info)
        {
            var attr = info.GetCustomAttribute<TwinRelationshipAttribute>();

            return new ModelRelationship(info.Name.ToCamelCase())
            {
                DisplayName = attr.DisplayName,
                Comment = attr.Comment,
                Description = attr.Description,
                MaxMultiplicity = attr.MaxMultiplicity **== 0 ? null : attr.MaxMultiplicity**,
                MinMultiplicity = attr.MinMultiplicity **== 0 ? null : attr.MinMultiplicity**,
                Target = info.PropertyType.GetModelPropertyType().GetDigitalTwinModelId()
            };
        }
    }

I have put up a PR for the fix as you suggested, along with some unit tests. (I did have to coerce the value to nullable though).
In the meanwhile, you have already noted the workaround, which is to simply specify the max multiplicity, e.g. MaxMultiplicity = 500 (which is the maximum the parser allows)

Thanks @sgryphon

The pull request has been merged and the bug has been fixed closing out this issue now


Add store map to allow saving memory

What changes do you make in this PR?
Store_map: True:

Checklist

[ ] I have merged the latest main branch into current branch.
[ ] I have run bash scripts/format.sh before merging.
Please use "squash and merge" mode.

Store Map==False:

Panda3D resource are successfully released when env.reset() is called. So the memory increasing speed is much lower than using store_map=True. However, some python variables still remain in memory, I am working on this

Should we merge this PR?


Remove LinuxPerf

We do not use perf tool anymore, lets remove his from the codebase.
Test: cmake --build .

Thanks Dimitry!


Bots don't split or feed on command with new userscript

My bots don't split or feed when I press E and R. It worked with the script that used to flash, but with the new one it doesn't.

Sorry it works now :)

Are you using windows or mac? Does this not work for windows? cuz it gave me errors when I npm install the package.json

Mark PM me on skype, I can help


organizations > destroy existing invitation

On the list of pending invitations there is an "X" button that should fire a delete call to destroy that invitation.
refs #188

PR: #216


Update Rust and C example

Closes #236

I'm going to do any documentation follow up off of this branch.

eh, nevermind. Let's get this one in.


Add systems job for maintenance tasks

One maintenance feature that datahub is lacking is the actual deletion of entities tied to deleted datasets.
Currently, when a dataset is deleted, datahub marks it's internal ID as deleted - but does not touch the entities in said dataset. From a user perspective all data in that dataset looks deleted because datahub respects the deleted state of datasets. But there are still entities in storage which are no longer accessed now. These stale entities should be physically deleted in a regular maintenance task.
The datahub should have an internal systems-job which runs in a configurable interval to do maintenance on stored data.
Regular entity deletion would be the first maintenance task executed by this systems job. But it should be possible to add more tasks later as well.
It should not be possible to delete or alter the job via the jobs APIs, but the job should be listed as scheduled or running job in respective API calls.

we do dataset gc now, but in form of an on-start go routine. this can still be improved with a job like described here

there are system jobs now, they are not visible in the regular job APIs but can be monitored using logs.


Retrieve DEFAULT_FORMAT from correct class during blade directive registration

The blade registration throws a "Undefined class constant", the default state is attempting to retrieve the constant from the Version class instead of the Constants supporting class. I have updated this with a simple two line fix, I did not add tests for this as it would be overly cumbersome for a single blade directive.

I'm also getting this undefined constant. Can this fix please be merged ASAP.

Sorry for taking too long to merge this, and thank you!

Version 0.2.6 tagged.


Handle failed report request to Abacus

Keep usage events in memory when POST to Abacus fails, so that reporting is retried on next iteration
Refactor reporting logic
[#101018516]

Signed-off-by: Georgi Sabev<EMAIL_ADDRESS>

Thanks for this contribution!


Fixed noData example typos

Example code showed 'data aint here' but rendered chart had 'No Data
For You!'.  Changed to be consistent.

Just an update, before we take in any further pull requests, we are going to put in place some unit tests to help us maintain stability.  Until that time, pull requests will be waiting.


Add js.special.instanceof(a, b) for a instanceof b.

Previously, we had no way to write code equivalent to a instanceof b where b is a dynamic value (rather than something directly available as js.constructorOf[C] for some C). Adding this method fills up a hole in our interoperability features.
This will also be necessary to desugar .isInstanceOfs on nested JS classes.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/4313/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/5789/
Test PASSed.

Just to check my understanding: This is necessary, because we need to desugar isInstanceOf on nested classes in PrepJSInterop (otherwise we have lost the nesting)?

Basically, that's the idea. It's quite complicated, though, and hard to thoroughly explain before you've seen the whole scheme for nested JS classes. But in a nutshell, we need to desugar x.isInstanceOf[path.C] early (before erasure) into instanceof(x, js.constructorOf[path.C]), and js.constructorOf[path.C] must also be desugared early into something like path.C$jsclass, where C$jsclass is a field used to store the nested class value that was created for the instance path. We need to do this before erasure because that phase will throw away the path. part, and replace it with Enclosing#C.
If you want to know more at this point, I suggest you read the big "strategy" comments at the beginning of ExplicitInnerJS and ExplicitLocalJS in my branch for nested JS classes.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/4314/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/5790/
Test PASSed.


failed to build the "minimal example"

I'm getting this error on Archlinux:
go build: when using gccgo toolchain, please pass linker flags using -gccgoflags, not -ldflags
# github.com/visualfc/goqt/ui
/usr/bin/ld: cannot find -lqtdrv.ui
collect2: error: ld returned 1 exit status

have you read readme.md? project is closed.

Oh. I opened this issue in the incorrect repo...


在got10k数据集上进行测试出现:out of memory

Hello when  I use
python tracking/test.py stark_st baseline_got10k_only --dataset got10k_test --threads 32
python lib/test/utils/transform_got10k.py --tracker_name stark_st --cfg_name baseline_got10k_only
Tested on the got10k data set, the result is less than 20 Sequences, and the oom error occurs.
My hardware platform is 3090

Hi, test.py by default assumes that the user is using 8 GPUs for evaluation and distributes threads evenly to each GPU. You can specify the number of threads and the number of GPUs by using "python tracking/test.py stark_st baseline_got10k_only --dataset got10k_test --threads <NUM_THREADS> --num_gpus <NUM_GPUS>"

Okay, thank you for your reply. I set the number of threads to 32 before, and then I changed it to 12 and started working. But currently, the FPS on GOT10k is only about 6.


Bot will not spin pokestops when bag is full

Expected Behavior
Bot continues to spin pokestops in order to gain experience.
Actual Behavior
Bot does not spin pokestops because the bag is full.
Steps to Reproduce
Other Information
OS: Ubuntu
Git Commit: dev

I have this issue on Windows 7 also.

Same problem here,I changed the config and fixed it
Just add ultraball and ultraball into the config.json
For example
"type": "RecycleItems",
"config": {
"item_filter": {
"Pokeball":       { "keep" : 20 },
"Potion":         { "keep" : 10 },
"Super Potion":   { "keep" : 20 },
"Hyper Potion":   { "keep" : 30 },
"Revive":         { "keep" : 30 },
"Razz Berry":     { "keep" : 50 },
"Greatball":     { "keep" : 40 },
"Ultraball":     { "keep" : 40 }

I hate my input program
Just add Greatball and Ultraball into the config.json


How to get the Example Split results?

Can we test our model in the example split experiment?

Hi,
Train and dev datasets in the example split contain some examples in our unreleased test dataset. So we won't release them.
Best,
Tao


No support for "deprecated" on properties

It appears as though the Property model doesn't have support for becoming deprecated, although the spec permits it.

swagger-core 1.5 (the version you point to) is for Swagger/OpenAPI 2.0 which does not support deprecated at the property level. That has changed in OAS3 and is supported in the 2.0 branch.


Dynamic RTL/LTR

What about a dynamic RTL/LTR direction with a less variable ....

There's already a RTL thread open—punting this discussion to that one.
We don't have the resources to do RTL yet. It's much lower on the priority list.

https://github.com/donaldducky/bootstrap-rtl

i'll try to do it using R2 method..
thanks...


Analog domain

This adds the possibility to specify the output domain when setting an analog output.
Tests are still missing, hence the draft status.
Edit: Tests are now there.

Note: This will be breaking ABI, but not API.

I am not sure it this is the right way. I would prefer to guide people setting this up on the teach pendant. Though they need to remember to save the installation

We could also add a third state "don't change" which could be the default. One problem there would be that at least for ROS 1 the default value would be 0 which I would not like to remap to "don't change" since 0 on the low-level interface means "current".
Thinking about it, that might be the best solution: Provide a default value of -1 that will not change the domain.

I like the idea of giving the flexibility to do both.

I like the idea of giving the flexibility to do both.

I've updated the PR accordingly. When not passing the domain, it is not set in the RTDEWriter.


Gpu api change to include gpu count

Introduce gpuCount in caph to avoid sku modification when adding new gpu.
PM Spec: https://microsoft.sharepoint.com/:w:/t/MSk8s/ESFaAAlvDCBNqBgVNHlwm0wBS2Cn88FOxhjIFNTh6O1xqw?e=BBVaMs
Dev Spec: https://microsoft.sharepoint.com/:w:/r/teams/MSk8s/_layouts/15/Doc.aspx?sourcedoc={6E21E07E-228F-4776-9E02-F68F666F022F}&file=GPUP Proposal.docx&action=default&mobileredirect=true&DefaultItemOpen=1&share=IQF-4CFujyJ2R54C9o9mbwIvAcUoSR87WF93Bg0cb3Cs1Z4

@wenjlu-lgy please rebase


.webm and .mp4 support?

Are there any plans for .webm and .mp4 support?
What I would like is for featherlight to detect and display these filetypes like it already does for images.
//rhwol

No plans per say. Should at least be a plugin. Wanna write a PR?

How about outsourcing detectors etc. as plugins or simple wiki examples how to implement? Because this is what FL is all about, barebones but extensible...

Closing this, until there's a PR or more interest


[Bug]: can't run android app

Version
v2.0.1
Which operating systems have you used?

[X] Android
[ ] iOS

Environment that reproduces the issue
Oneplus 8T android 13
Is it reproducible in the example application?
Not tested
RTMP Server

Reproduction steps
Update the package and yarn android
Expected result
App should be installed
Actual result
app building is failing with this error
Execution failed for task ':app:mergeExtDexDevelopmentDebug'.
Could not resolve all files for configuration ':app:developmentDebugRuntimeClasspath'.
Failed to transform camera-viewfinder-1.4.0-alpha06.aar (androidx.camera:camera-viewfinder:1.4.0-alpha06) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=21, org.gradle.category=library, org.gradle.dependency.bundling=external, org.gradle.libraryelements=aar, org.gradle.status=release, org.gradle.usage=java-runtime}.
Execution failed for DexingWithClasspathTransform: /Users/talhashafique/.gradle/caches/transforms-3/a2f592b30fddcb727f0b2169f047a323/transformed/jetified-camera-viewfinder-1.4.0-alpha06-runtime.jar.
Error while dexing.
Additional context
No response
Relevant logs output
Execution failed for task ':app:mergeExtDexDevelopmentDebug'.
> Could not resolve all files for configuration ':app:developmentDebugRuntimeClasspath'.
   > Failed to transform camera-viewfinder-1.4.0-alpha06.aar (androidx.camera:camera-viewfinder:1.4.0-alpha06) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=21, org.gradle.category=library, org.gradle.dependency.bundling=external, org.gradle.libraryelements=aar, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingWithClasspathTransform: /Users/talhashafique/.gradle/caches/transforms-3/a2f592b30fddcb727f0b2169f047a323/transformed/jetified-camera-viewfinder-1.4.0-alpha06-runtime.jar.
         > Error while dexing.

Have you tried to look for this issue on google and stackoverflow?
As CI is properly running for Android, it does not seem to comme directly from this package.

I'm encountering the same issue after upgrading to version 2.0.1. The Android app runs successfully when I revert back to version 2.0.0.

Execution failed for task ':app:mergeExtDexDevelopmentDebug'.

yes @ThibaultBee i have tried looking over these forums but can't find anything about this issue. I think the issue might be that new version is conflicting with the project build gradle version.

Execution failed for task ':app:mergeExtDexDevelopmentDebug'.

yes @ThibaultBee i have tried looking over these forums but can't find anything about this issue. I think the issue might be that new version is conflicting with the project build gradle version.

Have you tried to look for this issue on google and stackoverflow? As CI is properly running for Android, it does not seem to comme directly from this package.
yes @ThibaultBee i have tried looking over these forums but can't find anything about this issue. I think the issue might be that new version is conflicting with the project build gradle version.

From what I undertand, you did find a way out of this issue. How did you manage to fix it?

From what I undertand, you did find a way out of this issue. How did you manage to fix it?

for me it worked by upgrading the build gradle version

So what is your new AGP version?

So what is your new AGP version?

8.6

Ok thanks.
I am closing this issue as it is resolved.


Suggestion: Add More Characters & Mappacks On DLC

like those ....

& More....

Please direct DLC suggestions to the Mari0pository: https://github.com/qixils/mari0pository
I've also added a link to this repository for future reference.


fix: Add ReactNode as allowed type of TranslationValue

Seems like ReactNode is acceptable as TranslationValue and it's useful if we want to use React Nodes as translation values.
Example:
<NextIntlProvider defaultTranslationValues={{ br: <br /> }}>
    {content}
</NextIntlProvider>

And then use it in JSON: Line 1 {br} Line 2
Adding it to types helps to omit unnecessary TypeScript transformations and @ts-ignore usages.
I'm not sure if it possible to implement such thing as Intl has out-of-the-box: https://formatjs.io/docs/react-intl/components#wraprichtextchunksinfragment to avoid "keys" issue in another way.

Hi @omnomnomka and thank you for your PR! This looks pretty good. Can you add some tests and fix the lint error that is reported on CI? Thanks! 🙌

Closing as stale …


Convert Markdown to Docbook

We've had a request for converting Markdown to Docbook.
Maybe we could do it like we did for DITA, using the XSLTs Oxygen uses for copy/paste.
But we can probably leave this at the end, after the first version of the add-on is released.

We first need to resolve an internal issue to properly generate a Docbook article when converting the content: EXM-40279.
So the conversion actions from Markdown or HTML or XHTML to Docbook need to be available only starting with Oxygen 20. We have an API which shows you what Oxygen version is currently running.
So you can commit the implementation but the extra actions should be mounted on the menus only if Oxygen 20 or newer is running.

Maybe we need to update the Readme.md to state the new capability Docbook conversion maybe with parentheses like (Oxygen 20 or newer)


Panic in listenPush

Originally posted in https://github.com/checksum0/go-electrum/issues/10.
When working on testnet and mainnet integration tests in https://github.com/keep-network/keep-core/pull/3599 we stumbled upon random panics when running tests:
=== FAIL: pkg/bitcoin/electrum TestGetLatestBlockHeight_Integration (unknown)
panic: assignment to entry in nil map

goroutine 511 [running]:
github.com/checksum0/go-electrum/electrum.(*Client).listenPush(0xc00036cc40, {0x120e9eb, 0x1c})
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/network.go:264 +0x74
github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders.func1()
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:40 +0x36
created by github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:39 +0xfd

=== FAIL: pkg/bitcoin/electrum TestGetLatestBlockHeight_Integration/electrs-esplora_tcp_get (unknown)

=== FAIL: pkg/bitcoin/electrum TestGetLatestBlockHeight_Integration/electrumx_wss_get (unknown)
panic: assignment to entry in nil map

goroutine 511 [running]:
github.com/checksum0/go-electrum/electrum.(*Client).listenPush(0xc00036cc40, {0x[120](https://github.com/keep-network/keep-core/actions/runs/5215398427/jobs/9424824985?pr=3599#step:5:121)e9eb, 0x1c})
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/network.go:264 +0x74
github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders.func1()
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:40 +0x36
created by github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:39 +0xfd

I think the problem lies in how the library handles closing the connection.
SubscribeHeaders creates a goroutine that writes to s.pushHandlers map in the listenPush function. If Shutdown() is called at the same time, unfortunate ordering of calls can lead to clearing s.pushHandlers = nil and then attempting to s.pushHandlers[method] = append(s.pushHandlers[method], c).
Separately from this issue, I think the goroutines created in SubscribeHeaders are leaking given there is no clear exit signal for them - the channel returned by listenPush is never closed and they are stuck on reading.

Some new cases of panic failures, after https://github.com/keep-network/keep-core/pull/3656 got merged to main:
=== FAIL: pkg/bitcoin/electrum TestGetLatestBlockHeight_Integration/electrumx_wss_get (unknown)
panic: assignment to entry in nil map

goroutine 545 [running]:
github.com/checksum0/go-electrum/electrum.(*Client).listenPush(0xc0002aecb0, {0x120fbc8, 0x1c})
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/network.go:264 +0x74
github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders.func1()
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:40 +0x36
created by github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:39 +0xfd

=== FAIL: pkg/bitcoin/electrum TestGetLatestBlockHeight_Integration/fulcrum_tcp_get (unknown)
panic: assignment to entry in nil map

goroutine 757 [running]:
github.com/checksum0/go-electrum/electrum.(*Client).listenPush(0xc000310c40, {0x121655d, 0x1c})
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/network.go:264 +0x74
github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders.func1()
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:40 +0x36
created by github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:39 +0xfd

=== FAIL: pkg/bitcoin/electrum TestGetLatestBlockHeight_Integration (unknown)
    electrum_integration_test.go:437: failed to initialize electrum client: [retry timeout [1m0s] exceeded; most recent error: [websocket: bad handshake]]
panic: assignment to entry in nil map

goroutine 757 [running]:
github.com/checksum0/go-electrum/electrum.(*Client).listenPush(0xc000310c40, {0x121655d, 0x1c})
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/network.go:264 +0x74
github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders.func1()
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:40 +0x36
created by github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:39 +0xfd

=== FAIL: pkg/bitcoin/electrum TestGetLatestBlockHeight_Integration/embedded/mainnet/wss://electrumx-server.tbtc.network:8443_get (unknown)
panic: assignment to entry in nil map

goroutine 566 [running]:
github.com/checksum0/go-electrum/electrum.(*Client).listenPush(0xc0024d2930, {0x[120](https://github.com/keep-network/keep-core/actions/runs/5459180187/jobs/9935141566#step:5:121)fbc8, 0x1c})
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/network.go:264 +0x74
github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders.func1()
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:40 +0x36
created by github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:39 +0xfd

Jul 4: https://github.com/keep-network/keep-core/actions/runs/5449452469/jobs/9921536617
Jul 5: https://github.com/keep-network/keep-core/actions/runs/5459180187/jobs/9935141566
Jul 10: https://github.com/keep-network/keep-core/actions/runs/5502620856/jobs/10027243275

Another case: https://github.com/keep-network/keep-core/actions/runs/5663040756/job/15344593515.
=== FAIL: pkg/bitcoin/electrum TestGetLatestBlockHeight_Integration/embedded/mainnet/wss://electrumx.prod-utility-eks-us-west-2.staked.cloud:443_get (unknown)
panic: assignment to entry in nil map

goroutine 595 [running]:
github.com/checksum0/go-electrum/electrum.(*Client).listenPush(0xc0000aed20, {0x[121](https://github.com/keep-network/keep-core/actions/runs/5663040756/job/15344593515#step:5:122)6560, 0x1c})
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/network.go:264 +0x74
github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders.func1()
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:40 +0x36
created by github.com/checksum0/go-electrum/electrum.(*Client).SubscribeHeaders
	/go/pkg/mod/github.com/keep-network/go-electrum@v0.0.0-20230524074410-befe891c2f8c/electrum/subscribe.go:39 +0xfd


allow users to submit photos to training set

What's your idea?
Add an option in our Streamlit app that allows users to submit their photo for us to include in our training set.
Describe the solution you'd like
Add a button to send the photo to a Google Drive photo. We won't be able to do this until Streamlit Sharing enables secrets, since the Google Drive API will need to be authenticated.

Related to #5


⚠ Empornium has degraded performance

In 9b60c99, Empornium ($EMP) experienced degraded performance:

HTTP code: 502
Response time: 1009 ms

Resolved: Empornium performance has improved in be1f38a after 6 minutes.


Add fallback fonts

Added fallback fonts to cater for when the variable "--w-font-sans" is unreached to fix the bug on "Wagtail userbar displays incorrect font" issue
https://github.com/wagtail/wagtail/issues/9290

Thanks @fyunusa I feel like this would fix the problem but if you could.dig a bit deeper into the root cause we may find a simpler solution.

Thanks @fyunusa I feel like this would fix the problem but if you could.dig a bit deeper into the root cause we may find a simpler solution.

Ok, will try to lookup on that

@fyunusa it looks like the variable is defined in the tailwind config here:
https://github.com/wagtail/wagtail/blob/b276b835e959bb773a1e97d0470e066c1b169538/client/tailwind.config.js#L73
and
https://github.com/wagtail/wagtail/blob/b276b835e959bb773a1e97d0470e066c1b169538/client/tailwind.config.js#L148
I'm not familiar with tailwind, but my understanding is that instead of providing a fallback we need to actually define --w-font-sans to the settings you've provided as a fallback.
Hope this helps!

@fyunusa what you did works but I think I’d prefer the approach I suggested to you on Slack, which is to update:
https://github.com/wagtail/wagtail/blob/6814d0beea34ca3e868811210ff2210934e72ec3/wagtail/admin/static_src/wagtailadmin/scss/userbar.scss#L54
to have:
.#{$namespace}-userbar-reset {
  all: initial;
  // Copy our font sans variable so it can be used without Tailwind
  --w-font-sans: -apple-system,BlinkMacSystemFont,"Segoe UI",system-ui,Roboto,"Helvetica Neue",Arial,sans-serif,Apple Color Emoji,"Segoe UI Emoji","Segoe UI Symbol","Noto Color Emoji";
}

The problem we’re facing is that the variables created with Tailwind, such as var(--w-font-sans), aren’t defined for the userbar. This is because the userbar is technically displayed on other people’s sites – it’d be very dangerous for us to add lots of global styles there.
Ideally we’d refactor the code much more so we don’t have those kinds of issues anymore, using a Web Component – but for the time being I think we can go with the approach I suggested of hard-coding the font.

Ok, noted i'll try to work on this and send update soon

Just updated the title to be a bit clearer also.

Thanks @thibaudcolas
Sorry for the confusion @fyunusa - I'm still learning parts of tailwind myself and thought we could still use its utilis in other scss files.


You don't need to unregister your events in the onDisable()

You don't need to unregister your events in the onDisable()

This repo isn't actively maintained anymore


stats: symbolize strings in HeapStatData and ThreadLocalStore

Description: On the heels of https://github.com/envoyproxy/envoy/pull/3927, and in pursuit of lower heap consumption for Envoy (https://github.com/envoyproxy/envoy/issues/3585), this implementation uses the new Stats::SymbolTable library to symbolize std::string instances within HeapStatData and ThreadLocalStore.
Preliminary heap allocation improvements here:

short
short
long
long

current impl
w/ symbol table
current impl
w/ symbol table
improvement
improvement

clusters
runtime
allocated on heap
allocated on heap
allocated on heap
allocated on heap
short
long

(#)
(sec)
(kB)
(kB)
(kB)
(kB)
(%)
(%)

10
30
2484
2481
2637
2564
-0.15%
-2.77%

100
45
7996
8008
9552
8854
0.14%
-7.31%

1000
60
63145
63158
78793
71702
0.02%
-9.00%

10000
90
3566177
3480433
5509859
4420003
-2.40%
-19.78%

(For instructions on how to run this sort of analysis, see PPROF.md.)
Here, 'short' character names are ~10 characters long, 'long' character names are ~60 characters long, and all runs are against an envoy-static binary with --disable-hot-restart.
Risk Level: Medium -- this affects non-hot-restart binaries, so there won't be any issues with trying to restart against a different internal memory representation. But any PR which changes internal memory stuff comes with some risk.
Testing: As a drop-in replacement for HeapStatData / ThreadLocalStore, there are no new tests for these classes.
Docs Changes: N/A
Release Notes: N/A, no user-facing changes.
Fixes https://github.com/envoyproxy/envoy/issues/3585

@mrice32 can you take an initial pass at this? Thanks

Looks like asan/tsan caught a mutex issue inside thread_local_store.cc -- fixing now.

@all, I'm continuing to profile this for both heap alloc. and time performance. This won't be ready to review (let alone merge) until I'm confident it doesn't trade memory for slowness.

@PiotrSikora  I'm running bazel build //source/exe:envoy-static -c opt and then  bazel-bin/source/exe/envoy-static --disable-hot-restart --config-path <path>, where <path> points to a file like https://gist.github.com/ambuc/50181fa5f64485041696730c67e9f29f, except I have variants containing 10, 100, 1000, and 10000 clusters, with both short "cluster" n and long "cluster.abcdefghijkl.abcdefghijklm.abcdefghijklmn.cluster" n names. I'm not passing any explicit concurrency arguments when running these tests.

@ambuc but how are you checking memory usage? Using RSS value from the ps ux output or using server.memory_allocated (i.e. curl -s <IP_ADDRESS>:9901/stats | grep server.memory_allocated)?
As for the concurrency, you can check it via curl -s <IP_ADDRESS>:9901/stats | grep server.concurrency, which should shed some light on your numbers.

Per Piotr's comment, can you edit the description in this PR to include a pointer to the "how to run memory profiling" thing that you wrote, right above or below the table?

Also you pinged me a table showing the CPU consumption impact of this PR.  Can you add that to the description as well?

@PiotrSikora I'm not convinced that using server.memory_allocated is a particularly good way of inspecting memory usage, since it actively consumes memory which was not in-use before the call. Check out https://github.com/envoyproxy/envoy/blob/master/source/server/http/admin.cc#L524-L535 for a by-inspection proof of this -- for large configs, the creation of all_stats is a nontrivial amount of storage. I will perform this analysis via ps ux.
@ all, is this a "bug"? Should I take out an issue to fix this behavior? It would be nice for a memory monitoring feature to not significantly change the memory on which it reports.

@ambuc yeah, I'm aware of this (and I had a patch that emits memory stats to logs upon receiving SIGUSR2), but the memory overhead of generating /stats is negligible (in grand scheme of things) and is much lower than the overhead of tcmalloc free lists, so it was still much more accurate in my tests than RSS from ps ux.

I suspect that in the context of this benchmark which generates large numbers of long-named clusters, the overhead of generating all /stats is likely to be significant, as it makes a map of fully elaborated strings for all stat names, thus defeating the purpose of storing them as a vector of shared symbol references in steady-state.
I think if we add a query-param to /stats to just emit a single stat without elaborating all of them into a map, the results would more accurately reflect the value of what James has done.


Fix Partitions Script in CAMDECMPS Schema

The MATS tables in the ECMPS schema have partitions going back to 1993Q1.  We started collecting this data in 2015 so the data in the partitions prior to 2015 will always be empty.  These empty tables were causing poor performance of a SQL query.
Check the partitions script to so it only creates the partitions based on the data that is available.
https://github.com/US-EPA-CAMD/easey-db-scripts/blob/develop/camdecmps/partitions/create-partitions.sql

Document what the earliest begin year should be for each table. Drop any empty partitions. Shouldn't be hardcoded to end in 2024.


Window resize support

How do you feel about adding support to "resizing" window support? I found that is a pretty useful functionality, but not sure if that fits in this module.
I can PR if you found this could be useful :)

Definitely a good idea! Feel free to submit a PR...


laravel Auth::attempt passed but redirect to login page

--

You have to regenerate your session in order to authenticate successfully. Here you can read more: https://laravel.com/docs/10.x/authentication#authenticating-users Try to add this line to your Code:
 $request->session()->regenerate();

SESSION_DOMAIN issue delete / end of line


How to add JavaScript into epub

Am sorry if this isn't the right place to ask for help. Am joining today.
I had my ebook as HTML file (within a webview app) with a tagger script in the head tag of my HTML file. The script automatically adds link tags to famous names, these links shows a dialogue box onclick, with a brief bio of the famous person.
Is there anyway I can put that JavaScript in this reader so it is injected into the ebook when they are opened in the FolioReader app?
Thanks

Meant to say
*with a tagger script in the head tag of my HTML file.

No help?

@Ijaycool i don't think, there is anyway to inject your own javascript into the reader.. but if you add the script in epub html, i think, it should call javascript function to display dialog box..

@mahavir155 thanks for responding.
Let me clear, if there is js in my HTML (zipped as epub) FolioReader will call the js function?
If that is correct, then that's amazing.

@Ijaycool Yes, i think it should call.. we don't remove existing JS functions.. we just inject some JS.. Can you please try if you have just epub and let us know if it work?

@mahavir155 thanks.
I tried, FolioReader ignored my js. Can't I hardcode the js into the reader?
Since it's the single ebook I wish to access.
Thanks


Add a test case for false positive

There was one false-positive in our project. Instead of opening an issue, I thought it would be better to have a failing test case.
If a dependant module uses resValue, then the generated resource usage is not detected.

Thanks for the test!
I'll be getting to this very soon.

Marking it as draft makes sense 😅
I actually tried to fix it. One thing that I realized is that you have a task dependency to buildConfig tasks but not to resValues. Very similarly in couple of lines, I did this but it did not help.
I debugged (by putting bunch of print statements 😂) everything seems to be fine but the resources source sets somehow do not include the generated resources folder. I couldn't figure out why.

@Dependabot recreate

@Dependabot recreate


[Release-1.28] - Spegel mirror returns 500 instead of 404 on unavailable images

Backport fix for Spegel mirror returns 500 instead of 404 on unavailable images

#10117

Version has been bumped and will have 500 errors correctly instead of 404s.


Create SECURITY_POLICY.txt

This adds a link to the security policy.
Depends on: https://github.com/maplibre/maplibre/pull/141

@nyurik This is ready for merge as https://github.com/maplibre/maplibre/pull/141 was accepted.


improve SDP-atom canonicalization efficiency (#877)

This PR addresses the issues mentioned in #877. Fewer variables and fewer equality constraints are introduced in the compilation process. The bmat atom is used more often.
The only questionable part of this PR is the introduction of a vec_to_upper_tri function in upper_tri.py. If you want me to make this function an actual atom, or to write documentation for it, let me know. Tests for this function have been added to test_atoms.py.
I did not write additional tests for any of the existing atoms (normNuc, lambda_max, sigma_max, matrix_frac).

It looks good to me! Thanks for doing this!


Implement DestructibleHider from the live vJASS version of the map

Note that this ticket is pretty hard.

@YakaryBovine I'll take a stab at this one


Remove webinterface json config

This folder was used to define user-submitted (not public submitted) parameters:
webinterface/configuration
Not needed anymore, but cannot be removed due to dependencies.

Decided to keep this as it is important for initializing default values for params. Also might be relevant for #477


Report missing required components

As a User, I wan't to get clear and quick feedback when I make a mistake, so that I avoid spending time guessing/debuging in order to make my game work.
For example the Body component needs the entity to also have a GlobalTransoform. And a Velocity works only on dynamic bodies.
When a required components is missing it can be considered a programming mistake of the user of this library, and Heron should at least report the situation, so the user is aware of it.

I've actually twice now forgotten to include RigidBody alongside CollisionShape, it might be nice to show a warning for that one (collision shape with no rigidbody or vice versa), unless there's some use case where you'd only want one


foreign key constraint

Sorry, I cannot work just with an image without any context.
Please, try to create a reproducible example in a repo, or at least, a decent amount of code.

Nevermind, this was related to my use of the manager in the subscriber:
Replacing manager.create(TicketEvent, {}).save() by manager.save({}) fixed the issue 👌


Better error message when failing to add users because of "login name" field not being unique.

Server version - v0.37.0-rc2
Local Auth enabled in rancher-server.
Create a user with login name say "testme1" .
Create another user with the same login name - "testme1". We are not allowed to create this user.
User is presented with following error message:
"Validation failed in API: publicValue is not unique"
Can this error message be improved to indicate that the "login name" field is not unique.

Closing due to inactivity.


feat: add settings to enable inlay hints individually

Summary of Changes
Inlay hints for

parameter names
assignee types

can now be turned on/off individually.

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

npm package (@latest dist-tag)
npm package (@latest dist-tag)
v0.10.0
GitHub release

Your semantic-release bot :package::rocket:


Install Snapshot CRDs & Snapshot Controller for K8s 1.17 or higher

Issue description
With K8s 1.17 the Snapshots are moving from alpha to beta and there are a lot of changes in the CSI. One change is that K8s requires to install the Snapshot Beta CRDs and the Snapshot Controller (https://kubernetes-csi.github.io/docs/snapshot-controller.html#deployment).
Improvement
It would be very easy for Trident to check in the installation phase (https://github.com/NetApp/trident/blob/master/cli/cmd/install.go) the K8s release and apply the required yaml files if the K8s version is 1.17 or higher.

See our blog here: https://netapp.io/2020/01/30/alpha-to-beta-snapshots/

Yes, I did read the blog. But as far as I understand and I have been said the beta CRDs and the snapshot-controller needs to be created manually before install Trident 20.01 or above. I'd like to see Trident checking the K8s version and creating the CRD and snapshot-controller by itself.

@pablogarciaarevalo That behavior is essentially what the alpha snapshot sidecar used to do for us. The Kubernetes community decided that it is not the place of the sidecars or storage provisioners to be adding these CRDs or controllers, especially while in alpha/beta state; this should need be explicitly enabled by the Kubernetes admin. Thus it is the responsibility of the admin or Kubernetes distribution to install these components themselves if they wish for the functionality to be enabled.
Having the alpha CRDs be auto-installed actually did get us into trouble, since they are not compatible with the beta versions and must be completely removed from the cluster before you can properly use beta level snapshots. This means that users who never wanted or intended to use the snapshots while in alpha are still stuck having to remove these components when upgrading. So, again, to avoid this we're requiring that admins, or distros, handle the installation of the CRDs and controller moving forward.

Understood. Thanks a lot for the detailed answer.


No carthage support?

as title .

Thanks for mentioning - apparently we lost it with the v1.0 release - will be added shortly with the next release.

This should help - https://github.com/quickbirdstudios/XCoordinator/pull/52


Development

Possibility of further development or making this project open source?

The project is open source. I think stable diffusion might be a better starting point if you're looking for text to image capabilities.


[WIP] fix: race condition on data

This is caused by the data loading late waiting for a static query that is in the preview and shouldn't be.
What kind of change does this PR introduce?
A bugfix to fix and close #226
Does this PR introduce a breaking change?
Should not introduce a breaking change.
What needs to be documented once your changes are merged?
There really is not anything to document, because this should be exactly the same process.
See notes in #226. Using a StaticQuery in a preview should be avoided. NetlifyCMS is not in the build with Gatsby and does not have access at this time to graphql.

@ZoltanVeres this is how I would do the previews with functional components.
React hooks is a better solution for these previews, because we can watch for changes on entry and setup state variables for the data with little effort. Hopefully we will have a solution soon that will support React hooks in the plugin.

We really need to figure out a way to use a Template for this entry without the blog data component that has the static query in it.

@talves The blog component is used in two pages the in the /blog and in index page.
I used staticQuery to save on build time, and becuse basically they are indentical in the markup level as well, if this is as big of a problem as we think then how about using useStaticQuery and cache the data only, this can be seen in use in the SiteMetadata component, this is relatively a trivial change.
I'm all about supporting React hooks, we can try and rewrite one tamplate, so i can understand what are steps to convert a template.

We can't use react hooks yet, but eventually.

Sounds good.


Improve documentation of Version usage for Swarm API

This fix is an attempt to improve the documentation of Version usage for Swarm API in Swagger docs.
As is shown in the docs updates, Version field in Swarm API is a way to ensure safe concurrency and determinism so that multiple updates to the same swarm related objects (Node, Service, etc) will not unintentially overwrite each other.
This fix fixes #29448.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

SGTM
cc @aaronlehmann @stevvooe

This is also needed for the Version fields of ClusterInfo, Task, and Secret. For some reason I don't see Network defined in swagger.yaml.

Please see https://github.com/docker/docker/pull/30954/files#r100929925.
I'll let @aaronlehmann take care of the bounds of "implementation-specific", but the constraint is that the client must write back the version number from when they read the object and the api may reject it.

Thanks all for the review. The PR has been updated. Now a definition ObjectVersion has been added for de-duplication to Node, ClusterInfo, Task, Service, Secret. The description has also been updated. Please take a look and let me know if there are any issues.

LGTM


Pagination for blocks mined by account

Rebased on #173
This PR implements pagination for mined blocks on the account details page. Also moves the GUI caching code into its own file which means we don't need to be concerned with the mutex handling outside of this file. Should also make #167 easier.

Closing - these changes will be included in #180 instead


Create an analyzer to identify usage of APIs that UA can't upgrade

This will involve identifying namespaces, types, or members that users need to upgrade manually and displaying relevant information and links to documentation.
The analyzer should use config files so that users and extension authors can easily add their own APIs mapped to messages that the analyzer will use.

Note that this will require the source update step to change (or a new step to be added) so that analyzers without associated code fix providers can be used.


Please tutorial how to fix "error:0909006C:PEM routines:get_name:no start line" with algorithm: "RS256"

Please do not report security vulnerabilities here. The Responsible Disclosure Program details the procedure for disclosing security issues.
Thank you in advance for helping us to improve this library! Please read through the template below and answer all relevant questions. Your additional work here is greatly appreciated and will help us respond as quickly as possible. For general support or usage questions, use the Auth0 Community or Auth0 Support. Finally, to avoid duplicates, please search existing Issues before submitting one here.
By submitting an Issue to this repository, you agree to the terms within the Auth0 Code of Conduct.
Description

Provide a clear and concise description of the issue, including what you expected to happen.

Reproduction

Detail the steps taken to reproduce this error, what was expected, and whether this issue can be reproduced consistently or if it is intermittent.
Where applicable, please include:

Code sample to reproduce the issue
Log files (redact/remove sensitive information)
Application settings (redact/remove sensitive information)
Screenshots

Environment

Please provide the following:

Version of this library used:
Version of the platform or framework used, if applicable:
Other relevant versions (language, server software, OS, browser):
Other modules/plugins/libraries that might be involved:

I have the same issue.
please give me solution if you have

I have the same issue. you must use default algorithm for now.

@taylankarakas
I've hidden your suggestion. The point behind using an RS private key is so that noone but you can produce the signatures but everyone with the knowledge of your public key can verify it. HS256 is an HMAC based symmetric key (secret) algorithm and you'd be using the octets of malformed private key as the shared symmetric secret. DON'T DO THAT.

error:0909006C:PEM routines:get_name:no start line

How to fix it? Provide a properly formatted pkcs8, pkcs1, or sec1 PEM private key. That's really it.

Create JWT Token using the command shown here

For us we had this issue while loading a private key from ENV instead of files (because of automated deployment in aws).
We fixed it by replacing \n in the env var with real line breaks
process.env.JWT_PRIVATE_KEY?.replace(/\\n/gm, '\n')
Hope this helps someone

Continuing with @derN3rd 's answer, I had to approach this slightly differently.
For me, I was storing my private rsa key in a Gitlab CI/CD environment variable, which I was then reading into a file (this file was then read by the code I was testing).
touch priv_key.pem
echo "$MY_PRIV_KEY_ENV_VARIABLE" > priv_key.pem

Where I was going wrong was in the echo statement.  As you see above, I am surrounding the environment variable with double-quotes.  This is significant because by surrounding the variable with double-quotes, it preserves the \n character in the private key.
When I was just using the statement echo $MY_PRIV_KEY_ENV_VARIABLE > priv_key.pem, it was adding spaces where the \n character was and causing the error mentioned in this issue error:0909006C:PEM
Source - https://stackoverflow.com/a/50016491/7437737

For us we had this issue while loading a private key from ENV instead of files (because of automated deployment in aws).
We fixed it by replacing \n in the env var with real line breaks
process.env.JWT_PRIVATE_KEY.replace(/\\n/gm, '\n')
Hope this helps someone

Very useful, tks.

Just to add a bit of clarification to @derN3rd 's solution, which is great btw, adding \ns to the env variable is a necessary step, prior to replacing them on the client side.
I also did not use quotes to surround the value.
Something like this:
-----BEGIN PRIVATE KEY-----\nLONG_STRING_HERE\n-----END PRIVATE KEY-----

i mean if we validate the file's contents with openssl then there must be some other problem going on?

For Windows users with PowerShell and OpenSSL.Light installed who needs to extract everything between ----BEGIN CERTIFICATE----- and ----END CERTIFICATE-----:
openssl s_client -showcerts -connect YourWebsite:port | Select-String -Pattern '-----BEGIN' -Context 0,21 | Foreach { $_ -replace "  ", ""} | Foreach { $_ -replace "> ", ""} | Foreach { $_ -replace "/\\n/gm", "\n" } | Set-Content 'OutputFile.crt'

I got this because I was accidentally signing with my public key 🤦♀

For us we had this issue while loading a private key from ENV instead of files (because of automated deployment in aws).
We fixed it by replacing \n in the env var with real line breaks
process.env.JWT_PRIVATE_KEY.replace(/\\n/gm, '\n')
Hope this helps someone

I selected every reaction. This helped me so so so much. THANK YOU

For us we had this issue while loading a private key from ENV instead of files (because of automated deployment in aws).
We fixed it by replacing \n in the env var with real line breaks
process.env.JWT_PRIVATE_KEY.replace(/\\n/gm, '\n')
Hope this helps someone

For me it did not work in Google Cloud Platform Cloud Functions. Deploy works but function crashes with the error code.
I got tired of the error so I use a javascript string litteral and copy pasted my private key there instead of the process.env variable

For us we had this issue while loading a private key from ENV instead of files (because of automated deployment in aws).
We fixed it by replacing \n in the env var with real line breaks
process.env.JWT_PRIVATE_KEY.replace(/\\n/gm, '\n')
Hope this helps someone

For me it did not work in Google Cloud Platform Cloud Functions. Deploy works but function crashes with the error code.
I got tired of the error so I use a javascript string litteral and copy pasted my private key there instead of the process.env variable

Try this:
iconv -c -f UTF8 -t ASCII  myprivate.key >> myprivate.key
Converting from utf-8 to ASCII made it work for me 😄
ref: https://stackoverflow.com/questions/43729770/nginx-godaddy-ssl

Try this:
iconv -c -f UTF8 -t ASCII  myprivate.key >> myprivate.key
Converting from utf-8 to ASCII made it work for me 😄
ref: https://stackoverflow.com/questions/43729770/nginx-godaddy-ssl

I was placing the key and crt interchangeably. So placing it rightly solve mine.

I was placing the key and crt interchangeably. So placing it rightly solve mine.

I accidentally exchanged private key and certificate.

This is the complete solution of the problem.
First to generate SSL certificates, then create a HTTPS server via these certificates, after that implement Secure Web Sockets.
1st:
Generate SSL certificates via OPENSSL.
These are the 3 commands
openssl genrsa -out abels-key.pem 2048
openssl req -new -sha256 -key abels-key.pem -out abels-csr.pem
openssl x509 -req -in abels-csr.pem -signkey abels-key.pem -out abels-cert.pem
2nd: Code
const express = require("express");
const https = require("https");
const fs = require("fs");
const WebSocket = require("ws");
const app = express();
let cert = fs.readFileSync("abels-cert.pem");
let key = fs.readFileSync("abels-key.pem");
const options = {
key,
cert,
};
app.get("/", async (req, res) => {
res.send("Server is Running on HTTPs and WSS");
});
var server = https.createServer(options, app);
server.listen(443, () => {
console.log("Server is Running on PORT 443");
});
const wss = new WebSocket.Server({ server });
wss.on("connection", function connection(ws) {
console.log("Connection has been established successfully");
ws.on("message", function incoming(message) {
console.log("received: %s", message);
});
ws.send("something");
});
Note:
In Online server you may face 3 problems,
1st PORT
2nd (URL), WSS will not work with IP Address (In my Case new WebSocket("wss://localhost") its work fine, new WebSocket("wss://<IP_ADDRESS> or wss://<IP_ADDRESS>:443")) not working as expected.
3rd Certificates issues.
There are some online resources which helps us to validate our certificates

Hello, everyone! Another possible way is to have both: private and public key already (.crt. and .key), then:

cat file.key > file.pem
cat file.crt >> file.pem

Because our .pem is a concatenation of both files

npm install jsonwebtoken
npm install jwk-to-pem
const jwt = require("jsonwebtoken");
const jwkToPem = require('jwk-to-pem')
const keyObjectInJWTformat =
{
use: "sig",
kty: "RSA",
kid: "ins_21yJfbfkj-shortstring-Rk9vqfdsfdshasCu1i",
alg: "RS256",
n: "0Cbcyd-longltring-j5hzvZViw",
e: "AQAB"
},
const pem = jwkToPem(keyObjectInJWTformat)
console.log(PEM)
-----BEGIN PUBLIC KEY-----
MIIBIjANBgkqhkiG9dsfdsfdsfgKCAQEA0Cbcyd+01Wb8X6eWSct1Qz3qG8txsfsdfdApvWhopetosaveyouadayxGYq+S4EEFvO/z1luNhZeNXRPLgg9fsdlsdjaPk5FWvYWbMgNmTt/rpdZYSChda4opensourceh*llAme0zPUp+TbkX+OQ/cdffsfsQJ84uVjmjiBeHmQgZSWWOHNOcqGA6icap7JY0erBNIstoh1yfsdUH0Fs9WowBXiwci9B8lAjQtD8YOLk/dnEznt91tAp3C6vsdfds2zePSIgxCUT6sbytwj5hzvZViwIDAQAB
-----END PUBLIC KEY-----
jwt.verify(token, pem,  { algorithms: ['RS256']}, (err, decoded) => {
if(decoded) console.log(decoded);
if(err) console.log(err.name, err.message);
});

Had this same issue. My problem was I used the auth0.pem file downloaded from Auth0 dashboard > tenant settings > Signing keys, but that is actually a provate key!. To validate the JWT token you need to generate the .pub file from that certificate. So, I had to run:
openssl x509 -pubkey -noout -in auth0.pem  > pubkey.pem
And use the pubkey.pem to verify your JWT tokens.


Maven annotationProcessorPaths (APT) required for NullAway intereres with other existing APT processors on project classpath

I'm exploring using NullAway (v0.3.2) in a (large) existing Maven-based project which is not going to switch from Maven to e.g. Gradle any time soon, and have set it up as documented on https://github.com/uber/NullAway/wiki/Configuration#maven.
Some modules of this large existig project use at least one existing APT based processor; I've hit the snag related to http://immutables.github.io. This APT processor is not registered via the maven-compiler-plugin's annotationProcessorPaths but is a regular Maven <dependency> on some modules classpath; as recommended by Immutables documentation (also because their same JAR contains both an APT processor as well as annotations).  The problem is not related to Immutables.org per se and would occur with any other APT processor as well, if its a regular <dependency>.
Because the maven-compiler-plugin, apparently, for annotationProcessorPaths "If specified, the compiler will detect annotation processors only in those classpath elements. If omitted, the default classpath is used to detect annotation processors.", using NullAway deactivates an existing APT processor such as Immutables.org ... :-(
I can put together a simple standalone project to illustrate this problem, if it would help to clarify.
Some questions / points for discussion:

Does NullAway have to be on the annotationProcessorPaths of the maven-compiler-plugin? Couldn't it also at least optionally integrate with error-prone as a dependency of maven-compiler-plugin, just like error_prone_core itself?

Would you consider this "interference" of an APT processor on maven-compiler-plugin's annotationProcessorPaths with one it detects on the regular dependencies consider a maven-compiler-plugin problem? Then all hope is lost... :smile:

One possible workaround I've found is that if I put Immutables.org on the annotationProcessorPaths in addition (!) to being on the project's dependencies then it works. The trouble with that is I'm not sure that is feasible at scale - doing this would mean that hundreds of Maven modules suddenly have that particular APT processor activated, whereas before it was a dependency of only a handful of them... not sure about the build performance overhead this causes and would love to find a better solution - if anyone has any ideas?

BTW: Working around "the other way", and having nullaway as a regular dependency instead of an annotationProcessorPaths doesn't work, but that's probably because error_prone_core is on the classsloader of the maven-compiler-plugin so it cannot the nullaway via ServiceLocator... but "polluting" (uncessearily) the project dependencies classpath with nullaway isn't great either anyway.
com.google.errorprone.InvalidCommandLineOptionException: NullAway is not a valid checker name

@vorburger the proper fix would be to unwind immutables into tow artifacts. One for annotations and one for the compiler plugin. Many processors do it this way today like Dagger, Buttekrnife etc.
Autovalue is moving to this format as well. This seems like an issue with the maven compiler plugin really. The proper way to go about it would be to make an upstream issue on immutables to also provide an option for separate artifacts one for annotations and one with the apt compiler plugin. They can continue to ship the artifact which bundles both together as well.
In the meanwhile, you could split up the artifact into two yourself for use in your codebase till the issue is fixed upstream.

Thanks for the report @vorburger.  I think @kageiit has a good solution which will hopefully be workable.  Amazingly, just two days ago we were debugging an incompatibility between NullAway and Lombok when using Maven, and I now realize it's the same problem as this issue!

Ok, I thanks for your feedback. I'll write up an issue proposing split Immutable.org JARs hopefully tomorrow, and then probably close this.

==> https://github.com/immutables/immutables/issues/750


DuplicateDefinitionError when defining a :post factory

In spec/factories/post_factory.rb:
FactoryGirl.define do
  factory :post do

    user
    title Faker::Lorem.sentence
    body Faker::Lorem.paragraphs

  end
end

With that, I get this error:
... factory_girl-2.6.1/lib/factory_girl/registry.rb:39:in `add_as': Factory already registered: post (FactoryGirl::DuplicateDefinitionError)

If I change factory :post to factory :a_post, class: Post or any other name, it works fine. The only other factories I have defined are user and session (and those work fine).
Am I doing something wrong, or is there some internal post factory that is automatically defined which prevents me from using that name?

There are a couple of potential issues, assuming you're 100% positive no other post factory exists (that's the first thing I'd check). Are you using the factory_girl_rails gem and Rails 3? If you are, make sure you're not requiring factory_girl_rails anywhere in your spec_helper or any other file that'd be loaded during your tests. Also, make sure you're not calling FactoryGirl.find_definitions, which will load all factories; running this twice is a surefire way to see this error. Finally, are you calling require "spec/factories/post_factory" anywhere? factory_girl_rails will load up all the factories so this is unnecessary.
FactoryGirl doesn't create any factories for you by default so it's not going to interfere with your factory.

Welp. Apparently when I generated the post model, a test/factories/posts.rb was created, but I was only looking in the spec directory since I use RSpec.
I removed that and everything works. Thanks.

Ah, awesome! I figured that was likely the culprit.


Math with units parameter doesn't work with vlookup linked value

vlookup pulls a value from a table and displays the correct unit it retrieved. When a link is created for that value, the unit is not stored in the link. When you reference that linked value in Math in order to convert the result to a common unit, the Math function doesn't know what unit it is dealing with. For example...
Tile1:
!PU(vlookup):link=table1;val=www.azure.easytravel.com;col=1;base=low;warn=3;crit=5;size=28px
!PU(link):shady1
Tile2:
!PU(math):exp=(a);scope=a:shady1
Tile3:
!PU(math):exp=(a);scope=a:shady1;unit=S
Tile1 shows: 777 ms
Tile2 shows 777
Tile3 causes the following error:
POWERUP: ERROR  ReferenceError: val is not defined
at Object.pub.findLinkedUnit (:4:25214)
at :4:103630
at Array.forEach ()
at HTMLParagraphElement. (:4:103557)
at Function.each (jquery-3.6.0.min.js:2)
at S.fn.init.each (jquery-3.6.0.min.js:2)
at S.fn.init. (:4:102562)
at Function.each (jquery-3.6.0.min.js:2)
at S.fn.init.each (jquery-3.6.0.min.js:2)
at Object.pub.PUMath (:4:102389)
at Object.pub.fireAllPowerUps (:4:141732)
at mutationsDone (:4:143839)
at :4:143387
at rxthandler (ruxitagentjs_ICA2EQSVfhjqrtux_10223210811140219.js:573)

typo fixed


Color goes to white between each change

I've got a B450 Tomahawk, the tool works great except that between each color change the color is changing to white. So it's like color1 - white - color2 - white - color3 - white and so on. Is it supposed to be like that?

It isn't supposed to do that on supported motherboards. I'm afraid the B450 TOMAHAWK mobo is not supported.

Hi, I have the same mobo and it is actually supported, however you first have to run MSI Mystic Light software and turn the RGB off in there, otherwise the effect selected there is causing issues to effects from this utility:3.
@ixjf you might want to add a note to the readme about it. Anyway thanks, your app allows for much smoother control than the Mystic Light app.

thanks!:)

By supported motherboards I was referring to the list of motherboards that are on the README in the 'Supported motherboards' section. As it says on the README, it's entirely possible the LED control works the same as in those but is not one of the motherboards I can (kind of) guarantee the tool will work properly on.
I'd expect that disabling MSI Mystic Light is an obvious thing to do before attempting to run MSIRGB. Perhaps alternatively I'll add a check to MSIRGB to warn the user if Mystic Light/Gaming App is on.
Anyway, so the tool works properly on the MSI B450 TOMAHAWK then?


rootfs on external (SATA) device

Hello!
I'm wondering if there is a way to use SATA as default rootfs location.
As I know u-boot should be located at eMMC partition anyway, but maybe it's possible to keep u-boot and rootfs separately on different devices?
My goal is mender integration + rootfs on SATA, so I can't just copy rootfs to SATA from eMMC after flashing
Thank you in advance

Which Jetson are you planning on using?  You mention U-Boot, so I'm guessing either a Nano or a TX2.  I think U-Boot for Nano includes SATA support, but I'm not sure if it's there on the TX2.  That would be the first thing to check.
A few other things off the top of my head:

You would need to develop an installer/recovery image to help with getting the SATA drive partitioned, formatted, and loaded up with the rootfs as part of initial flashing.
You would need to adjust the Mender variables that indicate where to boot from to configure the right name and partition numbers on the SATA drive in U-Boot and in Linux.  The default setup assumes, of course, booting from the eMMC.
Double-check that the SATA drive you're looking to use works properly with the driver/controller on the Jetson.  I haven't used SATA for booting, but I have used it for data storage in a couple of products in the past, and ran into some compatibility issues.

An alternative to directly booting from the SATA drive from U-Boot, you could put a kernel/initramfs into a partition on the eMMC, point U-Boot at that, and then have the init script in the initramfs mount the SATA drive for you.

Thank you for reply!
Forgot to mention, I'm using TX2. I'm sure that U-boot is able to boot SATA-based rootfs, already tested with Ubuntu 16.04.
As I understand, partitioning for SATA should be added to flash.xml file, right?
But how can I specify the root partition for flasher? Original nVidia "flash.sh" script can read special file called "l4t-rootfs-uuid.txt" to determine rootfs drive, but I was unable to find any mention of this file in meta-tegra
Your concern regarding SATA drive is quite unexpected, can you please tell what kind of compatibility issues you faced with?

As I understand, partitioning for SATA should be added to flash.xml file, right?

I don't think the tegraflash tool supports "flashing" to a SATA drive, which is why I suggested the two-step process - flash an installer/recovery image to the MMC using tegraflash.  Have the tools in that image to format and install your final rootfs onto the SATA drive.

But how can I specify the root partition for flasher? Original nVidia "flash.sh" script can read special file called "l4t-rootfs-uuid.txt" to determine rootfs drive, but I was unable to find any mention of this file in meta-tegra

All that file does is tell tegraflash to assign a UUID to the APP flash partition, and to update the kernel command line with a root= parameter to specify that UUID.  You could do something similar, although if you're using Mender, you need to specify device names and partition numbers.  U-Boot also may need customization to get it to look at the SATA drive to find the rootfs.

Your concern regarding SATA drive is quite unexpected, can you please tell what kind of compatibility issues you faced with?

It's been a few years, but IIRC it had to do with DEVSLP (device sleep) support.  The controller or driver didn't fully support that feature, and I think I had to hack the driver to just ignore it when it was advertised by the drive.  That may not be an issue any more, or may have been something unique to the drive I was testing at the time.


install-plugins.sh: line 58: $3: unbound variable

commands that you execute
pluginList="active-directory:2.12 basic-branch-build-strategies:1.2.0 blueocean:1.14.0 cloudbees-bitbucket-branch-source:2.4.2 kubernetes:1.14.9 lockable-resources:2.5 matrix-auth:2.3 pipeline-stage-view:2.10"
echo  ${pluginList} > /var/jenkins_home/pluginList
export REF=/var/jenkins_home/plugins; /usr/local/bin/install-plugins.sh $(cat /var/jenkins_home/pluginList)
if [ $? -ne 0 ]
then
  rm -v /var/jenkins_home/pluginList # force re-installation on next run.
  echo ".... install-plugins failure."
  exit 1
else
  echo ".... install-plugins success."
fi

Actual result
In the event that one or more plugin(s) fail to download...
Example log (some log content removed for clarity. full log)
...
curl: (18) transfer closed with 2469386 bytes remaining to read
13:50:12 Failure (18) Retrying in 1 seconds...
...
curl: (18) transfer closed with 2281307 bytes remaining to read
13:50:14 Failure (18) Retrying in 1 seconds...
...
curl: (18) transfer closed with 2469318 bytes remaining to read
13:50:24 Failure (18) Retrying in 1 seconds...
...
13:50:25 Failed in the last attempt (curl -sSfL --connect-timeout 20 --retry 3 --retry-delay 0 --retry-max-time 60 https://updates.jenkins.io/2.164/latest/git-client.hpi -o /var/jenkins_home/plugins/git-client.jpi)
/usr/local/bin/install-plugins.sh: line 58: $3: unbound variable...
...

After manipulating the plugin name, the download is re-attempted... However, because $url is not passed to doDownload...  $3 is unbound.
In my experience the unbound variable runtime error does not cause the install-plugins.sh script to fail...
...
workflow-scm-step:2.7
workflow-step-api:2.19
workflow-support:3.2
Cleaning up locks
.... install-plugins success.

Interestingly checkIntegrity logs an error as well, but does not cause the script to fail.
[/var/jenkins_home/plugins/git-client.jpi]
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.

Expected outcome
one of the multiple logged errors to cause install-plugins.sh to return non-zero.

It seems to me that at a minimum. $url should be included in the second call to doDownload

After further review, I can see why the unbound variable error is not caught...
download is called asynchronously and expected to log any failures to $FAILED which is checked later to determine success/failure.

However, I don't understand why checkIntegrity is logging failure, but failing to trigger failure via $FAILED
the logic is sound/tests well in the example below.
$ timeout 1 curl -sSfL --connect-timeout 20 --retry 3 --retry-delay 0 --retry-max-time 60 https://updates.jenkins.io/2.164/latest/git-client.hpi -o git-client_incomplete.jpi
$ unzip -t -qq ./git-client_incomplete.jpi >/dev/null
[./git-client_incomplete.jpi]
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of ./git-client_incomplete.jpi or
        ./git-client_incomplete.jpi.zip, and cannot find ./git-client_incomplete.jpi.ZIP, period.
$ echo $?
9
$ curl -sSfL --connect-timeout 20 --retry 3 --retry-delay 0 --retry-max-time 60 https://updates.jenkins.io/2.164/latest/git-client.hpi -o git-client.jpi
$ unzip -t -qq ./git-client.jpi >/dev/null
$ echo $?
0

ah... it is not checkIntegrity that is logging the unzip error..
... checkIntegrity is never called for "git-client" because doDownload throws the unbound variable error before checkIntegrity can be called...
instead... the unzip error that is logged seems to be coming from get_plugin_version

With #815 merged, this issue can be closed.


rna-transcription: don't transcribe both ways

I can't remember the history of this, but we ended up with a weird non-biological thing in the RNA transcription exercise, where some test suites also have tests for transcribing from RNA back to DNA. This makes no sense.
If this track does have tests for the reverse transcription, we should remove them, and also simplify the reference solution to match.
If this track doesn't have any tests for RNA->DNA transcription, then this issue can be closed.
See https://github.com/exercism/x-common/issues/148

This issue can be closed I believe. It doesn't look like the test does anything for RNA->DNA.

Thank you :heart: @mienaikage!


BPF Recorder Cleanup + Child Process Tracing

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
This PR cleans up the BPF C code to remove some duplication.
It also adds support for tracing child processes in AppArmor profiles. I have more AppArmor improvements ready for a separate PR, but this particular one is a bit hard to take out of the existing commit chain. Hope that's ok. :)
Which issue(s) this PR fixes:
None
Does this PR have test?
Yes, in #2141.
Special notes for your reviewer:
The best way to review this is probably to go through the individual commits, I've tried to keep them fairly targeted on doing one specific thing. :)
Does this PR introduce a user-facing change?
NONE

/ok-to-test

Codecov Report

Merging #2155 (a1a755b) into main (11d77f4) will decrease coverage by 4.55%.
Report is 76 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2155      +/-   ##
==========================================
- Coverage   45.50%   40.95%   -4.55%
==========================================
  Files          79      102      +23
  Lines        7782    15106    +7324
==========================================
+ Hits         3541     6187    +2646
- Misses       4099     8482    +4383
- Partials      142      437     +295


Issue101 compute flow

Fixes #101  .
Note: this PR follows up from a previous PR, here
Changes proposed in this PR:

LGTM but I'll let @trentmc review this one, since he is closer to this functionality.

LGTM but I'll let @trentmc review this one, since he is closer to this functionality.

Shoulda marked it as draft, sorry. Mehmet and I have been iterating daily (except today). Next call is tomorrow, we will discuss next steps then.

We won't be using this PR. Closing.


Operator trips over provided super-user secret

When creating a superuser secret:
oc create secret generic quay-superuser-secret --from-literal=superuser-username=test --from-literal=superuser-password=test<EMAIL_ADDRESS>And specifying that in the Quay CR:
apiVersion: redhatcop.redhat.io/v1alpha1
kind: QuayEcosystem
metadata:
name: quay
spec:
quay:
imagePullSecretName: quay-pull-secret
superuserCredentialsName: quay-superuser-secret
configSecretName: quay-config-secret
database:
volumeSize: 50Gi
registryStorage:
local:
persistentVolumeAccessMode:
- ReadWriteOnce
persistentVolumeSize: 50Gi
The following error occurs:
{"level":"info","ts":1564163861.5151386,"logger":"controller_quayecosystem","msg":"Reconciling QuayEcosystem","Request.Namespace":"quay","Request.Name":"quay"}
{"level":"error","ts":1564163861.5152035,"logger":"controller_quayecosystem","msg":"Secret Validation","Namespace":"quay","Name":"quay-superuser-secret","error":"Failed to validate provided secret with required parameters","stacktrace":"github.com/go-logr/zapr.(*zapLogger).Error\n\t/home/travis/gopath/pkg/mod/github.com/go-logr/zapr@v0.1.0/zapr.go:128\ngithub.com/redhat-cop/quay-operator/pkg/controller/quayecosystem/validation.validateSecret\n\t/home/travis/gopath/src/github.com/redhat-cop/quay-operator/pkg/controller/quayecosystem/validation/validate.go:198\ngithub.com/redhat-cop/quay-operator/pkg/controller/quayecosystem/validation.Validate\n\t/home/travis/gopath/src/github.com/redhat-cop/quay-operator/pkg/controller/quayecosystem/validation/validate.go:25\ngithub.com/redhat-cop/quay-operator/pkg/controller/quayecosystem.(*ReconcileQuayEcosystem).Reconcile\n\t/home/travis/gopath/src/github.com/redhat-cop/quay-operator/pkg/controller/quayecosystem/quayecosystem_controller.go:127\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/home/travis/gopath/pkg/mod/sigs.k8s.io/controller-runtime@v0.1.10/pkg/internal/controller/controller.go:215\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func1\n\t/home/travis/gopath/pkg/mod/sigs.k8s.io/controller-runtime@v0.1.10/pkg/internal/controller/controller.go:158\nk8s.io/apimachinery/pkg/util/wait.JitterUntil.func1\n\t/home/travis/gopath/pkg/mod/k8s.io/apimachinery@v0.0.0-20181127025237-2b1284ed4c93/pkg/util/wait/wait.go:133\nk8s.io/apimachinery/pkg/util/wait.JitterUntil\n\t/home/travis/gopath/pkg/mod/k8s.io/apimachinery@v0.0.0-20181127025237-2b1284ed4c93/pkg/util/wait/wait.go:134\nk8s.io/apimachinery/pkg/util/wait.Until\n\t/home/travis/gopath/pkg/mod/k8s.io/apimachinery@v0.0.0-20181127025237-2b1284ed4c93/pkg/util/wait/wait.go:88"}
Also in the CR:
status:
conditions:
- lastTransitionTime: "2019-07-26T17:56:49Z"
lastUpdateTime: "2019-07-26T17:56:49Z"
message: Configuration Updated Successfully
status: "True"
type: UpdateDefaultConfigurationSuccess
- lastTransitionTime: "2019-07-26T17:56:49Z"
lastUpdateTime: "2019-07-26T17:57:41Z"
message: 'Failed to validate provided secret with required parameters. Namespace:
quay, Name: quay-superuser-secret'
reason: ProcessingError
status: "False"
type: QuayEcosystemValidationFailure

Same with Config Secret

I ran into this the other day when I was testing, but never got around to opening an issue. For the SuperUser, you need to use:
superuserCredentialsSecretName
This might just be a docs bug. I didn't get a chance to run into the problem with config secret. That looks like it should be okay, but I don't know if the validation maybe failed on your config secret content vs the name?

@stencell @wkulhanek Wanted to add a few comments related to this issue. PR #29 was submitted and merged to help address them. Some of the highlights:

Quay requires a superuser password of at least 8 characters. Added new validation that checks this after resource creation
Setup error handling was not properly implemented. Even though an error occurred during the setup process, the provisioning process continued to deploy Quay. Since setup failed, due to internal validation, the secret that is generated after the setup process completes successfully was not generated.
The superuserCredentialsSecretName was not mapped properly on the QuayEcosystem object. This has been corrected
A complete rewrite of how registry storage is managed was previously completed in order to eventually support fully defining replication scenarios. README was out of date #30 submitted to address this

I would like to push our a new tag for the repository that will provide a stable baseline prior to integrating clair. Looking to do it this week after additional community testing

Fixed.


ClusterLoader - refine wait_for_controlled_pods.go clients data structure

Why we need it:
https://github.com/kubernetes/perf-tests/issues/414
What this PR does:
Related to https://github.com/kubernetes/perf-tests/pull/423

/assigin @wojtek-t

/lgtm
/approve
/ok-to-test


key switches

@todo

[ ] decide for key switches and possible vendors

see: https://www.youtube.com/watch?v=dqai0_nnTgs

linear: tangerine, roselios, taelios
tactile: cherry mx brown, zealios, holy panda
clicky:  kailh box white, box jade, sherbet

info
So standard and probably original choc switches as mentioned have the size:

15mm x 15mm x 11mm

The choc mini switch size is:

14.5mm(W) x 13.5mm(D) x 8.2mm(H)
https://www.anandtech.com/show/11851/kailh-introduces-low-profile-mini-chocolate-keyswitches
https://imgur.com/a/yrECe
https://imgur.com/r/mechanicalkeyboards/HZD6d
https://www.thingiverse.com/thing:3744277

seller

http://www.kailh.com/en/Products/Ks/NotebookS/
http://www.kailh.com/en/Products/Ks/CS/

Datasheet for the MX switches: https://github.com/Gondolindrim/AcheronLibrary/blob/master/datasheets/MX_switch_datasheet.pdf


[Question] How to ignore query params when caching?

Hi, I'm trying how to find a way to omit query string when caching html pages in next.js app.
The URLs look like this:
http://localhost:3000/content?timestamp=1
http://localhost:3000/content?timestamp=2
http://localhost:3000/content?timestamp=3

and I would like to tell serwist, that these urls are equal and the content is the same. So to cache it as http://localhost:3000/content by removing the query string.
So something like this code from: https://web.dev/articles/cache-api-quick-guide#retrieving
const options = {
  ignoreSearch: true,
};

const response = await cache.match(request, options);

Any ideas how to do this? Thanks

I came up with this which seems to work, is there some easier way how to do this?
self.addEventListener('fetch', (event) => {
  const url = new URL(event.request.url);
  if (url.pathname === '/content') {
    // Clone the request and remove the query string
    const newUrl = url.origin + url.pathname;
    const newRequest = new Request(newUrl, {
      ...event.request,
    });

    event.respondWith(
      new NetworkFirst({
        cacheName: 'page-live',
        plugins: [
          new ExpirationPlugin({
            maxEntries: 1,
            maxAgeSeconds: 60 * 60,
          }),
        ],
      }).handle({
        event,
        request: newRequest,
        url: new URL(newUrl),
        params: {},
      }),
    );
  }
});

@MartinPavlik if you are looking for ignoreSearch, its an attribute inside of the matchOptions, example:

      new NetworkFirst({
        matchOptions: {
          ignoreSearch: true,
        }
...

Thank you! I'm closing this.

Why ignoreSearch does not work in CacheFirst strategy ?

@llz383455526 what do you mean? It should work just fine.


vscode-extensions.pythagoratechnologies.gpt-pilot-vs-code: init 0.1.7

vscode-extensions.pythagoratechnologies.gpt-pilot-vs-code: init 0.1.7

https://github.com/Pythagora-io/gpt-pilot/

https://marketplace.visualstudio.com/items?itemName=PythagoraTechnologies.gpt-pilot-vs-code&

For those wondering: Works with local LLMs too. (See demo)
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks @drupol


Sorting left pane with header

Is it possible to use sorting (like in a standard table) to sort / filter rows in the left pane of gantt?

Is it possible to use sorting (like in a standard table) to sort / filter rows in the left pane of gantt?

We currently have no plans to support this feature in ngx-gantt，but you can achieve this requirement through custom template
<ngx-gantt>
  <ngx-gantt-table>
    <ngx-gantt-column>
      <ng-template #header>

      <ng-template/>
      <ng-template #cell let-row>
        ...
      <ng-template/>
    </ngx-gantt-column>
  </ngx-gantt-table>
</ngx-gantt>


Crash on Android 5.0

Hi, I like the idea of this App. However is crashes on my (rather old) Android Phone SAMSUNG Galaxy Alpha at start.
I use the f-droid version 1.6. It works on more recent phones (Android 9).

Sorry, I built it using Flutter, and I can't figure out what versions of Android it should work on.  I tested on my phone, which has Android 9.


Paragraph_layout takes up lots of CPU time for complex layout

Steps to Reproduce
Shared code with @Hixie
Logs
debug.txt
Flutter Doctor
dynin-macpro:parlayout dynin$ flutter doctor
[✓] Flutter (on Mac OS, channel master)
    • Flutter at /Users/dynin/flutter
    • Framework revision e742b901b8 (22 hours ago), 2017-02-07 17:15:34
    • Engine revision cbf49d0652
    • Tools Dart version 1.22.0-dev.10.6

[✓] Android toolchain - develop for Android devices (Android SDK 23.0.3)
    • Android SDK at /Users/dynin/Library/Android/sdk
    • Platform android-N, build-tools 23.0.3
    • Java(TM) SE Runtime Environment (build 1.8.0_112-b16)

[-] iOS toolchain - develop for iOS devices (Xcode 8.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.2.1, Build version 8C1002
    • ios-deploy 1.8.6
    x ios-deploy version >= 1.9.0 not available; this is used to deploy to connected iOS devices.
      Install via 'brew install ios-deploy'.

[-] IntelliJ IDEA Community Edition (version 2016.3.1)
    • Dart plugin not installed; this adds Dart specific functionality.
    • Flutter plugin not installed; this adds Flutter specific functionality.
    • For information about managing plugins, see
      https://www.jetbrains.com/help/idea/2016.2/managing-plugins.html

[✓] Connected devices
    • iPhone SE • B817AD64-1015-45FD-A9D6-6CE407F6DA21 • ios • iOS 10.2 (simulator)

Thanks!

I haven't had time to look at this yet, sorry. Still on my queue.

I got around to looking at this just now but I couldn't find a hang. I did notice that the initial build and layout of the Explore tab skipped a few frames, but given how complex that layout is that seems unsurprising.

I tried again in release mode and the performance seems fine.
Do you have steps to reproduce?

We determined that what's happening here is as follows:

In debug mode, complex layouts are O(N²) in the depth of the tree are rely heavily on uncached computation of text layout (each text node is laid out a half dozen times for each node in the tree, basically).
On MacOS, text layout is especially slow.

MacOS isn't a supported configuration, and debug is intentionally making speed tradeoffs to catch bugs, so this is WONTFIX unless it can be reproduced on a device or simulator.


Termbox API

I got curious if I can run termbox https://github.com/nsf/termbox/ test example on tdl and it appeared that it is possible. Not sure if it will qualifies as a full fledged example, but submitting in case it will be found interesting.

Well I think it's interesting.  Hopefully I can add features that will finish the API for this.  An event.wait function has been on my todo list for a while.

@HexDecimal yes, there was an interesting pattern in termbox of waiting with timeout for an event to appear.


Activating borderless then deactivating fullscreen produces a bordered window

Activating borderless then deactivating fullscreen produces a bordered window (toggling borderless again fixes that).

I can not reproduce in 0.54.

Can't reproduce on 0.55.2 either, the steps above produce a borderless window as expected.


Support copy and deepcopy?

juliacall.AnyValue.__copy__ and __deepcopy__ could call Julia's copy and deepcopy.
Conversely, copy(::Py) and deepcopy_internal(::Py, ::IdDict) could call Python's copy.copy and copy.deepcopy.
https://docs.python.org/3/library/copy.html

We can support memos by including the Julia memo dict at a special key in the Python memo dict, and vice versa.

Note that you'll probably want to define deepcopy_internal to at minimum increment the reference count, since otherwise calling deepcopy(::Py) in Julia will use the default implementation (that will just copy the pointer).

See https://github.com/JuliaPy/PyCall.jl/pull/1039 for how PyCall.jl does it.


Style change 2.0

This is a corrected version of https://github.com/k2-fsa/icefall/pull/690.

(107df3b) First pass: We apply black formatting without the --experimental-string-processing feature which was causing some files to get deleted entirely.
(d31db01) Second pass: I manually went through all the modifications and corrected several of the sort:

            f"expected sample rate: {expected_sample_rate}. "
            f"Given: {sample_rate}"

being converted to:
           f"expected sample rate: {expected_sample_rate}. " f"Given: {sample_rate}"

and a small number of cases where inline comments made the formatting weird.
At the very least, these changes should not cause anything to break. If there is something I missed, I am hoping they can be corrected as people come across them in the future.

@csukuangfj I would request you to please briefly check the files in the main icefall/ directory since they are used in all recipes, to ensure there are no breaking changes. (You can skip the changes in egs)

Thanks! Looks good to me. Left a minor comment.

Thanks! Looks good to me. Left a minor comment.

I think it's ready for merge?


Make subpackage checks opt-in by default

PROBLEM:

Each subdirectory is a separate CircleCI job, so someone adding a new subdirectory needs to remember to add a new CircleCI job to test it
The check that tries to skip the directory if it doesn't have changes will silently skip & pass if the user copy-pastes the name  (this was the case with redis-voting-app)
The check_latest_version requires someone to remember to add the new subdirectory to it (also the case with redis-voting-app)

SOLUTIONIZING:
Have a single check job with a for loop that iterates over all subdirectories by default, skipping the directories that don't have changes (coded in the for loop, so impossible to get a mismatch with dir name)
Have check_latest_version do the same with a for loop, so users can't forget to add it.

Note as a part of this we should also run kurtosis clean -a between jobs so that there ins't resource contention
Also note we have temporarily updated resource_class to xlarge, we should remove that after this for loop is in place.

Perhaps also use YAML anchors to cleanup CI as a part of this

@victorcolombo probably add more parameters; to test other permutations (maybe could be a separate ticket)


Should set priority class for istio components to prevent preempted

As Pod Priority and Preemption is GA in kubernetes 1.14.  And the Preemption is enabled by default in kubernetes 1.11.  So we should set the highest priority for each istio components.
priorityClassName: system-cluster-critical

Thanks, agreed @hzxuzhonghu!
Morven, can you take this one too?

I would suggest instead of giving the highest possible priority, we should leave room for users to decide. What about adding one or two magnitudes to the globalDefault priority?

Actually we have provide the users the option of priorityClassName , see: https://github.com/istio/istio/blob/master/install/kubernetes/helm/istio/values.yaml#L374_#L379
Currently default value of priorityClassName is empty, should we update to system-cluster-critical?

+1 @morvencao that works as the default.  And if user wants to tweak it, he/she can.

Can we close this? As we have empty value for priorityClassName and the user is free to update to system-cluster-critical if needed.

Thanks for your work @morvencao

I don't see why this should have been closed

I feel we need a better default than ""
How do we feel about this now that we are more than a year past this discussion?

@tariq1890  Any comments on this? What default value it should be?
End user is able to set the priorityClassName other than default "" by the k8s settings.

Yes. We are setting it in our helm charts, but I am asking for better defaults.
If more customizations of ours are sent upstream, our helm chart management becomes easier.
If you agree on the default to be system-cluster-critical, I can send a PR for that

Feel free to raise PR to update it and add @linsun @hzxuzhonghu as reviewers.


Raspberry Pi auto start

Installed on Raspberry Pi 3, all worked after loading Python3 utilities.
Runs fine executing python3 mqtt.py and /usr/bin/python3 /home/pi/broadlink-mqtt/mqtt.py (command in service file)
Contents of /lib/systemd/system/broadlink-mqtt.service
[Unit]
Description=Broadlink MQTT Service
After=multi-user.target
<EMAIL_ADDRESS>
[Service]
Type=simple
ExecStart=/usr/bin/python3 /home/pi/broadlink-mqtt/mqtt.py
StandardInput=tty-force

[Install]
WantedBy=multi-user.target

sudo systemctl start broadlink-mqtt.service gives no error
sudo systemctl status broadlink-mqtt.service gives the following:
● broadlink-mqtt.service - Broadlink MQTT Service
   Loaded: loaded (/lib/systemd/system/broadlink-mqtt.service; enabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Wed 2020-10-21 16:08:09 BST; 7min ago
  Process: 8674 ExecStart=/usr/bin/python3 /home/pi/broadlink-mqtt/mqtt.py (code=exited, status=1/FAILURE)
 Main PID: 8674 (code=exited, status=1/FAILURE)

Oct 21 16:08:09 bg1mqtt systemd[1]: Started Broadlink MQTT Service.
Oct 21 16:08:09 bg1mqtt systemd[1]: broadlink-mqtt.service: Main process exited, code=exited, status=1/FAILURE
Oct 21 16:08:09 bg1mqtt systemd[1]: broadlink-mqtt.service: Failed with result 'exit-code'.

Good work, integrated with HomeAssistant great.

@VincentSC maybe you could help?


Upadting Readme.

Correcting the Readme.

By an odd coincidence I fixed that code sample yesterday, but I did miss an option for multi-player environments, thanks!