Checking the prediction_function signature in case of non-pandas df input

We currently have:
    def upload_model_and_df(
            self,
            prediction_function: Callable[[pd.DataFrame], Iterable[Union[str, float, int]]],
            etc...

This hint should do the following (see example below):
def greeting(name: str) -> str:
    return 'Hello ' + name

greeting(1.5)
>>>
Traceback (most recent call last):
File "<string>", line 7, in <module>
  File "<string>", line 5, in greeting
TypeError: can only concatenate str (not "float") to str

but... giving a prediction_function that has a list (instead of pandas) doesn't produce an error:
def prediction_function(input: List[int]):
    return input

def test_func(prediction_function: Callable[[pd.DataFrame], Iterable[Union[str, float, int]]]):
    print(prediction_function)

test_func(prediction_function)

This was observed by Etam, that had a problem with the upload due to the fact that the input of their prediction_function wasn't a pandas dataframe

it seems to me a rare case where the user is aware that he should give a pandas.dataframe to give to Giskard, but keeps their prediction_function input as xgb.matrix
It's difficult to make a check other than the one we did here:
    def _validate_model_execution(self, prediction_function, df: pd.DataFrame, model_type,
                                  classification_labels=None, target=None) -> None:
        if target is not None and target in df.columns:
            df = df.drop(target, axis=1)
        try:
            prediction_function(df.head(1))
        except Exception:
            raise ValueError("Invalid prediction_function input.\n"
                             "Please make sure that prediction_function(df.head(1)) does not return an error "
                             "message before uploading in Giskard")

If they got here, it means that they passed to upload_model_and_df a pandas.dataframe


event: Reduce potential for lock contention while executing dispatcher post callbacks.

Risk Level: low, minor code refactor
Testing: n/a, minimal functional changes expected.
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]

/wait


Prevent expo-av from crashing in bare apps

Temporary workaround for #3864, since bare apps have no experienceId which causes a crash.
I've verified that this stops the app from crashing in iOS. 👍

Oops... wrong base fork...


Getting Started documentation does not work

I have tried repeatedly to follow the documentation for setting up a local dev environment and building the "hello world" web part. Everything seems to work as expected up until I run gulp serve and the workbench doesn't load. First it appeared that @microsoft/sp-webpart-workbench was not installed.
I tried the same exercise in WSL 2 - Ubuntu, and the project was able to start as expected.

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

ID: 6c25a90e-829c-b436-d8ab-f9c010588ee1
Version Independent ID: 768fa9a1-2375-fb5a-be58-cb66320cc789
Content: Build your first SharePoint client-side web part (Hello World part 1)
Content Source: docs/spfx/web-parts/get-started/build-a-hello-world-web-part.md
Product: sharepoint
Technology: sharepoint-framework
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

Can you provide us with the version of SPFx you are using, as well as any helpful information such as any errors that are popping up for you.

It looks like this issue was caused by a transient error while the yo @microsoft/sharepoint command was being run. I am not able to recreate, and running the yeoman generator on a different machine behaved as expected. I should note that I did not see any obvious errors the first time running the generator which did not make me consider that something had failed to download/install during that script run.


Fixing cert name in calico when we delegate to etcd[0]

This fixes a bug with upgrades.
TASK [network_plugin/calico : Calico | wait for etcd] ********************************************************************************************************
task path: /kubespray/roles/network_plugin/calico/tasks/main.yml:82
The full traceback is:
Traceback (most recent call last):
  File "/tmp/ansible_vRbB7m/ansible_modlib.zip/ansible/module_utils/urls.py", line 1044, in fetch_url
    client_key=client_key, cookies=cookies)
  File "/tmp/ansible_vRbB7m/ansible_modlib.zip/ansible/module_utils/urls.py", line 951, in open_url
    r = urllib_request.urlopen(*urlopen_args)
  File "/usr/lib/python2.7/urllib2.py", line 154, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib/python2.7/urllib2.py", line 429, in open
    response = self._open(req, data)
  File "/usr/lib/python2.7/urllib2.py", line 447, in _open
    '_open', req)
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/tmp/ansible_vRbB7m/ansible_modlib.zip/ansible/module_utils/urls.py", line 417, in https_open
    return self.do_open(self._build_https_connection, req)
  File "/usr/lib/python2.7/urllib2.py", line 1167, in do_open
    h = http_class(host, timeout=req.timeout, **http_conn_args)
  File "/tmp/ansible_vRbB7m/ansible_modlib.zip/ansible/module_utils/urls.py", line 428, in _build_https_connection
    return httplib.HTTPSConnection(host, **kwargs)
  File "/usr/lib/python2.7/httplib.py", line 1264, in __init__
    context.load_cert_chain(cert_file, key_file)
IOError: [Errno 2] No such file or directory

fatal: [kube02.stargazer.local -> <IP_ADDRESS>]: FAILED! => {
    "attempts": 10,
    "changed": false,
    "content": "",
    "failed": true,
    "invocation": {
        "module_args": {
            "attributes": null,
            "backup": null,
            "body": null,
            "body_format": "raw",
            "client_cert": "/etc/ssl/etcd/ssl/node-kube02.local.pem",
            "client_key": "/etc/ssl/etcd/ssl/node-kube02.local-key.pem",
            "content": null,
            "creates": null,
            "delimiter": null,
            "dest": null,
            "directory_mode": null,
            "follow": false,
            "follow_redirects": "safe",
            "force": false,
            "force_basic_auth": false,
            "group": null,
            "headers": {},
            "http_agent": "ansible-httpget",
            "method": "GET",
            "mode": null,
            "owner": null,
            "regexp": null,
            "remote_src": null,
            "removes": null,
            "return_content": false,
            "selevel": null,
            "serole": null,
            "setype": null,
            "seuser": null,
            "src": null,
            "status_code": [
                200
            ],
            "timeout": 30,
            "unsafe_writes": null,
            "url": "https://localhost:2379/health",
            "url_password": null,
            "url_username": null,
            "use_proxy": true,
            "validate_certs": false
        }
    },
    "msg": "Status code was not [200]: An unknown error occurred: [Errno 2] No such file or directory",
    "redirected": false,
    "status": -1,
    "url": "https://localhost:2379/health"
}

ci check this

@bradbeam I came across this PR when I encountered a problem deploying a new cluster. Lines 87-88 mean that the check expects the file:
"client_cert": "/etc/ssl/etcd/ssl/node-project-env-cloud-etcd-001.pem",
"client_key": "/etc/ssl/etcd/ssl/node-project-env-cloud-etcd-001-key.pem",
but that directory has the certificates for the etcd hosts prefixed with member- or admin-:
/ # ls -lah /etc/ssl/etcd/ssl/*etcd-001*
-rw-r--r--    1 root     root        1.6K Feb 23 19:40 /etc/ssl/etcd/ssl/admin-project-env-cloud-etcd-001-key.pem
-rw-r--r--    1 root     root        1.5K Feb 23 19:40 /etc/ssl/etcd/ssl/admin-project-env-cloud-etcd-001.pem
-rw-r--r--    1 root     root        1.6K Feb 23 19:40 /etc/ssl/etcd/ssl/member-project-env-cloud-etcd-001-key.pem
-rw-r--r--    1 root     root        1.5K Feb 23 19:40 /etc/ssl/etcd/ssl/member-project-env-cloud-etcd-001.pem

When I changed L87-88 locally to:
-    client_cert: "{{ etcd_cert_dir }}/node-{{ groups['etcd'][0] }}.pem"
-    client_key: "{{ etcd_cert_dir }}/node-{{ groups['etcd'][0] }}-key.pem"
+    client_cert: "{{ etcd_cert_dir }}/member-{{ groups['etcd'][0] }}.pem"
+    client_key: "{{ etcd_cert_dir }}/member-{{ groups['etcd'][0] }}-key.pem"

I'm able to resume deployment successfully.

@bradbeam
@mattymo
@clolin
I'm building customized version based on kubespray upstream, just testing current master and this PR breaks creating new cluster.
There should definitely be
-    client_cert: "{{ etcd_cert_dir }}/node-{{ groups['etcd'][0] }}.pem"
-    client_key: "{{ etcd_cert_dir }}/node-{{ groups['etcd'][0] }}-key.pem"
+    client_cert: "{{ etcd_cert_dir }}/member-{{ groups['etcd'][0] }}.pem"
+    client_key: "{{ etcd_cert_dir }}/member-{{ groups['etcd'][0] }}-key.pem"

, because etcd members, does not have certificates with name node-{{ groups['etcd'][0] }}

https://github.com/kubernetes-incubator/kubespray/pull/2386 I'm fixing this


Wrong placement of Room Gizmo

Version of Sollumz used : Last version from GH
Version of Blender : 3.4.1
Description :
When I import v_int_4.ytyp file for editing purpose, room Gizmo is centered through origin and not correctly placed.
How to Reproduce :

export v_int_4.ytyp
import it
profit

Blender view :

CW view ( not the exported one ) :

No issue on CW after exporting the file without editing something.
CW view ( after export the file ) :

could you try exporting the ytyp XML to make sure its just the gizmo and not the import / export code?

This issue has already been reported here https://github.com/Skylumz/Sollumz/issues/466


Xamarin.auth problem in .NET MAUI

Xamarin.Auth Issue
IF BUG, INCLUDE THIS PART:
Version

[ ] nuget version = 1.7.0
[ ] component version =
[ ] sample

Steps to reproduce

Install Xamarin.Auth
Run the project

Platform:

[] .NET version = 7.0
[] mono version =

Expected behaviour
Tell us what should happen
I log in with an online service and get the token
Actual behaviour
After installing the xamarin.auth nuget package, i tried to compile the project but it says:
Error MSB4062: non è stato possibile caricare l'attività "Xamarin.Android.Support.BuildTasks.VerifyVersionsTask" dall'assembly /Users/alex/.nuget/packages/xamarin.android.support.annotations/<IP_ADDRESS>/build/monoandroid90/Xamarin.Android.Support.BuildTasks.dll. Could not load file or assembly 'Microsoft.Build.Utilities.v4.0, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified.
Verificare che la dichiarazione  sia corretta, che l'assembly e tutte le relative dipendenze siano disponibili e che l'attività contenga una classe pubblica che implementa Microsoft.Build.Framework.ITask. (MSB4062)
Tell us what happens instead
Can you also include a screen shot?

