Remove de.gematik.tim.room.name and de.gematik.tim.room.topic in a future version

The state events de.gematik.tim.room.name and de.gematik.tim.room.topic described in https://gemspec.gematik.de/docs/gemSpec/gemSpec_TI-Messenger-Client/gemSpec_TI-Messenger-Client_V1.1.2/#5.4.18 currently are just exact copies of m.room.name and m.room.topic. Currently they don't seem to provide any additional value for TIM clients or servers, however they complicate implementing the spec a lot since they have additional requirements (like requiring m.room.name to be empty) and don't integrate with existing parts of the Matrix specification (shortcuts to specify a room name during room creation, displaying a name for spaces or public rooms using their respective APIs, etc). It also complicates integration with Matrix based solutions since SDKs and clients need to be modified to be compatible on basic features like room names and topics.
Considering that my suggestion would be to remove them from the TIM specification (with an appropriate migration path possibly?). This should simplify developing TIM compliant solutions going forward. I think the idea behind them was to allow encrypting them in the future, however that would already be a breaking change and such custom event types could just be introduced at that point. Alternatively if the idea behind them was extensibility by adding custom fields, that can be more easily served by adding such custom fields in a namespaced manner to existing event types.
Possibly there is a different reason for these events, that I am unaware of however. If such a reason exists, it might be a good idea to document it on this issue and then close it to serve as a future reference.

Yes, the idea was to start encrypting them, once that is possible. Additional room-specific data is meant to go into separate state events, again, once they can actually be encrypted.
I agree that in their current form the custom name and topic fields are entirely useless and I've raised this topic internally when I joined the team six months ago. The issue is we already have specifications that require the custom fields out there and implementations that comply with them. If we now remove them, there'll be a compatibility problem. If we then reintroduce them later once state can be encrypted, there'll be another compatibility problem. So I'm not sure how we could feasibly remove the custom events again without causing even more pain. 😕

The intent to be able to decrypt them is very likely to break compatibility once that is possible, which is why imo it should be removed now. If you encrypt the event by keeping the event type plain text and the same, then the event will suddenly have a different content. That is a breaking change. And if you encrypt it and encrypt the event type as well, then the event type doesn't matter, since you will need to implement a new field as well.
The problem is that having those events right now, while they "do nothing", has a permanent cost associated with it, since Matrix APIs are not designed with them in mind. This means if the TIM specification wants to support spaces, all homeservers need to be patched to also return the name field when using the /hierarchy APIs. Similarly for the room directory it is not possible to see the names for public rooms. And a lot of other edge cases, where the TIM spec doesn't provide consistent solutions for them. Removing those event types (with a migration over several steps) is imo easier than specifying all those edge cases. One could as a first step allow using both (m.room.name and de.gematik.tim.room.name), where they should always have the same value with the intent to remove them in a future version. This does cause friction, but I think that is easier than fixing all the cases where those events are causing problems right now.
And once encrypted state events are possible, a lot more stuff will have to change, so the current solution is exactly as future proof (imo) as m.room.name is.

And a lot of other edge cases, where the TIM spec doesn't provide consistent solutions for them.

Chapter 5 has been streamlined a little in the latest draft release. The idea was that TIM is not going to list all the various cases in which servers need to evaluate the custom events rather than the standard events. Instead A_25820 just makes it a blanket SHOULD requirement.

One could as a first step allow using both (m.room.name and de.gematik.tim.room.name), where they should always have the same value with the intent to remove them in a future version. This does cause friction, but I think that is easier than fixing all the cases where those events are causing problems right now.

Let's assume in the next release (probably some time in 2025) we add a new requirement for clients to always send both types of events with equal content. We already have implementations out in the wild that only send the custom events. So new implementations would still have to support the case where only the custom events are filled with content. Once the old implementations exceed their certification (3+ years from today), we can then make another release and remove the obligation to send and evaluate the custom events.
This seems fine per se. But then since the transition period is 3 years you're still going to have to fix all the cases where the event types interfere on the server. We'd only actually improve things for people starting an implementation in 3 years. Maybe that is still worth it. I'm just trying to call out that the current problems would still have to be solved in the meantime.

I've re-raised this topic internally and am waiting for feedback and opinions.

Against all odds, I've been able to get the first step (sending the custom and standard events with identical values) into this week's patch release: https://gemspec.gematik.de/docs/gemSpec/gemSpec_TI-M_Basis/gemSpec_TI-M_Basis_V1.1.1/#<IP_ADDRESS>


workspaces: allow filtering out files from workspace root

I want to move tvix from a separate views/tvix/workspace.josh file to a workspace.json file shipped inside the directory, mostly to remove filter ambiguity as described in https://github.com/josh-project/josh/issues/1155#issuecomment-1449047761, as well as aligning more with how josh seems to be used elsewhere.
However, I need to exclude some files from the tvix directory the workspace.josh would then be in. Specifically, most default.nix files in there should be excluded (see here).
It seems it's not possible to customize the implicit :/<workspace root> filter, so that certain files are excluded, at least not while just using workspace=tvix as the only filter parameter (during clones).

Would something like ,,:exclude[::path/to/workspace/path/to/file]:workspace=path/to/workspace do what you want?

I think it would, but the URL would become quite messy. We want to exclude default.nix and **/default.nix (except these inside tvix_tests) from the workspace.
I don't want to push the burden of specifying these filters to people cloning it, this should be possible to specify from workspace.josh itself.

