BUG: fix issue in io/wavfile for float96 input.

This resulted in 24 test errors on my 32-bit Python install.
Introduced in gh-5990. @endolith could you please check this fix?

https://github.com/scipy/scipy/pull/5990#issue-142524080
Can it get the list of numbers from numpy's list of available types instead of being hardcoded?

There's not a very nice way to get numpy to tell you that AFAIK. One could test for existence of all known float types, but that's just a clumsy way to still hardcode the same info.

Well I think you know more about this than me.  :)  I didn't even know float96 existed.  It doesn't on my machine, nor does complex256, which was causing problems in https://github.com/scipy/scipy/pull/5608#issuecomment-209197767.  So if the types that numpy supports can vary from one install to the next, doesn't this list need to be dynamic, too?

set(8*dtype(t).itemsize for t in flatten(numpy.sctypes[kind] for kind in ('float', 'int', 'uint')))?  :)

So if the types that numpy supports can vary from one install to the next, doesn't this list need to be dynamic, too?

That's what didn't feel quite right about gh-5990 (which I didn't follow). We're trying to read wav files here. Why would the supported wav file formats depend on the available numpy types? If there's even such a thing as a valid 96-bit wav file, we can stuff it into either of float96/float128 I'd think.
And even if this code currently does the right thing, it seems wrong to support a 128-bit wav format only on systems that have numpy.float128 - not portable that way.

set(8*dtype(t).itemsize for t in flatten(numpy.sctypes[kind] for kind in ('float', 'int', 'uint')))? :)

Not exactly a win for readability .....

Well this function converts wav files to a bit-for-bit equivalent dtype, which is different from packages like pysoundfile or scikits.audiolab, which convert everything to float normalized from -1 to 1.  So it won't ever support 24 bit WAVs because there's no 24-bit dtype.  I don't know why it was done that way, but it's presumably useful to someone to be able to work with the int formats directly.  Adding a different function that converts any WAV format to normalized float would be useful, though, so people don't need a separate package to read any kind of WAV.

The numpy dtypes do not in my opinion restrict the supported float
types. Since the behavior is currently undefined, we can define it in
the future to mean to pick the next largest float type available.

Why not just add a normalize=False kwarg to the existing function? It seems simpler than making another function that does almost the same thing, at least at the reading end.

The numpy dtypes do not in my opinion restrict the supported float types. Since the behavior is currently undefined, we can define it in the future to mean to pick the next largest float type available.

Looks that way to me too.

Why not just add a normalize=False kwarg to the existing function? It seems simpler than making another function that does almost the same thing, at least at the reading end.

You mean to wavfile.read? Not sure that is necessary. I definitely agree that a new function is not needed though.
I read the wav file spec and all examples of actual formats, and there doesn't seem to be any real-life example with >64 bits. So we're talking about fixing a hypothetical issue here as far as I can tell. So proposal: simplest fix possible, remove the test and leave the list as is.
The one thing that is common but we don't support is 24-bit wav files. Those we really should stuff in float32 arrays, probably with a minor change to the existing read function.

Merging --- however, this is not a complete fix, because the dtype selection code needs to be made to support cases where numpy has no corresponding dtype.

@rgommers

I read the wav file spec and all examples of actual formats, and there doesn't seem to be any real-life example with >64 bits. So we're talking about fixing a hypothetical issue here as far as I can tell.

Yeah I don't think they exist.  Same issue as https://github.com/scipy/scipy/issues/6090 ? I misunderstood and thought you had an actual float96 file that was failing to open.
@Eric89GXL

Why not just add a normalize=False kwarg to the existing function? It seems simpler than making another function that does almost the same thing, at least at the reading end.

Well for what I'm imagining they would have different APIs and do substantially different things.  The function that converts to normalized floating point should return the file's encoding along with the data and sample rate, so a modified file can be written with the same encoding as the original.  Also because it's being converted to a different binary format, I don't think mmap can be used?  I couldn't figure out how to do that with 24-bit.  It would need to be read and processed in chunks instead?
Existing:
rate, data = scipy.io.wavfile.read(filename, mmap=False)
scipy.io.wavfile.write(filename, rate, data)

Normalized:
data, rate, enc = scipy.io.wavfile.readnorm(filename)
scipy.io.wavfile.writenorm(filename, rate, data, enc='pcm16')

Maybe with additional parameters in each to represent the start and stop point of a chunk to be read or written.  readf could specify the dtype to convert to, too?
data, rate, enc = scipy.io.wavfile.readnorm(filename, start=0, stop=None, dtype=float)
scipy.io.wavfile.writenorm(filename, rate, data, enc='pcm16', start=0)

The existing functions read/write numpy arrays as bit-identical WAV file formats, similar to saving an array using numpy.save or pickle, where the encoding of the WAV is implicitly the same as that of the numpy array, and saving and loading an array is lossless.
The proposed function would read any type of WAV file (could be expanded to other types in the future?), regardless of whether the encoding exists as a numpy type (potentially including compressed or mu-law, etc), and convert it to a normalized float format.  Saving and opening would not necessarily be lossless.  It would behave like scikits.audiolab.wavread or like a WAV-only soundfile.read.
Anyway, these other libraries already do all of this and more, so this would just be a simple way to use a limited set of sound files for people who don't want to install either of these libraries for some reason?

Yeah, those do seem sufficiently different to justify different names.

Anyway, these other libraries already do all of this and more, so this would just be a simple way to use a limited set of sound files for people who don't want to install either of these libraries for some reason?

Hmm... is that your motivation to add this function to scipy, then? Avoiding other dependencies?

Well actually I personally don't have any motivation to add it:  I was looking at scipy.io.wavfile because scikits.audiolab never worked reliably, but as I said in https://github.com/scipy/scipy/pull/5996, I found PySoundFile and that has been working fine.

So, I'm in the process of looking into 24-bit support, and I'm confused about this PR.
The issue in https://github.com/scipy/scipy/issues/6090 was that test_write_roundtrip was creating a float96 WAV file (dtype('>f12')) to see if it passes the round-trip writing and reading, and this was not supported by the WAV file reader, which was causing a test failure.  This was then fixed by adding support for float96 to the read() function in https://github.com/scipy/scipy/pull/6184/commits/485d1f046823de5d34a07e97c7661266424589e1?

But I thought everyone agreed that float96 WAVs don't exist?  So wouldn't the correct solution be to remove that test?
Also I'm not even sure how test_write_roundtrip was creating dtype('>f12') in the first place.  Trying to use Blame on that line, it looks like it always said for size in (1, 2, 4, 8):, so where did 12 come from?

Also I'm not even sure how test_write_roundtrip was creating dtype('>f12') in the first place.

I had a quick look and also can't make sense of how this happened

I added a print(dt) statement to est_write_roundtrip() in the current master and it tests these types on my machine:
>f4
>f8
>i2
>i4
>i8
float32
float64
int16
int32
int64
uint8


ModuleNotFoundError: No module named 'webview.platforms.qt' from compiled nuitka executable

I am using pywebview and PyQt5 to display a webpage.
Nuitka info:
1.5.7
Commercial: None
Python: 3.11.3 (tags/v3.11.3:f3909b8, Apr  4 2023, 23:49:59) [MSC v.1934 64 bit (AMD64)]
Flavor: CPython Official
Executable: C:\Users\Nether\AppData\Local\Programs\Python\Python311\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 10
Version C compiler: ~\AppData\Local\Nuitka\Nuitka\Cache\downloads\gcc\x86_64\11.3.0-14.0.3-10.0.0-msvcrt-r3\mingw64\bin\gcc.exe (gcc).

Using modules:
fastapi uvicorn pywebview[qt] pyperclip psutil requests tdqm tkinter
Code:
import urllib.request
import webview
class Api:
    def __init__(self):
        self._window = None

    def set_window(self, window):
        self._window = window

    def quit(self):
        window.destroy()

def on_closed():
    urllib.request.urlopen('http://<IP_ADDRESS>:36113/quit')
if __name__ == '__main__':
    api = Api()
    window = webview.create_window('test', 'http://<IP_ADDRESS>:36113/', width=600, height=500, resizable=False, js_api=api)
    window.events.closed += on_closed
    webview.start(gui='qt')

I encountered an error with the executable when compiled with nuitka, here is the parameters that i used to compile:
python -m nuitka --onefile --follow-imports --include-data-dir=ui=ui --enable-plugin=pyqt5 --enable-plugin=tk-inter frame.py
The error message when running executable:
[pywebview] QT cannot be loaded
Traceback (most recent call last):
  File "C:\Users\Nether\AppData\Local\Temp\ONEFIL~1\webview\guilib.py", line 29, in import_qt
ModuleNotFoundError: No module named 'webview.platforms.qt'
Traceback (most recent call last):
  File "C:\Users\Nether\AppData\Local\Temp\ONEFIL~1\frame.py", line 23, in <module>
    webview.start(gui='qt')
  File "C:\Users\Nether\AppData\Local\Temp\ONEFIL~1\webview\__init__.py", line 132, in start
  File "C:\Users\Nether\AppData\Local\Temp\ONEFIL~1\webview\guilib.py", line 102, in initialize
webview.util.WebViewException: You must have pythonnet installed in order to use pywebview.

