The triangle example crashes with UnsupportedDimensions on MacOS 10.12 + MoltenVK

I'm using a 2017 MacBook Pro with Intel Iris Graphics 550 1536 MB and a 2560 x 1600 display.
The example builds successfully, and when I run it, it starts fine, printing some messages, including:
[mvk-info] You require a license for the MoltenVK Vulkan Core feature set. Reverting to evaluation mode
After that, it panics when creating a new Swapchain. The error is UnsupportedDimensions. Debug-printing the dimensions passed to the new method, the dimensions turn out to be [2048, 1536]. The device capabilities are
Capabilities {
	min_image_count: 2,
	max_image_count: Some(2),
	current_extent: Some([1024, 768]),
	min_image_extent: [1024, 768],
	max_image_extent: [1024, 768],
	max_image_array_layers: 1,
	supported_transforms: SupportedSurfaceTransforms {
		identity: true,
		rotate90: false,
		rotate180: false,
		rotate270: false,
		horizontal_mirror: false,
		horizontal_mirror_rotate90: false,
		horizontal_mirror_rotate180: false,
		horizontal_mirror_rotate270: false,
		inherit: false
	},
	current_transform: Identity,
	supported_composite_alpha: SupportedCompositeAlpha {
		opaque: true,
		pre_multiplied: true,
		post_multiplied: true,
		inherit: false
	},
	supported_usage_flags: ImageUsage {
		transfer_source: true,
		transfer_destination: true,
		sampled: true,
		storage: true,
		color_attachment: true,
		depth_stencil_attachment: false,
		transient_attachment: false,
		input_attachment: false
	},
	supported_formats: [
		(B8G8R8A8Unorm, SrgbNonLinear),
		(B8G8R8A8Srgb, SrgbNonLinear),
		(R16G16B16A16Sfloat, SrgbNonLinear)
	],
	present_modes: SupportedPresentModes {
		immediate: false,
		mailbox: false,
		fifo: true,
		relaxed: false
	}
}

So it seems that it fails because of the max/min image extent checks.
I'm not knowledgeable enough to determine which of these values are wrong, but if I understand the meaning of max/min_image_extent capabilities right, it seems odd to me that the device supports only that. What should I do to fix this or to diagnose the problem further?

Probably a HiDPI problem.

I'm not knowledgeable enough to determine which of these values are wrong, but if I understand the meaning of max/min_image_extent capabilities right, it seems odd to me that the device supports only that. What should I do to fix this or to diagnose the problem further?

Presumably the device only supports 1024x768 when you create a swapchain on a window whose dimensions are 1024x768. If the window had other dimensions, it would report these dimensions.

Ah, indeed, it sounds like a HiDPI issue; the resolution 2048x1536 is an exact double of 1024x768, and if I adjust the screen scaling, I can make it report things like 2048x1510 and 1024x755, 2040x1190 and 1020x595, which are again, exact doubles.
It gets the larger resolution from the created window and the smaller from the device.

Looking at the API of winit and the source code of the example, I think that it's missing to multiply the resolution with the HiDPI factor. I'm on mobile so I can't test this right now, gonna check this out later.

Oh, okay, so the winit API has both functions for returning pixels and returning points (2048 x 1536 and 1024 x 768 on my machine, respectively.) It seems then that the device capability info is talking about points instead of pixels in this case? Should it be this way? I'd imagine that something that talks about hardware-y things would rather talk in pixels than points, which I take to be a HiDPI abstraction that correspond to multiple hardware pixels.

The problem is that we're talking about MoltenVK, so it's not really an officially-official thing endorsed by Vulkan.
Anyway, I think the best fix is simply to use the current_extent instead of the window size.

let caps = window.surface()
    .capabilities(physical)
    .expect("failure to get surface capabilities");
let mut dimensions = caps.current_extent.unwrap_or([1024, 768]);

and the application freezes after a while and when i start it up again it freezes immediately with a blue background and a red triangle

It would be good to get rid of that logging, but since it's in MoltenVK they would have to remove it.
Is there are reason to use a fence with a timeout of zero? If not, getting rid of the zero-timeout fences might quiet things down.

A timeout of zero is used in order to check whether a fence is signalled without blocking the thread.
It's a valid thing to do (the specs even talk explicitly about that case if I remember correctly) and I'd even say that it is probably a very common thing to do.

Then I guess we should ask the MoltenVK guys to stop logging it all the time.
Unfortunately, fences are not yet supported on Fuchsia so their use generates a log of logging there, too.

Then I guess we should ask the MoltenVK guys to stop logging it all the time.

Most of the MoltenVK problems should be reported to the MoltenVK people. Unfortunately I'm not really the right person for that, as I don't have any Mac to try things.

I left a comment in their support forums.
https://moltengl.com/forums/topic/moltenvk-error-vk_timeout-vulkan-fence-timeout-after-0-nanoseconds/

