No type error when non-comparable data types within a tuple

 Environment data

Language Server version: 2023.3.20
OS and version: darwin arm64
Python version (and distribution if applicable, e.g. Anaconda):
python.analysis.indexing: true
python.analysis.typeCheckingMode: strict

 Code Snippet

class A:
    pass

a = A()
b = A()

x = a > b
print(x)

at = (a,)
bt = (b,)

x = at > bt
print(x)

 Repro Steps

On     "python.analysis.typeCheckingMode": "basic"

 Expected behavior
Line with `at > bt` should be a type error since `tuple[A]` aren't comparable. This error should be similar to one in `a > b`

 Actual behavior
No type error
 Logs

XXX

Thanks for the issue.
This is expected. Tuple defines the __le__ operator while class A does not:
class tuple(Sequence[_T_co], Generic[_T_co]):
    def __new__(cls, __iterable: Iterable[_T_co] = ...) -> Self: ...
    def __len__(self) -> int: ...
    def __contains__(self, __x: object) -> bool: ...
    @overload
    def __getitem__(self, __x: SupportsIndex) -> _T_co: ...
    @overload
    def __getitem__(self, __x: slice) -> tuple[_T_co, ...]: ...
    def __iter__(self) -> Iterator[_T_co]: ...
    def __lt__(self, __x: tuple[_T_co, ...]) -> bool: ...
    def __le__(self, __x: tuple[_T_co, ...]) -> bool: ...
    def __gt__(self, __x: tuple[_T_co, ...]) -> bool: ...
    def __ge__(self, __x: tuple[_T_co, ...]) -> bool: ...
    @overload
    def __add__(self, __x: tuple[_T_co, ...]) -> tuple[_T_co, ...]: ...
    @overload
    def __add__(self, __x: tuple[_T, ...]) -> tuple[_T_co | _T, ...]: ...
    def __mul__(self, __n: SupportsIndex) -> tuple[_T_co, ...]: ...
    def __rmul__(self, __n: SupportsIndex) -> tuple[_T_co, ...]: ...
    def count(self, __value: Any) -> int: ...
    def index(self, __value: Any, __start: SupportsIndex = 0, __stop: SupportsIndex = sys.maxsize) -> int: ...
    if sys.version_info >= (3, 9):
        def __class_getitem__(cls, __item: Any) -> GenericAlias: ...

That's the definition of tuple in the typeshed.
https://github.com/python/typeshed/blob/a544b75320e97424d2d927605316383c755cdac0/stdlib/builtins.pyi#L918

Oh maybe you expected it to behave like the runtime does. Not that __gt__ wasn't defined on A, but that Pylance isn't doing what the runtime does for tuples.
Running this code produces the same TypeError we get from comparing the plain objects:
class A:
    pass

a = A()
b = A()

at = (a,)
bt = (b,)

greaterThan = at > bt
print(greaterThan)

produces:
TypeError: '<' not supported between instances of 'A' and 'A'

@erictraut I assume this isn't feasible in the type evaluator? We wouldn't know what the internal logic of tuple does because it's not written in python.

Pyright (the type checker upon which pylance is built) is doing the correct thing here. As @rchiodo mentioned, class A doesn't have a __gt__ method, but tuple does. The signature for tuple.__gt__ indicates that it accepts an argument of type tuple[_T_co, ...], which in this case is compatible with tuple[A]. From the perspective of a type checker, there is no type violation here.
Static type checkers are not guaranteed to catch all runtime errors. This is an example of where it's not possible to catch such an error.

Thanks both for replying to my issue.
I think it's quite clear to me now that the issue isn't in the type checker. It's following the typeshed.
I did expect it to behave it like the runtime in this case. I agree with @erictraut that the typechecker can't possibly detect all errors. This one though, does seem to be something that Python's type system should be able to model.
Would you agree that if we had a protocol called SupportsGE[_] which denoted anything that had __ge__, then we could've done def __gt__(self, __x: tuple[SupportsGE[_T_co], ...]) -> bool: ... ?
I'm just trying to make sure my understanding of Python's type system is correct.

When you say "could've done", are you positing an alternate definition of tuple.__gt__? It's more complicated than that because the Python interpreter's implementation of the > operator involves multiple steps where it first calls a __gt__ magic method on the LHS operand and then falls back to a __lt__ magic method on the RHS operand.

@erictraut Sorry for the late reply (my notifications are a bit unreliable).
Yes, I mean having an alternate definition of tuple.__gt__ in the stubs. Thanks for sharing an example of the complexity. I'm sure there are many others. I also realized that we'll have to add type to self which will create issues.
On the particular issue that raised:
So I'm not sure what the current signature implies:
def __gt__(self, __x: tuple[_T_co, ...]) -> bool: ...

To me it reads as: __gt__ is defined when the RHS is a homogeneous tuple.  Is that true? If it's the case then the case you're talking about becomes a little easier. We can simply type it as:
@overload
def __gt__(self: tuple[_SupportsGT[_T_co], ...], __x: tuple[_T_co, ...]) -> bool: ...
@overload
def __gt__(self, __x:  tuple[_SupportsLT[_T_co], ...]) -> bool: ...

@erictraut Sorry for the late reply (my notifications are a bit unreliable).
Yes, I mean having an alternate definition of tuple.__gt__ in the stubs. Thanks for sharing an example of the complexity. I'm sure there are many others. I also realized that we'll have to add type to self which will create issues.
On the particular issue that raised:
So I'm not sure what the current signature implies:
def __gt__(self, __x: tuple[_T_co, ...]) -> bool: ...

To me it reads as: __gt__ is defined when the RHS is a homogeneous tuple.  Is that true? If it's the case then the case you're talking about becomes a little easier. We can simply type it as:
@overload
def __gt__(self: tuple[_SupportsGT[_T_co], ...], __x: tuple[_T_co, ...]) -> bool: ...
@overload
def __gt__(self, __x:  tuple[_SupportsLT[_T_co], ...]) -> bool: ...


Fix quest type filter width

Since English strings can be longer

vs

Seems we could simply omit the redundant QUEST for each category selection

On Lumendark skin that is still (without Quest)

So maybe both drop Quest and increase the width just a bit?

Please feel free to replace all occurrence of QUEST, e.g. replace all Composition Quest with Composition

I did https://github.com/poooi/plugin-quest/pull/49/commits/cedb151c26377efbf1964d8d22fe0a765b3cc0ec and https://github.com/poooi/plugin-quest/pull/49/commits/afab0c95bce9b1b0117b598fb784a8f5f301e04b. Or you want to switch "Composition Quest" keys to "Composition"/etc., that would require editing i18n files for all languages as well as index.es.

I'll go with the latter because we could remove the redundant quest from the origin, which is more definitive

A project scope "find and replace" could make this in either VS Code or Atom. Or you could leave it to me if you use another editor

I'm using VS Code, I was just choosing between the two, the latter one is OK too.

This does not seems to be a plugin or translation issue to me. This only happen to certain themes, because those themes uppercases all button texts, which makes the text significantly wider and exceeds the edge of the button.
LumenDark with default uppercase:

LumenDark without uppercase:

Still looks like this with no upercase to me (paperdark)

@gnattu anyway the quest could be removed to leave some space, at least that's not bad

Going to reopen.


library: hide open/clear if empty

I was told that those buttons are confusing when the library is empty, and I agree.

Yeah! Makes sense. Why show it when it has no utility? +1

Yep agreed 👍


Only test constraints with the given variable.

It's probably not as bad as the old approach...

I disagree with this change. The original intent was "do not operate on memory locations such that there is an equality constraint touching a superset of the variables in the location", and this is now "do not operate on memory locations such that the variables in memory are a superset of the constraint's variables". I think at one point I had it the other way around, but it didn't work, so I thought about it really hard and decided that switching it would fix things.


Error: premake-vscode/vscode_project.lua:49: attempt to concatenate a nil value (local 'target')

