vsxmake "table index is nil"

Xmake Version
v2.6.5+HEAD.d239205f6
Operating System Version and Architecture
Windows 11
Describe Bug
xmake project -k vsxmake
Results in "table index is nil"
Expected Behavior
It generates the solution.
Project Configuration
No response
Additional Information and Error Logs
configure
{
    arch = x64
    ndk_stdcxx = true
    theme = default
    ccache = true
    kind = static
    plat = windows
    proxy_pac = pac.lua
    vs = 2022
    network = public
    mode = releasedbg
    buildir = build
    host = windows
}
using project kind vs2022
checking for debug.x64 ...
checking for cl.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\cl.exe
checking for Microsoft Visual Studio (x64) version ... 2022
checkinfo: cannot runv(dmd.exe --version), No such file or directory
checking for dmd ... no
checkinfo: cannot runv(ldc2.exe --version), No such file or directory
checking for ldc2 ... no
checkinfo: cannot runv(gdc.exe --version), No such file or directory
checking for gdc ... no
checkinfo: cannot runv(zig.exe version), No such file or directory
checking for zig ... no
checkinfo: cannot runv(zig.exe version), No such file or directory
checking for zig ... no
checking for unzip ... ok
checking for git ... ok
checking for gzip ... ok
checking for tar ... ok
git rev-parse HEAD
checking for cmake ... C:\Program Files\CMake\bin\cmake
checking for cmake ... C:\Program Files\CMake\bin\cmake
finding sentry-native from xmake ..
checking for xmake::sentry-native ... sentry-native 0.4.15
finding zlib from xmake ..
checking for xmake::zlib ... zlib v1.2.12
finding glm from xmake ..
checking for xmake::glm ... glm 0.9.9+8
finding mimalloc from xmake ..
checking for xmake::mimalloc ... mimalloc 2.0.6
finding tiltedcore from xmake ..
checking for xmake::tiltedcore ... tiltedcore v0.2.6
finding hopscotch-map from xmake ..
checking for xmake::hopscotch-map ... hopscotch-map v2.3.0
finding snappy from xmake ..
checking for xmake::snappy ... snappy 1.1.9
finding libsodium from xmake ..
checking for xmake::libsodium ... libsodium 1.0.18
finding protobuf-cpp from xmake ..
checking for xmake::protobuf-cpp ... protobuf-cpp 3.19.4
finding gamenetworkingsockets from xmake ..
checking for xmake::gamenetworkingsockets ... gamenetworkingsockets v1.4.0
finding catch2 from xmake ..
checking for xmake::catch2 ... catch2 2.13.9
finding libuv from xmake ..
checking for xmake::libuv ... libuv v1.44.1
finding mem from xmake ..
checking for xmake::mem ... mem 1.0.0
finding minhook from xmake ..
checking for xmake::minhook ... minhook v1.3.3
finding xbyak from xmake ..
checking for xmake::xbyak ... xbyak v6.03
finding directxtk from xmake ..
checking for xmake::directxtk ... directxtk 21.11.0
finding cef from xmake ..
checking for xmake::cef ... cef 100.0.24
finding cryptopp from xmake ..
checking for xmake::cryptopp ... cryptopp 8.4.0
finding spdlog from xmake ..
checking for xmake::spdlog ... spdlog v1.10.0
checking for nasm ... no
checking for perl ... C:\Program Files\Git\usr\bin\perl
finding openssl from xmake ..
checking for xmake::openssl ... openssl 1.1.1n
finding cpp-httplib from xmake ..
checking for xmake::cpp-httplib ... cpp-httplib 0.9.2
finding gtest from xmake ..
checking for xmake::gtest ... gtest 1.11.0
finding discord from xmake ..
checking for xmake::discord ... discord 2.5.6
finding imgui from xmake ..
checking for xmake::imgui ... imgui v1.87
finding entt from xmake ..
checking for xmake::entt ... entt v3.10.0
finding recastnavigation from xmake ..
checking for xmake::recastnavigation ... recastnavigation 1.5.1
checking for link.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe
checking for the linker (ld) ... link.exe
checking for link.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe
checking for the shared library linker (sh) ... link.exe
generating Code\common\BuildInfo.h.in to build\BuildInfo.h ..
  > replace GIT_BRANCH -> feat-clean
  > replace GIT_TAG -> v1.28.0-131-g78bd5f1a
generating Code\common\BuildInfo.h.in ... cache
checking for link.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe
checking for the static library archiver (ar) ... link.exe
checking for C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe ... ok
checking for flags (/subsystem:windows) ... ok
> link.exe "/subsystem:windows" "-nologo" "-dynamicbase" "-nxcompat" "-machine:x64"
checking for releasedbg.x64 ...
checking for cl.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\cl.exe
checking for Microsoft Visual Studio (x64) version ... 2022
checkinfo: cannot runv(dmd.exe --version), No such file or directory
checking for dmd ... no
checkinfo: cannot runv(ldc2.exe --version), No such file or directory
checking for ldc2 ... no
checkinfo: cannot runv(gdc.exe --version), No such file or directory
checking for gdc ... no
checkinfo: cannot runv(zig.exe version), No such file or directory
checking for zig ... no
checkinfo: cannot runv(zig.exe version), No such file or directory
checking for zig ... no
checking for unzip ... ok
checking for git ... ok
checking for gzip ... ok
checking for tar ... ok
checking for cmake ... C:\Program Files\CMake\bin\cmake
checking for cmake ... C:\Program Files\CMake\bin\cmake
finding sentry-native from xmake ..
checking for xmake::sentry-native ... sentry-native 0.4.15
finding zlib from xmake ..
checking for xmake::zlib ... zlib v1.2.12
finding glm from xmake ..
checking for xmake::glm ... glm 0.9.9+8
finding mimalloc from xmake ..
checking for xmake::mimalloc ... mimalloc 2.0.6
finding tiltedcore from xmake ..
checking for xmake::tiltedcore ... tiltedcore v0.2.6
finding hopscotch-map from xmake ..
checking for xmake::hopscotch-map ... hopscotch-map v2.3.0
finding snappy from xmake ..
checking for xmake::snappy ... snappy 1.1.9
finding libsodium from xmake ..
checking for xmake::libsodium ... libsodium 1.0.18
finding protobuf-cpp from xmake ..
checking for xmake::protobuf-cpp ... protobuf-cpp 3.19.4
finding gamenetworkingsockets from xmake ..
checking for xmake::gamenetworkingsockets ... gamenetworkingsockets v1.4.0
finding catch2 from xmake ..
checking for xmake::catch2 ... catch2 2.13.9
finding libuv from xmake ..
checking for xmake::libuv ... libuv v1.44.1
finding mem from xmake ..
checking for xmake::mem ... mem 1.0.0
finding minhook from xmake ..
checking for xmake::minhook ... minhook v1.3.3
finding xbyak from xmake ..
checking for xmake::xbyak ... xbyak v6.03
finding directxtk from xmake ..
checking for xmake::directxtk ... directxtk 21.11.0
finding cef from xmake ..
checking for xmake::cef ... cef 100.0.24
finding cryptopp from xmake ..
checking for xmake::cryptopp ... cryptopp 8.4.0
finding spdlog from xmake ..
checking for xmake::spdlog ... spdlog v1.10.0
checking for nasm ... no
checking for perl ... C:\Program Files\Git\usr\bin\perl
finding openssl from xmake ..
checking for xmake::openssl ... openssl 1.1.1n
finding cpp-httplib from xmake ..
checking for xmake::cpp-httplib ... cpp-httplib 0.9.2
finding gtest from xmake ..
checking for xmake::gtest ... gtest 1.11.0
finding discord from xmake ..
checking for xmake::discord ... discord 2.5.6
finding imgui from xmake ..
checking for xmake::imgui ... imgui v1.87
finding entt from xmake ..
checking for xmake::entt ... entt v3.10.0
finding recastnavigation from xmake ..
checking for xmake::recastnavigation ... recastnavigation 1.5.1
checking for link.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe
checking for the linker (ld) ... link.exe
checking for link.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe
checking for the shared library linker (sh) ... link.exe
generating Code\common\BuildInfo.h.in to build\BuildInfo.h ..
  > replace GIT_BRANCH -> feat-clean
  > replace GIT_TAG -> v1.28.0-131-g78bd5f1a
