OmniSharp-Roslyn v1.32.2+ HTTP is refusing connections

The Problem
The OmniSharp-Roslyn HTTP (v1.32.2) server is refusing all HTTP connections.
The problem seems to have occurred between v1.32.1 and v1.32.2.
Systems this occurs on
This issue has been reported to occur on these devices:

Fedora Linux 28 (OmniSharp-Roslyn HTTP Linux)
macOS High Sierra (OmniSharp-Roslyn HTTP OSX Mono)
Arch Linux (OmniSharp-Roslyn HTTP Linux)
Ubuntu 18.10 (OmniSharp-Roslyn HTTP Linux)

It does not occur on the Windows versions of the builds.
Running v1.32.2 of the server from the command line
This was run with the -v flag, the previous version of the server printed thousands of lines of debug: data, whereas v1.32.2 did not.

Click to view output.

[axvr@arctic Redacted.Auth master]$ ~/.omnisharp/omnisharp-roslyn/run -v -p 4242 -s "$(pwd)/Redacted.Auth.sln"
info: OmniSharp.Http.Startup[0]
      Starting OmniSharp on fedora 28.0 (x64)
info: OmniSharp.Cake.CakeProjectSystem[0]
      Detecting Cake files in '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth'.
info: OmniSharp.Cake.CakeProjectSystem[0]
      Could not find any Cake files
info: OmniSharp.Http.Startup[0]
      Project system 'OmniSharp.DotNet.DotNetProjectSystem' is disabled in the configuration.
info: OmniSharp.MSBuild.ProjectSystem[0]
      Detecting projects in '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.sln'.
info: OmniSharp.MSBuild.ProjectManager[0]
      Queue project update for '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Api/Redacted.Auth.Api.csproj'
info: OmniSharp.MSBuild.ProjectManager[0]
      Queue project update for '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Logic/Redacted.Auth.Logic.csproj'
info: OmniSharp.MSBuild.ProjectManager[0]
      Queue project update for '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Tests/Redacted.Auth.Tests.csproj'
info: OmniSharp.Script.ScriptProjectSystem[0]
      Detecting CSX files in '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth'.
info: OmniSharp.Script.ScriptProjectSystem[0]
      Could not find any CSX files
info: OmniSharp.Http.Startup[0]
      Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpWorkspaceOptionsProvider
info: OmniSharp.Http.Startup[0]
      Configuration finished.
info: OmniSharp.Http.Startup[0]
      Omnisharp server running on port '4242' at location '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth' on host -1.
info: OmniSharp.MSBuild.ProjectManager[0]
      Loading project: /home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Api/Redacted.Auth.Api.csproj
info: OmniSharp.MSBuild.ProjectManager[0]
      Successfully loaded project file '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Api/Redacted.Auth.Api.csproj'.
info: OmniSharp.MSBuild.ProjectManager[0]
      Adding project '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Api/Redacted.Auth.Api.csproj'
info: OmniSharp.MSBuild.ProjectManager[0]
      Loading project: /home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Logic/Redacted.Auth.Logic.csproj
info: OmniSharp.MSBuild.ProjectManager[0]
      Successfully loaded project file '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Logic/Redacted.Auth.Logic.csproj'.
info: OmniSharp.MSBuild.ProjectManager[0]
      Adding project '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Logic/Redacted.Auth.Logic.csproj'
info: OmniSharp.MSBuild.ProjectManager[0]
      Loading project: /home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Tests/Redacted.Auth.Tests.csproj
info: OmniSharp.MSBuild.ProjectManager[0]
      Successfully loaded project file '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Tests/Redacted.Auth.Tests.csproj'.
info: OmniSharp.MSBuild.ProjectManager[0]
      Adding project '/home/axvr/Documents/Projects/work/Redacted/Redacted.Auth/Redacted.Auth.Tests/Redacted.Auth.Tests.csproj'
info: OmniSharp.MSBuild.ProjectManager[0]
      Update project: Redacted.Auth.Api
info: OmniSharp.MSBuild.ProjectManager[0]
      Update project: Redacted.Auth.Logic
info: OmniSharp.MSBuild.ProjectManager[0]
      Update project: Redacted.Auth.Tests

OmniSharp-vim log:
OmniSharp-vim is unable to connect to v1.32.2 of the OmniSharp-Roslyn server, this is the error log produced.
NOTE: This does not seem to be an issue with OmniSharp-vim.

Click to view output.

2018-08-21 19:25:22,649 ERROR Could not connect to OmniSharp server: http://localhost:43993/v2/getcodeactions
Traceback (most recent call last):
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/util.py", line 214, in doRequest
    response = opener.open(req, body, timeout)
  File "/usr/lib64/python2.7/urllib2.py", line 429, in open
    response = self._open(req, data)
  File "/usr/lib64/python2.7/urllib2.py", line 447, in _open
    '_open', req)
  File "/usr/lib64/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib64/python2.7/urllib2.py", line 1230, in http_open
    return self.do_open(httplib.HTTPConnection, req)
  File "/usr/lib64/python2.7/urllib2.py", line 1200, in do_open
    raise URLError(err)
URLError:
2018-08-21 19:25:22,649 ERROR Error running python getCodeActions()
Traceback (most recent call last):
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/vimcmd.py", line 25, in wrapper
    ret = fxn(*args, **kwargs)
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/commands.py", line 108, in getCodeActions
    response = getResponse(ctx, '/v2/getcodeactions', parameters, json=True)
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/util.py", line 195, in getResponse
    json=json)
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/util.py", line 218, in doRequest
    raise ServerConnectionError(str(e))
ServerConnectionError:
2018-08-21 19:25:22,651 ERROR Could not connect to OmniSharp server: http://localhost:43993/v2/getcodeactions
Traceback (most recent call last):
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/util.py", line 214, in doRequest
    response = opener.open(req, body, timeout)
  File "/usr/lib64/python2.7/urllib2.py", line 429, in open
    response = self._open(req, data)
  File "/usr/lib64/python2.7/urllib2.py", line 447, in _open
    '_open', req)
  File "/usr/lib64/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib64/python2.7/urllib2.py", line 1230, in http_open
    return self.do_open(httplib.HTTPConnection, req)
  File "/usr/lib64/python2.7/urllib2.py", line 1200, in do_open
    raise URLError(err)
URLError:
2018-08-21 19:25:22,651 ERROR Error running python getCodeActions()
Traceback (most recent call last):
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/vimcmd.py", line 25, in wrapper
    ret = fxn(*args, **kwargs)
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/commands.py", line 108, in getCodeActions
    response = getResponse(ctx, '/v2/getcodeactions', parameters, json=True)
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/util.py", line 195, in getResponse
    json=json)
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/util.py", line 218, in doRequest
    raise ServerConnectionError(str(e))
ServerConnectionError:
2018-08-21 19:25:31,844 ERROR Could not connect to OmniSharp server: http://localhost:43993/updatebuffer
Traceback (most recent call last):
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/util.py", line 214, in doRequest
    response = opener.open(req, body, timeout)
  File "/usr/lib64/python2.7/urllib2.py", line 429, in open
    response = self._open(req, data)
  File "/usr/lib64/python2.7/urllib2.py", line 447, in _open
    '_open', req)
  File "/usr/lib64/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib64/python2.7/urllib2.py", line 1230, in http_open
    return self.do_open(httplib.HTTPConnection, req)
  File "/usr/lib64/python2.7/urllib2.py", line 1203, in do_open
    r = h.getresponse(buffering=True)
  File "/usr/lib64/python2.7/httplib.py", line 1121, in getresponse
    response.begin()
  File "/usr/lib64/python2.7/httplib.py", line 438, in begin
    version, status, reason = self._read_status()
  File "/usr/lib64/python2.7/httplib.py", line 402, in _read_status
    raise BadStatusLine(line)
BadStatusLine: ''
2018-08-21 19:25:31,845 ERROR Error running python updateBuffer()
Traceback (most recent call last):
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/vimcmd.py", line 25, in wrapper
    ret = fxn(*args, **kwargs)
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/commands.py", line 254, in updateBuffer
    getResponse(ctx, "/updatebuffer")
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/util.py", line 195, in getResponse
    json=json)
  File "/home/axvr/Documents/Projects/dotfiles/vim/.vim/pack/vivid/opt/omnisharp-vim/python/omnisharp/util.py", line 218, in doRequest
    raise ServerConnectionError(str(e))
ServerConnectionError: ''

Visiting the server in a web browser
Obviously you aren't meant to use OmniSharp from a web browser, but the responses do highlight a difference between the two versions of the server.
v.32.1
A blank web page is displayed and the server prints some connection info to STDOUT.
v.32.2
Nothing is printed to STDOUT, but the browser displays this error message:

If any more information is needed, please let me know. :smile:

Edit: Add more affected systems.

@david-driscoll the issue is occurring on all *nix's, including Ubuntu, see the issue description.
I've experienced it in an archlinux VM, and an archlinux flavoured WSL.

Any update or progress on this?

I can confirm that manually building the latest source from git on Arch Linux (following @nickspoons modifications to the build scripts) results in a working build of the HTTP driver, whereas installing any compiled release after version 1.32.1 from here gives me a broken build that refuses connections.
$ mono --version
Mono JIT compiler version 5.16.0 (makepkg/6e48ad4f7b1 Mon Oct 15 22:56:48 CEST 2018)
Copyright (C) 2002-2014 Novell, Inc, Xamarin Inc and Contributors. www.mono-project.com
        TLS:           __thread
        SIGSEGV:       altstack
        Notifications: epoll
        Architecture:  amd64
        Disabled:      none
        Misc:          softdebug
        Interpreter:   yes
        LLVM:          supported, not enabled.
        GC:            sgen (concurrent by default)

$ dotnet --version
2.1.403

@nickspoons
Maybe Eureka.
At https://github.com/OmniSharp/omnisharp-vim/issues/427#issuecomment-445017252 you said me to use omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461/OmniSharp.exe'. You didn't say to me to use omnisharp-roslyn/artifacts/publish/OmniSharp.Http.Driver/mono/OmniSharp.exe.
If you install the last version of OmniSharp:
rm -Rf ~/.omnisharp
cd ~/Downloads
wget https://github.com/OmniSharp/omnisharp-roslyn/releases/download/v1.32.8/omnisharp.http-linux-x64.tar.gz
mkdir -p ~/.omnisharp/omnisharp-roslyn
tar -xvzf ~/Downloads/omnisharp.http-linux-x64.tar.gz -C ~/.omnisharp/omnisharp-roslyn

Thanks the command :
$ diff -rq /media/data/home/omnisharp-roslyn/artifacts/publish/OmniSharp.Http.Driver/mono/ ~/.omnisharp/omnisharp-roslyn/omnisharp | grep "Only"

you could see that there is nothing diff between this two folders.
But thanks:
$ diff -rq /media/data/home/omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461 ~/.omnisharp/omnisharp-roslyn/omnisharp | grep "Only"

you could see there are missing files in the published version, especially
(...)
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.AppContext.dll
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.ComponentModel.Primitives.dll
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.ComponentModel.TypeConverter.dll
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.Console.dll
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.IO.FileSystem.dll
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.IO.FileSystem.Primitives.dll
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.Numerics.Vectors.dll
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.Runtime.InteropServices.RuntimeInformation.dll
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.Security.Cryptography.Encoding.dll
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.Security.Cryptography.Primitives.dll
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.Security.Cryptography.X509Certificates.dll
Only in omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461: System.Threading.Thread.dll
(...)