Here's what i get when compiling:
Nuitka:WARNING: The Python version '3.11' is only experimentally supported by Nuitka '1.5.7', but an upcoming release
Nuitka:WARNING: will change that. In the mean time use Python version '3.10' instead or newer Nuitka.
Nuitka-Options:INFO: Following all imports is the default for onefile mode and need not be specified.
Nuitka-Plugins:WARNING: pyqt5: For the obsolete PyQt5 the Nuitka support is incomplete. Threading, callbacks to compiled
Nuitka-Plugins:WARNING: functions, etc. may not be working.
Nuitka-Plugins:WARNING:     Complex topic! More information can be found at https://nuitka.net/info/pyqt5.html
Nuitka:INFO: Starting Python compilation with Nuitka '1.5.7' on Python '3.11' commercial grade 'not installed'.
Nuitka-Plugins:INFO: tk-inter: Injecting pre-module load code for module 'tkinter':
Nuitka-Plugins:INFO: tk-inter:     Need to make sure we set environment variables for TCL.
Nuitka-Plugins:INFO: multiprocessing: Injecting pre-module load code for module 'multiprocessing':
Nuitka-Plugins:INFO: multiprocessing:     Monkey patching "multiprocessing" load environment.
Nuitka-Plugins:INFO: multiprocessing: Injecting post-module load code for module 'multiprocessing':
Nuitka-Plugins:INFO: multiprocessing:     Monkey patching "multiprocessing" for compiled methods.
Nuitka-Plugins:INFO: pyqt5: Injecting pre-module load code for module 'PyQt5':
Nuitka-Plugins:INFO: pyqt5:     Adding binary folder to runtime 'PATH' environment variable for proper Qt loading.
Nuitka-Plugins:INFO: pyqt5: Injecting post-module load code for module 'PyQt5.QtCore':
Nuitka-Plugins:INFO: pyqt5:     Setting Qt library path to distribution folder. We need to avoid loading target
Nuitka-Plugins:INFO: pyqt5:     system Qt plugins, which may be from another Qt version.
Nuitka-Plugins:INFO: options-nanny: Note, when using 'PyQt5', consider using '--disable-console' option. Otherwise a terminal window will open. However for debugging, terminal output is the easiest way to see informative traceback information.
Nuitka-Plugins:INFO: options-nanny: Note, when using 'tkinter', consider using '--disable-console' option. Otherwise a terminal window will open. However for debugging, terminal output is the easiest way to see informative traceback information.
Nuitka-Plugins:WARNING: pyqt5: Unwanted import of 'tkinter' that is redundant with 'PyQt5' encountered. Use
Nuitka-Plugins:WARNING: '--nofollow-import-to=tkinter' or uninstall it for best compatibility with pure Python
Nuitka-Plugins:WARNING: execution.
Nuitka:INFO: Completed Python level compilation and optimization.
Nuitka:INFO: Generating source code for C backend compiler.
Nuitka:INFO: Running data composer tool for optimal constant value handling.
Nuitka:INFO: Running C compilation via Scons.
Nuitka-Scons:INFO: Backend C compiler: gcc (gcc).
Nuitka-Scons:INFO: Backend linking program with 316 files (no progress information available).
Nuitka-Scons:INFO: Compiled 636 C files using ccache.
Nuitka-Scons:INFO: Cached C files (using ccache) with result 'cache hit': 314
Nuitka-Scons:INFO: Cached C files (using ccache) with result 'cache miss': 2
Nuitka-Options:INFO: Included data file 'ui\d.html' due to specified data dir 'ui' on command line.
Nuitka-Options:INFO: Included data file 'ui\settings.html' due to specified data dir 'ui' on command line.
Nuitka-Plugins:INFO: data-files: Included data file 'certifi\cacert.pem' due to package data for 'certifi'.
Nuitka-Plugins:INFO: tk-inter: Included 87 data files due to Tk needed for tkinter usage.
Nuitka-Plugins:INFO: tk-inter: Included 835 data files due to Tcl needed for tkinter usage.
Nuitka-Plugins:INFO: data-files: Included data file 'webview\lib\Microsoft.Web.WebView2.LICENSE.md' due to package data directory 'lib' for 'webview'.
Nuitka-Plugins:INFO: pyqt5: Including Qt plugins 'iconengines,imageformats,mediaservice,platforms,platformthemes,printsupport,styles' below 'PyQt5\qt-plugins'.
Nuitka-Plugins:INFO: pyqt5: Found 2 files DLLs from 'OpenSSL' PyQt5 installation.
Nuitka-Plugins:INFO: dll-files: Found 1 file DLLs from clr_loader.ffi installation.
Nuitka-Plugins:INFO: dll-files: Found 1 file DLLs from pythonnet installation.
Nuitka-Plugins:INFO: dll-files: Found 4 files DLLs from webview installation.
Nuitka-Postprocessing:INFO: Creating single file from dist folder, this may take a while.
Nuitka-Onefile:INFO: Running bootstrap binary compilation via Scons.
Nuitka-Scons:INFO: Onefile C compiler: gcc (gcc).
Nuitka-Scons:INFO: Onefile linking program with 1 files (no progress information available).
Nuitka-Scons:INFO: Compiled 2 C files using ccache.
Nuitka-Scons:INFO: Cached C files (using ccache) with result 'cache hit': 1
Nuitka-Onefile:INFO: Using compression for onefile payload.
Nuitka-Onefile:INFO: Onefile payload compression ratio (26.77%) size 113821266 to 30471350.
Nuitka-Onefile:INFO: Keeping onefile build directory 'frame.onefile-build'.
Nuitka:INFO: Keeping dist folder 'frame.dist' for inspection, no need to use it.
Nuitka:INFO: Keeping build directory 'frame.build'.
Nuitka:INFO: Successfully created 'frame.exe'.

Help would be greatly appreciated, Thanks!

Does --include-module=webview.platforms.qt help this, I am almost assuming that is a kind of implicit dependency missing right there. The pywebview plugin combined with a qt binding plugin should probably add it.
And it does so, but not on Windows.
        # Make sure webview platforms are included as needed.
        if module_name.isBelowNamespace("webview.platforms"):
            if isWin32Windows():
                result = module_name in (
                    "webview.platforms.winforms",
                    "webview.platforms.edgechromium",
                    "webview.platforms.edgehtml",
                    "webview.platforms.mshtml",
                    "webview.platforms.cef",
                )
                reason = "Platforms package of webview used on '%s'." % getOS()
            elif isMacOS():
                result = module_name == "webview.platforms.cocoa"
                reason = "Platforms package of webview used on '%s'." % getOS()
            elif getActiveQtPlugin() is not None:
                result = module_name = "webview.platforms.qt"
                reason = (
                    "Platforms package of webview used due to '%s'."
                    % getActiveQtPlugin()
                )
            else:
                result = module_name = "webview.platforms.gtk"
                reason = (
                    "Platforms package of webview used on '%s' without Qt plugin enabled."
                    % getOS()
                )

BTW: Enabling the tk-inter and pyqt5 plugin means you are confused, using more than one is surprising, but not a problem by itself, the PyQt5 plugin just prevents tkinter from being included anyway.
Reordering these branches will work too and might be a sane choice. The above code replicates platform defaults I think, but selecting Qt should be doable I guess.
I believe the pythonnet message is probably just bad bug hiding of pywebview making guesses that are not warranted unless a pythonnet update is no longer working with Nuitka.

I decided to use pywebview with GTK and works perfectly with the built nuitka executable. 👍
And compiled with only --enable-plugin=tk-inter
import urllib.request
import webview
class Api:
    def __init__(self):
        self._window = None

    def set_window(self, window):
        self._window = window

    def quit(self):
        window.destroy()

def on_closed():
    urllib.request.urlopen('http://<IP_ADDRESS>:36113/quit')
if __name__ == '__main__':
    api = Api()
    window = webview.create_window('test', 'http://<IP_ADDRESS>:36113/', width=600, height=500, resizable=False, js_api=api)
    window.events.closed += on_closed
    webview.start(gui='gtk')

The PyQt5 support is also incomplete unlike PySide6 which is perfect, as is GTK for all I know. I am however keeping this open, to checkout if we can give some sort of warning to the user. If it's really Qt5 only, and we are speaking of Windows, there are indeed better options. Curious as to why you went with GTK and not the platform default, which I think is edge based stuff. With no argument for gui I guess, it would just work?

When gtk is not found, it defaults to mshtml on Windows, as far as I know.
When I use an older version of Windows 10 (2004), I get the following:
[pywebview] MSHTML is no longer supported. Details on how to use Edge Chromium can be found at https://pywebview.flowrl.com/guide/renderer.html#web-engine.
I could have used the default settings, as you suggested,  in the order: edgechromium, mshtml.
But i dont want to run into any rendering issues when using different web engines, so might as well stick with one.

Yes, fair point. Making things work on older systems is of course an art by itself. Recognizing the argument given is not yet practical, so this will remain the packagers obligation. I wonder if there is an environment variable, that I forgot to check and persist to runtime. Can you look that up? Because then, it could be done by forcing its value to the compile time value, avoiding unnecessary dependencies.

Any feedback on my last question?

Closing for lack of feedback


Fix Sync Issue with Upstash

Fixes #4563
Fixes #4583
Fixes #4658

@Dean-YZG Wow, thank you, this is my first open-source PR merge.


Can I embed these charts in a Jupyter Kernel?

I am working with Scala using the Almond.sh kernel. So far I quite like this library. However, one thing I wanted to ask was can this library work with a Jupter notebook?
I am using the Almond.sh kernel for Scala.

Sorry, I do not know if that's possible.


Publish the V1 package on https://open-vsx.org/

Is your feature request related to a problem? Please describe.
https://open-vsx.org/, which makes it available to non-MS clients that consume VS Code extensions. For example, RedHat uses it in CodeReady Workspaces, which is part of Open Shift.

It would also make it available in gitpod.io

Published on open-vsx


Clean up UI of toonin extension

looks messy, planning to hide some components until its needed.
Clarity for users

Check the new pull request has most of UI updates.

Solved in #60


Add keyboard shortcut for Upload button

Adding a keyboard shortcut for the Upload button in the Save panel would make it more convenient to save a changeset. Over time, as additional fields are added to the panel, the shortcut would become more important for making the Upload button easily accessible.
Personally as a Mac user, I’m a fan of ⌘↩ as a shortcut for the default action in any modal. But I don’t know whether CtrlEnter is similarly intuitive for Windows and Linux users. Another option would be to add accesskey="u" or somesuch, which could be more discoverable.

I thought about this more and I don't think we should do it.  I think the risk is too great that a user who is typing stuff in the comment box might accidentally trigger the keyboard shortcut that uploads before they are ready to do it.
Besides, a keyboard shortcut for the upload button doesn't really save any time, since the user will just be sitting there for some seconds waiting for the upload to finish.

