Error running npm-install

Today, I started getting errors when running npm install for my web application.  I seem to have traced it to npm-gyp.  Here is what I get when I try to install it:
$ npm install<EMAIL_ADDRESS>
><EMAIL_ADDRESS>install C:\Development\node_modules\microtime
> node-gyp rebuild

C:\Development\node_modules\microtime>if not defined npm_config_node_gyp (node "C:\Users\me\AppData\Roaming\npm\node_modules\n                                                                                pm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
Traceback (most recent call last):
  File "C:\Users\me\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\gyp\gyp_main.py", line 16, in <module>
    sys.exit(gyp.script_main())
  File "C:\Users\me\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 545, in script_main
    return main(sys.argv[1:])
  File "C:\Users\me\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 538, in main
    return gyp_main(args)
  File "C:\Users\me\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 514, in gyp_main
    options.duplicate_basename_check)
  File "C:\Users\me\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 98, in Load
    generator.CalculateVariables(default_variables, params)
  File "C:\Users\me\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\generator\msvs.py", line 1900, in CalculateVariables
    generator_flags.get('msvs_version', 'auto'))
  File "C:\Users\me\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\MSVSVersion.py", line 434, in SelectVisualStudioVersion
    versions = _DetectVisualStudioVersions(version_map[version], 'e' in version)