same

This seems to be only an issue with the 1.7.0 version of the package.
Steps to Replicate (Using Visual Studio 2022)

Create Android Application
Add NuGet Package Xamarin.Essentials version 1.8.1
Add NuGet Package Xamarin.AndroidX.Arch.Core.Runtime version <IP_ADDRESS>
Add NuGet Package Xamarin.Auth version 1.7.0.

After the auto build completes, the error MSB4044	The "VerifyVersionsTask" task was not given a value for the required parameter "TargetApiLevel". is returned.
Down-grading Xamarin.Auth to version <IP_ADDRESS> resolves the error.


Long term statistics for pressure multiplied by 100

Checklist

[X] I updated the card to the latest version available
[X] I cleared the cache of my browser
[X] I verified that I'm really running the lastest version in my browser console
[X] I checked if there is another issue opened with the same problem

Describe the bug
Long term statistics for pressure multiplied by 100.
Version of the card
Version: v2.0.1
To Reproduce
I have a sensor for atmospheric pressure: sensor.netatmo_casetta_studio_pressure with values in mbar (usually between 980 and 1045)
state_class: measurement
attribution: Data provided by Netatmo
unit_of_measurement: mbar
device_class: pressure
friendly_name: Netatmo Studio Pressure

Screenshots
The graphs are correct:
type: custom:apexcharts-card
graph_span: 1d
header:
  show: true
  title: Pressione
  show_states: true
  colorize_states: true
all_series_config:
  stroke_width: 2
yaxis:
  - decimals: 0
    min: ~980
    max: ~1045
entities:
  - entity: sensor.netatmo_casetta_studio_pressure
    name: Pressione
    group_by:
      duration: 1h
      func: avg

But the long term statistics show values multiplied by 100:
type: custom:apexcharts-card
graph_span: 1y
header:
  show: true
  title: Pressione
  show_states: true
  colorize_states: true
all_series_config:
  stroke_width: 2
  statistics:
    type: mean
    period: day
yaxis:
  - decimals: 0
    min: ~980
    max: ~1045
entities:
  - entity: sensor.netatmo_casetta_studio_pressure
    name: Pressione
    group_by:
      duration: 1w
      func: avg

Expected behavior
The correct value for the pressure (graph and axis)
Desktop (please complete the following information):

Browser: any (e.g. Safari, Chrome, Firefox)

Additional context
All the other long term statistics with the same settings are OK.

Problem still present

Issue still present

Issue still present

Isse still present

Issue seems solved (cannot reproduce anymore)


After eject theme to customize my theme App run very slow.

Hi! After eject theme to customize my theme App run very slow(debug - release). And turn on debug mode app working fine.
"react": "16.0.0", "react-native": "0.51.0", "native-base": "^2.3.5",
Screenshot of emulator/device

App load all of theme and redux.
## Bug present both on android & ios.

#1012


Unable to login nebula graph web ui(nebula studio)

Hi,
I successfully installed the nebula graph studio, and I am able to open the web ui at http://<IP_ADDRESS>:7001,
the login form appears, and I input the following information:
Host: <IP_ADDRESS>
Username: root
Password: nebula
But I login failed with ErrParam message appears on the ui,
I have no idea what happens, and would ask how to fix this problem ,thanks!

The response message is {code: "40004001", message:"ErrParam"}

the host part should be filled as host:port like "<IP_ADDRESS>:9669" 😊

the host part should be filled as host:port like "<IP_ADDRESS>:9669" 😊

Thanks @wey-gu for the helpful answer,it works for me now.
I would suggest that the Host input box choose Host:Port instead of Host.

@bithw1 thanks for your suggestion. I will update placeholder in next version


Add error msgs for AppSet error states

Issue: https://github.com/stolostron/backlog/issues/19963

Add error for not finding placement referenced in AppSet
Display error from Argo app resource for scenarios wrong git-repo path, revision, or path
Display Argo app sync status

/test lint-check-unittests-sonarcloud


need to remove/replace some more characters that are invalid for C# method names when using operationId and path segments

I have a swagger 2.0 file that uses a query parameter in the path and has an operationId that contains a $
This is hitting VMware's vCenter
    "/rest/vcenter/vcha/cluster/mode?vmw-task=true": {
        "put": {
            [... extra deleted ...]
            "operationId": "set$task"
        }
    },

when using SingleClientFromPathSegments, the method name generated is "RestVcenterVchaClusterMode?vmwTask=trueAsync" which is invalid (? and = need to be removed or replaced AFAICT?)
when using SingleClientFromOperationId, the method name generated is "Set$taskAsync" which is invalid ($ needs to be removed or replaced AFAICT?)
Hopefully filtering out characters that are invalid C# would do the trick?

As it clearly cannot work we should fix that directly in the NSwag code.
Can you create a PR?

@RicoSuter a first stab is up at https://github.com/RicoSuter/NSwag/pull/3405 but I'm fairly sure the cleanup should happen in a different place and needs more tests, but wanted to at least get something up as a starting point.


Mention link on profile screen

Describe the bug

There has been a minor bug in the profile screen for some time.
For example, if you press the Profile menu from the web and go to the Profile screen, the Mention link in the description set in the profile is functional as shown in the following image.

But if you transition from a link that was mentions in someone's post, the link is not set in the mentions as shown below.

To Reproduce
Steps to reproduce the behavior:
Web

Set up a Mention link in the Profile description
Transition to your profile screen from a post where someone mentions you

Mobile

Set up a Mention link in the Profile description
Transition to your profile screen from a post where someone mentions you
Or swipe to update on your profile screen

Expected behavior
In any case, the Mention link in the profile description should be valid.
Details

App version:

1.47.0

Additional context

I haven't checked the code, but it may be that the process of setting the link to the mentions is not working depending on the route taken to the profile screen.

seems to be a dupe of https://github.com/bluesky-social/social-app/issues/1223#issuecomment-1686592904

@intrnl
I read the Issue you shared, and I'm not sure if it's the same cause. But I imagine it's simply that the process of setting up the Mention link is missing only for certain routes.

Yeah this is a dupe issue. You're right that something seems to cause it depending on the route taken to show the profile; I haven't been able to isolate the cause yet.


Isssue generating declaration files with object containing keywords as property

TS playground does not have 4.8.2 yet, but this issue can happends in 4.8 beta:
https://www.typescriptlang.org/play?moduleResolution=99&target=99&jsx=0&module=199&ts=4.8.0-beta#code/C4TwDgpgBAMg9gYwIYBsIBEnCVAvFAJQgTgCcATAHgGdhSBLAOwHMAaKRiANwlID4AUKEhQAwnEY9SweMjTjGAM3rNKAFSgQAHsAiNy1WIlQYsOfLJOZsfPIWJkqAqFFoMWrZ1DUC+AbiFwaEt5CWVVDW1dfUMQ02w7OOskW3wiEgoaOiY2KAAFJGl6VHU+fwEBbTAyYCgmXVJFJARoAHEIGWM0agB5MGB6CWp1TR09AyM5eJSoAG8vFyQwMAAuKABXRgBrRjgAd0YAlxdyCCb1lGA1hSlOqYVw0qPjkiUVAH41uIeVUqgAHw2+jOTAg5GeLkYSAAthBPq5siwAgBfCpVGpQV60KDMDpxQz4EZRcaxLrTPgACnmxygSzAnhpUNhDOOp3OlzWbKQFzuJmoPzYC0xYRUa3W1F4+IFdlmyM8yLW7V53T6AyGpQAlNcJLdviKIqlbNTjqQOutSIwoFyeVL9SiAkA
The DTS result is wrong because default is a keyword:
export declare const getLocales: <T extends LocaleData>({ app, name, default, config, }: GetLocalesOptions<T>) => ConvertLocaleConfig<T>;

Meanwhile older ts versions work well:
https://www.typescriptlang.org/play?moduleResolution=99&target=99&jsx=0&module=199&ts=4.7.4#code/C4TwDgpgBAMg9gYwIYBsIBEnCVAvFAJQgTgCcATAHgGdhSBLAOwHMAaKRiANwlID4AUKEhQAwnEY9SweMjTjGAM3rNKAFSgQAHsAiNy1WIlQYsOfLJOZsfPIWJkqAqFFoMWrZ1DUC+AbiFwaEt5CWVVDW1dfUMQ02w7OOskW3wiEgoaOiY2KAAFJGl6VHU+fwEBbTAyYCgmXVJFJARoAHEIGWM0agB5MGB6CWp1TR09AyM5eJSoAG8vFyQwMAAuKABXRgBrRjgAd0YAlxdyCCb1lGA1hSlOqYVw0qPjkiUVAH41uIeVUqgAHw2+jOTAg5GeLkYSAAthBPq5siwAgBfCpVGpQV60KDMDpxQz4EZRcaxLrTPgACnmxygSzAnhpUNhDOOp3OlzWbKQFzuJmoPzYC0xYRUa3W1F4+IFdlmyM8yLW7V53T6AyGpQAlNcJLdviKIqlbNTjqQOutSIwoFyeVL9SiAkA
export declare const getLocales: <T extends LocaleData>({ app, name, default: defaultLocalesConfig, config: userLocalesConfig, }: GetLocalesOptions<T>) => ConvertLocaleConfig<T>;

I think this is a critial bug and should be fixed in 4.8.3 asap

Also I was curious:

why could this happends? Is related test cases missing?  Maybe the team should check test cases and add some releated test into it.
When will you update your website? Hours passes after releasing 4.8 to stable, maybe the team can update ts playground faster, and that DOES help developers.

@MartinJohns So why thumbs down.
So you think this is a completed snippet? Or it's normal for a project with 83.7k stars and 676 contributors should keep the website outdated days after new version? I do open source too, 1 is probably ok if no one everr comes out the idea to add related test. But I don't think it's ok for 2.

