Castle.WcfIntegrationFacility target framework warning

Hi,
When attempting to reference this nuget package from a web project that targets .netcore2.0, I get the following warning:
warning NU1701: Package 'Castle.WcfIntegrationFacility 4.1.0' was restored using '.NETFramework,Version=v4.6.1' instead of the project target framework '.NETCoreApp,Version=v2.0'. This package may not be fully compatible with your project.
Can anyone explain why I'm receiving such an error since I thought this project was updated to support .NET Core recently?
I was able to get rid of the warning by updating my target framework to be .NET 4.6.2.

Yes but not the WcfIntegrationFacility. WCF to my knowledge does not run on dotnet core. Have you checked out Wcf Connected Services?

If you are referring to how Visual Studio can generate SOAP client proxies, then yes. However, I don't like the code it generates so I currently rely on this facility to create them for me.
Thanks for clarifying that WCF (server side) isn't supported in .NET Core. I do believe the client side libraries are supported which is really how I wanted to use this library. Perhaps this package can be split to remove the server aspects so that the client api can be referenced as a .NET standard project.

I have never really used WCF, so I am not sure what this would entail. Do you think you could explain what features you are after specifically? It would help steer me in the right direction.

Sure. Let's say the are WCF services that are already built and published to some url, instead of using the VS tools to add a Connected Service, I would write registration code that looks like this.
container.Register(Component.For<IHelloService>() .AsWcfClient(WcfEndpoint.At("http://someurl.com/HelloService.svc")) .LifestyleTransient()
IHelloService was defined in a common assembly that both the client and the server references. IHelloService was also marked with the [ServiceContract] attribute so WCF knows it exists.
Then to call this code, I would just have to inject IHelloService and call any method that is marked with an [OperationContract] attribute.
So, I'm really after that .AsWcfClient(...) helper. I'm not trying to write services using WCF anymore (moved to Web API); but I do frequently have to call them and this helper seems to do all the work necessary to create the SOAP client proxy.
I hope this helps. I create this repo with some sample code; but it's targeting the full .NET framework instead of the cross-platform .NET core due to the library incompatibility.

Good stuff. Give me some time to absorb this and get back to you.

I forked: https://github.com/fir3pho3nixx/windsor-aspnet-core/tree/connected-services
After downloading and installing: https://marketplace.visualstudio.com/items?itemName=WCFCORETEAM.VisualStudioWCFConnectedService
I Generated a proxy client for: http://localhost:50214/HelloService.svc
Then added 1 line of code to register the client and I was consuming the client in Index.cshtml.cs using Windsor.
What is so hard about doing this?

https://github.com/dotnet/announcements/issues/70

https://github.com/dotnet/announcements/issues/72


[move-prover] Implemented the feature of automatic inconsistency checking

Added "--check-inconsistency" flag into the Prover CLI
With the flag above, Prover performs the inconsistency checking in  addition to the regular verification
In order to perform the inconsistency checking, Prover creates another function variant as a target, and assert false; is inserted at the end of the "return" block of the new function variant
If the assertion above passes, then Prover reports it as an inconsistency error

(Prover masks the expected errors (e.g., the "assert false" failures and the timeouts) from the negative test for the inconsistency checking, and does not report them to the users.)

Some examples are in .../functional/inconsistency.move.

Motivation
The presence of inconsistency is a serious issue. If there is an inconsistency in the verification assumption (perhaps due to a specification mistake or a Prover bug), any false post-condition can be proved vacuously. This PR adds to the Prover a new feature to automatically check the inconsistency and to ensure the absence of it.
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
cargo test
and see the .../functional/inconsistency.move.

Please advertise on Slack once the  PR lands that folks need to run dev_setup.sh once they sync up to the commit of this PR.

/land

:exclamation: Land has been canceled due to this PR being updated with new commits. Please issue another Land command if you want to requeue this PR.

/land

:broken_heart: Test Failed - ci-test-success

/land


Added feature for showing Ecosystem name in the table along with othe…

Issue description:
The summary reporter presents a table of packages that are recommended for upgrade. It does not show the ecosystem of the package and only shows the name and version. While this is fine for a scan where only a single lockfile was scanned, this is a problem where multiple lockfiles were scanned with different ecosystem
Screenshot-2023-04-09-170817
To start improving the UX, we should start by showing the Ecosystem name in the report report table.
Let me know if there are any changes instead of closing this PR.
What is drift I don't know I kept as it is.
#54

@tarunsamanta2k20 Please try to build your branch. If you have Go tool chain setup, you can build vet just with:
go build

@abhisek which ecosystem variable should i use? I mean it is defied in lots of files.

@abhisek

@abhisek

@tarunsamanta2k20 Looks great. Just a small request. Can we have Ecosystem as the first column? The justification for this is so that Package column which includes both name and version is followed by Update Version i.e. the latest version available for the package.
The table will be more intuitive if we have the the version to update just next to the package.

@abhisek please add me as a member of your organization. I wanna contribute more in tte future.


Add B

Bigquery allows for different field delimiters such as the pipe character "|" but this library does not support it
It would need to be added to the configuration.load.fieldDelimiter section of the job
https://github.com/spotify/luigi/blob/master/luigi/contrib/bigquery.py#L445

Can you edit issue title to something more descriptive please?

I already have a PR for this #1619. I'll just close this.


extended conformance - HTTPResponseHeaderModification

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
Support the HTTPResponseHeaderModification feature in conformance tests (extended)
Acceptance Criteria

[ ] the conformance test HTTPRouteResponseHeaderModifier is successfully ran

Can I work on this? Thanks!

Thanks reply ! According to my understanding, I should add an response-transform when a filter of responseHeaderModifier type is parsed, and I am confused that the conformance mentined here is just gateway-api or anything else? It's very possible I'm missing something, thanks for correcting me.

The conformance test mentioned in the issue is from the Gateway API suite: https://github.com/mlavacca/gateway-api/blob/a4e8bad07dedaf738cce83cb44a4e8641937fade/conformance/tests/httproute-response-header-modifier.go#L33
The final outcome of this issue should be having that test passed. As you can see in our conformance tests, we are currently skipping it.
For what concerns what needs to be done, we should convert that filter into a kong plugin (as already done for other filters), but may require an investigation of the problem and changes in other parts of the codebase, it needs to be figured out.

I have tried to implement this extended filter by generate a response-transform plugin with headers config, as requestHeaderModifier does. After remove the HTTPRouteResponseHeaderModifier from skip list, the gateway-api conformance test seems passed, would you like to give me some little hints about the other parts of the codebase? Thanks!

I have tried to implement this extended filter by generate a response-transform plugin with headers config, as requestHeaderModifier does. After remove the HTTPRouteResponseHeaderModifier from skip list, the gateway-api conformance test seems passed, would you like to give me some little hints about the other parts of the codebase? Thanks!

Cool! If you want to publish the PR, it'd be great :) As I said, this issue required a bit of investigation, it may be that we don't need to change anything else than what you already did :)

I have published a PR for this issue. Thanks!


SOP for data submitters to upload data files

This PR requests integrating into the master branch an SOP document for data contributors to upload data files to an upload area. Data wranglers would like to be able to link to these instructions when interacting with new data contributors.

Codecov Report

Merging #173 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #173   +/-   ##
======================================
  Coverage    86.9%   86.9%
======================================
  Files          34      34
  Lines        1291    1291
======================================
  Hits         1122    1122
  Misses        169     169

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 99c4b80...0dd38c8. Read the comment docs.

I have rebased to master and now all the checks have passed. I'm not sure what your policy is on merging one's own PRs. Happy to do it myself, or happy for anyone else to do it! Also @hewgreen could you approve this PR if you're happy with it?

@malloryfreeberg, you need one approval. You may squash and merge your changes into master.


nixos/httpd: add locations and directories options to virtualHosts

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@zimbatm how familiar are you with nginx configuration on NixOS? I'm adding a locations option to the httpd service, which is pretty much a copy+paste of the nginx code. Due to the differences in httpd and nginx configuration I figure a locations option won't be sufficient, and a directories option will be required too.
Thanks to your explanation of unsafeDiscardStringContext I'm now trying to decide if a directories option is "worth it" or not. I think overall it is, though maybe with some documented caveats.
A simple example of using directories safely with no problem:
services.httpd = {
  enable = true;
  virtualHosts.localhost = {
    directories."/var/www" = {
      alias = "/";
      index = "index.html";
      extraConfig = "Require all granted";
    };
  };
};

An example which might not be so safe:
services.httpd = {
  enable = true;
  virtualHosts.localhost = {
    directories."${builtins.unsafeDiscardStringContext pkgs.someWebApp}" = {
      alias = "/";
      index = "index.html";
      extraConfig = "Require all granted";
    };
  };
};

Usually web app modules will reference pkg.someWebApp elsewhere in the module, so practically speaking this probably isn't an issue.
I appreciate any and all feedback @zimbatm. Thanks!

Avoid using unsafeDiscardStringContext unless you can prove that it will work as intended. For example in your second example, because the only reference to pkgs.someWebApp is without context, nixos-rebuild switch won't build pkgs.someWebApp and the httpd config will be left with a dangling reference in there. So it's quite risky.
Probably the best approach is to copy the nginx model and have a set of HTTP aliases that map back to roots. Collect all the roots into a list and then generate the read-only directory entries.

I'm not familiar with nginx so I'll have to do some research. Thank you for advice!

@zimbatm I've looked through nginx documentation and understand what it is doing more specifically now. The approaches taken by the two web servers are fundamentally different it appears. With httpd you should never manage permissions with Location directives, only Directory directives. Not all Directory directives have one-to-one virtual mappings, so this doesn't quite line up with only a locations option.
A quick example of where generating both Directory and Location directives from a single virtualHosts.<name>.locations option would fall short:
DocumentRoot /var/www
<Directory /var/www>
  Require valid-user
</Directory>

With httpd you want to be able to specify real file level paths without having to specify a virtual mapping and this is why I continue to think that a directories option is appropriate. I guess the question becomes: is there a better way to implement a directories option than I have, that doesn't depend on unsafeDiscardStringContext :thinking:
As always, feedback is greatly appreciated. Thanks!

Fundamentally we are trying to map the httpd config format to JSON-like data structures. I don't remember the syntax well enough but I think it's not completely straight-forward. Which is why the current state of the art is to write the config by hand.
Try and take a look at the syntax and thing how it could be represented in pure nix data.
Alternatively the directories could also be transformed into lists to bypass that specific problem:
services.httpd = {
  // A list of directory configs
  virtualHosts.localhost.directories = [
    {
      path = pkgs.someWebApp;
      alias = "/";
      index = "index.html";
      extraConfig = "Require all granted";
    };
  ];
};

@zimbatm part of the problem I'm trying to solve is making directories and locations able to merge. The httpd already has a list of directories. Once you declare a directory in a list it can't be accessed and modified, or merged with something else.