```

If you copy this files in the released version thanks the command
```
$ cp -n /media/data/home/omnisharp-roslyn/bin/Debug/OmniSharp.Http.Driver/net461/* ~/.omnisharp/omnisharp-roslyn/omnisharp
```

You could see that the released version works ! (`cp -n` copy only files quoted above, without overwriting (updating) any existing file)`

Therefore, why  in `omnisharp-roslyn/artifacts/publish/OmniSharp.Http.Driver/mono`, the folder used to publish OmniSharp this files are missing ? If you fix this problem, you will fix this issue.

I'm a student, therefore I'm very beginner in c#. Hard to me to find any solution. But I think it could be very simple to find a solution !

Eureka. It seems to delete
void DeleteUnnecessaryAssemblies(string folder)
{
    FileHelper.Delete(CombinePaths(folder, "System.AppContext.dll"));
    FileHelper.Delete(CombinePaths(folder, "System.Numerics.Vectors.dll"));
    FileHelper.Delete(CombinePaths(folder, "System.Runtime.InteropServices.RuntimeInformation.dll"));
    FileHelper.Delete(CombinePaths(folder, "System.ComponentModel.Primitives.dll"));
    FileHelper.Delete(CombinePaths(folder, "System.ComponentModel.TypeConverter.dll"));
    FileHelper.Delete(CombinePaths(folder, "System.Console.dll"));
    FileHelper.Delete(CombinePaths(folder, "System.IO.FileSystem.Primitives.dll"));
    FileHelper.Delete(CombinePaths(folder, "System.IO.FileSystem.dll"));
    FileHelper.Delete(CombinePaths(folder, "System.Security.Cryptography.Encoding.dll"));
    FileHelper.Delete(CombinePaths(folder, "System.Security.Cryptography.Primitives.dll"));
    FileHelper.Delete(CombinePaths(folder, "System.Security.Cryptography.X509Certificates.dll"));
    FileHelper.Delete(CombinePaths(folder, "System.Threading.Thread.dll"));
}

I make a PR

Contrary to the hypothesis make above, the error is not caused by Travis. Therefore I have done a git bisect.
I've found:
b240da171358daeff827d9f7160379bb95c696b1 is the first bad commit
commit b240da171358daeff827d9f7160379bb95c696b1
Author: @david-driscoll
Date:   Sat Jun 30 11:23:43 2018 -0400
Updated to latest msbuild / roslyn versions

I hope it will help to merge https://github.com/OmniSharp/omnisharp-roslyn/pull/1361 :-)

Any indications on when this issue may be fixed? OmniSharp is unusable for me on vim without this since I have projects referencing each other.

@satiani Even if this is fixed, will you still have confidence in a project that left Linux users high and dry for months on end?

@ohjames It's an open source project run by volunteers. If you can't make the time to help fix this issue, then at least avoid negative comments.

@satiani The issue has been fixed but nobody is listening. Just because people are volunteering their time doesn't make them immune to criticism. Had plenty of criticism on some of my own open source projects from time to time and when it's deserved I'm thankful that somebody booted me into action.

constructive criticism != cynicism

As @david-driscoll as merged #1361 (thanks :smile: ) this issue could be closed :smile:

@rchande @nickspoons maybe it could be cool to make a new release for vim users ?

@JulioJu might as well wait for a release so we can test it, before closing this issue? (I don't have any control over releases, I'm just an OmniSharp-vim maintainer)

You can already test it by grabbing the pre-release using these instructions https://github.com/OmniSharp/omnisharp-roslyn#prerelease-versions
In this case, it would be the version 1.32.12-beta.51 or higher.
In the project, we have the approach that issues are closed as soon as the fix is merged (not when it's released). Feel free to open a new one (or reopen this one) when you still have problems with the prerelease - thanks!

Success! Thanks @filipw I hadn't realised the prerelease-versions were available like that.

Still open in v1.32.18 release on Windows 7 x64 when used with omnisharp-vim.
Server run from command line with omnisharp -v works but outputs nothing when visited with the web browser.
These workarounds in omnisharp.exe.config and installing the associated .NET developer pack did not solve it.
<configuration>
  <startup>
    <supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.7.2" />

  <runtime>
    <loadFromRemoteSources enabled="true"/>


Add "onlyCheckBundledFiles" option to enable checking files which actually bundled by webpack only

Feature motivation
As webpack will analyse the dependency and only pack the actually files in bundle, addtional file settings(include files and exclude) of tsconfig.json is redundant in a sense.
I'd like to have a option to make the type checking only check the files which be bundled. With this option, I can switch between default behavior and  'only check bundled files' behavior.
This is meaningful for some complex project which use one code repo to pack different implementation versions by different webpack config. In these case, some .ts file is only implemented for one specifial version, and it has the same name with the one in other versions.  If it's impossible to config with tsconfig.json, the current/default behavior of checker will cause typechecking failure. But in bundled files view, the type checking is pass really.
Feature description
This option modify the default behavior of checking. Like ts-loader option, when this option set to true, the checking won't respect the include files and exclude options in tsconfig.json.
It will simplify the maintainance of the project. In another hand, this feature will accelerate type checking process.
Feature implementation
(PS: sorry about my English, wish I make it clear)

i actually think it was doing this by default as of version 6.0.8, but when we updated to 6.1.1 suddenly it's type checking our test files (which are not included in the webpack bundle) and failing due to a lack of Jest types.
@eagleliang could this actually be a regression?

or maybe it was previously somehow including the jest types automatically and now isn't? either way the behavior changed between those two versions

If you want to exclude some directories from type-checking, you can use ‘configOverwrite’ option and add directories to 'exclude' list :)

@piotr-oles totally i get that.
i'm just confused what changed because previously our test files were either not being included (which would make sense with only bundle files being included), or somehow the jest types were included alongside them and are not now. i'm just wondering which it is and if either is a regression bug?

No, it's not. It's a fix for an issue when type-only files were not part of the web pack compilation, therefore if you changed something in type-only file it didn't trigger type-check. Starting from version 5 we add all files from typescript project as webpack dependencies.

@piotr-oles thanks for the update. apologies for the noise. it looks like the error happens on 6.0.8 as well so it must be due to jest types somehow not being included for another reason. not on you guys.

@eagleliang I'm closing this issue because I don't think I will implement it. You can manually control the list of type-checked files by overwriting the include and exclude list :)

@eagleliang I'm closing this issue because I don't think I will implement it. You can manually control the list of type-checked files by overwriting the include and exclude list :)

Yes, but not really conveniently - if we want to type-check only the files which are bundled it's not trivial to do it via include and exclude. This feature would be really helpful. For example, imagine you may have some devDependency installed into node_modules which has TS code incompatible with the target environment of your lib. You won't bundle this dependency, but still get errors like Import assignment cannot be used when targeting ECMAScript modules. Excluding each such dependency via 'exclude' is cumbersome.

My colleague found https://github.com/TypeStrong/fork-ts-checker-webpack-plugin/issues/714 and it seems to achieve what we want :)


Cannot save comment on procedure (PostgreSQL)

DBeaver 7.0.2
When we have edited a comment in a procedure window and are going to save it, the resulting SQL comes out malformed (procedure parameters are missed in SQL).
COMMENT ON PROCEDURE public.test() IS 'Test';
This leads to the error "ERROR: procedure public.test() does not exist"
Expected SQL is
COMMENT ON PROCEDURE public.test("integer", "text") IS 'Test';
This bug is new to 7.0.2; it worked correct in 7.0.1.

thanks for the bug report

It is a side effect of #8169 fix. Procedure parameters are cleared on save.
Now fixed.
@uslss also please check that #8169 is ok.

verified both #8399 and #8169


Fix build fail

What problem does this PR solve?
Issue Number: close #6627
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/merge

/check-issue-triage-complete


refactor: introducing the installer

This PR introduces the Installer which handles executing transactions and more. The installer can be observed with the Reporter trait. A default implementation is provided which uses indicatif.

Feel free to review, but please don't merge yet. I wanna test this out in pixi first.

With the Python API it is now easier than ever to create a conda environment from scratch:
# Solve an environment with Python 3.9 for the current platform
records = await solve(channels=["conda-forge"], specs=["python=3.9"])

# Link the environment in the directory `my-env`.
await link(records, target_prefix="my-env")

# That's it! You now have a functioning environment with Python 3.9 in the "my-env" directory.

Renamed link to install

I tested it in pixi as well: https://github.com/prefix-dev/pixi/pull/1438
@wolfv feel free to merge if you have no new objections!


ci: Update to stable GHA versions

Add workflow dispatch triggers to run CI on demand.
Update actions to latest stable versions
Fix CI so that flake8 and safety checks actually run.

In the meantime this was fixed by @talvandaalen in PR #245. :+1:

Report coverage to Codecov for all the Python versions tested by separating uploads with version flags.

@gordonwatts this is ready for review. The CI isn't running for some reason on these PRs but I've enabled the workflows on my fork and here's a screenshot of the CI passing there for this:

To reduce commit noise here I'm just going to override with a push of my rebased local version as the extra spaces make it hard to track in the future what the motivation for this was.

It might be we are both hacking on this right now - I'm trying to get this to a point we can get it merged. :-)

It might be we are both hacking on this right now - I'm trying to get this to a point we can get it merged. :-)

Sure. Do you know why your branch merges are adding extra spaces that aren't there in master?
this PR with branch merge to master
> git diff origin/ci/run-flake8 upstream/master -- .github/workflows/pypi.yaml
diff --git a/.github/workflows/pypi.yaml b/.github/workflows/pypi.yaml
index f9fbad5..0ac11e7 100644
--- a/.github/workflows/pypi.yaml
+++ b/.github/workflows/pypi.yaml
@@ -9,21 +9,21 @@ jobs:
     runs-on: ubuntu-latest

     steps:
-      - uses: actions/checkout@v3
-      - name: Set up Python
-        uses: actions/setup-python@v4
-        with:
-          python-version: "3.10"
-      - name: Install dependencies
-        run: |
-          python -m pip install --upgrade pip build
-      - name: Build the servicex wheel
-        env:
-          servicex_version: ${{ github.ref }}
-        run: |
-          python -m build --sdist --wheel
-      - name: Publish servicex to PyPI
-        uses<EMAIL_ADDRESS>-        with:
-          password: ${{ secrets.pypi_password }}
-          print_hash: true
+    - uses: actions/checkout@v2
+    - name: Set up Python
+      uses: actions/setup-python@v2
+      with:
+        python-version: 3.7
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip build
+    - name: Build the servicex wheel
+      env:
+        servicex_version: ${{ github.ref }}
+      run: |
+        python -m build --sdist --wheel
+    - name: Publish servicex to PyPI
+      uses<EMAIL_ADDRESS>+      with:
+        user: __token__
+        password: ${{ secrets.pypi_password_servicex }}

vs. my local branch to master
> git diff upstream/master -- .github/workflows/pypi.yaml
diff --git a/.github/workflows/pypi.yaml b/.github/workflows/pypi.yaml
index 0ac11e7..2d49b57 100644
--- a/.github/workflows/pypi.yaml
+++ b/.github/workflows/pypi.yaml
@@ -9,11 +9,11 @@ jobs:
     runs-on: ubuntu-latest

     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
     - name: Set up Python
-      uses: actions/setup-python@v2
+      uses: actions/setup-python@v4
       with:
-        python-version: 3.7
+        python-version: "3.x"
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip build
@@ -23,7 +23,7 @@ jobs:
       run: |
         python -m build --sdist --wheel
     - name: Publish servicex to PyPI
-      uses<EMAIL_ADDRESS>+      uses<EMAIL_ADDRESS>       with:
-        user: __token__
-        password: ${{ secrets.pypi_password_servicex }}
+        password: ${{ secrets.pypi_password }}
+        print_hash: true


MultipleRegistry support cluster

What is the purpose of the change
多注册中心支持 zk 集群的配置
#13328

2.7.x is EOL now.


Advice for using a sidecar file with gridded data

I'm having problems correctly using a sidecar file for a gridded dataset (IMERG). Here is the generalized problem.

I make a sidecar file for IMERG using STAREMaster_py ...

    $ python create_sidecar_files.py --workers 4 --grid IMERG

    IMERG_stare.nc
        >>>> ulong STARE_index(i=1800, j=3600);

And this matches up with the dataset I'm aiming to pod:
    DYAMONDv2_PE3600x1800-DE.prectot.20200116_0000z.nc4
        >>>> float PRECTOT(time=1, lat=1800, lon=3600);

But when I attempt to form a STAREPandas dataframe using it as so...

    gdf = starepandas.read_granule(granule_path, sidecar=True,
                                   latlon=False, read_timestamp=False,
                                   sidecar_path=sidecar_path,
                                   add_sids=False, adapt_resolution=True)

I get an error which leads me to that think I'm likely evoking the starepandas.read_granule() method incorrectly for this kind of sidecar (gridded data). Inside said method, I see that it gets the correct sidecar info, although the self.nom_res  = None parameter might be an issue.
    granule.read_sidecar_index():
        ds = <class 'netCDF4._netCDF4.Dataset'>
        ds.dimensions = 'i'          : size = 1800
                        'j'          : size = 3600
                        'l'          : size = 8
        ds.variables  = 'STARE_index': uint64 STARE_index(i, j)
                        'STARE_cover': uint64 STARE_cover(l)
        self.nom_res  = None

Here is where the error is raised (in starepandas.read_granule()).
    try:
        self.sids = ds['STARE_index_{}'.format(self.nom_res)][:, :].astype(numpy.int64)
    except IndexError:
        # If we don't have a nomres?
        >>>> 'STARE_index{}'.format(self.nom_res) = 'STARE_indexNone'
        self.sids = ds['STARE_index{}'.format(self.nom_res)][:, :].astype(numpy.int64)

        >>>> STAREPandas/starepandas/io/granules/granule.py", line 66, in read_sidecar_index
        >>>>   self.sids = ds['STARE_index{}'.format(self.nom_res)][:, :].astype(numpy.int64)
        >>>>   IndexError: STARE_indexNone not found in /

Any suggestions? Thanks.

Note, starepandas.read_granule() is inherited from the Granule class via a class I created) as should be clear from the code snippet below.
STAREPandas/starepandas/io/granules/
    __init__.py:
        from .imergl3 import L3IMERG, DYAMONDv2

        Added
            'L3IMERG': L3IMERG,
            'DYAMONDv2': DYAMONDv2
        to granule_factory_library

    imergl3.py:

        # Standard Imports
        import os
        import datetime

        # Third-Party Imports
        import numpy

        # STARE Imports
        from starepandas.io.granules.granule import Granule
        import starepandas

        class L3IMERG(Granule):
            def __init__(self, file_path, sidecar_path=None):
                # Use Granule.__init__() for this instance (self)
                super().__init__(file_path, sidecar_path)
                ##
                # Opens file_path using netCDF4
                self.netcdf = starepandas.io.s3.nc4_dataset_wrapper(self.file_path, 'r', format='NETCDF4')

            # Not implemented yet
            # def read_timestamps(self):
            #    self.ts_start = self.netcdf.time_coverage_start
            #    self.ts_end = self.netcdf.time_coverage_end

            def read_latlon(self):
                self.lat = self.netcdf['lat'][:].astype(numpy.double)
                self.lon = self.netcdf['lon'][:].astype(numpy.double)

        class DYAMONDv2(L3IMERG):
            """Special case for IMREG precip files with file names like
                DYAMONDv2_PE3600x1800-DE.prectot.20200116_0000z.nc4
            """
            def __init__(self, file_path, sidecar_path=None):
                # Use L3IMERG.__init__(), which calls Granule.__init__() for this instance (self)
                super().__init__(file_path, sidecar_path=sidecar_path)

Then in my code:
    from STAREpodder.cfg.def_IMERGPF import IMERGPF

    starepandas.io.granules.granule_factory_library['DYAMONDv2'] = IMERGPF

    gdf = starepandas.read_granule(granule_path, sidecar=True,
                               latlon=False, read_timestamp=False,
                               sidecar_path=sidecar_path,
                               add_sids=False, adapt_resolution=True)

I should add some potential dependency issues that might be at work here (e.g., Pandas 2.0).

Name
Version

astropy
5.2.2

cartopy
0.21.1

gdal
3.6.3

geopandas
0.12.2

geopandas-base
0.12.2

geos
3.11.2

h5py
3.8.0

hdf4
4.2.15

hdf5
1.12.2

hdfeos2
2.20

matplotlib
3.7.1

numpy
1.24.2

pandas
2.0.0

proj
9.1.1

pygeos
0.14

pyhdf
0.10.5

pyproj
3.5.0

pyshp
2.3.1

pytest
7.3.1

python
3.11.3

shapely
2.0.1

STARE
Installs

pystare
0.8.12 STARE

staremaster
0.0.4  STARE

starepandas
0.6.6  STARE

This is a bug. the nom_res is None, which should be mapped to a null string, not to the string "None". As a workaround, try setting the read_granule argument nom_res="".
Granules can have different resolution data, which implies a need to select between them. This is done via the argument nom_res, which stands for nominal_resolution. It is a string that is appended to the canonical variable names, i.e. if nom_res='750km', you'd refer to latitude_750km or latitude750km.
The fix for the Granule class is to check nom_res for None and set to "", if so.

Okay, I was toying with the "fix" to put something like:
nom_res_str = "" if non_res is None else non_res

at the top of read_granule().

Hmm, I think I need a bit more help. So the previous problem is solved, but now I get an error forming the STARE dataframe.
gdf = starepandas.read_granule(granule_path, sidecar=True,
                               latlon=False, read_timestamp=False,
                               sidecar_path=sidecar_path,add_sids=False,
                               adapt_resolution=True, nom_res=None)

in granule.to_df(xy=False):
        to_df(): self.lat.shape = (1800,)            flattened (1800,)
        to_df(): self.lon.shape = (3600,)            flattened (3600,)
        to_df(): self.sids.shape = (1800, 3600)      flattened (6480000,)
        to_df(): dtype = dtype('float64')
        to_df(): self.data[key].shape = (1800, 3600) flattened (6480000,)
        to_df(): dtype = dtype('float64')
        to_df(): series.shape = (6480000,)

raise ValueError("All arrays must be of the same length")
ValueError: All arrays must be of the same length

I haven't implemented the temporal part of the inputs yet, but I don't see how that gives this error.

The dataframe (Granule class I think) doesn't know how to integrate the 1D lat and lon and the 2D sids into a single frame. to_df will have to be specialized (or read_granule) to rectify lat and lon to the same size and shape as sids. This is a common issue if one has a simple 2D regular grid and one only keeps the 1D forms of the dimensional coordinates.
Alternatively, the sidecar constructor could write 2D lat and lon coordinate variables (commensurate with the data) and the existing Granule class (which was originally developed for Level 2 non-gridded data) would work. This approach takes more space in storage, but each grid type only needs one sidecar file.

The dataframe (Granule class I think) doesn't know how to integrate the 1D lat and lon and the 2D sids into a single frame. to_df will have to be specialized (or read_granule) to rectify lat and lon to the same size and shape as sids. This is a common issue if one has a simple 2D regular grid and one only keeps the 1D forms of the dimensional coordinates.
Alternatively, the sidecar constructor could write 2D lat and lon coordinate variables (commensurate with the data) and the existing Granule class (which was originally developed for Level 2 non-gridded data) would work. This approach takes more space in storage, but each grid type only needs one sidecar file.

Okay,
So something like
##
# Make a (lat=1800, lon=3600) mesh grid to match the sidecar SIDs array
#   Use Cartesian indexing lat[i, j]
lon, lat = numpy.meshgrid(lon, lat, copy=True, indexing='xy')

sids (1800, 3600)
lat (1800) = [ -89.950 ...  +89.950]
lat (3600) = [-179.950 ... +179.950]
lat (1800, 3600)
lon (1800, 3600)

    i     j   lat     lon
[   0,    0]: -89.950 -179.950
[   0,    1]: -89.950 -179.850
[   0,    2]: -89.950 -179.750
...
[   0, 1797]: -89.950   -0.250
[   0, 1798]: -89.950   -0.150
[   0, 1799]: -89.950   -0.050

...

[1799,    0]: +89.950 -179.950
[1799,    1]: +89.950 -179.850
[1799,    2]: +89.950 -179.750
...
[1799, 3597]: +89.950 +179.750
[1799, 3598]: +89.950 +179.850
[1799, 3599]: +89.950 +179.950

Yes, something along those lines should work.

@NiklasPhabian I'm working on a fix.


ORTools-Example 2-Parent Grain Reconstruction in Titanium alloys error

Hello.
I have trouble when I practice with ORTools-Example2-Paraent grain reconstruction in titanium alloys
Actually, the it is the same problem with Naga Chaitanya in comment.
I am also when I doing peak fitting(3:36 of video)
Undefined function 'findpeaks' for input arguments of type
'double'.
Error in gaussFit (line 46)
[peakValue,peakPositionIdx,peakWidth,peakHeight] = findpeaks(y,
'MinPeakDist',classInterval.fine,
'MinPeakHeight',ypos,'Annotate','extents','WidthReference','halfheight');
Error in defineORs (line 45)
[misoRange] =
gaussFit(classRange,classInterval,counts);
Error in ORTools_example2 (line 54)
job = defineORs(job);
this error is occurred.
How can I solve this problem?
thank you and have a nice day

Hi,
as stated in the readme, you need the Signal Processin Toolbox for the peak fitting functionality. The command which findpeaks in MATLAB returns the following on my system:
C:\Program Files\MatLab R2020a\toolbox\signal\signal\findpeaks.m
This tells me that the function exists and that it is part of the signal processing toolbox in MATLAB. If you don't get any return value, you don't have the toolbox installed and cannot use this functionality.
Cheers
Frank

I am really appreciated that your help and I have solved the problem!
Thanks


chore(deps): bump dependencies

build(deps): bump k8s.io/client-go from 0.25.4 to 0.26.1 in /operator
build(deps): bump sigs.k8s.io/controller-runtime in /operator

Superseeds https://github.com/SumoLogic/tailing-sidecar/pull/463 and https://github.com/SumoLogic/tailing-sidecar/pull/454


Feature request: Admin and non-admin modes

The UI should support admin and non-admin modes so that we can create two types of users:

Admin - Can create/update workspaces, add/delete files etc.
Non-admin - Can use the chatbot but cannot modify the workspace/files.

Closing this as similar request has been raised already


Create mongo image

Description
This pr creates a simple conf of mongo image
Fixes #5
Type of change

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

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

The build has not passing, because i do not setted setup linter, neither tests.


Fix the regression: fail make test on some parallel case.

This is regression by https://github.com/karen-irc/option-t/pull/299

This change is

bors r+


fix: 合并dev，修复合并dev后租户组件在else中不加载的问题

PR 类型

[ ] 新 feature
[x] Bug 修复
[ ] 功能增强
[ ] 文档变更
[ ] 代码样式变更
[ ] 重构
[ ] 性能改进
[ ] 单元测试
[ ] CI/CD
[ ] 其他

PR 目的

解决方案

PR 测试

Changelog

模块
Changelog
Related issues

其他信息

提交前确认

[x] PR 代码经过了完整测试，并且通过了代码规范检查
[ ] 已经完整填写 Changelog，并链接到了相关 issues
[ ] PR 代码将要提交到 dev 分支

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 3 committers have signed the CLA.:white_check_mark: xtanyu:x: AiMing317:x: Charles7cYou have signed the CLA already but the status is still pending? Let us recheck it.


Failed to precompile liquidart

my system is : Flutter 2.0.1 • channel stable • Dart SDK version: 2.12.0 (stable) on "macos_x64",
I just command with

pub global activate liquidart

, it failed on precompile with this message:

_fe_analyzer_shared 14.0.0 (19.0.0 available)
analyzer 0.41.2 (1.3.0 available)
args 1.6.0 (2.0.0 available)
async 2.5.0
buffer 1.1.0
charcode 1.2.0
checked_yaml 1.0.4 (2.0.1 available)
cli_util 0.3.0
codable 1.0.0 (discontinued)
collection 1.15.0
convert 2.1.1 (3.0.0 available)
crypto 2.1.5 (3.0.1 available)
file 6.1.0
glob 2.0.1
isolate_executor 2.0.2+3 (discontinued)
json_annotation 4.0.1
liquidart 4.0.1-b2
logging 0.11.4 (1.0.1 available)
meta 1.3.0
open_api 2.0.1 (discontinued)
package_config 1.9.3 (2.0.0 available)
password_hash 2.0.0 (discontinued)
path 1.8.0
pedantic 1.11.0
postgres 2.2.0 (2.3.1 available)
pub_cache 0.2.3 (0.3.0 available)
pub_semver 1.4.4 (2.0.0 available)
pubspec_parse 0.1.8 (1.0.0 available)
runtime 1.0.0-9 (discontinued)
safe_config 3.0.0-b2 (discontinued)
source_span 1.8.1
string_scanner 1.1.0
term_glyph 1.2.0
typed_data 1.3.0
watcher 1.0.0
yaml 2.2.1 (3.1.0 available)
Downloading liquidart 4.0.1-b2...
Downloading safe_config 3.0.0-b2...
Downloading runtime 1.0.0-9...
Downloading password_hash 2.0.0...
Downloading open_api 2.0.1...
Downloading codable 1.0.0...
Downloading yaml 2.2.1...
Downloading pub_cache 0.2.3...
Downloading logging 0.11.4...
Downloading pub_semver 1.4.4...
Downloading pubspec_parse 0.1.8...
Downloading checked_yaml 1.0.4...
Downloading isolate_executor 2.0.2+3...
Downloading args 1.6.0...
Downloading postgres 2.2.0...
Downloading buffer 1.1.0...
Downloading crypto 2.1.5...
Downloading charcode 1.2.0...
Downloading convert 2.1.1...
Downloading path 1.8.0...
Downloading json_annotation 4.0.1...
Downloading string_scanner 1.1.0...
Downloading source_span 1.8.1...
Downloading collection 1.15.0...
Downloading term_glyph 1.2.0...
Downloading typed_data 1.3.0...
Downloading meta 1.3.0...
Downloading analyzer 0.41.2...
Downloading _fe_analyzer_shared 14.0.0...
Downloading cli_util 0.3.0...
Downloading package_config 1.9.3...
Downloading glob 2.0.1...
Downloading async 2.5.0...
Downloading pedantic 1.11.0...
Downloading file 6.1.0...
Downloading watcher 1.0.0...
Precompiling executables... (7.1s)
Failed to precompile liquidart:liquidart:
../../../../../.pub-cache/hosted/pub.dartlang.org/isolate_executor-2.0.2+3/lib/src/source_generator.dart:6:8: Error: Error when reading '../../../../../.pub-cache/hosted/pub.dartlang.org/analyzer-0.41.2/lib/analyzer.dart': No such file or directory
import 'package:analyzer/analyzer.dart';
^
../../../../../.pub-cache/hosted/pub.dartlang.org/isolate_executor-2.0.2+3/lib/src/source_generator.dart:52:17: Error: Type 'ClassDeclaration' not found.
static Future _getClass(Type type) async {
^^^^^^^^^^^^^^^^
../../../../../.pub-cache/hosted/pub.dartlang.org/isolate_executor-2.0.2+3/lib/src/source_generator.dart:54:22: Error: Method not found: 'parseDartFile'.
final fileUnit = parseDartFile(uri.toFilePath(windows: Platform.isWindows));
^^^^^^^^^^^^^
../../../../../.pub-cache/hosted/pub.dartlang.org/isolate_executor-2.0.2+3/lib/src/source_generator.dart:58:28: Error: 'ClassDeclaration' isn't a type.
.where((u) => u is ClassDeclaration)
^^^^^^^^^^^^^^^^
../../../../../.pub-cache/hosted/pub.dartlang.org/isolate_executor-2.0.2+3/lib/src/source_generator.dart:59:28: Error: 'ClassDeclaration' isn't a type.
.map((cu) => cu as ClassDeclaration)
^^^^^^^^^^^^^^^^

@xijun-j
Thanks for reporting this. Currently, the packages shown as discontinued are being migrated to sound nullsafety, and will soon be published as new packages on pub.dev.
Please leave this issue open. I will update the thread when the migration and publishing is complete.

@xijun-j
I just tested the pub global activate liquidart with the latest version of the package and it is precompiling successfully. Make sure you are using the latest version of liquidart (5.0.0-b2) with Sound Null Safety. The original package of liquidart (which was called aqueduct) was so outdated (Last update in 2018) and its dependencies were also outdated that I had to migrate all those packages to Null Safety, just to make it compatible. So I cannot provide a non Null-Safe version of the package.
I am closing this issue as it has been resolved. Let me know if there is anymore problems on your end. Thanks again for reporting this.


Non-ergonomic display of long metadatavalue without space on admin metadata + bitstream edit AND public item pages

Describe the bug
When an item has a very long metadata value without space (e.g. a long URL stored in dc.identifier.uri for example), the display of the admin metadata edit is not ergonomic, as in the admin user has to scroll to the right to be able to see the edit/delete buttons for ALL metadata of the item.

This was tested on both Firefox and Chrome on demo 7 (7.2-SNAPSHOT at the time of writing this comment).
To Reproduce
Steps to reproduce the behavior:

Go to demo7
Log in as admin user
Edit any random item
Add a new metadata (e.g. dc.identifier.uri) with a very long metadata value without space
Check the impact on the display after adding this metadata

Expected behavior
A line break should be forced as from a certain width, to keep the buttons on the right side of the metadata always visible.
Related work
NA

Thanks @davidatmire , I agree this is a usability bug we should look at fixing soon.  For now, pulled it over to the 7.2 board to see if we can find someone to look at this for 7.2.  If your team is interested (or anyone else reading this later), let me know & please provide a rough estimate of the effort (in hours).

@tdonohue thank you for your prompt answer.
When checking with @artlowel, we have noticed that the same issues appear at different places :

Admin metadata edit (mentioned in my previous comment)
Admin bitstream edit:

Item view, both at the level of metadata and bitstream names:

The solution needs to be determined on a case-by-case basis and is going to be a compromise between completeness and usability (e.g. for admin pages it may be ok to simply cut off words halfway, for public facing pages we may need a more complicated solution).
Estimate : 4 hours to identify and implement the ad-hoc compromise solution for the cases mentioned above.

Thanks @davidatmire : I'll add your 4hrs estimate to this ticket & assign this back to you for implementation. Please feel free to delegate as needed to someone else within Atmire.


Broken git behaviour if authentication is needed

If I need to authenticate with GitHub it seems this prompt is ignored by Monokle - resulting in broken git behavior... for example the below project should detect that I already have a git repo configured for this folder - but the prompt to sign in is ignored by Monokle:

Easy workaround is to authenticate from the command-line or using something like https://github.com/GitCredentialManager/git-credential-manager before running Monokle

Need additional investigation


Adds navigation task (Spatial Reasoning)

This tasks tests the ability of neural language models to complete a simple spatial reasoning task. Given a series of navigation steps, the task is to determine whether or not an agent would end up back at the starting point.
Natural language often implicitly encodes information about the state of the world. Humans are able to maintain and update mental models of this state, which allows us to reason and plan. It is unclear to what degree neural language models -- which in contrast to symbolic models lack formal, logical representations of state -- are able to maintain "correct" world representations. We test a specific subset of this general concept within the context of navigation.

@googlebot I signed it!

@sosuperic We are now requesting that task authors please include an explicit section in the README.md file called "data source", listing exactly where you got your data from (or explicitly state that you made it up yourself). Many thanks!

@jaehlee are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

@sschoenholz are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

I really like this task! The idea of using navigation to probe the ability of language models to carry internal state and reason about it is clever and the motivation is made clear in the README description. I also really like how many examples you have included and how you span a range of difficulties. Finally, I appreciate that you have taken care to balance the dataset since generic paths must almost not return to the origin. I don't know how possible this is inside the BIG-bench API, but it might be nice to split the results by task difficulty. Can this be done by splitting the task into subtasks by whether the agent is always facing forward? One other question I have is why the authors chose not to balance the dataset across number of instructions? As written it seems like performance will be dominated by the 4-7 instruction examples.
Review Criteria

Correctness: The task is properly formatted and passes tests.
Formatting: The task formatting is clear and I had no problem interacting with it in the human interaction colab notebook.
Specificity: The motivation is clear and is captured in the README.
Thoroughness: The authors did a great job of thinking about confounders (balance and difficulty scale) and taking it into account. I do think it might be nice to have some way of seeing performance as a function of difficulty. There are also plenty of examples since they are generated synthetically.
Difficulty: While many of the easy examples seem quite simple, the difficulty definitely scales significantly as the number of instructions increases. From the authors' benchmarks current models are completely unable to solve the task.
Not solvable by memorizing the internet: Models cannot solve this task by memorizing the internet since the examples are generated synthetically.
Novelty: I think this task is fairly novel. As the authors note there is some overlap with arithmetic. Among tasks currently in BIG-bench it seems like the most similar task is the "Text Navigation Game" task, however they are very different (both in scope and format). It might be nice to discuss these similarities / differences in the README.
Justification: I think the README does a great job of explaining what the task is trying to measure and justifying why it is an important capability.
Size: The size of the task seems appropriate (with 1000 examples). Certainly enough examples to easily measure statistically significant signal.
Compute Resources: Although there are a lot of examples, each example is short. I imagine if the compute resources are too large some examples could be cut.

@sschoenholz thanks for the review and comments! Quick response right now:

Not sure about the BIG-Bench API either, but I can look into it.
Re: balancing dataset across number of instructions. That's a good point. No real reason not to balance it -- I can make that update pending any other comments.

The amount of tasks we received this round is tremendous.  With so many tasks to review we have decided to extend the review period to Tuesday, June 29.
Reviewers: if the submitter has made revisions please be sure to notify me with a formal ”accept” or ”does not meet criteria” by Tuesday, June 29  (if you haven't already done so).
this is an automated message

I am one of the reviewers and thank you for submitting the task. This task tests the ability of language model to complete a simple spatial reasoning task. Generally, I agree with @sschoenholz's assessment and agree that the task is nice and well thought out!
It'd be interesting to structure the task into subtasks with increasing difficulty as suggested by @sschoenholz, although in principle this could be custom analyzed with n_sentences field for each example is provided.

@chiafullo accept

@chiafullo accept

Your task has been accepted in the initial review stage.  The next stage of this process is for us to assign your task a meta-reviewer for final review and merge.  An assigned meta-reviewer will follow-up by commenting on the PR should it need further revisions.

Hello @sosuperic, I'm the meta-reviewer for this task. Thank you for this contribution! I will go ahead and merge.

@sosuperic - Also noting a non-blocking grammatical issue.
In principle the natural answers to the task_prefix, "If you follow these instructions, do you return to the starting point" should be "Yes" and "No", rather than "True" and "False". You might consider changing the choice values accordingly.
Alternatively you might consider changing the task prefix to "True or false: If you follow these instructions you will return to the starting point."


There are trace reuse problems in Kafka consumption

Describe the bug
When kafka consumption is performed, if there is no Trace information in the upstream, the consumer automatically creates a Trace. The Trace will not end after the call is completed, but will be reused along with the thread

Can you please answer all the required questions described in the issue template? For your convenience let me paste it here

Describe the bug
Please provide details of the problem, including the version of Spring Cloud that you
are using.
Sample
If possible, please provide a test case or sample application that reproduces
the problem. This makes it much easier for us to diagnose the problem and to verify that
we have fixed it.

use spring-boot version is 2.6.3 , sleuth  version is 3.1.0  ,kafka version is 2.8.2
#########provider (There is no trace for this project)############
@GetMapping("/test")
public void test() {
/*
* No Trace is set in this Kafka message
* */
kafkaTemplate.send("test_sleuth_topic", "test......");
}
##########consumer (this project has trace ) ###########
@KafkaListener(topics = "test_sleuth_topic")
public void listenDelete11(ConsumerRecord consumer, Acknowledgment ack) {
try {
log.info("consumer satrt .......");
} finally {
log.info("consumer end .....");
ack.acknowledge();
}
}

