Dynamically update JWKS URLs (Multi-tenancy support with Keycloak)

Summary
The list of JWKS URLs cannot be updated once initialized.
When working in a multi-tenant system, and upon boarding of a new tenant to the system, there is no option of validating the JWT token signature of the new tenant, because a new JWKS URL is introduced.

Actual Behavior
I'm working with Keycloak as the IDP. In Keycloak, each tenant (AKA 'realm') has its own URL and KID for validating the JWT token signature.
The URL contains the public key that is used to validate the token signature.
For example, the URLs for two different tenants can look like:
Tenant1 - https://idpHost/auth/realms/Tenant1/protocol/openid-connect/certs
Tenant2 - https://idpHost/auth/realms/Tenant2/protocol/openid-connect/certs
Therefore, I use the option of initializing the token store with a list of URLs for all current active tenants:
   @Bean("resourceServerTokenStore")
    public JwkTokenStore tokenStore(JwtAccessTokenConverter accessTokenConverter) throws Exception {
        return new JwkTokenStore( List.of(jwkSetUri.split(",")), accessTokenConverter, null);
    }

That works great. However, when a new tenant is created in the system, I have no way to update this list of URLs, and I must restart my service for that to take affect.
As you can see in class JwkDefinitionSource, the list is final and there is no way to update it:
private final List<URL> jwkSetUrls;

Expected Behavior
I'd like a way to update the list of URLs dynamically, so if a new token with a new KID (for new tenant) is sent to my server, it will lookup for that KID in an updated list of URLs.
This can be achieved adding a set method to JwkTokenStore class:
For example,  'addJwkUrl (String URL)' will add a single URL to the saved list.
And also 'updateJwkSetUrls (List jwkSetUrls)' will override any URL that is already set with the new list.
After having this, I will be able to update the list each time I get an event of new tenant in my system.

Version

2.3.5.RELEASE

Any update on this? thanks

Any update on this? thanks

Hi @cfpperche , please note the deprecation notice on this github repository (check readme.md).
No point on investing on that here.
My solution was to migrate my service to use the new spring security :
https://docs.spring.io/spring-security/site/docs/current/reference/html5/
The new one allows you to use multiple JWK URLs and dynamically update the list when new tenant token is retrieved.
It is based on NimbusJwtDecoder and allows you to configure the jwtDecoder and also have multiple JWK URLs by using setJWTClaimsSetAwareJWSKeySelector.
Check the multi-tenancy example and usage of TenantJWSKeySelector:
https://docs.spring.io/spring-security/site/docs/current/reference/html5/#oauth2resourceserver-multitenancy
Here is s code snippet from what I did:
@Override
public JwtDecoder getJwtDecoder() {
ConfigurableJWTProcessor jwtProcessor = new DefaultJWTProcessor<>();
jwtProcessor.setJWTClaimsSetAwareJWSKeySelector(new KeycloakTenantJWSKeySelector(authenticationServerLocation));
NimbusJwtDecoder jwtDecoder = new NimbusJwtDecoder(jwtProcessor);
jwtDecoder.setClaimSetConverter(new KeycloakJwtClaimAdapter());
SetJwtDecoderValidators(jwtDecoder, audience);
return jwtDecoder;
}


Implement conversions for Line, GeometryCollection, Rect, Triangle

Currently, this causes a run-time panic!(), which is not great. Line, Triangle and Rect should be easy to implement, but what about GeometryCollection?

Line maps to GeoJSON LineString
Triangle and Rect map to GeoJSON Polygon
GeometryCollection maps to GeoJSON GeometryCollection

Relevant code that needs to change:
https://github.com/georust/geojson/blob/f394a5b3424c2b44e5e762d5b98787854b1edd73/src/conversion/from_geo_types.rs#L101-L121
We should stop using _ as a matching pattern here and make it exhaustive.


[EUWE] tree_autoload - don't assume we're always autoloading the active tree

This is an euwe version of https://github.com/ManageIQ/manageiq/pull/12986 - heed params[:tree] for tree_autoload, instead of relying on x_active_tree
https://bugzilla.redhat.com/show_bug.cgi?id=1398239

:clap:


How to set hourly view only for 24h

Hi I try to create gantt chart where in columns I have all day listed for month and time line should be only 24 hours
so for GanttViewOptions I set
start :today (00:00:00)
end: today (23:59:59)
but it always show me entire week not just one day

I'm having the same question, did you resolve this?


How to sink JSON objects in a JSON array as individual records in a collection

I would like to sink elements of a given JSON Array into Mongo as individual records. For example given the following array:
[ { "name" : "john",  "age" : 20 }, { "name" : "jane", "age" : 22 } ]
each object should be one record in the given collection. I am getting the following error. One option is to read from one topic and write into another one breaking array into individual objects, but I wonder if it is possible to do it out-of-the box with kafka-connect and or with this connector.
kafka-connect_1    | org.apache.kafka.connect.errors.DataException: error: no converter present due to unexpected object type java.util.ArrayList kafka-connect_1    | 	at at.grahsl.kafka.connect.mongodb.converter.SinkConverter.getRecordConverter(SinkConverter.java:78) kafka-connect_1    | 	at at.grahsl.kafka.connect.mongodb.converter.SinkConverter.convert(SinkConverter.java:49) kafka-connect_1    | 	at at.grahsl.kafka.connect.mongodb.MongoDbSinkTask.lambda$buildWriteModel$3(MongoDbSinkTask.java:213) kafka-connect_1    | 	at java.util.ArrayList.forEach(ArrayList.java:1257) kafka-connect_1    | 	at at.grahsl.kafka.connect.mongodb.MongoDbSinkTask.buildWriteModel(MongoDbSinkTask.java:212) kafka-connect_1    | 	at at.grahsl.kafka.connect.mongodb.MongoDbSinkTask.processSinkRecords(MongoDbSinkTask.java:143) kafka-connect_1    | 	at at.grahsl.kafka.connect.mongodb.MongoDbSinkTask.lambda$null$0(MongoDbSinkTask.java:118) kafka-connect_1    | 	at java.util.ArrayList.forEach(ArrayList.java:1257) kafka-connect_1    | 	at at.grahsl.kafka.connect.mongodb.MongoDbSinkTask.lambda$put$1(MongoDbSinkTask.java:117) kafka-connect_1    | 	at java.util.HashMap.forEach(HashMap.java:1289) kafka-connect_1    | 	at at.grahsl.kafka.connect.mongodb.MongoDbSinkTask.put(MongoDbSinkTask.java:112) kafka-connect_1    | 	at org.apache.kafka.connect.runtime.WorkerSinkTask.deliverMessages(WorkerSinkTask.java:565) kafka-connect_1    | 	at org.apache.kafka.connect.runtime.WorkerSinkTask.poll(WorkerSinkTask.java:323) kafka-connect_1    | 	at org.apache.kafka.connect.runtime.WorkerSinkTask.iteration(WorkerSinkTask.java:226) kafka-connect_1    | 	at org.apache.kafka.connect.runtime.WorkerSinkTask.execute(WorkerSinkTask.java:194) kafka-connect_1    | 	at org.apache.kafka.connect.runtime.WorkerTask.doRun(WorkerTask.java:175) kafka-connect_1    | 	at org.apache.kafka.connect.runtime.WorkerTask.run(WorkerTask.java:219) kafka-connect_1    | 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) kafka-connect_1    | 	at java.util.concurrent.FutureTask.run(FutureTask.java:266) kafka-connect_1    | 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) kafka-connect_1    | 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) kafka-connect_1    | 	at java.lang.Thread.run(Thread.java:748) kafk

hi @zbrsy great to hear you are trying the sink connector. two important aspects w.r.t. the issue you posted:

for several reasons, the sink connector has been designed to work with JSON objects for keys and values. this means the value of a kafka records is expected to be represented as a JSON object instead of a JSON array. what's possible is of course to work with a value that only uses a single field holding the array in question e.g.

{ "myArray":
  [
   { "name" : "john", "age" : 20 },
   { "name" : "jane", "age" : 22 }
  ]
}

so all you need to do is change this value structure of yours to reflect that, then it should work.

Storing individual documents for the array entries is not possible and will also not get implemented. The idea of kafka connect is basically to provide a 1:1 mapping between records (sources -> kafka or kafka -> sinks). There are reasons why it's not the best idea to try to perform this "fan out" i.e. split a single kafka record into multiple ones towards the sink. Also the single message transform (SMT) mechanism of kafka-connect itself operates on a single message only for that matter. The solution to this is to employ a stream processor before (KStreams and/or KSQL are your friends here) which flatMaps the array into separate kafka records, writes them into a separate topic which can the be used with the sink connector.

let me know if you need any further info/help.

@hpgrahsl Thanks for the reply. It all make sense. Second option is the path I was planning take but first I wanted to check if I missed anything. Again thanks for clarifications and the reply.

Glad to hear that my reply was helpful. Feel free to reach out anytime you need something. Also it might be wise to checkout the official MongoDB connector which became available several weeks ago. Find the repo here: https://github.com/mongodb/mongo-kafka also the migration is pretty smooth since the sink connector is fully based on this code-base :)

I have the same issue as zbrsy.
Are you sure Kafka will not implement JSON arrays in mongo connector sink?
How can I implement stream processor as you mentioned.
Thanks a lot.


Does v1.2.0 still have full PHP 5.3+ support?

I noticed that some of the recent updates are removing support for PHP v5 and will only support v7 going forward, but I wanted to confirm that v1.2.0 still has full support for PHP v5.3+.
I have PHP Cron Expression Parser included in an open source project of mine, and I want to upgrade from v1.1.0 to v1.2.0, but I can't do so if it's going to lose support for PHP v5.

Yes, 1.2.x will continue to have PHP 5.x support. At this point I do not think there will be many more updates to that version though, as most of the fixes being worked on will break backward compatibility with the expressions that used to be allowed.
It should be safe to upgrade from 1.1.0 to 1.2.0 though.


[hotfix] [pytket-qiskit] [pytket-ionq] fix #128

This is for a hotfix solving bug #128
All remote tests for pytket-qiskit and pytket-ionq are passing.

Can you please update the changelog file for each extension, mentioning the bug fix in each?


ProtectionChanges doesn't encapsulate all possible values

The ProtectionChanges object used by BranchProtectionRuleEvent does not include all the possible return values. Currently it looks like:
type ProtectionChanges struct {
	AuthorizedActorsOnly *AuthorizedActorsOnly `json:"authorized_actors_only,omitempty"`
	AuthorizedActorNames *AuthorizedActorNames `json:"authorized_actor_names,omitempty"`
}