And an exclude filter inside the workspace.josh doesn't work, even making sure to target the files as if the workspace folder was the root?
(That might well be, especially since the change to how we compose this implicit filter with the workspace one, I don't have my computer to check right now.)

Another option right now could be to add this filter as an implicit prefix to all clones. There is a CLI option for this.  That would mean you can never get those files through josh tho.
Be aware this prefix only applies to git clones, not to graphql queries.

Currently there is no way to do this unfortunatly. It is good to know though that you are having this use case as we are investigating a new feature for workspaces anyway that will allow "pre-selecting" files from the tree before applying general filters. So knowing about this we can make sure your case is covered.

The other way around we have it currently also causes problems - people checking out a workspace, and then pushing back their commits now always include a change to a workspace.josh file - see https://cl.tvl.fyi/c/depot/+/9985.
This was a plain commit adding a foo.txt into the workspace checkout:
commit f07638db7ba5b7082a9d4c7ba15b67ec9889530b (HEAD -> canon)
Author: Florian Klink<EMAIL_ADDRESS>Date:   Mon Nov 6 13:56:06 2023 +0200

    test - please ignore.

    Change-Id: I28568b95b9c2f72e977a7d048423bb4730c1733f

diff --git a/foo.txt b/foo.txt
new file mode 100644
index 0000000..9daeafb
--- /dev/null
+++ b/foo.txt
@@ -0,0 +1 @@
+test

(using https://code.tvl.fyi/depot.git:workspace=views/tvix.git at 9b90ccb7089b1e21763126336125cbd354c20f00).
On the other side (in Gerrit), this showed up as a commit also editing views/tvix/workspace.josh to an earlier version of it - even though the change changing the file is already part of 9b90ccb7089b1e21763126336125cbd354c20f00.


Convertions

Description
Please include a summary of the change and state which issue is fixed or which new feature implemented. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #139
Feature ColorMap in image Processing

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality not to work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Also, include screenshots for verification and reviewing purpose.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented on my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Can you add some screenshots to verify this?

@hs05june can you add these screenshots?


New version: dotenvx.dotenvx version 1.32.0

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.9 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Manual Validation ended with:

2024-12-27 09:43:05.454 [FAIL] Installer failed security check. Url: https://github.com/dotenvx/dotenvx/releases/download/v1.32.0/dotenvx-1.32.0-windows-x86_64.zip Result: 0x80004005

(Automated response - build 931.)

@wingetbot run


Create DELETE dataset/{dataset_id}/genesets endpoint

Need
An endpoint to delete all genesets for a dataset
Approach

If the dataset is linked to a private collection, and the user is the owner of the collection, delete all genesets linked to a given dataset
Need to also delete the csv from s3

DOD

Return a 204 on success
If the user is not the owner of the collection return a 403
If the collection is not private return a 405

Closing. Per the 2022 Q1 plan, the design would embed gene sets in datasets.


Improve startup time

Ideas:

Start worker only on first .ts file.

+1
The TS package is the largest contributor to my package loading time, as reported by the TimeCop command.
I have a total of 105 packages, loaded in 675ms, of which atom-typescript consumes 584ms

+1
Currently adds 4510ms to my startup time

+1

It adds 87601ms to my startup time

Maybe you can improve the performance by inititalizing when a .ts file is opened or a project with tsconfig is opened

@prijindal that's absurd, I'm getting one to ten seconds

Any idea what's causing this delay?

Adds 4191ms to my startup. Wish there'd be a fix already. Atom isn't the fastest itself.

Bump for attention
Using this plugin on moderately sized TS projects can easily add 20s+ to startup time from this package alone. Not sure how such a mature and widely used package is so unoptimized, surely there's people with bigger projects than me who would have 40s+ startups due to this?

Closing this for housekeeping purposes. FWIW, this should've been fixed some time between v10 and v12. Feel free to open new issues if still experiencing this.


As an analyst, I want to navigate between records with comments, in order to find records more quickly.

Acceptance criteria:

[ ] Go to next record /w comment
[ ] Go to previous record /w comment

Can now navigate between records with comments using:

Previous comment  : Ctrl+Shift+Minus
Next comment  : Ctrl+Shift+Plus


UTF8 characters being displayed when shouldn't be

Azure Data Studio Version:
Version: 1.22.1
Commit: 1b5c54dd8c6eb03bf7ed1e04889d9f41de0523b3
Date: 2020-09-29T16:38:23.291Z
VS Code: 1.49.0
Electron: 9.2.1
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.4.0-51-generic

Steps to Reproduce:

Create the following function in a database that does not use a UTF8 collation.
END

GO

Go back to Sandbox and runt he 2 functions, the latter with 3 part naming:

USE Sandbox;

SELECT dbo.Bin2UTF8(0x48656C6C6F20F09F988A);
GO

SELECT UTF8.dbo.Bin2UTF8(0x48656C6C6F20F09F988A);

Both queries return 'Hello 😊', however, the first result should not be changed.

This only occurs in ADS. SSMS does not have this behaviour.
ADS:

SSMS:

As 1.23.0 came out after making the above Issue, I tested in that, same issue.


Publication in Journal of Open Source Software (JOSS)

@rmarkello and @danjgale - what do you guys think about publishing this toolbox in a short paper?
I haven't done it but I thought of the "Research and Ideas" (RIO) paper or perhaps the F1000, as mentioned by Ariel or was it Cameron? I imagine something short as this: https://riojournal.com/article/12342/
We can also just package into a zenodo thingy, but I like RIO or F1000 more as it is also considered by google scholar.

I think this is a great idea!! Super interested in being involved if I can be of help!✨✨

Excellent idea! I'm all for it

Perfect, we just need to figure out which format makes the most sense, i.e. RIO or F1000 (or something completely different?). What do you guys think?
Considering functionality, I think it's only "missing" that it reads out multiple peaks per cluster (as described in https://github.com/miykael/mni_atlas_reader/issues/23).
Or is there something else missing?

I mentioned RIO and F1000, but we could also publish in JOSS. I think it would be a perfect journal for such a software. What do you guys think?

@PeerHerholz - I've just updated your affiliation.

Sorry for the slow reply! This looks awesome and ready to go 🚀 🌟 ✨ 🌙

Looks great, really pleased with it! Good to submit imo

Perfect! Thanks for all the feedback, I will close this issue and start the submission process.


Fix (assumed) typo

not a typo, librarian creates a .tmp folder

not a typo, librarian creates a .tmp folder

That was rather my point :-) The text currently says that librarian creates a .tmp folder, but adds a tmp (no dot prefix) to the .gitignores. That seems like a typo to me.

doh, misread it sorry, and thanks!


Add Github Action to publish new charts

Action is configured to run any time an updated Chart.yaml is pushed main.
Essentially identical to the manual step listed in the README. There is a tool released by the Helm team, chart-releaser, that handles this in a similar way that is probably worth looking into more closely at some point. I played around with it but it wasn't clear to me that it added much value given how we currently publish new charts. I also couldn't get it to play nice.
TODO

Check to make sure you've run package-chart.sh (look for .tgz file with same version)
Run helm lint in the same action?

Both of your TODOs look actionable for this PR. What do you think of including them?

Both of your TODOs look actionable for this PR. What do you think of including them?

Sounds good to me. I just wanted to get this PR up before going too much further down any implementation, in case I was unwittingly doing a no-no.

@miker985 Updated to include a somewhat roundabout way of generating and linting the manifest files.
Tested this same action in an identical (separate) repo, and got the expected result using a known-bad file:

Also tested with a good file, and it passed. Let me know if you have any questions or suggestions.

Sorry to stick my nose in randomly.
Tom set up the gh-pages way of publishing charts as a temporary workaround when the Google Artifact Registry for Helm charts had a bug and was not yet working. I would strongly encourage switching to publishing charts in an actual chart repo and not perpetuating gh-pages hackery.

No worries, happy to switch away in that case!

@EricHorst can you comment on whether you're encouraging

something resembling the GAR "Store Helm charts" quickstart
The terraform setup @thomasjohns shared with us (which appears to be a major rework of how all of this is structured)
something else

Can you also quickly confirm that

there is related work with regard to fluxd would be to e.g., update this helm-chart-repository.yml in our dev setup and then, once we're confident it's working, make similar updates to eval/prod
any other bits of related work you understand need to be done (all I'm aware of is ☝ )


[ALLUXIO-3063] Implement common options for client and server

https://alluxio.atlassian.net/browse/ALLUXIO-3063

alluxio-bot, check this please

@calvinjia @aaudiber

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17657/
Test PASSed.

@aaudiber Thanks! I removed the inheritance.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17667/Failed Tests: 6org.alluxio:alluxio-tests: 6alluxio.cli.fs.command.TailCommandIntegrationTest.alluxio.cli.fs.command.TailCommandIntegrationTestalluxio.cli.fs.command.TestCommandIntegrationTest.alluxio.cli.fs.command.TestCommandIntegrationTestalluxio.client.ReadOnlyMountIntegrationTest.alluxio.client.ReadOnlyMountIntegrationTestalluxio.master.file.ConcurrentFileSystemMasterCreateTest.alluxio.master.file.ConcurrentFileSystemMasterCreateTestalluxio.master.file.FileSystemMasterIntegrationTest.alluxio.master.file.FileSystemMasterIntegrationTestalluxio.master.file.ConcurrentRenameIntegrationTest.alluxio.master.file.ConcurrentRenameIntegrationTest
Test FAILed.

Jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17671/
Test PASSed.

Merged build finished. Test PASSed.

Currently, there is no enforcement that every option has a common option. One way we could do it is via an interface for common options getters and setters?
And, there was no code gen for these java options...

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17689/
Test PASSed.


Internet-Connection Details

@duafatimaa , just approved these. Start giving this link out to everyone

Okay Sir, Today I will start giving this link out to everyone.


About "filament-sensing.g" and "pulse-counting.g"

Long time no see!
Nowadays I'm trying this project. And I have some problems while trying to read the code.
I noticed that you didn't use the toolhead in ERCF,cause I found nothing with the “hall switch” or "micro switch",maybe you used the pulse-counting.g file and compare the pulse number to do a work similar to the switch? The code as I understand it is this, I want to confirm my point.
I've read almost all the code, and I'm trying to write an explanatory document about ERCF and ERCFonRRF to help more RRF user，and now I have done part of the work.
Look forward to your reply！

And I wonder what's the difference between these two files.
I understand the difference between these two files are the "pulse-counting.g" file configed a trigger to count the pulse value , just like I mentioned above. The "filament-sensing.g" file does not have this function, but I don't know anything else.
Is that so？

The toolhead sensor is unused by this implementation. The way is similar to what is currently being developed in klipper for ercf's "sensorless detection"
As for filament-sensing vs. pulse-counting; pulse-counting is used for ERCF operations. filament-sensing is used for printing runout/clog/jam/tangle detection

Copy that. I'll read this code a little more.Thank you!


Update CommandTester.php

Add a way to access the command to create command specific tests related to the given command

I haven't seen any way to unit test the logging system while injecting logger into the command. So I'm thinking of accessing the Command->getLogger() to access the logging system and then access all the logs written in the command.
An other addition might be to add proper documentation in unit testing the Command output when combined with Monolog.


Investigate usefulness of creating a scivision docker image

@acocac noticed with a scivision plugin package he is creating that a because a requirement is detectron2, he needed things like PyTorch and torchvision pre-installed for this to work correctly.
I wonder if in the long run, we should also have a scivision docker image as well as the python package, which includes the python package, but also useful stuff like CUDA and PyTorch etc that are difficult to install?
@ots22 I would be interested to hear your opinion on this

Scivision models are separate from 'core' scivision, and can be completely arbitrary. A scivision image (or images) would have to make choices about which libraries to include, and their versions, which would necessarily not support every model!

We were discussing yesterday making scivision not so completely model-agnostic, and instead offering an interface to a number of popular frameworks for model authors to build on.  We could make sure these are supported by an image, at a minimum.
@miquelmassot


generated function

Not an issue, but, reading the code, I was just wondering why size was a generated function here
@generated function size{T,N,TCoefs,IT,GT,pad}(itp::BSplineInterpolation{T,N,TCoefs,IT,GT,pad}, d)
    quote
        d <= $N ? size(itp.coefs, d) - 2*padextract($pad, d) : 1
    end
end

Isn't it exactly equivalent to this version?
function size{T,N,TCoefs,IT,GT,pad}(itp::BSplineInterpolation{T,N,TCoefs,IT,GT,pad}, d)
    d <= N ? size(itp.coefs, d) - 2*padextract(pad, d) : 1
end

Yeah, it probably is. Probably an editing mistake, can you submit a PR?

Github's hooks didn't pick that up, but this was fixed in 22bf138.

To make it happen automatically, just put fixes #77 in the commit message somewhere.

Yeah, the PR had solves issue #77 in it, so I thought that would be enough, but I guess the "issue" in between confused it :)

It's a simple-minded little bot :smile:


Pierreroman profile add

OPS Build status updates of commit 72c5e3f:
:clock10: Preparing: average preparing time is 40 sec(s)

OPS Build status updates of commit 72c5e3f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/cloud-developer-advocates, working directory: W:\uwlz-s, target branch: master, target branch commit: 8b592e1ad2563a502713f089facc1b918ba2bcad, source branch: b34010c605564a77a4859af1674f1228/pierreroman-profile-add, source branch commit: .

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Test QR code with Bank Linth iOS app

Thank you!

Could you repeat your tests with the codes of v0.1.4 please? https://github.com/stapelberg/qrbill#qrbill-v014-2020-sep-21


⚠ ClamAV has degraded performance

In 282e5df, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in 2ccc14c.


COUR-2487 region based locations can no longer have duplicate ID's

Additional notes:

UKPRN now passed through as an integer
Each Provider gets a specific 4 digit export ID (0-9999) which forms the prefix of all location ID's
Address based locations get a specific 4 digit export ID (1000-1999) which is affixed to the prefix
Region based locations get a specific 4 digit export ID (2000-2999) which is affixed to the prefix
This guarantees that all locations share a context with the Provider, and that none are duplicated accross providers as per the original export.

CAVEATS:

supports 9999 providers only
supports 999 address based locations only
supports 999 region based locations only


Change language

How do I change the language using the resource libraries? Thanks.
Originally posted: 2015-07-07T10:45:40

To use one of the supported languages (en, fr, es), just change the culture in your main program (usually in Program.cs).
System.Threading.Thread.CurrentThread.CurrentCulture = new System.Globalization.CultureInfo("es");
System.Threading.Thread.CurrentThread.CurrentUICulture = new System.Globalization.CultureInfo("es");

If instead you're asking how to use a different language entirely, then you will need to work with me to complete a small translation of the resources into the language of your choice.
Originally posted: 2015-07-08T08:16:16


Update core version on the v1.39.x branch

I also mistakenly left this out from #26596. It looks like there have been core API changes in event engine and core credentials APIs since v1.38.x, so this bumps the core major version to 18.0.0

python windows: https://github.com/grpc/grpc/issues/25124
RBE MSAN: internal issue b/192270307
cloud to prod: https://github.com/grpc/grpc/issues/26595


Subscribed zero state tweaks

          1 Warning

:warning:
Please provide a linked tracker story for this PR in the description

  Generated by :no_entry_sign: Danger


Make Feature Researcher creation more of a form for the administrator.

Include User ID
Include Personal title auto filled by LDAP, but editable
Include photo
Include Introduction
ScholarSphere Use Link
Addition Information

We need to add an additional table in ActiveRecord to store all the elements separately and build the display from the items each time it is viewed. Form, Controller, DB record, and View.


Android 4.1.2 API Webview not loading map

I need to do Mapping in the Android 4.1.2 API version.
I have edited my index.html file as [https://leafletjs.com/examples/mobile/] on this link.
It works very well on Android 8, but does not work in version 4.1.2.
Although I added the parameter Window.L_DISABLE_3D = true as a global variable, I still failed to view it.
I have Webview xml inde error or Leaflet 4.1.2 version does not support?
java code:
final WebSettings webSetting = webView.getSettings ();
                        websetting.setbuiltınzoomcontrols (true);
                        websetting.setjavascriptenabled (true);
                        webView.addJavascriptInterface (new JavaScriptInterface (), "AndroidInterface");
                        webView.setWebViewClient (new android.webkit.WebViewClient () {
                            in @Override
                            public boolean shouldOverrideUrlLoading (WebView view, String url) {
                                view.loadurl (url);
                                return false;
                            }

                            in @Override
                            public void onPageStarted (WebView view, String url, Bitmap favicon) {
                                super.onPageStarted (view, url, favicon);
                            }
                            in @Override
                            public void onPageFinished (WebView view, String url) {
                                super.onPageFinished (view, url);
                            }
                        });

                        webview.loadurl ( "file: ///android_asset/index.html");

Make a Delayed Painting


Need a LICENSE.

This should probably be what is in ManageIQ/manageiq initially, since this was extracted from there.

@simaishi Can you please do this when you have a chance?


Add attribute for damage modifier

Since there are a number of things that modify the Ability modifier for damage without affecting the ability itself maybe that should be an attribute.  For example Attack Stance doubles the Ability modifier to Melee damage.  I can set up an effect to increase the Melee value but there's no way to bump just the additional damage.

You're right. Thanks for bringing this up. I'll add the data-point

This should be addressed in the most recent update. Note, the new data point controls a bonus to the damage modifier, not the damage modifier itself. This is because the value of the damage modifier is dependent on the value of the ability itself, and is thus calculated dynamically in the following way: ability-score + ability-damage-modifier-bonus. Thus, if you want to do something like double an actor's effective damage modifier, you would have to add the value of the ability score once more. See the attached screenshot.


PlasmidFinder database

Staramr manual says "By default, both databases are used" for --plasmidfinder-database-type. How can I select both databases at the same time? I mean I want staramr to look for in gram_positive and enterobacteriaceae databases, not just one.
Thanks.

Using both databases is the default behavior. So there's no need to pass anything on the command-line to select both of them.

All right, I understood. Now I got it! Thanks @apetkau


Final! Week 11 (Due Dec. 13) to-do

Wrap up, write the necessary doc, prepare for the presentation

@JornsenChao Can you specify how to get raw_terrain_fn = "example_dem.tif". Or if it's a click of download, please upload it in 'data'.

@JornsenChao I wasn't able to keep working because of missing raw files. Unable to fill the gap of height_acquire. Additionally, progressing with the 'geo_process' is not feasible without the raw files, as their structure is crucial for the process.
'''read existing files'''
building, extent, weather, boundary, terrain (set crs to 4326)
raw_building = gpd.read_file(raw_building_path)
raw_extent = gpd.read_file(raw_extent_path)
raw_weather = pd.read_csv(raw_weather_path) (I think we have this already)
raw_boundary = gpd.read_file(raw_boundary_path) (I think we also have this? the city limit?)

Please use this link to download the original geotif. My code includes trim the original tif with extent. (or can be the city boundary).
https://prd-tnm.s3.amazonaws.com/StagedProducts/Elevation/13/TIFF/historical/n48w123/USGS_13_n48w123_20230608.tif
I tried to upload but github won't allow big files
@LilacHo

Do we really need pylint workflow? I guess it is zero tolerance until it gets 10/10. My opinion is that if flake8 works the same, we don't need both. @krittin-kulrat @loriwon-uw

I think we should go with pylint and ditch flake8. The reason being, pylint does a more thorough job by flagging missing function documentation which is more important.

I understand. Can we use it as our own debugger and if it seriously cannot pass on Wed, we switch over? We can decide my suggestion later... But it gonna fail on my end forever. I just don't know how to fulfil all of its requests like "heat_island/height_acquire.py:236:0: R0914: Too many local variables (29/15) (too-many-locals)". If I genuinely addressed the issue of excessive local variables, it was necessary to reconstruct the function from scratch.

I think it's fine considering timeframe. I'm going to create a pull request so you can continue working on other stuff. Do you need any help? Here is my Zoom Link.

I am good at this point. Thank you for your offer! It is a function of around 200 lines of code so it's understandable to have too many local variables. I did try to split it into pieces (I promise I really tried) but for the sake of time, the most important is that it runs successfully instead of worrying about the warning.

I made a pull request, add functions to get building data and terrain data from a given study boundary.

https://drive.google.com/drive/folders/1LRoYHLRKRPzDxGWpCMemAKpcMFi9taGH?usp=sharing
here is the output file from these above functions. larger than 25mb so have to share through drive.
but I believe this module could get you these data.


@Scheduled doesn't work out of the box

I have a spring-boot project with tracer added (more details below). Configuration is just the default one, just the trace name is configured. I'm trying to get my trace value in a @Scheduled method, but I get IllegalStateException: X-Flow-ID has not been started.
Question: how is it supposed to work? I think the documentation on this is missing.
Description

I'm running on spring-boot:1.4.6.
I have only tracer-spring-boot-starter:0.16.0 in dependencies.
I have configured only this single property for tracer: tracer.traces.X-Flow-ID=flow-id
I have a @Scheduled method inside of which I'm calling tracer.get("X-Flow-ID").getValue() on an @Autowired Tracer tracer field.
Autowiring works, but I get the following exception:

java.lang.IllegalStateException: X-Flow-ID has not been started
	at org.zalando.tracer.DefaultTracer.getAndCheckValue(DefaultTracer.java:102) ~[tracer-core-0.16.0.jar:na]
	at org.zalando.tracer.DefaultTracer.access$000(DefaultTracer.java:15) ~[tracer-core-0.16.0.jar:na]
	at org.zalando.tracer.DefaultTracer$1.getValue(DefaultTracer.java:68) ~[tracer-core-0.16.0.jar:na]

Question
I didn't really find any docs of what a "support for scheduling" means. I'm sure I'm doing something wrong, but can you point out what's missing? I was expecting that the trace will be already initialized and I can just get it's value (and send in an outgoing request).

That sounds like the correct setup to me. Can you compare your setup to this test? Maybe there is an obvious difference somewhere? Do you register a custom TaskScheduler or something? Maybe you can put some breakpoints in the SchedulingAutoConfiguration  to see whether it's executed correctly.

I think the problem is in spring-boot version. The same code exactly works great on a fresh project running on spring-boot 2. It might be something else is different, but I don't really think so.

It works on an empty project indeed. I think the scheduler configuration doesn't work properly in this case. Not sure why, but I think the conditional taskScheduler bean is simply not created. In my log I see only:
org.springframework.scheduling.concurrent.ThreadPoolTaskScheduler - Initializing ExecutorService  'taskScheduler'

ANd no more relevant mentions of taskScheduler.
How is this supposed to work, if a bean with this name and type is already present?

Ah!
Anyway, maybe it would a great idea to somehow make this configuration not conditional, but make it extend whatever schedulers there are already in the context.
And if someone stumbles upon that thread some day: check if you already have a taskScheduler bean defined, guys! It ain't gonna work!!

Just got the same error. Had to copy paste bean definition from org.zalando.tracer.spring.TracerAutoConfiguration.AsyncAutoConfiguration#taskExecutor without condition. Worked, thanks

@innokenty it doesn't make sense to have both @Primary and @ConditionalOnMissingBean

@whiskeysierra  it doesn't make sense to have both @Primary and @ConditionalOnMissingBean


TOML Support

Overview
This discussion aims to explore the possibility of using .toml as the preferred format for saving API request data instead of .bru. For the progression of the Domain-Specific Language (DSL), see https://github.com/usebruno/bruno/discussions/360
I've been seriously exploring how we can represent api requests in toml
After careful consideration, factoring in various edge cases, toml seems to fulfil our requirements and might offer better long-term benefits.
Why TOML

Well established format
Utilize syntax highlighting in IDEs
Leverage existing tooling for toml file manipulation
Git providers (like GitHub, Bitbucket) offer syntax highlighting while reviewing pull requests on the web

Tasks

[ ] Experimental support for working with TOML in GUI
[ ] Experimental support for working with TOML in CLI
[ ] Gather Community feedback during our community call on Wed Jan 3rd 2024

More Tasks TBD based on how the experiment turns out
Looming Issues
Several features are currently blocked due to limitations in the bru language. Some include:

Describing headers
Managing choices for query parameters
File upload capabilities
Storing multiple payloads within a single request

To address these limitations, I see two potential paths:

Enhance Bru Lang
Utilize TOML

Next Steps
Before choosing a direction, I aim to implement experimental TOML support in our current app within the next 3-4 days (by January 1st, 2024). . BTW I am working on bruno Full Time from Jan 1st :)
Sample GET Request

I feel that TOML should be the way forward.  Would appreciate more articles on TOML vs bru as you move forward with the next steps.

any update on this ? this seems very interseting

Hey @helloanoop, just wanted to know did you also explore/consider YAML for the language?

Hey @helloanoop, just wanted to know did you also explore/consider YAML for the language?

My bad, went through #360 , indentation hell 🤔  interesting, I mean at one point it can become that but then at that point your API design is not good, It will be on the type of API imo.

I think TOML is a better format for humans. It's too easy to mess things up in YAML.

Any progress on this? There are so much bugs just going back to the custom bru DSL. It would be awesome to get rid of all of them. (e.g. the colon bug)

We will move very soon (max 2 weeks) to support YAML to save the requests
@lohxt1 on our team is already working on it.
There is a lot of nuance on TOML support, I will add detailed notes later
TLDR; - YAML support to arrive soon, TOML will need more time

@helloanoop my 2c is to stick with just one. If its yaml so be it. Who are very particular can setup conversion to/from toml
Maybe depreciate and support bru for a while but going down path of multiple config languages would be a pain

I do not think a custom language for settings a good idea.
Please try to stick to toml or yaml, do not reinvent the wheel.

Yeah. E.g. not allowing to have "colons" in your custom format for values is a bad joke. That is why inventing some custom configuration format is definitely a pain and just leads to bugs that normally would not be a problem with a basic standard format.

I agree one standard should be developed for. Bru is nice, with some issues. Personally I much prefer TOML over YAML as I find it easier to read.


Gem not releasing

https://github.com/metanorma/metanorma-plugin-datastruct/actions/runs/4644996886/jobs/8220586952
Something about MFA. Suspect I'm not even the gem owner, and GHA certainly doesn't think it is...

AFAIR we have two options here:

modify gemspec and set rubygems_mfa_required to false https://guides.rubygems.org/mfa-requirement-opt-in/
OR
Ask owners to enable MFA

@ronaldtse AFAIK we are did the second way for all our gems, I don't have access to do this, could you please help?

I will need to do manual release tomorrow if this is not resolved, it is dependency for standoc and is holding up release

@CAMOBAP please do the first option for now, i.e. rubygems_mfa_required should be set to false.

I've released accordingly. I thought I had disabled MFA my side.


Hello it's Babs!

Hey 👋  🐰 🐰 🐰  how are you doing?
:octocat: ❤

yo @babsbunny42 what's up?
☕  ⁉


Overriding user group not working

The following fails (Permission denied) even with lab image:
docker run -itP --rm --user 2019:2000 accetto/xubuntu-vnc:lab

Seems like only images build on Docker Hub have this problem. Locally build images work just fine. It'll be fixed in the next release.

Fixed in release 19.08.1


Can multiple stream uploads be active at the same time?

For example; there is a huge file upload that is going slowly in the first stream. Does that mean that all other streams, which might be significantly faster and more important, have to wait for the first one to finish, or may other streams be processed in parallel?
I am not sure if I am looking at the correct thing, but this excerpt might indicate that due to write lock, only one stream might be active at the time in either direction. Is this correct?
        private async Task<bool> SendInternalAsync(WatsonMessage msg, long contentLength, Stream stream, CancellationToken token)
        {
            if (msg == null) throw new ArgumentNullException(nameof(msg));
            if (!Connected) return false;

            if (contentLength > 0  && (stream == null || !stream.CanRead))
            {
                throw new ArgumentException("Cannot read from supplied stream.");
            }

            if (token == default(CancellationToken))
            {
                CancellationTokenSource linkedCts = CancellationTokenSource.CreateLinkedTokenSource(token, _Token);
                token = linkedCts.Token;
            }

            bool disconnectDetected = false;

            if (_Client == null || !_Client.Connected)
            {
                disconnectDetected = true;
                return false;
            }

            await _WriteLock.WaitAsync(token).ConfigureAwait(false);

            try
            {
                await SendHeadersAsync(msg, token).ConfigureAwait(false);
                await SendDataStreamAsync(contentLength, stream, token).ConfigureAwait(false);

                _Statistics.IncrementSentMessages();
                _Statistics.AddSentBytes(contentLength);
                return true;
            }
            catch (TaskCanceledException)
            {
                return false;
            }
            catch (OperationCanceledException)
            {
                return false;
            }
            catch (Exception e)
            {
                _Settings.Logger?.Invoke(Severity.Error,
                    _Header + "failed to write message to " + _ServerIp + ":" + _ServerPort + ":" +
                    Environment.NewLine +
                    e.ToString() +
                    Environment.NewLine);

                disconnectDetected = true;
                return false;
            }
            finally
            {
                _WriteLock.Release();

                if (disconnectDetected)
                {
                    Connected = false;
                    Dispose();
                }
            }
        }

Correct, you don't want multiple concurrent writers to a stream. The data would then be interleaved. You would want to use multiple separate connections.


Flutter_compass issue in apk build

FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':flutter_compass'.

Could not create an instance of type com.android.build.api.variant.impl.LibraryVariantBuilderImpl.
Namespace not specified. Specify a namespace in the module's build file. See https://d.android.com/r/tools/upgrade-assistant/set-namespace for information about setting the namespace.

 If you've specified the package attribute in the source AndroidManifest.xml, you can use the AGP Upgrade Assistant to migrate to the namespace value in the build file. Refer to https://d.android.com/r/tools/upgrade-assistant/agp-upgrade-assistant for general information about using the AGP Upgrade Assistant.

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
Get more help at https://help.gradle.org.

BUILD FAILED in 1s
Running Gradle task 'assembleRelease'...                         1,569ms
Gradle task assembleRelease failed with exit code 1

Same issue Here!

same here

FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':flutter_compass'.

Could not create an instance of type com.android.build.api.variant.impl.LibraryVariantBuilderImpl.
Namespace not specified. Specify a namespace in the module's build file. See https://d.android.com/r/tools/upgrade-assistant/set-namespace for information about setting the namespace.

 If you've specified the package attribute in the source AndroidManifest.xml, you can use the AGP Upgrade Assistant to migrate to the namespace value in the build file. Refer to https://d.android.com/r/tools/upgrade-assistant/agp-upgrade-assistant for general information about using the AGP Upgrade Assistant.

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
Get more help at https://help.gradle.org.

with the latest flutter version:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.3, on macOS 14.6.1 23G93 darwin-arm64, locale it-IT)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.1)
[✓] VS Code (version 1.93.1)
[✓] Connected device (6 available)
[✓] Network resources