What should happen if two different modules set values for the same location? Should they merge, fail or the last one wins? Potentially this could be handled by a filter operation after the fact.

I would expect some merging and mkForce semantics. extraConfig is lines so merge by default. index, etc... are nullOr so I imagine in many cases nothing required as its mostly extraConfig you want to modify.
extraConfig = "Require all granted" -> extraConfig = "Require valid-user, for example.


control when to run validation with do_early_stopping

This fix restores the behavior back to previous when do_early_stopping=False.

I think the inner train functionality (starting from the batch object) can be encapsulated into a function, and once that happens, this would clean up the code here and reduce it significantly.  WDYT?


[Identity] Update AzurePipelinesCredential

OIDC API version updated to 7.1.
Use the SYSTEM_OIDCREQUESTURI envvar for the base OIDC url
Added a new required parameter: "system_access_token".

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-identity

/check-enforcer override


Release Google.Cloud.Common version 2.2.0

Changes in this release:
New features

Change netstandard2.1 target to netstandard2.0 (commit 82bea85)

Created release for Google.Cloud.Common-2.2.0


SDk reference issues

Team,
We started seeing SDK issues with our builds. Can you share some insights of something changed ? this was all working fine till last Friday.
##[error]C:\Users\VssAdministrator\.nuget\packages\msbuild.sdk.sqlproj\3.0.0\Sdk\Sdk.targets(100,5): Error : The current SDK bundles target framework '6.0' which is not supported by MSBuild.Sdk.SqlProj. Either install a .NET SDK that supports 8.0 or 9.0, or file an issue at https://github.com/rr-wfm/MSBuild.Sdk.SqlProj/ to add support. If you have preview SDKs installed, you may have to use a global.json file: https://docs.microsoft.com/dotnet/core/tools/global-json

Example CSProj
<Project Sdk="MSBuild.Sdk.SqlProj/3.0.0">
  <PropertyGroup>
    <TargetFramework>netstandard2.0</TargetFramework>
    <SqlServerVersion>Sql150</SqlServerVersion>
    <SuppressTSqlWarnings>71558,71502</SuppressTSqlWarnings>
    <TargetName>CAPSHistoryDatabase</TargetName>
    <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
    <GenerateCreateScript>True</GenerateCreateScript>
  </PropertyGroup>

  <ItemGroup>
    <None Include="SchemaComparisons\SchemaComparison.scmp" />
    <None Include="shared.publish.xml" />
  </ItemGroup>
  <ItemGroup>
    <PreDeploy Include="Scripts\Script.PreDeployment.sql" />
    <PostDeploy Include="Scripts\Script.PostDeployment.sql" />
    <None Include="package.nuspec">
      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
      <Private>true</Private>
    </None>
  </ItemGroup>
  <ItemGroup>
    <Content Remove="Scripts\Script.PreDeployment.sql" />
    <Content Remove="Scripts\Script.PostDeployment.sql" />
    <Content Remove="Scripts\Post-Deployment\2446.0-PostDeployment.sql" />
    <Content Remove="Scripts\Pre-Deployment\2446.0-PreDeployment.sql" />
  </ItemGroup>
</Project>

@ErikEJ Can you share some thoughts?

Sorry just saw this https://github.com/rr-wfm/MSBuild.Sdk.SqlProj/pull/634

Weird that older versions have the same issue as well @ErikEJ

Ignore my comment

@ErikEJ do you think we are doing something wrong here ?
D:\a\.nuget\cache\Tiger\6.2.1\msbuild.sdk.sqlproj\2.9.1\Sdk\Sdk.targets(100,5): Error : The current SDK bundles target framework '9.0' which is not supported by MSBuild.Sdk.SqlProj. Either install a .NET Core SDK that supports 3.1, 5.0 or 6.0, or file an issue at https://github.com/rr-wfm/MSBuild.Sdk.SqlProj/ to add support. If you have preview SDKs installed, you may have to use a global.json file: https://docs.microsoft.com/dotnet/core/tools/global-json

For version 2.x you need the .NET 8 SDK, for version 3.0 you need either. Net 8 or. NET 9.

Yes, the error message is not correct..

I don't think it is incorrect, but I guess it is a bit ambiguous. The current SDK actually refers to version of the .NET SDK being used, not the version of MSBuild.Sdk.SqlProj. Would changing that bit to The current .NET SDK help?

It was the part referring to .NET Core 3.1 I was referring to.

Fair enough. Not sure it is worth patching this in older version though.

Agree, maybe just have a second look at the current error message?

Current message reads:
The current SDK bundles target framework '$(BundledNETCoreAppTargetFrameworkVersion)' which is not supported by MSBuild.Sdk.SqlProj. Either install a .NET SDK that supports 8.0 or 9.0, or file an issue at https://github.com/rr-wfm/MSBuild.Sdk.SqlProj/ to add support. If you have preview SDKs installed, you may have to use a global.json file: https://docs.microsoft.com/dotnet/core/tools/global-json

So I guess that's fine :)

Yes, it's fine until we forget to update it for .NET 10 😅


Exact time of day and better intervals between reports being sent

Should change the implementation to send, for example, everyday at 7am.
Current implementation is set to daily, weekly, 30 days (monthly) etc is just the interval being the number of seconds in that duration. Then the initial "nextReportTime" is set to 0. Every 10 mins the scheduler checks if the current time is greater than any nextReportTime, if there is, sends that scheduled report and increments the nextReportTime by the unix timestamp in interval.
Need to : Change UI to select an interval: Daily, Weekly, Monthly etc and also a time of day: 7am, 8am etc. and possibly, if it's weekly: What day of the week, if its monthly, what day of the month (Or potentially also "last day of month", "first of the month" etc??)
Not sure exactly what to do as there is quite a lot there. I suppose you would treat them all seperately, and have an initial step: Daily, Weekly, Monthly, then if it's daily: Time of day, otherwise weekly: Number of weeks (to handle fortnightly), day of week, then time of day. Then for monthly, number of months, day of week, time of day?
If it was done like this, I suppose you could map this into a cron scheduler string and then just start a cron job for each schedule.. but I'm not convinced this is a good idea, i'd rather save an exact time of the next report in a persistent db and check against that and increment, I think it seems safer! But determining a way to do that with these changes is more difficult, so I dunno.

@mark-prins  Been thinking about this a bit. I think the checking every 10 minutes for emails to be sent, sending them and updating the nextReportTime (next send time??) is fine, it's mainly the saving the details of calculating when the nextReportTime is..
So the two problems:

An initial send time: Currently set to 0 and this will send on the next 10minute schedule
Calculating the next send time: Currently just time now + hard coded number of seconds as per simplistic interval: 1month = 30days worth of seconds, 1 week = 7 days worth of seconds etc.

For the first: What if we choose the "Next report day" directly, then choose then time of day and and interval: Daily, Weekly, Monthly, fortnightly, 6 months, 12 months(???). . Then we save nextReportTime to be the date that was actually selected.
For the second: We actually save this as: TimeOfDay, Interval [int], Unit [Daily/Weekly/Monthly] and since we have the nextReportTime set, we just need the interval and unit and after each sent report, take the nextReportTime and nextReportTime.AddDate() with the unit and interval, and adjust the time of the day (Otherwise it might just get out of whack)
Although this doesn't handle "Last day of the month" very well.. I've tried looking around for implementations as this seems like a common enough problem but haven't been able to find any examples!


Unicode not supported

I call "WriteTag" with unicode characters and the MP3 title was updated with "???:????". Apparently it does not support unicode characters.

I've seen Unicode code in the code so there must be some type of support😃.  Can you upload the file?
Can you share more details?  What's the current title value?

I can reproduce this bug.
Example of code below :
tag.Title = "Kid Chameleon — Box ✅ Manual ✅";
[...]
mp3.WriteTag(tag, WriteConflictAction.Replace);

The result text in the tag is :
Kid Chameleon — Box ? Manual ?
It should be :
Kid Chameleon — Box ✅ Manual ✅

I can reproduce this bug. Example of code below :
tag.Title = "Kid Chameleon — Box ✅ Manual ✅";
[...]
mp3.WriteTag(tag, WriteConflictAction.Replace);

The result text in the tag is : Kid Chameleon - Box ? Manual ? It should be : Kid Chameleon — Box ✅ Manual ✅

I've solved your problem!
tag.Title = "Kid Chameleon — Box ✅ Manual ✅";// implicit convert string to TextFrame which is default encoding.

/***important line***/
tag.Title.EncodingType = Id3TextEncoding.Unicode;
/***important line***/
mp3.WriteTag(tag, WriteConflictAction.Replace);

Oh thanks @shamork !
I forgot to update this issue, i solved it the same way some time ago.
tag.Title = new Id3.Frames.TitleFrame()
{
    EncodingType = Id3TextEncoding.Unicode,
    Value = "Kid Chameleon — Box ✅ Manual ✅"
};


[BUG] Mount fails for a restored xfs volume.

Describe the bug
Longhorn is unable to mount the restored volume if the filesystem is XFS.
To Reproduce
Steps to reproduce the behavior:

Create a volume of XFS filesystem.
Write some data into it and compute md5sum.
Take a backup in S3 bucket.
Restore the backup and create PV with fsType with XFS or ext4.
Create a deployment/statefulSet with the restored volume at step 4. The mount fails with below error.

MountVolume.SetUp failed for volume "pvc-test-new" : rpc error: code = Internal desc = mount failed: exit status 32 Mounting command: mount Mounting arguments: -t xfs -o defaults /dev/longhorn/restore-new /var/lib/kubelet/pods/e492b530-192f-4c69-a28d-5338762269f9/volumes/kubernetes.io~csi/pvc-test-new/mount Output: mount: /var/lib/kubelet/pods/e492b530-192f-4c69-a28d-5338762269f9/volumes/kubernetes.io~csi/pvc-test-new/mount: wrong fs type, bad option, bad superblock on /dev/longhorn/restore-new, missing codepage or helper program, or other error.

Expected behavior
Mount should succeed.
Environment:

Longhorn version: Longhorn-master-head 05/28/2021
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: RKE1- K8s 1.20.4

Number of management node in the cluster: 1
Number of worker node in the cluster: 3

Node config

OS type and version: Ubuntu
CPU per node: 2 vcpus
Memory per node: 4 Gi
Disk type(e.g. SSD/NVMe): SSD
Network bandwidth between the nodes: upto 5 Gi

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): DO
Number of Longhorn volumes in the cluster: 2

