Add overwrite functionality

We might be able to use the load_resource_pack() with replace_files set to true, to easily overwrite for example textures.

That would make a mod like Brotato-Explosion-Mute as easy as locating the image file you want to replace and adding a new one at the mirrored location in the mod zip.
( In this case res://projectiles/rocket/explosion.png )

If not handled with care, this has potential for carnage

I successfully tested this with Brotato-Explosion-Mute, by allowing file replacement.
The complexity to achieve the desired result is greatly reduced.
KANA-ExplosionMuteV2.zip

From my reply on Discord:

What about a separate folder? Eg mods-overrides. That way it'd give them a standardised structure and would make it clear when a mod is overriding something in vanilla
Then we could just run _load_mod_zips twice, with the second call being passed an arg that says to use the "overrides" folder and use true for load_resource_pack

Darkly77 - Another use case for replacements: You could do stuff like make a balance pack for Brotato. Eg people often say Plank is weak, so someone could release a mod that improves its scaling by replacing the vanilla file.

From my reply on Discord:

What about a separate folder? Eg mods-overrides. That way it'd give them a standardised structure and would make it clear when a mod is overriding something in vanilla
Then we could just run _load_mod_zips twice, with the second call being passed an arg that says to use the "overrides" folder and use true for load_resource_pack

Now that I'm working on it - we can't do a separate folder because the only way to overwrite is to mirror the file paths inside the mod zip.
yourmod.zip
├───.import
└───mods-unpacked
    └───Author-ModName
        ├───mod_main.gd
        └───manifest.json
└───projectiles
    └───rocket
        └───explosion.png

There is also no point in adding something like "is_overwrite": true to the manifest.json That is loaded after we need to set the replace flag for ProjectSettings.load_resource_pack()
So the only way to send data in this stage to the mod loader is the name of the zip file?
That would mean we can't use arbitrary zip names any more.

(copied from Discord:)
Hey, I've been looking into this too! I realised that allowing full overrides would mean we're loading stuff from mod ZIPs before we get a chance to read their meta data. That means a mod could be published that doesn't even have a manifest
So my approach is to simply replace resources from within mod_main. That ensures mods stay within our current standards

Example mod with my approach. Changes the color of the logo. Has 3 different colors, the last one mentioned in the code is the one that gets used.
mod_main.gd (simplified):
extends Node

func _init(modLoader = ModLoader):
	var new1 = preload("res://mods-unpacked/Darkly77-LogoReplacement/overrides/ui/menus/title_screen/ui_logo__blue.png")
	new1.take_over_path("res://ui/menus/title_screen/ui_logo.png")

Download Mod (for ML v3.1.0):

Darkly77-LogoReplacement--ML-3.1.0.zip

🇦 ) Mod Loader Handling - Restart game once ( if a overwrite mod is added / removed )

Details

Creating an overwrites.gd inside the mod loader can work
By checking for an overwrites folder at the root of each mod folder
If detected it gets all files inside the overwrites and appends the required code to the overwrites.gd file

var overwrite_index = preload("res://mods-unpacked/ModAuthor-ModName/overrides/path/to/overwrite/resource")
overwrite_index.take_over_path("res://path/to/original/resource")

That overwrites.gd script gets initialized at the end of the mod loaders _init() function

Problem with this solution, is that the overwrites.gd file is created on the first run of the game.
To Allow the replacement resources to be preloaded the file needs to exist before the start.
That's why a game restart is required.

Currently a game can only be closed via script not restarted, this will most likely change in Godot 3.6:
https://github.com/godotengine/godot/pull/66168

🇧 ) Let mod authors create the overwrites.gd at the root of the mod folder

Details

My current proposed solution on how to handle the overwrites
Mod authors create the overwrites.gd script and overwrites folder in there mod folder:

We provide tooling to make the creation of the overwrites.gd script a one click thing
People ho don't want to use extra tooling can just write the overwrites.gd them self

Example Mod with this setup:
KANA-OverwritesTest.zip
Testing Godot Project
ModdingTest.zip

🇨 ) use --script run.gd

Details

Using the --script cmd arg to run a script before the game starts
Not tested, because with that setup we will need to unzip the mods in the run.gd
And we want to get rid of it anyway

Just to reference it here as well:
There is also the possibility to reload a scene with a new PackedScene.

For that I used get_tree().change_scene_to(PackedScene) in a script extension of the scene that is the owner of the overwritten resource.

That allows to use just load to load the replacement resource

load can take variable resource paths

That potentially allows to add a util function to the mod loader for mod authors to use

I didn't explorer this further because the preload solution currently seems way cleaner and doesn't require to reload / replace in a scenes _ready() function.

Example Brotato Mod of this solution:
Darkly77-LogoReplacement.zip

closed by https://github.com/GodotModding/godot-mod-loader/pull/74

Darkly77 - Another use case for replacements: You could do stuff like make a balance pack for Brotato [...]

I've found that replacing files can be a bad idea because it can mess up extended scripts.
I was trying a few things with the file character_selection.tscn, and after replacing it I found that the extended scripts for character_selection.gd (which is referenced in character_selection.tscn) no longer worked. Replacing .tres files likely has the same negative effect, where any .gd scripts that are referenced in that .tres file will ignore their extensions.
This needs further testing though.


notification system

added the possibility to write
'default', 'success','error','warning','info'
is documented

@Valecant don't bother the style, we will change the linting rules to fix it

@Valecant aren't supposed the errors to surface, for example, if the connection to the api fails? i stopped the api, the console reports the errors, but nothing happens in the ui

@Valecant check commit 533ee753c0e56c4cde52e3e0d0210cb67e7ed7ae
i tried to:

surface connectivity errors
use info to inform on success (for the change of user preference)

my code is just an example, but should help you to apply the same principles to the whole app.
for sure there are other client (at least apollo in app.js and maybe an axiom too) that don't open a pop up in case of errors, please tackle that

If the API's or Graphql are offline, the app is going to show a banner, and try to reconnect, if the services are again back online the app will automatically reconnect.

If the API's or Graphql are offline, the app is going to show a banner, and try to reconnect, if the services are again back online the app will automatically reconnect.

graphql, as by meeting discussion, should not be a blocking api. not need to fix this in this pr, can be tackled in relation to #38
in a software architecture you have essential dependencies, and dependencies that are not essential.
the ui should work fine with only:

keycloak
management-api

if graphql is down, we just show a message "preferences cannot be stored since configuration api is not reacheable"


When moving a namespace that has workloads with host ports/node ports from "system" projects , network policies relating to host ports/node ports do not get created resulting in the node ports/host ports not working.

Rancher versions: Build from master
Steps to Reproduce:
Create workloads with host port  and node ports in namespace that is part of "System" project.
Move this namespace out of  "System" project.
After the move,  network policies relating to host port  and node ports are not getting created in the namespace . This results in host port  and node ports not be accessible anymore.
Expected Behavior:
After the move,  network policies relating to host port  and node ports should get created in this namespace and host port  and node ports should continue to be accessible.

rancher/rancher:master 8/6
Network policies are being created when moving out of System project.
Created a namespace in System project, containing a workload with a nodeport.
kubectl get networkpolicies --all-namespaces should show no network policies for the new namespace in the System project.
Move the created namespace to the Default project.
kubectl get networkpolicies --all-namespaces shows network policies for the namespace and workload which was moved.


TextEditor.setDecorations() doesn't respects the decorations ordering

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.74.0
OS Version: Windows 11 x64

Steps to Reproduce:
I'm trying to apply multiple decorations in the same line. My intention is to highlight some parts of the text in other colors.
For example:

It seems to me that it is something possible, however, vscode is not respecting the order of the decorations, and ends up inverting them randomly.
The code is something like:
const decorationDefault = window.createTextEditorDecorationType({})

const flush = (editor: TextEditor) => {
  // Note: same range for all decorations.
  const range = new Range(new Position(1, 4096), new Position(1, 4096))

  editor.setDecorations(decorationDefault, [
    { range, renderOptions: { after: { color: "gray", contentText: "Hello, " } } },
    { range, renderOptions: { after: { color: "blue", contentText: "user" } } },
    { range, renderOptions: { after: { color: "gray", contentText: "!!!" } } },
  ])
}

My question is, is there any way to ensure order is maintained like this example? None of my attempts worked. It looks like a bug to me, or I'm misusing it.

After note: I managed to get this to work in a very "hackish" way. For each style variation, I need to create a new style. So if I need 8 variations on the same line, I'll need to manage 8 different styles. I really hoped that I could do it with one style, actually.

We store decorations in a red-black interval tree for performance reasons. Unfortunately, we can't guarantee that the reading order from the tree is the same as the insertion order due to the various rotation operations that occur in a red-black tree to keep it balanced. I'm sorry, but in the foreseeable future the only way is the workaround you have found.

