[WIP] Store RLP encoding to avoid encoding operations

PR description
The goal of this PR is to store RLP encoding bytes insides different objects  (transaction, transactionReceipt, BlockBody and BlockHeader) to avoid encoding operations when we need the get the RLP encoding.
This PR has also an improvement on Sync process to avoid transaction root validation and receipt root validation twice.
Before this PR

After this PR (We can notice that receipts root validation appears only once)

Fixed Issue(s)

Close to split into two PRs


⚠ deces.matchid.io search API has degraded performance

In 3b7c36b, deces.matchid.io search API (https://deces.matchid.io/deces/api/v1/search?q=pompidou) experienced degraded performance:

HTTP code: 200
Response time: 3253 ms

Resolved: deces.matchid.io search API performance has improved in ef3c7f8 after 12 minutes.


<EMAIL_ADDRESS>download via aria2 failed

Installing 'tmac' (6.0.7) [64bit]
Starting download with aria2 ...
Download: 10/06 01:09:31 [ERROR] CUID#8 - Download aborted. URI=https://technitium.com/download/tmac/TMACv6.0.7_Setup.zip
Download: Exception: [AbstractCommand.cc:351] errorCode=3 URI=https://technitium.com/download/tmac/TMACv6.0.7_Setup.zip
Download:   -> [HttpSkipResponseCommand.cc:219] errorCode=3 Resource not found
Download: Download Results:
Download: gid   |stat|avg speed  |path/URI
Download: ======+====+===========+=======================================================
Download: 97b55d|ERR |       0B/s|C:/Users/me/scoop/cache/tmac#6.0.7#https_technitium.com_download_tmac_TMACv6.0.7_Setup.zip
Download: Status Legend:
Download: (ERR):error occurred.
Download: aria2 will resume download if the transfer is restarted.
Download: If there are any errors, then see the log file. See '-l' option in help/man page for details.
ERROR Download failed! (Error 3) Resource was not found
ERROR https://technitium.com/download/tmac/TMACv6.0.7_Setup.zip
referer=https://technitium.com/download/tmac/
dir=C:\Users\me\scoop\cache
out=tmac#6.0.7#https_technitium.com_download_tmac_TMACv6.0.7_Setup.zip
ERROR & 'C:\Users\me\scoop\apps\aria2\current\aria2c.exe' --input-file='C:\Users\me\scoop\cache\tmac.txt' --user-agent='Scoop/1.0 (+http://scoop.sh/) PowerShell/7.0 (Windows NT 10.0; Win64; x64; Core)' --allow-overwrite=true --auto-file-renaming=false --retry-wait=2 --split=5 --max-connection-per-server=5 --min-split-size=5M --console-log-level=warn --enable-color=false --no-conf=true --follow-metalink=true --metalink-preferred-protocol=https --min-tls-version=TLSv1.2 --stop-with-process=10072 --continue

Url fixed, thanks.  but installation is still failing. EXE is a bit tricky and I had to open it in parser mode to be able to extract it.


Why do ctc.py and mmi.py use use_double_scores=True?

I am referring to the following:
https://github.com/k2-fsa/snowfall/blob/b7f76b663ba61c42c7b63aed4a101ed4e3f0fc16/snowfall/objectives/ctc.py#L38
https://github.com/k2-fsa/snowfall/blob/b7f76b663ba61c42c7b63aed4a101ed4e3f0fc16/snowfall/objectives/mmi.py#L85
This is a bit unusual to me. Was there a particular motivation? @csukuangfj it seems like you were the one who chose to use double instead of single precision.

I believe double precision is more accurate for log_sum_exp.
Maybe @danpovey has more experience about this.
I have not compared the speed and accuracy between single and double precision.

It was out of a concern that for long utterances, we might get roundoff errors being different in the forward vs backward computions, and posteriors that don't sum to 1, causing possible lack of cancellation between num and den.
However I don't recall whether this was an actual issue in practice.  We should test the effect on speed and WER again.

Okay. I definitely think it makes sense for long utterances to use double if we are in probability space. I will keep this in my mind as a potential knob to tune.

They are log-probs not probs (overflow/underflow are not the issue).
On Thu, May 6, 2021 at 11:47 PM Daniel Galvez @.***>
wrote:

Okay. I definitely think it makes sense for long utterances to use double
if we are in probability space. I will keep this in my mind as a potential
knob to tune.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/snowfall/issues/183#issuecomment-833629672,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO3V3JDOGJP3WANNGVLTMK22NANCNFSM44GMSH4A
.

Don't worry, I know that part (it would be concerning if I didn't!)


Provide examples for authentication etc.

Hey!
I love the idea behind this package, but have some troubles with getting started, as I am new to Supabase and PostgREST.
I would love to see some more examples on the usage of this package (with supabase), namely authentication.

(How) should I use the client.TokenAuth method?
Can I encode the credentials into the HTTP URI somehow?

For anything I tried, I get some not-so-descriptive error (like just EOF from net/url) which is very likely because I did something wrong.
All the examples I can find are just for a local setup without auth.
Maybe adding a few more tests would be ideal for this? I would love to contribute once I used this a bit more.
I would greatly appreciate any help :)

If it's okay, could you post the code snippet of the part where you got the error? Or the error itself?

Yeah sure, thanks!
At first I tried something like this:
connUrl := "postgresql://" + user + ":" + password + "@" + host + ":" + port + "/" + dbName

client := postgrest.NewClient(connUrl, "public", nil)
res, err := client.From("my_table").Select("*", "", false).Execute()

if err != nil {
    // I get an error from net/url: "invalid port \":myPassword after host"
	panic(err)
}

fmt.Println(res)

But having the credentials in the URL did not seem to work (even when I did connUrl = url.QueryEscape(connUrl) I got an error).
Then I tried to use the TokenAuth method like this:
client := postgrest.NewClient(connUrl, "public", nil)

client.TokenAuth("mySupabaseKey")

res, err := client.From("my_table").Select("*", "", false).Execute()

if err != nil {
    // I get a net/url Error: "EOF"
	panic(err)
}

using the API key I got from the supabase settings.
Does this loke somewhat right to you?

Alright, so I was able to fix my problems! One of them was simply me doing mistakes (this is why I would love better docs), but I think I found two bugs as well?

I got an error from Supabase/Kong: No API key found in request

Was able to fix this by adding an apikey header to the request, next to the Authorization one. (It is stated in the supabase docs, that this is required)

One of the headers in client.go caused an issue. Removing the extra  : solved it. I opened a PR for both fixes.

Yes, there is some deficiency in the document part, I will complete those parts as soon as possible, I am also looking at the errors you sent.
Thanks

I'm looking at the pr you sent

@muratmirgun There is still no example for the Auth on the README or at pkg.go.dev
I am thinking about using Supabase for multiple projects, but I have to integrate it with my own backend, so a documentation how to connect with Authentication would be really nice for people like me.

would also very much appreciate an auth example if this is possible


Option to use NAT instance with existing EIP

This PR gives support for static EIP by allowing instance to allocate itself an EIP passed as parameter to its public interface (eth0). This ensures network always flows through the same IP even in case of instance recreation / failure.
A few points to note in this PR:

Once assigned, a sleep 3 is issued to ensure the instance network networking is functional as I encountered cases where a direct egress interaction following EIP change would result in failure
I added IAM permissions on the documentation Cloudformation, without adding a way to actually pass an EIP allocation as I am fairly unfamiliar with Cloudformation
A few user data API calls were shifted from the eni_id scope to the global scope so that those variables may be reused in other blocks (in this case eip_id)

This PR only adds support for the service to have an EIP. This change is compatible with current implementations of CDK and Cloudformation, but they will need to be updated to pass the eip_id parameter to the configuration in order to have static EIP support.

lgtm

Is the difference here that an existing EIP can be associated with the fck-nat gateway?
We're currently doing this:
const elasticIp = new ec2.CfnEIP(ctx.stack, 'elasticIp', {
    domain: 'vpc',
  });

  const natGatewayProvider = new ec2.NatInstanceProvider({
    instanceType: ec2.InstanceType.of(ec2.InstanceClass.T4G, ec2.InstanceSize.NANO),
    machineImage: new ec2.LookupMachineImage({
      name: 'fck-nat-amzn2-*-arm64-ebs',
      owners: ['568608671756'],
    }),
  });

  vpc = new ec2.Vpc(ctx.stack, 'vpc', {
    maxAzs: 1,
    subnetConfiguration: [
      {
        name: 'privateLambda',
        cidrMask: 24,
        subnetType: ec2.SubnetType.PRIVATE_WITH_EGRESS,
      },
      {
        name: 'public',
        cidrMask: 24,
        subnetType: ec2.SubnetType.PUBLIC,
      },
    ],
    natGateways: 1,
    natGatewayProvider,
  });

  const eipAllocationIds = [elasticIp.attrAllocationId];
  natGatewayProvider.configuredGateways.forEach((gateway, idx) => {
    new ec2.CfnEIPAssociation(ctx.stack, `EIPAssociation${idx}`, {
      allocationId: eipAllocationIds[idx],
      instanceId: gateway.gatewayId,
    });
  });

  vpcPrivateSubnets = vpc.selectSubnets({ subnetType: ec2.SubnetType.PRIVATE_WITH_EGRESS });
  vpcPublicSubnets = vpc.selectSubnets({ subnetType: ec2.SubnetType.PUBLIC });

I'd just like to understand if we need to do something different as a result of this change? Or if we don't need to, maybe we should, so I'd like to grasp the difference. Cheers!

This change effectively enables an EIP attachment in fck-nat's HA mode. What you have there works if the instances are never replaced. But if you are using an ASG and a new instance is brought online, this change will automatically move the assigned EIP to the new instance.

This change effectively enables an EIP attachment in fck-nat's HA mode. What you have there works if the instances are never replaced. But if you are using an ASG and a new instance is brought online, this change will automatically move the assigned EIP to the new instance.

I see - thank you for explaining, much appreciated

if anyone struggling to use it in CloudFormation, this example works fine just adding the EIP Association ID parameter:
Parameters:
  vpc:
    Type: String
    Default: "vpc-xxxxxx"
  subnet:
    Type: String
    Default: "subnet-xxxxxx"
  CIDR:
    Type: String
    Default: "<IP_ADDRESS>/16"
  KeyName:
    Type: String
    Default: "renan"
  EipAssociationID:
    Type: String
    Default: 'eipassoc-xxxxxxxxx'

