feat!: switch session interface to have .data dependent on .status

This PR improves the typing of the useSession composable, either:

session.value.status === 'authenticated' and session.value.data will be available
session.value.status !== 'authenticated' and session.value.data will not be available, resulting in a TS error

This will force users of nuxt-auth to always check if the use is authenticated before relying on .data. Non TS users will not be impacted.

@valiafetisov my main interest would be: How do you like this design choice? I feel that it would be a great addition for people using TS and will not impact people using JS + it's a change we can still make in this stage of module-popularity. A possible downside is, that this deviates from NextAuth.js data-models, see here

I like the addition, but why type enforcing is not possible with the previous structure? I can imagine that { status, data } = await useSession() can be related in the same way as { status, data } = session types.

A possible downside is, that this deviates from NextAuth.js data-models, see here

Thanks for the context I was missing. Maybe then it's not such a good idea after all.

Well, I couldn't come up with a way to make the two related as independent variables. And it turns out that TS in general cannot keep track of refinements (eg, what happens when you check for one variant of a discriminate union): https://kyleshevlin.com/discriminated-unions-and-destructuring-in-typescript

Wheb you're saying maybe, do you have a preference going any one way? (:

I don't think it's possible to do it this way. I think the issue is if you destructure into multiple variables, Typescript is not able to link them anymore.
It should look like that if you're trying to achieve a discriminated union:
const session = await useSession();
if (session.status.value === 'authenticated') {
  session.data.value;
  // ^? -> Session
}
session.data.value;
// ^? -> Session | null

This, or actually a stricter version of this, is what's implemented in this PR at the moment. Atm. this code produces the following:
const { session } = await useSession({ required: false })
if (session.value.status === 'authenticated') {
  session.value.data;
  // ^? -> Session
} else {
  session.value.data;
  // ^? -> TS Error: `data` does not exist on session.value
}

I think the issue is if you destructure into multiple variables, Typescript is not able to link them anymore.

I thought the same, but as per https://stackoverflow.com/a/69033664 there is a way to make it work now (I've not tried it yet)

And it turns out that TS in general cannot keep track of refinements

...after destructuring. Cause in general it can

This, or actually a stricter version of this, is what's implemented in this PR at the moment

I thought that the comment above suggests that in order to enforce types you not only need to set correct union types, but also use variables without destructuring. I.e. update our readme examples:
- const { status, data, signIn, signOut } = await useSession()
+ const session = await useSession()

Or wait till typescript will support it.

Wheb you're saying maybe, do you have a preference going any one way? (:

I'm not sure it's a good idea to introduce breaking changes just to overcome some ts limitation (that might be fixed in the future and can already be overcome by using different syntax). Otherwise, someone might still write const { status, data } = session.value and loose type enforcing again.

