Ghost shadow element of MudDropZone's child Items' weird behavior when using complex templates

Bug type
Component
Component name
MudDropZone
What happened?
The shadow element which should be shown when an element is being dragged gets kind of faulty when the element has containers such as 'content' or 'actions' inside it (in the following example MudCardContent and MudCardActions elements). Showing the whole page ghost while dragging the single element.
Expected behavior
It should show the shadow of the only element you are dragging. just like the example I provided.
Reproduction link
https://try.mudblazor.com/snippet/mEcQEiFLIwJzwiGO
Reproduction steps
Run the example.
Relevant log output
No response
Version (bug)
6.0.14
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I think this is a duplicate of #4601
As a workaround you can add "DisableRipple" to your button.
I've updated your example: https://try.mudblazor.com/snippet/QkGQYCPidfecLAtu

@haas-daniel Yes it's duplicate, I didn't see that issue.
thank it's working fine.


Waybar ignoring config at ~/.config/waybar/config.json

I'm using waybar with river and couldn't get the river modules to work. On starting waybar I notice this [info] Using configuration file /etc/xdg/waybar/config. This is despite the fact that that I have a config.json in ~/.config/waybar.

try deleting the extension. call the file just "config".
Yes it's annoying, my VsCodium keeps thinking it is json and marking it all red, but it's apparently jsonc. i wish we could have the option of waybar looking for files with extension, and if none found look for the current file for compatibility.

I feel a bit silly now. I should've read more carefully.


feat: impl invModPUint512 hint

Resolves #559

Resolves #559

Hey, could you add this integration test to the integration tests directory and see if it works? Just add the name of the test file to the .env file and run make integration.

I also see, that your unit tests are not passing. Please run make unit and resolve all the issues. Try to take the expected values for the tests, by making prints in the hint itself (the code in the link I sent in the previous comment) and running the modified hint with the prints using cairo-compile and cairo-run commands. Thanks!

Sure, will do

directory

Concerning unit tests, effectively this is not working but since I don't have a math background I tried to solve this by myself but unfortunately didn't succeed. I'm not initiated to modular maths at all...

directory

Concerning unit tests, effectively this is not working but since I don't have a math background I tried to solve this by myself but unfortunately didn't succeed. I'm not initiated to modular maths at all...

Hey, no need to know maths in depth, just a couple of prints inside of the hint, compiled and run on python cairo-vm will do the job.


add API reference

more like a reminder for me. Something like this: http://sphinx-gallery.readthedocs.io/en/latest/reference.html. It will list all the functions + docstrings in the html pages.
but @titipata feel free to add it with your PR if you're in the flow :)

@pavanramkumar hope I am not adding too many issues! just trying to check of things which are necessary to make it look polished :)

@jasmainak no, your suggestions and contributions are amazing! but i'll continue to be issue nazi to make sure we don't get distracted...

I add API documentation in https://github.com/pavanramkumar/pyglmnet/pull/87, please feel free to take a look.


Remove/evolve FileIndexProvider

Master issue to track stabilizing the FileSearchProvider extension API...
Forked from #47058

Remove FileIndexProvider
Get shared fuzzy matching code into some npm module to share between vscode and Live Share

Instead of copying the fuzzy matching support code into a new repo, we can publish it to an npm module out of the main vscode repo, similar to what we do with monaco.

I'd like to understand the plan for the proposed FileIndexProvider API. In https://github.com/Microsoft/vscode/issues/47058#issuecomment-414759459 @roblourens flagged up the cost of URI-ifying the filenames before returning, and asked if we could return strings of relative paths instead. The response from @jrieken was simply

No - the API uses Uri

Since this is still only a proposed API I'm unclear why that aspect couldn't be changed. Or maybe it's that the caller would still have to URI-ify the results, and perhaps it's no cheaper there than if it was done in the EH?

We will not ship that API because of the inherent problems it has. It's easy to blame on uris (which means blaming the use of objects) but the concept of collecting all filenames (or uris) and sending them in a single batch is problematic. With uris (objects) that might happen sooner but no matter how low-level the abstract is chosen, given the right amount of data this will be a kamikaze API.

Thanks for the response. A get-all-in-one-batch will still happen when workspace.findFiles is called and an extension implements FileSearchProvider, correct?

Yeah, that will happen but on the when the query is *, no result limit is set, and all (also the default) search excludes are nuked. In that (rare?) case all files of a working will be send around but it also gives us the chance to bail out, e.g. like a-query-result-too-large-error. We don't wanna make this case the API default of working with the system.

I wanted to use FileSearchProvider for a remote fs extension I'm working on (for SAP systems, which are not actual remote filesystems but close enough to work).
Will it be possible to exploit this new API instead? BTW, fileindexprovider wouldn't really work in my case as it's very expensive to get a file index from the server.
I did get a "file search" working, but it's rather poorly integrated as I don't want to use unreleased APIs

@marcellourbani No, this API proposal will go away. For your scenario you wanna implement a FileSystemProvider and optionally the up coming search provider APIs

@jrieken Just to clarify my understanding - FileIndexProvider will go away, and FileSearchProvider will transition from an experimental API to a standard one. Correct?

yes

@marcellourbani No, this API proposal will go away. For your scenario you wanna implement a FileSystemProvider and optionally the up coming search provider APIs

