Sensitivity Labels not supported in OWA mini (mobile)

Hello,
I can't see any mention to OWA mini (the mobile browser version) not supporting Sensitivity Labels.
The new policy "Require users to add a label to emails" is bypassed if the user sends the email from OWA mini
Is expected the table on this article to publish this limitation?
Thank you in advance

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

ID: 2dd81b07-cfc4-2272-fa96-f6d053f7cb0e
Version Independent ID: 2dd81b07-cfc4-2272-fa96-f6d053f7cb0e
Content: Minimum versions for sensitivity labels in Microsoft 365 Apps - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/sensitivity-labels-versions.md
Service: o365-seccomp
GitHub Login: @cabailey
Microsoft Alias: cabailey

The only time I've heard this term is internally at Microsoft. You won't find it in any official documentation & I'm told the correct name is "Outlook in a mobile browser".
No, it doesn't support sensitivity labels and the documentation lists what we do support, not list email clients that we don't support. That's standard for documentation because the list could never be complete, and any omissions could be construed as therefore supported. For example, sensitivity labels aren't supported by our Windows built-in Mail app, either. Sensitivity labels are supported with Outlook for Windows, Outlook for Mac, Outlook for iOS, Outlook for Android, and Outlook on the web. These are the "flavors" of Outlook in the table.
Since you're a Microsoft employee, please feel free to reach out to me on internal channels about the documentation for sensitivity labels rather than using GitHub Issues.

Hello Carol,
Thank you very much for your input, it's really appreciated
I agree with everything you mention
My apologies for reaching out this way, I understood it was the correct one


move hook tests to adapter zone

resolves #

Description

Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

Could we move the .sql files into a python file and import them into the tests? Possibly combine the 2 SQL inputs into one seeds.py and import where needed.

Meant to open this as a draft, still iterating!


Failed to mine block, runtime error

Hi there,
I keep getting this error ' Failed to mine block, runtime error' in the mining log.
The node is syncing successfully with the main net Roma, and the cuda29 executable is in the same directory with epoch.
Steps to Reproduce the Problem

after synced, just ./epoch restart

I tried to run cuda29, seems it functions ok:
./cuda29
GeForce GTX 1070 Ti with 8119MB @ 256 bits x 4004MHz
Looking for 42-cycle on cuckoo30("",0) with 50% edges, 64*64 buckets, 176 trims, and 64 thread blocks.
Using 6976MB of global memory.
nonce 0 k0 k1 k2 k3 a34c6a2bdaa03a14 d736650ae53eee9e 9a22f05e3bffed5e b8d55478fa3a606d
Seeding completed in 66 + 46 ms
2-cycle found
30-cycle found
754-cycle found
findcycles edges 63447 time 27 ms total 358 ms
0 total solutions
Logs, error output, etc.
2018-11-30 18:24:48.723 [info] <0.12163.0>@aec_conductor:start_mining:674 Starting mining
2018-11-30 18:24:48.787 [error] <0.12163.0>@aec_conductor:handle_mining_reply:718 Failed to mine block, runtime error; retrying with different nonce (was<PHONE_NUMBER>8136980657). Error: {execution_failed,{status,127}}
(this error keeps repeating over and over again)

fixed, made a mistake, the cuda29 should be under '/node/lib/aecuckoo-0.1.0/priv/bin'


Update log_reports.py

What does this PR do?
Improved logging, with more detailed and structured logging messages.
Handling of various types of exceptions, with appropriate error messages and logging.
Enhancements to the Slack message formatting, using Slack's rich message features.
Caching and incremental processing of log files, to improve the script's efficiency.
Automated testing, with unit tests and integration tests to ensure the script's reliability and maintainability.

What's the status of this PR? I've converted it to a draft because it seems to me that this is its current state.


fix(deps): update module github.com/stackitcloud/stackit-sdk-go/services/sqlserverflex to v0.3.0

This PR contains the following updates:

Package
Type
Update
Change

github.com/stackitcloud/stackit-sdk-go/services/sqlserverflex
require
minor
v0.2.0 -> v0.3.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Update done in #409


Perform escaping of errors in templates

While fixing #355, I've found that we are escaping diff description in utils, that are used by assertions and actions, and step name in runner. In order to fix #355, I will need to escape diffs in assertions, and relatedSourceCode on server side. So after that we will have escaping in 4 different places, 3 on a client, and 1 on the server.
According to @kirovboris ' comment and @inikulin 's comment we are doing escaping to avoid breaking an error template parser because we use HTML markup for errors.
So I'm curious, why wouldn't we do this just right in error templates, because here we are forming HTML markup for an error and here we can escape all including data?
\cc @inikulin @AlexanderMoskovkin @kirovboris @helen-dikareva

The relatedSourceCode is always a string. We can't detect whether it's html or string with brackets on the templates side.

@kirovboris Related source code is always a js-code. So we can always escape it before inserting into template (since template is HTML)

Just the fun fact: can you find a tag in an expression below? parse5 can :)
ok(5<x-x>5)

Shame, no spoilers here, but tag is <x-x>.
What is more interesting, that it's a valid JS expression after all.

@inikulin, @AndreyBelym
I agree. Let's implement html escaping in the templates.

I agree too

@AndreyBelym Yap, any character except whitespace, > and / is perfectly valid in HTML5 tag name: https://github.com/inikulin/parse5/blob/master/lib/tokenizer/index.js#L767


Waratah v1.6.0 flagged as Trojan.Marsilla

Hello there,
Firefox blocked this download, and when allowed and scanned by Windows Defender, it came up clear. However, running through VirusTotal gives a number of hits against Trojan.Marsilia... I've checked the shasum across multiple machines and networks to ensure it wasn't something modifying in-flight
https://www.virustotal.com/gui/file/e7987ccb1859620039790f2b98399a1703d2c4be3ccd8b263426532c5d59822d/detection
Runs fine in a sandbox without anything weird, so seems like a false positive, but a pretty loud one that may need some clearing up.
Cheers,
Nick

Thanks mate for reaching-out about this.  We had a similar problem with Windows Defender (see closed issues), and I was able to get it validated and allow-listed.
I've reached-out to Firefox, but I don't have high hopes of getting this resolved (as there doesn't seem to be a formal request mechanism for malware)


Replace golicense with lichen

golicense is no longer actively maintained and does not support Windows
binaries. To avoid issues like #1663 in the future, we switch over to
lichen, a project heavily inspired from golicense. It's a fairly recent
project but works very well.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@57dcaec). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1800   +/-   ##
=======================================
  Coverage        ?   42.28%
=======================================
  Files           ?      192
  Lines           ?    16363
  Branches        ?        0
=======================================
  Hits            ?     6919
  Misses          ?     8461
  Partials        ?      983

Flag
Coverage Δ

kind-e2e-tests
42.28% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@57dcaec). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1800   +/-   ##
=======================================
  Coverage        ?   42.28%
=======================================
  Files           ?      192
  Lines           ?    16363
  Branches        ?        0
=======================================
  Hits            ?     6919
  Misses          ?     8461
  Partials        ?      983

Flag
Coverage Δ

kind-e2e-tests
42.28% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

/skip-all

@tnqn could I get another approval? I realized I had forgotten to remove the GITHUB_TOKEN env variable from the workflow file, and it is no longer needed with lichen

/skip-all


PFUser. loginWithUsernameInBackground() is crashing with an assertd[6900] error. Help!

Created a simple iOS program to demonstrate parse. When calling loginWithUsernameInBackground() in a simple IBAction, tapping the button causes the program to freeze, apparently within the loginWithUsernameInBackground().
Steps to reproduce
Please include a detailed list of steps that reproduce the issue. Include curl commands when applicable.

Launch program (see attached)
Tap on login button
Watch it crash, with the login button appearing that it is still
I can't attach the file, but here is the code in question (and a link to the zipped project is here).