closing this for now as outdated, we can revisit it later if we want to (:


New rule: prevent unsafe assignment of fp-ts types

export const foo: IO<string> = () => "hello";

// These assignments compile because the types are compatible, but the semantics are such that doing so is almost certainly an error.
export const bar: Lazy<string> = foo;
export const baz: () => string = foo;

We should also catch when a Promise is being assigned to an IO like IO.of(Axios.get(...)), or any async effect being assigned to an IO for that matter.

That latter point is covered by total-functions/no-nested-fp-ts-effects


[DEBUG] Test PR

A PR for debugging table rendering. To be cancelled / no merging.

Thanks for setting this up, this will help greatly with debugging.

Closing as this is now fixed in LWT v1.2.0 🥳


Connecting with Gmail

Hi,
I just can't get my code to connect to gmail. My go code is:
  m := mail.NewMessage()
  m.SetHeader("From", from)
  m.SetHeader("To", toList)
  m.SetHeader("Subject", subject)
  m.SetBody("text/html", msg)
  m.AddAlternative("text/plain", msgText)
  iport, _ := strconv.Atoi(port)
 	d := mail.NewDialer(host, iport, from, password)
	d.TLSConfig = &tls.Config{InsecureSkipVerify: true}
  if err := d.DialAndSend(m); err != nil {
    b.err = err.Error();
    return b.err
  }

  return "Success"

but it constantly comes back with username or password is incorrect. I've checked them and they are right and work with other clients. Not sure what else to try.

Never mind. Seeing it here I immediately saw my mistake.


Publish multiple events to a topic in bulk mode using a file.

Which version of the app are you running? vv0.4.0 [521ba0c] (https://github.com/provectus/kafka-ui/commit/521ba0c)

Is your proposal related to a problem? No

For testing purpose, we need to publish multiple events to kafka topic. Existing feature to produce one by one is not convenient and time consuming.
Describe the solution you'd like
With an option to upload a file with key value pair and multiple events would make such scenarios much simpler, faster and convenient.

With an option to upload a file, all the events from the file to be published in kafka topic.
Describe alternatives you've considered
manually published one-by-one events from ui.

n/a
Additional context
n/a

n/a

Hey, thanks for reaching out.
We already have something like this planned (#1344). Would it be enough for you to make the window stay open with the same key/value contents or would uploading a file be way more convenient in your case?

Thanks for update. Yes, uploading a file with key/value pair will be more convenient and easy to manage. e.g.
FileName: file1.json
{"key": "1111", "value": {"field1":"1111","field2":"dadada","field3":"9991","field4":null,"field5":null}}
{"key": "1112", "value": {"field1":"1112","field2":"tatata","field3":"9991","field4":null,"field5":null}}
{"key": "1113", "value": {"field1":"1113","field2":"rarara","field3":"9991","field4":null,"field5":true}}
{"key": "1114", "value": {"field1":"1114","field2":"kakaka","field3":"9992","field4":null,"field5":null}}
{"key": "1115", "value": {"field1":"1115","field2":"papapa","field3":"9992","field4":null,"field5":false}}


The recent 𝗹𝗲𝗮𝗸𝗲𝗱 of 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's original video shocked the online world and quickly became a hot topic among social media users. In no time, the video went 𝚟𝚒𝚛𝚊𝚕, attracting attention from people across platforms like Twitter, Instagram, and TikTok. 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞, previously known as a young and talented digital creator, gained even more popularity thanks to this unexpected video. In the 𝗹𝗲𝗮𝗸𝗲𝗱 footage, 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞 showcased her talent and creativity, which ultimately captured the attention of a wider audience.
The 𝚟𝚒𝚛𝚊𝚕 phenomenon sparked a range of reactions from internet users, from surprise to admiration for 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's intelligence and unique approach to her craft. Although the video was shared without 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's consent, the incident inadvertently introduced her to a broader audience. Many praised 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞 for her ability to create engaging and inspirational content.

The recent 𝗹𝗲𝗮𝗸𝗲𝗱 of 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's original video shocked the online world and quickly became a hot topic among social media users. In no time, the video went 𝚟𝚒𝚛𝚊𝚕, attracting attention from people across platforms like Twitter, Instagram, and TikTok. 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞, previously known as a young and talented digital creator, gained even more popularity thanks to this unexpected video. In the 𝗹𝗲𝗮𝗸𝗲𝗱 footage, 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞 showcased her talent and creativity, which ultimately captured the attention of a wider audience.
The 𝚟𝚒𝚛𝚊𝚕 phenomenon sparked a range of reactions from internet users, from surprise to admiration for 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's intelligence and unique approach to her craft. Although the video was shared without 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's consent, the incident inadvertently introduced her to a broader audience. Many praised 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞 for her ability to create engaging and inspirational content.

The recent 𝗹𝗲𝗮𝗸𝗲𝗱 of 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's original video shocked the online world and quickly became a hot topic among social media users. In no time, the video went 𝚟𝚒𝚛𝚊𝚕, attracting attention from people across platforms like Twitter, Instagram, and TikTok. 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞, previously known as a young and talented digital creator, gained even more popularity thanks to this unexpected video. In the 𝗹𝗲𝗮𝗸𝗲𝗱 footage, 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞 showcased her talent and creativity, which ultimately captured the attention of a wider audience.
The 𝚟𝚒𝚛𝚊𝚕 phenomenon sparked a range of reactions from internet users, from surprise to admiration for 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's intelligence and unique approach to her craft. Although the video was shared without 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's consent, the incident inadvertently introduced her to a broader audience. Many praised 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞 for her ability to create engaging and inspirational content.

The recent 𝗹𝗲𝗮𝗸𝗲𝗱 of 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's original video shocked the online world and quickly became a hot topic among social media users. In no time, the video went 𝚟𝚒𝚛𝚊𝚕, attracting attention from people across platforms like Twitter, Instagram, and TikTok. 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞, previously known as a young and talented digital creator, gained even more popularity thanks to this unexpected video. In the 𝗹𝗲𝗮𝗸𝗲𝗱 footage, 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞 showcased her talent and creativity, which ultimately captured the attention of a wider audience.
The 𝚟𝚒𝚛𝚊𝚕 phenomenon sparked a range of reactions from internet users, from surprise to admiration for 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's intelligence and unique approach to her craft. Although the video was shared without 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞's consent, the incident inadvertently introduced her to a broader audience. Many praised 𝐀𝐥𝐢𝐬𝐨𝐧 𝐁𝐫𝐢𝐞 for her ability to create engaging and inspirational content.


How to load ui-tinymce programmatically

I want load ui-tinymce in div programmatically when user click on this div. It means I want delay loading until user interact with element. Can I do that?

Are you trying to use tinymce in inline mode with the div being the container to invoke tinymce on?
If so why not just have it loaded when the page loads?
You could instantiate a tinymce directive on click though, by putting an ng-if on the directive with a variable that gets set to true when the correct location on the page is clicked.
Going to close this issue as it is not a bug or feature request, but feel free to respond back and I can help you more here if needed.


Improve HessianTracer performance

Couple key-type of HessianTracer dict to set-type
Move sorting kwarg into inner constructor of SortedVector
When merging HessianTracers, make a set out of the keys outside of the inner loop. This should avoid repeated sorting for unions with SortedVector.

Codecov Report
Attention: Patch coverage is 76.66667% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 73.61%. Comparing base (e3dca46) to head (acb22cb).

Files
Patch %
Lines

src/tracers.jl
76.00%
6 Missing :warning:

src/sortedvector.jl
80.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
- Coverage   73.98%   73.61%   -0.37%
==========================================
  Files           9        9
  Lines         319      326       +7
==========================================
+ Hits          236      240       +4
- Misses         83       86       +3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Merging now and leaving conversations unresolved for a follow-up PR.

closed the remaining conversations because I recorded the concerns in https://github.com/adrhill/SparseConnectivityTracer.jl/issues/40


Perelson transformation overflow issue

i've encountered an issue with transforming the measurement data of Perelson_Science1996. There in the measurement file measurementData_Perelson_Science1996.tsv The data is indicated as being log10 transformed, however, there is a math range issue with pow(10, 3208000).
it would be good to check this

observableTransformation=log10 is meant to indicate that we will take log10 of model and data in the objective function. We will try to clarify that in the documentation.

When #154 is resolved, we need to re-check all models.

Measurement file should be fine. See comment above. Now more clearly formulated in the PEtab documentation. Closing.


Multiple Configuration / Clients

Now is possible to load multiple configurations, this is essential in order to send push notifications to different apps.
Multiple clients are instantiated based on different configuration provided.
FCM.ID is a convenience struct for identifying a specific client with a specific configuration.

Hey @apemaia99 thanks for that great contribution! Sorry that it took so long to review it. It would be great if you also could update readme for these changes and I'd love to tag it as v3.0.0

Hey @MihaelIsaev, I'm happy for that, but unfortunately I found out a bug that I need to address before proceeding.
Since my implementation allows multiple clients at same time, each client tries to find the same KEY in the environment this can be very dangerous, I need to find another solution. Personally I don't use the env keys as configuration method and that's why I didn't catch it early in the development.

@MihaelIsaev I've found a solution for getting right keys based on FCM.ID, I try to explain it with an example.
let's take as example email env key "FCM_EMAIL", in this case:

If used on a specific instance (A as example) the key lookup will be: "FCM_EMAIL_A"
In case of default the key lookup will be:                                                "FCM_EMAIL"

this solution seems to not break current APIs.
Let me know your opinion and if is ok I'm going to update the readme.


Create route groups

Implements proposal at #1262.

I'm having a deprecation error (https://travis-ci.org/silexphp/Silex/jobs/83621808) with PHP 5.6 + SYMFONY_DEPS_VERSION=2.8:
Passing an instance of Symfony\Component\Validator\ValidatorInterface as argument to the Symfony\Component\Form\Extension\Validator\EventListener\ValidationListener::__construct method is deprecated since version 2.8 and will be removed in 3.0. Use an implementation of Symfony\Component\Validator\Validator\ValidatorInterface instead: 6x

    6x in ValidatorServiceProviderTest::testValidatorConstraint from Silex\Tests\Provider

@fabpot @igorw do you know the possible cause for this?

This looks weird to me. The service provider should be using the new validator implementation

@stof Do you think it can be some sort of cache inside Travis?

@fabpot, some comment about what could it be?

@stof some ideia about it?

@talyssonoc I suggest you to rebase your branch, which will retrigger a new Travis build. It may have been fixed by another change


Bump workspace to owo-colors 4.0

Requires adding a path dependency in color-eyre on color-spantrace so that the versions agree. Fortunately we can do that now!

Windows tests are failing.


strip speaker first and last names to remove any leading spaces

This is a permanent fix to issue shown in #9 .

Just realized strip on family_name was not needed :stuck_out_tongue_closed_eyes:


Add Open API conversion support

Purpose
In addition to default swagger support we will be providing OAS 3.0.0 based editor support from next release onward. So at anytime when we move ahead with open API based swagger editor, we should be able to represent ballerina with OAS 3.0.0. This PR is having that conversion logic and test cases.
Goals
Represent ballerina with OAS 3.0.0
Approach
https://docs.google.com/document/d/1fNeSq_Oha_AnHx9I2c8xtWiBvl93JJAs_awAo91kS-k/edit#heading=h.hu3u6zwzi49p
Test environment
java version "1.8.0_144"

Last commit message doesn't meet the standards. Commit message subject should start with a capital letter. https://chris.beams.io/posts/git-commit/


Fixed colors for "Swift for iOS" code.

I tried to use your tool to fix my problem with RGB color, but it's not work correctly. RGB values have a range of 0-1. The numbers that are given are always divided by 255.
https://stackoverflow.com/questions/39557590/graphics-uicolor-created-with-component-values-far-outside-the-expected-range
Also I don't know about other Codes and I can't to check them.
Thank you :)

@mazeina thx!


[Feature request] fail when import cannot be resolved

Currently when an import cannot be resolved an error is logged but compilation does not fail. This makes it hard to spot errors during development, as code will be broken but it's not very apparent why.
Especially if you are working in the browser, and not looking at the terminal, the message: Failed to resolve module specifier "foo". Relative references must start with either "/", "./", or "../". is not very helpful.
I suggest adding an option to allow failing when a module could not be resolved.
I can work on a PR for this.

That sounds good.  Could you make it do a babel exception like https://github.com/jamiebuilds/babel-handbook/blob/master/translations/en/plugin-handbook.md#-throwing-a-syntax-error (note I have not actually tested example code).

Will do. Do you want it to be configurable? And if so what should be the default behavior?

Throwing by default would be semver-major but it feels right.  I heard something at some point about browsers eventually getting 'built-in' modules, I'm unsure how that will work or if they'll be in a namespace.  Lets make it configurable and maintain current behavior by default for now.
I can do a semver-major after it's determined if anything can be done with bower_components?


New package: CommandComposer v0.1.0

Registering package: CommandComposer
Repository: https://github.com/singularitti/CommandComposer.jl
Created by: @github-actions[bot]
Version: v0.1.0
Commit: 773e828753e73ce1456630884ead40b4efe03d00
Reviewed by: @github-actions[bot]
Reference: https://github.com/singularitti/CommandComposer.jl/commit/773e828753e73ce1456630884ead40b4efe03d00#commitcomment-119850968

I am going to rename this repo as ComposableCommands.jl.

Hi @giordano, sorry. Could you please merge this for me? https://github.com/JuliaRegistries/General/pull/86523

That's slated to be automatically merged when the standard 3-day wait finishes.


Minimize Maximize Issue on Windows Terminal

I cannot able to see the minimize and all the stuff and I cannot able to exit app with the "X" Button
Please Help Me Out

Did you turn on "Focus Mode"?

linking #12542, but there's a better thread somewhere...


Sources: Add a source > Obtain login (OCP)

Describe the bug
Edits to the screen where you get your OCP login to adhere to brand standards for OpenShift, etc.
Expected behavior
Revised text:
Obtain your login credentials
To gather OpenShift Container Platform metering data, Cost Management requires the login token for the Operator Metering service account.

Log in to the OpenShift Container Platform cluster with an account that has access to the Operator Metering namespace.
To obtain your login token, run:
[command...]
To maintain security, store the token file on a file system with limited access.

Also, please add a popover with the text shown in the linked screenshot.
Screenshots
https://redhat.invisionapp.com/share/XJR3A3996KR#/screens/356616236

@dayleparker where the "learn more" link in the popover should take the user?


Vague error message "illegal seek" during simple ZST read/write

Repro is with zq commit 890be30.
I happened to try a simple operation of creating ZST output and reading it back, and it failed.
$ zq -version
Version: v0.29.0-104-g890be304

$ echo '{"foo": "bar"}' | zq -f zst - | zq -z -i zst -
stdio:///stdin: seek /dev/stdin: illegal seek

Closing this since the error message has since been improved...
% echo '{"foo": "bar"}' | zq -f zst - | zq -z -i zst -
stdio:stdin: zst must be used with a seekable input


Error reassigning partitions for topic __strimzi_canary while starting Strimzi Canary

Hello Team,
We are facing an issue while starting Strimzi Canary in our Kafka cluster. Here is the log after enabling the Verbosity_Level to 1.
I0314 09:46:12.466438       1 main.go:48] Starting Strimzi canary tool [0.2.0] with config: {BootstrapServers:[kafka-kafka-bootstrap.kafka:9093], BootstrapBackoffMaxAttempts:10, BootstrapBackoffScale:5000, Topic:__strimzi_canary, TopicConfig:map[], ReconcileInterval:30000 ms, ClientID:strimzi-canary-client, ConsumerGroupID:strimzi-canary-group, ProducerLatencyBuckets:[100 200<PHONE_NUMBER>], EndToEndLatencyBuckets:[100 200<PHONE_NUMBER>], ExpectedClusterSize:3, KafkaVersion:3.0.0,SaramaLogEnabled:true, VerbosityLogLevel:1, TLSEnabled:true, TLSCACert:[CA cert], TLSClientCert:[Client cert], TLSClientKey:[Client key], TLSInsecureSkipVerify:false,SASLMechanism:, SASLUser:, SASLPassword:, ConnectionCheckInterval:120000 ms, ConnectionCheckLatencyBuckets:[100 200<PHONE_NUMBER>], StatusCheckInterval:30000 ms, StatusTimeWindow:300000 ms}
I0314 09:46:12.466567       1 http_server.go:41] Starting HTTP server
I0314 09:46:12.466988       1 security.go:73] TLS_CA_CERT loaded from configuration directly
I0314 09:46:12.467189       1 security.go:73] TLS_CLIENT_CERT loaded from configuration directly
I0314 09:46:12.467215       1 security.go:73] TLS_CLIENT_KEY loaded from configuration directly
[Sarama] 2022/03/14 09:46:12 [Initializing new client]
[Sarama] 2022/03/14 09:46:12 client/metadata fetching metadata for all topics from broker [kafka-kafka-bootstrap.kafka:9093]
[Sarama] 2022/03/14 09:46:12 Connected to broker at [kafka-kafka-bootstrap.kafka:9093] (unregistered)
[Sarama] 2022/03/14 09:46:12 client/brokers registered new broker #[0 %!d(string=kafka-kafka-0.kafka-kafka-brokers.kafka.svc:9093)] at %!s(MISSING)
[Sarama] 2022/03/14 09:46:12 client/brokers registered new broker #[2 %!d(string=kafka-kafka-2.kafka-kafka-brokers.kafka.svc:9093)] at %!s(MISSING)
[Sarama] 2022/03/14 09:46:12 client/brokers registered new broker #[1 %!d(string=kafka-kafka-1.kafka-kafka-brokers.kafka.svc:9093)] at %!s(MISSING)
[Sarama] 2022/03/14 09:46:12 [Successfully initialized new client]
[Sarama] 2022/03/14 09:46:12 [Initializing new client]
I0314 09:46:12.791021       1 canary_manager.go:58] Starting canary manager
I0314 09:46:12.791045       1 connection_check.go:116] Creating Sarama cluster admin
[Sarama] 2022/03/14 09:46:12 client/metadata fetching metadata for all topics from broker [kafka-kafka-bootstrap.kafka:9093]
[Sarama] 2022/03/14 09:46:12 Connected to broker at [kafka-kafka-bootstrap.kafka:9093] (unregistered)
[Sarama] 2022/03/14 09:46:12 client/brokers registered new broker #[0 %!d(string=kafka-kafka-0.kafka-kafka-brokers.kafka.svc:9093)] at %!s(MISSING)
[Sarama] 2022/03/14 09:46:12 client/brokers registered new broker #[2 %!d(string=kafka-kafka-2.kafka-kafka-brokers.kafka.svc:9093)] at %!s(MISSING)
[Sarama] 2022/03/14 09:46:12 client/brokers registered new broker #[1 %!d(string=kafka-kafka-1.kafka-kafka-brokers.kafka.svc:9093)] at %!s(MISSING)
[Sarama] 2022/03/14 09:46:12 [Successfully initialized new client]
[Sarama] 2022/03/14 09:46:12 Connected to broker at [kafka-kafka-0.kafka-kafka-brokers.kafka.svc:9093 %!s(int32=0)] (registered as #%!d(MISSING))
[Sarama] 2022/03/14 09:46:12 Connected to broker at [kafka-kafka-0.kafka-kafka-brokers.kafka.svc:9093 %!s(int32=0)] (registered as #%!d(MISSING))
[Sarama] 2022/03/14 09:46:12 Closed connection to broker [kafka-kafka-0.kafka-kafka-brokers.kafka.svc:9093]
I0314 09:46:12.905506       1 connection_check.go:157] Connected to broker 0 in 17 ms
[Sarama] 2022/03/14 09:46:12 Connected to broker at [kafka-kafka-2.kafka-kafka-brokers.kafka.svc:9093 %!s(int32=2)] (registered as #%!d(MISSING))
I0314 09:46:12.919953       1 connection_check.go:157] Connected to broker 2 in 13 ms
[Sarama] 2022/03/14 09:46:12 Closed connection to broker [kafka-kafka-2.kafka-kafka-brokers.kafka.svc:9093]
[Sarama] 2022/03/14 09:46:12 Connected to broker at [kafka-kafka-1.kafka-kafka-brokers.kafka.svc:9093 %!s(int32=1)] (registered as #%!d(MISSING))
I0314 09:46:12.932029       1 connection_check.go:157] Connected to broker 1 in 12 ms
I0314 09:46:12.932069       1 topic.go:116] Creating Sarama cluster admin
[Sarama] 2022/03/14 09:46:12 Closed connection to broker [kafka-kafka-1.kafka-kafka-brokers.kafka.svc:9093]
[Sarama] 2022/03/14 09:46:12 [Initializing new client]
[Sarama] 2022/03/14 09:46:12 client/metadata fetching metadata for all topics from broker [kafka-kafka-bootstrap.kafka:9093]
[Sarama] 2022/03/14 09:46:12 Connected to broker at [kafka-kafka-bootstrap.kafka:9093] (unregistered)
[Sarama] 2022/03/14 09:46:13 client/brokers registered new broker #[0 %!d(string=kafka-kafka-0.kafka-kafka-brokers.kafka.svc:9093)] at %!s(MISSING)
[Sarama] 2022/03/14 09:46:13 client/brokers registered new broker #[2 %!d(string=kafka-kafka-2.kafka-kafka-brokers.kafka.svc:9093)] at %!s(MISSING)
[Sarama] 2022/03/14 09:46:13 client/brokers registered new broker #[1 %!d(string=kafka-kafka-1.kafka-kafka-brokers.kafka.svc:9093)] at %!s(MISSING)
[Sarama] 2022/03/14 09:46:13 [Successfully initialized new client]
[Sarama] 2022/03/14 09:46:13 Connected to broker at [kafka-kafka-0.kafka-kafka-brokers.kafka.svc:9093 %!s(int32=0)] (registered as #%!d(MISSING))
I0314 09:46:13.380792       1 topic.go:169] The canary topic __strimzi_canary already exists
I0314 09:46:13.380816       1 topic.go:392] Metadata for __strimzi_canary topic
I0314 09:46:13.380821       1 topic.go:394]     {ID:0 Leader:0 Replicas:[0 1 2] Isr:[0 1 2] OfflineReplicas:[]}
I0314 09:46:13.380833       1 topic.go:394]     {ID:1 Leader:1 Replicas:[1 2 0] Isr:[1 2 0] OfflineReplicas:[]}
I0314 09:46:13.380839       1 topic.go:394]     {ID:2 Leader:2 Replicas:[2 0 1] Isr:[2 0 1] OfflineReplicas:[]}
I0314 09:46:13.380847       1 topic.go:188] Going to reassign topic partitions if needed
I0314 09:46:13.380855       1 topic.go:322] Topic __strimzi_canary requested partitions assignments = map[0:[0 1 2] 1:[1 2 0] 2:[2 0 1]], minISR = 2
E0314 09:46:13.381850       1 topic.go:195] Error reassigning partitions for topic __strimzi_canary: failed to reassign partitions for topic:
kafka server: The client is not authorized to send this request type.
[__strimzi_canary-0]: kafka server: The client is not authorized to send this request type.
[__strimzi_canary-1]: kafka server: The client is not authorized to send this request type.
[__strimzi_canary-2]: kafka server: The client is not authorized to send this request type.
F0314 09:46:13.381894       1 canary_manager.go:86] Error starting canary manager: failed to reassign partitions for topic:
kafka server: The client is not authorized to send this request type.
[__strimzi_canary-0]: kafka server: The client is not authorized to send this request type.
[__strimzi_canary-1]: kafka server: The client is not authorized to send this request type.
[__strimzi_canary-2]: kafka server: The client is not authorized to send this request type.
stream closed