This issue is seen only when the PV and PVC is created using Rancher UI.
The PV is as below
apiVersion: v1
kind: PersistentVolume
metadata:
  annotations:
    field.cattle.io/creatorId: user-kpt8f
    pv.kubernetes.io/bound-by-controller: "yes"
  creationTimestamp: "2021-06-01T20:50:20Z"
  finalizers:
  - kubernetes.io/pv-protection
  - external-attacher/driver-longhorn-io
  labels:
    cattle.io/creator: norman
  managedFields:
  - apiVersion: v1
    fieldsType: FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          .: {}
          f:field.cattle.io/creatorId: {}
        f:labels:
          .: {}
          f:cattle.io/creator: {}
      f:spec:
        f:accessModes: {}
        f:capacity:
          .: {}
          f:storage: {}
        f:csi:
          .: {}
          f:driver: {}
          f:fsType: {}
          f:volumeAttributes:
            .: {}
            f:numberOfReplicas: {}
            f:size: {}
            f:staleReplicaTimeout: {}
          f:volumeHandle: {}
        f:persistentVolumeReclaimPolicy: {}
        f:volumeMode: {}
    manager: rancher
    operation: Update
    time: "2021-06-01T20:50:20Z"
  - apiVersion: v1
    fieldsType: FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          f:pv.kubernetes.io/bound-by-controller: {}
      f:spec:
        f:claimRef:
          .: {}
          f:apiVersion: {}
          f:kind: {}
          f:name: {}
          f:namespace: {}
          f:resourceVersion: {}
          f:uid: {}
      f:status:
        f:phase: {}
    manager: kube-controller-manager
    operation: Update
    time: "2021-06-01T20:50:40Z"
  - apiVersion: v1
    fieldsType: FieldsV1
    fieldsV1:
      f:metadata:
        f:finalizers:
          v:"external-attacher/driver-longhorn-io": {}
    manager: csi-attacher
    operation: Update
    time: "2021-06-01T20:51:26Z"
  name: restore-3
  resourceVersion: "28651"
  uid: 642cdf56-eecf-4f05-9a55-2b32d1dc8542
spec:
  accessModes:
  - ReadWriteOnce
  capacity:
    storage: 10Gi
  claimRef:
    apiVersion: v1
    kind: PersistentVolumeClaim
    name: restore-3
    namespace: default
    resourceVersion: "28409"
    uid: 42cb6510-9afc-4d38-a01d-b8cd5ad9aed4
  csi:
    driver: driver.longhorn.io
    fsType: xfs
    volumeAttributes:
      numberOfReplicas: "3"
      size: 2Gi
      staleReplicaTimeout: "20"
    volumeHandle: restore-3
  persistentVolumeReclaimPolicy: Retain
  volumeMode: Filesystem
status:
  phase: Bound

I was able to reproduce this on master-head.
Steps:

Created a volume, PV/PVC with XFS file system using Longhorn GUI.
Write data into it.
Took a backup.
Restored the backup.
Created PV/PVC with XFS file system using Longhorn GUI.
Created a deployment using the PVC, pod stuck with error.
MountVolume.MountDevice failed for volume "restore-1" : rpc error: code = Internal desc = mount failed: exit status 32 Mounting command: mount Mounting arguments: -t xfs -o defaults /dev/longhorn/restore-1 /var/lib/kubelet/plugins/kubernetes.io/csi/pv/restore-1/globalmount Output: mount: /var/lib/kubelet/plugins/kubernetes.io/csi/pv/restore-1/globalmount: wrong fs type, bad option, bad superblock on /dev/longhorn/restore-1, missing codepage or helper program, or other error.

longhorn-support-bundle_cf165717-2e60-47ae-905a-4ffc9f98eecc_2022-01-04T22-19-55Z.zip
Restored Volume name - restore-1
Time stamp - around 2022-01-04 22:18:00

I had the same problem.
Ask for help!!

@hellocloudnative What are the OS and kernel version of your nodes?


Blockheight and Block hash get params

Add another endpoint for https://github.com/getumbrel/umbrel-dashboard/pull/6
$ curl "http://localhost:3005/v1/bitcoind/info/block?height=1"

{"height":"00000000839a8e6886ab5951d76f411475428afc90947ee320161bbf18eb6048"}

$  curl "http://localhost:3005/v1/bitcoind/info/block?hash=00000000839a8e6886ab5951d76f411475428afc90947ee320161bbf18eb6048"

