[$1000] Web - Cursor moves to the end when the max number of digits is reached,regardless of where it was

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

Action Performed:

Go to URL https://staging.new.expensify.com/
Log in with any account
Go to any chat
Click on + button -> Request money
Enter some digits -> move cursor between entering digits -> keep entering digits until you reach the maximum number of digits

Expected Result:
The cursor stays in the previous position
Actual Result:
The cursor moves to the end
Workaround:
Unknown
Platforms:

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

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/226933041-d770db3c-677f-4dfc-8ee4-1bd059474657.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Interna Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01fb413670ab760e1a
Upwork Job ID:<PHONE_NUMBER>350471680
Last Price Increase: 2023-03-28

Triggered auto assignment to @JmillsExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@JmillsExpensify Huh... This is 4 days overdue. Who can take care of this?

I'm not really sure this is a bug, though I can reproduce it.

Regardless, if this is easy to fix and we can keep the cursor in the same position, I agree that is better, even if not incredibly important. Opening up to external so we can evaluate the complexity of the solution we're dealing with.

Job added to Upwork: https://www.upwork.com/jobs/~01fb413670ab760e1a

Current assignee @JmillsExpensify is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @s77rt (External)

Triggered auto assignment to @AndrewGable (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The cursor moves to the end after text exceed max length
What is the root cause of that problem?
it is setState issue, see explanation:
for some reason we use custom method to detect maxLength in app called calculateAmountLength rather than officail prop maxLength
so we let the user to enter more than maxLength then we format it and pass it down in input prop formattedAmount
for example
if the user enter 123456789 we will return previous value 12345678.
and force input to change it's value from 123456789 to 12345678, and this force lead the Cursor to reset at the end.
we aleardy have a method when value not vaild we will setState to prevState but it not triggering re-render becaue we return the same object.
and React would compare with the previous state using strict comparison === to decide whether to kick rerender. and  === would return true for the same object so no re-render will occured.
    getNewState(prevState, newAmount) {
        if (!this.validateAmount(newAmount)) {
            return prevState; //same object will not triggering re-render
        }
        const selection = this.getNewSelection(prevState.selection, prevState.amount.length, newAmount.length);
        return {amount: this.stripCommaFromAmount(newAmount), selection};
    }

What changes do you think we should make in order to solve the problem?
we need to make a shallow copy of the object
replace this return prevState; with return {...prevState, selection: {...prevState.selection}}; will fix the issue and not affect about anything because no diffrence between the two objects.
  getNewState(prevState, newAmount) {
        if (!this.validateAmount(newAmount)) {
            return {...prevState, selection: {...prevState.selection}};
        }
        const selection = this.getNewSelection(prevState.selection, prevState.amount.length, newAmount.length);
        return {amount: this.stripCommaFromAmount(newAmount), selection};
  }

and this the result.
https://user-images.githubusercontent.com/41129870/228268882-a739bc09-678e-4d16-961e-23b48d2e0291.mov

📣 @ahmedGaber93! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.

Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

@ahmedGaber93 Thanks for the proposal! You are saying that the bug is due to a missing re-render that should happen but it's not since we are using the same selection object where strict comparison fail.
I don't think this is about a missing re-render but rather an internal early return on the function that is responsible of changing the selection. If it was due to a missing re-render this issue would not exist since we always re-render the component due to this.state.amount being different. I took a look into this and found the root cause to be this piece of code here, setSelection won't even get called if we use the same selection object.
Your RCA is pretty accurate and the fix of using a shallow copy of selection looks good to me.
:ribbon: :eyes: :ribbon: C+ reviewed
cc @AndrewGable

📣 @ahmedGaber93 You have been assigned to this job by @AndrewGable!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

PR will be ready for review about after two days.

I sent Upwork proposal and waiting for acceptance on Upwork.

@JmillsExpensify, @AndrewGable, @s77rt, @ahmedGaber93 Eep! 4 days overdue now. Issues have feelings too...

@JmillsExpensify, @AndrewGable, @s77rt, @ahmedGaber93 Eep! 4 days overdue now. Issues have feelings too...

PR deployed to staging 3 days ago.
Waiting for production deployment...

@JmillsExpensify, @AndrewGable, @s77rt, @ahmedGaber93 Whoops! This issue is 2 days overdue. Let's get this updated quick!

The PR that introduced the bug has been identified: https://github.com/Expensify/App/pull/10147
The offending PR has been commented on: https://github.com/Expensify/App/pull/10147#issuecomment-1514043472
A discussion in #expensify-bugs has been started: https://expensify.slack.com/archives/C049HHMV9SM/p1681872199137209

Regression Test Proposal

Open App
Click FAB -> Request money
Type 8 digits e.g. 12345678
Type another digit
Verify that you were not able to add that digit and that the cursor is in the end
Move cursor to that middle i.e. after number 4
Type another digit
Verify that you were not able to add that digit and that the cursor is still in the middle

Ah in that case, isn't my math above incorrect? It should be $500 for each, right? Apologies if I misunderstood you.

@JmillsExpensify Why is that?

Ahhh, ok. I see now. The first part of your comment was completing the BZ checklist. I just missed the context.

Offers sent to both. Please accept on Upwork so I can issue payment.

Everyone is paid out at this point. Last remaining step is creating an issue for TestRail.

@JmillsExpensify, @AndrewGable, @s77rt, @ahmedGaber93 Still overdue 6 days?! Let's take care of this!

@JmillsExpensify

Last remaining step is creating an issue for TestRail

Any updates here?

Thanks for the bump! Apologies, I lost the thread on doing this one.


Cleanup Packer directory after VHD build

What this PR does / why we need it: Cleanup artifacts in packer dir at the end of VHD build. To avoid having to wait for a new image for the fix, also temporarily do this during CSE provisioning. Also cleanup rootfs- k8s artifacts.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #3961
Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

/lgtm pending E2E and manual validation of /home storage

$ df -h
Filesystem      Size  Used Avail Use% Mounted on
udev            3.4G     0  3.4G   0% /dev
tmpfs           697M  1.2M  696M   1% /run
/dev/sda1        30G  5.9G   24G  21% /
tmpfs           3.5G     0  3.5G   0% /dev/shm
tmpfs           5.0M     0  5.0M   0% /run/lock
tmpfs           3.5G     0  3.5G   0% /sys/fs/cgroup
/dev/sdb1        99G   60M   94G   1% /mnt
/dev/sdc1       252G  185M  239G   1% /var/lib/etcddisk
tmpfs           697M     0  697M   0% /run/user/1001

from a master on E2E-generated cluster?


Rectify example in C4178

Rectified some errors in the previous example, namely:

The usage of an uninitialized variable i, which is UB
The comments on the maximum value being for unsigned int but the type of i is int

The warning is also not emitted with just /W1 and the default settings of a fresh latest VS2022 C++ project. For some reason, the /permissive flag must be set for the warning to be shown. WIth the default of /permissive, the warning is not emitted even with /Wall and/or /w14178. Not sure if the root cause is some other more granular flag being implicitly set by the permissive flag.

@TylerMSFT

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Looking into why /permissive is required.


One exchange_client has one exchange and one exchange own one ringbuffer && void *** param for get msgs return

The api for get msgs is still need to discuss. A new api or use nng_send_aio?
The broken UT need to fix, wip....
NEW config seems like

And an exchange demo can see https://github.com/nanomq/nanomq/tree/moi_exchange_demo

ok remember to fix UT


Header Link Icon

Something that might be nice to is a way to get the link to the header. For example on the npm website, when you hover over a header, there is a link icon the pops up and clicking it gives you a direct url to that header (for example)

I've already done this for my site and might make a good addition for the main template.
Here's a sample pr: https://github.com/Adriel-M/adriel.dev/pull/10 where the necessary changes are:

source the icon (for my case, I used octicons)
change up rehype-autolink-headings config to prepend the element with the icon
Add some css to handle show/hiding when hovering/not hovering. I did have to upgrade to the latest rehype-autolink-headings so I can add a css class to the header element.

Thanks, I have added it to my project.

Thanks, I agree as well. It's a nice improvement - will add it to the to do list.


Discussion about future fix / feature : discord now available !

Hi @NPBruce, all,
Do you have a place where we can talk about new features, issues assignement, development questions and ideas to solve issues ( #111 for example)?
I would gladly help on a few topics on this nice project.
I'm thinking about a forum / channel in a place like Reddit or Discord.
It would also be nice for people to exchange tips when creating quests.

Discord is not a bad shout, I talk to a couple on IM within BGG but a forum for this might be handy.
Though I think a proper quest distribution and social platform is what's being discussed in #111.
I imagine somewhere where you can upload quests and then mods can approve to release them to the Valkyrie lists.
Also a place dedicated to rating and commenting on scenarios.

I personally am not in a position to engage in any real time communication (IM/audio/video). Happy for others to do so. I'm not against a forum/reddit, but I'm not sure that it will add a lot over the use of issues for discussion (like this one) and will fragment content between here and another place.
Perhaps scenario development specifically is a good case for discussion to live somewhere else.

A forum/reddit would be good for sure for players, and for content creators : those who do not want to create an account on a strange development website like 'github' :)
I agree it is not so good for developers as issues should be all listed on github.
A chat system like discord would be interesting for developers though.
BGG forum is good mostly to do some promotion of content, I guess, but difficult to use it as a centralized point.

I have created a discord : https://discord.gg/yrVeSVt
Please join :)
It would be great to share it on the README.md (GitHub first page).
Let's see if anyone comes and want to discuss :)

I added to the main wiki page (readme.md just links to it).

Thanks @NPBruce


Cerberus's SequenceR do not end when running in paralel

Hi,
I noticed that when running Cerberus with SequenceR (with 11 bugs in Defects4J) in parallel mode, Cerberus did not quit even after 1 hours. Although all threads have finished with a notification:

Status of all tasks in the summary screen still appears as "Validating predictions" or "Running," indicating that the tasks have not yet been completed.

It is quite minor as I still can manually check all thread and close.  However, I really appreciate if you could check this when you have time?
Many thanks.

Hi, was this running in debug mode? I have an idea about it and will make a fix later today

I tested multiple times. I think I ran with both debug and normal mode.

Debug mode does not close Cerberus when done. This is intended functionality, the non-debug mode should close but this has to be examined

Oh I see. However, the status was still not updated correctly as I observed that it is still "running" or "validating" though the corresponding thread is finished.

Yeah, that is a bug that should be checked out. Thanks for the info!

