Enable Disk Partition Resize when Size not specified - Fixes #162

Pull Request (PR) description

Disk:

Corrected message strings.
Added message when partition resize required but AllowDestructive
parameter is not enabled.
Fix error when Size not specified and AllowDestructive is $True
and partition can be expanded - Fixes Issue #162.
Fix incorrect error displaying when newly created partition is not
made Read/Write.

This Pull Request (PR) fixes the following issues

Fixes #162

Task list

[x] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md in the resource folder.
[ ] Resource parameter descriptions added/updated in schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[x] 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.

@johlju - would you mind reviewing when you have a moment?

This change is

Codecov Report

Merging #164 into dev will increase coverage by <1%.
The diff coverage is 100%.

@@         Coverage Diff         @@
##           dev   #164    +/-   ##
===================================
+ Coverage   94%    94%   +<1%
===================================
  Files        6      6
  Lines      784    795    +11
===================================
+ Hits       741    754    +13
+ Misses      43     41     -2

Codecov Report

Merging #164 into dev will increase coverage by <1%.
The diff coverage is 100%.

@@         Coverage Diff         @@
##           dev   #164    +/-   ##
===================================
+ Coverage   94%    94%   +<1%
===================================
  Files        6      6
  Lines      784    795    +11
===================================
+ Hits       741    754    +13
+ Misses      43     41     -2


fix not work

check can work normally, but the fix function does not respond. There are many same problems in the issue, please find out why.

Same here


Support Quake 2 & 3 BSP files

There are many BSP formats but a good starting point would be Q2 and Q3.
Some open questions:
Where are some resources for learning about the differences between the formats?
Is this outside the scope of this project?
Other possible BSP formats https://github.com/zturtleman/bsp-sekai

Definitely within the scope of this project!  I've vaguely started supporting other formats but didn't get very far as it wasn't a priority for my project.
One thing to be careful of is that most of the tools and engines out there that read and write BSP files are under the GPL license, so copying code from them isn't compatible with the MIT license.  There are some resources floating around explaining the formats that were from reverse engineering the format (like before the source was released).
For example, this info about the Quake 2 format came out before the source code was released: https://www.flipcode.com/archives/Quake_2_BSP_File_Format.shtml

Is it ok to use Valve map format in TB? It seems like qbsp has no problems with it, and my test maps seem to work fine as well. Valve format supports better texture projections, so you can transfer textures between faces much better with it, and it doesn't create brushes with some textures inverted as the standard format does (noticeable with textures that have text in them, like triggers, clips, etc).
It seems qbsp limits filenames to 15 characters, though, and I don't know if that can be changed.
WARNING: texture DiamondPlate008C name too long for Quake miptex

Yep.  That only impacts the .map file, so if you have a modern qbsp compiler that can handle the valve format, it's fine.
And, yeah, the texture names are pretty limited, which is a down side.  I think the Q2 format supports longer texture names, but q2bsp support would have to be implemented, which might take a while.
If you wanted to have more properly named materials, you could use longer names for the materials, shortened names for the quake texture names, and use the material remap dictionary on the importer, but that's kind of yucky.  In this case, you could strip off a 0 or two.  Probably not going to have more than 99 diamond plates. :D

this is a year too late probably, anyway https://github.com/jitspoe/godot_bsp_importer/pull/24


Install grpc error : Could not find a package configuration file provided by "modules"

Host Environment

OS: wsl2 ubuntu20.04
Compiler: gcc9

To Reproduce
Steps to reproduce the behavior:
./vcpkg install grpc
Failure logs
Install successfully :
...
The package grpc:x64-linux provides CMake targets:

    find_package(gRPC CONFIG REQUIRED)
    # Note: 7 target(s) were omitted.
    target_link_libraries(main PRIVATE gRPC::gpr gRPC::grpc gRPC::grpc++ gRPC::grpc++_alts)

    find_package(modules CONFIG REQUIRED)
    target_link_libraries(main PRIVATE re2::re2 c-ares::cares)

Additional context
I add find_package(modules CONFIG REQUIRED) to CMakeLists.txt , I raise error :
[main] Configuring folder: test
[proc] Executing command: /usr/bin/cmake --no-warn-unused-cli -DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE -DCMAKE_BUILD_TYPE:STRING=Debug -DCMAKE_C_COMPILER:FILEPATH=/bin/gcc-9 -DCMAKE_CXX_COMPILER:FILEPATH=/bin/g++-9 -H/home/zhaodachuan/test -B/home/zhaodachuan/test/build -G "Unix Makefiles"
[cmake] Not searching for unused variables given on the command line.
[cmake] /home/zhaodachuan/test/vcpkg/scripts/buildsystems/vcpkg.cmake
[cmake] -- Found RE2 via CMake.
[cmake] CMake Error at vcpkg/scripts/buildsystems/vcpkg.cmake:857 (_find_package):
[cmake]   Could not find a package configuration file provided by "modules" with any
[cmake]   of the following names:
[cmake]
[cmake]     modulesConfig.cmake
[cmake]     modules-config.cmake
[cmake]
[cmake]   Add the installation prefix of "modules" to CMAKE_PREFIX_PATH or set
[cmake]   "modules_DIR" to a directory containing one of the above files.  If
[cmake]   "modules" provides a separate development package or SDK, be sure it has
[cmake]   been installed.
[cmake] Call Stack (most recent call first):
[cmake]   CMakeLists.txt:26 (find_package)
[cmake]
[cmake]
[cmake] -- Configuring incomplete, errors occurred!

Possible workaround: Replace the call to find_package(module CONFIG REQUIRED) with find_package(protobuf) then you can use functions like protobuf_generate_cpp within the same directory.

I think I didn't fully understand the initial problem, but I hope that I can clear things up.
If you just intend on using gRPC after installing it with vcpkg then the following will do:
find_package(gRPC CONFIG REQUIRED)
add_executable(my_app main.cpp)
target_link_libraries(my_app PRIVATE gRPC::grpc++)

in other words, simply ignore find_package(modules CONFIG REQUIRED). Why does that work? Looking at <vcpkg_root>\packages\grpc_x64-windows-static\share\grpc\gRPCTargets.cmake you will see that it declares re2::re2 and c-ares::cares as INTERFACE_LINK_LIBRARIES which means anything that links with gRPC::grpc++ will also link with those. Additionally, in <vcpkg_root>\packages\grpc_x64-windows-static\share\grpc\gRPCConfig.cmake you can see the calls to find_package(re2) and find_package(c-ares) which take care of making those packages available.
If you want to use re2 or c-ares directly then I recommend that you vcpkg install re2 (or even better add it to your manifest file) and:
find_package(re2 CONFIG REQUIRED)
target_link_libraries(my_app PRIVATE re2::re2)

Now my thought was that you want to use protobuf_generate_cpp in which case find_package(gRPC CONFIG REQUIRED) will be enough to make it available in the current directory. No need to find_package(protobuf) as I initially thought.
The source of confusion for vcpkg regarding find_package(modules CONFIG REQUIRED) seems to come from the subdirectory called modules in <vcpkg_root>\packages\grpc_x64-windows-static\share\grpc\ which contains Findre2.cmake and Findc-ares.cmake. I suspect that modifying gRPC's portfile (<vcpkg_root>/ports\grpc) by adding file(REMOVE_RECURSE ${CURRENT_PACKAGES_DIR}/share/grpc/modules) and patching the gRPCConfig.cmake file to remove the list(APPEND CMAKE_MODULE_PATH ${CMAKE_CURRENT_LIST_DIR}/modules) from it should remove the message. I know that @JonLiu1993 is working on updating the gRPC port at the moment, so I wouldn't want to interfere for now.

Thanks for the  explanation! It makes more sense now.
I tried setting it up on a windows machine instead and it almost works.
My generated protos files has a hard time finding PROTOBUF_NAMESPACE_OPEN inside google/protobuf folder.
It says this declaration has no storage class or type specifier

I haven't seen that error before but here is what I would try

Make sure you are using the most recent vcpkg version. E.g. if you have cloned it then

cd <vcpkg_root>
git checkout master
git pull
./bootstrap-vcpkg.bat
./vcpkg install grpc

Make sure that the proto files were generated with the same version of protobuf that you are linking with. If you are using protobuf_generate_cpp then that should be taken care of though. If you are looking for the protoc.exe it will be in <vcpkg_root>\packages\protobuf_x64-windows\tools\protobuf\

On my Windows machine it works fine using MSVC and clang-cl and using the windows-x64 and windows-x64-static vcpkg triplet. So keep trying!

@NangiDev did you have time to test my example and if so can you say what the problem was? I am asking because other people that stumble across this issue might find it helpful to know :)

Hey @Tradias
I am sorry I havn't had time.
We found a completelly different solution that is not related to this discussion at all, but I will try this at home as soon as possible for completeness of the discussion :)

This wrong information had been fixed now with latest vcpkg.
Installing 11/11 grpc:x64-linux...
Building grpc[codegen,core]:x64-linux...
-- Downloading https://github.com/grpc/grpc/archive/v1.51.1.tar.gz -> grpc-grpc-v1.51.1.tar.gz...
-- Extracting source /data/phoebe/vcpkg/downloads/grpc-grpc-v1.51.1.tar.gz
-- Applying patch 00001-fix-uwp.patch
-- Applying patch 00002-static-linking-in-linux.patch
-- Applying patch 00003-undef-base64-macro.patch
-- Applying patch 00004-link-gdi32-on-windows.patch
-- Applying patch 00005-fix-uwp-error.patch
-- Applying patch 00009-use-system-upb.patch
-- Applying patch snprintf.patch
-- Applying patch 00012-fix-use-cxx17.patch
-- Applying patch 00014-pkgconfig-upbdefs.patch
-- Applying patch 00015-disable-download-archive.patch
-- Using source at /data/phoebe/vcpkg/buildtrees/grpc/src/v1.51.1-1066d25324.clean
-- Configuring x64-linux
-- Building x64-linux-dbg
-- Building x64-linux-rel
-- Fixing pkgconfig file: /data/phoebe/vcpkg/packages/grpc_x64-linux/lib/pkgconfig/gpr.pc
-- Fixing pkgconfig file: /data/phoebe/vcpkg/packages/grpc_x64-linux/lib/pkgconfig/grpc++.pc
-- Fixing pkgconfig file: /data/phoebe/vcpkg/packages/grpc_x64-linux/lib/pkgconfig/grpc++_unsecure.pc
-- Fixing pkgconfig file: /data/phoebe/vcpkg/packages/grpc_x64-linux/lib/pkgconfig/grpc.pc
-- Fixing pkgconfig file: /data/phoebe/vcpkg/packages/grpc_x64-linux/lib/pkgconfig/grpc_unsecure.pc
-- Fixing pkgconfig file: /data/phoebe/vcpkg/packages/grpc_x64-linux/debug/lib/pkgconfig/gpr.pc
-- Fixing pkgconfig file: /data/phoebe/vcpkg/packages/grpc_x64-linux/debug/lib/pkgconfig/grpc++.pc
-- Fixing pkgconfig file: /data/phoebe/vcpkg/packages/grpc_x64-linux/debug/lib/pkgconfig/grpc++_unsecure.pc
-- Fixing pkgconfig file: /data/phoebe/vcpkg/packages/grpc_x64-linux/debug/lib/pkgconfig/grpc.pc
-- Fixing pkgconfig file: /data/phoebe/vcpkg/packages/grpc_x64-linux/debug/lib/pkgconfig/grpc_unsecure.pc
-- Installing: /data/phoebe/vcpkg/packages/grpc_x64-linux/share/grpc/copyright
-- Performing post-build validation
Stored binaries in 1 destinations.
Elapsed time to handle grpc:x64-linux: 10 min
Total install time: 11 min
grpc provides CMake targets:

    # this is heuristically generated, and may not be correct
    find_package(gRPC CONFIG REQUIRED)
    # note: 7 additional targets are not displayed.
    target_link_libraries(main PRIVATE gRPC::gpr gRPC::grpc gRPC::grpc++ gRPC::grpc++_alts)