@IBAction func login(sender: AnyObject) {
print(emailTF.text)
print(passwordTF.text)
PFUser.logInWithUsernameInBackground(emailTF.text!, password: passwordTF.text!, block:{(user, error) -> Void in
if error != nil{
print(error)
}
else {
// Everything went alright here
self.displayAlertWithTitle("Success!", message:"Login successful")
    }
}
)
print("We are done")

}
The first two print() statements execute ... the second is never reached. There is an assertd message in the iOS simulator logs.
Expected Results
loginWithUsernameInBackground() shouldn't lock up
Actual Outcome
It does.
Environment Setup
Server
parse-server version (Be specific! Don't say 'latest'.) :
Operating System: iOS 9.3
Hardware: iPhone 6Plus
Localhost or remote server? (AWS, Heroku, Azure, Digital Ocean, etc): back4app.com
Database
MongoDB version: N/A
Storage engine: N/A
Hardware: N/A
Localhost or remote server? (AWS, mLab, ObjectRocket, Digital Ocean, etc): N/A
Logs/Trace
Oct 18 14:43:40 theviewahvalous assertiond[6900]: assertion failed: 16A323 13E230: assertiond + 15801 [3C808658-78EC-3950-A264-79A64E0E463B]: 0x1

So, a little more information might be helpful. I can create a starter project and it always works. If I add a file -- Movie -- even though I'm not instantiating any Movie objects -- it crashes. The file is attached. Does anyone see a problem with this file? This seems so weird ...
Movie.swift.zip


Is .NET 5 supported?

So .NET 5 has been out since November, as far as I can tell it will not work properly with OData, Swagger, and Api-Versioning.
Background:
I have a netcoreapp3.1 project that has been running with the following libraries
Microsoft.AspNetCore.OData.Versioning 4.1.1
Microsoft.AspNetCore.OData.Versioning.ApiExplorer 4.1.1
I have 3 groups of controllers
v1 - that is using ControllerBase
v1.1 - that is using a mix of ODataController and ControllerBase
v1.2 - that is using ODataController
this has all worked correctly for the most part and we have tests around everything, exercising the different scenarios.
We are only using attribute routing ie: api-version=1.1 etc...
We are updating our services to .NET 5 and I have done the following:

[x] Update Microsoft.AspNetCore.OData.Versioning to 5.0.0
[x] Update Microsoft.AspNetCore.OData.Versioning.ApiExplorer to 5.0.0
[x] Update other Microsoft libraries to 5.0.0 (if applicable)

What i'm seeing is that all of my OData test routes are no longer working using the route attributes (api-version=1.1)
etc..
Is this expected to work or am I too early on getting this integrated in to .NET 5

Please notice that the version 5.0 of Microsoft.AspNetCore.OData.Versioning.ApiExplorer and Microsoft.AspNetCore.OData.Versioning is not related to .NET 5(they depend on Microsoft.AspNetCore.Mvc.Versioning 4.2.0 which doesn't have a version '5.0').
In fact the author of ApiVersioning has no plan to support .NET 5(see https://github.com/microsoft/aspnet-api-versioning/issues/695#issuecomment-725195588). But you can still use ApiVersioning and .NET 5 together because .NET 5 is downward compatible with .NET Core.
This issue is a problem with Microsoft.AspNetCore.OData.Versioning.ApiExplorer 5.0.0 itself, not only in .NET 5 but also happens in .NET Core 3.x. See https://github.com/microsoft/aspnet-api-versioning/issues/710#issuecomment-748806590.

Please notice that the version 5.0 of Microsoft.AspNetCore.OData.Versioning.ApiExplorer and Microsoft.AspNetCore.OData.Versioning is not related to .NET 5(they depend on Microsoft.AspNetCore.Mvc.Versioning 4.2.0 which doesn't have a version '5.0').
In fact the author of ApiVersioning has no plan to support .NET 5(see https://github.com/microsoft/aspnet-api-versioning/issues/695#issuecomment-725195588). But you can still use ApiVersioning and .NET 5 together because .NET 5 is downward compatible with .NET Core.
This issue is a problem with Microsoft.AspNetCore.OData.Versioning.ApiExplorer 5.0.0 itself, not only in .NET 5 but also happens in .NET Core 3.x. See https://github.com/microsoft/aspnet-api-versioning/issues/710#issuecomment-748806590.

@anranruye Thank you for the concise information. You are correct just downgrading to 4.1.1 fixed my issues and is working fine with .net 5.

@anranruye Thank you for the concise information. You are correct just downgrading to 4.1.1 fixed my issues and is working fine with .net 5.


Add ssh_destroy_resource

Adds a new resource named ssh_destroy_resource which acts exactly like ssh_resource, but only runs during destroy. The destroy functionality reuses the schema and functions from ssh_resource.
The create function stores the empty state with a unique ID. Read and update operations are no-ops.
Closes #37

@arbourd looking at the changeset and at the issue you linked to in the Terraform repo what if we introduce a when argument to the existing ssh resource. The possible values would be:
create - Default value, same behaviour as now
destroy - Triggers the file and command uploads at destroy time
This would satisfy your use case without introducing a new resource type and the code duplication. What do you think?

I think that is very elegant, especially if you are coming from provisions @loafoe

Hi @arbourd , please see the when attribute implementation in #39 . I've also added some acceptance tests and it works as expected on my test SSH server. If the syntax looks good I'd like to merge and release it for your consideration.


Feature suggestion: use custom filter to split sections

Hello! Earlier in the TiddlyWiki Talk, I mentioned that I would like to be able to segment tiddlers not only based on first and second-level headers, but also in a more flexible way. After that, I tried to write a custom filter to achieve this.
First, add a new tiddler to define the new filter:
/*\
title: $:/core/modules/filters/sectionsplit.js
type: application/javascript
module-type: filteroperator
author: Gk0WK(nmg_wk @yeah.net)

Split sections of wikitext

\*/
(function() {

    /*jslint node: true, browser: true */
    /*global $tw: false */
    "use strict";

    /*
    Export our filter function
    */
    exports.sectionsplit = function(source, operator, options) {
        var options = {};
        operator.operand.split('+').forEach(option => {
            options[option] = true;
        })
        var results = [];
        source(function(tiddler, title) {
            var section = [];
            title.split('\n').forEach(line => {
                var newSection = false;

                if ((options.all || options.h1) && /^!\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.h2) && /^!!\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.h3) && /^!!!\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.h4) && /^!!!!\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.h5) && /^!!!!!\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.h6) && /^!!!!!!\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.hr) && /^---+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.blank) && /^\s*$/.test(line)) {
                    newSection = true;
                }

                if (newSection && section.length > 0) {
                    results.push(section.join('\n'));
                    section = [];
                }
                section.push(line);
            });
            results.push(section.join('\n'));
        });
        return results;
    };

})();

This toddler also has:

type: application/javascript
module-type field: filteroperator
title: $:/core/modules/filters/sectionsplit.js

Then, change the filter expression in line 11 of source/section/macros/main.tid to [<sourceText>sectionsplit[h1+h2+h3]!is[blank]regexp<nonWhitespace>].
Then TW-Section can split toddler by h1, h2 and h3. You can change it to [h1+h2+h3+h4] and so on to take more split rules. Now this filter support:

h1 ~ h6
hr
blank, means blank line
all, means all above

If you think this filter is useful, I can add more rules to it. 😄

if ((options.all || options.h1) && /^!\s+[^\n]+$/.test(line)) {

That's not how the TW parser tests for headings. See: !!.myStyle heading text comes here and the whitespace after !! is optional.

Hello! Earlier in the TiddlyWiki Talk, I mentioned that I would like to be able to segment tiddlers not only based on first and second-level headers, but also in a more flexible way. After that, I tried to write a custom filter to achieve this.
Then, change the filter expression in line 11 of source/section/macros/main.tid to [<sourceText>sectionsplit[h1+h2+h3]!is[blank]regexp<nonWhitespace>].

Thank you, I welcome your contribution! Unfortunately I know little JS. but this is a nice filter if it can accept different rules!
Would you please also add rules to split on <tag ...> contents </tag> a tag can be  section,  or article or other sematic html5 tag!

Then TW-Section can split toddler by h1, h2 and h3. You can change it to [h1+h2+h3+h4] and so on to take more split rules. Now this filter support:

h1 ~ h6
hr
blank, means blank line
all, means all above

does this means I can pass  sectionsplit[h1+h2]  and split only on heading one and two?
Also as @pmario said, in Tiddlywiki we have !!.myStyle heading text comes here which means you can apply styles through css classes! Would you please correct the filter to handle such cases!
Thank you

If you think this filter is useful, I can add more rules to it. 😄

That's not how the TW parser tests for headings. See: !!.myStyle heading text comes here and the whitespace after !! is optional.

@pmario Thank you! I have changed the expressions of header checking.

does this means I can pass sectionsplit[h1+h2] and split only on heading one and two?

@kookma Yes, sectionsplit[] (but useless), sectionsplit[h2], sectionsplit[h1+h2], sectionsplit[hr+h3+h1] can also work. The order and number of rules do not matter.

It is possible to split tiddler with HTML tags, but there are some problems:

I need to write a state machine to find opened/closed tags, the regex of </[^>]+>/gim can work on most HTML text but some special ones like <tag attr=">"></tag>.
The HTML section may not start at the head of the line, and may not end at the tail of the line. For example:

* Hello <div>
World
</div>!

The HTML sections may not close or close wrongly, how to solve them?

So, I think there are some questions for us to discuss before adding this important rule.

I also have another question. How does the plugin work? If I remove some blank lines and trim returned text (for example, \n123\n   -> 123), can the plugin work in the proper way?
Thanks all.

Updated js code:

JS Code
/*\
title: $:/core/modules/filters/sectionsplit.js
type: application/javascript
module-type: filteroperator
author: Gk0WK(nmg_wk @yeah.net)

Split sections of wikitext

\*/
(function() {

    /*jslint node: true, browser: true */
    /*global $tw: false */
    "use strict";

    /*
    Export our filter function
    */
    exports.sectionsplit = function(source, operator, options) {
        var options = {};
        operator.operand.split('+').forEach(option => {
            options[option] = true;
        })
        var results = [];
        source(function(tiddler, title) {
            var section = [];
            title.split('\n').forEach(line => {
                var newSection = false;

                if ((options.all || options.h1) && /^!(\.[\w\-]+)?\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.h2) && /^!!(\.[\w\-]+)?\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.h3) && /^!!!(\.[\w\-]+)?\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.h4) && /^!!!!(\.[\w\-]+)?\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.h5) && /^!!!!!(\.[\w\-]+)?\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.h6) && /^!!!!!!(\.[\w\-]+)?\s+[^\n]+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.hr) && /^---+$/.test(line)) {
                    newSection = true;
                } else if ((options.all || options.blank) && /^\s*$/.test(line)) {
                    newSection = true;
                }

                if (newSection && section.length > 0) {
                    results.push(section.join('\n').trim());
                    section = [];
                }
                section.push(line);
            });
            results.push(section.join('\n').trim());
        });
        return results;
    };

})();

@Gk0Wk thank you!
I decided to keep the simple version only a wikitext plugin (without JS). Then we can have subplung or advanced SE to include the JS and more sophisticated parsing!
I think for now

only sectionize the part started by a heading (e.g. !, !!, etc.)
ignore the part at the beginning of tidler with no heading

Next

support for ...
OR some custom tag like ....

but not any tag!

@kookma Hi. I'm sorry to inform you that the TiddlyWiki core parser does not parse results with substring location information, which makes the syntax tree unusable for splitting sections. I think a better way is still to write a separate syntax parser. As it happens, I am implementing such a parser in my TW5-CodeMirror-Enhanced project, and if I finish a simple parser I will try to use it in TW_Section.
BTW, I encountered some problems in using TW_Section, which are supposed to be caused by overly simple regular expressions. TW_Section does not correctly divide and render the following two cases:
case 1:
<<<
! a
<<<

case 2:

```
! a
```


[Bug]: Behavior of pixel_metrics/image_metrics

Describe the bug
When I setup my engine, I want to log all metrics available:
# https://anomalib.readthedocs.io/en/latest/markdown/guides/reference/metrics/index.html
engine = Engine(
                task=self.TASK,
                default_root_dir=self.models_path,
                logger=wandb_logger,
                max_epochs=max_epochs,
                callbacks=callbacks,
                pixel_metrics=[
                    "AUPIMO",
                    "AUPR",
                    "AUPRO",
                    "AUROC",
                    "AnomalyScoreDistribution",
                    "BinaryPrecisionRecallCurve",
                    "F1AdaptiveThreshold",
                    "F1Max",
                    "F1Score",
                    "ManualThreshold",
                    "MinMax",
                    "PIMO",
                    "PRO",
                ],
                accelerator="auto",
            )

However, the Engine always loads the AnomalibMetricCollection:
   ┃ Name                  ┃ Type                     ┃ Params ┃ Mode  ┃
