minor issue in code( base_language.py) comments

While going through base_language.py code , in _get_num_tokens_default_method .. code makes instance of gpt2 tokenizer while comment says "    # tokenize the text using the GPT-3 tokenizer" . this needs to be corrected to gpt-2
# create a GPT-2 tokenizer instance
tokenizer = GPT2TokenizerFast.from_pretrained("gpt2")

# tokenize the text using the GPT-3 tokenizer
tokenized_text = tokenizer.tokenize(text)

I'd approve the PR!

please some one assign to me . I will do this small change .

please review my pull request https://github.com/hwchase17/langchain/pull/4084


pydiscourse.exceptions.DiscourseClientError: The requested URL or resource could not be found.

Error when querying for hot_topics = client.hot_topics()...

I ran into this same error with a few other API calls and it's looking suspiciously like this might be the error we get when the key we are using doesn't have access to the call we are trying to make.  Can anyone else confirm/disprove this?  If this is the case, the bug really is that this error is a red herring.

hi @jsayles ,
Are you still seeing this behaviour with recent versions? There have been various updates to API handling in recent months which could affect your results.
Karl.


Failing to compile when I create a new vue file in pages directory

C:\Users\ayoub\Desktop\Vue Projects\auth-example\pages\new.vue
1:11  error    Replace ⏎$END$⏎ with $END$                           prettier/prettier
1:11  error    The template root requires an element rather than texts  vue/no-multiple-template-root
7:1   error    Replace name:·"new" with ··name:·'new',              prettier/prettier
7:7   warning  Property name "new" is not PascalCase                    vue/component-definition-name-casing
11:15  error    Delete ⏎⏎
everytime it gives me the same errors even If I stopped the development server and started it again.
all the errors highlighted above are not present in the code

I ran ' npm run lint ' and it fixed the issue

I ran ' npm run lint ' and it fixed the issue


Doc: fix how_juicefs_store_files markdown link

fix how_juicefs_store_files.md link can not redirect problem

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.xieke seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[incompatible OpenCypher] with size() for degrees

Describe the bug (must be provided)
Syntax parsing is not expected.
How To Reproduce(must be provided)
MATCH p=(n:player)-[]->(m:player) WITH p, size((m)-[]->()) as right_out, size((n)<-[]-()) as left_in \
WHERE right_out == 0 AND left_in == 0 RETURN p;

(root@nebula) [basketballplayer]> MATCH p=(n:player)-[]->(m:player) WITH p, size((m)-[]->()) as right_out, size((n)<-[]-()) as left_in \
                               -> WHERE right_out == 0 AND left_in == 0 RETURN p;
