Question about new Converter (Format)

Hello @SmetDenis
As I've already said, I'm working on a new SARIF Converter for your project.
Even if the POC is already available,  I've a little question about your project.
Did you think to open your API to add external converter without to embed it in your kernel code ?
BTW, I've noticed that PHPMD converter lose information details about violation that is not easy to retrieve/reuse by other converter later.
On my POC, I've modified the SourceCase class to store extra info (violation data structure)
diff --git a/src/Formats/Source/SourceCase.php b/src/Formats/Source/SourceCase.php
index 641a52f..b702e3c 100644
--- a/src/Formats/Source/SourceCase.php
+++ b/src/Formats/Source/SourceCase.php
@@ -17,6 +17,7 @@ declare(strict_types=1);
 namespace JBZoo\CIReportConverter\Formats\Source;

 use JBZoo\CIReportConverter\Formats\AbstractNode;
+use JBZoo\Data\Data;

 final class SourceCase extends AbstractNode
 {
@@ -27,6 +28,8 @@ final class SourceCase extends AbstractNode
     public ?int    $line      = null;
     public ?int    $column    = null;

+    public Data    $violation;
+
     // Output
     public ?string $stdOut = null;
     public ?string $errOut = null;

Could you explain why you split / re-implement for each format a *Case.php and *Suite.php.
I've not follow it and my SARIF converter is only  130 lines of code (of course with help of my project https://github.com/llaville/sarif-php-sdk)
Oh, FYI, here is a preview of what my POC is able to do on PHPMD
Command invoked
phpmd vendor/povils/phpmnd/src/ json unusedcode | ./ci-report-converter convert -S phpmd-json -T sarif --profile -vv

Gave somethink like this

PHPMD report converted to SARIF
{
    "$schema": "https://json.schemastore.org/sarif-2.1.0.json",
    "version": "2.1.0",
    "runs": [
        {
            "tool": {
                "driver": {
                    "name": "PHPmd",
                    "rules": [
                        {
                            "id": "UnusedPrivateMethod",
                            "defaultConfiguration": {
                                "enabled": true,
                                "level": "error"
                            },
                            "properties": {
                                "ruleSet": "Unused Code Rules"
                            }
                        },
                        {
                            "id": "UnusedFormalParameter",
                            "defaultConfiguration": {
                                "enabled": true,
                                "level": "error"
                            },
                            "properties": {
                                "ruleSet": "Unused Code Rules"
                            }
                        }
                    ]
                }
            },
            "originalUriBaseIds": {
                "SRCROOT": {
                    "uriBaseId": "/shared/backups/forks/CI-Report-Converter"
                }
            },
            "logicalLocations": [
                {
                    "name": "castToNumber",
                    "fullyQualifiedName": "Povils\\PHPMND\\Command\\RunCommand->castToNumber()",
                    "kind": "function"
                },
                {
                    "name": "Povils\\PHPMND\\Command",
                    "kind": "namespace"
                },
                {
                    "name": "getParser",
                    "fullyQualifiedName": "Povils\\PHPMND\\Container->getParser()",
                    "kind": "function"
                },
                {
                    "name": "Povils\\PHPMND",
                    "kind": "namespace"
                },
                {
                    "name": "getLexer",
                    "fullyQualifiedName": "Povils\\PHPMND\\Container->getLexer()",
                    "kind": "function"
                },
                {
                    "name": "Povils\\PHPMND",
                    "kind": "namespace"
                }
            ],
            "results": [
                {
                    "message": {
                        "text": "Avoid unused private methods such as 'castToNumber'."
                    },
                    "ruleId": "UnusedPrivateMethod",
                    "locations": [
                        {
                            "physicalLocation": {
                                "artifactLocation": {
                                    "uri": "vendor/povils/phpmnd/src/Command/RunCommand.php",
                                    "uriBaseId": "SRCROOT"
                                },
                                "region": {
                                    "startLine": 254
                                }
                            }
                        }
                    ]
                },
                {
                    "message": {
                        "text": "Avoid unused private methods such as 'getParser'."
                    },
                    "ruleId": "UnusedPrivateMethod",
                    "locations": [
                        {
                            "physicalLocation": {
                                "artifactLocation": {
                                    "uri": "vendor/povils/phpmnd/src/Container.php",
                                    "uriBaseId": "SRCROOT"
                                },
                                "region": {
                                    "startLine": 57
                                }
                            }
                        }
                    ]
                },
                {
                    "message": {
                        "text": "Avoid unused private methods such as 'getLexer'."
                    },
                    "ruleId": "UnusedPrivateMethod",
                    "locations": [
                        {
                            "physicalLocation": {
                                "artifactLocation": {
                                    "uri": "vendor/povils/phpmnd/src/Container.php",
                                    "uriBaseId": "SRCROOT"
                                },
                                "region": {
                                    "startLine": 62
                                }
                            }
                        }
                    ]
                },
                {
                    "message": {
                        "text": "Avoid unused parameters such as '$nodes'."
                    },
                    "ruleId": "UnusedFormalParameter",
                    "locations": [
                        {
                            "physicalLocation": {
                                "artifactLocation": {
                                    "uri": "vendor/povils/phpmnd/src/PhpParser/Visitor/DetectionVisitor.php",
                                    "uriBaseId": "SRCROOT"
                                },
                                "region": {
                                    "startLine": 26
                                }
                            }
                        }
                    ]
                },
                {
                    "message": {
                        "text": "Avoid unused parameters such as '$nodes'."
                    },
                    "ruleId": "UnusedFormalParameter",
                    "locations": [
                        {
                            "physicalLocation": {
                                "artifactLocation": {
                                    "uri": "vendor/povils/phpmnd/src/PhpParser/Visitor/ParentConnectorVisitor.php",
                                    "uriBaseId": "SRCROOT"
                                },
                                "region": {
                                    "startLine": 17
                                }
                            }
                        }
                    ]
                },
                {
                    "message": {
                        "text": "Avoid unused parameters such as '$node'."
                    },
                    "ruleId": "UnusedFormalParameter",
                    "locations": [
                        {
                            "physicalLocation": {
                                "artifactLocation": {
                                    "uri": "vendor/povils/phpmnd/src/PhpParser/Visitor/ParentConnectorVisitor.php",
                                    "uriBaseId": "SRCROOT"
                                },
                                "region": {
                                    "startLine": 31
                                }
                            }
                        }
                    ]
                }
            ]
        }
    ]
}

Forget to said that I've modified both SourceCase (previous patch) and also PhpMdJsonConverter
diff --git a/src/Converters/PhpMdJsonConverter.php b/src/Converters/PhpMdJsonConverter.php
index 5c4a0b0..30af09f 100644
--- a/src/Converters/PhpMdJsonConverter.php
+++ b/src/Converters/PhpMdJsonConverter.php
@@ -56,6 +56,8 @@ final class PhpMdJsonConverter extends AbstractConverter
                     self::getDetails($violation),
                 );