ci: cleanup no longer used test logic related to memlimit

It seems that as part of #912, the memlimit files became unused. This PR removes them so that people like me don't end up confused about them or debugging why they aren't used etc.
Pinging @yuvipanda as the author of the test scripts being cleaned away from this repo.

thanks @consideRatio


Configure Soft Delete Policy fails on Emulator

I am getting the following error while trying to set the soft delete on Azure storage emulator. This is happening with 2 versions of emulator 1.12 and 1.15.1
{
"name": "StorageError",
"message": "XML specified is not syntactically valid.\nRequestId:046520c8-e84c-45fe-86bd-3a144bcad2b7\nTime:2020-10-01T03:18:01.6645487Z",
"stack": "StorageError: XML specified is not syntactically valid.\nRequestId:046520c8-e84c-45fe-86bd-3a144bcad2b7\nTime:2020-10-01T03:18:01.6645487Z\n    at Function.StorageServiceClient._normalizeError (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\azure-storage\lib\common\services\storageserviceclient.js:1205:23)\n    at Object.StorageServiceClient._processResponse (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\azure-storage\lib\common\services\storageserviceclient.js:751:50)\n    at Request.processResponseCallback [as _callback] (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\azure-storage\lib\common\services\storageserviceclient.js:319:37)\n    at Request.self.callback (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\request\request.js:185:22)\n    at ZoneDelegate.invokeTask (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\zone.js\dist\zone-node.js:275:35)\n    at Zone.runTask (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\zone.js\dist\zone-node.js:151:47)\n    at Request.ZoneTask.invoke (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\zone.js\dist\zone-node.js:345:33)\n    at Request.emit (events.js:210:5)\n    at Request. (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\request\request.js:1157:10)\n    at ZoneDelegate.invokeTask (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\zone.js\dist\zone-node.js:275:35)\n    at Zone.runTask (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\zone.js\dist\zone-node.js:151:47)\n    at Request.ZoneTask.invoke (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\zone.js\dist\zone-node.js:345:33)\n    at Request.emit (events.js:210:5)\n    at IncomingMessage. (C:\Program Files (x86)\Microsoft Azure Storage Explorer\resources\app\node_modules\request\request.js:1079:12)\n    at Object.onceWrapper (events.js:299:28)",
"code": "InvalidXmlDocument",
"linenumber": "1",
"lineposition": "66",
"reason": "Element 'DeleteRetentionPolicy' is not recognized.",
"statusCode": 400,
"requestId": "046520c8-e84c-45fe-86bd-3a144bcad2b7"
}

@asgoe soft delete is not supported by either storage emulator. You can open a feature request against azurite here: https://github.com/Azure/Azurite

Storage Emulator doesn't support it at all. Azurite is on the way supporting it. You can see Azurite respond to the configure soft delete policy requests but the policy doesn't have any effect.


Zh trans access cluster api.md

zh-trans: add /content/zh/docs/tasks/administer-cluster/access-cluster-api.md

CLA registered

/i signed

/signed

/i-signed

/check-cla

/check-cla

i signed

/check-cla

/check-cla

/retest

/ok-to-test

/check-cla

/check-cla

close this & reopen a new pr


Add profile awareness to get namespace config

Is your feature request related to a problem? Please describe.
Currently, get_namespace_config is not profile aware: https://github.com/dojoengine/dojo/blob/8a1a652d9ea06f141645316f22e3916a148403ec/crates/dojo-lang/src/utils.rs#L17
So it reads the table in different orders and may match different part of the configuration if several profiles are provided.
Describe the solution you'd like
Add the profile as parameter to try:

Perfect match profile.<PROFILE_NAME>.tool.dojo.world.namespace.
If not found -> fallback in tool.dojo.world.namespace.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Scarb loads everything under tool. into a hash map.
With the rework scheduled in #2193, we should then identify how to access scarb specific profile tool_metadata to ensure correct loading of the different entries in the toml for this profile.

fixed in #2285.


Replace ex units with ems for input lengths

It was recently questioned why we were using ex units (a unit equivalent to the font's x-height) to define the width of inputs. Ultimately, the original reasoning appears to have been lost to time.
This PR replaces the use of ex units with em equivalents for Input lengths.
This makes sense as:

em units are based on typeface width, rather than the height
em and ex are both based on the font size, both affect the width of the input when the body font size is changed
ex units are calculated differently by different browsers — IE 'cheats' and makes ex always equal half of em

Rather than make one-to-one replacements of ex for em, I rounded the em units up to the nearest .25. This prevents an excessive amount of decimal places in the resulting pixel measurements, but does mean that these inputs are up to a few pixels wider than previously.

Input length
old ex
new em
old px
new px

2
5.4
2.75
51.217
52.25

3
7.2
3.75
68.283
71.25

4
9.0
4.50
85.350
85.50

5
10.8
5.50
102.417
104.50

10
23.0
11.50
218.117
218.50

20
41.0
20.50
388.817
389.50

30
59.0
29.50
559.517
560.50

I'm pretty happy with this, looks good and I don't think the slight difference in width should have a big impact on anyone 👍
@owenatgov - you did the initial review. Do you want to re-review? Also happy for you to look at the failing Percy build and approve it if you'd like. First time I think we're actually doing that!


style: Grid view cards styling changed

Cards styling in grid view changed
Closes #1222

Description
The current UI of cards is not very eye-pleasing and does not provide a good user experience. This pull request proposes to improve the UI of cards by making them more visually appealing and easier to use.
Screenshots

Responsiveness

Checklist

[x ] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@M-A-N-I-K the design still has many bugs

@JasonDsouza212 like what?

As you can see, the navbar is not aligned and there is uneven spacing in the application. It has an entirely different style, and we need the card to have a minimalist and less animated design. For these reasons, we cannot move forward with this design.

@M-A-N-I-K is not active I'll be closing this PR


Avoid printing the entire source code in Python UI as a "log message"

Log messages intended to be a brief explanation of internal events to inform us about whats is actually going in the program. printing the entire source code of python UI doesn't help at all. It only pollutes the Terminal.
Like this:

this fix instead only shows the parameters and not the source code.

Result:

this does not work with all constructors closing

@mmertama
I fixed the merge request to work with all constructors. We will clip the main command name for logging and add (...) if it gets too long plus we will make sure that the parameters gets printed in the log. 🧑💻

How about just set that source code dump as a Debug_Trace log level? - I have been using that for most detailed (and likely  screen polluting) logging that is mostly usable if redirected to file?

Allow me to disagree. I strongly believe that the GUI command is important enough to print it as a DEBUG kind of log message. Because it can literally make your windows not to launch properly. However we can also print the whole thing as a TRACE message when the command gets too long.
...
    if(appui.length()>20) {
        std::string_view clipped_appui(appui.c_str(),20) ;//for debug logging
        GempyreUtils::log(GempyreUtils::LogLevel::Debug, "gui_cmd:", clipped_appui,"(...)", cmd_params);
        GempyreUtils::log(GempyreUtils::LogLevel::Debug_Trace, "gui_cmd:", appui, cmd_params);
    }
...

The 1.0.2 has solution for this issue so this can be omitted.


Failed to load project file: NullReferenceException

Environment data
dotnet --info output:
.NET Core SDK (reflecting any global.json):
 Version:   3.0.100
 Commit:    04339c3a26

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.17763
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\3.0.100\

Host (useful for support):
  Version: 3.0.0
  Commit:  7d57652f33

.NET Core SDKs installed:
  1.1.0 [C:\Program Files\dotnet\sdk]
  2.0.0 [C:\Program Files\dotnet\sdk]
  2.0.2 [C:\Program Files\dotnet\sdk]
  2.0.3 [C:\Program Files\dotnet\sdk]
  3.0.100 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 1.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 1.1.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

VS Code version: 1.39.1
C# Extension version: 1.21.4
Steps to reproduce
I'm trying to use VSCode to load a Unity project. I'm not sure about any specific reproduction steps, as the same project works fine on another machine.
Expected  behavior
Omnisharp should load the project.
Actual behavior
Omnisharp fails with this error:
[warn]: OmniSharp.MSBuild.ProjectManager
        Failed to load project file 'd:\game\Assembly-CSharp.csproj'.
d:\game\Assembly-CSharp.csproj(1,1)
System.NullReferenceException: Object reference not set to an instance of an object.
   at Microsoft.Build.Shared.BuildEnvironmentHelper.get_Instance()
   at Microsoft.Build.Utilities.Traits.get_Instance()
   at Microsoft.Build.Evaluation.ProjectCollection..ctor(IDictionary`2 globalProperties, IEnumerable`1 loggers, IEnumerable`1 remoteLoggers, ToolsetDefinitionLocations toolsetDefinitionLocations, Int32 maxNodeCount, Boolean onlyLogCriticalEvents, Boolean loadProjectsReadOnly)
   at Microsoft.Build.Evaluation.ProjectCollection..ctor(IDictionary`2 globalProperties)
   at OmniSharp.MSBuild.ProjectLoader.EvaluateProjectFileCore(String filePath) in D:\a\1\s\src\OmniSharp.MSBuild\ProjectLoader.cs:line 123
   at OmniSharp.MSBuild.ProjectLoader.BuildProject(String filePath) in D:\a\1\s\src\OmniSharp.MSBuild\ProjectLoader.cs:line 72
   at OmniSharp.MSBuild.ProjectFile.ProjectFileInfo.Load(String filePath, ProjectIdInfo projectIdInfo, ProjectLoader loader) in D:\a\1\s\src\OmniSharp.MSBuild\ProjectFile\ProjectFileInfo.cs:line 98
   at OmniSharp.MSBuild.ProjectManager.<>c__DisplayClass29_0.<LoadProject>b__0() in D:\a\1\s\src\OmniSharp.MSBuild\ProjectManager.cs:line 297
   at OmniSharp.MSBuild.ProjectManager.LoadOrReloadProject(String projectFilePath, Func`1 loader) in D:\a\1\s\src\OmniSharp.MSBuild\ProjectManager.cs:line 308

[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: d:\game\Assembly-CSharp-firstpass.csproj
[warn]: OmniSharp.MSBuild.ProjectManager
        Failed to load project file 'd:\game\Assembly-CSharp-firstpass.csproj'.
d:\game\Assembly-CSharp-firstpass.csproj(1,1)
System.NullReferenceException: Object reference not set to an instance of an object.
   at Microsoft.Build.Shared.BuildEnvironmentHelper.get_Instance()
   at Microsoft.Build.Utilities.Traits.get_Instance()
   at Microsoft.Build.Evaluation.ProjectCollection..ctor(IDictionary`2 globalProperties, IEnumerable`1 loggers, IEnumerable`1 remoteLoggers, ToolsetDefinitionLocations toolsetDefinitionLocations, Int32 maxNodeCount, Boolean onlyLogCriticalEvents, Boolean loadProjectsReadOnly)
   at Microsoft.Build.Evaluation.ProjectCollection..ctor(IDictionary`2 globalProperties)
   at OmniSharp.MSBuild.ProjectLoader.EvaluateProjectFileCore(String filePath) in D:\a\1\s\src\OmniSharp.MSBuild\ProjectLoader.cs:line 123
   at OmniSharp.MSBuild.ProjectLoader.BuildProject(String filePath) in D:\a\1\s\src\OmniSharp.MSBuild\ProjectLoader.cs:line 72
   at OmniSharp.MSBuild.ProjectFile.ProjectFileInfo.Load(String filePath, ProjectIdInfo projectIdInfo, ProjectLoader loader) in D:\a\1\s\src\OmniSharp.MSBuild\ProjectFile\ProjectFileInfo.cs:line 98
   at OmniSharp.MSBuild.ProjectManager.<>c__DisplayClass29_0.<LoadProject>b__0() in D:\a\1\s\src\OmniSharp.MSBuild\ProjectManager.cs:line 297
   at OmniSharp.MSBuild.ProjectManager.LoadOrReloadProject(String projectFilePath, Func`1 loader) in D:\a\1\s\src\OmniSharp.MSBuild\ProjectManager.cs:line 308
[omnisharp.log](https://github.com/OmniSharp/omnisharp-vscode/files/3720119/omnisharp.log)

I've attached the full log:
omnisharp.log
Solutions attempted
I've reinstalled:

dotnet
mono
c# extension
visual studio (now using 2019)

I've tried

{"omnisharp.path": "latest"}
solution described in https://github.com/OmniSharp/omnisharp-roslyn/issues/1188
regenerating my csproj and sln files

Any help would be appreciated, as I've been trying to figure this out for hours.

can you share this project?

visual studio (now using 2019)

according to the log you provided, there is no Visual Studio 2019 available on the machine? you could try installing Build Tools for VS2019 https://visualstudio.microsoft.com/downloads/#build-tools-for-visual-studio-2019

@filipw Thanks for the response.
I've reinstalled Build Tools for VS2019, but it hasn't made any difference.
I created a new dotnet project with dotnet new console --name testproj. This issue happens on it for me. Repository (including an omnisharp log file) here: https://github.com/Ratismal/omnisharp_testproj

@nguerrera do you have any idea? https://github.com/Ratismal/omnisharp_testproj/blob/master/omnisharp.log
it fails with MSBuild tools for vs2019 too, with some internal MSBuild exception. Is this something for the SDK team or should we open a bug in MSBuild repo?

Just an update, I've completely reinstalled my computer and this problem no longer occurs. I'm going to close this issue because I think it was likely related to something in my previous windows installation. Feel free to reopen if you think this isn't the case, but I won't be able to provide any more information.


Add BMFont library

This code works well for bitmapped fonts and should be listed.

Thank you for the PR, and sorry for the slow response :)


wtclient: improve startup performance

From this discussion and the various profile dumps
shared there, it is clear that if a node has many tower client sessions, then start-up of the tower client, and hence
LND, can be very slow.
The issue:
After some investigation, it is clear that the current way the tower client starts is super sub-optimal as it does
the following on each startup (and for each client):

the client keeps an in-mem map of map[channelID] max-commit-height. This map represents the maximum commitment height of a backup for a channel that the client has received. It uses this map in various places.
the issue is the population of this map (which is done at startup) : it will currently range over every session (which could be in the ten thousands) and for every session, it goes through each of its range indices to get the max height per channel backed up with the session.

The solution(s):

[x] The first thing that needs to be done that will have the most impact is removing the need to iterate through all the sessions & range indices on each start-up. So we should: keep track of the max-height for each channel in the channel details bucket and update that accordingly & then read from that on startup. This will require a migration since we need to iterate once over all sessions to populate this new field for existing channels. - #8222
[ ] The next thing we should do is to not iterate over sessions for towers that users have marked as removed/inactive.

re-opening cause there a second step that hasnt been addressed yet :)