I have the tendency to give the down thumb to issues ignoring the issue templates. Weird flex, I know.

I did a bisect, and #41044 did change this, yes.

I agree this is a 4.8 patch candidate. cc @andrewbranch


do we need setup.py ? replace setup.py with project.toml for installing dependancies and single point of version control

#35


"termination_time" parameter for CI jobs

It's possible to configure a server or batch of servers to be automatically deleted after provisioning at a specific time by setting the "termination_time" parameter. There is a brief mention of this in the spot market page currently at https://support.packet.com/kb/articles/spot-market
The observation is that this would be a tremendously useful parameter to add for folks doing batch deployments for short-lived CI runs, and we should document this with an eye towards those applications.

@Naji-F-Pride @zalkar-z
I'd look at https://github.com/packethost/docs/blob/master/products/servers/deployment-options/spot-market.md first and would see if there could be another paragraph in there of description. Let me draft a PR.

PR in #60 please look?

Merged. Closing this issue. Thanks @vielmetti @Naji-F-Pride


Use fully qualified name for ::std::result::Result

The following code fails to compile
E0244: wrong number of type arguments: expected 1, found 2
#[macro_use]
extern crate diesel;

use diesel::types::*;

type Result<T> = ::std::result::Result<T, ()>;

#[derive(Debug, QueryableByName)]
pub struct Row {
  #[sql_type = "Text"]
  pub column: String,
}

Related issue:
https://github.com/diesel-rs/diesel/issues/1372

Can you add a change log entry for this?

Sorry I have to create a new one: https://github.com/diesel-rs/diesel/pull/1426 because I can't figure out how to edit this pull request.
It says from unknown repository.


Strange artifact when rendering shapes

Using your wonderful renderer inside openFrameworks, I get a strange artifact at the edge of shapes.

This artifact is only visible when the shapes are moving, and is visible on the edges that are in the direction of the motion (in the screenshot the velocity of the shapes is (-1, -1)). If I pause the movement, the artifact disappears.
This is another screenshot with velocity set to (5, 5)

I'm able to replicate this on both Mac and Windows using GL2 and GL3.
As I said, I'm using it inside openFrameworks, which I'm aware may be the cause of this artifact, but thought you might have some insights on whats going on.
Thanks!

Look strange indeed. Do you clear stencil buffer each frame?

Yes! I mean no, I wasn't clearing the stencil buffer. Now that I do, the artifact is gone!
Thanks you so much!


HTML-Elements are interpreted instead of printing them

Hi,
I printed a work item that contains e.g. <head>, <header> and <footer> (see first attached screen) and the printout did not show them at all (see second attached screen).

I assume that elements in angle brackets are not escaped and interpreted in the browser before printing.

Thanks for the report. I'll get this into the next version.


Collections

OpenHexa collections are a way to group different type of content around a single theme or purpose :

DHIS2 data elements or indicators
S3 files or folders
Postgres tables
Pipelines
Visualizations