Hi, I figured out the bug. It was related with the way condition variables were used. It is fixed in dev now. Will close as I am not able to reproduce it anymore


Deploy Connext UI

Prepare new xpollinate.io

[x] host/deploy connext.li.finance (+ xpollinate.io)
[x] Customize UI (header, footer, background)
[x] store configuration in URL query (partly - wait for mainnets)
[x] redirect old xpollinate query parameters (partly - wait for mainnets)
[x] parse big numbers
[x] reduced UI of popup
[x] history
[x] hide scroll bars
[x] closable info
[x] embed logo text in li.finance logo
[x] make "No route" retry
[x] disable optimism
[x] social media preview
[x] account change (delete messaging token)
[x] Add BSC testnet
[x] fix fee calculation for history
[x] mobile logos
[x] favicon
[x] host testnet version (testnet.xpollinate.io / testnet-connext.li.finance)
[x] link testnets to other domain
[x] Nice Info text
[x] check if token approval is needed or not
[x] direct xpollinate domain to new nameservers
[ ] show errors if step fail
[ ] hide failed transfers from active transactions list
[ ] find replaced transaction (eg. speedup)
[ ] show tx confirmations on the sender prepare and receiver prepares
[ ] update listeners in nxtp.tsx to check transaction_id and listen to multiple events (to handle multiple transactions in parallel)
[ ] toggle active/historic transactions by clicking the small arrow (https://github.com/lifinance/lifi-web/pull/20)
[ ] show warning if receiving address different to the own address is set (https://github.com/lifinance/lifi-web/pull/20)
[ ] save options in local storage (https://github.com/lifinance/lifi-web/pull/20)

Optional:

[ ] (wait for sdk support) avoid multiple messaging sign in
[ ] connected with unknown network
[ ] switch connected account / better disconnect
[ ] serve font from our domain
[ ] show balance and max in form
[ ] check bundle size

Mainnet:

[x] add mainnet token and chains
[x] store configuration in URL query
[x] redirect old xpollinate query parameters

Launch:

[ ] CNAME settings for xpollinate.io
[ ] Caching for xpollinate.io

Thanks @Tommy-R8 and @andyranitzsch for your feedback. I added the tasks.
The "less important" ones are good to keep in mind, but we will for noch focus on getting the first release done and then take care of the optional ones.


Update link to implementation guide in README

Change link location (prior link was dead)
Changed wording from deployment guide to implementation guide to match site

Issue #, if available:
Description of changes:
I performed minor changes to text and links.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@billkidwell Thanks for finding a wrong information.
We will update this one to the next release.


Deploy app to the web

A small DO droplet or EC2 instance will probably do. I think we'll need 16GB for the database, to be safe.

Done!


Feature: use extendSchema to generate schemas

Right now, we use typeDef arrays to generate a schema that is in multiple pieces. This is not a very nice notation, and it was mostly a workaround because it wasn't possible to extend schemas. We should refactor the code to use the extendSchema utility of graphql-js.

@helfer if you can give me some direction on this I'd love to pick it up!

@jbaxleyiii actually, never mind. After double-checking, it seem that extendSchema isn't quite as useful as I thought. Our original array syntax was so that you could import definitions from other files and still have everything work (which was easier than defining a custom loader for .graphql files). extendSchema doesn't really solve the same problem. I think for the time being, the best approach to use is @nnance's graphql-loader.


Remove crawlers from analytics

1.	semalt.semalt.com
2.	buttons-for-website.com

Maybe some others.
@reederz do you want to handle it?

I'll do it
On 02/18/2015 04:59 PM, Dan Mindru wrote:

|1.  semalt.semalt.com
2.  buttons-for-website.com
|
Maybe some others.
@reederz https://github.com/reederz do you want to handle it?
—
Reply to this email directly or view it on GitHub
https://github.com/fadeit/fadeitdk/issues/32.


Automating the setup process

So the setup readme has improved but there is still a bit of tedius work (Just switching a bunch of bundle ids) in order to get the program running on your phone.
I wrote a small python script to go and change the needed files.
I have no idea how safe this is. I tested it out and was up and running quickly. Since it's used during setup worst case should be a re-cloning of the base repo. That being said I don't know enough to know if this is a terrible idea.
All files are backed-up under FILENAME.EXTENSION.bak and those backups are only deleted if no error is thrown.
I've attached the python script (As a txt, github wont allow .py)
I already reworded the setup guide and added the script in properly in case this ends up becomes a PR.
quickstart.txt

@Huddie I was wondering how do I negotiate renaming these files with Git?

@BrianLitwin As in when you push? Right now I dont commit these files. This way they stay with my bundle name on my local machine but the base repos bundle name on my github repo

@telip007 Oh cool!!! I had a feeling someone already tried tackling this.
Sent with GitHawk


vcpkg - cannot open included file

Microsoft (R) Build Engine version <IP_ADDRESS>428 for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.

Build started 2/16/2018 7:34:14 PM.
     1>Project "C:\vcpkg\vcpkg\buildtrees\vcpkg-qt-test\src\head\vcpkg-qt-test-master\vsproject\test.sln" on node 1 (Rebuild target(s)).
     1>ValidateSolutionConfiguration:
         Building solution configuration "Release|x64".
     1>Project "C:\vcpkg\vcpkg\buildtrees\vcpkg-qt-test\src\head\vcpkg-qt-test-master\vsproject\test.sln" (1) is building "C:\vcpkg\vcpkg\buildtrees\vcpkg-qt-test\src\head\vcpkg-qt-test-master\vsproject\Test.vcxproj" (2) on node 1 (Rebuild target(s)).
     2>CoreClean:
         Creating directory "x64\Release\".
       PrepareForBuild:
         Creating directory "x64\Release\Test.tlog\".
       InitializeBuildStatus:
         Creating "x64\Release\Test.tlog\unsuccessfulbuild" because "AlwaysCreate" was specified.
       ClCompile:
         C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.12.25827\bin\HostX86\x64\CL.exe /c /Zi /nologo /W3 /WX- /diagnostics:classic /sdl /O2 /Oi /GL /D _MBCS /Gm- /EHsc /MD /GS /Gy /fp:precise /permissive- /Zc:wchar_t /Zc:forScope /Zc:inline /Fo"x64\Release\\" /Fd"x64\Release\vc141.pdb" /Gd /TP /errorReport:queue ..\code\main.cpp
         main.cpp
     2>..\code\main.cpp(2): fatal error C1083: Cannot open include file: 'QtWidgets\qapplication.h': No such file or directory [C:\vcpkg\vcpkg\buildtrees\vcpkg-qt-test\src\head\vcpkg-qt-test-master\vsproject\Test.vcxproj]
     2>Done Building Project "C:\vcpkg\vcpkg\buildtrees\vcpkg-qt-test\src\head\vcpkg-qt-test-master\vsproject\Test.vcxproj" (Rebuild target(s)) -- FAILED.
     1>Done Building Project "C:\vcpkg\vcpkg\buildtrees\vcpkg-qt-test\src\head\vcpkg-qt-test-master\vsproject\test.sln" (Rebuild target(s)) -- FAILED.

Build FAILED.

       "C:\vcpkg\vcpkg\buildtrees\vcpkg-qt-test\src\head\vcpkg-qt-test-master\vsproject\test.sln" (Rebuild target) (1) ->
       "C:\vcpkg\vcpkg\buildtrees\vcpkg-qt-test\src\head\vcpkg-qt-test-master\vsproject\Test.vcxproj" (Rebuild target) (2) ->
       (ClCompile target) ->
         ..\code\main.cpp(2): fatal error C1083: Cannot open include file: 'QtWidgets\qapplication.h': No such file or directory [C:\vcpkg\vcpkg\buildtrees\vcpkg-qt-test\src\head\vcpkg-qt-test-master\vsproject\Test.vcxproj]

    0 Warning(s)
    1 Error(s)

Time Elapsed 00:00:01.38

So, this works just fine if I try to build and run it through Visual Studio, but not when I try to install it. (I can even go into the buildtree directory, open the solution and build it.)
Source:
https://github.com/AshwinKumarVijay/vcpkg-qt-test
I put the vcpkg-qt-test folder (containing just the CONTROL and cmake file) in the ports directory.
I know that vcpkg allows for packages to depend on other packages, so I'm trying to figure out why this isn't working.

Hi @AshwinKumarVijay, thanks for the issue!
Currently, vcpkg_build_msbuild(PROJECT_PATH ${SOURCE_PATH}/vsproject/test.sln) won't add the current MSBuild integration into the project, which is why your build isn't finding the files! You can add OPTIONS /p:ForceImportBeforeCppTargets=${VCPKG_ROOT_DIR}/scripts/buildsystems/msbuild/vcpkg.targets to enable it.
See also [1] and [2].
[1] https://github.com/Microsoft/vcpkg/issues/891
[2] https://github.com/Microsoft/vcpkg/blob/a144a32fb2a11b5cd03f9577564aec03b4ada68d/ports/xalan-c/portfile.cmake#L48-L49

Works! (I wonder if maybe this should be the a default option?)
Either way, thank you very much! vcpkg is an awesome tool!


Add camera-open-status check

The program crashes on a specific device that does not support raw data.
Cuz when configureTransform called, the mCharacteristics still null.
This will cause a crash.
SoI added a piece of code to avoid executing the rest of the code if openCamera return false.

@googlebot I signed it!


[Bug] be abnormally quit on mac

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
master branch
may be 1.2.x
What's Wrong?
when inserting data to one table, no response is returned.
then, be quit.
What You Expected?
be reponse, and inserting data successfully
How to Reproduce?
1、mac book pro os:13.4
2、master branch, then compile fe, compile be
3、priority_network: <IP_ADDRESS>/32
4、sqls：
(1) create database maps
(2)create table test6(baks int, ts int) distributed by hash(baks) buckets 1 properties("replication_allocation"="tag.location.default:1");
(3) insert into test6(baks, ts) values (1,2)
then be quit abnormally.
Anything Else?
no other response.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

You can try adding get_stack_trace_tool=glog to be.conf
i will fix it soon

I want to know why this happens,thx.

Thx, this solution helps!

This is the real bug, feedback from others, stack_trace.size is wrong

L


add button to improve contribution

To increase engagement and facilitate users' contribution.

ref: https://docusaurus.io/docs/next/api/plugins/@docusaurus/plugin-content-docs#editUrl

Before

After

FYK @idan89


Add missing fields to SecurityAdvisoryEvent

Add missing fields to SecurityAdvisoryEvent

Ah!  I see that you are fixing some legacy naming mistakes.  Thank you, @nikpivkin !
Marking this PR as "breaking API changes" out of necessity.

Thank you, @WillAbides !
Merging.


[5.8] Collection::reject() supports string callable

The current implementation's behavior is inconsistent with Collection::filter(),  which allows string callables for global helper/static methods.
Example:
$collection = collect(['a', '', null,'b']);

// Current: reject() performs a comparison between $value and 'blank' string literal

>>> $collection->reject('blank');
=> Illuminate\Support\Collection {#5774
     all: [
       "a",
       "",
       null,
       "b",
     ],
   }

// filter() works as intended:

>>> $collection->filter('blank');
=> Illuminate\Support\Collection {#5771
     all: [
       1 => "",
       2 => null,
     ],
   }

// After:

>>> $collection->reject('blank');
=> Illuminate\Support\Collection {#5774
     all: [
       0 => "a",
       3 => "b",
     ],
   }

Send to master as this could be a breaking change?


nice-to-have: Register-opinion https://hubchart-tarine.rhcloud.com/ on hubzilla.org and demo.hubzilla.org

I suggest the register-opinion/ link for https://hubchart-tarine.rhcloud.com/ on https://hubzilla.org -Homepage and on https://demo.hubzilla.org to offer. Thereby the visitor can as the case may be itself 	immediately register on the Hubzilla-grid.

There is actually /pubsites for that. I don't think https://hubchart-tarine.rhcloud.com is the proper place to find a public hub.

/pubsite lacks of information, I also use "tarine" site to look for a new site/hub ;) ,  I see there some important information to decide on which hub to subscribe.

I agree that https://hubchart-tarine.rhcloud.com has important information for a new user that /pubsites doesn't supply. It's presented in a neutral informative way, very well done.
The only thing missing is the rather improtant info about server roles (pro/standard). Are you  affilliated with the site @elvecio @HappyPony ? Maybe you can pass on the suggestion to incorporate this info?

I asked also for contact with https://hubchart-tarine.rhcloud.com/ -operator by https://www.openshift.com/ -operator via Twitter.

I am not affiliated with the site @elvecio .


Warn Link.to_gpu unless compatible with to_device

Takeover #4139.

I resolved the conflicts with #5986.

Is this feature still need?
We now have to_device and the original issue #4129 has been closed.

to_gpu should be warned anyway.
We no longer need sticky option.

I'm afraid if I understand the review comments well, but I addressed them.

The force-push did nothing important.  Just github.com was broken.

Jenkins, test this please

Jenkins CI test (for commit f611323e68a5a75478c3f915c9c60dd468fadf54, target branch master) failed with status FAILURE.

@niboshi This pull-request is marked as st:test-and-merge, but there were no activities for the last 3 days. Could you check?

Jenkins, test this please.

Jenkins CI test (for commit f611323e68a5a75478c3f915c9c60dd468fadf54, target branch master) failed with status FAILURE.

Please check test failure:

TestStandardUpdaterDeviceArgumentFallback_param_2_{omit_device=False, omit_input_device=True}.test_gpu_to_gpu_transfer _

Jenkins, test this please

Jenkins CI test (for commit ae3935a142022bb3ca6a6b40dbbf6e47a13e6908, target branch master) succeeded!


Svn Task caching user details

Hi All,
I have a small problem with the SVN task. When I try to call any of the actions with a username and password then Tortoise is caching the user details.
I tried to find the solution, but as I can see the extension should use the no-auth-cache, so this should not happen.
What am I doing wrong?
Thanks,
Peter
I am using TortoiseSVN 1.8.11
Task:
<UsingTask TaskName="MSBuild.ExtensionPack.Subversion.Svn" AssemblyFile="$(MSBuildProjectDirectory)\packages\MSBuild.Extension.Pack.1.6.0\tools\net40\MSBuild.ExtensionPack.dll " />
  <Target Name="SVNUpdate" AfterTargets="ExcludeFilesFromPackage">
    <MSBuild.ExtensionPack.Subversion.Svn TaskAction="Update" Items="$(SVNROOT)" UserName="BuildUser" UserPassword="SuperSafePassword:)"  />
   </Target>

so you are saying that after running that, the task works but a side effect is that tortoiseSVN uses some cached credential?

The task works, but after running it the user is cached.
Which also means if you have a user already using the SVN then that one is going to be knocked out and replaced by the user defined in the task. When no-auth-cache is used this should not happen.
For example. Lets say one of the devs runs a publish, then his actual user is going to be replaced with the user defined in the task. If they don't log out that user and log in again with their own user then all of their changes are going to be committed with the task's user.


Start a Local Cluster (Insecure): slow shutdown

Re: Start a Local Cluster (Insecure)
Issue Description
For step #7, "Stop the Cluster", I experienced two issues going through this:

The slowness in shutdown that is mentioned started with the 3rd node that was shut down, not the 4th node as described.
The shutdown process for each of the nodes 3, 4, and 5 took much longer than "about a minute".  Each took well over 10 minutes.

Suggested Resolution

Verify with the development team whether the slowness in shutdown is expected to be with the 3rd or with the 4th node that is shut down.
Verify with the development team what the expected shutdown time should be for the latter nodes, and document accordingly.

Note: This occurred using a 2020 MacBook Pro with 32GB RAM.

cc @taroface


Option to give existing canvas element as winit window

This PR extends WindowDescriptor resource with canvas field.
This is a Document.querySelector() string allowing pointing winit to an existing HTMLCanvasElement to use.
This allows embedding Bevy game in an existing web page.

Ooh does this mean we might be able to have multiple canvases that behave as different "bevy windows"?

We are able to do so already. Every Bevy/winit window creates a separate <canvas> HTML element.
This PR gives an option to use pre-existing canvas element instead of creating and attaching one.

Looks good to me. Its worth noting that I don't see an error/panic like I would expect to when the custom canvas doesnt exist.

You need to configure it. If it is not configured, canvas element is is created and attached to HTML.
Works for me.
diff --git a/examples/wasm/index.html b/examples/wasm/index.html
index 33f18ca2..f10014d8 100644
--- a/examples/wasm/index.html
+++ b/examples/wasm/index.html
@@ -21,7 +21,7 @@
     </style>
   </head>
   <script type="module">
-    import init from './target/headless_wasm.js'
+    import init from './target/winit_wasm.js'
     init()
   </script>
 </html>
diff --git a/examples/wasm/winit_wasm.rs b/examples/wasm/winit_wasm.rs
index 7759d141..c3ab5568 100644
--- a/examples/wasm/winit_wasm.rs
+++ b/examples/wasm/winit_wasm.rs
@@ -16,6 +16,7 @@ fn main() {
         .add_resource(WindowDescriptor {
             width: 300,
             height: 300,
+            canvas: Some("#nonexistant".to_string()),
             ..Default::default()
         })
         .add_default_plugins()

gives:


netbox: 3.3.0 -> 3.3.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for netbox is: "IP address management (IPAM) and data center infrastructure management (DCIM) tool"
meta.homepage for netbox is: "https://github.com/netbox-community/netbox"
meta.changelog for netbox is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.3.2 with grep in /nix/store/0ycciik1i795kiwcp3si0yy5gfma4zbw-netbox-3.3.2
found 3.3.2 in filename of file in /nix/store/0ycciik1i795kiwcp3si0yy5gfma4zbw-netbox-3.3.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
netbox

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/0ycciik1i795kiwcp3si0yy5gfma4zbw-netbox-3.3.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A netbox https://github.com/r-ryantm/nixpkgs/archive/c908fca315b139f049e0c43d5b8a41754a941d2b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/0ycciik1i795kiwcp3si0yy5gfma4zbw-netbox-3.3.2
ls -la /nix/store/0ycciik1i795kiwcp3si0yy5gfma4zbw-netbox-3.3.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

netbox

Maintainer pings
cc @n0emis @RaitoBezarius for testing.

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

1 package built:

netbox

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

1 test built:

nixosTests.netbox


decrypt key during get

As we as storing key as encrypted mode need to decrypt it during fetch it.

Thanks for this pull request!


fix(example): Added abiFilters 'armeabi-v7a' to build.gradle

Description
As I descripted in #82 abiFilters 'armeabi-v7a is not used in the example, but is a part of the setup. So I added it to app/build.gradle.
Please check, if everything is correct!
Related Tickets
Closes #82

it should not be added to example, it can be installed on a simulator if you use Android Studio to build


Fix newline scheme labels

Mac has been basically a Unix since OS X 10.0, and Unix has always used LF.
I'm not too attached to these strings, though,
as long as Unix is listed in the LF label
and the CR label clearly refers to old Mac.
(I was going to call it "Ancient Mac" and use "etc."
instead of "*", but those might need translating.)

Hi @SamB, I slightly changed the labels in develop branch, almost the same as you did but with 'less than' symbol. It looks better to me. Thanks for reporting


[Bug]: Customers with disable status are still able to order items

What happened?
When a customer is registering at the front end, by default has its status to disabled (You can see that at the admin->customers list). This means that he can not checkout until he gets his status to Enabled according to this condition in the OrderManager Class:
public function validateCustomer($customer)
    {if (!setting('guest_order') AND (!$customer OR !$customer->is_activated))
            throw new ApplicationException(lang('igniter.cart::default.checkout.alert_customer_not_logged'));
    }

Instead it ignores the part of the code OR !$customer->is_activated that checks if the customer is enabled  and allow him to proceed to checkout!
I even tried to set the customer status to enabled and then back to disabled many times at the Back office as administrator but it still keeps ignoring it and allows always the customer to proceed to checkout and order no matter what his status is !!
Checked on TI version 3.0.7
What did you expect to happen?
Customers with their status set to disabled should not be able to order items.
Version
Other (please detail)
What browser are you seeing the problem on?
Firefox, Chrome, Safari
Relevant log output
No response

Again I'm not able to replicate this. Can you share screens showing your order settings config and your customer config so I can be sure the fields are set as explained.

ok one moment please.

Here is the user that is already registered on the site. As you can see at the screenshot below his status is disabled.

When i am logged in at the front end as this user i am still able to order even if my status is disabled!!!
According this code:
public function validateCustomer($customer)
    {if (!setting('guest_order') AND (!$customer OR !$customer->is_activated))
            throw new ApplicationException(lang('igniter.cart::default.checkout.alert_customer_not_logged'));
    }

i should not be able to order. Am i right??

If the user status is disabled I can't even log in.
And when I am logged in, if i make it disabled, I get logged out.
Struggling to replicate your issue. Have you made any changes to the core code? What theme are you using?

Ryan i just did a clean install with the 3.0.7 release and the default orange theme. The bug is still there. I can login at the front end even if the user is disabled at admin panel. What branch do you use?

Yes I think its due to the is_approved column not having a value or default.
It’s one for @sampoyigi to decide how to handle.

On 29 Aug 2021, at 20:28, Designer @.***> wrote:

Ryan i just did a clean install with the 3.0.7 release and the default orange theme. The bug is still there. I can login at the front end even if the user is disabled at admin panel. What branch do you use?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

If the user status is disabled I can't even log in.
And when I am logged in, if i make it disabled, I get logged out.

How did you fixed that ??

@greatdesign can you test the referenced commits to see if the issue has been fixed.

Fixed


[Event Request] Codeunit 333 Req. Wksh.-Make Order – InitPurchOrderLine

Could we please have a publisher OnBeforePurchOrderLineValidateLineDiscount in function InitPurchOrderLine 333 Req. Wksh.-Make Order?
procedure InitPurchOrderLine(var PurchOrderLine: Record "Purchase Line"; PurchOrderHeader: Record "Purchase Header"; RequisitionLine: Record "Requisition Line")
begin
   with RequisitionLine do begin
      PurchOrderLine.Init();
      PurchOrderLine.BlockDynamicTracking(true);

    ...
      PurchOrderLine."Prod. Order Line No." := "Prod. Order Line No.";
      PurchOrderLine.Validate(Quantity, Quantity);
      if PurchOrderHeader."Prices Including VAT" then
         PurchOrderLine.Validate("Direct Unit Cost", "Direct Unit Cost" * (1 + PurchOrderLine."VAT %" / 100))
      else
         PurchOrderLine.Validate("Direct Unit Cost", "Direct Unit Cost");
      // >>>>>>
      OnBeforePurchOrderLineValidateLineDiscount(PurchOrderLine, PurchOrderHeader, RequisitionLine);
      // <<<<<<
      PurchOrderLine.Validate("Line Discount %", "Line Discount %");
      PurchOrderLine."Vendor Item No." := "Vendor Item No.";
      PurchOrderLine.Description := Description;
      PurchOrderLine."Description 2" := "Description 2";
    ...

end;

[IntegrationEvent(false /*IncludeSender*/, false /*GlobalVarAccess*/)]
local procedure OnBeforePurchOrderLineValidateLineDiscount(var PurchOrderLine: Record "Purchase Line"; PurchOrderHeader: Record "Purchase Header"; RequisitionLine: Record "Requisition Line")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Upgrade to Quarkus 3.0

Quarkus migration tips: https://github.com/quarkiverse/quarkiverse/wiki/Migrating-to-Quarkus-3.x

Resolved - Thank you @gsmet for taking care of it!
Release 1.0.0 build was triggered
Resolved by merging https://github.com/quarkiverse/quarkus-opensearch/pull/52


feat: onRetry lifecycle callback

Summary

Adds a new option onRetry that allows that gives client access to information about retry attempts

Is a callback with params (response, retryTimeoutsIndex)

Adds tests to validate when onRetry should or shouldn't be called

Checklist

[x] Does your PR title have the correct title format?
Does your PR have a breaking change?:

A few small items, but overall, looks good

Looks like there are some issues with the tests @jooohhn . Once those are all passing, I'll approve


Eve Smoke - Anyone have one to help?

I've been able to decode "most" of the recipe to have a smoke sensor be presented as an Eve Smoke in the EveHome app, however have been unable to work out the required signatures etc
Does anyone have access to an Eve Smoke to help with some debugging and capturing of characteristic data??

Is there a way to help on a windows system? The main page only says something about a mac, which I don't have.

I believe you need a Mac as need to install Xcode to compile an app. Unless someone knows how you could get HMCatalog app distributed via TestFlight on iOS...

found this project. might help if you wanted to try
https://github.com/Supereg/hap-proxy


Support fuzzing without sanitizers

There are use cases for fuzzing binaries with cargo-fuzz without sanitizers:

Fuzzing entirely safe code to test for panics
Building initial corpus to feed it to more resource-intensive fuzzers later on

However, this is not currently possible with cargo-fuzz: passing no -s flag defaults to address sanitizer, passing "" has no effect, passing "none" says there's no such sanitizer.
For now I'm forced to use leak sanitizer, which seems to be the least resource-hungry of the bunch, but it still introduces a performance penalty. It would be nice to be able to disable sanitizers altogether.

We need to build with sanitizers to get the hooks that make libfuzzer work
On Sat, Jun 23, 2018, 4:15 PM Shnatsel<EMAIL_ADDRESS>wrote:

There are use cases for fuzzing binaries with cargo-fuzz without
sanitizers:

Fuzzing entirely safe code to test for panics
Building initial corpus to feed it to more resource-intensive
fuzzers later on

However, this is not currently possible with cargo-fuzz: passing no -s
flag defaults to address sanitizer, passing "" has no effect, passing
"none" says there's no such sanitizer.
For now I'm forced to use leak sanitizer, which seems to be the least
resource-hungry of the bunch, but it still introduces a performance
penalty. It would be nice to be able to disable sanitizers altogether.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rust-fuzz/cargo-fuzz/issues/158, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABivSNsmVLHTyKsrr_FRfpxbbLvV1X0Cks5t_swYgaJpZM4U09g-
.

According to libfuzzer documentation you only need -fsanitize=fuzzer but not e.g. -fsanitize=fuzzer,address
Or is this something specific to Rust?

For the record, I can say that it's not related to Rust because honggfuzz-rs works well with and without sanitizer.

No, I think that sanitize option did not exist or was not recommended in the libfuzzer docs when I wrote this crate. Feel free to make it the default.

I think changing the default should result in a major version bump, since it would effectively break existing deployments e.g. on CI that expect Address Sanitizer to be the default.

-fsanitize=fuzzer is only available on clang, not rustc. libFuzzer requires sanitizers in order to work; when using -fsanitize=fuzzer with clang, I would imagine that it does sanitization as well as linking the fuzzer and creating the fuzz harness.
libFuzzer, however, doesn't seem to support our approach anymore:
-fsanitize-coverage=trace-pc-guard is no longer supported by libFuzzer.
Please either migrate to a compiler that supports -fsanitize=fuzzer
or use an older version of libFuzzer

For C/C++, -fsanitize=fuzzer works fine w/o any sanitizers.
However there are some libFuzzer features that will kick in only when coupled with -fsanitize=address,
e.g. interceptors for memcmp.
I have not followed the Rust support for libFuzzer.

This is supported in latest release:
    -s, --sanitizer <sanitizer>    Use a specific sanitizer [default: address]  [possible values: address, leak, memory, thread, none]

Closing.


Use new location of manifests

This is a part of my effort to move manifest files to a saner location within the scripts repository.

Moving to flatcar-website repo.

https://github.com/flatcar/flatcar-website/pull/282


Issues with 2.2.1

I updated to 2.2.1 for both ios and Android and I think the component is missing something.
First of all, the AirshipBindings dll is not put into the list of references. I can add the dll manually  but NotificationActionButton is not available.

2.2.2 fixes the issue


Ability to update password when using Windows Authentication

Hi.
I changed my password for my windows login yesterday and after that the Windows authitication didn't work anymore (I know it's because docker uses a local user with the same username and password) however it would be nice if we could update the password for the user in the container and maintaining the Windows Authentication.

If you open the PowerShell prompt of the container and run this:
$credential = Get-Credential
Set-LocalUser -Name $credential.UserName -Password $credential.Password
That should do the trick

Thank you Freddy for the speedy reply.


The inference speed on Jetson TX2

Hello, @sacmehta
I run the ESPNet on  jetson TX2 and the JetPack SDK verson is  4.1.1, pytorch version is 4.0.
I find that when the input image is 360x640, the inference time is about 0.112s which means FPS is less than 10.  (I am sure without image loading and image writing time.)
In your paper, the inference Speed is more than 16 when the image is 360x640.  Can you give me more details about it?
Beside, I use the erf_net code to measure the inference time of  ESPNet, https://github.com/Eromera/erfnet_pytorch/blob/master/eval/eval_forwardTime.py

Could you please ensure that CUDA and CUDNN installed properly?
P.S.: we used PyTorch v0.3

Hi, do your meet this problem when test the ENet and ERFNet? I can not fix the bug,
RuntimeError: cuda runtime error (7) : too many resources requested for launch at /pytorch/torch/lib/THCUNN/im2col.h:120

No, I didn’t encounter this issue.
PS: I encountered issues with bilinear interpolation on TX2, so youmight wantto use deconvolution for upsampling.


Accessing UI within a VPC without public API

is it possible for UI to be deployed in private subnet and use ec2 to access it? I deployed the UI and I know it uses the public api as a proxy to login to UI. I am trying to avoid using the public API and have it completely isolated from the internet.

Hi @nikobellic3, thanks for reaching out!
PCUI deployment in isolated subnets is not supported as of version 2023.10.0.
Such feature will be probably included in the next release, but we do not have a date for it yet.

I have it deployed there and the UI keeps crashing after I login. It refers me to the something wrong pop-up message. How can I fix this issue?

+1
My AWS account is controlled by my company and for security reasons they don't allow to have instances with public IPs so I ImageBuilder cannot create the EC2 instance required to create images.
Note that my corporate cloud support team has denied the request to enable "auto-assign public IPv4 address" for security reasons and have already allocated the full range of subnets for my VPC, so I cannot even create a temporary subnet to allow ImageBuilder to complete successfully when I deploy the PCUI Cloudformation template.
So I really need a template that allows to deploy with only private IPs (we are required to use a VPN to interact with the AWS account).

I have the same problem and can't have PCUI on a public IP. Is there any estimate for this? Thanks!

Hi all,
thank you for your interest in ParallelCluster UI!
The above discussion surfaces two different, but similar, use cases:

make ImageBuilder use a custom Subnet to build PCUI image: to do this, you need to set the PCUI stack parameters ImageBuilderVpcId and ImageBuilderSubnetId. This feature is available since PCUI 2023.02.
deploy PCUI in a private networking: to do this, you need to set the PCUI stack parameter VpcEndpointId to a VPC Endpoint of type Interface for the service com.amazonaws.REGION.execute-api and the parameter LambdaSubnetIds to the subnets where the VPCE is deployed. This feature is available since PCUI 2023.12.0.

I hope this helps.

Closing this issue as both use cases are covered


[Intents] Handling question marks

Hi guys !
I'm running into a small problem today, with the question mark '?' character.
I noticed that when I fit my model's intent with a sentence composed of a question mark (at the end of the sentence), when I input the sentence with and without the question mark, I get different results and important probability score diff.
When trained with question mark :
{ input: 'es-tu réel',
  slots: [],
  intent: { intentName: 'whoAreYouFR', probability: 0.46640502699361525 } }

{ input: 'es-tu réel ?',
  slots: [],
  intent: { intentName: 'whoAreYouFR', probability: 1 } }

When trained without question mark :
{ input: 'es-tu réel',
  slots: [],
  intent: { intentName: 'whoAreYouFR', probability: 1 } }

{ input: 'es-tu réél ?',
  slots: [],
  intent: { intentName: 'whoAreYouFR', probability: 0.40673316662106906 } }

When trained with both sentences 'es-tu réel' and 'es-tu réel ?' :
{ input: 'es-tu réel ?',
  slots: [],
  intent: { intentName: 'whoAreYouFR', probability: 1 } }

{ input: 'es-tu réel',
  slots: [],
  intent: { intentName: 'whoAreYouFR', probability: 1 } }

Do you have any idea why it's doing this ? Is it normal ?
Thanks in advance ! :)

Hey @lucasvercelot ,
This is actually an expected behavior, even though it can be confusing. The SnipsNLUEngine contains two intent parsers that are called successively, the second one being called only when the first one doesn't find any match.
The first parser, that we called the DeterministicIntentParser, is based on pattern matching (regular expressions) and ensures a perfect accuracy on the training data. This parser will match with a probability of 1.0 any input which has been provided in the training data, and even small variations around it.
When the input hasn't been seen at training time, this parser will fail to find a match and then the second parser, the ProbabilisticIntentParser, will be called. This one is based on machine learning (logistic regression and Conditional Random Fields) and is able to extract intents and slots from inputs that can be very different from what was provided at train time.
In your example with the ?, all the cases where you get a classification probability of 1.0 correspond to inputs that were parsed by the DeterministicIntentParser because they were in the training data.
I guess the confusion comes from the probability gap for sentences that are very close. You can disable the first parser by removing it from the NLUEngineConfig used when instantiating the SnipsNLUEngine, so you'll have consistent probabilities. However, this might degrade a bit the parsing accuracy.
To have a better understanding of the whole pipeline, you can have a look at the blogpost we published when we open sourced the library.
I hope this helps :)