which is all that is ever returned in GitHub's documentation, but that isn't all that is possible. It should look something more like:
type ProtectionChanges struct {
	PullRequestReviewsEnforcementLevel       *PullRequestReviewsEnforcementLevel       `json:"pull_request_reviews_enforcement_level,omitempty"`
	DismissStaleReviewsOnPush                *DismissStaleReviewsOnPush                `json:"dismiss_stale_reviews_on_push,omitempty"`
	RequireCodeOwnerReview                   *RequireCodeOwnerReview                   `json:"require_code_owner_review,omitempty"`
	AuthorizedDismissalActorsOnly            *AuthorizedDismissalActorsOnly            `json:"authorized_dismissal_actors_only,omitempty"`
	RequiredStatusChecks                     *RequiredStatusChecks                     `json:"required_status_checks,omitempty"`
	RequiredStatusChecksEnforcementLevel     *RequiredStatusChecksEnforcementLevel     `json:"required_status_checks_enforcement_level,omitempty"`
	SignatureRequirementEnforcementLevel     *SignatureRequirementEnforcementLevel     `json:"signature_requirement_enforcement_level,omitempty"`
	LinearHistoryRequirementEnforcementLevel *LinearHistoryRequirementEnforcementLevel `json:"linear_history_requirement_enforcement_level,omitempty"`
	AdminEnforced                            *AdminEnforced                            `json:"admin_enforced,omitempty"`
	CreateProtected                          *CreateProtected                          `json:"create_protected,omitempty"`
	AllowDeletionsEnforcementLevel           *AllowDeletionsEnforcementLevel           `json:"allow_deletions_enforcement_level,omitempty"`
	RequiredDeploymentsEnforcementLevel      *RequiredDeploymentsEnforcementLevel      `json:"required_deployments_enforcement_level,omitempty"`
	RequiredConversationResolutionLevel      *RequiredConversationResolutionLevel      `json:"required_conversation_resolution_level,omitempty"`
	AuthorizedActorsOnly                     *AuthorizedActorsOnly                     `json:"authorized_actors_only,omitempty"`
	AuthorizedActorNames                     *AuthorizedActorNames                     `json:"authorized_actor_names,omitempty"`
}

with one or two more fields related to require deployment successes.

Thank you, @eolso !
Would you like to put together a PR for this issue?

Sure thing!

Sure thing!

Thank you, @eolso ! It is yours. 😁


Add initial support to Meadow F7Micro board

Summary
Add support to Wilderness Labs F7Micro board.
Impact
Users will be able to use this board in the NuttX mainline.
Testing

@duduita please squash your change into one patch by git rebase --interactive HEAD~6 and change all pick to squash except the first one.


Upgrade to v4.x of the AWS Terraform provider

This is a leftover from https://github.com/wellcomecollection/platform/issues/5664
Doing a plan/apply in the critical stack requires you to:

have access to the Azure containers
have permission to make policy changes to the permanent storage S3 buckets (you don't strictly need it, but Terraform will try to re-put the bucket ACL)

Devs don't normally have access to either of those, because it prevents us from inadvertent changes – but at some point we should do those upgrades.

Likewise, the API Gateway wants to redeploy when I fiddle with things and I don't want to deal with that right now.


Incorrect minimal cmake version requirement.

Are you using SoftEther VPN 5.x?

[X] Yes, I'm using SoftEther VPN 5.x, not 4.x.

Version
5.02.5185 / c2a7aa548137dc80c6aafdc645cf4dc34e0dc764
Component
VPN Server
Operating system & version
Ubuntu 22.04
Architecture or Hardware model
amd64
Steps to reproduce
git checkout 5.02.5185
./configure

✔ Expected Behavior
Successfully finished configuration.
❌ Actual Behavior
$ ./configure
---------------------------------------------------------------------
SoftEther VPN for Unix

Copyright (c) all contributors on SoftEther VPN project in GitHub.
Copyright (c) Daiyuu Nobori, SoftEther Project at University of Tsukuba, and SoftEther Corporation.

Licensed under the Apache License, Version 2.0 (the License).

Read and understand README, LICENSE and WARNING before use.
---------------------------------------------------------------------

Welcome to the corner-cutting configure script !

'rpm' executable found, CPack will generate RPM packages.

CMake Error at CMakeLists.txt:20 (cmake_policy):
  Policy "CMP0091" is not known to this version of CMake.

-- The C compiler identification is GNU 7.5.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Looking for sys/auxv.h
-- Looking for sys/auxv.h - found
-- Build date: 12/08/2024
-- Build time: 12:38:01
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.28")
-- Checking for one of the modules 'libsodium'
-- Found Curses: /usr/lib/x86_64-linux-gnu/libcurses.so
-- Found OpenSSL: /usr/lib/x86_64-linux-gnu/libcrypto.so (found version "1.1.1")
-- Looking for EVP_PKEY_get_raw_public_key
-- Looking for EVP_PKEY_get_raw_public_key - found
-- Looking for SSL_CTX_set_num_tickets
-- Looking for SSL_CTX_set_num_tickets - found
CMake Error at src/Mayaqua/CMakeLists.txt:38 (add_compile_definitions):
  Unknown CMake command "add_compile_definitions".

-- Configuring incomplete, errors occurred!
See also "/home/eugene/SoftEtherVPN/build/CMakeFiles/CMakeOutput.log".

Anything else?
add_compile_definitions has been introduced in cmake 3.12 however the project requires  3.10.

do you wish to submit a PR ?

@chipitsine I can, yes, why not. I'd like to know what is the desired solution, to workaround keeping cmake 3.10 or simply upgrade the minimal version to 3.12?

Feel free to choose a solution ))

we are not bound to 3.10


Maximized & Blacked-out GRC window? Temporary glitch!

Hey,
I experienced a strange glitch. My laptop had lost internet connectivity, I switched to my Gridcoin client and it looked like: https://usercontent.irccloud-cdn.com/file/gLNQnoWA/image.png
When I regained internet connectivity, the window had resized to the normal size and returned to normal.
Whilst it was in this glitched state, I was unable to move the window nor resize it.
Anyone else experienced this? If you're experiencing it, just give it time.
Regards,
CM.

I had something similar, multiple times, but it was not black. It was maximized and non responsive.
My internet connection was up, but I had quite much CPU load when it happened.

I saw this once as well but I don't know if it's our fault or something in Qt. I also had high CPU load when this happened.

Closing this issue as unable to duplicate. If we can reproduce or get another report, we can reopen it.


/user/username/liked

https://www.reddit.com/dev/api#GET_user_{username}_liked

80ccf65fec3d4a1b4b465d33999cd3f23d3e036a


Setup Logging Service

This service should:

consume all the logs from all pods
keep the logs for at least 3 days
provide a UI to view the logs

We'll add the manifests for deploying the log service to our repo, and either deploy this service as part of make launch or provide a different make target.
For now, I'm running w fluentd / elasticsearch / kibana a la http://kubernetes.io/docs/getting-started-guides/logging-elasticsearch/

Scheduling this for 1.3. And assigning to @sjezewski since he's working on it.

I'm still figuring out how these components fit together, but the basics are:

We spin up a fluentd pod per node (VM) that is provisioned. These gather logs from each node and send the logs to the elasticsearch-logging pods
The elasticsearch-logging pods, which store the logs and expose them via a REST API
The kibana pod provides a web UI for viewing the logs

The above is probably a valid path if we want to vendor in some logging tools w a given pachyderm cluster deployment. But! GCE and AWS seem to have builtin cluster wide logging already.
In particular, GCE provides monitoring (of CPU/MEM) via stackdriver, and logs (w search and past a few days) via their dashboard.


Control over whether a space is added between supplement and numbering

In non-latin languages like Chinese, we do not always split two words by a space, so when I reference Figure 1, I may want to get 图1 instead of 图 1.
Besides I know, there are some papers using Fig.1 to reference a figure (I found a Japanese paper case. https://www.jstage.jst.go.jp/article/aijjse/69B/0/69B_1/_pdf/-char/ja), now we cannot combine supplement and numbering without space.
My solution
Add a spaced field for ref function and default to true.
Then we can set the ref like following, after which we get Figure1 instead of Figure 1 when we write @fig1.
#set ref(spaced: false)
@fig1
#figure(
	image("x.png")
) <fig1>

Besides, add a spaced filed for figure function and it will work when showing the caption of the figure.
How about it? I am working on this and if you agree with me, I will open a PR then.

This can be very useful, especially when using Chinese numberings (e. g. 图一).


use nightly mono channel

The nightly build is already running quite far ;) Good idea!

looking at a previous failure, we may need to clear caches more aggressively:
https://s3.amazonaws.com/archive.travis-ci.org/jobs/260712135/log.txt?X-Amz-Expires=30&X-Amz-Date=20170803T214906Z&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAJRYRXRSVGNKPKO5A/20170803/us-east-1/s3/aws4_request&X-Amz-SignedHeaders=host&X-Amz-Signature=ff0a92e021f630ead41ad360e77ae461df5add1285272f90b039693597f97b9f

[31m3) Error : Paket.IntegrationTests.RestoreSpec.#2496 Paket fails on projects that target multiple frameworks
System.IO.IOException : Disk full. Path /home/travis/build/fsprojects/Paket/.paket/paket.exe" or "/home/travis/build/fsprojects/Paket/integrationtests/scenarios/i002496-empty-target-framework/temp/../.paket/paket.exe
  at System.IO.File.InternalCopy (System.String sourceFileName, System.String destFileName, System.Boolean overwrite, System.Boolean checkHost) [0x00036] in :0
at System.IO.FileInfo.CopyTo (System.String destFileName, System.Boolean overwrite) [0x00035] in :0

https://travis-ci.org/fsprojects/Paket/builds/260712134

yup, disk full again: https://travis-ci.org/fsprojects/Paket/builds/260788299#L6956