A good example collection would be a "Malaria data repository" collection for a specific country (let's say Senegal), which would include all the DHIS2 data elements, indicators and Postgres tables used by a team of data scientists / analysts working on Malaria in Senegal.

We are leaving this feature out for the moment, in favour of workspaces


[2.1] Remove AuthZ caching from AuthorizeFilter

2.1 Port of https://github.com/aspnet/AspNetCore/issues/9779
Customer Impact: When mixing multiple authentication schemes and authorize attributes (on controller + methods), we end up caching incorrectly, so the wrong authorization policy is applied on subsequent requests. This usually results in valid requests being rejected, I don't think there's any way this would let people who shouldn't be allowed access in.
Fix description: is to remove all the caching of authorization policies in MVC, so authorization policies are computed every time
Risk: Should be low (just removing caching logic, so just slower)

@rynowak @mkArtakMSFT can we get a review from someone on MVC (since this changes a bunch of AuthorizeFilter stuff). It's mostly a port from 3.0.

Hold off on the review for a bit, looks like there were some 2.2 to 2.1 differences that I need to resolve, ah backporting fun

OK. I'm ready to look at this when you are :)

Do not merge until / unless this goes back to shiproom and is approved.

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=213310

I'm seeing some failures on Linux for 5 MVC Modelbinding tests (this should be totally unrelated to my changes except for maybe adding a reference to Mvc.Core).  Rerun didn't work either

This comment was made automatically. If there is a problem contact<EMAIL_ADDRESS>I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/2626

Ok looks like the linux model binding failures are unrelated

Closing for now


[FEATURE] Render Y axis values inside the chart

When you have multiple charts one above the other and values on their Y axes are of different length, then the Y axes don't line up.
e.g.
the top chart may have Y axis values like: 1, 5, 10, ....
the bottom chart may have: 1 req/sec, 5 req/sec, 10 req/sec
the bottom chart's Y axis values are much "wider" than values in top image, so the bottom chart will have its Y axis pushed further right, which means charts won't align.
One cool way to solve this problem would be to allow the values on the Y axes to be printed "inside" or "on top of" the chart, like in this image: https://docs.newrelic.com/sites/default/files/styles/full_size/public/thumbnails/image/screen-app-overview-031816.png

@otisg there is a ticks.mirror option on the axis that moves the values in a way that might be beneficial to you.

Great, thanks @etimberg . Is there anything in ChartJS that would also help me fade the colours behind Y values once they are moved on top of the chart, like in https://docs.newrelic.com/sites/default/files/styles/full_size/public/thumbnails/image/screen-app-overview-031816.png ?

@otisg I don't think there is anything that can do that short of changing the draw function on the axis class.

Thanks @etimberg! Got this:

Any way to make those Y values either darker or push their layer to the top, so they are not "faded"?

Just to clarify: The white rectangle was drawn on the chart with the simple plugin in afterDatasetsDraw  method.

@prog8 that would mean that it drew after the scale. The draw order is scales then datasets. You could try using beforeDraw but it would be drawn before the gridlines

I tried beforeDraw earlier but it also didn't work


correct login status codes and enhance ldap logging

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1663/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1664/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1722/
Test PASSed.


update swagger base to not require paths when defined

@qjhart we don't really need this change for AE, so up to you if you want to create a new tag release.
This change is needed for DAMS though, but since FIN is built differently in that project, this change just needs to be in sandbox pretty much.


Intermittent ActiveRecord::StatementInvalid: PG::CharacterNotInRepertoire Error

I'm getting the following error when I try to create a model with a UUID primary key in rails 4.0.2:
ActiveRecord::StatementInvalid: PG::CharacterNotInRepertoire: ERROR:  invalid byte sequence for encoding "UTF8": 0xd6 0x40

running reload! from the console seems to fix the problem so I'm wondering if this is a race condition in startup. I know rails added uuid's in 4 at some point and so I'm wondering if it's a conflict with the new uuid column type. Any ideas?
Full error: https://gist.github.com/D1plo1d/1f5a57b90e2a0d331a94

@D1plo1d can you please elaborate on this? I'm also noticing an issue with intermittent behavior, but it's reproducible in tests. Initial run of tests fail when using spring, but saving any file and re-running the tests they pass. I'm guessing this is similar to the reload you were doing. Do you know how to use this gem on rails 4.0.2+ and get consistent results?

@claude Sorry, that was all from a year ago on my end. I can't recall the details of the issue any more then what I wrote above.

thanks for the response, so did you get uuid's working another way?

Yeah. It's actually baked into Rails 4 so we just use the built in helpers now (instead of activeuuid). Here's an example:
http://johnkolovos.blogspot.ca/2014/10/rails-4-uuid.html

Ahh, yeah that works great for postgres. Unfortunately mysql doesn't have a native uuid type and so you need this gem to do the string to binary conversion for you.  That way you don't have to store the uuid as a string in mysqll. Thanks for the help though!

np, good luck!


Terminal Hung and Crashed

Windows Terminal version
1.18.2822.0
Windows build number
10.0.22621.2428
Other Software
No response
Steps to reproduce
Download and open this file: https://1drv.ms/u/s!Aqq-Je19xlrfhR-DZMD3v-q0uqvx?e=gdWnI5
Trust me it is safe.
Expected Behavior
It should load the file.
Actual Behavior
It hung and crashed.

Trust me it is safe.

That's gonna be a nah from me dawg. Got a better way of repro'ing the issue than having me download a file off your onedrive?
Also, the Feedback Hub can actually record dumps for us: /feedback
Lastly, there's always the following the steps in https://github.com/microsoft/terminal/wiki/Troubleshooting-Tips#capture-automatically to get a post-mortem dump, then send us that to look at. Take your pick!

@skanda890 this is the content of the file Print+.py you posted on OneDrive:
print ("Hello, World")

o=9
i=9
p=98
p+i+o

and running it won't crash the terminal:
PS G:\Temp\onedrive> cat .\Print+.py
print ("Hello, World")

o=9
i=9
p=98
p+i+o

PS G:\Temp\onedrive> python .\Print+.py
Hello, World
PS G:\Temp\onedrive> python .\Print+.py
Hello, World
PS G:\Temp\onedrive> python .\Print+.py
Hello, World

Is this the file that hangs your terminal, or something is missing:

That feedback points at 4edb0d16-d70b-43ec-9be0-cecf58752fa5
points to g:6896174828363099
crashes:

MSFT:44758668 - looks like the same as the below:
MSFT:46783328 - APPLICATION_HANG_HungThreadIsIdle_cfffffff_WindowsTerminal.exe!WindowThread::_messagePump. This is usually the "the store killed us for an update"

Neither of those are what I'd expect here.
When you're "opening this file" - are you just double clicking it, or running it from the command-line?

You will need to have 7 more applications open. Or if that doesn't hang and crash the terminal. I have a computer, it is very very old. So that may be the reason why the terminal hanging and crashing. Probably because it doesn't have much resources.

Could you capture a dump of Windows Terminal while it's hung using Task Manager? This'll give us a bit more information than what generally gets uploaded.

This only happened when I had 'Python' installed.

Okay - again, could you grab a dump while it's hung/? That'll let us actually see the stack that it's hung at to further diagnose this.

Now it doesn't crash somehow.

Is it because it was downloaded from another provider?


Automatically arrange windows to test game side by side - UI Feature Proposal

Godot version:
Godot 3.0.6
Issue description:
Since the game is run in a separate window, when I'm testing a game I manually set up the window sizes so the game is side by side with the editor and I'm able to quickly test and make changes with "Sync Scene Changes" and "Sync Script Changes".
Feature Proposal:
How about a feature for automatically arranging the windows side by side. You just set the width of the editor and the height / aspect ratio for the game (in case you are testing for mobile with a vertical aspect ratio)
Here's an example of the feature:

possible dupe / related to #2326

A starting arrangement selection for game window may be added to project settings may be. Like "center", "avoid overlap", etc ...

This feature likely depends on #576 being solved first so that it be done for all platforms at once.

@aaronfranke Couldn't this be done already by comparing the screen resolution to the editor window size and position?

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Clarifications on table 5 of paper

For the experiments in table 5, what is the intervention method applied (mass mean shift vs probe weight direction)?
Particularly, I don't understand how mass mean shift could be calculated for point-wise selection.....since only specific dimensions of ecah head are selected, does that mean (i) you calculate one single mass mean vector across all the selected dimensions and (ii) within each head, you apply intervention at only the selected dimension positions by setting thetha to be zero at the unselected dimension positions?

Hello,
IRRC the first and second rows from Table 5 are using probing direction with all attention heads combined. The first row doesn't select heads but the second does, by setting theta to zero as you said.
In fact, the calculation of sigma is irrelevant to how the direction was found. For any direction in the space (masked or not), you can always first project all sample features onto it and calculate the std therein.
Sigma is the std on certain direction in the space. @CedarSnowy if you are asking theta, the theta of the third row is from the mass mean direction. That row is exactly the same as the last row in Table 3.

It's only used to calibrate intervention strength, alpha. It captures how dispersive the features are.


Temporarily disable all HMAC calls

(The service has disabled this non-GA feature temporarily, causing all tests to fail.)
Note: I'll self-merge with admin super-powers if Chris doesn't have time to review today; I want to get this in before the integration tests run internally tonight.

Integration tests passed locally.


metrics.json file is overwritten multiple times in the same shard

When running shard tests every execution of dart test, flutter test try to add their output to the same file in /metrics.json causing problems parsing the results and adding garbled content. Instead the files should be generated in a temporary folder.

I believe we have same issues for other log files uploaded to GCS.

This is different, metrics file should be used only within the test runner.


Async middleware and middleware getter

Hi !
I needed to use an async middleware on my project

Coverage remained the same at 100.0% when pulling b51fce78f7527c513da4d403943e02cb183b4545 on Aetherall:master into 585b4c7e86a58835cf3d3c5d3fe0478676bfa332 on 60frames:master.

@Aetherall Would you be able to explain a bit more about what you're trying to achieve here?

Closing due to inactivity.


🚀: Make trashy famous

I would like to see trashy in various official distributions.
Linux
Repology can cover status of huge number of Linux distributions.
[![Packaging status](https://repology.org/badge/vertical-allrepos/trashy.svg)](https://repology.org/project/trashy/versions)

Some information here https://repology.org/project/trashy/information needs cleaning up.
MacOS

[ ] brew formulae

Windows

[ ] Scoop: https://github.com/ScoopInstaller/Main/issues/5211

I totally support this, but do note that trashy is probably going to have a major release soon with big breaking changes like renaming the executable to trashy and also dropping support for the trash <PATH> invocation. So it might be best to pause these efforts until the next release because things will probably have to change in these upstream packaging efforts when that drops.

@stelcodes what's the current status?
 Is it still maintained? I would like to file an RFP for Debian.

Adding trashy to maintained-modern-unix: https://github.com/johnalanwoods/maintained-modern-unix/issues/11

trashy is already in my distribution, so not much for me to do there :)
Although the version of trashy in the nixos repo (and also the latest release on github) does not include this PR fixing #97, so in practice I still install trashy from the github repo...  The fix here would be to first (a) release a new version of trashy, and then (b) bug the package maintainer for nixos to update the package.

@j-lakeman I'm still maintaining trashy.
I've been pretty busy, but I'll probably make a new release of trashy soon.

Gr8, no worries!
I'll try to get it packaged for Debian then.


Various problems with trailing spaces in directory names in Windows and WSL

Summary:
Bash is able to create directory names with trailing spaces, but Windows does not handle them properly.
Trying to rename the directory to remove the trailing space in bash causes it to hang (I have to reboot to be able to use it again).
Expected:
Maybe Windows not able to handle trailing spaces in directory names is expected, but bash/WSL completely hanging is not.
Windows version: 10.0.14393
Steps to reproduce:
Create a directory with at least one trailing space:

In the Explorer, try to access the directory's properties, and notice info are missing in the General (the folder contains several non empty files) and Security tabs:

Try to rename the folder in the explorer, and notice you can't:

Try to rename the folder in bash to remove the trailing space, and notice the process hangs:

It's not just the mv command, the whole WSL does not respond anymore, and I have to reboot to use it again.
I can see the mv process in the task manager, even though I don't have bash running. Trying to kill it does nothing.

Just tried this and this was fixed in Creators Update.


how can i set number to chinese character

Hi,
is it possible set number to chinese character
2015/06/25 = 貳零壹伍/零陸/貳伍
0,1,2,3,4,5,6,7,8,9,10
零、壹、貳、叄、肆、伍、陸、柒、捌、玖、拾
Thanks

Well, if these characters are not used to designate things (like minutes, months), then its possible to support parsing (and printing, but I don't think people want that).

Do we even have facilities in moment to support this?  I would think we'd need something in the locale API to support this.
Also note that even in the above example, 0 and 10 have different symbols in Chinese.  I do not speak it, but I would guess that there may be other specific symbols for 11, 12, etc. and higher numbers, and other languages would have to be handled differently.
So the API would have to be a function that exchanged a number for a symbol, not a single digit.

Funny, If I run that array through Google Translate, I get:

Zero, one, two, three, wantonly, Wu Lu, Qi, eight, Nine, pick up

And if I run "one, two, three, four, five, six, seven, eight, nine, ten" through, I get different symbols:

一二三四五六七八九十

I'm sure there are cultural or contextual reasons why one set of symbols would be preferred over the other.
Still, I think asking for this feature is similar to asking for natural language parsing in English and other languages, which we've rejected many times before.   My vote is to close.

@hanzzhang - care to comment since you are fluent in Chinese?

Regard the questions:
is it possible set number to chinese character
2015/06/25 = 貳零壹伍/零陸/貳伍
0,1,2,3,4,5,6,7,8,9,10
零、壹、貳、叄、肆、伍、陸、柒、捌、玖、拾
those above Chinese characters are not used for date time. they are used for currency.
To represent 2015/06/25 in Chinese, you can do the following:

casual: 2015/25/06 , or 25/06/2015.       Note that date is before the month
a little formal: 2015年6月25日                       (年=year； 月=month， 日=day)
more formal: 二零一五年六月二十五日

I would say, 1 is used 20% of time, 2 is used 60% of time and 3 is used 20% of time.

Thanks @hanzzhang!

thanks for the help
now ,
I just use a array replace the time
var chinese_word = ['零', '壹', '貳', '叁', '肆', '伍', '陸', '柒', '捌', '玖', '拾'];

Just add a preparse function, like here: https://github.com/moment/moment/blob/develop/src/locale/my.js#L69
Pull requests welcome.

@david43, would you submit a PR to get this fixed, please? Or it isn't needed at all if used only 20% of time? Thanks!

https://github.com/moment/moment/pull/3952 adds support for both 1 and 2, which I believe is sufficient as a Chinese speaker myself. Even articles published in Chinese newspapers (eg. http://politics.people.com.cn/n1/2018/1106/c1024-30383446.html, https://tw.appledaily.com/politics/realtime/20181106/1461431) use the format shown in 2. The format using all Chinese characters for the numerals (shown in 3) is rare.


Documentation/Code issue: ondata not triggered

Summary
Hello,
First of all - great plugin with many customization possibilities :).
I'm trying to follow documentation and it's written that

For brevity we’ll only look at the process property. All other properties can be configured with the same configuration object.

However, this seems to be not true as ondata is being triggered only for process property.
Looking into the code it seems to be true as only createProcessorFunction has sendRequest with wrapper data with ondata.
How to reproduce
$(".message-attachment").filepond({
    server: {
        process: {
            url: "./UploadAttachments",
            method: "POST",
            ondata: function (formData) {
                console.log("load");
                return formData;
            }
        },
        load: {
            url: kendo.format("./LoadAttachment/{0}/", message.Id),
            method: "GET",
            ondata: function (formData) {
                console.log("load");
            }
        },
        revert: {
            url: "./RemoveAttachment",
            method: "DELETE",
            ondata: function(formData) {
                console.log("revert");
            }
        }
    },
    files: files,
    allowMultiple: true
});

Expected behaviour
Ondata is being triggered or documentation directly mentions that this property is useful only for process.
Additional information
Filepond version: 4.2.0

Environment
Version

OS
Windows 10

Browser
Chrome 73.0.3683.86 (64-bit)

The ondata callback is indeed only available for the process end point. Please submit a PR to the docs repository with the necessary changes.


show activation functions for Dense layer in the model plot

Summary
I added activation functions in the model plot to help debugging for complicated networks.  An example is given below:
Before:

After:

PR Overview

[n] This PR requires new unit tests [y/n] (make sure tests are included)
[n] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[y] This PR is backwards compatible [y/n]
[n] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thanks for the PR. I don't think we should merge this change, because:

It special-cases Dense, but many other kinds of layers may have activation functions, including custom layers.
The activation is not any more special a configuration parameter than any other parameter. It is not clear why displaying it (rather than any other parameter) helps with debugging. I believe the most important information for debugging is the input/output shape.
Dense(activation) seems like a weird notation. In Keras syntax it implies that we are applying the dense layer to an activation tensor, but that isn't the case.


Extension causes high cpu load

Issue Type: Performance
Extension Name: vscodeintellicode
Extension Version: 1.1.3
OS Version: Darwin x64 17.7.0
VSCode version: 1.31.1

:warning: Make sure to attach this file from your home-directory: /Users/myUser/VisualStudioExptTeam.vscodeintellicode-unresponsive.cpuprofile.txt :warning:
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

Thanks for the report! Closing as a duplicate of #60. This should be fixed soon with version 1.1.4 of the extension.


[WIP] remove jira support

WTF?!

Not discussed, not documented and not explained. WTF indeed??

Jira support was never working correctly and it had a bug for many months. We can think of adding it back any moment provided those things get solved, but since some of the adapters were totally falling off support I decided to remove them.

branch:changelog works for me!

@glensc Is branch:changelog the only command you use? what other commands were you using from Jira adapter?

i'm not using full power of gush, for now, branch:changelog was first touch using jira adapter, i do not know what's out there. there's no documentation (anymore?).
and i (nor you) shouldn't be the base of decision what features to keep... that's my opinion


Bump graphql experimental dependency

Hello,
Graphql just released an update and is now available under the 17.0.0-alpha.3 tag.
It's a breaking change update, running a front using the alpha 3 and a server using this integration break the defer and stream features.
An update will solve that.

This change needs to land in Apollo Server itself. It's something I might eventually get to, but if you'd like it ASAP I'd be grateful to receive a PR for it!

Why this MR included the specific tag ?
https://github.com/apollo-server-integrations/apollo-server-integration-fastify/pull/246/files#diff-78a8a19706dbd2a4425dd72bdab0502ed7a2cef16365ab7030a5a0588927bf47R56

@Maxou44 thats for testing purposes. That tag is the canary release of graphql that AS4 currently tests against. At the time, that was the most current version of the defer implementation to build against.


Initial draft at converting Python to cloudbuild.

Not ready to merge, but creating this for discussion purposes.

/cc @dlorenc


Fix GH actions

Hi.. I fixed the GH actions now the CI is passing again can you have a look?

I don't know why the docblocks have been removed, and I don't have a lot experience with Github Actions, and I don't know what the benefit is exactly, but I am not against it.

The docblocks have been removed due to php-cs-fixer newest version.. I guess this rule is include inside the @symfony ruleset.

https://github.com/moneyphp/money/blob/master/.php_cs.dist#L5

Ok, let's merge it in then.


Buttons on the extension view are not working

I have two UIButtons on the extension view. When the extension view is not moved up, the buttons are working fine. When the extension view is shifted up (stick to the top), the buttons are not working anymore. Any idea?

I don't have a solution, but I have seen this problem as well and may have some insight.  Short version:  everything works if I avoid using autolayout constraints in the extensionview.
I'm using an collectionview with a custom extensionview containing two buttons (one far left, one far right) with a label between them.  In the viewController, I use initWithFrame to instantiate the extensionview from a XIB that uses AutoLayout, and then I use setExtensionView to add it to the shyNavBarManager:
// TLYShyNav
self.extendedLayoutIncludesOpaqueBars = YES;
self.shyNavBarManager.scrollView = self.collectionView;
self.shyNavBarManager.expansionResistance = 0;
self.shyNavBarManager.contractionResistance = 0;

self.qrBar = [[MessageListQRBar alloc] initWithFrame:
              CGRectMake(0,
                         0,
                         self.view.bounds.size.width,
                         self.navigationController.navigationBar.bounds.size.height)];

[self.shyNavBarManager setExtensionView:self.qrBar];

In all cases, the extensionview appears/displays properly
In all cases, my far left button works as expected
The right-hand button does not work at all

Here are some things I've noticed:

If I scroll the collectionview down and then up, so that the extensionview is visible but is over one of the collectionview items, clicking on my right-button actually selects the collectionview item underneath.  I believe the reason my right button never works is that the underlying collectionview is always receiving its touch events. (Without scrolling, the events are going to the collectionview's empty background).
If I eliminate all AutoLayout constraints from the extensionview's XIB, all buttons work exactly as expected.  There is a conflict in my autolayout constraints that, I believe, is causing the problem above.

Don't know how to fix this yet, but hopefully the above will be helpful in finding a solution.

My extension view does not have any auto layout constraints.

Go into TLYShyNavBarManager.m and change the following line in init
self.extensionViewContainer.autoresizingMask = UIViewAutoresizingFlexibleWidth|UIViewAutoresizingFlexibleBottomMargin;

To this
self.extensionViewContainer.autoresizingMask = UIViewAutoresizingFlexibleBottomMargin;

This should be categorized as a bug because the buttons work fine on iPhone 6 but any screen size of 320 the autoresizing mask shortens the view and limits the clickable region on the right hand side.

@joshbernfeld I tried your suggestion, but it didn't resolve the issue in my case.  I have discovered a workaround, at least for my issue.  I'm using a universal storyboard and loading my extension view from a custom XIB (see code above) in viewDidLoad for my VC.  Turns out that the autolayout issues above were a red herring.
My problem occurred when buttons in my extensionview are on the right side of the screen, when the screen-width is greater than iPhone portrait (e.g. iPhone landscape, and any iPad resolution).  Basically, buttons on the left work, buttons beyond mid-screen do not (touch event passes through to the view below the extensionview).  I've worked around this by making the XIB for my extensionview 1024 points (max for iPad), and including a transparent view (0p high, 1024p wide) in that view.  After doing this, all my autolayout and buttons work in any position on the extensionview, and in any resolution.  I use autolayout to scale down for the iPhones, but this is working on all device layouts now.

@zhifeng-huang I updated the test with a button that you can test and see how it works.
@joshbernfeld Thanks for your suggestion, but I don't believe it's a bug in the library. Why shouldn't the extension view resize its width? I don't think the issue you described is applicable, since the extensionViewContainer inherits the bounds from the extension view. So, try setting the extensionView's frame before assigning it?
@dbleicher Glad it worked out :)

@Mazyod as long as my extensionview contains a full-width (e.g. 1024p wide), life is grand.  Deployed in production today with no problems.  TLYShyNavBar is awesome!

I am also facing the same issue. I am creating the elements in extensionView through code and does not use xib / storyboard / autoLayout at all. Still my buttons are not working when extensionView is shifted. Any of the above solutions worked for me.
Any suggestions / fixes guys ?


Documentation on networking restrictions

Is it possible to place any networking restrictions on the storage account without affecting the diagnostic collection?

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

ID: 0ca576e1-fafa-3a02-6d29-394f342bc737
Version Independent ID: a6a83de5-41fd-2dce-eb86-cee6c086c07d
Content: Create diagnostic settings to send platform logs and metrics to different destinations - Azure Monitor
Content Source: articles/azure-monitor/platform/diagnostic-settings.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@haymansfield Thank you for your query, we will look into it and get back to you at the earliest.

https://docs.microsoft.com/en-us/azure/azure-monitor/platform/roles-permissions-security#monitoring-within-a-secured-virtual-network
Yes. If you turn on firewall or other network access controls to the storage account, you must check "Allow trusted Microsoft Services" to allow diagnostic settings to flow.

Thanks a lot @nkiest for helping on this thread.
@haymansfield Apologies for delay in getting back to this thread. Please review the information shared by @nkiest and feel free to revert back if you have any further queries.
Hope the above information helps. At this point it appears that there's no further edits or updates needed to the documentation. We will go ahead and close this issue, however if you have any further queries please tag us in your reply and will be happy to reopen the thread and continue the conversation. Thank you


Couldn't find pack in packagist

Hi!
First of all, thanks for creating db tool, it looks promising.
The pack-fr isn't available on packagist, so we cannot require it. See :

Is it possible to make it available ? Or is this intended to delcare the repo in the composer.json ?

Hi,
sorry for the delay, I hadn't seen the issue. Pakcage has been registered with the wrong name, should be good now.


Error validate the Job.setting.json

I have problems when uploading the video in the Input Bucket, I receive a notification in my email indicating that the download of the file failed, I leave the specific message:
"Mistake": {
"Message": "Failed to download and validate the job-settings.json file. Please check its contents and location. Details on using custom settings: https://nam10.safelinks.protection.outlook.com/?url=https% 3D%7C3000%7C%7C%7C&sdata=QFshicn4hdr3fy9JCYHt5GjrxDzkM7mfInJ7KrcbZfY%3D&reserved=0",
"Error": "AccessDenied: Access Denied"

Thank you for opening this issue. We have added this to our backlog to look into.

seeing same issue

I have escalated this to get it prioritized in the backlog. Thanks.

Same issue here

Don't forget to add your uploaded file to /assets01. Solved the error for me.

Hi @cdeavila we are looking into this. can you clarify for us.. "I have problems when uploading the video in the Input Bucket", are you uploading into the root of the bucket?

hi @cdeavila please refer to this section of the Implementation Guide for reference: https://docs.aws.amazon.com/solutions/latest/video-on-demand-on-aws-foundations/components.html
For the next release, we will update the Implementation Guide so that it is more specific.


InteractiveElement.type() Fails When Text is Single Quote Character

InteractiveElement:301 - If javascript type mode is on and the text is a single quote character, the javascript that gets executed becomes "arguments[0].value=''';" which fails to evaluate.
In my project, we have some test cases that need to type a single quote into a text box.  For reasons outside of the scope of this issue, my test enters this using the JS type mode.
Instead of putting it on the users to constantly remember to escape their single quotes, I believe it would be easier and cleaner for EWD to escape the characters automatically (if they haven't been escaped by the user) before evaluating the Javascript.
Thoughts?

@mikeastarb Sounds like a good idea.  Why don't you go ahead and make a PR?

@kood1 Sure thing, I'll see if I can get it done this afternoon

@mikeastarb We can close this now, yes?

Ah, yes. I didn't realize it hadn't auto-closed


Enable CloudFront logging for Data Portal

Including logging of web browser exceptions.

Enabled in lower environments waiting on GDPR approval to enable in prod.

Enabled in prod @theathorn


Emojis not displayed correctly in the editor windows

Issue Type: Bug

Type the heart emoji – ❤–  in a Markdown file.
Open Markdown Preview
Expected: The heart in the editor should display as a red and nice heart, like it does in the preview.
Actual: The heart displays as a small character (still heart shaped, but not the real emoji).

VS Code version: Code - Insiders 1.41.0-insider (27cd33d891eb23e370b672e751be9fe8b3748ac1, 2019-11-08T19:11:22.061Z)
OS version: Darwin x64 19.0.0
(The issue was introduced with VS Code 1.40.0. I just used Insiders to check that it hadn't already been fixed.)

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.29GB free)

Process Argv
-psn_0_3580778

Screen Reader
no

VM
0%

Extensions: none

Hey @alexandrudima, I think issue is that the editor itself doesn't display the colorful emoji (while the markdown preview does, as expected)

Assigning back to you

This now seems fixed in 1.40.1.

Actually. That is not the case at all. On my computer (a MacBook Pro, see system info in the OP) I have two user accounts. If I start code --disable-extensions from one of the accounts I have the issue, from the other I don't. Please let me know if there is some diagnostics I can provide that could help pinpoint the source of the problem.

@PEZ The emoji is ultimately rendered with a font. Can you please:

Clarify if the emoji is not rendered correctly in the editor or in the markdown preview. Please include a screenshot to remove any possible misinterpretation.
Check to see what are the differences in settings.json between the two user accounts
(if you have customized the font family) Check that you have installed fonts in a way that they are available to all users.


Fix crash when a shell extension crashes

Fix #138
example of misbehavior:
colcon.colcon_core.shellFailed   <<< orocos_kdl [ Exited with code 1 ]                                                                                ERROR Exception in shell extension 'bat': local variable 'done' referenced before assignment                                                                                Traceback (most recent call last):
File "c:\python38\lib\site-packages\colcon_core\subprocess.py", line 176, in _async_check_call
done, _ = await asyncio.wait(callbacks, return_when=ALL_COMPLETED)
File "c:\python38\lib\asyncio\tasks.py", line 426, in wait
return await _wait(fs, timeout, return_when, loop)
File "c:\python38\lib\asyncio\tasks.py", line 523, in _wait
await waiter
asyncio.exceptions.CancelledError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "c:\python38\lib\site-packages\colcon_core\shell_init_.py", line 275, in get_command_environment
return await extension.generate_command_environment(
File "c:\python38\lib\site-packages\colcon_core\shell\bat.py", line 123, in generate_command_environment
env = await get_environment_variables(cmd, cwd=str(build_base))
File "c:\python38\lib\site-packages\colcon_core\shell_init_.py", line 312, in get_environment_variables
output = await check_output(cmd, cwd=cwd, shell=shell)
File "c:\python38\lib\site-packages\colcon_core\subprocess.py", line 104, in check_output
rc, stdout_data, _ = await _async_check_call(
File "c:\python38\lib\site-packages\colcon_core\subprocess.py", line 183, in _async_check_call
for task in done:
UnboundLocalError: local variable 'done' referenced before assignment

Can you please provide reproducible steps.

Yeah. On Windows just try running colcon from a normal terminal, without running the MSVC initialization scripts. You'll get the above errors if you don't have this patch. If you do, you'll get a more meaningful actual error:
Starting >>> fastcdr
...
--- stderr: fastcdr
Traceback (most recent call last):
  File "c:\users\dan\documents\colcon-core\colcon_core\executor\__init__.py", line 91, in __call__
    rc = await self.task(*args, **kwargs)
  File "c:\users\dan\documents\colcon-core\colcon_core\task\__init__.py", line 92, in __call__
    return await task_method(*args, **kwargs)
  File "c:\python38\lib\site-packages\colcon_cmake\task\cmake\build.py", line 85, in build
    rc = await self._reconfigure(args, env)
  File "c:\python38\lib\site-packages\colcon_cmake\task\cmake\build.py", line 150, in _reconfigure
    raise RuntimeError(
RuntimeError: VisualStudioVersion is not set, please run within a Visual Studio Command Prompt.
---
Failed   <<< fastcdr    [ Exited with code 1 ]

On Windows just try running colcon from a normal terminal, without running the MSVC initialization scripts.

I just ran the following command on a command prompt without the Visual Studio environment and got the desired  output - that is without this change:
$ colcon build --packages-select fastcdr --merge-install
Starting >>> fastcdr
--- stderr: fastcdr
Traceback (most recent call last):
  File "c:\python37\lib\site-packages\colcon_core\executor\__init__.py", line 91, in __call__
    rc = await self.task(*args, **kwargs)
  File "c:\python37\lib\site-packages\colcon_core\task\__init__.py", line 92, in __call__
    return await task_method(*args, **kwargs)
  File "c:\python37\lib\site-packages\colcon_cmake\task\cmake\build.py", line 85, in build
    rc = await self._reconfigure(args, env)
  File "c:\python37\lib\site-packages\colcon_cmake\task\cmake\build.py", line 151, in _reconfigure
    'VisualStudioVersion is not set, '
RuntimeError: VisualStudioVersion is not set, please run within a Visual Studio Command Prompt.
---
Failed   <<< fastcdr    [ Exited with code 1 ]

Summary: 0 packages finished [1.89s]
  1 package failed: fastcdr
  1 package had stderr output: fastcdr
VisualStudioVersion is not set, please run within a Visual Studio Command Prompt.

local variable 'done' referenced before assignment

I could see how this could happen (even though I can't reproduce it atm). I would suggest to initialize done with an empty set before the try block.

I’ll have to look at it closer. I think what I was seeing might only happen with concurrent build jobs.

Codecov Report

Merging #245 into master will increase coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #245      +/-   ##
==========================================
+ Coverage   80.04%   80.12%   +0.07%
==========================================
  Files          54       54
  Lines        3107     3099       -8
  Branches      514      513       -1
==========================================
- Hits         2487     2483       -4
+ Misses        582      579       -3
+ Partials       38       37       -1

Impacted Files
Coverage Δ

colcon_core/subprocess.py
66.37% <100%> (+1.08%)
:arrow_up:

Continue to review full report at Codecov.

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

I’ll have to look at it closer. I think what I was seeing might only happen with concurrent build jobs.

Ok, please provide the steps once you were able to reproduce it.

please provide the steps once you were able to reproduce it.

@rotu Friendly ping.

Yes, I was able to reliably reproduce this by trying to build multiple packages at once in parallel without --continue in a non-VS windows terminal. One package fails gracefully, causing the others to fail clumsily.

rebased on master

@rotu Can you please double check if this change is still necessary after #284 (which fixes the problem that CancelledError changed its type in Python 3.8).

@rotu Friendly ping.


Updated Blob Auditing API Files

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month. - Feb 2022
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month. - Feb 2022
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@jianyexi can you help to review and merge this PR? Thanks~

/azp run

@msdogra, you need to study the materials described in the submission form and attest to that by checking the box in the submission form. This is normally required before ARM API review. In this case, since this is going into an aggregation branch, with the goal of ARM-reviewing earlier in the process, I've gone ahead and reviewed it anyway. Please observe the requirement in all future PRs.
General feedback: this is an extremely noisy PR, even though there are hardly any actual changes. Please don't combine changes at the REST layer (which require ARM review) with other updates, e. g. reordering, refactoring, renaming, reformatting, and the like. Those changes make the ARM reviewer's job so, so much harder and slower. This would be much better if the new property was added in its own PR, and the refactoring was a separate PR where only the first would require ARM review.

Signed off for ARM with comments.

@qiaozha @ArcturusZhang , can you check the potential SDK breaking change?


There is an unrelated bug on Release Notes in New section

Storage Explorer Version: 1.11.0
Build: 20191101.5
Branch: rel/1.11.0
Platform/OS: Windows 10/ Linux Ubuntu 19.04/macOS High Sierra
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Launch Storage Explorer -> Help -> View Release Notes.
Check the bug #1249 appended to settings description 'Display and hide service nodes' on the New section.

Expect Experience:
Show the appended bug related to the description.
Actual Experience:
The bug is not related to the description.

More Info:
Related bugs to the description 'Display and hide service nodes'.
Bugs: #1877, #2146

Can't seem to find the correct bug, so we'll just remove it, unless someone knows what the correct bug number is?

Found it. Bug in question is #1877.


wireguard: 0.0.20180118 -> 0.0.20180202

Simple version bump.
  * curve25519-fiat32: uninline certain functions

  This results in much smaller code size and significanat speed gains on smaller
  hardware.

  * poly1305: add poly-specific self-tests

  Poly is easy to get wrong, so we've added quite a few tests that examine
  certain edge cases and places where other implementations of historically
  failed.

  * tools: dedup secret normalization
  * tools: share curve25519 implementations with kernel
  * contrib: keygen-html: share curve25519 implementation with kernel

  There is now only one place where we ship 25519 code.

  * qemu: disable PIE for compilation
  * qemu: disable AVX-512 in userland
  * qemu: update base versions

  Test suite enhancements.

  * device: let udev know what kind of device we are

  This enables folks to query the device type via udev, which is what systemd's
  networkctl uses.

  * tools: fread doesn't change errno

  This fixes clearing pre-shared keys on old glibc.

  * chacha20poly1305: use existing rol32 function
  * chacha20poly1305: better buffer alignment

  Small enhancements.

  * curve25519: verify that specialized basepoint implementations are correct

  Since some implementations have a specialized function for computing
  basepoints, it's important to do some basic sanity checking with them.

  * curve25519: replace hacl64 with fiat64

  For about 24 hours, fiat64 was faster.

  * curve25519: replace fiat64 with faster hacl64

  Then hacl64 caught up, so we moved back to it.

  * curve25519: break more things with more test cases

  These extra test cases help break the current "rfc7748_precomputed"
  implementation, which we're not using here at the moment, but it is still
  useful to ensure that we don't fall victim to the same bugs.

@GrahamcOfBorg build linuxPackages_testing.wireguard linuxPackages_latest.wireguard linuxPackages.wireguard linuxPackages_4_15.wireguard linuxPackages_hardened.wireguard

Failure on x86_64-darwin (full log)
Partial log (click to expand)
error: while evaluating 'callPackageWith' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:113:35, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:13136:17:
while evaluating 'makeOverridable' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:72:24, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:117:8:
while evaluating anonymous function at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/wireguard/default.nix:1:1, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:74:12:
while evaluating the attribute 'linux_testing' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:12967:3:
while evaluating 'callPackageWith' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:113:35, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:12967:19:
while evaluating 'makeOverridable' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:72:24, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:117:8:
while evaluating anonymous function at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/linux-testing.nix:1:1, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:74:12:
while evaluating anonymous function at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/generic.nix:1:1, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/linux-testing.nix:3:1:
assertion failed at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/generic.nix:33:1

Success on aarch64-linux (full log)
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/kjz8djrzljcjq7vdks9yj55mk5ff4i1h-wireguard-0.0.20180202
strip is /nix/store/xmpjypwjmp2qi1chs5kr0hacnh161ls4-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/kjz8djrzljcjq7vdks9yj55mk5ff4i1h-wireguard-0.0.20180202/lib
patching script interpreter paths in /nix/store/kjz8djrzljcjq7vdks9yj55mk5ff4i1h-wireguard-0.0.20180202
checking for references to /build in /nix/store/kjz8djrzljcjq7vdks9yj55mk5ff4i1h-wireguard-0.0.20180202...
/nix/store/kjz8djrzljcjq7vdks9yj55mk5ff4i1h-wireguard-0.0.20180202
/nix/store/mjhxz0hl0bpl3dnnp21f14jyxq84a54f-wireguard-0.0.20180202
/nix/store/5imc1b20y2vpcgiadw4n6plafg71b22f-wireguard-0.0.20180202
/nix/store/mjhxz0hl0bpl3dnnp21f14jyxq84a54f-wireguard-0.0.20180202
/nix/store/5vm3p11azpfc9zh74d6i09zrkmb6ikj5-wireguard-0.0.20180202

Failure on x86_64-linux (full log)
Partial log (click to expand)
install flags: install
  INSTALL /tmp/nix-build-wireguard-0.0.20180202.drv-0/WireGuard-0.0.20180202/src/wireguard.ko
  DEPMOD  4.15.1
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/87s2riji72md3axn42dbc8jlf2yaddkn-wireguard-0.0.20180202
strip is /nix/store/5qj61lcvzlap87rf6blvf8p577d482bv-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/87s2riji72md3axn42dbc8jlf2yaddkn-wireguard-0.0.20180202/lib
patching script interpreter paths in /nix/store/87s2riji72md3axn42dbc8jlf2yaddkn-wireguard-0.0.20180202
checking for references to /tmp/nix-build-wireguard-0.0.20180202.drv-0 in /nix/store/87s2riji72md3axn42dbc8jlf2yaddkn-wireguard-0.0.20180202...
error: build of ‘/nix/store/zxwnbmlgxqp3r6ys23hjrgvai166dd00-wireguard-0.0.20180202.drv’ failed

Failure due to 1800s timeout. I tested it on one of my machines using nox-review and all kernel versions seem to work fine.
Thanks for keeping it updated @zx2c4 ! :+1:

Backported in 467097464c75b455da656688da5c17bb31f79698


Design initial database schema

Relationships

One-to-Many:

A User can create multiple Recipes.
A Recipe can have multiple Comments.
A Recipe can have multiple Ratings.
A Recipe can have multiple Ingredients.

Many-to-Many:

Recipes and Ingredients could have a many-to-many relationship through a RecipeIngredient junction table.

Entity Relationship Diagram
erDiagram
    USER {
        int id PK
        string username
        string email
        string password_hash
    }

    RECIPE {
        int id PK
        string title
        text description
        int cooking_time
        int servings
        string category
        int user_id FK
    }

    INGREDIENT {
        int id PK
        string name
        string unit
    }

    RECIPE_INGREDIENT {
        int id PK
        int recipe_id FK
        int ingredient_id FK
        string quantity
    }

    COMMENT {
        int id PK
        text content
        datetime created_at
        int user_id FK
        int recipe_id FK
    }

    RATING {
        int id PK
        int value
        int user_id FK
        int recipe_id FK
    }

    USER_FAVORITE {
        int id PK
        int user_id FK
        int recipe_id FK
    }

    USER ||--o{ RECIPE : "creates"
    USER ||--o{ COMMENT : "makes"
    USER ||--o{ RATING : "gives"
    USER ||--o{ USER_FAVORITE : "favorites"

    RECIPE ||--o{ RECIPE_INGREDIENT : "uses"
    INGREDIENT ||--o{ RECIPE_INGREDIENT : "is used in"

    RECIPE ||--o{ COMMENT : "receives"
    RECIPE ||--o{ RATING : "receives"
    RECIPE ||--o{ USER_FAVORITE : "is favorited by"

    USER ||--o{ USER_FAVORITE : "favorites"


Template fixes

Ping @balajismaniam
Without this fix the selinux feature source is not working in DaemonSet

We can wait till the CI passes and then merge.


fix: price_gte to only be included when min is explicitly specified

blocked by: #6868
Buy Now filter
listed=true
Shows items that has 0 price or more
Well i don't know if it's by design.
This has been considered by @daiagi as issue which i have linked in the issue #6893
Further investigation shows me that useSearchPriceRange returns wrong defaultParams
It is because default params have price_gte=0
it is illogical to have that argument together with price_gt=0 when price is actually 0
unless explicitly specified by the min filter
This PR corrects this issue
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #6893

[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖 Generated by Copilot at 769abca
Refactored the code for opening the shopping cart modal from the navbar. Moved the logic and configuration from ShoppingCartModalConfig.ts and ShoppingCartButton.vue to Navbar.vue to improve code organization and performance.

🤖 Generated by Copilot at 769abca

ShoppingCartButton
Simpler, no more emit -
Autumn leaves falling

@roiLeo  That's because of #6868

@roiLeo That's because of #6868

@roiLeo let me explain a bit further. I'm sorry for the confusion.
#6868 fixes the filter of buy now
Basically without that, Buy Now is not working.
This on the other hand, fixes the query for the graphql side. Because the query is incorrectly returning items with 0 price.
I hope that clears it up 😂
I like your memes 🤭

@yangwao 20th PR 🎉🙏🥳


Need push transport

The code in knife-push can be most likely pretty easily adapted to this, although it'll bring in the chef gem and Chef::REST as run-time deps to the project (and if some upstream consumer of this gem also depends on chef, or if chef at some point pulls in this gem, then you wind up with diamond or circular deps).

@lamont-granquist Great idea to add more transports. In general we try to keep the dependencies in train reduced in order to ensure it is fast and lightweight. This project focuses on transport only. Can we find an abstraction e.g. Rest Commands for Chef Push and leave other parts within a knife implementation?

Well the knife-push code could be copypasta'd, its not much, so you don't have to bring in knife-push as a dependency.  The problem is that you'd want to leverage Chef::REST or Chef::ServerAPI and right now those aren't available as libraries so that would slurp in the entire chef-client as a runtime dep.

The idea of train is to be a lightweight framework for command execution and file transfer to the machine. I believe that we should not include the chef gem here. May it make sense to extract the required modules and turn this into a small gem with no dependencies?

I would love to see Chef::HTTP/Chef::REST/Chef::ServerAPI extracted out to a library for exactly this kind of purpose.

As there has not been much motion on this in the last 3 years, I'm going to close this - if there is demand for this, please feel free to re-open.


Conda build

This PR is more for reference in case it's useful - feel free to close as it looks like it breaks Travis. If a conda build is added in future this may serve as a useful starting point.
I need windows conda packages so I added conda build scripts. The built packages can be installed with
conda install -c dhirschfeld autograd

Closing as it's not suitable for merging as is.


Add Laravel Pushwoosh To Docs?

@shaders is it possible to add a link to this library in the Pushwoosh documentation? http://docs.pushwoosh.com/docs/

@shaders do you think it would be okay?

done! http://docs.pushwoosh.com/docs/createmessage

Thanks!


Node Version

Is there any chance of getting a supported version for Node?  Thanks!

Unfortunately, this isn't on our immediate roadmap. If you are a commercial customer, reach out to your account manager and we'll see what can be arranged.

Thanks for the response. It's too bad, as it looks like the only piece missing for a node implementation of smart speakers that are in your blog posts as both Leopard and porcupine have it. I totally get it tho, thanks again.


basic completion for variant constructors

This is a crude attempt at continuing https://github.com/rescript-lang/rescript-vscode/pull/493 by completing variant constructors in labelled argument assignments. I looked around and tried to follow roughly how things have been done before. It's not complete, but it's a start, and it does actually do completion.

Will look at it later. Need to cover basic earlier steps first.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/515 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target.git

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

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-06-16-07-37-49 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-06-16-07-37-49

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Cannot merge namespace data from multiple files

Hello, vs code seems to fail at merging information about a namespace, coming from multiple files.

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.77.3
OS Version: Ubuntu 22.10

Steps to Reproduce:

Define a global namespace in a .d.ts file
Extend the definition in another .d.ts file
You notice that namespace is not usable in the second file, until it has an import.

Here's a video, to see the bug in action.

Please share a code example that demonstrates the issue

https://deinstantway.com/$/uniqueFileUpload/download/iedZBwgrqpeZvea3S1u9Dr5uSYWvoFiVRd7ZtDxBBTSk

Please share the code, not videos

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


[GHSA-fwr7-v2mv-hh25] Prototype Pollution in async

Updates

Affected products

Comments
Apologies as I couldn't find much information on this one, but the NVD entry does suggest that prior to 2.x should be considered affected as well (I understand that isn't always particularly reliable though).  I did see this comment which suggested perhaps the vulnerability doesn't actually apply to prior versions, but I wasn't really sure so would appreciate any help in verifying that.  Thanks for maintaining such a great dataset!

@westonsteimel Thank you! The 1.5 documentation didn't mention the mapValues function at all, so I did some spelunking and found that it was added in this PR, so the vulnerable code did not exist prior to 2.x.

Ah great, thanks for taking a further look at this one.  Much appreciated!

No problem!


Make script to upload a new public NER corpus

AS A Anonymiser Application user
I WANT TO execute the command line script
IN ORDER TO upload the public NER corpus to Github.
Acceptance Criteria:

[ ] Script can be executed via command line
[ ] Script connects to ReSQL via Ruuter and gets the NER corpus.
[ ] Corpus is formatted according to Git requirements.
[ ] Script uploads the corpus to Git via PUT https://api.github.com/repos/${GIT_OWNER}/${GIT_REPO}/contents/corpus_${corporaId}.json

@alimuhammadahmer @kunnark

This issue is marked as "Done" but I can not see any code commits;
ACs must contain all REST endpoints used to get the result.

GIT upload Endpoint:
PUT https://api.github.com/repos/${GIT_OWNER}/${GIT_REPO}/contents/corpus_${corporaId}.json

Closing this issue as we ended up using https://huggingface.co/buerokratt instead.


Please add following freezer

It would be great to add my freezer.

I wasn't aware of this, I thought I did support it already since I also added support for the fridge.
Thanks for notifying. Will look into this.

As I do not have a freezer myself nor a fridge, please test and report any issues. It should behave largely the same as a fridge tested by another person I hope.

:tada: This issue has been resolved in version 3.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Hey Sander,
Thanks for your quick response.
Now the freezer is added in HomeKit. Unfortunately there is only +C*. Minus is missing :-(

Minus 15 is the Minimum

Yes i noticed the same after I published.
It will need some more time as I need to pull the range from the API response to re-use the fridge code. Your HomeKit Freezer is now using the fridge code and therefore limited to 1-9 degrees (that is the fridge limits as set by Miele).

Hello!
side effect of this new 3.2.0 version: fridge temperature is now correct (without the .1), see #24 :-)

fridge temperature is now correct

Maybe the internal homebridge rounding changed since  widened the valid range to support sub zero temperatures:
    this.mainService.getCharacteristic(this.platform.Characteristic.CurrentTemperature)
      .on('get', currentTemperatureCharacteristic.get.bind(currentTemperatureCharacteristic))
      .setProps({
        minValue: -50,
        maxValue: 100,
      });

Maybe but I did not update Homebridge since a while (at least way before plugin 3.1.0) and with plugin 3.1.0 the 4.1 was still visible.

I meant: with the same Homebrige version but with the changed plugin code (larger range for sub zero temperatures) it triggers different internal Homebridge rounding.
Not sure, but could be a cause.

Make sense !

Hello Sander,
after the last update my freezer is shown in Homekit but I cannot change anymore the temperature :-(

I wonder if this is due to the issues I am see with the Miele server currently (see #37). All my Miele appliances are seen as "OFF" with all characteristics to 0 (the appliances are correctly displayed in the Miele app though).

@Sebbl81 Could you retest? The European event server is back in working condition (don't know what region you are from and how many Miele servers there are though).


Bump distro + update labels

Bumps distro commit and updates label needs version info to version-info-needed

Is this for July still?


cavalier: fix missing libGL runtime dep

Description of changes
libGL is required at runtime for Cavallier to work.
This PR fixes #319369
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Build and runs well, thank you.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1745


Failure to parse date fields in WHOIS record: parsing time "2022-06-03" as "2006-01-02T15:04:05Z07:00": cannot parse "" as "T"

While troubleshooting #115 I opted to explicitly specify the WHOIS server for a particular domain and received this error:

failed to parse domain updated date: parsing time "2022-06-03" as "2006-01-02T15:04:05Z07:00": cannot parse "" as "T"

Relevant code block:
https://github.com/atc0005/check-whois/blob/2c907e24b8f0f35e08f9bcbded579da0455075bb/internal/domain/domain.go#L74-L80

Small prototype:
package main

import (
	"fmt"
	"time"
)

func main() {
	testTime := "2021-09-15T21:10:12Z"
	_, err := time.Parse(time.RFC3339, testTime)
	if err != nil {
		fmt.Println("Failed to parse sample:", err)
	}
	fmt.Println("Sucessfully parsed sample")
}

Works without issue on the 2021-09-15T21:10:12Z value pulled from the WHOIS record.

From https://github.com/likexian/whois-parser/blob/81fca9d7ed2ab07bf306d250d7921174fff00723/util.go#L116-L159:
func parseDateString(dateString string) (time.Time, error) {
	formats := [...]string{
		"2006-01-02T15:04:05Z",
		"2006-01-02",
		"2006-01-02 15:04:05",
		"2006. 01. 02.",
		"02-Jan-2006",
		"02/01/2006 15:04:05",
		"02.01.2006",
		"02-01-2006",
		"02.01.2006 15:04:05",
		"02.1.2006 15:04:05",
		"2.1.2006 15:04:05",
		"2006-01-02 15:04:05-07",
		"02-Jan-2006 15:04:05",
		"January _2 2006",
		"02/01/2006",
		"01/02/2006",
		"2006-01-02 15:04:05 MST",
		"2006-Jan-02",
		"2006-Jan-02.",
		"2006-01-02 15:04:05 (MST+3)",
		time.ANSIC,
		time.UnixDate,
		time.RubyDate,
		time.RFC822,
		time.RFC822Z,
		time.RFC850,
		time.RFC1123,
		time.RFC1123Z,
		time.RFC3339,
		time.RFC3339Nano,
		time.Stamp,
		time.StampMilli,
		time.StampMicro,
		time.StampNano,
	}

	for _, format := range formats {
		result, err := time.Parse(format, dateString)
		if err != nil {
			continue
		}
		return result, nil
	}

I added a log statement emit which format was chosen (debugging). I see this output:
2022/11/08 05:03:14 The "2006-01-02" time format was chosen
2022/11/08 05:03:14 The "2006-01-02" time format was chosen
2022/11/08 05:03:14 The "2006-01-02T15:04:05Z" time format was chosen

Two thoughts:

the logic applied by this plugin needs to gracefully handle not being able to parse the returned dates in the expected time format (it should fallback to a basic value)
the library should probably reorder the date formats so that all of the more specific/detailed formats are tried first before falling back to a basic YYYY-MM-DD format

While troubleshooting #115 I opted to explicitly specify the WHOIS server for a particular domain

No, this didn't work as expected. After fixing the date format (PR incoming), I realized that the WHOIS record for org itself was returned when I explicitly chose to use the whois.iana.org server.
Direct lookup using stock whois tool:
$ whois org
% IANA WHOIS server
% for more information on IANA, visit http://www.iana.org
% This query returned 1 object

domain:       ORG

organisation: Public Interest Registry (PIR)
address:      11911 Freedom Drive 10th Floor,
address:      Suite 1000
address:      Reston, VA 20190
address:      United States

contact:      administrative
name:         Director of Operations, Compliance and Customer Support
organisation: Public Interest Registry (PIR)
address:      11911 Freedom Drive 10th Floor,
address:      Suite 1000
address:      Reston, VA 20190
address:      United States
phone:        +1<PHONE_NUMBER>
fax-no:       +1<PHONE_NUMBER>
e-mail<EMAIL_ADDRESS>
contact:      technical
name:         Senior Director, DNS Infrastructure Group
organisation: Donuts Inc.
address:      10500 NE 8th Street, Suite 750
address:      Bellevue, WA 98004
address:      United States
phone:        1.425.298.2200
fax-no:       1.425.671.0020
e-mail<EMAIL_ADDRESS>
nserver:      A0.ORG.AFILIAS-NST.INFO <IP_ADDRESS> 2001:500:e:0:0:0:0:1
nserver:      A2.ORG.AFILIAS-NST.INFO <IP_ADDRESS> 2001:500:40:0:0:0:0:1
nserver:      B0.ORG.AFILIAS-NST.ORG <IP_ADDRESS> 2001:500:c:0:0:0:0:1
nserver:      B2.ORG.AFILIAS-NST.ORG <IP_ADDRESS> 2001:500:48:0:0:0:0:1
nserver:      C0.ORG.AFILIAS-NST.INFO <IP_ADDRESS> 2001:500:b:0:0:0:0:1
nserver:      D0.ORG.AFILIAS-NST.ORG <IP_ADDRESS> 2001:500:f:0:0:0:0:1
ds-rdata:     26974 8 2 4fede294c53f438a158c41d39489cd78a86beb0d8a0aeaff14745c0d16e1de32

whois:        whois.publicinterestregistry.org

status:       ACTIVE
remarks:      Registration information:
remarks:      http://publicinterestregistry.org

created:      1985-01-01
changed:      2022-06-03
source:       IANA

These are the fields that were triggering the warning:
created:      1985-01-01
changed:      2022-06-03


Status of this repo

Just so people can have a definitive source for this: I am no longer actively maintaining this extension. Working on other things now and this was more of a pet project that I wanted for myself in JS projects.
Now, if someone wants to continue the reigns of this project, I wholeheartedly approve of a hard fork of this repo, and if you get it off the ground again, I can point people to stop using this extension and instead use yours.

Thanks for all the hard work you put in @lannonbr, I think sometimes people don't appreciate open source creators enough. Cheers 🍻

Thanks @lannonbr .
Any other plugins recommend with TS/JS project?

I've just released an extension that does the same: https://github.com/imliam/vscode-inline-parameters
It was originally just for PHP, but since I saw this was no longer maintained I also pulled in the AST parsing logic to support JS/TS in it too. It uses a different mechanism to get the parameter names (using hover providers) so may behave a little differently than this extension, but I intend to support it and continue adding to it.


Locality sensitive hashing annotation

Adds a TLSH hash value as annotation (when possible).
This aims to ease the computation of OSCAR diffs (each document has a URL+TLSH, which could help build a "similarity score" that would help identify similar pages throughout differrent OSCAR releases.

TODO:

[ ] Push changes on tlsh-rs