XCode build failed

Hi Matthiew.
I'm trying to build AudioTK on XCode.
I installed Boost, libsndfile et fftw via macports and the CMake generation process seem to work more or less, except for FFTW but I tried to run a build and hoped to see work not every module but at least get ATK Core to build.
My cmake output is :
_INFO Build shared libraries: ON
INFO Build static libraries: OFF
INFO Build tests: ON
Could NOT find FFTW (missing:  FFTW_LIBRARY_FFTW3 FFTW_LIBRARY_FFTW3F FFTW_LIBRARIES FFTW_INCLUDES)
Boost version: 1.61.0
Boost version: 1.61.0
Found the following Boost libraries:
unit_test_framework
system
Boost version: 1.61.0
Found the following Boost libraries:
unit_test_framework
timer
system
chrono
Configuring done
CMake Warning (dev):
Policy CMP0042 is not set: MACOSX_RPATH is enabled by default.  Run "cmake
--help-policy CMP0042" for policy details.  Use the cmake_policy command to
set the policy and suppress this warning.
MACOSX_RPATH is not specified for the following targets:
ATKAdaptive
ATKCore
ATKDelay
ATKDistortion
ATKDynamic
ATKEQ
ATKIO
ATKMock
ATKPreamplifier
ATKReverberation
ATKSpecial
ATKTools
ATKUtility
This warning is for project developers.  Use -Wno-dev to suppress it.
Generating done_
the build fails in TypedBaseFilter.cpp and there was to issue there :
At first in the with RValue constructor
   template<typename DataType>   TypedBaseFilter<DataType>::TypedBaseFilter(TypedBaseFilter&& other)   :Parent(static_cast<Parent &&>(std::move(other))),
There was an error if I was not casting std::move(other) to RValue type.
I confess I'm not fluent at all with C++11 Feature and I'm lost there.
I hope you have a OS X and you could try a build on your machine, or maybe you got an idea of what's happenning ?
I'm on OS X El-Capitain. I use LLVM.
Thank you

By default, C++11 is not activated on the main compilers used by the users (although it starts coming with VS and GCC6).
In your case, you need to add -std=c++11 to the CMAKE_CXX_FLAGS and that should do the trick ;)

Thank you Matthiew ! I was this I had to enable C++ 11 option on every subtarget.
Strangely it's a custom post build who fails now : And builds fails after ATKCore !
PhaseScriptExecution CMake\ PostBuild\ Rules build/ATK/Core/AudioTK.build/Debug/ATKCore.build/Script-FEBD7229655F40AEA2FDC66A.sh
cd "/Users/francoisreme/Project From Internet/AudioTK"
/bin/sh -c "/Users/francoisreme/Project\ From\ Internet/AudioTK/build/ATK/Core/AudioTK.build/Debug/ATKCore.build/Script-FEBD7229655F40AEA2FDC66A.sh"
echo "Staging dSYM for ATKCore"
Staging dSYM for ATKCore
cd /Users/francoisreme/Project\ From\ Internet/AudioTK/build/ATK/Core && dsymutil libATKCore.dylib
error: cannot parse the debug map for "libATKCore.dylib": No such file or directory
make: *** [ATKCore_buildpart_0] Error 1

That's annoying... There may be an issue with the dependency ordering at the moment. Is the file available in the build folder /Users/francoisreme/Project\ From\ Internet/AudioTK/build/ATK/Core?

Here come the content of the generated Core directory :
/Users/francoisreme/Project\ From\ Internet/AudioTK/build/ATK/Core
├── AudioTK.build
│   └── Debug
│       └── ATKCore.build
│           ├── Objects-normal
│           │   └── x86_64
│           │       └── ATKCore.LinkFileList
│           ├── Script-FB573317575F4A62959C636C.sh
│           ├── dgph
│           └── dgph~
├── CMakeFiles
├── CMakeScripts
│   ├── ATKCore_postBuildPhase.makeDebug
│   ├── ATKCore_postBuildPhase.makeMinSizeRel
│   ├── ATKCore_postBuildPhase.makeRelWithDebInfo
│   └── ATKCore_postBuildPhase.makeRelease
├── CTestTestfile.cmake
└── cmake_install.cmake

I see, it's in the subfolder.
Well, I'll have to have a closer look later next week, I'm too busy to fix the issue in the next few days :/
What you can do is either use a release build, or you can change the main CMakeLists.txt to add Debug in the dsym target :/

Changing in main CMakeList to :
POST_BUILD
COMMAND dsymutil Debug/$<TARGET_FILE_NAME:${target}>
Works like a charm. Thank you for you help, I will now be able to perform tests !
Thanks !

Really strange, I can't reproduce the issue. Maybe because my xcode project lived through several CMake versions :/ I'll have to start from scratch one day :/

Oh!
You may have had CMAKE_BUILD_TYPE set to empty or not defined?

Yes, it's empty...and that's cool ?

It should be set to Debug or Release, depending on your build. That must be why you have that error.

Ok. It's works. I think there is a better way to handle this with CMake.
I personnaly use the $<TARGET_FILE:tgt> command who work pretty well :
https://cmake.org/cmake/help/v3.0/manual/cmake-generator-expressions.7.html
Cheers :-)


IO-like objects handled gracefully

I noticed in writing a small sample script to try Saxy out that ARGF wasn't handled. Threw an error about trying to get the class name of the object. ARGF is IO-like, but not actually an ancestor of IO. Seems like Nokogiri gracefully duck-types this IO object in either parse or parse_io.
Could https://github.com/monterail/saxy/blob/master/lib/saxy/parser.rb#L71 check for an IO-like interface instead of class ancestry?
I'd be more than happy to supply a PR, just let me know if you're interested.

@jbielick definitely something I would put into saxy. Looking forward to PR :)

https://github.com/monterail/saxy/pull/4


fix: Fix logging problems. Fixes #2589

Checklist:

[x] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
[x] The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
[x] I have written unit and/or e2e tests for my change. PRs without these are unlike to be merged.
[x] Optional. I've added My organization is added to the USERS.md.
[x] I've signed the CLA and required builds are green.

Fixes #2589
Fixes bullet points 3 and 5 from #2589. I believe that point 2 is caused by Go routine scheduling. May fix 1.

These changes LGTM, but we should make sure we fix points 2. and 4. in the original issue before closing it

I'm not sure we should fix 4 (even though it is a breaking change), lets discus at stand-up.


Django/Python: timeout / resume session

When a website visitor enters their passphrase, once their session is inactive for a 30 minutes, Django must automatically lock them out and require the visitor to re-enter their passphrase. When they re-enter their valid passphrase, they must return to / resume their last known web page.
The timeout setting (e.g. 15/30/45 mins) needs to be a variable that can be adjusted in the Django Admin Dashboard from the accounts app.

The problem was actually with my refactoring of accounts/models.py > AuthToggle class attribute change from On to is_published. I had to update the references to On in middlewares.py as well as in the _navbar.html partial template.
Next @UmarGit showed me how to reset the accounts tables. First you need to login to the Heroku Dyno and then invoke this migrate command:
(local venv) $ heroku run bash -a tarot-prod
(remote host) ~ $ python manage.py migrate accounts zero
(remote host) ~ $ python manage.py makemigrations
(remote host) ~ $ python manage.py migrate

It's important that you run python manage.py migrate accounts zero and not  python manage.py migrate zero because without the accounts in between, I would lose all my precious content data in all the tables!


Include VNL in build

On a related issue it has come up that I have no VNL built into my ITK. I have Looked into CMakeLists.txt, and found utterly nothing on VNL. How can I set this to be build, or if it is an external piece of software (it appears to be listed as a third-party module), where do I get it, and how can I signal to ITK to use the system distribution?
Currently I am using this command to build:
cmake -C /var/tmp/portage/sci-libs/itk-5.0_rc1/work/itk-5.0_rc1_build/gentoo_common_config.cmake -G Unix Makefiles -DCMAKE_INSTALL_PREFIX=/usr -DBUILD_SHARED_LIBS=ON -DITK_USE_SYSTEM_DCMTK=ON -DGDCM_USE_SYSTEM_OPENJPEG=ON -DITK_USE_SYSTEM_DOUBLECONVERSION=ON -DITK_USE_SYSTEM_CASTXML=ON -DITK_USE_SYSTEM_HDF5=ON -DITK_USE_SYSTEM_JPEG=ON -DITK_USE_SYSTEM_PNG=ON -DITK_USE_SYSTEM_SWIG=ON -DITK_USE_SYSTEM_TIFF=ON -DITK_USE_SYSTEM_ZLIB=ON -DITK_USE_KWSTYLE=OFF -DITK_BUILD_DEFAULT_MODULES=ON -DITK_COMPUTER_MEMORY_SIZE=1 -DWRAP_ITK_JAVA=OFF -DWRAP_ITK_TCL=OFF -Ddouble-conversion_INCLUDE_DIRS=/usr/include/double-conversion -DExternalData_OBJECT_STORES=/var/tmp/portage/sci-libs/itk-5.0_rc1/work/InsightToolkit-5.0rc01/.ExternalData -DBUILD_TESTING=OFF -DBUILD_EXAMPLES=OFF -DITK_USE_REVIEW=OFF -DITK_INSTALL_LIBRARY_DIR=lib64 -DVNL_CONFIG_ENABLE_SSE2=yes -DITK_WRAP_PYTHON=OFF -DCMAKE_BUILD_TYPE=Gentoo -DCMAKE_TOOLCHAIN_FILE=/var/tmp/portage/sci-libs/itk-5.0_rc1/work/itk-5.0_rc1_build/gentoo_toolchain.cmake  /var/tmp/portage/sci-libs/itk-5.0_rc1/work/ITK-5.0rc01

