Retaining conversation id's between process executions?

Hello, I just started messing with this library and from what I can tell, the conversation ID's are only valid during an active process session, but if the process/application is restarted, then there seems to no longer be any context when trying to re-use any of these previous conversation ID's.
Is this expected behavior? Is there a way to make these previous conversation ID's re-usable across process executions?

Hi @djriffel, yes this is the expected behaviour since we expected a conversational session to be brief. In order to re-use any of the ids, you can persist them either externally somewhere (i.e., file, or memory). Otherwise, you can use Redis itself to do that, the conversational history would persist in Redis unless you instantiate RedisDataStore with the flag do_flush_data.

So I think I have a little more info on what's going on here. While the do_flush_data parameter will in fact prevent deletion of documents upon construction of the RedisDataStore, the behavior during a service restart is effectively the same either way. This is because in the destructor of the MemoryManager it will call the flush method anyways via self.datastore.flush_all_documents() on line 70 in the clear() method.
One thing that made it more confusing for me, and a bit intermittent, is that I would rapidly hit CTRL+C out of habit, which would sometimes quit the process too fast before it could perform this destructor logic.
Having said all that. This seems like it would behaviorally be a bug, no? Because if one were to provide the False value in the do_flush_data parameter (which is actually the default value, by the way), then would it not be out of line with expectations if it would clear it anyways after the process quits? If you disagree in that, I suppose the follow-up question would be: How can this expected behavior be invoked/reconciled with the library in its current state?

@nps1ngh shall we disable this behavior here.
Because as described above you'd lose memory if you restart the process. WDYT?

Yeah, we should change that 👍

Hi, thanks a lot for your interest in the project.
Due to time constraints, we are unable to continue development on this repository.
Please check out OpenAI's official retrieval plugin, offering similar functionality and active development: openai/chatgpt-retrieval-plugin


[stable/nfs-client-provisioner] Revert running as nobody

What this PR does / why we need it:
Revert PR  #12180
Which issue this PR fixes

fixes nfs volume deletion, restores PSP

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign davidkarlsen

/assign viglesiasce

/assign briceruzand

I would just like to get this reverted asap, it's breaking existing setups.

/lgtm


Compile errors

bolt is a dependency of influxdb. I tried building a simple file that depends on the influxdb client and it's failing with bolt compile errors. It looks like the source of the error is coming from this PR.

Are you building with Go 1.4.x?

Yes, go version produces: go version go1.4.2 darwin/amd64.

Hmm, I'm trying to use build tags because Go 1.4 doesn't recognize arm64 as a GOARCH but that doesn't seem to be working: https://github.com/boltdb/bolt/pull/417
I'm going to revert the arm64 change until I can get this figured out.

Can you verify the removal works for you on Go 1.4? https://github.com/boltdb/bolt/pull/418

Yup. Works fine now. Thanks!


Fix a kind of nested groupBy sql will caused the infinite loop by removing SortRemoveRule.

Fixes #10842.
Description
The root cause is the sql described in #10842 will go to infinite loop by these rules.

AggregateRemoveRule
AggregateProjectMergeRule
SortRemoveRule
SortProjectTransposeRule

Removing any of above rules which can avoid infinite loop. This patch remove SortRemoveRule  to resolve it.

@asdf2014 Hi，Benedict，could you please review it or is there any better resolution?

@kaka11chen - is this no longer an issue? can we close the issue as well if so?


FEAT: update axios to 0.18

Coverage increased (+15.4%) to 88.462% when pulling 24dc94757837612b00fe0c36e06ac59b1e41d69b on feature/update-axios into 6a0fd46e7351868689d1aca81a6fbb22d322b726 on master.


Remove unused device kernels

Gather and Pad operators have JIT kernels.
Device kernels are no longer in use hence removing them.

I get cleaning these up but do we ever want to keep this around for debug purposes?

I get cleaning these up but do we ever want to keep this around for debug purposes?
I see value in that but we can use git to retrieve older files. Also we had gather and pad JIT kernels for a while now. So unlikely it has any bugs now.


Example 1, fixed

Corrected, only delivery receipts are suppressed.

Docs Build status updates of commit ab95e26:
:clock10: Full build: the average full build time is 11 min(s) 48 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit ab95e26:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/mail-flow/Set-RemoteDomain.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 5847f95:
:clock10: Full build: the average full build time is 11 min(s) 48 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 5847f95:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/mail-flow/Set-RemoteDomain.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Visually distinguish internal from external links (through little trailing icon)

Describe the feature you'd like
I think it would be a good idea to trail external links with a related icon, so they can directly distinguished from internal links (without the need to specifically pay attention to the underlaying URL). The icon also comes in handy as separator when lining up multiple external links (no need for separation with comma).
Following is an example how external links currently look in BookStack...

... and how the same links look in Edgewall’s Trac Wiki

would be a great idea to implement

One could consider to trail internal links also with an icon, to distinguish it from coloured text without link. The icon could be as in the above example, but with the arrow pointing into the rectangle (instead of out of it).

This is a trivial CSS change, e.g. see
https://jsfiddle.net/ZkbKp/

Thanks for the hint, I don't wanna tweak my BookStack installation however, as any update would like break the tweak. So it should be a default implementation, though it needs a better designed item as in your example ;)

@Wookbert I just recently implemented exactly this using the "Custom HTML Head Content" in BookStack, which is stored to the database and wouldn't break if you were to update the application.
a[href*="//"]:not([href*="base-domain.com"])::after {   content: "external link content"; }
If something like this were to be considered for inclusion in the default implementation, my thought would be that it would be the other way around: internal links to other parts of the book would be visually distinguished (and perhaps dynamically updated) instead of on the external link.


Button without text selection

Prevents text selection (as in the image) when the user clicks and drags over the sound button, using CSS applied to the key class.

A possible alternative is to use the ::select selector, and then set background-color to transparent and color to currentColor or inherit.


runtime: implement WASM-GC

Go applications compiled to WASM with the browser as target are quite  inefficient and large. Both the download payload and performance could be improved by implementing support for WASM-GC.
Chrome recently enabled WASM-GC
https://developer.chrome.com/blog/wasmgc/

Another good article: https://v8.dev/blog/wasm-gc-porting

This is what I want most in future Go version iterations.

Once Go ships with Wasm-GC support, the binaries produced should be super small... eager to see how the Go team progresses on this!

CC @golang/wasm

After looking briefly at various articles about this, I expect this will be an enormous amount of work, almost comparable to writing an entirely separate compiler. There would have to be special cases at many levels within the compiler. I would also love to see this, but I think it will be hard without considerable effort.

+1 to what @johanbrandhorst said. Another big blocker to this is the fact that the Wasm GC, IIUC, doesn't yet support interior pointers, which are ubiquitous in Go code.

Who do I need to pay to make this happen?
tinygo is great, but cannot handle anything beyond basic go apps.
Please. God. This.

I think folks are presuming that WASM-GC would mean that the generated programs would change a lot. What changes are folks presume? From the comments above, there is the perception that it would make the binaries smaller, but that's not the case. The code to run the GC is tiny in compared to the rest of the program.

I am ignorant, so please excuse that fact, but would completely removing the 1.5mb+ of additional GC golang bulk, only reduce WASM size?

Where are you seeing this 1.5MB?

@daveshanley removing the Go GC from the WASM build will indeed obviously reduce its size, but I have found no information about the "1.5mb+" GC size allegation

bump: any plan for this compiler feature

@omar391 No plans at present, the model of WASM-GC isn't compatible with the Go language.

@evanphx could you provide reference links to documentation about that?

wasm is  fortunately still work in progress. If someone is brave enough feel free to submit a proposal

I just read this article: https://web.dev/case-studies/google-sheets-wasmgc
Seems that Java, Kotlin, Dart and Flutter are making progress. Bit worried that Go is getting behind….