If you need a solution use this in your pubspec.yaml:
 flutter_map_location_marker:
    git:
      https://github.com/dariocavada/flutter_map_location_marker

I create this fork with the updated compass (dchs_flutter_compass)

that fixxed it for me. Thanks!

If you need a solution use this in your pubspec.yaml:
 flutter_map_location_marker:
    git:
      https://github.com/dariocavada/flutter_map_location_marker

I create this fork with the updated compass (dchs_flutter_compass)

That fixxed it for me. Thanks!

For anyone facing this issue, consider trying v10.0.0-beta.0. In this new version, FMLM has dropped the flutter_compass dependency and now uses flutter_rotation_sensor.
The flutter_rotation_sensor offers two great features: customizable update intervals and coordinate transformation. Additionally, you can use any device heading provider; flutter_rotation_sensor is just the default implementation. Check out the custom stream example to learn how to set it up.

For anyone facing this issue, consider trying v10.0.0-beta.0. In this new version, FMLM has dropped the flutter_compass dependency and now uses flutter_rotation_sensor.
The flutter_rotation_sensor offers two great features: customizable update intervals and coordinate transformation. Additionally, you can use any device heading provider; flutter_rotation_sensor is just the default implementation. Check out the custom stream example to learn how to set it up.

This creates a new error for me when trying to build the ios app.
Swift Compiler Error (Xcode): Cannot find 'RotationSensorPlugin' in scope
/Users/builder/.pub-cache/hosted/pub.dev/flutter_rotation_sensor-0.1.0/ios/Classes/FlutterRotationSensorPlugin.swift:14:19