There actually seem to be quite a few files with names including “vnl” installed by my ITK: itk_files.txt.

Not a bug with ITK
Mismatched ANTs and ITK versions resulting in incompatibility seem to be the issue, see updated comments on https://github.com/ANTsX/ANTs/issues/612

I see, thank you.


Add HTTP Gateway Protocol specification

Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

[ ] You have updated the .github/CODEOWNERS file according to your changes.
[x] You have registered new documents to /sidebars.js if any.

The actual HTTP Gateway Protocol spec has been moved to the ic-interface-spec repository: https://github.com/dfinity/interface-spec/pull/147. Once that is merged I will update this PR to ensure it is included in the portal.


"No such interface supported" after upgrade to windows store version of wsl

Version
Microsoft Windows [Version 10.0.19044.2311]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
No response
Distro Version
Ubuntu
Other Software
No response
Repro Steps
I had wsl2 working just fine then when i saw wsl 1.0 was released for win10 i wanted to give it a try.
installed KB5020030 and then ran a "wsl --update"
store version of wsl was downloaded and installed
after this was done, any attempt to run wsl.exe results in
No such interface supported
Error code: Wsl/0x80004002

and then wsl doesnt work any more.
interesting side note:
using a different account on the same computer that never used wsl before, i was able to get wsl started and installed, most likely using the pre-windows store version because linux kernel ended up being 4.4 instead of 5.10 something.
on that account, running wsl --update makes wsl break with above "No such interface" error and uninstalling the store version from Settings -> Apps makes it work again and wsl session uses the same old 4.4 kernel.
Expected Behavior
A working startup of wsl like before
Actual Behavior
Error: "No such interface supported" and then it doesnt start
Diagnostic Logs
No response

Tried uninstalling the store version of wsl on the regular user account and my old Ubuntu wsl dist came back and started to work.
and as expected i got this at the top:
Windows Subsystem for Linux is now available in the Microsoft Store!
You can upgrade by running 'wsl.exe --update' or by visiting https://aka.ms/wslstorepage
Installing WSL from the Microsoft Store will give you the latest WSL updates, faster.
For more information please visit https://aka.ms/wslstoreinfo

Obviously i'm not running wsl.exe --update again until someone can explain why it fails or how to fix it.

any update to this?
the issue still happens with kb5021233 installed (the non preview version)

got that error to

I am also getting this error. I've scoured google for troubleshooting tips and so far nothing has changed the dysfunctional behavior. I'm trying to get docker setup and I'm stuck unable to get WSL running.

I‘ve been hitting this issue recently. After some random effort, I can reproduce this and fix it quite easy.
As @tjnsn said. run wsl --update will corrupt your wsl command.
Remove the Appxpackage that updating installed will get your wsl back.
Run Get-AppxPackage |? { $_.Name -like "*WindowsSubsystemforLinux*" } in powershell to get the package fullname, and remove it using Remove-AppxPackage -Package "<packag-full-name>"
The actual wsl you're using is probably in "Setting -> Application" called "Windows Subsystem for Linux Update".

Unfortuneately,
Get-AppxPackage |? { $_.Name -like "*WindowsSubsystemforLinux*" }
did not return anything for me when ran in powershell as admin.

any update to this? only the inbox version can running correctly. once install the store version then everything broken

same problem here.  Can't uninstall the "store app" - it seems like it's in a bad state.  Please help - this is a huge productivity killer for me.  Also, it's pretty shameful that you recommended people to perform this upgrade and it completely bricks their wsl environments.

I‘ve been hitting this issue recently. After some random effort, I can reproduce this and fix it quite easy. As @tjnsn said. run wsl --update will corrupt your wsl command. Remove the Appxpackage that updating installed will get your wsl back. Run Get-AppxPackage |? { $_.Name -like "*WindowsSubsystemforLinux*" } in powershell to get the package fullname, and remove it using Remove-AppxPackage -Package "<packag-full-name>" The actual wsl you're using is probably in "Setting -> Application" called "Windows Subsystem for Linux Update".

This did the trick for me. Thank you very much @AZ-201

As another sample point I ran into also this issue after a fresh install of Windows 10. After installing Windows I installed the WSL from the Windows Store. After installation completed, running the wsl command would give
This application requires the Windows Subsystem for Linux Optional Component.
The system may need to be restarted so the changes can take effect.
Error code: Wsl/WSL_E_WSL_OPTIONAL_COMPONENT_REQUIRED

A reboot does not fix this.
Also, running wsl --install ubuntu leads to the following output:
Installing: Ubuntu
Ubuntu has been installed.
Launching Ubuntu...
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80004002
Error: 0x80004002 No such interface supported

Press any key to continue...
The installation process for distribution 'Ubuntu' failed with exit code: 1.
Error code: Wsl/InstallDistro/WSL_E_INSTALL_PROCESS_FAILED

which is the error mentioned here and at #9014.
To get things working I removed both WSL and the Ubuntu install using "Apps & Features". Once that was done, from the command line I ran wsl --install --no-distribution:
The requested operation requires elevation.
Installing: Virtual Machine Platform
Virtual Machine Platform has been installed.
Installing: Windows Subsystem for Linux
Windows Subsystem for Linux has been installed.
Installing: Windows Subsystem for Linux
Windows Subsystem for Linux has been installed.
The requested operation is successful. Changes will not be effective until the system is rebooted.

After rebooting, I could install Ubuntu (and other distributions happily). It seems that when installing from the Windows Store directly it does not install the required dependencies:

Virtual Machine Platform
Windows Subsystem for Linux
and this is confirmed in the "Windows Features" control panels settings window:

I was not sure that it was installing the Windows Store version of WSL, but going into the Windows Store app it only has the option to "open" and running X apps works correctly, so I am assuming that it has installed the store version of the app.

@chrisjaquet This works for me absolutely. Thank you.

As another sample point I ran into also this issue after a fresh install of Windows 10. After installing Windows I installed the WSL from the Windows Store. After installation completed, running the wsl command would give
This application requires the Windows Subsystem for Linux Optional Component.
The system may need to be restarted so the changes can take effect.
Error code: Wsl/WSL_E_WSL_OPTIONAL_COMPONENT_REQUIRED

A reboot does not fix this.
Also, running wsl --install ubuntu leads to the following output:
Installing: Ubuntu
Ubuntu has been installed.
Launching Ubuntu...
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80004002
Error: 0x80004002 No such interface supported

Press any key to continue...
The installation process for distribution 'Ubuntu' failed with exit code: 1.
Error code: Wsl/InstallDistro/WSL_E_INSTALL_PROCESS_FAILED

which is the error mentioned here and at #9014.
To get things working I removed both WSL and the Ubuntu install using "Apps & Features". Once that was done, from the command line I ran wsl --install --no-distribution:
The requested operation requires elevation.
Installing: Virtual Machine Platform
Virtual Machine Platform has been installed.
Installing: Windows Subsystem for Linux
Windows Subsystem for Linux has been installed.
Installing: Windows Subsystem for Linux
Windows Subsystem for Linux has been installed.
The requested operation is successful. Changes will not be effective until the system is rebooted.

After rebooting, I could install Ubuntu (and other distributions happily). It seems that when installing from the Windows Store directly it does not install the required dependencies:

Virtual Machine Platform
Windows Subsystem for Linux
and this is confirmed in the "Windows Features" control panels settings window:

I was not sure that it was installing the Windows Store version of WSL, but going into the Windows Store app it only has the option to "open" and running X apps works correctly, so I am assuming that it has installed the store version of the app.

it is not that simple as missing optional features, this is something i have checked many times in my troubleshooting of this.
also i have a brand new computer that was working fine with the store version of wsl (1.2.5 i believe) and then all of a sudden broke and i have no idea why.
it now behaves just like my older computer i used when i initialy created this ticket.
so new comptuer was working with store, then broke, tried reinstalling and now i have "No such interface supported" as long as store version of wsl is installed.
removing it and falling back pre-windows store version works

I'm prety sure i know what is triggering this error for me.
due to company policy many security related settings have been adjusted, one of them is removal of the default groups "interactive" and "authenticated users" from the local "Users" group.
this in turn causes something in wsl to fail, not sure what exactly, but i suspect it is some user related service or something, perhaps related to the integration of wsl and windows side.
and then when you try to do somethign with wsl, all you get is "no such interface" and the error code.
wsl --version works but wsl --list does not and errors
adding back the two groups plus a reboot makes it work again, but that is not realy a workabl solution in the long run.

My workaround is to add back: "NT AUTHORITY\Authenticated Users" to the local Users group and then restart the service WslService
This service is unfortunately listed with the name: ms-resource:AppName so it was not immediately obvious it is WSL related.
(would be nice with a proper name for the service)
Would be nice with some input from microsoft about this, for example is it a must to have authenticated users in the users group or is there a way to make it work without it (prefered due to company policies)
@OneBlue ?

My workaround is to add back: "NT AUTHORITY\Authenticated Users" to the local Users group and then restart the service WslService This service is unfortunately listed with the name: ms-resource:AppName so it was not immediately obvious it is WSL related. (would be nice with a proper name for the service)
Would be nice with some input from microsoft about this, for example is it a must to have authenticated users in the users group or is there a way to make it work without it (prefered due to company policies) @OneBlue ?

From what I can see with Process Monitor, the wslservice.exe gets ACCESS DENIED when trying to do a CreateFile operation on C:\ProgramData\Microsoft\Windows\AppRepository\Packages\MicrosoftCorporationII.WindowsSubsystemForLinux_<IP_ADDRESS>_x64__8wekyb3d8bbwe\PackagedCom\WslServiceProxyStub.dll.
I'm not sure how it's related, but I was able to get by with just adding "BUILTIN\SYSTEM" to the users group instead of all authenticated users.