{"block":"00000000839a8e6886ab5951d76f411475428afc90947ee320161bbf18eb6048","confirmations":626872,"size":215,"height":1,"blocktime":1231469665,"prevblock":"000000000019d6689c085ae165831e934ff763ae46a2a6c172b3f1b60a8ce26f","nextblock":"000000006a625f06636b8bb6ac7b960a8d03705d1ace08b1a19da3fdcc99ddbd","transactions":["0e3e2357e806b6cdb1f70b54c3a3a17b6714ee1f0e68bebb44a74b1efd512098"]

I thought it was a draft PR, didn't realize you already got it working lol. Tested and merged, cheers!


Package Dependencies - swap "sf" for "rgdal" and "sp"

Describe the bug
Packages "rgdal" and "sp" to be deprecated at the end of 2023.
Switch to "sf".
To Reproduce
NA
Expected behavior
NA
Screenshots
NA
Additional context
Add any other context about the problem here.

package rgdal

package sp

For context:
https://www.r-bloggers.com/2022/04/r-spatial-evolution-retirement-of-rgdal-rgeos-and-maptools/

Modify rgdal::readOGR(dsn, layer) to sf::st_read(dsn, layer).

Removed sp references in global.R by removing "test" code section in the file.

Update map_base in global.R.  Use geom_sf() directly on the sf object layer instead of geom_polygon() and a fortified layer.

Also had to remove coord_fixed() from the map.


Allow custom tolerance levels for ScaledObject

Use-Case
We have a configuration for HPA called horizontal-pod-autoscaler-tolerance in Kube Controller Manager, defaulted to 0.1.
A ScaledObject should be able to set a custom value for this, overriding the default value. This has already been done for polling interval.
Specification
Add an optional float field tolerance in ScaledObject.spec. This value must override the default value from Kube Controller Manager.

@GautamSinghania this setting is set on K8s Controller level? Or is it a HPA setting that can be set on HPA resource? If the latter then we can add this option, but if it is something on a controller level, then we have no means to do that.
Could you point me to the specific documentation of this feature?

It is a controller level config. You can search for horizontal-pod-autoscaler-tolerance in Kube Controller Manager, it is defaulted to 0.1.
If it cannot be handled by KEDA, please feel free to close this ticket. I have raised a parallel ticket in K8, see https://github.com/kubernetes/kubernetes/issues/116984.


Fix Refresh Button to Follow PatternFly Guidelines

Currently, the refresh button in the logging-view-plugin doesn't follow PatternFly guidelines since it's a primary button which conflicts with the Run Query button. According to PatternFly, there can only be one primary button on a page. This PR is to address that.

Great addition, thank you @jatinsu!
/lgtm

/cherry-pick release-6.0


Pair Programming Contact Section

In this section the following changes/additions occurred:
-Addition of the contact section heading and paragraph
-Addition of the contact form
-Styling the contact section heading and form

@Olamarx Please check the code. There's a very huge contribution from my partner.

First pair programming session merged. Progress...!!!


fix: added omitempty on children type on struct block

added ommitempty on children block, so when passed nil, it will ommit ant not make nil. right now when creating pake will make error
body failed validation: body.children[0].paragraph.children should be an array or `undefined`, instead was `null`.

Awesome! Thanks!!


HELP: Integration with django-storages (sftp or ftp)

Hello I'm trying to work an integration using django-storages (sftp or ftp), but it is not working.
Can someone help me, please?
When I work using django.core.files.storage.FileSystemStorage it works normally.
Should it work using django-storages (sftp)? Or not?
Thanks.
class AbstractChunkedUpload(models.Model):
    def append_chunk(self, chunk, chunk_size=None, save=True):
        self.file.close()
        with open(self.file.path, mode='ab') as file_obj:  # mode = append+binary
            file_obj.write(chunk.read())  # We can use .read() safely because chunk is already in memory

        if chunk_size is not None:
            self.offset += chunk_size
        elif hasattr(chunk, 'size'):
            self.offset += chunk.size
        else:
            self.offset = self.file.size
        self._md5 = None  # Clear cached md5
        if save:
            self.save()
        self.file.close()  # Flush

api_1  | This backend doesn't support absolute paths. Traceback (most recent call last):
api_1  |   File "/usr/share/scapole/upload/models.py", line 12, in append_chunk
api_1  |     print(self.file.path)
api_1  |   File "/home/scapole/.local/lib/python3.9/site-packages/django/db/models/fields/files.py", line 57, in path
api_1  |     return self.storage.path(self.name)
api_1  |   File "/home/scapole/.local/lib/python3.9/site-packages/django/core/files/storage.py", line 116, in path
api_1  |     raise NotImplementedError("This backend doesn't support absolute paths.")
api_1  | NotImplementedError: This backend doesn't support absolute paths.

class MyChunkedUpload(ChunkedUpload):     def append_chunk(self, chunk, chunk_size=None, save=True):         self.file.close()         # We can use .read() safely because chunk is already in memory         chunk_read = chunk.read()         # mode = append+binary         SFS.save_custom(self.file.name, chunk_read)         if chunk_size is not None:             self.offset += chunk_size         elif hasattr(chunk, 'size'):             self.offset += chunk.size         else:             self.offset = self.file.size         self._md5 = None  # Clear cached md5         if save:             self.save()         self.file.close()  # Flush

class MyChunkedUpload(ChunkedUpload):
    def append_chunk(self, chunk, chunk_size=None, save=True):
        self.file.close()
        # We can use .read() safely because chunk is already in memory
        chunk_read = chunk.read()
        # mode = append+binary
        SFS.save_custom(self.file.name, chunk_read)
        if chunk_size is not None:
            self.offset += chunk_size
        elif hasattr(chunk, 'size'):
            self.offset += chunk.size
        else:
            self.offset = self.file.size
        self._md5 = None  # Clear cached md5
        if save:
            self.save()
        self.file.close()  # Flush

from chunked_upload.models import ChunkedUpload
# 'ChunkedUpload' class provides almost everything for you.
# if you need to tweak it little further, create a model class
# by inheriting "chunked_upload.models.AbstractChunkedUpload" class

from scapoleaffiliate.storage_backends_sftp import ProductionFilesSSHStorage

SFS = ProductionFilesSSHStorage()

class MyChunkedUpload(ChunkedUpload):
    def append_chunk(self, chunk, chunk_size=None, save=True):
        self.file.close()
        # We can use .read() safely because chunk is already in memory
        chunk_read = chunk.read()
        # mode = append+binary
        SFS.save_custom(self.file.name, chunk_read)
        if chunk_size is not None:
            self.offset += chunk_size
        elif hasattr(chunk, 'size'):
            self.offset += chunk.size
        else:
            self.offset = self.file.size
        self._md5 = None  # Clear cached md5
        if save:
            self.save()
        self.file.close()  # Flush

from chunked_upload.models import ChunkedUpload
# 'ChunkedUpload' class provides almost everything for you.
# if you need to tweak it little further, create a model class
# by inheriting "chunked_upload.models.AbstractChunkedUpload" class

from scapoleaffiliate.storage_backends_sftp import ProductionFilesSSHStorage

SFS = ProductionFilesSSHStorage()

class MyChunkedUpload(ChunkedUpload):
    def append_chunk(self, chunk, chunk_size=None, save=True):
        self.file.close()
        # We can use .read() safely because chunk is already in memory
        chunk_read = chunk.read()
        # mode = append+binary
        SFS.save_custom(self.file.name, chunk_read)
        if chunk_size is not None:
            self.offset += chunk_size
        elif hasattr(chunk, 'size'):
            self.offset += chunk.size
        else:
            self.offset = self.file.size
        self._md5 = None  # Clear cached md5
        if save:
            self.save()
        self.file.close()  # Flush

import posixpath
from storages.backends.sftpstorage import SFTPStorage

class ProductionFilesSSHStorage(SFTPStorage):
    def save_custom(self, name, content):
        """Save file via SFTP."""
        path = self._remote_path(name)
        dirname = posixpath.dirname(path)
        if not self.exists(dirname):
            self._mkdir(dirname)
        f = self.sftp.open(path, 'ab')
        f.write(content)
        f.close()
        # set file permissions if configured
        if self._file_mode is not None:
            self.sftp.chmod(path, self._file_mode)
        if self._uid or self._gid:
            self._chown(path, uid=self._uid, gid=self._gid)
        return name


Issue using Oracle JDBC with logstash 6.7 and Java 11

Hello,
I am using the logstash JDBC plugin to move data from Oracle to Elasticsearch.  Everything works great in Java 8 environments... however I also have Java 11 environments, and Logstash 6.6 is not compatible.
Logstash 6.7 (which is supposed to be Java 11 compatible) was released today, and the new version starts up fine in Java 11. However, the JDBC plugin gets an error when it tries to load the driver.  Switching to Oracle's latest ojdbc jar file did not fix the problem.
Here is my configuration info I'm using in the conf file:
jdbc {
jdbc_driver_library => "${ES_BIN}/ojdbc8.jar"
jdbc_driver_class => "Java::oracle.jdbc.OracleDriver"
jdbc_connection_string => "${JDBC_CON}"
jdbc_user => "${JDBC_USER}"
jdbc_password => "${JDBC_PASSWORD}"
And here's the error message:
[ERROR][logstash.inputs.jdbc     ] Failed to load /usr/nlm/vsac/apps/es/bin/ojdbc7.jar {:exception=>#<TypeError: failed to coerce jdk.internal.loader.ClassLoaders$AppClassLoader to java.net.URLClassLoader>}
Failed to load /usr/nlm/vsac/apps/es/bin/ojdbc8.jar {:exception=>#<TypeError: failed to coerce jdk.internal.loader.ClassLoaders$AppClassLoader to java.net.URLClassLoader>}
{ 2032 rufus-scheduler intercepted an error:
2032   job:
2032     Rufus::Scheduler::CronJob "/1 * * * * *" {}
2032   error:
2032     2032
2032     LogStash::ConfigurationError
2032     Java::oracle.jdbc.OracleDriver not loaded. Are you sure you've included the correct jdbc driver in :jdbc_driver_library?
2032       /usr/nlm/vsac/apps/es/logstash-6.7.0/vendor/bundle/jruby/2.5.0/gems/logstash-input-jdbc-4.3.13/lib/logstash/plugin_mixins/jdbc/jdbc.rb:163:in open_jdbc_connection'   2032       /usr/nlm/vsac/apps/es/logstash-6.7.0/vendor/bundle/jruby/2.5.0/gems/logstash-input-jdbc-4.3.13/lib/logstash/plugin_mixins/jdbc/jdbc.rb:221:in execute_statement'
2032       /usr/nlm/vsac/apps/es/logstash-6.7.0/vendor/bundle/jruby/2.5.0/gems/logstash-input-jdbc-4.3.13/lib/logstash/inputs/jdbc.rb:277:in execute_query'   2032       /usr/nlm/vsac/apps/es/logstash-6.7.0/vendor/bundle/jruby/2.5.0/gems/logstash-input-jdbc-4.3.13/lib/logstash/inputs/jdbc.rb:258:in block in run'
2032       /usr/nlm/vsac/apps/es/logstash-6.7.0/vendor/bundle/jruby/2.5.0/gems/rufus-scheduler-3.0.9/lib/rufus/scheduler/jobs.rb:234:in do_call'   2032       /usr/nlm/vsac/apps/es/logstash-6.7.0/vendor/bundle/jruby/2.5.0/gems/rufus-scheduler-3.0.9/lib/rufus/scheduler/jobs.rb:258:in do_trigger'
2032       /usr/nlm/vsac/apps/es/logstash-6.7.0/vendor/bundle/jruby/2.5.0/gems/rufus-scheduler-3.0.9/lib/rufus/scheduler/jobs.rb:300:in block in start_work_thread'   2032       /usr/nlm/vsac/apps/es/logstash-6.7.0/vendor/bundle/jruby/2.5.0/gems/rufus-scheduler-3.0.9/lib/rufus/scheduler/jobs.rb:299:in block in start_work_thread'
2032       org/jruby/RubyKernel.java:1411:in loop'   2032       /usr/nlm/vsac/apps/es/logstash-6.7.0/vendor/bundle/jruby/2.5.0/gems/rufus-scheduler-3.0.9/lib/rufus/scheduler/jobs.rb:289:in block in start_work_thread'
2032   tz:
2032     ENV['TZ']:
2032     Time.now: 2019-03-26 14:14:11 -0400
2032   scheduler:
2032     object_id: 2010
2032     opts:
2032       {:max_work_threads=>1}
2032       frequency: 0.3
2032       scheduler_lock: #Rufus::Scheduler::NullLock:0xeca2440
2032       trigger_lock: #Rufus::Scheduler::NullLock:0x30ea1d7f
2032     uptime: 13.49963 (13s499)
2032     down?: false
2032     threads: 2
2032       thread: #Thread:0xfdbda16
2032       thread_key: rufus_scheduler_2010
2032       work_threads: 1
2032         active: 1
2032         vacant: 0
2032         max_work_threads: 1
2032       mutexes: {}
2032     jobs: 1
2032       at_jobs: 0
2032       in_jobs: 0
2032       every_jobs: 0
2032       interval_jobs: 0
2032       cron_jobs: 1
2032     running_jobs: 1
2032     work_queue: 0
} 2032 .
Is there any way around this issue without downgrading to Java 8?

Same issue here using the Postgres driver. If I run 7.0.1 with Java 8 it works fine, if I run with Java 11 it fails.
[2019-05-16T09:49:38,870][ERROR][logstash.inputs.jdbc     ] Failed to load /Users/jason/Projects/xxx/elasticsearch/logstash-7.0.1/postgresql-42.1.1.jar {:exception=>#<TypeError: failed to coerce jdk.internal.loader.ClassLoaders$AppClassLoader to java.net.URLClassLoader>}
[2019-05-16T09:49:38,888][ERROR][logstash.javapipeline    ] A plugin had an unrecoverable error. Will restart this plugin.
  Pipeline_id:main
  Plugin: <LogStash::Inputs::Jdbc jdbc_user=>"xxx", jdbc_password=><password>, statement=>"select 1", jdbc_driver_library=>"/Users/jason/Projects/xxx/elasticsearch/logstash-7.0.1/postgresql-42.1.1.jar", jdbc_connection_string=>"jdbc:postgresql://xxx:5432/xxx?user=xxx", id=>"05944e84f25244efb3fcedede97fc299d084c38644b27df1ba33fa62d5585065", jdbc_driver_class=>"org.postgresql.Driver", enable_metric=>true, codec=><LogStash::Codecs::Plain id=>"plain_e082ca4f-16f9-4876-8cd5-d1558cc71449", enable_metric=>true, charset=>"UTF-8">, jdbc_paging_enabled=>false, jdbc_page_size=>100000, jdbc_validate_connection=>false, jdbc_validation_timeout=>3600, jdbc_pool_timeout=>5, sql_log_level=>"info", connection_retry_attempts=>1, connection_retry_attempts_wait_time=>0.5, parameters=>{"sql_last_value"=>2019-05-16 14:39:10 UTC}, last_run_metadata_path=>"/Users/jason/.logstash_jdbc_last_run", use_column_value=>false, tracking_column_type=>"numeric", clean_run=>false, record_last_run=>true, lowercase_column_names=>true>
  Error: org.postgresql.Driver not loaded. Are you sure you've included the correct jdbc driver in :jdbc_driver_library?
  Exception: LogStash::ConfigurationError

Same problem on jdbc_static filter
I explain the problem here.
I give a work around here.
We are working on a proper solution.

An additional workaround:
1 - Change the jdbc_driver_library value in logstash conf to "". i.e.:
jdbc_driver_library => ""
2 - Put the jdbc jar in the <logstash>/logstash-core/lib/jars/ directory

It's just a work around. Just copy the driver Jar file to <logstash_install_dir>/logstash-core/lib/jars/ directory.

same issue here for MariaDB, workaround works for me.

Pull Request by me is here

Seems like this issue applies to all databases. Workaround works fin but at this point I think the workaround needs to be in the official docs of the plugin.. Any installation attempt on JVM 9+ will fail without the workaround. That includes elastic.co official docker images since 7.0...

Closed by #343

Works great, thanks!

@neoghost Try upgrading the jdbc input driver to the latest version (4.3.18). This issue should be resolved.


NGINX Help for SREF\CREF uploads

Looking for some help from the author to team please.
Everything works great except SREF\CREF uploads for Midjourney fail with a 404 error when locally hosted.  Vercel is fine.
NGINX error;
"POST /mjapi/mj/submit/upload-discord-images HTTP/2.0" 404 181
Any suggestions?  I would use vercel except the uploads for Base\Blend ect.. generally fail due to the small upload size but work when locally hosted.
Thank-you

Vercel 不支持吧？

Vercel doesn't support it, right?

Works fine in Vercel, except I am restricted by the size of the image.
Thanks for the reply worked out what it was, it is an issue with the open source mid journey proxy (which I used locally).
When testing in vercel I used "https://api.openai-hk.com/relax" this endpoint supports uploads.  openai-hk is just slow for me in Australia.  Not the image create, but the download after the create, it's like the old dial-up days.
Excellent project, let me close the issue.

@Dooy fixed the speed issue with https://api.openai-hk.com added a find\replace in nginx.
sub_filter 'mjimg.open-hk.com' 'cdn.discordapp.com';
sub_filter_types application/json;
sub_filter_once off;

中国大陆是被封掉了 cdn.discordapp.com
其实 mjimg.open-hk.com 就是代理 cdn.discordapp.com 而已
老哥真不是中国大陆的？

No Australia, which reminds me it's time for a nap here, almost midnight.


TestClass.java:71-74: Replace with isJUnitExtension...

The puzzle 193-8e3fe201 from #193 has to be resolved:
https://github.com/volodya-lombrozo/jtcop/blob/6884c619a14d844da9fb635db993fc2d7f440f54/src/main/java/com/github/lombrozo/testnames/TestClass.java#L71-L74
The puzzle was created by @volodya-lombrozo on 05-Jul-23.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd the puzzle #210 is still not solved.

@0pdd the only puzzle #210 is solved here.


is stripe-react-native support affirm?

Is stripe-react-native support affirm??
if yes then can someone please provide some source or docs to integrate support for affirm with stripe in my react native application?
in release note of 0.11.0 defined
"Added Affirm and AU BECS Direct Debit support to Payment Sheet. https://github.com/stripe/stripe-react-native/pull/940"
but not able to see Affirm option in Payment sheet, event i enabled Affirm from stripe dashboard.

Im also wants to integrate affirm in react native. any news?

hi! this is in progress here- https://github.com/stripe/stripe-react-native/pull/1036

@charliecruzan-stripe Thanks to add affirm support as payment method with stripe,
Can you please help to show this model same as below image with this affirm integration flow, because when i used latest pull, there is no option or method to show this kind of affirm view or model.

Also can you please help us on one more thing,
Same as affirm SDK provide promotional text and button to
show pre-booking amount/month, how can i show this kind of button using stripe-react-native ?


Feat/generic content block

Context
Link to Notion ticket
Additional information
The component is called generic/rich-text
I haven't added the tailwind prose styling (suggested by Jan), but have the code for this if we decide to add this at a further date.
Testing (tester)
The reviewer should check on this branch:

The code
The component in Bookshop browser ( generic/rich-text )
The site in CloudCannon (link here)

Before merge (PR owner)

Delete the test site in CloudCannon

This will be used inside the simple-text-block component


Update preprocessing.py

fixes bug where setCheckState was passed an int but expected CheckState

This happens when using PySide2 as Qt backend. Will add some basic tests to make sure I am not breaking PyQt5 in the meantime.

Thanks! I definitely don't have much useful testing in this package.


Fixed coinotron rate duration api method

public domain

Now that the "m" is not being displayed by Coinotron, the above pull is showing the round duration of IXC.

api_key_duration_hour_min: BTC.\s.\s.\s.>([0-9]+) h ?([0-9]+)? ?m? works for you?
Removing the m broke it before, so it must be intentional...
Does the api method need to approximate m?

That's what the ?'s are for, the character before is option.  When coinotron hits their 12 hour mark, or whatever, and removes the "m"...bitHopper will search, but return NoneType I believe...which is then 0 for bitHopper.
api_method: re_rateduration
api_key_ghashrate: ([ 0-9]+).([ 0-9]+) GH
api_key_duration_hour_min: BTC.\s.\s.\s.>([0-9]+) h ?([0-9]+)? ?m?
api_strip: ''
The above code works for me.
Coinotron	547,337 50.18%	    *18h:0m

I am getting 4,536,000 415.87% and no round time display?


Programatically control when the VM should stop its current execution

Feature request
The Miden assembly is turning complete and it is trivial to write an infinite loop with it:
push.1 while.true push.1 end

This is a problem for transactions executed by the network. An operator must be able to stop execution of the VM under some policy, for example the current number of cycles, or if a fee schedule similar to gas is implemented, once the gas is depleted.

There could be two potential ways of handling this:

Abort the program without generating the proof.
Abort the program and generate the proof that it exceeded the number of allowed cycles.

The first one should be fairly easy to implement - but may not be as useful as, for example, we won't be able to charge transaction fees for failed transactions.
The second one will require deep structural changes in the VM and is a part of a bigger problem of how to prove execution of programs which result in an error.

This made me think, somehow we have to agree on what is the max number of cycles, because we don't want an operator to choose arbitrarily low cycles, like 1, and have users being forced to pay for the "failed transaction". One solution is to have this at the protocol level, either as a hard constant, or use something similar to block gas limit.
But a more interesting approach would be to have arbitrary limits. As long as an operator can verify the user will pay for the computation with a conditional P2ID, there shouldn't be much of a reason not do a long computation (I'm assuming transaction execution is not done in lock step to sealing blocks, but that instead blocks are sealed with whatever proofs are available, the P2ID is not really required, just nice to have because otherwise multiple operators could work on the same long computation, and only one of them is paid, which is a lot of waste).
This makes some smart contracts easier to write, because one doesn't have to be conservative on smart contract calls that have loops (for example, the depth of Merkle trees has to be limited on Ethereum smart contracts, otherwise it is possible to grow the tree to such a size that verifying a Merkle proof wouldn't fit in a block)

The first one should be fairly easy to implement - but may not be as useful as, for example, we won't be able to charge transaction fees for failed transactions.

BTW, this can be useful for running the VM outside of a client. I did write an infinite loop, and on a Mac the system will handle the process gracefully, but on a Linux machine with swap usually the complete system becomes unusable. So even though it is not that useful for the rollup/clients, it can be useful for use to limit memory usage on CLI runs.

Closed by #991 and #998.


Host telegram bot on heroku error

Hi all!
I have host an example of this library on heroku but when deploy after a few minutes I get this error on log:

Error R10 (Boot timeout) -> Web process failed to bind to $PORT within 60 seconds of launch

this is my code:
index.js:
const TelegramBot = require('node-telegram-bot-api');

// replace the value below with the Telegram token you receive from @BotFather
const token = 'xxxxxxxxxxxxxxxxxxxxxxx';

var bot= new TelegramBot(token,{polling: true});

// Matches "/echo [whatever]"
bot.onText(/\/echo (.+)/, (msg, match) => {
    // 'msg' is the received Message from Telegram
    // 'match' is the result of executing the regexp above on the text content
    // of the message

    const chatId = msg.chat.id;
    const resp = match[1]; // the captured "whatever"

    // send back the matched "whatever" to the chat
    bot.sendMessage(chatId, resp);
});

// Listen for any kind of message. There are different kinds of
// messages.
bot.on('message', (msg) => {
    const chatId = msg.chat.id;

    // send a message to the chat acknowledging receipt of their message
    bot.sendMessage(chatId, 'Received your message');
});

package.json:
{
  "name": "bot-telegram",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "start": "node index.js",
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "node-telegram-bot-api": "^0.30.0",
  }
}

How can I change the port?
Thanks in advance!

Having the exact same issue, let me know if you find a workaround

Please try webhook like this:
webhook: {
url: process.env.APP_URL || 'your app url from heroku',
port: process.env.PORT || 5000
}
like  :
webhook: {
url: process.env.APP_URL || 'https://*****.herokuapp.com',
port: process.env.PORT || 5000
}

Heroku requires you to use the env variable port, like so:
const Telegram = require('telegram-node-bot');
const tg = new Telegram.Telegram(BOT_API_TOKEN,{
webAdmin: {
port: process.env.PORT
}
});

@jgeewax Hey what has been done for me to of found this on my phone ????


proposal: Go2: error handling with #err suffix

Go Programming Experience
4 years
Other Languages Experience
C, C++, Java, Dart, zig, rust
Related Idea

Has this idea, or one like it, been proposed before?
yes  issues/21161

Does this affect error handling? yes

Is this about generics? no

Is this change backward compatible? compatible

Proposal
scr: https://github.com/mainjzb/goerr
Make # as error symbol, I believe people can learn and understand it quickly. (Maybe other symbol replace it)
#@ as handler error symbol. Help people read code and simplify code.
I have read go/issues/21161
My imaging is something similar to that.
before
n, err := io.Write(x) // 1. err as value
n, _ := io.Write(x)  // 2. ignore error

// 3. return error immediately、
n, err := io.Write(x)
if err != nil {
   return 0, err
}

// 4. wrap additional information
n, err := io.Write(x)
if err != nil {
   return 0, fmt.Error("tcp closed: %w", err)
}

// 5. panic err
n, err := io.Write(x)
if err != nil{
    panic(err)
}

now
n := io.Write(x) #err       // 1. err as value

n := io.Write(x) #@ignore   // 2. ignore error

n := io.Write(x) #@done     //  3. return error immediately、

n := io.Write(x) #@wrap("tcp closed: %w") // 4. wrap additional information

n := io.Write(x) #@must     // 5. panic err

err as value. everything is like before. just error at suffix.
ignore error. make it no easier to ignore error than before. Encourage people with additional information. igmore 6 char more than other 4 char
return error immediately. Many times, Especially in libiary. we just need returen error nothing need to do. For example: url.parseAuthority
wrap additional information. maybe more easy way is  #@wrap equal to #@wrap("io.Wirite err:"), omit parameter make it easier for people to handle error instead of ignore error.
Many third-party libraries have MustXxx() and Xxx() two sets api. Now we just need only one. Just like go keyword, we no longer need sync and async functions.

People should reduce the discussion of compilation details. Focusing on achieving agreement on lang style.
Method chaining
move error to suffix have other benefit that separate return value and error. now we can Method chaining. Some people dislike Method Chaining but I like it.
func div(a, b float64) (float64, error) {...}
func sum(a, b float64) (float64, error) {...}

func handler() (float64, error){
  res := sum(div(0,7), div(4,5)) #err
  if err != nil{
      log.Error("do calc err: ", err)
      return 0, err
  }
  return res, nil
}

Is this change backward compatible?
No
Orthogonality: How does this change interact or overlap with existing features?
This is an orthogonal new feature that can be used not only for error handling, but also to reduce repetitive boilerplate in other cases.
Would this change make Go easier or harder to learn, and why?
Most language errors are more complex.  Only by making people aware that this is complex can they pay attention to it
Cost Description
No response
Changes to Go ToolChain
All of them.
Performance Costs
Compile time: extra injection work. Run time cost: none.
Prototype
No

Thanks, but this is not like anything else in Go.  A change in Go for error handling has to do more than just reduce error boiling boilerplate: it has to fit into the rest of the language.

No change in consensus, so declined.
— rfindley for the language proposal review group


get_tables.py

The above file is in python2 and the 2to3 library doesn't work on it. Is it possible to get a python3 version of it

And also  "previous tables.json with hand-corrected names" what is this file which has to be also provided as input to get the tables.json file


Data table modal todos

This PR addresses the remaining issues in #656.
depends on https://github.com/VEuPathDB/web-http-utils/pull/4
closes #656

@eikonomega I noticed that the buttons for this feature are not using the same theme as elsewhere on the site. Any idea why?

@eikonomega I noticed that the buttons for this feature are not using the same theme as elsewhere on the site. Any idea why?

I realized this isn't an issue, since the button won't appear here when we release.


The azure account doesn't expire when configuring proxy 'Use app proxy settings/Use environment variables' without Fiddler starting

Storage Explorer Version: 1.31.0-dev
Build Number: 20230619.1
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Ventura 13.4 (Apple M1 Pro)
Architecture: x64
How Found: From running test cases
Regression From: Previous release (1.30.0)
Steps to Reproduce

Launch Storage Explorer -> Do not start Fiddler.
Click 'Edit -> Configure Proxy' -> Set 'Use app proxy settings' as the Source.
Type 'http://localhost' to the URL field.
Type '8888' to the Port field -> Click 'OK'.
There is an info bar telling 'Unable to establish connection using current proxy settings'.
Switch to the 'ACCOUNT MANAGEMENT' panel.
Check whether the singed azure account expires.

Expected Experience
The azure account expires.

Actual Experience
The azure account doesn't expire.
Additional Context

This issue also reproduces when selecting 'Use environment variables'.
Fail to expand resources on the tree view.

Verified this issue on the main build 20230627.1. Fixed.


Get callback with found words?

When using
idx.search("search")

I would like to get an return value with an array containing all the words that were found in the entry, like:
[{
    "ref": 1,
    "score": 0.87533,
    "words": ["searching", "search", "searched"]
}]

This would give me the opportunity to e.g. highlight exactly those words which were found.
Is there a way?
My problem:
When using lunr in combination with something that handles search results, like a highlighting component, inconsistent behavior will happen. For example if you search for "searched" all entries with "search" or "searched" will be found. However, just "searched" will become highlighted (because those components may not contain things that lunr has built in, like stemming). So I need a way to determine exactly those words which were found in the entry by lunr. Rebuilding the algorithm seems quite excessive and unnecessary to me.

lunr does not keep the unstemmed words for a document around, so any highlighting tool would need to be able to highlight based on the stem of the word, in your example "searched" will actually never appear in the index, it will be stemmed to just "search" (or whatever the stemmer reduced it to).
For this reason I think the only way to solve this is to be able to return the positions of the tokens in the document, this should also lead to far greater highlighting performance, rather than having to search through the entire text of a document to find the terms to highlight.

I think rebuilding the algorithm of lunr would cause unwanted issues:

double logic
double stemming time
double maintenance and maybe inconsistent behaviour
from the adapter point of view non-reusable code as other libraries may have different algorithms
Therefore I would like to exclude this option.

Isn't there an option to hook into lunr to save the unstemmed words?
What exactly do you mean with position? The character index inside an element?

:balloon: I just saw that we've got a anniversary, the 200th issue :bowtie:

Hehe, yes 2x :100:
I think, as it stands, the current implementation of lunr has taken us as far as it's possible to go. As you say, this, and other features, are kind of incompatible with the current data structures in use, and I'm hesitant to try and shoe horn in features like this.
A better approach is rethinking some of the internals to provide better support, which is what I'm currently working on, as I'm sure you understand though this is not exactly quick and time is always limited.
Providing a means for storing the positions of terms in the index allows for highlighting matched terms as well as querying by locality. The changes to implement this also open up third-party code the ability to store other meta data about tokens that might open up some other interesting possibilities.
At a high level, I'd like to have positions stored something like this:
    'document-id': {
      'tf': 0.1234,
      'positions': [{
         'start': 3,
         'length': 6
       }]

I know this doesn't really solve you're problem right now, but I think, long term, its the right decision.

Oliver, I didn't get you in that position thing. What exactly is this position about? Is it the position in the whole text of the document (stripped HTML)?

Yes, so you would know the character in text that a token started at, as well as how long it is. This should be enough to wrap those characters in some markup that can show the highlighting.
For example, in the following document:
index : 012345678
doc   : hurp durp

hurp would have a start index of 0 and a length of 4
durp would have a start index of 5 and a length of 4

Well I don't know a way to handle this in a highlighting component. Such components search in HTML elements and also inside each text node in an those elements. So there an adapter would be necessary to find out which text node stores the word with a specific index and I don't know hot to start with that.

@olivernn To make things clear: Highlighting works based on an index if it is an index relative to the parent element. But, as you meantioned, you are thinking to implement the index of the whole document. I don't know how to map a document index with an index relative to the parent DOM element.
Can you help?

@olivernn I am willing to to implement an adapter, but I need your help in order to do this!

+1 to returning match positions. It would be easy to roll our own highlighting with that. How far is the current implementation from doing that?

@kanatzidis As @olivernn seems to be quite inactive, maybe you can help. My problem with an index of matches globally is that a conversion to an index based on HTML elements would be necessary.
Take this HTML as an example:

Lorem ipsum dolor sit amet Test Lorem ipsum dolor sit amet

Now, assume that lunr would return the index position of "Test": start 27, end 31. Now a highlighting component would need to map this index with an index relative to , which would be 0 and 4. I don't see a way to realize this.

I'm not sure how you would do it operating on dom elements. In my use case I have plain text which I can modify with spans that add a highlight style.

@kanatzidis You're absolutely right, I've had a wrong way of thinking. Sometimes it helps to talk with others. So if lunr would return the index of matched terms an adapter could seek it out. Then a highlighting component could be called with that term.
So the only left question is, @olivernn do you need any help to realize this soon?

Sorry for the lack of response, I've been spending some much needed time away from a computer!
The wrapping idea came from a library a friend create a while ago: https://github.com/benpickles/wapper, it is slightly old now, but should give some ideas for implementing the wrapping of characters in some dom element.
I'm actively working on the changes required to provide the positional information of tokens, the change is part of a larger re-work of the internals of the library. I can't commit to a date etc, but it is in progress and I hope to share what I have as soon as possible.

@olivernn Any update on this?

I've been working on some changes that should enable what you need. I'm pretty close to getting at least a preview build out with some documentation, I'll be sure to update this issue when that happens.
At a high level what will happen is that the tokenizer will keep track of the positions of a token within a document, this can then be returned along with some other details about the matched term with the search results. My hope is that this should provide all the information possible to enable users to highlight matched words.

Good read here. Any updates on it? Thanks!

Wow, how time flies.
As discussed earlier in this thread, I've been working on a new version of lunr that will support highlighting matched terms.
I've finally got round to putting together an alpha release, as well as an example showing how the highlighting will work.
This is an early alpha release, but should be fairly stable. I wouldn't recommend using it in production yet, but it should be possible to start testing using it, install with $ npm install lunr@alpha.
The example repository shows how to use this new version, the interface is largely the same as the current version of lunr, but with a few small differences.
Specifically the code for highlighting search terms, which is not part of lunr, is here.
The plan is to get some feedback on this (and other) alpha releases before getting a final release together. The more people who test it out the better.

I'm wondering why you've reinvented the highlighting functionality in the wrapper file? There is already e.g. mark.js out there which is specifically built for this task.
You're using createTreeWalker with an acceptNode property. I don't think this will work in IE9. mark.js on the other side is heavily cross-browser unit-tested and knows this and further issues.

The wrapper.js script is an example of how to use the data that is now returned by lunr to highlight words. It is by no means intended to be a production ready library that is ready to use out of the box.
I wasn't aware of mark.js, having taken a look at it it seems that it will not be compatible with the results from lunr. Unless I'm mistaken it assumes it will be passed words to highlight, but, as mentioned earlier in this thread, that is something that lunr cannot provide efficiently.

One last thing, lets keep the discussion going in #25 so its all in one place.


SA1513 (ClosingCurlyBracketMustBeFollowedByBlankLine) : yield return property initialize incorrectly report on };

The following Code reports SA1513 on the }; line, where it should not have:
        private static IEnumerable<Test> Method()
        {
            yield return new Test()
            {
                prop = "A"
            };
        }

Hi @StefanSchoof,
Thanks for reporting this, and welcome to the project! :+1:
:memo: Additional note for whoever wants to fix this: it looks very similar to #615, but in this case we have an object initializer instead of an array initializer.


[BUG]  sizeof(uint32_t) not considered contant

Describe the bug
Code generated using sizeof(uint32_t) behaves like the value could change at runtime
To Reproduce
Code to reproduce the behaviour:
from libc.stdint cimport uint32_t

cdef extern from *:
    """
/* padding to align pointers to 4-bounds */
static const int int32_pad[] = {0, 3, 2, 1};
    """
    const int[4] int32_pad

cdef object test(Py_ssize_t pos):
    pos += int32_pad[pos % sizeof(uint32_t)]  # would you like some padding?
    return pos

Result
The above generated the following code for the calculation:
  __pyx_t_1 = (sizeof(uint32_t));
  if (unlikely(__pyx_t_1 == 0)) {
    PyErr_SetString(PyExc_ZeroDivisionError, "integer division or modulo by zero");
    __PYX_ERR(0, 314, __pyx_L1_error)
  }
  __pyx_v_pos = (__pyx_v_pos + (int32_pad[(__pyx_v_pos % __pyx_t_1)]));

It is... more than unlikely that t_1 will be 0 :smile:
Expected behavior
I would expect the same code generated for [pos % 4], i.e.
__pyx_v_pos = (__pyx_v_pos + (int32_pad[__Pyx_mod_Py_ssize_t(__pyx_v_pos, 4)]));

(well, actually I would have expected pos % sizeof()... but I guess you have reasons to use that macro instead of the % operator).
Environment:

OS: Linux
Python version 3.8.5
Cython version 3.0a6

If it's causing you problems then it can be turned off by putting it in a with cython.division(False): block (similarly for the macro too). https://cython.readthedocs.io/en/latest/src/userguide/source_files_and_compilation.html#compiler-directives
I don't think there's a good mechanism within Cython for marking an integer as "Cython doesn't know what value this has but it's definitely not 0". Which practically makes it quite hard to fix. Hopefully the C compiler is smart enough not to bother with this branch.

I agree that we could optimize this away (though not sure how easy it would be). However the code above is still correct and should be optimized away by any decent optimizing C compiler.

We've always had a policy to let the C compiler do what it does well, and not bother Cython with it. I think this is such a case.


how to unmount COMPLETELY

Hello. I've been using react-google-maps so far without any issues and it has been great.
There is a problem though... that if you leave google maps up and running for ... 3 days or so.  suddenly the tiles no longer work and you will get a 403 error. https://issuetracker.google.com/issues/65212482
This is an issue that google said is intended behavior. Basically when google maps starts up, it creates a session... which expires eventually (understandable)
Google's suggested resolution to this is to use an iframe for the map and then tell the iframe to reload... this is an option that is not possible for my application because of the tons of complex logic going from app to map.
SO, I've been looking into figuring out a way how to do this without refreshing the page.  I believe the issue is that the main google js (https://maps.googleapis.com/maps/api/js) is the script that creates the "session" that expires after a few days.
I noticed that when I first mount the maps, I can see "react-google-maps" add's a bunch of scripts to the header of the page, ex:

But I noticed that when I unmount the map, the script's remain in the header.
Now I do not know if this would work, but my idea was to try to figure out a way to remove those scripts from the header, and then add them back in  (hoping to essentially refresh the session)
SO... is there any way to do this with your library? Do you think what I am trying to do is possible?

after further research, I've learned that just removing the script tags will not clear all the variables that were instantiated, meaning the problem would still exist. Looks like the only way is to literally reload the page. :(

Hello @dogofpavlov We are developing new package react-google-maps-api currently. I will make sure to cleanup those scripts on unmount. It will have simplified API, and will support latest React. Please wait.


Resource specific metadata getting cutoff

It seems resource specific metadata is not getting displayed properly (getting cutoff on the left side) in both view and edit modes for resource types like Time Series, Multidimensional, Geographic Feature, Geographic Raster. Here is an example of the problem:

Originally posted by @pkdash in https://github.com/hydroshare/hydroshare/issues/3102#issuecomment-462347432

@pkdash, do we have a way of creating resource of these types in local development, or maybe a database I can test with?

I wonder if Neal D. has this in his scripts.
On Feb 11, 2019, at 10:17 AM, Maurier Ramirez <notifications<EMAIL_ADDRESS>wrote:
@pkdashhttps://github.com/pkdash, do we have a way of creating resource of these types in local development, or maybe a database I can test with?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/hydroshare/hydroshare/issues/3141#issuecomment-462364851, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArS3Ttn4mozLjJKNMTBMO7kXH7omASdyks5vMYl0gaJpZM4a0ZYr.

Whoops, didn't mean to reply all, was just mentioning for Mauriel.
On Feb 11, 2019, at 10:18 AM, cuahsimarko <notifications<EMAIL_ADDRESS>wrote:
I wonder if Neal D. has this in his scripts.
On Feb 11, 2019, at 10:17 AM, Maurier Ramirez <notifications@github.commailto:notifications<EMAIL_ADDRESS>wrote:
@pkdashhttps://github.com/pkdash, do we have a way of creating resource of these types in local development, or maybe a database I can test with?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/hydroshare/hydroshare/issues/3141#issuecomment-462364851, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArS3Ttn4mozLjJKNMTBMO7kXH7omASdyks5vMYl0gaJpZM4a0ZYr.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/hydroshare/hydroshare/issues/3141#issuecomment-462365323, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArS3TrK-oWG7eSfi0zogCZYeeV7Qnyo5ks5vMYnAgaJpZM4a0ZYr.

@Maurier

do we have a way of creating resource of these types in local development, or maybe a database I can test with?

Yes, You can edit the create-resource.html file to insert an li element like this: https://github.com/hydroshare/hydroshare/blob/develop/hs_core/templates/pages/create-resource.html#L81 for a specific resource type.

Tested on beta.


[1.0] Tap gesture does not trigger when calling this.click in IE10

Considering the following contrived element and usage:
var XTapable = Polymer({
  is: 'x-tapable`

  listeners: {
    tap: '_onTap'
  },

  _onTap: function() {
    console.log('Tapped');
  }
});

var tapable = new XTapable();

tapable.click();

"Tapped" is not logged to the console in IE10.
Here is a live JSBin example.

/cc @azakus


FetchWithContext not exposed within ClientInterface

FetchWithContext is not exposed within ClientInterface so it's difficult to mock the implementation of ListUsages

I think the issue here is that methods such as FetchWithContext return a concrete implementation. It would be better to return an interface e.g.
type UsageLister interface {
	FetchWithContext(ctx context.Context) error
	CountWithContext(ctx context.Context) (*int64, error)
	HasMore() bool
	Data() []Usage
	...
}

I think the issue here is that methods such as FetchWithContext return a concrete implementation. It would be better to return an interface e.g.
type UsageLister interface {
	FetchWithContext(ctx context.Context) error
	CountWithContext(ctx context.Context) (*int64, error)
	HasMore() bool
	Data() []Usage
	...
}

Hey @abMatillion and @speza, thanks for letting us know about this.
I think that returning an interface from these functions makes sense and will make it easier to test against. I'm not sure that we'll be able to make this kind of change in the 3.x version of the client, though, as it would be a breaking change.
We will look into getting this included into the next major version of the client, which we hope to be releasing this quarter. I'll leave the issue open to keep you updated.

Hey @abMatillion and @speza, thanks for letting us know about this.
I think that returning an interface from these functions makes sense and will make it easier to test against. I'm not sure that we'll be able to make this kind of change in the 3.x version of the client, though, as it would be a breaking change.
We will look into getting this included into the next major version of the client, which we hope to be releasing this quarter. I'll leave the issue open to keep you updated.

Thanks @douglasmiller, that's understandable. We have a workaround which isn't too bad. We've basically extended the recurly.ClientInterface to have a ListUsageExt(...) UsageLister method which wraps the standard ListUsage method.

Thanks @douglasmiller, that's understandable. We have a workaround which isn't too bad. We've basically extended the recurly.ClientInterface to have a ListUsageExt(...) UsageLister method which wraps the standard ListUsage method.

@speza / @abMatillion We just merged #80 which will include an interface for the List* methods. As previously mentioned, this will be for the 4.x version of the client.
Thanks again for all of the great feedback and critique on this client library!

Thank you @douglasmiller, happy to feedback! Just out of interest, when is v4 likely to be released?

We are targeting end of February


Add instructions for data collection and processing with Spectacular AI tools

Adds easy support for new device types: OAK-D, RealSense and Azure Kinect, as well as an alternative pipeline for iOS (+ Android coming soon)
Differences to other approaches: Uses VISLAM (with post-processing, unlike ARKit) instead of purely image-based methods to compute camera poses -> more robust especially with quick scans. No need to upload the data to a 3rdparty server for processing

ping @maturk

Rebased on latest main and fixed the conflict in README.md


Update the Arduino diagram in the UI with a Wi-Fi shield

We need to update the diagram in the UI and docs with the Wi-Fi shield.  Need your help to do this, please.

I am closing this JIRA as Arduino is not shipped with WSO2 IoT Server 3.1.0

Please check this.

Thank you, Rasika! This is perfect. I updated the docs. Will we update the Arduino part in the device management console too?

Sure thing Shavindri. I have updated the diagram on device management console.


feat: continuous download

The hive engine-api test suite, along with other hive suites, expect that we are able to sync without a configured tip. This is a working draft for supporting this feature so it can be used for tests.
The idea behind this was to introduce a --continuous flag, and have the headers stage increment the downloader's sync target if it's set. This required refactoring SyncTarget to accept a number, instead of just a hash or header. The reverse header downloader is modified to support using block numbers as well.
This is extremely WIP, as the refactor has gotten a bit invasive, and is currently broken. Ideally we can support this without as invasive changes, in a way that works for hive testing purposes only.

Codecov Report

Merging #1744 (5953571) into main (043098a) will decrease coverage by 28.38%.
The diff coverage is 8.82%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #1744       +/-   ##
===========================================
- Coverage   72.95%   44.57%   -28.38%
===========================================
  Files         391      392        +1
  Lines       46575    46777      +202
===========================================
- Hits        33978    20853    -13125
- Misses      12597    25924    +13327

Flag
Coverage Δ

integration-tests
20.26% <0.00%> (-0.10%)
:arrow_down:

unit-tests
35.59% <8.82%> (-32.14%)
:arrow_down:

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

Impacted Files
Coverage Δ

bin/reth/src/node/mod.rs
0.00% <0.00%> (-4.31%)
:arrow_down:

crates/interfaces/src/p2p/error.rs
78.43% <ø> (-7.85%)
:arrow_down:

crates/interfaces/src/p2p/headers/downloader.rs
39.28% <0.00%> (-34.79%)
:arrow_down:

crates/stages/src/pipeline/builder.rs
0.00% <0.00%> (-60.72%)
:arrow_down:

crates/stages/src/pipeline/mod.rs
6.87% <0.00%> (-79.56%)
:arrow_down:

crates/stages/src/stages/headers.rs
24.62% <6.66%> (-70.79%)
:arrow_down:

...tes/net/downloaders/src/headers/reverse_headers.rs
49.93% <13.28%> (-23.91%)
:arrow_down:

... and 182 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@onbjerg as discussed PTAL


How to realize Visualization of the Learned Similarity

Hi, Edward～Thank you for your excellent work！
In section 4.2 of your paper, this part of the work is really interesting.
I have two problems: (1) Is this subsection of the experiment implemented with grad-cam? (2) Can you share the relevant code? This trick for the highlight is very clever.
Happy New Year to you! Good work in the new year!

Hi there and thank you~ It is not implemented by grad-cam. We get the similarity map just as we do in the LFM. The code is uploaded in main_vis_sim.py.

Thank you for the great work!
Thank you for the help😄


ENH upgrade conda to 4.3.11

@stephen-hoover @jmemich
This version upgrades conda while pinning python to 3.5.2. We don't have to move to 3.6 if we don't want to. Happy to close, but thought it might be useful.

I think @stephen-hoover wanted to take the opportunity to move to python 3.6. He had been working on updating the dependencies here.

Sure yeah. I have no horse in this race. Pinning and reinstalling python explicitly might be good practice.

I think we can close #10 in favour of this PR. Only substantive pre-CR question that remains is whether to bump to python 3.6.

Please also include the motivation for the conda upgrade in the PR text and the commit message.

@stephen-hoover I will include notes on the upgrade when I squash to merge.

Ugh. We did not do this last time, but sure.


Demo at https://example.universallogin.io/ can't create account

What i did:

Input a name following the example given in the ui, like:
yo.workthistime.eth
(tried several others later)
Click on "create"

What I expected to happen
The identity to be created and the demo to proceed.
What happened
See next screen for a fraction of a second, then:

Failed to load resource: the server responded with a status of 404 () universal-login-relayer.herokuapp.com/identity:1 Failed to load resource: the server responded with a status of 500 (Internal Server Error) rinkeby.infura.io/:1 Failed to load resource: net::ERR_NETWORK_CHANGED ethers.min.js:1 Uncaught (in promise) Error: invalid response - 0     at XMLHttpRequest.a.onreadystatechange (ethers.min.js:1) universal-login-relayer.herokuapp.com/identity:1 Failed to load resource: the server responded with a status of 500 (Internal Server Error)
`
Request
curl 'https://universal-login-relayer.herokuapp.com/identity' -H 'Referer: https://example.universallogin.io/' -H 'Origin: https://example.universallogin.io' -H 'User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/71.0.3578.98 Safari/537.36' -H 'Content-Type: application/json; charset=utf-8' --data-binary '{"managementKey":"0x0948C0a8E2372a67260CAEd307c5C842BbFdf4FA","ensName":"yo.workthistime.eth"}' --compressed
Response:
500
{"error":"Error: domain not existing / not universal ID compatible"}
`

Closing, fixed by #201.


[Explorer] Generate SQL but error type is error-answer-parse a143cdc1-37e1-4bae-ab7a-d6b18ae2caa3: What are the most popular repos related to ChatGPT

Hi, I have some problems with the question a143cdc1-37e1-4bae-ab7a-d6b18ae2caa3 What are the most popular repos related to ChatGPT (errorType = error-answer-parse):

Error message
Unexpected RIGHT_BRACE("}") in state VALUE

Encountered several similar errors today, generated SQL but error type is error-answer-parse.

Fixed


refactor(grid): update scss folder structure and mixins

Closes #274
Closes #365
Update grid package from feedback, optimize layout and examples for readability and extensibility. Simplify logic for no-gutter, hang, and condensed layouts.
Changelog
New
Changed

@carbon/layout

Add variable for condensed grid gutter

@carbon/grid

Refactor scss folder
Update layout options to include no-gutter and hang
Update logic for condensed grid, add support for hybrid grids with condensed rows
Update example to be easier to work with, reference, and preview
Add additional SassDoc to package

Removed

looks great, quick comments:
can we update the "1" here in each span, not sure it really makes sense

Update no gutter example to be on every column:


Improve search experience

Final release.

Thank you, cool search method. Why did you change the method of getting token? Defining tokens in code isn't good experience.

I changed it to what @sudheesh001 suggested. Now you can safely merge. I also add some new function like doing /user:CBS

@sudheesh001 @sevazhidkov When I am trying to fix the compatibility with Python 3, I find out that one should not have a file named token.py in the dictionary because it is a system module. It only cause trouble in Python 3 on my laptop. However, we should not do that. Now the code is working under Python 3.

@sudheesh001 Do you think that defining token in code is a good idea?

@sevazhidkov You can safely merge right now. There is no token.py.


How to communicate the philosophy of type annotations in Python?

A question from the Typing summit:
How to communicate the philosophy of type annotations in Python?

Beginners  don't need annotations
"Catching bugs" is how to sell it to your engineering director :-)
Adding improvements like new Union to simplify the syntax