Swift Compiler Error (Xcode): Value of type 'CMDeviceMotion' has no member 'orientation'
/Users/builder/.pub-cache/hosted/pub.dev/flutter_rotation_sensor-0.1.0/ios/Classes/FlutterRotationSensorPlugin.swift:45:35

Any ideas?

For anyone facing this issue, consider trying v10.0.0-beta.0. In this new version, FMLM has dropped the flutter_compass dependency and now uses flutter_rotation_sensor.
The flutter_rotation_sensor offers two great features: customizable update intervals and coordinate transformation. Additionally, you can use any device heading provider; flutter_rotation_sensor is just the default implementation. Check out the custom stream example to learn how to set it up.

This creates a new error for me when trying to build the ios app.
Swift Compiler Error (Xcode): Cannot find 'RotationSensorPlugin' in scope
/Users/builder/.pub-cache/hosted/pub.dev/flutter_rotation_sensor-0.1.0/ios/Classes/FlutterRotationSensorPlugin.swift:14:19

Swift Compiler Error (Xcode): Value of type 'CMDeviceMotion' has no member 'orientation'
/Users/builder/.pub-cache/hosted/pub.dev/flutter_rotation_sensor-0.1.0/ios/Classes/FlutterRotationSensorPlugin.swift:45:35

Any ideas?

