Allow plus(+) in username regex

Adding +(plus) in username regex allow all email address with plus to be unblocked.

Coverage decreased (-0.4%) to 94.032% when pulling ac14705f75488bfec83cb098c5d9993f2eae2051 on pathakamit88:master into d18e535b4bf0c0800251d5c2765d1727e92f8df9 on kencochrane:master.

@pathakamit88 thank you, would it be possible to add a unit test to make sure it works, and to prevent regressions in the future. Unit test would go here, and let me know if you have any question. Thanks again

Closing fixed with #77


Warn if configuration file is present, but not executable

Proof of concept (shamelessly stolen from SO):
// is_executable.c

#include <stdio.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <unistd.h>

int main(int argc, char **argv)
{
    if (argc > 1) {
        struct stat sb;
        printf("%s is%s executable.\n", argv[1], stat(argv[1], &sb) == 0 &&
                                                 sb.st_mode & S_IXUSR ?
                                                 "" : " not");
    }
    return 0;
}

> clang is_executable.c -o is_executable
> ./is_executable ~/.yabairc
/Users/${USER}/.yabairc is executable.

I’m all for that, but people would still need to check their logs to notice - which does not seem to be all that common until told to do so in an issue they create.
Sent with GitHawk

I have wanted to investigate anyways if there was a way to detect if yabai is running from launchctl and then spawn a notification instead. Wrote it down for later.

A solution would be to simply call osascript to create a notification and then exit. The only issue is that the launchservice would keep restarting the process and the user would be spammed woth notifications until fixed.
I’d almost prefer to have yabai fork itself to launch as a background upon start, and provide a yabai —quit command.
Sent with GitHawk

The only issue is that the launchservice would keep restarting the process and the user would be spammed woth notifications until fixed.

Notifications can have identifiers to avoid this sort of situation. I'll play around with this for a bit I think, but let's move this discussion to some future issue.
I think the check I mentioned in the first post should still be implemented and a warning should be printed to stdout.
Another possibility would be to change the Homebrew formula to redirect logs to some file that shows up in Console.app. I will investigate this right now.

Another possibility would be to change the Homebrew formula to redirect logs to some file that shows up in Console.app. I will investigate this right now.

Even better, NSLog prints to both stderr and Console.app at the same time. No need to change the formula. All this would take is implementing some debug function in an Objective-C source file somewhere. Sounds like something that should be done so there's a place to read logs easily.
#import <Foundation/Foundation.h>
#import "log.h"

void debug(const char *message,...)
{
    va_list args;
    va_start(args, message);
    NSLog(@"%@",[[NSString alloc] initWithFormat:[NSString stringWithUTF8String:message] arguments:args]);
    va_end(args);
}

If the config does not have its executable bit set for the owner, we try to set it before executing the config. If we fail to set the required permission, then we abort with an error. I think this should be sufficient for now.


Keep training tera if the initial maximum step is not enough

Dear developer, I'm grateful for the provided repo. As the title, I have train
a tera model, but consider resource limitation, I restrict total training epoch to
about 115 on LibriSpeech 100 hour subset with batch size 8. But I found the result is a bit lower than one report by the paper, and I think it's due to the small batch size and smaller total epoch(but I have run more step than paper, I'm not sure whether running more step will get better result comparing to more epoch). I don't want to train the model again by reset the maximum training step and retrain. But tera train the model with learning rate linearly decay to 0 at maximum training step. I don't know how to keep training it. I'm wondering is there any good way for solving this problem?

Hi, currently we do not support this function.
However, there is a quick workaround:
At the beginning of the pre-training script, manually add a few line of code to load your pre-trained TERA model, and continue the pre-training script like before (you can adjust the optimizer settings in the config file to suit your need). I believe this will solve your problem.

Thanks for the quick reply. I really appreciate it!
If I didn't misunderstand your advice. If I load the pretrain
model first, and start training from script again(assume I use the same script, so it's equivalent to double the training epoch, and the learning rate will cycle for two cycles.)
My experience about training the transformer is, it may be sensitive to learning rate scheduling process. For example, if I didn't use enough warm up step, then the training may crash sometimes. It's there any paper using this kind of learning rate scheduling process(multiple cycle)?May I ask the title of those papers?

Hi, I wouldn't suggest using two cycles. Instead you can modify the optimizer to have no schedule.
Change this line to something else, for example this optimizer by changing the string to "AdamW".
You can also use a different number of epoch on the second attempt.


Fix multilayer routing

Fixes #706

MultiCrossSectionAngles weren't being passed into generate_manhattan_waypoints and round_corners as tuples of CrossSection, (angle, angle) pairs, but instead as lists of CrossSections, which is now fixed.

_get_unique_port_facing now calls get_layer in case the layer is a string rather than a tuple.

Thank you Thomas!


Doesn't work on v45

I use /restart and get the confirmation message but typing confirm after does nothing.
Using /restart again after this doesn't even give me a confirmation and just shows /restart in chat.
Would be nice if this could be fixed when you have the time.

It seems the method name may have changed in v45 - possibly "EndPlayersFiredSequenceClientRpc" now? I'm not 100% sure.

Nevermind, cloned locally and recompiled it. Worked correctly with no issues. Seems like it just needed to be compiled with the latest version of the game's dlls.

Thanks for the info, I'll release a version soon with the latest dlls.


[📑 Docs]: Improve content under Message section on Event-Driven Architectures webpage

What Dev Docs changes are you proposing?
In the Core Concepts section, under the Message sub-heading, I propose to restructure the sentences and improve grammar.
Webpage link is here: https://www.asyncapi.com/docs/getting-started/event-driven-architectures#message
This is the current content of the section:

My suggestions are as follows:

Clean up unnecessary articles (like "the" and "they") and break up the sentences to be more concise.
Use the active voice when talking about message content.

Expected Outcome
The final content under the Message section will be as follows:

Message
A message is a piece of information that's sent by publishers to the broker, and received by all interested subscribers. The messages can contain anything, and are frequently classified as events or commands. As you saw above, events communicate a fact that occurred. On the other hand, commands are very similar to requests in REST APIs. They instruct the subscribers to “do this”.
To be precise, events and commands share the same structure, but differ conceptually.

Please let me know what you think of the changes to this section @alequetzalli and I can incorporate your feedback and file a PR accordingly. Thank you!
Code of Conduct

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

Hi, can I work on this project?

Hello, can I please work on this project?

Hey @glowreeya-01 I have already submitted a PR for this issue. It is seen in the activity log above
https://github.com/asyncapi/website/pull/726

@glowreeya-01  You can also start by reviewing the PR that @meghthak created, and learn from the change she made ❤
Feel free to join our 13_docs channel to pick up more work or suggest a new Docs improvement by creating a Docs GH issue ⭐


Segmentation fault

Hey there,
I get the error " Segmentation fault" when I use any grabc option accept -h.
Am I doing something wrong?

Are you using the version that available in ubuntu? If so, that version is old and not maintained. Please try to fetch the source and compile it yourself. It only requires X11 library.
Thanks.

I'm using it on my Raspberry Pi, using X11.
So you got any thoughts?

The old version 1.1 which is available on pi also works on my pi 3 b+. In any event I compiled uploaded the latest 1.0.2 binary for pi. Please download it from the releases page and see if it works for you.
Thanks


better-initramfs update ?

Hello,
Are you planning to update better-initramfs? There are a couple of updates available since last release.
Only one without update being vconfig if I'm correct.
alpine 3.14.2 > 3.17.2

busybox 1.33.1 > 1.34.1
cryptsetup 2.4.0 > 2.4.3
dropbear 2020.81 > 2022.83
json-c 0.15 > 0.16
libaio 0.3.112 > 0.3.113
libblkid 2.37 > 2.38.1
libgcrypt 1.8.8 > 1.10.1
libgpg-error 1.42 > 1.46
libuuid 2.37 > 2.38.1
lvm2 2.02.188 > 2.03.19
mdadm 4.1 > 4.2
popt 1.18 > 1.19
zlib 1.2.11 > 1.2.13

Thanks!
Cheers

Yes, I just found the project to be mostly feature completed and I do not invest much time into it. There's however many parts that I'd like to entirely rewrite, and add static eudev for proper /run/udev that is then required for systemd and lvm running under systemd.
Anyhow thanks for showing interest, I've just pushed update with new sysroot, I will update deps to new versions and maybe finally add some distfiles checksum validation while at it too.

I'm really glad to hear that @slashbeast, I have been working on it on my own as well.
BTW have you had a chance to revisit my PRs for ip4ll and dns? I was thinking of just rebasing and combining my networking updates, since they are behind master now.

Thanks @slashbeast
I forgot to mention missing ecdsa support for dropbear but noticed you added it already, nice.
https://github.com/slashbeast/better-initramfs/commit/732c3617417762faffb60e53683d4404bf1d9303
I personally added a kexec-tools lebuild a few months to be able to kexec to a different kernel (or pass kernel options) straight from initramfs. It works fine. I don't know if you feel like to add it ? Otherwise, I just let this here for those interested to try.
Size of the compiled binary : 255K
From: jean<EMAIL_ADDRESS>Date: Fri, 17 Mar 2023 10:54:16 +0100
Subject: [PATCH 1/2] add kexec 2.0.24 lebuild

---
 bootstrap/lebuilds/kexec-tools.lebuild | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)
 create mode 100644 bootstrap/lebuilds/kexec-tools.lebuild

diff --git a/bootstrap/lebuilds/kexec-tools.lebuild b/bootstrap/lebuilds/kexec-tools.lebuild
new file mode 100644
index 0000000..dd42607
--- /dev/null
+++ b/bootstrap/lebuilds/kexec-tools.lebuild
@@ -0,0 +1,20 @@
+#!/bin/sh
+name='kexec-tools'
+version='2.0.24'
+sources=( "https://www.kernel.org/pub/linux/utils/kernel/kexec/${name}-${version}.tar.xz" )
+homepage='https://kernel.org/pub/linux/utils/kernel/kexec'
+license='GPLv2'
+
+build_cmd="
+    export CFLAGS=\"\$CFLAGS -static\" && \
+    export CXXFLAGS=\"\$CXXFLAGS -static\" && \
+    cd /source/${name}-${version} && \
+    ./configure --prefix=/binit/prefix && \
+    make -j ${cores_num:-1} && \
+    make install
+"
+deploy="/binit/prefix/sbin/kexec"
+
+fetch_source
+install_source
+build_source
--
2.39.2

