msRestUserAgentPolicy.js errors - cannot resolve "os"

Package Version: 1.1.0_

[ ] nodejs

nodejs version: ______
os name/version: _____

[ ] browser

name/version: ____
Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

Updating our ms-rest-js package from 1.0.x to 1.1.0
changed
from
"ms-rest-js": "^1.0.x",
to
"@azure/ms-rest-js": "^1.1.0",
in my package.json file

Expected behavior
It worked before I updated this package
Screenshots
I get this error
I do npm run serve on my angular website and get this error
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'os' in
....\ClientApp\node_modules@azure\ms-rest-js\es\lib\policies'
which seems to indicate the import statement is failing
import * as os from "os";
Additional context
HELP

It seems that we should not expect ms-rest-js and @azure-ms-rest-js to be identitical even with the same version number

I'm a little confused. Is there actually an issue here, or is everything fine now?

Still doesnt work after clearing all:

npm node modules from \users...\appdata\roaming\npm and npm-cache
cleared the autorest and reinstalled: \users....autorest:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
package.json: 		"@azure/ms-rest-js": "^1.1.0",
in my typescript client:
import { TokenCredentials } from '@azure/ms-rest-js';
I regen my client but still get error even though the file exists:
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'os' in 'D:\Phils\GIT\Projects\sra\src\Sra.Web\ClientApp\node_modules@azure\ms-rest-js\es\lib\policies'
Which DOES EXIST:
// Copyright (c) Microsoft Corporation. All rights reserved.
// Licensed under the MIT License. See License.txt in the project root for license information.
export function getDefaultUserAgentKey() {
return "x-ms-command-name";
}
export function getPlatformSpecificData() {
var navigator = window.navigator;
var osInfo = {
key: "OS",
value: (navigator.oscpu || navigator.platform).replace(" ", "")
};
return [osInfo];
}
//# sourceMappingURL=msRestUserAgentPolicy.browser.js.map

Okay, I have two things for you to try:

See if changing @azure/ms-rest-js to 1.1.1 solves the problem. We did have a bundling bug that we fixed just a couple of days ago that seems related to this issue. If this doesn't solve the problem, then:
See if adding @types/node to your devDependencies solves the problem. If it does, then I'll move @types/node from our devDependencies to our dependencies.

Can confirm this is an issue. I am getting the same problem described in the original submission. I have also tried all of the steps suggested.

Hi there,
Tried the version 1.1.1 - same error
Tried adding the @types/node and still the error
npm install @types/node  --save-dev
Also tried on another developers machine