Personally as a Mac user, I’m a fan of ⌘↩ as a shortcut for the default action in any modal. But I don’t know whether CtrlEnter is similarly intuitive for Windows and Linux users

Just to note, the convention from many sites is enter sends, and shift-enter gives a newline.


can 'kubevpn proxy' support  user customizing local port

Hello!
After using this fantastic tool for some time, i have to say the kubevpn proxy is so convenient. And i encountered few problems when using kubevpn proxy.
In my k8s cluster, there is a testservice deployment, there shows the containers manifests:
Containers:
   hello:
    Image:      xx
    Port:       8003/TCP
    Host Port:  0/TCP
   hi:
    Image:      yy
    Port:       8004/TCP
    Host Port:  0/TCP

For now, after running kubevpn proxy deployment/testservice --headers a=1.
the envoy config:
- Ports:
  - containerPort: 8003
    protocol: TCP
  - containerPort: 8004
    protocol: TCP
  Rules:
  - Headers:
      a: "1"
    LocalTunIPv4: <IP_ADDRESS>
    LocalTunIPv6: efff:ffff:ffff:ffff:ffff:ffff:ffff:999b
  Uid: deployments.apps.testservice

Requesting port 8003 of testservice with headers a=1, the req will be proxied to local port 8003.
Requesting port 8004 of testservice with headers a=1, the req will be proxied to local port 8004.
And it works fine in this scenario.
BUT IN THE BELOW TWO SCENARIO, LOCAL PORT CONFLICTS OCCURS:
scenario 1:
when i have two service 'testservice1' and 'testservice2', the container manifest:
testservice1:
Containers:
   hello:
    Image:      xx
    Port:       8003/TCP
    Host Port:  0/TCP

testservice2:
Containers:
   hi:
    Image:      xx
    Port:       8003/TCP
    Host Port:  0/TCP

And i want to proxy the two service to my local pc, but the two service will both proxied to port 8003 of my local pc, it will not be feasible.
scenario 2:
when i have a service 'testservice3', the container manifest:
Containers:
   hello:
    Image:      xx
    Port:       43029/TCP
    Host Port:  0/TCP

and i want to proxy this testservice to my local pc, but the container port 43029 has been used by a import process in my local pc. it will not be feasible neither.
So i wonder if kubevpn proxy cmd can add an proxy port option for special user, like kubevpn proxy deployment/testservice --headers a=1 --portMap 8003:28003. And the envoy config can be like this:
ENVOY_CONFIG:
----
- Ports:
  - containerPort: 8003
    protocol: TCP
  - containerPort: 8004
    protocol: TCP
  Rules:
  - Headers:
      a: "1"
    LocalTunIPv4: <IP_ADDRESS>
    LocalTunIPv6: efff:ffff:ffff:ffff:ffff:ffff:ffff:999b
    PortMap:
    - 8003:28003
  Uid: deployments.apps.testservice

So control-plane can create local endpoint <IP_ADDRESS>:28003 . Request with header 'a=1' will be proxied to local pc' 28003 without port conflict.
And containerPort 8004 can not find it's PortMap, it can still be proxied to local pc' 8004.
So this feature will not influence the user who do not need customized local port in proxy scenario.

Hello! After using this fantastic tool for some time, i have to say the kubevpn proxy is so convenient. And i encountered few problems when using kubevpn proxy. In my k8s cluster, there is a testservice deployment, there shows the containers manifests:
Containers:
   hello:
    Image:      xx
    Port:       8003/TCP
    Host Port:  0/TCP
   hi:
    Image:      yy
    Port:       8004/TCP
    Host Port:  0/TCP

For now, after running kubevpn proxy deployment/testservice --headers a=1. the envoy config:
- Ports:
  - containerPort: 8003
    protocol: TCP
  - containerPort: 8004
    protocol: TCP
  Rules:
  - Headers:
      a: "1"
    LocalTunIPv4: <IP_ADDRESS>
    LocalTunIPv6: efff:ffff:ffff:ffff:ffff:ffff:ffff:999b
  Uid: deployments.apps.testservice

Requesting port 8003 of testservice with headers a=1, the req will be proxied to local port 8003. Requesting port 8004 of testservice with headers a=1, the req will be proxied to local port 8004. And it works fine in this scenario.
BUT IN THE BELOW TWO SCENARIO, LOCAL PORT CONFLICTS OCCURS: scenario 1: when i have two service 'testservice1' and 'testservice2', the container manifest: testservice1:
Containers:
   hello:
    Image:      xx
    Port:       8003/TCP
    Host Port:  0/TCP

testservice2:
Containers:
   hi:
    Image:      xx
    Port:       8003/TCP
    Host Port:  0/TCP

And i want to proxy the two service to my local pc, but the two service will both proxied to port 8003 of my local pc, it will not be feasible. scenario 2: when i have a service 'testservice3', the container manifest:
Containers:
   hello:
    Image:      xx
    Port:       43029/TCP
    Host Port:  0/TCP

and i want to proxy this testservice to my local pc, but the container port 43029 has been used by a import process in my local pc. it will not be feasible neither.
So i wonder if kubevpn proxy cmd can add an proxy port option for special user, like kubevpn proxy deployment/testservice --headers a=1 --portMap 8003:28003. And the envoy config can be like this:
ENVOY_CONFIG:
----
- Ports:
  - containerPort: 8003
    protocol: TCP
  - containerPort: 8004
    protocol: TCP
  Rules:
  - Headers:
      a: "1"
    LocalTunIPv4: <IP_ADDRESS>
    LocalTunIPv6: efff:ffff:ffff:ffff:ffff:ffff:ffff:999b
    PortMap:
    - 8003:28003
  Uid: deployments.apps.testservice

So control-plane can create local endpoint <IP_ADDRESS>:28003 . Request with header 'a=1' will be proxied to local pc' 28003 without port conflict. And containerPort 8004 can not find it's PortMap, it can still be proxied to local pc' 8004. So this feature will not influence the user who do not need customized local port in proxy scenario.

yes, it's a good idea, i will add this feature as soon as possible

@kyr1eeee i finished this feature at this pr: https://github.com/kubenetworks/kubevpn/pull/176, you can build kubevpn by yourself, or you can wait maybe one weeks to release new version ~

@kyr1eeee i finished this feature at this pr: #176, you can build kubevpn by yourself, or you can wait maybe one/two weeks to release new version ~

ok, thanks

it seems to be not perfect. If user A proxy deployment/test-svc with portMap 18003:8003, when user B proxy deployment/test-svc with portMap 28003:8003，user B's rule will not work. The portMap should added to a rule?

it seems to be not perfect. If user A proxy deployment/test-svc with portMap 18003:8003, when user B proxy deployment/test-svc with portMap 28003:8003，user B's rule will not work. The portMap should added to a rule?

Yes. You are right. It's my fault. I will fixed it as soon as possible

it seems to be not perfect. If user A proxy deployment/test-svc with portMap 18003:8003, when user B proxy deployment/test-svc with portMap 28003:8003，user B's rule will not work. The portMap should added to a rule?

Yes. You are right. It's my fault. I will fixed it as soon as possible

@kyr1eeee fix it at pr https://github.com/kubenetworks/kubevpn/pull/182

already released new version v2.2.3, add this feature, you can have a try ~

it still has a bug, in daemon/action/proxy.go:Proxt() func,  the portMap should be updated as the req.PortMap, otherwise the control-plane rule will be mixed.
	svr.connect.Workloads = req.Workloads
	svr.connect.Headers = req.Headers
        //svr.connect.PortMap= req.PortMap
	err = svr.connect.CreateRemoteInboundPod(ctx)

@wencaiwulue

it still has a bug, in daemon/action/proxy.go:Proxt() func, the portMap should be updated as the req.PortMap, otherwise the control-plane rule will be mixed.
	svr.connect.Workloads = req.Workloads
	svr.connect.Headers = req.Headers
        //svr.connect.PortMap= req.PortMap
	err = svr.connect.CreateRemoteInboundPod(ctx)

Yes, you are right. here is pr https://github.com/kubenetworks/kubevpn/pull/190

@kyr1eeee already redo-released version v2.2.3. You can use command kubevpn upgrade to have a try ～. Thanks for report this bug

@kyr1eeee already redo-released version v2.2.3. You can use command kubevpn upgrade to have a try ～. Thanks for report this bug

OK, i just wanna have a try for the proxy mode.


Add ignoreErrors option

This is useful if compile process has errors but we want to ignore them e.g. Typescript compile errors

I'd turn this into a callback like (errors) => {return true}. We pass the errors to func and it return true if errors should be raised and false if they should silenced.

Or maybe even better, a function that filters the errors.

Feel free to re-open and update the PR if this is still desired.


Execute commands using different user

A way to login with another user and execute commands could be useful. I was trying to use a user that I created (outside of the default ubuntu) and didn't see an in library way of doing so. A different user would contain a different name and key file (created by the user).

Thats a good idea. I think I will add a couple extra optional parameters to execute/upload/download that allow you to override the standard username and private key file.
It will look something like:
def execute(self, command, retry=3, retry_interval=10), username=None, private_user_key_file=None)
If the the username and/or private_user_key_file and not passed in, the behavior will be the same as it is now.

This has been implemented and supported via https://github.com/fabric-testbed/fabrictestbed-extensions/blob/main/fabrictestbed_extensions/fablib/node.py#L1295
Closing the issue.


tools/syz-cover: add missing DiscoverModules()

This fixes syz-cover tool breaking after the removal of DiscoverModules() down the stack from MakeReportGenerator().
I'm not sure if syz-cover should overwrite modules if they're provided on the command line, or if they should be added to the manager config modules. If we're using the config, then it might even make sense to just strip that command line option entirely.
Fixes #5086

I'm just waiting to hear back from my manager on what we're going to do about the CLA


DinD sidecar: Docker daemon fails to start