~/dev/vulkan_1 >>> premake5 gmake2                         // gmake2 target works
Building configurations...
Running action 'gmake2'...
Generated Makefile...
Generated lucre.make...
Generated engine.make...
Generated vendor/glfw3.make...
Generated vendor/yaml-cpp.make...
Generated vendor/atlas/Makefile...
Generated vendor/shaderc/shaderc.make...
Generated vendor/shaderc/shaderc_util.make...
Generated vendor/shaderc/SPIRV-Tools-opt.make...
Generated vendor/shaderc/SPIRV-Tools.make...
Generated vendor/shaderc/MachineIndependent.make...
Generated vendor/shaderc/OSDependent.make...
Generated vendor/shaderc/GenericCodeGen.make...
Generated vendor/shaderc/OGLCompiler.make...
Generated vendor/shaderc/SPIRV.make...
Generated vendor/sdl_mixer.make...
Generated vendor/libvorbis.make...
Generated vendor/libogg.make...
Generated vendor/sdl.make...
Generated vendor/box2d/Makefile...
Generated vendor/pamanager/libpamanager/Makefile...
Generated resource-system-gnu.make...
Done (758ms).
~/dev/vulkan_1 >>> premake5 vscode                          // fails
Building configurations...
Running action 'vscode'...
Generated vulkanRenderEngine.code-workspace...
Generated lucre/.vscode/tasks.json...
Generated lucre/.vscode/launch.json...
Generated lucre/.vscode/c_cpp_properties.json...
Generated engine/.vscode/tasks.json...
Generated engine/.vscode/launch.json...
Generated engine/.vscode/c_cpp_properties.json...
Generated vendor/glfw3/.vscode/tasks.json...
Generated vendor/glfw3/.vscode/launch.json...
Generated vendor/glfw3/.vscode/c_cpp_properties.json...
ln: '/home/user/dev/vulkan_1/vendor/yaml-cpp/include/yaml-cpp/anchor.h' and '/home/user/dev/vulkan_1/vendor/yaml-cpp/include/yaml-cpp/anchor.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/yaml-cpp/include/yaml-cpp/binary.h' and '/home/user/dev/vulkan_1/vendor/yaml-cpp/include/yaml-cpp/binary.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/yaml-cpp/include/yaml-cpp/contrib/anchordict.h' and '/home/user/dev/vulkan_1/vendor/yaml-cpp/include/yaml-cpp/contrib/anchordict.h' are the same file
[..]
ln: '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/stream.cpp' and '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/stream.cpp' are the same file
ln: '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/stream.h' and '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/stream.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/streamcharsource.h' and '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/streamcharsource.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/stringsource.h' and '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/stringsource.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/tag.cpp' and '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/tag.cpp' are the same file
ln: '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/tag.h' and '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/tag.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/token.h' and '/home/user/dev/vulkan_1/vendor/yaml-cpp/src/token.h' are the same file
Generated vendor/yaml-cpp/.vscode/tasks.json...
Generated vendor/yaml-cpp/.vscode/launch.json...
Generated vendor/yaml-cpp/.vscode/c_cpp_properties.json...
Generated vendor/atlas/SpriteSheetGenerator/.vscode/tasks.json...
Generated vendor/atlas/SpriteSheetGenerator/.vscode/launch.json...
Generated vendor/atlas/SpriteSheetGenerator/.vscode/c_cpp_properties.json...
Generated vendor/shaderc/shaderc/.vscode/tasks.json...
Generated vendor/shaderc/shaderc/.vscode/launch.json...
Generated vendor/shaderc/shaderc/.vscode/c_cpp_properties.json...
Generated vendor/shaderc/shaderc_util/.vscode/tasks.json...
Generated vendor/shaderc/shaderc_util/.vscode/launch.json...
Generated vendor/shaderc/shaderc_util/.vscode/c_cpp_properties.json...
Generated vendor/shaderc/SPIRV-Tools-opt/.vscode/tasks.json...
Generated vendor/shaderc/SPIRV-Tools-opt/.vscode/launch.json...
Generated vendor/shaderc/SPIRV-Tools-opt/.vscode/c_cpp_properties.json...
Generated vendor/shaderc/SPIRV-Tools/.vscode/tasks.json...
Generated vendor/shaderc/SPIRV-Tools/.vscode/launch.json...
Generated vendor/shaderc/SPIRV-Tools/.vscode/c_cpp_properties.json...
Generated vendor/shaderc/MachineIndependent/.vscode/tasks.json...
Generated vendor/shaderc/MachineIndependent/.vscode/launch.json...
Generated vendor/shaderc/MachineIndependent/.vscode/c_cpp_properties.json...
Generated vendor/shaderc/OSDependent/.vscode/tasks.json...
Generated vendor/shaderc/OSDependent/.vscode/launch.json...
Generated vendor/shaderc/OSDependent/.vscode/c_cpp_properties.json...
Generated vendor/shaderc/GenericCodeGen/.vscode/tasks.json...
Generated vendor/shaderc/GenericCodeGen/.vscode/launch.json...
Generated vendor/shaderc/GenericCodeGen/.vscode/c_cpp_properties.json...
Generated vendor/shaderc/OGLCompiler/.vscode/tasks.json...
Generated vendor/shaderc/OGLCompiler/.vscode/launch.json...
Generated vendor/shaderc/OGLCompiler/.vscode/c_cpp_properties.json...
Generated vendor/shaderc/SPIRV/.vscode/tasks.json...
Generated vendor/shaderc/SPIRV/.vscode/launch.json...
Generated vendor/shaderc/SPIRV/.vscode/c_cpp_properties.json...
Generated vendor/sdl_mixer/.vscode/tasks.json...
Generated vendor/sdl_mixer/.vscode/launch.json...
Generated vendor/sdl_mixer/.vscode/c_cpp_properties.json...
Generated vendor/libvorbis/.vscode/tasks.json...
Generated vendor/libvorbis/.vscode/launch.json...
Generated vendor/libvorbis/.vscode/c_cpp_properties.json...
Generated vendor/libogg/.vscode/tasks.json...
Generated vendor/libogg/.vscode/launch.json...
Generated vendor/libogg/.vscode/c_cpp_properties.json...
ln: '/home/user/dev/vulkan_1/vendor/sdl/include/SDL.h' and '/home/user/dev/vulkan_1/vendor/sdl/include/SDL.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_assert.h' and '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_assert.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_atomic.h' and '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_atomic.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_audio.h' and '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_audio.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_bits.h' and '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_bits.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_blendmode.h' and '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_blendmode.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_clipboard.h' and '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_clipboard.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_config.h' and '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_config.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_config.h.cmake' and '/home/user/dev/vulkan_1/vendor/sdl/include/SDL_config.h.cmake' are the same file
[..]
ln: '/home/user/dev/vulkan_1/vendor/sdl/src/hidapi/SDL_hidapi.c' and '/home/user/dev/vulkan_1/vendor/sdl/src/hidapi/SDL_hidapi.c' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/SDL_gamecontroller.c' and '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/SDL_gamecontroller.c' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/SDL_gamecontrollerdb.h' and '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/SDL_gamecontrollerdb.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/SDL_joystick.c' and '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/SDL_joystick.c' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/SDL_joystick_c.h' and '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/SDL_joystick_c.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/SDL_sysjoystick.h' and '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/SDL_sysjoystick.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/controller_type.h' and '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/controller_type.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/dummy/SDL_sysjoystick.c' and '/home/user/dev/vulkan_1/vendor/sdl/src/joystick/dummy/SDL_sysjoystick.c' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/src/video/yuv2rgb/yuv_rgb_sse_func.h' and '/home/user/dev/vulkan_1/vendor/sdl/src/video/yuv2rgb/yuv_rgb_sse_func.h' are the same file
ln: '/home/user/dev/vulkan_1/vendor/sdl/src/video/yuv2rgb/yuv_rgb_std_func.h' and '/home/user/dev/vulkan_1/vendor/sdl/src/video/yuv2rgb/yuv_rgb_std_func.h' are the same file
Generated vendor/sdl/.vscode/tasks.json...
Generated vendor/sdl/.vscode/launch.json...
Generated vendor/sdl/.vscode/c_cpp_properties.json...
Generated vendor/box2d/box2d/.vscode/tasks.json...
Generated vendor/box2d/box2d/.vscode/launch.json...
Generated vendor/box2d/box2d/.vscode/c_cpp_properties.json...
Generated vendor/pamanager/libpamanager/libpamanager/.vscode/tasks.json...
Generated vendor/pamanager/libpamanager/libpamanager/.vscode/launch.json...
Generated vendor/pamanager/libpamanager/libpamanager/.vscode/c_cpp_properties.json...
Error: .../dev/vulkan_1/vendor/premake-vscode/vscode_project.lua:49: attempt to concatenate a nil value (local 'target')

fixed in fork https://github.com/kamtar/premake-vscode


receipt exchange rate

Description:

Add various Sendable conformances (They should exist, just, easy to miss them)
Add exchangeRates to TransactionReceipt.

Note that unlike JS, Java, and Go, (but like CPP), this is the current and next. However, an equivalent exchangeRate property exists anyway, so, in practice this isn't really a divergence.

Related issue(s):

Fixes #259

Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Hi @izik1,
Thanks for the quick help. I observe that exchangeRate in TransactionReceipt is nullable but in Java SDK it is not nullable. Any specific reason?


Update README example to be compatible with both 2.7 & 3.x

Since #374 has landed, Python 3 is supported. This PR wraps all print calls with ( ) so that is it compatible with both Python 2.7 & Python 3. This should be one less thing for new users that are using Python 3 to trip on.
Note this is different than __future__.print_function which makes Python 2.7 actually use the real print function. That isn't needed here for these simple examples where wrapping in ( ) simply works in both in the simple case you are printing a single string.

/morph test


[vite:build-html] EISDIR: illegal operation on a directory, read

Describe the bug
Build process fails if html contains an image tag with a blank src attribute:
<img src="" />