Hi @adrienball ,
I already read this article, which is great btw, a few months ago, and I completely forgot that the NLUEngine has two parsers, my bad ! :)
I'll try to disable the first parser and see what I got !

Ok, so I tried without the DeterministicIntentParser and now the probability scores between the sentence with ? and the one without it are much closer, but much lower aswell :
{"input": "es-tu réel ?", "slots": [], "intent": {"intentName": "whoAreYouFR", "probability": 0.5505541897507736}}
{"input": "es-tu réel", "slots": [], "intent": {"intentName": "whoAreYouFR", "probability": 0.38014287318577317}}
I was expecting a higher score.
Now, for some intents, trained with words of short sentences, it doesn't event get an intent at all.
I wonder if this is a normal behavior, or if I should change the way I fill the intents' sentences.
I don't know if it's best to create entities if I want to train some intents with juste words or small groups of words (2-3-4 words sentences). I noticed that event for bigger sentences the probability score is pretty low, lower than when both parsers are enabled.

The probability of theProbabilisticIntentParser is not affected by the fact that the other parser is enabled or not.
The feedback we get about probabilities is that they are usually lower than what people expect. For now there is not much we can do about it if we stick with the current model (logistic regression) which performs quite well in terms of accuracy. In theory, you shouldn't have to worry about the absolute probabilities, as the intent which is returned is supposed to be the most likely one. However, comparing probabilities among the top intents could be useful but for now only the most likely intent is returned. See the thread about this: https://github.com/snipsco/snips-nlu/issues/623.
You shouldn't have to use entities if you don't need to, but that can be faster indeed to generate entity values with the intent vocabulary (if you know it exhaustively), instead of generating multiple sentences trying to cover all the vocabulary. So I guess that's something to try.
Intent classification relies on characteristic words so you can also try to add more sentences and have them contain some domain specific words.