KeyError: '2017'
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (C:\Users\me\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\configure.js:305:16)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Windows_NT 6.1.7601
gyp ERR! command "C:\\Program Files (x86)\\nodejs\\node.exe" "C:\\Users\\me\\AppData\\Roaming\\npm\\node_modules\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "re                                                                                build"
gyp ERR! cwd C:\Development\node_modules\microtime
gyp ERR! node -v v7.4.0
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok
<EMAIL_ADDRESS>C:\Development\Advaxis\Advaxis.norm\Advaxis.Norm.web
+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>`-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN<EMAIL_ADDRESS>No license field.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\microtime):
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm WARN optional SKIPPING OPTIONAL DEPENDENCY: Exit status 1

+1

+1

This has brought our dev team to a halt. When trying to install microtime directly which is kicking off the node-gyp rebuild here is the error(s) I get:
C:\Users\k_phillips\test<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS><EMAIL_ADDRESS>install C:\Users\k_phillips\test\node_modules\microtime
node-gyp rebuild
C:\Users\k_phillips\test\node_modules\microtime>if not defined npm_config_node_gyp (node "C:\Users\k_phillips\AppData\Roaming\npm\node_modules\npm\bin\node-gyp-bin\....\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
TRACKER : error TRK0005: Failed to locate: "CL.exe". The system cannot find the file specified.  System Windows_NT 6.1.7601
gyp ERR! command "C:\Program Files\nodejs\node.exe" "C:\Users\k_phillips\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\k_phillips\test\node_modules\microtime
gyp ERR! node -v v6.10.0
gyp ERR! node-gyp -v v3.5.0
gyp ERR! not ok
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN<EMAIL_ADDRESS>No license field.
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\k_phillips\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install"<EMAIL_ADDRESS>npm ERR! node v6.10.0
npm ERR! npm  v4.3.0
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the microtime package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs microtime
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls microtime
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\k_phillips\AppData\Roaming\npm-cache_logs\2017-05-11T17_59_58_211Z-debug.log

@ukphillips That is where I actually started with this.  This has been a huge time waste for my team too.  https://github.com/wadey/node-microtime/issues/49

I'm using windows, after the steps described here https://github.com/nodejs/node-gyp#installation, the error was gone

@im1dermike I just noticed that - I am now where you are. VERY frustrating that this has all blown up without a single change to any of our code. All I can assume is a package somewhere in the dependency chain change that has caused this mess.

@ukphillips NPM's biggest issue IMO.  I'm thinking that changing my package.json file to exclude the ^  prefix to packages so it won't be affected by issues like this with new versions would be smart.

+1

Is this the same issue? The symptom sounds the same https://github.com/nodejs/node-gyp/issues/1144

@skkonstantin Which steps did you follow?  I ran npm install --global --production windows-build-tools and still have the issue.

@im1dermike , I did the same thing, with administrator root . i think this error came after OS update

@nodejs/platform-windows any ideas? Looks like the original issue is
File "C:\Users\me\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\MSVSVersion.py", line 434, in SelectVisualStudioVersion
    versions = _DetectVisualStudioVersions(version_map[version], 'e' in version)

which is indeed the same as https://github.com/nodejs/node-gyp/issues/1144. (so cc/ @refack )

It looks like someone (the author of pdbdf2?) has fixed it.  https://github.com/wadey/node-microtime/issues/49.  I just blew out my node_modules folder and ran npm install and it looks like it worked!

It looks like someone (the author of pdbdf2?) has fixed it. wadey/node-microtime#49. I just blew out my node_modules folder and ran npm install and it looks like it worked!

So a native module was added to the dependencies, and now it's been removed again. FWIW it sounds like you can't install any native modules, so the issue is still there.

Not 1144, there the error is KeyError: '2017'
@im1dermike 's error has this in log: node-gyp -v v3.4.0
@ukphillips last log shows also old node-gyp: node-gyp -v v3.5.0
First thing to try is to update npm or just node-gyp

For posterity doing
npm install<EMAIL_ADDRESS>or even
npm i -g<EMAIL_ADDRESS>Is not sufficient, since when triggering the build on native Addon via npm i npm will use it's own version of node-gyp, so if you want to use Visual Studio 2017, you'll need to update npm to a version >= 4.5.0

Found out that npm was using<EMAIL_ADDRESS>while running npm i. But node-gyp -v yields v3.6.1.
I went into the npm folder, /Program Files (x86)/.../npm and ran npm i --save<EMAIL_ADDRESS>to bring node-gyp up to date with my global install.
Errors remains (SelectVisualStudioVersion...). Version change reflected in same error stack trace.

Forgot to mention, I'm running<EMAIL_ADDRESS>and<EMAIL_ADDRESS>And SelectVisualStudioVersion is same error as KeyError: '2017'.

First make sure you see
gyp ERR! node -v v7.10.0
gyp ERR! node-gyp -v v3.6.1
gyp ERR! not ok

In the error output.
If you do: look at #1179
Else: do the npm upgrade dance - https://docs.npmjs.com/getting-started/installing-node
npm install npm@latest -g

@refack I ran into this issue today and I've seen that some others have been befuddled by it as well, with seemingly no resolution. I found out something and wanted to share it in case anyone else finds it useful.
Here: https://github.com/nodejs/node-gyp/blob/v3.6.2/lib/find-vs2017.js#L17
I changed that to log.warn instead of log.silly just so I could see what was coming out, and I see a pretty JSON object like I should... except before the pretty JSON object, I have a pesky WARNING: Could not find ssh-agent thanks to posh-git. Looking up and taking care of that issue resolved this issue for me.

@tuespetre Just FYI, you can make npm log at silly level with npm -ddd i.
So I'm assuming posh-git in your PS profile script tried to autorun and fails, so I'll add -NoProfile to https://github.com/nodejs/node-gyp/blob/b5b52f7bffb55064a623e2478252a8939259cf3f/lib/find-vs2017.js#L9-L11

@refack I did --loglevel=silly and for some reason I did get a ton of those logs but not the ones I was hoping to see from that. Only reason 😛

I did --loglevel=silly and for some reason I did get a ton of those logs but not the ones I was hoping to see from that. Only reason 😛

Well it is called silly 🤷♂
Refs: https://github.com/dahlbyk/posh-git/pull/484

Cross-ref: https://github.com/nodejs/node-gyp/pull/1292

@tuespetre FYI, @dahlbyk landed a fix in posh-git and we landed a fix here (but it still needs to get into a release, then be picked-up by npm)

Has this been fixed?
 System Windows_NT 10.0.14393
gyp ERR! command "C:\Program Files\nodejs\node.exe" "C:\Users\Paulp\AppData\Roaming\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\electronDevelopment\chap5\exampleEdge2
gyp ERR! node -v v8.2.1
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok

@paulprendergast is seems like your folder C:\electronDevelopment\chap5\exampleEdge2 does not have a binding.gyp file.

Has this been resolved? if it has been resolved can you directed to page and location on page or give me direct feedback
versions = _DetectVisualStudioVersions(version_map[version], 'e' in version)
KeyError: '2017'
attached are my logs
KeyError2017.pdf
I have tried this with
electron-edge-js
electron-edge
electron 1.7.9
it seems to be with node-gyp.
I can node-gyp build at the root at electron-edge-js and electron-edge but when i try under \tools\build.bat release 7.9.0
electron-edge-js
supports Electron 1.7.x - Node.js v7.9.0.
electron-edge
Supports Electron 1.7.x - Node.js v7.9.0.
My Computer:
Win10
VS2015
Python=2.7.9
angularCLI =latest
Environment Vars:
GYP_MSVS_VERSION= 2015

run this command : meteor npm install --save @babel/runtime
All errors will disappear! Magic!!!

After trying for many days, I found that my problem is that the nodejs aerospike client uses node-gyp for building, and the client depends on libss-dev which was not installed on the box I use (VSTS Linux Build Agent).
I added the repo and installed the package before running the npm install:
add-apt-repository 'deb http://security.ubuntu.com/ubuntu xenial-security main'
sudo apt update
sudo apt install g++ libssl1.0.0 libssl-dev

<EMAIL_ADDRESS>install C:\Users\Lenovo\AppData\Roaming\npm\node_modules\appium\node_modules\heapdump
node-gyp rebuild

C:\Users\Lenovo\AppData\Roaming\npm\node_modules\appium\node_modules\heapdump>if not defined npm_config_node_gyp (node "C:\Program Files (x86)\nodejs\node_modules\npm\node_modules\npm-lifecycle\node-gyp-bin\....\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "C:\Program Files (x86)\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" rebuild )
gyp ERR! configure error
gyp ERR! stack Error: Can't find Python executable "python", you can set the PYTHON env variable.
gyp ERR! stack     at PythonFinder.failNoPython (C:\Program Files (x86)\nodejs\node_modules\npm\node_modules\node-gyp\lib\configure.js:483:19)
gyp ERR! stack     at PythonFinder. (C:\Program Files (x86)\nodejs\node_modules\npm\node_modules\node-gyp\lib\configure.js:508:16)
gyp ERR! stack     at C:\Program Files (x86)\nodejs\node_modules\npm\node_modules\graceful-fs\polyfills.js:284:29
gyp ERR! stack     at FSReqWrap.oncomplete (fs.js:152:21)
gyp ERR!  Try to UPDATE NPM (npm install -g npm) that's crazy how many days I lost because of that. Couldn't you check that and advice us to do that somewhere (when the package fails maybe and in the readme). Anyway thanks @refack.
Everything has broken when I update my OS. I reinstalled everything from node to meteor but also all packages deleted node-gyp, reinstalled node-gyp, every tools required via every possible way and did one thousand other things

@NitroBAY Thank you!!! npm install -g npm solved!

only this option helped https://github.com/nodejs/node-gyp#option-2

Be careful, You have to use Python 2.7 (and not 3.7 which is the current latest version).

after install tools
npm install --global --production windows-build-tools

You need to set new python path  in environment-variable PATH. be sure not to install different version of python.

whene i run this command ::::: npm i --s firebase @angular/fire
i get this error :;:
Failed to execute 'C:\Program Files\nodejs\node.exe C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js configure --fallback-to-build --library=static_library --module=C:\Users\saado\Desktop\Angular-Crud\AngularFirestore\node_modules\grpc\src\node\extension_binary\node-v64-win32-x64-unknown\grpc_node.node --module_name=grpc_node --module_path=C:\Users\saado\Desktop\Angular-Crud\AngularFirestore\node_modules\grpc\src\node\extension_binary\node-v64-win32-x64-unknown --napi_version=3 --node_abi_napi=napi --napi_build_version=0 --node_napi_label=node-v64' (1)

I had installing windows-building-tools  , python2.7 and the error still occurred.
But after updating npm with npm i -g npm , installing gone done!!!

On Mac OS doing :
nvm install 10.13.0        nvm use 10.13.0
solved for me the problem.

On Windows, what I did to fix it were the following:

Download the latest Node installer for Windows.
Install it.
At this point, the node-gyp issue is still not resolved.
Go to my C:\Users\MY_USERNAME\AppData\Roaming.
Delete the npm folder.
Run the same installer in step 1, to repair missing files, registry settings.
Once finished, npm install -g npm.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: concurrently "ng serve --proxy-config proxy.conf.json" "json-server --watch db.json --routes routes.json"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\dell\AppData\Roaming\npm-cache_logs\2020-04-29T00_57_00_039Z-debug.log

Can you help me ??

From the node-gyp docs
These are the steps if you are on Windows to solve the issue

Download latest python version (If you're getting issues try python 2.7)
Download latest MSVS here
Install just the build tools or install the entire Visual Studio installation using the "Desktop development with C++" workload
Go to your project folder and run npm config set msvs_version {{ version of msvs you installed }}. So if you installed the 2019 edition the command would be npm config set msvs_version 2019.
Finally run npm install -g node-gyp and it should work!


Cannot read property 'runtime-images' of undefined

BUG REPORT
Migrating this from serverless repo https://github.com/serverless/serverless-kubeless/issues/140#issuecomment-526726537 as they state it is a problem with Kubeless.
Running Kubernetes 1.13 on Amazon EKS with Kubelss v1.0.4.
What happened:
serverless deploy -v
Serverless: Packaging service...
Serverless: Excluding development dependencies...
Unable to find required information for authenticating against the cluster
Unable to find required information for authenticating against the cluster

  Type Error ---------------------------------------------

  TypeError: Cannot read property 'runtime-images' of undefined

What you expected to happen:
Deploy successfully to Kubernetes cluster running on AWS EKS.
Details:
Setting the two required envar's KUBECONFIG and AWS_PROFILE.
Then I am able to run kubectl commands:
kubectl get pods --namespace kubeless
NAME                                          READY   STATUS    RESTARTS   AGE
kafka-0                                       1/1     Running   0          7m41s
kafka-trigger-controller-69cf947d6b-q62jg     1/1     Running   0          7m41s
kubeless-controller-manager-646876977-qb46g   3/3     Running   0          3d19h
zoo-0                                         1/1     Running   0          7m40s

I am using aws-iam-authenticator could that be the issue?

I am using aws-iam-authenticator could that be the issue?

Yes, the amount of authentication methods supported by the plugin is quite limited. That's likely caused by an old version of the package kubernetes-client that we are using there. An upgrade will probably help but we never had the time to contribute to that. External contributions are welcomed! :)

@andresmgot I am happy to try and assist as this is blocking me from run serverless on a Kubernetes cluster on Amazon EKS. Can you perhaps point me in the right direction?

@nodesocket sure. Let me guide you through the code. Note that this only affects to the serveleress-kubeless plugin so all the code is in that repository. So the code is failing here:
https://github.com/serverless/serverless-kubeless/blob/master/lib/config.js#L65
  get(key, opt) {
    if (opt && opt.parse) {
      return JSON.parse(this.configMag.data[key]);
    }
    return this.configMag.data[key];
  }

That is because configMap is undefined because it was not able to retrieve the Kubeless configuration. TBH, that should fail before, when it's not able to resolve configMap but that's a different issue.
This configMap is in the same file. And it's not able to retrieve it because here:
    this.connectionOptions = Object.assign(
      helpers.getConnectionOptions(helpers.loadKubeConfig()),
      { url, json: true }
    );

Those connection options are not properly configured to use aws-iam-authenticator. The code that resolves the authentication info is that function helpers.getConnectionOptions. You can find the code here:
https://github.com/serverless/serverless-kubeless/blob/master/lib/helpers.js#L129
As you can see in that function, the supported mechanisms to authenticate are either using a token, a user and pasword or a client-certificate.
So having that we can either:

Discard that getConnectionOptions implementation and use a similar method of the library kubernetes-client for NodeJS. I assume that they have something to be able to connect to the cluster and retrieve resources like the configMap we are talking.
Improve the getConnectionOptions method to support other types of auth, like the one you mention.

Hope that helps, I can answer more questions if you have them.

@andresmgot thanks for the reply. So, just to get a test working I can generate a standard token using https://docs.aws.amazon.com/cli/latest/reference/eks/get-token.html. Then do I set this token so severless will use it?

yes, that should work. Have you tried?

@andresmgot I am not sure how to set the Kube token so serverless uses it. I was able to successfully deploy the test py function using kubeless described in the quick start guide https://kubeless.io/docs/quick-start/.
However, when running serverless deploy it fails with:
Serverless: Packaging service...
Serverless: Excluding development dependencies...
Unable to find required information for authenticating against the cluster
Unable to find required information for authenticating against the cluster

  Type Error ---------------------------------------------

  TypeError: Cannot read property 'runtime-images' of undefined


[DOC] Document how to load GeoJSON using Spark SQL's built-in JSON da…

…ta source.
Did you read the Contributor Guide?

Yes, I have read Contributor Rules and Contributor Development Guide

Is this PR related to a JIRA ticket?

No, this is a documentation update. The PR name follows the format [DOCS] my subject.

What changes were proposed in this PR?
Document how to use the built-in JSON data source in Spark SQL to read GeoJSON
How was this patch tested?
Did this PR include necessary documentation updates?

I also added documentation on how to work with JDBC data sources.
Reading geometries should be quite general across different databases. Writing I think is more specific. I only added documentation on how to write to Postgis.

I accidentally pushed some other changes. Now this PR only contains documentation updates. I really shouldn't multitask.


How can i get geometric shape

How can i detect if a layer is a rectangle or a circle? Where are this parameter in Layer object.
Many thanks!

Hi,
First of all, if the layer is a shape you should be able to extract information from:
node.get('vectorOrigination')  or even node.get('solidColor')

Then if you parse it:
node.get('vectorOrigination').data
you can extract the following:
{ class: { name: '', id: 'null' },
  keyDescriptorList:
   [ { class: [Object],
       keyOriginType: 1,
       keyOriginShapeBBox: [Object],
       keyOriginIndex: 0 } ] }

The "keyOriginType" property represent the type of shape.

Rectangle : 1
Rounded Rectangle : 2
Ellipse/Circle : 5
Line : 4

I didn't find an API reference.


small bugfixes

can you request a review and a reviewer when you're done?

unfortunately I don't think anyone but you has the rights to do that!
no option to do so here:


go get install error

from go-github-selfupdate..
https://github.com/rhysd/go-github-selfupdate/issues/30

https://github.com/google/go-github/issues/1549


Get-IisLog.ps1 should better support nonstandard methods

Right now, the $_ variable is being used, except that's the exception, not the current item.
PropertyNotFoundException: A:\bscripts\Get-IisLog.ps1:269
Line |
 269 |          catch{Write-Verbose "Nonstandard verb $($_.Method) at $($_.Fi …
     |                                                  ~~~~~~~~~
     | The property 'Method' cannot be found on this object. Verify that the property exists.

A new filter should be used to "deconstruct" and type the fields.

Fixed!


How to config maximum range can be selected

How to config maximum range can be selected.
Example: maximum 30 days

Bump


External Access for Shopcart Microservice

As a developer
I need to create a secure, scalable route in OpenShift for the shopcart microservice
So that customers can interact with their shopping carts through our e-commerce platform's frontend, enhancing user experience.
Details and Assumptions

The shopcart microservice is developed with RESTful API endpoints for cart operations.
TLS termination is set up at the route for secure HTTPS communications.
Load balancing is considered for handling peak traffic loads effectively.

Acceptance Criteria
Given the shopcart microservice is deployed in an OpenShift pod
When I create a secure and scalable route with TLS termination for the service
Then customers can securely access and manage their shopping carts through the e-commerce platform's frontend interface

https://shopcarts-whykay-01-dev.apps.sandbox-m4.g2pi.p1.openshiftapps.com/


{neatvnc,wayvnc}: 0.7.2 -> 0.8.0

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 295423 run on x86_64-linux 1

6 packages built:

neatvnc
wayvnc
weston
westonLite
xwayland-run
xwayland-run.man

Tested weston vnc backend.


Bug: Making new transaltion of a content does not copy values from master to translated version

By clicking on translation link, I expected all values present in master content will be copied into new translation content, as in 1.8.1 feature.
this does not happen in 1.9.x version. Seems to be a bug...

Can you give us the steps to reproduce this. I am currently fixing the problem with Widgets, is it doing this somewhere else ?

Yes, here the steps:

Enable Content Localization
Add another culture to site (it-IT in my case)
Edit a Content ( a page for example )
Set a culture to it (en-US)
Publish content
click + New translation
No fields will be filled up with master content values (Title is empty, Body is empty, etc...)

I'm not sure we want to copy those values in that specific scenario since they will be replaced with translated content. This makes more sense with Widgets for instance; where you need to copy the Layer and Zone to the Widget you are translating. Though let's see other people opinions about this. I think other CMS's like Wordpress gives a checkbox option to copy those values or not when you are translating a content item, this could be how we could implement this.

In the case of a content with fields and parts that need translations, you are right.
But imagine to have a content with many fields (Numeric Fields, Enum fields, Media Library picker fields image and so on) and/or parts that don't need for a translation. You have to re-enter all values and contents.
In Orchard 1.8.1 (and previous versions) all data were copied from master to translation.
So, i agree for a checkbox, but the default behavior should be "copy contents from master to translation". let's see other people opinions. Thanks.

@Jetski5822

@sfmskywalker Can we copy Layouts from one content item to another easily ? @Jetski5822 has it been dropped in 1.9.1 because of the changes made in the Localization features ? Would it be a proper way to use the handler to do this then ?

This feature never existed.

That's what I thought too. I don't remember that in 1.8.x it was copying the content. Though that's a good feature idea to consider implementing. I'm asking myself now if this is a specific case scenario or if it would be needed everytime we have a LocalizationPart on a content item. If it's the second then, we should maybe consider doing something more proper than what I did with the Widgets.

I am quite sure this feature was present:
at this history point, logic within Translate Action was changed:
https://github.com/OrchardCMS/Orchard/commit/fa1de09a4f5688f2fb4fe9abd270e1a0106d1dde
before:
MasterContent was used as Model for buildeditor of AdminController/Translate. The effect was that the MasterContent worked as initial value for the translation... may be this could help...

@Skrypt Yes very easily, simply copy the LayoutData and TemplateId properties over (string and int?).

So here it's not about copying the properties, but using the master content item to modelbind the UI, which I think is he best solution. We should fix the regression.

Yes thanks @sebastienros  for the precision

I need to revert the fix as this is preventing from selecting the translation. The real solution needs more work. I think we need to leverage the Clone function then to initialize the cloned culture. But then we need to change Clone in two ways:

not save the content item as it is done by default during a cloning
extend the clone action with some events so that parts like Autoroute can reinitialize the value, like when the cloned content is the homepage.

@sebastienros I worked on a PR to add clone handling, and also solving this translation issue. #6540


Add guidance on decorative images in our automated message

What
A decorative image should not have alt text and instead should have null alt text: alt=''.
Currently we do not have guidance on decorative images in our automated message. Let's add some!

@kendallgassner Since images on GitHub all render as a link, should we continue to require alt text for images to avoid links with no name, for now?

Oh good thought 🤔!
Maybe we can add that as a note instead.


New Remote Connect UI: Connect to XXXX server command can get lost at the bottom of the kernel picker.

After using new remote connection Ux, I found it to be confusing when I cannot see that I can connect back to local or another remote. The commands at the bottom should either be on the top, or at least somewhere discoverable.

The Jupyter extension cannot control this placement.

This is now handled by the new MRU kernel picker.


[1.4] Use PodDisruptionBudget v1

Backport of https://github.com/knative-sandbox/eventing-kafka/pull/1278

Need to merge this one now, because there are some issues related to it in Serverless Operator, after I had to upgrade eventing-kafka (not broker) there


gf.path.transition(width_type='linear') coming out with nonlinear width variation for some cases

Describe the bug
GDSFactory 7.23.0, Python 3.10.  Apologies if it's been corrected, I will upgrade to the latest version after my tapeout.
I'm using gf.path.transition, between two different gf.cross_section.rib_width_trenches.
Despite the two cross-sections having the same width=1, the actual width varies between ~0.58 and ~1.44.
To Reproduce
import gdsfactory as gf

xs1 = gf.cross_section.rib_with_trenches(width=1,width_trench=5,layer=(1,0),layer_trench=(2,0))
xs2 = gf.cross_section.rib_with_trenches(width=1,width_trench=1,layer=(1,0),layer_trench=(2,0))
xst = gf.path.transition(xs1,xs2,width_type='linear')
xs1b = gf.cross_section.CrossSection(sections=[gf.cross_section.Section(width=5, layer=(1,0))])
xs2b = gf.cross_section.CrossSection(sections=[gf.cross_section.Section(width=1, layer=(1,0))])
xstb = gf.path.transition(xs1b,xs2b,width_type='linear')
path1 = gf.path.straight(length=100)
path2 = gf.path.euler(angle=90,p=0.2,radius=50,use_eff=True)
path3 = gf.path.arc(angle=90,radius=50)
top = gf.Component()
e1 = top << gf.path.extrude(path1,cross_section=xst)
e2 = top << gf.path.extrude(path2,cross_section=xst)
e3 = top << gf.path.extrude(path3,cross_section=xst)
e4 = top << gf.path.extrude(path3,cross_section=xstb)
e2.move((0,-75))
e3.move((0,-150))
e4.move((0,-225))
top.show()

Image: From top to bottom: Top (straight) is fine, Second (Euler) and Third (Arc) have erroneous width variation, Fourth (Arc using single layer waveguide) is fine.
Expected behavior
Slab total width changes linearly. waveguide width changes linearly, which in this case is staying constant.
Suggested fix
Update to gf.cross_section module, assuming it's not already fixed in v8.

hi @jonnyroth ,
interesting... not sure why exactly this is, but I imagine it must have to do with the offset in the trench section. i never usually use this cross section. i would usually draw as shown below, and then derive the trench via a boolean. please see the modified sample script:
import gdsfactory as gf

def my_rib_with_trenches(width=1, width_trench=5, slab_offset=0.3, layer=(1,0), layer_trench=(2,0), layer_slab=(3,0)):
    core_width = width
    trench_width_full = 2* width_trench + core_width
    slab_width_full = trench_width_full + 2 * slab_offset
    return gf.cross_section.CrossSection(sections=[
        gf.cross_section.Section(name="rib",width=core_width, layer=layer),
        gf.cross_section.Section(name="trench",width=trench_width_full, layer=layer_trench),
        gf.cross_section.Section(name="slab", width=slab_width_full, layer=layer_slab)])

xs1 = my_rib_with_trenches(width=1,width_trench=5,layer=(1,0),layer_trench=(2,0))
xs2 = my_rib_with_trenches(width=1,width_trench=1,layer=(1,0),layer_trench=(2,0))
xst = gf.path.transition(xs1,xs2,width_type='linear')

path1 = gf.path.straight(length=100)
path2 = gf.path.euler(angle=90,p=0.2,radius=50,use_eff=True)
path3 = gf.path.arc(angle=90,radius=50)
top = gf.Component()
e1 = top << gf.path.extrude(path1,cross_section=xst)
e2 = top << gf.path.extrude(path2,cross_section=xst)
e3 = top << gf.path.extrude(path3,cross_section=xst)
e2.move((0,-75))
e3.move((0,-150))
top.show()

I've run @jonnyroth 's sample code (with some minor tweaks to get it in a compatible state, listed at the end of this comment)  and gdsfactory v8.1 appears to have the same behavior

From what I can gather, this seems to be an issue with Path._centerpoint_offset_curve(...)
To try to help track down the issue, I ran some data analysis on some variables of interest within Path.extrude_transition(...)

A couple things of note:

There are some deviations in the values of dy between the upper and lower trenches, though these deviations are much smaller than the deviations we see in the ridge width.
The non-constant width issue is immediately present in the trench polygons (created from the points generated by _centerpoint_offset_curve(...))
Interestingly, the "Deviation from Ideal Ridge Width" plot indicates that the deviation very closely approximates a sin function, though I have not run the analysis under different conditions to make sure this isn't just coincidence)

I'm curious if anyone else has an idea as to why this might be the case - granted I haven't actually spent a whole lot of time digging into _centerpoint_offset_curve(...), so it may very well be obvious once actually looked at, but I've spent way too much time drafting up this comment in the first place, so I'll leave that part to someone else.

This is actually my first time seeing this issue because I do basically the same thing that @tvt173 does. As he explained in his comment, just block out the general shape you want, draw out the trenches, and then do some boolean subtraction to get the final shape.

code used to generate the analysis plots (note that any variable with _upper or _lower in its name denotes that it is derived from the trench_1 or trench_2 sections, respectively):
fig, ax = plt.subplots(1, 3, figsize=(15,5))

ax[0].plot(width_upper / 2, label="Upper Trench")
ax[0].plot(width_lower / 2, label="Lower Trench")
ax[0].set_xlabel("Point Index")
ax[0].set_ylabel("'$dy$' (um)")
ax[0].title.set_text("Trench Width Offsets")
ax[0].legend()

ax[1].plot(upper_trench[:, 0], upper_trench[:, 1], label="Upper Trench")  # taken from 'points_poly'
ax[1].plot(lower_trench[:, 0], lower_trench[:, 1], label="Lower Trench")  # taken from 'points_poly'
ax[1].set_xlabel("X (um)")
ax[1].set_ylabel("Y (um)")
ax[1].title.set_text("Upper & Lower Trench Polygons")
ax[1].legend()

# First glace, this line doesn't seem right to me, but plotting the raw upper and lower trench data indicates that 'points1_upper'
# and 'points2_lower' correspond to the inner edges of the polygons
s_diff = np.sqrt((points1_upper[:, 0] - points2_lower[:, 0]) ** 2 + (points1_upper[:, 1] - points2_lower[:, 1]) ** 2)

ax[2].plot(s_norm := s_diff - 1, linewidth=2)
x_min, x_max = ax[2].get_xlim()
ax[2].plot([-100, 1000], [0, 0], 'k',ls='dotted', linewidth=1)
ax[2].set_xlabel("Point Index")
ax[2].set_ylabel("$\Delta$w (um)")
ax[2].set_xlim(x_min, x_max)
ax[2].set_yticks([s_norm.min(), 0, s_norm.max()])
ax[2].title.set_text("Deviation from Ideal Ridge Width")

fig.tight_layout()
fig.show()

Minor tweaks to sample code needed for gdsfactory v8.1 compatibility:

Provide wg_marking_layer to xs1 and xs2: rib_with_trenches type hints it as optional, but since wg_marking_layer is given as the main layer of the cross-section, it cannot be None in order to extrude the cross-section
xs1b and xs2b both need to have a name parameter defined in the Section defined in their CrossSections. The same name must be used for both


Remove duplicities in kfto_training_test.go

Description

Remove volume which is not needed.
Remove duplicit environment variables.
How Has This Been Tested?

Manually by running on OCP cluster.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

Thanks @sutaakar !!


Update to work with zig-0.12.0

Also updates raylib to v5.0 (necessary to work with zig-0.12.0)

I wont update this binding to an unreleased Zig version

I wont update this binding to an unreleased Zig version

Is there some issue with just creating a separate branch? The changes to migrate from 0.11 to 0.12 are relatively trival. Missing support for the latest version seems like an important thing to ignore.
It also has the added benefit of making it easier for yourself in the future when 0.12 LTS is released in the future.

We aren't missing support for the latest version, just support for development versions. These can change dramatically without notice and I honestly just can't be asked to keep up with that, given that the main repo is often updated 10 times a day.


PR: Fix variable explorer sort by size bug

Description of Changes
This PR fixes a bug where columns in the variable explorer could get out of sync when sorting on size.
Issue(s) Resolved
Fixes #14155
Affirmation
By submitting this Pull Request and typing my username below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.
I certify the above statement is true and correct: hengin

Hello @hengin! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/plugins/variableexplorer/widgets/collectionseditor.py:

Line 288:80: E501 line too long (95 > 79 characters)


feat: allow setting container memory & cpu from task memory & cpu

Description
With the addition of #162 if a user includes extra container definitions and those definitions require CPU/Memory allocations the deployment will fail as the atlantis container definition takes the entire task cpu and memory allocation.
Motivation and Context
This allows a user to optionally fine tune the allocations of cpu and memory for the atlantis container definition. Most users won't require this feature, unless they're using extra_container_definitions.
fixes #168
Breaking Changes
This does not break compatibility as by default the values are null and if null they will default to the ecs_task_* definitions, as it was before this change.
How Has This Been Tested?
module "datadog_fargate_container_definition" {
  source  = "cloudposse/ecs-container-definition/aws"
  version = "v0.41.0"

  container_name  = "datadog-agent"
  container_image = "datadog/agent:latest"

  container_memory_reservation = 256
  container_cpu                = 10

  essential = true

  environment = [
    {
      name  = "ECS_FARGATE"
      value = "true"
    }
  ]
}

module "atlantis" {
  source = "github.com/stacklet/terraform-aws-atlantis?ref=issue-168"

  name = "atlantis"

  # DNS
  route53_zone_name = "example.com"

  # Atlantis
  extra_container_definitions = [module.datadog_fargate_container_definition.json_map_object]
  atlantis_github_user           = "marcoceppi"
  atlantis_github_user_token     = data.aws_kms_secrets.atlantis.plaintext["github_user_token"]

  atlantis_hide_prev_plan_comments = "true"

  container_cpu    = 502
  container_memory = 768

  ecs_task_cpu    = 512
  ecs_task_memory = 1024

  allow_repo_config = "true"
}

hi @marcoceppi - thanks for the PR. could you update the github-complete example project to reflect the changes here so that we can both capture how they are utilized as well as test the changes. thank you!

Can and will do!

@bryantbiggs I've rebased and updated github-complete with the new values

@bryantbiggs I've rebased and updated github-complete with the new values

Thanks for the review @bryantbiggs I've updated the description strings and rebased on the latest in master and fixed the errors from pre-commit 0.14.7 check

Thanks @marcoceppi !
v2.33.0 has been just released.


Support for workdir on S3

Resolves #1437.
Changes in bcbio:

Refactoring of distributed/transaction.py: break up the code into smaller functions, add unit tests.
Change default tmp dir location to /tmp/bcbiotx. We do not want to create a tmp dir in a working directory (unless specified otherwise in a config file).
Attempt to unify the way file_transaction is used in the codebase. To make the behaviour of file_transaction dependent on configuration provided by user, we must always pass the config as an argument to the file_transaction. Otherwise, the behaviour specified in config cannot be consistently enforced. Only small number of file_transaction calls was changed.
When moving a file from temp dir to a work dir, create an empty "flag" file, then check the sizes of the files are equal, and remove the "flag" file. In case of error, the flag file remains in the working directory.
Wrap certain steps of rnaseq and variant calling pipelines into file_transaction (see the changes in code).
Minor bug fixes in multiqc, utils, alignprep
Add integration tests for the pipelines which must work with a workdir on S3. To test if a pipeline works with a workdir on S3, one has to mount a bucket with goofys and change a work dir for tests. I've added a mount.sh script to tests, which should be run with source path/to/mount.sh. Sadly, running such tests cannot be a part of CI, because it requires access to and S3 bucket.

Installation of goofys fs and mounting of S3 bucket should be a part of provisioning, so it is not included in the bcbio.
Changes in test suite:

Split tests into unit (do not require any additional dependencies, test only bcbio module) and integration (require tools and input data, perform disk IO, etc)
Keep tests which depend bcbio_nextgen_vm separately
Use py.test as a test running framework, because it is simpler, cleaner, outputs much more useful messages for failed tests, has a number of useful plugins, and takes full advantage of python's dynamicity.
Add test coverage to CI builds, separately for unit and integration tests.
Run ALL tests in CI that do not require installation of additional tools (like GATK) or databases. Integration tests are run in the docker container, built on top of bcbio/bcbio using the Dockerfile_dev.

WARNING!  To make some tests pass, I've had to install/upgrade the following dependencies:
multiqc-bcbio=0.1.6
goleft=0.1.6
tophat-recondition=1.4
As the list of dependencies is not a part of the codebase, I cannot include the requirement for those versions into the pull request.
WARNING! Some tests are currently failing (see the CI builds for logs). Those tests are related to the parts of bcbio that I didn't have a chance to work with, and although I did my best to make them pass, I couldn't figure out how to do that in case of test_7_cancer from test_automated_analysis.py and test_5_find_fastq_pairs from test_pipeline.py.

Tetiana;
This is so incredible, thanks for all the work that went into this. We pushed out the latest release and would be happy to get this merged. I think a couple of recent changes caused conflicts -- if you're able to get up to date with the latest master and resolve these we can get this in.
My major concern with the functionality changes is the decision to use a global temporary directory (/tmp/bcbiotx) as a default instead of local. I understand how this would be useful for your case, but for the currently more common shared filesystem case the other default is much more likely to work out of the box. /tmp is pretty likely to get filled up on most systems with big workflows, hence the default to a local disk unless specified otherwise. Could we swap this back to using the current default with an easy way to switch over to the global temporary directory?
For testing, would it also be possible to update the documentation for running the tests (http://bcbio-nextgen.readthedocs.io/en/latest/contents/testing.html#test-suite). I'm also sad to see run_tests.sh go as it avoiding folks needing to have the testing framework installed separately, although we could also add py.test as a top level dependency so it's present in the PATH alongside bcbio_nextgen.py. Did you have a specific reason for removing the small ./run_tests.sh wrapper.
Thanks again for this, definitely happy to discuss and get this pushed as soon as we can so folks can begin using it.

I agree, local temporary disks should be used wherever possible to prepare data for transfer to S3. @tetianakh, I believe that is also what we intended, isn't it?

@chapmanb thank you very much for the feedback! Sorry for the ling reply, I was on a conference, but now I'll implement the fixes ASAP.

@chapmanb why is it needed to pass a specific directory location for the temporary directory (the second argument for tx_tmpdir)?

Tetiana -- thanks so much for looking at these. The second argument for tx_tmpdir is optional and not required if you pass the data object since it pulls the choice from the configuration or else defaults to the current directory. base_dir just provides a different way to use this function if you didn't have a data object or wanted to specify some different location, but can generally be left as None for the work you're doing. Hope this helps.

@chapmanb
I've brought back run_tests.sh (changing the test runner inside to py.test), changed the default tmpdir to current working directory, brought back base_dir argument into tx_tmpdir, and updated the docs.
I advocate for keeping dev and testing dependencies separately from the rest of dependencies. Bcbio installs hundreds of packages -- 656 only via conda -- and throwing in something which is not required to run bcbio on production won't help in managing the dependencies. py.test and fiends can be installed in a virtual environment (or in a conda environment), eliminating the need to determine where the test runner is via run_tests.sh. The command to run tests is simply py.test,  or py.test tests/integration to run a subset of tests, so wrapping it in a script seems redundant in my humble opinion, that's why I removed it initially.

Tetiana and Sven-Eric -- thank you again for this great contribution. I merged into development and we can iterate and test with this all included. I also added you to the list of contributors, please let me know if you need a different affiliation or anything else. We really appreciate this. I'm looking forward to experimenting with the S3 mounted support and learning all about the wonders of py.test.

Tetiana;
I'm working on updating the development resources for the bcbio install so it gets everything we need for testing and have a quick question: what was the motivation for needing multiqc-bcbio, goleft and tophat-recondition with the standard bcbio install (as opposed to the tool installation)? Ideally we're only including python and other imported dependencies with bcbio and the tools are installed as a separate step.

@chapmanb yes, they belong to tools, of course you're right.


[SPARK-12862][SPARKR] Jenkins does not run R tests

Slight correction: I'm leaving sparkR as-is (ie. R file not supported) and fixed only run-tests.sh as @shivaram described.
I also assume we are going to cover all doc changes in https://issues.apache.org/jira/browse/SPARK-12846 instead of here.
@rxin @shivaram @zjffdu

Test build #49549 has started for PR 10792 at commit c50e351.

Test build #49549 has finished for PR 10792 at commit c50e351.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49549/
Test FAILed.

Test build #49552 has started for PR 10792 at commit fce7049.

Test build #49552 has finished for PR 10792 at commit fce7049.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49552/
Test PASSed.

Pretty sure test was broken by PR #10745 for SPARK-12575 committed ~ a day ago.

LGTM. Thanks @felixcheung for fixing this. Merging this to master


innernet: 1.5.4 -> 1.5.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/tonarino/innernet/releases.
meta.description for innernet is: "A private network system that uses WireGuard under the hood"
meta.homepage for innernet is: "https://github.com/tonarino/innernet"
meta.changelog for innernet is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.5.5 with grep in /nix/store/hn7iw81pidq7vxjwi65v9rl8s9rcd8xn-innernet-1.5.5
found 1.5.5 in filename of file in /nix/store/hn7iw81pidq7vxjwi65v9rl8s9rcd8xn-innernet-1.5.5

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
innernet

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hn7iw81pidq7vxjwi65v9rl8s9rcd8xn-innernet-1.5.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A innernet https://github.com/r-ryantm/nixpkgs/archive/7034d52ef87869ac46cc50e8973f192f1f5a6c2b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hn7iw81pidq7vxjwi65v9rl8s9rcd8xn-innernet-1.5.5
ls -la /nix/store/hn7iw81pidq7vxjwi65v9rl8s9rcd8xn-innernet-1.5.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

innernet

Maintainer pings
cc @tomberek @0x4A6F for testing.

Result of nixpkgs-review pr 192776 run on x86_64-linux 1

1 package built:

innernet


Wk 9 v1.2

Copy multiple emails to clipboard

Coverage decreased (-2.0%) to 90.77% when pulling e866d326332d48eb2c13db5e7ec68ad774b495ee on WK-9-V1.2 into e87446344320eb3ef2d8db958bb8678a1c9c42c4 on master.


Fix and renable safetydb importer

Fixes https://github.com/nexB/vulnerablecode/issues/244
Signed-off-by: Shivam Sandbhor<EMAIL_ADDRESS>

@pombredanne  from the discussion we had, you suggested we handle this  by using a config file which contains a list of licenses.
If an importer's license is not present in this list either run/don't run the importer.
I think this logic belongs in the importer_yielder.py file, no need to create another config file.

Commits included in https://github.com/nexB/vulnerablecode/pull/259. Hence closing


As a user a find it a bit confusing that I have to click (somewhere on the page) once I finish filtering. Could we have an OK and Canbel button instead?

An example of what I mean can be seen in the screenshot below or here: https://www.efsa.europa.eu/en/microstrategy/salmonella-dashboard
Got to the sub-page "Total untits tested of major fodd category" then check filter for food category as an example

An even nicer example can be seen on the BVl portal. Check the attached screenshot or here: https://gis.bvl.bund.de/
The select "Gast-Zugang", then "Zoonosen-Monitoring Dashboard", then "Zooosen_Monitoring Prävalenzen" --> "Hauptfilter" and look at the "Tierart/Matrix" filter window that opens. It seems very complex, so I understand if we can't implement something like it. I just wanted to show it,

there is Icon for open and close . we can explain it in the evaluation information)

Can we make the Icon to open and close the filter a bit bigger maybe to make it more clear?

Icon exists - following Material design standards


Allow for easy switching between Collectives

It is currently quite annoying when switching between collectives, especially with using the app from a homescreen icon.
It would be nice to store tokens for all currently logged in collectives and allow for a quick change. One way to do that could be by scoping all localStorage values by collective id so that we can store as many as wanted next to each other.

Changing still needs to logout and log back in, but by displaying visited Collective names and giving a back-button on the login screen this is already much easier now.


"posts_per_page": 6 doesn't work

i set  "posts_per_page": 6 , but  doesn't work
home page return all posts
Can you help me？

Casper has an infinite scroll plugin, if you look carefully only the first 6 posts will be loads, but as you scroll the rest would come in. You can remove this line from default.hbs to stop that https://github.com/TryGhost/Casper/blob/master/default.hbs#L79


Serious reliability issue

Thanks for providing this solution. Everything works fine initially.
SELECT * FROM replication_group_members was showing all group members online.

group_replication_applier
ab5c8ecd-ed92-11e8-9445-2e3b2c86be4a
mysql-0.mysql
3306
ONLINE

group_replication_applier
caddcd94-ed92-11e8-9a8b-f667d83e57a5
mysql-1.mysql
3306
ONLINE

group_replication_applier
f03a662f-ed92-11e8-9a13-c278ec5cf7bd
mysql-2.mysql
3306
ONLINE

But when, I tried to play around it & deleted 2 PODS mysql-1 & mysql-2. It was still showing

group_replication_applier
ab5c8ecd-ed92-11e8-9445-2e3b2c86be4a
mysql-0.mysql
3306
ONLINE

group_replication_applier
caddcd94-ed92-11e8-9a8b-f667d83e57a5
mysql-1.mysql
3306
UNREACHABLE

group_replication_applier
f03a662f-ed92-11e8-9a13-c278ec5cf7bd
mysql-2.mysql
3306
ONLINE

After that, I am not able to execute write/insert query. Read operations are working fine.
Please let me know, if someone has tried similar thing & faced such issue. Any solution?

This repo is DEPRECATED
Please use official MySQL operator instead --> https://github.com/oracle/mysql-operator


Inception project contains outdated references to I did it button

PR: https://github.com/NationalSecurityAgency/skills-service/pull/2038


lnrpc: add additional REST binding for QueryRoutes

Lists with non-primitive members aren't supported in the query string of a GET request with the current version of the grpc-gateway library. To allow route_hints to be set through REST, we also offer a POST endpoint for that call where the encoding of the request parameter can be specified as JSON.
Fixes https://github.com/lightningnetwork/lnd/issues/6752.

@guggero added the milestone on the issue instead


Support automatic plugin loading in Data Prepper core

Is your feature request related to a problem? Please describe.
Data Prepper plugins can depend on other Data Prepper plugins. They load these plugins manually using the PluginModel.
This results in duplicated logic such as the following:
https://github.com/opensearch-project/data-prepper/blob/91b6666512805bf502186bd683ae800b0943ba10/data-prepper-plugins/aggregate-processor/src/main/java/org/opensearch/dataprepper/plugins/processor/aggregate/AggregateProcessor.java#L83-L87
More importantly, it is also not compatible with our new support for schema and documentation generation.
Describe the solution you'd like
Provide a feature in Data Prepper core for loading plugins on behalf of other plugins. Data Prepper can have a new @UsesDataPrepperPlugin annotation which states that the type is a Data Prepper plugin.
In plugin configurations, use the desired plugin interface type. For example, replace
https://github.com/opensearch-project/data-prepper/blob/91b6666512805bf502186bd683ae800b0943ba10/data-prepper-plugins/aggregate-processor/src/main/java/org/opensearch/dataprepper/plugins/processor/aggregate/AggregateProcessorConfig.java#L31-L34
with:
    @JsonProperty("action")
    @NotNull
    @UsesDataPrepperPlugin
    private AggregateAction aggregateAction;

When loading plugins in Data Prepper core, detect this annotation and load the plugin.

A slightly different proposal which makes implementation easier without intrusive change in data-prepper-core:
We will keep the original
@JsonProperty("action")
@NotNull
@UsesDataPrepperPlugin(pluginType = AggregateAction.class)
private PluginModel aggregateAction;

Then we will register a custom deserializer and schema generator for PluginModel.class. The custom deserializer will validate PluginModel::name against the allowed pluginName in the annotated pluginType (allowed plugin names can be retrieved through reflection since they are implementations of the AggregateAction interface)


[Feature Request] Add a builtin rate limiter

As described in the API document

API may be requested at most 1 time per two seconds

Can the sdk also add a builtin rate limiter for this?

@Zxilly That's a great idea. I'll try to work on it when I have some time.


API documentation for ProwJobs

It would be great if the docs site had a API docs, especially API reference for ProwJobs would be very useful. Something similar to https://kubernetes.io/docs/reference/kubernetes-api/workload-resources/pod-v1/

For anyone's information:
K8s API reference docs are generated using some generators.
These are some related links:

Generating Reference Documentation Quickstart Guide

update-imported-docs.py

https://github.com/kubernetes-sigs/reference-docs
*gen* files & directory in https://github.com/kubernetes/kubernetes/tree/master/hack
*gen* directories in https://github.com/kubernetes/kubernetes/tree/master/cmd
...

The documentation is linked here:
https://docs.prow.k8s.io/docs/jobs/
Click on PodSpec api reference

There's a link to the ProwJob source code here: https://docs.prow.k8s.io/docs/life-of-a-prow-job/
Not as nice as the actual doc but it does the job.

/sig documentation


🐛 [ENG-1897] only generate affiliate fee msg send if amount > 0

The computed affiliate message fee can sometimes be 0. Currently, a bank send with a 0 amount will still be created for these cases causing the transaction to fail. This PR adds logic to check if the affiliate fee amount is greater than 0 before generating a bank send for it.

Added bug emoji to the PR name :)


make: *** [Makefile:106: docker] Error 1

What is the error
docker pull golang:1.13.6
1.13.6: Pulling from library/golang
Digest: sha256:04bdcb522081df9c95e5888ada7da60da18fd0b0eae9423fdf05a477e0894352
Status: Image is up to date for golang:1.13.6
docker.io/library/golang:1.13.6
docker run \
        --rm \
        -v "":"/gopath" \
        -w "/gopath/src/github.com/atlassian/gostatsd" \
        -e GOPATH="/gopath" \
        -e CGO_ENABLED=0 \
        golang:1.13.6   \
        go build -o build/bin/linux/gostatsd -ldflags "-s -X main.Version=$(git describe --abbrev=0 --tags) -X main.GitCommit=$(git rev-parse --short HEAD) -X main.BuildDate=$(date +%Y-%m-%d-%H:%M)" github.com/atlassian/gostatsd/cmd/gostatsd
can't load package: package github.com/atlassian/gostatsd/cmd/gostatsd: cannot find package "github.com/atlassian/gostatsd/cmd/gostatsd" in any of:
        /usr/local/go/src/github.com/atlassian/gostatsd/cmd/gostatsd (from $GOROOT)
        /gopath/src/github.com/atlassian/gostatsd/cmd/gostatsd (from $GOPATH)
make: *** [Makefile:106: docker] Error 1

Reproducing the error
By following the README documentation, I ran make setup -> make build. make run which uses stdout as the backend works for my machine. However, after I ran make run-docker, the above error occurs.
My operating system is POP_OS 20.04, the version of go is 1.13.6, docker is 19.03.13.
What have I tried
Because my GOPATH is /home/alex/go, so I tried changing the GP variable in Makefile from /gopath to /home/alex/go, then I ran make run-docker again and the same error occurs.

From -v "":"/gopath", it looks like you don't have a GOPATH set in your shell.  The result of this is no /gopath volume being mounted at all, and the source not being found.  $GP is the path inside docker, not outside.
Can you double check that GOPATH is an exported variable?

You are right, that's where the issue is. go env GOPATH yields /home/alex/go, but echo $GOPATH gives an empty string, so it looks like I have to have GOPATH as an exported variable in the shell instead.
The solution is then:

Edit $GOPATH as an environment variable and point it to $HOME/go. In Windows, can do it through the GUI menu. In Linux or Mac, we can do it by adding export GOPATH=$HOME/go in ~/.bashrc or ~/.zshrc


Play volume based on sound effect volume slider

don't know if its possible but it would be nice if the volume of the audio clips were based on the sound effect slider in the in game settings.

Awaiting plugin-hub approval at runelite/plugin-hub#2137
It is a config option within the plugin settings, not tied to the in-game volumes.


Add support for running a commandline in another WT window

Summary of the Pull Request
If you're reading this PR and haven't signed off on #8135, go there first.

This provides the basic parts of the implementation of #4472. Namely:

We add support for the --window,-w <window-id> argument to wt.exe, to allow a commandline to be given to another window.

If window-id is 0, run the given commands in the current window.
If window-id is a negative number, run the commands in a new Terminal window.
If window-id is the ID of an existing window, then run the commandline in that window.
If window-id is not the ID of an existing window, create a new window. That window will be assigned the ID provided in the commandline. The provided subcommands will be run in that new window.
If window-id is omitted, then create a new window.

References

Spec: #8135
Megathread: #5000
Project: projects/5

PR Checklist

[x] Closes #4472
[x] I work here
[x] Tests added/passed
[ ] Requires documentation to be updated - sure does

Detailed Description of the Pull Request / Additional comments
Note that wt -w 1 -d c:\foo cmd.exe does work, by causing window 1 to change
There are limitations, and there are plenty of things to work on in the future:

[ ] We don't support names for windows yet
[ ] We don't support window glomming by default, or a setting to configure what happens when -w is omitted. I thought it best to lay the groundwork first, then come back to that.
[ ] -w 0 currently just uses the "last activated" window, not "the current". There's more follow-up work to try and smartly find the actual window we're being called from.
[ ] Basically anything else that's listed in projects/5.

I'm cutting this PR where it currently is, because this is already a huge PR. I believe the remaining tasks will all be easier to land, once this is in.
Validation Steps Performed
I've been creating windows, and closing them, and running cmdlines for a while now. I'm gonna keep doing that while the PR is open, till no bugs remain.
TODOs

[ ] There are a bunch of GetID, GetPID calls that aren't try/caught 😬

[x] Monarch.cpp
[x] Peasant.cpp
[x] WindowManager.cpp
[ ] AppHost.cpp

[ ] If the monarch gets hung, then you can't launch any Terminals 😨 We should handle this gracefully.

Proposed idea: give the Monarch some time to respond to a proposal for a commandline. If there's no response in that timeframe, this window is now a hermit, outside of society entirely. It can't be elected Monarch. It can't receive command lines. It has no ID.

Could we gracefully recover from such a state? maybe, probably not though.
Same deal if a peasant hangs, it could end up hanging the monarch, right? Like if you do wt -w 2, and 2 is hung, then does the monarch get hung waiting on the hung peasant?

After talking with @miniksa, we're gonna punt this from the initial implementation. If people legit hit this in the wild, we'll fix it then.

Hey @jsoref, any ideas why the spellcheck bot seems to have exploded in the last couple of days? 147b187 has a comment (that's been minimized) where the bot just marks a ton of misspellings, but on text that's... not in the project?

@jsoref I'm not sure if I've figured it out. #8962 is the first PR who's commit failed the spellcheck. In that commit, some .PNG files were added. Not .pngs, but all-caps .PNGs. Looks like the bot might not filter those out? That's at least my theory.

Thanks. Made #9004 to fix that and unblock everyone.

@zadjii-msft I would be interested in reparenting Windows Terminal inside another Win32 window, either with the full terminal or just a single tab. We've been doing this successfully in the past in Remote Desktop Manager with powershell.exe and regular console programs, but our generic reparenting code doesn't work with wt.exe. I know this is a separate feature request, but it would be related to the current one as a further improvement. A quick search for "reparenting" didn't yield interesting tickets, would you know if there is a ticket already covering this feature request?

@awakecoding the closest thing to that discussion we've had is over in #7084. That's kinda what you're looking for? If it's not, feel free to open a new issue to track that discussion.

@zadjii-msft: you're going to need to merge w/ main and resolve the file rename (sorry).
Also, did you want someone to fix the case on those PNGs?

@jsoref yea I know, thanks. I figured I'd let whoever the second is on this PR give a chance to review it so I only need so switch branches once.
I don't really mind the PNG thing personally, I suppose I was mostly just worried that the change from PNG to png would get Windows all confused, that the files aren't actually different. I guess I didn't know if that mattered with extensions or just filenames. I might be totally misremembering though. If you want to do it, feel free to submit a PR and I'll happily sign off ☺

I think it's actually dangerous to change just the case of a filename -- so best practice is to pick a different filename (in addition to switching to lowercase): https://stackoverflow.com/questions/1793735/change-case-of-a-file-on-windows/15640313#15640313


[SASS-3718] Upgraded UnauthorisedPensionSchemeTaxReferenceControllerISpec to new pattern.

Various refactoring to make other scenarios more succinct.
Removed unnecessary variations of summaries where the user type + preferred language was irrelevant.

[ ]  Had a QA verify that all acceptance criteria have been met, with comment(s) in the ticket?
[ ]  Hosted a group review of your implementation?
[ ]  Rebased against main?
[ ]  Ensured that your commit(s) contain the ticket key?
[ ]  Ensured that your branch is named after the ticket key?
[ ]  Successfully run the unit tests?
[ ]  Successfully run the integration tests?
[ ]  Successfully run the journey tests? If not, please add a comment explaining why it wasn't necessary.
[ ]  Ensured that the PR job has passed.

Reviewer: Before Reviewing
Have you ...

[ ]  Pulled the branch down?
[ ]  Assigned yourself to this PR?
[ ]  Moved the Jira subtask to “in review”?
[ ]  Ensured that it is rebased against main?

Developer: After Approval
Have you ...

[ ]  Rebased against main?
[ ]  Moved the Jira subtask to “in pipeline”?

Developer: After Merging
Have you ...

[ ]  Deleted the branch?
[ ]  Scheduled a demo? If not, please add a comment.

Had a QA verify that all acceptance criteria have been met, with comment(s) in the ticket?

In lieu of this, I took Anthony through the work.

Successfully run the journey tests? If not, please add a comment explaining why it wasn't necessary.

Given that only tests were modified, it was agreed that we can bypass this.


[3.2][HEAD] Directories cannot be created anymore

Godot version:
https://github.com/godotengine/godot/commit/89f57ae12244f3269c9e3fe4684e16ec1fd2c989
Note that I did not test master, it might be an issue there aswell.
OS/device including version:
Tested it on:
Linux, Manjaro. Kernel: 5.6-4
Android 9
Windows
Issue description:
After the recent changes to Directory they have to be open()-ed before use, however Directory.open() will not actually open a non-existant directory, which means that new directories cannot be created, because now Directory.create() fails with an error saying make_dir: Directory must be opened before use.
Steps to reproduce:
var dir = Directory.new()
dir.open("user://test")
dir.make_dir("user://test")

If the test folder doesn't exists you get a make_dir: Directory must be opened before use. error on dir.make_dir("user://test")'s line, and that directory won't be created.
Minimal reproduction project:
dir_open_test.zip
output for me:
If Directory doesn't exists:
Trying to open directory: user://test
dir.open("user://test"): 31
Directory doesn't exists: try create
dir.make_dir("user://test"): 3
dir.make_dir_recursive("user://test"): 3

else (I made it delete the folder for testing purposes) (remove works):
Trying to open directory: user://test
dir.open("user://test"): 0
Directory exists: Removing
dir.remove("user://test"): 0

Human readable error codes:
Error 0 = OK
Error 3 = ERR_UNCONFIGURED
Error 31 = ERR_INVALID_PARAMETER

Yeah, I stumbled upon this in #40234, and I think I didn't got to actually test this properly because the directories existed, because most usages are in the form of:
static func make_dir(path):
	var fs = Directory.new()
	if not fs.dir_exists(path):
		fs.make_dir(path)

It likely also affects make_dir_recursive.
CC @nathanfranke 🙂

It likely also affects make_dir_recursive.

Yes it does, although I managed to not state it explicitly, my minimal reproduction project tests for it.

Ran into the issue with recursive as well.
In my case, I think I worked around it by opening it to user:// when creating the directories there.
Maybe there are still circumstances when it doesn't already exist during play?

In my case, I think I worked around it by opening it to user:// when creating the directories there.

That's really clever lol


Update SpaceHardware enum

(raised on slack -internal link- by @abhishekkrthakur)
Current SpaceHardware enum is outdated. This PR updates it with "zero-a10g", "a10g-largex2" and "a10g-largex4". Note: even when not added to the enum, users can still sets them using a simple string.
I chose to add the zero-a10g (cc @cbensimon) even though it is still in restricted access. Shouldn't be much of a problem (I don't think this API is much used by non-HF users) and at least it will be forward compatible.
cc @christophe-rannou who updated this list in the backend recently.

Thanks for the quick review :)

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


Cannot add repository to HA

I'm trying to add the repository to my HA v 2024.11.2
Settings->Add-on Store->Add Repository
But it complains:

Perhaps I misunderstood, I'm a complete newbie with HA.

You need to install HACS in order to install this repository via HACS
OR
download the repository and copy the content to config\custom_components\becker


Build failure on Linux

Hey, interested to try this out!
For me it fails to build on a new flutter project as soon as the dependency is added:

Details

❯ flutter run -d linux -v
[  +43 ms] executing: uname -m
[  +21 ms] Exit code 0 from: uname -m
[        ] x86_64
[   +3 ms] executing: [/home/merritt/Applications/tools/flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[   +5 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] 19c61fed0da681ba2068c97e22add3bde38e51e4
[        ] executing: [/home/merritt/Applications/tools/flutter/] git tag --points-at 19c61fed0da681ba2068c97e22add3bde38e51e4
[   +9 ms] Exit code 0 from: git tag --points-at 19c61fed0da681ba2068c97e22add3bde38e51e4
[        ] 2.5.0-5.2.pre
[   +6 ms] executing: [/home/merritt/Applications/tools/flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[   +4 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/beta
[        ] executing: [/home/merritt/Applications/tools/flutter/] git ls-remote --get-url origin
[   +3 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[  +55 ms] executing: [/home/merritt/Applications/tools/flutter/] git rev-parse --abbrev-ref HEAD
[   +4 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] beta
[  +47 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +2 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'WindowsUwpEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +49 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[        ] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'WindowsUwpEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +47 ms] Skipping pub get: version match.
[  +14 ms] Found plugin hotkey_manager at /home/merritt/.pub-cache/hosted/pub.dartlang.org/hotkey_manager-0.0.6/
[  +71 ms] Found plugin hotkey_manager at /home/merritt/.pub-cache/hosted/pub.dartlang.org/hotkey_manager-0.0.6/
[  +14 ms] Generating /home/merritt/Downloads/flutter_hotkey_demo/android/app/src/main/java/io/flutter/plugins/GeneratedPluginRegistrant.java
[  +71 ms] Initializing file store
[   +5 ms] Skipping target: gen_localizations
[   +2 ms] Skipping target: gen_dart_plugin_registrant
[        ] Skipping target: _composite
[   +1 ms] complete
[   +3 ms] Launching lib/main.dart on Linux in debug mode...
[   +3 ms] /home/merritt/Applications/tools/flutter/bin/cache/dart-sdk/bin/dart --disable-dart-dev
/home/merritt/Applications/tools/flutter/bin/cache/artifacts/engine/linux-x64/frontend_server.dart.snapshot --sdk-root
/home/merritt/Applications/tools/flutter/bin/cache/artifacts/engine/common/flutter_patched_sdk/ --incremental --target=flutter --debugger-module-names --experimental-emit-debug-metadata
-DFLUTTER_WEB_AUTO_DETECT=true --output-dill /tmp/flutter_tools.ZEJHLO/flutter_tool.PYCTQJ/app.dill --packages /home/merritt/Downloads/flutter_hotkey_demo/.dart_tool/package_config.json
-Ddart.vm.profile=false -Ddart.vm.product=false --enable-asserts --track-widget-creation --filesystem-scheme org-dartlang-root --initialize-from-dill
build/c075001b96339384a97db4862b8ab8db.cache.dill.track.dill --enable-experiment=alternative-invalidation-strategy
[  +18 ms] Building Linux application...
[  +14 ms] <- compile package:flutter_hotkey_demo/main.dart
[   +1 ms] executing: [build/linux/x64/debug/] cmake -G Ninja -DCMAKE_BUILD_TYPE=Debug -DFLUTTER_TARGET_PLATFORM=linux-x64 /home/merritt/Downloads/flutter_hotkey_demo/linux
[  +35 ms] -- Checking for module 'keybinder-3.0'
[        ] --   No package 'keybinder-3.0' found
[        ] -- Configuring incomplete, errors occurred!
[        ] See also "/home/merritt/Downloads/flutter_hotkey_demo/build/linux/x64/debug/CMakeFiles/CMakeOutput.log".
[        ] CMake Error at /usr/share/cmake-3.18/Modules/FindPkgConfig.cmake:545 (message):
[        ]   A required package was not found
[        ] Call Stack (most recent call first):
[        ]   /usr/share/cmake-3.18/Modules/FindPkgConfig.cmake:733 (_pkg_check_modules_internal)
[        ]   flutter/ephemeral/.plugin_symlinks/hotkey_manager/linux/CMakeLists.txt:10 (pkg_check_modules)
[   +8 ms] Building Linux application... (completed in 56ms)
[+5525 ms] Exception: Unable to generate build files
[        ] "flutter run" took 5,920ms.
[   +3 ms]
           #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
           #1      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:669:9)
           <asynchronous suspension>
           #2      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1125:27)
           <asynchronous suspension>
           #3      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #4      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
           <asynchronous suspension>
           #5      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:288:9)
           <asynchronous suspension>
           #6      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #7      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:236:5)
           <asynchronous suspension>
           #8      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
           <asynchronous suspension>
           #9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #10     main (package:flutter_tools/executable.dart:92:3)
           <asynchronous suspension>

[  +81 ms] ensureAnalyticsSent: 79ms
[   +1 ms] Running shutdown hooks
[        ] Shutdown hooks complete
[        ] exiting with code 1

❯ flutter --version
Flutter 2.5.0-5.2.pre • channel beta • https://github.com/flutter/flutter.git
Framework • revision 19c61fed0d (13 days ago) • 2021-08-18 17:10:31 -0700
Engine • revision 7a4c4505f6
Tools • Dart 2.14.0 (build 2.14.0-377.7.beta)

On linux platform it depends on keybinder-3.0, You need to install it.
sudo apt-get install keybinder-3.0

Thanks! On Ubuntu-based distros this is:

sudo apt install libkeybinder-3.0-dev

Now it compiles correctly. :+1:
On run however it throws an exception at instance.register():
  final _hotkey = HotKey(
    KeyCode.f6,
    modifiers: [KeyModifier.control, KeyModifier.shift],
  );

  await HotKeyManager.instance.register(
    _hotkey,
    keyDownHandler: (hotKey) {
      print('keyDown');
    },
    keyUpHandler: (hotKey) {
      print('keyUp');
    },
  );

_CastError (Null check operator used on a null value)

#0      MethodChannel.binaryMessenger (package:flutter/src/services/platform_channel.dart:121:86)
#1      MethodChannel.setMethodCallHandler (package:flutter/src/services/platform_channel.dart:375:5)
#2      HotKeyManager._init (package:hotkey_manager/src/hotkey_manager.dart:28:14)
#3      HotKeyManager.register (package:hotkey_manager/src/hotkey_manager.dart:85:24)
#4      main (package:flutter_hotkey_demo/main.dart:12:32)
#5      _runMainZoned.. (dart:ui/hooks.dart:145:25)
#6      _rootRun (dart:async/zone.dart:1428:13)
#7      _CustomZone.run (dart:async/zone.dart:1328:19)
#8      _runZoned (dart:async/zone.dart:1863:10)
#9      runZonedGuarded (dart:async/zone.dart:1851:12)
#10     _runMainZoned. (dart:ui/hooks.dart:141:5)
#11     _delayEntrypointInvocation. (dart:isolate-patch/isolate_patch.dart:283:19)
#12     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)

Can you provide the code for flutter_hotkey_demo

Please don't use it in main function.

Alright, tried moving it out of main and it works some of the time. (Where do you recommend to initialize?)
"some of the time" because when it doesn't work; after hot reload I have seen it:

Fire the call back 3 times per press of the hotkey
Trigger an exception:

"No element"
Stacktrace:

#0      ListMixin.firstWhere (dart:collection/list.dart:167:5)
#1      HotKeyManager._methodCallHandler (package:hotkey_manager/src/hotkey_manager.dart:62:33)
#2      MethodChannel._handleAsMethodCall (package:flutter/src/services/platform_channel.dart:386:55)
#3      MethodChannel.setMethodCallHandler. (package:flutter/src/services/platform_channel.dart:379:34)
#4      _DefaultBinaryMessenger.setMessageHandler. (package:flutter/src/services/binding.dart:379:35)
#5      _DefaultBinaryMessenger.setMessageHandler. (package:flutter/src/services/binding.dart:376:46)
#6      _invoke2. (dart:ui/hooks.dart:205:15)
#7      _rootRun (dart:async/zone.dart:1428:13)
#8      _CustomZone.run (dart:async/zone.dart:1328:19)
#9      _CustomZone.runGuarded (dart:async/zone.dart:1236:7)
#10     _invoke2 (dart:ui/hooks.dart:204:10)
#11     _ChannelCallbackRecord.invoke (dart:ui/channel_buffers.dart:42:5)
#12     _Channel.push (dart:ui/channel_buffers.dart:132:31)
#13     ChannelBuffers.push (dart:ui/channel_buffers.dart:329:17)
#14     PlatformDispatcher._dispatchPlatformMessage (dart:ui/platform_dispatcher.dart:544:22)
#15     _dispatchPlatformMessage (dart:ui/hooks.dart:92:31)

Code is just from a new flutter project:
import 'dart:developer';

import 'package:flutter/material.dart';
import 'package:hotkey_manager/hotkey_manager.dart';

Future<void> main() async {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(),
      home: const HomePage(),
    );
  }
}

Future<void> startHotkey() async {
  final _hotkey = HotKey(
    KeyCode.f6,
    modifiers: [KeyModifier.control, KeyModifier.shift],
  );

  await HotKeyManager.instance.register(
    _hotkey,
    keyDownHandler: (hotKey) => log('keyDown'),
    keyUpHandler: (hotKey) => log('keyUp'),
  );
}

class HomePage extends StatefulWidget {
  const HomePage({Key? key}) : super(key: key);

  @override
  State<HomePage> createState() => _HomePageState();
}

class _HomePageState extends State<HomePage> {
  @override
  void initState() {
    super.initState();
    startHotkey();
  }

  @override
  Widget build(BuildContext context) {
    return const Scaffold(
      body: Center(
        child: Text('Hotkey test'),
      ),
    );
  }
}

By the way, is there a specific reason this requires Flutter, and doesn't support a dart application? I could see it being very useful for that use case as well.

I reconfirmed that it works on main, but I need to add WidgetsFlutterBinding.ensureInitialized()
For hot reload, HotKeyManager.instance.unregisterAll() needs to be called;
No element error is fixed on macOS platform , Windows and Linux platform will fixed later.

void main() {
+  WidgetsFlutterBinding.ensureInitialized();

  final _hotkey = HotKey(
    KeyCode.keyF,
    modifiers: [KeyModifier.alt],
  );

+  HotKeyManager.instance.unregisterAll();
  HotKeyManager.instance.register(
    _hotkey,
    keyDownHandler: (hotKey) => print('keyDown'),
    keyUpHandler: (hotKey) => print('keyUp'),
  );

  runApp(MyApp());
}


Feature filter filename replacements

I wanted the ability to pull in relevant tasks to the current note (use case: a project overview note, for example). Inline dataview and templater expressions do not appear to process before the Todoist query, so I added some functionality to expand {{filename}} within the query filter. Spaces replaced with asterisks for compatibility.
I am not super familiar with typescript and this may be considered hacky. Future improvements would be to include a toggle in settings for this, maybe adding a map of other possible expansions, etc.

OK, this has been revised as discussed. This actually cleaned it up a bit as I didn't need to pass a new parameter around just for the filename, since the MarkdownPostProcessor context was already in the pendingQuery object. Added some comments on the commits with considerations.

Hey @anth-dinosaur, I've pulled this change into #259, added some tests and tweaked the implementation. I've added you as a co-author on the commit - thanks for the contribution! 🎉


Integration of nextsim.dg module in NEDAS workflow

added a JobSubmitter class to handle different ways to run jobs; replacing the job_submit_cmd approach (@yumengch could you help test this?)
moved nextsimdg restart and forcing file logic from top-level config file to its own default.yml config file. Adding a namelist() function.
moved perturb.py under models/nextsim/dg, keeping it as a nextsimdg specific perturbation scheme, along with the standalone scheme in scripts/perturb.py
added the missing bits in nextsim/dg/model.py methods, to integrate with core assimilate algorithm

Hi @myying,

I use sbatch command from the compute node all the time without needing
ssh (on fram, but maybe other servers are different?). Perhaps it is better
to simplify the logic and don't have the ssh option?
space in /tmp is limited since many people can be saving there (I've had
"no disk space" crashes from this) so I would agree the default location
for temp files should be in the run directory.

On Tue, 17 Dec 2024 at 09:39, Yue (Michael) Ying @.***>
wrote:

@.**** commented on this pull request.
On utils/job_submitters/oar.py
https://github.com/nansencenter/NEDAS/pull/7#discussion_r1888108772:
@aydogduali https://github.com/aydogduali had a question about the "ssh
" part and about the temporary file used in submit_job_and_monitor.
I remembered Yumeng mentioning that ssh is necessary when the job is
submitted from a compute node, so that they are accessing the login node to
submit additional jobs. If it is unnecessary, I suggest making additional
logic:
if self.job_submit_node is None: then don't add the "ssh
self.job_submit_node" part in the commands
For the temporary job script files. I used NamedTemporaryFile to create a
file with guaranteed unique name. I imagine that if the self.run_dir is the
same when submitting multiple jobs with the same job_name their job_script
can collide (all members run in the same directory, can happen sometimes).
The default name from NamedTemporaryFile is "/tmp/tmpXXXXXX.sh" if this is
not a good location. I suggest using NamedTemporaryFile(mode='w+',
delete=False, dir=self.run_dir, prefix=self.job_name, suffix='.sh') so that
it is created in the run_dir. We can also keep the job_script (don't
os.remove it) so that later we can check it.
—
Reply to this email directly, view it on GitHub
https://github.com/nansencenter/NEDAS/pull/7#pullrequestreview-2508186002,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATVYQLXT4HE22UV3XLMZ3T2F7PLTAVCNFSM6AAAAABS75MPGSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMBYGE4DMMBQGI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi @tdcwilliams, Thanks for chiming in.
I have a separate job_submitters/slurm.py to handle the sigma2 computers. As you see in slurm.py I didn't implement any ssh before sbatch command, since we don't need this additional step. The ssh stuff is only for oar.py for a different machine.
The temporary file is indeed not safe to rely on /tmp partition. So in the new commit I changed it to be written in self.run_dir which is specified when making the JobSubmitter instance.


Can't install module

I am trying to install but I get the same error always
242 verbose node v12.16.1
243 verbose npm  v6.14.2
244 error code ELIFECYCLE
245 error errno 1
246 error<EMAIL_ADDRESS>install: node-gyp rebuild
246 error Exit status 1
247 error Failed at the<EMAIL_ADDRESS>install script.
I have node v12
Win 10 x64

+1 It's because it contains package ogg-packet which contains ref-struct package which is NOT compatible with node 12+.
IMO this issue should be top priority for this repo.
My  node -v v12.16.2.
OS: Ubuntu 20.xx LTS
There is a fix in a comment https://github.com/Rantanen/node-opus/issues/93#issuecomment-570787267 which I'll try tomorrow

I'm archiving the repository for lack of interest in maintaining it. Sorry I never got to deal with this issue. Please see @discordjs/opus for an alternative implementation that is based this project but uses more stable Node.js module technology that should break less between Node.js versions.


G rules seems not working with loguru.

It seems G rules only works with built-in logging module not loguru. Is it by-design or a bug?
Testing script:
import logging
from loguru import logger

a = 100
logger.info(f"Value of a is {a}")
logging.info(f"Value of a is {a}")

logger.warn("This is a warning")
logging.warn("This is a warning")

ruff command (executed in empty project directory without config file) : ruff check --select G ./test.py.
The output:
  |
4 | a = 100
5 | logger.info(f"Value of a is {a}")
6 | logging.info(f"Value of a is {a}")
  |              ^^^^^^^^^^^^^^^^^^^^ G004
7 |
8 | logger.warn("This is a warning")
  |

test.py:9:9: G010 [*] Logging statement uses `warn` instead of `warning`
  |
8 | logger.warn("This is a warning")
9 | logging.warn("This is a warning")
  |         ^^^^ G010
  |
  = help: Convert to `warning`

Found 2 errors.
[*] 1 fixable with the `--fix` option.

ruff points out all errors on logging module but none of loguru's.
ruff version: 0.6.9

Yes, this seems intentional.
Ruff matches the upstream flake8-logging-format linter and its documentation states that it is specific to Python's logging module

Python logging supports a special extra keyword for passing a dictionary of user-defined attributes to include in a logging event.

Our current focus is on linting vanilla Python code, and we only make an exception for very popular third-party libraries that are considered community standards.


Multi Choice Allows for a Duplicate Shortcut Key

The multi choice parameter allows you to use the same letter as a shortcut key for more than one choice.
<choice label="&Private"
        help="Adds a Private folder to the module root."
        value="Private" />
<choice label="&Public"
        help="Adds a Public folder to the module root."
        value="Public" />

In the above example, if the letter "P" is used as a shortcut it will create both the Private and Public folder. My current work around is to use different letters.
<choice label="P&rivate"
        help="Adds a Private folder to the module root."
        value="Private" />
<choice label="P&ublic"
        help="Adds a Public folder to the module root."
        value="Public" />

It seems buggy that Invoke-Plaster, and perhaps Test-PlasterManifest, would allow for this behavior. The expected behavior would be an indication to the user that there's a conflict, and that a different value should be used for one of the choice labels.

+1 I just ran into the same where I had to choices starting with the same letter

I tried looking at the code, and it doesn't seem that this is something that Plaster does automatically, it is a function of the [System.Management.Automation.Host.ChoiceDescription] Label property. It is based on where you place the &.
https://docs.microsoft.com/en-us/dotnet/api/system.management.automation.host.choicedescription.label?view=powershellsdk-1.1.0#System_Management_Automation_Host_ChoiceDescription_Label
The workaround is the solution in this case I think.


Macro formatting

This doesn't get formatted:
macro_rules! passthrough {
    ($($t:tt)*) => ($($t)*)
}

passthrough! {
fn x(
) -> (
) {}
}

dup of #8


Issue with DateTime fields

IN the cortex framework it is expected that dates have the date format with SAP. I like to request that in the config you can format how the date fields and time fields are handled in Cortex.
My suggestion is for dates the following options.

date
datetime
string + format [yyyymmdd]

For time fields

time
string + format [HHMISS]

I suggest that there is a possibility that you can use also the string data type or date time data type in the solution.
The following functions are affected.

fiscal_case1.sql
fiscal_case2.sql
fiscal_case3.sql
currency_conversion.sql (function gdatu_to_date)

Further for the time fields used in the ordertocash reporting should also support string values.

OrderToCash.sql

here is an example:
/*OrderCycleTimeInDays*/   IF(Deliveries.ActualGoodsMovementDate_WADAT_IST IS NOT NULL,     TIMESTAMP_DIFF(CAST(CONCAT(CAST(Deliveries.Date__proofOfDelivery___PODAT AS DATE), ' ', CAST(Deliveries.ConfirmationTime_POTIM AS TIME FORMAT 'HH24MISS') ) AS TIMESTAMP),       CAST(CONCAT(CAST(SalesOrders.CreationDate_ERDAT AS DATE), ' ', CAST(SalesOrders.CreationTime_ERZET AS TIME FORMAT 'HH24MISS')) AS TIMESTAMP), DAY), NULL) AS OrderCycleTimeInDays,
This can be solved with the jinja script. Makes the integration with Fivetran's LDP solution much smoother.
If you want me to discuss happy to setup a meeting.

HI @rikat5tran , thanks for reaching out with a suggestion. The data foundation expects data landed and converted as in here: https://cloud.google.com/solutions/sap/docs/bq-connector/latest/planning#data_type_mapping
Because we cater to multiple integration solutions for SAP to BQ, and this specific request would have a massive impact in deployment, data handling and reporting downstream, I'll reject the suggestion and recommend you reach out to the Fivetran team to discuss transformations specific to this tool so the change is sustainable and compatible with the formats expected by the data foundation.

Hi
I’m the SAP specialist of Fivetran. So I that is the reason I request this.  But it is Googles framework so if you say it is a massive impact on the deployment it is what it is.
Cheers,
Richard Brouwer
Principal Sales Engineer Global SAP specialist
Sent from Outlook for iOShttps://aka.ms/o0ukef

From: Lucia Subatin @.>
Sent: Monday, September 25, 2023 7:29:36 PM
To: GoogleCloudPlatform/cortex-data-foundation @.>
Cc: Richard Brouwer @.>; Mention @.>
Subject: Re: [GoogleCloudPlatform/cortex-data-foundation] Issue with DateTime fields (Issue #34)
Closed #34https://github.com/GoogleCloudPlatform/cortex-data-foundation/issues/34 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/GoogleCloudPlatform/cortex-data-foundation/issues/34#event-10465680422, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXMMFDDVRFIY6M6X735A2WLX4G5QBANCNFSM6AAAAAA5GHQFAI.
You are receiving this because you were mentioned.Message ID: @.***>


chore(master): release 1.1.93

:robot: I have created a release beep boop
1.1.93 (2023-06-26)
Miscellaneous

deps: update pnpm to v8.6.5 (78eb5f2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/keywind/releases/tag/v1.1.93 :sunflower:


[Relay] Add list update to prelude

This can be useful to model TensorArrayWrite in TensorFlow.

@MarisaKirisame @slyubomirsky @jroesch please take a look. Test passed.

LGTM