Expected Behavior
I'm trying to build and push a Docker image to a private repository, using a task with a DinD sidecar. It should successfully build the image and then push it to the repo.
Actual Behavior
The task fails with the error Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?. Initially, the pipeline executed correctly. After a few days, it began failing intermittently. Now, a few days later, it fails every time.
Steps to Reproduce the Problem

Additional Info

Kubernetes version:
Output of kubectl version:

Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.4", GitCommit:"b695d79d4f967c403a96986f1750a35eb75e75f1", GitTreeState:"clean", BuildDate:"2021-11-17T15:48:33Z", GoVersion:"go1.16.10", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.3", GitCommit:"c92036820499fedefec0f847e2054d824aea6cd1", GitTreeState:"clean", BuildDate:"2021-10-27T18:35:25Z", GoVersion:"go1.16.9", Compiler:"gc", Platform:"linux/amd64"}

- Tekton Pipeline version:

**Output of `tkn version` or `kubectl get pods -n tekton-pipelines -l app=tekton-pipelines-controller -o=jsonpath='{.items[0].metadata.labels.version}'`**

Client version: 0.21.0
Pipeline version: v0.31.0
Triggers version: v0.18.0
Dashboard version: v0.23.0

Hey @HildaHay thanks for the issue report. I think maybe the most concrete next steps I can offer here are to suggest creating a plain Pod with the containers in them configured similarly to the DinD setup you're using in your Task/TaskRun. If the issue persists in this setup then I think the problem is likely outside the control of Tekton, and if not then hopefully we'll be able to compare the working Pod with the one Tekton generates and come to some conclusions about where the issue might be.


DWIM rewrite patterns for graphblas.reduce_to_vector and matrix multiply ops

This PR also:

Comments out unused code that causes warnings.
Refactors *::buildAlgorithm to have a smaller signature.
Adds functional tests.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 83.8%

Files with Coverage Reduction
New Missed Lines
%

mlir_graphblas/engine.py
1
75.84%

mlir_graphblas/tests/test_profile.py
4
51.95%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3678

Relevant Lines:
4389

💛  - Coveralls

I had an internal dialog with myself -- capturing here for others to consider.
I had the thought that DWIM rewriting should happen on the generic versions of operations. That way anyone who manually writes a generic call (useful for user-defined operators) will get the benefit of DWIM rewriting.
I have decided this is a bad choice because several operations have custom operators which don't go through the generic machinery. For example, ReduceToVector with count doesn't lower to generic. Instead, it writes the full lowering pass directly (because it is both easy to write and much more efficient than going through the generic looping). For the case of ReduceToVector, that would require DWIM rewriting to handle both ReduceToVector and ReduceToVectorGeneric. That is asking too much of DWIM rewriting.
So, I think we should only perform DWIM rewriting on the non-generic operations. That should cover the 99% of normal cases, giving users freedom in writing algorithms without worrying about the strict requirements of the lowering algorithm. And for those users who need to manually write generic operations, they will have to operate in strict mode. But they are already "expert" users, so this is not much of a burden.


websocket connection failed

Hi @yoryer
I tried to implement the SDK in my app, but I still got a connection failed code: 1006.
Before that, I tried to run the example app in this repository, but I still had an issue connecting to the Toky VoIP.
Evidence:
Chrome

Firefox

Am I missed something?
Thanks

Hi @dev-amirmalik
I suggest you get in touch through the support center so we can share confidential information there in case we need it to check your case.

Hi @yoryer
Yes you are right. You need to contact the support. Maybe the production has bugs or something that causing the issue.
Currently I can ues the sdk.
I close this issue.
Thanks


connects multiple times when uploading multiple files

Steps to reproduce

Make sure you are not connected
select multiple files in treeview
right click and select Upload

Expected
Connects to ftp server once and uploads files
Actual
Connects once for each file selected then uploads all the files

Fixed in the dev branch. Will be in the next release. If there are still any issue then please open a new issue.


ARROW-88: [C++] Refactor usages of parquet_cpp namespace

I also removed an unneeded Py_XDECREF from ARROW-30; didn't want to create a separate patch for that.

+1, LGTM

Thanks -- merging so you can rebase


chore(183909391): Remove existing Cloud dashboard code

Pull Request Description
Given that we have decided to go with a React-based approach for the combined IDE & Cloud UI, the cloud_dashboard written in EnsoGL is no longer applicable and is being removed. The bug fixes and refactors from that code that are still relevant have been separated into the following, smaller, PRs:
https://github.com/enso-org/cloud-v2/pull/116
https://github.com/enso-org/cloud-v2/pull/117
https://github.com/enso-org/cloud-v2/pull/118
https://github.com/enso-org/cloud-v2/pull/119
Meanwhile, the cloud_dashboard crate itself is being removed from the enso repo to ease future maintainability burdens.
Important Notes

Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide build and ./run ide watch.

Does it also remove all JS styles/things inserted to the codebase to make the Rust-based cloud dashboard working? I understand that the new implementation is on separate repo, so all CSS styles and JS changes introduced because of the Rust dashboard can be reverted now.

Unless I'm misunderstanding, we never added JS styles, only stylewatch styles, which were being added in the previously closed PR so they're not a concern.


BUG: Minimizing Terminal Crash

Environment
Windows build number: 10.0.18362.175
Windows Terminal version: 0.2.1831.0

Steps to reproduce
Execute a command that keeps outputting something for a while.
for example you can user apt list or ls in a directory with a lot of content.
Then, while the command is running, double click on the titlebar or click the minimize? button.
Expected behavior
The behavior I would have expected was for it to pause or keep running
Actual behavior
The Terminal Freezes and crashes after approximately 1,5 seconds
Other Terminal windows are unaffected

