C# Files have no syntax highlighting, intellisense, etc.

Issue Description
When I open .cs files, they are treated as Plain Text. The 'Autodetect' language mode leaves them as plain text, and trying to manually set the language mode is impossible because there's no csharp language mode to choose:

Steps to Reproduce

Open VSCode
Install this extension
Open a .cs file

Expected Behavior
The file is automatically detected to be csharp, and syntax highlighting, intellisense, etc. is enabled
Actual Behavior
The file is treated as plain text, and no syntax highlighting is possible.
Logs
OmniSharp log
Starting OmniSharp server at 12/8/2021, 10:22:41 AM
    Target: /Volumes/SSD/home/dev/Developer/PublicAPI/GordonPublicAPI
OmniSharp server started.
Path: /Volumes/SSD/Applications/code-portable-data/extensions/ms-dotnettools.csharp-1.23.17/.omnisharp/1.37.17/run
PID: 82307
    Starting OmniSharp on MacOS 12.0.1 (x64)

    Checking the 'DOTNET_ROOT' environment variable to find a .NET SDK

    DotNetPath set to /Volumes/SSD/home/dev/.local/share/dotnet/dotnet

    Located 1 MSBuild instance(s)
        1: StandAlone 17.0.0 - "/Volumes/SSD/Applications/code-portable-data/extensions/ms-dotnettools.csharp-1.23.17/.omnisharp/1.37.17/omnisharp/.msbuild/Current/Bin"

    MSBUILD_EXE_PATH environment variable set to '/Volumes/SSD/Applications/code-portable-data/extensions/ms-dotnettools.csharp-1.23.17/.omnisharp/1.37.17/omnisharp/.msbuild/Current/Bin/MSBuild.exe'

    Registered MSBuild instance: StandAlone 17.0.0 - "/Volumes/SSD/Applications/code-portable-data/extensions/ms-dotnettools.csharp-1.23.17/.omnisharp/1.37.17/omnisharp/.msbuild/Current/Bin"
        CscToolExe = csc.exe
        MSBuildToolsPath = /Volumes/SSD/Applications/code-portable-data/extensions/ms-dotnettools.csharp-1.23.17/.omnisharp/1.37.17/omnisharp/.msbuild/Current/Bin
        CscToolPath = /Volumes/SSD/Applications/code-portable-data/extensions/ms-dotnettools.csharp-1.23.17/.omnisharp/1.37.17/omnisharp/.msbuild/Current/Bin/Roslyn
        BypassFrameworkInstallChecks = true
        MSBuildExtensionsPath = /Volumes/SSD/Applications/code-portable-data/extensions/ms-dotnettools.csharp-1.23.17/.omnisharp/1.37.17/omnisharp/.msbuild

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpFormattingWorkspaceOptionsProvider, Order: 0

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.Completion.CompletionOptionsProvider, Order: 0

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.RenameWorkspaceOptionsProvider, Order: 100

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.ImplementTypeWorkspaceOptionsProvider, Order: 110

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.BlockStructureWorkspaceOptionsProvider, Order: 140

    Detecting Cake files in '/Volumes/SSD/home/dev/Developer/PublicAPI/GordonPublicAPI'.

    Could not find any Cake files

    No solution files found in '/Volumes/SSD/home/dev/Developer/PublicAPI/GordonPublicAPI'

    Queue project update for '/Volumes/SSD/home/dev/Developer/PublicAPI/GordonPublicAPI/GordonPublicAPI.csproj'

    Detecting CSX files in '/Volumes/SSD/home/dev/Developer/PublicAPI/GordonPublicAPI'.

    Could not find any CSX files

    Configuration finished.

    Omnisharp server running using Stdio at location '/Volumes/SSD/home/dev/Developer/PublicAPI/GordonPublicAPI' on host 81995.

    Loading project: /Volumes/SSD/home/dev/Developer/PublicAPI/GordonPublicAPI/GordonPublicAPI.csproj

    Successfully loaded project file '/Volumes/SSD/home/dev/Developer/PublicAPI/GordonPublicAPI/GordonPublicAPI.csproj'.

    Adding project '/Volumes/SSD/home/dev/Developer/PublicAPI/GordonPublicAPI/GordonPublicAPI.csproj'

    Update project: GordonPublicAPI

C# log
The C# Log is completely blank

Environment information
VSCode version: 1.62.3
C# Extension: 1.23.17
Mono Information
        OmniSharp using built-in mono
Dotnet Information
.NET SDK (reflecting any global.json):
 Version:   6.0.100
 Commit:    9e8b04bbff