@alexdima Oh, that seems very strange to me (probably because I didn't understand the red-black tree well).
The red-black tree update struck me as surprising. The performance was very valuable indeed. However, I still can't understand how it works in such a "random" way, since the data is always pushed in a certain order.
If the red-black tree decides the render order somehow, what parameter is used to make this happen? Perhaps it is possible to control this from my side?
I am not satisfied with the method I used. Well, it works fine, however, it seems too hackish (and probably much slower than if it were done natively).


module resolve issue - everything imports as type Any

How to import this module correct?
Is there somthing that need to be fixed?
Using python 3.12 on Windows for development.
Later it runs on a Debian under Docker where typings are not needed.

Your import is just fine.
The module is implemented as a native extension in cython.
It's quite possible that this is a tooling limitation where it can't introspect into the module.
Hopefully the documentation is clear enough even if code completion might not fully work in this instance.


Media features scoped as keyword_query

@media screen and (color) and (color: 8) {}

The first color is keyword_query whereas the second is feature_name. Both should be feature_name.
W3C documentation.

Like I mentioned in the other issue, we can't determine this dynamically - use queries to fine-tune this


Running prettier via the pre-commit hook creates invalid mdx

Describe the bug
Running husky creates invalid mdx.
To Reproduce
Steps to reproduce the behavior:

Standard setup
Create blog post containing Press Cmd+Shift+P and
Blog post works fine.
Run husky by committing the blog post.
The code snippet above is converted to

Press
    Cmd+Shift+P

and

Which generates the following error
Error: Build failed with 1 error:
_mdx_bundler_entry_point-526e1ca1-0ac1-4cfd-bcc7-534cd64daa30.mdx:20:0: ERROR: [plugin: esbuild-xdm] Expected a closing tag for `</kbd>` (20:7-20:12) before the end of `paragraph`

Expected behavior
Husky should not generate invalid mdx.
System Info (if dev / build issue):

OS: Mac OS 10.15.4
Node version: 16.9.1
Npm version: 8.3.0

Additional context
The problem appears also with other html tags.
A workaround is to disable husky temporarily by renaming the .husky folder to something else.
According to the mdx playground the markup is really invalid, i.e., it is not a problem with the mdx setup here.
This is probably an upstream bug, but I wanted to report this here as well.

Husky just runs lint-staged which runs prettier. Here's the line in package.json. Looks like prettier might not be that smart in formatting HTML in mdx. Try removing md|mdx and see if that works?
It could also be the case that the current prettier version is slightly outdated and maybe upgrading would fix the issue. Let me know if upgrading prettier helps?

I created an upstream bug with prettier, until this is fixed I am keeping husky disabled.

The bug is now tracked in https://github.com/prettier/prettier/issues/12209. I'm closing this issue until the bug is fixed as we can't do anything right now.

Re-opening the issue for awareness and to avoid similar issues being posted. Have also changed the title to more accurately reflect the bug.
If you encounter the issue, currently the best approach would be to exclude md/mdx files from being linted by prettier. You can do that by removing md|mdx in this line in package.json

Prettier supports only MDX v1 but MDX is already at version 3. Therefore, I suggest removing |mdx from https://github.com/timlrx/tailwind-nextjs-starter-blog/blob/2d8ac8061a2e423d0cbbec85411b723755ee556f/package.json#L69-L71 because Prettier is not suitable to lint MDX v2+!
There are other options to lint MDX

https://github.com/mdx-js/eslint-mdx
https://github.com/remarkjs/remark-lint
https://github.com/mdx-js/mdx/tree/main/packages/remark-mdx


[suitesparse] build failure

Host Environment

OS: [Win 10]
Compiler: MSVC 141

To Reproduce
Steps to reproduce the behavior:
vcpkg install suitesparse
Failure logs
C:\Users\ryanh>vcpkg install suitesparse:x64-windows
The following packages will be built and installed:
    suitesparse[core]:x64-windows
Starting package 1/1: suitesparse:x64-windows
Building package suitesparse[core]:x64-windows...
-- Note: suitesparse only supports static library linkage. Building static library.
-- Using cached E:/Documents/Visual Studio 2010/Projects/vcpkg-master/downloads/SuiteSparse-5.4.0.tar.gz
-- Using source at E:/Documents/Visual Studio 2010/Projects/vcpkg-master/buildtrees/suitesparse/src/teSparse-5-fc74a7e67e
-- Using cached E:/Documents/Visual Studio 2010/Projects/vcpkg-master/downloads/jlblancoc-suitesparse-metis-for-windows-v1.4.0.tar.gz
-- Using source at E:/Documents/Visual Studio 2010/Projects/vcpkg-master/buildtrees/suitesparse/src/v1.4.0-113b6dfb93
-- Copying SuiteSparse source files...
-- Copying SuiteSparse source files... done
-- Removing integrated lapack and metis libs...
-- Removing integrated lapack and metis libs... done
-- Configuring x64-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:58 (message):
    Command failed: ninja -v
    Working Directory: E:/Documents/Visual Studio 2010/Projects/vcpkg-master/buildtrees/suitesparse/x64-windows-rel/vcpkg-parallel-configure
    Error code: 1
    See logs for more information:
      E:\Documents\Visual Studio 2010\Projects\vcpkg-master\buildtrees\suitesparse\config-x64-windows-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_cmake.cmake:296 (vcpkg_execute_required_process)
  ports/suitesparse/portfile.cmake:48 (vcpkg_configure_cmake)
  scripts/ports.cmake:84 (include)

Error: Building package suitesparse:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: suitesparse:x64-windows
  Vcpkg version: 2019.08.16-nohash

**Additional context**
From the config-x64-windows-out.log it shows that the output directory path does not exist as it's been doubled up.

[1/2] cmd /c "cd ../../x64-windows-dbg && "C:/Program Files/CMake/bin/cmake.exe" "E:/Documents/Visual Studio 2010/Projects/vcpkg-master/buildtrees/suitesparse/src/v1.4.0-113b6dfb93" "-DBUILD_METIS=OFF" "-DUSE_VCPKG_METIS=OFF" "-DCMAKE_MAKE_PROGRAM=C:/Program Files (x86)/Microsoft Visual Studio/2017/Professional/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=OFF" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v141" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-DSUITESPARSE_INSTALL_PREFIX="E:/Documents/Visual Studio 2010/Projects/vcpkg-master/packages/suitesparse_x64-windows/debug"" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_INSTALL_PREFIX=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/packages/suitesparse_x64-windows/debug""
FAILED: ../../x64-windows-dbg/CMakeCache.txt
cmd /c "cd ../../x64-windows-dbg && "C:/Program Files/CMake/bin/cmake.exe" "E:/Documents/Visual Studio 2010/Projects/vcpkg-master/buildtrees/suitesparse/src/v1.4.0-113b6dfb93" "-DBUILD_METIS=OFF" "-DUSE_VCPKG_METIS=OFF" "-DCMAKE_MAKE_PROGRAM=C:/Program Files (x86)/Microsoft Visual Studio/2017/Professional/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=OFF" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v141" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-DSUITESPARSE_INSTALL_PREFIX="E:/Documents/Visual Studio 2010/Projects/vcpkg-master/packages/suitesparse_x64-windows/debug"" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_INSTALL_PREFIX=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/packages/suitesparse_x64-windows/debug""
CMake Error: The source directory "E:/Documents/Visual Studio 2010/Projects/vcpkg-master/buildtrees/suitesparse/x64-windows-dbg/2010/Projects/vcpkg-master/packages/suitesparse_x64-windows/debug" does not exist.
Specify --help for usage, or press the help button on the CMake GUI.
[2/2] cmd /c "cd .. && "C:/Program Files/CMake/bin/cmake.exe" "E:/Documents/Visual Studio 2010/Projects/vcpkg-master/buildtrees/suitesparse/src/v1.4.0-113b6dfb93" "-DBUILD_METIS=OFF" "-DUSE_VCPKG_METIS=OFF" "-DCMAKE_MAKE_PROGRAM=C:/Program Files (x86)/Microsoft Visual Studio/2017/Professional/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=OFF" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v141" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-DSUITESPARSE_INSTALL_PREFIX="E:/Documents/Visual Studio 2010/Projects/vcpkg-master/packages/suitesparse_x64-windows"" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/packages/suitesparse_x64-windows""
FAILED: ../CMakeCache.txt
cmd /c "cd .. && "C:/Program Files/CMake/bin/cmake.exe" "E:/Documents/Visual Studio 2010/Projects/vcpkg-master/buildtrees/suitesparse/src/v1.4.0-113b6dfb93" "-DBUILD_METIS=OFF" "-DUSE_VCPKG_METIS=OFF" "-DCMAKE_MAKE_PROGRAM=C:/Program Files (x86)/Microsoft Visual Studio/2017/Professional/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=OFF" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v141" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-DSUITESPARSE_INSTALL_PREFIX="E:/Documents/Visual Studio 2010/Projects/vcpkg-master/packages/suitesparse_x64-windows"" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=E:/Documents/Visual Studio 2010/Projects/vcpkg-master/packages/suitesparse_x64-windows""
CMake Error: The source directory "E:/Documents/Visual Studio 2010/Projects/vcpkg-master/buildtrees/suitesparse/x64-windows-rel/2010/Projects/vcpkg-master/packages/suitesparse_x64-windows" does not exist.
Specify --help for usage, or press the help button on the CMake GUI.
ninja: build stopped: subcommand failed.

E:/Documents/Visual Studio 2010/Projects/vcpkg-master
move to
E:/Projects/vcpkg-master
?

@gt1485a Thanks for reporting this issue.
It seems that this issue is caused by the fact that your vcpkg path contains spaces and is too long. Please shorten the vcpkg path and remove the spaces in the path.
Thanks.

CMake Error: The source directory "F:/Documents/Visual Studio 2010/Projects/vcpkg-master/buildtrees/suitesparse/x64-windows-rel/2010/Projects/vcpkg-master/packages/suitesparse_x64-windows" does not exist.

It seems that the source_path and installed_path are spliced in a path with spaces.

I can install many other packages without changing the path, i've only had the issue with suitesparse so far.  Thanks for the support @JackBoosY .

@gt1485a This may be because other ports do not have such a long path.

The code for paths in vcpkg has been updated. Does this issue still exist?

This issue still occurs in the latest vcpkg(2021-08-03-1a6d67b8150e94b7f473c6ab1c0cfd) when useing the long path.

The CMAKE_INSTALL_PREFIX variable set in vcpkg_configure_cmake should be enclosed in double quotes.

I can't reproduce this issue with path F:\Documents\Visual Studio 2010\Projects\vcpkg-master, please reopen this issue if it still exist.


how to get sender_params.json

how to get sender_params.json ,
the below  values:
"channelid": "", "authorization": "", "application_id": "", "guild_id": "", "session_id": "", "version": "", "id": "",

I gave an instruction in readme.md, you need you use Google Chrome's Developer Tools to find your own parameters


Added Merge Sort  [Java]

Fixed #61
By submitting this pull request I confirm I've read and complied with the below requirements.

[X] I have read the Contribution guidelines and I am confident that my PR reflects them.
[X] I have followed the coding guidelines for this project.
[X] My code follows the skeleton code structure.
[X] This pull request has a descriptive title. For example, Added {Algorithm name} [{Language}], not Update README.md or Added new code.

nopr


Basic CRA preset does not work on clean project

Hey I have tried setting up a new CRA project using CRA 3.3.0 and adding storybook (cli version 5.2.8) with the CRA preset and it immediately errors out. Below I am including the steps I took, the error message received, and a link to the git repo to reproduce the issue. Any help with what to do in this case would be great.
https://github.com/colingm/storybook-cra-error
npm i -g<EMAIL_ADDRESS><EMAIL_ADDRESS>create-react-app sanity --template typescript
cd sanity
sb init
npm i @storybook/preset-create-react-app
echo "module.exports = ['@storybook/preset-create-react-app'];" > .storybook/presets.js
npm run storybook

ERROR in ./node_modules/is-map/index.js
Module build failed (from ./node_modules/babel-loader/lib/index.js):
SyntaxError: /Users/cmaxfield/workspace/audit/sanity/node_modules/is-map/index.js: 'return' outside of function (12:1)

  10 | 		return false;
  11 | 	};