┡━━━╇━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━╇━━━━━━━┩
│ 0 │ model                 │ DraemModel               │ 97.4 M │ train │
│ 1 │ loss                  │ DraemLoss                │      0 │ train │
│ 2 │ _transform            │ Resize                   │      0 │ train │
│ 3 │ normalization_metrics │ MetricCollection         │      0 │ train │
│ 4 │ image_threshold       │ F1AdaptiveThreshold      │      0 │ train │
│ 5 │ pixel_threshold       │ F1AdaptiveThreshold      │      0 │ train │
│ 6 │ image_metrics         │ AnomalibMetricCollection │      0 │ train │
│ 7 │ pixel_metrics         │ AnomalibMetricCollection │      0 │ train │

I do not understand why? The same behavior can be observed in the notebook below. Here, "AUROC" is selected, however AnomalibMetricCollection shows. I am experimenting with the Draem model. I am not so sure if this is a bug or if I do not understand some part of it, but after googling for an hour I am kind of stuck here. https://github.com/openvinotoolkit/anomalib/blob/404e896c769aa069f77deb79081e19024e8596a8/notebooks/600_loggers/601_mlflow_logging.ipynb#L1083
Dataset
Other (please specify in the text field below)
Model
DRAEM
Steps to reproduce the behavior
See above
OS information
OS information:

OS: Ubuntu 24.04
Python version: 3.12
Anomalib version: recent
PyTorch version: recent
CUDA/cuDNN version: 12.4
GPU models and configuration: RTX 4090
Any other relevant information: Custom dataset in mvtec format. Simpler experiments with Padim worked.

Expected behavior
My selected metrics should be chosen.
Screenshots
No response
Pip/GitHub
pip
What version/branch did you use?
No response
Configuration YAML
---

Logs
---

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Also, there is no information about the AnomalibMetricCollection in the docs:

This is mostly an implementation detail. If you specify multiple metrics, they get stored inside a special object. This way you can update the values in all metrics simultaneously with a single update call and then compute them with a single compute call.
For more details, you can check AnomalibMetricCollection class, or it's parent class from pytorch lightning MetricCollection.
I think that the docs don't show it, as it's not added to __all__ inside __init__.py. @ashwinvaidya17 should this maybe be included?

As @blaz-r pointed out, it is not shown since AnomalibMetricCollection is not within __init__.py.
With that being said, the class does not have any specific implementational detail, but just wrapping the original Lightning MetricCollection class. One idea would be to expand its docstring with examples and a link to the Lightning documentation as they have comprehensive documentation
https://lightning.ai/docs/torchmetrics/stable/pages/overview.html#metriccollection
I need to double check how the new metric design will impact the metric collection though
https://github.com/openvinotoolkit/anomalib/pull/2326

Thanks for the quick response, however I am not sure if I understand. When I log the training with wandb, the metrics I chose do not show up. wandb only shows the following ones:

train_loss_step
image_F1Score
epoch
pixel_F1Score
pixel_AUROC
image_AUROC
train_loss_epoch

So I guess I assumed these were simply the metrics included in the AnomalibMetricCollection, but that does not seem to be correct?

That is quite unusual. Are the metrics logged in the shell, but not on wandb? I think that the problem here is unrelated to AnomalibMetricCollection.

Works now, not exactly sure what I did :D Reinstallation helped.

Great


no check for consecutive times lots when training

I've noticed that the get_item method in the dataset class does not proof if the 5 np.arrays used for training are actually 5 consecutive timeslots. I'm referring to this line:
arr = np.array(self.dataset.get(self.keys[i+j])) and
y = np.squeeze(processor.data_preprocessing(np.array(self.dataset[self.keys[i+4]])[np.newaxis,:,:])) #corrected after github issue
E.g. the first 3 arrays can be the 01.01.2020 5:00, 01.01.2020 5:05, 01.01.2020 5:10 but the next is 01.01.2020 5:20 and should not be used.

Hi @JKupzig,

Thank you for the brief explanation @hydrogo!


Out of Ordering Styling Between Dev and Build

[ x ] I have tried upgrading by running bundle update vite_ruby.
[ x ] I have read the troubleshooting section before opening an issue.

Description 📖
When there are multiple Vue (might happen in other frameworks) entrypoints that depend on the same child component, which causes a separate chunk, the shared component's css file gets loaded in different orders in dev and build mode.
Reproduction 🐞
Repro

Vite Ruby Info
Run bin/rake vite:info and provide the output:
bin/vite present?: true
vite_ruby: 3.2.14
vite_rails: 3.0.13
rails: 7.0.4
node: v16.13.1
npm: 9.2.0
yarn: 1.22.11
pnpm: 7.9.3
ruby: ruby 2.7.2p137 (2020-10-01 revision 5445e04352) [x86_64-darwin20]

installed packages:
vite-rails-test@ /Users/jerisman/Development/vite-rails-test
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
└──<EMAIL_ADDRESS>

Screenshots 📷
Dev:

Build:

Would changing this line to be imports + entries instead of entries + imports resolve the issue? What would be the drawbacks of doing so?

Would changing this line to be imports + entries instead of entries + imports resolve the issue? What would be the drawbacks of doing so?

In short, no.

This problem is related to how code is bundled vs how code is loaded during development, and is something that I've observed for years (using Webpack).
It boils down to CSS specificity, and usually comes up when you have CSS rules with the same specificity in different components. In your example, if the bundler puts SharedComponent before Entry, the .parent rule wins, and viceversa.
For example, the order could change if the bundler moves SharedComponent to a separate chunk.

Unfortunately, this kind of problem is hard to spot (unless you are doing visual regression testing).
A few ways to prevent it:

Use CSS rules with higher specificity when "overriding" component styles
Use props to control styling, letting the component handle variants


dependency bump

should fix a lot of issues other people were gettng when using --turbo in the dev environment

can we have the person who has a problem with turbo try this?
@Ayham1590


Validate / Implement RecentMessageCache

Need to understand if RecentMessageCache is implemented and working correctly.
This is used in Chat and ChatService. This has a dependency on RoomCache so that will need to be verified as part of this.

Update: waiting until justine/persisterrepository is merged as it has a lot of changes. In the meantime, found two places where  room.ChatMessages.Add(chatMessage); was being updated both in chat.cs and chatService.cs.
Need to look into how it's actually being used as Heather mentioned they had implemented RecentMessageCache as a list.

Added David Fowler's tests "GetRecentMessagesReturnsTheLastNMessages" and "ThreadSafety". Both pass.
After stepping through both old and new Jabbr, I see that old Jabbr's RecentMessageCache doesn't seem to cache messages effectively- in ChatService.cs, _recentMessageCache.Add(chatMessage); uses NoopCache in RecentMessageCache.cs, which is empty. Regardless, when you load up an old room it pulls messages from the database, which we don't have working in Jabbr core. Once that's working messages should be cached.

UPDATE: David Fowler said don't use RecentMessageCache in Jabbr-Core. He said to forget it for now.
Todo: remove the code for RecentMessageCache.


NFS folder root user

hello team,
I have a weird issue . i use NetApp storage as NFS  volume in my container as /data using complete NFS volume and IP address of target in my docker host/container  for example: /data using PV and PVC .  My PV, PVC is active and bounded to my POD.
I see my container mounted the volume from PV,PVC to my pod and i see the /data, however the files under /data path  ownership is not showing "root"   and i expect to have "root" owner . I get this weird ownership "99" number and even if do "chown -R root:root file" the ownership is not getting updated.
From our netapp storage volume is having full permission to these hosts and i dont see any issue becuase i mounted on another docker host and i see the permissions as root , however in the kubernetes container side it is not showign "root" ownership
dr-xr-xr-x.  6  99 sapinst 4096 Apr 27 18:20 K8S
Does anyone come across this issue and what is the root cause and fix .
BR
Rushi.

hi
i am using NFS4 in my PV mount options like below , when i use NFS=3 i get errors because the container seems
apiVersion: v1
kind: PersistentVolume
metadata:
name: pv0003
spec:
capacity:
storage: 5Gi
volumeMode: Filesystem
accessModes:
- ReadWriteOnce
persistentVolumeReclaimPolicy: Recycle
storageClassName: slow
mountOptions:
- hard
- nfsvers=4.0
nfs:
path: /tmp
server: <IP_ADDRESS>

error using NFS3 , i get this error.
ount.nfs: parsing error on 'nfsvers=' option
Warning  FailedMount  8s  kubelet, node4  MountVolume.SetUp failed for volume "vacik8snfs" : mount failed: exit status 32
Mounting command: systemd-run
Mounting arguments: --description=Kubernetes transient mount for /var/lib/kubelet/pods/11fb184d-a440-44df-8e12-679edb04a18f/volumes/kubernetes.io~nfs/vacik8snfs --scope -- mount -t nfs -o hard,nfsvers=3.0 host:/vol1 0001 /var/lib/kubelet/pods/11fb184d-a440-44df-8e12-679edb04a18f/volumes/kubernetes.io~nfs/vacik8snfs
Output: Running scope as unit run-17255.scope.
mount.nfs: parsing error on 'nfsvers=' option

it turned out a NFS release issue .


Release in website contains .git directories

phantom, steam and sticko themes in release on website contains .git directories.

Yes, these are only being included in the beta for testing. This is a manual process at the moment, but the release gulp task will eventually pull in the default theme files (not the entire repo) automatically :)


[CHEF-2894] - use same values for initialize as knife cookbook create

Fixes CHEF-2894. Using the same default values as knife cookbook create means that if the values are not present in the metadata we're not making decisions for the user, particularly the license.

Merged to master.


🐛 Further cleanup to tmc-related e2e tests

Signed-off-by: David Festal<EMAIL_ADDRESS>
Summary
Further cleanup to tmc-related e2e tests:

we should always add the CRDs for deployments, services, endpoints in the fake pclusters, since they are now all required by the Syncer to correctly work.
In a Syncer Virtual Workspace test, we should have been waiting for the Syncer finalizer bing set before trying to patch the resource.
Rewite of the Syncer virtual workspace tests to avoid starting 2 syncers when it's not necessary.
Cleanup the way Syncer are started to sync user workspaces, and remove the need of the BoundWorkload method.

Additional fixes on code bugs found at this occasion:

Skip the kcp-system in namespace and resource scheduling (was obviously bringing much load for nothing)
Small fix in the workload sync command, to avoid adding a nil APIExport reference in the SupportedAPIExports
...

Related issue(s)
Required changes that should fix some flakes and race-condition-related bugs.

/test e2e-shared

/test shared

/test e2e-shared

/test e2e-sharded

/lgtm

/approve


fix golangci-lint errors

see https://github.com/opiproject/sztp/actions/runs/9548346846/job/26315321754 for example

@glimchb i would like to work on this issue


ipfs service: dataDir backwards compatibility, fix dataDir existance detection

