subclassing FastAPI

I tried recently to subclass a FastAPI class to create a more concrete class. I didn't get any python errors but the response from FastAPI was the normal JSON error response. And all examples I've seen, I've not seen anywhere were FastAPI is subclassed so far.
So is it possible to subclass FastAPI class?

It is not possible to see where you made mistakes without any code, Can you follow the issue template?

from fastapi import FastAPI

class NewAPI(FastAPI):
      def __init__(self):
           super(NewAPI,self).__init__()
      def get(self,path:str = "/home"):
           return {"greeting":"welcome home"}

that's sort of an example of what i was talking about. the one i wrote didn't have any python errors. but instead of returning the response of NewAPI's get method, it instead return an error JSON file
{
    "detail": [
        {
            "loc": [
                "path",
                "item_id"
            ],
            "msg": "value is not a valid integer",
            "type": "type_error.integer"
        }
    ]
}

That error does not appear to be related to the code you have provided.
And to answer your question, no, the FastAPI class is not intended to be subclassed to implement routes.

from fastapi import FastAPI

class NewAPI(FastAPI):
      def __init__(self):
           super(NewAPI,self).__init__()
      def get(self,path:str = "/home"):
           return {"greeting":"welcome home"}

Ahhh, i see, use class-based views from dmontagu's fastapi-utils.
It will be way more cleaner solution than subclassing FastAPI.

Thanks for the help here everyone! 👏  🙇
Thanks for reporting back and closing the issue @ht-thomas  👍

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


NFT transaction failure from Katla testnet

Describe the bug
Transactions from katla testnet to holskey keep failing

Steps to reproduce
Steps to reproduce here.
Spam policy

[X] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

@rajmehta26 Probably one of those nft2me collections am I right? reach out on discord, those contracts don't follow the NFT standard correctly.

No, apparently its every nft that I am transacting, shows failure
Best regards,
Raj Mehta
On Thu, 7 Mar 2024, 23:18 Korbinian, @.***> wrote:

@rajmehta26 https://github.com/rajmehta26 Probably one of those nft2me
collections am I right? reach out on discord, those contracts don't follow
the NFT standard correctly.
—
Reply to this email directly, view it on GitHub
https://github.com/taikoxyz/taiko-mono/issues/16354#issuecomment-1984617153,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEMNYVDGPBAJVOJNFYCHJN3YXDRUTAVCNFSM6AAAAABELUFENKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBUGYYTOMJVGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Create new release

@felipe-augusto It seems there is no github release or git tag on the project.
Do you know where the npm v1.0.0 leads? It seems it's causing problems : https://github.com/felipe-augusto/linguee/issues/7
I think we should release at least a patch release and update it on npm. What do you think? :)

@Einenlum I will do this ASAP!

I think I misse something, haha. I never did that, can you check if everything is correct?


Niantic is shutting down scanners

So pokevision just shut down at the request of Niantic. Over the course of today an new patch was released for the game as well as older versions of the app not functioning properly. The Ceo warned of this in a recent interview and I think that Niantic has taken server side measures to prevent "rare farming". I think this includes limiting update rates, limiting event triggers by speed in general as well as detection software that will deny any further updates at all if triggered and huge changes in distance covered between nodes maybe a trigger. I've been experimenting with older versions and none seem to be capable of running multiple scans anymore and once even attempting to do so, it seem like no scan at all will work without completely rebooting. I'd be concerned about "watchers" now as these changes could mean other changes as well that arent as apparent.
0.1.2 is still working properly although limited to one instance of the app now. The speed is acceptable and seems to find everything. The unfortunate side is the reason that version was scrapped, the tunnel service was too problematic. Those that can set it up as a server will find it still function well but needs a signout between position changes so as not to trigger the huge distance gap detector (I just made that up but it certainly feels like its a thing now)
So the development of this App may soon be compromised as the next step is to shutdown any/all GIT Hub projects. I hope not but I am willing to bet with the removal of the footsteps and the shutdown of pokevision as well that Niantic will add functionality to the pokemon+ watch that simulates scanning to a very controlled and limited degree.  It probably wont be nearly as cool either but it would be the only way to actually track something. "Shenanigans!"

can't confirm this for sure, but i live in a rural area and since the update, scanning has seemingly stopped pokemon from spawning, my area is almost empty now.. if niantic have an algorithm to detect scanning, which seems possible, its also possible theyre punishing ppl for scanning, and i would be surprised if anyone else in my village plays, never mind scans.

intraum, that wouldn't make much sense since I would expect the more populated the area (such as New York City), the more likely people are playing pokemon go and the more likely people are using scanners.  If they punished entire areas for scanning, you'd have to assume that places like Central Park would abruptly stop spawning pokemon.

no but they changed pokemon nest spawn, that might be the reason for your troubles :/

Basically asking for too many updates too quickly gets you none. What still works is 0.1.6 of this App with a 2 radius and follow location on. The problem is a large radius now alerts the game. When the end of a huge spiral restarts its a large distance. For an app like this to function on the limitations theyve set, it need to abandon blanket scans. Currently I can manually changing coords by small amounts and spacing them out so as not to spam, with a radius of 2 is seems to pickup everything again. You may still be able to scan within a house distance of your current position.  As long as the update rate on the app is within the update rate set by Niantic seemingly yesterday afternoon then it will recieve an accurate scan. As long as the radius isnt over the distance threshold then it wont deny a query result. Basically you need to toggle the app on and off and use it within a small few block area.
An alternative is exclusively called pathed scans that dont reset automatically and force a delay between ending it and beginning it. Say for instance you scan nothing at all until feedback is recieved and the feedback is in the form of a drawn path. This would at least appear to the system that there is movement that falls under their acceptable rate. I have even noticed encounters dont pop up unless I am driving much slower than before. Seems liek the event trigger system got some nerfs

this game is more fun for me driving around and finding pokemon with friends. then walking around all over the city hoping for a squirtle to pop up.
when in reality actually scanning and seeing how far apart the pokemon you actually want, and now realizing they are making it way more difficult is going to draw people from the game

Pill0ws, instead of a delay the workers could reverse direction after completing the spiral.

@pill0ws if you can keep it up keep it up and F*ck niantic
if they cant fix their trackin system they have no right to cancel stuff like this.
If they email you give me the email and ill mail them back a nice screenshot of a botnet thats ready to ddos their servers :P

Doesn't Niantic have better things to worry about right now? like... I don't know... The fcking servers dont even work half the time, they delayed their pokemon+ device that they've been hyping up, they pushed back their trading and breeding update and they still have fcking pokestops in cemeteries and most of all the sole feature needed to actually play the one thing you can do in the game doesn't properly work, the nearby list. But yea, it seems appropriate for them to be focused scanners right now...


Fuzzing Yosys inputs may lead to better results

Description
Investigation in https://github.com/The-OpenROAD-Project/OpenLane/issues/1697 shows that no-ops in Verilog, such as whitespace and comment, actually have an effect in Yosys.
Per YosysHQ, this is a quasi-feature as it allows us to kinda sorta pick a different "seed" for ABC. @oharboe notes that minor changes to the input can drastically affect the PPA of the netlist.
Yosys for their part provide a command to exploit this, but there is also the rename command. https://github.com/YosysHQ/yosys/issues/3713#issuecomment-1485639838
Proposal
We can use either this command or automate fuzzing of the Verilog source files somehow (maybe by just adding a nondescript amount of line breaks.)

I think at one point or another, it's probably better to actively embrace the chaotic effect of Yosys than try to fight it; I think OpenLane is well-suited to such an idea, because different seeds can be qualified according to the output of post-synthesis STA and area.


RxBroadcast breaks w/rxJava 1.3.0

With RxBroadcast version: 1.1.1, RxJava version: 1.2.6, project builds fine.
But with RxJava 1.3.0 get error:
java.lang.NoSuchMethodError: No static method      fromEmitter(Lrx/functions/Action1;Lrx/Emitter$BackpressureMode;)Lrx/Observable; in class Lrx/Observable; or its super classes

closed with #17


Getting Error while setting up project

Hi team,
I am trying to setup the project but i am getting this error while give cmd:
yarn install
Error:

Hey @janrxyz ,
I followed all steps that were mentioned but still getting error.
Step-1: Node Setup:

Step-2: Npm / Yarn Setup:

Step-3: Download Indy-sdk and Adding  Path

Step-4: Confirm installation
run cmd;
npx -p<EMAIL_ADDRESS>is-indy-installed
I ran it....
when i tried to give same cmd it is asking me install again. But Y again...?

You installed Libindy, the underlying package needed to run AFJ. If you run the final command, it installs AFJ and checks if Libindy installation has succeeded.