I'm not sure if it's the same crash, but while trying to repro this, I managed to get an exception while restoring down from maximize while the app was outputting.
...
10 (Inline Function) --------`-------- conhost!wil::details::in1diag3::Throw_HrIf+0x17 [c:\ba\2009\s\dep\wil\include\wil\result_macros.h @ 5039]
11 0000008b`002ff6d0 00007ff6`ceb4e2e5 conhost!TextBufferCellIterator::TextBufferCellIterator+0x281 [c:\ba\2009\s\src\buffer\out\textbuffercelliterator.cpp @ 43]
12 (Inline Function) --------`-------- conhost!TextBuffer::GetCellDataAt+0x19 [c:\ba\2009\s\src\buffer\out\textbuffer.cpp @ 169]
13 0000008b`002ff740 00007ff6`ceb4cefa conhost!Microsoft::Console::Render::Renderer::_PaintBufferOutput+0x1c5 [c:\ba\2009\s\src\renderer\base\renderer.cpp @ 573]
14 0000008b`002ff8b0 00007ff6`ceb4ccca conhost!Microsoft::Console::Render::Renderer::_PaintFrameForEngine+0x1ea [c:\ba\2009\s\src\renderer\base\renderer.cpp @ 112]
15 0000008b`002ff9a0 00007ff6`ceb4f6a2 conhost!Microsoft::Console::Render::Renderer::PaintFrame+0x6a [c:\ba\2009\s\src\renderer\base\renderer.cpp @ 65]
16 (Inline Function) --------`-------- conhost!Microsoft::Console::Render::RenderThread::_ThreadProc+0x40 [c:\ba\2009\s\src\renderer\base\thread.cpp @ 165]
17 0000008b`002ff9d0 00007ffd`26ef18a4 conhost!Microsoft::Console::Render::RenderThread::s_ThreadProc+0x52

Looked like it was trying to draw one of the lines that's now outside the viewport. Perhaps we're not locking the terminal properly for certain resize types? Or we've moved the viewport to a bad position on restore down? Fortunately, the repro is easy enough, so we just need to dig in more.
Thanks!

Leaving notes here for me after the weekend:
Trying to repro this, the terminal moves between [118x26]<->[207x61]
When we crash, the conpty's buffer has 61 rows with 118 chars each. We're trying to render row 60, and we want all 207 characters from it. Seems to imply that the buffer's width wasn't resized, but the height was.
That's not right.

Okay debugging more into this. For my repro, the resize is from [207x61] -> [118x26]
When we resize in ConPty, we call both SetConsoleScreenBufferInfoEx and if that succeeds, SetConsoleWindowInfo. In SetConsoleScreenBufferInfoEx, we'll perform 2 ResizeWithReflow operations(!).

The first will resize from [207x61] -> [118x61], changing the width of all the rows.
The second will resize from [118x61] -> [118x26], changing the height of the buffer. The second resize is triggered by the ScreenInfo::SetViewportSize call near the bottom ofSetConsoleScreenBufferInfoEx.

During this resize, we'll copy the first 26 rows (up to iOldRow = 25) just fine. However, the problem comes when we try to newline the cursor. When that happens on the 25th row, we'll attempt to circle the buffer, to shift all the buffer contents up a row, and give us a new row to work with. Hypothetically, this will happen 35 more times, as we copy all the old lines into the new buffer.
HOWEVER, as we call TextBuffer::IncrementCircularBuffer, we'll also tell the renderer that it should give the render engines a chance to paint, before any old text was lost. This is helpful when a newline is normally output to the terminal, but isn't really necessary to do during a resize.
The renderer will ask the VT engine if it wants to paint to handle the buffer circling. The VT engine will always say "yes".
The renderer will then immediately render a new frame for the VT engine. Somewhere during this frame, the vt engine will throw an exception.
The exception will get caught way back up in SetConsoleScreenBufferInfoEx. The rest of the buffer will fail to get copied, and even worse, we won't even move the [118x26] text buffer into the screen buffer, leaving the screen buffer with a incorrectly sized buffer.

Later on, we'll try and render a new frame, and try to render the 60th line of the buffer, with a width of 207 chars. The buffer does incorrectly still have 61 lines, but it only has 118 chars per row, so the renderer throws an exception that's not caught here, crashing conpty. Conpty crashing causes the terminal to crash.
Now we just need to come up with the right solution for handling this resize. Very simply, we could maybe tell conpty to not paint at all during a resize. Might be too risky a change for 0.3, however.

Maybe, kind of, #1946? It got lost somewhere in the swarm of issues few weeks ago :(

@mcpiroman It's probably related to the larger class of "conpty resizing issues". I think it might be able to improve the QOL for the terminal by fixing this particular bug now, while a more complete design of the right way to handle conpty resizing is a while off. I do still owe you a response on that other issue though.


Add license to image and gemspec

This adds the LICENSE to the home directory of the Docker image.

The dependabot-* gems on rubygems currently have the license set as 'nonstandard' and do not include a license file.
TBD: rake gems:release will not include the LICENSE in the dependabot-* gems in this PR.

The scope of this PR has been reduced to only including the LICENSE file in the Dependabot-Core Docker image. We will punt on adding the LICENSE to the dependabot-* Gems themselves since it's non-trivial with this Nonstandard license.
Since the LICENSE will not be included in the Gems, we will also have to omit the dependabot-* Gems from being vendored with licensed, but that isn't that big of an issue since Core is owned by us and anyone can look at our required Gems for licensing info.

Thank you 4 being sk helpful. Truly appreciate it


Keywords color disappears when renaming open SQL files

Hi,
DBeaver version: 3.7.8
OS: Linux Mint 17.3
Steps to reproduce:

Create a new script file (Ctrl + F3). Its name is "script.sql";
Add a few SQL commands;
Rename the file without closing it (F2), without precising the ".sql" extension (it is done automatically);
In the SQL editor, words coloration has disappeared.

I did not notice this bug in older versions.
Regards,
Cedric

Thanks for report


Ability to use the equivalent of --build-arg when building an image

Is your feature request related to a problem? Please describe.
I would like the ability to pass arguments to the Dockerfile when building images with the Portainer UI. This would be equivalent to the following command:
docker build -t  --build-arg PORT=8080 .
Describe the solution you'd like
A similar solution to how environment variables can be added for creating containers. You can add as many as necessary with the ability to remove arguments too.
Describe alternatives you've considered
I've tried other cluster management tools but I enjoy working with Portainer more and it prevents me having to manually type commands to build images etc.

I'm happy to work on this feature, I would like to contribute to portainer if that's ok?

I'm happy to work on this feature, I would like to contribute to portainer if that's ok?

Hi @DomhnallBoyle that would be great! We always welcome contributions from the community!
For instructions on how to make a contribution make sure to read CONTRIBUTING.md which is in the Portainer code base.
Cheers,
The Portainer team

Is there an update for this?

@MGRashed this is not in our roadmap at the moment


Fix out of bounds accesses in imatcopy tests

Description
This patch fixes an issue where the imatcopy tests would occasionally segfault during the reference computation.
The issue was in the computation of the matrix size, for example taking the row major case, lda is used as the stride for the input matrix, so in row major, it means that the allocation needs at least lda number of rows. And lda is the maximum of m and n.
So if lda is m, there's no issue to use m x n as the size of the matrix, but if lda is n, that also means m < n, and therefore a m x n matrix size will lead to out of bound accesses, when using lda as a stride.
So here instead of taking lda x m for the row major matrix size, we need to take lda x n to ensure we have enough rows. And similarly, for row major we need lda x m.
This is also consistent with how it's done in
unit_tests/blas/extensions/imatcopy.cpp, I'm not sure why these two tests had it flipped.
May be related to: https://github.com/oneapi-src/oneMKL/issues/245
Checklist
All Submissions

[x] Do all unit tests pass locally? Attach a log.

I have tested this with cuBLAS and rocBLAS which were both showing occasional segfaults and they work fine, however they skip the actual tests so I'm not 100% sure the actual data is correct.

[x] Have you formatted the code using clang-format?

Bug fixes
To reproduce the crashes simply run the imatcopy tests with either cuBLAS or rocBLAS (although it should fail for all the backends since it's in the reference code), and run multiple times, as it depends on which random values of m and n are picked, or manually set n to be larger than m and it should trigger the issue.

[x] Have you included information on how to reproduce the issue (either in a
GitHub issue or in this PR)?

Ping @mmeterel can you give this short PR a quick second review so we can merge in this bug fix?

Ping @mmeterel can you give this short PR a quick second review so we can merge in this bug fix?

@andrewtbarker worth to check our internal tests.


Aggregate doc with account_id

Includes account_id as a part of the aggregated doc. https://github.com/cloudfoundry-incubator/cf-abacus/issues/175

Looks OK to me, merging. Thanks for the contribution!


Fix TypeError on empty translate key

Fixes #51, or rather the underlying bug.

Hm, I cannot reproduce the problem, i.e. when I click on 'publish' just nothing happens (besides the first two mandatory fields turning red (validation process?)). Deploying this PR doesn't change anything. I probably miss something.

@dr0i This is probably because the default schema has been updated in the meantime see https://github.com/dini-ag-kim/lrmi-profile/commit/6452adb5704962e7fbf0ccc8ab0e03944555a3ff. For testing this you should use this schema: https://raw.githubusercontent.com/dini-ag-kim/lrmi-profile/7aae393f58ebfa4a1e7c3c8f87ecc041f2a9747b/draft/schemas/schema.json

@acka47 ah - thx this did the trick :)
Getting no more blank side but now I get an Error.oneOf a the top of the side after clicking publish.

Deployd to test and production, closed.


nixos/wireless: disable SAE by default

Motivation for this change
Unfortunately this is not working as expected and results in
authentication failures in some cases. See issue #151729.
Things done

22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

WPA3 (SAE) requires 802.11w and it's no surprise that logins fail with pmf=0, and I firmly believe that adjusting our module towards compatibility with draft 802.11n adapters is a way in the wrong direction.

I want to support WPA3, but breaking any working setup is not acceptable. Yes, enabling 802.11w with pmf=1 kinda works, but older clients will still fail to authenticate. IMO wpa_supplicant should allow falling back to WPA-PSK if SAE failed, but I don't think this is possible as of now.

Issues like these come up when the AP requests MFP unconditionally. For that specific scenario hostapd offers the combination of ieee80211w=0 and sae_require_mfp=1, so that legacy adapters can bypass MFP.

Unfortunately access poins are usually locked down and end users can't really configure any of this.
So, I really don't want to just blame the AP and call it a day when users report they can't access their network anymore.

IMO wpa_supplicant should allow falling back to WPA-PSK if SAE failed, but I don't think this is possible as of now.

This expectation is understandable from the POV of a person having the goal to connect to a wireless network with the SSID & passphrase. The important part here is in fact, that this behavior would allow downgrade of the key negotiation to WPA2 + disabled PMF.
That being said, if i understand the current implementation correctly, this is also the case at the moment. Someone configures a network in expectation to use WPA3-SAE without specifying the key management in its config. This configuration however will happily connect using a regular PSK AKM.
Regarding older clients: The kernel exposes management-frame protection capability to userspace. I'm not sure how wpa_supplicant handles this currently, but you might be able to exclude AKMs with PMF-enabled this way.
This is also only a compromise. The only real solution (in my opinion) would be to force the decision about the key management to use upon the user, as using wpa_supplicant is not able to do this implicitly upon first connection like e.g. NetworkManager does.
(All this is from the perspective of a User who never used NixOS, so my assumptions might be completely off track)

To confirm if I understood your contribution correctly allow me to rephrase it:
The culrpit in our module is that our default setting for authProtocols enables all these key_mgmt types by default and we should instead make the user set this value explicitly.

and we should instead make the user set this value explicitly

Though most users probably don't care and shouldn't need to know about key management protocols (as explicited by the original issue reporter).
If we decide to officially drop support for old hardware by requiring PMF by default, that's fine I guess. However, I don't think expecting users to configure these details is a good idea.

Though most users probably don't care and shouldn't need to know about key management protocols (as explicited by the original issue reporter).

Then they should probably use a frontend (like NM) that takes care of that abstraction for them.

If we decide to officially drop support for old hardware by requiring PMF by default, that's fine I guess.

It is not NixOS requiring PMF by default, it's access points and WPA3s mandatory requirement for it.

However, I don't think expecting users to configure these details is a good idea.

The wpa_supplicant is a low level interface to configuring WLAN connections, so I do think that it can be expected, since the errors stems from our misleading default. I

The wpa_supplicant is a low level interface to configuring WLAN connections, so I do think that it can be expected, since the errors stems from our misleading default.

Then they should probably use a frontend (like NM) that takes care of that abstraction for them.

The networking.wireless module is actually pretty high-level and it barely mentions being wpa_supplicant.

Went with https://github.com/NixOS/nixpkgs/pull/154130 instead.


Add markdown linter GH Action

This PR introduces a new GH Action specifically for linting the markdown files in the repository. It relies on the https://github.com/github/super-linter project which seems to be the most maintained GH Action for linting and specifically for markdown. It only lints markdown files and does not use any of the other linters included, since golangci-lint is used for Go code in the project.

I suppose we could move this under the existing sanity/lint action?

I suppose we could move this under the existing sanity/lint action?
That seems reasonable to me.

Also: should we separate out the markdown linting violations into it's own commit?

Also: should we separate out the markdown linting violations into it's own commit?

Yes, they should be in their own commit. I squashed everything but will try to split it out 👍


[SPARK-34036][DOCS] Update ORC data source documentation

What changes were proposed in this pull request?
This PR aims to update SQL documentation about ORC data sources.
Why are the changes needed?
This document is not up-to-date. Apache Spark 3.2.0 can utilize new improvements from Apache ORC 1.6.6.
Does this PR introduce any user-facing change?
No, this is a documentation.
How was this patch tested?
Manual.
SKIP_API=1 jekyll build

BEFORE

AFTER

Could you review this please, @maropu , @viirya , @HyukjinKwon ?

Could you review this please, @maropu , @viirya , @HyukjinKwon ?

Test build #133759 has started for PR 31075 at commit b7baa0c.

Test build #133759 has started for PR 31075 at commit b7baa0c.

Test build #133759 has finished for PR 31075 at commit b7baa0c.

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

Test build #133759 has finished for PR 31075 at commit b7baa0c.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38347/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38347/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38347/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38347/

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

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

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

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

The PR is updated. Thanks, @sunchao and @viirya .

The PR is updated. Thanks, @sunchao and @viirya .

Test build #133762 has started for PR 31075 at commit d9279dc.

Test build #133762 has started for PR 31075 at commit d9279dc.

Test build #133762 has finished for PR 31075 at commit d9279dc.

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

Test build #133762 has finished for PR 31075 at commit d9279dc.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38350/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38350/

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

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

Test build #133765 has started for PR 31075 at commit 2024f01.

Test build #133765 has started for PR 31075 at commit 2024f01.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38350/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38350/

Test build #133765 has finished for PR 31075 at commit 2024f01.

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

Test build #133765 has finished for PR 31075 at commit 2024f01.

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

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

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

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

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

Thank you, @viirya . Merged to master.

Thank you, @viirya . Merged to master.

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

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


fix(example): feature

rename rust dependency feature from cheader to generate-c-header

Superseded by https://github.com/libbpf/libbpf-bootstrap/pull/187


updates to policies

based on convo with @withoutboats and @carols10cents

@bors: r+
🎊

:pushpin: Commit 6fb3e68 has been approved by steveklabnik

:hourglass: Testing commit 6fb3e6804f1471412aec5621e75ef1d6965321d9 with merge d8cf6027f2edffacceaa34082350ada80215f458...

:sunny: Test successful - status-appveyor, status-travis
Approved by: steveklabnik
Pushing d8cf6027f2edffacceaa34082350ada80215f458 to master...


Moving integration tests to proper folder

Hi @neil-greenwood since this wouldnt merge as it is I did the manual refactor and added the changes in 542e84 and gave you credit in there for the work. Thanks for the help!


Vault intermittently grinding to a halt while typing text into a note

Describe the bug
I have no idea what the bug is, simply opening a note and typing some text causes Iconize to do strange things in the background
To Reproduce
As above
Expected behavior
Typing text should not cause Iconize to lock up the vault
Screenshots
This is Iconize locking everything up with every keystroke

Additional context
See a sample call tree

I'm deprecating the use of iconize from my vaults, so I'm closing the issue


Ctrl + Backspace and Ctrl + Left/Right arrows skip underscores for subword navigation in the script editor/TextEdit

Godot version: 4.0 e1e52b373
OS/device including version: Windows 10
Issue description:
Ctrl + backspace and ctrl + left/right arrow only consume alphabats and digits but skips underscores.

Steps to reproduce:
Minimal reproduction project:
N/A

cc @bruvzg

Ah, nevermind there is PR opened to fix that, I missed it, sorry

Fixed by #47604.


"<" was missing in formatting

would show up in the assessment as "b>Multiple releases every cycle (sprint / iteration)"

thanks for fixing @saberprivateer !


Unable to deploy due to corporate proxy

I'm using Sparkle to deploy to our Office 365 Dynamics organisation.  It all works well on my personal laptop but fails from my work machine which needs to go through the corporate proxy.
I receive the following:
The application terminated with an error.
Metadata contains a reference that cannot be resolved: 'https://disco.crm6.dynamics.com/XRMServices/2011/Discovery.svc?wsdl&sdkversion=9'.
at System.ServiceModel.Description.MetadataExchangeClient.MetadataRetriever.Retrieve(TimeoutHelper timeoutHelper)
at System.ServiceModel.Description.MetadataExchangeClient.ResolveNext(ResolveCallState resolveCallState)
at System.ServiceModel.Description.MetadataExchangeClient.GetMetadata(MetadataRetriever retriever)
at System.ServiceModel.Description.MetadataExchangeClient.GetMetadata(Uri address, MetadataExchangeClientMode mode)
at Microsoft.Xrm.Sdk.Client.ServiceMetadataUtility.RetrieveServiceEndpointMetadata(Type contractType, Uri serviceUri, Boolean checkForSecondary)
at Microsoft.Xrm.Sdk.Client.ServiceConfiguration`1..ctor(Uri serviceUri, Boolean checkForSecondary)
at Microsoft.Xrm.Sdk.Client.ServiceConfigurationFactory.CreateManagement[TService](Uri serviceUri, Boolean enableProxyTypes, Assembly assembly)
at Microsoft.Xrm.Sdk.Client.ServiceConfigurationFactory.CreateManagement[TService](Uri serviceUri)
at Microsoft.Crm.Sdk.Samples.ServerConnection.GetProxy[TService,TProxy](Configuration currentConfig)
at Microsoft.Crm.Sdk.Samples.ServerConnection.GetDiscoveryProxy()
at Microsoft.Crm.Sdk.Samples.ServerConnection.GetOrganizationAddress()
at Microsoft.Crm.Sdk.Samples.ServerConnection.GetServerConfiguration(Boolean ignoreLocalPrincipal)
at SparkleXrmTask.Program.Run(CommandLineArgs arguments)
at SparkleXrmTask.Program.Main(String[] args)
The remote server returned an error: (407) Proxy Authentication Required.
Press any key to continue . . .