From 50621a9586490f7579027657656d67d5db10a918 Mon Sep 17 00:00:00 2001
From: jean<EMAIL_ADDRESS>Date: Fri, 17 Mar 2023 10:59:36 +0100
Subject: [PATCH 2/2] add kexec support

---
 bootstrap/bootstrap-all.conf | 2 +-
 scripts/meta                 | 6 ++++++
 scripts/release              | 2 +-
 3 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/bootstrap/bootstrap-all.conf b/bootstrap/bootstrap-all.conf
index 28de1c3..4864076 100644
--- a/bootstrap/bootstrap-all.conf
+++ b/bootstrap/bootstrap-all.conf
@@ -1 +1 @@
-packages='busybox libaio lvm2 zlib dropbear libuuid popt libgpg-error libgcrypt json-c libblkid cryptsetup askpass mdadm vconfig'
+packages='busybox libaio lvm2 zlib dropbear libuuid popt libgpg-error libgcrypt json-c libblkid cryptsetup askpass mdadm kexec-tools vconfig'
diff --git a/scripts/meta b/scripts/meta
index 7d59ce9..70ac9ff 100755
--- a/scripts/meta
+++ b/scripts/meta
@@ -95,6 +95,12 @@ prepare() {
 		ewarn "No mdadm binary found, Software Raid support will be disabled."
 	fi

+  if test -f kexec; then
+    install -m 755 kexec "${sourceroot}/bin/kexec"
+  else
+    ewarn "No kexec binary found, kexec support will be disabled."
+  fi
+
 	if test -f dropbearmulti; then
 		install -m 755 dropbearmulti "${sourceroot}/bin/dropbearmulti"
 		( cd "${sourceroot}/bin/" && rm -f dropbear dropbearkey dbclient && ln -s dropbearmulti dropbear && ln -s dropbearmulti dropbearkey && ln -s dropbearmulti dbclient )
diff --git a/scripts/release b/scripts/release
index 7c9bd08..bbabd84 100755
--- a/scripts/release
+++ b/scripts/release
@@ -97,7 +97,7 @@ for arch in ${arches}; do
 	# cleanup
 	rm -f "${workdir:-/LEFAIL}"/bootstrap/output/*
 	rm -f "${workdir:-/LEFAIL}"/output/initramfs.cpio.gz
-	rm -f "${workdir:-/LEFAIL}"/sourceroot/bin/{busybox,sh,cryptsetup,lvm,mdadm,dropbearmulti,dropbear,dropbearkey,resume}
+	rm -f "${workdir:-/LEFAIL}"/sourceroot/bin/{busybox,sh,cryptsetup,lvm,mdadm,kexec,dropbearmulti,dropbear,dropbearkey,resume}

 	(
 		cd "${workdir}" || die
--
2.39.2

It might make sense if better-initramfs will get features similar to https://github.com/open-power/petitboot but I do not see use for it. So I will leave it out, however if you feel like it might be of use for others feel free to make pull request with lebuild just leave it out of default installation list for now.

Well it has been like four years, but I reviewed everything. When binit_net_addr=dhcp is set, then nameservers will be set automatically, unless one or more binit_net_nss has been set. I also included link-local support from https://github.com/slashbeast/better-initramfs/pull/49, since that is usually included in dhcp clients anyway, and wouldn't be unexpected. Let me know what you think.


Update name of "DunogeonENG"

Done in https://github.com/arduino/library-registry/pull/2272#issuecomment-1362677068


Helm installation fails using FlexCD.

The CronJob “kube-image-keeper-registry-garbage-collection” has too long a name, and prefixing it with Namespace would exceed the 52 character limit.
This is not a problem with kuik, but could the name be changed to something shorter, for example, “kuik-registry-garbage-collection”?
Warning  InstallFailed       8s                     helm-controller  Helm install failed for release kuik-system/kuik-system-kube-image-keeper with chart<EMAIL_ADDRESS> 1 error occurred: * CronJob.batch "kuik-system-kube-image-keeper-registry-garbage-collection" is invalid:
metadata.name: Invalid value: "kuik-system-kube-image-keeper-registry-garbage-collection": must be no more than 52 characters

duplicate of #425, closing


[Bug] init crash when symlinks are present in plugins/themes/(probably others) on Windows

Describe the Bug

I recently was trying to symlink my plugin repo's plugins to the plugin folder to retain the organization of my repo while still enjoying my plugin updating in real time. I tried creating a symlink to accomplish this, but surprisingly, BD crashed. It looks like fs.statSync throws when called with a symlink path. It also looks like most places statSync is used, lstatSync could be a drop in replacement.
I am happy to submit a PR to fix this but wanted to check-in with the maintainers here to see what goals they had in mind. At the bare minimum, I think that crashing when symlinks are present in the plugins folder should be fixed, but I'd love to add functionality for symlinks.
To Reproduce
1.
New-Item -Pa "<src path>" -ItemT Sy -Val "<plugins folder>"

restart discord

Expected Behavior
No crash.
Discord Version
Stable

I used the command mklink "<plugins folder file>" "<source file>" in an admin command prompt and did not experience this crashing bug.

Well, this is embarrassing, tried to reproduce the issue and got nothing. I'm assuming the file I symlinked had errors causing BD to crash but I don't think we'll ever know.


Ingest a JSON file of voluntary exits

The below feature request from a large Teku user.
Assuming we might have 100s of exits, how could we easily pass these into teku.
teku voluntary-exit --validator-keys=<KEY_DIR>:<PASS_DIR> | <KEY_FILE>:<PASS_FILE>[,<KEY_DIR>:<PASS_DIR> | <KEY_FILE>:<PASS_FILE>...]

We were wondering if there is the possibility to pass a json file into the command teku voluntary-exit. The json file would include all the vals.

As an alternative here that's already implemented, it's possible to pass --config-file to voluntary exit...
So you could, for example, have the config file that specifies a bunch of files, and work that way to accomplish what's in this ticket.
For example
network: "minimal"
validator-keys: ["a.json:a.txt",
                 "b.json:b.txt"
                ]

any other configuration such as the beacon rest endpoint to submit the exit can be added to the file also, or configured on command line...

The configuration file approach seems like a reasonable workaround. Closing.


How to sortable with the  table

I'm trying to sort the tr of the table, it can sorting but the SortableGhost display not correctly.

I'm having the same issue, I was actually thinking of adding a css rule "body > tr { display: table }" but it doesn't seem to apply to my element.
Anyone have an idea ?

Just use helperClass prop to add a class to dragging item to style it the same way as your table.
.sorting-row {
  td {
    /* necessary css rules */
  }
}

I had this problem specifically with Bootstrap tables. I fixed it with a combination of @MartinsLapsa's and @darkrift's answers:
body > tr {
  display: table !important;

  td {
    padding: 8px;
    line-height: 1.428571429;
    vertical-align: top;
    border-top: 1px solid #ddd;
  }
}

@christiangenco: Did you not still have problems with the column widths of the draggable element?
I can't figure out how to get appropriate column widths, since they need to be calculated based on the other rows in the table, which aren't taken into consideration when the row is appended to .

@samharad Ahh yeah, that would be a problem. My table has constant column widths between rows so that wasn't an issue for me.
I think a more general solution here would be to append the cloned sorted element as a sibling of the original instead of on the body.

+1 to @christiangenco's idea of "append[ing] the cloned sorted element as a sibling of the original instead of on the body."

It is possible to do so with helperContainer. However, it does not solve all problems. At least in my case, rows where there's empty space inside columns still get squished because of display: fixed;. I have no idea why though.

@tohu12 currently, there is helper besides the node, so there is no need to query by className

Still getting this issue. Ideally, want a purely CSS solution without having to do stuff onSortStart 🤔

Works fine for me!

Hi,
One thing which I used is: SortableContainer's helperContainer.
My table has a tbody and I'm using useRef to get a ref to tbody and use it in helperContainer.
It works, but the dragged row doesn't take full width, maybe you know how to make it look better)))

Thanks for all the clues above. My current workarrount is:
First, set the width of every colum onSortStart:
    ({ node, helper }: any) => {
      node.childNodes.forEach((td: HTMLTableDataCellElement, index: number) => {
        helper.childNodes[index].style.width = `${td.offsetWidth}px`;
      });
    }

Then, add helperClass like dragging
Then, set helperContainer to the tbody of the table (since all my trs are inside the tbody)
Then, set the css of class dragging to
    display: table;

    // keep it as the your table table
    table-layout: fixed;

That's it

Much thanks sir, you saved my day.


Upgrade to Pulumi 3.9.1 and Pulumi-terraform-bridge v3.5.0

This PR needs the Makefile change like pulumi/pulumi-kubernetes#1664

great catch! fixed


missing method in opcodes

I've been using your getOpcodes() function to extract all the method hashes for a contrac's bytecode, and it seems like it's missing one of the methods.  You can see the details in an issue I posted in another repo that is actually a fork of yours, but is using some different logic under the hood: https://github.com/acuarica/evm/issues/49

Ok...the issue is that the hashed function selector turns out to have 2 leading 0's, therefore it is stored in a JUMP3 opcode, not a JUMP4.  Cheers


typo Fix formatting errors in CREATE-TABLE documents due to line indentation

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


Consider merging with file_selector_windows?

flutter/plugins has a relatively new file_selector plugin, which currently has an endorsed web version, and unendorsed (since we have no automated testing solution yet) desktop implementations in the FDE repository, which will move to flutter/plugins once we have test harness for them. The current file_selector_windows is a standard C++ Windows plugin (largely the same code as the older file_chooser FDE plugin it replaced), which is presumably mostly the same code as what this package is doing via FFI.
Long term, maintaining both versions doesn't seem ideal, so we should consider merging them. file_selector_windows would need to maintain a bit of traditional plugin code in order to get the HWND, but all of the rest could be Dart+FFI. Some options:

Discontinue this package, and replace most of file_selector_windows with code from this implementation.

Downside: no longer usable in a non-Flutter Dart application

Move this to flutter/packages, and tweak its API surface as necessary to allow it to be used as the implementation of most of file_selector_windows, so that the plugin just contains an adapter from the file_selector_platform_interface to this package, plus the small amount of glue code to get the HWND via method channel.

Downside: somewhat more complexity long term (two packages, two API surfaces)

/cc @cbracken

