IP address plugin for PowerToys Run

Summary of the Pull Request
This pull requests adds a new community-made plugin for PowerToys Run that allows you to get your public IPv4/IPv6 address by typing "ip" in the launcher.

PR Checklist

[x] Closes: #26665
[x] Communication: A collaborator said it's A-OK to make a PR.
[ ] Tests: Added/updated and all pass
[x] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments
This feature uses the icanhazip.com website (this can be changed if required) to get the user's IP address. It allows the user to get both their IPv6 and IPv4 address at the same time. If the user doesn't have IPv6, it will just tell them that they do not have an IPv6 address (this is done using a try/catch). The user can also press enter to copy the selected result to the clipboard, whether that is their IPv6 or IPv4.

Validation Steps Performed
My home network has on-and-off IPv6 support which decided to not work the day I was working on this plugin, so I tested IPv6 support by connecting to my mobile phone using hotspot. There is no automated testing done here (I am not even sure if that is possible, but then again, I have never written a test).

@microsoft-github-policy-service agree

Suggestion: Can we switch the symbol to a globe?

Suggestion: Can we switch the symbol to a globe?

Sure, why not.

Can't we just get the ip adress locally. We don't need a third-party website for this. Or am I missing something?

It might be, but I can't find any solution for that and NAT is a thing etc

Can't we just get the ip adress locally. We don't need a third-party website for this. Or am I missing something?

Not the global one of your router given by your isp.

Oh I see. For this to work safely we would have to create some sort of API (through for example Azure Functions) that then returns the users IP adress. I'm not sure if this is worth the effort.

Can't we just get the ip adress locally. We don't need a third-party website for this. Or am I missing something?

Not the global one of your router given by your isp.

Oh I see. For this to work safely we would have to create some sort of API (through for example Azure Functions) that then returns the users IP adress. I'm not sure if this is worth the effort.

I got a response from a maintainer saying that the team is not interested in maintaining a service for this plugin

Apart from maintenance issues, I believe that privacy concerns are more important.
Therefore, in order to apply this function, it will be necessary to provide a flow of agree for privacy.
However, as there has been no such case in the PowerToys feature to date, it seems difficult to persuade them even if everything is met.
If there is a similar case for external requests in PowerToys, it will help persuade them.

In case this is implemented, Microsoft should have its own server instead of using icanhazip.

Some improvement suggestions.

why depending on the website http://ipv4.icanhazip.com ? there are ways to get the string without using 3rd party websites. with  NetworkInterface.GetAllNetworkInterfaces() you can check and find the one that's online and is of type InterNetwork family.

Some improvement suggestions.

why depending on the website http://ipv4.icanhazip.com ? there are ways to get the string without using 3rd party websites. with  NetworkInterface.GetAllNetworkInterfaces() you can check and find the one that's online and is of type InterNetwork family.

Because your method Only returns the internal ip of the adapter which is connected to the internet. And this plugin should show your external ip given by your isp.

Some improvement suggestions.

why depending on the website http://ipv4.icanhazip.com ? there are ways to get the string without using 3rd party websites. with  NetworkInterface.GetAllNetworkInterfaces() you can check and find the one that's online and is of type InterNetwork family.

Because your method Only returns the internal ip of the adapter which is connected to the internet. And this plugin should show your external ip given by your isp.

I see. Well I'm still pretty sure that data is stored somewhere locally. I'd used a javascript code to retrieve it once, and I'm hopeful C# has the feature too. I'll look into it and come back when I found it

Can this work(I am not good at these stuff so pls dont bash me with comments lol)

using System;
using System.Net;

class Program
{
    static void Main()
    {
        // Get the host name
        string hostName = Dns.GetHostName();

        // Get the IP addresses associated with the host
        IPAddress[] ipAddresses = Dns.GetHostAddresses(hostName);

        // Print the private and public IP addresses
        foreach (IPAddress ipAddress in ipAddresses)
        {
            if (IsPrivateIpAddress(ipAddress))
            {
                Console.WriteLine("Private IP: " + ipAddress.ToString());
            }
            else
            {
                Console.WriteLine("Public IP: " + ipAddress.ToString());
            }
        }
    }

    // Helper method to check if an IP address is private
    static bool IsPrivateIpAddress(IPAddress ipAddress)
    {
        byte[] ipBytes = ipAddress.GetAddressBytes();

        // Check for private IP address ranges
        if (ipBytes[0] == 10)
            return true; // Class A private network
        if (ipBytes[0] == 172 && ipBytes[1] >= 16 && ipBytes[1] <= 31)
            return true; // Class B private network
        if (ipBytes[0] == 192 && ipBytes[1] == 168)
            return true; // Class C private network

        // Additional checks for other reserved IP address ranges
        if (ipBytes[0] == 169 && ipBytes[1] == 254)
            return true; // Link-local address range
        if (ipBytes[0] == 127)
            return true; // Loopback address range

        return false;
    }
}

In this code, we added a helper method IsPrivateIpAddress that checks whether an IP address is within the private IP address ranges (e.g., <IP_ADDRESS>/8, <IP_ADDRESS>/12, <IP_ADDRESS>/16). We also added a conditional statement to differentiate between private and public IP addresses and print them accordingly.
When you run this code, it will display all the private and public IP addresses associated with your machine.
maybe it helps!! maybee

@VihaanReddyM
No this helps not. Your code only retrieves the local IPs of the network adapters and not the one that you got from your isp.

@pistasjis , @crutkas
I don't think that we can get this work without external website service.
And to not have long useless discussions here I suggest the following: If @pistasjis provide the plugin under his/her name on his/her GH profile he can use external websites to get the IP.
What do you think?

This might be the best action for this plugin.
On Thursday 13. July 2023 07:01:48 (+02:00), Heiko wrote:
@pistasjis , @crutkas
I don't think that we can get this work without external website service.
And to not have long useless discussions here I suggest the following: If @pistasjis provide the plugin under his/her name on his/her GH profile he can use external websites to get the IP.
What do you think?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

https://stackoverflow.com/questions/735350/how-to-get-a-users-client-ip-address-in-asp-net

