Service Lifetimes definitions are ambiguous

In the section https://docs.microsoft.com/en-us/aspnet/core/fundamentals/dependency-injection?view=aspnetcore-2.2#service-lifetimes , it says "Transient lifetime services are created each time they're requested." and "Scoped lifetime services are created once per request."
The two definitions sound equivalent to me. How is the lifetime of a Transient different from a Scoped service?
Thanks!

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

ID: 0d78f540-81fd-1ba9-cb90-591e4ec27d9b
Version Independent ID: 2251f557-dd3a-ec35-2558-6243337db009
Content: Dependency injection in ASP.NET Core
Content Source: aspnetcore/fundamentals/dependency-injection.md
Product: aspnet-core
GitHub Login: @guardrex
Microsoft Alias: riande

Good idea @ScottHutchinson ... I agree ... "request" is overloaded here. I'll fix it up. Thanks for commenting.

@ScottHutchinson Do you agree that if it remarked for transient along the lines "(service is) requested from the container" that that would clear it up? I think adding "from the container" should do the trick in those spots. Whereas, most devs will process (for scoped) "the request" as an ordinary app request.

I think that's a lot clearer. Thanks for the prompt fix!
------ Original Message ------
Received: 11:45 AM PDT, 03/27/2019
From: Luke Latham<EMAIL_ADDRESS>To: aspnet/Docs<EMAIL_ADDRESS>Cc: Scott Hutchinson<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [aspnet/Docs] Service Lifetimes definitions are ambiguous (#11707)
@ScottHutchinson Do you agree that if it remarked for transient along the lines "(service is) requested from the container" that that would clear it up? I think adding "from the container" should do the trick in those spots. Whereas, most devs will process (for scoped) "the request" as an ordinary app request.

—
  You are receiving this because you were mentioned.
  Reply to this email directly, view it on GitHub, or mute the thread.
 {"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/aspnet/Docs","title":"aspnet/Docs","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/aspnet/Docs"}},"updates":{"snippets":[{"icon":"PERSON","message":"@guardrex in #11707: @ScottHutchinson Do you agree that if it remarked for transient along the lines \"(service is) requested from the container\" that that would clear it up? I think adding \"from the container\" should do the trick in those spots. Whereas, most devs will process (for scoped) \"the request\" as an ordinary app request."}],"action":{"name":"View Issue","url":"https://github.com/aspnet/Docs/issues/11707#issuecomment-477299518"}}} [ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/aspnet/Docs/issues/11707#issuecomment-477299518", "url": "https://github.com/aspnet/Docs/issues/11707#issuecomment-477299518", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]


Google Drive Folder Creation

A folder on Google Drive will be created for large sized file sharing.

—
Reply to this email directly or view it on GitHub
https://github.com/bounswe/bounswe2016group5/issues/3#issuecomment-185694464
.

<EMAIL_ADDRESS>18 Şubat 2016 Perşembe tarihinde, sevdacopur<EMAIL_ADDRESS>yazdı:

<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','sevdacopur019@gmail.com');>
18 Şub 2016 14:21 tarihinde "Kerim Gokarslan"<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
yazdı:

I shared google drive folder. @sevdacopur <https://github.com/sevdacopur
@mbugc https://github.com/mbugc can you send your e-mail addresses to
me?
—
Reply to this email directly or view it on GitHub
<
https://github.com/bounswe/bounswe2016group5/issues/3#issuecomment-185694464
.

—
Reply to this email directly or view it on GitHub
https://github.com/bounswe/bounswe2016group5/issues/3#issuecomment-185701609
.


No replica recoverer or minos for tier0-reaper and loadtest

To fix:
[haozturk@lxplus912 ~]$ k get pods | grep temp
daemons-minos-temporary-expiration-58bc84877d-srkqj               1/1     Running     0               53m
loadtest-daemons-minos-temporary-expiration-6ddcd9bc4-7dtnp       1/1     Running     0               53m
tier0-reaper-daemons-minos-temporary-expiration-5c5689fb49mxcsx   1/1     Running     0               53m

[haozturk@lxplus912 ~]$ k get pods | grep repl
daemons-replica-recoverer-6cf779cd4-5cvv9                         1/1     Running     0               53m
loadtest-daemons-replica-recoverer-78d55547b6-jv9pc               1/1     Running     0               53m
tier0-reaper-daemons-replica-recoverer-5969647d79-vw8bw           1/1     Running     0               53m

Agreed, created an issue for it: https://github.com/dmwm/CMSRucio/issues/846


Retrying failed CEK deployment breaks CMK

Hello,
My deployment (v21.08) failed on TAS installation due my config error.
When I retried the deployment after fixing TAS config, CMK pods went down to CrashLoopBackOff with 401 Unauthorized errors after TLS certs for CMK webhook have been recreated.
I will provide full ansible log later on with more details about the issue.

Hello and thanks much for your input.  Did you try please using the cleanup playbook (redeploy_cleanup.yml) prior to trying to redeploy?  Let us know please if this addresses the issue.

After triggering infra/redeploy_cleanup.yml playbook, cmk redeployment works fine, thanks!


ProductProductOdooModel not recognised

This C# line of your OdooJsonRpcClient example :
var repository = new OdooRepository(config);
is transformed into this one in VB
Dim repository = New OdooRepository(Of ProductProductOdooModel)(config)
But ProductProductOdooModel is not recognized by compiler
and in your examples ProductProductOdooModel is never declared
How can I make ProductProductOdooModel reference recognised ?

Read the readme carefully. You need to fetch the model.

Patrick,
Read this very good article about dynamicall building of class based on
source dictionary :
https://www.dotnetcurry.com/csharp/dynamic-class-creation-roslyn
On Wed, Apr 26, 2023 at 8:40 PM Jean-Regis BOULET @.***>
wrote:

Thanks Patrick,
For answer.
Creating constant class is a problem in an Odoo configuration in which
each model can be enriched at any time.
It means we have to update our code after each Odoo model enrichment which
is clearly unpredictable and never announced ...
Then, I suggest you to investigate in following direction :
As we are already able to fetch the Oddo model structure
Please build some code to dynamically create Odoo models class accepting
the table name as parameter (Template.Product or account.invoice ...)
fetching the odoo model and using the result to dynamically build the
class needed to use the rest of you API
Visual studio is offering dynamic construction of class :
https://learn.microsoft.com/en-us/dotnet/csharp/advanced-topics/interop/walkthrough-creating-and-using-dynamic-objects
Best regards
Jean-Regis
On Wed, Apr 26, 2023 at 4:41 PM Patrick @.***> wrote:

Read the readme carefully. You need to fetch the model.
—
Reply to this email directly, view it on GitHub
https://github.com/Intechnity-com/OdooJsonRpcClient/issues/71#issuecomment-1523540399,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHT5UEH46S77P2746UIYA33XDEXYXANCNFSM6AAAAAAXMRD7MQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

You can use dictionary with this library, read documentation and take a look on repository


Remove private call on FolderManager as those methods are being called elsewhere

The methods defined below the private definition aren't really private as they are being called in other files. On User#move_feed_to_folder for example.

To fix the bundler-audit warnings: #13

Where do you see those methods being called from outside their classes?
S3Client.key is a private method to calculate the key of an S3 asset. It is not called from any other class.
FolderManager.move_feed_to_folder is a public method that is called from User#move_feed_to_folder when a user requests moving a feed to a folder. However this method doesn't do the heavy lifting, it delegates to other private methods in the class; e.g. if the folder to which the feed is being moved already exists, it calls FolderManager.move_feed_to_existing_folder. The private methods of FolderManager are only invoked from methods in the same class.

However, looking at this I've realized that while those class methods are supposed to be private, they really aren't because a private_class_method line is missing in those classes (and the private line does not really affect class methods). Thanks for pointing this out.

@amatriain my explanation of "are being called in other files" wasn't right. Actually if you set these methods to private with private_class_method and run the unit tests you will see they will fail with:

NoMethodError:
private method `remove_feed_from_folder' called for FolderManager:Class

I'm closing this, those methods should indeed be private. There is a related issue, which is that private class methods were not declared correctly; this is addressed in PR #12


Update pojavlauncher section.

It now supports modpacks and auto installs modloaders like fabric forge and quilt

Thank you for the report! ❤
I added what I could see in launcher in https://github.com/TayouVR/MinecraftLauncherComparison/commit/a177d7d1ee0c7b83b1f4ff250ec7bcf97694dbd5
If there is anything missing or if any more features are added in the future feel free to tell me and I will update the list.


Shard parquet in download_and_prepare

Following https://github.com/huggingface/datasets/pull/4724 (needs to be merged first)
It's good practice to shard parquet files to enable parallelism with spark/dask/etc.
I added the max_shard_size parameter to download_and_prepare (default to 500MB for parquet, and None for arrow).
from datasets import *

output_dir = "./output_dir"  # also supports "s3://..."
builder = load_dataset_builder("squad")
builder.download_and_prepare(output_dir, file_format="parquet", max_shard_size="5MB")

Implementation details
The examples are written to a parquet file until ParquetWriter._num_bytes > max_shard_size. When this happens, a new writer is instantiated to start writing the next shard. At the end, all the shards are renamed to include the total number of shards in their names: {builder.name}-{split}-{shard_id:05d}-of-{num_shards:05d}.parquet
TODO:

[x] docstrings
[x] docs
[x] tests

cc @severo

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

This is ready for review cc @mariosasko let me know what you think


chore: Cluster sync should have short requeue, no backoff

Fixes #
Description

Right now, the cluster state syncing mechanism will have an exponential backoff which can create large gaps between runs of the provisioning or deprovisioning loop. This should not be an exponential backoff but a continued one second poll
Also, the counter controller should use cluster state sync when overwriting the status. Otherwise, there is a potential for us to overwrite the status with a smaller value and shoot over our limits

How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 25 (12.0%)  changed or added relevant lines in 2 files are covered.
91 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.2%) to 80.606%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controllers/deprovisioning/controller.go
3
4
75.0%

pkg/controllers/provisioning/controller.go
0
21
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/provisioning/controller.go
6
0%

pkg/controllers/state/cluster.go
6
84.47%

pkg/controllers/node/controller.go
7
73.53%

pkg/controllers/provisioning/scheduling/preferences.go
7
86.67%

pkg/controllers/deprovisioning/controller.go
23
76.7%

pkg/controllers/provisioning/provisioner.go
42
77.81%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
6546

Relevant Lines:
8121

💛  - Coveralls


Adding Bourbon Refills styles

Could you also add erb and slim templates?

Yep!

Updated with erb and slim templates

Could you squash the second commit into the first one?

Done

Thanks!

:smile:


[pipe-commands] feat: copy file contents to clipboard

Description

feat: add copy contents to clipboard file command
fix: cleanup inputType extraction logic, fix copy path command

Screencast
NA
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Done!

I must admit the code is not the clearest, I must have accepted a pull request without doing a proper review.
Could you explain to me the purpose of your in the extensions/pipe-commands/src/pipe-to-command.tsx file ?
Feel free to refactor the code to make it clearer instead of adding a bunch of comments :)

@iloveitaly are you still around?


WebPack Optimization

This is used for webpack to add the files in the final.js file.

I'm not familiar with WebPack, so I'll have to trust you this works

Thanks!

Thanks for you trust.


Support reading from custom file in HCL configs

Sometimes we need to provide a custom HL7 parser config to the the HL7 store, such as this example here: https://github.com/GoogleCloudPlatform/healthcare-data-protection-suite/blob/4276c1ae548ea7b872c1d996ad6ce72bf19ec91f/examples/tfengine/modules/team.hcl#L444
This custom config is a JSON file that could be several thousand lines long. It would be impractical to copy paste the entire JSON file into a HCL config file, can we have a custom function that reads a separate config file, something like templatefile in Terraform?

Fixed by https://github.com/GoogleCloudPlatform/solution-acceleration-toolkit/pull/1112


No string constructor/factory method for date-time

With this yaml :
  task:
    type: object
    properties:
      complete_before:
        type: string
        format: date-time
      complete_after:
        type: string
        format: date-time

Using the generated example for the post :
{
  "complete_before": "2016-07-25T13:15:53.208Z",
  "complete_after": "2016-07-25T13:15:53.208Z"
}

triggers this error :

[JsonMappingException: Can not instantiate value of type [simple type, class org.joda.time.DateTime] from String value ('2016-07-25T13:07:40.517Z'); no single-String constructor/factory method
 at [Source: [B@4e6bc7b1; line: 34, column: 22] (through reference chain: workload.yaml.NewTask["complete_before"])]

But this is working :
{
  "complete_before":<PHONE_NUMBER>,
  "complete_after":<PHONE_NUMBER>,
}

I also have this issue :
parsing default 0 value into a type: number, format: double.

[ClassCastException: java.lang.Integer cannot be cast to java.lang.Double]

@slavaschmidt can i have a try?

@ I started Top redesign marshalling part recently. I tend to generate play writeables and readables for that. Made a POC today but still need some time to code it properly

@nmcb sorry, previous comment was for you

@slavaschmidt nice! wanna sync this week?

Fixed in 0.2


For a short period of time after a new stable release, beta is an older version.

For instance, as of right now:
$ git checkout upstream/beta
Previous HEAD position was 71520442d4 [flutter_releases] Flutter beta 3.4.0-34.0.pre Framework Cherrypicks (#112971)
HEAD is now at 099b3f4bf1 Update Engine revision to 45c5586f2a9583d9511b742703529a92bf8f9db5 for beta release 3.7.0-1.5.pre (#118927)
$ flutter --version
Downloading Linux x64 Dart SDK from Flutter engine 45c5586f2a9583d9511b742703529a92bf8f9db5...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  238M  100  238M    0     0  87.3M      0  0:00:02  0:00:02 --:--:-- 87.3M
Building flutter tool...
Flutter 3.7.0-1.5.pre • channel unknown • unknown source
Framework • revision 099b3f4bf1 (2 weeks ago) • 2023-01-20 18:35:12 -0800
Engine • revision 45c5586f2a
Tools • Dart 2.19.0 (build 2.19.0-444.6.beta) • DevTools 2.20.1
$ git checkout upstream/stable
Previous HEAD position was 099b3f4bf1 Update Engine revision to 45c5586f2a9583d9511b742703529a92bf8f9db5 for beta release 3.7.0-1.5.pre (#118927)
HEAD is now at 7048ed95a5 Update Engine revision to 800594f1f4a6674010a6f1603c07a919b4d7ebd7 for stable release 3.7.1 (#119687)
$ flutter --version
Downloading Linux x64 Dart SDK from Flutter engine 800594f1f4a6674010a6f1603c07a919b4d7ebd7...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  238M  100  238M    0     0  96.3M      0  0:00:02  0:00:02 --:--:-- 96.4M
Building flutter tool...
Flutter 3.7.1 • channel unknown • unknown source
Framework • revision 7048ed95a5 (2 days ago) • 2023-02-01 09:07:31 -0800
Engine • revision 800594f1f4
Tools • Dart 2.19.1 • DevTools 2.20.1

Beta points to an older version of the Dart and Flutter SDK than stable. This is surprising, since I would expect that using the beta channel always gets me a version of Flutter that is >= stable.
One way to solve this would be to update beta to point to the same hash as stable when doing a stable release. We get out of this state once a new beta is cut after a stable.
/cc @zanderso who knows a lot about the release process and might have ideas about why this won't work. Ditto for @christopherfujino
/cc @CaseyHillers and @XilaiZhang since they're also involved in release.

\cc @itsjustkevin  from the release program side
tl;dr This is intentionally broken. If there's a workflow broken, please let us know so we can prioritize.
Generally, we try to ship a new beta as soon as possible after the stable, but there's some constraints. We will have gone ~2 weeks with this behavior, but we don't expect this to impact any people or tools.
One thing we've proposed is possibly shipping a hidden beta. For example, while 3.7 was on beta in January, we could've created 3.8 beta. While it would've never shipped, it'd be available for us to promote during the stable release, and would live for only ~2 weeks.


Lossless Compression for static assets

🚀 feature request
Relevant Package

 This feature request is for @angular/....
"@angular/core": "~8.2.14"
Description
 A clear and concise description of the problem or missing capability...
Lossless image compression for angular production builds.
Describe the solution you'd like
 If you have a solution in mind, please describe it.
Build managers like gulp offer great libraries for this alongside npm. Perhaps this might be a consideration for the future as a standard feature.
Describe alternatives you've considered
 Have you considered any alternative solutions or workarounds?
Yes, third party applications.

Hi, @artistry-intelligence-solutions,
Thanks for your feature request. IMHO, I think images should be compressed outside of the build pipeline. Assets don't change that often, and a better approach would be to export/save the assets with the optimal compression.
If you need to use a tool on top of your assets editing software to compress them even more. This doesn't need to be done in build pipeline either, as with this approach users can compress the assets using their desired compression method and tool. Also, with this approach an asset will be compressed once for every assets as opposed of having it as part of the build pipeline where each assets will be compressed each time a build is triggered.
Integrating compression in the build pipeline has the potential of becoming a foot-gun because builds will get slower because of the need to compress each assets.


Failure to create non-Identity integer PK column due to lack of SERIAL support

Related to:

https://github.com/Mause/duckdb_engine/issues/289
https://github.com/aaronsteers/target-duckdb/pull/1#issuecomment-1493725565

I understand that SERIAL types are not supported, and #289 should eventually add that feature. However, shouldn't an INTEGER column just be created as an integer rather than a SERIAL (auto-identity) column?
We're trying to create primary keys that may or may not be integers, and the failure to create the table in certain circumstance is unexpected.
Two questions:

Is there are workaround to prevent integers from being attempted as SERIAL type columns?
Is there a potential fix where IDENTITY/AUTOINCREMENT/SERIAL columns are just never considered, and the columns are created just as regular ints?

There's probably a lot of missing context on my side, since I'm not deep into Postgres internals. Thanks in advance for your help on this.

Hi @aaronsteers, you can disable this behaviour by setting autoincrement=False on the column declaration itself. This stuff is buried pretty deep inside the PostGres DDL compiler so it isn't straightforward for me to disable it from the DuckDB dialect

Thanks, @Mause!
cc @kgpayne ☝

@Mause - What do you think of an update to the docs here?: https://github.com/Mause/duckdb_engine#auto-incrementing-id-columns
That section calls out how to do auto-increment in a different way, but not how to just create a "normal" non-incrementing integer column.

If you think that documenting it there would be useful sure, I'm happy to accept a pr 😁


Inconsistent Autoformatting Behavior

From @nwayve on October 4, 2016 14:23

VSCode Version: 1.5.3
OS Version: 10.0.14393

Steps to Reproduce:

[Edit] File > Preferences > User Settings
[Set] "javascript.format.insertSpaceAfterOpeningAndBeforeClosingNonemptyBrackets": true
[Ensure] There is no workspace settings file interfering
[Create] test.js
[Add] var x = ['foo', 'bar'];
[Observe] Upon hitting the semi-colon key, the line will be formatted as such:
var x = [ 'foo', 'bar' ]; as expected
[Auto-Format] Trigger an auto-format ensuring nothing is highlighted (shift + alt + f)
[Observe] The line will be formatted back to: var x = ['foo', 'bar'];
[Select-All]
[Auto-Format]
[Observe] The line will be reformatted to the correct: var x = [ 'foo', 'bar' ]; again

Copied from original issue: Microsoft/vscode#13210

@nwayve thank you for opening this issue. Can you try with v1.6? I cannot reproduce the bug with the latest download.

From @nwayve on October 10, 2016 22:23
Just updated to v1.6.0.  I still get the same exact behavior following the steps provided except that when I select all to auto-format, it doesn't reformat it.
To verify (and provide myself with a sanity check), I perform the following:
User settings.json file:
{
    "editor.fontFamily": "Consolas",
    "editor.formatOnType": true,
    "files.autoSave": "afterDelay",
    "javascript.format.insertSpaceAfterOpeningAndBeforeClosingNonemptyBrackets": true
}

keybindings.json:
(Emptied this out to ensure I didn't inadvertently reassign any default keybindings)
[]

shift + alt + f Default Keyboard Shortcut:
{
"key": "shift+alt+f",
"command": "editor.action.format",
"when": "editorHasFormattingProvider && editorTextFocus && !editorReadonly"
}

No .vscode folder or workspace file(s).
Working directory: C:\Projects\temp

Create test.js
Add line var x = ['foo', 'bar']
When I press ; at the end of that line, it auto-formats to:

var x = [ 'foo', 'bar' ];

After pressing the semi-colon, I press to auto-format shift + alt + f and get:

var x = ['foo', 'bar'];

Then I select all ctrl + a
And shift + alt + f again to auto-format:

var x = [ 'foo', 'bar' ];

Or if I delete the semi-colon and re-enter it, it reformats it with the spacing as before:

var x = ['foo', 'bar']
```

```
var x = [ 'foo', 'bar' ];
```

SS of Sys Info (windows version) and VS Code Version:

![sysinfo_vscodeinfo](https://cloud.githubusercontent.com/assets/1729221/19252991/a9c64386-8f0d-11e6-9e9e-f7e38372fdea.png)

Can't think of anything else I could rule out as far as any custom configurations/settings that I may have set anywhere that might do this.

@waderyan edited for readability.

@nwayve thank you for your detailed instructions. This seems to be working as intended.
From what you've written I understand that you expect
var x = ['foo', 'bar']

To turn into
var x = [ 'foo' ,  'bar' ]

When you add a semicolon per "editor.formatOnType": true setting.
Your original issue was that when you selected formatted
var x = [ 'foo' ,  'bar' ];

It would undo the formatting. However you are not seeing that in V1.6 anymore.
Is my understanding correct?
At least I see what I am thinking is correct behavior. See here.

From @nwayve on October 13, 2016 20:55
The behavior you're displaying is correct.  With the line of code selected, when you Format Code, the formatting will remain as expected, with the spacing added within the brackets.  Now deselect the code and leave the cursor at the end of the line (or anywhere in the doc really, just don't have anything selected).  When you Format Code again, it will undo the formatting.
I'm still seeing this behavior in v1.6.1.

@nwayve thank you for your clarification. This is a language service issue. Moving to the language service repo.

This is working for me correctly in latest drop of TypeSript (typescript@next).

I should note that VSCode ships with an older version of TypeScript. Please see Using Newer TypeScript Versions documentation for more details on updating your VSCode to use a different version of TypeScript.

Makes sense.  Updated to the latest version of TypeScript and that fixed this behavior.  Thanks for the info.

thanks for confirming. closing.


Support the TOS DOC for alluxio

What changes are proposed in this pull request?
I have added the documentation for Tinder Object Storage (TOS), which includes information on how to integrate TOS with Alluxio.
Why are the changes needed?
We need documentation to provide customers with support for the current version of TOS.
Does this PR introduce any user facing changes?
No.

alluxio-bot, merge this please

alluxio-bot, merge this please


is this someting to worry about ?

Logger: homeassistant.components.sensor
Source: components/sensor/init.py:497
Integration: Sensor (documentation, issues)
First occurred: 5:56:57 AM (8 occurrences)
Last logged: 5:56:57 AM
Entity sensor.dynamic_prices_highest_energy_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
Entity sensor.dynamic_prices_average_electricity_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
Entity sensor.dynamic_prices_current_percentage_of_highest_electricity_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
Entity sensor.dynamic_prices_time_of_highest_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('timestamp') it is using; expected None; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
Entity sensor.dynamic_prices_time_of_lowest_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('timestamp') it is using; expected None; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author

Yes this is something to worry about, but not now yet :)
WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.current_electricity_market_price (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.next_hour_electricity_market_price (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.lowest_energy_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.highest_energy_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.average_electricity_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.current_percentage_of_highest_electricity_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.time_of_highest_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('timestamp') it is using; expected None; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.time_of_lowest_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('timestamp') it is using; expected None; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.


Clarify "You can request Key Vault to manage your storage account with a user principal, but not with a service principal."

I don't read why there is limitation you cannot let keyvault manage storage account with sp, actually we already do that.

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

ID: 8b568009-2a35-c627-cfa0-ff5cbfde027e
Version Independent ID: 07fc2d02-be89-1170-3b37-9e65808d5d89
Content: Manage storage account keys with Azure Key Vault and the Azure CLI
Content Source: articles/key-vault/secrets/overview-storage-keys.md
Service: key-vault
Sub-service: secrets
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@karataliu Thank you for pointing this out . We will have this checked and update the thread.

@karataliu, thank you for reaching out. I don't see any issue using Service Principal for Azure Key vault to manage Storage account keys. I would recommend you set this up with Service Principal and see if that works. Ideally, it should work without issue.

Closing this thread since got no queries on it.

@souravmishra-msft  I mean there is no issue, but the doc said there is an issue. Did you read my request clear?

@shashishailaj  @souravmishra-msft  Can you reopen this. My question is not answered.

@karataliu that command fails when run under service principal:
az keyvault storage add ...
https://github.com/MicrosoftDocs/azure-docs/issues/26079


Reduce compiler log level for iterations and cache log lines to debug (#4650)

Pull request opened by the merge tool on behalf of #4650

Merged into branches iso5 in 107dba722bdfaf4468f31007b6c79d8b4b9e413a


Manually trigger pipeline on latest commit

Clear and concise description of the problem
After my server went offline and I made a new commit, it won't run the pipeline on the latest commit because It didn't receive the webhook. I need a way to manually re-trigger the webhook now and I can't find a feature that can do this...
Suggested solution
Trigger Pipeline button with commit/branch id
Alternative
Pushing a random commit just to get it to trigger (not a good idea)
Additional context
No response
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Oops, i think #83 means this  is a duplicate question.

#93 will solve it. Its definitely often requested and still on the roadmap 😉


stylesheet_link_tag not processing properly

I have the following test_mailer/test_mail.html.erb:

  <%= stylesheet_link_tag 'email' %>

Test Site

Hi <%= @person.name %>

The style defined in the  tag is inlined correctly. However, the styles defined in email.scss are just dumped into the message's  tag:
<!DOCTYPE html>

    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">

Test Site

Hi Mister Test

I can see this if I add this to a Person view:

<%= Premailer::Rails::Hook.perform(TestMailer.test_mail(@person)) %>

Incidentally, email.scss also fails to load at all with "ActionView::Template::Error (Asset was not declared to be precompiled in production.
Add Rails.application.config.assets.precompile += %w( email.css ) to config/initializers/assets.rb and restart your server):" unless I do this in the initialiser:
Rails.application.config.assets.precompile += %w( email.css )

Is this right? There's no mention of needing to do this in the docs. (I assume loading from outside stylesheets is supported, as having to copy the same styles into each email isn't exactly DRY :)
Is there a clue in the fact that the loaded stylesheet is appearing in the , despite the command to load it being in the  section? Is the fact that it's SCSS causing the confusion
I'm using premailer 1.8.7, premailer-rails 1.9.4, nokogiri <IP_ADDRESS>, sass 3.4.22, sass-rails 5.0.6, bootstrap-sass 3.3.6, and rails 5.0.1 with Ruby 2.3.1.
Thanks!

That's correct, the stylesheet you are using must be one that rails precompiles. That can also be your default application.css, which is already configured to be precompiled in Rails.application.config.assets.precompile. The fact that you are using SCSS makes no difference; the asset pipeline will transform that into CSS.
I didn't quite understand, does everything work as expected when your stylesheet is set up correctly? Also, try looking at the example app in this repo.

Thanks — it appears I had some structural problems with the actual HTML of the email. Adding a doctype and  tag seems to have fixed it :)


chore(codemagic_app_preview): update version to 0.2.2

⬇ Builds generated by Codemagic for commit 78353ff6b1d1f84f2218ad423507260d4bb33238 ⬇
This is a custom message.

iOS

Android

macOS

  Download link

  Download link

  Download link

(expires Sat, 16 Dec 2023 12:17:33 GMT)Codemagic build: 655607739e94f8861f7d7126; comment generated with Codemagic App Preview


32 bit NeoDemo causes access violation in native DotNet Core helper function

When running NeoDemo as 32 bit an access violation occurs in the native DotNet Core helper function StubHelpers::SetLastError
This is the complete call stack: https://pastebin.com/hzr5C2LG
This is the exception:
Unhandled exception at 0x0FD9372E (coreclr.dll) in dotnet.exe: 0xC0000005: Access violation writing location 0x0FD9380E.

The Veldrid codebase is completely unmodified aside from the ImGui renderable code and the menu code being disabled in the NeoDemo program, as well as the backend being set to OpenGL. I ran it with the 32 bit dotnet executable and used the 32 bit SDL2 library provided in the Veldrid repository.
I have experienced similar issues when using Veldrid in my own 32 bit program, where it causes a managed stack frame cookie corruption. I think it may be caused by some OpenGL calls passing the wrong size to native code somewhere, but i can't be sure.
I tried to run the 32 bit version with the Vulkan backend; while the 64 bit version works just fine, the 32 bit one fails with this error: https://pastebin.com/QkyKwuSG
I also tried the DX11 backend: https://pastebin.com/GL3Ceyxp
I assume this is failing because the Veldrid.SPIRV library relies on a 64 bit native library that it can't find. The NuGet package directory only contains 64 bit libraries.
I'm not sure if this is related or not.

You're right, there is some problem with using the OpenGL backend in a 32-bit application. The fact that Veldrid.SPIRV doesn't contain a 32-bit Windows binary has been fixed in a newer beta version of the package, but the one referenced here doesn't yet. On my machine, the Vulkan backend works fine in a 32-bit mode -- what drivers do you experience that crash on?
I'm not sure yet why the OpenGL backend fails on 32-bit. I'm going to try to isolate some different chunks of code and see if I can identify any problems.

The problem seems to be pretty simply: OpenGL actually uses the standard calling convention, rather than the C calling convention. I'm going to push a fix that will correct the function declarations so that they use the correct calling convention. This fixes everything that I'm seeing with regards to OpenGL.
Running Veldrid under 32-bit .NET Framework gives a much more descriptive error message for this, which made it easier to find out.
By the way, I actually tried 32-bit NeoDemo under Vulkan again, and I see the same validation error that you do. When I turn off the validation layers (GraphicsDeviceOptions.Debug == false), everything still renders correctly. I think this may be caused by a driver error, so I'm updating my Nvidia drivers and checking again. I don't see why a shader would compile correctly in a 64-bit app and fail to compile in a 32-bit app (since they are identical).

Well, the latest Nvidia drivers seem really unstable (I was crashing in Visual Studio alone), but I went back a couple versions and the error about shader compilation no longer occurs, and everything renders properly. I will merge the change to fix 32-bit OpenGL, and bump the version of Veldrid.SPIRV so it includes 32-bit binaries.

This should be fixed now, let me know if you're able to test against the latest code from the master branch.

I ran the latest code and the OpenGL and Vulkan backends work in 32 bit mode. The D3D11 backend terminates with this exception: https://pastebin.com/0GteuRzh
I'll see if i can get my own program to run with these assemblies, since the beta hasn't been published in NuGet yet.

@SamVanheer That should be fixed now.

I can confirm that the D3D11 backend now works in both 64 and 32 bit builds.

Thanks for the confirmation. These fixes will be in the next release, so I'm closing this out.


Allow charts to select date-range

Add a dropdown to charts so that the date range can be specified.

I agree, there seems to be a reported issue with ApexCharts https://github.com/apexcharts/apexcharts.js/issues/3754


Strange error upon attaching (maybe javafx)

i'm attaching to lunar client (removed jvm args of disable attach) and this is happening and im not really sure. i think its this bit: java.lang.RuntimeException: java.lang.NullPointerException: Cannot invoke "java.lang.ClassLoader.getResource(String)" because "<local2>" is null
[2021-08-16 09:15:56.064] [info]  [OUTPUT] 09:15:56.059 [Attach Listener] WARN : Failed to delete old log file, will append instead
09:15:56.062 [Attach Listener] INFO : Recaf-2.20.1
09:15:56.063 [Attach Listener] INFO : Starting as agent...

[2021-08-16 09:15:56.296] [info]  [OUTPUT] 09:15:56.295 [Attach Listener] TRACE: Loading configuration

[2021-08-16 09:15:59.481] [info]  [OUTPUT] 09:15:59.480 [Attach Listener] INFO : Loaded instrumentation workspace

[2021-08-16 09:15:59.496] [info]  [OUTPUT] 09:15:59.489 [Attach Listener] INFO : Loaded 1 plugins

[2021-08-16 09:16:01.873] [info]  [OUTPUT] Exception in thread "JavaFX Application Thread" java.lang.RuntimeException: java.lang.NullPointerException: Cannot invoke "java.lang.ClassLoader.getResource(String)" because "<local2>" is null

[2021-08-16 09:16:01.884] [info]  [OUTPUT] 	at me.coley.recaf.ui.MainWindow.lambda$get$2(MainWindow.java:205)
	at javafx.graphics/com.sun.javafx.application.PlatformImpl.lambda$runLater$10(Unknown Source)
	at java.base/java.security.AccessController.doPrivileged(Unknown Source)
	at javafx.graphics/com.sun.javafx.application.PlatformImpl.lambda$runLater$11(Unknown Source)
	at javafx.graphics/com.sun.glass.ui.InvokeLaterDispatcher$Future.run(Unknown Source)
	at javafx.graphics/com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
	at javafx.graphics/com.sun.glass.ui.win.WinApplication.lambda$runLoop$3(Unknown Source)
	at java.base/java.lang.Thread.run(Unknown Source)
Caused by: java.lang.NullPointerException: Cannot invoke "java.lang.ClassLoader.getResource(String)" because "<local2>" is null
	at javafx.graphics/com.sun.javafx.css.StyleManager.getURL(Unknown Source)
	at javafx.graphics/com.sun.javafx.css.StyleManager.loadStylesheetUnPrivileged(Unknown Source)
	at javafx.graphics/com.sun.javafx.css.StyleManager.loadStylesheet(Unknown Source)
	at javafx.graphics/com.sun.javafx.css.StyleManager.processStylesheets(Unknown Source)
	at javafx.graphics/com.sun.javafx.css.StyleManager.gatherSceneStylesheets(Unknown Source)
	at javafx.graphics/com.sun.javafx.css.StyleManager.findMatchingStyles(Unknown Source)
	at javafx.graphics/javafx.scene.CssStyleHelper.createStyleHelper(Unknown Source)
	at javafx.graphics/javafx.scene.Node.reapplyCss(Unknown Source)
	at javafx.graphics/javafx.scene.Node.doProcessCSS(Unknown Source)
	at javafx.graphics/javafx.scene.Node$1.doProcessCSS(Unknown Source)
	at javafx.graphics/com.sun.javafx.scene.NodeHelper.processCSSImpl(Unknown Source)
	at javafx.graphics/com.sun.javafx.scene.ParentHelper.superProcessCSSImpl(Unknown Source)
	at javafx.graphics/com.sun.javafx.scene.ParentHelper.superProcessCSS(Unknown Source)
	at javafx.graphics/javafx.scene.Parent.doProcessCSS(Unknown Source)
	at javafx.graphics/javafx.scene.Parent$1.doProcessCSS(Unknown Source)
	at javafx.graphics/com.sun.javafx.scene.ParentHelper.processCSSImpl(Unknown Source)
	at javafx.graphics/com.sun.javafx.scene.NodeHelper.processCSS(Unknown Source)
	at javafx.graphics/javafx.scene.Node.processCSS(Unknown Source)
	at javafx.graphics/javafx.scene.Scene.doCSSPass(Unknown Source)
	at javafx.graphics/javafx.scene.Scene.preferredSize(Unknown Source)
	at javafx.graphics/javafx.scene.Scene$2.preferredSize(Unknown Source)
	at javafx.graphics/com.sun.javafx.scene.SceneHelper.preferredSize(Unknown Source)
	at javafx.graphics/javafx.stage.Window$12.invalidated(Unknown Source)
	at javafx.base/javafx.beans.property.BooleanPropertyBase.markInvalid(Unknown Source)
	at javafx.base/javafx.beans.property.BooleanPropertyBase.set(Unknown Source)
	at javafx.graphics/javafx.stage.Window.setShowing(Unknown Source)
	at javafx.graphics/javafx.stage.Window.show(Unknown Source)
	at javafx.graphics/javafx.stage.Stage.show(Unknown Source)
	at me.coley.recaf.ui.MainWindow.start(MainWindow.java:63)
	at me.coley.recaf.ui.MainWindow.lambda$get$2(MainWindow.java:203)
	... 7 more

[2021-08-16 09:16:02.251] [info]  [OUTPUT] Exception in thread "JavaFX Application Thread"
[2021-08-16 09:16:02.260] [info]  [OUTPUT] java.lang.NullPointerException: Cannot invoke "java.lang.ClassLoader.getResource(String)" because "<local2>" is null
	at javafx.graphics/com.sun.javafx.css.StyleManager.getURL(Unknown Source)
	at javafx.graphics/com.sun.javafx.css.StyleManager.loadStylesheetUnPrivileged(Unknown Source)
	at javafx.graphics/com.sun.javafx.css.StyleManager.loadStylesheet(Unknown Source)
	at javafx.graphics/com.sun.javafx.css.StyleManager.processStylesheets(Unknown Source)
	at javafx.graphics/com.sun.javafx.css.StyleManager.gatherSceneStylesheets(Unknown Source)
	at javafx.graphics/com.sun.javafx.css.StyleManager.findMatchingStyles(Unknown Source)
	at javafx.graphics/javafx.scene.CssStyleHelper.createStyleHelper(Unknown Source)
	at javafx.graphics/javafx.scene.Node.reapplyCss(Unknown Source)
	at javafx.graphics/javafx.scene.Node.doProcessCSS(Unknown Source)
	at javafx.graphics/javafx.scene.Node$1.doProcessCSS(Unknown Source)
	at javafx.graphics/com.sun.javafx.scene.NodeHelper.processCSSImpl(Unknown Source)
	at javafx.graphics/com.sun.javafx.scene.ParentHelper.superProcessCSSImpl(Unknown Source)
	at javafx.graphics/com.sun.javafx.scene.ParentHelper.superProcessCSS(Unknown Source)
	at javafx.graphics/javafx.scene.Parent.doProcessCSS(Unknown Source)
	at javafx.graphics/javafx.scene.Parent$1.doProcessCSS(Unknown Source)
	at javafx.graphics/com.sun.javafx.scene.ParentHelper.processCSSImpl(Unknown Source)
	at javafx.graphics/com.sun.javafx.scene.NodeHelper.processCSS(Unknown Source)
	at javafx.graphics/javafx.scene.Node.processCSS(Unknown Source)
	at javafx.graphics/javafx.scene.Scene.doCSSPass(Unknown Source)
	at javafx.graphics/javafx.scene.Scene$ScenePulseListener.pulse(Unknown Source)
	at javafx.graphics/com.sun.javafx.tk.Toolkit.lambda$runPulse$2(Unknown Source)
	at java.base/java.security.AccessController.doPrivileged(Unknown Source)
	at javafx.graphics/com.sun.javafx.tk.Toolkit.runPulse(Unknown Source)
	at javafx.graphics/com.sun.javafx.tk.Toolkit.firePulse(Unknown Source)
	at javafx.graphics/com.sun.javafx.tk.quantum.QuantumToolkit.pulse(Unknown Source)
	at javafx.graphics/com.sun.javafx.tk.quantum.QuantumToolkit.pulse(Unknown Source)
	at javafx.graphics/com.sun.javafx.tk.quantum.QuantumToolkit.pulseFromQueue(Unknown Source)
	at javafx.graphics/com.sun.javafx.tk.quantum.QuantumToolkit.lambda$runToolkit$11(Unknown Source)
	at javafx.graphics/com.sun.glass.ui.InvokeLaterDispatcher$Future.run(Unknown Source)
	at javafx.graphics/com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
	at javafx.graphics/com.sun.glass.ui.win.WinApplication.lambda$runLoop$3(Unknown Source)
	at java.base/java.lang.Thread.run(Unknown Source)

#399 this leads from here

Looks like its a known bug in JavaFX, says so in the source code.

It was also crashing because it tried to rename the extension to bss as you can see in the prior screenshot. I didn't look at much of the internals of why it does this, but thankfully there's a system property to disable this.
Then you can just set the thread's context classloader and it works.
Will push up a fix shortly.

Works :)

thank you so much for your time <3


Face Transformer Table I Query

I would like to ask one question about the table I in your Face Transformer paper.
How you calculate the speed of Img/Sec in Table I?
I am using the below library but I have no idea how to calculate inference speed?
https://github.com/sovrasov/flops-counter.pytorch

You can record the inference speed of n images, and then calculate the mean value.
import time

You can record the inference time of n images and then calculate the mean value.
import time

last_time = time.time()
backbone(batch_device)
forward_time += time.time() - last_time

Thank you @zhongyy

@zhongyy  Would it be possible for you to check this batch_device = batch.to(device) code again. I am getting errors in this line.

@khawar512 The code comes from "test_forward()" of "https://github.com/zhongyy/Face-Transformer/blob/main/util/utils.py", maybe you can try "https://github.com/zhongyy/Face-Transformer/blob/main/test_forward.py".


calculate ratio between tumor and normal sample’s WGS read counts

Hi
fragCounter using CBS algorithm calculate read couts ratio between tumor and normal sample in 200bp genomic bins.
I would like to ask which is the output parameter of the normal sample?  This code doesn't see the input parameters of normal.
frag -b inst/extdata/chr21.bam -d inst/extdata/gcMAP21/ -w 200

You will need to run fragCounter on your tumor bam and normal bam separately, then compute the tumor/normal ratio afterwards by dividing reads.corrected per bin of the tumor sample by the normal sample.

Hi, my tumor/normal data has different sequencing depth.
Should I use (tumor.reads.corrected / tumor.mean_coverage) / (normal .reads.corrected / normal .mean_coverage)
instead of  tumor.reads.corrected / normal .reads.corrected  as you mention


test(slider): add unit tests to Slider

add unit tests to Slider

Will you add tests for the Slider component?

adding in #429


Fix the bug where utilsd tries to add NoneType arguments to argparse

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

Comparison is base (d7389bd) 67.53% compared to head (dc265bd) 67.56%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
+ Coverage   67.53%   67.56%   +0.03%
==========================================
  Files          33       33
  Lines        1925     1927       +2
==========================================
+ Hits         1300     1302       +2
  Misses        625      625

Impacted Files
Coverage Δ

utilsd/config/cli_parser.py
72.09% <100.00%> (+1.36%)
: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.


Secure NFS configuration

In order to secure NFS configuration files like /etc/hosts.{deny,allow} would have to be filled with correct configuration data. The current NFS NixOS module does not allow one to configure these, so how is one supposed to deploy NFS on NixOS?
One could argue that it is not responsible to even give the suggestion that NFS is supported in any fashion on NixOS today.
See https://help.ubuntu.com/community/SettingUpNFSHowTo for documentation for Ubuntu NFS setup.

FWIW I'm pretty happy with current NixOS module for NFSv3 (non-secure) running in my home LAN.
For now you can fill those files by using configuration option environment.etc."hosts.allow".text = "foo". Of course, it would be nice to have the module support them instead. Probably we should highlight that we support only NFSv3 out of the box for now.

As part of https://github.com/NixOS/nixpkgs/pull/22303 I consider to disable tcp-wrappers and have users relying on NixOS firewall instead.

(triage) it looks like the environment.etc."hosts.allow".text solution has been given… maybe this issue can be closed? :)


Problem of exception coming from acquisition with pycromanger