Totally up for that. I'd be more than happy to discontinue this package or migrate it. I suspect 99% of the value of this package is within a Flutter app, and I can migrate a version of this into the win32 examples folder for those who want to use it outside of Flutter.
I've upgraded the code to FFI 1.0, at least -- let me know what you'd like to do and I'm happy to transfer the package and/or the repo.

I'll leave the final call to @cbracken since he'll ultimately be maintaining file_selector_windows. What's easier to maintain may also depend on what the UWP implementation looks like.


Unable to leave a thread when it is initiated and left in offline mode

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go offline
Open any report
Send any comment
Reply in thread to the comment
Send any comment in the thread
Click on the thread header
Click on Leave thread
Go online

Expected Result:
The left thread does not reappear on LHN
Actual Result:
The left thread reappears on LHN
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.22
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/bfbcd26e-71f9-4f65-9ea7-889425389366
https://github.com/Expensify/App/assets/43996225/c5e8142f-b35b-41d1-8282-238671316876
Expensify/Expensify Issue URL:
Issue reported by: @adeel0202
Slack conversation:
https://expensify.slack.com/archives/C049HHMV9SM/p1685430101536709
View all open jobs on GitHub

I'll test soon

I'll test tomorrow

I'm going to close, this one is very similar to the updates planned for these two GHs:
https://github.com/Expensify/App/issues/19982
and
https://github.com/Expensify/App/issues/20197


Derive from path

I asked ChatGPT to rewrite it entirely so that FileObject and DirectoryObject are derived from pathlib.Path, as requested by @liamhuber. I had to correct it a little bit but overall it could be good enough.

I thought about writing DocStrings but the functionalities are super obvious so I leave it like this.


offcanvas toggle bottom

bug:
when offcanvas toggle bottom.browser scrollbar can't hidden

Doesn't make sense.

@newyhj can you please confirm this is still a bug and if so, please describe it in a way we can understand.
Thanks

Close due to no activity.


Added 2 nodes of 1 user

Added:

@dufaultraquildis:matrix.org 2 nodes of the user

@ironoa Element handle of the user was verified


Add buildtest to bash.

This command was missing.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