[ERROR (-7)]: SyntaxError: syntax error near `]->()) a'

Thu, 22 Jul 2021 12:16:33 CST

Expected behavior
MATCH p=(n:Person)-[]->(m:Movie) WITH p, size((m)-[]->()) as left_out, size((n)<-[]-()) as left_in
WHERE right_out = 0 AND right_in = 0 RETURN p;

╒══════════════════════════════════════════════════════════════════════╕
│"p"                                                                   │
╞══════════════════════════════════════════════════════════════════════╡
│[{"louvain":5,"degree":0.0,"name":"Emil Eifrem","born":1978},{"roles":│
│["Emil"]},{"louvain":5,"degree":8.0,"tagline":"Welcome to the Real Wor│
│ld","votes":372,"title":"The Matrix","released":1999}]                │
├──────────────────────────────────────────────────────────────────────┤
│[{"louvain":5,"degree":0.0,"name":"Joel Silver","born":1952},{},{"louv│
│ain":5,"degree":8.0,"tagline":"Welcome to the Real World","votes":372,│
│"title":"The Matrix","released":1999}]                                │
├──────────────────────────────────────────────────────────────────────┤

...

Additional context
related topic in forum: https://discuss.nebula-graph.com.cn/t/topic/4947

We have not support use pattern in WITH yet


Fix typo

SSIA

Hi @Krout0n ，
I am not going to accept PR since the diff of the notebook is too huge.
Please create an issue to clarify the typo you found. We would fix that for you.
Thanks.


At some point in time, AAP export blew up.

Error: CsvView.Csv could not be found.
Fix it.

Didn't have friendsofcake/csv-view installed on UAT. Fixed.


meltwater.txt

Added Meltwater Entrepreneurial School of Technology (MEST) under lib/domains/org/meltwater.txt

@peter-ohara Pull request merged. Thank you!


[sample]  change OneToMany's data expression from String plus operato…

fixed #276

Seems like Travis in trouble with something , I'll figure out it later.

6b81a6 has removed ContentViewActivity.kt and it's menu item but SmokeTest.kt still use ContentViewActivity to connectedAndroidTest so Travis could not work as wished. I will send a PR to resolve it.


check traject 2.x with jruby 9k prerelease?

Just to give them feedback, if needed. optional. http://www.jruby.org/2015/01/20/jruby-9-0-0-0-pre1

Tested on my machine and added jruby-head to .travis.yml and everything is green.


"npm install --save-dev electron-rebuild "  Unsuccess

system：W7 32
npm:2.15.0
node:V4.4.2
electron:v1.1.0
Masters,I am new to electron and nodejs,please help me.
When I run "npm install --save-dev electron-rebuild", install unsuccess
Partial log:
14474 error Windows_NT 6.1.7601
14475 error argv "D:\WorkProgram\nodejs\node.exe" "D:\WorkProgram\nodejs\node_modules\npm\bin\npm-cli.js" "install" "--save-dev" "electron-rebuild"
14476 error node v4.4.2
14477 error npm  v2.15.0
14478 error code ELIFECYCLE
14479 error<EMAIL_ADDRESS>install: node-gyp rebuild
14479 error Exit status 1
14480 error Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
14480 error This is most likely a problem with the nslog package,
14480 error not with npm itself.
14480 error Tell the author that this fails on your system:
14480 error     node-gyp rebuild
14480 error You can get information on how to open an issue for this project with:
14480 error     npm bugs nslog
14480 error Or if that isn't available, you can get their info via:
14480 error
14480 error     npm owner ls nslog
14480 error There is likely additional logging output above.
14481 verbose exit [ 1, true ]
14482 verbose unbuild node_modules\electron-rebuild\node_modules\nslog
14483 info preuninstall<EMAIL_ADDRESS>14484 info uninstall<EMAIL_ADDRESS>14485 verbose unbuild rmStuff<EMAIL_ADDRESS>from C:\Users\Administrator\node_modules
14486 verbose unbuild rmStuff in C:\Users\Administrator\node_modules\electron-rebuild\node_modules
14487 info postuninstall<EMAIL_ADDRESS>14488 silly gentlyRm C:\Users\Administrator\node_modules\electron-rebuild\node_modules\nslog is being purged from base C:\Users\Administrator
14489 verbose gentlyRm don't care about contents; nuking C:\Users\Administrator\node_modules\electron-rebuild\node_modules\nslog
14490 silly vacuum-fs purging C:\Users\Administrator\node_modules\electron-rebuild\node_modules\nslog
14491 silly vacuum-fs quitting because other entries in C:\Users\Administrator\node_modules\electron-rebuild\node_modules
14492 verbose unbuild node_modules\electron-rebuild
14493 info preuninstall<EMAIL_ADDRESS>14494 info uninstall<EMAIL_ADDRESS>14495 verbose unbuild rmStuff<EMAIL_ADDRESS>from C:\Users\Administrator\node_modules
14496 silly gentlyRm C:\Users\Administrator\node_modules.bin\electron-rebuild.cmd is being gently removed
14497 silly gentlyRm verifying C:\Users\Administrator is an npm working directory
14498 silly gentlyRm containing path C:\Users\Administrator is under npm's control, in C:\Users\Administrator
14499 silly gentlyRm deletion target C:\Users\Administrator\node_modules.bin\electron-rebuild.cmd is under C:\Users\Administrator
14500 verbose gentlyRm vacuuming from C:\Users\Administrator\node_modules.bin\electron-rebuild.cmd up to C:\Users\Administrator
14501 silly gentlyRm C:\Users\Administrator\node_modules.bin\electron-rebuild is being gently removed
14502 silly gentlyRm verifying C:\Users\Administrator is an npm working directory
14503 silly gentlyRm containing path C:\Users\Administrator is under npm's control, in C:\Users\Administrator
14504 silly gentlyRm deletion target C:\Users\Administrator\node_modules.bin\electron-rebuild is under C:\Users\Administrator
14505 verbose gentlyRm vacuuming from C:\Users\Administrator\node_modules.bin\electron-rebuild up to C:\Users\Administrator
14506 info postuninstall<EMAIL_ADDRESS>14507 silly gentlyRm C:\Users\Administrator\node_modules\electron-rebuild is being purged from base C:\Users\Administrator
14508 verbose gentlyRm don't care about contents; nuking C:\Users\Administrator\node_modules\electron-rebuild
14509 silly vacuum-fs purging C:\Users\Administrator\node_modules\electron-rebuild
14510 silly vacuum-fs quitting because other entries in C:\Users\Administrator\node_modules

Do you have node-gyp installed globally? I had to manually install node-gyp with npm install -g node-gyp using Run as Administrator on the Node.js CMD.

@ahallicks  Thanks for your help,I run "npm install -g node-gyp" then run "npm install --save-dev electron-rebuild" ,
I still don't have a successful installation...

I re-install the node and npm,it still unsuccess.
error log:
Microsoft Windows [版本 6.1.7601]
版权所有 (c) 2009 Microsoft Corporation。保留所有权利。
C:\Windows\System32>npm install --save-dev electron-rebuild
<EMAIL_ADDRESS>install C:\Windows\System32\node_modules\nslog
node-gyp rebuild
C:\Windows\System32\node_modules\nslog>if not defined npm_config_node_gyp (node
"D:\WorkProgram\nodejs\node_modules\npm\bin\node-gyp-bin....\node_modules\node-gyp\bin\node-gyp.js" rebuild ) else (node "" rebuild )
gyp ERR! configure error
gyp ERR! stack Error: Can't find Python executable "python", you can set the PYT
HON env variable.
gyp ERR! stack at failNoPython (D:\WorkProgram\nodejs\node_modules\npm\node_
modules\node-gyp\lib\configure.js:401:14)
gyp ERR! stack at D:\WorkProgram\nodejs\node_modules\npm\node_modules\node-g
yp\lib\configure.js:356:11
gyp ERR! stack at FSReqWrap.oncomplete (fs.js:117:15)
gyp ERR! System Windows_NT 6.1.7601
gyp ERR! command "D:\WorkProgram\nodejs\node.exe" "D:\WorkProgram\nodejs\n
ode_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Windows\System32\node_modules\nslog
gyp ERR! node -v v6.2.0
gyp ERR! node-gyp -v v3.3.1
gyp ERR! not ok
npm WARN enoent ENOENT: no such file or directory, open 'C:\Windows\System32\pac
kage.json'
npm WARN System32 No description
npm WARN System32 No repository field.
npm WARN System32 No README data
npm WARN System32 No license field.
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "D:\WorkProgram\nodejs\node.exe" "D:\WorkProgram\nodejs\node
_modules\npm\bin\npm-cli.js" "install" "--save-dev" "electron-rebuild"
npm ERR! node v6.2.0
npm ERR! npm v3.8.9
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the nslog package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR! node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR! npm bugs nslog
npm ERR! Or if that isn't available, you can get their info via:
npm ERR! npm owner ls nslog
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR! C:\Windows\System32\npm-debug.log

@kcsm-lqk You haven't got a properly configured Python installation;

gyp ERR! stack Error: Can't find Python executable "python", you can set the PYTHON env variable.

Install Python 2.7, and your problem should go away. You should also close the duplicate issue you have opened on the main Electron repo (https://github.com/electron/electron/issues/5764).

@thomasjo Thanks fo your help and  I'm sorry for disturbing you again . I am new to nodejs and electron.
(1) I installednode-gyp with "npm install -g node-gyp"
(2)I installedpython 2.7
C:\Users\Administrator>python
Python 2.7.11 (v2.7.11:6d1b6a68f775, Dec  5 2015, 20:32:19) [MSC v.1500 32 bit (
Intel)] on win32
Type "help", "copyright", "credits" or "license" for more information.
Through this process I still  installation failed;
Error log:
gyp ERR! build error
gyp ERR! stack Error: C:\Windows\Microsoft.NET\Framework\v4.0.30319\msbuild.exe   failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (D:\WorkProgram\nodejs\node_modules\np
m\node_modules\node-gyp\lib\build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_proces
s.js:204:12)
gyp ERR! System Windows_NT 6.1.7601
gyp ERR! command "D:\WorkProgram\nodejs\node.exe" "D:\WorkProgram\nodejs\n
ode_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\Administrator\node_modules\mosca\node_modules\zmq
gyp ERR! node -v v6.2.0
gyp ERR! node-gyp -v v3.3.1
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS><EMAIL_ADDRESS>install: node-gyp rebuild
npm WARN<EMAIL_ADDRESS>Exit status 1

<EMAIL_ADDRESS>install C:\Users\Administrator\node_modules\nslog
node-gyp rebuild

C:\Users\Administrator\node_modules\nslog>if not defined npm_config_node_gyp (no
de "D:\WorkProgram\nodejs\node_modules\npm\bin\node-gyp-bin\....\node_modules
node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
Building the projects in this solution one at a time. To enable parallel build,
please add the "/m" switch.
main.cc
nslog_win.cc
C:\Users\Administrator\node_modules\nan\nan.h(41): fatal error C1189: #error :
This version of node/NAN/v8 requires a C++11 compiler [C:\Users\Administrator
node_modules\nslog\build\nslog.vcxproj]
gyp ERR! build error
gyp ERR! stack Error: C:\Windows\Microsoft.NET\Framework\v4.0.30319\msbuild.exe  failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (D:\WorkProgram\nodejs\node_modules\np
m\node_modules\node-gyp\lib\build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_proces
s.js:204:12)
gyp ERR!  Windows_NT 6.1.7601
npm ERR! argv "D:\WorkProgram\nodejs\node.exe" "D:\WorkProgram\nodejs\node
_modules\npm\bin\npm-cli.js" "install" "--save-dev" "electron-rebuild"
npm ERR! node v6.2.0
npm ERR! npm  v3.8.9
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the nslog package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs nslog
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls nslog
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\Administrator\npm-debug.log
npm ERR! code 1

Seems like you don't have a C++ compiler that supports C++11

C:\Users\Administrator\node_modules\nan\nan.h(41): fatal error C1189: #error :This version of node/NAN/v8 requires a C++11 compiler

Install VS2015, then try again.
https://www.visualstudio.com/en-us/products/visual-studio-community-vs.aspx

You don't need to install the full VS2015. You should be fine with just the build tools https://www.microsoft.com/en-us/download/details.aspx?id=48159

@thomasjo , @ahallicks   ,  I'm sorry for disturbing you again . Do I have to install node-gyp before installing electron-rebuild?
If I have to install node-gyp.must I need to install:
https://github.com/nodejs/node-gyp
On Windows:
Visual C++ Build Environment:
Option 1: Install Visual C++ Build Tools using the Default Install option.
Option 2: Install Visual Studio 2015 (or modify an existing installation) and select Common Tools
for Visual C++ during setup. This also works with the free Community and Express for Desktop         editions.
:bulb: [Windows Vista / 7 only] requires .NET Framework 4.5.1
Install Python 2.7 (v3.x.x is not supported), and run npm config set python python2.7 (or see below        for further instructions on specifying the proper Python version and path.)
Launch cmd,  npm config set msvs_version 2015
If the above steps didn't work for you, please visit Microsoft's Node.js Guidelines for Windows for       additional tips.
Until now ,I have  installed python 2.7, installed node-gyp with "npm install -g node-gyp" , installed
BuildTools_Full_2015 .When I run "npm install --save-dev electron-rebuild " ,it still error
Microsoft Windows [版本 6.1.7601]
版权所有 (c) 2009 Microsoft Corporation。保留所有权利。
C:\Users\Administrator>npm install --save-dev electron-rebuild

<EMAIL_ADDRESS>install C:\Users\Administrator\node_modules\nslog
node-gyp rebuild
C:\Users\Administrator\node_modules\nslog>if not defined npm_config_node_gyp (no
de "D:\WorkProgram\nodejs\node_modules\npm\bin\node-gyp-bin\....\node_modules
node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
在此解决方案中一次生成一个项目。若要启用并行生成，请添加“/m”开关。
C:\Users\Administrator\node_modules\nslog\build\nslog.vcxproj(19,3): error MSB4
019: 未找到导入的项目“C:\Microsoft.Cpp.Default.props”。请确认  声明中
的路径正确，且磁盘上存在该文件。
  Windows_NT 6.1.7601
npm ERR! argv "D:\WorkProgram\nodejs\node.exe" "D:\WorkProgram\nodejs\node
_modules\npm\bin\npm-cli.js" "install" "--save-dev" "electron-rebuild"
npm ERR! node v6.2.0
npm ERR! npm  v3.8.9
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the nslog package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs nslog
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls nslog
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\Administrator\npm-debug.log

I believe I am having a similar issue installing on Windows 10
node: 5.1.1
npm: 3.3.12
python: 2.7.11
msbuild: 14.0.25123.0
C:\test>npm install --save-dev electron-rebuild

><EMAIL_ADDRESS>install C:\test\node_modules\nslog
> node-gyp rebuild

C:\test\node_modules\nslog>if not defined npm_config_node_gyp (node "C:\nvm\v5.1.1\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node  rebuild )
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Platforms\x64\PlatformToolsets\v140\Toolset.targets(36,5): error MSB8036: The Windows SDK version 8.1 was not found. I
nstall the required version of Windows SDK or change the SDK version in the project property pages or by right-clicking the solution and selecting "Retarget solution".   Windows_NT 10.0.10586
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "install" "--save-dev" "electron-rebuild"
npm ERR! node v5.1.1
npm ERR! npm  v3.3.12
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the nslog package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls nslog
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\test\npm-debug.log

C:\test>

Thanks all, I have solved my problem. Following these steps  [https://github.com/nodejs/node-gyp] , and
I can install  electron-rebuild successfully!!

@Xiao-Yaran
Hi, Xiao:
I have got the same problem as yours but finally you got it resolved.
The error message is shown as this:
PS D:\Coder'sCode\node.js\electron\data_analyser> npm install --save-dev electron-rebuild
npm WARN prefer global<EMAIL_ADDRESS>should be installed with -g

><EMAIL_ADDRESS>install D:\Coder'sCode\node.js\electron\data_analyser\node_modules\nslog
> node-gyp rebuild

D:\Coder'sCode\node.js\electron\data_analyser\node_modules\nslog>if not defined npm_config_node_gyp (node "C:\Program Fi
les\nodejs\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebu
ild )
在此解决方案中一次生成一个项目。若要启用并行生成，请添加“/m”开关。
D:\Coder'sCode\node.js\electron\data_analyser\node_modules\nslog\build\nslog.vcxproj(20,3): error MSB4019: 未找到导入的
项目“D:\
Microsoft.Cpp.Default.props”。请确认 <Import> 声明中的路径正确，且磁盘上存在该文件。
  Windows_NT 10.0.14905
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "ins
tall" "--save-dev" "electron-rebuild"
npm ERR! node v6.4.0
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the nslog package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs nslog
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls nslog
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     D:\Coder'sCode\node.js\electron\data_analyser\npm-debug.log

I have build tool and node-gyp installed already. Could you please show me that how you finally figure out this problem? Thanks.

@iridescent-rsy installing electron-rebuild globally solved the problem for me
npm install -g electron-rebuild

@gjavi1 Thanks for reply!
I tried to add -g but it still got an error.... It shows a red error message like this:
019: 未找到导入的项目“C:\Microsoft.Cpp.Default.props”。请确认 <Import> 声明中的路径正确，且磁盘上存在该文件。````
It seems that something is missing but I don't know how to solve this. Do you have any idea? :eyes:

@iridescent-rsy is Visual C++ Build Tools installed on your machine?

@gjavi1 I have installed it. Maybe the msbuild was not installed correctly. I tried another command to install the develop-environment : npm install -g windows-build-tools. And it can solve the problem. Thanks a lot! 😄

Also, try setting a specific version for msvs of you do have the build tools installed. I have had to use --msvs_version=2013 and --msvs_version=2015 to get things working in the past.

@ahallicks Thanks. I have solved this problem. I believe the version of msbuild is the key.

So I've kind of been following this thread for the last few hours and reading through it looks like everyone has solved there problems in various ways.
I'll close this out now as everything looks good from here 👍

I'm sorry to reopen this, but I am having the same or a very similar problem and the help already in this thread has not fixed it. I have re-installed node and npm. I have all the Visual studio build tools installed. I have run this in Administrator: CMD and in Windows 7.1 SDK command prompt, to no avail. I have installed node-gyp globally, as well as node and npm. I don't know what else I can do.
Log:
gyp ERR! build error
gyp ERR! stack Error: 'C:\Program Files (x86)\MSBuild\14.0\bin\msbuild.exe' failed with exit code:<PHONE_NUMBER>
gyp ERR! stack at ChildProcess.onExit (C:\Users\path\to\npm\node_modules\node-gyp\lib\build.js:276:23)
gyp ERR! stack at emitTwo (events.js:106:13:)
gyp ERR! stack at ChildProcess.emit (events.js:191:7)
gyp ERR! System Windows_NT 6.1.7601
gyp ERR! Command "C:\Program Files\nodejs\node.exe" "C:\Path\To\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install" "--save-dev" "electron-rebuild""
gyp ERR! cwd C:\Path\to\Project\node_modules\nslog
gyp ERR! node -v v6.6.0
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN<EMAIL_ADDRESS>No license field.
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Path\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install" "--save-dev" "electron-rebuild"
npm ERR! node v6.6.0
npm ERR! npm v3.10.8
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: 'node-gyp rebuild'
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the nslog package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs nslog
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls nslog
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Path\To\Project\npm-debug.log

+1

Same issue here

+1

Hey all, I managed to solve this problem after a few nights of trying everything in this thread and elsewhere.
Currently I'm on a windows 10 PC and was getting all of these posted error logs.
My belief is that there was a install script bug related to paths, but I can't be sure.
The fix:
Clone or download the nslog repo,
copy it into your node_modules renamed to just nslog,
run node-gyp configure, which looks at the binding.gyp file.
run node-gyp rebuild
Now you should be able to do the npm install electron-rebuild as it now has that nslog dependency solved.

@iridescent-rsy  请问是不是 npm install --global --production windows-build-tools就能解决了


Remove queued query

Would it be possible to add a connection method that would remove a queued (waiting) query from the connection queue? It would take a single parameter the query object as returned from connection.query() and it would remove the query from the queue if the query is waiting. It should respond with one of three states: query removed, query currently executing (can't be removed), query not found (probably already executed).
A rough pseudocode what it would do (specific implementation details will need to be refined):
Connection.prototype.remove = function remove(query) {
  var idx = this._protocol._queue.indexOf(queue)
  if (idx < 0) {
    return false; // Query not found
  }
  if (idx === 0) {
    return false; // Query currently executing
  }
  this._protocol._queue.splice(idx, 1)
  return true;
}

We need this feature for implementing timeouts in Knex.js, see this discussion https://github.com/knex/knex/pull/4416#discussion_r608067856. We don't want to hack the driver internals and we would prefer to use a supported feature. I'm willing to create a PR with the implementation if the feature would be accepted.
I'm filing a symmetric feature request to mysql2 driver https://github.com/sidorares/node-mysql2/issues/1316. It would be good to implement it in a compatible way in both drivers.

Hi @martinmacko47 thanks for this feature request. I don't see any reason why a feature to remove a queued query cannot be implemented. One tweak I may suggest to our original ideal is the following: what about adding a method like .cancel() or .abort() onto the Query object itself? This would mean you do not have to know what the original connection was the query belongs to (especially helpful with pool.query, in which you never get the original connection object) and this module). A property could also be added to the Query object regarding it's state as well, which may be more useful than just the return value (i.e. you can see if it is executing before attempting a cancel). Thoughts?

@dougwilson Thank you for your proposal, your solution is much better.
About naming of cancel method - name should show, what method only does job only when query in queue, so users don't have expectations to cancel any query by this method. Maybe .dequeue() or smth?

@dougwilson I agree with the tweak. I created a draft PR #2480 with the implementation. I added Query.started() and Query.ended() methods to determine if the query is pending, already started or already ended. And method Query.dequeue() to remove the query from the connection queue unless it already started.

@maximelkin

Also have you any thoughts about pausing queue execution while we cancel current executing query? Potentially cancel is long and we do not want kill next query from execution queue.

Won't methods Connection.pause() and Connection.resume() help with this? Or they are for something else?

@martinmacko47 :thinking:  they actually for streaming, as for docs.
In default workflow everything is good, but not sure about Connection lost cases (slow connection creation & cancelling? exhausting mysql connection limit on server side?).

@dougwilson Hi, can you pls check my PR https://github.com/mysqljs/mysql/pull/2480?

@dougwilson Hello. Is there anything I can do to have the PR checked?


Implements #130 Case Sensitive Path

implements #130

@sclarson are you interested in revisiting this to resolve conflicts and address the feedback? If not, can you let us edit this PR?


Data collection link broken

https://github.com/google-research/open-covid-19-data/tree/master/data/exports/search_trends_symptoms_dataset/United States of America/subregions/Minnesota
Link in Readme is broken to data collection policy

This is fixed now (by commit 02e11f43ac6869fd6e4bca96772f407cb1ff0ca3).


Fix/13062 title label misaligned, spacing issues

closes: https://issues.redhat.com/browse/RHOAIENG-13062
Description
fix the misaligned title - archived label. removed some unnecessary wrappers that were stealing the flex display and set align children to center. all 3 places will look like this now:

in the mr version details, i removed isFillColumns which was adding some extra space, added margin below the titles (Model Location and Source model format). It looks like this now:

Hoping this is good, but I was a little confused with the 8px spacing mentioned in the figma (@yih-wang ) - it seems like the figma is saying these should all be 8px:

but PF has row gap between description groups at 24px, and the gap between a descriptiongroup's term and description at 8px.
this is the PF DescriptionList default spacing, this component is using the DashboardDescriptionListGroup components (a group is a label + value) and we aren't adding any extra styling there.

The space between (the darker lime color in the image above) is PF, from the docs:

And the same spacing appears between groups on the PF examples:
list:

group:

The name and version are already grouped together:

How Has This Been Tested?
Title and "archived" label should be vertically center aligned for both archived model and version details page. The mr versions details should be spaced properly
Test Impact
none - styling changes
Request review criteria:
Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[x] Included any necessary screenshots or gifs if it was a UI change.
[x] Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

The code LGTM here, but I'll wait for a review on the spacing from @yih-wang or @vconzola before approving this

@gitdallas To answer the space question, currently there's no clear visual distinction of where's the end of a group (model location, source model format). For example, how to tell which attributes belong to the Source Model Format. Does version belong to it? Does Author belong to it? I'm trying to use different spacings (i.e., smaller spaces within the group items) to clarify that relationship.
Another choice is using divider lines, but I'm not seeing any other places are using it, or any examples from the PF description list pattern.
Would love to get your thoughts on this.

Talking with @gitdallas, I'll bring the spacing/grouping issues to PF and see what they suggest. What we have now looks good to me. Thanks!

With the Dividers added

Thanks for the updates @gitdallas, looks good!


Small changes in section 2 of MXD tutorial to make it more intuitive

Description
This commit enhances documentation of section 2 of MXD Readme file as follows:

Explicitely add git command to clone this repo locally before issuing any other command that require its files
Change the place where the "cd" command is issued

<!--

## Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

- [x] DEPENDENCIES are up-to-date. [Dash license tool](https://github.com/eclipse/dash-licenses). Committers can open IP issues for restricted libs.
- [x] [Copyright and license header](https://eclipse-tractusx.github.io/docs/release/trg-7/trg-7-02) are present on all affected files

@moiarayam you need to sign the ECA for us to be able to accept your contribution.

Closing this pull request to create a new one with the proposed changes and the Email with ECA signature in the commits


What is coordinate of the corner?

System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04): 18.04
Apollo installed from (source or binary):
Apollo version (3.5, 5.0, 5.5, 6.0): 6.0
Output of apollo.sh config if on master branch:

I am making an apollo map.
Apollo_HDMap_OpenDrive_Specs_A3.0.1 is referred to.
I'm trying to link a traffic light to the map, but I don't know what the description of the traffic light means.

@phs4530 A traffic light is usually a rectangle with 4 corners corresponding to 4 coordinates. There're already some map in modules/map/data you can ref to.


Time series classification

Hello, could you please update the code on time series classification, as well as the example Thank you !

Hi @lys15163238308 ,
I've updated the README with a classification example. The main idea is to squash the entire hidden state output sequence to a fixed size vector with the sequence statistics (mean, max, min) for each output feature dimension. This is achieved by global mean, max and min pooling. Then a final fully-connected layer is used to classify the sequence.
ki = tf.keras.Input(shape=(None, 1))
scan = models.ScanRNNAttentionModel([10, 10], [10, 10])
avg_pool = tf.keras.layers.GlobalAveragePooling1D()
max_pool = tf.keras.layers.GlobalMaxPooling1D()
min_pool = tf.keras.layers.Lambda(lambda x: tf.reduce_min(x, -2))
conc = tf.keras.layers.Concatenate()
dense = tf.keras.layers.Dense(1, "sigmoid")

h = scan(ki)
avgp = avg_pool(h)
maxp = max_pool(h)
minp = min_pool(h)
mix = conc([avgp, maxp, minp])
o = dense(mix)

classification_model = tf.keras.Model(ki, o)
classification_model.compile("adam", "bce")


Different results for RAD51D depending on whether we select hg19 or hg38, even though there are no differences in the numbering

Please paste the url of the results page and the variant search string as text here (unless this isn't about a specific variant).
Hi there,
We're looking at RAD51D:c.715C>G (NM_002878.4). When we put "RAD51D:c.715C>G" in the search bar and select hg 19 we get a donor gain of 0.87 at 3bp; when we then keep the same in the search bar but change to hg38 we loose that donor gain even though there is no difference in RAD51D between hg 19 and hg 38. Can you explain the difference? Which one is correct?
link to my search in hg 19:
https://spliceailookup.broadinstitute.org/#variant=RAD51D%3Ac.715C>G&hg=37&distance=500&mask=1

link to my search in hg 38:
https://spliceailookup.broadinstitute.org/#variant=RAD51D%3Ac.715C>G&hg=38&distance=500&mask=1

Thanks,
Andrea
If you'd like to also share a screenshot:
MacOS:   Press Command+Shift+4, select a region of the page, then paste the image here.
Windows:  Press Windows Logo Key + PrtScn   or   Fn + Windows logo key + Space Bar. Then, in File Explorer, open the Pictures > Screenshots folder and drag the image here.

Hi Andrea,
SpliceAI-lookup use an Ensembl API to convert HGVS strings like "RAD51D:c.715C>G" into genomic coordinates before running SpliceAI on them.
In your example, the API urls are
https://grch37.rest.ensembl.org/vep/human/hgvs/RAD51D:c.715C>G?content-type=application/json&vcf_string=1
which returns a "vcf_string" of 17-33430485-G-C
and
https://rest.ensembl.org/vep/human/hgvs/RAD51D:c.715C>G?content-type=application/json&vcf_string=1
which returns a "vcf_string" of 17-35103277-G-C
However, these 2 positions aren't the same.
 Putting the lifted-over variant into SpliceAI-lookup for hg38 produces the same scores as the hg19 variant.
Since the RAD51D gene definitions are the same for GRCh37 and GRCh38, my only guess is that the Ensembl API isn't using the same isoform when doing the conversion.
Either way, this tells me that HGVS notation isn't as reliable/predictable as just specifying the genomic coordinates.


Fix 404 link to GitHub Annotation limitations

Hi, @oliviertassinari! We merged your PR to reviewdog! 🐶
Thank you for your contribution! ✨
We just invited you to join the @reviewdog organization on GitHub.
Accept the invite by visiting https://github.com/orgs/reviewdog/invitation.
By joining the team, you'll be a part of reviewdog community and can help the maintenance of reviewdog.
Thanks again!

Hi, @oliviertassinari! We merged your PR to reviewdog! 🐶
Thank you for your contribution! ✨
We just invited you to join the @reviewdog organization on GitHub.
Accept the invite by visiting https://github.com/orgs/reviewdog/invitation.
By joining the team, you'll be a part of reviewdog community and can help the maintenance of reviewdog.
Thanks again!

Hi, @oliviertassinari! We merged your PR to reviewdog! 🐶
Thank you for your contribution! ✨
We just invited you to join the @reviewdog organization on GitHub.
Accept the invite by visiting https://github.com/orgs/reviewdog/invitation.
By joining the team, you'll be a part of reviewdog community and can help the maintenance of reviewdog.
Thanks again!


[Discuss] Aliasing

The current implementation of Relax allows tensors to be aliased.
from __future__ import annotations
import os

import numpy as np
import tvm
import tvm.script
from tvm import relax
from tvm.script import relax as R

@tvm.register_func("test.increment")
def increment(inp):
    inp[:] = np.array([inp.numpy()[0] + np.array(1, dtype="int32")], dtype="int32")

@tvm.script.ir_module
class Alias:
    @R.function
    def main(x: Tensor((1,), "int32")):
        y = x
        p1 = R.call_packed("test.increment", y, type_args=(Void))
        p2 = relax.print(x, y, format="x: {}, y: {}")
        return p1

mod = Alias
target = tvm.target.Target("llvm", host="llvm")
ex = relax.vm.build(mod, target)
vm = relax.VirtualMachine(ex, tvm.cpu())

ret = vm["main"](tvm.nd.array([1]))

When executed, this prints x: [2], y: [2], suggesting that they point to the same tensor. This is not altogether unreasonable. However, if we permit tensor values to alias, then it might make sense to provide a way to copy them in cases when users want to keep the original value as well as a mutated one. As it stands, there is a builtin VM operator that does copying but it's apparently only used for the implementation of if-else expressions (note: we should test if that breaks aliasing, actually) -- perhaps we should expose it.
Any thoughts on aliasing? It's perhaps reasonable for users to expect values to alias (see the August 9 community discussion), so we may want to make sure that it's possible to make copies and check if values are aliased.

For the record, if-else expressions do not break aliasing. This experiment has the same result as the one above:
@tvm.script.ir_module
class TestIfAlias:
    @R.function
    def branch(cond: Tensor((), "bool"), x: Tensor((1,), "int32")) -> Tensor((1,), "int32"):
        if cond:
            ret = x
	else:
            ret = x
        return ret

    @R.function
    def main(cond: Tensor((), "bool"), x: Tensor((1,), "int32")) -> Object:
        y = branch(cond, x)
	p1 = R.call_packed("test.increment", y, type_args=(Void))
	p2 = relax.print(x, y, format="x: {}, y: {}")
        return p2

mod =  TestIfAlias
target = tvm.target.Target("llvm", host="llvm")
ex = relax.vm.build(mod, target)
vm = relax.VirtualMachine(ex, tvm.cpu())

ret = vm["main"](1, tvm.nd.array([1])) # prints "x: [2], y: [2]"

@slyubomirsky Thanks for bringing this up! This is an interesting problem.
I wonder what kind of sticky situations we would face if we keep the current behavior.
For example, what do we expect if we alias tensors within data flow block outside?
 @R.function
 def main(x: Tensor((1,), "int32")):
       with relax.dataflow():
             y = relax.add(x,x)
             z = relax.add(y,x)
       alias = y
       # play around with alias

If y is a DataflowVar, then it would not be visible outside the dataflow block. I wonder if there is some way to mess with the safety of dataflow blocks this way, yes.

Also curious if alias would complicate liveness analysis for memory footprint.

It adds some complexity, yes. You would have to think about the actual memory regions. A region of memory is live if there is any live reference to it, so you would have to track which references in fact point to the same region.

Right. Might be good to list out the trade-offs for having this alias support.
For now, imho, aliasing seems more like a UX feature rather than must-have feature (please correct me if I'm wrong).
Don't have a clear opinion yet, but have a doubt if aliasing is worth supporting despite the extra complication that it would bring. It might be okay starting from conservative, restricted approach (no alias) and gradually introduce new features like alias as we observe the real community needs.

Aliasing exists in the current implementation, the question is whether we want the semantics of the language to permit it or not. I would argue that it is actually the less complex option because preventing aliasing would require a lot of copying of NDArrays and asking packed funcs very, very nicely not to break the abstraction.

Right, it already exists haha
I guess I might subconsciously want to avoid alias if I can due to the PTSD from my limited experience with may-alias relationship - it made my optimization behave more conservative than it was supposed to!

preventing aliasing would require a lot of copying of NDArrays and asking packed funcs very, very nicely not to break the abstraction.

Would you give me an example?

Here are some choices we might make: If we don't want to permit aliasing, then we would have an assignment like y = x in the above example perform a copy. Then y and x are separate tensors. However, a PackedFunc can wreak havoc on this abstraction. Suppose we have a PackedFunc that simply takes in an NDArray x and returns that same NDArray. The result of the PackedFunc aliases its argument. If we want to prevent aliasing in the language, we would have to check whether the result is an alias for something else and make a copy. However, an even more evil example might be a PackedFunc that returns a pointer to a global NDArray it keeps. Every time we call this PackedFunc, the result aliases the results of past calls. We would have no way to automatically detect this, so we may just ask users very nicely to not do things like that in PackedFuncs or the program might break or to put in an explicit copy if they do something like that.
Those are not the only choices we could make, just one example. Another choice would be having some kind of COW layer on top of NDArrays in the language, but that would introduce overhead at runtime. I am sure there might be more options, too. What makes things very complicated is exposing PackedFuncs in the language, since they can do anything they want and users are certain to take advantage of that flexibility. Given that, I would actually advocate for permitting aliasing and being up-front about it, while also providing ways to make copies when needed and check if values are aliased.

@slyubomirsky Really appreciate the detailed explanation :)
Alias is much more common than my previous naive understanding - wasn't relating alias to function calls.
I'm convinced!
In summary, I agree with the following direction:

Support alias and provide a way to make copies
Address potential corner cases
(Optional) Take user annotation for better analysis

Oh boy, here's a tricky case I've thought of that we might want to think about: Suppose we have a function that captures a tensor from an outer scope. If tensor values can be aliased, then that might provide a way to mess with dataflow blocks
Example:
@R.function
def outer(x: Tensor((1,), "int32"):
    @R.function
    def inner():
        with R.dataflow():
            dx = x
            R.output(dx)
        return dx
    w = relax.print(inner())
    y = R.call_packed("increment", x, type_args=(Void))
    z = relax.print(inner())
    return z

With the argument [1], this prints:
[1]
[2]

Even though inner() is a pure function, its output can be affected by mutating the x that is captured. :zany_face:

Note: In-place mutation can also be done to scalars! This may actually be a reason to distinguish between pure ints/bools/etc and 0-rank tensors, but I would really rather we not distinguish them.
@tvm.script.ir_module
class IncrementScalar:
    @R.function
    def main(x: Tensor((), "int32")):
        y = R.call_packed("test.inc_scalar", x, type_args=(Void))
        return x # x will be incremented in-place

aliasing indeed create complication(need for alias analysis or do nothing). On the otherhand, they are somewhat needed mainly because of the abstraction and interfacing with python's data structure  (which themselves alias).
From practical pov, most of the optimizations focused on the dataflow block  and we do not need to worry about aliasing there. The capture example is indeed interesting, similar property holds there, since as we entering the dataflow block, the boundary (the vars that goes into it and implicit variables that are captured by closure) remains immutable during the dataflow block.

Agreed that this is less of a problem in dataflow blocks. We should adequately warn users of the potential for aliasing and provide facilities to be able to deal with aliased values when they are writing non-dataflow code.


fix: content-type super-type resolution for application/octet-stream …

…is now correct

LGTM :+1:


addVertexPair optimized (experiment)

Do math once for both vertices added in addVertexPair.
Measured 30-40% throughput increase.

Performance test results vs master:
#performance createLineGeometry segments=2
  min=0.0009ms (-35.26% vs 0.0014ms) sum=998.99ms (0% vs 998.96ms) repeats=692680.00 (46.09% vs 474149.00) throughput=693383.55/s (46.09% vs 474642.08/s)
  avg=0.0014ms (-31.55% vs 0.0021ms) med=0.0011ms (-38.31% vs 0.0018ms) med95=0.0023ms (-28.2% vs 0.0032ms)
  gcTime=25.641ms (-27.74% vs 35.4837ms) sumNoGc=973.34ms (1.02% vs 963.48ms) throughputNoGc=711649.47/s (44.61% vs 492110.20/s)

#performance createLineGeometry segments=4
  min=0.0026ms (-40.77% vs 0.0043ms) sum=995.31ms (-0.01% vs 995.39ms) repeats=257347.00 (83.18% vs 140487.00) throughput=258560.90/s (83.2% vs 141137.51/s)
  avg=0.0039ms (-45.41% vs 0.0071ms) med=0.0032ms (-39.12% vs 0.0052ms) med95=0.0058ms (-53.23% vs 0.0125ms)
  gcTime=32.758ms (-19.65% vs 40.771ms) sumNoGc=962.55ms (0.83% vs 954.62ms) throughputNoGc=267360.40/s (81.66% vs 147178.59/s)

#performance createLineGeometry segments=16
  min=0.016ms (-30.22% vs 0.023ms) sum=995.74ms (-0.42% vs 999.89ms) repeats=44236.00 (37.13% vs 32258.00) throughput=44425.38/s (37.7% vs 32261.50/s)
  avg=0.0225ms (-27.38% vs 0.031ms) med=0.0182ms (-33.87% vs 0.0275ms) med95=0.0334ms (-26.58% vs 0.0456ms)
  gcTime=42.815ms (3.17% vs 41.4981ms) sumNoGc=952.92ms (-0.57% vs 958.39ms) throughputNoGc=46421.43/s (37.32% vs 33804.95/s)

#performance createLineGeometry segments=64
  min=0.0622ms (-34.09% vs 0.0944ms) sum=999.95ms (0.01% vs 999.88ms) repeats=10958.00 (35.4% vs 8093.00) throughput=10958.54/s (35.39% vs 8093.98/s)
  avg=0.0913ms (-26.14% vs 0.1235ms) med=0.0739ms (-32.56% vs 0.1095ms) med95=0.1392ms (-26.47% vs 0.1893ms)
  gcTime=48.2008ms (21.04% vs 39.8221ms) sumNoGc=951.75ms (-0.87% vs 960.06ms) throughputNoGc=11513.53/s (36.59% vs 8429.08/s)

#performance createLineGeometry segments=256
  min=0.2764ms (-30.88% vs 0.3999ms) sum=1000.06ms (-0.02% vs 1000.29ms) repeats=2615.00 (34.45% vs 1945.00) throughput=2614.85/s (34.48% vs 1944.44/s)
  avg=0.3824ms (-25.64% vs 0.5143ms) med=0.3115ms (-31.64% vs 0.4557ms) med95=0.6779ms (-18.81% vs 0.835ms)
  gcTime=50.0151ms (-12.25% vs 56.9955ms) sumNoGc=950.04ms (0.72% vs 943.29ms) throughputNoGc=2752.51/s (33.46% vs 2062.43/s)


Optimize generated ToString compilation performance

By not calling std::string constructor on every struct field ever.
generated_vulkan_struct_to_string: 79s -> 65s
generated_vulkan_struct_decoders_to_string.cpp: 73s -> 57s
Related: https://github.com/LunarG/gfxreconstruct/issues/761
Found with https://github.com/aras-p/ClangBuildAnalyzer.

Author ishitatsuyuki not on autobuild list. Waiting for curator authorization before starting CI build.

CI gfxreconstruct build queued with queue ID 27988.

CI gfxreconstruct build # 2182 running.

CI gfxreconstruct build # 2182 passed.

@andrew-lunarg Mind taking a look? This should be fairly trivial to review.


Fix two out-of-bounds bugs in inchi interface

What does this implement/fix? Explain your changes.
We (https://patents.google.com/) encountered some out of bounds bugs while converting the following SMILES strings to InChI:

CC[C@H]1OC(=O)C[C@@H](O)[C@H](C)[C@@H](O[C@@H]2O[C@H](C)[C@@H](O)C(N(C)C)C2O)[C@@H](CC=O)C[C@@H](C)C(=O)/C=C/C(C)=C/[C@@H]1CO.CC[C@H]1OC(=O)C[C@@H](O)[C@H](C)[C@@H](O[C@@H]2O[C@H](C)[C@@H](O[C@H]3CC(C)(O)[C@@H](O)C(C)O3)C(N(C)C)C2O)[C@@H](CC=O)C[C@@H](C)C(=O)/C=C/C(C)=C/[C@@H]1CO[C@@H]1OC(C)[C@@H](O)[C@H](OC)C1OC.[3H][Y]([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])([3H])[3H]
Cc1cc(C)c(N2CCN(c3c(C)cc(C)cc3C)C2[Ru@OH28]2(Cl)(Cl)=Cc3cc([S@SP3](=O)(=O)N(C)C)ccc3O->2C(C)C)c(C)c1

Any other comments?
This may not be the most correct fix :/ but it's better than segfaulting :)
Thanks!

@chbrown Thanks for identifying the problem and proposing a fix!
I created a PR with an alternate solution, along with tests for the two cases you identified here:
https://github.com/rdkit/rdkit/pull/8126
I would like to make sure you get credit for this. Could you please create an issue reporting the problem? I will then link that from the other PR.

Ok, created issue #8129
Thanks!

Superseded by #8126


Unable to find named pipes/endpoints

Hi. I love your tool but I currently am facing an issue in Certipy 3.0.0. (main). I will describe the issues below but I cannot post any screenshots for obvious reasons.
I client of mine is running ADCS on a Server 2019 machine which is not a domain controller. The web interface is enabled and there are multiple vulnerable custom certificate templates. There are however many restrictions on who can enroll certificates.
When I attempt to abuse ESC1 using a machine account which has enrollment rights via the domain group Domain Computers and a template which is vulernable, the attack first fails with  "STSTATUS_OBJECT_NAME_NOT_FOUND" when Certipy attempts to find the required named pipe (cert?). After that the backup feature which attempts to identify dynamic endpoints kicks in but that also fails with "Failed to get dynamic TCP endpoint for CertSvc" and "NoneType object has no attribute 'request'".
What is strange is that abusing ESC8 against the same ADCS server works when authenticating using a  DC's machine account I coerce using PetitPotam.py and specifying the template KerberosAuthentication. I get the certificate but when I then attempt to get the matching TGT and NT hash it seems the authentication against the DC fails. I get the error "ERR_CLIENT_NOT_TRUSTED". According to nmap against the same DC, services that requires TLS are configured to use certificates which are issued by the same CA that is configured on the ADCS server so it seems it is not the CA that is not trusted.
Any suggestions? Thanks!

Hello. First of all, you should make sure that the certificate requests are targeted the CA server and not the domain controller, which might explain the "STSTATUS_OBJECT_NAME_NOT_FOUND". Secondly, it sounds like this is a standalone CA server and not an enterprise CA server, which might explain the "ERR_CLIENT_NOT_TRUSTED". Can you verify or did you resolve the issues?

Hi. Thanks for your reply. Unfortunately I can no longer check or test things since I do not have access to the mentioned infrastructure anymore. However, according to https://serverfault.com/questions/826444/difference-between-microsoft-adcs-standalone-ca-and-enterprise-ca only enterprise CA servers have templates and as I mentioned there were plenty of those. You can also see that the ESC8 attack worked against the same ADCS server.
It is of course obvious to you but I wonder if the web interface is used for all ESCX attacks as it is for the ESC8 attack? Or put in other words, does the ESC8 attack, which worked, require using named pipes? Also, I think it is possible to restrict access to named pipes using ACLs. Could that be a explanation?

Hello @jsdhasfedssad Sorry for the late reply. I still haven't figured out why this is the case. However, I have implemented web enrollment in the upcoming version of Certipy, such that if web enrollment is enabled (HTTP/HTTPS), then it's possible to abuse ESC1-ESC3 (certificate templates) through that in case RPC is not available.


When RayCast2D placed next to a Body(StaticBody2D) it activates.

Godot version
3.3 stable
System information
Windows 10, GLES3
Issue description
Using Grid Snapping with RayCast2D and placed next to a Body(StaticBody2D) it activates. It shouldnt happen as raycast is not placed in the collisionShape2D.
Steps to reproduce
Create a StaticBody2D and place RayCast2D next to it using Grid Snap. Then if placed directly in the StaticBody2D node disable exclude parent. Then write a script that displays a message when RayCast is colliding.
Minimal reproduction project
Raycast2D_bug.zip

3.3 stable

Can you reproduce this on 3.4? 3.3 is no longer supported (except for security fixes).

oh, i wrote the wrong version. Yes, it works in the latest version

very annoying tho , you have to disable grid snap and pixel snap and move it a little bit to fix it.

oh, i wrote the wrong version. Yes, it works in the latest version

Closing as resolved then.

very annoying tho, you have to disable grid snap and pixel snap and move it a little bit to fix it.

This is expected, as the RayCast must be located fully outside the StaticBody2D to avoid a collision. If you do not wish this to happen, configure the StaticBody2D's collision layers then configure the Raycast2D's cull mask accordingly, or add a collision exception to prevent the StaticBody2D from being registered as a collider for the RayCast2D.
PS: Please use the Edit button (hidden behind the … icon in the top-right corner of your comments) instead of multi-posting.


SwitchIOS and SwitchAndroid Styling

How can we style the border of a SwitchIOS?  See below -- how can I style that black border?

Also, how come there is no styling props for SwitchAndroid?

Looks like styling for Android isn't supported at the moment.
For styling the border on IOS you need to set the tintColor prop.
More information inside the docs:
https://facebook.github.io/react-native/docs/switchios.html#tintcolor
Please remember to close this issue if it solves your problem. Thanks!

Hi there! This issue is being closed because it has been inactive for a while.
If you'd like you can create an issue on Product Pains.
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub.
Also, if this issue is a bug or a feature request, please consider sending a PR. Thanks!


Add support for Vue.js plugin

No requirements, but the playground/innovation app that @wbprice is working on could use this if it supported Vue in addition to React.

@dlindahl I think we have the Vue project now!

Yes!


Exercise 8 Ingress

Might have them create an ingress resource - or explicitly call out the needed previous exercise as a pre-req. Probably best not to assume users are doing this all in one sitting.
Also, include command for getting IP kubectl get service istio-ingress -n istio-system

Andy, can you check it out? ^^

Closing per @nilesh32's request


tor: <IP_ADDRESS> -> <IP_ADDRESS>

Description of changes

new stable release: https://forum.torproject.net/t/stable-release-0-4-7-7/3108

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] 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.

Can repro the aarch64 test failures seen on onborg:
sandbox/open_filename: [forking]
  FAIL src/test/test_sandbox.c:158: open: Operation not permitted [1]
  [open_filename FAILED]
sandbox/opendir_dirname: [forking]
  FAIL src/test/test_sandbox.c:264: opendir: Operation not permitted [1]
  [opendir_dirname FAILED]
sandbox/openat_filename: [forking]
  FAIL src/test/test_sandbox.c:242: openat: Operation not permitted [1]
  [openat_filename FAILED]
sandbox/chmod_filename: [forking]
  FAIL src/test/test_sandbox.c:184: chmod: Operation not permitted [1]
  [chmod_filename FAILED]
sandbox/chown_filename: [forking]
  FAIL src/test/test_sandbox.c:202: chown: Operation not permitted [1]
  [chown_filename FAILED]
sandbox/rename_filename: [forking]
  FAIL src/test/test_sandbox.c:222: rename: Operation not permitted [1]
  [rename_filename FAILED]

Can repro the aarch64 test failures seen on ofborg:

Wondering what's the difference in sandboxing between x86_64-linux and aarch64-linux ...

Reported upstream: https://gitlab.torproject.org/tpo/core/tor/-/issues/40599

It doesn't seem like this is NixOS specific -- I noticed someone on the Tor side is working on adding arm64 CI, and the test fails similarly in their runs: https://gitlab.torproject.org/eighthave/tor/-/jobs/128518
Maybe we should build Tor on aarch64 with the sandbox disabled? (I think it can be optionally disabled, at least by not providing a libseccomp input?)

I think it can be optionally disabled, at least by not providing a libseccomp input?)

Disabled seccomp in 113ca3c (via a configure flag, removing libseccomp from inputs does not thing, because libseccomp is already present in the system as a dependency of something else).

@GrahamcOfBorg build tor


Browser zooms on Inputs in iOS

In Web versions, iOS browsers getting focus on Input components that their font sizes are less than 16px.

closed by #7826


chore(deps): update dependency cron to ^3.3.2

This PR contains the following updates:

Package
Type
Update
Change

cron
dependencies
patch
^3.3.1 -> ^3.3.2

[!WARNING]
Some dependencies could not be looked up. Check the warning logs for more information.

Release Notes

kelektiv/node-cron (cron)
v3.3.2
Compare Source
🐛 Bug Fixes

fix infinite loop on expressions resolving only inside a DST forward jump (#938) (efb8df5), closes /github.com/kelektiv/node-cron/pull/667/files#diff-c14c2dca8456f15417b39cfbd9758009f8eb4f3a190a415768d6e4ae6ae9dceeL473-L477 #919 #919

⚙ Continuous Integrations

action: update marocchino/sticky-pull-request-comment action to v2.9.0 (#930) (1e7bce9)
renovate: pin GitHub action digests to semver (#926) (6541167)

♻ Chores

deps: lock file maintenance (70c3339)
deps: lock file maintenance (afad454)
deps: lock file maintenance (b1dbf69)
deps: pin dependencies (#915) (dfcbd3c)
deps: update dependency @commitlint/cli to v19.6.1 (7999427)
deps: update dependency @semantic-release/release-notes-generator to v14.0.2 (93c9373)
deps: update dependency @types/node to v20.17.10 (9313ffd)
deps: update dependency lint-staged to v15.2.11 (100c9ff)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Hywax Assistant.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: pnpm-lock.yaml
Scope: all 2 projects
Progress: resolved 1, reused 0, downloaded 0, added 0
playground                               | Progress: resolved 1, reused 0, downloaded 0, added 0
undefined
 ERR_PNPM_FETCH_404  GET https://npm.pkg.github.com/@hywax%2Fvitepress-yandex-metrika: Not Found - 404

This error happened while installing a direct dependency of /tmp/renovate/repos/github/hywax/nuxt-cron

@hywax/vitepress-yandex-metrika is not in the npm registry, or you have no permission to fetch it.

An authorization header was used: Bearer ghp_[hidden]


Fix PythonVirtualenvOperator when using provide_context=True (#8256)

https://github.com/apache/airflow/pull/8256
This change will ensure that provide_context=True works on v1.10.12 for PythonVirtualenvOperator
Clean up code and simplified the solution to ignore just the unserializable items
adding pytest for PythonVirtualenvOperator

cc @lyft/dpe-airflow

Codecov Report

Merging #229 into airflowinfra will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##           airflowinfra     #229      +/-   ##
================================================
- Coverage         27.79%   27.79%   -0.01%
================================================
  Files               528      528
  Lines             36622    36625       +3
================================================
  Hits              10179    10179
- Misses            26443    26446       +3

Impacted Files
Coverage Δ

airflow/operators/python_operator.py
31.21% <0.00%> (-0.61%)
:arrow_down:

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 2999a99...50b31ab. Read the comment docs.

👍
Looks like this won't actually unblock the user though. If ti isn't supported then we can't pull xcoms, right?

Yeah. I'm working with him on that. Might as well merge this anyways though.

💨


add logging sidecar

Describe the Feature
Add a sidecar container to each of the tasks in the project to read logs and capture errors:

read logs from main container
capture errors and store in DynamoDB using a service identifier

needs:

sidecar container definition
sidecar container service (write the code)
dynamodb table to hold errors

questions:

if the primary container exits, does the sidecar also exit? Will we be able to still capture the error before the task is spun back up?

probably should introduce something like winston to log to a file rather than redirecting output from the container process https://www.npmjs.com/package/winston


Wrong dependency in package.json

Hello,
it seems that the generated "package.json" for angular-components gets a wrong dependency.
Reproduction: after a clean install via "yo ng-factory" and the question "What are you building today?" answered with "component" the undertaker-dependency is: "undertaker-app-tasks": "^0.2".
This leads to an error if you try to run a gulp-task (serve or build):
Error: Bower components directory does not exist at /path/to/your/project//app/bower_components
Well, changed this dependency to "undertaker-lib-tasks": "^0.3" resolves this error and builds the project the right way, without any issue.
Hope this detail can help make it even better ;)

@bkrieger1991 Thanks for the report ;-)!


Fixes the iOS scrolling problem

fixes #217
A Safari update introduced this bug where the whole window scrolls.
Note I branched from the firebase changes, so this should probably merge after that. It's really just one commit (the last one).

LGTM, one comment about something to try.


Update _chatListeners.mjs

Added ability to utilize amount in embedded function and to change the actor referenced by the embedded function.
Use case:  The embedded function requires a change amount and the actor to which it will be applied.  The catch is that only the GM has permission to make this change.  Using the Requestor framework, an automated request is made to the GM with the amount and correct target set in the button as derived from the arguments.  This allows the GM to click the button to implement it (or ignore / delete with the trash icon).
Issues have been that the macro tries to use the GM as the actor (which fails) or references the player and so returns a permission error.  This is the first work-around I've found to automate the process.
action: async () => {
          await game.hm3.macros.changeFatigue(amount, actor)
        }

While I can see a common use to pass an id or similar to declare actor as something else, I am not seeing the system-agnostic use of amount. Would this serve better as a system-specific method for the hm3 system?
If you agree, you can submit a collection of common functions to demos.mjs same as I have included common uses for the dnd5e system.
See here as well: https://github.com/krbz999/requestor/blob/main/scripts/_api.mjs

To my mind, amount would be anything needed for any macro (damage rolls, spell point loss, sanity points, etc.)  The intention was to keep it as generic as possible (in keeping with the approach).
It's possible I don't understand the correct way to submit such numbers, and you've already got this covered another way.
HM3 is the Harn system using the Foundy VTT module here: https://github.com/toastygm/HarnMaster-3-FoundryVTT
There is a walk-through with links to Using Macros and Example Macros.
I'm not playing/running any 5E games at the moment with Foundry.

Here's an example from our Harn game.  Amount holds all the information that needs to be passed to the macro so it can be unpacked and used.
rollResult.aspect = "Fire";     rollResult.impact = dmg_rndInt;     rollResult.aim = "Mid";     await Requestor.request({       description: result,       sound: "worlds/novendo/sounds/heatwave.ogg",       img: targetActor.thumbnail,       buttonData: [{         actor_from_token_ID: targetToken.id,         amount: rollResult,         permission: "GM",         limit: 1,         label: request_label,         action: async () => {           await game.hm3.macros.injuryRoll(actor, amount);         }       }]     });

Here's a spell example.  Again, everything needed can be packed into Amount to keep the parameters simple, flexible, but powerful.
const effectData = {
  actor_from_token_ID: targetToken.id,
  rollRes: rollResult.result,
  spellName: spellName,
  convSI: convSI,
  itemdataimg: item.data.img
}

let result = spellName + " mesmerizes."
if (rollResult.isCritical) {
  result = spellName + " transfixes!"
}
let request_label = spellName + " to " + targetActor.name
await Requestor.request({
  description: result,
  sound: "worlds/novendo/sounds/wow.ogg",
  img: targetActor.thumbnail,
  buttonData: [{
    actor_from_token_ID: targetToken.id,
    amount: effectData,
    permission: "GM",
    limit: 1,
    label: request_label,
    action: async () => {
      await game.hm3.macros.applyEffect(amount);
    }
  }]
});

Firstly, let me apologize for having let this linger for so long.
With v11 coming up, I've taken the opportunity to rewrite the module, with better code (and some great and also breaking changes on top). One being that any argument passed to a button will be directly accessible (eg passing scope: {amount: 0} will let you use amount as is, and secondly if passing tokenId or actorId in the scope object, then token or actor will be defined using those ids rather than the selected token/actor.
Though it does mean this PR isn't useable, I hope this satisfies the request.


fixed bug for port forward to support ipv4 and ipv6

Please provide a description of this PR:
fixed bug for port forward to support ipv4 and ipv6
may be related to issue#40605

@zhlsunshine: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
9f597d3943eb51bab0856a4ce9c06c284ddabd32
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhlsunshine: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
9f597d3943eb51bab0856a4ce9c06c284ddabd32
link
true
/test unit-tests-arm64

unit-tests_istio
9f597d3943eb51bab0856a4ce9c06c284ddabd32
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

can you add a release note for this and include the issue linked in the description?

Hi @GregHanson , sure, I will add the release note. Thanks for your commment!

Hi @GregHanson the release note has been added

!Hi @istio/wg-policies-and-telemetry-maintainers  & @istio/release-managers, PTAL for this PR. thanks!

removing do-not-merge after verifying with @zhlsunshine

/cherry-pick release-1.15

@GregHanson: once the present PR merges, I will cherry-pick it on top of release-1.15 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.15

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@GregHanson: new pull request created: #41464

In response to this:

/cherry-pick release-1.15

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Add Client namespace on README.md example

It's easier for users know witch kind of Client the service must receive.

Thanks guy! :metal:


The "types" here seems redundant

the types is hard to understand and according to the following content at line 535, so I think the "types" should not be existed.

/assign

/assign

Hi @lostsquirrel thank you for your contribution.  I don't believe this change is warranted. Several others agree as well. Because of this I am going to close this PR.
/cc @jimangel

Hi @lostsquirrel thank you for your contribution.  I don't believe this change is warranted. Several others agree as well. Because of this I am going to close this PR.
/cc @jimangel


android: new target sdk requirement in nov 2018

https://android-developers.googleblog.com/2017/12/improving-app-security-and-performance.html

https://perkeep-review.googlesource.com/c/perkeep/+/18666


Timed Action Lists list creature smart scripts instead of action lists

Describe the bug
When trying to load a timed action list via quickload, the loading window lists creature smart scripts instead of timed action lists. Though you still can load the action lists by adding the value to the search bar, there wont be any search results as the list does not contain action lists at all.
Version (commit hash)
7afcb458c0cd45a1f3e0f51457f090512ee8645c
To Reproduce

Hit the timed action list button on the quickload menu
See that the entries of the popping up list are no timed action lists

Expected behavior
The load list should contain timed action lists instead of creature smart scripts

Fixed in https://github.com/BAndysc/WoWDatabaseEditor/commit/b2d93720974f2c0bd1d232ab77779dce944b393e


How to host on my own machine?

I copied a Gemfile and Gemfile.lock , and ran bundle exec jekyll serve but it just does not work..

Which Gemfile? what error message did you received?

I just cloned this repo and tried with bundle exec jekyll serve jekyll run the project, however it comes up with a error msg Could not locate Gemfile or .bundle/ directory  so I copied some Gemfile from another sample project provided by jekyll...
I just don't know how to run it on a local machine since I'm a junior in front-end....
so I'm looking for an answer
thank you so much

Remove the Gemfile and Gemfile.lock and try running jekyll serve

Conversion error: Jekyll::Converters::Scss encountered an error while converting 'css/main.scss':
Invalid US-ASCII character "\xE2" on line 74

weired。。。there is no line 74 at all

This thread might help - https://github.com/jekyll/jekyll/issues/4268.
If you observe the last statement, it imports all the SCSS present in the _sass folder. So it has lines more than 100.

many thanks


Implement preset system

value of gs
low, hi in map(gs, _, _, low, hi)
window size
image size
glsl code (?)

With the canvas implementation I can use Tweakpane to import/export presets

Implemented via LazyGui in #16 and #84


Is this project abandoned?

 I honestly think you could be the next popular js ui framework.

It might be considered abandoned. I had to do some full-time work and haven't been able to get back to this project. I agree it has a lot of potential, but needs a lot of effort to make it usable.


Allow servers to send and receive messages directly

Added ability for servers to send and receive messages as their own entity.
Added account scoped events for connection created and connection closed.
Signed-off-by: Derek Collison<EMAIL_ADDRESS>/cc @nats-io/core

Coverage decreased (-0.04%) to 92.143% when pulling 8d05ed82ff3aa94c520ecd763aec8fad8a269eae on sys into 8a662a57605c21e01841197071526c5a05b1cd3a on master.


Representing SequenceAnnotations without Components

When a SequenceAnnotation does not have a Component, you should open a SequenceAnnotation editor that allows you to edit the Role and Sequence at that location.

SequenceAnnotations with Roles and no Component now appear on the Canvas.  There is also a new PartEditDialog for them.  It has several TODOs though marked in the code.  The biggest one is making this editor actually allow you to edit.  Right now it always opens read-only.
Another issue with represent SA's with Roles is that It does not handle mixed Components and SA w/roles.  I think this really only makes sense when the sequence is known.  In this case, you could likely use getSortedSequenceAnnotations, then check if there is a Component or Roles and act accordingly.  There should not be any Components without SequenceAnnotations when the sequence is known, so this should catch everything.  Also, they should all have explicit locations, so the SequenceConstraints should be safely ignored.
Finally, there is no way yet to add new SequenceAnnotations with Roles.

This was addressed with the sequence feature view enhancement.


Do not rely on /usr/bin/env in critical paths

rbenv uses a lot of sub shell script calls, and #!/usr/bin/env bash is executed each time. This is not very efficient depending on the PATH settings.
This changeset is to use $BASH rbenv-* when calling rbenv-* in critical paths to invoke ruby. This eliminates the overhead of calling /usr/bin/env.
Before:
$ time ruby -v
ruby 3.3.0dev (2023-02-20T01:33:06Z master 7d5794bad5) [x86_64-linux]

real    0m0.532s
user    0m0.022s
sys     0m0.087s

After:
$ time ruby -v
ruby 3.3.0dev (2023-02-20T01:33:06Z master 7d5794bad5) [x86_64-linux]

real    0m0.204s
user    0m0.018s
sys     0m0.028s

Note: My environment is WSL2 on Windows and my PATH setting contains some NTFS directories; NTFS access is very slow, so this problem is very noticeable.
The idea of reducing the overhead of /usr/bin/env by using $BASH comes from @naruse.

Hi, thanks for the suggestion and for sharing the speed comparison.
In case of a slow PATH, I think you will have many more problems with your dev environment than just with rbenv. The shebang /usr/bin/env is practically standard.
However, rbenv is known to add some overhead to every Ruby invocation. The biggest contributor to that is that bash is slow. In your case, another contributor to that is that searching the PATH is slow. To avoid any overhead to executing ruby, I would suggest moving to something like chruby instead.
I do want to take a deeper look into your PR, so please give me some time to do that. I want to make the speed comparison on systems where there are no NTFS directories in path, as well as try to think about the possible downsides of the change you've proposed.

Thank you for your quick reply!
Here is the minimum and average seconds for 1000 invocations of ruby -v when there is no NTFS directory in path.

minimum
average

Before my patch
0.0888
0.175

After my patch
0.0831
0.158

It is not so fast as the case of NTFS directory, but it seems to be faster.
However, I understand that you don't want to insert a lot of $BASH from a maintenance point of view. If you don't like it, feel free to close it. Thanks for your time!

Do we have documentation as to which versions of bash reliably set $BASH? (especially when it was introduced) Any known versions of bash that have bugs regarding that variable?
Would also want to double confirm this is fine on FreeBSD (it being the primary example for why /usr/bin/env is preferred over /bin/bash). I don't have any suspicion that it wouldn't, but would be prime suspect to confirm.

Do we have documentation as to which versions of bash reliably set $BASH? (especially when it was introduced)

Good question. I don't know when it was introduced, but it is already mentioned in the manpage of bash-1.14.7 (in 1996).
https://git.savannah.gnu.org/cgit/bash.git/commit/?id=726f63884db0132f01745f1fb4465e6621088ccf
Even in the case that $BASH is not defined, rbenv will work as before as long as $BASH is interpreted as empty.

Any known versions of bash that have bugs regarding that variable?

I don't know, but I hope it is stable enough.

Would also want to double confirm this is fine on FreeBSD (it being the primary example for why /usr/bin/env is preferred over /bin/bash).

I confirm that $BASH is defined in bash on my FreeBSD machine.
[mame@freebsd /usr/home/mame]$ echo $BASH
/usr/local/bin/bash

I made a quick check that my rbenv branch works great on the machine. A slight improvement was also observed on the machine.

min.
ave.

Before my patch
0.0218
0.0294

After my patch
0.0195
0.0251

Hi, thanks for your efforts to speed up rbenv, but I've taken a stance that if /usr/bin/env (or PATH resolution) is slow on a certain system, it sounds like rbenv isn't a good fit for that system. See https://github.com/rbenv/rbenv/wiki/Comparison-of-version-managers


NIFI-4675 Lifted restriction on demarcator and kafka.key usage together

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?
YES, NIFI-4675

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Given your description of the user understanding the decision they're making I think I agree with you.
@markap14  what do you think?
@jasper-k we'll want to make sure kafka 10, 11, and 1 are all updated for this but lets see what mark thinks too.
Thanks for contributing and sharing your findings

@jasper-k the travis-ci builds show there are some contrib-check failures.  Make sure you run that locally so you can see them more easily.  "mvn clean install -Pcontrib-check"  Please rebase to latest master, resolve the contrib-check, and force push.

@joewitt It seems to be checkstyle errors, but there is no obvious error. Any tip to setup IntelliJ to use the  embedded in the pom.xml?

@jasper-k @joewitt I think I am ok with this change. I do tend to expect Kafka keys to be unique because otherwise Kafka may lose the data. However, I know there are use cases where some users want to use the same key anyway, so given the apt description in the property this is probably a good improvement. Will merge to master.

OK tested locally and all looks good. +1 merged to master. Thanks for contributing the improvement!


Proposal for a PoC : Providing a tool for people to be able to crowd source their learning

I believe that as well as providing strategies we should enable teams to implement these strategies by providing tools they can use.
Problem Statement
We are requiring teams to spend 20% of their time for learning and continuous improvement. It is the managers responsibility to report on what the team has learned and improved. Currently, if the strategy were to role out, this would mean manual tracking and reporting which creates additional burdens on teams taking away from learning  and discouraging from adopting the strategy. We have created a CATS code for continuous learning however, it does not allow for insight on the actual learning and improvement
Possible Approach
Explore providing a tool for teams to be able to crowd source their learning on a team level. Users would be able to add links to resources they have viewed. This would provide two benefits:

Automated reporting for managers: Managers would be able to get detailed automated reports on team learning. No overburdening them with manual tracking
Automated knowledge sharing with the rest of the team: A living knowledge base is built on the go which would be searchable, that would allow fast knowledge sharing across the team

There are tools that do something similar to this but on an individual level such as Instapaper and Pocket whereas we are looking for more team and organizational level.
I am proposing we look at a proof of concept of either finding any tools out there that may have this functionality or building it ourselves.

@RachelMuston had the idea of adding a learning journal as well which would allow a person to be able to log over time their learning experiences

Minimum functionality hoping to achieve in two weeks

[ ] Be able to add resources ( links )  and automatically parse descriptions, title, image a user can correct and augment with data like custom categories
[ ] Be able to search in natural language for links and resources
[ ] Dashboard showing resources added and accessed

https://microlink.io/docs/api/getting-started/data-fields
for metadata extraction from resources... underlying library is open source and can be used to build our own endpoint in the future https://github.com/microlinkhq/metascraper

PostgreSQL has full text search capabilities can be used to provide search on titles and descriptions in English and French


Wrong Date/Time Display

Code of Conduct

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

Submission validity

[X] This is not a personal support request, that should be posted on discussions community.
[X] I checked current issues and this request isn't a duplicate of an existing issue, opened or closed.

Self troubleshooting

[X] I've read the documentation and made sure to follow it.
[X] I've read the troubleshooting first steps and frequent issues guides.

Version
1.9.1
Description
After Upgrade to the newest version, i have a wrong date/time output.

Expectation
No response
Reproduction steps
No response
Context
No response

Seems like an error in the translation file you're using, check with the translator


Supports runtime injection of Param.Expander

This supports runtime injection of Param.Expander. Implementing
contracts will assign MethodMetadata.indexToExpander using configured
values. When MethodMetadata.indexToExpander is unset, Feign has the
existing behavior, which is to newInstance each indexToExpanderClass.
Fixes #338

NetflixOSS » feign » feign-pull-requests #227 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » feign » feign-pull-requests #228 SUCCESS
This pull request looks good

:+1:

NetflixOSS » feign » feign-pull-requests #229 SUCCESS
This pull request looks good


Cannnot save into csv

CRITICAL:root:twint.output:_output:CSV:Error:'Config' object has no attribute 'Tabs'
'Config' object has no attribute 'Tabs' [x] output._output

same error with me

how did you overcome @irvanseptiar ?

Try doing t.Config['tweet'] - ["id","location","date","time"....] , make sure of 't.Store_csv = True' and 't.Output = ".csv"'. This works for me

Try doing t.Config['tweet'] - ["id","location","date","time"....] , make sure of 't.Store_csv = True' and 't.Output = ".csv"'. This works for me
Below is my code,  it just raise this error, could you please your implementation and help me out?

CRITICAL:root:twint.output:_output:CSV:Error:'location'
'location' [x] output._output
1321413272885669889 2020-10-28 07:27:29 -0400  @att your business fiber internet has been down since Monday.  Update on repair, please?  @ATTHelp #outage
import twint
c = twint.Config()
c.Custom['tweet'] = ['id', 'location', 'date', 'time']
c.Lang = 'en'
c.Profile_full = True
c.Search = 'outage ATT'
c.Store_csv = True
c.Limit = 10
c.Output = 'test.csv'
twint.run.Search(c)

@Shawn1206 Yes it turns out 'Location' has not been working since a few days. As for storing for csv, if the above doesn't work, you could always enable pandas and use it to save the csv, and maybe even clean it before.
import twint
import pandas
import nest_asyncio
nest_asyncio.apply()
t = twint.Config()
t.Search = "outage ATT"
t.Pandas = True
twint.run.Search(t)
twint.storage.panda.Tweets_df.to_csv("output.csv")

@Shawn1206 Yes it turns out 'Location' has not been working since a few days. As for storing to csv, if the above doesn't work, you could always enable pandas and use it to save the csv, and maybe even clean it before.
EDIT: Ran below code and got 180 tweets :)
import twint
import pandas
import nest_asyncio
nest_asyncio.apply()
t = twint.Config()
t.Search = "outage ATT"
t.Pandas = True
t.Since = "2020-10-01"
t.Lang = 'en'
t.Profile_full = True
twint.run.Search(t)
twint.storage.panda.Tweets_df.to_csv("output.csv")

Thanks a lot! From the output, it seems that there are a lot of empty cells in the "place" attribute, maybe people just don't like share their geo information.

@Shawn1206 A very naive question: How do you manage to use twint at this time? I thought it is still broken (see #604)? If I run your code example (without the nest_asyncio part), it gives me the usual CRITICAL:root:twint.run:Twint:Feed:noDataExpecting errors. How did you make this work?! :-)

@LinqLover
Twint was ported to the newer v2 twitter API endpoints a while ago,
You must be using the older installation if you are still getting the CRITICAL:root:twint.run:Twint:Feed:noDataExpecting
ERROR.
Try updating your twint installation from GitHub and check if it works for you.

@himanshudabas Woah, thank you very much! Looks like it was forgotten to report this update on #604, which is still open. :-)


Error code: SBOX_FATAL_MEMORY_EXCEEDED when loading segmentations

Bug Report
Describe the Bug
I created multi-segment segmentations using highdicom of size 167x512x512. When viewing these segmentations with the latest OHIF viewer from docker hub, my browser (Chrome) goes out of memory with Error code: SBOX_FATAL_MEMORY_EXCEEDED.
The segmentations were created with
seg_dataset = hd.seg.Segmentation(
        source_images=image_datasets,
        pixel_array=mask,
        segmentation_type=hd.seg.SegmentationTypeValues.BINARY,
        segment_descriptions=segment_descriptions,
        series_instance_uid=hd.UID(),
        series_number=20,
        sop_instance_uid=hd.UID(),
        instance_number=1,
        manufacturer='XYZ',
        manufacturer_model_name='XYZ',
        software_versions='v1',
        device_serial_number='XYZ',
        omit_empty_frames=False
    )

When I only write out the first segment, the segmentation is rendered just fine, but Chrome consumes several GB of memory.

Please review the latest code in the master branch. This issue might have been resolved. If it persists, kindly reopen the issue with updated details.
Try viewer-dev.ohif.org instead of viewer.ohif.org
Our viewer.ohif.org is deployed from release branch while viewer-dev.ohif.org is our master  branch
Read more about branch explanations here https://docs.ohif.org/development/getting-started#developing


Add command for transferring ownership & key backup

Adds a dat keys command a two subcommands: dat keys import and dat keys export.

dat keys - prints out dat key and discovery key
dat keys export - prints out secret key (does not remove ownership from existing dat)
dat keys import - prompts user for secret key to make a dat writable.

These are still somewhat advanced key features but will at least make it possible to backup or transfer ownership of a dat.
(#827)

LGTM! Really nicely implemented.

FYI: Waiting to fix tests before merge. Thinking that https://github.com/mafintosh/hypercore/issues/109 is causing the test errors but gonna debug a bit more when mafintosh gets into town.

this is documented in the README but I don't believe it's listed in the initial dat --help printout from the command line. dat keys --help will give details though. this could very well be intentional, but thought I'd mention it.

Thanks, yes it is a bit intentional. Importing/Exporting via dat keys still has the potential to corrupt a dat with incorrect use. We'll leave it for more advanced use cases until the multiwriter lands to avoid that.


Figure out how to fetch metrics via metricbeat for API Gateway

API Gateway metrics are sometimes not being fetched using ./metricbeat run -v from beats/x-pack/metricbeat. Metricbeat works fine but I am getting no events.
Possible issues:

No metrics can be fetched for any service -> using the same config with AWS/EC2 as namespace and with the appropriate metrics works. AWS/ApiGateway is the correct namespace, metrics used are also correct.

"metricbeat":{"aws":{"cloudwatch":{"events":42,"failures":0,"success":42}}} after fetching EC2 metrics for ~1 minute.
"metricbeat":{"aws":{"cloudwatch":{"events":0,"failures":0,"success":0}}} after fetching API Gateway metrics for ~15 minutes.

Too few metrics are being generated by the API Gateway?

???

Config in beats/x-pack/metricbeat/modules.d/aws.yml:
- module: aws
  period: 1m
  metricsets:
    - cloudwatch
  access_key_id: REDACTED
  secret_access_key: REDACTED
  metrics:
  - namespace: AWS/ApiGateway
    statistic: ["Average"]
    name:
    - 4XXError
    - 5XXError
    - CacheHitCount
    - CacheMissCount
    - Count
    - IntegrationLatency
    - Latency

Metrics via CloudWatch:

Metrics via AWS CLI command:
aws cloudwatch get-metric-statistics \
    --namespace AWS/ApiGateway \
    --metric-name Count \
    --start-time "$(date -u +"%Y-%m-%dT00:00:00Z")" \
    --end-time "$(date -u +"%Y-%m-%dT23:59:59Z")" \
    --period 300 \
    --statistics Average

First, check if the metrics are included in the time window the CloudWatch metricset uses:
https://github.com/elastic/beats/blob/f65554fdfb439f71f7107524c4c050767f2e9bb7/x-pack/metricbeat/module/aws/utils.go#L32-L37

Second, if the API Gateway service or CloudWatch takes time to publish the metrics, then Metricbeat may miss them.
We can leverage the latency configuration option to check and address this issue.
You could :

set up some script that generates metrics so you have a predictable data flow
bump latency to 5m

Five minutes is very high, so it may be a latency issue if it starts collecting metrics reliably. If so, find the threshold by decreasing the latency value until you stop receiving metrics reliably.