Make Java/JVM great again!Sent from my iPhoneOn 27 Jun 2024, at 23:15, Stephan Westen @.***> wrote:
I just read this article: https://web.dev/case-studies/google-sheets-wasmgc
Seems that Java, Kotlin, Dart and Flutter are making progress. Bit worried that Go is getting behind….
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Current compiled Go code assumes that the garbage collector (GC) will not move memory around. However the WasmGC proposed spec allows for compacting GC that moves memory around; thus this will break most Go code as presently compiled.
In fact, https://v8.dev/blog/trash-talk from 2019 says that the Orinoco garbage collector in the v8 javascript engine (in Chrome) is a moving GC, so this is not a theoretical issue, but a real and present one. Quoting that blog about Orinoco:
"The major GC also chooses to evacuate/compact some pages, based on a fragmentation heuristic. You can think of compaction sort of like hard-disk defragmentation on an old PC. We copy surviving objects into other pages that are not currently being compacted (using the free-list for that page). This way, we can make use of the small and scattered gaps within the memory left behind by dead objects."
Moreover, the current large wasm binaries from Go code I think are mostly attributable to including the fmt and other large standard library packages, and not to the Go runtime GC code; as mentioned above.
In addition, there is no interior pointer support at present in the WasmGC Chrome MVP implementation, which would mean that a whole new memory layout strategy would be needed for arrays that contains structs, and structs that contain structs whose addresses are taken.
Current Go code that pins memory using runtime.Pinner is likely to never work on WasmGC, as pinning is not even on the post-MVP feature list for WasmGC ( https://github.com/WebAssembly/gc/blob/main/proposals/gc/Post-MVP.md ).
For these reasons, it may be best to focus on using/improving the current Wasm support in the Go toolchain rather than holding your breath for WasmGC support. It looks like a ton of work, and it likely won't shrink binaries by much anyway.

I don't really understand the concerns about interior pointers, wouldn't you just use a fat pointer, address+offset to represent these?

@Splizard It's not feasible to use a different type for pointers to the start of an object and interior pointers, so that approach would require that all pointers be fat pointers.  That is probably doable in principle, but would be quite a lot of work in practice.

I was curious about where the space in the hello world .wasm file does get used. A basic hello world from the 1.22 Go toolchain is about 2MB on my darwin laptop.
So I did the following admittedly very crude analysis. My rough conclusion is that runtime takes up about 75% of the space, and of that the garbage collection is 10%. So, again very roughly, not including garbage collection routines would save about 7.5% of function space in the .wasm binary.
I'd welcome much more rigorous means of doing this analysis. Obviously I've used some quick and dirty approximations, simply because I don't know what tools are available to do any better. If you'd like to improve on it, please do, so I know how. Here is how I did my crude analysis.
$ go version
go version go1.22.4 darwin/amd64
$ cat main.go
package main

import "fmt"

func main() {
	fmt.Println("Hello wasip1.")
}

$ GOOS=wasip1 GOARCH=wasm go build -o main.wasm main.go

$ ls -al main.wasm
-rwxr-xr-x  1 me  staff  2112611 Oct 21 11:07 main.wasm
$ ls -alh main.wasm
-rwxr-xr-x  1 me  staff   2.0M Oct 21 11:07 main.wasm

$ # (install github.com/WebAssembly/wabt for analysis)

$file ./main.wasm
./main.wasm: WebAssembly (wasm) binary module version 0x1 (MVP)

$ wasmtime ./main.wasm
Hello wasip1.
$ wasm-objdump -x main.wasm > out.objdump

$ cat analyze.py

import re
from collections import defaultdict

# Read the wasm-objdump output from a file
with open('wasm-objdump-output.txt') as f:
    lines = f.readlines()

# Pattern to match function lines (example: 'func[0] size=120 <runtime.main>')
func_pattern = re.compile(r'func\[\d+\] size=(\d+)\s+<([^>]+)>')

# Dictionary to store cumulative size per package
package_sizes = defaultdict(int)

# Parse the lines
for line in lines:
    match = func_pattern.search(line)
    if match:
        size = int(match.group(1))
        func_name = match.group(2)

        # Extract the package from the function name (e.g., runtime.main -> runtime)
        package = func_name.split('.')[0]

        # Add the size to the corresponding package
        package_sizes[package] += size

# Print the results
for package, total_size in package_sizes.items():
    print(f"Package {package} uses {total_size} bytes")

$ python3 analyze.py |sort -nk 4
Package go_buildid uses 4 bytes
Package _rt0_wasm_wasip1 uses 21 bytes
Package wasm_pc_f_loop uses 42 bytes
Package cmpbody uses 60 bytes
Package internal_bytealg uses 69 bytes
Package memeqbody uses 69 bytes
Package memcmp uses 77 bytes
Package gcWriteBarrier uses 90 bytes
Package callRet uses 123 bytes
... (omit lots of little stuff)
Package os uses 8309 bytes
Package syscall uses 14588 bytes
Package internal_poll uses 16091 bytes
Package type_ uses 16444 bytes
Package internal_fmtsort uses 17271 bytes
Package sync uses 27732 bytes
Package strconv uses 52924 bytes
Package reflect uses 68515 bytes
Package fmt uses 87892 bytes
Package runtime uses 953200 bytes

$ cat out.objdump |grep func|grep size|grep runtime
978345
$ cat out.objdump |grep func|grep size|grep runtime|grep gc|sed 's/size=/size\ /'|awk '{sum+= $4} END {print sum}'
97699
$ ls -al main.wasm
-rwxr-xr-x  1 me  staff  2112611 Oct 21 11:07 main.wasm
$ python3 analyze.py |sort -nk 4 | awk '{sum+= $4} END {print sum}'
1310313  ## so the lines with "func" and "size" are accounting for 1310313/2112611 = 62% of the bytes in main.wasm
$

# So very roughly, a crude, back-of-the-envelope estimate suggests:
#
# Of the sized func sections, "runtime" is the largest, at 978345 / 1310313 = 74% of function bytes.
#
# Out of those, the runtime gc func are 97699 / 978345 = 10% of runtime, and 97699 / 1310313 = 7.5% overall.
#
# Hence we can conlude, very roughly, that not including the Garbage collection
# routines would reduce the .wasm binary size by about 7.5%.


ci: Clear unneeded runner dirs for stake pool job

Problem
The stake pool CI job is failing with "no space left on device".
Solution
Blast some unused directories and see if that helps!

maybe another source of unneeded disk consumption https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2004-Readme.md#cached-docker-images

maybe another source of unneeded disk consumption https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2004-Readme.md#cached-docker-images

ah nice find, done in 4ede450

where do these put us with free space, sufficient?

not sure is it help, but provide the pre-install software list here:
https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2204-Readme.md
if we don't need android stuff, we can delete them and get ~10G back 👻

I also found it installed lots of unnecessary tools in /usr/local/bin

Click me
ls -alh
total 1.2G
drwxrwxrwx  2 root   root   4.0K May 17 22:40 .
drwxr-xr-x 19 root   root   4.0K May 17 22:16 ..
-rwxr-xr-x  1 root   root   3.3K May 17 22:32 activate-global-python-argcomplete
-rwxrwxrwx  1 root   root    20M May 16 09:19 aliyun
lrwxrwxrwx  1 root   root     37 May 17 21:26 aws -> /usr/local/aws-cli/v2/current/bin/aws
lrwxrwxrwx  1 root   root     47 May 17 21:26 aws_completer -> /usr/local/aws-cli/v2/current/bin/aws_completer
-rwxrwxrwx  1 runner docker  30M Apr 20 22:03 azcopy
lrwxrwxrwx  1 root   root     21 May 17 21:23 azcopy10 -> /usr/local/bin/azcopy
lrwxrwxrwx  1 root   root     47 May 17 22:01 bazel -><EMAIL_ADDRESS>lrwxrwxrwx  1 root   root     47 May 17 22:01 bazelisk -><EMAIL_ADDRESS>lrwxrwxrwx  1 root   root     24 May 17 21:57 bcp -> /opt/mssql-tools/bin/bcp
-rwxrwxrwx  1 root   root    56M May 17 21:25 bicep
-rwxr-xr-x  1 root   root    556 May 17 22:05 bin-proxy
-rwxrwxrwx  1 root   root    14M Apr  4 19:10 ccmake
-rwxrwxrwx  1 root   root    14M Apr  4 19:10 cmake
-rwxrwxrwx  1 root   root    29M Apr  4 19:10 cmake-gui
-rwxr-xr-x  1 root   root    560 May 17 22:05 commander
lrwxrwxrwx  1 root   root     45 May 17 21:58 corepack -> ../lib/node_modules/corepack/dist/corepack.js
-rwxrwxrwx  1 root   root    14M Apr  4 19:10 cpack
-rwxrwxrwx  1 root   root    15M Apr  4 19:10 ctest
-rwxrwxrwx  1 root   root    13M May 17 21:13 docker-compose
-rwxr-xr-x  1 root   root    542 May 17 22:05 dotenv
-rwxr-xr-x  1 root   root    554 May 17 22:05 fastlane
lrwxrwxrwx  1 root   root     35 May 17 22:00 grunt -> ../lib/node_modules/grunt/bin/grunt
lrwxrwxrwx  1 root   root     36 May 17 22:00 gulp -> ../lib/node_modules/gulp/bin/gulp.js
-rwxrwxrwx  1 root   root    49M May 17 21:52 helm
lrwxrwxrwx  1 root   root     32 May 17 21:50 heroku -> /usr/local/lib/heroku/bin/heroku
-rwxr-xr-x  1 root   root    566 May 17 22:05 httpclient
-rwxrwxrwx  1 runner docker 9.4M Mar  5  2020 hub
-rwxr-xr-x  1 root   root    560 May 17 22:05 jmespath.rb
-rwxrwxrwx  1 root   root   6.7M May 17 21:52 kind
-rwxrwxrwx  1 root   root    47M May 17 21:52 kubectl
-rwxrwxrwx  1 root   root    15M May 17 21:52 kustomize
-rwxrwxrwx  1 root   root    14K May 17 21:52 lein
lrwxrwxrwx  1 root   root     37 May 17 22:00 lerna -> ../lib/node_modules/lerna/dist/cli.js
-rwxrwxrwx  1 root   root    81M May 17 21:52 minikube
lrwxrwxrwx  1 root   root     27 May 17 22:00 n -> ../lib/node_modules/n/bin/n
lrwxrwxrwx  1 root   root     43 May 17 22:00 netlify -> ../lib/node_modules/netlify-cli/bin/run.mjs
lrwxrwxrwx  1 root   root     40 May 17 22:00 newman -> ../lib/node_modules/newman/bin/newman.js
-rwxrwxrwx  1 root   root    87M May 17 21:58 node
lrwxrwxrwx  1 root   root     21 May 17 22:00 now -> /usr/local/bin/vercel
lrwxrwxrwx  1 root   root     38 May 17 21:58 npm -> ../lib/node_modules/npm/bin/npm-cli.js
lrwxrwxrwx  1 root   root     38 May 17 21:58 npx -> ../lib/node_modules/npm/bin/npx-cli.js
lrwxrwxrwx  1 root   root     43 May 17 22:00 ntl -> ../lib/node_modules/netlify-cli/bin/run.mjs
-rwxrwxrwx  1 root   root   125M May  2 22:14 oc
-rwxr-xr-x  1 runner docker 8.8M Mar 20 10:12 oras
-rwxr-xr-x  1 root   root   195M May  4 19:17 packer
lrwxrwxrwx  1 root   root     37 May 17 22:00 parcel -> ../lib/node_modules/parcel/lib/bin.js
-rwxr-xr-x  1 root   root   3.1M Feb 27 13:05 phpunit
-rwxr-xr-x  1 root   root    206 May 17 22:32 pipx
-rwxr-xr-x  1 runner docker  80M May 15 19:05 pulumi
-rwxr-xr-x  1 runner docker  245 May 15 19:08 pulumi-analyzer-policy
-rwxr-xr-x  1 runner docker 1.9K May 15 19:08 pulumi-analyzer-policy-python
-rwxr-xr-x  1 runner docker  23M Feb 13 17:44 pulumi-language-dotnet
-rwxr-xr-x  1 runner docker  66M May 15 19:05 pulumi-language-go
-rwxr-xr-x  1 runner docker  24M Apr 18 19:34 pulumi-language-java
-rwxr-xr-x  1 runner docker  24M May 15 19:05 pulumi-language-nodejs
-rwxr-xr-x  1 runner docker  24M May 15 19:05 pulumi-language-python
-rwxr-xr-x  1 runner docker  11K May 15 19:08 pulumi-language-python-exec
-rwxr-xr-x  1 runner docker  22M Apr 12 17:09 pulumi-language-yaml
-rwxr-xr-x  1 runner docker  238 May 15 19:08 pulumi-resource-pulumi-nodejs
-rwxr-xr-x  1 runner docker 1.9K May 15 19:08 pulumi-resource-pulumi-python
-rwxr-xr-x  1 runner docker 6.6M Jul 25  2022 pulumi-watch
-rwxr-xr-x  1 root   root   2.6K May 17 22:32 python-argcomplete-check-easy-install-script
-rwxr-xr-x  1 root   root   1.1M May 17 22:34 pzstd
-rwxr-xr-x  1 root   root   2.0K May 17 22:32 register-python-argcomplete
-rwxr-xr-x  1 root   root    540 May 17 22:05 rougify
lrwxrwxrwx  1 root   root     38 May 17 21:26 sam -> /usr/local/aws-sam-cli/current/bin/sam
lrwxrwxrwx  1 root   root     58 May 17 21:26 session-manager-plugin -> /usr/local/sessionmanagerplugin/bin/session-manager-plugin
lrwxrwxrwx  1 root   root     27 May 17 21:57 sqlcmd -> /opt/mssql-tools/bin/sqlcmd
lrwxrwxrwx  1 root   root     32 May 17 21:57 sqlpackage -> /usr/local/sqlpackage/sqlpackage
-rwxrwxrwx  1 root   root    54M May 17 21:50 stack
lrwxrwxrwx  1 root   root     30 May 17 21:29 swift -> /usr/share/swift/usr/bin/swift
lrwxrwxrwx  1 root   root     31 May 17 21:29 swiftc -> /usr/share/swift/usr/bin/swiftc
-rwxr-xr-x  1 root   root    608 May 17 22:05 terminal-notifier
-rwxr-xr-x  1 root   root    62M Apr 26 17:54 terraform
lrwxrwxrwx  1 root   root     38 May 17 22:00 tsc -> ../lib/node_modules/typescript/bin/tsc
lrwxrwxrwx  1 root   root     43 May 17 22:00 tsserver -> ../lib/node_modules/typescript/bin/tsserver
lrwxrwxrwx  1 root   root     19 May 17 22:34 unzstd -> /usr/local/bin/zstd
-rwxr-xr-x  1 root   root    219 May 17 22:32 userpath
lrwxrwxrwx  1 root   root     40 May 17 22:00 vc -> ../lib/node_modules/vercel/dist/index.js
lrwxrwxrwx  1 root   root     28 May 17 22:17 vcpkg -> /usr/local/share/vcpkg/vcpkg
lrwxrwxrwx  1 root   root     40 May 17 22:00 vercel -> ../lib/node_modules/vercel/dist/index.js
lrwxrwxrwx  1 root   root     42 May 17 22:00 webpack -> ../lib/node_modules/webpack/bin/webpack.js
lrwxrwxrwx  1 root   root     42 May 17 22:00 webpack-cli -> ../lib/node_modules/webpack-cli/bin/cli.js
-rwxr-xr-x  1 root   root    546 May 17 22:05 ww
-rwxr-xr-x  1 root   root    560 May 17 22:05 xcodeproj
-rwxr-xr-x  1 root   root    554 May 17 22:05 xcpretty
-rwxr-xr-x  1 root   root    656 May 17 22:05 xcpretty-travis-formatter
lrwxrwxrwx  1 root   root     36 May 17 22:00 yarn -> ../lib/node_modules/yarn/bin/yarn.js
lrwxrwxrwx  1 root   root     36 May 17 22:00 yarnpkg -> ../lib/node_modules/yarn/bin/yarn.js
-rwxr-xr-x  1 root   root   1.2M May 17 22:34 zstd
lrwxrwxrwx  1 root   root     19 May 17 22:34 zstdcat -> /usr/local/bin/zstd
-rwxr-xr-x  1 root   root   3.8K May 17 22:34 zstdgrep
-rwxr-xr-x  1 root   root    197 May 17 22:34 zstdless
lrwxrwxrwx  1 root   root     19 May 17 22:34 zstdmt -> /usr/local/bin/zstd

Merging this now to unblock CI, we can always refine later


Development only services?

I know there are external and executable services, what if I want to run a SQL database locally but use Azure SQL when deployed? How would I set up the configuration to run a SQL database only locally and not deployed?

How would you like to run the SQL database locally? If you running it inside of a container, for example running https://hub.docker.com/_/microsoft-mssql-server, you can have an image reference inside of your tye.yaml to it.
When you deploy, tye doesn't deploy image references, only .NET projects or references to local Dockerfiles. So the SQL database image won't be deployed; you'll need to provide a connection string for the SQL database, which can be your Azure SQL.
The voting sample has an example of using postgressql: https://github.com/dotnet/tye/tree/master/samples/voting
Going back to the original ask though, I think being able to specify environments (like development, production, etc) could be another way to approach this.

Ahh sounds like it was just an understanding on my part; I assumed that any service other than 'externa' or 'executable' would be deployed, now I see that 'image' services are also not deployed, so that alone seems to address my concern.
I think perhaps we can close this issue, but I love your idea of specifying different environments somehow - as a separate idea from my dumb question :)