Not issue just a Question on the Philips data

In the Espalexa.h i had tried to switch out some of the data referring to philips, namely the "Royal Philips Electronics" bit on line 226, however, changing this to seemingly anything breaks device and it can no longer be found by Alexa.
Is there any work around for this?

Old question actually. The answer is no.


Adding performance measurement infrastructure.

Fix #170
Added measurements for getting and parsing the UnityLoader and the scene
load.

merging now, thanks for all your help, @delapuente. I'll be updating the meta issue: #165.


Add support proper syntax highlighting and support for indented sass(NOT scss)

VSCode Version: 1.6.1
OS Version:Win 10

Steps to Reproduce:

the intellisense is not working for indented sass originally
then i used a marketplace extension sass(by Robin Bentley) which gives some syntax highlighting but not all.

first one is for css
second is for indented sass
you can see the diffrence

There are extensions for indented SASS. Please try these and help improving there.


Feature Request Persistent Check Sorting

Hi,
On the client page, it would be great if checks could be sorted by name or if the UI could remember our sorting preferences. If you have a lot of checks its a pain having to Ctrl+F for them or looking through a long list.

@AirCombat thank you for submitting this issue! We're tagging this for inclusion in the 0.15 release as a new feature. The client detail view currently displays checks in order of check status, in descending order (with no secondary sort condition). We will change this to sort by status, then check name.