Adding to the local Users group seemed risky, so I didn't try this - but I hope somebody with a clearer understanding of the consequences of adding users can help understand what that would cause.
I've found that on the base version of wsl, running wsl --install --inbox works to update the WSL kernel to <IP_ADDRESS>, but WSL itself doesn't play well with updating (I try with Add-AppxPackage and am able to revert with the Get-AppxPackage and Remove-AppxPackage) , I keep getting the "No such interface found".
Interestingly, the kernel gets updated alongside the wsl to the following versions:
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-3
WSLg version: 1.0.54
MSRDC version: 1.2.4240
Direct3D version: 1.608.2-61064218
DXCore version: 10.0.25880.1000-230602-1350.main
Windows version: 10.0.19045.3086
I've tried with a few different versions (you can see 1.3.11 (preview) above, but the store version of 1.2.5 also fails)
As mentioned above, reverting to the base version with the method from @AZ-201 still works, and my data isn't lost (I backed it up just in case)

I‘ve been hitting this issue recently. After some random effort, I can reproduce this and fix it quite easy. As @tjnsn said. run wsl --update will corrupt your wsl command. Remove the Appxpackage that updating installed will get your wsl back. Run Get-AppxPackage |? { $_.Name -like "*WindowsSubsystemforLinux*" } in powershell to get the package fullname, and remove it using Remove-AppxPackage -Package "<packag-full-name>" The actual wsl you're using is probably in "Setting -> Application" called "Windows Subsystem for Linux Update".

I'd still love to be able to use wslg from the releases - MobaXterm still lags a bit for my taste
Any help is appreciated!

Faced the same issue on a secured corporate machine. I ran this from an elevated cmd.exe shell:
net localgroup users "nt authority\authenticated users" /add
Seems to have fixed it. I got that command from https://learn.microsoft.com/en-us/troubleshoot/windows-server/identity/add-special-groups-to-built-in-groups

There is no solution to use new version of WSL if company don't allow to change the localgroup user?

I'm prety sure i know what is triggering this error for me. due to company policy many security related settings have been adjusted, one of them is removal of the default groups "interactive" and "authenticated users" from the local "Users" group.
this in turn causes something in wsl to fail, not sure what exactly, but i suspect it is some user related service or something, perhaps related to the integration of wsl and windows side. and then when you try to do somethign with wsl, all you get is "no such interface" and the error code. wsl --version works but wsl --list does not and errors
adding back the two groups plus a reboot makes it work again, but that is not realy a workabl solution in the long run.

Hi, good work! We experienced similar issues. Although the fix is clear, I am much more interested how you have been able to pinpoint it to this cause. Solutions are always nice but the road to find it is much more useful.

I'm prety sure i know what is triggering this error for me. due to company policy many security related settings have been adjusted, one of them is removal of the default groups "interactive" and "authenticated users" from the local "Users" group.
this in turn causes something in wsl to fail, not sure what exactly, but i suspect it is some user related service or something, perhaps related to the integration of wsl and windows side. and then when you try to do somethign with wsl, all you get is "no such interface" and the error code. wsl --version works but wsl --list does not and errors
adding back the two groups plus a reboot makes it work again, but that is not realy a workabl solution in the long run.

Hi, good work! We experienced similar issues. Although the fix is clear, I am much more interested how you have been able to pinpoint it to this cause. Solutions are always nice but the road to find it is much more useful.

well....
pure luck mostly.
on my old computer i always assumed the broken wsl was due to a fault of me messing something up.
all i knew was that only the pre windows store version worked, any atempt to run the store version caused wsl to break.
so i figured i give it a new try after getting a new computer.
and to my surprise the store version worked imediately and worked fine for a couple of weeks.
so i, incorrectly, assumed problem was solved and it was my fault somehow that it broke it on the old computer.
after a few weeks it broke again and this time it was definitely not my fault so that got me to troubleshoot it.
this happen to coincide with a change and apparently the policy related to authenticated users had been temporarily turned off for a few weeks (when it worked for me after computer replacement).
then it was just a matter to test and confirm it.
so it was down to a computer replacement, timing and some luck.
now it would be nice with some feedback from MS about this.
either fix the issue or bare minimum update the wsl documentation section regarding running wsl in a corporate environment so it includes the need for authenticated users.

I have found a workaround for the problem that does not require adding the "authenticated users" group to the "users" group.
The problem is that WSL wants to run a process in the App Package folder, but the permissions for the System Account do not allow it because the permission inheritance has been broken.
If you start a CMD as System Account using PSEXEC (psexec -i -s cmd.exe) and set the permissions of the folder with the command icacls C:\ProgramData\Microsoft\Windows\AppRepository\Packages\MicrosoftCorporationII. WindowsSubsystemForLinux_<IP_ADDRESS>_x64__8wekyb3d8bbwe\PackagedCom /grant System:(OI)(CI)(X) /inheritance:d corrected, WSL2 works with the latest release.
Why the permissions in the directory are wrong, Microsoft would have to answer. However, this is a workaround, with which you can
you don't have to soften the company policy.
Probably you will have to correct the permissions again after each update.

I have found a workaround for the problem that does not require adding the "authenticated users" group to the "users" group. The problem is that WSL wants to run a process in the App Package folder, but the permissions for the System Account do not allow it because the permission inheritance has been broken.
If you start a CMD as System Account using PSEXEC (psexec -i -s cmd.exe) and set the permissions of the folder with the command icacls C:\ProgramData\Microsoft\Windows\AppRepository\Packages\MicrosoftCorporationII.WindowsSubsystemForLinux_<IP_ADDRESS>_x64__8wekyb3d8bbwe\PackagedCom /grant System:(OI)(CI)(X) /inheritance:d corrected, WSL2 works with the latest release.
Why the permissions in the directory are wrong, Microsoft would have to answer. However, this is a workaround, with which you can you don't have to soften the company policy.
Probably you will have to correct the permissions again after each update.

Good catch, havent tested this yet but i'm prety sure it will work.
Using sysinternals procmon you can see that wslservice.exe is trying to open that path and fails with an access denied because of the permissions.
The service is running as SYSTEM and the permissions on the PackagedCom folder does not grant SYSTEM the execute permission and the only reason it does get execute permission is because authenticated users membership in users group makes SYSTEM part of the Users group and Users do have the needed permissions.

I have found a workaround for the problem that does not require adding the "authenticated users" group to the "users" group. The problem is that WSL wants to run a process in the App Package folder, but the permissions for the System Account do not allow it because the permission inheritance has been broken.
If you start a CMD as System Account using PSEXEC (psexec -i -s cmd.exe) and set the permissions of the folder with the command icacls C:\ProgramData\Microsoft\Windows\AppRepository\Packages\MicrosoftCorporationII.WindowsSubsystemForLinux_<IP_ADDRESS>_x64__8wekyb3d8bbwe\PackagedCom /grant System:(OI)(CI)(X) /inheritance:d corrected, WSL2 works with the latest release.
Why the permissions in the directory are wrong, Microsoft would have to answer. However, this is a workaround, with which you can you don't have to soften the company policy.
Probably you will have to correct the permissions again after each update.

This actually works! This should be the most perfect solution at the moment. Thank you very much.

I had no issues in Win 10 on my work PC with WSL. In place upgrade to Win 11 then running WSL it prompted me to run wsl.exe --upgrade which broke my VMs and uninstall reinstall isn't working. Working through some of the fixes here to hopefully solve.

now that wsl 2.0.9 is not a prerelease any more i'm going to close my own ticket.
the issue was caused by removal of authenticated users and 2.0.9 no longer have the problem


[DOC] TSC notebook

a few minor corrections

The phrasing is a bit weird here and got me confused :
We use 3D numpy even if the data is univariate: all classifiers work with shape (n_cases, n_timepoints), this can get confused with single multivariate time series, which are of shape (n_channels, n_timepoints).
Did you mean something like this?
We use 3D numpy even if the data is univariate: even though a univariate classifier could work using a 2D array of shape (n_cases, n_timepoints), this 2D shape can get confused with single multivariate time series, which are of shape (n_channels, n_timepoints). Hence, to differentiate both cases, we enforce the 3D format (n_cases, n_channels, n_timepoints) to avoid any confusion.


Tracing AMQP Workflow

For a workflow as follows:
Start (Service A) -- amqp --> General Processing (Service B) ---
--- amqp --> Fork into two queues Task C (Service C) and D (Service D)

The exact similar case of https://github.com/jaegertracing/jaeger-client-python/issues/122
From the code examples:
$tracer = $config->initTrace('Service A', '<IP_ADDRESS>:6831');
$spanContext = $tracer->extract(Formats\TEXT_MAP, $_SERVER);

$serverSpan = $tracer->startSpan('get data', ['child_of' => $spanContext]);
$serverSpan->addBaggageItem("version", "1.6.0");

$tracer->inject($serverSpan->getContext(), Formats\TEXT_MAP, $_SERVER);

Questions

what does extract(Formats\TEXT_MAP, $_SERVER) achieve?
Is it the server's spanContext that needs to be shared across all services and have them bi child_of of it?
How would you recommend to share a trace that goes from Service A to Service D to cover the mentioned workflow?

Much appreciated!

Does amqp implement Distributed Tracing?