different build, different error:
https://travis-ci.org/fsprojects/Paket/jobs/260792320#L7115
-> Could not find versions for package FSharp.Formatting on http://www.nuget.org/api/v2.
-> Source 'http://www.nuget.org/api/v2' yielded no results
	-  Request 'https://api.nuget.org/v3-flatcontainer/fsharp.formatting/index.json?semVerLevel=2.0.0' finished with error
	-> HttpRequestException: An error occurred while sending the request
	-> WebException: Error: TrustFailure (One or more errors occurred.)
	-> AuthenticationException: A call to SSPI failed, see inner exception.
	-> MonoBtlsException: Ssl error:1000007d:SSL routines:OPENSSL_internal:CERTIFICATE_VERIFY_FAILED
	     at /build/mono-<IP_ADDRESS>5/external/boringssl/ssl/handshake_client.c:1132
	-  Request 'http://www.nuget.org/api/v2/FindPackagesById()?semVerLevel=2.0.0&id='FSharp.Formatting'' finished with error
	-> Could not retrieve data from 'http://www.nuget.org/api/v2/FindPackagesById()?semVerLevel=2.0.0&id='FSharp.Formatting''
	-> HttpRequestException: An error occurred while sending the request
	-> WebException: Error: TrustFailure (One or more errors occurred.)
	-> AuthenticationException: A call to SSPI failed, see inner exception.
	-> MonoBtlsException: Ssl error:1000007d:SSL routines:OPENSSL_internal:CERTIFICATE_VERIFY_FAILED
	     at /build/mono-<IP_ADDRESS>5/external/boringssl/ssl/handshake_client.c:1132
	-  Request 'http://www.nuget.org/api/v2/Packages?semVerLevel=2.0.0&$filter=Id eq 'FSharp.Formatting'' finished with error
	-> Could not retrieve data from 'http://www.nuget.org/api/v2/Packages?semVerLevel=2.0.0&$filter=tolower(Id) eq 'fsharp.formatting''
	-> HttpRequestException: An error occurred while sending the request
	-> WebException: Error: TrustFailure (One or more errors occurred.)
	-> AuthenticationException: A call to SSPI failed, see inner exception.
	-> MonoBtlsException: Ssl error:1000007d:SSL routines:OPENSSL_internal:CERTIFICATE_VERIFY_FAILED
	     at /build/mono-<IP_ADDRESS>5/external/boringssl/ssl/handshake_client.c:1132

something with certificates ...

Sent from my iPhone

On 7. Aug 2017, at 23:44, Lukas Rieger<EMAIL_ADDRESS>wrote:
@matthid
???
¯_(ツ)_/¯
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I found it myself. I had tried to fix the "disk full" error on travis by deleting %temp%, but apparently, some important script was also living in temp. Reverting that commit fixed that issue. =)
av is green, travis is 2/3 green:

with one failed test:

https://travis-ci.org/fsprojects/Paket/builds/262466593

first failure:

second failure:

next try:
https://travis-ci.org/fsprojects/Paket/builds/262477362?utm_source=github_status&utm_medium=notification

https://travis-ci.org/fsprojects/Paket/jobs/262477365#L5593
let's try again

https://travis-ci.org/fsprojects/Paket/builds/262477362?utm_source=github_status&utm_medium=notification

https://travis-ci.org/fsprojects/Paket/builds/262585235

welp, I rebased it on master, now nothing works ™

https://travis-ci.org/fsprojects/Paket/builds/262652598?utm_source=github_status&utm_medium=notification

once more

I don't think travis will ever be green ...

https://travis-ci.org/fsprojects/Paket/builds/262684028?utm_source=github_status&utm_medium=notification

@matthid I think this is ready from my side.
Travis is still flakey, but each sub-part was green at least once ;-)


Create app wireframe

Changed the user profile to include the "rank XP/badge tracking system".
Removed Forum FAQ from the right drawer to a new screen, as it would become clumsy in smaller screens.
Changed the reply indicator to be a continuous line.
A mockup of the "private post" system. Since it is yet to be implemented in the website, it essentially shows how it could be implemented in the app.
Also added the option to create the same in "Start a Discussion" page.

Creating a new post is still not the way as we discussed.
Rest looks good.
Please change that one thing and we will be good to go.

Could you point out the thing which you are referring to?

Pg 13 start a discussion

Yes, could you elaborate what is missing?

Nothing is missing.
The design of that page was suppose to be the same as the original proposal.


Integration page update

Added freeCodeCamp to the integration page.

Please do not modify package lock.
On Dec 3, 2017 00:30, "Justin Dorfman"<EMAIL_ADDRESS>wrote:

@jdorfman https://github.com/jdorfman requested your review on:
MaxCDN/bootstrap-cdn#891
https://github.com/MaxCDN/bootstrap-cdn/pull/891 Integration page
update.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/MaxCDN/bootstrap-cdn/pull/891#event-1369047847, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAVVtQs0jiklc9gmZLar3DX3AyzMda7cks5s8c-TgaJpZM4QzfmW
.

And the rest don't merge. I have it many similar changes in my bootstrap 4
branch. No need for this to go in before that.
On Dec 3, 2017 00:31, "XhmikosR"<EMAIL_ADDRESS>wrote:

Please do not modify package lock.
On Dec 3, 2017 00:30, "Justin Dorfman"<EMAIL_ADDRESS>wrote:

@jdorfman https://github.com/jdorfman requested your review on:
MaxCDN/bootstrap-cdn#891
https://github.com/MaxCDN/bootstrap-cdn/pull/891 Integration page
update.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/MaxCDN/bootstrap-cdn/pull/891#event-1369047847, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAVVtQs0jiklc9gmZLar3DX3AyzMda7cks5s8c-TgaJpZM4QzfmW
.

@XhmikosR should I add package-lock.json togitignore?

No. It's a bug in npm like I told you the other day.
Please do not merge. For one you should have squashed tour patches. Second
the package file. Third I'm pretty sure I will achieve better image
compression. Fourth it will conflict with my bootstrap 4 branch where I
changed to use an alert. Generally this should be closed and we can add the
new entry after my PR is merged.
On Dec 3, 2017 00:34, "Justin Dorfman"<EMAIL_ADDRESS>wrote:

@XhmikosR https://github.com/xhmikosr should I add package-lock.json to
gitignore?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MaxCDN/bootstrap-cdn/pull/891#issuecomment-348725121,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVVtWflTSsoRpjqz4AH4G5sBCGydJ2Cks5s8dB9gaJpZM4QzfmW
.

k when you're done, merge this branch.