Can you provide a complete, minimal, verifiable sample that reproduces the problem rather than pasted code? It should be available as a GitHub (or similar) project or attached to this issue as a zip file.


Can not serve both http and websocket request on one server?

Describe the bug
When serve both http and websocket request on one server, can not connect websocket.
When connect websocket in command line using python websockets(v7.0) package.
➜  web python3 -m websockets wss://localhost:8000/ws/
Failed to connect to wss://localhost:8000/ws/: .

Got the following error in sanic console:
[2019-05-28 13:54:07 +0800] [50524] [ERROR] Transport closed @ ('<IP_ADDRESS>', 65350) and exception experienced during error handling
[2019-05-28 13:54:07 +0800] [50524] [DEBUG] Exception:
Traceback (most recent call last):
  File "/Users/jagger/projects/dingpan/server/venv/lib/python3.7/site-packages/sanic/server.py", line 267, in data_received
    self.parser.feed_data(data)
  File "httptools/parser/parser.pyx", line 193, in httptools.parser.parser.HttpParser.feed_data
httptools.parser.errors.HttpParserInvalidMethodError: invalid HTTP method

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/jagger/projects/dingpan/server/venv/lib/python3.7/site-packages/sanic/server.py", line 515, in write_error
    self.transport.write(response.output(version))
AttributeError: 'coroutine' object has no attribute 'output'
/Users/jagger/projects/dingpan/server/venv/lib/python3.7/site-packages/sanic/server.py:272: RuntimeWarning: coroutine 'handle_exception' was never awaited
Coroutine created at (most recent call last)
  File "/Users/jagger/.vscode/extensions/ms-python.python-2019.4.12954/pythonFiles/lib/python/ptvsd/__main__.py", line 317, in run_module
    run_module_as_main(target, alter_argv=True)
  File "/usr/local/Cellar/python/3.7.1/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/local/Cellar/python/3.7.1/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/Users/jagger/projects/dingpan/server/dingpan/app.py", line 45, in <module>
    workers=config['WORKERS'], protocol=WebSocketProtocol)
  File "/Users/jagger/projects/dingpan/server/venv/lib/python3.7/site-packages/sanic/app.py", line 1096, in run
    serve(**server_settings)
  File "/Users/jagger/projects/dingpan/server/venv/lib/python3.7/site-packages/sanic/server.py", line 770, in serve
    loop.run_forever()
  File "/Users/jagger/projects/dingpan/server/venv/lib/python3.7/site-packages/sanic/websocket.py", line 52, in data_received
    super().data_received(data)
  File "/Users/jagger/projects/dingpan/server/venv/lib/python3.7/site-packages/sanic/server.py", line 272, in data_received
    self.write_error(InvalidUsage(message))
  File "/Users/jagger/projects/dingpan/server/venv/lib/python3.7/site-packages/sanic/server.py", line 513, in write_error
    response = self.error_handler.response(self.request, exception)
  File "/Users/jagger/projects/dingpan/server/venv/lib/python3.7/site-packages/sanic/handlers.py", line 129, in response
    response = handler(request, exception)
  self.write_error(InvalidUsage(message))