I was able to get through the corporate proxy by adding a defaultProxy setting to the {app}.config file


Operators in Azure Cosmos DB for MongoDB vCore

hi,
in this article https://learn.microsoft.com/en-us/azure/cosmos-db/mongodb/vcore/compatibility#query-language-support it is mentionned that $map, $reduce & $addToSet are not yet supported. Any idea of when it would be ? As we need these operators for our app to work.
thank you in advance

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

ID: 43a8d9b2-59d2-0a37-b645-fdd1133b9e72
Version Independent ID: 30014099-4923-e3f6-03c2-7bfa69d4c14f
Content: Compatibility and feature support - Azure Cosmos DB for MongoDB vCore
Content Source: articles/cosmos-db/mongodb/vcore/compatibility.md
Service: cosmos-db
Sub-service: mongodb-vcore
GitHub Login: @suvishodcitus
Microsoft Alias: suvishod

@baouchemaya
Thanks for your feedback! We will investigate and update as appropriate.

@baouchemaya - Although these commands are not supported at the moment, we value your feedback. Your request has been acknowledged, and we are committed to making them available as soon as possible.
Thank you for your understanding.

Hey, these commands ($map, $reduce & $addToSet) are quite basic and widely used for mongo queries, would you guys have an estimation when it would be available?

+1, we use these in a lot of place in our project

Thank you for reaching out,
Unfortunately, this is the documentation GitHub repository, and we cannot troubleshoot issues, answer product questions, or provide technical support.
For technical support, please open a support ticket in the Azure portal. For questions about the product, please post them in Microsoft Q&A.
#please-close


Is nanodet working in tflite?

i am trying to use the file from saved_model_416x416//model_float16_quant.tflite
it loads and seems like inferencing but when i print interpreter.get_tensor(self.output_details[1]['index'])[0] i get:
[[ 1.6956738   0.9603495   0.08334054 ... -0.43478656 -0.615623
  -0.69419366]
 [ 1.3914261   1.064209    0.5480403  ... -0.3135112  -0.6691925
  -1.055948  ]
 [ 1.116673    0.7275144   0.21141839 ... -0.33032337 -0.8020203
  -1.3268852 ]
 ...
 [ 1.1918403   0.8403666   0.23102881 ... -0.78871083 -1.1012893
  -1.5689907 ]
 [ 1.1498222   1.2557286   0.9366938  ... -0.8202821  -1.1629286
  -1.481539  ]
 [ 3.8123865   2.9635813   0.75818133 ... -0.7893712  -1.0882326
  -1.1440537 ]]

do i need to post process it?
if yes, how? any help will be greatly appreciated.

do i need to post process it?

Yes.

if yes, how? any help will be greatly appreciated.

You will need to implement Post-Process yourself in the repository cited below.
https://github.com/RangiLyu/nanodet/blob/main/demo/demo.py

do i need to post process it?

Yes.

if yes, how? any help will be greatly appreciated.

You will need to implement Post-Process yourself in the repository cited below.
https://github.com/RangiLyu/nanodet/blob/main/demo/demo.py

Mmm i see... A tad more complicated than I thought. Also, code looks a bit inefficient, for example the sigmoid is called for each box (row) and not on the array. Thank you!

Mmm i see... A tad more complicated than I thought. Also, code looks a bit inefficient, for example the sigmoid is called for each box (row) and not on the array. Thank you!


Fixing tests that need to run locally

Alright, considering CI is passing here and there hasn't been any activity since January I'm gonna go ahead and merge.
Thanks @mauricioszabo


Preserve active sheet index when calling store()

This pull request follows on from the handy #987
A small fix to preserve the users active sheet index when a call is made to store() in the writer.
My use case is as follows (A spreadsheet with 2 sheets):

Excel::load('spreadsheet_with_2_sheets.xlsx', function($reader) {
	// makes some modifications to sheet 1
	// ...
})
->setActiveSheetIndex(0)
->store('xlsx');

Currently within the store() the call to $this->_render() will overwrite my index of 0 resulting in the outputted spreadsheet always showing sheet 2 by default when opened in Excel.

Wouldn't it be better to move this to the _render() method so it will work for ->export() and ->download() too?

Yep totally - have updated PR.

Is this accessible via a release tag yet?

Reply to this email directly, view it on GitHub, or mute the thread.


Bugfix: list.focusForeground defaulted to transparency

Issue: In the file explorer, when focusing an item the text disappeared
Defect: list.focusForeground defaulted to unspecified which, unless otherwise handled, equals transparency. This is according to vscode's default definition, but they often then fall back to handling it in some other convoluted way.
Fix: Set the default for list.focusForeground to the color of foreground

/azp run


⚠ Bofrost.be has degraded performance

In 60b5958, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1341 ms

Resolved: Bofrost.be performance has improved in a48a70d after 13 minutes.


Create day2-best-practices-for-supporting-developers-to-contribute-to…

…-open-source-and-or-innersource.md
Signed-off-by: Jonas van den Bogaard<EMAIL_ADDRESS>Signed-off-by: Jonas van den Bogaard<EMAIL_ADDRESS>

This PR is no longer needed. The notes are already added by Ana.