Unable to install Red Hat OpenShift Container Platform Self-Managed on Azure

Hi All, I am using Red Hat OpenShift Container Platform Self-Managed, from Azure marketplace, for openshift installation. I am facing this error msg, could you please help ?
{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details.","details":[{"code":"Conflict","message":"{\r\n "status": "Failed",\r\n "error": {\r\n "code": "ResourceDeploymentFailure",\r\n "message": "The resource operation completed with terminal provisioning state 'Failed'.",\r\n "details": [\r\n {\r\n "code": "VMExtensionProvisioningError",\r\n "message": "VM has reported a failure when processing extension 'deployOpenShift'. Error message: \"Enable failed: failed to execute command: command terminated with exit status=4\n[stdout]\noning into 'openshift-container-platform-playbooks'...\n - Retrieved playbooks successfully\nWed Sep 4 10:48:15 UTC 2019 - Create variable for routing certificate based on certificate type\nWed Sep 4 10:48:15 UTC 2019 - Create variable for master api certificate based on certificate type\nWed Sep 4 10:48:15 UTC 2019 - Create variable for master cluster address based on cluster type\nWed Sep 4 10:48:15 UTC 2019 - Creating Master nodes grouping\nWed Sep 4 10:48:15 UTC 2019 - Creating Infra nodes grouping\nWed Sep 4 10:48:15 UTC 2019 - Creating Nodes grouping\nWed Sep 4 10:48:15 UTC 2019 - Enabling HA mode for masters\nWed Sep 4 10:48:15 UTC 2019 - Create Ansible Hosts file\nWed Sep 4 10:48:15 UTC 2019 - Running DNS Hostname resolution check\n\nPLAY [all] *********************************************************************\n\nTASK [Wait for DNS hostname resolution - will try for up to 33 minutes] ********\nok: [myocpcluster-master02]\nok: [myocpcluster-master01]\nok: [myocpcluster-infra01]\nok: [myocpcluster-master03]\nok: [myocpcluster-infra02]\nok: [myocpcluster-infra03]\nok: [myocpcluster-node01]\nok: [myocpcluster-node02]\nok: [myocpcluster-node03]\n\nPLAY RECAP *********************************************************************\nmyocpcluster-infra01 : ok=1 changed=0 unreachable=0 failed=0 \nmyocpcluster-infra02 : ok=1 changed=0 unreachable=0 failed=0 \nmyocpcluster-infra03 : ok=1 changed=0 unreachable=0 failed=0 \nmyocpcluster-master01 : ok=1 changed=0 unreachable=0 failed=0 \nmyocpcluster-master02 : ok=1 changed=0 unreachable=0 failed=0 \nmyocpcluster-master03 : ok=1 changed=0 unreachable=0 failed=0 \nmyocpcluster-node01 : ok=1 changed=0 unreachable=0 failed=0 \nmyocpcluster-node02 : ok=1 changed=0 unreachable=0 failed=0 \nmyocpcluster-node03 : ok=1 changed=0 unreachable=0 failed=0 \n\nWed Sep 4 10:48:17 UTC 2019 - Create Ansible Hosts file\nWed Sep 4 10:48:17 UTC 2019 - Updating WALinuxAgent on all cluster nodes\nmyocpcluster-master02 | UNREACHABLE! => {\n \"changed\": false, \n \"msg\": \"SSH Error: data could not be sent to remote host \\\"myocpcluster-master02\\\". Make sure this host can be reached over ssh\", \n \"unreachable\": true\n}\nmyocpcluster-master03 | UNREACHABLE! => {\n \"changed\": false, \n \"msg\": \"SSH Error: data could not be sent to remote host \\\"myocpcluster-master03\\\". Make sure this host can be reached over ssh\", \n \"unreachable\": true\n}\nmyocpcluster-infra03 | UNREACHABLE! => {\n \"changed\": false, \n \"msg\": \"SSH Error: data could not be sent to remote host \\\"myocpcluster-infra03\\\". Make sure this host can be reached over ssh\", \n \"unreachable\": true\n}\nmyocpcluster-master01 | UNREACHABLE! => {\n \"changed\": false, \n \"msg\": \"SSH Error: data could not be sent to remote host \\\"myocpcluster-master01\\\". Make sure this host can be reached over ssh\", \n \"unreachable\": true\n}\nmyocpcluster-node01 | UNREACHABLE! => {\n \"changed\": false, \n \"msg\": \"SSH Error: data could not be sent to remote host \\\"myocpcluster-node01\\\". Make sure this host can be reached over ssh\", \n \"unreachable\": true\n}\nmyocpcluster-infra02 | UNREACHABLE! => {\n \"changed\": false, \n \"msg\": \"SSH Error: data could not be sent to remote host \\\"myocpcluster-infra02\\\". Make sure this host can be reached over ssh\", \n \"unreachable\": true\n}\nmyocpcluster-infra01 | UNREACHABLE! => {\n \"changed\": false, \n \"msg\": \"SSH Error: data could not be sent to remote host \\\"myocpcluster-infra01\\\". Make sure this host can be reached over ssh\", \n \"unreachable\": true\n}\nmyocpcluster-node02 | UNREACHABLE! => {\n \"changed\": false, \n \"msg\": \"SSH Error: data could not be sent to remote host \\\"myocpcluster-node02\\\". Make sure this host can be reached over ssh\", \n \"unreachable\": true\n}\nmyocpcluster-node03 | UNREACHABLE! => {\n \"changed\": false, \n \"msg\": \"SSH Error: data could not be sent to remote host \\\"myocpcluster-node03\\\". Make sure this host can be reached over ssh\", \n \"unreachable\": true\n}\n\n[stderr]\n % Total % Received % Xferd Average Speed Time Time Time Current\n Dload Upload Total Spent Left Speed\n\r 0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0\r100 6 100 6 0 0 904 0 --:--:-- --:--:-- --:--:-- 1000\n\"."\r\n }\r\n ]\r\n }\r\n}"}]}

This information is not helpful enough.  When opening an issue, please follow the template and provide the details that are needed for proper troubleshooting.

please make sure you have not set a passphrase on your ssh key.


B-20465-INT - Add Counseling Office Selection to Customer Application

Agility ticket
Summary
When a customer selects an origin duty location assigned to government counseling, then they will also be asked to select a counseling office within the same GBLOC. This field will be called "Counseling Office".
How to test as a Customer

Login as a Customer
Select a counseling Office while creating a new move
Check the Moves data table for corresponding counseling_office_id
Create another move
Select a prime counseled location (Ex. NS Mayport) and verify the customer is not prompted to pick a counseling office
Check the Moves data table for corresponding counseling_office_id (should be null)

How to test as a Service Counselor

Login as a Service Counselor
Verify the counseling queue's counseling Office column matches what was created in previous move

508 Screenshots
508Screenshots-20465.pdf

Nice work!

conflicts

Test coverage on server failed.
Last run: Mon 30 Sep 2024 11:21:49 PM UTC
Refer to this report to see coverage details for the server app.
Documentation for test coverage can be found at this link.


Removed Active Cases Chart from Daily Section

Description of PR
This PR Fixes the daily active cases graph which was showing the negative values if the recovery was greater than active cases that day.
Type of PR

[x] Bugfix

Relevant Issues
Fixes #1313
Checklist

[x] Compiles and passes lint tests
[x] Properly formatted
[x] Tested on desktop
[x] Tested on phone

Screenshots
before:

after:

#1313 was closed and invalid. Thanks for the PR, however, closing this.


turn alphabet into a list grabbed from a csv file

thanks a bunch to @gsingh93 for the inspiration (from #1048)

It could be worth creating a default CSV that gets shipped so users know the expected header structure, folder to put the CSV in etc.
It also would mean that you could remove the default alphabet from the code.

A settings/alphabet.csv should be created automatically the first time you boot Talon with this code (which has a header). Hopefully people will either find that or get directed to it one way or another.
You may also be interested in this discussion about settings/.csv vs included .csv files starting here: https://github.com/knausj85/knausj_talon/pull/1048#issuecomment-1336317288


chore: add extract translations job to check workflow

Summary
add extract translations job to check workflow
Fixes # 1873
How did you test this change?
Ran Github workflow.
Checklist:

[ ] I have performed a self-review of my code
[ ] 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
[ ] Implemented a user interface (UI) change, referencing our Figma design to ensure pixel-perfect precision.

Please check @nikaaru this


Dev/exce handling - Initial Contribution for Exception Handling (Issue #1884)

NB: This is an initial contribution, part 1 of 2 contributions.
Included in this contribution

Inside the classic interpreter only:

Initial handling of Tags
Initial handling of Exceptions based on W3C Exception Proposal (note this isn't a direct copy of traditional C++ exceptions)
Builds with no-regressions

To be included in Part 2

Bug fixes and naming convention improvements
Additional CI /CD changes to validate ENABLE_EXCE_HANDLING switch builds ok all platforms
Import and Export of Exceptions and Tags

Ugh.. ok.. let's try this one again.. closing this now.


Understanding the pv_picovoice_process() function.

Hello,
I am currently working on an STM32F407G-DISC1 attempting to use picovoice. I am able to read in PDM data and convert it to PCM, but I am struggling to understand what needs to be sent into the pv_picovoice_process() function. I understand that it is a different buffer of int16_t based on the API documentation. Mainly I need to understand what conversion is being done in the below loops present in the demo, what do these do? Why are we not just passing in the PCM buffer?

As stated in the API documentation, "the number of samples per frame can be obtained by calling pv_picovoice_frame_length." In other words, to use the process function, we need to pass 512 samples in this instance. Therefore, a helper ping pong buffer is employed instead of directly passing the output of the PDMToPCM function. This buffer accumulates the PCM samples, ensuring that the requirement is met


Add explicit shutdown to ActorCache when the Worker::Actor is shutdown

Previously, we allowed scheduled flushes to continue after the Worker::Actor experienced shutdown. This meant that we would effectively commit to storage whatever write operations were in the write buffer when an actor became broken. Instead, we now cancel any scheduled flushes. There may still be in-flight flushes containing coalesced batches of writes from before the actor shut down.

Turns out that we needed to also shutdown the actor cache inline to the abort promise because otherwise actor cache shutdown was sequenced after the flush promise enqueuing. Fun times!


Version expansion for NuGet

NuGet is almost semver, but not exactly: https://docs.microsoft.com/en-us/nuget/concepts/package-versioning#where-nugetversion-diverges-from-semantic-versioning

NuGet's implementation: https://github.com/NuGet/NuGet.Client/blob/dev/src/NuGet.Core/NuGet.Versioning/VersionComparer.cs


This component fouls up userHeight even not asked to replay

The spectator camera is being activated and set to 0,0,0 even when there is nothing being replayed.
This ignores the existing userHeight default in the scene.
I had to fix it via hard coding -
<a-scene avatar-recorder avatar-replayer="spectatorMode: true;loop: true;spectatorPosition 0 0 1.778">
My expectation was that even with spectatorMode active etc, if there is nothing to replay, then the default camera should be respected.

I ran into this as well, having avatar-replayer sets mobile to 0,0,0 giving a black screen


Square config grouping for location

Bug Description
Reported that the Square config is failing to pass the location_id when processing a DSAR
Looking to see if the params need to be grouped for the GET request for location
Steps to Reproduce

Set up Square config
process an access request

Expected behavior
The access request should succeed
Screenshots
n/a
Environment

Version: 2.4.0
OS:
Python Version:
Docker Version:

Additional context
Reported by a user, creating the issue on their behalf and continuing to investigate.

Copying this from the PR - I have been able to get a DSR to succeed using nox -s test_env on fides with both an email and a phone number without having to change the grouping, so this issue as written is appearing to be invalid. However, when using the phone number on the latest of fidesplus I encounter a failure (email still succeeds) planning to review further and address.


Implementation reverse Cuthill-McKee

See also #185 .
This changes the Cuthill-McKee algorithm implemented here to the reverse Cuthill-McKee algorithm. Starting vertices are choosen by a George-Liu pseudoperipheral vertex finder.
I have added two more test cases and verified the correctness of the exisiting one w.r.t. the new algorithm.
I did try to write in the style of the library, however I might not always have got it right when to use .index(), when to call .unwrap() vs .expect(), and so on. Feel free to change these details, as well as remove unneccesary comments or line breaks. I just can't get enough of those.
As benchmark possibilities are scarce, I did not optimize anything to aggressively. I really believe this crate should come with some sparse matrix collection as well as random matrices to benchmark against, automatically. That would make tweaking the details worthwile. That's an issue for another time though.
Concerning the minimum degree ordering: As it is significantly more complex, It will take some more time to implement it, as in a few days to weeks. I will do it eventually, but I don't want to block somebody else on it, if there is interest.

That looks nice, thanks.
First remarks:

I think it would be better to expose both cuthil_mckee and reverse_cuthill_mckee
could you make sure the comments are wrapped at the 80 characters limit? Running cargo fmt should do it automatically. (I'm wondering why no CI launched on your PR?)

I'll take more time to review the rest a bit later.

My bad, looks like CI did pass https://travis-ci.org/github/vbarrielle/sprs/builds/668095347, I don't know why github did not display it...

I am wondering why CI was not displayed as well, but I can't help you there, I am not that good with Github.

Exposing cuthill_mckee too ... I don't know.  It is easily simulated by just reversing the reverse order. Sure, that is definitely not optimal from a performance standpoint, but it shouldn't matter too much, as, firstly, the normal ordering should only be used rarely, and secondly, those O(n) algorithms probably aren't bottlenecks anyway. Still, if you really want it too, that's totally fine by me.

I contemplated if maybe the algorithm should be generic over a StartingStrategy. Then the consumer of the algorithm could decide on how to pick a starting node. Some options that immediately come to mind: PseudoPeripher, MinimumDegree, Next, Constant. Being able to optimally choose depending on the specific situation seems like a strong point to me. What do you think?

I did run cargo fmt, no idea why these style guidelines were not enforced. Seems like a problem with the project configuration. I am using VS Code on Windows, even manually running rustfmt with passed --config max_width=80 option didn't format the comments correctly. Any idea what to do?

Take your time to review, though the tests seem to pass I am not sure if I got all the library-specifics absolutely correct.

Don't worry about the CI, maybe it's simply an issue with github.
The main reason why I wanted to expose cuthill_mckee as well was to have an API exposing all the choices. It's quite a simple operation indeed, but i feel it's nice to present the user with all the possibilities.
In that spirit, I think it's a idea to expose the StartingStrategy, especially if the StartingStrategy is made to implement Default.
Don't worry about the comments line width, I'll fix them myself later on, and I'll try and find the proper rustmf configuration.

I will push the requested changes now.
After giving it some more thought, I believe the best compromise between ease of use and customizability is: exposing one function generic over the orderings direction and the starting strategy, cuthill_mckee_custom(mat, strategy, direction), along with one function reverse_cuthill_mckee(mat), which essentialy calls the custom function with the defaults as specified in this PR. I will take a day or so to push this customizable algorithm. Don't merge yet ;-)

I agree exposing two functions is the best way to enable choosing the init strategy while encouraging pseudoperipheral as a default.

Here is the refactored, extended version.
I decided to favour traits for the options. For the strategies that is the obvious choice, for the orderings direction not so much. However I do not see an other way to create this abstraction without considerable runtime costs.
Concerning the namespacing, that definitely needs to be overhauled. I introduced the modules start and order, but they need to be publicly re-exported in some sensible way or something. It's up to you to decide if and how to do that. Also, maybe a different file structure is needed.
Please comment what you would like to be clarfied etc., and if you believe there need to be more test cases covering all the different options, although most are quite trivial.

I have implemented all changes as requested. If there is nothing left to dicuss, this can now be merged.

That's perfect, I'm merging.

ANd thanks a lot!

And thanks a lot!

It's the magic of open source!


557-feat-add-inspection-request-direct-email-option-for-chelsea-residents

Added question and code to email inspection requests directly to the City of Chelsea for residents who want to, updated download screen, and added conditional logic to inspection request instructions based on the user's address/answer to directly emailing letter.

Fixed #557
In this PR, I have:

[X] Manually tested to ensure my PR is working
[X] Ensured issues that this PR closes will be automatically closed
[X] Requested a review

@nonprofittechy just so you know the email is currently sent from<EMAIL_ADDRESS>We could add a reply_to email for the user they have one filled out, but many potentially won't have one filled out.

Updated the question to yesnoradio!


There is no access restrictions when accessing weave-app by default

What you expected to happen?
The weave app should add access restrictions when a user accesses the weave app dashboard UI.
What happened?

Weave app is a dashboard UI that helps to manage the Kubernetes cluster, however, there are no access restrictions when accessing the weave app dashboard by default. Thus, without a proper weave-scope network policy, a malicious container can access the weave app dashboard inside a Kubernetes cluster, and delete any workloads in the whole cluster.
How to reproduce it?

Install the weave app following the official guide:

kubectl apply -f https://github.com/weaveworks/scope/releases/download/v1.13.2/k8s-scope.yaml
2. Install any web browser inside a malicious pod, and access the weave app dashboard.
3. A malicious pod can access the weave app dashboard UI by default, and it can leverage the dashboard UI to delete any workloads inside the whole cluster.
Anything else we need to know?

Kubernetes v1.24.2, installed by kubeadm. A three-node cluster, each node with Ubuntu 20.04 TLS.
Versions:

$ scope version
$ docker version
$ uname -a
$ kubectl version

Logs:
$ docker logs weavescope

or, if using Kubernetes:
$ kubectl logs <weave-scope-pod> -n <namespace>

Hello? Are there any updates or comments? Looking forward to your reply!


Update standard_library_types.md -Typo Fix

Change Summary

Related issue number

Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[ ] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review

Thanks @Yummy-Yums


Draft a release

Could we publish a first release v0.0.1, please? This would make it easier for me to rely on a specific set of exports of this package.

@YingboMa @ChrisRackauckas Do you have any objections to publishing a first release?


Added multi pixel support & unit tests

This PR introduces changes to the Nextdoor Pixel GTM custom template that enable the multi pixel feature support.
Updated the regex for Pixel IDs: each pixel id should be UUID that consists of 32 hexadecimal characters, separated by hyphens into five groups of 8-4-4-4-12 characters respectively. The string may begin and end with any number of whitespace characters, and each UUID, except for the last one, can be followed by a comma and any number of whitespace characters.
Each passed Pixel ID will be initialized and added to the Pixel ID array. Then every Pixel ID will be sent to the tracking call.
Slightly not backwards compatible however. For advertisers who currently have 1 pixel id in GTM but 2 pixels installed, it will double count, this change will immediately only count the one. The reason I opted against an additional parameter, is it seemed redundant/confusing to include the Pixel ID in one field and then create a list with that ID as well.
It also adds unit tests, that setup the Pixel script injection and multi pixel parameter extraction.

Can we add changelog in metadata.yaml file?

Slightly not backwards compatible however. For advertisers who currently have 1 pixel id in GTM but 2 pixels installed, it will double count, this change will immediately only count the one. The reason I opted against an additional parameter, is it seemed redundant/confusing to include the Pixel ID in one field and then create a list with that ID as well.

💯


通过 updateSheet 修改表格配置实现隐藏行后表格未刷新，需要提供手动刷新表格的API。

场景：需要隐藏表格的某些行。
代码：
const sheet = this.tableRef.current?.getAllSheets()[0];
const tableData = {
    ...sheet,
    config: {
        ...sheet.config,
        rowhidden: {
            1:0,
            2:0,
        },
    },
};
this.tableRef.current?.updateSheet(tableData);
//运行后代码表格没有变化。

this.forceUpdate();
//强制刷新后隐藏了对应行

English
Scenario: Some rows of a table need to be hidden.
Code:
const sheet = this.tableRef.current? .getAllSheets()[0];
const tableData = {
    ... sheet,
    config: {
        ... sheet.config,
        rowhidden: {
            1:0,
            2:0,
        },
    },
};
this.tableRef.current? .updateSheet(tableData);
// The code table does not change after running.

this.forceUpdate();
// The corresponding row is hidden after a forced refresh

you are looking to hide some rows on component initialisation ?

是的 我想要在使用表格过程中 手动隐藏一些符合条件的行。
上面的代码是通过修改表格配置并且重新渲染可以实现。
Yes I want to manually hide some eligible rows while using the table.
The above code can be achieved by modifying the table configuration and re-rendering.

rephrasing this, we need a direct method for hideRow and hideColumn in the workbook ref. perfectly justified ask, let me see what i can do about this. thanks for submitting the request @Rio-shuai

i have implemented this, will try to release a new version by tonight with the other fixes i had merged last week.

i have implemented this, will try to release a new version by tonight with the other fixes i had merged last week.


(Multi-platform version) When packaging for Linux, binaries aren't found by default and need to be moved

When packaging the plugin in a project for Linux, and running the ProjectNames.sh starting script, the following error is displayed:
error while loading shared libraries: libMicrosoft.CognitiveServices.Speech.core.so: cannot open shared object file: No such file or directory

This issues comes frome the fact that all the Azure library files (*.so) are placed in ProjectName\Binaries\Linux\ThirdParty\AzSpeech. If the libraries are moved instead two folders up, to ProjectName\Binaries\Linux, the build starts up fine.
If there is a way to define where the libraries are placed when making a build, it would be great to fix this. I personally haven't found how this can be defined, but my knowledge of the UBT process isn't so deep.

Removed the sub directories, now the plugin binaries will be moved to the same directory of the executable: https://github.com/lucoiso/UEAzSpeech/commit/090480b3481f65cec455f9a447474114753a51e5#r124331032


color for the scrollbars

I've got css code from Abhay which changes the bar color. How about a color option in the Scrollbars extension?

Thanks for the suggestion. I've been thinking about adding settings colors. The original idea was to avoid them and let themes handle that, but meanwhile I've encountered some situations where the extension has to add colors explicitly.
Can you give more details about your problem? What theme exactly are you using? Can you give me the css code block from your roam/css page?

Each of five graphs uses a different color scroll bar. This provides visual variety, in contrast to the straightforward black or white bar.
Roam Studio's Base Theme, Quattro, is used for all the graphs.
The css code used now is shown below. The original is in Slack and found here.
/* Scrollbar improvements HT: Palash Karia */
div::-webkit-scrollbar-track {
background-color: transparent !important;
}
div::-webkit-scrollbar-thumb {
background-color: #9C27B0      !important;
}
div::-webkit-scrollbar {
width: 15px;
height: px;
border-radius: 8px;
}
div::-webkit-scrollbar-track {
background: rgba(249,170,170,0);
border-radius: 20px;
}
div::-webkit-scrollbar-thumb:hover {
background-color: rgba(249,135,135,0.25);
border: 0px darkgrey solid;
}

Each of five graphs uses a different color scroll bar. This provides visual variety, in contrast to the straightforward black or white bar.
Roam Studio's Base Theme, Quattro, is used for all the graphs.
The css code used now is shown below. The original is in Slack and found here.
/* Scrollbar improvements HT: Palash Karia */
div::-webkit-scrollbar-track {
background-color: transparent !important;
}
div::-webkit-scrollbar-thumb {
background-color: #9C27B0      !important;
}
div::-webkit-scrollbar {
width: 15px;
height: px;
border-radius: 8px;
}
div::-webkit-scrollbar-track {
background: rgba(249,170,170,0);
border-radius: 20px;
}
div::-webkit-scrollbar-thumb:hover {
background-color: rgba(249,135,135,0.25);
border: 0px darkgrey solid;
}


No way to access settings.

Maybe I am missing something, but I see no way to access settings to try the dark mode.
In seabass1 the hamburger menu opens with 'settings', 'about', and 'help' and there is an icon to open the file browser on the right of the top bar. In seabass2 the hamburger opens the file browser, and there is no corresponding icon to the right for the other items.

There is no Settings page yet for Seabas2.
For now the only way to activate the dark mode to set the Suru dark theme globally (using UT Tweak Tool for example)

TODO: make these preferences available:

[ ] print margin
[ ] font size
[ ] optionalize soft tabs
[ ] default soft tab size

Toggling light/dark/system theme would be a nice setting to have too.

Toggling light/dark/system theme would be a nice setting to have too.

Thank you for the idea!
Toggling theme manually is, I believe, impossible within a pure QML application.
I don't want to add C++ until there are critical blockers for using pure QML.
So we probably won't have the feature for now

Seabass can now read indentation preferences from .editorconfig files


Metadata Import Error

Basic information

My operating system and version: Windows 10 Enterprise, version 10.0.19045 Build 19045
My mzmine version: 4.0.3

What happened
When importing metadata of my dataset, the import stops with the error message:
io.github.mzmine.taskcontrol.impl.WrappedTask run Error of task Importing metadata parameters from the selected file(s): Couldn't import metadata from D:\converts\desm_metadata.csv
The same happens if metadata read-in is part of the batch workflow. Processing works without any issues if no metadata is read in.
You asked me at the workshop to upload some data files and the metadata file I used. They are in a zip file here: https://drive.google.com/file/d/1HvYt0eXvdBHkVDdryh1GM4T-J43IhVMr/view?usp=sharing

Solved at the workshop, probably caused by manual generation of the metadata file.

Hello, please can you tell me how did you resolve this problem?


fields from a custom user model not passed upon save

May you help with how can I overide the serializer to include extra fields from a custom serializer

What's your progress right now? What you are trying to achieve can you share some piece of code?

I hv a multiple type user system.  I needbmy users to be categorized by the field category with choices of user types. The thing is those extra fields are on required fields and I even modified my serializer but sadly when I pass the user data upon sign up from an api,,,all those fields except for email, username and password are not passed,,,they are just null values

@DUMBALINYOLO  can you please share the code of your custom sterilizer, did you update the Djoser serializer settings with your customer serializer as I did.

@DUMBALINYOLO  can you please share the code of your custom sterilizer, did you update the Djoser serializer settings with your customer serializer as I did.


Hoist uses of Nullable.GetUnderlyingType() into ModelMetadata

MVC code contains too many instances of low-level code duplicating
!type.GetTypeInfo().IsValueType || Nullable.GetUnderlyingType(type) != null

and even more duplicates of
Nullable.GetUnderlyingType(type) ?? type

The first could be exposed as ModelMetadata.AllowsNullValue. Alternatively could provide ModelMetadata.IsValueType in addition to the existing ModelMetadata.IsNullableValueType, allowing the above to be elevated to
!metadata.IsValueType || metadata.IsNullableValueType

The second could be exposed as ModelMetadata.UnderlyingType.

This, is code cleanup, just do it IMO :+1:

Yeah, don't disagree. Self-assigning this "task".

Decided to add

IsReferenceOrNullableType, just to avoid a bool property that isn't named IsFoo
UnderlyingOrModelType

bf7e0f1


QueuedJob: Replaces params with param

People often ask why this doesn't work:
class MyJob < QueuedJob
  params(user_id : Int32)
  params(email : String)
end

And I believe the source of this confusion is that the crystal community has changed tack a bit. Years ago people feared macros and tried to avoid using them. Today people have come to embrace macros. The style of one macro to declare multiple attributes isn't intuitive.
Besides the confusion, this makes it difficult to implement composable behaviors. Modules can't add their own parameters because they'll get overwritten by subsequent calls to params(). With the param macro, this becomes possible:
module ValidateUserBehavior
  param user_id : Int32

  before do
    user = lookup_user(user_id)
    if user.banned?
      raise "user cannot run jobs anymore"
    end
  end
end

class AJob < QueuedJob
  include ValidateUserBehavior

  param how_much : Int32

  def perform
    # whatever users do in this system
  end
end

That would make a fan-out-in behavior easy to implement and ship with mosquito, which is another thing that people have asked for several times.

I agree. I myself have done this several times.

I like this a lot! Much more intuitive.


Fields not matching the fields in vehicle_data endpoint

Hi, I noticed that the fields that are accessible in the Payloads in this project do not cover all fields that are documented to be available for example in the vehicle_data endpoint. Especially for our use case, having access to the steering wheel heater is essential. Will fields like these be added, or is there some fundamental difference in the type of data that will be available via fleet-telemetry and the vehicle_data endpoints?

+1
I'm using the guest mode for our fleet but we can only enable/disable them through the commands but we can't see if they are active or not. I believe they should be accessible through the vehicle data endpoint


Target NETStandard2.0 TFM

Bringing back Microsoft.AspNetCore.Hosting.WindowsServices. (Will update coherence packages.csv)
Cross-compiling M.A.Hosting for netcoreapp2.0 and netstandard2.0 since it needs System.Runtime.Loader which is a netcoreapp only package.
cc @Eilon @DamianEdwards

:up::date:


Update people section

I like having an "Alumni" section. Comments:

I think it's better to link to the alumni's preferred web site (LinkedIn, github, whatever) rather than for us to maintain their old personnel summary page, as you're doing with Kavin. We can also list the start-end years they were part of the group, and what position they held.
Why not do the same with Ali (her name is dropped)? And Jordana and Keith?

I am fine with changing Kavin's old bio to LinkedIn, but with the template the headshot probably needs to be there -- I think it is better to just have names the links -- I could look into that later. I did think it is weird to have the old bio there, but that was the path of least work last night.
For Ali, Keith, and Jordana - I dropped their names because I don't think they have spent enough time in the group to be an integral part of it or actually contributed to the projects. Happy to discuss this.

I am fine with changing Kavin's old bio to LinkedIn, but with the template the headshot probably needs to be there -- I think it is better to just have names the links -- I could look into that later. I did think it is weird to have the old bio there, but that was the path of least work last night.

Ah. So, a stub person page has to exist in order for the person to show up at all?!

For Ali, Keith, and Jordana - I dropped their names because I don't think they have spent enough time in the group to be an integral part of it or actually contributed to the projects. Happy to discuss this.

We can discuss elsewhere, some other time.

Ah. So, a stub person page has to exist in order for the person to show up at all?!

I think so, but there may be some short cut around it.

Seems like this is what we need: https://github.com/wowchemy/wowchemy-hugo-themes/issues/1414
Not sure if it works with the updated version, but it's probably worth giving it a try.


Envoy does not release access log fd on sigusr1

Envoy does not release access log fd on sigusr1
Description:
In one of our more heavily used clusters we've seen that despite employing logrotate and kill -USR1 $ENVOY_PID the file descriptors are still present in /proc/$ENVOY_PID/fd.  We have also seen the reported memory usage increase until we force a reload either by restarting the process or kill -HUP $ENVOY_HOT_RESTART_PID.
Version:
envoy 08fadcc41f34b5816f2dc9412a1614efca4d9667/1.8.0-dev/Clean/RELEASE live 2829 44974 1

Repro steps:

ls -l /var/log/envoy/200

	lrwx------ 1 envoy envoy 64 Aug  6 12:39 130 -> /var/log/envoy/default_https.log
	lrwx------ 1 envoy envoy 64 Aug  6 12:39 131 -> /var/log/envoy/200/default_https_access.log
	lrwx------ 1 envoy envoy 64 Aug  6 12:39 146 -> /var/log/envoy/spark_default_https.log
	lrwx------ 1 envoy envoy 64 Aug  6 12:39 61 -> /var/log/envoy/default.log
	lrwx------ 1 envoy envoy 64 Aug  6 12:39 77 -> /var/log/envoy/spark_default.log

kill -USR1 $ENVOY_PID
sudo ls -l /proc/$ENVOY_PID/fd

	lrwx------ 1 envoy envoy 64 Aug  6 12:39 130 -> /var/log/envoy/default_https.log
	lrwx------ 1 envoy envoy 64 Aug  6 12:39 131 -> /var/log/envoy/200/default_https_access.log
	lrwx------ 1 envoy envoy 64 Aug  6 12:39 146 -> /var/log/envoy/spark_default_https.log
	lrwx------ 1 envoy envoy 64 Aug  6 12:39 147 -> /var/log/envoy/200/spark_default_https_access.log.1
	lrwx------ 1 envoy envoy 64 Aug  6 12:39 61 -> /var/log/envoy/default.log
	lrwx------ 1 envoy envoy 64 Aug  6 12:39 62 -> /var/log/envoy/200/default_access.log.1
	lrwx------ 1 envoy envoy 64 Aug  6 12:39 77 -> /var/log/envoy/spark_default.log
	lrwx------ 1 envoy envoy 64 Aug  6 12:39 78 -> /var/log/envoy/200/spark_default_access.log.1

Mitigation:
(this is for anyone that lands here from google searching in the future)
There are 2 ways to mitigate:

Restart the envoy process.
If using the envoy hot restarter script you can force a reload: kill -HUP $ENVOY_HOT_RESTART_PID

Here is the reopen logic: https://github.com/envoyproxy/envoy/blob/master/source/common/filesystem/filesystem_impl.cc#L196. I'm pretty sure it's working correctly.
Do you essentially see this leak forever in your env no matter how many times you rotate? @danielhochman might be able to spot check what's going at Lyft in this area.

bleh... I think this is me v_v
I did some more work and this is what i see:

we have 1 higher volume route generating a couple hundred mb in access logs every 10 minutes but we have a number of low volume routes that do not generate many logs
time-based logrotation would take an existing log and rotate it

a new log would be created, an old one would be rotated but envoy would keep the old fd until the new one received traffic then it would release the old fd.

size-based logrotation would take an existing log and rotate it

The reported memory usage by systemd was the sum of all log files in the log directory (weird).
Envoy would release the old fd as the new one received traffic immediately.

Cool, glad it's working. Closing.


Use cmd exit value for show_if

Is your feature request related to a problem? Please describe.
From what I found in documentation, it seems that show_if can't take exit value from cmd when using script module.
As example, I want to show available updates on my Arch and it seems little wastefull to call checkupdates twice.
  - type: script
    mode: poll
    cmd: checkupdates | wc -l | xargs -n1 echo "󰏕"
    interval: 1800000
    show_if: checkupdates

Describe the solution you'd like
Not sure what best approach would be, but maybe simply receive exit value from cmd as first argument?
Describe alternatives you've considered
Maybe somehow manipulate ironvars? Seem little overkill to me, but maybe its better way.

Hey, I don't think this would be feasible unfortunately.
The show_if option is handled outside of the module code, runs its own script, and has no visibility into the module itself so it can't easily be shared. I'd be hesitant to refactor the way that works to change that either, since the original behaviour would need to be kept for other module types.
Instead the best option here is definitely to use an ironvar. There's possibly opportunity to make that more elegant, especially since this is (I'd imagine) quite a common scenario, so I'm open to ideas around that.


Hide & seek

👍 You added your card two times, in html and in js that's why you got twelve cards rather than six. Kindly remove the HTML part because it is dynamic when you add it in JavaScript.

You duplicated some cards rather than creating a unique card


Use empty future instead of Forever

This PR removes unnecessary Forever future.
Empty already implements this behavior:
https://docs.rs/futures/0.1.18/src/futures/future/empty.rs.html#20-22

Thanks a lot for this PR!


"Add a Plugin"-Link doesn't work after adding a plugin

After adding a plugin I cannot click on the "Add a Plugin" link in the upper left corner. At least it doesn't do anything.
Chrome 28.0.1500.95 on OS X 10.8.4 here.

It is a link with href="#", so it points to current page without reloading.
Solution: make the url of last breadcrumb just plain text. Or is this linkified on purpose?

@dom-mel why are you linking to href="#"? looks useless in eyes of end users..


Suggestion: Put footer at bottom of page even when litlle page content

On the following example, the page content is not big enough to push footer off-site. The body content height should be at least of a size which would put the offset perfectly at the page bottom. This can be done in purely in sass.

Use Bootstrap position utility fixed-bottom? Need to test...

I prefer this option How to Make Footer Stay at Bottom of Page with Bootstrap to vertical align the footer to the bottom of the page (on pages with little content).


[Form validation] multiple fields

Hey all,
is it possible to get more than one field to give them a validation option?
...
fields : {
datum: {
identifier  : 'phone',
rules: [
{
type   : 'empty',
prompt : 'Example text',
}
]
},
...
Can I give them a class or something else to get them all "called"?

View contributing guide


NIFI-4303 Add routingKey to ConsumeAMQP processor

Description of PR
Adds routing key in ConsumeAMQP processor NIFI-4303
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[x] Have you verified that the full build is successful on JDK 8?
[x] Have you verified that the full build is successful on JDK 11?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

@doodzio Thank you for your first contribution.
As far as I see, you are not assigned the Contributor role yet. Could you please request for "Jira contributor access" on the mailing list<EMAIL_ADDRESS>Then please assign the Jira ticket to yourself and also set it to Patch Available status.
I tested your change and it works as expected. The code and tests look also fine.
What do you think about also adding exchange (from the envelope) and queue (from the processor property) to the outgoing FlowFile's attributes too?

@turcsanyip Thx. I tootaly overlooked those steeps, I did it today.
Adding exchange seems ok, I am not sure that we should append something that already is definied by user.

ConsumeKafka and ConsumeJMS add the destination as an attribute (kafka.topic / jms_destination).
It is true though, that the destination property can come from variable registry in case of those processors so it is more dynamic.
In general, it is easier to use this info in the downstream flow if it is available on the FlowFile.


RTPS payload pool [9544]

Documenting the new PayloadPool on the RTPS layer:

Add the IPayloadPool interface to the API-ref
Add the IChangePool interface to the API-ref
Describe the functionality of the pool
Describe how the default implementation works
Explain how to create endpoints with a custom pool
Provide an example using a custom pool

note: Keeping it as draft as tests will fail until all implementations are in place in Fast DDS.

Build status:

Build status:

Build status:

Build status:

Build status:


Change Open Knowledge International to Open Knowledge Foundation

Did the branding change again? https://okfn.org/ and other seemingly authoritative venues say International.

Hi @mlinksva. I work on the communications team at Open Knowledge (see https://okfn.org/team for proof).
We're building up to rebranding from Open Knowledge International back to Open Knowledge Foundation. This will be announced publicly on Monday 20th May 2019 so I'm starting to request changes on lots of sites so that everything can be in place by that date. Hope that makes sense.


MapLibre GL Core Build or Pull Request

Added a simple Github Workflow that will be triggered if any MapLibre GL Core path is changed. It will ignore all paths for any platform specific code.
test: Have tested with my local fork for include and src directory.
Next change is to deduce how we can export sources and binaries from this build that Android build can use. Or find a way for Android to rely on core build.

Also please add ccache, it will make subsequent builds much faster.

The root CMakeLists.txt needs to be added to the paths (this is true for other workflows as well).

Also please add ccache, it will make subsequent builds much faster.

Ah, was thinking about it. Wanted to start with something darn simple. Will add it in upcoming commit.

Reading up on reusable workflow. The moment I am done, will come back to this. :) Thank you for the quick responses. ❤

The root CMakeLists.txt needs to be added to the paths (this is true for other workflows as well).

This is taken care of.

Also please add ccache, it will make subsequent builds much faster.

This is also taken care of.

Ready to be reviewed again.

I would like to trigger the workflow to make sure it runs through.

Totally open to. Now that I read your comment, it feels like I didn't land my opinion.
I am saying that we need a strategy for backward incompatible changes in core. Especially the ones that are not trivial to address on the platforms in the single change. :)


Not Working in Android 13

This is my code.
await SoundMode.setSoundMode(RingerModeStatus.normal);
VolumeController().setVolume(1);
It is not working in vibrate mode

Hi @thelastknightahk , appreciate your interest in the plugin! Could you let me know the model of the device where you experienced the issue?
Additionally, I'd like more details on what exactly isn't working. Was the plugin unable to switch from normal ringer mode to vibrate, or vice versa?
I've successfully tested the plugin on an Android 13 emulator, so any additional information you provide will help me better understand and address the problem you're facing. Thanks!

In my device, it cannot convert silence to ringer mode
Best regards
On Sun, 10 Dec 2023, 12:57 pm Jan Owyeong, @.***> wrote:

Hi @thelastknightahk https://github.com/thelastknightahk , appreciate
your interest in the plugin! Could you let me know the model of the device
where you experienced the issue?
Additionally, I'd like more details on what exactly isn't working. Was the
plugin unable to switch from normal ringer mode to vibrate, or vice versa?
I've successfully tested the plugin on an Android 13 emulator, so any
additional information you provide will help me better understand and
address the problem you're facing. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/TryingOutSomething/sound_mode/issues/16#issuecomment-1848872449,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOHYC2763UYCPGXILQW7TBTYIVI45AVCNFSM6AAAAAA72GAHLSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBYHA3TENBUHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @thelastknightahk , it's possible that the plugin's lack of support for Android 13 is causing the issue. I've addressed this by adding Android 13 support to the plugin and pushing the update to pub. Please test it and inform me if it resolves your problem.

Yes, thanks sir. I will try.
Best regards
On Sun, 10 Dec 2023, 3:06 pm Jan Owyeong, @.***> wrote:

Hi @thelastknightahk https://github.com/thelastknightahk , it's
possible that the plugin's lack of support for Android 13 is causing the
issue. I've addressed this by adding Android 13 support to the plugin and
pushing the update to pub. Please test it and inform me if it resolves your
problem.
—
Reply to this email directly, view it on GitHub
https://github.com/TryingOutSomething/sound_mode/issues/16#issuecomment-1848895854,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOHYC24DAMAINRM2GQQNHDLYIVYBTAVCNFSM6AAAAAA72GAHLSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBYHA4TKOBVGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Specify a board type to python scripts. Add RGB FOV option

New options added:

for depthai.py:

  -brd BOARD, --board BOARD
                        BW1097, BW1098OBC - Board type from resources/boards/
                        (not case-sensitive). Or path to a custom .json board
                        config. Mutually exclusive with [-fv -rfv -b -r -w]
  -rfv RGB_FIELD_OF_VIEW, --rgb-field-of-view RGB_FIELD_OF_VIEW
                        Horizontal field of view (HFOV) for the RGB camera in
                        [deg]. Default: 68.7938deg.

for calibrate.py:

  -brd BOARD, --board BOARD
                        BW1097, BW1098OBC - Board type from resources/boards/
                        (not case-sensitive). Or path to a custom .json board
                        config. Mutually exclusive with [-fv -b -w]

Example usage:

python3 calibrate.py -brd BW1097
python3 calibrate.py -brd bw1098obc
python3 test.py -brd bw1098obc -e
(-e to write config and calib to EEPROM, or -o just to override it for current run when it is already programmed)
python3 test.py -brd path/to/my/custom_board.json
(please refer to resources/boards/ for examples on how to create the custom config)

Resolves issue https://github.com/luxonis/depthai/issues/73
Also includes a small fix for running with Python3.5.

@alex-luxonis the CLI options here work well. I tested the bw1098obc and bw1097, and I could see in the print statements that the correct values were applied. The custom config section is perfect for those using the BW1098FFC.
I had some trouble here, again, with storing/reading back from eeprom. I tried python3 test.py -brd bw1098obc -e, which worked to apply the settings on the immediate run. But when I tried a second time with python3 test.py to see if the board config would be loaded up and printed, it was not. Seems to be the same as what I was seeing in my other comment.

@Luxonis-Brian This behavior is expected. The host always sends a config, even if no command options are specified. It a file resources/depthai.calib is not found, it will use resources/default.calib, and the default parameters from the python scripts (or from the lib if certain fields are not initialized in the scripts) are used.
The host config (as in the logs starting with config_h2d json:) is used if the options -e or -o are passed, or if no valid data is found in EEPROM.
But yes, we should also implement retrieving data from EEPROM, for display and use in the host app.

Thanks for the explanation @alex-luxonis. Makes more sense now!

Thoughts on this one @szabi-luxonis ?  I'll try to test again today as well...

LGTM

LGTM


Add automatic JS license generation

Removed librejs file and replaced it with a plaintext file that is built from all JS dependencies that are included in the webpack build. It does not cover the few remaining statically vendored files and fomantic is added manually because it's not yet in the webpack build process.
Fixes: https://github.com/go-gitea/gitea/issues/11630
Sample License (source):
--------------------------------------------------------------------------------
<EMAIL_ADDRESS>- MIT
--------------------------------------------------------------------------------
# The MIT License

Permission is hereby granted, free of charge, to any person obtaining a copy of
this software and associated documentation files (the 'Software'), to deal in
the Software without restriction, including without limitation the rights to
use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
the Software, and to permit persons to whom the Software is furnished to do so,
subject to the following conditions:

The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

lint error

Lint fixed. Fingers crossed the integration test passes.

Completed GET /js/licenses.txt 404 Not Found in 331.473µs
    links_test.go:41:
          Error Trace:  integration_test.go:427
                              links_test.go:41
          Error:        Not equal:
                        expected: 200
                        actual  : 404
          Test:         TestLinksNoLogin
          Messages:     Request: GET /js/licenses.txt

I guess webpack files are not available by the time these test run.

@silverwind I think we can remove this url from the test

While I do think that there is some value in testing webpack build, it's probably not warranted the extra load it creates on CI. Frontend build was removed in https://github.com/go-gitea/gitea/pull/11159 from the testing steps, btw.

But it is not compatible with librejs imho

LibreJS is a meme imho. It's Firefox extension has a whopping 257 users and their approach can not possibly work with bundlers like webpack anyways.

I'm not against the idea of librejs but I see no sane way of handing out the licensing info in a parsable format. Of course, if there is a webpack plugin to handle this task, I'm all for adding it, but there appears to be none.

ping @bot

With vendored golang dependencies, it shouldn't be too hard to also add them to the output. Once they are unvendored, it will be more tricky thought and we'd probably need a tool to retrieve them.


Fix issue#1263

Small issues, small fixes... Moving forward and learning.

I checked it out from master. #1263 now closed.


[Test][Monitor] add environment variables needed by AAD tests

Apply fix similar to the one in PR #29262

build failure is unrelated and has been fixed in main. overriding
/check-enforcer override

/check-enforcer override


[HOLD for payment 2021-12-06] Own messages are marked as unread when opening the app in another device

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

I use Expensify on device "A"
Sent a bunch of messages from account "A"
Stopped using device "A" (exited app)
Started using device "B" with the same account "A"
Saw that I have unread messages in a conversation
All the unread messages are messages that were sent by me from another device

Expected Result:
Own messages should not be marked as unread.
Actual Result:
Own messages are marked as unread.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.1.0
Reproducible in staging?: yes
Reproducible in production?: yes
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @kidroca
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1632241925007700
View all open jobs on GitHub

Triggered auto assignment to @TomatoToaster (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

https://github.com/Expensify/App/pull/4887 Or https://github.com/Expensify/App/pull/5155 could be the cause.

Triggered auto assignment to @bfitzexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

These PRs are not the cause

#4874 have no effect on updateLastReadActionID, it only changes the marker handling
#5155 Involves updateLastReadActionID for componentDidMount but it was already called per each mount before the change
the issue is reproducible on prior commits

The bug comes from the call to ReportActionView.componentDidMount which would also call updateLastReadActionID

We're fetching Report summaries and actions (particularly the latest actions for the currently viewed report)
componentDidMount runs and calls updateLastReadActionID before we're done fetching
We setLocalLastRead with the last action we currently have for that report
We also tell the backend to set this as the last action we've read
Soon fetching report summaries and action completes and we have a new last action for that report
We have the received actions marked as new due to what happened just before

New more specific steps to recreate - looks like the issue would happen only when the same report is loaded when you start the app - you have to had ended the app on the same Report on both platforms for this to happen - the issue won't happen if you don't start on the same report for both devices.
https://user-images.githubusercontent.com/12156624/134339344-6e4fdcad-ecba-4370-aed0-84c7547db10a.mp4

Upwork posting

Triggered auto assignment to @johnmlee101 (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Reviewed the PR based on the request here https://github.com/Expensify/App/pull/6095#issuecomment-954062016.

Dropping this issue in here so we'll hopefully notice there when this issue closes. Curious if they're related
https://github.com/Expensify/App/issues/3981

reviewed!

Merged, and waiting to be moved to main app, out of staging

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.1.16-10 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/6095

If no regressions arise, payment will be issued on 2021-12-06. :confetti_ball:

Contract ended and paid - thank you @jolivervidal.
@kidroca, mind applying to this job so I can pay a reporting bonus?

Thank you! I'm glad to contribute for the first time.
I just want to ask one question @bfitzexpensify: mattallen said that in the slack channel. Would this PR be one of those that would have a 250$ bonus for the hold? Thanks

@bfitzexpensify when you post jobs in Upwork, will you please be sure to include the issue number in the title, it makes it easier to search for them in Upwork.
@jolivervidal correct, you are eligible for a $250 bonus based on my mistake mentioned in the slack link above.  I just added it to your contract, thanks for the help!


Fix Imputer. Added Imputer tests

Reference Issues/PRs
Closes #1643

Why after? Does it not make sense to have bug fixes in this release?

Why after? Does it not make sense to have bug fixes in this release?

Exactly my point! They were on the 0.9.0 milestone, ready to merge.
A few days of delay will result in sticking to the milestone plan and improved reliability, quality.
Let's not rush the release, @mloning.

Hi @aiwalter, yes the release was ready to go, that's why I wanted to hold back on this.


[EMB-203] Quick text changes

Purpose
Fix some additional text problems from QA
Summary of Changes

Capitalize Twitter
Change quickfiles to Quick Files

Side Effects / Testing Notes
None
Ticket
https://openscience.atlassian.net/browse/EMB-203
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

Coverage remained the same at 30.443% when pulling 225c44ffaa69ba3452386a8f54433e57496690ca on brianjgeiger:feature/support-text-fixes-2 into 9f2d8f7c0b25fb1f6f92f334d8df10e8d2e13fc4 on CenterForOpenScience:develop.


Add Show-HelpPreview -MarkdownFile <string[]> parameterSet

It's nice(at least for development) to have a one-liner to preview help engine output directly from markdown.
Please, up-vote, if you want this feature.
> Show-HelpPreview -MarkdownFile .\docs\Show-HelpPreview.md -AsObject

Will output help object directly from markdown.

@vors so the intent would be to convert to maml on the fly and render the output as it would be rendered from Get-Help?

Yes! This is just a matter of combining two steps into one, using a temp file and cleaning up.

@vors Could we just extend the existing Get-HelpPreview to handle markdown files and just do the render based on the type of file passed to -Path?

Yeap, totally make sense.

Thank you for this suggestion - With the upcoming release of Microsoft.Powershell.PlatyPS v1.0, the workflow for this has changed dramatically. Please evaluate your scenario and needs with the new version once its released.


bindOuterClick service doesn't work with angularui/ng-selelct

Hi Maurizio,
First of all, thanks for creating mobile-angular-ui. this is a great framework.
I'm trying to use this in my project. However, there seems to be some issue with bindOutClick service.
My project setting is similar to the demo project, except for I have a angularui /ng-select directive on the left sidebar.
The left sidebar suppose  to be only hide when click the main page, but after I put the angularui/ng-select in. it hides when I click any select item of the ng-select directive.
I've had a quick look, seems bindOuterClick service didn't realized that elements of  ng-select directive (which generated by angularjis) is within left sidebar. So triggered outclick call back.
I've tried to digger deeper, but I'm not good enough with angularjs so didn't go any further.
Do you have any idea what could be a proper fix for that?
Thanks,
Ryan

I am not sure if this is a outclick problem, it could surely be one.
But just to make sure, if you are using ui-toggle= on the sidebar it could be the following:
It could the ui-toggle-off="uiSidebarLeft" is causing this. You can replace it by ui-state="uiSideBar" which only initializes the ui-state as a shared state.
ui-toggle initializes the sharedstate and responds to clicks on the whole element.
Let me know if this solves your problem.

hi
I do have a ui-toggle in my sidebar left, but it is on a sub div rather than the main div

 Search

I've tried to change it to

 Search  one

which didn't help


Don't disconnect if insufficient peers

For 3 types of infractions, don't actively disconnect

timeout
useless response
IO exception (common from DeFramer)

And, don't do any outbound disconnects if EthPeers count is below maxPeers
With these changes, while below maxPeers, the only disconnects that happen with established peers are those found by removing peers if the connection becomes disconnected. This still happens quite frequently.
TODO - test this PR out on a mainnet node

Still logging when we otherwise would be disconnecting (outbound) and calls to registerDisconnect (inbound)
grep "Disconnect - " besu03.log
2023-11-29 13:04:32.541+10:00 | nioEventLoopGroup-3-6 | DEBUG | EthProtocolManager | Not attempting (insufficient peers) Disconnect - Outbound - 0x10 SUBPROTOCOL_TRIGGERED - 0xcc1a0cf5b7239909... - 6 peers left
2023-11-29 13:04:32.545+10:00 | nioEventLoopGroup-3-5 | DEBUG | EthProtocolManager | Did not register Disconnect - Inbound - 0x UNKNOWN - 0x46ad5ac2c66abe4f... - 6 peers left
2023-11-29 13:04:32.547+10:00 | nioEventLoopGroup-3-9 | DEBUG | EthProtocolManager | Not attempting (insufficient peers) Disconnect - Outbound - 0x10 SUBPROTOCOL_TRIGGERED - 0xd1905e4c62cf3d47... - 6 peers left
2023-11-29 13:04:36.314+10:00 | nioEventLoopGroup-3-2 | DEBUG | EthProtocolManager | Not attempting (insufficient peers) Disconnect - Outbound - 0x10 SUBPROTOCOL_TRIGGERED - 0xe0a51409471df7d0... - 6 peers left
2023-11-29 13:04:36.315+10:00 | nioEventLoopGroup-3-2 | DEBUG | EthProtocolManager | Did not register Disconnect - Inbound - 0x10 SUBPROTOCOL_TRIGGERED - 0xebc701ca568a41a4... - 6 peers left
2023-11-29 13:04:36.540+10:00 | nioEventLoopGroup-3-7 | DEBUG | EthProtocolManager | Not attempting (insufficient peers) Disconnect - Outbound - 0x10 SUBPROTOCOL_TRIGGERED - 0xf9c124545462969d... - 6 peers left
2023-11-29 13:04:36.565+10:00 | nioEventLoopGroup-3-7 | DEBUG | EthProtocolManager | Not attempting (insufficient peers) Disconnect - Outbound - 0x10 SUBPROTOCOL_TRIGGERED - 0x9798421d28da908c... - 6 peers left
2023-11-29 13:04:36.611+10:00 | nioEventLoopGroup-3-5 | DEBUG | EthProtocolManager | Not attempting (insufficient peers) Disconnect - Outbound - 0x10 SUBPROTOCOL_TRIGGERED - 0x32eb7568c362dfe0... - 6 peers left
2023-11-29 13:04:36.616+10:00 | nioEventLoopGroup-3-2 | DEBUG | EthProtocolManager | Not attempting (insufficient peers) Disconnect - Outbound - 0x10 SUBPROTOCOL_TRIGGERED - 0x1027c602936bcf35... - 6 peers left
2023-11-29 13:04:36.643+10:00 | nioEventLoopGroup-3-7 | DEBUG | EthProtocolManager | Did not register Disconnect - Inbound - 0x10 SUBPROTOCOL_TRIGGERED - 0xcf7662490b312ec8... - 6 peers left

this code only disconnects the peer when the underlying connection is already disconnected. So in this case, we do actually want to disconnect.


added some swedish translations

I translated some phrases to Swedish.

@simonljus  could you sign our CLA so we can accept your PR?

accepted


Add route based code splitting

I think you should do
() => import('<Component>')

instead of
import(() => import('<Component>'))

Also noticed you tried to import './containers//HomePage' instead of './containers/Homepage'

Yea that was a mistake. Did that locally and it still doesn't work. Still investigating

I think it would be more appropriate to create an example out of this instead of add this to erb. We should document this in erb site under a performance section


Add #each and #map to Object and Nil

I was going over some old episodes from Ruby Tapas and really like this idea from the Safe Navigation series.
https://www.rubytapas.com/2015/11/23/episode-361-safe-navigation-part-1-reliable-associations/
https://www.rubytapas.com/2015/11/26/episode-362-safe-navigation-part-2-try-try-again/
https://www.rubytapas.com/2015/11/30/episode-363-safe-navigation-part-3-everything-is-optional/
https://www.rubytapas.com/2015/12/03/episode-364-safe-navigation-part-3-refinements/
https://www.rubytapas.com/2015/12/07/episode-365-safe-navigation-part-5-adapter/
This addresses a few problems but I like how you can safely navigate through objects and arrays.
This involved adding #each and #map to Object and Nil.  I find the solution to be a little off putting but maybe another name to this solution would be good.  Something like #in and #through for #each and #map respectively.

Wouldn't this just hide the problem that you're trying to iterate nil instead of forcing you to actually handle it?

I mean this could be a pattern for handling/dealing with nil but I can see your point.  There are other was to handle nil as well.

Yes, we already have this.  #try.  arr.try &.map { |v| v + 1}.


Fixed package.json funding format

Reference Issues/PRs
none
What does this implement/fix?

Fixed package.json funding format

Object has fewer properties than the required number of 2

package-json#funding

Thanks, but the "type" doesn't seem to be required since it works as expected :)
Will merge it anyway.


Added support for SRP auth

#1
I've tested with python2.7 and with a user that don't require a password change on auth.
Please, review and comment.

Tested with python3.4 and requested changes are made.

bad ass, thanks @armicron


Questions about data files

Dear author, hello ! I am a graduate student majoring in statistics. I am fascinated by your article on the use of PyGSLIB for resource estimation of this geological algorithm. I have run through your code and understand the meaning of most of the data in the data file. However, the meaning of the CU data column in the assay _ BABBITT data file is not clear. Can you tell me the meaning of this data column ?

CU stands for the Copper values in %. So the length of the assay row denoted by from - to has a copper value as given in 'CU' column.


maybe add my badges

add my badge
even tho its for a kid lang it works well

Hi @ctrl-shift-make,
Thanks for the suggestion.
I'm afraid that this tool is not widely used by enough person, I won't add it. Sorry


Dev: While Entering magic code, code is blinking and not able to enter magic code

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to login Page
Enter a valid Email
Try to add magic code

Expected Result:
Magic code should be not blink, and enter properly
Actual Result:
Not able to add magic code. magic code is blinking
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.23-6
Reproducible in staging?: n/a
Reproducible in production?: n/a
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/78bc9cd4-74fa-4c66-a6c1-44d3053b5ac7
Expensify/Expensify Issue URL:
Issue reported by: @niravkakadiya25
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685769358664389
View all open jobs on GitHub

Thanks @jczekalski , closing out


Follow changes in std / rustc

This PR contains the changes to std::num currently.

I expect to add the changes of the second pass stabilization of std::string very soon.

String stabilization commit added.

thanks again!


feat: Add support for Safari, Brave and Opera browsers

Description
Add support for Safari, Brave and Opera browsers.
Type of change

[ ] Bugfix
[x] Feature

Checklist

[x] I have read the contribution guide: https://vaadin.com/docs/latest/guide/contributing/overview/
[x] I have added a description following the guideline.
[ ] The issue is created in the corresponding repository and I have referenced it.
[ ] I have added tests to ensure my change is effective and works as intended.
[ ] New and existing tests are passing locally with my change.
[ ] I have performed self-review and corrected misspellings.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Looks good to me, thanks!

Thank you. When should I expect a new release ?

8.5.0 is out


Refactor MNIST to subclass H5PYDataset

See this tutorial for more information.

@Thrandis This is your ticket.