Code snippet
from sanic import Sanic
from sanic.websocket import WebSocketProtocol

from .config import config, log_config
from .models import init_db, close_db, init_cache, close_cache
from .blueprints import (handle_exception, message_v1, ws, order_v1, stock_v1,
                         user_v1, wallet_v1)

app = Sanic(config['NAME'].capitalize(), log_config=log_config)
app.config.update(config)

app.error_handler.add(Exception, handle_exception)

app.blueprint(message_v1)
app.blueprint(order_v1)
app.blueprint(stock_v1)
app.blueprint(user_v1)
app.blueprint(wallet_v1)

app.add_websocket_route(ws, '/ws')

...

if __name__ == '__main__':
    app.run(host=config['HOST'], port=config['PORT'], debug=config['DEBUG'],
            auto_reload=config['AUTO_RELOAD'], access_log=config['ACCESS_LOG'],
            workers=config['WORKERS'], protocol=WebSocketProtocol)

Environment (please complete the following information):

OS: masOS
Version: 19.3.1

I already tried to force update websocket version to v7.0, even though Sanic require v6.0.

Are you having trouble connecting?
Here is a pretty simple example:
from sanic import Sanic, response

app = Sanic("request_stream")

@app.route("/text")
def handler_text(request):
    return response.text("Hello")

@app.websocket("/ws")
async def handler_ws(request, ws):
    print("Websocket opened")
    name = "<someone>"
    while True:
        data = f"Hello {name}"
        await ws.send(data)
        name = await ws.recv()

        if not name:
            break

if __name__ == "__main__":
    app.run(port=7777, debug=True)

As you can see from the output, I have no problem hitting the /text route even after a websocket is opened.
[2019-05-28 09:47:12 +0300] [29726] [INFO] Goin' Fast @ http://<IP_ADDRESS>:7777
[2019-05-28 09:47:12 +0300] [29728] [INFO] Starting worker [29728]
Websocket opened
[2019-05-28 09:47:20 +0300] - (sanic.access)[INFO][<IP_ADDRESS>:58448]: GET http://localhost:7777/text  200 5

Now, it looks like you are trying to use websockets as a CLI (a v7 feature) in the same virtual environment as your Sanic app. If you switch to a different virtualenv that has websockets 7, you can use that:
$ python3 -m websockets ws://localhost:7777/ws
Connected to ws://localhost:7777/ws.
< Hello <someone>
> Adam
< Hello Adam

On a side note, you do not need to manually configure protocol=WebSocketProtocol. Sanic will automatically set it up once you add a websocket route. ;-)
I am closing this for now because it does not appear to be a Sanic issue. Please feel free to continue the conversation.


[doc] Provide startup guide

The package has no startup guides so users might not know how to get started.

Thank you for this. I'll get to it


Drop support for Julia 1.5 and older

This seemed like the easier options since Documenter dropped these Julia version a couple weeks ago and is now causing compat issues.
https://github.com/invenia/FeatureTransforms.jl/actions/runs/2786559009

Looks like coverage reporting might be different across Julia releases? Also, I think the Julia 1.5 jobs is stuck cause it's required on main? I believe that should go away once merged?

This seems to work when the Julia 1.5 test isn't included: https://github.com/invenia/FeatureTransforms.jl/actions/runs/2791698474
And coverage seems to only be an issue because there are 5 less lines of code: https://github.com/invenia/FeatureTransforms.jl/pull/110/checks?check_run_id=7658996965

Small question - Is this actually a non-breaking release?
I had done something similar for 0.3 for NamedDims and someone commented that it's a minor release.
https://github.com/invenia/NamedDims.jl/pull/202#discussion_r937572855
(I've got to here because I'm trying to see which packages need NamedDims 1 and I also encountered this package due to a resolution issue from one of our internal packages)

Small question - Is this actually a non-breaking release?

Yes, indeed it is, sorry if it caused trouble. I imagine it can be resolved by bumping compat in a few places?


Remove tokio dependency

Substituted with a manually chunked std::thread approach.
While this does use a different dependency, it's significantly less dependencies overall.
Currently, this decreases performance by a couple milliseconds on my machine, but hopefully I will fix this in the next few commits.
I'm going to create some benchmarks to calculate the best buffer size for the channel as well.

Hi Spaceface16518.
Thank you for your merge request. I'm 100% open to remove the tokio dependency but I'll not do it while it decreases the performance.
By the way, I received a comment on the reddit thread that mention a 60% performance boost with a simple modification. That could be an interesting approach to evaluate: https://www.reddit.com/r/rust/comments/jyziio/a_fractal_i_rendered_with_rust_without_any/gd8xvl2?utm_source=share&utm_medium=web2x&context=3

I'm 100% open to remove the tokio dependency but I'll not do it while it decreases the performance.
￼
I agree, I'm working on performance right now.

By the way, I received a comment on the reddit thread that mention a 60% performance boost with a simple modification. That could be an interesting approach to evaluate: https://www.reddit.com/r/rust/comments/jyziio/a_fractal_i_rendered_with_rust_without_any/gd8xvl2?utm_source=share&utm_medium=web2x&context=3

This change uses the rayon library. While it would be a lighter dependency than tokio (tokio itself depends on rayon) but I still feel like avoiding dependencies that do everything for you is a goal here (even if the Go standard library does everything for you).


[prim_present/otp_ctrl] Add round index state IOs to primitive

This addresses #3784.
Signed-off-by: Michael Schaffner<EMAIL_ADDRESS>

Hey @udinator, this probably depends a bit how coverage will be affected by this. But from my side I think it is ok to just test the static version.


feat: creating random generators

Codecov Report

Merging #18 (c3c1c5f) into main (fe886d8) will decrease coverage by 0.50%.
The diff coverage is 95.65%.

