Marshalling char16_t* or wchar_t*

Marshalling char16_t* and wchar_t* produces the wrong marshalling type.
Windows, MSVC++ to C#.
C++ signature:
void Log(const char16_t* str)
will produce:
void _LogDelegateHook(global::System.IntPtr __instance, [MarshalAs(UnmanagedType.LPUTF8Str)] string str)
Here, LPUTF8Str is wrong. char16_t is UTF16-LE. Tests confirm using LPWStr would fix it.

We've had a pending PR that should help with this: https://github.com/mono/CppSharp/pull/983
It just needs a bit of a push to get it over the line.

@tritao I already took care of this issue as far as I know.
https://github.com/mono/CppSharp/blob/9b69f147059cebc487353997680053ec7e39e253/tests/CSharp/CSharp.cpp#L1846
The problem is that all my contributions are not available in the latest nuget package from nuget.org.

@josetr
When will you release the new version? It's been a long time since the last release

@josetr
When will you release the new version? It's been a long time since the last release

Soon. We had to fix some issues that came up after we upgraded everything to .NET Core to get this far and we also had to make some changes to support windows / linux / macos in a single package.
It would be nice if you can manually try the new nuget package 👍 .


Moving from dcmtk to pynetdicom3

Hi,
thanks for porting pynetdicom to python3. I am currently using dcmtk (called from python) but would love to use pynetdicom3.
What I am not sure is how I would translate the following dcmtk findscu command to pynetdicom3 equivalent
findscu -v -S -k 0008,0052=SERIES -aec <AEC> <IP> -aet <AE_TITLE> -k PatientName -k PatientBirthDate -k PatientID -k PatientSex -k StudyID -k StudyDate -k Modality -k AccessionNumber -k BodyPartExamined -k StudyDescription -k SeriesDescription -k SeriesNumber -k InstanceNumber -k ReferringPhysicianName -k InstitutionName -k StudyInstanceUID -k SeriesInstanceUID -k StudyDate=20160101 -k SeriesDate=20160101 -k SeriesTime=000000-075959