> 12 | 	return;
     | 	^
  13 | }
  14 |
  15 | var $mapHas = $Map ? Map.prototype.has : null;
    at Parser.raise (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:6983:17)
    at Parser.parseReturnStatement (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11027:12)
    at Parser.parseStatementContent (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10714:21)
    at Parser.parseStatement (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10666:17)
    at Parser.parseBlockOrModuleBlockBody (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11242:25)
    at Parser.parseBlockBody (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11229:10)
    at Parser.parseBlock (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11213:10)
    at Parser.parseStatementContent (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10742:21)
    at Parser.parseStatement (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10666:17)
    at Parser.parseIfStatement (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11020:28)
    at Parser.parseStatementContent (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10711:21)
    at Parser.parseStatement (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10666:17)
    at Parser.parseBlockOrModuleBlockBody (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11242:25)
    at Parser.parseBlockBody (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11229:10)
    at Parser.parseTopLevel (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10597:10)
    at Parser.parse (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:12107:10)
 @ ./node_modules/es-get-iterator/index.js 131:16-33
 @ ./node_modules/iterate-value/index.js
 @ ./node_modules/promise.allsettled/implementation.js
 @ ./node_modules/promise.allsettled/polyfill.js
 @ ./node_modules/promise.allsettled/shim.js
 @ ./node_modules/promise.allsettled/auto.js
 @ ./node_modules/airbnb-js-shims/target/es2020.js
 @ ./node_modules/airbnb-js-shims/target/es2019.js
 @ ./node_modules/airbnb-js-shims/target/es2018.js
 @ ./node_modules/airbnb-js-shims/target/es2017.js
 @ ./node_modules/airbnb-js-shims/target/es2016.js
 @ ./node_modules/airbnb-js-shims/target/es2015.js
 @ ./node_modules/airbnb-js-shims/target/es5.js
 @ ./node_modules/airbnb-js-shims/index.js
<EMAIL_ADDRESS> @ multi<EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/config.js (webpack)-hot-middleware/client.js?reload=true&quiet=true

ERROR in ./node_modules/is-set/index.js
Module build failed (from ./node_modules/babel-loader/lib/index.js):
SyntaxError: /Users/cmaxfield/workspace/audit/sanity/node_modules/is-set/index.js: 'return' outside of function (12:1)

  10 | 		return false;
  11 | 	};
> 12 | 	return;
     | 	^
  13 | }
  14 |
  15 | var $mapHas = $Map ? Map.prototype.has : null;
    at Parser.raise (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:6983:17)
    at Parser.parseReturnStatement (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11027:12)
    at Parser.parseStatementContent (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10714:21)
    at Parser.parseStatement (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10666:17)
    at Parser.parseBlockOrModuleBlockBody (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11242:25)
    at Parser.parseBlockBody (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11229:10)
    at Parser.parseBlock (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11213:10)
    at Parser.parseStatementContent (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10742:21)
    at Parser.parseStatement (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10666:17)
    at Parser.parseIfStatement (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11020:28)
    at Parser.parseStatementContent (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10711:21)
    at Parser.parseStatement (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10666:17)
    at Parser.parseBlockOrModuleBlockBody (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11242:25)
    at Parser.parseBlockBody (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:11229:10)
    at Parser.parseTopLevel (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:10597:10)
    at Parser.parse (/Users/cmaxfield/workspace/audit/sanity/node_modules/@babel/parser/lib/index.js:12107:10)
 @ ./node_modules/es-get-iterator/index.js 133:16-33
 @ ./node_modules/iterate-value/index.js
 @ ./node_modules/promise.allsettled/implementation.js
 @ ./node_modules/promise.allsettled/polyfill.js
 @ ./node_modules/promise.allsettled/shim.js
 @ ./node_modules/promise.allsettled/auto.js
 @ ./node_modules/airbnb-js-shims/target/es2020.js
 @ ./node_modules/airbnb-js-shims/target/es2019.js
 @ ./node_modules/airbnb-js-shims/target/es2018.js
 @ ./node_modules/airbnb-js-shims/target/es2017.js
 @ ./node_modules/airbnb-js-shims/target/es2016.js
 @ ./node_modules/airbnb-js-shims/target/es2015.js
 @ ./node_modules/airbnb-js-shims/target/es5.js
 @ ./node_modules/airbnb-js-shims/index.js
<EMAIL_ADDRESS> @ multi<EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/config.js (webpack)-hot-middleware/client.js?reload=true&quiet=true