generating Code\common\BuildInfo.h.in ... cache
checking for link.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe
checking for the static library archiver (ar) ... link.exe
checking for C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe ... ok
checking for flags (/subsystem:windows) ... ok
> link.exe "/subsystem:windows" "-nologo" "-dynamicbase" "-nxcompat" "-machine:x64"
checking for release.x64 ...
checking for cl.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\cl.exe
checking for Microsoft Visual Studio (x64) version ... 2022
checkinfo: cannot runv(dmd.exe --version), No such file or directory
checking for dmd ... no
checkinfo: cannot runv(ldc2.exe --version), No such file or directory
checking for ldc2 ... no
checkinfo: cannot runv(gdc.exe --version), No such file or directory
checking for gdc ... no
checkinfo: cannot runv(zig.exe version), No such file or directory
checking for zig ... no
checkinfo: cannot runv(zig.exe version), No such file or directory
checking for zig ... no
checking for unzip ... ok
checking for git ... ok
checking for gzip ... ok
checking for tar ... ok
checking for cmake ... C:\Program Files\CMake\bin\cmake
checking for cmake ... C:\Program Files\CMake\bin\cmake
finding sentry-native from xmake ..
checking for xmake::sentry-native ... sentry-native 0.4.15
finding zlib from xmake ..
checking for xmake::zlib ... zlib v1.2.12
finding glm from xmake ..
checking for xmake::glm ... glm 0.9.9+8
finding mimalloc from xmake ..
checking for xmake::mimalloc ... mimalloc 2.0.6
finding tiltedcore from xmake ..
checking for xmake::tiltedcore ... tiltedcore v0.2.6
finding hopscotch-map from xmake ..
checking for xmake::hopscotch-map ... hopscotch-map v2.3.0
finding snappy from xmake ..
checking for xmake::snappy ... snappy 1.1.9
finding libsodium from xmake ..
checking for xmake::libsodium ... libsodium 1.0.18
finding protobuf-cpp from xmake ..
checking for xmake::protobuf-cpp ... protobuf-cpp 3.19.4
finding gamenetworkingsockets from xmake ..
checking for xmake::gamenetworkingsockets ... gamenetworkingsockets v1.4.0
finding catch2 from xmake ..
checking for xmake::catch2 ... catch2 2.13.9
finding libuv from xmake ..
checking for xmake::libuv ... libuv v1.44.1
finding mem from xmake ..
checking for xmake::mem ... mem 1.0.0
finding minhook from xmake ..
checking for xmake::minhook ... minhook v1.3.3
finding xbyak from xmake ..
checking for xmake::xbyak ... xbyak v6.03
finding directxtk from xmake ..
checking for xmake::directxtk ... directxtk 21.11.0
finding cef from xmake ..
checking for xmake::cef ... cef 100.0.24
finding cryptopp from xmake ..
checking for xmake::cryptopp ... cryptopp 8.4.0
finding spdlog from xmake ..
checking for xmake::spdlog ... spdlog v1.10.0
checking for nasm ... no
checking for perl ... C:\Program Files\Git\usr\bin\perl
finding openssl from xmake ..
checking for xmake::openssl ... openssl 1.1.1n
finding cpp-httplib from xmake ..
checking for xmake::cpp-httplib ... cpp-httplib 0.9.2
finding gtest from xmake ..
checking for xmake::gtest ... gtest 1.11.0
finding discord from xmake ..
checking for xmake::discord ... discord 2.5.6
finding imgui from xmake ..
checking for xmake::imgui ... imgui v1.87
finding entt from xmake ..
checking for xmake::entt ... entt v3.10.0
finding recastnavigation from xmake ..
checking for xmake::recastnavigation ... recastnavigation 1.5.1
checking for link.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe
checking for the linker (ld) ... link.exe
checking for link.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe
checking for the shared library linker (sh) ... link.exe
generating Code\common\BuildInfo.h.in to build\BuildInfo.h ..
  > replace GIT_BRANCH -> feat-clean
  > replace GIT_TAG -> v1.28.0-131-g78bd5f1a
generating Code\common\BuildInfo.h.in ... cache
checking for link.exe ... C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe
checking for the static library archiver (ar) ... link.exe
checking for C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\bin\HostX64\x64\link.exe ... ok
checking for flags (/LTCG) ... ok
> link.exe "/LTCG" "-nologo" "-dynamicbase" "-nxcompat" "-machine:x64"
checking for flags (/OPT:REF) ... ok
> link.exe "/OPT:REF" "-nologo" "-dynamicbase" "-nxcompat" "-machine:x64"
checking for flags (/subsystem:windows) ... ok
> link.exe "/subsystem:windows" "-nologo" "-dynamicbase" "-nxcompat" "-machine:x64"
error: @programdir\core\main.lua:280: @programdir\plugins\project\vsxmake\getinfo.lua:544: table index is nil
stack traceback:
    [@programdir\plugins\project\vsxmake\getinfo.lua:544]:
    [@programdir\plugins\project\vsxmake\vsxmake.lua:223]: in function '?'
    [@programdir\plugins\project\main.lua:79]: in function '_make'
    [@programdir\plugins\project\main.lua:89]:
    [C]: in function 'xpcall'
    [@programdir\core\base\utils.lua:280]:
    [@programdir\core\base\task.lua:519]: in function 'run'
    [@programdir\core\main.lua:278]: in function 'cotask'
    [@programdir\core\base\scheduler.lua:371]:

stack traceback:
	[C]: in function 'error'
	@programdir\core\base\os.lua:854: in function 'os.raiselevel'
	(...tail calls...)
	@programdir\core\main.lua:280: in upvalue 'cotask'
	@programdir\core\base\scheduler.lua:371: in function <@programdir\core\base\scheduler.lua:368>

The nil error is fixed in master, the coloring isnt


Change gemini.circumlunar.space to geminiprotocol.net

Change gemini.circumlunar.space to the new address, geminiprotocol.net. Also remove https://transjovian.org/gemini that currently does not exist.

@discerno Thanks!


min, max and clamp

Big fan!
Would like to help 🙂  I started following the contributor instructions, and have some partial entries made here and there, but I need to figure out package development, because I don't think any of my changes are showing.
I tried adding min() and max() and clamp(). Seemed straightforward enough as starters, but I didnt get that far. Figured a checkin here might be nice.
I have ambitions to help with @property, place-items, inline-size / more logical properties, :is() and :where() and maybe more?.. but think that starting with these would be helpful enough to start?

Wow! Thanks for the contribution, @argyleink! I haven't been devoting time to maintaining CSS3 like I should, so I really appreciate the effort. I'll need some time to reacquaint myself with the codebase and then I'll try to get this reviewed and merged soon.
Thanks again!

Yay!
I think once one or two of my submissions merge, getting me more acquainted with the codebase as well, I'll be able to help out pretty efficiently. Lookin forward to hackin together on this 👍🏻

cool, got min() working pretty well. i ended up piggy backing hard on calc() as they share a ton of usage and flexibility.
old and broken:

included in the crew:

I want to write more tests, as a way for me to get my needs out before I tinker more. How small or big of a PR should i work for? I feel like min, max and clamp are gonna touch a ton of files already.. thoughts?


Example project doesn't work

I've tried to run your example project from the master without success (I have followed all steps mentioned in your README).
Upon react-native run-ios I've got the following log:
.../react-native-onesignal/examples/RNOneSignal/ios/build/Build/Products/Debug-iphonesimulator/libRCTOneSignal.a(UNUserNotificationCenter+OneSignal.o)
ld: 97 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

** BUILD FAILED **

The following commands produced analyzer issues:
	Analyze Modules/RCTRedBox.m
(1 command with analyzer issues)

The following build commands failed:
	Ld build/Build/Products/Debug-iphonesimulator/RNOneSignal.app/RNOneSignal normal x86_64
(1 failure)

Installing build/Build/Products/Debug-iphonesimulator/RNOneSignal.app
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=22):
Failed to install the requested application
The bundle identifier of the application could not be determined.
Ensure that the application's Info.plist contains a value for CFBundleIdentifier.
Print: Entry, ":CFBundleIdentifier", Does Not Exist

Command failed: /usr/libexec/PlistBuddy -c Print:CFBundleIdentifier build/Build/Products/Debug-iphonesimulator/RNOneSignal.app/Info.plist
Print: Entry, ":CFBundleIdentifier", Does Not Exist

react-native-cli: 2.0.1
Actually,
I have issues with migration to the last version of react-native and react-native-onesignal in my project. To find out the root cause I've tried to build and run your example project to be sure I should find something in react-native but as you can see it doesn't work.

I haven't tried the example app, will do that tomorrow. But I also get the error message clang: error: linker command failed with exit code 1(use -v to see invocation).

@nonameolsson Have you tried the example app?

@bitsal No, I haven't yet. Actually I got the code to work in my app, so I forgot to do that. If I have time today I will try it.
Can't you get OneSignal to work in your app?

No, there is probably related issue https://github.com/facebook/react-native/issues/12952, that is still opened.
We've started working on a new react-native app and it seems to work there with new RN and OneSignal. We still working on it from time to time.

@bitsal Ok! I managed to upgrade from 0.42.0 to 0.43.3 and got OneSignal to work. I'm not sure how, because tried different things during two days =/

The example project is not upgraded yet to the latest RN version, i'll try to do so and we'll see if any problem comes up.
Please let me know if you managed to solve the issue.

The main issue is "Example project doesn't work" as my summary. I wouldn't close this issue until the example project works as expected at least with RN version that is configured now.

I've updated the example project and this error should no longer occur.
If anyone encounters this issue again please feel free to respond and I will reopen the issue.

Command failed: /usr/libexec/PlistBuddy -c Print:CFBundleIdentifier build/Build/Products/Debug-iphonesimulator/RNOneSignal.app/Info.plist
Print: Entry, ":CFBundleIdentifier", Does Not Exist

I am seeing this same Print: Entry, ":CFBundleIdentifier", Does Not Exist error on my application, not demo app, after following the setup steps https://documentation.onesignal.com/docs/react-native-sdk-setup


Need to add web scraping in client side to scrap youtube.com

Completed this feature on previous commit


Ui::dnd_drop_zone() swallows the inner Response

The signature is
pub fn dnd_drop_zone<Payload>(
        &mut self,
        frame: Frame,
        add_contents: impl FnOnce(&mut Ui),
    ) -> (Response, Option<Arc<Payload>>)
    where
        Payload: Any + Send + Sync;

Note that add_contents: impl FnOnce(&mut Ui) doesn't return a result. It's thus unclear how we're supposed to know what happened to the contents.
I can do this:
let (inner_response, response, payload) = {
    let mut inner_response = None;
    let (response, payload) =
        ui.dnd_drop_zone::<Payload>(Frame::default(), |ui| {
            inner_response =
                Some(ui.add(Slider::new(&mut value, Normal::range()).text("Label")));
        });
    (inner_response, response, payload)
};
if let Some(inner_response) = inner_response {
    if inner_response.changed() {
      // do something
    }
}

... but that's quite a bit more cumbersome than something like this:
let (response, payload) = {
    ui.dnd_drop_zone::<Payload>(Frame::default(), |ui| {
        ui.add(Slider::new(&mut value, Normal::range()).text("Label"))
    })
};
if let Some(response) = response.inner_response {
    if response.changed() {
        // do_something
    }
}

(That hypothetical code assumes that dnd_drop_zone uses something like an InnerResponse rather than a plain old Response.)
I hope there isn't anything obvious I'm missing here, but I looked at the dnd_drop_zone source, and it looks like there was no provision for add_contents's response to be preserved:
let mut frame = frame.begin(self);
add_contents(&mut frame.content_ui);
let response = frame.allocate_space(self);

I'll give it a look today.