There's a nice community around typing that is responsive to ideas and thoughts.

I think sometimes we fall into the trap of thinking that proposals to change Python syntax in order to allow for more ergonomic type annotations are "changing Python just for the typing people" and are somehow harming non-typing users. I think this is actually backwards and it's important to reframe this tension. Typing is here to stay, and sooner or later people who start without type annotations (which is great!) will need to read annotated code. If we can make type annotations simpler, more ergonomic, better "fit" into the language, and easier to read, this doesn't just benefit active users of typing -- it perhaps has the most benefit for non-users of typing who still have to read annotated code in other peoples' libraries!
So I think the idea I've heard suggested that we need to "split" the language so typed Python can add syntax without affecting untyped Python is not the right direction. We all have to read each others' code, so the thing that benefits everyone, users of types and non-users of types, is to work hard to better integrate and streamline type annotations.

Having some evidence-based reasons for why to adopt annotation would be helpful in convincing projects to adopt it.  Many Python projects are very strong on the testing front partially because of the lack of static type checking, but it also means that the benefits of adding annotations are less evident.
In numpydoc types are specified inline; I think we'd benefit from having those be grabbed from annotations instead.
@carljm Fully agree with your sentiment above: "sooner or later..." — keeping it simple is beneficial for all Python users, preserving one of Python's best features: its ease of readability.

