Cannot import tensorflow

First time post on github.  trying to import tensorflow on local machine.  keep getting a runtime error.  please advise.
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No
TensorFlow installed from (source or binary): pip install tensorflow on miniconda3
TensorFlow version (use command below): 2.2.0
Python version: 3.8.5
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: cudnn-11.0-windows-x64-v<IP_ADDRESS>.zip   cuDNN8.0.4
GPU model and memory: NVIDIA Quadro K1100M.  This has compute capability 3.0 which is less than the minimum required... see https://www.tensorflow.org/install/gpu#hardware_requirements

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
tensorflow is not imported.  instead i get a runtime warning... see warning below in Other info / logs
Describe the expected behavior
Standalone code to reproduce the issue
import tensorflow as tf
Other info / logs
ImportError                               Traceback (most recent call last)
~\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow.py in
57
---> 58   from tensorflow.python.pywrap_tensorflow_internal import *
59
~\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py in
27             return _mod
---> 28     _pywrap_tensorflow_internal = swig_import_helper()
29     del swig_import_helper
~\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py in swig_import_helper()
23             try:
---> 24                 _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
25             finally:
~\miniconda3\envs\quant\lib\imp.py in load_module(name, file, filename, details)
241         else:
--> 242             return load_dynamic(name, filename, file)
243     elif type_ == PKG_DIRECTORY:
~\miniconda3\envs\quant\lib\imp.py in load_dynamic(name, path, file)
341             name=name, loader=loader, origin=path)
--> 342         return _load(spec)
343
ImportError: DLL load failed while importing _pywrap_tensorflow_internal: The specified module could not be found.
During handling of the above exception, another exception occurred:
ImportError                               Traceback (most recent call last)
 in
----> 1 import tensorflow as tf
~\miniconda3\envs\quant\lib\site-packages\tensorflow_init_.py in
39 import sys as _sys
40
---> 41 from tensorflow.python.tools import module_util as _module_util
42 from tensorflow.python.util.lazy_loader import LazyLoader as _LazyLoader
43
~\miniconda3\envs\quant\lib\site-packages\tensorflow\python_init_.py in
48 import numpy as np
49
---> 50 from tensorflow.python import pywrap_tensorflow
51
52 # Protocol buffers
~\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow.py in
67 for some common reasons and solutions.  Include the entire stack trace
68 above this error message when asking for help.""" % traceback.format_exc()
---> 69   raise ImportError(msg)
70
71 # pylint: enable=wildcard-import,g-import-not-at-top,unused-import,line-too-long
ImportError: Traceback (most recent call last):
File "C:\Users\Jeff\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\Jeff\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\Jeff\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "C:\Users\Jeff\miniconda3\envs\quant\lib\imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "C:\Users\Jeff\miniconda3\envs\quant\lib\imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: DLL load failed while importing _pywrap_tensorflow_internal: The specified module could not be found.
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

First time post on github. trying to import tensorflow on local machine. keep getting a runtime error. please advise.
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No
TensorFlow installed from (source or binary): pip install tensorflow on miniconda3
TensorFlow version (use command below): 2.2.0
Python version: 3.8.5
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: cudnn-11.0-windows-x64-v<IP_ADDRESS>.zip   cuDNN8.0.4
GPU model and memory: NVIDIA Quadro K1100M.  This has compute capability 3.0 which is less than the minimum required... see https://www.tensorflow.org/install/gpu#hardware_requirements

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
tensorflow is not imported. instead i get a runtime warning... see warning below in Other info / logs
Describe the expected behavior
Standalone code to reproduce the issue
import tensorflow as tf
Other info / logs
ImportError Traceback (most recent call last)
~\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow.py in
57
---> 58 from tensorflow.python.pywrap_tensorflow_internal import *
59
~\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py in
27 return _mod
---> 28 _pywrap_tensorflow_internal = swig_import_helper()
29 del swig_import_helper
~\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py in swig_import_helper()
23 try:
---> 24 _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
25 finally:
~\miniconda3\envs\quant\lib\imp.py in load_module(name, file, filename, details)
241 else:
--> 242 return load_dynamic(name, filename, file)
243 elif type_ == PKG_DIRECTORY:
~\miniconda3\envs\quant\lib\imp.py in load_dynamic(name, path, file)
341 name=name, loader=loader, origin=path)
--> 342 return _load(spec)
343
ImportError: DLL load failed while importing _pywrap_tensorflow_internal: The specified module could not be found.
During handling of the above exception, another exception occurred:
ImportError Traceback (most recent call last)
in
----> 1 import tensorflow as tf
~\miniconda3\envs\quant\lib\site-packages\tensorflow__init__.py in
39 import sys as _sys
40
---> 41 from tensorflow.python.tools import module_util as _module_util
42 from tensorflow.python.util.lazy_loader import LazyLoader as _LazyLoader
43
~\miniconda3\envs\quant\lib\site-packages\tensorflow\python__init__.py in
48 import numpy as np
49
---> 50 from tensorflow.python import pywrap_tensorflow
51
52 # Protocol buffers
~\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow.py in
67 for some common reasons and solutions. Include the entire stack trace
68 above this error message when asking for help.""" % traceback.format_exc()
---> 69 raise ImportError(msg)
70
71 # pylint: enable=wildcard-import,g-import-not-at-top,unused-import,line-too-long
ImportError: Traceback (most recent call last):
File "C:\Users\Jeff\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\Jeff\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\Jeff\miniconda3\envs\quant\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "C:\Users\Jeff\miniconda3\envs\quant\lib\imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "C:\Users\Jeff\miniconda3\envs\quant\lib\imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: DLL load failed while importing _pywrap_tensorflow_internal: The specified module could not be found.
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions. Include the entire stack trace
above this error message when asking for help.

@DJJD17
What is make/model of your cpu?
I suspect your cpu model does not support AVX instructions sets.See hardware requirements
Make sure to download the latest microsoft visual c++ redistributable from here.
.Also, please follow the instructions from to install from Tensorflow website.
Please, check Your CPU/Python is on 32 bits?Please, refer #36167 and see if it helps you.Thanks!


checkAuth should throw an error after the redirect

At the moment, the checkAuth function just redirects, but does not actually return a rejected promise. This often seems to work fine, but it can lead to issues, because under certain conditions, the code continues to execute even after the redirect has started. For example, using requireAuth does not work properly in this case, because it checkAuth just returns normally, so it is assumed that the user is authenticated.
Even worse, this can lead to checkAuth being called again (resulting in a new login attempt), if that is required by specific routes. And even checkError might be called, because we a re not authenticated yet, but some routes might result in backend calls.
Combined, this can result in extremely nasty race conditions, where the login sometimes works, and sometimes doesn't, depending on when the browser decides to stop the execution of JavaScript when the redirect happens. This can also differ between browsers. I don't have any resources, but from my experience, Firefox tends to continue executing JavaScript for longer than Chrome, so the problems are more likely to manifest there.
Also, at the moment, checkAuth doesn't do anything if the redirectOnCheckAuth option is set to false.
In total, I would recommend to add a simple throw new Error() at the end of checkAuth, to handle both the problem after the redirect and the redirectOnCheckAuth === false case properly.

Thanks for the report!


settings: sdkPath/sdkVersion does not work

Issue Description
settings: sdkPath/sdkVersion does not work.
One workaround is to use global.json + "omnisharp.dotnetPath".
Steps to Reproduce
Use dotnet install from non-default location.
Set "omnisharp.sdkVersion", "omnisharp.sdkPath" correctly - OmniSharp fails to start.
Set "omnisharp.dotnetPath", "omnisharp.sdkVersion", "omnisharp.sdkPath" correctly - OmniSharp fails to start.
Expected Behavior
OmniSharp starts successfuly.
Actual Behavior
OmniSharp fails to start.
The highest version SDK (7.0.100-rc.2.22477.23) has some problems, but that is why I set sdkVersion/sdkPath.
Maybe OmniSharp server fails due to SDK defects, but I could not see why it do not starts with the specified version.
(The problem is that with sdk=7.0 on my machine, dotnet build etc. mystically fails, but dotnet <MSBuild.dll ...> succeeds.)
Logs
OmniSharp log

Starting OmniSharp server at 10/28/2022, 7:56:27 PM
    Target: [deleted]
OmniSharp server started with .NET 7.0.100-rc.2.22477.23
(/home/ulp/.dotnet).
Path: /home/ulp/.vscode/extensions/ms-dotnettools.csharp-1.25.0-linux-x64/.omnisharp/1.39.0-net6.0/OmniSharp.dll
PID: 3738114
    Starting OmniSharp on Unknown 0.0 (Unknown)

    Checking the 'DOTNET_ROOT' environment variable to find a .NET SDK

    DotNetPath set to /home/ulp/.dotnet/dotnet

    The Sdk version specified in the OmniSharp settings was not a valid semantic version. Configured version is ''6.0.402''. Please update your settings and restart OmniSharp.

    The Sdk path specified in the OmniSharp settings does not exist. Configured path is ''/home/ulp/.dotnet/sdk/6.0.402''. Please update your settings and restart OmniSharp.

    Located 0 MSBuild instance(s)

Could not locate MSBuild instance to register with OmniSharp.
[ERROR] Error: OmniSharp server load timed out. Use the 'omnisharp.projectLoadTimeout' setting to override the default delay (one minute).

### C# log ###
Post the output from Output-->C# here
Environment information
VSCode version: 1.72.2
C# Extension: 1.25.0
Mono Information
        OmniSharp using mono :6.12.0
Dotnet Information
.NET SDK:
 Version:   7.0.100-rc.2.22477.23
 Commit:    0a5360315a