We have provided the All permission on __Strimzi_Canary topic and Read permission to the consumer group, here is what our Kafka-user ACL looks like-
...
spec:
  authentication:
    type: tls
  authorization:
    acls:
    - operation: All
      resource:
        name: __strimzi_canary
        patternType: literal
        type: topic
    - operation: Read
      resource:
        name: strimzi-canary-group
        patternType: literal
        type: group
    type: simple
...

Can you please provide some information on what extra permission the Strimzi Canary client requires?

You should check your broker logs -> they should log what exact permissions are missing.

Thanks, @scholzj , We missed adding Alter permission on Cluster and that's why the partition re-assignment request was failing while starting the Canary.


Missing definition for optional_spid

optional_spid is not documented here and i believe it should be.
http://esdbp.createaforum.com/general-9/whats-the-optional_spid-in-sys-dm_exec_plan_attributes/

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

ID: a2a77d63-0ef7-54c8-4f02-59f736298c70
Version Independent ID: 1688ab2b-7894-bbb5-d19d-8d26b6e820aa
Content: sys.dm_exec_plan_attributes (Transact-SQL) - SQL Server
Content Source: docs/relational-databases/system-dynamic-management-views/sys-dm-exec-plan-attributes-transact-sql.md
Product: sql
Technology: system-objects
GitHub Login: @CarlRabeler
Microsoft Alias: carlrab

@brettshearer -- Brett, thank you for your feedback.
@CarlRabeler -- Carl, please look into this issue.

#resassign: pmasl

@brettshearer Apologies for the long delay. @WilliamDAssafMSFT could you take a look at the link provided and other anecdotal research on the internet to come up with a potential description of the optional_spid attribute for this reference article. When you have a potential description, we can get it approved by @pmasl. Leaving both of you assigned on this for now. #assign:WilliamDAssafMSFT

@brettshearer Apologies for the long delay. @WilliamDAssafMSFT could you take a look at the link provided and other anecdotal research on the internet to come up with a potential description of the optional_spid attribute for this reference article. When you have a potential description, we can get it approved by @pmasl. Leaving both of you assigned on this for now. #assign:WilliamDAssafMSFT

@brettshearer Thanks for contacting Microsoft with feedback about our product documentation. A document update is in flight.
The connection session_id (spid) becomes part of the cache key in order to reduce the number of re-compiles. This prevents recompilations for a single session's re-use of a plan involving non-dynamically bound temp tables. You may find more information about this scenario here.
It should take a few business days to appear live. We'll contact you if we need further information, and you'll receive an email message each time we "@mention" you in a comment.
Thanks again for submitting your feedback. Your suggestions help improve our documentation.

@brettshearer Thanks again for submitting your feedback, a doc update has been approved and should be live later today.


[CI]: Remove Ashish Tiwari(Revolyssup) as Reviewer from dependabot

Current Behavior

We have a reviewer Ashish in dependabot yml file.
Desired Behavior

We need to remove him from reviewers.
Implementation

Acceptance Tests

Contributor Guides and Resources

📝 Meshery Adapters Spreadsheet
🛠 Meshery Build & Release Strategy
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

#136 successfully completes this issue ticket.


zuul: allow local builds

Will add this play to other container image repositories after we merged it here.


Added gizmo specific 'IsOver' function

I implemented object picking in my 3D project and had to make sure it only picks when my mouse is not hovering ImGuizmo but the current IsOver detects for all three gizmos even if they're not visible. This simple addition does it based on operation, since thats a client-side thing anyway.

Thank you!


Optimize gas fee

Issue: #399: Optimize gas fee for deregisterProvingKey function in VRFCoordinator contract
Change description:

[1] Cache length of s_provingKeyHashes array to memory before running for loop.
[2] Default value of uint type is 0 so we don't need to assign it value to 0 again.
[3] Proving key hashes will be pushed by only contract owner so it can't be overflow 256 bit. So we can add ++i inside unchecked keyword

Hello @HuyHuynh3103
Thank you for your contribution!
As you probably know the length of s_provingKeyHashes is not expected to be very large, and the gas savings will be proportional to the number of keys in the array. We need to balance the optimal gas usage together with a good code readability.
Did you compute the gas savings for this particular fix?

What is the gas savings for caching storage variable s_provingKeyHashes compared to repeated reading of its length?
What is the gas savings of unchecked iterator increment?

I believe that the first proposal would be good to include, but I am more hesitant about the second one. I do not expect the gas saving be significant while the code readability suffers from the second one.
I am looking forward to reading your answer!
Cheers,
Martin

Hello @martinkersner
I calculate the number of gas that the iterator used in this way:
        uint gas = gasleft();         // for loop code         gas -= gasleft();         emit GasUsed(gas);
I have tested on an array with 4 elements
Procedure:

Execute registerProvingKey 4 times to push 4 key hashes into s_provingKeyHashes
Execute deregisterProvingKey with the last proving key

The testing result is:

before fix: 19105
cache length of array: 18792
unchecked iterator increments: 18730
combine two of optimized gas way: 18417

I believe my proposal will reduce gas significantly if s_provingKeyHashes grows very large.

@HuyHuynh3103 Thank you for the explanation!
If I understand it correctly, you are testing the worst case scenario when we need to remove the item located at the end of the array. To generalize the gas savings per iteration, we would end up with following

(19,105-18,792) / 4 = 78.25 gas by caching length array
(19,105 - 18,730) / 4 = 93.75 gas by unchecking iterator increments

(The gas values are in floating point because you did not deduct gas used for your gas measurement.)
As you have mentioned in your comment, the optimization reduce gas significantly if s_provingKeyHashes grows very large. The problem with this assumption is that it will never grow large, there is no reason for it. The number of items in the array will be most likely always kept in low single digits, and it will not be executed often.
I would like to suggest to keep only the optimization for caching length of array, and please format your code according to standards in this repository. After those changes, I am going to merge it.

@martinkersner, I have just updated upon your suggestion. Please review my changes again.

@HuyHuynh3103 Please fix the code formatting.

@martinkersner I've done


using extrema_finding_kwargs in FrequencyFits

Looking at the code briefly, it seems like theFrquencyFits does not actually use and let the user specify the extrema_finding_kwargs.