Resources:
  FckNatInterface:
    Type: AWS::EC2::NetworkInterface
    Properties:
      SubnetId: !Sub "${subnet}"
      GroupSet:
        - Fn::GetAtt:
            - NatSecurityGroup
            - GroupId
      SourceDestCheck: false

  FckNatAsgInstanceProfile:
    Type: AWS::IAM::InstanceProfile
    Properties:
      Roles:
        - Ref: NatRole

  FckNatAsgLaunchConfig:
    Type: AWS::AutoScaling::LaunchConfiguration
    Properties:
      ImageId: ami-05b6d5a2e26f13c93
      InstanceType: t4g.nano
      KeyName: !Sub "${KeyName}"
      IamInstanceProfile:
        Ref: FckNatAsgInstanceProfile
      SecurityGroups:
        - Fn::GetAtt:
            - NatSecurityGroup
            - GroupId
      UserData:
        Fn::Base64: !Sub |
          #!/bin/bash
          echo "eni_id=${FckNatInterface}" >> /etc/fck-nat.conf
          echo "eip_id=${EipAssociationID}" >> /etc/fck-nat.conf
          service fck-nat restart
    DependsOn:
      - NatRole

  FckNatAsg:
    Type: AWS::AutoScaling::AutoScalingGroup
    Properties:
      MaxSize: "1"
      MinSize: "1"
      DesiredCapacity: "1"
      LaunchConfigurationName:
        Ref: FckNatAsgLaunchConfig
      VPCZoneIdentifier:
        - !Sub "${subnet}"
      Tags:
        - Key: Name
          Value: Fck-NAT
          PropagateAtLaunch: TRUE
    UpdatePolicy:
      AutoScalingScheduledAction:
        IgnoreUnmodifiedGroupSizeProperties: true

  NatSecurityGroup:
    Type: AWS::EC2::SecurityGroup
    Properties:
      GroupDescription: Security Group for NAT
      SecurityGroupIngress:
        - CidrIp: !Sub "${CIDR}"
          IpProtocol: "-1"
      SecurityGroupEgress:
        - CidrIp: <IP_ADDRESS>/0
          Description: Allow all outbound traffic by default
          IpProtocol: "-1"
      VpcId: !Sub "${vpc}"

  NatRole:
    Type: AWS::IAM::Role
    Properties:
      AssumeRolePolicyDocument:
        Statement:
          - Action: sts:AssumeRole
            Effect: Allow
            Principal:
              Service: ec2.amazonaws.com
        Version: "2012-10-17"
      Policies:
        - PolicyDocument:
            Statement:
              - Action:
                  - ec2:AttachNetworkInterface
                  - ec2:ModifyNetworkInterfaceAttribute
                Effect: Allow
                Resource: "*"
            Version: "2012-10-17"
          PolicyName: attachNatEniPolicy
        - PolicyDocument:
            Statement:
              - Action:
                  - ec2:AssociateAddress
                  - ec2:DisassociateAddress
                Effect: Allow
                Resource: "*"
            Version: "2012-10-17"
          PolicyName: associateNatAddressPolicy

If you are down for that, the documentation might need to be updated with the newest features for the various ways of deployment. A PR adding in static EIP support for the CloudFormation deployment would definitely be a good thing :)

I think it's more related to Cloudformation syntax because the docs used Fn::Join, and Fn::Sub is not working inside Fn::Join.
Maybe changing the Cloudformation docs to use Fn::Sub syntax is more clean and easy for not Cloudformation experts


Set scenarios to keep_compatible and apply missed migrations

One of our clients has created a number of scenarios for which the keep_compatible attribute is set to false. For the scenarios the attribute needs to be set to true. In the meantime, migrations have been applied that these scenarios have missed. The question is how to apply the missed migrations to the scenarios after they have been set to true. Perhaps this is done automatically by ETEngine?
Scenarios are still confidential, so contact me for the scenario-links.

[ ] Set keep_compatible for marked scenarios to true with an API call
[ ] Check how missed migration can be applied to marked scenarios
[ ] Apply missed migrations to marked scenarios

Notifying @DorinevanderVlies

The most important question is: (how) do we know which migrations have been missed?
Unfortunately it's not that straightforward to apply them. If values were changed based on older calculations we cannot see anymore what it should have been. We also don't know if some migrations have been carried out when the scenarios were updated (we migrate for 1 month after the last update), followed by some missed migrations when the scenarios were untouched again for a while, and then again maybe some applied migrations after an update.
Anyhow, we have to do it all by hand.
Did they create the scenario's through the API? Then they should have explicitly put keep_compatible to truelike it says in our API docs. If they did not create them through the API, but via the font-end we have to investigate if that is the behaviour that we want - that scenario's get stale so quickly without users apparently being aware of it.

This issue also applies to the II3050v2 scenarios, though I'm still awaiting confirmation from our client to check that keep_compatible = false was unintentional.

Closed in 7687e51


Updated isotonic regression

A new score or metric should be developed on a separate feature branch, rebased against the main branch. Each merge request should include:
The implementation of the new metric or score in xarray, ideally with support for pandas and dask
100% unit test coverage
A tutorial notebook showcasing the use of that metric or score, ideally based on the standard sample data
API documentation (docstrings) using Napoleon (google) style, making sure to clearly explain the use of the metrics
A reference to the paper which described the metrics, added to the API documentation
For metrics which do not have a paper reference, an online source or reference should be provided
For metrics which are still under development or which have not yet had an academic publication, they will be placed in a holding area within the API until the method has been properly published and peer reviewed (i.e. scores.emerging). The 'emerging' area of the API is subject to rapid change, still of sufficient community interest to include, similar to a 'preprint' of a score or metric.
Add your score to summary_table_of_scores.md in the documentation

All merge requests should comply with the coding standards outlined in this document. Merge requests will undergo both a code review and a science review. The code review will focus on coding style, performance and test coverage. The science review will focus on the mathematical correctness of the implementation and the suitability of the method for inclusion within 'scores'.
A github ticket should be created explaining the metric which is being implemented and why it is useful.

If this gets merged, then this pull request isn't needed https://github.com/nci/scores/pull/112/files

I have pushed up a change to rename isoreg to isoreg_impl to be consistent with the naming pattern of other scores.


test: jestify invoke contract in connector besu

Migrate Tap to Jest
File Path:
packages/cactus-plugin-ledger-connector-besu/
src/test/typescript/
integration/plugin-ledger-connector-besu/
deploy-contract/
invoke-contract.test.ts
This is a PARTIAL resolution to issue #238
Signed-off-by: Youngone Lee<EMAIL_ADDRESS>

@Leeyoungone If you rebase onto upstream/main it should start passing the CI


Line by line on a sleep function breaks

Testing https://github.com/microsoft/vscode/issues/133905
I would've expected to pause the execution/timer
https://user-images.githubusercontent.com/35271042/135150397-237506fe-d172-4ec5-a8a1-6c19990e7564.mp4

Filed https://github.com/microsoft/vscode-jupyter/issues/7715 instead as i was confused by the continue action


Serial console is not re-connected after SerialIo tests (Bugzilla Bug 3866)

This issue was created automatically with bugzilla2github
Bugzilla Bug 3866
Date: 2022-03-11T06:54:26+00:00
From: Felix <>
To:   Gayathri Thunuguntla <>
CC:   edhaya.chandran, edk2+bugs+int+994+563148131503455288, gaojie, gayathrit, sundaresans
Last updated: 2024-02-27T01:11:12+00:00

Comment 18597
Date: 2022-03-11 06:54:26 +0000
From: Felix <>

Industry Specification: ---
Release Observed: EDK II Master
Releases to Fix: EDK II Master
Target OS: ---

SCT disconnects serial console (TerminalDxe) before running SerialIo tests and does not connect it back once execution as completed. As a result, it's impossible to run SCT via the serial console.