Not with PHP, no. There is a Go package for that though https://github.com/opentracing-contrib/go-amqp not sure if it helps (new to this realm, sorry).
On the other hand, it is not necessary to track within AMQP, what matters is to trace the spans in the services (it's alright to skip AMQP for tracing).
Would that work? i.e. if we were to pass the SpanContext through message headers of AMQP somehow?

I think you need to inject SpanContext to  message body. Like this： Service A： $tracer->inject($serverSpan->getContext(), Formats\TEXT_MAP, $amqpMsg) and Service C : $tracer->extract(Formats\TEXT_MAP, $amqpMsg)


Flutter Video Player - Android Controller input gets stuck on on pressing left key (dpad left key)

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Flutter video player is not accepting any input from Android Controller or Physical or Logical Keyboard when arrow left key is pressed.
I/flutter (10294): RawKeyDownEvent#23f68(logicalKey: LogicalKeyboardKey#860f3(keyId: "0x100000302", keyLabel: "Arrow Left", debugName: "Arrow Left"), physicalKey: PhysicalKeyboardKey#52951(usbHidUsage: "0x00070050", debugName: "Arrow Left"), repeat: false)
It do not print anything in log after left key is pressed.
Once the left key is  pressed the player does not accept any input unless I press Right key and everything gets to normal
Expected results
The Inputs should not get stuck
Actual results
The Inputs are getting stuck and not able to do anything in player till right key is pressed
Code sample
Code sample
 FireTVRemoteListener(
                      onUp: () => '',
                      onDown: () => '',
                      onLeft: ()  {

                        setState(() {
                          isBackward = true;
                          _controller.seekTo(Duration(seconds: _controller.value.position.inSeconds - 4));
                          isBackward = false;
                        }
                        );
                      },

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
```console
I/flutter (10294): RawKeyDownEvent#23f68(logicalKey: LogicalKeyboardKey#860f3(keyId: "0x100000302", keyLabel: "Arrow Left", debugName: "Arrow Left"), physicalKey: PhysicalKeyboardKey#52951(usbHidUsage: "0x00070050", debugName: "Arrow Left"), repeat: false)[Paste your logs here]
```

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.7.5, on Microsoft Windows [Version 10.0.19045.2965], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[√] Chrome - develop for the web
[X] Visual Studio - develop for Windows
    X Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components
[√] Android Studio (version 2021.3)
[√] Connected device (4 available)
[√] HTTP Host Availability

</details>

Thanks for the report @sa1dude23
Are you trying to use the video player on Android TV or using emulator or physical device on which this behavior is occuring ?
Also, can you upgrade to latest stable or latest master version and run your scenario again to see if the issue still persists or not ?

It is happening for Android TV - Both Actual TV and Emulator. Tried with both DEBUG and RELEASE builds.
Regarding the upgrade part I will test it today and post you an update.


Added quick sort program

_ No description provided. _

Please add a readme file with detailed desciption of the algo implemented through this code.

Why are you adding two files for the same ?

Why are you adding two files for the same ?

Moved the program file to the folder along with readme file describing the implementation of the code.


image classification code sample error

Thank you for submitting a TensorFlow documentation issue. Per our GitHub
policy, we only address code/doc bugs, performance issues, feature requests, and
build/installation issues on GitHub.
The TensorFlow docs are open source! To get involved, read the documentation
contributor guide: https://www.tensorflow.org/community/contribute/docs
URL(s) with the issue:
https://www.tensorflow.org/beta/tutorials/images/classification
Description of issue (what needs changing):
acc = history.history['accuracy']
val_acc = history.history['val_accuracy']

should be
acc = history.history['acc']
val_acc = history.history['val_acc']

Clear description
When I copy and paste the code and run it directly, the model is trained for a few minutes/a while but I get KeyError: 'accuracy' and then when I change that, get KeyError: 'val_accuracy'
Someone would use this to perform image recognition with code they take directly from the Tensorflow docs on this page.
Correct links
It is not correct yet. I am thinking of making a PR to the docs repo
Parameters defined
n/a
Returns defined
n/a
Raises listed and defined
n/a
Usage example
to train images
Request visuals, if applicable
There's a few.
Submit a pull request?
Are you planning to also submit a pull request to fix the issue? See the docs
contributor guide: https://www.tensorflow.org/community/contribute/docs and the
docs style guide: https://www.tensorflow.org/community/contribute/docs_style
I am!

@elizabethsiegle Thanks for reporting this issue. The code in the tutorial is correct for TF2.0. If you want to run it in TF1.x, you need to change those two lines from
acc = history.history['accuracy']
val_acc = history.history['val_accuracy']

to
acc = history.history['acc']
val_acc = history.history['val_acc']

Thanks! Please close this issue if it was resolved? Thanks!

I am closing the issue as it was resolved. Please feel free to open it if the issue persists again. Thanks!


Add Implementation on IDependable

We are facing the following error when using a dependency on the SopsSecret:
"Error: * does not implement IDependable. Use "Dependable.implement()" to implement"
Please add an implementation for IDependable. Thanks!

Solved - IDependable is already implemented.


Allow setting variables for intermediateType in AggregateFunctionSignature

Fix #9105.

Hi @mbasmanova @kagamiori. Could you help review?

@liujiayi771 CI is red.

@liujiayi771 CI is red.
I see. But I find that many other PRs failed, because of these two tests:

CustomTypeTest.getCustomTypeNames
GenericViewTest.toString

But I find that many other PRs failed, because of these two tests in velox_expression_test:

Would you try rebasing? If tests still fail, please, create GitHub issue. We'll need to fix these tests before merging PRs.

@mbasmanova I have already moved the validation into the constructor, and the AggregateFunctionSignature passes the usedVariables to FunctionSignature for validation through a lambda function.

Hi @mbasmanova, I added documents to the new parameter, and the issue I hope to solve is being able to use signature variables in AggregateFunctionSignature's intermediateType_. This might be described in more detail in the issue. The primary role of the lambda function is to pass the variables used in intermediateType_ to the constructor of FunctionSignature for use during validation.

@liujiayi771 Would you make sure the PR is rebased onto latest main?

@liujiayi771 Would you make sure the PR is rebased onto latest main?

Rebased.


v2: Add sorting to facilities table

Description
User should be able to sort the facilities table.
Proposed solution
This table should be sortable in the following ways:

Name - sort A-Z, Z-A

Tests conducted, positive tests, negative tests, positivity rate: high-low, low-high

Additional context
Figma file: https://www.figma.com/file/NPSPM5SBzQ7Ra3mmr9M9cB/SimpleReport-Public-Site?node-id=3996%3A13764

Closing; will evaluate the dashboard more holistically to determine user needs.


About GPU accelerate

I tried to complie with cmake .. -DBUILD_VGICP_CUDA=ON, but I found when run python_tester/gicp_odometry2.py to test. It can run but the fps is less than 10. And it didn't use my gpu.
I wonder how can I get a gpu version of fast_gicp. Thanks!

I've get a cuda version successfully, but during my experience cuda version seems to be slower. That's strange, and using FGICP can't reach a very high fps.


Request for a scale function on the shape

Hey there!
Is it possible we can have an argument for scale in the Intersects function? If not, what would be your recommendation about changing the scales of shapes? Just create a new shape for each scale or?
With friendly regards.

If you have a fixed number of scales (i.e., less than 10) it's definitely best to just create a shape for each scale. If you don't have a fixed number of scales I'll need a bit more context to know if this function would be helpful, as it's not obvious if there's a fast way to implement this


Grid: editable list issue

Hey Vitali,
I encountered a new issue with the grid.
But first i like to understand your vision on drop-down lists in grids.
The value of the dropdown field doesn't seem to be a simple id, but rather an object={id:...,text:...}.
This seems to be the case for setting & getting.
However when populating the grid, i set the field-value to an id integer instead of the {id:...,text....}
This seems to work as well, but is that the correct way???
And is there a mode where getting and setting works with pure id's instead of objects?
(so the text gets looked up dynamically from the source (items))
Fyi: i don't use the url to set and get the grid records, i use my own ajax mechanism,
(which loads multiple grids and forms simultaneously)
and i use grid.add for setting and grid.records for getting data.
When i want to save the grid, i convert the list field value from object to its id like field=field.id
The problem that i have is when i add a new blank row and want to select somthing from the list there is a js error, it is below in the screenshot.
I altered your code to include a check if oldvalue===null to make it work.
I think the reason it fails is because of the way i add a new record,
first i create it in the table, so it has an id, and after it is loaded into the grid with grid.add,
now the dropdownlist has a value of null instead of an object with {id:null:text:''}, so that's where w2ui gives an error unless i change the code as illustrated below!

Hi
how to add cascading dropdown column in w2grid.
mean when i select country than get state data and bind in ddl state than city data


[BUG] - Can't focus/type on Input when using it inside a Card with isPressable prop

Describe the bug
When using the Input component inside a Card component with the isPressable prop, I can't focus or click on the input to type. It works without the isPressable. Even if I disable the animations or ripple effect it still has the problem.
<Card isPressable isHoverable>
  <Card.Body>
    <Input placeholder="Add Tag" size="xs" />
  </Card.Body>
</Card>

Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

Create a Card component with the isPressable prop
Add an Input component inside it
Try to click/type in the Input

Expected behavior
Expected to be able to focus/type on the input
Screenshots or Videos
https://user-images.githubusercontent.com/25394736/196060014-384d66b8-3c2d-4cfb-939b-e331173e4325.mov
Operating System Version
macOS
Browser
Chrome

Hey please update to v2.0 https://nextui.org/blog/nextui-v2 due to v1 will not receive more updates.
https://nextui.org/docs/components/card
check the isPressable and allowTextSelectionOnPress props.


set the height of the cylinder

When I set the height of the cylinder, I can't use the variables I set before, I can only use float.
hfss["H"] = "1.6mm"
Cylinder = hfss.modeler.create_cylinder(cs_axis="Z", position=["L1", "0", "0"], radius=0.6, height="H", numSides=0, name="Feed", matname="pec")
height[float]
Height of the cylinder.

Hi @asuperwang,
What do you mean exactly?
I tested your code snippet and I was able to get a cylinder with the expected height:

@asuperwang I think you just need to modify the parameter value:
hfss["H"] = "1.6mm"
Cylinder = hfss.modeler.create_cylinder(cs_axis="Z", position=["L1", "0", "0"], radius=0.6, height="H", numSides=0, name="Feed", matname="pec")

hfss["H"] = "5mm"

Please let us know if you need anything else, or we will close the Issue

@asuperwang我认为你只需要修改参数值：
hfss["H"] = "1.6mm"
Cylinder = hfss.modeler.create_cylinder(cs_axis="Z", position=["L1", "0", "0"], radius=0.6, height="H", numSides=0, name="Feed", matname="pec")

hfss["H"] = "5mm"

如果您需要其他任何信息，请告诉我们，否则我们将关闭该问题

Thank you for answering my question,even though the editor shows"Expected type 'float' but got 'str'"I can still create the required model.


Can ConnectHOC be a PureComponent?

I was running into an issue earlier today where I noticed that one of my components was rerendering much more frequently than I expected it to.  It was hooked up to the ConnectHOC.  I swapped out my component implementation for one that used the Connect component, and my rerendering problems went away.  I checked, and ConnectHOC is extending Component instead of PureComponent.
I'm not a React expert, so I could be way wrong.  But I think the ConnectHOC should be able to be a PureComponent.  But if it can't, it would probably be good to define some kind of shouldComponentUpdate definition for it.
I'm happy to submit a PR to add it, but I always like to get feedback to make sure my proposed changes are wanted/make sense before I start working on a PR.

v1 is released now which means this issue was resolved / is out-of-date 👌
Some docs and a proper announcement for v1 are still in progress in pending. It's completely usable so far but the API has changed, so I have to ask for some more patience, but everything will be ready quite soon ✌


Add model and architecture for Omnivore model

Summary:
Omnivore is a vision multimodal model that able to classify RGB image, video, and depth image (RGBD) with a shared trunk parameters.
With this PR, we aim to have the model and architecture class for Omnivore that able to load converted pretrained weight from original author: https://github.com/facebookresearch/omnivore and produce same result.
Test plan:

Add notebook in examples/omnivore/LoadOriginalPretrainedWeightAndCompare.ipynb that load the original omnivore pretrained weight in torchhub and make sure the output is the same
Add unittest

Q: how did you make sure that the order of pretrained model keys is the same as that of your model keys?
For example, if I implement the same model in a slightly different way (e.g., additional variables, layers), will it be a problem for me to match the original implementation?
key_mapping = {pretrained_keys[i]: model_keys[i] for i in range(len(model_keys))}

Q: how did you make sure that the order of pretrained model keys is the same as that of your model keys?
For example, if I implement the same model in a slightly different way (e.g., additional variables, layers), will it be a problem for me to match the original implementation?

Sometimes this can be a problem, especially if we have branching operation. For instance in omnivore, the head that use ModuleDict need to have the same order as the original implementation or otherwise it will not match the correct key using that method (need a more complex way to auto assign like doing a matching with same size).
However from what I understand, if there is no branching (like the swin transformer encoder only, in fact the implementation here is different with the original one: https://github.com/facebookresearch/omnivore/blob/main/models/swin_transformer_3d.py ) usually the layout will follow the same order on how the input will be processed, and in this case it should be okay. Some of the problem that may occur here is that we might store same data in different dimension like 100 x 100 (2D) or 10000 (1D).

Codecov Report

Merging #43 (e7869d7) into main (3beffd9) will increase coverage by 0.31%.
The diff coverage is 97.11%.

@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
+ Coverage   88.96%   89.27%   +0.31%
==========================================
  Files          33       38       +5
  Lines        1722     2061     +339
==========================================
+ Hits         1532     1840     +308
- Misses        190      221      +31

Impacted Files
Coverage Δ

...al/modules/encoders/swin_transformer_3d_encoder.py
96.87% <96.87%> (ø)

torchmultimodal/models/omnivore.py
97.22% <97.22%> (ø)

torchmultimodal/architectures/omnivore.py
100.00% <100.00%> (ø)

torchmultimodal/architectures/clip.py
100.00% <0.00%> (ø)

torchmultimodal/models/albef.py
78.26% <0.00%> (ø)

...ultimodal/modules/encoders/albef_vision_encoder.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Issue #1323

Updated Links and text #1323

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.


feat(form)!: use native events for primitive components

Removed the HoC withinput on all primitive input components:

Checkbox
Date
Number
Pass
Radio
Select
MultiSelect
Text
TextArea

BREAKING CHANGE: these components now use React.SyntheticEvent in onchange prop parameter
This means using things like react-hook-form is somewhat easier, typing of the components is much better, and overall, it improves DX across the board by having expected event coming back from inputs components.

TODO : créer une branche v3 depuis la master pour merger cette PR


Minor build improvements

Minor build improvements:

#include <stdio.h> moved to ch32v003fun.h
added build_all.cmd to build all examples

@cw2 I am deliberately adding #include <stdio.h> so that people who look at the code are consciously aware they can just printf(...) from the code.  Like there is no technical reason to add it, but I wanted to express it from a intellectual standpoint.
Is there a technical reason to remove it?

I see, good point - no need to remove #include <stdio.h> then (I have removed/reverted the commit).
Also, I have split .option arch, +zicsr, will do a separate PR associated with the respective issue.

Blehhhhhhhh I can't get this to compile on windows with del /Q /F
What Make are you using?

I do not have any extra tools installed except what comes with the toolchain - there is make.exe in the bin directory...

I must be using a different make.  They appear to be mutually exclusive.  I spent about an hour trying to get something to autodetct if I even could execute del but I couldn't figure it out.  Maybe I need to do it the other way around, i.e. see if I can execute rm.
Well, don't install anything extra.  I want to figure this out.


Upgrade Velocity version

Describe the feature
velocity version used is 1.7 which is old,
Can upgrade to velocity-engine-core version 2.0 latest stable version
Additional context
maven dependency:

org.apache.velocity
velocity-engine-core
2.0

Would you like to submit a PR?

When I fork the project, I don't see any velocity reference, can you advice?

The dependency is in the pom.xml file: https://github.com/tipsy/javalin/blob/master/pom.xml#L95-L100


Document SQLite function translations

Note that some of these are over-simplified for the purposes of communication.
Part of #504

Hmm, it's probalby worth breaking the table up into sections. Something like...

Binary functions
Date and time functions
Numeric functions
String functions

Generally should break on the type. That is generally followed in various database documentations.

Breaking on type is not exactly individual types but rather groups - numeric/ datetime/ string etc. Anyway I can update when I "add" other providers.


Public constructors for extension

The constructors for HiveMetaStoreJUnitExtension and HiveServer2JUnitExtension are package-private, so they cannot be used outside the com.hotels.beeju.extensions package. This PR fixes that problem.

@mateoi BeeJU 3.0.1 has been released and will appear on Maven soon! Thank you for the PR :)


Test main API functions

We should test that API calls are filling the correct data structures and checking for error codes when something goes wrong.

Check this for mocking command line methods
https://gianarb.it/blog/golang-mockmania-cli-command-with-cobra


Updates Should statements in Pester tests to use dashes before parameters

Pull Request (PR) description
Updates Should statements in Pester tests to use dashes before parameters
This Pull Request (PR) fixes the following issues
None
Task list

[x] Added an entry under the Unreleased section of the change log in
CHANGELOG.md. Entry should say what was changed, and how that affects
users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as
appropriate.
[ ] Examples appropriately added/updated.
[x] Unit tests added/updated. See
DSC Resource Testing Guidelines.
[x] Integration tests added/updated (where possible). See
DSC Resource Testing Guidelines.
[x] New/changed code adheres to
DSC Resource Style Guidelines and Best Practices

This change is

Codecov Report

Merging #564 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##            dev   #564   +/-   ##
===================================
  Coverage    74%    74%
===================================
  Files        27     27
  Lines      4030   4030
  Branches      4      4
===================================
  Hits       3009   3009
  Misses     1017   1017
  Partials      4      4

Hi @mhendric - I've got an issue raised for this. It is to meet Pester 4 standards (the dashes are part of the v4 standard). However there are some additional things that need to be met for v4. So perhaps we shouldn't close the v4 in part of this as we do still need a bunch of work on the tests. What do you reckon?

Hey @PlagueHO , yeah that's what I was thinking. This PR will help us get closer to v4 standards, but there's still work that needs to be done.


HTML5 number input attributes seem split across multiple interfaces

Let's say I want a number input like this:
<input
    id="size"
    name="size"
    type="number"
    min="10" step="10" value="100"
    required="true">

It seems that the only way to do this (Python 3, WTForms 2.0.2) is like so:
#!/usr/bin/env python
from wtforms import Form
from wtforms.fields.html5 import IntegerField
from wtforms.widgets.html5 import NumberInput

class TestForm(Form):
    size = IntegerField(default=100, widget=NumberInput(step=10))

print(TestForm().size(min=10, required=True))

This seems very strange to me — one attribute (default) is right there in the field. Another one (step) must be given as an argument to an argument of the field, overriding the default widget. Another two can only be given when the widget is rendered.
None of these attributes seem to be any more fundamental or UI specific than any others. They are all simply UX enhancements implemented by browsers. It took a fair few hours of searching through the documentation just to figure out how to do this.
Am I doing this wrong? If not, is there some overarching logic to this that would help guide me when trying to construct such forms?

Looking at widgets.html5.py:84, it might be sensible just to add the allowed HTML5 attributes (min, max) there alongside step, and maybe required should go in a common HTML5 input widget base... would such a change be useful?

This should be resolved now in #233


Don't toggle visibility switch on adding/removing layer

Fixes #4093
This corrects a previous change d066c9d6a21a55f8056207dadc396a65dae4f3eb that introduced the bug; when a layer was added/removed it faulty changed the visibility switch state to the new layer, rather than the layer that the current layer the view represents.
Now, the add/removal only maybe updates the visibility of the switch, but not the actual switch state (that is still handled by the event listener on the own layer model the view represents).
ping @javisantana

haha, a 1.5years old bug :es:

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)


New pio build targets

This adds a couple of targets to the platformIO config, specifically the funhouse, qtpys3_n4r2 and esp32s3devkitc.
Also adds a CI build target for the debug build of reverse tft S3 feather, superseding #599

@brentru this is ready for review


fix(docs): fix getUpdateComplete example in lifecycle

I copied the example into my code and got an type error. This aligns the docs with the type description.

Thanks @eins78!
We need the CLA signed though, even for small changes.

@justinfagnani CLA is signed


SUBMARINE-213. [SDK] Add pysubmarine-tracking example in mini-submarine

What is this PR for?
Add a tracking api example in mini-submarine
What type of PR is it?
[Improvement]
Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-213
How should this be tested?
https://travis-ci.org/pingsutw/hadoop-submarine/builds/599649334
Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

LGTM.

Thanks @liuxunorg for the review


fix: fail to output json with batch option 'aws'

duck fails to output json report when error occurs.
The reason is that it cannot parse message including 'logUrl' to JSON.
Error message is like the below. In this case, the line : https://ap-northeast-1.console... is logUrl.
Unexpected non-JSON error: /opt/nodejs/node_modules/google-closure-compiler-linux/compiler --rewrite_polyfills=true ...
[{"level":"error","description":"The result of a goog.define call must be assigned as an isolated statement.", ...},{"level":"info","description":"70 error(s), 0 warning(s)"}]

: https://ap-northeast-1.console.aws.amazon.com/cloudwatch/home?region=ap-northeast-1#logEventViewer:group=%2Faws%2Flambda%2Ffaast-b0d763dd-d988-4d38-a357-485f28d8ef5e;stream=2020%2F01%2F30%2F%5B%24LATEST%5D12d386dbf1fe4ca9933e07671ca6956c;filter=%22575f34df-23ac-47a3-b0d9-9bb5f8a081e0%22

logUrl has been included since v3.0.14.
see the commit: https://github.com/faastjs/faast.js/commit/4c65cffa1e522e886f3948ee85b1943925e78281
So, this problem does not happen before v3.0.13.
FaastError document indicates the below.
The logUrl will be appended to the end of FaastError.message and FaastError.fullStack as well.
This PR deletes 'logUrl' from error message to be able to parse it to JSON.

Note: the colon separated error message format for nested error objects is created by verror.

@teppeis
I have some ideas. It was hard to choose one for me because I didn't know what is better. If you don't mind, please see the ideas and give me your idea if you have.

@shisama Thank you for your great work and proposals!!! That's awesome 👍
I prefer the approach of 4. it seems the most rigid approach for me because the message might be added other than logUrl in the future release.
https://gist.github.com/shisama/d182500cf8f70344fea36a4ecc355e10#gistcomment-3164710
I feel that we should isolate code for faastjs, but it would require a lot of works.
So it would be better to work on it as another task.
But getting a command string and CompilerErrorItem[] has already messed up, so I feel that it would be nice if the implementation is separated as a function like the following.
const { command, items } = parseErrorReason(result.reason, config);
:
function parseErrorReason(
  // FIXME
  reason: unknown,
  config: DuckConfig
): { command: string; items: CompileErrorItem[] } {
  const { message: stderr, info } = reason as CompilerError & {
    info?: Record<string, any>;
  };
  const message = config.batch === "aws" && info ? info.message : stderr;
  const [command, , ...messages] = message.split("\n");
  return { command, items: JSON.parse(messages.join("\n")) };
}

Codecov Report

Merging #378 into master will decrease coverage by 0.48%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #378      +/-   ##
==========================================
- Coverage   50.72%   50.24%   -0.49%
==========================================
  Files          24       24
  Lines        1029     1039      +10
  Branches      184      186       +2
==========================================
  Hits          522      522
- Misses        468      478      +10
  Partials       39       39

Impacted Files
Coverage Δ

src/commands/buildJs.ts
16.84% <0%> (-1.99%)
:arrow_down:

@shisama Thank you! Could you fix the conflicts?

@koba04 Thanks. I rebased this and resolved the conflicts.

@teppeis Could we merge this PR? Your review status isn't in approval yet.

@shisama @koba04 LGTM 👍


Better error message for tag filtering mismatch

When tag filtering filters out all of the op templates in a workload, there are some specific cases which we could detect and provide better feedback on:

a tag name was specified, but none of the op templates have that tag name.

This is not an error per se, as it might be a valid scenario for automation
It is likely an error if it is driven from user behavior.
For cases like this, we don't want to interrupt the scenario with an error, but we do want to inform the user.
It is possible to use levenschtien distance to match what was intended and provide a prescriptive solution.

a tag name and literal tag value are specified, the tag name matches, but the tag value does not and there are no op templates which could match the tag value.

as with the above scenario, this may not always be an error. (Imagine automation over many files which are not in consideration here)
As above, we would like to inform the user with a useful error.

This should be handled with the same mechanism as in #12

We currently are finding an error here. The change requested is fold that message through NBAdvisorOutput.test


Do not handle uncaught exceptions

When winston is configured to handle uncaught exceptions, all uncaught
exceptions will result in a LoggerNoContext error being thrown, which
obscure the real source of the error and makes it very hard to debug
problems. Additionally, this behaviour interferes with application level
exception handling.
Change-type: major
Signed-off-by: Lucian Buzzo<EMAIL_ADDRESS>

@balena-ci rebase

@balena-ci I self-certify!


Always escape when writing pubspec.yaml's 'description' field.

Closes https://github.com/flutter/flutter/issues/80013.
Before:
$ flutter create test1 --description "a: b"
Creating project test1...
Error detected in pubspec.yaml:
Error on line 2, column 15: Mapping values are not allowed here. Did you miss a colon earlier?
  ╷
2 │ description: a: b
  │               ^
  ╵
Please correct the pubspec.yaml file at /Users/matan/Developer/scratch/test1/pubspec.yaml

After:
$ flutter create test1 --description "a: b"
Creating project test1...
Resolving dependencies in test1...
Got dependencies in test1.
Wrote 129 files.

All done!
You can find general documentation for Flutter at: https://docs.flutter.dev/
Detailed API documentation is available at: https://api.flutter.dev/
If you prefer video documentation, consider: https://www.youtube.com/c/flutterdev

In order to run your application, type:

  $ cd test1
  $ flutter run

Your application code is in test1/lib/main.dart.

It's worth noting that this always escapes a non-empty project description, which means that descriptions that were not previously wrapped in "s' will be. I'm not sure how worth it is to do a conditional escape (i.e. only escape if not escaping would cause a problem), but willing to change.
Side-note: I had no idea where to list this test in the (very large) create_test.dart, so I did my best :)