It seems that we have few duplicated issues for this feature, I'll close this one in favor of https://github.com/sensu/uchiwa/issues/156 so it will be easier to keep everyone informed about the progress.


Add maven artifact

Release a maven artifact on a plublic repo.

Done on OSS plateform.
https://oss.sonatype.org/content/repositories/releases/com/github/jsr203hadoop/jsr203hadoop/1.0.0/


fix(adapter-planetscale): Use instanceof for checking for Client instance

Prevoulsly, we tried to check for constructor name, which theoretically
should've been more reliable and work with several copies of
@planetscale/database. However, name gets mangled during minification
and check fails. Since minification is pretty common usecase (unlike
having multiple copies of the @planetscale/database), reverting back
to instanceof check.

Note: found via https://github.com/prisma/ecosystem-tests/pull/4226

Heya, this has caused some issues (#21956) for me :/

Thanks! Let's continue the discussion in https://github.com/prisma/prisma/issues/21956
We definitely want to fix this.


Automate releases

The release process would be

run npm run release locally, which:

bumps the version in package.json
commits and pushes the change
creates the release via the GitHub API

GitHub Actions is triggered by the release, and publishes the package to the npm registry (https://docs.github.com/en/actions/publishing-packages/publishing-nodejs-packages#publishing-packages-to-the-npm-registry)

https://github.com/changesets/changesets

Publication of packages to the npm registry is now automated (https://github.com/cerbos/cerbos-sdk-javascript/pull/852), but version bumps and changelogs are still manual.


Configurable id and text properties

I've used select2 for quite some time now and have always found the need to either manipulate data or format selections/results so that the correct properties are selected and displayed to be a continuous annoyance. The library itself is excellent, but simple things like this act as a deterrent. Is there an easy way to set these properties in the latest release? I know the docs aren't finished being updated just yet.

While we're waiting on the docs to be updated (just realized I never moved the code snippets over), you can find the code snippets for re-mapping the id and text properties here:
https://select2.github.io/announcements-4.0.html#changed-id
That also explains why we removed the ability to configure the id and text properties in Select2 4.0.0.

Duplicate of https://github.com/select2/select2/issues/4086.


New Logo Proposal

Good day, I am a graphics designer and i have a new logo design for your project. Please reply on this issue if you want to see it.

Hey, thanks! I would love to see it, please email me at<EMAIL_ADDRESS>


Add a new constructor for MemoryPackUnionAttribute by using FullName,Assembly info of a type in addition to typeof(Type)

Suggested new constructor:  [MemoryPackUnion(0, "MyNamespace.MyType, MyAssemblyName")]
this is useful when the implementing type is not in the same assembly with the Interface, which is usually isolated from its implementation and MemoryPackUnionAttribute may not possibly make a reference to it by typeof() operator.
this is a similar measure in:
[Editor("MyNamespace.ByteArrayEditor, MyAssemblyName", "System.Drawing.Design.UITypeEditor, System.Windows.Forms")]

It is undesirable to apply dependencies from the inside to the outside with strings.
#17 proposal would solve the problem, and I think it would be better to implement it.

[MemoryPackFormatter] is not a bad idea! late-binding union info is more practical and flexible.
just waiting for new NuGet package release with this update, and my project can't work without it.


$util.clipboard

Using same syntax as $util.share (items array), with confirmation field for text translation.
"action": {
                        "type": "$util.clipboard",
                        "options": {
                          "items": [
                            {
                              "type": "text",
                              "text": "{{text}}",
                              "confirmation": "copied"
                            }
                          ]
                        }
                      }

I didn't check if it's aligned with iOS version.

Hmmm, "copy to clipboard" is present in share chooser action on my android 5.1.1.
We may not need this. If we keep it, we'd need a way to highlight/flash what was copied like a "select all" because toast on android is really too small.

@j2l sorry about not merging this in, I actually tried it out and was going to, but then found that it didn't support images or videos, and I didn't want to overwhelm you with "we need video and image support otherwise this can't be merged in!" so was going to think about how to proceed with this, and then forgot about it. I apologize!

ok


How to get private key?

https://github.com/breadwallet/breadwallet-ios/blob/47c17e4ddf12c3db6176e7b1649e31257e454f6b/breadwallet/src/BRCore.swift#L156
In the code I have seen Work-In-Progress for fetching private key. Is it In progress?
or Is it completed?
Please let me know can I use this library code for following:

Crating and fetching private key?
Can I use the library code for Singapore based wallet app?
Can I buy bitcoin/ethereum in Singapore.
Can I use for transaction creation for cryptocurrency?

We do not currently have plans for exporting your private key. Tools exist to convert the BIP39 phrase to a private key if you require this.


SQL

Do you know of an example where this boilerplate is used with mysql or other sql?

Yes, you can use MySQL or others. Check out the TypeORM documentation for more details.

Yes, you can use MySQL or others. Check out the TypeORM documentation for more details.


File download in JupyterLab does not work if jupyter-fs is installed

Describe the bug
If jupyter-fs is installed, in Lab 'File' > 'Download' or right-click on file in standard Lab file browser > 'Download' do not work (nothing happens). Right-click in standard file browser > 'Copy Download Link', then pasting link into browser address bar yields '503 service unavailble'. The whole raw file request mechanism via /files API endpoint stops working.
To Reproduce
Steps to reproduce the behavior:

Install JupyterLab and jupyter-fs
Right-click some file in Lab's file browser. Then 'Download'.

Expected behavior
Webbrowser starts download (asks where to save the file or saves it to standard location).
Desktop (please complete the following information):

OS: Debian 12
jupyter-fs 0.3.1 with minor bugfix in is_hidden (main already has this fix)
JupyterLab 4.0.4

Log output
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]: [E 2023-10-06 20:45:33.958 ServerApp] Uncaught exception
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:     Traceback (most recent call last):
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:       File "/opt/conda/envs/jhub/lib/python3.11/site-packages/tornado/http1connection.py", line 276, in _read_m>
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:         delegate.finish()
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:       File "/opt/conda/envs/jhub/lib/python3.11/site-packages/tornado/httpserver.py", line 399, in finish
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:         self.delegate.finish()
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:       File "/opt/conda/envs/jhub/lib/python3.11/site-packages/tornado/routing.py", line 268, in finish
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:         self.delegate.finish()
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:       File "/opt/conda/envs/jhub/lib/python3.11/site-packages/tornado/web.py", line 2395, in finish
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:         self.execute()
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:       File "/opt/conda/envs/jhub/lib/python3.11/site-packages/tornado/web.py", line 2417, in execute
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:         self.handler = self.handler_class(
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:                        ^^^^^^^^^^^^^^^^^^^
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:       File "/opt/conda/envs/jhub/lib/python3.11/site-packages/tornado/web.py", line 238, in __init__
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:         self.initialize(**kwargs)  # type: ignore
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:         ^^^^^^^^^^^^^^^^^^^^^^^^^
Oct 06 20:45:33 2ba3e1fbebad jupyterhub-singleuser[98500]:     TypeError: StaticFileHandler.initialize() missing 1 required positional argument: 'path'
Oct 06 20:45:33 2ba3e1fbebad jupyterhub[93466]: 20:45:33.965 [ConfigProxy] error: 503 GET /whz-mathe-test/user/u21302575108/files/test.ipynb socket hang up
Oct 06 20:45:33 2ba3e1fbebad jupyterhub[93460]: [D 2023-10-06 20:45:33.968 JupyterHub pages:584] No template for 503
Oct 06 20:45:33 2ba3e1fbebad jupyterhub[93460]: [I 2023-10-06 20:45:33.970 JupyterHub log:191] 200 GET /whz-mathe-test/hub/error/503?url=%2Fwhz-mathe-test%2Fuser%2Fu21302>

Solution
In metamanager.py the MetaManager class has to set files_handler_params = {'path': self.root_dir} member variable, which defaults to {} making Jupyter Server's FilesHandler fail (cf. Jupyter Server's standard contents manager FileContentsManager).

Have to correct my initial solution: add
    @property
    def files_handler_params(self):
        return {'path': self.root_dir}

to MetaManager class in metamanger.py. Else loading the jupyter-fs extension will fail (but initial solution works if code is modified after extension has been loaded!?).


[WFCORE-3168] warning when setting or removing secret value for alias

https://issues.jboss.org/browse/WFCORE-3168
https://issues.jboss.org/browse/JBEAP-12642

I'll just fix that here.


Adding functionality to the xDSCWebService resource (might be a 'BREAKING CHANGE')

Hi there,
I would like to discuss with you the following request, as it might end up as a 'BREAKING CHANGE':
I'm currently working on the possibility to have an on premises HTTPS DSC Pull Server being generated by making use of an Azure DSC server.
To make it more clear: I generate an on premises VM which gets its configuration from an Azure DSC server. Purpose is to make this new VM an on premises HTTPS DSC Pull Server, totally automated.
And this last two words are the catch 22, since you currently have to supply xDSCWebService with the valid thumbprint of the SSL certificate you want to be used. But you don't now the thumbprint yet, as it is not being generated at that point in time. The certificate will be requested using the xCertReq resource within the same config file.
So my proposal is to add the value of CertificateGeneratedInSameConfig to the list of possible values for Thumbprint within the xDSCWebService resource (besides an actual Thumbprint and AllowUnencryptedTraffic).
Furthermore it should be possible to supply the value of Subject to the resource when CertificateGeneratedInSameConfig is used. Since this is the key value of the xCertReq resource, based on this value the determined.
This all can be accomplished by adding a small amount of code to the xDSC (aprox. 7 lines).
Please let me know what you think of it.
Kind regards,
Sven

Hi Sven,
I've seen your question on powershell.org as well but had no time this week to pitch in.
This sounds like you're trying to augment the purpose of xDSCWebService to play nicely with a very specific scenario and overall that's not what DSC resources are in general. It seems you're more trying to answer the problem DSC has with lack of orchestration tools, and the fact DSC isn't working well with logical evaluation in runtime (only at design time), although bare in mind DSC does support WaitFor and similar so you can chain various DSC resources to act in some sort of workflow and you can use the Script DSC resource to run additional steps before and after the DSC configuration. Remember DSC resources need to be used for immutable state, not necessarily orchestrating.
The scenario you're describing shouldn't be hard to accomplish without any changes to xDSCWebService, simply on the Azure side. Now, I'm not an Azure automation expert but I would look at
options within Azure automation first.
Overall outside of DSC (this is the orchestration part):
You generate a certificate from the on premise CA, give the thumbnail to Azure DSC to create a MOF file for the on premise DSC pullserver, at the same time transfer the certificate to the on premise DSC pull server and install the certificate.
Now that stage can be done by a job, a scheduled task, a running script, even an Azure function (you can run PowerShell scripts in it), as it has nothing to do with DSC.
Only then you send the mof file to the on premise DSC pull server and enact it.
This is exactly the same if you would have wanted a Azure DSC to create an on premise IIS webserver
and wanted to use HTTPS, thus would have needed a certificate installed before hand and using its thumbnail or subject
Just my 0.5cent :)

I'm running into a similar issue with a few resources, I'm writing a configuration that's using xCertReq to request it's certificate in the same configuration as I'm using that certificate but most resources require the Thumbprint.
Obviously I'd rather not have two configs or any manual intervention in this process so the solution I'm currently using is to add CertificateSubject as a property on the DSC resource, within the resource I then check if CertificateSubject has a value and if it does I grab the Thumbprint from Cert:\ and set CertificateThumbprint property to have that value. If I do this near the top of the Set and Test then it lets the resource continue to work as normal but still provides the functionality I require. I'm doing a bit more testing on it before I start doing PRs into various resources but I'd be happy to add xDSCWebService to that list of things to look at.
I think a similar approach could be taken here, the only additional property needed is CertificateSubject and then the resource itself handles that.

@svenvanrijen Did you get your question answered? How did it work out for you?

I'm closing this as not fixed due to lack of response. Please reopen or comment if this was closed in error.


Fix clippy warnings

Codecov Report

Merging #9 (9c0175e) into main (4a94c50) will decrease coverage by 13.04%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main       #9       +/-   ##
===========================================
- Coverage   95.65%   82.60%   -13.05%
===========================================
  Files           3        3
  Lines          23       23
===========================================
- Hits           22       19        -3
- Misses          1        4        +3

Flag
Coverage Δ

unittests
82.60% <0.00%> (-13.05%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

python/src/lib.rs
60.00% <0.00%> (-30.00%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a94c50...9c0175e. Read the comment docs.


Docs website url

Hi there, the docs url on the renderlet.com homepage seems to be invalid.   Could you please suggest how to access, thanks!

Hi, will get this fixed this week. The docs are not currently live, will push to gh-pages

sounds good, looking forward to it. i've been trying out various examples from the folder in the meantime and modifying for my use case. thanks for all the great work so far!


Simple image cropping examples and tutorial in React

This PR is regarding the issue #546.
Work done,
Simple image cropping examples and tutorial in React

Sorry for the delay. Reviewing it now

no problem christy take your time

Very well written @spiritbro1
I would suggest you also run a linter to format all the code and push one more commit and then we're good to go

@christyjacob4 how bout this one? https://github.com/appwrite/demos-for-react/pull/4#discussion_r502900032

@christyjacob4 how bout this one? #4 (comment)

Yea I understood now. I have left a reply

@spiritbro1 I would suggest you also run a linter to format all the code and push one more commit and then we're good to go

@christyjacob4 linter done please check again

Great Job! 🥳

thanks
On Sun, Oct 11, 2020, 10:35 PM Christy Jacob<EMAIL_ADDRESS>wrote:

Great Job! 🥳
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/appwrite/demos-for-react/pull/4#issuecomment-706722809,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO5B4AN4KSBDWM4WWKWE3YLSKHGEHANCNFSM4R4P6VCA
.

Hey @eldadfux feel free to merge it. Looks good from my side


DynTree Worlgen Mod Compatibility

I have seen this behaviour in combination with multiple Dungeon mods now,
the one i am using right now is Pillar: https://github.com/Vazkii/Pillar
but this isn't an issue only with them.
Mods cannot generate their Structures correctly with DynTrees installed often generating them on top of the trees like the leaves were solid blocks.
It may be the leaves or the trunk but all of the mods i had this occur with, do not have this problem with vanilla trees.
How to recreate:
Install DynTrees+Pillar
Create an Structure using StructureBlocks
Import to pillar using /pillar-copy
Crank spawn rates in /pillar folder to make it easier to find
Most if not all of the structures will generate on top of the trees in densly grown areas.
I had this issue occur with the mod recurrent complex as well, but there it is much harder to replicate.
It may be something about in which order worldgen works in forge/mc but i really don't know.
Is DynTrees doing things differently than vanilla?

Structure mods like this have a hard time dealing with Dynamic Trees because if they break the branch blocks they get messed up trees, if they just generate around them they get messed up structures.  It's a no-win situation regardless.  The trees in Dynamic Trees generate as an IWorldGenerator with a modGenerationWeight of 20.  But regardless of order this will still happen because of chunk edges.


GatherScatterModel bug?

Hi,
Thanks for adding the new AML typologies!
In GatherScatterModel.java line 91 reads:
sendTransaction(step, minAmount, orig, bene, isSar, alertID);
Is this a bug? Should minAmount be replaced with scatterAmount?
In any case line 85 always seems to result in scatterAmount being set to minAmount, so there is never an event when the scatter account gets the 10% margin. Is this the right behavior?
Thanks

Thank you for pointing out the bug. Now I fixed it (minAmount -> scatterAmount).
We add several AML typologies, but these are preliminary so we are working on updating documents.


Update third-party BUILD files for NVIDIA Jetson (TF 1.14)

Why? The NVIDIA dev community is struggling hard to compile TensorFlow: https://devtalk.nvidia.com/default/topic/1055131/jetson-agx-xavier/building-tensorflow-1-13-on-jetson-xavier/post/5354734/
Basically the same PR for 1.14 as the already approved https://github.com/tensorflow/tensorflow/pull/26985 for 2.0 beta except that I also had to slightly modify third_party/gif.BUILD. Let me know if you think this change can cause trouble. Changing the compiler config might also work, but I've already spent 7 days compiling TF in various configurations and would like to focus on something else now... THANK YOU :)
Maybe you can release this with v1.14.1 soon?
More information about the Jetson (Nano) in case haven't heard of it: https://developer.nvidia.com/embedded/jetson-nano-developer-kit

I've signed the CLA. Maybe it needs some time to update the status on GitHub?

We've published our libtensorflow 1.14.0 binaries for download: https://dl.photoprism.org/tensorflow/
From what I understood, S_IREAD is deprecated in favor of S_IRUSR. I couldn't find a compiler or setting that would have compiled libgif without defining S_IREAD, S_IWRITE etc and it's the only library that needs it: https://github.com/photoprism/tensorflow/commit/31104dcca0912288b28e045b7d2c8158061d6ddf
If somebody can tell us how to compile libgif without those changes using GCC 4.8 / 4.9 (as recommended by the TF docs), we can remove them again. Otherwise and/or if there is no negative impact, I would remove the conditions and always define them (see referenced commit in our fork):
    defines = [
        "S_IREAD=S_IRUSR",
        "S_IWRITE=S_IWUSR",
        "S_IEXEC=S_IXUSR"
    ],

This will get into 1.14.1, which will be released soon. We don't yet have a timeline, but I wanted to update the PR.


wip(ssr): fix export package for gen runtime helpers

Good catch! I implemented an easier fix in 8f6b669 so that we don't need to worry about calling with the correct arguments in transforms.


Nodes are bounced back when they are dragged around

Description
Nodes are bounced back when they are dragged around.
Also after the change eventually succeeds, sometimes the destination is not the desired location where the node is dragged to.
Steps to Reproduce

Simulate a slow network
Drag any nodes on a map

Expected Behaviour
Nodes should be following the cursor.
Relevant Logs / Console output
N/A
Your Environment

NR1 CLI version used: Observability maps v0.10.0
Browser name and version: Chrome, latest
Operating System and version: macOS big sur

Additional context
In older versions of observability maps, the position change happens right after mouse left button is released, however, at the moment, it does not happen until the network communication finishes, which creates a weird situation that the nodes are bouncing back constantly.
What makes it un-usable is that now users can not drag and drop smoothly if their network is slow. They are forced to wait until each change finishes, to get a view where next node should be dragged to, and is big time waster.

Believe it is always a problem to fight against a slow network, but what if the status of the whole map could be stored in local, and finally sent back to server when users choose to end editing? It might help solve the conflict between real-time editing and slow network communication.
Anyway, just food for thought.

@aswanson-nr When I was originally implementing it I had some issues with the behaviour of the graph package going between two objects and things going out of sync one of which was also related to slow networks. So I intentionally removed it some time ago. So it is per design currently.
I do want to re-implement the feature to improve usability. I'll see if I can put something together this week to go out with another catalog update otherwise we'll have to schedule it for later.


Add optional enable attribute to XML generator classes

Description:

It would be helpful to support an optional enabled= attribute to the xml gen class node. If specified, the value would be either internal or debug. If "internal" was set, then the entire generator would be ignored if INTERNAL_TESTING was not set. If "debug" was set, then the entire generator would be ignored if _DEBUG was not set.
This would be primarily useful for interfaces, though it could also be useful for experimental generators such as "DocViewApp".

I'm just going to go with "internal" for now and skip "debug". Presumably the classes or interfaces are experimental, so the only time they should not be shown is a non-internal release build. We shouldn't need the additional granularity of a debug-only class or interface.


Provider\Internet::domainWord() should produce lowercase results

Using $faker->domainName(), I noticed the results were capitalized. companyEmail() is also affected, as it uses domainName().
It is because the underlying domainWord() uses Provider\Person::lastName() (with localization and transliteration), but keeping the initial capital.
I think the expected behaviour is undoubtedly to have lowercase results.

This happens only when intl extension missing. See code of Provider\Internet::transliterate(), and its latest changes.

Capital letters are valid in domain names and emails - even if ignored by browsers and mail clients. I wonder what's best for Faker: generate expected output, or unexpected output? I think the latter...

I think the contrary, as it is very uncommon to encounter capital letters.
If you want to cover more cases, you might consider:

hyphens, which are allowed unless leading/trailing
IDN

Fixed by #725

I just tripped over this. Good to see that it is already fixed.
@fzaninotto is there any chance we see a new release (bug fix/patch or minor, I don't care) which contains this fix any time soon?
The last release, v1.5.0, was released quite some time ago and people could make use the those 192 commits integrated since. I'd be happy to help, if I can.

@ravage84 not in 2015. If you want to help, well there is a lot to do:

review pending PRs
generate changelog
write a blog post about the new stuff
????
profit!


Limited fonts available in arduino api for coreink

I'd like to have some more fonts than the standard 2 fonts in the api.

Ascii8x16.cpp
Ascii24x48.cpp

Can someone add some bold fonts to the library?

You might try contacting this person who provided the word molds
https://github.com/lovyan03/LovyanGFX/blob/19a2e711e70a5a072d5463be6c09c06204a35835/src/lgfx/Fonts/Ascii24x48.h#L4


[ECS] Unexpected token "//", expected '/**' at offset 0

Using ECS version 6.1.0
Symplify\CodingStandard\Fixer\Property\ArrayPropertyDefaultValueFixer

In FixerFileProcessor.php line 158:

  [Symplify\EasyCodingStandard\FixerRunner\Exception\Application\FixerFailedException]
  Fixing of "XXX.php" file by "Sympli
  fy\CodingStandard\Fixer\Property\ArrayPropertyDefaultValueFixer" failed: Un
  expected token "//", expected '/**' at offset 0 in file /var/www/project/ve
  ndor/phpstan/phpdoc-parser/src/Parser/TokenIterator.php on line 183

Exception trace:
  at /var/www/project/vendor/symplify/easy-coding-standard/packages/FixerRunner/src/Application/FixerFileProcessor.php:158
 Symplify\EasyCodingStandard\FixerRunner\Application\FixerFileProcessor->processTokensByFixer() at /var/www/project/vendor/symplify/easy-coding-standard/packages/FixerRunner/src/Application/FixerFileProcessor.php:106
 Symplify\EasyCodingStandard\FixerRunner\Application\FixerFileProcessor->processFile() at /var/www/project/vendor/symplify/easy-coding-standard/src/Application/SingleFileProcessor.php:63
 Symplify\EasyCodingStandard\Application\SingleFileProcessor->processFileInfo() at /var/www/project/vendor/symplify/easy-coding-standard/src/Application/EasyCodingStandardApplication.php:139
 Symplify\EasyCodingStandard\Application\EasyCodingStandardApplication->Symplify\EasyCodingStandard\Application\{closure}() at /var/www/project/vendor/symplify/easy-coding-standard/src/Application/EasyCodingStandardApplication.php:166
 Symplify\EasyCodingStandard\Application\EasyCodingStandardApplication->processFileInfoWithCallable() at /var/www/project/vendor/symplify/easy-coding-standard/src/Application/EasyCodingStandardApplication.php:140
 Symplify\EasyCodingStandard\Application\EasyCodingStandardApplication->processFoundFiles() at /var/www/project/vendor/symplify/easy-coding-standard/src/Application/EasyCodingStandardApplication.php:96
 Symplify\EasyCodingStandard\Application\EasyCodingStandardApplication->run() at /var/www/project/vendor/symplify/easy-coding-standard/src/Console/Command/CheckCommand.php:88
 Symplify\EasyCodingStandard\Console\Command\CheckCommand->execute() at /var/www/project/vendor/symfony/symfony/src/Symfony/Component/Console/Command/Command.php:255
 Symfony\Component\Console\Command\Command->run() at /var/www/project/vendor/symfony/symfony/src/Symfony/Component/Console/Application.php:987
 Symfony\Component\Console\Application->doRunCommand() at /var/www/project/vendor/symplify/package-builder/src/Console/HelpfulApplicationTrait.php:28
 Symplify\EasyCodingStandard\Console\EasyCodingStandardConsoleApplication->doRunCommandAndShowHelpOnArgumentError() at /var/www/project/vendor/symplify/package-builder/src/Console/HelpfulApplicationTrait.php:19
 Symplify\EasyCodingStandard\Console\EasyCodingStandardConsoleApplication->doRunCommand() at /var/www/project/vendor/symfony/symfony/src/Symfony/Component/Console/Application.php:255
 Symfony\Component\Console\Application->doRun() at /var/www/project/vendor/symplify/easy-coding-standard/src/Console/EasyCodingStandardConsoleApplication.php:54
 Symplify\EasyCodingStandard\Console\EasyCodingStandardConsoleApplication->doRun() at /var/www/project/vendor/symfony/symfony/src/Symfony/Component/Console/Application.php:148
 Symfony\Component\Console\Application->run() at /var/www/project/vendor/symplify/easy-coding-standard/bin/ecs:80

In TokenIterator.php line 183:

  [PHPStan\PhpDocParser\Parser\ParserException]
  Unexpected token "//", expected '/**' at offset 0

Exception trace:
  at /var/www/project/vendor/phpstan/phpdoc-parser/src/Parser/TokenIterator.php:183
 PHPStan\PhpDocParser\Parser\TokenIterator->throwError() at /var/www/project/vendor/phpstan/phpdoc-parser/src/Parser/TokenIterator.php:80
 PHPStan\PhpDocParser\Parser\TokenIterator->consumeTokenType() at /var/www/project/vendor/phpstan/phpdoc-parser/src/Parser/PhpDocParser.php:33
 PHPStan\PhpDocParser\Parser\PhpDocParser->parse() at /var/www/project/vendor/symplify/coding-standard/packages/TokenRunner/src/DocBlockParser.php:33
 Symplify\CodingStandard\TokenRunner\DocBlockParser->parse() at /var/www/project/vendor/symplify/coding-standard/packages/TokenRunner/src/DocBlock/DocBlockManipulator.php:122
 Symplify\CodingStandard\TokenRunner\DocBlock\DocBlockManipulator->resolvePhpDocNodeIfFound() at /var/www/project/vendor/symplify/coding-standard/packages/TokenRunner/src/DocBlock/DocBlockManipulator.php:81
 Symplify\CodingStandard\TokenRunner\DocBlock\DocBlockManipulator->resolveVarTagsIfFound() at /var/www/project/vendor/symplify/coding-standard/packages/TokenRunner/src/DocBlock/DocBlockManipulator.php:91
 Symplify\CodingStandard\TokenRunner\DocBlock\DocBlockManipulator->resolveVarTagIfFound() at /var/www/project/vendor/symplify/coding-standard/packages/TokenRunner/src/DocBlock/DocBlockManipulator.php:35
 Symplify\CodingStandard\TokenRunner\DocBlock\DocBlockManipulator->isArrayProperty() at /var/www/project/vendor/symplify/coding-standard/src/Fixer/Property/ArrayPropertyDefaultValueFixer.php:77
 Symplify\CodingStandard\Fixer\Property\ArrayPropertyDefaultValueFixer->fixProperties() at /var/www/project/vendor/symplify/coding-standard/src/Fixer/Property/ArrayPropertyDefaultValueFixer.php:65
 Symplify\CodingStandard\Fixer\Property\ArrayPropertyDefaultValueFixer->fix() at /var/www/project/vendor/symplify/easy-coding-standard/packages/FixerRunner/src/Application/FixerFileProcessor.php:156
 Symplify\EasyCodingStandard\FixerRunner\Application\FixerFileProcessor->processTokensByFixer() at /var/www/project/vendor/symplify/easy-coding-standard/packages/FixerRunner/src/Application/FixerFileProcessor.php:106
 Symplify\EasyCodingStandard\FixerRunner\Application\FixerFileProcessor->processFile() at /var/www/project/vendor/symplify/easy-coding-standard/src/Application/SingleFileProcessor.php:63
 Symplify\EasyCodingStandard\Application\SingleFileProcessor->processFileInfo() at /var/www/project/vendor/symplify/easy-coding-standard/src/Application/EasyCodingStandardApplication.php:139
 Symplify\EasyCodingStandard\Application\EasyCodingStandardApplication->Symplify\EasyCodingStandard\Application\{closure}() at /var/www/project/vendor/symplify/easy-coding-standard/src/Application/EasyCodingStandardApplication.php:166
 Symplify\EasyCodingStandard\Application\EasyCodingStandardApplication->processFileInfoWithCallable() at /var/www/project/vendor/symplify/easy-coding-standard/src/Application/EasyCodingStandardApplication.php:140
 Symplify\EasyCodingStandard\Application\EasyCodingStandardApplication->processFoundFiles() at /var/www/project/vendor/symplify/easy-coding-standard/src/Application/EasyCodingStandardApplication.php:96
 Symplify\EasyCodingStandard\Application\EasyCodingStandardApplication->run() at /var/www/project/vendor/symplify/easy-coding-standard/src/Console/Command/CheckCommand.php:88
 Symplify\EasyCodingStandard\Console\Command\CheckCommand->execute() at /var/www/project/vendor/symfony/symfony/src/Symfony/Component/Console/Command/Command.php:255
 Symfony\Component\Console\Command\Command->run() at /var/www/project/vendor/symfony/symfony/src/Symfony/Component/Console/Application.php:987
 Symfony\Component\Console\Application->doRunCommand() at /var/www/project/vendor/symplify/package-builder/src/Console/HelpfulApplicationTrait.php:28
 Symplify\EasyCodingStandard\Console\EasyCodingStandardConsoleApplication->doRunCommandAndShowHelpOnArgumentError() at /var/www/project/vendor/symplify/package-builder/src/Console/HelpfulApplicationTrait.php:19
 Symplify\EasyCodingStandard\Console\EasyCodingStandardConsoleApplication->doRunCommand() at /var/www/project/vendor/symfony/symfony/src/Symfony/Component/Console/Application.php:255
 Symfony\Component\Console\Application->doRun() at /var/www/project/vendor/symplify/easy-coding-standard/src/Console/EasyCodingStandardConsoleApplication.php:54
 Symplify\EasyCodingStandard\Console\EasyCodingStandardConsoleApplication->doRun() at /var/www/project/vendor/symfony/symfony/src/Symfony/Component/Console/Application.php:148
 Symfony\Component\Console\Application->run() at /var/www/project/vendor/symplify/easy-coding-standard/bin/ecs:80

After changing only this line in the class:
private const MY_CONST = 12; // a comment

to:
private const MY_CONST = 12; /** a comment */

Makes the error disappear and ECS runs smoothly.
Also running phpstan alone on same code doesn't make the error happen.

I recommend moving all such comments above the node with Slevomat cosing standard. If it's fixed  here, it will fail later somewhere else. Technicaly they belong to next node, php-parser will add it to wrong place.
The code should be fixed instead.
/**
  * Comment
  "/
public const VALUE = 123;

But ECS should not throw an expeption on this, right? It should parse the file without problems as the syntax is correct?

It depends. It will break something else eventually, so better fix your code.
If you find a way to fix it, I'd be happy to merge it of course. I don't thing it's worth investing time, as comment is wrong for some doc parsers.

There is nothing to be fixed in my code though, it is correct.

I'm aware of like 5 phpdocs parsers. Each treats doc nodes differently, PSR was really needed.
That's why I look forward to PHP 8.0, where this will be solved by attributes:  https://wiki.php.net/rfc/attributes_v2

And a simple way to "fix" it would be catching this "exception" and just skipping the line that can't be properly parsed, so the rest of fixers can go forward.


Fix slow import when window is unfocused

Fixes #93877
Effectively, the importation speed was dependent on the VSync and Update continuously settings. That was because there is a lot of progress popup update while importing scenes. Disabling VSync and Update continuously settings while importing files accelerated the importation process on my computer and I got the same speed now when the editor is focused or not. On my computer, it accelerated the importation of the MRP project from 132 sec. to 88 sec. even when focused.
At the same time, I made a couple of little tweaks in the progress popups to prevent redrawing at each step progress passing p_force_redraw to false to step method.
There was also a bug in ProgressDialog when more then one step was present. If the second task was updated quickly or with p_force_redraw at true, the first step was never updated if the first task was using p_force_redraw at false. That was because last_progress_tick was global and always updated only for the second task. When task_step was called for the first task, last_progress_tick was always up to date causing it to return and not update the step.

Yes, I think it's because of Main::iteration(). When VSync is enabled, the DisplayServer waits before rendering the frame (I think) and with low_processor_usage_mode, OS::add_frame_delay adds low_processor_usage_mode_sleep_usec each frame.

Thanks for testing. I'm glad that you are able to get the same performance has the "optional settings".
For the difference in the time for the importation, I took a quick look at all the modifications between https://github.com/godotengine/godot/commit/92c8e87cd987e8d55d176608c3038e9c4646cb2c and https://github.com/godotengine/godot/commit/4d984b63697d840f2c823608d2c46740fb44651a
https://github.com/godotengine/godot/compare/92c8e87..4d984b6
The only think I see that could have affected the importation is that commit which was suppose to optimize the load/saving of scenes: https://github.com/godotengine/godot/commit/b83c64faacd317bd375b90b5f24cfd62cf72d96c (#93723)
I don't known, maybe it reloads all the meshes to load scene groups?!? I could look into it a bit later.

I don't known, maybe it reloads all the meshes to load scene groups?!? I could look into it a bit later.

After testing 4.3.beta2, 4.3.dev2 and 4.2.2, I can't reproduce my time of 12 seconds with any of their official builds. I'm not sure how I managed to get that time in the first place. If anything, the latest master branch is a bit faster than 4.2.2.

That's a good news!!! I'll not investigate further then. Thanks!

Needs rebase, then it should be good to merge.

Rebased done! I retested with the MRP project from the issue. Performance optimization was as expected.

Thanks!


Fixed optional type imports in core

Context: we want to be able to refer to "optional types" such as pdfmake, nodejs Buffer, etc.  We want MedplumClient to work seamlessly in browser environments and nodejs environments.
To do this, we have found 3 requirements:

Use peerDependenciesMeta to flag dependencies as optional
Use import type rather than import to indicate that you only want the types
Add a magic comment of // @ts-ignore before import type statements that are optional

2 additional changes are required:

Removing /// <reference types="node" />.  Apparently any reference to Buffer automatically includes /// <reference types="node" />.  We worked around this by using Uint8Array rather than Buffer, which then removes /// <reference types="node" />

Preserving the // @ts-ignore comments by using /** @ts-ignore */ (h/t https://github.com/microsoft/TypeScript/issues/29866#issuecomment-715140696 )

Coverage increased (+0.004%) to 95.125% when pulling deafb497c4023b849e8a2c6237d87101a8606c76 on cody-fix-optional-type-imports into bae3032a960b9f7b8cfdf5c42463af0345134f1e on main.


Improve readme

Adding Development instructions to README.md

Confirmed the instructions worked with eclipse dsl tools 2021-12.


GraphQL API for fetching data

Create a graphql api file so the frontend can fetch data from the backend
methods:
fetchCongressman()
fetchBills(year or congressional term)
- This should join congressman onto bills

not doing a backend

not doing a backend


Unpin numpy

Acados was updated in https://github.com/commaai/openpilot/commit/4c1b8b71d096e7658fe56598944e3f8bd3e347f0, numpy does not need to be pinned anymore.

Note for future: Acados does not need to be recompiled for that, the only thing from there which relies on numpy is acados_template/acados_ocp_solver_pyx.pyx, which is recompiled every time solver is generated