For example, the find_peaks function is called here
https://github.com/vijayvarma392/gw_eccentricity/blob/main/gw_eccentricity/eccDefinitionUsingFrequencyFits.py#L603
and I do not see a way the user would be able to override the arguments, for example, the distance.
Also we want to update the extrema_finding_kwargs that is used to find the peaks so that we can later access it and see what values have been used.
I understand that since the code proceeds segment by segment and therefore probably uses different values for different segments. But it would be nice to store these values somewhere. Also, we should be clear about this in the doc. Right now the user would think that they are passing some kwargs but in reality, those are never used here.
@haraldp271 let me if I am understanding this situation correctly or not.

Done in #137


Add handler for other message types

It'd be helpful if there was a handler to handle non-media messages from the server. I have a case where I want the server to close the connection. I can send a message over the phoenix channel to tell the client that the channel is closing in N ms, but the client has no way of catching that message and handling it at all. So my server just forcibly closes the channel and then the client code has to handle an error.
I think this is because the library is handling opening and managing the socket channel for us. So we don't have access to the channel to attach custom listeners (which is possible in the membrane-webrtc-js)

Hi, thank you for your insight.
You're right; this library shouldn't manage the socket channel. We plan to remove the dependency on the Phoenix library entirely. Users of this library should be responsible for managing the signaling channel.
We have that task high on our backlog, and we plan to implement this soon.

I'm not necessarily advocating for separating out the signaling channel into a peer dependency that needs to be managed separately. There's something to be said for a "batteries included" library that enables end to end functionality. Simply exposing the underlying phoenix library might be sufficient

We are currently using react-native-membrane-webrtc as a client library for RTC-Engine. RTC-Engine does not impose a specific communication channel for transmitting events. Initially, we utilized Phoenix Channels for this purpose as it was most convenient in our Elixir application. However, we acknowledge that users may have reasons to avoid this implementation.
We want to provide flexibility to users, allowing them to choose the communication channel for events. We do not want to enforce the use of Elixir Phoenix channels. If someone decides to use this library, they should also decide how to transmit events. react-native-membrane-webrtc will be used solely for interpreting these events.
Simultaneously, we are developing Jellyfish Media Server and its client libraries, which are designed to be "battery included." Users do not need to implement anything themselves. In Jellyfish, events are sent via websockets without the Phoenix overlay.
So, we have two approaches: a low-level one where users configure everything themselves, and a simpler path for those who prefer minimal configuration, such as Jellyfish and its SDK.

@incubo4u Is this a "close won't fix" sort of resolution? I didn't see a recent PR that addressed it or any updates to the documentation.

Hi @Tonyhaenn 👋
I want to clarify that closing the issue doesn’t mean we consider it a “close won’t fix.”
We truly appreciate your report, and it’s definitely on our radar 📡
While there hasn’t been recent activity in the form of a PR or updates to the documentation, please know that it doesn’t imply dismissal 😊
Your concerns are important to us, and our team is actively working on addressing the issue.
We recognize its significance and are committed to finding a solution.
If you have any further questions or suggestions, feel free to let us know!


Make API URL configurable

DO NOT MERGE

Codecov Report

Merging #345 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #345   +/-   ##
========================================
  Coverage      17.3%   17.3%
  Complexity      137     137
========================================
  Files           174     174
  Lines          2254    2254
  Branches        290     290
========================================
  Hits            390     390
  Misses         1827    1827
  Partials         37      37

Continue to review full report at Codecov.

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


Help, decoding gt02a gps trackerr

I setup gt02a device and set to 5002 port but it shows wrong position and time iz 2 dayss in past,here is hex
283032373034333432353037334252303031353039303241343435382e393538344e30323030392e35393337453030302e373230303732363132302e333030303030303030304c303030303030303029
Thanks.

Solved , after 20 minutes device start  sending correct position and time... port 5002

Cold GPS fix can take 15 or more minutes.


As a user, I want a changelog documented on the docs site

Background: when discussing version bumps with Josh, I mentioned that a changelog that listed out major changes would be helpful (ie: changes of class name or component helpers). For example when we updated the tables there was a change to the helper names from <Table.HeadSmall> to <Table.HeadStacked>. These are things that people would need to be aware of if they had to upgrade because that way they know what areas they need to check/fix so that the component doesnt break. The point is, is to make peoples lives easier when they have a large gap between versions when they upgrade.

It doesn't look like the current "releases" page is accurate enough. One of the version bumps has 24 commits and yet it doesn't have a description of what was changed.
I see that there is a changelog file within the project but its empty. I'm not sure if this was meant to be set up so that it automatically updated the changelog or what.

Scenario: View Changelog on Docs Site

[ ] On the docs site, when I click the version number, I want to be directed to a changelog.
[ ] When I view the changelog, I see a more descriptive list of updates made for each version.

If this isn't done automatically, we might need to consider adding instructions for whoever pushes up the new versions so that they know to add in the list of changes.


feature: Avoid using JOINs to get stats for the Dashboard

Test Plan

[x] Manually confirm the queries match the previous numbers

@icerove I agree that we should keep migrating to TS; unfortunately, I don't have capacity to do that myself these days.


Fix contraflow cycle via system::metrics pipeline

Pull request
Description
Before this PR is was possible to trigger a contraflow event cycle when connecting any pipeline to the system::metrics pipeline. In that case contraflow messages might be sent via the system::metrics pipeline which used to account for all its input pipelines (all pipelines are input to the system::metrics pipeline). So the cycle went from offramp to pipeline X to system::metrics pipeline to all of its inputs (among them pipeline X) and to system::metrics pipeline and so forth.
This is avoided by having system::metrics not account for its input pipelines, so it is a dead end for contraflow messages.
Related

Checklist

[x] The RFC, if required, has been submitted and approved
[x] Any user-facing impact of the changes is reflected in docs.tremor.rs
[x] The code is tested
[x] Use of unsafe code is reasoned about in a comment
[x] Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
[x] The performance impact of the change is measured (see below)

Performance

Coverage decreased (-0.008%) to 83.81% when pulling 3ff72a6f3979f4b7f7a1fbaa950a056e37f998b9 on system_metrics_cycle into a667992e7cb5420e287fe97b2ba1aed7063c3582 on main.


Update connecting-your-database.md

Added a note that the schema field is a required field in dev credentials
Note: Heads up! The schema field in the 'Developer Credentials' section is a required field.
Description & motivation

To-do before merge

Pre-release docs
Is this change related to an unreleased version of dbt?

[ ] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[x] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects

thanks @nghi-ly , committed those changes and let me know when i can go ahead and merge 🙏

@mirnawong1


[backport] Fix output of show scalaInstance

libraryJars is an array, so we can't just call toString on it to display
its content.
This is a backport of 4ae22604cc5d855badb686d7182f16609a76d4b8.

To validate this pull request against other sbt modules, please visit this link.


Not possible to start scan

Describe the bug
Some phones(Huawei P30, ..) running Android 10 have problem to start BLE scan.
To Reproduce
just call
rxBleClient.scanBleDevices(                                 ScanSettings.Builder()                                     .setScanMode(ScanSettings.SCAN_MODE_BALANCED)                                     .build(),                                 ScanFilter.Builder()                                     .setServiceUuid(ParcelUuid(UUID.fromString(Constants.UUID_SERVICE_BATTERY)))                                     .build()                             )
Expected behavior
A clear and concise description of what you expected to happen.
Smartphone (please complete the following information):

Huawei P30
OS Android 10
Library version: 1.11.1

Logs from the application when bug occurs (this will greatly help in quick understanding the problem)
No logs
Additional context
When the scan is stareted an Bluetooth is tunred off and on again the scan will deliver ScanResults.
The same applies when during scan phone is locked and unlocked
https://stackoverflow.com/questions/61315872/rxble-start-scan-does-not-work-on-android-10-huawei-p30

Hello,
Thank you for using the library.
Your post seems to be a bug report but it lacks some crucial information. Could you edit to contain information according to the issue template?
Here you state that scan does not start but on stackoverflow it does not start immediately but only after certain conditions happen. Please add reproduction steps and logs from the application — maybe then it will be possible to help you.
Best regards

Thank you for quick resposne.
As you can see in Additional commetn, there is exactly the same description as in stackoverflow.
Repro step:

turn on BT and location
start application
start scan

in this case the state is READY, but no scan result are delivered.
There are no logs from appliaction containing BLE scan issue(I do not have access to Huawei P30 at the moment).
What is maybe important, I am using rxble library INSIDE my library

Without logs I cannot be sure if RxAndroidBle is called at all.

@ramlok I am closing this issue as I do not have any actionable info. Feel free to add new things here and I will reopen.

I have too this Bug ;( on Huawei P30 Lite

The service uuid filter seems broken on Huawei P30 Lite with latest RxAndroidBle (1.11.1)


Enhance architecure diagram

Fixes #104
Changes
Added two alternative architecture diagram enhancements

OTEL compoenents shown as part of the main architectural diagram
OTEL componenets shown as a seperate diagrm

Thanks for the contribution! This looks pretty good. Is there a way to tweak the contrast on the background at all? Right now, on the topmost image, going between the grey and black looks a bit odd.

I'm not a big fan of the variation on the top graph. The top graph should be about the actual application service connections, and not necessarily the telemetry flow.  The bottom chart represents the telemetry flow.

Thanks for the contribution @algebrajones!
It's hard to comment on a graph so I'll try a different approach:

Would it help if we callout that all the color-coded components are instrumented (any grey/black components are not instrumented)

Then in the telemetry flow diagram, we can abstract it out a bit, e.g. "all the instrumented components are sending data to Collector", which helps to make the diagram more readable.

@reyang's approach makes lots of sense and also allows us to easily document (perhaps optional styling) to indicate where vendors would plug into the Collector to export to their backend

+1 to @reyang's approach -- less busy, which is nice!

I really like the proposal @reyang I'll work on putting something together like that. Have to admit I prefer the second option, and as @puckpuck says it gives that info on how other vendors can link into the demo.
I'll link out to the actual documentation on what is instrumented i.e. metrics and traces, IIRC the collector isn't setup for log collection currently, I'll double check that as I document this though.

🤞 this works.
@puckpuck I've not added any custom styling on the exporters, but I have added text and links above the diagram to say where the exporters are configured.
To save people jumping around here's an image of the new version

Small nit. The titles of the colored boxes fades in dark mode.
But otherwise looks excellent! I’m a fan 🚀

These updates are fantastic!


info

Hello, what will be the difference ? with https://github.com/RVC-Project/Retrieval-based-Voice-Conversion-WebUI
Thank You

It's for developers.
Still under development again, so please be patient.


Remove flake8-to-ruff

Summary
We stopped releasing this a while ago and no longer advertise it anywhere. IMO, we should remove it so that we stop paying the cost of maintaining it. If we want to revive it, we can always do so from Git.

Ah, it might be a good idea to push one more (binary-identical?) <IP_ADDRESS> release to PyPI with a prominent "This project has been archived, see https://github.com/astral-sh/ruff/pull/9329" warning in the readme...


Exceptions when reading tf in emtfxml format

Two errors have appeared since the last widescale test in early June 2023.

An error occurred when the "attachment" field of the xml had value None

This error originates in emtfxml.py around line 308, during the iteration over element_keys.  When the element == "attachment" the line attr.read_dict(root_dict) fails.
This occurs on line 32 of attachment.py in the line
helpers._read_element(self, input_dict, "attachment")
The input_dict does have a key "attachment", but the value is None
I suggested a modification to read_dict() in attachment.py to return if input_dict["attachment"] is None.

Traceback (most recent call last):
File "/home/kkappler/software/irismt/mt_metadata/mt_metadata/transfer_functions/io/emtfxml/metadata/helpers.py", line 80, in _read_element
cls.from_dict(element_dict)
File "/home/kkappler/software/irismt/mt_metadata/mt_metadata/base/metadata.py", line 631, in from_dict
meta_dict = helpers.flatten_dict(meta_dict[class_name])
File "/home/kkappler/software/irismt/mt_metadata/mt_metadata/base/helpers.py", line 310, in flatten_dict
for key, value in meta_dict.items():
AttributeError: 'NoneType' object has no attribute 'items'

And a zipped copy of the xml is here
8P_CAR04_RRCAZ12.xml.gz

One of the SPUD xml, (which used to read-in) chokes on the "site" key
Specifically I think the error occurs when it is parsing the "orientation" info, but the orientation values in the offending "site" dict (pasted below) don't appear different from xml that do read in ... puzzling.

"site": {
"acquired_by": null,
"country": null,
"end": "1980-01-01T00:00:00+00:00",
"id": null,
"location.elevation": 0.0,
"location.latitude": 0.0,
"location.longitude": 0.0,
"name": null,
"orientation.angle_to_geographic_north": 0.0,
"orientation.layout": "orthogonal",
"project": null,
"run_list": "",
"start": "1980-01-01T00:00:00+00:00",
"survey": null,
"year_collected": 1980
}
}