How to bind the key on windows?

I just pushed an update of Proto REPL with new keybindings that should work for Linux, Windows, and OSX. Try the newest version (0.3.0) and take a look at the keybinding documentation https://github.com/jasongilman/proto-repl#keybindings-and-events


Thanks for the excellent integration

Dear bropat. I express my great gratitude to you for the excellent integration. You can't imagine what emotional joy 😁 I experienced today when I discovered that an update has been released and my camera has already been added to it: Hijacking Indoor Cam 2K Pan & Tilt T8410. Unfortunately, there is no other topic here besides God, and I created a topic for gratitude in the bug. I apologize.
I made a manual on how to implement everything
https://community.home-assistant.io/t/eufy-camera-integration/121758/1392?u=divanx10

@DivanX10 Thank you so much!


[Discuss] Allow "parent" links to contain expanded content.

https://trello.com/c/vm54jvVo/477-send-hard-coded-breadcrumbs-to-publishing-api is the underpinning need for this work or something similar to accommodate expanded links.
This change to the publishing api allows for expanded links to pass through to the content store which unblocks rendering breadcrumbs and other dependency related content.

/cc @rboulton - I'd be interested in what you think the appropriate key would be here now that we've changed the approach to serving breadcrumbs slightly, parent or all_parents given the conversion on this travel advice publisher PR?


Using PnP.PowerShell

Hello,
I downloaded the module, but when I try to connect, I got a warning that the module is being depreciated and should move to PnP.PowerShell. Is this possible to update the documentation on that?
Thanks
[Enter feedback here]

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

ID: 5e257652-6b19-287b-7f39-21c2a0909c09
Version Independent ID: 10c59abb-83b5-1ae4-2008-0a53e9ca952e
Content: PnP PowerShell Overview
Content Source: sharepoint/docs-conceptual/sharepoint-pnp/sharepoint-pnp-cmdlets.md
Service: sharepoint-powershell
Product: sharepoint-server-itpro
GitHub Login: @Techwriter40
Microsoft Alias: pamgreen

Hello!
how are you?
Even though the PnP documentation articles are on this repository, I need to kindly redirect you to open PnP cmdlets issues/pull requests here, in that repository, where a member of the PnP community will be able to correct the article/review your PR with your feedback.
The documentation for PnP cmdlets is then synced from that repo into this one.
Thank you!

@get-itips Thank you very much for the contribution and sharing this explanation. @nicoride93 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


fix typo

Coverage decreased (-1.8%) to 86.765% when pulling ef6b32d4d60d55618bc427863c28c765831fa7df on atefBB:patch-1 into 2c06059e53009e7dd1a3aa0f9e5f20833ae6a76b on dwmkerr:master.

Thanks!


AssertationError if no initial call of javascript method in javascript

I just checked your React tutorial out. I tried to add a new javascript method, which will be called from Python, but on eel.init  python always throws an AssertationError.
AssertionError: eel.expose() call contains '(' or '='. But when I call the JavaScript method in the same JavaScript file first- the code works as expected.
function getClipboard(  ) {
}
window.eel.expose( getClipboard, 'get_clipboard' )

// and without the following initial call it leads to the AssertationError.
getClipboard( );

So the work around is quite easy, but I don't think it's the desired behavior.
May I overlooked something.
best regards.

Hi @slo-ge. Could you please provide a working example and a not-working example? I'm not following exactly what the problem is for now, but from the error message it looks partly to do with one of Eel's dark secrets in how it parses the functions exposed on the JavaScript side.

I'm having a similar issue. When testing out your example React App Create-React-App .
Everything works as expected when running in develop mode out of the "src" directory. However, when running out of the "build" directory I get this error:
Traceback (most recent call last):
File "/Applications/PyCharm.app/Contents/plugins/python/helpers/pydev/pydevd.py", line 1438, in _exec
pydev_imports.execfile(file, globals, locals)  # execute the script
File "/Users/rpm886/Documents/ZoomRoomsProjects/zoom_test/eel_CRA.py", line 120, in
start_eel(develop=len(sys.argv) == 2)
File "/Users/rpm886/Documents/ZoomRoomsProjects/zoom_test/eel_CRA.py", line 79, in start_eel
eel.init(directory,['.tsx', '.ts', '.jsx', '.js', '.html'])
File "/Users/rpm886/Documents/ZoomRoomsProjects/zoom_test/venv/lib/python2.7/site-packages/eel/init.py", line 106, in init
assert rgx.findall(r'[(=]', expose_call) == [], msg
AssertionError: eel.expose() call contains '(' or '='


Adding Indy RHDG8 Maven Repository

@pradeshm I'm closing this. I'll create a maven settings.xml downstream with the necessary repos. is that ok for you?

@pradeshm I'm closing this. I'll create a maven settings.xml downstream with the necessary repos. is that ok for you?

Hello @pruivo yes that make sense,Thank you!


Add gitpod badge to readme