I am getting this error when i gave yarn install can some one help me
yarn install
yarn install v1.22.19
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "eslint-config-react-app ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "eslint-config-react-app ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "workspace-aggregator-fc13f99e-a41a-44a9-977d-444e91fd4d5b > client ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^16.0.0 || ^17.0.0".
warning "workspace-aggregator-fc13f99e-a41a-44a9-977d-444e91fd4d5b > client ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-fc13f99e-a41a-44a9-977d-444e91fd4d5b > server ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-fc13f99e-a41a-44a9-977d-444e91fd4d5b > server ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-fc13f99e-a41a-44a9-977d-444e91fd4d5b > client > kbar ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
warning "workspace-aggregator-fc13f99e-a41a-44a9-977d-444e91fd4d5b > client > kbar ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
warning "workspace-aggregator-fc13f99e-a41a-44a9-977d-444e91fd4d5b > client > kbar ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "workspace-aggregator-fc13f99e-a41a-44a9-977d-444e91fd4d5b > server > ts-node-dev ><EMAIL_ADDRESS>has unmet peer dependency "@types/node@*".
warning "workspace-aggregator-fc13f99e-a41a-44a9-977d-444e91fd4d5b > client > kbar > @reach/portal ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
warning "workspace-aggregator-fc13f99e-a41a-44a9-977d-444e91fd4d5b > client > kbar > @reach/portal ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
[4/4] Building fresh packages...
[-/7] ⠁ waiting...
[-/7] ⠁ waiting...
[-/7] ⠂ waiting...
[-/7] ⠂ waiting...
error C:\Users\ACER\Desktop\animo-demo\node_modules\indy-sdk: Command failed.
Exit code: 1
Command: node-gyp rebuild
Arguments:
Directory: C:\Users\ACER\Desktop\animo-demo\node_modules\indy-sdk
Output:
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| win32 | x64
gyp info find Python using Python version 3.11.3 found at "C:\Users\ACER\AppData\Local\Programs\Python\Python311\python.exe"
gyp ERR! find VS
gyp ERR! find VS msvs_version not set from command line or npm config
gyp ERR! find VS VCINSTALLDIR not set, not running in VS Command Prompt
gyp ERR! find VS checking VS2022 (17.6.33723.286) found at:
gyp ERR! find VS "C:\Program Files\Microsoft Visual Studio\2022\Community"
gyp ERR! find VS - found "Visual Studio C++ core features"
gyp ERR! find VS - found VC++ toolset: v143
gyp ERR! find VS - missing any Windows SDK
gyp ERR! find VS could not find a version of Visual Studio 2017 or newer to use
gyp ERR! find VS looking for Visual Studio 2015
gyp ERR! find VS - not found
gyp ERR! find VS not looking for VS2013 as it is only supported up to Node.js 8
gyp ERR! find VS
gyp ERR! find VS **************************************************************
gyp ERR! find VS You need to install the latest version of Visual Studio
gyp ERR! find VS including the "Desktop development with C++" workload.
gyp ERR! find VS For more information consult the documentation at:
gyp ERR! find VS https://github.com/nodejs/node-gyp#on-windows
gyp ERR! find VS **************************************************************
gyp ERR! find VS
gyp ERR! configure error
gyp ERR! stack Error: Could not find any Visual Studio installation to use
gyp ERR! stack     at VisualStudioFinder.fail (C:\Users\ACER\Desktop\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:122:47)
gyp ERR! stack     at C:\Users\ACER\Desktop\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:75:16
gyp ERR! stack     at VisualStudioFinder.findVisualStudio2013 (C:\Users\ACER\Desktop\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:363:14)
gyp ERR! stack     at C:\Users\ACER\Desktop\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:71:14
gyp ERR! stack     at C:\Users\ACER\Desktop\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:384:16
gyp ERR! stack     at C:\Users\ACER\Desktop\animo-demo\node_modules\node-gyp\lib\util.js:54:7
gyp ERR! stack     at C:\Users\ACER\Desktop\animo-demo\node_modules\node-gyp\lib\util.js:33:16
gyp ERR! stack     at ChildProcess.exithandler (node:child_process:427:5)
gyp ERR! stack     at ChildProcess.emit (node:events:513:28)
gyp ERR! stack     at maybeClose (node:internal/child_process:1091:16)
gyp ERR! stack     at ChildProcess._handle.onexit (node:internal/child_process:302:5)
gyp ERR! System Windows_NT 10.0.19045
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\ACER\\Desktop\\animo-demo\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\ACER\Desktop\animo-demo\node_modules\indy-sdk

You need to follow the steps from the stacktrace: https://github.com/nodejs/node-gyp#on-windows

I have followed it and installed the Desktop with C++ in visual Studio 2019 but still getting the same error.

Have you tried reinstalling the dependencies and/or re-creating the yarn.lock after your changes? Make sure that it refers to the most recent installation of visual studio.

I have re-cloned the repo. I am getting the same error.
D:\animo-demo\client> yarn install
yarn install v1.22.19
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "eslint-config-react-app ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "eslint-config-react-app ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "workspace-aggregator-26b36de9-dfe3-4906-b3a8-31bbb530ba97 > client ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^16.0.0 || ^17.0.0".
warning "workspace-aggregator-26b36de9-dfe3-4906-b3a8-31bbb530ba97 > client ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-26b36de9-dfe3-4906-b3a8-31bbb530ba97 > server ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-26b36de9-dfe3-4906-b3a8-31bbb530ba97 > server ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-26b36de9-dfe3-4906-b3a8-31bbb530ba97 > client > kbar ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
warning "workspace-aggregator-26b36de9-dfe3-4906-b3a8-31bbb530ba97 > client > kbar ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
warning "workspace-aggregator-26b36de9-dfe3-4906-b3a8-31bbb530ba97 > client > kbar ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "workspace-aggregator-26b36de9-dfe3-4906-b3a8-31bbb530ba97 > server > ts-node-dev ><EMAIL_ADDRESS>has unmet peer dependency "@types/node@*".
warning "workspace-aggregator-26b36de9-dfe3-4906-b3a8-31bbb530ba97 > client > kbar > @reach/portal ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
warning "workspace-aggregator-26b36de9-dfe3-4906-b3a8-31bbb530ba97 > client > kbar > @reach/portal ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
[4/4] Building fresh packages...
[-/7] ⠈ waiting...
[-/7] ⠈ waiting...
[-/7] ⠈ waiting...
[7/7] ⠁ ffi-napi
error D:\animo-demo\node_modules\indy-sdk: Command failed.
Exit code: 1
Command: node-gyp rebuild
Arguments:
Directory: D:\animo-demo\node_modules\indy-sdk
Output:
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| win32 | x64
gyp info find Python using Python version 3.11.3 found at "C:\Users\ACER\AppData\Local\Programs\Python\Python311\python.exe"
gyp ERR! find VS
gyp ERR! find VS msvs_version not set from command line or npm config
gyp ERR! find VS VCINSTALLDIR not set, not running in VS Command Prompt
gyp ERR! find VS checking VS2022 (17.6.33723.286) found at:
gyp ERR! find VS "C:\Program Files\Microsoft Visual Studio\2022\Community"
gyp ERR! find VS - found "Visual Studio C++ core features"
gyp ERR! find VS - found VC++ toolset: v143
gyp ERR! find VS - missing any Windows SDK
gyp ERR! find VS could not find a version of Visual Studio 2017 or newer to use
gyp ERR! find VS looking for Visual Studio 2015
gyp ERR! find VS - not found
gyp ERR! find VS not looking for VS2013 as it is only supported up to Node.js 8
gyp ERR! find VS
gyp ERR! find VS **************************************************************
gyp ERR! find VS You need to install the latest version of Visual Studio
gyp ERR! find VS including the "Desktop development with C++" workload.
gyp ERR! find VS For more information consult the documentation at:
gyp ERR! find VS https://github.com/nodejs/node-gyp#on-windows
gyp ERR! find VS **************************************************************
gyp ERR! find VS
gyp ERR! configure error
gyp ERR! stack Error: Could not find any Visual Studio installation to use
gyp ERR! stack     at VisualStudioFinder.fail (D:\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:122:47)
gyp ERR! stack     at D:\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:75:16
gyp ERR! stack     at VisualStudioFinder.findVisualStudio2013 (D:\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:363:14)
gyp ERR! stack     at D:\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:71:14
gyp ERR! stack     at D:\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:384:16
gyp ERR! stack     at D:\animo-demo\node_modules\node-gyp\lib\util.js:54:7
gyp ERR! stack     at D:\animo-demo\node_modules\node-gyp\lib\util.js:33:16
gyp ERR! stack     at ChildProcess.exithandler (node:child_process:427:5)
gyp ERR! stack     at ChildProcess.emit (node:events:513:28)
gyp ERR! stack     at maybeClose (node:internal/child_process:1091:16)
gyp ERR! stack     at ChildProcess._handle.onexit (node:internal/child_process:302:5)
gyp ERR! System Windows_NT 10.0.19045
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "D:\\animo-demo\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd D:\animo-demo\node_modules\indy-sdk

Could you run npm config set msvs_version 2019 --global? NPM is probably not able to locate the correct Visual Studio installation.

Hey @janrxyz  I have done it but got same error.
PS D:\animo-demo> npm config set msvs_version 2019 --global
PS D:\animo-demo> yarn install
yarn install v1.22.19
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "eslint-config-react-app ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "eslint-config-react-app ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "workspace-aggregator-6e603cfe-9ece-4055-b3fc-c81adfb05365 > client ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^16.0.0 || ^17.0.0".
warning "workspace-aggregator-6e603cfe-9ece-4055-b3fc-c81adfb05365 > client ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-6e603cfe-9ece-4055-b3fc-c81adfb05365 > server ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-6e603cfe-9ece-4055-b3fc-c81adfb05365 > server ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-6e603cfe-9ece-4055-b3fc-c81adfb05365 > client > kbar ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
warning "workspace-aggregator-6e603cfe-9ece-4055-b3fc-c81adfb05365 > client > kbar ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
warning "workspace-aggregator-6e603cfe-9ece-4055-b3fc-c81adfb05365 > client > kbar ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "workspace-aggregator-6e603cfe-9ece-4055-b3fc-c81adfb05365 > server > ts-node-dev ><EMAIL_ADDRESS>has unmet peer dependency "@types/node@*".
warning "workspace-aggregator-6e603cfe-9ece-4055-b3fc-c81adfb05365 > client > kbar > @reach/portal ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
warning "workspace-aggregator-6e603cfe-9ece-4055-b3fc-c81adfb05365 > client > kbar > @reach/portal ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".
[4/4] Building fresh packages...
[-/7] ⠁ waiting...
[-/7] ⠁ waiting...
[-/7] ⠁ waiting...
[7/7] ⠁ ffi-napi
error D:\animo-demo\node_modules\indy-sdk: Command failed.
Exit code: 1
Command: node-gyp rebuild
Arguments:
Directory: D:\animo-demo\node_modules\indy-sdk
Output:
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| win32 | x64
gyp info find Python using Python version 3.11.3 found at "C:\Users\ACER\AppData\Local\Programs\Python\Python311\python.exe"
gyp ERR! find VS
gyp ERR! find VS msvs_version not set from command line or npm config
gyp ERR! find VS VCINSTALLDIR not set, not running in VS Command Prompt
gyp ERR! find VS checking VS2022 (17.6.33723.286) found at:
gyp ERR! find VS "C:\Program Files\Microsoft Visual Studio\2022\Community"
gyp ERR! find VS - found "Visual Studio C++ core features"
gyp ERR! find VS - found VC++ toolset: v143
gyp ERR! find VS - missing any Windows SDK
gyp ERR! find VS could not find a version of Visual Studio 2017 or newer to use
gyp ERR! find VS looking for Visual Studio 2015
gyp ERR! find VS - not found
gyp ERR! find VS not looking for VS2013 as it is only supported up to Node.js 8
gyp ERR! find VS
gyp ERR! find VS **************************************************************
gyp ERR! find VS You need to install the latest version of Visual Studio
gyp ERR! find VS including the "Desktop development with C++" workload.
gyp ERR! find VS For more information consult the documentation at:
gyp ERR! find VS https://github.com/nodejs/node-gyp#on-windows
gyp ERR! find VS **************************************************************
gyp ERR! find VS
gyp ERR! configure error
gyp ERR! stack Error: Could not find any Visual Studio installation to use
gyp ERR! stack     at VisualStudioFinder.fail (D:\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:122:47)
gyp ERR! stack     at D:\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:75:16
gyp ERR! stack     at VisualStudioFinder.findVisualStudio2013 (D:\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:363:14)
gyp ERR! stack     at D:\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:71:14
gyp ERR! stack     at D:\animo-demo\node_modules\node-gyp\lib\find-visualstudio.js:384:16
gyp ERR! stack     at D:\animo-demo\node_modules\node-gyp\lib\util.js:54:7
gyp ERR! stack     at D:\animo-demo\node_modules\node-gyp\lib\util.js:33:16
gyp ERR! stack     at ChildProcess.exithandler (node:child_process:427:5)
gyp ERR! stack     at ChildProcess.emit (node:events:513:28)
gyp ERR! stack     at maybeClose (node:internal/child_process:1091:16)
gyp ERR! stack     at ChildProcess._handle.onexit (node:internal/child_process:302:5)
gyp ERR! System Windows_NT 10.0.19045
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "D:\\animo-demo\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd D:\animo-demo\node_modules\indy-sdk