Traceback (most recent call last):
File "/home/kkappler/software/irismt/mt_metadata/mt_metadata/transfer_functions/io/emtfxml/metadata/helpers.py", line 80, in _read_element
cls.from_dict(element_dict)
File "/home/kkappler/software/irismt/mt_metadata/mt_metadata/base/metadata.py", line 631, in from_dict
meta_dict = helpers.flatten_dict(meta_dict[class_name])
File "/home/kkappler/software/irismt/mt_metadata/mt_metadata/base/helpers.py", line 310, in flatten_dict
for key, value in meta_dict.items():
AttributeError: 'str' object has no attribute 'items'

Here is a zip of the XML with the second error
18645148_8P_CAS03.xml.gz

Hint on error 2:
On a file that does read in fine, when we hit the function _read_element in metadata_helpers.py, the value of element dict is:
element_dict
Out[3]:
{'orientation': OrderedDict([('angle_to_geographic_north', '0.000'),
('value', 'orthogonal')])}
But in the failing case, we have:
element_dict
Out[6]: {'orientation': 'sitelayout'}, and since the value of orienation is not a dict, we get the error, since the string has no items() method.
So, starting from emtfxml.py, when element == site (around line 306),
the attr is ______ and this goes into attr.read_dict()
read_dict loops over site keys, and when it gets to orientation, for the workng case in site.py,
lines ~95-110 we have this output

element
Out[3]: 'orientation'
attr
Out[4]:
{
"orientation": {
"angle_to_geographic_north": 0.0,
"layout": "orthogonal"
}
}

But in the broken case I also see (and cut and paste):

element
Out[6]: 'orientation'
attr
Out[7]:
{
"orientation": {
"angle_to_geographic_north": 0.0,
"layout": "orthogonal"
}
}

In orientation.py, in the read_dict method in the working case, we have:

input_dict
Python 3.10.10 (main, Mar 21 2023, 18:45:11) [GCC 11.2.0]
Type 'copyright', 'credits' or 'license' for more information
IPython 8.12.0 -- An enhanced Interactive Python. Type '?' for help.
PyDev console: using IPython 8.12.0
Out[1]:
OrderedDict([('project', 'USMTArray'),
('survey', 'CONUS SoCal'),
('year_collected', '2019'),
('country', 'USA'),
('id', 'CAR04'),
('name', 'The Meadows Slough, CA, USA'),
('location',
OrderedDict([('latitude', '38.245768'),
('longitude', '-121.489255'),
('elevation', '0.000'),
('declination',
OrderedDict([('epoch', '1995.0'),
('value', '13.200')])),
('datum', 'WGS84')])),
('orientation',
OrderedDict([('angle_to_geographic_north', '0.000'),
('value', 'orthogonal')])),
('acquired_by', 'National Geoelectromagnetic Facility'),
('start', '2019-05-03T22:29:40'),
('end', '2019-05-09T00:25:42'),
('run_list', 'CAR04a'),
('data_quality_notes',
OrderedDict([('rating', '1'),
('good_from_period', '20.000'),
('good_to_period', '400.000'),
('comments',
OrderedDict([('author',
'Adam Schultz and Esteban Bowles-Martinez'),
('value',
'Very poor data quality will require careful removal of large frequency bands before inverting.')]))])),
('data_quality_warnings',
OrderedDict([('flag', '1'),
('comments',
OrderedDict([('author',
'Adam Schultz and Esteban Bowles-Martinez')]))]))])

but in the broken case we have:

input_dict
Out[2]:
OrderedDict([('project', 'USMTArray'),
('survey', 'CONUS SoCal'),
('year_collected', '2019'),
('country', 'USA'),
('id', 'CAS03'),
('name', 'Pleasanton Ridge, CA, USA'),
('location',
OrderedDict([('latitude', '37.658830'),
('longitude', '-121.959752'),
('elevation', '484.050'),
('declination',
OrderedDict([('epoch', '1995.0'),
('value', '15.000')])),
('datum', 'WGS84')])),
('orientation', 'sitelayout'),
('acquired_by', 'National Geoelectromagnetic Facility'),
('start', '2019-10-24T22:07:08'),
('end', '2019-11-08T00:49:32'),
('run_list', 'CAS03a CAS03b'),
('data_quality_notes',
OrderedDict([('rating', '1'),
('good_from_period', '30.000'),
('good_to_period', '100.000'),
('comments',
OrderedDict([('author',
'Adam Schultz and Esteban Bowles-Martinez'),
('value',
'Signal overwhelmed by cultural noise near electric train in San Francisco Bay Area. Very poor data quality that will require careful removal of large frequency bands before inverting.')]))])),
('data_quality_warnings',
OrderedDict([('flag', '1'),
('comments',
OrderedDict([('author',
'Adam Schultz and Esteban Bowles-Martinez')]))]))])


Feature request: make bamtofastq output reads without mate in a separate file instead of skipping

I pre filtered a bam file so that it only includes reads mapping in a given region. Some mate pairs may have been on both sides of the boundaries of the region, but I'm interested in the reads having mapped in the region, even if their mate mapped outside. It would be useful to have an option to get those reads written in a third fastq file instead of having the following warnings and loosing them:
*****WARNING: Query 37433727 is marked as paired, but its mate does not occur next to it in your BAM file.  Skipping.
*****WARNING: Query 37533486 is marked as paired, but its mate does not occur next to it in your BAM file.  Skipping.
*****WARNING: Query 37538659 is marked as paired, but its mate does not occur next to it in your BAM file.  Skipping.
*****WARNING: Query 37540273 is marked as paired, but its mate does not occur next to it in your BAM file.  Skipping.
*****WARNING: Query 37568157 is marked as paired, but its mate does not occur next to it in your BAM file.  Skipping.
*****WARNING: Query 37683188 is marked as paired, but its mate does not occur next to it in your BAM file.  Skipping.
*****WARNING: Query 37741932 is marked as paired, but its mate does not occur next to it in your BAM file.  Skipping.

This feature would also be very useful to me.
I have reads that are marked as paired, but a mate is missing and it would be fantastic to have a third file written that captures those reads.
Note: the unpaired mates in my case aren't due to a filtering step, so this feature should be useful to folks who aren't filtering, as well as to those who are.
Thank you.

I would also like to request for this feature. Please provide option to output such mate missing / single / unpaired reads into separate file.
Thanks,


Blazor WASM security updates for 5.0

Addresses #19503
Internal Review Topics ...

Hosted AAD
Hosted AAD B2C

Alright ... so ... the good news is that everything is working here :+1:. The bad news is that ...
Scopes ... scopes ... and SCOPES!
The scopes scenario between api://-based and https://{TENANT}.onmicrosoft.com/-based App Id URIs is resolved now with Jean-Marc's assistance. The difference is based on the tenant's domain: If it's verified, you get full control and api://-based. I'm updating the PR to react to this new understanding.
WRT additional scopes (openid, offline_access, User.Read)
I'm dropping the hosted INCLUDES from yesterday for these hosted scenarios. Testing indicates that the app doesn't :boom: and the access token doesn't :boom: if the dev just goes with the solution config guidance and ends up with a client app that only specifies the API scope as the default access token scope. There's no need or reason to do anything with openid, offline_access, or User.Read for the access token scope AFAICT. Correct me if I'm wrong about that.
B2C broken config from CLI command
Another little side note on a broken scenario with the hosted B2C scenario ...
When creating the solution from the template with the CLI command, one still has to keep an :eye: out for a duplicated scheme and host in the App ID URI. Even with the 5.x template, the command still yielded ...
options.ProviderOptions.DefaultAccessTokenScopes
    .Add("https://XXXXX.onmicrosoft.com/https://XXXXX.onmicrosoft.com/eaee236e-faf8-442e-b472-7ff311d5ae76/API.Access");

... so it had to be manually changed to ...
options.ProviderOptions.DefaultAccessTokenScopes
    .Add("https://XXXXX.onmicrosoft.com/eaee236e-faf8-442e-b472-7ff311d5ae76/API.Access");

This is an on-going problem from 3.x. Idk if you want to discuss it further. The topic does call it out as a gotcha to keep an :eye: out for.

@captainsafia ... I just hit a 🧠🔥 brain fry 🧠🔥 for the day wrapping up these draft updates. In order not to have broken preview docs over the weekend, I'm going to go ahead and merge this now and take it live. I'll go back over these updates on Monday morning and possibly make further updates. The next major PR is probably Tuesday (Wednesday at the latest) and will focus on the AAD groups and roles topic.


Fix file github branch to match actual branch name