I'm assuming you have imported the dependency? If not try flutter pub add  flutter_rotation_sensor:0.1.0. Else can you show some code where the error occurs?

flutter_rotation_sensor is updated to v0.1.1. The build issue of ios app should be fixed.

Waiting for flutter_rotation_sensor v0.1.1 to be merged into flutter_map_location_marker: ^10.0.0-beta.1


Add API for creating hierarchical PartitionKeys

[x] The purpose of this PR is explained in this or a referenced issue.
[x] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[x] Tests are included and/or updated for code changes.
[x] Updates to module CHANGELOG.md are included.
[x] MIT license headers are included in each file.

This PR adds a builder for creating PartitionKey structs with multiple keys in order to support hierarchical partitioning keys.

@microsoft-github-policy-service agree company="Boyum IT"

@simorenoh and/or @analogrelay - can you take a look at this when you get a chance?

The coverage tests are failing, but the error message doesn't seem quite right
FAIL
coverage: 84.4% of statements
FAIL	github.com/Azure/azure-sdk-for-go/sdk/data/azcosmos	49.481s
FAIL
Searching for coverage files in D:\a\_work\1\s
Reading config data from D:\a\_work\1\s\/eng/config.json
(data/azcosmos) Failing if the coverage is below 0.40
D:\a\_work\1\s\coverage.xml
Status: Succeeded	Coverage file: D:\a\_work\1\s\coverage.xml	 Coverage Amount: 0.8439


Support for non-personalized ads

Hey,
In some cases (like when a European user doesn't give its consent for some tracking cookies), it may be required to show only non-personalized ads. Would it be possible to add support for this in your plugin ?
I haven't checked your source code yet, some I cannot submit a pull request, but here are some links :

Android implementation.
iOS implementation.

If you're okay and if you don't have the time to work on this, I may be able to send a pull request.
Thanks !

The links you posted are the legacy ones, so they will no longer be valid in a near future. The current way to request authorization for users are in the documentation:

Request App Tracking Tranparecy Authorization for iOS
EU Consent 🔗
Complying with the Precise Location Data Policy

Can this be closed?

EU Consent 🔗

The biggest problem with this one, is that Funding Choices has a big caveat that stops ads from being served. Therefore the only workaround is to use that npa parameter. In fact, even Google added a such parameter in their brand new plugin (link here).
But that's up to you, it's your project after all 😁

Hmm this is interesting. I've read the whole admob documentation, from top to bottom, and I never saw anyone ever mention the nonPersonalizedAd. I took a look at google-mobile-ads source and here's how they implement it:
Android:
    if (nonPersonalizedAds != null && nonPersonalizedAds) {
      final Bundle extras = new Bundle();
      extras.putString("npa", "1");
      request.addNetworkExtrasBundle(AdMobAdapter.class, extras);
    }

iOS:
            let extras = GADExtras()
            extras.additionalParameters = ["npa": "1"]
            request.register(extras)

Tho I don't have a mac, so I can't implement the iOS part, it depends on the community. You can join my discord server so we can talk better.


CreateAccountWithSeedParams error or not aligned with docs

Hey there :)
I have been experimenting with your lib and it works pretty well. Yet, I found a problem with the CreateAccountWithSeedParams tuple. According to the docs, the seed should be of type str, see here
While using the tuple and providing a string as seed I get the following error:
ValueError: not enough values to unpack (expected 2, got 1)
It turns out that I have to provide a dict in the following form: seed={"length": len(seed), "chars": seed},
There are two possibilities:

We update the docs
We fix it in the code. If str given, transform to dict. If dict given and contains both keys, great :)

What would you prefer?

Can you provide a code sample? I can't reproduce this error

Sure, here is an entire piece of code:
import yaml
import json
from os import path
from solana.rpc.api import Client
from solana.keypair import Keypair
from solana.publickey import PublicKey
from solana.transaction import Transaction, TransactionInstruction
from solana.system_program import create_account_with_seed, CreateAccountWithSeedParams

client = None
payer = None
greeted_public_key = None
program_id = None

LAMPORTS_PER_SOL = 1000_000_000

def getConfig():
    config_path = path.normpath(path.join(path.expanduser('~'), '.config', 'solana', 'cli', 'config.yml'))
    with open(config_path, 'r') as f:
        try:
            config = yaml.load(f, Loader=yaml.FullLoader)
        except FileNotFoundError as fnfe:
            print(fnfe)
            print('Please setup solana-cli config first')
            sys.exit(1)
    return config

def getRpcUrl():
    config = getConfig()
    if 'json_rpc_url' in config:
        return config['json_rpc_url']
    print("Failed to read RPC url from config file. Falling back to localhost")
    return "http://<IP_ADDRESS>:8899"

def getPayer():
    """
    Tries to read the secret key from the file referenced in the config file. If no file is found, a new keypair is generated.
    """
    config = getConfig()
    if 'keypair_path' in config:
        with open(config['keypair_path'], 'rb') as f:
            keypair = Keypair.from_secret_key(b"".join(map(lambda x: x.to_bytes(1, "big"), json.loads(f.read().decode('utf-8')))))
            return keypair
    print('Failed to create keypair from cli, falling back to new random keypair')
    keypair = Keypair()
    return Keypair()

def establishConnection():
    """
    Establishes a connection to the Solana cluster.
    """
    global client

    rpc_url = getRpcUrl()
    try:
        client = Client(rpc_url)
    except Exception as e:
        print(e)
        print("Failed to establish connection to RPC server")
        sys.exit(1)

def establishPayer():
    global payer

    fees = 0

    feeCalculator = client.get_recent_blockhash()["result"]["value"]["feeCalculator"]

    fees += client.get_minimum_balance_for_rent_exemption(1)["result"]

    fees += feeCalculator["lamportsPerSignature"] * 100

    payer = getPayer()

    balance = client.get_balance(payer.public_key)["result"]["value"]
    print(f"Balance for {payer.public_key} is {balance} lamports")

    if balance < fees:
        print("Not enough balance on account. Going to request an airdrop")
        tx = client.request_airdrop(payer.public_key, 2 * LAMPORTS_PER_SOL)

        print('Waiting for transaction to be finalized...')
        client.confirm_transaction(tx["result"])

        balance = client.get_balance(payer.public_key)["result"]["value"]
        print(f"Balance for {payer.public_key} is {balance} lamports")

if __name__ == '__main__':
    establishConnection()
    establishPayer()
    lamports = client.get_minimum_balance_for_rent_exemption(1)["result"]

    program_id = Keypair().public_key
    test_seed = "hello"
    greeted_public_key = PublicKey.create_with_seed(payer.public_key, test_seed, program_id)

    transaction = Transaction().add(
        create_account_with_seed(CreateAccountWithSeedParams(
            from_pubkey=payer.public_key,
            base_pubkey=payer.public_key,
            #seed={"length": len(test_seed), "chars": test_seed},
            seed=test_seed,
            new_account_pubkey=greeted_public_key,
            lamports=lamports,
            space=1,
            program_id=program_id
        ))
    )
    tx = client.send_transaction(transaction, payer)
    client.confirm_transaction(tx["result"])

