Proposal: Spec update - replacePin should support "update" like operations

Please read https://github.com/ipfs-shipyard/pinning-service-compliance/issues/124 and https://github.com/web3-storage/web3.storage/issues/1547 for more details.

I believe the intention was for replace operation to already cover atomic "update" operations  like this.
User should be able to replace an existing pin with one that pins the same CID, but has new/modified  Pin.origins (to unblock stuck pinning job) or Pin.meta to add/modify metadata attributes.
If this is unclear from the spec, then we should clarify.

If this is unclear from the spec, then we should update the description of this method.

I think a one part in the spec makes this somewhat unclear:

Replace an existing pin object (shortcut for executing remove and add operations in one step to avoid unnecessary garbage collection of blocks present in both recursive pins)
-- https://ipfs.github.io/pinning-services-api-spec/#tag/pins/paths/~1pins~1{requestid}/post

As I mentioned in pinning-service-compliance#124, I interpret this exactly as a replacement operation. Especially because the CID is required. If the method were described as update instead, with CID being optional, then it would make sense that anything/everything could be updated/changed.
maybe that's just me. I would love to hear @2color and @en0ma's interpretations and feedback.

P.S. Replace should definitely delete/remove any data that's not provided during a replace call, but for update, I feel like it should only modify provided data. There are significant connotations that come with calling something an update operation, and replace does not come with those exact meanings.
Connotations that come with update/replace operations
That I know of, and your experience may vary, but I thought this could help the conversation:
Update

Do not corrupt/change/modify existing data that were not specified to update.

Didn't supply Meta field? provider should leave it alone.

Only require uniquely identifying properties for update operations.

Requiring anything additional is unnecessary and can result in accidental updates that modify/corrupt data unintentionally.

Merging may be possible

Highly varies by services/products, but update operations for objects frequently include merge support. (unless you want to write add/edit/delete APIs for each object that exists across your ecosystem).

Replace

The item being replaced will be thought of as "garbage" for either deletion/cleanup after a successful replacement.
The item being provided (replacer/new item) should match the same type as the replaced object, but there are zero other expectations on it's shape.
There is a unique identifier that acts as a pointer to an object, and a replacement operation is simply altering the location being pointed to.

This should be required as well as any required properties for the update operation's type

Hey, I'm looking at the  https://github.com/web3-storage/web3.storage/issues/1547 and I had the same doubts raised by @SgtPooki here.
We interpreted the spec to be a replace operation rather than an update.
To make it obvious we are also returning an error if the same CID is provided.
From the discussion in https://github.com/ipfs-shipyard/pinning-service-compliance/issues/124, it looks like we reached a consensus it should support update operations as well.
I'm going to update our implementation, please shout if that shouldn't be the case.
On top of what already discussed, and the fact the endpoint is meant to do updates (on top of replace), it'd make more sense to update the same psa_pin_request (using the same requestid) rather than create and return a new instance.
This isn't the case atm in web3.storage (we create a new psa_pin_request while deleting the old one), but would you agree that'd be a better approach?
@lidel @SgtPooki keen to hear your thoughts!

Looking at this now but not in the right headspace. Will have to come back to this one

Just gave things another read, but on mobile and just jotting down some thoughts.

User should be able to replace an existing pin with one that pins the same CID, but has new/modified Pin.name, Pin.origins (to unblock stuck pinning job) or Pin.meta (to add/modify metadata attributes).

Pin === requestid, so CID should be able to be the same.
scenario 1 - same CID
Require one of name|origins|meta. If none of those are different, we could fail “No_changes_required”, or silently succeed, or always loop over fields.
For each field, update the pin’s relevant field with the new value.
scenario 2 - different CID
If CID is not the same, inputs should be a partial of those required by “addPin” and in addition, requestid is also required.
This scenario seems to be the original intent behind replace; allowing the service to combine the deletion of the old CID, with the pinning of different content, while allowing consumers to keep the same requestid.

I think CID should not be required by replace, but should be allowed to be the same as already contained CID.
If a requestid already exists(PinStatus), it must already have a CID. If it does, and I want the same CID and just to replace other data in the Pin, then the service already knows the CID.
If i pass a CID, and it’s the same, it shouldn't matter. I should be able to pin the same CID with different name/metadata (different origins shouldnt matter for successfully pinned content, right?).
Edgecases
If i try to pin the exact same name, CID, and metadata, under different requestid’s, that should be blocked with a “already have that at x requestid.” or does this have some particular benefit i’m not aware of?
What are the minimal viable differences required for a separate Pin to be relevant? CID+name probably, right?
I feel like an error should only be thrown when the same CID is passed if the replacement has no increase in value to the existing pin, but I don’t think the spec calls that out And as long as the same CID is allowed to be added or replaced multiple times, the spec is satisfied.
However, i think the spec should grant providers some lenience in setting up their customers for success; i.e. Erroring out in cases where the customer might be doing something unnecessary.


CUDA: Cooperative groups in Windows without TCC mode

Feature request
It is great that cooperative groups was added in release 0.53. I'm wondering when/if it will be available in Windows, without requiring the GPU to be in TCC mode?

ping @gmarkall

This is a limitation imposed by the driver, rather than Numba - the documentation describes the limitation mainly to save Windows users with WDDM mode some frustration / confusion if CGs don't work for them (rather than making them hunt for the issue in other docs).
If the driver supports CGs with WDDM devices in future, then that will also work in Numba, but it's something that's outside our control for now.

Thanks for clarification! I'll use linux for now.
Again, thanks for the amazing work you are doing!


Does redux-token-auth work with OmniAuth?

I'm setting up a solution with devise_token_auth authorizing users through Github. Does redux-token-auth support that?

I'm trying to do the same thing with a Rails 5 API using https://github.com/lynndylanhurley/devise_token_auth. I want users to be able to sign in with an oauth provider (Google, FB, etc.), have the user authenticated and inserted into the table on the backend and then get back to my React frontend (a separate app) authed by the new token.
Have you gotten something like that working @drothschild ?


[artifacts]Gather the whole /etc

There are many places in /etc the EDPM ansible roles are modifying like tuned config or ssh_known_hosts file. To be able to troubleshoot such configuration in CI runs this PR switch to gather the whole /etc directory from the EDPM nodes.
As a pull request owner and reviewers, I checked that:

[ ] Appropriate testing is done and actually running
[ ] Appropriate documentation exists and/or is up-to-date:

[ ] README in the role
[ ] Content of the docs/source is reflecting the changes

/approve
While I don't really like getting the full of /etc, it seems to be needed. It's also planed to move to sosreport when dust settles a bit, so we may get some even better debugging capabilities.
At least, we won't get the symlink, hopefully this should lower a bit the amount of potentially useless data :).


Document process

what

Implement turnkey coldstart for our most common reference architectures

why

Make it insanely easy to get up and running

 Document Each Account


Remote RabbitMQ Connection Timeout using .NET Core 2.0

I am not sure if this is due to Core 2.0 but I can run it fine on non core.
The code I am using is below.

Options

  "RabbitMQ": {
    "Host": "rabbitmq://<IP_ADDRESS>/",
    "Username": "test",
    "Password": "1"
  },

Bus

            var busControl = Bus.Factory.CreateUsingRabbitMq(cfg =>
            {
                var host = cfg.Host(new Uri(_rabbitMQOptions.Host), h =>
                {
                    h.Username(_rabbitMQOptions.Username);
                    h.Password(_rabbitMQOptions.Password);
                });

                cfg.ReceiveEndpoint(host, nameof(TestCommand), e =>
                {
                    e.Consumer(() =>
                    {
                        return new TestConsumer();
                    });

                    e.PrefetchCount = 2;
                });

            });

            _logger.LogInformation("Starting bus");
            busControl.Start();
            _logger.LogInformation("Bus started");
            _logger.LogInformation("Waiting for messages");

Command & Consumer

    public class TestCommand
    {
    }

    public class TestConsumer : IConsumer<TestCommand>
    {

        public TestConsumer()
        {
        }

        public async Task Consume(ConsumeContext<TestCommand> context)
        {
            Console.WriteLine("Consumed");
        }
    }

When I run the code, it fails with
'Connect failed: test@<IP_ADDRESS>:5672/'
TimeoutException: The operation has timed out.

This issue occurs with both 3.5.7 and <IP_ADDRESS>6-develop.
I have confirmed that I can connect fine by following the rabbitmq hello world tutorial.
Each connection creates the following log entries in rabbitmq.
=INFO REPORT==== 25-Jul-2017::05:00:19 ===
accepting AMQP connection <0.31858.0> (<IP_ADDRESS>:51098 -> <IP_ADDRESS>:5672)

=ERROR REPORT==== 25-Jul-2017::05:00:19 ===
closing AMQP connection <0.31858.0> (<IP_ADDRESS>:51098 -> <IP_ADDRESS>:5672):
{handshake_timeout,handshake}

I just installed a localhost server and it worked.
Getting remote connections is still quite important. Any tips on what to look out for?

I have been using it fine with remote connection, there is so many things that could block a remote connections, local or remote firewall is a common one.
Simplest way to replicate.

Install docker for windows
docker run -d --hostname my-rabbit --name some-rabbit -p:5672:5672 -p 15672:15672 rabbitmq:3-management

Connecting locally change your rabbitmq options to
  "RabbitMQ": {
    "Host": "rabbitmq://localhost:5762/",
    "Username": "guest",
    "Password": "guest"
  },

or remotely change rabbitmq to
  "RabbitMQ": {
    "Host": "rabbitmq://<IP_ADDRESS>:5762/",
    "Username": "guest",
    "Password": "guest"
  },

@thedumbtechguy did you get it working?

Remotely no, so I'm using a local install for now.
Unfortunately, I'm not willing to install docker on my windows machine for
now due to space constraints.
On 4 Aug 2017 12:31 a.m., "Gert Jansen van Rensburg" <
<EMAIL_ADDRESS>wrote:

@thedumbtechguy https://github.com/thedumbtechguy did you get it
working?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MassTransit/MassTransit/issues/946#issuecomment-320125089,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4XZVhIpanobOATg8GqiplBKVYq-KKwks5sUmZ8gaJpZM4OiElK
.

In that case you could try Little Lemur - For Development from CloudAMQP for free.

@phatboyg I have been using the pre-release package on netcore1.1 and netcore2 without any issue. think we can close this issue.