@@            Coverage Diff             @@
##             main      #18      +/-   ##
==========================================
- Coverage   96.15%   95.65%   -0.51%
==========================================
  Files           5        7       +2
  Lines          26       46      +20
  Branches        1        1
==========================================
+ Hits           25       44      +19
- Misses          1        2       +1

Impacted Files
Coverage Δ

lib/services/pokemon/model.ts
94.44% <92.30%> (-5.56%)
:arrow_down:

lib/services/pokemon/stat.ts
100.00% <100.00%> (ø)

lib/services/pokemon/types.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


add new scalability consideration

IoTHubQuotaExceed

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/iot/data-ingestion.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update sites.yml

This PR is:

[x] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the Cloudflare report
[ ] This site is not an ultra minimal site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: claudiu-ivan.com
  url: https://www.claudiu-ivan.com/
  size: 20.93
  last_checked: 2024-07-14

Cloudflare Report: https://radar.cloudflare.com/scan/5a50cece-f44a-4916-99a4-6bbaa29b10fa

I'm sorry, but I have to categorize this as ultra minimal, as defined by our FAQ. Thanks anyway!


Update financial-consolidations-currency-translation.md

Updating some lines on the document for better accuracy.

@aprilolson - Can you please review and verify that these updates are correct?

Please assign this to @Jinnie<EMAIL_ADDRESS>as she owns both of these areas. I’ll work with her on any updates and questions.
April Olson
Dynamics 365 Finance
Principal Group Product Manager
Check out our Dynamics 365 Finance bloghttps://community.dynamics.com/365/financeandoperations/b/dynamics-365-finance-blog.
Our workplace allows for a flexible work schedule, which is an amazing benefit. I am sending this now because it is convenient for me. I do not expect you to read or respond until it’s convenient for you.
From: Tonya Fehr<EMAIL_ADDRESS>Sent: Friday, September 18, 2020 8:24 AM
To: MicrosoftDocs/dynamics-365-unified-operations-public<EMAIL_ADDRESS>Cc: April Olson<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/dynamics-365-unified-operations-public] Update financial-consolidations-currency-translation.md (#2742)
Merged #2742https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-unified-operations-public%2Fpull%2F2742&data=02|01|aolson%40microsoft.com|d212a8d4079d4d19752a08d85bd6208d|72f988bf86f141af91ab2d7cd011db47|1|0|637360323218428941&sdata=zEIpsCCqbr0ye1FUFOD598tuH9wY0b%2FIGJT3fsXC8Bg%3D&reserved=0 into live.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-unified-operations-public%2Fpull%2F2742%23event-3782596681&data=02|01|aolson%40microsoft.com|d212a8d4079d4d19752a08d85bd6208d|72f988bf86f141af91ab2d7cd011db47|1|0|637360323218438935&sdata=lsvZkSVtsaEOBchCm4IC%2BuhHSjY%2BiqekgtEdJ7uw9i4%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAF2E2XOREGCSTDDZQB3BPRLSGNNPTANCNFSM4RAEW5JA&data=02|01|aolson%40microsoft.com|d212a8d4079d4d19752a08d85bd6208d|72f988bf86f141af91ab2d7cd011db47|1|0|637360323218438935&sdata=6h3NlGo8PrQpTA6XnJd5TQbuta2w3matrawAxjISlJs%3D&reserved=0.


Initial commit for security related config rules.

Initial commit of security related config rules including language breakouts.

I left some feedback in the acorley-development branch


Added caching support

Review #1 first and then I can rebase this on that one so that it is smaller and easier to review.

add support for caching previously loaded data
add support for populating the cache with static data via the cacheData static method, clear it again with the clearCache static method, and checking to see if the data is already loaded with the checkCache method
data can be added to the cache by loading it from files (sync or async) or by the cacheData method. The ensureLocale method is not implemented yet. (Yet another PR to come.)

Everything is in #3 instead


Misc prowler fixes

Add GetEbsEncryptionByDefault wherever Prowler policies are mentioned
Update Extra718 check to be aware of access denied responses
Update Extra726 check to be more verbose for non-failure items
Update Extra73 check to be aware of access denied responses
Update Extra734 check to be aware of access denied responses and parse policies with jq for better accuracy
Update Extra742 check for verbiage
Update Extra756 check for verbiage and parameter order
Update Extra761 check for failure scenarios (requires most recent awscli and addition to Prowler IAM policy)
Added Extra763 check to verify that object versioning is enabled on S3 buckets
Added Extra764 check to verify that S3 buckets enforce a secure transport policy

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@toniblyx I've opened a new pull request for Prowler

wow! Thanks for this amazing PR @zfLQ2qx2 I´ll review it as soon as I can. I gave it a quickly review and it looks awesome, thanks!

Your suggested extra764 seems to do the same as extra734

Your suggested extra764 seems to do the same as extra734

@toniblyx extra764 checks for a secure transport policy (denying S3 requests made over http) and extra734 is checking for server side encryption policy. What I did for both was instead of the awk stuff in the original extra734 I used jq to parse the bucket policy and do a little strong validation. For example, in the case of check764,  its not enough to have an aws:SecureTransport - false condition, it has to be a Deny policy, for all s3 actions, applying to all principles. Its still just jq - someone could iterate all possible s3 actions individually and while technically correct the check would still report failure, but I think that person can just be unhappy. ;) The check for extra734 is similarly precise but could be toned down to the original just check for the condition if desired.

@zfLQ2qx2 I have one stupid question to this PR why is extra734 now enforcing an S3 policy for denying every request without a s3:x-amz-server-side-encryption:true header? I cannot find a documentation stating that you need that header if you have default encryption enabled. Wouldn't it be better to make this a separate check or checking for a policy that denies requests WITH a s3:x-amz-server-side-encryption:false header?


Logout command for Prisma CLI

Feature Request
What feature are you missing?
We can add a command (prisma logout) to log out from Prisma CLI.
How could this feature look like in detail? Tradeoffs?
We can programmatically remove the cloudSessionKey from ~/.prisma/config.yml and document this in prisma help.
Related issue: https://github.com/graphcool/prisma/issues/1861
I can take this up, please let me know.

Thanks for raising this issue @divyenduz! That approach totally makes sense!
Feel free to start implementing it, you have the go from my side!
If you have any further questions, you know where to reach me ;)

Hey ! I can help you on that if needed ! Let me know how I can help 🚀

This is now available at 1.13-beta.


Fix capitalization in service fee query (id 4017925)

Trivial PR that lowercases the service fee query and uses snake case throughout for column names (4017925)

@fhenneke In case you have other stylistic suggestions, this might be the PR to apply them. E.g, column names. Should we use underscores and lowercase letters instead of names such as bondingPool ?

@fhenneke In case you have other stylistic suggestions, this might be the PR to apply them. E.g, column names. Should we use underscores and lowercase letters instead of names such as bondingPool ?

Actually pushed a change to snake case throughout

LGTM, what's the reason for removing capitalization? I thought capitalizing SQL specific words was the standard syntax.

Yeah that's a valid point. I guess currently we are inconsistent and i decided to propose going with lower case. Whatever we choose, it would be nice to stick to it.
Do you have any preference @fhenneke ?

I do not really have an opinion on capitalization. The most important part to me would be would be that it is consistens (first within a query and then across queries).
If we cafe about a specific styling, we can add it to the linting check, e.g. via
[sqlfluff:rules:capitalisation.keywords]
capitalisation_policy = lower

Ok, i would suggest we keep this as is then (lower case) and try to be consistent throughout. Merging so that we can finally start fixing the query


Getting Error while fetching signatures of a CLA Group on Staging environment

error: "Bad Request"
message: "{"Code":"404","Message":"EasyCLA - 404 Not Found - unable to lookup company by ID: 0012M00002MnADGQA3 - error: no company matching company record - company ID: 0012M00002MnADGQA3","x-request-id":"b257e240-67ce-11eb-a587-9d3f3a63a2ec"}"
metadata: {request: {,…},…}
statusCode: 400
https://admin.staging.platform.linuxfoundation.org/api/pages/uuids/afbf9482-1a72-11eb-8554-b763d67d8e09/query?queryName=getEmployeeSignatures
Request Method: POST
Response
{"statusCode":400,"error":"Bad Request","message":"{"Code":"404","Message":"EasyCLA - 404 Not Found - unable to lookup company by ID: 0012M00002MnADGQA3 - error: no company matching company record - company ID: 0012M00002MnADGQA3","x-request-id":"b257e240-67ce-11eb-a587-9d3f3a63a2ec"}","metadata":{"request":{"url":"https://api-gw.staging.platform.linuxfoundation.org/cla-service/v4/company/0012M00002MnADGQA3/project/a097c000007ny98AAA/contributors","method":"GET","body":null,"headers":{"User-Agent":"Retool/2.0 (+https://docs.tryretool.com/docs/apis)","Authorization":"Bearer eyJhbGciOiJSUzI1NiIsInR5cCI6IkpXVCIsImtpZCI6Ik9UY3lOVUZFUkRrME1rTTVRMFUxT1Rjd01UQTVOemM0TVRGQk5rRkNNRFZGUlRWQ1FrSTNPQSJ9.eyJpc3MiOiJodHRwczovL2xpbnV4Zm91bmRhdGlvbi1zdGFnaW5nLmF1dGgwLmNvbS8iLCJzdWIiOiJhdXRoMHxtYW5pc2h2LmNvbXBybWdyIiwiYXVkIjpbImh0dHBzOi8vYXBpLWd3LnN0YWdpbmcucGxhdGZvcm0ubGludXhmb3VuZGF0aW9uLm9yZy8iLCJodHRwczovL2xpbnV4Zm91bmRhdGlvbi1zdGFnaW5nLmF1dGgwLmNvbS91c2VyaW5mbyJdLCJpYXQiOjE2MTI1MzkwNTQsImV4cCI6MTYxMjU2MDY1NCwiYXpwIjoiYzFqVlVoMlBZME82V3IzNlpuQ3AyMkJWNGNCdDg2Y1QiLCJzY29wZSI6InByb2ZpbGUgZW1haWwgb3BlbmlkIG9mZmxpbmVfYWNjZXNzIn0.dxPCYb0U6zGV4HQ2mxpSnwi-aUsGgX9j0QCGyZwB9dhX_K1uaSF_o5zbgVo0HyuQKjzwDxlrV1r3JPni3p1A5hCXeyKH4lc8gOG5G5kx_t8SDAxH-hrT22cH5dzKslKLl-6bPbb3ni8v90hDYgzMY2YK3GtUOeP1EDBgE2gaJCbot2uKl9_RzEEkQuXfhW4Y6C7veKFtczMa1SLR4Tvrn2Gabk30-hTuWfXJP2EQmdjh0IP37ZVDnOLuVZQcVk4nUFCTvms40i4I2OtpwoWutt_XrK7DQSUmXhkAaNW4Vn5JCQpSPHruMaTo0bZu32vDRB7w6toCbXqATGunnrNuRw","X-Retool-Forwarded-For":"<IP_ADDRESS>"}},"headers":{"date":["Fri, 05 Feb 2021 16:25:01 GMT"],"content-type":["application/json"],"content-length":["235"],"connection":["keep-alive"],"vary":["Origin"],"x-request-id":["b257e240-67ce-11eb-a587-9d3f3a63a2ec"]},"status":404,"statusText":"Not Found"},"data":null}