Results in the following npm console error:
file: {PROJECT_ROOT_DIRECTORY}/index.html
error during build:
Error: EISDIR: illegal operation on a directory, read
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `vite build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

Full npm debug log:
0 info it worked if it ends with ok
1 verbose cli [
1 verbose cli   'C:\\Program Files\\nodejs\\node.exe',
1 verbose cli   'C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js',
1 verbose cli   'run',
1 verbose cli   'build'
1 verbose cli ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prebuild', 'build', 'postbuild' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
8 verbose lifecycle<EMAIL_ADDRESS>PATH: C:\Program Files\nodejs\node_modules\npm\node_modules\npm-lifecycle\node-gyp-bin;C:\Users\{SOME_USER}\Documents\My Projects\Portfolio\node_modules\.bin;C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Windows\System32\OpenSSH\;C:\Program Files\nodejs\;C:\Program Files\Git\cmd;C:\Users\{SOME_USER}\AppData\Local\Microsoft\WindowsApps;;C:\Users\{SOME_USER}\AppData\Local\Programs\Microsoft VS Code\bin;C:\Users\{SOME_USER}\AppData\Roaming\npm;C:\Program Files\heroku\bin
9 verbose lifecycle<EMAIL_ADDRESS>CWD: C:\Users\{SOME_USER}\Documents\My Projects\Portfolio
10 silly lifecycle<EMAIL_ADDRESS>Args: [ '/d /s /c', 'vite build' ]
11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
12 info lifecycle<EMAIL_ADDRESS>Failed to exec build script
13 verbose stack Error<EMAIL_ADDRESS>build: `vite build`
13 verbose stack Exit status 1
13 verbose stack     at EventEmitter.<anonymous> (C:\Program Files\nodejs\node_modules\npm\node_modules\npm-lifecycle\index.js:332:16)
13 verbose stack     at EventEmitter.emit (events.js:315:20)
13 verbose stack     at ChildProcess.<anonymous> (C:\Program Files\nodejs\node_modules\npm\node_modules\npm-lifecycle\lib\spawn.js:55:14)
13 verbose stack     at ChildProcess.emit (events.js:315:20)
13 verbose stack     at maybeClose (internal/child_process.js:1048:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:288:5)
14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd C:\Users\{SOME_USER}\Documents\My Projects\Portfolio
16 verbose Windows_NT 10.0.19042
17 verbose argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "build"
18 verbose node v14.15.3
19 verbose npm  v6.14.9
20 error code ELIFECYCLE
21 error errno 1
22 error<EMAIL_ADDRESS>build: `vite build`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>build script.
23 error This is probably not a problem with npm. There is likely additional logging output above.
24 verbose exit [ 1, true ]

Reproduction
See bug description.
System Info
System:
    OS: Windows 10 10.0.19042
    CPU: (8) x64 Intel(R) Core(TM) i7-7700K CPU @ 4.20GHz
    Memory: 10.07 GB / 15.96 GB
  Binaries:
    Node: 14.15.3 - C:\Program Files\nodejs\node.EXE
    npm: 6.14.9 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1023.0), Chromium (91.0.864.59)
    Internet Explorer: 11.0.19041.1
  npmPackages:
    vite: ^2.3.8 => 2.3.8

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

I ran into this same error, except slightly different cause: <link rel="icon" href="" /> in my index.html file.

An error is expected. But the message can be improved.

btw this also happens with empty src attribute in <img> and <video>

These appear to be the lines that cause this error to throw. I'd love for this to either not throw, or to give a better error message (file/line/row/column causing the issue) because the error that's being throw, while technically correct, isn't useful

I also ran into this issue with links referencing MPA output (href="/my/mpa/output"). The PR linked above should resolve both the referenced named output issue that causes this error, and the empty attribute issue that causes this error.


arg undef added to school domains

I just follow jetbrains instructions to add a new school
Here and here to check existence of the university

@Chavis00 First stage of verification passed, official domain for Facultad de Ingenieria del Ejercito is fie.undef.edu.ar as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@Chavis00 Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Add Angular 7 support

@chymz Angular 7 was released. Please update dependencies in package.json for the ng2-ckeditor module.

@chymz : hi Simon! May I we ask you t update the references please?


Final round of doc updates

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind api-change
/kind bug
/kind cleanup
/kind deprecation
/kind design
/kind failing-test
/kind feature
/kind flake
/kind code-refactoring

What does this PR do / why we need it:
Final round of doc updates
Have you updated the necessary documentation?

[x] Documentation update is required by this PR.
[x] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
How to test changes / Special notes to the reviewer:

/assign @gaganhegde
/assign @chetan-rns

/test v4.5-integration-e2e

/assign @bigkevmcd

/LGTM


feat(enricher): Adds the rh-metering flag for adding Red Hat Runtimes metering to your application.

The new flag is --rh-metering.  can also be used as --rh-metering.component-version=NODE_VERSION

This is currently an undocumented feature since it is more for those using Red Hat Runtimes build of Node.js

Coverage decreased (-0.3%) to 98.299% when pulling 30d26e79c049dd1aa4e54f0824f3d4fc70972c5f on lholmquist:NODE-880-meterting-metadata into 089809e0fe7c8b95be1c6334175dd87b937075b6 on nodeshift:master.


Fix graphics

Two things in this set of changes:

Fixed a long standing bug that messed up scaling of plots in the "wtvaimg" mode.    Fix may not be perfect, but is an improvement over present form that produces really bad plots in that mode most of the time.   The possible residual issue is that we may need to handle the "aspect" parameter for image plotting more elegantly.   I'm not sure with these changes a user can actually change the aspect ratio manually.   I don't expect that to be a very common need though so I didn't think the effort required to test all the possibilities was worth the trouble.
I changed the default style to "wtva" (wiggle trace variable area) instead of "wtvaimg" (the style that previously was giving trouble).   Maybe should be the simplest "wt" but in most cases wtva plots are a better data visualization that wt - an option though.

@wangyinz I urge you to merge this branch with master.   It fixes a long standing bug in graphics with the "wtvaimg" style of plot.   The only check that is flagging anything is coverage.  It is complaining the new sections aren't being tested.   I have no idea how to create a pytest function to test this kind of functionality.   The best way to test the graphics module is to run the graphics tutorial and verify it works correctly.   Anyway, I would urge you to review what I did and merge this branch with master.


Fix all the "# noqa: F401" lint skips the right way

I added # noqa: F401 to a bunch of lines to get the code passing the linter.
We should fix all these lines the right way.

Linting exceptions have been correctly applied.


Fix static build by using integer-gmp variant of the compiler

Purpose
cborg is not compatible with integer-simple and 8.10.4.
Changes
The cborg dependency did not have the flag to disable integer-gmp dependency. concordium-client now depends on our own fork that adds this flag.
Checklist

[ ] My code follows the style of this project.
[ ] The code compiles without warnings.
[ ] I have performed a self-review of the changes.
[ ] I have documented my code, in particular the intent of the
hard-to-understand areas.
[ ] (If necessary) I have updated the CHANGELOG.

I have verified that all of these work now.


Zotero-Web-Clipper (might also impcat other citation software)

Description
the weblipper is no longer able to recognize the right doi/ title authors. this makes the htmml version less useful
(Optional:) Please add any files, screenshots, or other information here.
No response
(Required) What is this issue most closely related to? Select one.
Choose One
Internal issue ID
ca88244f-e73a-4a9a-b96c-09996cffad5e
Paper URL
https://arxiv.org/html/2410.03131v1
Browser
Firefox/132.0
Device Type
Desktop

@MarioMuehlematter could you report this to Zotero directly? We would like to know if there are issues in the HTML that cause problems, but this report is not specific enough yet.

Reported it to the zotero forum.


A Space needed in notification email

I've noticed that there needs to be a space between: Yeah! Your print job and the gcode filename. It makes it sometimes difficult for me to read.

Thanks

Thank you @phcoetzee for reporting this issue. Just fixed it.


Documentation doesn't explain record states

I can see that records have these four properties:

isPending
isRejected
isRequested
isResolved

I gather that when isResolved = true the record.content property is fulfilled and I can render my template. I'm guessing that isRejected is true if the load failed. What is the difference between isPending and isRequested?
Probably all four should be explained in the documentation with some sort of sample template logic. Example:
{{#if record.isResolved}}
    {{#with record.content as |model|}}

{{model.name}}

    {{/with}}
{{else if record.isRejected}}

Error

{{else}}

Loading...

{{/if}}

https://github.com/flexyford/impagination/blob/master/src/page.js#L10
isPending - when we wait on the fetch to return
isResolved - when the fetch is resolved
isRejected - when the fetch is rejected
isRequested - record isPending OR isResolved OR isRejected
isRequested is false whenever we initialize our dataset with unrequested pages.
  // fetch a page of records at the pageOffset
    return this.store.query('record', params).then((data) => {
      stats.totalPages = data.get('totalPages'); // Initialize the dataset with Unrequested Pages of Unrequested Records all the way up to stats.totalPages
      return data.get('records');
    });

This is a record attribute we inherit from impagination and should be explained in the README. Leaving this issue open for now as a result.


When will Windows support be available?

Hello,
When will Windows support be available?
Thank you very much for your great widget. 💯
Regards,
Jose

any update on this?


Add ability for rust uniwig to create output files from input .bam files

Some work accomplished with PR #40.
We would like to use this code as a drop in replacement for bamSitesToWig.py from PEPATAC: https://github.com/databio/pepatac/blob/master/tools/bamSitesToWig.py
bamSitesToWig.py creates three files as output:

[ ] exact.bw

[ ] smoothed.bw

[ ] shift.bed

Currently, uniwig can take an input file of either:
bed, narrowPeak, bam
and create an output of:
wig
npy
bedGraph
bw (via an intermediate bedGraph conversion)
Some items to accomplish for this task:

[ ] add bed output support if given a non-bed-like file as input (i.e. bam).
[ ] Original bamSitesToWig adjusts positions based on flags. Noodles does support Flags but on first glance they are named differently and must be reviewed.
[ ] I believe PEPATAC pipeline only needs starts counts? So add ability to only output any counts the user wishes instead of always outputting all three (starts,ends, core).
[ ] Scaling is used in bamSitesToWig.py. Currently, uniwig does not scale counts. Should it?
[ ] Smooth vs Exact, currently uniwig only offers the smooth option, though the user can set smoothing =0. Is this the same output as the exact code? If so, why were two perl scripts used? For original counting algorithms, see:
https://github.com/databio/pepatac/blob/master/tools/cutsToWig.pl
https://github.com/databio/pepatac/blob/master/tools/smoothWig.pl

Nice to have:

[ ] if associated .bai does not exist, have uniwig create it (using noodles crate)

Working proof of concept in #40 , however, it uses an intermediate BedGraph file written to disk.
Therefore, we are exploring an alternative method in #47 which streams values directly to bigtools bw writer.
However, some challenges remain, namely:

[ ] when spawning separate threads to handle consumer/producer workflows, I see issues with error handling where an error does not seem to close the producer, causing the consumer to "hang"


DependsOn, condition and variables with stage template

Are there more advanced examples on this?
Like setting dependsOn, condition and variables in the stage template?

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

ID: 49ed15b8-f956-4839-5caa-611f69f207b7
Version Independent ID: 49ed15b8-f956-4839-5caa-611f69f207b7
Content: stages.template definition
Content Source: content/stages-template.md
Product: devops
Technology: devops-cicd
GitHub Login: @steved0x
Microsoft Alias: sdanie

Thank you for your feedback. We are migrating this issue to our new documentation feedback system, and closing this issue.
As we phase out GitHub issues for this repository, we encourage you to propose changes using public PRs or submit feedback through our updated control at the top of the page. For more information on our feedback systems, see https://aka.ms/contentuserfeedback.


Make array_reader and schema modules private

These have been experimental for a while as a stepping stone to eventually making them private, let's do that now

Oh I forgot the benchmarks make use of this :cry:


Unable to build on windows with msvc

warning: couldn't execute llvm-config --prefix (error: program not found)
warning: set the LLVM_CONFIG_PATH environment variable to the full path to a valid llvm-config executable (including the executable itself)
error: failed to run custom build command for silk-rs v0.1.0 (https://github.com/lz1998/silk-rs?branch=main#3951ee61)
Caused by:
process didn't exit successfully: F:\pyproject\rsilk\target\debug\build\silk-rs-19638ae05635062f\build-script-build (exit code: 101)
--- stderr
thread 'main' panicked at 'Unable to find libclang: "couldn't find any valid shared libraries matching: ['clang.dll', 'libclang.dll'], set the LIBCLANG_PATH environment variable to a path where one of these files can be found (i
nvalid: [])"', C:\Users\LAB.cargo\registry\src\github.com-1ecc6299db9ec823\bindgen-0.53.3\src/lib.rs:1956:31
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
error: cargo failed with code: 101

Install llvm before cargo build.
Download the llvm release name same as LLVM-${version}-win64.exe from https://github.com/llvm/llvm-project/releases/

Thanks. Have fixed it using github action.


parser: support group_concat as window function in parser

What problem does this PR solve?
group_concat as a window function cannot be resolved in the parser. The detail you can see in TIDB #10996
What is changed and how it works?
Check List
Tests

Unit test
Integration test

Is there anything wrong with the license checker? @zz-jason

Is there anything wrong with the license checker? @zz-jason

I think so.

/run-cherry-picker


Added hooks for additional image formats (including GIFs)

I’ve added some de/serialisation blocks to the HNKCacheFormat to allow overwriting of the process for none-native formats (GIF, Webm, etc). I’ve also added an image processing hook to the HNKNetworkFetcher for the same purpose.
In addition I’ve implemented an example using GIFs from Giphy to showcase the changes (available by changing the HNK_USE_GIFS define to 1). Gif serialisation was achieved with snippets from Mattt Thompson's (http://github.com/mattt) AnimatedGIFImageSerialization (https://github.com/mattt/AnimatedGIFImageSerialization)

Is this still active? GIF support in Haneke would be great.

Hello, any update on this?


Updated default commits to latest with ResFinder and PlasmidFinder

Updated the default database versions to the latest commits. For ResFinder it's up until 2019-10-01 and the PlasmidFinder is updated to 2019-10-18. Had to change one of the test cases to accurately reflect the new changes.
Another thing that was updated was the ARG_drug_key files since this is how staramr identifies the name of the drug. I automated this update process by creating a script that converts either the resfinder or the plasmidfinder into tsv files given an excel sheet.
In order to run this program, set the command python update_staramr_db.py <file> <resfinder,pointfinder>. The script requires pandas, sys and xlrd>=1.0.0 as dependencies, I suggest creating a new conda environment for this script

It looks like this PR is old enough that it's still assuming TravisCI will be run (it won't be, only Github actions). I am closing this PR and will re-open as a new PR which should hopefully fix the issue.

It looks like this PR is old enough that it's still assuming TravisCI will be run (it won't be, only Github actions). I am closing this PR and will re-open as a new PR which should hopefully fix the issue.


KeyError: 'regions'

hi every body
i replace train and val folder with my train  and val folder then i ran the inspect_custom_data.ipynb but i faced this error
KeyError                                  Traceback (most recent call last)
 in ()
3 # https://github.com/matterport/Mask_RCNN/releases
4 dataset = custom.CustomDataset()
----> 5 dataset.load_custom(CUSTOM_DIR, "train")
6
7 # Must call before using the dataset
~/Deep-Learning/mask_rcnn_damage_detection/custom.py in load_custom(self, dataset_dir, subset)
118         # The VIA tool saves images in the JSON even if they don't have any
119         # annotations. Skip unannotated images.
--> 120         annotations = [a for a in annotations if a['regions']]
121
122         # Add images
~/Deep-Learning/mask_rcnn_damage_detection/custom.py in (.0)
118         # The VIA tool saves images in the JSON even if they don't have any
119         # annotations. Skip unannotated images.
--> 120         annotations = [a for a in annotations if a['regions']]
121
122         # Add images
KeyError: 'regions'
can every body help me

Show us the contents of your Json file

{"_via_settings":{"ui":{"annotation_editor_height":25,"annotation_editor_fontsize":0.8,"leftsidebar_width":18,"image_grid":{"img_height":80,"rshape_fill":"none","rshape_fill_opacity":0.3,"rshape_stroke":"yellow","rshape_stroke_width":2,"show_region_shape":true,"show_image_policy":"all"},"image":{"region_label":"via_region_id","region_color":"via_default_region_color","region_label_font":"10px Sans","on_image_annotation_editor_placement":"NEAR_REGION"}},"core":{"buffer_size":18,"filepath":{},"default_filepath":""},"project":{"name":"ok"}},"_via_img_metadata":{"1.jpg216044":{"filename":"1.jpg","size":216044,"regions":[{"shape_attributes":{"name":"polygon","all_points_x":[345,463,527,544,509,427,328,220,170,163,305],"all_points_y":[135,138,179,261,348,418,436,388,293,216,150]},"region_attributes":{"name":"not_defined","type":"unknown","image_quality":{"good":true,"frontal":true,"good_illumination":true}}}],"file_attributes":{"caption":"","public_domain":"no","image_url":""}},"img.jpg162851":{"filename":"img.jpg","size":162851,"regions":[{"shape_attributes":{"name":"circle","cx":406,"cy":311,"r":202},"region_attributes":{"name":"not_defined","type":"unknown","image_quality":{"good":true,"frontal":true,"good_illumination":true}}}],"file_attributes":{"caption":"","public_domain":"no","image_url":""}},"Img_1_1_2.jpg162884":{"filename":"Img_1_1_2.jpg","size":162884,"regions":[{"shape_attributes":{"name":"circle","cx":407,"cy":319,"r":189},"region_attributes":{"name":"not_defined","type":"unknown","image_quality":{"good":true,"frontal":true,"good_illumination":true}}}],"file_attributes":{"caption":"","public_domain":"no","image_url":""}}},"_via_attributes":{"region":{"name":{"type":"text","description":"Name of the object","default_value":"not_defined"},"type":{"type":"dropdown","description":"Category of object","options":{"bird":"Bird","human":"Human","cup":"Cup (object)","unknown":"Unknown (object)"},"default_options":{"unknown":true}},"image_quality":{"type":"checkbox","description":"Quality of image region","options":{"blur":"Blurred region","good_illumination":"Good Illumination","frontal":"Object in Frontal View"},"default_options":{"good":true,"frontal":true,"good_illumination":true}}},"file":{"caption":{"type":"text","description":"","default_value":""},"public_domain":{"type":"radio","description":"","options":{"yes":"Yes","no":"No"},"default_options":{"no":true}},"image_url":{"type":"text","description":"","default_value":""}}}}

Refer to this: https://github.com/matterport/Mask_RCNN/issues/1045


Pulls: 'View file' broken links

Moved to here from https://support.github.com/ticket/1094756

'A REAL URL': https://github.com/elementary/wallpapers/pull/138/files

Look at the link, blob//backgrounds.

It will be fixed in the next release. For now disable enable-file-links-in-compare-view

21.4.6 coming right up! Given the bugs it's worth releasing early

https://github.com/sindresorhus/refined-github/releases/tag/21.4.6 Just wait for it


valid micromamba installation not found by openff utils

Currently, micromamba can be installed and run correctly even if its installation location is not in the user's $PATH: https://github.com/mamba-org/mamba/issues/2577
This leads to a discrepancy where the micromamba installation is completely valid, but then openff fails with a openff.utilities.exceptions.CondaExecutableNotFoundError.
I would expect that if micromamba (or conda, mamba, etc.) runs successfully, that openff should be able to detect it as well. Can https://github.com/openforcefield/openff-utilities/blob/main/openff/utilities/provenance.py be updated to be more robust to this, and less dependent on searching for specific installation locations?

This is a really interesting proposal. I'm trying to think of the case where a user is in a properly initialized micromamba env, but doesn't have micromamba in their path, are there other environment variables we can look at to determine where the micromamba executable is?

I am not sure yet how to do it, but I do think making find_executable aware of shell functions would resolve this issue.

Also, should this trigger an exception? Perhaps a warning about being unable to check package versions would be better.

This has become too much work compared to the benefit provided, I'm just going to have it warn and return an empty dictionary when something goes wrong.

Thanks @mattwthompson!

If you do conda install conda within your environment, you can pass the find_executable function

Yes, installing conda or mamba into an environment will fix this (for future people running into this error).

Look for this in the next release, probably version 0.1.13 and sometime on the week of December 2nd
(I don't make releases on Fridays and next week is a holiday week in which many of us are half-off.)


new source scaling: TMG2017

Request feedback on the coding style, and also how/what to write the Copyright Statement.

Something is not so right with this pull request.


add imagesignature role

To support pulling images which are signed from the internal image registry, imagesignature role needs to be granted to kabanero-pipeline-role.

fixes #734


Release v0.1

Tracking what it will take to do an initial release for v0.1. We should do a simultaneous release of pywhy-graphs v0.1, so that way.
Some left-over PRs I want to get merged:

#83 for CMI kNN test: done
#85 for Classifier CMI test: still WIP
#86 for an initial look at basic time-series causal discovery
#72 @robertness is in charge of this, so if necessary, we can slate this for v0.2. However, seems like if it's wrapping causal-learn, just needs someone to take it over the finish line.
We should update the dependency for pywhy-graphs to v0.1 once we release there
We should probably create some more documentation talking about how to use Context and the constraint-based causal discovery

Other possible improvements, we can stack in are #29, but this really also requires someone to champion.

I will slate a release once:

#146
and the #138 (S-FCI) algorithm are implemented in dodiscover

I kind of want to remove the Context object because it is pretty difficult to use both from a developer and user perspective. I think it is in the right direction, but there's a lot tbd about the API


ActiveRecord unlink() could be with the same feature as link ($extraColumns)

Hi,
Is there a reason, why unlink in 'BaseActiveRecord' class is without $extraColumns option?
Right now it is possible to save extra info with link, but during unlink, it deletes all records disregarding info in ExtraColumns.
What steps will reproduce the problem?
I have many2many relation table with columns: id (PK) | meeting_id | customer_id | customer_role
A customer may have different roles: initiator, participant, lector etc. (1, 2, 3 etc)
So, during linking, I can fill table with proper values eg:
1st person:
... | 307 | 45 | 1 - initiates meeting
... | 307 | 45 | 2 - participated on meeting as quest
... | 307 | 45 | 3 - participated on meeting as lector
2ndt person:
... | 307 | 46 | 2 - participated on meeting as quest
... | 307 | 46 | 3 - participated on meeting as lector
etc
But if I would like to delete customer nr 45 from lector list using unlink of  'BaseActiveRecord' class, it deletes all records, where meeting_id = 307 and customer_id = 45.
Additional info
I checked the doc of 'link' and 'unlink'. 'Unlink' is missing part of $extraColumns, what 'link' has.
I added $extraColumns to unlink:
public function unlink($name, $model, $delete = false, $extraColumns = [])
{ ...

and in the code added:
foreach ($extraColumns as $k => $v) {
                $columns[$k] = $v;
            }

now it works as I would like it to work.

Q
A

Yii version
2.0.12

Makes sense but since there's a signature change, it could be adjusted in Yii 2.1 only.

Is there a reason, why unlink in 'BaseActiveRecord' class is without $extraColumns option?

extra columns are used to store extra information for a relation. A relation between two records should have only one entry in the table, not multiple entries with different extra column data. The reference to related tables should be a primary key.

@cebe well, under normal conditions - yes. But in my case scenario, one person can be connected to event as different type (lector, participant, secretary, etc). So in Many2Many relation table (using via method), I relate Event with Person via Participant table (M2M relation table). Participant Model is based on Samdark Cookbook ideas: "Single table inheritance", so there is different types of participants. And the only way to understand the type of participation is by using additional column 'type'.
Therefore it is very useful Extracolumn during link method (as I link event with person and marking participation type), but I am not able to delete only certain type of participation without additional query for finding id in related table (where I still have to use Event+Person+ParticipationType). With existing unlink() all participation's of person/event will be deleted, disregarding type of participation.
Maybe it is only matter of my case scenario. But anyway, thanks for nice framework.

Yes, that's very specific scenario indeed...

BTW, another places we are using similar structure:

sending emails from our CMS

attaching files to emails.

A mail receiver can be in list of TO / CC / BCC. So when (after saving draft) I would like to delete somebody from BCC (for example), he is also gone from TO and/or CC.
We found, that using one table with 'relation type' is better choice, than using many tables with similar structure or one table with another table holding that 'type' information. Searching and/or adding new types is much easier within one table.

We have to send with email public and private files. We found, there is no reason for duplicating methods in 'AttachmentsPublic' model and 'AttachmentsPrivate' model. Rather than using the same logic (Samdark STI) and having only one model/table/ParentClass with child classes.

Yes, use case is understood and isn't very common. I think the best way to solve it is to issue two delete() statements in transaction.

you can of course create such relations but they are out of the scope that is supported by link() and unlink(). If you have that scenario, you have to write your own code for creating and removing relations.

OK, thank you!


style: remove row-gap overwritting style in MuiAccordion

What it solves
Resolves #3509
How this PR fixes it
Removes style that was suppressing row-gap in the last MUI accordion for no apparent reason
How to test it
Open a transaction deeplink view and the layout shall not be cramped anymore.
Screenshots

The removed style doesn't seem to be have any practical effect other than not cramping the last accordion content which I believe it's a leftover style. I've tested how it affects the layout and doesn't seem to have any negative impact.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3228

Relevant Lines:
8613

💛  - Coveralls

Looks good now. thanks


Angular ng-repeat animation transition not working with ie10

Any expert to help to answering this question:
http://stackoverflow.com/questions/31659373/angular-ng-repeat-animation-transition-not-working-with-ie10

Sorry, usage questions are not in the scope of the github issues, see https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#-got-a-question-or-problem


error when running the startup demo

create a demo: deno run -A -r https://fresh.deno.dev
running: deno task start
errors when opening or refreshing the page:
 🍋 Fresh ready
    Local: http://localhost:8000/

An error occurred during route handling or page rendering.
An error occurred during route handling or page rendering.
An error occurred during route handling or page rendering.
An error occurred during route handling or page rendering.
Error: Invalid gzip data in archive: InvalidInput: invalid gzip header
    at extractFileFromTarGzip (https://deno.land/x/esbuild@v0.20.2/mod.js:1838:11)
    at installFromNPM (https://deno.land/x/esbuild@v0.20.2/mod.js:1790:22)
    at eventLoopTick (ext:core/01_core.js:175:7)
    at async install (https://deno.land/x/esbuild@v0.20.2/mod.js:1877:12)
    at async https://deno.land/x/esbuild@v0.20.2/mod.js:1948:23
    at async Module.initialize (https://deno.land/x/esbuild@v0.20.2/mod.js:1777:3)
    at async initializeEsbuild (https://deno.land/x/fresh@1.7.2/src/build/esbuild.ts:45:5)
    at async EsbuildBuilder.build (https://deno.land/x/fresh@1.7.2/src/build/esbuild.ts:61:21)
Error: Invalid gzip data in archive: InvalidInput: invalid gzip header
    at extractFileFromTarGzip (https://deno.land/x/esbuild@v0.20.2/mod.js:1838:11)
...
[several same messages]

deno version:
deno 1.46.2 (stable, release, aarch64-apple-darwin)
v8 <IP_ADDRESS>-rusty
typescript 5.5.2

maybe the root cause is npm registry


[BUG] Installation from mirror fails

Version
v1
Describe the bug
Installation from mirror fails
I noticed a failure to download helm in my env:  Error: Error: Did not find file "helm-linux-amd64.tar.gz" in the given hash file
I could also see that example_mirror.yml fails during executions here as well.
Steps to reproduce, workflow links, screenshots
I'm using an Ubuntu runner (22.04) on GitHub Enterprise.
runs:
  using: composite
  steps:
    - name: Install OpenShift Client
      uses: actions/redhat-actions_openshift-tools-installer@v1
      with:
        oc: "4"
        helm: "3"

+1 Same issue

Its working again.


Implemented nestedContentSubmitWatcher directive to get around the pr…

…oblem of child controls that use the formSubmitted event to save their values back to the scope.
This relates to issue #20 - Saving Macro Container properties to Nested Content doesn't always work and is based on work in Archetype by @kjac.
This implementation is simplified and doesn't require creating another broadcast event as the directive binds to the callbacks on the $parent instead of the scoped node.  As such it may be a little more optimised in terms of performance as we're only relying on the formSubmitted event and not broadcasting another one.

Merged

I've merged this in, but I get the feeling I might need to test this with #40 having been merged in too cos I get the feeling it might fail with the editor being removed by ng-if (where as ng-show would still be present in the dom)

I've implemented this in a slightly different way now, but formSubmitting now gets called on the inner editors so the problem this fixes should still be resolved


parameters.py needs to be rewritten as a class.

I found that param.reduction_structural_embeddings_file cannot be called when running train.py, then I found that there is no class defined in parameters.py. I wrote the class myself and it works fine! Just to let you this :)

Hi, @blueorris, Thank you for the reminder~
I am also reproducing this paper, could you please send me a copy of your revised code?
If possible, thank you very much!


Fishing boats ~upvote=0|downvote=0

Boats that specialize in the catching of fish. In PW/PK fishing mostly in shallow water. If possible the ability to catch fish in open waters. If fishing can be done in shallow water also, perhaps add a buff to the fishing on open sea. Fishing on open sea would give either considerable more fish, or a different breed of fish that is worth more in order to encourage it. Fish boat has a storage to store said fish. Reason: Make fishing much more engaging

🏰 Discord Suggestion from Younes@215866926033928193

user BridgeTroll@7127 voted +


[Very important] metagpt need to send some question and related choices to human, to get feedback from human

It needs to do:

During its developing
(1) Is this part a key point during total developing circle? If so, (or human decide to) ,  trigger (2)...
(2) What kind of question and choices should be sent to human?
(3) Then send question and choices to human and receive feedback
(4) How to change its following work, after receiving human choices.

At the begging of work
And at the begging of incremental work(or normal work),
It also needs to show its understanding of current user's code repo and user's requirement.....and to let human to answer
"correct understanding" or "incorrect understanding"
additionally,
sending key question and choices it designed , to clarify its understanding.

Product rules:
To choose "A B C D", or to answer "correct or incorrect" , is easy for human.
To write complex words and code, is hard for human.

Yes. These are available in our 1.0 version. However, this problem is actually not as simple as we think. For example, "Buy me an apple" is simple for humans, but it is very complex for machines.
Because we don’t know whether we are looking for mobile phones or fruits, we don’t know the required brand, quality, price, quantity, distance, time, transaction person, transaction channel, transaction method, etc. "Buy three apples that ripen on a full moon night on the Tianshan Mountains, buy them within a month, trade with Bitcoin, pay later" also meets the requirements of "buying apples".
Although we would intuitively say that it has some optimization points, it is very difficult to completely solve this problem.

Yes. These are available in our 1.0 version. However, this problem is actually not as simple as we think. For example, "Buy me an apple" is simple for humans, but it is very complex for machines.
Because we don’t know whether we are looking for mobile phones or fruits, we don’t know the required brand, quality, price, quantity, distance, time, transaction person, transaction channel, transaction method, etc. "Buy three apples that ripen on a full moon night on the Tianshan Mountains, buy them within a month, trade with Bitcoin, pay later" also meets the requirements of "buying apples".
Although we would intuitively say that it has some optimization points, it is very difficult to completely solve this problem.

At the beginning, building accurate user 's portrait and  task's doc from by llm itself , is pretty important...
It can narrow down the following project 's question range.

It needs to do:

During its developing
(1) Is this part a key point during total developing circle? If so, (or human decide to) ,  trigger (2)...
(2) What kind of question and choices should be sent to human?
(3) Then send question and choices to human and receive feedback
(4) How to change its following work, after receiving human choices.
At the begging of work
And at the begging of incremental work(or normal work),
It also needs to show its understanding of current user's code repo and user's requirement.....and to let human to answer
"correct understanding" or "incorrect understanding"
additionally,
sending key question and choices it designed , to clarify its understanding.

Product rules: To choose "A B C D", or to answer "correct or incorrect" , is easy for human. To write complex words and code, is hard for human.

myabe somthing like autogen? now the agent message subscription mechanism is not enough though

It needs to do:

During its developing
(1) Is this part a key point during total developing circle? If so, (or human decide to) ,  trigger (2)...
(2) What kind of question and choices should be sent to human?
(3) Then send question and choices to human and receive feedback
(4) How to change its following work, after receiving human choices.
At the begging of work
And at the begging of incremental work(or normal work),
It also needs to show its understanding of current user's code repo and user's requirement.....and to let human to answer
"correct understanding" or "incorrect understanding"
additionally,
sending key question and choices it designed , to clarify its understanding.

Product rules: To choose "A B C D", or to answer "correct or incorrect" , is easy for human. To write complex words and code, is hard for human.

myabe somthing like autogen? now the agent message subscription mechanism is not enough though

you men , metagpt 's agent message subscription mechanism is not enough,
or you mean this general mechanism in all alternative, is not good enough

You mean metagpt's agent message subscription mechanism not good enough,
or you mean general mechanism and theory,  not good enough in all alternatives projects

My design intent is to be a pure pubsub, and can be replaced by a queue. I understand pubsub solves all problems.


Added functionality to remove attachment from a document by attachment name.

Thanks for your hard work, please ensure all items are complete before opening.

[x] You have signed the CLA as per the instructions in CONTRIBUTING.md
[X] You have added tests for any code changes
[X] You have updated the CHANGES.md
[X] You have completed the PR template below:

What
Added functionality to cloudant-client to remove attachment from a document by attachment name.
How
Needed ability to remove an attachment from a document by attachment name.
Modeled new functionality after the remove() methods that remove a document from the database. Leveraged functionality already available in the REST API to sent a delete request with attachment name and _rev as query parameter (info found at https://docs.cloudant.com/attachments.html#delete).
Testing
Added test to com.cloudant.tests.AttachmentTest that creates a document, adds an attachment, verifies attachment has been added, and then calls the new removeAttachment method. Asserts the bar.getAttachments() is null after removal.
Issues
N/A

Closing and creating new pull request...Travis build timed out on jdk 1.8


Views counter

We can think of increment views on nfts using functions and using ThreadDB for that? Let's discuss it later @vikiival

Hey, time travel. Is this what we're doing? :D

links to #4593

https://backyardcoder.me/posts/implementing-a-page-view-counter/

be carefull with time travel

#4862

Yes continue to

#4593


Guard against race conditions

While #79 described a situation that could happen commonly, there are some less likely but still possible race conditions that we might consider guarding against.
Concurrent downscale + rollout
If a downscale is in progress but hasn't updated the last-downscale annotation yet, it's possible that a rollout could start concurrently, bypassing the min-time-between-zones-downscale check in the controller.
Concurrent downscale + downscale
It downscales for two zones occur concurrently, they can both be accepted before either has updated the last-downscale annotation, bypassing the min-time-between-zones-downscale check in the prepare-downscale webhook.
Potential fix
One reliable way to serialize operations across different statefulsets is to lock on a common resource, such as with an annotation on a common CRD, using a resourceVersion on updates. Absent that, we could add a lock annotation to a statefulset and use a double check when adding it to guard against races, though I'm not positive this is completely safe.

Concurrent downscale + downscale
It downscales for two zones occur concurrently, they can both be accepted before either has updated the last-downscale annotation, bypassing the min-time-between-zones-downscale check in the prepare-downscale webhook.

This situation isn't possible. Zone A is controlled by HPA and zones B and C are controlled by the rollout-operator. The rollout operator only adjusts replicas in zones B and C down if the number of replicas in zone A has already been adjusted down.

I had Loki in mind for that scenario, where they use an HPA for each zone.


Feat: add has class

Description
Add has class
Motivation & Context
How is this being tested ?
Screenshots (if appropriate):

This is a great PR, not extra files and no merge conflicts. Great Job! At this point though I want to be really careful on what gets added to the project because the main call is for it to be small. The initial goal was to keep it around 500 bytes so that this library can be used in conjunction with bigger libraries. I am not opposed to adding new features but I think they should be discussed before and at minimum check this boxes:
[  ] It it something complex or verbose that would benefit from being a utility?
[  ] Is it used often ?
[  ] Creating the new utility makes writing the code a lot easier (cleaner, simpler) ?
I don't think this as used that often to warrant it to be added to the library. Maybe you can make a better case for it, but I almost never need to check if an element has a specific class, I think in those rare cases it's okay to use the el() to utilize the HTMLElement API.
With this project the goal is not to add everything and eventually become jQuery but carefully pick and choose what gets added. What I would like to look into adding is a way to make this module tree-shakeable or have different different modules that a user can import to limit how much code they or importing or add more utilities if they need that.
Also please fill out the PR prompt so I can get more context as to why you think these changes would be good. By simply omitting and description in your PR I don't get any context as to why this might be a good change from another persons perspective.


Simplify postinstall

Use || instead of && … || true

Coverage remained the same at 100.0% when pulling a10bcf2debdab2bd5b16fa50940326642edd9675 on simplify-postinstall into 4daf6d7dc7beea6a6723e38620315afc1d7b1bb9 on master.


Set RNAME and POS of unmapped read identical to mate

As recommended in the SAM specification:
"For a unmapped paired-end or mate-pair read whose mate is mapped,
the unmapped read should have RNAME and POS identical to its mate."
No reason is given, but this is probably to make the unmapped read be
adjacent to its partner in a coordinate-sorted BAM file.
See #212

I had a quick look and was first confused by mate_name1 and mate_name2, then I realised that this is probably the reference chromosome ID that the mate is mapped to, right? perhaps it should be named ref_id_mate1 or something instead?

Good point; I just refactored the code to use somewhat more consistent variable names. The code in the Sam class uses the term "name" to describe reference ids and query ids because that’s consistent with what the SAM spec uses (RNAME, QNAME), so I opted for mate_reference_name1.

Great, happy to merge!Sent from my iPhoneOn 8 Jun 2023, at 10:06, Marcel Martin @.***> wrote:
Good point; I just refactored the code to use somewhat more consistent variable names. The code in the Sam class uses the term "name" to describe reference ids and query ids because that’s consistent with what the SAM spec uses (RNAME, QNAME), so I opted for mate_reference_name1.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>


Sept Demo | Payment Calculator

Requirement 1:
The following changes to be updated:

Regarding Petition Fee - refer pucardotorg/Dristi_kerala#942 or section 3.2 Efiling the document below
Regarding Summons E Post payment refer section 3.1 Summons

https://docs.google.com/document/d/1CANeDVZr8MoFJSf3f_3sG_vxLuw7-8haXJb5JKDtv9Y/edit
Requirement 2:
The Delay Condonation Application is filed during the E Filing process if there is a delay of more than 1 month in filing a case. A payment of Rs 2 is to be added to the E Filing fee if a delay condonation application is filed.

for requirement two, at any chance in future it will be slab based penalty for delay in application , eg. for 1 month =2rs for 2  month=5rs  @devarajd94

@prasanna2400 it is an application fee and not penalty fee. Hence it will remain constant. The rate may change in future. No slab based implementation expected. Good call out.

thanks for clarification @devarajd94 , is delay duration is also constant?

Delay duration will vary. But we have only 2 scenarios. If it is before 30 days then no delay condonation. If it is past 30 days then there is a delay condonation. This is already covered as a part of E Filing. @prasanna2400

@devarajd94  do we need to show  Delay Condonation Application in breakdown?

Yes @prasanna2400

For other channels like SMS and email, does the court fees calculation follow section 3.1.5 of the HLD Payment Calculator?  @devarajd94

Hi @Ramu-kandimalla
I have verified it in the Dev solution,it's working fine for E-filing.
url: https://dristi-kerala-dev.pucar.org/digit-ui/employee/dristi/pending-payment-inbox/pending-payment-details?filingNumber=F-C.1973.002-2024-002053&businessService=application-voluntary-submission&consumerCode=APPLICATION-NO-2024-09-20-000274_APPL_FILING&paymentType=Application Filing Fee
Test cases:
1.verify that if Cheque Amount <= 200000 then, amount for Petition Fee is 500.
reference link: https://jam.dev/c/65e41573-44c3-4f93-9be0-ae17851d45f1
2.verify that if Cheque Amount <= 5000000 then, amount for Petition Fee is 5000.
reference link:  https://jam.dev/c/83d89c98-22d9-4afc-8cb9-570438744d40
2.verify that if Cheque Amount > 5000000 then, amount for Petition Fee is 10000.
reference link: https://jam.dev/c/e88f6ef9-b3af-4358-8905-d7b8c3e58c81
3.verify that Application Fee per Application = INR 2
4.verify that Vakalathnama Fee = INR 6
5.verify that Advocate Welfare Fund = INR 25
6.verify that Advocate Clerk’s Welfare Fund = INR 12
7.verify that  if there is a delay of more than 1 month in filing a case for Delay Condonation Application, Delay Condonation Application Fee = INR 2

Hi @kashish384  Please verify in the UAT env

Hi @Ramu-kandimalla, verified in UAT env for  E-Filing

url: https://dristi-kerala-uat.pucar.org/digit-ui/employee/dristi/pending-payment-inbox/pending-payment-details?filingNumber=F-C.1973.002-2024-000451&businessService=task-summons&consumerCode=TASK-NO-2024-09-25-000363_EMAIL_COURT&paymentType=Summons Email Court Fee

Reference link: https://jam.dev/c/abf82876-5713-4f85-9ab8-ef0ba65a05aa

Test cases:
1.verify that if Cheque Amount <= 200000 then, amount for Petition Fee is 500.
2.verify that if Cheque Amount <= 5000000 then, amount for Petition Fee is 5000.
3.verify that if Cheque Amount > 5000000 then, amount for Petition Fee is 10000.
4.verify that Application Fee per Application = INR 2
5.verify that Vakalathnama Fee = INR 6
6.verify that Advocate Welfare Fund = INR 25
7.verify that Advocate Clerk’s Welfare Fund = INR 12
8.verify that if there is a delay of more than 1 month in filing a case for Delay Condonation Application, Delay Condonation Application Fee = INR 2

Hi @devarajd94  @Taherabharmal  Please verify and confirm

Hi @Ramu-kandimalla, for summons payment related still not fixed.
https://jam.dev/c/6e5be060-74d9-4de2-ae3c-7cd33ab8af64

Hi @kashish384
Summons related payment changes to show breakup from api still in progress.
Will update once done.

[ ] Restrict amount to 2 significant digits
[ ] Round up the second digit if 3rd digit is 5-9, Round down if its 0-4

fixed in solution qa and deployed

*verified in QA[sol] env for Summons_its working fine.
Summons SMS Court Fee:
https://jam.dev/c/f86d9129-47fc-4bc0-9bec-73891d40ea38
Summons Email Court Fee:
https://jam.dev/c/3077568a-71c2-419f-9435-9f31225e8844
Summons iCops Court Fee:
https://jam.dev/c/1b84c5b0-0ccf-4fcb-88a2-9ef015d1dca4
Summons Rpad Court Fee:
https://jam.dev/c/a995a770-9c66-4140-adae-a9ea201b51b5
*Regarding Summons E Post payment

Summons Post Court Fee:
https://jam.dev/c/4e29b7f9-8a2b-4247-ac0a-0d0e494db831
Summons Post Process Fee
https://jam.dev/c/4e29b7f9-8a2b-4247-ac0a-0d0e494db831

*verified in QA[sol] env, its working fine.

*verified in UAT env, its working fine.

*verified in UAT env, its working fine.
Summons SMS Court Fee:
https://jam.dev/c/5155332e-ca03-4571-9fcd-f93c06f16929
Summons Email Court Fee:
https://jam.dev/c/64facc45-3589-4f2b-b9a1-7eba6c8800fd
Summons iCops Court Fee:
https://jam.dev/c/40677908-b776-4e47-92f5-5c8c78cf883b
*Regarding Summons E Post payment
Summons Post Court Fee & Summons Post Process Fee:
https://jam.dev/c/33205583-9679-47fd-929f-86028e07cbd3

*verified in UAT env, Not working.
Summons Rpad Court Fee:
https://jam.dev/c/48e1219e-88d9-4e06-a0dd-8ae0a68767bb

@kashish384  can you please verify weather end point is getting called or not.

@Ramu-kandimalla  please assign ui developer to check this issue

Hi @kashish384  please recheck it's working fine

cc @rajeshcherukumalli @Ramu-kandimalla

*verified in UAT env, its working fine.
Summons Rpad Court Fee:

Verified


[Bug Report] Could not resolve "lodash"

Element UI version
2.15.12
OS/Browsers version
_
Vue version
2.7.14
Reproduction Link
https://stackblitz.com/edit/vitejs-vite-uu5e2r?file=package.json&terminal=dev
Steps to reproduce

: run dev

What is Expected?
fix bug
What is actually happening?
[ERROR] Could not resolve "lodash"
node_modules/.pnpm/element-ui@2.15.12_vue@2.7.14/node_modules/element-ui/lib/element-ui.common.js:230:25:
230 │ module.exports = require("lodash");

Just a moment, I'll check the problem

@XioDone Will be fixed in a minor release, watch for post-release updates


PDF Time stamp should be readable

05 Nov 2020 5 PM IST
Timezone should be based on user who run the program.
Twitter timestamp are UTC needs to be converted to local TZ.

Here: https://github.com/twindle-co/twindle/pull/623

DOne finally. Without all those files https://github.com/twindle-co/twindle/pull/626

@PuruVJ next time in the PR description just add "closes #"

Also, I don't see much value in having the timestamp for all tweets? The main timestamp (or date) should give a good indication

I think we should replace the timestamp value of format "MMM d, yyyy h:mm aaaa" with just "MMM d, yyyy"
ie Nov 12, 2001 12:23 pm -> Nov 12, 2001
(BTW, that above is my birthday 😁🤣)


Transaction list issues

There seems to be something off with the transaction list in v2.0.0. on staging
Safe on production

Safe on staging

There's some transactions that don't appear on staging but on production and others that only appear on staging. Then there is this loading spinner in the type filed which is similar to the issue we had with the Safe Apps origin field.
Safe address: 0xCfF7eB7C0cAc85188dF7748486A3ab8a20F56315

I pushed the fix for endless Loading  in PR https://github.com/gnosis/safe-react/pull/807. I'm tackling  Apps problems in that PR so we can review/merge all together


Add support for filtering services discovered by "ConsulDiscoveryClient#getAllInstances" and "ConsulDiscoveryClient#getServices"

My use case for it is filtering services by an API Gateway.
The gateway is auto discovering all services registered in consul, filtering from the list only services with "Edge" tag in order to allow the services to decide on exposing routes through the API Gateway

Why can't this be handled by a ServerListFilter from ribbon?

@spencergibb ServerListFilter is filtering servers from the list for already created route, this filter will allow filtering services and not creating the routes at all.

consul doesn't know anything about routes. Why couldn't you create a ServerListFilter that is only used in Zuul? I don't see the need for another filter.

Sorry, I misunderstood. I still don't think it's a consul specific problem. If anything it's a DiscoveryClientRouteLocator problem, or enhancement.

I agree, enhancing the DiscoveryClientRouteLocator would be better, the only issue with it is that the DiscoveryClient doesn't have an interface for returning the services with the metadata since it is not consul specific.
As I see it the only way to do it using the DiscoveryClientRouteLocator is by adding some logic that will add all the "excluded" services to the ignore list before running the DiscoveryClientRouteLocator#locateRoutes

It does. ServiceInstance.getMetadata(). I opened https://github.com/spring-cloud/spring-cloud-netflix/issues/2747


Package Structure - Part 1

$ABAPGIT_CTS:
ZCL_ABAPGIT_CTS_API
ZCL_ABAPGIT_DEFAULT_TRANSPORT
ZCL_ABAPGIT_TRANSPORT
ZCL_ABAPGIT_TRANSPORT_2_BRANCH
ZCL_ABAPGIT_TRANSPORT_MASS
ZCL_ABAPGIT_TRANSPORT_OBJECTS
ZIF_ABAPGIT_CTS_API
#4253

getting some dumps when pulling, but seems ok with standalone

https://github.com/abapGit/abapGit/issues/4274


Handle teardown failure to avoid blocking cleanup

When CNI isn't configured, the pod creation fails and leaves the network namespace uncleared. The cleanup process shouldn't depend on CNI being configured to properly remove the pod

Sample flow shared by @sameersaeed
pod-config.json:, per crictl docs
{
  "metadata": {
    "name": "nginx-sandbox",
    "namespace": "default",
    "attempt": 1,
    "uid": "65fb5845de97d8a4"
  },
  "log_directory": "/tmp",
  "linux": {
  }
}

Current behaviour from running sandbox pod - receive CNI plugin error, but pod still comes up:
root@32748c76b2ba:/src# crictl pods
POD ID              CREATED             STATE               NAME                NAMESPACE           ATTEMPT             RUNTIME

root@32748c76b2ba:/src# crictl runp pod-config.json
E1004 17:42:14.071632   31488 remote_runtime.go:176] "RunPodSandbox from runtime service failed" err="rpc error: code = Unknown desc = failed to setup network for sandbox \"803615d3f284991954f660a4d85295c15839ba8fa574640d58dd02de6a248d80\": cni plugin not initialized"
FATA[0000] run pod sandbox: rpc error: code = Unknown desc = failed to setup network for sandbox "803615d3f284991954f660a4d85295c15839ba8fa574640d58dd02de6a248d80": cni plugin not initialized

root@32748c76b2ba:/src# crictl pods
POD ID              CREATED             STATE               NAME                NAMESPACE           ATTEMPT             RUNTIME
803615d3f2849       1 second ago        NotReady            nginx-sandbox       default             1                   (default)

Changes made
Current code (release 1.7):
https://github.com/containerd/containerd/blob/45967a7be455b7232f915c5926ed79059d035343/pkg/cri/server/sandbox_run.go#L323-L326
https://github.com/containerd/containerd/blob/45967a7be455b7232f915c5926ed79059d035343/pkg/cri/server/sandbox_run.go#L444-L447
Changes were added to the above sandbox_run.go code to skip the pod network teardown if no CNI plugins are initialized. This allows for the rest of the deferred cleanup steps to run so that the CNI plugins failure does not block the pod from being fully cleaned up as expected:
// Teardown network if an error is returned.
if cleanupErr = c.teardownPodNetwork(deferCtx, sandbox); cleanupErr != nil {
	log.G(ctx).WithError(cleanupErr).Errorf("Failed to destroy network for sandbox %q", id)

	// ignoring failed to destroy networks when we failed to setup networks
	if sandbox.CNIResult == nil {
		cleanupErr = nil
	}
}

// Teardown network if an error is returned.
if cleanupErr = c.teardownPodNetwork(deferCtx, sandbox); cleanupErr != nil {
	log.G(ctx).WithError(cleanupErr).Errorf("Failed to destroy network for sandbox %q", id)

	// ignoring failed to destroy networks when we failed to setup networks
	if sandbox.CNIResult == nil {
		cleanupErr = nil
	}
}

This makes it so that a pod cannot be started if no CNI plugins are initialized:
root@32748c76b2ba:/src# crictl pods
POD ID              CREATED             STATE               NAME                NAMESPACE           ATTEMPT             RUNTIME

root@32748c76b2ba:/src# crictl runp pod-config.json
E1004 17:40:26.373297   29655 remote_runtime.go:176] "RunPodSandbox from runtime service failed" err="rpc error: code = Unknown desc = failed to setup network for sandbox \"bd9a6545dc2f9f1484b36a3b72b2c87d2e44b9f74fe5ffc90f53fa220d73462b\": cni plugin not initialized"
FATA[0000] run pod sandbox: rpc error: code = Unknown desc = failed to setup network for sandbox "bd9a6545dc2f9f1484b36a3b72b2c87d2e44b9f74fe5ffc90f53fa220d73462b": cni plugin not initialized

root@32748c76b2ba:/src# crictl pods
POD ID              CREATED             STATE               NAME                NAMESPACE           ATTEMPT             RUNTIME

/ok-to-test

@dmcgowan does the below error need to be fixed before pr is merged?
[FAIL] [k8s.io] Container OOM runtime should output OOMKilled reason [It] should terminate with exitCode 137 and reason OOMKilled
Link to that is here


Metadata v31

Hi, sorry for the wrong place for the issues. I am using your PR for Il2CppInspector for metadata v29. Unfortunately most games use the newer version. So fields placed in wrong order and processing fails. Is there any chance that you will look into?

There is no metadata v31.

Custom engine or something. There is not - as of unity 6 - a metadata v31.

So metadata is obfuscated? I had that with some games, they all using unity 2022+ with error:
Il2CppInspector Command-Line Edition
Version 2021.1+a165b63b5aecb76c50b7ddb435b9d35bf7a44fe7
(c) 2017-2021 Katy Coe - www.djkaty.com - www.github.com/djkaty

Using plugin: IL2CPP API Discovery
Using plugin: Binary metadata field order deobfuscator
Using plugin: Metadata strings XOR decryptor
Using plugin: Binary file XOR decryptor
Extracting metadata from (archive)/assets/bin/Data/Managed/Metadata/global-metadata.dat
Detected metadata version 29
Extracting binary from lib/arm64-v8a/libil2cpp.so
Processed XXXXXX relocations
Plugin Binary file XOR decryptor: Detecting encryption
Container format: ELF64
Container endianness: Little
Architecture word size: 64-bit
Instruction set: ARM64
Global offset: 0x0000000000000000
Symbol table(s) found with XXX entries
No matches in symbol table
No matches via code heuristics
Required structures acquired from data heuristics
CodeRegistration struct found at 0x000000000XXXXXXX (file offset 0x0XXXXXXX)
MetadataRegistration struct found at 0x000000000XXXXXXX (file offset 0x0XXXXXXX)
The detected Il2CppCodeRegistration / Il2CppMetadataRegistration structs do not pass validation. This may mean that their fields have been re-ordered as a form of obfuscation and Il2CppInspector has not been able to restore the original order automatically. Consider re-ordering the fields in Il2CppBinaryClasses.cs and try again.
Analyze IL2CPP data: 100.20 sec

Just tried Cpp2IL, works unlike Il2CppInspector

Detected metadata version 29
from your il2cppinspector log

no idea what dumper is doing or where it's getting 31 from but this is just v29

What about minor versions of v29? Can we get exact version (29.1, 29.2, ...) without unity version like inspector does?

--verbose will log minor version.

I'm talking about il2cppinspector, else can/will Cpp2IL generate stuff for disassembler like Ghidra or IDA?

Custom engine or something. There is not - as of unity 6 - a metadata v31.

I test unity6 and unity 22.3.35f1, it is confused that unity 6 are metadata v29 and unity22.3.35f1 are metadata v31

Cpp2IL now have v31 support


graph/internal/set: make complex set operations pure functions

Looking at the benchmarks for this approach, we have a better outcome than the original code for community.KCliqueCommunities and a better outcome than the pointer approach in #900 for topo.BronKerbosch and topo.CliqueGraph.
Original versus this approach

nameoriginal time/oppure function time/opdelta
KCliqueCommunities/simple-k=3-876.1µs ± 2%61.1µs ± 3%-19.81%(p=0.000 n=17+20)
KCliqueCommunities/Batagelj-Zaversnik_Graph-k=3-8230µs ± 3%184µs ± 1%-19.83%(p=0.000 n=20+17)
KCliqueCommunities/Batagelj-Zaversnik_Graph-k=4-8229µs ± 2%183µs ± 1%-20.02%(p=0.000 n=19+18)
BronKerbosch/wikipedia_example-819.0µs ± 2%21.4µs ± 3%+12.39%(p=0.000 n=20+20)
BronKerbosch/Batagelj-Zaversnik_Graph-8106µs ± 6%111µs ± 6%+4.62%(p=0.000 n=20+20)
CliqueGraph/simple-854.5µs ± 3%58.1µs ± 2%+6.52%(p=0.000 n=20+20)
CliqueGraph/Batagelj-Zaversnik_Graph-8149µs ± 6%156µs ± 1%+4.75%(p=0.000 n=20+16)

Pointer method approach versus this approach

namepointer method time/oppure function time/opdelta
BronKerbosch/wikipedia_example-824.7µs ± 6%21.4µs ± 3%-13.22%(p=0.000 n=20+20)
BronKerbosch/Batagelj-Zaversnik_Graph-8123µs ± 5%111µs ± 6%-9.78%(p=0.000 n=20+20)
CliqueGraph/simple-862.1µs ± 1%58.1µs ± 2%-6.53%(p=0.000 n=18+20)
CliqueGraph/Batagelj-Zaversnik_Graph-8167µs ± 4%156µs ± 1%-6.18%(p=0.000 n=20+16)

I also think the clarity of the calls is not bad (surprisingly to me).
Finally, the purity of the functions means that other potentially surprising behaviours that I did not notice previously are now not possible.
All up I think this is the correct approach.
Please take a look.
Fixes #863.
Closes #900.

I'm happy to make that change.

PTAL


get-credentials fails for kubernetes cluster

Hey, I just spun up my first k8s cluster on Azure. After it got running, I wanted to get the credentials, but they fail with the following error. I was able to SSH to the master and grab from there, but wanted to see if there was a bug somewhere:
~/d/c/k/acs ❯❯❯ az acs kubernetes get-credentials --resource-group=$RESOURCE_GROUP --name=$CLUSTER_NAME                                                                                master ✱ ◼
Password:
Could not deserialize key data.
Traceback (most recent call last):
  File "/Users/slokas/lib/azure-cli/lib/python2.7/site-packages/azure/cli/main.py", line 37, in main
    cmd_result = APPLICATION.execute(args)
  File "/Users/slokas/lib/azure-cli/lib/python2.7/site-packages/azure/cli/core/application.py", line 157, in execute
    result = expanded_arg.func(params)
  File "/Users/slokas/lib/azure-cli/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 358, in _execute_command
    raise ex
SSHException: Could not deserialize key data.

I will try to repro. Are you sure you typed the correct key password?

I assume the password is my Mac password? It's the same prompt when something needs elevated permissions.

No, it would be the password for your SSH key in ~/.ssh/rsa_pub You probably created it a long time ago...

Closing this issue, please re-open if you can't resolve by moving and re-creating that key.

Hey Brendan, sorry for the late reply, yes you are correct it was asking for my ssh passphrase, but looked exactly like the Mac admin password. I just upgraded to latest OSX and things have gone downhill from there.
Thanks for keeping up with me on this!

I'll update the message to include 'ssh'
--brendan

From: Steve Sloka<EMAIL_ADDRESS>Sent: Wednesday, April 5, 2017 6:07:33 AM
To: Azure/azure-cli
Cc: Brendan Burns; State change
Subject: Re: [Azure/azure-cli] get-credentials fails for kubernetes cluster (#2636)
Hey Brendan, sorry for the late reply, yes you are correct it was asking for my ssh passphrase, but looked exactly like the Mac admin password. I just upgraded to latest OSX and things have gone downhill from there.
Thanks for keeping up with me on this!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli%2Fissues%2F2636%23issuecomment-291855399&data=02|01|bburns%40microsoft.com|9b82b4ef808146e90ac108d47c24ba0c|72f988bf86f141af91ab2d7cd011db47|1|0|636269944556252861&sdata=iB6SGTe3xAz3sqfqNpSv86Bicm%2BqZ0cUsbtx3y4hzy8%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFfDgn5gkn66mxIw5NiDq9IuoY9puZhAks5rs5IVgaJpZM4MqpfM&data=02|01|bburns%40microsoft.com|9b82b4ef808146e90ac108d47c24ba0c|72f988bf86f141af91ab2d7cd011db47|1|0|636269944556252861&sdata=PDNIlcbkCZRpjY1fhh2Eq7mmlQPugn8L48vPgIdGOcc%3D&reserved=0.


Enhance University Form-Filling Process with Conversational AI

We face challenges with students providing incorrect or incomplete information in forms, leading to delays and inaccuracies. Following up manually is inefficient and frustrating.
Integrate a conversational AI interface for real-time interaction. The AI should prompt students for missing or incorrect details, allowing changes directly in the conversation. It should use an LLM to understand natural language, update fields, validate information, and provide feedback.
We considered sending follow-up emails or messages, but this is time-consuming and lacks immediate feedback. Strict validation checks within the form also don't offer a conversational, interactive experience.
Additional context
A student enters an incorrect date of birth (e.g., "31/02/2000").
The AI prompts: "The date of birth you entered seems incorrect. Could you provide the correct date?"
Student responds: "It's 28/02/2000."
AI updates: "Thank you! I've updated your date of birth to 28/02/2000."
For missing phone number: "Could you provide your contact number?"
Student: "Sure, it's<PHONE_NUMBER>."
AI updates: "Great, I've added your phone number as<PHONE_NUMBER>."

@kevin4801 Hello, I can work on this feature. I have similar experience working on this type of project.

@adeprez This is the same idea as #445 - do you want to discuss with @ChaikhiBelaid about this :)

@ChaikhiBelaid If you'd like to take a look at this, we would probably recommend implementing this as an optional additional engine. How the agent asks for additional data could be extensive & require many integrations - so can grow and involve many contributors
Do you want to discuss further on Discord?

Hello, I’m interested in working on this feature for my startup. I have relevant experience and am excited about the potential for collaboration. Let's work together to build this.

Hello, Yes of corse I am excited to work on such a feature together.
@lyie28 @kevin4801 @adeprez

I made a Discord channel where we can discuss this feature further @ChaikhiBelaid @kevin4801 : https://discord.com/channels/1216089456134586388/1265250774913388605

@lyie28 Thanks. But i couldn't join the channel. Can you add me this is my discord: belaid0635

@ChaikhiBelaid - I think you have to join the LaVague server first

@lyie28 It is done. Thank you!


dcp: add --chunksize and --blocksize options

Add command-line options for --chunksize and --blocksize, and add
them to the dcp.1 man page, since these options now exist for both
dcp and dcp1.
Improve the usage message for these options in dcp.1 to not be
self-referential, and explain what they are actually used for.
It was otherwise unclear what the difference between them was.
Keep the defaults at 1MB for this patch, but it probably makes
sense to make them at least the Lustre stripe size to minimize
contention between multiple threads reading/writing the same file.
Signed-off-by: Andreas Dilger<EMAIL_ADDRESS>

Great, thanks @adilger !


Cordova 9

What version of Streaming-Meda-Cordova-Plugin are you using?
Lastest version
What version of Cordova are you using?
9.0.0
What devices are affected?
Android
Please describe the issue in detail, with relevant code samples
App Crash when play Video. (iOS works perfect, only fails on android)

+1

+1

+1

If found the two activities were disappearing.  They were there the first time I built after re-adding the platform, on the next build one of them disappeared (audio first), the on the next build the video activity disappered.  Solve it by removing android platform.  Adding the following to my config.xml, and then re-adding Android platform.  It's worked so far:
<config-file parent="/manifest/application" target="AndroidManifest.xml">
    <activity android:configChanges="orientation|screenSize" android:label="Simple Audio Stream" android:name="com.hutchind.cordova.plugins.streamingmedia.SimpleAudioStream" />
    <activity android:configChanges="orientation|screenSize" android:label="Simple Video Stream" android:name="com.hutchind.cordova.plugins.streamingmedia.SimpleVideoStream" />
</config-file>


[request-for-external] page 233 "Apply Vendor Entries"

Please make the following functions protected
protected procedure CalcApplnRemainingAmount(Amount: Decimal): Decimal
protected procedure FindAmountRounding()
protected procedure HandleChosenEntries(Type: Enum "Vendor Apply Calculation Type"; CurrentAmount: Decimal; CurrencyCode: Code[10]; PostingDate: Date)

Has already been asked again in
https://github.com/microsoft/ALAppExtensions/issues/18103
but it is not included in latest w1 onprem 20.3.42673.43026

We do not expose new procedures in PROD versions, it will be available in version 21 only.


⚠ AVS (kty) has degraded performance

In 726dde5, AVS (kty) (https://avus.com.br) experienced degraded performance:

HTTP code: 200
Response time: 8489 ms

Resolved: AVS (kty) performance has improved in ca67b29.


Add "PATCH" as a MethodType to be used by Request or fetch

As per https://tools.ietf.org/html/rfc5789

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


sbt crashes on Lustre and NFS because lock0 isn't implemented

sbt uses a file locking mechanism that isn't supported on networked filesystems like Lustre and NFS.
Using the latest sbt downloaded from the official website. Java version is 1.7.0_45
$ sbt --version
java.io.IOException: Function not implemented
        at sun.nio.ch.FileDispatcherImpl.lock0(Native Method)
        at sun.nio.ch.FileDispatcherImpl.lock(FileDispatcherImpl.java:90)
        at sun.nio.ch.FileChannelImpl.tryLock(FileChannelImpl.java:1063)
        at java.nio.channels.FileChannel.tryLock(FileChannel.java:1154)
        at xsbt.boot.Locks$GlobalLock.withChannel$1(Locks.scala:86)
        at xsbt.boot.Locks$GlobalLock.xsbt$boot$Locks$GlobalLock$$withChannelRetries$1(Locks.scala:78)
        at xsbt.boot.Locks$GlobalLock$$anonfun$withFileLock$1.apply(Locks.scala:97)
        at xsbt.boot.Using$.withResource(Using.scala:10)
        at xsbt.boot.Using$.apply(Using.scala:9)
        at xsbt.boot.Locks$GlobalLock.ignoringDeadlockAvoided(Locks.scala:58)
        at xsbt.boot.Locks$GlobalLock.withLock(Locks.scala:48)
        at xsbt.boot.Locks$.apply0(Locks.scala:31)
        at xsbt.boot.Locks$.apply(Locks.scala:28)
        at xsbt.boot.Launch.locked(Launch.scala:238)
        at xsbt.boot.Launch.app(Launch.scala:147)
        at xsbt.boot.Launch.app(Launch.scala:145)
        at xsbt.boot.Launch$.run(Launch.scala:102)
        at xsbt.boot.Launch$$anonfun$apply$1.apply(Launch.scala:35)
        at xsbt.boot.Launch$.launch(Launch.scala:117)
        at xsbt.boot.Launch$.apply(Launch.scala:18)
        at xsbt.boot.Boot$.runImpl(Boot.scala:41)
        at xsbt.boot.Boot$.main(Boot.scala:17)
        at xsbt.boot.Boot.main(Boot.scala)
Error during sbt execution: java.io.IOException: Function not implemented

I don't see a good solution for this. Maybe implement a flag that says "YOLO I don't need locking" and run one instance of sbt at a time?
/cc @jsuereth

I have the same error. Please fix it.

Same with me. Please provide a no-lock flag or something similar

Same with me.

Upstream issue: https://bugs.openjdk.java.net/browse/JDK-8193594

The (long-winded, but currently available) workaround involves creating a custom launch config file and defining lock in the [boot] section as false.

what is the solution to fix it

One solution is fix the JDK (if possible). Another is to change the code in sbt/launcher to avoid using lock0 (if possible). Another is to change the launchconfig file in sbt to expose a system property for the lock file so you can opt-out (what Eugene is referring to as the "YOLO I don't need locking" flag).
I'm happy to try and give more details if you let me know what solution you're interested in.

I think the second option to change the launcher.
find bellow my sbt-launcher.jar
" zip.vim version v28    2 " Browsing zipfile /nfs/2017/b/brabo-hi/.brew/Cellar/sbt/1.1.0/libexec/bin/sbt-launch.jar    3 " Select a file with cursor and press ENTER    4    5 META-INF/    6 org/    7 org/apache/    8 org/apache/ivy/    9 org/apache/ivy/core/   10 org/apache/ivy/core/cache/   11 org/apache/ivy/core/deliver/   12 org/apache/ivy/core/event/   13 org/apache/ivy/core/event/download/   14 org/apache/ivy/core/event/resolve/   15 org/apache/ivy/core/event/retrieve/   16 org/apache/ivy/core/install/   17 org/apache/ivy/core/module/   18 org/apache/ivy/core/module/descriptor/   19 org/apache/ivy/core/module/id/   20 org/apache/ivy/core/module/status/   21 org/apache/ivy/core/publish/   22 org/apache/ivy/core/report/   23 org/apache/ivy/core/repository/   24 org/apache/ivy/core/resolve/   25 org/apache/ivy/core/retrieve/   26 org/apache/ivy/core/search/   27 org/apache/ivy/core/settings/   28 org/apache/ivy/core/sort/   29 org/apache/ivy/osgi/   30 org/apache/ivy/osgi/core/   31 org/apache/ivy/osgi/util/   32 org/apache/ivy/plugins/   33 org/apache/ivy/plugins/circular/   34 org/apache/ivy/plugins/conflict/   35 org/apache/ivy/plugins/latest/   36 org/apache/ivy/plugins/lock/   37 org/apache/ivy/plugins/matcher/   38 org/apache/ivy/plugins/namespace/   39 org/apache/ivy/plugins/parser/   40 org/apache/ivy/plugins/parser/m2/   41 org/apache/ivy/plugins/parser/xml/   42 org/apache/ivy/plugins/report/   43 org/apache/ivy/plugins/repository/   44 org/apache/ivy/plugins/repository/file/   45 org/apache/ivy/plugins/repository/url/   46 org/apache/ivy/plugins/resolver/   47 org/apache/ivy/plugins/resolver/util/   48 org/apache/ivy/plugins/trigger/   49 org/apache/ivy/plugins/version/   50 org/apache/ivy/util/   51 org/apache/ivy/util/extendable/   52 org/apache/ivy/util/filter/   53 org/apache/ivy/util/url/   54 sbt/   55 scala/   56 scala/collection/   57 scala/collection/convert/   58 scala/collection/generic/   59 scala/collection/immutable/   60 scala/collection/mutable/   61 scala/compat/   62 scala/math/   63 scala/reflect/   64 scala/runtime/   65 scala/sys/   66 scala/util/   67 scala/util/control/   68 scala/util/hashing/   69 scala/util/matching/   70 scala/xml/   71 xsbt/   72 xsbt/boot/   73 xsbti/   74 module.properties

So the line of the launcher that eventually calls lock0 is: https://github.com/sbt/launcher/blob/v1.0.2/launcher-implementation/src/main/scala/xsbt/boot/Locks.scala#L88
Looks like some quite tricky code (I've never seen it before) what with dealing with potential dead locks and catching null pointer exceptions.
What I suggest is to attack this problem by testing: try and write an interesting test around that code, run it and see it fail on your NFS disk, and then try and see if you can rewrite it in a way that the intended behaviour is preserved and works on NFS.
Let me know how it goes.

I'm not scala developper, but java. So to writing test could take me time. What is the easiest way to fix it

Then I think the 3rd option, for which I've created https://github.com/sbt/sbt/pull/3927.

In which file should i add sbt.boot.lock-true

In the file sbt/sbt.boot.properties inside ~/.sbt/launchers/1.1.0/sbt-launch.jar.

I've extract the file sbt-launch.jar using (jar xf sbt-launch.jar), made changes and recreate the jar (jar cvf sbt-launch.jar *) . But got manifest error when i run (java -jar sbt-launch.jar)  "no main manifest attribute, in sbt-launch.jar"

Is it what i have to add ?
[boot]
directory: ${sbt.boot.directory-${sbt.global.base-${user.home}/.sbt}/boot/}
lock: ${sbt.boot.lock-true}

correct

lock: ${sbt.boot.lock-true} should i put it true or false  (sbt.boot.lock) i mean?


Getting started link in README is broken

The "Java Getting Started on Google Cloud Platform" link points to https://cloud.google.com/java/getting-started which 404s.

Thanks for reporting this. We're working on getting that page out.


Hello World in C++

What the program do ?

In what programming language it is written?

Hello World in C# and Python language

Hello World in HTML language


Update Goes16.ipynb

Add brier skill score (in addition to brier score); add another visualization and tweak others; expand commentary

I really like the new visualization in the notebook.


Support for .vue-Files

depcheck supports Vue-Syntax with .vue-Module.
Is there any possibility to enable this in npm-check? In Vue.js-Apps you can't use the unused packages feature

Duplicate of #290, where there is some more information.


Setting limit for number of open files

I haven't played with upstart scripts before but this appears to be the right way to set the limit,
http://upstart.ubuntu.com/wiki/Stanzas#limit


HIVE-26286: Fix NPE when starting WebHCat Service

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

@zabetak Could you please help review this PR? thanks!


Problem to add ng2 slides to my project

Hi,
I've follow the instructions but I think there is empty part regarding the systemconfig.js
In addition, I think it's a little bit problematic not to bundle it.
What do you think?
Can you help me with it?

Hey there, I've rewritten a large portion of this package and I will update it tonight! after that things should be back up and running

I've fixed a number of issues now, however I'm still having some small bugs around the slide flashing on animation. I will work on this soon.
The good news is, is you can now bundle the package!

Hey, Spittal..
That's a great news.
Thanks a lot for the quick response.. it's not obvious


Inference script

Can you release the inference script of this project? Thanks!

Do you need a demo for this?

yeah!

@jetwu-create I've made a gradio based web demo for Qwen2-VL. You can upload multi-image, or video data.
The fps for the video data is set to 1.0.

Closing the issue for now updates.