Bug: Browser crashes when there are several nodes

Description
When displaying several nodes (e.g 500 nodes) the browser crashes
Steps to Reproduce
Provide a ruge array of nodes (e.g 500 nodes) to the flow
Expected Result
The nodes should show (virtual scroll?)
Actual Result
The browser becomes slower and may crash
Environment Details
Browser: 130.0.6723.70 (Official Build) (arm64)
Library Version: 16.0.5
Angular Version: 18.2.0
Screenshots or Videos
No response
Additional Context
No response

I will check it. Please provide screenshot and stacktrace.

@anderson-896 Not reproduced. I added 2500 nodes


Added localAddress on SocketConfig

With this configuration, it will be much easier for servers with multiple IPs to rotate their connection IPs.
Below is an example of how to list the machine's current IPs:
import { networkInterfaces } from "os"

let localAddresses = []
for (let i of Object.values(networkInterfaces())) {
    for (let info of i) info.family === 'IPv4' && !info.internal && localAddresses.push(info.address)
}

let randLocalAddress = localAddresses[Math.floor(Math.random() * localAddresses.length)]

makeWASocket({
  // ...
  localAddress: randLocalAddress,
  // ...
);

Sorry for the duplication. I ended up deleting the new repository instead of the oldest one I had and ended up closing the previous pull request.

Updated exemple...

Can't we just set a premium proxy system-wide to rotate the IP?

Can't we just set a premium proxy system-wide to rotate the IP?

Yes, it is possible. However, this way is much simpler and easier.
And it's even possible to do it today by extending the 'WebSocketClient' class and replacing the 'connect' function. After setting it in 'makeWASocket({ socket, ... })'.
Once again, this approach significantly simplifies the process.


Introduce "Step Declaration" File

Summary
Right now, you can define Feature Scenarios in the .feature file. However, by looking at only the Feature file, you are not able to tell what are the exhaustive list of available Steps to be used in every Scenario. To know the list of Steps, one must check the Step Definition source code, which could be located in a place where the tester is unfamiliar, or might not even be in the same repository (for multi-repo architectures, where multiple repositories depend on one Behavior definition repository). Not only this makes it hard for the person who writes the Feature definitions to know all the steps made available to them, it also removes any possibility in developing a compile-time or syntax-highlighter check if the Feature files are correct.
Expected Behavior
For us to be able to: review/access a list of steps available for Feature definitions, build syntax highlighters, and check the correctness of Feature files at compile-time rather than runtime.
Current Behavior
Right now, none of this is possible.
Possible Solution
Introduce "Step Declaration" File
Ideally, we can "declare" Steps in a file, just like how we define Features. Imagine a *.step file, containing just the list of step regular expressions, and no implementation. A Feature file could then import multiple *.step files at the header, which opens up so many possibilities in terms of scoping, syntax highlighting, and compile-time checking.
Context & Motivation
When you're working across multiple repositories, where many repos implement the Feature definitions in their tests, it's very useful to have a list of Steps that is agreed as "services" that a system provides. The writer and maintainer of the Feature files do not need to know where and how the Steps are defined - which could be at multiple repositories.
Think of it as an "interface to Step definitions". Just the list of steps the system(s) support, the syntax and parameters it expects, and nothing else. This could take Cucumber to the next level as a BDD framework for large codebases sitting across multiple repositories.
Your Environment

Link to your project: https://github.com/graknlabs
We haven't made our BDD feature definition repository public

I'm closing this as it seems to be a duplicate of #521

Issue #521 is exactly the same as the one I'm proposing, @aslakhellesoy. They are similar, yes. But not the same.

Issue #521 is not exactly the same as the one I'm proposing, @aslakhellesoy. They are similar, yes. But not the same.
In this solution I'm proposing, the developer will be able to write and maintain the "Step Declaration" file as source code.


Documentation update

Fix contribute badge
Clean up README
Remove TODO.md
Squashes miscellaneous commits (will require a force push to rewrite git history)

This PR squashes some commits that were meant to be squashed prior to merging, and also fixes the README and removes TODO.md


integration-guide-and-jdbc-updates.md

Updating this per our recomendation of including partner-source
What are you changing in this pull request and why?

Checklist

[ ] Review the Content style guide so my content adheres to these guidelines.
[ ] For docs versioning, review how to version a whole page and version a block of content.
[ ] Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

Adding new pages (delete if not applicable):

[ ] Add page to website/sidebars.js
[ ] Provide a unique filename for the new page

Removing or renaming existing pages (delete if not applicable):

[ ] Remove page from website/sidebars.js
[ ] Add an entry website/static/_redirects
[ ] Run link testing locally with npm run build to update the links that point to the deleted page

merging this @rpourzand and @amychen1776 !


Added Zoroark

I have added a new Pokemon which is not repeated on the original index.html. Please review & check.

Done. Thank you for contributing!


Importing Requests from apiDoc file

Tell us about the task you want to perform and are unable to do so because the feature is not available
APIDOC Website
APIDOC GitHub
Add support for importing requests from the apiDoc file. The developed code should be able to parse the apiDoc file, after processing let user choose the requests he wants to add via a dialog and create the corresponding requests in the API Dash collection.

Update: The frontend is already implemented.
The only change required will be the addition on apiDoc in ImportFormat enum and adding the parser code.


PB6 C-Outout

The new alpha versions of PB6 are shipped with pbcompiler.exe and pbcompilerc.exe, which can output c-code.
I'd like to see a new version supporting that. :-)

There seems also a "Optimize generated code" option now. I guess this activates the optimizer of the c compiler and was the reason why they changed to C.

From what I understood, that's part of the options that can be passed to GCC, which is something that will need to be considered when integrating the C backend into PBAsmUI — my guess is that in most cases users will require just the raw (and unoptimized) C code for this tool, since the goal would be having access to the C sources to tweak them before recompiling them again.
If I remember correctly, GCC optimizations operate on bytecode directly, so the goal for PBAsmUI should be to allow compiling to C sources, edit them, and then recompile them again (with whatever options GCC offers).
But these are just guesses based on my past experience with GCC, and I haven't tried it PB6 yet.

I'm using windows most of the time, too. But I think I could also do simple tests with linux, if you tell me what do you want to have tested.

That's fine. I know some Linux and macOS PB users which I might ask to join for testing.
Once I've managed to finish and publish the current project I'm working on, I'll start looking into this.

@SIRprise, I've been looking into this and I realize that we might be facing a problem regarding integration of the IDE code to handle version info, and other code that might be required to interface with the C backend, i.e. that this project is MIT Licensed and the PB IDE repository is GNU GPL, which are incompatible licenses.
One of the main reasons the PBAsmUI project was never finished was because we didn't have access to the required info on how to handle application version info, so that we could reuse that info when recompiling from custom tweaked FASM sources. That, and also some complicate aspects of handling PB project files (which can become quite entangled when some options are enabled, and which is not documented). Probably similar needs will be encountered for the new backend.
Now the IDE code is public, but due to license incompatibility it can't be integrated here. I could ask the original author if it's OK to switch to the GPL license, but the majority of the other libraries and assets I've collected so far are all MIT licensed. Also, the IDE project is dual licensed under GPL and Fantaise Software license, which might complicate things even further.
I'm no so sure it's worth the effort. Currently I'm using some custom scripts to generate ASM and/or preprocessed files from PB sources, which I've integrated directly in my file navigation tool (I don't use File Explorer, I use Altap Salamander, a proprietary tool which allows integration of scripts and plugins). Using a batch script, or creating a Ruby or Python script is quicker and doesn't require IDE integration via a tool.
Let's see what PB6 will look like once it's released in stable version. Honestly, I don't have big expectations like many others do. My general impression is that the PB language is becoming every year less usable, mostly due to poor standards choices which make it impractical to use it in production projects, especially in collaborative projects, since it doesn't blend-in well with the general ecosystem of the open source community.
Also, I don't really use much the PB IDE any more, I find it very clunky and impractical, and I prefer modern editors like SublimeText and VSCode. I only use the IDE for debugging, and a few other operations which are more practical therein, but almost never to actually edit code. So in the end, it would be better to invest time in a PB package for a decent editor, which could also support these features natively, rather than spending time writing a PB IDE tool.

@tajmone I don't understand that licensing problem: It is just a GUI for some CLI tools. I like the tool, but I don't see that there is an intellectual property to protect.
You are right with your thoughts about the editor and I also mensioned some problems I have with it in the german PB forum. And I also think the language could be dead in a few years. But for now I think the language itself is ok, it feels consistent if I don't use the examples from the help file.

I don't understand that licensing problem: It is just a GUI for some CLI tools. I like the tool, but I don't see that there is an intellectual property to protect.

The PBAsmUI was written by Horst Schaeffer, who originally made it available on his website as a freeware but without a license, then I asked him if it could releases it under MIT License, so I (and others) were free to change it, reuse it in other projects, etc.
Any source code is copyright of its creator by default, so it can't be reused in any way without explicit permission of its author. The MIT License is very permissive, it allows to reuse the code in any way (including commercial use) with the condition that the original MIT License text is fully quoted in the final application.
The PB IDE source code is released under the GNU GPL license + Fantasie Software License. The former is a well known open source license which imposes a long list of restrictions to its use (among them, the obligation of releasing any derivative work under the same license, and providing the full source code to end user). Unfortunately, the GPL license doesn't work well with other open source licenses due to some of its legal clauses being incompatible, which ultimately means that practically you can only use GPL licensed code and components in projects with the same license, and were all other components have the same license too. The additional Fantasie Software License complicates things further, for it's not clear whether it implies that derivative works should also be governed by the same license too, and whether it might affect compatibility with GPL licensed libraries, etc.
Legally speaking, the GPL license(s) are among the harder to interpret, and you really need to consult a lawyer to ensure that you're not violating its terms when using third party components under that license in your work.
For a tool like this one, it's strongly desirable that we might reuse part of the IDE sources, e.g. to handle scanning project files (the PB project file has undergone some tweaks in the last year), to handle the Manifesto info for Windows binaries (also stored in the project files), etc. Otherwise, we'll have to reimplement the whole projects systems from scratch, and tweak it whenever its format changes (undocumented), which is probably going to happen with PB6, since it's introducing a secondary backend and it will have to store C preferences in project files too (either implicitly on a per file or per folder base, depending on user settings, or explicitly in case of a project file).
I remember that when I was working on the PBAsmUI I wasn't able to handle some edge cases, e.g. the IDE can compile a single module with different settings, depending on whether you're compiling it as part of a project or as a standalone file. The PBAsmUI needs to be able to parse project files and IDE settings to work out these options in order to correct recompile the modified ASM/C sources, since it has to bypass the PBCompiler to achieve that. For these same reasons the original author never managed to complete the project.
Originally, PB projects files were neither documented nor their code available for study, whereas now we have at least access to their sources to study them, but we can't reuse that code except at the conditions of the GPL License.
Licenses compatibility is a known problem. With open source languages the tendency is to release libraries for that language in the same license as the language project, to maximize usability in its ecosystem, so in the Python world the most used license is MIT, in Rust MIT + Apache, etc. With proprietary languages is harder, since every user will chose an arbitrary FOSS license for its code, depending on personal choices, so the best we can hope if that most users will use permissive licenses (MIT, Apache, BSD, etc.).
Truth is that in the PB community it's common practice to just share code without a license, by saying "enjoy" or similar phrases which clearly indicate the author's will to share it with others, but which provides no legal ground for its reuse in any contexts beside personal use. When publishing third party code and assets here on GitHub, you have to strictly abide to Copyright laws and licenses, as indicated in the GitHub terms of use and EULA.

I'm moving this Issue to Discussions, since it's implementation is still an open possibility once PB6 is out and/or we can solve the license issues.


Jerryliu sc 69749 support signing modal on android

Make activity context mandatory in all APIs

This would prevent enabling UI from Auth Relayer side crashes the app

Add @Keep to nested EIP712TypedData

LGTM. OOC, seeing as we are now directly setting the provider context here, is there any concern that it won't get passed to the overlay?

"The overlay is a Webview widget set with internal access only, and cannot be accessed from external libraries such as our extension library. To allow for mutations, we provide the context property, but prevent access to the overlay from outside sources."


How to fill bird_ring_number in cr_birding_birds?

Premise for Lisenka
INBO "birds" table (called tbl_kleurring) is bird-oriented: one row = one bird, while SOVON table cr_birding_birds is ring-oriented: one row = one (color) ring.
The column MetaalringNummer in INBO  birds table contains the most recent metalring number! So, every time the metal ring is changed, this field is manually updated. Tracking the changes of the metal ring is practically almost impossible: the old metal ring numbers are saved in textual notes in some observation tables and not easy to filter.
As @EricStienen commented in https://github.com/inbo/sovon/issues/41#issuecomment-456115509 experts at INBO didn't bother too much about metal rings. And from Lisenka's comment https://github.com/inbo/sovon/issues/7#issuecomment-431014379 it seems not a primary information for SOVON db, being a color ring oriented database as well.
Question for Lisenka
May we leave this field filled with the most recent metal ring as in INBO db although it could be not the real one at the moment of ringing?
Example (bird 2 got metal ring before color ring)

bird_id
bird_reference
bird_shorthand
bird_ring_number
bird_date_begin
bird_date_end

SOVON's field
1
AAAA
L....00001
2010-01-01
2014-05-04

SOVON's field
1
AABB
L....00001
2014-05-04

SOVON's field
2

L....00002
1999-03-01
2003-01-01

SOVON's field
2
BBAJ
L....00002
2003-01-01
2005-01-01

SOVON's field
2
BBAY
L....00002
2005-01-01

Lisenka's answer:

Het is wat mij betreft prima om de laatst bekende metalen ring in te vullen.

Our approach is approved by SOVON. Issue can be closed.


#general

Hi chat

hi fifty :)

hi holo :)