Closing this out as it needs an active service :(


nx required?

Should I use Nx and monorepo in order to use these library?
I'am using Angular v13 with @angular-architects/module-federation 12.5.3, so following the readme I should use ngx-mfe 1.0.2

Hi, Nx monorepo isn't required

Try to update to last version, in past versions was a bug with production build, see -

Sorry, my mistake, I am using Angular v12 can I update to the last version?

Update to angular v13, angular-architect/module-federation to v14 and this lib to v2.1.0

Thank you!
Marco

From: Denis Khrunov @.>
Sent: Tuesday, July 26, 2022 11:27:16 AM
To: dkhrunov/ngx-mfe @.>
Cc: marco24690 @.>; Author @.>
Subject: Re: [dkhrunov/ngx-mfe] nx required? (Issue #13)
Update to angular v13, angular-architect/module-federation to v14 and this lib to v2.1.0
—
Reply to this email directly, view it on GitHubhttps://github.com/dkhrunov/ngx-mfe/issues/13#issuecomment-1195241237, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQJXH4JSRAAE62R2QM7URN3VV6VPJANCNFSM54VFURMQ.
You are receiving this because you authored the thread.Message ID: @.***>

Happy to help!
Denis


Assumptions fail and leaky abstractions

I use pysolr because there is nothing better yet. Sadly every time I need to do something unusual I'm stuck because pysolr makes some painful assumptions about what I wan't to do. Most of all it makes assumptions about what parts of response I'm interested in and does not give me anything more.
Currently we can access only those fields that:

are accessed as attributes of Results() class
Solr() class API methods kindly pass to Results() class as instance args/kwargs

This approach has following problems:

This isn't future-proof. Whenever something new is added to Solr, pysolr users can't access it because assumption was made ("you ain't gonna need that") and can't do nothing about that. Only solution is to add a new "feature" as a PR that lands in limbo (#140, #141, #148, #142, #55; note that some of them are years old!).

There is no separation of concerns. Results() do nothing more than storing fields. It really could be a dict. There is almost no benefit in using object here. And most of the code in methods like search() is creating a result field by field:
 if result.get('stats'):
     result_kwargs['stats'] = result['stats']

This IMHO should be a responsibility of Results() class - take a raw response an make it something more python-like. As a side note i feel that the need for altering this response is doubtful.

Cognitive burden - result attributes are different from those used in Solr (like: qtime vs. QTime). So if you are well familiar with Solr you need to remember two sets of results fields. Also if someone wants to switch to pysolr due to other features then migration of existing code can be tedious because of that.

Introducing new solr features requires new tests while it actually should not. Since Solr is quite mature project there is no need to tests things that are documented and outside of the lib scope. But since every not covered response field becomes actual Results() class and Solr() class code it becomes pysolr's feature and thus needs testing. Take for example the cursorMark proposed by #140 & #141.

Too much maintenance. Every new field needs new code. Trivial or not this creates need for PRs, reviews, merging, releases. No one likes doing that for trivial code so this is the reason of so long PR backlog piling up on GitHub.

But hey! pysolr is generally good but not "future proof" and thus needs constant care from maintainers. So much that they lost the steam and this is the reason than mentioned PR-limbo exists. This isn't a vicious rant. I have suggestion how to fix this situation in a few steps:

quick hotfix so impatient devs (like me) can still can use pysolr despite the fact it does not assumed there are features they would like to use. Solution is simple: make Results() object contain also a raw solr response dictionary. Like:

def search(self, q, **kwargs):
    params = {'q': q}
    params.update(kwargs)
    response = self._select(params)

    result = self.decoder.decode(response)
    result_kwargs = {}
    # nasty "dicting" here
    # ...
    response = result.get('response') or {}
    numFound = response.get('numFound', 0)
    # `raw_result=result` for those that we forget about
    return Results(response.get('docs', ()), numFound, raw_result=result, **result_kwargs)

code refactoring to improve separation of concerns between Client and Results class. Let client api methods like Solr.search() pass whole response json to Results() class instead of creating intermediary dict and then unpacking it into Results args and/or kwargs like:

def search(self, q, **kwargs):
    params = {'q': q}
    params.update(kwargs)
    response = self._select(params)

    return Results(self.decoder.decode(response))

pluggable results classes so you could extend existing Results() class if something is missing via subclassing without the need for fixing whole pysolr package. Or even replacing whole stuff with something different. Like:

class Solr(object):

    def __init__(self, url, decoder=None, timeout=60, results_cls=Results):
        self.decoder = decoder or json.JSONDecoder()
        self.url = url
        self.timeout = timeout
        self.log = self._get_log()
        self.session = requests.Session()
        self.session.stream = False
        self.results_cls = results_cls

    # ...

    def search(self, q, **kwargs):
        params = {'q': q}
        params.update(kwargs)
        response = self._select(params)

        return self.results_cls(self.decoder.decode(response))

With that you can even do solr = pysolr.Solr(url, response_cls=dict) and have exactly the same thing you would get using something like only requests but still benefitting from anything else Solr class provides you.

Future improvements in existing Results class. I'm sure there is a mass of people appreciating fact that you can access response fields via attributes instead of just keys (sadly, I'm not one of them) but maybe there is a way to make it more generic and future proof so there is no need to iterate over every Solr response field. Im sure there is a way to make it more auto-magical and still ensure backwards compatibility.

All those steps ensure backwards compatibility (except 4. that is like optional) while making pysolr compatible with a load of existing Solr's features like cursors, maxScore etc and more that even does not exist yet. This will in fact reduce overall amount of code and make pysolr more maintainable (lot of existing PRs could be closed).
I can handle code changes and prepare PRs for every step described here (or just jump to 3.) but I would like to be sure that someone agrees with me and wants at least to review proposed changes.

Hah, found this line that is here from the very beginning of pysolr:
# TODO: make result retrieval lazy and allow custom result objects

So I did not wait and just did it (#167)

I owe you a longer response but I'm :+1: on doing this. pysolr is a fairly old project and long-overdue for some refactoring.


Listen to NFC events on iOS

Requirements:

[ ] Listens for NFC events
[ ] Is able to determine whether an NFC card is a Satscard/Tapsigner
[ ] Has a means to enable/disable NFC listening at will

This won't work out as there will be conflicts between this plugin and the popular nfc_manager plugin. Instead the library should depend on nfc_manager to provide communication with NFC cards and the library can focus on simply being a proxy for Nunchuk's C++ tap-protocol library.
https://github.com/PeteClubSeven/cktap-protocol-flutter/issues/12
https://github.com/PeteClubSeven/cktap-protocol-flutter/issues/13


fix (a11y): Colour-blind friendly mechanism to distinguish between issue and PR states

At the moment an issue chip renders with a leading "dot in a square", coloured either green if the issue is open or red if the issue is closed.
This is problematic for a couple of reasons.

The colour scheme feels backwards (for some cultures) -- an open issue indicates a problem (bad), a closed issue indicates a problem has been resolved (good)
The red/green are not differentiable under some forms of colour blindness.

I've attached a file that demonstrates this. If you save this file, and then load it in to the simulator at https://www.color-blindness.com/coblis-color-blindness-simulator/ you'll see that for some people the colours are either very close or indistinguishable.
I'd recommend removing the use of red/green entirely to communicate the state, and use different icons to clearly distinguish between them.
Thanks.
Edit to add: PRs have a similar issue; the state of the PR (unmerged or merged) is communicated solely through colour, the icons for unmerged and merged are the same.

@nikclayton thanks for bringing this up! So for this I'm using the primer library, which is the CSS library github uses for colors: https://github.com/primer/css
It looks like I might have some mismatch on determining when to use which color. Open issues should be green, closed should be purple. This is the convention that github uses. I'll also see if I can find a good closed issue icon; github replaces the dot with a checkmark once the issue is closed.

Open issues should be green, closed should be purple

FWIW I see open PRs are green and merged PRs are purple. However, again, their state (open or merged) is only communicated through the colour of the icon, there's no colour-blind friendly alternative indication of the state.

Just noticed that the GitHub Android app uses different icons to distinguish between merged (top) and unmerged PRs, whereas the plugin just uses the bottom icon.

Aha, perhaps this is a different issue then - I'm also using the pull request icons, the single dots are for issues. Perhaps it's not detecting that correctly. Can you share a couple of the links you're using? Or links that show the same behavior.

Sure (I've updated this issue's title/body to cover PRs as well as issues as well).
At the time I've writing, if I open an Obsidian file and paste in
https://github.com/pachli/pachli-android/pull/473
https://github.com/pachli/pachli-android/pull/480

it renders as the following screenshot:

Of those, the first PR is unmerged, the second is merged. As you can see, they render with identical icons, just the colour is different.
The screenshot from the GitHub app that I included in https://github.com/nathonius/obsidian-github-link/issues/68#issuecomment-1970672178 shows what I think is the correct icon to use for merged PRs.
In the specific case of pull requests I think you might only be checking whether or not the PR is closed. You might also need to make an explicit check to see if it's been merged (https://docs.github.com/en/rest/pulls/pulls?apiVersion=2022-11-28#check-if-a-pull-request-has-been-merged) to show the correct icon.
So for PRs arguably this is at least a tri-state, and the icon + colours could indicate:

PR is open
PR is closed (and not merged)
PR is closed (and merged)

That could be extended further to indicate:

PR is open and mergeable (all reviews approve, no checks that are blocking the merge)
PR is open and not mergeable

Completely understand if that last one makes sense as a separate feature request.

Okay, so it seems like there's a number of separate issues here:

Closed as completed issues should be purple in color, not red
Icons should more clearly communicate the status of an issue or PR

Thirdly, there's a couple feature requests here:

Show mergibility of PRs
Better a11y in general

I'm going to spin this off into a few extra issues to split it up.

Simple mergeable / not mergeable PR status checks have been added.


Cannot deploy CloudFormation because of deprecated python version (3.7) in Lambda

CloudFormation errors out. Python version needs to be updated.

The runtime parameter of python3.7 is no longer supported for creating or updating AWS Lambda functions. We recommend you use the new runtime (python3.12) while creating or updating functions. (Service: Lambda, Status Code: 400, Request ID: 318fecf5-0598-4cf2-84c6-09e350369b86)" (RequestToken: 00a811ba-b0d9-9897-4df4-244f2af5fb85, HandlerErrorCode: InvalidRequest)

solved by #19


Update people.md  Edward Capriolo

according to Linkedin , Edward Capriolo is no longer worked at Hortonworks but Deutsche Bank
https://www.linkedin.com/in/edwardcapriolo/

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

Is the change a dependency upgrade?

How was this patch tested?

We don't take linkedin as source for these informations but rather let the people themselves update these thing if they want. if they don't it stays as is.
Closing!!!


Cannot read properties of undefined (reading 'createInvoice')

Hi,
I am trying to use this package in my NestJS backend with Typescript and the easyinvoice import is logging to undefined and I am getting the error "Cannot read properties of undefined (reading 'createInvoice')."
Any idea why this could be? I've tried uninstalling and reinstalling the package a few different times, but am still having the same issue. I am working with the newest version of easyInvoice: 3.0.23.
Thanks

Thanks for your message, we will have a look ASAP and try to fix it. Can you send me a bit of sample code?

I just tried to import easyinvoice in a NestJS application, that worked fine for me. Types were recognized. Please provide me with some code so I can reproduce. Also the ts.config would help.


[SPARK-42085][CONNECT][PYTHON] Make from_arrow_schema support nested types

What changes were proposed in this pull request?
Make from_arrow_schema support nested types
Why are the changes needed?
do not need to get the schema by self.schema due to from_arrow_schema not supporting some nested types
Does this PR introduce any user-facing change?
No
How was this patch tested?
existing UT

merged into master


Issue with Codeignter3 with network calls functions

Hello,
I haver tried both functions on CI2 and CI3 and found there is an issue with CI3.
On welcome controller I have added index2 method and execute it.
public function index()
{
$this->load->view('welcome_message');
}
public function index2()
{
for($j=0;$j<=10;$j++){
echo file_get_contents("https://google.com");
}
}
While the index2 is executing on CI3, I wasn't able to access any other function on the controller unless it finishes the execution, When I access index() as example the browser keep waiting until the execution of index2 finishes the it load the page.
I tried same to do the same thing on CI2 and it worked without any issue.
Could you please check what could be the issue.
Thanks

It looks like you're having an issue with your own project, rather than a CodeIgniter issue. For that, please post this on the CodeIgniter forums.

I see this has been raised on the forum. Closing the issue.


fix(UI): fixed navbar menu hover bg color

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #49774

Hey @Akshay1997a
Thank you for opening this pull request.
This is a standard message notifying you that we've reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


feat/Include drivers for Postgres and MySQL/MariaDB by default

Current docker image only includes the driver for SQLite.
I believe those of us running many instance of Hummingbot, or want a centralised db storage (for post-trade analytical purposes) would prefer logging trades in a different RDBMS such as PostgreSQL or MySQL/MariaDB. However, this means we would need to build the docker image ourselves every time a new release of Hummingbot is out, in order to work with a non-SQLite database.

To help me understand why this is useful: So your workflow would be to install the (updated) Docker image, then use the new Python modules to do what? Wouldn't you need to recompile your own Docker images anyways, if you would add Python scripts allowing you to store the data to your remote DBs?

To help me understand why this is useful: So your workflow would be to install the (updated) Docker image, then use the new Python modules to do what? Wouldn't you need to recompile your own Docker images anyways, if you would add Python scripts allowing you to store the data to your remote DBs?

No these are not for custom scripts, but to enable storing of trade info into a centralised DB vs a SQLite file (cf. https://hummingbot.org/global-configs/external-db/)
If you are running 20+ docker containers and each of them do frequent trades, SqlAlchemy's ORM via SQLite isn't exactly the most performant out there (cf. https://stackoverflow.com/questions/11769366/why-is-sqlalchemy-insert-with-sqlite-25-times-slower-than-using-sqlite3-directly), and you will end up with lots of disk thrashing.
It's also useful to have a central location to do analysis of trades instead of having to get data from individual SQLite file.

Each user should configure that in their own installation, and it should not be part of the official repository.

But it makes sense to only have dependency for SQLite built-in? 🤷♂
Joke aside, would it be better to say, have a different environment.yml file only for the purpose of building a docker image so that it can run out-of-the-box with a db driver other than SQLite? Because when running a cluster of docker containers and all of them write to different SQlite files at the same time, it really cause problems with disk thrashing (backlog disk I/O leading to stalled instance, leading to high memory consumption).

Or perhaps there is a better way to do it so that the official docker image we pull can already support other RDBMS?

@cheemeng @aarmoa Mike opened a discussion in CommonWealth for this topic. This is the link: https://commonwealth.im/hummingbot-foundation/discussion/6013-support-for-external-dbs

Closing PR, snapshot vote: failed


Fix queue bind returning an array instead of self.

Queue#bind should return self. in its current state it instead returns [self]

Coverage remained the same at 100.0% when pulling ff47b79b88826ebfec633b5da7150c0871b41e5f on fugufish:master into 09b5cb735f26c87d63664816d46247767e914b94 on arempe93:master.

Thanks for your contribution @fugufish - sorry its been so long in the queue. Work has been very time consuming lately


docs: Document manual approval steps for progressive delivery

Closes #846
What changed?
Documentation added to the site about Flagger and manual gating in Enterprise.
Why was this change made?
Often users will want to retain a manual approval mechanism so that a human operator can account for the decision to promote a new application version.
How was this change implemented?
N/A
How did you validate the change?
Asked for a review.
Release notes
Excluded.
Documentation Changes
Website.

re-run failed jobs

Error: request to https://storage.googleapis.com/upload/storage/v1/b/staging.docs.gitops.weave.works/o?name=846-document-flagger-gating%2Fassets%2Fjs%2Fff7192ae.00181232.js&uploadType=resumable failed, reason: socket hang up


Fix IBM identifier in TPM_MANUFACTURERS

Describe the issue
The TPM spec says that TPMManufacturer values in a TPM attestation should use capital letters exclusively:

The value of the TPMManufacturer attribute MUST be the ASCII representation of the
hexadecimal value of the 4 byte vendor identifier defined in the TCG Vendor ID Registry[3]. Each
byte is represented individually as a two digit unsigned hexadecimal number using the characters
0-9 and A-F. The result is concatenated together to form an 8 character name which is appended
after the lower-case ASCII characters “id:”.
For example, the vendorId 0x12 0x34 0x56 0xEF would be encoded as “id:123456EF”.

The identifier for IBM has included a lowercase "d" for a while now. I should fix that. See here:
https://github.com/MasterKale/SimpleWebAuthn/blob/dc70416e781c9ab11625ba9afbf092809391874e/packages/server/src/registration/verifications/tpm/constants.ts#L117

Frustratingly the lowercase letter is present in this other spec listing manufacturer IDs:
https://trustedcomputinggroup.org/wp-content/uploads/TCG-TPM-Vendor-ID-Registry-Family-1.2-and-2.0-Version-1.06-Revision-0.96_pub.pdf

A couple of options here:

Update the existing entry so IBM has a single entry with 'id:49424d00'
Add a second entry for IBM with 'id:49424D00' so IBM ends up with two entries
Do some bytes comparison via string manipulation on the values after id:

Option 2 is looking pretty good to me 🤔


Cannot find module 'next/dist/next-server/server/api-utils.js' from 'node_modules/next-test-api-route-handler/dist/index.js'

The problem
Using Next 11.0.1 and NTARH 2.0.1, when running tests I receive the following error:
Cannot find module 'next/dist/next-server/server/api-utils.js' from 'node_modules/next-test-api-route-handler/dist/index.js'
If I look in the next node_modules folder, the correct path is 'next/dist/server/api-utils.js'

Ah, yeah, the NextJS devs have the api-utils in four different locations across several versions 😅.  This should be accounted for in the latest version of NTARH (v2.2.1). If you are able to update, does the problem go away?

So if I update, I end up with a bunch of errors saying that ‘undefined’ should be string value or Array Buffer, or something like that. Can’t remember off the top of my head - for now, I just reverted to Next 11.0.1 and everything works fine!
Will Nixon
Software Developer
310-463-0567
willnixon.dev https://www.willnixon.dev/
Lemmi - the app that lets you chat https://www.lemmichat.com/

On Sep 1, 2021, at 4:59 PM, Bernard @.***> wrote:
Ah, yeah, the NextJS devs have the api-utils in four different locations across several versions 😅. This should be accounted for in the latest version of NTARH (v2.2.1). If you are able to update, does the problem go away?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Xunnamius/next-test-api-route-handler/issues/320#issuecomment-910423611, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH7LOCARUIEKGBRUE5UKTPLT7ZEXVANCNFSM5DGUZI2Q.

Just to be sure: I meant only update NTARH, not NextJS. The latest NTARH should work with the older NextJS v11.0.x (integration tests are passing at least).

So, updating to 2.2.1 gives me this error:

      at ServerResponse.apiRes.end (node_modules/next/server/api-utils.ts:77:33)
      at onerror (node_modules/next-connect/dist/index.cjs:4:47)
      at node_modules/next-connect/dist/index.cjs:16:16
      at apiResolver (node_modules/next/dist/server/api-utils.js:101:9)```

Interesting, this seems like the NextJS resolver function itself might have a problem with your handler, since NTARH doesn't call res.end (only your handler does).
First, does your handler code run in NextJS normally without errors?
Second, can you show me the code around your usage of the testApiHandler function and the handler you're testing? Or better yet, if you can spin up a tiny dummy repo that reproduces your error, I can pinpoint the problem 🙂


hyperlink to header brings header to bottom

instead of top. which is weird.
https://d2lang.com/tour/grid-diagrams/#cells-expand-to-fill

yeah idk about this one. its fine locally for me. Might just have to rebuild and redeploy


Notification and Pre/Post pipeline similar to MediaR

Is there a notification and pre/post pipeline similar to MediaR? If so can you please share an example.

Unfortunately not yet, I'm working on it.

@girish66 I have implemented the Notification pattern for now without Pre/Post pipeline

Thank you will try it out.


initdb should write the full configuration

Currently, we are only writing a partial configuration during the initdb job. We should proceed to write the full configuration.

related to #1373 ?


Support multiple client connections over Websocket

Looked at the WebSocketAudioSource class to realise that the server will only support one client at a time. Also leads to this weird (but expected, i guess) behaviour - when two clients connect sequentially, the later one has response offsets that carry over from the first one.
I can see a way to add multi-client support - happy to make a PR - but wondering if this choice is intentional? And if i am missing something?
New here, and really appreciate the detail and thought that has gone into this work already!

Hi @juanmc2005, i have opened the PR. Sorry for the delay, look forward to working on this!


logged in user on front end

add code to see which user is logged in and display their name to nav bar or header (example:  "hello dave")

This was implemented in src/components/Navbar.js


Removed FreeCAD environments

Removed FreeCAD docker environments for testing & deployment.

cqparts-env:ubuntu-py2
cqparts-env:ubuntu-py3

To avoid confusion, take unnecessary load of Travis, and just cleaning.

Not sure why Appveyor isn't showing up anymore.
At the start of this PR, Appveyor was responding to the webhook with:
{"message":"PR and project repositories do not match."}

I've since changed the project repo within Appveyor from fragmuffin/cqparts to cqparts/cqparts, but still nothing.
Perhaps Appveyor needs to be working from the start of a PR... anyway, it's not critical, just annoying.


Generics

So I guess this came up before, but anyways :) I want to be able to inject generic implementations and right now Magnet hinders me to do that because of some class name validation issue:
const val RATINGS_STORAGE = "ratings-storage"

data class RatingsModel(...)

interface ModelStore<T> {
    fun load(): T?
    fun save(data: T)
}

@Instance(type = ModelStore::class, classifier = RATINGS_STORAGE)
fun provideRatingsModelStore(): ModelStore<RatingsModel> {
    return /* create the store for the RatingsModel */
    }
}

@Instance(type = RatingsManager::class)
class RatingsManager internal constructor(
    @Classifier(RATINGS_STORAGE) private val store: ModelStore<RatingsData>
) {
   ...
}

This fails with
error: Method must return instance of path.to.ModelStore as declared by interface magnet.Instance
    public static final de.aoksystems.ma.abp.modelstore.ModelStore<RatingsData> provideRatingsModelStore()

Now I thought maybe I could outsmart the validation and use
fun provideRatingsModelStore(): ModelStore<*> { ... }

but this didn't work either. And naturally, Kotlin doesn't let me use the plain ModelStore type without any generic arguments, as I could do with Java:
public class StaticProvision {
    @Instance(type = ModelStore.class, classifier = RATINGS_STORAGE)
    static ModelStore provideRatingsModelStore() {
        return /* create the store for the RatingsModel */;
    }
}

This version of course compiles just fine.
What are my options here (beside starting to write Java code again :))?

That was quick, thank you! 🙏

You are welcome. The fix is available in 3.1-beta1.


[stable/ghost] Can't evaluate field port in type interface

What this PR does / why we need it:
Fix default installation that is broken since https://github.com/helm/charts/pull/6999.
Current error:
Error: render error in "ghost/templates/deployment.yaml": template: ghost/templates/deployment.yaml:51:56: executing "ghost/templates/deployment.yaml" at <.Values.externalData...>: can't evaluate field port in type interface {}

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

/ok-to-test
/lgtm

/lgtm

/retest

/lgtm cancel

/lgtm

Sorry about that! I'm new here... :)


Adding custom insert embed code plugin

I tried to replicate upload plugin and willing to insert embed code (just  code snippet).
When I try to use insertImage method in my own embed plugin, it will create an image and putting my data to image src. But can't find where this image is creating. If i find it, I will change it for embed plugin.
I just want to put code snippet which is written on editorModal to editor, just like insertImage.
Where should I check in files?
My plugin snippet:
embed: {
    func: function(params, tbw) {
        var embed, pfx = tbw.o.prefix;
        var $modal = tbw.openModalInsert(
            tbw.lang.embed,
            {
                embed: {
                    label: 'embed',
                    required: true
                }
            },
            function(){
                tbw.execCmd('insertEmbed', $('[name="embed"]').val());
                setTimeout(function(){
                    tbw.closeModal();
                }, 250);
            }
        );
    },
    ico: 'insertEmbed'
}

hi!
did you managed to figure this out?
Trying to do the same :(

I need to improve the documentation.
Maybe this branch can help you: https://github.com/Alex-D/Trumbowyg/pull/189


⚠ DDoS-Relay lin-EU-C-fra-02 has degraded performance

In 38bf302, DDoS-Relay lin-EU-C-fra-02 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 312 ms

Resolved: DDoS-Relay lin-EU-C-fra-02 performance has improved in bff329f after 12 hours, 23 minutes.


The expected value is a boolean, not a string.

A New PR
THANK YOU - We love to get PR's and really appreciate your time and help to improve this module
Accepting a PR
https://github.com/sqlcollaborative/dbachecks/wiki
Please confirm you have 0 failing Pester Tests
[X] There are 0 failing Pester tests
Invoke-Pester -show Failed,Describe,Summary .\tests\Unit.Tests.ps1 is clean:
Tests Passed: 2133, Failed: 0, Skipped: 0, Pending: 0, Inconclusive: 0
Changes this PR brings
Tiny, one-line bug fix. The original line compares $allServerInfo.PowerPlan to a string 'true', but $allServerInfo.PowerPlan needs to be compared to a boolean true. If you dig deeper, you will see that this gets set in Sever.Assertions.ps1, line 66:
$PowerPlan = (Test-DbaPowerPlan -ComputerName $ComputerName blahblahblah).IsBestPractice
It's a little weird that the 'PowerPlan' in the Server.Assertions.ps1 sets $PowerPlan to a string in the catch{} blocks when Test-DbaPowerPlan fails (for whatever reason) but $PowerPlan is a boolean when Test-DbaPowerPlan succeeds. I think that is OK, for our purposes here.
It's also a little weird that DbaChecks basically says "Your Power Plan is/is not "High Performance"", while I'm not sure that Test-DbaPowerPlan "IsBestPractice" is necessarily looking for plan named "High Performance" . IIRC, some some AMD-specific "go fast as possible" Power Plans are named "High Performance (AMD)" or something along those lines. If you look into the Test-DbaPowerPlan code, it actually is looking for a specific GUID as the 'best plan'-maybe different languages call that plan something besides "High Performance"? ("Alto Rendimiento"?) IOW, we want to use the "IsBestPractice" attribute that Test-DbaPowerPlan provides and not compare (English) strings.
I think that covers everything.
-d

Thank you
Expected value is only a boolean if we set errors to be a boolean also
We were setting them to be a string hence the requirement for a string
However, changes in underlying code means that I cannot parse the result easily so I have changed the values to fit with your change.


ImportError: umap.plot requires matplotlib, datashader, colorcet and holoviews to be installed

Install umap using below commands:
pip install umap-learn
pip install "umap-learn[plot]"

and they run successfully.
but when I import umap.plot, it occurs:

But actually, these packages matplotlib, datashader, colorcet and holoviews have been installed.
so what I should do to fix this error?
Big thanks!

I had the same issue and had to install the imgcat package as well.

Recently, ran into this above didn't help.

Error of mine goes from 'nansum_missing' from 'datashader.utils' ... digging into...

Sadly same problem here

Hi!
It gave me the same error. So I manually imported each of these (matplotlib, datashader, colorcet and holoviews), to see if they were all fine. Turns out there was an error while importing holoviews.
So i did pip install holoviews. And turns out it got updated to the recent version. I think umap didn't install the required latest package. Hope this helps. :)
Also, small note for just in case: I did fit_transform and was wondering why it was throwing error. I remembered I had to only fit() as use that object.
Here's what it looked like:

Hi!
A suggestion, not solution.
I got this error in my jupyter notebook as well. The problem was with importing holoviews. But since, thats not your issue:
This is the code that from which the error is being raised. You could try executing this code and see if you could find any error. It may not solve your problem, but give it a shot.
`    import pandas as pd
import datashader as ds
import datashader.transfer_functions as tf
import datashader.bundling as bd
import matplotlib.pyplot as plt
import colorcet
import matplotlib.colors
import matplotlib.cm
import bokeh.plotting as bpl
import bokeh.transform as btr
import holoviews as hv
import holoviews.operation.datashader as hd`

This was a great suggestion for me - I found my datashader needed numba != 0.50.1 to use CUDA, which umap.plot wants, so a quick numba update solved the error.

@pramitchoudhary Also, a great suggestion for me! Thanks

pip install umap-learn[plot] solves the issue

I still have this problem with numba version 0.51.2 & umap-learn  0.5.0

I still have this problem with numba version 0.51.2 & umap-learn  0.5.0


WIP: Fuzz wasmtime

Fuzzing @Vurich 's wasmtime. Experimental run.

Why is that a PR in the Substrate repo?

no reason, just an experiment


Add RSS Feed

It's been requested 👍

@jlord -- maybe not what you were intending, but I've created a little go app that:
a) runs on a Heroku free dyno
b) pulls down a CSV every 30 minutes
c) converts it to an RSS feed
d) responds with the RSS feed to https://federal-gov-rss.herokuapp.com/rss
Not the best example of Go code, but it's here.


1.19.2?

Is this planned at some point?

I am working on this right now :)

I lied, I am now working on this! Check the 2.x branch.

This has been released as version 2.0.0 @Aces-and-Jacks!
https://modrinth.com/mod/compact-chat/version/2.0.0


PageSpy icon invalid on click (cannot reappear stably)

Describe the bug
https://github.com/HuolalaTech/page-spy-web/assets/5585835/c9a30069-1df0-42ed-aee5-b4ceb96dae59
Steps to reproduce
No response
System Info
System:
    OS: macOS 12.7.1
    CPU: (8) x64 Intel(R) Core(TM) i5-1038NG7 CPU @ 2.00GHz
    Memory: 268.21 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.16.1 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 9.5.1 - /usr/local/bin/npm

Logs
No response
Validations

[X] Read the docs.
[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

The issue (see https://github.com/HuolalaTech/page-spy/pull/20) has been confirmed, and will be fixed in the next release.
Thank you!


exempt 3 call method chaining

fix #36

@simonihmig any objections to merging this?

@simonihmig Keine Ursache! I had forgotten it until now 😄


Settings shortcut shows name instead of symbol

VSCode Version: 1.13.0 - insiders
OS Version: Windows 10

-- Sorry for the English, I used Google Translator --
Currently:
Ctrl + Comma
How should I show (in my opinion)
Ctrl + ,
It happens if I set a new shortcut instead of the default.

@bpasero May I know who generate the labels for keybindings?

@sandy081 this is done by Electron.

Thanks @bpasero


Fix RFC5424 timestamp output

This fixes issue #3, "RFC5424 output seems to format zone offset incorrectly"
Adding the second 'Z' to the timestamp format outputs the offset with the colon (see date format docs here). I tested this with syslog-ng as the receiving syslog and it now happily accepts RFC5424 messages sent to it via logstash-output-syslog where before it complained and wouldn't parse them.

@daph can you rebase please?

@suyograo Rebase done.

@daph can you sign the CLA? https://github.com/elasticsearch/logstash/blob/master/CONTRIBUTING.md#contribution-steps

@suyograo I could have sworn I already did, but oh well. I just went and signed it all now.

@daph make sure to sign the CLA with the same email as the one you use for commit. I see a skytap email here, but I think you signed with a different one.

@daph if it helps, you can also create a brand new PR with the new email.

@suyograo Woops, sorry about that. Didn't realize that's the email I have for the commit. Is it fine if I just resign the CLA again, but with the skytap email?

CLA check ok. LGTM. manually merging

@daph thanks for your patience and contribution to Logstash!


Add files via upload

https://www.openpolytechnic.ac.nz/
https://www.openpolytechnic.ac.nz/qualifications-and-courses/op7091-bachelor-of-information-technology/

@odysseynz Pull request merged. Thank you!


How can I reuse an already acquired token via interactiveBrowserCredentialOptions for Microsoft Graph using Azure.Identity

Query/Question
I am using the below code to acquire tokens interactively by signing users in.
https://github.com/microsoftgraph/msgraph-sdk-dotnet/blob/dev/docs/upgrade-to-v4.md#example-using-tokencredential-class
The problem is, once the application is stopped, I lose the logged in user and will have to sign in the user again. Is there a way to save and use the token that is obtained in the first sign in? I see some TokenPersistenceCache options but there is no sample of how to use it other than this below,
https://docs.microsoft.com/en-us/dotnet/api/azure.identity.tokencachepersistenceoptions?view=azure-dotnet
I tried to use this part of the code snippet
//Call AuthenticateAsync to fetch a new AuthenticationRecord.
    authRecord = await credential.AuthenticateAsync();

which is different from the suggested code for Microsoft.Graph which is below
User me = await graphClient.Me.Request()
                .GetAsync();

But it fails with a "InteractiveBrowserCredential authentication failed: AADSTS650057: Invalid resource." error.
I don't want the user to be asked for credentials every time they start the app if the user has already logged in once. How can I achieve this for MS Grpah with Azure.Identity? Could you help please?
Environment:

Azure.Identity 1.5
Microsoft.Graph(Tried both 4.3 and 4.7)
OS and .NET runtime version ( Windows 10 .NET Framework 4.7.2)
IDE and version : Visual Studio 16.6.5

Hello, @schaabs , @christothes Just want to check again on this-what would be a neat way to logout the persisted user? How can I change the user who is logged in other than by deleting the saved Authentication Record and allowing a new Authentication Record to be created. Is there a better way to do that? Because the MSAL Cache is still persisted regardless of deleting the AuthenticationRecord and MSAL holds the sensitive info per the statement here.

The AuthenticationRecord which is returned from the Authenticate and AuthenticateAsync, contains data identifying an authenticated account. It is needed to identify the appropriate entry in the persisted token cache to silently authenticate on subsequent executions. There is no sensitive data in the AuthenticationRecord so it can be persisted in a non-protected state.

Probably the safest way would be to delete the persisted cache and start over. Although MSAL does persist the cache, the TokenCachePersistenceOptions have a Name property that uniquely identifies the cache. So changing that name to a new unique name effectively makes it a new cache without any history.
https://github.com/Azure/azure-sdk-for-net/blob/6a2213dd974db0b934c7d62c309b416c371adbcd/sdk/identity/Azure.Identity/src/TokenCachePersistenceOptions.cs#L62

Ok, Thanks for your help on this. I will close this query now.


CI: Migrate from Travis CI to GitHub Actions

Because the billing plan of Travis CI has been changed.
https://docs.travis-ci.com/user/billing-overview/
https://blog.travis-ci.com/2021-05-07-orgshutdown
In addition, GitHub Actions has a high affinity to GitHub.

Hmm, it seems that running GitHub Actions is blocked in this repository, because it's my first contribution on this.
@tmm1 Could you grant to run it?

What do I need to grant and where?

Hmm, don't you see the button here?
https://docs.github.com/en/actions/managing-workflow-runs/approving-workflow-runs-from-public-forks
Now I merge to try trigger it.

Triggered: https://github.com/tmm1/http_parser.rb/actions
Sorry,  I misunderstood it.


Child columns without gutters in gutters grid

Hi,
If you create a gutter grid with inside a nested grid.
The columns of the nested grid will have gutters to.
It'd be great to make this nested grid no guttered, adding something like a "no-gutter" class to it.
What do you think ?
Anyway, Thanks for building concise.

@atlza
Thank you so much for pointing this out! I never thought about nested grids unintentionally inheriting the gutter styling. I have fixed this in my latest commit and it will be bundled with the next patch release.
If you want to nest a grid with gutters and you don't want the nested grid to have gutters, just add the .gutters-off class.
Best,
Keenan


Basicnat

Sorry about this, the PR was meant for Igalia's repo.

No problem :).
You are welcome to mirror your branch on the SnabbCo repo if/when you want to promote it for other people to check out. Just send a PR to add it to src/doc/branches.md.


Add a simple test

Since a lot of the logic is in the flag parsing and return code
generation, it seems worth including it in the test, which requires a
bit of a refactor.
I'm afraid I don't know enough about GitHub actions to guess if this will run as part of ci.yml, but let me know if I should add go test -v . there or if it's unnecessary.

I've updated the PR with a test for -rpc-header. Sorry for the breakage in the other PR!


Slide looses "index" prop value

Describe the bug
Slide looses "index" prop value after adding or removing slides.
To Reproduce
https://codesandbox.io/s/644pkk7y0z
After changing the number os slides, the slides loose their index, making the is-active and is-current classes not being properly set.
Basically, this setter stops working (https://github.com/baianat/hooper/blob/master/src/Hooper.vue#L285), leaving the index with the default value of 0
Expected behavior
It would be expected for the slider to get the current index so that the classes is-active and is-current get correctly applied

I've added a quick fix using slide element attributes, but didn't made a pull request as it's a really lousy fix. I'm sure there's a better way to change props on the child component.
https://github.com/filipepina/hooper/

Thank you for reporting, we will fix it very soon.


AZ GROUP CREATE NOT WORKING AFTER SUCCESSFUL LOGIN

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az group create
Errors:
Please run 'az login' to setup account.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az group create --name {} --location {}

Expected Behavior
Environment Summary
Windows-10-10.0.19042-SP0
Python 3.10.5
Installer: MSI

azure-cli 2.40.0

Additional Context

@zhoxing-ms for awareness

The issue has been fixed. It was a system time zone setting issue.


Bug/postgres migration

Most of the bugs associated with the postgres migration should be solved at this point. Next step, deployment!

Okay boys, big day..... we'll see if this works.


Add to travis PHP 7.4nightly and try to run tests on PHP 7.4 and nightly

Fix issue #2293
Description
Travis configuration.
Checklist:

[X] Securely signed commits
[X] Component(s) with PHPdocs
[X] Unit testing, with >80% coverage
[X] User guide updated
[X] Conforms to style guide

ould this be done as two matrices, one for PHP7.2 & 7.3 and one for the others?
Your approach is likely to get unwieldly as we add more database drivers

I don't think there is much we can do. While I'm not expert in Travis config file, I prepared some simplifies.

I'm not a fan of going to a full matrix currently, honestly. I do think we should add 7.4 in right away (I'll take care of that momentarily.) It's always possible we get another solution in the place in the near future. Closing.


Make a website version

It would be nice if we could have a website version of this tool. Type in the module name et voila! No CLI or node required.

Heh, I did that!
https://npmtrace.appspot.com


(GH-702) added documentation for TeamCity

Seems exposing vcsroot.branch is required for Cake.Recipe to run.
fixes #702

@nils-a thanks for this, really appreciate it!
Does this configuration go into a file in the repository, or does this go into a build configuration within TeamCity?  Historically, TeamCity was very much UI driven.  How would I take have you have provided here and get it into a build?

@gep13 no, that's the DSL, as displayed in TeamCity - as far as I see the DSL can be edited in the UI but not checked into the repository like the others. (Probably should mention that, though.)

TeamCity can store its configration in a VCS, see https://www.jetbrains.com/help/teamcity/storing-project-settings-in-version-control.html. But I would suggest to keep it to the user how he uses TeamCity (through UI, with settings stored in XML, with settings stored in Kotlin)

@gep13 @pascalberger I have added a sentence to explain the exported Kotlin DSL.
Should we better use screenshots to document this?

@nils-a screenshots never hurt, so if you wanted to add some here, I certainly wouldn't say no 😄

@gep13 added a textual guideline thru the TeamCity-UI, including two screenshots. (and I was able to add the img-responsive-calass thanks the video from the week before 😃 )

@nils-a thanks for getting this documented!


[BUG] Poor performance on MacOS

What
I'm experiencing strange performance issues on MacOS. The rgb_depth_aligned sample runs at about 10 FPS which should run at 30 FPS. And basically all samples that include highish rgb resolutions run either very slow or have a really high latency ~3 Sec.
How to reproduce:
Clone the depthai-core repo
Build with this configuration:
cmake -S. -Bbuild -D’BUILD_SHARED_LIBS=ON’ -D’DEPTHAI_BUILD_EXAMPLES=ON’ -D’CMAKE_INSTALL_PREFIX=/usr/local’
sudo cmake --build build -j16 --target install
run build/examples/rgb_depth_aligned or build/examples/rgb_video
Expected behavior
Smooth video and depth with 30 FPS
Result
Slow, laggy video with high latency
Additional context
I benchmarked frame acquisition times and it seems to be stuck at 100ms (10 FPS).
The same code and all samples work as expected on Ubuntu 20.04 (Stable, expected framerate and no lags at all)
getUsbSpeed returns SUPER in all tests. Verbose logs show no issues.
OAK:
OAK-D Pro W (IMX378 RGB)
Mac:
MacBook Pro 16" 2019 - 2,4 Ghz Intel Core i9 - 16GB RAM - AMD Radeon Pro 5300M
Running MacOS Ventura 13.0
Ubuntu:
Intel Core i7 11800H - 32GB RAM - Nvidia RTX 3070 Laptop GPU

Thanks for the report @ZumWoelkchen
Do you mind cross checking with Python library & examples? I think this boils down to displaying the frames.
If you remove the imshow calls and just log the framerate, is it running as expected?

Getting back a little late but after some extensive testing i can confirm that the issue lies in single-threaded imshow/OpenCV on MacOS.


Interpolation by series

This is not bug, rather a feature request (is there a template for that?).
I'd be great if we could define different interpolation functions via the series-config.
Currently <px-vis-timeseries> allows to set 1 interpolation-function at the chart level which is then passed down to the individual <px-vis-line-*> components, but it does not allow to set different interpolation functions for different series.
I am aware that Polymer does not allow to pass functions via a data-binding. One workaround would be, to pass a key which is then used to lookup an interpolation function. I'd be happy to discuss this further and could also send a PR for <px-vis> if that feature gets accepted.

Adding in v1.1.
I've added the ability for interpolation curves to be passed in the seriesConfig and the lookup by key.
The reason I didnt do a lookup originally is that many of the functions require additional parameters. So I've added that ability in, but only for the "simple" functions that do not require any parameters:
"curveBasis"
"curveBasisClosed"
"curveBasisOpen"
"curveLinear"
"curveLinearClosed"
"curveMonotoneX"
"curveMonotoneY"
"curveNatural"
"curveStep"
"curveStepAfter"
"curveStepBefore"

If you want to use other interpolators, you'll still have to pass them in as functions.
It is true that you cannot directly bind a function, which is annoying, but the workaround is:
<px-vis-timeseries
    ...
    interpolation-function="[[ _returnInterpolator() ]]">
</px-vis-timeseries>

<script>
    _returnInterpolator: function() {
        return Px.d3.curveBasis;
    }
</script>

Now that you can pass in a per series interpretor, your seriesConfig now can look like:
{
    "mySeries2":{
        "type":"line",
        "name":"mySeries2",
        "x":"x",
        "y":"y2",
        "color": "rgb(250,164,58)",
        "interpolationFunction": Px.d3.curveBasis
    }
}

or
{
    "mySeries2":{
        "type":"line",
        "name":"mySeries2",
        "x":"x",
        "y":"y2",
        "color": "rgb(250,164,58)",
        "interpolationFunction": "curveBasis"
    }
}

That's great news! Thanks!


Icon doesn't appear with full size

Hello, i'm trying to display icon
                    <Icon style = {{textAlign: 'center', fontSize: 40}} name = 'ios-contacts-outline'/>
the problem is the icon doesn't appear with full size like this

Any ideas?

@skokon try setting the width to null for Icon.
Like
<Icon style = {{width:null, textAlign: 'center', fontSize: 40}} name = 'ios-contacts-outline'/>

thanks a lot, how i can set your answer as the correct answer?

Thank you @akhil-ga ! It didn't solve my problem completely. I had to give those styles:
paddingTop: 4,
height: 21,
width: null,

This is the right solution for me, it can vary on different stacks

Do not use the fontSize style for the Icon, instead use the size and boxSize props of the Icon component.


stable/cockroachdb "bring-your-own-certs" scenario

bump cockroachdb version to 19.2.1->19.2.2
Signed-off-by: Artur<EMAIL_ADDRESS>Is this a new chart
no
What this PR does / why we need it:
Which issue this PR fixes

added "bring your own certs" scenario for secure cockroachdb installation.
bumped cockroachdb version to the latest as of now 19.2.1 -> 19.2.2

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

[x ] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

I'd recommend splitting the backupper change into a separate PR, so we can discuss it separately.

I'd recommend splitting the backupper change into a separate PR, so we can discuss it separately.

sorry, accidently leaked into my master. fixed it already. Yes, I will open another PR once I properly test it


GUACAMOLE-980: used tomcat-jre8 Docker-Image seems to be deprecated

Let's go ahead with this ?
As it seems to be a prerequisite before going further with #469...
@mike-jumper, without @asutosh936 response, perhaps you can update the commit message by yourself when merging the PR ?
Many thx 👍

... perhaps you can update the commit message by yourself when merging the PR?

As a rule, no. It is self-defeating for an open source project to step in and take over for a contributor, particularly regarding compliance with contribution guidelines, regardless of how small that change might be. I made the mistake of stepping in on behalf of contributors in the early days of the project (before the move to the ASF), back when I was too shy to ask contributors to make minor changes themselves, and it was horrible.
If GUACAMOLE-980 becomes critical and there is no movement on this PR, then yes, we might step in, make the change ourselves via a new PR and close this one. I don't think we're at that point yet.

OK, thank you @mike-jumper for your detailed explanation, understood 👍 :)

@mike-jumper - Have updated commit message. Please review and merge.

@asutosh936 - You've updated the PR title, but not the commit message.  Commit message still lacks the ":" between the JIRA issue number and the message.

Looks like it is having some issues, have raised separate PR for the same with correct message with :.
PR: https://github.com/apache/guacamole-client/pull/553
Once PR is merged, will close current PR.

Merged the other pull request.


SqrlOnly and Hardlock do not have access to HttpContext making it very hard to work with databases

Describe the bug
When a method for SqrlOnly or Hardlock are implemented there not passed HttpContext like other are making it hard to get hold of a database or other services
Expected behavior
A HttpContext is passed to the methods

Version 1.2.0 will address this and introduce an braking change to include this


Sankaku support?

This probably isn't the first thread of this, but where the heck is Sankaku?

There is no suppot anymore for Sankaku see : https://github.com/Bionus/imgbrd-grabber/issues/2128#issuecomment-697037426 .

There's a way to use Sankaku. Keep searching for the right support ticket. I forget exactly how I hacked it together, but I wrote out an explanation somewhere in a Sankaku-related support thread some time last year, I think. It involved using a file from an old installation of the program, manually marking it as "do not delete" so the program wouldn't delete the data for Sankaku, and then updating the program.
I do suggest you move on to other boorus, as Sankaku's admin has been throwing fits and running the site into the ground over the last few years and doesn't show signs of improving.
Remember, the first thing you're supposed to do is search the issues section for a few minutes with key terms of your issue to see whether it's been talked about before and what it's status is. The Sankaku question has been coming up a LOT, so I don't think you bothered to search for it. Part of me giving you breadcrumbs here is to encourage you to learn to use the search features on the issues page.

People have found workaround, but Sankaku is not officially supported anymore.
See issue #1888 for more details. 🙂


Feature/boss mock

Description
Add a boss abstract class and add Wither and EnderDragon to mockable entities
Progresses  #825
Checklist
The following items should be checked before the pull request can be merged.

[x] Code follows existing style.
[x] Unit tests added (if applicable).

Hey, Can you please rebase this and check coverage of your tests?


python311Packages.nettigo-air-monitor: 2.2.1 -> 2.2.2

Diff: https://github.com/bieniu/nettigo-air-monitor/compare/refs/tags/2.2.1...2.2.2
Changelog: https://github.com/bieniu/nettigo-air-monitor/releases/tag/2.2.2
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

5 packages built:

home-assistant-component-tests.nam
python310Packages.nettigo-air-monitor
python310Packages.nettigo-air-monitor.dist
python311Packages.nettigo-air-monitor
python311Packages.nettigo-air-monitor.dist


Create 0092-reverse-linked-list-ll.java

File(s) Modified: 0092-reverse-linked-list-ll.java
Language(s) Used: java
Submission URL: _https://leetcode.com/problems/reverse-linked-list-ii/submissions/933023749/_

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

Thanks!


Move init command to aerogear/create-graphql

graphback-cli requires a peer dependency of graphql to execute the generate command, which is used to generate GraphQL resources such as queries, and schema.
The CLI also executes init, which would typically be executed outside of a folder with any grapHQL dependency. This causes #1530 to happen.
These two commands are incompatible to have in the same CLI package. It makes sense to move the init command to a different package which will not require graphql to be installed.
Let's move the init command/code to aerogear/create-graphql

This will fix #1530

create-graphql does a lot of cool file generation with Yeoman. It would be a lot of work to retrofit this to use Graphback.
What about extracting the init command code to a package create-graphback-app?
Then you can do:
npm init create-graphback-app my-cool-project

Or
npx create-graphback-app my-cool-project

Then work could be done on create-graphql post v1 release to make it compatible with Graphback while staying true to its original file generation.

I have no strong opinion, but this is going to create yet another CLI.
Less then better IMHO. Can we just add graphback to existing stuff?

Are we prepared to also support what currently exists in create-graphql?

I know create-graphql does not get much current usage, so maybe it is fine to scrap it all.

We give the same features by different way. Cli also generates resolves..
I would simply copy paste our current CLI and remove non init stuff there.
That is all.

We give the same features by different way. CLI also generates resolves..

The Graphback CLI does not currently generate resolvers.
create-graphql does not generate a schema file, it generates several JavaScript files that export GraphQLObjectType with mutation and queries as fields on the object (not separate like the format from graphql-tools).
Check its commands for more: https://github.com/aerogear/create-graphql/blob/master/docs/Commands.md
Also, I could not find anything to suggest that it currently generates TypeScript files - only JS.

Agrement - No create-graphql for 0.14.0. We can have create-graphback for the moment


New user email

Man oh man, this is UGLY to get hold of the password. But it works. Thanks stack overflow!
@xybrnet

:+1:


karate fork / exec cli behavior broken in 1.X

opening to investigate, more details shortly

@maxandersen here's the change I just made, you may have more experience with CLI args to review: https://github.com/intuit/karate/commit/f84409df3ffc5a187d5d5f5bf35d825e5d252b3d#diff-2b896dd948abdfb58f1f4f3b8dc98314692ac1cba2413122605f53b0169ca662R153


Auto-focus input field when editing a global setting

Go to Global Setting => Advanced Settings.
Pick a setting and click the 3 dots and 'Edit Setting'.
The value field should automatically get focus.
@rak-phillip Added a directive to do this, so this should be straight forward.

The v-focus directive currently handles cases for for LabeledInput, LabeledSelect, and SelectPriciple. We might need to add more to support any focusable components as we encounter them.

Test Environment:
Rancher version: v2.6-head 10dfee4
Rancher cluster type: single-node docker install
Docker version: 20.10
Downstream cluster type: n/a
Downstream K8s version: n/a
====================================
Test Steps:

go to the Main menu
click Global Settings

Advanced Settings

Pick a setting with an input field (they have string values on UI on the Advanced Settings page)
Click on three dots
Click on "Edit Setting"

Confirmed: the input field is focused when the setting page open

additional notes:
adding autofocus didn't change the functionality for updating the fields


IndexError: index 0 is out of bounds for axis 0 with size 0

This was discovered in the CI: https://github.com/boschresearch/pcg_gazebo/actions/runs/3335964277/jobs/5520547249
The reason is probably a degenerate triangle used in the orientation check: https://github.com/mikedh/trimesh/issues/1753

Should be fixed in<EMAIL_ADDRESS>


fix(matches_related_rule): Add check to match Kind and Resource for m…

Fixes https://github.com/metacontroller/metacontroller/issues/883

Going to cleanup the tests with a test table.

@grzesuav I have add the fix and the tests, ready for review when you have a chance, thanks.

/retest

Thanks @grzesuav , very much appreciated.


constraints are not activate with argument

using Chronos on Ubuntu Ubuntu 12.04.2 LTS
chronos                             2.3.4-1.0.183.ubuntu1404             amd64
mesos                                    0.23.0-1.0.ubuntu1204
When I use constraint without argument, mesos launch the job on the correct server.
But if I add <<	"arguments": [  "toto"],  >> on the same job, constraint are not used, and the job is launch on any slaves.
{
"name": "fred",
"command": "/tmp/toto.sh",
<<	"arguments": [  "toto"],  >>
"shell": false,
"epsilon": "PT10M",
"executor": "",
"executorFlags": "",
"retries": 2,
"owner"<EMAIL_ADDRESS>"ownerName": "",
"description": "",
"async": false,
"lastError": "",
"cpus": 0.1,
"disk": 256,
"mem": 128,
"disabled": false,
"softError": false,
"dataProcessingJobType": false,
"errorsSinceLastSuccess": 0,
"uris": [],
"environmentVariables": [],
"highPriority": false,
"runAsUser": "root",
"constraints": [
[
"role",
"EQUALS",
"alimentation"
]
],
"schedule": "R/2015-09-17T15:15:00.000Z/PT10S",
"scheduleTimeZone": "Europe/Paris"
}

I guess to problem is in JobUtils.scala
def getJobWithArguments(job: BaseJob, arguments: String): BaseJob = {
need to copy constraints too ...

This is very important for us. Can anyone kindly confirm when 2.5.0 will be released. The fix at https://github.com/mesos/chronos/pull/562 works for us perfectly well, can someone please confirm by when it will atleast be merged in code base?


Add Wait instruction support

Purpose

Support Wait construct in BIR and jvm codegen

Approach

A wait in BIR,

bb3 {
		%8 = MOVE %10;
		%12 = MOVE %5;
		%13 = MOVE %8;
		%14 = WAIT %12|%13;
	}

A wait after a codegen,

            case 3:
                var16 = 3;
                if (!var13.isDone) {
                    var0.blocked = true;
                    var0.blockedOn = var13.strand;
                    var0.yield = true;
                    break;
                } else {
                    var14 = (String)var13.result;
                    var17 = var14;
                }
            case 4:

Remarks

BIR does not support wait for all. It is a separate instruction.

Codegen only supports wait for a single future.

Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@184b01d). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #15051   +/-   ##
=========================================
  Coverage          ?   68.85%
=========================================
  Files             ?        5
  Lines             ?      122
  Branches          ?       21
=========================================
  Hits              ?       84
  Misses            ?       38
  Partials          ?        0

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 184b01d...18b6874. Read the comment docs.


Empty rgb files written to disk

Hello,
I am trying to write a PiCamera application capable of the following after motion is detected:

Save a stream of before and after video to h264 format
Save a stream of uncompressed and down-scaled images from the motion corresponding exactly to those within the motion event.
Eventually save the uncompressed stream at a much reduced framerate compared to the video stream, so I do not save too much data.

I have assigned a PiCameraCircularIO buffer for both the rgb and h264 streams. When I copy data from the rgb buffer to file, the before.dat file is always empty, while after.dat, before.h264 and after.h264 always contain data. My suspicion is this is because the data in the ring buffer is too large to copy to disk quickly, so I added a delay between copying and clearing the buffer. No avail.
Can someone help?
Below I show a sketch of my application, based on one of the examples in the docs:
from ctypes import resize
import io
import random
import picamera
from PIL import Image
from time import sleep

def detect_motion(camera):
    result = random.randint(0, 5) == 0
    return result

with picamera.PiCamera() as camera:
    camera.resolution = (1920, 1080)
    camera.framerate = 25
    camera.annotate_frame_num = True
    stream_h264 = picamera.PiCameraCircularIO(camera, seconds=10,splitter_port=1)
    stream_image = picamera.PiCameraCircularIO(camera, seconds=10, splitter_port=0)
    camera.start_recording(stream_h264, format='h264', splitter_port=1)
    camera.start_recording(stream_image, format='rgb', splitter_port=0,  resize=(416, 416))
    try:
        while True:
            camera.wait_recording(1)
            if detect_motion(camera):
                print('Motion detected!')
                # As soon as we detect motion, split the recording to
                # record the frames "after" motion
                camera.split_recording('after.h264', splitter_port=1)
                camera.split_recording('after.dat', splitter_port=0)
                print(list(enumerate(stream_image.frames)))
                # Write the 10 seconds "before" motion to disk as well
                stream_h264.copy_to('before.h264', seconds=10)
                stream_image.copy_to('before.dat', seconds=10)
                sleep(20)
                stream_h264.clear()
                stream_image.clear()
                # Wait until motion is no longer detected, then split
                # recording back to the in-memory circular buffer
                while detect_motion(camera):
                    camera.wait_recording(1)
                print('Motion stopped!')
                camera.split_recording(stream_h264, splitter_port=1)
                camera.split_recording(stream_image, splitter_port=0)
    finally:
        camera.stop_recording()

https://picamera.readthedocs.io/en/release-1.13/api_streams.html#picamera.PiCameraCircularIO.copy_to

copy_to(output, size=None, seconds=None, first_frame=PiVideoFrameType.sps_header)
Warning
Note that if a frame of the specified type (e.g. SPS header) cannot be found within the specified number of seconds or bytes then this method will simply copy nothing (but no error will be raised).

An RGB stream will never have any frames of type PiVideoFrameType.sps_header, so it will never copy any data.
stream_image.copy_to('before.dat', seconds=10, first_frame=PiVideoFrameType.frame) is more likely to work.

Thank you


New package: PostgREST.postgrest version 12.2.3

Notes

External dependencies required, add "InstallationNotes" for it.
See https://github.com/microsoft/winget-pkgs/issues/189017#issuecomment-2460520948

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Resolve #189017

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\postgrest.exe returned exit code: -1073741515

Automated error analysis suggests -1073741515 may mean STATUS_DLL_NOT_FOUND - possibly due to a dependency on Microsoft.VCRedist.2015+.x64 (or .x86).
(Automated response - build 897.)

@stephengillie yes, it depends on psql.dll (and its' dependencies)

Hi @Dragon1573,
Is a dependency missing? The application refuses to run, instead giving this message on each launch.

Verify:

libpq.dll can be downloaded and installed via package identifier PostgreSQL.psqlODBC.
The "dependency" package will not add itself to $env:PATH.
postgrest.exe can't load libpq.dll and failed to launch.

@stephengillie , I don't have good solution for resolving this problem. 😞

So the package has a dependency, and a workaround is needed to address a bug.

[!WARNING]
The package requires libpq.dll, which can't be correctly added to $env:PATH. Unable to resolve this problem.


Move Eventbus to Application scope rather request scope

We need to raise events tenant and application wide

I think there might be an issue if it's registered at a higher scope as it will resolve some services, and a singleton cannot resolve a scoped service.

Yup, its a problem as we need to use the IEventBus everywhere... problem is that I really like its not currently reliant on Http.... hmmm need to have a think about this.

I think EventBus must be registered in AddHostCore
  public static IServiceCollection AddHostCore(this IServiceCollection services) {
            services.AddSingleton();

            services.AddSingleton();
            {
                services.AddScoped();
                services.AddSingleton();

                services.AddSingleton();

                services.AddSingleton();
                {
                    services.AddSingleton();
                    {
                        services.AddSingleton();
                    }

                    services.AddSingleton();
                }
            }

            services.AddSingleton();

            services.AddSingleton();

            return services;
        }

You need to understand the difference between Application singletons and Tenant singletons. Application singletons are unique for all tenants, whereas Tenant singletons are unique inside a specific tenant. All the services which are registered using services.AddSingleton will be application singletons. All the ISingleton interfaces will be tenant singletongs.
The EventBus state needs to be per tenant, because messages can't go across the tenant boundary, so the state needs to be defined on the tenant services collection, which is what ShellContainerFactory is building.
How application singletons are built is that they are resolved by ShellContainerFactory, and injected as instances in the tenant's container, as a singleton. So all the tenants will actually share the same instance.
What events need to be raised application wide?
Another solution is for the EventBus state to store the state for every tenant.


Typo fix

count down -> countdown

@swift-ci please smoke test

Hm, both this patch and the existing docs read correctly. @natecook1000 What do you think?

Reads correctly to me; the sense being conveyed is that “down” is the direction of iteration. To put it more explicitly, one might say “iterate over the countdown down to zero,” but that does not read well.

Reads correctly to me; the sense being conveyed is that “down” is the direction of iteration. To put it more explicitly, one might say “iterate over the countdown down to zero,” but that does not read well.

I understand your point, I think it makes sense too. The way I see it is that "iterate over the countdown to zero", countdown here should be a noun. And if we google "count down" as a noun, it's "countdown" instead of "count down". That's why I created this PR.

The current wording could surely be improved, but right now the sentence means "iterate downward over the count". I don't think the new version perserves that meaning.

Closing this for now — @weitieda feel free to re-open if you'd like to revisit!


Posctrl Z position hold with LIDAR and PX4Flow

The Quadrotor has a optical flow px4 flow and Lidar lite V3(i2c). While flying manually the drone is performing normally even when switching to Altitude Hold mode the altitude hold is good. If I put my hand just below the  lidar drone goes up and comes down to actual height.
Subsequently, while testing position control mode it is holding the position earlier there were oscillations but after tuning it is pretty stable and stays at the position without GPS. But when I try to put my hand below the sensor the drone goes up and for some second it stays there and then it start to comes down but then it actually goes up.
Reviewing the flight logs I observed that Z setpoint changes to -6 instead of -2 and drone somewhat stays there and we have to bring it down. And this thing is repeatable. It is the test case if a car comes below it  should go up and come back to its original altitude.
Is this z setpoint change is as expected or some bad parameter tuning or worse some bug in firmware?
Px4 Flight logs:
https://logs.px4.io/plot_app?log=2e693261-f0df-4f75-867b-4ac39b73f19c
The timing when I bring my hand below it can be observed from distance  sensor output i.e. 16:03 & 16:05

I again tested the flight with GPS on EKF2-aid-mask set to 3. Similar, issue happened.
Local position z
I realised for small distance reading change posctrl is good i.e. it comes back down to its position but if estimated position crosses 0 or becomes positive it changes its set point near to -5.5 or so. Also estimated position shouldn't  cross 0  as Distance reading is good for low values and it is not becoming negative.
@priseborough  @ChristophTobler

Seems like the ekf was deadreckoning for too long and then reset the height.

By that time the z estimate has already diverged too much and that stays after the reset.

https://github.com/PX4/Firmware/pull/8183 is a fix (partly). With this one your vehicle would stay at ~3m after the reset instead of applying the delta between estimated z and setpoint z.
The other issue is that you have quite a lot of vibrations and that's why your z estimate goes the wrong direction -> resulting in a large delta.


Error trying to copy file (Windows)

Hi there!
I'm getting an error that I can't seem to figure out. Wondering if you've seen it before.
OS: Windows 10
CLI: PowerShell
Repro steps:

npm install -g squido
mkdir test; cd test
squido new

Results:
PS C:\Users\user7\Documents\portfolio> npm install -g squido
C:\Users\user7\AppData\Roaming\npm\squido -> C:\Users\user7\AppData\Roaming\npm\node_modules\squido\cli.js
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\squido\node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

+<EMAIL_ADDRESS>updated 1 package in 15.428s
PS C:\Users\user7\Documents\portfolio> mkdir test; cd test

    Directory: C:\Users\user7\Documents\portfolio

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
d-----         6/22/2021   9:45 PM                test

PS C:\Users\user7\Documents\portfolio\test> squido new
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/404.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/index.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/page.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/post.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/tag.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/layouts/layout.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/package.json
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/robots.txt
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/content/images/favicon.png
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/content/images/squido.png
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/content/images/squido.svg
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/content/javascripts/site.js
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/content/stylesheets/style.css
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/about.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/alter-tradere.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/colore-non-nec-robora-altera.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/conspicit-celeberrima-suo-cui-cum-meque-strata.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/cruor-videntur.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/dederat-venias-noctis.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/ibi-talia-non-caruit-thisbes-vitae-thyrsos.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/intumuere-sine-viso-in.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/movit-plumis-studio-receperat-dicentem-mollia.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/noxque-deum-miseris-id-silvis.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/pectora-remittit-pudorem-ardentior-securi-leve-mihi.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/quid-io-sine-rima-leves-fortuna-non.markdown
Error: Source and destination must not be the same.
[Setup complete]

PS C:\Users\user7\Documents\portfolio\test> ls

    Directory: C:\Users\user7\Documents\portfolio\test

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
-a----        10/26/1985   1:15 AM           1813 config.js
-a----        10/26/1985   1:15 AM            192 package.json

The LastWriteTime above is suspect, no?
Thanks!

Suspect this relates to #2.

Fixed in 6cad43d8487690603e61126476f5999ae29526b1.

Cool. Thanks for peeking at this issue. I'll test again now.

Thanks. New code is not on NPM yet, you can grab the new code easily with: npm i -g https://github.com/mrvautin/squido

Hello. I'm seeing the original issues above when trying to run squido new even though I installed Squido using npm install -g https://github.com/mrvautin/squido.gi
I Tried:

npm -g install squido

Error

npm uninstall squid
npm install -g https://github.com/mrvautin/squido.git

Error

PS E:\dataDocuments\ktree\vault\assets\squido> npm uninstall squido
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN<EMAIL_ADDRESS>No license field.

up to date in 5.376s
found 0 vulnerabilities

PS E:\dataDocuments\ktree\vault\assets\squido> npm install -g https://github.com/mrvautin/squido.git

C:\Users\user7\AppData\Roaming\npm\squido -> C:\Users\user7\AppData\Roaming\npm\node_modules\squido\cli.js
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\squido\node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

+<EMAIL_ADDRESS>updated 1 package in 13.655s

PS E:\dataDocuments\ktree\vault\assets\squido> squido new

Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/404.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/index.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/page.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/post.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/swagger.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/tag.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/layouts/layout.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/partials/header.hbs
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/package.json
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/robots.txt
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/content/images/favicon.png
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/content/images/squido.png
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/content/images/squido.svg
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/content/javascripts/site.js
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/content/stylesheets/style.css
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/about.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/alter-tradere.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/api.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/colore-non-nec-robora-altera.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/conspicit-celeberrima-suo-cui-cum-meque-strata.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/cruor-videntur.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/dederat-venias-noctis.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/ibi-talia-non-caruit-thisbes-vitae-thyrsos.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/intumuere-sine-viso-in.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/movit-plumis-studio-receperat-dicentem-mollia.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/noxque-deum-miseris-id-silvis.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/pectora-remittit-pudorem-ardentior-securi-leve-mihi.markdown
Error: Source and destination must not be the same.
Error trying to copy file: C:/Users/user7/AppData/Roaming/npm/node_modules/squido/source/posts/quid-io-sine-rima-leves-fortuna-non.markdown
Error: Source and destination must not be the same.
[Setup complete]

Resulting in:
PS E:\dataDocuments\ktree\vault\assets\squido> ls

    Directory: E:\dataDocuments\ktree\vault\assets\squido

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
-a----        10/26/1985   1:15 AM           2546 config.js
-a----         1/29/2022   2:59 PM             80 package-lock.json
-a----        10/26/1985   1:15 AM            202 package.json

Hm.. after running npm list, here are the results. Don't know if this helps...
PS E:\dataDocuments\ktree\vault\assets\squido> npm list
<EMAIL_ADDRESS>E:\dataDocuments\ktree\vault\assets\squido
`-- UNMET DEPENDENCY squido@latest

npm ERR! missing: squido@latest, required by<EMAIL_ADDRESS>

@SeaDude This should be fixed in 829d196. If you still have issues, please let me know.

Still seeing the same issue. Am I missing a step?
Steps to repro

npm uninstall squido
npm uninstall https://github.com/mrvautin/squido.git
npm install https://github.com/mrvautin/squido.git
squido new

It's possible you have it installed globally with the -g tag.
Can you try?:

npm uninstall -g squido
squido -V to check it's not installed
npm install -g https://github.com/mrvautin/squido.git
squido new

You got it!


[unittests] Simplify ligature test

Not expecting ligatures to be inactive when they're active with
parameters gives more flexibility. One might want to check that a
ligature works both with and without certain parameters.

LGTM