Motivation for this change
Because of #25531, older instances of IPFS directories weren't being detected (see #25759). Also the check for the existance was incorrect originally. This PR fixes both of these issues.
I tested this on the release-17.03 branch, since master currently doesn't build for me, the change is pretty trivial though.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Pinging @mguentner

Added a release note for this change including the script to migrate, ready to merge.

@fpletz Did it, used squashing for the first time, it's not as bad as I thought it would be :D

Cool, thanks. I just finished some tests. Let's merge this.


Create a Contributing.md for the project

I want to create a contributing.md file for this project so that the new contributors can understand the process of contribution process.

I want to create a contributing.md file for this project so that the new contributors can understand the process of contribution process.

go ahead

conversation moved to #12


Removed unused code

Was looking at the code and it seems that this is no longer in use, this PR clean up the unused code.

Thanks for this! 👍


Is it possible to load local images inside HTML?

Is this a bug report or a feature request?
Feature request

Have you read the guidelines regarding bug report?
Yep.
Have you read the documentation in its entirety?
(Write your answer here.)
Yep.
(Write your answer here.)
Is the bug specific to iOS or Android? Or can it be reproduced on both platforms?
Both

Duplicate #82. I'll re-open it if you can't find a way to make it work.


Add login_strategy to config

Currently strategy methods for logging in must be called directly, eg:
Doorman.Strategy.login.

Rename Strategy to Login.
Add Login behavior.
Add login_strategy config options.
Add public methods on Doorman to delegate to Login modules.

:shipit:


Missing authentication header for appComponent dependencies

This fixes issue #162.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Multiple instructions/program calls in one transaction.

I am quite the Solana noob and I can't figure out how to include multiple instructions or program calls in a single transaction such that it is only necessary to sign with the wallet once.
For example, instead of having to first end staking and afterwards end cooldown I would like to include both in a single click (and single wallet signing).
I would highly appreciate if someone could provide some code examples.

I don’t know that this is possible unless you deploy your own version of the rust program and remove that step in the process.
@ilmoi - why is there a cooldown period that is not optional to include in the farm anyway? This doesn’t seem to be a common thing in most projects, so why would it be a mandatory step?
I hope that helps.
On Mon, Mar 28, 2022 at 4:51 AM, torbentemp @.***> wrote:

I am quite the Solana noob and I can't figure out how to include multiple instructions or program calls in a single transaction such that it is only necessary to sign with the wallet once.
For example, instead of having to first end staking and afterwards end cooldown I would like to include both in a single click (and single wallet signing).
I would highly appreciate if someone could provide some code examples.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

@numisman Do you mean specifically just the "end staking" + "end cooldown" combination or generally any combination of program calls?
It is possible to include multiple instructions in a transaction on Solana and the program calls are just instructions, no?

@numisman Do you mean specifically just the "end staking" + "end cooldown" combination or generally any combination of program calls?

Yes, add "end staking" + "end cooldown" instruction in one transaction, I do that on my project.

It should be possible to include multiple instructions in a transaction on Solana and the program calls are just instructions, no?

Also yes, take a look at this multiple NFT mint in one transaction.
https://github.com/maxwellfortney/next-candy-machine/blob/a3fc776a789ebee766552325e3362b1aa5b0ec0d/utils/candyMachine.ts#L363
join the instruction(s) and signers in an array matrix and sign it in a transaction once

That's pretty awesome!  How's the cost impact on something like that?  Does it grow based on the number of internal transactions or is it one fee for the container transaction?
Sorry - not too clear on the Rust side right now.....looking to learn.

Thank you so much! I will look into the code you linked!


Fix the terrible holdover hack in Heuristics

This is awful to do at module level:
"use strict"
var LETTER_FREQUENCY;
var LETTER_SHARE;
var COMPACT_DICT;

function setLetterShare(letterShare) {
  LETTER_SHARE = letterShare;
}

function setLetterFrequency(letterFrequency) {
  LETTER_FREQUENCY = letterFrequency;
}

function setCompactDict(compactDict) {
  COMPACT_DICT = compactDict;
}

Fixed


Implement WSGI app interface

Zappa does not actually implement a WSGI app wrapper, it does translate a JSON dict into a WSGI environ, perhaps the function in wsgi.py should be named create_wsgi_environ instead.
The zappa_django project contains a lambda handler however that almost implements a proper WSGI app wrapper.
That handler should be extracted into zappa and fixed to truly call a WSGI app, this would make it much easier to add support for any framework using WSGI.

I can take a quick stab at a PR todo this.

I have made some progress towards this here: https://github.com/Doerge/Zappa but my WSGI understanding is limited so any comments or other efforts are most welcome!

You might take a look at the werkezeug stuff in https://github.com/Miserlou/django-zappa/issues/9

Btw. @bbangert join us on Slack here: https://slackautoinviter.herokuapp.com/

I took a rough guess at this func here:
https://github.com/bbangert/Zappa/blob/master/zappa/wsgi.py#L117

Awesome!
One thing that springs to mind: I think we need exceptions raised on anything but 200. There is a small explanation as to why over here on issue 28.

Implemented in 0.14.0


default addr for network '<IP_ADDRESS>' unknown

Hello, I met across a problem, could you help me?
My Environment

Mac os Catalina
xk6_0.4.5_mac_amd64.tar.gz

What I did
// test.js
import sql from 'k6/x/sql';

const db = sql.open('mysql', 'mysql://root:<password>@<IP_ADDRESS>/<database>?charset=UTF8MB4');

export function setup() {

}

export function teardown() {
    db.close();
}

export default function () {
    db.exec('SELECT NOW();');
    let result = sql.query(db, 'SELECT NOW();');
    console.log(result);
}

./k6 run test.js

What happened
it returns...
ERRO[0000] default addr for network '<IP_ADDRESS>' unknown
        at go.k6.io/k6/js/common.Bind.func1 (native)

What I want to do
connect to mysql successfully

Hi, I just merged #9 which adds a MySQL example.
The issue you're having is because you need to specify the IP in the connection string between parenthesis.
So the following worked for me (tested with Docker):
const db = sql.open('mysql', 'root:<password>@(<IP_ADDRESS>)/<database>?charset=UTF8MB4');


Need to bump version of vulkano-win and vulkano-shaders to 0.23.0

Unless I am missing something on my part (always possible :) ), it appears that the .toml in the sub-crates has been updated to depend on 0.23.0, but the crates themselves have not had their version bumped, therefore the change is lost on crates.io and the subcrates continue to depend on 0.22.0.
This breaks building (vulkano_win::required_extensions() returns a different type than Instance::new() expects, for example) when using the crates.io repositories:
vulkano = "0.23.0"
vulkano-shaders = "0.22.0"
vulkano-win = "0.22.0"

The subcrates need to be bumped too if their dependencies have been modified.

It looks like that @Eliah-Lakhin just forgot to cargo publish the other two crates. The version is already bumped.

Well, vulkano-win/Cargo.toml also still specifies the crate version as 0.22.0.

@apessino, @faulesocke, @Rua Many apologies for the issue. I totally forgot to publish vulkano-win and vulkano-shaders to crates.io. Now the issue resolved, all missing 0.23 crates just published and should be available for pulling from crates.io. Thank you very much to pointing out to this issue!


Defining a document title with Merge endpoint

I'm currently using ghostcript to merge PDFs and would like to migrate to this nice gotenberg API.
With gs, I use an option to specify the title :
'-c \'[ /Title MyTitle /DOCINFO pdfmark\'',
Is there such a possibility with gotenberg ?

Not currently, but it may be interesting to set/update metadata to generated PDFs :)

Related:

#207
#330


How to process the whole batch

Currently I am relying on __get_item(self,index)__ to tokenize sentence index. However, there is a way to more effectively tokenize the whole batch, instead of individual sentence. Could this be done in pytorch-meta, I have yet found one in the examples.

Most datasets currently available in Torchmeta rely on a hierarchy of three objects:

Dataset, which is simply a PyTorch dataset, which is responsible for getting the individual examples for a given label. For example it can be a dataset containing all the (20) examples of the letter A in Omniglot.
ClassDataset, which is producing the datasets for different classes. Each index of this class corresponds to a single label. For example in Omniglot, this contains 1028 elements, and class_dataset[0] returns an instance of Dataset (above) containing all the examples of images_background/Alphabet_of_the_Magi/character01.
CombinationMetaDataset combines multiple indices (for example (0, 1, 2, 3, 4)) to create a task over the corresponding labels, the individual indices corresponding to the ones in ClassDataset above.

Something you could do in your case is to tokenize all the elements of Dataset at once, because this is essentially a batch of data (from which the sampler is going to sample from to create the actual datasets for the task).
Another option could be to look into how to allow __getitem__(index) to get a batch (list) of indices for index. This is already possible in standard PyTorch datasets, and since Torchmeta datasets are essentially instances of PyTorch datasets this could be possible. I have tried to include that at some point in Torchmeta to improve sampling, but there was no particular improvement for image datasets, especially since processing by Torchvision transforms (e.g. image loading, Resize, etc...) only accept single images, so I ended up not continuing further.


cmd/cue: support stdin via - for import

Many of CUE's commands accept using - (hyphen) to specify that the input is from stdin, however cue import does not. This would be helpful when piping output from another program or API response to CUE.

This is possible today:
stdin x.json
exec cue import -o x.cue json: -
cmp x.cue x.cue.golden

-- x.json --
{
	"x": 5
}
-- x.cue.golden --
x: 5

Assuming this covers your use case?

Yes, this works great @myitcv (this issue was filed on my behalf based on a Slack question). Thanks for the clarification.
(I had missed the bit about the json: tag from the cue filetypes help text.)

The bug was opened on behalf of @christophermaier's question on Slack, and it appears Paul is right, so I'm going to close this for now :)


Add avatar_src to the Panoptes JWT

Can we add avatar_src to the encoded payload of our web tokens, so that the front-end doesn't have to make an API request to get the URL for the user avatar?
Related front-end PR: https://github.com/zooniverse/front-end-monorepo/pull/4803

If it's as straightforward as adding avatar: user.avatar_src here, then I can open a PR.
https://github.com/zooniverse/panoptes/blob/889e8459f467446eb262449fb03095f3df6bf34c/config/initializers/doorkeeper.rb#L75C1-L82

I’m not sure about that previous comment, because avatar_src is added by the User serialiser.

This has come up while adding auth to the new Next.js app. When we get a user from the Panoptes JWT, we don’t get the user avatar URL.

This would also be helpful for the user stats and logged in homepage.


Failed to initialize MelonLoader: Failed to load library!