Thank you.
I have FileSystemProvider working already, looking forward to the search provider. Now it's very annoying as code occasionally complains not having a search provider but I can't provide one (well, I can but then I couldn't publish the whole thing)
Will have to wait the upcoming API or stay where it is in a context menu entry

As part of this we also have to reintroduce the cacheKey/clearCache() to the FileSearchProvider. I was thinking that we had resolved this already, but no, I remember we got rid of it when moving to the FileIndexProvider...
 	export interface FileSearchProvider {
                 // ...

		clearCache?(cacheKey: string): void;
	}

        export interface FileSearchOptions extends SearchOptions {
		// ...

		cacheKey?: string;
	}

Originally when we discussed this, we considered making the lifecycle of the request more explicit with the concept of a search "session" which is initialized, searches, and is disposed to clear the cache. I think that's overkill for this API. Let's keep this simpler option. I think that most search providers besides ones that we are involved with (liveshare) will ignore the lifecycle and cache anyway.

 How about a rich type in the options bag, maybe a cancellation token or something similar. Like so
export interface FileSearchOptions extends SearchOptions {
  // a token that is being reused per session. will fire cancellation
  // when the session isn't needed anymore
  session: CancellationToken;
}

Then, as an extension I can use that token to manage my caches.

That's a good idea


Revert "logging: Assert fail on pthread without log lock"

Reverts envoyproxy/envoy#2619

@lita can you expand on the rationale for the rollback? Preferably in the commit message at top.

My bad - wrong pr to revert. Closing


ISPN-12953 Zero Capacity Node TopologyJoinCommand frequently timing out

https://issues.redhat.com/browse/ISPN-12953
The DistributedServerTask and TaskParameter had to be removed from server/runtime/proto.lock as these messages now exist in server/core. As these are internal types that are never persisted this should have no impact on the user.

Merged, thanks Ryan!


Lemma with an ensures clause is not being applied

We are using the following lemma:
rule [keccak-eq-conc-false]: keccak(A)  ==Int B => false
    ensures #notEq(keccak(A), B, "keccak-eq-conc-false") [symbolic(A), concrete(B), simplification, comm]

The idea is that sometimes we want to assume that the keccak of a symbolic expression will not collide with a pre-existing concrete value, but we are using the ensures clause to record this assumption in the configuration (with a new #notEq symbol) so we can check afterwards that the cases where this lemma is applied are sound.
However, we have the following claim:
claim [lookup-keccak]:
    <k> runLemma ( #lookup ( STORAGE:Map [ 1 <- 2 ] , keccak ( X ) ) )
    => doneLemma ( #lookup ( STORAGE:Map , keccak ( X ) ) ) ... </k>

It should be passing by the lemma above and this lemma about #lookup, but it fails with the lemma as it is. However, if we remove the ensures clause, it passes.
I'm attaching a keccak-lemmas.k file with the lemma and claim above. The behavior should be reproducible using kprove. It's worth noting that this claim is based on a proof that we were running with the booster, where this subexpression was also not being simplified.
keccak-lemmas.k.txt

I mentioned this issue to @goodlyrottenapple in our meeting earlier today.

Would it be sufficient to log it whenever this equation applies, then inspect the log after the proof is finished?
Rather than building it into the ensure clause, basically make it meta-level feature (which it is), by just telling the backend to log applications of this rule?

@ehildenb Can we have the backend log applications of certain rules? Yeah, if we can do that we wouldn't need the ensures clause.

I think that should be an easy enough feature to implement, yes. I think it already can do that actually, not sure it will also log the substitution it applies, but we can tell it to log rules applied, and then pyk can extract those logs.
I guess it wolud be nice to tell it to only log the specific rule we care about, because otherwise th elogs will be huge and the backend much slowed down.
@geo2a is it possible? Does this information exist in the logs? Can we tell (over RPC), to only log specific equations by name, and also to give us the sustitution that equation applied with in the log?

At the moment the only option is "log-successful-simplifications": true which will return entries of this shape
{
  "tag": "simplification",
  "original-term": ...,
  "result": {
    "tag": "success",
    "rewritten-term": ...,
    "rule-id": ...
  }
}

perhaps the easiest thing would be to modify the "log-successful-simplifications": true parameter to either take a bool or a list of rule labels, i.e. "log-successful-simplifications": ["keccak-eq-conc-false"]?

Yeah, that would be great. And I guess the original-term field contains the needed information about what the arguments to keccak were.

Out experience shows that simplification logs very quickly become prohibitively large (10s of GB over one request), therefore we will implement a more specialized version, as @goodlyrottenapple suggested. @ehildenb @lucasmt do I understand correctly that we need to log the arguments too, not only the tag? Could you come up with the minimal amount of information we need to log? How many applications of the "tracked rule" do you envision during one proof?

@geo2a Yeah, we would want to know the arguments as well. That way if we ever have an application of the rule that looks like this, for example:
keccak(A)  ==Int keccak(b"") => false

we would be able to catch it. So ideally we'd like to log the subexpression that matches the left-hand side of the rule, or else the substitutions (like A |-> A, B |-> keccak(b"")). I don't actually know how many applications we would expect, because they happen behind the scenes. Maybe @PetarMax has some idea?


Add Custom Gradient Color Picker

Hey sorry for the late response! I checked your code and it looks good, it just doesn't really fit the design.
I am on the fence about merging it because V1 won't really be updated anymore and I have yet to start on V2.
Sorry to disappoint, but perhaps ill merge just so you can be contributor!


Fixing a few broken links

Hey @justin-schroeder
I've noticed a few broken internal links on the website. Here is a quick fix.
Peter

Super helpful. Thank you!!

No problem!


unable to read property list from file: /Users/testuser/Desktop/bluetooth-ble/ios/bluetoothble/Info.plist: The operation couldn’t be completed. (XCBUtil.PropertyListConversionError error 2.)

Prerequisites

[X] I checked the documentation and FAQ without finding a solution
[X] I checked to make sure that this issue has not already been filed
[X] I'm sure that question is related to the library itself and not Bluetooth Low Energy or Classic in general. If that so, please post your question on StackOverflow.
[ ] I'm running the latest version

Question
Hi, I get the following error when debugging on iOS. I researched this error on the internet but could not find a solution. Does anyone have any ideas on this subject?
Error is here:
unable to read property list from file: /Users/testuser/Desktop/bluetooth-ble/ios/bluetoothble/Info.plist: The operation couldn’t be completed. (XCBUtil.PropertyListConversionError error 2.)

This is likely unrelated to the library itself but rather to formatting thst plist file. Can you share it here?

This is likely unrelated to the library itself but rather to formatting this plist file. Can you share it here?

The problem was that in the info plist file, the line where I gave the bluetooth permission affected other lines. I solved this problem, thank you.

Closing as issue seems to be resolved


Add CI check to validate install size?

Given the recent issue with install size increase (#158), should we add a CI check to validate the project deps haven't increased between versions with an explicit confirmation this was expected?
This comment shows how to do this for a project: https://github.com/ai/size-limit/issues/65#issuecomment-395982967
Could also compare against previous version size using this API<EMAIL_ADDRESS>

There's also this tool to help with this (https://github.com/siddharthkp/cost-of-modules) and this comment (https://github.com/siddharthkp/cost-of-modules/issues/51)


Window won't render/restore

Type: Bug
A couple of times in the last week I've been running multiple VS Code windows simultaneously and found that one of them won't show its window any more. The other works fine.
I've tried using windows 11 app/monitor docking shortcuts but it doesn't work. The faulty window shows on the task bar and in the list of apps to alt-tab through. Tried restarting explorer.exe (in case it was the problem), but that didn't help either.
The faulty window was maximised and it was performing a long-running debugpy (python) debug session.
I access this computer via remote desktop and in the latest instance I had been logged-out (inactivity) before logging back in again to find one of the windows wouldn't display any more.
VS Code version: Code 1.87.2 (863d2581ecda6849923a2118d93a088b0745d9d6, 2024-03-08T15:20:17.278Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-10980XE CPU @ 3.00GHz (36 x 3000)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
127.72GB (83.01GB free)

Process Argv
E:\dev\\.vscode\fbcli.code-workspace --crash-reporter-id f7ab30df-e6d5-46d4-8630-294e93f9777b

Screen Reader
no

VM
0%

Extensions (41)

Extension
Author (truncated)
Version

Bookmarks
ale
13.4.2

debugpy-old
ata
2023.1.12492010

githistory
don
0.6.20

python-environment-manager
don
1.2.4

xml
Dot
2.5.1

gitlens
eam
14.9.0

communicationhubtools
Ele
1.1.0

json-tools
eri
1.0.2

nsis
idl
4.2.4

vscode-graphviz
joa
0.0.9

vscode-azurefunctions
ms-
1.13.4

vscode-azureresourcegroups
ms-
0.8.4

vscode-bicep
ms-
0.26.54

vscode-cosmosdb
ms-
0.20.1

csdevkit
ms-
1.4.29

csharp
ms-
2.22.3

vscode-dotnet-runtime
ms-
2.0.2

vscodeintellicode-csharp
ms-
0.1.26

autopep8
ms-
2023.8.0

debugpy
ms-
2024.3.10731008

isort
ms-
2023.10.1

pylint
ms-
2023.10.1

python
ms-
2024.2.1

vscode-pylance
ms-
2024.3.1

jupyter
ms-
2024.2.0

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.17

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

remote-wsl
ms-
0.86.0

azure-account
ms-
0.11.6

azurecli
ms-
0.6.0

cmake-tools
ms-
1.17.17

cpptools
ms-
1.19.7

cpptools-extension-pack
ms-
1.3.0

powershell
ms-
2024.0.0

azurerm-vscode-tools
msa
0.15.11

cosmosdbsqlapi
Sav
0.0.60

markdown-preview-enhanced
shd
0.8.12

cmake
twx
0.0.17

markdown-all-in-one
yzh
3.6.2

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vscod805cf:30301675
binariesv615:30325510
vsaa593:30376534
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
vscrpc:30673769
2i9eh265:30646982
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
accentitlementst:30887150
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfcontrol:30979541
d34g3935:30971562
fegfb526:30981948
bg6jg535:30979843
dsvsc020:30976470

Seems to be connected to remote connections, not sure if I correctly assigned, feel free to reassign

Was this involving floating windows?

I had multiple VS Code sessions open. It was their main windows.

Do you have steps to reproduce for me?

No, sorry.  I'd say it was an intermittent problem. My current work doesn't have me using VS Code as much and I haven't seen it in the last couple of weeks.


Updating AvatarView Components

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes
Updated borders and overflow for AvatarView.
Verification
Light and dark mode was tested.

Before
After

Pull request checklist
This PR has considered:

[x] Light and Dark appearances
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)

Microsoft Reviewers: Open in CodeFlow

looking at your screenshot, it seems like your extra small case, the initials are being cut off when the face stack has borders

does your border change impact demo app > avatar view circle style with image-based border and just regular border?

It has updated image-based borders as well for consistency but I can revert if needed. Updated PR with images

are the image of the pr description updated?


Add regex and wildcard operators on fields

You should be able to do "find +name=foo*" and "find +name~^foo\d+$".

Regex operator added in 8695c706a057ca2cb5acc34d7827874e5e2b7528.


Headers field unused on WASM

It seems that the headers field of the Request struct remains unused when compiling to WASM. From what I can tell this is because they are set via a getter here and therefore remain unused.
From a quick glance at the web-sys docs the correct way to set headers is by passing a Headers struct to the RequestInit instead. I might be able to test this and make a PR later this week.

@Strosel I got around this by using custom functions/implementations of Request
Post json content for exemple:
fn post_json(url: String, body: Vec<u8>) -> Request {
    Request {
        method: "POST".to_owned(),
        url: url,
        body,
        headers: Headers::new(&[
            ("Accept", "*/*"),
            ("Content-Type", "application/json; charset=utf-8"),
        ]),
        #[cfg(target_arch = "wasm32")]
        mode: Mode::default(),
    }
}


Error in building

I can't seem to build successfully. These are my errors.
Downloading android-arm-profile/linux-x64 tools...
Downloading android-arm-release/linux-x64 tools...
Downloading android-arm64-profile/linux-x64 tools...
Downloading android-arm64-release/linux-x64 tools...
Downloading android-x64-profile/linux-x64 tools...
Downloading android-x64-release/linux-x64 tools...
Launching lib/main.dart on sdk gphone64 x86 64 in debug mode...

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':ncnn_yolox_flutter:externalNativeBuildDebug'.
> Build command failed.
  Error while executing process /home/dnth/Android/Sdk/cmake/3.18.1/bin/ninja with arguments {-C /home/dnth/Desktop/ncnn_yolox_flutter/android/.cxx/cmake/debug/armeabi-v7a ncnn_yolox_flutter}
  ninja: Entering directory `/home/dnth/Desktop/ncnn_yolox_flutter/android/.cxx/cmake/debug/armeabi-v7a'
  [1/2] Building CXX object CMakeFiles/ncnn_yolox_flutter.dir/home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp.o
  FAILED: CMakeFiles/ncnn_yolox_flutter.dir/home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp.o
  /home/dnth/Android/Sdk/ndk/21.1.6352462/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ --target=armv7-none-linux-androideabi24 --gcc-toolchain=/home/dnth/Android/Sdk/ndk/21.1.6352462/toolchains/llvm/prebuilt/linux-x86_64 --sysroot=/home/dnth/Android/Sdk/ndk/21.1.6352462/toolchains/llvm/prebuilt/linux-x86_64/sysroot -DUSE_NCNN_SIMPLEOCV -Dncnn_yolox_flutter_EXPORTS -isystem _deps/ncnn-src/armeabi-v7a/include/ncnn -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -march=armv7-a -mthumb -Wformat -Werror=format-security   -O0 -fno-limit-debug-info  -fPIC -fno-exceptions -pthread -MD -MT CMakeFiles/ncnn_yolox_flutter.dir/home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp.o -MF CMakeFiles/ncnn_yolox_flutter.dir/home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp.o.d -o CMakeFiles/ncnn_yolox_flutter.dir/home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp.o -c /home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp
  In file included from /home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp:20:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/layer.h:18:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/mat.h:38:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/allocator.h:23:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/platform.h:61:
  /snap/flutter/current/usr/include/pthread.h:682:6: error: 'regparm' is not valid on this platform
       __cleanup_fct_attribute;
       ^~~~~~~~~~~~~~~~~~~~~~~
  /snap/flutter/current/usr/include/bits/pthreadtypes-arch.h:103:50: note: expanded from macro '__cleanup_fct_attribute'
  # define __cleanup_fct_attribute __attribute__ ((__regparm__ (1)))
                                                   ^            ~
  In file included from /home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp:20:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/layer.h:18:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/mat.h:38:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/allocator.h:23:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/platform.h:61:
  /snap/flutter/current/usr/include/pthread.h:694:3: error: 'regparm' is not valid on this platform
    __cleanup_fct_attribute;
    ^~~~~~~~~~~~~~~~~~~~~~~
  /snap/flutter/current/usr/include/bits/pthreadtypes-arch.h:103:50: note: expanded from macro '__cleanup_fct_attribute'
  # define __cleanup_fct_attribute __attribute__ ((__regparm__ (1)))
                                                   ^            ~
  In file included from /home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp:20:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/layer.h:18:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/mat.h:38:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/allocator.h:23:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/platform.h:61:
  /snap/flutter/current/usr/include/pthread.h:717:6: error: 'regparm' is not valid on this platform
       __cleanup_fct_attribute;
       ^~~~~~~~~~~~~~~~~~~~~~~
  /snap/flutter/current/usr/include/bits/pthreadtypes-arch.h:103:50: note: expanded from macro '__cleanup_fct_attribute'
  # define __cleanup_fct_attribute __attribute__ ((__regparm__ (1)))
                                                   ^            ~
  In file included from /home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp:20:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/layer.h:18:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/mat.h:38:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/allocator.h:23:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/platform.h:61:
  /snap/flutter/current/usr/include/pthread.h:730:3: error: 'regparm' is not valid on this platform
    __cleanup_fct_attribute;
    ^~~~~~~~~~~~~~~~~~~~~~~
  /snap/flutter/current/usr/include/bits/pthreadtypes-arch.h:103:50: note: expanded from macro '__cleanup_fct_attribute'
  # define __cleanup_fct_attribute __attribute__ ((__regparm__ (1)))
                                                   ^            ~
  In file included from /home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp:20:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/layer.h:18:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/mat.h:38:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/allocator.h:23:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/platform.h:61:
  /snap/flutter/current/usr/include/pthread.h:735:6: error: 'regparm' is not valid on this platform
       __cleanup_fct_attribute __attribute__ ((__noreturn__))
       ^~~~~~~~~~~~~~~~~~~~~~~
  /snap/flutter/current/usr/include/bits/pthreadtypes-arch.h:103:50: note: expanded from macro '__cleanup_fct_attribute'
  # define __cleanup_fct_attribute __attribute__ ((__regparm__ (1)))
                                                   ^            ~
  In file included from /home/dnth/Desktop/ncnn_yolox_flutter/ios/Classes/yolox.cpp:21:
  In file included from _deps/ncnn-src/armeabi-v7a/include/ncnn/net.h:26:
  /home/dnth/Android/Sdk/ndk/21.1.6352462/toolchains/llvm/prebuilt/linux-x86_64/sysroot/usr/include/android/asset_manager.h:140:5: error: expected function body after function declarator
      __RENAME_IF_FILE_OFFSET64(AAsset_seek64);
      ^
  /home/dnth/Android/Sdk/ndk/21.1.6352462/toolchains/llvm/prebuilt/linux-x86_64/sysroot/usr/include/android/asset_manager.h:169:5: error: expected function body after function declarator
      __RENAME_IF_FILE_OFFSET64(AAsset_getLength64);
      ^
  /home/dnth/Android/Sdk/ndk/21.1.6352462/toolchains/llvm/prebuilt/linux-x86_64/sysroot/usr/include/android/asset_manager.h:181:5: error: expected function body after function declarator
      __RENAME_IF_FILE_OFFSET64(AAsset_getRemainingLength64);
      ^
  /home/dnth/Android/Sdk/ndk/21.1.6352462/toolchains/llvm/prebuilt/linux-x86_64/sysroot/usr/include/android/asset_manager.h:199:5: error: expected function body after function declarator
      __RENAME_IF_FILE_OFFSET64(AAsset_openFileDescriptor64);
      ^
  9 errors generated.
  ninja: build stopped: subcommand failed.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1m 0s
Exception: Gradle task assembleDebug failed with exit code 1
Exited (sigterm)

Output for flutter --version
Flutter 3.0.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision cd41fdd495 (12 days ago) • 2022-06-08 09:52:13 -0700
Engine • revision f15f824b57
Tools • Dart 2.17.3 • DevTools 2.12.2

Output for flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.0.2, on Ubuntu 20.04.4 LTS 5.13.0-51-generic, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[✓] Android Studio (version 2021.1)
[✓] VS Code
[✓] Connected device (3 available)
[✓] HTTP Host Availability

Is there any suggestion how to fix this error?

Yes it is on Android emulator.

Hi any update on this?


chore: Update nokogiri to v1.16.2

Depends on https://github.com/github/erblint-github/pull/126
Resolves https://github.com/github/erblint-github/security/dependabot/29

Superseded by https://github.com/github/erblint-github/pull/127


Sc 14489: contact migration

Scope of changes
Adds a new gdsutil command to migrate vasp contacts to the model contacts namespace.
Type of change

[ ] bug fix
[x] new feature
[ ] documentation
[ ] other (describe)

Acceptance criteria
Describe how reviewers can test this change to be sure that it works correctly. Add a checklist if possible
Author checklist

[x] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

There is a story for Ben in the email verification epic to do the migration using this command. That will need to happen by Friday so we should probably just go ahead and merge this.


--ignore-installed can break existing conda environment

Minor issue, but this broke my conda environment as pip pulled numpy=2.1.1, which was incompatible with another package.
The ReadMe suggests to install the package using pip and the --ignore-installed flag. However, if this is is used in a conda envoronment, problems might arise as pip will ignore all currently installed packages by conda, and will (or could) overwrite installed version with non-compatible ones or just install a second version leading to a complete import mess.
Unless there is a specific reason I am not aware of, It would be helpful to change the flag to --force-reinstall, or at least include a warning to first run pip with both the --ignore-installed and the --dry-run flags to inspect what would happen upon installation.
Cheers, and thanks for the package!

Hey @G-Francio,
I had a look and updated the instructions within the README accordingly. Thank you for bringing this to our attention and for your patience. This will likely save people time and frustration in the future.
Note: I wasn't able to fully recreate the issue except when using specific versions of our package (some older versions had loose restrictions for numpy) but either way that step in the instructions was out of date. If you continue to encounter problems feel free to open a new issue and we can provide further assistance.


Support for more than 3 nodes dev cluster

Overview
Add support for dev cluster with more than 3 nodes. Even though we have -n <N> option for dev/run it has a couple of limitations.

The number of nodes cannot be more than 5. Otherwise we try to allocate tcp ports greater than  65535
The number of nodes is assumed to be 3 in couple of places

in setup application
in haproxy config

This PR removes the limitation and adds --random-ports which we are planing to use to run Elixir test suite. So we can instantiate a second instance of the cluster specifically for test suite.
Testing recommendations
Testing --random-ports option

clean development environment rm -rf dev/lib
run cluster as follows dev/run --admin=adm:pass --random-ports (note the port number of the cluster)

verify that logs printed on console use random ports for nodes

let it start and call _membership endpoint (use the port noted in step 2)
(optional): remsh into cluster and run mem3:nodes()

Testing more than 3 nodes

clean development environment rm -rf dev/lib
run cluster as follows dev/run --admin=adm:pass -n 10 (note the port number of the cluster)

verify that logs printed on console use random ports for nodes starting from 4th node

let it start and call _membership endpoint (use port noted in step 2)

verify that there are 10 nodes returned

(optional): remsh into cluster and run mem3:nodes()

Related Issues or Pull Requests
Checklist

[x] Code is written and works correctly;
[ ] Changes are covered by tests;
[ ] Documentation reflects the changes;

On Mon, 10 Dec 2018, at 19:16, Joan Touzet wrote:

wohali approved this pull request.
Fix the deprecation, then +1.
WARNING: If you are planning on introducing haproxy as a mandatory
element for testing elixir stuff with a cluster, you need to know that
there is no haproxy on Windows. Period. You'll need an alternative.

coughs and runs away very quickly:
https://crlog.wordpress.com/2012/09/29/install-haproxy-on-windows-cygwin-good-for-testing/
If this really is a requirement, maybe consider an erlangy reverse proxy. https://github.com/heroku/vegur comes to mind, and https://github.com/edwardt/somdune was used in iriscouch days, but probably doesn't have the plumbing you'd need. There are a few  elixir based ones on hex.pm but I haven't ever used them, all of them use cowboy in some form under the hood.

+1 to an Erlang-y reverse proxy for testing if necessary.
I am hoping @iilyak only updated the haproxy section because it needed it for the >3 node stuff, not because it'll actually be necessary in testing. We'll see ;)

We are not making haproxy a requirement for testing. We just fixing the configuration for the case when people start cluster with dev/run --with-haproxy -n 10.


Consistency in the order of applying a transform and using thunder.jit

grad transform has to be applied after thunder.jit while autocast has to be applied before thunder.jit
import torch
import torch.nn as nn
import thunder
from thunder.core.transforms import grad, autocast
from thunder.examine import examine

def foo(x):
    return x

x = torch.randn(3, device='cpu')

jfoo = thunder.jit(foo)

# RuntimeError: Can only transform compiled thunder functions
# o = thunder.jit(grad(foo))(x)

# Works
o = grad(thunder.jit(foo))(x)

# Works
o = thunder.jit(autocast(foo, dtype=thunder.dtypes.bfloat16))(x)

# NotImplementedError: Attempting to execute outside of a tracing context, which is not supported
# o = autocast(thunder.jit(foo), dtype=thunder.dtypes.bfloat16)(x)

We should aim at all transforms being applied after jit, so I'd say this is a bug in autocast.


Print output paths of artifact produced by the different make targets

I will rework the log output to be more informative.

PR #90 merged :+1:


Clean up when launch fails

If Binder fails to launch a JupyterHub server, it should try to explicitly stop it and not leave pods randomly lying around orphaned. https://github.com/jupyterhub/kubespawner/issues/95 is related

It's hard for me to tell if this is fixed now, though I'll note that we have a decent amount of failed launches, and to my knowledge this doesn't result in orphaned pods. Can we call this fixed?

I don't believe that we actively attempt top stop the server. We have gotten better at failing to proceed if the request stops, for instance, so some orphaned server cases (e.g. user disconnected before launch started) are less likely to occur now.


7tv Transparent Animated Emotes scuffed on Linux

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
The transparent emotes dont seem to be deleting 'old' frames and overlap instead, and it seems to be happening only to 7tv emotes.
e.g. the cat emote https://7tv.app/emotes/63acb7b5acef59270e7cf4b0
Screenshots

OS and Chatterino Version
Chatterino 2.4.4 (commit 53b044c4) built with Qt 5.12.12 Running on Debian GNU/Linux trixie/sid, kernel: 6.1.0-7-amd64

are you using the appimage

are you using the appimage

yes

yes

use the deb

yes

use the deb

that one works, although I had to manully download and install the old package of libboost-filesystem-1.71.0, is there any reason why 1.74 or 1.81 aren't supported

It's built specifically for Ubuntu 20.04, and it's built in a bit of a special way. We're working on PPAs for Ubuntu, and I would be open to have a PPA for debian too if that's not too much extra effort. Your best bet being on Debian might be to just build it from source.
You can explore the source package & launchpad recipe here if that is useful to build with

It's built specifically for Ubuntu 20.04, and it's built in a bit of a special way. We're working on PPAs for Ubuntu, and I would be open to have a PPA for debian too if that's not too much extra effort. Your best bet being on Debian might be to just build it from source.
You can explore the source package & launchpad recipe here if that is useful to build with

This is meant for the .deb package right? Is there any way to fix the AppImge one, or is it also somehow built for ubuntu?  (I'm not really familiar with AppImages)

the appimage is also built for ubuntu, and I'm not looking to support the appimage very much going forward. There's flatpak you can try too, it will probably work better than the appimage

Okay, I guess I'll keep using the .deb one until it stops working, gotta remember to add the proxy settings into .desktop file (there's imo no need for global environment clutter).
In-app setting of the env-vars would be nice tho.
I'll close the issue then if there's nothing much to add.


question about "make_priors"

> self.priors = torch.Tensor(prior_data).view(-1, 4)

hello, i modify your code for my task, and running modified code failed. I notice that the above line generates a cpu tensor. So I change it to following line:

self.priors = torch.Tensor(prior_data).view(-1, 4).cuda()

and, code runs successfully.
But your code here doesn't use ".cuda()". why can your code work?

That's because of this line in the setup:
https://github.com/dbolya/yolact/blob/605b3c8fdf283642e7753c355bc6ee02f8db1380/train.py#L90
(there's a similar line in eval.py)
That sets the default torch.Tensor to return torch.cuda.FloatTensor, so it's cuda by default.

That's because of this line in the setup:
https://github.com/dbolya/yolact/blob/605b3c8fdf283642e7753c355bc6ee02f8db1380/train.py#L90
(there's a similar line in eval.py)
That sets the default torch.Tensor to return torch.cuda.FloatTensor, so it's cuda by default.

thanks for your lighting reply

:^)


SASL Login to ZNC

It would be nice to see users able to log into their ZNC account via SASL.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/101799-sasl-login-to-znc?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).

Does this mean SASL to ZNC or Cyrus? I think this means SASL to IRC as in PLAIN/EXTERNAL/ecdsa-nist256p-challenge but after talking on #znc at freenode, I am not sure.

@Mikaela there is already a cyrusauth module which checks password against saslauthd.
So this issue must be about https://github.com/ircv3/ircv3-specifications/blob/master/extensions/sasl-3.1 (I think)

Ok, so IAL. There is also https://github.com/ircv3/ircv3-specifications/blob/master/extensions/sasl-3.2.md which should probably also be supported.

Yes this is about cap sasl.

Has anything happened with this? I guess at first the supported mechanisms should be PLAIN and EXTERNAL (when *certauth is loaded).
Would this work the same way as now, SASL username Mikaela@jolla/freenode and SASL password as password?

Even with sasl, clients still send USER, so probably sasl username can be
without network and client embedded
2015-02-26 2:56 GMT-08:00 Mikaela Suomalainen<EMAIL_ADDRESS>
Has anything happened with this? I guess at first the supported mechanisms
should be PLAIN and EXTERNAL (when *certauth is loaded).
Would this work the same way as now, SASL username Mikaela@jolla/freenode
and SASL password as password?
—
Reply to this email directly or view it on GitHub
https://github.com/znc/znc/issues/296#issuecomment-76158935.

Even with sasl, clients still send USER, so probably sasl username can be without network and client embedded

There are cases where the user cannot control USER though, like webchat (someone on IRC wanted to use KiwiIRC with znc)

10:03 (Mikaela) grawity: I am not sure what your latest comment means as KiwiIRC works with ZNC withut issues. Just check "I have a password" and write your Mikaela@kiwi/freenode:swordfish

Disregard the last comment then... [aFreshMelon in #znc was very insistent that kiwi requires SASL support, though.]

Maybe how Kiwi works should be opened a little, to user it works like this:

Nick is used as SASL Username (https://github.com/prawnsalad/KiwiIRC/issues/674) and password is used as SASL password.
If SASL fails, KiwiIRC sends the given password normally with PASS.

One issue related to ZNC not supporting SASL for logging in is that clients often default to using SASL instead of PASS and it's difficult for some users to understand that they should be using some random option instead of it. This has happened on #znc recently.
I think SASL is nowadays more widely understood by people than server password.

I have implemented SASL PLAIN in ZNC 1.6.3. It still needs some work and testing, but I can make a PR if there is any interest when it is ready.
SASL Debug Output

Yay!
But it should go to master branch, not to 1.6

I figured as much, but I needed to do it on 1.6.3 for our own purposes first. I will be moving the changes over to a master-based branch after.

I have completed moving my changes so far to a master-based branch.
It can be viewed here
I will hold off on making a PR until I implement handling of numeric 905/ERR_SASLTOOLONG (possibly later tonight), but if anyone would care to look it over and comment, I'd appreciate it.
I committed my .gitignore changes separately as I doubt you guys are too interested in adding VS specific stuff to it (my development environment for this has been VS with the Linux Development addon and Bash/Ubuntu for Windows)

@MrLenin, what status of PR-readiness? :)

Ready as it has been since the last commit. Really was just looking for some more in depth review before making a PR as I touch znc-core a bit

Worked "properly" with  my minimal testing wrt hexchat, almost no testing with other clients

My code is as free as can be, if the znc devs what to hack it into znc in a way that fits their their goals and agenda, they are more than welcome to. My code is written with a bias towards the network we use it on. unfortunately. the only reason I keep my stuff separate.

@MrLenin You didn't open the PR as promised, so I thought you had abandoned this.
So there was a miscommunication :(
There should be an integration test for it.

I was unable to get cyrusauth working using saslauthd. The saslauthd needs to be integrated into ZNC.
The current default auth method is absolutely ridiculous and the use of the user string should not extend beyond their username. The servers/etc are all a completely abstracted idea to the concept of the ZNC. I shouldn't have to configure 5 networks just to change the user string to /network for each and having them identical otherwise.

I was unable to get cyrusauth working

That's because cyrusauth module has nothing to do with SASL login.

The current default auth method is absolutely ridiculous

Yes, it is, but SASL wouldn't improve it. No existing SASL mechanism has a concept of IRC networks and clients.

I shouldn't have to configure 5 networks

This sounds like you want some kind of multiplexing of connections, which also has nothing to do with SASL.
You may want to take a look at https://github.com/ircv3/ircv3-ideas/issues/39

PR open at: https://github.com/znc/znc/pull/1573


Fix Go version [v1.1]

/assign @rjbez17

/assign @erikgb
Erik thanks for lgtm'ing the other one! Can you please lgtm this one too? Thanks!

/lgtm


KAFKA-1880: Add support for checking binary/source compatibility

This PR aims to extend the gradle build with a task that can generate a report for API/ABI compatibility.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@asasvari @andrasbeni @smurakozi @akatona84 would you please review this?

@ijuma at this point it is only a report that can be run manually, it's not part of the build task. The reason for this is that the compare tool (japicmp) requires two sets compiled jars (old and new) and it is quite lengthy to compile both: it takes about 3-4 minutes and I think it should be enough if only Jenkins bounces the commit back. Improvement idea 1. should reduce it to the half of this time. Let me know if you think we should add this to the local build as well.
There is the failOnSemanticIncompatibility option that I added which fails the task if there binary incompatible changes on a minor/maintenance release branch. Just propagated the corresponding cli option from here: http://siom79.github.io/japicmp/CliTool.html. Alternatively the failOnBinaryIncompatibility option could be used too on minor and maintenance branches to achieve the same effect.
For now I specified 3 packages as API to check: org.apache.clients.*, org.apache.common.* and org.apache.connect.*. I'm hoping you can give me more input on how to refine this list, I can easily imagine that this list is by far not complete (for instance shall we include streams too?).
Adding it to the Jenkins jobs however can absolutely be done with some extra steps. For instance in case of trunk: since the previous release was 2.0.0 we need to specify that to the Jenkins build as well (unfortunately "specifying the previous release" can't be automated, we need to figure it out manually for each job).
I've created a test Jenkins job on which I've done something like this:
# do the build/test before this report
#
# branches must be local for now, we need to check them out
git checkout $BASE_BRANCH #2.0.0
git checkout $BRANCH #trunk (2.1.0.dev)
# this is to check out the PR
git fetch origin pull/5620/head:api-compatibility-report
git checkout api-compatibility-report
#
# This can be specified in an "Invoke Gradle Script" build step but copied the command line launch here
./gradlew apiCompatibilityReport -PbaseBranch=$BASE_BRANCH -PnewBranch=$BRANCH -PfailOnSemanticIncompatibility=true

So if you're on the 2.1.x development branch, you need to set 2.0 as the base branch
I didn't want to add it to the standard build because generating this report usually takes 3-4 minutes as it practically checks out the two branch to compare and jars them.
Some possible ways to improve this:

Use the currently built artifacts if they are available. For instance in the case above we wouldn't need to compile trunk once again as during the build process all the required jars were built. This would spare 1-2 minutes for the build.
Checking dependencies: binary incompatible dependencies might be an issue in case of minor and maintenance releases. For instance we upgrade the Jackson dependency to one which is binary incompatible with the previous one could break a Kafka user when they want to upgrade by swapping out the client jar.

Added WIP to the PR title as I'm actually pretty close on implementing idea #1 in my previous comment. It might take 1-2 more days with testing it out.

Some answers:
Public packages are specified in the gradle build in the following way (confusing, I know):

javadoc {
include "/org/apache/kafka/clients/admin/*"
include "/org/apache/kafka/clients/consumer/"
include "**/org/apache/kafka/clients/producer/"
include "/org/apache/kafka/common/*"
include "/org/apache/kafka/common/acl/"
include "**/org/apache/kafka/common/annotation/"
include "/org/apache/kafka/common/errors/*"
include "/org/apache/kafka/common/header/"
include "**/org/apache/kafka/common/resource/"
include "/org/apache/kafka/common/serialization/*"
include "/org/apache/kafka/common/config/"
include "**/org/apache/kafka/common/config/provider/"
include "/org/apache/kafka/common/security/auth/*"
include "/org/apache/kafka/common/security/plain/"
include "**/org/apache/kafka/common/security/scram/"
include "/org/apache/kafka/common/security/token/delegation/*"
include "/org/apache/kafka/common/security/oauthbearer/"
include "**/org/apache/kafka/server/policy/"
include "**/org/apache/kafka/server/quota/*"
}

Generally, the goal is to compare the APIs with a released version. Can we not simply rely on the released artifacts instead of compiling manually?

@ijuma yea I was thinking about adding that feature. Basically I would need to create dependencies out of the provided versions and resolve them. I think it can be done but last time I looked it didn't seem trivial. Let me look into it and come back with what I'll find. Thanks for the public packages.

@ijuma A couple of questions to clarify the use-case. So I was presuming that generally there are 5 use-cases.

The developer wants to check if they are compliant, therefore they check their changes against trunk
We want to enable a check in the PR Jenkins jobs that check PRs if they are compliant (so enforce binary compatibility rule in minor and patch/maintenance releases)
We want to compare random releases to see how did they change from version to version to evaluate how risky it is for users to migrate (or just compare releases just for fun).
When creating patches it is a good thing to see if the patch is good and doesn't break binary compatibility, so let's check against a released version that is being patched.
When upgrading dependencies we'd like to see if we're breaking binary compatibility in them. (I don't see this generally as a big problem but might be an issue though.)

I'm curious if you think we'd aim for these changes or do we need something else?
Also I managed to figure out a way for downloading dependencies (it doesn't seem to be too complex afterall), but it requires a bit more time to implement a well-working solution. Also I'll limit the minimum version to be <IP_ADDRESS> for version downloads. The reason is that certain projects are added during the life of Kafka and going back too much would overcomplicate the whole checker (as I need to add exceptions for certain projects etc.) and I don't see the value in those situations. I think it is not often a use-case when such an old version is asked. And if it is, we can fall back to "compile then compare" instead of "download and compare". If needed I can do it after more important things are ready.

The developer wants to check if they are compliant, therefore they check their changes against trunk

Yes.

We want to enable a check in the PR Jenkins jobs that check PRs if they are compliant (so enforce binary compatibility rule in minor and patch/maintenance releases)

Yes.

We want to compare random releases to see how did they change from version to version to evaluate how risky it is for users to migrate (or just compare releases just for fun).

Nice to have, but could be done separately.

When creating patches it is a good thing to see if the patch is good and doesn't break binary compatibility, so let's check against a released version that is being patched.

Is this different from the Jenkins PR issue?

When upgrading dependencies we'd like to see if we're breaking binary compatibility in them. (I don't see this generally as a big problem but might be an issue though.)

Helpful, but one would hope that the dependencies do this on their own.

Also I managed to figure out a way for downloading dependencies (it doesn't seem to be too complex afterall), but it requires a bit more time to implement a well-working solution. Also I'll limit the minimum version to be <IP_ADDRESS> for version downloads.

I'd say we could even just do 2.0 and trunk.

@ijuma answeing your question: yes it is a subclass of the PR point technically not different.
I've reworked this pretty much and accomodated for most of these use cases (except the dependency checking).
Does the following by default.

It compares the last release with the current branch that we're building on. This is good for a generic development activity and doesn't require any extra time as it downloads the last release from the configured maven repos.
it is part of the check lifecycle step, so whenever a user runs a gradle build or gradle check it'll run after the assemble task.
You can run it against two arbitrary branches / tags and even released versions. The git references must be local, otherwise it won't work.
The release download only works for Kafka 2.0.0+. If somebody adds a new project which is not released yet, then the download will fail (and therefore it fails the build). The build detects though if the source or target references are released versions or not (and therefore has to be built) but if a released version doesn't contain a project (such as 0.7 didn't contain connect) it'll fail when downloading that. This is an unsolved problem as it might be nicer to maybe ignore these but I didn't spend too much time with this as projects aren't added that often and I tried to exclude any "examples" and "system-test" project (that might be added more frequently). Anyway, I think this is a bridge that we need to cross when we're there.
By default it only shows incompatibilities but this can be set through the onlyIncompatible gradle property.
By default it doesn't fail the build in any circumstances but it can be set. There are three flags for this: failOnBinaryIncompatibility, failOnSourceIncompatibility and failOnSemanticIncompatibility. The first two I think straight enough but the third triggers the mode which we actually need. So it will enable failing the build if there is a binary incompatible change between the source and the target if they are minor or patch (maintenance) releases. The reason why I didn't enable this by default is that there is already a bunch of incompatibilities in 2.1 and it would immediately fail the build. I think it would make sense to keep it this way until releasing 2.1 and then switching the default from false to true on this when bumping the version.
I've added an extra gradle property called lastRelease that must be updated whenever the version is bumped after a release.

@TolerableCoder would you please also review it?

@ijuma is this still needed? What can I do to make this eligible for merging?

Yes! Maybe @omkreddy can help review.

Thanks :)
It's pretty much the end of the day for me but tomorrow I'll whip this into shape, rebase it, review it once more.

@omkreddy @ijuma updated this PR, please check it if it's good from a usability perspective and otherwise.
I only have one doubt which is that the report generator by default prints the report to stdout. It might be a bit annoying. Unfortunately in the JApiCmp lib itself there is no way to work this around except reusing some of it's code (25-30 lines). If that's OK to do I can push a change with that.
As of its current state . it builds before the build task but after assemble, therefore part of the build pipeline. Not a huge overhead as by default it downloads the artifacts of the latest release from maven and builds a report against that. The description I wrote about general behavior in https://github.com/apache/kafka/pull/5620#issuecomment-422412156 is still valid.

In fact I pushed it so it can be compared if needed.

restest this please

retest this please

@omkreddy would you please review this once you get some time?

retest this please

Updated the PR.
@omkreddy would you please review this? It might be a good time to add this after the 2.3 release.
CC @cmccabe

retest this please

I think we should get rid of the complexity of implementing git checkout, building, installing, etc. and just download prebuilt versions like @ijuma suggested earlier.
One thing to keep in mind here is that Apache does not keep around binary artifacts for very long on official mirrors.  That's the reason why in ducktape we download binary artifacts from https://s3-us-west-2.amazonaws.com/kafka-packages.  I think we should do that here.  We should also let users specify their own places to download prebuilt artifacts from, of course.

@cmccabe thanks a lot for reviewing this PR. I've updated it considering your suggestions. Now I use the S3 bucket to download the released artefacts instead of resolving them from maven repositories. The user can use -ParchivesLocation=http://whatever-i-want.com/my-kafka-packages to specify their own location. If a custom local branch is specified then the code will attempt to check it out to a temp folder and build it.

retest this please

@ijuma @cmccabe any opinions if this can get merged in if I rebase it?
I'm trying to clean up my open PR list so if you guys think this one is useful and something that the project needs I can rebase it, otherwise I'll close this.

Sorry, I'm really overloaded right now, but I do think this is important.  Can I check back in a week?  Mention me on the PR then?

@cmccabe just checking back, would you please let me know if you have time ro review this or shall we postpone it?

@cmccabe just checking back, please let me know if you can set aside some time to review this.

@cmccabe just checking back if you have some bandwidth for a review?


included 'transaction_timeout' variable in call to 'wallet'

This includes the call to 'transaction timeout'. Trivial changes include my wallet public address and its balances of play tokens
Cheers,
mnkyNtigr

Fixes #3


Mechanism to detect whether the plugin is running on Mobile or Desktop

As of now there is no sanctioned way of detecting inside a plugin code that it is running in the mobile app vs the desktop.
This comes handy for the following reasons:

Usage of a feature that is only available on Desktop or Mobile
Change design and visuals based on whether is running on Mobile or Desktop.

As of now, I am aware of the possibility of using app.isMobile, but that involves having to extend the typings declaration, plus the fact that I am using an undocumented API that could be changed/removed in the future

Is added by the new insider release and already documented https://github.com/obsidianmd/obsidian-api/commit/48bd2b91f66aa7d3768629be48c52e5351ec6696

For whoever still reading, this has been available for some time and the docs can be found at:
export const Platform {

https://github.com/obsidianmd/obsidian-api/blob/master/obsidian.d.ts#L2067


Support storing / loading / sharing parts of a program

Currently we can only store and load a complete program. As you would like to reuse a special part of a program, e.g. a function for line following, you have to start from scratch or from the full copy of an already written program.
It would be helpful to have the option to mark a special part of an already existing program and to store it for reuse. Once you would like to reuse it, you simply load and attach it to the actual program (or maybe just link to the part?).
There are many ways to do it. One idea is to limit it to blockly functions, e.g.

The challenge might be to find a good way for global variables and maybe we should support scope variable before to make it easier.

+100


feat: Bump roux version to latest

Move roux to latest version as data structures have changed.

Hey @dark0dave, thanks for the PR! I'm going to close it in favor of #7, which also implements the update and which I just merged. Sorry for the long wait!


PMG DCIPHER data issue

Good afternoon James, LaFreda,
Happy Friday! You recently assisted us with understanding the submitterIDs coming out of ReportStream, so figured you all might be the right people for this question. If not, could you please let us know who we can reach out to?
Our team has been reviewing OTC data coming through the pipeline and noticed duplicates coming through reportstream_v3_cleaned. It appears that majority of the duplicates are associated with the submitterID pmg and duplicated 464 times.
We wanted to flag this observation and ask a couple of questions:
Can you please confirm there are duplicates on your end? If so, we would like to understand what event as caused this duplication and how they should be handled. Many of the duplicates from pmg are coming in groups of 464, so it appears it could be caused by one event.
Can you verify if there are duplicates across deidentified data?
For reference, I have provided the number of duplicates we have identified (4,346,176) by submitterID and a sample of 30 duplicates, attached.
Please let me know if you have any questions or concerns!
Best,
Norris K.

Sent email to Norris:
Hi Norris,
Please see below for answers:
1.) Can you please confirm there are duplicates on your end?
      - Yes, it does appear that we have duplicate results from PMG for the examples listed.
If so, we would like to understand what event as caused this duplication and how they should be handled.
- The event causing this duplication was PMG sending the same set of results multiple times.
how they should be handled
- I'm not quite sure how to answer this question. It looks like on 07/06/2022 we implemented a configuration for PMG to reject duplicates so you                  shouldn't be seeing any duplicates from PMG past that point. If you are, I can investigate to see why that is. As far the duplicates that were sent                    prior to 07/06/22 I'm not sure I have a good answer.
Many of the duplicates from pmg are coming in groups of 464, so it appears it could be caused by one event.
2.) Can you verify if there are duplicates across deidentified data?
- Yes, these do appear to be true duplicates as they have the same unique identifier.
Let me know how else I can help.
Thanks,
James"

Closing as no further requests regarding this from DECIPHER


Add shift; several error messages displayed to user at once

When adding a new shift several error messages are displayed to the user at once when the format is incorrect. Should be less messages and better explained.

Would you propose to delete the flash or the messages? Messages are nice as they explain what is going on, perhaps adding them to the form code (inline as in patternfly) would make it easier:

red box around the error
error message below the error explaining it


Addec caml_blocking_section

Please consider the following addition.  My software use large polygons and ability to run multithreaded is important.  I've checked and the gpc.c module appears re-entrant: no global variables or shared state.

I know nothing about multithreading, but this appear innocuous, so merged.


Fixed '[Errno 36] File name too long' issue making it impossible to save comment scrapes with long titles.

Overview
Summary
Added _check_len() function to the Export.NameFile() class to ensure generated filenames are not too long (and thus causing an error when trying to write scrapes to files). Added _check_len() call to Subreddit, Redditor, and comments scraping. These changes should prevent scrapes from failing due to overly long generated filenames.
Motivation/Context
When trying to use the comment scrape option, scrapes are automatically written to a file which includes the title of the comment thread in the filename. In a case where a thread title is rather long (140 chars+) it is possible that the overly long filename will cause an error and the scrape will fail to write to the designated file. In a nutshell, when scraping comment threads with long titles you would sit and wait for it to finish only to find out that your data was lost due to a bad filename :).
New Dependencies
None
Issue Fix or Enhancement Request
Not applicable
Type of Change

[x] Bug Fix (non-breaking change which fixes an issue)

Breaking Change
Not applicable (I have included some scrape logs for reference anyways)
List All Changes That Have Been Made

Created _check_len() function to the Export.NameFile() class

Checks the length of a raw filename and shortens it if need be

Added _check_len() call to all scrape types right before incorrect char validation

How Has This Been Tested?

Target

Attempted to run a comments scrape with limit set to 0:
*Comment url was for a post with a lengthy title (> 140 chars).
+ Ran python3 Urs.py -c https://www.reddit.com/r/AskReddit/comments/j5jb71/how_do_you_deal_with_an_overly_friendly_neighbor/ 0 --json.
* output: python                             [2020-10-05 22:49:19,876] [CRITICAL]: AN ERROR HAS OCCURED WHILE EXPORTING SCRAPED DATA. [2020-10-05 22:49:19,877] [CRITICAL]: [Errno 36] File name too long: '../scrapes/10-05-2020/c-How do you deal with an overly friendly neighbor who asks too many questions about your life when you happen to be outdoors at the same time_-RAW.json'

Test Configuration

Python version: 3.8.2

Running on Linux Mint 20 Ulyana

Kernel version: 5.4.0-48-generic
x86_64

Dependencies
astroid==2.4.1
attrs==19.3.0
certifi==2<IP_ADDRESS>
chardet==3.0.4
colorama==0.4.3
coverage==5.1
idna==2.9
isort==4.3.21
lazy-object-proxy==1.4.3
mccabe==0.6.1
more-itertools==8.3.0
packaging==20.4
pluggy==0.13.1
praw==7.0.0
prawcore==1.3.0
prettytable==0.7.2
py==1.8.1
pylint==2.5.2
pyparsing==2.4.7
pytest==5.4.3
pytest-cov==2.10.0
requests==2.23.0
six==1.14.0
toml==0.10.0
update-checker==0.17
urllib3==1.25.9
wcwidth==0.2.4
websocket-client==0.57.0
wrapt==1.12.1
Checklist
Tip: You can check off items by writing an "x" in the brackets, e.g. [x].

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code, including testing to ensure my fix is effective or that my feature works.
[x] My changes generate no new warnings.
[x] I have commented my code, providing a summary of the functionality of each method, particularly in areas that may be hard to understand.
[x] I have made corresponding changes to the documentation.
[x] I have performed a self-review of this Pull Request template, ensuring the Markdown file renders correctly.

Codecov Report

Merging #19 into master will decrease coverage by 0.97%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
- Coverage   73.07%   72.10%   -0.98%
==========================================
  Files          25       25
  Lines        1998     1932      -66
==========================================
- Hits         1460     1393      -67
- Misses        538      539       +1

Impacted Files
Coverage Δ

urs/utils/Export.py
94.44% <87.50%> (-1.21%)
:arrow_down:

urs/utils/Titles.py
63.15% <0.00%> (-10.92%)
:arrow_down:

urs/utils/Validation.py
34.54% <0.00%> (-7.40%)
:arrow_down:

urs/utils/Comments.py
32.11% <0.00%> (-6.74%)
:arrow_down:

urs/utils/Logger.py
43.06% <0.00%> (-6.29%)
:arrow_down:

urs/utils/Subreddit.py
31.75% <0.00%> (-4.73%)
:arrow_down:

urs/utils/Basic.py
24.42% <0.00%> (-2.78%)
:arrow_down:

urs/utils/Redditor.py
30.40% <0.00%> (-1.10%)
:arrow_down:

urs/utils/Cli.py
94.23% <0.00%> (-0.11%)
:arrow_down:

... and 4 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 6b0a80b...1f7540b. Read the comment docs.

🌚🥴🌝


Enhancement: Extract ResetProcessAction from ForgotController

This PR

[x] extracts a ResetProcessAction from the ForgotController

Follows #912.

Thank you, @chartjes!


Documentation: Update examples in documentation.

Polaris Bug Report
Description of the bug
I realized that in the docs for New-PolarisStaticRoute the examples don't seem to work, cause the parameter 'Path' does not exist anymore. I don't know if it's the same case for other cmdlets in the module, but I think it would be a good idea to check the examples in the documentation, to see if there is more obsolete stuff in there.
Cheers, Kevin!

Hey, I just saw that the parameter was probably renamed to 'FolderPath' instead of 'Path', I ceated a PR to fix it in the docs --> #189

🎉🎉🎉
Thank you!


Prepare User Study

Once hypothesis and scenarios re ready, and inputs have been given from the service, we will need to create a solution that uses the 5.1.0-beta.4 SDK and solves the scenarios.
Send the solution to @meeraharidasa

Sample code, scenarios, credentials, and documentation links have been sent to @meeraharidasa


code generation

https://github.com/microsoft/TypeScript/blob/main/lib/lib.dom.d.ts
How is the above generated?  Is it generated via a tool which parses webidl?  The reason for asking, I'm wondering if we could do the same for blazor/c#/js interop?
https://github.com/yoshiki-okawa/BlazorJSProxy/issues/1#issuecomment-1046793751

https://github.com/microsoft/TypeScript-DOM-lib-generator


Some existing mailbox on domain without CatchAll are identified as unreachable

Hi,
I tried SMTP check on personnal outlook.fr mailbox and it seems that mx record eur.olc.protection.outlook.com don't like consecutive RCPT TO requests.
On smtp.go, I moved the catchAll block code below the test of the real username and the username is correctly detected as deliverable with 250 smtp response code, however this time the RCPT TO request with random username is rejected by mx record with 452 response code.
This issue probably doesn't concern only outlook.fr.
More generally, it may be a good idea to add a feature "safe check" that check the smtp response code and if it is different to 450 the user’s mailbox was unavailable or similare code it doesn't qualified the mailbox as unreachable. I put this idea here but it may be a different issue.
Thanks for all your work

@plorgue
Thanks for your feedback on this issue, it's a good one, the way we handled catchall before may be a bit rough and simple, I'll investigate if there's a better way to handle it. Of course, if you have good suggestions, please feel free to feedback!


Add "Start dragging object" to enable custom picking of object to be dragged

Default dragging requires the cursor to be over an object. This is not great for fabrics of small objects, because there is a lot of space that will not trigger a drag to start.  This change adds a new action to manually start dragging, so game creators can choose when and which objects to grab.
To enable this logic, I added a new option to the Mouse Button property: "None (disable automatic dragging)".
Do you like this implementation, or should I make a separate property, such as: "Enable automatic dragging"?

Example game
https://gd.games/victrisgames/building-monsters

Note: I'm repeating this question so it is more obvious that I am seeking feedback.
I added a new option to the Mouse Button property: None (disable automatic dragging)
Do you like this implementation, or should I make a separate property, such as: Enable/Disable automatic dragging?

Any feedback on my question?  Which is the best implementation:

Add a new option to the Mouse Button property: None (disable automatic dragging)
OR
Add new property Enable/Disable automatic dragging

Any feedback on my question? Which is the best implementation:
* Add a new option to the `Mouse Button` property: `None (disable automatic dragging)`
  OR

* Add new property `Enable/Disable automatic dragging`

I'm not sure the 1st option is possible because the "mouse button" parameter type can't have this new choice in the action to change the property.

Any feedback on my question? Which is the best implementation:

* Add a new option to the `Mouse Button` property: `None (disable automatic dragging)`

  OR

* Add new property `Enable/Disable automatic dragging`

I'm not sure the 1st option is possible because the "mouse button" parameter type can't have this new choice in the action to change the property.

This is actually what my commit does. I'm using a "list of strings".

This is actually what my commit does. I'm using a "list of strings".

Yes, but, ideally the "mouse button" parameter type should be used because the editor can make a better UI with translations and icons for instance.
The same types could be added to properties in the future as it seems useful for input mapping behaviors.

This is actually what my commit does. I'm using a "list of strings".

Yes, but, ideally the "mouse button" parameter type should be used because the editor can make a better UI with translations and icons for instance. The same types could be added to properties in the future as it seems useful for input mapping behaviors.

OK, I think what you are saying is:

There are no current "mouse button" parameter types, but in the future, there might be.
Changing the list of strings to match the standard "mouse button" format will make it easier to transition to the above.
Therefore, I should create a new property to enable/disable automatic dragging

Did I understand correctly?

I've changed this to a new property and removed the option on the mouse buttons.
Next, I will create an update to the existing example game for "draggable physics" to test this change.

I tested this on the default example, but I couldn't think of a good way to add it to the game that made sense.


Multicast

Hi
I have added the code that implements addMembership and dropMembership in this fork:
https://github.com/jmc420/react-native-udp
Do you want me to create a pull request into this repository? I am not an Objective C developer so you might want to compare the fork.
I have tested the changes and they appear to work.
James

@jmc420 thanks for the PR. May I ask how you tested it?
@aprock these methods adhere to the dgram spec, where they are synchronous. So they don't have callbacks to report to the JS side on whether the invocation was successful or not. Do you have any suggestions on how to handle errors? Maybe provide an optional callback param or emit an event for that socket?

An optional callback seems the right approach.  It has the least impact on existing integration.

@jmc420 up for adding some callbacks?

The NodeJS API does not support error handling. There is no return value or callback and it does not say that it raises an "on" event if there is a failure in the underlying joinGroup call:
https://nodejs.org/api/dgram.html#dgram_socket_addmembership_multicastaddress_multicastinterface
I experimented with how NodeJS handles errors in the addMembership call.
Calling addMembership without binding the socket results in no "on" events or runtime exceptions - the socket just never receives any data. Calling addMembership twice after the socket is bound does not trigger an event in the "on" handler but causes a javascript run time error which has to be handled by a try catch block if you want to trap it. Rather than firing an "on" event or invoking a callback, I guess an error in the React native module should trigger a javascript runtime error (i.e. throw new Error). It does not look as though the React native bridge/modules support this though?
In the react-native-udp java version, the exceptions generated by the call to addMembership on line 182 of UdpSockets.java are swallowed and logged to FLog. I guess the equivalent in the Objective C version is just to write to NSLog or maybe Facebook have an Objective C equivalent of FLog? Maybe just leave the Objective C version to do the same as the Java version - i.e. just log the error?

for ios logging use RCTLog.
Yes, unfortunately node.js' synchronous api's have this issue.  We should continue logging it, and extend the api with a callback on both platforms.
@jmc420 I can add it to the android implementation, if you want to take the ios one.

I am not sure about this because changing the signature of the NodeJS
addMembership function will cause the typescript compiler to generate an
error. Just logging the error and generating an exception when FB makes
this possible sounds better.
On Fri, 11 Nov 2016 at 19:20, Andy Prock<EMAIL_ADDRESS>wrote:

for ios logging use RCTLog.
Yes, unfortunately node.js' synchronous api's have this issue. We should
continue logging it, and extend the api with a callback on both platforms.
@jmc420 https://github.com/jmc420 I can add it to the android
implementation, if you want to take the ios one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tradle/react-native-udp/issues/29#issuecomment-260034151,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALLjCXUVW5UULtCnlj0osW-6m5sC8UjOks5q9MABgaJpZM4Ku-GC
.

@jmc420 agreed (see my earlier edit).  this fix needs to be in for both addMembership and dropMembership.

@aprock @jmc420 I have a use case for this, looking forward to seeing the merged PR. If there is someway I can help, I'd be happy to try. I've not done any objective-c before :-D

Hi Jason
Perhaps try out the existing patch to see if it works for you? I am not an Objective C programmer so it is just a bodge.
After you have tried it, perhaps it can be merged as is until someone who is an Objective C programmer wants to do something with it.

@jmc420 your fork is working for me, many thanks. Not sure about the error discussion above though.

merged, since this adds parity with the android implementation. the exceptions will be added later.
available in 1.2.9


Add leetcode #98 solution

Pull Request Template
Description
This PR added solution for Leetcode question #98 in Javascript.
Make sure all below guidelines are followed else PR will get Reject:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code so that it is easy to understand
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules

@lokendra1704 done.

Thanks for the contribution @ongzx
Merging this PR !!!

@GouravRusiya30 possible to tag it with hacktoberfest-accepted?


Alert.trigger api should send a json formatted payload

In Alert trigger the request data should be sent in a json format

@rozenm if you don't mind, I'd welcome a PR


Nested header is not showing on scrolling vertically.

Nested header is not showing on scrolling vertically.
Note: This issue is not available in the jExcel version 2.1.0

Experiencing the same issue in Jspreadsheet v4.10.1
Workaround for me as an adjusted version of @heidashuaiheniwawa suggestion:
.jexcel > thead {
            top: 0 !important;
            position: sticky !important;
        }


GPG signing key for  SHA256SUMS.txt expired in March, 2024

Though somewhat difficult for me to find, the GPG key for the SUMS file expired earlier this year.  While the primary key has no expiration set, the subkeys all have expirations of 2024-03-14, which was 3 years from original creation.
My apologies if an updated key is available from a source that I'm unaware of, but the key available at the link above and from keyserver.ubuntu.com are unchanged.
I appreciate the effort it takes to manage keys and sign releases and recognize that almost no one actually checks them - just weirdos like me - but if the signatures are being produced then the signing key will require routine expiration updates and should probably be linked to from some About or FAQ section.
Once expired, there's little reason to continue signing as verification fails even though the signature otherwise checks out.  We should indeed distribute the keys from a different domain than the signatures, but users may need help in finding those keys.
Thanks for all that you do.

Hi,
The expiration date has been updated on 2024-05-15, and was published to openpgp.org: https://keys.openpgp.org/search?q=456958E85A185DD5C2D1E4E80C822B298461FA03
curl https://keys.openpgp.org/vks/v1/by-fingerprint/456958E85A185DD5C2D1E4E80C822B298461FA03 | gpg --show-key --with-subkey-fingerprint

Indeed, I did not update my about page (now it's done). Thank you for the reminder :wink:
And I also updated on keyserver.ubuntu.com: https://keyserver.ubuntu.com/pks/lookup?search=456958E85A185DD5C2D1E4E80C822B298461FA03&fingerprint=on&op=index

My apologies as my keyserver didn't have the update.  Thank you and I will yet again change my keyserver ;).


Updated the page with corrections for Issue #601

Updated the page with corrections and replaced the second screenshot in both languages on
Close issue #601

Pages to review:

https://deploy-preview-621--a11ycanada.netlify.app/en/making-accessible-emails/
https://deploy-preview-621--a11ycanada.netlify.app/fr/rendre-vos-courriels-accessibles/

Unclassified | Non classifié
Looks good! Thank you Joseph
Ellen Syracopoulos
Accessibility, Accommodations and Adaptive Computer Technologies (AAACT) Program
CIO Directorate, Corporate Services Branch
Shared Services Canada / Government of Canada
Programme d’accessibilité, d’adaptations et de technologie informatique adaptée (AATIA)
Direction générale du DPI, Direction générale des Services ministériels
Services partagés Canada / Gouvernement du Canada
From: Joseph Antle @.>
Sent: Friday, July 26, 2024 11:57 AM
To: gc-da11yn/gc-da11yn.github.io @.>
Cc: Syracopoulos, Ellen (SSC/SPC) @.>; Review requested @.>
Subject: Re: [gc-da11yn/gc-da11yn.github.io] Updated the page with corrections for Issue #601 (PR #621)
Pages to review:

https://deploy-preview-621--a11ycanada.netlify.app/en/making-accessible-emails/
https://deploy-preview-621--a11ycanada.netlify.app/fr/rendre-vos-courriels-accessibles/

—
Reply to this email directly, view it on GitHubhttps://github.com/gc-da11yn/gc-da11yn.github.io/pull/621#issuecomment-2253049866, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BEPQVEEPBA6LWTY3GB7EF3DZOJWTXAVCNFSM6AAAAABLFF3WEWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJTGA2DSOBWGY.
You are receiving this because your review was requested.Message ID: @.***>


Add support for using base template in etc/containerd/config.toml.tmpl

Proposed Changes
Right now you need copy-paste the default containerd config template from github into config.toml.tmpl only to add a few lines.
This PR allows to do :point_down: instead which is much cleaner and removes the need to keep the base template up-to-date from :point_up: :
/var/lib/rancher/k3s/agent/etc/containerd/config.toml.tmpl
{{ template "base" . }}

[plugins."io.containerd.grpc.v1.cri".containerd.runtimes."custom"]
  runtime_type = "io.containerd.runc.v2"
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes."custom".options]
  BinaryName = "/usr/bin/custom-container-runtime"

Types of Changes
The PR adds the base template to the template list, so that it can be used.
Verification

Build
Use above config.toml.tmpl
cat config.toml

Testing

Linked Issues

#7992

User-Facing Change

NONE

Further Comments

Thanks! Would you mind opening an issue describing the problem you're trying to solve, so that our QA folks have something to work against for validation?

Thanks for the contribution!


Not responsive

On mobile, the price widget is not responsive.

Hey @princessfrombabylon99, thank you for filling out the issue. 😊
What behavior would you expect? The table was intentionally designed to be horizontally scrollable.
I'll close the issue, but feel free to open a PR with an alternative solution.

Okay


Upload without using helper

How can I upload an image without using the helper in the package with form data?
This is a shortened code that I have to first show the image in html5 before upload.
I am using cropper.js to crop the image first, before sending it to cloudinary with the formData.
// Deleted Cropper code.
function readURL(input) {
  if (input.files && input.files[0]) {
    var reader = new FileReader();

    reader.readAsDataURL(input.files[0]);

    reader.onload = function (event) {
      $('#avatar-preview').attr('src', event.target.result);
    }
  }
}

Template.foo.events({
  'click .upload': function () {
    // I want to initiate uploading photo from $('#avatar-preview > img') with form data. Here
  }
});

Hi! You would have to do something kind of like this:
				Meteor.call("cloudinary_upload",reader.result,options,function(err,res){
					if(err){
						_cloudinary.remove(options.db_id);
						console.log(err);
					} else {
						callback && callback(res);
					}
				});

That is what we're calling when we're using the normal "cloudinary_upload" method. I also have a "cloudinary_upload_stream" method that should work but I'm not certain. Have a look at this file.

@Lepozepo Thank you! Is it possible to add form data to cloudinary_upload call?

Hey! I completely revamped the code. It is much more efficient now! Although it still needs a bit more work for auth. It should be less confusing to build on it now that there is no helper.


Unable to open OneNote

The onenote-desktop app does not start because it fails to parse a JSON file:
Uncaught Exception:
SyntaxError: Unexpected non-whitespace character after JSON at position 709
    at JSON.parse (<anonymous>)
    at Conf._deserialize (/snap/onenote-desktop/20/resources/app.asar/node_modules/conf/dist/source/index.js:67:43)
    at get store [as store] (/snap/onenote-desktop/20/resources/app.asar/node_modules/conf/dist/source/index.js:278:43)
    at new Conf (/snap/onenote-desktop/20/resources/app.asar/node_modules/conf/dist/source/index.js:131:32)
    at new ElectronStore (/snap/onenote-desktop/20/resources/app.asar/node_modules/electron-store/index.js:69:3)
    at Object.<anonymous> (/snap/onenote-desktop/20/resources/app.asar/lib/appConfig.js:21:16)
    at Module._compile (node:internal/modules/cjs/loader:1269:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1324:10)
    at Module.load (node:internal/modules/cjs/loader:1124:32)
    at Module._load (node:internal/modules/cjs/loader:965:12)

Can you tell me which json file is being read, so that I can check it ?

Hi, it seems like your config file may be corrupted?

Close the app.
Go to /home/<user>/snap/onenote-desktop/current/.config/OneNote/.
Delete the config.json file.
Restart the app.

Let me know if this works. And, if it doesn't, I could use some more details like:

Is this a fresh install? Or did it start after a crash or something?
Your OS.
Write permissions in the user's snap folder, etc.

Thank you, I fixed it my modifying the config.json. It was indeed corrupted and looked like this (extra " and extra { at the end):
{ 	"homepage": "https://www.onenote.com/notebooks", 	"autoHideMenuBar": false, 	"minimizeToTray": false, 	"closeToTray": false, 	"autoShowAboutWindow": false, 	"enableGPUAcceleration": false, 	"wasMaximized": true, 	"width": 800, 	"height": 600, 	"currentVersion": "2.2.2", 	"windowBounds": { 		"x": 0, 		"y": 64, 		"width": 1920, 		"height": 1136 	}, 	"lastLink": "<my_data>", 	"lastVersion": "2.2.2" }" }
It started to happen while the application was opened (I started getting the exception in a popup) and not on a crash. Ubuntu 22.04.3 LTS /  Gnome 42.9
Anyway it is fixed now, thank you.

Cool, let me know if this happens again. Thanks.


Worm feature and listing usage by region

This checklist is used to make sure that common guidelines for a pull request are followed.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/williexu/azure-cli-extensions.git@worm#subdirectory=src/$EXT&egg=$EXT"


[InputBase] Improve documentation for custom inputComponent

Closes #16037
Turns out we already document it but are not very explicit about it.

@material-ui/core: parsed: +0.07% , gzip: +0.16%

Did some testing before this PR and it was not useful to minify the error messages. Might be at some point though. Should keep an eye on it.

I'm so tired of browser tests. Icing this until this is important.


Check Podspec data of requests against OpenAPI spec

At the moment we are just looking at methods and requestURIs of communications with kube-apiserver. We could expand this by looking at the request Podspec data submitted. Doing this we could:

Obtain stats on individual field coverage in the Podspec
Validate individual requests to the endpoint against the OpenAPI spec

There has been interest in using this feature to help efforts surrounding increasing conformance test coverage for POD APIs.
People interested: aish, zhenw

/cc

closing due to staleness.  You can view an updated roadmap here: https://github.com/cncf/apisnoop/blob/master/org/roadmap.org


Implement queryPurchases

This library currently only implements queryPurchaseHistoryAsync() which returns all purchases, even refunded ones.
queryPurchases() returns only the uses current purchases (I think).

Looks like this has been added now :)


get distance of Chinese phrases of the different length

Can this algorithm combine with other, such as Levenshtein Distance, to get get distance of Chinese phrases of the different length?

You can use this algorithm together with DTW

@marina-danilevsky , again this is another algorithmic question, and maybe you have some insights.

Sure, you can use this in aggregate - the input is pairwise, so certainly you can use it on different length phrases, you'll just need to decide how to resolve pairing up input with mismatched lengths. Basically, each pairwise calculation is independent, and you need to choose the method of aggregation. If you are using it as part of a Levenshtein distance approach, for example, when you make substitutions to turn one phrase into another, you could calculate the cost of each substitution using this metric.


Event mouse Scroll doesn't work in dxPopup

Hi all,
I meet a issue when I integrate a ngx-codemirror in a dxPopup, when I want to scroll with the mouse wheel, it doesn't works.
Can you help me, please ?
regards,

Sorry, you can close the issue, I just add dx-scroll-view wrap the component ngx-codemirror


Convert JSON blobs to nix files

There's really no point in doing it any other way right now.

Fixed by #41


net: lwm2m: SenML CBOR zcbor adaptation and floating-point support

This will require zcbor v0.4.0 before it can be merged.

I switched to using the stack for the input and output data.

@jheiskan81 @SeppoTakalo please review and once you're done this can be merged. I'll enable the support with PR #13.

Once PR jheiskan81/sdk-nrf#3 is in this one can be merged.


fix: jupyterlab not found with docker exec bash -c

Also, can you explain why we need to remove the --user scope on the pip install?

@jkeech  --user makes pip install packages in the home directory (ie /home/$USER/.local/bin ). This PATH is not known when we  run docker exec $ID bash -c "jupyter-lab --version". Hence, it currently fails.
The cli takes care of PATH issues with containerEnv. But couldn't find a way to add it there as /home/$USERNAME/.local/bin is dynamic and depends on the $USERNAME.

FYI - These commands would also work if you ran them interactively.  docker exec -i ...  What is the use case for doing docker exec directly?

FYI - These commands would also work if you ran them interactively. docker exec -i ... What is the use case for doing docker exec directly?
@Chuxel

@jungaretti would be be able to answer it better, but it is required by codespaces for jupyterlab integration. Codespaces tries to run docker exec -u codespace <container-id> jupyter server list  and docker exec -u codespace <container-id> jupyter lab --version.
@jungaretti could we try what Chuck suggested docker exec -i, if that works then we don't need to make Feature/universal image changes?

Yeah...
$ docker run -d --privileged --rm mcr.microsoft.com/devcontainers/universal sleep infinity
$ docker exec -it -u codespace <container ID> bash -ic 'jupyter --version'

...works with no errors
Part of the problem here with the universal image is that even the version of python in the path can be dynamic at startup, so there may need to be some special work done in that case verses elsewhere.  Files in the source tree can cause a different version of python to be in the path.

@samruddhikhandale Can we close this? We've fixed the Python feature and this no longer repros in the Universal image.
docker run --rm -u codespace mcr.microsoft.com/devcontainers/universal:latest bash -c 'jupyter lab --version'

3.4.7


Added checks for false cases

In addition to error case, false checks are added for all component verification.
Signed-off-by: Gagandeep Singh<EMAIL_ADDRESS>
What this PR does / why we need it:
This is to fix the logical flaw that if each component status is false, it was being ignored. only error case was being returned.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
fixes #305
Special notes for your reviewer:
Relying on CI for build and verification.

cc: @egurnov

@prateekpandey14 If yes=false always implies err!=nil and vice versa, why do we need both?


ENTESB-19636 - Add JSLT Action Kamelet

https://issues.redhat.com/browse/ENTESB-19636

camel-quarkus-jslt is already available in CEQ 2.13.0:
https://github.com/jboss-fuse/camel-quarkus/pull/184

@oscerd Thanks. Could you merge this?


Change the way to import ONNX model. (cf. #186)

As I wrote in #186 , This seems a better way to import ONNX model.

In the Jenkins testing environment, apparently onnx.load() returns a GraphProto object, while in my environment it returns a ModelProto object...

I will withdraw this PR, because it is from my master branch. I did it due to my slacking off, but apparently it's no good for future potential contributions. I'll do the same PR again later.

You didn't do anything wrong. The latest docs said that onnx.load() returns a ModelProto according to this.
However, it was about 23 days before, they redefined the function load(). Before that, we seriously have a GraphProto returned. How devastating.

What a disaster. The release note of v0.2 also tells:

Introduction of ModelProto to represent top-level model in addition to GraphProto


Option for having output tensors allocated in device memory?

Hello,
I'm loading a model of a frozen graph and run it. I then check for the device of the resulting output-tensors and they all return to me
/job:localhost/replica:0/task:0/device:CPU:0
implying they reside in the host memory. I don't know if this is a result of how the graph is constructed, or if there are options missing I have to set, but I would prefer if they stay in the device memory, so I can access and process the data further via CUDA.

Hi ,
were you able to solve this ?  I faced problem for a related task, meaning loading frozen_model - In my case , the error I got is terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
I can see it has something to do with memory.  When I was loading the original model , there was no issue at all. I faced this only after trying the frozen model. The two model are nearly same in size but there could be difference in structure of the graph though which I didn't check.
Here is how I load my frozen model :
cppflow::model model("Froozen_model_dir", cppflow::model::TYPE::FROZEN_GRAPH);
and here is how I call inference on it with sampple input
output_tensor = model(input_1);
and I got this :
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
Aborted (core dumped)
Any tip on how to solve this


[FEATURE REQUEST]Split downloaded epub per volume or per 100 or so chapter

Some of the novels have very huge number of chapters (2000-3000) and that casues long loading time because i read from my phone.Is it possible to split epub per 100 chapters or so.Then i can or someone who read from the phone can read more smothly.
Is your feature request related to a problem? Please describe.
There is not a problem just very convenient feature.
Describe the solution you'd like
Option to choose split epub according to your will for example 100 chapters per epub or 1 volume per epub and per epub should have cover page and introduction etc.
Describe alternatives you've considered
I dont know you know there is another app called lightnovel-crawler that has this feature but it split according to volume so sometimes it wont work beacuse site has novel as a bulk.1 epub has 3000chapters it took too long to load.
Additional context
Thank you for creating this app it is very usefull and well made.
Best regards

This sounds like a eReader problem to me. I can recommend Moon+ Reader on Android, maybe try that? Alternatively you can always download chapters in steps.

@mskomek
Using moon reader or pocket-book reader. Open epub like instant, then what's your reader?

@mskomek
Using moon reader or pocket-book reader. Open epub like instant, then what's your reader?
the feature something of Splitting the novel into several parts is useless for me, even I really support a complete epub until it's finished even though it has ten thousands of chapters, Lol .. n I have many of it


Fixed critical going out of bounds on -20 modifier

Resolves moo-man/WFRP4e-FoundryVTT#1623

Let's just change the default behavior to be minOne = true

Let's just change the default behavior to be minOne = true

Wasn't sure if it wouldn't affect something else, so wanted to be on the safe side. But yeah, that works too, gimme sec :)