Yeah, we have an issue for different ENVs here: https://github.com/dotnet/tye/issues/413


System info:
OS: MacOS Ventura 13.0
ISA: aarch64
Installation steps followed:

Disabled system protection.  Rebooted.
Installed via homebrew.
Configured scripting addition.
Ran command yabai --start-service

Any help is appreciated.

I am having the same issue

same here
workaround: source ~/.yabairc

If you you upgraded recently via homebrew you'll have to re-run sha256 in this step:
https://github.com/koekeishiya/yabai/wiki/Installing-yabai-(latest-release)#configure-scripting-addition and update /private/etc/sudoers.d/yabai with the new hash.

@orthotypos it doesn't help me in current release

same here workaround: source ~/.yabairc

Could you please elaborate?  Have you found a way around the issue?

Not sure what the issue would be. Config file has been correctly executed for me since the very early days of development, so I would have to assume that it is something somewhere in your setup.
Try to run yabai from a terminal with the --config argument, and see if it runs the provided file.


Daniel --> master

This PR introduces 2 hotfixes:

Table class init
AbstractTable class init

Both needed for proper functionality (fix crashes) - instantiating a DB in FL, DB nodes functionality etc. However a deeper thought about the implementation is probably needed (they are just hotfixes after all).

I merged it but it shouldnt remain in the code for long time @YoungMasterGandalf

Please prepare fix PR

@DovaX Yes, I will it ASAP


Openshift Multi-tenant Sandbox Hooks

Describe what this PR does and why we need it:
Implementation of #572
Changes proposed in this pull request

Adding post sandbox create/destroy hooks
Adding actual join/isolate network hook implementation
Adding cluster specific shouldJoinNetworks method to retrieve the hooks to be used.
Using openshift api to get structures for openshift calls partial fix for #590

 (5) Implement network isolation proposal for multi-tenet environment.

Changes Unknown when pulling 30dae9d1831969ed4d38665a6ab3de64ddeb5e40 on shawn-hurley:network-iso-impl into ** on openshift:master**.

Changes Unknown when pulling 8e2b60ab5c9203ab44e230db5dee182d5b5f6c23 on shawn-hurley:network-iso-impl into ** on openshift:master**.

Changes Unknown when pulling d8a6f0bda36b62941cc9ef162a65d7a09be18d77 on shawn-hurley:network-iso-impl into ** on openshift:master**.

Changes Unknown when pulling b3df86bb114743b68076a3c17b7f28c88e52346d on shawn-hurley:network-iso-impl into ** on openshift:master**.

This PR is related to https://github.com/openshift/openshift-ansible/pull/6536.
They do not block each other, but they are needed to work together.

Changes Unknown when pulling 63b6aac0efb873425e76888d1a92044fe24d87f2 on shawn-hurley:network-iso-impl into ** on openshift:master**.


bag2tf.py: Error when number of images is below 6000

Thank you very much for this script. Just a heads up that for small data sets, sub 6000 images, the user will encounter an error per below. Its easy to debug, but a warning might help future users. Thanks again.
File "script/bag2tf.py", line 475, in main
splits=split_list, center_only=center_only, debug_print=debug_print)
File "script/bag2tf.py", line 207, in init
writer = ShardWriter(self._outdir, s[0], scaled_images, max_num_shards=scaled_shards)
File "script/bag2tf.py", line 135, in init
self.num_entries_per_shard = num_entries // max_num_shards
ZeroDivisionError: float divmod()

Hello! I got the same error, could you please share how you solved this problem? Very appreciated


Slight updates regarding blob storage accounts

Blob storage accounts can be updated through portal; also changed language slightly

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

@xyh1 : Thanks for your contribution! The author, @hux, has been notified to review your proposed change.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We have closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions


Masking layer doesn't work

I encountered some problems using the masking layer. The network, instead of skipping the padded timestamps, computes the gradients obtaining nan values.  More in detail, I have padded the sequences with the value -1.0 using the pad_sequences function implemented in keras. Then, I have trained the model using the train_on_batch method.
Do you already face these kinds of problems?
Can be this explanation a reason for such problems? "If any downstream layer does not support masking yet receives such an input mask, an exception will be raised." --  keras documentation