Newer developer sees a type annotation

Don't panic
How to read documentation
Why someone would use the TA

Like unit tests, it improves code quality

It's okay as a new developer to be aware of TA but feel no pressure to use until you are creating production code at scale

TODO: SciPy sprint for Carpentries lesson re: typing esp. tensors as a compelling use case; code quality; reproducibility; scaling. Reach out to Turing Way too.

Thanks Carol for starting a discussion here!
I'm interested in getting the ball rolling on some centralized documentation for the Python type system, which ideally could also include some opening or introductory pages that discuss things you just mentioned like (1) how to read documentation, (2) why someone might want to use type annotations, why this is particularly helpful for coding at scale, etc.
Right now it's a bit difficult to find any source of truth for learning about the type system outside of finding disjoint pieces scattered across PEPs, or picking a type checker and going to read their website. If we can get a documentation source that the larger Python community has contributed to and agrees on, I could see this providing a lot of value to new and existing users of types in the language.
I recall there was a suggestion during the summit to get this started as a subdirectory of the typing repository, where we can incrementally iterate, then potentially moving this to somewhere discoverable alongside other python documentation.
Thoughts?  I also plan to start some discussion on this in the typing sig mailing list.

I recall there was a suggestion during the summit to get this started as a subdirectory of the typing repository, where we can incrementally iterate, then potentially moving this to somewhere discoverable alongside other python documentation.