Okay, thanks for giving that a try. Can you post a copy of how your code consumes the ms-rest-js package (specifically the lines where you're importing from ms-rest-js in your code)? Did you use AutoRest to generate a TypeScript client?

Thanks for looking into this for me. Details requested below:
This is the command I use to autogenerate the client:
autorest --typescript --input-file=http://localhost:61817/ApiCore/swagger/v1/swagger.json --output-folder=gentypescript --add-credentials --package-name="apiCore" --package-version="1.0" --license-header=MICROSOFT_MIT_NO_VERSION
The Output:
AutoRest code generation utility [version: 2.0.4283; node: v10.13.0]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
Loading AutoRest core      'C:\Users\x.autorest@microsoft.azure_autorest-core<EMAIL_ADDRESS>(2.0.4289)
Installing AutoRest extension<EMAIL_ADDRESS>(~2.0.12)
Installing AutoRest extension<EMAIL_ADDRESS>(2.3.51)
The Output from autorest --version:
AutoRest code generation utility [version: 2.0.4283; node: v10.13.0]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
Showing All Installed Extensions
Type       Extension Name                           Version      Location
core       @microsoft.azure/autorest-core           2.0.4289     C:\Users\x.autorest<EMAIL_ADDRESS>extension  @microsoft.azure/autorest.csharp         2.3.82       C:\Users\x.autorest<EMAIL_ADDRESS>extension  @microsoft.azure/autorest.modeler        2.3.51       C:\Users\x.autorest<EMAIL_ADDRESS>extension  @microsoft.azure/autorest.modeler        2.3.55       C:\Users\x.autorest<EMAIL_ADDRESS>extension  @microsoft.azure/autorest.typescript     2.0.683      C:\Users\x.autorest<EMAIL_ADDRESS>

Can you try running autorest --reset and then regenerating? We have a newer version of the TypeScript generator out and it would be nice if telling autorest to refresh its generator cache fixes this problem.

Have you changed anything? I get the same error after doing an autorest --reset ...
Versions below:
AutoRest code generation utility [version: 2.0.4283; node: v10.13.0]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
Showing All Installed Extensions
Type       Extension Name                           Version      Location
core       @microsoft.azure/autorest-core           2.0.4289     C:\Users\philipwilford.autorest<EMAIL_ADDRESS>extension  @microsoft.azure/autorest.csharp         2.3.82       C:\Users\philipwilford.autorest<EMAIL_ADDRESS>extension  @microsoft.azure/autorest.modeler        2.3.51       C:\Users\philipwilford.autorest<EMAIL_ADDRESS>extension  @microsoft.azure/autorest.modeler        2.3.55       C:\Users\philipwilford.autorest<EMAIL_ADDRESS>extension  @microsoft.azure/autorest.typescript     2.0.683      C:\Users\philipwilford.autorest<EMAIL_ADDRESS>

Too bad that those options didn't solve the problem.
How're you referencing your generated client in your application? Our generated clients are intended to be standalone packages that are imported by your applications. If you generate the client using --generate-metadata=true then it will generate several files that help in creating a package for your generated client. One of those generated metadata files is a rollup.config.js file (such as this one) and a package.json file (such as this one). In the package.json file you'll see that the main property is set to dist/<your-package-name>.js. This is the browser-ready build that is created by your rollup configuration file. In the browser scenario, we assume that you have already imported the ms-rest-js package as shown here.
Since I haven't seen how you're referencing your generated client or ms-rest-js from your actual code files, I can't know for sure what you need to do, but I'm hoping that adding --generate-metadata=true to your generation step might provide a few extra clues.

I'm going to assume that my previous instructions were enough to help you out. Feel free to reopen this issue if the problem persists and you're still stuck.

Hi there,
I still get the same error and cannot see any info in those extra files it generates.
autorest --typescript --use=<EMAIL_ADDRESS>--generate-metadata=true --input-file=http://localhost:61817/SraWebApiCore/swagger/v1/swagger.json --output-folder=gentypescript --add-credentials --package-name="sra" --package-version="1.0" --license-header=MICROSOFT_MIT_NO_VERSION
rollup.config.js
import rollup from "rollup";
import nodeResolve from "rollup-plugin-node-resolve";
import sourcemaps from "rollup-plugin-sourcemaps";
/**

@type {rollup.RollupFileOptions}
/
const config = {
input: "./esm/sraAPI.js",
external: [
"@azure/ms-rest-js",
"@azure/ms-rest-azure-js"
],
output: {
file: "./dist/sra.js",
format: "umd",
name: "Sra",
sourcemap: true,
globals: {
"@azure/ms-rest-js": "msRest",
"@azure/ms-rest-azure-js": "msRestAzure"
},
banner: `/
Copyright (c) Microsoft Corporation. All rights reserved.
Licensed under the MIT License. See License.txt in the project root for license information.

Code generated by Microsoft (R) AutoRest Code Generator.
Changes may cause incorrect behavior and will be lost if the code is regenerated.
*/`
},
plugins: [
nodeResolve({ module: true }),
sourcemaps()
]
};

export default config;
package.json
{
"name": "sra",
"author": "Microsoft Corporation",
"description": "SraAPI Library with typescript type definitions for node.js and browser.",
"version": "1.0",
"dependencies": {
"@azure/ms-rest-js": "^1.1.0",
"tslib": "^1.9.3"
},
"keywords": [
"node",
"azure",
"typescript",
"browser",
"isomorphic"
],
"license": "MIT",
"main": "./dist/sra.js",
"module": "./esm/sraAPI.js",
"types": "./esm/sraAPI.d.ts",
"devDependencies": {
"typescript": "^3.1.1",
"rollup": "^0.66.2",
"rollup-plugin-node-resolve": "^3.4.0",
"rollup-plugin-sourcemaps": "^0.4.2",
"uglify-js": "^3.4.9"
},
"homepage": "https://github.com/azure/azure-sdk-for-js",
"repository": {
"type": "git",
"url": "https://github.com/azure/azure-sdk-for-js.git"
},
"bugs": {
"url": "https://github.com/azure/azure-sdk-for-js/issues"
},
"files": [
"dist//*.js",
"dist//.js.map",
"dist/**/.d.ts",
"dist//*.d.ts.map",
"esm//.js",
"esm/**/.js.map",
"esm//*.d.ts",
"esm//.d.ts.map",
"lib/**/.ts",
"rollup.config.js",
"tsconfig.json"
],
"scripts": {
"build": "tsc && rollup -c rollup.config.js && npm run minify",
"minify": "uglifyjs -c -m --comments --source-map "content='./dist/sra.js.map'" -o ./dist/sra.min.js ./dist/sra.js",
"prepack": "npm install && npm run build"
},
"sideEffects": false
}
In my package.json file for my project
{
"name": "sraweb",
"version": "1.0.0",
"license": "MIT",
"scripts": {
"ng": "ng",
"start": "ng serve",
"serve": "npm run start -- --port 60518 --base-href=/SraWebCore/",
"build": "ng build",
"build:production": "npm run build -- --prod",
"build:ssr": "npm run build --app=ssr --output-hashing=media",
"test": "ng test",
"lint": "ng lint",
"e2e": "ng e2e"
},
"private": true,
"dependencies": {
"@angular/animations": "^7.0.3",
"@angular/common": "^7.0.3",
"@angular/compiler": "^7.0.3",
"@angular/core": "^7.0.3",
"@angular/forms": "^7.0.3",
"@angular/http": "^7.0.3",
"@angular/platform-browser": "^7.0.3",
"@angular/platform-browser-dynamic": "^7.0.3",
"@angular/platform-server": "^7.0.3",
"@angular/router": "^7.0.3",
"@fortawesome/fontawesome-free": "^5.5.0",
"@ng-bootstrap/ng-bootstrap": "^4.0.0",
"@nguniversal/module-map-ngfactory-loader": "^7.0.2",
"@okta/okta-angular": "^1.0.7",
"angular": "^1.7.5",
"aspnet-prerendering": "^3.0.1",
"babel-polyfill": "^6.26.0",
"bootstrap": "^4.1.3",
"core-js": "^2.5.7",
"guid-typescript": "^1.0.9",
"lodash": "^4.17.11",
"@azure/ms-rest-js": "^1.1.0",
"@azure/ms-rest-azure-js": "^1.1.0",
"ngx-bootstrap": "^3.1.1",
"oidc-client": "^1.5.4",
"rxjs": "^6.3.3",
"stacktrace-js": "^2.0.0",
"zone.js": "^0.8.26"
},
"devDependencies": {
"@angular-devkit/build-angular": "^0.10.5",
"@angular/cli": "^7.0.5",
"@angular/compiler-cli": "^7.0.3",
"@angular/language-service": "^7.0.3",
"@types/jasmine": "^2.8.11",
"@types/jasminewd2": "^2.0.6",
"@types/node": "^10.12.7",
"codelyzer": "^4.5.0",
"jasmine-core": "~3.3.0",
"jasmine-spec-reporter": "~4.2.1",
"karma": "^3.1.1",
"karma-chrome-launcher": "~2.2.0",
"karma-coverage-istanbul-reporter": "^2.0.4",
"karma-jasmine": "^1.1.2",
"karma-jasmine-html-reporter": "^1.4.0",
"protractor": "~5.4.1",
"ts-node": "~7.0.1",
"tslint": "~5.11.0",
"typescript": "^3.1.6"
}
}
And this is my import statement
import { TokenCredentials } from '@azure/ms-rest-js';

I believe you will have to use the browser bundle. Take a look at this sample which includes the msRestBundle.js file and then instantiates the token credentials off it.
I believe import { TokenCredentials } from '@azure/ms-rest-js'; would give you the node.js version.

os cannot be resolved because os is a node.js standard module. How will that be available in the browser?

os is defined as external in rollup.config.js. By following the explanation provided in the rollup documentation over here, we may have to use this plugin for import * as os from "os" to work in node and browser.

@adha-admin - One more interesting thing to note:

What the error relates to is the fact that the "os" cannot be resolved.
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'os' in 'D:\x\git\Projects\sra\src\sra.web\ClientApp\node_modules@azure\ms-rest-js\es\lib\policies'

You are using this with Angular that is supposed to be used in the browser. However the code is using the file (msRestUserAgentPolicy.js) that should be used for node.js environment rather than the browser one msRestUserAgentPolicy.browser.js.
@daschult
I think, this line in package.json
is the culprit.
"./es/lib/policies/telemetry/msRestUserAgentPolicy.js": "./es/lib/policies/telemetry/msRestUserAgentPolicy.browser.js"

It is pointing to a file that does not exist at that location. It should rather be something like this
remove the telemetry folder from the path for both the files.
"./es/lib/policies/msRestUserAgentPolicy.js": "./es/lib/policies/msRestUserAgentPolicy.browser.js"

@adha-admin - Can you make the above change in<EMAIL_ADDRESS>file and try again? Let us know if that solves the problem

GREAT !
That change in the package.json file as fixed the issue.
"./es/lib/policies/telemetry/msRestUserAgentPolicy.js": "./es/lib/policies/telemetry/msRestUserAgentPolicy.browser.js"
to
"./es/lib/policies/msRestUserAgentPolicy.js": "./es/lib/policies/msRestUserAgentPolicy.browser.js"
I presume this will go in the next release?

Yes, it will. Thanks @amarzavery for catching this.

Thanks guys,
Please let me know when you have published this fix and what the version is.

@adha-admin - New version (1.2.1) of @azure/ms-rest-js has been published to npm.


⚠ Bofrost.lu has degraded performance

In dab31ae, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1504 ms

Resolved: Bofrost.lu performance has improved in e7e3126 after 1 hour, 11 minutes.


Broken sql format with many long column names

Describe the bug
The sql is broken after formatting.
To Reproduce
The code below:
with
    cte1 as (select * from {{ ref("model1") }}),
    cte2 as (select * from {{ ref("model2") }}),
    cte3 as (
        select
            cte2.*,
            this_column
            - that_super_long_long_column_name
             as super_long_column_name,
            another_long_column_name,
            this_column,
            yet_another_column
        from cte2
        left join cte1 using (project_id)
    )
select *
from cte3

Once formatted turns into this:
with
    cte1 as (select * from {{ ref("model1") }}),
    cte2 as (select * from {{ ref("model2") }}),
    cte3 as (
        select
            cte2.*,
            this_column - that_super_long_long_column_name as super_long_column_name,
            another_long_column_name,
            this_column,
            yet_another_column
        left join cte1 using (project_id)
    )
        from cte2
select *
from cte3

The from cte2 statement is removed from the cte3 definition.
Additional context
sqlfmt, version 0.19.0

See my comment on your other issue. I'm almost certain it's not my software doing this.

See https://github.com/tconbeer/sqlfmt/issues/476


ProxyConnector doesn't always set correct request line

I tried using aiohttp with a HTTP proxy and ran into several problems. Most proxies simply broke and told me the client made a malformed request, some other notified me that I need to enable transparent proxy support.
I think this is because the correct request line is only set when the first connection to the proxy is made.
The relevant code to update the request line for the proxy should be here: https://github.com/KeepSafe/aiohttp/blob/master/aiohttp/connector.py#L523
_create_connection seems to be called in BaseConnector.connect, which is only used in ClientSession.request, if I understand the code correctly.
If you look at the source of BaseConnector.connect you can see that _create_connection is only called as long as there is no existing connection - thus request that reuse an existing connection don't get an updated request line and will fail, as long as the HTTP proxy doesn't have transparent proxy support.
This also happens when reusing the same ProxyConnector for multiple requests, even if not using a Session object, since the connection to the proxy is stored in the Connector.

Hmm. Would you try ProxyConnector(proxy_addr, force_close=True)?
I'm inclined to use force_close=True as default value for ProxyConnector.

Seems to work... using a request session like this:
PROXY_CONNECTOR = aiohttp.ProxyConnector(proxy="http://<IP_ADDRESS>:12345", force_close=True)
SESSION = aiohttp.ClientSession(connector=PROXY_CONNECTOR)

SESSION.get(...)

Fixed in 792b980 -- force_close=True by default for ProxyConnector


does it works with graphite

Hi,
I would like to make an histrogram over data stored into graphite.
the panel remains white. In the upper left corner there is  a red triangle.
It says
"undefined is not an object(evaluating '$q.when')"
any hint
thanks
Ale

What version Grafana use?
Do you use package? Or build from source?
This panel should work with Graphite or other time series data sources.
But I only use this with Prometheus, so I don't test the panel well...

Grafana 3.0.0 from build
this is the time serie I would like to make an histogram for

but the same serie with histogram does not work

is it correct how I'm trying to use it ?
thanks in advance
Ale

Yes, it is correct.
I also use Grafana, build from source.
I specify the path of "dist", not "src" in config file.
I use Mac and Google Chrome.
If you tell me the commit hash of your Grafana build, I'll try to replicate :-)

Hi, I try to replicate issue by using Graphite in my env.
I don't install Graphite locally, so I use play.grafana.org site Graphite by Grafana proxy mode.

(specify "http://play.grafana.org/api/datasources/proxy/1/")
and then, I set query like this.

The histogram panel works.
The git commit hash of Grafana is https://github.com/grafana/grafana/commit/7bc15ec6c18193f1e6ea3a71437812aa6ee7ec20 .

first of all thanks for answering.
tha same source does not work for me
root@grafana:~# dpkg -l | grep grafana
ii  grafana                        3.0.1                     amd64        Grafana
root@grafana:~#
I suppose tha plugin is corrected loaded.
How could I debug this issue ?

Your environment might be Ubuntu.
And the Grafana is installed by deb package. (download from http://grafana.org/download/builds ?)
Currently I don't have idea how you can resolve issue.
You use Ubuntu and you use deb package, I use Mac and I use local build Grafana, these are difference of our environment.
I'll try to replicate your issue on Ubuntu with package installed Grafana.
I also get bug report of this panel, they also use package version. https://github.com/mtanda/grafana-histogram-panel/issues/7
There are difference between package and local build.
I'm not sure but the difference could cause your reported error.
If you have experience of building from source, please try it.
It's great helpful :-)
http://docs.grafana.org/project/building_from_source/

The error message might be caused by this change https://github.com/mtanda/grafana-histogram-panel/issues/7 .
It is only place, I use "$q.when".
I don't test this change with package installed Grafana, I test only Homebrew installed Grafana.
I should check package installed Grafana.
Sorry for inconvenience.

@aleita I replicate your issue on Ubuntu.
And I fix it by this  PR. Please renew panel source and test it.
https://github.com/mtanda/grafana-histogram-panel/pull/15

Yesterday I was able to create some histogram
but today I'm facing a new issue.
When I add panel, selecting Histogram, It seems to work.

Reload means "refresh button of Grafana" or "browser reload button"?

I have a similar problem.In my setup I have two dashboards:
Dashboard 1: with two histograms

Dashboard 2: with a graph panel and an histogram panel

It seems all right, but if I open a new browser and I load first Dashboard 2 the result is:

As you can see there's an empty histogram and no errors in my browser console. Now if I navigate to Dashboard 1 histograms don't appear also here. After this moment no other histogram will be loaded until I force a page reload.
It seems that your plugin needs to be loaded first and if not it stop working.
I hope this can be helpful.
Thanks

I'm encountering this issue on both cases "refresh button of Grafana" and "browser reload button".

@Bellovitello Thanks for sharing reproduce step!
I succeed to replicate the issue.

@aleita I got it. I'll investigate the bug.

When I change base class to MetricsPanelCtrl, panel is appeard.
Finally, I give up to reuse GraphCtrl code.
https://github.com/mtanda/grafana-histogram-panel/pull/16

it works for me

Thanks for testing 👍
I close this issue.


Fixing a typo (-backend)

ozp-center, ozp-backend, and ozp-react-commons all have this typo and a PR to match. (Civilian vs Civillian)
@mannyrivera2010 @lsemesky @mleeBoeing @J-Fid

:+1:


[PackageTool] Add package edit subtool

We should have test coverage for the tool behaviors.

functional?

I think so, yeah? Doesn't have to try to do everything, but I think it should vet the behaviors.

@swift-ci please test

@swift-ci please test


[NFC] Delete unused static methods.

Just found that there are unused methods when fixing mhlo.conv verifier tests.


Chinese

Write a comment with the translation or make a PR after #72 is merged to the dev branch, please. Thanks 😄 🍺
Search

Status

All Languages

Reset

Settings / Donate

Status

Donate to CodeHub on Patreon

Settings / General

Status

Make any donation to enable the option to disable ads.

Settings / About

Status

What's new in this version?

Developed by Aalok Singh

Time Ago converter

Status

one second ago

seconds ago

a minute ago

minutes ago

an hour ago

hours ago

a day ago

days ago

one month ago

months ago

one year ago

years ago

Hi, can I work on this?

Of course.


group privileges not picked up by v_generate_user_grant_revoke_ddl

I'm trying to drop a group from my redshift cluster. Using the latest version of v_generate_user_grant_revoke_ddl, I'm running the following query to get the DDL statements to run for the group privileges:
select ddl from admin.v_generate_user_grant_revoke_ddl where ddltype='revoke' and username='<groupname>' order by grantseq;

This is the statement documented at https://aws.amazon.com/premiumsupport/knowledge-center/redshift-user-cannot-be-dropped/ except that the sequence number column has been renamed since the article was written (NB: I've also provided feedback on the page saying it's out of date and needs to be updated to reflect the column name changes).
The query returns an empty result set, which leads me to believe I'm clear to drop the group, however when I do so, I get the following error:
SQL Error [500310] [55006]: [Amazon](500310) Invalid operation: group "<groupname>" cannot be dropped because the group has a privilege on some object;

I'm gonna have to dig into the view definition and see if I can figure out where the missing privileges are. I will add comments to this ticket with my findings.

I am not able to delete  it still getting the error like
SQL Error [55006]: ERROR: group "buying_alias" cannot be dropped because permission dependency is found

Getting this same exact problem for a group we have. I have tried to do all these steps above and also
ALTER DEFAULT PRIVILEGES IN SCHEMA <schema>
    REVOKE ALL ON TABLES FROM GROUP <group_i_want_to_drop>;
ALTER DEFAULT PRIVILEGES IN SCHEMA <schema>
    REVOKE ALL ON FUNCTIONS FROM group <group_i_want_to_drop>;

All of the admin views return empty and SQLs @sworisbreathing posted. Still getting
SQL Error [55006]: ERROR: group "<group_i_want_to_drop>" cannot be dropped because permission dependency is found


Youtube Integration with VS Code

While learning lot of JavaScript libraries or frameworks, I use YouTube  coding tutorials. But opening YouTube in browser and working with tutorial in VS code is difficult. If VS code can have integrated youtube same as integrated terminal, it will be great help.

An integrated browser has been a common request but I don't think it fits in to VS Code's general way of doing things. I suggest you arrange your Code and browser windows so you can see both 😁


[BUG]Incorrect launguage will not have prompt in english screenshots will explain

Describe the bug
This is the lanugage I am getting with screenshot

Version:
Newest
Tool:
I am using stable swarm/comfy add on inside it
To Reproduce
I rebuild re scan all images show but the language different this was an awesome tool.
Expected behavior
I would like to have english for easy access copy and paste you know
Affected Image(s)
Just when open everything else is english
Screenshots

Additional context
please let me know any data you need

Can you post a the sample image? This is not related to translations. It's probably bad/unsupported metadata

I have this same issue using Stable Swarm. SD Prompt Reader can read the meta prompt data but for some reason Diffusion Toolkit shows in in Chinese> Every setting I can find is English (windows 11, Diffusion Toolkit setting)

This is related to #250 which is caused by a third party library. Stable Swarm support will be added soon.


fix(sshtunnel): argument params to properly setting server_port

SUMMARY

Using configuration for tuple to set service_host + server_port. Realized there was an upstream issue with the lib properly setting the port before establishing the connection. Will submit upstream fix later but want to unblock superset users being able to set the port properly.
with sshtunnel.open_tunnel(
    ssh_address_or_host=(SERVER, SERVER_PORT),
    ssh_username=SSH_USER,
    ssh_pkey=private_key,
    remote_bind_address=(remote_host, remote_port),
    local_bind_address=("<IP_ADDRESS>",),
    # debug_level='TRACE',
) as server:
   # establish connection...

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

🏷 preset:2023.7


Getting "value" property of select element.

The select element behaves differently to input, as they don't have their own value attribute - instead, the selected option element's value should be returned.
Setting the value of a select element should set the selected attribute to the matching option, if one matches.
Potential pitfall: when a select element has no "selected" option, the value should default to the first option.

Kindly implemented by @ognjen-petrovic
Thanks, closing this issue now.


feat(app): App robot settings other tabs robot is busy

Overview
This will fix the rest of #10322  (Networking tab & Calibration tab)
Details
https://github.com/Opentrons/opentrons/pull/10368

Changelog

Added functionality that checks whether a robot is busy or not to RobotSettings - Networking tab
Added functionality that checks whether a robot is busy or not to RobotSettings - Calibration tab
Added test cases to each tab

Review requests

Risk assessment
low

[ ] solve conflicts

[ ] merge #10368 into edge
[ ] change the merge direction to edge and merge


How to pass table as vector to a c++ function?

I have a C++ function looks like this:
void foo(const std::vector<int>& arr) { ... }
I want to call it from lua script:
foo({1, 2})
But I get error:

stack index 1, expected userdata, received table: value is not a valid userdata

If you ask for a reference -- const or non-const -- sol2 expects the data to be an actual, real std::vector type. If you want sol2 to automatically convert, take a std::vector<int> arr (notice no reference: by value!).

Otherwise, you need to make a wrapper function and bind that to sol2.

I just remove the reference, but still have the same error
void foo(std::vector<int> arr) { ... }

Thank you!


memo

Auto function memo-iser

Hi i couldnt see any repo-specific contribution guidelines, would it be okay if i worked on a PR adding this?

If you'd like to! :) I can always clean up things to make it fit in with the rest of the library afterwards!
Btw, I'm planning to do a re-organise of this repo soon anyway, so I can port over anything you do to the new version (if you do want to go ahead)

Opened #57

Done: #57


miniziti still has a ziti binary dependency

There is a single remaining ziti edge command in the miniziti.bash script. Replacing this line with ziti_wrapper() invocations completely eliminates any dependency of having ziti on the host.

Not sure if same invocation of ziti CLI, but miniziti.bash login gets:
+ minikube kubectl --profile miniziti -- --context miniziti get secrets ziti-controller-admin-secret --namespace miniziti --output 'go-template={{index .data "admin-password" | base64decode }}'
+ xargs ziti edge login --cli-identity miniziti --yes --username admin --password
Enter controller host[:port] (default localhost:1280): error: EOF

Yes, correct. This happens because when we moved the ziti binary to the container, we never login on the host. We don't currently store enough state to do this after the first run, unless we inspect the coredns configmap to find the ingress zone. I wanted to discuss this with you this morning, and was part of why I was trying to address a few other issues, but decided to wait to a more opportune time. 90/90 rule, sadly.

To be clear, there is another ziti edge command in the start path of the stack.


pub publish on a plugin has warning

While pub publishing a plugin I was given the following warning:
Suggestions:
* Don't depend on "flutter" from the sdk source. Use the hosted source instead. For example:

  dependencies:
    flutter: ^0.0.20

  Using the hosted source ensures that everyone can download your package's dependencies along with your package.

Package has 1 warning. Upload anyway (y/n)?

The offending line was generated by flutter create --plugin. Should I follow the suggestion? If yes, should we update what flutter create --plugin generates? Can we document somewhere what to do?
/cc @mit-mit

cc @szakarias @jakobr-google

The warning is wrong. The "hosted" flutter package is just a very empty placeholder, and using sdk is correct. We should make sure pub doesn't emit this warning for flutter.
Which version of pub did you use?

We are also seeing this with <flutter sdk dir>/bin/cache/dart-sdk/bin/pub. We probably need to "teach" pub about this special case.

@kasperl @kevmoo any idea who might be able to help with this?

Looks like this warning is gone? At least I didn't see it when I just updated a bunch of plugins on pub.

@kevmoo did anything change in pub?

Yes, I think this is fixed. I tried with a fresh plugin and no warnings anymore. Closing.


Issue #1335 Fix

Small changes to fix Issue #1335. State of fresh counters problem solved problem solved. Do not review yet.

Jenkins: Test this please!

Jenkins: test this please

@dwightguth please review

looks good. I'm going to merge this. Eventually we will want to remake the Executor interface over KORE, although it may be slightly different by then. That will be when we remove these extra methods, which I've decided aren't that nice. But it's fine for now as a fix for Traian's problem.


AngularJS?

Currently ngrx is ng2 only. There were some community efforts to bring it to ng1, but nothing official.

Wow that fast. :)
Yeah, I really like Angular 2, but I accepted a "challenge" to build something in AngularJS. I scaffolded out "TodoMVC" with no Redux design and it seems like a big cluttered mess. :/

You can do regular Redux itself in AngularJS, or anything for that matter. It's really nothing but an event store. NgRx on the other hand is a whole thing, that's woven around ng2, RxJs, etc, so it would be too much to move it to ng1 I'd imagine. :/

There is an Angular bindings for Redux called ng-redux - https://github.com/angular-redux/ng-redux. It makes bridges between angular app and regular redux.


Add support for custom methods for connecting to Gremlin

We currently call Gremlin Server without any authentication headers. While this gets the job done, it would be nice to support some form of authentication method in case users deploy the application to a managed service (e.g. Cloud Run) or want to integrate the API with a hosted Gremlin-compatible database such as AWS Neptune.
One option for this would be to abstract out logic for generating Gremlin DriverRemoteConnection objects and allow users to specify these when starting the API server. (e.g. by specifying a name with a flag like --connection-factory my.custom.GremlinConnectionFactory or something similar).

In the end, we keep the gremlin server safe behind some network rules in the VPC, and the API should be the only one connecting.


Make the hotkeys work only when the game window is focused.

Make the hotkeys work only when the game window is focused.

This would require hook to win32 API which is unavailable from node.js land(probably could be done with node-ffi-napi), and I have no idea as of now how to replicate that on Linux. But overall interesting idea, that I might revisit at some point in the future.


double-beep: Watch frozen\W*four

double-beep requests the watch of the watch_keyword frozen\W*four. See the MS search here and the Stack Exchange search here.

frozen\W*four has been seen in 1 true positive, 0 false positives, and 0 NAAs.

I'm going to decline this, as I don't think it covers many of the cases which should be caught.


Fix issue with frost-tabs

This project uses semver, please check the scope of this pr:

[x] #patch# - backwards-compatible bug fix
[ ] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Add icons
Fix issue with frost-tabs

Visual Acceptance tests:
No changes

Approved

Also curious, what does the title of this review have to do with the content?

Yep I got it for the icons my bad.

Approved


Improve Bramble extension loading via URL params, add extra/ dir

Travis needs to be fixed by adding these new modules to this section of the gruntfile: https://github.com/humphd/brackets/blob/bramble/Gruntfile.js#L362-L365

@sedge can you re-review?  I've fixed some other stuff in the grunt build process, too.  Tested locally in src/ and dist/ and all is well.  You can test with a url like ?enableExtensions=CodeFolding,brackets-cdn-suggestions,JSLint&disableExtensions=HTMLCodeHints,UrlCodeHints

r+

r+, very good patch


Vigenère Cipher Implementation in C++

🛠 Issue (Number)

Issue no #516
✔ Check List

My code follows the code style of this project.
This PR does not contain plagiarized content.
The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
Definition
👉 In cryptography, the Vigenère Cipher is a method of encrypting alphabetic text by using a series of interwoven Caesar Cipher.
Steps
⇒ Convert all uppercase character into lowercase in the plain text.
⇒ Check if plain text has no intergral characters.
⇒ Input cipher key.
⇒ No. all characters from a - z as 0 - 25.
⇒ Find cipher text by : (characterNum + keyNum) mod 26.
, where characterNum and keyNum are the no.s associated with plain text and cipher key characters.
📷 Screenshots

@Kumar-laxmi @072arushi please assign me this in java and C

@Kumar-laxmi @072arushi please assign me this in java and C

This is a PR opened by someone else


WIP : this is my dev branch if you want to discuss what I'm working on

What i've modified / done so far :

FIX : force uppercase in shapes, as otherwise we crash due to some entry being lowercase (they must have not been case sensitive)
FIX : filter element from the UI which targets a -1 id
FEAT : I've implemented full texture caching to improve going back & forth in menu (at the cost of memory)
Changed : Shape's sprite are now loaded all at once to reduce re-reading palette. Anyway we always use all the sprites stored within the shape for animation/display purposes
WIP : 2 types for UI element, one for button one for the rest (this will have to be modified later on but it's better than just one)
WIP : I'm beggining to do one file per 'scene', currently they are mostly empty and do nothing #48
WIP : I've changed how texture are rendered, this'll most likely break convert tools and other stuff so I'll need to take a look, instead of having full bitmap with alot of void, instead it only renders the part that has content and save the offset to be used for display later on (it reduce memory usage)

Next up :

Make pause menu usable, actually it's the only thing rendered into the island scene, this'll allow us to go back to the main menu
Move hardcoded stuff  (gametitle + stars) back into the MainMenu scene #48
Rendering islands, but no bridge connection or other fancy stuff #48
Rendering animation (this will be painfull as we'll need to parse them and the format seems like hell)
Rendering creation dynamic content (game difficulty + tribe selected, but no text) + Dummy global-state for game parameters
Dummy global state that allows game-creation config to alter rendered islands (world size)

What is there currently looks good 👍

So what does this PR actually does ?

Massively improve shapes loading #49 #30 :

loading a whole shapefile all at once to reduce re-reads of pallets
Uses cache to avoid reading file several time ( ⚠ this will increase memory usage, but since the textures are all low res it should not be an issue, maybe we should avoid caching in-game planet texture later on ? )
Output object are now object with an offset to reduce memory usage (skips alot of transparents pixels, reduce the memory usage of caching)
Rendering has been modified to handle the new offseted objects

Several fixes
Refacto for scenes
WIP : first version of the pause menu (act as a big button, just allows to go back to the main screen)

It's possible extractor tools for shapes are brokken i've not tested them.
Other changes discussed in the first message will be done on another PR otherwise it's gonna be hard for you to look at (too many changes)


Errors should respect source maps & unhandled rejections

Logging is great! I'm just thinking that in any real production application (mobile, web) uncaught errors are usually funneled to some logging platform such as BugSnag etc. If error occurs only in production, sometimes that's the only way to notice an issue. I wonder how this should be done here.

I just found my way here because I was trying to figure out why we don't get a proper sentry.io error from a service that fails in strict mode. Turns out that the error is just logged to console and not propagated to Sentry's unhandled promise rejection handler. While logging is great for backend applications this happens in frontend code where we rely on Sentry error collection as it groups the errors based on callstack and other values.
As a side note the error logging does not respect source maps in Chrome as it's concatenating a string like console.error("Error: " + error). The error should be sent to the logger as an error object to allow Chrome to handle sourcemaps, ie. console.error("Error:", error).
Originally posted by @qtiki in https://github.com/davidkpiano/xstate/issues/463#issuecomment-604947112

Hey @urig-40seas!
We will be working on onError for XState v5 soon, which should solve these kinds of issues.

@Andarist Can this be closed as finished?

We no longer use console.error to report any errors so I think this isn't an issue in v5.


LSNBLDR-455 - When importing from cartridge HTML should be inline rather...

... than linked

I'm going to do this a different way, that won't break imports from other systems.


Provide a RS256 Example with public and private key

Hello,
It would be helpful to provide an example with Jwt with RS256.

Tests are running all RSA algorithms.

Create public and private keys
Encode using the private key
Validate using the public key

You can also generate random keys and check everything works
Is that good enough or would you prefer a more structured example?

It would be awesome to have an example how to use RS256 with jwt-play sessions (is it working?) and maybe also external providers like Auth0. I am currently using HS256 with Auth0 and it works great!
I am willing to contribute some time if you need help.

I made this code to works with auth0 and playframework
`def url(config: Config) = s"https://${config.domain}/.well-known/jwks.json"
/**
*
* @param token a jwt token (RS256) to verify using public key
* @return true if token is verified
*/
def verify(token: String)(implicit config: Config): Future[Either[Unit, SignedJWT]] = {
Try(SignedJWT.parse(token.replace("Bearer ", ""))).fold(
_ => Future.successful(Left(())),
signedJWT => {
val keyId = signedJWT.getHeader.getKeyID
for {
jwtjsString <- loadjwtksJson(config)
} yield {
val publicKeys = JWKSet.parse(jwtjsString.toString())
val rsaPublicKeyOpt = Option(publicKeys.getKeyByKeyId(keyId).asInstanceOf[RSAKey])
val result = rsaPublicKeyOpt.map(rsa =>
new RSASSAVerifier(rsa.toRSAPublicKey)
).exists(signedJWT.verify(_))
Logger.info(s"jwt keyId = $keyId, valid = $result, token=$token")
if (result) Right(signedJWT) else Left(())
}
}
)
}
def getToken(code: String)(implicit req: play.api.mvc.RequestHeader, config: Config): Future[Either[JsValue, Token]] = {
wsclient.url(s"https://${config.domain}/oauth/token")
.withHttpHeaders(HeaderNames.ACCEPT -> MimeTypes.JSON)
.post(Json.obj(
"client_id" -> config.clientId,
"client_secret" -> config.secret,
"redirect_uri" -> config.callbackURL,
"code" -> code,
"grant_type" -> "authorization_code"
))
.flatMap(t => t.status match {
case 403 => Future.successful(Left(t.json))
case 200 => Future.successful(t.json.validate[Token].asEither.fold(
err => Left(JsError.toJson(err)),
t => {
Logger.info(s"access_token = ${t.access_token}")
Logger.info(s"id_token = ${t.id_token}")
Right(t)
}
))
}
)
}
def getUser(accessToken: String)(implicit req: play.api.mvc.RequestHeader, config: Config): Future[Either[JsObject, UserAuth0]] = {
wsclient.url(String.format(s"https://${config.domain}/userinfo"))
.withQueryStringParameters("access_token" -> accessToken)
.get().flatMap(response => response.json.validate[UserAuth0].fold(
errors => Future.successful(Left(JsError.toJson(errors))),
u => Future.successful(Right(u))
))
}`

@dgouyette awesome thank you!
I am currently using a custom ActionBuilder to authenticate the user:
request.jwtSession.claimData.asOpt[JWTToken] match {
        case Some(token) =>Future.successful(Ok)  // Valid
        case _ => Future.successful(Unauthorized) // Invalid
}

It would be nice if JwtPlayImplicits would support RS256 out of the box.

Thanks for sharing @dgouyette
PR is welcome to support RS256. You would need to check if algorithm is symmetric or not at bootstrap and depending on that use one (HMAC) or two keys (RSASSA or ECDSA, public and private).


[FLINK-12485][python] Add completeness test for Table and TableEnviro…

What is the purpose of the change
After FLINK-12407 Python Table API will align with current Java Table API. So, In the following Table API development, it's better to add a new user interface to the Java Table API will also need to be added to the Python Table API. So In this Jira, we want to add the test_completeness to check the user interface of Python Table API aligns with Java Table API.
I'm not sure if we want to force contributors to contribute to the Python Table API while developing the Java Table API. But at least we should have the ability to automatically check if Python is aligned with Java functionality.
Brief change log

Add PythonAPICompletenessTestCase which is base class for Python API completeness tests.
Add TableAPICompletenessTests which test whether the Python Table is consistent with Java org.apache.flink.table.api.Table.
Add EnvironmentAPICompletenessTests which test whether the Python TableEnvironment is consistent with Java org.apache.flink.table.api.TableEnvironment.

Verifying this change
This change is only adding the test case.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature?  (no)
If yes, how is the feature documented? (not documented)

Thanks for the quick review! @dianfu
I have update the PR according your comments!
Best,
Jincheng

Merging...
@flinkbot approve all

Will be merged after CI turn green.
@flinkbot approve all

@sunjincheng121 There is a very high-frequency test failure in table module may cause the Travis to fail. I have reported it, the Jira ID is FLINK-12512. I also found it caused your PR to fail. Can you try to fix it? thanks.

@sunjincheng121 There is a very high-frequency test failure in table module may cause the Travis to fail. I have reported it, the Jira ID is FLINK-12512. I also found it caused your PR to fail. Can you try to fix it? thanks.

We know the fail reason, Python API add the code style check: you can see the error log  ./pyflink/testing/test_case_utils.py:164:101: E501 line too long (102 > 100 characters)
And it already fixed in my last commit in this PR. :)  but thanks for your kind mention that.

@sunjincheng121 I mean TableSourceTest  test error. It also caused your PR's failure, you can see here : https://travis-ci.org/apache/flink/jobs/532617757

Do you think this problem is caused by this PR? Otherwise, we can discuss it in the JIRA you created, what do you think?

@sunjincheng121 No, I am sorry to comment these unrelated messages in this PR. I just saw it also affected your PR and you guys focus on Table modules may fix it as soon as possible. Because it has broken my PRs many time recently.

So, can we merge this PR first, and then talk the test issue on the JIRA page? @yanghua

@sunjincheng121 Absolutely, it's OK.


Update README.md

Description
Make the README look nicer
Type of change

[x] New feature (non-breaking change which adds functionality)

I have read the CLA Document and I hereby sign the CLA

@andrewstech fixed


When someone votes everyone gets keys

I need more basic information, like what server you were on.

Found it, the issue is the crate command itself. Passed on to @HackuJacku

Fixed on our end, also forwarded to the developer :)


Is it possible to change cp1252 to utf-8?

Hi, I'd like to know if is it possible to change cp1252 to utf-8 with pdf.UnicodeTranslatorFromDescriptor("utf-8"), when I type that it appears open utf-8.map: The system cannot find the file specified.
The problem is I want to write those unicodes  ☑  ☐  inside string and generate pdf then I doesn't show.
Thanks for your time.

The problem you'll face is that gofpdf works only with code pages, that is, fixed size tables that contain at most 256 glyphs. You can manually create your own code page with the glyphs you select from the full unicode pool; see the documentation for UnicodeTranslator and the makefont utility (in subdirectory makefont) for more details. Even then, with some of these new utf-8 glyphs you'll be in uncharted territory.
For checkboxes and common symbols, you may also want to check out Zapf Dingbats. These are included without any special preparation.


Vulkan support

This revision supports Vulkan.

This feature will be done in CubbyFlow-v0 and Jet Framework.


Verify slqlite ledgers exist in node store

This only checks if the header is present in the node store compared to ledger_cleaner which would fetch the whole SHAMap tries, right?

Jenkins Build Summary
Built from this commit
Built at 20180313 - 16:04:31
Test Results

Build Type
Log
Result
Status

docs
logfile
1 cases, 0 failed, t: 2s
PASS :white_check_mark:

clang.debug.unity
logfile
1003 cases, 0 failed, t: 243s
PASS :white_check_mark:

msvc.debug
logfile
999 cases, 0 failed, t: 547s
PASS :white_check_mark:

coverage
logfile
1003 cases, 0 failed, t: 602s
PASS :white_check_mark:

clang.debug.unity, PARALLEL_TESTS=false
logfile
1003 cases, 0 failed, t: 467s
PASS :white_check_mark:

clang.debug.nounity
logfile
1001 cases, 0 failed, t: 300s
PASS :white_check_mark:

gcc.debug.unity
logfile
1003 cases, 0 failed, t: 236s
PASS :white_check_mark:

gcc.debug.nounity
logfile
1001 cases, 0 failed, t: 368s
PASS :white_check_mark:

clang.release.unity
logfile
1002 cases, 0 failed, t: 322s
PASS :white_check_mark:

gcc.debug.unity -Dstatic=true
logfile
1003 cases, 0 failed, t: 235s
PASS :white_check_mark:

gcc.release.unity
logfile
1002 cases, 0 failed, t: 340s
PASS :white_check_mark:

msvc.debug, PROJECT_NAME=rippled_classic
logfile
999 cases, 0 failed, t: 1128s
PASS :white_check_mark:

msvc.release
logfile
998 cases, 0 failed, t: 492s
PASS :white_check_mark:

Codecov Report

Merging #2430 into develop will decrease coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           develop   #2430      +/-   ##
==========================================
- Coverage    70.32%   70.3%   -0.03%
==========================================
  Files          702     702
  Lines        53429   53435       +6
==========================================
- Hits         37573   37566       -7
- Misses       15856   15869      +13

Impacted Files
Coverage Δ

src/ripple/app/ledger/impl/LedgerMaster.cpp
45.54% <0%> (-0.31%)
:arrow_down:

src/ripple/server/impl/BaseHTTPPeer.h
87.03% <0%> (-3.09%)
:arrow_down:

src/ripple/protocol/impl/STVar.cpp
85.71% <0%> (-2.6%)
:arrow_down:

src/ripple/protocol/impl/Serializer.cpp
70.06% <0%> (-0.35%)
:arrow_down:

src/ripple/app/misc/SHAMapStoreImp.cpp
76.02% <0%> (+0.24%)
: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 8648440...7f15861. Read the comment docs.

@MarkusTeufelberger Correct. This additional check detects when the node store in out of sync with sqlite, making complete_ledgers more accurate. The desync usually occurs when a user deletes their node store and forgets to clear the sqlite DB.

@ximinez I don't believe so as this is called after a network sync and backfilling begins.

@miguelportilla That's what I was thinking, too. Oh well.

In 1.0.0-b3 as 4869a0d00e4c5854b1760ca89ee93b1a7c79b8f1


oe-gdb: oe-gdb script needs to be relocatable to any OE install location

Currently the oe-gdb script is generated via a configure_file command that uses the installation directory specified during cmake-configure time.
cmake -DCMAKE_INSTALL_PREFIX=foo
However  installation can also be done via the following syntax:
make install DESTDIR=bar
The above install would not work since the oe-gdb script has been configured to work at foo.

CC: @andschwa @CodeMonkeyLeet @johnkord

Resolved by #1591.


Update buf plugins, cockroach with massive performance gain, other deps

Cockroach v22.2 is now even with Postgres. This was not the case with v22.1 and below.

Codecov Report

Merging #95 (e5f69f6) into master (1d56df4) will decrease coverage by 0.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
- Coverage   70.93%   70.75%   -0.18%
==========================================
  Files          10       10
  Lines        1118     1118
==========================================
- Hits          793      791       -2
- Misses        206      207       +1
- Partials      119      120       +1

Impacted Files
Coverage Δ

memory.go
90.00% <0.00%> (-3.34%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Adding support for ISE 3.3 release

Description of changes:
Adding support for ISE 3.3 release. Making 3.2 as default version.

/do-e2e-tests


Multi user multi site

resolves #462 #463

Can be tested on https://opev-webgen-dev.herokuapp.com

[Current coverage][cc-pull] is 30.74%

Merging [#464][cc-pull] into [master][cc-base-branch] will decrease coverage by 1.18%

@@             master       #464   diff @@
==========================================
  Files             3          3
  Lines           332        348    +16
  Methods          34         34
  Messages          0          0
  Branches         42         42
==========================================
+ Hits            106        107     +1
- Misses          226        241    +15
  Partials          0          0

Powered by Codecov. Last updated by [6e357c2...014a413][cc-compare]
[cc-base-branch]: https://codecov.io/gh/fossasia/open-event-webapp/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/fossasia/open-event-webapp/compare/6e357c25076085cdc39529882c3578167b298705...014a413255929c8eb664c0ad794daf548d91d461?src=pr
[cc-pull]: https://codecov.io/gh/fossasia/open-event-webapp/pull/464?src=pr


UDFs should have unique identifiers

Description
Currently you can only apply a single udf to a method and can only apply a single named udf to a column. This seems like it will present scaling issues in the future.
As a user I'd like to apply multiple udfs to the same method to apply to different columns without needing to replicate the method over and creating code bloat.

[x] I have reviewed the Guidelines for Contributing and the Code of Conduct.

reopening as this is still relevant


Refactor offline mode in package resolution

Purpose

Refactor offline mode in package resolution
Prerequisite for #30757

Approach
Removed the usage of the system property ballerina.offline.flag to enable offline mode. Instead, the offline flag is set to ResolutionRequest. At the package resolution phase, this flag is retrieved from project.buildOptions()
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@hevayo ack


346 duckdb for benchmarking

Unfortunately a very large PR, however, it is for a much-needed and positive upgrade to the benchmarking code.
Before:

Lots and lots of opening and closing of phenopacket files. For each run, the phenopacket file would be opened and the same methods for retrieving the information would be called. This means, that when running the benchmarking on the 4k phenopacket-store for 4 different run configurations, the same batch of phenopackets would be opened, call retrieval methods, and closed ~16k times.
Used dictionaries and lists to store data from the benchmarking - making the code hard to read.
Lots of files output from the command, making it difficult at times to navigate to the data that you want to access.
Limited customisation to plots output from the command.

Now:

A unique corpus of phenopackets is read in once. The data retrieval occurs once. All the data is stored in a single duckdb table for reference. So now for benchmarking the 4k corpus on 4 different run configurations, each operation is completed once.
Eliminated dictionaries and most lists for storing data, now data is taken from a duckdb table.
All outputs are now contained to a single db - apart from the svg plots, making it easier to navigate.
You can now customise the output of the plot from the command - custom plot titles for all plots & naming of the runs in the plot keys are named after a specified run identifier.

This change was necessary, the overall benchmarking process is now more efficient in terms of speed, storing of data & code readability (also managed to cut down the codebase).
Finally, I have added some much-needed documentation for running a benchmark.
A general idea of how things work now:

Run configuration YAML file is consumed and parsed - providing all information required for benchmarking run(s).
Unique corpus of phenopackets is parsed and relevant gene/variant/disease information (if specified) is stored in a duckdb table (TABLE A).
Depending on whether gene/variant/disease benchmarking is specified for that run configuration, the standardised TSV result is read in as a duckdb table (TABLE B) and used to compare against what is known from the phenopacket (retrieved from TABLE A) and the rank is retrieved and added to TABLE A.
Summary stats, plots, and comparisons between runs are generated from TABLE A

This PR fails due to the MakeFile pipeline test that executes the template runner and the benchmark command. The MakeFile could be separated out of the main PhEval repo and moved into monarch-pheval - @souzadevinicius agreed on this statement in another PR. Ultimately this PR is blocked until the test is refactored in the MakeFile pipeline OR this is moved to another repo to test the pipeline.

This PR fails due to the MakeFile pipeline test that executes the template runner and the benchmark command. The MakeFile could be separated out of the main PhEval repo and moved into monarch-pheval - @souzadevinicius agreed on this statement in another PR. Ultimately this PR is blocked until the test is refactored in the MakeFile pipeline OR this is moved to another repo to test the pipeline.

The pipeline test needed to be modified due to the pheval-utils methods refactor. Sorry for this PR blocking. @yaseminbridges, @julesjacobsen and @matentzn, wherever you decide to put the Makefile code, I agree 😅.


Scalability enhancement: sharding support

For some large scale applications, it is fairly easy to "shard" the application to provide scalability ... eg. PeerPad documents have no dependencies on each other, so it would be easy to sort them into individual shards so that the backend server assets needed for collaboration could be scaled.
Two things that could benefit from some sharding management are:

selecting different websocket-star rendezvous servers based on the shard
selecting different pubsub gossip channels based on the shard

With consistent hashing (already used in peer-star-app), it should be possible to add or remove shards dynamically so the clustered/sharded services can be scaled up and down to match load.

Moved to: https://github.com/ipfs-shipyard/peer-star/issues/20


Send composition assumptions from client to server

Composition currently does not work in client/server mode because receipts are not sent from client to server. This PR adds the required plumbing to send the assumptions from client to server.
Resolves: #1236

Note that I moved ReceiptClaim from core to base. My inuition is that this is the better place for it under the understanding that api is the surface with which external developers are expected to interact, and core contains less stable details such as the MemoryImage. Since the ReceiptClaim is part of the public API surface, and also used internally, it seems like it needs to be in base.

The distinction between api, core, and base is:

api: of the wire messages between client/server.
core: the core data types, these are meant to be persistent formats for objects that may be stored in disk, in a DB, and temporarily over the write in the api. Ideally, the API would treat these types as opaque blobs. The idea is to split wire from storage following this best practice: https://protobuf.dev/programming-guides/api/#use-different-messages
base: the glue that works in both api and core.

Given this I think ReceiptClaim belongs in core rather than base.

Ok, I've refactored this and got test coverage by refactoring most of the executor tests to run in client/server mode in addition to local mode based on the RISC0_EXECUTOR env var. I'm not convinced this is the best option, but I haven't thought of a better one yet. @flaub , do you have any thoughts?
https://github.com/risc0/risc0/pull/1241/files#diff-7829468e86c1cc5d5133195b5cb48e1ff6c75e3e9203777f6b2e379d9e4882b3R178-R181

In https://github.com/risc0/risc0/pull/1241/commits/b4d07bbc93470f65d82af89c536fbf6190d8d7e3 I added a step to build r0vm. Given caching, I'm not sure whether or not that will have a significant impact on CI run times.
Given the fact that tests are being run with -F prove, an alternative to this would be to provide some way of running the test in a sort of "loopback server mode" where, without starting a new process or in a fork, the test process creates an IPC connected to itself/ the fork. This would be a deeper change.

In b4d07bb I added a step to build r0vm. Given caching, I'm not sure whether or not that will have a significant impact on CI run times. I will check the difference once CI completes here.
Given the fact that tests are being run with -F prove, an alternative to this would be to provide some way of running the test in a sort of "loopback server mode" where, without starting a new process or in a fork, the test process creates an IPC connected to itself/ the fork. This would be a deeper change.

Looks like the new steps add about 2-4 minutes


UX: Parameter-Names with whitespaces cause problems but don't give error messages

Within a Standard Logic App we've used a parameter name which includes a whitespace:

No problem, if it causes a problem, we can rename it.
But the issue was that saving the parameters works fine, saving a workflow works fine, no errors anywhere. But when you use the Logic App you notice that the workflow hasn't been compiled.
When you take a look at the Resource health and dig deeper into the menues you'll find the issue. But I would expect to already get an error when I try to save parameters with whitespaces.
AB#15951998

@hartra344 - we are missing an api at workflow app level to validate any artifact files, only in the context of any workflow we can use the validate api. Currently saving connections or parameters from their blade is just a file save call on the site resource.

I am moving this isue to backend because we will need a validate api just on the artifact to make this working for the customers.


update dependencies

follow up to https://github.com/keboola/docker-bundle/pull/688

ah, we need to roll back that message change


"Furduiești" is listed in the drop-down menu

Description:
"Furduiești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Furduiești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Furduiești, Alba (Câmpeni)".


Add CWE support in all importers

issues: #1093

@ziadhany Thanks++, before associating any vulnerability with CWE please check if it exists in cwe2 DB or not like this https://github.com/nexB/vulnerablecode/pull/1256, and also please do this for NVD importer too.


Support GitHub Actions

Create a simple workflow to test building and installing the package as well as building and launching a Bobber container using the application.
Closes #3
Signed-Off-By: Robert Clark<EMAIL_ADDRESS>

This initial Action does the following:

Installs the dependencies in requirements.txt
Checks the code with pycodestyle
Builds/installs the Python wheel
Builds the Docker image using the wheel

This isn't as exhaustive as it could be, but this initial phase should suffice to ensure we aren't making major changes that would impact the suitability of the Python wheel.
In the future, the plan is to run the checks against actual hardware using private runners. Those tests will go through a clean run of the tests and parse results, checking to see if they meet a certain threshold to ensure the code functions properly.


Skip empty requirement lines.

Closes: #17.
This simply skips such lines to solve #17 immediately.
Personally though longer term, I'd like it if tox-ini-fmt actually preserved + used the newlines -- usually I use empty lines there to denote requirements "blocks"/groups. So e.g. if I'm using factor groups I'll put empty lines between the factor groups if they have a bunch of different requirements. Let me know if you agree/disagree with that.

Codecov Report

Merging #20 into main will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #20   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          246       246
  Branches        55        55
=========================================
  Hits           246       246

Flag
Coverage Δ

#tests
100.00% <100.00%> (ø)

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

Impacted Files
Coverage Δ

src/tox_ini_fmt/formatter/requires.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

I'll need some more selling on groups, open an issue and post some example, and say why it's better. but this pr on this own works for now.


Update measure tools to use eslint and raise itwin.js minimum

did you bump the core ver? or mean to?

did you bump the core ver? or mean to?

@aruniverse yup, just pushed that change. A couple of the changes I made to clean up the lint were added in 2.19.

@aruniverse mind taking another quick look? Pushed a few more updates while I was already updating the minimum version needed.
Also, do we want to try and get a release out with the cherry-picked changes and the 2.17 minimum prior to updating all the way to 2.19?

@aruniverse mind taking another quick look? Pushed a few more updates while I was already updating the minimum version needed.
Also, do we want to try and get a release out with the cherry-picked changes and the 2.17 minimum prior to updating all the way to 2.19?

new changes look fine to me, i dont think theres much usage of this pkg, so id go straight to 2.19, but whatever is easiest for you

@aruniverse that's easier for me. We'll go straight there. Thanks!


[3.1.0] Add support to puppet 8

The camptocamp-augeas dependency must be removed,  because this module is only compatible with the deprecated puppet 6.
See https://forge.puppet.com/modules/camptocamp/augeas/readme
Puppet >= 4.10.0 < 7.0.0

In f10082aee7f12f172a42f13ab2bdf8779fb22e71 some lenses were dropped, reducing the need for camptocamp/augeas but 84a211ba40523dda73b64746d60cf585d777727b completely dropped the dependency. Since #358 we already tested with it.


Review linter checks

We need to review if the current linter checks are appropriate and what additional ones could be interesting to add.

Similarly, it'd be good to also review how each of these checks is being done (rules behind each of them) to fine tune them. They can be found hovering over the ? icon on each of them in the project detail view.

Cc: @caniszczyk

We've checked quite a few orgs for the .github repo, but very few are using it.
We are experimenting a bit with a new concept: multiple repositories kinds. We have two at the moment, primary and secondary, and each kind is linted and scored in a different way (different checks, weights, etc). When computing the final project score, the primary has much more weight as well.
It's looking good but it still requires a bit of work. Hopefully we'll have something ready by tomorrow 🙂

I think checking for .github is still fine, we WANT more projects to use it.
I think primary/secondary repos is the way to do it, basically we can
specify a primary repo and a bunch of secondary repo. To make our lives
easier, we may also have support for second github orgs too, because some
projects span across multiple github orgs like kubernetes, kubernetes-sigs
etc
On Mon, Feb 14, 2022 at 11:41 AM Sergio C. Arteaga @.***>
wrote:

We've checked quite a few orgs for the .github repo, but very few are
using it.
We are experimenting a bit with a new concept: multiple repositories
kinds. We have two at the moment, primary and secondary, and each kind
is linted and scored in a different way (different checks, weights, etc).
When computing the final project score, the primary has much more weight as
well.
It's looking good but it still requires a bit of work. Hopefully we'll
have something ready by tomorrow 🙂
—
Reply to this email directly, view it on GitHub
https://github.com/cncf/clomonitor/issues/15#issuecomment-1039368882,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPSIN4IWVBOQ75OZLQ2B3U3E5EZANCNFSM5NYGDADQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Cheers,
Chris Aniszczyk
https://aniszczyk.org

Support for primary/secondary repos is ready:

and these are the checks applied to secondary repos at the moment:

I think checking for .github is still fine, we WANT more projects to use it.

We can also add support for this, it's not a problem. This kind of check will require cloning a remote repo even when running the linter locally on an already checkout repo though, which may not be very nice. Plus it's Github specific. Some of the requested checks will also require Internet access, like the releases one, etc, unless we opt for handling them differently (not talking to the Github API).


Use new Pester syntax: -Parameter for Pester in SDK and Provider tests

PR Summary
Use new Pester syntax: -Parameter for Pester in SDK and Provider tests
Resolves part of #6245
PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Change is not breaking
[X] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[X] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[X] Not Applicable
OR
[NA] User-facing Documentation needed

[NA] Issue filed - Issue link:

Testing - New and feature

[X] Not Applicable or can only be tested interactively
OR
[NA] Make sure you've added a new test if existing tests do not effectively test the code changed

[NA] Add [feature] if the change is significant or affects feature tests

@TravisEz13 @SteveL-MSFT I can not merge PRs. I see "You're not authorized to merge this pull request". Is policy changed?

@iSazonov I replied you in another PR about the permission issue. You should be able to merge now.
Also, please require [feature] test runs for all Pester v4 syntax change PRs. We merged a few Pester syntax change PRs last Friday and our daily build on Linux has been broken since then, so we need to be more cautious.

@daxian-dbw Thanks - I get permissions again.
Clear about Feature.

@KevinMarquette Please add [Feature] in commit header to pass full test set.


Installation error

I'm getting this when I install on a Windows 7 32 bit machine:

node lib/install.js

‼ spawn ENOENT
‼ optipng pre-build test failed
i compiling from source
× Error: Command failed: '.' is not recognized as an internal or external
and,
operable program or batch file.
at ChildProcess.exithandler (child_process.js:648:15)
at ChildProcess.emit (events.js:98:17)
at maybeClose (child_process.js:756:16)
at Socket.<anonymous> (child_process.js:969:11)
at Socket.emit (events.js:95:17)
at Pipe.close (net.js:465:12)

<EMAIL_ADDRESS>node_modules\imagemin-optipng
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Cheers,
Dave

Building is not supported on Windows platforms.


Use latest rubocop on both gemspec and CC

We were already using this in Gemfile.lock

Pull Request Test Coverage Report for Build 179

0 of 0   changed or added relevant lines in 0 files are covered.
11 unchanged lines in 6 files lost coverage.
Overall coverage remained the same at 92.363%

Files with Coverage Reduction
New Missed Lines
%

test/dummy/config/initializers/migration_class.rb
1
75.0%

test/test_password_expired_controller.rb
1
100.0%

lib/devise-security/controllers/helpers.rb
1
50.94%

lib/devise-security/rails.rb
2
100.0%

test/test_security_question_controller.rb
3
100.0%

test/dummy/config/environments/test.rb
3
100.0%

Totals

Change from base Build 176:
0.0%

Covered Lines:
895

Relevant Lines:
969

💛  - Coveralls

Pull Request Test Coverage Report for Build 179

0 of 0   changed or added relevant lines in 0 files are covered.
11 unchanged lines in 6 files lost coverage.
Overall coverage remained the same at 92.363%

Files with Coverage Reduction
New Missed Lines
%

test/dummy/config/initializers/migration_class.rb
1
75.0%

test/test_password_expired_controller.rb
1
100.0%

lib/devise-security/controllers/helpers.rb
1
50.94%

lib/devise-security/rails.rb
2
100.0%

test/test_security_question_controller.rb
3
100.0%

test/dummy/config/environments/test.rb
3
100.0%

Totals

Change from base Build 176:
0.0%

Covered Lines:
895

Relevant Lines:
969

💛  - Coveralls

Pull Request Test Coverage Report for Build 179

0 of 0   changed or added relevant lines in 0 files are covered.
11 unchanged lines in 6 files lost coverage.
Overall coverage remained the same at 92.363%

Files with Coverage Reduction
New Missed Lines
%

test/dummy/config/initializers/migration_class.rb
1
75.0%

test/test_password_expired_controller.rb
1
100.0%

lib/devise-security/controllers/helpers.rb
1
50.94%

lib/devise-security/rails.rb
2
100.0%

test/test_security_question_controller.rb
3
100.0%

test/dummy/config/environments/test.rb
3
100.0%

Totals

Change from base Build 176:
0.0%

Covered Lines:
895

Relevant Lines:
969

💛  - Coveralls

Pull Request Test Coverage Report for Build 179

0 of 0   changed or added relevant lines in 0 files are covered.
11 unchanged lines in 6 files lost coverage.
Overall coverage remained the same at 92.363%

Files with Coverage Reduction
New Missed Lines
%

test/dummy/config/initializers/migration_class.rb
1
75.0%

test/test_password_expired_controller.rb
1
100.0%

lib/devise-security/controllers/helpers.rb
1
50.94%

lib/devise-security/rails.rb
2
100.0%

test/test_security_question_controller.rb
3
100.0%

test/dummy/config/environments/test.rb
3
100.0%

Totals

Change from base Build 176:
0.0%

Covered Lines:
895

Relevant Lines:
969

💛  - Coveralls

Pull Request Test Coverage Report for Build 179

0 of 0   changed or added relevant lines in 0 files are covered.
11 unchanged lines in 6 files lost coverage.
Overall coverage remained the same at 92.363%

Files with Coverage Reduction
New Missed Lines
%

test/dummy/config/initializers/migration_class.rb
1
75.0%

test/test_password_expired_controller.rb
1
100.0%

lib/devise-security/controllers/helpers.rb
1
50.94%

lib/devise-security/rails.rb
2
100.0%

test/test_security_question_controller.rb
3
100.0%

test/dummy/config/environments/test.rb
3
100.0%

Totals

Change from base Build 176:
0.0%

Covered Lines:
895

Relevant Lines:
969

💛  - Coveralls


Version number in settings

There should be an app ver number in seetings in order to know which app user use

Let the number vesrion looks like in this screenshot

62d0b27a3083787d3d5046a4886c479a3ba2840b @mBoiler mBoiler committed 15 hours ago


v7.1 Support: handle mimetype=octet-stream

The initrd file for v7.1 no longer appears to be compressed:
file --mime-type --brief /tmp/initrd.img application/octet-stream
We will need to handle this file type case in order to support v7.1

Hm...works on my machine ;)

Which OS are you running? Which version of the file command are you using?

Hi, thanks for looking into it. I'm running Centos 7.9.2009 which ships with file 5.11-37

That what it was, everything worked fine once I switched to using file-5.41


Publish a manylinux wheel to pypi

It would great if a manylinux wheel could be published to pypi for this project. For more information about manylinux, see PEP 513
Many linux allows for users to use pylibmc on the majority of linux platforms (regardless of version), without having to deal with binary dependencies. (Which is useful for server deployments to AWS Lambda for example).
The basic idea is to add some lines to the travis.yaml file, which can build and deploy a manylinux wheel to pypi.  This uses a special image to build the wheel and link in the binary dependencies.  (See https://github.com/pypa/python-manylinux-demo for another example and more info). Special docker images from a PyPA repo are used to build a wheel that will work on almost any linux system.
I wrote an example of what this would look like at https://github.com/bxm156/pylibmc-manylinux . The wheels can be downloaded at: https://github.com/bxm156/pylibmc-manylinux/releases.
I'm using the above wheels in my own projects.

Any chance we could get this in? Want to use pylibmc on Lambda and can't currently since the libmemcached-dev headers are not installed.

I did a wheelie
☸


Update README.md to explain how to avoid potential failure on lists i…

…nside a row

Thanks for this Tom


Option to write Date strings in ISO-8601 format

Roland asked for this: ISO-8601 formatted dates in the 'map-scale' files that Ferret writes within LAS.  It'd be a good general capability to be able to write out dates as 2018-12-18  instead of the default 18-DEC-2018.
Add a MODE DATE_FORMAT which can take the values DEFAULT or ISO8601.  All date-strings are formed in a few routines so it's straight forward to reformat them as they're needed.  So,
yes? use monthly_navy_winds

! default date formatting

yes? list/L=33:35/x=180/y=0 uwnd
             VARIABLE : ZONAL WIND (M/S)
             FILENAME : monthly_navy_winds.cdf
             FILEPATH : /home/users/tmap/ferret/linux/fer_dsets/data/
             SUBSET   : 3 points (TIME)
             LONGITUDE: 180E
             LATITUDE : 0
                     180E
                     65
 16-SEP-1984 / 33: -1.549
 17-OCT-1984 / 34: -0.538
 16-NOV-1984 / 35: -1.526

yes? set mode date:iso

yes? list/L=33:35/x=180/y=0 uwnd
             VARIABLE : ZONAL WIND (M/S)
             FILENAME : monthly_navy_winds.cdf
             FILEPATH : /home/users/tmap/ferret/linux/fer_dsets/data/
             SUBSET   : 3 points (TIME)
             LONGITUDE: 180E
             LATITUDE : 0
                     180E
                     65
 1984-09-16  / 33: -1.549
 1984-10-17  / 34: -0.538
 1984-11-16  / 35: -1.526

yes? say `uwnd,return=tstart`
1982-01-16 20:00:00

yes? shade/L=41 uwnd   ! will have the time label in the upper left as 1985-05-18

yes? list/norow/noheader/L=33:35 tax_datestring(t[gt=uwnd], t[gt=uwnd], "day")
 "1984-09-16"
 "1984-10-17"
 "1984-11-16"

yes? list/norow/noheader/L=33:35 tax_datestring(t[gt=uwnd], t[gt=uwnd], "minute")
 "1984-09-16 20:00"
 "1984-10-17 06:30"
 "1984-11-16 17:00"

A further step could be to allow entering date-strings in this way,  say in a DEFINE AXIS/T= or LIST/T= command.  For that, Ferret could just detect the formatting and accept either sort, independent of how the MODE is set.
@noaaroland

As I understand it, the date strings must always contain all of yyyy-mm-dd.
So for modulo time axes, which don't have a year associated with them, I think it's best to write the dates with year 0000
 yes? list/x=300/y=-40 uwnd
             VARIABLE : ZONAL WIND (M/S)
             FILENAME : coads_climatology.cdf
             FILEPATH : /home/users/tmap/ferret/linux/fer_dsets/data/
             SUBSET   : 12 points (TIME)
             LONGITUDE: 61W
             LATITUDE : 41S
                     61W
                    140
 0000-01-16  /  1:  0.503
 0000-02-15  /  2:  0.727
 0000-03-17  /  3:  1.267
 0000-04-16  /  4:  2.392
...

This is done.  I'll make a new ticket with lower priority about being able to read dates that are input to commands in this format.

Modulo time axes may start in 0000 or 0001 and extend a year or less.  Andrew's seasonal climatological axes start, for instance, in December 0001 and go into year 2.  So, we need to use the actual year from the axis in date-strings for climatologies:

yes? use clim_axes_gregorian
yes? set mode date:ISO

yes? sh axis/t NDJ_REG_GRG
 name       axis              # pts   start                end
 NDJ_REG_GRG TIME               4mr   0001-12-17 00:34:39  0002-09-16 22:56:33
T0 = 01-JAN-0001 00:00:00
   Axis span (to cell edges) = 365.2425 (modulo length = axis span)

       L     T                       TBOX          TBOXLO                TSTEP (days)
       1>  0001-12-17 00:34:39       91.31062   0001-11-01 08:51:00         350.0241
       2>  0002-03-18 08:01:57       91.31062   0002-01-31 16:18:18         441.3347
       3>  0002-06-17 15:29:15       91.31062   0002-05-02 23:45:36         532.6453
       4>  0002-09-16 22:56:33       91.31062   0002-08-02 07:12:53         623.9559

--- and when not in mode DATE:ISO ,  listings now will NOT show year 0002.
checked in; changes to a couple of benchmark script outputs.


[css] @supports and/or/not should be case insensitive

Issue Type: Bug
Example of code that is valid in browsers but is marked up as a syntax error in VS Code (with the result that code inside the braces is also mis-parsed):
@supports (column-width: 1rem)
  OR (-moz-column-width: 1rem)
  OR (-webkit-column-width: 1rem) {
/* create columns, with fallback properties */
}

On testing, it seems that VSCode expects the combinators (and, or, not) to always be lowercase. But, like most of CSS, they should be case insensitive.  Lowercase could be a linting recommendation, but uppercase is definitely not a syntax error!
VS Code version: Code 1.23.0 (7c7da59c2333a1306c41e6e7b68d7f0caa7b3d45, 2018-05-03T16:44:55.614Z)
OS version: Windows_NT x64 10.0.17134

Verified it works, also double checked this behavior is correct:

CSS style sheets are generally case-insensitive, and this is also the case for media queries.

From https://www.w3.org/TR/css3-mediaqueries/#syntax (sent there from https://drafts.csswg.org/css-conditional-3/#at-supports )


Error msg: Synchronous XMLHttpRequest on the main thread is deprecated

Chrome 45
Full msg: Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check http://xhr.spec.whatwg.org/.
Line 7: alasql.min.js
I envoked this console error msg when doing this:
alasql('SOURCE "myfile.sql"');
As described here:
https://github.com/agershun/alasql/wiki/How-to-run-SQL-script-from-text-file
I thought you could never use Synchronous with an XHR. Can this be fixed to not only use Async but Promises?

Please use this construction for async call of SOURCE statement:
    alasql('SOURCE "myfile.sql"', [], function() {
         // do the rest of the program here...
     Very active development. Great work guys!

😊👍


hinted compilation fails on pandas core

Hi:
I've been building with nuitka for standalone compilation, and ran into a piece of code that has lots of imports that causes the compilation time to become unreasonably high. So, I'm trying to see if the hinted compilation will help resolve the runtime issue.
However, I get a nuitka warning about an issue with pandas that causes the compilation to stop moving forward. I'm just looking for some suggestions what what the culprit might be.
bash-4.2$ python3 nuitka-hints.py myfile
NUITKA v<IP_ADDRESS> on Python 3.8.2 (linux) is compiling 'myfile' with these options:
--standalone
--recurse-none
--hinted-json-file=myfile-38-linux-64.json
Nuitka-Plugins:INFO: hinted-mods: 'hinted-mods' is adding the following options:
Nuitka-Plugins:INFO: hinted-mods: Enabling Nuitka plugin 'numpy' as needed.
Nuitka-Plugins:WARNING: numpy: matplotlib may need hinted compilation for non-standard backends
Nuitka-Plugins:INFO: hinted-mods: Enabling Nuitka plugin 'qt-plugins' as needed.
...
Problem with <Node 'STATEMENT_ASSIGNMENT_VARIABLE' with {'variable': <LocalVariable 'new_style' of 'convert'>}> at /vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/pandas/core/internals/blocks.py:2797
Problem with <Node 'STATEMENT_LOOP'> at /vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/pandas/core/internals/blocks.py:2796
Problem with <Node 'STATEMENT_TRY'> at /vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/pandas/core/internals/blocks.py:2796
Problem with <Node 'STATEMENT_TRY'> at /vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/pandas/core/internals/blocks.py:2783
Nuitka:WARNING: Problem creating function code 'pandas$core$internals$blocks$$$function_153_convert'.
Nuitka:WARNING: Problem creating code for module <Node 'COMPILED_PYTHON_MODULE' with {'filename': '/vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/pandas/core/internals/blocks.py', 'module_name': }>.
Traceback is really long, I'm just skipping to the end to show that I'm confused by that also:
File "/vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/nuitka/codegen/LoopCodes.py", line 67, in generateLoopCode
generateStatementSequenceCode(
File "/vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/nuitka/codegen/CodeHelpers.py", line 222, in generateStatementSequenceCode
_generateStatementSequenceCode(
File "/vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/nuitka/codegen/CodeHelpers.py", line 211, in _generateStatementSequenceCode
generateStatementCode(
File "/vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/nuitka/codegen/CodeHelpers.py", line 171, in generateStatementCode
statement_dispatch_dict[statement.kind](
File "/vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/nuitka/codegen/VariableCodes.py", line 79, in generateAssignmentVariableCode
getVariableAssignmentCode(
File "/vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/nuitka/codegen/VariableCodes.py", line 361, in getVariableAssignmentCode
variable_declaration.getCType().emitVariableAssignCode(
File "/vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/lib/python3.8/site-packages/nuitka/codegen/c_types/CTypeNuitkaBools.py", line 37, in emitVariableAssignCode
assert not in_place
AssertionError
I can include more information if necessary; however, I apologize that I can't provide a simpler case of what created the error, because there are many imports that use pandas under the hood. If someone can advise me on how to further isolate the issue, that would help the creation of a small testcase as well.

Nuitka version, full Python version and Platform (Windows, OSX, Linux ...)
<IP_ADDRESS>
Python: 3.8.2 (default, Jun 16 2020, 17:24:50)
Executable: /vendor/tools/FOSS/python/3.8.2_20200616a/x86_64/bin/python3.8
OS: Linux
Arch: x86_64

Pandas version: 0.25.0

How did you install Nuitka and Python (pip, anaconda, deb, rpm, from source,
what is a virtualenv ...), this is very important usually.
pip

Thanks,
Alfred

That's actually something that <IP_ADDRESS> fixed I believe.

Sorry about this and thank you, it is confirmed to be fixed in <IP_ADDRESS>.
Nuitka is really great work, and hinted_compilation is a great addition to the setup.


Docs: Broken Link

Description
Fixes #3761. Changed /theming/overview to /overview
How Has This Been Tested?

Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

Thanks!


tools/benchmark: stream results into reports

Reports depended on writing all results to a large buffered channel and
reading from that synchronously. Similarly, requests were buffered the
same way which can take significant memory on big request strings. Instead,
have reports stream in results as they're produced then print when the
results channel closes.

Just tested this code in my local machine:
./benchmark --endpoint="<IP_ADDRESS>:12378" put --clients=100
10000 / 10000 Booooooooooooooooooooooooooooooooooooooooooooooo! 100.00 % 46s

Summary:
  Total:	46.1354 secs.
  Slowest:	0.8960 secs.
  Fastest:	0.1153 secs.
  Average:	0.4590 secs.
  Requests/sec:	216.7535

Response time histogram:
  0.115 [1]	|
  0.193 [0]	|
  0.271 [18]	|
  0.349 [824]	|∎∎∎∎∎∎∎∎
  0.428 [2771]	|∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎
  0.506 [3766]	|∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎
  0.584 [1945]	|∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎
  0.662 [539]	|∎∎∎∎∎
  0.740 [122]	|∎
  0.818 [12]	|
  0.896 [2]	|

Latency distribution:
  10% in 0.3526 secs.
  25% in 0.4045 secs.
  50% in 0.4507 secs.
  75% in 0.5205 secs.
  90% in 0.5676 secs.
  95% in 0.5997 secs.
  99% in 0.6698 secs.

LGTM. Defer to @xiang90.

One question, then LGTM.

LGTM again. Thanks for fixing the nits.


Add Tobii Eye Tracking

(?:^|/)Tobii.EyeX.Client.dll$
(?:^|/)Tobii(?:\.|_)GameIntegration(?:_x64|_x86)?\.dll$
Is there a better website to link to ? i didn't want to link to tobii's main website because it's intended for marketing

https://gaming.tobii.com/games/
Could help to check if any games (that are on steam :) ) are missing

There's a bunch of different names, however it appears to be safe to just do (?:^|/)Tobii.


Test egress from a container group does not return firewall public IP

[Enter feedback here]
We are following the instruction to create a lab environment. The container group and firewall are all successfully deployed, however there is an issue to verify the egress traffic.
https://learn.microsoft.com/en-us/azure/container-instances/container-instances-egress-ip-address#test-egress-from-a-container-group
We create the 'testegress' container group and it will run 'curl -s http://checkip.dyndns.org' command. In the document, it says we should get the firewall public IP, however we always get the ACI private IP.
To troubleshoot the issue, we do the following test:

In the testegress container, we run 'curl -s http://ifconfig.me' command before we change the firewall rule. The traffic is blocked, which means the firewall is working as expected.
We modify the firewall rule to whitelist the URL. The traffic is successful, which is the expected result.
The URL 'ifconfg.me' returns the firewall public IP. This indicates that the issue is caused by 'checkip.dyndns.org'

What is the impact of the issue:
The customer follows the document to try our feature. However, due the egress traffic verification failure, they think the firewall is not properly configured. Please review this document and modify it accordingly.

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

ID: 6a2b459d-cbbb-100d-a10c-6652c55885b7
Version Independent ID: d45dcb68-316d-d166-426a-88feb1fd05e3
Content: Configure static outbound IP - Azure Container Instances
Content Source: articles/container-instances/container-instances-egress-ip-address.md
Service: container-instances
GitHub Login: @tomvcassidy
Microsoft Alias: tomcassidy

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

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

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, we can help redirect you to the right support channel to get an answer to your question: [reference to alternative product/service or community endpoint]. Because this issue does not relate to documentation, we are closing this issue. #please-close


git hook: windows script

Is this a correct script on Windows:
   The script is the same for all operating systems:
   ```shell
   #!/bin/bash

   pytest example.py
   ```

In particular, #!/bin/bash

Thanks. This is probably not right.

From my experience (Git Bash on Win 10), pre-commit hooks work the same as on other systems. They also run in my Anaconda Prompt, although there are some weird issues with their content (maybe shell issue?). I can investigate more if you want....
Btw., I had major issues with hooks when testing them in GitHub Desktop on Win, but on some other win installations they worked in GitHub Desktop, too. Annoying, but well, requesting all contributors to use git in terminal if possible.

This issue is not relevant anymore since the local hook episode got removed in #111.