Flip indices of cv images in addLayerFromImage to be consistent with use in toCvImage (#56)

As described in #56, index use between both methods appears to be inconsistent. This makes things work as expected for my use case (grid map -> cv image, cv operations, new cv image -> new grid map layer).

Superseded by #58.


Test

@ebeshero Here is my test

@Janman813 I am finally merging in old pull requests from the Git test! Good work on this round. You can safely delete your pituchTest branch on your local version of the textAnalysis-Hub if you have not done so already.


Update physica.typ

Makes i and j dotless when under a vector arrow.

Thanks! But I prefer creating minimum surprises for the users. I've seen math expressions where the accented i and j still have their dots, so I'd like to allow users to write just that. If a user want to have dotless i and j when an accent is present, they can do so explicitly, e.g. va(dotless.i), vu(dotless.i) .
.

Thanks! But I prefer creating minimum surprises for the users. I've seen math expressions where the accented i and j still have their dots, so I'd like to allow users to write just that. If a user want to have dotless i and j when an accent is present like , they can do so explicitly, e.g.
$ va(dotless.i), vu(dotless.i) $

or if they want dotless i and j throughout the paper:
#show "i": sym.dotless.i
...
$ va(i), vu(i) $

I understand that! It just seemed obvious to me that accentuated i and j should be dotless, since all of my teachers have been doing that.
I added a commit that add a dotless option set to false by default. Users can do let va(a) = vectorarrow(a, dotless: true) to make their writing more convenient instead of having to use dotless.i every time.
Tell if I should update the manual too.

Hi, I find it kind of clumsy, imho.. 😢 Typst currently doesn't support set rules for custom functions, so it seems you will have to specify the dotless arg every time. Moreover, this argument just takes effect on i and j, and it seems too "specialized" from an API design perspective, and it is doing something that should've been done at the symbol level.
Yeah I get that in many contexts accented vectors of i and j are dotless, as I've ran into them myself, but I still don't feel the the weight is heavy enough to tip the balance.
Maybe you could define a variables with va(dotless.i) etc. and use these variables whenever you want such vectors?

Typst currently doesn't support set rules for custom functions
Is there already an issue on the Typst compiler repo? This seems to me like a very useful feature for packages.

I definitely think that an option to get the behavior of the dotless letters would improbe this package, but I am not sure what the best implementation would be.
To circumvent the non-existence of set rules for custom functions, you could create a boolean variable, which the user can switch and then in the package code an if-expression which defines the function either in the old way or in the new way of the original commit. But to be honest I don't really like this solution, as this is really clumsy.
I am not sure, if the let va(a) = vectorarrow(a, dotless: true) solution of OPs comment works, but if it does and one doesn't need to specify it every time I would be in favor of adding this to the package, as I think a way to get this behavior with just setting one global variable would improve the package. (But it should be wrapped into a boolean variable, which does this.)

After further thinking, I'd probably wait until there is some easier way for setting global package options supported. Right now it feels quite hacky.
(FYI: This is what 🔥PgSuper🔥  on Discord said regarding package options:
egarding global package options , the most idiomatic equivalent thing nowadays would be using state with a global show rule, like
#show: package.with(option1: true, option2: false)

// becomes
#state("package", (:)).update((option1: true, option2: false))

// then, when you use a function...
#something-from-package()

// ...it retrieves the data
#let something-from-package() = context {
  let (option1, option2) = state("package", (:)).get()
  [Yay: #option1 #option2]
}

however , the problem is that it makes package functions contextual, so they all return opaque content
removing the context  keyword will help with that (they wont return opaque content anymore), but then you will always need to be inside a context to use something from the package, i.e. you'd write #context something-from-package()
which is a minor annoyance but maybe not that bad depending on the package
but the true equivalent to global, immutable options  (The previous approach is the only possibility when you want to be able to change options mid-document) would be to make a single function returning functions
i.e. your package is defined as
#let init(option1: true, option2: true) = {
  let something-from-package() = [Yay: #option1 #option2]

  (something-from-package: something-from-package)
}

then, in your document
#import "package": init
#let (something-from-package,) = init(option1: true, option2: false)

// Works, no `context` needed
#something-from-package()

right now these are the two approaches for global options (state and function returning functions), in the future it's possible we will have more ergonomic approaches though
previous discussions:
here https://github.com/typst/typst/issues/2403 and here https://github.com/typst/typst/issues/2636 )

Thanks! I'll close this for now.


presenting views on top of presenter issue

i have a presentr view shown on top of of the screen if I present a view on top of presentr then dismiss the view the is a slight flicker of the presentr view, it shows it full screen then it goes to the top of the scree the way it is suppose to be. have you experience this issue or known of it.

@pprevalon I has some issue with you.

Use Presentr present show ViewControllerA with half type.
Present ViewControllerB in ViewControllerA
Try dismiss ViewControllerB. The ViewControllerA show FullScreen and fast show half screen.

How fix it? Thx!

Same with https://github.com/IcaliaLabs/Presentr/issues/68

I try secondVC.modalPresentationStyle = .overFullScreen fixed my issue.

Yeah, it's a known Apple SDK issue with custom view controller presentation.
I outlined a couple of fixes/workarounds in #68


Revert redis-py and Kombu updates

Description of change
Celery 4.2.2 was released but they restricted redis to <3 and Kombu to <4.4 in it (even though Celery 4.2.1 has been out for some time and did not have a redis<3 restriction).
Downgrading redis-py to 2.x now seems risky so this reverts things to how they were in the last release.
When Celery 4.3 is released, we can look at updating these again.
Checklist

[ ] Has a new newsfragment been created? Check changelog/README.rst for instructions
[ ] Have any relevant search models been updated?
[ ] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[ ] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[ ] Has the admin site been updated (for new models, fields etc.)?
[ ] Has the README been updated (if needed)?

After we upgraded redis py and Kombu the issue with MI job disappeared. We should monitor that once this is merged.

@elcct This puts this back as they are on master, so we shouldn't have to worry about that.

Ah true! Great


Update README.md and add LIKE operator example

This PR adds an example with LIKE operator to README.md.

Merged cc: @dmikurube


Set texture

Dynamically change textures using an url/image/canvas/data by providing one of this on the scene.config.textures.  Ex:
scene.config.textures.orbits.url = "http://..../image.png";
scene.updateConfig({ rebuild: true });

// or

var canvas = new document.createElement("canvas");
...
scene.config.textures.orbits.canvas = canvas;
scene.updateConfig({ rebuild: true });

// or

var image = new document.createElement("image");
...
scene.config.textures.orbits.image = image;
scene.updateConfig({ rebuild: true });

// or

var raw_data = { data: [], width: 500, height: 500 }
...
scene.config.textures.orbits.data = raw_data;
scene.updateConfig({ rebuild: true });

Can you please add a description of changes, and usage examples?

Please also remember to not commit the dist/ files in the future, we should remove them from this PR, but even adding them in a single commit adds them to history forever :/

Should I delete and create again this branch?

I don't think so, we can rebase before merging.
On Thursday, December 10, 2015, Patricio Gonzalez Vivo <
<EMAIL_ADDRESS>wrote:

Should I delete and create again this branch?
—
Reply to this email directly or view it on GitHub
https://github.com/tangrams/tangram/pull/226#issuecomment-163653804.

sorry, I don't remember when I include them

We don't commit them except when a new release is cut'm
On Thursday, December 10, 2015, Patricio Gonzalez Vivo <
<EMAIL_ADDRESS>wrote:

sorry, I don't remember when I include them
—
Reply to this email directly or view it on GitHub
https://github.com/tangrams/tangram/pull/226#issuecomment-163656582.

Now that we have this working, can we combine canvas and image into a single element parameter for textures? We can detect based on type, and other aspects are similar (both can be provided directly as WebGL textures).

Done! Also, that lint error on text.js is annoying. Any idea how to solve it?

Re: lint, just remove let texture =  and let the new Texture call be on its own. I don't really like that syntax wise, but it's a result of how the rest of the system works. Maybe I can replace with a static instantiation method that would be clearer (Texture.create).

:+1:

should I merge master on it? or you want to do it before merging this brunch into master?

No please wait, I will review more later, thanks.

@patriciogonzalezvivo with https://github.com/tangrams/tangram/commit/4172dc5534f89740ee3ba09d106536c54c3ee59d, the element can be set to a CSS selector string.
So you can setup HTML like:
<canvas class='awesome-texture'>
And then reference it as a texture with:
textures:
   awesome-texture:
      element: .awesome-texture

Or you could reference an element by id, but the use of # in CSS selector means you need to be careful to quote the YAML so it's not interpreted as a comment :) For example:
<canvas id='texture-id'>
textures:
   awesome-texture:
      element: '#texture-id'  # without quote, the # will be interpreted as a YAML comment!

:+1: nice!

yay!


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/kn-plugin-operator-approvers
/assign knative-sandbox/kn-plugin-operator-approvers
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/hack 9b303d6...7331bb1:
  > 7331bb1 rename function (# 149)
bumping knative.dev/operator d16dced...cfbc645:
  > cfbc645 Keep the default image name the same as the original (# 958)
  > fc76718 Remove the continual tests for serving upgrade (# 959)

/lgtm
/approve


Js ubuntu support

Made changes to support Ubuntu. This is needed to support NDv2 and NDv4 fleet health checking

@JonShelley can you please split this PR into 2, one for OpenPBS and one with the ubuntu support. I would suggest you create a new branch for OpenPBS and update only files for it. Once done it would be easier to merge in the master.
once we have merge this OpenPBS in the master, you can merge the master in this ubuntu branch, and solve the conflicts.

@JonShelley maybe the easiest would be to leave the jumpbox running on CentOS but allow compute nodes and job scheduler to run on Ubuntu. Doing this you need smaller updated in the python code.
@edwardsp please comment on this too.


Device Compliance

Hello,
Would be very useful to query device compliance status by Dynamic groups.
User vote created, but maybe, there is a way already?

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

ID: 5f7a4070-1be8-5f1f-0092-68fb6b9e2305
Version Independent ID: cd34312f-b507-affe-555e-1e0c63e7d494
Content: Dynamic automatic group membership rules - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/groups-dynamic-membership.md
Service: active-directory
GitHub Login: @curtand
Microsoft Alias: curtand

@lightupdifire Thanks for the feedback! We are currently investigating and will update you shortly.

@BryanTrach-MSFT ,
Thanks! It would be very useful for combination of multiple Conditional Access policies and other rules.

Hi @lightupdifire ,
I'm not sure if this is what you are looking for, but you can create groups by applying a rule to determine membership based on device properties. When the attributes of a device change, Azure AD evaluates all dynamic group rules in the Azure AD tenant and performs any adds or removes. See the guide for this: https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/groups-create-rule

If you are looking for something more detailed than this I'm not sure if it is supported.
We will proceed to close this thread now but if you have further questions feel free to reach out to me on MSDN. https://social.msdn.microsoft.com/Forums/en-US/home?forum=WindowsAzureAD

@MarileeTurscak-MSFT,
Thanks, I know how Dynamic groups works, we do query already different groups with condition as:
Device = Corporate -and Device managed = MDM -and Account enabled = true; etc.
Question is how to use Dynamic group for condition "isCompliant",
The "isCompliant" not exist in Dynamic group documentation and not available in options while creating group, just wanted to check if there is a way to get that other, logic way... but seems will have to figure out myself.
Please make a note for "isCompliant" with Dynamic Groups, as this is very-very useful for Conditional Access rules and Configuration or Compliance Profiles.
Thanks anyway for all feedback!


GN build parity: doxygen

The GN build needs support for building doxygen to reach parity with automake.

Tentatively marking this non blocking. I don't think anyone depends on doxygen generation, and personally I find doxygen documentation less useful than browsing the source even with a text editor. Browsing on https://cs.opensource.google/ with xrefs or VSCode with libclang is much better.


Fix travis: build scala to 2.10.4

failed here: https://travis-ci.org/andypetrella/spark-notebook/builds/97757066
cc @andypetrella

ah nice, it would also be interesting to, at least launch the tests on 2.11 for the last spark version   :-/


Add Context, Failure, Formatter types

Added following types:

Context struct
Failure struct
Formatter interface

Fixes: #69

Coverage remained the same at 93.955% when pulling 89e8d24e894fd99297a35beee826a1d673117b0e on goku321:add-context-failure-formatter into 4cfa969e232100a06000d730abb34c300e54f9cb on gavv:master.

Thanks a lot for your PR and sorry for huge delay. I'll review it in upcoming days.

LGTM!
Since we'll have multiple PRs, I created a feature branch for this issue (named context):
https://github.com/gavv/httpexpect/commits/context
We'll merge related PRs into it, and when the feature is done, I'll merge context into master.


Modelmesh is not running after Kserve installed and uninstalled.

/kind bug
What steps did you take and what happened:
This issue is always reproducible when you install kserve and uninstall kserve then install modelmesh.
Modelmesh controller failed to run because the conversion webhook in the inferenceservice CRD.
What did you expect to happen:
kserve and modelmesh should run without any issues.
What's the InferenceService yaml:
No need to have isvc yaml
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

Kserve has conversion webhook in the inferenceservice CRD but modelmesh does not have it.
CRD will stay even though kserve controller is uninstalled.
inferenceservice CRD still looks for kserve controller as a webhook server even after kserve operator is uninstalled. That's why modelmesh controller failed to run.

  conversion:
    strategy: Webhook
    webhook:
        conversionReviewVersions: ["v1beta1"]
        clientConfig:
          # this is "\n" used as a placeholder, otherwise it will be rejected by the apiserver for being blank,
          # but we're going to set it later using the cert-manager (or potentially a patch if not using cert-manager)
          caBundle: Cg==
          service:
            namespace: $(kserveNamespace)
            name: $(webhookServiceName)
            path: /convert

At the moment, I removed the conversion webhook from CRD to solve this issue. However, if kserve api version is changed, the conversion webhook should be added then this issue will be back.
What would be a good idea to solve this issue?
Environment:

Istio Version:
Knative Version:
KServe Version: 0.11
Kubeflow version:
Cloud Environment:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm]
Minikube/Kind version:
Kubernetes version: (use kubectl version): v1.27.8
OS (e.g. from /etc/os-release):

I know Kserve have a plan to change api version. It means conversion webhook will be needed for kserve and modelmesh both. Then it will cause CRD conflict issue as well because modelemsh CRD will point to modelmesh controller pod and kserve CRD will point to kserve controller pod.
Why don't we create a common service pointing to one of them and use the service in the conversion webhook?
@yuzisun WDYT?

FYI @ckadner
This issue might require some changes in modelmesh-serving

@Jooho -- I don't know enough about conversion webhooks, but reading your analysis, it sounds like the webhook should get uninstalled/removed when KServe is uninstalled. Is that what should be done but somehow it is not technically feasible?

@ckadner conversion webhook is a kind of special one and it is not a separate webhook like validating/mutating webhook. It is in the CRD. InferenceService CRD in KServe has the webhook
So if kserve is installed first and uninstalled, the CRD will exist. Then if Modelmesh is installed, the CRD will check webhook server(kserve controller). Definitely, the KServe controller is not there so it blocked modelmesh to run.

Ensure Clean Uninstallation of Kserve:

Verify complete Kserve uninstallation.
Check for lingering Kserve CRDs using: kubectl get crds | grep kserve

Remove Kserve-related CRDs:

Delete remaining Kserve CRDs: kubectl delete crd inferenceservices.serving.kserve.io

@Syntax-Error-1337 I don't think deleting CRDs cleanly is the right way.
The scenario that I explained in the description is one of the examples. Let me give you another scenario.

The main issue of this problem is 2 controllers sharing CRDs with different variables for conversion webhook.
That's why I recommend that we should share a Service to use one of the controllers because it does not matter which operator can handle the conversion webhook.
@ckadner WDYT?

I took a look at this issue more and found some more stuff.
Issues

There is no proper uninstallation script or doc for Kserve/modelmesh

CRD clean up issue
If users do something like this, it will remove CRDs as well even modelmesh/kserve were both installed together. Then it causes removing all existing ISVCs for modelmesh as well.
KSERVE_VERSION=0.11.1
wget https://github.com/kserve/kserve/releases/download/v${KSERVE_VERSION}/kserve.yaml
kubectl delete -f ./kserve.yaml

mutating/validating webhooks clean up issue
If mutating/validating webhooks stays after each component is uninstalled, it can impact one or the other component   because the component is not running.

Generated CRD by kustomize for KServe/Modelmesh might be different based on patch yamls

Conversion webhook should use a sharable Service object.
Conversion webhook should be set in CRD and modelmesh/kserve are shaing CRDs. If each component uses its Service then it can cause an issue.

Ideas to solve these issues
CRD clean-up issue

We need to uninstall process or script (I think we can remove all objects except CRDs)

mutating/validating webhooks clean up issue

This will be solved if we provide a proper uninstall script or guide.

Generated CRD by kustomize for KServe/Modelmesh might be different based on patch yamls

Need to add a new task to check the final CRD generated by kustomize or Helm in Release process

Conversion webhook should use a sharable Service object

Both components should use a sharable Service : likeapiVersion: v1
kind: Service
metadata:
  labels:
    app.kubernetes.io/instance: kserve-modelmesh-shared
    app.kubernetes.io/managed-by: kserve-modelmesh-shared
    app.kubernetes.io/name: kserve-modelmesh-shared
  name: kserve-modelmesh-webhook-server-service
  namespace: kserve
spec:
  ports:
  - port: 9443
    protocol: TCP
    targetPort: webhook
  selector:
    control-plane: kserve-modelmesh-controller

Both components controller need to add the same label : like controller-plane=kserve-modelmesh-controller
We should document that if users want to use both components (modelmesh,kserve), the user must use the same namespace for them.

@danielezonca @ckadner @yuzisun

@ckadner @yuzisun I send a PR to use the final CRDs for HELM but it has the conversion webhook for Kserve. It would not work for modelmesh. So we should sharable a Service for it.

At the moment, kserve does not have the conversion webhook  so removing the conversion webhook related information from manifests and change the logic to generate CRDs for now

https://github.com/kserve/kserve/pull/3476
https://github.com/kserve/kserve/pull/3392

However, we should work closely with modelmesh when KServe API version need to be updated.
Here is the list we should work for conversion webhook when it is needed:
Action items:
KServe/Modelmesh part

Create a sharable Service manifests and update conversion webhook service name in CRD.
Add sharable label component: kserve in the Deployment manifest for both controller.
Update SAN list for a new service name (cert-manager/ self-signed cert script and so on)

ModelMesh part

Copy conversion webhook logic from kserve to modelmesh.

@ckadner I think this way would be the right approach for now. please review this.

@Jooho -- I see https://github.com/kserve/kserve/pull/3476 is merged now. Do all of the action items above still apply. If so, did/could you you create respective (self-contained) issues?


⚠ Bofrost.lu has degraded performance

In c207f69, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1035 ms

Resolved: Bofrost.lu performance has improved in cfc62d7.


fix: Removes chat.js from build

frappe js build errors during frappe.chat import which was removed previously. This PR removes chat.js from build.json

@DeveloperMetal, did you mean chat or chart? 😅

@Alchez good catch 😅 although chart.js is throwing some errors while installing now as well. Although I think Felipe might have done this for JHAudio

@Alchez @rvpasari Ah, oops. I meant Chart, Sorry! I believe @neilLasrado removed the dependency and this fixes an importing issue you can see while you run the build command.
I'll edit things


where is the config file for Peds Are Always Visible

where is the config file for Peds Are Always Visible

I found it


Need labels to triage/filter issues related to setup.py test

This repository doesn't make use of labels. It should be.

This repository doesn't make use of labels.

It does. It just doesn't have labels for what you're expecting. What are you expecting (more generally)? Can you propose a scheme that the project could use that would help?
I'm a little reluctant to add a command-<name> label for each command. test is too generic and could easily be confused with setuptools' tests.
Plus, I'd like to deprecate the test command.

I think that every part of setuptools functionality is an area that deserves a separate keyword. Even it testing means mixing both test command and setuptools tests, but I'd like to concentrate on setup.py test only as it is more valuable for users.

I think the mention of setup.py test or test command is sufficient to free-text search for a feature that's largely deprecated. Thanks for the suggestion.


about the decrease of the loss

Hello. Thanks for your excellent work!
I transplant the loss function and memory bank of your code to my own code, and run my code on Cityscapes dataset and my own dataset. But it turns out that the decrease of the contrast loss is very small. For example, the contrast loss drops from 1.27 to 1.11 after 80k epoch on Cityscapes, while the ce loss drops from 1.26 to 0.15. And the same thing happens on my own dataset. It seems that the contrast loss is not being useful during train.
I wonder is it normal for the contrast loss to decrease so little? And what can I do to make full use of the contrast loss, like tuning the hyper-parameters?

I have the same problem too, looking forward to the author's answer

@hhxzzy I'm sorry to bother you.I added the contrast loss function to my model and code, and this problem also occurred.Has your problem been solved?

@hhxzzy I'm sorry to bother you.I added the contrast loss function to my model and code, and this problem also occurred.Has your problem been solved?

Hello. I try to make some change to the code, but the contrastive loss is still not going to drop much. But it seems that adding the contrastive can help improve the mIOU on Cityscapes by about 1%. Maybe it is the nature of contrastive loss that it doesn't drop much.
By the way, do you use warmup_iter in your code. I found that using a warm up strategy can be helpful. The author use this strategy in his code but not metion it in the paper.

已收到

@hhxzzy
I am a novice postgraduate, and my coding ability is not high. I want to use ContrastiveSeg method on my own two-class segmentation model. Which py files should I read or what should I pay attention to?
Looking forward to your answer, thank you very much.

@tfzhou @hhxzzy @wingkeihon @guwen007
I am a novice postgraduate, and my coding ability is not high. I want to use ContrastiveSeg method on my own two-class segmentation model. Which py files should I read or what should I pay attention to?
Looking forward to your answer, thank you very much.

@tfzhou @hhxzzy @wingkeihon @guwen007 I am a novice postgraduate, and my coding ability is not high. I want to use ContrastiveSeg method on my own two-class segmentation model. Which py files should I read or what should I pay attention to? Looking forward to your answer, thank you very much.

If you want to run the author's scripts directly, you can take configs/cityscapes/H_48_D_4.json as an example. You need to change "data" and "loss“/"params"/"ce_weight" according to your dataset. If you wanna use contrastive loss on your own code instead of running the author's scripts, you should look at lib/loss/loss_contrast.py and lib/loss/loss_contrast_mem.py to read the implement of contrastive loss.

Thank you for your prompt attention.


Authentication mechanism

There's a few authentication mechanisms that I can think of to support:

Our own signup/login
LinkedIn https://docs.microsoft.com/en-us/linkedin/consumer/integrations/self-serve/sign-in-with-linkedin?context=linkedin/consumer/context
Google Sign-in https://developers.google.com/identity/sign-in/web/sign-in

LinkedIn
Pros

ability to load work profile for job seekers
ability to load company profile for job posters

Cons

user base needs to have LinkedIn account

Google
Pros

more likely to have a gmail account for most user base

Cons

not much information to pre-populate other than email address

Would we want to only help users with LinkedIn account? Older non tech savvy folks might not have LinkedIn account, but then again they might not even use our app.

My initial thought was if we want to share about a job post on LinkedIn it's going to be easier if we already have LinkedIn authentication in place otherwise we can always add it later.

I think we should go with LinkedIn anyway, as the more I think about it, I think the user population are likely to have LinkedIn account. And prepopulating the job search profile might be very useful in helping new sign ups

I am closing this issue as we decided to go with LinkedIn for first version. We can revisit this later after the first version is released.


CI build

Initial CI build support (#1):

Installation of repotool
Poky setup (stub, replaced by #2)
Simple bitbake

⚠  Note:
While GH Actions requires root user (doc), Bitbake doesn't allow it.
Therefore bitbake commands use /opt/yocto/ as working directory and run with a yocto user shell (as in the image).

lgtm


[Event Hubs] Support exponential retry policy

We are adding support for exponential retries in core-amqp as part of #3331
This issue is to absorb these changes in Event Hubs so that our customers can make use of the exponential retries
Use this issue to also remove the concept of "jitter" that we are using in a couple of places for determining the delay between retries. Since the exponential retry adds an element of randomness we dont need to use jitter

Closed with #4400


Kubernetes upgrades maintenance

Can you explain what those changes mean?

Those are mostly upgrades to our k8s manifests file and small update to github actions file to stay up to date.


return error if get user failed

this will destroy vm immediately.
rearrange codes by the way.
Signed-off-by: Gao feng<EMAIL_ADDRESS>

LGTM


Release drafter for automatic release descriptions

I noticed that we don't always have release notes.
We used the following Github Action (https://github.com/marketplace/actions/release-drafter) in some projects in the past which can automatically create release notes based on PR/commit messages. This could be a low effort  approach to some nice looking release notes :)

We use the auto-generate feature for release notes by now, as you can see here: https://github.com/dfinity/sdk/releases


Integration with Canadian data portal

Refactored local.py into two separate scripts.
One script is used to combine and filter data into an xz-compressed JSON file, or append to an existing file.
A second script is used with this xz-JSON as the primary input for analysis.
python3 convert.py working/n1e4.fasta.xz --xz working/n1e4.pangolin.csv working/metadata.tsv --delimiter '\t' --name strain --outfile test.json.xz

may equivalently be run as:
python3 convert.py working/n1e4.fasta.xz --xz working/n1e4.pangolin.csv working/metadata.tsv --delimiter '\t' --name strain | xz > test.json.xz

Encountering some significant problems with the metadata TSV from the data portal:

no region (North America) or country (Canada) fields
there is no label that can be used to match rows in metadata table to sequence records in the FASTA file

Country and sequence labels are available as optional fields (see Columns drop-down).

Can now append to JSON xz file:
art@Wernstrom covizu % python3 convert.py working/n1e4.fasta.xz --xz working/n1e4.pangolin.csv working/metadata.tsv --delimiter '\t' --name strain | xz > test.json.xz
art@Wernstrom covizu % unxz -c test.json.xz | wc -l
    8722
art@Wernstrom covizu % gunzip -c working/virusseq-consensus-export-2021-05-25T17_26_15.259802Z.fasta.gz | grep -c ">"
20
art@Wernstrom covizu % python3 convert.py working/virusseq-consensus-export-2021-05-25T17_26_15.259802Z.fasta.gz working/virusseq-pangolin-2021-05-25.csv working/virusseq-metadata-export-20210526.tsv --gzip --delimiter '\t' --name Isolate --coldate 'Sample Collection Date' --region Region --country Country --append test.json.xz
🏄 [0:00:00.081432] Rejected 0 short genomes, 0 records with bad dates
art@Wernstrom covizu % unxz -c test.json.xz | wc -l
    8742

Oops - serialized Python dicts are not equivalent to JSON after all because of the single quotes:
art@orolo:~/git/covizu$ python3 process.py test.json.xz --outdir working/iss282/
Traceback (most recent call last):
  File "process.py", line 153, in <module>
    analyze_feed(handle, args, callback=cb.callback)
...
json.decoder.JSONDecodeError: Expecting property name enclosed in double quotes: line 1 column 2 (char 1)

Fixed.

🏄 [0:01:41.084760] start B, 327 entries
🏄 [0:01:57.955842] start A, 257 entries
🏄 [0:02:05.278295] start B.1, 2924 entries
--------------------------------------------------------------------------
Open RTE was unable to open the hostfile:
    mfile
Check to make sure the path and filename are correct.
--------------------------------------------------------------------------
Traceback (most recent call last):
  File "process.py", line 158, in <module>
    analyze_feed(handle, args, callback=cb.callback)
  File "process.py", line 125, in analyze_feed
    result = make_beadplots(by_lineage, args, callback, t0=timestamp)
  File "/home/art/git/covizu/covizu/utils/batch_utils.py", line 103, in make_beadplots
    subprocess.check_call(cmd)
  File "/usr/lib/python3.6/subprocess.py", line 311, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['mpirun', '--machinefile', 'mfile', 'python3', 'covizu/clustering.py', 'data/by_lineage.json', 'B.1', '--nboot', '100', '--outdir', 'data', '--timestamp', '2021-05-26T20:57:00']' returned non-zero exit status 1.

Should have an option for running without an MPI environment.

Disabled MPI by default.
Ran into another problem:
art@orolo:~/git/covizu$ python3 process.py test.json.xz --outdir working/iss282/ --ft2bin fasttree2-mp
🏄 [0:00:00.845342] aligned 0 records
...
🏄 [0:09:04.698638] start B.1.468, 2 entries
Traceback (most recent call last):
  File "process.py", line 163, in <module>
    analyze_feed(handle, args, callback=cb.callback)
  File "process.py", line 130, in analyze_feed
    result = make_beadplots(by_lineage, args, callback, t0=timestamp)
  File "/home/art/git/covizu/covizu/utils/batch_utils.py", line 92, in make_beadplots
    beaddict = beadplot_serial(lineage, features, args)
  File "/home/art/git/covizu/covizu/utils/batch_utils.py", line 38, in beadplot_serial
    for coldate, accn, label1 in intermed:
ValueError: not enough values to unpack (expected 3, got 2)

Fixed a problem with running MPI with -np flag.
Much faster even with 6 cores.
Reproduced error with some more info using this modification to batch_utils.py:
        try:
            for coldate, accn, label1 in intermed:
                beaddict['nodes'][variant].append([coldate, accn, label1])
        except ValueError:
            print(intermed)
            raise

🏄 [0:03:42.880638] start B.1.468, 2 entries
[['2020-02-13', 'SAU/545/2020'], ['2020-02-18', 'SAU/596/2020']]

Successful run of backend with commit ca7a0a5
Front end fails to parse data files:
Uncaught TypeError: Cannot read property 'replace' of undefined
    at parse_variant (beadplot.js:153)
    at parse_clusters (beadplot.js:380)
    at Object.<anonymous> (covizu.js:150)
    at fire (jquery.js:3496)
    at Object.fireWith [as resolveWith] (jquery.js:3626)
    at done (jquery.js:9786)
    at XMLHttpRequest.<anonymous> (jquery.js:10047)

Need to load country annotations into JSON data

For some reason district keys are not being carried over from by_lineage.json:
🏄 [0:01:39.482498] start B, 327 entries
Traceback (most recent call last):
  File "process.py", line 163, in <module>
    analyze_feed(handle, args, callback=cb.callback)
  File "process.py", line 130, in analyze_feed
    result = make_beadplots(by_lineage, args, callback, t0=t0.timestamp())
  File "/home/art/git/covizu/covizu/utils/batch_utils.py", line 93, in make_beadplots
    beaddict = beadplot_serial(lineage, features, args)
  File "/home/art/git/covizu/covizu/utils/batch_utils.py", line 28, in beadplot_serial
    trees, labels = clustering.build_trees(features, args, callback=callback)
  File "/home/art/git/covizu/covizu/clustering.py", line 217, in build_trees
    union, labels, indexed = recode_features(records, callback=callback)
  File "/home/art/git/covizu/covizu/clustering.py", line 34, in recode_features
    label = '{label}|{region}|{country}|{district}|{coldate}'.format(**record)
KeyError: 'district'

Gah!  keyword is division, not district

Running into some difficults with front-end because we stopped requiring sample accessions in the data.
We were using accession numbers as unique identifiers for SVG elements.
Also running into this problem:
drawtree.js:277 Uncaught TypeError: mutations is not iterable
    at mutations_to_string (drawtree.js:277)
    at mouseover (drawtree.js:324)
    at SVGTextElement.<anonymous> (drawtree.js:416)
    at SVGTextElement.<anonymous> (d3.js:1526)

For some reason tips are not getting annotated with mutations.

We'll continue to try to modify the front-end to remove all dependency on sample accession numbers, and assume that sequence names are unique (need to add a check for this in backend).
That means that sequence names are re-used as element IDs, in addition to being text labels in some cases.

If we omit accession numbers then these are not available to the user via the front-end.

Close and populate specific issues, e.g., #311


Unable to run example tests with reporter - Cannot read properties of undefined (reading 'rootSuite')

Hi,
I created repository for problem reproduction https://github.com/shwarcu/playwright-reportportal-example.
Here is how I got up to this point:
I deployed ReportPortal locally with docker and created initial tests setup with npm init playwright@latest.
Then I added agent-js-playwright reporter to the config and changed testDir to take tests from ./tests-examples.
When running example tests I keep getting this error:
Running 50 tests using 4 workers

  ✓  1 [chromium] › demo-todo-app.spec.ts:61:3 › New Todo › should show #main and #footer when items added (1s)
  ✓  2 [chromium] › demo-todo-app.spec.ts:14:3 › New Todo › should allow me to add todo items (1s)
  ✓  3 [chromium] › demo-todo-app.spec.ts:47:3 › New Todo › should append new items to the bottom of the list (1s)
  ✓  4 [chromium] › demo-todo-app.spec.ts:37:3 › New Todo › should clear text input field when an item is added (1s)
  ✓  5 [chromium] › demo-todo-app.spec.ts:81:3 › Mark all as completed › should allow me to mark all items as completed (795ms)
     6 [chromium] › demo-todo-app.spec.ts:90:3 › Mark all as completed › should allow me to clear the complete state of all items
     7 [chromium] › demo-todo-app.spec.ts:99:3 › Mark all as completed › complete all checkbox should update state when items are completed / cleared
     8 [chromium] › demo-todo-app.spec.ts:122:3 › Item › should allow me to mark items as complete
/home/maciej/workspace/playwright-reportportal-example/node_modules/@reportportal/agent-js-playwright/build/reporter.js:462
                        rootSuiteName = parentObj.rootSuite;
                                                  ^

TypeError: Cannot read properties of undefined (reading 'rootSuite')
    at RPReporter.<anonymous> (/home/maciej/workspace/playwright-reportportal-example/node_modules/@reportportal/agent-js-playwright/src/reporter.ts:483:37)
    at step (/home/maciej/workspace/playwright-reportportal-example/node_modules/@reportportal/agent-js-playwright/build/reporter.js:60:23)
    at Object.next (/home/maciej/workspace/playwright-reportportal-example/node_modules/@reportportal/agent-js-playwright/build/reporter.js:41:53)
    at /home/maciej/workspace/playwright-reportportal-example/node_modules/@reportportal/agent-js-playwright/build/reporter.js:35:71
    at new Promise (<anonymous>)
    at __awaiter (/home/maciej/workspace/playwright-reportportal-example/node_modules/@reportportal/agent-js-playwright/build/reporter.js:31:12)
    at RPReporter.onTestEnd (/home/maciej/workspace/playwright-reportportal-example/node_modules/@reportportal/agent-js-playwright/build/reporter.js:423:16)
    at /home/maciej/workspace/playwright-reportportal-example/node_modules/@playwright/test/lib/reporters/multiplexer.js:69:131
    at wrap (/home/maciej/workspace/playwright-reportportal-example/node_modules/@playwright/test/lib/reporters/multiplexer.js:119:5)
    at Multiplexer.onTestEnd (/home/maciej/workspace/playwright-reportportal-example/node_modules/@playwright/test/lib/reporters/multiplexer.js:66:45)
    at Dispatcher._reportTestEnd (/home/maciej/workspace/playwright-reportportal-example/node_modules/@playwright/test/lib/dispatcher.js:574:159)
    at Worker.onTestEnd (/home/maciej/workspace/playwright-reportportal-example/node_modules/@playwright/test/lib/dispatcher.js:240:12)
    at Worker.emit (node:events:390:28)
    at ChildProcess.<anonymous> (/home/maciej/workspace/playwright-reportportal-example/node_modules/@playwright/test/lib/dispatcher.js:633:12)
    at ChildProcess.emit (node:events:390:28)

Related issue https://github.com/reportportal/agent-js-playwright/issues/42
Probably regression from last release?

I also see these errors lately:
(node:9318) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'rootSuite' of undefined
583    at RPReporter.<anonymous> ([<path hid by me>]/node_modules/@reportportal/agent-js-playwright/src/reporter.ts:483:37)
584    at step ([<path hid by me>]/node_modules/@reportportal/agent-js-playwright/build/reporter.js:60:23)
585    at Object.next ([<path hid by me>]/node_modules/@reportportal/agent-js-playwright/build/reporter.js:41:53)
586    at fulfilled ([<path hid by me>]/node_modules/@reportportal/agent-js-playwright/build/reporter.js:32:58)
587(Use `node --trace-warnings ...` to show where the warning was created)
588(node:9318) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
589(node:9318) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

The reportportal shows that the jobs are running even tho they were finished on CI. Nothing was changed by us around configuration and it was working in past with no issues.
it is present on versions (the only ones I checked so far):
"@reportportal/agent-js-playwright": "^5.0.2",
"@reportportal/agent-js-playwright": "^5.0.5",

Something has happened between 15.08.2022 11:30 CET time - this was the last run we've had in a project and everything was okay. The first occurrence I've spotted at 15.08.2022 21:00 CET time after this date almost every run contains this error.

Hello guys!
We have released a version 5.0.6 with a fix.
Could you confirm that this fixes the issue?

@AmsterGet Hi, I will check that afternoon when I'm done with my daily duties ;)

Hi, sorry for delayed response. After updating the package, it is indeed working without errors 💯


reflect: memory corruption when operating on pointer types

What version of Go are you using (go version)?

$ go version
go version go1.12 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN="/home/acln/bin"
GOCACHE="/home/acln/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/acln/src/gopath"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build938821885=/tmp/go-build -gno-record-gcc-switches"

What did you do?
I ran the following program: https://play.golang.org/p/Oawu1xbQTSN
The number of * in that one is 250. Moving forward, this program is named P250. P250 runs fine, both in the playground and on my local machine. For all x >= 0 && x <= 250, the respective program Px appears to work fine.
P251
This is P251: https://play.golang.org/p/mm0AO0I8F4M. It is P250, but with an added *.
In the playground, P251 appears to be an infinite loop: https://play.golang.org/p/-P4HziKyv9W
On my local machine, however, I get the following from P251, which is perhaps more enlightening:
: forge:reflectcorruption ; ./reflectcorruption
panic: runtime error: growslice: cap out of range

goroutine 1 [running]:
fmt.(*buffer).WriteString(...)
	/usr/local/go/src/fmt/print.go:83
fmt.(*fmt).padString(0xc000088040, 0x63202c6f6c6c6568, 0x772074707572726f)
	/usr/local/go/src/fmt/format.go:110 +0xfa
fmt.(*fmt).fmtS(0xc000088040, 0x63202c6f6c6c6568, 0x772074707572726f)
	/usr/local/go/src/fmt/format.go:347 +0x61
fmt.(*pp).fmtString(0xc000088000, 0x63202c6f6c6c6568, 0x772074707572726f, 0xc000000076)
	/usr/local/go/src/fmt/print.go:448 +0x132
fmt.(*pp).printArg(0xc000088000, 0x49da40, 0xc0000101f0, 0x76)
	/usr/local/go/src/fmt/print.go:684 +0x880
fmt.(*pp).doPrintln(0xc000088000, 0xc000041f78, 0x1, 0x1)
	/usr/local/go/src/fmt/print.go:1159 +0xb2
fmt.Fprintln(0x4e00a0, 0xc00000e018, 0xc000082f78, 0x1, 0x1, 0x98, 0x0, 0x0)
	/usr/local/go/src/fmt/print.go:265 +0x58
fmt.Println(...)
	/usr/local/go/src/fmt/print.go:275
main.main()
	/home/acln/src/reflectcorruption/main.go:12 +0x157a

Note the bogus addresses in the fmt.(*pp).fmtString frame:

0x63202c6f6c6c6568 is c ,olleh
0x772074707572726f is w tpurro

P252
P252 is also an infinite loop in the playground, but crashes on my local machine like so:
: forge:reflectcorruption ; ./reflectcorruption
unexpected fault address 0x0
fatal error: fault
[signal SIGSEGV: segmentation violation code=0x80 addr=0x0 pc=0x488567]

goroutine 1 [running]:
runtime.throw(0x4c60a8, 0x5)
	/usr/local/go/src/runtime/panic.go:617 +0x72 fp=0xc00008a778 sp=0xc00008a748 pc=0x427ba2
runtime.sigpanic()
	/usr/local/go/src/runtime/signal_unix.go:397 +0x401 fp=0xc00008a7a8 sp=0xc00008a778 pc=0x43a301
main.main()
	/home/acln/src/reflectcorruption/main.go:12 +0x1517 fp=0xc00008af98 sp=0xc00008a7a8 pc=0x488567
runtime.main()
	/usr/local/go/src/runtime/proc.go:200 +0x20c fp=0xc00008afe0 sp=0xc00008af98 pc=0x4294ec
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:1337 +0x1 fp=0xc00008afe8 sp=0xc00008afe0 pc=0x450ad1

P253 and onwards
These seem to be infinite loops wherever I run them.
What did you expect to see?
I expected P251 and onwards to work just as P250 did.
What did you see instead?
P251 and subsequent programs appear to corrupt memory, loop infinitely, or crash in mysterious ways.

I will note that this bug most certainly did not show up in a real program, and is merely the result of the #darkarts channel on the Gophers Slack poking at the reflect package.

Interesting test case.  This is an instance of #29312.  Closing in favor of that issue.

@acln0 of note, P500000 crashes the builder with a stack overflow.


Apply flutter_lints

To improve and apply standard to code, is recommended use flutter_lints.
Take a look to this document to get context

Solved in version 1.0.3


Get Block Range API

Allows for getting information on blocks in a range with a single call.
This is a very heavy api and should not be enabled on public nodes. To make sure this doesn't happen, a config feature has been added to allow disabling certain RPC methods. The default config disables the new method.

One more thing: please make sure that the getStatus API itself can never be disabled. It should always be possible to check on a node's status.

Resolved in https://github.com/hive-engine/hivesmartcontracts/pull/16/commits/7c2b045e47c6ccb60b92c2070ff185e5563a1486.
While the operator can still place it in the blocklist, the block of getStatus won't be respected.


Address readability in the Abstract

Everyone, I'm really excited about this project. I've been thinking hard about these issues and find so much of value here. I hope to become a contributor, and... well, diving in.  My way has tended toward the blunt and matter-of-fact, so let me say... great work!  And as I make suggestions, please let me know if my words are ever harsh.
The Abstract is going to be the first thing people read, in many or most cases. I'm finding it to be super dense, full of ideas, and hard to read. An abstract should be a high-level overview, present conclusions, and invite the reader to study & explore the rest of the material. I'd propose rewriting it to be less dense, more inspiring, clearer. Maybe break up the paragraph. Avoid too much enumeration, too many buzzwords. Write it for a non-techie and make it easy to grasp the exciting part.

Hey @stenzer .. thanks, great thing about GitHub and opensource .. we are open to suggestions. Please submit a proposed re-write to make more inspiring and clear.


When I have trained a model in the PaddleDetection, how to use the trained model to finetune?

When I have trained a model in the PaddleDetection, how to use the trained model to finetune?

We have no experience with Paddle, but you can try converting Paddle checkpoints to the pytorch format.

Get numpy array of the weights from the checkpoint of paddledetection
Find the corresponding architecture in mmdet. If there is not a corresponding one, implement it. (eg, paddle often use resnet_vd, but mmdet does not support resnet_vd backbone right now.)
Do variable names mapping. PaddleDetection has a diffrenent nameing policy. eg:

    'conv1_1_weights': 'conv1.weight',
    'bnv1_1': 'bn1',
    'conv1_2_weights': 'conv2.weight',
    'bnv1_2': 'bn2',

@PanJianning vd has been supported in https://github.com/open-mmlab/mmdetection/pull/2252, and will be released in v2.0.


Persist URL hash fragments across a login redirect

Can't try this out right now, concept looks good. Does this remove the returnto part when the redirect has happened? So there isn't an ever increasing list of URL parameters...
What happens if the user is logged out and then forced to log in (maybe a quick server restart) mid browsing? Is a long argument list, such as a diff view with many custom filters, properly restored?

@whisperity Yes, for both of your questions:

The returnto part will be removed after login.
If the session of the user has been expired he will be forced to log in again but the query parameter will be properly restored.


Print dependcygraph when circular dependencies occurs

Print a simple dependency graph : (--verbose)
From unknown source [error] Circular dependencies among remaining modules: [test/base/CLIParserTest, sdk, ooc-math, ooc-collections, ooc-base]
Dependency Graph:
test/base/CLIParserTest : 3 [
	test/base/CLIParserTest => sdk
	test/base/CLIParserTest => ooc-collections
	test/base/CLIParserTest => ooc-base
]
sdk : 1 [
	sdk => ooc-math
]
ooc-math : 1 [
	ooc-math => sdk
]
ooc-collections : 1 [
	ooc-collections => sdk
]
ooc-base : 2 [
	ooc-base => sdk
	ooc-base => ooc-collections
]

[FAIL]

Works great:

Merged, thanks!
(Although I don't think enough snowmen in there, but it's a matter of personal taste).

(Btw, test repo: https://github.com/fasterthanlime/issue850 )


:sparkles: add cache for source manifestwork client

Summary
Related issue(s)
https://github.com/open-cluster-management-io/ocm/issues/476

/assign @qiujian16

Thanks @skeeey & @qiujian16 for working on this.
I reviewed the PR, it touches some points we discussed in our today call and I've summarised the challenges in this discussion issue https://github.com/open-cluster-management-io/ocm/issues/483

You need to update this document - https://github.com/open-cluster-management-io/sdk-go/blob/main/pkg/cloudevents/README.md

You need to update this document - https://github.com/open-cluster-management-io/sdk-go/blob/main/pkg/cloudevents/README.md

yeah, I will udpate the readme after this pr is done

/hold

/unhold

/approve
/lgtm


[FLINK-35430][cdc-connector][kafka] Passed the zoneId to JsonSerializationSchema.

Refer to FLINK-35430
Passed the zoneId to JsonSerializationSchema.

Hi @joyCurry30, could you please create another PR to backport this to release-3.1? We're preparing for the next patch release, and it would be nice to include this bugfix.

Hi @joyCurry30, could you please create another PR to backport this to release-3.1? We're preparing for the next patch release, and it would be nice to include this bugfix.

https://github.com/apache/flink-cdc/pull/3377
Hi, cc plz. @yuxiqian


Added Materia color scheme (https://github.com/Defman21/base16-materia)

Merged in https://github.com/auduchinok/base16-builder along with other pull requests.
I'm going to maintain this project for some time, so this pull request can be closed.

Thanks, would you mind maintaining this in your own repository like https://github.com/chriskempson/base16-default-scheme. The new base16 builders can then pull this scheme from your repository and inherit any updates/tweaks you make in the future.

https://github.com/Defman21/base16-materia

Thanks for the link. Any chance you could rename the yml file to yaml? Also don't forget to add yourself here: https://github.com/chriskempson/base16/blob/master/README.md#scheme-repositories 😄

Okay!


google_benchmark_vendor: 0.1.2-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository google_benchmark_vendor to 0.1.2-1:

upstream repository: https://github.com/ament/google_benchmark_vendor.git
release repository: https://github.com/ros2-gbp/google_benchmark_vendor-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.1.1-2

google_benchmark_vendor
* Resolve Clang compilation issue with benchmark 1.5.3 (#28 <https://github.com/ament/google_benchmark_vendor/issues/28>)
* Contributors: Adam Seiwert

Holding for an upcoming Humble sync

I'm going to merge this, as it's needed for the Humble patch.


Add obs-status.rb helper

This is part of the story to trigger Gating for OBS release requests automatically.

The code itself should work as it is but I have to add documentation and have to test how it works if uses by a trigger CI job which I also have to add.

Why don't you instead change https://github.com/openSUSE/osc to add the missing functionality ? It already has some of what this does.

@JanZerebecki There is no missing functionality. This is a wrapper around it, osc is a general tool while this is Cloud specific e.g. the comments and the corresponding status.

It is not a wrapper around it, it never calls or loads osc.

this is Cloud specific

I agree that storing a status as a comment containing a specific string and filtering by that is a cloud specific hack. But listing requests, printing them, making requests to obs is not.
It is in ruby and most other osc related code is in python, thus it is harder to upstream common code and harder to reuse common code. (That would be the same if this were written in python and osc in ruby. And I'm glad that osc is not written in bash.) (See e.g. https://github.com/SUSE-Cloud/automation/blob/22f475984e357633074522c50d55b69d585a0ea1/jenkins/ci.opensuse.org/openstack-prepare-staging.py .) Is it in some way beneficial to add ruby to this and define common data structures in two languages?

I think, osc has a mechanism to allow plugins - maybe that also works if those are written in non-python? might integrate nicer.

The only plugins I know of are in python. I checked with zypper info --requires osc-plugin* while having the repo openSUSE:Tools enabled.

Here is the situation: That is not perfect is clear and there are many ways to make it better but we need something functional now and there is nobody available right now who can spend time on this. I created this during some spare time I had just to make some progress on this topic and ruby is the language where I'm more comfortable. Beside this I assume that this is only a temporary solution till the OBS team implemented a common way of handling this. So I don't want to spend much time on this because it will be anyway obsoleted sooner or later.

we need something functional now

Is this requirement new?
Or was there some spec or ticket about this?

Beside this I assume that this is only a temporary solution till the OBS team implemented a common way of handling this.

Do you have a link handy that shows they plan to take this on?

we need something functional now

Is this requirement new?
Or was there some spec or ticket about this?

This is partly covert by the Crowbar CI spec. The Goal is to automate everything to free up resources.

Beside this I assume that this is only a temporary solution till the OBS team implemented a common way of handling this.

Do you have a link handy that shows they plan to take this on?

Unfortunately there is nothing puplic I can share but there was a slide deck where this was mentioned.


this isn't really an issue

hey, super sorry to contact you like this.  i didn't want to open an issue because this isn't an issue.  how did you reverse the websockets api?  i'm trying to recreate this in another language but i'm having a heck of a time even testing the alarm websocket out.
i'm using this client just for ease:
https://chrome.google.com/webstore/detail/browser-websocket-client/mdmlhchldhfnfnkfmljgeinlffmdgkjo/related?hl=en
should i be able to connect to that URL from anywhere or do they have origination restrictions? time limit? how long is the auth code valid from?
wss://${connections_response.server}/?authcode=${connections_response.authCode}
i can telnet to port 443 on the server i'm getting back but if i try to connect to it with a websocket client of any kind the server immediately closes the connection.  any hints on how you figured this out? if you want to reach out through another medium feel free.  i'm on gmail for example.

@codahq I'm happy to discuss your setup here.  Hopefully it can be useful if someone else is trying to implement another language down the road.  To answer you first question, the initial discovery on the websocket api was done by @joeyberkovitz who submitted a PR to doorbot.  The most important take-away from that PR is that ring is using socket.io.  He also had some good examples of how to request and interpret the different data types through the websocket connection.  Hopefully there is a socket.io client for whichever language you are using, because that will make you life a whole lot easier.  Using the socket.io client, I had no issues getting connected.  I didn't run into any issues with request origin or timing with the auth token (although I was using it from code, so it there wasn't much delay).  Once you are connected, the best way to learn how the interactions work is to pop open dev tools and visit the ring web app to observe the websocket interactions.  Let me know if you run into any other roadblocks, and good luck!

Very interesting.  It appears that I was not on current version of the web app until today.  This will make some parts of this trivial now that I can see this traffic going through a browser.  Until today I was trying MITM with the app.  This was okay until the WS calls where they are potentially certificate pinning.
I think there must also be a flag on the account not to allow WS connections on that URL until the web app supports alarm functionality.  I'm doing the exact thing today that I did last week that I'm doing today but now it works.  Hopefully this helps somebody as well.


Add --legacy flag, for compatibility with Xcode 16

Some related discussion can be seen here - https://github.com/fastlane/fastlane/issues/22132

@jozefizso - this is my first attempt at Typescript - I welcome any testing pointers.

Does anyone what the non-deprecated alternative is for this command?
xcrun xcresulttool -h
OVERVIEW: Xcode Result Bundle Tool (version 23028).

USAGE: xcresulttool <subcommand>

OPTIONS:
  --version               Show the version.
  -h, --help              Show help information.

SUBCOMMANDS:
  get                     Get Result Bundle contents.
  export                  Export Result Bundle contents.
  graph                   Print Result Bundle Object Graph. This subcommand is deprecated and will be removed in a future release, consider using
                          `xcresulttool get test-report` instead.
  metadata                Result Bundle Metadata.
  formatDescription       Result Bundle Format Description. This subcommand is deprecated and will be removed in a future release.
  merge                   Merge Result Bundles.

graph and formatDescription have been deprecated. For graph, it says: consider using xcresulttool get test-report instead.
xcrun xcresulttool get -h
OVERVIEW: Get Result Bundle contents.

USAGE: xcresulttool get <subcommand>

OPTIONS:
  --version               Show the version.
  -h, --help              Show help information.

SUBCOMMANDS:
  object (default)        Get Result Bundle Object. This subcommand is deprecated and will be removed in a future release, consider using
                          `xcresulttool get test-report` instead.
  test-results            Get a high level description of what was tested, the destinations on which tests ran, and other metadata along with all
                          failures reported.
  build-results           Get a high level description of build action, the run destination for which the build was performed, and other metadata
                          along with warnings and issues.
  log                     Get build, action, or console log from Result Bundle.

  See 'xcresulttool help get <subcommand>' for detailed help.

For get it's odd that a deprecated subcommand object is the default.
It also tells you to "consider xcresulttool get test-report instead."
However, test-report is not listed as subcommand.... but it does exist.
xcrun xcresulttool get test-report -h
OVERVIEW: Get Result Bundle Object. This subcommand is deprecated and will be removed in a future release, consider using `xcresulttool get
test-report` instead.

USAGE: xcresulttool get object [--legacy] --path <path> [--id <id>] [--version <version>] [--format <format>]

OPTIONS:
  --legacy                Required to enable legacy API.
  --path <path>           The result bundle path.
  --id <id>               The ID of the object [optional, assumes rootID if not specified].
  --version <version>     For incomplete result bundles (lacking Info.plist), specify version explicitly [optional, assumes latest version if not
                          specified].
  --format <format>       The output format. (values: json, raw; default: raw)
  --version               Show the version.
  -h, --help              Show help information.

This command is deprecated as well.... it appears to be an alias for get object, which is also mentioned in the usage?
All in all it's super confusing and unclear what the undeprecated way is.

Relevant topic: https://forums.developer.apple.com/forums/thread/763050
Looks like xcrun xcresulttool get test-results summary --path X.xcresult is a close alternative, at least for our use case it contains the information we need.

Hello, we are sorry we missed this pull request. I will take a look at it as we are at the transition to Xcode 16.

Hi @gemmakbarlow, I have some feedback for the PR:

It is always better to create PR from a topic/feature branch and not the master branch. This would allow us to make small adjustments to the code before merging if it was on a topic branch.
I would prefer script to continue working if it cannot determine the Xcode version instead of bailing out
The code formatting and styling should match other source code.

Hi @gemmakbarlow, I have some feedback for the PR:

It is always better to create PR from a topic/feature branch and not the master branch. This would allow us to make small adjustments to the code before merging if it was on a topic branch.
I would prefer script to continue working if it cannot determine the Xcode version instead of bailing out
The code formatting and styling should match other source code.

All very valid, thank-you for the feedback. I hope to get some time to address those comments later in the week.
If you need to proceed with another approach in the interim, please go ahead, as I'm a little swamped at the moment.

I am closing this PR as the fix was implemented in #10


how can openblas supprt andoid x86_64

The script near the middle of the wiki page on building for Android has a section with the settings for x86_64, please see
https://github.com/xianyi/OpenBLAS/wiki/How-to-build-OpenBLAS-for-Android#alternative-script-which-was-tested-on-osx-with-ndk2136528147

The script near the middle of the wiki page on building for Android has a section with the settings for x86_64, please see
https://github.com/xianyi/OpenBLAS/wiki/How-to-build-OpenBLAS-for-Android#alternative-script-which-was-tested-on-osx-with-ndk2136528147

thank you , I will try it.


Allow dropTableIfExists to accept multiple table names as arguments

When doing migrations I will often find myself writing 10+ dropTableIfExists() calls in sequence - (usually in the .down()) -- it would be great if there was either a dropTablesIfExists method or if dropTableIfExists was extended such that an arbitrary number of table names as arguments, or an array of table names, could be passed.
Examples:
// Multiple table names as arguments
trx.schema.dropTableIfExists("alpha", "beta", "gamma");

// Array of table names as single argument
trx.schema.dropTableIfExists(["alpha", "beta", "gamma"]);

Both of which would produce
drop table if exists "alpha"
drop table if exists "beta"
drop table if exists "gamma"

I don't see this feature valuable enough that I would like to have it in the codebase. For consistency we should review also other similar methods and implement it to all of them, write tests, update docs etc.
for me its enough to do this:
    trx.schema
        .dropTableIfExists("alpha")
        .dropTableIfExists("beta")
        .dropTableIfExists("gamma");

@elhigu That's fair, maybe we can keep the idea open for a while to see if any other users are interested. It would be fairly trivial to implement. Also there are several places in the codebase that would could be cleaned up if we implemented this, e.g. https://github.com/tgriesser/knex/blob/master/test/integration/schema/index.js#L24


diagnostic: remove duplicate checks.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

Remove duplicate checks added in #896.

may have to disable that "Required" thing ... I guess something in test-bot is out of order so it's still using the old bits?

@ilovezfs Nope, valid test failure 😀

@ilovezfs Nope, valid test failure 😀

ah because you didn't update the tests that go with the diagnostic changes... I see

:green_heart:

@MikeMcQuaid thanks!


Replace fetch.dart with new http library and roll mojo

 Issue by collinjackson
Wednesday Nov 04, 2015 at 18:41 GMT
Originally opened as https://github.com/flutter/engine/pull/1943

None

collinjackson included the following code: https://github.com/flutter/engine/pull/1943/commits

 Comment by abarth
Wednesday Nov 04, 2015 at 18:52 GMT

LGTM

 Comment by Hixie
Thursday Nov 05, 2015 at 03:05 GMT

Travis is unhappy.

 Comment by abarth
Thursday Nov 05, 2015 at 20:24 GMT

LGTM