AFAICS, this was started in typing docs.

Note the specific section linked is specifically aimed at library authors considering whether to ship type hints to their users via PEP 561.
We should add more docs to typing.readthedocs.io to address more user profiles.


[Bug]: EntrySensor gets no updates on cloud data refresh and no push messages

Client version
2.1.1
Node version
16.16.0
Operating System type
Linux
Operating system version
deb
Describe the bug
Hi!
I have had a debugging session with @federicodigiacomo who has issues with his eufy sensors (T8900s). The state of theses sensors is updated correctly one time only on startup of the client. In subsequent refreshs of the cloud data the correct value for CMD_ENTRY_SENSOR_STATUS is retrieved from the cloud, but the raw property is not updated. In consequence the DeviceSensorOpen property is also not updated. In short: the sensor always shows the state that was loaded at startup. Any sensor changes are ignored.
I can provide a log. Since these are not mine and have sensitive data in them, I'd like to share them via google drive or something. I can also share snippets of the log, if that's helpful.
But to summarise the log: after every polling Interval the device data is fetched. It shows the correct state in the device data but when we try to get the sensor state in the homebridge plugin it is not updated. Also there is no raw property change or property change emitted. So I can confirm that the internal value is not correctly updated. The p2pconnection to the station is not disconnected, so this doesn't seem to be it.
Also something peculiar: there's also no push notification regarding the sensor state change. Not even a raw push message in the logs. Do these just not work for these sensors?
Push in general does seem to work. At least I have seen two other push messages in the log regarding guard mode changes.
To reproduce
Observe internal state of sensor over a period of time.
Screenshots & Logfiles
No response
Additional context
No response