Thanks!

@matanlurey btw, you needed two org member approvals and for one of them to add the autosubmit label because you are not a flutter GitHub org member. Would you like me to nominate you for membership? The process is: https://github.com/flutter/flutter/wiki/Contributor-access


one on one chat implementation using chime sdk

What demo do you have issue with?
To setup the one on one messaging by using Amazon Chime SDK(direct message).
please provide a documentation for implementing direct messaging

@charantejapandalaneni if you're referring to data messages, please refer to this documentation: https://aws.amazon.com/about-aws/whats-new/2020/05/amazon-chime-sdk-data-messages-real-time-signaling/
There are additional links at the bottom to help you get started.

@michhyun1 i am looking for one on one chat feature(direct messaging to a person) without creating a channel so that we can communicate directly to end user.  any documentation for this?

You would still create a channel (probably private) but then only add the two app-instance-users who will be members of the channel.  I think you are correct that this would be good to demo, but using aws cli (with admin creds) it would be something like the following:
Create users (typically done on first user sign in, for directions uuids are user1 and user2)
aws chime create-app-instance-user \
    --app-instance-arn "APP_INSTANCE_ARN" \
    --app-instance-user-id "user1" \
    --name "backend-worker"

aws chime create-app-instance-user \
    --app-instance-arn "APP_INSTANCE_ARN" \
    --app-instance-user-id "user2" \
    --name "user2"

