STAC - Catalog Links have label extension items.

This is a 0.8 catalog that I got from Aaron, which is from an Annotate export:
{
    "stac_version": "0.8.0-rc1",
    "id": "01749366-9e25-4b42-a4d1-5aae9f8c9eec",
    "title": "AL Project Test 5",
    "description": "Exported from Raster Foundry 2019-10-04 15:55:13.759",
    "links": [
        {
            "href": "s3://rasterfoundry-development-data-us-east-1/stac-exports/01749366-9e25-4b42-a4d1-5aae9f8c9eec/catalog.json",
            "rel": "self",
            "type": "application/json",
            "title": "Catalog 01749366-9e25-4b42-a4d1-5aae9f8c9eec",
            "label:assets": []
        },
        {
            "href": "catalog.json",
            "rel": "root",
            "type": "application/json",
            "title": "Catalog 01749366-9e25-4b42-a4d1-5aae9f8c9eec",
            "label:assets": []
        },
        {
            "href": "1a8c1632-fa91-4a62-b33e-3a87c2ebdf16/collection.json",
            "rel": "child",
            "type": "application/json",
            "title": "Layer Collection",
            "label:assets": []
        }
    ]
}

The label:assets property on the link should not be there - label is an Item-only extension. As it's an optional property, the label:asset should also not be in the JSON if it is not utilized.

Task is to remove label:assets for anything that isn't an item - then see if we have more to do after that.


[EVENT REQUEST] Skip OnOpenPage Trigger in calculate plan reports

We want to set the parameters for report 99001017 "Calculate Plan - Plan. Wksh." and report 699 "Calculate Plan - Req. Wksh." via a parameter text. In the Report.Execute(ParameterString) function the OnOpenPage-Trigger is executed and the UseForecast variable is overwritten with the setup value.
We need this events:
report 699 "Calculate Plan - Req. Wksh."
...
        trigger OnOpenPage()
        var
            IsHandled: Boolean;
        begin
            IsHandled := false
            OnBeforeOnOpenPage(IsHandled);
            if not IsHandled then begin
                MfgSetup.Get();
                UseForecast := MfgSetup."Current Production Forecast";
            end;
...

    [IntegrationEvent(false, false)]
    local procedure OnBeforeOnOpenPage(var IsHandled: Boolean)
    begin
    end;

report 99001017 "Calculate Plan - Plan. Wksh."
...
        trigger OnOpenPage()
        var
            IsHandled: Boolean;
        begin
            IsHandled := false
            OnBeforeOnOpenPage(IsHandled);
            if not IsHandled then begin
                MfgSetup.Get();
                UseForecast := MfgSetup."Current Production Forecast";
                if MfgSetup."Combined MPS/MRP Calculation" then begin
                    MPS := true;
                    MRP := true;
                end else
                    MRP := not MPS;
            end;

            OnAfterOnOpenPage;
        end;
...

    [IntegrationEvent(false, false)]
    local procedure OnBeforeOnOpenPage(var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


code format and not skip javadoc

Ⅰ. Describe what this PR did
code format and not skip javadoc
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #378 into develop will not change coverage.
The diff coverage is 0%.

@@            Coverage Diff             @@
##             develop     #378   +/-   ##
==========================================
  Coverage      29.06%   29.06%
  Complexity       570      570
==========================================
  Files            165      165
  Lines           6588     6588
  Branches         786      786
==========================================
  Hits            1915     1915
  Misses          4469     4469
  Partials         204      204

Impacted Files
Coverage Δ
Complexity Δ

...e/protocol/transaction/BranchRegisterResponse.java
93.1% <ø> (ø)
7 <0> (ø)
:arrow_down:

...fescar/core/protocol/AbstractIdentifyResponse.java
44.89% <ø> (ø)
4 <0> (ø)
:arrow_down:

...libaba/fescar/tm/api/GlobalTransactionContext.java
76.92% <ø> (ø)
5 <0> (ø)
:arrow_down:

...m/alibaba/fescar/tm/DefaultTransactionManager.java
14.81% <ø> (ø)
2 <0> (ø)
:arrow_down:

.../rm/datasource/exec/BaseTransactionalExecutor.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../alibaba/fescar/rm/datasource/ConnectionProxy.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


[FR] Support module map files

Brief Issue Summary
Support exposing the contents of module maps created by CMake to LSP engines.
CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
CMake creates module map files which holds module mapping of translation units to used module files on disk. This information should be exposed to LSP engines without user intervention as examplified here @bobbrow mentioned that this part of the integration is the onus of CMakeTools, not CppTools.

Thanks for the issue, I've placed it in our backlog and we will prioritize it when we can based on bandwidth and demand for other issues. Thanks!


[Psy-X] Texture filter switch per polygon (using PSYX_* primitive)

The low resolution nature of the HUD and game text doesn't look great when filtered. Short of texture replacements or a nearest neighbour upscale, it would be nice to mark these textures as not needing to be filtered.

@The-King-of-Toasters Psy-X already supports Hi-res 32 bit texture replacement and it's made for font and number textures, fonts are in process. But PSX renderer itself should have the filtering switch - maybe some day.

No need in that yet.


Create spnttw

sv.txt

@address20001030 lib/domains/spnttw: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@address20001030 lib/domains/spnttw: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@address20001030 Regrettably, I have to close the request without merging, because you made a mistake in your request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


No compile instructions

I tried to make a feature via this starter.

Download the template
Make a new feature or change the ID of color or hello feature
Error Blob /tmp/features-output-1699945273175/devcontainer-feature-.......tgz does not exist.

Via this comment https://github.com/devcontainers/features/issues/752#issuecomment-1811349223 it seems I should also have compiled something but I don't see those instructions in this feature-starter readme. So please can we make it clearer?

Hey there!  This repo utilizes a GitHub action following these steps in the README for publishing.  We also provide this guide, although the steps are quite similar.   "Under the hood" this GitHub action invokes the devcontainer features publish command, which has the arguments as shown below
[~]$ devcontainer features publish --help
devcontainer features publish <target>

Package and publish Features

Positionals:
  target
           Package and publish features at provided [target] (default is cwd), where [target] is either:
              1. A path
           to the src folder of the collection with [1..n] features.
              2. A path to a single feature that contains a d
           evcontainer-feature.json.
                                                                                [string] [required] [default: "."]

Options:
      --help       Show help                                                                                [boolean]
      --version    Show version number                                                                         [boolean]
  -r, --registry   Name of the OCI registry.                                               [string] [default: "ghcr.io"]
  -n, --namespace  Unique indentifier for the collection of features. Example: <owner>/<repo>        [string] [required]
      --log-level  Log level.                                      [choices: "info", "debug", "trace"] [default: "info"]

Hope this is helpful! If you're still stuck, please provide step-by-step instructions so we can better assist you.


OSSM-3600 E2E tests for external prometheus

https://issues.redhat.com/browse/OSSM-3600
PR needs to deploy the SMCP, and user workload monitoring stack.
PR needs to  enable collecting traffic metrics.
PR needs to deploy bookinfo and generate some ingress traffic and check the Query Thanos and verify that metrics istio_requests_total exist.
For more details how to test this integration, look at this PR: https://github.com/maistra/istio-operator/pull/1094.

Closing in favor of #545.


Feature: option to sort coins based in stability (price change)

Assign me this issue under JWOC. @arnab2001

Would like to give it a try

Would like to give it a try

Let's do this together, I will fork your repo, will work together and you make the final PR.  What say? @TuhinBanerjee31 Could be a great leaning experience.

Yeah @pooranjoyb , sounds excellent and surely would be a great learning experience.

Sure , i can assign you both of that's fine

Go ahead , I'll be there if you guys need any help


[NNPA] Support ULP-based tolerance setting for backend test on NNPA by the TEST_ULPTOL environment.

Support ULP(Unit in the Last Place)-based tolerance setting for backend test on NNPA.
For example, export TTEST_ULPTOL="DFLOAT:10" specifies the following.
Tolerance = ref_output * RTOL + ATOL + ULP * ULPTOL
ULP is calculated for each reference output value with assuming the format "DFLOAT" (or "BFLOAT16").
Currently only "DFLOAT" is supported by this PR.
Signed-off-by: Yasushi Negishi<EMAIL_ADDRESS>

@jenkins-droid test this please

Let me know if this is ready for review

@negiyas let me know if it is ready to be reviewed.

I am afraid that this PR may not be used practically unfortunately, and I am not sure if I should close or merge it.
Can you give me your comments?

@negiyas please asses if this PR is still needed and either merge or close. Thanks

@negiyas if you believe there is a reasonable change that we might use it in the future, then push; otherwise just close it.

I am closing this PR, because it will not used probably.


fix copy url button, add deep link sanity check, make file icons solid

fixes https://github.com/nf-core/nf-co.re/pull/514#issuecomment-719806594


VS unit tests fail on master

After #1105, getting cursor positions moves to ThreadPool.
Setting cursor position might occur after the first TagChanged event, therefore, test assertions fail.
These unit tests need to be updated.

I've read it 10 times and still don't understand :) Could you point at problematic code?

The code is fine. The test infrastructure is problematic.
DocumentEventListener always invokes updateCursorPosition when a tagger is created. Before we check cursor positions on UI thread so the checks often occur after we set cursor positions on VS unit tests. Now, getting cursor positions and setting them are asynchronous, their order is non-deterministic.
I changed the unit tests to always set cursor positions very early in #1108.


Add support for imports of UTF-16 encoded text w/ Byte Order Mark (BOM)

Before this PR, we blindly assumed that all text imported into speedscope was UTF-8 encoded. This, unsurprisingly, is not always true. After this PR, we support text that's UTF-16 encoded, with either the little-endian or big-endian byte-order-mark.
Fixed #291

Coverage increased (+0.07%) to 47.178% when pulling e62112f2761c130091140c8789b6aa30a39828df on jlfwong/encoding into f3a1c09c9b21644f816b90035fbe198cc787eefd on master.


GKE cluster Terraform plan. Wrong CLI version. There is no 2.7 version.

Describe the bug
Update the required Azure CLI version. It says "Install or update Azure CLI to version 2.7" and to date the latest Azure CLI version is  2.16.0

Thank you @saffiali. It should be 2.7.0, i will fix it


Extract requests of current assignee from current project

User story

As a user
I want to see requests that belong to the current project
So that I could get an overview of the current project

Acceptance criteria

The method should be updated to extract issues from the current project. Those requests don't have to be shown in the calendar, only as a list for example.

Definition of done

Can be demoed
Fully tested with the unit test
Code well structured and formatted (hold to code conventions)
Clear explaination of code
No syntax/runtime errors
Codes are peerly reviewed
Communicate issues by leaving comments on github

Is a current project does not exist - Is that correct?

Group Decission: This task will be cancelled because lack of time.


Google Fonts support

Description
Being able to specify a font from Google Fonts, and have the compiler pull it from the internet if not already present.
https://developers.google.com/fonts/docs/developer_api
Use Case
Google Fonts is a large curated database of fonts, and being able to pull fonts from there would instantly give typst users access to thousands of fonts, without having to have them installed as system fonts.