Hi thanks for comment! Do you have a reproducible example? I've never used pad_sequences myself.
In any case (when it's working) mask layer will multiply loss function by 0/1 mask if all above layer propagates the mask. So if any of the outputs is NaN then endresult would be NaN after summation


from torchaudio.functional import resample missing in soundstream.py

The Soundstream class in soundstream.py uses resample in the forward call. However, the necessary import does not exist in the file.

@sohananisetty oh yes, thanks!


Update README

[x] Update rationale
[ ] Update screenshot
[ ] Update data
[ ] Verify "how it works"
[ ] Describe data format

Reminder to self (@peterdesmet) to fix this.


Advanced Query Capabilities

When describing a service, Advanced Query Capabilities node is not included in the response.  Here is how it looks in the browser when hitting a service.
Advanced Query Capabilities:
Supports Statistics: true
Supports OrderBy: true
Supports Distinct: true
Supports Pagination: false
Supports TrueCurve: true

Yea, there is quite a lot of missing information from the ServiceLayerDescriptionResponse class. I only put in the more common properties since that response can vary between layers and services. Adding more properties in is fine, though be aware that some of the functions described are not exposed via the operations in the library e.g. statistics, curved geometry.


traceback error doesn't  indicate which cell and line error occurs

Describe the bug
marimo traceback error doesn't  indicate which cell and line error occurs. Not sure what is happening or if I made a mistake that caused this. I experience this after updating marimo.
Example:
Traceback (most recent call last):

FileNotFoundError: [Errno 2] No such file or directory: 'dataset_hand_over_face_check_frame_by_frame.csv'

This example is not the best as it is easier to find where it happens but there are instances of errors that is harder to spot on the code.
Environment
{   "marimo": "0.1.81",   "OS": "Linux",   "OS Version": "<IP_ADDRESS>-microsoft-standard-WSL2",   "Processor": "x86_64",   "Python Version": "3.9.18",   "Binaries": {     "Chrome": "--",     "Node": "--"   },   "Requirements": {     "black": "23.11.0",     "click": "8.1.7",     "jedi": "0.19.1",     "pymdown-extensions": "10.7",     "tomlkit": "0.12.3",     "tornado": "6.3.3",     "typing_extensions": "4.7.1"   } }
This is on WSL2
Code to reproduce
No response

@themissingCRAM, thanks for reporting. It looks like we introduced this in version 0.1.79, when we added support for PDB. Sorry about that! I'll look into this.


VSTest not correctly supporting pre-enumerated theories xunit

Description

VSTest not correctly supporting pre-enumerated theories xunit

Steps to reproduce
Here is the link to the xUnit issue with all the details and it looks like a problem in vstest.
https://github.com/xunit/xunit/issues/1483#issuecomment-343691972

@AbhitejJohn Profiling for xunit Theory tests do not work, because while creating the file for profile report, we fail to do so as the name of the file is invalid.
for e.g. in the attached project the file name that we try to create ends with ""any_theory(input: "1")"", where Path.Combine Api fails
Method public string CreateProfilerReportPath(string testName)
File: ProfilerHost.cs in devdiv.

@shyamnamboodiripad @AbhitejJohn It seems that the problem mentioned in the linked ticket concerns Test Window.
Here is the link to the comment with reproducer: https://github.com/xunit/visualstudio.xunit/issues/114#issuecomment-343691544
Here is the error trace in the test output window:
[21/07/2022 2:41:29.066 pm] [Error] System.ArgumentException: Illegal characters in path.
   at System.IO.Path.CheckInvalidPathChars(String path, Boolean checkAdditional)
   at System.IO.Path.Combine(String path1, String path2)
   at Microsoft.VisualStudio.TestWindow.Profiler.ProfilerHost.CreateProfilerReportPath(String testName)
   at Microsoft.VisualStudio.TestWindow.Profiler.ProfilerLauncher.StartProfiling(String testName, String targetFramework, ExecutionPlatform testExecPlatform)
   at Microsoft.VisualStudio.TestWindow.Profiler.ProfilerController.Start()
   at System.EventHandler`1.Invoke(Object sender, TEventArgs e)
   at Microsoft.VisualStudio.TestWindow.Logging.ILoggerExtensions.CallInternalWithLogging(IInternalLogger log, Action action, Boolean shouldThrow)

I am also seeing another one that wasn't mentioned at first:
[21/07/2022 2:41:31.027 pm] [Error] System.ArgumentNullException: Value cannot be null.
Parameter name: profilerReport
   at Microsoft.VisualStudio.TestWindow.Profiler.ValidateArg.NotNull[T](T arg, String parameterName)
   at Microsoft.VisualStudio.TestWindow.Profiler.ProfilerHost.DisplayReport(String profilerReport)
   at Microsoft.VisualStudio.TestWindow.Profiler.ProfilerController.DisplayResult()
   at Microsoft.VisualStudio.TestWindow.Profiler.ProfilerController.<Stop>b__28_0()
   at Microsoft.VisualStudio.TestWindow.Logging.ILoggerExtensions.CallInternalWithLogging(IInternalLogger log, Action action, Boolean shouldThrow)

@MarcoRossignoli @jakubch1 Do you guys know if the Profile entry is supposed to work? I am getting a popup with the following error message "File contains no data buffer".

I have opened the issue on Test Window side as the problem is on their side.


[Docker/CI] Embedded toolchain in Docker image -> available on DockerHub

see https://hub.docker.com/r/guillaumedua/cpp-toolchain-dev

closed as duplicate with #4


Please regenerate the processed landscape.yml

I was unable to get generation working as a GitHub action.
How do we hook up this bot again?

figured it out

thanks Ry!


Not able to see CPU, Memory ,Read and Writes

Version: pg_activity-1.4.0-1.rhel7.1.noarch
Postgres Version: postgresql10

You should run pg_activity with postgres system user like this:
$ sudo -u postgres pg_activity

we are already in postgres user.
then run pg_activity command there.

Can you let us know if any tool needs to be installed to collect these stats

hi,
pg_activity uses the os and psutils libraries to get the info from the OS / processes.
pg_activity displays the information only if:

it is launched on the database the PostgreSQL server is running on. The test checks if inet_server_addr() is equal to inet_client_addr(). [1]

the user launching it can access the process / OS infos, the steps are :

connect to the database
get the data_directory parameter value
read the pid file
get info from the postmaster process

Are you running pg_activity :

on the server you are running the database ? (it look like it is from your screenshot)
with the user who is running PostgreSQL ?

Is the following code working (replace POSTGRES_PID with the pid of the postmaster)?
$ python3 <<EOF
import psutil
print(psutil.virtual_memory())
p = psutil.Process(POSTGRES_PID)
print(p.cpu_times())
exit()
EOF

[1] https://www.postgresql.org/docs/10/functions-info.html


Trigger does not run on Update

I have a Rollup summary built to roll up completed activities to Opportunity. The rollups work when the tasks are created with the status of completed, but not when the task is edited to have the status go from open to complete.
Am I missing something here?

I am facing the same problem. Any tips how to resolve it?

For me, I had to add the fields I was using in the criteria to the "Relationship Criteria Fields" field and then it started working.

Thanks, that was my mistake too. It works now!


Routing to same url should fire event

What problem does this feature solve?
I need to update the current page (request new data from server) whenever the link in the navigation or breadcrumb is clicked. (like it is implemented on amazon)
Currently the router-link will do absolutely nothing when clicked, while already being on the target page.
Neither beforeRouteUpdate or router.beforeEach are triggered.
But it would be nice to be able to notice a route request to the same route.
What does the proposed API look like?
Maybe a beforeRouteReload hook, which returns false per default and does nothing (as it is now), but when it returns true the normal events get fired and I can handle them.
Or a global parameter which allows me to enable events even if from and to are the same. Ideally on route level, individually for each route.

Closing in favour of #974


Add Elu as a backend ranking function

@havardpe Please review.
Note that I haven't regenerated the tensor conformance test here.

a test should also be added to the c++-only tensor conformance test
if the tensor conformance spec file is not in sync with the program generating it, unit testing will break

@havardpe I'll add the conformance test spec as soon as I get this version built on my machine.

LGTM


add Payload constructors and refactor internal data structure

Codecov Report

Merging #56 (0ff67b3) into master (c371209) will decrease coverage by 0.05%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
- Coverage   64.16%   64.11%   -0.06%
==========================================
  Files          20       20
  Lines         734      758      +24
==========================================
+ Hits          471      486      +15
- Misses        263      272       +9

Impacted Files
Coverage Δ

ergo/flask_http_invoker.py
47.36% <0.00%> (ø)

ergo/function_invocable.py
35.93% <0.00%> (-1.16%)
:arrow_down:

ergo/amqp_invoker.py
29.26% <20.00%> (ø)

ergo/payload.py
57.14% <58.82%> (-5.36%)
:arrow_down:

test/integration/test_amqp.py
94.68% <90.00%> (-0.56%)
:arrow_down:

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 c371209...0ff67b3. Read the comment docs.


Figure out the way how to deploy application to the web

What should we use:

Azure
NodeJS package

smth else?

#45


[commonjs--resolver] No known conditions for "./cid" entry in "multiformats" package when using Sveltekit

System Info:
OS: Windows 11 21H2
CPU: Intel Core i3-10105
RAM: 8GB
Code:
`

`
Error:
`D:\Code\You-apply-for-a-job-at-McDonalds>pnpm run build

<EMAIL_ADDRESS>build D:\Code\You-apply-for-a-job-at-McDonalds
vite build

vite v4.0.4 building for production...
✓ 34 modules transformed.
transforming (50) node_modules.pnpm\ipfs-core@0.17.0\node_modules\ipfs-core\src\components\refs\local.js14:25:57 [vite-plugin-svelte] dom compile done.
package                                 files    time      avg
you-apply-for-a-job-at-mcdonalds            4   0.15s   36.5ms
[commonjs--resolver] No known conditions for "./cid" entry in "multiformats" package
file<EMAIL_ADDRESS>error during build:
Error: No known conditions for "./cid" entry in "multiformats" package
at bail (file:///D:/Code/You-apply-for-a-job-at-McDonalds/node_modules/.pnpm/vite@4.0.4/node_modules/vite/dist/node/chunks/dep-5e7f419b.js:14799:8)
at resolve$1 (file:///D:/Code/You-apply-for-a-job-at-McDonalds/node_modules/.pnpm/vite@4.0.4/node_modules/vite/dist/node/chunks/dep-5e7f419b.js:14856:32)
at resolveExports (file:///D:/Code/You-apply-for-a-job-at-McDonalds/node_modules/.pnpm/vite@4.0.4/node_modules/vite/dist/node/chunks/dep-5e7f419b.js:21865:12)
at resolveDeepImport (file:///D:/Code/You-apply-for-a-job-at-McDonalds/node_modules/.pnpm/vite@4.0.4/node_modules/vite/dist/node/chunks/dep-5e7f419b.js:21883:31)
at tryNodeResolve (file:///D:/Code/You-apply-for-a-job-at-McDonalds/node_modules/.pnpm/vite@4.0.4/node_modules/vite/dist/node/chunks/dep-5e7f419b.js:21572:20)
at Object.resolveId (file:///D:/Code/You-apply-for-a-job-at-McDonalds/node_modules/.pnpm/vite@4.0.4/node_modules/vite/dist/node/chunks/dep-5e7f419b.js:21336:28)
at file:///D:/Code/You-apply-for-a-job-at-McDonalds/node_modules/.pnpm/rollup@3.9.1/node_modules/rollup/dist/es/shared/rollup.js:23660:40
 ELIFECYCLE  Command failed with exit code 1.`
Is an issue with orbit, as ipfs works just fine alone

You can try: https://github.com/orbitdb/orbit-db/issues/1023#issuecomment-1300931772

This should be fixed in 1.0.0. Note the new package name.
If the issue persists, feel free to re-open the issue.


Workflow boolean input example

Why:
Many people are confused by the usage of boolean. https://github.com/actions/runner/issues/1483
invalid
if: ${{ inputs.perform_deploy }}
if: inputs.perform_deploy
Must be treated as string
if: ${{ inputs.perform_deploy == 'true' }}
What's being changed:
Update examples of the usage of workflow inputs of type boolean
Check off the following:
I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
For content changes, I have completed the self-review checklist.
Writer impact (This section is for GitHub staff members only):
This pull request impacts the contribution experience
I have added the 'writer impact' label
I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Note for reviewer: this got a technical review in https://github.com/github/docs/pull/15349.

Madison chambers or Chamberlin POW MIA Afghanistan IRAC,  impersonated by Pamela mceandry, calling themselves Steve and Mary Female identity change to Paris Pichai or Pichilli Apaguasa. Secured identity with "memory" software accounts of DreamWorks high jacked used against her by known criminals and theives.Stalking of female documented with federal restraining order permanent. Paris<PHONE_NUMBER> Sent from my  U.S.Cellular© Smartphone-------- Original message --------From: mc4928184 @.> Date: 2/28/22  15:53  (GMT-05:00) To: @. Subject: FW: Urgent HelpSent from my  U.S.Cellular© Smartphone-------- Original message --------From: mc4928184 @.> Date: 2/28/22  3:51 PM  (GMT-05:00) To: @. Subject: FW: Urgent Data pushesMasked ipPhone info changesAll obsevedSent from my  U.S.Cellular© Smartphone-------- Original message --------From: mc4928184 @.> Date: 2/24/22  12:07 AM  (GMT-05:00) To: @. Subject: Urgent Crime Sent from my  U.S.Cellular© SmartphoneSent from my  U.S.Cellular© Smartphone
-------- Original message --------From: Laura Coursen @.> Date: 3/3/22  17:03  (GMT-05:00) To: github/docs @.> Cc: Subscribed @.***> Subject: Re: [github/docs] Workflow boolean input example (PR #16028)
Note for reviewer: this got a technical review in #15349.
—Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

@tevans-submittable
Thanks so much for opening a PR! I'll get this triaged for review :zap:


Update run error.

Version
0.7.0-beta.3
Environment
window10
Reproduction link
https://stackblitz.com/edit/sfddsa
Steps to reproduce
ng serve
Error: ENOENT: no such file or directory, stat 'G:\ng-zorro-antd\site\src\tsconfig.app.json'
at Object.fs.statSync (fs.js:948:11)
at AngularCompilerPlugin._setupOptions (G:\ng-zorro-antd\node_modules@ngtools\webpack\src\angular_compiler_plugin.js:80:16)
at new AngularCompilerPlugin (G:\ng-zorro-antd\node_modules@ngtools\webpack\src\angular_compiler_plugin.js:45:14)
at _createAotPlugin (G:\ng-zorro-antd\node_modules@angular\cli\models\webpack-configs\typescript.js:77:16)
at Object.getNonAotConfig (G:\ng-zorro-antd\node_modules@angular\cli\models\webpack-configs\typescript.js:100:19)
at NgCliWebpackConfig.buildConfig (G:\ng-zorro-antd\node_modules@angular\cli\models\webpack-config.js:41:37)
at Class.run (G:\ng-zorro-antd\node_modules@angular\cli\tasks\serve.js:71:98)
at check_port_1.checkPort.then.port (G:\ng-zorro-antd\node_modules@angular\cli\commands\serve.js:123:26)
at
at process._tickCallback (internal/process/next_tick.js:188:7)
What is expected?
success
What is actually happening?
success
Other?

pls check your env, all ci passed now.

i have the same problem


[EN] add eBPF

Describe your changes
Added a new term - eBPF
Related issue number or link (ex: resolves #issue-number)
Resolves #1448
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Thanks, @MaryamTavakkoli. We are still working on pod and stateless apps. Next is runtime and then we'll tackle this PR . Thanks for your patience!

@CathPag I applied all the suggestions, please have another look :)

Great! Next time, the easiest thing is to simply click on "sign off and commit suggestion." That will automatically resolve the comment. To merge, all comments must be resolved. If you have a different point of few, you can also start a conversation :) One suggestion had a question that we need to address, though.

@CathPag I see you have pushed the suggestions, thanks! Can you help me understand what is the next step? It shows some checks were not successful (un-signed commits) and some spell-checking. But both seem correct to me (all commits are signed and spells sound fine!), and I don't understand what is wrong.

We need a technical reviewer to have a look as well. Unfortunately, we have a little bit of a backlog right now. But another round of reviews needed.
As to the spellcheck error, eBPF needs to be added to the "wordlist.txt" file. You can find instructions here.

Thanks for this @MaryamTavakkoli!
I've gone through and made some specific technical suggestions in line.
We should also consider mentioning other areas where eBPF is useful, like networking, observability, and security. I don't know if these are key to the definition, but they may provide more context around why eBPF is important.

@nate-double-u Thanks for the suggestions. I applied those suggestions now. Is this in good shape now or we should still add more content?

Friendly ping @MaryamTavakkoli

Thanks, @iamNoah1. Sorry for the delay, I was on holiday! I'll get back to this soon.

@MaryamTavakkoli welcome back. I left one comment :)

Yes, sorry. I missed that and only rebased the MR yesterday. But now I applied the suggestion as well.

thanks, tech review passed.

Hi @CathPag,
How could we proceed with this?

Looks great. Here are some edits. Haven't gone through the entire definition yet, though.

Thanks. I now applied those suggestions :)

Also, quick note that we improved our review process, which is now more transparent, @MaryamTavakkoli. You can see our review board here. I just completed runtime, so I started with the eBPF editorial review (the last step). We know the process isn't super fast, but hope that, with the increased transparency, it will help contributors understand where we are in the process.

Also, quick note that we improved our review process, which is now more transparent, @MaryamTavakkoli. You can see our review board here. I just completed runtime, so I started with the eBPF editorial review (the last step). We know the process isn't super fast, but hope that, with the increased transparency, it will help contributors understand where we are in the process.

Thanks for the explanation @CathPag, it makes it easier to understand how the process works :)

Looks great! I made some minor edits, but this is a solid definition. If these look good to you, @MaryamTavakkoli, we can finalize the edit review and merge 🎉
...and thanks again for your patience while we were onboarding new reviewers and figuring out a new review process!

Thanks for the great comments. I applied the suggestions and it seems fine to me to be merged.


Build out draft user admin panel

One of the ongoing goals for this app is to make sure that we get it to a place where engineers need to maintain fewer non-code parts of it; to that end, @nimiro is doing some heroic work building out some beautiful designs. Check them out here: https://github.com/colinxfleming/dcaf_case_management/files/520274/DCAFwireframe101016_user_account_management.pdf
The steps here are as follows:

Add a view and a route for users/index
Code up the first page of that in hard HTML (no functionality needed yet)
If possible, add a simple controller test to confirm that the route is accessible

Stems from #626

I'll take a stab at this!

Which page of the mockup are we working from? Great work @nimiro !

@itsWill - can you touch base with @nimiro before you start, and keep an eye on our convo in #626? Thanks!

@mebates I just picked the first page, I understand the frontend design is not ready yet so this mainly sets up the backend, we can then change the view according to the design you and @nimiro come up with once its decided.

Demoting from active development so we can focus on required stuff for Dec 1 -- thanks a ton to @itsWill for getting this process started!

Here are some super dope designs courtesy @nimiro --https://github.com/colinxfleming/dcaf_case_management/files/571543/DCAFwireframe110316_user_account_management.pdf
I'm ready to start moving on this, and I think it'd be a nice thing to get into place. Marking active.


fix add spi=0 to ipSecKeysRemovalTime

This commit prevents the LoadIPSecKeys function to add to the ipSecKeysRemovalTime map the value SPI 0. To the best of my understanding, we don't allow to provide a key with spi=0, which will cause cilium to return an error with the proper "invalid spi value" message. At this point, the 1st time that an IPsec key is loaded, this function performs an unneeded update of the map with the key=0 and value=time.Now(), which will never be removed.

/test


map context instances using createContext as key

@TobbeLino could you please try the CodeSandbox package build for this PR and give us feedback if it solves the problem?
npm i https://pkg.csb.dev/reduxjs/react-redux/commit/cd8a93bd/react-redux

@phryneas Great! Yes, my test cases (and also our preact apps) run fine now with this PR!

Let's combine this with  #2050


Posts in the root directory

My current ghost blog has urls at the root. Like www.mysite.com/post-slug. But this theme assumes that all the posts are under /post/post-slug. I want the same url structure as per my current blog structure.
How can I achieve this with this theme? Which files needs modifications? Can someone provide some guidance.
Thank you so much.

I have done with the help of parmalinks.

👍


refactor: consolidate activeMasternodeInfo{Cs} into CActiveMasternodeManager, create NodeContext alias, reduce globals usage

Additional Information

CActiveMasternodeManager, unlike other managers, is conditionally initialized (specifically, when the node is hosting a masternode). This means that checks need to be made to ensure that the conditions needed to initialize the manager are true or that the pointer leads to a valid manager instance.
As the codebase currently checks (and fast-fails) based on the node being in "masternode mode" (fMasternodeMode) or not, we will continue with this approach, but with additional assertions after the masternode mode check if the manager exists.

Though, since activeMasternodeInfo(Cs) are global variables, they can be accessed regardless of whether the corresponding manager exists. This means some parts of the codebase attempt to fetch information about the (nonexistent) active masternode before determining if it should use the masternode mode path or not (looking at you, CMNAuth::ProcessMessage)
Moving them into CActiveMasternodeManager meant adding checks before attempting to access information about the masternode, as they would no longer be accessible with dummy values (here) on account of being part of the conditionally initialized manager.

In an attempt to opportunistically dereference the manager, CDKGSessionManager (accepting a pointer) was dereferencing the manager before passing it to CDKGSessionHandler. This was done under the assumption that  CDKGSessionManager would only ever be initialized in masternode mode.
This is not true. I can confirm that because I spent a few days trying to debug test failures. CDKGSessionHandler is initialized in two scenarios:

In masternode mode
If the -watchquorums flag is enabled

The latter scenario doesn't initialize CActiveMasternodeManager.
Furthermore, the DKG round thread is started unconditionally (here) and the CDKGSessionHandler::StartThreads > CDKGSessionHandler::StartThread > CDKGSessionHandler::PhaseHandlerThread > CDKGSessionHandler::HandleDKGRound > CDKGSessionHandler::InitNewQuorum > CActiveMasternodeManager::GetProTxHash call chain reveals an attempt to fetch active masternode information without any masternode mode checks.
This behaviour has now been changed and the thread will only be spun up if in masternode mode.

Dereferencing so far has been limited to objects that primarily hold data (like CCoinJoinBroadcastTx or CGovernanceObject) as they should not have knowledge of node's state (that responsibility lies with whatever manager manipulates those objects), perform one-off operations and static functions.

activeMasternodeInfo allowed its members to be read-write accessible to anybody who asked. Additionally, signing and decrypting involved borrowing the operator secret key from the active masternode state to perform those operations.
This behaviour has now been changed. The internal state is now private and accessible read-only as a const ref (or copy) and Decrypt/Sign functions have been implemented to allow those operations to happen without having another manager access the operator private key in order to do so.

You cannot combine a WITH_LOCK and an Assert (in either mutex or accessed value), doing so will cause errors if -Werror=thread-safety is enabled. This is why asserts are added even when it would intuitively seem that Assert would've been more appropriate to use.

Future Considerations
Currently there are no unit tests that test the functionality of CActiveMasternodeManager as it's never initialized in test contexts, breakage had to be found using functional tests. Perhaps some (rudimentary) tests for CActiveMasternodeManager may prove to be valuable.
Breaking Changes
Not really. Some behaviour has been modified but nothing that should necessitate updates or upgrades.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas (note: N/A)
[x] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have made corresponding changes to the documentation (note: N/A)
[x] I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

Since this isn't backports; it'd be nice to have Clang-Diff format clean: https://github.com/dashpay/dash/actions/runs/8316932637/job/22757066218?pr=5940

Since this isn't backports; it'd be nice to have Clang-Diff format clean: https://github.com/dashpay/dash/actions/runs/8316932637/job/22757066218?pr=5940

I look at these clang-diff suggestions and I hate most of it tbh...

I look at these clang-diff suggestions and I hate most of it tbh...

I have an idea: https://github.com/dashpay/dash/pull/5942

please consider: https://github.com/PastaPastaPasta/dash/commit/c52d1692dcc7e0756817bb9ecd46099ea908236c


Implement NSH over MAVLink

@tridge has added a wrapper packet to the protocol and implemented it for APM here, and it looks super useful as it deconflicts the USB port and makes it easier to access NSH in general.
https://github.com/diydrones/ardupilot/blob/master/libraries/AP_HAL_PX4/NSHShellStream.cpp

@julianoes needs collaboration with @DonLakeFlyer and @dogmaphobic on the QGC side.

done


Adding missing caching from python code

Summary of the Pull Request
What is this about?
PR Checklist

[ ] Applies to work item: #xxx
[ ] CLA signed. If not, go over here and sign the CLI.
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

Codecov Report

Merging #2467 (5d8e8bd) into main (0c4cd54) will increase coverage by 5.42%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2467      +/-   ##
==========================================
+ Coverage   29.88%   35.31%   +5.42%
==========================================
  Files         288      135     -153
  Lines       35180    16958   -18222
==========================================
- Hits        10513     5988    -4525
+ Misses      24667    10970   -13697

Impacted Files
Coverage Δ

src/agent/coverage/src/test.rs
92.85% <0.00%> (-7.15%)
:arrow_down:

src/agent/coverage/src/code.rs
8.29% <0.00%> (-6.12%)
:arrow_down:

src/agent/onefuzz/src/machine_id.rs
51.21% <0.00%> (-5.63%)
:arrow_down:

src/agent/onefuzz/src/sanitizer.rs
69.69% <0.00%> (-2.91%)
:arrow_down:

src/agent/onefuzz-agent/src/worker/tests.rs
99.10% <0.00%> (-0.11%)
:arrow_down:

src/agent/onefuzz/src/asan.rs
0.00% <0.00%> (ø)

src/agent/coverage/src/cache.rs
0.00% <0.00%> (ø)

src/agent/onefuzz-agent/src/setup.rs
0.00% <0.00%> (ø)

src/agent/onefuzz-agent/src/reboot.rs
0.00% <0.00%> (ø)

src/agent/onefuzz/src/input_tester.rs
0.00% <0.00%> (ø)

... and 164 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Write a C program to check vowel

Description
Write a C program to check vowel
Vowel check is a simple program to check whether a character is a vowel or not.
Input  : a
Output : Vowel

How to contribute

Save the solution in CheckVowel.c file
Add CheckVowel.c file in check-vowel folder

!assign


[Fusion] Execution Error on 13.6.0-preview.7

Is there an existing issue for this?

[X] I have searched the existing issues

Product
Hot Chocolate
Describe the bug
I'm getting this error:
{
  "errors": [
    {
      "message": "Unexpected Execution Error",
      "extensions": {
        "message": "Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index')",
        "stackTrace": "   at System.Collections.Generic.List`1.get_Item(Int32 index)\r\n   at HotChocolate.Fusion.Planning.QueryPlannerHelpers.GetBestMatchingSubgraph(FusionGraphConfiguration configuration, IOperation operation, IReadOnlyList`1 selections, ObjectTypeMetadata typeMetadataContext, IReadOnlyList`1 availableSubgraphs)\r\n   at HotChocolate.Fusion.Planning.Pipeline.ExecutionStepDiscoveryMiddleware.CreateExecutionSteps(QueryPlanContext context, Queue`1 backlog, ObjectTypeMetadata selectionSetTypeMetadata, IReadOnlyList`1 selections, ISelection parentSelection, Boolean preferBatching)\r\n   at HotChocolate.Fusion.Planning.Pipeline.ExecutionStepDiscoveryMiddleware.Invoke(QueryPlanContext context, QueryPlanDelegate next)\r\n   at HotChocolate.Fusion.Planning.QueryPlanPipelineBuilder.<>c__DisplayClass5_1`1.<Use>b__1(QueryPlanContext context)\r\n   at HotChocolate.Fusion.Planning.QueryPlanner.Plan(IOperation operation)\r\n   at HotChocolate.Fusion.Execution.Pipeline.OperationQueryPlanCompiler.OptimizeOperation(OperationOptimizerContext context)\r\n   at HotChocolate.Execution.Processing.OperationCompiler.CreateOperation(String operationId, OperationDefinitionNode operationDefinition, ObjectType operationType, DocumentNode document, ISchema schema)\r\n   at HotChocolate.Execution.Processing.OperationCompiler.Compile(String operationId, OperationDefinitionNode operationDefinition, ObjectType operationType, DocumentNode document, ISchema schema, IReadOnlyList`1 optimizers)\r\n   at HotChocolate.Execution.Pipeline.OperationResolverMiddleware.CompileOperation(IRequestContext context, String operationId, OperationDefinitionNode operationDefinition, ObjectType operationType)\r\n   at HotChocolate.Execution.Pipeline.OperationResolverMiddleware.InvokeAsync(IRequestContext context)\r\n   at HotChocolate.Execution.Pipeline.OperationComplexityMiddleware.InvokeAsync(IRequestContext context)\r\n   at HotChocolate.Execution.Pipeline.OperationCacheMiddleware.InvokeAsync(IRequestContext context)\r\n   at HotChocolate.Execution.Pipeline.DocumentValidationMiddleware.InvokeAsync(IRequestContext context)\r\n   at HotChocolate.Execution.Pipeline.DocumentParserMiddleware.InvokeAsync(IRequestContext context)\r\n   at HotChocolate.Execution.Pipeline.DocumentCacheMiddleware.InvokeAsync(IRequestContext context)\r\n   at HotChocolate.Execution.Pipeline.TimeoutMiddleware.InvokeAsync(IRequestContext context)\r\n   at HotChocolate.Execution.Pipeline.ExceptionMiddleware.InvokeAsync(IRequestContext context)"
      }
    }
  ]
}

When loading this FGP file (Make sure to unzip it before loading):
Gateway.zip
And executing any valid query to the schema.
Steps to reproduce

Create a new Fusion Gateway
Load from File (Above)
Query any valid query

Relevant log output
No response
Additional Context?
Error does not happen on 13.5.1
PS: sub-graphs doesn't need to be running for this problem to occur
Version
13.6.0-preview.7

Thanks for reporting!

I cannot reproduce this error. Important to know here is that we are changing the fusion graph configuration at the moment and the tooling must match the gateway. Also you need to fully recompose. When running your gateway config in our unit tests it just works.

Sorry, found the issue in your graph ...
we will have a fix with the next preview.


CPUManager word variants in Control CPU Management Policies on the Node

There is some variants of the word "CPUManager" in this document: Control CPU Management Policies on the Node

CPUManager
CPU Manager
CPUManager

IMO in the context they are same thing that is a mechanism to control CPU affinity, not a K8s resource or others (CMIIW),
so should we update the doc to change these variants to a consistent name?

/kind doc
/language en

/kind documentation

/assign

/sig docs
/priority backlog
Any updates for this issue? @haripriya9647

/triage accepted

I was waiting for the triage/accepted label @aj11anuj
I will add pr


Improve footer of divine activities page

https://akshitagupta15june.github.io/Moksh/story.html
add the footer of augmented reality meditation page to the divine activities page https://akshitagupta15june.github.io/Moksh/meditation.html

I am interested to work on this issue under Quine. So please assign me this task to me.

Can I work on this issue ?

Hey @a

https://akshitagupta15june.github.io/Moksh/story.html
add the footer of augmented reality meditation page to the divine activities page https://akshitagupta15june.github.io/Moksh/meditation.html

hey, i have gone through the project and understand the issue, can you assign this issue to me under GSSOC'23


Add Replaceable Monitoring Cluster Definition

📖 Description
This PR adds the replaceable monitoring cluster definition (replaceable-monitoring-cluster.xml) zap template.
🔎 Details

Template is the specification of the open PR (https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/6487)
Template is added to various lists in various files (see #26223)
Code generation ran with zap_regen_all.py

🧪 How Has This Been Tested

chip-tool was successfully built using ./scripts/examples/gn_build_example.sh examples/chip-tool out/chip-tool

🔗 Links/References

closes #26223

Please separate out re-ordering of things from functionality changes. These should be 2 separate PRs.

Done ✅


Change how LWJGL path is set. Improves compat with newer JRE

We had our own NativeHelper code to try to manipulate private variables on the classloader. It didn't work very well across JRE versions, as seen in #3742. Fortunately, LWJGL has its own mechanism for dealing with library paths.
In specific, this PR addresses the error that appeared in logs as:
ERROR o.terasology.utilities.NativeHelper - Couldn't link static libraries.
java.lang.NoSuchFieldException: usr_paths
at java.base/java.lang.Class.getDeclaredField
at org.terasology.utilities.NativeHelper.addLibraryPath

How to test

Run Terasology in a Java 8 environment.
Run Terasology in a Java 14 environment.

If it still works under Java 8, and it gets to the main screen in Java 14, that's better than it was before!
The other things in natives/ could potentially be affected by this. That includes what, OpenAL, controller support, and Lua?

Sound is working, and it looks like it's using the bundled libopenal. I set another property that took care of libjinput.
Does lua work? So far I have managed to start a world with KComputers in it that hasn't crashed, but I don't know if that proves lua works or not.

Okay. I give up, for the moment, on testing lua. Until I see an easier test plan.
Apparently OpenComputers will fall back to LuaJ if JNLua can't load, so I'm not sure if running a minimal Computer would prove anything? I'd have to specifically test whether the suspend/persistence features work.
or have a specific idea of what to look for in the logs. It might log something when it succeeds or fails in loading the lua engine, but I'm not sure.
Given that the stuff that uses JNLua already needs special launch-time arguments to work (for the permissions requirements), I think testing JNLua is not-release-blocking. JNLua users could do their own path settings at launch time, alongside their other settings.

Okay @Cervator, I've added examples of how to set the native library path.

for gradle, in :facades:PC:game
for IDEA, in the permissive security run config. Requires going to Settings / Path Variables / TERASOLOGY_NATIVES to set.

having to manually set that variable value is a stopgap, but streamlining how the build handles the natives directory is for another time.

Okay tested around a bit more and updated KComputers with extra details including the potential workarounds: https://github.com/Terasology/KComputers/commit/7b2c93cf2324bb4b270da8ca504841d372433cb9
Still thinking we should remove from lineup right now, but at least this way it'll be more doable to play with it anyway, and hopefully later recover it fully and move forward :-)
Thanks @keturn


[requirements] update requirements and add script for updating

to: @ryandeivert or @austinbyers
cc: @airbnb/streamalert-maintainers
size: small
resolves #439
Background
As @austinbyers pointed out in the relevant issue, many deps were out of date and it was unclear which were top level.
Changes

Add a top level requirements
Add a script to manage and update reqs
Update the pylint script so I know it has started

Testing
Ran all the testing scripts locally

Coverage remained the same at 95.687% when pulling 7c82c3dcfe5509493da3da62c3cf7fe328fddee4 on jacknaglieri-top-level-reqs into bd8b80f7d3417921e5ea69aaf93f1d2d5915d012 on master.

Coverage remained the same at 95.687% when pulling 83e84aa797024e7bf5540a8f200548966c4e1db5 on jacknaglieri-top-level-reqs into ef9941939c6b7db279d15bf2efde980453e107ed on master.

Coverage remained the same at 95.687% when pulling c6eba01c181c5093f8f0101d6cda96d416e9f8fd on jacknaglieri-top-level-reqs into 75722f64cbca0628321b5c882ff825950b51039c on master.

Thanks for making the change! A few comments:

I would recommend specifying the version of every native package you use in requirements_top_level.txt. For example, specify which version of cryptography you are using so that the virtualenv is consistent with the natively compiled package we provide.
virtualenv need not be in the requirements. You need virtualenv installed before you pip install the requirements
I'm a bit confused by the requirements.txt - why is there a comment halfway down? It should just be the output of pip freeze

I can make these changes on Friday if they are agreeable

I would recommend specifying the version of every native package you use in requirements_top_level.txt. For example, specify which version of cryptography you are using so that the virtualenv is consistent with the natively compiled package we provide.

Definitely, feel free to make this change.

virtualenv need not be in the requirements. You need virtualenv installed before you pip install the requirements

That's fair, feel free to make this change too.

I'm a bit confused by the requirements.txt - why is there a comment halfway down? Why not just make it the output of pip freeze?

This happens automatically via the script that was checked in (inspired from the link you provided in the issue)


Forward only valid properties to DOM nodes

See https://facebook.github.io/react/warnings/unknown-prop.html
Fixes #7

It seems this has been fixed in commit e82ddd33. Would it be ok to close this PR and the related issue to clarify this is not an issue anymore?


Update git in base image to prevent 'The repository will be downloaded using the GitHub REST API'

Description
As discussed on Discord;

jonathanperret
Anybody else noticed this message when building on GitHub actions with unityci/editor (using 0.8.0 images)?

Getting Git version info
  Working directory is '/__w/PecoPeco/PecoPeco'
  /usr/bin/git version
  git version 2.17.1
Deleting the contents of '/__w/PecoPeco/PecoPeco'
The repository will be downloaded using the GitHub REST API
To create a local Git repository instead, add Git 2.18 or higher to the PATH

Context

Ubuntu 18.04's git current version is 2.17.1
We might want to move from Ubuntu's freezed version and move to at least 2.18
git is installed here: https://github.com/game-ci/docker/blob/0be9208/base/Dockerfile#L29

Would this be fixed by updating to Ubuntu 20.04 for our images?

It would, but we used to start with 20.04 as seen here:
https://github.com/game-ci/docker/commit/1918fe864767801eb41b22ebb241602ba2647890#diff-b093d34eec92adf6e0cc6e445edb7147ea45d25fb0c98f2f84a60f5aae6f34f7R1
And we had to downgrade since officially supported ubuntu version for Unity is is still 18.04 🤦
ref: * 438d326 - Change to ubuntu 18.04 (#44) (3 months ago)

And we had to downgrade since officially supported ubuntu version for Unity is is still 18.04 🤦
Just FYI Ubuntu 20 is now officially supported by Unity 2020.2. https://docs.unity3d.com/2020.2/Documentation/Manual/system-requirements.html
Not sure if it's worth maintaining differently built images for different Unity versions though.

Yea it's officially support for the latest unity versions. We tried to go Ubuntu 20.04 but it was creating new bugs for older versions of Unity.
If we have reason to believe that this git error only occurs the latest versions of Unity we might consider creating a second base image for newer versions. Otherwise it will be far simpler to fix the problem across the board, at least for now.

I'm using the latest LTS (long term support) version of Unity which is currently 2019.4.19f1. If this version is only supported on Ubuntu 18.04 I think that should remain the base image.
Here's what I'm doing at the start of my job to update the Git version :
apt-get -qq update && apt-get -qq -o=Dpkg::Use-Pty=0 --no-install-recommends install gnupg1

echo 'deb http://ppa.launchpad.net/git-core/ppa/ubuntu bionic main' > /etc/apt/sources.list.d/git.list
apt-key adv --keyserver keyserver.ubuntu.com --recv-keys E1DD270288B4E6030699E45FA1715D88E1DF1F24

apt-get -qq update && apt-get -qq -o=Dpkg::Use-Pty=0 --no-install-recommends git

The more conventional way to add a PPA is to install software-properties-common and use its add-apt-repository script but I felt it added to much bloat so I just added the minimum necessary to install the additional Git PPA.
One more note: the warning about Git 2.17 only appears if you run the @actions/checkout action in the unityci/editor image. This happens for me because I added container: { image: unityci/editor:2019.4.19f1 } at the job level, for simplicity, so all of my job's steps run under the unityci/editor image. I guess if you only use the Docker image through actions such as https://github.com/marketplace/actions/unity-builder , the @actions/checkout action will run on the host and probably have a recent enough Git version.

Thanks for adding the additional information. I was wondering where the whole GitHub Rest API magic came from :)
In light of this information I would argue that the change should be out of scope for now. As it is indeed not necessary for any of the workflows in the documentation. Adding the latest git above the OSes stability level could technically cause other problems on the way.
Let me know if this would be an acceptable outcome.

That's fine by me, as mentioned I already know how to update Git in the container. 👍


What kind of feature is suitable for this scenery? (deal with synonym)

Train data: Who is Michael Jordan?  and Who is Air Jordan?  are one right match.
The test data is Who is Jumpman Jordan?  and Who is Air Jordan?
What feature should I add for training? Can the model capture the match info for this case?
Thank you！！！
@Wrosinski

Best to take a look at the Extraction - Kernel Features 14.05.ipynb notebook. Features capturing straight text similarities (similar words etc) are mostly there.


Gradle endlessly starts OpenJDK Platform Binaries

Expected Behavior

Running Gradle task 'assembleDebug'...  to finish
Current Behavior

It just spawns OpenJDK Platform Binaries until it takes up all the RAM
Context

I'm trying to start my application. The issue occurs on a newly created project on Android Studio + A newly created Flutter app on VS Code as well
Steps to Reproduce

The issue occurs on a newly installed Windows 10 with default settings
Your Environment

Build scan URL:
After completely taking up the RAM the following error gets printed
Error:
FAILURE: Build failed with an exception.

* What went wrong:
Unable to start the daemon process.
This problem might be caused by incorrect configuration of the daemon.
For example, an unrecognized jvm option is used.
Please refer to the User Manual chapter on the daemon at https://docs.gradle.org/6.7/userguide/gradle_daemon.html
Process command line: E:\Programs\Android Studio\jre\bin\java.exe -Xmx1536M -Dfile.encoding=windows-1252 -Duser.country=US -Duser.language=en -Duser.variant -cp C:\Users\sunny\.gradle\wrapper\dists\gradle-6.7-all\cuy9mc7upwgwgeb72wkcrupxe\gradle-6.7\lib\gradle-launcher-6.7.jar org.gradle.launcher.daemon.bootstrap.GradleDaemon 6.7
Please read the following process output to find out more:
-----------------------
Error occurred during initialization of VM
Could not reserve enough space for object heap

* 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
Running Gradle task 'assembleDebug'...
Running Gradle task 'assembleDebug'... Done

Downgrading Gradle to 6.0 and Gradle Build Tools to 3.6.0 Solved the issue

Would you please be able to share the error with the debug options enabled as suggested in the output?
* 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.

See also Configure the JVM memory to learn about increasing the memory for your builds. Depending on your build scripts, this may be necessary.
If the problem persists, it would be great to get a heap dump  to see if there are any leaks.
Also, have you tried running your build using a more recent version (e.g. Gradle 7.0)?

Closing for now. @Sunney-X please let me know if you got more information and we can reopen


Branch lagom-1.5.x is stale

@renatocaval branch lagom-1.5.x and may probably be removed. Want to do the honours?

Done


The archive button is confusing

Since the archive process is the action of applying rules, perhaps it should be clearer that this is what's going on when you click the archive button? Certainly it's strange when the archive icon is present inside the archive (given in theory your files are already in the archive).
Maybe the icon should be changed, and the operation be renamed to something like, 'Run rules'?

I've updated this to use the wand icon, which I think is a little clearer, and certainly makes a lot more sense when it appears when you've selected items that are already in the archive. I've also added a tooltip.

Shipped in https://github.com/jbmorley/fileaway/releases/tag/macOS_0.4.0.


KeepAlive workaround causes Geofence to malfunction

Hi
After installed this addon my Android phone has started to show aiohue_xxxx user in location using devices. This means that geofencing features (like turn lights on when reacing home) is not working while one device is always at home.
If I remove this 'aiohue_xxx' device from Android phone /Location based devices it will within minutes come back.
Is there somewhere a plugin setting wich related on this which can be turned of?
BR
Janne

Aiohue is a library. It's used by an application that you're using which is not setting a correct user string.

Actually, this is caused by the keep alive workaround but it shouldn't mess with the geofencing. At least it didn't so far.

I can confirm that the aiohue user is blocking geofence from working.
I've tried to replicate this by turning off my Home Assistant instance, removing the aiohue user, and running a 'comming home' route via the route mocking application 'lockito' on android. I've tried this 5 times and all 5 times the 'comming home' scene was activated about 100 meters before my house.
After starting the Home Assistant instance it stopped working completely.
I understand that this may be a case of wrong implementation of the library by Home Assistant, but maybe someone else can take a deeper look in this.

I don’t think that we should re-open this issue, because it doesn’t feel like it adequately describes the issue. We could open a new one, if it is in the lib.
@marcelveldt is the workaround in Home Assistant or in the lib?

I think the workaround is defined here: https://github.com/home-assistant-libs/aiohue/blob/58b24e2ad0dd212f34a780689d0d90aa050a80dd/aiohue/v2/controllers/events.py#L284
Of course i can open a new issue if that's desirable.

Correct, that is the workaround. Problem is that the workaround is still needed. I'm kind of surprised you're the first one in all these months that has noticed this so I guess the geofence in Hue is not so popular within the HA users ;-)
I'll inform with Signify if there's any sight on the solution where there is a keep alive message sent by the EventStream.

Yeah i'm pretty confused myself. I haven't really found any other threads about this issue except this reddit post.
In the PR that introduced this error https://github.com/home-assistant-libs/aiohue/pull/93 you mentioned something about feature-toggling this function. Is it perhaps possible to introduce this so i can at least disable this keep-alive feature when i'm disabling the hue integration?

Why not use geofencing from within HA instead of Hue ?
Since you're literally the only/first person mentioning this I'm not going to create a feature toggle, simply because that is a lot of work to implement into the entire path.
I'd rather fix the underlying issue, detection if the connection is still alive.
I'll have a look if we can adjust the workaround so the fake device is not at home.

Why not use geofencing from within HA instead of Hue ? Since you're literally the only/first person mentioning this I'm not going to create a feature toggle, simply because that is a lot of work to implement into the entire path.

I understand you'd want to fix the underlying issue. The reason I'm not using HA geofencing instead of hue is that i'm using more scene's of Hue that are not available in HA like mimicking presence, and i want to have one place to manage my automations for a certain product because otherwise i'll lose track of where all my automations are.

Since you're literally the only/first person mentioning this

I have this issue too. I have two users setup in Hue app to turn off all Hue lights. I was relying on this and didn't realize it wasn't working for a while.

Hi, I have the same problem with geofencing Hue App with aoihue_ . I think, Noth Pole as Location would help us.
It would be graet, if you can adjust the workaround.

Disabling the Hue integration doesn't stop it doing this either.

I will take a look at this today, I'll keep you informed.
@vjpr disabling the Hue integration will work if you also delete the aiohue user in the app

Hi
Should this work now? I tried to re-install this and geofencing is still not working. I used Lockito to simulate going a way and arriving home and without hue integration geofening works ok. If I activate this (aiohue_ appears as a device on android app) geofencing stops wroking.
BR
Janne

No, it is not released yet

No, it is not released yet

Any news when it will be released? I have had to uninstall this because I need phone's gefencing to work.

I think that many people don't even know that the problem is caused by home assistants, which is why there are so few reports. It also took me some time to find out.
Is there a solution in the meanwhile?

The fix should be in HA release 2023.2 which beta is released today

does that mean there is a geofence entity too? (in the old days, we had a device_tracker for the Hue app device)

The fix should be in HA release 2023.2 which beta is released today

It has the same behavior

I'm also struggling with this issue.
I'm only noticing it now, because I had a roommate who never left the house for the last 9 months or so. They have recently moved out, and I am now noticing the issue.
For what it's worth, the only reason I think more people aren't noticing this issue is because Geofencing has been broken in iOS for the last few years meaning the Hue app on iOS doesn't reliably (if ever) trigger the lights as it should. Apple doesn't care to fix this issue, so most iOS users simply don't use the feature. I personally have since switched to Android because of this, and all has been well until this bug came up.
Considering Geofencing not working was a big enough bug that made me, a life-time iOS user, and once Apple employee change platforms, it should indicate how important it is for this to work properly.
This is a deal breaker if it's not fixed soon.

Aaannddd, now it's working. I don't know why. But it wasn't working for the last week and I tried a few different things. Now it seems to be working, and I'm not complaining, so long as it doesn't stop working again.


⚠ Adacta-fintech.com has degraded performance

In fd76693, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 200
Response time: 1687 ms

Resolved: Adacta-fintech.com performance has improved in f1f0a03 after 11 minutes.


\simplify{ {new data type} } subs in the JME version of {new data type}, not the TeX version

For example, if w = 001 is a codeword, $\simplify{{w}}$ gives $codeword("001",2)$. It should give the TeX version, $\mathtt{001}$.

That's not what's happening! The \simplify command does jme.subvars on the argument, which renders subbed-in expressions as JME. The whole expression is then run through exprToLaTeX.
Because there's no codeword literal, it's rendered in JME as codeword(_,_). I just need to add code to render the codeword function in LaTeX.


Feature/add dashboard and onboarding

Component adds:
Client Dashboard:
When a user who is a client logs in they'll be routed to the client dashboard.
Groomer Dashboard:
When a user logs in who is a Groomer they'll be routed to the Groomer Dashboard.
Onboarding Form:
If a user logs in and does not have an assigned role they'll be allowed to select one as well as enter any other needed information.

Agreed with Elijah, but we could always refactor later. Thank you for your work!


Add VSWhere support

As discussed in #1369 , please assign to me and I am going to try to get this added this weekend.

@devlead I already have a lot of this stubbed out.  I need to think about the use cases that are supported by the tool and decide what to add.  I may just add getting the installation path as that is the most needed.

@phillipsj getting the FilePaths first sounds good to me.

Here is currently what I have, I am still thinking through the implementation.  Feedback welcomed.
https://github.com/phillipsj/cake/tree/GH1517-add-vswhere-support

@phillipsj great that you took this on so quickly. I think this is on critical path for cake users who need to use the new MSBuild 15 BuildTools on their CI servers. (It is for me anyway).
Open a [WIP] PR and lgtm?

That is a great suggestion, I will do it.
On Mar 5, 2017 12:18 PM, "Damian Hickey"<EMAIL_ADDRESS>wrote:

@phillipsj https://github.com/phillipsj great that you took this on so
quickly. I think this is on critical path for cake users who need to use
the new MSBuild 15 BuildTools on their CI servers. (It is for me anyway).
Open a [WIP] PR and lgtm?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cake-build/cake/issues/1517#issuecomment-284244561,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABfthUAZceelph433wUKh4mpkcjBU4F6ks5riu5XgaJpZM4MTDu6
.

@damianh The pull request is here now.

@devlead so, this can be closed, right?

@gep13 correct, closed by #1520

I wonder if this changes anything?
https://blogs.msdn.microsoft.com/heaths/2017/04/21/vswhere-is-now-installed-with-visual-studio-2017/
The package would only have to be downloaded if %ProgramFiles(x86)%\Microsoft Visual Studio\Installer\vswhere.exe and %ProgramFiles%\Microsoft Visual Studio\Installer\vswhere.exe do not exist.

@jnm2 It would just change how the tool is found.  May need to add a custom resolver that looks for it in that default location first.

vswherewhere


Update data type for bounding box coordinates to float32

This pull request updates the data type for bounding box coordinates to float32. This change ensures compatibility with Apple M3 pro Chips #656

In the future, we should add test for MPS chips (Apple Silicon). Exploring current practices, it appears that many tests are conducted using purchased cloud or local instances.

@ethanwhite did you request changes? I can see your review is blocking.

@ethanwhite did you request changes? I can see your review is blocking.

I was thinking about the additional uses of .astype(float) in utilities.py so we don't get caught by this again with the helper functions, but I'll approve this and open a new issue for those.

Thanks for the contribution @hesamossanloo! Since @bw4sz has approved this as well I'm merging.

Cheers @ethanwhite, @bw4sz :) I should thank you for keeping this great project running.


Test Issue created at Wed, 14 Apr 2021 15:31:38 GMT

Test issue body  Wed, 14 Apr 2021 15:31:38 GMT

Comment on issue at Wed, 14 Apr 2021 15:31:42 GMT


Add Reference APIs in doc

It would be better to automate the integration of the API reference to mkdocs.yaml. For this, please see
https://niklasrosenstein.github.io/pydoc-markdown/getting-started/
and
https://niklasrosenstein.github.io/novella/

pydoc-markdown:
https://niklasrosenstein.github.io/pydoc-markdown/
Documentation recommend to install from pipx:
$ pipx install novella
$ pipx inject novella pydoc-markdown[novella]

For this firstly we need to install the pipx :
python3 -m pip install --user pipx

When I install the package:
pipx install novella
Not installing to existing directory '/home/devfullstack/.local/pipx/venvs/novella'. Pass '--force' to force installation

Because:
The default binary location for pipx-installed apps is ~/.local/bin. This can be overridden with the environment variable PIPX_BIN_DIR.

pipx's default virtual environment location is ~/.local/pipx. This can be overridden with the environment variable PIPX_HOME.

I also check some of the issues and PRs:
pipx install --force not overwriting correctly: https://github.com/pypa/pipx/issues/582
"pipx install --force --python ..." doesn't change Python version: https://github.com/pypa/pipx/issues/447

My suggestion is that: We can merge this PR and create a separate ticket for it so we will work on it.


[MEDIEVO OSCURO] Weapon damage changed to a text field instead select class:"dtype"

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Hey @VJRolmasters
Looks like this PR is empty - did it upload the correct files?

Character Sheet Info Roll20 Internal Use only.

Hi BronsonHall
Yes, I uploaded the wrong file.
Now it's fine if you can check it.
Thank's


easyAutocomplete is not a function

I have jquery included within the  tag
And then, I have this in my  (I know, not the best practice, but for my CMS usage purpose, it's easier for me)
<script src="jquery.easy-autocomplete.min.js"></script>
<script>
$(document).ready(function(){
var options = {
url: "villes_france.json"
};
 $("#city-search").easyAutocomplete(options);
});
</script>
The urls are good and tested
When I load my page, I have the error I put in the title

$(...).easyAutocomplete is not a function

You need to include jqueryui.


Add new extension

but doesn't provide any query translations for methods/members on DateOnly/TimeOnly

Hmm...  Never thought of that, I have created https://github.com/ErikEJ/EFCore.SqlServer.DateOnlyTimeOnly/issues/1 and will investigate

You can check out https://github.com/dotnet/efcore/pull/24989 for the translations (and tests!) done for SQLite, I recommend looking at the PG provider too, it should have more extensive query translations.

Thanks  I will do that


How to control the counts of events in schedule?

https://github.com/ClickerMonkey/dayspan/issues/20

https://github.com/ClickerMonkey/dayspan/issues/18


drt: coverity fixed (1)

There are many other coverity issues to be fixed, but I decided to start with these ones to avoid a huge PR.

Please fix the errors before I review

Please fix the errors before I review

Fixed.


[material-ui][IconButton] disableRipple on MuiButtonBase doesn't disable ripple effect on IconButton

Steps to reproduce
Link to live example: https://codesandbox.io/p/sandbox/friendly-sunset-w3d9ft
Steps:

Create a theme that disables the ripple effect globally through MuiButtonBase as described in the documentation.
Add IconButton components.
Also create a theme that disables the ripple effect directly on MuiIconButton.
Add IconButton components here too.

Current behavior

In the first example, the ripple effect won't be disabled on IconButton components. The prop is completely ignored, where in v5 this would disable the ripple.
For the second example, now not only is the ripple disabled, but so is the hover effect.

Expected behavior

Disabling ripple on MuiButtonBase disables it for all button types, as v5 used to do.
Disabling ripple on MuiIconButton does not disable any hover effects, as this property does when applied to MuiButtonBase. This issue is also present in v5.

Context
No response
Your environment
No response
Search keywords: ripple icon button

Looks like this regression is from https://github.com/mui/material-ui/pull/43271
This is how disabledRipple got resolved in v5
check for IconButton prop (if not defined) -> check for MuiIconButton theme default props (if not defined), check for MuiButtonBase theme default props.
But this behaviour was changed here, disabledRipple was explicitly set to false if it's not defined, so check couldn't go for step 3.
I think we can solve this like below, but i'm not sure how efficient solution is

const IconButton = React.forwardRef(function IconButton(inProps, ref) {
  const props = useDefaultProps({ props: inProps, name: 'MuiIconButton' });
+  // eslint-disable-next-line material-ui/mui-name-matches-component-name
+  const buttonBaseProps = useDefaultProps({ props: inProps, name: 'MuiButtonBase' });
  const {
    edge = false,
    children,
    className,
    color = 'default',
    disabled = false,
 disableFocusRipple = false,
- disableRipple = false,
+    disableRipple: disableRippleProp,
    size = 'medium',
    ...other
  } = props;

+  const disableRipple = disableRippleProp ?? buttonBaseProps.disableRipple ?? false;

I am taking this up as my first issue. Will be updating the thread with my progress.

@sai6855 Let's revert the change in #43271 and fix the variants to:
variants: [
      {
-        props: { disableRipple: false },
+       props: (ownerState) => !ownerState.disableRipple,
        style: {
          '&:hover': {
            backgroundColor: theme.vars
              ? `rgba(${theme.vars.palette.action.activeChannel} / ${theme.vars.palette.action.hoverOpacity})`
              : alpha(theme.palette.action.active, theme.palette.action.hoverOpacity),
            // Reset on touch devices, it doesn't add specificity
            '@media (hover: none)': {
              backgroundColor: 'transparent',
            },
          },
        },
      },

This bug also applies to plain normal Button's as well.


fully functional application. Bug testing will continue

Added proper icons and weather data to 5 day forecast. Code could use slight refactoring. Application is otherwise fully functional.


Add generic JobService

Short Description
This adds a generic job service that can be used with the JobServiceFactory to get a service for the different jobs. The jobs are wired up using injection.

[ ] Removing the already implemented Services

Changes

Add generic job + interface
Add tests for the generic job
Add the service factory that correctly wires the different parts of a service together

Feedback

Which parts of the code would you like feedback on?
Questions that are open are also good (e. g. "Do you see any tests missing for class x?")

How to review this Pull Request
Link to Confluence (internal)

I think that are several solid principles could be violated.
Anyway, the layered architecture shouldn't be affected, what means, the application layer shouldn't know anything about the api layer.


inputField method

There exist methods such as objectField and mutationField that allow iterative additions of fields to existing types.
Is there a way to do the same thing for inputType such that one could call a method inputField? I cannot find it in the API, is it not implemented or just done differently?

Input types can't be changed after they are defined because the definition defines the shape (type) of the input type.  If you later add more input fields, the types for the input type when used as an arg wouldn't be correct.

Going to close this, but let me know if you have any more questions!


Add Fumec University in 'br/' directory out of 'edu/'

Fumec University already exists, it is only in edu/ directory. The email of the students finishes with @fumec.br and not @fumec.edu.br probably this one is destined for professors.
Official Site: http://www.fumec.br

Do I need to add something? Could you please check this Pull Request? Thank you all.

@luisbt07 Thank you for your patience while waiting for a review and processing your request. Pull request merged. Thank you!