@tomaka The UnsupportedDimensions panic is not related to macOS or MoltenVK.
I am on Linux and I have the same issue.
I was never able to run any of the Vulkano examples(LunarG examples were ok) - I always had only an Intel processor with integrated graphics(because I don't game) - all the examples at one point started panicking because my drivers didn't satisfy the Vulkan API version requirement(before that change, the examples were panicking for some other reasons).
A week ago I got an nVidia card(newer Vulkan) and tried Vulkano examples again but these were panicking anyway, so I decided to ignore it.
Now I stumbled on this and realized that it is the HiDPI issue.
I am on Linux with a 4K monitor and I get this panic:
thread 'main' panicked at 'failed to create swapchain: UnsupportedDimensions', /checkout/src/libcore/result.rs:860
stack backtrace:
   0:     0x556be89db423 - std::sys::imp::backtrace::tracing::imp::unwind_backtrace::hcab99e0793da62c7
                               at /checkout/src/libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1:     0x556be89d8346 - std::sys_common::backtrace::_print::hbfe5b0c7e79c0711
                               at /checkout/src/libstd/sys_common/backtrace.rs:71
   2:     0x556be89dd4ca - std::panicking::default_hook::{{closure}}::h9ba2c6973907a2be
                               at /checkout/src/libstd/sys_common/backtrace.rs:60
                               at /checkout/src/libstd/panicking.rs:355
   3:     0x556be89dd0cb - std::panicking::default_hook::he4d55e2dd21c3cca
                               at /checkout/src/libstd/panicking.rs:371
   4:     0x556be89dd8db - std::panicking::rust_panic_with_hook::ha138c05cd33ad44d
                               at /checkout/src/libstd/panicking.rs:549
   5:     0x556be89dd7b4 - std::panicking::begin_panic::hcdbfa35c94142fa2
                               at /checkout/src/libstd/panicking.rs:511
   6:     0x556be89dd6e9 - std::panicking::begin_panic_fmt::hc09fe500d9b7be81
                               at /checkout/src/libstd/panicking.rs:495
   7:     0x556be89dd677 - rust_begin_unwind
                               at /checkout/src/libstd/panicking.rs:471
   8:     0x556be8a02b2d - core::panicking::panic_fmt::h883a028e9f4b4457
                               at /checkout/src/libcore/panicking.rs:69
   9:     0x556be877e559 - core::result::unwrap_failed::ha1e1d8b7cb93815a
                               at /checkout/src/libcore/macros.rs:29
  10:     0x556be8746e91 - <core::result::Result<T, E>>::expect::hb16888bd05d6e26a
                               at /checkout/src/libcore/result.rs:762
  11:     0x556be8810dfb - triangle::main::h8066a67b6ac93b0f
                               at triangle/main.rs:188
  12:     0x556be89e47ca - __rust_maybe_catch_panic
                               at /checkout/src/libpanic_unwind/lib.rs:98
  13:     0x556be89ddfd0 - std::rt::lang_start::ha09816a4e25587ea
                               at /checkout/src/libstd/panicking.rs:433
                               at /checkout/src/libstd/panic.rs:361
                               at /checkout/src/libstd/rt.rs:59
  14:     0x556be8816b42 - main
  15:     0x7f20bb7af82f - __libc_start_main
  16:     0x556be86e3588 - _start
  17:                0x0 - <unknown>

The code from: https://github.com/vulkano-rs/vulkano-examples
The command: RUST_BACKTRACE=full cargo run --bin triangle

I hardcoded the size to [1024, 768] and it started but instantly went out of memory and almost crashed the system(UI freezed and the music was looping):
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: OutOfHostMemory', /checkout/src/libcore/result.rs:860

After a super busy month, I was able to return to this. Giving the Swapchain::new method a [1024, 768] starts the example app, and it seems to run fine. (There was the earlier comment about memory usage, but mine, on MacOS, seems to run at a steady level.)
So, it's a HiDPI issue, but it's hard to pinpoint yet where the exact issue is.
About the MoltenVK logging thing, I send a supporting comment:
https://moltengl.com/forums/topic/moltenvk-error-vk_timeout-vulkan-fence-timeout-after-0-nanoseconds/#post-1332

@tomaka As a confirmation, did you mean this, mentioned here: https://www.khronos.org/registry/vulkan/specs/1.0/man/html/vkWaitForFences.html

If timeout is zero, then vkWaitForFences does not wait, but simply returns the current state of the fences. VK_TIMEOUT will be returned in this case if the condition is not satisfied, even though no actual wait was performed.

is @tomaka able to test this library in a HiDPI environment?

I don't have an HiDPI monitor.

As a confirmation, did you mean this, mentioned here: https://www.khronos.org/registry/vulkan/specs/1.0/man/html/vkWaitForFences.html

Yes.

So, I think I solved the HiDPI bug. In Vulkano-win, when setting up the CAMetalLayer, the scaling factor of the layer has to be set, like this: layer.set_contents_scale(view.backingScaleFactor());. However, the wrapper for the setContentsScale API wasn't implemented in metal-rs, so I sent a pull request: https://github.com/gfx-rs/metal-rs/pull/10 Hopefully it will be pulled in quickly.
Also, according to the docs ( https://developer.apple.com/documentation/appkit/nsview/1483695-wantslayer ) if you set the layer of the view manually, setWantsLayer must be called AFTER the layer has been set:

In addition to creating a layer-backed view, you can create a layer-hosting view by assigning a layer directly to the view’s layer property. In a layer-hosting view, you are responsible for managing the view’s layer. To create a layer-hosting view, you must set the layer property first and then set this property to true. The order in which you set the values of these properties is crucial.

Currently setWantsLayers is set before:
let view = wnd.contentView();
view.setWantsLayer(YES);
view.setLayer(mem::transmute(layer.0)); // Bombs here with out of memory

Fixing these things, it reports the extents of the surface correctly and the resolution is correct.

I have a similar issue on linux with an intel gpu (part of my cpu). I don't know if I should start a separate issue. I'll put my details here for now.
It seems that on my card, the min/max extent listed in the capabilities matches the exact size of the window when it's created. Trying to create the swapchain with any other size results in UnsupportedDimensions.
I figured this out by adding a println inside swapchain.rs and comparing that against requested dimensions. I'll see if I can figure out how to make the examples use a size from the current capabilities (current_extent?) instead of the window size.

--- a/examples/src/bin/triangle.rs
+++ b/examples/src/bin/triangle.rs
@@ -110,8 +110,7 @@ fn main() {
     // Get the dimensions of the viewport. These variables need to be mutable since the viewport
     // can change size.
     let mut dimensions = {
-        let (width, height) = window.window().get_inner_size_pixels().unwrap();
-        [width, height]
+        window.surface().capabilities(physical).unwrap().current_extent.unwrap()
     };

     // The next step is to choose which GPU queue will execute our draw commands.
@@ -185,6 +184,7 @@ fn main() {
         let format = caps.supported_formats[0].0;

         // Please take a look at the docs for the meaning of the parameters we didn't mention.
+        println!("dimensions = {:#?}", dimensions);
         Swapchain::new(device.clone(), window.surface().clone(), caps.min_image_count, format,
                        dimensions, 1, caps.supported_usage_flags, &queue,
                        SurfaceTransform::Identity, alpha, PresentMode::Fifo, true,
@@ -340,8 +340,7 @@ void main() {
         if recreate_swapchain {
             // Get the new dimensions for the viewport/framebuffers.
             dimensions = {
-                let (new_width, new_height) = window.window().get_inner_size_pixels().unwrap();
-                [new_width, new_height]
+                window.surface().capabilities(physical).unwrap().current_extent.unwrap()
             };

             let (new_swapchain, new_images) = match swapchain.recreate_with_dimension(dimensions) {

This change works for me in the triangle example. It's so small that it seems silly to make a pull request, but I can do that if it would be useful.

dagit: What is the resolution reported by the window dimensions and what is the one reported by the surface capabilities API? Are you using a HiDPI setup? If you manage to start the example, does the resolution look normal or too low/pixely?

I'm not using a hidpi display. So it does look normal. The numbers reported by the capabilities matches what I would expect (1024x768). The only discrepancy that was causing issues was that inner pixel size was not exactly equal to the window size and my drivers didn't like that.

So they were off, but only slightly? That sounds like a problem with winit, more than Vulkano. But since Vulkano seems to care about the extent of the surface rather than the size of the window, that change might be good to do.

I think winit was correctly calculating the inner size. It was just a few pixels smaller than the window size.
I think the issue here is that my drivers report the expent (min/min) to be equal to the window size. So then trying to use any different size is a problem in vulkano and who knows, maybe it would be an issue for the driver as well. Reusing the current_extent is easy and seems to be fool proof.

Huh, I wonder how the Surface is created on Linux? Do you get it directly from the window or do you create it separately, choose the resolution yourself and link it with the Window...?

I expect you are using X. From Vulkano's source/docs:

Surface::from_xlib
Creates a Surface from an Xlib window.
The surface's min, max and current extent will always match the window's dimensions.

That means that if the dimensions are different, it's a bug somewhere. I agree that using the current_extent here is a foolproof thing to do, but that bug should be investigated also.

Okay, so the creation of the surface is handled by the platform-specific extensions, and on X, the surface size is always the window size, you can't affect that. Which means that X just calculates the window size using the whole window size, and expecting the inner area size to match the surface size is wrong. Or so it seems?

expecting the inner area size to match the surface size is wrong.

I agree with that, but I would also expect the driver to be reasonable and accept the window size or something slightly smaller (viewport size?). My only guess is that the driver isn't required to do so.

Sent a PR that addresses multiple of the problems discussed in this thread!


Don't require a PR approval before allowing merge

It will still be standard to have a PR reviewed before merging, but it's needlessly strict to have github enforce it.

Settings are on the develop branch here:
https://github.com/ncsa/phylodiver-viz/settings/branches
A PR is still required to merge to develop but not an approval.


Any way to delete/remove word from user dictionary?

Hi.
I see there is an add method to add words to user dictionary.
Is there a way to remove those words?
Thanks.

Sorry, found a duplicating #22

I was working again on this feature.
Tried just removing words from system files.
On Windows it is okay.
And on Mac I have a trouble that words added by this module in Electron to ~/Library/Spelling/en file. After those words are removed from that file, they are persisted in spellchecker - still marked as correct.
Seems a bug?


create handlebars

create handle bars so that we can make more pages and be able to switch between them.

show all monsters page, create monster page


Add signature to prevent griefing

Fixes issue in audit:

52

Has stack to deep errors preventing contract compilation

Needs #24 merged


Compatible with another plugin typewriter-mode

Are you using the latest version of the plugin?

[X] I have verified that I am using the latest version

Have you checked for similar issues?

[X] I have verified that there are no similar issues

What happened?
I am using another plugin at the same time https://github.com/davisriedel/obsidian-typewriter-mode
When two plugins are turned on simultaneously, once I start typing, the input box will run to the top and I cannot see the text I am typing
What did you expect to happen?
I hope that when typing text, the input box can automatically focus on the center of the screen
I'm not sure if this is a problem with this plugin or typewriter-mode
If you need typewriter-mode plugin to do something, please reply to me and I will go to that plugin to raise an issue
Relevant console output
No response
Obsidian version
1.6.3
OS
Windows

Thanks for your reply, I will raise an issue with typewriter-mode


/vt/ and /e/ archive domain.

https://archive.palanq.win/ has an api as it's running FoolFuuka :)
Thanks for adding all the domains btw! Love ya <3
Could you possibly add a way to continuously monitor the latest threads?
Not sure logistically how that'd work but if it's possible that'd be awesome :)

Another request for the script is to remove the first 65 characters of the 8chan files it downloads due to the incredibly long file names.

/vt/ can be archived right now, kind of, because I scrape warosu.org. I hadn't thought about /e/ though, I'll have to test on that one
To "monitor" all the threads I use the recently added -n flag with hdg_archive.py to scrape the last N threads. So for 4 threads you add -n4. Then I run that in a loop for all the boards I want to archive every half hour or so.

And thanks for the palanq.win link btw, had no idea that existed

And thanks for the palanq.win link btw, had no idea that existed

No problem! Really appreciate your script :)
Glad I could tell you about it :)

Is there any chance you could add an option to sanitize the file names to exclude emojis/non utf8 characters?
Some photo indexing software can't handle emojis in filenames :/

So I managed to add support for /e/ but it seems like FoolFuuka is ignoring me when I pass it the ?board=e parameter, even though &text=voldy works fine. So I just skip any posts that don't match the passed board. According to this it's supposed to be the board shortname. Would you have any information about this, am I do it wrong?

Is there any chance you could add an option to sanitize the file names to exclude emojis/non utf8 characters?
Yeah I can do that.

So I managed to add support for /e/ but it seems like FoolFuuka is ignoring me when I pass it the ?board=e parameter, even though &text=voldy works fine. So I just skip any posts that don't match the passed board. According to this it's supposed to be the board shortname. Would you have any information about this, am I doing it wrong?

Is there any chance you could add an option to sanitize the file names to exclude emojis/non utf8 characters?

Yeah I can do that.

It's because their recent posts don't actually have "voldy" in it at all lmfao.
I was using the https://rentry.org/sdmodels url as the search term. Going way back on the archive to find the possible text in the threads also led me to seeing that they didn't include voldy but linked to other github repos

do you know why this isn't working on my linux install? @space-nuko
I'm getting this error
Saving: \mnt\sd\Archive\archiveofsins\h\7185614\1676419900217452_look how hard i can cum.png Saving: \mnt\sd\Archive\archiveofsins\h\7185614\1676419989324428_03984-778644185-15_Feb_2023-d5f8393fea.png Saving: \mnt\sd\Archive\archiveofsins\h\7185614\catbox\4btpon.png Saving: \mnt\sd\Archive\archiveofsins\h\7185614\1676420235241543_tally test.jpg Traceback (most recent call last):   File "/mnt/sd/Archive/#sd-webui-utilities/hdg_archive.py", line 759, in <module>     for res in p.imap_unordered(worker, links):   File "/usr/lib/python3.10/multiprocessing/pool.py", line 873, in next     raise value   File "/usr/lib/python3.10/multiprocessing/pool.py", line 125, in worker     result = (True, func(*args, **kwds))   File "/mnt/sd/Archive/#sd-webui-utilities/hdg_archive.py", line 713, in worker     save_link(basepath, url, real_name, mtime)   File "/mnt/sd/Archive/#sd-webui-utilities/hdg_archive.py", line 691, in save_link     os.makedirs(os.path.dirname(path), exist_ok=True)   File "/usr/lib/python3.10/os.py", line 225, in makedirs     mkdir(name, mode) FileNotFoundError: [Errno 2] No such file or directory: ''

Almost certainly because of this fn:
https://github.com/space-nuko/sd-webui-utilities/blob/fbce11d401600ba0beb5639e6d9f77915aa0a5b7/hdg_archive.py#L660-L668
because it sanitizes filepaths based on Windows formatting
I think I'll need a better filepath sanitize function if Linux is being used

What OS are you running? Windows? @space-nuko

Yeah I run Windows only right now so I can't test unfortunately, I can just spitball things until they work for you maybe

Could you try https://github.com/space-nuko/sd-webui-utilities/commit/30c2341ef241f140ac77cf002d2d1e90a4a2619a maybe? Made it so it should sanitize Linux pathnames correctly

Could you try 30c2341 maybe? Made it so it should sanitize Linux pathnames correctly

That worked!
Thanks!

Do you think you could also add the creation date to when the photo was posted? Not just the date modified?

I already set the date modified on the downloaded files to the time the post was uploaded on *chan, is that what you mean?

It was because they changed the name of the cookie that validates you've clicked through their disclaimer, should be fixed in 3545995

Is there a good way to import these all into hydrus?
Would love to see a way to implement the script so that you can import all catbox images with metadata as well as being able to import all the files you already downloaded. Maybe parsing the filenames as well? :D


Adding code-of-conduct in repo.

why code-of-conduct:- We propose adding a comprehensive Code of Conduct to our repository to ensure
a safe, respectful, and inclusive environment for all contributors and users. This code will
serve as a guideline for behavior, promoting diversity, reducing conflicts, and attracting a
wider range of perspectives.
Issue type

[✅] Docs

@thinkwee  If you were planning this issue kindly assign it to me! I would love to work on it ! Thank you !

I am not an admin of the repo.
Regards.

Thank you very much for your proposal. We will consider your pull request.


add gif images to light source docs

Change List

add gif images

Coverage remained the same at 62.427% when pulling a2ceba2ceb30cde1a3cb3f35a5d807b717eaf71e on improve-doc into 647e4c5d8998528e73eed7ab7eb20c7e043c252b on master.


Feature Request: rules for codeformatting

Hi,
first of all, thanks for the project! 🍻
Is it possible to add some formatting/code style rules? EditorConfig can fix many of these issue, but it would be better if the linter would throw an error if there are

trailing spaces
no newlines at EOF
consecutive blank lines
the indention level is wrong

Do you think this is possible to add?

Hey @timbru31 I can add that, probably this weekend, will keep you posted. Also feel free to make prs if you have any suggested changes/additions ^ ^

I added rules for all of these with version 0.1.0. Take a look at the documentation of how you configure the rules (https://github.com/vsiakka/gherkin-lint#rule-configuration) because a couple of them give you more options than most rules.
Let me know if you have any issues :)

Awesome, I'll give it a try later! 🍻
Sent from my iPhone

On 06 Jun 2016, at 08:12, vsiakka<EMAIL_ADDRESS>wrote:
I added rules for all of these with version 0.1.0. Take a look at the documentation of how you configure the rules (https://github.com/vsiakka/gherkin-lint#rule-configuration) because a couple of them give you more options than most rules.
Let me know if you have any issues :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Issue 45 | Update Dockerfile to fix pcre compilation error

Change base of Docker image from CentOS 7 to Debian Bookworm as CentOS is EOL'd.

Include automake package to fix error compiling the pcre library.

Closed in favor of #53


low, noisy audio v2

I have dropped v2 in the SG-1000 console which is effectively the same as an SC-3000 computer. Things are working well for the most part. For audio, this machine has a sn76489. When i jumper to pin 7 which is audio out the signal is very noisy and low. I saw some other comments for this console that suggested putting a cap between pin 7 and ground. That was for standard RCA jack modification. Any thoughts on how I might approach fixing sound for the TN-VPD v2?

Look my latest v3 board (Tang 20K only). There I added a trimpot for voltage divider and also a 10uF cap in series. Sorry I moved into Tang 20K and I don't want to make new 9K boards. You can try adding this mod. Other thing to help would be connecting the audio straight to the board instead of the socket and removing the pin on the header to avoid creating an antenna and using a shielded cable to pull the audio from the system. You can just start modding by adding a 10uF in series, see what you get but the voltage divider really helps.

Put the capacitors close to the ADC not on the source
On Mon, Aug 28, 2023 at 8:55 AM dantaipan @.***> wrote:

Hmm. I was trying something similar to your last suggestion with a fixed
3k3 resistor like so but still very noisy. I can try something variable or
keep playing around with it. Thanks.
[image: image]
https://user-images.githubusercontent.com/642598/263755738-4d5c5326-6621-4189-9ddd-278836f11683.png
—
Reply to this email directly, view it on GitHub
https://github.com/lfantoniosi/tn_vdp/issues/16#issuecomment-1695941101,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCJWUZOAQRUGYZGRJQZGR3XXS5PRANCNFSM6AAAAAA4BOHAMI
.
You are receiving this because you commented.Message ID:
@.***>

And should be a voltage dividee not a resistor in series
On Mon, Aug 28, 2023 at 8:55 AM dantaipan @.***> wrote:

Hmm. I was trying something similar to your last suggestion with a fixed
3k3 resistor like so but still very noisy. I can try something variable or
keep playing around with it. Thanks.
[image: image]
https://user-images.githubusercontent.com/642598/263755738-4d5c5326-6621-4189-9ddd-278836f11683.png
—
Reply to this email directly, view it on GitHub
https://github.com/lfantoniosi/tn_vdp/issues/16#issuecomment-1695941101,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCJWUZOAQRUGYZGRJQZGR3XXS5PRANCNFSM6AAAAAA4BOHAMI
.
You are receiving this because you commented.Message ID:
@.***>

Thanks, I think I get it.

Like so?

Yes
On Mon, Aug 28, 2023 at 9:59 AM dantaipan @.***> wrote:

Like so?
[image: tn-vdp audio]
https://user-images.githubusercontent.com/642598/263770055-a557d10d-e445-4336-bcc6-8abc208d59dc.png
—
Reply to this email directly, view it on GitHub
https://github.com/lfantoniosi/tn_vdp/issues/16#issuecomment-1696033776,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCJWU2Z55MWL4MUCBOG3KDXXTE57ANCNFSM6AAAAAA4BOHAMI
.
You are receiving this because you commented.Message ID:
@.***>

It is not pretty but this did significantly approve the audio. Still a certain amount of hiss even when tuned but it did remove a lot and helped with volume.


Timed tasks trigger repeated executions, 2 times at the same time,why??

Version: <IP_ADDRESS>
Timed tasks trigger repeated executions, 2 times at the same time,why??
Is it possible to cause this problem if server cluster time synchronization and time rewind?？？？

quartz will execute the job  server time rewind, even if the job has finished at the last same time

Version: <IP_ADDRESS>
Timed tasks trigger repeated executions, 2 times at the same time,why??
Is it possible to cause this problem if server cluster time synchronization and time rewind?？？？

quartz will execute the job server time rewind, even if the job has finished at the last same time


Whitespaces changes not ignored in Merge Conflict View

Issue Type: Bug
It seems the case that there is either no option for ignoring whitespace changes for Merge Conflicts. There is one for the Diff Editor, and it is working there. But If you change lets say the line ending sequence of a file and that is a Merge Conflict or there is another merge conflict in the file, the whole file lights up as a change also if you only want to fix the merge conflict that is a non-whitespace change.
VS Code version: Code 1.57.0 (b4c1bd0a9b03c749ea011b06c6d2676c8091a70c, 2021-06-09T17:18:42.354Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Pentium(R) Gold G5400 CPU @ 3.70GHz (4 x 3696)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.89GB (0.89GB free)

Process Argv
--crash-reporter-id 2ecdc59c-7d55-48a0-beaf-451bf71a6cf4

Screen Reader
no

VM
0%

Extensions (48)

Extension
Author (truncated)
Version

better-comments
aar
2.1.0

jenkins-status
ale
4.2.1

conflict-squeezer
ang
1.0.1

vscode-apollo
apo
1.19.4

atlascode
atl
2.9.1

cyberpunk-2020
Car
0.1.4

path-intellisense
chr
2.3.0

vscode-jq-playground
dav
4.2.0

vscode-eslint
dba
2.1.23

gitlens
eam
11.6.0

prettier-vscode
esb
8.0.1

vscode-firefox-debug
fir
2.9.4

linter
fna
0.0.4

auto-close-tag
for
0.5.11

auto-rename-tag
for
0.1.7

vuetify-snippets
Gai
0.1.2

gitlab-workflow
Git
3.26.0

vue-snippets
hol
1.0.4

rest-client
hum
0.24.5

output-colorizer
IBM
0.1.2

vscode-postfix-ts
ipa
1.9.4

jenkins-pipeline-linter-connector
jan
1.2.0

jq-syntax-highlighting
jq-
0.0.2

vscode-codemetrics
kis
1.24.0

inline-parameters
lia
0.2.1

codeacejumper
luc
3.3.2

dotenv
mik
1.0.1

vscode-scss
mrm
0.10.0

remote-ssh
ms-
0.65.7

remote-ssh-edit
ms-
0.65.7

remote-wsl
ms-
0.56.5

test-adapter-converter
ms-
0.0.10

vsliveshare
ms-
1.0.4580

vsliveshare-audio
ms-
0.1.91

gremlins
nho
0.26.0

angular-console
nrw
17.6.1

vetur
oct
0.34.1

vscode-versionlens
pfl
1.0.9

polacode
pnp
0.3.4

vuetify2-jest-snippets
pra
0.0.2

jenkinsfile
rec
1.2.0

vscode-yaml
red
0.21.1

vue-vscode-snippets
sdr
2.2.1

jenkins-jack
tab
1.1.5

vscode-conventional-commits
viv
1.21.3

vuetify-vscode
vue
0.2.0

quokka-vscode
Wal
1.0.386

wallaby-vscode
Wal
1.0.298

(6 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt551cf:30311713
vspre833:30321513
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscus158:30321503
pythonvsuse255:30335481
vscod805:30301674
pythonvspyt200cf:30331938
vscextlang:30333561
binariesv615:30325510
vsccppwt:30329788
bridge0708:30335490

The merge conflict is given as textual markers in the file by Git. So it's not VS Code computing the diff.
https://stackoverflow.com/questions/9776527/merging-without-whitespace-conflicts might be of help.

@eamodio Is there a way to add options to the Git merge commands triggered through the UI?

There isn't any option for adding flags to the merge command. You might be able to get the desired result by setting the following git config: https://git-scm.com/docs/git-config#Documentation/git-config.txt-applyignoreWhitespace but it would always apply which may not be desired.


Convenience function for flashing NSErrors

Related to #329
Here is the code I use to flash an NSError to the screen. Perhaps this may be added as a class function.
extension MBProgressHUD {
    func flashError(error: NSError) {
        let window = UIApplication.sharedApplication().keyWindow
        MBProgressHUD.hideAllHUDsForView(window, animated: false)

        let hud = MBProgressHUD(forView: window)
        hud.mode = .CustomView
        hud.removeFromSuperViewOnHide = true

        let view: UITextView = UITextView(frame: CGRectMake(0, 0, 200, 200))
        view.text = error.localizedDescription
        view.font = hud.labelFont
        view.textColor = UIColor.whiteColor()
        view.backgroundColor = UIColor.clearColor()
        view.sizeToFit()

        hud.customView = view
        hud.show(true)
        hud.hide(true, afterDelay: 1.2)
    }
}

#329 has been addressed.


WIP: Mattermost Load Test Install Guide

Working with Josh to draft Mattermost Load Test Guide
Starting with plain-vanilla Ubuntu install instructions
Josh helping draft install steps for Mattermost Load Test, and we'll test, review and update

Goal is clear, reproducible set up on plain vanilla Ubuntu and we'll get more fancy in future guides.

Closing as this is replaced by https://github.com/mattermost/mattermost-load-test/pull/11


Allow stoping admission from a specific LocalQueue.

What type of PR is this?
/kind feature

What this PR does / why we need it:
To allow stoping admission from a specific LocalQueue.
Which issue(s) this PR fixes:

Fixes #2109
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Allow stoping admission from a specific LocalQueue.

/assign

/assign @trasc

/lgtm

/cc @mwielgus
/cc @alculquicondor

/assign @alculquicondor

/hold
Let's not merge this until we have released 0.7
Please work on other commands for now.

/unhold

/lgtm
/approve


Panic in GetBaseColor()

Description
Alternative XLSX editors can make Excelize panic on styles loading.

Steps to reproduce the issue:

Save file with some cell background and text colors changed in alternative client "R7 Office"
Call GetStyle(cellStyleID)

Describe the results you received:
Get "invalid memory address or nil pointer dereference" panic
Describe the results you expected:
code continues to modify and apply new style
Output of go version:
go version go1.19.13 linux/amd64

Excelize version or commit ID:
github.com/xuri/excelize/v2 v2.8.1

Environment details (OS, Microsoft Excel™ version, physical, etc.):

Thanks for your issue. This duplicated with #1963, this issue has been fixed in the master branch, please upgrade to the master branch code by go get -u github.com/xuri/excelize/v2@master, and this patch will be released in the next version.

I'll close this, if you have any questions, please let me know, and you can reopen this anytime.


Add sendmail SMTP configuration option

Summary

Ticket Link

Hello @bbodenmiller,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@coltoneshaw - Are you open to reviewing this documentation PR for technical accuracy?

Thanks, @coltoneshaw. Great feedback! I agree with incorporating this new content into the bullet point that reads "If you don't have an SMTP service, here are simple instructions..."
@bbodenmiller - Are you open to updating your PR with this feedback? If you'd prefer, I'm can help complete this update.

Might get done faster if you make the tweak.
On Wed, Jun 30, 2021, 6:47 AM Carrie Warner (Mattermost) <
@.***> wrote:

Thanks, @coltoneshaw https://github.com/coltoneshaw. Great feedback! I
agree with incorporating this new content into the bullet point that reads
"If you don't have an SMTP service, here are simple instructions..."
@bbodenmiller https://github.com/bbodenmiller - Are you open to
updating your PR with this feedback? If you'd prefer, I'm can help complete
this update.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mattermost/docs/pull/4682#issuecomment-871419693, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJDGTDD4IRKQJ3T4WSPWDTTVMN7BANCNFSM47PPX6YA
.

@cwarnermm / @bbodenmiller - I adjusted the requested structure. Let me know if it flows okay.


Server authoritative bytes data gets corrupted on TypeScript

Description
When transferring bytes data into a TypeScript server authoritative match and relaying it back the bytes data gets corrupted on the process
Steps to Reproduce
Send bytes data from Unity.
nakamaManager.Socket.SendMatchStateAsync(match.Id, (long)code, bytes);
Relay the message back to unity from the server.
dispatcher.broadcastMessage(message.opCode, message.data, null, message.sender);
Read bytes data received in Unity.
Expected Result
The byte data received in Unity should be exactly the same as the sent data
Actual Result
I sent this from unity: (Bytes to string with a space between each one)
12 0 0 0 8 0 16 0 14 0 4 0 8 0 0 0 253 96 66 65 142 250 66 191 0 0 20 0
I received this after relaying the message: (Bytes to string with a space between each one)
12 0 0 0 8 0 16 0 14 0 4 0 8 0 0 0 239 191 189 96 66 65 239 191 189 239 191 189 66 239 191 189 0 0 20 0
Context
Unity + TypeScript server authoritative
Your Environment
Nakama 3.4.0-r1
Unity Nakama 3.0.0 package

@LucasGaspar what's the reason you're sending raw bytes in the messages? The JS runtime expects an utf-8 string for the messages, so the issue could be that the bytes are wrongly decoded as such if the payload is not utf-8.

@sesposito I wanted to use flatbuffers

Thank you a lot @sesposito 😄


Remove Gemfury API key

I don't think we publish anything to Gemfury any more, so there's no point in having an API key on the CI machines.

There are a couple of things still in gemfury I think.  I'll just check.

I don't have Gemfury access, so I can't check for you, I'm afraid @alexmuller.

Yep, there are a few that are still in use - a modified capistrano strategy, and the mirroring related ones.
Details here: https://docs.google.com/a/digital.cabinet-office.gov.uk/spreadsheets/d/1D0TrBlMFuC-eOJy8YaFUaaQnhVpHsx5RrNSo1hHc73Q/edit?usp=sharing

@alext - who still has Gemfury access?


Long time evaluate EACH helper

This is example.
console.time('aaa')
bbb={}; //This string is optimal
bbb[100000000]=1
Handlebars.compile("{{#each products as |p pid|}}111{{/each}}")({products:bbb})
console.timeEnd('aaa');
console.time('aaa')
bbb=[];  //This string is NOT optimal
bbb[100000000]=1
Handlebars.compile("{{#each products as |p pid|}}111{{/each}}")({products:bbb})
console.timeEnd('aaa');

If have created a jsfiddle for your second example.
As you can see, Handlebars iterates the whole array, also displaying the "undefined" elements.
With bbb[100000000], these are a lot of elements and a long runtime can be expected.
The behavior of Array.prototype.forEach() on arrays is different.
The Handlebars-behavior is what I would have expected, and I don't think this will be changed without a good reason. (Although I'm not a collaborator of the project, so it would not be my decision).

@nknapp is right. This is expected, given JavaScript.


ctx.forwardMessage 'optional' options object causing typescript error 2345

SOLVED: https://github.com/grammyjs/grammY/issues/270#issuecomment-1228709629
Reproduction
const targetChatId = 123;
				const fromChatId = 456;
				const message_id = 789

				ctx.forwardMessage(targetChatId, {
					from_chat_id: fromChatId, // from_chat_id does not exist 'Other<"forwardMessage, "chat_id" | "from_chat_id" | "message_id">'
					message_id
				})

Workaround
bot.api.forwardMessage(targetChatId, fromChatId, message_id); // QED.

Notes

Using latest versions of all grammY packages.
from_chat_id and message_id are required fields, even though I am passing them as an 'optional' object. I could be misusing it.
ctx.reply has an almost identical type definition format, triggering autocomplete beautifully. Using the options object for ctx.forwardMessage however fails show anything.

Solution
ctx.forwardMessage, much like ctx.reply is a shortcut that is bound to the current message i.e. its from_chat_id and message_id are filled in for you. All that's required is the target chatId. Neat!
To define other values, use ctx.api.forwardMessage. Noice.
Thanks to @Loskir for the rapido response.

Also to be found at 65https://grammy.dev/guide/context.html#available-actions
Do you think there's a better way to explain it, so that this becomes easier to understand?

@KnorpelSenf I have a suggestion.
Currently, the snippet in the link you shared is like,
bot.on("message", async (ctx) => {
  await ctx.reply("I got your message!");
});

// Or, even shorter:
bot.on("message", (ctx) => ctx.reply("Gotcha!"));

A little below under "Telegram Reply Feature", there is this text:

Naturally, every other method on ctx.api has a shortcut with the correct pre-filled values, such as ctx.replyWithPhoto to reply with a photo, or ctx.exportChatInviteLink to get an invite link for the respective chat.

My suggestion is to showcase a few more common methods such as forwardMessage, answerCallbackQuery, replyWithPhoto, exportChatInviteLink (because we're talking about it above) etc.
Even better, we could also do a ctx.api.forwardMessage vs. ctx.forwardMessage kinda thing for those methods as  etc.
ctx.api.forwardMessage(
  chat_id,
  from_chat_id,
  message_id,
);

// shorter, from_chat_id and message_id are pre-filled:
ctx.forwardMessage(chat_id);

What do you think?

Sure, that'd be simple to do.
Was this the reason you opened this issue? In other words, did you read the respective section but didn't relate it to your case?
I'm thinking, if you didn't see the part of the docs at all, then the problem isn't that it's not clear enough, but rather that it's not easy enough to find. Hence, we should look into ways to increase the visibility, such as

linking it from more places in the docs
linking it from JSDoc
describe it in other places in short

@KnorpelSenf I had seen this section, albeit only during my initial exploration of grammY. Never been back there since, as ctx.reply was sufficient and intuitive for my then level. I spent a lot of time reading and carefully re-reading this section, as that was what gave me the original 'aha' moment for ctx. After reading the Context/AvailableActions section, I think I'd just assumed that chatId would be abstracted away in nearly all cases. Today's issue was that I didn't expect ctx to be that helpful i.e. taking care of from_chat_id and message_id too! I could only suggest a gentle teaser of ctx's power, similar to @dcdunkan 's suggestion of succinct comparisons, as this may have induced an even grander 'aha' and built more anticipation for the subsequent Context section.
With regards to clarity/findability - a slight boost in both would have alleviated my issues. Regarding findability, I had literally Cmd+K searched forwardMessage in the vain hope of finding an example; all it would have taken is one mention of a 'best practice' with ctx, or a cheatsheet of sorts, that would have solved my issue. Regarding clarity, a more explicit reason than 'alias' to use ctx, such as the proposition of saving time and writing cleaner code, would have also led me sooner to the possibility that I may have been overcomplicating things!
Once I had started using the plugins intensively and began getting creative with grammY's capabilities, revisiting the Context section simply didn't cross my mind; I thought I had understood ctx for the fantastically intuitive multitool that it was. However, when a roadblock was encountered, the instinct was to search directly for the 'more advanced' term forwardMessage than for the more basic one 'Context'. Perhaps ctx's conveniences are a bit too seamless?
The documentation is quite possibly the clearest and most proactively helpful I have read, so it really does take a short essay to express the nuanced context of the problem I faced today. I've enjoyed every minute of using grammY and look forward to contributing more, if only in the form of documenting solutions on GitHub Issues.

Alright, then I think adding a few more examples is the right way to go. Do you feel like working on this?

I had literally Cmd+K searched forwardMessage in the vain hope of finding an example

We cannot add examples for all API methods, that's too much work to maintain. It would be awesome to be able to crawl https://doc.deno.land/https://deno.land/x/grammy/mod.ts and add the search results to the index, but I am afraid that Algolia can only crawl one page in the free OSS version. We may be able to do some manual work and add entries for the API to the index, but until then we provide https://grammy.dev/guide/#about-the-documentation which explains where to look.

Perhaps ctx's conveniences are a bit too seamless?

They will be reworked for 2.0 in the distant future, you are more than welcome to participate in this process!

I understand; thanks for the consideration.
I'd be happy to give it a shot. Wednesday will have time.
I'll keep watch in the issues and group chat for the rework.

Great to hear!


[hotfix] forgot password should not give unnecessary info

Purpose
Requests for password resets of email accounts that had not been registered were informing the requestee that they didn't exist. Generally these should fail silently so we don't inadvertently leak information to a malicious party.
Changes
Requester shall see the same status message informing them an email has been sent with reset information regarding the account in question regardless of whether that email is registered or not.
Side Effects
This could confuse legitimate users that type in the wrong email address and don't read the status message.
Closes-Issue: #2355

ping @lbanner


feat-handle-updateToken

preview

This one took me a while to dig through... But 👍 lgtm!

uff, finnaly :smile: :dancers:


iot-platform-deployment-scenarios.md: fix typos

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


Redis asyncio testing

This PR adds functionality to use asyncio as part of Redis

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop-redis-addons@f45335a). Click here to learn what that means.
The diff coverage is n/a.

@@                   Coverage Diff                   @@
##             develop-redis-addons     #881   +/-   ##
=======================================================
  Coverage                        ?   81.37%
=======================================================
  Files                           ?      185
  Lines                           ?    18506
  Branches                        ?     3203
=======================================================
  Hits                            ?    15060
  Misses                          ?     2563
  Partials                        ?      883

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


Numerous documentation tweaks and improvements

Updates to various README files - in part inspired by @jamuir's experience described in https://github.com/openssl/project/issues/253

CI is relevant

All feedback addressed. Please take another look.

Latest round of feedback addressed. Please take another look.

This pull request is ready to merge

Merged to master and 3.2. Thank you.


Update OnboardingView DocC - Version 2 With Signed Commits

Update OnboardingView DocC - Version 2 With Signed Commits
:recycle: Current situation & Problem
I noticed that the documentation for the initializers and overall descriptions of this screen were a bit unclear vague when I was trying to use them.
All of these changes were copied over from my previous pull request for signed commits.
:gear: Release Notes
N/A
:books: Documentation

I added documentation to provide a coding example of the initializer of OnboardingView, as well as a more detailed explanation.
Also added an example fo the custom initializer

:white_check_mark: Testing
Please ensure that the PR meets the testing requirements set by CodeCov and that new functionality is appropriately tested.
This section describes important information about the tests and why some elements might not be testable.
:pencil: Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[x] I agree to follow the Code of Conduct and Contributing Guidelines.

@philippzagar - Here is my updated pull request! I am unable to request a review from you, but if you would be able to review it that would be great! I have copied over all of the changes that we discussed in the previous PR that had been approved with no other differences.


Add endswith function #7

...I'm so sorry.  I was way too eager in pushing commits here.

No worries, looks good! Thank you very much.

Oops, I forgot to add it to the README!

Don't worry. There should be an automated way to do this, anyway (see #10).


Installation fails on firefox 52.0

I can't install add-on on Firefox 52.0, also can't install on Basilisk browser (updated Firefox)
I receive an error:
Installation aborted because the add-on appears to be corrupt.
Manually installation by downloading file doesn't work either.

@ewuc are you installing from https://addons.mozilla.org/firefox/addon/smart-prevent-duplicate-tabs/?

@ewuc Basilisk uses XUL and don`t supports WebExtensions, see details in: https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/Browser_compatibility_for_manifest.json

@ewuc tested on Firefox 52.0, installing:

About window:


Screen Recoding

🐛 Bug Report
I am trying to implement screen recording but NodeModules object is coming blank
To Reproduce
import {NativeModules} from 'react-native';
const { RecorderManager } = NativeModules;
console.log(NativeModules) - object {} (i.e. blank object)
Expected Behavior
NativeModules object should have RecorderManager.

^ In case you're wondering why the bot added that, it's due to the missing Environment section. Can you edit that in? You can run react-native info to get the information we need.

This looks like a bug with your installation of https://github.com/ycswaves/react-native-screen-recorder


create trouble widget

Codecov Report

Merging #65 (c1dfae3) into main (b37860e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #65   +/-   ##
=========================================
  Coverage     33.75%   33.75%
  Complexity      140      140
=========================================
  Files            67       67
  Lines           862      862
  Branches         42       42
=========================================
  Hits            291      291
  Misses          557      557
  Partials         14       14

Flag
Coverage Δ

unittests
33.75% <ø> (ø)

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

Continue to review full report at Codecov.

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


Point based image registration

Hi,
I am working on a multi-modal affine registration task which is too difficult. To help the registration, I am defining corresponding points between moving and fix image.
The images are Nifti format volume and from each one I am selecting 90 corresponding points using BrainSuite. The format of text file for points is similar to following:
point
90
137 141 564
137 141 563
137 141 562
...
I have 2 separate .txt for to represent points in moving and fixed image volume. These points are mostly samples from a small central region of the fixed and moving image. However, including fixed and moving points seems to give unexpected result. I tried adjusting weights but still it does not work.
(Metric0Weight 0.2)
(Metric1Weight 0.8)
Since these points are coordinates from BrainSuite, I even tried "index" instead  of "point" in the point text file and that even did not help.
I am using Elastix framework on Windows 10. Can you please let me know how to solve this issue? Also please let me know if there is any example for point-based affine registration. I was able to find example for SimpleElastix but not for Elastix. Also, the manual section for point based selection was informative but very short.
Is there any way to do affine registration in Elastix only based on selected corresponding points without using any similarity matrix? (I tried achieving this but setting Metric0Weight=0 and Metric1Weight=1 but did not seem to work!).
Thank you for your help!

Hi @maz369,
Could you share you command line call and your point sets and images, so I could try to reproduce your problem?
If you're using point sets to aid your registration, be aware that the point sets should be in world coordinates if you use "point" in your point set file. If you use "index" in the point set file, elastix uses the coordinates relative to the fixed image origin.
And setting " Metric0Weight=0 and Metric1Weight=1 ", should be the correct way of doing registration without using any similarity matrix in elastix.

Hi @maz369 , thanks for raising the issue, I'm closing it for now. Feel free to reopen it, when you find that this issue is not fixed in the newest release of elastix.


Use HeroIcons package instead of creating icons ourselves

Replace icon usage with the icon package. Might have to do a little magic with the imports because the Icon component is doing some stuff with sizing and additional classes that might not work with the icons right out of the box but who knows 🤷♀ we'll find out!
https://github.com/tailwindlabs/heroicons#react

This might be something to put lower on the backlog if Josh creates icons to replace the current ones.


Adds helper method to remove node lists

Adds helper method to remove node lists, which also reduces the jshint
complexity of the _renderMap method.

@monfresh Please merge if https://github.com/smcgov/SMC-Connect/commit/a71c892416fd8def9e2b9dc3f6a590efae5026ed looks good.

https://github.com/smcgov/SMC-Connect/commit/a71c892416fd8def9e2b9dc3f6a590efae5026ed merged in https://github.com/smcgov/SMC-Connect/pull/67


Consider to use an icon without context-fill

https://addons.mozilla.org/zh-TW/firefox/addon/image-picka/reviews/

toolbar button is not displayed in FF 52 ESR/Win7.
please fix it.
來自 circcc，2 天前

Added in 0.6.0.


project_docs, project_doc_sections in data | marineenergy.app - Google Sheet

Transfer:

from: documents data | marineenergy.app - Google Sheet

project
doc NAME
ATTACHMENT NAME
url

to: unique sheets that use drop-downs to get at project_docs, project_doc_sections

and consume into db with update_ferc_docs() in /scripts/update.R

Hi @bbest, it looks like I need access to the data | marineenergy.app - Google Sheet - either<EMAIL_ADDRESS>or<EMAIL_ADDRESS>should work. Thanks!

Ah whoops, I should've done that already. Ok, just gave you access :)

No worries, just got it! @bbest

Ok, looks like you got this to work well in Google Sheets but the multiple levels of hierarchy seem too complicated to handle with its data validation:

projects

documents

attachments (aka sections)

Ideally, something like Access subdatasheet would be available in Shiny DT::datatable(), eg Access 2016 Tables: Subdatasheets - YouTube @ 8:20:

But here's what we found:

Extending DT child rows example
most promising, hopefully the child table could also be editable
Child Rows
javascript - Parent/Child Rows in R shiny Package - Stack Overflow
R Shiny DT Child Row Selection and Info · Issue #660 · rstudio/DT
Shiny DT Child Row Selection and Info Issue · Issue #659 · rstudio/DT

This solution seems to work :) Manually edit child row in shiny datatable - Stack Overflow:

See DTedit-subdatasheet/test.R

Hi @cdobbelaere, here also is a more thorough explanation of this NestedData() approach:

Child tables with DT, editing, exporting | R-bloggers

Thanks @bbest! Very helpful.

Actually @cdobbelaere, I'm thinking we could handle all of this more simply with selectizeInput(..., create = T) inside a renderUI() and some observe() functions to detect a new list item plus updateSelectizeInput(). See:  Shiny - Using selectize input:
# allow creation of new items in the drop-down list
selectizeInput(
  'foo', label = NULL, choices = state.name,
  options = list(create = TRUE)
)

@bbest - Sounds like a good plan of action - will look into it right now.
Are we thinking that we'd update the project choices (via create = TRUE) for the DTedit ferc docs table as a whole, or would this be for a separate tab with the docs/sections/URLs corresponding to each project?
I was initially using this approach for updating the project choices for the DTedit ferc docs table, but the selectizeInput() and selectize.options didn't seem to be working (they were from the DavidPatShuiFong modularized version of DTedit), similarly to how we were having difficulties adding in checkboxInputs.

Hi @bbest, exciting news!! I was finally able to get the correct DavidPatShuiFong version of DTedit installed, and we now have checkboxInputs functioning! Now to see if we can get the selectize.options from this particular commit in order.

Screenshots showing

checkboxInput functionality and

ability to create a new project within edit panel

We can use update.R's update_ferc_docs() and update the GoogleSheet data | marineenergy.app - Google Sheets with googlesheets4::sheet_write().


[Swift] wait for DLO segments to show up when Close()ing the writer

Not just when Commit()ing the result. This fixes some errors I observed when the layer (i.e. the DLO) is Stat()ted immediately after closing, and reports the wrong file size because the container listing is not yet up-to-date (usually recognized by the error message "digest invalid: provided digest did not match uploaded content").
Related to #1013 and maybe to #1539 aka docker/docker#21290.

Current coverage is 51.18%

Merging #1650 into master will decrease coverage by -6.76% as of 9a67520

@@            master   #1650   diff @@
======================================
  Files          122     122
  Stmts        10683   10687     +4
  Branches       838     749    -89
  Methods          0       0
======================================
- Hit           6190    5470   -720
+ Partial        838     749    -89
- Missed        3655    4468   +813

Review entire Coverage Diff as of 9a67520

Uncovered Suggestions

+0.30% via .../driver/s3-aws/s3.go#732...763
+0.28% via .../driver/s3-aws/s3.go#268...297
+0.21% via .../driver/s3-aws/s3.go#884...906
See 7 more...

Powered by Codecov. Updated on successful CI builds.

LGTM.  @nevermosby : second opinon please

Bump. I want to extend this pattern to more request types to further stabilize the DLO support, so it'd be nice to have this merged.

Yep, looks good


unplugged device

Noob allert!
Is there a way to asynchronously detect FTDI was unplugged?
Maybe "setEventNotification" can be helpful. Any good examples of usage?

I don't know that it's possible from FTDI library. The OS may generate an event. If you're using Windows, you could try pywin32. Here's a blog I found via web search: http://timgolden.me.uk/python/win32_how_do_i/detect-device-insertion.html

thank you @snmishra for hint


Class-defined block doesn't render when nested.

In Twill 3.0, I have nested blocks. The block I am nesting is defined by class instead of as a view-file inside of views/twill/blocks. Rendering this block works fine on the top-level. But when I try to nest it, it is looking for the wrong view path.
In the frontend I render the nested blocks with {!! $renderData->renderChildren('subblocks') !!}
This works fine for view-file defined blocks, but the class defined block gives this error:
An error occured trying to render:

View [site.blocks.app-related] not found. in (path to project)\vendor\laravel\framework\src\Illuminate\View\FileViewFinder.php

The correct view file path is defined in the render function of the block, but this doesn't seem te be used:
    public function render(): View
    {
        return view('components.twill.blocks.related');
    }

Is this a bug?

This is fixed on the 3.x branch and will be released shortly. You can try it with 3.x-dev as your composer version in the meantime.


Refactor: Created new refactoring classes and functions to clean up the code

In this refactoring I created new classes to remove some responsibilities from 'SpotifyHttpManager' and created a class 'SpotifyHttpManagerExecute' to divide the responsibilities and cleaned up the functions to make the code easier to understand and maintain, I decoupled the Builder class to another file
Please, tell me your feedback on the suggested improvements

Ok, thanks you for this feedabacks <3
I agree with you, i think really will help in terms of the code quality.
But at the moment i don't have much knowledge in Kotlin to help, sorry about that and for now i dont have any other issues related to quality at the moment, i will close this PR

Thank you for all your work on code quality! :rocket:


Update default url for R&R service

I think this is fixed in dev and will be merged in the next release

See https://github.com/watson-developer-cloud/nodejs-wrapper/blob/dev/services/retrieve_and_rank/v1.js


Misleading dates in Region-specific deployments (UK)

General availability for the UK is quoted as "October 2022" - it is not unreasonable to assume that means right now since we are in the month of October.
After purchasing licences and spending significant amounts of time troubleshooting why we can not use voice in our environment I have now been told by support that it is not generally available yet.
The date should be amended to be specific or have an astrix/footnote to say the release is TBC in October.

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

ID: f7db03e4-1bb2-317d-43a4-609149b0bf86
Version Independent ID: 22c180d9-42ee-d114-0791-823dc5804b5c
Content: Supported cloud locations, languages, and locale codes
Content Source: ce/customer-service/voice-channel-region-availability.md
Service: dynamics-365-customerservice
GitHub Login: @neeranelli
Microsoft Alias: nenellim

Thanks for your comment, @Dan-Newton. This issue has been assigned to the doc author for review.

Hello @Dan-Newton,
I am sorry that you are facing issues regarding the availability of the feature in your region. I have shared the issue with the stakeholders and am waiting for their response.
Best regards,
Neeraja

@neeranelli - Apologies, however this is now GA in our region so the documentation is correct. For future feedback however I think it is very confusing using non specific release dates in documentation e.g. "October 2022". In this instance the product was not available GA until mid-October however at the start of October my organisation, our licence supplier and even Microsoft Support initially thought it was available and could not work out why it did not work.
These dates should be specific or simply say 'TBD' etc

Thank you for your feedback, @Dan-Newton. Let me work with the stakeholders and see if we can incorporate it. This might take some time.
If there's no immediate action item, I'd like to close this issue.
Best regards,
Neeraja

@Dan-Newton, We have included exact dates now for the upcoming supported regions. This change will be published in a couple of hours.
Thank you for your help with improving the documentation.
Best regards,
Neeraja


[SQL Lab] Can't scroll in SQL Lab Query History (again)

Can't scroll in SQL Lab query history - the scroll bar is not there.
Expected results
User was previously able to scroll using a scoler on the right side.
Actual results
User can't scroll to see previous query history details
Screenshot

FYI @yousoph

This was resolved last month and now recently the regression happened again: https://github.com/apache/superset/issues/12922

Fixed with: https://github.com/apache/superset/pull/13822


RevEng: Use nameof in data annotations

Currently when creating an ef view caffolding with attributes enabled, all of the attributes are using hard coded strings from the properties names of the related entity.  This is very brittle and can make changes in the entity model extremely difficult and error prone.  The request is to change the code generation logic to output attributes that leverage nameof instead to address this issue.
Further technical details
EF Core version: 2.1.0-preview2-final
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10
IDE: Visual Studio 2017 15.4

Meta question for the team: What is the lowest version of C# we need to support?

C# 6 is my vote

Discussed in triage and decided that we will not do this yet because we still support people using EF Core with .NET Framework on earlier versions of VS.

So this is more of a compiler issue than a framework issue?

Yes, we can't generate nameof() unless everywhere you can use EF Core is able to compile it.

Removing from backlog temporarily to discuss the command line switch.

@ravensorb We discussed this and came to the conclusion that:

It would be good to add a "language version" in parallel with the currently supplied "language".
This should be read from the user's project by default, just like the language is.
Based on this, we can start using features that are supported by the language version being used.

Marking this a up-for-grabs since this should not be too difficult to implement--advice from @bricelam is to "follow the language" code.

@ajcvickers this sounds like an excellent solution! I am looking forward to it :)

I was going to take a look at this but wanted to confirm which code generation we are looking to update.
Running this command:
Scaffold-DbContext "Server=(localdb)\mssqllocaldb;Database=Northwind;Trusted_Connection=True;" Microsoft.EntityFrameworkCore.SqlServer -DataAnnotations -OutputDir Models
This creates models that include relationship properties annotated like so:
        [ForeignKey("CategoryId")]
        [InverseProperty("Products")]
        public virtual Categories Category { get; set; }

Updating this to use nameof, we would want something like:
        [ForeignKey(nameof(CategoryId))]
        [InverseProperty(nameof(Categories.Products))]
        public virtual Categories Category { get; set; }

Is this the right changes we are talking about here? Any other annotations that would need updating?

That sounds right.
We also need to investigate what maximum version of C# we can use. But don’t let that stop you from submitting a PR.

I would love to do this. I'm migrating a webapp that uses EF 6 to EF Core, and the DB and it's edmx were a mess, so we decided to redo the entities via scaffold. We had to rename some stuff and BOM, InverseProperty names are hardcoded :( Wish it had nameof
This would have saved me couple of hours on runtime validation.

@bricelam  Can I do this now, that .NET Standard 2.1 is minimum for EF Core 3.0?

@Muppets already has an open PR. But yes, I think we can move forward with this since C# 8.0 will be the default for projects targeting .NET Standard 2.1 and .NET Core 3.0.

@bricelam Sorry, missed that

@bricelam TableAttribute miss nameof, can you add it?

@Varorbc Personally, I don't think it belongs there since the table name is independent of the type name. Note that the places it was added ([ForeignKey] and [InverseProperty]) refer to CLR type and property names, not table and column names.

One important scenario that would be lost by using nameof in [Table]:
After scaffolding, we want you to easily rename your CLR types but still have your model map to the same database schema. If we used nameof, it would also change the table mapping.


PBSCluster hard codes account string for project

Currently PBSCluster accepts a project code as an argument and uses this with the account string, -A, option
https://github.com/dask/dask-jobqueue/blob/master/dask_jobqueue/pbs.py#L96
PBS Pro has account string (-A) and project (-P) options:
https://github.com/PBSPro/pbspro/blob/master/src/cmds/qsub.c#L892-L898
For my local HPC setup the -P option is used for project based accounting.
It would be preferable to have the ability to pass either account or project as options for PBSCluster and use the appropriate flag. This would break existing configs that assume the -A flag is used.
One possible solutions would be to allow the flags associated with options to be specified in a dask config file.
Would the maintainers be happy to accept a PR with this implemented?

Hi @aidanheerdegen, thanks for sharing this.

Would the maintainers be happy to accept a PR with this implemented?

Certainly!!

This would break existing configs that assume the -A flag is used.

You can mitigate this by using project kwarg as the account variable if no account kwarg is provided. Not sure if it is really needed, or if it can break something else...

One possible solutions would be to allow the flags associated with options to be specified in a dask config file.

Isn't it possible with job-extra kwarg or equivalent config entry?

I am facing a similar problem on a cluster (PBS version 18.2.1) and tried job_extralist = [' -P  XXXXXX'] without success.  For now, I am overriding the line:
https://github.com/dask/dask-jobqueue/blob/bff96a329439c23414bd0cb11c0ee8e67145d96b/dask_jobqueue/pbs.py#L96
with a -P.

job_extralist = [' -P XXXXXX']

There's a problem in jobqueue.dask.org rendering API. The correct argument is job_extra only.
job_extra = ['-P XXXXX']

should do it. Could you try again?

This works.
Thanks :)

We should do this in 0.8.1 has we already prepared a few changes to kwargs.


HDDS-7159. Run integration tests on Java 8

What changes were proposed in this pull request?
Follow-up to #3707: run integration tests on Java 8.  TestSecureOzoneCluster is failing on Java 11.
https://issues.apache.org/jira/browse/HDDS-7159
How was this patch tested?
https://github.com/adoroszlai/hadoop-ozone/runs/7957021440?check_suite_focus=true

Thanks @kaijchen, @kerneltime for the review.


Issue found on page 'Enum Data Type': document enum ordering

It seems that enums implement the > and < operators, however, their behaviour does not seem to be described anywhere.
Are enums compared as strings? Or are they compared based on the order the elements are defined?
Based on my experiments the later seems to be true:
CREATE TYPE bar AS ENUM ('b', 'a')
SELECT 'a'::bar > 'b'::bar

returns
┌───────────────────────────────────────┐
│ (CAST('a' AS bar) > CAST('b' AS bar)) │
│                boolean                │
├───────────────────────────────────────┤
│ true                                  │
└───────────────────────────────────────┘

Would be nice to add that explicitly to the enums documentation page.
Page URL: https://duckdb.org/docs/sql/data_types/enum.html

This is a great point, thanks for raising this issue. A PR is incoming!


More numeric types

Right now, HVM only supports U32. The numeric types above should be supported too. I32 and F32 should be easy to add, since they are unboxed, like U32. I64, U64 and F64 require implementing boxed numbers, since they don't fit inside a 64-bit Lnk (due to the 4-bit pointer), but shouldn't be hard. Discussion on whether we should have unboxed 60-bit variants is valid.

I'd be interested in adding support for at least the unboxed numeric types as this seems like a low hanging fruit. I just wanted to check what the concensus was between implicit/explicit casting and how operators should function in regards to mixed input types. My instinct would just be to cast up to the 'bigger' type but maybe operators should only operate on the same type - combined with explicit casts.
@VictorTaelin what do you think?

Originally posted by @samtoth in https://github.com/Kindelia/HVM/issues/38#issuecomment-1037127813

[ ] unboxed

[ ] integer: I32
[ ] float: F32

[ ] boxed

[ ] integer: U64, I64
[ ] float: F64

F60 and U60 are now merged on Master. I don't think we should have other integer types.

@VictorTaelin Can you comment on / give links to some papers on how to lower signed integers (in higher-level languages) to unsigned integers (in HVM as a backend)? It's not immediately obvious to me.

Hmm perhaps adding Int isn't a bad idea to be fair. Should be very simple to do now (see F60.hvm and U60.hvm).

Do we have f32 and i32 already?

No, I think f32/i32/u32 should just be implemented as a lib, but not too sure about that. Any reason you think these should be native? I believe a minimalist design is important, must have caution not to become a bloated spec

If f32/i32/u32 can be implemented as a lib without performance issues I'm totally fine with it. I was just thinking about use-cases for games where f32's are used heavily. I'd assume we'd want to be able to take advantage of hardware float operations, which I wasn't sure if we could do in a lib or not.

In that case you should just use a F60. Since the pointer size is 64, a F32 would just have 28 empty bits. What we could (should?) do in a future perhaps is have a 32-bit version of HVM?

That's an interesting question. I'm not sure if using 64 bit floats everywhere would have negative impact on performance or not.
Maybe a 32-bit HVM does make sense. I don't know. 🤷♂


Incorrect handling of TLS 1.2 CBC IV field

I apologize if I am mistaken about this but I believe I have found a discrepancy between the behavior of OpenSSL and the TLS 1.2 spec with regard to the encoding of a record when using a block mode cipher. While working on my own TLS library (for fun) I was using curl (version 7.58.0 on ubuntu bionic compiled with OpenSSL 1.1.1) to test and I found something that I had trouble making sense of at first. When as the server, decrypting the first encrypted TLS record sent by a curl client, being the Finished record, the IV field of the GenericBlockCipher struct appears to be included in the portion that is being encrypted by curl (and OpenSSL). If the record decryption doesn't start with the IV field, the Finished record does not decrypt correctly. If the record decryption starts with the IV field, the Finished message data is decrypted correctly with record type 20 and length 12 following the IV field. According to the TLS 1.2 spec, the IV field is not part of the encrypted record for CBC ciphers and the encryption should only include the payload, MAC, padding length, and padding. Here is the definition of the GenericBlockCipher struct from the spec:
struct {
    opaque IV[SecurityParameters.record_iv_length];
    block-ciphered struct {
        opaque content[TLSCompressed.length];
        opaque MAC[SecurityParameters.mac_length];
        uint8 padding[GenericBlockCipher.padding_length];
        uint8 padding_length;
    };
} GenericBlockCipher;

At first, I was discouraged because I thought that there must be some magic that was not in the spec or else I was misinterpreting it. However, I now suspect that the OpenSSL implementation is actually just incorrect since here is the GenericBlockCipher from the TLS 1.1 spec:
block-ciphered struct {
    opaque IV[CipherSpec.block_length];
    opaque content[TLSCompressed.length];
    opaque MAC[CipherSpec.hash_size];
    uint8 padding[GenericBlockCipher.padding_length];
    uint8 padding_length;
} GenericBlockCipher;

Note the difference here that the entire struct is "block-ciphered", including the IV field. This makes me think that the OpenSSL implementation for TLS 1.2 was just inherited from the TLS 1.1 implementation and this change in the spec was overlooked. This would make sense since the section in the TLS 1.2 spec regarding changes from TLS 1.1 does not mention this difference.
I'm no expert on the OpenSSL source code, but reviewing the tls1_enc function in ssl/record/ssl3_record.c seems to confirm that the IV field is written before encryption. I am assuming that tls1_enc is used for both TLS 1.1 and TLS 1.2. I don't see a version of tls1_enc that would be specific to TLS 1.2.
I have tried to duplicate this behavior using s_client, however, I've been unable to get s_client to offer the specific cipher that my library supports for the moment, which is TLS_RSA_WITH_AES_128_CBC_SHA (0x00, 0x2F). I'm initially implementing that cipher because it's relatively simple and because the TLS 1.2 spec explicitly states that all TLS 1.2 implementations must support this cipher.
Assuming the behavior of OpenSSL is actually wrong, I imagine it is too late to change it at this point since I assume that it would be incompatible with other (all?) TLS implementations. It also seems like CBC ciphers are now rarely used. I mostly am just trying to understand if OpenSSL is correct or not or if I am misunderstanding the TLS 1.2 spec.

When as the server, decrypting the first encrypted TLS record sent by a curl client, being the Finished record, the IV field of the GenericBlockCipher struct appears to be included in the portion that is being encrypted by curl (and OpenSSL).

What makes you think we encrypt the IV? This does not make sense since the IV is an input to the encryption process.

My understanding was that the IV used to encrypt the first record is derived from the master secret and the IV in each record is used to encrypt/decrypt the following record. I was mistaken about this. I think I must have gotten this understanding from something I read that was not the TLS 1.2 spec. Indeed, the TLS 1.2 spec says this:
Currently, the client_write_IV and server_write_IV are only generated
for implicit nonce techniques as described in Section 3.2.1 of
[AEAD].

That being said, it seems that encryption must be starting with the IV rather than from the start of the Finished message + MAC + padding. Here is what I am seeing. When receiving an encrypted handshake message following a ChangeCipherSpec message during the initial handshake from the client (curl), the record type is handshake (22) and then length is 64. The negotiated cipher suite is TLS_RSA_WITH_AES_128_CBC_SHA so the encryption block length (and IV length) is 16 bytes and the MAC length is 20 bytes. When beginning decryption at offset 16 (skipping the 16 byte IV field), using the first 16 bytes of the handshake message as the IV, the Finished message does not decrypt correctly since it has random bytes where the handshake type and length should be, though the padding at the end of the final block is correct. When beginning decryption starting at the first byte of the message using the first 16 bytes of the message as the IV, the decrypted data starting at offset 16 appears to be a correctly formed Finished message with type 20 and length 12 followed by 32 (seemingly random) bytes (12 for the handshake hash and 20 for the record MAC) and then the 1-byte padding length field (with value 11), again with correct padding. I have not yet verified that the handshake hash or MAC are correct in this case but I assume thus far that they are. Is decryption supposed to start at the start of the IV? Why would decryption begin with the IV if encryption did not?

My understanding was that the IV used to encrypt the first record is derived from the master secret and the IV in each record is used to encrypt/decrypt the following record.

That's how TLSv1.0 works. TLSv1.1 changed this to have an explicit IV.

When beginning decryption at offset 16 (skipping the 16 byte IV field), using the first 16 bytes of the handshake message as the IV, the Finished message does not decrypt correctly since it has random bytes where the handshake type and length should be, though the padding at the end of the final block is correct. When beginning decryption starting at the first byte of the message using the first 16 bytes of the message as the IV, the decrypted data starting at offset 16 appears to be a correctly formed Finished message with type 20 and length 12 followed by 32 (seemingly random) bytes (12 for the handshake hash and 20 for the record MAC) and then the 1-byte padding length field (with value 11), again with correct padding.

This makes no sense. AES decrypts blocks of 16 bytes at a time. When using CBC mode you xor in an additional 16 bytes of data - which is either an IV, or the ciphertext from the previous block. In both the scenarios you describe the latter blocks should both decrypt the same way since the preceding blocks are the same.
In other words, if you're ciphertext looks like this:
IV: 16-block-IV
Ciphertext: 16-block-CT1, 16-block-CT2, 16-block-CT3 ... 16-block-CTn
Then your output from the decryption should look like this:
Plaintext: 16-block-PT1 16-block-PT2 16-block-PT3 ... 16-block-PTn (including padding)
If instead you feed this in:
IV: 16-block-IV
Ciphertext: 16-block-IV 16-block-CT1, 16-block-CT2, 16-block-CT3 ... 16-block-CTn
Then your output from the decryption should look like this:
Plaintext: 16-block-random-data 16-block-PT1 16-block-PT2 16-block-PT3 ... 16-block-PTn (including padding)
So, the two outputs should be identical except that in the latter case you get 16 bytes of random data (the "decrypted" IV) at the beginning.
The fact that you don't see that suggests there is a problem with the way you are decrypting.

Thinking about it more, it seems likely that you are not passing the correct IV into the process somehow, i.e. doing this:
IV: 16-block-random-data
Ciphertext: 16-block-CT1, 16-block-CT2, 16-block-CT3 ... 16-block-CTn
Will result in this:
Plaintext: 16-block-random-data 16-block-PT2 16-block-PT3 ... 16-block-PTn (including padding)
But this:
IV: 16-block-random-data
Ciphertext: 16-block-IV 16-block-CT1, 16-block-CT2, 16-block-CT3 ... 16-block-CTn
Will still give you:
Plaintext: 16-block-random-data 16-block-PT1 16-block-PT2 16-block-PT3 ... 16-block-PTn (including padding)

I very much appreciate your time spent considering and responding to my problem. Your description of how it should work (which is detailed and well written) matches my expectation. I could not, however, see how your conclusion could be correct since there are only the 3 calls to EVP_DecryptInit_ex(), EVP_DecryptUpdate(), and EVP_DecryptFinal_ex() and the parameters are all fairly straightforward.
However, when trying to come up with an easy to show example using the bytes of an encrypted record coming from curl and openssl, I managed to not only show that it works precisely as you say it should but also that your conclusion was indeed correct. I must admit I'm ashamed to say that I was passing NULL as the pointer to the IV when calling EVP_DecryptInit_ex() because the pointer to the record payload was only being initialized after that and before calling EVP_DecryptUpdate() since I had originally believed that the IV field was not yet needed at the point of calling EVP_DecryptInit_ex(). I'm guessing that the IV used in that case is some default since EVP_DecryptInit_ex() was clearly not dereferencing the NULL pointer I was giving it.
Thanks again! I am reassured that OpenSSL's handling of CBC IV fields is correct.

Great. Glad you got it sorted.

Hello, I apologize if this is not the right place to ask, but I have a question related to what was mentioned above. Apologies also because I am no expert in openssl nor in interpreting rfcS.
I can't see the explicit IV that the client should (?) send over the wire. I coded and configured a dummy aes_128_cbc engine that prints the IV, the key, and some of the first and last blocks processed; plain and cipher texts. I run the following command:
openssl s_client -tls1_2 - debug -connect www.google.com:443 -cipher AES128-SHA -servername www.google.com
In addition to looking at the output of this command, in the debug output of the engine I see the IV with which the cipher was initialized, and when checking with wireshark what I understand would be the GenericBlockCipher, I do not see that it starts with the value of that IV. Instead, I find the result of the encryption of the IV xored with the first block of the plain text. In the following records, I see that the last encrypted block is used as IV.
I would like to clarify that I am not asking about decryption; I understand that block would be discarded, what I can't find is the explicit IV.
Thank you.

@arielm33  Perhaps you could post some of the wireshark output that leaves you confused?
It would be most appropriate to do so as an email to<EMAIL_ADDRESS>though opening a new issue would be okay as well.

Thank you @kaduk
I've added more information and details in #11866


Song Likes and Dislikes displayed as Zero, even on the main bot

Describe the bug
When playing a song and getting the song info, the likes and dislikes are displayed as zero.
Expected behavior
It is supposed to display the actual number of likes and dislikes.
Additional context
It happens on my bot too, not just the DisTube bot, which probably means it's an API issue :>

Due to ytdl-core cannot get this when using embed video.
https://github.com/fent/node-ytdl-core/issues/800

Is there any way I can fix it?

Is there any way I can fix it?

When the above issue closed

Same reason as #21


a small addition to the changed flag

Description
a small addition to the changed flag

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
[ ] Ideally, include a test that fails without this PR but passes with it.
[ ] Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

[ ] Run the tests with pnpm test:ci.

Documentation

[ ] If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

[ ] Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

@sheremet-va Should I fix something else?


Fix Spark/Istio ports

Spark workers need to have the block-manager port exposed.
This PR prefixes all Spark service ports with tcp- to force Istio to use TCP, as well as adds the block-manager port to the Spark worker service.
This PR was successfully tested on a deployment using the following script:
def install_pyspark():
    import importlib.util
    if importlib.util.find_spec('pyspark') is None:
        import subprocess
        print("Installing PySpark...")
        subprocess.run(['pip', 'install', 'pyspark==3.1.1'])
        print("PySpark successfully installed!")

install_pyspark()

import pyspark
sc = pyspark.SparkContext()
words = ['cat', 'elephant', 'rat', 'rat', 'cat']
wordRdd = sc.parallelize(words, 4)
pairRdd = wordRdd.map(lambda word: (word, 1))
counts = pairRdd.reduceByKey(lambda a, b: a + b)
print(counts.collect())

Need to remove driver ports that aren't strictly necessary.

Can we add the Jira (DOM-31740) to the commit summary line? This will link the PR back to the Jira ticket, and it will also automatically creating a link in Jira.

Since this repo is open source, we avoid adding links to Domino-internal stuff, like JIRA or other repos. Happy to link the PR from within the ticket, though!


fix(window.ts): unresponsive window popup message [NFC]

The word no longer seems a bit too definitive, implying that it would not get back to be responsive again, but there is an option of waiting it to be responsive.

Why not 👍


commands.spawn_batch() causes panic

Bevy version
main#348e2a3d
Operating system & version
Arch Linux 5.11.6
What you did
use bevy::prelude::*;

fn main() {
    App::build()
        .add_plugins(DefaultPlugins)
        .add_system(setup.system())
        .run();
}

fn setup(
    mut commands: Commands,
) {
    commands.spawn(());

    let mut tiles = Vec::new();

    for _ in 0..=20 {
        tiles.push(SpriteBundle {
            ..Default::default()
        })
    }

    commands.spawn_batch(tiles);
}

What you expected to happen
No panic
What actually happened
thread 'main' panicked at 'assertion failed: len <= self.capacity()', ~/.cargo/git/checkouts/bevy-f7ffde730c324c74/348e2a3/crates/bevy_ecs/src/storage/blob_vec.rs:120:9
Additional information
Doesn't happen when changing the 20 to any number over 62 or 1

We should ensure that this is fixed as part of #1562. Adding the 0.5 milestone to this as a result.

Looking now. This isn't specific Commands. It behaves the same way for direct World access:
use bevy::prelude::*;

fn main() {
    App::build()
        .add_plugins(DefaultPlugins)
        .add_startup_system(setup.exclusive_system())
        .run();
}

fn setup(world: &mut World) {
    // does not panic on upper bounds of 0..63, panics on 64...
    for _ in 0..64 {
        let mut tiles = Vec::new();

        for _ in 0..=20 {
            tiles.push(SpriteBundle {
                ..Default::default()
            })
        }

        world.spawn_batch(tiles);
    }
}

Note that I changed the add_system to add_startup_system to show that this fails after spawn_batch is repeated X times.


Image resolution too small, can't read content

Description
Image resolution too small, can't read content. In documentation chapter "Using CMake for Configuring and Building ITK" https://itk.org/ITKSoftwareGuide/html/Book1/ITKSoftwareGuide-Book1ch2.html#x22-180002.2
Expected information
The image resolution should be high enough so that the text in it is readable.
Versions
Book1ch2

While it doesn't solve this issue, the resolution is high in printable version. Generally, HTML version has lower-resolution images than PDF version.


Create code-of-coduct.md for this project

Every open source project should have a code of coduct that establishes expectation of behaviour from contributors and everyone participating on the project.
KubeSimplify uses CNCF code of conduct
Add a file code-of-conduct.md stating above and linking to CNCF's code of conduct

Hey @anubha-v-ardhan , i can do this one!

Sure, go ahead!

Except the link to CNCF code of conduct, anything else to be added in there?

took reference from other CNCF projects! So, only adding the link would be sufficient 🙂


The function I need to finish out of repo maps except I don't know how to code in rust

Yeah, we've got two map PRs open and I don't want to merge either of them. Help getting this working is appreciated
@ninjanomnom @SpaceManiac
10$ bounty

Repo download seems to work right, haven't tested jobs system yet

This isn't necessary with the HTTP stuff, still need a .zip handler


Failed to build_pdsh() and no need to self-build pdsh

Problem
I am trying to coral build on CentOS 8 container.
I got an error during build_pdsh() like below.
make  check-TESTS
make[3]: Entering directory '/barreleye/coral_build_2021-08-16-23_49_30-thsefkuc/pdsh_build/BUILD/pdsh-2.34/tests'

...

ERROR: t0004-module-loading.sh - exited with status 1
ERROR: t0006-pdcp.sh - exited with status 1

...

============================================================================
Testsuite summary for pdsh 2.34
============================================================================
# TOTAL: 158
# PASS:  141
# SKIP:  5
# XFAIL: 2
# FAIL:  8
# XPASS: 0
# ERROR: 2
============================================================================
See tests/test-suite.log
============================================================================

...

make[4]: *** [Makefile:742: test-suite.log] Error 1
make[3]: *** [Makefile:850: check-TESTS] Error 2
make[2]: *** [Makefile:939: check-am] Error 2
make[1]: *** [Makefile:634: check-recursive] Error 1
make: *** [Makefile:491: check-recursive] Error 1
error: Bad exit status from /var/tmp/rpm-tmp.0JOtDo (%build)
    bogus date in %changelog: Tue Jul 20 2016 Albert Chu<EMAIL_ADDRESS>    bogus date in %changelog: Fri Feb 23 2006 Ben Woodard<EMAIL_ADDRESS>    bogus date in %changelog: Thu Feb 22 2006 Ben Woodard<EMAIL_ADDRESS>    bogus date in %changelog: Wed Feb 16 2006 Ben Woodard<EMAIL_ADDRESS>    Bad exit status from /var/tmp/rpm-tmp.0JOtDo (%build)
]
[2021/08/16-23:55:04] [ERROR] [coral_build.py:180] failed to build PDSH
[2021/08/16-23:55:04] [ERROR] [coral_build.py:344] failed to download dependent RPMs on host [35e927afbac9]
[2021/08/16-23:55:04] [ERROR] [coral_build.py:758] failed to download dependent rpms
[2021/08/16-23:55:04] [DEBUG] [cmd_general.py:509] command failed with status -1

Idea
It seems to be test errors. But I don't know and don't have investigated the detail well.
Because the pdsh rpm package is available on epel repository actually. ( pdsh-2.34-5.el8.x86_64.rpm CentOS 8 Download)
So no need to self-build pdsh and can avoid this error by using package. Please consider using the package on epel repository.

This issue should have been fixed since pdsh is now downloaded from YUM repo.


the curser setting takes both my screen as one

i have a 2 monitors and i play minecraft on one, fullscreen, when i try to configuer the mouse settings it takes both my screens and not just one monitor. it could be that it is an convigueration setting or something but its really frustrating. i play on forge 1.20.1 i think latest version of the mod
edit: it also isnt taking the bottom of my screen as the outside boundry
it fixed itself i think, oh wait no they didnt
i had a good setup but as soon as i joined the game it was completly messed up and it broke

the hole thing seems to be at an offset, i have a video but its too big

every time i load the configuration the point of the zero changes so one time its in the middle of my screen the next its all the way on the left

There is a key to reset the cursor's origin, by default its the R key but you can change it among other things in the vanilla keybinds menu

You can just place the cursor at the position you want to be the new center, and press R to set it as the new center (it should also work when the cursor is hidden)


[bug]: When using Proxy, cannot see funded wallet details

Prerequisites

[X] I have searched for existing issues that already report this problem.

Device
Laptop/Desktop
Device OS
MacOS
Device OS Version
13.0.1
Browser
Firefox
Browser Version
104.1
Current Behavior
after funding a wallet, a 502 error is showing when trying to access wallet details
Expected Behavior
Wallet details should show without error
Steps to Reproduce

install JAM
create wallet
fund wallet
view wallet

Anything else?
No response

The issue is when using Bitcoin Proxy Server, using core as the only dependency fixes this.

Issue is when using Bitcoin Proxy as a dependency, changing it bitcoin core fixes this.

Removed Proxy as an option.


UPDATING CSS

Fixes Issue🛠

Closes #
Description👨💻

Type of Change📄

[ ] Style (non-breaking change which improves website style or formatting)

Checklist✅

[ ] I am an Open Source contributor
[ ] I have performed a self-review of my code
[ ] My code follows the style guidelines of this project
[ ] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@BHUVANESHWARTS Good! But, try to do significant contributions!


Add an option to disable auto searching socket colors

Most of the time it doesn't matter what color your sockets are only how many links there are. Please and a setting to turn off auto socket searching items.
Ps thanks for making such a cool tool :)

Hi, thanks for creating an issue. The colors of sockets are ignored. Only the selected count will be searched. Currently the sockets and links are connect with following setting:

Sorry I had an issue that I thought had been created by that. Thanks for make such a cool tool :)

Sorry I had an issue that I thought had been created by that. Thanks for make such a cool tool :)

No problem. Glad you like my tool! :rocket: