Intel: Panic when acquirng swapchain image

Short info header:

GFX version: 0.4.0
OS: Ubuntu Linux
GPU:VGA compatible controller: Intel Corporation HD Graphics 515 (rev 07)

Also tested on another machine, Linux with NVIDIA Geforce RTX 2060. There the same code works flawlessly, i can open 50 windows without any problems.
I have a program with 2 threads. One thread is running the event loop with winit, the other is handling graphics and interfacing wgpu (etc.). They are connected using a crossbeam.
When clicking a button the gpu thread gets the mouse event over crossbeam, reports a user event back to the winit thread, which creates a window & a corresponding surface. The surface is passed to the graphics thread to set up the rendering.
It seems to me like the swapchain tries to render in the old window while the new one is created. This causes the intel driver to return a strange texture index, and not an error.  Thus rust panics. This happens every time.
I have made a workaround (attached) that sanitises the texture indices, and this seems to work. (i could create 10-20 windows without panic now)
Possible workaround:
diff --git a/src/backend/vulkan/src/window.rs b/src/backend/vulkan/src/window.rs index 6e852e51..420d8a15 100644
--- a/src/backend/vulkan/src/window.rs +++ b/src/backend/vulkan/src/window.rs
@@ -501,6 +501,11 @@ impl w::PresentationSurface<Backend> for Surface {          timeout_ns = timeout_ns.saturating_sub(moment.elapsed().as_nanos() as u64);
        let fences = &[ssc.fence.0];
+        if(!(index < ssc.frames.len() as u32)) +        {
+           return Err(w::AcquireError::OutOfDate); +        }
+          match ssc.device.0.wait_for_fences(fences, true, timeout_ns) {
            Ok(()) => {                 ssc.device.0.reset_fences(fences).unwrap();
`
Panic:
thread '<unnamed>' panicked at 'index out of bounds: the len is 3 but the index is 32767', /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libcore/slice/mod.rs:2806:10 note: run with RUST_BACKTRACE=1 environment variable to display a backtrace.
[Switching to Thread 0x7ffff79ef700 (LWP 6198)]
Thread 2 "customcode" hit Breakpoint 1, rust_panic () at src/libstd/panicking.rs:521 521     src/libstd/panicking.rs: No such file or directory.
(gdb) bt #0  rust_panic () at src/libstd/panicking.rs:521
#1  0x0000555555db15a6 in std::panicking::rust_panic_with_hook () at src/libstd/panicking.rs:491 #2  0x0000555555db104e in rust_begin_unwind () at src/libstd/panicking.rs:375
#3  0x0000555555dcc2de in core::panicking::panic_fmt () at src/libcore/panicking.rs:84 #4  0x0000555555dcc2a5 in core::panicking::panic_bounds_check () at src/libcore/panicking.rs:62
#5  0x0000555555a17937 in <usize as core::slice::SliceIndex<[T]>>::index (self=32767, slice=...) at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libcore/slice/mod.rs:2806 #6  0x0000555555a19628 in core::slice::<impl core::ops::index::Index for [T]>::index (self=..., index=32767)
   at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libcore/slice/mod.rs:2657#7  0x00005555559e57fb in <alloc::vec::Vec as core::ops::index::Index>::index (self=0x55555629b6b0, index=32767)
   at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/liballoc/vec.rs:1871#8  0x00005555559b8716 in <gfx_backend_vulkan::window::Surface as gfx_hal::window::PresentationSurface<gfx_backend_vulkan::Backend>>::acquire_image (self=0x55555629b638,
   timeout_ns=999910700) at /home/freqmod/Downloads/gfx/src/backend/vulkan/src/window.rs:512#9  0x0000555555833fec in wgpu_core::swap_chain::<impl wgpu_core::hub::Global>::swap_chain_get_next_texture (self=0x5555560ff860, swap_chain_id=..., view_id_in=...)
   at /home/freqmod/Downloads/wgpu/wgpu-core/src/swap_chain.rs:148#10 0x00005555558995a2 in wgpu_swap_chain_get_next_texture (swap_chain_id=...) at /home/freqmod/Downloads/wgpu/wgpu-native/src/device.rs:410
#11 0x00005555556df5d7 in wgpu::SwapChain::get_next_texture (self=0x7ffff012aec0) at /home/freqmod/Downloads/wgpu-rs/src/lib.rs:1480 #12 0x0000555555629eef in customcode::RenderingPipeline::redraw (self=0x7ffff79ee960, win_idx=...) at src/main.rs:454
#13 0x000055555562ada2 in customcode::RenderingPipeline::handle_event (self=0x7ffff79ee960, event=...) at src/main.rs:553 #14 0x000055555562b67e in customcode::RenderingPipeline::listen_for_events (self=..., event_receiver=...) at src/main.rs:612
#15 0x000055555562da9d in customcode::main::{{closure}} () at src/main.rs:633 #16 0x000055555562366d in std::sys_common::backtrace::__rust_begin_short_backtrace (f=...) at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libstd/sys_common/backtrace.rs:136
#17 0x000055555565ad5d in std::thread::Builder::spawn_unchecked::{{closure}}::{{closure}} () at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libstd/thread/mod.rs:469 #18 0x00005555556575cb in <std::panic::AssertUnwindSafe as core::ops::function::FnOnce<()>>::call_once (self=..., _args=())
   at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libstd/panic.rs:318#19 0x000055555568a78e in std::panicking::try::do_call (data=0x7ffff79eeb60 "\000") at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libstd/panicking.rs:292
#20 0x0000555555db4e3a in __rust_maybe_catch_panic () at src/libpanic_unwind/lib.rs:78 --Type  for more, q to quit, c to continue without paging--
#21 0x0000555555689daa in std::panicking::try (f=...) at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libstd/panicking.rs:270 #22 0x000055555565766d in std::panic::catch_unwind (f=...) at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libstd/panic.rs:394
#23 0x000055555565ab5a in std::thread::Builder::spawn_unchecked::{{closure}} () at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libstd/thread/mod.rs:468 #24 0x000055555566fec3 in core::ops::function::FnOnce::call_once{{vtable-shim}} () at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libcore/ops/function.rs:232
#25 0x0000555555da5b9f in <alloc::boxed::Box<F> as core::ops::function::FnOnce<A>>::call_once () at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/liballoc/boxed.rs:1022 #26 0x0000555555db4270 in <alloc::boxed::Box as core::ops::function::FnOnce>::call_once () at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/liballoc/boxed.rs:1022
#27 std::sys_common::thread::start_thread () at src/libstd/sys_common/thread.rs:13 #28 std::sys::unix::thread::Thread::new::thread_start () at src/libstd/sys/unix/thread.rs:80
#29 0x00007ffff7d58669 in start_thread (arg=<optimized out>) at pthread_create.c:479 #30 0x00007ffff7c66323 in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95
vulkaninfo:
========== VULKANINFO
==========
Vulkan Instance Version: 1.1.114

Instance Extensions: ====================
Instance Extensions	count = 18 	VK_EXT_acquire_xlib_display         : extension revision  1
	VK_EXT_debug_report                 : extension revision  9 	VK_EXT_debug_utils                  : extension revision  1
	VK_EXT_direct_mode_display          : extension revision  1 	VK_EXT_display_surface_counter      : extension revision  1
	VK_KHR_device_group_creation        : extension revision  1 	VK_KHR_display                      : extension revision 23
	VK_KHR_external_fence_capabilities  : extension revision  1 	VK_KHR_external_memory_capabilities : extension revision  1
	VK_KHR_external_semaphore_capabilities: extension revision  1 	VK_KHR_get_display_properties2      : extension revision  1
	VK_KHR_get_physical_device_properties2: extension revision  1 	VK_KHR_get_surface_capabilities2    : extension revision  1
	VK_KHR_surface                      : extension revision 25 	VK_KHR_surface_protected_capabilities: extension revision  1
	VK_KHR_wayland_surface              : extension revision  6 	VK_KHR_xcb_surface                  : extension revision  6
	VK_KHR_xlib_surface                 : extension revision  6 Layers: count = 4
======= VK_LAYER_LUNARG_standard_validation (LunarG Standard Validation Layer) Vulkan version 1.0.114, layer version 1
	Layer Extensions	count = 0 	Devices 	count = 1
		GPU id       : 0 (Intel(R) HD Graphics 515 (Skylake GT2)) 		Layer-Device Extensions	count = 0
 VK_LAYER_MESA_overlay (Mesa Overlay layer) Vulkan version 1.1.73, layer version 1
	Layer Extensions	count = 0 	Devices 	count = 1
		GPU id       : 0 (Intel(R) HD Graphics 515 (Skylake GT2)) 		Layer-Device Extensions	count = 0
 VK_LAYER_VALVE_steam_overlay_32 (Steam Overlay Layer) Vulkan version 1.0.3, layer version 1
	Layer Extensions	count = 0 	Devices 	count = 1
		GPU id       : 0 (Intel(R) HD Graphics 515 (Skylake GT2)) 		Layer-Device Extensions	count = 0
 VK_LAYER_VALVE_steam_overlay_64 (Steam Overlay Layer) Vulkan version 1.0.3, layer version 1
	Layer Extensions	count = 0 	Devices 	count = 1
		GPU id       : 0 (Intel(R) HD Graphics 515 (Skylake GT2)) 		Layer-Device Extensions	count = 0
 Presentable Surfaces:
===================== GPU id       : 0 (Intel(R) HD Graphics 515 (Skylake GT2))
Surface type : VK_KHR_xlib_surface Formats:		count = 2
	B8G8R8A8_SRGB 	B8G8R8A8_UNORM
Present Modes:		count = 3 	IMMEDIATE_KHR
	MAILBOX_KHR 	FIFO_KHR
VkSurfaceCapabilitiesKHR: 	minImageCount       = 3
	maxImageCount       = 0 	currentExtent:
		width       = 256 		height      = 256
	minImageExtent: 		width       = 256
		height      = 256 	maxImageExtent:
		width       = 256 		height      = 256
	maxImageArrayLayers = 1 	supportedTransform:
		VK_SURFACE_TRANSFORM_IDENTITY_BIT_KHR 	currentTransform:
		VK_SURFACE_TRANSFORM_IDENTITY_BIT_KHR 	supportedCompositeAlpha:
		VK_COMPOSITE_ALPHA_OPAQUE_BIT_KHR 		VK_COMPOSITE_ALPHA_INHERIT_BIT_KHR
	supportedUsageFlags: 		VK_IMAGE_USAGE_TRANSFER_SRC_BIT
		VK_IMAGE_USAGE_TRANSFER_DST_BIT 		VK_IMAGE_USAGE_SAMPLED_BIT
		VK_IMAGE_USAGE_STORAGE_BIT 		VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT
VkSurfaceCapabilities2EXT: 	supportedSurfaceCounters:
		None
GPU id       : 0 (Intel(R) HD Graphics 515 (Skylake GT2)) Surface type : VK_KHR_xcb_surface
Formats:		count = 2 	B8G8R8A8_SRGB
	B8G8R8A8_UNORM Present Modes:		count = 3
	IMMEDIATE_KHR 	MAILBOX_KHR
	FIFO_KHR VkSurfaceCapabilitiesKHR:
	minImageCount       = 3 	maxImageCount       = 0
	currentExtent: 		width       = 256
		height      = 256 	minImageExtent:
		width       = 256 		height      = 256
	maxImageExtent: 		width       = 256
		height      = 256 	maxImageArrayLayers = 1
	supportedTransform: 		VK_SURFACE_TRANSFORM_IDENTITY_BIT_KHR
	currentTransform: 		VK_SURFACE_TRANSFORM_IDENTITY_BIT_KHR
	supportedCompositeAlpha: 		VK_COMPOSITE_ALPHA_OPAQUE_BIT_KHR
		VK_COMPOSITE_ALPHA_INHERIT_BIT_KHR 	supportedUsageFlags:
		VK_IMAGE_USAGE_TRANSFER_SRC_BIT 		VK_IMAGE_USAGE_TRANSFER_DST_BIT
		VK_IMAGE_USAGE_SAMPLED_BIT 		VK_IMAGE_USAGE_STORAGE_BIT
		VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT VkSurfaceCapabilities2EXT:
	supportedSurfaceCounters: 		None

Groups : ========
	Device Group Properties (Group 0) : 		physicalDeviceCount = 1
 			Intel(R) HD Graphics 515 (Skylake GT2) (ID: 0)
 		subsetAllocation = 0
 	Device Group Present Capabilities (Group 0) :
 		Intel(R) HD Graphics 515 (Skylake GT2) (ID: 0)
		Can present images from the following devices: 			Intel(R) HD Graphics 515 (Skylake GT2) (ID: 0)
 		Present modes:
			VK_DEVICE_GROUP_PRESENT_MODE_LOCAL_BIT_KHR

Device Properties and Extensions : ==================================
GPU0 VkPhysicalDeviceProperties:
=========================== 	apiVersion     = 0x401066  (1.1.102)
	driverVersion  = 79699976 (0x4c02008) 	vendorID       = 0x8086
	deviceID       = 0x191e 	deviceType     = INTEGRATED_GPU
`	deviceName     = Intel(R) HD Graphics 515 (Skylake GT2)
...

The returned index appears to be -1, which hints on invalid usage or some kind of invalid internal state in the driver and/or the surface. Do you have validation layers installed? If you do, you should see some output when running the application in debug, and it may tell us what's going wrong.

At the very least, we could check if the index is specifically -1 and issue a warning in this case, as a workaround for this unspecified driver behavior.

I recompiled, and had to update winit (from version 20 to version 21), now i can't reproduce the error anymore. (I had to rewrite some stuff as i couldn't clone events anymore, so that can affect races).
I suggest closing this for now and see if anybody else has this problem in the future. (or possibly validate the indices of the textures as a preventive measure).

I have the same issue when resizing a window with wgpu-rs. For example clone the lyon repo and run the wgpu example https://github.com/nical/lyon/tree/master/examples/wgpu (cargo build in the examples/wgpu folder).
I've also had this with other wgpu related things but this one is probably the simplest.


Drop asyncqlio

The dev for asyncqlio hasn't been maintaining the library despite it being extremely broken. She is also extremely arrogant making it hard to contribute to the library,
There are 3 ways to go about this:

Actually start filing issues, which will most likely go unnoticed.
Drop the library entirely. Doing this will involve a major rewrite and will probably be harder as I'd have to use actual SQL.
Keep most of the code in my library and mirror the whole thing. This is the hardest one of them all due to all the weird nuances in the code. But since I'm only using PostgreSQL I don't have to worry about supporting the other drivers.

This issue will be kept for future reference, and this is my first issue lol.

Closing this because I've already pushed the commit.


Add P2SH Outputs

The ability to add P2SH outputs will be added to the library whilst maintaining backwards compatibility of the library interface.
The NetworkType class already has the scriptHash prefix which can be recognised in addressToOutputScript. This function should return an output script for P2SH outputs according to the address. Currently this function creates P2PKH scripts in a convoluted fashion. For P2SH outputs, a new function will be created in p2sh.dart called createP2shOutputScript. It shall take the script hash and create a P2SH output script. This will allow P2SH outputs to be created programmatically from the script hash without being overcomplicated as in p2pkh.dart.
This solution is not perfect but keeps the library interface intact and is simple to implement.

This sounds like a viable way to me.
Since we probably have to start over at some point, we should now do what gets us there fastest (while reliably working.)

Sure thing, I shall get started. I will make a PR when it is ready. I'll name the branch 16-p2sh-outputs unless there is an alternative preferred naming scheme?

sure


Config structure for using multiple transformers with source-filesystem

Description
The current configuration file structure seems a bit awkward when using multiple transformers with source-filesystem plugin.
What I Expected to Work

{
      use: '@gridsome/source-filesystem',
      options:[
        {
          path: 'markdowns/**/*.md',
          typeName: 'Markdown',
          remark: {
            plugins: [
              // ...local plugins
            ]
          }
        },
        {
          path: 'yamls/**/*.yaml',
          typeName: 'Yaml',
          yaml: {
            plugins: [
              // ...local plugins
            ]
          }
        },
      ]
    },

What Actually Works

 plugins:[ {
      use: '@gridsome/source-filesystem',
      options:
        {
          path: 'markdowns/**/*.md',
          typeName: 'Markdown',
          remark: {
            plugins: [
              // ...local plugins
            ]
          }
        }
    },
    {
      use: '@gridsome/source-filesystem',
      options:
        {
          path: 'yamls/**/*.yaml',
          typeName: 'Yaml',
          yaml: {
            plugins: [
              // ...local plugins
            ]
          }
        }
    }],

The difference is that what I expected is that you should only state the usage of a plugin once and give multiple options. But what actually works is when you state that you use the plugin twice with different options
I know this is a very trivial issue but this seems more consistent with how webpack configs are written. I find it a bit awkward, never mind if it's just me.

Yes it seems quite awkward, and also the gridsome.config.js can get rather complicated and hard to read in a project with many data sources (e.g. an ecommerce site, dozens of schemas).


feat: wildcard support for AuthZ

Issue #, if available:
Description of changes:
Add glob wildcard (*) and MQTT wildcards (# and +) to resources in AuthZ.

'*' matches any character but '/'
'#' matches anything at and below its level, meaning it can only be used as the last character preceded with a '/', adding it anywhere else will result in being treated as a normal character.
'+' matches any character but within its level, meaning it should always be within two '/' (eg: 'abc/+/def'). Which will ultimately behave as '/*/'.

Why is this change necessary:
It allows users to add a group of resources with common prefix/suffix or a pattern.
How was this change tested:
basic Unit tests have been added to just test the wildcard logic. More tests would be added later on
Any additional information or context required to review the change:
Design Doc
IoT MQTT wildcards
Checklist:

[ ] Updated the README if applicable
[x] Updated or added new unit tests
[ ] Updated or added new integration tests
[ ] Updated or added new end-to-end tests
[ ] If your code makes a remote network call, it was tested with a proxy
[ ] You confirm that the change is backwards compatible

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Please name this PR in accordance with the conventional commit guidelines.
Please also fix the issues identified by LGTM

This pull request introduces 2 alerts when merging 5693a28 into e36c60d - view on LGTM.com
new alerts:

2 for Dereferenced variable may be null

The variable, if Null, would always fall into one of the handled conditions before dereferencing.

Please address the LGTM bot's issues.


Support more then one connection string

Add support for more than one connection string. One could choose which Connection String (and thus SSIS server) monitor from a specific menu item that should be added to the UI.
In this way, with just one dashboard, one can monitor more than one server.

Is support for more than 1 connection string coming. This is a great dashboard, but we have more than one ssis server, and it would be nice to see the two of them in one dashbaord.

Definitely but I haven't a schedule yet. I should take no more than one day to implement it. Btw a mutltiple SSISDB is already available here, which is a fork of my SSIS Dashboard: https://github.com/niphlod/ssis_dash

Thanks for the link. Not to sure on the other framework that is being used. I will wait for your update once it is ready

Hi @yorek,
Does the app still only support one connection string?
Thanks!

Yep, haven't been able to work on it...


docs: add a README to the Nixfiles, improve formatting

Motivation

There is already documentation for this, so just link to the existing docs

just handy to have a back-link for those less familiar with Nix and who stumble upon it
Similar to #10969

Modifications

add a README with a one-line link to dev/nix/

also improve some grammar and formatting in the Nix docs

use code backticks for all commands
use markdown convention of using 1. for all items of an ordered list and having the renderer determine the numbering
use the name of the doc (Running Locally) instead of "this"

more consistent with the rest of the docs, as well as k8s style guide: https://kubernetes.io/docs/contribute/style/style-guide/#links
also merge 1 and 2 as they're both mentioned in Running Locally

separate run-on sentence

Verification

Double-checked that the relative link worked locally

cc @isubasinghe

@isubasinghe  sorry I didn't realize Terry tagged you. Please review and comment it.

I didn't change any content, so thought it'd be ok for anyone to review, otherwise would cc @isubasinghe myself who knows Nix better than I currently do

LGTM since its just markdown


Breaks Rails 5.1

For Rails 5.1.6 the gem breaks rails itself:
rails aborted!
Don't know how to build task 'console' (see --tasks)
/Users/pawel/Sites/gratan-test/bin/rails:9:in `<top (required)>'
-e:1:in `<main>'
(See full trace by running task with --trace)

with --trace:
rails aborted!
Don't know how to build task 'console' (see --tasks)
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/rake-12.3.1/lib/rake/task_manager.rb:59:in `[]'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/rake-12.3.1/lib/rake/application.rb:159:in `invoke_task'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/rake-12.3.1/lib/rake/application.rb:116:in `block (2 levels) in top_level'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/rake-12.3.1/lib/rake/application.rb:116:in `each'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/rake-12.3.1/lib/rake/application.rb:116:in `block in top_level'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/rake-12.3.1/lib/rake/application.rb:125:in `run_with_threads'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/rake-12.3.1/lib/rake/application.rb:110:in `top_level'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/commands/rake/rake_command.rb:21:in `block in perform'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/rake-12.3.1/lib/rake/application.rb:186:in `standard_exception_handling'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/commands/rake/rake_command.rb:18:in `perform'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/command.rb:46:in `invoke'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/commands.rb:16:in `<top (required)>'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in `require'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in `block in require'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:258:in `load_dependency'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in `require'
/Users/pawel/Sites/gratan-test/bin/rails:9:in `<top (required)>'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:286:in `load'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:286:in `block in load'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:258:in `load_dependency'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:286:in `load'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/commands/rails.rb:6:in `call'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/command_wrapper.rb:38:in `call'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:201:in `block in serve'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:171:in `fork'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:171:in `serve'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:141:in `block in run'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:135:in `loop'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:135:in `run'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application/boot.rb:19:in `<top (required)>'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/Users/pawel/.rbenv/versions/2.3.7/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
-e:1:in `<main>'

It does not seem to be gratan's problem.


[BUG] Tray Icon doesn't have proper naming

Go Hass Agent Version
v9.6.0
Describe the bug
The app name is not defined and it's using the AppID value from prefs.go com.github.joshuar.go-hass-agent
To Reproduce
Maybe this is a KDE thing, I know Gnome has another Tray behaviour. But repro steps on KDE are:

Install app
Click on tray icon
Observe the issue

Expected behaviour
A proper name for the app, something along the lines of "Go Hass Agent"
Screenshots

Desktop (please complete the following information):
Operating System: EndeavourOS
KDE Plasma Version: 6.1.3
KDE Frameworks Version: 6.4.0
Qt Version: 6.7.2
Kernel Version: 6.10.2-arch1-1 (64-bit)
Graphics Platform: Wayland
Processors: 16 × AMD Ryzen 7 7800X3D 8-Core Processor
Memory: 30,6 GiB of RAM
Graphics Processor: NVIDIA GeForce RTX 3080/PCIe/SSE2
Manufacturer: ASUS

Hey thanks for reporting. I agree, the agent should use a better name. I've made commit 87d9ae4ba97a0e50c50b7a3a923fe610f201a6f9 which will make the agent display "Go Hass Agent". This will be in the next release.
Cheers!

Hey there, since v10 Go Hass Agent now displays a nicer tooltip. I'll close this issue off. Cheers!


refactor: small improvements for BottomTabsModel

This PR resolves []
Changes

rename unreadConversation to conversations
rename unreadActivityPanelNotifications to notifications
rename unreadActivityPanelNotificationsCountChanged to setUnreadNotificationsCount
rename unreadConversationCountChanged to setUnreadConversationsCount

PR Checklist

[ ] I tested my changes on iOS / Android.
[ ] I added screenshots or videos to illustrate my changes.
[x] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[ ] I hid my changes behind a feature flag.
[ ] I have prefixed changes that need to be tested during a release QA with [NEEDS EXTERNAL QA] on the changelog.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

small improvements for BottomTabsModel - dimatretyak

Need help with something? Have a look at our docs, or get in touch with us.

Way better 👌


Updating swagger-spec for kubernetes.io

It was last updated on Feb 26.
PR generated by running swagger-spec/update.sh

cc @bgrant0607 @vmarmol

LGTM


Update troubleshooting-shared-images.md

Issue description : https://github.com/MicrosoftDocs/azure-docs/issues/66107

@benhurjoel : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#please-close Looks like the string in the docs will be updated to match the new error message when the source is updated. This also has a merge conflict.


Clarify licensing?

Currently the artifacts in the repository suggest a confusing mishmash of licenses for this project:

the pom.xml file implies Apache-2.0 only
the COPYING.GPL file implies GPL-3.0
the COPYING.LESSER file implies LGPL-3.0
and the LICENSE file appears to be attempting to imply Apache-2.0 OR LGPL-3.0+ (but because it uses the abbreviated "file header" texts instead of the full license texts, tools that attempt to detect licenses from such files, including GitHub's own license detection, are unable to do so)

Suggested solution
Assuming the JFFI authors do intend to publish the work under Apache-2.0 OR LGPL-3.0+, I'd suggest:

remove the COPYING.GPL and COPYING.LESSER files
add a second <license> block within the existing <licenses> block within the pom.xml file, containing the name and URL of the LGPL-3.0+ license
add the full Apache-2.0 license text and the full LGPL-3.0+ license text to the LICENSE file, separated by a number of blank lines, and with text such as OR, AT YOUR DISCRETION: in the middle of the whitespace
(optional, but may keep FSF type folks happy) duplicate the (updated) LICENSE file and name it COPYING

Thanks for pointing out the mishmash. At one point the original author wanted to use GPL 3, but I convinced him to switch to Apache-2. It seems we have some cleanup to do.

@pmonks I have made these changes in #151, minus the duplicate COPYING file. If someone really wants that alongside LICENSE they can ask us for it.

Thanks @headius, and no need to apologise! Licensing is one of those chores that's super easy to mix, especially if it's changed over time!  Really appreciate you taking the time to sort this out.

Thanks for the help!


Raspberry Pi support

@homunkulus r+

:pushpin: Commit b9e1426 has been approved by japaric

:hourglass: Testing commit b9e14260d73e5e000dc6b73a813fbac3b9801a06 with merge 296d0af8b26a32a19a5ba6ccdaa5c52071069160...

:sunny: Test successful - status-travis
Approved by: japaric
Pushing 296d0af8b26a32a19a5ba6ccdaa5c52071069160 to master...


fix version masking

I noticed when testing that my recipe.json was being rebuilt when I updated my package version number

Seems to also be included in the Cargo.lock file, and there's currently no logic to mask it there. Should that also be done?

I am not 100% sure on what behaviour you are changing - isn't the old main version equivalent to your branch version?

So the code I replaced makes it seem that the Cargo.toml package.version value should be replaced with 0.1.0, but when I run cargo chef prepare, it is not getting replaced and the resulting recipe.json file is different (meaning my cache is invalidated and needs a rebuild of dependencies)

Understood - would you mind adding a test, so that we do not regress on this functionality going forward?

Almost there - just run cargo fmt and then we can merge 😁

Almost there - just run cargo fmt and then we can merge grin

There's always something :facepalm:


WebView not found

System:
OS: Linux 4.18 Ubuntu 18.04.2 LTS (Bionic Beaver)
CPU: (4) x64 Intel(R) Core(TM) i3-6006U CPU @ 2.00GHz
Memory: 2.04 GB / 11.65 GB
Shell: 4.4.19 - /bin/bash
Binaries:
Node: 8.10.0 - /usr/bin/node
npm: 3.5.2 - /usr/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
npmPackages:
"react": "16.8.6",
"react-native": "0.60.2",
"react-native-gesture-handler": "^1.0.5",
"react-native-webview": "^6.3.1",
"react-navigation": "^3.11.1"
npmGlobalPackages:
react-native-cli: 2.0.1

By any change are you importing WebView from react-native instead of using react-native-webview?

Same issue, RN 0.60.3, please help

Same issue...
I'm upgrading a project to RN 0.60.3 . Unfortunately, I'm getting the following error when I attempt to run on Android:
WebView has been removed from React Native. It can now be installed and imported from 'react-native-webview' instead of 'react-native'. See https://github.com/react-native-community/react-native-webview
I have run 'npm install --save react-native-webview' and confirmed that it is appropriately linked into the my underlying Android project (i.e. settings.gradle, build.gradle, imports, etc...). I've also made the appropriate changes to ensure I import WebView like so:
import { WebView } from 'react-native-webview';
and ensure that I've removed all occurrences of:
import { WebView } from 'react-native;
It compiles fine, but I get the run-time error above.
Any thoughts?

RNCwkwebview was not found
this problem is bothering me...

Same issue here on fresh React Native 0.60 install

Guys, I solved this problems, by check my node_modules, I have some dependencies, that import WebView from react-native, I replace them by react-native-webview, or update packages to new version

I had the same occurrence as @Ivan-Stashak-CardinalPeak but using RN 0.60.4
@tarasvakulka can you please elaborate more on which part of the dependencies?

@zodlanier , I remove react-native-qrcode from node_module, that use WebView from recta-native, now I use react-native-qrcode-svg

still searching for solutions

i have the same issue


Youtube mix/playlist request?

Documentation is not really clear on how to compose a Youtube request: you might want to add a section in the readme file.
As far as I'm aware the existing request methods are the following:
Request output

http://localhost:5005/youtube/direct/<youtubeID> → A simple redirect to final stream URL. (faster, no disk usage, sponsorblock not works)
http://localhost:5005/youtube/bridge/<youtubeID> → Remuxing on fly. (fast, no disk usage)
http://localhost:5005/youtube/download/<youtubeID> → First download full video then it's served. (slow, temp disk usage)

Request output type

http://localhost:5005/youtube/direct/<youtubeID>-audio → Return an audio only streaming
http://localhost:5005/youtube/direct/list-<youtubeID> → Return a streaming of an entire playlist (?)
http://localhost:5005/youtube/direct/list-<youtubeID>-audio → Return an audio only streaming of an entire playlist (?)

I'm not currently able to make the latest two in list works, hence the title of the issue: how shall I compose a request starting from a Youtube playlist such as https://www.youtube.com/playlist?list=PL6kFT_LlrYKW32II058F7c_NcJ3uCMNh6 ?
Also please feel free to expand the list if any other request possible is missing.
Thanks again for your time and the amazing work put into this app.

Sorry for the documentation, when I can I will work to update it and make it clearer.
The list should be declared in channel_list.json like
[
       "list-PL6kFT_LlrYKW32II058F7c_NcJ3uCMNh6"
]

After that, ytdlp2STRM will create a strm for each video in the playlist.

Currently extractaudio does not work with list prefix.

The list should be declared in channel_list.json

This is clear, I’d like to create a stream of a whole playlist through a request instead. Is it not possible to compose a request that return a single strm containig all the youtube videos of the playlist queued?

Currently extractaudio does not work with list prefix.

Assuming that part 1 is doable, will this be implemented?
I’m probably using your program in a different way than the one intended, hence my reliance on http requests.


perf: avoid unnecessary byte/string conversion

PR Details
We can use alternative functions/methods to avoid unnecessary byte/string conversion calls.
Description

(*Builder).WriteString(string(<a byte>)) -> (*Builder).WriteByte(<a byte>)
(*Regexp).Match([]byte(<a string>)) -> (*Regexp).MatchString(<a string>)

Related Issue

Motivation and Context

How Has This Been Tested

go test ./...
ok  	github.com/xuri/excelize/v2	63.499s

Types of changes

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

Checklist

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

Codecov Report

Merging #1541 (f0b8454) into master (a246db6) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master    #1541      +/-   ##
==========================================
- Coverage   98.82%   98.79%   -0.03%
==========================================
  Files          31       31
  Lines       24494    24494
==========================================
- Hits        24206    24200       -6
- Misses        191      195       +4
- Partials       97       99       +2

Flag
Coverage Δ

unittests
98.79% <100.00%> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

numfmt.go
100.00% <100.00%> (ø)

table.go
99.15% <100.00%> (ø)

... and 1 file with indirect coverage changes


Fix live video event and remove the latencyCompensator

After switching to AWS Media Services, I noticed that it doesn't show the live tracker anymore. So you can't go back in the previous 8 hours anymore. It is kinda okay in my opinion.
However, the dashboard needs to adjust some of the events that it is using to show that the video is live.

After switching to AWS Media Services, I noticed that it doesn't show the live tracker anymore. So you can't go back in the previous 8 hours anymore. It is kinda okay in my opinion.
However, the dashboard needs to adjust some of the events that it is using to show that the video is live.

Please see: https://mistywest.slack.com/archives/C02MGT5GK44/p1718068403460679?thread_ts=1717862617.770879&cid=C02MGT5GK44

LGTM!


Win10 can't download mjpeg stream as mp4 using ffmpeg

I use the Windows 10 operating system, I copied the ip address in the app and opened it on my computer's browser and it worked fine，and I use the following command in the cmd window:
ffmpeg -i http://<IP_ADDRESS>:8080/cam.mjpeg -c:v libx264 -crf 23 output.mp4

It can't save the mjpeg as an mp4 video, I just get a 25kb output.mp4, it's not able to play, it's just a frame.
How should I operate so that the video shot on the mobile phone can be transmitted to the computer in real time and saved as mp4 ?

If I were you I would just use OBS to receive the stream, and use its UI to record. In the Readme you will find the right settings to receive the stream.
I am sure there is an ffmpeg command that would work though, but I am no expert.

Ah ! can you try to add -f mjpeg to your command? :
ffmpeg -f mjpeg -i http://<IP_ADDRESS>:8080/cam.mjpeg -c:v libx264 -crf 23 output.mp4

Thanks a lot, the command you provided above it worked! Now ffmpeg is indeed able to save mjpeg streams to mp4. But your project only supports video now? Do you consider transmitting the sound through the mobile phone, so that I can get the mp4 video with sound through the following ffmpeg command:
ffmpeg -f mjpeg -i http://<IP_ADDRESS>:8080/cam.mjpeg -i /dev/video0 -c:v libx264 -crf 23 -c:a aac -b:a 128k output.mp4

my obs can't show, but browse is ok
my obs is diferent with yours.

Careful only the last client gets a feed. You should close the browser tab and only open the OBS source. If it is not that, the problem on your OBS side.


Parent Child Component extension example

Hi @dev4jam  ,
Thanks for the great example, could you explain please, why do we need
protocol ParentDependencyChild : Dependency {}

and how to use it, because I could not find the use case for it. As I saw in your example and uber one, we always declare our Dependency's in a
protocol ChildDependency: Dependency {}

and use them as provided in example
extension ParentComponent: ChildDependency {
    // TODO: Implement properties to provide for Child scope.
}

Thanks.

Hi @dimaosa,
the whole point of this is to give a Child an opportunity to ask something (not common with other children) from it's Parent i.e. sub-dependency from services or maybe (from Uber TicTacToe tutorial) ask Parent to provide a view controller conforming to a specific Child's view protocol.
Clear or need some code?
PS: We have a slack community uber-ribs.slack.com. Would you like to join?

Yeah, clear enough @dev4jam , code example will also be good. Thank you, I would like to join uber-ribs.slack.com ( my email is<EMAIL_ADDRESS>

👍🏻 added!


Make documentation very easy to understand

Beyond describing all of the commands that are to be used when running locally or committing, we should document the process from start to finish assuming the end-user has none of the tools...
At the mininum, our documentation should explain the following tasks:
How to install things

brew?
git
ruby (rvm?)
bundle

Link to other, helpful applications

Sublime Text / Atom

Run locally
Flesh out the following a bit better?
git clone git@github.com:Widen/widen.github.io.git
cd widen.github.io
bundle install
jekyll serve --drafts

Add content
With examples to Jekyll documentation along with any other examples that might be helpful (syntax highlighting, style guides, required formatting?)

Post
Page

How to deploy content
Perhaps highlight a sustainable branching policy and fully explain that anything pushed to devlop will be build with travis and deployed.
How to view CI stuff
Should we post links/details to the travis stuff that might need to be maintained at some point down the road?
How to modify the architecture
Beyond allowing a user to create or edit content, we should also layout some instructions on editing the templates and configuration of our Jekyll implementation itself...

Modifying layouts
Adding assets
Leveraging plugins

It's already a bit easier to get started, instead of:
git clone git@github.com:Widen/widen.github.io.git
cd widen.github.io
bundle install
jekyll serve --drafts

You just need:
git clone git@github.com:Widen/widen.github.io.git
cd widen.github.io
make setup start

Both approaches failed for me because jekyll wasn't on my path .. Fixed by just opening a new console window, but maybe something that could be improved.

FWIW, I've always run it via bundle exec jekyl ...

Now that you say that, it's entirely likely that it was my fault because I've got rbenv in the mix.

Makefile should probably be updated to run bundle exec. The travis targets
already do this.
On Fri, Aug 28, 2015 at 4:33 PM Mark Feltner<EMAIL_ADDRESS>wrote:

FWIW, I've always run it via bundle exec jekyl ...
—
Reply to this email directly or view it on GitHub
https://github.com/Widen/widen.github.io/issues/8#issuecomment-135893518
.

Makefile should probably be updated to run bundle exec. The travis targets
already do this.

Done in a7072fac0ae1b1fbd336f0ed401ac7b6b0e8f295


Bottom ActivityBar overlapping with sidebar border

When sideBar.border has a color, bottom activity bar seems to overlap with it. resulting in the situation in the image. Toggling activity bar to be invisible restores the border area there. This overlap scales with the size/height of activity bar

  "workbench.colorCustomizations": {
      "sideBar.background": "#ffffff00",
      "activityBar.background": "#ffffff00",
      "sideBar.border": "#ff0000"
  }

it seems the border area there is controlled by "activityBar.border": "#ff0000" as using this fills the right border. But that also colors the top of the activityBar which may not be desired

technically nothing is wrong here
you still have borders, they are just transparent
it's a matter of personal taste, I want to have it
I can try to set the border height to 0 if there is no set color

more precisely, I will set only sidebar border color on activitybar border-right
unfortunately, I will leave it to the users to add the desired styles themselves, because this cannot be a general solution

Agreed and thanks. Feel free to close the issue when you deem appropriate


Feature get region name from region_id

Currently Magento 2 only offers the region_id as a number, from the Store Information,
While the Country ID was readable, e.g. EN.
The region id only returns a number, which makes it really hard to do something with it.

Fixed with latest change


Feature/adding dashboard banner

What does this PR do?
Adding the illustration made by @ekidneyrh to the dashboard.
Screenshot / video of UI

What issues does this PR fix or reference?
Fixes https://github.com/containers/podman-desktop-extension-ai-lab/issues/905
How to test this PR?
Open dashboard page

Wondering PNG vs SVG ?

The svg provided has a lot a properties related to the tool used, therefore the png was a better alternative as it does not come with all of this metadata

usually we cleanup metadata ( you can drop them with tools or manually etc)  but I would go with svg

I would define it as a svelte component as well

@benoitf @jeffmaury  I created a svelte component for the svg after removing the meta data


Questions

It appears that while iorejson utilizes ioredis, from the documentation it appears that iorejson does not support a Redis cluster and does not support the other Redis commands.  Is that correct?
If not using clustering, I would have to create two clients:  one using ioredis for other commands/data-types and one using iorejson for managing JSON data types.  Correct?
Are there any plans to extend iorejson to support Redis clusters and the rest of the Redis command set?

It's correct that it does NOT support redis cluster at the moment. It's also unclear whether the rejson module itself is suitable for cluster use yet, but I suppose it should be doable since the underlying client is transparent.
As for other regular redis commands, you can access the underlying client directly:
var Rejson = require('iorejson');

var instance = new Rejson();
yield instance.connect()
var client = instance.client; // the ioredis client

client.get('foo'); // do your work here

Okay, thanks for the clarification.


Exploration Perk Tree

Perk Tree

Name: Exploration
Art: Road
Unlocked by: Finding a significant landmark (i.e. Blackreach, Throat of the World)
Improved by: Discovering new places (or just walking around, if the former isn't feasible)
Perks: Provides general navigation and exploration buffs when traveling to many different areas of the map

Unlocking can be done by detecting player cell changes, improving might be possible with the same technique. If it isn't, we can simply copy Frostfall's method for advancing when the player walks around.
Perks

Shelved for now.


Search for Player Returns 0

`var searchParameters = new PlayerSearchParameters
{
//ResourceId = 188567
Page = 1,
Level = Level.Silver,
MaxBuy = 10000,
MinBid = 200
};
        var searchResponse = await futClient.SearchAsync(searchParameters);
        Console.WriteLine(string.Format("Found {0} auctions", searchResponse.AuctionInfo.Count));`

When i search by the ResourceId it returns results but when i search by using the Parameters. Am i using the correct search Parameters?

I'm having the same issue as well, I'm not getting any results even with no filters applied...


Defining roadmap to a feature complete web documentation

We should define actionable tasks for reaching the point where we can say the web documentation is good and complete.
And for that there are a few decisions to make:

Sidebar: I would like to have the main topics displayed there for ease of navigation, like the CHM. Is this possible to achieve?
It's fine even having a javascript array of topic-titles somewhere, that I can parse to build it.
Index and interface: keep how it's currently implemented, as separate page, or make something else? It wouldn't be hard to make something like the chm, where you can filter the results.
It would also be helpful if I have access of it as an array, so I can match things like Character.Say exactly and offer it as a search result that positions the user at the correct anchor.
Web vs CHM: Replace the CHM with a web version?
I can find ways to make it fully work locally as a SPA without a webserver, but that will require having all the contents embedded either in the HTML or a javascript. Or, we could ship one of those third party micro webservers and serve it on localhost:somenumber.
It's also possible to render markdown on the fly straight from javascript, or even build search indexes locally and saving them on localstorage and then checking against build number to regenerate them.
Mobile: hamburger menu and stuff. But does anyone unironically use the docs on a phone?

I'd say only 1&2 should be considered for the short term. I would like to at least have the genindex content on a javascript array, so I can load the page with the correct anchor.
Related issues:

#122
#130
#101

I use the docs on a phone. It's the primary way I read stuff, even with the computer on. I even have a epub version I made for myself to read on a Kindle... In the reader I use on the phone (moon reader), it has a nice bar below which has icons for menu options including the "chapters", which are the headers. It's also how I revise text usually.

About SPA, I really feel static pages are better, they get better indexed on Google and other search engines, and they usually work with any browser. The fact SPAs work offline is nice - I have not seen evidence people actually install them though.
About the chm, if we want to replace it, the new build possibly won't be that different html wise, as it would need to function in a limited html capacity - we probably don't want to throw a huge browser along with the AGS Editor to run the docs, and there isn't a huge offer of HTML reading C# projects or embeddable Web Views. It would be more about getting a good indexing system and then making the ui in C#.
Lastly, about the Sidebar, I think we need to manually maintain it, with only a script to check there's exactly one link for every document, reporting missing and duplicates. There's anything that differentiates a "see also" link from any other link currently, so there's no way to grasp a tree structure from just looking into the files.

But you're using an app, so any web improvement is probably useless for you! 😛
About SPA/CHM We can have both static and spa (not the installable kind), it's just that if we use the file:// protocol we must only use only one page,or the localstorage will be different for evey single document. Issue that would be solved with a micro-webserver like php or python's, or anything else that can serve files through the http protocol, if we want to keeping it offline.
Running from the http: protocol also has the advantage that I can async load other pages content via ajax and pretend the url has changed without actually leaving the page. Without http I can still use the # to fake page navigation, but I won't be able to make ajax calls, which sadly means I need a javascript way to have the contents.
There are many options and hybrid approaches to make one's head spin. In the end, it's all about the end result we want to achieve.

The localStorage is the same across a domain, so it's shared between our pages - they are served in the same domain.

In my mind there are two separate parts to the manual:

the script API documentation
the part of the manual that describes the concepts and general usage of the programs

Something like a printed manual or public website would likely incorporate both of those parts, and that is also where the indexing becomes more complicated. The script API is potentially much easier to index if it acts purely as a lookup of a symbol, potentially just an index entry that points to the function signature and a brief description of what it does (perhaps with a short example). Technically, if required, this could also be synced to docstrings in the source code to make sure that everything is actually documented and stays in sync with branches.
The other part of the manual is more like a book, so that needs something that reads in a more structure way, but equally someone may be interested in how to do X, where the ideas, concepts, and examples surrounding X need to be somehow located, probably across multiple pages.
I don't think anyone is using the CHM viewer because that is their preferred way to read the tutorial, more likely they would prefer that in a browser or exported as a PDF. I think what it provides is the fast API index lookup from the current script editor and the special focus rules for its window (which allows it to co-exist more nicely with windows of the application which invoked it). To replace the CHM viewer I think whatever replaces it should target editor support, and I'm suggesting any editor, not the Editor.
It probably isn't a good idea to get hung up on the tooling before we know what the goal is. To suggest some ideas, how about trying to create the following two components:

Generic editor support tool
Along with the general move of the build functionality from the Editor and into external tools, that should also include generic editor support. The tooling should allow the look-up of API documentation to be made by an editor and then returned back to the editor. This acts as a true replacement for the CHM viewer because the editor is responsible for displaying the content and that can cover cases where window focus needs to be handled differently. This should remove the main argument for moving away from the CHM viewer because the new viewer has the context and ability to provide "fixes" to complaints.
Maybe this falls within or alongside something like an LSP server, maybe it has to be entirely separate, but effectively this is some sort of symbol lookup and a return of some kind of structured data. Maybe something like an HTML page would be an optional return type, but the focus is on the data not the presentation. I think more likely you would return the content itself as text (String.Format function signature), with optional links ("read more about formatting strings"), which an editor can treat as something like a tooltip. If it is the world's worst LSP server that just returns the docstrings for the current context, that would do as a start.

Public website
This is essentially the AGS book. The contents of the site are structured in a way that works as both an introductory guide and a reference, the script API reference is present but listed separately.
The contents page of the site describes chapters which introduce ideas and concepts, the relevant keywords that relate to content and concepts are listed in the general index. The key thing here is that the script API index is separate, and just lists all functions and objects. The script API index entries point to the "docstring" reference for that symbol, with additional entries for where the same symbol is explicitly mentioned anywhere in the manual.
I'll also suggest it should be printable and equally usable in its printable form, to literally function as book, on paper or electronically with a document viewer.

Potentially there is substantial cross-over in how both of these are built and function, I imagine there would be multiple options for bundling the equivalent of the website with editor tooling and searching it, but does the above sound feasible as a way to replace the CHM viewer and have a better web-based manual (potentially resolving all three of the related issues linked in this ticket)?


refactor: simplify project structure

Thanks for providing the module @Larpon.
It does its magic to monitor directories on a server that are the target of a scanner, to then trigger a cloud sync event when a user scans a document in the office :magic_wand: .

This PR updates makes small updates the project structure to simplify and clean things up. If the PR doesn't introduce unwanted changes, if I would make further small updates that are imo. improvements in a follow-up PR.
Regarding an installation with the submodule: Instead of updating the install command in the readme in this PR to clone the submodule, the previously submitted PR that updates the install command would cover it.

Thanks for the PR. I'm happy to jear you find the project useful ❤
I don't use git submodules in any of my V (nor any other) projects for the following reasons:

needs an extra flag when manually cloning
no easy way to provide patches to the submodule sources (several of my projects patches or tweaks the code)
including the source directly is a "batteries included" experience, git submodules is not

If you're missing anything or the module / C source needs updating just open an issue.
I'm not a git submodule kind of guy 🙂🤷🏼♂

Thanks, for the review! I see and understand the reasons for NOT using a submodule, so it's all fine.
At first I was also about to submit the changes without the last commit to use a submodule, but added it in the end as well. So the main changes that were intended in this PR are included in the penultimate commit. If that's a thing to merge repo state at 206d8c0.


Update step-07.markdown

Wrong curl request params

Hey, thanks for the PR. I know it is only a small commit, but we really need you to sign the CLA

. I signed it!
Il 15/Apr/2016 19:55, "Paul Kinlan"<EMAIL_ADDRESS>ha scritto:

Hey, thanks for the PR. I know it is only a small commit, but we really
need you to sign the CLA
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/google/WebFundamentals/pull/2807#issuecomment-210564690

Now, iirc we've changed this a number of times in the past.
Every example that I have seen of this has the " around all of the Content-Type header, and not after the ":"..... which is correct and why?

I've tested curl on MacOsx and it doesn't work.
After many tests I have succeeded by doing the same thing that you have
done in your Demo:
https://simple-push-demo.appspot.com/
I thing that the problem is curl's version on MacOsx because now i tried on
linux (curl version 7.47.0) and it works in both ways!
Il 15/Apr/2016 21:51, "Paul Kinlan"<EMAIL_ADDRESS>ha scritto:
Now, iirc we've changed this a number of times in the past.
Every example that I have seen of this has the " around all of the
Content-Type header, and not after the ":"..... which is correct and why?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/google/WebFundamentals/pull/2807#issuecomment-210616204

It looks like the proper syntax for curl is to quote the whole header, not just the value.
I'm going to close for now, if you can find something that indicates otherwise, please re-open.


Brandeis attorney program updates

Update Brandeis attorney program section on Students and Graduates page (issue 53)
Additions

PDF

Changes

Text about applying

Testing

Navigate to http://localhost:8000/about-us/careers/students-and-graduates/ and check that Brandeis program subheader is updated & links to pdf

Review

@contolini
@sebworks

Screenshots

Notes

Is the download link styled correctly?

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

LGTM, @schaferjh what do you think?

Could we break the bolded h5 text into two lines? Once that's done, looks good. thanks!
(download link looks fine)

Line break added:

👍

Do we need a period on the second line?

@sebworks Good question. Wasn't sure if it would look strange with the icon/spacing:

yeah, I'm not sure. LGTM. Merge when ready.


localtunnel.me

since switching to localtunnel.me sharing is unusable, just stucks while loading page.
$ curl -v https://urtbzajshd.localtunnel.me/
*   Trying <IP_ADDRESS>...
* Connected to urtbzajshd.localtunnel.me (<IP_ADDRESS>) port 443 (#0)
* TLS 1.2 connection using TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256
* Server certificate: *.localtunnel.me
* Server certificate: COMODO RSA Domain Validation Secure Server CA
* Server certificate: COMODO RSA Certification Authority
* Server certificate: AddTrust External CA Root
> GET / HTTP/1.1
> Host: urtbzajshd.localtunnel.me
> User-Agent: curl/7.43.0
> Accept: */*
>

.... waiting till getting timeout

< HTTP/1.1 504 Gateway Time-out
< Server: nginx/1.10.1
< Date: Tue, 26 Jul 2016 08:26:28 GMT
< Content-Type: text/html
< Content-Length: 183
< Connection: keep-alive
<

<title>504 Gateway Time-out</title>

504 Gateway Time-out

nginx/1.10.1

* Connection #0 to host urtbzajshd.localtunnel.me left intact

The new version seems to time out as well after about 30 minutes even if you use a PTC Account to login.

Same. Timing out after about 30 mins.


996 refactor dietary restriction

Issue: The client requested that the previous dietary restrictions page be made into a page where they can print information about their volunteers such as dietary restrictions emergency contact info, etc.
Fixes: We implemented a refactor on the page which included two available view modes, a card view, and a table view. We added checkboxes so users could choose which information they view from the volunteers. We also added a print button that will automatically print the table or card view based on which view the user is currently on.  Additionally, implemented Data Table for displaying the data in a table view.
Test: Add Event participants, and rsvps, rsvps should include both people on the waitlist and those not on the waitlist. There should be some members with dietary restrictions. Ensure proper rendering of data, and that checkboxes properly toggle the respective data to be shown or hidden.
Resolves #996

This issue#996 is almost done, the only thing left is to make sure the barcode on the Cardview is correct and can be scanned. When we checked using a scanner, none of them could be scanned with the phone.


feat: invoke branch version of sidekick

Fix #60

:tada: This PR is included in version 5.3.0 :tada:
The release is available on:

GitHub release
v5.3.0

Your semantic-release bot :package::rocket:


Use 'app.mountpath' instead of 'base' option

The base option is not stable enough and doesn't respect express philosophy.
Its purpose can be replaced by internally using the Application.mountpath express property.

Fixed in https://github.com/league-of-legends-devs/league-tooltips/commit/073faa0ef66bfb433dbcf5487f3be864882aa549.


Show seed encoded with the PGP Word List.

Fixes #12.

Note that for this bitcoin version of Paymetheus, the word list encoding for the seed does not (currently) include any extra information such as a checksum or information about when the seed was generated (useful for rescans).  These details are abstracted out to the WalletSeed.EncodeWordList method, so Decred (which uses a single checksum byte, and which I dislike for various reasons) will only need to change the implementations in the WalletSeed class.

OK


BUG: Could not start v0.11.0 from Docker Compose

我也是最新版的镜像起不来

me too.+1

@Minamiyama @yanmao2023 @XiaoCC Could you please help me test this:
Build a new image based on our offical image:
FROM xprobe/xinference:v0.11.0

RUN pip install torchvision==0.17.1

And then test it?
In my own machine with two GPUs, I can use xinference it normally with above method.

@Minamiyama @yanmao2023 @XiaoCC Could you please help me test this: Build a new image based on our offical image:
FROM xprobe/xinference:v0.11.0

RUN pip install torchvision==0.17.1

And then test it?
In my own machine with two GPUs, I can use xinference it normally with above method.

It seems not work for me

@Minamiyama @yanmao2023 @XiaoCC Could you please help me test this: Build a new image based on our offical image:
FROM xprobe/xinference:v0.11.0

RUN pip install torchvision==0.17.1

And then test it?
In my own machine with two GPUs, I can use xinference it normally with above method.

It seems not work for me

Paste your error stack. And your related commands.

@Minamiyama @yanmao2023 @XiaoCC Could you please help me test this: Build a new image based on our offical image:
FROM xprobe/xinference:v0.11.0

RUN pip install torchvision==0.17.1

And then test it?
In my own machine with two GPUs, I can use xinference it normally with above method.

It seems not work for me

Paste your error stack. And your related commands.

@Minamiyama @yanmao2023 @XiaoCC Could you please help me test this: Build a new image based on our offical image:
FROM xprobe/xinference:v0.11.0

RUN pip install torchvision==0.17.1

And then test it?
In my own machine with two GPUs, I can use xinference it normally with above method.

It seems not work for me

Paste your error stack. And your related commands.

What's the error here? You can also add --log-level debug in your entrypoint command.
Could you just test:

build the new image
run it

docker run -p 9997:9997 --gpus all <the new image> xinference-local --log-level debug -H <IP_ADDRESS>

And then use it

@Minamiyama @yanmao2023 @XiaoCC Could you please help me test this: Build a new image based on our offical image:
FROM xprobe/xinference:v0.11.0

RUN pip install torchvision==0.17.1

And then test it?
In my own machine with two GPUs, I can use xinference it normally with above method.

It seems not work for me

Paste your error stack. And your related commands.

What's the error here? You can also add --log-level debug in your entrypoint command. Could you just test:

build the new image
run it

docker run -p 9997:9997 --gpus all <the new image> xinference-local --log-level debug -H <IP_ADDRESS>

And then use it

is this message useful?

@Minamiyama @yanmao2023 @XiaoCC Could you please help me test this: Build a new image based on our offical image:
FROM xprobe/xinference:v0.11.0

RUN pip install torchvision==0.17.1

And then test it?
In my own machine with two GPUs, I can use xinference it normally with above method.

It seems not work for me

Paste your error stack. And your related commands.

What's the error here? You can also add --log-level debug in your entrypoint command. Could you just test:

build the new image
run it

docker run -p 9997:9997 --gpus all <the new image> xinference-local --log-level debug -H <IP_ADDRESS>

And then use it

is this message useful?

What's this? It seems no relation with xinference and it may be the issue with your cuda environment.
The docker image uses pytorch image as the base image. You can try that whether you can use this image directly: `pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel`

@Minamiyama @yanmao2023 @XiaoCC Could you please help me test this: Build a new image based on our offical image:
FROM xprobe/xinference:v0.11.0

RUN pip install torchvision==0.17.1

And then test it?
In my own machine with two GPUs, I can use xinference it normally with above method.

It seems not work for me

Paste your error stack. And your related commands.

What's the error here? You can also add --log-level debug in your entrypoint command. Could you just test:

build the new image
run it

docker run -p 9997:9997 --gpus all <the new image> xinference-local --log-level debug -H <IP_ADDRESS>

And then use it

is this message useful?

 What's this? It seems no relation with xinference and it may be the issue with your cuda environment. The docker image uses pytorch image as the base image. You can try that whether you can use this image directly:
pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel

cause by adding --log-level, mayby it's my wrong usage

@Minamiyama @yanmao2023 @XiaoCC Could you please help me test this: Build a new image based on our offical image:
FROM xprobe/xinference:v0.11.0

RUN pip install torchvision==0.17.1

And then test it?
In my own machine with two GPUs, I can use xinference it normally with above method.

It seems not work for me

Paste your error stack. And your related commands.

What's the error here? You can also add --log-level debug in your entrypoint command. Could you just test:

build the new image
run it

docker run -p 9997:9997 --gpus all <the new image> xinference-local --log-level debug -H <IP_ADDRESS>

And then use it

is this message useful?

 What's this? It seems no relation with xinference and it may be the issue with your cuda environment. The docker image uses pytorch image as the base image. You can try that whether you can use this image directly:
pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel

cause by adding --log-level, mayby it's my wrong usage

nothing new shown, and auto shut down as well

pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel

Just run
docker run --gpus all pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel tail -f /dev/null

still auto shutdown?

@Minamiyama @yanmao2023 @XiaoCC Could you please help me test this: Build a new image based on our offical image:
FROM xprobe/xinference:v0.11.0

RUN pip install torchvision==0.17.1

And then test it?
In my own machine with two GPUs, I can use xinference it normally with above method.

It seems not work for me

Paste your error stack. And your related commands.

What's the error here? You can also add --log-level debug in your entrypoint command. Could you just test:

build the new image
run it

docker run -p 9997:9997 --gpus all <the new image> xinference-local --log-level debug -H <IP_ADDRESS>

And then use it

is this message useful?

 What's this? It seems no relation with xinference and it may be the issue with your cuda environment. The docker image uses pytorch image as the base image. You can try that whether you can use this image directly:
pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel

cause by adding --log-level, mayby it's my wrong usage

nothing new shown, and auto shut down as well

The host machine is windows OS. May cannot use <IP_ADDRESS>. I haven't tried windows. Remove -H <IP_ADDRESS> and try again.

pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel

Just run
docker run --gpus all pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel tail -f /dev/null

still auto shutdown?

running normally

pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel

Just run
docker run --gpus all pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel tail -f /dev/null

still auto shutdown?

running normally

Cannot reproduce.
docker pull xprobe/xinference:nightly-bug_torchvision_version

This image is built by #1485 . And I can use it normally on my ubuntu machine.

pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel

Just run
docker run --gpus all pytorch/pytorch:2.1.2-cuda12.1-cudnn8-devel tail -f /dev/null

still auto shutdown?

running normally

Cannot reproduce.
docker pull xprobe/xinference:nightly-bug_torchvision_version

This image is built by #1485 . And I can use it normally on my ubuntu machine.

failed as well

@Minamiyama Try this image:
docker pull xprobe/xinference:nightly-docker_crash_due_to_llama

0.11.1 is ok


initial draft - Realworld workshop

This isn't meant to be public yet, just getting initial stuff out there. When it's ready it can be added to the toolbar.


#To set up login and signup pages on the website

The login button already exists but is not working.

Hey @anuragverma108 ,Can you assign me this issue

Hey, @anuragverma108 can you please add the "gssoc-ext" label too


TestTcpMetrics fails after merging release-1.1 to master

TestTcpMetrics started failing after merging release-1.1 to master. See https://github.com/istio/istio/pull/10186
Circle:
https://circleci.com/gh/istio/istio/259212?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
Prow:
https://k8s-gubernator.appspot.com/build/istio-prow/pr-logs/pull/istio_istio/10186/e2e-mixer-no_auth/8869

PR #10218 hopefully fixes.

Marking as fixed.


jsonschema~=2.6 dependency causes DeprecationWarning under Python 3.7

cfn-lint version: 0.20.2
The jsonschema~=2.6 dependency causes a DeprecationWarning under Python 3.7:
  …/lib/python3.7/site-packages/jsonschema/compat.py:6: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
    from collections import MutableMapping, Sequence  # noqa

This was fixed in jsonschema 3.0.0. Can you please bump your jsonschema dependency to allow (or require) 3.0? (See also.)

We should be able to update after https://github.com/awslabs/serverless-application-model/issues/922 is complete.

This should be fixed in v0.22.0.


want open file with tab in macOS

OS: macOS 11.2.3 (20D91)
goneovim version: v0.4.11
Current: when I using open -a goneovim foo.md, the foo.md wiil opened and replaced current that I'm editing(saved).
Want: when I open file, goneovim will open file with tabe.

Hi :D
Added an option for this issue.
https://github.com/akiyosi/goneovim/actions/runs/679561026
You can specify the command to open the file as follows.
[Editor]
FileOpenCmd = ":tabe"

@akiyosi
Awesome! it works for me.


skip index testing in favor of new action based generation

Signed-off-by: Sam Curren<EMAIL_ADDRESS>

@TelegramSam - can we do this now?  It will be really great to have this in place sooner than later!
@dhh1128 ?

Merging would be fine with me.
On Fri, Dec 20, 2019, 4:57 PM Stephen Curran<EMAIL_ADDRESS>wrote:

@TelegramSam https://github.com/TelegramSam - can we do this now? It
will be really great to have this in place sooner than later!
@dhh1128 https://github.com/dhh1128 ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hyperledger/aries-rfcs/pull/357?email_source=notifications&email_token=AAQ3JCGHQCECRTPSKA3DIQLQZVLU7A5CNFSM4J4EEF5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHOPKAI#issuecomment-568128769,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ3JCCW5UEHCXYJHXF36LTQZVLU7ANCNFSM4J4EEF5A
.

Something odd is blocking this PR. I'm going to close this and reopen another with the same change.


Posting to different channels per pod with newer Slack APIs

We'd like to be able to use annotations such as the existing kube-slack/slack-channel one to control which Slack channel events for particular pods post to. However, if my understanding is correct, Slack's current (and recently changed, I think?) recommended path to set up webhooks now restricts things such that a given webhook is only allowed to post to a single channel. Reading from https://api.slack.com/custom-integrations/incoming-webhooks:

The majority of your legacy code for sending messages using incoming Webhooks should continue to work within a Slack app without much modification; the only thing you can no longer do is customize the destination channel and author identity at runtime.

So, I think I can't do what I want using this code as it is, because AFAICT the webhook URL is only globally specifiable and not configurable via a pod-level annotation. Is that correct? I'm happy to consider providing a patch to fix this but just want to ensure my understanding is correct first. Thanks for providing this library!

I haven't been following Slack's recent API changes as well. Thanks for informing us.
From what I see in the incoming webhook docs

You can't use Incoming Webhooks with Workspace Apps right now; those apps can request single channel write access and then use chat.postMessage in the Web API to post messages, providing very similar functionality to Incoming Webhooks.

I suppose incoming webhook will no longer works. I think here's what need to be done to move to web API

Migrate to official Slack library, since the one we're using doesn't support web API
Make sure that proxy configuration works for people who depends on that
Make our library support tokens for authentication, and send message using the web API.

This appears to be working since I upgraded (was on 3.1.1 when I opened the issue).


New JSON structure

Proposed JSON structure for next version:
{
  "admin": {
    "web": "https://trashserver.net",
    "name": "Thomas Leister",
    "email"<EMAIL_ADDRESS>  },
  "api": {
    "version": 1
  },
  "location": {
    "name": "Servercow",
    "coordinates": {
      "long": 666,
      "lat": 42
    }
  },
  "software": {
    "name": "Prosody",
    "version": "0.9.12"
  },
  "vhosts": [
    {
      "connections": {
        "s2s": {
          "incoming": {},
          "outgoing": {}
        },
        "c2s": {
          "count": 0
        }
      },
      "name": "uploads.security-enforced.de",
      "users_connected": 0
    },
    {
      "connections": {
        "s2s": {
          "incoming": [
            {
              "from": "trashserver.net"
            },
            {
              "from": "jabber.de"
            }
          ],
          "outgoing": [
            {
              "to": "trashserver.net"
            },
            {
              "to": "jabber.de"
            }
          ]
        },
        "c2s": {
          "count": 2
        }
      },
      "name": "security-enforced.de",
      "users_connected": 1
    },
    {
      "connections": {
        "s2s": {
          "incoming": {},
          "outgoing": {}
        },
        "c2s": {
          "count": 0
        }
      },
      "name": "conference.security-enforced.de",
      "users_connected": 0
    }
  ]
}

Implemented in https://github.com/ThomasLeister/mod_serverinfo/tree/745b1614b095e4e4c9a2291f8ab1ef8b51705d96
Waiting for feedback.

Installed on 5222.de and working.

@linkmauve do you have an opinion on the new JSON format? Any suggestions before I use it in the new module version?

Why are you using JSON instead of plain XMPP btw?  It would be way easier for servers to simply publish their data over PubSub than through HTTP, with all of the proxying, JSON encoding, etc. here.

I'm used to JSON are barely know the XMPP standard, so it's simpler for me to publish data in JSON format. The second reason is that JSON is wider spread when we are talking about web applications / javascript applications / etc. Also my backend is written in Golang, and there seems to be no adequate XMPP library.
I saw that you have quite a lot experience in XMPP / PubSub. So if you like to, you can add additional PubSub / XMPP support for this module, so we end up in having two different methods of publishing information. PubSub via XMPP for more advanced usage and JSON for "normal" usage.


I think this is because the language version is set to 1.7.x which doesn't exist.

I'll try updating and see if that resolve this.

The original issue is resolved now.
But I see this message :/

Caution: This kata does not currently have any known supported versions for Go. It may not be completable due to dependencies on out-dated libraries/language versions.

Did you republish the Go version with any changes in either author solution or text fixture so it triggers a re-validation?

I republished after adding a blank line and the message is gone :expressionless:
Thanks @Voileexperiments


Test improvements

It isn't clear that the changes in the tests accomplish the same thing as the original tests. The Original tests were organized around real-world examples of how one might use the API (or, rather, that was the intent). Its not obvious this has been translated into the replacement tests. It might help to clarify "X is covered by Y" and then enhance the fn doc to explictly state what is being covered by the test.

It isn't clear that the changes in the tests accomplish the same thing as the original tests. The Original tests were organized around real-world examples of how one might use the API (or, rather, that was the intent). Its not obvious this has been translated into the replacement tests. It might help to clarify "X is covered by Y" and then enhance the fn doc to explictly state what is being covered by the test.

I updated the commit message to indicate which new tests replace which old ones. I also added a multi-key, single-context test which I realized was missing in the new tests. Let me know if anything seems to need further clarification.


Update Composer dependencies (2020-05-19-00-11)

Visual regression test passed!
View the visual regression test report


chore: release workflow

Addresses #51
I have made the main.yml workflow reusable and used it as a test job upon which the publish job depends.

a pull_request of a branch named release/* will trigger a dry run
a push will trigger a deployment if the tags match this pattern:

'envied-v[0-9]+.[0-9]+.[0-9]+*'
'envied_generator-v[0-9]+.[0-9]+.[0-9]+*'

I didn't use melos here as I saw no need and because melos uses dart pub publish under the hood anyway.
I have left a comment for you in the workflow where you will have to create an environment with the name pub.dev (or you can name it differently) just as it's outlined here in this guide.

@techouse what would be the workflow for making a release with this action? It seems like you would need to have a tag or a release branch. You also would need to manually update the version and changelog before this is run.
I am thinking that either you need to make a "release" PR or we make it part of the contributing PR step to have contributors run the Melos script to generate a new version and changelog before the PR is merged.
I am curious what your thoughts are?

what would be the workflow for making a release with this action?

You push a tag, just like one of these that you already have here https://github.com/petercinibulk/envied/tags.
If you make a release/ branch PR it will run a dart pub publish --dry-run, essentially testing if it can publish without publishing.

Okay. Those tags and changelogs are created with the '''melos version''' command btw. I'll get that env added and merge this to test

Hold off on merging it just yet. I realised, that providing the environment for the dry run job is not desired and one can not just conditionally add it. I'll refactor this into a reusable workflow and then call it from 2 separate workflow files with their own environments respectively.

To avoid any future confusion I've renamed main.yml to test.yml (as that is essentially what it does).
I have also changed the publish.yml workflow to only trigger on pushing tags:
on:
  push:
    tags:
      - 'envied-v[0-9]+.[0-9]+.[0-9]+*'
      - 'envied_generator-v[0-9]+.[0-9]+.[0-9]+*'

for example:

envied-v1.2.3
envied-v4.5.6+780
envied_generator-v1.2.3
envied_generator-v4.5.6+780

Separately I have created a publish_dry_run.yml which as the name implies only does a dry run and is triggered on any pull request that has the branch name:
on:
  pull_request:
    branches:
      - 'release/v**'
      - 'release/envied/v**'
      - 'release/envied_generator/v**'

for example:

release/v1.2.3
release/envied/v4.5.6
release/envied_generator/v7.8.0+123


Package request: memprof-limits

Project description
Memprof-limits — Memory limits, allocation limits, and thread cancellation for OCaml
Memprof-limits is an implementation of (per-thread) global memory limits, (per-thread) allocation limits, and cancellation of CPU-bound threads, for OCaml. Memprof-limits interrupts a computation by raising an exception asynchronously and offers features to recover from them such as interrupt-safe resources.
This library is important for the development of OCaml interactive stuff.
Metadata

homepage URL: https://gitlab.com/gadmm/memprof-limits
source URL: https://gitlab.com/gadmm/memprof-limits
license: LGPL3
platforms: unix, linux, darwin, windows

ping @NixOS/ocaml


Missing numbers on compass ring on auto pilot

Theme with numbers on the ring:

Theme without

se earlier for more info

The problem is the use of apText in svg-autopilot.component.html
on text elements with numbers.
90
The numbers is on the ring where there are dashes
With hard coded black
<path
id="use6664"
d="m 98.386053,135.48218 14.374497,5.23484"
style="display:inline;fill:#000000;fill-opacity:1;stroke:#000000
Black is the better choise for that ring but class will be better
The ring is <ellipse class="compass-inner-ring"

Thanks for the feedback! I'll fix it.
There are plans to align the both Wind and AP dials to the same styles and also build the AP Widget with a responsive design which is lacking at the moment. One contributor said he was working on something but I've not seen his design sketches yet.
Also note that the Material styles need to be fixed to upgrade to angular 16+. It's a bit messy at the moment. We also plan to remove some themes to reduce code base and styling complexity in widgets. Most themes are not used.

now part of v2.6


test change

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Use TypeProcessor to discover javax.validation hints in a more dynamic way

Should allow to discover @Constraint(validatedBy = HostKeyAndAlgoBothExistValidator.class) automatically, simplify https://github.com/spring-projects-experimental/spring-native/blob/main/spring-native-configuration/src/main/java/org/hibernate/validator/ValidatorHints.java and revert https://github.com/spring-projects-experimental/spring-native/commit/147ff21b4e3796a9570d44c7bf040d8377c056ca.

Please use the repro described in https://github.com/spring-projects-experimental/spring-native/issues/1300#issuecomment-982488944 to check it works.

Spring Native is now superseded by Spring Boot 3 official native support, see the related reference documentation for more details.
As a consequence, I am closing this issue, and recommend trying your use case with latest Spring Boot 3 version. If you still experience the issue reported here, please open an issue directly on the related Spring project (Spring Framework, Data, Security, Boot, Cloud, etc.) with a reproducer.
Thanks for your contribution on the experimental Spring Native project, we hope you will enjoy the official native support introduced by Spring Boot 3.


net.sf.jsqlparser.JSQLParserException: Encountered unexpected token: "desc" "DESC"

Describe the bug
parser error.
To Reproduce
Steps to reproduce the behavior:

Example SQL : select a from b group by c desc;
Parsing this SQL using JSqlParser with this statements
Exception : JSQLParserException

Expected behavior
parser success.
System

mysql 5.7
Java 1.8
JSqlParser 4.0

I have never seen such a construct. Is group by c desc a shortcut for group by c order by c desc?

This is an old way of writing. This SQL can be run on MySQL.


Link to grandchild route broken in 2.3.0

Reproduction
https://stackblitz.com/edit/remix-run-remix-gbbwif?file=app%2Froutes%2F_.%24userId.blog.tsx

Have a grandparent route (_), parent (_.$userId) and child (_.$userId.blog).
From grandparent, use a Link <Link to="blog" relative="path">

System Info
It's same in stackblitz

Used Package Manager
npm
Expected Behavior
Clicking "Blog" shows the Blog route (Blog stuff goes here).
Actual Behavior
On 2.3.0, the route changes to /blog.
On 2.2.0, (the version in the blitz), the route correctly navigates  to  /$userId/blog

This is due to a bug fix in React Router 6.19.0 where when relative="path" was specified, we were losing the contextual route starting point.  This meant that the link href changed values every time you navigated since it was not properly tied to the contextual route.
All routing in React Router should be relative to the current route hierarchy, so relative links (or links starting with ./) start at the current location in the tree.  Specifying path=relative changes the behavior of how .. works - by default it traverses the route hierarchy, but you can tell it to instead traverse the path by URL segment.
In your example, the Link in _.tsx is relative to that route (path /), so <Link to="blog" relative="path> now correctly results in /blog.  If you want it to tack onto the end of the current pathname you have a few options:
// Use the current location
let location = useLocation();
<Link to={`${location.pathname}/blog`}>

// Or construct via the current params
let params = useParams();
<Link to={`${params.userId}/blog`}>

I guess your bug was my feature!
But thanks for the quick resolution, appreciate it


fix: regression caused by new error thrown

Description of changes
Previously, we were ignoring errors that had the type of "NotFoundException" when an S3 bucket did not exist and a user tried to delete it. If this is the case, we would exit without throwing an error.
The problem is that the "NotFoundException" was changed to "BucketNotFoundError" of AmplifyError type.
The remove-env-from-cloud.ts file was not updated to match, so it is throwing errors when S3 bucket does not exist/is already deleted.
Passing unit tests:

E2e run pending

Issue #, if available

Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #11118 (d208d98) into dev (85696bd) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev   #11118   +/-   ##
=======================================
  Coverage   49.63%   49.64%
=======================================
  Files         666      666
  Lines       31910    31910
  Branches     6496     6496
=======================================
+ Hits        15840    15841    +1
+ Misses      14590    14589    -1
  Partials     1480     1480

Impacted Files
Coverage Δ

...xtensions/amplify-helpers/remove-env-from-cloud.ts
100.00% <100.00%> (ø)

...li/src/domain/amplify-usageData/getUsageDataUrl.ts
100.00% <0.00%> (+12.50%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


SHA-256 sum of json-3.10.5.tar.xz changes over time (but not the content itself)

I noticed when running my build toolchain, that the md5sums of this file changed. I could extract an old file from a docker image and confirm this with a recently downloaded version of the file also with SHA-256. The archive contents itself are identical.
What is the issue you have?
I don't necessarily consider it a bug, because the  expected behaviour (md5sums or SHA-256 sums of archive remaining constant) is not promised anywhere but maybe this is an artifact of the build process, which can be fixed.
The equivalent zip file does seem to keep the SHA-256 sum I guess - because it is in the description of the release and did not change since the release I think. So maybe something is different about the creation of this zip file (maybe rerun of some github action in case of .tar.xz file involved).

Please describe the steps to reproduce the issue.

wget https://github.com/nlohmann/json/releases/download/v3.10.5/json-3.10.5.tar.xz
sha256sum json-3.10.5.tar.xz > sha256sum1.txt
wait some time (unknown amount. maybe a week)
wget https://github.com/nlohmann/json/releases/download/v3.10.5/json-3.10.5.tar.xz
sha256sum json-3.10.5.tar.xz > sha256sum2.txt
diff  sha256sum1.txt sha256sum2.txt

Can you provide a small but working code example?

What is the expected behavior?
SHA-256 sum of release file json-3.10.5.tar.xz does not change.

And what is the actual behavior instead?
SHA-256 sum of release file json-3.10.5.tar.xz does change over time.

Which compiler and operating system are you using?

Compiler: clang
Operating system: Debian 11 (stable) & Debian 11 (stable) in docker container

Which version of the library did you use?

[x] latest release version 3.10.5
[ ] other release - please state the version: ___
[ ] the develop branch

If you experience a compilation error: can you compile and run the unit tests?

[ ] yes
[ ] no - please copy/paste the error message below

Thanks for reporting. The feature of having the xz files is experimental still (see discussion https://github.com/nlohmann/json/discussions/3255). I am still struggling to make the tar.xz creation reproducible. The content already is, but it seems that the tar hash changes every time I re-create the archive.
Any idea how to fix this?
Once I know how to make the archive creation reproducible, I will officially announce the feature and will also list the expected hash in the release notes.

Currently at work - I can have a closer look at it this evening concerning the differences between the different files on a tar level. Maybe this stackoverflow answer points in the right direction:
https://stackoverflow.com/a/54908072

Yes, it is most likely the timestamps.  Why do you need to recreate and republish the archive?

I don't need to or want to recreate or republish the archive myself. I just pull the file from github in a build script for my own software and the checksum changes when I do this at different times.
In the interest of reproducible builds on my own part I'd like to get a bit identical tar.xz file.
I was suggesting to fix the workflow/action here at github, which creates this file, so that the downloaded file will always be identical. This might be fixed with giving some options to the tar command, but I'm not sure how this file is created here.

Why do you need to recreate and republish the archive?

I had to republish the archive because I forgot a file needed by MSVC.
Of course, this changes the hash etc., but in the end, I want to add a Makefile target to create the file and make it possible for anyone to reproduce this step.

@5balls

I don't need to or want to recreate or republish the archive myself.

I know, I was asking @nlohmann.

I had to republish the archive because I forgot a file needed by MSVC.

That makes sense.
@5balls it sounds like this bit may not be accurate:

The archive contents itself are identical.

Did you only compare file contents, or did you check for new files in the new archive?
If this only changed because a republish because of a broken zip file, and we don't expect this to be a regular thing, then it may not make sense to try to make it so that the archive creation is bitwise reproducible, as long as the contents can be verified.

Once I know how to make the archive creation reproducible, I will officially announce the feature and will also list the expected hash in the release notes.

As long as the release notes aren't part of the archive itself, then you can do that if you publish the archive before the release notes.  Otherwise, that's not possible.
If you do list it in the release notes, and you have to publish a new version of the archive because of a missing file, you can always change the archive name from json-3.10.5.tar.xz to json-<IP_ADDRESS>.tar.xz and add the new MD5 to the release notes with a note of the differences between the archives.

Did you only compare file contents, or did you check for new files in the new archive?

@gregmarr I think I did a diff on the directories and compared the individual md5sums of the files but I may have overlooked a file (although that should have been caught by the diff). I remember being surprised that the contents were the same. I also downloaded the .zip file and calculated the sha256sum afterwards, and this was matching the one in the release notes. Unfortunately yesterday the power supply of the machine I have the files on seems to have failed, so I will only be able to check this when I have replaced the faulty power supply.

If this only changed because a republish because of a broken zip file, and we don't expect this to be a regular thing, then it may not make sense to try to make it so that the archive creation is bitwise reproducible, as long as the contents can be verified.

@gregmarr Ah, yes, if the archive was just recreated due to manual intervention of @nlohmann in this case and it is a rare occurence I'm fine with that. I thought it could have been an - on the fly running github action - like encoding the archive at the time when downloading or something like that. I only thought that, because it was already released with a version tag and I thought it would be "frozen". Of course it would be a nice thing, if anyone could reproduce the archive with the identical hash at any time.

Maybe it would be good practice to create a new version if something changes?

This feature is still experimental. I added the xz files to the releases to test whether everything works as expected with FetchContent. Once this all is stable, there will be a single xz archive per release with a single (and hopefully reproducible) hash. Until then, don't make any assumption on these files - not even about their presence ;)

The feature is now documented: https://github.com/nlohmann/json/commit/c11f98228dac03fdddc89e5991861fc306874ef9
Please let me know what you think!

Thanks a lot, it looks great! I will test it but probably can't do so before next weekend when I have time again.


Add prepend-icon support on select tags + treeview more options

Prepend-icon
A small fix to allow prepend-icon on select tag with the logic used on input.
Tree-view
On treeview I added "before" option that allow us to specify where we want the leaf and not at the end.
Plus the option to directly add an icon to a leaf.
Example with one of my functions:
addLeaf: function( ev )
{
      var node = this.treeview.addLeaf(
        $( ev.currentTarget ).parent().parent()
        , "New " + $( ev.currentTarget ).attr( 'data-target' ) // can be parent or child
        , {
          before     : $( ev.currentTarget )
          ,icon      : "link"
          ,className : $( ev.currentTarget ).attr( 'data-target' )
        } );
      node.attr( 'data-type', 'url' );
}

please read this http://forum.metroui.org.ua/viewtopic.php?id=69

My bad was looking for this but didn't looked in the forum :)