I had to install following packages:
pip install solana pyyaml
I am running a local test validator:
solana-test-validator
That's why I am reading the key and url in. If you need more details, let me know :)
Once I run this code, I get the following error:
Traceback (most recent call last):
  File "main.py", line 101, in <module>
    create_account_with_seed(CreateAccountWithSeedParams(
  File "/Users/jacek.varky/project/solana/Solana-Intro/PR_code/venv/lib/python3.8/site-packages/solana/system_program.py", line 491, in create_account_with_seed
    data = SYSTEM_INSTRUCTIONS_LAYOUT.build(
  File "/Users/jacek.varky/project/solana/Solana-Intro/PR_code/venv/lib/python3.8/site-packages/construct/core.py", line 336, in build
    self.build_stream(obj, stream, **contextkw)
  File "/Users/jacek.varky/project/solana/Solana-Intro/PR_code/venv/lib/python3.8/site-packages/construct/core.py", line 348, in build_stream
    self._build(obj, stream, context, "(building)")
  File "/Users/jacek.varky/project/solana/Solana-Intro/PR_code/venv/lib/python3.8/site-packages/construct/core.py", line 2144, in _build
    buildret = sc._build(subobj, stream, context, path)
  File "/Users/jacek.varky/project/solana/Solana-Intro/PR_code/venv/lib/python3.8/site-packages/construct/core.py", line 2657, in _build
    return self.subcon._build(obj, stream, context, path)
  File "/Users/jacek.varky/project/solana/Solana-Intro/PR_code/venv/lib/python3.8/site-packages/construct/core.py", line 3924, in _build
    return sc._build(obj, stream, context, path)
  File "/Users/jacek.varky/project/solana/Solana-Intro/PR_code/venv/lib/python3.8/site-packages/construct/core.py", line 2144, in _build
    buildret = sc._build(subobj, stream, context, path)
  File "/Users/jacek.varky/project/solana/Solana-Intro/PR_code/venv/lib/python3.8/site-packages/construct/core.py", line 2657, in _build
    return self.subcon._build(obj, stream, context, path)
  File "/Users/jacek.varky/project/solana/Solana-Intro/PR_code/venv/lib/python3.8/site-packages/construct/core.py", line 2133, in _build
    context.update(obj)
  File "/Users/jacek.varky/project/solana/Solana-Intro/PR_code/venv/lib/python3.8/site-packages/construct/lib/containers.py", line 117, in update
    for k,v in seqordict:
ValueError: not enough values to unpack (expected 2, got 1)

@jacekv I think this issue was fixed here:
https://github.com/michaelhly/solana-py/commit/21228ef9ace459d06b294d8f5919f6a165f779b3
@kevinheavey would you mind creating a patch release?

https://github.com/michaelhly/solana-py/pull/215#issuecomment-1084976583


Installing native applications (<IP_ADDRESS>.1.1)

It can be system-wide by default unless user specifies override path?

I prefer this to be specified explicitly, running npm i in a directory would surprise you if it put things outside the directory. Same with our tool, lets be specific with our -g.


[RUM-1830] 128 bit trace IDs

What does this PR do?
Introduces 128 bit trace IDs.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)
[ ] If this PR is auto-generated, please make sure also to manually update the code related to the change

I have changed the implementation, here is how it works now:
Separating Span ID and Trace ID
The TracingIdentifier constructor is now private.
It can only be created through TracingIdentifier.createSpanId() or TracingIdentifier.createTraceId(), and it returns different types respectively.
Span IDs will always be generated as 64 bits BigInt(s) and Trace IDs as 128 bits BigInt(s)
String representation takes into account the type of Tracing Identifier
TracingIdFormat (previously TracingIdRepresentation) is now more generic, and the .toString implementation has been modified accordingly.
When requesting a padded HEX value, the bit length of the current type will be taken into account.
Requesting the low bits of a 128 bit Trace ID will get you a 64 bit ID, while requesting the low bits of a 64 bit Span ID will get you a 32 bit ID.
Here is a runnable example.
Here is an example output for reference:

FORMAT
VALUE

Trace ID FULL HEX
667ed836000000000d8dbaedc8adcf59

Trace ID PADDED FULL HEX:
667ed836000000000d8dbaedc8adcf59

Trace ID HIGH HEX
667ed83600000000

Trace ID PADDED HIGH HEX:
667ed83600000000

Trace ID LOW HEX
d8dbaedc8adcf59

Trace ID PADDED LOW HEX:
0d8dbaedc8adcf59

Span ID FULL HEX
378ad79b0519da05

Span ID PADDED FULL HEX
378ad79b0519da05

Span ID HIGH HEX
378ad79b

Span ID PADDED HIGH HEX
378ad79b

Span ID LOW HEX
519da05

Span ID PADDED LOW HEX
0519da05

I will add a few more tests to match the RFC specs more accurately, and verify @ganeshnj's comment https://github.com/DataDog/dd-sdk-reactnative/pull/680#pullrequestreview-2141204325.

Example of generated headers:

Header
Value

x-datadog-origin
rum

x-datadog-sampling-priority
1

x-datadog-trace-id
17731188736533748218

x-datadog-parent-id
17034905968583185706

x-datadog-tags
_dd.p.tid=6682b2ee00000000

traceparent
00-6682b2ee00000000f611d636468059fa-ec6824b93f95152a-01

tracestate
dd=s:1;o:rum;p:ec6824b93f95152a

b3
6682b2ee00000000f611d636468059fa-ec6824b93f95152a-1

X-B3-TraceId
6682b2ee00000000f611d636468059fa

X-B3-SpanId
ec6824b93f95152a

X-B3-Sampled
1


Mimic pinns namespace path layout

What type of PR is this?
/kind feature
What this PR does / why we need it:
CNI expects namespaces available under /var/run/netns, which means we now change the overall layout of the namespaces to match pinns.
Which issue(s) this PR fixes:

Fixes https://github.com/containers/conmon-rs/issues/1065
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

Mimic pinns namespace path layout for `conmonrs pause`

Codecov Report

Merging #1067 (42e8c47) into main (8306edd) will decrease coverage by 0.18%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1067      +/-   ##
==========================================
- Coverage   33.39%   33.21%   -0.18%
==========================================
  Files          13       13
  Lines        1126     1126
  Branches      390      391       +1
==========================================
- Hits          376      374       -2
  Misses        494      494
- Partials      256      258       +2

LGTM, thanks!

@rphillips @haircommander I'd say we merge this and iterate on the open issues

/lgtm


Can I write the Fibonacci sequence in cue?

What version of CUE are you using (cue version)?

$ cue version
cue version v0.4.3 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?

For the following code snippet (fib.cue), I run with cue export fib.cue
#Fib: {
    n: int,
    if n <= 2 {
        value: 1
    }
    if n > 2 {
        _n1: #Fib & {
            n: n - 1
        },
        _n2: #Fib & {
            n: n - 2
        }
        value: _n1.value + _n2.value
    }
}
fib8: #Fib & {
    n: 8
}

What did you expect to see?
{
    "fib8": {
        "n": 8,
        "value": 21
    }
}

What did you see instead?
cycle error:
    ../fib.cue:3:8
cycle error:
    ../fib.cue:6:8

It seems that for #Fib in different positions, cue treats them as the same instance?

import (
    "strconv"
)

#Fib: {
    n: int
    #seq: [int] * (n + 1)
    #fib: [Num=string]: {
        let N = strconv.Atoi(Num)

        val: int
        if N < 2 {
            val: N
        }
        if N >= 2 {
            val: #fib["\(N-1)"].val + #fib["\(N-2)"].val
        }
    }
    for idx, i in #seq {
        #fib: "\(idx)": _
    }
    value: #fib["\(n)"].val
}

fib8: #Fib & {
    n: 8
}

Generally speaking, you cannot do general recursion in CUE.
You can use an iterative method or the direct calculation like: https://github.com/hofstadter-io/cuetorials.com/blob/main/code/patterns/recursion/fib.cue

@verdverm, good point about the back reference. My answer above takes ~6 seconds on my machine if n=10000. However this iterated solution only takes ~0.5 second for n=10000. Perhaps by calculating forward, every dependency is already resolved, and thus doesn't need to unify a large graph? It's also much simpler:
import (
    "list"
)

#Fib: {
    n: int
    #seq: [for i in list.Range(0, n + 1, 1) {
        if i < 2 {
            i
        }
        if i >= 2 {
            #seq[i-1] + #seq[i-2]
        }
    }]
    value: #seq[n]
}

fib8: #Fib & {
    n: 10000
}

My final version, seems my method reaches the limits of floating point accuracy
import (
	"list"
	"math"
)

_num: float | *23.0 @tag(N,type=number)

_s5:  math.Pow(5.0, 0.5)
_Phi: (_s5 + 1.0) / 2.0
_phi: _Phi - 1.0

#FibN: {
	n: number
	f: math.Round((math.Pow(_Phi, n) - math.Pow(_phi, n)) / _s5)
}

#Fib1: {
    n: number
    #seq: [for i in list.Range(0, n + 1, 1) {
        if i < 2 {
            i
        }
        if i >= 2 {
            #seq[i-1] + #seq[i-2]
        }
    }]
    value: #seq[n]
		l: len(#seq)
}
#Fib2: {
    n: number
    #seq: [ for i, I in list.Range(1, n+1, 1) { (#FibN & {n: I}).f } ]
    value: #seq[n-1]
		l: len(#seq)
}