Hello
I take the liberty of contacting you to ask you for help with pycro-manager
Currently I am considering using pycro-manage to control the micro-manager software but I am having exceptional problems with Bridge.
I want to integrate all the features of micro-manager on my python code with pycro-manager but I can't do it.
That is why I am asking for your support.
Thanks in advance
Sincerely,
-- coding: utf-8 --
"""
Created on Thu May  2 09:49:07 2024
@author: baraf
"""
import numpy as np
from pycromanager import multi_d_acquisition_events, Acquisition
Generate the events for a single z-stack
events = multi_d_acquisition_events(z_start=0, z_end=10, z_step=0.5)
Create an Acquisition object and use it to acquire images
with Acquisition(directory=r'C:\Users\baraf\OneDrive\Bureau\ecole', name='acquisition_name', show_viewer=True) as acq:
acq.acquire(events)
Problème
Python 3.12.2 (tags/v3.12.2:6abddd9, Feb  6 2024, 21:26:36) [MSC v.1937 64 bit (AMD64)]
Type "copyright", "credits" or "license" for more information.
IPython  -- An enhanced Interactive Python.
runfile('C:/Users/baraf/OneDrive/Bureau/sanstitre3.py', wdir='C:/Users/baraf/OneDrive/Bureau')
C:\Users\baraf\AppData\Local\Programs\Python\Python312\Lib\site-packages\pyjavaz\bridge.py:308: UserWarning: Version mistmatch between Java ZMQ server and Python client.
Java ZMQ server version: 4.0.0
Python client expected version: 5.1.0
To fix, update to BOTH latest pycromanager and latest micro-manager nightly build
warnings.warn(
Traceback (most recent call last):
File ~\AppData\Local\Programs\Python\Python312\Lib\site-packages\spyder_kernels\py3compat.py:356 in compat_exec
exec(code, globals, locals)
File c:\users\baraf\onedrive\bureau\sanstitre3.py:16
with Acquisition(directory='C:\Users\baraf\OneDrive\Bureau\ecole', name='acquisition_name', show_viewer=True) as acq:
File ~\AppData\Local\Programs\Python\Python312\Lib\site-packages\pycromanager\acquisition\acq_constructor.py:45 in new
return JavaBackendAcquisition(**named_args)
File ~\AppData\Local\Programs\Python\Python312\Lib\site-packages\pycromanager\acquisition\java_backend_acquisitions.py:279 in init
self._create_remote_acquisition(**named_args)
File ~\AppData\Local\Programs\Python\Python312\Lib\site-packages\pycromanager\acquisition\java_backend_acquisitions.py:521 in _create_remote_acquisition
self._acq = acq_factory.create_acquisition(kwargs['directory'], kwargs['name'], show_viewer,
File ~\AppData\Local\Programs\Python\Python312\Lib\site-packages\pyjavaz\bridge.py:488 in
): instance._translate_call(signatures_list, args, static=_java_class == 'java.lang.Class')
File ~\AppData\Local\Programs\Python\Python312\Lib\site-packages\pyjavaz\bridge.py:698 in _translate_call
valid_method_spec, deserialize_types = _check_method_args(method_specs, fn_args)
File ~\AppData\Local\Programs\Python\Python312\Lib\site-packages\pyjavaz\bridge.py:870 in _check_method_args
raise Exception(
Exception: Incorrect arguments.
Expected java.lang.String, java.lang.String, boolean, boolean, int, int, int, int, boolean
Got <class 'str'>, <class 'str'>, <class 'bool'>, <class 'int'>, <class 'bool'>

As it says in the message: To fix, update to BOTH latest pycromanager and latest micro-manager nightly build

thank you I got it.
Now I have a labview source code which allows me to control the helmholtz coils with three axes x,y,z and I want to integrate it into micromanager
Do you have an idea on how to integrate it into micromanager?

https://micro-manager.org/Micro-Manager_Programming_Guide


ssl problem maxtv.hrvatskitelekom.hr

Site
maxtv.hrvatskitelekom.hr
Description
ssl problem

If you're encountering an issue, could you please provide the error message you're seeing or share a screenshot?

I think that the epg api link should be changed to theirs from the desktop or android app, because it works well on those platforms.


TSX Wrong Emit Handler Return Type

Version
3.2.1
Reproduction link
https://jsfiddle.net/986ukynp/
Steps to reproduce
Event handler should return void not validation value of emit.
What is expected?
No type error
What is actually happening?
Type error is happening

@webfansplz I edited type code with your pull request and nothing changed. The problem is here

I don't have a good knowledge about typing types but if I change it to this, I don't get any type error in component file but in type file.

Hi @kadiryazici,Maybe you forgot to return it when you invoked it
import { defineComponent } from 'vue';

const Component = defineComponent({
   emits: {
      something: (value: string) => true
   }
});

const OtherComponent = defineComponent({
   render() {
      return (
         <Component
            onSomething={() => {
               console.log('yeah');
               return true  // return true
            }}
         />
      );
   }
});

export default OtherComponent

@webfansplz  it is not the return type of emit, it is just validator. I return true from validator because I don't need validaton. Vscode type checking is enough for me.
Emit handler should return any or void always.

@kadiryazici You are right, I have mentioned PR to fix this


Rounding for layers

Description
As the title suggests, the ability to be able to set border radius for layers aka rounding would be great cuz enabling blur on layer based apps like rofi whilst also setting the border radius to >1 in the rofi config, resultant layer has sharp corners as theres no option to set rounding for borders

No.

resultant layer has sharp corners as theres no option to set rounding for borders

layerrule = ignorezero, <regex>

Okay it works but it does come with its quirks. the borders (if they are rounded) on a  layer surface like  waybar look kinda ugly.
See the top left corner of this screenshot:

since then ignorezero has been deprecated in favor of ignorealpha [val]. val goes from 0-1. Play around with the value until it looks good.

Okay it seems like changing the blur contrast, and brightness values to 1 solved the issue for me. Great work man !

No.

resultant layer has sharp corners as theres no option to set rounding for borders

layerrule = ignorezero, <regex>

what is in the


Some general concerns

I have recently discovered that someone reuploaded my patch to GitHub and uploaded a video showcasing the installation process to YouTube, both without asking for permission. I think that I've made it clear enough in the readme that you should not do that. After lurking around this repository for a bit, I want to ask the following questions:

Why did you not ask me for permission before performing both of the aforementioned actions? You could have, for example, sent me an email or opened an issue on the main patch repository
What does this project have to do with Lutris? It seems completely unrelated, but still has "Lutris" in the name for some reason

sir please check https://github.com/VHSgunzo/sr_patch/issues/1 he ignored me

Hmm. It seems like @strycore might want to do something about this. Still, I'm waiting for a response to both of my questions.

Delete or rename your repository, remove any occurence of the word Lutris.

It seems like you have removed the repository where the reuploaded patch was hosted, and instead included it as an archive concatenated to the install script. That does not change the fact that the patch is still reuploaded to GitHub. Also, removing the repository is not even what I asked for, so both questions remain unanswered. Waiting for your response

You should also listen to strycore and at least rename your repository

Why did you not ask me for permission before performing both of the aforementioned actions? You could have, for example, sent me an email or opened an issue on the main patch repository

@mkrsym1

What kind of request for use permissions are we talking about, is this written in the README or at least somewhere?
Have you even read the license under which you distribute your patches?
If not, I will remind you that the patches are distributed under the MIT license and here is a small excerpt from it:

...
Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software
...

Initially, my project was as an addition to Lutris, adding certain functionality and simplifying work with it, and also Wine runner from Lutris was used by default in my project, and the name itself was taken precisely because of this, and also because the use of other runners other than Wine was not intended, then the project grew up. Now you can still install Wine and some libraries for the prefix from Lutris, but not necessarily.

Delete or rename your repository, remove any occurence of the word Lutris.

@strycore
I am not going to rename or delete a project that is already a little more than two years old! (a year of which there is a video on YouTube on launching the game using the patch @mkrsym1) And I don't understand, do you have any personal rights to the word "Lutrsi"? Something I strongly doubt this ...
I hope the Admin from AUR has also clearly explained to you why the package cannot be deleted or renamed.
Although to be honest, I myself have been thinking about renaming the project for a long time, because I'm tired of explaining to people every time that this is not the "inconvenient" lutris, which can't even run the EXE from explorer, because that's what they start telling me at the mention of the word lutris...

It seems like you have removed the repository where the reuploaded patch was hosted, and instead included it as an archive concatenated to the install script. That does not change the fact that the patch is still reuploaded to GitHub. Also, removing the repository with the reuploaded patch is not even what I asked for, so both questions remain unanswered. Waiting for your response

@mkrsym1
The patch is not in the open and its installation is not possible separately from my project! (only if you really want it and you know what you're doing)
Better look here

If not, I will remind you that the patches are distributed under the MIT license and here is a small excerpt from it:

If you would also look at the license.txt file in the patch repo it would say under the following condition:
"The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software."

little more than two years old!

This is not possible as the patches that were made are all less than a year old since we are talking about sr here and not gi

If not, I will remind you that the patches are distributed under the MIT license and here is a small excerpt from it:

If you would also look at the license.txt file in the patch repo it would say under the following condition:
"The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software."

Yes, but the author of the patch in README forbade any mention of the author and the patches!

any mention of the author and the patches!

and for sr it also forbid the sharing as noted in the very top of the readme

gi

No matter what kind of game we are talking about, the essence does not change from this

and sharing also ofc includes any files no matter in which format.

forbade any mention of the author and the patches!

That's why it says "permission notice" as well and not just license but you didn't ask for permission.

And I'm also amused by the fact that for some reason you guys think that the developers of games and anti-cheat do not know that there is an unofficial way to run their games on linux, which also lies in the public domain and Googles in a couple of minutes and has several vedeo on YouTube (not only mine)...

What kind of request for use permissions are we talking about, is this written in the README or at least somewhere?

The readme states that the project should not be shared in public. So, if you intend to share it in public, you should at least ask.

Have you even read the license under which you distribute your patches?

I am aware that this is not legally enforceable. However, sharing might attract unnecessary attention, so it is in the interests of the users to not share it.

a year of which there is a video on YouTube on launching the game using the patch

You are thinking of the GI patch, which is not maintained by me. However, that repository has a similar notice about sharing in public.

The patch is not in the open and its installation is not possible separately from my project!

That does not change the fact that you are sharing it. Is asking really that hard?

Yes, but the author of the patch in README forbade any mention of the author and the patches!

I did not forbid that specifically, but instead everything that can be considered "sharing in public". You mostly violated that already.

And I'm also amused by the fact that for some reason you guys think that the developers of games and anti-cheat do not know that there is an unofficial way to run their games on linux, which also lies in the public domain and Googles in a couple of minutes and has several vedeo on YouTube (not only mine)...

Not an excuse for provoking them further. The patches do depend on their good will, but until they've stated officially that this is allowed, we should not assume so.

Oh, We know we also keep reminding people that post without permission that they did something wrong and some follow through and delete their videos instead of being stubborn like you.
But just following the rules that are put out still helps out.

OK, what do you want from me now?
I have now renamed the YouTube video (But Youtube's algorithms don't work that easily)

@mkrsym1
Also, if we have a misunderstanding on the basis of the need to request permission to use, then I want you to add it to the README

At least:

Clearly state that the patch is not yours (I've seen some people with that misconception under a different video on your channel). I do not request more attribution, however you should still contact and ask the GI patch maintainer
Resolve the issue with @strycore . As your project no longer has any relation to Lutris, the name is simply misleading
If you are going to do something like this again, please ask for permission. You may receive guidance on what should and should not be done

@mkrsym1 Also, if we have a misunderstanding on the basis of the need to request permission to use, then I want you to add it to the README

I will add it in the future

Clearly state that the patch is not yours (I've seen some people with that misconception under a different video on your channel). I do not request more attribution, however you should still contact and ask the GI patch maintainer

But I didn't mention anywhere that these were my patches, and I didn't say anywhere that I or anyone else did them, I didn't leave any links, as required in the README. I will even say more, anyone can add a patch to help launch a particular game or application in my project.
By the way, genshin does not need a patch to run now

But I didn't mention anywhere that these were my patches

You did not mention that they are not yours anywhere too. I've seen multiple people with that misconception

By the way, genshin does not need a patch to run now

And yet you are still distributing it.

And yet you are still distributing it.

This is done only in case of a sudden change of mind on the part of the developers.
So far they have not made official statements. because now it's like running a game on steam deck

Let me repeat that again. You are distributing it. You have to provide attribution. I'm telling you to ask the maintainer about this. It's not that hard

@strycore

I don't think it matters if your project has been using this name for a few years now. If you want for your project to gain any kind of notoriety in the future, I don't see how it could happen while having another project's name.

Perhaps in the future I will create a new project that will include developments from my current project, but the key feature will be a completely redesigned GUI and the project will have a different name unrelated to Lutris, but for now I think everything will remain as it is.

Then there's the fact that your project distributes stuff that you've been asked not to distribute. I would appreciate if the Lutris branding wasn't associated with that

But Lutris has installers for gi and sr that violate the licenses of these games, and as I understand it, they are used when installing games from the Lutris library. But in my project, the use of these patches is optional.

But Lutris has installers for gi and sr that violate the licenses of these games, and as I understand it, patches are used when installing games from the Lutris library. But in my project, the use of these patches is optional.

You have completely skipped over this line "Then there's the fact that your project distributes stuff that you've been asked not to distribute. " while the patches for both game were distributed with permission as the launchers don't directly link to it.

But in my project, the use of these patches is optional.

Yea, sure try running SR without it

You have completely skipped over this line "Then there's the fact that your project distributes stuff that you've been asked not to distribute. "
while the patches for both games were distributed with permission as the sr launcher doesn't directly link to it or share it and for gi the person came to the patch maintainer and asked.

I said this because @strycore cares about the branding of lutris in terms of the fact that there is the word lutris in the name of my project and I did not ask permission to use patches that initially violate the license of these games, although the same thing happens in lutris itself, and the patch for genshin in lutris is generally distributed completely in open form, no matter who there what and from whom I asked

Yea, sure try running SR without it

This is another question, in the Lutris installer, both patches violating the license of these games are enforced

Lol, what? What exactly are the "Nvidia drivers" that I "distribute"?

Quite literally all of it:
https://github.com/VHSgunzo/runimage-nvidia-drivers/releases/tag/v530.41.03

Lol, what? What exactly are the "Nvidia drivers" that I "distribute"?

Quite literally all of it: https://github.com/VHSgunzo/runimage-nvidia-drivers/releases/tag/v530.41.03

These are not "nvidia drivers", as such, and if you don't understand what it is, then it's better not to climb

@cybik
Nvidia LICENSE:
...
2.1.2  Linux/FreeBSD Exception.  Notwithstanding the foregoing terms
of Section 2.1.1, SOFTWARE designed exclusively for use on the Linux or
FreeBSD operating systems, or other operating systems derived from the
source code to these operating systems, may be copied and redistributed,
provided that the binary files thereof are not modified in any way
(except for unzipping of compressed files).
...

@cybik Nvidia LICENSE:
...
2.1.2  Linux/FreeBSD Exception.  Notwithstanding the foregoing terms
of Section 2.1.1, SOFTWARE designed exclusively for use on the Linux or
FreeBSD operating systems, or other operating systems derived from the
source code to these operating systems, may be copied and redistributed,
provided that the binary files thereof are not modified in any way
(except for unzipping of compressed files).
...

You remove libraries. You are altering the software package itself. And thus:

            No Reverse Engineering. Customer may not reverse engineer, decompile, or disassemble the SOFTWARE, nor attempt in any other manner to obtain the source code.

            No Separation of Components. The SOFTWARE is licensed as a single product. Its component parts may not be separated for use on more than one computer, nor otherwise used separately from the other parts.

You violate section 2.1.3 Limitations of the driver license available at https://www.nvidia.com/en-us/drivers/nvidia-license/.

Package managers in standard Debian, Ubuntu, Pop, Redhat/Fedora, Suse and all derivatives, also have multiple packages and a system designed so that the integrity of the original nVidia driver installer runtime's software package is preserved (unless the user manually says to not install one component or another - which is, at that point, up to the user themselves and not the redistributor, clearing the redistributor's responsibility).
If you had multiple squashfs images containing all the things your packaging script removed, you COULD argue you're doing this.
Which brings us to this:
3. TERMINATION

    This LICENSE will automatically terminate if Customer fails to comply with any of the terms and conditions hereof. In such event, Customer must destroy all copies of the SOFTWARE and all of its component parts.

As you are in direct violation of 2.1.3 per what your script does, the license would compel you to stop what you're doing and delete the package you're building and distributing, at least.

If you had multiple squashfs images containing all the things your packaging script removed

This container module is loaded or assembled only on
linux computers of Nvidia users, and they already have all the deleted items from the container module

If you had multiple squashfs images containing all the things your packaging script removed

This container module is loaded or assembled only on linux computers of Nvidia users, and they already have all the deleted items from the container module

That is completely irrelevant. You can't assume the presence of any other components. Either you redistribute it in whole, or you don't redistribute it at all.

Just no one really wants to bother with you anymore cause you just don't listen and it's pointless fighting with you at that point.
Instead at this point it's better to escalate this than to keep fighting with you on a personal level.

Said the guy who is doing the same thing...

Said the guy who is doing the same thing...

Well, at least I'm not saying that I will "escalate" the situation and not inflate the tragedy out of nothing here... If you really would like no one to know about the patch, then you didn't need to put it in open access, you didn't need to make instructions for use, separate launchers and other things that you did yourself, and now you're whining that someone uses it and distributes ...
Honestly speaking of which, I don't really care about these anime games, I don't play them and added patches because I was asked to do it
If you don't like what I'm doing, then don't use it! I don't force anyone to use it, I don't sell my projects to anyone and I don't take money for it, even in the form of donations!

added patches because I was asked to do it

Yea but you got linked to them which was already breaking the do not share part, then proceeded to make a repo to share it without asking the maintainer and again breaking the do not share part.

If you really would like no one to know about the patch, then you didn't need to put it in open access, you didn't need to make instructions for use, separate launchers and other things that you did yourself

The reason the patch is out there for the public to use is cause it was meant for that reason the reason we say why YOU should not share it is to prevent the damage to minimal costs everywhere else where you can find the patch either has an agreement with the patch maintainer.
the "but then the launchers and etc shouldn't exist" thing doesn't apply here and is irrelevant as they are simply meant to make it easier for users to use the stuff without leaking information out as we never link to the stuff or share the files right away unlike YOU
The videos that exist all also were made with permission as long as they followed our guidelines like:

not mentioning the game by name
not sharing the links or files in description or comments

These guidelines are all also followed by the launchers in the repos as well.

The reason the patch is out there for the public to use is cause it was meant for that reason

Well, again, you contradict yourself... What constructive dialogue can go on with you?

Just admit that you're using the word lutris to masquerade as some sort of trustable Lutris addon or something, and possibly for SEO.


The size of the images of the training and the test was inconsistent

Hello, this is a great job! I noticed that the paper says to resize the image of the dataset, which is 1024×512 in Cityscapes and 256×832 in KITTI. It was found that the picture would be intercepted to 256×256 in the training stage why not enter the whole picture, but the entire images was included in the testing stage, why the size of the images of the training and the test was inconsistent?
training stage: imgs = self.aug_seq(imgs, 256, 256)
testing stage: imgs = Full-size image

Thank you for your question. In the low-level field, previous researches are done in the same way as this setting (training on small images, testing on full-size). When training, if you use the full-size, it will cause problems such as too small batchsize, out-of-memory, and slow training.


Misc cleanup before 2.0 release

I plan to do some misc fixes and cleanup here, before the 2.0 release.
Currently there are some fails in master:
https://travis-ci.org/astropy/pyregion/builds/287512820
If there's anything difficult or controversial, I'll ask. No need to review at this point.
Work in progress ...

@bsipocz - Do you remember what the fix was to suppress this list intersphinx issue?
https://travis-ci.org/astropy/pyregion/jobs/287523737#L1007
I remember seeing it before, but can't remember or find it ...

Coverage remained the same at 63.504% when pulling 60bfe13e3fc3e2e373b4d68e3b9fb047147c82d3 on cdeil:cleanup into 7bde9dd1561c5b1a47b989a3a57232b6f15f7754 on astropy:master.

Coverage increased (+0.04%) to 63.542% when pulling f5bf208e8f928f9055fa05364198d54a307f5008 on cdeil:cleanup into 7bde9dd1561c5b1a47b989a3a57232b6f15f7754 on astropy:master.

Because of the list intersphinx issue, for now I've just allowed Sphinx warnings in the Python 2 docs build: 32fceb7
Very few people build the docs of this package anyways and there's really no reason why they should use Python 2 any more ...
If someone wants to change back, sure, just go ahead with a commit in master or a pull request.

@cdeil - probably the simplest is to add them as exceptions to a docs/nitpick-exceptions file.

btw, I'm doing a new helpers release right now, if you want to included that here (though probably nothing major is in there that would affect pyregions).

I don't think there's a reason to do a helpers update for this package.

Coverage increased (+0.04%) to 63.542% when pulling 32fceb772c29351e775c892b73e512a12b26aded on cdeil:cleanup into 7bde9dd1561c5b1a47b989a3a57232b6f15f7754 on astropy:master.

It looks like changes to PYTEST_HEADER_MODULES in conftest.py have no effect for this package? I'm always seeing the modules from Astropy core listed.
The conftest.py file is executed, but changes to PYTEST_HEADER_MODULES have no effect on the prinout!???
@bsipocz - Can you reproduce? Do you have an idea what's wrong?

I don't think there's a reason to do a helpers update for this package.

at all? That case I can remove it from the auto-update list.

I'll check on the conftest stuff soon.

I would suggest to keep this package on the astropy-helpers update list. I'm just saying if there are no known issues with what we have and  CI is all green and this package is already at a recent astropy-helpers v2, then it's OK to release here and have other astropy-helpers updates and other fixes in future releases, no?

@cdeil - PYTEST_HEADER_MODULES seems to be working for me, could you give more details of what you see?
Maybe are you on the astropy slack?

I put this in conftest.py:
from collections import OrderedDict
PYTEST_HEADER_MODULES = OrderedDict([
    ('numpy', 'numpy'),
    ('matplotlib', 'matplotlib'),
    ('Astropy', 'astropy'),
    ('spam', 'spam'),
])

and I still get this, which I think is what Astropy has:
Numpy: 1.13.3
Scipy: 0.19.1
Matplotlib: 2.1.0
h5py: 2.7.0
Pandas: 0.20.3

Why?

It inherits the default from astropy, not sure what happens if you try to override that variable, it seems that nothing.
But what I see on this branch is what we usually do, delete the keys that are unneeded, and add new ones. Is there a problem with that approach?
try:
    PYTEST_HEADER_MODULES['Astropy'] = 'astropy'
    del PYTEST_HEADER_MODULES['h5py']
    del PYTEST_HEADER_MODULES['Scipy']
except NameError:
    pass

Is there a problem with that approach?

Yes, Astropy keeps getting new dependencies (like pandas now) that are not a dependency in affiliated packages. IMO it's simpler / cleaner to just have a separate / independent list here, instead of doing additions / deletions wrt Astropy core and having to change that over time.
Obviously this is not a big deal. If it's not clear how to do the list from scratch, I'll go back to the other scheme. @astrofrog - Maybe you have a tip?

OK, then it's really an issue for the package template. What I would do is to hack around with a list to keep and clear the dict automatically for everything else.
Happy to come up with something after I see the helpers release out of the door.

Hmm, actually, this can be fixed in astropy. The dict is defined separately in the plugin, and there is indeed no reason to keep pandas or h5py or any astropy specific entry there.
I would suggest to keep numpy and mpl there, but even the latter could be removed from the default one.

@bsipocz - I followed your suggestion to clear the dict and then fill it how I like. Works fine. Thanks!
Thinking about it a bit, it's not really surprising that making a new PYTEST_HEADER_MODULES didn't work. One has to change the existing dict that is already defined elsewhere and only brought into this namespace via the * import at the top.

Coverage increased (+0.04%) to 63.542% when pulling a01d89e3669e615303a9d686781a36d04a172fad on cdeil:cleanup into 7bde9dd1561c5b1a47b989a3a57232b6f15f7754 on astropy:master.


PANIC: process uncaughtException Error: retry update after advisory lock release iostats

Environment info

NooBaa Version: VERSION
Platform: Kubernetes 1.14.1 | minikube 1.1.1 | OpenShift 4.1 | other: specify

Actual behavior

web server crashes

Apr-3 18:38:38.008 [WebServer/797197] [ERROR] core.util.postgres_client:: findOneAndUpdate failed { system: 642a9d052383132854a9f199, resource_id: 642a9dff7f1b5a2a19edd252, resource_type: 'NODE', start_time:<PHONE_NUMBER>000, end_time:<PHONE_NUMBER>99 } { '$set': { system: 642a9d052383132854a9f199, resource_id: 642a9dff7f1b5a2a19edd252, resource_type: 'NODE', start_time:<PHONE_NUMBER>000, end_time:<PHONE_NUMBER>999 }, '$inc': { read_count: 8, write_count: 8, read_bytes: 8192, write_bytes: 8192, error_read_count: 0, error_write_count: 0, error_read_bytes: 0, error_write_bytes: 0 } } { upsert: true, returnOriginal: false } Error: retry update after advisory lock release iostats
    at PostgresTable._upsert (noobaa-core/src/util/postgres_client.js:1239:35)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async IoStatsStore._update_io_stats (noobaa-core/src/server/analytic_services/io_stats_store.js:62:21)
    at async IoStatsStore.update_node_io_stats (noobaa-core/src/server/analytic_services/io_stats_store.js:31:9) {
  retry: true
}

Apr-3 18:38:38.008 [WebServer/797197] [ERROR] CONSOLE:: PANIC: process uncaughtException Error: retry update after advisory lock release iostats
    at PostgresTable._upsert (noobaa-core/src/util/postgres_client.js:1239:35)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async IoStatsStore._update_io_stats (noobaa-core/src/server/analytic_services/io_stats_store.js:62:21)
    at async IoStatsStore.update_node_io_stats (noobaa-core/src/server/analytic_services/io_stats_store.js:31:9)

Expected behavior

no uncaught exceptions

Steps to reproduce

using backingstores with erasure coding and occasionally it fails like so (when updating stats).

More information - Screenshots / Logs / Other output

specifically here we should probably handle the error in the code using io_stats_store.
in general the we need to revisit the way we store stats in the DB and try to get rid of the upsert implementation and the advisory locks


Add School UAEM

Add school Universidad Autónoma del Estado de Morelos(Autonomous University of the State of Morelos)
The official page from the university is https://www.uaem.mx/

@alejandropzz First stage of verification passed, official domain is uaem.mx as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.Proof of domain ownership:https://www.uaem.mx/avisos/obten-tu-correo-institucional-uaemedumx

@alejandropzz Pull request merged. Thank you.


p4a: Add metadata to maximize window

https://phabricator.endlessm.com/T33607

I expect jenkins to fail as this depends on https://github.com/endlessm/python-for-android/pull/5 being merged and pinned in the Makefile.

Hmm, jenkins failed saying p4a was missing the option, but that seems wrong. Let's try that again.

Ohh... it's because build.py is loaded from the p4a cached dist directory. I think we might really want to stop any p4a caching in jenkins. It's too broken.

retest this please

Yep, it succeeded after I manually deleted the docker volume being used as the p4a cache.

Can we look at #39 (the last commit in particular) before merging this so that the normal build doesn't break reusing the wrong p4a?

#39 got merged, so taking this now.


Fix null pointer error when adding endpoints

Purpose

$subject

Codecov Report
Patch coverage has no change and project coverage change: +54.68 :tada:

Comparison is base (031c589) 25.42% compared to head (78f7d7e) 80.10%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1401       +/-   ##
===========================================
+ Coverage   25.42%   80.10%   +54.68%
===========================================
  Files         129        9      -120
  Lines       14952      548    -14404
  Branches     2442      246     -2196
===========================================
- Hits         3801      439     -3362
+ Misses      11019      109    -10910
+ Partials      132        0      -132

Flag
Coverage Δ

idp-domain-service
80.10% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 138 files with indirect coverage changes

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


Simplified Predicates test. Thx @lukaseder

Current coverage is 94.71%

Merging #1173 into master will not affect coverage as of 5ea2030

@@            master   #1173   diff @@
======================================
  Files           78      78
  Stmts         9185    9185
  Branches      1746    1746
  Methods          0       0
======================================
  Hit           8700    8700
  Partial         90      90
  Missed         395     395

Review entire Coverage Diff as of 5ea2030

Powered by Codecov. Updated on successful CI builds.


Enhance to work with OIDC Authentication

Associated PRs:

https://github.com/ManageIQ/manageiq-appliance/pull/287

I was initially thinking of protecting ui/service under OIDC (web redirects with Apache config changes https://github.com/ManageIQ/manageiq-appliance/pull/287), even if we fetched the X-Remote-User and related headers, this is not going to help us.  Since we're running as an SPA, the API calls are going through Apache (not on server side locally), so sending any of those headers will be ignored, essentially rendering any API calls from SUI unauthorized.
Unless we have the Apache/IDP fetch an access token for the SPA in the headers, not even sure if this is even an option generally, the next solution is to support resource owner password grant authentication in Apache for the API, see lower priority issue in https://github.com/ManageIQ/manageiq/issues/19867,  with this we'd be able to do basic auth with the API, which will just have the SUI work with OIDC with no change.  Not web SSO like classic, but users will be able to enter their OIDC IDP credentials in the SUI login screen.

See the Talk Post for an approach a community member took on his own.  His work was originally implemented for the Hammer version, but he recently updated to extend this feature to Ivanchuk.
/cc @guilrom

Thanks @chessbyte for adding me to the discussion!
In our sandbox implementation, the authentication process is indeed handled at the Apache level (using apache2 mod_auth_openidc module with Keycloak as an IDP) exactly as for the admin UI (as described in the official documentation). So we do are using X-Remote-User and related headers in the SUI (in the authentication-api.factory.js).
But to make it work, we had to tweak the API, and add a few routes to perform authentication / authorization for this special OIDC case.
To get a better understanding, you can have a look into this 2 main commits, respectively in the SUI and the API:
https://github.com/Plemi/manageiq-ui-service/commit/249af259fe32bc48a7aefe752536c909cf909280 https://github.com/Plemi/manageiq-api/commit/47af2f6346228b92e71f4ff6e9937f9111643988
It might looks quite tinkered, but it is working. Don't hesitate if you have any question.

Ok, beating my head on this a bit more, it's actually a lot simpler than I thought for Jansa.
Once you protect a page with OIDC, i.e.:
<LocationMatch ^/ui/service.*$>
  AuthType  openid-connect
  Require   valid-user
</LocationMatch>

The redirect that occurs from the IDP (Keycloak in this case) to the protected page actually includes the access token for the authenticated user.  We receive the OIDC access_token as
a header as follows:
  HTTPD_OIDC_ACCESS_TOKEN=eyJhbGc......

The SUI can simply use that access token (if there) to target the /api/auth with that access token as a Bearer authorization:
 -H "Authorization Bearer: eyJhbGc...."

To get the API token for that user, without having to show the login screen in the SUI giving us OIDC SSO and no changes needed in the API for this.
The SUI login would still need to trigger the /oidc_login/redirect_uri?logout=https:/..../ui/service
upon logout. Also, the IDP would need to be configured to allow /ui/service* as valid redirects URLs.

We may still need to enhance the /api/product_info endpoint similar to @guilrom did in https://github.com/Plemi/manageiq-api/commit/d2a25af71a9fa77a92ecb66a147dcb6e8490dc9e#diff-0ef0779741ad3209b46314a66c353351 to expose the current authentication settings needed for the SUI to drive the login screen logic if no HTTPD_OIDC_ACCESS_TOKEN is found, i.e. present the  Login to corporate system button or just redirect automatically if enable sso is checked.

Related WIP PRs:

[ ] https://github.com/ManageIQ/manageiq-appliance/pull/287
[ ] https://github.com/ManageIQ/manageiq-api/pull/856

Need to add the following to the OP - I can't edit it.

[ ] https://github.com/ManageIQ/manageiq-ui-service/pull/1651

And related documentation ...

[ ] https://github.com/ManageIQ/manageiq-documentation/pull/1460

sorry, closed too soon, I guess
https://github.com/ManageIQ/manageiq-pods/issues/556


Make a payment throught GraphQL

I’m traying to make a payment using GraphQL requests. The steps I follow are the following:

Make a checkoutPaymentCreate request
Make a paymentCapture request

When I make the  paymentCapture request there is a PAYMENT_ERROR error that has the following message:  Cannot find successful {kind.value} transaction. How can I solve this problem?

Hi, you have to authorize the payment first through checkoutComplete see https://github.com/mirumee/saleor-docs/blob/99d83417f25f6fe39c854a0f197ea3896f1125f1/docs/api-process/check-out.md#completing-the-checkout for more information.

Thank you @NyanKiyoshi


minimal_by_hdr_and_img_gz fails on Windows

This is not a new problem, I just waited a long time to report it.
When I run cargo test --release --features ndarray_volumes, minimal_by_hdr_and_img_gz fails on Windows and WSL. Moreover, it stops the other tests from running. To see the tests from volume.rs and writer.rs, I must remove the minimal_by_hdr_and_img_gz test.
I didn't investigate yet but I'm probably the only one with Windows so I'll do that "soon". What we could do to confirm the problem is to tell travis to test all OS. They added Windows support in the last months.
os:
  - windows
  - linux
  - osx
language: rust
...

Is this a different issue from #4? But yes, whatever the problem is here, we could add osx and windows to Travis.

Hahaha, wow, I totally forgot about this issue. I'll confirm that it has the same cause and, if so, I'll fix the tests. We can't really ask other contributors or travis to use the hack you suggested.

Closing as fixed by #29.


fix: update expected images for redact samples test

Fixes: #77

[ ] Tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

This is probably the quickest fix.
It has been an issue in the past, however - so a more in-depth fix would be to scan the resulting image for an appropriate sub-image (at a fixed position). We might be able to use sharp for that.

Thanks @ace-n! I opened an issue regarding the potential fix.


Make depth clears reliable in OpenGL.

Mirror of https://github.com/KTXSoftware/Kha/commit/1b75d8d6fc5bdafe5f50763bafa9c5ed95e06626.
Seems to work well, anything suspicious?

Looking good.


Add option AllowHeredoc to Metrics/LineLength

This option can either be true or a list of heredoc delimiters. When set to true, long lines in all heredocs are allowed. When set to a list, long lines are only allowed in heredocs delimited by the specified delimiters.
Closes #1407
/cc @jfelchner

This is nice! But I think the whitelist feature is not particularly useful.
Personally, I would prefer if Metrics/LineLength simply allowed long lines in heredocs all the time, without the need for an extra config parameter.

I'm improving this right now since this doesn't work with nested heredocs à la
puts <<-FOO, <<-BAZ
Hello #{<<-BAR}, good morning!
John
BAR
Hope you're doing well.
FOO
Second puts arg
BAZ

@alexdowad You can just set it to true. I guess we can also discuss whether the default value of this option should be true.

You can just set it to true

Yes, certainly; but in my opinion, it is better not to add gratuitous configuration "knobs". It just complicates things.
But if the config parameter is there, my vote is that it should definitely be set to true by default.

Usage-wise I love it, but it's ultimately up to @bbatsov. The reason I like this is because:
a) it can be set to true in the default configuration so that users don't have to set what would probably be a standard default
b) it makes this ripe for a refactoring into a module that can be mixed into any other cop where we'd like to exclude HEREDOCs

Thanks to @alexdowad's suggestion, I'm now extracting the information from the AST, which makes the code simpler. I've added a commit on top and haven't rebased, just so one can see the difference. Let me know if I should rebase before merging.
One more thing I'm not sure about is where to store the extracted heredocs. Right now I just create this heredoc variable in #investigate (line). Would it be OK to have that as an ivar or is the instance reused across many files?

The new code is much cleaner. Good job. Thank you @alexdowad and @fphilipe.

@fphilipe so happy to see this. :smile: Thank you!

You should add LineLength: AllowHeredoc in config/default.yml.

You should add LineLength: AllowHeredoc in config/default.yml.

@jonas054 With what value? true or false?

Looks good. Squash all commits together and rebase on top of the current master.

@bbatsov Done!

:+1:

@fphilipe really happy about this.  Thank you so much for looking at it and getting a solution knocked out. :grinning:

Thanks @fphilipe.

 Happy to help :smile:


Allow overriding default Speckle paths

We should allow overriding of default speckle paths via Env Variables and programmatically.
Currently, this has been allowed in core for the UserApplicationDataPath folder, see: https://github.com/specklesystems/speckle-sharp/pull/1879
What still needs to be done:

[x] propagate the change in SpecklePy
[ ] (optional) propagate to the installers
[ ] look at other paths that will need a similar treatment (eg the InstallApplicationDataPath)
[x] align with proposed changes in the Blob storage PR https://github.com/specklesystems/speckle-sharp/blob/core/blobSupport/Core/Core/Helpers/Path.cs#L28

@teocomi do we need to add this behavior to the install path too now?

Closing this ticket as we are moving to Jira to reduce noise. You can access it there.