Create channel and add members
We will create it from user1, which will auto add user1 as a member.  user1 will then add user2 as a member. Alternatively you could create it from the backend and then have the backend add user1 and user2.  If the backend were to do it then neither user1 and user2 would have permissions to add other users.
aws chime create-channel \
    --chime-bearer "APP_INSTANCE_USER_ARN/users/user1" \
    --app-instance-arn "APP_INSTANCE_ARN" \
    -- type PRIVATE \
    --name "firstChannel"

note: channel is PRIVATE so it isn't discoverable by others
aws chime create-channel-membership
--channel-arn <CHANNEL_ARN_VALUE_RETURNED FROM ABOVE CALL> \
--member-arn "<APP_INSTANCE_USER_ARN>/users/user2" \
--chime-bearer "<APP_INSTANCE_USER_ARN>/users/user1" \
--type DEFAULT

Send / list messages where app-instance-user arn can be user1 or user2 (in form <APP_INSTANCE_USER_ARN>/users/user2), any other user would get 403
aws chime send-channel-message \
    --chime-bearer "APP_INSTANCE_USER_ARN" \
    --channel-arn "CHANNEL_ARN" \
    --content "hello world" \
    --type STANDARD \
    --persistence PERSISTENT

aws2 chime list-channel-messages \
    --chime-bearer "APP_INSTANCE_USER_ARN" \
    --channel-arn "CHANNEL_ARN"

