0.11.1 Docs misinformation

Rasa Core version:
0.11.1
Python version:
3.6
Operating system (windows, osx, ...):
Ubuntu 18.04
Issue:
The HTTP API "/parse" endpoint is missing on this release (according to server.py), yet the documentation points to it in the "Start a Conversation" section of the HTTP API docs
If reproducing is needed, simply run rasa_core.run --enable_api and use the curl example from the docs above. It will return a 404.
Content of domain file (if used & relevant):

Thanks for raising this issue, @tmbo will get back to you about it soon.

Yes that needs correcting. I created a PR to fix this. https://github.com/RasaHQ/rasa_core/pull/928


History viewed cars

Asana task: https://app.asana.com/0/1202525679663009/1202720801120419

@oleksandr-danylchenko Please, check my PR


[FR] Sales order templates

Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find a similar feature request

Problem statement
Where I am using Inventree just had an event where we had to create and issue a large number of sales orders with the same line items with only a few variations
Suggested solution
A template for sales orders where a group of line items are added automatically to the sales order
Describe alternatives you've considered
At the moment we are just manaully adding every line item by hand but this is very slow
Examples of other systems
No response
Do you want to develop this?

[ ] I want to develop this.

@ATLAS2246 we currently have the option to "duplicate" a sales order - but this only duplicates the order details, not the line items.
Perhaps a clean solution here would be to allow the user to also duplicate the line items from an existing order?

that would probably work extremely well, id just set up a template order that doesnt get completed and use that


Clarifying questions/X-refs; MONDO:0010173 and Mayer-Rokitansky-Kuster-Hauser syndrome(s)

Mondo term (ID and Label):
MONDO:0010173 Mayer-Rokitansky-Kuster-Hauser syndrome type 1
Xref that should be fixed (ID and label):

SNOMED CT 8793008 looks like it should actually be on the parent concept of MRKH syndrome/ Rokitansky sequence

OMIM's pref name on 277000 is "MAYER-ROKITANSKY-KUSTER-HAUSER SYNDROME" (abbreviated as MRKH) and it describes multiple, related phenotypes in the Description, so I don't know that it is correct to map this OMIM record to "type 1" child term, perhaps it should also move to the parent concept?

I see that Orphanet ORDO:247775 points to 277000 as 'exact' and Orpha's term is "...type 1" but I think this equivalency should be reviewed?

UMLS X-refs that I think may be the best matches (but UMLS also has multiple CUIs for the parent concept, I think, I will follow up with them separately)
MRKH syndrome: C0431648 / C1698581
MRKH1 : C5566555
MRKH2: C4305568
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here
Other comments:
Summary of Requests, pending review/discussion from @ahamosh and @Orphanet @ana-rath-orphanet  et al.

[ ] Move SNOMEDCT 8793008 to MONDO:0017771
[ ] Add "Rokitansky sequence" (source- SNOMED CT 8793008)) and "Rokitansky Kuster Hauser syndrome" (Source: MeSH C537371) syns to MONDO:0017771
[x] ? Move OMIM 277000 to MONDO:0017771 ? Pending review by Orphanet ?

thanks @kanems!

Question to OMIM:
Does OMIM:277000 represent a specific disease  ('Mayer-Rokitansky-Kuster-Hauser syndrome type 1') or a "grouping term" ?

On the Expert call (Jan 25, 2024):
OMIM:277000 represents 'Mayer-Rokitansky-Kuster-Hauser syndrome type 1'
@ahamosh, please update the OMIM record to show this disease name (ie 'Mayer-Rokitansky-Kuster-Hauser syndrome type 1'). Thank you!

To do:  ? Move OMIM 277000 to MONDO:0017771 ? Pending review by Orphanet ?
See previous comment, the OMIM ID is correct.

I did:

moved the SCTID to the more general term - MONDO:0017771
Added UMLS x-refs to MONDO:0017771 and children (notes that these might also be part of the x-refs import from MedGen)
Added synonyms as requested (note that the suggested MESH id is equivalent to a parent term "mullerian aplasia", and not to MONDO:0017771. Therefore, to avoid confusion, the orcid instead of the MESH-id as a x-ref for the synonym)

Note: Based on here and Orphanet:247775, Rokitansky sequence refers to type1, and not to parent term as suggested by SCTID:8793008.


Running the action optionally, including pre and post scripts

We tried to make running Harden Runner optional:
    steps:
    - name: Harden Runner
      if: ${{ vars.USE_HARDEN_RUNNER }}
      uses: step-security/harden-runner@0080882f6c36860b6ba35c610c98ce87d4e2f26f
      with:
        egress-policy: audit

However, this seems to impact only the main script and not the pre & post scripts https://github.com/step-security/harden-runner/blob/0080882f6c36860b6ba35c610c98ce87d4e2f26f/action.yml#L37-L39
Is there a way to en/disable all of them? The runs documentation mentions pre-if and post-if and I’m not quite sure if this is something we ought to implement, or the action?
Do you guys have a suggestion?

Thank you for raising this issue!
You’re correct that pre-if and post-if could potentially enable conditional execution of the pre and post scripts.
We’ll experiment with implementing pre-if and post-if to see if it can address your use case effectively. Once we’ve explored this approach and validated whether it works as expected, we’ll update you here with our findings or next steps.
Out of curiosity, could you share a bit more about your use case for conditional execution? Understanding why you’re looking to enable or disable Harden-Runner in certain scenarios will help us provide an alternate solution in case pre-if/ post-if does not work.

[…] Once we’ve explored this approach and validated whether it works as expected, we’ll update you here with our findings or next steps.

Thank you!

Out of curiosity, could you share a bit more about your use case for conditional execution? Understanding why you’re looking to enable or disable Harden-Runner in certain scenarios will help us provide an alternate solution in case pre-if/ post-if does not work.

The problem for us is runtime. Having multiple repos and quite a number of action workflows, each starting out with hardening the runner, the setup runtime of ~3m has become a noticeable portion of the billing time.

So instead of removing the step from our workflows we wanted to make it optional…

From the attached screenshot, i can see the workflow is container-based, currently harden-runner doesn't supports container-based workflows. Ideally pre-step should have just skipped, as we have logic to detect this scenario. checkout this sample run
Can you please attach screenshot of logs from pre-step ? I am just curious what's happening in there.

Can you please attach screenshot of logs from pre-step ? I am just curious what's happening in there.

As in debug logs when I rerun the workflow?

As in debug logs when I rerun the workflow?

you can attach these as well for more context.
I was actually referring to just clicking on the Pre Harden Runner step to expand it. Something like below

I should mention that we’re running v1.5.0 so we have:
Run step-security/harden-runner@2e205a28d0e1da00c5f53b161f4067b052c61f34
Step Security Job Correlation ID: 4088e16b-5dc9-46c5-984a-fb2b8360bb5d
Unable to fetch cacheURL
/usr/bin/tar xz --warning=no-unknown-keyword --overwrite -C /home/runner/work/_temp/60e81887-14e5-4fd7-909e-db82915b827e -f /home/runner/work/_temp/50ca6b7f-be68-4ae9-9ab3-f01514233df1
View security insights and recommended policy at:
https://app.stepsecurity.io/github/org/repo/actions/runs/12471801516
Initialized

These appears to be interesting logs. Ideally harden-runner shouldn't have logged them in-case of container-based workflows because of a conditional-check.

I should mention that we’re running v1.5.0

I tried running v1.5.0 on my test workflow, it appears to be working fine checkout this run.
If possible can you run the workflow with v2.10.2 ?
Anyways i will keep updated. Just curious are you guys using self-hosted runners ?
Thanks

I tried running v1.5.0 on my test workflow, it appears to be working fine checkout this run.

Try adding an
if: ${{ false }}

or some such here. It should still run the pre- and post-steps, which is what we’d like to avoid.

[…] Once we’ve explored this approach and validated whether it works as expected, we’ll update you here with our findings or next steps.

Thank you!

Out of curiosity, could you share a bit more about your use case for conditional execution? Understanding why you’re looking to enable or disable Harden-Runner in certain scenarios will help us provide an alternate solution in case pre-if/ post-if does not work.

The problem for us is runtime. Having multiple repos and quite a number of action workflows, each starting out with hardening the runner, the setup runtime of ~3m has become a noticeable portion of the billing time.

@jenstroeger harden-runner should not be taking 3m. This is likely because of use of a very old version v1.5 which uses node16. Once you upgrade to the latest version, it should not take more than 5-7 seconds. Please let us know if this is not the case. Given that updating to a more recent version should fix the root cause, we would prefer to not spend time researching/ implementing optional execution.

So instead of removing the step from our workflows we wanted to make it optional…


Converted Service editor to angular.

@dclarizio @AparnaKarve @himdel please review

@himdel addressed your comments.

@h-kataria Thanks, found an extra unused method, but apart from that, looks great! :)

@himdel
$scope.$watch("serviceModel.name", function() {
        $scope.form = $scope.angularForm;
        $scope.model = "serviceModel";
      });

added this back to fix Abandon Changes prompt.

@h-kataria you should just need that $scope.form = $scope.angularForm; and you should be able to do it in the same place you moved that model to. But if this works better.. :)

@himdel ready for review again

@dclarizio Tested the form in the UI. Looks good :+1:


[Android] Ionic Native stops working after upgrading to WKWebView

When I upgrade my app to WKWebView, the native plugins (camera, geolocation) stop working on Android. They work fine on iOS though. I don't know what is causing this. I tried creating a new app with WKWebView preinstalled and native plugins work fine there.
This is my first issue so I don't know what info I have to submit.
Here's my ionic info:

Ionic:

   ionic (Ionic CLI)  : 4.1.1 (/usr/local/lib/node_modules/ionic)
   Ionic Framework    : ionic-angular 3.5.3
   @ionic/app-scripts : 2.0.2

Cordova:

   cordova (Cordova CLI) : 8.0.0
   Cordova Platforms     : android 7.0.0
   Cordova Plugins       : no whitelisted plugins (5 plugins total)

System:

   Android SDK Tools : 26.1.1 (/home/samsep10l/Android/Sdk)
   NodeJS            : v8.10.0 (/usr/bin/node)
   npm               : 6.3.0
   OS                : Linux 4.15