Comment 18889
Date: 2022-04-21 18:11:02 +0000
From: Felix <>
(In reply to Edhay from comment #1)
> Hi Felix,
>   Could you please give details of the point of disconnection and when it
> should be reconnected. Also please provide the logs for this case.
Edhay,
The logs look fine, no errors, warnings or anything like this is reported.
The serial console is disconnected write before the tests. The last message I'm seeing my serial terminal is "Remaining test cases: 8". After that I can see another non-printable character. The VGA console keeps working fine. Once SCT test execution is completed I can re-connect serial terminal by running "connect -r" in Shell using USB keyboard.

Comment 18933
Date: 2022-05-05 10:13:44 +0000
From: Barton Gao <>
Hi Felix,
We have sct tests on multiple platforms and never see this issue. Could you provide more information regarding your test environment ?

Comment 18948
Date: 2022-05-11 04:55:56 +0000
From: sundaresans
Hi Barton,
In SCT tool source, before performing EfiSerialIoProtocol test ,SCT code invokes OpenProtocol() on SerialIo Handle with EXCLUSIVE attribute. As Terminal driver already opened SerialIo protocol with BY_DRIVER attribute, it will get disconnected as EXCLUSIVE attribute is passed for OpenProtocol().
After completion of test, CloseProtocol() is invoked but no attempt made to reconnect the SerialIo Handle. After closing the protocol, Serial I/O protocol handle should be connected recursively.
Same issue was reproduced with PciSioSerialDxe & TerminalDxe from EDK2 source.
SCT code:
// Add one new logic to filter the SerialIo Protocol
//
if (SctCompareGuid (Guid, &gEfiSerialIoProtocolGuid) == 0) {
Status = tBS->OpenProtocol (
HandleBuffer[HandleIndex],
Guid,
(VOID **) &Interface,
HandleBuffer[HandleIndex],
NULL,
0x00000020    //EXCLUSIVE
);
Referred edk2-test-edk2-test-stable202108 source code.
edk2-test-edk2-test-stable202108\uefi-sct\SctPkg\TestInfrastructure\SCT\Framework\Execute\Execute.c (ExecuteBbTestCase())

Comment 19022
Date: 2022-06-02 10:20:05 +0000
From: Edhay <<edhaya.chandran>>
Thank you Sundaresan for your inputs.
We will analyse further by reproducing this issue.

Comment 18777
Date: 2022-04-07 10:19:18 +0000
From: Edhay <<edhaya.chandran>>
Hi Felix,
Could you please give details of the point of disconnection and when it should be reconnected. Also please provide the logs for this case.

Comment 21762
Date: 2023-07-21 09:54:55 +0000
From: Gayathri Thunuguntla <>
Comment on attachment 1385
Saved the Driver binding handle that connects SerialIo handle before SCT tool takes exclusive ownership.  Passed the saved Driver binding handle to ConnectController() in order to connect back serial.
Attached the changes to save the driver binding handle that binds the SerialIo handle before SCT tools exclusive take ownership of the SerialIo handle.
After SerialIo test, passed the saved driver binding handle to ConnectController in order to connect back serial Io handle to the old driver.
Unit Test:
Verified after SCT serial Io test, SerialIo handle got connected to the Terminal driver and verified redirection works fine.
Attachment: SCT_SerialIo.zip

Comment 22518
Date: 2024-02-04 13:31:18 +0000
From: Edhay <<edhaya.chandran>>
Thank you, Gayathri for the patch and verification results.
Could you please send the patch in https://edk2.groups.io/g/devel for review.
Also mention the link of the message in this ticket.

Comment 22657
Date: 2024-02-27 01:11:12 +0000
From: Edhay <<edhaya.chandran>>
Hi Gayathri,
The solution must be provided as a patch.
The attachment contains the actual source files.
Could you please provide the same.
Also could you please send the patch in https://edk2.groups.io/g/devel for review.
Please refer to https://github.com/tianocore/tianocore.github.io/wiki/Laszlo's-unkempt-git-guide-for-edk2-contributors-and-maintainers

Comment 21761
Date: 2023-07-21 09:52:52 +0000
From: Gayathri Thunuguntla <>
Created attachment 1385
Saved the Driver binding handle that connects SerialIo handle before SCT tool takes exclusive ownership.  Passed the saved Driver binding handle to ConnectController() in order to connect back serial.

Changes to save the Driver binding handle that connects SerialIo handle before SCT tool takes exclusive ownership of the SerialIo handle.
Passed the saved Driver binding handle to ConnectController() in order to connect back serial Io handle to the old driver.

Attachment: SCT_SerialIo.zip

Comment 19929
Date: 2022-11-03 10:55:07 +0000
From: Edhay <<edhaya.chandran>>
@Sundaresan: Can AMI provide a patch for the solution?


Can't define multiple tests

stack test makes no error but hie make errors with tests.
Could not load module ‘Test.Hspec’
It is a member of the hidden package ‘hspec-2.6.1’.
Perhaps you need to add ‘hspec’ to the build-depends in your .cabal file.
Use -v to see a list of the files searched for.

https://github.com/falsandtru/haskell-examples/blob/master/basic/test/LibSpec.hs

Accidentally fixed in debugging. Seems like Stack has unexpected side-effects.

Seems like Stack has unexpected side-effects.

Could you explain what you mean by this in more detail please?

This problem was fixed by rebuilding. Probably this is a problem of build cache management.


Add missing array extension methods for Sugar

https://www.motoslave.net/sugarcube/2/docs/object-methods.html#array-arrayprotopluckmany

To preview the use of these extensions before the develop branch is merged with master, download the following files and place them anywhere in your project:
https://github.com/daterre/Cradle/blob/develop/.src/Cradle/StoryFormats/Sugar/StoryVarExtensions.cs
https://github.com/daterre/Cradle/blob/develop/.src/Cradle/StoryFormats/Sugar/VarTypes/SugarDictionaryService.cs
https://github.com/daterre/Cradle/blob/develop/.src/Cradle/StoryFormats/Sugar/VarTypes/SugarListService.cs
And then add these 2 lines somewhere before you call story.Begin:
StoryVar.RegisterTypeService<List<StoryVar>>(new SugarListService());
StoryVar.RegisterTypeService<Dictionary<string, StoryVar>>(new SugarDictionaryService());

Hi! Thanks for doing this! It did not work out fo the box, unfortunately. Should I also change js extensions accordingly? https://github.com/daterre/Cradle/blob/master/Editor/js/StoryFormats/Sugar/sugar.extensions.js_

StoryVarExtensions needed minor fix on line 254 (Max to Min). Now I'm stuck with lack of vars capturing support in my story, unfortunately, which is technically out of the scope for this arrays issue. I'll try to simplify my example and let you know.

Hey mate, anything I could help with?


Update Shoukou

About the PR
Several fixes after first-day stress test.

Evil Twin stuff is failing the tests :<


Enhancement: Add PHP 7.2 to Travis build matrix

This PR

[x] adds PHP 7.2 to the Travis build matrix

Thank you, @richardfullmer!


Catalogue use case: Should we add a property specifying if the seller-product and the customer-article are fully specified?

Catalogue use case: Should we add a property specifying, if the seller-product and the customer-article are fully specified?

When you get the representation of a seller-product or customer-article, the presence of properties with the plural or range termination shows that it is not fully specified. When all the properties have a single value, it is fully specified.
When you get the identifier of an item that belongs to a seller-product or customer-article, the presence or the absence of the paper (or pulp, ...) property makes it clear if it is fully specified of not.

What if the customer wants to create a customer-article without specifying all the open properties as it can accept all the values proposed by the seller. (...) fullySpecified: true | false. We will have to invent more scenario to play with that property within the Catalogue used case.

The idea of this fullySpecificied property is to distinguished between not providing-a-specific-value-for-a-specific-property-because-I-will-specificy-it-late (fullySpecified: false); and not providing-a-specific-value-for-a-specific-property-because-I-let-the-seller-choose (fullySpecified: true). We still want to clearly differentiate between these tow cases, but we don't think fullySpecificied is a good property name.
We suggest definitionState: fully-defined-by-the-customer | will-be-further-defined-by-the-customer-on-order | can-be-further-defined-by-the-seller-on-order (default). The property is not required.

This discussion will be continued in issue #141...


Design a PCB that is RFI resistant for remote location close to high power antennas

A version built on proto PCB with lots of short wire connection in a plastic case, located within 25ft of the several VHF antennas often locks up when transmitting with high power levels like 150W to 500W.  Several remedies have help a bit, but it is time to create a PCB with particular attention to RFI resistance,  To that end I have a nearly completed PCB waiting for parts to arrive and do a part fit verification before ordering boards.

Created an optocoupler based board with only ethernet cable inside on 3/31/2022. Ordered 8 boards and built 1 of them.    Made several tweaks to the board as I built the first one including routing the USB host and ethernet cables wired onto the board and fixed a missing 3.3V trace.


Update Homebrew installation instruction

See https://github.com/Homebrew/homebrew-core/pull/124851

Oh man thank you so much for doing this!! I admit though I am not familiar with their process on how to update homebrew-core when a new release is out and I just pushed out v1.3.0. If it's alright with you would you mind making changes in homebrew-core and the readme to reflect that? I would deeply appreciate it!

Here you go: https://github.com/Homebrew/homebrew-core/pull/124959
Anyone can update a formula with the following command
brew bump-formula-pr --version 1.3.0 erdtree

Thank you! I'm adding this back into the README


how to install old version or rollback to old version of electron?

If there is an existing Electron issue for this, please link it here:
https://github.com/atom/electron/issues
Describe your problem:
i have an app the that i build using electron long time ago, after updating electron globally it doesn't work anymore, how i can rollback to the old version or reinstall old version of electron?

never mind i found out how thanks

Just for those who find this:
npm install electron@<version_number>


meta function for max_pool missed the non-pooling dimensions.

What does this PR do?
Fixes #365

[ ] Adding test for meta function.

Thanks! Should we augment the test too?

Yes we definitely should. @mruberry mentioned the same in https://github.com/Lightning-AI/lightning-thunder/issues/365#issuecomment-2108606263
This is meant to be a quick fix for @kiya00 so I didn't properly look at that part. Thanks for the quick merge :bow:


MH-12618, SMIL tests fail when doctype url can't be resolved

remove the doctype with the dtd url and it's flakey http protocol in the test smil file. live smil files have no doctype anyway

I'm confident that the reports are failing for spurious reasons, but I've restarted the build anyway.  IMO should just merge as is.


Invalid format for Authorization header [code: 6111]

Which Cloudflare product(s) does this pertain to?
Wrangler core
What version(s) of the tool(s) are you using?
3.30.1
What version of Node are you using?
No response
What operating system and version are you using?
Linux
Describe the Bug

root@kr1:~# wrangler whoami
 ⛅ wrangler 3.30.1 (update available 3.31.0)
-------------------------------------------------------
Getting User settings...

✘ [ERROR] A request to the Cloudflare API (/user) failed.

  Invalid request headers [code: 6003]
  - Invalid format for Authorization header [code: 6111]

  If you think this is a bug, please open an issue at:
  https://github.com/cloudflare/workers-sdk/issues/new/choose

Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
No response

@a224327780 can you try to logout (wrangler logout) and then login again to get a new code? It looks like there may have been an issue during the auth flow. I've tested with a couple users myself and can't seem to reproduce this.

hi @a224327780 we haven't heard from you in a while, so we’re going to close this issue for now. if this is still a problem, please feel free to open a new issue referencing this one. please be sure to include an updated summary and any additional details, followup, or context (particularly in response to @admah 's question above) and we will revisit and investigate further. thanks!


packaging request: etherpad

Project description
Etherpad is a real-time collaborative editor scalable to thousands of simultaneous real time users. It provides full data export capabilities, and runs on your server, under your control.
Metadata

homepage URL: https://etherpad.org/
source URL: https://github.com/ether/etherpad-lite
license: Apache License v2
platforms: unix, linux, darwin, windows

a module for this would be nice too

I gave this a bit of a go today; ran into quite a lot of issues trying to package with buildNPMModule:

etherpad is using the old npm lockfile version, I had to regenerate it with npm and add a patch, otherwise fetch-npm-deps complained about missing hashes
etherpad's install script does some kind of funky symlinking of itself back into node_modules - without this trying to resolve dev deps fails (even if you set npmInstallFlags = [ "--omit=dev" ]) because etherpad-cli-client depends on ep_etherpad-lite. To fix this I patched to remove etherpad-cli-client from devdependencies
There's no build step, so you need to set dontNpmBuild = true;
It tries to find it's own base path and makes assumptions about what it looks like, so you have to patch AbsolutePaths.js to account for it
Once you actually get it to run, it shells out to npm to try to find installed plugins, so you need to set makeWrapperArgs to include a path to npm.
And even then, once you get it to run with a config file so it doesn't try to create a database file in the nix store, it will break due to trying to read with hard-coded path prefixes.

At this point I decided that was enough computering for one day. Here's a gist with where I got to: https://gist.github.com/wlcx/b6cec22ddc9e6b1869bc9a30262ae08f

Looks like someone worked on this during Summer of Nix 2022: https://github.com/ngi-nix/etherpad-nix
Not tested or documented though.


Page alert example refresh after closing?

Not sure what this means:
Jon thinks this may be interpreted as a button to reset the code demo after messing with it.

@aaronhans @xjensen I dont know what this issue is for. Any guidance on how to confirm it is working/fixed?

The page alert component page viewable in the release preview here https://1-0-1-rc-post-launch-fixes.pr.designsystem.webstandards.ca.gov/pr/1-0-1-rc-post-launch-fixes/components/page-alert/readme/
Has  new "reset demo" link above the interactive demo embedded in the readme documentation. This is supposed to let the user return the demo to its initial state which is helpful with this component because clicking the x dismisses it.

Oh I get it. Confirmed clicking reset demo reload the page alert. Minor thing is there's no hover state for "Reset Demo" so you don't know it does anything until you click it. Tolerable behavior though.

I'll let @jaredwyee make the call on urgency.
Once we have a fix I'd like to make sure @kkoryaka gets to review it because he was changing some button styles recently

Sorry I inadvertently moved this DONE not realizing there was more comments. @xjensen @aaronhans. Will move this back into in progress.
If a hot fix is safe to do, go a head but this isn't urgent by any means so if we'd rather adhere to the new release schedule, let's do that.

I've put a fix into the current sprint at #666. Not going to do a hotfix here.


TopTabNavigator - tabStyle: width auto

Current Behavior
When I use createMaterialTopTabNavigator with tabStyle: { width: 'auto' } it will break the width of indicator.

It also affects scroll alignment
Expected Behavior

tab indicator should adjust based on tab width.

Your Environment

software
version

react-navigation
3.6.1

react-native
0.57.7

node
10.15.0

yarn
1.10.1

https://github.com/react-native-community/react-native-tab-view/issues/698

then let's close this issue. Tried to search for same issue 🤓


Travis not running all test fixtures

@rajasegar the output from Travis does not show all the new tests.  Can you look into this, please?

Should have been fixed by https://github.com/rajasegar/ember-angle-brackets-codemod/pull/16 I think 🤔

ya, that looks right!  thanks at @rwjblue.  I was looking at my Travis jobs from the other day.


can't import properly in typescript

what import statement do we need to get ResizeSensor imported into typescript when using the ResizeSensor.d.ts file that comes with the package?
When I use import ResizeSensor from 'css-element-queries/src/ResizeSensor';
the TS compiler translates:
new ResizeSensor(element, cb);
to
new ResizeSensor_1.default(element, cb)
I have tried all the forms of import I know about but I haven't been able to get TS to generate the proper call.
Any suggestions?
Thanks

OK. I think that the ResizeSensor.d.ts is wrong. I could not find an import statement coupled with a new statement for the class that would work. So I started reading this section of the typescript documentation:
Declaration Files
specifically this page  has a section on how to build a definition file for a UMD modules (which I believe ResizeSensor is).
It shows that the client code should look like this:

var x = require("bar");
// Note: using 'new' operator on the imported variable
var y = new x("hello");

And that the definition file should follow the module-class.d.ts template.
most of the existing ResizeSensor.d.ts file follows that template, except for the final export. Right now the final export is:
export default ResizeSensor;
but it should be
export = ResizeSensor
Once I changed the file to use the export = line, I was able to get TS to generate the correct code to call the constructor for ResizeSensor.
Could we get a new version with that changed? I am not really setup to create a PR, or I would do that.

I am back again. Importing this into TS is messy.
I am sure that there is some combination of TS options that allow the supplied defintion file to properly import the module and use it from TS. I don't know what they are though.
What I provided above works great when the TS options are:

target: es5
module: amd

with these options you have to use export = ResizeSensor in the .d.ts file and you import with:
import ResizeSensor = require('...');
but due to some other features I want to use, I had to change my TS options to:

target: es5
module: esnext

with these options, I had to go back to the supplied .d.ts file (with the export default ResizeSensor) and modify it to export the class. Here is my the .d.ts file I am using now:
export declare type ResizeSensorCallback = (size: { width: number; height: number; }) => void;

export declare class ResizeSensor {
    constructor(element: Element | Element[], callback: ResizeSensorCallback);
    detach(callback?: ResizeSensorCallback): void;
    reset(): void;

    static detach(element: Element | Element[], callback?: ResizeSensorCallback): void;
    static reset(element: Element | Element[]): void;
}

export default ResizeSensor;

But I also had to modify the source code slightly to get the module to return an object that TS was expecting.
I had to change the final return statement to return an object with ResizeSensor in it rather than return the function. It now looks like:
return {
    ResizeSensor
};

I should also mention that I am using webpack to bundle this up as that might be changing the import handling in some way I am unaware of. But with this source modification and the supplied .d.ts file, I am now able to do this:
import { ResizeSensor } from 'Libs/css-element-queries/ResizeSensor';
let sensor = new ResizeSensor($container[0], determineBounds);

What happens in the underlying JS code is that the module that is imported now contains an object with the ResizeSensor constructor function in it. The new ResizeSensor call is converted into module.ResizeSensor(), which is why the return needed to be an object rather than the actual function.
I suspect this messes up use in the global space so I am not suggesting a change in any way. I am just recording this so somebody else running into the issue can benefit from my effort.
I guess if you did change the return as I pointed out above, you could keep the existing CommonJS and global variable behaviour by changing the UMD template to look like this:
(function (root, factory) {
    if (typeof define === "function" && define.amd) {
        define([], factory);
    } else if (typeof exports === "object") {
        module.exports = factory().ResizeSensor;
    } else {
        root.ResizeSensor = factory().ResizeSensor;
    }

My temp fix for now is just require the JS source file:
const sensor = require('css-element-queries/src/ResizeSensor.js');

// setup the sensor
this.sensor = new sensor.ResizeSensor(this.element.nativeElement, (event: ResizeSensorEvent) => {});

But I would like to see the d.ts file updated to properly use the TS definition.

Works here fine using import { ResizeSensor } from 'css-element-queries';.
https://github.com/vdolek/angular-resize-event/blob/6e2a58a6305005e093d682d0dec2bc9995c3ade4/projects/angular-resize-event/src/lib/resized.directive.ts#L2

PR https://github.com/marcj/css-element-queries/pull/266


Does it handle the access token expiration scenario?

for instance, a background long running job keep getting data from blob with delegated user https://storage.azure.com/user_impersonation scope and able to refresh token even after the token expires.
In that case do we need offline_access scope so that refresh token is also cached?

The sample actually caches both the refresh tokes and access tokens. Refresh tokens are the ones that need caching to enable refreshing access tokens at a regular intervals. So yes, the offline_access scope is required


Introduction of [-useBestContainerOS] parameter prohitting running of images on Insiders OS Build

I updated the NAVContainerHelper to the latest version and now hitting error while intend to run a new NAV image using command New-NAVContainer.
I am running Insider OS 19H1 build on my machine (Build no. 18272.1000)

      What error are you getting?

I have fixed this in the Dev. branch - will do a pull request later and update navcontainerhelper today.
Until then - you can install an older version of navcontainerhelper - or clone the depot and run the navcontainerhelper.ps1  (in the dev branch)

I will do a module update after  your update :)

Thanks for reporting this.

You are welcomed :)

fixed in this https://www.powershellgallery.com/packages/navcontainerhelper/<IP_ADDRESS>


Help me!!

newsyue@ubuntu1404-newsyue:~/dotnet/test$ cat project.json
{
"version": "1.0.0-*",
"compilationOptions": {
"emitEntryPoint": true
},
"dependencies": {
"Microsoft.NETCore.App": {
"type": "platform",
"version": "1.0.0"
}
},
"frameworks": {
"netcoreapp1.0": {}
},
"runtimes": {
"ubuntu.14.04-x64": { }
}
}
newsyue@ubuntu1404-newsyue:~/dotnet/test$ cat Program.cs
using System;
namespace ConsoleApplication
{
public class Program
{
public static void Main(string[] args)
{
Console.WriteLine("Hello World!");
}
}
}
newsyue@ubuntu1404-newsyue:~/dotnet/test$ dotnet run
Compiling test for .NETCoreApp,Version=v1.0
Compilation succeeded.
0 Warning(s)
0 Error(s)
Time elapsed 00:00:05.4426753
Hello World!
－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－
newsyue@ubuntu1404-newsyue:~/dotnet/test$ ls
Program.cs project.json  project.lock.json
newsyue@ubuntu1404-newsyue:~/dotnet/test$ dotnet build --native --ilcpath /home/newsyue/dotnet/corert/bin/Product/Linux.x64.Debug/packaging/publish1
Compiling test for .NETCoreApp,Version=v1.0
Compilation succeeded.
0 Warning(s)
0 Error(s)
Time elapsed 00:00:05.2255700
Input Assembly: /home/newsyue/dotnet/test/bin/Debug/netcoreapp1.0/test.dll
clang: error: no such file or directory: '/home/newsyue/dotnet/test/obj/Debug/netcoreapp1.0/native/test.obj'
Compilation of intermediate files failed.
newsyue@ubuntu1404-newsyue:~/dotnet/test$ dotnet /home/newsyue/dotnet/test/bin/Debug/netcoreapp1.0/test.dll
Could not load host policy library from [/home/newsyue/dotnet/test/bin/Debug/netcoreapp1.0]
－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－
Test build tests/src/Simple/Hello is also wrong
Where am I wrong?
Please help me，Thanks very much！

This repo is still on a pre-release version of dotnet tool that needs different package versions.
Can you please try project.json from one of our test cases, like https://github.com/dotnet/corert/blob/master/tests/src/Simple/Hello/project.json?

@jkotas
Test build tests/src/Simple/Hello is also wrong
newsyue@ubuntu1404-newsyue:~/dotnet/corert/tests/src/Simple/Hello$ cat project.json
{
"version": "1.0.0-*",
"compilationOptions": {
"emitEntryPoint": true
},
"dependencies": {
    "NETStandard.Library": "1.0.0-rc2-23819"
},

"frameworks": {
    "dnxcore50": { }
},

"runtimes": {
    "win7-x64": { },
    "osx.10.10-x64": { },
    "ubuntu.14.04-x64": { }
}

}
newsyue@ubuntu1404-newsyue:~/dotnet/corert/tests/src/Simple/Hello$ cat Hello.cs
// Licensed to the .NET Foundation under one or more agreements.
// The .NET Foundation licenses this file to you under the MIT license.
// See the LICENSE file in the project root for more information.
using System;
// Name of namespace matches the name of the assembly on purpose to
// ensure that we can handle this (mostly an issue for C++ code generation).
namespace Hello
{
internal class Program
{
private static void Main(string[] args)
{
if (args.Length == 0)
{
Console.WriteLine("Usage: hello name");
return;
}
        Console.WriteLine("Hello " + args[0]);
    }
}

}

newsyue@ubuntu1404-newsyue:~/dotnet/corert/tests/src/Simple/Hello$ ls
Hello.cmd  Hello.cs  Hello.sh  project.json  project.lock.json
newsyue@ubuntu1404-newsyue:~/dotnet/corert/tests/src/Simple/Hello$ dotnet build -n --ilcpath /home/newsyue/dotnet/corert/bin/Product/Linux.x64.Debug/packaging/publish1
Compiling Hello for DNXCore,Version=v5.0
/home/newsyue/dotnet/corert/tests/src/Simple/Hello/project.json(8,49): warning NU1007: Dependency specified was NETStandard.Library >= 1.0.0-rc2-23819 but ended up with NETStandard.Library 1.5.0-rc2-24027.
Compilation succeeded.
1 Warning(s)
0 Error(s)
Time elapsed 00:00:05.3551580
Input Assembly: /home/newsyue/dotnet/corert/tests/src/Simple/Hello/bin/Debug/dnxcore50/Hello.dll
clang: error: no such file or directory: '/home/newsyue/dotnet/corert/tests/src/Simple/Hello/obj/Debug/dnxcore50/native/Hello.obj'
Compilation of intermediate files failed.
newsyue@ubuntu1404-newsyue:~/dotnet/corert/tests/src/Simple/Hello$ dotnet /home/newsyue/dotnet/corert/tests/src/Simple/Hello/bin/Debug/dnxcore50/Hello.dll
Could not load host policy library from [/home/newsyue/dotnet/corert/tests/src/Simple/Hello/bin/Debug/dnxcore50]
newsyue@ubuntu1404-newsyue:~/dotnet/corert/tests/src/Simple/Hello$

dotnet cli has been has been replaced by initial msbuild driver. Check https://github.com/dotnet/corert/blob/master/Documentation/how-to-build-and-run-ilcompiler-in-console-shell-prompt.md
@newsyue If you still have issues building the repo, please open a new issue on it.


Add RSpec test framework

After certification and before getting into new features it is time to get serious.

http://blog.carbonfive.com/2013/06/28/sinatra-best-practices-part-two/


Lazy loading image

There are a lot of images, so if I use lazy loading, but after the generation, there are no images, if I don't use lazy loading, sometimes the last few images are not shown

Please share the complete code to reproduce the issue.

请分享完整的代码以重现该问题。

              {infraredUrl && (

              )}

I rendered a lot of images using lazy loading, but when I generated the PDF it didn't load

请分享完整的代码以重现该问题。

              {infraredUrl && (

              )}

I rendered a lot of images using lazy loading, but when I generated the PDF it didn't load

I rendered a lot of images using lazy loading, but when I generated the PDF it didn't load

Sorry, but that's not the complete code. That's a snippet from your ?React? application that has only little to do with jsPDF. Please share a complete project (optimally plain HTML/JS, no framework). I don't have the time set up a React project, manually copy the JSX snippet (since you sent it as image), and then guess what jsPDF-related code you might have in order to maybe run into the same issue as you.


Fix a few Azure.Provisioning issues

Fix a few Azure.Provisioning issues including using params for Location, changing the entropy source for RGs, avoiding Location on existing resources, sorting params/outputs, fixing params in interpolation, adding an overload of AssignRole not tied to user assigned identities, and making a few helpful APIs public.  This also adds back part of the InfrastructureResolver module splitting prototype to sort params/outputs generically.

/check-enforcer evaluate

/check-enforcer evaluate


Add test-naming-conventions-check

I'm think it would be nice to have this in project, @volodya-lombrozo what do you think?

@l3r8yJ What "this"? Do you mean the plugin itself?

@volodya-lombrozo add test-naming-conventions check into test-naming-conventions build


Move release blog template to PULL_REQUEST_TEMPLATE

Description of change

Checklist

[ ] npm test passes
[ ] documentation is changed or added (if applicable)
[ ] permission has been obtained to add new logo (if applicable)
[ ] contribution guidelines followed here

Close it as misunderstanding of pull request template, which is useful for the default text of the pull request when created not for the PR itself.


Bug 1950419: ceph: update podDisruptionBudget controller for OSD

This PR has following changes around OSD pdbs

Detect node drains more reliably. When OSD is down and OSD pod is not scheduled to any node or if the scheduled node is not ready, then assume that node is draining.
Don't set no-out flag on the failure domain if the OSD is down due to reasons other than node drain (say disk failure)
update unit tests
update design doc to reflect above changes

Signed-off-by: Santosh Pillai<EMAIL_ADDRESS>(cherry picked from commit aee8f568534d50a290f5c7fb74845c1407a4934f)

Description of your changes:
Which issue is resolved by this Pull Request:
Resolves  https://bugzilla.redhat.com/show_bug.cgi?id=1950419
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
[ ] Skip Tests for Docs: Add the flag for skipping the build if this is only a documentation change. See here for the flag.
[ ] Skip Unrelated Tests: Add a flag to run tests for a specific storage provider. See test options.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.
[ ] Pending release notes updated with breaking and/or notable changes, if necessary.
[ ] Upgrade from previous release is tested and upgrade user guide is updated, if necessary.
[ ] Code generation (make codegen) has been run to update object specifications, if necessary.

@sp98 No need for a downstream backport for 4.8, we are still syncing directly from the release-1.6 to release-4.8 branch. We will pick this up in the next sync.


Respect the --user option when pip install.

Make experiment to mock a typical user behavior who have no root permission and wants to install graphscope by pip3 install graphscope --user. Make sure that all the stuffs and logs doesn't goes outside $HOME, even better, not outside of $home/.local.
Check the permission of the default log location, if the default location can't be used, then prompt user to provide one.

I think weibin has a commit merged about the log location,
@siyuan0322 Could you test and verify your proposal?

@siyuan0322  prompting user to provide a log location during runtime seems wired.  GraphScope would chose to output log to $HOME/.local/log/graphscope if user does not has write  permission  to default location (/var/log/graphscope)

Closed as it worked as expected.


Feature/add forms

Pull Request - Add Forms to the Blog Application
Description:
In this pull request, the implementation of forms in the Blog application is presented, complying with all the specified project requirements.
General Requirements:

Verification of the absence of linter errors.
Followed Git workflow correctly.
Professionally documented work.

Ruby Requirements:

Adherence to the best practices of Ruby throughout the implementation.

Project Requirements:

Created a method called current_user in ApplicationController to make user data available to all controllers.

This method returns the first user from the database.
Proper authentication has not been implemented at this time, as it will be addressed in future projects.

Implemented Functionalities:

Form for creating a post on behalf of the current_user.
Form for creating a comment on behalf of the current_user.
Functionality that allows users to add "likes" to posts.

Changes Made:
Controllers

app/controllers/application_controller.rb: The current_user method has been added to make user data available to all controllers.

# app/controllers/application_controller.rb

class ApplicationController < ActionController::Base
  # ... (existing code)

  def current_user
    @current_user ||= User.first
  end
end

Forms
app/views/posts/new.html.erb: Form for creating a post on behalf of the current_user.

<!-- app/views/posts/new.html.erb -->

<%= form_for @post do |f| %>
  <!-- Form fields for creating a post -->
  <%= f.submit 'Create Post' %>
<% end %>

app/views/comments/new.html.erb: Form for creating a comment on behalf of the current_user.

<!-- app/views/comments/new.html.erb -->

<%= form_for @comment do |f| %>
  <!-- Form fields for creating a comment -->
  <%= f.submit 'Create Comment' %>
<% end %>

app/views/posts/_likes_count.html.erb: Form to allow users to add "likes" to posts.

<!-- app/views/posts/_likes_count.html.erb -->

Notes:
At this moment, proper authentication has not been implemented, as indicated in the project requirements. This functionality will be addressed in future projects.
Please review these changes and confirm that they meet the specified requirements before merging them into the develop branch.
Thank you for your review and consideration.
Regards,
@ClaudiaRojasSoto

Thank you @ridwanediallo!


Update scrcpy-noconsole.vbs

Doesnt Open a new Scrcpy window if one is already open. This prevents echoing when playing Audio on pc. In addition, if the window is already open, it will be brought to foregroud (unless it is minimized). This wont quite work yet, as the window title is dependent on the device. Maybe one could get this name dynamically or rename the window to scrcpy on all devices. just a suggestion

+1


[FLINK-1650] [logging] Suppress wrong netty warnings on akka shutdown

Maybe we should also turn off the DefaultChannelPipeline in the log4j-test.properties files, which don't turn logging off, if we don't want the RejectedExecutionException to appear in the test outputs. Currently the flink-yarn-tests module sets the logging to warn in log4j-test.properties.
Having said that, LGTM.

+1, as well. I agree with Till regarding the log4j-test file. If you add it there as well, don't forget to add it to tools/log4j-travis.log as well.

I addressed your comments and merged the pull request.


feat: port data.int.basic

[x] depends on #583

The final section /-! # units -/ has not been ported, because it still depends on some earlier files.

The /-! # units -/ no longer needs to be ported, as it has been moved out to a separate file in mathlib3 as mathlib#17508.

bors merge


DOCS: Unify project layout sections

I have several times found myself trying to remember where to put examples, and getting frustrated looking for it in the guide - because its not there, its in the manifest description, which I find to be a non-intuitive place for it. I thought it might help to unify the project layout sections between "The Guide" and "Manifest Description", and have them both link to each other for explanations where needed.
Of course, I am very willing to take any suggestions on my changes; I'm not wedded to any of text in specific, I just want the two sections to not be so separate and hard to find, especially from each other.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

So in general, the Cargo docs are a mess, and I've been promising @alexcrichton that I'd fix it for a few months now, but I still haven't gotten to it :(
I would argue that the project layout does belong in the manifest section, because they're defaults that can be changed by the manifest. However, you're right that the current state is confusing.
Alex, what do you think? Ultimately, when the docs are re-done, this will all get changed anyway... so might be worth merging for now, even if it's not perfect.

So in general, the Cargo docs are a mess, and I've been promising @alexcrichton that I'd fix it for a few months now, but I still haven't gotten to it :(
I would argue that the project layout does belong in the manifest section, because they're defaults that can be changed by the manifest. However, you're right that the current state is confusing.
Alex, what do you think? Ultimately, when the docs are re-done, this will all get changed anyway... so might be worth merging for now, even if it's not perfect.

So in general, the Cargo docs are a mess, and I've been promising @alexcrichton that I'd fix it for a few months now, but I still haven't gotten to it :(
I would argue that the project layout does belong in the manifest section, because they're defaults that can be changed by the manifest. However, you're right that the current state is confusing.
Alex, what do you think? Ultimately, when the docs are re-done, this will all get changed anyway... so might be worth merging for now, even if it's not perfect.

So in general, the Cargo docs are a mess, and I've been promising @alexcrichton that I'd fix it for a few months now, but I still haven't gotten to it :(
I would argue that the project layout does belong in the manifest section, because they're defaults that can be changed by the manifest. However, you're right that the current state is confusing.
Alex, what do you think? Ultimately, when the docs are re-done, this will all get changed anyway... so might be worth merging for now, even if it's not perfect.

So in general, the Cargo docs are a mess, and I've been promising @alexcrichton that I'd fix it for a few months now, but I still haven't gotten to it :(
I would argue that the project layout does belong in the manifest section, because they're defaults that can be changed by the manifest. However, you're right that the current state is confusing.
Alex, what do you think? Ultimately, when the docs are re-done, this will all get changed anyway... so might be worth merging for now, even if it's not perfect.

So in general, the Cargo docs are a mess, and I've been promising @alexcrichton that I'd fix it for a few months now, but I still haven't gotten to it :(
I would argue that the project layout does belong in the manifest section, because they're defaults that can be changed by the manifest. However, you're right that the current state is confusing.
Alex, what do you think? Ultimately, when the docs are re-done, this will all get changed anyway... so might be worth merging for now, even if it's not perfect.

So in general, the Cargo docs are a mess, and I've been promising @alexcrichton that I'd fix it for a few months now, but I still haven't gotten to it :(
I would argue that the project layout does belong in the manifest section, because they're defaults that can be changed by the manifest. However, you're right that the current state is confusing.
Alex, what do you think? Ultimately, when the docs are re-done, this will all get changed anyway... so might be worth merging for now, even if it's not perfect.

So in general, the Cargo docs are a mess, and I've been promising @alexcrichton that I'd fix it for a few months now, but I still haven't gotten to it :(
I would argue that the project layout does belong in the manifest section, because they're defaults that can be changed by the manifest. However, you're right that the current state is confusing.
Alex, what do you think? Ultimately, when the docs are re-done, this will all get changed anyway... so might be worth merging for now, even if it's not perfect.

Ah! Great to hear that the documentation will get an overhaul. Hurray!
Yes, I certainly don't see my solution as perfect, and I certainly see your argument. However, the "default" project layout gives you functionality that doesn't require modifying Cargo.toml at all; as a frequent "casual" Cargo user, when I am looking for a reference on where to put tests, benchmarks, or examples so that they just work with Cargo without modifying Cargo.toml, I certainly don't think to look in the description of Cargo.toml for anything but how to change those defaults. Maybe that's just me, though!
Here's another idea: I could make a shorter section on "Project Layout" in The Guide, with a link to a more detailed description in The Manifest Format. I'm not sure how to split things, but I could try. What do you think?

So in general, the Cargo docs are a mess, and I've been promising @alexcrichton that I'd fix it for a few months now, but I still haven't gotten to it :(
I would argue that the project layout does belong in the manifest section, because they're defaults that can be changed by the manifest. However, you're right that the current state is confusing.
Alex, what do you think? Ultimately, when the docs are re-done, this will all get changed anyway... so might be worth merging for now, even if it's not perfect.

oh my gosh, my spotty wifi meant this got sent a zillion times, so sorry :(

I could make a shorter section on "Project Layout" in The Guide, with a link to a more detailed description in The Manifest Format

This sounds like a good idea to me! I agree with @steveklabnik that we may as well take incremental improvements for now, it's certainly not hurting anyone :)

OK, its back to the previous split format, but with a little more detail and (most importantly to my mind) a link from the section in guide.md to the section in manifest.md. Any other suggestions?

looks great, thanks :)

@bors: r+

:pushpin: Commit b7f08d6 has been approved by steveklabnik

:hourglass: Testing commit b7f08d6 with merge 19af88c...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64


6.0.0 and  Custom controls

Hi everyone.
How to create one small button with two states and one onClick event?
I have no idea...Are there some examples or tuts?
It'll be great...
Thank you!

Have you seen these demos: http://tomchentw.github.io/react-google-maps/basics/overlay-view ?

Yees....why are you asking? I don't see any buttons on this map...

I believe you can either use an OverlayView, or try something like this: https://developers.google.com/maps/documentation/javascript/examples/control-custom.

@n0ne I was wondering the exact same thing.
It seems however that we are out of lack. Peeking at the SearchBox code, I see hints everywhere that it's custom designed for maps usage  (google maps specific controlPosition prop) and as such no plain react component can be used.
Any thoughts on that @tomchentw ?
or even a workaround for such a common scenario?
Thanks.

@kstratis Were you able to find a solution to this / workaround? I am trying to add a legend on the map

@jm90m None I'm afraid...

@kstratis Oh ok thanks for the reply

Please refer to Getting Help section in the README (or #469).


This article doesn't make sense

You recommend not to check the PAT key into source control and yet the only way for the PAT key to work correctly is to add it into a config file.

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

ID: a1a158fb-d1e3-a0d9-492e-3f78dde5d97c
Version Independent ID: 71055426-3ba1-27f3-11db-cadb688ae123
Content: Use dotnet with Azure DevOps Services feeds
Content Source: docs/artifacts/nuget/dotnet-exe.md
Product: devops
GitHub Login: @elbatk
Microsoft Alias: elbatk

Agreed. Doesn't make any sense. I cannot find any valid article on how to restore the Azure Artifacts nuget package via command line on a build server. Every article out there points to using Visual Studio and Interactive mode to authenticate and pull the packages. How do we automate this process in CI server?

Using the old method, you'd want to securely store that token somewhere in your build system and then construct your nuget.config on the server.
However, there's now an easier way to do this with the Azure Artifacts Credential Provider. I've updated the doc with the new guidance.

@alexmullans, thx! I am interested to know the setup for non-interactive scenario like in build servers on Linux. Following this article now i am able to download and setup the credential provider from here https://raw.githubusercontent.com/Microsoft/artifacts-credprovider/master/helpers/installcredprovider.sh
I have also setup these two env variables..
NUGET_CREDENTIALPROVIDER_SESSIONTOKENCACHE_ENABLED
VSS_NUGET_EXTERNAL_FEED_ENDPOINTS
However "dotnet restore" still could not download my packages from my private vsts feed. I am pretty sure it is not searching from my private feed. On my laptop i can add sources via "nuget sources" command. How do i add sources to the nuget source list on linux?

okay i did add a local nuget.config and have specified the source. However I don't want to specify my credentials in there because this will be checked-in. I was hoping to pass the credentials using the environment variables specified here - https://github.com/Microsoft/artifacts-credprovider#environment-variables
However setting it doesn't seem to have any effect. "dotnet restore" now hits my private feed but returns a 401. Doesn't the cross platform credential provider not take the credentials from the ENV variable?

To add more context, I am running dotnet restore inside docker as part of my build pipeline.
Here is the Docker File....
FROM microsoft/dotnet:2.1-sdk AS build-env
ENV NUGET_CREDENTIALPROVIDER_SESSIONTOKENCACHE_ENABLED true
ENV VSS_NUGET_EXTERNAL_FEED_ENDPOINTS {"endpointCredentials": [{"endpoint":"https://xxx/_packaging/xxx/nuget/v3/index.json","username":"optional","cleartextpassword":"xxx"}]}
WORKDIR /app
COPY ./src/*.csproj .
RUN wget -qO- https://raw.githubusercontent.com/Microsoft/artifacts-credprovider/master/helpers/installcredprovider.sh | bash
COPY ./NuGet.Config /root/.nuget/NuGet/
RUN dotnet restore
The restore fails with this error
/usr/share/dotnet/sdk/2.1.401/NuGet.targets(114,5): error : Unable to load the service index for source https://xxxx.pkgs.visualstudio.com/_packaging/xxxx/nuget/v2/index.json. [/app/xxxx.csproj]
/usr/share/dotnet/sdk/2.1.401/NuGet.targets(114,5): error :   Response status code does not indicate success: 401 (Unauthorized). [/app/xxxx.csproj]
Here is my nuget.config file

Here is my nuget.config file.....

Ah, sorry, I've conflated a few things. Use "password" instead of "cleartextpassword" in your Dockerfile. cleartextpassword is something specific to the nuget.config file, which we're bypassing altogether by using the Credential Provider.

I'm closing this issue as it's been over a month. @alexmullans let me know if you think another doc update (it looks like you've already mae one) should come out of this.

I am curious what additional info is needed?  I think a number of us are running into the same issue -- how to use a PAT token without using some sort of credential store in Azure.

@ravensorb yeah - you're hitting a weak spot in our documentation - sorry about that. I've queued a feature on our internal backlog (hoping to get to it in December, but may slip to January with holidays coming up) to ensure we've clearly documented using authenticated feeds in Docker containers.


EMP-80 Suppressor ship weapon magazine size

The magazine size for this weapon is "1", instead of being "0" like other non-automatic ship weapons. This causes the "recharge" indicator on the ship HUD to simply switch from "0" to "100" when it's recharged, instead of smoothly recharging like other non-automatic ship weapons.

Details
Form ID 002184C1 [WEAP]
Game Version
<IP_ADDRESS>
New Game
Yes
Reproduction Steps
Observe the recharge rate in game, and look at the WEAP record in Starfield.esm via xEdit.

https://github.com/Starfield-Community-Patch/Starfield-Community-Patch/commit/a51cf6dbb83968aa19b2ab8b3bb16811260bd46d


Ensure health probe is stopped when a container is removed

Signed-off-by: Kenfe-Mickael Laventure<EMAIL_ADDRESS>--
- What I did
Fixes #30107 by preventing an health check to keep on running after a container has been removed
- How I did it
By always stopping health checks on container exits. Health checks are always started if present on container start (see daemon.StateChanged()), so there's no fear of missing any and it avoid unneeded exec while the container is still down.
- How to verify it
Given it's a race, it has to be manually testing unfortunately. Instructions provided by @esbite were used (https://gist.github.com/esbite/d0c639e72435b4721d44445b0972dff8)
- Description for the changelog
Ensure health probe is stopped when a container exits
- A picture of a cute animal (not mandatory but encouraged)
Not Today :scream_cat:

@vdemeester @cpuguy83 changed the fixed (and updated the PR description) PTAL

powerpc failure known flaky, let's merge this 👼


feat: add meta frameworks support

Search meta framework languages that support typescript embedded language based on grammars property in language extension package.json. Refactored to simpler dynamic registration.
For people who need to use this PR in advance (unzip it and drag to VSCode extension sidebar):
pretty-ts-errors-0.2.4-pr2.vsix.zip

I can do it. :)

Thank you for your work. This is very great. :)


MDRaisedButton critical clock

Description of the Bug
Reopening https://github.com/kivymd/KivyMD/issues/1011
I think the warning [CRITICAL] [Clock ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute shows if:
The button is added to a screen that is not currently displayed.
The button is a child of a BoxLayout with padding>0.
Here is a reproducible example:
Code and Logs
from kivy.lang import Builder
from kivymd.app import MDApp
from kivymd.uix.screen import MDScreen

class ScreenA(MDScreen):
    pass

class ScreenB(MDScreen):
    pass

KV = """
<ScreenA>:
    BoxLayout:
        orientation: 'vertical'
        padding: 10
        MDLabel:
            text: "Hello world A!"
        MDRaisedButton:
            text: "Go world B"
            size_hint_x: 1.0
            on_press: app.root.ids.screen_manager.current = "ScreenB"

<ScreenB>:
    BoxLayout:
        orientation: 'vertical'
        padding: 10
        MDLabel:
            text: "Hello world B!"
        MDRaisedButton:
            text: "Go world A"
            size_hint_x: 1.0
            on_press: app.root.ids.screen_manager.current = "ScreenA"

Screen:
    ScreenManager:
        id: screen_manager
        ScreenA:
            name: "ScreenA"
        ScreenB:
            name: "ScreenB"
"""

class MyMDApp(MDApp):
    def build(self):
        return Builder.load_string(KV)

if __name__ == "__main__":
    MyMDApp().run()

Versions

OS: Ubuntu 20.04.3
Python: 3.8.10
Kivy: 2.0.0
KivyMD: 0.104.2

Here is the dirty workaround I use at the moment:
class CustomMDScreen(MDScreen):
    are_button_already_resized = BooleanProperty(False)

    def on_pre_enter(self, *args):
        super().on_pre_enter(*args)
        if self.are_button_already_resized:
            return
        for widget in self.walk():
            if isinstance(widget, MDRaisedButton):
                widget.size_hint_x = 1.0
        self.are_button_already_resized = True

#1141 hopefully fixes this? Solves it for me, anyway.

@gbm001


API: Implement tensordot and matmul

Tracking issue #21
Hi @willow-ahrens @hameerabbasi,
This WIP PR introduces finch.tensordot(x1, x2, axes) and finch.matmul(x1, x2)/x1 @ x2.
I still need to complete exhaustive testing.

@willow-ahrens I've got one question regarding matmul. I just noticed that there's a slight difference between tensordot and matmul in Array API for >2D input, namely tensordot aggregates non-contracted dims from one input and the other, where matmul takes two innermost dims for multiplication, and the rest of dims is treated as a stack/batch dimensions.
Here's a NumPy code showing it (described in notes in docs):
  >>> a = np.ones([9, 5, 7, 4])
  >>> c = np.ones([9, 5, 4, 3])
  >>> np.tensordot(a, c, axes=((-1,), (-2,))).shape
  (9, 5, 7, 9, 5, 3)
  >>> np.matmul(a, c).shape
  (9, 5, 7, 3)
  >>> # n is 7, k is 4, m is 3

So I'm not sure I can implement matmul as self.tensordot(other, axes=((-1,), (-2,))). WDYT?

Yeah, perhaps we need to define einsum better in Finch and express both matmul and tensordot in terms of that.
Side note:
I think we should implement mul! in Julia for Finch Tensor and call that here.

@willow-ahrens The PR is ready from my side! (I also want to benchmark these functions)
For now I constrained matmul input to 2D tensors, once we have mul that can perform matmul on stack of matrices I will recur to this.


[Improve][dataproxy] Added sink tests for tubeMQ and kafka.

Description
There is no sink test for tube MQ and kafka right now. Added sink tests for tubeMQ and kafka.
InLong Component
InLong DataProxy
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

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

KafkaSink has not been implemented yet, so test TubeSink first.


Use nearest mipmaps for both minification and magnification

Port of https://github.com/godotengine/godot/pull/40523 to master. Follow-up to https://github.com/godotengine/godot/pull/51533 and https://github.com/godotengine/godot/pull/51531.
This is generally the expected behavior when using a nearest + mipmaps mode, as it's often used for pixel art games.
This closes https://github.com/godotengine/godot-proposals/issues/1222.
Preview
The floor texture uses the Nearest Mipmap filter mode. Click to view at full size, as the difference isn't really noticeable on the preview.
Before

After

Needs rebase.

Thanks!

Thank you!


Add code examples from manuscript submission to ReadTheDocs documentation

The code examples can be found here: https://docs.google.com/document/d/1R964Tz8aORehmL0-8EzZsNvIOkkqp5sh9uBE6O-s8dY/edit?usp=sharing
and here: https://drive.google.com/file/d/11FPwR585h24hA71ubkmFeQ4wZAqpHCXN/view?usp=sharing

Created two ipynb files here (b5a7ceb43270a4467f21d0a9978d88cf59d42105):
https://github.com/biothings/mygeneset.info/blob/master/docs/ipynb/mygeneset_workflow.ipynb  (supp fig 2)
https://github.com/biothings/mygeneset.info/blob/master/docs/ipynb/mygeneset_versioning_caching.ipynb (supp fig 3)


ECommerce-shop and taxamo (European laws VAT)

Jeremy, you're going to include support in your eshop for new VAT European laws?
PrestaShop, WooCommerce, Magento, .... have already built plugins for it.
http://www.woothemes.com/2014/12/handling-eu-vat-woocommerce/
Now, it is a must for a shopping cart.
The best way is through www.taxamo.com.
You will develop a plugin to use taxamo?
Thanks,
Best regards,
Jose A.

Hi @Josua2012 - definitely having global tax support is going to give this module a wider audience. I'm all for having this kind of support added to the module, at least architecting it in such a way that taxamo could be added on as a module.
However, I personally don't have plans to undertake this work. @markguinn do you have/ know of any plans?

Relates to #12

Jeremy, If I pay for the work of making taxamo support. How much could it cost?
Thanks,
Best regards,
Jose A.

@Josua2012 that would require some figuring out time, which unfortunately I don't have. I'm also not in a position to do the work either. Happy to answer any questions from someone who will do the work however.

No, we don't have any clients in Europe so we've not had this feature requested at all. Thanks for bringing this to our attention @Josua2012.
On the one hand this seems like it would be pretty simple to implement as they give you a prewritten spec for what a plugin needs to do: https://www.taxamo.com/doc/dev_guide/plugin_spec/. I think it's possibly as simple as implementing a subclass of TaxModifier for a basic integration, especially when using stepped checkout, maybe also needing a few extra checkout steps created.
On the other hand, there's a lot of logic to digest there (dynamic v universal pricing, sms verification, payment integration, etc). So there's potential for this to take a lot of time to do it right.
I'm also happy to help in an advisory role but I don't see myself being able to do the work any time soon.

The problem is not only for European vendors, but for any foreign seller who wants to sell digital products in Europe. So any shopping cart must be prepared for this, otherwise you can not legally make sales to Europe. Thank you @markguinn .

Maybe https://github.com/mpociot/vat-calculator is an alternative to Taxamo.


New feedback pop-up page design (for issue #856)

Design a new feedback pop-up page (issue #856) that contains Dr. Cleo's survey questions.
Dr. Cleo's survey questions:

Please help make this health app better for you and all Canadians.
1. How satisfied are you with the app? Stars 1-5
2. Are you currently attached to/registered with a family doctor or nurse practitioner?
-Yes
-No
3. Please give any suggestions for improvement regarding the app:
(free text response)

The issue will be closed because the requirement has changed. Please refer to #860


Added VSIISExeLauncher autorebuild support

Added VSIISExeLauncher autorebuild support.
When searching for devenv process in FindAncestorVSProcess method, if "VSIISExeLauncher" is found in the process tree, then we are likely in a debugable IIS hosted app.
Retrieving the initial launcher to find the owningPid number, actually the current devenv instance.
Returning devenv process if found and valid.
This behavior will fail of course if the current executing process lack of permissions, like not being in the administrator group.
In my case, added my current ApplicationPool in the Administrators group, then restarted IIS, was sufficient to allow to read devenv process.
Correct me if i'm wrong, but as the VSIISExeLauncher will be called only if launched from Visual Studio, this behavior should never happen in production.
NOTE: FindIISHostedVSProcess is called within FindAncestorVSProcess while loop if its relevant, however if you prefer by design that this method is called after FindAncestorVSProcess in TryCreate (if relevant), i will change this.

Damn, actually commited something not related to this but to an other pull request. I will revert back shortly.


Inelegant failure after first install

Install scriv 0.8.0
Run "scriv create"
Look:

$ scriv create
Creating changelog.d/20200804_133229_nedbat.rst
Traceback (most recent call last):
  File "/usr/local/virtualenvs/webhooks/bin/scriv", line 8, in <module>
    sys.exit(cli())
  File "/usr/local/virtualenvs/webhooks/lib/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/virtualenvs/webhooks/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/local/virtualenvs/webhooks/lib/python3.8/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/virtualenvs/webhooks/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/virtualenvs/webhooks/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/virtualenvs/webhooks/lib/python3.8/site-packages/scriv/create.py", line 60, in create
    with open(file_path, "w") as f:
FileNotFoundError: [Errno 2] No such file or directory: 'changelog.d/20200804_133229_nedbat.rst'

Fixed in 351ba2e4034caa800742ca427e9043ef71ae03d4.


Initial setup not running

🐛 Bug Report

[ X] 📁 I've Included a ZIP file containing my librephotos log files
[ X] ❌ I have looked for similar issues (including closed ones)
[ ] 🎬 (If applicable) I've provided pictures or links to videos that clearly demonstrate the issue

📝 Description of issue:
Setup does not run on fresh install. No questions to set up admin user ect.
🔁 How can we reproduce it:
Clean install
Please provide additional information:

💻 Operating system: Red Hat Stream 9
⚙ Architecture (x86 or ARM): x86
🔢 Librephotos version: latest
📸 Librephotos installation method (Docker, Kubernetes, .deb, etc.): docker (other users report issues with pi and unraid

🐋 If Docker or Kubernets, provide docker-compose image tag:

📁 How is you picture library mounted (Local file system (Type), NFS, SMB, etc.): n/a
☁ If you are virtualizing librephotos, Virtualization platform (Proxmox, Xen, HyperV, etc.):  n/a
lplog.zip

I think I have tracked down the issue. the docker-compose.yml was changed.
- ADMIN_EMAIL=${adminEmail}
- ADMIN_USERNAME=${userName}
- ADMIN_PASSWORD=${userPass}
- ADMIN_EMAIL=${adminEmail:-admin@localhost}
- ADMIN_USERNAME=${userName:-admin}
- ADMIN_PASSWORD=${userPass:-admin}
Pinged the author of the change. Awaiting response.

@DigitalWarrior remember to close this!

nope leaving it open. 😂


Given mount not allowed

Hi, I'm having difficulties sharing folders with the containers. I keep getting

ERROR: given mount not allowed: /home/files:/project/files

Using docker or nvidia-docker this is no problem, so I guess this has to do with the permissions in the config file. This is how the relevant part looks like at the moment.
VOLUME_MOUNTS_ALWAYS = []
VOLUME_MOUNTS_AVAILABLE = ['/home/files:/project/files']
VOLUME_MOUNTS_DEFAULT = [
    # default mount user's home
    user_home + ':' + user_home,
]

Do you see any obvious mistakes?
Thanks in advance!

hmm, cannot reproduce, could you paste the commandline you're trying?

might've been related to #13 and possibly also fixed by 5ffb9b6edc00debd91a1f4739381fe1bccd33752
closing for now, reopen if not


Webview CSS: data-vscode-theme-name attribute changes when the language changes

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.68.0-insider
OS Version: Windows 10

Steps to Reproduce:

Write webview CSS depending on data-vscode-theme-name, e.g. see https://github.com/microsoft/vscode/issues/97663#issuecomment-630558320
Change the language by installing a language pack - the CSS will stop working

Would it be possible to use an ID instead of a name string for this attribute (e.g. a new attribute to prevent breaking changes), or perhaps to force this attribute to be English only?

@pwang347 @mjbvz
I submitted a PR to address the issue. Please take a look when you have a moment.


Site favicon

Anyone feeling favicon-inspired? I hate seeing lots of favicon.ico 404s in our web logs.

I create favicon.ico and placed it in the /static/img dir. It's a simplified version of the house from the sheltr.org banner.

Nice! In 1a795a60bfb6735f5f30802250745b6edb9257d9 I moved it to the root of static so it should just show up.


Overriding deep-level fields in response object

Describe the PR
Recursively parse content in { } to overriding deep-level fields in response object
Relation issue
Enhancement of #650
#670

Codecov Report

Merging #672 into master will decrease coverage by 0.48%.
The diff coverage is 82.50%.

@@            Coverage Diff             @@
##           master     #672      +/-   ##
==========================================
- Coverage   84.98%   84.50%   -0.49%
==========================================
  Files           7        7
  Lines        1805     1813       +8
==========================================
- Hits         1534     1532       -2
- Misses        172      178       +6
- Partials       99      103       +4

Impacted Files
Coverage Δ

operation.go
87.67% <82.50%> (-1.64%)
:arrow_down:

Continue to review full report at Codecov.

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


add support for LM model

Support Language Model from https://huggingface.co/blog/wav2vec2-with-ngram
solve issue #10

this looks great - thanks for the pr


Support import.meta.hot

Describe the feature you'd like:

As part of looking at #1, I discovered the Vite has another piece of import.meta functionality that we aren't supporting yet: import.meta.hot.
Link to docs.
Suggested implementation:

I think we would be pretty safe to simply remove any references to it as in most cases we are transforming for test code where HMR is not applicable (AFAIK) and/or environments where HMR is not available or available through different APIs.
As per the other transforms, it should be able to be disabled from the preset so other babel plugins could handle it in specialised ways or it can be left in if the target environment supports it.
Describe alternatives you've considered:

None.
Teachability, Documentation, Adoption, Migration Strategy:

Hello, would this be possible?

Hi @neoighodaro, sorry for the delayed response.
I haven't looked into it at all since raising it, but as usual PRs are always welcome.  Would you be interested in adding this feature?


DrawerNavigator screen shouldn't unmount

Hi at all, maybe it's a misunderstanding or just my fault but I googled for hours, I made a StackOverflow post and I wrote on Discord but no one seems to have a solution to my problem.
Current Behavior
I built a very simple example which shows my problem. I have a DrawerNavigator with 2 Screens
class HomeView extends Component {

	componentDidMount() {
		console.log("mount home");
	}

	render() {

		return(
			<View>
				<Text>HomeView</Text>
			</View>
		);

	}

}

class SecondView extends Component {

	componentDidMount() {
		console.log("mount second");
	}

	render() {

		return(
			<View>
   		                 <Text onPress={ () => this.props.navigation.navigate('HomeView') }>SecondView</Text>
			</View>
		);

	}

}

const MyNavigator = DrawerNavigator({

	HomeView: {
		screen: HomeView
	},

	SecondView: {
		screen: SecondView
	}

});

My steps:

The HomeView gets mounted
I open the drawer
I press on Second View
Second View gets mounted
I press on "Second View"
It navigates to HomeView and the componentDidMount function gets called

Expected Behavior
I'm expecting the HomeView should stay mounted. If I'm calling this.props.navigation.navigate('HomeView') I'm expecting my app should show the HomeView Screen as I had it before leaving it. Am I the only one who's facing this issue? Or is my concept completely wrong and someone can tell me a better way to do this? This is a very basic example but think of I'm having a Drawer and each object is a StackNavigator with multiple subroutes. Consider to go navigate 3 times in the first stack, then call another Route from the Drawer and then going back to the first route. You don't want to rerender the whole Stack....
I'm lucky with any suggestions because I'm getting tired of try and fail
Your Environment

software
version

react-navigation
1.0.0-beta.11

react-native
0.46.4

node
8.0.0

npm or yarn
yarn 0.24.6

why do you not call goBack()?

@vonovak because I don't want to rerender my screen when I press the Drawer.
In the mean time nobody has answered or helped me out I started to dig and search for a solution to my problem. Now I have a kind of solution but it seems not to work so I created a detailed new issue with my newest try at #2495
As this is not longer a problem for me, because I found a kind of workaround, the issue can be closed. But I'm still interested in finding a solution or the answer that is is an antipattern or something like this

@vanBrunneren my guess would be that goBack() does not cause a re-render, but I have not tested it

@vonovak oh sorry I was i bit confused at the moment. Of course you are right goBack() does not a rerender. But I have multiple routes in my stack and I want to jump maybe from the 6th to the 3th and this is not realizable with a goBack(). And If I do a goBack() the route which I press goBack() on will get unmounted. And when I go from maybe the 5th route to the 7th and then switch again to the 5th the 5th would be unmounted and has to mount again. This is not what I want, I want a function like "jumpTo" where you can jump to the routes in the stack

+1
same problem
if you go to B from A, A will be unmounted and changed states of A will be lost. I dont want that.

Pinging OP @vanBrunneren since this issue has not been active for a while, and it's related to an old version of the lib.
Please let me know if you want this to remain open; if I get no answer in the next 7 days I will close it.

@kelset the issue is still persisting. I've updated the library to the newest version and the same thing still happens. Is this even a case someone will ever fix? Or let's ask the other way around, what is the default way to do this? Are @nhuthan and I the only ones who are trying to do this?

Ok, thanks for letting me know.

Are @nhuthan and I the only ones who are trying to do this?

Maybe you are the only ones trying to achieve this without a state manager (redux or MobX)?

Is this even a case someone will ever fix?

Can't really tell atm. The Drawer is a "portion" of the navigation I personally don't use so I don't have much knowledge of best practices (I know @GantMan was writing some articles/tutorials which you can find linked in the README); and I think this issue, in particular, seems quite "low priority" in the grand scheme. But nothing is set in stone.

@vanBrunneren  Did you manage to find a simple solution for the Drawer nav problem ? I am facing the same issue and it is really irritating when there is no straight solution for a problem like this ...

@amirmatin no not at the moment, I didn't had the time at the moment to work on this, as soon as I have more time I definitely give this another effort

@vanBrunneren  thanks for your comment.. I am giving this other lib called "react native navigation" a shot and will let you know if I had any luck .. from their example it seems that they don't have the re-mounting issue ..

+1. I would also like to see this implemented. If you have a very 'heavy' screen, you shouldn't be forced to re-render it every time.

@Jaftem I am using this other lib called "react native navigation" which doesnt have this issue.. It has less documentation but the example they provide is really all you would need ..

I have also ran into this issue and it's a pain.
I swapped over to react-native-navigation as suggested by @amirmatin and it initially worked well. However I ran into a bunch of limitations/bugs with that component and I am therefore hoping that there will soon be a fix to the unmount problem in react-navigation so I can go back to using it.

I agree this behavior should not be the default for drawer.. I think it should work just like tabs.

possibly (?) related: https://github.com/react-navigation/react-navigation/pull/3164

@brentvatne Do you know if this is going to be addressed to work just like tabs? And if so, is there any timeline for when it might be looked at?
Thx!

@brentvatne Please add an option to toggle between the two behaviors instead of replacing the current behavior entirely.
Ideally, add a unmountAfterSwitch property in RouteConfigs so that we can customise the behavior for different screens; eg. set unmountAfterSwitch: true for rarely accessed screens so that they are not kept in memory if the user only switched to it once.

@jaltin - not until after we release 2.0. you could fork drawernavigator and make your own version though, eg: https://github.com/react-europe/react-europe-2018-mobile/blob/master/src/Navigation.js

Thanks for the update @brentvatne.
Do you have any idea of how long it might be before 2.0 is released?

So, since the drawer goBack() is broken / no working as intended, I've tried to find a work around. Here it is, combining drawerNavigator and StackNavigator:
// this const is here only to make thing easier to maintain, no duplicate declarations
const routes = {
  RemotePage: { screen: RemotePage },
  ReferralPage: { screen: ReferralPage },
  TestPage: { screen: ReferralPage },
}

// StackNavigator with all you routes so you can navigate easely between them
const stack = createStackNavigator({
  Home: { screen: HomePage },
  OwnerPage: { screen: OwnerPage },
  ...routes,
}, {
  initialRoute: 'Home'
})

// DrawerNavigator and main menu on the left.
export default createDrawerNavigator({
  Home: { screen: stack,
    navigationOptions: ({ navigation }) => ({
      drawerLockMode: 'locked-closed',
    }),
  },
  ...routes,
});

This solution is quite easy to implement.
With this, the goBack() button is working as intended, and the home page is not re-mounting.
I'm using react-navigation "react-navigation": "^2.3.1", tested wit expo SDK 28 on iOS and Android.
@brentvatne Do you validate this kind of use or is it not intended and will be fixed in the future?

@poptocrack - https://github.com/react-navigation/react-navigation/issues/4474

moved this issue to new repo https://github.com/react-navigation/react-navigation-drawer/issues/4


Multi-file search & replace incorrectly selecting contents based on regular expression

VSCode Version: Code 1.12.1 (f6868fce3eeb16663840eb82123369dec6077a9b, 2017-05-04T21:26:50.689Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

rainbow-brackets
2gua
0.0.6

Example file:
  "name": "Legion - Bronze - Teleporter",
  "to_build": "LegionStructureBasicTeleporter",
  "instance_count": 1,
  "priority": 1,
  "max_num_assisters": 10,
  "builders": [
    "LegionCommander",
    "AnyLegionFabberBasic",
    "AnyLegionFabberAdvanced",
    "LegionOrbitalBasicFabber"
  ],

Steps to Reproduce:

Switch to Search
Expand Search to include replace field
Enable regular expression
In FIND box enter: ,\n      "builders":
In REPLACE box enter: ,\n      "base_sort": "FromBuilder",\n      "builders":

EXPECTED BEHAVIOUR:
File will become as follows:
  "name": "Legion - Bronze - Teleporter",
  "to_build": "LegionStructureBasicTeleporter",
  "instance_count": 1,
  "priority": 1,
  "max_num_assisters": 10,
  "base_sort": "FromBuilder",
  "builders": [
    "LegionCommander",
    "AnyLegionFabberBasic",
    "AnyLegionFabberAdvanced",
    "LegionOrbitalBasicFabber"
  ],

CURRENT BEHAVIOUR:
File becomes the following:
  "name": "Legion - Bronze - Teleporter",
  "to_build": "LegionStructureBasicTeleporter",
  "instance_count": 1,
  "priority": 1,
  "max_num_as
  "base_sort": "FromBuilder",
  "builders":
  "builders": [
    "LegionCommander",
    "AnyLegionFabberBasic",
    "AnyLegionFabberAdvanced",
    "LegionOrbitalBasicFabber"
  ],

NOTE: use CTRL-H search & replace within a file behaves according to expected behaviour.
example files.zip

A further example:
  "name": "Bronze - Basic Metal Extractor - cdr",
  "to_build": "BasicMetalExtractor",
  "instance_count": 1,
  "max_num_assisters": 3,
  "shared_instance_count": "BasicMetal",
  "priority": 261,
  "base_sort": "FromBuilder",
  "base_sort": "FromBuilder",
  "builders": [
    "UberCommander"
  ],

Find: "base_sort": "FromBuilder",\n      "base_sort": "FromBuilder",
Replace: "base_sort": "FromBuilder",
Output:
  "name": "Bronze - Basic Metal Extractor - cdr",
  "to_build": "BasicMetalExtractor",
  "instance_count": 1,
  "max_num_assisters": 3,
  "shared_instance_count": "BasicMetal",
  "priority": 261,
  "base_sort": "FromBuilder",
  "base_sort": "FromBuilder",
  "builders": [
    "UberCommander"
  ],

The problem is multi-file FIND only highlights the first base_sort line rather than both of them as it should.
And again, search & replace within a single file, not using the sidebar search function, works as expected correctly highlighting both base_sort lines and outputting the following:
  "name": "Bronze - Basic Metal Extractor - cdr",
  "to_build": "BasicMetalExtractor",
  "instance_count": 1,
  "max_num_assisters": 3,
  "shared_instance_count": "BasicMetal",
  "priority": 261,
  "base_sort": "FromBuilder",
  "builders": [
    "UberCommander"
  ],

Duplicates #13155


How can  we run ansible in rundeck with the alternate config file other that /etc/ansible/ansible.cfg ?

Docs say to use the ANSIBLE_CONFIG environment variable. I don't know off the top of my hat if the env of Rundeck gets passed to the Ansible process. If so, you could set this env var for Rundeck.
Anyhow, we could probably add another project-global option, like #45 for this.

right, you can still pass the environment variable to rundeck by defining it in /etc/rundeck/profile it will get passed to all child processes.

Hi @anochand , closing this. please let us know if you still have issues defining the env vars in rundeck. You can also post a question/issue in rundeck project github if you need more input on this.

This worked for me
project.ansible-config-file-path=


Wandsworth remove 2nd weds monthly visit slot

Tested on preprod - passed (for 2nd Weds of October).

I'll port these to the new data structure after merge.


Not working when use the latest version of jQuery

Not working when use the latest version of jQuery (v3.1.1)

I had the same issue, created a pull request for that here
https://github.com/cozeit/czMore/pull/2
In the mean time works on this fork using the jquery.czMore-latest.js file
https://github.com/t14/czMore


Fix page centering of auth index

I'm unable to repro - is this a specific Browser?

Hmm I'm unable to recreate now as well so will close out.


Hope that prov-json will be in ES documents?

Is it the plan for prov-json to be included in the ES documents? That would be my preference (fewer api requests, less risk of inconsistency, ...), but if it's hard, it's not essential.
Preferences in order:

It's already been done, and I'm just not looking in the right place.
It will be done.
It won't be done.
Saying it will be done, but never having it delivered. :)

@shirey : This is the issue I'm remembering. I will close this, if it would not be in the index but only added after the ES processing was done. Multiple API calls on my side is a little messier on my side, but if either service has problems, I'll know exactly where the problem is, and parallel requests from the client will be more performant that serial processing at the server. Thanks!