Aight, we had our fun. I'm closing this in an effort to clean up issues a bit :P


Add initial version of MusicVAE deeplearn implementation.

@hapticdata
@torinmb

for the standalone build I highly recommend changing --standalone music-vae to --standalone musicVAE.
Using a hyphen would require the user to access it as new window["music-vae"].MusicVAE() as opposed to just new musicvae.MusicVAE().
Also, because deeplearn would be a required dependency to install this via npm/yarn it should be a dependency and not a devDependency
Similarly http-server should be a devDependency and not a dependency because it is only necessary for viewing the example, not as part of a build.


Set Up and Generate a New SAP Fiori App Project

When I generate a project and then start it, I get the following error:

Error [ERR_SOCKET_CLOSED]: Socket is closed
at Socket._writeGeneric (net.js:772:8)
at Socket._write (net.js:794:8)
at doWrite (_stream_writable.js:377:12)
at clearBuffer (_stream_writable.js:517:7)
at Socket.Writable.uncork (_stream_writable.js:317:7)
at ClientRequest._flushOutput (_http_outgoing.js:883:10)
at ClientRequest._flush (_http_outgoing.js:852:22)
at _http_client.js:323:47
at callSocketMethod (_http_client.js:818:7)
at ClientRequest.onSocket (_http_client.js:823:7)

It is probably due to the middleware fiori-tools-proxy. Is it possible that the error occurs because of the company proxy? The environment variables http_proxy, https_proxy and no_proxy are already set.
Thanks and best regards

@whitenrock can you please confirm that your proxy is correctly configured and accessible?
Also, could you please detail how you created your project?  Was it using the Fiori elements Generator?

Thank you for your feedback. We haven't heard from you in a while, so we are closing the issue.
If you still have questions, feel free to reopen the issue.


Optimize generated dissector code

Currently, conditions for protocol fields are checked multiple times. The conditions of all variants of a field are checked in the validation function to decide if any variant is valid. If one valid variant was found and the corresponding accessor function is used, the conditions of the variants are checked again to find the currently valid variant and determine the location of the field. We should try prevent these double executions.

We had the following idea to realize that optimization:
All paths through a PDU model can be identified by a unique path number. Each node (PDU field) on a path gets a position number. The validation function is changed to return a token instead of a boolean value. The token (represented by a private record with path and position components) specifies which path is valid for a given input buffer, and until which node (field) it has been validated. The introduction of an additional token validation function is needed, which establishes the necessary postconditions for the accessor function. Finally, the accessor function is able to decide based on the values in the token which variant is valid instead of checking all conditions again.
The usage could look like this:
T := Valid_Field (Buffer, T);
if Valid_Token_Field (T) then
   X := Field (Buffer, T)
end if;

We are thinking about using the same mechanism also between PDUs. We will need a function, which checks that the conditions are met (e.g., a type field is set to a specific value), and exchanges in this case the token of the lower layer protocol to a token of the appropriate higher layer protocol.

Fixed by #32.


Fix OpenSearch integration fields type error

Description
Fix the error on OpenSearch Integration when accessing the Discover module.
The updated fields are:

data.virustotal.scan_date
data.osquery.calendarTime

Evidence
https://github.com/user-attachments/assets/3ce18699-78c1-4450-84ff-08fe268dc996
Related Issues
Resolves #448

Check List

[ ] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

Discover view doesn't throw any errors either:


sortByRank function failing in python 3.7

Hello - thanks for the great paper and for making the code available.
It adjusted a few lines to be compatible with python 3.7+, but I am stuck on this one given I don not get the logic:
		def sortByRank(a, b):
			for phase in range(len(params['phases'])):
				i = len(params['phases']) - phase - 1
				comp = cmp(b['finished'][i]['aveEff'], a['finished'][i]['aveEff'])
				if comp != 0:
					return comp
				comp = cmp(a['finished'][i]['aveMon'], b['finished'][i]['aveMon'])
				if comp != 0:
					return comp
			return comp

the cmp variable is noit referened (typo ?), and the call: vaccines.sort(key=sortByRank) fails due to
TypeError: sortByRank() missing 1 required positional argument: 'b'

Any help ? Thanks a lot !

Hi,
This function sorts vaccines based on their average success at each phase (backwards). If they are the same it compares the average times instead. cmp is a function that seems to have gone away in Python 3. Someone suggests replacing it with this:
def cmp(a, b):
return (a > b) - (a < b)
I haven't tested this.
Cheers
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Steve Lloyd
Emeritus Professor, School of Physics and Astronomy
Queen Mary University of London, Mile End Road, London E1 4NS, UK
E-mail<EMAIL_ADDRESS>+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

On 20 Oct 2020, at 10:59, hdavit<EMAIL_ADDRESS>wrote:
Hello - thanks for the great paper and for making the code available.
It adjusted a few lines to be compatible with python 3.7+, but I am stuck on this one given I don not get the logic:
  def sortByRank(a, b):
  	for phase in range(len(params['phases'])):
  		i = len(params['phases']) - phase - 1
  		comp = cmp(b['finished'][i]['aveEff'], a['finished'][i]['aveEff'])
  		if comp != 0:
  			return comp
  		comp = cmp(a['finished'][i]['aveMon'], b['finished'][i]['aveMon'])
  		if comp != 0:
  			return comp
  	return comp

the cmp variable is noit referened (typo ?), and the call: vaccines.sort(key=sortByRank) fails due to
TypeError: sortByRank() missing 1 required positional argument: 'b'
Any help ? Thanks a lot !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Error fetching abapGit Repositories

I recently updated the ADT backend eclipse package, and now I get an error when accessing the functionality in eclipse:

My the ADT_Backend branch is synced with origin.
-abapGit for ABAP Development Tools (ADT)	1.4.0

We are working on providing a fix.
Till that time please use an older version of the abapGit plugin for ADT, v1.3.1 recommended.
abapGit for ABAP Development Tools (ADT) 1.3.1 can be manually installed in ADT via zip, that can be found at  https://github.com/abapGit/ADT_Frontend/releases/tag/v1.3.1.


[Release-0.4] Fix kmesh daemon graceful exit

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #654
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/lgtm
/approve