[BUG] Wrong mouse position when entering [[ in the list

Describe the bug
Wrong mouse position when entering [[ in the list
To Reproduce
enter [[ in list
Expected behavior
The mouse positoin should be in the middle like this: [[|]].
But if I entering [[ in list with outliner-plugin on. the result is: [[]|]. Meanwhile, i cannot move the mouse position left.
I think the problem is in the plugin because error does not occur with the plugin off.
Screenshots

Desktop (please complete the following information):

OS: Windows
Obsidian Version: 1.0.3
Outliner Versoin: 4.2.0

I also just started running into this after the update to 4.2.0.

@Miroier @lamdor thank you for the feedback
@sitaktif it looks like it's because of https://github.com/vslinko/obsidian-outliner/pull/401. it would be great if you could fix it today. if not, i'll fix it.

Added a broken test to the main branch

I see what’s going on, I’ll submit the (straightforward) fix as soon as I can sit in front of my computer (hopefully in the next hour).

Cmd-a is also broken. Looks like getContentStartCh should include checkboxLength. But this change is potentially very dangerous and it needs to be refactored very carefully.

In light of your last two comments, maybe we should we revert the change.

Actually https://github.com/vslinko/obsidian-outliner/issues/402#issuecomment-1358220450 was addressed, and https://github.com/vslinko/obsidian-outliner/issues/402#issuecomment-1358236978 is not a regression, so maybe the right way is forwards!

Fixed in https://github.com/vslinko/obsidian-outliner/releases/tag/4.2.1


what is the ⚠ in top-right corner of instance icon indicative of?

Sorry if I'm too blunt. It's driving me crazy and can't seem to find much info about it. Help? :D

From reading the code it means the instance state is in one these states, creating, pending, or provisioning:
https://github.com/aws-solutions/workload-discovery-on-aws/blob/ef4c6b16066d537fe210fe9da12678c6e9c5dc93/source/frontend/src/Utils/Resources/ResourceStateParser.js#L24C5-L24C5

I didn't really have time to run through the code (also not very clear about its structure, etc), but definitely thanks for the heads up, @svozza
what's interesting about them is that they have been in a running state since Oct 3 / Oct 10 / Oct 25, yet all three of them have the icon - I find that to be inconsistent

I'll look through the code some more, but if anyone has any idea what might be going on, I'd love to hear about it. Thanks :D

same for the database - and that one has been running on since "forever"

Load Balancers of both network and application type seem to have it too.

so I'm either doing something wrong or there is a bug somewhere if this is unexpected behaviour. Probably the former, but what could it be?!

Looks like this might be a bug to me.

adding bug information below
Describe the bug
EC2 instance, RDS instance, ELB icons all have a ⚠ sign next to them although they're all running and have been running for a long time (are not in a different state than running).
To Reproduce
Steps to reproduce the behavior:
unsure how to fully reproduce, as I am running against an already-existing set of resources, but maybe:

create EC2 instance, RDS instance and Network/Application Load Balancer in AWS, wait for them to be in a running state
let Workload Discovery on AWS discover them
import them to a new diagram
observe

Expected behavior
EC2, RDS, LB instances should not have a ⚠ icon on the top-right corner of their icons
Screenshots

Browser (please complete the following information):

Name: Chrome
Version: 118.0.5993.117 (Official Build) (arm64)

Additional context
¯_(ツ)_/¯

I dug into this an it looks like the frontend is expecting the state info for these resource types to be in a different field from the one that is being returned from the backend. Not too difficult to fix either. We will be aiming for a 2.1.3 release with this a few other bugfixes for the end of the year/early next year.

This issue has been fixed in v2.1.3 that was released yesterday.


Generate and display password

Generate password if user picked at least one type of character to be included.
Display it to the webpage.
Otherwise generate an alert explaining that they need to pick at least one character type and repeat prompts

Generate password based on character pool.
Display password to webpage.


RetrievalQA chain with chain_type map_reduce fails for custom prompts

System Info
langchain 0.0.173
python 3.9.16
Who can help?
@hwchase17 @agola11 @vowelparrot
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[X] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[X] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[X] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction

custom_prompt_template = """Use the context to generate an appropriate reply to the query

Context: {context}
Query: {question}
Response:"""
CUSTOM_PROMPT = PromptTemplate(
    template=learn_mode_prompt_template, input_variables=[
        "context", "question"]
)

def generate_response(text: str, query: str):
  retriever = create_document_vectorstore(
            page_text=text)
  chain_type_kwargs = {"prompt": CUSTOM_PROMPT }
  qa = RetrievalQA.from_chain_type(llm=OpenAI(
            openai_api_key=openai_api_key), chain_type="map_reduce", retriever=retriever)
  qa.run(body.query)

Expected behavior
tl;dr trying to use RetrievalQA chain with chain_type of map_reduce (and refine) errors out when using a custom prompt but successfully works with chain_type=stuff
Note this errors out with
ValidationError: 1 validation error for MapReduceDocumentsChain
prompt
  extra fields not permitted (type=value_error.extra)

however if chain_type is changed to stuff the code generates a completion without a problem

qa_chain = RetrievalQA.from_chain_type(
    llm=llm_model,
    chain_type="stuff",
    retriever=retriever,
    #return_source_docummets=True,
)

i think return_source_docummets is problem

i came to the same problem using ConversationalRetrievalChain. you may try this
chain_type_kwargs={"combine_prompt": CHAT_COMBINE_PROMPT, "question_prompt": QUESTION_PROMPT}

`prompt_template = """
{context}
{question}
"""
PROMPT = PromptTemplate(
template=prompt_template, input_variables=["context", "question"]
)
chain_type_kwargs = {"prompt": PROMPT}
qa = RetrievalQA.from_chain_type(llm=chat, chain_type="stuff", retriever=retriever, chain_type_kwargs=chain_type_kwargs,verbose=True)
`
I did it this way

i came to the same problem using ConversationalRetrievalChain. you may try this
chain_type_kwargs={"combine_prompt": CHAT_COMBINE_PROMPT, "question_prompt": QUESTION_PROMPT}

is it wored for you @Willy-J
?

i came to the same problem using ConversationalRetrievalChain. you may try this
chain_type_kwargs={"combine_prompt": CHAT_COMBINE_PROMPT, "question_prompt": QUESTION_PROMPT}

is it wored for you @Willy-J ?

yes, it works on versin 0.0.179

Okay thanks 👍. Let me try with the same version.

I was able to fix this.
you need to look, for each chain type (stuff, refine, map_reduce & map_rerank) for the correct input vars for each prompt.
Check the attached file, there I described the issue in detail. And how figured out the issue looking at the Langchain source code for the original/default prompt templates for each Chain type.
bing_chain_types.md

Thank you bro. It's very detailed

I was able to fix this. you need to look, for each chain type (stuff, refine, map_reduce & map_rerank) for the correct input vars for each prompt.
Check the attached file, there I described the issue in detail. And how figured out the issue looking at the Langchain source code for the original/default prompt templates for each Chain type.
bing_chain_types.md

Thank you for your resource. However I found that the approach to map_reduce as stated in your file is incorrect. My code is as follow:
map_template = """Make a summary for these documents. Keep the summary as accurate and concise as possible.\n\n{text}"""
MAP_CHAIN_PROMPT = PromptTemplate(template=map_template, input_variables=["text"])

combine_template = """Use the following summary to answer the question at the end. If you don't know the answer, just say that you don't know, don't try to make up an answer. However try to guess the closest answer as you can. Keep the answer as concise as possible.

{summaries}

Question: {question}

Return the answer in this format. Use only the capital letters for the answer.
{{ "Answer": "A" , "Explanation": "Explanation about the option"}}

"""
COMBINE_CHAIN_PROMPT = PromptTemplate(template=combine_template, input_variables=["summaries", "question"])

map_reduce_qa_chain = RetrievalQA.from_chain_type(
    llm,
    retriever=vectordb.as_retriever(),
    chain_type="map_reduce",
    chain_type_kwargs={"map_prompt": MAP_CHAIN_PROMPT, "combine_prompt": COMBINE_CHAIN_PROMPT, "document_variable_name": "summaries"},
)

The error is as follow:
TypeError: langchain.chains.combine_documents.map_reduce.MapReduceDocumentsChain() got multiple values for keyword argument 'document_variable_name'
I'm using langchain=0.0.264. Do you have any idea what might cause the issue?

I was able to fix this. you need to look, for each chain type (stuff, refine, map_reduce & map_rerank) for the correct input vars for each prompt.
Check the attached file, there I described the issue in detail. And how figured out the issue looking at the Langchain source code for the original/default prompt templates for each Chain type.
bing_chain_types.md

Thank you for your resource. However I found that the approach to map_reduce as stated in your file is incorrect. My code is as follow:
map_template = """Make a summary for these documents. Keep the summary as accurate and concise as possible.\n\n{text}"""
MAP_CHAIN_PROMPT = PromptTemplate(template=map_template, input_variables=["text"])

combine_template = """Use the following summary to answer the question at the end. If you don't know the answer, just say that you don't know, don't try to make up an answer. However try to guess the closest answer as you can. Keep the answer as concise as possible.

{summaries}

Question: {question}

Return the answer in this format. Use only the capital letters for the answer.
{{ "Answer": "A" , "Explanation": "Explanation about the option"}}

"""
COMBINE_CHAIN_PROMPT = PromptTemplate(template=combine_template, input_variables=["summaries", "question"])

map_reduce_qa_chain = RetrievalQA.from_chain_type(
    llm,
    retriever=vectordb.as_retriever(),
    chain_type="map_reduce",
    chain_type_kwargs={"map_prompt": MAP_CHAIN_PROMPT, "combine_prompt": COMBINE_CHAIN_PROMPT, "document_variable_name": "summaries"},
)

The error is as follow: TypeError: langchain.chains.combine_documents.map_reduce.MapReduceDocumentsChain() got multiple values for keyword argument 'document_variable_name'
I'm using langchain=0.0.264. Do you have any idea what might cause the issue?

In the map_reduce_chain you are trying to create, try it without passing the document_variable_name, you are using the default input vars so you do not need to pass them again.
Hope that will fix your issue.

Just to put everything together with regards to  map_reduce
from langchain.embeddings.sentence_transformer import SentenceTransformerEmbeddings
from langchain.vectorstores import Chroma
from langchain.prompts import PromptTemplate
from langchain.chains import RetrievalQA
from langchain.chat_models import ChatOpenAI
from decouple import config

TEXT = ["Python is a versatile and widely used programming language known for its clean and readable syntax, which relies on indentation for code structure",
        "It is a general-purpose language suitable for web development, data analysis, AI, machine learning, and automation. Python offers an extensive standard library with modules covering a broad range of tasks, making it efficient for developers.",
        "It is cross-platform, running on Windows, macOS, Linux, and more, allowing for broad application compatibility."
        "Python has a large and active community that develops libraries, provides documentation, and offers support to newcomers.",
        "It has particularly gained popularity in data science and machine learning due to its ease of use and the availability of powerful libraries and frameworks."]

meta_data = [{"source": "document 1", "page": 1},
             {"source": "document 2", "page": 2},
             {"source": "document 3", "page": 3},
             {"source": "document 4", "page": 4}]

embedding_function = SentenceTransformerEmbeddings(
    model_name="all-MiniLM-L6-v2"
)

vector_db = Chroma.from_texts(
    texts=TEXT,
    embedding=embedding_function,
    metadatas=meta_data
)

combine_template = "Write a summary of the following text:\n\n{summaries}"
combine_prompt_template = PromptTemplate.from_template(
    template=combine_template)

question_template = """Use the following pieces of context to answer the question at the end. If you don't know the answer, just say that you don't know, don't try to make up an answer. Use three sentences maximum. Keep the answer as concise as possible. Always say "thanks for asking!" at the end of the answer.
{context}
Question: {question}
Helpful Answer:"""
question_prompt_template = PromptTemplate.from_template(
    template=question_template)

# create chat model
llm = ChatOpenAI(openai_api_key=config("OPENAI_API_KEY"), temperature=0)

# create retriever chain
qa_chain = RetrievalQA.from_chain_type(
    llm=llm,
    # mmr > for diversity in documents
    # Set fetch_k value to get the fetch_k most similar search. This is basically semantic search
    retriever=vector_db.as_retriever(
        search_kwargs={'fetch_k': 4, 'k': 3}, search_type='mmr'),
    return_source_documents=True,
    chain_type="map_reduce",
    chain_type_kwargs={"question_prompt": question_prompt_template,
                       "combine_prompt": combine_prompt_template}
)

# question
question = "What areas is Python mostly used"

# call QA chain
response = qa_chain({"query": question})

print(response)

print("============================================")
print("====================Result==================")
print("============================================")
print(response["result"])

print("============================================")
print("===============Source Documents============")
print("============================================")

print(response["source_documents"][0])


Running the tutorial takes too long

Hi Circle_finder team,
I am using your circle_finder pipeline and I tried to run the tutorial example code (with the two 5GB fastq files you provided as input). I specified 16 cores but it takes too long to finish. It has been about 24 hours and it is still not finished.
the code I use:
bash ./microDNA.InOne.sh /data2/kjiajinlong/reference/hg38 /data2/kjiajinlong/data/test/Index11_1.fq /data2/kjiajinlong/data/test/Index11_2.fq 16 testrun 49 10000 /data2/kjiajinlong/software/Circle_finder/scripts
Is there something wrong with my code? Or is this pipeline inherently slow?
Thanks,
Jiajinlong Kang

Hi，l meet the same error, could you tell me some advice to deal with this error?

I also encountered the same problem, could you tell me how to solve it？


Feature Request: Provide way to delete individual entries from history

Currently, if I want to remove a specific entry from the history, the only way I see to do that is to clear the entire history. Perhaps I'm missing something, but I would like to be able to activate Clipy, navigate to one of the items in the history and delete the selected item, perhaps by pressing the delete key.

I think that simply pressing the Delete key might lead to accidentally deleting items you didn't want to delete. But I also don't like the way that some applications try to guard against those accidents by asking for confirmation for every deletion! That gets tiring very quickly, in my opinion.
What I prefer is sliding/swiping the mouse or trackpad over the desired item gives it a more intentional feel. But I'm open to other suggestions.
All of that said, I'm not a programmer! I don't know how difficult it is to implement any of this.

I don't really care what keyboard or mouse action is necessary to delete an item, but I really would like a way to remove one or more selected items. Even something as simple as right clicking on an item to get to a delete option.

Barring the ability to remove an individual entry, is there a way to manually edit the history. Even if I have to quit Clipy, edit and restart Clipy, I would really like to be able to remove specific entries from Clipy history without clearing everything.

For what it is worth, I have tried the beta feature "Delete history" where you assign a modifier to use with a mouse click to delete a specific entry from the history. This completely fills my needs. I suggest renaming it to "Delete item from history" so that it is clear that you are not deleting the whole history.

danny thank you!


Documentation: Complex Number

Do the checklist before filing the issue:

[x] Do you have Node.js and all the relevant dev-dependencies installed?
[ ] Is this a bug fix?
[x] Is this an enhancement?
[ ] Is this a feature request?

NOTE: Provide a clear and concise description of the feature that needs to be added! Or if its a bug, then provide the necessary steps to reproduce it along with screenshots.

Give your answer below:

Provide brief documentation on M.Complex() inside the docs folder!
A blank index.md is already provided for each module
For exact formatting, refer here!

I want to work on this

@sweezy793 you're assigned


Revert "Add iOS exception handling for beta review"

Reverts Expensify/Expensify.cash#3166

Merging to fix broken iOS staging deploys

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Document memory-efficient Entity-Repository pattern

The Entity-Repository pattern is perfect for complementing usage of this repo, but something not obvious from the Entity-Repository Pattern is how it can be utilised in a memory-efficient way.
Basically, any function that returns a collection of Entities should do it via a callback to the original ResultSet.
function getChildren(Entity $parent):EntityCollection<Entity> {
    $resultSet = $this->db->fetchAll("getChildrenByParentId", $parent->getId());

    $entityCollection = new EntityCollection();
    $entityCollection->setIteratorCallback(function() use $resultSet:?Entity {
        $resultSet->next();
        if(!$resultSet->valid()) {
            return null;
        }
        return $this->rowToEntity($resultSet->current());
    });

    return $entityCollection;
}

Original issue text:
The repository pattern is a design pattern that describes an object that represents a mutable list of data, and is perfect for answering the questions raised by #80.
Problem: $row = $database->fetch("customer/getById", 123) it is obvious that here we are querying the customer table and retrieving the row with an ID of 123, but to the code and IDE it is not obvious to what functionality or parameters $row has.
Solution: Use of CustomerRepository, a developer-defined class which is constructed with reference the Customer query collection (promoting database encapsulation).
class CustomerRepository {
	protected $db;

	public function __construct(QueryCollection $db) {
		$this->db = $db;
	}

	public function getById(int $id):Customer {
		$row = $this->db->fetch("getById", $id);
		return new Customer(
			$row->firstName,
			$row->lastName,
			// ... etc.
		);
	}
}

The idea is simple: let the developer have complete control over how the Entity objects are created. For example, it may not be that all Entity object constructors take a Row parameter... so this allows arbitrary classes to be used as Entities.
Strict rule: All database mutations should be done within their own Repository classes. Entities can have functionality, but only functionality that is calculating something based on its own internal data structure.

<?php
abstract class Entity {}

class User extends Entity {
	public function getUsername():string { return "bob"; }
}

/** @template T */
class Repository {
	/** @return T */
	public function getById(string|int $id):Entity {

	}
}

/** @extends Repository<User> */
class UserRepository extends Repository {

}

$userRepo = new UserRepository();
$user = $userRepo->getById(123);

// This is now type-hinted correctly!
$user->getUsername();


--editable does not work with manifest mode

Describe the bug
./vcpkg install --editable
...
-- Cleaning sources at /Users/leanderSchulten/git_projekte/Lichtsteuerung/vcpkg/buildtrees/aubio/src/0.4.9-f1b891adc9.clean. Use --editable to skip cleaning for the packages you specify.

Even if you add the --editable argument, it cleans the sources.
To Reproduce
Steps to reproduce the behavior:
See description
Expected behavior
Respect the --editable argument

@autoantwort, Currently, the option --editable is not supported in manifest mode
PS F:\Feature-test\test\vcpkg> ./vcpkg install --editable
Error: The option --editable is not supported in manifest mode.
Using manifest file at F:\Feature-test\test\vcpkg\vcpkg.json.

Hello,
is there maybe some update or some workaround possibility expect not to use manifest mode?
Maybe by using a vcpkg-configuration.json file? Here you can select an overlay-port where you can reference an edited fork commit -> but this would force you to commit every little code change which is not sensefull.

No there is no update and it still requires classic mode.

Hi all,
Can you please advise a good workflow for when you're working in manifest mode and need to make a package editable?
My project is setup so that I have vcpkg imported as a git submodule, a vcpkg.json manifest in the project root and then I configure with cmake using the following
cmake -DCMAKE_TOOLCHAIN_FILE=../3rd_party/vcpkg/scripts/buildsystems/vcpkg.cmake ../

This all works well until I need to make a quick patch to a large dependency (OpenCV) which I'd like to do in vcpkg/buildtrees rather than go for the long winded alternative which I think would be 'patch opencv repo, update port file, repull all source and rebuild from scratch'. (On my machine OpenCV takes almost an hour to build from scratch).
In this situation is the best solution currently to do the following?

Run 'vcpkg install PACKAGE --editable' (in a separate root folder so that vcpkg.json doesn't interfere)
Edit the source code in the new dir that lacks the .clean suffix
Run 'vcpkg remove PACKAGE' followed by 'vcpkg install PACKAGE --editable'
Manually copy the new files from vcpkg/installed to the cmake build/vcpkg_installed folder (or hack your CMakeLists.txt file somehow)

Any advice here would be much appreciated. It all seems a bit clunky so if I've got the wrong end of the stick here or have missed something obvious please let me know. :)
Many thanks
Alan

As a workaround you can set _VCPKG_EDITABLE property in a triplet file like this:
if (PORT MATCHES "<port-name>")
  set(_VCPKG_EDITABLE ON)
endif()

It will build a particular port as editable, but unfortunately it will rebuild all other ports.

I have serveral projects that depend on gRPC:

gRPC is built with native triplet, which doesn't need to be edited, but required to keep the source code so that I can step into when debugging. Without --editable, however, sometimes the source code will be cleaned.
In these projects, I have to use manifest mode for specifing versions of gRPC and its dependencies, otherwise they will be compiled again after updating vcpkg, which takes a lot of time.

I think the lack of option --keep causes the problem:

--editable means that the source code in buildtrees is necessary, and may be "dirty". It is dangerous to be able to edit the source of a package in manifest mode, because the modifications from different projects may have conflicts.
clean means users doesn't take care of the source code of packages, so vcpkg have permission to remove sources when rebuilding for a new project.
So we can add a option for users to tell vcpkg that never removing sources, and reverts all modifications when rebuilding.

+1 to this.
Having --editable under manifest mode will drastically speed up project bring-ups and migrations. Current methods to patch dependencies make iteration extremely long-winded and unnecessarily complex:

Clone the dependency repo, make changes, commit, push to some repository, create overlay portfile for the dependency with the new commit (slowest)
Clone the dependency repo, make changes, make patchfile, create overlay port for the dependency, specify patchfile under PATCHES (less slow)
Clone a the dependency repo, make changes, compile, copy binaries/headers to VCPKG_INSTALLED_DIR (not scalable for projects with lots of build artifacts)
Set _VCPKG_EDITABLE in an overlay triplet (unnecessarily rebuilds everything else; doesn't necessarily work because binaries could be cached)

editable and head should be able to be set in the vcpkg.json manifest for particular ports because eg. currently I'm hitting bugs that I can't reproduce in classic mode but the bug exists in manifest mode only.
In my case, a library is linked statically against MSVC CRT even if I'm using x64-windows triplet and I can't reproduce it in classic mode. Something like: cmake cl : Command line warning D9025 : overriding '/MDd' with '/MTd'
It would be very helpful to set head and editable in the manifest vcpkg.json so I can add debug messages to the CMakeLists.txt to find out why is this happening, it would save a lot of commits and git reset and similar things.

Having --editable under manifest mode will drastically speed up project bring-ups and migrations. Current methods to patch dependencies make rapid iteration impossible, especially given that manifest mode is the recommended way to manage packages.

Expanding on @sharadhr 's answer, I think one of the most advantageous things about something like
{
  "dependencies": [
   {
    "name": "foo",
    "default-features": false,
    "features": ["bar"] ,
    "editable": true
   }
  ]
}

is the ability to pseudo-cache.
I agree with what others have suggested for a single-project workflow; just use the CLI to develop with --editable. However, some projects which use CMake or CMakePresets might be in development themselves, using Manifest  Mode to include many many dependencies. If something in the root-level CMakeLists.txt file breaks, a rebuild will always happen in the current implementation.

I am also interested in potential workflow for manifest mode. We are looking to migrate our project to vcpkg and our project uses a fair bit of dependencies. For us it is important to maintain reasonable iteration times when working on our dependencies, because we do active debugging and development on them as required. Naturally, safety is not a priority here - fast iteration times and ability to debug from IDE are. If something breaks we can always clear files of dependency in question and rebuild it.

Is there any progress at all on this? It has been almost 4 years since this was reported.
This is a very glaring shortfall in vcpkg. The competition, Conan has this functionality which works seamlessly and as expected.

Here is the simplest workaround I can think of:

Rename the vcpkg/buildtrees/<port-name>/src/xxx.clean directory (which contains the source code) to something that does not end in .clean, say a
Comment out vcpkg_from_git / vcpkg_from_github etc in the portfile.cmake to skip the repo cloning.
Let's say this is a cmake-based port that calls  vcpkg_cmake_configure . Hard code the path into the SOURCE_PATH parameter so it looks something like : vcpkg_cmake_configure(  SOURCE_PATH "vcpkg/buildtrees/<port-name>/src/a" ...)
Now if you can run vcpkg install, vcpkg should not be cleaning up sources so you can edit as you like. Just remember to undo the previous steps when you are done.


Connect endpoints to db

I added a library called bookshelf that allows us to create models of our database objects. I also added bluebird for some convenient functions involving promises.

Something to note is that the tests will run with a database connection to a sqlite3 DB that is held in memory so you don't have to start a DB server for testing. This allows the automated tests in GitHub actions to run without their own database server. This was all done in commit 9abbfff if you want to take a look. I don't know if there are significant differences between sqlite3 and postgres that will lead to differences in testing vs. real behavior. I don't think we should have any problems since knex will deal with differences in the interface between the two. Ideally I would use a postgres instance for testing, but postgres cannot be run in memory 🤷♂.


Chat: persist tool events

Thank you for contributing to the Cohere Toolkit!

[ ] PR title: "area: description"

Where "area" is whichever of interface, frontend, model, tools, backend, etc. is being modified. Use "docs: ..."  for CI changes.
Example: "deployment: add Azure model option"

[ ] PR message: Delete this entire checklist and replace with

Description: a description of the change
Issue: the issue # it fixes, if applicable
Dependencies: any dependencies required for this change

[ ] Add tests and docs: Please include testing and documentation for your changes

[ ] Lint and test: Run make lint and make run-tests

AI Description

This PR introduces a new ToolCall model and makes changes to the database, CRUD operations, and routers to support tool calls.

Database Models:

A new ToolCall model is added to src/backend/database_models/tool_call.py.
The Message model in src/backend/database_models/message.py is updated to include a relationship with ToolCall.

CRUD Operations:

create_tool_call and list_tool_calls_by_message_id functions are added to src/backend/crud/tool_call.py for creating and listing tool calls.
ToolCall is imported and used in the test_create_tool_call and test_list_tool_calls_by_message_id test functions in src/backend/tests/crud/test_tool_call.py.

Database Migration:

A new table, tool_calls, is created in the database using Alembic commands in src/backend/alembic/versions/038075cd8530_.py.
The table includes columns for name, parameters, message_id, id, created_at, and updated_at.

Routers:

The chat_stream and chat functions in src/backend/routers/chat.py are updated to include next_message_position as a parameter.
The get_conversation function in src/backend/routers/conversation.py is modified to exclude messages with tool calls temporarily until the frontend is ready to handle them.

Schemas:

The MessageBase class in src/backend/schemas/message.py is updated to include a list of ToolCall objects.

Services:

tool_call_crud and ToolCallModel are imported and used in src/backend/services/chat.py.
A new function, save_tool_calls_message, is added to save tool calls and their associated messages to the database.
The handle_stream_tool_calls_generation function is modified to call save_tool_calls_message when should_store is True.
The is_custom_tool_call function is updated to check if any tool name is not in the AVAILABLE_TOOLS.

Codecov Report
Attention: Patch coverage is 88.78505% with 12 lines in your changes missing coverage. Please review.

Project coverage is 85.43%. Comparing base (f223dbe) to head (ef9b875).

Files
Patch %
Lines

src/backend/services/chat.py
61.53%
5 Missing :warning:

src/backend/chat/custom/custom.py
63.63%
4 Missing :warning:

src/backend/alembic/versions/038075cd8530_.py
80.00%
3 Missing :warning:

Additional details and impacted files
@@                   Coverage Diff                    @@
##           luisa/persist_events     #282      +/-   ##
========================================================
+ Coverage                 85.34%   85.43%   +0.08%
========================================================
  Files                       153      158       +5
  Lines                      5719     5821     +102
========================================================
+ Hits                       4881     4973      +92
- Misses                      838      848      +10

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


Added SQL Server Diagnostic Book to marketplace

This PR adds my extension (https://tsql.tech/the-sql-diagnostic-jupyter-book/) to the Marketplace
https://github.com/EmanueleMeazzo/SQL-Server-Diagnostic-Book

@EmanueleMeazzo looks like you forgot to update the total count for the insider gallery file.

I also need to make changes to this file, to avoid conflicts, i will merge your PR and then make changes on your behalf.


Running flutter leads to BITS Transfer which fails

I just installed flutter and ran the flutter command for the first time, which led to:
Checking Dart SDK version...
Downloading Dart SDK from Flutter engine fee001c93f25a1e7258e762781a7361f122d29f5...
Downloading the Dart SDK using the BITS service failed, retrying with WebRequest...

This process takes an eternity.

@Tobi696 Is it still relevant?

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of ' flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.

I just installed flutter and ran the flutter command (flutter --version) for the first time its showing these

@azrinsultana

Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of ' flutter doctor -v' please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


.aar Library Support For Extension

From Many Days AppInventor Team Is Able To Add Support Of Many Features  ....Which Was Supposed To Impossible To For External component :/
But adding .aar support For Extension Will Be More Awesome :) If AppInventor Team Will Help To Add This Feature than It Would Be Great :)

Duplicate of https://github.com/mit-cml/appinventor-sources/issues/2196


Added lib generated files to send telemetry data to VAC API

Added all generated files for go-client to send telemetry data of cluster to VAC API.

Closing this PR as this was just testing purpose. @mpermar, will check your comment in this PR


Write a Kotlin program to convert temperature from celsius to kelvin

Description
Write a Kotlin program to convert temperature from celsius to kelvin
Input  ( C ) : -40
Output ( K ) : 233.15

Contributed by - @magar51

How to contribute

Comment !assign to assign this issue to yourself
Fork this repository
Create a new branch
Save the solution in program/program/convert-temperature-from-celsius-to-kelvin/ConvertTemperatureFromCelsiusToKelvin.kt
Commit the changes
Create a pull request

!assign


Error with campaign ID when generating report

[+] A total of 1 campaign IDs have been provided for processing.
[+] GoReport will process the following campaign IDs: 88
[+] Now fetching results for Campaign ID 88 (1/1).
[!] There was a problem fetching this campaign ID's details. Make sure your URL and API key are correct. Check HTTP vs HTTPS!
L.. Details: super() takes at least 1 argument (0 given)
[+] Success!
[!] Looks like campaign ID None does not exist! Skipping it...
[!] There was a problem processing campaign ID 88!
L.. Details: 'NoneType' object has no attribute 'timeline'

Any suggestions on how to fix this?
btw, both url and id are correct and im running http.

Hi Chris,
I experience the same issue here, please help to fix this.

Hey Chris,
Getting the same issue, please help!

Facing same issue , kindly guide to resolve
Thanks

I am using version 0.9.0. I even tried to copy the database to the older version op Gophish, where Goreport used to work in the past, and that didn't work as well. Perhaps it has something to do with campaigns that have been conducted via the latest version?

Solved the issue by :
python3 -m pip install --upgrade urllib3

I traced the issue to error :
urllib3.exceptions.LocationParseError: Failed to parse: https://:3333/api/campaigns/
the solution was :
python3 -m pip install --upgrade urllib3

I traced the issue to error :
urllib3.exceptions.LocationParseError: Failed to parse: https://:3333/api/campaigns/
the solution was :
python3 -m pip install --upgrade urllib3

Thanks for the suggestion but it did not work for me. Hope that @chrismaddalena can fix this issue.

I am also facing the same issue.
[+] A total of 1 campaign IDs have been provided for processing.
[+] GoReport will process the following campaign IDs: 5
[+] Now fetching results for Campaign ID 5 (1/1).
[!] There was a problem fetching this campaign ID's details. Make sure your URL and API key are correct. Check HTTP vs HTTPS!
L.. Details: Failed to parse: https://:3333/api/campaigns/5
[+] Success!
[!] Looks like campaign ID None does not exist! Skipping it...
[!] There was a problem processing campaign ID 5!
L.. Details: 'NoneType' object has no attribute 'timeline'

I think the tool is broken...

Okay, so I did some debugging.
The error is triggered here: https://github.com/chrismaddalena/Goreport/blob/master/lib/goreport.py#L433

I also ran into a similar issue a while ago and my fix was to DISABLE my
iptables firewall. Once I disabled the firewall, the GoReport script ran
and generated reports as expected.
Hope this helps someone.
Cheers
On Fri, Jul 10, 2020 at 8:14 AM Vincent Cox<EMAIL_ADDRESS>wrote:

Okay, so I did some debugging.
The error is triggered here:
https://github.com/chrismaddalena/Goreport/blob/master/lib/goreport.py#L433
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/chrismaddalena/Goreport/issues/25#issuecomment-656698720,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH5LZHQ2TYX2IEL64FHL34DR24O5TANCNFSM4JUGSHKA
.

@kpomeroy1979, did you ran it over HTTP instead of HTTPS? I am digging in the code and I think there is something about that.

@kpomeroy1979 , just tested, it has no effect... I have no idea why it's failing

I followed the trace and I ended up at: https://github.com/chrismaddalena/Goreport/blob/master/lib/goreport.py#L230
This is where the error occurs

Okay I found the fix!
git clone git://github.com/urllib3/urllib3.git
python setup.py install

Have a nice weekend guys!

I ran it over HTTPS.
Have you ever been able to successfully run a report?
Also make sure your permissions are right on the directories (just a shot
in the dark).
Check your goreport/gophish.conf and make sure your gp_host is correct. It
should be https://x.x.x.x:3333/ and that your API key is correct.
Also, when I had my firewall ENABLED and was getting errors, I saw this in
the output and fixed it like so. There was also another reference to an ot
of date python utility but I can't remember now off the top of my head
which one it was.
ERROR: gophish 0.4.0 has requirement python-dateutil==2.6.0, but you'll
have python-dateutil 2.8.1 which is incompatible.
pip uninstall python-dateutil
apt-get remove --auto-remove python-dateutil
apt-get purge --auto-remove python-dateutil
pip install python-dateutil==2.6.0
On Fri, Jul 10, 2020 at 8:31 AM Vincent Cox<EMAIL_ADDRESS>wrote:

@kpomeroy1979 https://github.com/kpomeroy1979, did you ran it over HTTP
instead of HTTPS? I am digging in the code and I think there is something
about that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chrismaddalena/Goreport/issues/25#issuecomment-656706978,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH5LZHXM2XLBRPL2HTOQSLLR24Q5FANCNFSM4JUGSHKA
.

That was the other one I had to fix, urllib3.
good job.
On Fri, Jul 10, 2020 at 8:49 AM Vincent Cox<EMAIL_ADDRESS>wrote:

Okay I found the fix!
git clone git://github.com/urllib3/urllib3.git
python setup.py install
Have a nice weekend https://www.youtube.com/watch?v=IwLSrNu1ppI guys!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chrismaddalena/Goreport/issues/25#issuecomment-656715484,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH5LZHQJZQQY5GDKOJ3BJCDR24S6TANCNFSM4JUGSHKA
.


Refactor match step

I suggest a new method signature for match step which accept a list of functions.

:100:


Handle highway=construction

We need a way to handle highway=construction
This route is not possible because of construction:
https://routing.anyways.eu/projects/bikeforbrussels/#14.43/50.8567/4.4219?sb=true&o=4.4173059,50.8537517&d=4.4153025,50.8550967&p=profile1

https://www.openstreetmap.org/changeset/80093939#map=16/50.8591/4.4110

We need to render the construction tagged highways I think, to make this clearer.
WDYT @pietervdvn ?

Slap a 'bicycle=dismount' on them by default would do the trick IMHO ;p
This way, we would still go over it, but it'll cost a bit more so will only take it pays of (kindof)


Fix poetry install url

Description
the previous url send to a 404
Related Issue
Nan
Type of Change

[ ] 📚 Examples / docs / tutorials / dependencies update
[x] 🔧 Bug fix (non-breaking change which fixes an issue)
[ ] 🥂 Improvement (non-breaking change which improves an existing feature)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🔐 Security fix

Checklist

[x] I've read the CODE_OF_CONDUCT.md document.
[x] I've read the CONTRIBUTING.md guide.
[x] I've updated the code style using make format-code.
[x] I've written tests for all new methods and classes that I created.
[x] I've written the docstring in Google format for all the methods and classes that I used.

Thanks @Tonow ! I merge the change in a separate merge request (needed to change the Makefile as well).


Provide a decompress method which limits the size of the output buffe…

…r allocated when the decompressed size is known.
Motivation:
Limiting the size of buffer allocation can help mitigate the risks of compression bombs. In cases where these could be a legitimate attack vector, we should provide the library caller with the option of limiting the buffer size.
This is necessary to provide Brotli support for RFC8879 https://datatracker.ietf.org/doc/rfc8879/ in netty.
Changes:

Add decompressKnownLength method to Decoder.java to allow for decompression of known-size payloads.
Add decompressKnownLength method to DirectDecompress.java to be consistent with decompress.
Modify DecoderJNI.java and decoder_jni.cc to allow users to specify the output buffer size.

Result:
Users which believe that they have known-size payloads can decompress those payloads in linear space complexity based on the expected size rather than unknown space complexity, based on the actual size. This was verified with async-profiler which clearly shows reduced allocation when providing a known size.

@slandelle PTAL

ping @slandelle @hyperxpro please LMK if there's anything I can help with to move this one along. thanks.

@CamW Can you do PR @google/brotli directly? Once the change is pulled, I will sync Brotli4j with upstream and release.

Any updates? @CamW


ignore root path, no need strip traling slash

Sometimes I just want use that {{ "/" | SafeUrl }}, but in this case will generate // path,
so we need ignore root path.

Merged.

Oh, and thanks for the contribution.
The license change in Hugo is dragging out in time, so could you also sign off on
discuss.gohugo.io/t/switching-to-apache-2-license/173/32

@bjornerik done.


Clear variables when getting i18n files

Not sure how you prefer:
(a) clear variables before passing to method
(b) clear variables inside METH
This PR implements (b).
Problem:
A non-cleared variable lv_ext
https://github.com/abapGit/abapGit/blob/1269360946d0625b2b671bc3171b01d0d6a96c9d/src/objects/core/zcl_abapgit_objects_files.clas.abap#L407

Contributes to #6980

I prefer (b). But in this case, the clear should be inside of zcl_abapgit_filename_logic=>i18n_file_to_object 😉


context

Hi,
I have an idea / suggestion / found a bug / have question / want to contribute.
Here's some code
using Entitas;

public sealed class MyComponent : IComponent {
}

Cheers

Your issue "context" is missing context ;)

I guess that's just the issue template. Feel free to create a new issue or to reopen, if you have a question


MISC: PREMs Overwrite LENGTH_OF_STAY_DAYS with calculation

Added configuration to LengthOfStayMapper: Overwrite existing length by default
Changed length of stay logic to ignore time of day when calculating stay duration
Re-added Discard-IP-ShortStay discard filter

If you encounter issues running generate_compose_yaml due to a tzlocal error, there is a test branch here which may fix your issue.
Testing instructions

Build with -Pdocker and navigate to compose-cluster
Clean up any old CARDS docker images: docker-compose down && docker-compose rm && docker volume prune -f && ./cleanup.sh
Prepare docker with the following flags: python3 generate_compose_yaml.py --mssql --cards_project cards4prems --oak_filesystem --dev_docker_image --composum --adminer
Start docker: docker-compose build && docker-compose up -d
Nacigate to http://localhost:1435/ and sign in:

System: MS SQL (beta)
Server: mssql
Username: sa
Password: testPassword_
Database: master

Select Import on the left bar
Upload cards/compose-cluster/mssql/processorImportTest.sql and execute. It should say 38 queries executed OK
Login to CARDS as admin
Navigate to http://localhost:8080/Subjects.importClarity. A mostly empty page should show success and Data import started
Navigate to the daashboard http://localhost:8080/content.html/Questionnaires/User and check which of patients 34 through 36 were imported.

Expected result:
Patients 34 and 35 should not be imported due to a length of stay of less than 1 day (10 hours or 23 hours 58 minutes)
Patient 36 should be imported as an inpatient or long form inpatient due to an 11 minute visit crossing midnight

docker-compose down && docker-compose rm && docker volume prune -f && ./cleanup.sh
Repeat steps 3 through 8
Delete the configuration for LengthOfStayMapper using http://localhost:8080/system/console/configMgr
Repeat steps 9 and 10. Verify the results are the same as before due to default handling if configuration is not present.
docker-compose down && docker-compose rm && docker volume prune -f && ./cleanup.sh
Repeat steps 3 through 8
Modify the configuration for LengthOfStayMapper using http://localhost:8080/system/console/configMgr to disable overwrite
Repeat steps 9 and 10. Verify that all of patients 34-36 are present as imported default LENGTH_OF_STAY of 1 is accepted

Oh, and I think this deserves a Jira issue, please create one and amend the commit and PR title.


[CSL-2974] Fix select question options images height

Image options looking good. The background image for select questions that we discussed still doesn't fill 100% of the height when the view width gets smaller

Are you sure you're on the right branch? @mocca102  🤔

Yes @esezen, this was intentional as agreed with @mocca102.
We wanted to fix the image ratio to unify how images look in case we get different sized images in a single question.
PS: The aspect-ratio: 1/1 property was always there but didn't take correct effect as it was placed on the wrong element.


Added autoassign parameter for agent category assignment

This update adds a new parameter "autoassign" that it's stored in the agent_category pivot table and it's meant to solve our particular scenario, where I have at the moment two categories:

The first one, is a category where I have one agent that has to check all new tickets. If it's not for her, she has to be able to assign that ticket to another agent in the category (nothing new in this case)
The second one is the IT main category:

We want to configure a fake agent with a department shared mail account, that gets all new tickets. We all have that email configured so everyone is going to know about new tickets.
Later, we want to distribute these tickets manually to whom is the best agent on each case, at least for the first support step

I've tried to improve eloquent queries a bit. Also I have'nt applied new template fields to translation files, so If you think it's worth to add this code I'll finish that part up as soon as I can.
Comments and suggestions are all welcome!

I guess I've published my modifications the wrong way :( I'm going to publish my modifications again as a new branch.


Swift Auth V3 support

Hi, the library you use for Swift access (https://github.com/ncw/swift/) supports Identity v3 for auth. It would be great if Aptly could use this, so that we can publish to providers that only support V3 for authentication.

Hopefully fixed in #441

If it doesn't work with aptly 1.0.0, please feel free to re-open


Don' failure a connection when user doesn't have permissions on a device type.

My setup doesn't have garage doors. It seems that permission is also omitted from my vendor.
When doing async_update, it wants to find all entries, including garage doors. There's no garage doors anyway, but I get a 423
back because I have no permission to list them.
I believe when checking for new devices, it's sensible to just omit them if you get a 423 back. There might also be reasons you i.e. don't want your home assistant to see certain devices, and it'd make sense to limit the rights of the alarm.com user that you give to home assistant.
I'll likely take a stab at this myself, if noone else gets around to do it before - but my hacking schedule isn't always predictable :)

Seems it's actually already solved.


Create migration for latest kops improvement

As discussed in https://github.com/reactiveops/pentagon/pull/151, create a migration to go from one kops ig to one-per-az

I'd also like to use this issue as a discussion point for what sorts of changes should also come with a migration:
Pentagon Migrations were intended as a way to make sure that project organization, file contents, and standards could easily be applied to existing projects so that all new changes can be 'immediately' adopted by existing projects and those projects can be pinned at a version.
So my current stance is that any changes in the defaults (content or organization) should include a migration. Unless your project is incorporating the latest standards, it can't be considered the latest version can it? Upgrading the project may involve changes to the cluster or other cloud resources and that is a natural result of upgrading the project version to me.

After looking at the linked PR one thing that sticks out is that some migrations might need a transition state. We'd want to add the new IG's and leave the existing IG in place in some form to be decommissioned intentionally.

That's an interesting point...
Creating an automated process for a Shrodinger's version sounds like it could be very challenging.
With the current way the migrations are done with git, ( commit to a new branch -> update cloud resources -> merge to master) is that enough of a transitional state?
Thoughts on that?

Yeah that would probably work. Or maybe in some cases preserving existing code with comments to cleanup and/or moving to another file that can be cleaned up manually. I just wouldn't want to encourage situations where running apply without understanding would cause major problems.

Agreed we should not be encouraging anyone to run anything by rote. I hope no one would do that anyway but I take your meaning. In the past, strongly worded log messages have the safeguard around that.
The code should be preserved in the other branch so I'm not sure I'd want to do anything thing tricky with commenting in place but migration instructions for something like this would be important,
For the migration in question here, the user would likely understand that applying this change will incur some risk and apply the new instance groups carefully but later might forget to delete the old instance group and/or forget to reconfigure the cluster auto-scaler and that wouldn't cause any obvious issues until later.

Follow up to in person discussion:
Migrations should be made when files change
There may still exist time when migrations might not be feasible so discuss when in doubt.

resolved by #158


tftest.hcl command conversion

I have file with test sth.tftest.hcl:
run "test_nothing" {
    command = plan
}

I attempt to convert it to json:
and i get:
{
  "run": {
    "test_nothing": [
      {
        "command": "${plan}"
      }
    ]
  },
}

when I try to run terraform test on first version it's fine but when i try to run it on json version it errors with:
│ Error: Invalid "command" keyword
│
│   on tests-out/sth.tftest.json line 5, in run.test_nothing[0]:
│    5:         "command": "${plan}"
│
│ The "command" argument requires one of the following keywords without
│ quotes: apply or plan.

Is this hcl2json conversion issue or terraform issue?

It's a "accurately converting hcl to json in all cases requires knowledge of the schema" problem.
This is basically the same problem as https://github.com/tmccombs/hcl2json/issues/12


filter stream cleanup

a few patches that I thought could be useful after my first review

Going to wait for Travis to pass, then will merge away.


Add enrollment mode to django-admin interface. Add attributes of search, list display and filter fields

Done!!!

Enrollment model is visible in the admin interface and allows decent detail

search fields added
filter fields added
foreign key search added
sensitive fields are read-only


Failed to install sysinternals, GoogleChrome, sysinternals.vm, pestudio,vm

Description

C:\Windows\system32>Call PowerShell -NoProfile -ExecutionPolicy bypass -command "Import-Module 'C:\ProgramData\boxstarter\Boxstarter.Bootstrapper\boxstarter.bootstrapper.psd1';Invoke-Boxstarter -RebootOk -NoPassword:$True"
Boxstarter Version <IP_ADDRESS>
(c) 2018 Chocolatey Software, Inc, 2012 - 2018 Matt Wrock. https://boxstarter.org

Boxstarter: Disabling Automatic Updates from Windows Update
 *** LOADING BOXSTARTER ***
Boxstarter: *** HELLO WORLD ***
++ Boxstarter starting Calling Chocolatey to install flarevm.installer.vm. This may take several minutes to complete...
Chocolatey v1.3.1
Installing the following packages:
flarevm.installer.vm
By installing, you accept licenses for the packages.

flarevm.installer.vm v<IP_ADDRESS>21201 (forced)
flarevm.installer.vm package files install completed. Performing other installation steps.
 *** LOADING BOXSTARTER ***
Boxstarter: *** HELLO WORLD ***
[+] Packages to install:
        [+] pestudio.vm
        [+] sysinternals.vm
[+] Installing: pestudio.vm
+ Boxstarter starting Calling Chocolatey to install pestudio.vm. This may take several minutes to complete...
Chocolatey v1.3.1
Installing the following packages:
pestudio.vm
By installing, you accept licenses for the packages.
Progress: Downloading pestudio.vm 9.47... 100%

pestudio.vm v9.47
pestudio.vm package files install completed. Performing other installation steps.
 *** LOADING BOXSTARTER ***
Boxstarter: *** HELLO WORLD ***
Downloading pestudio.vm
  from 'https://www.winitor.com/tools/pestudio/current/pestudio.zip'
Progress: 100% - Completed download of C:\Users\mathias\AppData\Local\ChocoCache\pestudio.vm\9.47\pestudio.zip (1.08 MB).
Download of pestudio.zip (1.08 MB) completed.
Error - hashes do not match. Actual value was '93C9CCB715DE39370070656D3B1959F01DD4DC16ACBB0AD9A6A3CFF32C1BCB00'.
2023/03/31 17:16:06 [pestudio.vm] vm.common.psm1 [+] ERROR : [ERR] Checksum for 'C:\Users\mathias\AppData\Local\ChocoCache\pestudio.vm\9.47\pestudio.zip' did not meet 'E1CA8E305C080EF6880173BD7F17F7E4773C420F8545DB9A72F6958F1AEDDC0A' for checksum type 'sha256'. Consider passing the actual checksums through with --checksum --checksum64 once you validate the checksums are appropriate. A less secure option is to pass --ignore-checksums if necessary.
At C:\ProgramData\chocolatey\helpers\functions\Get-CheckSumValid.ps1:210 char:5
+     throw "Checksum for '$file' did not meet '$checksum' for checksum ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
ERROR: Checksum for 'C:\Users\mathias\AppData\Local\ChocoCache\pestudio.vm\9.47\pestudio.zip' did not meet 'E1CA8E305C080EF6880173BD7F17F7E4773C420F8545DB9A72F6958F1AEDDC0A' for checksum type 'sha256'. Consider passing the actual checksums through with --checksum --checksum64 once you validate the checksums are appropriate. A less secure option is to pass --ignore-checksums if necessary.
The install of pestudio.vm was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\pestudio.vm\tools\chocolateyinstall.ps1'.
 See log for details.

Chocolatey installed 0/1 packages. 1 packages failed.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Failures
 - pestudio.vm (exited -1) - Error while running 'C:\ProgramData\chocolatey\lib\pestudio.vm\tools\chocolateyinstall.ps1'.
 See log for details.
+ Boxstarter finished Calling Chocolatey to install pestudio.vm. This may take several minutes to complete... 00:00:12.2375074
[+] Installing: sysinternals.vm
+ Boxstarter starting Calling Chocolatey to install sysinternals.vm. This may take several minutes to complete...
Chocolatey v1.3.1
Installing the following packages:
sysinternals.vm
By installing, you accept licenses for the packages.
Progress: Downloading sysinternals.vm 2023.3.9... 100%

sysinternals v2023.3.9 [Approved]
sysinternals package files install completed. Performing other installation steps.
Sysinternals Suite is going to be installed in 'C:\ProgramData\chocolatey\lib\sysinternals\tools'
Downloading sysinternals
  from 'https://download.sysinternals.com/files/SysinternalsSuite.zip'
Progress: 100% - Completed download of C:\Users\mathias\AppData\Local\ChocoCache\sysinternals\2023.3.9\SysinternalsSuite.zip (45.6 MB).
Download of SysinternalsSuite.zip (45.6 MB) completed.
Error - hashes do not match. Actual value was '3852A080B773E198B18CC25F6E3935EB935DC17D471AAFCC9B09EFA276D9992B'.
ERROR: Checksum for 'C:\Users\mathias\AppData\Local\ChocoCache\sysinternals\2023.3.9\SysinternalsSuite.zip' did not meet 'f8e060da4267f7c72d7cbe1b89ae5765b4b3247d999de6d185c6821ab9ecdee4' for checksum type 'sha256'. Consider passing the actual checksums through with --checksum --checksum64 once you validate the checksums are appropriate. A less secure option is to pass --ignore-checksums if necessary.
The install of sysinternals was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\sysinternals\tools\chocolateyInstall.ps1'.
 See log for details.

sysinternals.vm v2023.3.9
sysinternals.vm package files install completed. Performing other installation steps.
2023/03/31 17:16:22 [sysinternals.vm] vm.common.psm1 [+] ERROR : [ERR] Cannot find path 'C:\ProgramData\chocolatey\lib\sysinternals\tools' because it does not exist.
At C:\ProgramData\chocolatey\lib\sysinternals.vm\tools\chocolateyinstall.ps1:16 char:19
+ ... coToolDir = Join-Path ${Env:ChocolateyInstall} 'lib\sysinternals\tool ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
ERROR: Cannot find path 'C:\ProgramData\chocolatey\lib\sysinternals\tools' because it does not exist.
The install of sysinternals.vm was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\sysinternals.vm\tools\chocolateyinstall.ps1'.
 See log for details.

Chocolatey installed 0/2 packages. 2 packages failed.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Failures
 - sysinternals.vm (exited -1) - Error while running 'C:\ProgramData\chocolatey\lib\sysinternals.vm\tools\chocolateyinstall.ps1'.
 See log for details.
 - sysinternals (exited -1) - Error while running 'C:\ProgramData\chocolatey\lib\sysinternals\tools\chocolateyInstall.ps1'.
 See log for details.
+ Boxstarter finished Calling Chocolatey to install sysinternals.vm. This may take several minutes to complete... 00:00:15.3260456
[+] Installation complete
2023/03/31 17:16:26 [flarevm.installer.vm] chocolateyinstall.ps1 [+] INFO : Packages installed:
010editor.vm|12.0.1
7zip-15-05.vm|15.05
apimonitor|<IP_ADDRESS>10213
apimonitor.vm|<IP_ADDRESS>20224

2023/03/31 17:02:27 [flarevm.installer.vm] chocolateyinstall.ps1 [+] ERROR : Failed to install: sysinternals
2023/03/31 17:02:27 [flarevm.installer.vm] chocolateyinstall.ps1 [+] ERROR : Failed to install: GoogleChrome
2023/03/31 17:02:27 [flarevm.installer.vm] chocolateyinstall.ps1 [+] ERROR : Failed to install: sysinternals.vm
2023/03/31 17:02:27 [flarevm.installer.vm] chocolateyinstall.ps1 [+] ERROR : Failed to install: pestudio.vm
2023/03/31 17:02:27 [flarevm.installer.vm] chocolateyinstall.ps1 [+] ERROR : For each failed package, you may attempt a manual install via: choco install -y <package_name>
2023/03/31 17:02:27 [flarevm.installer.vm] chocolateyinstall.ps1 [+] ERROR : Failed package list saved to: C:\Users\mathias\Desktop\failed_packages.txt
2023/03/31 17:02:27 [flarevm.installer.vm] chocolateyinstall.ps1 [+] ERROR : Please check the following logs for additional errors:
2023/03/31 17:02:27 [flarevm.installer.vm] chocolateyinstall.ps1 [+] ERROR : 	C:\ProgramData\_VM\log.txt (this file)
2023/03/31 17:02:27 [flarevm.installer.vm] chocolateyinstall.ps1 [+] ERROR : 	%PROGRAMDATA%\chocolatey\logs\chocolatey.log
2023/03/31 17:02:27 [flarevm.installer.vm] chocolateyinstall.ps1 [+] ERROR : 	%LOCALAPPDATA%\Boxstarter\boxstarter.log

Steps to Reproduce

./install.ps1
wait
See error.

Expected behavior:
To successfully install these applications.
Actual behavior:
Shows an error that they are not installed.
Environment
2023/03/31 17:09:26 vm.common.psm1 [+] INFO : Host Information

VM OS version and Service Pack
-----

Version                 : 10.0.19045
BuildNumber             : 19045
OSArchitecture          : 64-bit
ServicePackMajorVersion : 0
Caption                 : Microsoft Windows 10 Pro

VM OS RAM (MB)
-----
8192

VM OS HDD Space / Usage
-----

DeviceID DriveType ProviderName VolumeName              Size         FreeSpace
-------- --------- ------------ ----------              ----         ---------
C:       3<PHONE_NUMBER>32<PHONE_NUMBER>8
D:       5                      CCCOMA_X64FRE_EN-US_DV9<PHONE_NUMBER>   0

VM AV Details
-----
AntiVirusProduct classname does not exist...

Per the logs the packages need to be updated so the versions/hashes match again. We track these here: https://github.com/mandiant/VM-Packages. Would you mind opening separate issues for Chrome and pestudio there? Sysinternals should be handled by https://github.com/mandiant/VM-Packages/pull/283.

This packages have been fixed in https://github.com/mandiant/VM-Packages


Key off previewNumber instead of additional bool for UPM pack

Overview
Slightly simplifies the UPM pack pipeline and script, renaming buildNumber to the more descriptive previewNumber and removing an extra bool that represented whether to use the previewNumber. It now keys off whether previewNumber is provided or not.

/azp run


Wrap child process code in fork() in try/catch

Otherwise an exception in the child process might lead to catch-blocks from the parent process to be executed (that were copied during forking).
Paired with @alexispeter
@tomaszdurka fyi

Interesing - lgtm.


Coverage breaks sourcemaps

Issue :
If user enables test coverage with Jest then the sourcemaps are not correct.
Expected behavior :
Lines should be shown correctly, even if developer enables coverage report with Jest.
Minimal repo :
Github project: https://github.com/maxpolski/ts-react-jest/ Please kindly note that this is NOT my repository
Run jest with coverage option.
You will notice that the sourcemaps are not working correctly. To make sure, you can add console.log() in any React component and compare the line where console.log occured with the one displayed in the jest output.
Interestingly, sourcemaps work absolutely fine in test files, but not in source files.

I confirmed that this was the cause of my inability to debug tests via vscode-jest. Turning off code coverage allowed the sourcemaps to be built properly.

Any thoughts on the cause/fix? I started seeing this today. My index.ts file has around 100 lines but jest is reporting console.log calls on lines > 1000.

I can confirm that this is an issue for me as well.
It was driving me crazy because it also prevented debuggers from stepping through the code. Glad I found that I was not the only one experiencing this.

Can confirm, VSCode reports line numbers >6000 in file with 300 lines, only when using ts-jest with coverage turned on.

I am getting this problem as well when I turn on coverage. Looking through similar issues I found https://github.com/kulshekhar/ts-jest/issues/484 which has some commits to fix this same issue though, and I am using the latest build, so I am kind of confused...

Hi all,
I am having the same issue. I took some time and created a small repository that reproduces the issue: https://github.com/wesleygrimes/ts-jest-coverage-incorrect-line-numbers
I am seeing this issue as well with jest: 24.8.0 and ts-jest: 24.0.2.
jest.config.js:
module.exports = {
  testMatch: ['**/+(*.)+(spec|test).+(ts|js)?(x)'],
  transform: {
    '^.+\\.(ts|js|html)$': 'ts-jest'
  },
  roots: ['<rootDir>/src'],
  moduleFileExtensions: ['ts', 'js', 'html'],
  collectCoverage: true,
  coverageReporters: ['html']
};

We need collectCoverage enabled. Is there a fix planned for this? Would be happy to help with a PR if someone can point me in the right direction.
Thanks!

Was fixed in 23.1.2, see . But reproducible in:
jest 24.9.0 + ts-jest 24.0.2
jest 23.6.0 + ts-jest 23.10.5 (latest 23.* versions as of today)
Workaround for debugging:

remove collectCoverage from config
add --coverage to jest cli in test script of package.json:

"test": "jest --config ./jestConfig.json --coverage"

I can confirm that pinning to version 23.1.2 does resolve the issue with vscode v1.38.0 .  Thank you @antonovicha for unblocking us over here!

any news on when this may be fixed?  Being locked on v 23.2.1 It's making moving to jest 24 a painful experience.

In order to get debugging to work, I removed "collectCoverage" from my jest configuration and instead changed my npm script to jest --coverage. Then, in VSCode's launch configuration I did not add that flag. Who needs coverage during debugging anyways? It also makes the test execution slower.
Of course, this is just a workaround until this is fixed.

I found a fix that works well for me.  If you add --coverage false to the command line when the jest debugger is run, it works just fine.  The solution is to add this block to your launch configuration in .vscode/launch.json:
    {
      "type": "node",
      "request": "launch",
      "name": "vscode-jest-tests",
      "cwd": "${workspaceFolder}",
      "args": ["--inspect-brk", "${workspaceRoot}/node_modules/.bin/jest", "--runInBand", "--config", "--coverage", "false"],
      "console": "integratedTerminal",
      "internalConsoleOptions": "neverOpen"
    },

This is not ts-jest issue. Please check PR https://github.com/facebook/jest/issues/5739