Latest-Bootstrap.log
What does this mean?

Please ask for support in https://discord.gg/PuUCBZ3V
Also, repository is only meant for the installer. MelonLoader has its own separate repository.


More Packer migration guide

Seeing how many people struggled to convert their packer plugin spec to lazy, myself included. Especially it wasn't clear how different it's to set mode in packer vs lazy.
Hopefully this extra example can help a couple of poor soul out there in their exodus.

Thank you for the suggestion though. Will make the migration a bit clearer in the docs.


Reduce dart-services startup time from a minute to under 2 seconds.

Dart Services does a lot of work at startup, which needs to be either moved to image build time or eliminated completely.

A mixture of bumping up the minimum number of Cloud Run instances and https://github.com/dart-lang/dart-services/pull/660/files seems to have sorted out the issue of serving 500s while spinning up instances.


Reverse order query causes performance bottleneck of cassandra cluster

CassandraJournalStatements.scala
  def selectHighestSequenceNr =
    s"""
     SELECT sequence_nr FROM $tableName WHERE
       persistence_id = ? AND
       partition_nr = ?
       ORDER BY sequence_nr
       DESC LIMIT 1
   """

journal messages table of cassandra below:
CREATE TABLE IF NOT EXISTS pekko.messages (
  persistence_id text,
  partition_nr bigint,
  sequence_nr bigint,
  timestamp timeuuid,
  timebucket text,
  writer_uuid text,
  ser_id int,
  ser_manifest text,
  event_manifest text,
  event blob,
  meta_ser_id int,
  meta_ser_manifest text,
  meta blob,
  tags set<text>,
  PRIMARY KEY ((persistence_id, partition_nr), sequence_nr, timestamp))     --- sequence_nr default is asc order
  WITH gc_grace_seconds =864000
  AND compaction = {
    'class' : 'SizeTieredCompactionStrategy',
    'enabled' : true,
    'tombstone_compaction_interval' : 86400,
    'tombstone_threshold' : 0.2,
    'unchecked_tombstone_compaction' : false,
    'bucket_high' : 1.5,
    'bucket_low' : 0.5,
    'max_threshold' : 32,
    'min_threshold' : 4,
    'min_sstable_size' : 50
    };

When business application actors are restored from cassandra disks, there will be a large number of slow queries. Is there any way to eliminate this reverse order query? Thank you！

I am not sure why the Cassandra plugin needs this at recovering.
but we could optimize the execution plan by cutting the result set. i.e., WHERE sequence_nr > ? it will reduce the ordering on RDBMS.

Can we just use max?
SELECT max(sequence_nr) FROM $tableName WHERE
       persistence_id = ? AND
       partition_nr = ?`

This stackoverflow answer says that Cassandra supports it.
https://stackoverflow.com/questions/41717163/query-to-select-max-value-of-a-column-using-max-using-cassandra

@ZhiXingHeYiApple as a workaround until we can fix the code, is it possible for you to add an index to pekko.messages?
You could have one that indexes based on (persistence_id, partition_nr), sequence_nr DESC. I'm not a Cassandra expert. I know more about RDBMS.

Can we just use max?
SELECT max(sequence_nr) FROM $tableName WHERE
       persistence_id = ? AND
       partition_nr = ?`

This stackoverflow answer says that Cassandra supports it. stackoverflow.com/questions/41717163/query-to-select-max-value-of-a-column-using-max-using-cassandra

It will ignore the null value, but still sorting a large set. https://stackoverflow.com/questions/34246403/performance-of-max-vs-order-by-desc-limit-1


Update grpc-okhttp to 1.40.0

Updates io.grpc:grpc-okhttp from 1.39.0 to 1.40.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.grpc", artifactId = "grpc-okhttp" } ]

labels: library-update, semver-minor

@mergifyio update


How to config JNDI in datasource

No sample,below is error:
spring.shardingsphere.datasource.ds0.type=JNDI
spring.shardingsphere.datasource.ds0.jndiname=java:comp/env/jdbc/mymaster
What is correct?

type mean is your data source pool class name, JNDI is not  data source pool class name obviously.
You can use java API to generate datasource from JNDI and pass them to sharding rule config.

The above code is just a guess.
Hard code is not simple，we must bind datasource and all rules， even a bit cumbersome...
Additionally, Spring boot 2.0 supports JNDI now. So I hope Shardingsphere could support it.


Loc Automation - partial handback for Branch: ResxAC1035

Loc Automation - partial handback for Branch: ResxAC1035

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

jobs/test/500/Feature/index.zh-CN.resx
:white_check_mark:Succeeded

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

jobs/test/500/Feature/index.es-ES.resx
:white_check_mark:Succeeded

jobs/test/500/Feature/index.zh-CN.resx
:white_check_mark:Succeeded

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


install --force generates unnecessary warning

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?
[x] Confirmed this is problem with Homebrew/brew and not specific formulae? If it's a formulae-specific problem please file this issue at https://github.com/Homebrew/homebrew-core/issues/new

Pick an arbitrary formula, e.g., wallpaper, and forcibly install it: brew install --force wallpaper.  Observe: Warning: wallpaper: this formula has no --force option so it will be ignored!
This warning should not appear.  Rather, "--force" should be scrubbed out of the list of "options" passed to the formula (or should be whitelisted, or whatever).
This warning is especially jarring (and potentially confusing) when a user has just tried brew install foo (without the "--force") and been told by brew to "try again but with --force".  (E.g., perhaps foo is keg-only and and older version is already installed.)

Fixed. Thanks for the report.


Typo

In new_chapter_1 in vulkan_mainloop.md under the section Image Layouts in the second paragraph there is the word "scren." I believe this is supposed to be "screen." it is the second to last word in the second paragraph.
Not trying to be a stickler. this typo is very small and insignificant.

Fixed


[BUG/ISSUE] bundled vcredist is out of date

Bundled Vcredist 2015+ isn't the latest version. Latest one according to Winget installer is:
https://download.visualstudio.microsoft.com/download/pr/a16e1596-7fef-4570-8d63-fb9d4e72c820/D0CAFF8097BB9C43A3685686826867506BF6EDD667F7B90092CB0EB1B964A85E/VC_redist.x64.exe
and
https://download.visualstudio.microsoft.com/download/pr/a16e1596-7fef-4570-8d63-fb9d4e72c820/BB66E74A41ECF5DC09487608E477ACC4F78A38A0EC4FFE3100069C8A067DA29A/VC_redist.x86.exe
Context: after a fresh install, WingetUI detects an update for Vcredist 2015+

Thanks!
The file will ship with the next updates!


feat: switch to sqlite

Fixes #60
I did make one query now be fallible - it will now return an error if there is an invalid zap in the DB.
Unfortunately I'm not sure how to test this end-to-end :sweat_smile:

Looks like I left the dbg in as well. I'll take a look at this on Wednesday; it was working for me at some point, so maybe I undid the thing that made it work

I get the same error as @luckysori
docker logs -f nostrdice
2024-08-21 07:54:48  INFO nostr_dice::logger: Initialized logger
2024-08-21 07:54:48 DEBUG hyper::client::connect::dns: resolving host="bob"
2024-08-21 07:54:48 DEBUG hyper::client::connect::http: connecting to <IP_ADDRESS>:10009
2024-08-21 07:54:48 DEBUG hyper::client::connect::http: connected to <IP_ADDRESS>:10009
2024-08-21 07:54:48 DEBUG hyper::client::pool: pooling idle connection for ("https", bob:10009)
2024-08-21 07:54:48  INFO nostr_dice: Connected to LND: 03eb56540e181b297224c8a4820db70e8818b6925dd52e77a2a7f593f6f503bf53
Error: Failed to open database file

Caused by:
    0: error returned from database: (code: 14) unable to open database file
    1: (code: 14) unable to open database file

How are you running this (compose or just the container)? Also, did you get any other errors (e.g. during container build)? I'm not sure how this is possible given that the settings dictate that it should create the database file if it's missing. Perhaps the data folder itself is missing?

For me, just all works:
docker logs -f nostrdice
2024-08-22 13:55:47  INFO nostr_dice::logger: Initialized logger
2024-08-22 13:55:47 DEBUG hyper::client::connect::dns: resolving host="bob"
2024-08-22 13:55:47 DEBUG hyper::client::connect::http: connecting to <IP_ADDRESS>:10009
2024-08-22 13:55:47 DEBUG hyper::client::connect::http: connected to <IP_ADDRESS>:10009
2024-08-22 13:55:47 DEBUG hyper::client::pool: pooling idle connection for ("https", bob:10009)
2024-08-22 13:55:47  INFO nostr_dice: Connected to LND: 0226e59f462f11c864a466f7451493d2ab2f85e16990279e7b705f93b74a633e27
2024-08-22 13:55:47  INFO nostr_dice: Webserver running on http://<IP_ADDRESS>:1993
2024-08-22 13:55:47  INFO nostr_dice::subscriber: Starting invoice subscription
2024-08-22 13:55:47 DEBUG hyper::client::pool: reuse idle connection for ("https", bob:10009)
2024-08-22 13:55:47 DEBUG tungstenite::handshake::client: Client handshake done.
2024-08-22 13:55:47 DEBUG nostr_dice::social_updates: No winners in this round, not posting anything
2024-08-22 13:55:47 DEBUG reqwest::connect: starting new connection: http://nostr-rs-relay:8080/
2024-08-22 13:55:47 DEBUG nostr_dice::nonce: New active nonce commitment=0751573ddcaa823a9bf3102217354bf758af38f70825c6e9e01c1a38a3af1d73 commitment_event_id=fa918f889b92c154711cd49fc52af07375bf26237d1de09798c72dc43654b26f
2024-08-22 13:56:17 DEBUG nostr_dice::nonce: Nonce has expired commitment=0751573ddcaa823a9bf3102217354bf758af38f70825c6e9e01c1a38a3af1d73
2024-08-22 13:56:17 DEBUG nostr_dice::nonce: Waiting to reveal expired nonce commitment=0751573ddcaa823a9bf3102217354bf758af38f70825c6e9e01c1a38a3af1d73
2024-08-22 13:56:17 DEBUG nostr_dice::nonce: New active nonce commitment=e9f63aa0dbd6e940f8fdabe5dd93b4d6751fc3fe503b8eb09fa0f41ed43a69f3 commitment_event_id=86ca8bb8e80649cb8bcea4b3d44e71bc19be44ff5c13c6b0304b46f47ca0ada7