Fix buildfromsource build

Cherry-pick of #4771.

@KevinRansom - can you explain why this property is specific to the BuildFromSource.targets?  Why wouldn't it be the same in normal builds?  Does it just need to be factored out to somewhere shared?

We do this in our regular builds here, but this was missed because the src/buildfromsource directory is all SDK projects where src/fsharp is old-style projects.


Evaluate and reduce INFO logging where practical from Providers

There seems to be excessive INFO message written to the EVM logs contributing
to logging related bugs.
An evaluation and reduction of INFO logging where practical needs to be done.

@miq-bot add_label providers "technical debt"

@miq-bot add_label providers

@jvlcek do you have a couple examples of log messages that clutter the logs?

I do have example logs that I can share privately. What I did notice is that within 1 second a lot of duplicate messages are being reported. For example:
file % grep 03T17:14:42 evm.log | wc
104    2753   34585
file % grep 03T17:14:41 evm.log | wc
64    1992   22098
It appears to be related to C&U on vmware.

I think this is better handled as issues on individual provider repos as those dependencies are really the 'issue'. Closing this but feel free to open on the providers themselves!


Cookbook does not contain binaries

No binaries allowed in community cookbooks.
Are there legit use cases for binaries in a public cookbook? (I have seen it used, usually as a workaround to not having a real artifact server, in private cookbooks).
Does Supermarket already reject binaries?

+1

:+1:


Missing 2D example

Need to create specific 2D example and scripts.

2D example added by rmvermeulen, thanks!


Inconsistency when using long press to add accented letters

When using Mac, pressing and holding a letter such as 'e' raises the options to add an accented version of the letter:

When selecting an accented letter in rich mode, both the original letter and the accented letter are displayed:

When selecting an accented letter in plain mode, only the accented letter is displayed

I think that the latter behaviour is the better behaviour of the two, and that the behaviour should at least be consistent

Additionally to this, you can't create accented letters manually like ` + e for example


add smkn2malang domain

@rickyseptiandypph Thank you for your patience while waiting for a review and processing your request. Pull request merged.


[Tizen] Implement date / time chooser proxy

Date / Time chooser proxy is a DBus object proxy that communicates
with dialog launcher service.

Let's close this.


Updating copyright headers to match current best practices

Resolve #9
Signed-off-by: Daniel Lim Wee Soong<EMAIL_ADDRESS>

@daniellimws Can you squash these two commits together.

@mithro I just realized this

[ ] Add or update the AUTHORS files to have the text as shown at https://opensource.google/docs/releasing/authors/

I'm not sure what to put inside. Whose names?

https://makandracards.com/makandra/15909-quick-git-contributors-list maybe?

@mithro Is the AUTHORS file in the right format? If so, I can add it to the v2x repo later.

@mithro ping


Fix icons and manifest generation

Currently blocked by two Real Favicon Generator issues:

https://github.com/RealFaviconGenerator/grunt-real-favicon/issues/21
https://github.com/RealFaviconGenerator/realfavicongenerator/issues/207

Once fixed, the grunt task copy:manifest will be removed.

Check this: https://realfavicongenerator.net/api/interactive_api#overlapping_markups


Add X/Y offset to flat pattern drawing view placement on sheet

@rmcanany Please see the slightly modified code. I took inspiration from your user-defined scale value and also added an X/Y offset for better control over drawing placement.
I think I followed your new task conventions for a complete feature. I added a couple lines to the help text. I did not add any CheckStartconditions code as I'm not totally sure off the top of my head that there's anything to check other than that it can be cast to a double.
This just takes a positive or negative X and Y value and adds it to the center anchor point of the drawing view. It should let us dial in the placement a little bit better to dodge other elements in the template.
I did not add any logic to convert from the default unit of meters. I couldn't find any precedent for how units of measure should handled in Housekeeper. Before putting in a bunch of work to add unit conversions I thought I'd kick the change up to you to see if you have any thoughts on that. I have some simple helper functions I use to bounce between inches, meters, rads, and degrees in my own code and my shop always uses inches so I don't have any good boilerplate for that.

That's a good idea!  Thank you!
I'll have to think about units.  I guess we could dodge the issue by using percent of height or width, but that doesn't seem very intuitive.

That's a good idea! Thank you!
I'll have to think about units. I guess we could dodge the issue by using percent of height or width, but that doesn't seem very intuitive.

I had the same thought and came to the same conclusion. Couldn't think of a way to make that intuitive so you could punch in a number and have a good idea of exactly what it's going to look like on paper.


miscellaneous fixes / improvements

Don't recursively clean on Windows
This was left over from an early version of the Makefile, with recursive use of Make

Don't hard-code Makefile name
This was previously in, but was removed when I was accidentally using an older version of Make

Print warning when not on Windows and HHC is defined
HHC return codes need to be inverted, so the shell command that does this is not portable (you can't just use chmcmd here)

Use subshell to avoid directory change and get a wildcard match
Script block issue in Travis-CI seems to be that shell globbing doesn't function in the actual block
my final attempt before I never touch this again
seems to have worked

oh, you scratched, I would answer as "famous last words" ! Anyway, tested good here too. :]


Linters

It'd be great if you use Linters on your PR.

Thank you @abdullah97-pro  I appreciate it,
I will implement it,
Thank you


Change type of 'Authors' from text to multi-select

First of all, thanks for this integration!
However, if i had a wish, it will be the functionality to change the type of authors property to multi-select.
With this, it would be simple to add a sort by authors
Thanks!

I would love to see that enhancement as well ! Thanks for reading my mind :)

Same opinion, for us, the first author and corresponding author are often the points we pay more attention to. If they can be separated and changed into options instead of text, it will have a better performance.
In addition, I think changing time to a single choice would also be better.

@ctruillet @CarlosLegba777 @Dx-wmc, do you have a desired format for the values of the multi-select options? I currently format each author as Last name, First name, but Notion does not allow commas within multi-select options.
Some other formats I've considered are:

Last name; First name (using ; instead of ,)
First name Last name

What do you think?

Hi. Thanks for reaching out. Personally I use " First name Last name " in
my database, but I suppose others might have other preferences.
Le jeu. 28 déc. 2023 à 08:26, David Hoff-Vanoni @.***>
a écrit :

@ctruillet https://github.com/ctruillet @CarlosLegba777
https://github.com/CarlosLegba777 @Dx-wmc https://github.com/Dx-wmc,
do you have a desired format for the values of the multi-select options? I
currently format each author as Last name, First name, but Notion does
not allow commas within multi-select options.
Some other formats I've considered are:

Last name; First name (using ; instead of ,)
First name Last name

What do you think?
—
Reply to this email directly, view it on GitHub
https://github.com/dvanoni/notero/issues/146#issuecomment-1870896572,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATYMBKUS4OXEJOIH2B3A3TLYLUNLXAVCNFSM6AAAAAAQ4J3TVOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZQHA4TMNJXGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm exploring adding support for this. I ran a test to add ~2500 multi-select options to a property. I noticed that some parts of Notion start to become sluggish when working with that many options, so this is something to keep in mind if you have a lot of different authors in your database.
I want to support both text and multi-select, so if you try multi-select when it becomes available and you find that it slows down Notion too much, you should be able to switch back to text and re-sync your items to restore them to how they were before.

That's a good point, @pcuellar. We could consider adding another option to select the separator used for text properties.
In the meantime, you can try disabling the "Wrap column" option on the Authors property:
https://github.com/user-attachments/assets/2ad565ec-932d-467a-8ff4-a54048a360de

Controlling the separator will be ideal, for sure. Thanks for the suggestion, I am using a local script to fix the trailing spaces. Will keep an eye on the repo in case you fold this into a future release


Data Deletion with Version 1.2 Deployment issue

I would like to report an issue that occurred following the deployment of Version 1.2.
Upon completing the deployment, we noticed that:
All previously uploaded files were deleted from the Blob storage. including content and uploaded container
All existing records were deleted from the Cosmos DB.
This data loss was not intended and has impacted the availability of previously uploaded files and records. Please investigate the cause of this issue and suggest corrective actions to prevent it from happening in future deployments.
If any recovery options are available for the deleted files and records, please let me know what the next steps.
I am looking forward to your prompt response.

Hi Ravi,
I am currently looking into this issue.  Which version did you upgrade from?  When you deployed the upgrade, did you have
your prior TF state files from your previous deployment?

I have upgraded from v1.1.1,
Yes when I upgraded prior TF state files was there

Please review your TF plan file and see if the Storage Account and Cosmos DB were marked for deletion and recreation.
In regards to data recovery, since the entire Storage Account and Cosmos DB were deleted, they are not recoverable via the portal.  I advise submitting an MS ticket to see if there's anything they can do if the data is not easily replaced.