I had the same issue as described here. In order to resolve it, I added to the windows\linux host file new hostname - "<IP_ADDRESS> rabbitmq_server" (<IP_ADDRESS> - the IP address of the server) and in the connection string put "rabbitmq://rabbitmq_server" and it works.
I guess that there is some bug in the code. ("rabbitmq://<IP_ADDRESS>" - doesn't work)

I had the same issue as described here. In order to resolve it, I added to the windows\linux host file new hostname - "<IP_ADDRESS> rabbitmq_server" (<IP_ADDRESS> - the IP address of the server) and in the connection string put "rabbitmq://rabbitmq_server" and it works.
I guess that there is some bug in the code. ("rabbitmq://<IP_ADDRESS>" - doesn't work)

@zamiramos I created another example for #966 using the latest development packages where I connect to either my host ip address or docker container ip address.

@zamiramos similar things reported in #966 but it seems to be the OS/RabbitMQ limitation rather than MT.

Ok thanks,
The strange thing is that if I running the same code under .net framework
4.5.1 console application on the same environment(and on the same OS) with
IP address it works great.
2017-08-14 19:48 GMT+03:00 Alexey Zimarev<EMAIL_ADDRESS>
@zamiramos https://github.com/zamiramos similar things reported in #966
https://github.com/MassTransit/MassTransit/issues/966 but it seems to
be the OS/RabbitMQ limitation rather than MT.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MassTransit/MassTransit/issues/946#issuecomment-322243733,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXwamuUvBr0ujcTZ4JtnPjB5vtAdPixnks5sYHpfgaJpZM4OiElK
.

Well, you can try the same MT package version (BTW we are on 4.5.2, you can't have it on 4.5.1) and see if it works. If it does - this is definitely the .NET itself or RabbitMQ since MT code will be the same.

I have same issue. Geting timeout when connecting by ip. It works with localhost or host name defined in hosts file but fails when I use ip address.
Tried with standard RabbitMQ.Client and it works with ip correctly.
.net core 2.0.
masstransit <IP_ADDRESS>5-develop.

The same problem here, remote connection to RabbitMQ broker results a Timeout Exception. however: after 3 - 4 restarts of docker container results sucessfully connection and stable work.
<IP_ADDRESS>1-develop with Masstransit in the same version

I think the oddity that has been mentioned again which I also observed is
being able to connect with RabbitMQ.Client directly but not with MT.
On Wed, Aug 30, 2017 at 12:06 PM, Tival<EMAIL_ADDRESS>wrote:

The same problem here, remote connection to RabbitMQ broker results a
Timeout Exception. however: after 3 - 4 restarts of docker container
results sucessfully connection and stable work.
<IP_ADDRESS>1-develop with Masstransit in the same version
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MassTransit/MassTransit/issues/946#issuecomment-325970375,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4XZYjAGoTJEsXsMc47LgHKEKR0fgiuks5sdVBdgaJpZM4OiElK
.

--
Regards,
Stefan Froelich.

Same issue. I stop rabbitmq and started it. Issue went away.

Hello.
I have the exact same problem  here and i should take a decision on how i'm going to proceed with a long-term project that i have been assigned to. Any info please?
Thanks!

Does bus.StartAsync() work?
To my understanding timeout is caused by deadlock , occuring due to single threaded SynchronizationContext in bus.start() and lack of .ConfigureAwait(false) in RabbitMq client.
Created PR in client's repo.

Hello.
Indeed  bus.StartAsync() works! What does this mean?
BTW it is always tried to be connected using ipv6 which fails and so an exception is caught but it then continues to ipv4.
Is this normal?

I guess it's better to use async version until an updated version of rabbitmq dotnet client is released.
Also, SocketFrameHandler in that lib does actually try to connect using ipv6 and than falls back to ipv4 on failure.

IBusControl.Start() is just a wrapper for the StartAsync. I am wondering how this can be a problem.

BTW it is always tried to be connected using ipv6 which fails and so an exception is caught but it then continues to ipv4

@tasoss it depends on your DNS resolution. If your IPv6 address is resolved first - it will be used.

@jacobpovar I see it now, they use different API for Core. Makes sense. Your PR is accepted, so I assume we just need to wait a little :)

@alexeyzimarev  thanks for the information

Same issue. I stopped rabbitmq service and started it. Issue went away. Perhaps, the previous connection is still alive and stopping the service kills it?

I'm seeing the same problem, it happens after my PC goes to sleep and then wakes up again. Restarting the RabbitMQ service always fixes it.
I'm using rabbitmq://localhost and username guest password guest so I don't think it's just an issue with IP addresses or remote servers etc.
I'm using .NET 4.7 "full framework" (nothing .NET Core in my app) so I don't think it's a .NET Core specific problem either.
I'm using bus.Start(); in a regular console app (not using bus.StartAsync()) so I'll update my code and see if that fixes the issue and report back.

@todthomson are you hosting rabbit in docker? might be dockers stupid tcp proxy.

@gertjvr Just hosting locally on Windows (for local development purposes) :)

Hi, guys are there any updates on the problem?
I faced with the same issue in our projects when we trying to connect to RMQ server through IP address.
A test code is pretty simple.
            var bus = Bus.Factory.CreateUsingRabbitMq(cfg =>
            {
                cfg.Host(new Uri("rabbitmq://<IP_ADDRESS>/vHost"), h =>
                {
                    h.Username("user");
                    h.Password("****");
                });
            });

            bus.Start();
            while (true)
            {
                bus.Publish<RegistryLogItem>(logitem);
            }
            bus.Stop();

Packages:

Framwork: .Net Core 2.0

My understanding is waiting on a RabbitMQ.Client update.

My understanding is waiting on a RabbitMQ.Client update.

Just as an FYI RabbitMQ.Client version 5.1.0-pre1 doesn't fix it for me.
Also, using bus.StartAsync() vs bus.Start() doesn't fix it for me either.
I'm happy to test potential fixes if you need me to. On .Net 4.7 "full framework" so I'm not even sure that my issue is related to what the .Net Core 2.0 people are seeing here.
Restarting the RabbitMQ service (in Windows) seems to be the only thing that with "fix" it for me.

We had this problem, it worked for Erlang 20 but not for some 19.something version.

Any update on this? We are trying to connect from our Azure ASE App service to a VM with RabbitMQ and Mass Transit on it. Works fine with the API running locally against a Docker container of RabbitMQ with port forwarding to 5672, but trying to either hit the IP of the VM or go through the DNS in Azure, it times out.

This is weird because I am running a couple of services inside Linux containers, reaching out to RabbitMq hosted on Windows VMs (production) and I experience no issues at all.

well, we have had some issues with different rabbitmq instances on cloudamqp, while for some instances of erlang 19 we have been able to use it (while not for others).

What version of RabbitMq and Erlang are you using @alexeyzimarev? We are using RabbitMq containers, 3.6.14 with Erlang 19.2.1. I tried updating to 3.7.0 yesterday with Erlang 20.1 and the permissions_topic kept blowing up in management. I tried a local install as well as the container, but both had the same issue. So I went back to 3.6.14

@mmoser-rivermoss I can check tomorrow.

@alexeyzimarev After a bit of troubleshooting and trying things, our problem turned out to be an NSG not taking into account the Subnet IP range, so we weren't allowing the traffic through properly. We are now able to communicate to the VM from the app service.

I'm going to close this, enough  have it working with core that this isn't an MT issue.


The slot content was not successfully mounted

Version
2.6.10
Reproduction link
https://github.com/Veath/micro-frontend-qiankun
Steps to reproduce

installation and startup  portal and micro a project
click navigation in the browser ，click in order home 、 micor-a、 about links
about page  show blank

What is expected?
about page display text
What is actually happening?
about page display blank

Hey, this is not clear enough for a reproduction: why two apps? which one to open.
The only things I could see are

add a trailing slash to base: https://router.vuejs.org/api/#base
Do not add a domain to the publicPath (https://github.com/Veath/micro-frontend-qiankun/blob/master/micro-a/vue.config.js). That cannot work
To go to a different app under a different base, use a regular a with the proper url like /other-base/about

Hey, this is not clear enough for a reproduction: why two apps? which one to open.
The only things I could see are

add a trailing slash to base: https://router.vuejs.org/api/#base
Do not add a domain to the publicPath (https://github.com/Veath/micro-frontend-qiankun/blob/master/micro-a/vue.config.js). That cannot work
To go to a different app under a different base, use a regular a with the proper url like /other-base/about

two apps because I want to integrate multiple projects, like a single page.
add domain to the publicPath because the micor-a project will be run in portal page when the route ishttp://localhost:8080/vue/.  portal page fetch (the absolute path) micor-a  content.e.g html and js and runing.

startup portal (http://localhost:8080) and micor-a (http://localhost:8081)
open  http://localhost:8080/ in  browser,
click home link, page display content,  and than click micro-a project Home linkalso display content，and than click micro-a project About link display blank


(QA-2403) Change "pry-byebug" Version Pin

Merge #9 first!
The "3.3" version of "pry-byebug" is not compatible with Ruby 1.9.3.
Changing the version to 1.2.x to guarantee compatibility.

LGTM.


Got stuck at Creating completion session with arguments with full cpu usage

Hi! I am using cquery with LanguageClient-Neovim. However it seems that cquery got stuck after Creating completion session with arguments, occupying half of my cores while reporting progress as {"indexRequestCount":0,"doIdMapCount":0,"loadPreviousIndexCount":0,"onIdMappedCount":0,"onIndexedCount":0}.
Debug logs from LanguageClient-Neovim:
14:10:22 DEBUG   [MainThread] => {"jsonrpc":"2.0","method":"textDocument/didChange","params":{"textDocument":{"uri":"file:///home/lz/code/c10k-server/a.cpp","version":5},"contentChanges":[{"text":"#include <iostream>\n\nint main()\n{\n\tstd::cin.\n}\n"}]}}
14:10:22 DEBUG   [MainThread] => {"jsonrpc":"2.0","id":5,"method":"textDocument/completion","params":{"textDocument":{"uri":"file:///home/lz/code/c10k-server/a.cpp"},"position":{"line":4,"character":10}}}
14:10:22 DEBUG   [RPC-cpp   ] <= {"jsonrpc":"2.0","method":"$cquery/progress","params":{"indexRequestCount":0,"doIdMapCount":0,"loadPreviousIndexCount":0,"onIdMappedCount":0,"onIndexedCount":0}}
14:10:22 WARNING [RPC-cpp   ] no handler implemented for $cquery_progress
14:10:22 DEBUG   [RPC-cpp   ] <= {"jsonrpc":"2.0","method":"$cquery/progress","params":{"indexRequestCount":0,"doIdMapCount":0,"loadPreviousIndexCount":0,"onIdMappedCount":0,"onIndexedCount":0}}
14:10:22 WARNING [RPC-cpp   ] no handler implemented for $cquery_progress
14:10:22 DEBUG   [RPC-cpp   ] <= {"jsonrpc":"2.0","method":"$cquery/progress","params":{"indexRequestCount":0,"doIdMapCount":0,"loadPreviousIndexCount":0,"onIdMappedCount":0,"onIndexedCount":0}}
14:10:22 WARNING [RPC-cpp   ] no handler implemented for $cquery_progress
# No completion result received after 5 minutes

I tried to debug it and found threads occupying 100% CPU were all running std::string::_S_construct. GDB log is attached at https://gist.github.com/htfy96/61969e3939b69f9e3320e25ad63b5794, and here is
LanguageServer.log.
I was just wondering is this a bug or the expected behavior?

This feels like a bug to me. Can you try testing on a small project and see if it still happens with neovim?
Also, feel free to comment out EmitProgress in command_line.cc to remove the "$cquery/progress" messages.

@jacobdufault Actually, I tried it on my toy project https://github.com/htfy96/c10k-server with clang_args as follows:
-xc++
-std=c++14
-Isrc/
-Ivendor/Catch/include
-Ivendor/spdlog/include

More information could be found at https://github.com/autozimu/LanguageClient-neovim/issues/181

Let me give more input.
Dead lock may happen for trivial config like https://gist.github.com/anonymous/6e6cf05d934bd90dd27f2cd3051c84f4
One "indexer*" thread
llvm::RegisterTarget<(llvm::Triple::ArchType)18, false>::getArchMatch(llvm::Triple::ArchType) ()

Another "indexer*" thread
llvm::TargetRegistry::lookupTarget(std::string const&, std::string&) ()

(gdb) x/xg 0x7f030b2408a0
0x7f030b2408a0 <_ZZN4llvm18getTheX86_64TargetEvE15TheX86_64Target>:     0x00007f030b2408a0

Assuming clang_parseTranslationUnit2FullArgv can never be called concurrently I'm fine with serializing requests to it. It may be that we need to move all index jobs to a separate process - I'd like to avoid doing that though (that is how cquery used to be implemented, but it seemed to work well enough in-process, and in-process avoids a lot of extra complication).

One possible mitigation is to partially serialize indexer tasks, guaranteeing that the time different threads calling clang_parseTranslationUnit2FullArgv differs by at least, say 100ms.

If on-disk storage will ever be added to cquery, multi-process poses more problems because IPC is difficult and many embedded database does not have good multi-process support.

One possible mitigation is to partially serialize indexer tasks, guaranteeing that the time different threads calling clang_parseTranslationUnit2FullArgv differs by at least, say 100ms.

sg, though I think having a dedicated job queue for doing this would work faster (no need for 100ms delays). I need to re-investigate implementing a proper job system.

If on-disk storage will ever be added to cquery, multi-process poses more problems because IPC is difficult and many embedded database does not have good multi-process support.

I'd prefer to avoid on-disk storage unless it is relatively transparent to the rest of cquery. In that case the IPC setup should not care about the actual db storage.

Turns out that having a dedicated job/serializing these requests will really hurt index performance, as that is where most of the indexing work happens.
Ideally we can fix libclang here.
One alternative instead of a true multi-process model is to run clang_parseTranslationUnit and then call clang_saveTranslationUnit in a separate binary. The main cquery process then loads it via clang_createTranslationUnit. This will almost certainly hurt indexing performance but it may be significantly simpler than trying a full multi-process model.


Why do I get "Skpped" when trying to convert all files in folder?

Hi!
First of all, thanks for the tool!
I am using tochd v0.13 with Mint 21.3 (Ubuntu 22.04), installed the script and I can convert an individual file to .chd it works without any issues. But when I run the command inside a folder with several .iso files, I will get "Skipped" for all of them. I have tried different options but I haven't been able to make it work. I am sure it may be something quite simple, but I still wasn't able to make it work  : (
Any suggestions?

Ok. Yes, it was something quite silly, had typed wrong the *.iso.
So "tochd *.iso" (and with any other options) works fine

You're welcome, and glad you figured it out. You can also just do tochd *, and give all files over. The script will figure out which files are supported and not and what has been converted already. In example it skips, if there is already same file but with .chd extension already.
Instead files, you can also give a directory, which tochd will find all files in that directory and do the same as described above. For current folder it is enough to use the dot like tochd . .
These are just suggestions, in case you didn't know about.


Multiple WA settings

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request

Module Version

4.2.0

Affected Resource(s)

ArcGIS_Server

Configuration Files

{
    "AllNodes": [
        {
            "NodeName": "<IP_ADDRESS>",
            "Role": [
                "Server",
                "WebAdaptor"
            ],
            "WebAdaptorConfig":[
                {
                    "Role": "Server",
                    "ServerContext":"maps",
                    "AdminAccessEnabled": false
                },
                {
                    "Role": "Server",
                    "ServerContext":"maps-admin",
                    "AdminAccessEnabled": true
                }
            ]
        },
        {
            "NodeName": "<IP_ADDRESS>",
            "Role": [
                "Server",
                "WebAdaptor"
            ],
            "WebAdaptorConfig":[
                {
                    "Role": "Server",
                    "ServerContext":"maps",
                    "AdminAccessEnabled": false
                },
                {
                    "Role": "Server",
                    "ServerContext":"maps-admin",
                    "AdminAccessEnabled": true
                }
            ]
        }
    ],
    "ConfigData": {
        "Version": "11.1",
        "ServerRole": "GeneralPurposeServer",
        "ServerContext": "maps",
        "DownloadPatches": false,
        "ForceLicenseUpdate": false,
        "Credentials": {
            "ServiceAccount": {
                "Password": "xxx",
                "UserName": "svcusr",
                "IsDomainAccount": false,
                "IsMSAAccount": false
            }
        },
        "Server": {
            "LicenseFilePath": "C:\\prep\\licenses\\generated.prvc",
            "Installer": {
                "IsSelfExtracting": false,
                "Path": "C:\\prep\\preextractedInstalls\\ArcGIS 11.1\\ArcGISServer\\Setup.exe",
                "InstallDir": "C:\\Software\\ArcGIS\\Server",
                "InstallDirPython": "C:\\Python27",
                "PatchesDir": "C:\\Software\\PatchDownloads",
                "PatchInstallOrder": [

                ]
            },
            "ServerDirectoriesRootLocation": "\\\\<IP_ADDRESS>\\gis-server\\arcgisserver\\directories",
            "ConfigStoreLocation": "\\\\<IP_ADDRESS>\\gis-server\\arcgisserver\\config-store",
            "PrimarySiteAdmin": {
                "UserName": "theadmin",
                "Password": "xxx"
            }
        },
        "WebAdaptor": {
            "AdminAccessEnabled": true,
            "Installer": {
                "Path": "C:\\prep\\preextractedInstalls\\ArcGIS 11.1\\WebAdaptorIIS\\Setup.exe",
                "IsSelfExtracting": false,
                "DotnetHostingBundlePath": "C:\\prep\\installs\\11.1\\dotnet-hosting-6.0.27-win.exe",
                "WebDeployPath": "C:\\prep\\installs\\11.1\\WebDeploy_amd64_en-US.msi"
            }
        }
    }
}

Expected Behavior

I would expect that two contexts are configured, one with admin enabled, and one without.
Actual Behavior

Dot Sourcing the Configuration:- ArcGISInstall
ArcGIS\ArcGIS_Install : A duplicate resource identifier '[ArcGIS_Install]WebAdaptorIIS-Server-mapsInstall' was found while processing the specification for
node '<IP_ADDRESS>'. Change the name of this resource so that it is unique within the node specification.
At C:\Program Files\WindowsPowerShell\Modules\ArcGIS\4.2.0\Configurations-OnPrem\ArcGISInstall.ps1:480 char:29
+                             ArcGIS_Install "$($WAName)Install"
+                             ~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [Write-Error], ParentContainsErrorRecordException
    + FullyQualifiedErrorId : DuplicateResourceIdInNodeStatement,ArcGIS\ArcGIS_Install

ArcGIS\ArcGIS_Install : A duplicate resource identifier '[ArcGIS_Install]WebAdaptorIIS-Server-mapsInstall' was found while processing the specification for
node '<IP_ADDRESS>'. Change the name of this resource so that it is unique within the node specification.
At C:\Program Files\WindowsPowerShell\Modules\ArcGIS\4.2.0\Configurations-OnPrem\ArcGISInstall.ps1:480 char:29
+                             ArcGIS_Install "$($WAName)Install"
+                             ~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [Write-Error], ParentContainsErrorRecordException
    + FullyQualifiedErrorId : DuplicateResourceIdInNodeStatement,ArcGIS\ArcGIS_Install

Compilation errors occurred while processing configuration 'ArcGISInstall'. Please review the errors reported in error stream and modify your configuration code
appropriately.
At C:\Windows\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:3917 char:5
+     throw $ErrorRecord
+     ~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (ArcGISInstall:String) [], InvalidOperationException
    + FullyQualifiedErrorId : FailToProcessConfiguration

Steps to Reproduce

Deploy as defined
Important Factoids

I don't see any documentation precluding this approach, although the related code sample doesn't run multiple WAs on one machine.
The idea is that I can allow the admin URL internally to my org, but block it from the public.  I have a second site on the same subdomain so I can't just use /arcgis/ for that purpose. I'm really just exploring as the WebAdaptorConfig is pretty new.
References

4.2.0 variable
MultiWA Code Sample

Hi @jhevenor,
This a valid use case that should work. The error is indicating that both Node.Name entries in the json config were set to the same ip <IP_ADDRESS>. However, in the json example provided above, each Node.Name is unique and looks correct.
Perhaps at some point both node names were set to <IP_ADDRESS> then one was updated to <IP_ADDRESS> but the file was not saved before the DSC run?
Let's verify the json config is up to date with each NodeName value being unique and re-run.
Thanks,
Cameron K.

Hello again @cameronkroeker. You were right, I had a copy/paste that had svr1 listed twice.   I adjusted the original posts error message to show what I'm getting. It's still a dup config.  Even when I changed the contexts to something else (foo, fooadmin) I still see the [ArcGIS_Install]WebAdaptorIIS-Server-mapsInstall message given.  I'm running these in succession and I'm wondering if there's something cached that I need to clear?

Hello again @cameronkroeker. You were right, I had a copy/paste that had svr1 listed twice. I adjusted the original posts error message to show what I'm getting. It's still a dup config. Even when I changed the contexts to something else (foo, fooadmin) I still see the [ArcGIS_Install]WebAdaptorIIS-Server-mapsInstall message given. I'm running these in succession and I'm wondering if there's something cached that I need to clear?

Perhaps the LCM is using a cached MOF from it being stuck in a pending state. Let's try the following:

Close and launch a fresh powershell window
Run command Stop-DscConfiguration
Run command Get-DscLocalConfigurationManager to check to see what state LCM is in
Run command Remove-DscConfigurationDocument -Stage current,pending,previous to remove any current/pending/previous configurations
Run command Get-DscLocalConfigurationManager to check to see what state LCM
Re-run the ArcGIS DSC Module

I tried this with a fresh deployment and had the same errors.  Config:
{
    "AllNodes": [
        {
            "NodeName": "<IP_ADDRESS>",
            "Role": [
                "Server",
                "WebAdaptor"
            ],
            "WebAdaptorConfig":[
                {
                    "Role": "Server",
                    "ServerContext": "maps-cartes",
                    "AdminAccessEnabled": false
                },
                {
                    "Role": "Server",
                    "ServerContext": "maps-cartes-admin",
                    "AdminAccessEnabled": true
                }
            ]
        },
        {
            "NodeName": "<IP_ADDRESS>",
            "Role": [
                "WebAdaptor"
            ],
            "WebAdaptorConfig":[
                {
                    "Role": "Server",
                    "ServerContext": "maps-cartes",
                    "AdminAccessEnabled": false
                },
                {
                    "Role": "Server",
                    "ServerContext": "maps-cartes-admin",
                    "AdminAccessEnabled": true
                }
            ]
        }
    ],
    "ConfigData": {
   ...

Resulting in the same error:
Dot Sourcing the Configuration:- ArcGISInstall
ArcGIS\ArcGIS_Install : A duplicate resource identifier '[ArcGIS_Install]WebAdaptorIIS-Server-maps-cartesInstall' was
found while processing the specification for node '<IP_ADDRESS>'. Change the name of this resource so that it is
unique within the node specification.
At C:\Program Files\WindowsPowerShell\Modules\ArcGIS\4.2.0\Configurations-OnPrem\ArcGISInstall.ps1:480 char:29
+                             ArcGIS_Install "$($WAName)Install"
+                             ~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [Write-Error], ParentContainsErrorRecordException
    + FullyQualifiedErrorId : DuplicateResourceIdInNodeStatement,ArcGIS\ArcGIS_Install
...
# error repeats for other server

I'm going to drop this for time being but will hopefully get a chance to dig in again in a couple weeks when I finish some other things.


[FEA] Add 'Experimental' Repo

Morpheus needs a repo for features that aren't quite ready for general acceptance, so let's make one!

complete- https://github.com/nv-morpheus/morpheus-experimental


doit: 0.30.3 -> 0.31.0

Semi-automatic update. These checks were done:

built on NixOS
ran /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0/bin/.doit-wrapped --help got 0 exit code
ran /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0/bin/.doit-wrapped help got 0 exit code
ran /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0/bin/.doit-wrapped --version and found version 0.31.0
ran /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0/bin/doit --help got 0 exit code
ran /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0/bin/doit help got 0 exit code
ran /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0/bin/doit --version and found version 0.31.0
found 0.31.0 with grep in /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0
found 0.31.0 in filename of file in /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0

@GrahamcOfBorg eval

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘python3.6-pyinotify-0.9.6’ in /private/var/lib/ofborg/checkout/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/top-level/python-packages.nix:13557 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Partial log (click to expand)
Successfully installed doit-0.31.0
/tmp/nix-build-doit-0.31.0.drv-0/doit-0.31.0
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0/lib  /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0/bin
patching script interpreter paths in /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0
checking for references to /tmp/nix-build-doit-0.31.0.drv-0 in /nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0...
wrapping `/nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0/bin/doit'...
/nix/store/y6r8fjncv9axs205hmi0vjv8rb2s4i6v-doit-0.31.0

Success on aarch64-linux (full log)
Partial log (click to expand)
Successfully installed doit-0.31.0
/build/doit-0.31.0
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/i4lw4l9grrz8q7bycgz3370fcb6iq6xj-doit-0.31.0
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/i4lw4l9grrz8q7bycgz3370fcb6iq6xj-doit-0.31.0/lib  /nix/store/i4lw4l9grrz8q7bycgz3370fcb6iq6xj-doit-0.31.0/bin
patching script interpreter paths in /nix/store/i4lw4l9grrz8q7bycgz3370fcb6iq6xj-doit-0.31.0
checking for references to /build in /nix/store/i4lw4l9grrz8q7bycgz3370fcb6iq6xj-doit-0.31.0...
wrapping `/nix/store/i4lw4l9grrz8q7bycgz3370fcb6iq6xj-doit-0.31.0/bin/doit'...
/nix/store/i4lw4l9grrz8q7bycgz3370fcb6iq6xj-doit-0.31.0


Unable to change animated class name

This is:

Bug

Specifications

AOS version: 2.3.1
OS: Catalina
Browser: safari, google

Detailed Description

Any setting about ClassName is not work, for instance, replacing animatedClassName to 'animated', etc.

Any update on that one? I'm also facing the same issue.


Some date/time string are not formatted according to CLDR

Some date/time string are not formatted according to CLDR http://unicode.org/Public/cldr/30.0.3/
According to German CLDR definition (common/main/de.xml) , the date formatting "20. Febr. 2017" is incorrect, the abbreviated text for February in CLDR is "Feb", and applying follow the format
"d. MMM U" , it should be "20. Feb. 2017".

I think this is fixed by #3829 . What do you think @jha-nishant ?

closing in favor of #3829


LightSlider not adapting to different slide heights

When we have different heights to each slide, LightSlider will just take the highest height and set it as the height of the slider. When this is done, small images get a blank space underneath to fill the height. I tried to set "adaptiveHeight" both to true and false, and none of them had any effect.
Example in fiddle: http://jsfiddle.net/2patspw2/2648/
Can you help me? Is this a bug? Thanks!

I still facing same problem like you.
Anyone can suggestion?

Hello! I couldn't resolve this issue. I ended up using an algorithm to crop the images to a specific size (I was using Wordpress and all sliders had to be configurable). If you are not obligated to use  and you can set the height of the slider, a easier way would be to use s instead of  and s and set the background-image with background-size: cover.

Here is a fix that I just implemented in the file: lightslider.js
you will also need to add this to the lightslider.min.js if you want use the minified version.
line 934:
if (settings.adaptiveHeight === true && settings.vertical === false) { 					var sheight = $children.eq(scene).outerHeight(true);                     $el.css('height', $children.eq(scene).outerHeight(true)); 					$el.parent().css('height', sheight);                 }
line 1027:
if (settings.adaptiveHeight === true && settings.vertical === false) { 					var sheight = $children.eq(scene).outerHeight(true);                     $el.css('height', $children.eq(scene).outerHeight(true)); 					$el.parent().css('height', sheight);                 }


Add .asar support

It would be great if sharpcompress could handle .asar archives.

It’s for electron? https://github.com/electron/asar/
I’ll look into it.  Looks simple enough, unlike Tar :)

Yes, it is.
Glad to know that this would be "easily" possible. :)

I haven't personally tested this, but Razorvine.Pyrolite implements Pickle for a limited mapping of Python objects.  It is on GitHub, MIT licensed, targets .Net Standard 2.0, and the documentation explicitly says that the pickle and un-pickle (Protocol 2) can be used stand-alone (Razorvine.Pyrolite/Pickle)


VS generator RC file missing include paths

Xmake Version
master
Operating System Version and Architecture
Windows 11
Describe Bug
In RC files, one can use #include like in C/C++ files, currently this will not use the include paths specified for the project.
Expected Behavior
#include directives resolve like in C/C++ files.
Project Configuration
No response
Additional Information and Error Logs
No response

https://github.com/xmake-io/xmake/pull/2183


shim: allow for concurrent calls to handler

This is a mirror of this apex issue, specific for Up.

oops missed this, copied in dd950dc!


Login Page, Signup Page and Forgot Password Page

Add Login Page, Signup Page and Forgot Password Page
Login Page (with example input)

Signup Page (with example input)
(currently only implemented partial communication logic to backend since signup backend is not implemented yet)

Forgot Password Page (with example input)

#2 #6 #12

just wondering: is any of the login/signup/forgot password implemented yet as of now?

If I didn't understand your word wrongly, I think currently in this main repo, the login page is just a dummy page, and the signup page and forgot password page is empty (has no functionality)
https://github.com/CS3219-AY2425S1/cs3219-ay2425s1-project-g36/blob/main/frontend/src/pages/user-service/LoginPage.tsx
https://github.com/CS3219-AY2425S1/cs3219-ay2425s1-project-g36/blob/main/frontend/src/pages/user-service/SignupPage.tsx
https://github.com/CS3219-AY2425S1/cs3219-ay2425s1-project-g36/blob/main/frontend/src/pages/user-service/ForgotPasswordPage.tsx
And for this pull request, LoginPage and ForgotPasswordPage is fully implemented and SignupPage is implemented without communication to the backend

since milestone deadline is approaching, anymore reviews regarding this PR? if not can merge and close related issues, thanks

scope for milestone 3 is just containerisation of user service (the backend) and question service (backend) if you've seen in the chat so the current improvements here are just for future milestones, feel like they dw to bully us too much during midterm season
since we almost got penalised during the milestone 2 submission for doing user service early
however, this is very much in scope for milestone 4 so do keep working on it! will adjust the milestone deadlines accordingly

So you mean will merge this pr after captcha logic and signup API logic on the backend side is totally done and then I finish frontend logic according to the backend logic? If so seems that I'll wait to continue my work and merge this PR until backend logic is done. Thanks

So you mean will merge this pr after captcha logic and signup API logic on the backend side is totally done and then I finish frontend captcha logic and signup API logic according to the backend logic? If so seems that I'll wait to continue my work and merge this PR until the corresponding backend logic is done. Thanks

it's based on the given milestone requirements but since all of us are busy with midterms we also can't exactly work on the features now so yea do wait for now

I think we can merge this for now as can finally have a formal user service UI rather than the dummy one. For the parts that haven't implemented maybe can implement later (e.g. captcha)


Add dev VM to cors_domains

Added the VM to allowed CORS domains. Address is in cors_conf.py which is in .gitignore. Import is in a try/except block so the conf file doesn't need to be present in production or local developement.

Next time - let's try to fix these big commit chains


fix(list): 클래스 이름 수정

:tada: This PR is included in version 1.4.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(deps): update Dockerfile to SPDK longhorn-v24.09 branch

Which issue(s) this PR fixes:

Issue https://github.com/longhorn/longhorn/issues/9700
What this PR does / why we need it:
Special notes for your reviewer:
Additional documentation or context

@DamiaSan The PR doesn't pass the Conventional Commits Check.

@DamiaSan Some checks failed

Conventional Commits Check.

Done, now it want the prefix in PR's name too.

build / Build AMD64 binaries (pull_request)

It doesn't depend by this commit, it seems that in one test the tgt is not ready when the assert is checked.
If I run the same test for example on my PC or on an equinix hosted machine, all is ok.


About Navbar collapse

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the contributing guidelines

Proposal
When i reduce the screen size that time navbar-toggler-icon is come.
After that i click on particular text i navigate to another page but collapsed screen doesn`t hide.
i want to hide the scrren when i click the anchor tag.
please solve this issue in future update,
Thank You.

Motivation and context
When you make this change i dont nedd to write extra code for this.
and also in my point of view it is the most important update in bootstrap History.

IMO this is a normal behavior.

With anchor elements the page will be reloaded by default and so the menu will be collapsed.
If target="_blank" is used, the menu will stay open and that's normal for the user to continue to navigate.

IDK if you're using an SPA or something like that, but in this case, you probably need to close yourself the menu after clicking on an anchor link and routing the app to another page.

This sounds like a project-specific problem and not a bug in Bootstrap, so closing out. Please ask around in the Discussions or on Stack Overflow.


Search: Design Review

As discussed on a call just now, we'd like a design review of the new search pages, how they mesh together, how the new components looks, colour choices, etc. Dom will review them and let us know of any preferred changes, which we can then discuss here if need be.

As discussed, the overview page is in review and so isn't yet available on staging/main. Happy to send screenshots or do a screenshare if you did want to do this before it was merged in @DominiqueMarshall

@rcantin-w I don't mind waiting until it's on staging/been merged

@DominiqueMarshall it was approved quickly 😄 so it's now available.

(Comments in progress)

Stories headings are missing an underline hover state

Images tab - There’s not enough spacing between the bottom of the image filter buttons and the black background. Increase the margin to 32px to match the design: https://www.sketch.com/s/1cc6a024-668b-4d76-9617-115910d3a797/a/qemAoEy#Version

The ‘Reset filters’ selector needs reworking as it looks like another filter.

Font sizing between the filters, pagination and sort by are different and should be consistent (14px)

Border width should be 1px across all filter buttons. It's currently 1px on the sort filter and 2px on the others.

Image modals aren’t showing any titles

The arrows on the Overview buttons (all stories etc) are massive compared to the mockups. Decrease the sizing to 18px

Consider increasing the padding between the pagination at the bottom of the page and the footer.

Stories headings are missing an underline hover state ✅

Images tab - There’s not enough spacing between the bottom of the image filter buttons and the black background. Increase the margin to 32px to match the design: https://www.sketch.com/s/1cc6a024-668b-4d76-9617-115910d3a797/a/qemAoEy#Version
Done locally, I'll show you when we meet!

The ‘Reset filters’ selector needs reworking as it looks like another filter.
We can do some live coding together if it helps?

Font sizing between the filters, pagination and sort by are different and should be consistent (14px)

Border width should be 1px across all filter buttons. It's currently 1px on the sort filter and 2px on the others.

The two above I'd like to review with you because of the way they are coded. They are currently "buttons" and the padding doesn't match the "small" button. I can explain better when we share screen and discuss! I think a new component might be the way to go, or a different way of styling all the variables we're playing with at the moment.

Image modals aren’t showing any titles ✅

The arrows on the Overview buttons (all stories etc) are massive compared to the mockups. Decrease the sizing to 18px
This would be a change on all icons in buttons. Do we have other examples where we might think they are too big? otherwise maybe the icon svg itself needs a rework?

Consider increasing the padding between the pagination at the bottom of the page and the footer.
Let's discuss!

Leaving this here as discussed, need to decide if we want to try something new or stick with this until we've got a set "button strategy".

As explained here, we could be ready for round 2! I'll move it out of In progress and back to Next as this is more of a "parent" ticket. Let me know if there's anything!

We just had a chat and we're happy to close this! thanks so much @DominiqueMarshall ⭐


blake2s

There's support for blake2b. Why not blake2s too?

If someone has time to write an implementation, I would be happy to merge it.

https://github.com/ebfe/rust-blake2/tree/master/src
It exists!


Cache doesn't work because API always return HTTP OK

I'm not pretty sure if this repository is the right place to ask this kind of stuff, but, doing a request using cURL like the one presented below:
curl -v --header "If-None-Match: W\/\"{E-Tag}\"" https://myapp.zendesk.com/api/v2/help_center/pt-br/sections
Will always return a 200 OK response even if the E-Tag's match. Is this the expected result? Or the API should return a 304 NOT MODIFIED on the subsequent requests?

@nelsonsar: it's probably too late already, but it's better to write to<EMAIL_ADDRESS>for actual issues with the API. Closing this


feat: configurable date ranges

Description

[x] close #49
[x] add start_date and end_date as configuration props in report.toml
[x] change count_pull_requests query to utilize new date config
[x] change repo-participants query to utilize new date config

bors r+


[Bug]: Documentation has wrong component import

Tremor Version
latest
Link to minimal reproduction
none
Steps to reproduce
The documentation for the DatePicker has an incorrect component import. It's in the 'Usage example date picker' section.

What is expected?
import { DatePicker } from "@tremor/react";

What is actually happening?
import { DateRangePicker } from "@tremor/react";

What browsers are you seeing the problem on?
No response
Any additional comments?
No response

Thanks!


Profile gn gen and no-op engine builds.

This based on an observation that no-op engine builds during development were getting steadily slower over time.
Usually, even for no-op builds, Engine developers run gn gen (invoked by ./flutter/tools/gn) followed by the ninja invocation to generate build artifacts. Just running ninja is sufficient to pick up changes to GN rules themselves but this does not pick up changes to GN args. So, most just run the two invocations in order. This workflow has been getting alarmingly slow over time.
On my eng Macbook Pro, gn gen takes ~4s today. It used to take less than 300ms earlier (say 3 years ago).
$ time ./flutter/tools/gn --runtime-mode debug
Generating GN files in: out/host_debug
Generating Xcode projects took 241ms
Done. Made 739 targets from 228 files in 1608ms
./flutter/tools/gn --runtime-mode debug  3.98s user 3.40s system 263% cpu 2.799 total

Just running ninja on the other hand seems to take 300ms which seems fine to me.
$ time ninja -C out/host_debug_unopt -j999
ninja: Entering directory `out/host_debug_unopt'
ninja: no work to do.
ninja -C out/host_debug_unopt -j999  0.29s user 0.12s system 98% cpu 0.425 total

Taking >4s for a no-op builds breaks flow. I used to be able to build on save and I can't do that anymore. Also, given the trend, this is probably going to get worse without some effort to arrest the increase. I can't tell how much of the increase is from additional targets, additional use of scripts in GN, or, just the effects of running GN on my laptop in the WFH environment vs. using the beasts in the office.
GN and Ninja have built in tools to profile the various steps and we should track what the most expensive steps are. Typically, the guidance has been to avoid Python or Dart scripts referenced in GN rules as they are typically the bottleneck. To avoid having these scripts, newer versions of GN have built-in functions that we could probably use. I am positive there are tons of low hanging fruit that will significantly improve productivity of engine developers. Separately, we don't have (or adhere to) documented best practices for authoring GN rules. Perhaps now is the time to set that up. Also, perhaps there is a way to benchmark how long no-op builds take so we can figure out trends.

Ok, that was easier than I expected. As anticipated, it was script execution. A couple of the scripts, we can get rid of and use GN built-ins. There are some that can be parallelized. Generating Xcode projects can also probably be opt-in. Adding --tracelog=<file> in gn gen generates the following trace for just that one step.

The ios_sdk.py and mac_sdk.gni bottlenecks can be tackled by being smarter about when and how many times we generate symlinks for GOMA (cc @dnfield).
The bottlenecks then become script executions in Dart. Specifically, sdk_args.gni is in an import so it get executed multiple times and is slow. Perhaps that can be put in an arg. list_dart_files.py also seems to be slow and performing tasks redundantly. Is it possible to list the sources directly in GN rules like we do for all other targets? (cc @a-siva @zanderso).
Then there a are a couple scripts in Dart as well as engine that attempt to read Git revisions and such for versioning. This can also be done once upfront and stashed away in a GN variable.
Once these are tackled, I believe everything should complete in well under a second.

cc @aam @rmacnak-google: Is there a way we can reduce the usage of script files in GN?

I expect all the list_dart_files.py can be removed; I believe it predates the ability of our tools to generate depfiles.

xref https://dart-review.googlesource.com/c/sdk/+/196981. Thanks! I'll stash the iOS and Mac SDK details in a GN variable. I already patched compile_commands.json generation so its now not a separate step (saves 1 second). With the three fixes, I think we should be well under two seconds. I'll see if we can get rid of engine versioning from Git SHAs. We never exposed that to users so I don't think its absolutely necessary (we do use it to version caches).

I'll see if we can get rid of engine versioning from Git SHAs

We do have a test in the framework that verifies flutter_tester --version outputs the same hash that we asked to download from GCS:
https://github.com/flutter/flutter/blob/b8833afcc9b7939b7f3dd886ad60233aa75ede9c/dev/bots/test.dart#L137-L146

https://github.com/dart-lang/sdk/commit/8d436a20113d7032da6aa7097fec5d5818619b3e is causing unnecessary operations to run in no-op engine builds.
  With the above change, it's rerunning the list_dart_files_as_depfile.py actions and the Dart snapshotting actions that depend on those depfiles.
Can you revert https://github.com/dart-lang/sdk/commit/8d436a20113d7032da6aa7097fec5d5818619b3e?
Regarding the broader issue highlighted in this bug: in my workflow I typically only need to run gn after syncing to a new engine commit or selecting a new target configuration.  Rebuilding the engine after changing source code should only require running ninja, which is generally efficient.
@chinmaygarde what scenarios require frequently rerunning gn?

@jason-simmons Can you run ninja -d explain -C out/host_debug_unopt on the second run to see why ninja is doing work. AFAIK, such issues occur because of subtle difference in depfile listings. For instance, absolute vs relative paths.

Based on the explain output it looks like ninja expects the depfile to contain a relative path to the depfile itself.
@rmacnak-google I can get this to work as expected in the Flutter engine build by changing https://github.com/dart-lang/sdk/commit/8d436a20113d7032da6aa7097fec5d5818619b3e#diff-40dd701de08faf64c714b08263634be72ff06ced2cbb199e67c5ec4f516c2d9eR32
to use os.path.relpath(depfile)


Update utilities/arxiv.py

Refactor file removal logic in for-loop

Problem:
The file removal using os.remove(file_name) was placed inside a for-loop, causing unnecessary and potentially incorrect deletion attempts.

Fix:
Moved the file removal code outside the for-loop and added a check to ensure the file exists before attempting deletion. The updated code now includes the following check:
if os.path.exists(doc_file_name):
os.remove(doc_file_name)

I'm sure that this change improves the efficiency and accuracy of file deletion in the program.

I will review and request again.
Thanks.


feat: replace @type with type and roundTrip testing

WHAT
This PR replaces the reference to @type with type in the example and the required field section.
Additionally added the round trip testing of the examples JSON-LD processing + schema validation
Closes #88
How was the issue fixed?
Briefly state why the change was necessary.
More context
List other areas that have changed but are not necessarily linked to the main feature. This could be naming changes,
bugs that were encountered and were fixed inline, etc.

This is great. Are we actively deciding to keep the option between @type and type in the schemas? I'd feel more comfortable if we add "type": "@type" to the dcp context and remove @type from the schemas. That may however break non-json-ld-aware applications.

@wolf4ood is there anything additional you need to do before this is merged?

I'll add another issue for removing the @type option from the schemas.


[Bug]: ERROR: Failed to set up chrome-headless-shell v121.0.6167.85!

Error string
ERROR: Failed to set up chrome-headless-shell v121.0.6167.85! Set "PUPPETEER_SKIP_DOWNLOAD" env variable to skip download.
Background
I have a puppeteer project that I build in Github Actions to a Docker image.
Last week I was building this project fine, with no issues, but suddenly this week I started to get a puppeteer download error.
The following is my package.json:
{
  "name": "puppeteer-project",
  "version": "1.0.0",
  "main": "App.js",
  "type": "module",
  "scripts": {
    "start": "node src/App.js",
    "test": "c8 mocha"
  },
  "dependencies": {
    "cors": "2.8.5",
    "express": "4.18.2",
    "moment": "2.30.1",
    "pdfkit": "0.14.0",
    "puppeteer": "21.11.0"
  },
  "devDependencies": {
    "c8": "9.1.0",
    "chai": "5.1.0",
    "fetch-mock": "9.11.0",
    "mocha": "10.3.0",
    "nyc": "15.1.0",
    "pdfreader": "3.0.2",
    "sinon": "17.0.1"
  }
}

The following is my Dockerfile:
FROM node:20@sha256:cb7cd40ba6483f37f791e1aace576df449fc5f75332c19ff59e2c6064797160e

# https://pptr.dev/guides/docker
# https://github.com/puppeteer/puppeteer/blob/main/docker/Dockerfile
RUN apt-get update \
    && apt-get install -y wget gnupg \
    && wget -q -O - https://dl-ssl.google.com/linux/linux_signing_key.pub | gpg --dearmor -o /usr/share/keyrings/googlechrome-linux-keyring.gpg \
    && sh -c 'echo "deb [arch=amd64 signed-by=/usr/share/keyrings/googlechrome-linux-keyring.gpg] https://dl-ssl.google.com/linux/chrome/deb/ stable main" >> /etc/apt/sources.list.d/google.list' \
    && apt-get update \
    && apt-get install -y google-chrome-stable fonts-ipafont-gothic fonts-wqy-zenhei fonts-thai-tlwg fonts-khmeros fonts-kacst fonts-freefont-ttf libxss1 dbus dbus-x11 \
      --no-install-recommends \
    && service dbus start \
    && rm -rf /var/lib/apt/lists/* \
    && groupadd -r pptruser && useradd -rm -g pptruser -G audio,video pptruser

ENV DBUS_SESSION_BUS_ADDRESS autolaunch:

WORKDIR /app

COPY package.json .

RUN npm install

COPY src/* .

EXPOSE 8080

CMD [ "node", "App.js" ]

The following is my Github Actions:
name: main-build

on:
  workflow_dispatch:

jobs:

  build-image:
    runs-on: ubuntu-latest
    permissions:
      id-token: write
      contents: read
    steps:
      - id: checkout
        name: GIT checkout
        uses: actions/checkout@v4

      - name: Use Node.js 21.x
        uses: actions/setup-node@v3
        with:
          node-version: 21.x

      - name: Install dependencies
        run: npm install
        env:
          PUPPETEER_SKIP_DOWNLOAD: true

      - name: Run Tests
        run: npm test

      - id: ecr
        name: Setup ECR
        uses: ./.github/actions/ecr_setup

      - id: build
        name: Build
        env:
          DOCKER_REGISTRY: ${{ steps.ecr.outputs.docker_registry }}
          AWS_ACCESS_KEY_ID: ${{ steps.ecr.outputs.aws_access_key_id }}
          AWS_SECRET_ACCESS_KEY: ${{ steps.ecr.outputs.aws_secret_access_key }}
          AWS_SESSION_TOKEN: ${{ steps.ecr.outputs.aws_session_token }}
        run: |
          docker build -t ${{ env.DOCKER_REGISTRY }}/puppeteer-project:latest .
          docker push ${{ env.DOCKER_REGISTRY }}/puppeteer-project:latest

Whenever the npm install runs, either in the pipeline itself during the test phase (that for now I'm skipping because I'm not testing puppeteer) or the docker build phase, I get a download error.
Expectation
I'd expect both the test-phase and docker-image to build successfully, as last week it did.
Reality
However, I'm getting the following error:
#10 [5/6] RUN npm install
#10 13.53 npm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
#10 16.74 npm notice
#10 16.74 npm notice New minor version of npm available! 10.1.0 -> 10.4.0
#10 16.74 npm notice Changelog: <https://github.com/npm/cli/releases/tag/v10.4.0>
#10 16.74 npm notice Run `npm install -g<EMAIL_ADDRESS>to update!
#10 16.74 npm notice
#10 16.74 npm ERR! code 1
#10 16.74 npm ERR! path /app/node_modules/puppeteer
#10 16.75 npm ERR! command failed
#10 16.75 npm ERR! command sh -c node install.mjs
#10 16.75 npm ERR! Error: ERROR: Failed to set up chrome-headless-shell v121.0.6167.85! Set "PUPPETEER_SKIP_DOWNLOAD" env variable to skip download.
#10 16.75 npm ERR!     at file:///app/node_modules/puppeteer/lib/esm/puppeteer/node/install.js:81:27
#10 16.75 npm ERR!     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
#10 16.75 npm ERR!     at async Promise.all (index 1)
#10 16.75 npm ERR!     at async downloadBrowser (file:///app/node_modules/puppeteer/lib/esm/puppeteer/node/install.js:87:9) {
#10 16.75 npm ERR!   [cause]: Error: Download failed: server returned code 500. URL: https://edgedl.me.gvt1.com/edgedl/chrome/chrome-for-testing/121.0.6167.85/linux64/chrome-headless-shell-linux64.zip
#10 16.75 npm ERR!       at file:///app/node_modules/@puppeteer/browsers/lib/esm/httpUtil.js:64:31
#10 16.75 npm ERR!       at ClientRequest.requestCallback (file:///app/node_modules/@puppeteer/browsers/lib/esm/httpUtil.js:42:13)
#10 16.75 npm ERR!       at Object.onceWrapper (node:events:629:26)
#10 16.75 npm ERR!       at ClientRequest.emit (node:events:514:28)
#10 16.75 npm ERR!       at HTTPParser.parserOnIncomingClient (node:_http_client:693:27)
#10 16.75 npm ERR!       at HTTPParser.parserOnHeadersComplete (node:_http_common:119:17)
#10 16.75 npm ERR!       at TLSSocket.socketOnData (node:_http_client:535:22)
#10 16.75 npm ERR!       at TLSSocket.emit (node:events:514:28)
#10 16.75 npm ERR!       at addChunk (node:internal/streams/readable:376:12)
#10 16.75 npm ERR!       at readableAddChunk (node:internal/streams/readable:349:9)
#10 16.75 npm ERR! }
#10 16.75
#10 16.75 npm ERR! A complete log of this run can be found in: /root/.npm/_logs/2024-02-22T18_30_51_131Z-debug-0.log
#10 ERROR: process "/bin/sh -c npm install" did not complete successfully: exit code: 1

Puppeteer version
21.11.0
Node version
20.0.0
Package manager
npm
Package manager version
10.1.0
Operating system
Linux

Same issue here

I kind of fixed it by upgrading to version 22.2.0, guessing the binaries should be downloaded from here https://googlechromelabs.github.io/chrome-for-testing/ but for older versions they aren't. That's my guess because there is no version older than 122 in this page.
Although I still think the project should be compatible with older version.

My builds are also failing because of the 500 error on edgedl. I was able to get it working as per @schmittjoaopedro suggestion of updating to 22.2.0

My builds are also failing for the same reason. This download is failing for us and I can recreate by visiting the URL (500 error): https://edgedl.me.gvt1.com/edgedl/chrome/chrome-for-testing/115.0.5790.98/linux64/chrome-linux64.zip

I tried npm update, but I got the same error

if you want to get around this issue quickly, you can upgrade puppeteer to the newest release: 22.2.0, as @schmittjoaopedro mentioned.
modified my package.json:
-"puppeteer": "^21.3.7",
+"puppeteer": "^22.2.0",

Unfortunately we use a package we don't have control over and can't just update the version. Fingers crossed that server comes back.

Thanks for reporting. I think it’s a duplicate of https://github.com/puppeteer/puppeteer/issues/11967
Please switch to the alternative URLS used by newer versions or wait till the server is fixed (it’s outside of Puppeteer’s control).


Example box build

Might be helpful to some users.
Sorry if this is duplicate.

Thanks for the PR, but this is not how the settler box is built.


Allow setup.py to be merged with an existing cfg

This fixes #6.
I also added some convenience functions for the test utilities to make it easier to write these and future tests, as well as a tox.ini.
Might be a good idea to set up Travis CI.

Looks good :+1: Thanks!


[SP-3274][PDI-15922]-Missing data when using delimiters like "aa" in the CSV file

fix for regression BACKLOG-14781
fixed unittest

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/steps/csvinput/CsvInput.java
  engine/test-src/org/pentaho/di/trans/steps/csvinput/CsvProcessRowInParallelTest.java

Still Broken Tests:
org.pentaho.di.trans.steps.syslog.SyslogMessageConcurrentTest.concurrentSyslogMessageTest:
:red_circle: java.lang.Exception
java.lang.Exception: test timed out after 2000 milliseconds
	at java.lang.Class.getSuperclass(Native Method)
	at org.mockito.cglib.core.ReflectUtils.getClassInfo(ReflectUtils.java:427)
	at org.mockito.cglib.core.ReflectUtils$3.getClassInfo(ReflectUtils.java:403)
	at org.mockito.cglib.proxy.MethodInterceptorGenerator$1.transform(MethodInterceptorGenerator.java:71)
	at org.mockito.cglib.core.CollectionUtils.bucket(CollectionUtils.java:32)
	at org.mockito.cglib.proxy.MethodInterceptorGenerator.gene

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.auth.core.impl.DefaultAuthenticationConsumerFactory

Branch Change:  + 7.14%
Complexity Change:  + 7.69%

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -1.64%:small_red_triangle_down:
Line Change: -1.97%:small_red_triangle_down:

org.pentaho.di.trans.steps.csvinput.CsvInput

Branch Change: -.28%:small_red_triangle_down:
Complexity Change: -.80%:small_red_triangle_down:
Instruction Change:  + .13%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change:  + 2.35%
Complexity Change:  + 2.70%
Instruction Change:  + .35%
Line Change:  + 1.07%

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change:  + .53%


Failed to update nightly on Raspberry Pi

Here is what I got today:
$ rustup update
info: syncing channel updates for 'stable-armv7-unknown-linux-gnueabihf'
info: syncing channel updates for 'nightly-armv7-unknown-linux-gnueabihf'
info: latest update on 2018-04-27, rust version 1.27.0-nightly (7f3444e1b 2018-04-26)
error: component 'rust-docs' for 'armv7-unknown-linux-gnueabihf' is unavailable for download
info: checking for self-updates

       stable-armv7-unknown-linux-gnueabihf unchanged - rustc 1.25.0 (84203cac6 2018-03-25)
  nightly-armv7-unknown-linux-gnueabihf update failed - rustc 1.20.0-nightly (622e7e648 2017-06-21)

I got the same today on stable:
$ rustup update
info: syncing channel updates for 'stable-arm-unknown-linux-gnueabihf'
info: latest update on 2018-05-10, rust version 1.26.0 (a77568041 2018-05-07)
error: component 'rust-docs' for 'arm-unknown-linux-gnueabihf' is unavailable for download
info: checking for self-updates

  stable-arm-unknown-linux-gnueabihf update failed - rustc 1.20.0 (f3d6973f4 2017-08-27)

Same here, any update on the issue?

@lennartS Yes I can install beta (1.27.0) on my raspberry Pi now. But still not nightly (1.28.0).

Dup of #1410


Fix warning when returning from vm_loop.

Issue #930

LGTM (FWIW)

LGTM, but please add "Related issue: #930" to the commit message before land.


increasing number of output pdb

Is it possible to increase the number of output pdb?
For example, the default of AF and chai output 5 PDBs.

Hi, yes there’s an option for this called --diffusion_samples


Write a test that proves that you can override an env var with a blank string value in the configuration object

Done. You can override an environment variable with an empty string, but you can't make it undefined.
This means that you can override environment variables with explicit values, but you cannot fully unset them.


feat: support tgz for bundled plugins

closes #48
This feels like the code could be dry'd a bit, maybe doing a seperate file extension lookup, then checking the naming convention, then offloading to the correct downloader, but possibly fine for now.
can be tested by
git@github.com:pact-foundation/pact-plugins.git
git checkout feat/targz_plugin_binaries
cd cli
cargo run install https://github.com/austek/pact-avro-plugin/releases/tag/v0.0.4

This will fail, is run from master or from the released binary
pact-plugin-cli install https://github.com/austek/pact-avro-plugin/releases/tag/v0.0.4

The JVM driver will need to be updated to support this as well.


Select just id on withRelated

Hi there,
I have a problem when I want to get a list of id of the related table.
I have a Card model with a Model Feedback  connected via card_id
This is the connection inside the Card:
    feedback: function() {
      return this.hasMany(Feedback);
    },

And then when I query the Card, I want an array of all related ids inside the Card.
Here is the code I'm trying to get this done:
this.forge().query(function(qb) {
        qb.where('status', '!=', cardStatus.DELETED);
        qb.andWhere('public_id', '=', publicId);
      })
      .fetch({
        withRelated: [
          {
            feedback: function(qb) {
              qb.select(['id']);
            }
          }
        ]
      });

The problem I have is that the related feedback is an empty array.
As soon as I add card_id to the select array, it works, but without it it's always an empty array.
Any idea how to solve this?
I'm using bookshelf v0.9.5.

Actually No. because without the card_id field its impossible to attach feedbacks to their cards.
So, I suggeste you to add a method to Card model which retrieves a card and its feedback ids
Card.findWithFeedbacks = function (publicId) {
  return this.forge().query(function(qb) {
    qb.where('status', '!=', cardStatus.DELETED);
    qb.andWhere('public_id', '=', publicId);
  })
  .fetch().then(card => {
    return card.feedback().pluck('id')
    // store the ids somewhere and return the card
    .then(ids => card.fbIds = ids).return(card)
  })
}

It's not a optimal solution, but it will give an insight to a better one

Currently using a similar workaround with pluck(). Suspect that wanting to fetch just the ids is a fairly common use-case.
Read through available bookshelf plugins on npm. None seem to support id only on withRelated.
@rhys-vdw Would you be open to a PR that used column alongside withRelated,e.g.?
Parent.forge().fetch({withRelated: ['child'], columns: ['parent_attr', 'child.id']})

This issue is closed, however I'm also interested in this behaviour.
It seems very similar to #937 - I think there is room to expand the ability of bookshelf to pass options to the relation queries?


In game is wrong

Few things, first league updated their in game scoreboard and the in game tab and team games need to be updated with positions and color. The scoreboard now shows turrets natively so that can be removed. Third the level up seems bugged and doesn’t show the color behind the number instead it just shows the black text number.
Thank you for making this free and public, your work is very appreciated!

Hi there :)
We have already fixed the overlay issues coming from the spectator changes in the spring-cleanup branch, but have not merged the changes yet since it also contains some structural changes that aren't finished. A workaround is below.
As for the color, could you make sure you don't have any custom CSS in your OBS browser source? The default CSS included with every browser source likes to eat colors and placements.
Workaround for spectator changes:

Download the updated modules for ingame  and teams
Delete the contents of the folders ...\league-prod-toolkit-develop\modules\league-in-game and ...\league-prod-toolkit-develop\modules\rcv-teams
Copy the contents from the downloaded modules into their respective folders
edit config.json in ...\league-prod-toolkit-develop\modules\config and rename the league-in-game object to module-league-ingame:

"module-league-in-game": {
    "items": [],
    "level": []
  }

Open PowerShell and navigate to the toolkit (or shift-rightclick in the toolkit directory to open PowerShell directly)
Run npm run build:modules
Run npm start or start.bat

resolved with release 0.9, updating by hand is no longer necessary


Update react-native-libraries.json

Why
turns out we have some more to in order to fully support dualscreen on Windows
(sorry for the noise :) )
CC @kmelmon

np, thanks for updating!


[suggestion] support pattern array in skipInterpolation meta option

I think sometimes only one pattern can't describe the rules of a complex template very well. So how about pattern array? For example:
{
  "skipInterpolation": [
    "src/**/*.{vue,js}",
    "assets/**/*",
    "ios/**/*",
    "android/**/*",
    ...
  ]
}

Thanks.

I'll give it a shot

I'll give it a shot


Adds jq to dockerfile

@saivarunr What are you trying to achieve? Contrary to the title the pull requests adds a bunch of more changes.
What would be the reasoning for adding jq?


Issue while deploying Azure IoT pcs device simulator

Type of issue

[ ] Bug
[ ] New feature
[ ] Enhancement

Description
I am trying to deploy the Azure IoT device simulation. I followed the mentioned instructions for visual basic deployment but I am getting errors while running "npm run deploy".  I also followed the steps from issue #34. But now i am getting this error on running "npm run deploy":
...
Steps to reproduce
Bring up deployment folder in Visual Studio Code
Modify the parameters.json and add the config values
Run the command npm install from the terminal window in VS Code. It ran cleanly.
In the same window, run npm run deploy and it generates the errors above.
Expected behavior
...
Current behavior
src/aadApp.ts:49:9 - error TS2322: Type 'string | null | undefined' is not assignable to type 'string'.
Type 'undefined' is not assignable to type 'string'.
49         return appCreationResponse.appId;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/auth.ts:84:9 - error TS2322: Type 'null' is not assignable to type 'credentials'.
84         return null;
~~~~~~~~~~~~
src/auth.ts:108:13 - error TS2322: Type 'null' is not assignable to type 'credentials'.
108             return null;
~~~~~~~~~~~~
src/deploymentManager.ts:50:48 - error TS2345: Argument of type 'string | undefined' is not assignable to parameter of type 'string'.
Type 'undefined' is not assignable to type 'string'.
50     await assignContributorRoleOnResourceGroup(rgResult.name, creds, sp.id, config);
~~~~~~~~~~~~~
src/deploymentManager.ts:73:5 - error TS2322: Type 'string | null | undefined' is not assignable to type 'string'.
Type 'undefined' is not assignable to type 'string'.
73     parameters.aadClientId.value = sp.appId;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/deploymentManager.ts:75:5 - error TS2322: Type 'string | undefined' is not assignable to type 'string'.
Type 'undefined' is not assignable to type 'string'.
75     parameters.aadClientServicePrincipalId.value = sp.id;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/deploymentManager.ts:77:5 - error TS2322: Type 'string | undefined' is not assignable to type 'string'.
Type 'undefined' is not assignable to type 'string'.
77     parameters.storageEndpointSuffix.value = creds.environment.storageEndpointSuffix;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/deploymentManager.ts:113:58 - error TS2532: Object is possibly 'undefined'.
113         throw new Error(Deployment validation failed: ${validationResponse.error.message});
~~~~~~~~~~~~~~~~~~~~~~~~
src/deploymentManager.ts:133:62 - error TS2345: Argument of type 'string | undefined' is not assignable to parameter of type 'string'.
Type 'undefined' is not assignable to type 'string'.
133     const lro = await client.deployments.beginCreateOrUpdate(rg.name, buildDeploymentName(config), deployment);
~~~~~~~
src/deploymentManager.ts:146:8 - error TS2454: Variable 'latestState' is used before being assigned.
146     if(latestState === 'Failed'){
~~~~~~~~~~~
src/deploymentManager.ts:149:35 - error TS2454: Variable 'latestState' is used before being assigned.
149         console.log(Deployment ${latestState}.);
~~~~~~~~~~~
Found 11 errors.
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>build: tsc
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
Known workarounds
...
Possible solution
...
Context and Environment
npm version:
Windows 10
I downloaded the rep from github: https://github.com/Azure/azure-iot-pcs-device-simulation.git.

Operating System: ...
Windows 10
GitHub branch: ...
.NET Runtime: ...
2.1.812

Would you please let me know the version of Node.js do you have installed? I'm running v15.5.1 on my machine.
Do you see any output that says "AAD application '' created..."?
It appears as if AAD application registration is failing without an exception being thrown.

Would you please let me know the version of Node.js do you have installed? I'm running v15.5.1 on my machine.
Do you see any output that says "AAD application '' created..."?
It appears as if AAD application registration is failing without an exception being thrown.

Also, do you have permissions/authorization/privileges in your Azure subscription to create AAD application registrations? Perhaps this is an AAD auth issue and ADD is returning a warning/error message that the code should be catching (a test that I didn't think of) and because the call to AAD wouldn't throw an exception in this case, we end up with this error.

Also, do you have permissions/authorization/privileges in your Azure subscription to create AAD application registrations? Perhaps this is an AAD auth issue and ADD is returning a warning/error message that the code should be catching (a test that I didn't think of) and because the call to AAD wouldn't throw an exception in this case, we end up with this error.

I created pull #44 to hopefully catch this case and output an error message. I wasn't able to repro the issue on my end though. I removed the app admin role for my account and ran a deployment but, for me, the call to AAD to register the app threw an exception, as expected. I don't understand why the app registration is apparently failing without the Graph API client throwing.
Please pull the code from pull #44 and see if you receive the new error (that would at least tell us that AAD app registration is failing for sure).

I created pull #44 to hopefully catch this case and output an error message. I wasn't able to repro the issue on my end though. I removed the app admin role for my account and ran a deployment but, for me, the call to AAD to register the app threw an exception, as expected. I don't understand why the app registration is apparently failing without the Graph API client throwing.
Please pull the code from pull #44 and see if you receive the new error (that would at least tell us that AAD app registration is failing for sure).

I created pull #44 to hopefully catch this case and output an error message. I wasn't able to repro the issue on my end though. I removed the app admin role for my account and ran a deployment but, for me, the call to AAD to register the app threw an exception, as expected. I don't understand why the app registration is apparently failing without the Graph API client throwing.
Please pull the code from pull #44 and see if you receive the new error (that would at least tell us that AAD app registration is failing for sure).

Hello, I pulled your code and got the new error (as you mentioned). I was again getting few errors but I solved them by changing the node.js version. The workspace was somehow taking the older version of it. So now everything is set.
Thanks!

I created pull #44 to hopefully catch this case and output an error message. I wasn't able to repro the issue on my end though. I removed the app admin role for my account and ran a deployment but, for me, the call to AAD to register the app threw an exception, as expected. I don't understand why the app registration is apparently failing without the Graph API client throwing.
Please pull the code from pull #44 and see if you receive the new error (that would at least tell us that AAD app registration is failing for sure).

Hello, I pulled your code and got the new error (as you mentioned). I was again getting few errors but I solved them by changing the node.js version. The workspace was somehow taking the older version of it. So now everything is set.
Thanks!


feat: wallet backend communication

Description

[ ] start wallet when unlocking
[ ] get wallet identity from backend
[x] get wallet balance when showing wallet page
[ ] get both string and emoji representation of wallet address from backend

Motivation and Context
#257
How Has This Been Tested?

balance update pending:

@alex1307
I'm getting GRPC error: status: Unknown, message: "transport error", details: [], metadata: MetadataMap { headers: {} }
when calling wallet_identity from frontend
could you take a look at it?


Allow the workflow to push directly to the default branch

For complete automation it was required that files are synced without the need for user interaction. If the option SKIP_PR is set to true (defaulted to false), it will push the changes without a PR.

Hey @villainous-hippogriff, what wasn't working with this PR? Are going to create a new one?

Hey @villainous-hippogriff, what wasn't working with this PR? Are going to create a new one?

Created a new one,
https://github.com/BetaHuhn/repo-file-sync-action/pull/14


Security Vulnerability: merge > 1.2.1

Review and fix the merge security vulnerability listed on GitHUb.

This security vulnerability is six levels deep in the heirarchy.  aws-appsync-codegen needs to upgrade their usage of Jest in order to update this.


Space suggestion: Infinite earrings

The space is defined by $(\mathbb{Z}\times S^1)/\sim$, where $(m,x)\sim (n,y)$ if and only if $x=y=1$. Equivalently, it can be defined as $\mathbb{R}/\mathbb{Z}$, identifying all points of $\mathbb{Z}$.
The space is sequential, not first countable (https://math.stackexchange.com/q/1017757/), and not locally compact (https://math.stackexchange.com/questions/118005/).

We already have this space: https://topology.pi-base.org/spaces/S000139

We have several spaces including S139 with these properties. https://topology.pi-base.org/spaces?q=Sequential%2B~First+countable%2B~Locally+compact
But we should probably add "countable wedge sum" to the aliases for S139 so it can be more easily found, if that's how folks are searching for it.

By the way (not related to this post), although the space S139 is not homeomorphic to $\displaystyle\bigcup_{n=1}^\infty {(x,y):(x-n)^2+y^2=n^2}$, the two spaces are homotopically equivalent (see https://math.stackexchange.com/q/1741457/1039170), meaning that it is actually more difficult to distinguish these two spaces than to distinguish S139 and Hawaii earring.

The last space you mention does not seem to be closed in $\mathbb R^2$: it's missing the $y$-axis.  And it does not seem to be locally compact, but neither is S139.  But it is first countable, even second countable as a subspace of $\mathbb R^2$, whereas S139 is not.  It could be possibly be added to pi-base as a contrasting case with S139.  Even more instructive would be to add the Hawaiian earring, I would say.

I personally know Hawaiian colleagues who would strongly prefer the so-called "Hawaiian earring" not be called such, so I'll exercise my editorial privilege to say we need a different canonical name for it on the pi-Base. See the conversation at https://en.wikipedia.org/wiki/Talk:Hawaiian_earring (which is as contentious as one might guess such topics on the internet might be). "Infinite earring" (as used by Munkres) is a more semantic description as well.
But yes, I'd like to see this standard example added to pi-Base.

The last space you mention does not seem to be closed in R 2 : it's missing the y -axis. And it does not seem to be locally compact, but neither is S139. But it is first countable, even second countable as a subspace of R 2 , whereas S139 is not. It could be possibly be added to pi-base as a contrasting case with S139. Even more instructive would be to add the Hawaiian earring, I would say.

Ah yes, I missed the fact that $\bigcup_{n=1}^\infty \{(x,y):(x-n)^2+y^2=n^2\}$ is not locally compact. In a metric space, local compactness at a point is equivalent to some closed ball centered at the point being compact, but for this space no such closed ball is compact because $y$-axis is missing. Thanks!

I personally know Hawaiian colleagues who would strongly prefer the so-called "Hawaiian earring" not be called such, so I'll exercise my editorial privilege to say we need a different canonical name for it on the pi-Base. See the conversation at https://en.wikipedia.org/wiki/Talk:Hawaiian_earring (which is as contentious as one might guess such a topic on the internet would be). "Infinite earring" (as used by Munkres) is a more semantic description as well.
But yes, I'd like to see this standard example added to pi-Base.

That's a good name. How would you prefer to call $\bigcup_{n=1}^\infty \{(x,y):(x-n)^2+y^2=n^2\}$ (I hate to type this space for each time)? "Stretching" infinite earring?

"Stretching" infinite earring?

Possibly, but we first need to do at least one of the following:

Find where the space is used in the literature (and use any appropriate canonical name if possible)
Find a search query that we cannot answer without this space

(Disclaimer: I'm guilty of introducing https://topology.pi-base.org/spaces/S000199 but I guess that hints at a third possible rationale: it was a convenience in order to provide a useful characterization of https://topology.pi-base.org/spaces/S000044 as a product.)

Edited title to reflect where the conversation led to.

I personally know Hawaiian colleagues who would strongly prefer the so-called "Hawaiian earring" not be called such, so I'll exercise my editorial privilege to say we need a different canonical name for it on the pi-Base. See the conversation at https://en.wikipedia.org/wiki/Talk:Hawaiian_earring (which is as contentious as one might guess such a topic on the internet would be). "Infinite earring" (as used by Munkres) is a more semantic description as well.

Yes, I remember that conversation.  With due respect, one could view those who oppose the Hawaiian earring name as driven by a misguided sense of ... something.  I would be curious, are those colleagues you refer to actually native Hawaiians, or whites who think they know better?  The canonical name in the literature is Hawaiian earring, and that should at least be an alias.

I won't go into more detail on this forum because they got harrassed for it, but the particular vocal opponent of the name who I have in mind is a native Hawaiian, yes.
I agree it should be an alias; besides, we will likely cite https://en.wikipedia.org/wiki/Hawaiian_earring as a reference.

Sadly, I only find "inverse Hawaiian earring" once on the Internet, in this link: https://mathoverflow.net/questions/43411/homotopical-immersion-of-the-wedge-product-of-countable-many-circles-in-rn

My take is that S201 Infinite earring is important as a well-established example to contrast with S139 Countable bouquet of circles (and maybe we should add cross references between their descriptions to point this out). But this inverse infinite earring feels less motivated and at this time isn't a useful example to add, unless it is able to answer a question open to pi-Base.

I believe this issue is settled; please re-open if I'm overlooking anything.


Backport several additions / fixes to PHP / Ruby xDS Interop tests

Add support to PHP timeout test: https://github.com/grpc/grpc/pull/25696
Add support to PHP fault_injection test: https://github.com/grpc/grpc/pull/25943
Fix Ruby fault_injection test: https://github.com/grpc/grpc/pull/26006
Fix PHP and Ruby header_matching test: https://github.com/grpc/grpc/pull/26017

Adhoc run, all ruby test cases passed: https://fusion2.corp.google.com/invocations/3ca71c75-00c8-4a68-9c01-1c736b1d6a86/targets

PHP also all passed: https://fusion2.corp.google.com/invocations/f68fe335-ad1d-4727-a863-0b1c0d60f51e/targets


Update version for the next release (v0.10.2)

This version makes this package compatible with Meilisearch v1.0.0 :tada:
Check out the changelog of Meilisearch v1.0.0 for more information on the changes.

Not required


Please add support for Amazon website in Chinese!

I am a Chinese Kindle User and I have tried the global one, and ended up in vain.

I am convinced that many of the users of the plugin is from China, so it is necessary to add support for the Chinese website: Amazon.cn.
Last but not least, thank you for your developing for people who love reading!

Can you confirm that your books are accessible on this URL please: https://read.amazon.cn/notebook

I believe this is a show stopper.
The plugin's cloud sync works by accessing Amazon's online Kindle Reader and scraping the HTML. Checkout the following screenshot to see what that read looks like.

It seems that Amazon does not have such an experience available for Chinese users. I just had a read online and there are several news reporting that Kindle books are no longer available in China as of this year.
Your other option hence is to sync your highlights using the My Clippings file on the device using a USB cable.

It's a real shame, but thanks for your work! Maybe Amazon didn't offer this kind of online note viewing solution for Chinese users from the beginning 🥲