What I did:
| setup | result |
|<EMAIL_ADDRESS>| O |
|<EMAIL_ADDRESS>+<EMAIL_ADDRESS>| X |
|<EMAIL_ADDRESS>| X |
|<EMAIL_ADDRESS>-<EMAIL_ADDRESS>| O |
|<EMAIL_ADDRESS>+ preset-cra@* | X |
In other words, the problem appears to be an incompatibility between @storybook/preset-create-react-app and something else in the ecosystem (latest version of CRA? some change in inspect-js (see also: https://github.com/inspect-js/is-map/pull/3)?

It seems that this rule tries to transpile is-map and fails?
https://github.com/facebook/create-react-app/blob/master/packages/react-scripts/config/webpack.config.js#L455

I've found the issue here and will push it shortly.

Fixed in @storybook/preset-create-react-app@1.5.0!! Please upgrade and try it out! Thanks @mrmckeb for the fix 🙌

Upgrading takes care of it, thank you so much for the quick turn around everyone!

@mrmckeb and @shilman -- What was the cause and solution of this issue? I am running into the same problem with another (more mature) CRA project. Downgrading deep-equal to 1.1.1 works around the problem but unsure of the true origin or "correct" fix. Any insight is appreciated.
Please see:
deep-equal issue #78

Top-level return is perfectly valid inside node modules; transpiling code you didn't author is unsafe; if you're going to transpile node_modules, then you should be sure to use settings that work on valid node modules :-)

@n8sabes, this was caused by the CRA preset applying the wrong loader to files imported by Storybook.
The correction is here:
https://github.com/storybookjs/presets/pull/77/files#diff-80b3f28ef184f3c7a4ac83b1974c3dc6R147
Which ensures that an earlier loader (we use oneOf) catches the file, so that it isn't accidentally processed as application code.

@ljharb you're right - but this was never meant to be transpiled, it was simply omitted from the original preset - the new preset was forked from that, and the bug was carried over.
This is the related code:
https://github.com/SeedJobs/react-scripts/blob/master/packages/react-scripts/config/webpack.config.js#L489

Hello guys !
I just get the exact same problem with newly generated project.
Steps :

npx create-react-app myproject --typescript
npx -p @storybook/cli sb init --type react
yarn add -D @storybook/preset-create-react-app
in ".storybook/main.js" > addons: ['@storybook/addon-actions', '@storybook/addon-links'],

But it results to :
Module parse failed: Unexpected token (10:33)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
| };
|
> export const ToStorybook = () => <Welcome showApp={linkTo('Button')} />;
|
| ToStorybook.story = {
Error: Module parse failed: Unexpected token (10:33)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
| };
|
> export const ToStorybook = () => <Welcome showApp={linkTo('Button')} />;
|
| ToStorybook.story = {
    at Object../stories/0-Welcome.stories.js (http://localhost:6006/main.bfbb4c2484bd25f72ff3.bundle.js:62:7)
    at __webpack_require__ (http://localhost:6006/runtime~main.bfbb4c2484bd25f72ff3.bundle.js:786:30)
    at fn (http://localhost:6006/runtime~main.bfbb4c2484bd25f72ff3.bundle.js:151:20)
    at webpackContext (http://localhost:6006/main.bfbb4c2484bd25f72ff3.bundle.js:36:9)
    at http://localhost:6006/vendors~main.bfbb4c2484bd25f72ff3.bundle.js:3849:31
    at Array.forEach (<anonymous>)
    at http://localhost:6006/vendors~main.bfbb4c2484bd25f72ff3.bundle.js:3848:22
    at Array.forEach (<anonymous>)
    at http://localhost:6006/vendors~main.bfbb4c2484bd25f72ff3.bundle.js:3847:14
    at render (http://localhost:6006/vendors~main.bfbb4c2484bd25f72ff3.bundle.js:1897:13)


Path not valid for this data type

Hello, I wish to host my images of pages other than self@, I thought to make destination: image:// to have it here user/images. I manage to add them in the right folder since the admin, unfortunately, I can not delete them, I have an error "Path not valid for this data type".

How are you managing them? Via a file field?  That could well be an issue that needs looking at.
Another approach is to simply create a folder user/pages/images and then use that as an empty page that just contains images.  Then you can easily access those images from any page:
![my image](/images/my-sample-image.jpg)

Yes, I handle this via file field.
From the moment I change the destination, I could add files but not delete them. Even if I put destination: user/pages images it does not work to delete the added images.
I found this to be my problem: "https://github.com/getgrav/grav-plugin-admin/issues/1619" unfortunately its solution does not work.


Revive USING MODEL in BQL

ESTIMATE ... FROM <population> MODELED BY <metamodel> USING MODELS <modelnos>
SIMULATE ... FROM <population> MODELED BY <metamodel> USING MODELS <modelnos>

Related: https://github.com/probcomp/cgpm/issues/208
Will cause the test suite to catch fire since all method in bqlfn.py will admit an additional parameter and break all associated tests in test_bql.py.

What is the revision you are suggesting?

Sorry "revive", not "revise", and I am referring to to cgpm_metamodel dropping all models.
https://github.com/probcomp/bayeslite/blob/master/src/metamodels/cgpm_metamodel.py#L275-L285

And, this ticket was a duplicate (from a stale tab):
https://github.com/probcomp/bayeslite/issues/544

And, this ticket was a duplicate (from a stale tab): #544

#544 was another ticket for DROP MODELS. This ticket is indeed still outstanding.
Progress on this ticket in 20170624-fsaad-using-model.

Design choice:

USING MODELS must have a MODELED BY .
ESTIMATE ... FROM <pop> [MODELED BY <metamodel> [USING MODELS <modelnos>]]

USING MODELS need not have a MODELED BY .
ESTIMATE ... FROM <pop> [MODELED BY <metamodel>] [USING MODELS <modelnos>]
If there is no MODELED BY <metamodel>

If there is just 1 metamodel in the population (the common case), use its modelnos .
If there are several metamodels for the population, try using modelnos for all: and crash if any do not have those modelnos.

Branch is tested and ready for merge at
https://github.com/probcomp/bayeslite/compare/20170624-fsaad-using-model
Outstanding items are:

Addressing the design choice from the previous comment;
Deciding where to report a BQLError for invalid model indexes (currently throws IndexError or similar).

Separate tickets:

Updating documentation to include modelnos where appropriate (separate ticket, since places like ANALYZE, DROP MODELS etc all accepts modelnos which are currently undocumented).

https://github.com/probcomp/bayeslite/commit/a6056a7cc2ea77c5b6d818d46ae3c574a9fd5a88


Initial workflow and MCLI test yaml (fork)

What does this PR do?
Test MCLI submission
What issue(s) does this change relate to?

Before submitting

[ ] Have you read the contributor guidelines?
[ ] Is this change a documentation change or typo fix? If so, skip the rest of this checklist.
[ ] Was this change discussed/approved in a GitHub issue first? It is much more likely to be merged if so.
[ ] Did you update any related docs and document your change?
[ ] Did you update any related tests and add any new tests related to your change? (see testing)
[ ] Did you run the tests locally to make sure they pass?
[ ] Did you run pre-commit on your change? (see the pre-commit section of prerequisites)

Since pull_request_target works off the base repo we'll need to work on this off of the composer repo directly.  Forks will only trigger once the initial workflow has been committed.


ORT-HF Benchmark Integration

-Add HF Benchmarker class.
-Add sample to benchmark HF model.
Example:
    python -m benchmarks.hf_model_benchmark --num_iterations=10 --model_name="microsoft/MiniLM-L12-H384-uncased"

shark/benchmarks

so is this current placement OK? or should I move it into another directory? haha

Yes current is fine. Thanks


Any planned release date for 4.1.0?

Hi,
I noticed that you have started some work on a 4.1.0 release a while back. Looks very interesting.
Do you have any plan for when to release a stable version of that? :)
Or if you would like to share what needs to be completed for it to qualify as a non preview in your point of view?
Thanks,
//Thomas

Yes, I'm looking at releasing it within the next couple of weeks.

Skipping over 4.1 and going to 5.0. Planning to release it sometime this week. Closed the last task.


GraphQLSchema "The specified key does not exist."

Describe the bug
amplify push fails with:

Following resources failed
Resource Name: qexdsgmbbrc4ddsothvfik7oauGraphQLSchema (AWS::AppSync::GraphQLSchema)
Event Type: update
Reason: The specified key does not exist. (Service: Amazon S3; Status Code: 404; Error Code: NoSuchKey; Request ID: 0F36192E4BFA9CC0; S3 Extended Request ID: 2Rf0PcdJSrHP9SFRQLhwKyDovNA76tncyiCAQVA5Z71GOMCXanaqCAUKj/dn1zcMzdLgFrfaQWo=)

Amplify CLI Version
4.21.4
To Reproduce

Modify lambda resource
amplify push

Expected behavior
Stack deploys successfully.
Desktop (please complete the following information):

Ubuntu 18.04.4 LTS
Node Version: v10.16.3

Additional context
I've created a GraphQL resource and some Lambdas. Gave Lambdas access to "GraphQL API" resource.
Modifying lambdas and pushing them worked perfectly until some changes in GraphQL Schema. (added some fields and connections), after which, pushing only lambda resources fails.
But if I modify GraphQL schema along with Lambda, everything works again.

there is a workaround, revert to CLI version 4.21.3 and then force push

there is a workaround, revert to CLI version 4.21.3 and then force push

this works. thank you.

Closing as duplicate of #4612.


C# Not Listed in Language Selection After Installing C# Extension

Environment data
dotnet --info output:
.NET Core SDK (reflecting any global.json):
Version:   2.1.401
Commit:    91b1c13032
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.17134
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\2.1.401\
Host (useful for support):
Version: 2.1.3
Commit:  124038c13e
.NET Core SDKs installed:
2.1.401 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
VS Code version:
Version: 1.27.2 (system setup)
Commit: f46c4c469d6e6d8c46f268d1553c5dc4b475840f
Date: 2018-09-12T16:17:45.060Z
Electron: 2.0.7
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64
C# Extension version:
1.16.1
Steps to reproduce
Install C# extension.

Expected  behavior
It should have C# in the "Select Language Mode" dropdown and should should select this automatically when a .cs file is selected.
Actual behavior
Syntax highlighting doesn't work and there isn't even a "C#" option in the "Select Language Mode" dropdown.

.NET Core SDK (reflecting any global.json):
Version:   2.1.402
Commit:    3599f217f4
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.17134
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\2.1.402\
Host (useful for support):
Version: 2.1.4
Commit:  85255dde3e
.NET Core SDKs installed:
2.0.2 [C:\Program Files\dotnet\sdk]
2.1.2 [C:\Program Files\dotnet\sdk]
2.1.4 [C:\Program Files\dotnet\sdk]
2.1.100-preview-007326 [C:\Program Files\dotnet\sdk]
2.1.100 [C:\Program Files\dotnet\sdk]
2.1.201 [C:\Program Files\dotnet\sdk]
2.1.202 [C:\Program Files\dotnet\sdk]
2.1.301 [C:\Program Files\dotnet\sdk]
2.1.400-preview-009063 [C:\Program Files\dotnet\sdk]
2.1.400-preview-009197 [C:\Program Files\dotnet\sdk]
2.1.401 [C:\Program Files\dotnet\sdk]
2.1.402 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.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 2.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.0.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download
VSCode:
Version: 1.27.2 (user setup)
Commit: f46c4c469d6e6d8c46f268d1553c5dc4b475840f
Date: 2018-09-12T16:17:45.060Z
Electron: 2.0.7
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64
C# Extension: 1.16.1
The Select Language Mode works fine for me.
@rchande Do you have any ideas here?

@blackwire It is strange. In the left pane on your editor I do see "C#" which indicates that somewhere the language version is being recognized. We have the 1.16.2 version of the extension out. Could you try that ?

@akshita31, it is most strange indeed. I updated to the latest version and still appear to be having the issue. I restarted the application as well and it did not seem to make any difference. I'm tempted to re-install my vscode, but... am worried about various things surrounding doing that. It takes a lot of time to re-setup things.

@blackwire Does this problem reproduce for you in the basic "dotnet new console" project as well ?

@blackwire I am really unsure about why that would happen. The extension folder lives inside the vscode directory so I don't think that should be the case.
I will go ahead and close this for now. Please re-open if you experience the issue again so we can dive deeper.


Add notification when trying to stop entry with unmet constraints

:star2: What does this PR do?
For now it only prints error from backend, which is descriptive enough, imho. We might actually add some actions pointing to ws settings, or something.
:bug: Recommendations for testing
⚠ All changes should be tested across Chrome and Firefox. ⚠

Set some constraints in ws settings
Try to stop Time Entry in button without ☝ fields filled in
There should be a notification

Try pomodoro with "stop time entry" option set too
:memo: Links to relevant issues or information
Closes #1430

Good to go? 🙏

Good to go? 🙏

I guess it should be 😂


Versioning/Releases and Tags

:wave: @jinankjain @JonPulsifer
Do we want to version kubeaudit?

[x] Create releases and tags in Github
[x] add a flag to the kubeaudit binary that reports the current version.

So that you can install specific version via e.g. glide and check which version is currently install with e.g. --version.

@klautcomputing How do we name the version like v1.0.0 or v0.0.1?

How do you feel about using https://github.com/goreleaser/goreleaser ?

https://github.com/Shopify/kubeaudit/pull/9 adds the flag to kubeaudit

https://github.com/Shopify/kubeaudit/pull/3 adds goreleaser

last(?) but not least: @jinankjain suggested to use: https://github.com/hashicorp/go-version

This can be closed now with #14 #9 #6 #7 @klautcomputing

Yes :)


feat(generate:buildVersion): Add support for RC versions

Adds support for the 2.0 release candidate version scheme. To make the implementation simpler, the RC version scheme is the same as the internal version scheme but uses "rc" as the prerelease identifier instead of "internal". For example, 2.0.0-rc.1.0.0 will be the first RC version.
Pre-release/dev builds of RC versions will be of the form 2.0.0-dev-rc.<IP_ADDRESS>456.
AB#6142

REQUIRED_PRERELEASE_IDENTIFIER.

stale

Refers to: build-tools/packages/version-tools/src/internalVersionScheme.ts:120 in 6c86ed2. [](commit_id = 6c86ed25244c0e2801d022d6e23c9c44b6a9dc1b, deletion_comment = False)

internalVersion: semver.SemVer | string,

docs and function name could use an update when possible.

Refers to: build-tools/packages/version-tools/src/internalVersionScheme.ts:61 in 6c86ed2. [](commit_id = 6c86ed25244c0e2801d022d6e23c9c44b6a9dc1b, deletion_comment = False)

Thanks for the review, @jason-ha!


Find the essential prime implicants

Description
This is gonna another function that will find the essential prime implicants from a list of the prime implicants.
Like the case of the function find_prime_implicants(), we will need to use other functions to success with the process we want to create.
Idea for the code
def find_essential_implicants (prime_implicants):
 return essential_implicants

Functions to complement
It was fundametal to create other functions to make the process easier like
#Function to remove the indicator at the beginning of the groupings of 1s
def eliminate_indicator(grouping_of_1s):
    return grouping_of_1s

#Function to remove empty lists from a list of lists.
def eliminate_empty(list_of_lists):
    non_empty_list = []
    return non_empty_list

#Function to compare two minterms.
def compare(min1, min2):
    return bool_value, new_min

#Function to check if the minterm has already been grouped before or not.
def grouped_min(min1, min2):
    return bool_value

Functions to complement
It was fundametal to create other functions to make the process easier like
#Function to find the minterms that make up the grouped minterms
def search_minterms(min):
    minterms = []
    return minterms


fix: specifying "main" in npm breaks webpack setups

is it reasonable to omit the main entry altogether?
when installing via npm and building with webpack, this breaks the webpack build as the ./assets/react/index.js will be picked up as a module and bundled, causing broken dependencies (if you don't want react in the project).
BTW, thanx! unsemantic is awesome!

Hmm, without specifying "main" then this wouldn't work…
import { Grid } from 'unsemantic'

Let me do some research and see if there's a different file name I can use, that might get Webpack to ignore "index.js"

So, as it turns out, I don't need {"main":"…"} in package.json, as long as I have an index.js file at the root. Related commit:
https://github.com/nathansmith/unsemantic/commit/15c1f94b438f76fcf446c27da0ed0f97883554d2
NOTE: I bumped the version number to 1.1.3.
Now, users will still be able to do…
import { Grid, GridContainer} from 'unsemantic'

…but also, Webpack won't erroneously pick up on any "main" file, because that doesn't exist in the package.json anymore.
Best of both worlds, win-win.
Thanks for bringing this to my attention.
:)

awesome, thanx for that fix. i wasn't around for a while, and just saw your comments..
cheers :)


src/core/src/pso.rs: remove assert

When running the Cube example in piston-examples, I noticed that resizing the window does not update the sizes of the buffers. I discovered that the pipeline data needs to be updated according to the window's dimensions, and it resulted in the following commit (not yet pushed to piston-examples), which has successfully solved the problem for me (along with the assert removal in this pull request):
https://github.com/da-x/piston-examples/commit/f88489145cf0d873ad2479285a5b0db9bee7ffbe
Is there a better way to fix window resizing in piston-examples, or can we have this assert removed?
@bvssvni , @sectopod , @Bastacyclop

If you updated both output surfaces, then the sizes should still match. Hence, the question - why were you still hitting the assert?

IIRC, the PixelTargetSet dimensions are always initialized to None before it is filled. So this assertion means that you have multiple render targets (probably the color and depth-stencil ones) of different size for the same draw call, did you update all those render targets after resize ?

You're both right. Wasn't realized they are compared against each other. I'll proceed with the piston-examples fix separately.

Thanks @da-x !

We could have probably returned an error instead of asserting there...


Lin 425 delete artifact bugfix

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Also please update PR description.


feat: support tilt since 0.33.15

Description
Minor update to remove the out dated suffix of _ALPHA for tilt when installing on ARM.
Motivation and Context

[ ] I have raised an issue to propose this change, which has been given a label of design/approved by a maintainer (required)

How Has This Been Tested?
make test

If updating or adding a new CLI to arkade get, run:
go build && ./hack/test-tool.sh tilt

Types of changes

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

Documentation

[x] I have updated the list of tools in README.md if (required) with ./arkade get --format markdown
[x] I have updated the list of apps in README.md if (required) with ./arkade install --help

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s

[x] I have tested this on arm, or have added code to prevent deployment

Output for ./hack/test-tool.sh
+ ./arkade get tilt --arch arm64 --os darwin --quiet
+ file /Users/brian/.arkade/bin/tilt
/Users/brian/.arkade/bin/tilt: Mach-O 64-bit executable x86_64
+ rm /Users/brian/.arkade/bin/tilt
+ echo

+ ./arkade get tilt --arch x86_64 --os darwin --quiet
+ file /Users/brian/.arkade/bin/tilt
/Users/brian/.arkade/bin/tilt: Mach-O 64-bit executable x86_64
+ rm /Users/brian/.arkade/bin/tilt
+ echo

+ ./arkade get tilt --arch x86_64 --os linux --quiet
+ file /Users/brian/.arkade/bin/tilt
/Users/brian/.arkade/bin/tilt: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, BuildID[sha1]=00faf54f3aa1a26f966d027921512ef8a6e71156, for GNU/Linux 3.2.0, stripped
+ rm /Users/brian/.arkade/bin/tilt
+ echo

+ ./arkade get tilt --arch aarch64 --os linux --quiet
+ file /Users/brian/.arkade/bin/tilt
/Users/brian/.arkade/bin/tilt: ELF 64-bit LSB executable, ARM aarch64, version 1 (SYSV), dynamically linked, interpreter /lib/ld-linux-aarch64.so.1, for GNU/Linux 3.7.0, Go BuildID=5-F9CBcRhjlrErxbHipA/208amWVCzRf9QYm9Xw5O/D_cs7mDqfYrHF-p2N_J5/9ZDXoxYzZd-JEmN6TM68, BuildID[sha1]=804dc4ac4489922e015ce1cf4dff61a0c18c8ec5, stripped
+ rm /Users/brian/.arkade/bin/tilt
+ echo

+ ./arkade get tilt --arch x86_64 --os mingw --quiet
+ file /Users/brian/.arkade/bin/tilt.exe
/Users/brian/.arkade/bin/tilt.exe: PE32+ executable (console) x86-64 (stripped to external PDB), for MS Windows
+ rm /Users/brian/.arkade/bin/tilt.exe
+ echo

What are the next steps for this PR?


Return iterator of matching routes instead of first match

For something like fall-through routing, it would be useful to return an iterator of values that match the route.
An example might be:
let mut router = Router::new();
router.insert("/users/:id", 1);
router.insert("/users/:name", 2);
router.insert("/users/*", 3);

let mut matches = router.at("/users/ari").iter();
assert_eq!(matches.next(), Some(1));
assert_eq!(matches.next(), Some(2));
assert_eq!(matches.next(), Some(3));
assert_eq!(matches.next(), None);

This would be a pretty fundamental change to the router. For fallback routing I was thinking of supporting a much simpler model, where conflicting inserts would return the conflicting value. From there you could handle routing guards, etc. yourself.

That sounds simpler. In my case, we forked matchit and modified it to our needs, so if it's not in the scope of matchit, feel free to close this issue


Syntax lookup: Fix routing

Hi @ryyppy
As discussed, this PR is a suggestion for fixing the Syntax Lookup routing.
Notes
[1] I've tried to stay as close to the current implementation as possible and minimise changes, with one exception ...
[2] MAIN CHANGE: In the current code, when the search changes we have an effect to update the URL. In this PR I've reversed that, so when the URL changes it updates the search. I've tried to comment the flow of the code, see what you think.
[3] For route changes I'm using Next.Router.push so we get some history. The replace behaviour has been removed (no longer needed due to point [2] above).
[4] I've removed the GithubSlugger.slug(item.id) code. If we stick to the current id pattern using hyphenated lowercase names then I assume it's not needed? But let me know what you think.
[5] I've added a scrollToTop() when the router triggers a Detail view. The only time this might feel a bit weird is when you have scrolled down the page a bit and manually type in the full name of an item.
[6] BEHAVIOUR CHANGE: One behaviour change compared to the current implementation. If you are on a ShowDetails view and then change the search box, the URL anchor is not cleared.
What do you think of this direction? Do you think that continuing with the current approach of syncing the route to the search (rather than syncing the search to the route) might be worth exploring more?
Testing

Navigating to /syntax-lookup shows the ShowAll view.
Full page reload of an anchored page such as /syntax-lookup#module-decorator shows the ShowDetails view.
Clicking a tag shows the ShowDetails view and sets the URL anchor
Clearing the search box via the "x" shows the ShowAll view and clears the URL anchor
Clearing the search box using the keyboard shows the ShowAll view and clears the URL anchor
Entering text in the search box not matching an item shows the ShowFiltered view, noting that the existing URL anchor remains unchanged.
Entering text in the search box exactly matching an item name shows the ShowDetails view.
ShowDetails view consistently scrolls to the top.
URL history of ShowDetails and ShowAll views are retained.

Okay this feels way nicer and fixes quite a few issues. Code changes make sense to me as well. What else is there to explore?

Thanks @ryyppy, should be fine to merge as-is. I've also just added links to the raw pages.

This is great!


Eventlog: Staking request data to BALN token SCORE

Add an event log to setStake transactions indicating the change in stake.
For example, if Bob currently has 1,000 BALN staked and changes it to 800 BALN staked, the event should return "-800" labeled changeInStake

Hmm, I think you mean to ask for an eventlog that returns the change in staked amount, which would be -200 in the case you describe. I think it would be better to have a more general eventlog since just reporting the delta leaves out data. I'd suggest SetStake(from, old_stake, new_stake)


[engine-1.21] Update wharfie usage in windows code path

Proposed Changes
Fix https://github.com/k3s-io/k3s/pull/4671 for windows. We don't test or build any of the Windows code that's present in k3s until it's pulled through into RKE2, so this was missed.
Types of Changes
bugfix
Verification
update k3s in rke2; notice build failure.
Linked Issues

https://github.com/k3s-io/k3s/issues/4663

User-Facing Change
Further Comments

CI failed due to running disk space issues; merging


Error in JSON parser when running larva scenario

When running larva scenario the larva tool crashes with the following error

This works with snapshot FF 7.8.1-20230420.190150 and now fails when trying to upgrade to FF 7.8.1
Reporter
Dirk-Jan / Ali
To Reproduce
For more info about the larva test and the adapter handling this test you can contact me.

Larva is expecting json because the file name ends with .json but the adapter is returning xml, hence it is not expecting char 6 (<). So it's not strange that Larva cannot parse the "json" but a more user friendly error message would be nice

This is already fixed in september 2022 and works in 7.9.0 and higher already. See #3736


Limit Database Access Within Workflows, Database Authentication.

Currently, the database can be accessed freely within workflows via the VirtoolDatabase fixture and the fixtures created for each database collection. This may become problematic in future when developers outside of the organization begin contributing workflows.
Database access is required to setup the fixtures needed for the analysis workflows. Additionally it is required to send results of analysis workflows for the front-end to display.
We require a solution to securely allow access to the database by the runtime while blocking access within workflows and fixtures, execept with specific exceptions.

Sandboxing
The fundumental problem here is that the runtime process must imports a
module (workflow.py) which is written by a user. As such there is
nothing preventing the workflow.py module from accessing values from
the global namespace, environment variables, and other runtime specific items.
Adding authentication to the database via SCRAM or some other mechanism would make it a bit more difficult to access the database from within workflows. However there is nothing preventing a workflow developer from gaining access to the auth token which would need to be accessable within the runtime code, and as such could be discovered within the workflow code.
Ultimatley we need to sandbox the workflow code in order to
prevent it from accessing the runtime code which provides access to the
database (or credentials for accessing it).
There are several ways to do this, each providing different levels of security. Here are some of those methods.
Use eval() and override import
The eval function allows you to set the locals and globals in the context of the code being evaluated. This would limit a workflow's access to the global namespace.
__import__ could be overridden to prevent the workflow module from importing packages from the runtime. If they can't import the runtime modules, and don't have access to the full global scope of the runtime, it would be somewhat difficult to access the database.
The workflow.py module still has access to the same filesystem as the runtime, as well as it's enviroment variables.However, it does make it more difficult to access the database and other runtime specific constructs. It won't deter a determined attacker, but anyone attempting to access the database would clearly know that they're not supposed to be doing it.
Full Containerization
The most secure way to sandbox the workflow module is to execute it in it's own container. Any database lookups required would be executed within the runtime process prior to running the workflow container and the required data would be provided to the container via arguments or a mounted volume. The container could produce a file containing the output results dictionary, which could then be taken in by the runtime process and sent to the database.
The problem with this approach is that it would require docker-in-docker or something similar, in the context of the planned K8S deployment.
Run Workflows in a Separate Process Without Root Priveleges
Similarly to full containerization, Any database lookups would be performed in the runtime process and the data passed along to the workflow process.
The MongoDB connection string, Redis connection string, and other credentials could be stored in a file which requires root privelleges to open. The runtime process will have access to the file and thus full access to the database. The workflow process will be able to import modules from virtool_workflow_runtime but will not be able to access the database without the proper credentials, which will be inaccessable to it since it lacks root privelleges.

Closing as #83 details the chosen solution.


OSMTilePackager tests are appearing to hang forever

Issue Type
[ ] Build system related
Description and/or steps/code to reproduce the problem
gradlew clean build
Environment
win10
Version of osmdroid the issue relates to:
head
not sure what's going on, but the test is hanging forever. Need to alter the gradle scripts to output std out to std out and figure out what the issue is. I'm guessing the tool was banned from openstreetmaps. Quick fix is to disable the tests

issues

test uses http, instead of https
server is returning 429, too many requests, so it's probably banned.

i'll commit the changes but leave the tests disabled


Protect trailing = in signature

Some signatures can end in an equal sign (ex: QE5GoUkrrI1SC/XmORhLGEeGe/c= )
This fix stops the param splitter from over-splitting, instead splitting only on the first equal sign and leaving subsequent equal signs as part of the value in the KV pair.

Thanks for the patch!

FYI There was a conflict between this and the next pull request.
I believe I've resolved the conflict in a way that still works for keys with "=" in them, and I have committed a test to demonstrate this in c1e13f5cdfc1e742c233498d78cc2edc182407f2
Please let me know if I've overlooked anything!

I don't think so, looks good to me!


Static library build

This Makefile builds libUpsurge.a and Upsurge.swiftmodule. These two files can be added to any Swift project that requires Upsurge. Once the two files are added to an Xcode project, import Upsurge will use the module map to access Upsurge objects in the static library.
This is a temporary workaround for the problem that Xcode 7.1 does not build static libraries or usable frameworks that include swift code.
Note:

To reference the module map and static lib, in the end project, Build Settings ->  Swift Compiler - Search Paths -> Import Paths needs to have $(SOURCE_ROOT)/path/to/files set. i.e. if added under a Lib group, then set $(SOURCE_ROOT)/Lib

The Upsurge code in the static library references the Accelerate framework but does not include it. The end project will require import Accelerate to ensure object file references link correctly.

The Complex.swift object is not included at present. swiftc has an odd problem where it fails to build code that uses hypot or atan2

Merged with https://github.com/aleph7/Upsurge/commit/eb568e97ecba0f19d07b2cf96a5ecc932b6505c4
thanks!


Question about the aggregated binary mask

Great work and thanks for sharing codes.
I have a question about the aggregated binary mask, specifically according to the paper, the sum will be set to one if the resulting value is 0.

But in the codes, i didn't find the corresponding impletation. https://github.com/saic-vul/imvoxelnet/blob/3512e89ca98e48aebb21a4c9e9fbe5037220b3a4/mmdet3d/models/detectors/imvoxelnet.py#L71
here the summed masks are directly used for the average calculation. Could you please give me some insights?
Thanks.

Hi @jianingwangind,
I think in paper we write 1 here to avoid dividing 0 by 0. And in code we set not valid voxel values to zero. If i'm not missing something, it should be the same.

Thanks for the reply.
Assuming that i understand correctly, if you don't set 1 for 0 summing values in advance, the division already return nan before you set invalid voxel values to zeros, which will interrupt the training.

We are getting nan and in the next line replace it with 0, so the overall voxel volume is fine.

I see. Thanks for your help.


Add your solution!

:jack_o_lantern: Hacktoberfest 2020 FizzBuzz

This repository is designed for everyone interested in coding. All skill-levels welcome!
Go ahead and experiment a bit with a language of a choice.
We're searching for creative, weird, new, interesting ways to solve FizzBuzz!
If you don't know what that is, here's a tl;dr:

FizzBuzz is a program that prints the numbers from 1 to 100. But for multiples of three it prints “Fizz” instead of the number and for the multiples of five, it prints “Buzz”. For numbers which are multiples of both three and five it prints “FizzBuzz”.

Read more about it here: https://www.tomdalling.com/blog/software-design/fizzbuzz-in-too-much-detail/

Information on how to get started can be found in the readme.
Happy hacking!

can i work on this???

Let me in !!

Please assign me this.

Request

Nice! I submitted a swift version of fizz buzz, could you check it out?

Yes

Amazing platform for coding

Can you pls assign to me, I'll do it with well written code.

I would like to work on this using python

i would like to work on this

interesting lets do that

Please assign me this. I would like to write it in Python

Please assign me this... I would like to write it in C++

I would like to work on this using C++.

I would like to work on this using C,C++,Python.

Hi @NLDev  , I have made a PR of what I believe to be the simplest python solution to fizzbuzz as a part of Hacktoberfest 2020. Please review.

I have added a one-liner using python lambda and map functions.
Please review.

Please check my PR. It was the 1st single line solution

making a function of this on python

Hello, I would like to work on this using kotlin/scala. Assign me please :).

Sir I have made a pull request after adding a folder for Cpp

I would like to work in Python

Just for clarification:
I wont assign this issue to anyone because everyone is welcome to contribute!
Cheers

can I please be assigned to this?

Can i please be assigned to this? I would like to work in Java!

Can i please be assigned to this? I would like to work in Java!

No. See here.

I will try to do it in Ruby

Added fizzbuzz with go routines #171

I would like to add my own solution in Ruby

I would like to work on this using Ruby one-liner

Can I help?
and with what language can I do it?

Can I help?
and with what language can I do it?

@JossJoestar Any, as stated in the readme :)

let me in, I give it a try.

can i work on this project?

I can do this. Please let me in

@NLDev, I guess it will be good if this issue is /pinned

I would like to contribute

I would like to contribute, please let me in

I have solved the Problem, Please check it out #274

I would like to work on this using python

I'd love to work on this repo with javascript :100:

I would love to work on this with python😃

I would like to work on this using python.

@NLDev, are you in all seriousness arguing that this repository is in the spirit of Hacktoberfest? Wow. IMO, you are clearly contributing to hurting open source by asking for random contributions that don't help anyone.

@keikoro The link you provided argues that Hacktoberfest itself is hurting the OSS Community.
Either way, yes. the repository is in alignment with Hacktoberfest's values. Please read the participation rules as well as hacktoberfest's values before accusing me of such things. The hacktoberfest welcomes beginners as well as intermediate programmers and professionals. In contrast to other repositories where people only commit their name to a list or so (most of those repositories have already been flagged by the way) this repository here is is for people of any skillset to challenge themself. I put a lot of effort into reviewing every single PR on its own and label obvious +1 attempts, as well as duplicates and off-topic PR's as invalid. You can check out the list of invalid PR's here: https://github.com/NLDev/Hacktoberfest-2020-FizzBuzz/pulls?q=is%3Apr+is%3Aclosed+label%3Ainvalid
I'd also recommend consulting the ReadMe of the repository before making such claims.
Cheers

While your project may not go against Hacktoberfest's "values", it's clearly not an existing OS project but designed solely to accept Hacktoberfest-related PRs. The repo name alone says as much, no need to read the README.
So my point stands: your project is not in the spirit of Hacktoberfest, which has always been about making open source better. It's not about getting people to submit their solutions for exercises, that's what Computer Sciences courses are for, it's about giving back to OS.

But since you linked to the official website: I wouldn't agree this project meets the fourth and last point in the "Quality Standards" section.

Something that's clearly an attempt to simply +1 your pull request count for October.

Individual PRs might meet that requirement, and PRs are, in fact, what the section focuses on, but I'd argue that's a limited reading of the rules (or, if you will, a failing on Hacktoberfest's part to not spell this out more clearly for people who nitpick over words rather than overall meaning).

While your project may not go against Hacktoberfest's "values", it's clearly not an existing OS project but designed solely to accept Hacktoberfest-related PRs. The repo name alone says as much, no need to read the README.

This might have been the "spirit" of Haktoberfest. However the website clearly states:
Hacktoberfest is open to everyone in our global community. Whether you’re a seasoned contributor or looking for projects to contribute to for the first time, you’re welcome to participate.
Don't get me wrong. I know what you mean. Yes, the hacktoberfest is meant to support existing open source projects that need help. However, anyone is welcome to participate and this repository isn't hurting anyone. It's not like everyone would just contribute to "serious" open source project if this repository here wouldn't exist.

It's not about getting people to submit their solutions for exercises, that's what Computer Sciences courses are for, it's about giving back to OS.

Again, the website states the it's mean for learning as well.

But since you linked to the official website: I wouldn't agree this project meets the fourth and last point in the "Quality Standards" section. "Something that's clearly an attempt to simply +1 your pull request count for October."

As I said, I manually review every single PR and attempts to effortlessly +1 will be labeled as invalid. There have been a lot of creative solutions so far. For instance, someone wrote the first and only FizzBuzz implementation in the esoteric language OMGROFL with instructions on how to build it.
Sure, this doesn't do much for the poor OSS community. Still no need in taking the fun out of it.

Individual PRs might meet that requirement, and PRs are, in fact, what the section focuses on, but I'd argue that's a limited reading of the rules (or, if you will, a failing on Hacktoberfest's part to not spell this out more clearly for people who nitpick over words rather than overall meaning).

Thankfully, this isn't for you to decide.
Cheers.

I'm on @NLDev side here

Good on ya, I guess...

I will try@NLDev  it's opportunity to prove myself

please assign this to me i will use python 3 to solve the aforesaid problem

i will use python 3 in this

@NLDev I've added my solution using Segment Tree ( #456 ).

Thank you for participating!


Don't use unzip -u modifier; fix for #93

Thank you for the fixes and the test.

@burtlo can you release the new version of ark cookbook? Thanks!


How to write this package as a Flask extension and not merely a Python package?

Hey, you might want to check out http://flask.pocoo.org/docs/0.10/extensiondev/

Extensions are all located in a package called flask_something where “something” is the name of the library you want to bridge. So for example if you plan to add support for a library named simplexml to Flask, you would name your extension’s package flask_simplexml.

The name of the actual extension (the human readable name) however would be something like “Flask-SimpleXML”. Make sure to include the name “Flask” somewhere in that name and that you check the capitalization. This is how users can then register dependencies to your extension in their setup.py files.

Flask sets up a redirect package called flask.ext where users should import the extensions from. If you for instance have a package called flask_something users would import it as flask.ext.something. This is done to transition from the old namespace packages. See Extension Import Transition for more details.

So basically you'd need to rename the project, afaik.

Many many thanks, @spikergit1 !
Just did and worked perfectly :)


Capture closures around megamorphic dispatches

The performLayout and build callsite are highly megamorphic because they
dispatch into a large number of clients. However, for a given caller, the
callee is always of the same type, which means the megamorphic lookup exactly
factors by the caller. We can speed up the dispatch by capturing a closure at
initialization and then monomorphically dispatching through the closure.

@Hixie

the travis madness is because _builder is asserted to null. should we checked against the _buildNothing call now.


Login form should never display on live

Whitelist preview domains should be positively identified before displaying the login form. Currently, new or unknown production domains might accidentally display the login form because preview is assumed to be the inverse of production (live).

This should provide a fix once it's resolved: https://github.com/abcnews/env-utils/issues/2


Indeed scraper - Scraping not scraping latest jobs.

I tried to scrap all the latest jobs from indeed using specific country and state but seems like it's not scraping latest job. I am sure there are latest jobs reason for that is, I have built an scraper using selenium and it does get the job done but since that is bit slow. I also tried Premium Static Residential proxies, here is the code:
import csv
from jobspy import scrape_jobs

jobs = scrape_jobs(
    site_name=["indeed"],
    location="Dubai",
    results_wanted=50,
    hours_old=24,
    linkedin_fetch_description=True,
    country_indeed='United Arab Emirates'
)

print(f"Found {len(jobs)} jobs")
print(jobs.head())
jobs.to_csv("jobs.csv", quoting=csv.QUOTE_NONNUMERIC, escapechar="\\", index=False) # to_xlsx

Am I missing something? I tried Linked-in one, it went good but it crashes due to error 429 after scraping couple of pages.

Stay tuned for release later tonight, will fix this

pip install -U python-jobspy
should be good

Hi, @cullenwatson
Seems like issue still persists, if I try to scrap for past 24 hours it will scrap 1 job. If I do 48 hours, it scrapes 2 jobs and for 72 hours it scrapes 3 jobs. Seems like it is scraping only first job of each page or something similar.
Output for past 72 hours:
2024-03-11 10:11:51,320 - JobSpy - INFO - Indeed search page: 1
2024-03-11 10:11:51,679 - JobSpy - INFO - Indeed search page: 2
2024-03-11 10:11:52,010 - JobSpy - INFO - Indeed found no jobs on page: 2
2024-03-11 10:11:52,010 - JobSpy - INFO - Indeed finished scraping
Found 3 jobs
     site  ...                                      ceo_photo_url
0  indeed  ...                                                NaN
1  indeed  ...                                                NaN
2  indeed  ...  https://d2q79iu7y748jz.cloudfront.net/s/_ceoph...

[3 rows x 26 columns]

Process finished with exit code 0

I have latest version of the package (1.1.48).

fixed. try version 1.1.49. I'm getting over 1k jobs now for 72 hours in Dubai

Thanks, @cullenwatson ! It's working now and it's soo fast, really appreciate the work you have done!

for some reason the backend api doesnt give salary as often as the direct method did


Coordinator timeout based on number of devices

Coordinator timeout based on number of inverters, 5 second per inverter.

Need to consider number of attached devices too (meters, batteries)

Reduce some timing in PR #350

Targeting this update for release v2.3.5

Now targeting v2.4.0

Timeouts are based on a worst case 1 second read response plus a 200 ms buffer.


Separate the Chat logic from the system logic

Is there a way to create or separate the chat logic from the system/administrative logic? Its good to be able to add a feature that allows server owners to disable/enable features or settings or administrator privileges (at build or at runtime), such as:

Chat Only
Disable Model View (disable dropdown selection of models)
Have a default model
Hide download
Enable default model
Disable Settings View (or at least, show only parts of system settings). This is beneficial to protect accidental deletion of models.
Enable search view (in stead of chat interface, bring the text area to the center of the screen (just like google search).

If contributors are busy, I'd like to be able to know the actual files that deals with this view, so I can play around with it.
Overall, I'm happy with the result of the latest release. Kudos to the contributors.
Thanks.

Hi, Thanks for the suggestion. RBAC is a feature that's being developed, stay tuned. Thanks. Duplicate of #68

Hi, Thanks for the suggestion. RBAC is a feature that's being developed, stay tuned.
As for the all other numerous features you're requesting, I do not plan on adding them (non-core) in the near future, and will be only focusing on adding core features, so you would have to fork the repo and add features to the frontend and the backend yourself. Please refer to the code in the /src/lib and /src/routes for the frontend, /backend directory for the backend. Thanks. Duplicate of #68

Would you welcome PRs that add non-core features, or would you rather someone fork the project if they want to add non-core features?


Spiral Navigator  does not return WorkerResult.RUNNING to the tick...?


entering text and 2 query params into a single field

Is entering multiple values supported?
For example:
Query::something some text here Query::another
Or using Append or Prepend together with Query?

You can have two actions and perform append query on same.

But what's the syntax of appending a query?
for https://test.getautoclicker.com/?aaa=bbb I tried Append::Query::aaa but instead of placing "bbb" it places "Query::aaa"

Its a bug we will address this


Implementing a few switch statements

I really like how patterns can be used for variable assignment and avoiding duplicated logic. (related: #150942)
// before
final GestureRecognizer? recognizer = info.recognizer;
if (recognizer is TapGestureRecognizer) {
  if (recognizer.onTap != null) {
    configuration.onTap = recognizer.onTap;
    configuration.isLink = true;
  }
} else if (recognizer is DoubleTapGestureRecognizer) {
  if (recognizer.onDoubleTap != null) {
    configuration.onTap = recognizer.onDoubleTap;
    configuration.isLink = true;
  }
}

// after
switch (info.recognizer) {
  case TapGestureRecognizer(:final VoidCallback? onTap):
  case DoubleTapGestureRecognizer(onDoubleTap: final VoidCallback? onTap):
    if (onTap != null) {
      configuration.onTap = onTap;
      configuration.isLink = true;
    }
}

It sort of reminds me of how Python has quirky inbuilt tips and tricks that serve to reduce verbosity but make it much harder to read and non-obvious to a newcomer.

doesn't help that : and final have the same color highlight

Yes, I can identify with that. Same thing goes with enum values: they have a distinct color in VS Code, but on GitHub they look just like other static class members.

Thank you Victor!
I think we're good to merge, since test updates are included in this PR.
(My plan was to ping @test-exemption-reviewer anyway, but it looks like they're pretty backed up as of now.)


Select with number values?

Is it possible to have a select field with number values? Suppose I have something like this:
export const Foo = t.struct({
  userId: t.Num,
});

I provide an option to the form like this:
userId: {
  factory: t.form.Select,
  options: _.map(collection, (x) => { return { value: x.id, text: x.name } }),
}

Everything works fine until validation, at which point I get the following (validationResult.firstError()): "Invalid value "3" supplied to /userId: Number"
This was surprising to me because I thought I'd read somewhere that tcomb will automatically cast numbers to Number?
EDIT: Oops, I just realized that it probably is the case that tcomb converts js numbers to tcomb's Num, but in my case, the value is a string. So really, I am not surprised :)
I tried using a transformer approach whereby I parse the value into a string. This works in my example, but not in another case where I use t.maybe(t.Num). It seems that null cannot reduce to a Number (ok, makes sense), but that's what the maybe is for?

Hi @mohanzhang,

Is it possible to have a select field with number values?

Yes but it's a little verbose:
export const Foo = t.struct({
  userId: t.Num
});

userId: {
  factory: t.form.Select,
  transformer: { // handles numbers and Nil
    format: (value) => t.Nil.is(value) ? '' : String(value),
    parse: (value) => value === '' ? null : parseInt(value, 10)
  },
  options: _.map(collection, (x) => ({ value: x.id, text: x.name }))
}

Another option:
const collection = [
  {id: 1, name: 'name 1'},
  {id: 2, name: 'name 2'}
];

const UserID = t.enums(_.transform(collection, (result, x) => {
  result[x.id] = x.name;
  return result;
}, {}), 'UserID');

const Foo = t.struct({
  userId: UserID
});

const App = React.createClass({

  onSubmit(evt) {
    evt.preventDefault();
    var value = this.refs.form.getValue();
    if (value) {
      // here convert userId to a number...
    }
  },

  render() {
    return (
      <form onSubmit={this.onSubmit}>
        <t.form.Form
          ref="form"
          type={Foo}
        />
        <button className="btn btn-primary">Save</button>
      </form>
    );
  }

});

p.s.

tcomb converts js numbers to tcomb's Num

Actually tcomb doesn't convert anything: tcomb types are just identity functions containing an assert to ensure the correct type:
var n1 = t.Num(2); // => n1 = 2
var n2 = t.Num('a'); // => throws

The only exceptions are the structs which don't merely return the input (but an instance of a "class")
var Foo = t.struct({
  userId: t.Num
});

var foo = Foo({userId: 1}); // => foo instanceof Foo === true

Ah fantastic! Thank you so much @gcanti! I've noticed searching throughout the issues that your answers are always very thorough. And of course thank you for bringing some semblance of type safety to the js world ;)


Python pip - could not find the version 9.1.9490

What version of OR-Tools and what language are you using?
Version: 9.1.9490.
Language: Python
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
Routing Solver
What operating system (Linux, Windows, ...) and version?
Ubuntu 20.04
What did you do?
Steps to reproduce the behavior:

Install ortools version 9.1.9490 using python3 -m pip install ortools==9.1.9490
See error

What did you expect to see
Installation should have been successful as it was working well till last week.
What did you see instead?
Installation was unsuccessful by throwing the below error:
Step #0: Step 7/21 : RUN python3 -m pip install ortools==9.1.9490
Step #0:  ---> Running in 59c7a70e82b3
Step #0: ERROR: Could not find a version that satisfies the requirement ortools==9.1.9490 (from versions: 7.8.7959, 8.0.8283, 8.2.8710, 9.0.9048, 9.2.9972, 9.3.10497, 9.4.1874, 9.5.2237, 9.6.2534, 9.7.2996)
Step #0: ERROR: No matching distribution found for ortools==9.1.9490
Step #0: The command '/bin/sh -c python3 -m pip install ortools==9.1.9490' returned a non-zero code: 1

Make sure you include information that can help us debug (full error message, model Proto).
Anything else we should know about your project / environment
I would like to know if this version has been removed from the library.
Thank you,
Kishor

We had to delete versions. We were hitting the max size of pypi

Note that the whl files should be on the GitHub release artifacts


Potential regressions with collapsed panels

For context, I tried updating from 0.0.53 to 0.0.61 and noticed potential regressions.
Using 0.0.53, I have a panel that's collapsed by default (defaultSize={0} + collapsed), and using ImperativePanelHandle I'm able to expand/collapse it, and it'll remember the last committed values. Note that I'm also using resize(40) if it's the first expansion.
https://github.com/bvaughn/react-resizable-panels/assets/2852660/2a1bfe4c-b492-4bb5-bdbf-fdd256f332a9
Following https://github.com/bvaughn/react-resizable-panels/blob/main/packages/react-resizable-panels/CHANGELOG.md#example-migrating-panels-with-percentage-units, I updated defaultSize to defaultSizePercentage, getCollapsed to isCollapsed, and onCollapse to onExpand, however using 0.0.61, it seems like it forgets its previous size before it was collapsed?
https://github.com/bvaughn/react-resizable-panels/assets/2852660/deed8e30-67c7-4a80-a140-b91b3c810d53
I then did a bit of looking through the code, and I believe in 0.0.53 minSize was defaulted to 10, but the default has since been removed. So I added minSizePercentage={10}, but it still doesn't behave like it used to - that is, it expands to minSizePercentage instead of the last size (40%)
https://github.com/bvaughn/react-resizable-panels/assets/2852660/3c2ad8cc-f4f9-4732-94bd-e6f9f75ca5c1

If you share a Replay or as least a Code Sandbox of the bug, I’ll take a look.

I did share 2 Code Sandboxes, they're hidden behind hyperlinks for 0.0.53 and 0.0.61 respectively

My apologies @NMinhNguyen. I didn't realize those were Code Sandbox links. I thought they were links to my own release notes or something.

No problem and thanks for looking into this. I'll try to record a replay in the coming days

I updated the 0.61 Sandbox to use 0.63 and recorded a replay: https://app.replay.io/recording/react-app--383118c2-aa91-4623-9bc7-d2c3d9f184e6

Thanks, @NMinhNguyen. That will be very helpful when I get time to look at this.

Thank you for sharing the Replay.
Looking at the Replay, I believe it shows things working as expected. Internally, I keep track of a Panel's size before collapse using a ref, panelSizeBeforeCollapseRef. That ref is only written do when the Panel.collapse() API is called and it's only read from when the Pane.expand() API is called. In the Replay you shared, you "collapse" the panel by drag-resizing it, so the ref is never written do. So when you re-expand it, it just re-expands to the min size.
It's not clear to me what (other than this) you'd expect to happen in this case?

Going to close this issue since I don't plan to take action on it. Happy to continue talking here though.

Personally, I think the 0.53 behaviour is more intuitive - I don't think it should matter whether collapse() API was explicitly called or whether the panel was collapsed as a result of a drag interaction. The new behaviour may appear to the user as if the last expanded size was forgotten. And the minSize merely determines the threshold where collapsing happens - I don't think the user would expect clicking on an expand button to expand to that.
Alternatively, if you have suggestions as to how I can bring back the old behaviour in user land, I'm happy to use that approach instead.

You can see the VS Code behaviour here, although my theme makes it hard to see the resize handle:
https://github.com/bvaughn/react-resizable-panels/assets/2852660/32452b4b-0180-47cf-bd5c-e4fb2a0d6828

Looks like VS Code remembers the most recent dragged size too, on some kind of debounce/throttle maybe so that it doesn't always end up being min size (the last non-collapsed size). If this matters to you, feel free to open a PR that adds this behavior (after #230 lands). I don't think I will add it myself.


Enable building and hosting the documentation on readthedocs

The result of building this PR can be seen here:
https://mbed-tls.readthedocs.io/en/readthedocs/

@daverodgman I can see the newest advisory - here and here.
Maybe your browser is caching the webpages?


[KYUUBI #5367] Spark crashes with ClassCastException when resolving a join of masked tables

:mag: Description
Issue References 🔗

This pull request fixes #5092
Spark crashes with ClassCastException when resolving a join of masked tables
Describe Your Solution 🔧
As per @sauliusvl comment in this unfinished PR Stage0Marker column ids are not updated on the right side of the join which leads to Stage1Marker being incorrectly applied which in turn causes
java.lang.ClassCastException: org.apache.kyuubi.plugin.spark.authz.ranger.datamasking.DataMaskingStage1Marker cannot be cast to org.apache.spark.sql.catalyst.plans.logical.Join

So the solution was to copy the scan plan together with the child plan in withNewChildInternal method so that column ids are aligned.
Types of changes :bookmark:

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

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Self join of tables containing masked columns using joinExprs arg fails with ClassCastException.
Behavior With This Pull Request :tada:
Self join of tables containing masked columns using joinExprs arg succeeds.
Related Unit Tests
DataMaskingTestBase

Checklist 📝

[x] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (47a1091) 61.16% compared to head (1fdb496) 61.10%.

:exclamation: Current head 1fdb496 differs from pull request most recent head 8644837. Consider uploading reports for the commit 8644837 to get more accurate results

Files
Patch %
Lines

...thz/rule/datamasking/DataMaskingStage0Marker.scala
50.00%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6012      +/-   ##
============================================
- Coverage     61.16%   61.10%   -0.06%
  Complexity       23       23
============================================
  Files           623      623
  Lines         37060    37063       +3
  Branches       5024     5025       +1
============================================
- Hits          22666    22647      -19
- Misses        11958    11976      +18
- Partials       2436     2440       +4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

seems the test failed in Spark 3.1 and 3.5


lucenepp: install header files of lucene-contrib

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

According to result/lib/pkgconfig/liblucene++-contrib.pc
includedir=${prefix}/include/lucene++
lib=lucene++-contrib

header files of lucene-contrib should move to $out/include/lucene++

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/501