Issue Replication Scenario:
**1.**Current Version (V.1.1.1):

The system has storage files and database (DB) data.

**2.**Upgrade to V1.2:

Perform the deployment using Terraform (TF) files. If any TF file is removed or modified, it will recreate resources (like storage and DB) from scratch after deleting the existing ones.
The deployment is successful, but the issue occurs where all storage files and database data are wiped out (cleared).

**3.**Post-Upgrade to V1.2:

Attempt to upload new files. However, all files get stuck in the "Upload" state.
Alternatively, trying to resubmit files via the UI results in them getting stuck in the "Queue" status (similar to the issue documented here).

Another Scenario:

After V1.2 Upgrade:

Some files were uploaded successfully, or they are still stuck in the "Uploaded" or "Queue" state, and corresponding DB entries exist.

Downgrade Attempt to V1.1.1:

Attempting to downgrade back to V1.1.1 faces issues and doesn't succeed. The TF files are then updated back to V1.1.1.

Upgrade Again to V1.2:

After resolving the downgrade issue, upgrading again to V1.2 results in a successful deployment.

Same Issue:

Despite the successful deployment, the same issue occurs: all previously uploaded files (whether in "Uploaded" or "Queue" status) are wiped out, and both the storage account and Cosmos DB are cleared out again.

This sequence of events highlights that upgrading/downgrading between versions, especially with changes in Terraform files, can lead to unintended deletion of storage and database data, causing file uploads to get stuck or wiped out.

All right, might that not at this moment big issue because not working on directly production environment, but as issue arrives in Development time so same could be possible for the Production as well and thats why want to know what exactly it’s causing

The terraform plan file will let you know what has been marked for deletion and recreation.  You can inspect that file to see what has changed and the proposed action from Terraform.


New command : ~addalias

~addalias on all commands (or custom commands only)
If added, redo all aliases to use this suite (add/remove alias)

This could be done on all commands, but I only really want to do it for custom commands. BTW, before we get too involved with custom commands we should read more about what discord 2.0 will look like and the future of discord bots. Some of this stuff may be useless if, for instance, bots can no longer read messages. wouldn't want to spend our next 2 months developing custom commands if they get nuked come January.


Use Dependency-Lint

https://github.com/charlierudolph/dependency-lint

added https://github.com/Originate/lodash-match-pattern/pull/23


Fixes reversed parameters for new NameInfo call in FromOptionSpecification

The ShortName/LongName parameters are reversed when creating a new NameInfo in FromOptionSpecification.

Excellent sharp eyes @cfeilen! PR merged. :+1:
Thanks for contributing! :smile:


fix: display property edit button when accessing with not author account

Proposed Changes
Implementation

care of loggedInWallet === undefined && authorAddress === undefined pattern

Thanks!


Fails and hangs if filename contains a '[hash]'

Currently, the plugin only looks for assets with the exact same name mentioned in the config file. But if a hash is included in the filename, the plugin won't be able to find it.
This results in a unhandled promise rejection, and the compilation hangs. It seems that the callback is never called if that problem happens. That may be related to #19.

+1


Add NASA Earthdata GIS Rest API into QGIS image

Context
Both VEDA and GHG jupyterhubs have deployed QGIS images for their respective jupyterhubs. We would like tighter integration of those hubs with the NASA Earthdata GIS data catalog. The Earthdata GIS layers are availble via a Rest API and a connection can currently be established manually. It would be great if the Rest API connection was preloaded in the QGIS image so the Earthdata catalog would be available by default.
Proposal
No response
Updates and actions
No response

I'd love for help from devseed folks here, as this basically is just qgis work. I know they have folks in there with qgis expertise, and it would be extremely helpful here. The alternative is that I basically dive into learning more about qgis itself, and I'm not sure how that would work from a resourcing / timing perspective.

Some information that may be useful.

We're currenly running this fork of the stac plugin (https://github.com/slesaad/qgis-stac-plugin/tree/support_gdal_assets) to allow qgis to pick up AWS credentials automatically. It is basically just bringing in a few unmerged PRs from upstream (https://github.com/stac-utils/qgis-stac-plugin) that was needed because upstream looks dead :( I don't know what the long term plan for this would be - currently, just by dint of owning the repo, @slesaad is on the hook for it and idk if that's intentional :D
The image itself if coming from https://github.com/jupyterhub/jupyter-remote-desktop-proxy/tree/base-image/images/qgis. I'll probably move it to its own repo, as we're installing custom forks of qgis plugins and stuff so it's probably not the best fit for an upstream image.

So I think whatever integration is needed, would look like:

Figuring out what combination of qgis plugins + config is needed to make that happen locally, on someone's machine
Modifying the docker image to have those plugins + config
Figuring out authentication, if any is needed.

Does that help?

Does that help?

Hugely. Thank you!

Modifying the docker image to have those plugins + config

This might be a naive question, but do you have a recommended workflow to test whether "this works"? i.e. run docker image locally some-how and try connecting to the running QGIS or so? Or do we just push changes to the Dockerfile and wait for help to get it deployed to a test environment?

Figuring out authentication, if any is needed.

Would be good to know more about what this might entail? Is this authentication from within QGIS to access the required datasets / catalogs? Or is this some-how managing access to the QGIS setup (which I assume is just handled by the Jupyterhub infrastructure?). We can come back to this if it becomes an issue.

FYI @jmunroe. Discussion here can likely be scooped into improved documentation to enable more community managed customization.

Is this authentication from within QGIS to access the required datasets / catalogs?

Yes, this is what needs to be handled. If it requires AWS creds, it just needs to recognize it's in an environment where it can pick up AWS creds from the env and pass them on. Nothing JupyterHub specific.

This might be a naive question, but do you have a recommended workflow to test whether "this works"? i.e. run docker image locally some-how and try connecting to the running QGIS or so?

You can definitely run it locally!  You should be able to access jupyterlab with that tokenized URL, and launch qgis from there to see how that goes.
I think to test AWS creds you may need to run in-cluster, but you can also do that easily by building and pushing the image (i recommend a test account on quay.io as registry), and then using the feature of the veda hub that lets you write in the image to use (thanks for helping building that, @batpad ;)). This should reproduce the exact set up so you can see what needs to happen.

cc @Rub21 @sunu - let's figure a time to look at next steps here.

The Earthdata GIS layers are availble via a Rest API and a connection can currently be established manually.

@freitagb is it possible to get a bit more clarity on what this looks like, for those of us with a bit less overall context? Ideal would be a screencast of the manual process, or even just screenshots of having the layers loaded would be great, just so we have a precise idea of the goal / what needs to be replicated in the image definition.
Once we understand the goal correctly / what manual process in QGIS we just need to automate at startup, we can go ahead and experiment with changes to the Docker image to get things to the desired state.
@yuvipanda thanks again for all the super-clear guidance! It feels like there's quite a lot we might want to do with the "QGIS in the browser" feature down the line - figuring out the Earthdata GIS layers sounds like a good first step to get our feet wet, before getting into stuff that could be significantly more complicated, which we can worry about later.

I was chatting with @batpad about QGIS integration so thought I'll chime in here with some ideas. This is very exciting — I love getting QGIS preloaded with data sources, have them show up as layers etc.
1. Use a QGIS project file (.qgs)
This is probably the least effort method. The steps might look like this — manually add the layers once in QGIS (locally or otherwise), save the QGIS project. Put the project file somewhere the container can read from or pull it while building the image. Write a small startup script that runs when QGIS launches which then loads the project. The project file is quite powerful. They can hold a lot of configuration including default enabled plugins. They can be shared, including any spatial analysis or visualization. Making them almost like notebooks within QGIS!
2. Use a QGIS profile with custom configuration files
QGIS uses a default profile by default. We can change this to have a custom profile which then can have a custom configuration (.conf, .ini, .plist or so) that allows us to edit any QGIS attribute. For example, we can edit projOpenAtLaunchPath variable so QGIS opens the project at launch with no startup scripts. Profiles are really cool because all plugin configuration also lands in the main configuration. For example, the QGIS STAC plugin can take custom STAC connections very easily.
3. A QGIS plugin
A plugin will be really be a nice way to do this but a bit more involved. We can manage the plugin code separately, have qgis configured to load the plugin by default. The plugin can then configure all aspects of working with EGIS data sources including custom layers, styles, managing auth etc. Some folks on our team built plugins extensively (example) that has a lot of UI components, disk reading, network requests, metrics etc.
I'm happy to outline any of these above if helpful.

Thanks for outlining this from the QGIS end @geohacker ! This is all very exciting.
It seems like the quickest way to get the Earthdata GIS Rest API into the hosted QGIS is to just auto-load a custom Project file? This would also allow us to do simple things like ensuring the STAC plugin is activated by default.
It does seem like if we want to start adding more complex features and data sources, the best architecture would be to develop that as a standalone QGIS plugin that we load by default, where we can have a lot of flexibility in what we do.
@j08lue @freitagb we might need a bit of help in getting the exact configuration options to load https://gis.earthdata.nasa.gov/image/rest/services into QGIS, and if any authentication is required, but it seems like then it should be pretty straightforward to create the project file and make the changes required in the container to load that project file on startup, which should give us what we want here.
@geohacker couple more things from your notes that we should follow up on:

They can be shared, including any spatial analysis or visualization. Making them almost like notebooks within QGIS!

Would be great to see if we can build any features to support this?

The plugin can then configure all aspects of working with EGIS data sources

I like the idea of encapsulating all the common data sources and things we know users will want as a separate plugin(s) that can be maintained separately, and useful beyond just QGIS inside JupyterHub. I think how we architect this will depend a bit on if we see this as a one-off feature, or if we envision building more integrations into QGIS in the future. For eg. if we want to allow users to "open dataset in QGIS" from a webpage, we'd definitely need to build it out as a plugin.
Thanks again for the notes @geohacker - it makes it really clear on how we should structure things on the QGIS side and what good short-term and long-term approaches look like.

Would be great to see if we can build any features to support this?

I've never done this in practice but we have passed around the project file a couple of times and it worked great. You could combine it with version control and then allow people to fork/commit changes. Very speculative, could be possible.