If the users were connected via websocket for the duration of this call, they would get realtime notifications of channel membership added to new channel, and the channel messages when sent

Closing due to inactivity. Feel free to reopen if you have further questions.


Add shard proxying and add (1-shard) sharded CI job with front-proxy

add shard awareness to front-proxy, watching shards and workspace assignment and then reverse-proxying to the right shard URL
add CI job with front-proxy and tests running against the front-proxy

Note: this is still a one-shard setup. But it gives us the test setup to run with 2 or more shards.

/test boilerplate

/retest

/retest

/test e2e

@ncdc

/test test


[Event Request] Table 5406 Prod. Order Line - Function DeleteRelations

Dear ALAppExtensions Team,
We would like to request the following event publisher in table 5406 - Function DeleteRelations, of course without the prefix DIN.
begin
    OnBeforeDeleteRelations(Rec);

    ProdOrderLine.SetRange(Status, Status);
    ProdOrderLine.SetRange("Prod. Order No.", "Prod. Order No.");
    ProdOrderLine.SetRange("Routing No.", "Routing No.");
    ProdOrderLine.SetFilter("Line No.", '<>%1', "Line No.");
    ProdOrderLine.SetRange("Routing Reference No.", "Routing Reference No.");
    if not ProdOrderLine.FindFirst then begin
        ProdOrderRoutingLine.SetRange(Status, Status);
        ProdOrderRoutingLine.SetRange("Prod. Order No.", "Prod. Order No.");
        ProdOrderRoutingLine.SetRange("Routing No.", "Routing No.");
        ProdOrderRoutingLine.SetRange("Routing Reference No.", "Routing Reference No.");
        if ProdOrderRoutingLine.FindSet(true) then
            repeat
                ProdOrderRoutingLine.SetSkipUpdateOfCompBinCodes(true);
                ProdOrderRoutingLine.Delete(true);
            until ProdOrderRoutingLine.Next() = 0;
    end;

    ProdOrderComp.Reset();
    ProdOrderComp.SetRange(Status, Status);
    ProdOrderComp.SetRange("Prod. Order No.", "Prod. Order No.");
    ProdOrderComp.SetRange("Prod. Order Line No.", "Line No.");
    // DINNL <<
    DINDeleteRelationsOnBeforeProdOrderCompDeleteAll(ProdOrderComp, Blocked, DINIsHandled);
    if not DINIsHandled then
    // DINNL >>
    ProdOrderComp.DeleteAll(true);

    if not CalledFromComponent then begin
        ProdOrderComp.SetRange("Prod. Order Line No.");
        ProdOrderComp.SetRange("Supplied-by Line No.", "Line No.");
        OnDeleteRelationsNotCalledFromComponentFilter(Rec, ProdOrderComp);