It would also open up a huge privacy hole. You could mitigate this from the web side with a cache on your server side, but the CLI would leave lots of exposure. If you go down this road figure out something more generic for remote fonts such that alternative providers could be used and no network actions would be taken without the knowledge of somebody rendering on their own system.

It would also open up a huge privacy hole. You could mitigate this from the web side with a cache on your server side, but the CLI would leave lots of exposure. If you go down this road figure out something more generic for remote fonts such that alternative providers could be used and no network actions would be taken without the knowledge of somebody rendering on their own system.

Instead of fetching the respective font from Google Fonts during each compilation, Typst could instead download the font once and then include the .ttf file (or whatever) in the project source. There could be a dedicated .fonts folder where Typst would look for the desired font before downloading it.

Compiler-wise this is not desirable I think. Web-app wise we have our own fonts infrastructure, which also requires other metadata than Google fonts, privacy issues aside.


Readiness probe failure demo

Instructions a little incorrect, as we don't ship this alert out of the box. Can you either remove the screnshot and Robusta tie-in, or keep it but add instructions on applying the PrometheusRule?

Fixed it.


Application crash when use precompiled .lib with "AVX2" optimization.

It seems on legacy processors. box2d in pre-compiled .lib library format with "Advanced Vector Extensions 2 (AVX2)" optimization,
causes “Illegal Instruction” exception, and application freeze/crash.
This mostly happens when physical bodies collide, before the first collision everything works.

There is no pre compiled lib located here. And yes, you should compile box2d without AVX2 if your CPU doesn't support those instructions. I'm not sure what the problem is.


sudo pass: can it be defined in the deployment whether a sudo password is needed?

If not, maybe we can make it configurable in a user config file (e.g. match on path of deployment file and set some options for it)


Issue abot preprocessing.py

Hello @irdanish11 @junaid340
I got this error while preprocessing Avenue Dataset

Thank you

@junaid340 can you please have a look at the issue?


MessageMissingError when using predicates as macros and uuid_v4

Describe the bug
When implementing predicates as macros, and the uuid_v4? predicate, it appears there is not default error message like there is for the other macros.
To Reproduce
class ExampleContract < ApplicationContract
  params do
    required(:uuid).filled(:string)
  end

  rule(:uuid).validate(:uuid_v4?)
end

ExampleContract.new.call({ uuid: 'not a uuid' })

Dry::Validation::MissingMessageError:
       Message template for :uuid_v4? under "uuid" was not found
     # /usr/local/bundle/gems/dry-validation-1.4.0/lib/dry/validation/messages/resolver.rb:73:in `message'
     # /usr/local/bundle/gems/dry-validation-1.4.0/lib/dry/validation/messages/resolver.rb:34:in `block in call'
     # /usr/local/bundle/gems/dry-validation-1.4.0/lib/dry/validation/message.rb:55:in `evaluate'
     # /usr/local/bundle/gems/dry-validation-1.4.0/lib/dry/validation/message_set.rb:90:in `block in freeze'
     # /usr/local/bundle/gems/dry-validation-1.4.0/lib/dry/validation/message_set.rb:88:in `each'
     # /usr/local/bundle/gems/dry-validation-1.4.0/lib/dry/validation/message_set.rb:88:in `freeze'
     # /usr/local/bundle/gems/dry-validation-1.4.0/lib/dry/validation/result.rb:178:in `freeze'
     # /usr/local/bundle/gems/dry-validation-1.4.0/lib/dry/validation/result.rb:26:in `new'
     # /usr/local/bundle/gems/dry-validation-1.4.0/lib/dry/validation/contract.rb:94:in `call'

Expected behavior
I would expect that there be an error message like uuid is not a valid UUID. Some of the predicates have messages gteq? for example, others don't. I would expect consistency here in that all of the dry-logic built-in predicates have default error messages built-in.
Your environment

Affects my production application: NO
Ruby version: ruby 2.4.1
OS: macOS catalina

I've moved this to dry-schema because it's errors.yml is simply missing a default message for uuid_v4? predicate so it's a bug here.


Update to Bitcoin 0.17

Let's update and then we can have access to new PSBT RPC endpoints 😁

Hmm it looks like 0.17 was put in the releases page and put back in the tags. May have been accidental by the core repo maintainers

It's out now!


[Bug] relay.transform.AnnotateSpans does not work on modules imported from Pytorch

Expected behavior
AnnotateSpans should work on modules imported from Pytorch.
Actual behavior
It fails, because the PyTorch importer creates variables with "." in their names. AnnotateSpans works by pretty-printing and then re-parsing a module. Unfortunately, the parser (tokenizer?) chokes on variables with "." in the name.
This makes it very difficult to debug other transformations when working with PyTorch models.
Environment
Ubuntu LTS, latest TVM main, torch 1.8.0
Steps to reproduce
class Net(nn.Module):
    def __init__(self):
        super().__init__()
        self.dense = nn.Linear(4, 13)

    def forward(self, x):
        return self.dense(x)

net = Net()
data = torch.randn(17, 4)
traced = torch.jit.trace(net, data).eval()

mod, params = relay.frontend.from_pytorch(traced, [('x', tuple(data.shape))])
print(mod['main'].astext())
mod = relay.transform.AnnotateSpans()(mod)