I like the idea of encapsulating all the common data sources and things we know users will want as a separate plugin(s)

For a plugin I might suggest structuring it somewhat as a standalone QGIS server. Then you can easily have triggers and a REST interface to configure, load data, perform other operations etc.

@j08lue @freitagb we might need a bit of help in getting the exact configuration options to load https://gis.earthdata.nasa.gov/image/rest/services into QGIS, and if any authentication is required

Who would be the right person to connect to, to get these details, @freitagb?

I've moved this image out to its own repo, in https://github.com/2i2c-org/nasa-qgis-image. I've invited @batpad, @freitagb and @slesaad as admins on that repo - feel free to send out access as you wish. It gets automatically built and pushed to https://quay.io/repository/2i2c/nasa-qgis-image?tab=tags

Thanks @yuvipanda !
As I see next steps here:

Test local setup. See @yuvipanda's comment above
Modify Docker image / start commands to:

Automatically open QGIS in the desktop environment
Auto-load a project file that we have pre-loaded into the docker container

After that or in parallel, we can work on creating a QGIS project file pre-loaded with all the data sources and configuration we need to add the Earthdata Rest APIs and anything else we'd like always pre-loaded by default.

I can start on just figuring out local setup. Once that makes sense, @geohacker @Rub21, we can follow up on next steps to auto-configure QGIS.

Going to close this as it's tracked in the veda-jupyterhub issue now. Thanks @batpad


DUOS-760 [risk=no] Initial Page layout and styling of Chair Console

Addresses DUOS-760
NOTE: Do not merge this branch in until feature is complete. This branch will serve as a feature base, with each new ticket branching off of this one. On completion the branch will be merged into develop
Initial styling and table layout of the new Chair Console, as well as modal depicting DAR details on click. Some scaffolding added for pagination and search, but they serve as as an initial stepping stone and are not indicative of the end result of that option. As such the table right now will show all DAR entries up front since view control (since view control is tied to pagination and search).
Have you read Terra's Contributing Guide lately? If not, do that first.
I, the developer opening this PR, do solemnly pinky swear that:

[x] PR is labeled with a Jira ticket number and includes a link to the ticket
[x] PR is labeled with a security risk modifier [no, low, medium, high]
[x] PR describes scope of changes

In all cases:

[ ] Get a minimum of one thumbs worth of review, preferably 2 if enough team members are available
[ ] Get PO sign-off for all non-trivial UI or workflow changes
[ ] Verify all tests go green
[ ] Squash and merge; you can delete your branch after this
[ ] Test this change deployed correctly and works on dev environment after deployment

See comments inline ... the page starts to load fine, but fails when looking at an election dac. We need to know if this is a UI problem or an API problem, but since this is not using the correct API, it's hard to tell at the moment.
This is my screen as a DAC Chairperson:

Code loads fine on my end. Error is indicating that there's no name attribute on null, which would mean there's no DAC assigned to an election? Is that possible?

See comments inline ... the page starts to load fine, but fails when looking at an election dac. We need to know if this is a UI problem or an API problem, but since this is not using the correct API, it's hard to tell at the moment.
This is my screen as a DAC Chairperson:

Code loads fine on my end. Error is indicating that there's no name attribute on null, which would mean there's no DAC assigned to an election? Is that possible?

@JVThomas Yes, it could be true (especially in dev) that there are elections missing a DAC. We also need to be looking at the /api/dar/manage/v2 endpoint for data.

@JVThomas Yes, it could be true (especially in dev) that there are elections missing a DAC. We also need to be looking at the /api/dar/manage/v2 endpoint for data.

@JVThomas Yes, it could be true (especially in dev) that there are elections missing a DAC. We also need to be looking at the /api/dar/manage/v2 endpoint for data.

For the elections missing a DAC, is that a possible format for an election or is it a bugged record that needs to be filtered out?

@JVThomas Yes, it could be true (especially in dev) that there are elections missing a DAC. We also need to be looking at the /api/dar/manage/v2 endpoint for data.

For the elections missing a DAC, is that a possible format for an election or is it a bugged record that needs to be filtered out?

@JVThomas

For the elections missing a DAC, is that a possible format for an election or is it a bugged record that needs to be filtered out from the view?

The point is moot - the API this PR is using is buggy & being deprecated. The v2 dar manage API should be doing the right things with respect to getting the DACs for the datasets of the DARs instead of trying to get it from the election.

@JVThomas

For the elections missing a DAC, is that a possible format for an election or is it a bugged record that needs to be filtered out from the view?

The point is moot - the API this PR is using is buggy & being deprecated. The v2 dar manage API should be doing the right things with respect to getting the DACs for the datasets of the DARs instead of trying to get it from the election.


dotnet dev-certs https --trust requires install

Add dotnet tool install --global dotnet-dev-certs and dotnet dev-certs https -c
see https://github.com/aspnet/AspNetCore.Docs/issues/14776#issuecomment-537815699
The troubleshoot section only applies to Kestrel. For  VS/IIS Express, run the installer repair tool. See https://github.com/aspnet/AspNetCore/issues/16892#issuecomment-556080754

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

ID: c7ddfb2f-5dea-5e31-2a27-02a4dc16104c
Version Independent ID: 12ca0b9a-2638-a4ce-c45f-d4579c83dafb
Content: Enforce HTTPS in ASP.NET Core
Content Source: aspnetcore/security/enforcing-ssl.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Might be good to add when this issue is worked ...
The only thing we say about install is that the SDK install does it on first run. However, I had an expired cert and needed to put a new one in place (turns out that a new SDK install doesn't replace it automatically). The topic could call out that running dotnet dev-certs https will provide a new cert, then one trusts it, so the sequence should be clean-install-trust for this scenario.

Moved to Enforce HTTPS Master issue #17356


[WIP] Adding the remaining PMO tasks

This PR adds the remaining chemistry-related black boxes, which are based on the PMO benchmark.
Thankfully, most of these are already available as TDC oracles.
(Closes #166 )

Running thorough-er tests, I've realized there's a small bug in PyTDC. I'll keep implementing the other PMO black boxes.


Save the random seed in the SCORM suspend data #634

Hi, I'm not sure if it's right, but i did basically what you said.
Also, i guess i could help you with Russian translation for your project (I'm native).
#634

Thanks for this! Could you add your name to the CONTRIBUTORS file please?

@christianp yeah sure, that's it

Thanks!


Show icon if image preview is missing

Rather than a blank empty space, looks like a regular file instead now

@flamerohr I've reviewed and added a suggested tweak at https://github.com/open-sausages/silverstripe-framework/pull/18 (PR to your PR, if that's ok).
Sorry I didn't catch this issue earlier. :)

I've merged https://github.com/silverstripe/silverstripe-asset-admin/pull/255, by the way. :D

I'd be happy with your Pull Request rather than this one!

So can this be closed now @flamerohr ?

actually yes, lets close this and have the other one @tractorcow did merged :)


Fix DBus for multiple independent handlers

Now that we're going down the libtcmu path for supporting handlers other than tcmu-runner, if we're going to present a coherent picture of the available handlers to the user then they need to cooperatively register in one place. We can use Telepathy "mission-control-5" as a model for how to accomplish this.

Hey Andy, I think the embedded cases (ie large applications linking libtcmu.a) are orthogonal to the tcmu-runner's system/dbus registration and interfaces. That is a system would have the tcmu-runner with its plugins or a larger application with libtcmu linked.
Here is how I see the embedded case:

the application wants to reuse SCSI command processing and the user/kernel protocol implementation
the device registration process is tricky/awkward/proprietary
the application would have little interest in responding DBus messages

it would want to minimize the dependencies

Yes it's not relevant to the embedded case. It's relevant to the distro use case, and I think we can accommodate both.

Does it make sense to handle the dbus communications in libtcmu?

I think so.. it's just a matter of figuring out how to do it, given that multiple processes could be using libtcmu, and only one entity can back a DBus service. I looked into how Telepathy does this, but got lost in all the code.

I think we can extent tcmu-runner's org.kernel.TCMUService1 interface and let it be the mid-man coordinator between multiple handlers and targetcli.
Currently it only implements a CheckConfig method to verify the cfgstring, and provides object introspection for enumerating available subtypes, for targetcli. If we extent this service for libtcmu users as well, targetcli user interface is extended automatically. Specifically, we can add a "RegisterHandler" method (family) alongside CheckConfig, which accepts subtype string from libtcmu.
One more related feature I'd like to discuss is file descriptor passing.
In a typical virtualization setup, QEMU doesn't open files by itself, instead it gets FDs passed from libvirt, which opens image files on its behalf, via unix socket control messages. This is so because of privilege limitation for security concerns. In TCMU, QEMU process would need to open /sys/class/uio and /sys/device/tcm_user, which is not desired. A solution is let tcmu-runner who gets DBus messages do this, and pass it back to libtcmu through a unix socket. I think this is doable.
In the case of qemu-tcmu (a standalone TCMU handler utility) we may not need this, just like when qemu-img is invoked by libvirt, it has access to images etc, which kinda suggests we could run qemu-tcmu with the permissions to uio. But in my first take on TCMU support in QEMU, I would like to introduce an embedded handler, like the NBD server in qemu-system-* processes. Because: 1) this is easier to do and 2) this is easier to justify - it allows QMP which is the control path to manage the targets, and it enables virtualization use cases that virtual disk data is accessed in the host, through LIO fabrics, with SCSI commands, compared to which the current NBD protocol is poorer (see also the recent discussions around extending NBD command set to allow querying block allocation states in qemu-devel@nongnu.org).
What do you think, Andy?
Fam

I think extending DBus to allow libtcmu users to RegisterHandler with tcmu-runner is good, some design issues I have are deregistration, or what we do when another process supporting handlers goes away? I don't have an answer (and was hoping Telepathy's implementation did). And also, how would tcmu-runner handle proxying check_config calls, that sort of thing.
Regarding FD passing, I have no immediate objections, but maybe we can open a separate issue to discuss further, or when you have a PR?