Please add the following code snippet to the bottom of the readme file
[![Open in Gitpod](https://gitpod.io/button/open-in-gitpod.svg)](https://gitpod.io/#https://github.com/AccessibleForAll/AccessibleWebDev)

Hey Emma, I would love to work on this issue. Assign it to me

Sure!

Thank you

as @MrKrishnaAgarwal already resolved this issue. So, I don't have to create a PR now.

as @MrKrishnaAgarwal already resolved this issue. So, I don't have to create a PR now.

Yes, I have resolved #20 and #15 both

@Ashish-khanagwal since you are assigned to this issue I will merge your pull request over someone else’s as I don’t feel it’s good etiquette that people take assigned issues from others, especially during hacktoberfest. So please go ahead and make your PR.

@Ashish-khanagwal since you are assigned to this issue I will merge your pull request over someone else’s as I don’t feel it’s good etiquette that people take assigned issues from others, especially during hacktoberfest. So please go ahead and make your PR.

Should I close my PR then?

@Ashish-khanagwal since you are assigned to this issue I will merge your pull request over someone else’s as I don’t feel it’s good etiquette that people take assigned issues from others, especially during hacktoberfest. So please go ahead and make your PR.

Should I close my PR then?

Yes, please only makePRs for issues you are assigned to. If you see an issue that’s already assigned but seems abandoned after a couple of weeks feel free to ask about having it reassigned. I’m also hoping to add more issues later this weekend

@Ashish-khanagwal since you are assigned to this issue I will merge your pull request over someone else’s as I don’t feel it’s good etiquette that people take assigned issues from others, especially during hacktoberfest. So please go ahead and make your PR.

Should I close my PR then?

Yes, please only make PRs for issues you are assigned to. If you see an issue that’s already assigned but seems abandoned after a couple of weeks feel free to ask about having it reassigned. I’m also hoping to add more issues later this weekend

yes sure, please add them

okay, then I'll make PR on this issue


Uncaught SyntaxError: ambiguous indirect export: default index.esm.js:1:7

Having an issue with a getting it to work.. I've followed the installation and getting started documentation to the letter, but when I run my project I'm met with the error labeled in the title, in the console.. It's related to the following dependency..
http://localhost:3000/node_modules/@googlemaps/markerclusterer/dist/index.esm.js?v=e9585d40
Seems like perhaps #33 is sort of in the same vein.
I'd be happy to provide any further details that you might need to troubleshoot, please just let me know what you'd need. Would really like to use this as there's limited options right now for making working with Google Maps and Vue 3 easier.

Hello! Sorry for the delay on answering.
Isn't this related to #4 somehow? Did you made this configuration while using our library?


refactor: migrate from Node.js streams to Web Streams API

Continuing the work of making node-zwave-js more portable, this PR refactors the usage of streams to be compatible with the Web Streams API, rather than Node.js streams.

@zwave-js-bot automerge


Add github configuration

Copied (and modified appropriately) the github actions from ssh-executor. And added a .gitignore file.

closing this as we are going to redesign CI/CD in AgnostiqHQ/covalent#448


templates: update dashboards to include tenant

Add a tenant variable to the composer dashboard, with the option
to select multiple tenants. Add tenant filter to queries accordingly.
link to dashboard: https://grafana.stage.devshift.net/d/JLSCLdqnk/image-builder-worker-with-tenant?orgId=1
This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

Sorry been meaning to get back to this, I had a look this morning and changing from tenant=\"$tentant\" to tenant=~\"$tentant\" should do the trick.

dashboards updated, codebase/tests not affect. admin merging


Replace outdated GH action with new gradle/actions/setup-gradle

Replace outdated GH action with new gradle/actions/setup-gradle.

The failing InterpreterTest seemed to be caused by too many non-deterministic retries with a long delay which then led to exceeding the test timeout. I was not able to reproduce the problem locally.


CI wrongly considers noxfile.py is designed for all sessions to pass

The sessions in noxfile.py were not originally designed to be evaluated in terms of "passing".
Instead, by analysing the README we can see that the original intent of the noxfile.py was to generate an informative table that will include situations where combining methods will pass and other where it will not pass... Making all the sessions in nox pass kind of defeat this purpose, isn't it?
Probably a clearer approach (however more costly in terms of computing resources and time), would be to run all the sessions in noxfile.py, build the new table.md using the reporting instructions described in the README.md and then fail if there is any uncommitted alteration in that file specifically.

Sounds good! I'll take a look at that.

This might be useful in the context of running the whole nox matrix to generate the complete compatibility report:
https://github.com/actions/setup-python/releases/tag/v4.4.0

fail if there is any uncommitted alteration in that file specifically

One concern I have is that there might be different results across different Python versions (that's what this repository is intended to determine, right), for which the user can't possibly determine the compatibility, thus running the whole workflow with different Python versions on a pull request will most probably result in a CI failure?

Hi @chrysle, I believe the best is to run all the Python versions from a since CI job.
Probably https://github.com/actions/setup-python/releases/tag/v4.4.0 can be used for that.
Otherwise, it would be necessary to find out a way of integrating the report from different jobs, before converting them to a table. Individually running nox for a single Python version is not really useful to determine if the tests fail or not.
This happens because the noxfile.py was designed to run all the sessions (with the different Python versions) at once and only at the end of all sessions, generate a report file (which can then be processed by the script that creates the table).
I don't think this fits in the idea of mapping each Python version to a different CI job (running in a different container/VM).

I believe the best is to run all the Python versions from a since CI job

I agree with that. But if I've understood correctly, the different table reports for the Python versions will be combined into one compatibility matrix. And if running the nox sessions results with, for example, 3.10, in a different table than with 3.8,
which the user employed to update the table, won't the CI process on a PR fail?

Umm, I don't know if I understand the problem.
Let's suppose that in the CI, we use setup-python to install in the same CI job worker, Python versions: 3.8, 3.10 and 3.12. To ensure noxfile.py works well, then we have to set the environment variable the environment variable USE_PYTHON_VERSIONS="3.8 3.10 3.12"="3.8 3.10 3.12"before runningnox. This way it should not matter (🤞) which we use to install nox, for each different section nox` should take care of spawning the correct virtual environments and combining then consistently when reporting.
Now if contributors want to update the table.md file, in their personal computer they will also need to install the same Python versions used in the CI and set the environment variable USE_PYTHON_VERSIONS="3.8 3.10 3.12" before using the procedure described in the README.md.
This is how I suppose it would work, but the best would be actually check that against the github CI. Maybe other unforeseen errors will appear...
If that is too complicated, we can also drop all togheter this idea of checking if table.md is updated accordingly... Just pick the combinations of Python versions and nox sessions that we know should work and only run those...

Thanks for your patience, I suppose I had a little short-circuit here. I'll look into implementing that soon.

@chrysle this is a quick test I did to see if a single job can run multiple nox sessions:
https://github.com/abravalheri/experiment-setup-python-multiple-versions/actions/runs/7891544198/job/21536102814
I used a .python-version to maintain a single source of truth for both Github Actions and nox.

Yeap, there should be no problem with that.


IIS Web App Manage Task - HTTP error code 405 on all PUT & DELETE request post release.

VSTS IIS Web App Manage Task
Issue Description
HTTP error code 405 on all PUT & DELETE request post release.
Per the docs - "The task updates websites, webapps and application pools but does not install or configure IIS web server on the machines."
However the first option is "Enable IIS" which appears runs the cmndlet Add-WindowsFeature web-server -includeallsubfeature cmd which I'm not familiar with.
After release completes using this check box, even if IIS was enabled on the server all PUT AND DELETE requests trigger a 405 result. For every app on the IIS server, not just the targeted site. Also note that WebDAV Module was set to "disabled" for the affected site.

@JCanlett If you enable "Enable IIS" option, then all the sub-features of "Web Server (IIS)" feature will get enabled, including WebDAV publishing. So please disable WebDAV manually and re-deploy you website with "Enable IIS" option disabled. See if that works.

@rajatagrawal-dev  Yes that is exactly what we had to do. Just a case of features going faster than documentation I guess. Also note that while the installation is "seemless" the uninstall is not and requires server restart. So this was a bit of an issue for our team.


refactor(ts client): remove url object usage in fetch.ts

This PR removes the use of URL object from the typscript client to avoid compatibility issues with react-native
https://github.com/electric-sql/electric/issues/2072 and adds a generic check with urlite

Closing as discussed in the issue https://github.com/electric-sql/electric/issues/2072#issuecomment-2524844376


Should we rename to GeocodeConverter?

My intentions are to support multiple OpenStreetMap-based providers, not only nominatim in the long run. Would it make sense to rename it then?

Yes. Any proposals?
GraphHopperGeocoderConverter?

yes something like this or instead camel casing using dashes graphhopper-geocoder-converter as this will be cloned into files and there I don't like upper cases due to windows etc

Ok, I renamed the repository.
Github recommends to run
git remote set-url origin new_url


add event listener for cancel lecture comments button

This fixes #514. I added a missing event listener that for the cancel lecture comments button (it is done exactly the same way for the cancel buttons in the other accordion folds).

Remember to "Squash and merge"


Added the bias-variance trade-off question and answer

Thank you a lot for your second contribution!

I'm preparing for data science interviews so it helps to write down answers to some questions


How can I redirect to a page by clicking on the ionic push notifications while the mobile application is in the background?

This code works when the application is in the foreground, but not when it is in the background. What can I do to redirect me to a specific page of the application in ionic 3 for android and ios.
@Injectable()
export class NotifyService {
constructor(private push: Push, public alertCtrl: AlertController, public app: App) {

}

initPushNotification(){
    // to initialize push notifications
    const options: PushOptions = {
        android: {
            senderID:'your_id_proyect',
            sound: true,
            forceShow: true,
            vibrate: true,
            icon: 'icon',
            iconColor: 'color'
        },
        ios: {
            alert: 'true',
            badge: true,
            sound: 'false'
        },
        windows: {},
        browser: {
            pushServiceURL: 'http://push.api.phonegap.com/v1/push'
        }
    };
    const pushObject: PushObject = this.push.init(options);

    pushObject.on('notification').subscribe((notification: any) =>
        {

            if(notification.message=='Ha realizado una solicitud de permiso'){
                this.app.getActiveNavs()[0].push(RequestPage);
            }

            if(notification.message=='Aceptó la solicitud de permiso'){

                this.app.getActiveNavs()[0].push(RequestPage);
            }

            if(notification.message=='Rechazo la solicitud de permiso'){
               /* var nav = this.app.getActiveNav();
                nav.setRoot(RejectedPage); */
                this.app.getActiveNavs()[0].push(RejectedPage);
            }

        }
    );
    pushObject.on('error').subscribe(error => console.error('Error with Push plugin', error));
}

}

@joelgo if you want it to happen when the app is in the background then you need to send a background notification:
https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/PAYLOAD.md#background-notifications
https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/PAYLOAD.md#background-notifications-1

I added it as the tutorial says but likewise the application when it is in the background after receiving the notification and clicking on it just opens the app but does not execute on (Notification); where I have a function that redirects me to a specific page.
tokens_and.append(item['token'])
extra = {'body': mensaje, 'title': name, 'color': 'Blue', 'icon': 'drawable-ldpi-icon.png'}
dataRaw_android = {
"notification": extra,
"registration_ids": tokens_and,
"content_available": "1"
}

@joelgo Having the same problem. Did you find a solution?

[image: image.png]
Of course, yes, for android apply the following and it works normal.
El jue., 20 sept. 2018 a las 14:32, 2gen<EMAIL_ADDRESS>escribió:

@joelgo https://github.com/joelgo Having the same problem. Did you find
a solution?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/phonegap/phonegap-plugin-push/issues/2548#issuecomment-423305362,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK_PAs_igeAeP6Y0fjNUaTwElOdevdPdks5uc-1EgaJpZM4WnaOJ
.


custom text TextInputWidget

hi! i was wondering if it's possible to custom style the text input fields. if so, how?

Hello,
you can use widgetStyles prop in the TextInputWidget
widgetStyles={{
textInput: {
// your style
}
}}
or you can set a global style using formStyles prop in the GiftedForm
widgetStyles={{
TextInputWidget: {
textInput: {
// your style
}
}
}}

thanx, for the info. i initially tried that , but it seems not to work for me. i tried changing the background color and height, but nothing happened. my react-native version: 0.19.0
any ideas?

please try
widgetStyles={{
textInputInline: {
// your style
}
}}

sweet! that was it—thanx!

@FaridSafi  Thanks for the amazing form widget. I just want to know how can I style country widget touchable opacity? I want to add a 'tick' when a country is selected. Just like it happens in selection of gender in Gender widget. Thanks


Deleting project and datasets doesn't remove it from the UI

You can't click on the project, but it is still there.

I suspect it is deleting the project from HDFS, but there is definitely still the row for the project left in the project table.

Actually, this could be the problem:
the foreign key in projects is disabled.
FOREIGN KEY (inode_pid,inode_name) REFERENCES hops.hdfs_inodes(parent_id,name) ON DELETE CASCADE ON UPDATE NO ACTION
I still don't know what the fix is.

I just tried replicating it and it works for me. Projects and datasets are removed from the UI, hdfs and the database. Not sure what the difference in our configurations might be.

Fixed by fixing foreign keys.


Add magic CL script.

This adds a new script written in Common Lisp for generating fonts. Includes help, format options, and a custom language for defining your permutations.
It works by using the Hack project as a submodule. See the updated README.md for examples and usage.
It has a few features to help unwitting users:

Fails nicely for bad specifications (eg. two different variants of zero).
Has a help that shows up.
Just specify which variants you want from the list.

Addresses https://github.com/source-foundry/alt-hack/issues/43

This has been here awhile.
Is there anything I can do to help it along? Perhaps it must be written in a more ubiquitous language like Perl to be accepted?

This is great and an approach like this is definitely desirable but I won't be able to review or maintain Lisp sources. I hate to have you refactor all of this work.  Will you maintain this in a downstream and we can point to it from the README?

This is great and an approach like this is definitely desirable but I won't be able to review or maintain Lisp sources. I hate to have you refactor all of this work.

I chose lisp because it is a powerful language for solving complex problems. It would have been really hard to do it in a shell script without making a prototype like this first.

Will you maintain this in a downstream and we can point to it from the README?

Yes, I will take care of my baby.

Don't take this the wrong way.  I trust your opinion that this may be a good approach.  I don't know Lisp.  It is really as simple as that.  :)

I just saw the pull request from @barrycohen, and I think that is going to be the way forward.


Fix typo

a -> an

@changeworld : Thanks for your contribution! The author, @dan-wesley, has been notified to review your proposed change.

Verified editorial correction.


xlights: 2024.09 -> 2024.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/smeighan/xLights/releases.
meta.description for xlights is: xLights is a sequencer for Lights. xLights has usb and E1.31 drivers. You can create sequences in this object oriented program. You can create playlists, schedule them, test your hardware, convert between different sequencers.
meta.homepage for xlights is: https://xlights.org
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2024.10 in filename of file in /nix/store/k598kbnirnpr4kp1dp5lw36ls46dpy6h-xlights-2024.10

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

xlights

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/k598kbnirnpr4kp1dp5lw36ls46dpy6h-xlights-2024.10 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A xlights https://github.com/r-ryantm/nixpkgs/archive/24c1185b547cad889e6d2efbe09db07f6d2b4792.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/24c1185b547cad889e6d2efbe09db07f6d2b4792#xlights

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/k598kbnirnpr4kp1dp5lw36ls46dpy6h-xlights-2024.10
ls -la /nix/store/k598kbnirnpr4kp1dp5lw36ls46dpy6h-xlights-2024.10/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

xlights

Maintainer pings
cc @kashw2 for testing.

@NixOS/nixpkgs-merge-bot merge


jQuery

First of all thank you for the beautiful minimalist theme.
I'm having a few issues that I believe are caused by jQuery not loading correctly. Random resources throwing 500 errors instead of getting returned. The favicon is 404ing. The blog post content is being rendered three times. FA icons aren't showing properly. Etc.
I've tried loading it in head.html.twig and in the Custom JS plugin, neither of which should have been necessary because Grav itself should be getting it but apparently something is preventing it from getting loaded before all the other JS.

I'm not sure what was causing the issues but I fixed them by adding security headers.
The favicon problem was fixed by replacing the twig {{ variables }} in user/themes/canvas/templates/partials/head/favicon.html.twig to relative links. Not sure what the variable was being set as elsewhere.


Remove fade effect

Hello how can I remove fade effect when a page load ?

Hi
You can remove the loader div or else remove it from the script.

thanks but chere in the code ?


Cow: Watch (?-i:qAx1muGCWJg)(?# youtu.be)

Cow requests the watch of the watch_keyword (?-i:qAx1muGCWJg)(?# youtu.be). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:qAx1muGCWJg)(?# youtu.be) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Added Sum,NanSum, CumSum and NanCumSum

PR Summary
Added Sum, NanSum, CumSum and NanCumSum to both APyFixedArray and APyFloatArray. Related to #303.
PR Checklist

[N/A] "closes #0000" is in the body of the PR description to link the related issue
[x] new and changed code is tested
[ ] relevant changes added to CHANGELOG.md
[x] new functionality is documented

If you need help to rebase this on top of main and resolve the conflicts, come talk to me in my office tomorrow.


responsive members page with dummy data

What does this PR do?

add fontawesome and material design icons
design responsive members page with dummy data

Description of Task to be completed?

responsive members page

How should this be manually tested?
npm run start:dev or yarn start:dev
Screenshots (if appropriate)


Pods restart because their log volume exceeds its limit

Pods restart because their log volume exceeds its limit:
Usage of EmptyDir volume "log" exceeds the limit "11Mi".

This was observed in an OpenShift cluster for ZooKeeper 3.8.0 and HBase.

The issue can be reproduced on our OpenShift cluster.
The problem is that the disk usage per log file is not just required blocks * block size. For instance, a zookeeper.log4j.xml with 4,127,151 bytes occupied 4,032 blocks. Then log entries were written and the actual file size increased to 4,132,477 bytes which occupy suddenly 8,128 blocks.  The number of reserved blocks can also decrease. For instance, the zookeeper.log4j.xml.1 contains 5,243,985 bytes and occupies only 5,124 blocks.


remix.config.js devServerPort setting not being observed and there is no configuration to alter public URL for proxied socket connections.

Have you experienced this bug with the latest version of the template?
yes
Steps to Reproduce
This is not a duplicate of https://github.com/remix-run/remix/issues/2620 as it contains some nuances relating to devServerPorts vs socket ports.
ive installed and run indie project and edited remix.config.js  to set devServerPort according to instructions here
/**

@type {import('@remix-run/dev').AppConfig}
/
module.exports = {
cacheDirectory: "./node_modules/.cache/remix",
ignoredRouteFiles: [".", "/*.css", "/*.test.{js,jsx,ts,tsx}"],
devServerPort: 3301
};

The default port is 8002 and when I run in the brower i see:
(index):26 WebSocket connection to 'wss://remix1.syntapse.co.uk:8002/socket' failed:

which appears to imply that devServerPort is being ignored.
There is another issue here
Expected Behavior

devServerPort: 3301 should result in the browser attempt to connect to websocket at port 3301 and not the default 8002.
there is no way to separate the devServerPort from the devServer socket port though I believe they are not always the same thing.
Port specs on reverse proxied connections dont make sense and there is no way to override

'wss://remix1.syntapse.co.uk:8002/socket'

with a client URL similar to:
'wss://remix1.syntapse.co.uk/my-overriden-url-spec-or-nothing-at-all/socket
In previous versions of CRA and Next I have been using formats like this to access socket ports for dev purposes and in most recent versions of each there is no need because the main URL and sockets work off the same port.  This I believe would be an optimium solution also for remix to enable a more seamless post-installation developer experience.
Actual Behavior

devServerPort: 3301 is being ignored and the default port is being used.
the default socket url format is inappropriate for proxied connections and it would advantageous to have some way to remove the port spec from the URL to allow nginx or other proxied router to route based on custom paths. create-react app offers  a variety of environment variables to achieve this.

part of the problem is fixed by use of REMIX_DEV_SERVER_WS_PORT in .env.
Raising separate issue for the other part...


[device_info] More clearly define scope

There has been interest in adding desktop support to device_info (see https://github.com/flutter/flutter/issues/39674, https://github.com/flutter/plugins/pull/2942#issuecomment-680119751); it's not really clear to me how we would evaluate whether to accept or reject any given information being added there; there are obvious easy things (e.g., exact OS version), and all kinds of things that obviously wouldn't belong, but a huge grey area that could arguably be "device" information, but would potentially be too niche. I think we should more clearly articulate the intended scope of device_info somewhere in the README or docs.
It would also be nice to take a step back as we extend it to new platforms and consider extracting a base class for the info structure. There are many reasonable things that could be included that would be common across at least all of desktop, and maybe all devices, and having a few common abstractions for those things would be good.

// Subscribing, this is very relevant for any web implementation later.

I think we should use neofetch data model for defining base class for the info structure. Atleast it would work on all linux distros and macos.

The screenshots of the default information display are a clear illustration of the problem, IMO. Taking a few examples:

OS Clearly yes
Resolution It's a property of the device, but it's not included on iOS or Android currently. (I assume that's because it's available other ways, as it will be on desktop via screen APIs in the framework, but that's a guess on my part; we should have an articulation of why it's not there.)
Memory Similar to resolution, except that I have no idea why it's not there.
Uptime Not really a property of the device, and utility seems niche; I would personally say no
Terminal Font Clearly no

I think, we just need to specify the base class for all desktop platforms (linux, macos and windows) all together and not include Android and iOS in it.

I think you misunderstood my references to iOS and Android in my last comment. I'm not saying it's technically difficult to include things like resolution and memory because they aren't there for iOS and Android, I'm saying that we need clearly articulated scope criteria for what should be included in this plugin, in general (and that whatever those criteria are, they should almost certainly give the same answer for whether or not to include very general concepts like memory and screen resolution for any platform).

I am sorry for that misunderstanding. I've made a Google docs for it. Here's the link. This will help in maintaining the information. I have added some things for linux and windows. Please review it. (You can edit if you want to make some changes in it.)

@yash1200 Articulating the desired scope of the plugin at a higher level is a precondition of reviewing specific proposals for what fields it should have on desktop platforms; thus this issue.

/cc @yjbanov It looks like you created this plugin; are you aware of any discussions about the desired scope?

@matthew-carroll you're listed as the code owner for this plugin; thoughts on the above?

Please read my comment above; this issue is not about discussing technical difficulty, it's about clearly defining and articulating the intended scope of the plugin.

Closing as device_info is deprecated in favor of device_info_plus.


Fix re-require issues in tests

Description
Fix re-require issues for mock-require.
Fixes # (issue)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Replaced with https://github.com/adobe/asset-compute-sdk/pull/160


Not working - keeps "loading"

I can see the JIRA tickets panel in storybook but it doesnt load data for a ticket and gives this error in console:

I have this for endpoint, not sure if that is right:
STORYBOOK_JIRA_API_ENDPOINT=https://studio-henk.atlassian.net/rest/api/latest/issue/

This PR should fix this error https://github.com/lennartdeknikker/storybook-addon-jira/pull/5 , I think we should wait for the release.
Just in case, this solution worked for me meanwhile: https://github.com/lennartdeknikker/storybook-addon-jira/issues/4#issuecomment-1379073705


Auto-generating Django migrations using Django's own machinery

Up for discussion! I think I prefer the idea of having Django more autonomous, but:

Django's code is not very modular, so I can have minimal code
This means the corresponding code is not very readable
And I have to reimplement a few details that are already implemented in Django's own code but I couldn't reuse.
That being said, the code here is more testable than the script it replaces (well... differently testable. The script was not easily testable because it was not part of the python package. That version is more easily testable in this way, but we'll need more integration tests, vs unit tests)

It's kind of funny because Django here does a lot of very smart things, but they have so many unrelated code in the same functions, that there's always one little detail that makes many things not reusable. That being said, this code in django was most probably not supposed to be hacked, so I expected it to be hard.
Hmm. I'm wondering about grouping all the missing SQL migrations in a single django migrationwith multiple operations. This would simplify the code a lot !

Successful PR Checklist:

[ ] Tests

[ ] (not applicable?)

[x] Documentation

[ ] (not applicable?)

[ ] Had a good time contributing? < I've known better
[ ] (Maintainers: add PR labels)

@elemoine Ready for review !

@ewjoachim you may want to add PR labels.


Kobweb Plugin: Have a way to not generate font awesome  in index.html

Right now we're always generating the link to font awesome icons, which might be expensive if you have an app that isn't using them.
Instead, there should be a way for the user to indicate they don't want it. Maybe just a kobweb.conf.yaml setting? Or we can figure it out if they don't have a dependency on the font awesome icon kobweb artifact?

This bug should let users add any number of header tags (or should we limit it to scripts and stylesheets?) Look at highlight.js, which requires added a handful of different tags.


post-workshop survey

OBIS has some?

carpentries has some

https://docs.carpentries.org/topic_folders/assessment/assessment.html#post-workshop-survey

OBIS has some?

Hi! As far as I know OBIS doesn't have a specific/standardized pre-post surveys for courses/workshops. In general, most of OBIS courses are delivered through the Ocean Teacher Global Academy and they are the one who send the surveys to the participants. I could try to get one example for us. However it would be very helpful to create a survey that OBIS could adopt for future "internal" workshops (not the ones delivered by the OTGA).

Survey draft here.
I want to remove some of the carpentries questions. Maybe we can talk through that briefly in the run through today.

I'd like to leave this open until we add a link to the survey here:
https://github.com/ioos/bio_mobilization_workshop/blob/gh-pages/_includes/custom-schedule.html


duplicate symbols for architecture x86_64 when using --no-package-directories

Using a simple project just one file:
package com.example;

public class SharedClass {

  public static int add(int a, int b) {
    return a+b;
  }

}

build.gradle:
// The 'java' plugin must be applied before the 'j2objc' plugin
apply plugin: 'java'
apply plugin: 'eclipse'
apply plugin: 'com.github.j2objccontrib.j2objcgradle'

sourceCompatibility = 1.7
version = '1.0'

buildscript {
  repositories {
      maven {
          url "https://plugins.gradle.org/m2/"
      }
  }
  dependencies {
      // Current Version: https://plugins.gradle.org/plugin/com.github.j2objccontrib.j2objcgradle
//      classpath "gradle.plugin.com.github.j2objccontrib.j2objcgradle:j2objc-gradle:0.2.2-alpha"
    classpath files('/Users/mg/Downloads/j2objc-gradle/build/libs/j2objc-gradle-0.2.2-alpha.jar')
  }
}

repositories {
    mavenCentral()
    flatDir {
      dirs 'lib'
    }
}

dependencies {
}

// Plugin settings:
j2objcConfig {
    xcodeProjectDir "${projectDir}/../ios"   // Xcode workspace directory
    xcodeTarget "ios"                        // Xcode application target name
    // More: https://github.com/j2objc-contrib/j2objc-gradle/blob/master/src/main/groovy/com/github/j2objccontrib/j2objcgradle/J2objcPluginExtension.groovy#L25

    translateFlags "--no-package-directories"
}

Xcode compile error:

@confile  - i cannot help much with a pod-generated file, but I see you have a -lPods-ios-j2objc-shared argument to the linked.  that is bug #128 and causes this issue as well.  we'd appreciate your help in fixing 128 - it'll fix this too =)

dupe of #128

@advayDev1 I cannot fix anything until I get my project running with the new plugin.

My point is that this bug is because of the pod task.  I think you are the local expert on that, so helping in #128 is the only way to fix your issue here.


Duration texts on the Progress Bar don't scale based on device text scaling factor

Hello,
The progress and total duration of ProgressBar don't change their text scales based on the device text scale factor.
Is it possible to add text scaling?

Yes, this is a bug that should be fixed. I'm not sure when I'll get to it, but hopefully before too long. PRs welcome.

This is now fixed in version 2.0.1. Thank you for letting me know about the bug.
Before:

After:


Adversarially training

In the paper you mentioned about experiment of training Adversarially and you said
"it is challenging because it is impossible to backpropagate through sampling from the generator"
could you please elaborate more on this issue and whether you have found a solution?

Not the author, but this might be a better question for StackOverflow or the like. Essentially, gradient descent cannot be applied when there are stochastic nodes in the computation graph (what is the gradient through the sampling step? this is equivalent to asking what is the derivative of a random variable with respect to the source of the randomness, which can't be computed [in general]). I think a more common way this problem manifests itself is in the context of the reparameterization trick, so it might help to look into that too.


AttributeError: 'BodyPart' object has no attribute 'get_position'

What I Did
I tried running real-robots-demo at the terminal, as well as the python code in the readme, but I seem to get the same error. See below. I tested it directly, and the important seems to execute fine. Any thoughts? Thanks!
Output
The default interactive shell is now zsh.
To update your account to use zsh, please run chsh -s /bin/zsh.
For more details, please visit https://support.apple.com/kb/HT208050.
  Call .resolve and .require separately.
result = entry_point.load(False)
WARN: gym.spaces.Box autodetected dtype as <class 'numpy.float32'>. Please provide explicit dtype.
WARN: gym.spaces.Box autodetected dtype as <class 'numpy.float32'>. Please provide explicit dtype.
/Users/christopherkeown/anaconda3/lib/python3.6/site-packages/pybullet_envs/data/kuka_gripper_description/urdf/kuka_gripper.urdf
Traceback (most recent call last):
File "/Users/christopherkeown/anaconda3/bin/real-robots-demo", line 11, in
sys.exit(demo())
File "/Users/christopherkeown/anaconda3/lib/python3.6/site-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/Users/christopherkeown/anaconda3/lib/python3.6/site-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/Users/christopherkeown/anaconda3/lib/python3.6/site-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/Users/christopherkeown/anaconda3/lib/python3.6/site-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/Users/christopherkeown/anaconda3/lib/python3.6/site-packages/real_robots/cli.py", line 59, in demo
run_episode(env, pi)
File "/Users/christopherkeown/anaconda3/lib/python3.6/site-packages/real_robots/cli.py", line 28, in run_episode
observation = env.reset()
File "/Users/christopherkeown/anaconda3/lib/python3.6/site-packages/real_robots/envs/env.py", line 164, in reset
return self.get_observation()
File "/Users/christopherkeown/anaconda3/lib/python3.6/site-packages/real_robots/envs/env.py", line 210, in get_observation
retina = self.get_retina()
File "/Users/christopherkeown/anaconda3/lib/python3.6/site-packages/real_robots/envs/env.py", line 193, in get_retina
self.robot.object_bodies["table"]
AttributeError: 'BodyPart' object has no attribute 'get_position'

Dear @ckeown ,
from the error message I would guess it is a problem with the pybullet version you are using.
Are you running real-robots-demo in an Anaconda virtual environment?
If you create the virtual environment as suggested here (using this yml file) it should work.
The suggested pybullet version is: 2.5.3


feat: align section title and link to adopted trees

align section title and link to adopted trees

@aeschi  yes makes sense. I added text-end to it (which works for all screens as far as I can tell). Let me know what you think :)


PRODUCTION build script not working

Please fix the production build script as deploying a hotfix is impossible with a broken build script

Dominic ,I can not understand, i should use cross env for build? or no


Create an end-to-end tutorial for pipeline building

Summary
To convey the value of LineaPy's pipeline support, we want to have a more comprehensive end-to-end tutorial on the topic, with more relatable examples that can intrigue the target audience upfront.
Changes

[x] Create a new notebook tutorial on pipeline building

Testing
CI will test the notebook.
Ticket(s)
None.

This PR merge does not update existing bitly link to the pipeline demo notebook (https://bit.ly/3SJewuO) as it is pointing to v0.2.x branch (the motivation for this needs is to be clarified with @lionsardesai). Hence, either of the following should be done:

Update the bitly link to make it point to main
Merge main into v0.2.x branch (or cherry-pick this PR's commits)


WIP Edit read.me

/azp run Mac Public BuildXL PR Validation

Just blocking accidental merge to master

Just blocking accidental merge to master

WIP in teh PR title does the blocking automatically. Andy taught me this trick

/azp run Mac Public BuildXL PR Validation


python312Packages.lupa: 2.0 -> 2.1

Changelog: https://github.com/scoder/lupa/blob/lupa-2.1/CHANGES.rst
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 298872 run on x86_64-linux 1

4 packages failed to build:

python312Packages.optuna
python312Packages.optuna.dist
python312Packages.pyannote-pipeline
python312Packages.pyannote-pipeline.dist

10 packages built:

python311Packages.lupa
python311Packages.lupa.dist
python311Packages.optuna
python311Packages.optuna.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.pyannote-pipeline
python311Packages.pyannote-pipeline.dist
python312Packages.lupa
python312Packages.lupa.dist


Support to run with snowflake MFA-enabled user

Is your feature request related to a problem? Please describe.
In current snowflake integration, it only supports to login by basic authentication (user/password). If the user enable the MFA, there is no way to use piperider
Describe the solution you'd like
Reference of snowflake profile in dbt

Open an external browser to authenticate
Support Key Pair Authentication

Describe alternatives you've considered
N/A
Additional context
N/A

Hi @DaveFlynn  @jonycfu,
wonder you have any idea on the above comment ☝ ?
I'm keen to contribute to this feature and has implemented locally and worked well.

Hi @chenxuanrong
Thanks for taking the time to contribute!
To submit your contribution please fork the repo, make your changes on a branch and submit a pull request which will be reviewed and merged.
Dave


added Nuestra Señora de Lourdes' domain

@MyersKing I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host nsdl.edu.pe not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Add UFP themes for "Dice so Nice!"

This patch introduces color and icon themes for Dice so Nice!. The icons are derived directly from the SVG present in the icons folder. This currently only contains UFP dice configurations.
(partially) Fixes #47 and #11

Currently moving this issue over to testing, thank you for doing this work!


Specify more temp files

Rotary generates a lot of files that seem extraneous. We should add more of these to the outputs of rotary rules and make them temp().

@jmtsuji I used snakemake's file graph command to make this image. Temped files are highlighted in red, and my suggestions are highlighted in green. Thoughts?

I was also thing we could temp all the polishing steps. There's a few rules here where we kept output diaganotic txt output files. I think we could add a rule to compile these to TSVs and then we would be able to temp out their entire directories.

Here are the folder sizes of the output:
25M     ./assembly/end_repair
14M     ./assembly/flye/20-repeat
5.5M    ./assembly/flye/00-assembly
17M     ./assembly/flye/30-contigger
13M     ./assembly/flye/40-polishing
5.7M    ./assembly/flye/10-consensus
66M     ./assembly/flye
90M     ./assembly
189M    ./qc/long
143M    ./qc/short
331M    ./qc
9.5M    ./polish/polypolish/input
239M    ./polish/polypolish
16M     ./polish/polca
422M    ./polish/medaka
16M     ./polish/cov_filter
691M    ./polish
2.3M    ./annotation/checkm/protein_files
2.7M    ./annotation/checkm
3.1M    ./annotation/eggnog
310M    ./annotation/coverage
6.7M    ./annotation/dfast/ddbj
6.9M    ./annotation/dfast/genbank
73M     ./annotation/dfast
388M    ./annotation
5.6M    ./circularize/combine
9.5M    ./circularize/polypolish/input
239M    ./circularize/polypolish
12M     ./circularize/circlator
5.6M    ./circularize/filter
261M    ./circularize

Here are the file sizes:
25M     ./assembly/end_repair
14M     ./assembly/flye/20-repeat
5.5M    ./assembly/flye/00-assembly
17M     ./assembly/flye/30-contigger
13M     ./assembly/flye/40-polishing
5.7M    ./assembly/flye/10-consensus
66M     ./assembly/flye
90M     ./assembly
189M    ./qc/long
143M    ./qc/short
331M    ./qc
9.5M    ./polish/polypolish/input
239M    ./polish/polypolish
16M     ./polish/polca
422M    ./polish/medaka
16M     ./polish/cov_filter
691M    ./polish
2.3M    ./annotation/checkm/protein_files
2.7M    ./annotation/checkm
3.1M    ./annotation/eggnog
310M    ./annotation/coverage
6.7M    ./annotation/dfast/ddbj
6.9M    ./annotation/dfast/genbank
73M     ./annotation/dfast
388M    ./annotation
5.6M    ./circularize/combine
9.5M    ./circularize/polypolish/input
239M    ./circularize/polypolish
12M     ./circularize/circlator
5.6M    ./circularize/filter
261M    ./circularize
1.8G    .

Thanks for the helpful diagram and proposed changes! Is it okay if I wait to review this until after the holidays, or would you like input on this sooner?

Thanks for the helpful diagram and proposed changes! Is it okay if I wait to review this until after the holidays, or would you like input sooner?

After the holidays it is okay. I planned to run some large datasets over the break, but that appears to have been delayed. In the worst-case scenario, I will build and use a branch with temp() changes.

@jmtsuji Created a branch for this: https://github.com/rotary-genomics/rotary/tree/more_temp_files

@jmtsuji I might take this approach. https://snakemake.readthedocs.io/en/stable/snakefiles/rules.html#shadow-rules
I used it for the hot-fix for https://github.com/rotary-genomics/rotary/pull/105.

@jmtsuji I might take this approach. https://snakemake.readthedocs.io/en/stable/snakefiles/rules.html#shadow-rules
I used it for the hot-fix for #105.

So, this doesn't work the way I thought it would. I need to do some tests, but it appears to move all the files back, even those not listed in the output parameter of the snakemake rule.

@LeeBergstrand Do you think we can close this issue now that #114 has been merged?

Yes!


feature/issue-34: Support for var subsetting when vars are in a different group as coordinate vars

Github Issue: #34
Description
Variable subsetting in OCO3 would not work for variables in a different group as the latitude variable
Overview of work done
Added variable list as an argument in subset_bbox and extend the group list if the variables are present.
Overview of verification done
Test 3 variables in 3 different groups for OCO3 files.
Overview of integration done
Explain how this change was integration tested. Provide screenshots or logs if appropriate. An example of this would be a local Harmony deployment.
PR checklist:

[ x] Linted
[ x] Updated unit tests
[ x] Updated changelog
[ ] Integration testing

See Pull Request Review Checklist for pointers on reviewing this pull request

@nlenssen2013 I'm also seeing a few linting warnings with both flake8 and pylint. Can you address those? Thanks!


Accessibility

Do the regular stuff: check font sizes and that one can edit amenities using a text-to-speech interface.

[x] All IconButtons should have tooltips.
[x] The app should work fine with increased system font size.
[ ] Check for screen readers: RendererBinding.instance.setSemanticsEnabled(true).

https://docs.flutter.dev/development/accessibility-and-localization/accessibility

Could not enable semantics and decided that screen readers are not the target, other than that it is done in 0.10.

Sidenote: You can also check the accessibility of the Every Door website with ie. Google Lighthouse and W3C validators (sorry, I'm just stating the obvious).


ECS Fargate enhancemente and exec doc

Enhancements:

Updated screenshots for ecs fargate efs volumes.
Added Instructions to exec commands on fargate instance

Some minor things like changing fargate to Fargate


ValidationError fail OpenAPI Schema validation

First Check

[x] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from fastapi import FastAPI

app = FastAPI()

@app.get("/")
async def root():
    return {"message": "Hello World"}

Description

Running openapi-generator against the schema generated by FastAPI fails.
ValidationError schema cannot be processed.
Generation fails with FastAPI v0.75.0 or higher
Generated successfully with FastAPI v0.74.1

$ openapi-generator-cli generate -g typescript-axios -i http://localhost:8000/openapi.json

[main] INFO  o.o.codegen.DefaultGenerator - Generating with dryRun=false
[main] INFO  o.o.codegen.DefaultGenerator - OpenAPI Generator: typescript-axios (client)
[main] INFO  o.o.codegen.DefaultGenerator - Generator 'typescript-axios' is considered stable.
[main] INFO  o.o.c.l.AbstractTypeScriptClientCodegen - Hint: Environment variable 'TS_POST_PROCESS_FILE' (optional) not defined. E.g. to format the source code, please try 'export TS_POST_PROCESS_FILE="/usr/local/bin/prettier --write"' (Linux/Mac)
[main] INFO  o.o.c.l.AbstractTypeScriptClientCodegen - Note: To enable file post-processing, 'enablePostProcessFile' must be set to `true` (--enable-post-process-file for CLI).
[main] INFO  o.o.codegen.utils.URLPathUtils - 'host' (OAS 2.0) or 'servers' (OAS 3.0) not defined in the spec. Default to [http://localhost] for server URL [http://localhost/]
[main] INFO  o.o.codegen.utils.URLPathUtils - 'host' (OAS 2.0) or 'servers' (OAS 3.0) not defined in the spec. Default to [http://localhost] for server URL [http://localhost/]
[main] WARN  o.o.c.l.AbstractTypeScriptClientCodegen - The import is a union type. Consider using the toModelImportMap method.
Exception in thread "main" java.lang.RuntimeException: Could not process model 'ValidationError'.Please make sure that your schema is correct!
        at org.openapitools.codegen.DefaultGenerator.generateModels(DefaultGenerator.java:492)
        at org.openapitools.codegen.DefaultGenerator.generate(DefaultGenerator.java:888)
        at org.openapitools.codegen.cmd.Generate.execute(Generate.java:441)
        at org.openapitools.codegen.cmd.OpenApiGeneratorCommand.run(OpenApiGeneratorCommand.java:32)
        at org.openapitools.codegen.OpenAPIGenerator.main(OpenAPIGenerator.java:66)
Caused by: java.lang.StringIndexOutOfBoundsException: begin 1, end 0, length 0
        at java.base/java.lang.String.checkBoundsBeginEnd(String.java:3751)
        at java.base/java.lang.String.substring(String.java:1907)
        at java.base/java.lang.String.substring(String.java:1880)
        at org.openapitools.codegen.languages.TypeScriptAxiosClientCodegen.postProcessModels(TypeScriptAxiosClientCodegen.java:233)
        at org.openapitools.codegen.DefaultGenerator.processModels(DefaultGenerator.java:1302)
        at org.openapitools.codegen.DefaultGenerator.generateModels(DefaultGenerator.java:487)
        ... 4 more

Process finished with exit code 1

Operating System
macOS
Operating System Details
No response
FastAPI Version
0.75.2
Python Version
3.10.4
Additional Context
No response

same on 0.76.0

Well, it is due to this change: #4806
But I would rather argue this is an issue of openapi-generator, as the schema is valid according to the specification of OpenAPI. There is even an open issue for that: OpenAPITools/openapi-generator#4576

It seems so.
I used openapi-typescript instead.
This one generates without any problem.
Close this issue.

This issue with typescript-axios generator is fixed in the main branch of openapi generator, but it's not packaged yet. Using git bisect I pinned the fix down to this commit


Extend API to support backends that permit Compare-and-Swap for caching without a race condition

Seems like
if (!await keyvMap.has(someKey)) {
  const someValue = ...expensive computation and or side effects ...
  keyvMap.set(someKey, someValue)
}

is used by folks a lot but it has a race condition when that code block is re-entrant (e.g., an HTTP or GraphQL handler). keyv has a nice API, but it doesn't expose a proper Compare&Swap or SetIfUnset/SetNX functionality to allow using a backend such as redis that could support this pattern without the race condition.
Perhaps there could be an optional 4th argument to .set() to indicate the desire to return the old value or an new .getSet() function that returns a promise for the existing value instead of just uselessly returning true?
Then we can write:
if (!await keyvMap.getSet(someKey, 'in_progress')) {
  const someValue = ...expensive computation and or side effects ...
  keyvMap.set(someKey, someValue)
}

Or similar.  I just want to get rid of the race condition when the first processor hitting the await switching to a second processor at the same codepoint happens so both .has() invocations return false and we do the computation and have the side-effects multiple times.
Thoughts?

@gjbadros - thanks for sending this in and I do think that this would break many of the storage adapters by adding this in. I do understand that this is not as performant as you can get with using Redis directly.
We are trying to stay as much on the Map() API layer with some additional functions that have been slowly rolled out.
I would love to see a couple examples of other API's / libraries that are doing this and code examples to learn more before we just close this as something we are not going to do.
We could move to async but the issue is that removes the compatibility with others. Thoughts?

And it seems like making the set possibly return the prior value for backends that support it would an easy and good change. Given that some backends would see a performance hit to get and set, you could make it a new method or an extra argument to the setter that only some backends support.
What do you mean by "move to async"?

Most likely it would have to be a new method as we try not to break other clients out there and standardize on our current interfaces.

So just a getAndSet() method supported by redis and memcache backends?
On Sat, Apr 22, 2023, 10:18 AM Jared Wray @.***> wrote:

Most likely it would have to be a new method as we try not to break other
clients out there and standardize on our current interfaces.
—
Reply to this email directly, view it on GitHub
https://github.com/jaredwray/keyv/issues/756#issuecomment-1518710764,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALOHTKYVW5LGPGIBUGBHD3XCQHGPANCNFSM6AAAAAAWPECGTI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yeah. I think at this time we will most likely not add this as I would want most of the storage adapters to support this. Closing this option for now and will rethink it over this year as something that we will want to extend.

I recommend at least documenting the race condition.
On Sat, Apr 22, 2023, 4:57 PM Jared Wray @.***> wrote:

Closed #756 https://github.com/jaredwray/keyv/issues/756 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/jaredwray/keyv/issues/756#event-9073900667, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AALOHTPHP34Z5CLBAJMG3VDXCRV6VANCNFSM6AAAAAAWPECGTI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add deployment.environment tag

We should tag traces with this tag as we do for service.name and service.version
See: https://docs.datadoghq.com/tracing/setup_overview/open_standards/#opentelemetry-collector-datadog-exporter

It has been introduced in #10


supports --anonymous-auth=false for hub apiserver

What would you like to be added:
Why is this needed:

In k3s, anonymous-auth is explicitly set to false.
https://github.com/k3s-io/k3s/blob/f18b3252c0fbdfdc2a8e4d0a6d8deb4921052af9/pkg/daemons/control/server.go#L183-L185
Actually this is not a prerequisite.


feat(mountebank): allow existing Mountebank

Add ExistingMountebank: like Mountebank, but uses an existing
mountebank process, managed outside of mountepy.
Usage:
with ExistingMountebank(host, port) as mb:
    ...

@butla Not sure why the commit style check is failing in CI. It passes for me:
$ tox
GLOB sdist-make: /home/roman/dev/python/mountepy/setup.py
py34 inst-nodeps: /home/roman/dev/python/mountepy/.tox/dist/mountepy-0.4.0.zip
py34 installed: astroid==1.4.8,bottle==0.12.10,coverage==4.2,isort==4.2.5,lazy-object-proxy==1.2.2,mccabe==0.5.2,mountepy==0.4.0,pipdeptree==0.8.0,pkg-resources==0.0.0,port-for==0.3.1,py==1.4.31,pylint==1.6.4,pytest==3.0.3,python-dateutil==2.5.3,requests==2.10.0,six==1.10.0,wrapt==1.10.8
py34 runtests: PYTHONHASHSEED='413817490'
py34 runtests: commands[0] | /bin/bash -c ci-helpers/check_version_changed_accordingly.sh > /dev/null
py34 runtests: commands[1] | /bin/bash -c pylint mountepy --rcfile=.pylintrc || echo Fix Pylint issues! Or disable them if your absolutely shure that's the right thing to do.
py34 runtests: commands[2] | coverage run -m py.test tests/
================================================================================================================================== test session starts ===================================================================================================================================
platform linux -- Python 3.4.5, pytest-3.0.3, py-1.4.31, pluggy-0.4.0
rootdir: /home/roman/dev/python/mountepy, inifile:
collected 25 items

tests/test_http_service.py ............
tests/test_mb_mgmt.py .....
tests/test_mountepy.py ........

=============================================================================================================================== 25 passed in 18.41 seconds ===============================================================================================================================
py34 runtests: commands[3] | coverage report -m
Name                       Stmts   Miss  Cover   Missing
--------------------------------------------------------
mountepy/__init__.py           4      0   100%
mountepy/http_service.py      80      0   100%
mountepy/mb_mgmt.py           47      0   100%
mountepy/mountebank.py        78      0   100%
--------------------------------------------------------
TOTAL                        209      0   100%
________________________________________________________________________________________________________________________________________ summary _________________________________________________________________________________________________________________________________________
  py34: commands succeeded
  congratulations :)

@romanlevin I've traced Snap output and I know what's the problem. I didn't take merge commits into account when creating my scripts that validate commits. I'm not sure if I'll handle it in the script, because that would make it much more complex, but the problem doesn't appear if you rebase onto master.

@butla So this PR is fine in terms of passing tests? Should I change something on my end?

It's ok, I can rebase it if you want to. Though I'll try to refactor the code later (something doesn't seem right with those NotImplementedErrors, but I don't know how to structure it more elegantly right now).
Will you be adding Docker support, or is this enough for you now?

Rebased and merged (though I should have probably done that through the Github button)

It's definitely enough for one PR, I think. If I do add Docker support it will be based on top of this work.
I'm not happy about the NotImplementedErrors either. I'd rather just not define those methods on MountebankWrapper, but pylint won't let me :smiling_imp:.
My purpose here was to allow the new functionality without changing the existing API at all, There's probably a more elegant way of doing that. I considered giving Mountebank a provider object that specifies whether and how to control the mountebank process, but that's overkill just for this one feature.

Thanks for merging this! Any idea when it will make it's way to PyPI?

Never mind! I see it's already there!

Thanks to my magic pipeline :)
https://bultrowicz.com/cd-with-angularjs-commits/

Cool!


Pylance not working with pixi editable packages

Environment data

Pylance version: 2024.7.1
OS and version: WSL2 linux
Python version (& distribution if applicable, e.g. Anaconda): 3.10.14

Code Snippet

A reproducible example and description is done here:
https://github.com/clement-chaneching/pixi-pylance-test
It s a minimal example with 2 packages, 1 depending on the other on pixi as editable.
Repro Steps

Create 2 packages as above with pixi and hatch/setuptools build, with one depending on the other as editable
Navigating from package 1 to package 2 with pylance doesnt work.

Expected behavior
Having pylance to work with pixi editable paths.
Actual behavior

Currently it works fine without "editable".
Going into pixi shell and manually running pip install -e . --config-settings editable_mode=compat also doesnt help.
This also has also been raised in pixi repo
I tried using both hatch and setuptools and doesnt make any difference.

Logs

Happy to get any help or redirection on this topic.
Edit : @baszalmstra, Thanks for your PR on adding pixi on vscode python. Is this something you could have a look on?

This is working for me. Do you have the Pixi environment selected within VS Code?

Btw, when I first tried this I couldn't get Pixi support to work within vscode-python. The Pixi environment was not shown in the environment selector list. I think this is because I was in an experiment that changed my VS Code to use https://github.com/microsoft/python-environment-tools (which doesn't have Pixi support) rather than the environment locators in vscode-python. If you're not seeing the Pixi environment in the list of options, try setting the following VS Code setting and see if that helps:
    "python.locator": "js"

Hello,
Indeed, it looks like it works for me now as well!
Not sure what was happening last friday...


Feat/Add Status Updates to Orchestration Remote Interface module - Basic availability

Before submitting this PR, please make sure:

[X] Your code builds clean without any errors or warnings
[X] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Adds basic availability status updates to the remote iface/MQTT orchestration module
Tests performed by the developer:
Tested it.
Tips for QA testing:

@klpanagi

@klpanagi @TheHolyRoger the purpose of this pr is to check if the bot is running or not with that message? I thought that KLPANAGI it's not possible to infer this from another channel? For me seems okey since it's just another way to ask if the bot is running a Strat or not but idk if it was already implemented.

@cardosofede At the moment the bot sends a heartbeat message, but for the purposes of my new home assistant integration it's much better to have this connection state broadcast. I'd like to add more status info to this channel in the future too


issue in SAML

I am unable to pinpoint the source of the issue.

Zip of export from Firefox

Your RequestedAuthnContext response must match the request.
Request
<samlp:RequestedAuthnContext Comparison="minimum">
        <saml:AuthnContextClassRef>urn:oasis:names:tc:SAML:2.0:ac:classes:X509</saml:AuthnContextClassRef>
    </samlp:RequestedAuthnContext>
</samlp:AuthnRequest>

Response
<saml:AuthnStatement AuthnInstant="2021-07-20T10:20:39.893Z"
                             SessionIndex="da95fd1f-9629-43d2-9688-1013c19ce2a0::3dbaf278-94fe-4df0-af5d-a012375401d5"
                             SessionNotOnOrAfter="2021-07-20T20:20:39.893Z"
                             >
            <saml:AuthnContext>
                <saml:AuthnContextClassRef>urn:oasis:names:tc:SAML:2.0:ac:classes:unspecified</saml:AuthnContextClassRef>
            </saml:AuthnContext>
</saml:AuthnStatement>

i have configured it , attaching the stack trace. But still the response is invalid.
SAML-tracer-export-2021-07-25T06 31 11.677Z.zip
.

this is my configuration in GLPI Plugin

Your response is almost there but the claims are incorrect.  See the valid response below compared to your previously attached response.
Correct

Your Response

Closing due to no response.  Reopen if needed.


CacheFilter: Add date metadata/ internal header to cached responses

When a response is being inserted to the cache, the cache filter should add a date internal header or metadata to the cached response. This should be used instead of the response Date header throughout the CacheFilter module as there might be a clock skew between the origin and Envoy.

Among the updates that should be made when this is implemented:

Update CacheabilityUtils::isCacheableResponse() has_validation_data conditions.
Update LookupRequest::requiresValidation() and remove the check for a future response_time.

I think we should add things like this to the plugin API. Plugins can turn them into header if needed, and we can avoid the cost of modifying headers for in-memory caches.
Offhand, I think we should add a Metadata struct (or proto?) that would be included in makeInsertContext and LookupResult.


fix: openssl should be set to lib

Without this,  prisma can't find the libraries and will error out

So that's related to #1030 ? Is there any work around before this is getting merged?

@baptisteArno you can manually add openssl as a lib via the --libs cli arg

Looks like some relevant tests are failing. Could you please look into whats going on there. It might be an issue with older versions of prisma?

I'd like to revive this PR as I think it is still needed. Some of the tests are failing and can check it out more deeply next week. It would also be a good idea to add a test for prisma 5