I'm on the latest develop version and I get updates just fine. I have push notifications enabled and get notifications/updates just fine.

Released version 2.1.2 based on the latest develop version which works according to @Palmke.

Will update the plugin to 2.1.2. in the coming days and let you know if this fixes it.
@Palmke so you own at least one T8900 also? I don't know if there are different kinds of sensors from eufy.

I have three. And checked for logs for one when I opened a door after reading this issue.

@federicodigiacomo did let me know, that his issue is resolved. Thx!


Update how-to-virtual-machine-mtu.md

minor edits to ping verification.

@asudbring
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@mattreatmsft - Can you respond to the comment above?

I sent an email to @mattreatMSFT today.

After further review I realized this change is not needed. Thanks for the comments. Allen and @Court72


error while evaluation?

task.py -- validation_epoch_end:
"Trainer" object has no attribute 'use_ddp'

I also meet this problem. How do you solve this?

What is the version of pytorch-lightning you using?

@RangiLyu in 1.4.0, some attributes like use_ddp got removed.

Removed deprecated trainer attributes - on_cpu, on_tpu, use_tpu, on_gpu, use_dp, use_ddp, use_ddp2, use_horovod, use_single_gpu

Source.


connectors: Add envFrom configuration option to connectors chart

Closes #1165

Configures a environment variable configuration provider based on Kafka EnvVarConfigProvider
Adds option to pass envFrom to deployment.yaml template
Adds deployment.extraEnvFrom to values.yaml

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

LGTM, thanks!

@tomasz-sadura @RafalKorepta Thanks for the feedback! Updated the version as requested.


docker build should support option for specifying a custom seccomp profile

Output of docker version:
Client:
 Version:      1.10.2
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   c3959b1
 Built:        Mon Feb 22 22:37:33 2016
 OS/Arch:      darwin/amd64

Server:
 Version:      1.10.2
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   c3959b1
 Built:        Mon Feb 22 22:37:33 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 1
 Running: 0
 Paused: 0
 Stopped: 1
Images: 3
Server Version: 1.10.2
Storage Driver: aufs
 Root Dir: /mnt/sda1/var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 10
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.1.18-boot2docker
Operating System: Boot2Docker 1.10.2 (TCL 6.4.1); master : 611be10 - Mon Feb 22 22:47:06 UTC 2016
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.956 GiB
Name: default
ID: IJCF:IXCO:C32D:S3BP:3WRN:VFRE:DKKL:ZOPL:ZJAZ:ORJG:GJSV:K2WC
Debug mode (server): true
 File Descriptors: 10
 Goroutines: 22
 System Time: 2016-03-01T15:34:54.098983226Z
 EventsListeners: 0
 Init SHA1:
 Init Path: /usr/local/bin/docker
 Docker Root Dir: /mnt/sda1/var/lib/docker
Username: msimons
Registry: https://index.docker.io/v1/
Labels:
 provider=virtualbox

Currently there doesn't seem to be a way to change the seccomp profile used during docker builds.  It would be nice if you could specify  --security-opt seccomp:<profile path> when building.
Scenario:
FROM microsoft\dotnet

WORKDIR /myapp
# TODO: obtain your dotnet app - pull from SCC or use COPY

# restore the dependencies
RUN dotnet restore

# build and run the app on entry
ENTRYPOINT ["dotnet", "run"]

Building the above Dockerfile will fail because the dotnet CLI tooling relies on the LTTng library which makes a blacklisted syscall.  (see #20818)
This feature request was mentioned in #20634 as a possible solution.

The reason I'd be weary of this is we use seccomp to help provide protection for the host. Part of this protection is in place to ensure that builds can't change host properties, thus making a non-portable image.

I don't think this is a good solution, it is not going to be widely supported in autobuild environments, and will make builds much less portable.

All the issues with blacklisted syscalls for build that have been raised have been resolved, so I am not sure there is any need for this.

yes, I'll close this one; the direct issue at hand is resolved, and it's unlikely we'll make changes to the profile used during build for the reasons mentioned above


Always use 'absolute' imports with coc.nvim

[x] I've gone through the existing config options

The current implementation of coc.nvim when it comes to autoImport is pretty smart in terms of absolute vs relative imports.
I encountered a requirement where for a project I would like to have just 'absolute' imports, this was done to avoid any confusion.
Wondering if there is a way to achieve this?

The server could provide such kind of option which can't be controlled by coc.nvim.
Should be problem of your language server, checkout https://github.com/neoclide/coc.nvim/wiki/Debug-language-server#using-output-channel

Looks like you need send feature request to typescript team, the option have to be provided by tsserver.

@chemzqm Thank you for point me there. Found this helpful option which we already support https://github.com/microsoft/TypeScript/issues/21860#issuecomment-389307342. My bad on this.
Here's how you do it in vim:
# Opens coc-tsserver config
:CocConfig

# Add this to the config object
"typescript.preferences.importModuleSpecifier": "non-relative"