My idea is like this:
At the time of RegisterHandler, tcmu-runner <--> $FOO_HANDLER open an out-of-bound unix socket. The socket can be used by tcmu-runner to do check_config calls upon targetlib's DBus.CheckConfig call: tcmu-runner sends a check_config query to $FOO_HANDLER through the unix socket, and wait for reply. (We need a mini protocol here, which could be built on, for example, json-rpc.)
With that, the deregistration would be implicit, when the unix socket is closed (G_IO_HUP).
If this sounds okay, I can work on patches implementing it, and probably extend it to support FD passing, in the coming PR.

I feel like DBus should also be used here, instead of our own rpc mechanism. But if you'd like to focus on getting your stuff working and want to implement what you describe in the short term, that's fine, and I can worry about converting to dbus (I do believe dbus can pass fds, fwiw). This is an internal interface this future change should not cause disruption.

It is better if we can stick to dbus, but I get questions I cannot answer. I am not a DBus expert, but reading through the documents I couldn't find a way to do a callback from DBus service (tcmu-runner) to clients (handler process).
How would you implement check_config with DBus? And how would tcmu-runner know when the handler is killed?
I'd like to do some work here for long term, after all QEMU cannot rely on any internals of a daemon.

I'm not a dbus expert, but I'll look more into doing what we want to do today, and ask the experts on the dbus mailing list as well.

Opened #40 to discuss the FD thing in a separate issue from the DBus stuff.

I think from looking at this if we have each user of libtcmu provide a distinct well-known bus name with the TCMUService1 API or something similar, then tcmu-runner can get signals when bus names show up or disappear, reply to the current API with the complete list of handlers, and proxy API calls like CheckConfig through to the specific handler instance.


New tor circuit?

How can I request a new Tor circuit (to change my output IP) from a started TorClient?
I do not found any documentation to help...
Thanks

yeah i would love to know the same thing, and maybe even implementing 2 connections at the same time so if one fails it automatically falls back to the other one so I dont get disconnected

In my experience, every time you create a new socket connection over tor your output ip will change.
Sudo Code:
        TorClient client = //initialized somehow
        String host = //initialized somehow
        String path = //initialized somehow
        String query = //initialized somehow
        Socket socket = client.getSocketFactory().createSocket(host, 80);
        PrintWriter writer = new PrintWriter(socket.getOutputStream(), true);
        InputStream inputStream = socket.getInputStream();
        writer.println("GET "+path+query+" HTTP/1.0");
        writer.println("Host: "+host);
        writer.println("Connection: close");
        writer.println("");
        writer.flush();

For instance, I have a wrapper project tor Orchid. By slightly modifying one of my HTTP GET request example you will be able to see the change in IP address with every request.
        TorRequest.openTunnel();
        TorRequest whatIsMyIp = TorRequest.getInstance("http://ip-api.com/json");
        try{
            whatIsMyIp.executeRequest();
            System.out.println("Attempt #1: "+whatIsMyIp.getResponse().getContent());
            Thread.sleep(100l);
        } catch(Exception e){
            System.out.println("Attempt #1 Failed: "+e.getMessage());
        }
        try{
            whatIsMyIp.executeRequest();
            System.out.println("Attempt #2: "+whatIsMyIp.getResponse().getContent());
            Thread.sleep(100l);
        } catch(Exception e){
            System.out.println("Attempt #2 Failed: "+e.getMessage());
        }
        try{
            whatIsMyIp.executeRequest();
            System.out.println("Attempt #3: "+whatIsMyIp.getResponse().getContent());
        } catch(Exception e){
            System.out.println("Attempt #3 Failed: "+e.getMessage());
        }
        TorRequest.closeTunnel();


fix: change input type of amount and budget fields to number (#256)

What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[x] Bug Fix
[ ] Optimization

Description
Related Tickets & Documents

Related Issue #256
Closes #256

Added/updated tests?
We encourage you to keep the code coverage percentage at 80% and above.

[ ] Yes
[x] No, and this is why:
[x] I need help with writing tests

Oh, it seems some tests have failed, and I think my changes were the cause.

@AmirAbdollahi ,
Yes, but no problem, you can run the tests locally easily. just run the application as usual and than go the folder 'easyfinance.client' and run the command npx cypress open. Then you can run the tests and see step by step what happen. Maybe just need to change something in the tests.
Let me know if you need some help.

@AmirAbdollahi
I modified the contributing.md to have more detail how to run the tests. If you need help to analyze them, just let me know.

@AmirAbdollahi
I modified the contributing.md to have more detail how to run the tests. If you need help to analyze them, just let me know.

Sure! Thank you very much. @FelipePSoares

@AmirAbdollahi
I modified the contributing.md to have more detail how to run the tests. If you need help to analyze them, just let me know.

Hi @AmirAbdollahi,
I saw that you comment empty. Can I help you with something?

Hi @AmirAbdollahi,
I saw that you comment empty. Can I help you with something?

Oh, I'm sorry. That was a small mistake on my part.

@AmirAbdollahi ok, no problem, just asking if you need something, but if everything is fine, good.
Have a nice day.

I'm so thrilled to see all the tests are green.

Hi @AmirAbdollahi,
I found some issues in specific languages and browser, but don't worry about that, thank you for your help, I'm working on your branch changing some details, when I finish it I'll update this MR with this changes and I advice you to check.
Sorry for not approve yet. But you make a good job, just some small points that I don't thing when I wrote the issue. hahaha

Hi @AmirAbdollahi ,
After a lot of study and errors. kkk I finally find a solution for the problem, but unfortunately I need to change from type="number" to type="text" again, because when it's type="number" the navigator just send the value when is a valid number, otherwise it send null and null in this situation is a valid value, because you can create a expense just with budget without value.
I will apply now the solution for the others forms, but I think that this way with inputmode="decimal" and validation based on user's language will be the best solution.
Thank you a lot and sorry to create a issue to put type="number" and now make it different, I really believed that type="number" was the best solution, but, no.

@AmirAbdollahi ,
No it's perfect, it's my fault, I'll finish now this issue. The other one that you are working on will be so helpful also. Thank you

Hi @AmirAbdollahi,
Finally I finished it. If you want to see what I did you can see in the changes. Now I'll just wait you finish yours to avoid conflits and then I apply this for edit forms also.
Thank you for your help again.

Hi @AmirAbdollahi,
Finally I finished it. If you want to see what I did you can see in the changes. Now I'll just wait you finish yours to avoid conflits and then I apply this for edit forms also.
Thank you for your help again.

Thank you very much @FelipePSoares .
Ok. I'll try to finish it tonight.

Resolving conflicts ...

Resolving conflicts ...

Hi @FelipePSoares,
Merge conflicts have been resolved. Please review the changes again before merging the branch.

Ok thanks

@AmirAbdollahi,
I need to apply the changes to the edit forms that you worked also.

Hi @AmirAbdollahi,
Finally we finalized this issue. thanks you for your help.


In YogaUIView's layoutNodes method, return early for nested YogaUIViews to prevent redundant frame calculations.

Follow-up to PR #1996

[x] CHANGELOG.md's "Unreleased" section has been updated, if applicable.

Whoa. Nice! Just FYI. We've been digging into bugs here where we're not getting enough layout passes for views that get more children and should resize but don't. Not sure how this will play into that
@aridokmecian @edahle

Interesting--this was found while trying to solve the opposite problem. When a YogaUIView is nested within another YogaUIView (in the case I was investigating, it was a Column within a Row), then the child view calls layoutSubviews twice in a row on appearance, which is totally unnecessary--I made some local changes so that the second call no-op'ed, and the resulting layout was still as expected.
This double layoutSubviews call is still occurring with this change in place, but this felt like a step in the right direction regardless.


cloud_firestore package, missing file(s)

I'm working on an app and i wanna use cloud_firestore, but if i add the package in the dependencies, I get this error:
Xcode's output:
↳
In file included from
...app/ios/Pods/FirebaseFirestore/Firestore/core/src/firebase/firestore/nanopb/writer.cc:17:
In file included from
...app/ios/Pods/FirebaseFirestore/Firestore/core/src/firebase/firestore/nanopb/writer.h:28:
...app/ios/Pods/Headers/Public/gRPC-C++/grpcpp/support/byte_buffer.h:22:10: fatal error:
'grpc/byte_buffer.h' file not found
#include <grpc/byte_buffer.h>
If i just use these packages:
firebase_core: ^0.4.3+3
firebase_analytics: ^5.0.11
firebase_auth: ^0.15.4
everything works fine, the problem appears when i add cloud_firestore.
I don't know if it can help but all my dependencies are:
flutter:
sdk: flutter
http: ^0.12.0+4
webview_flutter: ^0.3.19+5
flutter_staggered_grid_view: ^0.3.0
url_launcher: ^5.4.1
email_validator: ^1.0.4
firebase_core: ^0.4.3+3
firebase_analytics: ^5.0.11
firebase_auth: ^0.15.4
cloud_firestore: ^0.13.0+1
I already flutter upgrade, reinstall pods, cocoapods... and flutter doctor have no issues
I also try to change the channel to dev but nothing changes
I hope someone can help !

I have same error.
I hope someone can help too!!!!

Same issue here!
[✓] Flutter (Channel stable, v1.12.13+hotfix.7, on Mac OS X 10.15.3 19D76, locale en-IN)
• Flutter version 1.12.13+hotfix.7 at /Users/anmolverma/flutter
• Framework revision 9f5ff2306b (13 days ago), 2020-01-26 22:38:26 -0800
• Engine revision a67792536c
• Dart version 2.7.0
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/anmolverma/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.3
• Java binary at: /Users/anmolverma/Library/Application
Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.6010548/Android
Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.3.1, Build version 11C504
• CocoaPods version 1.8.4
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 43.0.1
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] Android Studio (version 3.5)
• Android Studio at /Users/anmolverma/Library/Application
Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.6010548/Android Studio.app/Contents
• Flutter plugin version 43.0.1
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] VS Code (version 1.41.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.8.0
[✓] Connected device (2 available)
• SM N960F       • 4a44434350413598                         • android-arm64 • Android 10 (API 29)
• Anmol’s iPhone • b79dd3e8a73f8a6b4777dc90cc2e6d42ddb9b5ac • ios           • iOS 13.3.1
• No issues found!

I found a solution.
sudo gem install cocoapods --pre

I'm glad if you can help!

@bakatsuyuki Thanks a lot! I ran pod install after that command and whoosh! its working!

Works fine for me thanks a lot !
I just change the package to the new one
cloud_firestore: ^0.13.0+1 to cloud_firestore: ^0.13.1+1
and after I ran exactly as you said
sudo gem install cocoapods --pre pod install
Thank you so much !


Fix rendering

Description
fixes weird rendering
Type of change
Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)