+                $case->violation = $violation;   // because getDetails() lose information
+
                 $package = $violation['package'] ?? null;
                 if ($package !== null) {
                     $case->class     = $package;

Found a better approach (about little patches in your kernel code). I'll propose soon

I can help you integrate the new format into my converter. No problem. Really.
To do this, I need to

A maximally finished example of your utility report.
A detailed description of your report format.
Any additional context that I could hypothetically use.

Feel free to attach these files and descriptions here in the comments.
Unfortunately, I am not familiar with your project and can't see how I can help you yet.
Thank you in advance.

Could you explain why you split / re-implement for each format a *Case.php and *Suite.php.

Each format has a function for parsing an source format (CheckStyle.xml, JUnit.xml, phpmd, etc) that understands the syntax and construction of the original report and parses it into cases and case groups (suites) or even sub-cases, sub-suites.
A case is an object that contains the text of the error message, error level, file location, code word, rule name, snippets, etc. As much as possible about original report without losing details (if it's possible).
After parsing, this is converted into an internal format (src/Formats/Source) which is an intermediate step between source and target formats.
Thus, in order to recognize a format as a source, you need to implement toInternal(). For the converter to know how to convert to target, it needs to implement fromInternal().
As real example you take a look at https://github.com/JBZoo/CI-Report-Converter/blob/master/src/Converters/JUnitConverter.php

You can use the project code as an SDK for any available reports. This API is stable and will not change, only expand.
https://github.com/JBZoo/CI-Report-Converter?tab=readme-ov-file#use-tool-as-sdk-to-generate-reports
Thus, you can prepare one data format, save it as a Source/* object, and using existing functions, convert it to any format you like or to all of them at once - https://github.com/JBZoo/CI-Report-Converter/blob/master/src/Commands/Convert.php#L119-L123.
As a benefit, you only need to make a report once, and it will convert to all at once, and my project can be just a dependency that you use as a framework.
Alternatively, use it as a console utility in the CI pipeline.
PS There are actually a huge number of formats out there that are supported by CI tools (GA, Jenkins, Drone, TeamCity). For example - super popular junit and checkstyle. Frankly speaking, I don't understand why you need to reinvent the wheel. But it's just my personal opinion, nothing more.

Btw, just examples how to add format as part of my project

Output - https://github.com/JBZoo/CI-Report-Converter/pull/9/files
Input - https://github.com/JBZoo/CI-Report-Converter/pull/5/files

Thanks for answer. I've no much free time (other projects whith highest priority) to continue to implement my SARIF converter, but I'll let you know when I'll be back active on this task !

Very welcome @llaville
Btw, there is another example to use it as lib.

https://github.com/JBZoo/Csv-Blueprint/blob/master/src/Validators/ErrorSuite.php#L55-L62
https://github.com/JBZoo/Csv-Blueprint/blob/master/src/Validators/ErrorSuite.php#L155-L165

Feel free to ask me any questions.
Thank you!


ParameterizedType in type/2

Environment

Elixir version (elixir -v): 1.12
Database and version: PostgreSQL 11.8
Ecto version (mix deps): 3.8.0-dev
Database adapter and version (mix deps): "0.15.13"

Initially I was trying to do this:
"users"
|> select([u], fragment("CASE WHEN ? THEN ? ELSE ? END", is_nil(u.joined_at), :pending, :active))
|> limit(1)
|> Repo.all()

 Error
The error is a bit cryptic, is it possible to improve it?
** (FunctionClauseError) no function clause matching in Ecto.Adapters.Postgres.Connection.expr/3

    The following arguments were given to Ecto.Adapters.Postgres.Connection.expr/3:

        # 1
        :pending

        # 2
        {{[34, "users", 34], [117 | "0"], nil}, []}

        # 3
        #Ecto.Query<from u0 in "users", limit: 1,
         select: fragment("CASE WHEN ? THEN ? ELSE ? END", is_nil(u0.joined_at), :pending, :active)>

    (ecto_sql 3.7.1) lib/ecto/adapters/postgres/connection.ex:590: Ecto.Adapters.Postgres.Connection.expr/3
    (elixir 1.12.2) lib/enum.ex:1582: Enum."-map/2-lists^map/1-0-"/2
    (elixir 1.12.2) lib/enum.ex:1582: Enum."-map/2-lists^map/1-0-"/2
    (ecto_sql 3.7.1) lib/ecto/adapters/postgres/connection.ex:648: Ecto.Adapters.Postgres.Connection.expr/3
    (ecto_sql 3.7.1) lib/ecto/adapters/postgres/connection.ex:1282: Ecto.Adapters.Postgres.Connection.intersperse_map/4
    (ecto_sql 3.7.1) lib/ecto/adapters/postgres/connection.ex:340: Ecto.Adapters.Postgres.Connection.select/3
    (ecto_sql 3.7.1) lib/ecto/adapters/postgres/connection.ex:113: Ecto.Adapters.Postgres.Connection.all/2
    (ecto_sql 3.7.1) lib/ecto/adapters/postgres.ex:102: Ecto.Adapters.Postgres.prepare/2
    (ecto 3.7.1) lib/ecto/query/planner.ex:180: Ecto.Query.Planner.query_without_cache/4
    (ecto 3.7.1) lib/ecto/query/planner.ex:150: Ecto.Query.Planner.query_prepare/6
    (ecto 3.7.1) lib/ecto/query/planner.ex:125: Ecto.Query.Planner.query_with_cache/7
    (ecto 3.7.1) lib/ecto/repo/queryable.ex:203: Ecto.Repo.Queryable.execute/4
    (ecto 3.7.1) lib/ecto/repo/queryable.ex:19: Ecto.Repo.Queryable.all/3

Current behavior
"users"
|> select(
  [u],
  type(
    fragment("CASE WHEN ? THEN ? ELSE ? END", is_nil(u.joined_at), "pending", "active"),
    {:parameterized, Ecto.Enum, Ecto.Enum.init(values: [:active, :pending])}
  )
)

error:
** (Ecto.Query.CompileError) type/2 expects an alias, atom or source.field as second argument, got: `{:parameterized, Ecto.Enum, Ecto.Enum.init(values: [:active, :pending])}`
    (ecto 3.8.0-dev) expanding macro: Ecto.Query.select/3
    iex:8: (file)
    (elixir 1.10.4) expanding macro: Kernel.|>/2
    iex:8: (file)

Expected behavior
Support parameterized types in Ecto.Query.API.type/2

You should use the type of the column instead:
type(fragment, t.column_with_type_enum)

Other than that, we should indeed raise for atoms in fragments (or anywhere in a query really, unless they have been cast to a parameterized type).

Ah, I see, there is a similar issue https://github.com/elixir-ecto/ecto/issues/3767, so virtual field is a solution.

Other than that, we should indeed raise for atoms in fragments (or anywhere in a query really, unless they have been cast to a parameterized type).

Should current issue be used for that? Feel free to close if not.

I think there is value in supporting parameterized types on type/2, as I pointed in the comment at #3767. I'm just not sure about how can we do it. Perhaps adding type/3? I just wouldn't like to expose the {:parameterized, type, params} to the user.

Closing this. Either refer to a field or use @my_type Ecto.ParameterizedType.init(Ecto.Enum, values: ...) to instantiate the type that you can pass as second argument. :)


I'd like to ask you how to add slashes to polygons

I've been troubled by this problem for a long time

Take a look here https://github.com/cloudybay/leaflet-polygon-fillPattern


feat: add normalise and validate functions

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@dcb2b45). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master       #26   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?        12
  Lines             ?        57
  Branches          ?         2
==========================================
  Hits              ?        57
  Misses            ?         0
  Partials          ?         0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Error handling

First, awesome work guys!
I've been trying out JAWS and at the beginning was a bit hard to debug what are the errors that AWS API  Gateway returns.
Since context.fail expects a message (string) and all examples (sign in, signup...) are returning an object [1], as a response you usually get something like:
{
  "errorMessage": "[object Object]"
}

[1] Example error thrown:
{
      status: 401,
      message: 'Invalid login credentials'
}

Temporarily I changed some of the functions to return a string instead of object (e.g. 401: Invalid login credentials). With that I'm able to use "Integration response" that uses lambda regex and on the end returns proper response.
Also a mapping template like this:
{
    "error" : $input.json('$.errorMessage'),
    "status" : 401
}

This seems like an ugly hack. Is there any other way to do this? Like with using the error model lambda has or anything else? Otherwise, it is obviously a limitation on AWS' side.

@B-Scan This is something I need to research as well, but have been too busy working on other components for JAWS v1.
With your mapping template approach, are you able to get API Gateway to return the same HTTP response code?  I believe you must return an error string to trigger the lambda error regex and render the right status code, correct?  It's been about a month since I've looked into this issue, so I'm a bit rusty.
@B-Scan If you find a good solution for this, please put it in here. :+1:

@ac360 yes, with setting custom integration response that was triggered with lambda error regex, I was able to get the right status code.
I posted a thread on AWS forum to see is there any other way to return an object instead of a string for errors. If not, I can make some changes and submit a PR with string only version. If you are happy with that.

@B-Scan @ac360 Hi. I've had the same issue and here's the workaround I use:
In your lambda function:, use:
context.fail(JSON.stringify(error));
# Instead of:
# context.fail(error);

For your integration response's Lambda Error Regex and Method response status, use:

\{"status":4.* => 400
\{"status":5.* => 500
and so on...

It works as far as the status is the first property in the error object.
Finally, in your integration response's mapping template, use:
$input.path('$.errorMessage')

Very ugly and incomplete, but now you can pass your error as is from lambda to API gateway's HTTP response body.

is there anyway to put the Lambda Error Regex in the s_function.json?

I have the same question as @ialmetwally - where is this to be placed ? what do you place in there ?
...
         "responses": {
           "400": {
             "statusCode": "400"
           },
           "default": {
             "statusCode": "200",
             "responseParameters": {},
             "responseModels": {},
             "responseTemplates": {
               "application/json": "IT GOES HERE"
             }
           }
         }
....

@iongion I found out how to do it
"404" : {
"selectionPattern" : "pattern_here",
"statusCode": "404"
}

@ialmetwally I do not understand, I see no response template velocity stuff, do you have a complete s-function.json ?

@iongion I just put the statusCode, you can add the rest of params inside the same object
"404" : {
"selectionPattern" : "pattern_here",
"statusCode": "404"
}
or
"pattern_here" : {
"statusCode": "404"
"responseParameters": {},
"responseModels": {},
"responseTemplates": {
"application/json": "IT GOES HERE"
}
}

@ialmetwally awesome !!!

To be able to have my own custom error responses with at least a 400 error status code I do the following:

in case of error, from the lambda I return an expected Error object

var fault = { type: 'api', message: 'could not continue', detail: 'not-allowed', code: 4332 };
var error = new Error(JSON.stringify({ status: 'failure', fault: fault }));
context.fail(error);

then my s-function.json I have the responses section as following:

...
       "responses": {
         "400": {
           "selectionPattern": "(.|\\n)*\\\"status\\\"\\:\\s*\\n*\\s*\\\"failure\\\"(.|\\n)*",
           "statusCode": "400",
           "responseParameters": {},
           "responseModels": {},
           "responseTemplates": {
             "application/json": "#set($inputRoot = $input.path('$.errorMessage'))\n$inputRoot"
           }
         },
         "default": {
           "statusCode": "200",
           "responseParameters": {},
           "responseModels": {},
           "responseTemplates": {
           }
         }
       }
...

This means that whenever a complete match of errorMessage value is found against this regex:

(.|\n)*\"status\"\:\s*\n*\s*\"failure\"(.|\n)*

Then the response should be the $input.path('$.errorMessage'), but the $.errorMessage coresponds to the unserialized JSON string that was passed to the Error constructor
This should result in an error object that is returned to the client which looks like this:

{
  status: 'failure',
  fault: {
    type: 'api',
    message: 'could not continue',
    detail: 'not-allowed',
    code: 4332
  }
}

which is awesome.
Now, the problem is that the s-function.json configuration is not loaded at API Gateway configuration, the integration response has no 400 handling defined, how did you get that loaded by API Gateway ?

My take on time is to find a way to pass context.fail with a string value


Coerce non-numeric values to numeric in the datadog backend

In order to address the case where data is dropped from the datadog backend if the aggregation result is a non-numeric value (NaN, +Inf, -Inf), this PR modifies the component's behaviour to coerce such values into a numeric. This serves as a basic fix to #342.

Inf+ --> float.Max
Inf- --> float.Min
NaN --> -1

NaN's conversion to -1 was chosen because negative values are (anecdotally) less frequent compared to zero counters.

lgtm

Should we not do this for all agregations produced instead of just for the datadog backend?

We can pull it out if needed.  I think InfluxDB allows infinity for instance (the query language can check for it, but that might only be usable if you produce an infinity, not query an infinity).

Thanks team!


Hactoberfest Excluded Repository

If you guys are trying to get swags by using this repo then this is totally useless as this repo is EXCLUDED.

Do you happen to have a curated list of legit hacktoberfest 2022 repositories?

This repository has been permanently excluded from Hacktoberfest 2022. This means that any pull requests (PRs) made to this repository will not be counted towards your contributions for Hacktoberfest 2022.
Please contribute to valuable repositories instead of sending pull requests (PRs) to repositories that are there to defeat the whole purpose of Hacktoberfest 2022.
Any repeated, continuous exclusions will automatically ban your account from participating in Hacktoberfest 2022.


[Bug]: Icon Cloning not working with glob patterns

Describe the bug
"material-icon-theme.files.customClones": [
  {
    "name": "typescript-lib",
    "base": "typescript",
    "color": "red-300",
    "lightColor": "light-green-600",
    "fileNames": ["model.*.ts"]
  }
]

"fileNames": ["model.*.ts"]   // here * = any name
It is not warking. Plz, give me solution

Sadly, vscode's icons API lacks support for globs/regex in file/folder icons names assignment.
While what you're asking for is not possible to do, this, on the other hand will work:
// ...
{
  "name": "typescript-lib",
  "base": "typescript",
  "color": "red-300",
  "lightColor": "light-green-600",
  "fileExtensions": ["model.ts"]  // instead of fileNames, use fileExtensions
},
// ...

This means that instead of model.foo.ts you would have to use a foo.model.ts naming convention:

And by the way, this is also possible:
// ...
"fileExtensions": ["model.ts", "models/ts"], //notice "models/ts"
// ...

Which means that any *.ts inside a models folder will use the new colors:

PS: this was not an icon request, next time reporting a bug, please use the "Bug" button :)

Let's close this. We have enough issues reporting this already


.content-ios CSS missing $text-ios-color variable

Short description of the problem:
In content.ios.scss
.content-ios {
  background-color: $background-ios-color;
}

should include $text-ios-color variable, just like md
.content-md {
  color: $text-md-color;
  background-color: $background-md-color;
}

What behavior are you expecting?
ios should match md and wp with .content customization.
Gulp version:  CLI version 3.9.1
Ionic Framework Version: 2.0.0-rc.0
Ionic CLI Version: 2.1.0
Ionic App Lib Version: 2.1.0-beta.1
Node Version: v6.7.0

I see the PR for this. https://github.com/driftyco/ionic/pull/8421


Is it possible to use selects in migrations?

I would like something like this:
async up(): Promise<void> {
    const otherTableId = await this.execute('SELECT id FROM other_table WHERE ...');
    this.addSql('alter table test add other_table_id int');
    this.addSql('update table test set other_table_id = ' + otherTableId);
}

But I haven't found a way to make a select and get the result of its execution in the migration..

There is a driver instance on the Migration class, so something like this should work:
const res = await this.driver.execute('...');

But thinking about that now, it will probably not use the same transaction.

I try it:

KnexTimeoutError: Knex: Timeout acquiring a connection. The pool is probably full. Are you missing a .transacting(trx) call?
at Client_MySQL2.acquireConnection (/app/node_modules/knex/lib/client.js:347:26

Yeah thats the issue, migrations are executed with single connection in the pool. In next rc there is execute method that handles this.


wr_hier.py only return GOterms in namespace 'BP'

Dear author, i was running
$ scripts/wr_hier.py BP MF CC --gene2go=gene2go --taxid=9606 --dash_len=17 --concise -o human_BP_MF_CC.txt
but the results in txt only includes those GOterms in 'BP'. I've checked the gene2go file and it does include MF and CC.
Full results are here:
EXISTS: go-basic.obo
go-basic.obo: fmt(1.2) rel(2020-03-23) 47,232 GO Terms
HMS:0:00:04.080605 330,518 annotations, 20,562 genes, 18,403 GOs, 1 taxids READ: gene2go
18499 IDs in loaded association branch, BP
NoneGoSubDag: 47232 sources in 47232 GOs rcnt(True). 2701 alt GO IDs
NoneGoSubDag: namedtuple fields: NS level depth GO alt GO_name dcnt D1 tcnt tfreq tinfo id
NoneGoSubDag: relationships: set()
NoneGoSubDag: 12378 sources in 15696 GOs rcnt(True). 0 alt GO IDs
NoneGoSubDag: namedtuple fields: NS level depth GO alt GO_name dcnt D1 id
NoneGoSubDag: relationships: set()
WROTE: human_BP_MF_CC_test.txt

Thanks for reporting this and thank your for the specific command to replicate the issue.
You are correct. We changed the underlying infrastructure of the annotation readers so that they all share a common base class and in addressing another issue to separate the GO terms more easily into BP, MF, and CC, this command wrongly no longer prints MF and CC.
I'll fix this shortly. Not catching this earlier was due to missing functionality in the tests, which will also be corrected.
Thank so much for being alert and for taking the time to report this issue. And thank you for your interest in GOATOOLS.

@tanghaibao , Can you please create a new version? This is an important fix and should be available to all. Thanks.

Thanks for your reply

@dvklopfenstein @NewTearingHeaven
Updated the PyPi version to v1.0.5.

thanks for your reply
---Original---
From: "Haibao<EMAIL_ADDRESS>Date: Sat, May 16, 2020 11:48 AM
To<EMAIL_ADDRESS>Cc: "Mention"<mention<EMAIL_ADDRESS>Subject: Re: [tanghaibao/goatools] wr_hier.py only return GOterms in namespace 'BP' (#163)
@dvklopfenstein @NewTearingHeaven
Updated the PyPi version to v1.0.5.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


feat: sbtc modal

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

[ ] Refactor
[X] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update

Description
Adding modal for sbtc
Issue ticket number and link
https://github.com/hirosystems/explorer/issues/1944
Checklist:

[X] I have performed a self-review of my code.
[X] I have tested the change on desktop and mobile.
[X] I have added thorough tests where applicable.
[X] I've added analytics and error logging where applicable.

Screenshots (if appropriate):

:tada: This PR is included in version 1.239.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Use dynamic returns for ray_execute_bundle

Signed-off-by: Jiajun Yao<EMAIL_ADDRESS>With dynamic returns feature, we no longer need to calculate the number of returns beforehand.

GH has some issue right now so CI is not triggered. Don't merge it until I run all tests.

this is awesome. Thanks LGTM!


ObservableArray binding

I'm am having serious troubles getting this to work.
I'm getting JSON data from a remote server, I made and ObservableArray for the items inside the Json (which I parsed). Everything is ok for this. I checked using browser console and my data is valid.
Now I'm trying to foreach this array in the UI using

 //properties.Title are object in my Array

This just won't work, I tried numerous combinations. Is there a problem with the fact that I am doing this on initial load? I am populating my ObservableArray in the constructor.
Any help, please? Would be very thankful if someone would help me out.

I suspect the issue is the use of 'this' in your foreach binding.
If 'Data' is an ObservableArray on your View Model (with 'PreserveCase' so that it doesn't become camel case in the script) - you would simply use data-bind="foreach: Data"
For an example of this see - the html that renders a template foreach contact - https://github.com/scottdurow/SparkleXrm/blob/ebd4a3579cf656fc38a26119f781fbe781f0c9ec/SparkleXrmSource/DebugWeb/WebResources/dev1_/html/ContactCardViewer.htm#L67
This bound to the 'Contacts' field - https://github.com/scottdurow/SparkleXrm/blob/ebd4a3579cf656fc38a26119f781fbe781f0c9ec/SparkleXrmSource/Client/InlineSubGrids/ViewModels/ContactCardViewModel.cs#L15
Notice that since [PreserveCase] is not used, the script name is camelCase.
You can read more about the binding context at http://knockoutjs.com/documentation/binding-context.html
Hope this helps!

Oh dear lord. It was indeed the "this".
Thank you very much :)


Cannot import name 'SOM' from partially initialized module 'sompy'

I get
Traceback (most recent call last):
File "D:\Projects\AI\SOM\data\sompy_clustering.py", line 1, in
import sompy
File "C:\Python\lib\site-packages\sompy_init_.py", line 2, in
from sompy import SOM
ImportError: cannot import name 'SOM' from partially initialized module 'sompy' (most likely due to a circular import) (C:\Python\lib\site-packages\sompy_init_.py)

If you still have problem, try these passages from anaconda cmd if it is what you use:

pip uninstall sompy
git clone <code link of this library>
cd_sompy
python setup.py install

Then on your notebook you just have to:

import sompy

Probably you already have a version of sompy that is the one that has recursively problems with python>3.0


Update styles for the individual transaction page

New styles are available here: https://discordapp.com/channels/435916207162654722/437556042864394251/476037720138055700

Related:
It was brought up in the August 6th meeting that we also need to support multiple action types, not just transfers. We also need a display screen for member registration and unregistration. In addition we'll need all other eosDAC related actions such as registering as a custodian, voting on custodian, creating a worker proposal, voting for a worker proposal, etc.


upgrade chart versions

change  to version change jenkins-x/tekton to version 0.0.37

/test tekton

/test ng


chore: bump version of node bot in release workflow

Why
Semantic release bot only runs on node >= 18 and we were using node 14.
How
Bumped the version of node in the workflow
Checklist

[x] Commit messages are formatted as per the convention, and breaking changes are marked

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix issue #10: --twitch-oauth-token argument was removed in Streamlink 2.0.0

This PR fixes issue #10 by removing the --twitch-oauth-token argument to Streamlink. It doesn't seem to have had any effect even in older version of Streamlink, so it should be backwards-compatible.
Tested manually with Streamlink 2.0.0.

StreamLink changelog notes that --twitch-oauth-token can be replaced with --http-header="Authorization=OAuth oauth-token-from-twitch-website". Maybe I should do that instead?

I also just removed the --twitch-oauth-token and it seems to run without problems, did that ever do anything?

StreamLink changelog notes that --twitch-oauth-token can be replaced with --http-header="Authorization=OAuth oauth-token-from-twitch-website". Maybe I should do that instead?

I also just removed the --twitch-oauth-token and it seems to run without problems, did that ever do anything?

I had the same issue. And separate from this PR, I attempted to simply remove the --twitch-oauth-token and was able to successfully get the stream.

I had the same issue. And separate from this PR, I attempted to simply remove the --twitch-oauth-token and was able to successfully get the stream.

@mlindgren thank you

@mlindgren thank you


What about docs?

Let's work on the documentation:

[ ] #7
[ ] #8
[ ] #9
[ ] Decide if we want to go with something more complex

So what you're saying is

[ ] Create a 'docs' folder
[ ] Create new 'index.md' document
[ ] Turn on GitHub pages
[ ] Decide if we want to go with something more complex.

Thanks me. I created the issues for the first three suggestions.


Refactor solution for Metrics ++ Polish properties

The changes in this PR are:

Change the tracing/metrics format to follow the same format as the rest of the properties

This change allows to enable/disable/configure these extensions per module.

Refactor the metrics module to decouple the metrics of prometheus and have another type of exporter: file.

My idea is to integrate the metrics within the running Quarkus application, so we can assert metrics during the test execution without needing to have a Prometheus.
Also, we're exporting the metrics into a file (/target/logs/metrics.out).

@pjgg I've already addressed all your comments:

module_name remains the same, but I've added an additional label scenario_name with the running class name
renamed module_duration_seconds to scenario_duration_seconds
renamed all the metrics from success to succeed and fail to failed


Added new descriptions and updated images

@gsanchezu to review the changes as well.

@asanokhin this is ready to be merged.


Test mrcli setup - owning company creation

Test the creation of owning companies by:

Installing mediumroast_js via rpm i -g mediumroast_js
After installation run mrcli setup and follow all required steps
Once the CLI completes check out /tmp for company.json to see if your updates were stored

Variation 1
Perform the test with a public company like IBM or Hitachi
Variation 2
Perform the test with a non-public company having a wikipedia page like Cerebras
Variation 3
Perform the test with a non-public company having no wikipedia page like Mediumroast, Inc.

Closing as the tests are going to be different due to change to GitHub


Kj/induced subgraphs

Pull Request Test Coverage Report for Build 659178277

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 659129830:
0.0%

Covered Lines:
79

Relevant Lines:
79

💛  - Coveralls


Suggestion for Debugging the code-base

I know that this might not be acceptable in the current way. I just want to point out that this helped me a lot in understanding particular code-places with the debugger. It's just a lot more useful to see the code in the debugger instead of line and column numbers.
So feel free to just close this PR or maybe we can find a way to somehow integrate it and improve the debugging experience with range. Or just tell me what you think about changes like these in general.

@dsyme like this?
Note that I don't feel like this is a good idea, debuggers really should be able to handle exceptions here. It is a basic and fundamental thing...

I'm more concerned about the implemantation. There should be a better way, correct? (Though for the the given use-case it probably is just fine)

Seems fine to me


All functions should pass NamedNodes not URIs

in general in the whole rdflib.js and in the solidos stack a RDF object is  passed as a NamedNode object rather than a string.
Here is an example where URI strings are used instead.
https://github.com/solid/solid-logic/blob/main/src/util/UtilityLogic.ts#L138
There may be other.  Please change them all to pass NamedNodes -- like the function before not like the one after.
Developers using the code should get used to this consistency.
This also allows us to use Typescript more usefully.  If all parameters are strings, then we don't get the value out of TS.
also when it comes to routing naming, include the word 'load' instead of 'get' when there is a web fetch.

@bourgeoa @theRealImy
(case I get the push running) how would then for example https://github.com/solid/solid-logic/blob/main/src/util/UtilityLogic.ts#L138 look like ?
const containerMembers = this.store.sym(url); possibly ? we have to consider the variable name, the passed parameter, the syntax.
as baby programmer I need some guidance. where would we find other occurences of URIs to be changed to named nodes ?
talking to Timea we found this could be a good first issue for me.

@ewingson a few lines above in same code you see a good example: https://github.com/solid/solid-logic/blob/8239044333ed2a06ca09596760e4783f6c95c85a/src/util/UtilityLogic.ts#L115
You can worry about making this consistent in solid-logic and then I can take it over and propagate it to other repos where they are used.
Basically, you can even change the name of variables if really really needed. However, I expect most work to be a chnage from string to NamedNode type and that is all.

Beware solid-logic is also used in the https://github.com/solid/test-suite https://www.npmjs.com/package/solid-logic

see https://github.com/SolidOS/solid-logic/pull/38
please review.


Revert functools.wraps to functools.update_wrapper

Description
Context: I recently noticed that the majority of tests in azure-keyvault-certificates and azure-keyvault-secrets haven't been getting collected by pytest since the morning of 11/16/21. Specifically, no tests with decorators have been getting collected: compare the 161 collected tests for azure-keyvault-secrets on 11/15/21 with the 31 collected tests on 11/16/21 when executing the same command.
It turns out that these tests stopped getting collected by pytest after this change to AbstractPreparer.__call__ was made, replacing functools.update_wrapper with functools.wraps. This change was made while migrating mgmt-plane tests to the test proxy (https://github.com/Azure/azure-sdk-for-python/pull/21746) because decorated tests weren't being collected when using the former method, but were collected when using the latter. I wasn't able to reproduce the error that necessitated this change today, so it's possible that a secondary error (which had since been fixed) was causing the collection issue.
This reverts the function change. functools.wraps is equivalent to a partial call of functools.update_wrapper, so this change shouldn't break tests (especially considering that functools.update_wrapper had been used up until the 11/15/21 change).
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

N/A Pull request includes test coverage for the included changes.

/azp run python - keyvault


Fix undefined var problem in isObserve()

When I tried to use the coap proxy server, I got a curious error.  I created a proxy and target servers in separate processes, based on the example proxy code, and then used the coap-cli module to construct a message to the target through the proxy (i.e., I added the proxyUri option).  The message and response worked, but shortly after the client received the response, the proxy and the target got locked into an endless loop of debug like this:
Target receives [request is not defined] in port [8976] from port [6780].
I tracked the problem down to isObserve(), where it appears request is being used instead of packet, and which is also not even testing anything because it doesn't return a value.
All of the unit tests pass in spite of this.  Apparently, that's because it takes too much time for the error to percolate up, and I guess none of the tests check for unexpected requests anyway.
I could use some advice on the test I added.  I had to disable the sinon fake timers to get it to pass (otherwise, it would hang forever), which is probably bad form.  I'm sure there is a way to use them and make it work, but I'm not familiar enough with them to know how.

This is breaking unit tests on travis: can you check? https://travis-ci.org/mcollina/node-coap/builds/108278145

The error is an unhandled ECANCELED in send that happens as a result of calling server.close() in afterEach.  It happens only after the "resend notifications in an observe connection" test.
I don't know what exactly triggers an ECANCELED, but if I do a clock.restore() before the test, it doesn't happen.  Is this a timing thing?  I see the same error on a bunch of previous commits, but I don't see how it has been fixed in the past...

Well...that didn't work.
The problem seems to be that sendObservation opens a new socket via the request() mechanism, and somehow that socket is breaking after the end of the observable test.  In particular, strace shows that there are asynchronous ACKs going on from the observer to the proxy and from the proxy to the target.  I think ECANCELED happens when it breaks during the ACK send, and I think the "not running" error happens when the socket breaks and then an attempt to send the ACK is made.
I think the call to done() needs to be somehow hooked into the finish event of the server and the target, but my attempt to do so just doesn't work (I get finish after one ACK and then never see another finish again).
I give up for now.  Maybe I will look at this again one day.

It seems I have been working on this exact same problem (I forgot to check PRs before starting), as you can see in PR #104. The isObserve() function was quite wrong, as it was making a strange or of objects. I modified it and checked with Wireshark this time to ensure the interchanged packets were the appropriate ones.
The ECANCELLED problem remains, anyway, but not in all Node versions. I haven't been able to reproduce it in v0.10.42, while it's easily reproducible with versions v0.12.10 and above. I'll try to fix the errors in my PR considering yours.

This raises another interesting question, BTW. What versions of Node.js is node-coap expected to support? 0.10 to 5?

This should support node v0.10 -> v5.

@dmoranj does your pr supersedes this?

@mcollina Yes it does. I'm facing the exact same problem with the ECANCELED, and the solution for the isObserve() problem needs for some other corrections to work properly. We can continue the discussion there if you prefer.

Concerning the supported versions of Node.js, I would suggest limiting to v0.10 -> v4, as, even skipping the whole proxy test suite,  there are plenty of errors runnning v5. v0.12 and v4.2.4 on the other hand, seem to run the tests, if we skip one faulty test in the proxy code. I would suggest to disable v5 from Travis and add an issue for the v5 support, as it may have some more work to do than simply fixing a couple of bugs.
Working in the proxy tests, they seem to have some interaction with tests in other suites (maybe something is left open or some value is globally changed) as, with @zacwbond solution it seems tu run fine on its own, but failed when running along the other tests. Concerning the referred solution, it really works but I'm afraid I don't understand why (as I'm not quite sure what the error is in first place). I'm sure the best solution would be to refactor the test but I'm facing the same problem with the finish events, so it doesn't seem clear to me how to modify that. I'll be trying to make things work on versions up to v4, and wait to hear your opinion about v5.

@dmoranj I don't have any bandwidth for this ATM.

@mcollina Then I think it's better to temporarily disable v5 while we work on the integration. I have disabled it in the other PR, and I'm going to create an issue right now. I think I may have some time to look at this this week.

I'm also closing this PR, as it completely overlaps with #103


hud not shown until method next view controller pushed

I have something like this:
-(IBAction)longProcess
{
[SVProgressHUD showWithMaskType:SVProgressHUDMaskTypeClear networkIndicator:NO];
[self takeLongTime]
[self.navigationController pushViewController:viewController animated:YES];
}
And the hud is dismissed by the navigationController delegate with

(void)navigationController:(UINavigationController *)navigationController willShowViewController:(UIViewController *)viewController animated:(BOOL)animated
{
[SVProgressHUD dismiss];
}

However, when do (IBAction)longProcess, the hud doesn't show. A split second before viewController is pushed, I see the hud appear and disappear.

Hi, because the takeLongTime method probably blocks your main UI thread nothing is updated before the lengthy work is done. You need to execute the heavy method on another thread. This is really easy in iOS:
Replace [self takeLongTime] with this:
[NSThread detachNewThreadSelector:@selector(takeLongTime) toTarget:self withObject:nil];
Now the method is executed on another NSThread and the UI thread can complete its work and view your viewController.
I don't see why you dismiss the HUD in the Navigation Controller delegate?? A better way would be to dismiss it when the heavy work is done, like this:
-(void) takeLongTime {
//Some heavy code...
dispatch_async(dispatch_get_main_queue(), ^ {
[SVProgressHUD dismiss];
});
}
Also note that your threaded method is not necessarily executed on the main thread and therefore we need to wrap the call to SVProgressHUD in dispatch_async() as done in the example above.
Hope that helps!

@magsto is spot on, @autodidakto. Although I would personally use GCD over NSThread:
[SVProgressHUD show];

dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^(void) {
    // heavy code
});

[SVProgressHUD dismiss];

Thanks guys. This was my first need for threads and you cleared it right up.
The nav controller delegate idea was due to all-nighter coding and other bad ideas :P

@lernfolio the GCD code I posted earlier sounds right for your use case:
[SVProgressHUD show];

dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^(void) {
    // heavy code

    dispatch_async(dispatch_get_main_queue(), ^ {
        // this code will be executed only once the heavy code is done
        [SVProgressHUD dismiss];
    });
});

@Sam:
In some cases this worked, icy you do some Backgroundtasks.
For me this helped:
Function1:
[SVProgressHUD<EMAIL_ADDRESS>networkIndicator:YES];
[self performSelector:@selector(function2) withObject:self afterDelay:0.1];

Function2:
//theheavycode
I split the call in two functions, "[...afterDelay:0.1];" finally leaves some time for the UI to do the things.
Now your wonderful Library is back in business!
Thanks for that!


[Intel MKL]: Dequantize Op

This PR adds support for Dequantize Op using MKL-DNN.

Hi @penpornk , could you please review this PR? also if you can review #26585, it will be great. Thanks!

@penpornk Thank you so much for reviewing this. I have addressed all the comments.

@penpornk I have addressed the missed comment. Thanks!


Unit Test - cmdImp package

Determine a list of go files and functions in those files for which we can write unit tests, create one test file per go file
Go files under cmdImp package:
rootImp.go
shared.go
undeployImp.go

close as those files removed.

@lzbj rootImp.go and undeployImp.go are removed, but shared.go still exists.  Will change title to reflect this.


Start-SPOSiteRename cmdlet is missing in docs

Start-SPOSiteRename is lacking documentation alike other similar cmdlets (see #1386)
There is no scaffolding page either on GitHub under office-docs-powershell

@nikrampi Thank you for this feedback
I will get this over to the PowerShell team so we can get this documented.
I was able to get this into the backlog for the PowerShell team. The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

While documenting I've found that the cmdlet is not working properly (issue #3086) ...

@VesaJuvonen  if you find it suitable please merge this PR

@j0rt3g4 Thank you very much for the contribution and updating the documentation with PR. @nikrampi Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@officedocsbot close


Verify that ${} is supported

Error happened while parsing a "variable name" in paraconf ie: ${test%3d_2} abort the program.
This is a copy of jbigot/paraconf#1

Mentioned in commit e11f3febef1ffd75ec858bad34be75216f3efb7d

In GitLab by @croussel on Nov 14, 2016, 16:39
Mentioned in commit 43ec133a5cc2c12f4a5b56196ada04b04b27501b

We need a sample code to debug this. We also need to add it to tests.

Status changed to closed by commit e11f3febef1ffd75ec858bad34be75216f3efb7d


Hyphenated words

First of all, thanks for trying to tackle this problem :)
I played around a bit with harper and one issue I stumbled upon pretty quickly is hyphens.
For example, pre-existing is not in the dictionary, but it's also not possible to add it to the dictionary, as pre is detected as its own word.
It seems that there's currently no support for well-known hyphenated phrases.

This should actually be a pretty quick and straightforward fix. I'll give an update here once a release is pushed out. Thanks for letting me know!

The changes are made, but I am a little hesitant to create a release until the changes I've got stashed are finished. As before, I'll update here once it's truly ready.

Alright there's a release with the tentative changes available now. Do me a favor and try out some other hyphenated words? We may need to expand the dictionary a bit.

As per #107 this has turned out to be more complex. I've decided that the prefix should remain it's own token. These prefixes just need to be added to the dictionary as necessary. If there are any other prefixes that need to be added, please feel free to reopen this ticket and I'll make the changes promptly.


Update to Julia 1.8

Also update the Manifest.toml and add pluto_now() to run without system image.

Codecov Report
Base: 0.00% // Head: 0.00% // No change to project coverage :thumbsup:

Coverage data is based on head (4e426d5) compared to base (2813884).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@          Coverage Diff          @@
##             dev     #23   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          3       3
  Lines         42      43    +1
=====================================
- Misses        42      43    +1

Impacted Files
Coverage Δ

src/HelloJulia.jl
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Set up placeholder image for all non-working images

Figured out #10
Did not do full verification but the fix worked locally.
I took way to long trying to add the onerror function in the original creation of the cards.

Looks good! Thanks @stuin


trip_id bugfix & SIGTERM graceful shutdown

This PR fixes a bug in our ingestion application that resulted in trip_id values from GTFS-RT vehicle position files not being included in parquet files.
This PR also adds adds ECS graceful shutdown functionality to streamline CD deployments.
Asana Task: https://app.asana.com/0/1203655790984044/1203958635792900
Asana Task: https://app.asana.com/0/1203655790984044/1203958635792903

added entity_is_deleted for BUS_VEHICLE_POSITIONS to PR.

LGTM


there was a bug processing patients with multiple expected values for…

… multiple measures associated iwth them. The code broke out of the loop instead of continuing to the next item in the array. there was also an issue where some expected_value items were empty dictionaries. This is now handled.

this bug was found on the production database on bonnie-review. user account<EMAIL_ADDRESS>measure CMS72


Add bot whitelist

This updates the default logic for deciding on server side render to
handle a whitelist of bots. Eventually we may want a more robust solution
to this problem, but the whitelist should suffice for a temporary fix.

!merge


Is this working with Phonegap Build?

I'm trying to use this plugin with Phonegap Build.
But always get this error message:

Error - The following 3rd-party plugin is causing the build to fail and may need to be updated to a newer version: com.omarben.inappreview

It's not tested with phonegap build but should work. I will try it and let you know.

Thanks!
Would love if it works.

Is the plugin added to your config.xml?
like:
<plugin name="com.omarben.inappreview" spec="~0.0.5"/>

Looks good but not working for me unfortunately.
Using:
<plugin source="npm" name="com.omarben.inappreview" />
Causes build error:
Plugins/com.omarben.inappreview/SKstorereview/SKstorereview.m:30:6: error: use of undeclared identifier 'SKStoreReviewController'
[SKStoreReviewController requestReview];
                 ^

Suspect Phonegap Build does not support the latest SDK yet.


Downloaded 1.0.1 Windows 10 64 Bit does not run , please help

Hi I have downloaded the binary zip , unpacked it and run striprouter from the command line and from explorer and nothing happens.
Am I supposed to regeister the  glew.dll or do anything else ?
No errors and no messages.  Would appreciate any help or direction

I tried it now, and the 1.0.1 version doesn't work for me either, on current Windows 10. However, version 1.0.0 works for me. I've deleted the 1.0.1 version for now. Let me know if you have trouble with 1.0.0.


Adding CP features to ggplot visualization

Also adding CP+DP features and the F1 score notebook

Results summary

Feature space
Number of top scoring phenotypes

CP_and_DPtest
10

CPtest
4

DPtest
2

F1 Score results

PR Curve results

@roshankern - I also had a question: what is Weighted? Should I include this class in the F1 score bar chart?

@roshankern - I also had a question: what is Weighted? Should I include this class in the F1 score bar chart?

Weighted refers to the weighted F1 score (the mean F1 score that takes into account occurrences of each label). I think this is good to include as it gives an idea of the overall performance of the model.

Another small comment: consider changing the final pr curve figure to 3 rows x 5 comments instead of 4x4 format. I think this would look better for a final figure, especially since there are 15 models. This is nitpicky, so feel free to ignore.

There might be something to address in this PR now that the threshold values changed for multiclass models in #26.

Another small comment: consider changing the final pr curve figure to 3 rows x 5 comments instead of 4x4 format. I think this would look better for a final figure, especially since there are 15 models. This is nitpicky, so feel free to ignore.

This is a very good suggestion.
I've also implemented all other suggestions, I am going to merge!

Weighted refers to the weighted F1 score (the mean F1 score that takes into account occurrences of each label). I think this is good to include as it gives an idea of the overall performance of the model.

One last thing - i changed this label to increase clarity


SyntaxError after Restarting with reloader in Windows

For some reason it appears the reloader is attepting to launch mapproxy-util.exe with the Python interpreter, instead of just executing mapproxy-util.exe
Had a look at the source code in mapproxy\util\ext\serving.py but could not figure out what it was doing wrong.  Problem encountered in version 1.9.0 and still present in latest development tarball:
C:\app\mymapproxy>ver

Microsoft Windows [Version 6.1.7601]
C:\app\mymapproxy>python --version
Python 2.7.11 :: Continuum Analytics, Inc.
C:\app\mymapproxy>..\mapproxy_venv\Scripts\activate
(mapproxy_venv) C:\app\mymapproxy>mapproxy-util --version
MapProxy 1.10.0a0.post20161130
(mapproxy_venv) C:\app\mymapproxy>mapproxy-util serve-develop mapproxy.yaml
[2016-11-30 09:43:35,720] mapproxy.config - INFO - reading: C:\app\mymapproxy\mapproxy.yaml
[info]  * Running on http://<IP_ADDRESS>:8080/ (Press CTRL+C to quit)
[info]  * Restarting with reloader
  File "C:\app\mapproxy_venv\Scripts\mapproxy-util.exe", line 1
SyntaxError: Non-ASCII character '\x90' in file C:\app\mapproxy_venv\Scripts\mapproxy-util.exe on line 1, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details

I am also having the exact same problem. Any information on how to fix this would be greatly appreciated.

Previous versions of PIP created mapproxy-util.exe and mapproxy-util.py. The reloader called python mapproxy-util and this loaded the .py file. More recent versions of PIP only created .exe files, but they were still Python files (or somehow directly executable by Python). I've updated MapProxy to call  python mapproxy-util.exe. Apparently PIP changed the way it installs scripts again.
@stanleyck are you using Anaconda as well?

@olt: I'm not using anaconda. I'm just using python 2.7, and followed the install instructions as given in https://mapproxy.org/docs/1.9.0/install_windows.html. Running on Windows 7, 64bit.

I tried reinstalling python, and now using version 2.7.9, and now it works!

I had same problem and it works too on python 2.7.9 (not with python 2.7.12), thanks @stanleyck

I can also reproduce this problem.  I was able to get everything working while keeping Python 2.7.12 and Mapproxy 1.9, but I had to rollback virtualenv to 15.0.3 (was 15.1.0) and setuptools to 26.0.0 (was 28.8.0).

There is a new 1.9.1 release which should fix this.

Thanks @olt :smile:


SOLR-16691 : Use Jackson for JSON serialization

WIP
Do not merge

I could be mistaken, but iiuc most normal writes (what we've been benchmarking in the related dev@solr discussion) run through here.
I wonder in that case how much of the speedup we've observed from this change might simply be related to bypassing the shim OutputStream that only overrides write(int byte). Put another way: what would be the impact of simply overriding write(byte[] buf, int offset, int length) on that shim OutputStream?
writeStrRaw is basically equivalent to both the gen.writeRawValue(String) and gen.writeRaw(String) methods. The distinction between writeRawValue() and writeRaw() is I think a bit of an impedance mismatch between Solr's TextWriter API and the Jackson JsonGenerator API -- the clue is in the no-op methods for writing separators in WriterImpl in this PR, and I think the consequence is that because we're no-opping those, we should always use gen.writeRawValue() from the context of Solr. Related: I also note that gen.writeRawValue(String) says nothing about wrapping Strings in quotes -- in fact the gen.writeRawValue() methods don't know the type context at all, so I think that may requires a change in the else clause of writeStr(), to:
    public void writeStr(String name, String val, boolean needsEscaping) throws IOException {
      if (needsEscaping) {
        gen.writeString(val);
      } else {
        gen.writeRawValue("\"");
        gen.writeRaw(val);
         gen.writeRaw("\"");
     }
    }

I don't have any means to compare the performance.

You can run the wget command and see the transfer time, its speed and the amount of data sent.
wget --report-speed=bits --server-response -O /dev/null --header='Accept-Encoding: ' --post-data 'indent=false&q=*%3A*&rows=1000000&wt=RESPONSE_WRITER' http://SOLR_IP:8983/solr/TEST_COLLECTION/select
Replace SOLR_IP, TEST_COLLECTION with your values, and run the command with various RESPONSE_WRITER (json or javabin or smile or jacksonjson). The collection documents should be large enough and the number of rows should be such that at least several gigabytes are transferred from the database. This will allow you to see the difference in speed. I tested on different collections and data, it always turned out faster from 1.7x to 7x. The disk and the network must be fast (for example, NVME disk or ram disk), otherwise the difference will not be noticeable due to long data reading from the disk or network limitations. To avoid a bottleneck in the network, the command can be executed on the Solr server by replacing the SOLR_IP with <IP_ADDRESS>.

@Fikavec , I'm not sure if there is a release planned for 8.11.2 . if yes, we will try to incorporate this. However, We can create a patch and make it available for anyone who wishes to use it

TestRandomFlRTGCloud is failing in this PR. @magibney  Can you give me any hint as to why it is failing ?

I can take a closer look tomorrow, but (esp. based on the fact that you tagged me) I suspect this will be around handling of the "json" raw doc transformer ... just have to figure out how to translate the special handling of raw json values to the new context.

@noblepaul take a look at https://github.com/apache/solr/pull/513 -- in particular, the RawShimTextResponseWriter which together with TextResponseWriter (from which it is invoked) actually handles flipping raw transformer fields over such that TextWriter.writeStrRaw() will be called. I think you need an analogous multiplexing (flipping between raw and non-raw) somewhere in JsonResponseWriter.
Again it's possible that any concrete performance improvement might be largely a consequence of avoiding this, which IMO looks really really wrong, and would be the first thing I'd change, even if pursuing using Jackson for JSON serialization anyway.

looks like the  problem was here
@magibney


ci: actions bump

Ci actions updated to v3
Checklist

[ ] This PR makes changes to the public API
[ ] I have included links for closing relevant issue numbers

thanks for contributing!


WIP: CircleCI 2.0

Migrate to CircleCI 2.0 as 1.0 reaches EOL in August.

Codecov Report

Merging #235 into master will decrease coverage by 3.56%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #235      +/-   ##
==========================================
- Coverage    74.3%   70.73%   -3.57%
==========================================
  Files           7        7
  Lines        2440     2440
  Branches      487      487
==========================================
- Hits         1813     1726      -87
- Misses        454      545      +91
+ Partials      173      169       -4

Looks like we are good to go here:
https://96-1551431-gh.circle-artifacts.com/0/html/index.html

Thanks @larsoner


Chip styles not working

I now have a problem with Chips styles,
since 2 or 3 months the chip styles look very bad on my side
We updatet rmwc but the problem remains on every browser exept firefox mobile
there the styles work perfect

It’s likely related to some local styles you have interfering. Can you provide a repro?

The Issue is located in \src\modules\cash_register\itempick\cash.js
https://github.com/knarxy/mpg-sas2020

That’s a private repository.
If you can reproduce your issue using the provided code sandbox I might be able to be of some help. It’s unlikely to be a problem with RMWC and more likely to be an issue with how you have it setup, or are importing the chip styles.

ok thanks


New release into Rolling

@asorbini Could you please do a new release of rmw_connextdds into Rolling?  That should help us get the packages building with all of the recent changes for unique network flows and the like.  Thanks.

I should also add: if you don't have time for it today, just let me know.  I can do the release as well.

@clalancette I was hoping to merge #22 before a new release, and bump the version number to 0.5.0, but from test results and @ivanpauno's review, I see I still have some work on that, and probably won't be able to merge it before tomorrow at the earliest, to allow for any additional review (assuming I'll be able to get the code in final shape today).
If you can do the release today I would appreciate it. Maybe bump the version to 0.4.1? (or 0.5, and we'll go to 0.6 after #22)
Thanks!

Sounds good, I'll go ahead and do it.  Numbers are cheap, we can always bump again :).  Thanks for the reply.

@clalancette I'm going to let CI run overnight to validate the recent changes (#22, #28). If everything is as expected, I'll go ahead and create 0.6.0 tomorrow.

CI was green/as expected, going ahead with 0.6.0 release now.
For reference:

nightly_linux_release:
nightly_osx_release:
nightly_win_rel:
Rci__nightly-connext_ubuntu_focal_amd64:

Tagged and bloomed 0.6.0 (ros/rosdistro#29138)


Terminal doesn't activate selected conda environment

Type: Bug

Behaviour
Expected vs. Actual
I am using Windows 10 and Anaconda for virtual environments.
recently, I change from spyder to vs code. it works fine.
But yesterday, when I started a new terminal or run  a python file, new terminal did not activate the selected virtual env (Conda).
It DO work before. But didn't after I installed the follows on Friday.

New Python (3.9.9) outside of Anaconda to test a simple web server.
WSL 2 and Docker for RATH (https://rath.kanaries.net/).

currently, I tried to uninstall New Python (3.9.9) and reinstall vs code (after delete everything, it's a clean new install), but still doesn't work.
I know this is the same issue as #20885, but I tried all solutions mentioned in # 20885 and failed.
XXX
Steps to reproduce:

Install new Python (3.9.9) outside of Anaconda to test a simple web server.
WSL 2 and Docker for RATH (https://rath.kanaries.net/).

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.7.10
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Conda
Value of the python.languageServer setting: Default

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

User Settings

condaPath: "<placeholder>"

languageServer: "Pylance"

terminal
• executeInFileDir: "<placeholder>"

Extension version: 2023.16.0
VS Code version: Code 1.82.2 (abd2f3db4bdb28f9e95536dfa84d8479f1eb312d, 2023-09-14T05:55:25.390Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
Intel(R) Xeon(R) Gold 5122 CPU @ 3.60GHz (8 x 3592)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
62.66GB (54.99GB free)

Process Argv
--crash-reporter-id 72a488d0-fc85-4685-952b-366d42b0aba1

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
py29gd2263cf:30792227
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
3biah626:30602489
89544117:30613380
showlangstatbar:30737416
962ge761:30841074
03d35959:30757346
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
pythonnosmt12:30797651
pythonidxptcf:30805731
pythonnoceb:30805159
copilotsettingt:30839829
synctok:30821570
dsvsc013:30795093
dsvsc014:30804076
diffeditorv2:30821572
pythonmpsinfo:30842941
dsvsc015:30829745

I found a solution, although I don't know what's the reason for this issue.
I added "conda activate " as
"${env:HOMEDRIVE}${env:HOMEPATH}\anaconda3 & conda activate "
Before this revision, it's just
"${env:HOMEDRIVE}${env:HOMEPATH}\anaconda3"
The full related setting:
"terminal.integrated.profiles.windows": {
    "Anaconda Prompt": {
        "path": [
            "${env:windir}\\Sysnative\\cmd.exe",
            "${env:windir}\\System32\\cmd.exe"
        ],
        "args": [
            "/k",
            "${env:HOMEDRIVE}${env:HOMEPATH}\\anaconda3\\Scripts\\activate.bat",
            "${env:HOMEDRIVE}${env:HOMEPATH}\\anaconda3 & conda activate <my-env-name>"
        ],
        "icon": "terminal"
    }

Please provide the debug logs as mentioned in the template:

Run Developer: Set log level command
Select Python -> Select Trace
Run Developer: Reload Window command
Provide the output:

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

@karrtikr
Thanks a lot for the reply.
Please refer to attached PDF for the output.
As mentioned above, after added "conda activate my-env", it works fine.
It's output in this case in Pages 1-43,
Output.pdf
I deleted  "conda activate my-env" and again terminal did not activate selected conda environment.
please find output in this case in Page 44-87.
Sorry for the long log.
Hope this be useful to locate the issue.
Thanks again.

Thanks. Couple things:

Try installing pre-release version of the extension and reload window:

If that doesn't help, can you send your User settings? I'm wondering if you have shell integration turned off.

@karrtikr
I switch to pre-release version and reload window, but it doesn't work.
It's still base env, but not selected virtual env.

I am not quite sure what kind of User setting you need.
Anyway, just send settings.json firstly.
If you need other info, please let know.
Thanks.

Hi @suipz
This is a very old problem that I have been facing at least on macOS  since I started using virtual environments.
One working fix to this problem that I found was to change the "code-runner.executerMap" in settings.json under the key "python" to "$pythonPath $fullFileName".
I found setting "python.terminal.activateEnvironment": false; to be efficient in programs that take user input. Because it keeps on running till the environment gets activated (program takes this as the user input, thus causing a bug).
Although it does not solve the problem but all the code will run with the environment as selected in the interpreter path.
I understand it is a macOS issue, but if you use code-runner, this should solve the problem nonetheless.

The reason for this was, terminal would activate my environment after a delay, due to which the program runs without the environment being activated (it gets activated after that). So, in my setup, I found selecting a default helps in removing the lag and hence the environment gets activated before the code is run. Cannot say the same for Windows-10 but worth a try.

@suipz Can you also remove the "args" parameter activating conda base env from User settings? Alternatively please switch to default powershell profile using Select default profile command and create a new terminal.

@the-psychedelic
Thanks for your reply.
I can solve the issue by user settings as mentioned above.
I just wonder why.

@karrtikr
I tried again.

Use "terminal.integrated.defaultProfile.windows": "Anaconda Prompt", but remove 'args'.
It looks fine if I start  a new terminal, selected virtual env activated. If I [Run Python File] from the button at right-upper editor after started a new terminal, It's also no problem, although the python file was run in a different terminal.
however, if I [Run Python File] directly from the button at right-upper editor without manually started new terminal, it did not work since no selected virtual env activated. I mean, after I started vscode, I need to start new terminal once manually to activate virtual env. This cannot be done automatically when [Run Python File].

Use "terminal.integrated.defaultProfile.windows": "PowerShell", it's same situation as No.1 above.

Appreciate you giving it a try. If I understand correctly, once args is removed:

The issue is fixed for any new terminals created
The first terminal or existing terminal which shows up does not have the environment activated

if I [Run Python File] directly from the button at right-upper editor without manually started new terminal, it did not work since no selected virtual env activated.

I'm not very clear on this bit. I would ideally expect all terminals to behave the same.
Can you dispose all terminals by clicking the 🗑 icon and try it? Would really help if I could get some screenshots on how you're inferring whether the correct env was activated.

Btw note, if you hover over terminal's name, it should also tell you which environment is currently activated:

@karrtikr
If I start a terminal by click Terminal menu/New terminal. It's OK and selected virtual env (py37) activated.

But if I don't do it, alternatively run python file directly after started vs code, library cannot be loaded since it's not installed properly under base env.
(if virtual env activated, it's no problem to load library.)

BTW, it's showed that py37 activated in the 2nd screenshot, but it's not in fact.

This is the screenshot to run the python file successfully if I started terminal in advance.

Thanks for the deets, we're very close. I'm able to reproduce the issue but under very specific conditions.

alternatively run python file directly after started vs code

Note environments take some time to activate (we don't a good job surfacing this, something we can improve), I think if you wait long enough before running the Python file (or opening a new terminal), you should not see the ⚠ icon or this issue.
Also, this should only happen the first time an env is selected (and reasonable time is given for it to activate). Once you dispose all terminals with ⚠ icon and reload window (Reload window command), the issue should not appear as long as py37 is selected.

If that still does not work for you, please provide the logs again https://github.com/microsoft/vscode-python/issues/22068#issuecomment-1732455320 after doing the steps needed to reproduce the issue.

@karrtikr
I tested again. Yes, just as you said, it's no problem after waited for 15min.
maybe no need to wait 15min. I just did something else.
Anyway, thanks a lot for your confirmation.
This issue could be closed now, I think.

No worries! I'll just make the quick fix to show a progress indicator to help you with how long it takes to re-activate the environment, after which we can close this. Thanks for the help.

@karrtikr
Sorry, I am new to github.
Do you mean I could do something from #22082 to know the time for the activation of environment?
I don't know how to do. Please give me some guidance. Thanks.

Yep, you'll see an indicator on the bottom left which indicates how long it takes for activation an environment. Fix should be out in the pre-release version of the extension in a hour, use the following to try it out:

Once you reload the window after installing it, you should be able to see the indicator.

@karrtikr
Thanks a lot for your help.
I switched to pre-release version and found it's very fast to re-load window. Maybe only 1 or 2 sec.
I even cannot see the indicator.
Also, I close the vs code and start again to try to reproduce the issue.
But I failed to reproduce it with pre-release version.
And then I switched back to release version and tried again.
I can reproduce the  issue, and re-activation of environment need about 5-10 sec.
I think it's not a long time.
That's all. Thanks again.


[MB-1284] Clean up output for GHC pricing parser

Description
The output for the GHC pricing parser CLI was quite verbose and could be hard to follow.  This PR uses the open-source pterm library to help style the output.  I also did some refactoring along the way to help make the output more consistent and keep the code DRY.
Reviewer Notes

I'm glad to walk through some of the features I use from pterm.
In general, the output should be showing the same information as before, but hopefully now easier to visually follow.

Setup

make clean server_build bin/ghc-pricing-parser
make server_test to ensure server tests still pass for parsers.
See this PR for instructions on running ghc-pricing-parser and pricing-import.
Be sure to try running ghc-pricing-parser both with and without the --display option.
Ensure that the same number of records were created in the rate engine tables for this branch and master.  Spot-check a few rows if you have time.

Code Review Verification Steps

[ ] Code follows the guidelines for Logging
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Messages

:book:
:link: MB-1284

  Generated by :no_entry_sign: dangerJS against 5d6115f941b4a8c93f2be4ced5b24e67758fc46b


Cant change projectile of Dragon arrows?

With Twisted bow, I transmogged the dragon arrows to be harpoonfish, but for some reason the first shot will work, but the shots after will revert back to the normal animation. I don't know if this is a bug with the quiver, or with enemies that are bigger than 1xq.

do you either a) have a f2p option to replicate this bug? I tried cursed goblin bow on hill giants and it works fine. or b) are there enemies on which it does work?

It works for me. Here's my settings


minor ui + utility bugs fixes

skeleton-loader added	in Our Store
cart model updated with real-data with image,	price and name
table	header and cupon removal for empty cart
footer contact-us refactor used in the header-nav section

Fixes #236

Deployment has failed, please check by running npm run build and debug accordingly @SandipGyawali

Deployment has failed, please check by running npm run build and debug accordingly @SandipGyawali

@pooranjoyb i guess it's fixed

@SandipGyawali your branch has merge conflicts in Navbar.tsx and Shop.tsx` kindly resolve, otherwise it can't be merged

Also resolve the conflicts @SandipGyawali


CORE-2572: Do not explicitly set content-type in result coming out of log filter

Addresses issue for:
"Explicitly set HTTP header 'Content-Type: application/json' is ignored, explicit `Content-Type` header is not allowed."

Somehow, the logging filter re-adds the content-type that had previously been removed.
Tested this while running proxy locally

Seems like a bit of a hack. Is the filter actually adding the content-type header or is it coming from somewhere upstream?

@mbryzek Context here is that WARN log messages are being thrown that show:
"Explicitly set HTTP header 'Content-Type: application/json' is ignored, explicit `Content-Type` header is not allowed. Set `HttpResponse.entity.contentType` instead."

and up to 40 million+ of these are being written to the logs daily. these warning messages aren't useful and they eat up our credits with sumo: https://service.us2.sumologic.com/ui/#/search/U24LcAfzx33vwXYCkuKtEDe3SmPj9M7gRAO5uIjP
@benwaffle i tried to narrow down where this happens and couldn't find in the code where we re-add it. note we already attempt to remove it here:
https://github.com/flowvault/proxy/blob/3ab950fdf856a6cc07b8a4d8b5932761dcb4dd4d/app/handlers/GenericHandler.scala#L146
https://github.com/flowvault/proxy/blob/3ab950fdf856a6cc07b8a4d8b5932761dcb4dd4d/app/lib/ProxyRequest.scala#L299
Note that for this one:
https://github.com/flowvault/proxy/blob/3ab950fdf856a6cc07b8a4d8b5932761dcb4dd4d/app/handlers/GenericHandler.scala#L152
I already tried not explicitly passing the content-type in the headers, but somehow it still gets re-added by the time it gets to the filter.

@benwaffle just to prove it happens somewhere in the LoggingFilter, i tried removing loggingFilter: LoggingFilter from https://github.com/flowvault/proxy/blob/main/app/filters/Filters.scala#L17 and the warning went away. Somehow, the content-type was being added back in LoggingFilter


null out pythonpath

fixes #233
cc @cs01 @laixintao @jayvdb

Let me fix the test before merging. Let me know if there’s an easier way to test this.

@laixintao I fixed the tests for python 3.6, added a changelog entry for <IP_ADDRESS>, and bumped the version correspondingly.

A force push was recently done, so this PR will have to be rebased onto origin/master. Apologies for the inconvenience. See https://github.com/pipxproject/pipx/issues/270.

@cs01 @michaeljoseph @laixintao just bumping this PR. tests now pass. anything left to fix?

Looks good to me! Thanks


Add alertdialog for survey open or close.

[ ] Add an "alertdialog" to show when survey isn't available on the day.
[ ] Add an "alertdialog" to show when survey is completed on the day.

Delete the old branch, and create a new branch from "AdaptsBranchV3" named "liying-alertdialog-survey"

Liying, is. this. completed already? I believe it is. If you say yes then I will close the issue.

Completed already. ...


util.inherits not working in Webpack

https://github.com/Richienb/node-polyfill-webpack-plugin/issues/57
https://github.com/browserify/node-util/issues/80
(This module uses the same one that I have been)

I think this should be covered when you polyfill both util and inherits and use bundler functionality for browser environment (as is mentioned in comment). Do you have minimal reproduction where this doesn’t work? I have tried locally and it works as it should
Other than that, I think proxying module and reimplementing util.inherits to use inherits/inherits_browser should be way to go.

cc @barnabasbartha


Fix archiving crash

Fix the archiving crash describes in issue #32 by archiving structs manually.

Nice! This looks good to me, merging.

Your fixes have been released in 2.2.5, thanks for contributing!


Livereload reloads whole page even if only the css has changed

I guess this has more to do with ember-cli (or even broccoli) but it impacts the effectiveness of live-reload.
When changing the css, and only the css, livereload signals the whole page to reload, since ember-cli will recompile the whole app and dump it on the dist folder again.
Do you know if there's any plans to support partial recompiling, so we can get the best of livereload? Reloading CSS only is a pleasure when it comes to style difficult-to-reach scenarios.
Thanks!

I think this issue can be closed.

Thanks @lukemelia !


no textures

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
I've been using a no texture fast flag for a while now and had been working fine. But now it hasn't, I've tried different versions but still hasn't worked. May I know what's the problem and how to fix it.

https://github.com/pizzaboxer/bloxstrap/wiki/A-guide-to-FastFlags#old-material-textures


support offset/size parameters to ::get() and drop stream_copy_to_stream in httpget?

Hi,
i'm implementing a virtual file system using \FSExt\File und thus i provide the get() method which shall return either a file resource or the requested content string.
now my problem is that the requested file is not hosted locally, so i need to grab the given file by myself from a remote host using a custom api. but since i dont have offset and size given, i need to query the whole file and provide just the slice requested by the dav client. in my case this is a pretty overhead cause of big files.
so i would kindly like to ask for the following changes:

add $offset=0,$size=0 parameters for get() calls so one can provide the parts needed
in server.php in function httpGet the function stream_copy_to_stream is used on a temporary php://temp because a few lines below httpResponse->sendBody() doesnt handle parts of the body - a simple solution would be to just change sendBody($body) to sendBody($body, $size=0) and just send $size bytes if greater zero

i could provide the code changes if above modifications get approved or any other thoughts on how to handle this?
thx

No feedback


Input decorators, obscure number and CVV, validations

Added input decorators, some simple card validations, optional obscure card number and CVV, this has #21 and #13 as a base but its improved in many ways, fixes #17, fixes #16, fixes #15, fixes #9, fixes #2

Hi @simformsolutions ,
This is a great addition to your package, please merge.
Thanks

Waiting to change the English labels to other languages. :)


Add tests for duplicate semantic tokens

Contributing a Pull Request
If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check.
Fixes #1428

Raised https://github.com/OmniSharp/csharp-language-server-protocol/issues/516 to try and figure out the exception in the semantic tokens test.

Does this work ok with O# 19 now?


Update out of date dependencies

npm WARN deprecated<EMAIL_ADDRESS>This module has moved and is now available at @hapi/joi. Please update your dependencies as this version is no longer maintained an may contain bugs and security issues.

Thanks for your fast response, when do you have planned to release these changes?

About a minute ago 😄. Check out v2.0.1.


Readonly point2d, point3d, point4d throws an error

I'm trying to monitor a vector(3) in my project, but I'm getting errors when setting readonly: true in my binding. I don't have much experience with this library yet so possibly I'm just using it wrong...
const pane = new Pane();
const PARAMS = {
  offset: {x: 50, y: 25},
};
pane.addBinding(PARAMS, 'offset', { readonly: true });

{
    "message": "No matching controller for 'offset'",
    "name": "TpError",
    "stack": "Error: No matching controller for 'offset'\n    at new TpError (https://tweakpane.github.io/docs/assets/tweakpane.js:85:22)\n    at PluginPool.createBinding (https://tweakpane.github.io/docs/assets/tweakpane.js:7290:15)\n    at RackApi.addBinding (https://tweakpane.github.io/docs/assets/tweakpane.js:1694:31)\n    at Pane.addBinding (https://tweakpane.github.io/docs/assets/tweakpane.js:2210:30)\n    at <anonymous>:4:6",
    "type": "nomatchingcontroller"
}

I have the same problem...

I have not yet implemented 2D/3D point monitoring because I was unsure about the demand for them.

It would be cool to have it work, so for example I would like to use this to output the actual position of a 3D model (vector3) in my scene.
This will definitely be helpful :)