[#OSF-3882]

Looks good to me @kch8qx! Done with code review 1! :sunny:

The sidebar menu displays the branch name (test_branch) I might expect, but the file content displayed on the page is actually from the master branch.
Thus this affects actual functionality in addition to the cosmetic issue originally reported in [#OSF-3882]. I've created a followon ticket with some examples. @kch8qx , can you investigate further? I've created that other ticket as a blocker- once the actual functionality is addressed I'll take this back up for code review.
https://openscience.atlassian.net/browse/OSF-4611


Apply 'getEditor' logic of producing a React component to other elements

Thinking about using this plugin in a project I am working on and it would be really nice if the same logic implemented by the getEditor was applied across the other components as well in order to make it more customizable.
The 3 I have in mind right now are:

[ ] field selector
[ ] operator selector (almost there with the getOperators)
[ ] rule removal button

I would be potentially willing to contribute by making a fork/PR if needed

That sounds good. I am assuming that you are trying to introduce more conditional control to decide the list of fields/operators/editors/actions on the fly. I think we can call that out with a configuration object instead of passing multiple props into the QueryBuilder.
How about a configurator that can do the following:
const configurator = {
  getEditor(field, operator){}, // Gives <SomeEditor />
  getFields(){}, // array of fields
  getActions(ruleOrGroup){} // array of valid actions (edit, delete)
};

The above might not be an exhaustive set, but hopefully gives you a sense of where I am going.
Of course, would love a PR 😃

Yeah the configuration object sounds fine. I was more interested in using the semantic ui react components in place of native browser elements http://semantic-ui.com/introduction/integrations.html
So I would probably have to take a closer look to see if it would be easy to implement as long as it makes the appropriate callback to your components


Fix failing test

To count the correct number of requests, don't reuse exactly the
same request as the other test.

Coverage remained the same at 100.0% when pulling 746fc331231026a221166254e33ea459b9d923c7 on fix-test into 6298dabe584f522e7b7811a5806f99b0c88e23b1 on master.


bugfix: eth_call for pyevm should not check for known account

What was wrong?
closes https://github.com/ethereum/web3.py/issues/2198
How was it fixed?

Build an unsigned, spoofed tx rather than a signed tx for eth_call via py-evm. The account should not need to be a known account since we aren't signing for eth_call because it does not change state.

Todo:

[x] Clean up commit history
[x] Add entry to the release notes

Cute Animal Picture

This: https://github.com/ethereum/eth-tester/issues/211
was opened in relation to the original web3.py issue. Can this close that too?


Add iOS frontend for Counter sample.

Not merging as this implementation doesn't work yet. Initializing the runtime results in a kotlin.TypeCastException for the update and view functions due to the generic types in Oolong.runtime.


POSIX Requirement

Q
A

Issue Type
Question

Deployer Version
4.0.2

Local Machine OS
Unix

Remote Machine OS
Docker php:5.6

Description
I am trying to deploy in a gitlab setup using .gitlab-ci.yml but am unable to due to the requirement for POSIX is there anywhere around that?
This happens every time I try to deploy:
Shell on your server is not POSIX-compliant. Please change to sh, bash or similar.     Usually, you can change your shell to bash by running: chsh -s /bin/bash
Deploy.php
`<?php
namespace Deployer;
require 'recipe/common.php';
// Configuration
set('repository', 'git@github.com:example/deployer.git');
set('shared_files', [
'web/.htaccess',
'.env'
]);
set('shared_dirs', [
'web/app/uploads'
]);
set('writable_dirs', [
'web/app/uploads'
]);
set('application_name', 'deployer');
// Servers
server('staging', 'example.com', '1234')
->user('user')
->password('password')
->set('deploy_path', '/var/www/{{application_name}}/staging')
->set('branch','develop')
->set('stage', 'staging');
// Tasks
desc('Deploy your project');
task('deploy', [
'deploy:prepare',
'deploy:lock',
'deploy:release',
'deploy:update_code',
'deploy:shared',
'deploy:writable',
'deploy:vendors',
'deploy:clear_paths',
'deploy:symlink',
'deploy:unlock',
'cleanup',
'success'
]);
after('deploy', 'success');`

How did you run dep?

I ran php deployer.phar deploy staging through the gitlab ci using docker.

I also have same problem

Try run ssh echo $0 in gitlab ci using docker. Whats output?

This is what happens when I do that.

Warning: Identity file .......... not accessible: No such file or directory.
ssh: Could not resolve hostname echo: Name or service not known

I think error is:

Warning: Identity file .......... not accessible: No such file or directory.
ssh: Could not resolve hostname echo: Name or service not known

It appears that GitLab can't run in an interactive shell since all jobs have to be automated. Is there a way to run this in a non-interactive shell?

I actually found the problem. Deployer wasn't getting the variables I was setting in GitLab and was failing authentication to the server. Passing the variable in directly resolved the issue. Just need to figure out how come its not accepting the gitlab variables. How would I get environment variables?

What is your GitLab CI config? In php you can get env variables by getenv()

I am using hidden variables and was trying to use get() to get the environment variables. I am using getenv() now and it is working fine. The error was confusing though maybe look into error handling when there is an empty or no password?

empty or no password for that?
Please, show your gitlab ci config.

stages:     - deploy before_script:     - curl -LO https://deployer.org/deployer.phar     - mv deployer.phar /usr/local/bin/dep     - chmod +x /usr/local/bin/dep     - dep self-update Deploy to staging:   stage: deploy   script:     - dep deploy:unlock staging      - dep deploy staging --revision=$CI_BUILD_REF -v   environment:     name: staging     url: https://staging.example.com   only:     - develop Deploy to production:   stage: deploy   script:     - dep deploy:unlock production      - dep deploy production --revision=$CI_BUILD_REF -v   environment:     name: production     url: https://example.com   only:     - master   when: manual

You can omit dep self-update as https://deployer.org/deployer.phar already contain latest version.

Yeah, that is just left over from when I was trying different ways of deploying to try and resolve the problem.


Incompatible with current version of Keras

Hi,
I've found a couple of issues related to use of this code with Keras 2.2.4 in sim-gan.py that cause errors.

Line 72 needs to be changed to y = layers.merge.Add()([input_features, y])
Line 177 needs to be changed to data_format='channels_last')
Thanks for providing the code, it's very helpful.

Thanks a lot for providing the code. Very simple and easy to follow.
layers.merge is no longer available in new versions of the keras.
So I changed layers.merge to layer.Add and it solved the version incompatibility issue. However,  later it causes a 'shape miss match error between input_feature and y.
Any idea how to solve this?
P.S: I am using the .h5 files from the Kaggle repo so the size of input data is (55, 35) for both datasets.
Best,
noushin


Drop fileowner and use current ansible_user

I think this was fixed by https://github.com/icpcsysops/ansible/commit/dcb5516f7a7e84e76417b353509d37fe71e7d48a#diff-50d00459294eabe8fd4a8d279be4ca46f5956980082c05216348e016aba5980dL9-L10, so we can probably close this?


Create otfk.txt

https://otfk.od.ua/

@Traer3 I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host otfk.odessa.ua not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


⚠ ClamAV has degraded performance

In 4d4ac18, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 193 ms

Resolved: ClamAV performance has improved in 3d5f24f after 6 minutes.


Task0

added Log messages in the task

SourceCode/2023/Oct2023/Basics/SachinRawal/Task2/project.json
Hey there,

For the error An exception was thrown while activating..., it normally happens when Uipath doesn't have the right access to certain resources. Ensure to check if all dependencies have been installed, Studio or the Robot is run with Admin rights and the robot service is executed correctly.

On Cannot perform project analysis because some workflows have errors, I'd suggest to try identifying the specific workflow that causes this error. You could do this by running each workflow one by one to catch the one throwing this error. Once identified, review it meticulously for errors and correct them, before trying to analyze again.

Stay focused, you're doing fine. Keep polishing your work until it shines.
Keep going!

SourceCode/2023/Oct2023/Basics/Sudheer Devabattula/Test0/project.json
Hey there, I stumbled upon a few roadblocks while trying to review your code.

Main.xaml - There seems to be an issue related to a 'LogMessage' activity that the program doesn't recognize. It's possible that either the activity is not being utilized in the right way or maybe it hasn't been installed. Check and correct it.

The second error states that your project analysis could not be executed due to the presence of errors in some workflows. Before you proceed with the project analysis, make sure you've addressed and rectified all errors present within your workflows.

Hope this helps, let me know if you need more information.

Means No Need of log message activity for this simple task?

SourceCode/2023/Oct2023/Basics/SachinRawal/Task2/project.json
Hey there,
Just had a look on your code, and here are some suggestions for you:

"An exception was thrown while activating ?:UiPath.Studio.Platform.Contracts.IPlatformWrapper."

Looks like an issue when you trigger the UIPath Studio Platform Contracts. Make sure that you're using the latest version of UiPath Studio and all installed packages are up-to-date. Resolving these steps might fix the exception.

"Cannot perform project analysis because some workflows have errors."

Simply, you've got errors in your workflows which are stopping the project analysis. I would recommend you to validate your workflows and debug them for any runtime errors. Once you fix those errors, the project analysis might just get through.
Hop on it, and hit me up if you need any further help.

SourceCode/2023/Oct2023/Basics/Sudheer Devabattula/Test0/project.json
Hey there! Noticed a hiccup in your 'Main.xaml' file. It seems like there's a MessageBox activity type that the system's having trouble recognizing. Make sure everything's correctly defined and spelled. Also, before carrying out project analysis, all workflow errors need to be sorted out. Make sure all your workflows are error-free. Let’s get those errors fixed and the code up and running again.

In project analysis it is showing Redundant Sequence The sequence Sequence is redundant, but the code from my end is running good.

SourceCode/2023/Oct2023/Basics/SachinRawal/Task2/project.json
Please make sure that the file uipath.studio.platform.contracts.iplatformwrapper functions properly. Looks like it is triggering an error. Also, ensure that all workflows are error-free before initiating the project analysis, as the code currently seems to have faulty workflows.
To fix the issues,

Investigate what's causing the exception in uipath.studio.platform.contracts.iplatformwrapper. Check whether the file is located in the correct directory and whether all dependencies have been added.

Review your workflows. If possible, run them individually to identify the erroneous one. Fix the problematic workflows and then run the project analysis again.

SourceCode/2023/Oct2023/Basics/Sudheer Devabattula/Test0/project.json
Hey there,
Seems like there's an issue with your Main.xaml file. The error message indicates that it can't load because of an unknown type, specifically {http://schemas.uipath.com/workwork/activities}MessageBox.
Please ensure that all your types are correctly defined and that MessageBox is installed. It looks like it might be missing or unavailable in your current setup. Also check your file for any other typos or incorrect type references.
On another note, the project analysis also appears to be failing because there are some workflows with errors. Clean these up, and it should help your project run smoother.
Remember to review and test your files carefully before reviewing.
Hope this helps!

@Sudheer2331
Welcome to Vajrang Org Let's kickstart your basic training and embark on your RPA journey with us.
To track your progress effectively, establish a daily routine of completing tasks and regularly submitting pull requests on GitHub.
Don't forget to seek code reviews to improve your skills further.
Test -0
Default activity names


Ampersand in title

see issue reported in theme, but it is a problem of core-code: learn/issues/2

fixed with 1.4.7

fixed with version 1.4.7 (just published)


replace clang with /opt/opencilk/bin/clang

I have replaced clang with /opt/opencilk/bin/clang throughout. The one thing I am unsure of is how this impacts Mac OS, so I wonder if @ailiop can please verify this? Thank you :-)
When we develop instructions to configure clang, then we'll want to update this.

@ailiop I merged this only because I realized too late that this came from the main branch of my origin repo. I would still appreciate your review here, especially w.r.t. Mac OS. Thank you.

Also, I just noticed that the "Example" block for our assumed setup in the Install page now uses a Markdown section instead of just bold italics to style the "Example:" label.  That is, ## Example vs ***Example:***.  I personally prefer the latter; the section inside the box feels awkward to me.

Thanks @ailiop. I understand the awkwardness of the Example, and it's largely because of my limitations with CSS and the various layers in our stack. (I used the header as a sure way to generate anchor tags that I needed from the Getting started page.)


posAssertion doesn't work

In the peggy documentation it is stated that

but this doesn't seem to work. Is the documentation out of date?
How to reproduce

Install peggynpm i -D peggy

Codeconst ab_parser = peggy.generate('posAssertion = "a" &"b"')

ab_parser.parse("ab")

Generates the error SyntaxError: Expected end of input but "b" found.

Never mind, I just realized that the examples have the match and rest patterns to make these work.


enhancement allow table in url when do two-phase commit (#15246)

Make it works even if user provide us with (unnecessary) table info in url. i.e. curl -X PUT --location-trusted -u user:passwd -H "txn_id:18036" -H \ "txn_operation:commit" http://fe_host:http_port/api/{db}/{table}/_stream_load_2pc can still works!
Signed-off-by: freemandealer<EMAIL_ADDRESS>Proposed changes
Issue Number: close #15246
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[X] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[X] No Need

Has document been added or modified:

[X] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[X] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[X] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.75 seconds
load time: 633 seconds
storage size:<PHONE_NUMBER>4 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221221134229_clickbench_pr_66415.html

Why should we support this url? Since {table} is meaningless?

What if user specify a different table?

It would commit the transaction based on specific txnid, table name only works as a placeholder and is ignored by the service.

Why should we support this url? Since {table} is meaningless?

Yes, {table} is unnecessary. I just try to make the url more consistant with the one in precommit command. I saw users copied&pasted the url from precommit command and confused why it won't works in commit.


Ability to customize error messages

The error messages this produces aren't exactly user-friendly:
https://github.com/simonw/asgi-csrf/blob/f1a7e2889b5e7df7d67dbb7d6f483c22dbffae83/asgi_csrf.py#L146-L150
https://github.com/simonw/asgi-csrf/blob/f1a7e2889b5e7df7d67dbb7d6f483c22dbffae83/asgi_csrf.py#L168-L188

This came up because Datasette serves confusing messages and I'd like to be able to change those to include links to the relevant documentation: https://docs.datasette.io/en/stable/internals.html#csrf-protection


WARN ignoring uplink error

I've setup a Dragino LPS8 as a Light-GW. But I can not see that my device (Dragino LGT92) connects to helium properly.
Release v1.0.0-alpha.8
Warning message
daemon.info helium_gateway[1710]:  WARN ignoring uplink error: Service(Rpc(Status { code: Unimplemented, message: "unknown service helium.router", metadata: MetadataMap { headers: {"content-type": "application/grpc", "trailer": "Grpc-Status", "trailer": "Grpc-Message", "trailer": "Grpc-Status-Details-Bin"} } })), module: router

Feedback from @madninja via discord

I’ll do a release that fixes that today. The settings in alpha.8 are not pointing at the staging router and they should

This should be fixed in f9efa3d and part of release v1.0.0-alpha.9


Dead man notification does not work

Avoid duplicates

[x] Bug is not mentioned in the FAQ
[x] Bug is not already reported in another issue

Describe the bug
As reported previously background checks are ~not working on my iPhone~ only work when I connect my phone to a charger for unknown reasons.
So I was patently waiting for the new dead man notification that was added with 1.3.0. Unfortunately I received no notification within 43 hours (at which point I opened the app to force a check).

Expected behaviour
After 36 hours of no check a local notification should prompt me to open the app.
Steps to reproduce the issue
Wait for 36 hours after the last check without opening the app.
Technical details

iOS Version: 13.7
Device: iPhone 8 Plus

Possible Fix
I have no idea really but I'm wondering if the local notification is only scheduled in an actual background task, not when the check is executed while the app is in the foreground. Because in my case the checks are only happening when I'm opening the app.

Internal Tracking ID: EXPOSUREAPP-2994

Hi @databus23 , thanks for sharing your content. Could you wait another 30 hours for a return time and then check the time of the risk calculation?
Thanks,
MP
Corona-Warn-App Open Source Team

@Marco2907 Not sure I fully understand. What do you mean be a "return time"? By "check the time of the risk calculation" you mean the last time a check was logged in Settings-> Exposure notifications?

This is also difficult to test the notification check at the moment only for the background task and not the risk calculation. This means if the background task was startet but canceled without a successful risk calculation the notification will be rescheduled. We plan to extend this to the risk calculation.

@thomasaugsten Not sure I fully understand, but this sounds like my assumption was correct that the dead man notification only works if the background task was executed previously.

No this means every time the background task was executed the notification is rescheduled in 36h. But an executed background task doesn't mean a successful risk calculation because iOS can stop the background task after 1sec because of memory or battery issues and this means the risk calculation is also stopped.

I see, so this could be the problem here.

Is there any news on this? I also have the case now, that after more than 36 hours there was no background check and no notification.
iPhoneXS
newest iOS and app version

This will be improved in v1.5

This will be improved in v1.5

Thanks for the fast reply! Can you give an estimation when 1.5 will be released? I think 1.4 was planned for mid october.

Not yet there is already an internal planning but not yet ready to publish.

@databus23
Can you please check if the Dead Man Notification now works with Version 1.5.1 (or above) from CWA?

I think this Issue can be closed since Dead Man Notifications are working with 1.5.1 and above

Should be fixed with https://github.com/corona-warn-app/cwa-app-android/pull/1513 and release 1.7.

Corona-Warn-App Open Source Team

I can't really reproduce the issue anymore as the background task execution started to work somewhat reliable for me.

@heinezen Please leave this Issue open.
With the (buggy) release of 1.6.0 and the (not optimal) hotfix 1.6.1 there are some users who complain about not working Dead Man Notifications (f.e.: https://github.com/corona-warn-app/cwa-app-ios/issues/916#issuecomment-727920973).
I can also confirm what @PalminX stated in #916:
The last check on my phone was performed at 15.11. at 22:42. Until now, no new check happened (only a failed one) and also no Dead Man Notification.

@Ein-Tim We will leave it open until 1.7 has been rolled out to all users and check for feedback.

Corona-Warn-App Open Source Team

No deadman notification showed up for me, up to 65 hrs after the laste foreground check. Then, another background check happened (see https://github.com/corona-warn-app/cwa-app-ios/issues/916#issuecomment-728935091)

@Ein-Tim We will leave it open until 1.7 has been rolled out to all users and check for feedback.
Corona-Warn-App Open Source Team

Is this fix also for iOS? Asking since the PR was for the Android repo: https://github.com/corona-warn-app/cwa-app-android/pull/1513

@PalminX

Is this fix also for iOS?

Apologies, I have confused the fixes for the different apps. I've updated my initial post.
The Dead Man Notification Bug for iOS is currently being investigated and there are unfortunately no updates on this.
Sorry for the confusion,
CH

Corona-Warn-App Open Source Team

Can anybody here report a working dead man notification with 1.6.0 or above?

Hi @PalminX,
FYI, I have added your report to the internal ticket for the developers.
Best wishes,
DS

Corona-Warn-App Open Source Team

I received a dead-man notification yesterday, Nov 19, at 21:!5, exactly 40 hrs after the last logged background check (which was at 5:15 on Nov 18).

Well, at least they seem not reliable.
See here

Well - seems not reliably. I got none, see details here

Dear @ndegendogo, @PalminX,
Many thanks for feedback, we forward these observation to the developers!
Best wishes,
DS

Corona-Warn-App Open Source Team

Dear community,
We would appreciate some feedback here. This issue was resolved in CWA version 1.7.1.
References: EXPOSUREAPP-2332, PR https://github.com/corona-warn-app/cwa-app-android/pull/1513.
Best wishes,
DS

Corona-Warn-App Open Source Team

@dsarkar I am a bit confused now - the PR you mentioned is for Android but this ticket is in the iOS repo?

Can anybody confrim that Dead man notificaitons are working with 1.7.1?
Or are the background checks to reliable with this version? 😉

@dsarkar maybe the devs could confirm that there is a related PR also for iOS?
Two weeks ago they were still investigating - but that comment might well be outdated today.
In the first week of 1.7.1 my checks were perfect. Now since two days they start to have large gaps, yesterday it was 8 hours instedad of 4, today I had a gap of 12h. Still by far better than before 1.7.1.
I'll continue to watch.

Dear @ndegendogo,
There is no PR that correlates directly to a fix of the deadman issue. However, this problem is understood to have resolved by fixing other issues.
Related internal tickets: EXPOSUREAPP-3880, EXPOSUREAPP-3821.
Let's monitor for the moment, any feedback will be forwarded to the developers! Many thanks,
Best wishes,
DS

Corona-Warn-App Open Source Team

@ndegendogo, I got some feedback, which might explain some recent irregularities in the checks. In den night from Sunday to Monday and Monday to Tuesday, time 0100 - 0200, the server was prepared for ENF v2 (future CWA version 1.8). Possibly, checks in that time frame were skipped.
Best wishes,
DS

Corona-Warn-App Open Source Team

Thanks @dsarkar.
I am just preparing to submit another ticket on some of the irregularities I saw last Thursday. (Nothing very serious, but it would be interesting if there is an explanation).

@ndegendogo, sure, see https://github.com/corona-warn-app/cwa-app-ios/issues/1631#issuecomment-739326690. Best, DS
Corona-Warn-App Open Source Team

Did some additional testing, Dead Man notifications are working with 1.7.1 👍

@Ein-Tim wow - so you saw such a notification? 🎉🎉

@Ein-Tim, thanks for letting us know. Could you let us know, how you could trigger such notifications? In any case, we will forward this information to the developers.
We will close this issue now. However, re-open if appropriate. Thanks to everybody for contributing here.
Best wishes,
DS

Corona-Warn-App Open Source Team

@ndegendogo

wow - so you saw such a notification?

Yes, this one:

@dsarkar

Could you let us know, how you could trigger such notifications?

Yeah, just make sure that your phone is not connected to the Internet for a bit more than 36h and you will revive one. 👍
But I saw that it was not exactly 36h after the last check, more like 37 1/2h. But I think this is ok 👍

@Ein-Tim, thanks for the information, very much appreciated. We will forward your observation to them and we would suggest closing this issue after monitoring a few more days. Best wishes,
DS

Corona-Warn-App Open Source Team

Dear @Ein-Tim, @ndegendogo, @databus23, @PalminX, and community,
Thank you for your contributions.  This issue will be closed now. Please, re-open if necessary.
Best wishes,
DS

Corona-Warn-App Open Source Team


Corrupted Files

I'm trying to extract the files of the card artworks, but the extracted files are all corrupt. Jpg files can't be viewed and gim files can't be viewed or extracted with programs like Noesis. Is there a problem or am I doing something wrong? I can't get the program to open with Command Prompt either. Thank you.

How exactly are you using this utility? You're meant to launch it from a shell, so it should just work from there.
Also which game (and its region) and which files did you use to extract?
I used GimConv to convert the image to PNG and it worked just fine. Below is an example extracted from Tag Force 1 cardh_e.cip (by dragging and dropping the file into the tool).

It should work for all formats except Tag Force Special's JPEGs which needs the '-s' argument to extract those.

I've taken a look at Noesis and it indeed doesn't open the file.
That is because the GIM header from the game isn't technically correct but official GimConv from Sony (and real PSP) doesn't really care about it so it just works.
GimConv throws this:
WARNING : illegal image order 1
Which seems to break whatever Noesis has, so just stick to tools based on GimConv.

I'm trying to extract from the Japanese version of Tag Force Special actually. How do I extract those? Does dragging the files onto the exe still work?

For Tag Force Special, you can drag and drop any CIP file except CARDJ_J.CIP which is the JPEG pack. For that you need to invoke the utility like this
CIPTool -s CARDJ_J.CIP
Then the JPEGs will also be properly extracted.

I'm also having getting GimConv to work. I tried downloading it from two sources, but I can't get either to work from different problems.

You can find GimConv in RCOmage here: https://github.com/kakaroto/RCOMage/releases/tag/1.1.1
I'm not sure if there's a good GUI utility that you could use (there is MinoriTool I guess but that thing shouldn't even exist in the state that it does).
There is also Gim Viewer which you could try using (but it doesn't support all formats).

Also on another note - to extract with GimConv you need to invoke the utility something like this:
gimconv 4007_0.gim -o 4007_0.png
This will produce a proper png file.

I just downloaded GimConv again but had the same problem. When I open the program the command prompt ends with "press any key to continue" and when I do the prompt closes.
For CARDJ_J.CIP do I change the file name to CIPTool -s CARDJ_J.CIP? How do I invoke the program as that?

For CARDJ_J.CIP do I change the file name to CIPTool -s CARDJ_J.CIP? How do I invoke the program as that?

This is exactly why I asked you this question:

How exactly are you using this utility? You're meant to launch it from a shell, so it should just work from there.

So you have to launch and use it as a command-line utility. This goes for gimconv as well!
Here are simple steps (assuming you're on Windows):

Extract CIP files next to CIP tool
In Explorer in that folder, hold SHIFT and right click in an empty spot
In the context menu, select "Open PowerShell window here"
Type in "cmd" to enter the command prompt
Type in the command I wrote: CIPTool -s CARDJ_J.CIP

And you should end up with the extracted files.

Specifically, I'm trying to extract the higher quality card artworks. On the Yugioh Wikia there's art from the older games at 544x544px. I'm trying to find those artworks from TF Special.

Tag Force Special doesn't come with 544x544 images. It comes with 256x256 pictures.
Tag Force 3 has higher res of 512x512 (and are currently, to my knowledge, the highest quality digital artwork we have available).
For JPEGs, you don't even need the gimconv tool. They should already be in JPEG when extracted, so you should be able to open them with an image viewer just fine.
Where the Wikia got them from though, I have no idea...

Also it's CARDJ_A.CIP in Tag Force 3. For Special it's CARDJ256_A.CIP or something like that (I can't remember, I don't have the game at hand).

It's no problem. I really appreciate you taking the time to explain all this. I'm not very knowledgeable with command prompt stuff.
I finally got it to work with the Power Shell and got all of the jpg files. It's really unfortunate Special only has 256x256 images. I was hoping to get decent sized images of some of the artworks that have never been used anywhere else like the World Championship prize cards. But it's great to have the 512x512 images of the anime exclusive cards since they still look decent up scaled for the wallpapers I make.
Also, Noesis works for converting the other gim files in the game, but it can't do them as a batch. Can GimConv do that?

Also, Noesis works for converting the other gim files in the game, but it can't do them as a batch. Can GimConv do that?

Technically it can. As it is a command-line utility, the batch processing is done by the means of a script file (Windows has it, it's called a .bat or Batch file, or PowerShell scripts like .ps1 files). This is a bit of a complicated method to make on your own, but a very effective one.
If you need it, I could write up a script for you that can handle it. (It'll have to wait for a little while before I can do that though)
I think there may be also some tools floating around the internet that also handle batch processing of GIMs but I can't seem to find them right now (except that MinoriTool I mentioned previously).

But it's great to have the 512x512 images of the anime exclusive cards since they still look decent up scaled for the wallpapers I make.

Offtopic - I've had great results with Cupscale and a few upscaling models (such as Lollipop and Ultra Yandere). You can use that to get some decently upscaled stuff!

Specifically, I'm trying to extract the higher quality card artworks. On the Yugioh Wikia there's art from the older games at 544x544px. I'm trying to find those artworks from TF Special.

Tag Force Special doesn't come with 544x544 images. It comes with 256x256 pictures.
Tag Force 3 has higher res of 512x512 (and are currently, to my knowledge, the highest quality digital artwork we have available, without censoring or watermarking).

For context, the Tag Force artworks uploaded to the YGO wiki were captured in-game and stitched together (since apparently when you view the full artworks, you can only see a quarter of the artwork at a time; this must have been very tedious indeed for the thousands of artworks across the series!), which is why they're over-resolution. At some point we intend to extract the artworks directly from the game files and upload replacements (this will also allow us to replace a bunch of JPEGs with PNGs which is something we're a bit particular about), it's just a matter of doing.
(Full disclosure: I'm an editor on Yugipedia, not the YGO Wikia, but I was an editor on there for almost a decade before Yugipedia forked from there.)

Pretty cool!

At some point we intend to extract the artworks directly from the game files and upload replacements (this will also allow us to replace a bunch of JPEGs with PNGs which is something we're a bit particular about), it's just a matter of doing.

Feel free to use this tool then! I've made it for this exact purpose (and among other stuff, modding the game).
A bit more technical info - the image data in the game is stored in JPEG with 4:2:0 subsampling. You won't get anything in terms of quality by converting it to PNG (unless you're doing web related optimizations, then I understand).
The filenames are Card IDs, which are the internal Konami IDs used in their database. (so 4007 = Blue Eyes White Dragon)
(You can find it by editing the cid in the URL: https://www.db.yugioh-card.com/yugiohdb/card_search.action?ope=2&cid=4007&request_locale=en - the cid=4007 part)
The suffix ("_0" and "_1") denotes the alternative artworks available.
And lastly, all IDs before 4007 are temporary ones used for pre-release/game-exclusive/temporary/token cards.
I hope this will be helpful!

Feel free to use this tool then! I've made it for this exact purpose (and among other stuff, modding the game).

That's the plan! ;)

A bit more technical info - the image data in the game is stored in JPEG with 4:2:0 subsampling. You won't get anything in terms of quality by converting it to PNG (unless you're doing web related optimizations, then I understand).

Aah right, the games using JPEG already, slipped my mind. In that case we'll be uploading them as JPEG, without conversion (though probably with a pass through a lossless filesize optimizer - unfortunately the options for optimizing JPEGs are somewhat slimmer than for lossless formats such as PNG).

The filenames are Card IDs, which are the internal Konami IDs used in their database. (so 4007 = Blue Eyes White Dragon)
(You can find it by editing the cid in the URL: https://www.db.yugioh-card.com/yugiohdb/card_search.action?ope=2&cid=4007&request_locale=en - the cid=4007 part)
The suffix ("_0" and "_1") denotes the alternative artworks available.
And lastly, all IDs before 4007 are temporary ones used for pre-release/game-exclusive/temporary/token cards.
I hope this will be helpful!

We did know all of this already, but a reminder never hurts. =)

If you need it, I could write up a script for you that can handle it. (It'll have to wait for a little while before I can do that though)

That would be great if it's not too much trouble. It would certainly make things easier.

Offtopic - I've had great results with Cupscale and a few upscaling models (such as Lollipop and Ultra Yandere). You can use that to get some decently upscaled stuff!

Thanks for the suggestion. I usually use Waifu2x, but I'll give this one a shot.

For context, the Tag Force artworks uploaded to the YGO wiki were captured in-game and stitched together

That's really cool. I can't imagine how much work that must have been.
Updating the images with pngs would be great.

@BlazeAccelerator Hi! It's been a while... Sorry for taking this long, but it's better late then never.
Use the tool I attached in this post. I included instructions on how to use it inside.
BatchGimConv.zip
I'll include this utility in the release as well so people don't wander around.


Feature/sg 996 add reports to s3

This PR includes

Scripts to automatically reproduce the reports. This is not meant for common use, only for the dev team to regenerate new reports when wanted.
Adding classmethods to instantiate AnalysisManager quickly which simplifies the code. Not mandatory for the PR but I think it's a good opportunity

To add:

[ ] Readme to the reports

This is a bit weird to me... why not use the from_x methods to generate a dataset and then pass that to a manager? why did you decide to create a different manager by calling from_x?

Renamed to analyze_x


Better experience

Hey @hacdias,
Would love to help you improve the directory navigation bar. :)
Check it out 😆

Please run build.sh and commit again.

Thank you.


Revert "subkey"

This reverts commit f6ed28a8a09bff10b573166c6b8f29cf82a8e3f0.
The commit was introduced in #6 but unfortunately came with a bug. I attempted to fix the bug in #7 but new errors appeared: Error: error:0906D06C:PEM routines:PEM_read_bio:no start line. I don't understand exactly what's going on in subkey and how to fix it, so for now I think it's safer to revert it and go back the original code.

3.0.0


The type of composer package was added

Without type it's impossible to install a plugin like composer dependency.

:+1:

Is there any reason why this pr is bad idea?

@NemoD503 Is there some way to configure composer.json to install plugins to /plugins directory?

This should do it:
"type": "october-plugin",
"require": {
    "composer/installers": "~1.0"
},


Use canonicalUrl parameter for canonical tag in meta headers

If defined. Fallback to .Permalink as before.
Useful for example to provide a copy of an article on your personal blog which is officially published on the company blog, without a SEO penalty for the main article.

Ok, Thanks!


rebranding project to Repo Flow

Adds project description

Hey Ryan,
This re-brands the project and adds a more complete project description. If you are okay with the changes, you can pull this and rename your project to git-repo-flow.
Regards,
Pieter

Hey Ryan,
Should I separate the renaming from the description then?
Regards,
Pieter