Added divider

Done


shims/super/cc: Use -idirafter instead of -isystem [Linux]

Fix
error: stdlib.h: No such file or directory
 #include_next <stdlib.h>

See https://github.com/Linuxbrew/brew/issues/724

Codecov Report

Merging #733 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #733   +/-   ##
=======================================
  Coverage   68.28%   68.28%
=======================================
  Files         393      393
  Lines       20983    20983
=======================================
  Hits        14329    14329
  Misses       6654     6654

Continue to review full report at Codecov.

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

As for solutions, one way is to add -idirafter HOMEBREW_PREFIX/opt/glibc/include in the end of gcc spec file for the users who use brew glibc. Another way is using -idirafter HOMEBREW_PREFIX/include instead of -isystem=HOMEBREW_PREFIX/include in the superenv wrapper.

https://github.com/Linuxbrew/brew/issues/724#issuecomment-397803958
Your other suggestion was to add HOMEBREW_PREFIX/opt/glibc/include to homebrew_extra_isystem_paths. If it works, I think I'd prefer that to this PR.

The another solution would be like this: https://github.com/Linuxbrew/homebrew-core/pull/8108

Thanks, @xu-cheng !
Closed without merging in favour of https://github.com/Linuxbrew/homebrew-core/pull/8108


Incorrect project link

Plugins > JsonViewer > About, Click on the website link which shows "https://github.com/kapilratnani/JSON-Viewer".
It opens up not the github, but the sourceforge link.
The link here should be changed
https://github.com/kapilratnani/JSON-Viewer/blob/76a71f6df47982e6860a21ec52a51b07d78f2661/NppJSONViewer/PluginDefinition.cpp#L99

Thanks for pointing this out..

Fixed


[HOTFIX] rename field "thread_pool_size" to match camel case

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[x] Any interfaces changed? NO

[x] Any backward compatibility impacted? NO

[x] Document update required? NO

[x] Testing done YES
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA. OK

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1911/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10165/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2122/

Can you please explain , why need to rename?

Can you please explain , why need to rename?

thanks for review, thread_pool_size doesn't match java code style.
for example, field names may be carbonLRUCache, dictionaryColumnUniqueIdentifier etc

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1956/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10209/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2231/


Unable to uninstall theme?

Hi,
I tried out your theme, but then I didn't like the "shade" effect on my other panes (if for example they are synced, I don't clearly see the text unless I go to that pane) so I decided to remove it.
Nothing works from commenting out and sourcing .tmux.conf, and also removing it from tmux/plugins folder.
Any tips?

When you create a new tmux session you will have default + your ~/.tmux.conf settings again.

In case anyone else wants an easy way to toggle fading the inactive pane text, there's a PR here: https://github.com/odedlaz/tmux-onedark-theme/pull/26


wrap demo in "cog"

Hi!  Really cool demo.  I wanted to share it with someone who wasn't a dev - so I tweaked it to work in "replicate"
I wrapped it with cog  - a tool replicate.com built to make deploying ML models easier:

deploy on replicate.com (I pushed a test to https://replicate.com/anotherjesse/facial-landmark-detection to see if it works) - it lets you run via a web http on their server.
run local with cog predict -I<EMAIL_ADDRESS>(it runs in a docker container)

I am sending a PR of what I did to make it so it works (as you can see, there isn't really any changes from your code.
To test locally I ran: cog predict -I<EMAIL_ADDRESS>then when I had it working locally I uploaded with: cog push r8.im/anotherjesse/facial-landmark-detection
I will delete the replicate project in a couple days as it is your work not mine.  If you want to re-create the replicate project, you can use this PR
Feel free to delete this PR :)

Hi Jesse!
Thank you for the PR :D I'll have a look and see if I'd like to merge it or maybe collaborate on a repo for this. Appreciate it!


[Feature Request]: add codeql workflow

Is your feature request related to a problem? Please describe.
The repository contains code in javascript, it does not have workflow for code scanning.
Describe the solution you'd like
I want to add the codeql workflow to automate security checks. CodeQL is the code analysis engine developed by GitHub to identify vulnerabilities in code. It will analyze your code and display the results as code scanning alerts. It will be enabled on every push, commit and pull request using GitHub actions.
Record

[x] I agree to follow this project's Code of Conduct
[x] I'm a GSSoC'23 contributor
[x] I want to work on this issue

Okay, this looks like a good feature. You can go ahead and add it.


Error:  Executor failed to create kernel. No registered 'Snapshot' OpKernel for GPU devices,while running image label example

System information

OS Platform: Windows10
TensorFlow installed from source  (GPU support)
Python version: 3.5
Compiler version : Microsoft Visual Studio 2015 x64
CUDA/cuDNN version:CUDA9.0/cuDNN7.0
GPU: GTX 1070 8G
-CMake settings:

Problem
I successfully built tensorflow.lib and tensorflow.dll, then I linked these to my visual studio project.Everything goes well when I run the example code which does matrix multiplication (the example code on:https://joe-antognini.github.io/machine-learning/windows-tf-project)
But when I tried to run the Label_Image example code(https://github.com/tensorflow/tensorflow/blob/master/tensorflow/examples/label_image/main.cc), I got the error:

E C:\tensorflow_gpu\tensorflow\tensorflow\core\common_runtime\executor.cc:644] Executor failed to create kernel. Not found: No registered 'Snapshot' OpKernel for GPU devices compatible with node Subtract = Snapshot[T=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"](ResizeBilinear)     .  Registered:  device='CPU'; T in [DT_INT64]device='CPU'; T in [DT_INT32] device='CPU'; T in [DT_UINT16] device='CPU'; T in [DT_INT16] device='CPU'; T in [DT_UINT8] device='CPU'; T in [DT_INT8] device='CPU'; T in [DT_HALF] device='CPU'; T in [DT_BFLOAT16] device='CPU'; T in [DT_FLOAT] device='CPU'; T in [DT_DOUBLE] device='CPU'; T in [DT_COMPLEX64] device='CPU'; T in [DT_COMPLEX128] device='CPU'; T in [DT_BOOL]      [[Node: Subtract = Snapshot[T=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"](ResizeBilinear)]] 2018-03-14 17:13:05.408206: E Image_Label.cpp:385] Not found: No registered 'Snapshot' OpKernel for GPU devices compatible with node Subtract = Snapshot[T=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"](ResizeBilinear)     .  Registered:  device='CPU'; T in [DT_INT64] device='CPU'; T in [DT_INT32] device='CPU'; T in [DT_UINT16] device='CPU'; T in [DT_INT16] device='CPU'; T in [DT_UINT8] device='CPU'; T in [DT_INT8] device='CPU'; T in [DT_HALF] device='CPU'; T in [DT_BFLOAT16] device='CPU'; T in [DT_FLOAT] device='CPU'; T in [DT_DOUBLE] device='CPU'; T in [DT_COMPLEX64] device='CPU'; T in [DT_COMPLEX128] device='CPU'; T in [DT_BOOL]      [[Node: Subtract = Snapshot[T=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"](ResizeBilinear)]]
I found out the code stopped at:
TF_RETURN_IF_ERROR(session->Run({ inputs }, { output_name }, {}, out_tensors));
in
Status ReadTensorFromImageFile(const string& file_name, const int input_height,     const int input_width, const float input_mean,     const float input_std,     std::vector<Tensor>* out_tensors)
Does anyone know how to fix this problem??

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow version
Bazel version
GPU model and memory
Exact command to reproduce

@tensorflowbutler
Have I written custom code: no, using example code only
OS Platform and Distribution: Windows10
TensorFlow version: v1.7
Bazel version: N/A
GPU model and memory:  nvidia geforce 1070 8G
Exact command to reproduce: N/A, the error code is from visual studio project after build and run the example code
Thanks!

The error says that the SnapshotOp kernel is not available for GPU.  It's a relatively new kernel.  You said you compiled from source, so I believe this kernel should have compiled for GPU.  @rmlarsen do you see anything wrong with the application of this kernel?

Same problem with Win10 and tf-nightly-gpu installed.

Same problem here. I'm on Win10 and I've compiled this yesterday (19MAR) using latest sources from https://github.com/tensorflow/tensorflow.git. Built using cmake following instructions from
https://github.com/tensorflow/tensorflow/tree/master/tensorflow/contrib/cmake.
The full error is:
2018-03-20 17:34:41.133280: I D:\temp\tensorflow\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:1041] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 6522 MB memory) -> physical GPU (device: 0, name: GeForce GTX 880M, pci bus id: 0000:01:00.0, compute capability: 3.0)
2018-03-20 17:34:53.281846: E D:\temp\tensorflow\tensorflow\tensorflow\core\common_runtime\executor.cc:644] Executor failed to create kernel. Not found: No registered 'Snapshot' OpKernel for GPU devices compatible with node conv1/truncated_normal = SnapshotT=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"

@btud did you try the uninstall/reinstall suggested by @rancheng ?

Is this problem only related to the tensorflow branch:master?
How can I get the lastest tensorflow-gpu packages besides cloning from https://github.com/tensorflow/tensorflow.git and build it with cmake and MCVS 2015 ?
@rancheng Did you get the source code else where?

@poxvoculi , I tried installing the prebuilt package (hope I understood correctly what @rancheng was suggesting) using "pip3 install --upgrade tensorflow-gpu". It seems the default tensorflow-gpu requires the exact 9.0 version of CUDA. I have 9.1 installed - this is what I built against.  So with prebuilt package It fails even for a hello world program - the error is: ImportError: Could not find 'cudart64_90.dll. Not sure if multiple cuda versions can coexist on windows, so I may have to downgrade to 9.0 and then try again.

What if I'm trying to build C++ dynamic library?

@GilesCJ What version of TensorFlow (exact git commit) are you trying to build? @jhseu recently fixed this for the Windows build in d392b1c9ebf131b9ac64ff289d26e43afea21c10, which as far as I can tell is only on the r1.7 branch.
@yifeif @jhseu Does this fix need to be backported to the master branch?

Yeah, we usually merge the r1.7 branch back into master sometime near the final release.