@manishv09
For this use case endpoint path parameter for Company now uses internalID instead of External(salesforce) IDs. So change will have to be updated on the UI end.
cc @dealako @vinod-kadam

@manishv09 You can QA this on DEV
PR link https://github.com/LF-Engineering/admin-console/pull/646

I closed this one as have created LFX ticket LFX-3088 as the fix is made on Console side.


fiona /add: how to apply steps for payment agents

Changes:

added new steps for how to appy

...

Type of change

[ ] Bug fix
[ ] New feature
[x] Update feature
[ ] Refactor code
[ ] Translation to code
[ ] Translation to crowdin
[ ] Script configuration
[ ] Improve performance
[ ] Style only
[ ] Dependency update
[ ] Documentation update
[ ] Release

noted, I'll work on that


[FEATURE] Memory-games #194Add files via upload

Memory game issue #194 .please accept my pull request

Awesome contribution @Dimple-Choudhary 👏


networking: remove pod.spec.vpc & pod.vpc.nics fields

What would you like to be added:
cleanup pod.spec.vpc & pod.spec.nics which are not in use anymore.
Why is this needed:
current pod type includes spec.vpc & spec.nics fields, which are required by the old vpc network design proposal. The new design of networking uses network type to represent networks of any types (including vpc). No need to keep vpc&nics fields in pod type definition.

not priority. pounded.


Fix README typo

Current coverage is 39.68% (diff: 100%)

Merging #61 into master will not change coverage

@@             master        #61   diff @@
==========================================
  Files             5          5
  Lines            63         63
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits             25         25
  Misses           38         38
  Partials          0          0

Powered by Codecov. Last update a1b087c...8173bf9

awesome, thanks @yurimalheiros 👍


Sylius 1.3 support

Hello,
Do you plan to support Sylius 1.3?
Currently when trying to make an payment, i'm getting the following error message:
Call to a member function setApiKey() on string
Thanks!

Sylius 1.4 is now supported :)


Packaging MediaPipe with OVRLipSync Error