Runtime Environment:
OS Name:     ubuntu
OS Version:  20.04
OS Platform: Linux
RID:         ubuntu.20.04-x64
Base Path:   /home/ulp/.dotnet/sdk/7.0.100-rc.2.22477.23/
Host:
Version:      7.0.0-rc.2.22472.3
Architecture: x64
Commit:       550605cc93
.NET SDKs installed:
6.0.402 [/home/ulp/.dotnet/sdk]
7.0.100-rc.2.22477.23 [/home/ulp/.dotnet/sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.10 [/home/ulp/.dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.0-rc.2.22476.2 [/home/ulp/.dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.30 [/home/ulp/.dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.10 [/home/ulp/.dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.0-rc.2.22472.3 [/home/ulp/.dotnet/shared/Microsoft.NETCore.App]
Other architectures found:
None
Environment variables:
DOTNET_ROOT       [/home/ulp/.dotnet]
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

Visual Studio Code Extensions

Extension
Author
Version

cmake
twxs
0.0.17

cmake-tools
ms-vscode
1.12.27

cpptools
ms-vscode
1.12.4

cpptools-extension-pack
ms-vscode
1.3.0

cpptools-themes
ms-vscode
2.0.0

csharp
ms-dotnettools
1.25.0

intellicode-api-usage-examples
VisualStudioExptTeam
0.2.6

java
redhat
1.12.0

jupyter
ms-toolsai
2022.9.1202862440

jupyter-keymap
ms-toolsai
1.0.0

jupyter-renderers
ms-toolsai
1.0.10

powershell
ms-vscode
2022.10.0

python
ms-python
2022.16.1

ruby
rebornix
0.28.1

vscode-back-forward-button
grimmer
0.1.6

vscode-fileutils
sleistner
3.5.0

vscode-java-debug
vscjava
0.45.0

vscode-java-dependency
vscjava
0.21.0

vscode-java-pack
vscjava
0.25.3

vscode-java-test
vscjava
0.37.1

vscode-jupyter-cell-tags
ms-toolsai
0.1.6

vscode-jupyter-slideshow
ms-toolsai
0.1.5

vscode-maven
vscjava
0.39.0

vscode-pylance
ms-python
2022.10.40

vscode-ruby
wingrunr21
0.28.0

vscodeintellicode
VisualStudioExptTeam
1.2.29

I suspect that because there is ' ' character surround both parameter during the process of omnisharp vscode parse the setting value and send to omnisharp host
Not sure is it because here?
https://github.com/OmniSharp/omnisharp-vscode/blob/66bbce261bf57d0c3ae506644010897ba871f03a/src/omnisharp/server.ts#L404-L412

confirmed. Just delete both ' make it work

Closing as resolved.


CI pipeline is not running Unit tests

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
CI pipeline is not running Unit tests.
To Reproduce
1 . Open a PULL request against the main branch.
2. See that tests do not run => Is clear when observing the ci.yml file
Code snippets
No response
OS
ANY
Python version
ANY
Library version
ANY

This is for security reasons – an OpenAI engineer must manually approve each contributor to allow tests to be run. I know it's surprising and disappointing, sorry about that.


[Do Not Merge] Creates Automating Subusers Page

Do Not Merge Yet
Please review this PR and page to make sure that it meets what you were looking for before any merger. Please let me know any and all changes that need to be made and I will be happy to do so. Opening this up as a PR just makes it easier to share my Commits/edits and contributions with you so we can collaborate.
I will remove the [Do Not Merge] tag from the title once I get the thumbs up from you. :+1:
Description of the change:
Per #4175, this PR creates a page called Automating Subusers and includes the curl references in proper Markdown, stylized for Github.
Closes #4175

Hi @Whatthefoxsays! You're awesome to review all of this for me, thank you! I'll get on making those changes in the next few days. :+1:

Hello @crweiner,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Thank you again @Whatthefoxsays and @ksigler7 for reviewing this! Really, I greatly do appreciate your time and help!!
I just wanted to make sure of one thing that popped up,  does the Username field at Line 489 here return the email address in question, or is it actually the username from Line 38 here ? I just want to make sure that I have that correctly.
Also, this is the first time I've tried to use the Review feature of Github, so if I did something wrong, please let me know so I can correct it!

@crweiner - lgtm! you have the username correctly. Thank you SO much for all of your effort on this!!


Maually esiting profile / understanding profile

Hi,
Referring the picture below. Can you tell me how to retrieve key from tuya clud api?

What is address finish, address ssid, ssid padding and how to retrive the same?

This is outside the scope of CloudCutter.
CloudCutter includes the list of DPIDs in the schema (the id fields), but doesn't determine how to use them.  Beyond that you'll need to use external tools such as LTChipTool's UPK2ESPHome Schema Download using the key from the CloudCutter profile as either the firmware key or product key.

Okay.
What about the adrress finish, address ssid, ssid padding?
Trying to understand it to find a workaround for devices using esp based tuya ic.
I myself happen to have lots of them.
Tuya convert is able to detect them but unable to exploit.

What about the adrress finish, address ssid, ssid padding?

Those are for CloudCutter exploiting, and are not a part of device configuration.


Add more code coverage for OAuthClient

Improved test code coverage
Changes
Updated OAuthClientTest include a test case for makeApiCall with parameters, and updated test for generateOauth1Sign to be more precise.

Notes
I have another set of changes coming with more coverage for OAuthClientTest1

Coverage increased (+0.8%) to 88.063% when pulling e4a1c2ff63dfb635746f0a2edb6a203650fd548a on oscargrp:add_more_code_coverage into 73aa0dd7375fb8c12b5293b4ed434b49f700af38 on intuit:develop.

Coverage increased (+0.8%) to 88.063% when pulling e4a1c2ff63dfb635746f0a2edb6a203650fd548a on oscargrp:add_more_code_coverage into 73aa0dd7375fb8c12b5293b4ed434b49f700af38 on intuit:develop.

@oscargrp: Thanks

🚀 PR was released in v2.0.0  hacktoberfest


Update brew for release-0.21.2

Issue #, if available:
Description of changes:
Update brew for release-0.21.2
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/meow

/moo

/retest

/cherry-pick release-0.21


fix: audit log bug

https://github.com/StarRocks/starrocks/issues/53786

 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.pengliang3 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.

@lldlhd PR already reaised for this issue. duplicate of #53697

Writing it in ConnectProcessor is better than writing it in StmtExecutor


Error when user deactivates their account.

The following SQL error occurs when a user deactivates their account. It appears that a new user record is being inserted with the same credentials and failing on the duplicate id.
SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '1' for key 'PRIMARY'

Looking at the onDeactivate() method in components/Account.php, the issue seems to occur when the user is soft deleted then logged out. T
Auth::logout();

This in turn calls October\Rain\Auth\Manager->logout() where the forceSave() method is called.
/**
     * Logs the current user out.
     */
    public function logout()
    {
        if ($this->user) {
            $this->user->setRememberToken(null);
            $this->user->forceSave();
        }

        $this->user = null;

        Session::forget($this->sessionKey);
        Cookie::queue(Cookie::forget($this->sessionKey));
    }

Stack trace attached
stack.txt

Can confirm I have encountered this bug as well, is there a way to get around it?
My steps that reproduce it are;

create new account, get redirected back to the same page
now deactivate this account just created.

Bug: error thrown 'Integrity constraint violation'

@sgreener @pmhou Please confirm that #245 fixes this for you


Test Status in fileScreenings overview

The appartent mingling in the downloaded Observations table was due to an old file (ESAS_BE_DATA_test_obis_tab_ERID.csv) that was in the database, instead of the most recent upload (ESAS_BE_DATA_test_obis_tab.cs), both with the same campaignID. Very confusing because in the overview of file screenings it seems to suggest that the latter file (88) was uploaded to the database, hereby overwriting the former file (86):

[ ] So concluding: it remains to be tested whether the status "File already uploaded to the database" actually holds true.

Originally posted by @nicolasvanermen in https://github.com/ices-tools-dev/esas/issues/88#issuecomment-1161421056

The appartent mingling in the downloaded Observations table was due to an old file (ESAS_BE_DATA_test_obis_tab_ERID.csv) that was in the database, instead of the most recent upload (ESAS_BE_DATA_test_obis_tab.cs), both with the same campaignID. Very confusing because in the overview of file screenings it seems to suggest that the latter file (88) was uploaded to the database, hereby overwriting the former file (86):

* [ ]  So concluding: it remains to be tested whether the status "File already uploaded to the database" actually holds true.

Files were uploaded to the database, but deleted after!

OK, this could explain this but hard for me to check. The only confusing thing remaining was the reported submission date, that was set on 17/06 while the false file (screening 86) was pushed to the DB at 8/06.

[ ] To be tested!

@cmspinto: comparable problem now, see my file screening 133. File was submitted (resubmission of file with screening 130), file status says that file has been uploaded, but it is not in the download... The other resubmission (screening 132) did make it in the download. What am I missing?

Nicolas,
You are the DataRightHolder to organization 202 but not for Organization 630.

I have not implemented yet what we talked in the morning. I need to look carefully before changing the upload permission logic. Does this make sense?

OK Carlos, but the problem is that the list of file screenings does not notify me that there was no actual resubmission. It simply says: "File already uploaded to the database". This is also the subject of this issue.
Did you change it now, because I tried it again an now both resubmissions are in the download...??

SO to be clear: I screen a file, it says there are no warnings, it says it is a resubmission, I opt to resubmit, it says data are downloaded to the database, in the fileScreenings overview it says the file has been uploaded... Only: the file is not in the database!

Yes, have changed it now to always allow a user to overwrite their own files!

Both resubmissions are now in the list of the submissions; just a few questions:

[ ] clicking on the numbers gives an empty page, what is supposed to be there? (@cmspinto )
[ ] files that are resubmitted are deleted from this list, is that OK? (@peterdesmet)
[ ] why do the submission ID's jump from 131 to 135 to 140, I did not delete any file after the meeting? (@cmspinto )

why do the submission ID's jump from 131 to 135 to 140, I did not delete any file after the meeting? (@cmspinto )

Because I tested some files and deleted them immediately after!

ESAS_BE_DATA_test_resubmission_Carlos.csv
Or could you upload this file?

[ ] DRH NIOZ, uploaded by Carlos, so I shouldn't be able to overwrite it, and the application should warn me thereof

Done, I can see both:

I'm 630 DataRightHolder, so I can see, overwrite or delete your file.

Cleared this out during chat with Carlos: in case the file includes campaigns for which the submitter has no rights to overwrite them, there will be a warning and the submitter will not be able to upload the file (even if this file also includes brand new campaignID's!).

[ ] test warning pop-up

OK I tested this as well and it worked fine. Would be nice if the warning simply stated:

[ ] "You do not have permission to overwrite the campaign(s) in the database"


fix ability for guarded models to be filled with default fields

…as id, created_at, and updated_at

Set exists to true too. So saving it won't give integrity constraint error

When newInstance is called, exists is set to true because of the second parameter

Thanks @dallincoons


Python unit tests

Built on top of #2175.
Test generated Python clients with nox.
Test generated unit tests, and skip testing system tests (#2205 will set up credentials for system tests) for later.
This only tests PubSub and Speech, because Logging generation is broken by #2209

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2177   +/-   ##
=========================================
  Coverage     86.86%   86.86%
  Complexity     5139     5139
=========================================
  Files           456      456
  Lines         20385    20385
  Branches       2184     2184
=========================================
  Hits          17707    17707
  Misses         1908     1908
  Partials        770      770

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 866d38b...2858bed. Read the comment docs.

Blocked on #2197, #2160 which break the generated python client :(

PTAL

Ok, only running tests for a language if the language's client generation succeeded. There will be calls to artman to (try to) generate all language clients, without short circuiting if any generation fails.
PTAL

Ok, changed gapic_smoketest.py to generate_clients.py
PTAL


Trigger automatic actions defined by auction stages in delegate server

This task requires to introduce polling logic placing in queue HydraEvents  which are triggered by:

DelegateEvent Start
DelegateEvent (AuctionStageStarted ...)

Actual implementation is out of scope of this task.
Blocked by #140

@mlabs-haskell/hydra-auction Given that our consumer is running non-concurrently we need a way to handle one event a time.
Previously we discussed, that we may not just put everything in queue concurrently - cuz client requests were state-dependent => should use DelegateServerT, which complicates everything. But now it seems like all input producers are stateless.
Theoretically we could return back to concurrent threads. While I am not sure that this is required, cuz it should work just with single threaded main calling bunch of producers and
The only thing that definitely should be changed - is adding queue. While I guess now we could just use single DelegateInput queue.


Polybar not displaying text color properly

I have never had a problem with this config until i reinstalled polybar 2 days ago now it wont display text color properly

This is a known issue as reported in #666 and #639.
This does not occur in the latest stable release 3.0.5, so downgrading to that will fix this

Thank you that fixed it

This issue will be tracked in #639


Build and use Fladle plugin together with Flank

Author the user story for this feature
As a Developer, I want to use Fladle together with Flank using composite build so I can develop plugin even better.
Describe the solution

Setup Fladle as git submodule
Use a composite build to substitute the flank dependency.
Add an integration test for the fladle plugin in the flank project. This is a good idea whether or not we are going to integrate both projects to prevent regressions.

Resources

Investigation
composite builds
Composite Builds Declared Substitutions
Composite Builds Plugin Development

do we need the submodule for a composite build?

We need to substitute in somehow with providing path to module and to make it more automated submodules seems to be good solution

Closing this ticket as #1963 #1964 were created.


feat: add config option to exclude modules from native yaml check

Fixes: #228

Codecov Report

Merging #236 (4ab4fde) into main (dde7eae) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #236   +/-   ##
=======================================
  Coverage   18.10%   18.10%
=======================================
  Files           9        9
  Lines        1077     1077
=======================================
  Hits          195      195
  Misses        882      882

Impacted Files
Coverage Δ

ansiblelater/settings.py
83.15% <ø> (ø)

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 dde7eae...4ab4fde. Read the comment docs.


Regarding backward implementation in CrossScan and CrossMerge

Dear Authors,
Thank you for your work and for releasing your code. I have a question regarding the CrossScan and CrossMerge classes. The forward functions seem clear to (the former produces the 4 sequences to perform selective scan and the later merges them afterwards). But I am not sure what is exactly being done in the backwards function?
I would be grateful if you could please explain those in a bit more detail.
Kind regards

It just follow the rule that "c=a+b => da = dc; db=dc;". That is: the backward function of  CrossScan is CrossMerge, and the backward of CrossMerge is CrossScan.


Override LdapUserBase "Property set method not found"

Hello, while trying to override LdapUserBase like documented:
  public sealed class CustomLdapUser : LdapUserBase
  {
      [LdapAttribute(Schema.ActiveDirectory, "displayName")]
      [Claim(CustomClaimTypes.Nome)]
      public override string DisplayName => base.DisplayName;
   }

I get:
  {"Property set method not found."}
   at System.Reflection.RuntimePropertyInfo.SetValue(Object obj, Object value, BindingFlags invokeAttr, Binder binder, Object[] index, CultureInfo culture)
   at Visus.LdapAuthentication.LdapUserBase.Assign(LdapEntry entry, LdapConnection connection, ILdapOptions options)
   at Visus.LdapAuthentication.LdapAuthenticationService`1.Login(String username, String password)

I'm currently using
  [LdapAttribute(Schema.ActiveDirectory, "displayName")]
  [Claim(CustomClaimTypes.Nome)]
  public string DisplayName { get; internal set; }

This correctly maps the claims I want but obviously I'm not able to properly access the property ldapUser.DisplayName (it's value is null)

I was able to repro the issue and searching for a fix.

Fixed in Nuget package 1.9.0


The 2decomp module is too big

EVERYTHING lives in the 2decomp module, these should be factored out into modules that the 2decomp module can then use.
As an example, anything that is #include'd (e.g. transposes, halo) would seem a natural fit to be its own module.
Likewise there are related things (decomp2d_abort, decomp2d_warn) that could live in a decomp2d_messages module.
This will be a breaking change for codes using 2decomp, however it should be a relatively simple fix to just go through and fix compiler errors, without any change in functionality.

This is related to https://github.com/xcompact3d/2decomp-fft/issues/6. But not a duplicate because the other one is more generic.

Are you planning to split 2decomp into multiple smaller modules. Or have many submodules and a 2decomp module mostly limited to the declaration of interfaces ?

Ah I'd overlooked #6.
I think a combination of the two, currently decomp_2d.f90 is >2000 lines long (discarding #include'd files) making it very difficult to work with/change.
So as an example of how the two might work together:

move decomp2d_abort, decomp2d_warn -> decomp2d_messages module
move all transpose interfaces into transpose module
-- Implementations of transposes would be submodules of this (essentially transpose_x_to_y.f90 would be a submodule that extends transpose.f90 with the actual x->y implementation)

Does that sound sensible? Other views would be appreciated

I think this would also tie in with @rfj82982 desire to have decomp2d without MPI eventually

Also, to avoid a "mega commit" I think it will be better to periodically merge this branch into main - the first case might help clarify the plan

I think this is a good idea. But this move should occur after we release a clean version of the library with backward-compatibility.


it seems too many uncertain for sha256sums.gpg for new user

Please describe the question or issue you're facing with "Ubuntu WSL documentation".
when i clicked the file of "sha256sums.gpg", the browser of brave showed the link is not https and possibly be tamper with. I also have not seen ubuntu listed several optional tool to verify it in windows10. this likes ubuntu also didn't list several option in burn iso software.
i saw a website suggest using ventoy1.0.98 to install unbuntu24.04. is it stable to use?
The most important for me is how do I verify the sha256sums.gpg, ubuntu hasn't shown it or i don't understand https://ubuntu.com/tutorials/how-to-verify-ubuntu#4-retrieve-the-correct-signature-key
then i try several way to verify it, but all show me problems. the attachment is these result

Reported from: https://canonical-ubuntu-wsl.readthedocs-hosted.com/en/latest/

Thanks for your report. It is not a problem with Ubuntu WSL but a general support question. You can ask for support from Launchpad https://answers.launchpad.net/ubuntu
Do no hesitate to report any bug you may find.


Proxy support in tokio migration

Adding a git dependency of a hyper_proxy version with tokio 1.0 support. It is not yet merged, and as far as I see there was no reaction of the maintainer: tafia/hyper-proxy#18.

@Johannesd3 While you are at it, do you think you could place this behind a feature flag?
(from back then https://github.com/librespot-org/librespot/pull/191#issuecomment-424040306)

@Johannesd3 While you are at it, do you think you could place this behind a feature flag?
(from back then https://github.com/librespot-org/librespot/pull/191#issuecomment-424040306)

Yes, sure. But while we're at it, would you please explain me why hyper_proxy and proxytunnel.rs are necessary?

Yes, sure. But while we're at it, would you please explain me why hyper_proxy and proxytunnel.rs are necessary?

Most crates are in our dependency tree anyway. I'm more worried that hyper_proxy is no longer actively maintained.

Most crates are in our dependency tree anyway. I'm more worried that hyper_proxy is no longer actively maintained.

Good question, but from a quick glance, the AP resolution stuff could also be handled by adapting proxytunnel.rs -- @janderholm would be the right person to answer..

Good question, but from a quick glance, the AP resolution stuff could also be handled by adapting proxytunnel.rs -- @janderholm would be the right person to answer..

If I recall, it was quite a long time ago, proxy servers are usually configured to only support CONNECT if the destination port is 443. Any normal HTTP request such as apresolve need to implement the full proxy standard.
If it's possible to resolve through port 443 then the proxy tunnel could probably be used. The server is going to need to support CONNECT anyway so requiring it for resolve as well is reasonable.

If I recall, it was quite a long time ago, proxy servers are usually configured to only support CONNECT if the destination port is 443. Any normal HTTP request such as apresolve need to implement the full proxy standard.
If it's possible to resolve through port 443 then the proxy tunnel could probably be used. The server is going to need to support CONNECT anyway so requiring it for resolve as well is reasonable.

HTTPS://apresolve.spotify.com/ seems to work, so is there any reason not to use proxy tunnel?

HTTPS://apresolve.spotify.com/ seems to work, so is there any reason not to use proxy tunnel?

In my opinion, if you manage to hook it up, go for it :-)

In my opinion, if you manage to hook it up, go for it :-)

Wait, is hyper used only for apresolve? We could replace it with reqwest which supports proxies.

Wait, is hyper used only for apresolve? We could replace it with reqwest which supports proxies.

@Johannesd3 no, we use hyper for discovery as well (but as a server)

@Johannesd3 no, we use hyper for discovery as well (but as a server)

Reqwest uses hyper internally, so probably it won't add many dependencies even if we keep hyper. So what do you think?

Reqwest uses hyper internally, so probably it won't add many dependencies even if we keep hyper. So what do you think?

Reqwest uses hyper internally, so probably it won't add many dependencies even if we keep hyper. So what do you think?

Personally, I'd prefer not to pull in reqwest just to handle this AP query..
If we can't use the existing proxytunnel.rs a simple(r) option would be to just always use the fallback AP when using a proxy?

Reqwest uses hyper internally, so probably it won't add many dependencies even if we keep hyper. So what do you think?

Personally, I'd prefer not to pull in reqwest just to handle this AP query..
If we can't use the existing proxytunnel.rs a simple(r) option would be to just always use the fallback AP when using a proxy?

It's already odd to use hyper for this simple query. There are at least some projects that rely on core but not on connect (like ncspot) and I really wonder how much it would decrease compile time if hyper was no dependency.
Using reqwest it could be something like this:
async fn request() -> reqwest::Result<APResolveData> {
    Client::new()
        .get(APRESOLVE_ENDPOINT)
        .send().await?
        .json().await
}

reqwest uses the system proxy which is given by the http_proxy and https_proxy env vars by default. I think this is really tempting, but this decision is not up to me.
We must implement this trait for proxytunnel. It doesn't seem to hard, I will take a look at it.

It's already odd to use hyper for this simple query. There are at least some projects that rely on core but not on connect (like ncspot) and I really wonder how much it would decrease compile time if hyper was no dependency.
Using reqwest it could be something like this:
async fn request() -> reqwest::Result<APResolveData> {
    Client::new()
        .get(APRESOLVE_ENDPOINT)
        .send().await?
        .json().await
}

reqwest uses the system proxy which is given by the http_proxy and https_proxy env vars by default. I think this is really tempting, but this decision is not up to me.
We must implement this trait for proxytunnel. It doesn't seem to hard, I will take a look at it.

https://apresolve.spotify.com/ requires a TLS library, which is a whole other can of worms.
I’d say using reqwest saves us a lot of trouble. Actually I’ve even been thinking of opening an issue on reqwest to split out the proxy code (or at least make it public). Then we can replace proxytunnel.rs, which, unlike reqwest, does not support https or socks proxies.
If people actually care about build times / binary size that much, we can make apresolve (always use the fallback) and proxy support optional. Then all the hyper/reqwest depencies go away (obviously discovery still needs hyper).  I’d rather not until someone actually complains about it though.
——
I don’t know how I feel about reqwest using environment variables when we also have a flag for it. Maybe if proxytunnel is replaced by reqwest’s proxy implementation then we can start using the environment variables and deprecate the flag. Until then we should tell reqwest to use a proxy if and only if the flag is set.

https://apresolve.spotify.com/ requires a TLS library, which is a whole other can of worms.
I’d say using reqwest saves us a lot of trouble. Actually I’ve even been thinking of opening an issue on reqwest to split out the proxy code (or at least make it public). Then we can replace proxytunnel.rs, which, unlike reqwest, does not support https or socks proxies.
If people actually care about build times / binary size that much, we can make apresolve (always use the fallback) and proxy support optional. Then all the hyper/reqwest depencies go away (obviously discovery still needs hyper).  I’d rather not until someone actually complains about it though.
——
I don’t know how I feel about reqwest using environment variables when we also have a flag for it. Maybe if proxytunnel is replaced by reqwest’s proxy implementation then we can start using the environment variables and deprecate the flag. Until then we should tell reqwest to use a proxy if and only if the flag is set.

Apresolve should work now with proxytunnel, but I'm not sure about the problems @janderholm mentions.

Apresolve should work now with proxytunnel, but I'm not sure about the problems @janderholm mentions.

I thought extern crate was more or less phased out in rust 2018

@ashthespy We could merge this, and think about using hyper_proxy or reqwest later, or what do you think? At least it works now (I think), which was previously not the case.

@Johannesd3 Don't have the time to test this now, but if you are happy with it, will merge in..

Yes, I would prefer this. Testing with a local proxy server instance worked, and after all it's just a WIP branch...


[Enhancement Request] Versioning Scheme

What would you like to be added:
I propose we should come up with a good package versioning scheme. This will create clearance and guidance around many other aspects in KUDO including e.g. dependency management, version policies, version lifecycle and for a compatibility matrix. It also will touch how the official kudobuilder/frameworks registry will look like and how other tools like the KUDO cli will manage KUDO applications.
One way I propose is coming from a familiar friend. Borrowing from here:
Packages are versioned with an a.b.c-x.y.z format, where a.b.c is the version of the KUDO and x.y.z indicates the version of the Base Technology, e.g. Kafka. For example, 0.1.0-2.11-2.4.0 indicates version 1.10.0 of the DC/OS integration and version 2.11-2.4.0 of Kafka.
We should not offer support services for the base technology (e.g. Kafka). The base technology version is denoted as the second version in the package number (e.g. 1.2.3-4.5.6).
And with inspiration from here:

Major Versions (X.y.z) are releases for providing major and minor features and improvements or optimizations to existing features. They incorporate all applicable bug fixes made in earlier Major, Minor, and Maintenance versions.

Minor Version (x.Y.z) are versions for delivering minor features, improvements or optimizations to existing features, and bug fixes. They incorporate all applicable bug fixes made in earlier Minor versions, and Maintenance versions.

Maintenance Version (x.y.Z) are versions for providing bug fixes that are highly impactful to a number of users and who cannot wait for the next Major or Minor version. They incorporate all applicable bug fixes made in prior Maintenance versions.

“End of Life (EOL)” versions are no longer supported by KUDO, upgrading to a later version is highly recommended.

Why is this needed:
Versioning is hard and there are many ways this can be done. Other KEPs like KEP-0006 for stable Kafka ( https://github.com/kudobuilder/kudo/pull/157 ) but also KEP-0003 CLI depend on it in my opinion. This should also go along with deciding on specific naming conventions, which could be another ticket or discussed here as well.

What is a Package?
What is a Framework?
What is an Application?

Getting guidance around this terms, adding them to concepts and so on will help also when looking into designs for kudobuilder/frameworks and/or dependency models but also the actual versioning. E.g. do we have a Kafka framework or package? What is the difference? This should not be just answered in the FAQ ( https://github.com/kudobuilder/kudo/pull/111 ) but also manifested under concepts. There are also plenty more questions that need to be answered that way. For Example:

How will all this under the light of dynamic CRDs change or not?
Will the CLI come with a core binary similar to the DC/OS CLI and then have its specific Kafka or Flink cli functionality in a binary package zip file that entails the specific Kafka/Flink CLI for KUDO that is pluggable?
Etc..

Thanks @fabianbaier. This is a great start.
I actually think we should only release with base version of the technology. KUDO is unlike the SDK in that there are no libraries against which these base frameworks are written.
There is something else here that needs to be versioned, though, and it is the CRD that users are using. The only time that these should have revisions are when the public-facing API changes (in this case, the spec field for each given CRD/instance).

It looks like @fabianbaier had similar ideas to what we were discussing over at https://github.com/kudobuilder/kudo/pull/889.
@zen-dog did you make any progress with this?
@gerred this is one of the issues I'd consider "blockers". Should we raise the priority?

@mpereira We may not be able to block on this -- it's a huge story that changes the way this is all currently implemented, and we don't even have the start to it scoping. So we may have to ship GA as-is, and then plan a migration story (this shouldn't be a problem, we can account for this). My current understanding is we aren't worrying about patch releases for minor versions, since none have came out and unless we're planning on backporting versions, this won't be a problem until the next minor release with an N-1 patch release. wdyt?

To further clarify, I think what we have now works, albeit is a little "weird", is an unsolved problem in even Linux distributions, and we should get some usage to understand the exact semantics we want since there are so many versions installed. Nothing competing does this right now as well.

Instead, we can guarantee an upgrade path from current versioning scheme to new versioning scheme, if that helps. It'll just be really difficult to invent an entirely new versioning scheme, refactor our existing packages, break all of the operators again, etc., in the weeks coming up to GA when we can safely and progressively do this work without impact now.

@gerred fair. We should create a KEP for it.


feat: chainsync blocking

This PR wires in the blocker component to the chainsyncer so that chain-syncing becomes mandatory.

This change is

Codecov Report

Merging #2585 (d5b8713) into master (1f87bee) will increase coverage by 0.00%.
The diff coverage is 100.00%.

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


[MQ] Tool to generate live test resources

Have a standard tool across languages to generate the resources for live tests on command, before the tests actually run. This will probably be in PowerShell and bash, have a NPM script as part of our package.jsons, and use the Azure CLI to authenticate, as well as set the environment variables before running the recording tests.
The Engineering team is already working on this, but we can help. Ask Daniel Jurek.

I'll close this on favor of this epic: https://github.com/Azure/azure-sdk-for-js/issues/6647


Revert "Export package for asm based secret store implementation." MERGEOK

Reverts vespa-engine/vespa#32461

@gjoranv FYI


Client Allocation Request for: Jim

Client Allocation Questions
Core Information

Name: Jim
Website / Social Media: oanode.com, (@apojii, Twitter)
Region: Greater China
DataCap Requested: 1 TiB
Addresses to be Notarized: f1jjtc2tdy7svnsaqcgxlauta455iwqjbbjiulddi
Notary Requested: Neo Ge

This user’s identity has been verified through filplus.storage


Can't modify frozen hash on model destroy

Having trouble deleting models with attachments on them--It just fails with the following trace:
   (0.3ms)  BEGIN
  SQL (0.4ms)  DELETE FROM "app_revisions" WHERE "app_revisions"."id" = $1  [["id", 8]]
   (0.3ms)  ROLLBACK
RuntimeError: Can't modify frozen hash
	from /Users/adam/.rvm/gems/ruby-2.3.1/gems/activerecord-<IP_ADDRESS>/lib/active_record/attribute_set/builder.rb:45:in `[]='
	from /Users/adam/.rvm/gems/ruby-2.3.1/gems/activerecord-<IP_ADDRESS>/lib/active_record/attribute_set.rb:51:in `write_from_user'
	from /Users/adam/.rvm/gems/ruby-2.3.1/gems/activerecord-<IP_ADDRESS>/lib/active_record/attribute_methods/write.rb:50:in `write_attribute_with_type_cast'
	from /Users/adam/.rvm/gems/ruby-2.3.1/gems/activerecord-<IP_ADDRESS>/lib/active_record/attribute_methods/write.rb:32:in `write_attribute'
	from /Users/adam/.rvm/gems/ruby-2.3.1/gems/activerecord-<IP_ADDRESS>/lib/active_record/attribute_methods/write.rb:20:in `__temp__6696c656f59646='
	from /Users/adam/.rvm/gems/ruby-2.3.1/gems/refile-0.2.5/lib/refile/attachment.rb:23:in `id='
	from /Users/adam/.rvm/gems/ruby-2.3.1/gems/refile-0.2.5/lib/refile/attachment.rb:64:in `delete!'
	from /Users/adam/.rvm/gems/ruby-2.3.1/gems/refile-0.2.5/lib/refile/attachment/active_record.rb:21:in `block in attachment'

Is this a compatibility issue with Rails 5 maybe?  Looks like it's trying to write a temporary attribute after the model has been destroyed?
Simplified model is as follows:
class AppRevision < ApplicationRecord
  validates_presence_of :app, :file, :version

  belongs_to :app

  attachment :file
end

Would appreciate any help or insight!

Were you able to fix this?

No.  I haven't really looked at it lately--the project I'm using refile for has been on my back burner for a bit...

Likely outdated refile (0.2.5 as your backtrace).
Give it a shot on latest release or even with the master, if you can replicate that, feel free to reopen the issue, thanks! <3


v1.0.0-beta.15 won't run without installing glob module

$ docker run --rm node:lts-alpine npx<EMAIL_ADDRESS>--version
npx: installed 86 in 9.634s
1.0.0-beta.14
$ docker run --rm node:lts-alpine npx<EMAIL_ADDRESS>--version
npx: installed 89 in 10.73s
Cannot find module 'glob'
Require stack:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>
I'm not too well-versed in npx, but this should work without manually installing dependencies, shouldn't it?

Thanks for your bug report! It was fixed in #215 and released as a v1.0.0-beta.16.


Bot randomly has stopped working

The bot was working fine until someone added another song, and it randomly spit this error, and now it won't do anything but return this error... I tried updating youtube_dl, nothing changed. Tried redownloading the bot and nothing changed.
Error:
ERROR: Signature extraction failed: Traceback (most recent call last):
  File "C:\Users\FineNerds\AppData\Local\Programs\Python\Python35-32\lib\site-packages\youtube_dl\extractor\youtube.py", line 997, in _decrypt_signature
    video_id, player_url, s
  File "C:\Users\FineNerds\AppData\Local\Programs\Python\Python35-32\lib\site-packages\youtube_dl\extractor\youtube.py", line 912, in _extract_signature_function
    res = self._parse_sig_js(code)
  File "C:\Users\FineNerds\AppData\Local\Programs\Python\Python35-32\lib\site-packages\youtube_dl\extractor\youtube.py", line 975, in _parse_sig_js
    initial_function = jsi.extract_function(funcname)
  File "C:\Users\FineNerds\AppData\Local\Programs\Python\Python35-32\lib\site-packages\youtube_dl\jsinterp.py", line 241, in extract_function
    raise ExtractorError('Could not find JS function %r' % funcname)
youtube_dl.utils.ExtractorError: Could not find JS function 'ur'
 (caused by ExtractorError("Could not find JS function 'ur'",))

Same here.

Getting essentially the same error as well.

Youtube changed something and youtube-dl needs to be updated to fix it.

You can try manually updating youtube-dl by downloading its source then pasting it into the Lib\site-packages. It worked for me.

^ Please no one do this.  Update youtube-dl properly, run update-deps.bat, or pip install --upgrade youtube-dl

same error as well, i have no idea how to update youtube-dl despite the instructions

run update_deps.bat
On Thu, Jun 23, 2016 at 12:12 AM, yoczed<EMAIL_ADDRESS>wrote:

same error as well, i have no idea how to update youtube-dl despite the
instructions
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SexualRhinoceros/MusicBot/issues/365#issuecomment-227945329,
or mute the thread
https://github.com/notifications/unsubscribe/AFSg4Jb3kxYLM4s7jQqL-LmltAEOBcAQks5qOge-gaJpZM4I8ZXR
.


auto size of the legend if width or heigth is less than 0

only compatible with external legend for the moment...

Thank you very much for your pull request, unfortunately we will not merge this to the branch.
The given width within the config has priority otherwise it obtains the width of it's parent container. Ensure your legend container (divId) does not fulfill the whole space and has a fixed width.


Selection sort added

Selection sort algorithm implemented in c++

@the-code-meister : It would be great if you can add the description along with the issue number.
Feel free to use our contribution guidelines (https://github.com/fnplus/interview-techdev-guide/blob/master/CONTRIBUTING.md#how-to-contribute-an-implementation-code) for reference

Closes #329

@the-code-meister File format need to be changed.
Do not use .md for your algorithms

Thanks for contributing!


assetmgr: asset reflection properties

allow adding properties to asset wrapper objects
classes which subclass Asset
This would be useful for modifying procedural or file based assets.
Issues:
should references to an asset should hold the properties? - or the flyweight ? maybe both...

done.
https://github.com/tweakoz/orkid/commit/e9e6b3657810d46e0ad4efc317d2ce7565cb8b94


Implementing view_as

issue #629

GREAT first PR! Don't forget to add integration and unit tests to close issue #629


Update form to enable JSON Payload Type to store files in a blobstore service.

Need to be able to save files to blobstore service and then send URL's in payload.
Depandant: https://github.com/typerefinery-ai/typerefinery/issues/459

Save the file to the blob store and pass the result to the Endpoint.

[ ] Create a structure for putting the files. (/path_to_form/GUID/file_name.mimetype)
[ ] Click on submit should wait until the files are uploaded.
[ ] When the files are uploading, some kind of indication or progress bar for the files should be visible and then once the files are upload, then link to view the file should be added.
[ ] Allow drag and drop option accept files.
[ ] Allow option to remove the files added.

Update the flow to save file payload to the blob store

Completed


Don't set User-Agent header by default

Updated from #540

@seanmonstar just to make sure, did you see https://github.com/seanmonstar/reqwest/pull/540#discussion_r312740784 and your are not doing it here on purpose? Not a major point, just asking...

@lucab I don't quite understand the comment actually


instant-ngp-bounded method fail before Setting up evaluation dataset... and freeze

my environment
wsl2 + RTX3050 laptop
nerfacc                   0.5.2
nerfstudio                0.3.4
torch                     2.0.1+cu118
torchvision               0.15.2+cu118
Describe the bug
when running the command

ns-train instant-ngp-bounded --data data/poster

nerfstudio stuck at

Setting up training dataset...
Caching all 204 images.

Nerfstudio gets stuck, and experiences a significant increase in memory usage, eventually leading to a Linux OOM error. The process get killed.
(The RAM assigned for wsl is more than 8GB and is complete enough for default nerfacto method.)
Besides, default port of ns-viewer would appear Renderer disconnected.
It's worth mentioning that the first try for instant-ngp-training succeeded. Described in To Reproduce.
To Reproduce
Steps to reproduce the behavior:

first train with ns-train instant-ngp-bounded succeeded.

Then I ran ns-viewer --load-config, and the viewer failed to load. The viewer got stuck at

Setting up training dataset...
Caching all 204 images.

Maybe that have something todo with https://github.com/nerfstudio-project/nerfstudio/issues/1835#issuecomment-1527983032 , I'm not sure. Anyway, after waiting for about an hour I ctrl+c ended that.

Then each time I try ns-train instant-ngp-bounded I meet this error.

Also, when I try ns-viewer for nerfacto configs, the default port appears Renderer disconnected. But after changing the port the ns-viewer could function well.

Such error appears on multiple datasets.

Expected behavior
after the output

Setting up training dataset...
Caching all 204 images.

it should be

Setting up evaluation dataset...
Caching all 22 images.

Screenshots

nerfacc is likely failing silently. Can you try reinstalling nerfacc. https://github.com/KAIR-BAIR/nerfacc

@tancik I updated to the new version of nerfstudio. And I'm working on implementing a new Model Class where I use the VolumetricSampler and pass the nerfacc occupancy grid estimator.
When I run the following snippet, my code freezes.
self.occupancy_grid = nerfacc.OccGridEstimator(
            roi_aabb=self.scene_aabb,
            resolution=self.config.grid_resolution,
            levels=self.config.grid_levels,
        )

self.sampler = VolumetricSampler(
            occupancy_grid=self.occupancy_grid, density_fn=self.field.density_fn
        )
....
 with torch.no_grad():
              ray_samples, ray_indices = self.sampler(
                  ray_bundle=ray_bundle,
                  near_plane=self.config.min_near,
                  far_plane=self.config.min_far,
                  render_step_size=self.config.render_step_size,
                  alpha_thre=self.config.alpha_thre,
                  cone_angle=self.config.cone_angle,
              )

I have tried uninstalling and installing nerfacc but that hasn't helped. If you have any advice or guidance, that would be great!

After some digging, I have found that the code gets stuck at this ray_aabb_intersect of nerfacc (nerfacc/grid.py)
t_mins, t_maxs, hits = _C.ray_aabb_intersect(
        rays_o.contiguous(),
        rays_d.contiguous(),
        aabbs.contiguous(),
        near_plane,
        far_plane,
        miss_value,
    )


[api-docs] Fix broken link in Gateways

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/1c1641b0be45313a9ad8d87ad3ed07f796b75b3c

This commit will not effect our lib's code. No changes required.


Better support for angular filters

I couldn't really get mox to mock my angular custom filter. It would be nice if it was supported as a top level method next to mockServices, mockDirectives, mockControllers etc: mockFilters.
This is the code that I currently use to make it work:
    module(function ($provide) {
      $provide.value('myFltrFilter', function (input) {
        return input;
      });
    });

    mox
      .module('myModule')
      .mockServices('$state', 'SomeService')
      .setupResults(function () {
        return {
          SomeService: {}
        };
      })
      .run();

I did try to mock my filter with mox in various ways, since it is suggested in the documentation, but I could not get it to work.. if it is supported, can I get a clear example of how to do it?
I tried for instance:
mox.mockServices('myFltr')
mox.mockServices('myFltrFilter')

mox.setupResults(function () {
        return {
          MyFltr: '',
          MyFltrFilter: ''
        };
      })

Hi @Athaphian, thanks for creating this issue.
Mocking filters shouldn't be any problem via .mockServices(function() { ... }). I will soon (I'm on holiday right now) update the fvanwijk/mox-example repo because you are right that this should be more clear from the documentation.

I'm not sure if your example code is exactly what you tried, because the MyFltr key of setupResults will throw an error because Angular cannot inject it.
I think it also happens to MyFltrFilter because the first letter should be a lowercase 'm'. This typo might be the reason why the Mox way didn't work in your code, while the $provide.value('myFltrFilter', ...) way works.
As promised I updated the examples, so you can see how to test and how to mock a filter with and without Mox:
https://github.com/fvanwijk/mox-example/pull/1/files


Small changes for more readability and performance

@steveatinfincia are you still maintaining this repository?

@rodoufu Some of us could not get Steve answer for a while and forked https://github.com/maciejhirsz/tiny-bip39 off this repo.

@rodoufu Some of us could not get Steve answer for a while and forked https://github.com/maciejhirsz/tiny-bip39 off this repo.

Thanks, I will take a look.


Let ChatGPT optimize dereferenceSync

https://twitter.com/anttiviljami/status/1624718451599110145

Messed up the merge, but here's the relevant commit for those interested:
https://github.com/anttiviljami/dereference-json-schema/commit/ab6d56230a81957e50242812326f7aa4704e7b19


feat: dask expr __invert__, is_null, n_unique methods

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

[ ] 💾 Refactor
[x] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[ ] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #637

Checklist

[x] Code follows style guide (ruff)
[ ] Tests added
[ ] Documented the changes

If you have comments or can explain your changes, please do so below.
Convers a few methods and refactor is_null test

thanks! could you resolve the conflict please?


🐞[bug]: Images, text size & color, padding problems.

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
The text size is very small and its color is very light causing visibility problem for a mobile user.
The background image is not properly aligned.
The padding for text area for contact details is not correctly defined.
The button used is not much appealing for the website.
Please assign me these issues  under gssoc'24 EXTENDED and hacktoberfest.
Expected behavior
The Heading of the page can be highlighted in order to distinguish it from the other texts.
The images can be aligned as centre to make the page more responsive.
Add ScreenShots

What browsers are you seeing the problem on?
Chrome
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

Thank you for assigning me these issues. Also could you please add the level.

@akansha-02 i will add it to the PR.

Is this issue assigned? But it shows that no one is assigned. If not, can you please assign it to me? I would like to work on it.

you can assign that task to me under gssoc-ext and hacktoberfest label..

Would you assign this project to me under gssoc-ext . I will try to add it PR


Signal the end of away status to the client after disconnecting

Connect to a server

Mark yourself as away

Disconnect
(You are still away according to your client)

Reconnect
(Once again, you are still away according to your client, but not to the server)

Maybe there's a way to tell the client to cancel the away status without messing anything up?

That depends on your client. Does it let server know that it's away when connecting?
Also, any modules like simple_away used?

That depends on your client. Does it let server know that it's away when connecting?

When I said connect/disconnect, I meant znc connecting and disconnecting from the server. The client has no idea of what's going on.

Also, any modules like simple_away used?

No.

I see. Probably it should send the away status to server instead upon connect.

That seems like an even better solution.

I think this is a bit of a larger problem than just away, there's various states that a client may have which are not reset upon disconnect, or new connection. We can perhaps think about this a bit more holistically.

Joined channels, and subsequently the NAMES list (which might indicate to clients nicks that are online), and modes of those channels
Prior ISUPPORT state
AWAY state
MONITOR state (possibly connected to ISUPPORT, however reconnecting to server could leave client thinking it is monitoring something it is not anymore) - ties into  #1725
User's hostmask (unsure, clients might update from other numerics).

User mode seems to be one of the things which is updated upon disconnect along with capabilities (providing cap-notify was used). Some clients might already have logic to reset some of these states in 001 welcome numeric such as resetting isupport/away state.


google map Key move from index.html

keeping map key in index.html not a good idea in live environment.
Is there way can move the key from index.html to app.module.ts or environment.ts

you may want to use an npm package such as 'load-script2' and implement a dynamic loader such as this:
import loadScript from 'load-script2';
let scriptPromise: Promise<any> | null = null;
export const scriptLoader = (scriptSrc: string, attrs?: object, parentNode?: HTMLElement, apiExists?: (any) => boolean): Promise<any> => {
  // if(!scriptPromise)
  scriptPromise = new Promise((resolve, reject) => {
    if (apiExists && apiExists(window as any)) {
      console.log('scriptLoader:apiExists', scriptSrc);
      resolve((window as any));
    } else {
      console.log('scriptLoader:loadScript', scriptSrc);
      loadScript(scriptSrc, attrs, parentNode)
      .then(() => {
        resolve((window as any));
      })
      .catch((err) => {
        reject(err);
      });
    }
  });
  return scriptPromise;
};

you can pass in your Google maps api url (and the key can be stored basically any way you like):
example:
scriptLoader(
      "YOUR SCRIPT HERE",
    ).then((win) => {
      // script loaded
    }).catch(e => {
      // something broke
    });


Rename injection token for clarity

Notice that the base branch is #5635.
This PR doesn't do anything else than rename routeInjectionToken to frontEndRouteInjectionToken. This is done to make it easier to distinguish back-end routes from front-end routes.

Change Route to FrontEndRoute?

Could be done if we start having trouble distinguishing them. Currently I haven't had any issues with that.

Made mistake with merge. Merged to non master base branch.


Tauri App doesn't run after successful installation (Windows 10 NSIS)

Describe the bug
So I built my tauri application using npm run tauri build and then it created the target folder and release as expected. The app.exe file in the release folder works perfectly fine, also when I try to install the application using NSIS installer in release/bundle/nsis the application is installed successfully and the shortcut to the app is created too. But the application doesn't open up or there is no trace of it opening up anywhere in the task manager too.
Reproduction
No response
Expected behavior
The application should run & open as intended on the Windows platform after successful installation with NSIS.
Platform and versions
><EMAIL_ADDRESS>tauri
> tauri info

[✔] Environment
    - OS: Windows 10.0.19045 X64
    ✔ WebView2: 120.0.2210.61
    ✔ MSVC:
        - Visual Studio Build Tools 2019
        - Visual Studio Community 2019
    ✔ rustc: 1.65.0 (897e37553 2022-11-02)
    ✔ Cargo: 1.65.0 (4bc8f24d3 2022-10-20)
    ✔ rustup: 1.25.1 (bb60b1e89 2022-07-12)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 18.16.1
    - npm: 8.4.1

[-] Packages
    - tauri [RUST]: 1.4.1
    - tauri-build [RUST]: 1.4.0
    - wry [RUST]: 0.24.3
    - tao [RUST]: 0.16.2
    - @tauri-apps/api [NPM]: 1.4.0 (outdated, latest: 1.5.2)
    - @tauri-apps/cli [NPM]: 1.4.0 (outdated, latest: 1.5.7)

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../dist
    - devPath: http://localhost:1420/
    - framework: React (Next.js)
    - bundler: Webpack

Stack trace
No response
Additional context
No response

Without a reproduction or logs or anything, we can't really help, so please provide a reproduction. Also try building with tauri build --debug and see if you can get any errors or logs.

It was solved!

I ran into the same problem, Can you please describe how the issue was solved?

It was an issue with the application code itself. To debug this you can check the error logs which the app.exe (installed) generates when it is run using cmd. (not the shortcut lnk file but the actual exe)


Deduplicate sampling/demo code

This PR deduplicates the code inadvertently readded in #69 to use the refactored helpers implemented in #57.
Most of the commits are simple mechanical refactorings, aside from:

sgm.util.load_model_from_config had to be changed not to model.eval() (which was already done in #60)
the last commit, which merges the edited do_sample and do_txt2img from the sampling demo with the refactored helpers, and adds a new move_model context manager argument to implement low-VRAM mode. (I think it turned out pretty elegant.)

👉  Since I do not have a beefy enough CUDA device at my easy disposal, I didn't test the streamlit demo yet.

I didn't see this before starting on #102 or it would have saved me some work :( Regardless, I think I addressed the same things here, but let me know if you spot anything missing or have feedback on it. I believe my model swapping method ends up more flexible?


Fix query and fragment removed from request

Motivation:
DefaultSimpleHttpClient did not used query and fragment from original
URI from request.
Modifications:

Build string from path, query and fragment in original URI so query
and fragment are not removed.

Result:
Proper request while using SimpleHttpClient

Current coverage is 64.06% (diff: 33.33%)

Merging #224 into master will decrease coverage by 0.12%

@@             master       #224   diff @@
==========================================
  Files           277        277
  Lines         10603      10611     +8
  Methods           0          0
  Messages          0          0
  Branches       1534       1536     +2
==========================================
- Hits           6806       6798     -8
- Misses         3059       3071    +12
- Partials        738        742     +4

Powered by Codecov. Last update 07e767f...86a888e

Thanks! :+1:


Support URL object for fetch and fix getBaseUrl

A solution for #140.
Fixed request.js to support URL object for fetch.

Changed to convert URL object to string when the input variable is an instance of URL object.

Fixed getBaseUrl() function in url.js to support fetch URL that has no host for any environment.

Changed to return an url whose origin is the same as the url where the storybook is hosted (= location.href) if the url has no host.
To test a function that refers to window.location.href, set Jest test environment to jsdom and add mock of window.location.href``.

Note : new URL(url, base) returns url without changing anything if it has host.
e.g. new URL('https://a.com/foo/bar', 'https://b.com') -> https://a.com/foo/bar

@akkadaska  can you resolve your merge conflicts?

@nutboltu
Merge conflicts have been successfully resolved. However, tests are failing, which seems to be a pre-existing issue with the master branch.

@akkadaska Can you pull the changes from the main repo into your build. This is a problem that we are running into as well and would like to use your fork if possible.

@buggedcom Pulled the changes and my fork is updated.
@nutboltu I resolved conflicts again. I would appreciate it if you could consider merging.


replace loader.dropped content with ingestor.dropped [#155996794]

Signed-off-by: Warren Fernandes<EMAIL_ADDRESS>

DO NOT MERGE. Story is rejected

@AmberAlston 10-4. Blocking story until we hear different.

@ljarzynski Thanks! Acceptance of https://www.pivotaltracker.com/story/show/155996794 is what can unblock this merge. @kbakas-piv and I can also merge upon acceptance.

Looks good to me. Merging and unblocking, will accept story @ljarzynski


trace_call uses incorrect gas limit when it's not provided in the input

Describe the bug
When no gas is provided in the trace_call tx object (first input argument), we derive it from the block the call is executed at (third input argument): https://github.com/paradigmxyz/reth/blob/d6092cf1d3bf6dcd274815ce12c15726293a529b/crates/rpc/rpc/src/eth/api/transactions.rs#L541 while both geth and Erigon uses the pre-configured constant of 50 million:
https://github.com/ledgerwatch/erigon/blob/eae2d9a79cb70dbe30b3a6b79c436872e4605458/cmd/rpcdaemon/commands/trace_adhoc.go#L956
https://github.com/ledgerwatch/erigon/blob/eae2d9a79cb70dbe30b3a6b79c436872e4605458/eth/ethconfig/config.go#L94
Reth's behaviour seems to be logically correct, as we inherit the whole environment from the specified block we trace the call at. But it's not consistent with what geth and Erigon currently do.
Steps to reproduce

Start reth and erigon
Call trace_call:

{
  "jsonrpc": "2.0",
  "method": "trace_call",
  "params": [
    {
      "to": "0x6b175474e89094c44da98b954eedeac495271d0f",
      "data": "0x70a082310000000000000000000000005d3a536e4d6dbd6114cc1ead35777bab948e3643"
    },
    [
      "trace"
    ],
    "0xe67702"
  ],
  "id":<PHONE_NUMBER>51704834
}

Observe the difference in responses:

type                index       reth       erigon
──────────────────────────────────────────────────
value   trace.0.action.gas   0x1c8fd59   0x2fa9cc8
                              29949273   49978568

reth's 29949273 gas limit is the mainnet block 15103746 gas limit 29970705 minus the 21432 gas (min tx gas + initialization costs)
erigon's 49978568 gas limit is the constant gas limit 50000000 minus the same 21432 gas
Code of Conduct

[X] I agree to follow the Code of Conduct

The solution is to just change the max gas to 30M instead of 50M right?

yes, not sure why it is 50M by default though

The solution is to just change the max gas to 30M instead of 50M right?

and use it in the trace_call instead of block's actual gas limit

Turns out, same applies to eth_call RPC calls: if no gas limit is specified in the transaction object, Geth fallbacks to the default gas cap which is 30M: https://github.com/ethereum/go-ethereum/blob/84b05d4f343dea9d89862d0bbaa7e9d0c3f0acd5/internal/ethapi/api.go#L1037


[DCOS-46650] Cassandra base tech upgrade to 3.11.3

What does this PR aim to do?

Upgrade Cassandra base tech version for cassandra package release.
Defaults to the package CQL sh version which is 5.0.1

How were the changes tested?

By running a CI build.

@samvantran @akirillov one more look please.

@akirillov one last time.

Thanks, @soumasish, LGTM. Let's wait for CI builds to finish.


✨ Added support for custom CA contexts within ZabbixAPI

This PR provides the possibility for an user to run the script with a custom CA bundle file, fixing issue #82.
While pynetbox uses the requests package, which listens to the environment variable REQUEST_CA_BUNDLE, zabbix_utils uses urllib.request, which provides a SSLContext variable for the user to implement.
This PR implements this SSLContext, when the user has defined a REQUEST_CA_BUNDLE environment variable, so that Zabbix will also use this ca bundle, keeping the configuration of customized certificates simple.

Nice elegant fix! Thanks!


Sequence diagrams are visible in AppMaps

This requires the release of sequence diagrams in appmpa-js.

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

v0.68.0
GitHub release

Your semantic-release bot :package::rocket:


Add hot reload workflows into dwds and flutter-tools

This is instead being tracked by https://github.com/dart-lang/sdk/issues/54934. Closing this out.

Correction it's being tracked by https://github.com/dart-lang/webdev/issues/2516


Add github actions as build test

This could be instead of travis but for now should be an alternative.

It appears that github actions comes with several problems, will ignore it for now.


Add semantic versioning on GH

In order to track everything more sufficiently and have automated release notes linked to the specific commit.

@rogankool Can we prioritize this one?

@bosko-m I will present it to the team, you will get the feedback on this and other issues, and what is going in the next release - next Monday the latest.

Hello, we tried to solve the issue.
This is what we did:
Modified the pull request template to include a required field for the related GitHub issue number. This will help track changes more efficiently and enable automated release notes linked to specific commits.
You can review changes in this commit: https://github.com/mnmnk43434/IntersectMBO-cc-portal-481/commit/44a0ed1dbda1f2fd6a06d2371d61471569078bba.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


Is it available on windows system?

Hi, I have watched your video on youtube and really appreciate your work. I want to run it on windows but I am a bit new on this.
The setup.sh requires a linux package, is it still possible to run on windows?

Hello!
That is a great question. You have a few options to run it on Windows, although you will need some technical skills for all of them:

Run in WSL
Run in Docker
Install PyAudio and ffmpeg directly, without using the script (setup.sh is just for this).

Running server should be pretty straightforward, but running client might be more difficult, as it will require access to microphone.
If you do manage to get it working, please post your steps here?

Hello! That is a great question. You have a few options to run it on Windows, although you will need some technical skills for all of them:

Run in WSL
Run in Docker
Install PyAudio and ffmpeg directly, without using the script (setup.sh is just for this).

Running server should be pretty straightforward, but running client might be more difficult, as it will require access to microphone.
If you do manage to get it working, please post your steps here?

Thank you, maybe I will try to get it in a few days..
I paused my idea because I was really frustrated that I can't get the windows system sound output simultaneously, when it came from my bluetooth speaker. I was hoping to develop an easy-to-install open source project that provides real-time subtitle for any media playing on windows. But it always involves extra conditions or operations. :<


ROX-9962: Fix import policy error display when not a duplicate

Description
As part of the work to deprecate pre-versions 1.1 policy formats, backend found that the error display for errors like "bad format..." were confusingly showing duplicate resolution choices.
This, PR into the backend work branch, fixes the error display for those types of errors.
As often happens, I noticed many opportunities for refactoring and improvement in the existing import handling code, but I resisted the urge to do more than necessary to fix the problem. (Full disclosure: I wrote the original frontend import code, and so it was particularly hard for me to resist the urge to improve it out of shame.)
Checklist

[ ] Investigated and inspected CI test results

Testing Performed
Confirmed that all existing UI e2e tests still pass
Manual testing
(new) Correct display of errors not related to duplicates

(confirmed existing behavior) Duplicate problems are still displayed correctly

(confirmed existing behavior) Policies with no errors or duplication import immediately

@vjwilson Huge thank you!


refactor(core): Replace segmentio-encoding/json

Description
Replaces segmentio-encoding/json by wandb/simplejsonext (which is necessary to support encoding/decoding Infinity and NaN; otherwise we'd use the standard json package).
NOTE: There are important semantic differences that could cause issues. I tried to be careful, but we might get unexpected behavior in places that are not well tested. Here are a few errors that caught me off guard:

JSON numbers are always unmarshalled as either int64 or float64. ~There was system metrics code that asserted that numbers were float64 (which works for encoding/json, I think) which resulted in panics.~ (updated: system metrics can use encoding/json, but I left some of the cleanup that I did)
Only string-keyed maps can be marshalled to JSON. I had to modify runconfig.go because it was using an int-keyed map for telemetry and "metrics".
{} is unmarshalled as a nil map. I had to update the part of runconfig.go that modifies the "_wandb" key to allocate a map if the value is nil before writing into it.

should we test this case in history:
{} is unmarshalled as a nil map. I had to update the part of runconfig.go that modifies the "_wandb" key to allocate a map if the value is nil before writing into it.

in case users provide an empty dict will things break?

should we test this case in history:
{} is unmarshalled as a nil map. I had to update the part of runconfig.go that modifies the "_wandb" key to allocate a map if the value is nil before writing into it.

in case users provide an empty dict will things break?

It only matters if we're mutating the data we read, which I think only happens in this one instance. Otherwise, a nil map is the same as an empty map for reading in Go.

should we test this case in history:
{} is unmarshalled as a nil map. I had to update the part of runconfig.go that modifies the "_wandb" key to allocate a map if the value is nil before writing into it.

in case users provide an empty dict will things break?

It only matters if we're mutating the data we read, which I think only happens in this one instance. Otherwise, a nil map is the same as an empty map for reading in Go.

Ah thanks!


Question: RouterTSP with already calculated distance matrix

I already have a distance matrix calculated. may I use the TSPRouter class with and already calculated distance matrix?

Ok. I found the solution.


Add platform for Cloud Hypervisor (via libvirt)

LISA currently has a qemu platform that orchestrates the tests on a QEMU VM. It interacts with QEMU via libvirt. Most of this code can be used for Cloud Hypervisor too by changing the libvirt connection URI scheme from qemu:// to ch://.
Extract all the re-usable code from qemu platform into a base class called BaseLibvirtPlatform. Both the qemu and the new cloud-hypervisor platform inherit from this base class. BaseLibvirtPlatform is an abstract class and cannot be used as a platform on its own. It must be extended. BaseLibvirtPlatform will use BaseLibvirtPlatformSchema and BaseLibvirtNodeSchema and both can be extended if required.
Platform
  |
  |==> BaseLibvirtPlatform
    |
    |==> QemuPlatform
    |
    |==> CloudHypervisorPlatform
    |
    |==> SomeOtherLibvirtBasedPlatform

All the common handling will be done in the base class and derived classes can override wherever any special handling is required based on the hypervisor used.
The following things need special handling for Cloud Hypervisor:

The Cloud Hypervisor driver for libvirt doesn’t support any flags during domain creation. So LISA’s domain.createWithFlags call throws an error. For Cloud Hypervisor no flags should be sent.  domain.undefineFlags should be handled along similar lines.
Raw disk images should be used because qcow2 support will be dropped from cloud-hypervisor.
Domain XML should specify virtio devices.

NOTE: The upstream version of libvirt has limited support for Cloud Hypervisor so cloud-hypervisor/libvirt should be used.
Sample runbook:
extension:
  - /home/anirudh/src/lisa/microsoft/testsuites
testcase:
  - criteria:
      name: "cpu_count_check"
variable:
  - name: keep_environment
    value: "no"
  - name: qcow2
    value: "/home/anirudh/imgs/focal-server-cloudimg-amd64.img"
  - name: extra_user_data
    value: ""
  - name: admin_private_key_file
    value: "/home/anirudh/imgs/id_rsa"
notifier:
  - type: console
platform:
  - type: cloud-hypervisor
    admin_username: anirudh
    admin_private_key_file: $(admin_private_key_file)
    keep_environment: $(keep_environment)
    cloud-hypervisor:
      network_boot_timeout: 60
      hosts:
        - address: "<IP_ADDRESS>"
          username: "anirudh"
          private_key_file: $(admin_private_key_file)
    requirement:
      cloud-hypervisor:
        disk_img: $(qcow2)
        disk_img_format: "qcow2"
        firmware: "/home/anirudh/hypervisor-fw"
        cloud_init:
          extra_user_data: $(extra_user_data)

Cc: @vyadavmsft @pupacha

@squirrelsc @cwize1 please provide your feedback on this approach. Alternative approach suggestions are also welcome.

@squirrelsc @cwize1 please provide your feedback on this approach. Suggestions for a completely new approach are also welcome.

@squirrelsc @cwize1 please provide your feedback on this approach. Suggestions with alternative approaches are also welcome.

The design looks good to me.

@anirudhrb Overall, it's good to me, and I'm happy to see it happens. Do you know how other libvirt based implemented? I want to make sure we can make a more reusable libvirt base class.

@anirudhrb Overall, it's good to me, and I'm happy to see it happens. Do you know how other libvirt based implemented? I want to make sure we can make a more reusable libvirt base class.

Thanks! I didn't quite understand the question.

@anirudhrb Overall, it's good to me, and I'm happy to see it happens. Do you know how other libvirt based implemented? I want to make sure we can make a more reusable libvirt base class.

Thanks! I didn't quite understand the question.

In current PR, it supports cloud-hypervisor and qemu. The base libvirt platform exposes enough interface for their different behavior. But if other libvirt based platforms need to be supported in future, is the current interface enough? It doesn't need to support other platforms for now, but it needs to understand the impact in future, and try the best to construct a sharable interface to prevent big churn in future. When the code is used widely, it's almost impossible to do big refactoring without regression.

@anirudhrb Overall, it's good to me, and I'm happy to see it happens. Do you know how other libvirt based implemented? I want to make sure we can make a more reusable libvirt base class.

Thanks! I didn't quite understand the question.

In current PR, it supports cloud-hypervisor and qemu. The base libvirt platform exposes enough interface for their different behavior. But if other libvirt based platforms need to be supported in future, is the current interface enough? It doesn't need to support other platforms for now, but it needs to understand the impact in future, and try the best to construct a sharable interface to prevent big churn in future. When the code is used widely, it's almost impossible to do big refactoring without regression.

I don't have extensive knowledge of the internals of libvirt and the other hypervisors supported.
AFAIK, Libvirt exposes common interfaces for all hypervisors. The current APIs should mostly work for other hypervisors too. In case there is some differences the new platform can override that method and handle it there. For example, in this PR, the Cloud Hypervisor platform doesn't support the flags while creating VM (domain.createWithFlags). So, I have overridden the start code and did it without the flags. Other than a couple more instances like this most of the existing QEMU platform code worked for Cloud Hypervisor too.
In the future I don't expect any huge refactoring to be required to support new hypervisors via libvirt. It will either be overriding a method in the platform or extracting some part into a new method in the base platform and then overriding it.

@anirudhrb Overall, it's good to me, and I'm happy to see it happens. Do you know how other libvirt based implemented? I want to make sure we can make a more reusable libvirt base class.

Thanks! I didn't quite understand the question.

In current PR, it supports cloud-hypervisor and qemu. The base libvirt platform exposes enough interface for their different behavior. But if other libvirt based platforms need to be supported in future, is the current interface enough? It doesn't need to support other platforms for now, but it needs to understand the impact in future, and try the best to construct a sharable interface to prevent big churn in future. When the code is used widely, it's almost impossible to do big refactoring without regression.

I don't have extensive knowledge of the internals of libvirt and the other hypervisors supported.
AFAIK, Libvirt exposes common interfaces for all hypervisors. The current APIs should mostly work for other hypervisors too. In case there is some differences the new platform can override that method and handle it there. For example, in this PR, the Cloud Hypervisor platform doesn't support the flags while creating VM (domain.createWithFlags). So, I have overridden the start code and did it without the flags. Other than a couple more instances like this most of the existing QEMU platform code worked for Cloud Hypervisor too.
In the future I don't expect any huge refactoring to be required to support new hypervisors via libvirt. It will either be overriding a method in the platform or extracting some part into a new method in the base platform and then overriding it.

It'll be pretty difficult to predict what differences there will be, if we ever add support for other Hypervisors. So, IMHO it is probably better to keep things simple for now.

Is this change ready for review? It is still marked as draft.

Now it is.

@anirudhrb The change is ok to me, but I need @cwize1 to approve it firstly.


revert node16 changes of GitHubReleaseV1

Task name:  GitHubReleaseV1
Description: Revert node 16 changes of https://github.com/microsoft/azure-pipelines-tasks/pull/17248
Documentation changes required: (Y/N)
Added unit tests: (Y/N)
Attached related issue: (Y/N)
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

/azp run

/azp run

/azp run


[ownership] Move OME after CopyForwarding

This may break pass restart in the interim, since OME is a module pass and is being moved into a function pass pipeline.

@swift-ci benchmark

Performance: -O

Regression
OLD
NEW
DELTA
RATIO

IterateData
827
960
+16.1%
0.86x (?)

ObjectiveCBridgeStringHash
83
92
+10.8%
0.90x (?)

DataAccessBytesMedium
47
52
+10.6%
0.90x (?)

Array2D
4368
4768
+9.2%
0.92x (?)

ObjectiveCBridgeStubToNSDateRef
2160
2340
+8.3%
0.92x (?)

PrefixArray
13
14
+7.7%
0.93x (?)

Improvement
OLD
NEW
DELTA
RATIO

FlattenListFlatMap
4537
2699
-40.5%
1.68x (?)

LessSubstringSubstring
30
22
-26.7%
1.36x

LessSubstringSubstringGenericComparable
30
22
-26.7%
1.36x

EqualSubstringSubstring
29
22
-24.1%
1.32x

EqualSubstringSubstringGenericEquatable
29
22
-24.1%
1.32x

EqualSubstringString
29
22
-24.1%
1.32x

EqualStringSubstring
30
23
-23.3%
1.30x

UTF8Decode_InitFromData
169
136
-19.5%
1.24x

UTF8Decode_InitFromCustom_contiguous
165
139
-15.8%
1.19x (?)

UTF8Decode_InitDecoding
165
141
-14.5%
1.17x

Breadcrumbs.MutatedUTF16ToIdx.Mixed
317
273
-13.9%
1.16x

Breadcrumbs.MutatedIdxToUTF16.Mixed
323
280
-13.3%
1.15x (?)

UTF8Decode_InitFromBytes
159
140
-11.9%
1.14x (?)

StringComparison_longSharedPrefix
373
336
-9.9%
1.11x

Set.isSubset.Seq.Empty.Int
84
76
-9.5%
1.11x (?)

Calculator
158
143
-9.5%
1.10x (?)

NSDictionaryCastToSwift
1060
960
-9.4%
1.10x (?)

FlattenListLoop
1517
1374
-9.4%
1.10x (?)

Set.subtracting.Empty.Int
28
26
-7.1%
1.08x (?)

Code size: -O

Regression
OLD
NEW
DELTA
RATIO

ObserverForwarderStruct.o
2828
2932
+3.7%
0.96x

ObserverUnappliedMethod.o
5113
5249
+2.7%
0.97x

Improvement
OLD
NEW
DELTA
RATIO

RandomTree.o
12167
11999
-1.4%
1.01x

Performance: -Osize

Regression
OLD
NEW
DELTA
RATIO

IterateData
828
950
+14.7%
0.87x (?)

ObjectiveCBridgeStringHash
83
92
+10.8%
0.90x

StrComplexWalk
2970
3240
+9.1%
0.92x (?)

StringWalk
1520
1640
+7.9%
0.93x (?)

Improvement
OLD
NEW
DELTA
RATIO

LessSubstringSubstring
30
22
-26.7%
1.36x

EqualStringSubstring
29
22
-24.1%
1.32x

EqualSubstringSubstring
30
23
-23.3%
1.30x

EqualSubstringSubstringGenericEquatable
30
23
-23.3%
1.30x

EqualSubstringString
30
23
-23.3%
1.30x

LessSubstringSubstringGenericComparable
30
23
-23.3%
1.30x

UTF8Decode_InitFromBytes
171
140
-18.1%
1.22x

UTF8Decode_InitFromData
163
136
-16.6%
1.20x (?)

UTF8Decode_InitFromCustom_contiguous
165
139
-15.8%
1.19x

UTF8Decode_InitDecoding
167
141
-15.6%
1.18x

Breadcrumbs.MutatedIdxToUTF16.Mixed
324
280
-13.6%
1.16x (?)

Breadcrumbs.MutatedUTF16ToIdx.Mixed
317
274
-13.6%
1.16x (?)

StringComparison_longSharedPrefix
374
337
-9.9%
1.11x

Set.isStrictSubset.Int.Empty
52
47
-9.6%
1.11x (?)

StringToDataEmpty
550
500
-9.1%
1.10x (?)

Set.subtracting.Empty.Int
28
26
-7.1%
1.08x (?)

Code size: -Osize
Performance: -Onone

Regression
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeStringHash
83
92
+10.8%
0.90x

StrComplexWalk
4990
5500
+10.2%
0.91x (?)

ObjectiveCBridgeStubNSDateRefAccess
4235
4610
+8.9%
0.92x (?)

StringWalk
3120
3360
+7.7%
0.93x (?)

DataCreateEmptyArray
44350
47750
+7.7%
0.93x (?)

Improvement
OLD
NEW
DELTA
RATIO

EqualSubstringSubstringGenericEquatable
33
26
-21.2%
1.27x

LessSubstringSubstringGenericComparable
33
26
-21.2%
1.27x

UTF8Decode_InitFromBytes
176
141
-19.9%
1.25x

UTF8Decode_InitFromData
168
135
-19.6%
1.24x

LessSubstringSubstring
34
28
-17.6%
1.21x (?)

EqualStringSubstring
35
29
-17.1%
1.21x

EqualSubstringString
35
29
-17.1%
1.21x

UTF8Decode_InitDecoding
174
147
-15.5%
1.18x

UTF8Decode_InitFromCustom_contiguous
177
150
-15.3%
1.18x

EqualSubstringSubstring
34
29
-14.7%
1.17x (?)

Breadcrumbs.MutatedUTF16ToIdx.Mixed
334
290
-13.2%
1.15x

Breadcrumbs.MutatedIdxToUTF16.Mixed
340
297
-12.6%
1.14x

Code size: -swiftlibs

Regression
OLD
NEW
DELTA
RATIO

libswift_Differentiation.dylib
233472
237568
+1.8%
0.98x

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac mini
  Model Identifier: Macmini8,1
  Processor Name: 6-Core Intel Core i7
  Processor Speed: 3.2 GHz
  Number of Processors: 1
  Total Number of Cores: 6
  L2 Cache (per Core): 256 KB
  L3 Cache: 12 MB
  Memory: 64 GB

Converting the OME to a function pass is not an option. Doing so, we may end up inlining ossa functions to non-ossa functions and vice versa due to callgraph scc cycles. Added a comment to document this interim issue.

@swift-ci test

@swift-ci test source compatibility

Build failed
Swift Test Linux Platform
Git Sha - 88166079b9e925e4bf114d60aaa562ce5dabc0f2

Build failed
Swift Test OS X Platform
Git Sha - 88166079b9e925e4bf114d60aaa562ce5dabc0f2

@swift-ci smoke test OS X platform

@swift-ci smoke test OS X platform

@swift-ci test OS X platform

Build failed
Swift Test OS X Platform
Git Sha - 542af449ae7baa839ea3fa4d61493aa109d02fb0

https://github.com/apple/swift/pull/33205 converts ome into a function pass. We support inlining of ossa functions into non-ossa functions in the inliner. The ossa function is converted on demand in the SILCloner. Updated the PR to reflect this, also this can be merged only after https://github.com/apple/swift/pull/33205.

@swift-ci test


Output FastJet clusters as a list rather than a MaskGroup

MaskGroup doesn't have any special useful functionality, it just makes things more complex with the string wrapped numerical indices. Return it as a list.

During refactor, also move into select module, and add parameter to limit making the top_k masks by pT.


Support build for ARM64 processor

Dev Notes:
https://github.com/docker/build-push-action/blob/master/docs/advanced/multi-platform.md

I had hard time in installing libc-compat for arm64 so I had to add Github Action setup-qemu as indicated in the link ☝


Repo improvements

[ ] Update README to give detailed information about the repository. You can use the README structure from our other extensions.
[x] Remove cytoscape.js and cytoscape.min.js files in the root of the source code. They should not be necessary.
[ ] While naming the file names and variables, always use camel case (commonstream -> commonStream).
[ ] Format the code. Use same indentation amount everywhere (e.g. tab with 2 spaces), delete commented code snippets and unnecessary empty lines.
[ ] Write explanatory comments to your code.
[ ] Make menu in the demo more user-friendly as discussed.

Are these resolved, should I check them?


DirectoryLister with office file

Hi,
there's a possibility to open office file(doc, docx, xlsx, xls, ppt) without download it?

Sorry, but the only way to open a file is to download it.