Runtime Environment:
OS Name:     Mac OS X
OS Version:  12.0
OS Platform: Darwin
RID:         osx-x64
Base Path:   /Volumes/SSD/home/dev/.local/share/dotnet/sdk/6.0.100/
Host (useful for support):
Version: 6.0.0
Commit:  4822e3c3aa
.NET SDKs installed:
6.0.100 [/Volumes/SSD/home/dev/.local/share/dotnet/sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.0 [/Volumes/SSD/home/dev/.local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.0 [/Volumes/SSD/home/dev/.local/share/dotnet/shared/Microsoft.NETCore.App]
To install additional .NET runtimes or SDKs:
https://aka.ms/dotnet-download

Visual Studio Code Extensions

Extension
Author
Version

csharp
ms-dotnettools
1.23.17

I would just like to note that, after hearing about the new C# Dev Kit extension and the updates to this extension, I came back to try it again >1.5 years later and am still seeing the same issue, on a completely separate installation of VS Code.

I have discovered that, when I create a new VS Code profile from an empty profile, and then install this extension, everything appears to work as expected. That suggests that either my settings are incorrect (even then, as far as I can tell, none of my defaults have changed in any way that would affect C#), or somehow my default profile is corrupt. Regardless, the issue itself is now resolved, since I can successfully use syntax highlighting for C# in VS Code, so I am closing this issue.


Update README.md

-change the title.
-change the travis build URL to use https://travis-ci.org/exercism/javascript.

Thanks! and congrats on your first contribution! 🎉


Start WebSocket connection early for fast startup.

This makes the time to a live server connection and WASM load almost simultaneous, so startup feels almost twice as fast.

bors r+


Multipart request headers do not work properly for values of empty string

Continuing the discussion from https://github.com/sigmavirus24/requests-toolbelt/issues/162, attempting to create a RequestField which is then made multipart via make_multipart does not work properly if the filename given is an empty string.
urllib3 test code:
from urllib3.fields import RequestField
field = RequestField(name="somename", data="somedata", filename="")
field.make_multipart(content_type="application/octet-stream")
print(field.headers)

Expected output:
{'Content-Type': 'application/octet-stream', 'Content-Location': None, 'Content-Disposition': 'form-data; name="somename"; filename=""'}

Actual output:
{'Content-Type': 'application/octet-stream', 'Content-Location': None, 'Content-Disposition': 'form-data; name="somename"'}

A test made in Chrome 55 on macOS shows that the filename field should be present in the content disposition even if the value is empty (such as when an optional input tag on a webpage accepts multiple files and none are provided):
------WebKitFormBoundaryCvXp0tseQuZuGTru
Content-Disposition: form-data; name="somename"; filename=""
Content-Type: application/octet-stream

------WebKitFormBoundaryCvXp0tseQuZuGTru--

Flask server code I wrote to test the request payload from Chrome:
from flask import Flask, request
app = Flask(__name__)

@app.route('/', methods=['GET'])
def multipart_form():
    return '<form name="testform" action="" enctype="multipart/form-data" method="post"><input id="file" type="file" multiple="multiple" name="somename"><input id="post" type="submit" value="Go"></form>'

@app.route('/', methods=['POST'])
def upload_handler():
    print(request.files['somename'])
    return 'ok'

I'm happy to accept a PR to change this behaviour. =)


EmrEtlRunner: add CI/CD to update AMI bootstrap scripts

From https://github.com/snowplow/snowplow/tree/master/3-enrich/emr-etl-runner/bin to s3://snowplow-hosted-assets
Hey @BenFradet, am I right that we can use existing credentials for that?

I think we should make it very explicit in CI/CD process that we want to upload/update those scripts, it looks like a bit scary for me to re-upload them on each EmrEtlRunner release.

Hey @BenFradet, am I right that we can use existing credentials for that?

yup


Support for SSH Public Key Provisioning for vr-sros

Added support to automatically provision SSH public keys into vr-sros nodes.
Used the embed function to store my config command template in a dedicated external file. I liked this approach better than to specify the commands directly in the vr-sros.go file.
I created a dedicated function for the health check/readiness probe.
I would like to improve sessions handling a bit though. I would prefer to re-use the same session for the readiness probe, the applyPartialConfig and the applyDefaultConfig functions. Although it works as is, I would be also happy about a pointer on how to improve this.

Thanks for the pointer. Did that

superseded by #1706


fix empty biz body unwrapBinary EOF error

What type of PR is this?

Check the PR title.

[ ] This PR title match the format: <type>(optional scope): <description>
[ ] The description of this PR title is user-oriented and clear enough for others to understand.
[ ] Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.
修复空业务body在unwrapBinaryMessage遇到的EOF错误
(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):
(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

 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.xielei.xielei seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Columns not in columns prop at first render are pushed to end of table

material-react-table version
1.2.6
Describe the bug and the steps to reproduce it
If the array passed in "colums" props have more columns than previously, added columns are pushed at the end of the table
Your Minimal, Reproducible Example - Sandbox (Recommended) or code snippets
https://codesandbox.io/s/thirsty-bohr-2y5wom?file=/src/App.js
Screenshots or Videos (Optional)
No response
Do you intend to try to help solve this bug with your own PR?
No, because I don't know how
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

That will happen, because the column order is managed internally based on the initial column order.
To fix, you will need to manage the columnOrder state yourself
See this guide in the docs
Though you will probably need to pass it as fully managed state, rather than initialState

Code sandbox showing how you can fix it:
https://codesandbox.io/s/hopeful-danilo-m3gy29
export default function App() {
  const [condition, setCondition] = useState(false);

  const cols = useMemo(
    () =>
      [
        {
          header: "Name",
          accessorKey: "name"
        },
        condition && {
          header: "Address",
          accessorKey: "address"
        },
        {
          header: "Lastname",
          accessorKey: "lastname"
        }
      ].filter(Boolean),
    [condition]
  );

  const columnOrder = useMemo(() => cols.map((col) => col.accessorKey), [cols]);

  return (
    <>
      <button onClick={() => setCondition(!condition)}>Toggle condition</button>
      <MRTable columns={cols} data={data} state={{ columnOrder }} />
    </>
  );
}


Select appropriate VPN connection for routes resource

This PR refactors the logical conditions involved in selecting which VPN connection to create (default, tunnel or preshared). It then applies those logical conditions to the resource that creates vpn connection routes in order to select the right VPN connection id.
Since the 3 different logical conditions are mutually exclusive, this PR adds conditionals to determine if tunnel should be used, else if preshared should be used, or (by exclusion) default should be used.
Fixes #6

@dbadrak could you please test your use case against this PR?

@dbadrak could you please test your use case against this PR?

Sure thing, as soon as I figure out how to grab the pull request (I'm a git noob).

@dbadrak something like git fetch origin pull/15/head:pr/15 will create a branch called pr/15 with the code of this PR. Then git checkout pr/15 will navigate your repo to that branch.

@dbadrak something like git fetch origin pull/15/head:pr/15 will create a branch called pr/15 with the code of this PR. Then git checkout pr/15 will navigate your repo to that branch.

@miguelaferreira  Ok, I did that, put it into a test module, included it (remove the old, did a terraform init).  terraform plan showed it was removing the old routes and adding the new. Then, the apply crashed and generated a crash.log. It was all about the destroy. When I re-ran the plan, the old ones were not there, and it showed the new ones.  Apply actually applied them.  So, it appears to have done the right thing.  Thanks.

Great, thanks @dbadrak
I will rone some tests myself and after that remove the WIP from the PR

I've tested all the examples, and after a few fixes, all ran as expected. So I think this PR is good to go @antonbabenko
I'll remove the WIP from the title

I tested this PR in a production environment and it works as expected.

Thank you @Farzy

when you merge this?

THANKS, @miguelaferreira @Farzy @dbadrak !
v1.6.0 has been released.


Don't allow remove on non-stopped machines

Closes #1179
Please review and after first code review I will add some tests cc @ehazlett @hairyhenderson
There is also some general cleanup of messaging on actions, etc. to make it more consistent and informative.
Signed-off-by: Nathan LeClaire<EMAIL_ADDRESS>

What do we do about providers that do not support stop?  I.e. RAX?

Also, not sure what is happening, but a stop action on DO just hangs.

What do we do about providers that do not support stop? I.e. RAX?

Argh, I didn't know that was a thing unfortunately.  A SupportsStop method on the driver seems kludgey, maybe we will have to selectively allow them based on the driver type i.e. a switch in the host.Remove method.

What do we do about providers that do not support stop? I.e. RAX?

Argh, I didn't know that was a thing unfortunately.

And it's likely to be more of a thing in the future, too - there was a PR for a Triton/SDC driver recently, and it doesn't support stop and a few other operations too...

Re-visiting this, I think we need to at least prompt before destroying instances.  Our current method is extremely dangerous.

@nathanleclaire ok if we close it for bookkeeping for now?

Yeah let's close, I at least want to have a prompt (You are going to destroy these machines.  Proceed? [y|n]) in the future though.


Feature request: do not autosave

Right now, when choose "edit value under cursor" and changing the value, the modification is immediately written to disk: it's quite different from the behavior of VSCode which won't automatically (unless you have files.autoSave set to 'on') save your file but put a "modified" marker next to the filename instead.
How about only autoSaving when the files.autoSave option is set?

Looks like there is no autoSave actually: since there was no "modified" marker appended on the original file nor the ".hexdump" tab, and since that closing the ".hexdump" tab and opening it again shows the modified state, I thought it had been saved.

You are right, identifying the document as being modified would help the user to better understand the situation. Hopefully, renaming the tab and colorize modifed bytes (#28) should help


[SECURESIGN-298] Add uninstall & template for default values

closed https://github.com/securesign/sigstore-ocp/pull/104 in favor of this PR - see #104 for previous review
This PR:

adds @JasonPowr 's remove keycloak commit from https://github.com/securesign/sigstore-ocp/pull/102
introduces template for default values that is embedded in binary with go compile
adds helm lister to check for existence of release to either install or upgrade
adds uninstall cmd that
a) performs helm uninstall
b) removes extra namespaces that were created but not managed by helm
in absence of a custom values file, the OIDC issuer endpoint https://oauth2.sigstore.dev/auth will be configured. This is set in the default template/values file.
adds constants for secret file names, certs dir

I do like the use of go templates to set the host values its much cleaner than my method, I'm assuming something similar could be done for the OIDC provider?

I unable to install Helm chart (0.1.25) of TAS via installer because of existing sigstore-monitoring namespace. This namespace does not exist before execution of tas-install.
Namespace "sigstore-monitoring" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; label validation error: missing key "app.kubernetes.io/managed-by": must be set to "Helm"; annotation validation error: missing key "meta.helm.sh/release-name": must be set to "trusted-artifact-signer"; annotation validation error: missing key "meta.helm.sh/release-namespace": must be set to "trusted-artifact-signer"

I unable to install Helm chart (0.1.25) of TAS via installer because of existing sigstore-monitoring namespace. This namespace does not exist before execution of tas-install.

Did you run the easy install script first? It would seem to me that this line ran, creating it without the annotations for it being managed by helm. If that is the case you can just delete the project and try again from the insatller. This issue is related to how close some of this work is to the other active PRs in this repo, there are other PRs to handle abstraction of pull-secrets and cert generation.

this should be made against the release-1.0.gamma branch, closing


parameter "r" is ratio or absolute number

Hi,
If I set the Dimond parameter top =10, and r=10, how many hits will be included to do the LCA of ORF? 10 or 1?

Hi @zhaoxvwahaha,
Both parameters are a percentage. A --top of 10 means that only hits within 10% range of the best alignment score are included in the DIAMOND output file. So if the bit-score of the best hit is e.g. 100, all hits with a bitscore >= 90 are included. The -r parameter does the same, but after alignment while parsing the alignment file. It is meant to change the fraction of included hits without having to remake the alignment file every time again. For this reason, -r should always be set lower than --top.
Let me know if you have further questions!
Bastiaan


Add minted/burned dToken to number dTokens

With the DFIP the futures swap was introduced and with it dTokens can be minted (without a loan) and burned. These numbers should be added to the evaluation "Number dTokens"
Data can be acquired from the command "listgovs"

Also the locked dToken should be included in this graph, because they also reduce the circulating amount


Unable to dotnet build when %20 in folder name

Spent about 2 hours trying to figure out why I couldn't build a repo after I cloned it, it turned out dotnet core doesn't like it when there's "%20" in a folder name.
With "%20" in folder name:

Without "%20" in folder name:

@wli3 @dsplaisted it looks like we're url-decoding paths? Can you help route this?

@rainersigwald Is this an MSBuild issue (can't build with %20 in the path to a project?

Yes: microsoft/msbuild#2178. Unlikely to be fixable, unfortunately.

Closing as duplicate of https://github.com/dotnet/msbuild/issues/2178


provide me a pretrained model which is compatible for Yolov8 model with the object365

Please provide me a model that has to be compatible with the yolov8 architecture with the object365 classes.
I tried the above models but it has only the coco classes and not the object365

Have you double checked your statement and question, haven't you?
Pretrained weights on object365 are already avail, compatibility with another model doesn't make sense you know that right?

Have you double checked your statement and question, haven't you?
Pretrained weights on object365 are already avail, compatibility with another model doesn't make sense you know that right?

The pretrained model which was provided is a ".pth" file but usually for model prediction we use the ".pt" file. The pretrained model file's architecture is not mentioned, like which version of YOLO is used for the model.

Please provide me with some reference code on how to use the pretrained checkpoint model

@AnishMathewOommen you should first understand that this is like asking, "Give me a Mercedes for driving that is compatible with Ford !".
You should first understand that D-FINE is a completely new model (with new architecture), different from YOLO.

Please provide me with some reference code on how to use the pretrained checkpoint model

Have you gone through this part?


Do not run containers in interactive mode

Docker run options '-i' and '-t' are not needed for
fully automated operations (when user doesn't
interact with an application inside a container).
At the same time those options require valid TTY as
input device, which makes tests execution impossible
in closed virtual environment (e.g. Jenkins job).

@tomdee What is the next step for this PR?

@tmjd One of the maintainers should be reviewing and merging it @projectcalico/libcalico-go-maintainers

LGTM.


Ensure all HTTP errors returned as JSON error messages

As a Developer
I need to guarantee that HTTP Errors messages are returned as JSON
So that we can take advantage of Swagger Documentation
Details and Assumptions
Flask- RESTX layout are being followed
Acceptance Criteria
correct JSON format are returned with the acceptable key/values
Given Flask- RESTX layout is available
When an error gets triggered
Then an error in JSON format should be returned

This issue is already covered in previous issues.


Migrate qiskit_nature.properties

What should we add?
As part of #701, all existing modules of Qiskit Nature will be migrated.
This issue outlines the requirements for the existing properties module.
This issue has high priority because it blocks further migration of other modules.
This issue is (partially) blocked by #702.

⚠ This migration needs to copy the code in order to abide the deprecation process!

Below are the current module contents:
qiskit_nature/
    properties/
        Property
        GroupedProperty
        second_quantization/
            DriverMetadata
            SecondQuantizedProperty
            electronic/
                AngularMomentum
                ElectronicDipoleMoment
                ElectronicEnergy
                ElectronicStructureDriverResult
                Magnetization
                ParticleNumber
                bases/
                    ElectronicBasis
                    ElectronicBasisTransform
                integrals/
                    ElectronicIntegrals
                    IntegralProperty
                    OneBodyElectronicIntegrals
                    TwoBodyElectronicIntegrals
            vibrational/
                OccupiedModals
                VibrationalEnergy
                VibrationalStructureDriverResult
                bases/
                    HarmonicBasis
                    VibrationalBasis
                integrals/
                    VibrationalIntegrals

Here are the new locations for all of these classes.
qiskit_nature/
    second_quantization/
        operator_factories/
            electronic/
                AngularMomentum
                ElectronicDipoleMoment
                ElectronicEnergy
                Magnetization
                ParticleNumber
                bases/
                    ElectronicBasis
                    ElectronicBasisTransform
            vibrational/
                OccupiedModals
                VibrationalEnergy
                bases/
                    HarmonicBasis
                    VibrationalBasis

The following classes will not be migrated because of they are superseded as indicated:

the base class Property will no longer be needed. There may instead be a new OperatorFactory
base class
the GroupedProperty will also not be needed as the new Problem classes will take on the
responsibility of storing multiple OperatorFactory instances
DriverMetadata will no longer be needed as the problem will store its "provenance" information
(see upcoming issue on the planned HDF5 schema)
SecondQuantizedProperty will not be needed
ElectronicStructureDriverResult will be superseded by the new ElectronicStructureProblem
VibrationalStructureDriverResult will be superseded by the new VibrationalStructureProblem
ElectronicIntegrals (and its subclasses) will be superseded by PolynomialTensor (see #665)
VibrationalIntegrals will be superseded by PolynomialTensor (see #665)

I’m not sure I completely understand but if all that is needed to resolve is to copy the modules around I can do it when #702 is solved and #714 is merged.

Thank you for the offer!
The properties are a bit more involved as there are various API refactorings going on around them at the same time.
Although an initial copy might indeed be straight forward, I will likely handle this particular module myself for time reasons.
I can however suggest that I will send a smaller issue your way once this has progressed a bit further :+1:

No problem! Just let me know when and how I can help!


[Bug] 判断消息是否拒绝逻辑取反？ org.apache.rocketmq.store.timer.TimerMessageStore#isReject

Before Creating the Bug Report

[X] I found a bug, not just asking a question, which should be created in GitHub Discussions.

[X] I have searched the GitHub Issues and GitHub Discussions  of this repository and believe that this is not a duplicate.

[X] I have confirmed that this bug belongs to the current repository, not other repositories of RocketMQ.

Runtime platform environment
无
RocketMQ version
5.0 develop分支
JDK Version
No response
Describe the Bug
org.apache.rocketmq.store.timer.TimerMessageStore#isReject
if (RANDOM.nextInt(1000) > 1000 * (congestNum - storeConfig.getTimerCongestNumEachSlot()) / (storeConfig.getTimerCongestNumEachSlot() + 0.1)) {
return true;
}
这里的逻辑是否取反，return true 表示拒绝，
假设左边随机数固定，右边congestNum 越大 总体值不是越大，这样就拦截不了。
Steps to Reproduce
无
What Did You Expect to See?
无
What Did You See Instead?
无
Additional Context
No response

代码没错，是看错了。

更新

CLOSE


Add some operator improvements

Remove namespace from title. Current grafana-operator versions are already grouping dashboard by namespace, so there is no need to add it to the title because it is done automatically by the operator
Update default namespace on examples Makefile to use the new one with operator-sdk v1 (prometheus-exporter-operator-system)
Re-structure ansible role to make easier the addition on new exporters:

Every exporter customization is located on its own exporter directory, so all ansible templates are generic (deploymet, service, servicemonitor, grafanadashboard)
Grafana dashboard management will be easier, because the direct json jinja template is located on its own exporter dir (so without indentiation)
Every exporter container params (args, command, env, volumeMounts..), if they are present, are located on its own exporter dir
Every exporter volumes, if they are present, are located on its own exporter dir

Fix error on mysql dashboard with current qps query when having more than one timeseries db because there are more than 1 pod withing defined interval.
Add new manual kustomize to make make deploy/undeploy targets work. On default kustomize (intended for olm) envvar WATCH_NAMESPACE is taken from olm annotation, and when used with make deploy (so no olm), operator fails because envvar WATCH_NAMESPACE is empty, it thinks it is working on cluster scope mode (which is disabled intenationally, operator is  using role not cluster-role). So with new manual kustomize, we are overriding  WATCH_NAMESPACE to use the default namespace WATCH_NAMESPACE=prometheus-exporter-operator-system
Refactor docs, and add documentation about how to install the operator (with the upgrade to operator-sdk v1) and how to do an operator release
Create new alpha release 0.3.0-alpha.12

/lgtm


Git history is getting huge due to Jupyter Notebooks

Just cloned the repo and noticed that it was taking quite some time, downloading 550 MB for a fairly low-LOC repo:

I wonder if there's any merit towards switching to a fork/squash-merge approach instead of the branch based? Realistically, that history is only ever going to become larger.
Counterpoint:

This repo is not really something many people outside devs clone so it might be overengineering.

wdyt @giovp @LucaMarconato ?

I wonder if what is proposed here is an acceptable solution: https://stackoverflow.com/questions/1209999/how-to-use-git-to-get-just-the-latest-revision-of-a-project.

I don't think that having these extra 500 MB on disk is the issue (mildly inconvenient at best), but realistically this history will very quickly grow even larger in size if we keep working on it without squashing these Jupyter Notebook commits - same for spatialdata-notebooks.
Maybe we could implement a "squash-and-merge-only" rule to counter-act that a little bit?

That could be a good approach @giovp wdyt?

It would be interesting to decrease the size of this repo by squashing comments, not sure how to do it though. The solution you pointed to @LucaMarconato is interesting for dev perspective indeed. And also @timtreis you are right that the whole purpose of having the notebooks repo separate is that we knew this would happen in the future.


add edit in kebab dropdown of address space detail header and purge option in address detail header

Type of change

Enhancement / new feature

Description

add download option in kebab of address space list
add edit option in kebab of address space detail header
add purge option in kebab of address detail header

@enmasse-ci run tests testcase=standard.web.*

@enmasse-ci run tests testcase=standard.web.**

@drcoolsanjeev so Ive just tried your changes and I see that when I open edit menu win address menu it has set wrong addressplan

:heavy_check_mark: Test Summary :heavy_check_mark:
TEST_PROFILE: pr
TEST_CASE: standard.web.**
TOTAL: 28
PASS: 28
FAIL: 0
SKIP: 0
BUILD_NUMBER: 417

@kornys , I fixed the issue. Please review the changes.


make acceptsGzip public

It would make my use of this library much easier if my code can see whether the library will consider a request for possible compression.
Would making acceptsGzip -> AcceptsGzip as a pull request be welcome?

I think this has popped up a few times so I'm into adding the option to alter the default.
Instead of making that func public, perhaps we can add a func AcceptsGzip(func(*http.Request) bool) option to let folks optionally extend it?
Would that fit your use case?

That is a slick api addition and I could see it being useful to other, but I don't have an issue with the way it is determining compression. I just want to access the logic independently of the way the public api uses it. Feel free to change the title of this issue/repurpose/close for AcceptsGzip.

This is a dupe of #34 and #48, and I think the same recommendation applies. I don't see any problem with to exporting the func, but there's already a better implementation in gddo.

@meirf I have an implementation based on gddo available at tmthrgd/httputils if that's useful to you.


Option for seconds

I love this. I would like to make seconds/minutes optional :)

also weeks/months/years


V1 - Export Files from Dev-App

This PR is part of implementation of task #47 .

Files can now be exported from desired Dev-App working repository.
HTTP GET request sent to 'api/file/download' with query string as the path of file to be downloaded will return a 'Save As' dialog in the browser.
Only a single file at a time can be downloaded. No support for direct folder download.
File inclusion vulnerability is put in check so that files outside Dev-App's repos folder (such as /etc/passwd ) cannot be accessed.
SetTimeout() is used for making this work in Firefox.

LGTM


cannot use Custom Network in UnRAID

Its working if choose Bridge network,
Got following error if switching Custom: br0 Network and assgin a static ip
Install: Downloading extractor...
Installation failed (code 6)

Thanks @kroese i am using macvlan network, followed by your readme file, and apply macvlan to docker
docker network create -d macvlan
--subnet=<IP_ADDRESS>/24
--gateway=<IP_ADDRESS>
--ip-range=<IP_ADDRESS>/28
-o parent=eth0 vlan
Still get same error from container logs:
Install: Downloading extractor...
Installation failed (code 6)

Thanks for the update. I am off the computer for a while. Will check it later when I am in the town.
-------- 原始信息 --------发件人： Kroese @.> 日期: 2023/4/13  08:47  (GMT-04:00) 收件人： kroese/virtual-dsm @.> 抄送： Yu Sun @.>, Mention @.> 主题： Re: [kroese/virtual-dsm] cannot use Custom Network in UnRAID  (Issue #26)
@tysunyu Any updates?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Hi @kroese now i got code:0 when switch host machine dns to <IP_ADDRESS>
...
Starting Virtual DSM for Docker v2.97...
Install: Downloading extractor...
Failed to download extractor, code: 0
...

Custom: br0 is with macvlan driver. i would like to get a dedicated internal ip address for vDSM, rather using unRAID ip


Fixed required extension in README and enhanced it structure

There was a problem with wrong required extension installation name and link in README file. This pull request fixes the #1 issue and enhances the structure of README - Creating the issue from the link in #Supported Domains now lead to new issue creation page and will automatically attach the add to supported pages label.
Should be merged to master ASAP!

No issues detected, merging to master. #1 resolved


[doc] add runtime doc

This doc helps users understand the wenet models so I will merge this asap. @mikelei please help refine this in a new PR.


Customize Assembly Info

(How) is it possible to set the company inside the assembly info? Is there a possibillity to add "company": "My Company Name" to manifest.json file?
Thanks in advance!

Yes, but how do you want to use it? It's any variable?

Available in Electron.NET 8.31.1 - Please use the new electron.manifest.json file.
{
  "executable": "{{executable}}",
  "splashscreen": {
    "imageFile": ""
  },
  "name": "{{executable}}",
  "author": "",                      <--- Here you can use the company name
  "singleInstance": false,
  "build": {                          <--- alternative use the electron-builder config https://www.electron.build/configuration/configuration
    "appId": "com.{{executable}}.app",
    "productName": "{{executable}}",
    "copyright": "Copyright © 2020",
    "buildVersion": "1.0.0",
    "compression": "maximum",
    "directories": {
      "output": "../../../bin/Desktop"
    },
    "extraResources": [
      {
        "from": "./bin",
        "to": "bin",
        "filter": ["**/*"]
      }
    ],
    "files": [
      {
        "from": "./ElectronHostHook/node_modules",
        "to": "ElectronHostHook/node_modules",
        "filter": ["**/*"]
      },
      "**/*"
    ]
  }
}


implement spin -vvvvv

This provides an alternative to RUST_LOG=spin=trace.
Most non-rust users are familiar with the -vvvvv pattern for command-line interfaces. This provides an easy way to change between the five different log levels Rust provides:

0: "off"
1: "error"
2: "warn"
3: "info"
4: "debug"
5: "trace"

Remembering the number of occurrences of a flag is easier to remember than the string values for each log level, as each language/framework can use different terminology or letter casing for different log levels.
closes #608.
$ ../../target/debug/spin -vvvvv bindle push --bindle-server http://localhost:8080/v1
2022-06-28T19:27:14.367304Z TRACE new: bindle::standalone: Computed invoice file path invoice_path=/tmp/.tmp55Qe0T/64b479265e1f51ccc669de307a054e2c3fd4756eb0eae6ebfbe1bfde7ca37eeb/invoice.toml
2022-06-28T19:27:14.367367Z TRACE new: bindle::standalone: Listing parcels in parcels directory parcels_dir=/tmp/.tmp55Qe0T/64b479265e1f51ccc669de307a054e2c3fd4756eb0eae6ebfbe1bfde7ca37eeb/parcels
2022-06-28T19:27:14.367759Z  INFO bindle::client: Provided base URL missing trailing slash, adding...
2022-06-28T19:27:14.391917Z TRACE get_yanked_invoice{invoice_id=spin-hello-world/1.0.0}: bindle::client: req=RequestBuilder { method: GET, url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(8080), path: "/v1/_i/spin-hello-world/1.0.0", query: Some("yanked=true"), fragment: None }, headers: {} }
2022-06-28T19:27:14.392072Z TRACE get_yanked_invoice{invoice_id=spin-hello-world/1.0.0}: hyper::client::pool: checkout waiting for idle connection: ("http", localhost:8080)
2022-06-28T19:27:14.392146Z DEBUG get_yanked_invoice{invoice_id=spin-hello-world/1.0.0}: reqwest::connect: starting new connection: http://localhost:8080/
2022-06-28T19:27:14.392229Z TRACE get_yanked_invoice{invoice_id=spin-hello-world/1.0.0}: hyper::client::connect::http: Http::connect; scheme=Some("http"), host=Some("localhost"), port=Some(Port(8080))
2022-06-28T19:27:14.392337Z DEBUG hyper::client::connect::dns: resolving host="localhost"
2022-06-28T19:27:14.392592Z DEBUG get_yanked_invoice{invoice_id=spin-hello-world/1.0.0}: hyper::client::connect::http: connecting to <IP_ADDRESS>:8080
2022-06-28T19:27:14.392712Z TRACE get_yanked_invoice{invoice_id=spin-hello-world/1.0.0}: mio::poll: registering event source with poller: token=Token(1), interests=READABLE | WRITABLE
2022-06-28T19:27:14.392819Z TRACE get_yanked_invoice{invoice_id=spin-hello-world/1.0.0}: mio::poll: deregistering event source from poller
2022-06-28T19:27:14.392887Z TRACE get_yanked_invoice{invoice_id=spin-hello-world/1.0.0}: hyper::client::connect::http: connect error for <IP_ADDRESS>:8080: ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })
2022-06-28T19:27:14.392959Z TRACE get_yanked_invoice{invoice_id=spin-hello-world/1.0.0}: hyper::client::pool: checkout dropped for ("http", localhost:8080)
2022-06-28T19:27:14.393093Z TRACE push: bindle::standalone: Loading invoice file from disk path=/tmp/.tmp55Qe0T/64b479265e1f51ccc669de307a054e2c3fd4756eb0eae6ebfbe1bfde7ca37eeb/invoice.toml
2022-06-28T19:27:14.393745Z DEBUG push: bindle::standalone: Attempting to create invoice invoice_id=spin-hello-world/1.0.0
2022-06-28T19:27:14.393931Z TRACE push:create_invoice{id=spin-hello-world/1.0.0}: bindle::client: req=RequestBuilder { method: POST, url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(8080), path: "/v1/_i", query: None, fragment: None }, headers: {"content-type": "application/toml"} }
2022-06-28T19:27:14.394018Z TRACE push:create_invoice{id=spin-hello-world/1.0.0}: hyper::client::pool: checkout waiting for idle connection: ("http", localhost:8080)
2022-06-28T19:27:14.394076Z DEBUG push:create_invoice{id=spin-hello-world/1.0.0}: reqwest::connect: starting new connection: http://localhost:8080/
2022-06-28T19:27:14.394132Z TRACE push:create_invoice{id=spin-hello-world/1.0.0}: hyper::client::connect::http: Http::connect; scheme=Some("http"), host=Some("localhost"), port=Some(Port(8080))
2022-06-28T19:27:14.394192Z DEBUG hyper::client::connect::dns: resolving host="localhost"
2022-06-28T19:27:14.394286Z DEBUG push:create_invoice{id=spin-hello-world/1.0.0}: hyper::client::connect::http: connecting to <IP_ADDRESS>:8080
2022-06-28T19:27:14.394355Z TRACE push:create_invoice{id=spin-hello-world/1.0.0}: mio::poll: registering event source with poller: token=Token(16777217), interests=READABLE | WRITABLE
2022-06-28T19:27:14.394433Z TRACE push:create_invoice{id=spin-hello-world/1.0.0}: mio::poll: deregistering event source from poller
2022-06-28T19:27:14.394479Z TRACE push:create_invoice{id=spin-hello-world/1.0.0}: hyper::client::connect::http: connect error for <IP_ADDRESS>:8080: ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })
2022-06-28T19:27:14.394514Z TRACE push:create_invoice{id=spin-hello-world/1.0.0}: hyper::client::pool: checkout dropped for ("http", localhost:8080)
Error: Failed to push bindle to server

Caused by:
    0: Failed to push bindle from '/tmp/.tmp55Qe0T' to server at 'http://localhost:8080/v1'
    1: Error creating request
    2: error sending request for url (http://localhost:8080/v1/_i): error trying to connect: tcp connect error: Connection refused (os error 111)
    3: error trying to connect: tcp connect error: Connection refused (os error 111)
    4: tcp connect error: Connection refused (os error 111)
    5: Connection refused (os error 111

Signed-off-by: Matthew Fisher<EMAIL_ADDRESS>

Okay, pushed a fix in 3ca315e.

Got @bacongobbler's approval to close this for now. We'd still like to see this feature but whoever implements can open a fresh PR.


VFT Token

About VFarm
VFarm system is a proven, research backed farming technology licensed by Agrosky Sdn. Bhd (ASSB), a vertical farming specialist. The system provides a DLT blockchain layer that creates an immutable record for each batch of produce as to enable verification of source, planting, packaging and distribution. This in turn enables ecosystem users to not only have total traceability of the produce on food safety but also drive social impact via informed choices on their purchases.
Document
Whitepaper
Summary
Main website https://vfarm.global
Twitter https://twitter.com/vf4rm
Telegram https://t.me/Vfarmofficial
GitHub https://github.com/vfarm
Facebook https://www.facebook.com/vf4rm
gas limit:60000

建议一个 pr 提交一个代币，这个先关了，请按要求重新提交。


SolaceIO: refactor to allow inheritance of BasicAuthSempClient

Users were not able to create a custom BasicAuthSempClient due to some package-private fields. This PR fixes that and adds example on how to create and use a custom client.
Depends on https://github.com/apache/beam/pull/32060, waiting for that to be merged

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Run Java PreCommit

Run Java_Solace_IO_Direct PreCommit

Run Java_Examples_Dataflow PreCommit

@Abacn now that @sjvanrossum lgtm'ed, can I ask you for a review?


Storage Explorer doesn't work with certs generated from dotnet dev-certs

We want to be able to use the certs generated by dotnet dev-certs to help create a seamless dev exp across Azurite, ASP.NET, Storage Explorer, and the SDKs.  This issue is blocking that exp.

Generate a cert

Install dotnet SDK
dotnet dev-certs https --trust -ep cert.pfx -p azurite

Start Azurite with that cert
npm install azurite
azurite --oauth basic --cert cert.pfx --pwd azurite --blobHost localhost --queueHost localhost

Create connection to Azurite using this connection string
DefaultEndpointsProtocol=https;AccountName=devstoreaccount1;AccountKey=Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==;BlobEndpoint=https://localhost:10000/devstoreaccount1;QueueEndpoint=https://localhost:10001/devstoreaccount1;

Add cert to Storage explorer

Edit -> SSL Certificates -> Import Certificates
Import the cert.pfx file you created in first step

Reload Storage Explorer

Try to access Azurite node and you get this:

I tried importing the cert into the Root Certificate store, but still got the same issue. Here's the command I tried:
certutil -f -user -enterprise -p azurite -importpfx root cert.pfx
This same cert works with the Azure SDKs, so I think there's something about Storage Explorer that doesn't support this cert.  The source for how this cert is generated is here:
https://github.com/dotnet/aspnetcore/blob/master/src/Tools/dotnet-dev-certs/src/Program.cs

Cert was of wrong format. @jongio and his team are going to help us with writing code to convert these certs into the cert format we need.

@jongio , can you ping me with an update on what you'd like done about this?

We are still trying to find an existing package that can convert from pfx to pem - but have been unable to do so.  It's still on radar.  We don't have an ETA at the moment.

We are still trying to find an existing package that can convert from pfx to pem - but have been unable to do so. It's still on radar. We don't have an ETA at the moment.

Is there any more changes that also needs to be applied to the certificate? I cannot get it to work with a manually converted PFX -> PEM.

I don't think anyone has it working.

I don't think anyone has it working.
Actually, I got it mostly working after a reboot and fiddling with it some more.
I say mostly as it seems I can do most things, but I get cert error when trying to delete a blob.

Thanks.  Can you post here what you had to do to get it working and the error you are getting?

Unfortunately, in trying to reproduce what I thought was the successful steps (after resetting storage explorer), I could not get it to work again...
I popped back into the docs and read about the "Use system proxy (preview)" proxy setting that's supposed to make the explorer use the certificates from the OS etc. and after setting that it all (mostly) worked again just as it did before the reset. I must have changed that setting at some point while trying things for hours and hours and not realizing that it was the resolution.
So although it appears I didn't have solution for the import certificate issue - it solves my issue and I can now use the explorer with the dev-certs.
The error I've gotten so far is still an certificate error on trying to delete a blob. But I expect this will be resolved when the system proxy comes out of preview, as the docs does say it yet does not support deleting blob's for instance.


Update NPS URL to include context parameters

Backporting changes made in ADS to be able to capture Product version and other parameters via context params.
ref PR: https://github.com/microsoft/azuredatastudio/pull/24723
Survey URL format details: https://meganvwalker.com/passing-variables-through-the-forms-pro-survey-url/
We internally received this information from the Survey teams.

The link parameters work as intended for VS Code - the VS Code NPS survey is hosted on Survey Monkey, while it appears the ADS NPS survey is hosted on MSFT Customer Voice.  I believe this would explain the difference in parameter requirements.
Looking at our survey data, >99% of responses are correctly assigned a MachineId, Version, and OS.  Therefore I think this fix is not suitable for the upstream VS Code branch.


Reference the correct app in LUIS Change Model quickstart

@n01d : Thanks for your contribution! The author, @diberry, has been notified to review your proposed change.

@n01d Thank you for taking the time to submit this correction. #sign-off


settings in init was missing a self.

Wizard didn't work for me, so I had to rename the settings variable

@fredo I think this is related to what you just saw. From glancing at it, it doesn't sound like the right fix.
Related to #115

we don't need that anymore


Add test/ to .dockerignore to exlcude tests and test assets from frontend image build context

Fixes #856 by @dhruvkb
Description

Adds test/ to the file openverse/frontend/.dockerignore
Testing Instructions

Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@dhruvkb I was looking up online. I have an idea of how to make a volume and mount it to a docker container. But I am not able to figure out how to add the test/ directory to the volume. Let me know where I can know more about this.

@sehgxl the tests folder will need to be added to the volume mount here:
https://github.com/WordPress/openverse/blob/8afb04bc81429cda3f0aa3f8be1d22ad4bd8cbc7/frontend/docker-compose.playwright.yml#L10-L13
Likely something along the lines of  - ./test:/frontend/test. Although, it looks like those files should be mounted anyway, so I'm a bit surprised that adding them to the ignore is causing the tests to fail 🤔

I've kicked off the failed tests again, perhaps it was a flake!

@AetherUnbound @dhruvkb Thanks guys for the help!

@sehgxl I'm not a guy, but I'm still happy to have been of assistance 😁

@sehgxl I'm not a guy, but I'm still happy to have been of assistance 😁

My bad, have edited the comment :)


E0000 No JSON object could be decoded

Version:
cfn-lint 0.9.1
Error:
Command: cfn-lint --template master.yaml
Error message: E0000 No JSON object could be decoded
master.yaml:1:1
This is occuring on Ubuntu running on Windows 10, latest version of AWSCLI, Python version 2.7.15rc1
I am new to using this setup, so excuse any missing info and please let me know what additional info you may need.

Sorry for the delay in responding.  I'm trying to mimic your test environment to see what I can figure out.

I was able to get a Windows Server 2016 (1709), Ubuntu, with python version 2.7.12, cfn-lint v0.9.1 working.
What version of Ubuntu are you using 18.04 or 16.04?  Going to see what an upgrade to Python brings me.

Hey --
Interesting. Any chance you can share your template? That often means the JSON in the template is malformed to the point we can't see it as valid.
Maybe run it through a JSON parser?

Hi there,
Thank you for your replies. Interestingly, my colleague has just successfully run the command and pushed the update using the same file and the same setup. I just ran the master.yaml through a josn parser and it fails saying:
SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data
So I am bewildered as to why it's worked for him and failed for me. I will continue to investigate with him and update you as soon as I can.

@MrBeardyDrummer were you able to figure this out or any more insights I could take to try repeat this?

Closing this from inactivity.  We can always re-open if needed.

FWIW, I ran into this today using 0.26.0 of cfn-lint with a yaml template. It turned out to be a trailing tab char on one of the lines. I used cfn-flip to try to convert from yaml to json while troubleshooting and it blew up with a json parse error. Hopefully this saves someone time in the future.


[Improvement][Document] low version of Mysql jdbc driver may cause serious problem

Describe the question
Some low version of Mysql jdbc driver may cause serious problem. In my case 5.1.12 breaks the fundamental function of quartz and causes scheduler not working, though 5.1.47+ should promise work.
For better searhing for this issue, basically quartz will throw an exception sth like syntax error around 'OPTION SQL_SELECT_LIMIT=1'   ....
After I changed to mysql driver 5.1.47, the error is gone.
What are the current deficiencies and the benefits of improvement

A clearer requirement on mysql jdbc driver version in document (like 5.1.47+) would save people from hitting such annoying serious problem.

Which version of DolphinScheduler:
1.3.4
Describe alternatives you've considered

good suggestion
is there anybody could add some text on official github web : https://github.com/apache/incubator-dolphinscheduler-website

good suggestion
is there anybody could add some text on official github web : https://github.com/apache/incubator-dolphinscheduler-website

I want to add it

I want to add it

I want to add it

Looking forward to your PR submission, please let me know when you are done and I will review it for you

I want to add it

Looking forward to your PR submission, please let me know when you are done and I will review it for you

I want to add it

Looking forward to your PR submission, please let me know when you are done and I will review it for you

https://github.com/apache/incubator-dolphinscheduler-website/pull/281

I want to add it

Looking forward to your PR submission, please let me know when you are done and I will review it for you

https://github.com/apache/incubator-dolphinscheduler-website/pull/281

close by https://github.com/apache/incubator-dolphinscheduler-website/pull/281

close by https://github.com/apache/incubator-dolphinscheduler-website/pull/281


Issue with grpc for loading of pbjs 6 object when proto file references imports to other proto files.

Hello,
I have used the master branch's source code (which contains support for pbjs v6) and have resolved the path for imports for protobufjs object (v 6.7.3).
The grpc.loadObject(root) works when proto file does not contain any import but I get below exception when the proto file references to other proto files via imports.
Error: no such type
      at Service.lookupType (node_modules/protobufjs/src/namespace.js:357:15)
      at Method.resolve (node_modules/protobufjs/src/method.js:138:45)
      at Service.resolveAll (node_modules/protobufjs/src/service.js:109:20)
      at getProtobufServiceAttrs (node_modules/grpc/src/node/src/protobuf_js_6_common.js:115:11)
      at loadObject (node_modules/grpc/src/node/src/protobuf_js_6_common.js:143:25)
      at node_modules/grpc/src/node/src/protobuf_js_6_common.js:150:22
      at node_modules/lodash/lodash.js:4944:15
      at baseForOwn (node_modules/lodash/lodash.js:3001:24)
      at node_modules/lodash/lodash.js:4913:18
      at Function.forEach (node_modules/lodash/lodash.js:9359:14)
      at loadObject (node_modules/grpc/src/node/src/protobuf_js_6_common.js:149:7)
      at node_modules/grpc/src/node/src/protobuf_js_6_common.js:150:22
      at node_modules/lodash/lodash.js:4944:15
      at baseForOwn (node_modules/lodash/lodash.js:3001:24)
      at node_modules/lodash/lodash.js:4913:18
      at Function.forEach (node_modules/lodash/lodash.js:9359:14)
      at loadObject (node_modules/grpc/src/node/src/protobuf_js_6_common.js:149:7)
      at node_modules/grpc/src/node/src/protobuf_js_6_common.js:150:22
      at node_modules/lodash/lodash.js:4944:15
      at baseForOwn (node_modules/lodash/lodash.js:3001:24)
      at node_modules/lodash/lodash.js:4913:18
      at Function.forEach (node_modules/lodash/lodash.js:9359:14)
      at loadObject (node_modules/grpc/src/node/src/protobuf_js_6_common.js:149:7)
      at node_modules/grpc/src/node/src/protobuf_js_6_common.js:150:22
      at node_modules/lodash/lodash.js:4944:15
      at baseForOwn (node_modules/lodash/lodash.js:3001:24)
      at node_modules/lodash/lodash.js:4913:18
      at Function.forEach (node_modules/lodash/lodash.js:9359:14)
      at Object.loadObject (node_modules/grpc/src/node/src/protobuf_js_6_common.js:149:7)
      at Object.loadObject (node_modules/grpc/src/node/index.js:100:41)

Thank you.

Are you calling grpc.loadObject directly, or through grpc.load? If you're calling it directly, that implies that you are loading the proto file with Protobuf.js yourself; have you tried doing anything with the loaded object yourself to verify that it loaded correctly? If you're using grpc.load, you should use the alternate argument format and pass an object with root and file properties, where root is the path that the proto imports are relative to, and file is the relative path from root to the file that you want to load.

grpc.load using root and file properties works and hence closing. Thank you.


[docs] Demo LoadingButtonsTransition crashes

[ ] I have searched for similar issues in this repository and believe that this is not a duplicate.

Steps to Reproduce

Visit https://mui.com/material-ui/react-button/#main-content
??
demo LoadingButtonsTransition crashes

Your Environment

Tech
Version

MUI
v5.6.4

netlify deploy
https://627113f59699500008a660b9--material-ui-docs.netlify.app

Browser
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/100.0.4896.127 Safari/537.36

@maxculturait I could not reproduce it. Can you check again?

It's Ok, not crashes in this moment. 💪😁
El mié, 11 may 2022 a las 2:31, Zeeshan Tamboli @.***>)
escribió:

@maxculturait https://github.com/maxculturait I could not reproduce it.
Can you check again?
—
Reply to this email directly, view it on GitHub
https://github.com/mui/material-ui/issues/32642#issuecomment-1123203473,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYBMNOJL5PBBOMU3K6UFS33VJNA4TANCNFSM5VFKCMIA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
http://www.culturait.com
Max Santana
Front-End React Js Developer
@.***


the window is very small

fresh download i tried deleting and reinstalling but i keep getting a small window with all the buttons bassically unusable. pls help
https://i.gyazo.com/526bc30fbd05d60d70e4060a9cc6ecd8.png

Try https://github.com/runelite/runelite/wiki/DPI-scaling-on-Windows , also it seems you created a duplicate issue.


Laboratory work #3, Zvarichuk Nikolay - 23FPL1

I can't understand how to solve problem with "Returning Any from function declared to return "Union[SupportsDunderLT[Any], SupportsDunderGT[Any]]"  [no-any-return]"

@Niko-Zvar do not use russian language in code comments and git commit messages, get used to English everywhere - it is international language of IT


Issue/92 2

#92

I fetched the branch and will make some minor changes (e.g. removing the committed node_modules).


How do you add custom deck images?

I searched and could find no folder named "Decks".
I created a folder named Decks and added two folders with cards in them, but the only thing in the card options are "Tarot" and "Standard". I did a search throughout my vault for names like "Decks", "Tarot", and "Standard" but nothing comes up pertaining to this plugin.
Where do I add the custom deck images?

Yeah, you have to create the folder yourself in your vault, and it must be at the root of your vault by default. If you'd like to nest your decks somewhere else, you can change the folder in plugin settings (e.g. set the option to Assets/Decks or something).
Also worthy of note, your folder needs to be named Decks and not decks, but I think you're good there.
You also need to hit the refresh button, or switch to another tab and back, for changes to take place 😓

After watching Josh P's latest video, it also appears that the images of the cards in your Decks folder must be PNG format. JPG and WEBP display blanks when drawn.

JPG format should work correctly if you're on version 0.3.6 or later
There is a PR for WEBP support https://github.com/alexkurowski/solo-toolkit/pull/8

After watching Josh P's latest video, it also appears that the images of the cards in your Decks folder must be PNG format. JPG and WEBP display blanks when drawn.

Not true, at least in the versions I downloaded. I converted a bunch of cards from gif to jpg and they work just fine in the program.

There indeed was a bug in prior versions where files with extension ".jpeg" wouldn't load, but ".jpg" worked fine. So that might be the reason.
Also, I should add support for gifs! Would be fun to have animated cards.

There indeed was a bug in prior versions where files with extension ".jpeg" wouldn't load, but ".jpg" worked fine. So that might be the reason. Also, I should add support for gifs! Would be fun to have animated cards.

...and disregard all that work I did converting the gifs to jpg files? Oh, the nerve! J/k
It would be awesome if you could support for gif files!
Hopefully you'll fix the issue with it creating a new image whenever you drag a card to canvas or excalidraw.
I'm happy either way.


License?

What is the license of this ontology?
On https://auroral.iot.linkeddata.es/def/shipment/ there is only a placeholder for the license, so no luck there.

http://purl.org/NET/rdflicense/cc-by4.0 according to the code.

@mariapoveda thank you for your reply. I've found a reference to CC-BY 4.0 in https://github.com/oeg-upm/auroral-shipmentBiomass-ontology/blob/main/OnToology/ontology/shipment.ttl/documentation/ontology.rdf
But it is weird, because there is no license information in https://github.com/oeg-upm/auroral-shipmentBiomass-ontology/blob/main/ontology/shipment.ttl
I am wondering which RDF file represents the latest state.


Updating vaccination language across all SAVI Vaccine plots and tables

Closes #47

@michaelarikard I went ahead and finished all the vaccine updates for plots, tables, and maps.
Here's what they look like rendered:
plot_vaxcoverage

[ ] Can we check that the bottom notes don't normally overflow?

They do overflow here, but that might be because I didn't set the width correctly.
plot_vaxcurve

Looks good
map_vaccinations

Looks good
table_10vaccinations(., "fully")

Good.
table_10vaccinations(., "people")

Good.
table_countriesofconcern()

Good.
Can you and @kimkimroll check over and make any changes?

Closed #47


Conda package name for parasail is 'parasail-python'

The conda package name for parasail is parasail-python:
https://anaconda.org/bioconda/parasail-python
But it's listed as parasail in the environment file:
https://github.com/jts/ncov-tools/blob/511f5efbdb9f3f13b37529cfe287408f30b0a0a7/workflow/envs/environment.yml#L19

Thanks Dan, good catch. I’ll fix that soon.

On Dec 18, 2020, at 5:58 PM, Dan Fornika<EMAIL_ADDRESS>wrote:

The conda package name for parasail is parasail-python:
https://anaconda.org/bioconda/parasail-python
But it's listed as parasail in the environment file:
https://github.com/jts/ncov-tools/blob/511f5efbdb9f3f13b37529cfe287408f30b0a0a7/workflow/envs/environment.yml#L19
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Oh, just saw the PR - thanks for that too.


Difference between base and large tokenizer?

I understand that a cased tokenizer and an uncased one are surely different because their vocabs are different in casing, but how does a base tokenizer different from a large tokenizer? Does a large tokenizer have a larger vocab?

I haven't looked in the details of the vocabularies for each model.
If you investigate this question, be sure to share the results here, it may interest others as well!

I did a diff on the two vocabulary files and there is no difference. As long as you use the uncased version at least. I haven't investigated others.

I can validate @mhattingpete 's research.
I tokenized a big collection of text with the uncased tokenizer from both the base and the large model and both tokenizations are identical.


Feature/protege refactoring reflection

Feature branch updated with latest commits in version4. Successful re-testing of Ontop Endpoint and Protégé Plugin (Java 8 and Java 17). Ready to merge.

Hi @kontchakov, @bcogrel,
I integrated the last changes in version4 into the feature branch and successfully re-tested everything.
I remember that previously there was an automatic build & test feature of the branch code to be merged, which I relied on in a previous pull request. Now I could not find it anymore, and similarly I don't know which merge policy you have for the project.
Therefore, I just created a pull request ensuring that everything is ready for the merge, but I leave you the task of carrying out the actual merge (or to inform me about the aforementioned issues so that I can proceed myself).

Done, thanks a lot!


API Logging doesn't work without providing version

Description
When using API logging feature [1], if an API is invoked as the default version without specifying the version, there are no logs printed in the api.log file. When version is used, the logs are getting printed.
[1] https://apim.docs.wso2.com/en/latest/observe/api-manager/monitoring-api-logs/
Steps to Reproduce

Deploy the PizzaShack API
Enable 'Make this the default version' property and redeploy
Change the API log level to full using devops API
Invoke the API with the version and without the version
Check the api.log file

Affected Component
APIM
Version
4.1.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

HI Team,
Appreciate your input on this.
Thank You
DilshanF

Hi Team,
Kind reminder on this.
Thanks,
Methma

closing as the public fix: https://github.com/wso2/carbon-apimgt/pull/12063 has already been merged.


Add IOBufOutputStream

Adds an OutputStream that allows retrieving the content as
folly::IOBufs. This is for use in partitioned output to get zero copy
message passing between Velox and the exchange server in presto_cpp.
Changes the Position type of ByteStream to a number so as to be
compatible with C++ streams.

Assuming this PR is no longer needed. Feel free to re-open if needed.

In seekp, it should be ranges_.empty() && position == 0. If we do a seekp into a stream without content, the only legitimate offset is 0. Anything else should error out.
Fixed extend. Extend is covered in the test, where we write for the first time and then rewrite everything.
Added some comments about the relation of members of ByteRange.
From: Deepak Majeti @.>
Sent: Monday, March 7, 2022 6:37 AM
To: facebookincubator/velox @.>
Cc: oerling @.>; Mention @.>
Subject: Re: [facebookincubator/velox] Add IOBufOutputStream (PR #1123)
@majetideepak commented on this pull request.
@oerling https://github.com/oerling  I made a high-level pass. Left a couple of comments. Thanks.

In velox/common/memory/ByteStream.cpp https://github.com/facebookincubator/velox/pull/1123#discussion_r820752529 :

int64_t size = 0;

for (auto& range : ranges_) {
if (&range == current_) {

 return current_->position + size;

}
size += range.numValues();
}
VELOX_FAIL("ByteStream 'current_' is not in 'ranges_'.");
+}

+void ByteStream::seekp(std::streampos position) {

int64_t toSkip = position;
if (current_ && current_->position > current_->fill) {
current_->fill = current_->position;
}
if (ranges_.empty() && position == 0) {

should this be (ranges_.empty() || position == 0)?

In velox/common/memory/ByteStream.cpp https://github.com/facebookincubator/velox/pull/1123#discussion_r820758340 :

@@ -31,4 +66,55 @@ void ByteStream::flush(OutputStream* out) {
}
}

+void ByteStream::extend(int32_t bytes) {

if (current_) {
current_->fill = current_->position;
}
if (current_ && current_ != &ranges_.back()) {
++current_;
current_->fill = current_->position;
if (current_->fill < current_->position) {

This block of code is not clear. We have current_->fill = current_->position; on line 75 and then
if (current_->fill < current_->position) on line 76 without any body.

In velox/common/memory/ByteStream.cpp https://github.com/facebookincubator/velox/pull/1123#discussion_r820763667 :

@@ -31,4 +66,55 @@ void ByteStream::flush(OutputStream* out) {
}
}

+void ByteStream::extend(int32_t bytes) {

if (current_) {
current_->fill = current_->position;
}
if (current_ && current_ != &ranges_.back()) {
++current_;
current_->fill = current_->position;
if (current_->fill < current_->position) {

Probably add a test for ByteStream::extend
—
Reply to this email directly, view it on GitHub https://github.com/facebookincubator/velox/pull/1123#pullrequestreview-901757980 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKPPPT6MS64AICSCIINKAP3U6YIATANCNFSM5P36OSZA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you were mentioned.Message ID: @.***>

Thank you, Masha.  Edited as requested.
For ByteRange, we can look at it like a folly::Range, spo buffer and size (count of addressable elements) are the right names. Fill only makes sense for the last one. So I put this in ByteStream. The point is that all but the last range are filled to size and the last range is filled to lastRangeEnd_. A seekp can set the writing position to something else, so we must keep track of the end (e.g. file length) separately. The use case is when writing stuff into Page serialization and then seeking back to the start to fill in size and checksum.
The end of last range must be set on seekp and is convenient to set before flush.

Updated the follow up item 1127. would you take a look?


Linux support?

I was wondering if Linux support is planned.
Thanks!

Hi neuromancer! In fact, code is multiplatform from the beginning so it should run on Linux if you compile it. There is no binary package yet though, since this is VERY early stage of development.
However, thanks for interest! :)

In fact, code is multiplatform from the beginning so it should run on Linux if you compile it.

Given that I see no INSTALL.md or COMPILE.md or any instructions on how to compile it, and as there is no CMakelist.txt, Makefile, or any other usual prebuild file to set up the compilation process, how is one supposed to compile this?
Also, have you thought about using Github's Travis-CI to set up automated builds to verify and test that compilation always succeeds as development continues?

@OvermindDL1 But there is *.cbp file which is a Code::Blocks project file. You can open it with C::B and compile the code.
There is no README and other instructions because I put this code here just for my convenience. You can use it as is. It's my private free-time project and I can't make any guarantees.

@OvermindDL1 But there is *.cbp file which is a Code::Blocks project file. You can open it with C::B and compile the code.

What is the command for doing so?  Via a quick google search I was unable to deduce how to compile a project via the codeblocks executable short of spooling up an X Server (which I do not have available on my build system, only the libraries).  Even something as simple as 'codeblocks --buildorcodeblocks --build --no-splash-screen` or any mixture of a variety of flags that I could find were unproductive as it kept throwing the error of:
11:59:15: Error: Unable to initialize GTK+, is DISPLAY set properly?

Why would a build system need a running GTK+ instance?

@OvermindDL1 Code::Blocks is an open source IDE for C++, not a command. You open project file in the IDE and build it from that level. That's why it needs GTK+, which is the library Code::Blocks is made with. I'm not aware of any way to build code through C::B in text mode.
Well, you can make Makefile by yourself if you really need it - cbp file is really an XML file. You can open it by any text editor, check what necessary libraries are and make Makefile from that.
However, game itself needs OpenGL to run, since... well, it's meant to be 3D game so it renders graphics in 3D. So you need X Windows environment and graphics card ready for OpenGL 3.
Regarding work progress - I'm now just playing with parsing sc2 save game file. For now I can read heights for all terrain tiles and display it in 3D with some simple texture. So it's really early stage of development. SC2 save game format is a proprietary one and there is no official documentation.

@OvermindDL1 Code::Blocks is an open source IDE for C++, not a command. You open project file in the IDE and build it from that level. That's why it needs GTK+, which is the library Code::Blocks is made with. I'm not aware of any way to build code through C::B in text mode.

Ah, my build server is headless so I have no running X server.  How odd, I've never seen a build system require a GUI, that is weird.  ^.^
Have you thought of using CMake?  It can output Codeblock files (as well as files for about any other IDE in existence as well), it has essentially taken over as 'the' C++ build system over the past decade.  :-)
I compile everything remotely as my desktop is... insufficient, the compilation speeds are astounding, but I do test the final binaries locally so OpenGL is available here, just not the build system.

Regarding work progress - I'm now just playing with parsing sc2 save game file. For now I can read heights for all terrain tiles and display it in 3D with some simple texture. So it's really early stage of development. SC2 save game format is a proprietary one and there is no official documentation.

Is any of the data being read in to display model data yet, or is it still at the parsing stage?

As I said - terrain heights for tiles. Code displays those with simple texture:

That's all for now.

I saw a recent commit in this project. Maybe you should add a screenshot (even that one), to show the state of this re-implementation?


Bump client-go and fix lint

Please provide a description of this PR:
fix: https://github.com/istio/istio/pull/49080

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
155fa8d9ee15f49ba8d55cbe3190777cc9e26155
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Quick Accent doesn't have greek characters anymore

Description of the new feature / enhancement
I noticed that in Powertoys v0.74.0 the greek accents aren't available anymore, this is very sad because those characters are massively used in scientific papers and I really loved the feature, I think it was very useful.
Scenario when this would be used?
Scientific papers, mathematics texts in general
Supporting information
No response

Duplicate of #28785

/dup #28785


fix code coverage windows

On windows path to grcov is:
info: grcov installed at /c/Users/runneradmin/.install-action/bin/grcov.exe
+ grcov --version
grcov 0.8.19

On unix:
info: grcov installed at /home/runner/.cargo/bin/grcov
+ grcov --version
grcov 0.8.19

In both cases they should be available in $PATH
Tested on my fork: https://github.com/piotrkwiecinski/coreutils/actions/runs/8926781245/job/24518622684?pr=7
Fixes https://github.com/uutils/coreutils/issues/6331

Fantastic! The CI failure is #6275 and #6333, so not a blocker.


[ImageResizer] Fix blurry images in some cases

Summary of the Pull Request
In some cases a resized image becomes very blurry.
Loading the input image file into cache in BitmapDecoder resolves the blurriness.
PR Checklist

[x] Closes: #10675
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] ~Tests: Added/updated and all pass~
[ ] ~Localization: All end user facing strings can be localized~
[ ] Dev docs: Added/updated
[ ] ~New binaries: Added on the required places~

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

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

Detailed Description of the Pull Request / Additional comments
I tried to find the technical reason why caching resolves it, or why it is blurry in the first place, but was unable to locate it.
The issue seems to occur only on specific images. I was not able to identify a strong parameter that would cause it.
Nevertheless, with the images I will attach to the PR, I hope to make an evident case for the factual resolution of the issue. And maybe or hopefully someone with more knowledge about the respective used APIs will be able to provide the factual, technical reason for it.

ResizeOperation uses BitmapDecoder to load the image file. In main the decoder is instantiated with BitmapCacheOption.None. Changing it to BitmapCacheOption.OnLoad resolves the issue.
The pass-along of image data is as follows:

create decoder
decoder -> image file metadata
decoder -> BitmapFrame -> TransformedBitmap -> BitmapFrame -> encoder (Frames.Add()) -> encoder.Save()

transformdoc mentions the following about loaded cached image data potentially mismatching target pixel size, which made me wonder whether the no-cache delayed reading in combination with the transformation leads to reduced-size-reading. I was not able to confirm it. Given that most images I tested did not have a problem, I don't think it's a plausible cause.

If you don't do this, the application will cache the image as though it were rendered as its normal size rather than just the size that is displayed.

For multiple test images, any differences were hard to spot, they did not result in blurriness. The image I had issues with in the past being very colorful made me wonder whether amount of color information could make the difference. But with the technical context, that would only make sense if the encoder caching somehow handled color information differently, which seems unlikely too.
Validation Steps Performed
Manual tests with multiple image files with variance. I will attach test cases in the following comment.

Given that I don't have a full technical explanation why this works, I'm creating this PR as a draft. I'm looking for feedback whether this would be fine, or for help in explaining why it resolves it.

test case 1: blurry result
original:

scaled to 480p, and face 2x  zoomed to more obviously show the difference:

main
cached

test case 2: subtitles with text online slightly different
original:

main
cached

no obvious/barely visual differences

main
cached

main
cached

slight differences in:

main
cached

Hi @Kissaki ,
Thanks for opening this PR!
Can you please provide a sample image that's affected by this change? Inside a zip file might be better, since GitHub might manipulate the image to embed in the comment.
Thanks, in advance.

imageresizer blur example files.zip

@Kissaki
Thank you for opening this PR!
I had the chance to give it a try. On images with text I found it that it looked like it made the text harder to read, so this might be something not everyone would want.
I think this might be a good Setting with a Toggle to have on the Settings page of Image Resizer, under encoding. Something like "Use bitmap cache when decoding the original image", with the description text that it might help avoid blurriness in some edge cases.
What do you think?

That seems like a reasonable workaround/implementation.


Adding a Keyboard keylogger written in C++

What type of PR is this? (check all applicable)

[Yes ] 🚀 Added Name
[Yes ] ✨ Feature
[Yes ] 🌟 stared the repo
[ ] 🐛 Grammatical Error
[ ] 📝 Documentation Update
[ ] 🚩 Other

Description
This is a keyboard keystroke logger or keylogger for short and it has been written in C++. This program creates a file called 'Record.txt' and stores every recorded keystroke into that file.
Add Link of GitHub Profile
https://github.com/Raven1233

@keshavsingh4522 Can this be assigned the hacktoberfest 2021 label?


Compile for Linux // Create Snap Package

Snap package cannot found system installed binary, for example
cat plan | loop-rs 'curl -O -J -L $ITEM'

it say:
sh: 1: curl: not found

When I specify the path like /usr/bin/curl
it say
sh: 1: /usr/bin/curl: not found

Is it just curl? Or any system binary?

Any system binary(but snap also provide some common binary)
for example loop-rs 'ls /bin|wc' --until-success
    133     133     991

but execute ls /bin|wc directly
   4767    4767   50358

Snap is actually using it's own rootfs, only /home folder are shared with host OS


Action fails when lint errors are found in unchanged files

Hi, I'm trying to use buf lint in an existing protobuf code base. There are files that does not pass the buf lint rules, in this case buf-lint-action itself fails because those files are not present in the pull request diff:
Failed to write comments in-line: Unprocessable Entity: "Path is invalid and Pull request review thread review comments is invalid"

I think our use case is common for teams that has some protobuf files before introducing buf as their linter tool. Please consider supporting this use case.
As an alternative solution, this is what I chose to do:

Run buf lint --error=format=json
Use jq to format output in this format so that GitHub Pull Request can show them under "Unchanged files with check annotations" section (in "Files changed" tab).

Also because the source code does not provide a FOSS license, I am not allowed to fix your program.
Thanks!

Thanks for the issue! You're right, this is a common use case for anyone migrating to buf. I appreciate the recommendation for the format that renders the errors in the "Unchanged files with check annotations" section, too. In this case, we shouldn't need to use jq though - we can just rely on the functions we already have available in Typescript (similar to what we do for in fileAnnotationToComment.
In the meantime, the action should show you the errors you care about in the console alongside the error you shared above. This is handled here so that we continue to surface lint warnings in the presence of errors just like this. Was the buf lint output available for you there?
As for the license, the license we use in our private repositories was accidentally used. We'll get this updated to the same license in the buf repository right away. We welcome contributions if you're still interested in lending a hand! Otherwise, we'll track this and get a fix out for it soon.

@amckinney Thank you so much for your quick response and changes to the license :)
I did saw buf lint outputs in GitHub Actions, so that part is working as expected.
For now I think we'll use our own solutions for now, to print ::error messages in the step, because the configuration is simpler; we don't have to pass GitHub token in order to run buf checkers.
We also have a minimal config for buf breaking instead of using buf-breaking-action: checking out the base branch in another folder (add another step of actions/checkout), and run buf breakingin the head checkout, to compare the code in the other folder. This way we can avoid considering howbuf` interacts with the git repository internally.
Looking forward to see the improvements of your GitHub Actions :)

We have now updated this Github Action to use the file annotations, so updating to our new release will remove the requirement for the github.token to be set in the configurations: https://github.com/bufbuild/buf-lint-action/releases/tag/v0.3.0
Thank you for filing this and using our Github Actions and feel free to re-open if there are any concerns with the release! :)

@doriable Thanks. The new version works perfect with our project!


Docker engine provision

This is the last version of our distrinet, the main changes include the replacement of lxd container by docker, architecture improvements, and the replacement of ryu by onos controller cluster and the specific implementation of openstack or alcor.


helm upgrade doesn't pick up new values

here my scenario:

first i installed my chart using:
helm install ./my-example-chart.tgz  --set key1=value1
then i introduced new variable in my helm chart and i put it to my values.yaml file with default value.
i again created chart and used this command:
helm upgrade my-existing-chart ./my-example-chart.tgz --reuse-values

after upgrade i see that helm doesn't pick new introduced values it only uses old key-value which is (key1-value1) from first installation.
i was expecting even i supply reuse-values helm should pick new introduced key-value pairs from values.yaml file

@bilalyasar I tested on 2.8.1 and 2.8.2 and I'm getting the same behavior you're seeing but I also expect (like you) to see my new values.

--reuse-values uses exactly the same values it used the previous time. It does not re-merge values. So by design it will ignore any new defaults set in a chart's values.yaml file.
Unfortunately, this behavior cannot be changed during the 2.x lifecycle because it is a compatibility issue.

Thanks for the clarification @technosophos! I've removed the bug label here.
@bilalyasar - To achieve what you're trying to do here, we suggest overriding the values file on the command line.
helm get values REL > values.yaml && helm upgrade REL CHART -f values.yaml

@michelleN thanks for suggestion.
now I have another problem.
when I make helm get values RELEASE it shows to me last upgrade values, It doesn't list to me existing values.
my latest upgrade command was:
helm upgrade REL CHART --reuse-values --set newkey=newvalue
then helm get values REL
returns only:
newkey=newvalue
I was expecting more than 20 key=value pairs.

@technosophos if I understand this correctly, the documentation  should probably be changed:

  --reuse-values         when upgrading, reuse the last release's values, and merge in any new values. If '--reset-values' is specified, this is ignored.

The and merge in any new values bit is kind of misleading.

@bilalyasar What do you get when you use the --all flag: helm get values RELEASE --all ?

@danielcb Does the change in #3713 help make things more clear here?

yep. Thanks.

The new description is indeed more clear, but still doesn't make sense, why not merge set values with default values?
An automated deployment pipeline will break when introducing new values, even if you specified the default value.
Furthermore, I am experiencing another bug with --reuse-values, but I will create a new issue for that.

I am experiencing the same problem when using the upgrade command.
It took me a while to find out how to reproduce:
# install new chart
$ helm install --set foo=bar -n test <chart>
$ helm get values test
foo: bar # all as expected
$ helm upgrade test <chart>
$ helm get values test
foo: bar # still as expected
# all fine from here, now upgrade and add an extra value
$ helm upgrade test <chart> --reuse-values --set foo2=bar2
$ helm get values test
foo2: bar2 # where is foo?
$ helm get values test -a
foo: bar # oh there it is
bar2: bar2
# now upgrade without specifying values
$ helm upgrade test <chart>
$ helm get values test
foo2: bar2 # still no foo
$ helm get values test -a
foo2: bar2 # foo is gone and deployment probably failed

now i am using this command as workaroung // @michelleN
helm get values RELEASE --all > values.yaml && helm upgrade REL CHART -f values.yaml

if is there any other workaround it would be awesome :)

@yoshz yeah that does seem odd. Digging into that a bit more now.

ah so it looks like that's happening because foo was originally set on the command line via the --set flag. Is foo a value in the values.yaml file in the chart? @yoshz

Hello @michelleN / @technosophos I could really use some clarification on the mechanism of --reuse-values ... in v2.x are the default values of a chart ever reloaded / remerged with an upgrade?


TEXT-201 - Simplify assertion

Test expression may be simplified

Coverage decreased (-0.02%) to 97.963% when pulling 086d03d9d1f0d151acbfa407a2294be4a612df50 on arturobernalg:feature/TEXT-201 into 857a3ed9eeda0b197ebe57035d7de52cdf551b60 on apache:master.

Merged, thanks!


String not available in Transifex

hi,
here (http://www2.qgis.org/en/site/getinvolved/development/index.html), the text in note frame about Osgeo userid doesn't appear in transifex and therefore is unstranslated.
There are also other strings not available in Transifex to be translated :

http://www2.qgis.org/pt_BR/site/forusers/index.html : documentation part
the new text in the footpage of every page about fixing translation or reporting bug

Hi, I added it some weeks ago, and have not updated the transifex strings since.
Upon next update it will be available (Normally I do the transifex update after some bigger updates of the website, not for every sentence (it takes me some time to do it).
So probably with the 2.8 release (within a couple of days) I'll update the strings.
Closing for now. Ok?

ok.
Thanks for replying. Don't take into account my completion...


Question: Is long polling supported by Thruway?

Hi Guys,
I am trying to create a real time notification using Thruway and ZMQ. see below code
$loop   = React\EventLoop\Factory::create();
$pusher = new \Thruway\Peer\Client("realm1", $loop);
$pusher->on('open', function ($session) use ($loop) {
    $context = new React\ZMQ\Context($loop);
    $pull    = $context->getSocket(ZMQ::SOCKET_PULL);
    $pull->bind('tcp://[IP]:5555');

    $pull->on('message', function ($entry) use ($session) {
        $entryData = json_decode($entry, true);
        if (isset($entryData['category'])) {
            $session->publish($entryData['category'], [$entryData]);
        }
    });
});

$router = new Thruway\Peer\Router($loop);
$router->addInternalClient($pusher);
$router->addTransportProvider(new Thruway\Transport\RatchetTransportProvider("<IP_ADDRESS>", 8080));
$router->start();

It is located on Server A
When i tried to connect to Thruway using Autobahn JS and transport as websocket which is located on the Server B, it's working,
but when i tried to use longpoll i'm recieving an error of "Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://local.app1.dev' is therefore not allowed access. The response had HTTP status code 400."
I also tried using socket.io to connect but it doesn't work as well, XHR cross-origin issue.
I tried different solution that i found, but unfortunately it doesn't work for me. can someone help me please?
Thanks

Thruway does not support long polling.
If you do need to access a Thruway router using HTTP, please see: https://github.com/voryx/WampPost
While this allows you some access to the router, this is still not long-polling and may also suffer from the CORS issues you are having.

Is WampPost gonna work on older browser that doesn't support websocket as a fallback?

WampPost is not a substitute for long polling.


ruff server fails when calling ruff.applyAutofix from IDE

I just upgraded my LSP setup and packages, and I got a message that ruff-lsp was deprecated and I should use ruff instead.  After updated everything, a command that was previously working in my IDE is no longer working, and I can't figure out why not.  I suspect it might be something to do with my environment, but I've hit a wall trying to debug it.
This is what I'm trying to do, taken from docs / github somewhere from ruff-lsp (I'm using neovim as my IDE):
map("n", "<leader>fa", "", {
  callback = function()
    vim.lsp.buf.execute_command {
      command = 'ruff.applyAutofix',
      arguments = {
        { uri = vim.uri_from_bufnr(0) },
      },
    }
  end,
  desc = "Fix all (ruff-lsp, in python)"
})

After updating, most functionality still works, but this command fails.  I see this in the logs:
[DEBUG][2024-11-20 08:53:25] ...m/lsp/client.lua:565	"LSP[ruff]"	"client.request"	3	"workspace/executeCommand"	{  arguments = { {      uri = "file:///some/path/to/a/file.py"    } },  command = "ruff.applyOrganizeImports"}	<function 1>	1
[DEBUG][2024-11-20 08:53:25] .../vim/lsp/rpc.lua:282	"rpc.send"	{  id = 7,  jsonrpc = "2.0",  method = "workspace/executeCommand",  params = {    arguments = { {        uri = "file:///some/path/to/a/file.py"      } },    command = "ruff.applyOrganizeImports"  }}
[DEBUG][2024-11-20 08:53:25] .../vim/lsp/rpc.lua:404	"rpc.receive"	{  jsonrpc = "2.0",  method = "window/showMessage",  params = {    message = "Ruff encountered a problem. Check the logs for more details.",    type = 1  }}
[DEBUG][2024-11-20 08:53:25] .../vim/lsp/rpc.lua:404	"rpc.receive"	{  error = {    code = -32602,    message = "missing field `version`"  },  id = 7,  jsonrpc = "2.0"}

I can't figure out what thing is missing a version field.  Is this an issue with my .toml settings?  With how I'm calling the LSP command from within the IDE?  Something else?  I don't know what changed in the updated version to make the previously-working configuration no longer work.

I suspect that the problem is that ruff server expects a version argument, which is the version of the document but I don't know how to configure this in neovim. I'm sure @dhruvmanila can help you tomorrow morning. He's our neovim expert.

It's the table within the arguments table that's missing the version key:
map("n", "<leader>fa", "", {
  callback = function()
    vim.lsp.buf.execute_command {
      command = 'ruff.applyAutofix',
      arguments = {
        { uri = vim.uri_from_bufnr(0), version = 0 },
      },
    }
  end,
  desc = "Fix all (ruff-lsp, in python)"
})

You can just set it to 0 for now. I'm not exactly sure of the original intent for this argument because it's currently only being used for the formatting edits (not others):
https://github.com/astral-sh/ruff/blob/f8c20258aec5148029da904fe1ea91c1c8f1be50/crates/ruff_server/src/server/api/requests/execute_command.rs#L79-L84

I'm not exactly sure of the original intent for this argument because it's currently only being used for the formatting edits (not others):

The version allows clients to easily detect outdated server responses (unless they use another mechanism for it). We should use the version for organize imports and fixes too. For example:

The user makes edit, hits save (version = 1)
The LSP sends a format and organize import request to the LSP
The format request finishes first and there are changes. The client applies the changes (version = 2) and it sends another organize import requests
The first organize import request finishes and there are changes. However, it's not safe to apply the edits now because the client is at version 2, but the edits sent by the server are for version 1.

I see, thanks for the explanation.
I did a quick search as to any reference for document version in the lsp client code in Neovim and I found this: https://github.com/neovim/neovim/blob/07db909eb5ae2a559771068be64439eba394cd61/runtime/lua/vim/lsp/util.lua#L2165-L2171 which is then being used in multiple places across the client code.
I think that can be used to get the current version of the document on the client side which can then be passed to the server:
map("n", "<leader>fa", "", {
  callback = function()
	local bufnr = vim.api.nvim_get_current_buf()
    vim.lsp.buf.execute_command {
      command = 'ruff.applyAutofix',
      arguments = {
        {
		  uri = vim.uri_from_bufnr(bufnr),
          version = vim.lsp.util.buf_versions[bufnr],
		},
      },
    }
  end,
  desc = "Fix all (ruff-lsp, in python)"
})

You could also assert that Neovim always keeps track of the versions as it should do so:
        version = assert(
          vim.lsp.util.buf_versions[bufnr],
          ('No version found for buffer %d'):format(bufnr)
        )

Thanks a lot for your help!  I can confirm that setting version as suggested solves the issue for me; querying vim.lsp.util.buf_versions as described above worked without issue.  Setting it to 0 also worked initially, which I tried before finishing reading the comments, though I didn't keep editing to see if it would have failed later.


Yet another error has occurred...

Again, I'm reaching up to you for yet another issue that popped up out of nowhere...
I think I'll just leave this image here and it should tell you enough about the error... at least I hope it will.

It shows up almost towards the end of the booting sequence.
P.S.: I'm sorry for having so many issues... >.>

P.S.: I'm sorry for having so many issues... >.>

No need to be sorry ^^
The source repo for hd has a different name on huggingface now. HDv17 was also released 10 days ago. I have updated the notebook to use the new repo url alongside using the newest checkpoint.
Let me know if you need any help or would like to see any additions to the notebook.

Will do! Thank you again!^^


Updated to the latest dot parser

This should fix a potential problem if a dot file generating tool was
using anonymous subgraphs (for example gvpack does that).
Latest ngraph.fromdot should handle these graphs properly.

@anvaka Thanks!