So it does a Series level query and retrieves those attributes (all -k ones). The filter is set on a specific time. The basic example from (here)[https://github.com/scaramallion/pynetdicom3/blob/master/docs/scu_examples.rst] is running.
Thanks a lot for your time. If there are some easy fixes (documentation, code) I can try to contribute.

I think the general approach would be to:

Set the logging to verbose mode:

pynetdicom_logger = logging.getLogger('pynetdicom3')
pynetdicom_logger.setLevel(logging.INFO)

Create an AE and associate with the peer.
Read in the dataset using pydicom and add/modify/delete the tags you want to override.
Send a C-FIND request to the peer using study root info model:

response = assoc.send_c_find(dataset, query_model='S')

Which yields pynetdicom3.SOPclass.Status, pydicom.dataset.Dataset
5. And finally, release the association.
You might be able to get the general idea from apps/findscu, but keep in mind that it's really bare-bones at the moment (I mostly use it for testing the C-FIND operations).

Thanks for your answer.


PF-1352: Add git as a passthrough app

https://docs.google.com/document/d/14sn6ValS3Mtq9HDlYmUioJ3RzpDRLwsp5Yfby63JVKE/edit?usp=sharing

Also add instructions & examples to the README file.


[Bug] Uses up tons of disk space very fast

Checks

[X] I confirm that I have searched for existing issues / pull requests before reporting to avoid duplicate reporting.
[X] I confirm that I followed the MCDC Docs and read the Changelog, but this problem still exists.
[X] I confirm that I noted that if I don't follow the instructions, the issue may be closed directly.

Java Version
17.0.7
Minecraft Version
1.20
MCDiscordChat Version
2.2.0
Description
After about 2 days I have noticed that my disk space has increased almost half a GB from this mod. It is an amazing mod but I assume that this feature of taking up disk space is not intentional. Today from simply staring the server it rose 1 KB which might not seem like a lot but adds up. Is there any way to prevent this or to get rid of the extra files. What files are they?
To Reproduce
Simply use the mods installation instructions, then monitor disk space.
Log
N/A
Config
I dont think that the config file would assist but If it does I will post it in replies

??? Can you check where are the files located?

Thats part of the problem I dont know where they are or even why they are getting created

MCDC does not create any additional files other than the mcdiscordchat.json config file.

Today from simply staring the server it rose 1 KB which might not seem like a lot but adds up.

This increase sounds more like server logs at /log/, normal.

my disk space has increased almost half a GB

It may be someone in your server exploring new chunks?

Ok, you're welcome.


Unable to detect pedestrians

Hello,
I'm working on a deepsort implementation of this repo, using pose bbox info and features from backbone, but the model is unable to detect pedestrians consistently. Weird thing is that, when I use another detector, like YOLOv3, and give the output bbox of that to this model, crop the detected areas of the image, it works beautifully. Any ideas? Is the kernel size of conv layers too big perhaps?

Hi! I believe the pedestrian height to image height ratio is too small for the default parameters. You can increase network input size with --height-size option. Try to set it to 512 or 1024, and see if it works.

Hi! I believe the pedestrian height to image height ratio is too small for the default parameters. You can increase network input size with --height-size option. Try to set it to 512 or 1024, and see if it works.

Thanks for the reply. Does the network need to be retrained for this?

Thanks, size 512 is able to detect even the smallest pedestrians.

Great, that it helped!


Add support for injecting values from environment variables

This PR introduces a CUE attribute in the format @timoni(env:[string|number|bool]:[ENV_VAR_NAME]). The attribute can be placed in bundles next to any field to set its value from the environment at runtime.
Motivation
As shown here, a Timoni bundle can be made out of multiple files, so we can have a CUE file that contains some values generated at runtime from various sources e.g. an encrypted store, CI vars, etc. While this approach works, it forces uses into storing or generating CUE files in CI, which can be cumbersome. By introducing support for environment variables, users can easily inject values without having to modify their pipelines, as most workflow engines can set env vars before running a command.
Another reason for implementing environment variables is offering a similar feature to Helm's --set-file and --set-json. Some modules may require users to provide TLS certs, or app config files (JSON, INI, XML). Instead of forcing users into embedding the file text content in a bundle, the content can be stored in an env var and Timoni will inject it at runtime in the bundle as  multi-line string, preserving the original indentation.
An alternative to using the environment to inject file contents, would be adding an attribute for reading files from disk e.g. @timoni(import:string:ssh/id_ecdsa.pub). But then we'll have to deal with relative paths (to the bundle or to the execution root?), symlinks, path traversal protection, etc. I think for now, we can ship the environment solution and see what issues users will have with it, then decide if we really need a file reader.
Example
appName: string @timoni(env:string:APP_NAME)

bundle: {
	apiVersion: "v1alpha1"
	name:       "\(appName)"
	instances: {
		"\(appName)": {
			module: {
				url:     "oci://my-registry/modules/app"
				version: "1.0.0" @timoni(env:string:APP_VER)
			}
			values: {
				// required string (can be multi-line)
				sshKey: string @timoni(env:string:SSH_KEY)
				// optional boolean (defaults to false)
				isAdmin: false @timoni(env:bool:IS_ADMIN)
				// required number
				age: int @timoni(env:number:AGE)
			}
		}
	}
}

Export the env vars and run the timoni bundle apply command.
EXPORT APP_NAME="my-app"
EXPORT SSH_KEY=$(cat .ssh/id_ecdsa.pub)
EXPORT IS_ADMIN="true"
EXPORT AGE="41"

timoni bundle apply -f bundle.cue

At apply time, Timoni injects the fields values from the environment, if a specified env var is not found and if a default is not provided, the build operation with fail with an incomplete value error.

As a meta-comment: I really like the style of conversational, but carefully considered, rationale given here.


passing a txt file with passwords not working

launching :
crowbar -b rdp  -s <IP_ADDRESS>/32 -u IEUser -C /usr/share/wordlists/rockyou.txt
result:
2020-08-22 17:55:31 START
2020-08-22 17:55:31 Crowbar v0.4.1
2020-08-22 17:55:31 Trying <IP_ADDRESS>:3389
File: /usr/share/wordlists/rockyou.txt doesn't exists
does anyone has experimenting the same issue?

Hey, I just had this issue with rockyou.txt, but I think I've been able to work around it.
Assuming you are using kali and used "apt install crowbar" to install the package to /usr/share

Navigate to /usr/share/crowbar/lib/
Make a backup of main.py (I used "cp main.py main.py.backup")
Once you have a backup, open up the main.py file (Ex. sudo nano main.py).
Look for "def sshlogin(self, ip, port, user, keyfile, timeout): as the start of the next function (the function before it should be def rdp(self): but we need to make the change at the bottom of this function, hence why I suggest looking for def sshlogin.
A few lines above that you should see a try statement followed by "passwdfile = open(self.args.passwd_file, "r").read().splitlines()"
This is the line causing problems with rockyou.txt.
I modified the open command to add encoding="latin-1" instead of Python3 using "UTF-8" that throws the error.
ie. "passwdfile = open(self.args.passwd_file, "r", encoding="latin-1").read().splitlines()
Seems to be working (I think), at least crowbar is running and the passwords it's attempting look okay. Worth a try at least.

It's the rockyou.txt and some of the passwords in there not working well with UTF-8.

Feel free to open up a PR!

Alternative solution which doesn't require crowbar to change: Convert rockyou.txt to UTF-8.
wpscanteam/wpscan#190 (comment)
iconv -f ISO-8859-1 -t UTF-8 /usr/share/wordlists/rockyou.txt > rockyou_utf8.txt

This works for me, thanks.

For anyone who stumbles upon this issue...
Alternatively you can create another rockyou file and update that one.
sudo cp /usr/share/wordlists/rockyou.txt /usr/share/wordlists/rockyou_utf8.txt
sudo vim -c ':set nobomb' -c ':set fileencoding=utf-8' -c ':wq' /usr/share/wordlists/rockyou_utf8.txt

The commands, :set nobomb, :set fileencoding=utf-8 and :wq work in vim too; for when you have vim already opened.

https://en.wikipedia.org/wiki/Byte_order_mark
https://unix.stackexchange.com/questions/381230/how-can-i-remove-the-bom-from-a-utf-8-file
https://stackoverflow.com/questions/778069/how-can-i-change-a-files-encoding-with-vim

that's weird, my rockyou.txt is already utf-8 and even with these workarounds I'm still getting "file doesn't exists"

that's weird, my rockyou.txt is already utf-8 and even with these workarounds I'm still getting "file doesn't exists" Copying rockyou.txt to the same folder I'm running crowbar worked, though.

it works. you will end up with a rockyou_utf8.txt file, try to fix it because you will need it. this post helped me a lot for the oscp


Add Rose At Nightfall High Pitch by NemesisTheory

Song Name
Rose At Nightfall High Pitch
Artist Name
NemesisTheory
Source
Youtube
Youtube Link
Im2LMh2hDeOuP8ZY
Direct File Link
https://www.mediafire.com/file/0h27bki2c04j91o/193466_Rose_At_Twilight.mp3/file
Song ID
153668
Start Offset [ms]
1
Extra
Created from Auto Nong.
Created for level: 69119062

reject
invalid YouTube link


Update token.go

Modify the function annotation so that the swagger can parse correctly

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


[Changes] Qualify medication request

Medication request with type (intent) PLAN cannot be qualified
child of edenlabllc/ehealth.api#3474

Tested


feature/bug: 6502 operands have several display issues when referencing memory

I'd like to start a small discussion about how to improve the display of operand references in 6502 disassembly.
I've been looking at 6502 disassembly for the better part of a year, so I'm fairly accustomed to the quirks in Ghidra's 6502 disassembly. I understand that 6502 isn't a priority, but I think these display changes probably aren't major in terms of code changes .. and maybe someone would be inspired to fix them.  I'd also be willing to tackle them myself, if there's some guidance on where to start.
Basically, there are several places where memory references should be changed to be more consistent with 6502 assembly code.

Any memory reference to zero-page address $0 gets a memory reference with "=>" on the right of the operands, when it really should just be like any other memory reference and replace the operand with the reference.

Offset references should be displayed in code.  The most glaring one is when you do something like "sta buffer+5".  Ghidra would create a dynamic label to buffer[5] (which isn't really ever seen in 6502 assembly), but when you apply an Offset to the reference of -5, it changes the display to "offset buffer" and loses all sense of what the offset is.  It would be best to show "buffer+5" where +5 isn't part of the label.

Jump tables are quite frustrating to work with when they're offset by a value, which is quite common in 6502 where the RTS instruction adds 1 to the return address.  So jump tables often have the label with an offset of 1, but Ghidra doesn't show that in the disassembly.

Does anyone have suggestions for how to work with these quirks, or is this an issue that we could see a fix for?  You can look at the assembly in my github repository (at https://github.com/sarchar/GhidraBugFeature), and here below is my demonstration of what I see in Ghidra when I start analyzing the file:

Cheers, and I much appreciate everyone's time and work they've put into this project.

@sarchar I believe your first point is already covered by #982.

@sarchar I believe your first point is already covered by #982.

Ah, yes. Indeed it looks like the same issue.


Unbound returns "error: memory exhausted" with include directive on large file with version 1.21.0

Describe the bug
Starting with version 1.21.0, when loading unbound and using include directive to load a large file with lots of NXDOMAIN (244 859 entries), unbound gives a memory exhausted error and does not load on freebsd 13.3-p5.
To reproduce
Steps to reproduce the behavior:

add include directive to file with lots (over 244000) of entries like this  "local-zone: "vip.xvpn.io" always_nxdomain"
try to start unbound with service start unbound

Expected behavior
Unbound should start ( as it did before with version 1.20.0_1 )
System:

Unbound version: 1.21.0
OS: freebsd 13.3-p5
unbound -V` output:

> unbound -V
Version 1.21.0

Configure line: --with-libexpat=/usr/local --with-ssl=/usr --enable-dnscrypt --disable-dnstap --with-libnghttp2 --with-dynlibmodule --enable-ecdsa --disable-event-api --enable-gost --with-libevent --disable-subnet --disable-tfo-client --disable-tfo-server --with-pthreads --prefix=/usr/local --localstatedir=/var --mandir=/usr/local/share/man --infodir=/usr/local/share/info/ --build=amd64-portbld-freebsd13.3
Linked libs: libevent 2.1.12-stable (it uses kqueue), OpenSSL 1.1.1w-freebsd  11 Sep 2023
Linked modules: dns64 dynlib respip validator iterator
DNSCrypt feature available

BSD licensed, see LICENSE in source package for details.
Report bugs to<EMAIL_ADDRESS>or https://github.com/NLnetLabs/unbound/issues

Additional information
The behaviour appeared right after updating package from 1.20..0_1 ( still on 13.3-p5 ).
Here is the output when restarting package and including file :
> sudo service unbound start
/usr/local/etc/unbound/monblocker.conf:12163: error: memory exhausted
read /usr/local/etc/unbound/unbound.conf failed: 1 errors in configuration file
Obtaining a trust anchor...
/usr/local/etc/unbound/monblocker.conf:12163: error: memory exhausted
read /usr/local/etc/unbound/unbound.conf failed: 1 errors in configuration file
/usr/local/etc/rc.d/unbound: WARNING: failed precmd routine for unbound

And of course monblocker.conf is the included large file.
If I use a sharp sign (#) to comment out the include directive, unbound starts just fine.
If I use just a couple of lines out of the 24000, the include directive works just fine and  unbound loads and returns NXDOMAIN for the few remaining domains.

Is this the same issue as #1127 ? It sounds like it could be. Perhaps the bug fix for that issue also fixed this issue here?

Yes exactly. Sorry, I though I search for "memory exhausted" before posting.  What is the optimal way of closing this issue now ?

Okay, nice that it fixes this problem too.

Well, I can't vouch for that yet.  But bug reports seems  to refer to the same issue.

If it does not fix the issue, this can be reopened. It would be nice to have the config that fails then; like a short version of the main config file and the include described with a sample line, because that would apparently then be a different config bug.

Here are the three files needed to load my config.  Hopefully, unbound.conf is cleaned up correctly.
Archive.zip

With the archive, I have verified that the fix works. Without the fix, with version 1.21.0, it gives the error like in the issue post, monblocker.conf:12163: error: memory exhausted. With the fix, there is no error. So the issue is already fixed by the other bug fix.

Thank you very much.

I am sorry to bring this back, but when can we expect the fix to be released ?


Action execution is not ideal

The code found here is problematic because:

failure mode is not properly taken into account
there is no isolation between executing actions for different contexts etc

Example scenario of problematic behaviour:

Assume three matches:

Context 1, Rule 1
Context 1, Rule 2
Context 1, Rule 3
Context 2, Rule 1

If, for example an action fails for Rule 2, everything else is run anyway regardless if we want engine-level or context-level failure.

Obsoleted with changes in #30


Add move option in backup

Implement for https://github.com/ysugimoto/aws-lambda-image/issues/133
Added move option for backup configuration.
If move: true, original file will delete from Bucket after processes completed.
And I refactored some:

S3FileSystem returns Promise directory because return value won't use anywhere
Ignore lint for node_modules/

@kdybicz plz review again if you have a time.

Sorry it took so long...


An example isnt work

Did u try it before post?

Or i do something wring?

Hi @qwertukg Which example are you referring to?

Hello!
spring cloud with ldap. I have a connection to my LDAP server, but getting a bad credentials error for correct user. If make auth in LDAP manually, by LdapTemplate and query builder, its ok too

Also, using  inMemoeryAuth manager its ok too

Thanks. Is this the article you're referring to: http://www.baeldung.com/spring-security-ldap and this project: https://github.com/eugenp/tutorials/tree/master/spring-security-mvc-ldap ? Just making sure, because it's not a Spring Cloud project.

Checked the spring-ldap project. Everything works ok. Closing.


[WIP][Enhancement] Optimize shuffle join with skew data

Why I'm doing:
What I'm doing:
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@Youngwb please fill the commit message to make others know what you have done in this CL. Otherwise I know nothing about it.

@Youngwb please fill the commit message to make others know what you have done in this CL. Otherwise I know nothing about it.

Done, PTAL

https://github.com/Mergifyio backport branch-3.1


Staging environment for PRs

Whenever we open a pull request, it would be nice to have the GitHub publish action publish to a staging environment so that we can QA/preview it.
The fission drive application has a working implementation we can look at.
NOTE: This will be a last-PR-wins solution. The most recent PR to receive a push will be the PR on the staging server... unless you re-run the action on a PR you want to "claim" staging.

This is done, for Vercel, in the main repo, at least.

Would be nice to use Fission's Publish infra for staging servers per branch/PR at some point. Not today.


Could not decrypt token.

I followed the instructions on Github to obtain
NOTION_TOKEN

and NOTION_SPACE_ID,

but when executing it, an error 'Could not decrypt token' appeared.

@date-0604  have you tested it using the Python code example provided? Alternatively, could you show me your code so I can review it?

This is my code.

I use Chrome and his TOKEN looks very strange, it's different from yours.

@date-0604 I see, notion just change the token format, may be need some work on this

Thank you, the code is now working properly!


8-buffer PreSieve (used to be 4-buffer), estimated 1%-4% speed improvement

I've been trying to squeeze a bit more performance of out the multi-buffer PreSieve idea, and at the same time reduce the total memory taken by these buffers. Let me know what you think.
Single-core performance
primesieve 1e11 -t1

Compiler
-march
Original (seconds)
Modified (seconds)
Improvement
Original (Giga-cycles)
Modified (Giga-cycles)
Improvement

clang
skylake
14.680
14.262
2.85%
60.729
59.219
2.49%

clang
x86-64
14.811
14.370
2.98%
61.064
59.400
2.72%

gcc
skylake
14.800
14.588
1.43%
61.541
60.700
1.37%

gcc
x86-64
15.027
14.436
3.93%
62.720
60.142
4.11%

Multi-core performance
primesieve 1e11

Compiler
-march
Original (seconds)
Modified (seconds)
Improvement
Original (Giga-cycles)
Modified (Giga-cycles)
Improvement

clang
skylake
4.916
4.788
2.62%
134.955
131.069
2.88%

clang
x86-64
4.965
4.778
3.75%
134.378
130.901
2.59%

gcc
skylake
4.871
4.829
0.87%
134.192
131.280
2.17%

gcc
x86-64
4.832
4.646
3.85%
134.118
130.446
2.74%

Each reported value is the average from 5 runs. Each run was preceded by 30 seconds of sleep so that the CPUs can start non-throttled.

I've been trying to squeeze a bit more performance of out the multi-buffer PreSieve idea, and at the same time reduce the total memory taken by these buffers. Let me know what you think.

I think it is great! I tested it on 2 of my CPUs and it indeed ran faster using both a single thread and multiple threads. On top of that the new code also uses less memory! I slightly improved the distribution of the primes in the bufferPrimes vector, this saves about 100 bytes.
Note that last weekend I tested whether a single shared PreSieve object (i.e. a singleton using primes <= 59) would improve performance. But it ran equally fast. I prefer the current one PreSieve object per thread design as this should theoretically scale better on servers with a large number of CPU cores (because many threads accessing the same memory could cause a bottleneck).


cmd/internal/obj/arm64: golang assembler handles arm64 instruction madd/msub incorrectly.

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version devel +59051da Fri Jun 16 06:05:57 2017 +0000 linux/arm64
What operating system and processor architecture are you using (go env)?
GOARCH="arm64"
GOBIN=""
GOEXE=""
GOHOSTARCH="arm64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/fanzha02/go"
GORACE=""
GOROOT="/home/fanzha02/work/asm_test/golang_us/golang_1/golang"
GOTOOLDIR="/home/fanzha02/work/asm_test/golang_us/golang_1/golang/pkg/tool/linux_arm64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build468264647=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
What did you do?
cd src/cmd/asm/internal/asm
add arm64enc.s test file
enable the ARM64Encoder test command
go test -run "ARM64Encoder"
arm64enc.s test file and endtoend_test.go you can load from this patch:
https://go-review.googlesource.com/c/45851/
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
What did you expect to see?
go test successfully
What did you see instead?
the madd/maddw/msub/msubw test cases are failed

Was this in go1.8? Or it's new on tip (so, a recent regression)?

I just load the latest golang source code and build using go1.6 for bootstrapping. Then run the test and failed.
I also load go1.8 and run the test.  And it shows the error infomation "package cmd/internal/objabi: cannot find package "cmd/internal/objabi"" .

Looking at the git history it looks like it was encoded like that for at least 2 years, so I'll guess that no, it's not a recent regression.

Ok, I looked into this. The tests that @zhangfannie added is expecting MADD R1, R2, R3, R4 to mean R4 = R1*R2 + R3, but currently it means R4 = R1*R3 + R2.
I agree that the first form looks nicer. But this is potentially a breaking change. There may be user code that works that way, especially given that this has been like that for several releases. We need to be sure that no user will be affected unexpectedly, if we want to make this change.
On the other hand, on PPC64, similar FMADD instruction means /* FMADDx fra,frb,frc,frt (t=a*c±b) */. So maybe we shouldn't make this change.

@cherrymui  I found the comment ("madd/msub Rm,Rn,Ra,Rd ") in the current code, if the comment is right,  the current code gets "Rn" value from prog.From3.Reg and gets "Ra" value from prog.Reg. Refer to the instructions assemble process(asm.go), the assignment in that way is wrong.

One possibility is to fix the comment to match the implementation. I'm not sure.

@cherrymui @ALTree  I will fix the comment and keep the source code  not changed.


Find another statistical libraries

 Ask for another statistical libraries

The only 'statistical' library suggested out of our team, for use in C++ was boost. No one knows more, what's not in the net.
This closes the issue.


Make it clear where extensions come from

Would it be possible to make it clear where extensions come from, for example by putting the link to the github repo at the start of the extension page? For example, I want to have to python tool. From this:

I suppose this is the "official" VSC extension, right? Any way I can be sure of it? Could you add some way to easily understand who the author is?

Good point - we should put more context here to help validate the source.
Also related to #2345


Heatmap: Improve support for datasets with >10M columns

How can I  create a heatmap beyond 10 Million values?  Using a very large dataset of size similar to 32k x 26k columns and rows
In this example I am using some random data created and I commonly find this exception

Addtitionally, I attach an example of a very large dataset
https://drive.google.com/drive/folders/1nTLX5hkQowqwnaogATYP6nts5pjOfxZi?usp=sharing
**ScottPlot Version: 4.1
**Operating System: Win 10
**Application Type: Winforms, VB.net

Hi @leGusz,
Right now the best way to do this is to down-scale the data yourself before plotting it as a heatmap.
Here's an example downscaling function:
https://github.com/ScottPlot/ScottPlot/pull/2497#issuecomment-1475277336
A half-size downscaling option was recently added:

#2497

If someone creates one with more features (e.g., arbitrary scaling size) I'd be happy to merge it in! I created an issue to track progress there: #2518

Thank you very much, I am really enjoying ScottPlot, your support and the community are awesome. I will try this method to downscale the data before plotting


Secret is really secret

Maybe in next release, there could be some visual clue which could indicate where is secret located, or it could be said when picking up specific item, that is required for unlocking said secret.

Closed this issue for now, maybe the hint with shovel could be added later


ci(github-actions): update actions/setup-node to v4

coverage: 97.619%. remained the same when pulling 5e0b9afc220940c9c223de411b47a415d6d93332 on actions-setup-node-v4 into 82ad7c1bf6fecc7d4ceede7de619ded476ee2a07 on master.


fix: fetching entity w/o id within firestore doc

Fixes #119

@wovalle, could you please look at this?
We would like to use fireorm in our production code (add it as dependency), but this is blocking...


Venue blackout dates

We need the ability to assign blackout dates to a venue.  This should probably be a simple model with a venue FK, start datetime and end datetime.
Some care will need to be made to incorporate consideration of these blackout dates into the existing logic that determines if a venue is available, or what venues are available during a specified period.

I'm guessing blackout dates define a period during which the venue is not available because of some outside activity, like facility maintenance or other scheduled events (versus events scheduled through openassign). Do I understand this correctly?
I gather we're really talking about blackout periods, each defined as a pair of dates, correct?
Can we assign n blackout periods to a venue?
Does each require explanation or description?
What roles / permissions are required to assign, change, and remove blackout dates?

Correct on all counts.  A description is a good idea.
For now, let's just give permission to Admin.

A few detail questions regarding blackout periods:
I'm currently planning to use methods on the VenueManager to add / edit / remove blackout periods for a given venue. In the interest of uniformity, should I instead create a BlackoutPeriodManager for the new model, with standard create/update methods?
I assume the start and end dates for a blackout period are whole days, and not time values (ie, "ending at 2pm EST on Oct 20"). I'm looking for ways to enforce this "coarseness" on the field or its widgets. Suggestions are welcome if there's a standard idiom in Django or PR.
I assume blackout periods will affect calls to these methods:
RoomManager.get_available_rooms( )
VenueManager.get_available_venues( )
What about resource checks?
ResourceManager.get_available_resources( )
I'm guessing that any resources in a blacked-out venue would be actually be available. (The resource manager currently seems to treat resources as "portable" across venues.) Or, does a resource belong to its venue, and therefore be "otherwise occupied" (unavailable) during a blackout period?
I assume we won't offer blacked-out venues when scheduling new sessions. But should we block the creation or assignment of a session in this situation? If so, is there an existing exception class that would be most appropriate here?
OperationNotPermittedException     <== This seems most appropriate to me
InvalidUsageException
AttributeNotUpdatedException
ValidationException
InvalidInputException
Similarly, should we block the modification of start / end dates for an existing blackout period, if this would conflict with already-scheduled sessions in this venue? If so, I assume we throw the same exception type as above.

I think there should be a BlackoutPeriodManager.
I'm not sure about the whole days issue.  I've queried Ryan, and we'll
see what he says.
We do not need to worry about resources with this.
Let's not block assignment creation if a session ends up scheduled
during a blackout period.
Let's not prevent modification or creation of a blackout period even
if there is a scheduled session that overlaps.  At the very least, the
blackout period can prevent additional sessions from being scheduled,
and then the conflicting sessions can be dealt with manually- either
being rescheduled, or leaving them as-is.
On Mon, Jan 16, 2012 at 12:03 AM, Jim Allman
<EMAIL_ADDRESS>wrote:

A few detail questions regarding blackout periods:
I'm currently planning to use methods on the VenueManager to add / edit / remove blackout periods for a given venue. In the interest of uniformity, should I instead create a BlackoutPeriodManager for the new model, with standard create/update methods?
I assume the start and end dates for a blackout period are whole days, and not time values (ie, "ending at 2pm EST on Oct 20"). I'm looking for ways to enforce this "coarseness" on the field or its widgets. Suggestions are welcome if there's a standard idiom in Django or PR.
I assume blackout periods will affect calls to these methods:
 RoomManager.get_available_rooms( )
 VenueManager.get_available_venues( )
What about resource checks?
 ResourceManager.get_available_resources( )
I'm guessing that any resources in a blacked-out venue would be actually be available. (The resource manager currently seems to treat resources as "portable" across venues.) Or, does a resource belong to its venue, and therefore be "otherwise occupied" (unavailable) during a blackout period?
I assume we won't offer blacked-out venues when scheduling new sessions. But should we block the creation or assignment of a session in this situation? If so, is there an existing exception class that would be most appropriate here?
 OperationNotPermittedException     <== This seems most appropriate to me
 InvalidUsageException
 AttributeNotUpdatedException
 ValidationException
 InvalidInputException
Similarly, should we block the modification of start / end dates for an existing blackout period, if this would conflict with already-scheduled sessions in this venue? If so, I assume we throw the same exception type as above.

Reply to this email directly or view it on GitHub:
https://github.com/TireSwingSoftware/openassign-server/issues/51#issuecomment-3506271

It would be beneficial to specify a blackout period with greater granularity than just days.  For example, Jan 16 11:30AM to Jan 16 5:15PM.   Such a blackout might be due to any number of facility needs which do not need to be known to the system in any greater detail.  (we could just use the reservation system, but then we would have a reporting problem due to mixing in facilities 'events' with the kinds of events we actually care about)  A blackout of a partial day may have the effect of preventing a training event from being assigned on the same day - but it may not. That depends on the event requirements and should not be dictated by a system which fails to represent an accurate picture of blackout reality.
I suggest handling this in a manner similar to how we set up reservations for learning events.   Of course, this would also allow a blackout period be used to make entire days or weeks unavailable as you were probably imagining.
Agreed - resources can be handled at a later date.
Michael's comments are spot on.  A blackout period should mask the venues such that no NEW events can be scheduled on top of the blackout period.  If something is already there the admin will need to deal with it.  When assigning a blackout period, it would be appropriate to see what is already scheduled and alert the admin, but not prevent assignment of the blackout.

I'm grabbing this issue for a bit to add a couple of minor features that the UI needs.


make expects a command "esc" that I do not have

nor do I know what it's supposed to be. any hints?
esc -pkg assets -o cmd/query/app/ui/actual/gen_assets.go -prefix jaeger-ui/packages/jaeger-ui/build jaeger-ui/packages/jaeger-ui/build
make: esc: No such file or directory
make: *** [Makefile:233: cmd/query/app/ui/actual/gen_assets.go] Error 127

I'm on

jaeger 1.21.0 (72b9a062e57afa466f30ebe0f547d34a9b57ea26)
go version go1.15.6 linux/amd64
archlinux 5.9.14-arch1-1

@nolash it looks like you'll need to run make install-tools as outlined in: https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING.md.

@nolash it looks like you'll need to run make install-tools as outlined in: https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING.md.

That was it. Thanks @albertteoh

That was it. Thanks @albertteoh


Emit batch when no data

Hello.
I need a help. I'm trying to calculate errors rate of my application based on data from InfluxDB.
Here is my tick script
`// get errors
var errors = batch
|query('''
SELECT sum("count")
FROM "telegraf"."autogen".errors
WHERE "status_code" =~ /^5\d+/
''')
.period(5m)
.every(10s)
.groupBy('host')
.fill(0)
.align()
|last('sum').as('value')
// get views
var views = batch
|query('''
SELECT count(*)
FROM "telegraf"."autogen".request_time
''')
.period(5m)
.every(10s)
.groupBy('host')
.fill(0)
.align()
|last('count_request_time').as('value')
errors
|join(views)
.as('errors', 'views')
|eval(lambda: int(("errors.value" / "views.value") * 100.0))
.as('value')
|alert() ...`
Everything works fine when in both measurements i have data, but it is not always present.
I'm sending data to influxdb on my own and only if there is an error/request.
So if i have no data (no requests and errors) for tha period of time (5m) my batches are not emitted.
What i'm doing wrong?
I was trying to use DefaultNode but no effect.

@segseriy I'm having the same problem - did you find a workaround?

Unfortunately no. I did not find workaround, just replaced my task...


Add groq llm integration

This PR adds groq as a new module in distilabel.llm, as an integration for the Groq API. Groq provides access to Llama2-70B and Mixtral 8x7B models running on an LPU engine, with inference speed up to 18x faster than regular GPUs.
Because Groq tries to make it easy to transfer from OpenAI API to their API, GroqLLM is structured in a similar way to OpenAILLM.
I have:

Added Groq as an optional dependency to pyproject.toml.
Added _GROQ_AVAILABLE check to imports.py.
Completed basic GroqLLM class. Left #TODO notes to double-check whether Groq supports a few things that OpenAI does, but current code should be functional. I hope to be approved for Groq's alpha release soon so I can test it out and create a basic tutorial demonsrating how to use GroqLLM.

Need to:

Add a section in distilabel docs specifying that Groq is available as an extra.
Add a tutorial that uses GroqLLM. This first requires Groq access.

Hi @kcentric, thanks for the PR, it's awesome! Haven't tested it, but looks really nice. We are currently doing a refactor that would affect the way the LLMs are defined in distilabel, and this will need some modifications. Let's keep this PR as a guide for the moment

Hi @kcentric thanks for the PR, awesome job! I guess you still need to clarify some things and tackle some TODOs so no worries, we won't merge until you let us know it's finished!
Also to confirm to @plaguss I told him we can merge that within the main branch and then we adapt it ourselves to the refactor, so that the PR is not left stale, so we can have it as a reference within the main branch, so that anyone can still use it when installing from source 👍🏻

Thanks guys! @plaguss @alvarobartt I did hear that distilabel is undergoing refactoring, so yes I'm sure we can modify the LLM definition later. The Groq code here should work based off the current version of distilabel already. We just need to verify it live by trying it out in a Colab notebook.
@alvarobartt I wonder how I should test whether Groq works? I'm waiting for Groq to approve me to access their API, but even so, if we haven't merged this PR by then, I won't be able to just do something like from distilabel.llm import Groq. How would I make sure everything's functional?

Oh cool, thought you already tried the GroqLLM implementation and the official Groq client, but if that's not the case because you don't have access yet, then we can wait until you try it out, since the tests should be mocked, so those will work as long as everything's mocked properly.

Yes, I'm still waiting on their early access approval. But if anyone else
has access to their API then is welcome to test it out.
I had tried their platform already because they were offering a UI
interface too.
On Tue, Feb 27, 2024, 2:10 AM Alvaro Bartolome @.***>
wrote:

Oh cool, thought you already tried the GroqLLM implementation and the
official Groq client, but if that's not the case because you don't have
access yet, then we can wait until you try it out, since the tests should
be mocked, so those will work as long as everything's mocked properly.
—
Reply to this email directly, view it on GitHub
https://github.com/argilla-io/distilabel/pull/360#issuecomment-1965994610,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A43V2KVRWDWGISEAO3NIAYDYVWIFVAVCNFSM6AAAAABDYHNN2SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRVHE4TINRRGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey guys, just an update: I still haven't got access to Groq API (a fact I don't really like, because I applied several weeks ago, but I think they have just way too many applicants). I'm wondering if you would like me to update the LLM code inside Groq to accommodate the new refactored LLM interface?
@plaguss @alvarobartt

Hi @kcentric! Sure, if that's not too disruptive feel free to go ahead, but note that the base branch with the refactor is named core-refactor and some things may be still under documented, so maybe you can wait ~1 week? Otherwise, feel free to go ahead and drop us some feedback in the meantime!

@kcentric If you want, there are a lot of open and closed PRs for the new core-refactor LLMs so you would be able to continue work on this :)

@davidberenstein1957 Actually, I got access to Groq API beta a couple days ago after chatting with them a bit. I'm thinking to test this implementation then as soon as I have time. And then maybe we could merge this and anyone could refactor it later along with the other LLM modules, maybe.

Sure @kcentric, feel free to continue with it on the current branch and we'll merge it, then we can take over and implement the refactored version ourselves! 👍🏻

Also wondering @alvarobartt, since Groq is actually an API serving inference for several models, is it better to just a GroqLLM just like OpenAILLM, or rather have a Groq Inference LLM just like we have a Together Inference LLM?
Currently I set it up as GroqLLM because they've basically tried their best to make their API very OpenAI compatible. But just throwing the idea out there.

Hey guys, I got GroqLLM working locally - fully integrated into distilabel. 🥳 I'm using PyCharm though, so I'm hoping to transfer the code to a notebook and be able to share.

Hi here @kcentric, do you want me to take over this and refactor it in the latest version of distilabel? Or is that something you're planning to do by yourself? Thanks in advance! I'd like to see QroqLLM merged within the 1.1.0 release ⚡

You can do it @Alvaro!
On Thu, Apr 18, 2024, 2:43 AM Alvaro Bartolome @.***>
wrote:

Hi here @kcentric https://github.com/kcentric, do you want me to take
over this and refactor it in the latest version of distilabel? Or is that
something you're planning to do by yourself? Thanks in advance! I'd like to
see QroqLLM merged within the 1.1.0 release ⚡
—
Reply to this email directly, view it on GitHub
https://github.com/argilla-io/distilabel/pull/360#issuecomment-2063232231,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A43V2KW75N2ZOKBQLTQXIU3Y552TFAVCNFSM6AAAAABDYHNN2SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRTGIZTEMRTGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

You can do it @alvaro!

Cool, thanks @kcentric, I'll make sure to credit you within the PR description 👍🏻


onFilterTextChange disabling all items

I'm using ngx-dropdown-treeview and for some reason whenever I use the filter every item.disabled is set to true in my tree. I can't edit onFilterTextChange since it's inside lib.
My itemTemplate (I added the "item.disabled" part just to verify this problem):
<ng-template #itemTemplate let-item="item" let-onCollapseExpand="onCollapseExpand">

         {{ item.text }}

         {{ item.text + ' - ' + item.disabled }}

</ng-template>

And here's two images with an example (that "true" right after the text means item.disabled is set to true, the one I added in itemTemplate):
Before using the filter (only the items I disabled are so):

After using the filter (everything in the tree is disabled):

This means no item in the tree is selectable anymore. Is there any solution to this problem?
EDIT: Is it possible that the filter function assumes every child from a disabled element is disabled too? Shouldn't the decoupleChildFromParent option make sure that doesn't happen?

Any news about this issue?


added mix test.watch alias

Added alias for mix test.watch, because I use it a lot during development. And it seems to be pretty popular in the community for TDD.

Thank you @praveenperera!


googleapis_auth Unhandled Exception: Null check operator used on a null value

I am trying to use the googleapis_auth 1.1.0 package from a macos desktop app, using some of the sample code.
void prompt(String url) async {
if (await canLaunch(url)) {
await launch(url);
} else {
throw 'Could not launch $url';
}
}
        var id = ClientId(
            "-----my client id ---------------.apps.googleusercontent.com",
            null);
        var scopes = ['https://mail.google.com/'];

        var client = http.Client();
        await obtainAccessCredentialsViaUserConsent(
                id, scopes, client, prompt)
            .then((AccessCredentials credentials) {
          print('=============  ${credentials.toString()}');
          // Access credentials are available in [credentials].
          // ...
          client.close();
        });

The code almost works but I get the following error:
[ERROR:flutter/lib/ui/ui_dart_state.cc(186)] Unhandled Exception: Null check operator used on a null value
#0      obtainAccessCredentialsUsingCode (package:googleapis_auth/src/oauth2_flows/auth_code.dart:62:62)
#1      AuthorizationCodeGrantAbstractFlow._obtainAccessCredentialsUsingCode (package:googleapis_auth/src/oauth2_flows/auth_code.dart:131:7)
#2      AuthorizationCodeGrantServerFlow.run (package:googleapis_auth/src/oauth2_flows/auth_code.dart:210:19)

#3      MyApp.build.. (package:rudy_googleapis/main.dart)

And the browser window jumps someplace and gives me an HTTP ERROR 500 .
Also,  the print statement never executes.
Any ideas?

I have something similar : https://github.com/google/googleapis.dart/issues/254

Which version of the package are you using? This should be fixed in the latest version of googleapis_auth

googleapis_auth 1.1.0 was what I tested with.

Ah! That was fixed in https://github.com/google/googleapis.dart/pull/197
Just need to publish the latest googleapis_auth package!


[Torch] Better support in-place variant of ops (aten::relu_ etc)

So far we've been kept adding "underscore ops" like aten::relu_ as they come up, but this is annoying. This change lets us support all _ ops automatically.
@comaniac

This change makes sense to me. On the other hand, would that be a potential correctness issue as we discussed before? Such as copy_.

This change makes sense to me. On the other hand, would that be a potential correctness issue as we discussed before? Such as copy_.

Yeah, this is a gray area. Disallowing all _ ops would be too restrictive, since most likely they are benign. I haven't heard or encountered correctness issues arising from those  _ ops that are already in our converter. It is hard to decide where to draw the line, but as we discussed we don't support copy_ for now.
I'll add a warning when we encounter _ ops.

Thanks @masahi


When using Generate Unit Test multiple tabs are opened at the same time

Describe the bug
When using Generate code or Generate Unit Test multiple tabs are opened in the same time
How to reproduce
In an Angular application, open a component's TS file, select all the lines then right click and select Generate Unit Test
Expected behavior
Generate Unit Test should only open one tab
Screenshots

Additional information

Version of the extension impacted: v1.13.0

Thanks for reporting, this happened to me as well. I'll look into it.


correctly refer to "using directives"

Address incorrect use of "using statement" to refer to a "using directive".
and a VB comment syntax fix

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

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


font-variations

Putting this in a separate issue
earthlng wrote

layout.css.font-visibility.level looks nice at first glance but doesn't it just add another vector to detect locale?
Like, fe on my system the arabic fonts are "hidden" in my OS and font-visibility.level=1 would make them not accessible for CSS anymore but someone with an en-US FF (spoofed or not) on an OS in arabic would have those fonts exposed in the list. I don't see how this improves privacy TBH

more to follow. I removed my earlier comments in the diffs, as I wanted to reframe the narrative, and use cleaner tests: rather than some quick results. I've been following this particular anti-FPing setup since at least FF76 (and others from well before that)

reserved

reserved

but I don't think it's worth it

I block doc fonts, I use TB's whitelist with Twemoji Mozilla added). Top is visibility 3, bottom is visibility 1. As you can see I don't gain much: except that the fingerprintjs2 type test is awesome to have empty: that's the fast common test using in FPing. The fallback one is a lot slower but catches more: I don't think anyone uses it.

Update:
This is (essentially) no different to just curating your own whitelist in font.system.whitelist and the feeling (so far) seems to be, engineering wise, when font.system.whitelist is used, to ignore the visibility pref, so that change may or may not happen: I wouldn't call it high priority because Tor Browser can control the pref in ESR78+, and the pref is at default 3 anyway (because it's early days and experimental)
That said, I wouldn't mind adding it as a FYI. It's quite good in the sense that users don't have to fiddle around with non-ANSI chars (e.g. 굴림 ) in a pref, and the kBase, kLangPack lists are already somewhat curated for them. And bundled fonts are auto allowed. Level 2 is a good setting for most people (but I still expect Linux to be somewhat messy)
Also, thanks for everyone's input and help on this :) Also, where is everyone?
@earthlng What do you think?

oof there's a lot here ... 1st of all thanks for writing this all down and all the testing etc! 👍
I think I misinterpreted a dev's comment(s) and the call to isHidden() (or something like that) in the code. I thought it uses an OS API to retrieve the visible/hidden state of installed fonts but apparently that's only used for a few weird system fonts on Mac OS. Everything else seems to based on those manually curated fontlists and therefore the OS locale leak I was worried about shouldn't happen.
There are a few other weird things that I don't quite understand like fe in your test when you set the level to 2 it didn't detect Meiryo and MingLiU_HKSCS anymore even though they're both in the kLangPackFonts list.
MingLiU-ExtB from the kBaseFonts also mysteriously dropped with level 2. Any idea why that is?
Overall I think this looks promising and a good first step forward, especially once they make value 2 the default. Maybe a that point we could even consider relaxing "no-doc-fonts" and just use this instead.

in your test when you set the level to 2 it didn't detect Meiryo and MingLiU_HKSCS anymore even though they're both in the kLangPackFonts list. MingLiU-ExtB from the kBaseFonts also mysteriously dropped with level 2. Any idea why that is?

 There may be a few others like this. Meiryo UI, MingLiU, MingLiU_HKSCS-ExtB show up FWIW.
There's clearly a difference in how whitelist and visibility accept font names/families or something. line7 "TODO: check whether we need to list legacy styled family names like "... Light" <-- I don't think this it, since it can pick up on Arial Narrow for example.
I have a gut feeling it's to do with CJK names (western vs non-western): I am so not a font expert. Maybe it's to do with GDI? IDK
PS: sent you an email
PPS: I'll ping the moz dev

... aaaaand so it begins ... https://old.reddit.com/r/firefox/comments/i1jfzl/several_fonts_gone_in_80_beta/  .. RFP

when the font whitelist is used, font vis will be ignored - starting in FF81, may be back ported to FF80 for Tor Browser - I'll keep you posted


transfer: small improvements to authz

Description

This PR fixes 2 out of the 3 Improvements suggested by @fedekunze here.
closes: #XXXX
Commit Message / Changelog Entry
N/A

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[x] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Codecov Report

Merging #3460 (0543e7b) into main (c043dbd) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3460   +/-   ##
=======================================
  Coverage   78.54%   78.55%
=======================================
  Files         181      181
  Lines       12581    12584    +3
=======================================
+ Hits         9882     9885    +3
  Misses       2265     2265
  Partials      434      434

Impacted Files
Coverage Δ

...ules/apps/transfer/types/transfer_authorization.go
91.56% <100.00%> (+0.31%)
:arrow_up:


Project does not compile as described in README

Running the command:
$ ./gradlew consumer
Yields the following output:

Task :plugin FAILED

FAILURE: Build failed with an exception.

What went wrong:
Project directory '/XXX/kotlin-xcode-compat/plugin' is not part of the build defined by settings file '/XXX/kotlin-xcode-compat/settings.gradle.kts'. If this is an unrelated build, it must have it's own settings file.

If you're using Gradle 5, it now requires settings.gradle https://docs.gradle.org/current/userguide/upgrading_version_4.html#changes_4.5
example:
settings.gradle
rootProject.name = 'ProjectName'


Make containers resizable

Is your feature request related to a problem? Please describe.
All containers are currently not resizable and have a fixed height/width. This makes it hard to adjust the several components to one's needs (e.g. make input data more visible).
Describe the solution you'd like

[ ] Make containers resizable

[ ] properties panel (width)
[ ] editor/left side (width)
[ ] preview/right side (width)
[ ] input data (height)
[ ] output data (height)

[ ] Consider: having similar snap behavior as we have in the Desktop Modeler properties container
[ ] Add the possibility to persist these values (like open).

Describe alternatives you've considered
Keep things fixed.
Additional context
Related to https://github.com/camunda/camunda-modeler/issues/3126

As discussed with @christian-konrad, let's wait for further feedback before we act on this.


added drag-and-drop-puzzle

Added a drag-and-drop puzzle

Thanks. Can we improve it for responsiveness as well

Sure Sure


made Box behave like h2d.comp.Interactive

Proposition for adding more interactivity to Box
Here I just copied some from h2d.comp.Interactive
It has two problems:

copy-paste and fact that scripts bound
h2d.comp.Parser binds scripts to properties like 'onclick' only to h2d.comp.Interactive

Another way to do this is to just sub-class Box form Interactive
So what would be preferred way?

h2d.comp is deprecated


Small typo in README

In the config section of the README, algorithm is spelled wrong -- 'algorithem'.

oops! Thanks I've updated it to the correct spelling.


Does RxJS async.series example really execute in series?

I have a use case where I need to execute multiple calls to functions that return AsyncSubject in order, similar to async.series. Here is the primary requirement, quoted from that page:

The async.series runs an array of functions in series, each one running once the previous function has completed.

Here is the code that I've come up with that works, using TypeScript, RxJS 5:
    private _wrapArray<T> (items: T[]): Observable<any[]> {
        return Observable
            .fromArray(items)
            .concatAll()
            .toArray();
    }

    execSync<TItem, TResult>({ items, func }: {
        items: TItem[],
        func: (item: TItem) => AsyncSubject<TResult>
    }): Observable<TResult[]> {
        // defer call is required to execute in order
        var wrapped = this._wrapArray(items.map(t => Observable.defer(() => func(t))));
        // var wrapped = this._wrapArray(items.map(t => func(t))); // doesn't execute in order
        return wrapped;
    }

Notice that I had to use Observable.defer (Thanks SO+Brandon!) to get it to execute synchronously, which after the fact makes sense, as the call to map is executing the func(t). So the array that is passed into _wrapArray is already running each function, and thus not in series.
But this is not what is in the RxJS async.series example code:
wrapArray([
    Rx.Observable.just('one'),
    Rx.Observable.just('two')
  ])
  .forEach(
    function (results) {
      console.log(results);
    },
    function (err) {
      console.log('Error: %s', err);
    }
  );

The purpose of the async.series operator is so that you can call AsyncFuncA, AsyncFuncB, etc., in order, so I'm not sure if the Rx.Observable.just example is very helpful. Perhaps it could be changed to include an async func and use the call to Observable.defer to be slightly more helpful?
wrapArray([
    Rx.Observable.defer(Rx.Observable.just('one')),
    Rx.Observable.defer(Rx.Observable.just('two'))
  ])
  .forEach(
    function (results) {
      console.log(results);
    },
    function (err) {
      console.log('Error: %s', err);
    }
  );

To check the synchronous aspect of the example solution (without defer), I threw the following into one of my mocha unit tests:
var Rx = require('rxjs');

function wrapArray (items) {
  return Rx.Observable
    .from(items)
    .concatAll()
    .toArray();
}

function delayFunc(s: string, delayMs: number): Rx.Observable<string> {
    let result = new Rx.Subject();
    setTimeout(() => {
        console.log(s);
        result.next(s);
        result.complete();
    }, delayMs);
    return result.asObservable();
}

wrapArray([
    Rx.Observable.of('one yo'),
    delayFunc("two yo", 500),
    delayFunc("three yo", 10),
    Rx.Observable.of('four yo'),
  ])
  .forEach(
    function (results) {
      console.log(results);
    },
    function (err) {
      console.log('Error: %s', err);
    }
  );

describe('common', () => {
    it('should buy time', (done) => {
        setTimeout(x => {
            done();
        }, 10000);
    });
});

With console output:
  common
three yo
two yo
[ 'one yo', 'two yo', 'four yo' ]

Notice that three is before two, and it doesn't even show up in the final results array.
If I change it to use defer:
var Rx = require('rxjs');

function wrapArray (items) {
  return Rx.Observable
    .from(items)
    .concatAll()
    .toArray();
}

function delayFunc(s: string, delayMs: number): Rx.Observable<string> {
    let result = new Rx.Subject();
    setTimeout(() => {
        console.log(s);
        result.next(s);
        result.complete();
    }, delayMs);
    return result.asObservable();
}

wrapArray([
    Rx.Observable.defer(() => Rx.Observable.of('one')),
    Rx.Observable.defer(() => delayFunc("two yo", 500)),
    Rx.Observable.defer(() => delayFunc("three yo", 10)),
    Rx.Observable.defer(() => Rx.Observable.of('four yo')),
  ])
  .forEach(
    function (results) {
      console.log(results);
    },
    function (err) {
      console.log('Error: %s', err);
    }
  );

describe('common', () => {
    it('should buy time', (done) => {
        setTimeout(x => {
            done();
        }, 10000);
    });
});

with output:
  common
two yo
three yo
[ 'one', 'two yo', 'three yo', 'four yo' ]

Now it makes sense. Two comes before three, and all four are in the results array.

@bill-mybiz Yes, this is the distinction between Observable creation and Observable subscription. In both your examples, you're synchronously creating Observables, but when you wrap the call to func in Observable.defer(), you're delaying the call to func to the point at which each inner Observable is subscribed to by concatAll.
You can also delay creating each inner Observable until the one before it is completed by using concatMap:
execSync<TItem, TResult>({ items, func }: {
    items: TItem[],
    func: (item: TItem) => AsyncSubject<TResult>
}): Observable<TResult[]> {
    return Observable.from(items).concatMap(func).toArray();
}```

@trxcllnt I was in the middle of testing changes that you recommended and writing a response when life intervened. I apologize for the delay, and I thank you for your quick response. RxJS's super-active community and extensive documentation is the primary reason we chose RxJS over Bacon and others.
With that said, it's actually that documentation that I posted this issue about. That execSync code was given for some background (i.e. my use case). You phrased the heart of the issue nicely in that it was highlighting the difference between creation and subscription, and the change made to the execSync code worked.
However, it's the async.series example code that I tried to focus on, as that code is what people learning Rx from the async library will be looking at. I only posted the issue because It is **not ** "my example", rather it is the official RxJS example. And as I stated, it doesn't seem to execute synchronously as given, at least to non-expert learners (who I would think compose the target audience). This seems to stem from the example using an observable that acts synchronously (the just in the documentation, of in my example), as opposed to the actual use case for async.series.
And so, do you think the current documentation is correct as-is?

@bill-mybiz ah, I understand now. Yes, this example should probably be updated cc: @staltz who has been focusing on this area.


[network] Reduce network log frequency

Decreasing duplicate logs where sometimes we'd log an entry message and
then immediately different paths which acted like a trace log.  In
addition, remove usage of the crit! macro.

error / crit seem strange for exiting a loop when we should expect that upon normal termination, no?

Yeah, it is strange.  I guess the thought is it that it would be an error if it unexpectedly happened.  Except if it happens on purpose, obviously it's not an error.

/land


Add OpenSearch support

Add an OpenSearch description document according to the specification.
I'm not sure how well other browsers support this, but this addition will allow metasmoke to be added as a search engine in Firefox. This allows me to search for any keywords directly from my browser like this:

Chrome and Safari probably support this too.


Add support for HTTP Basic Auth in endpoints

Describe the feature request
In HTTP endpoints, e.g. when checking "[STATUS] == 200", I would like to see the availability to pass on HTTP Basic credentials (username and password), similar to the security section.
Example endpoint configuration could be:
  - name: Authenticated endpoint
    url: "http://myservice:8080"
    interval: 30s
    conditions:
      - "[STATUS] == 200"
    security:
      basic:
        username: admin
        password: YWRtaW4=  # base64 encoded

Why do you personally want this feature to be implemented?
It would be perfect to monitor the HTTP status codes of some authenticated pages
How long have you been using this project?
No response
Additional information
No response

I haven't tested this myself, but yeah, I think sending the header should work 🤔


fix: Fix printing of generic function parameters

Fixes #482

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.58%. Comparing base (dd669c1) to head (aa68b93).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #516   +/-   ##
=======================================
  Coverage   91.58%   91.58%
=======================================
  Files          59       59
  Lines        6170     6170
=======================================
  Hits         5651     5651
  Misses        519      519

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


Permission check for the protected handler

Add an optional array of the required permissions to be check in protected handlers

@krzysztofwolski please release to npm after merge


Enable dm_verity, dm_bufio, virtio_console, and nfsd v4 security label

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./LICENSES-AND-NOTICES/SPECS/data/licenses.json, ./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md, ./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

DM_VERITY Kconfig
DM_BUFIO Kconfig
VIRTIO_CONSOLE Kconfig
NFSD_V4_SECURITY_LABEL Kconfig Security labels allow security modules like SELinux and Smack to label files to facilitate enforcement of their policies.
REED_SOLOMON Kconfig
Change Log

Enable dm_verity and dm_bufio by default
Enable virtio_console by default
Enable nfsd v4 security label

Does this affect the toolchain?

YES
Associated issues

https://microsoft.visualstudio.com/OS/_workitems/edit/52468301

Test Methodology

https://dev.azure.com/mariner-org/mariner/_build/results?buildId=634981&view=results

Closed as these were addressed in 2 PRS
Enable virtio console by default and build e1000 drivers as modules by rlmenge · Pull Request #10604 · microsoft/azurelinux (github.com)
Enable nfsd v4 security label by rlmenge · Pull Request #10605 · microsoft/azurelinux (github.com)


Update asn1.js

Thanks for all the help getting these through. Still ironing out the bn.js fork here.
See https://github.com/indutny/asn1.js/pull/32 as well.

so this isn't stricly needed except to prevent weird cache issues, but doesn't hurt

I had to unpublish this as it causes an peer invalid error, ping me when its safe to publish again

The peer dependency conflict will exist so long as bn.js 0.15 is a peer dependency is in the same tree.
The way around this would be to bump the major.

bump the major where? I thought we were just waiting on indutny/asn1.js#32 to upgrade?

Ahh right, sorry I thought you meant the bn.js 0.15 peer dependency issue was still happening, but of course that's been resolved.


opentelemetry-instrumentation-rdkafka blocking opentelemetry-instrumentation-all update

I'm currently unable to bundle update opentelemetry-instrumentation-all opentelemetry-exporter-otlp opentelemetry-api opentelemetry-sdk due to this error:
Bundler could not find compatible versions for gem "opentelemetry-api":
  In Gemfile:
    opentelemetry-exporter-otlp (= 0.24.0) was resolved to 0.24.0, which depends on
      opentelemetry-api (~> 1.1)

    opentelemetry-instrumentation-all (= 0.28.0) was resolved to 0.28.0, which depends on
      opentelemetry-instrumentation-rdkafka (~> 0.2.0) was resolved to 0.2.0, which depends on
        opentelemetry-api (~> 1.0.0)

This is happening because opentelemetry-api is locked to ~> 1.0.0 here: https://github.com/open-telemetry/opentelemetry-ruby-contrib/blob/3e85d4436d338f326816c639cd2087751c63feb1/instrumentation/rdkafka/opentelemetry-instrumentation-rdkafka.gemspec#L28
Should that be changed to  ~> 1.1.0?

Hi @pb-dod - thanks for the report, and sorry for the trouble. We'll take a look.

As noted in Slack, this is another occasion on which this issue's general domain comes to mind: https://github.com/open-telemetry/opentelemetry-ruby-contrib/issues/176


3.3.0 release causes semver breakage

The recent relese (3.3.0) breaks backwards compatibility with 3.2.1 but is interpreted by cargo as backwards compatible. For example, this script compiles with 3.2.1:
fn main() {
    let mut wtr = tagger::new(tagger::upgrade_write(Vec::<u8>::new()));
    let _ = wtr.elem("svg", |d| {
        d.attr("xmlns", "http://www.w3.org/2000/svg")
            .attr("xmlns:xlink", "http://www.w3.org/1999/xlink");
    });
}

When Cargo.toml includes:
[dependencies]
tagger="=3.2"

Converting the version requirement to tagger="3.2" (removing the =), this example continues to compile if the Cargo.lock cache is maintained. However, upon doing cargo update or deleting Cargo.lock, tagger 3.3.0 is brought in because the tools assume it is "semver compatibile" with 3.2.1. However, the compilation will then fail with:
   Compiling tagger-test v0.1.0 (C:\Users\drand\Documents\src\rust-play\tagger-test)
error[E0599]: no method named `attr` found for enum `Result` in the current scope
 --> src\main.rs:5:14
  |
5 |             .attr("xmlns:xlink", "http://www.w3.org/1999/xlink");
  |              ^^^^ method not found in `Result<(), std::fmt::Error>`

error[E0308]: mismatched types
 --> src\main.rs:3:33
  |
3 |       let _ = wtr.elem("svg", |d| {
  |  _________________________________^
4 | |         d.attr("xmlns", "http://www.w3.org/2000/svg")
5 | |             .attr("xmlns:xlink", "http://www.w3.org/1999/xlink");
6 | |     });
  | |_____^ expected enum `Result`, found `()`
  |
  = note:   expected enum `Result<_, std::fmt::Error>`
          found unit type `()`

Some errors have detailed explanations: E0308, E0599.
For more information about an error, try `rustc --explain E0308`.
error: could not compile `tagger-test` due to 2 previous errors

I think the correct solution here would be to yank 3.3.0 and re-release it as 4.0.0.

Sorry about this. I followed @link2xt's advice and yanked those versions and released a 4.0.0

Thanks, works for me.


Back to top button

🎮 feature Request
This feature proposes the addition of a "Back to Top" button functionality to enhance user experience in navigating through lengthy pages within the project. The "Back to Top" button would allow users to swiftly return to the beginning of the page, offering convenience and improved usability.
Point down the features
The proposed solution involves implementing a visually discreet yet easily accessible button, positioned at the bottom right corner of the viewport. Upon clicking or tapping, the button smoothly scrolls the page back to the top, providing a seamless navigation experience. The button's design will align with the project's aesthetic principles while maintaining clarity and functionality.

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Please assign me this issue under GSSoC'24.

Please assign me this issue , so i can work on it under GSSoC'24.

@Aleena-Memon Please raise any other issue, i'll assign you

If you have any doubt you can directly send message on our discord (https://discord.gg/rZb46cCMmK), we have created seprate chennals for all projects...
Thank you :)

As I learned MERN stack development i easily create this issue assign to me

hey @Durgesh4993 , I would like to contribute to this issue if no one is working on it currently.

Hello @Durgesh4993 , It has been a month for this issue and not resolved yet , please assign it to me I will create PR within 1 day.


[SLAM] Backport the depth image fixing to the new implementation

Fixes #124

@kenny-y please help to review.

Merging. @kenny-y please merge if LGTM.


Mapping object for VS 2017

Hi. I'm a Dot Net Developer and I'm looking for a 3rd party tool that allows me to show objects on a USA map (taken from a database) and once you click on each individual object on the map, it takes you 1+ level deeper, so you can obtain more information about that particular object. Is anyone aware of such tool that is integrated into Visual Studio?
Thanks in advance
Al

Sorry, this is not a place to ask questions or recommendations. We're only keeping a list of awesome libraries/tools and if you can't find something for your needs we can't help you with that. You can probably ask on Stack Overflow or any other apropriate place.


MRG: Add VectorView and OPM rest example

Follow-up of #5514.
Closes #5510.
Implements the Brainstorm pipeline for resting state data. Looks good and seems robust to me on VectorView and OPM data.
Marked WIP pending verification from @bloyl that it looks similar in Brainstorm and/or that @dengemann can replicate on mne-hcp data. @ktavabi has reported that it works for some of his VectorView data.

https://10077-1301584-gh.circle-artifacts.com/0/html/auto_examples/datasets/plot_opm_rest_data.html

As per chat with @larsoner I re-ran this resting state script on data for a 17 years-old participant in 5 minutes, eyes-open session on a 306 channel VectorView system. Noise covariance is computed from ERM recording processed with SSS using default  Maxfilter parameters, raw data was denoised with SSS in MNE using in regularization, 0.98 tSSS correlation with 300. seconds integration window. Projections were computed with component averaging with compute_proj_exg with more or less default parameters. For resting state example I made minimal adjustments to script to better suit my needs and reproduce source images.
LMK if you need more info.
PCA Topo Maps

Raw PSD

Alpha band results

Beta band results

Updated naming and comment, good to go for you @dengemann ?

I had another question. Why didn‘t we keep the 1 projector per channel type

average == True? In my experience that is a good default to go (remove
good amount of artifact with 1 SSP).

On Thu 8 Nov 2018 at 15:56, Eric Larson<EMAIL_ADDRESS>wrote:

Updated naming and comment, good to go for you @dengemann
https://github.com/dengemann ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/5692#issuecomment-437024404,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fio7-hRNQ_irEWrOirvoWyHuXiRufks5utEY2gaJpZM4YMxle
.

Why didn‘t we keep the 1 projector per channel type + average == True?

The default is already average=True but I'll lower it to 1 for mag

Ah. Ok. Right. Nevermind.
On Thu 8 Nov 2018 at 16:12, Eric Larson<EMAIL_ADDRESS>wrote:

Why didn‘t we keep the 1 projector per channel type + average == True?
The default is already average=True but I'll lower it to 1 for mag
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/5692#issuecomment-437030340,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fiobNdqhhLEPesSgxuT0Mokzu9deUks5utEnygaJpZM4YMxle
.

Pushed fixes

I think this example would deserve an entry in what‘s new. We have long
waited to have that.
On Thu 8 Nov 2018 at 16:20, Eric Larson<EMAIL_ADDRESS>wrote:

Pushed fixes
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/5692#issuecomment-437033037,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fipvzK0wBSzwBPMMwYYflAWqFqPTeks5utEumgaJpZM4YMxle
.

Done

Here's the link to the last rendering
https://10146-1301584-gh.circle-artifacts.com/0/html/auto_examples/datasets/plot_opm_rest_data.html
LGTM

CIs look happy (AppVeyor is slow but should follow) so I'll merge. Thanks for working through this one everyone!


net/http: tls with large number of connections

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go version go1.7 linux/amd64

What operating system and processor architecture are you using (go env)?
go version go1.7 linux/amd64

What did you do?
Start a HTTPS server with ListenAndServeTLS

What did you expect to see?
Handling a lot of short connections

What did you see instead?
Not handling a lot of short connections

So basically, the TLS handler is bad at handling large number of short connections,
and works a lot better when I removed the "tcpKeepAliveListener" from this line:
tlsListener := tls.NewListener(tcpKeepAliveListener{ln.(*net.TCPListener)}, config)
(I disabled KeepAlives with SetKeepAlivesEnabled anyway)

I'm afraid we need more information. Can you write a simple benchmark that demonstrates the problem and show how much faster it gets with the change you propose?

ok, this is pretty embarrassing
When I was working on the problem, at some point I was forced to update go version on the machine I was working on, and that was what actually helped me.
So this was probably some already resolved problem with handling cryptography in only one thread.
Sorry for wasting your time.


Temporarily remove Jitsi integration for F-Droid build

Remove Jitsi integration for the F-Droid version of the application, due to the fact that Jitsi library include non free software (package "co.apptaillor.googlesignin")
More details here: https://github.com/vector-im/riot-android/issues/3121
F-Droid applicationId set to "im.vector.app"

Replaced by #3276


iOS 13 Safari is load time is Sloooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooow

Problem
non-private mode safari on ios 13 is super slow to load.

Further complications
iOS 12 seems to work fine:

As does iOS 13 private mode:

If you load up the profiler do you see anything at all interesting?

I did some investigating:

On iOS, our app is fast to load in Private browsing mode but slow to load in Regular mode. The difference is about an order of magnitude. Having done some testing and research, I believe the cause is a combination of these two factors: 1) Private mode on iOS does not store the databases to disk and 2) our database, IndexedDB, is slow on iOS.
I did some googling and found articles claiming that inserts to IndexedDB on iOS can take multiple seconds(!). This is orders of magnitude slower than other platforms.
Looking at the Storage tab in Safari's debug mode, I found that the Regular mode uses IndexedDB but Private mode does not. This suggests that they use an in-memory database only for Private mode. This would virtually eliminate the time required to write to the database.
At first glance it's odd that, excepting iOS, writing to disk is as fast as writing to memory. However, blocking on network I/O should take up almost all of the load time. The remainder is within the margin of error except on iOS.
Note that this is still speculative. A definitive test would involve measuring write time to IndexedDB on iOS Regular vs Private modes. This should be added to the existing Augur app if possible, to eliminate variables like IndexedDB size and the numbers of writes, databases, and indexes. If write times are very large in Regular mode but minute in Private mode then we know that the problem is with iOS's IndexedDB implementation.
A potential solution is to use an in-memory database for fast load then leisurely replicate it onto disk. (credit: Paul)
tl;dr: It's probably a problem with iOS. We can run a definitive test by adding debugging info to our app on our next deploy.

Using https://augurproject.github.io/pouchdb-perf/ we have confirmed the cost stems from indexedDB db and index creation.
questions here:

Will reducing the number of active dbs/indexes speed up mobile safari? (Yes)
Is it a similar result for other browsers?

To research:

A replication scheme similar to the one stated above that only builds indexes in memory.

Ruled out:

The Webworker does not seem to be source of the slowness.

Filed: https://github.com/pouchdb/pouchdb/issues/7928
Hopefully they can brining it to the attention of interested parties.... now to find a short term fix.....

if I navigate to another app, it seems to get messed up
debug may be enabled which makes it slower on our site
each subsequent performance test is a lot slower on https://augurproject.github.io/pouchdb-perf/. This isn’t true for dexie. Idk why

switched to dexie


chore(GroupWithZero,Valuation): move ℤₘ₀ notation earlier, under Multiplicative scope

Change the name of scope containing ℤₘ₀ from DiscreteValuation to Multiplicative.

Requested by @YaelDillies

I am happy with it, I permitted myself to modify a bit the description of the PR. I think that also the title of the PR should change (it makes no reference to the scope business) but I let @pechersky see if he agrees.

Great by me

I am happy with the change as well.

Thanks :tada:
bors merge


feat: remove logging in prod mode

Oh this has indeed confused me before, tbh I'd vote for removing logging altogether. Most cases where I've been confused is in dev

I think it’s still good practice to log in dev mode to hint users when they are doing something wrong (like using a string key instead if an array)

:tada: This PR is included in version 4.0.0-alpha.6 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


QA: Does it possible to use several PVC in JBOD?

Hi, amazon have limits on performance for EBS, and it is just economically and performance profitable to have 4 volumes per node instead of big one
Like 4 gp3 128GB volumes with default 125 MB/s & 3000 iops = 512 GB 600 MB/s and 3000 iops
vs 1 gp3 512GB volumes with increased limits to 500 MB/s & 12000 iops

yes it possible
you could use something like this
spec:
  configuration:
    files:
      config.d/storage_policy_jbod.xml: |
       <yandex>
          <storage_configuration>
            <disks>
              <disk2>
                <path>/var/lib/clickhouse2/</path>
              </disk2>
              <disk3>
                <path>/var/lib/clickhouse3/</path>
              </disk3>
              <disk4>
                <path>/var/lib/clickhouse4/</path>
              </disk4>
            </disks>
            <policies>
              <default>
                <volumes>
                  <default>
                    <disk>default</disk>
                    <disk>disk2</disk>
                    <disk>disk3</disk>
                    <disk>disk4</disk>
                  </default>
                </volumes>
              </default>
            </policies>
          </storage_configuration>

  defaults:
    podTemplate: multi-disk
  templates:
    podTemplates:
    - name: multi-disk
      spec:
        containers:
        - image: clickhouse/clickhouse-server:22.6
          name: clickhouse
          volumeMounts:
          - mountPath: /var/lib/clickhouse
            name: default-gp3
          - mountPath: /var/lib/clickhouse2
            name: disk2-gp3
          - mountPath: /var/lib/clickhouse3
            name: disk3-gp3
          - mountPath: /var/lib/clickhouse4
            name: disk3-gp3

    volumeClaimTemplates:
    - name: default-gp3
      reclaimPolicy: Retain
      spec:
        accessModes:
        - ReadWriteOnce
        resources:
          requests:
            storage: 128Gi
        storageClassName: gp3
    - name: disk2-gp3
      reclaimPolicy: Retain
      spec:
        accessModes:
        - ReadWriteOnce
        resources:
          requests:
            storage: 128Gi
        storageClassName: gp3
    - name: disk4-gp3
      reclaimPolicy: Retain
      spec:
        accessModes:
        - ReadWriteOnce
        resources:
          requests:
            storage: 128Gi
        storageClassName: gp3
    - name: disk4-gp3
      reclaimPolicy: Retain
      spec:
        accessModes:
        - ReadWriteOnce
        resources:
          requests:
            storage: 128Gi
        storageClassName: gp3

But this approach has tradeoff.
clickhouse-server splits data by parts (file level) not block level, and use round_robin (least_used in 22.6+) for JBOD balancing.
So, when new part creates after insert or after background merge / mutation then parts / data doesn't shuffle / spread between disks smoothly
you would get the "hot" data parts which will store on the only one disk and read / write iops can't be spread between disks the same way as RAID


[Event Parameter Change] Report 698 - Get Sales Orders - procedure InsertReqWkshLine - event OnInsertReqWkshLineOnBeforeCode

We would like the parameter IncludeSender added to the following event.
Report 698 - Get Sales Orders - procedure InsertReqWkshLine - event OnInsertReqWkshLineOnBeforeCode
Original:
[IntegrationEvent(false, false)]
local procedure OnInsertReqWkshLineOnBeforeCode(var ReqLine: Record "Requisition Line"; SalesLine: Record "Sales Line"; SpecOrder: Integer; LineNo: Integer; ReqWkshName: Record "Requisition Wksh. Name"; GetDim: Option Item,"Sales Line"; var IsHandled: Boolean)
begin
end;

Change: IncludeSender = true
[IntegrationEvent(true, false)] // <-- we need IncludeSender true
local procedure OnInsertReqWkshLineOnBeforeCode(var ReqLine: Record "Requisition Line"; SalesLine: Record "Sales Line"; SpecOrder: Integer; var LineNo: Integer; ReqWkshName: Record "Requisition Wksh. Name"; GetDim: Option Item,"Sales Line"; var IsHandled: Boolean) <-- we need LineNo var
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 49416.


Squawker causing account being locked

Checklist

[X] I am able to reproduce the bug with the latest version.
[X] I made sure that there are no existing issues - open or closed - which I could contribute my information to.
[X] I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
[X] This issue contains only one bug, that only occurs in Squawker

Affected version
v3.7.7+300014723 (Probably also applicable to newest version)
Steps to reproduce the bug

Browse some tweets
The feeds stops working
Re login into account and saying account locked due to suspicious activity

Expected behavior
Account should not be locked and should be working as usual.
Actual behavior
Sometimes app causes account being locked
Screenshots/Screen recordings
N/A
Logs
N/A
Affected Android/Custom ROM version
Android 14 / One UI 6.0
Affected device model
Samsung Galaxy S22+
Additional information
N/A

Update: Seems account lock occurs when you log into Squawker, regardless of VPN usage.

Got the issue as well, never used VPN.


Update parseOneOf.ts

Remove TypeScript types from schema to fix the issue of generating scheme that cannot be invoked directly by new Function.

Produces bad ts though. Either add an option+cli flag to disable all types or refactor the output code so it's not needed.


Missing Fields on TabMon DataSource

Hi
I installed TabMon on a Windows Server 2012 R2 server, and when I am trying to check the dashboards on TabMonWorkbook, I noticed that there are many missed fields in the TabMon datasource. Due this issue, obviously many of dashboards does not show information.
Here is a list with some of the missing fields
1.- active_sessions
2.- average_batch_failure
3.- average_batch_size
4.- average_batch_latency
5.- average_request_latency
6.- External Abstract Query Hit Ratio
7.- External Abstract Query Hit Ratio
8.- External Abstract Query Hit Ratio
9.- external_abstract_query_cache_misses
10.- external_abstract_query_cache_misses
Notes
1.- TabMon appears is working OK.
2.- I'm using Tableau Desktop 9.2.6 to open the workbook.
Attached Images:

Do you know why I have these missing fields?
I would appreciate any help.
Best Regards
José Alberto

Hey Jose.  This is a know problem detailed in issue 55. In essence,  many of the JMX MBeans which TabMon polls for information have been moved (or their names have changed)  in 9.3

Hi Sussch.
Thanks for reply so quickly. I noticed that I forgot to enable the JMX ports.
Thanks again.
Regards.

Closing this out in favor of https://github.com/tableau/TabMon/issues/55


Fix tremor common

Description
No functional changes just mopping up missing dependencies veatures for sub crates that are used stand alone
Checklist

[x] The RFC, if required, has been submitted and approved
[x] Any user-facing impact of the changes is reflected in docs.tremor.rs
[x] The code is tested
[x] Use of unsafe code is reasoned about in a comment
[x] Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
[x] The performance impact of the change is measured (see below)

Performance
No code changes

Coverage remained the same at 83.187% when pulling 102cc13261ce2ff0cf7c02a8ddb6c892efe1e283 on fix-tremor-common into 04f674ff3029ce3ee4caab87c6af342386667e68 on main.


feat: Original on left

#61
Doesn't close it, we need to enforce the 'Original' property in manx-search-data

We need to enforce the 'Original' property in manx-search-data


Crafted images in 400 bytes or less take over a minute to decode

Found with the fuzzer. Several test cases that take over 60s to decode, ranging from ~90 to ~400 bytes: webp_slow_to_decode.tar.gz
Tested on commit cc5ec05d1dddfe8a389c738c34c9789eefa08d32

Did some investigation of the 86-byte test case...
The core issue seems to be that this library believes the file is a valid WebP image. Once you account for the claimed 11024x13994 image dimensions, the 34 second decoding time (on my machine) works out to a not totally unreasonable 17 MB/s decoding speed.
From what I've gathered so far, the reason the code's OK with such a small file encoding so large an image is because the reader simply treats a truncated bitstream as containing infinitely more zero bytes:
https://github.com/image-rs/image-webp/blob/67a0f4eb181525f1cccbf0a73c58cd41977f74cf/src/vp8.rs#L731-L736

FWIW dwebp just errors out with "not enough data" instead.

I don't think treating the incoming bitstream as containing zero bytes is even needed for partial decoding. If the image is truncated, we will decode all the valid parts and write them to the input slice, and then once we raise an error about a premature EOF the valid data stays there. So I think we should just raise an error on an unexpected EOF.

I'll do more fuzzing once this is fixed, because right now I am only getting ~50 executions per second per core.

I am getting at around 10,000 executions per second per core on the fuzzer. Nice!


iOS : MediaView is too small for video

Plugin Version
google_mobile_ads: 5.0.0

Duplicate of https://github.com/googleads/googleads-mobile-flutter/issues/844
See comment https://github.com/googleads/googleads-mobile-flutter/issues/844#issuecomment-1652432974. This is working as intended.


Don't duplicate tests from testng xml suites when maxForks > 1

Context
This pull requests makes test xml suites for TestNG runnable in parallel through a single task, enabling fine-grained control of test execution in parallel, forked JVMs.
Fixes Issue: #2783 - The use of TestNG suites in combination with maxParallelForks causes tests to be execute multiple times.
I think this pull request is a nice first stab at fixing the problem, but I'm not familiar enough with the codebase to say that it's the correct way. I'm willing to implement it differently if need be.
My personal use case is that I work on a legacy project, and certain tests are not allowed to run in parallel within the same JVM.
These tests are in the same module however, and I'd like to speed up our tests by runnign them in parallel in separate JVMs, but cannot, since they are in the same gradle project.
Background
When using TestNG in with gradle, there are two configuration options:
1) Configure TestNG & class scanning through gradle.
This is generally the preferred method as it easily grants access to things like parallelization, rerunning tasks first, and other gradle driven enhancements.
2) Specify or build suite xml files for TestNG to use.
This (in my mind) is usually for legacy projects that need finer grained control. A major draw back is that you cannot run the xml files in parallel JVMs.
The way xml files are handled currently doesn't play well with setting max parallel forks.
The root cause is that the xml files would get passed to every test executor fork as they are created, and get executed one per test executor.
This initial fix moves test suite xml tracking from the test class processor creation to be handled more like test classes - passed around between the executors so they can be assigned one by one.
That fixes Issue: #2783 in the following ways

When one test suite is specified but maxForks > 1, the suite gets executed a single executor only once
When more than one test suite is specified and maxForks > 1, the test suites get assigned round-robin to the test executor, before they are run.
When maxForks = 1, things behave as they have been behaving

Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@big-guy You commented in the issue that sparked this pull request. Could I ask you to take a look at this?

Are there any updates on this one? We've started facing this issue recently and although there is a config workaround, it's quite nasty

@samsieber is this still an issue? If so, can you rebase this on master?


Fix: #346 Voyager hilt support broken with Dagger 2.51

Fixes #346 due to breaking changes done in Dagger 2.51
HiltViewModelFactory
DefaultViewModelFactories

Hey @nvkleban. thanks for the contribution, can you update the API Dump? by calling ./gradlew apiDump