My mistake 🙈 the issue was that data/zaps.db was still there from sled-db times and it was a directory. just wipe didn't work because it only deletes files not directories, hence, the file was still there and nostrdice couldn't create a new file.
Thank @Restioson and sorry for the confusion.

Tested it locally successfully including running just test.


Updating category for Advanced Tests

Hey Robert, can you please take a look? I'm updating categories to help ease automation.

+1 LGTM.

Thanks Robert.


Use flake8 settings from setup.cfg

pep8speaks allows setting up flake8 parameters using the config option pycodestyle. For example:
https://github.com/pandas-dev/pandas/blob/master/.pep8speaks.yml#L8
But Python projects/flake8 already defines a way to specify those, which is in the flake8 section of setup.cfg. For example:
https://github.com/pandas-dev/pandas/blob/master/setup.cfg#L14
I think it'd be nice to keep the pycodestyle setting, but in case it's not present, it should look for the options in setup.cfg and use them. This way, projects don't need to maintain two different copies of the flake8 settings, and there are no mismatches in the errors the pep8speaks reports, and what the CI of the projects are linting.

Hello @datapythonista !
Thank you so much for the wonderful suggestion. A new release of the app has been pushed, tested and deployed with your suggestion. You can now remove the pycodestyle section from .pep8speaks.yml and it will read configurations from the [flake8] section of setup.cfg.

thanks @OrkoHunter, I created this issue in pandas, will let you know how it works: https://github.com/pandas-dev/pandas/issues/25605
thank you!

Now, pep8speaks seem to ignore flake8 configurations in setup.cfg.


CC0068 - false positives with WPF callbacks

When you define callback in XAML to private method in codebehind then the method is marked as unused.

@kindermannhubert Can you provide a code sample ?

Sorry, it's little bit more complicated than I thought.
Repro steps:

Create new WPF app 'WpfApplication1'.
Add new class:

    public class MyControl : UserControl
    {
    }

Add new item to project "User Control (WPF)" with name 'MyControlDerived'
Change XAML of new user control (to derive from MyControl class):

<local:MyControl x:Class="WpfApplication1.MyControlDerived"
             xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
             xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
             xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
             xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
             xmlns:local="clr-namespace:WpfApplication1" Loaded="MyControl_Loaded">
    <Grid>
    </Grid>
</local:MyControl>

Change code-behind of new user control:

using System.Windows;

namespace WpfApplication1
{
    public partial class MyControlDerived
    {
        public MyControlDerived()
        {
            InitializeComponent();
        }

        private void MyControl_Loaded(object sender, RoutedEventArgs e)
        {
        }
    }
}

Now the 'MyControl_Loaded' method will be detected as unused.

I can confirm that the problem happens. It does not happen to regular controls as the MyControlDerived.g.i.cs generated file create an event binding. This is not happening in this scenario.
Without a .cs file to investigate there is nothing we can do to find the method usage.
The only ways I see to fix this is either you suppress the diagnostic there:
#pragma warning disable CC0068 // Unused Method
        private void MyControl_Loaded(object sender, RoutedEventArgs e)
#pragma warning restore CC0068 // Unused Method
        {

        }

Or you make the method public.
Any clues on how to find this reference?


PHPStorm debugging guide link missing

hi,
https://laradock.io/documentation/#phpstorm-debugging-guide leads to https://laradock.io/guides/ which is a 404

That link is still broken

https://github.com/laradock/laradock/commit/5c10a024f1d61c99c42d5cb53694f760f00d051e

That link is still broken


{CI} Add execution time

Description
Add execution performance check on CI via azdev perf benchmark, sampling on version command and network vnet subcommands
Testing Guide
https://github.com/Azure/azure-cli-dev-tools/pull/222/checks?check_run_id=825286911
After https://github.com/Azure/azure-cli-dev-tools/pull/220 is megred, the CI should be fine.
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

add to S172


Tests for GetComimitsInfo

In particular, adds test coverage for the bug fixed by #98.

for a read-only test, like this one, why not using the code repository itself ? That's how I did it in repo_test.go. I know it may be fragile but seeing those objects in the repository sounds wrong to me. Maybe the cleanest would be for an initialization script to be run to create the repo ...

@strk's idea is what I want. But it's difficult to do in this repo. It could be in github.com/go-gitea/gitea's integration tests.

@strk: The disadvantage of using the repository itself is that you are limited in the number of scenarios you can test for (the commit history is essentially linear, and relying on branches other than master seems fragile since those branches may be deleted).
By adding a separate repository, we can create more scenarios for tests (e.g. two branches that have significantly diverged)
@lunny: It may be worthwhile to add an integration test in gitea, but I think it is important to also add a unit test in this repo. Otherwise, we could introduce a bug in this library, and will not realize until we update gitea's dependencies.

LGTM


Reccomended version

Hello,  If I am using faker for creating data is the recommended version to still use 2 or 3.  I was running into trouble using 3.1 and was looking for the correct version

Nvm I figured out what I actually needed to do.  Just using 3 for now.


Massive performance degradation and loss of results in SPARQL queries between RDF4J 2.5.2 and 3.7.2

Current Behavior
We are currently running RDF4J version 2.5.2 but need to upgrade to the latest version for security reasons. We tried upgrading to 3.7.2 but we found that the performance of many of our SPARQL queries have massively degraded - actually, they don't even complete execution. These same queries complete in under 20 milliseconds in version 2.5.2.
We can roughly see that the problematic queries have UNIONs in them. For one of these queries that hangs forever, we found a way to rewrite it without the UNION keyword which caused it to work again. This was not a trivial exercise, so it is not feasible for us to replace the hundreds of queries we have in our system.
I isolated and stripped down a query and RDF store data to repro this. I found that the performance of this query scales with the (decreasing) size of the RDF store. A full TTL export of our test RDF store is about 15 MB. The query hangs when run on the full store (I didn’t bother waiting after many minutes). When I cut down the store to 7 MB, the query took about 56 seconds. From 5 to 2 MB, the query went from 46 to 8 seconds. Note that the sizes I refer to here are the ttl export sizes. The memorystore.data file for the attached example RDF store comes to 509 KB.
I also found that the problematic queries run on 3.7.2 don’t return any results at all (they should and do return results on 2.5.2).
The attached zip file contains the following files, which describe and can be used to reproduce this issue:
Screenshots show the query running in the workbench and in our internal tool.
Logs are prefixed by 252 for the test machine with RDF4J 2.5.2 and 372 for version 3.7.2.
Here are some extracts showing the time difference for the two versions:
372:
2021-09-08 12:07:35,424 10008 [repositories/NativeJStore] DEBUG c.g.z.f.c.CompressingHttpServletResponse - Committing response without compression
2021-09-08 12:07:42,949 17533 [repositories/NativeJStore] INFO  o.e.r.h.s.r.TupleQueryResultView - Request for query -2131556384 is finished
252:
2021-09-08 12:07:23,803 28861 [repositories/NativeJStore] DEBUG c.g.z.f.c.CompressingHttpServletResponse - Committing response without compression
2021-09-08 12:07:23,805 28863 [repositories/NativeJStore] INFO  o.e.r.h.s.r.TupleQueryResultView - Request for query 3392903 is finished
I include two sets of logs (example1 and 2), no functional difference, I just ran the test twice.
Expected Behavior
The same query run on RDF4J 2.5.2 and 3.7.2 should complete in similar times and return the same results.
Steps To Reproduce
rdf4j_query_bug3.zip
(EDIT I updated this zip file with new data, new file is called rdf4j_query_bug3.zip).
Repro steps:
1. Create a memory store repository and fill it with data from rdfstore_clean.ttl using Add RDF. See screenshots.
2. In repro_query_clean.rq, there are two queries available. Run the stripped-down one to get similar test results as I've described. I've also included our full actual query that does not complete even with this cut-down RDF store (> few minutes). Note the stripped-down query may not make sense, I obtained it by just removing stuff out of the full query to get it as simple as possible while still demonstrating this problem.
3. Observe that when run on a machine with RDF4J 2.5.2, the query completes in under 20 ms; whereas when run on a machine with RDF4J 3.7.2, the query takes about 8 to 9 seconds to complete.
4. Also observe that the query returns no results when run on 3.7.2, while it does on 2.5.2.
Version
3.7.2
Are you interested in contributing a solution yourself?
Perhaps?
Anything else?
Recall that this is a heavily stripped-down example - in our production environments, this query never finishes.
If relevant, version of tomcat used on our systems:
server.info=Apache Tomcat/9.0.43
server.number=<IP_ADDRESS>
server.built=Jan 28 2021 20:25:45 UTC
Thanks for your help with this.

Thanks for reporting the issue and the detailed test material.
Would it perhaps be possible to do a quick test with e.g. version 3.4.0, to narrow down a bit the version(s) where the degradation occurs ?

Hi Bart, thanks for your response - we reproduced this issue in 3.3.1

It appears the problem was first introduced in release 3.0.3. The full query executes for me in ~130ms (with 8 results) when using version 3.0.2, but does not complete in reasonable time when using 3.0.3. A likely suspect is that this is a side effect of the fix we did for GH-1642. I'll take a look at the query plans.

We spent sometime on last Friday/Saturday to try 10+ times using different versions/commits and did identify the exact code change which brought us this issue -- yes, the same one you mentioned before; we thought it'd be fine to update you here later on Monday. But then we found out that you're very much proactive and  had already posted not only the identified code change, but also the commit of fix! Wow, that's amazing!
Earlier today, we did a quick test using the v3.7.2 code plus your commit, and confirmed that the issue was gone. We still need to do a full regression testing to verify if this new version works fine with our application. So we'll update you guys a little bit later (like tomorrow).
Appreciated your help, and will update you soon!

Thanks for confirming the fix works! Please note it may still need a small refinement, but I'm hoping to find time to get that sorted and do a patch release this weekend.

Hi Jeen,
Good to hear that a patch release is coming this weekend!
We did run a regression today with our real testing data and we can confirm that the fix did work. So we'll have another round of testing with the coming release. Look forward to that!

Robin

Jeen, thank you very much for your work and the super-fast release.
It looks good in our quick tests and I'm sure it will be fine for the full test.
Thanks again, we really appreciate it. :)


Methylartist noes not recognize cram index

Hi,
Methylartist segmeth works just fine with .bam files but generates an error when I am trying to use .cram input
bam not indexed:
Is it an expected behavior? Is there a way to fix it?
Thank you