Hi Wong Fei,
I've tested packaging with only the mediapipe plugin aswell as only the Oculus VR Lip Sync plugin and both are able to be packaged fine on their own. However trying to package them together produces an error that my colleague and I haven't been able to figure out.
If you have any idea why this is happening know of a fix it would be greatly appreciated.
(It seems that Github will wrap the text to fit so it's still difficult to read even when marked as code, sorry)
The Output Log for the Packaging Process:
LogUObjectHash: Compacting FUObjectHashTables data took   0.80ms LogMainFrame: Selected target: MediaPipeDemo LogLauncherProfile: Unable to use promoted target - D:/Unreal Projects/ue4-mediapipe-plugin/Binaries/Win64/MediaPipeDemo-Win64-Shipping.target does not exist. UATHelper: Packaging (Windows (64-bit)): Running AutomationTool... UATHelper: Packaging (Windows (64-bit)): Parsing command line: -ScriptsForProject="D:/Unreal Projects/ue4-mediapipe-plugin/MediaPipeDemo.uproject" BuildCookRun -nocompileeditor -installed -nop4 -project="D:/Unreal Projects/ue4-mediapipe-plugin/MediaPipeDemo.uproject" -cook -stage -archive -archivedirectory="D:/Unreal Projects/MediaPipeBuild" -package -ue4exe ="C:\Program Files\Epic Games\UE_4.27\Engine\Binaries\Win64\UE4Editor-Cmd.exe" -clean -compressed -ddc=InstalledDerivedDataBackendGraph -pak -prereqs -distribution -nodebuginfo -targetplatform=Win64 -build -target=MediaPipeDemo -clientconfig=Shipping -utf8output UATHelper: Packaging (Windows (64-bit)): Setting up ProjectParams for D:\Unreal Projects\ue4-mediapipe-plugin\MediaPipeDemo.uproject UATHelper: Packaging (Windows (64-bit)): ********** BUILD COMMAND STARTED ********** UATHelper: Packaging (Windows (64-bit)): Running: C:\Program Files\Epic Games\UE_4.27\Engine\Binaries\DotNET\UnrealBuildTool.exe MediaPipeDemo Win64 Shipping -Project="D:\Unreal Projects\ue4-mediapipe-plugin\MediaPipeDemo.uproject" -Clean -NoHotReload "D:\Unreal Projects\ue4-mediapipe-plugin\MediaPipeDemo.uproject" -NoUBTMakefiles  -remoteini="D:\Unreal Proj ects\ue4-mediapipe-plugin" -skipdeploy -distribution -nobuilduht -log="C:\Users\Dex-lab\AppData\Roaming\Unreal Engine\AutomationTool\Logs\C+Program+Files+Epic+Games+UE_4.27\UBT-MediaPipeDemo-Win64-Shipping.txt" UATHelper: Packaging (Windows (64-bit)):   Cleaning MediaPipeDemo binaries... UATHelper: Packaging (Windows (64-bit)): Took 1.1431025s to run UnrealBuildTool.exe, ExitCode=0 UATHelper: Packaging (Windows (64-bit)): Running: C:\Program Files\Epic Games\UE_4.27\Engine\Binaries\DotNET\UnrealBuildTool.exe MediaPipeDemo Win64 Shipping -Project="D:\Unreal Projects\ue4-mediapipe-plugin\MediaPipeDemo.uproject"  "D:\Unreal Projects\ue4-mediapipe-plugin\MediaPipeDemo.uproject" -NoUBTMakefiles  -remoteini="D:\Unreal Projects\ue4-mediapipe- plugin" -skipdeploy -distribution -Manifest="D:\Unreal Projects\ue4-mediapipe-plugin\Intermediate\Build\Manifest.xml" -NoHotReload -log="C:\Users\Dex-lab\AppData\Roaming\Unreal Engine\AutomationTool\Logs\C+Program+Files+Epic+Games+UE_4.27\UBT-MediaPipeDemo-Win64-Shipping_2.txt" UATHelper: Packaging (Windows (64-bit)):   Using 'git status' to determine working set for adaptive non-unity build (D:\Unreal Projects\ue4-mediapipe-plugin). UATHelper: Packaging (Windows (64-bit)):   Parsing headers for MediaPipeDemo UATHelper: Packaging (Windows (64-bit)):     Running UnrealHeaderTool "D:\Unreal Projects\ue4-mediapipe-plugin\MediaPipeDemo.uproject" "D:\Unreal Projects\ue4-mediapipe-plugin\Intermediate\Build\Win64\MediaPipeDemo\Shipping\MediaPipeDemo.uhtmanifest" -LogCmds="loginit warning, logexit warning, logdatabase error" -Unattended -WarningsAsErrors -abslog="C:\User s\Dex-lab\AppData\Roaming\Unreal Engine\AutomationTool\Logs\C+Program+Files+Epic+Games+UE_4.27\UHT-MediaPipeDemo-Win64-Shipping_2.txt" -installed UATHelper: Packaging (Windows (64-bit)):   LogInit: Display: Loading text-based GConfig.... UATHelper: Packaging (Windows (64-bit)):   Reflection code generated for MediaPipeDemo in 4.7631709 seconds UATHelper: Packaging (Windows (64-bit)):   Writing manifest to D:\Unreal Projects\ue4-mediapipe-plugin\Intermediate\Build\Manifest.xml UATHelper: Packaging (Windows (64-bit)):   Building MediaPipeDemo... UATHelper: Packaging (Windows (64-bit)):   Using Visual Studio 2019 14.29.30137 toolchain (C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133) and Windows 10.0.19041.0 SDK (C:\Program Files (x86)\Windows Kits\10). UATHelper: Packaging (Windows (64-bit)):   Building 42 actions with 8 processes...  See log for more details. (C:\Users\Dex-lab\AppData\Roaming\Unreal Engine\AutomationTool\Logs\C+Program+Files+Epic+Games+UE_4.27\UBT-MediaPipeDemo-Win64-Shipping_2.txt) UATHelper: Packaging (Windows (64-bit)): AutomationTool exiting with ExitCode=6 (6) UATHelper: Packaging (Windows (64-bit)): BUILD FAILED PackagingResults: Error: Unknown Error

seems not mediapipe issue, but maybe related to precompiled header mode,
CookFrameSequenceAsync.cpp(7): fatal error C1083: Cannot open include file: 'Async.h'
OVRLipSyncPlaybackActorComponent.h(26): fatal error C1083: Cannot open include file: 'Classes/Components/AudioComponent.h'
possible fix:
#include "Async/Async.h"
#include "Components/AudioComponent.h"

MY GOODNESS, THANK YOU SO MUCH WONG FEI THIS WAS LITERALLY THE ANSWER! <3


Identifying lichess account

https://lichess.org/@/OneEyedOwl_PH

Some users may feel uncomfortable sharing their lichess accounts publicly, perhaps a better method would be to privately message hyperchessbotauthor in Lichess?
Either way I think it's a great idea to be able to identify a user with their bot, it really makes things more organized and, should issues arise, easier to identify the causes from credible sources.

@hyperbotauthor this is urgent i need your assistance in pm


updated broken links in documentation

Issue: https://github.com/scicloj/tablecloth/issues/14

@genmeblog Does this PR also require updating the changelog and version?

No, no, that's ok as it is.


Write reactor tests for checking transaction payments

Write one or more reactor tests that set up sending transactions and check payment logic.

https://github.com/EdHastingsCasperAssociation/casper-node/pull/20
https://github.com/EdHastingsCasperAssociation/casper-node/pull/21
https://github.com/EdHastingsCasperAssociation/casper-node/pull/25
https://github.com/EdHastingsCasperAssociation/casper-node/pull/26
https://github.com/EdHastingsCasperAssociation/casper-node/pull/27
https://github.com/EdHastingsCasperAssociation/casper-node/pull/29


[WIP] fix(builder): generate key on boot

closes #3132

On second thought, this means that we're going to get an unknown key warning every time builder is restarted, right? Hm....
We may want to generate keys on first boot and then store them in etcd or something to be templated later.

I could try introducing a function that pulls the key from etcd or auto-generates a new one. /deis/builder/sshPrivateKey seem like a good endpoint to store the key?

Does /deis/builder/sshPrivateKey seem like a good endpoint to store the key?

SGTM

done according to @carmstrong's suggestion. the ssh keys, once generated, are now saved in etcd to persist for future reboots.

Much easier to follow - thanks @bacongobbler! Code LGTM.

Code LGTM.


⚠ Bofrost.de has degraded performance

In 49cf2cf, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2435 ms

Resolved: Bofrost.de performance has improved in 5d87843 after 6 minutes.


docs(static-deploy): added deployment instructions for AWS with Flightcontrol

Description
This PR aims to add instructions for deploying a Vite project to AWS using Flightcontrol.
Flightcontrol provides fully automated deployments to AWS.
This is a new PR that was created as a result of the feedback given in PR one
What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[x] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

@patak-dev just wanted to remind you about this


Core Initialization Routine

As described in #354 splitting core and flex will need changes on the core side as well.
The startup routine for the core should therefore look like

Parse the comand line looking for the gRPC port argument
Initialise a Tree and offer it via gRPC
Wait for configuration to be complete via a configured property (do-while)
Initialise the client with given parameters from the tree
Additionally initialise a server on port +1

Questions are:

Can we, technically, hold back SDL2 from initialisation until it is configured

Additionally we will benefit from a new clean initialization and shutdown routine as far as that we can more clearly determine when libCEF has to be started and terminated, as well as making it easier to work on #321
The TODO for the next major release is:

[ ] Overhaul the current initialisation routine
[ ] Remove the command line and cubescript parsing as much as possible
[ ] Do the same for the server, which is related to #339

Hmm, this is a duplicate of #354 ? What you described is the main intention of #354.

It is the copy list for the corresponding C++ implementation. Just to keep that in mind.

Ok, the C++ implementation details / progress goes here.
Yes we can and have to hold back SDL2 initialization until the Inexor Tree is fully initialized.
The main difficulty is that the current implementation of the C++ sub modules cannot be initialized with arguments and we have to pass the instance id/port to the RPC sub module.
Another difficult issue is, that Inexor Flex will populate parts of the tree and Inexor Core will populate other parts of the tree. We have to care about priorities during initialization.

Moved the remaining tasks to  #427


Metadata: Allow indexes and foreign keys to use inherited properties

The ModelBuilder should lift the FK or index when there is one declared on the same properties on a derived type

@AndriySvyryd if you don't think this is critical for 7.0.0 then come discuss as we may postpone it

7.0.0 and pri0 are appropriate for this

@AndriySvyryd can you add some more details on what this means?

Consider
class Base
{
  public int Id { get; set; }
}
class Derived : Base
{
  public string Name { get; set; }
}

You should be able to create an index like this:
modelBuilder.Entity<Derived>().Index(b => new [] { b.Id, b.Name });

Also if you had created an index on the derived entity type:
  modelBuilder.Entity<Derived>().Index(b => new [] { b.Id });

You should be able to override it on base:
  modelBuilder.Entity<Base>().Index(b => new [] { b.Id });

and viceversa.
Same applies to FKs.

@AndriySvyryd can you discuss with folks what we want to do before doing it. Either design meeting, or just in the team room.

Partially implementing this for indexes alongwith #700

for indexes implemented in bc4609bcb34f57325635e3f3c0a1c20606509cd5
Pending for foreignkeys

Should base type lift the FK from derived type if defined on same features? or block it?


Undefined property: stdClass::$pagination

i dont understand how i can do the pagination for getPopularMedia()
code of my controller is
$popular = $this->instagram->getPopularMedia();
$testp = $this->instagram->pagination($popular);

when i return $testp i get a Undefined property: stdClass::$pagination error
what am i doing wrong ?

popular media does not have pages, see in instagram api documentation.

aahhhahah ok :D


Changing units for a python measurement

orchid.net_quantity has many functions for converting between .NET and python objects. For the .NET objects it has the functionality to change the units for a measurement but does not have the ability for python measurements.
When comparing results from 2 .ifrac files (1 in UsOilField and 1 in Metric). The screenshot shows my code for converting the bbl to m3 units. It would be nice to have a orchid.net_quantity.as_measurement_in_unit(measurement, to_unit) function.

We need to implement this function.

We need to implement this function.


fix(api): use generic error message if CSRF error

fix: show generic error if CSRF error
test: update tests to reflect new response
refactor: generic500 -> genericError
fix: use default to serialize all generic errors
fix: provide correct schema for save-challenge
fix: allow for CSRF errors as well as other 403s

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

There's probably a better way to serialize 'default' errors, but I haven't found it. So, for now, I've gone with sharing the default error schema and using unions when the endpoint has a custom 500.
In the future, we probably won't need many 500 errors, since it should mean "something unexpected occurred" and so we're unlikely to be able to tell the user anything more specific than that. For now, I think this slightly messy approach will have to do.

tested locally, all tests pass.


Conflict with new menu bar

After installing 3D-pack, I have been getting thick ribbon below the menu bar, which is quite annoying.
I would appreciate it if anyone tells me how to remove it.
Thank you.

I have the same problem and can't close it

我也遇到了这个问题，我是windows.11系统电脑，顺便问一下，你的UI界面很漂亮，是什么软件？
I encountered this problem too. I’m using a Windows 11 computer. By the way, your UI looks very nice. What software is that?

I have the same problem


ArchLinux install instructions

I've added an AUR package to easily install this on ArchLinux. Updated the wiki here
https://github.com/wsny/aws-okta-wiki/blob/aur-pkgbuild/Installation.md
Let me know if you want control of the PKGBUILD on the AUR, I can add you as a maintainer so you can make changes/updates if needed.
https://aur.archlinux.org/packages/aws-okta-bin/

Neat, I added section in our wiki.
I personally don't have the bandwidth to maintain it, but will put a call out internally to see if any of our Arch users would be willing.
Thanks!

I'm "watching" releases here, so hopefully I'll notice them. It's a simple change to bump the version and sha, so I'll see I'll try to keep it up to date. AUR will let someone else take it over if I become unresponsive ;)


[openldap] Build error

CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:112 (message):
Command failed: G:/Cone/vcpkg/downloads/tools/msys2/d7266db249278763/usr/bin/bash.exe --noprofile --norc --debug -c "V=1 CPP='compile cl.exe -E' CC='compile cl.exe' CC_FOR_BUILD='compile cl.exe' CPP_FOR_BUILD='compile cl.exe -E' CXX_FOR_BUILD='compile cl.exe' CXX='compile cl.exe' RC='windres-rc rc.exe' WINDRES='windres-rc rc.exe' AR='ar-lib lib.exe' LD='link.exe -verbose' RANLIB=':' STRIP=':' NM='dumpbin.exe -symbols -headers' DLLTOOL='link.exe -verbose -dll' CCAS=':' AS=':' ./../src/openldap-2-83710ce236.clean/configure --build=x86_64-pc-mingw32 "--without-cyrus-sasl" "--disable-cleartext" "--disable-mdb" "--disable-relay" "--disable-slapd" "--disable-syncprov" "--with-tls=openssl" "--without-systemd" "--without-fetch" "--without-argon2" "ac_cv_lib_iodbc_SQLDriverConnect=no" "ac_cv_lib_odbc_SQLDriverConnect=no" "ac_cv_lib_odbc32_SQLDriverConnect=no" "ac_cv_prog_ac_ct_STRIP=:" "gl_cv_double_slash_root=yes" "ac_cv_func_memmove=yes" "--enable-shared" "--disable-static" "--prefix=/G/Cone/vcpkg/installed/x64-windows/debug" "--bindir=\${prefix}/../tools/openldap/debug/bin" "--sbindir=\${prefix}/../tools/openldap/debug/sbin" "--libdir=\${prefix}/lib" "--includedir=\${prefix}/../include" "--datarootdir=\${prefix}/share/openldap""
Working Directory: G:/Cone/vcpkg/buildtrees/openldap/x64-windows-dbg
Error code: 1
See logs for more information:
G:\Cone\vcpkg\buildtrees\openldap\config-x64-windows-dbg-config.log
G:\Cone\vcpkg\buildtrees\openldap\config-x64-windows-dbg-out.log
G:\Cone\vcpkg\buildtrees\openldap\config-x64-windows-dbg-err.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_make.cmake:838 (vcpkg_execute_required_process)
ports/openldap/portfile.cmake:38 (vcpkg_configure_make)
scripts/ports.cmake:168 (include)

config-x64-windows-dbg-config.log
config-x64-windows-dbg-err.log
config-x64-windows-dbg-out.log

Works "as expected". x64-windows is not in the set of supported platforms.
https://github.com/microsoft/vcpkg/blob/830f86fb309ad7167468a433a890b7415fbb90a5/ports/openldap/vcpkg.json#L8

The window says' mingw 'after it,
This means that msys2 can be used for complete compilation

The window says' mingw 'after it, This means that msys2 can be used for complete compilation

mingw means that a compiler based on the MinGW-w64 runtime (maybe even MinGW) can be used to compile this package. It is somewhat related to the msys2 tools. But mingw does not mean you can use a compiler which doesn't provide a mingw runtime, such as MSVC (x64-windows).

Can you port the complete msys2 to vcpkg?

I can't find the Windows package in openldap.org

I took a look and found that there is indeed openldap in vcpkg
However, no packages with Windows can be found on the official website
So, can we think of porting Openldap to Windows? This idea should be possible

You can consult the upstream official openldap whether it supports porting to windows.

mingw: provides some POSIX runtime functions on top of native windows.
msys2: provides some Unix tools on top of mingw.
vcpgk: Use msys tools for some builds on windows. Can use mingw with mingw triplets and toolchains.

Can you port the complete msys2 to vcpkg?

No.

I can't find the Windows package in openldap.org

Because openldap isn't supported on native Windows.

So, can we think of porting Openldap to Windows? This idea should be possible

The idea is possible. But it needs work on the upstream source code. Once it upstream supports it, we can enable windows in the vcpkg port.
That's a lot to write down just to repeat what the vcpkg port has already declared.

@RainbowTaro2022 Do you want to adapt openldap yourself to compile on the windows platform?

@RainbowTaro2022 Do you want to adapt openldap yourself to compile on the windows platform?

sure

i found an error
/usr/lib/gcc/x86_64-pc-msys/13.2.0/../../../../x86_64-pc-msys/bin/ld: libbackends.a(mdbmdb.o): in function mdb_mutex_failed': /g/Cone/openldap-2.6.6/servers/slapd/back-mdb/./../../../libraries/liblmdb/mdb.c:10311:(.text.unlikely+0x313): undefined reference to pthread_mutex_consistent'
/usr/lib/gcc/x86_64-pc-msys/13.2.0/../../../../x86_64-pc-msys/bin/ld: libbackends.a(mdbmdb.o): in function mdb_env_setup_locks': /g/Cone/openldap-2.6.6/servers/slapd/back-mdb/./../../../libraries/liblmdb/mdb.c:4924:(.text.unlikely+0x734): undefined reference to pthread_mutexattr_setrobust'

i use msys2

./configure C=cl.exe CXX=cl.exe --prefix=/g/OpenLDAP-windows/
make depend
make

and an error

You are building binaries for the MSYS environment. This is not native windows, even less than mingw: It uses a cygwin-based CRT.
https://www.msys2.org/docs/environments/

MSYS2 comes with different environments and the first thing you have to decide is which one to use. The differences among the environments are mainly environment variables, default compilers/linkers, architecture, system libraries used etc. If you are unsure, go with UCRT64.
The MSYS environment contains the unix-like/cygwin based tools, lives under /usr and is special in that it is always active. All the other environments inherit from the MSYS environment and add various things on top of it.

Even msys2 doesn't build openldap for the MSYS environment. They build openldap for mingw-w64 environments, with three patches.
https://github.com/msys2/MINGW-packages/tree/master/mingw-w64-openldap

You are building binaries for the MSYS environment. This is not native windows, even less than mingw: It uses a cygwin-based CRT. https://www.msys2.org/docs/environments/
Even msys2 doesn't build openldap for the MSYS environment. They build openldap for mingw-w64 environments, with three patches. https://github.com/msys2/MINGW-packages/tree/master/mingw-w64-openldap

shouid i use cygwin?

I don't think that cygwin, msys or mingw is what you want to have openldap for. Unless you already know what these environments mean.

i sorry
i can make a idea

I've been planning my own game these days
This game is about people with disabilities
I can't think of a good way to handle this

I have always claimed to be a code wizard
But in fact, I'm just an ordinary person

@RainbowTaro2022 Will you continue to adapt this port to the Windows platform?

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.