Output:
#[version = "0.0.5"]
fn (%x: Tensor[(17, 4), float32], %dense.weight: Tensor[(13, 4), float32], %dense.bias: Tensor[(13), float32]) {
  %0 = transpose(%dense.weight, axes=[1, 0]);
  %1 = transpose(%0, axes=[1, 0]);
  %2 = nn.dense(%x, %1, units=13);
  add(%2, %dense.bias)
}
error: expected a local variable found `.`
 --> GeneratedSource:115:48
     |
 115 |  def @main(%x: Tensor[(17, 4), float32], %dense.weight: Tensor[(13, 4), float32], %dense.bias: Tensor[(13), float32]) {
     |                                                 ^

Unfortunately it's not totally trivial to fix this because tuples use "." for element access. Maybe the PyTorch importer / all variable creation should excise "."s from variable names, but I could see that breaking scripts that currently work.

Thanks I'll take a look.

There is a comment suggesting "x.y" should be parsed as an identifier ideally:
https://github.com/apache/tvm/blob/2c67d7131bc820481aaaca921f9e76f5709051b3/src/parser/parser.cc#L1547
So we should fix the tokenizer / parser

Oh in PT models we can have a variable name like %features.1.conv.1_bias. We cannot distinguish '1' there with tuple access...
I'll write a pass to replace "." with "_" in a variable name, and let users optionally run it after PT model import.

Thanks so much!


Allow indexing with symbols

TypeScript now has a ES6 target mode which includes definitions Symbol. However when trying to index an object with a symbol, I get an error (An index expression argument must be of type 'string', 'number', or 'any').
var theAnswer = Symbol('secret');
var obj = {};
obj[theAnswer] = 42; // Currently error, but should be allowed

That is part of the ES6 Symbol support we @JsonFreeman is working on. Your code sample should be supported in the next release.

@wereHamster, with pull request #1978, this should become legal, and obj[theAnswer] will have type any. Is that sufficient for what you are looking for, or do you need stronger typing?

Will it be possible to specify the type of properties which are indexed by symbols? Something like the following:
var theAnswer = Symbol('secret');
interface DeepThought {
   [theAnswer]: number;
}

Based on the comments in that PR, no:
This does not cover symbol indexers, which allows an object to act as a map with arbitrary symbol keys.

I think @wereHamster is talking about a stronger typing than @danquirk. There are 3 levels of support here. The most basic level is provided by my PR, but that is just for symbols that are properties of the global Symbol object, not user defined symbols. So,
var theAnswer = Symbol('secret');
interface DeepThought {
    [Symbol.toStringTag](): string; // Allowed
    [theAnswer]: number; // not allowed
}

The next level of support would be to allow a symbol indexer:
var theAnswer = Symbol('secret');
interface DeepThought {
   [s: symbol]: number;
}
var d: DeepThought;
d[theAnswer = 42; // Typed as number

This is on our radar, and can be implemented easily.
The strongest level is what you're asking for, which is something like:
var theAnswer = Symbol('secret');
var theQuestion = Symbol('secret');
interface DeepThought {
   [theQuestion]: string;
   [theAnswer]: number;
}
var d: DeepThought;
d[theQuesiton] = "why";
d[theAnswer] = 42;

This would be really nice, but so far we have not come up with a sensible design for it. It ultimately seems to hinge on making the type depend on the runtime value of these symbols. We will continue to think about it, as it is clearly a useful thing to do.
With my PR, you should at least be able to use a symbol to pull a value out of an object. It will be any, but you will no longer get an error.

@wereHamster I did a little writeup #2012 that you may be interested in.

I've merged request #1978, but I will leave this bug open, as it seems to ask for more than I provided with that change. However, with my change, the original error will go away.

@wereHamster can you post an update of what more you'd like to see happen here? Wasn't immediately clear to me what we have implemented vs what you posted

We would take a PR for this. @JsonFreeman can provide details on some of the issues that you might run into.

I actually think adding a symbol indexer would be pretty straightforward. It would work just like number and string, except that it wouldn't be compatible with either of them in assignability, type argument inference, etc. The main challenge is just making sure you remember to add logic in all the appropriate places.

@RyanCavanaugh, it would be nice to eventually have the last example in https://github.com/Microsoft/TypeScript/issues/1863#issuecomment-73668456 typecheck. But if you prefer you can split this issue up into multiple smaller issues which build on top of each other.

Was there any update on this front? AFAIU the latest version of the compiler supports only the first level described in https://github.com/Microsoft/TypeScript/issues/1863#issuecomment-73668456.

We would be happy to accept PRs for this change.

It might be worth tracking the two levels as two separate issues. The indexers seem fairly straightforward, but the utility is not clear. The full support with constant tracking seems quite difficult, but probably more useful.

Constant tracking is already tracked in https://github.com/Microsoft/TypeScript/issues/5579. this issue is for adding support for a symbol indexer, similar to string and numberic indexers.

Got it, makes sense.

@JsonFreeman @mhegazy an issue is available at #12932

Just thought I'd throw my use case into the ring. I'm writing a tool that allows queries to be described by specifying plain text keys for matching against arbitrary object properties, and symbols for specifying matching operators. By using symbols for well-known operators, I avoid the ambiguity of matching an operator versus a field whose key is the same as that of the well-known operator.
Because symbols can't be specified as index keys, in contrast to what JavaScript explicitly allows, I'm forced to cast to <any> in a number of places, which degrades the code quality.
interface Query {
  [key: string|symbol]: any;
}

const Q = {
  startsWith: Symbol('startsWith'),
  gte: Symbol('gte'),
  lte: Symbol('lte')
}

const sample: Query = {
  name: {
    [Q.startsWith]: 'M',
    length: {
      [Q.lte]: 25
    }
  },
  age: {
    [Q.gte]: 18
  }
};

Use of "unlikely" first characters such as a $ character is not a suitable compromise, given the variety of data that the query engine may need to inspect.

Hi guys. Is there any movement in this? I need it so I would be willing to contribute the necessary changes. Haven't contributed to TS before though.

@mhegazy @RyanCavanaugh I know you guys are incredibly busy, but could you weigh in when you get a chance? Symbols are a really important tool for architecting libraries and frameworks, and the lack of ability to use them in interfaces as a definite pain point.

I'm asking is there anything in progress? Sincerely hope this feature be supported.

https://github.com/Microsoft/TypeScript/pull/15473 looks related.

Yeah still looking for this today, this is what I see in Webstorm:

That actually works
var test: symbol = Symbol();

const x = {
    [test]: 1
};

x[test];

console.log(x[test]);

console.log(x['test']);

but the type of x is not right, being inferred as
{
  [key: string]: number
}

Yeah still looking for this today, this is what I see in Webstorm:

Please note that JetBrains' own language service that is enabled by default in WebStorm, intelliJ IDEA, and so on.

This works in TS 2.7
const key = Symbol('key')
const a: { [key]?: number } = {}
a[key] = 5

Any update on this?
My problem:
export interface Dict<T> {
  [index: string]: T;

  [index: number]: T;
}

const keyMap: Dict<number> = {};

function set<T extends object>(index: keyof T) {
  keyMap[index] = 1; // Error Type 'keyof T' cannot be used to index type 'Dict<number>'
}

But this also does not work, because symbol cannot be an index type.
export interface Dict<T> {
  [index: string]: T;
  [index: symbol]: T; // Error: An index signature parameter type must be 'string' or 'number'
  [index: number]: T;
}

Expected behavior:
symbol should be a valid index type
Actual behavior:
symbol is not a valid index type
Using the workaround with casting as string | number seems for me very bad.

How is util.promisify.custom supposed to be used in TypeScript? It seems that using constant symbols is supported now, but only if they are explicitly defined. So this is valid TypeScript (aside from f not being initialized):
const custom = Symbol()
interface PromisifyCustom<T, TResult> extends Function {
	[custom](param: T): Promise<TResult>
}
const f: PromisifyCustom<string, void>
f[custom] = str => Promise.resolve()

But if promisify.custom is used instead of custom, the attempt to reference f[promisify.custom] results in the error Element implicitly has an 'any' type because type 'PromisifyCustom<string, void>' has no index signature.:
import {promisify} from 'util'
interface PromisifyCustom<T, TResult> extends Function {
	[promisify.custom](param: T): Promise<TResult>
}
const f: PromisifyCustom<string, void>
f[promisify.custom] = str => Promise.resolve()

I would like to assign to a function's promisify.custom field, but it seems (given the behavior described above) that the only way to do this is to cast the function to an any type.

I cannot understand why symbol is not allowed as key index, below code should works and is accepted by Typescript 2.8 but it is not allowed by Typescript 2.9
/**
 * Key can only be number, string or symbol
 * */
export class SimpleMapMap<K extends PropertyKey, V> {
  private o: { [k: string ]: V } = {};

  public has (k: K): boolean {
    return k in this.o;
  }

  public get (k: K): V {
    return this.o[k as PropertyKey];
  }

  public set (k: K, v: V) {
    this.o[k as PropertyKey] = v;
  }

  public getMap (k: K): V {
    if (k in this.o) {
      return this.o[k as PropertyKey];
    }
    const res = new SimpleMapMap<K, V>();
    this.o[k as PropertyKey] = res as any as V;
    return res as any as V;
  }

  public clear () {
    this.o = {};
  }
}

I tried below, which is more 'correct' to me but it is not accepted by both version of Typescript compiler
/**
 * Key can only be number, string or symbol
 * */
export class SimpleMapMap<K extends PropertyKey, V> {
  private o: { [k: K ]: V } = {};

  public has (k: K): boolean {
    return k in this.o;
  }

  public get (k: K): V {
    return this.o[k];
  }

  public set (k: K, v: V) {
    this.o[k] = v;
  }

  public getMap (k: K): V {
    if (k in this.o) {
      return this.o[k];
    }
    const res = new SimpleMapMap<K, V>();
    this.o[k as PropertyKey] = res as any as V;
    return res as any as V;
  }

  public clear () {
    this.o = {};
  }
}

The status of this ticket indicates that what you are suggesting is the desired behaviour, but the core team is not at this point committing resources to add this feature enhancement, they are opening it up to the community to address.

@beenotung Although this is not an ideal solution, assuming that the class you posted is the only place where you need such a behaviour, you can do unsafe casts inside the class, but keeping the class and methods signatures the same, so that the consumers of the class won't see that:
/**
 * Key can only be number, string or symbol
 * */
export class SimpleMapMap<K extends PropertyKey, V> {
  private o: { [k: string]: V } = {};

  public has(k: K): boolean {
    return k in this.o;
  }

  public get(k: K): V {
    return this.o[k as any];
  }

  public set(k: K, v: V) {
    this.o[k as any] = v;
  }

  public getMap(k: K): V {
    if (k in this.o) {
  	return this.o[k as any];
    }

    const res = new SimpleMapMap<K, V>();
    this.o[k as any] = res as any as V;
    return res as any as V;
  }

  public clear() {
    this.o = {};
  }
}

Because the signatures are the same, whenever you use this class, you will have the type validation applied correctly, and, when this issue is solved, you will just need to change this class (it will be transparent to the consumers).
An example of a consumer is like bellow (the code won't need any change when this issue is fixed):
const s1 = Symbol(1);
const s2 = Symbol(2);

let m = new SimpleMapMap<symbol, number>()
m.set(s1, 1);
m.set(s2, 2);
m.get(s1);
m.get(1); //error

Yeah I have been suffering with this one for awhile sadly, my latest question regarding this topic:
https://stackoverflow.com/questions/53404675/ts2538-type-unique-symbol-cannot-be-used-as-an-index-type

@jhpratt there's a PR up at #26797 (note the caveat about well-known symbols). There are recent design meeting notes about it in #28581 (but no resolution recorded there). There's a bit more feedback about why that PR is held up here. It seems to be regarded as a fringe/low impact issue, so maybe more upvotes on the PR may help raise the profile of the issue.

Thanks @yortus. I've just asked Ryan if the PR is still planned for 3.2, which is what is indicated by the milestone. Hopefully that's the case, and this will be resolved!

The PR pointed by @yortus seems a huge change,
Shouldn't the fix for this bug be very minor? e.g. adding an or statement in the condition check.
(I have not located the place to change yet.)

+1

temp solution here https://github.com/Microsoft/TypeScript/issues/24587#issuecomment-412287117, kinda ugly but gets the job done
i guess one of the gotchas i've noticed in using symbols though is if you have a project involving the child_process module, yes you can share types/enums/interfaces between the two process but never symbols.

Don't know, if anyone has started something to fix this, but if not, I have now.
My time is limited, though and I have zero knowledge about the Typescript sources. I've made a fork, but no pull request, yet, because I don't want to molest the devs with unfinished (?) changes. I would ask everyone to contribute what they can to my fork. I will eventually issue a PR then.
So far I have found a way to fix the interface index type restriction. I don't know, if there's more to it. I was able to index an object with a symbol in TS 3.4 without any fixes. (https://www.typescriptlang.org/play/#src=const o %3D {}%3B
const s %3D Symbol('s')%3B

o[s] %3D 123%3B)
Have a look at my commit to see what I found: https://github.com/Neonit/TypeScript-SymbolKeys/commit/11cb7c13c2494ff32cdec2d4f82673058c825dc3
Missing:

Tests: I haven't had time to look into how tests in TS are organized and structured.
Localization: The diagnostic message has only been updated for the English variant. Maybe the other languages get the old message still. I don't know. I could only provide a German translation anyway.

I hope this will get things started finally after years of waiting.

the fix looks good. can TypeScript dev take a look on it?

hello, any progress for this?

Just opened a SO thread about this: https://stackoverflow.com/questions/59118271/using-symbol-as-object-key-type-in-typescript
Why isn't that possible? Isn't symbol another primitive type like number - so why's there a difference?

Hello, any progress for this?
FIVE years have been passed!

You're not gonna believe how long it took C++ to get closures 😲

lol fair, but C++ isn't marketing itself as a superset of a language that has closures :-p

@ljharb keep beating that horse, it's still twitching 😛

For those who are targeting newer runtimes, why not use a Map? I've anecdotally found that a lot of developers don't know Maps exist, so I'm curious whether there's another scenario that I'm missing.
let m = new Map<symbol, number>();
let s = Symbol("arbitrary symbol!");

m.set(s, 1000);
let a = m.get(s);

Maps and objects have different use cases.

@DanielRosenwasser well-known symbols are used as protocols; a Map key of Symbol.match, for example, won't make the object RegExp-like (and any object may want a Symbol.iterable key to make it iterable without having to explicitly use TS built-in Iterable types).

@aaronpowell the error appear when you enable 'strict' flag in the 'compilerOptions' in tsconfig.json.
So under the current version of typescript compiler, you've to either turn off the strict mode or cast the target into any ...

Sure, but an any cast isn't really ideal and disabling strict mode is just loosening restrictions in the type safety.

Reading messages i imagine the next "solution" will probably be to "disable typescript".
We shouldn't have to search for stopgap solutions neither have to explain why we need it.
It's a standard feature of javascript so we need it in typescript.

@DanielRosenwasser my use case is similar to that of @aaronpowell - a seeming mismatch in the ProxyHandler interface and TypeScript's rules preventing me from properly typing proxy handler traps.
A boiled down example demonstrating the issue:
const getValue = (target: object, prop: PropertyKey) => target[prop]; // Error

As far as I can tell, it's impossible to craft any type for target that will avert the error yet allow only objects that can legitimately be accessed by PropertyKey.
I'm a TypeScript newbie so please forgive me if I'm missing something obvious.

Another use case: I'm trying to have a type {[tag: symbol]: SomeSpecificType} for callers to provide a map of tagged values of a specific type in a way that benefits from the compactness of object literal syntax (while still avoiding the name clash risks of using plain strings as tags).

Another use case: I'm trying to iterate all enumerable properties of an object, symbols and strings both. My current code looks something like this (names obscured):
type ContextKeyMap = Record<PropertyKey, ContextKeyValue>

function setFromObject(context: Context, object: ContextKeyMap) {
    for (const key in object) {
        if (hasOwn.call(object, key)) context.setKey(key, object[key])
    }

    for (const symbol of Object.getOwnPropertySymbols(object)) {
        if (propertyIsEnumerable.call(object, symbol)) {
            context.setKey(symbol, object[symbol as unknown as string])
        }
    }
}

I'd very strongly prefer to be able to just do this:
type ContextKeyMap = Record<PropertyKey, ContextKeyValue>

function setFromObject(context: Context, object: ContextKeyMap) {
    for (const key in object) {
        if (hasOwn.call(object, key)) context.setKey(key, object[key])
    }

    for (const symbol of Object.getOwnPropertySymbols(object)) {
        if (propertyIsEnumerable.call(object, symbol)) {
            context.setKey(symbol, object[symbol])
        }
    }
}

I have also issue with indexing with symbols. My code is as follow:
const cacheProp = Symbol.for('[memoize]')

function ensureCache<T extends any>(target: T, reset = false): { [key in keyof T]?: Map<any, any> } {
  if (reset || !target[cacheProp]) {
    Object.defineProperty(target, cacheProp, {
      value: Object.create(null),
      configurable: true,
    })
  }
  return target[cacheProp]
}

I followed the solution by @aaronpowell and somehow managed to workaround it
const cacheProp = Symbol.for('[memoize]') as any

function ensureCache<T extends Object & { [key: string]: any}>(target: T, reset = false): { [key in keyof T]?: Map<any, any> } {
  if (reset || !target[cacheProp]) {
    Object.defineProperty(target, cacheProp, {
      value: Object.create(null),
      configurable: true,
    })
  }

  return target[cacheProp]
}

Casting to any from symbol isn't so nice indeed.
Really appreciated for any other solutions.

@ahnpnl For that use case, you'd be better off using a WeakMap than symbols, and engines would optimize that better - it doesn't modify target's type map. You may still have to cast it, but your cast would live in the return value.

A workaround is to use Record<Key, Value>.
The key can be of any type you prefer.

A workaround is to use generic function to assign value ...
var theAnswer: symbol = Symbol('secret');
var obj = {} as Record<symbol, number>;
obj[theAnswer] = 42; // Currently error, but should be allowed

Object.assign(obj, {theAnswer: 42}) // allowed

A workaround is to use generic function to assign value ...
var theAnswer: symbol = Symbol('secret');
var obj = {} as Record<symbol, number>;
obj[theAnswer] = 42; // Currently error, but should be allowed

Object.assign(obj, {theAnswer: 42}) // allowed

I'm not agree. These three lines are equal to each other:
Object.assign(obj, {theAnswer: 42});
Object.assign(obj, {'theAnswer': 42});
obj['theAnswer'] = 42;

@DanielRosenwasser
I have this use case, In the playground link, I also solved it by using maps, but take a look, its ugly.
const system = Symbol('system');
const SomeSytePlugin = Symbol('SomeSytePlugin')

/** I would prefer to have this working in TS */
interface Plugs {
    [key: symbol]: (...args: any) => unknown;
}
const plugins = {
    "user": {} as Plugs,
    [system]: {} as Plugs
}
plugins[system][SomeSytePlugin] = () => console.log('awsome')
plugins[system][SomeSytePlugin](); ....

Playground Link
Using symbols here rules out the possible accidental overwrite that happens when using strings. It makes the whole system more robust and easier to maintain.
If you have an alternative solution that works with TS and has the same readability in the code, I'm all ears.

Any official explain for this issue?

A workaround is to use generic function to assign value ...
var theAnswer: symbol = Symbol('secret');
var obj = {} as Record<symbol, number>;
obj[theAnswer] = 42; // Currently error, but should be allowed

Object.assign(obj, {theAnswer: 42}) // allowed

You're looking for
Objet.assign(obj, { [theAnswer]: 42 });

However there isn't a way to read x[theAnswer] back out without a cast AFAIK

For the love of God, please make this a priority.

You're looking for
Objet.assign(obj, { [theAnswer]: 42 });

However there isn't a way to read x[theAnswer] back out without a cast AFAIK

As pointed out by mellonis and MingweiSamuel, the workarounds using generic function are:
var theAnswer: symbol = Symbol("secret");
var obj = {} as Record<symbol, number>;

obj[theAnswer] = 42; // Not allowed, but should be allowed

Object.assign(obj, { [theAnswer]: 42 }); // allowed

function get<T, K extends keyof T>(object: T, key: K): T[K] {
  return object[key];
}

var value = obj[theAnswer]; // Not allowed, but should be allowed

var value = get(obj, theAnswer); // allowed

Found a work-around on this case, it not generic but work in some case:
const SYMKEY = Symbol.for('my-key');

interface MyObject {   // Original object interface
  key: string
}

interface MyObjectExtended extends MyObject {
  [SYMKEY]?: string
}

const myObj: MyObject = {
  'key': 'value'
}

// myObj[SYMKEY] = '???' // Not allowed

function getValue(obj: MyObjectExtended, key: keyof MyObjectExtended): any {
  return obj[key];
}

function setValue(obj: MyObjectExtended, key: keyof MyObjectExtended, value: any): void {
  obj[key] = value
}

setValue(myObj, SYMKEY, 'Hello world');
console.log(getValue(myObj, SYMKEY));

@james4388 How is your example any different from the one from @beenotung?

FYI: https://github.com/microsoft/TypeScript/pull/26797
(Just found it - I'm not actually part of the TS team.)

Can someone tell me how it makes sense that this is legit:
const keepAs = <
  R extends Record<string, unknown>,
  From extends keyof R,
  M extends Record<string, From>,
  To extends keyof M,
  S extends { [P in To]: R[M[P]] }
>(
  a: R[],
  m: M,
): S[] => a.map((r) => Object.assign({}, ...Object.entries(m).map(([k, v]) => ({ [k]: r[v] }))));

but the analogous
const keepAs2 = <
  R extends Record<PropertyKey, unknown>,
  From extends keyof R,
  M extends Record<PropertyKey, From>,
  To extends keyof M,
  S extends { [P in To]: R[M[P]] }
>(
  a: R[],
  m: M,
): S[] => a.map((r) => Object.assign({}, ...Reflect.ownKeys(m).map((k) => ({ [k]: r[m[k]] }))));

isn't?
Feels like, [2015] here's the final standard; Symbols make great keys! No name collision! Etc!
Typescript [2020]: wanna be modern and use not just ES3 types? Are you in for a surprise, buddy ¯_(ツ)_/¯

My poor workaround for some cases :
const bar: Record<any, string> = {};
const FOO = Symbol('foo');

// eslint-disable-next-line @typescript-eslint/no-explicit-any
const aFOO = FOO as any;

bar[aFOO] = 'sad';

My poor workaround for some cases :
const bar: Record<any, string> = {};
const FOO = Symbol('foo');

// eslint-disable-next-line @typescript-eslint/no-explicit-any
const aFOO = FOO as any;

bar[aFOO] = 'sad';

@Gnucki It’s probably better to do, since the as any type assertion gets removed during compilation:
const bar: Record<any, string> = {};
const FOO = Symbol('foo');

bar[
	// eslint-disable-next-line @typescript-eslint/no-explicit-any
	FOO as any
] = 'sad';

which compiles to:
const bar = {};
const FOO = Symbol('foo');

bar[FOO] = 'sad';

whereas your code compiles to:
const bar = {};
const FOO = Symbol('foo');

const aFoo = FOO as any;

bar[aFOO] = 'sad';

@Gnucki It’s probably better to do, since the as any type assertion gets removed during compilation:
const bar: Record<any, string> = {};
const FOO = Symbol('foo');

bar[
	// eslint-disable-next-line @typescript-eslint/no-explicit-any
	FOO as any
] = 'sad';

which compiles to:
const bar = {};
const FOO = Symbol('foo');

bar[FOO] = 'sad';

whereas your code compiles to:
const bar = {};
const FOO = Symbol('foo');

const aFoo = FOO as any;

bar[aFOO] = 'sad';

Is there any explanation of this from typescript maintainers team?
I can't see any reason why this is not handled.
This is very useful when creating 3 party libraries.
Without this fix we'll have to use @ts-ignore comments all over the place

I'll reiterate my suggestion to make a TS language-level distinction between user-instantiated symbols and 'well-known' JS symbols.
Quoting myself from earlier:

Well-known symbols are just sentinels that happen to be implemented using Symbol. Unless the goal is object serialization or introspection, code probably should treat these sentinels different from other symbols, because they have special meaning to the language. Removing them from the symbol type will likely make (most) code using 'generic' symbols more safe.

@riggs that would not fit a number of use cases; it's absolutely critical that everything for which typeof x === 'symbol' is in the symbol type, otherwise APIs can't be typed properly.

Given that TypeScript is a superset of JS created for the explicit purpose of type safety, create two new types, LanguageSymbol and Sentinel:

LanguageSymbol is the union of the 'well-known' static properties on Symbol.
Sentinel is every Symbol that isn't a LanguageSymbol.

Symbol remains untouched, but maybe gets a flag warning about direct usage in a future version.
Sentinels are, by definition, freed from the concerns relating to well-known symbols. Preserving this behavior at runtime does incur a small overhead, but would likely only occur when doing introspection, at which point extra care is likely warranted. I suspect most code doing runtime type checks for symbols are expecting to not be dealing with a LanguageSymbol in the first place.

Nit: TypeScript is not a superset of JS, and nor it is a programming language; it specifies little in the way of behaviors. One can't program in "TS 4.2". One programs in eg. ES2015, with TS 4.2 specified type annotation syntax and ES2015 types. Novice programmers can't learn to "code in TS". They learn to code in JS, with type annotations. No TS spec says anything about what [].map() does, and it can't, as the runtime is JS and what the [].map() does depends on the ES version, not on the TS version.
TS is more of a type annotation overlay to assist code linting, which also happens to add minor shorthands to the JS syntax via transpilation, and in practice, remove significantly from JS capabilities too (because it makes certain patterns very hard or impossible to properly type annotate; eg. it allows extra properties). TS does not live independently of JS, it has no language spec. And sure, there's an associated toolchain with source code transformation (mostly, removes the type notations), linter, IDE plugins etc.
So it's best to think of TS

a linter friendly type notation syntax, with
rather partial type libraries for constructs of ES versions, and
a toolchain consisting of linters, pessimizing transpilers (slow, unneeded polyfills) etc., plus
some macros for localized JS syntax augmentation (eg. in class),
which altogether constrain the coding style to the subset of JS patterns that are somewhat conducive to the otherwise not well specified, and not particularly consistent so-called structural typing concepts
associated with TypeScript versions, which bundle specifics of the above things, but still isn't a superset of JS

We're working on an implementation, but the conversation here isn't useful nor is it contributing anything to the implementation.

Implemented in #44512 which is now in the main branch.


Fix the bug introduced in #5132 where ScanBasedDocIdIterator.isMatch() is still required

ScanBasedDocIdIterator.isMatch() is still required when the query is in the following format:
SELECT ... WHERE (filterA OR filterB) AND filterC
And filterC is working on a scan-based column (column without inverted index)
Enhance the HybridClusterIntegrationTest to catch this issue

Codecov Report

Merging #5328 into master will decrease coverage by 9.39%.
The diff coverage is 1.78%.

@@            Coverage Diff             @@
##           master    #5328      +/-   ##
==========================================
- Coverage   66.21%   56.82%   -9.40%
==========================================
  Files        1072     1072
  Lines       54552    54578      +26
  Branches     8137     8141       +4
==========================================
- Hits        36123    31012    -5111
- Misses      15801    21130    +5329
+ Partials     2628     2436     -192

Impacted Files
Coverage Δ

...core/operator/dociditerators/AndDocIdIterator.java
54.38% <0.00%> (-10.32%)
:arrow_down:

...or/dociditerators/ExpressionScanDocIdIterator.java
0.00% <0.00%> (-49.70%)
:arrow_down:

...e/operator/dociditerators/SVScanDocIdIterator.java
49.42% <0.00%> (-20.09%)
:arrow_down:

...e/operator/dociditerators/MVScanDocIdIterator.java
40.62% <14.28%> (-28.87%)
:arrow_down:

...a/org/apache/pinot/minion/metrics/MinionMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/BrokerQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/minion/metrics/MinionQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/core/query/reduce/ComparisonFunction.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...pinot/minion/exception/TaskCancelledException.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ot/minion/events/EventObserverFactoryRegistry.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 323 more

Continue to review full report at Codecov.

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


C# intellisense

C++ glue project works with Visual studio intellisense, but C#  does not work - no 'ImGUI' class having controls detected, so cannot see exact invoke convention. Somehow Cooking is working, but code editing in C# environment is far inferior than C++ situation.
Can intellisense be available for C# script?

Make sure you configuration is set to Editor.Development and the architecture is set to Win64.


OTP: GenServer, Application, and Supervisor

Now that there is an API that mimics the Erlang processes (I put it that way in case the underlying implementation changes), We need to be able to do something with them.
We don't need to bring full-on OTP here, but we need enough to support the Elixir API. Most notably GenServer, Application, and Supervisor

I think I have a possible way forward. I managed to start working on a GenServer implementation at the processes repo. Specifically you can see it here: https://github.com/bryanjos/processes/blob/master/src/test.js

Closing this. Should be figured out as part of #99 and/ or #154


Removed SharedPreferences migration

After release, we no longer need the migration from SharedPreferences.

 Remove Shared Preferences migration


Are all "extras" installed by default?

What is the default behaviour? Will extras install when not explicitly requested during pip install?
I am working on setup.py for package, which has optional dependency on OpenCV for webcam support, but it can work even without webcam.
Problem is that on ARM systems, there is no OpenCV package available in PyPI. So i would like my package to install without it. I've understood, that i can add it to extras_require={}, but i am not sure how this will work.
When people do pip install mypackage i want webcam support to install if available, but to be silently ignored if not available. But i am afraid that it will install only if people will do pip install mypackage[webcam].
What is the default behaviour? Will extras install when not explicitly requested during pip install?

I don't think you want extras_require. That is always required to be explicit, and as far as I know there's no way to make it silently fail.

When people do pip install mypackage i want webcam support to install if available, but to be silently ignored if not available.

If you really want this, you probably need to do some architecture detection in setup.py and conditionally add it to the install_requires list. And I wouldn't be totally silent. I'd warn when you detect that you can't install webcam, and warn again in the inevitable try/except thing you use to feature-gate your webcam support:
try:
    import cv2
except ImportError:
   warnings.warn("OpenCV not installed: webcam support disabled")

...or whatever you do to detect it.

try: import cv2

We already do this. But people complain that they cannot install out app on raspberry pi, because opencv-python can't be installed by pip install on ARM.

As far as I know there is no real way to do this, and it's a shame. You could possibly do some funkiness with setup.py, but that will obviously not work if you provide any binary distribution.
It's also not something we can fix in setuptools for the same reason. setuptools is a build backend and as such can only create the package and the metadata that goes with it. We need to fix the standardized dependency metadata to allow for "soft-failing" dependencies and several other unconventional dependencies as well (including the unfortunately ill-fated "default extras_require: #1139).
For the moment, I think you have the following options:

If possible, use environment markers to remove the opencv dependency for any platform that can't handle it.
Add an option to setup.py that removes the cv2 requirement, users on ARM would need to then do pip install mypkg --install-option="--no-cv2" or something to this effect. If possible, I would use environment markers and provide this option as an "escape hatch" in case your environment markers don't work well
Make cv2 an optional dependency for everyone, and everyone has to opt-in by installing mypkg[cv2]. This is the simplest but unfortunately gives the wrong default.
Do not release a binary distribution for platforms that do not support opencv (or don't release a binary distribution at all), and do some sort of hack in setup.py to detect whether opencv can be installed on your platform.

As for the correct solution, I suggest we take that up on the packaging problems repository, because this is not a problem that can be solved in setuptools alone. I've created an issue there, pypa/packaging-problems#214.


Installing through brew gets an error

brew install neilotoole/sq/sq
Cloning into '/opt/homebrew/Library/Taps/neilotoole/homebrew-sq'...
remote: Enumerating objects: 164, done.
remote: Counting objects: 100% (164/164), done.
remote: Compressing objects: 100% (119/119), done.
remote: Total 164 (delta 74), reused 18 (delta 4), pack-reused 0
Receiving objects: 100% (164/164), 17.71 KiB | 4.43 MiB/s, done.
Resolving deltas: 100% (74/74), done.
Error: Invalid formula: /opt/homebrew/Library/Taps/neilotoole/homebrew-sq/Formula/sq.rb
sq: wrong number of arguments (given 1, expected 0)
Error: Cannot tap neilotoole/sq: invalid syntax in tap!

Still seeing this error..

==> Tapping neilotoole/sq
Cloning into '/opt/homebrew/Library/Taps/neilotoole/homebrew-sq'...
remote: Enumerating objects: 164, done.
remote: Counting objects: 100% (164/164), done.
remote: Compressing objects: 100% (119/119), done.
remote: Total 164 (delta 74), reused 18 (delta 4), pack-reused 0
Receiving objects: 100% (164/164), 17.71 KiB | 2.95 MiB/s, done.
Resolving deltas: 100% (74/74), done.
Error: Invalid formula: /opt/homebrew/Library/Taps/neilotoole/homebrew-sq/Formula/sq.rb
sq: wrong number of arguments (given 1, expected 0)
Error: Cannot tap neilotoole/sq: invalid syntax in tap!

Working again!


[Bug]: Unreliable returns in JS function

Is there an existing issue for this?

[X] I have searched the existing issues

Description
I have a simple js function that returns a value:
myFun1: () => { 			return 1; 	}
When I try to run the function, in the first try, there is no return value. In the second try it works. If I try to change "1" to "2", sometimes the run works and sometimes not.

conversation here: https://theappsmith.slack.com/archives/C02K0SZQ7V3/p1652951480887279
Steps To Reproduce
explained above
Public Sample App
No response
Version
Cloud

Duplicate of #13512


Q: Async serilog sink and performance

Hiya,
Quick question - i'm seeing some locking happen in my app and it seems to be doing with logging.
I use the following sinks:

File
Console
MSSQL

I want to move the sinks into Serilog.Async and to keep it consistent i'm considering moving all sinks into the async package.
I ofcourse could also only move File and Console to the async serilog sink and leave the MSSQL sink alone
Just a quick peer-check:
Would there be any significant performance degradations in doing this?
Would it be better to not put the MSSQL sink in the Async sink or would it not matter much?
Reading the docs:

If you initialize the sink with WriteTo then it uses a batched sink semantics. This means that it does not directly issue an SQL command to the database for each log call, but it collectes log events in a buffer and later asynchronously writes a bulk of them to the database using SqlBulkCopy.

I'd say there would be some overhead (given the double queueing but nothing major.

Hi  @sommmen!
Sorry for the delay in response. Like the documentation states, if you use the MSSQL sink in normal mode (WriteTo()) then the log operation just writes into a buffer which is asynchronously written to the DB in batches in the background. I guess there would be no performance or resposibility gain in your app if you call that asynchronously. On the other hand doing so would not be a big problem either. Check out the code starting here for details: https://github.com/serilog-mssql/serilog-sinks-mssqlserver/blob/5e9dc3a5a470a35292dc80ab612c1eaac2c84fe6/src/Serilog.Sinks.MSSqlServer/Sinks/MSSqlServer/MSSqlServerSink.cs#L125
If you use the MSSQL sink in audit mode (AuditTo()) then each log call is written to the database directly without any buffering or delay in the sink. If you call this async your app would continue executing while the log message is written to the DB but this would also defeat the purpose of the audit sink.
I hope this helps.

Hi @sommmen!
Sorry for the delay in response. Like the documentation states, if you use the MSSQL sink in normal mode (WriteTo()) then the log operation just writes into a buffer which is asynchronously written to the DB in batches in the background. I guess there would be no performance or resposibility gain in your app if you call that asynchronously. On the other hand doing so would not be a big problem either. Check out the code starting here for details:
https://github.com/serilog-mssql/serilog-sinks-mssqlserver/blob/5e9dc3a5a470a35292dc80ab612c1eaac2c84fe6/src/Serilog.Sinks.MSSqlServer/Sinks/MSSqlServer/MSSqlServerSink.cs#L125
If you use the MSSQL sink in audit mode (AuditTo()) then each log call is written to the database directly without any buffering or delay in the sink. If you call this async your app would continue executing while the log message is written to the DB but this would also defeat the purpose of the audit sink.
I hope this helps.

Hello @ckadluba,
Thanks for your answer - i'll be able to make an informed decision now.


[bionic 18.04] robotpkg-osg-dae and robotpkg-ur5-description does not exist

Hello everyone,
Im posting this issue because i'm trying to view my robot on gepetto-gui. I get an error which says that it is impossible to read .dae files and the latter should be converted in .osg. I tried to install robotpkg-osg-dae, but this package doesn't exist in robotpkg bionic (I checked on this link : http://robotpkg.openrobots.org/packages/debian/pub/dists/bionic/robotpkg/binary-amd64/). I had the same issue for installing robotpkg-ur5-description.
To avoid this problem, I added the xenial version of robotpkg (instead of bionic, which is my ubuntu version) and then tried to install both packages, using these commands :

sudo sh -c "echo 'deb [arch=amd64] http://robotpkg.openrobots.org/packages/debian/pub xenial robotpkg' >> /etc/apt/sources.list.d/robotpkg.list"

curl http://robotpkg.openrobots.org/packages/debian/robotpkg.key | sudo apt-key add -

sudo apt-get update

sudo apt install robotpkg-ur5-description

sudo apt install robotpkg-osg-dae

This works fine for the ur5-description package which gets installed. But, the robotpkg-osg-dae can't get installed, I get the following error :
_Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
robotpkg-osg-dae : Depends: libboost-filesystem1.58-dev (>= 1.34.1) but it is not installable
Depends: libboost-filesystem1.58.0 (>= 1.34.1) but it is not installable
Depends: libboost-system1.58-dev (>= 1.34.1) but it is not installable
Depends: libboost-system1.58.0 (>= 1.34.1) but it is not installable
E: Unable to correct problems, you have held broken packages._
After that, I installed libboost-all-dev, but it actually changes nothing. Thank you for your feedback

A quick answer to your issue is that you have to add robotpkg-wip to the list of known servers. See http://robotpkg.openrobots.org/robotpkg-wip.html for further details.
PS: for information, wip packages are not totally stable but almost ready for production.
@nim65s We should move osg-dae to the main robotpkg repository I think.

Thank you very much, it works fine with wip :)
I followed the instructions there : https://github.com/stack-of-tasks/pinocchio/issues/705#issuecomment-467835859

Great. I will close this issue then.

libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
List of GroupNodes :
gepetto-gui
hpp-gui
python-pinocchio
BOX does not exist.: iostream error
BOX does not exist.: iostream error
BOX does not exist.: iostream error
BOX does not exist.: iostream error
Node "world/pinocchio/collisions/ee_link_0" does not exist.
Node "world/pinocchio/collisions/tool0_0" does not exist.
Node "world/pinocchio/collisions/tool0_1" does not exist.
Node "world/pinocchio/collisions/tool0_2" does not exist.
You must initialize a Geode on world/pinocchio/visuals/base_link_0 to use Alpha
You must initialize a Geode on world/pinocchio/visuals/shoulder_link_0 to use Alpha
You must initialize a Geode on world/pinocchio/visuals/upper_arm_link_0 to use Alpha
You must initialize a Geode on world/pinocchio/visuals/forearm_link_0 to use Alpha
You must initialize a Geode on world/pinocchio/visuals/wrist_1_link_0 to use Alpha
You must initialize a Geode on world/pinocchio/visuals/wrist_2_link_0 to use Alpha
You must initialize a Geode on world/pinocchio/visuals/wrist_3_link_0 to use Alpha
BOX does not exist.: iostream error
BOX does not exist.: iostream error
When I try to visualize robot ur5.urdf in gepetto-gui, I get this message and only one segment is showed in the visualization screen. Can somebody explain what is happening and how I can fix this?
Thank you in advance.

Can you share your code to see what is missing. By the way, in the folder examples, you have an example of using Pinocchio with the viewer.

Line BOX does not exist.: iostream error means a) you are not up to date and b) that you have a specific UR5 model because I think there is no box in the official UR5 model.

@jcarpent
This is my code( I followed the example):
from pinocchio.robot_wrapper import RobotWrapper
import pinocchio
import gepetto
import numpy
pinocchio.switchToNumpyMatrix()
URDF = '/opt/openrobots/share/ur5_description/urdf/ur5_gripper.urdf'
robot = RobotWrapper.BuildFromURDF(URDF)
robot.initViewer(loadModel = True)
#rgbt = [0.0, 0.0, 1.0, 1.0]  # red, green, blue, transparency
#robot.viewer.gui.addSphere("world/sphere", .1, rgbt)  # .1 is the radius
visualObj = robot.visual_model.geometryObjects[4]  # 3D object representing the robot forarm
visualName = visualObj.name                        # Name associated to this object
visualRef = robot.getViewerNodeName(visualObj, pinocchio.GeometryType.VISUAL)
rgbt = [0.0, 0.0, 1.0, 1.0]  # red, green, blue, transparency
q1 = (.5, .1, .2, 1.,0.,0.,0. )
robot.viewer.gui.addSphere("world/sphere", .1, rgbt)  # .1 is the radius
robot.viewer.gui.applyConfiguration("world/sphere", q1)
robot.viewer.gui.refresh()
This is the photo of a gepetto-gui when I introduce a robot in it. Should it look like this? It seems that only the meshes can be drawn in gui.

you can do robot.display(robot.q0) to display the neutral configuration. You will see the whole robot then.

It worked. One more question: Although I see seven links, I get this message:
BOX does not exist.: iostream error
BOX does not exist.: iostream error
BOX does not exist.: iostream error
BOX does not exist.: iostream error
Node "world/pinocchio/collisions/ee_link_0" does not exist.
Node "world/pinocchio/collisions/tool0_0" does not exist.
Node "world/pinocchio/collisions/tool0_1" does not exist.
Node "world/pinocchio/collisions/tool0_2" does not exist.
Is it true that only meshes can be shown in gepetto gui? My robot now looks like this:


Too much rounding.

In the new interface, it only shows 9.00 Ark
In arkcoin.net, it shows the total value 9.00299711 Ark
If the new interface shows the exact value, it would be great!
You should at least see the unrounded values somehow.

Agreed, especially when you offer a website for your voters and your own site and the explorer show different numbers.

@luciorubeens I think this is a priority other things are not so pressing.


Exception: A new session could not be created. W3C capabilities: 'deviceName' can't be blank

Hi I'm updating classic window app test case by replacing DesiredCapabilities with AppiumOptions and AppiumCapablilities, but I currently have a problem I cannot resolve.
This is error in log file:
[Appium] Could not parse W3C capabilities: 'deviceName' can't be blank
This is the updated test script:
AppiumOptions appiumOptions = new AppiumOptions();
appiumOptions.AddAdditionalCapability("app", @ExecutablePath);
appiumOptions.AddAdditionalCapability("platformName", "Windows");
appiumOptions.AddAdditionalCapability("deviceName", "WindowsPC");
session = new WindowsDriver(new Uri(WindowsApplicationDriverUrl), appiumOptions, TimeSpan.FromSeconds(1.5));
At this point, exception is throw,  test case aborts but the application is launched.

OpenQA.Selenium.WebDriverException: OpenQA.Selenium.WebDriverException: The HTTP request to the remote WebDriver server for
URL http://<IP_ADDRESS>:4723/wd/hub/session timed out after 1.5 seconds. ---> System.Net.WebException: Timeout für Vorgang überschritten.
Appium Log file:
Appium REST http interface listener started on <IP_ADDRESS>:4723
[HTTP] --> POST /wd/hub/session
[HTTP] {"desiredCapabilities":{"app":"C:\Program Files (x86)\XXX\XXX\XXX\XXX.exe","platformName":"Windows","deviceName":"WindowsPC"},"capabilities":{"firstMatch":[{"platformName":"Windows"}]}}
[debug] [W3C] Calling AppiumDriver.createSession() with args: [{"app":"C:\Program Files (x86)\XXX\XXX\XXX\XXX.exe","platformName":"Windows","deviceName":"WindowsPC"},null,{"firstMatch":[{"platformName":"Windows"}]}]
[debug] [BaseDriver] Event 'newSessionRequested' logged at<PHONE_NUMBER>238 (12:37:15 GMT+0100 (GMT+01:00))
[Appium] Could not parse W3C capabilities: 'deviceName' can't be blank
[Appium] Trying to fix W3C capabilities by merging them with JSONWP caps
[BaseDriver] The capabilities ["app","deviceName"] are not standard capabilities and should have an extension prefix
[Appium] Creating new WindowsDriver (v1.4.0) session
[Appium] Capabilities:
[Appium]   app: C:\Program Files (x86)\XXX\XXX\XXX\XXX.exe
[Appium]   deviceName: WindowsPC
[Appium]   platformName: Windows
[debug] [BaseDriver] Creating session with MJSONWP desired capabilities: {"app":"C:\Program Files (...
[BaseDriver] The following capabilities were provided, but are not recognized by appium: app.
[BaseDriver] Session created with session id: e05f0406-e54c-4c5f-b239-a94d1244b66d
[WinAppDriver] You must use WinAppDriver version 1.1
[WinAppDriver] Verifying WinAppDriver version 1.1 is installed via comparing the checksum.
[debug] [WinAppDriver] WinAppDriver changed state to 'starting'
[WinAppDriver] Killing any old WinAppDrivers on same port, running: FOR /F "usebackq tokens=5" %a in (netstat -nao ^| findstr /R /C:"4724 ") do (FOR /F "usebackq" %b in (TASKLIST /FI "PID eq %a" ^| findstr /I winappdriver.exe) do (IF NOT %b=="" TASKKILL /F /PID %a))
[WinAppDriver] No old WinAppDrivers seemed to exist
[WinAppDriver] Spawning winappdriver with: 4724/wd/hub
[WinAppDriver] [STDOUT] Windows Application Driver listening for requests at: http://<IP_ADDRESS>:4724/wd/hub
[debug] [JSONWP Proxy] Proxying [GET /status] to [GET http://<IP_ADDRESS>:4724/wd/hub/status] with no body
[WinAppDriver] [STDOUT] Press ENTER to exit.
[WinAppDriver] [STDOUT]
Any idea on the source of the problem?
Appium.WebDriver version: <IP_ADDRESS>-beta
Selenium.WebDriver version: 3.14
Selenium.Support version: 3.14
Newtonsoft.Json  version=11.0.2

It is because of dotnet driver has no fix for W3C adaption, I assume.
e.g.: https://github.com/appium/python-client/blob/b6795db715d8b86b5ca5da980d597f42b31343d5/appium/webdriver/webdriver.py#L62
Then, Appium fallback to previous spec.

did you find any solution for this problem?

@afgaiel do you still get this exception with the latest server and driver?

@akinsolg: thanks very much. the problem is solved.  I think the device name had a space and I corrected it.


Change Unit Tests from MS Test to NUnit

Issue
The current test frame is MS Test
Fixes
Modernization with NUnit
Acceptance
All current tests are now executing in NUnit Framework

This would be a good time to bring in Bogus


Version 1.6.0 has bugs

I updated to version 1.6.0. When refreshing the screen, an error is appearing (Error: Hydration failed because the initial UI does not match what was rendered on the server.), and the component layout is broken.

Hi @reynierhdez 👋
Thanks for the feedback.
I did the documentation update with version 1.6.0 like you and I don't have the same problem. Can you show me your code or tell me how I can reproduce this problem.

I still can't reproduce the error. Can you reproduce it in a codeandbox and send the link

Now it's working.
When customizing the component, I set containerClassName to "py-0 my-0". I added "relative w-full text-gray-700" and it's working again.
Thanks

OK thank you for closing this issue.


Add "Never Gonna Give You Up"

Describe your PR, what does it fix/add?
Adds another splash under the song lyrics category.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
No
Is it ready for merging, or does it need work?
Ready to merge.

please do not make splash MRs


change deps

chore(deps): add missing devdeps
feat(deps): require react and reactDOM since they are peerdeps

Travis failed because of rollup config (since it thinks it's importing react). Should bundle only happen on the application level?

Bundle needs to happen on prepublish. You can modify the rollup config (module.exports in rollup.config.js) to have externals for React and ReactDOM so it won't try and bundle them. That might work.

Hey @stevemao just got to merging your other PRs. Can you have another look at this one when you have a chance pls? Thanks!

Yes, of course.

lgtm

I don't have a lot of context to this but it looks pretty weird. It looks like it should work though.

lgtm

@bradleyayers the idea is that react and react dom can be required as they'd be peer dependencies and provided by the consumer anyways. This build stuff can eventually be contained in skatejs/build (as it already handles dependencies, just not peerDependencies).

lgtm

LGTM

lgtm

lgtm!

lgtm


Remove support for C semantics

This pull request removes regression tests related to the C semantics from our repository.

Review checklist
The author performs the actions on the checklist. The reviewer evaluates the work and checks the boxes as they are completed.

[ ] Summary. Write a summary of the changes. Explain what you did to fix the issue, and why you did it. Present the changes in a logical order. Instead of writing a summary in the pull request, you may push a clean Git history.
[ ] Documentation. Write documentation for new functions. Update documentation for functions that changed, or complete documentation where it is missing.
[ ] Tests. Write unit tests for every change. Write the unit tests that were missing before the changes. Include any examples from the reported issue as integration tests.
[ ] Clean up. The changes are already clean. Clean up anything near the changes that you noticed while working. This does not mean only spatially near the changes, but logically near: any code that interacts with the changes!

This is because the backend lacks proper support for +Int patterns on the left-hand side of rules, particularly in Map keys.


Flathead Valley Community College

Please update Flathead Valley Community College
Home Page: https://www.fvcc.edu/

@vncloudsco First stage of verification passed, official domain is fvcc.edu as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.


keybase doesn't see my facebook proof

I posted my proof on facebook with a public audience settings but after 2 days, I got the message that keybase has given up.

I tried this today and have the same failure, but when I try to post I get an error screen from keybase but the proof is still posted.

Tried today and got the same problem as well

I can add a me too here. My facebook name is robert.smigielski.589 and I can see the posting from keybase. My facebook account uses 2 factor authentication in case that is part of the issue. Each time I try to perform the process from Chrome on Windows 10, Android, or the windows command line I get the same error:  Error 500 something went wrong.

I can add a me too here. My facebook name is robert.smigielski.589 and I can see the posting from keybase. My facebook account uses 2 factor authentication in case that is part of the issue. Each time I try to perform the process from Chrome on Windows 10, Android, or the windows command line I get the same error: Error 500 something went wrong.

Exactly the same case.

I have a similar issue. When I first posted the proof on facebook, I did not notice that it had my default "friends" visibility, so one initial failure was expected. But then I made it public, told keybase I had fixed it and to try again, but still got a failure. Then tried removing my claim and starting again from the beginning, making sure that the "new" proof post was public before I told keybase to check. Still repeatedly failing...

Our bug, we're working on it today
Thanks for your patience.

This should be fixed now. Go ahead and try making the proof again.
Any existing proofs that didn't work due to this bug won't automatically recover, but redoing the proof should work now.

ok I just deleted the previous post, revoked my claim, and started again from scratch. It works for me now!

It's working now.

Not working for me.  Tried multiple times, revoking, etc.  Still getting 500.

Just tried again this afternoon and still doesn't work for me. As before I
can see the proof on my Facebook page.
Robert Smigielski
On Thu, Nov 15, 2018, 5:20 PM seclorum<EMAIL_ADDRESS>wrote:

Not working for me. Tried multiple times, revoking, etc. Still getting 500.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/14537#issuecomment-439211338,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG_1NRhDINzWnqJGDtEQ82-jUpLaFmSxks5uveihgaJpZM4YHmEd
.

Facebook proof not working. Seeing Error 500. Something is not right. We will look into it. on posting prefilled post to facebook. Post is created but keybase is unable to see it.

I have the same issue, following.

Here's a link to my latest FB proof from yesterday and it still doesn't work: https://www.facebook.com/Haprog/posts/10157487956996030

Same here

Last night I deleted the proof because I set all my public posts to Facebook and forgot that keybase was on it.  So I deleted and was going to redo the process.  Now the facebook option isn't there anymore.  I see my twitter and github, but Facebook is missing.  Any thoughts?

Facebook is broken, see #14744. I would say this is a duplicate of 14744 but it is not up to me to decide


Exclusion regexps do not appear to work

coveralls \
    --build-root $(pwd) \
    --gcov $(which llvm-cov) \
    --gcov-options gcov \
    --verbose \
    --exclude-pattern ".*"

This should skip every file, because the regex matches all paths, but instead I see it generating a ton of gcov files.

Indeed, run_gcov does not ever touch the values in args.regexp.

You can reference my explanation here #145
tldr; --exclude-pattern isn't used for selecting what gets run with gcov, as you've discovered. It's used for filtering out files/directories from the final report. You have to use --exclude to exclude things from being run with gcov.


JSONPath template language

@brendandburns Thanks. Here are features in this pull request:
given template：hello ${.jq.title}
and struct: {jq: {title: "world"}}
we will get: hello world
I add quote "" function so that user can input string will not be evaluated:
expression: hello ${"${"}
output: hello ${

@brendandburns Thanks for reviewing my code. I correct all mistakes except for the third_party code problem. Actually, for the third_party code, I write 30% of them.

I rewrite parser so the parser code do't need to be put in third party directory.

@brendanburns The table driven test saves me a lot of code! Thanks mentor. I am still working on the filter part.

Cool, please ping me when this is ready for a final review.
--brendan

This is targeted towards the gsoc branch, but if it's done soon, I think we could pull it into master. As a standalone library/tool, it's not much of a liability.

@bgrant0607 the major features are done, but I need to add more test cases to find the minor bugs.

@brendanburns  dear mentor, do you think it seems redundant to always add a $ operator in front of JSONPath expression? It is not used elsewhere. e.g. ${.bicycle.} can be {.bicycle.}

Here are some things that I do with jq:

kubectl get nodes -o json | jq . to pretty-print with color
kubectl get nodes -o json | jq '.items[] | {name: .metadata.name, cap: .status.capacity}' to list node capacity
kubectl get pods/memhog -o json | jq .status.containerStatuses[].state.terminated.reason to check reason for failure

I hope these will be similarly easy with this PR, but I haven't checked.  Perhaps you could.

Sorry for the lack of feedback. We're in our 1.0 crunch at the moment.
golang-derived code must go in third_party/golang, even if modified.

I'm fine with dropping the $.

Please search the code base for uses of the existing "-o template" and determine whether any additional features are necessary to handle them. For example:
https://github.com/GoogleCloudPlatform/kubernetes/blob/5520386b180d3ddc4fa7b7dfe6f52642cc0c25f3/cluster/update-storage-objects.sh#L52
https://github.com/GoogleCloudPlatform/kubernetes/blob/8ebd896351513d446d56bc5785c070d2909226a3/release-0.20.0/docs/man/man1/kubectl-config-view.1#L168
https://github.com/GoogleCloudPlatform/kubernetes/blob/e5670c104d1c8cdc5bd31f472b3ca806c5b2df35/cluster/libvirt-coreos/util.sh#L186

Thanks! I will use some of them as test cases.

Thanks everybody your kind midterm evaluation :smile:. Here are some test cases I used in this PR now. I think they are enough for common tasks.

{range .items[*]}{.metadata.name}, {end}{.kind}  <IP_ADDRESS>, <IP_ADDRESS>, List
{.items[*].metadata.name}  <IP_ADDRESS> <IP_ADDRESS>
{.items[*]['metadata.name', 'status.capacity']}  [<IP_ADDRESS>, {cpu: 4}] [<IP_ADDRESS>, {cpu: 8}]
{.users[?(@.name=="e2e")].user.password} secret

This is failing Shippable, I think we need to rebase the gsoc branch to HEAD (and in general, Shippable doesn't seem to handle testing PRs to branches other than master very well)

@daizuozhuo Please ping any open PRs and/or issues with current status at least once a week to ensure they don't drop off our radar.

@brendandburns I have finished all features. Could you review it?

yes, will look by the end of the week.  Thanks!
--brendan

@brendandburns I just added doc.go. Can you take a look at it?

Thanks.
More examples:
Getting info about last termination(s):
kubectl get pods/pod-w-message -o template -t "{{range .status.containerStatuses}}{{.lastState.terminated.message}}{{end}}"

Getting external IP of a service of type LoadBalancer, which I found easier to do with grep than with a template:
kubectl get svc my-nginx -o json | grep \"ip\"

While awaiting another review on this, you could write another PR that adds this as a new output format to kubectl, such as -o jsonpath.

I would like to eventually use this to specify field references in the downward API.

Few small comments and a request for parser tests for ill-formed input.

@bgrant0607 To get external IP of a service of type LoadBalancer, we can use "{..ip}" to find the key ip recursively in JSONPath.

Ok, since we're now in post 1.0, can you rebase this on master, and we can merge it in.
Thanks!
--brendan

@brendandburns Do you mean reopen this pull request on master? If not, can you tell me how to do this?


make two lints notice instead of warn,

As discussed in #492 these two lints don't make sense to be Warning level. The BR they cover is not possible to be verified by a linter (BR <IP_ADDRESS>).
closes #492

Looks good to me. Will give other folks some time to chime in. Otherwise, we can get merged next day or so.

Thanks for the reminder @cpu. Should we try to get a fix in for #476 for that update as well, then? Are there other lints that could use a result change we should try to get in?

@cardonator Sorry I missed your follow-up comment. It looks like #476 was resolved by a docs update, was there something else you were thinking needed to be done?

Are there other lints that could use a result change we should try to get in?

I can't think of any such lints off-hand. It would be nice to fix https://github.com/zmap/zlint/issues/418 as part of this major release but otherwise I'm probably in favour of cutting the major release ASAP. It will make it easier to not have to juggle multiple branches in the event we need to do a point release for a bugfix or something in the meantime. Better to release smaller releases more often IMO.
I'll try to grab #418 this weekend and will start making plans for a v3 release soon after unless anything else comes up.

No worries @cpu the only other one I can think of is fixing the 398 day lint which has a warning and error on the same lint.

the only other one I can think of is fixing the 398 day lint which has a warning and error on the same lint.

oh, good call! Thanks - https://github.com/zmap/zlint/issues/496


Wrong width on vertical scroll

Hello, this is example http://codepen.io/delch/pen/wBQOmK. What i do wrong ?

See console errors:
Plugin .js and .css are not found (404 error)

Fixed, nothing change


added a compile hook for hardhat-vyper plugin

This PR creates a workflow to compile Vyper contracts using hardhat. It also adds a preprocessing functionality using C like pragmas ( #pragma once, #include "subContract.vy", ...).
Summary of additions and changes

The hardhat configuration is seperated in its on config file and also a new build:vyper script is added to package.json.
A new compile hook is introduced for the @nomiclabs/hardhat-vyper plugin to add the preprocessing functionality.

Awesome! Have you run yarn audit after updating dependencies?

@0age , yes, there seem to be 2 low severity vulnerabilities related to ethereum-waffle > ... > web3 which wasn't added by the new dependencies:
Insecure Credential Storage in web3


Add Pinmux PWM test

See the commit messages for more detailed explanations. This PR introduces a new pinmux_pwm_test which tests that a PWM instance can be muxed, and that the existing PWM loopback testing works only when the pinmux is configured correctly. If running in CI, this test would be sufficient to catch the issue fixed in #369.
I've tested this on both FPGA and in simulation locally, and the FPGA in CI is already set up with a jumper cable so this test should be passing. I've also added the loopback to the simulation model so this can pass there as well.

Force push just addresses the typo in the commit message.


Fixes #43, tweaselORG/cli#24: Use iproxy for ssh

This enables SSH connections via USB and minimal setup after the jailbreak. Running (rooted) SSH commands via the mobile user is also supported via this PR. I tested it on freshly jailbroken iOS 15 and 16.

When trying to run this, I get an error on both phones:
/home/benni/coding/JS/tweasel/appstraction/src/util.ts:393
        throw new Error(`Usbmux proxy exited with code ${code}: ${proc.stderr?.read()}`);
        ^

Error: Usbmux proxy exited with code 0: Exception ignored in: <module 'threading' from '/home/benni/.cache/autopy/python/3.11.3/lib/python3.11/threading.py'>
Traceback (most recent call last):
  File "/home/benni/.cache/autopy/python/3.11.3/lib/python3.11/threading.py", line 1583, in _shutdown
    lock.acquire()
TypeError: <lambda>() takes 1 positional argument but 2 were given

    at ChildProcess.<anonymous> (/home/benni/coding/JS/tweasel/appstraction/src/util.ts:393:15)
    at ChildProcess.emit (node:events:525:35)
    at ChildProcess.emit (node:domain:489:12)
    at ChildProcess._handle.onexit (node:internal/child_process:291:12)

Now, ensureDevice() works but I get this error in installCertificateAuthority() (I uninstalled various Sileo packages to make sure the autosetup is still working):
Installed apps: [
  'com.airbnb.app',
  'com.audible.iphone',
  'com.deliveroo.orderapp',
  'com.deutschebahn.navigator',
  'com.gimb.paydirect',
  'com.okcupid.app',
  'de.check24.check24',
  'de.dwd.warnapp',
  'de.otto.shop',
  'de.wetteronline.RegenRadar',
  'net.whatsapp.WhatsApp'
]
/home/benni/coding/JS/tweasel/appstraction/src/ios.ts:171
                throw new Error(`SSH command failed with exit code ${res.code}: ${res.stderr}`);
                      ^

Error: SSH command failed with exit code 127: [sudo] password for mobile: zsh:1: command not found: sqlite3
    at Object.ssh (/home/benni/coding/JS/tweasel/appstraction/src/ios.ts:171:23)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at Object.installCertificateAuthority (/home/benni/coding/JS/tweasel/appstraction/src/ios.ts:436:9)
    at <anonymous> (/home/benni/coding/JS/tweasel/appstraction/examples/ios-device.ts:25:9)

Oh, I think I've found the problem:
https://github.com/tweaselORG/appstraction/blob/f6ab51ffc4e836953eec1445eb7afc92f7eee9c9/src/ios.ts#L184
Uninstalling sqlite3 left behind this line:
# apt list --installed
libsqlite3-1/stable,now 3.34.1 iphoneos-arm [installed,auto-removable]

That needs to be pL.split().some(l => l.startsWith(p + '/'))[^1].
[^1]: Failing to escape the backtick in the template string I had there originally took me longer than finding a fix. :'D


OSError: [Errno 8] Exec format error

I am unable to run this on a Raspberry Pi.
Everytime I run fijibin.macro.run(macro) I get this error
Traceback (most recent call last):
File "", line 1, in
File "/home/pi/miniconda3/lib/python3.4/site-packages/fijibin/macro.py", line 77, in run
stderr=subprocess.PIPE, env=env)
File "/home/pi/miniconda3/lib/python3.4/subprocess.py", line 859, in init
restore_signals, start_new_session)
File "/home/pi/miniconda3/lib/python3.4/subprocess.py", line 1457, in _execute_child
raise child_exception_type(errno_num, err_msg)
OSError: [Errno 8] Exec format error
I am trying to use fijibin to an existing python script with GUI to do some image analysis.
Would appreciate the help.

Hi, and thanks for reporting. Do you have steps to reproduce?

Installed ImageJ according to this site: https://imagej.net/Raspberry_Pi
Installed fijibin based on this: https://pypi.org/project/fijibin/
Python Script:
import fijibin
import fijibin.macro
new_macro = """ open("/home/pi/Desktop/My files/cameraui.png");
run("Measure");
saveAs("Results", "/home/pi/Desktop/My files/Results.csv");
close();
"""
fijibin.macro.run(new_macro)
Error
Traceback (most recent call last):
File "test.py", line 9, in
fijibin.macro.run(new_macro)
File "/home/pi/fijibin/fijibin/macro.py", line 77, in run
stderr=subprocess.PIPE, env=env)
File "/usr/lib/python3.7/subprocess.py", line 775, in init
restore_signals, start_new_session)
File "/usr/lib/python3.7/subprocess.py", line 1522, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
OSError: [Errno 8] Exec format error: '/home/pi/.bin/Fiji-20141125.app/ImageJ-linux32'

Probably because raspberry is an ARM processor. You could try setting fijibin.BIN to the executable that you have installed.


Boarding pass QR

Look into the possibility of bringing QR code boarding passes once checked in, along with other useful info I end up finding after checking in.

This has been added in 2023.9.4


Add library append to sys path

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


Adding<EMAIL_ADDRESS>targetting<EMAIL_ADDRESS><EMAIL_ADDRESS>is compr…

…ised of pinned cordova android/ios/windows/wp8 platforms and core plugins matching<EMAIL_ADDRESS>tacokit@\
3.0.0 will allow users to seamlessly target<EMAIL_ADDRESS>without having to worry about "latest" breaking changes in core cordova plugins`

@meenakb @digeff can you guys take a look?

:+1:

:+1:


DC-274: integration tests with snapshots

Add an integration test for all service operations. The test creates, updates, retrieves and deletes a catalog entry for a TDR snapshot.
While testing we found that the HTTP status code for a missing catalog entry record was 500, but should've been 404. This PR also fixes this behavior and adds a unit test for this case.
Note that permissions tests are not part of this PR, that work will be done in https://broadworkbench.atlassian.net/browse/DC-297

This is great! We could use some cases that test failures, but there's no reason they cant be added later in th e future :)

I agree, although I think the next most important set of tests to write are ones involving permissions. I created https://broadworkbench.atlassian.net/browse/DC-297 for this work.


User Story: Browse Categories

As a user I want to browse categories of items so that I can easily search for my products.

Completed during #77


Allow TextInput to be auto focused

Is your feature request related to a problem? Please describe.
It would be handy to have this prop available
Describe the solution you'd like
Add autoFocus prop to the TextInput

Duplicate issue, this change have already been done


RTL Support

FSCalendar doesn't support the RTL mode, meaning it doesn't flip scrolling and dates order for RTL

@WenchaoD what do you mean by fixed? I cannot find anything related to RTL support in the documentation 😢

如何支持RTL ?


Copying events with overnight shifts creates broken copies

Describe the bug
When copying an event with shifts overnight to the next day, these shifts are created with wrong times.
To Reproduce

Create an event with the following shifts (as shown in the screenshot "original event")

22:30-23:30 ("normal shift")
23:30-00:30 ("overnight shift")

Create a copy of the event to the same day (using the configuration from the screenshot)
In contrast to the original event that ends at 01:00 (end of the overnight shift), the copy ends at 23:30 (end of the normal shift), as shown in the screenshots. However, the individual shift are both shown correctly on the copy.
The incorrect shift data also beaks the single day calendar view (see screenshot)

Expected behavior
When copying an event, all shifts including overnight shifts should be copied according to their configuration on the original event. The end time of the copied event should be identical to the end time of the latest shift end time.
Screenshots
Original event:

Copy configuration:

Copied event:

Broken single day calendar view:

Environment
Tested with:

ephios v0.16.3 (docker deployment with Postgres 12)
ephios on commit 5025baa (local development instance with SQLite)

Both times using Firefox 124.0.1
Additional context
When editing the overnight shift on the copy event and saving it unchanged (i.e., just visiting the edit view and clicking save), the event seems to be fixed as issues described above can't be observed afterwards.

I was able to reproduce the issue you described and prepared a fix in #1252. Came down to using the wrong timezone when handling the times.


How Aggregation Happens - Single table or Multiple Tables

The aggregations can happen only on one single table multiple table joins are not possible.
Comments
How to join the multiple tables need to be defined?
Please give multiple file and it's processing expectations.

This issue not covered in v5.0 and it has to be analyzed


Web & Device Extension by InfinityLoop

Web & Device
Created by InfinityLoop
Web & Device is an extension created by InfinityLoop (InfinityLoopGaming on Scratch), to interact and get data from websites, your device, and your network.
Features include:

Change & get page title
Get network stats
Get browser, OS, and memory stats
Toggle & get fullscreen
Open and redirect tabs, get current tab, get data from URL
Get screen width/height

Hello @InfinityLoopGames, Currently, the blocks from this extension are present in these other extensions:
(get data from url ()) Exists in the fetch extension.
<connected to internet>, (network type), (network generation), (browser), (current url), open url () in new tab, redirect to url () Exists in the Networking Extension.
(operating system) and (device memory) Exist in Navigator Info Extension.
(page title), set page title to (), <is page fullscreen>, enter fullscreen, exit fullscreen, (screen width), (screen height) Exist in Window Controls Extension.

@CubesterYT
Yes, this is kind of meant to combine similar extensions, they are also changed a bit.
I credited them in the code.

Alright, but I'll want to let you know that this extension may not be merged because
a. General Utility Extensions aren't allowed
b. Combination Extensions don't usually fly
c. Personal type extensions should just stay personal, instead of submitting a PR for public use.
These are part of the guidelines, but you are still allowed to keep this up till GarboMuffin reviews this.

I understand how it is the first two things you said, but how is this a personal type extension?

I understand how it is the first two things you said, but how is this a personal type extension?

@InfinityLoopGames, I didn't mean that this was a personal one, I'm just stating general rejected extensions.

@CubesterYT oh ok

Oh ok, I'll do that next time I make an extension.

I understand how it is the first two things you said, but how is this a personal type extension?

i mean, there is always sharkpool's gallery.

Fixed some issues with the extension ID, should work correctly now when ran as a file.

i'm not going to say no at this point, but as these all kind of exist elsewhere, it'll probably be a while
i know these blocks are currently very disorganized; centralizing some of the random ones into an extension that people will actually be able to find is not a bad idea

Added block icons


Insufficient commenting and Not up to coding standard behaviours

Comments should be added but universal indenting will cause lots of merge conflicts for students.

In process of adding comments.


bug: Error message every time open init.lua under nvim config

Did you check docs and existing issues?

[X] I have read all the lazy.nvim docs
[X] I have searched the existing issues of lazy.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.4
Operating system/version
Mac OS 14.1.1
Describe the bug
Error message every time open init.lua file after setting up lazy nvim:
Failed to source /Users/username/.local/share/nvim/lazy/nvim-ts-context-commentstring/plugin/ts_context_commentstring.vim
vim/_editor.lua:0: nvim_exec2()..BufReadPre Autocommands for ""..script nvim_exec2() called at BufReadPre Autocommands for "":0../Users/username/.local/share/nvim/lazy/nvim-ts-context-commentstring/plugin/ts_context_commentstring.vim, line 1: Vim(lua):E5108: Error executing lua [string ":lua
"]:1: attempt to index a boolean value
Steps To Reproduce
cd .config/nvim
nvim .
open init.lua
Expected Behavior
No error message
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

would help if you provided a minimal repro
that's the kind of error you normally get when you require a module which doesn't exist and then try to index it.

Thanks for your reply. I migrated from Packer to Lazy. I will try to figure out if I include all the plugins it needs.