hmm the cram index (.crai) should be picked up via pysam, if you try the following in a python interactive session or as a script what happens (sorry I don't have a .cram file handy but will test as soon as I can):
import pysam
bam = pysam.AlignmentFile("/path/to/your.cram")
bam.check_index()

@adamewing I apologize about the late response. You are right, version update has solved the issue. Thank you!


[Bug]: Select widget options does not list all the options, only lists 1 and 2.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I bind a select widget options to say a text widget, only label and value are displayed, the third option added is not displayed to the user.
Steps To Reproduce

Drag and drop a select widget
In the options for select widget - add the following
[
{
"label": "Blue",
"value": "BLUE",
"key": "first"
},
{
"label": "Green",
"value": "GREEN",
"key": "second"
},
{
"label": "Red",
"value": "RED",
"key": "third"
}
]
Bind the select widget to a text widget using - Select1.options
Note that only label and value are displayed from the options .. key is not displayed.

https://github.com/appsmithorg/appsmith/assets/101863839/a19ae017-ce8a-4a33-b44c-0157fe489a4d
This was working in earlier versions.
This change is breaking the internal Regression App functionality
Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Cloud

As per slack thread discussion, reducing the priority of the issue.

Tagging it has high priority until we understand that users are not facing this as a blocker. The pin can be removed after the sprint.


Increase amount of medal brackets

Now we have greed, grey, red.
Add more brackets based on percentiles
1 purple
2 purple
3 purple
pink
10 %
orange
25 %
yellow
50 %
green
75 %
blue
90 %
white
Colours from: https://wiki.guildwars2.com/wiki/Item


Refactored package to use PHPUnit 6 and namespaces (closes #20)

Reopened #22, now that I'm part of the PHP-VCR org.

@adri I'm part of the PHP-VCR org, but I don't have write access to this repo. If it's ok for you, you can give me write access and I'll merge this, and then release v3.0 with these changes 🙂

🤔 can you lease try again?


How to exclude certain folder from bundling

Please provide us with the following information:

OS?

Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)
Windows 10

Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:
angular-cli: 1.0.0-beta.21
node: 6.9.0
os: win32 x64

Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.

The log given by the failure.

Normally this include a stack trace and some more information.

Mention any other details that might be useful.
When doing ng build does this take the whatever is in the src folder and bundle it? if so I have put extra folder in the src folder which I want to exclude during the build so it won't bundle up but still copies it to the /dist after the build.
Say my folder is called 'excludethisfolder', I want this to be excluded but still want it to be in the dist folder.

Thanks! We'll be in touch soon.

It does not compile everything, only what's imported from your code. Also, it seems you want an assets folder which we support.

I actually meant to include certain folder, but I found it where to add them now.


BeamGuards don't protect a location when it has query parameters

Example (from the docs), tested with the latest master:
BeamGuard(
    pathBlueprints: ['/books/2'],
    check: (context, location) => false,
    showPage: forbiddenPage,
  ),

Trying to enter /books/2 fails, which is good.
But when I add a query parameter to the uri, e.g. /books/2?test I can see the page which should definitely not be the case, especially not in applications where the user has to be authenticated to enter specific pages.
I know that this could be solved by changing the path blueprint in the guard to /books/2*, but in my opinion /books/2 should also protect /books/2?test etc.
And I think we should definitely add a section to the docs which extensively explains BeamGuards.

@devj3ns good point, it should guard the path, regardless of query.

@devj3ns good point, it should guard the path, regardless of query.

I am working on a PR to fix this 😄

am working on a PR to fix this

Great! Be sure to also add some tests for this.


Page references depreciated functions?

This page talks about functions such as Office365Users.SearchUser yet this page https://docs.microsoft.com/en-us/connectors/office365users/ says these are depreciated and replaced by V2 equivalents
[Enter feedback here]

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

ID: 9434b232-2bf5-14f0-9861-6726dc216b96
Version Independent ID: d073f002-01ac-541b-d365-60affab50cae
Content: Overview of the Office 365 Users connection - Power Apps
Content Source: powerapps-docs/maker/canvas-apps/connections/connection-office365-users.md
Service: powerapps
GitHub Login: @lancedMicrosoft
Microsoft Alias: lanced

Thank you for reaching out. Documentation has been fixed, please allow some time to reflect the changes.


New role for installing keycloak >= 17.0.0 (quarkus)

Fix #21

Please note that the new Role 'keycloak-quarkus' is missing from both  Github and Ansible Galaxy tarballs:
[ https://github.com/ansible-middleware/keycloak/archive/refs/tags/1.0.2.tar.gz ]
[ https://galaxy.ansible.com/download/middleware_automation-keycloak-1.0.2.tar.gz ]

Please note that the new Role 'keycloak-quarkus' is missing from both Github and Ansible Galaxy tarballs

Indeed it has been merged into 1.0.3-devel; will be included in next release 1.0.3


update freewheel bider adapter doc

When updating our bidder adaptor to our new company name, there was some mistake in the documentation,.
I've remove the stickyadtv md file, replacing it with freesheel-ssp doc.
also added prebid_1_0_supported:true in it.
(as suggested by Roffray in this thread: https://github.com/prebid/prebid.github.io/issues/620#issuecomment-371770251 )

confirmed the status of the adapters in PBJS. Looks good.


Implement division for I256

Currently, the 256-bit signed integer type does not implement division (oops!). This issue captures the work for implementing division and division related operations:

Implement Div and Rem traits for division and remainder (modulo) calcuations.
Implement {overflowing,checked,wrapping,saturating}_{div,rem,div_rem} methods for all the different flavours of standard division and remainder calculations.
Implement {overflowing,checked,wrapping,saturating}_{div,rem,div_rem}_euclid for euclidean versions of the above calculations. Note that euclidian division and remainder have slightly different semantics than standard integer division when the divisor is negative.

@bh2smith Take a look if you're interested!

Sounds like a nice juicy task however I will require more clarification on the meaning of {overflowing,checked,wrapping,saturating}_{div,rem,div_rem} and your definition of the difference between euclidean and standard integer division when divisor is negative.


Use etcdutl instead of etcdctl for restoring an etcd cluster

This is a Bug Report

Problem:
This issue refers to the page
Kubernetes Documentation > Tasks > Administer a Cluster > Operating etcd clusters for Kubernetes
https://kubernetes.io/docs/tasks/administer-cluster/configure-upgrade-etcd/
In the section titled "Restoring an etcd cluster", the commands use the etcdctl utility to restore from an existing snapshot.
Using etcdctl for restoring snapshots is deprecated since etcd version 3.5.x
Usage of this command displays a deprecation warning and suggests to use etcdutl utility instead as shown below.
ubuntu@kubemaster:~$ etcdctl version
etcdctl version: 3.5.10
API version: 3.5
ubuntu@kubemaster:~$ ETCDCTL_API=3 etcdctl --endpoints <IP_ADDRESS>:2379 snapshot restore snapshot.db
Deprecated: Use `etcdutl snapshot restore` instead.

I feel this change is necessary as the etcd official documentation has already changed to reflect the use of the etcdutl utility, as can be seen in the etcd documentation referenced in this section - https://etcd.io/docs/v3.6/op-guide/recovery/#restoring-a-cluster.
Proposed Solution:
I have raised a PR to depict the use of the etcdutl utility for restoring from an existing snapshot.
https://github.com/kubernetes/website/pull/44050
Before submitting the PR, I have tested the changes on a local Kubernetes cluster for v1.28.0 and I can confirm everything works as expected.
Page to Update:
https://kubernetes.io/docs/tasks/administer-cluster/configure-upgrade-etcd/

Kubernetes Version: 1.28.x and above

//assign

/assign

/retitle Use etcdutl instead of etcdctl for restoring an etcd cluster
/language en

/sig etcd

/triage accepted

Also see https://github.com/kubernetes/website/issues/46757


[Core][Share] Folder from "Shared with me" doesn't disappear automatically if to rename original one. Note that chat located in shared folder disappears in this case

By user1 create folder with chat inside -> share the folder
By user2 click on the link -> folder with char appears in Shared with me
By user1 renamed the folder -> shared icon disappears (ok)
By user2 refresh the browser and note that the chat disappears (ok), but folder is still in "Shared with me" structure -> bug

Expected result: folder with its elements should disappear from "Shared with me" if the original shared folder was renamed
Note: user2 can delete such folder from Shared with me thru context menu

Related to https://github.com/epam/ai-dial-core/issues/221

Verified on staging successfully


Can't edit a kata due to a rounding error reported by the server

http://www.codewars.com/kata/paginationhelper/edit/ruby

Fixed


Named Indices should not have column positions assigned

CrateDB version
master
CrateDB setup information
No response
Steps to Reproduce
// from one of the integration test: test_alter_table_add_column_keeps_existing_meta_information()

CREATE TABLE tbl (
author TEXT NOT NULL,
INDEX author_ft USING FULLTEXT (author) WITH (analyzer = 'standard')
);

ALTER TABLE tbl ADD COLUMN dummy text NOT NULL;

Expected Result
{"default":{"dynamic":"strict","_meta":{"constraints":{"not_null":["dummy","author"]},"indices":{"author_ft":{}}},"properties":{"author":{"type":"keyword","position":1,"copy_to":["author_ft"]},"author_ft":{"type":"text","analyzer":"standard"},"dummy":{"type":"keyword","position":2}}}}

Actual Result
// index mapping information obtained while debugging, notice duplicate position 2
{"default":{"dynamic":"strict","_meta":{"constraints":{"not_null":["dummy","author"]},"indices":{"author_ft":{}}},"properties":{"author":{"type":"keyword","position":1,"copy_to":["author_ft"]},"author_ft":{"type":"text","position":2,"analyzer":"standard"},"dummy":{"type":"keyword","position":2}}}}

cr> select column_name, ordinal_position from information_schema.columns where column_name like 'author%';
+-------------+------------------+
| column_name | ordinal_position |
+-------------+------------------+
| author      |                1 |
+-------------+------------------+
SELECT 1 row in set (0.028 sec)

ordinal_positions also suggest that named indices should not be assigned position values, otherwise it will be a breaking change.

Why shouldn't they have a position? What's the issue if they do?

Actually, there is no issue if they do.
Also, I was wrong:

ordinal_positions also suggest that named indices should not be assigned position values, otherwise it will be a breaking change.

I will close this now and continue at https://github.com/crate/crate/pull/12675


[5.10] cherry-pick: build: add SwiftIDEUtils to the Swift Syntax modules

Add SwiftIDEUtils to the modules that we distribute with the toolchain. The motivation here is to share the build of swift-syntax across the compilers and the LSP. Using the shared linkage this way shaves ~6MiB off of the LSP binary.

@swift-ci please test

This is needed for compnerd/swift-build 5.10 build to work for building the Windows toolchain.

I'll note that we will want this for the official builds as well (CC: @shahmishal)


More updates 3/5

-Redirection
-Jimmy Feedback

Docs Build status updates of commit 24b6076:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

windows-iotcore/tutorials/dragonboard.md
:white_check_mark:Succeeded

windows-iotcore/tutorials/intel.md
:white_check_mark:Succeeded

windows-iotcore/tutorials/nxp.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Subdivision of translation items

( I have corrected it and will pull request again. )
Translated items have been subdivided to conform to the display of Japanese government applications.
English and French have been adjusted to display the same as before.
Please check it.

Thanks. Yet another good addition to allow the translations to be more fine-grained and accurate.


Handle missing token errors

Start a project with no Mux creds anywhere, i.e. no env.local
Add a video to /videos
Run next-video sync

It'd be good to handle this as a known error in next-video, rather than unknown from mux-node, and maybe directly people back to that part of the next-video docs.

This doesn't look like an up to date error, we added a big try/catch there a little while ago. I'm not sure what we should do here re: known errors, that kind of integration-specific stuff sits a level below the CLI in the handlers.

Maybe related, it'd be nice to provide a bit nicer message when a user doesn't set up their creds correctly. I intentionally broke my creds and got this. It's not bad but could be more helpful, and I expect this to be a common issue.


Bug in update access time and expiry in org.ehcache.internal.store.offheap.AbstractOffHeapStore.OffHeapStoreIterator#next

By calling setAccessTimeAndExpiry(thisEntry.getKey(), thisEntry.getValue(), timeSource.getTimeMillis()); outside of lock scope, memory corruption can occur.
This update from within the iterator needs to be performed differently.

This was introduced by the fix for #559 and missed during review by me.


Can't reproduce the result

I run the training and evaluation code and find the result on ScanRefer and Multi3DRefer are exectly bad, I'm not sure is there any bug in the code

Such as, I'm curious why obj_id are different between training and evalutation in prepare_scanrefer_annos.py, is this a bug?

Did you change any training config such as the GPU number? Since the learning rate is multiplied by GPU number here, we find that the results is currently unproducible when set GPU numer to 8 (we got the similar results as yours with very low grounding performance).
If you did not change any config, maybe you can try a lower max_grad_norm here. We use max_grad_norm=0.01 in recent experiments and it seems to be more stable.
This code snippet has no bug. Actually the "obj_id" in train annotation is not used during training, and the "obj_id" in val annotation is representing the id of the gt object in the gt segmentations, which will be used to retrieve the gt bbox for calculating the IoU.

Did you change any training config such as the GPU number? Since the learning rate is multiplied by GPU number here, we find that the results are currently unproducible when setting GPU numer to 8 (we got the similar results as yours with very low grounding performance).
If you did not change any config, maybe you can try a lower max_grad_norm here. We use max_grad_norm=0.01 in recent experiments and it seems to be more stable.
This code snippet has no bug. Actually the "obj_id" in train annotation is not used during training, and the "obj_id" in val annotation is representing the id of the gt object in the gt segmentations, which will be used to retrieve the gt bbox for calculating the IoU.

That's an interesting phenomenon, it seems like doubling the GPU number from 4 to 8 would have a huge impact on the grounding tasks? just because the 2x learning rate?

The grounding and captioning tasks are all based on object ids. So the low performance indicates that the object id is not well-trained, and we can't directly observe this from the loss curve (the loss curve looks normal when it produces poor grounding results). We've did some ablation studies but still haven't found the true reason. Basically, I think a possible reason is about data scale. Current data (number and diversity) is not enough for feeding the trainable weights and the model may be trained towards a wrong direction that neglects the learning of object ids (the process could be vulnerable to learning rate). So we may plan to add more data in the future, especially those designed for learning object ids.
Nonetheless, It's not that hard to reproduce the reported results if you stick to our training config.

Thanks for your explain, I didn't change the GPU number but do the following change in run.sh because I can't install slurm

In the evaulation stage it hints: WARNING 2024-08-07T14:34:51 | py.warnings: /data/projects/15003900/Chat-3D-v2/utils/distributed.py:18: UserWarning: do_sample is set to False. However, top_p is set to 0.6 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset top_p.
builtin_warn(*args, **kwargs)
Is this the reason?

It looks no problem. Can I see your training log (the train.log file under output directory)?
I'm re-training this code today to see if it can be reproduced in my environment now. Maybe we can compare the training log to find some difference.

Oh, I change the training epoch from 3 to 2, the learning rate seems to drop faster than before, there is the training log:
train.log
But I train the code in 3 epoch with the setting add_scene_token=True, and get the similiar result( very poor grounding peformance), there is the training log:
train.log
seems I can't change any setting in the config, maybe I should try max_grad_norm=0.01 to make it more stable. If it's still so unpredictable, maybe there are some problems unsovled in the code or in the design?

I used the original config and have trained it for 2 epochs (out of 3). The results are reproducible in my environment.
train.log
The add_scene_token should be set to False as default. Because we didn't use it in our experiments and we also didn't mention it in the readme instructions.
I recommond training it with default setting first. If it is still unreproducible in your environment, we can further help you find the problem.

Thanks for your patience, I reproduce the results with the default setting, it's a good work.


Add weather alert sensor to meteo france component

Description:
Add weather alert sensor to the meteo_france component. It will show the status of all weather alert type  monitored in the department of the location set in configuration.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#9239
Example entry for configuration.yaml (if applicable):
meteo_france:
  - city: 'Paris'
    monitored_conditions:
      - weather_alert

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly (example).
[x] New dependencies have been added to requirements in the manifest (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Hi @oncleben31,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Is anybody available to review my PR?
Is something missing?

Hey @fabaff and @amelchio, I've seen you have reviewed a PR on a similar subject: "weather alerts". Are you available to review my PR or notify the good reviewer?

Thanks


Release on crates.io

This project can't currently be released on crates.io, as two of its dependencies aren't:

https://github.com/japaric/nb
https://github.com/japaric/embedded-hal

Once those two are available, we should release a crates.io version ourselves.
Things to remember when releasing:

Add additional badges to README
Update links to documentation in README and Cargo.toml

I've opened issues about this upstream:

nb
embedded-hal

This is blocked by #1. Cargo won't allow me to release a crate with a [patch] section, nor can I set the dependency to my cortex-m fork, unless I release that on crates.io, too (which I don't want to do).
I wanted to get the first release out before taking on the task of upgrading to the new versions of all dependencies, but if that's what needs to happen, so be it.


malformed URL?

Hi,
I receive the following error for some images ... can you help me understand how to fix it?

2014-07-14 15:57:04 thumbor:WARNING Malformed URL: /jyq3vYWwfonZ53WCmP_wOw0Sn5Y=/103x/http://1.bp.blogspot.com/-XEpVqBPZxKo/Ue4mLe8_MoI/AAAAAAAACpc/LCiGjWdHWrM/s1600/fantasy+environment+07.jpg

May be there is a problem with url containing the "minus" sign?
thanks

I think I've fixed this in https://github.com/webfactory/phumbor/pull/1, you could try webfactory/phumbor: ^1.2.3 on Composer.


Change pinned post detection, fixes #2142

Fixes #2142.
The property that indicated whether a post is pinned is not available anymore. As a workaround, the first five posts are treated differently (as pinned posts were): if they were already downloaded, they are skipped, but they do not cause the iteration to stop.

Is it just a proof of concept?* No

Is the documentation updated (if appropriate)?* Yes

Do you consider it ready to be merged or is it a draft?* Ready

Can we help you at some point?* I'm open to suggestion for improvement

Tried this branch, it tried to retrieve the already downloaded profiles' metadata (i.e. all posts) even though it was given the "-F"/"--fast-update" command-line argument and all the posts were logged as "exists" and "unchanged".

You're right, there was a wrong condition that was considering more than the desired first five posts. Hopefully it's fixed now.

Nice change and a good idea to simply disregard the fast-update and takewhile stop criterium for the first five posts. Thanks for also updating the docstring of is_pinned.
I only have two minor suggestions:
Only up to three posts can be pinned, so it would suffice to limit the check to the first three posts.

You can pin up to 3 posts you share on Instagram to the top of your profile.

(https://help.instagram.com/318456537074409/?cms_platform=iphone-app)
Also, posts_download_loop is used for every target that yields post, not only for Profiles, whereas only for Profiles Posts can be pinned. I would prefer to have that criterium only for Profiles and for other targets it should still stop as soon as the first already-downloaded post is encountered.

Only up to three posts can be pinned, so it would suffice to limit the check to the first three posts.

I used five posts instead of three as a safeguard in case IG changed the limit. But it would only work if they changed the limit to four or five, if they decided one can pin six posts, the code would have to be changed anyway. So I guess it's fine to use three.

Also, posts_download_loop is used for every target that yields post, not only for Profiles, whereas only for Profiles Posts can be pinned. I would prefer to have that criterium only for Profiles and for other targets it should still stop as soon as the first already-downloaded post is encountered.

How would one detect that it's Posts being downloaded? The closest I found was the owner_profile argument, but it is passed for posts and also IGTV posts (is this still a thing?).
However the method, one problem is that the conditionals would become even more complex with a check for the iteration type.

How would one detect that it's Posts being downloaded? The closest I found was the owner_profile argument, but it is passed for posts and also IGTV posts (is this still a thing?).

It could be passed as a parameter flag to posts_download_loop.

However the method, one problem is that the conditionals would become even more complex with a check for the iteration type.

Indeed, it would become a little less readable. However, depending on the use case (e.g. downloading :saved) it could then likely save requests when updating an almost-up-to-date collection. I think that's a good trade off.

That function already has a lot of parameters, but I guess that's one solution.
Instead of a flag, I thought of passing a number representing the number of initial posts to not consider for stopping. When downloading posts, I'd pass 3 to this new parameters, in other cases 0 is passed (or rather, nothing is passed, and it receives the value 0 by default). What do you think?

Instead of a flag, I thought of passing a number representing the number of initial posts to not consider for stopping. When downloading posts, I'd pass 3 to this new parameters, in other cases 0 is passed (or rather, nothing is passed, and it receives the value 0 by default). What do you think?

I like that idea. I doubt that they will allow different numbers of pins for different targets, but it would move the number of maximum pins into the profile download function, and as such semantically closer to where it is relevant.

I've added the parameter, now the extra check only applies to profile posts.

Thanks!
I noticed that you are missing in our AUTHORS.md file where we list the most important contributors. We would be pleased if you'd add yourself in that file.