fibN: (#FibN & {n: _num}).f
fib1: (#Fib1 & { n: math.Round(_num) }).value
fib2: (#Fib2 & { n: math.Round(_num) }).value

if you remove the .value suffix on those last 2 lines, you can see the entire sequences

I'm curious if CUE, with the plan for arbitrary decimals, should not lose accuracy in this example (cc @mpvl @myitcv)
$ cue export fib.cue -t N=200.0
{
    "fibN":<PHONE_NUMBER>92510140037722000000000000000000,
    "fib1": 2.80571172992510140037617E+41,
    "fib2":<PHONE_NUMBER>92510140037722000000000000000000
}

//<PHONE_NUMBER>92510140037722
//<PHONE_NUMBER>92510140037617

Thank you for all, very amazing solutions.

@Peefy to take a step back: a core design principle in CUE is that computation and configuration should be strictly separated. In general, computation is better handled by a programming language. Also, the whole point of a configuration language is that there are benefits to declarative specification. From experience we know that specifying computation in a configuration language, and intermixing the two, results in a rather unmaintainable mess.
That said, we also know that, in practice, it is not possible to exclude computation from the configuration layer. Another approach is to allow users to specify their own functions, not unlike CUE's builtin functions, written in a programming language of choice. We plan to move into this direction with CUE.
The aim here is to keep CUE relatively simple and move the complexity to functions, not unlike how spreadsheets work.
We believe that wrapping computation with configuration, rather than wrapping data and configuration with computation, is a much more useful paradigm.

@mpvl. Thanks for your answer. I'm very much in favor of separating the configuration data itself from the computational logic. In addition, using a programming language to write CUE functions sounds like a very cool thing, maybe it is a language such as Go, Python, etc. ? I am very much looking forward to it.


If something needs to be fixed I will fix it. I dont like the idea of adding new features because there are too many in the program and its hard to maintain all of them. You can open a pull request if you want, I will most likely accept it.

          If something needs to be fixed I will fix it. I dont like the idea of adding new features because there are too many in the program and its hard to maintain all of them. You can open a pull request if you want, I will most likely accept it.

Originally posted by @shaked6540 in https://github.com/shaked6540/YoutubePlaylistDownloader/issues/427#issuecomment-2523878452

Hi,
Appreciate your time and support.
I thought to let you know that when using YTDownloader if one link (for video or playlist) pasted and click "download" button, the  progress bar... showing number videos and completion (%) bar does display. If I add the links to queue "Add to Queue" the progression does not display, at all. It creates confusion about whether download is working or not, no way to know if download is completed.
Also, for video and playlists it works fine but not for channels. It would be great if you can fix it too.
Many thanks advance.
-Enthu


Add frequency to make it looks more real.

Hi guys, this project is really cool! However, people can not always keep a ~400 days streak without a break off. So I have added some frequency feature, which makes the result more convincible.
For example, if I set the frequency to 0.80 (the default is 1.00):
from RockStar import RockStar

rock_it_bro = RockStar(frequency=0.80)
rock_it_bro.make_me_a_rockstar()

Then the result will be like this:

+1 to have it. Looks nice :+1:

It looks nice, however I am not sure if this the right way to go about it. I am just trying to wrap my head around this line:
if random() <= self.frequency:
random generates 0 to 0.999...9 i.e. less than 1. Lets say you have set the frequency of 0.9. That means you want to have maximum no. of contributions, but not daily. Following will be the if block:
if random() <= 0.9
The random can generate any float and if its keep generating anything more than 0.9, above if block will fail and you don't get git commits for that day. Now, imagine, for some reason, random has returned every number more than 0.9, your contributions will be zero, even if you have set the frequency of 90%.
Thoughts?

No response from author since 10 days. Closing this issue.


Add Warps crystal positions

Introduction
I need to add warps crystals to the spawn because of promote map exploration. The new map is very beautiful and it merit to be explored !
The other thing pointed is an experience in the last map, we haven't teleportation stuff like this so... All players were remained near the spawn, and they hadn't explored the WorldMap entirely !
Let's do it, let's offer an easy solution to explore the WorldMap !
Add many Warps crystals positions to the server

[x] North West (not very far from -20000, y, 25000)
[x] North East (not very far from 25000, y, 25000)
[x] South West (not very far from 25000, y, -20000)
[x] South East (not very far from -20000, y, -20000)
[x] Add this xarps positions to the spawn tower and add pedestals/signs to explain that (maybe also add lights)

Over, available at the spawn tower, lower floor. :)


Revert "restructuring project file structure"

Reverts Python-World/s-tool#11

Codecov Report

Merging #12 (2e23435) into main (c9d2e89) will decrease coverage by 50.63%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #12       +/-   ##
==========================================
- Coverage   52.23%   1.60%   -50.64%
==========================================
  Files           5       6        +1
  Lines         134     125        -9
==========================================
- Hits           70       2       -68
- Misses         64     123       +59

Impacted Files
Coverage Δ

s_tool/driver.py
0.00% <0.00%> (-94.00%)
:arrow_down:

s_tool/parser.py
0.00% <0.00%> (ø)

s_tool/utils/__init__.py
0.00% <0.00%> (ø)

s_tool/utils/driver_exceptions.py
0.00% <ø> (ø)

s_tool/utils/driver_utils.py
0.00% <0.00%> (ø)

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 c9d2e89...2e23435. Read the comment docs.

Why suddenly ?

@aahnik
webdriver_manager using public API of GitHub . so its failing sometime
After merging PR into main one testcase got failed .


CF password (in deployment manifest) cannot have '@' character

Spent some time on this issue. Before I checked on of the logs and figured it out. The passwords that are provided in the cloud foundry deployment manifest cannot have '@' in them. If they do the deployment fails while updating the jobs.
Mine failed at the following step Started updating job nats_z1 > nats_z1/0 (canary).
Having '@' also does not allow you to bosh ssh into the instances.

Hi there!
We use Pivotal Tracker to provide visibility into what our team is working on. A story for this issue has been automatically created.
The current status is as follows:

[ ] #99824346 CF password (in deployment manifest) cannot have '@' character

This comment, as well as the labels on the issue, will be automatically updated as the status in Tracker changes.

Closing due to inactivity.
For the record, we investigated this and there are numerous places where this is the case.  This was further complicated by the fact that Ruby and Go libraries behaved differently in terms of how they handled URL-encoded/unencoded input.  It was turning out to be a significant undertaking, and not a high priority so we're leaving this as won't-fix.


Download datasets from google drive

current workflow needs downloading datasets to make sure all the results is reproducible

its not working now #91


Close #10 - [core] Add numeric.NonNegInt

Close #10 - [core] Add numeric.NonNegInt

Codecov Report
Patch coverage: 100.00% and project coverage change: +2.22 :tada:

Comparison is base (e6d7a20) 77.77% compared to head (8e0cd9b) 80.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   77.77%   80.00%   +2.22%
==========================================
  Files           3        3
  Lines           9       10       +1
==========================================
+ Hits            7        8       +1
  Misses          2        2

Impacted Files
Coverage Δ

...core/shared/src/main/scala/refined4s/numeric.scala
50.00% <100.00%> (+16.66%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


fix(test): wait enough time to Trigger Running Hook. Fixes: #12844

sleep 1s is sometimes too short to trigger the running hook

Fixes #12844
Motivation

Modifications

Verification

I saw there are many comment:
	// TODO: Temporarily comment out this assertion since it's flaky:
	// 	  The running hook is occasionally not triggered. Possibly because the step finishes too quickly
	//	  while the controller did not get a chance to trigger this hook.

This one we can sleep longer. Because it only verifies running hooks, unlike others that verify other types of hooks.

Could you let us how you tested it to make sure it doesn't flake

Could you let us how you tested it to make sure it doesn't flake

It is indeed not guaranteed, but now sleep 1s does often lead to a higher probability of failure of this test. In many other tests, the running hook check is skipped. If it is better for you to skip the task, you can skip it.

I wonder if we can run https://argo-workflows.readthedocs.io/en/stable/running-locally/#running-e2e-tests-locally a few times to reproduce the original issue

Could you let us how you tested it to make sure it doesn't flake

It is indeed not guaranteed, but now sleep 1s does often lead to a higher probability of failure of this test. In many other tests, the running hook check is skipped. If it is better for you to skip the task, you can skip it.

I wonder if we can run https://argo-workflows.readthedocs.io/en/stable/running-locally/#running-e2e-tests-locally a few times to reproduce the original issue

It is difficult to reproduce, but it may happen in the test after submitting the PR. Like TestStoppedWorkflow and TestTemplateExecutor, It is almost difficult to reproduce locally when running a test alone. It is easy to occur only after the PR is submitted and several people trigger the pipeline at the same time.

I think we can keep monitor and see if this change reduces flakiness

Yes, I think so. Thanks

poor spec of CI runner (2core, 7mem).

This did get doubled recently (see also Slack thread), it's now 4 vCPU & 16GB RAM. But 4 vCPU is still likely less vCPU than most local dev setups


Restarting and Patching S2D clusters

It would be worthwhile to add to this page some additional information / warnings about the correct restart process required for S2D clusters - namely https://docs.microsoft.com/en-us/windows-server/storage/storage-spaces/maintain-servers or note that Cluster-Aware Updating is recommended for patch maintenance. If only two nodes are used as suggested (i.e. two-way mirror) you can only restart one node and must wait until the S2D is resychnronised before restarting ANY nodes or you may break the storage in the cluster.

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

ID: af26cb8a-f7a0-291b-7ac3-e38f679df4ba
Version Independent ID: dd04d12d-05cb-6fb4-4a2c-ef85ebb15d0e
Content: SQL Server FCI - Azure Virtual Machines - Azure SQL Server VM
Content Source: articles/virtual-machines/windows/sql/virtual-machines-windows-portal-sql-create-failover-cluster.md
Service: virtual-machines-sql
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@chris-bowman
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@MikeRayMSFT
Can you please check and add your comments on this doc update request as applicable.

This article was recently refreshed under the direction of @MashaMSFT #please-close


Unable to upload package to Pypi using twine, Getting error requests.exceptions.ConnectionError: ('Connection aborted.', error(10054, 'An existing connection was forcibly closed by the remote host'))

Describe the bug
I am not able to upload the new version of my package using pypi even after generating token for my project.
Expected behavior
Whenever i try to upload it throws error "requests.exceptions.ConnectionError: ('Connection aborted.', error(10054, 'An existing connection was forcibly closed by the remote host'))"
To Reproduce
twine upload dist/* --repository-url https://pypi.org/project/FaceReco/ -u token -p pypi_token
My Platform
Windows
Additional context

Hi @1297rohit, sorry you're having issues.
The --repository-url you're trying to use isn't valid. It should be the upload endpoint for PyPI instead, not the project URL.
You can omit this flag entirely and twine will default to https://upload.pypi.org/legacy/

It starts to upload with my current command but stops just seconds after starting so i think command is right but some other issue.
If i dont specify the url then how will it find ny package.

Specifying the directory dist/* tells twine where to find your distribution. PyPI will determine where the release goes based on the project name.

Thanks for the help was able to upload it by creating .pypirc file and changing the repository url to https://upload.pypi.org/legacy/. Pypirc file format can be found from the link https://packaging.python.org/specifications/pypirc/#pypirc


[Nautiljon] Cloudflare issue - cURL 8.3 removes support for NSS

Is your feature request related to a problem? Please describe.
I implemented a bridge for Nautiljon because the RSS they provide is behind a Cloudflare gate.
The bridge requires a cURL version compiled with NSS so it can mimic the TLS handshake as done by browsers.
However, it's now no longer possible to compile cURL with NSS, so, soon, it will either require to stay on versions < 8.3 (not advisable on the long-term for security reasons) or find another workaround or remove this bridge.
The reason for removal of NSS is described here and changes are low it will ever come back: curl/curl#11459
Describe the solution you'd like
cURL supports other libraries.
My system provides support for the following:

gnutls
mbedtls
openssl
rustls

Maybe one of them will be close enough to NSS and Cloudflare will let us pass.
Describe alternatives you've considered

Contacting the webmaster to remove Cloudflare on specific pages was unsuccessful.
Using specific options from cURL to mimic a browser handshake, but I'm not an expert on this.
If there are other bridges that were able to workaround Cloudflare issues, I would be interested to know if there are other things I can try.

Looks like it works now with OpenSSL (which I think is the default?):
curl 'https://www.nautiljon.com/actualite/' --compressed -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/117.0' -H 'Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,*/*;q=0.8' -H 'Accept-Language: en-US,en;q=0.5' -H 'Accept-Encoding: gzip, deflate, br'

Also works in RSS bridge.
I don’t know if something changed since my implementation on RSS bridge side to make it more look like a browser, or on Cloudflare side, or on cURL side (and OpenSSL and NSS implementations are now closer), but either way, it seems to work now, so I will remove that requirement.


[rcore][desktop_glfw] Keeping CORE.Window.position properly in sync with glfw window position

CORE.Window.position is now properly kept in sync with the glfw window position.

WindowPosCallback added
SetWindowPos now updates CORE.Window.position as well
Fullscreen & Borderless Fullscreen now properly user CORE.Window.position for setting CORE.Window.prevposition

Those changes should not change any behavior other than fixing ToggleFullscreen() not restoring the window to the correct previous window position.

FYI: There will be another PR for fixing the remaining problems with fullscreen & borderless fullscreen

restoring window size correctly and using already existing CORE.Window.prevscreen for it
BorderlessFullscreen using glfw SetWindowMonitor function as well to fix borderless fullscreen on macOS
some small misc fixes.

There is a small issue in RayLib 5 when calling ToggleFullscreen() changes the angle Camera3D looks at. (And calling ToggleFullscreen() again changes the angle back). I wonder if this fix fixes that issue as well.

@raysan5 @SoloByte With the raylibdll.lib built from https://github.com/SoloByte/raylib/tree/glfw-window-pos-callback the bug with Camera3D angle jumping when switching between resolutions is indeed fixed. I tested it a 3d project. So this PR can be considered as a bug fix, not a just-in-case improvement. The relevant code I tested it with:
inline constexpr int SCREEN_WIDTH = 1920;
inline constexpr int WINDOWED_SCREEN_HEIGHT = 1080;
inline constexpr int FULL_SCREEN_HEIGHT = 1200;

if ((IsKeyDown(KEY_LEFT_ALT) || IsKeyDown(KEY_RIGHT_ALT)) && (IsKeyPressed(KEY_ENTER) || IsKeyPressed(KEY_KP_ENTER)))
        {
            if (IsWindowFullscreen())
            {
                ToggleFullscreen();
                screenWidth = SCREEN_WIDTH;
                screenHeight = WINDOWED_SCREEN_HEIGHT;
                SetWindowSize(screenWidth, screenHeight);
            }
            else
            {
                screenWidth = SCREEN_WIDTH;
                screenHeight = FULL_SCREEN_HEIGHT;
                SetWindowSize(screenWidth, screenHeight);
                ToggleFullscreen();
            }
        }

Thanks for testing this :) I did not think this would fix something right away ^^

The new commit should set CORE.Window.position correctly from the start in InitPlatform(). I also had to fix the code for centering the window on start-up when the high dpi flag is enabled so that CORE.Window.position would be set correctly as well.
@casperbear You could recheck it, it should not have changed anything (hopefully).

@SoloByte yes, Camera3D angle is still preserved when changing from windowed to fullscreen and back with RayLib from commit https://github.com/raysan5/raylib/pull/4190/commits/7bd4888a9dbbe0c42b7b85240ec201a391515ee4
In both cases: when windowed window is centered on the desktop and not centered on the desktop.
My setup is Windows 7 64-bit and one 1920x1200 monitor.

@SoloByte The improvement looks good to me, thanks for the review and sorry for the late review.

@SoloByte The improvement looks good to me, thanks for the review and sorry for the late review.

Thanks, and no worries because of the delay :)


Cross platorm - Received id is stuck in loading state in verifier device

Describe the bug
the received id is stuck in loading state in the verifier device
To Reproduce
Prerequiste: VC is downloaded and stored in sharing device
Device A - verifier device
Device B - (IOS) wallet device
steps:

open qr code in Device A
open scanner in Device B
scan the QR code from Device B
Perform a sharing method
Click on save id button on the device A
and go to Received ID page

Expected behavior
The VC should be downloaded, or else it shouldn't be presented at all
Actual behavior
The VC is stuck in the loading state
Video :
https://user-images.githubusercontent.com/102220709/222445320-716dc3b0-e35c-4510-8f12-cae0eb6f1c85.mp4
Add the screenshot of the profile page with commit id

Smartphone :
Wallet : Device B

Phone make/model: Apple iphone 7
OS: IOS 15.6
BLE version : 4.2

Verifier : Device A

Phone make/model: Redmi 7A
OS: Android 10
BLE version : 4.2

MOSIP Version : <IP_ADDRESS>
MOSIP server : qa-1201-b2.net
APP link : https://github.com/mosip/inji/actions/runs/4293687772
IPA Testflight  version : 0.4.1 (21)

The fix is given in fa142c69aaaeccc08f239060b56e0b142bef2574 to increase the DB space. This issue shouldn't happen after the db space has been increased. The video also shows the missing UIN in history page and instead shows an incremental index number.

The fix is given in fa142c69aaaeccc08f239060b56e0b142bef2574 to increase the DB space. This issue shouldn't happen after the db space has been increased. The video also shows the missing UIN in history page and instead shows an incremental index number. This could also be a sqlite db issue.

The issue is fixed by PR - #630. Kindly test with the build that includes the mentioned PR.

This issue again can come up when the stored data reaches to 30MB, which is configured in https://github.com/mosip/inji/commit/fa142c69aaaeccc08f239060b56e0b142bef2574 PR. As part of #617 enhancement, there would be an error dialog pop up comes when the data can't be stored after the VC is transfered to the verifier.

@Sujithbn As per Ravi's comments, #617 #630 might have fixed the issue, as they are not assigned for testing. re assigning the bug to you.

@Raginikrishnamurthy : #617 is a enhancement targeted for 16th MAR release which is still not released to QA team and #630 is a PR, will assign this bug along with #617 post QA handover today.

@Anushree09-N : As discussed on triage call, Please reverify and share the inputs

verified as fixed in the build with commit id in the below screenshot.


fix copy/pasta typos wikipedia->arxiv

just updates a few module level docstrings from Wikipedia -> Arxiv

Thanks!


NetStandard 2.0 support ?

Are there any plans to support also NetStandard 2.0 ?

Not from me, I'm afraid. Does much need to change?

Normally not, see here or here

If you are able to add support without breaking existing functionality, I'll happily take a pull request.