I also have issue with native plugins using the camera on Android (using Ionic DevApp) from a new project created today (by following the tutorial). Same project is fine on iOS using Ionic DevApp. Here is my ionic info:
Ionic:

   ionic (Ionic CLI)             : 4.6.0 (C:\Users\Steph\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework               : @ionic/angular 4.0.0-rc.0
   @angular-devkit/build-angular : 0.11.4
   @angular-devkit/schematics    : 7.1.4
   @angular/cli                  : 7.1.4
   @ionic/angular-toolkit        : 1.2.2

Cordova:

   cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>   Cordova Platforms     : android 7.1.4, ios 4.5.5
   Cordova Plugins       : not available

System:

   NodeJS : v10.13.0 (C:\Program Files\nodejs\node.exe)
   npm    : 6.4.1
   OS     : Windows 10

When I tried platform.platforms() on any of my Android device, I don't have the 'cordova' string, but it does show on my iOS devices. Can this be an hint to resolve the issue?

Hi Vivek,
Thank you for your hint. I managed to test it directly via an apk and 'cordova' is returned properly so the camera is working fine now. I guess I will debug in Ionic DevApp only with iOS assuming everything is fine with Android until I build and test via apk.
Thanks again for your help!

I can't reproduce, Ionic Native is working fine on my apps if I build them or use live reload (ionic cordova run android -l)
DevApp is probably out of date, so some things might now work there

what version of DevApp on Android do you have?

No, I mean the DevApp is out of date, not that the version you use is out of date.
I didn’t try on DevApp, but if it doesn’t work there then it’s a DevApp bug, not a webview plugin bug


Comment author/timestamp label baselines not aligned

Testing #143545

The roblourens does not line up with the 33 minutes ago, it looks a little off

It's clearly not aligned properly in your screen shot, but I can't reproduce it. Can you think of anything that might be different about your view? I tried moving the Comments view to the sidebar, but still can't repro there.

They have different font sizes for me:

.9em resolves to 11.7px

The different font size is intended, but they don't end up with different baselines when I see it. I'll keep looking.

Can repro after changing zoom.

This looks a bit similar to https://github.com/microsoft/vscode/issues/113223, but adding margin-top: auto to the various elements doesn't help anything. @misolori do you have any ideas? CSS isn't my forte, but I don't see a problem in the git SCM view were we are using the same font-size: 0.9em for the description of each changed file..

This is the same issue across all tree views and the root problem is the difference in font-size:

I'd love to solve this during debt week and move away from using percentage & ems to be consistent
cc @TylerLeonhardt

@daviddossett FYI in case you were interested in looking into this!

Also @hbons FYI.


update restore-artifacts script

Hi,
When testing the code in the example 'assemble' script, the syntax didn't seem to work:
mv /tmp/artifacts/. ./

resulted in the error
mv: cannot move '/tmp/artifacts/.' to './.': Device or resource busy

Actually, I would have expected it to be:
mv /tmp/artifacts/* ./

Any thoughts on this?  Could you reproduce the error?
A more thorough modification would be the following. This catches hidden files also.
shopt -s dotglob
mv /tmp/artifacts/* ./
shopt -u dotglob

@sdarwin can you give us more thorough recreate steps?

@bparees , sure.  The issue is the trailing dot "/." in combination with the mv command.
Again, the original was:
mv /tmp/artifacts/. ./

To recreate, try this:
[ec2-user@ip-172-31-6-107 ~]$ mkdir test1
[ec2-user@ip-172-31-6-107 ~]$ mkdir test2
[ec2-user@ip-172-31-6-107 ~]$ touch test1/test1.txt
[ec2-user@ip-172-31-6-107 ~]$ touch test1/test2.txt
[ec2-user@ip-172-31-6-107 ~]$ touch test1/test3.txt
[ec2-user@ip-172-31-6-107 ~]$ mv test1/. test2/
mv: cannot move 'test1/.' to 'test2/.': Device or resource busy
[ec2-user@ip-172-31-6-107 ~]$ mv test1/. ./
mv: cannot move 'test1/.' to './.': Device or resource busy
[ec2-user@ip-172-31-6-107 ~]$

@sdarwin yeah it looks like we previously had this as * and (w/o testing I guess) switch it to .:
https://github.com/openshift/source-to-image/commit/dd9ba6571183ca78800196e401e5a3cdbda1a7e6
I think the dotglob option is probably best to ensure we continue to catch hidden files.  Can you update your PR to that?

I think the dotglob option is probably best to ensure we continue to catch hidden files. Can you update your PR to that?

updated.

/lgtm
thanks @sdarwin

/ok-to-test


Moved asset installation to before database migration and fixed task name

Q
A

Bug fix?
Yes or No

New feature?
Yes or No

BC breaks?
Yes or No

Deprecations?
Yes or No

Fixed tickets
N/A or xx

Do not forget to add notes about your changes to CHANGELOG.md. A command line tool has been included to make this easy, see CONTRIBUTING.md for details.

Superseded by #2143


Update strath.txt

please merge my request

@abmnukmr Your request seems to be incorrect, because the University of Strathclyde is already in the repository, it's not necessary to add it again.
If you wish to add an email domain of National Institute of Technology, Hamirpur, you may need to add a domain nith.ac.in (or another one, if students do not have their emails in @nith.ac.in domain). In order to do that you have to submit a new request, with a file containing a single line:
National Institute of Technology, Hamirpur
and the file has to have a name of lib/domains/in/ac/nith.txt (if the emails are in @nith.ac.in domain).
Thus, I am closing this request without merging. Please submit a correct one. Thank you!


Bug in intToBigInt?

const intToBigInt = (key, val, {source}) => typeof val === "number" && val % 1 === 0 ? BigInt(source) : val;

AFAICT, this code is going to fail for (e.g.) source === '123e2':
> BigInt('123e2')
SyntaxError: Cannot convert 123e2 to a BigInt

intToBigInt is intended to illustrate a particular example rather than completeness, but I understand the risk of developers naïvely copying code off the Internet. Does #38 satisfy your concerns?

Looks good, yes!


Uppercase in username doesn't accept.

Example AlexGluck not working, but in lowercase alexgluck accepted properly

@AlexGluck the Telegram documentation says that "Usernames are case-insensitive".
So I don't think that it's right to equalize uppercase and lowercase nicknames in third-party software.
If someone wants to change something in their username - he should update alerts as well.


Responsive design

Website is responsive to different devices you can easily navigate, nav  bar and home section.
my issue no #158
Before making responsive: home section items are not visible completely .

after making responsive:-
https://github.com/Suchitra-Sahoo/AgriLearnNetwork/assets/149225785/8fda4233-c6e3-4c67-b2a7-596010ef3854

Now user can use website in different devices easily.

Resolve the conflicts of this branch.

Recently various PR has been merged so the home page designing is changed. You may look it by clicking the website's link. Make sure those changes are visible. Currently the video which you have sent, I can see it is showing the website with previous styling.

here is the responsive design with latest updates
https://github.com/Suchitra-Sahoo/AgriLearnNetwork/assets/149225785/e2ded8e0-3bde-4f48-9dbd-703232212a23


Fixup after is_basis refactor

Removed some lemmata that got merged into mathlib, and fixed up the issues after is_basis got refactored into a bundled basis.

Decided that fighting against Lean was more engaging than finding all the subfields of ℚ(ζₙ) for n way too large :) please let me know if the style is no good!

Thank you!


Bug: Button Press Duration in Frontend

Button Press Duration should be hardcoded in the backend

Does it reference to that the duration should be handled in backend?

Yes the duration should be handled in the backend.
Short : 200ms
Medium: 1500ms
Long: 3000ms

This was debugged and is ready for testing in "debugging" branch


Firewall/NAT

Which version are you using?
rtsp-simple-server v0.12.2
Question
[1/1/0] [client 195.x.x.x:50922] ERR: no packets received recently (maybe there's a firewall/NAT in between)
My system is Ubuntu 18.04.5
When I add source from localhost ( gst-launch-1.0 videotestsrc ! clockoverlay halignment=right valignment=top text="SERVER h264 " shaded-background=true font-desc="Sans, 36" !  videoconvert ! videoscale ! video/x-raw,format=I420 ! x264enc tune=zerolatency bitrate=2000 speed-preset=ultrafast  ! h264parse ! queue !  rtspclientsink location=rtsp://localhost:8554/mystream ) - rtsp server works well.

Thanks!

Hello, the problem with UDP mode is that it works only if the server and the client are both on the same LAN.
You can either

use a VPN
toggle TCP mode with rtspclientsink location=location protocols=tcp

Hello, the problem with UDP mode is that it works only if the server and the client are both on the same LAN.
You can either

use a VPN
toggle TCP mode with rtspclientsink location=location protocols=tcp

Thank you. rtspclientsink location = location protocols = tcp, it worked.

Thank you. rtspclientsink location = location protocols = tcp, it worked.


The server<EMAIL_ADDRESS>do not start

Hello everyone
I installed openVPN in my raspberry pi but when I try to boot the server gives the following error:
●<EMAIL_ADDRESS>- OpenVPN connection to server
Loaded: loaded<EMAIL_ADDRESS>disabled; vendor preset: enabled)
Active: failed (Result: exit-code) since Fri 2017-11-24 11:59:26 CET; 6min ago
Docs: man:openvpn(8)
https://community.openvpn.net/openvpn/wiki/Openvpn23ManPage
https://community.openvpn.net/openvpn/wiki/HOWTO
Process: 32286 ExecStart=/usr/sbin/openvpn --daemon ovpn-server --status /run/openvpn/server.status 10 --cd /etc/openvpn --config /etc/openvpn/server.conf --writepid /run/openvpn/server.pid (code=exited, status=226/NAMESPACE)
Nov 24 11:59:26 Pi systemd[32286]<EMAIL_ADDRESS>Failed at step NAMESPACE spawning /usr/sbin/openvpn: Bad file descriptor
Nov 24 11:59:26 Pi systemd[1]<EMAIL_ADDRESS>Control process exited, code=exited status=226
Nov 24 11:59:26 Pi systemd[1]: Failed to start OpenVPN connection to server.
Nov 24 11:59:26 Pi systemd[1]<EMAIL_ADDRESS>Unit entered failed state.
Nov 24 11:59:26 Pi systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.
My piVPN debug is:
::: Generating Debug Output
:::					:::
::		PiVPN Debug		 ::
:::					:::
::	Latest Commit			 ::
:::					:::
commit 7ed7e4652ebe322e51e85a0582907e83eea0d103
Author: Carlos Colaço<EMAIL_ADDRESS>Date:   Fri Oct 20 17:00:23 2017 +0200
Added info regarding Raspbian Stretch

added info about support for raspbian Strech, and changed the preference from Jessie to Stretch.

Edited and merged directly since it is a minor edit.

::: Copy saved to /tmp/debug.txt
:::
The version of kernel is:
3.4.112-sun7i
The server.conf is:
dev tun
proto udp
port 1194
ca /etc/openvpn/easy-rsa/pki/ca.crt
cert /etc/openvpn/easy-rsa/pki/issued/server_ntQsz4gFFmzJKgoq.crt
key /etc/openvpn/easy-rsa/pki/private/server_ntQsz4gFFmzJKgoq.key
dh /etc/openvpn/easy-rsa/pki/dh4096.pem
topology subnet
server <IP_ADDRESS> <IP_ADDRESS>
server and remote endpoints
ifconfig <IP_ADDRESS> <IP_ADDRESS>
Add route to Client routing table for the OpenVPN Server
push "route <IP_ADDRESS> <IP_ADDRESS>"
Add route to Client routing table for the OPenVPN Subnet
push "route <IP_ADDRESS> <IP_ADDRESS>"
your local subnet
push "route <IP_ADDRESS> <IP_ADDRESS>"
Set your primary domain name server address for clients
push "dhcp-option DNS <IP_ADDRESS>"
push "dhcp-option DNS <IP_ADDRESS>"
Override the Client default gateway by using <IP_ADDRESS>/1 and
<IP_ADDRESS>/1 rather than <IP_ADDRESS>/0. This has the benefit of
overriding but not wiping out the original default gateway.
push "redirect-gateway def1"
client-to-client
keepalive 10 120
remote-cert-tls client
tls-version-min 1.2
tls-auth /etc/openvpn/easy-rsa/pki/ta.key 0
cipher AES-256-CBC
auth SHA256
comp-lzo
user nobody
group nogroup
persist-key
persist-tun
crl-verify /etc/openvpn/crl.pem
status /var/log/openvpn-status.log 20
status-version 3
log /var/log/openvpn.log
verb 1
Generated for use by PiVPN.io
Thanks in advance for yours help

Hello
My OS is Raspbian Stretch 9


D455 How to get the azimuth angle with python code?

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D455

Firmware Version
<IP_ADDRESS>

Operating System & Version
win10

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC

SDK Version
<IP_ADDRESS>2

Language
python

Segment
{Robot }

Issue Description
<Describe your issue / question / feature request / etc..>
D455 How to get the azimuth angle with python code?

Hi @Dontla   Do you require further assistance with this case, please?   Thanks!

Thanks, it's OK

Thanks very much for the update!


Unable to test with DateRangePicker

react-dates version
21.8.0
Describe the bug
I use the "DateRangePicker" component in a project and I need to test the user flow with the Testing Library. The problem is that when I do the first click (userEvent.click()), on the first date, the modal is closed, not waiting for the selection of the second date.
 "@testing-library/react": "^14.0.0",
 "@testing-library/user-event": "^14.4.3",
 "react-dates": "^21.8.0",

Source code (including props configuration)
Steps to reproduce the behavior:
    <DateRangePicker
      noBorder
      minimumNights={0}
      startDate={startDate}
      startDateId="start-date-id"
      endDate={endDate}
      endDateId="end-date-id"
      onDatesChange={handleDatesChange}
      focusedInput={focusedInput}
      onFocusChange={(focusedInput) => setFocusedInput(focusedInput)}
      isOutsideRange={isOutsideRange}
      onClose={handleOnClose}
      withPortal={true}
      customInputIcon={label}
      hideKeyboardShortcutsPanel={true}
    />

We recommend testing with enzyme, but in particular, no testing solution's event simulations (enzyme and RTL included) are actually true matches of browser behavior - they just invoke props - so they're not very useful.
The click you want doesn't go on the picker directly, it goes on one of the smaller elements contained within it.

@ljharb Thanks for the answer.
I use the Testing Library to test cover the entire application. At the moment there are more than 600 test scenarios. Many of them simulating user interaction and working perfectly. It is not an option to switch to Enzyme. The only issue we had was with the DateRangePicker.

Understood, but I'm trying to convey that "simulating user interaction" with either RTL or with enzyme doesn't actually give you value, even if it seems to "work", because it doesn't actually match user interaction.
At any rate, the issue at hand is likely that your .click() isn't targeting the right element. If you can make a codesandbox that can reproduce the problem I'm happy to take a look.

I got the elements for the start and end date through the getByRole method. Note below:
    const firstDate = screen.getByRole('button', {
      name: /choose terça-feira, 1 de agosto de 2023 as your check-in date. it’s available./i,
    });
    const lastDate = screen.getByRole('button', {
      name: /choose terça-feira, 22 de agosto de 2023 as your check-in date. it’s available./i,
    });

Sounds like an issue with RTL then; i'm not sure how getByRole works.

@ljharb
https://github.com/xereda/datepicker-test

Sounds like an issue with RTL then; i'm not sure how getByRole works.

getByRole
The getByRole method from the Testing Library provides a way to select an element by its accessible role. This query is recommended because it encourages developers to write tests that rely on the accessibility information of elements, which ultimately leads to more accessible applications.
https://testing-library.com/docs/queries/byrole/

@ljharb
https://github.com/xereda/datepicker-test
In this repository I managed to reproduce the problem.

src/components/Datepicker/Datepicker.test.js
https://github.com/xereda/datepicker-test/blob/main/src/components/Datepicker/Datepicker.test.js

yarn test:watch
Note that the modal is closed on the first click.

up

I have the same issue for what it's worth.


Is Scorex a good choice for a sidechain?

We are writing an app that should have a high TPS (ex. 100 TPS) and periodically sync the state of a sidechain to a mainnet of an established blockchain (ex. Ethreum). For this reason, we are comparing different lightweight blockchains with PoS consensuses for the role of a sidechain in this app and we came across Scorex.
Are there any examples of using Scorex as a sidechain?
If yes, can you provide the links?
What about integration with popular blockchains like Bitcoin or Ethereum?

Are there any examples of using Scorex as a sidechain?

As far as I know, there are no examples for this. (@kushti, am I right?)  The main example that we provide and maintain is Twinscoin (https://github.com/ScorexFoundation/Scorex/tree/master/examples).

What about integration with popular blockchains like Bitcoin or Ethereum?

Also, no example for this to the best of my knowledge. But nothing prevents you from implementing your own integration. Scorex provides abstractions to help you implement your own blockchain. If you would like to allow your blockchain to communicate with Bitcoin's blockchain or Ethereum's blockchain, it is your responsibility to implement a component inside your Scorex application that would interact with a Bitcoin node or an Ethereum node/app and post data of your blockchain to Bitcoin's blockchain or Ethereum's blockchain.

Thank you!


Can't find ssh config file

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed: https://github.com/icetee/remote-ftp/issues

Description
I'm trying to use ProxyCommand through my ssh config file to access a server through a jumpbox. This seems theoretically possible but it doesn't seem like Remote-FTP is finding my config file. I supplied the path in Remote-FTP > Settings as /Users/Josh.Temple/.ssh/config, but I'm getting the following error:

Error: getaddrinfo ENOTFOUND spark spark:21.

My .ftpconfig file looks like this:
{
    "protocol": "sftp",
    "host": "spark"
}

My ssh config file is as follows:
Host spark
	Hostname [IP Address of Destination Server]
	IdentityFile [Path to Identity File]
	User [Username]
	ProxyCommand ssh -l [Jumpbox User] -i [Path to Private Key] -q [Jumpbox URL] -W %h:%p
	DynamicForward localhost:[Port to Forward]

Versions
Mac OS Sierra 10.12.6
Atom: 1.19.6
Electron: 1.6.9
Chrome: 56.0.2924.87
Node: 7.4.0

Unfortunately, I do not know how ProxyCommand and Dynamic Forward work. I'm trying to look at it. With these settings, it works perfectly well on Windows and MacOS. Related of https://github.com/icetee/remote-ftp/issues/990
My simple .ftpconfig file:
{
    "protocol": "sftp",
    "host": "domain.country",
    "hostName": "Hello Server",
    "remote": "/home/icetee/.meteor/"
}

And my partial ssh config (in Windows)
Host domain.country
    HostName domain.country
    IdentityFile C:\Users\icetee\.ssh\id_rsa
    User icetee
    Port XYZW

I'm facing the same problem as joshtemple above.
Atom 1.22.1x64
Remote FTP 2.1.0
On Ubuntu 16.04.3 LTS

I think port is missing in the config file, 21 port is simple FTP.

@icetee in your example above domain.country appears to be the same value in .ftpconfig as the actual address configured against HostName in the config file. Part of the purpose of the ssh config file is to allow a user to use different local pseudonyms to connect to the same host in different ways (e.g. with different keys). Hence could you try a configuration like this:
.ftpconfig:
{
    "protocol": "sftp",
    "host": "somename",
    "hostName": "Hello Server",
    "remote": "/home/icetee/.meteor/"
}

and config:
Host somename
    HostName domain.country
    IdentityFile C:\Users\icetee\.ssh\id_rsa
    User icetee

When I do this I get the error described above.


General mechanism for custom widgets

Needed by #10 and #11 and #12.

I'm going to try guiding this entirely using templates for each widget.
For any query, take the names of columns, order alphabetically, then concatenate then into a template name, eg:
count-value.html

That template would live here:
django_sql_dashboard/widgets/count-value.html

Uses select_template() (via the Django {% include iterable %} mecho and to try to load that template, but fall back on widgets/default.HTML which displays a regular table (and has HTML comments indicating what template it tried first).
Maybe support wildcards? So you can say "I like latitude and longitude and anything else" in the file name somehow. This is harder though because need to scan all available templates and prioritize them.
Instead could have latitude-longitude-labels.html and let labels be a JSON column so you can bundle in a bunch of labels for each row using a JSON build SQL function.
This would mean that people can easily define their own widgets by defining new templates.
The widget templates can use {% load %} to load special tags eg for markdown display.


Allow GCP buckets to be used for backup, without deploying nexus itself on Google Cloud

got my workflow wrong here - raised a PR to tackle this and forgot/didn't realise that would raise its own issue (#202 ).  So this one is a duplicate.


[8.x] Fix bug discarding input fields with empty validation rules

This PR fixes a breaking change that was introduced with conditional validation rules in v8.55.0.
Before the change in v8.55, if input fields were being validated with an empty list of validation rules, no validation would be performed but those fields would be included in the data returned from $request->validated().
Now, fields with an empty list of validation rules ([] or '') are removed from the list of rules entirely, so matching input fields in the request are not included in $request->validated().
v8.54 and earlier:
validator([
    'name' => 'Jacob', // Input
], [
    'name' => '', // Rules
])->validated();

// ['name' => 'Jacob']

v8.55 and later:
validator([
    'name' => 'Jacob', // Input
], [
    'name' => '', // Rules
])->validated();

// []

Background
The conditional validation rules feature added in v8.55.0 runs filter() on the collection of validation rules so that conditional rules are not applied if the conditional is false.
This unintentionally introduced the side-effect that input data is excluded from the output of validated() for non-conditional rules with empty ([] or '') rules, and for conditional rules where the conditional is false. In the second case, if the only rule on a field is a conditional rule and the condition evaluates to be false, instead of just not running the conditional rules the field will be removed from validated() completely.
I'm not suggesting in this PR that the previous behaviour is or isn't correct (I asked some colleagues about this and most actually agree that the new behaviour is better), but it seems like a breaking change, so I think it would be ideal to avoid making it in a minor release. At Tighten we've already seen multiple applications break as a result of this change.
The code change in this PR is very small and completely backwards-compatible—it just stops filtering out validation rules that are [] or ''. If filtering these fields out is the preferred behaviour, it could easily be reintroduced in v9.
Fixes #38505.

this make me to change all my empty rule to "sometimes" but its ok


json: cannot unmarshal number ERROR

https://github.com/hummingbot/deploy-examples/tree/main/hummingbot_gateway_compose.yml
When running "docker-compose up -d", I got the following error:
"json: cannot unmarshal number into Go struct field LogConfig.HostConfig.LogConfig.Config of type string"
The FIX for this issue is to change the 'logging: max-file: 5' to use a "5" with the quotes
logging:
driver: "json-file"
options:
max-size: "10m"
max-file: 5  --> "5"
I didn't check all the other deploy-examples docker-compose.yml files, but if using a number without the "", it will generate same error

Thanks for the suggestion! @david-hummingbot can you make this change globally?

Great you were able to reproduce it. However, your solution doesn't really solve the problem for everyone. If configurations are put in as strings but then used appropriately as numbers in processing, then it solves the problem.
If you leave it as is, then expect a lot of people to have this problem and they're going to keep asking for help on the support discord.
A key note. I did not experience this problem on my windows environment where I did have Docker Desktop.
However, I decided to run the Bot in Linux and spun up an Oracle A1 and that is where I had this problem. There is no Docker Desktop. So I installed docker-compose there via: sudo apt  install docker-compose
That is what most people would do in a Linux environment. So if the version that is available there is not the latest that fixes this problem, then they'll see this problem like I did.


Add logic for handling a stopped body

Bodies should never be completely stopped ever in the current iteration. Will probably just add a reset button to stop never ending bodies.


Performance regression in arrow_batch_points/query benchmark

Somehow it seems that #3162 caused a serious performance regression for this specific benchmark:

(See it live here)

For 0.9: let's see if this has significant impact on a real point cloud example, or if we can punt on this to 0.10

I suspect some of Jeremy's optimizations got lost in the above PR, in particular the bytemuck deserialization of Position3D and other trivial types.

Just as a note: real-life point cloud performance has improved by 3x from 0.8.2. to 0.9: https://github.com/rerun-io/rerun/issues/1136#issuecomment-1748716579

The regression only impacts the legacy query APIs, which do not exist anymore.
$ git co 905fcaf56  # pre-offending PR

$ cargo bench --all-features -p re_query -- 'arrow_batch_points/query'  # legacy APIs
arrow_batch_points/query
                        time:   [5.4049 µs 5.4188 µs 5.4373 µs]
                        thrpt:  [183.92 Melem/s 184.54 Melem/s 185.02 Melem/s]

$ cargo bench --all-features -p re_query -- 'arrow_batch_points2/query' # new APIs
arrow_batch_points2/query
                        time:   [7.4022 µs 7.4485 µs 7.4947 µs]
                        thrpt:  [133.43 Melem/s 134.26 Melem/s 135.09 Melem/s]

$ git co 0a2258a  # offending PR

$ cargo bench --all-features -p re_query -- 'arrow_batch_points/query'  # legacy APIs
arrow_batch_points/query
                        time:   [14.340 µs 14.359 µs 14.382 µs]
                        thrpt:  [69.531 Melem/s 69.642 Melem/s 69.734 Melem/s]
                 change:
                        time:   [+164.89% +165.74% +166.47%] (p = 0.00 < 0.05)
                        thrpt:  [-62.472% -62.369% -62.248%]
                        Performance has regressed.

$ cargo bench --all-features -p re_query -- 'arrow_batch_points2/query'  # new APIs
arrow_batch_points2/query
                        time:   [7.4420 µs 7.4699 µs 7.4966 µs]
                        thrpt:  [133.39 Melem/s 133.87 Melem/s 134.37 Melem/s]

Oh, and for completeness, here's main:
$ git co main

$ cargo bench --all-features -p re_query -- 'arrow_batch_points2/query'
    Finished bench [optimized + debuginfo] target(s) in 0.16s
     Running benches/query_benchmark.rs (target/release/deps/query_benchmark-78136916f96ddd02)
arrow_batch_points2/query
                        time:   [2.6338 µs 2.6503 µs 2.6766 µs]
                        thrpt:  [373.60 Melem/s 377.32 Melem/s 379.67 Melem/s]
                 change:
                        time:   [-64.660% -64.496% -64.320%] (p = 0.00 < 0.05)
                        thrpt:  [+180.27% +181.66% +182.96%]
                        Performance has improved.

Which is indeed 3x faster than the legacy.


Padding class helper as vuetify has

This function solves the problem (这个功能解决的问题)
like pl-2 for padding left, pr-2 for padding right... the same for margins
Expected API (期望的 API)
like pl-2 for padding left, pr-2 for padding right... the same for margins

I'm not sure this should be added. Since if we don't provide many classes, it would be very easy for user to create some needed classes by themself. However if we provide many classes, tailwind or bootstrap is the best choice.

I'm not sure this should be added. Since if we don't provide many classes, it would be very easy for user to create some needed classes by themselves. However if we provide many classes, tailwind or bootstrap is the best choice.

so, do you recommend bootstrap be used within naive, in this case?

I'm not sure this should be added. Since if we don't provide many classes, it would be very easy for user to create some needed classes by themselves. However if we provide many classes, tailwind or bootstrap is the best choice.

so, do you recommend bootstrap be used within naive, in this case?

Any relevant library is okay.

As a Vuetify user i can attest to the fact that these types of classes are better when linked to the UI library because you can use the same variables used in the components.
For example (forced to prove the point): If i have a n-list with a n-list-item inside and would like to align an element outside the list with the n-list-item, i would have to to inspect the list, see that the left padding is 20px and add a style to the element. It would be much easier if i had a ml-<global padding step> class knowing that the classes have the same padding steps used in the component.
A workaround would be to modify Tailwind to have the same variables as the components but that would be kind of tedious.
I didn't use Naive UI long enough to know if the paddings and margins used in components are equivalent to Tailwind for example. If that is true i think it should be stated in the documentation to reduce confusion.

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


feat: improve the ergonomics of empty leaf nodes

Objective

Currently in Rust Sitter, writing the AST can quickly become an ergonomics nightmare as all leaf nodes must be fields.
This PR improves this situation by allowing the #[rust_sitter::leaf] attribute on structs and enum variants with no fields; the changes are summarised below:

// old syntax:
struct Zero(#[rust_sitter::leaf(text = "0")] ());

enum Operator {
    Add(#[rust_sitter::leaf(text = "+")] ()),
    Sub(#[rust_sitter::leaf(text = "-")] ()),
    Mul(#[rust_sitter::leaf(text = "*")] ()),
    Div(#[rust_sitter::leaf(text = "/")] ()),
}

// new syntax:
#[rust_sitter::leaf(text = "0")]
struct Zero;

enum Operator {
    #[rust_sitter::leaf(text = "+")]
    Add,
    #[rust_sitter::leaf(text = "-")]
    Sub,
    #[rust_sitter::leaf(text = "*")]
    Mul,
    #[rust_sitter::leaf(text = "/")]
    Div,
}

Looks like there's a clippy lint failing, but otherwise looks great! Thanks for the improvement @soqb!


Several questions

I'm an experienced ssh user and new to assh. I have several questions about assh, some of which may be actual issues. I'm grouping them into one issue as a first pass to keep noise down.

I have serveraliveinterval: 60 in my defaults. assh config list displays this as ServerAliveInterval: <. Why is it displaying < instead of 60?

Is support planned for ProxyJump? It's the preferred alternative to -W/nc.

When my assh config has a host entry with no hostname (for example. "bastion.domain.com"), assh config list displays this as user@[hostname_not_specified]. Shouldn't this fallback to displaying the given host? Currently, I need a redundant hostname key in my config to fix the output.

Prepending "[custom options] foo=bar" in assh config list is hard to parse visually, because regular options are at a different indent level. Could this be "foo=bar [override]" instead?

assh wrapper ssh doesn't support -vv. I expected this to pass through to ssh.

Running assh connect host.domain.com --dry-run seems to always fail with "No such available gateway", even when assh wrapper ssh host.domain.com works.

Is there any way to see the full ssh command generated by assh connect? Currently, "Executing proxy command: exec assh connect --port=22 host.domain.com", which helps for debugging assh problems, but not one level down for problems with ssh.

Why does alias create a separate entry in ~/.ssh/config? ssh already supports "Host name othername" for multiple aliases.

Hi @pikeas, thank you for all these questions; it helps me to understand the usage and feeling of users 👍

I have ServerAliveInterval: 60 in my defaults. assh config list displays this as ServerAliveInterval: <. Why is it displaying < instead of 60?

< is the ascii representation of 60, see ; so basically I need to be sure that I print integer values as integer values and not as characters

Is support planned for ProxyJump? It's the preferred alternative to -W/nc.a

Good idea, I need to give more tests to this feature so it can be integrated transparently in assh.
Depending on my tests, it will probably become the new default way of connecting to hosts

When my assh config has a host entry with no hostname (for example. "bastion.domain.com"), assh config list displays this as user@[hostname_not_specified]. Shouldn't this fallback to displaying the given host? Currently, I need a redundant hostname key in my config to fix the output.

Ok, I need to think a little bit more about doing it globally or only for hosts that aren't wildcards or regexp
Two main options:

always applying the host key in place of a missing Hostname
using host key for bare names and keeping something special for wildcards and regex

What do you think?

Prepending "[custom options] foo=bar" in assh config list is hard to parse visually, because regular options are at a different indent level. Could this be "foo=bar [override]" instead?

Good idea, I will do it

assh wrapper ssh doesn't support -vv. I expected this to pass through to ssh.

Yes, the library used to parse CLI commands does not support "Count boolean" flags yet, so I need to send a PR to this library first, or use a custom parsing just for this command
As a quick fix, you should be able to call ssh without the wrapper, i.e., by prepending a backslash before ssh to tell your shell not to use aliases: $ \ssh ...

Running assh connect host.domain.com --dry-run seems always to fail with "No such available gateway", even when assh wrapper ssh host.domain.com works.

That comes from the way --dry-run is developed; this option does everything normally except the network connection that will always fail
This option is particularly useful when developing, which usage do you expect from this option, as a user?

Is there any way to see the full ssh command generated by assh connect? Currently, "Executing proxy command: exec assh connect --port=22 host.domain.com", which helps for debugging assh problems, but not one level down for problems with ssh.

Try $ assh -D connect or $ export ASSH_DEBUG=1;  assh connect

Why does alias create a separate entry in ~/.ssh/config? ssh already supports "Host name othername" for multiple aliases.

I can't remember if it was intentional or not; I will try to switch to the built-in alias support for assh alias and known hosts wildcard hosts

Thanks for the reply, @moul! Would you like separate issues for the changes in 1, 2, and 4?

(hostname not specified) - No strong feelings on my end about what's displayed.

(ssh -vv) - Thinking about it some more, this is a bigger scope than -vv. What I'm really looking for is assh wrapper ssh to look and act as much like ssh as possible, since the generated ~/.ssh/config isn't usable by ssh. That's a harder problem to solve.

(assh connect --dry-run) - My expectation is that this command will succeed if my config is valid, and show me what the connection will look like. When it failed, I assumed that meant I had a problem with that host. I think your answer to 7) is what I really need. :-)

(ssh aliases) - I've realized there's another issue here.

Example ssh config:
Host foo*.bar
    IdentityFile ~/ssh/special

Host *.bar
    User me
    IdentitifyFile ~/ssh/regular

ssh uses the first value found for each setting. This config will use the regular key for all .bar servers but use the special key for all foo.bar servers. But if I create the same config in assh:
hosts:
    "foo*.bar":
        identityfile: ~/ssh/special
    "*.bar":
        identityfile: ~/ssh/regular
        user: me

This generates the following config file:
Host *.bar
    IdentityFile ~/ssh/regular
    User me

Host foo*.bar
    IdentityFile ~/ssh/special

The ordering has been reversed! This will use the wrong key file for foo servers.

Thanks for the reply, @moul! Would you like separate issues for the changes in 1, 2, and 4?

Yes, definitely! 🍰

3: (hostname not specified) - No strong feelings on my end about what's displayed.

Ok, so I will start with option 2

5: (ssh -vv) - Thinking about it some more, this is a bigger scope than -vv. What I'm really looking for is assh wrapper ssh to look and act as much like ssh as possible

Totally agree; my goal for this wrapper is to act 100% like ssh

continuing 5: since the generated ~/.ssh/config isn't usable by ssh. That's a harder problem to solve.

That's very strange, the standard ssh should always be able to use the generated ~/.ssh/config :hushed:
Do not hesitate to give me an example if you encounter this issue again

6: (assh connect --dry-run) - My expectation is that this command will succeed if my config is valid, and show me what the connection will look like. When it failed, I assumed that meant I had a problem with that host. I think your answer to 7) is what I really need. :-)

I think that you should be interested in this feature idea: https://github.com/moul/advanced-ssh-config/issues/82

8: (ssh aliases) - I've realized there's another issue here.
[...]
The ordering has been reversed! This will use the wrong key file for foo servers.

Nice catch, you can open a new bug for this :)

There is already an open issue for 1.:  #181

I've pulled out most of these into separate issues.
Regarding generated ~/.ssh/config not being compatible, here's an example of what I mean:
Host foo
IdentityFile ~/.ssh/foo
User foo
# Inherits: [other]
# Gateways: [example.com]
# HostName: example-bastion.com
This entry is only usable by plain ssh because assh connect is set as the default ProxyCommand. Remove that and plain ssh will fail.
In other words, assh locks you in to connecting via assh, it's currently not possible to use assh only for managing ~/.ssh/config and not as a connection wrapper.


How to disable other credential helpers?

On Windows Server 2019 using:
PS git --version
PS git version 2.39.2.windows.1

with this .gitconfig :
[credential]
	helper = "D:/git-credential-oauth/git-credential-oauth.exe -device"
[credential "https://gitlab.example.com"]
	oauthClientId = redacted
	oauthScopes = read_repository write_repository
	oauthAuthURL = /oauth/authorize
	oauthTokenURL = /oauth/token
	oauthDeviceAuthURL = /oauth/authorize_device

I was expecting not to see any dialogue popping up but I am seeing this:

I then need to close the dialogue without signing in and then I can see the device code flow prompt:

I am able to open the url and enter the code and it works as expected from thereon.
Is the dialogue expected to show if -device is set?

Your first screenshot shows Git Credential Manager https://github.com/git-ecosystem/git-credential-manager
Run git config --get-all credential.helper to see which helpers you have configured.
Then try:
[credential]
	helper =
	helper = "D:/git-credential-oauth/git-credential-oauth.exe -device"
[credential "https://gitlab.example.com"]
	oauthClientId = redacted
	oauthScopes = read_repository write_repository
	oauthAuthURL = /oauth/authorize
	oauthTokenURL = /oauth/token
	oauthDeviceAuthURL = /oauth/authorize_device

following https://git-scm.com/docs/gitcredentials#Documentation/gitcredentials.txt-helper

If credential.helper is configured to the empty string, this resets the helper list to empty

On git version 2.39.2.windows.1 it seems the credential-cache is disabled:
git credential-cache
fatal: credential-cache unavailable; no unix socket support

Thus adding cache --timeout 21600 as a helper won't work.
As an attempt at a workaround I added wincred as a helper and after that I do not reed to reauthorize again and again. Would this be a good workaround on Windows @hickford or does it has some side effects? I guess the token might not be refreshed as expected and I guess it is not technically cached then?
[credential]
        helper =
	helper = wincred
	helper = "D:/git-credential-oauth/git-credential-oauth.exe -device"
[credential "https://gitlab.example.com"]
	oauthClientId = [appId]
	oauthScopes = read_repository write_repository
	oauthAuthURL = /oauth/authorize
	oauthTokenURL = /oauth/token
	oauthDeviceAuthURL = /oauth/authorize_device

Nevermind, I just updated "git for windows" ant it seems credential-cache is now supported on "git for windows" too. I just did not get the impression that this was fixed from the various issues floating around.


can't configure

Installed as per instructions. Most recent version online.
When I try to configure it, I get this error:

I can't get any logs. Sorry, please be patient with me I am still very new to HA.
Configuration
Add your logs here.

Describe the bug
When trying to configure with my username and password (verified they are correct) I get the error in the screenshot.
Debug log

Add your logs here.

Logs attached.
home-assistant_2024-09-22T23-57-03.854Z.log


What is going here

https://github.com/Muhammet-Yildiz/Mern-Blog/blob/226f8f15461961ea0a1f5aeb0435bdb62b61cdfc/Backend/Models/story.js#L66-L74
why did you write this hook and why you don't need to call the next()

If a story is deleted, all comments must be deleted before the next()
operation is done, because there is nothing to be done after this is
deleted.
Raihanur Rahman @.***>, 30 Haz 2022 Per, 09:44
tarihinde şunu yazdı:

https://github.com/Muhammet-Yildiz/Mern-Blog/blob/226f8f15461961ea0a1f5aeb0435bdb62b61cdfc/Backend/Models/story.js#L66-L74
why did you write this hook and why you don't need to call the next()
—
Reply to this email directly, view it on GitHub
https://github.com/Muhammet-Yildiz/Mern-Blog/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARGPQJPOJZDW7USFP7BACMTVRU633ANCNFSM52IF5UYA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Implement DNS-over-TCP redirection

The ztunnel side already supports this, but we do not do redirection. This is critical for correctness, since Ztunnel's DNS server is giving alternative DNS responses

We could do this in the CNI or in ztunnel by having a bypass mode in outbound


DRILL-8112: Excel Reader Ignores HeaderRow Config Param

DRILL-8112: Excel Reader Ignores HeaderRow Config Param
Description
Drill was ignoring the headerRow parameter in the Excel reader, which allowed Drill to skip to the first row of actual data.  This PR fixes that.
Documentation
N/A
Testing
Ran unit tests, and tested manually.

The JDK 11 test run included a failure for TestHttpPlugin#testSlowResponse which I'm going to attribute to test reproducability issues and not this PR which has nothing to do with the HTTP plugin.


Monitor global states after fixture

Hello,
we would like to validate that each fixture cleans up global variables, static classes and so on.
The following code snippet is our suggestion:
IFixtureMonitor = interface
    procedure PostTearDownFixture;
end;

{ snippet from DUnitX.TestRunner.pas }
procedure TDUnitXTestRunner.ExecuteTearDownFixtureMethod(const context: ITestExecuteContext; const threadId: TThreadID; const fixture: ITestFixture);
begin
  try
    Self.Loggers_TeardownFixture(threadId, fixture as ITestFixtureInfo);
    fixture.ExecuteFixtureTearDown;
    fFixtureMonitor.PostTearDownFixture; // use the new interface
  except
    on e: Exception do
    begin

      Log(TLogLevel.Error, Format(SFixtureTeardownError, [fixture.Name,e.Message]));
      raise;
    end;
  end;
end;

TFixtureMonitor = class(TInterfacedObject, IFixtureMonitor)
public
  procedure PostTearDownFixture;
end;

procedure TFixtureMonitor.PostTearDownFixture;
begin
  Assert.IsNull(pGlobalPointer);
end;

This is just an simplified example, the IFixtureMonitor could contain more methods.
What do you think about this proposal?

I guess my first question has to be, why are your tests messing with global state? Since the logger methods are now called (thanks for the PR) are you able to use that to achieve what you need?

We are able to use the logger methods.


Adapt Incubator Documentation into Incubator wiki

Overview
Currently, technical documentation is hosted within Incubator's repo. As as project stakeholder, I would like to have to technical documentation in the wiki so I can begin deploying my project to HFLA's incubator.
Action Items

[ ] Audit https://github.com/hackforla/incubator/tree/main/documentation

double-check branches to see if there's any other articles

[ ] Move articles with content to https://github.com/hackforla/incubator/wiki

Resources/Instructions
https://github.com/hackforla/incubator/tree/main/documentation

@JasonEb See below:
$ git push --set-upstream origin newWiki
remote: Permission to hackforla/incubator.wiki.git denied to joshuayhwu.
fatal: unable to access 'https://github.com/hackforla/incubator.wiki.git/': The requested URL returned error: 403

Any thoughts on setting git permissions?

@joshuayhwu I added you to ops and ops-write on Github, but you shouldn't have write markdown and git push from our local machine. You should be able to see the Wiki tab in the repo now and be able to directly edit the markdown in the Wiki. Lmk if that's not the case for you

Hey Josh, whenever good - please provide an update with this template. Lmk if you have any issues

Progress: "What is the current status of your project? What have you completed and what is left to do?"
Blockers: "Difficulties or errors encountered."
Availability: "How much time will you have this week to work on this issue?"
ETA: "When do you expect this issue to be completed?"
Pictures or links* (if necessary): "Add any pictures or links that will help illustrate what you are working on."

remember to add links to the top of the issue if they are going to be needed again.

Hey Josh, whenever good - please provide an update with this template. Lmk if you have any issues
Progress: "What is the current status of your project? What have you completed and what is left to do?"
Blockers: "Difficulties or errors encountered."
Availability: "How much time will you have this week to work on this issue?"
ETA: "When do you expect this issue to be completed?"
Pictures or links* (if necessary): "Add any pictures or links that will help illustrate what you are working on."
remember to add links to the top of the issue if they are going to be needed again.

Progress: completed at the ops wiki page.
Blockers: none.
Availability: depending on new job hours.
ETA: done
P.S.: If it's on the wrong wiki, simply copy the part from "Incubator Hosting" to "Dedicated Hosting" and move to wherever desire.


UAA + Credhub ops-files become default

Will the features of the credhub.yml and uaa.yml ops-files become default in bosh-deployment.yml? Would you accept a PR that enabled these features by default (now that they are stable and no longer experimental) and created ops-files that disable these two releases?

@genevieve i've been thinking about it as well, though i do like current separation and pluggability. as of now, im planning to keep them as it is. (they are also not experimental since they leave at the root).

We've got dozens of teams testing this with uaa+credhub enabled though, and fewer running recent versions of bosh-deployment with them disabled, so we should call out not enabling these as more outside the norm and slightly less likely to work. With two ops files that disable uaa and credhub individually, operators would have the same configurability (and the same frustration around not understanding which features depend on which, but at least fewer people would have to experience that).

i would recommand to hold it off until they fixed
https://github.com/cloudfoundry-incubator/credhub/issues/33

I still think it should be the default, we have not observed the issue described with Credhub.

The issue with growing DB has been fixed in CredHub with 1.9.2 when they moved to writing audit-relevant data to a logfile instead of the DB. If that was the only reason keeping us from using UAA and CredHub as a default, then go for it.

Hey @evanfarrar, I know Genevieve isn't on your team anymore. Is the infrastructure team still interested in submitting a PR for this issue? We'd be interested in receiving one.

😢


Version 3.9.0 changelog and main page favicon scroll rail

Adding upcoming update to the changelog

Thank you for the information!


Uncaught TypeError: Cannot read property 'Keyboard' of undefined

I am getting the following message in the debug console when I try to run an ionic app using corodva-plugin-googlemaps in the visual studio code IDE:
 Uncaught TypeError: Cannot read property 'Keyboard' of undefined
I am assuming it is referring to the reference to the keyboard property in the .run function of the app.js code.
However, the app builds successfully and loads on my device.
I tried removing and reinstalling both the android and iOS platforms.  Still I see this error message and the map is not loading.  I only see a blank white screen.  Any help would be appreciated.  thanks.

What plugins do you installed this plugin else?
If you share your WHOLE project, I can check yours.

this is what i have:
cordova-plugin-console
cordova-plugin-device
cordova-plugin-keyboard
cordova-plugin-splashscreen
cordova-plugin-whitelist
I read that i need to install com.plugin.keyboard
I typed this command: cordova plugin add com.plugin.keyboard
Fetching plugin "com.plugin.keyboard" via npm
Error: Failed to fetch plugin com.plugin.keyboard via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
Error: Registry returned 404 for GET on https://registry.npmjs.org/com.plugin.keyboard

I think you have to install cordova-plugin-keyboard or ionic-plugin-keyboard.

I installed both and added a reference to the ionic-plugin-keyboard js file like so:

Now I am not getting the original error message but I am getting a new message now:
Uncaught ReferenceError: plugin is not defined
Any ideas why this is showing up?  thanks.

Could you share your whole project to save my time? I'm not familiar with ionic also.
Github or bitbucket.org, please.

Hi. Here is a link to the project on github. Thanks.
https://github.com/Plex768/cordova-plugin-googlemap-project

Thanks, I will check it later.

Ok. Thank you.

When I executed $ ionic state restore, I got these errors. What should I do?
$ ionic state restore
Attempting to restore your Ionic application from package.json

Restoring Platforms

cordova platform add ios
cordova platform add android

Restore platforms is complete

Restoring Plugins

cordova plugin add cordova-plugin-device
cordova plugin add cordova-plugin-console
cordova plugin add cordova-plugin-whitelist
cordova plugin add cordova-plugin-splashscreen
cordova plugin add com.ionic.keyboard

/Users/masashi/.nvm/v4.2.1/lib/node_modules/ionic/node_modules/ionic-app-lib/lib/state.js:543
        throw 'Error executing "' + pluginCommand + '":\n' + output;
        ^
Error executing "cordova plugin add com.ionic.keyboard":
Error: Registry returned 404 for GET on https://registry.npmjs.org/com.ionic.keyboard

I am asking the same question. I don't know how to solve the issue associated with that error message.

Well, this is out-of-range of the plugin support.
Because this is related with com.ionic.keyboard project.
Please ask the project author @obetomuniz

Ok. thanks for your time. :)

Sorry, I'm not familiar with ionic.


python312Packages.pymfy: refactor

Description of changes
Fix build (https://hydra.nixos.org/build/270899140)

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 340505 run on x86_64-linux 1

4 packages built:

python311Packages.pymfy
python311Packages.pymfy.dist
python312Packages.pymfy
python312Packages.pymfy.dist


Move and merge some functions about issue

Merge updateAttachments and Comment.UpdateAttachments as UpdateCommentAttachments. They are duplicated.
Split issues_model.UpdateComment as two parts, one moved to service layer and keep the simple one in issue_model.

Closed as two refactor parts have been extracted.


[Bug] Cannot load trained adapter correctly into the InternVLChatModel model

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.
[X] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.

Describe the bug
The bug is I can save the adapter model but after I load the trained adapter weight and do continual training, the loss is not continual going down. The loss curve looks like the training is restarting instead of continuing from the first training.
Reproduction
Here is code about how I add adapter and save the adapter.
if model_args.use_llm_lora:
        model.wrap_llm_lora(r=model_args.use_llm_lora, lora_alpha=2 * model_args.use_llm_lora)
        model.config.use_llm_lora = model_args.use_llm_lora
        if model_args.train_llm_adapter and model_args.llm_adapter_name is None:
            lora_config = LoraConfig(
                lora_alpha=2 * model_args.use_llm_lora,
                r=model_args.use_llm_lora,
                base_model_name_or_path=tokenizer_path,
                target_modules=["q_proj", "o_proj", "k_proj", "v_proj", "gate_proj", "up_proj", "down_proj"],
                task_type="TOKEN_CLS",
            )
            model.language_model = get_peft_model(model.language_model, lora_config)
            model.add_adapter(lora_config, adapter_name=training_args.output_dir.split("/")[-1])
            model.set_adapter(training_args.output_dir.split("/")[-1])

class SavePeftModelCallback(TrainerCallback):
        def on_save(
            self,
            args: TrainingArguments,
            state: TrainerState,
            control: TrainerControl,
            **kwargs,
        ):
            checkpoint_folder = os.path.join(args.output_dir, f"{PREFIX_CHECKPOINT_DIR}-{state.global_step}")

            peft_model_path = os.path.join(checkpoint_folder, "adapter_model")
            kwargs["model"].save_pretrained(peft_model_path)

            pytorch_model_path = os.path.join(checkpoint_folder, "pytorch_model.bin")
            if os.path.exists(pytorch_model_path):
                os.remove(pytorch_model_path)
            return control

    trainer = Trainer(
        model=model,
        args=training_args,
        train_dataset=train_dataset if training_args.do_train else None,
        eval_dataset=eval_dataset if training_args.do_eval else None,
        tokenizer=tokenizer,
        data_collator=concat_pad_data_collator,
        callbacks=[SavePeftModelCallback],
    )

Here is the code about how I load the adapter and do continue training.
if model_args.model_name_or_path is not None:
        if model_args.train_llm_adapter and model_args.llm_adapter_name is not None:
            logger.info("Loading adapter for InternVLChatModel...")
            lora_config = LoraConfig.from_pretrained(model_args.llm_adapter_name)
            config = InternVLChatConfig.from_pretrained(model_args.model_name_or_path)
            model = InternVLChatModel.from_pretrained(
                model_args.model_name_or_path, torch_dtype=torch.bfloat16, config=config
            )
            model.language_model = PeftModel.from_pretrained(model.language_model,
                model_args.llm_adapter_name,
                is_trainable=True, #
                adapter_name=model_args.llm_adapter_name.split('/')[-2]
                # adapter_name="InternVL_LLM"
            )
            model.language_model.enable_input_require_grads()

Environment
--model_name_or_path "OpenGVLab/InternVL2-1B" \
--freeze_llm True \
--freeze_mlp True \
--use_llm_lora 16 \

Error traceback
No response

This bug has been resolved by using the save_pretrained method from peft. But since the embedding layer dimension of InternVL2 is different with Qwen2. Using save_pretrained will save the embedding layer which is the main reason to cause it saves a huge model. Set save_embedding_layers=False will address the issue and only save the adapter model.


Filter edit still throws exception

On seekbar change

Done https://github.com/CherryPerry/Amiami-android-app/commit/42adaacc2af7eaec1c9a6fb4999be35407cd5c72


fix incorrect syntax for ad-hoc regexp filter

Steps to reproduce:

Create a panel
Apply a key-value regexp filter
Expected result: Filter is applied
Actual result: a query fails due to incorrect syntax

error querying the database: clickhouse [execute]:: 400 code: Code: 62. DB::Exception: Syntax error (additional filter): failed at position 7 ('~'): ~ 'https://open.spotify.com/track/44ymejne8Roi7uT4lhiwCR' . Unrecognized token: '~'. (SYNTAX_ERROR)

After fix:

Changelog
Fix support for ad-hoc regexp filters.

@mshustov could we add a test case for this too? It looks great!

@asimpson added, PTAL

@mshustov could you add ILIKE to cspell.config.json? That will get Drone passing and we can merge.

@asimpson done. could you merge on green CI, please?


[Android] DropdownMenu not working

Hello! Thanks for this cool library!
We just tried a small snippet from the example project and found that it only works on iOS (and maybe web), but on Android, nothing is being shown when I tap on the Trigger. I this a known issue because I see that Android is supported from the documentation? Thank you!
Repro
 <DropdownMenu.Root key={filter}>
    <DropdownMenu.Trigger>
      <ThemedChip
        key={filter}
        isSelected
        label={filter}
        style={tw('mx-2 my-2')}
        rightComponent={
          <ThemedIcon
            name="x"
            color="white"
            size={16}
            style={tw('ml-1')}
          />
        }
        // onPress={() => removeFilter(filter)}
      />
    </DropdownMenu.Trigger>
    <DropdownMenu.Content
      style={{
        minWidth: 220,
        backgroundColor: 'white',
        borderRadius: 6,
        padding: 5,
        borderWidth: 1,
        borderColor: '#fff8',
      }}
    >
      <DropdownMenuLabel>Help</DropdownMenuLabel>

      {[1, 2, 3].map(i => (
        <DropdownMenuItem key={`list-${i}`}>
          <DropdownMenuItemTitle>{`Item ${i}`}</DropdownMenuItemTitle>
        </DropdownMenuItem>
      ))}
    </DropdownMenu.Content>
  </DropdownMenu.Root>

hey, thanks for checking! android is on the TODO at the bottom, though i thought it worked. does it just render nothing?

it should be a JS version only. i assume you installed the peer deps?

@nandorojo thanks for the prompt response! Yes I have made sure to install all peer deps. On Android, it renders nothing when I tap on a View that is wrapped by the <DropdownMenu.Trigger /> from the example code.

@axeldelafosse @intergalacticspacehighway were planning on adding the native menu support to Android, it should work after that is added.
i don't own an android so i'm sadly not exactly able to reproduce this one

to be sure, can you just put a text node inside of the trigger?

@nandorojo Using a Text node instead works! Does this mean that I have to menunify my custom components for it to work on Android?

ahh my guess is that your component isn't forwarding down the ref and the onPress prop. try that

@nandorojo sorry I realised I should have tried debugging more first before replying, but I found that it actually did work (somewhat), but this is what it looks like after I applied some debugging styles (padding and backgroundColor: "black"):

I am using a Pressable component in my ThemedChip which is stealing touch, and it works if I replace it with a View. Could you give an example of how I could forward the ref from Trigger to my Pressable component? Apologies as I am not very familiar with that 😔 Thank you!

@thespacemanatee can you remove Pressable from ThemedChip and just use a View? Zeego also adds a Pressable on Trigger so it won't work well with your Pressable. I am working on a native menu on android, which will solve it in the future.

As for forwarding the ref, it should look like this:
import { forwardRef } from 'react'

export const ThemedChip = forwardRef((props, ref) => {
  return <View ref={ref}>{...}</View>
})

@nandorojo thanks for the example. I am familiar with that and did try it, but wasn't very sure what it does since DropdownMenu.Trigger doesn't seem to be forwarding any ref to its children.
Simply using a View instead of a Pressable works for the moment, though I can't implement any onPressed styles.

Yeah, there is some magic going on here for Android. Under the hood, it's cloning the child and passing down a ref.
It first extracts the Trigger child here: https://github.com/nandorojo/zeego/blob/master/packages/zeego/src/menu/create-android-menu/index.android.tsx#L119
Update: now that I'm looking at it again, I don't think forward ref should be necessary, since we're measuring the Pressable: https://github.com/nandorojo/zeego/blob/master/packages/zeego/src/menu/create-android-menu/index.android.tsx#L149-L156
So the only thing that really matters is not using a Pressable I believe. This will be fixed when we move to a native menu.

Thank you guys for the informative discussion, and since there isn't an 'issue' with the Android implementation and a better implementation is coming up, feel free to close this issue! 😃

The Android native menu is now live in version 0.4.0. It requires installing @react-native-menu/menu.
Shout out to @intergalacticspacehighway for the #6 PR

@ddikodroid thanks for sharing! that's a pretty menu.
using the best native library for. a menu is very attractive. i'll admit that it's a bit unlikely that i'm able to add that, since i don't even know Java / native code itself, and i worry about maintaining something so out of my reach. i'd prefer to keep the API surface area within using RN libraries.
maybe there could be some sort of binding for these extra cases, like zeego/dropdown-menu/cascade. it's hard to say the best way to do it
anyway, i'll keep this in mind. appreciate the suggestion.


Not catching any requests

Hey
I've added the proxy and followed the instructions for vercel. I checked my frontend for plausible events, there were indeed triggered.
The plausible.io dashboard did not seem to show any visits. Do you know what could be the reason? How to debug it further?

Hi @hvitis! Sorry for the late response.
Can you paste your next.config.js file and your code using <PlausibleProvider>?

Hi @4lejandrito,
facing the same issues as @hvitis.
There doesn‘t seem to be an error but my Plausible Dashboard is simply showing Waiting for first pageview
on my-url.com.
HTTP Request shows 200.

Request URL:
https://my-url.com/js/script.js
Request Method:
GET
Status Code:
200 (from disk cache)
Remote Address:
[2a05:d014:275:cb01::c8]:443
Referrer Policy:
strict-origin-when-cross-origin

Script tag is correctly added to the frontend.
<script src="/js/script.js" async="true" defer="true" data-api="/proxy/api/event" data-domain="https://my-url.com/" data-nscript="afterInteractive"></script>
Here is my _app.js
import '../styles/global.css'; import PlausibleProvider from 'next-plausible'; import {ubuntu, neuton, norican} from '../src/utils/fonts'; const plausibleDomain = "https://my-url.com/";`
export default function App({ Component, pageProps }) {     return (         <PlausibleProvider domain={plausibleDomain}>         <div className={${ubuntu.variable} ${neuton.variable} ${norican.variable}}>         <Component {...pageProps} />         </div>         </PlausibleProvider>     ) }
Here is my next.config.js
const { withPlausibleProxy } = require('next-plausible') const nextConfig = {     /* config options here */   };`
// module.exports = nextConfig; module.exports = withPlausibleProxy()(nextConfig);
what could be the issue here? :(

Hey @niqwithq,
I believe you must remove https:// and the trailing / from your domain. Se a real life example.

That actually did the trick for me! Thank you so much! And thanks for the quick reply!


ts-jest causes Babel to throw while generating cache keys

Issue :
My test suite will not run and shows the error:

[BABEL] unknown: Configuration contains string/RegExp pattern, but no filename was passed to Babel

I’ve tracked it down far enough to know that the error is being thrown in TsJestTransformer.getCacheKey() because that calls the getter ConfigSet.babel(), which calls loadOptions() in Babel.
I also know that commenting out the error throw site in Babel works and allows my test suite to run, which makes me thing that ts-jest is using Babel in an unexpected (but previously valid) way.
Expected behavior :
I expect my test suite to run without Babel throwing an error.
Debug log:

log file content
# content of ts-jest.log :
{"context":{"logLevel":20,"namespace":"Importer","package":"ts-jest","version":"23.10.5"},"message":"creating Importer singleton","sequence":1,"time":"2019-01-04T22:49:48.405Z"}
{"context":{"allowJs":false,"logLevel":20,"namespace":"jest-preset","package":"ts-jest","version":"23.10.5"},"message":"creating jest presets not handling JavaScript files","sequence":2,"time":"2019-01-04T22:49:48.410Z"}
{"context":{"actualVersion":"23.6.0","expectedVersion":">=22 <24","logLevel":20,"namespace":"versions","package":"ts-jest","version":"23.10.5"},"message":"checking version of jest: OK","sequence":3,"time":"2019-01-04T22:49:48.414Z"}
{"context":{"baseOptions":{},"logLevel":20,"namespace":"jest-transformer","package":"ts-jest","transformerId":1,"version":"23.10.5"},"message":"created new transformer","sequence":4,"time":"2019-01-04T22:49:48.414Z"}
{"context":{"fileName":"/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro/src/dummy.tests.ts","logLevel":20,"namespace":"jest-transformer","package":"ts-jest","transformOptions":{"instrument":false,"rootDir":"/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro"},"transformerId":1,"version":"23.10.5"},"message":"computing cache key for /Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro/src/dummy.tests.ts","sequence":5,"time":"2019-01-04T22:49:48.415Z"}
{"context":{"logLevel":30,"namespace":"jest-transformer","package":"ts-jest","transformerId":1,"version":"23.10.5"},"message":"no matching config-set found, creating a new one","sequence":6,"time":"2019-01-04T22:49:48.415Z"}
{"context":{"config":{"automock":false,"browser":false,"cache":true,"cacheDirectory":"/var/folders/31/qs9vmhyd09g85lkk1ktcz5tw0000gn/T/jest_dx","clearMocks":false,"coveragePathIgnorePatterns":["/node_modules/"],"cwd":"/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro","detectLeaks":false,"detectOpenHandles":false,"errorOnDeprecated":false,"filter":null,"forceCoverageMatch":[],"globals":{"ts-jest":{"babelConfig":"babel.config.json"}},"haste":{"providesModuleNodeModules":[]},"moduleDirectories":["node_modules"],"moduleFileExtensions":["js","jsx","json","ts","tsx","d.ts"],"moduleNameMapper":[],"modulePathIgnorePatterns":[],"name":"fe199b29939e30882d9be5087fe59bba","prettierPath":null,"resetMocks":false,"resetModules":false,"resolver":null,"restoreMocks":false,"rootDir":"/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro","roots":["/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro"],"runner":"jest-runner","setupFiles":[],"setupTestFrameworkScriptFile":null,"skipFilter":false,"snapshotSerializers":[],"testEnvironment":"jest-environment-jsdom","testEnvironmentOptions":{},"testLocationInResults":false,"testMatch":["**/__tests__/**/*.js?(x)","**/?(*.)(spec|test).js?(x)","**/tests.(js|jsx|ts|tsx)","**/*.tests.(js|jsx|ts|tsx)"],"testPathIgnorePatterns":["/node_modules/"],"testRegex":"","testRunner":"/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro/node_modules/jest-jasmine2/build/index.js","testURL":"http://localhost","timers":"real","transform":[[".(js|jsx|ts|tsx)","/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro/node_modules/ts-jest/dist/index.js"]],"transformIgnorePatterns":["/node_modules/"],"watchPathIgnorePatterns":[]},"logLevel":20,"namespace":"backports","package":"ts-jest","transformerId":1,"version":"23.10.5"},"message":"backporting config","sequence":7,"time":"2019-01-04T22:49:48.416Z"}
{"context":{"jestConfig":{"automock":false,"browser":false,"cache":true,"cacheDirectory":"/var/folders/31/qs9vmhyd09g85lkk1ktcz5tw0000gn/T/jest_dx","clearMocks":false,"coveragePathIgnorePatterns":["/node_modules/"],"cwd":"/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro","detectLeaks":false,"detectOpenHandles":false,"errorOnDeprecated":false,"filter":null,"forceCoverageMatch":[],"globals":{"ts-jest":{"babelConfig":"babel.config.json"}},"haste":{"providesModuleNodeModules":[]},"moduleDirectories":["node_modules"],"moduleFileExtensions":["js","jsx","json","ts","tsx","d.ts"],"moduleNameMapper":[],"modulePathIgnorePatterns":[],"name":"fe199b29939e30882d9be5087fe59bba","prettierPath":null,"resetMocks":false,"resetModules":false,"resolver":null,"restoreMocks":false,"rootDir":"/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro","roots":["/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro"],"runner":"jest-runner","setupFiles":[],"setupTestFrameworkScriptFile":null,"skipFilter":false,"snapshotSerializers":[],"testEnvironment":"jest-environment-jsdom","testEnvironmentOptions":{},"testLocationInResults":false,"testMatch":["**/__tests__/**/*.js?(x)","**/?(*.)(spec|test).js?(x)","**/tests.(js|jsx|ts|tsx)","**/*.tests.(js|jsx|ts|tsx)"],"testPathIgnorePatterns":["/node_modules/"],"testRegex":"","testRunner":"/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro/node_modules/jest-jasmine2/build/index.js","testURL":"http://localhost","timers":"real","transform":[[".(js|jsx|ts|tsx)","/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro/node_modules/ts-jest/dist/index.js"]],"transformIgnorePatterns":["/node_modules/"],"watchPathIgnorePatterns":[]},"logLevel":20,"namespace":"config","package":"ts-jest","transformerId":1,"version":"23.10.5"},"message":"normalized jest config","sequence":8,"time":"2019-01-04T22:49:48.416Z"}
{"context":{"fromPath":"babel.config.json","logLevel":20,"namespace":"config","package":"ts-jest","toPath":"/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro/babel.config.json","transformerId":1,"version":"23.10.5"},"message":"resolved path from babel.config.json to /Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro/babel.config.json","sequence":9,"time":"2019-01-04T22:49:48.417Z"}
{"context":{"logLevel":20,"namespace":"config","package":"ts-jest","transformerId":1,"tsJestConfig":{"babelConfig":{"kind":"file","value":"/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro/babel.config.json"},"compiler":"typescript","diagnostics":{"ignoreCodes":[6059,18002,18003],"pretty":true,"throws":true},"isolatedModules":false,"transformers":[],"tsConfig":{"kind":"file"}},"version":"23.10.5"},"message":"normalized ts-jest config","sequence":10,"time":"2019-01-04T22:49:48.417Z"}
{"context":{"logLevel":20,"namespace":"Importer","package":"ts-jest","requireResult":{"exists":true,"given":"babel-core","path":"/Users/rehkugler/Documents/Development/ts-jest-babel-no-filename-repro/node_modules/babel-core/index.js"},"version":"23.10.5"},"message":"loaded module babel-core","sequence":11,"time":"2019-01-04T22:49:48.443Z"}
{"context":{"logLevel":20,"namespace":"Importer","package":"ts-jest","version":"23.10.5"},"message":"patching babel-core","sequence":12,"time":"2019-01-04T22:49:48.444Z"}
{"context":{"actualVersion":"7.0.0-bridge.0","expectedVersion":">=6 <7 || 7.0.0-bridge.0","logLevel":20,"namespace":"versions","package":"ts-jest","version":"23.10.5"},"message":"checking version of babel-core: OK","sequence":13,"time":"2019-01-04T22:49:48.445Z"}

Minimal repo :
https://github.com/paulrehkugler/ts-jest-babel-no-filename-repro

I can't reproduce anymore the error after upgrading all your repo's dependencies to latest version so I'll close this issue. If you still can reproduce the same error with latest jest and ts-jest, feel free to open another issue.


SYMP-109: Update COVID Test REcommendation

Looks like you need to update questions.json in the cypress folder (line 136), with the new title. That should fix the tests.

Let me try that! Also noticed, the [contents] isn't quite accurate - does that matter @dbayly-freshworks ?

Let me try that! Also noticed, the [contents] isn't quite accurate - does that matter @dbayly-freshworks ?

Yes and no - Its good practice to have content of questions.json match the content of the form.
However, our cypress tests do not check the contents of that paragraph at any point, so it would not change anything functionally.

Thanks, that did it @dbayly-freshworks !


[TRI-1166] Help text in job dashboard makes it seem like every job is a scheduled job

For example, on a job that is triggered through a stripe webhook:

We should either

Detect when a job is a scheduled job and only show that help text then
Remove the help text to avoid confusion

This was first reported to me by a user in Discord so confident this confusion is happening "in the wild"
From SyncLinear.com | TRI-1166

@ericallam as part of frameworks support work im doing, i've also had to tweak the Callouts. Info callouts now look more muted – and for this panel, i've also moved it below the instructions.


Don't overwrite records from stale workflows

When we update records through the holdingpen, it's possible that the record has changed in the meantime and the workflow data is stale. We should avoid updating the record in that case and cause an error instead. This affects both record updates on the regular article workflow and the new core_selection workflow.

Tech:

Set version_id in all record schemas.
Add check in update which will compare version_id from record.model and incoming data (simillar to this one). Version in update HAS TO BE the same as in model.version_id otherwise raise an exception
Update function should have a flag which when enabled will change behavior of update to ignore missing/wrong version_id (for revert method and for migration)
revert should call update with flag to ignore version_id check
all migrator functions should call record.update with flag to ignore version check
On before_models_committed signal remove version_id from record json blob.
Update our fork of invenio-records to v1.3.2 (this version we are currently using)
In https://github.com/inspirehep/invenio-records/blob/050e7e1836c1d1b225e847f20fd0151a1df519a1/invenio_records/api.py#L114 call https://github.com/inspirehep/invenio-records/blob/050e7e1836c1d1b225e847f20fd0151a1df519a1/invenio_records/api.py#L117 with a copy of obj.json with version_id added into json.
On Next be sure that when we send record to HEP we have version_id inside record data. If not then add it (from obj.extra_data[head_version_id])

stupid question: can't we just set the If-Match header that is already checked in invenio-records-rest: https://github.com/inspirehep/invenio-records-rest/blob/c7db3d3d615ccea56e59f921dff78321beee904a/invenio_records_rest/views.py#L880 (which calls https://github.com/inveniosoftware/invenio-rest/blob/master/invenio_rest/views.py#L250-L285) and used by the editor, and modify the put method in our fork to disallow PUTs that don't have that header? seems a lot easier than inventing a new record field that needs to be checked manually.
Note that internal modifications of the form "read a record from DB, modify it, write back to DB" are already handled automatically by the SQLAlchemy version counter check, so it's really the REST layer, which doesn't start from a DB read in the same transaction, that needs this feature.

good point @michamos


Email Validation Available Offline ?

Hello,
Based on the feature description, I think that it's not something available offline so maybe to be added as information/limitation.

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

ID: de592fbb-8f65-109c-b659-4f19d5dbb1a1
Version Independent ID: de592fbb-8f65-109c-b659-4f19d5dbb1a1
Content: Email address validation for email columns in Dataverse - Power Apps
Content Source: powerapps-docs/maker/data-platform/data-validation-email-column.md
Service: powerapps
Sub-service: dataverse-maker
GitHub Login: @Mattp123
Microsoft Alias: matp

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


Add tool launcher

Fix #186

Install https://github.com/CosyVerif/tool-graphinf (in the dev branch).

please rember to replace string graphinfo.test  by  string graphinfo-test  in graphinfo-test.lua


By default, add logging to dask workers

It's common to want to log behavior when using dask. The best way to do this is to assign a logger to each dask worker via a worker plugin. That way when workers are restarted, they restart and assign a new logger, too.

Trying to create a logger local to LEAP Problem.evaluate() calls isn't efficient since it would have to be created for each instance and sent over the wire to the distant worker.  Better to have the worker install a logger once and use that.  And, it doesn't hurt anything if the user doesn't use the logger.  It's available if needed.

Added default logging for workers.  Tested.  Works.


chore: Updating the login and signup page for cloud hosting as per new design

Description
Updating the login and signup page for cloud hosting as per new design
Fixes #32267
Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!CAUTION]
🔴 🔴 🔴 Some tests have failed.
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8708497857
Commit: a3977dce76dd99f8b830a47c194f8f4e6de5b0d7
Cypress dashboard:  Click here!
The following are new failures, please fix them before merging the PR:
cypress/e2e/Regression/ServerSide/Datasources/ConnectionErrors_Spec.ts
cypress/e2e/Regression/ServerSide/Datasources/Redis_Basic_Spec.ts
cypress/e2e/Regression/ServerSide/QueryPane/EmptyDataSource_spec.js
cypress/e2e/Sanity/Datasources/ArangoDataSourceStub_spec.js
cypress/e2e/Sanity/Datasources/ElasticSearchDatasource_spec.js
cypress/e2e/Sanity/Datasources/MongoDatasourceURI_spec.ts
cypress/e2e/Sanity/Datasources/MySQL_spec.js
cypress/e2e/Sanity/Datasources/PostgresDatasource_spec.js
cypress/e2e/Sanity/Datasources/RedshiftDataSourceStub_spec.js
cypress/e2e/Sanity/Datasources/SMTPDatasource_spec.js
To know the list of identified flaky tests - Refer here

Summary by CodeRabbit

New Features

Enhanced titles and messages across login, signup, and forgot password pages for clarity and engagement.
Added new analytics event for tracking visits to self-hosting documentation.
Introduced new content and layout adjustments in user authentication pages to improve user experience.
Implemented conditional rendering to optimize content display for mobile devices and cloud hosting scenarios.

Bug Fixes

Updated footer links to use consistent capitalization.

Refactor

Major structural and styling overhaul of forms and user authentication components to utilize modern CSS practices and improve maintainability.

Documentation

Added direct links to self-hosting documentation to facilitate user access.

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true


BleakDBusError('org.bluez.Error.NotPermitted', 'Not paired')

Getting the following error during setup with a supported BT adapter that worked for at least one other device successfully. The adapter is a Plugable USB-BT4LE (BCM20702A0).
Retrying setup: An error occurred updating mug: e=BleakDBusError('org.bluez.Error.NotPermitted', 'Not paired')

Things I've tried (besides the normal setup instructions in the README):

Restart Home Assistant
Restart the host
Enter/exit pairing mode on the mug at different times during setup
Reinstalled the integration
Hard reset the mug

Since I was able to pair another device and set it up fully, I have a feeling this is specific to either this integration, or some characteristic of the mug itself as a bluetooth device which makes it different from others...

Ha, I figured it out. Seems to be some kind of interference -- using an extension cable to move the dongle away from the USB ports and Pi in general allowed it to pair.

Awesome! Glad you got it working and thanks for providing your solution for others too. :)


add OIDC connect authenticator

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