@mrry  I've rebuild master three days ago, and the problem remains. I'll try to build r1.7 today and see how it goes. Is this problem possible related to static library or dynamic library??

As far as I understand, the problem affects all versions of the GPU build using Windows/CMake since the snapshot op was added.

@mrry The problem is solved when I build the library with r1.7 branch!! Thank you guys very much!
Will r1.7 merge to the master?? Or I'll have to use r1.7 from now on?


add data directory configuration component to advanced tab in rke2 create/edit cluster interface

Summary
Fixes #10824

Occurred changes and/or fixed issues

add data directory configuration component to advanced tab in rke2 create/edit cluster interface
add unit test for data directory configuration component

Technical notes summary

full manual test is dependent on https://github.com/rancher/rancher/issues/45038

Areas or cases that should be tested

Go to RKE2 cluster creation interface and make sure the data configuration area is present on the Advanced tab of Cluster Configuration
Make sure data persists and all works fine
Make sure that cluster editing, the data configuration is disabled (not allowed to change the values)

Areas which could experience regressions

Screenshot/Video

NOTE: this video still doesn't cover the edit scenario properly, since we still don't have the changes needed on the backend to persist the data, but it proves the payload sent is correct
https://github.com/rancher/dashboard/assets/97888974/2f09bc61-8c02-4927-8ebf-98257261fc6d
Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

I think even though you are not allowed to change the values you should still be able to see the exact values you entered when you created the cluster. For example, I added separate paths for each of the three inputs when I created the cluster but this is what I see in edit mode:

I think even though you are not allowed to change the values later, you should still be able to see the exact values you entered when you created the cluster. For example, I added separate paths for each of the three inputs when I created the cluster but this is what I see in edit mode:

That's because the backend part is still not done. The data doesn't persist yet. It's normal, for now 😬
But they payload is sent correctly. Check the RFC doc linked to the SURE issue where they show the agreed data struct.

I think even though you are not allowed to change the values later, you should still be able to see the exact values you entered when you created the cluster. For example, I added separate paths for each of the three inputs when I created the cluster but this is what I see in edit mode:

That's because the backend part is still not done. The data doesn't persist yet. It's normal, for now 😬
But they payload is sent correctly. Check the RFC doc linked to the SURE issue where they show the agreed data struct.

now that makes sense! 😅

I removed the optional chaining before (I agree it should not be needed) but this caused failures in at least another unit test... I just re-added them to move this forward :P

I removed the optional chaining before (I agree it should not be needed) but this caused failures in at least another unit test... I just re-added them to move this forward :P

It seems you can fix all those failing tests by adding:
dataDirectories: {
   systemAgent:  '',
   provisioning: '',
   k8sDistro:    '',
}

to: https://github.com/rancher/dashboard/blob/b42c20ed5640d91b20a1f100e79c99a2522c4a50/shell/edit/provisioning.cattle.io.cluster/tests/utils/cluster.ts#L62

I removed the optional chaining before (I agree it should not be needed) but this caused failures in at least another unit test... I just re-added them to move this forward :P

It seems you can fix all those failing tests by adding:
dataDirectories: {
   systemAgent:  '',
   provisioning: '',
   k8sDistro:    '',
}

to:
https://github.com/rancher/dashboard/blob/b42c20ed5640d91b20a1f100e79c99a2522c4a50/shell/edit/provisioning.cattle.io.cluster/__tests__/utils/cluster.ts#L62

@momesgin code updated 🙏

I am getting
"[Vue warn]: Invalid prop: type check failed for prop "value". Expected Object, got Undefined
found in
--->  at shell/edit/provisioning.cattle.io.cluster/tabs/DirectoryConfig.vue
 at shell/edit/provisioning.cattle.io.cluster/tabs/Advanced.vue
 at shell/components/Tabbed/Tab.vue
 at shell/components/Tabbed/index.vue
 at shell/components/CruResource.vue
 at shell/edit/provisioning.cattle.io.cluster/rke2.vue
 at shell/components/CruResource.vue
 at shell/edit/provisioning.cattle.io.cluster/index.vue
 at shell/components/ResourceDetail/index.vue
 at shell/pages/c/_cluster/_product/_resource/create.vue
<Shell/components/templates/default.vue> at shell/components/templates/default.vue

error thrown if I am trying to save rke2 cluster with default configuration.
https://github.com/rancher/dashboard/assets/133266076/d885acc6-7484-44a0-ac3a-3c190e4f73af

Gm @eva-vashkevich . This was dependent on a backend issue https://github.com/rancher/rancher/issues/45038 but all of the work has been merged, afaik. I am going to give this a test with a new backend image. Since I've done the frontend part without the backend being finished, I totally forgot to test it manually and merged this (all based on the RFC doc). 🙏

Looking at the associated PRs, I don't think the backend part is completed to this date.... 🤔 The console message is not pretty, but I don't think there's a high risk with the current work merged. It allows for creating/edit of a cluster + the console message doesn't appear unless you're running the UI locally.
I've just pinged the backend team to get more information about the current status of their work

@aalves08 Thank you for looking into it


Dashboard should have separated assets

I need to implement saleor with the foundation. With this approach I need to remove all current styles and scripts. The problem is that we have one package.json file and I don't knew which packages I can remove without breaking dashboard.
I propose to move all dashboard static to salor/dashboard/static and create separate grunt and webpack files for it.

What about this issue? Is it still valid? Usually projects have single package.json file and I'd personally keep it like it is. What do you think?


New features in patch versions

Question
I've noticed that Typia sometimes includes feature additions in patch upgrades. For example, version 6.0.6 includes the commit feat: Migrate to ESM and use tsup for building, and version 6.0.4 includes Feat: apply RFC3339 standard to typia.tags.Format<"date-time"|"date"|"time">.
Typically, I would expect new features to trigger a minor version bump rather than just a patch version bump, meaning these feature additions would result in a 6.1.0 release.
Could you please explain your strategy for deciding how to bump the version number?

Thank you for your quick reply (and while I'm at it: also thank you for your open source contributions in this project over time).
It seems that we agree in theory about the versioning, even if we might differ on the implementation into practice.
If I may be so bold as to make a recommendation, I propose to trigger a new minor version whenever the release includes a feat:  commit.

You are right, I had to upgrade minor patch after adding UniqueItems, but I missed it.


Support SPFx v1.15.0 for spfx project doctor command

SPFx v1.15.0 is not supported yet for the command spfx project doctor.

I'm willing to work on this one.

Fantastic milan, all yours!


emacsWrapper: Put autoloads behind optional flag

Description of changes
This puts the recently introduced autoloads for emacs behind an optional
flag, which is off by default.
See https://github.com/NixOS/nixpkgs/pull/168954 for more information.
@myguidingstar can you please check if this works for you?
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I've looked at the code and it looks great, but I don't have the computer to really run it soon

I can't say I like the idea of having flags to work around bugs in our wrapper code.. I think we should either disable the functionality outright or fix it.
Not add more flags.

The added autoload code for this wrapper has helped me to reduce my configs a good bit because I can now use functions without having to know what package they are coming from and requireing them. This is also very helpful if you want to call emacs functions from the CLI. But this code can cause problems when there is a badly misbehaving package like what arose here https://github.com/NixOS/nixpkgs/pull/168954#issuecomment-1114806352 , when though I made sure to call the #'load function with the NO-ERROR argument.

I think we should either disable the functionality outright or fix it.

I am fine with both. I think that others can also benefit from this autoloading but, since I cannot reproduce the error, I can not work on any fix. In which way should we proceed?

I have attempted to fix the underlying issue, with the hope that we can get rid of that flag and close this PR. I attempted to build a wrapper with more and more packages from emacs.pkgs and I saw more and more errors, meaning that not all packages are able to being autoloaded. However, when wrapping the generated (load ...) calls with ignore-errors everything worked fine.
My code for this is here:
Commit: https://github.com/nagy/nixpkgs/commit/f854fccb2d2905c8a6bd952739f82e079f52f0c1
Branch: https://github.com/nagy/nixpkgs/tree/emacs-ignore-autoload-errors
@myguidingstar could you verify that this fix would solve the issue for you? If so, then we can close this PR here and I can make another one for that fix.

Closing in favor of https://github.com/NixOS/nixpkgs/pull/181174


Software engineer

Is there a way to download an entire folder from the blob container rather downloading individual images. Please help.

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

ID: 60e56e69-1c44-2056-ad5f-319535e4b525
Version Independent ID: 49fa6098-0b8e-ebf5-efe7-f50f3604e325
Content: Azure Quickstart - Upload, download, and list blobs in Azure Storage using .NET | Microsoft Docs
Content Source: articles/storage/blobs/storage-quickstart-blobs-dotnet.md
Service: storage
GitHub Login: @tamram
Microsoft Alias: tamram

@rizwaans Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks, I have found the solution
var myBlockDirectory = myContainer.GetDirectoryReference("myFolder");
var myBlob = myBlockDirectory.ListBlobs().ToList(); // this gives the list of all the blobs inside the folder


Solution level stylecop.json & stylecop.ruleset

I have a solution with multiple projects in it. I want to define the same stylecop configuration and ruleset for all projects in the solution. In Visual Studio 2015, I used to define this configuration like this in project.json
"buildOptions": {
"additionalArguments": [ "/ruleset:../../../stylecop.ruleset", "/additionalfile:../../../stylecop.json" ]
}
I define stylecop.ruleset and stylecop.json as the solution items (only once) and specify the relative path in each project.json.
How can I do this in VS 2017 csproj.

The equivalent of this is to create a file called "Directory.Build.props" alongside your solution (or in any parent folder above the project) and add the following props pointing to this:
<Project>

  <PropertyGroup>
    <CodeAnalysisRuleSet>stylecop.ruleset</CodeAnalysisRuleSet>
  </PropertyGroup>

  <ItemGroup>
    <AdditionalFiles Include="stylecop.json" />
  </ItemGroup>

</Project>

Note, you'll can also make use of one of the MSBuild reserved properties, such as $(MSBuildThisFileDirectory) in front of the file names to get paths that resolve correctly for all projects regardless of where they are in the tree.

That worked. Thanks.
I defined the props file at the solution level and used  $(MSBuildThisFileDirectory) for the full path.