local procedure DINDeleteRelationsOnBeforeProdOrderCompDeleteAll(var ProdOrderComponent: Record "Prod. Order Component"; Blocked: Boolean; var IsHandled: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[v1] fix: mark some peer dependencies as optional and remove some unnecessary peer dependencies

v1 version of #1093
Not 100% sure about peer dependency version ranges. Let me know if some of those need to be extended.

@splincode demo app build seems to be failing because of some missing dependencies in the main package.json.
https://github.com/taiga-family/tui-editor/actions/runs/9446429617/job/26016105773


Update sbt-wartremover, wartremover to 2.4.13

Updates

org.wartremover:sbt-wartremover
org.wartremover:wartremover

from 2.4.12 to 2.4.13.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.wartremover" } ]

labels: library-update, semver-patch

Resolved by #181


Can you make it so we have the Riot client API port/auth-token in global variables or similar ?

Hello again !
In ranked champ select the player names are hidden. I want to make a plug-in that unhides them. I found that it's possible to retrieve the real usernames by using the Riot client API, which is different to the LCU, with different endpoints.  (The interesting one being /chat/v5/participants/champ-select)
It uses a different port, and use an auth-token, which changes everytime the client starts up.
On Window you can retrieve the port by using in a terminal the command : wmic PROCESS WHERE name='LeagueClientUx.exe' GET commandline
In Mac OS you can do : ps -A | grep LeagueClientUx
I searched a little bit, and apparently it's not possible to do these commands in javascript without nodejs.
So I was wondering, I'm not sure exactly how league-loader works, but I very much suppose there's some sort of "pre-load" that uses C/C++ or another low-level language, which could be used to retrieve the Riot client API/auth-token, and made them available in global variables in the DevTools
I put a red arrow on the two informations here :

It's quite easy. Just use this API:
GET /riotclient/command-line-args

The response is the command line of LeagueClient.exe.
You have to parse them to get auth token and password.
// up-to-date APIs: http://www.mingweisamuel.com/lcu-schema/tool/#/

/riotclient/command-line-args

Once again, thank you, you're spot on everytime

Riot Client APIs have response header with origin diff to LCU port, so any request will fail due to CORS.
You can bypass them by disabling CEF web security in INSECURE_OPTIONS.
Next update I'll add something magical to the network layer to bypass them.


fix: Add check before activating service accounts in Kaniko build jobs

Context
With the changes in PR #357, there is a need to make additional changes to the Dockerfiles of the Kaniko image building templates - instead of authenticating immediately with GOOGLE_APPLICATION_CREDENTIALS, we now first check if that env var is defined, if so we will authenticate with the credentials stored in the file path pointed to by that env var; otherwise, the gcloud auth activate-service-account will not be run in the Dockerfile. In the second scenario, the Google service account credentials are expected to be passed to the Kaniko image building jobs via a mounted Kubernetes service account.
A similar change has been implemented in Merlin: https://github.com/caraml-dev/merlin/pull/352/files#diff-931d889c572814da3e253178b316471ac9878c45bf0497dd1792b5442eab359cR24

Thanks for the quick review! :D Merging this now!


Make rubydoc.brew.sh workflow more like formulae.brew.sh

Verification

[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

Provide a detailed description of the proposed feature
Homebrew/rubydoc.brew.sh has a "Scheduled regeneration" GitHub Action which commits changes to the repository and pushes them. This is a bit like what Homebrew/formulae.brew.sh used to do, too.
Instead, it should make use of the GitHub Pages v4 deployment mechanism which doesn't require generated files to be committed.
This can also mean that PRs are required on this repository.
What is the motivation for the feature?
Enabling additional security features on Homebrew/rubydoc.brew.sh.
How will the feature be relevant to at least 90% of Homebrew users?
It won't be.
What alternatives to the feature have been considered?
(note: opening on Homebrew/brew for improved visibility)

This was done, thanks @Rylan12!


[AutoComplete] use sublime text like search instead of Levenshtein Distance for fuzzy search

The problem was that fuzzy search used Levenshtein Distance for matching.
It resulted that Peach was not matched by ea.
So i use search similar to the sublime text fuzzy search in ctrl + p also inspired by https://github.com/mattyork/fuzzy
Basically pattern will match if you will get it from search text by deleting some letters.
If someone would like Levenshtein Distance there is still filter for them.

This is for issue #4159

@ddeath The logic looks good to me. Thanks for adding some tests! Could you have a look at my comments?

@oliviertassinari I already did :smile:

@ddeath Awesome.
Could you squash down the commits and use this PR name in the commit message?

@oliviertassinari done

@ddeath can you add a couple more tests (with slightly longer and more complex strings)? A a couple of  "close but no cigar" non-matches would be good too 👍  Would be cool to have the limitations documented.

@nathanmarks
Hi I added some more test, but I could not came up with anything else, since it appears to be quite simple algorithm...
If you have any ideas feel free to share it with me I can write some more tests then.

@ddeath great, thanks!

@ddeath Thanks!


⚠ AZ-Launcher has degraded performance

In 832aaa3, AZ-Launcher (https://az-launcher.nz/) experienced degraded performance:

HTTP code: 200
Response time: 16736 ms

Resolved: AZ-Launcher performance has improved in ccaff73 after 30 minutes.


chore(main): release 0.0.3

:robot: I have created a release beep boop
0.0.3 (2023-04-02)
Bug Fixes

change to goreleaser action (#3) (94dca82)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/others-git/watch-dawg/releases/tag/v0.0.3 :sunflower:


dev/ttyACM0 not found and other errors

zigbee2mqtt add-on version (if edge, please report commit hash):
version 1.15.0-dev (commit #fee6310)

Operating environment (HassOS, Virtual Machine, Device/platform running Home Assistant):

Supervised install on an intel nuc running Ubuntu
Description of problem:
When starting the addon reutrns some errors. If I try to restart it the error changes and becomes "dev/ttyACM0 not found. However if i unplug the CC stick and plug it back in again it appears in the listed serial devices, and if I try to start the addon it seems to start but still gives errors
Your entire configuration from the frontend (with sensitive fields redacted):
{
  "data_path": "/share/zigbee2mqtt",
  "external_converters": [],
  "devices": "devices.yaml",
  "groups": "groups.yaml",
  "homeassistant": true,
  "permit_join": false,
  "mqtt": {
    "base_topic": "zigbee2mqtt",
    "server": "mqtt://<IP_ADDRESS>",
    "user": "mqttusername",
    "password": "mqttpassword"
  },
  "serial": {
    "port": "/dev/ttyACM0",
    "disable_led": true
  },
  "advanced": {
    "log_level": "info",
    "pan_id": 6754,
    "channel": 11,
    "network_key": [
      1,
      3,
      5,
      7,
      9,
      11,
      13,
      15,
      0,
      2,
      4,
      6,
      8,
      10,
      12,
      13
    ],
    "availability_blocklist": [],
    "availability_passlist": []
  },
  "device_options": {},
  "blocklist": [],
  "passlist": [],
  "queue": {},
  "frontend": {
    "port": 8099
  },
  "experimental": {
    "new_api": true
  },
  "socat": {
    "enabled": false,
    "master": "pty,raw,echo=0,link=/dev/ttyZ2M,mode=777",
    "slave": "tcp-listen:8485,keepalive,nodelay,reuseaddr,keepidle=1,keepintvl=1,keepcnt=5",
    "restartdelay": 1.0,
    "initialdelay": 1.0,
    "options": "-d -d",
    "log": false
  }
}

Your logs from Home Assistant
if it finds the adapter after it has been replugged: https://pastebin.com/PDU8rcu7
if I restart it after (the adapter doens't show up anymore in the serial devices): https://pastebin.com/NURTaBC0

@igiul667 I'd recommend you try and use the named serial port:  Supervisor → System → Host system → ⋮ → Hardware. It should look something like:  /dev/serial/by-id/usb-1a86_USB_Serial-if00-port0
Also try using the add-on from: https://github.com/zigbee2mqtt/hassio-zigbee2mqtt as it has a few startup improvements (to test things out).

Thanks for the reply, I'll try what you're telling me soon

Using the release version of zigbee2mqtt everything seems to start correctly however I'm unable to send commands to the zigbee devices. Using the edge version addon it also starts no problem, but as soon as i try pressing a button on a zigbee remote the addon crashes with error:
Zigbee2MQTT:error 2020-11-03 08:26:47: Adapter disconnected, stopping
Zigbee2MQTT:info  2020-11-03 08:26:47: MQTT publish: topic 'zigbee2mqtt/bridge/state', payload 'offline'
Zigbee2MQTT:info  2020-11-03 08:26:47: Disconnecting from MQTT server
Zigbee2MQTT:error 2020-11-03 08:26:47: Failed to stop zigbee
I was thinking maybe I need to reflash the coordinator, do you think that would help?

Might be that you need to do so. From the looks of it you either have:

adapter problem
virtualization problem - if you have some sort of passthrough for the serial adapters
driver problems ? unsure how your intel nuc setup is on this part

I think I'll have to reflash the adapter, since it was working, but after a reboot it stopped, so I wouldn't think the issue is with the host. I'll let you know how that goes

Fixed everything by reflashing the coordinator. I've had some prolems binding new items but I managed to do it.


SDPPP button/icon on top of inpaint icons

You can't change the size of your brush because the SDPPP is there.

this icon is able to be dragged.

On desktop, yes, but not on mobile.

On desktop, yes, but not on mobile.

I'll support it later


reset all views

Hi, is there something like nv1.reset() which sets all camera positions back to after loading? If not, could we please add it? I looked at nv1.screenSlices but it looked complex.

@haehn if you try the latest commit to the thresholding branch you will see a new option nv1.setDefaults(). I think this name is more consistent with other functions than nv1.reset(), but do not have a strong opion on this. I added buttons to demonstrate this if you run npm run dev as well as npm run demo (the connectome demo for the latter).
Note that it resets the options, but does not change values in your web page. For example, if you have a checkbox that allows the user to toggle isRadiologicalConvention, you will want to make sure it reflects the updated options.

Thank you!


Solve #45 - Update new table structure

Closes #45
Closes #46
Context
The new structure for the PostgreDB shared storage table was updated on this PR. Also, a new dynamic to keep the asynchronous process was added, where when a bot uses a record from the shared storage, the record is marked as "in process", and after is processed is marked as "processed".

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

314 of 331 (94.86%)  changed or added relevant lines in 36 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.0%) to 98.9%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/bas/bot/fetch_birthdays_from_notion.rb
8
9
88.89%

lib/bas/bot/fetch_next_week_birthdays_from_notion.rb
8
9
88.89%

lib/bas/bot/base.rb
17
21
80.95%

lib/bas/write/postgres_update.rb
11
22
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-1.0%

Covered Lines:
1709

Relevant Lines:
1728

💛  - Coveralls


Revert "ALPN upgrade to http/2.0 (#5618)"

Revert until we figure out a certain way of knowing if the other side supports h2.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mandarjog
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [mandarjog]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mandarjog: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
9100a07288fb612f69e7ad6236631f23d667b648
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests

@mandarjog: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
9100a07288fb612f69e7ad6236631f23d667b648
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mandarjog: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
9100a07288fb612f69e7ad6236631f23d667b648
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
9100a07288fb612f69e7ad6236631f23d667b648
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mandarjog: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
9100a07288fb612f69e7ad6236631f23d667b648
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
9100a07288fb612f69e7ad6236631f23d667b648
link
/test e2e-simple

prow/istio-pilot-e2e.sh
9100a07288fb612f69e7ad6236631f23d667b648
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests

/test e2e-simple

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mandarjog, rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [mandarjog,rshriram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Force merging as this fixes some correctness issues. Test failures are flaky.


B 20526 main  - Backend | Sort Customer dropdown of Counseling Offices

Agility ticket
Summary
The list from which the customer makes their counseling office selection will be sorted by nearest to farthest from origin duty location.
How to test

Login as a customer, create a move with a non-primed counseled duty location (Scott AFB, IL 62225)
Verify the counseling options presented are displayed from closest to furthest distance and in accordance with the names in the screenshot below.

Integration PR
Integration PR INT-B-20526

This has a couple extra files in this PR than your INT PR.

This has a couple extra files in this PR than your INT PR.

great catch


UI is using SimulationDatasetId instead of DatasetId

Checked for duplicates
No - I haven't checked
Is this a regression?
Yes - This worked in a previous version
Version
2.1.0
Describe the bug
This bug was likely introduced in #1017, L305 of +page.svelete
Instead of grabbing the actual dataset_id from the simulation dataset, it is treating the simulation_dataset_id as dataset_id. This causes problems when the dataset_id and simulation_dataset_id fall out of sync (for example, via adding external datasets). Once this desync happens, the UI will either fail to get any results for a sim run, or will fetch incorrect results.
Reproduction

Create a plan
Add an external dataset via the API
Simulate the plan

Example video on a localhost venue where the desync was caused by running the external dataset E2E tests in Aerie core:
https://github.com/NASA-AMMOS/aerie-ui/assets/62574120/b2100b6d-c6c7-4792-a27d-9f9d9ca30f68
Database entry for this request. Notice how the dataset id does not equal the request's id:

Query that the UI made to get the spans. Notice that it used 107, the request id, instead of 133, the dataset id

Logs
No response
System Info
MacOs, Firefox

Severity
Major

This is a blocking bug for Clipper.


Create nano tutorial

Added logos to associate different editors.
Added tutorial for nano text editor.
closes #6
closes #12

@xuwenzhi is this good to go?

@xuwenzhi is this good to go?

LGTM!

😃


GLQUAKE brightness issues

So, glide wrappers has always had an issue with adjusting the brightness of glquake, so this is not a new issue, but I was wondering if you could take a look at the issue. Glquake will just not change brightness properly. And I know it could work because I tried it on 86box.
But then again, I really don't know anything currently.

also, have you considered making a discord server?


[T3A2][T16-A1] Alwinson Au-yong

Ready to Review

@alwinsonauyong closing this PR as there is nothing much to review,


User dialog validation logic steals focus while trying to correct warnings

Azure Data Studio Version: main

Steps to Reproduce:

Open Create User dialog
Change to Contained user with SQL Login auth
Click Script button with empty passwords
Try to fix the passwords by typing two new values in text boxes

Expected: The warning can be corrected.
Actual: After each character is typed into password text box focus is stolen and moved to the warning on dialog.  This makes it very difficult to correct the errors.

I also noticed this yesterday and the fix is in https://github.com/microsoft/azuredatastudio/pull/22781/files#diff-79b7efb31cab44e8b386bea529ce174e39c3f4a601c3acb8e4494e321e3c860dR615, I'll send out a seperate PR to fix the issue.


42 USA, Freemont

Institution/School Name
{42 USA}
Instiution/School Website
{http://42.us.org}
Email Users
Please place an x between the square brackets if yes

[x] Students<EMAIL_ADDRESS>[x] Academic/Teaching Staff<EMAIL_ADDRESS>[ ] Other/Support Staff

Education Levels
Please place an x between the square brackets if yes

[x] Non-Profit coding University
[ ] Post-graduate research
[ ] Graduate School
[ ] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

Hi,
It would appear that you have opened a pull-request containing commits from JetBrains/swot which is a fork of this repository (leereilly/swot).
If you intended to create a pull request for JetBrains/swot, may I suggest you create a pull request there.
We cannot accept your pull request if it contains more than one institution, as is the case here.
If you want your institution to be included please follow the following steps:

Delete your fork of swot
Create a new fork of swot based upon this repository
Create the file (and directories if necessary) to add the institution to your fork
Open a new pull request against this repository to merge your fork.

I am closing this pull request, as it is for the wrong repository.


THEEDGE-1633 Convert .Fatal() calls to LogErrorAndPanic() and ginkgo …

…stub files
Description
Changed *.Fatal() calls to log errors, flush buffers, then panic in a consistent way [LogErrorAndPanic()]
Added stub ginkgo (BDD) files
Fixes # THEEDGE-1633
Type of change
What is it?

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[X] Tests update
[ ] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[X] I run go fmt ./... to check that my code is properly formatted
[X] I run go vet ./... to check that my code is free of common Go style mistakes

Codecov Report

Merging #688 (924259e) into main (3c1c5a3) will decrease coverage by 0.70%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #688      +/-   ##
==========================================
- Coverage   32.88%   32.18%   -0.71%
==========================================
  Files          41       42       +1
  Lines        4345     4440      +95
==========================================
  Hits         1429     1429
- Misses       2718     2813      +95
  Partials      198      198

Impacted Files
Coverage Δ

main.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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