Persist events instead of simulation state

This issue is concerned with refactoring the OpenDC simulation model to persist events instead of simulation state.
At the moment, at every second in the simulation, the state of machines and task is recorded and written to the database. However, this means that even when a task or machine never changes during the simulation, we do need to record all these states and write them to the database for this entity. This will cause a single simulation to write a huge amount of rows to the database. A benefit of this approach is that getting the state for a single point in time can be accessed quite quickly.
The model should be refactored so it writes events that occur to the database instead of these states. This means that entities that do not have any events associated with them will not occupy space in the database. A downside of this approach is that getting the state for a single point in time requires replaying the events starting from some checkpoint. This change will also require significant changes in the database as well as the current frontend.
See more about Event Sourcing here:

https://martinfowler.com/eaaDev/EventSourcing.html
https://docs.microsoft.com/nl-nl/azure/architecture/patterns/event-sourcing

This feature will be implemented in the 2.x release.
In addition, we are currently looking into InfluxDB for persisting fine-grained machine and task states.


Documentation for the possible error codes?

Looking through https://github.com/palantir/tslint/blob/master/test/executable/executableTests.ts:

0 means either no errors or --force is on
1 means no arguments were passed
2 means lint errors were found

Can this be documented somewhere? It'd be nice as reference.

💀 It's time! 💀
TSLint is being deprecated and no longer accepting pull requests for major new changes or features. See #4534. 😱
If you'd like to see this change implemented, you have two choices:

Recommended: Check if this is available in ESLint + typescript-eslint ✅
Not Recommended: Fork TSLint locally 🤷♂

👋 It was a pleasure open sourcing with you!
If you believe this message was posted here in error, please comment so we can re-open the issue!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


multi-class

Hello, i was wondering if node-svm can perfom multi-class classification by itself, or do i need to perform it using 1-vs-1 or 1-vs-all. This is the example of a dataset:
[
 [[ 262, 333,<PHONE_NUMBER>.249186, 25.64758, 0.431, 0 ], 1],
 [[ 262, 326,<PHONE_NUMBER>.26529, 25.64758, 0.431, 0 ], 1],
 [[ 262, 326,<PHONE_NUMBER>.282071, 25.64758, 0.431, 0 ] 1]
.....

This a short example of the features of one class, and i have many many classes, thousands to be exact, and every class has around 20 examples or features.
Im using node-svm with:
Classifier: CSVC
C: [0.03125, 0.125, 0.5, 2, 8],
kernelType: 'RBF',
gamma: [0.03125, 0.125, 0.5, 2, 8],
nFold: 5,
normalize: true

When i train a class what i do is, i select the same number of negative samples from other classes has i have positive from the new class, and train the classifier with positive and negative samples, thus making the classifier binary.
My question if i need to do this, or i can just train node svm with all the classes all together and the implementation alone already does this for me.

You should use built-in multiclass support with the entire dataset.
Note: each class must be represented by an unique integer. Example:

person0 => 0
person1 => 1
person2 => 2
...

Since the number of feature is small (6) you should disable PCA:
Classifier: CSVC
C: [0.03125, 0.125, 0.5, 2, 8],
kernelType: 'RBF',
gamma: [0.03125, 0.125, 0.5, 2, 8],
nFold: 5,
normalize: true,
reduce: false

Thank you for your help, i am already getting better results!! But if i am constantly adding new classes, i have to constantly be training a new classification model? Any idea on how can i improve this a bit
Thanks for your help

I think there is no other solution when using SVM.
Maybe you should take a look to other machine learning techniques (see http://scikit-learn.org/stable/supervised_learning.html#supervised-learning)
Best regards


Added ability to free

Adds a free function so you can use nfd from a dll on windows that uses a different version of msvcrt and thus a diffrent heap than your main application.
Cheers and awesome library! :)

I'm aware of the static library limitation of NFD; thanks for taking a step in the right direction.  Did you manage to update the Premake scripts to optionally generate DLLs?  It seems that this PR makes sense in the context of a build feature that doesn't yet exist.
I'd happily take a build script PR that adds DLL support.

Hey, I'm wondering if you want to weigh in here on my question.  I'm going to release the next version of NFD in a couple weeks; we've had some updates I'll be moving from the devel branch to master.  If not, no worries.


How to update both the domain and subdomain through docker compose?

Hello! As mentioned here https://github.com/mietzen/porkbun-ddns/issues/19 , multiple concurrent updates seem implemented.
However, after setting up docker compose with the environment variables as so:
"DOMAIN=my.domain",
"SUBDOMAINS=*",
only the A record of *.my.domain is getting updated , while my.domain is not. I am not very tech savvy, so there's probably something I am missing here.

Try:
SUBDOMAINS: "@,my_other_subdomain,my_subsubdomain.my_subdomain"
For more Infos, see: https://github.com/mietzen/porkbun-ddns/issues/7#issuecomment-1597079658

As I am not really an expert on DNS, could you explain what the @ sign means? Indeed, everything seems to work now, when I specify the @ sign for the subdomains environment variable. Before I just left it empty.

The "@" symbol is a placeholder used to represent "the current domain" as in root domain.
@ A free standing @ is used to denote the current origin. [0]
It's handy if you want to set the root domain and subdomains. But I also tested it without subdomain, you should also be able to leave it empty.
[0] RFC1035 - Page 35

I updated the readme, if you have still have trouble updating the root domain please reopen the issue.

thank you so much! Indeed, this fixed it :)

Is there a way I can use the "@" to configure my root domain, while still using the wildcard? I tried "SUBDOMAINS: "@,*" and it didn't seem to help.

This works, just test it:
version: "3"
services:
  porkbun-ddns:
    image: "mietzen/porkbun-ddns:latest"
    container_name: porkbun-ddns
    environment:
      DOMAIN: "my-domain.com" # Your Porkbun domain
      SUBDOMAINS: '@,*' # Subdomains comma spreaded
      SECRETAPIKEY: "xxx" # Your Porkbun Secret-API-Key
      APIKEY: "xxx" # Your Porkbun API-Key
    restart: unless-stopped

porkbun-ddns  | 2024-04-02 17:22:12,094 DEBUG    Create new record, with:
porkbun-ddns  | {"name": "my-domain.com", "type": "A", "content": "<IP_ADDRESS>"}
porkbun-ddns  | 2024-04-02 17:22:13,797 INFO     Creating A-Record for my-domain.com with content: <IP_ADDRESS>, Status: SUCCESS
porkbun-ddns  | 2024-04-02 17:22:16,222 DEBUG    Create new record, with:
porkbun-ddns  | {"name": "*.my-domain.com", "type": "A", "content": "<IP_ADDRESS>"}
porkbun-ddns  | 2024-04-02 17:22:17,937 INFO     Creating A-Record for *.my-domain.com with content: <IP_ADDRESS>, Status: SUCCESS
porkbun-ddns  | 2024-04-02 17:22:19,338 INFO     Sleeping... 300s

Please test it again, if it still fails open a new issue including the content of your docker-compose.yml file and the log output of docker compose up

Oh, weird! Will try it again, maybe I just messed something up somewhere. Thanks for the quick response!

I had it set in reverse before ("*,@"), but setting it like in your example seems to have worked! Weird! Thanks for the help.

@kandykarter yes that is a bug, I fixed it in #49


Getting "Cannot find module 'zone.js/dist/zone-node'" when deploying on production server

Hi there,
Thank you for the great starter template!
I have been using it to create application and when i'm about to deploy to production server i get "Cannot find module 'zone.js/dist/zone-node'. The steps that i did was:

yarn universal
copy dist/ and dll/ folder to server
Runnode dist/server.js
The fuill stacktrace is as below
Error: Cannot find module 'zone.js/dist/zone-node'
at Function.Module._resolveFilename (module.js:476:15)
at Function.Module._load (module.js:424:25)
at Module.require (module.js:504:17)
at require (internal/module.js:20:19)
at Object.zone.js/dist/zone-node (/var/.../myapp/dist/server.js:9138:18)
at webpack_require (/var/.../myapp/dist/server.js:20:30)
at Object. (/var/.../myapp/dist/server.js:8647:130)
at Object../src/server.ts (/var/.../myapp/dist/server.js:8711:30)
at webpack_require (/var/.../myapp/dist/server.js:20:30)
at /var/.../myapp/dist/server.js:63:18

I can run it on the generated server.js in local but not on server. Any help will be appreciated. Thanks!

Hi,
I have same problem did you find a solution ?

Hi Samuel,
I ended up performing yarn install on server to have the package installed on server.

Hi,
When we run node dist/server.js I am getting following error:
Error: Invalid provider for undefined. useClass cannot be undefined.
Usually it happens when:
1. There's a circular dependency (might be caused by using index.ts (barrel) files).
2. Class was used before it was declared. Use forwardRef in this case.
at syntaxError (C:\Users\nagaraajuk\Desktop\New folder\dist\node_modules@angular\compiler\bundles\compiler.umd.js:287:21)
Node Version: 9.11.0
Webpack version: 4.0.0
NPM version: 5.6.0
Please help us to resolve the above issue.
Thanks.

@nagarajukmenlo unfortunately your solution did not help. we get still the same error

@nagarajukmenlo @tonihuang
I have the same problem. The problem started when i was trying to fix another problem . My universal app kept failing during build but doing the following in webpack.server.config.js allowed the app to build. But cause the same problem you guys described above.
const nodeExternals = require('webpack-node-externals');
module.exports = {
...
   externals: [nodeExternals()],
...
}

Did you guys find a solution? are you still working on it?


chore: add workflow dispatch

Also make version from a single file instead of multiple places

This PR added VERSION file on the top directory of this repository.
This breaks goenv on case-insensitive file system environment, such as macOS.
❯ go help
goenv: version '2.0.8' is not installed (set by /Users/skaji/.goenv/version)

Oops! Will fix it.

@skaji please pull latest!

@ChronosMasterOfAllTime Thanks! Now it works again.

@ChronosMasterOfAllTime Thanks! Now it works again.

Awesome; I always forget about case insensitive FS 😂


client.CreateV4SignedUrl() does not work when object_name == ""

In storage/tests/client_sign_url_integration_test.cc
client.CreateV4SignedUrl() does not work when object_name == ""
The actual url is different than the expected url.
EXPECT_EQ(expected, *actual);

fails.
Related to #2250

Can you be more specific about what it does wrong when object_name is empty?

@remyabel
sorry, just updated the ticket.

Can you please give me an example? This is what I get from testing:
https://storage.googleapis.com/thisisatest1234567
?x-goog-signature=...
&x-goog-algorithm=GOOG4-RSA-SHA256
&x-goog-credential=service-account%40project.iam.gserviceaccount.com%2F20190327%2Fus-east1%2Fstorage%2Fgoog4_request
&x-goog-date=20190327T162007Z
&x-goog-expires=600
&x-goog-signedheaders=host

formatted for readability. It looks pretty much identical to output from gsutil.

@remyabel
This is related to a new PR #2250 that has not been merged yet.
If you look at the new file client_sign_url_integration_test.cc that is under review at line 148    EXPECT_EQ(expected, *actual); fails for object_name == "" ( line 112)

There are two potential causes I think:

Either your assert is firing at line 147 EXPECT_THAT(*actual, HasSubstr(object_name));, or:
The test case with no object specified, i.e https://storage.googleapis.com/test-bucket/ is tripping it up (notice the trailing slash. The example I showed has no trailing slash)

For what is worth, we should check if the signed URL created by the C++ client library works when object_name == "".  If it does, then it is probably a bug in the expectations from the .json file.  If it does not work, then we need to fix our library.

@remyabel
Just commented out
 EXPECT_THAT(*actual, HasSubstr(object_name));

and it still failed.
I cannot see anything at
https://storage.googleapis.com/test-bucket/?X-Goog-Algorithm
I get

Try removing the trailing slash. Neither the library nor gsutil output a trailing slash after test-bucket.

After rereading the .NET code for a couple of hours I think there might be a bug (or oversight). Please check my math:
This is the code where we are getting the JSON data:
https://github.com/googleapis/google-cloud-dotnet/blob/b1c4be61412aeb6e261e2be38c568678b9ed73f3/apis/Google.Cloud.Storage.V1/Google.Cloud.Storage.V1.Tests/UrlSignerTest.V4SignerTest.cs#L114
And these lines are where we decide whether or not to append objectName:
https://github.com/googleapis/google-cloud-dotnet/blob/b1c4be61412aeb6e261e2be38c568678b9ed73f3/apis/Google.Cloud.Storage.V1/Google.Cloud.Storage.V1/UrlSigner.V4Signer.cs#L144-L148
The problem I think is that since it is checking for null instead string.Empty, and because object is "" instead of NULL, it's appending a trailing slash.

@remyabel  you are right
if (objectName != null)

seems to be the problem. the string is empty but is not null.

FYI, I've opened a PR over there.

@remyabel it is possible that the error is in the JSON file that drives these tests, maybe it should be setting the object_name field to null here:
https://github.com/googleapis/google-cloud-dotnet/blob/b1c4be61412aeb6e261e2be38c568678b9ed73f3/apis/Google.Cloud.Storage.V1/Google.Cloud.Storage.V1.Tests/UrlSignerV4TestData.json#L116
@frankyn FYI.

Question, the trailing slash should work, could I get a full example that doesn't work?
https://storage.googleapis.com/test-bucket/?X-Goog-Algorithm is not a complete signed URL.

Question, the trailing slash should work, could I get a full example that doesn't work?

I apologize for the confusion. The question is whether or not there should be a trailing slash if object_name is empty or null. Currently, the .NET library only checks if it is null, so it outputs a trailing slash if the string is empty.
if (objectName != null)
{
    _resourcePath += $"/{Uri.EscapeDataString(objectName)}";
}

The C++ library checks for empty() and only outputs a trailing slash if the string is not empty.
if (!object_name().empty()) {
    os << '/' << curl.MakeEscapedString(object_name()).get();
}

gsutil does not output a trailing slash if you omit the object name. IMO, only checking if it's null is a logical error because the JSON value can either be null or "", hence my PR.
And the test data we are using is from the .NET library (linked to by @coryan above):
https://storage.googleapis.com/test-bucket/?X-Goog-Algorithm=GOOG4-RSA-SHA256&X-Goog-Credential=test-iam-credentials%40dummy-project-id.iam.gserviceaccount.com%2F20190201%2Fauto%2Fstorage%2Fgoog4_request&X-Goog-Date=20190201T090000Z&X-Goog-Expires=10&X-Goog-SignedHeaders=host&X-Goog-Signature=2a1d342f11ddf0c90c669b9ba89ab5099f94049a86351cacbc85845fd5a8b31e1f9c8d484926c19fbd6930da6c8d3049ca8ebcfeefb7b02e53137755d36f97baab479414528b2802f10d94541facb888edf886d91ba124e60cb3801464f61aadc575fc921c99cf8c52e281f7bc0d3e740f529201c469c8e52775b6433687e0c0dca1c6b874614c3c3d09599be1e192c40ad6827416e387bf6e88a5f501f1d8225bce498d134599d0dfe30c9c833c244d3f90cf9595b9f8175658b788ee5c4a90b575fde5e83c645772250c7098373ca754b39d0fc1ebca2f50261a015931541c9827920eba67a1c41613853a1bd23299a1f9f5d583c0feb05ea2f792ba390d27

So it seems to be a little inconsistent and if you could tell us the correct behavior, that would be much appreciated.

I chatted with a PoC for V4 Signature support in the GCS team about it.
There is no recommendation as they both should work.
Given gsutil does not have a trailing slash, let's at least be consistent with the CLI tool.
Thanks for raising this @rsimion and for clarifying @remyabel.

Assigning to myself for now at least until triaged.

@rsimion is fixing this as part of #2250.


How to use LinqKit  with LiteDB Find operation?

I use LinqKit  PredicateBuilder  for complicated  and/or filter  with linq to object syntax.
LiteDB  works fine in production with 2 thousands records but  slower significantly  with 7 thousands ones. I think LiteDB performance advantage is not Linq2Object  but  Query index-based documents.  How do I use LinqKit with LiteDB Queries?
BTW:  I have built Index on filter field before importing data to LiteDB  as,
col.EnsureIndex(x => x.Id, true);
col.EnsureIndex(x => x.Gender);
col.EnsureIndex(x => x.Name, "$.Name[*]");

and abbreviated entity is ,
[BsonId]
public Guid Id { get; set; }
public String Gender{ get; set; }
public List<String> Name { get; set; }

valuable suggestion. There is some real improvement. Thanks


<EMAIL_ADDRESS>does not support staticStyle yet?

I try to use  staticStyle  and  v-bind:style, staticClass and v-bind:class in a component,
But when I add the style property it does not work out ， Is it because this feature removed it? but why v-bind:class and staticClass is support ?
I see the style of unit testing, there is no such use cases,  but class of unit testing, has the cases
example
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>Vue.js spec example</title>

There's no more staticStyle, it's removed. Just use style.

ohh, I was in version 1.0 component, this is the use of style


readme: add python3 to dependencies

distutils isn't included in all base python3 installations (e.g. my Ubuntu WSL install didn't have it) and the V8 python deps step complains when it isn't there. I also added a reference to Bazelisk since, in my experience, that tends to be the recommended way to use Bazel at Cloudflare.

Hello, we can probably close this PR as changes containing pointers on using bazelisk and python deps have landed:

https://github.com/cloudflare/workerd/pull/578
https://github.com/cloudflare/workerd/pull/561

Thanks


[ENH] ResNet and MLP are missing the use_mini_batch_size parameter

Describe the feature or idea you want to propose
When ResNet, MLP and FCN were proposed, they were proposed with a batch size formula, check this line in FCN. This should be added to ResNet and MLP.
Describe your proposed solution
will add it myself
Describe alternatives you've considered, if relevant
No response
Additional context
No response

[ ] ResNet #433
[ ] MLP


How to Install?

Hi, I've tried to installl using the app gallery link, but when it opens in the Fitbit app it says that there is no connection, any suggestions?

Hi! There can sometimes be connectivity issues due to Fitbit's messaging internals. Best I can advise is  attempting to restart both phone and watch. Sometimes re-opening the app on the watch can help.
Please let me know if you're able to connect!

Hey Thanks for getting back to me.
I uploaded the code to fitbit studio in the end and compiled myself, which was actually pretty easy. I've only had my Ionic for about a week but am finding the whole app side of the fitbit a bit disappointing, having come from the Pebble Time Steel which was still really good. The authenticator was one of my most used Pebble apps, so I am really grateful that you have started this project!

Oh I misunderstood, thought you were running with a Versa. I currently have a beta build for the Ionic if you'd like to try that out:
https://gam.fitbit.com/gallery/app/b853f37e-1951-4720-88c3-07d1587f694d
I definitely share your misgivings (as a fellow former-pebbler) about the fitbit apps but I'm working with it best I can :D

I got it working on the Ionic emulator by changing adding Ionic as a build target in the project, haven't actually tried on the physical device yet, but assume it would work?
I was using an android wear device in between the Pebble and Ionic and started using https://www.authenticatorplus.com/ which is pretty good, a fitbit app for that would be great!
You mention in the Readme that if the app is uninstalled, the data goes as well. Is it going to be possible to create some sort of backup do you think? I only ask as i dont really trust the performance reliability of the Fitbit app, so can see my self reinstalling it at some point and dont want to loose my code/seed data! :)

No way of storing a backup at this time. I'm wary of getting into that as there are further security considerations with downloading files from the fitbit. That's why I suggest not using this as the sole source of token generation, personally I use a mix of this and other applications to store my MFA.

Yeah i totally get that, i work in IT support myself so am aware of the potential risks there! :)

I've updated the main app to now be available on Ionic as well!


How many Project should Application Layer have?

If our enterprise has 5 applications (example: clientWeb, clientMobile, OperatorWeb, logisticWeb,ReportMobile),
how many projects should Application Layer should have? If we use Clean Architecture
more than 5 projects in Application Layer + 5 in Presentation layer
Or
1 project in Application Layer + 5 projects in Presentation ?
Because I just see 1 projects in Application + 2 In Presentation (this template)
Thanks

Typically I recommend having one Solution per Application that you ship/deploy. And this is a solution template. So if you have 5 applications I would have 5 solutions each with Core/Infrastructure/Web. If you find having a distinct application layer valuable, then add that as well - total of 4 (non-test) projects per solution.
But what about shared code!
That's what a SharedKernel project is for, which should be in a separate solution and should publish a NuGet package that the other solutions consume. If it's early days and you're just getting started, a high-churn NuGet package is going to slow you down, so in early rampup of the project it's fine to just add it as a class library to your first solution or two. But once things get a bit more stable, you'll want it to be just a NuGet reference rather than a class library.

Thanks for your recommend!


Live Stream PubSub Notifications

Can I use the library in order to notify viewers of my live stream that the new video segment is ready for download?

No probably not. You probably want to send an email for that.

@alexhultman Maybe I put it wrong. I have segments of live stream, each 1 second length. I have a p2p network in which these segments are shared. As soon as a new such segment is generated I want to notify all py peers that they can download it

@cyrillicw Look:
https://github.com/uNetworking/uWebSockets/issues/1048


Cannot mock laravel contracts in constructor of commands using mockery

Laravel Version: 5.4.24
PHP Version: 7.1.5
Database Driver & Version: mysql  Ver 14.14 Distrib 5.7.18

Description:
Not sure if I am making a basic mistake. I can mock Laravel contracts and facades that are resolved by the Laravel container in the handle() method but not the constructor of a class that extends Laravel's Illuminate\Console\Command class.
Steps To Reproduce:
Basic mockery mock;
use Mockery as m;
use Illuminate\Contracts\Filesystem\Factory as File;

class myClassTest
{
    public function my_test()
    {
        $mock = m::mock(File::class);
        $mock->shouldReceive('put')->once();
        $this->app->instance(File::class, $mock);

        $this->artisan('call-my-command');
    }
}

Using the code below and inspecting $file in the constructor gives Illuminate\Config\Repository
use Illuminate\Console\Command;
use Illuminate\Contracts\Filesystem\Factory as File;

class MyClass extends Command
{
    public function __construct(File $file)
    {
        parent::__construct();
        $this->file = $file;
    }

    public function handle()
    {
        $this->file->put('/here.txt', 'stuff');
    }
}

Using the code below and inspecting $file in the handle() method gives Mockery_3_Illuminate_Contracts_Config_Repository as expected
use Illuminate\Console\Command;
use Illuminate\Contracts\Filesystem\Factory as File;

class MyClass extends Command
{
    public function handle(File $file)
    {
        $file->put('/here.txt', 'stuff');
    }
}

I am injecting other classes into handle() so I would like to keep the Laravel contracts being injected into the constructor. It works when running the command - its just the mocks that are not working in the constructor.
What am I missing?

Please ask on the forums. This repo is for bug reporting only.


level=error msg="Could not unmarshal JSON: unexpected end of JSON input"

Haven't pried into the code about it but I'm pretty sure I put the account and token in. Any thoughts why this is happening?

Not sure why Oanda won't return JSON, or an error or something i'll have to check their enpoint

I realized It's because I don't have a v20 account...

Thanks for the report! I need to improve the error handling in this package so it is clearer.
Please use this package for the older REST API.


Eliminates backpointers from RuntimeData structures.

This cleanup is a step toward eliminating the double-linked non-sharing behavior
in the schema compiler, because maintaining these double-linked pointers
for the runtime was forcing them to exist in the schema compiler also.
Turns out the runtime didn't need them except to check an Assert in one unparser. That check is removed, and all the backpointers can go away from RuntimeData and friends.
DAFFODIL-1444

Nevermind. Broken idea.


Distance function bias seems off, picks wrong colors sometimes

I use a bias when calculating distance between colors, seems a bit wrong, the program tends to favor the wrong color sometimes

original

srcery-ized

:D
Takes ~0.03s to convert

neat, that was quick too. I tried some 4k images and that took a good while. Colors don't seem that off either. Calling that a success ;)

have you run with --release?
[rustemb@machine srcery-convert]$ time cargo run  -- ~/Pictures/Saturn.png Saturn.png
    Finished dev [unoptimized + debuginfo] target(s) in 0.03s
     Running `target/debug/srcery-convert /home/rustemb/Pictures/Saturn.png Saturn.png`

real	0m16.007s
user	0m15.846s
sys	0m0.034s
[rustemb@machine srcery-convert]$ time cargo run --release  -- ~/Pictures/Saturn.png Saturn.png
    Finished release [optimized] target(s) in 0.09s
     Running `target/release/srcery-convert /home/rustemb/Pictures/Saturn.png Saturn.png`

real	0m0.732s
user	0m0.648s
sys	0m0.029s

No never actually tried that :D, that quite a bit of improvement I must say


PR for testing - ignore

PR for issue
What is being addressed
Describe the current behavior you are modifying.  Please also remember to update any impacted documentation.
How is this addressed

Describe the changes made, and if appropriate, why they are addressed this way
Note any pending work (with links to the issues that will address them)
Update documentation

/help

/help

/help

/help

/help

/help

/help

/help

/help

/help

/help

/help

/help

/test

/force-approve

/test

/test


infer

Can you provide code for reasoning demonstration?

Sorry, what do you mean by reasoning demonstration? Chinese is good here.


chore: Upgrade dependency on github.com/jenkins-x/jx-api/v4

Also ran go mod tidy
Fixes that jx secret removes issueProvider from jx-requirements.yaml

/ok-to-test
/lgtm

/hold

Just ran go mod tidy on main branch, and did not remove anything ... which version of go are you on @msvticket?

I have 1.17.2.

can you downgrade to 1.15 (or use gvm to switch the version)? I have a few PRs open to upgrade to 1.17 soon though ...

/hold cancel


CSP breaks injection

Content Security Policy: The page’s settings blocked the loading of a resource at inline (“script-src”).
Because the contentScript.js injects a payload into the user-space to access the Three objects, CSP settings can prevent inline script injections. No immediately solutions come to mind for debugging arbitrary three scenes, although we might be able to tag the script such that site owners can make an exception to the injected code (maybe by not inlining it and instead reference the allowlisted extension source, maybe an SRI attribute although that'd change for every release).

Looks like react devtools has the same problem, although possibly only with Firefox, bugzilla/1267027


MLJFlux super slow / stackoverflow

using MLJ

N = 10^4
y = rand([0,1], N)
x1 = Float32.(y .+ 1)
df = DataFrame(y = y,
    x1 = x1,
    x2 = x1,
    x3 = x1,
    x4 = x1);

y, X = unpack(df, ==(:y), !=(:y); :y=>Multiclass{2});

CLS = @load NeuralNetworkClassifier
nnc = CLS(epochs = 1)

mach = machine(nnc, X, y)

@time fit!(mach)
 48.405561 seconds (101.52 M allocations: 6.914 GiB, 2.21% gc time, 0.46% compilation time)

and it stackover flows if you set N=1850786

@Moelf Thanks indeed for reporting. This behaviour is terrible.
Have identified the culprit in the way one-hot encoding is being done. Working on a fix now.


[5.x] Bump yarn version

Is your feature request related to a problem?
Yes, it's a yarn bug
Describe the solution you'd like
$ yarn config unset httpsProxy
Unknown Syntax Error: Extraneous positional argument ("unset").

The yarn version in Docker image is too old. See yarnpkg/berry#2667
Describe alternatives you've considered
yarn set version latest

to update the yarn version

I've tried already to upgrade to yarn 3 but breaks all test, I don't want to invest time on this because master brach uses pnpm and works fine, the 5.x branch has expire date.


align the week data without messing up everything

Hi Matt,
Really thanks for this great plugin.i am unable to load huge record(over 300000 records).take much time to load.any idea?.after loading the calendar if i change the year (using next and previous button)  data still binding.how can i clear it?

This plugin isn't designed for that many events, trying to do anything with 300000 items in angular will cause performance issues (the recommended maximum amount of watchers is 2000). I'd suggest changing your UX to support filtering of events as you'd never need to display that many to the user anyway.


Add Fedora-based dev-containers

For use within a Fedora-based environment, starting with F#.

Thank you for the review, @samruddhikhandale. I've addressed the issues you've raised.


Create Dockerfile

https://github.com/MakiShire/hello-github-actions/tree/MakiShire-patch-1/action-a

https://github.com/MakiShire/hello-github-actions/tree/MakiShire-patch-1/action-a


fix: redis instrumentation startup stream check #666

Which problem is this PR solving?
All work done by @jonbcampos-alto in #666 :imp:
Short description of the changes
Added the fix for the lint and recreated the PR to close #666 and get the fix merged.

Enough people approved #666, so I'll take the freedom to merge this.


missing sourcemap?

Hi there,
I have an Electron (11) project in which Im using an opensource library named electron-store which requires the library conf which requires finally ajv.
It seems that on the last updates something happened with the sourcemap of ajv and I started to get the following errors:
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/ajv.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/context.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/validate/dataType.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/rules.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/validate/applicability.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/errors.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/codegen/index.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/codegen/code.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/codegen/scope.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/names.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/util.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/validate/index.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/validate/boolSchema.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/validate/iterate.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/validate/defaults.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/validate/keyword.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/vocabularies/code.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/subschema.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/resolve.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/uri-js/dist/es5/uri.all.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/core.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://./Applications/MyApp.app/Contents/Resources/app.asar/node_modules/conf/node_modules/ajv/dist/compile/error_classes.js.map: Unexpected end of JSON input

Is there a configuration fix/workaround for that (beside going into electron's devtools and clearing or filtering these warnings)?
What version of Ajv are you using? Does the issue happen if you use the latest version?
this issue happens on the versions mentioned here below, note that ajv is a depndency of a dependency of a dependency 🙃
"electron-store": "7.0.2"
----------------------------- > requires "conf": "^9.0.0"
-------------------------------------------------------------> requires "ajv": "^7.0.3", "ajv-formats": "^1.5.1"
Your code (usage inside electron-store -> conf library)
https://github.com/sindresorhus/conf/blob/ce1dcee195e95cbd71493b1dfd6efcf7fe185a51/source/index.ts#L97-L108
Thanks!

Seems like it happens when the library is imported inside a web worker (Bundled with webpack). Any ideas?

Looks to me like this is actually an issue with the conf library. It uses version 8.1 of ajv. I think it needs to be updated to use 8.2+.

@MattLish did something related to the issue changed in versions greate than 8.2 ?

Any progress？

Has anyone fixed this issue? If so, may you explain how?

Looks to me like this is actually an issue with the conf library. It uses version 8.1 of ajv. I think it needs to be updated to use 8.2+.

This should not matter. It's using a ^ semver specifier, so it should receive the latest minor version of Ajv 8.

 Any update on this one?  Has someone found a trick of any kind?

i have the same problem

I have found a workaround mentioned here. In short, one can either create a symlink to make the path reachable or copy the source map files over.
Symlinking would be a better solution obviously but that caused a memory leak as the path would be full circle. So i just copied over, something along the lines of:
mkdir -p public/home/myuser/projects/my_test_repo/
rsync -R $(fd .js.map)  public/home/myuser/projects/my_test_repo/

Did this ever get fixed? I'm having issues with it.
electron-store 8.1.0
conf 10.2.0
ajv 6.12.6


which version can be used in win 10 x64 ?

I am not sure what are you asking for.

I am not sure what are you asking for.

My programe can NOT get the right TypeInfo from the OCX,
So I search the tlfin32.dll on my system,
I get 3 version of this file,
Is this important for the programe to run ?


Running nbqa with flake8 complains about blank lines on notebook cells with functions

Consider a cell containing a function
# cell contents
def get_dataset():
    return make_classification(
        n_samples=1000, n_features=20, n_informative=15, n_redundant=5, random_state=7
    )

I am getting the warnings

E302 expected 2 blank lines, found 1.
E305 expected 2 blank lines after class or function definition, found 1

I'm not sure how we could consistently put the correct number of blank lines between cells...I'm tempted to just have a "suggested configs" document in which we suggest which errors to ignore for which tools. Or do you think this is solvable?

I was about to type exactly the same "suggested configs".

Is there any issue appending 2 spaces after each cell content in the temp python file and removing those spaces when writing back to the source notebook?

I think there was some issue with isort from doing that...would that solve the problem?

Adding two new lines seems to solve the problem. I tested the following code with isort
Before
#%%
import numpy as np
import pymc3 as pm
import arviz as az

#%%
from matplotlib import pyplot as plt
from scipy.special import betaln
from scipy.stats import beta

After
#%%
import arviz as az
import numpy as np
import pymc3 as pm

#%%
from matplotlib import pyplot as plt
from scipy.special import betaln
from scipy.stats import beta

isort removes a newilne when it sorts the imports. But this wouldnt be an issue if we are splitting by just #%%. After the split, we can strip \n from the cell contents. I think this should work.

After the split, we can strip \n from the cell content

yes but if isort makes any changes, then won't it (potentially) return the wrong exit code? e.g. like this it could happen that you run nbqa isort notebook.ipynb and you get
fixing notebook.ipynb

but then if you do git diff you see that nothing has changed

After the split, we can strip \n from the cell content

yes but if isort makes any changes, then won't it (potentially) return the wrong exit code? e.g. like this it could happen that you run nbqa isort notebook.ipynb and you get
fixing notebook.ipynb

but then if you do git diff you see that nothing has changed

Great point. I didn't think isort would change the file when no imports has to be sorted.


Switch the neighborhood structure according to search status

The swap neighborhood for selection constraints which are automatically detected seems to interfere to find a feasible solution in the early stage of the optimization.
A strategy to switch neighborhood according to search status is preferred.
A concrete  idea is to enable the swap neighborhood if and only if the current solution is feasible.

The issue that optimization with the swap neighborhood for selection fails to find feasible solutions was caused by the inappropriate neighborhood detecting algorithm. It was fixed by #20.


fix: use same version

Thanks, good catch,
Can you amend your commit message to be: "Use Elixir 1.13.4 for the prod build stage"

Done

Cool, thanks again for the patch!


Added many organizations

Summary

Now the user can have many organizations and the organizations have projects
Edited the nav bar to show only the correct menu
Added link action to github

TODO: Review flows
TODO: Analyze if the github information for the organization should be in another model

:+1:


Configurability of P-Graph Widget @fussel178

From the looks of it, this is outdated. Feel free to re-open if you want.


NFT ids cashing

This component provides info on which NFT ids are owned by which addresses.

@quantumagi fixes applied


Proof of concept: Allow reloading automation without restarting HA

Proof of concept! Not for merging
Depends on the automation entity PR, hence a PR against that branch to just show changes.
This PR adds a new service to the automation component to allow reloading automations without restarting Home Assistant.
This is a proof of concept as it now depends on a copy of the config validation code of bootstrap. This should probably be replaced with the extracted function that @kellerza is working on in #2912
Adds the following changes to helpers:

entity has a new method remove() which removes itself from the state machine
entity component has a new method reset() which removes all its entities and cancels its listeners.

Checklist
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Any idea what happens to automations that are running when this service is called?

I would hope the world implodes in such a case :-P
They get turned off https://github.com/home-assistant/home-assistant/pull/3002/files#diff-13395835104ee1e614bcafd45602a496R288

Any actions will finish but they won't fire again

The PR was still pointing at the other automation branch when I pushed an update that was rebased on latest dev so it had a week worth of dev in it for a couple of minutes. Looks like Farcy found some issues that slipped in ;-)

This PR is ready for review.

Still needs a services.yaml description file.
Is there any way to provide feedback on reload? (persistent_notification / browser push / event where you can roll your own)

It's currently not providing any feedback on reload. I guess we could add a persistent notification to the frontend if something failed to reload. However, that is for a future PR 👍

@kellerza - services.yaml added
@Teagan42 - I disagree with calling it reload_automations because it is a method of the automation component, which implies it only applies to automation.

Looks good and a very helpful feature! :dolphin:


Deprecates IEventLogController in favor of DI

Resolves #4242
The PR:

Deprecates the interface
Deprecates the implementation
The implementation beeing internal and the interface having a warning to not implement by 3rd parties, we can replace it's usage in other downstream classes without too much worry
I did only one usage replacement in RelationshipController as a proof of concept and adjusted the related test on it.
There are many many usages of this in Platform, so if thise PR is ok, we have a plan forward for the other usages, any feedback welcome.

Summary

@bdukes Does this make sense in the spirit of the issue comments ?


Setting header again appends to header, not replaces

Hi.
Related issue: #150
As title says, when again setting res.setHeader(key, value) appends to same header, not replaces.
It is bug? Or am i doing something wrong?

There is no res.setHeader, you only have res.writeHeader

Yep, setHeader in my app :)
I forgot to say, yes, same bug applies to writeStatus too


Move Draw Calls to Thread

move the sfml drawing to its own thread.
This should be done in GameBackboneExe
Test to see if this fixes the slow navigator movement on laptops.

Draw calls cannot be threaded out because OpenGL and by extension SFML cannot have the window used by different threads.
The update function can be threaded out, but this causes a severe dip in performance that is likely due to cash collisions between the draw, and update threads since the updateables and drawables are often the same.
Halting work on this for now.


Add ban/kick/timeout commands?

Adding ban, kick and timeout commands.

Timeout command already listed #43

Why make a ban command? Discord already has the rest of those features.


Ability to export chart image ?

Is it possible to generate or export chart created to png/jpeg through some method ?

WriteXLSX can create xlsx file which include charts, but doesn't have method to create chart images directly.


Downgrade bisect_ppx

Version 2.8.1 of bisect_ppx has an issue that results in a HTML report with missing coverages. This PR downgrades its version to 2.7.1.

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

The issue has been closed and a new patch version was released https://github.com/aantron/bisect_ppx/issues/409. Can we upgrade to the 2.8.2 instead?

I will test if the problem is fixed in this new version and change the PR to update it if that's the case.

Unfortunately, upgrading bisect_ppx requires a series of package version bumps that break the code (it boils down to upgrading janestreet libs, which have breaking changes), so it's a rather intrusive change.

The underlying issue here seems to have been resolved. Closing.


Content Categories mess

I think the whole IAB Content Categories is one big mess and I would suggest either amending them or changing the Content Categories system completely.
At the moment I'm implementing some OpenRTB stuff where I have to make a mapping between our Content Categories and those used in OpenRTB. Doing this I notice that the IAB Content Categories are missing some thing and have some very weird choices.
There is an "Automotive" category with a lot or really specific car and motorcycle related subcategories. There however is no category or subcategory for anything related to Boats, Trains or Aircrafts.
"Travel" contains subcategories like "Australia & New Zealand", "Africa", "Europe" but doesn't contain a subcategory for Asia. It does however for some reason contain a "Japan" subcategory as only country in Asia.
Another category or subcategory that is missing is anything that has to do with Weapons. For example websites for gun owner clubs, websites that sell weapons, websites about ancient weapons.

+1
IAB categories are one of worst classification systems I've ever used and practically useless for the variety of sites out there today.

I tried emailing the iab about this<EMAIL_ADDRESS>but they never replied.

@ErikDubbelboer im also working with IAB Content categories, and am looking for a parseable source (not PDF), do you know where I might find one? Id prefer not to copy-paste/scrape the PDF if possible, and i fear if i email<EMAIL_ADDRESS>i too will get no response.. thanks in advance for any pointers you can give, thanks

@farridav
Try this: https://gist.github.com/crowdmatt/5040911

@farridav because the IAB categories are such a mess we stopped using them.

Thanks @manigandham and @ErikDubbelboer, Ill use a static data source until we either stop using them, or they become maintained by the appropriate body


Header and footer

In the model I .......
-[ ] Added header to project
-[ ] Added footer to project

Good work. It is working properly. ✅
The menu items and the footer information correct.


Delivery for Map and App Gallery Template AGOL5.3

Implemented following gitHub tickets:
#217: Add support for vector basemap group
#221: Re-assess shared theme on mobile devices

@CTLocalGovTeam Please submit this pull request against the develop branch


getExperimentCrossCorrelation for sample pairs?

The current implementation of getExperimentCrossCorrelation calculates n x m correlation matrix between two input experiments.
There are many use cases, where we need to calculate correlation between sample pairs, when the columns of the two input experiments are paired. In that case, the calculation would return a correlation vector of length n instead, assuming that both experiments have n columns that are matched (by order, by name, or both). This is a subset of the full correlation matrix between two experiments, but remarkably faster to compute when only the paired correlations are needed.
Is it feasible to implement this as an additional argument in the getExperimentCrossCorrelation function? For instance, paired=TRUE or a similar argument. Or would it be better to implement an entirely new function? We would have an immediate use case in miaTime, if this can be added in one way or another.

Hmmm, yes I think the implementation should be relatively easy. However, I think we should avoid adding too much stuff to one function, because it makes it too complicated to use. Because this is as special case, we could add new function for this, but we can think about that

Implementation and maintenance will be easier if it is a separate function. Then we can cross-reference these two functions in the @seealso field.
Another important thing: calculating correlation is not sufficient, we will need to have support for general dis/similarity measures.
Perhaps a new function with a name getExperimentPairedCorrelation and/or getExperimentPairedDissimilarity.
(@YagmurSimsekk might like to wait that this is implemented, before optimizing the time divergence code)

I was starting to implement this and I realized that this function calculates correlations only for feature-pairs. For example, correlation between taxa and metabolites that share same samples.
Currently we do not have functionality that calculates correlations between samples. We should implement this, I create some kind of draft

Might be easiest to have this in a single function but I am not sure. You can consider something analogous like we have transformSamples and transformFeatures and transformCounts if this seems necessary/clearer to separate.


Don't extend form the wrapper class Object

Don't extend form the wrapper class Object as object is a reserved word in PHP >= 7.
This branch is based on tag 1.0.1 and should result in 1.0.2

@burzum I don't know when CakeObject was defined.
But I know, that object is a reserved word in PHP7 so using the Cake-Class will produce an fatal error.

I'm aware of this, CakePHP 2 and the according version of this plugin is just ancient. But I know that Cake2 was made php7 compatible. So I dislike the idea of introducing a BC breaking change.
I think this change can be worked-around by adding a class_exists('Object') check and if it exists alias it? I'm not sure if class aliasing was already available in 5.x?

Here is the code from CakePHP...
File: lib/Cake/Core/Object.php
App::uses('CakeObject', 'Core');
class_alias('CakeObject', 'Object');

As you can see, Object is already an alias for CakeObject.
So extending from CakeObject instead of Object will be the right way.

Thanks for checking this! I've merged it.


Line numbering with page scope fails if the page break results from previous content

Description
If you set the line numbering later in the document and the page break results from content before the #set, the line numbering is not correctly reset when flowing onto the next page.
#set page(height: 100mm, width: 200mm, margin: 15mm)
#lorem(100)

#set par.line(
  numbering: "1",
  numbering-scope: "page",
)

#lorem(200)

Results in

Reproduction URL
https://typst.app/project/ptwa1v6askfg7bvPktADwJ
Operating system
Web app
Typst version

[X] I am using the latest version of Typst

This is documented here: https://typst.app/docs/reference/model/par/#definitions-line-numbering-scope

Note: The line numbering scope must be uniform across each page run (a page run is a sequence of pages without an explicit pagebreak in between). For this reason, set rules for it should be defined before any page content, typically at the very start of the document.

Ideally there could be some diagnostic to identify this pattern, but it could always be legitimate to change the line number scope setting midway (if you're about to pagebreak), so I'm not sure it'd be worth it.

Ah, sorry, I was actually looking for something like that but missed the note section.
For anyone coming across this, this works as excpected:
#set par.line(
  numbering-scope: "page",
)

#set page(height: 100mm, width: 200mm, margin: 15mm)

#set par.line(numbering: none)
#lorem(50)

#set par.line(numbering: "1")
#lorem(200)


Avoid exception when registering shortcut

Napari does not allow registering an existing shortcut.
This can occur when:

Another plugin or Napari itself or the user has registered "Shift-L", "Shift-E" before opening napari-spatialdata
The user opened and closes napari-spatialdata multiple times in the same Napari session

As a consequence, Napari raises an exception and all code after shortcut registration cannot be used anymore.
This PR adds the optional argument override to silence the error. If you prefer override=True, I can change that (should have the same effect on avoiding the error).

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.11%. Comparing base (e9ad739) to head (5ab0e30).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #201   +/-   ##
=======================================
  Coverage   78.11%   78.11%
=======================================
  Files          17       17
  Lines        1819     1819
=======================================
  Hits         1421     1421
  Misses        398      398

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks @aeisenbarth, merging it. Please notice that the labels_key usage will be removed when the code from https://github.com/scverse/napari-spatialdata/issues/180, which is currently merged to the multi_table branch, will be merged to main.


HIVE-28360: Upgrade jersey to 1.19.4

What changes were proposed in this pull request?
Upgrade jersey to 1.19.4
Why are the changes needed?
HIVE-28360
After upgrading to Hadoop 3.3.5, the Hive WebHCat server fails to start because of inconsistent versions of the Jersey JAR package. Hive HCat lacks the jersey-server-1.19 jar.
Does this PR introduce any user-facing change?
None, that I am aware of.
How was this patch tested?
CI

mvn_dependency_tree.txt
Hi @lvyankui This is the dependency tree with this change and it would be a bit cleaner practice to exclude 1.19 versions which are shown in dependency tree as a part of org.apache.tez:tez-api:jar:0.10.3 and org.apache.atlas:atlas-client-common:jar:2.3.0.Otherwise this change looks good.

mvn_dependency_tree.txt Hi @lvyankui This is the dependency tree with this change and it would be a bit cleaner practice to exclude 1.19 versions which are shown in dependency tree as a part of org.apache.tez:tez-api:jar:0.10.3 and org.apache.atlas:atlas-client-common:jar:2.3.0.Otherwise this change looks good.

Thank you for your feedback. I have already taken care of it

@lvyankui have you verified the hcatalog ok with the latest change? seems like the https://github.com/apache/hive/blob/master/common/pom.xml#L214 still has direct reference to the old jersey

@lvyankui have you verified the hcatalog ok with the latest change? seems like the https://github.com/apache/hive/blob/master/common/pom.xml#L214 still has direct reference to the old jersey

@dengzhhu653 I have processed all the references. Please help review again, thanks!


Definitions of a variable share the same reference to the Var object

Consider this small function which produces the following IR
@njit
def calc(n):
    arr = [0]
    for i in range(1, n+1):
        arr += [i]
    return arr

---------------------------------IR DUMP: calc----------------------------------
label 0:
    n = arg(0, name=n)                       ['n']
    $const0.1 = const(int, 0)                ['$const0.1']
    $0.2 = build_list(items=[Var($const0.1, a.py (12))]) ['$0.2', '$const0.1']
    del $const0.1                            []
    arr = $0.2                               ['$0.2', 'arr']
    del $0.2                                 []
    jump 6                                   []
label 6:
    jump 8                                   []
label 8:
    $8.1 = global(range: <class 'range'>)    ['$8.1']
    $const8.2 = const(int, 1)                ['$const8.2']
    $const8.4 = const(int, 1)                ['$const8.4']
    $8.5 = n + $const8.4                     ['$8.5', '$const8.4', 'n']
    del n                                    []
    del $const8.4                            []
    $8.6 = call $8.1($const8.2, $8.5, func=$8.1, args=[Var($const8.2, a.py (13)), Var($8.5, a.py (13))], kws=(), vararg=None) ['$8.1', '$8.5', '$8.6', '$const8.2']
    del $const8.2                            []
    del $8.5                                 []
    del $8.1                                 []
    $8.7 = getiter(value=$8.6)               ['$8.6', '$8.7']
    del $8.6                                 []
    $phi22.1 = $8.7                          ['$8.7', '$phi22.1']
    del $8.7                                 []
    jump 22                                  []
label 22:
    $22.2 = iternext(value=$phi22.1)         ['$22.2', '$phi22.1']
    $22.3 = pair_first(value=$22.2)          ['$22.2', '$22.3']
    $22.4 = pair_second(value=$22.2)         ['$22.2', '$22.4']
    del $22.2                                []
    $phi24.1 = $22.3                         ['$22.3', '$phi24.1']
    $phi38.1 = $22.3                         ['$22.3', '$phi38.1']
    del $phi38.1                             []
    del $22.3                                []
    $phi38.2 = $phi22.1                      ['$phi22.1', '$phi38.2']
    del $phi38.2                             []
    branch $22.4, 24, 38                     ['$22.4']
label 24:
    del $22.4                                []
    i = $phi24.1                             ['$phi24.1', 'i']
    del $phi24.1                             []
    $24.4 = build_list(items=[Var(i, a.py (13))]) ['$24.4', 'i']
    del i                                    []
    $24.5 = inplace_binop(fn=<built-in function iadd>, immutable_fn=<built-in function add>, lhs=arr, rhs=$24.4, static_lhs=Undefined, static_rhs=Undefined) ['$24.4', '$24.5', 'arr']
    del $24.4                                []
    arr = $24.5                              ['$24.5', 'arr']
    del $24.5                                []
    jump 22                                  []
label 38:
    del $phi24.1                             []
    del $phi22.1                             []
    del $22.4                                []
    jump 40                                  []
label 40:
    $40.2 = cast(value=arr)                  ['$40.2', 'arr']
    del arr                                  []
    return $40.2                             ['$40.2']

This IR defines the variable arr twice. One on label 0 and another on label 24:
label 0:
  arr = $0.2                               ['$0.2', 'arr']

label 24:
  arr = $24.5                              ['$24.5', 'arr']

When creating the statements for those lines, numba uses the same object to both Stmt. In other words, the target field of both Assign statements points to the same Var object. The problem with this approach is that if one wants to change the name field of the first arr, the second is updated as well since they are the same object.
I don't think this is the expected behavior. I would expect that every definition to create a copy of it's LHS Var object. Therefore, I think a plausible solution would be to replace target by a copy of it on file interpreter.py:274

Var is treated as immutable.  It should not matter rather Var is referenced in multiple places.  I would prefer all rewrite passes to always create new nodes instead of mutating existing nodes.  I'd think the actual problem is that the IR nodes do not prevent mutation.

Hi @sklam, following up our discussion on Gitter. One part of the SSA transformation is variable renaming. For instance, when inserting a PHI node for arr at the beginning of label 22: arr.phi22 = PHI [(arr, 0), (arr, 24)], one would rename the first and the second arr` to distinguish them in further analysis.
If I am not wrong, LLVM does this! Every assignment creates a new variable and names are not shared among the LHS of assignments.

The trouble with mutating the Var.name attribute is a limitation of the design in the Numba IR. LLVM IR is designed for rewrite passes. When the Numba IR is created, we didn't envision that we need an extensive suite of rewrite passes.  Thus, it is very limited.  There're plans to improve the Numba IR but it will not be ready soon.
As for adding SSA, have you considered doing it when the Numba IR is generated so that you don't need to rewrite them later?  At numba/interpreter.py, the Interpreter class translates Python bytecode into Numba IR.  I think it has sufficient information to covert the stack based bytecode into SSA form in the Numba IR.

The trouble with mutating the Var.name attribute is a limitation of the design in the Numba IR. LLVM IR is designed for rewrite passes. When the Numba IR is created, we didn't envision that we need an extensive suite of rewrite passes. Thus, it is very limited. There're plans to improve the Numba IR but it will not be ready soon.

There are places in the code where mutating Var seems necessary (ref). Is Var immutable only because of the name?
The "problem" is that renaming variables require iterating over the RHS of each instruction in the Numba IR and as far as I am aware, there is no easy way of doing that without implementing it for every possible instruction type.
For instance, imagine that we have an instruction I which the RHS is an Expr and we want to 1) check if arr is in the RHS of I and 2) if yes, rename it (replace) by something else.
But, there are currently 18 types of Expr and they all define different operands:

binop (fn, lhs, rhs)
inplace_binop (fn, immutable_fn, lhs, rhs)
unary (fn, value)
call (func, args, kws, vararg)
build_tuple (items)
build_list (...)
build_set
build_map
pair_first
pair_second
getiter
iternext
exhaust_iter
getattr
getitem
static_getitem
cast
make_function

Therefore, adding the step of renaming variables would increase the coupling between PHI nodes and other IR instructions. Any design changes/addition in the IR would require the step of renaming variables to be aware of those changes.
Maybe before this, we could come up with a solution for iterating over instruction operands. Perhaps implement a getOperand/setOperand in a base class?
Let me know if that makes sense to you.

@guilhermeleobas, there were some IR manipulation tools added in numba.ir_utils.  They are mostly done by @ehsantn and @DrTodd13 during the development of the parfors passes.  I have not a good mental map of what is available in it.  From a quick look at it, I think replace_var_names should work for you.
We'll need better docs for everything in numba.ir_utils.


How to add "choose all" button?

Hi,
i want to add "choose all" action to left bar button. how can i choose all contacts?

You'd have to use the - (void)addContact:(id)contact withName:(NSString *)name; within a loop to add all contacts to the view. Adding a "choose all" button to your bar buttons is outside the scope of this project, but you may check out StackOverflow's answer to the question.

thanks @jguffey .


d4mac, d4win stable 1.13.1 release notes

Docker for Mac, Docker for Windows Stable 1.13.1 release notes
@ebriney @friism @samoht just preparing docs for tomorrow
Signed-off-by: Victoria Bialas<EMAIL_ADDRESS>

@mstanleyjones thanks, I changed the daemon.json thing but this is all direct from dev and they include it on the UI too. Unless it's really bad, I'd rather not change their wording. Next time I'll catch them all before the release goes out, and make them change it.

LGTM, merging


The tests take too long to run

Hello guys, I need your help.
I have a project in Angular 8 with about 400 unit tests implemented with Karma / Jasmine.
My computer has 8 cores and in the maxConcurrentTestRunners property I tested both by placing 2 and 4 - on the Striker website it is said for Angular applications to use half the number of cores.
When I run the styker run command it is said that approximately 3600 mutants have been generated and that the estimated execution time is 40 hours.
Is there anything to be done to reduce this time of execution? 40 hours becomes impracticable.
Unfortunately I can't publish code because it is all in a business environment.
For installing the Styker in the Angular project I followed the basic tutorial available on the website:

$ npm install -g stryker-cli
$ npm install --save-dev @ stryker-mutator / core
$ stryker init (at this stage I made the appropriate selection of Angular and Karma projects)
$ stryker run

Does anyone have any idea what it is or is it really?

Well... if we somehow could add all mutants to the code at the same time, that might save some time... 🤔
Oh wait. We're doing that right now 😅
Please try out the latest beta. You can read more about it here: https://stryker-mutator.io/blog/2020-07-13/announcing-stryker-4-beta-mutation-switching
We've found a > 50% performance improvement in our angular performance test.

nico 50% of 40h is still 20h :x

@belmirofss we've released v4 yesterday with this improvement. Would you mind checking it out? https://stryker-mutator.io/blog/2020-10-07/announcing-stryker-4-mutation-switching

Angular component tests are slow. A few things to try:

Enable Ivy, there seems to be performance improvements (2x)
Migrate to Jest to run suites in parallel (>2x)


Empty form action should resolve to document's address (should disregard baseuri set with base element)

Empty form action should resolve to document's address - i.e. it should ignore any baseuri set with base element.
See the spec:
http://www.whatwg.org/specs/web-apps/current-work/multipage/association-of-controls-and-forms.html#dom-fs-action
To reproduce:
<html>
<head>
  <base href="http://example.com/"/>
</head>
<body>
  <form action="">
    <input type="submit" value="foo"/>
  </form>
</body>
</html>

Parse and run form.absurl("action")
Observe it returns "http://example.com/", although it should return the document address (the one it was retrieved from).

Note:

This bug only applies to form's action; empty links should resolve to uri specified by base element

Fix:

The problem is that Node.absurl does not know about this HTML "specificity"
Obviously, there is a workaround on the client side, but I believe the expectations on jsoup would be to "behave as close to browsers as possible".

(Closing out old, dormant bugs. If you are still impacted by this, please reopen & vote.)


AL Gateway not connecting

Kip called me directly.  He installed sensors set with gateway ending in 193 today, turned everything on and isnt showing up on dashboard.
He already updated the firmware on this set with Esleyther.
He has to go back to farm tomorrow anyway, will reset, and i am going to help him go through synching using his tablet if needed.

His gateway started sending data and a backlog of data this morning at 7am. He didnt reset it, it just happened.

Kip's gateway sent data from 7am-2pm on 5/25. Has only been sending data from 3/4 nodes.
I tried to help him troubleshoot. 1. connected the node that isnt showing up on dashboard to tablet, node is working, but not synching to gateway
2.connected gateway to tablet (got stuck in loop trying to connect to hologram)
3.tried cellular scouting (reported GOOD signal)
4. tried to synch again and got stuck in loop trying to connect to cell again, never pinged nodes
I called Esleyther, he said similar thing happening in VA, he is going to talk to Alondra when she gets back.  I think Tuwana is having similar issue?
Kips other set he installed yesterday isnt conneceting to dashboard either.  He isn't going to go reset it until this issue gets sorted.
@elhenriq

@annabirnbaum @elhenriq
The gateway shouldn't be getting stuck in loop trying to connect to Hologram. It will try a few times, I don't remember how many, but it should eventually give up and try to connect to the nodes. I'll double check the firmware.
Just to make sure I'm understanding, this issue is that the gateway has good cellular reception, but it's getting stuck in an infinite loop?

This set seems to be working now.  Issue solved.


[language][move] Add parsing tests [part 1]

This is the start of some tests for the parser, both positive and negative tests. I went through the current grammar and tried to construct examples with variations of various constructs, starting from the higher level things:

File
Module
Address
Use declarations
Struct definitions
Function declarations

Some of these things are likely to change as we refine the language but I went ahead and added tests for everything I could think of as it stands right now.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
I ran these new tests and they all pass

:umbrella: The latest upstream changes (presumably 49788a86d8c3d4f9b8f83e796625e086adcc02d6) made this pull request unmergeable. Please resolve the merge conflicts.

I've updated this for the latest parser changes. It is ready to review now.

I just realized that Todd recently moved some tests into a "parser" directory, where I had been using "parsing". I've changed this PR to match Todd's name.

@bors-libra r=tnowacki

:pushpin: Commit 5f4408f has been approved by tnowacki

:hourglass: Testing commit 5f4408f531ac4e0b5cfa5eb82ad69403da891129 with merge 4c0cc184bf08bada21be08739ee3390d8617f7da...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra r=tnowacki

:bulb: This pull request was already approved, no need to approve it again.

This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.

:pushpin: Commit 5f4408f has been approved by tnowacki

:hourglass: Testing commit 5f4408f531ac4e0b5cfa5eb82ad69403da891129 with merge 98fba3198670a55c2403f9349867ffbdaed9a0a2...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: tnowacki
Pushing 98fba3198670a55c2403f9349867ffbdaed9a0a2 to master...


Question: the "ref" attributes

Just a quick question was the "ref" attribute found in version 0.23 replaced with something else in the newer version? Or just removed?
I get the following when upgrading and using "ref":
[!] Error: 'currentContext' is not exported by node_modules/solid-js/web/dist/web.js
I can learn to manage without it if it was removed, so no biggy.

Looks like a mismatch between solid version and babel preset solid version. Refs are definitely still there.

Thanks, thats all I needed to know.


completed

3.75 of 6 (-0.5 on Q1; -0.25 on Q2, Q3; -0.75 on Q4; -0.5 on Q5)
Not bad! Keep at it, and eventually this stuff will stick.


Multiple accounts not supported

Currently the app is only written to deal with one account, if the user has multiple accounts it will use the first. Most people won't have this issue but this could become an issue as Mondo grows.

Now an issue what with current accounts coming out.


Missing text function from MAGE

In the newest version, the text function is missing, user reported he was using it before updating the version and noticed it was missing.
Source

1 from the community: https://discord.com/channels/842007348272169002/842008652377751582/1236942628944220160

Please add text function back. Thanks!


Moving some filesystems to Transactional API part 1

This PR uncomments previously introduced changes and replaces the argument order to match inputs first and outputs last order for some Filesystems. Follow up PRs will move more filesystems to new API.

Retriggering builds since it seems Ubuntu CPU fialed

This passed as flaky in GPU build but still failing in CPU build. Not sure if the issue is related with this PR or the state of the time of the branch.

Build should be green on master. Let's try resyncing?

If we can have #42013 merged soon, I can make use of it in this PR after rebase. If you don't want that I will rebase this and make another PR for using the macro. What do you think?

#42013 should land soon

I rebased this one to todays master. Lets see if this pass.

@mihaimaruseac it looks like it passed after rebase, though it requires re-approval.

@mihaimaruseac current failure is not related with this PR. It seems something else changed on the master again.

There seem to be internal failures too on the internal testing filesystems. I'll have to import manually.

Rebasing and adding the new macro should fix all these errors. It is
possible that name hiding causing issues in internal places that I couldn't
see.
On Tue, Aug 4, 2020, 9:12 PM Mihai Maruseac<EMAIL_ADDRESS>wrote:

There seem to be internal failures too on the internal testing
filesystems. I'll have to import manually.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/41946#issuecomment-668969037,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQNEFDQQXQLBP4MLGHMRITR7DL3FANCNFSM4PQ6A3LA
.

Let's attempt that first then (rebasing and converting to use the macro)

@mihaimaruseac, done. Would require re-approval though


Crash

Terminating app due to uncaught exception 'CALayerInvalid', reason: 'layer <CALayer: 0x60000002b860> is a part of cycle in its layer tree'
It crashed on this line:
self.view.addSubview(commentView)
My view controller is UITableViewController subclass

I honestly haven't looked at this code in years, but if my memory serves, it won't work with a UITableViewController due to the way it is implemented.


Issue with k8s.io/docs/setup/pick-right-solution/

This is a...

[ ] Feature Request
[ x] Bug Report

Problem:
Link to IBM private cloud is broken
Proposed Solution:
Page to Update:
https://kubernetes.io/...

@Vaccano I tried to look for IBM cloud private on the IBM website and i get a 404 error. Just raised an issue on the IBM website.

@Vaccano The link to IBM private cloud works fine now. Please take a look and close this issue if it addressed.

/close


How do I pin the version of executables?

There seems to be no writeup on this matter in the documentations (aka README). I'm using vg with dep and my current way of installing executables is putting them in required field inside Gopkg.toml

Good question! By putting them in required of Gopkg.toml you are already pinning them in Gopkg.lock. The version is only updated if you run vg ensure -update or vg ensure -update github.com/user/repo-with-executable. If you want to exclude some specific (major) versions you can use a [constraint] block in your Gopkg.toml, in the same way as regular dependencies with dep.
I'll keep this open to remember to add something about it to the README for this (probably a link to dep docs).

@JelteF Thanks for the response 👍

I'm asking @sdboyer about this on slack to confirm, but I'm quite sure everything that is in the required list won't be pruned. Because that's the way you can insert packages in the dependency graph that are not actually imported (as far as I know).

I got a response in the #vendor slack channel:

I just tried it myself to see how dep prune behaves with required executable. So, I created a new project and added github.com/golang/dep/cmd/dep package in required. And then I ran ensure. The whole dep repo was downloaded to vendor/. Then I ran prune, and it cleaned all the packages that are not required by github.com/goland/dep/cmd/dep package. And then I tried building dep from what was left in vendor, and it worked.
Also, when the require is added to the lock file, dep adds a list of all the packages the required package uses. And this list of project packages is used by prune to create a list of packages to keep.
So, if you add the proper package name of the executable, it's safe.
Also, you're right about using constraint to set version of a required package.

So it will keep working fine even with prune. As long as you put the actual full path to the executable package, so github.com/golang/dep/cmd/dep instead of github.com/golang/dep. Luckily, you already should do that, because otherwise vg doesn't know which package to install.

confirming what was verified experimentally - required are equivalent to an import from a pruning perspective. wouldn't be much point in them otherwise ☺

Thanks @JelteF @sdboyer It's clear to me now 👍


fix(fs_lstat): enable S_IFLNK

thanks to @brimworks
fixes #153

Thanks! I would have sent this as a pull request, but I'm quite a bit behind on my branch (need to get some time to accommodate all the latest changes).
LGTM


high temp if (bad) BG data jumps high just for one reading even if BG is falling

ppl who are not using Dex for BG monitoring but Libre with Limitter get some times one high readings >200mg/dl just for one data set what causes high temp  even if BG is going down like few times in my case. 2nd problem is with flat line if Libre sensor expires after 14.5 days, last value will be read and send  continuously to NS if transmitter still attached and sending data

@libxmike This is a serious issue, but I don't think it's a oref0 issue. Because of the Seperation Of Concern principles (https://en.wikipedia.org/wiki/Separation_of_concerns ) these issue should imho not be fixed in oref0, but in the Libre application uploading the BG data (Limitter) in your case.
I think CGM devices should:

do some filtering of the (bad) BG glucose data if the technology requires that (prevent spikes that are inaccurate). If you compare raw data from a G4 you'll see that the Dexcom also does some kind of filtering. I think Limitter should also implement this

don't provide BG data if it's not accurate (for example sensor expired). I know the Libre sensor can detect that. It should not sent bad BG data to openaps in that case. By design if no fresh BG data has been entered OpenAps/Oref0 will not sett temps.

I noticed there is a mechanism in place for xdrip to sent an expiry message after 5 BG's that are the same
https://github.com/JoernL/LimiTTer/issues/10
I think 5*5 = 25 minutes is too long, but that's an implementation responsibility of the CGM-device.

the code from Joern is not in use anymore, we use one from @keencave which is more advanced

I talked about the issue with Scott yesterday and as there are too many transmitters and software for Libre sensors oref0 solution would be the best way

I disagree with @scottleibrand on this one. It would not be the best way, but the easiest way. Fixing somebody elses problems in another layer or application is plain wrong (sorry I met Edsger Dijkstra once).  OpenAPS/oref0 should be able to trust the cgm device (Nightscout in think in this case) to provide valid CGM-data.
What can be done is implement a (configurable) limit to oref0 of the maximum bg glucose (lets say default 280).  That would also benefit openaps/oref0 safety. Libre users can set that lower, but that's a workaround for the root cause (bad CGM data).

@AdrianLxM do you know what the current situation is with Libre?  Is the sensor-expired issue something we'll need to work around in OpenAPS, or will they be fixing it properly in xDrip+?

As far as I understood @jamorham, it was never an issue in xDrip+.
It was only people having set their input method to xBridge and not LimiTTer.
xDrip even detects whether it is a LimiTTer or an xBridge by the BT name.
So far I have only seen this in the German group where they passed an unofficial pre-release firmware for the LimiTTer that falsely reported to be an xBridge.

@libxmike, if you use the keencave firmware please update it. The latest version should now report itself as "LimiTTer".
@jamorham, I remember we had a general deduping of exactly the same raw values in for Dexcom as well. It seems to have gone - or did it not make it the xDrip+ fork? This would have prevented this as it would have not sent any more (false) data that was used for looping and would have risen a missed readings alert after a short while.

Anything else we need to do for this in oref0?  Or should we close this?

We've since added a check to determine-basal which should help with this: https://github.com/openaps/oref0/blob/dev/lib/determine-basal/determine-basal.js#L683-L687
And see also https://github.com/openaps/oref0/pull/690 and https://github.com/NightscoutFoundation/xDrip/issues/191


unable to run on web

ERROR in ./node_modules/react-native/Libraries/Image/AssetSourceResolver.js 1:407-439
Module not found: Error: Can't resolve '../Utilities/Platform' in 'D:\projects\rn\WebAndMobile-RN\node_modules\react-native\Libraries\Image'
resolve '../Utilities/Platform' in 'D:\projects\rn\WebAndMobile-RN\node_modules\react-native\Libraries\Image'
using description file: D:\projects\rn\WebAndMobile-RN\node_modules\react-native\package.json (relative path: ./Libraries/Image)
Field 'browser' doesn't contain a valid alias configuration
using description file: D:\projects\rn\WebAndMobile-RN\node_modules\react-native\package.json (relative path: ./Libraries/Utilities/Platform)
no extension
Field 'browser' doesn't contain a valid alias configuration
D:\projects\rn\WebAndMobile-RN\node_modules\react-native\Libraries\Utilities\Platform doesn't exist
.web.tsx
Field 'browser' doesn't contain a valid alias configuration
D:\projects\rn\WebAndMobile-RN\node_modules\react-native\Libraries\Utilities\Platform.web.tsx doesn't exist
.web.ts
Field 'browser' doesn't contain a valid alias configuration
D:\projects\rn\WebAndMobile-RN\node_modules\react-native\Libraries\Utilities\Platform.web.ts doesn't exist
D:\projects\rn\WebAndMobile-RN\node_modules\react-native\Libraries\Utilities\Platform.web.js doesn't exist
.js
Field 'browser' doesn't contain a valid alias configuration
D:\projects\rn\WebAndMobile-RN\node_modules\react-native\Libraries\Utilities\Platform.js doesn't exist
as directory
D:\projects\rn\WebAndMobile-RN\node_modules\react-native\Libraries\Utilities\Platform doesn't exist
@ ./node_modules/react-native/Libraries/Image/resolveAssetSource.js 1:96-128
@ ./node_modules/react-native-video/Video.js 1:1484-1542
@ ./node_modules/react-native-video-player/index.js 1:1958-1987
@ ./src/VideoView.js 1:322-358
@ ./AppNavigator.js 1:707-733
@ ./App.web.js 1:348-373
@ ./index.web.js 1:202-218

should be fixed, since react-native-video works on web. Check with react-native-video > 6.8.0


The sample in README seems not correct

Hello,
I am trying to understand how Hummingbird works by sample in README, but I have a problem with the matrix C.
The second and third rows of the matrix C seems incorrect. If the input feature is [1.5, 6, 3.5, 5, 4], then input path is [1, 1, 0, 1], the product of input path and matrix C will be [2, 2, 0, 0, -2], and the output path is [1, 1, 0, 0, 0].
I think the matrix C should be like this:
[[1, 1, 1, -1, -1], [1, -1, -1, 0, 0], [0, 0, 0, 1, -1], [0, 1, -1, 0, 0]]
and then for the same input feature [1.5, 6, 3.5, 5, 4], the input path is still [1, 1, 0, 1], the product of input path and C will be [2, 1, -1, -1, -1]，and the output path is [1, 0, 0, 0, 0].
I wonder if there is something wrong with my understanding?

Thanks @Qianxi! You are right, good catch! We are going to contact who generate the figure and see whether they can fix it.

@Qianxi Thank you very much! You are right, it's my mistake, I will change with the correct one.


Possibly bug in contract creation

py: 3.8
stories: 0.10.2
https://github.com/dry-python/stories/blob/75954a70b13c092281571b1da510de0d40ec9d3c/src/_stories/contract.py#L207-L212
Seems isinstance cannot be applied here because spec is a type/class. Maybe  you mean  issubclass ?

combine_contract also broken, because isinstance cannot be applied to parent.origin and child.origin (origin it's a class)
https://github.com/dry-python/stories/blob/75954a70b13c092281571b1da510de0d40ec9d3c/src/_stories/contract.py#L524-L531

:man_facepalming:  I didn't take it into account that PydanticSpec is metaclass

Yeh, it's a complicated package :slightly_smiling_face:


No staged files match any of provided globs

Description
Folder Structure

Both client and server have package.json with different lint-staged config
client/package.json
"husky": {
    "hooks": {
      "pre-commit": "lint-staged",
      "pre-push": "lint-staged"
    }
  },
  "lint-staged": {
    "*": [
      "prettier --write",
      "eslint",
      "git add"
    ]
  }

server/package.json
"husky": {
    "hooks": {
      "pre-commit": "lint-staged",
      "pre-push": "lint-staged"
    }
  },
  "lint-staged": {
    "*.js": [
      "prettier --write",
      "eslint",
      "git add"
    ]
  }

lint-stage is working on the staged files of the server folder but not working staged files of the client folder instead it complains about No staged files match any of provided globs. But if I run npx lint-staged -d  from the client folder using cmd, lint-stage works on staged files of the client folder.
Environment

OS: Windows 10
Node.js: v10.15.0
lint-staged: 9.2.0

This seems like a monorepo problem, which we cover in our README.md.
I think the problem boils down to the git directory being different than the working directory (client/server).

I started getting this issue.
I have a normal repo, my config looks like
{
  "scripts": {
    "build": "echo 'OK'",
    "prettify": "prettier --write",
    "prettify-all": "prettier --write \"./**/*.{js,scss,json}\"",
    "lint": "eslint",
    "lint-all": "eslint \"**/*.js\""
  },
  "devDependencies": {
    "eslint": "^6.8.0",
    "eslint-config-prettier": "^6.10.0",
    "eslint-loader": "^3.0.3",
    "eslint-plugin-jest": "^23.7.0",
    "eslint-plugin-prettier": "^3.1.2",
    "eslint-plugin-react": "^7.18.3",
    "husky": "^4.2.1",
    "lint-staged": "^10.0.7",
    "prettier": "^1.19.1"
  },
  "lint-staged": {
    "*.{js}": [
      "npm run prettify",
      "npm run lint"
    ],
    "*.{json}": [
      "npm run prettify"
    ]
  },
  "husky": {
    "hooks": {
      "pre-commit": "lint-staged && test && npm run build"
    }
  },
  "dependencies": {}
}

I have the same issue.
Folder structure is the following:

I have the same issue.
Folder structure is the following:
client
  package.json // with local pre-commit rules (src/**/*.{js,jsx})

server
  package.json // with local pre-commit rules (src/**/*.ts)

The one in the 'server' folder works fine, but the one in client just throws:
No staged files match any of provided globs
Both husky and lint-staged versions in both directories are the same.

removing the node_modules folder and running npm install solved this for me.

It's going to be a year-old issue soon (or 2+ months old for me personally) and nothing has changed.
I'm currently building a template for the students' projects and if this is not going to work I should look into an alternative.

@emirotin if that’s the case, it sounds like an issue with the upstream dependency micromatch.

Could be, for me it's just that the patch upgrade has broken the existing config

I’m sorry, but I’ll need those debug logs and configurations to be able to do anything.

You can find my config in my comment from Feb https://github.com/okonet/lint-staged/issues/682#issuecomment-585729946

I'm having the same issue, which is indeed fixed by omitting the curly braces {}.

OS: Windows 10
Node.js: v12.2.0
lint-staged: 8.1.7

As a curiosity, do the curly braces only work when there are multiple items inside it? Maybe this could be reported as a bug to micromatch.

@emirotin  Have you tried configuring lint-staged using the .lintstagedrc.json file instead of package.json? This resolved the issue for me.

Didn't try anything new since I figured out the braces issue.
On Sat, May 2, 2020, 02:45 Michal Kozakiewicz<EMAIL_ADDRESS>wrote:

@emirotin https://github.com/emirotin Have you tried configuring
lint-staged using the .lintstagedrc.json file instead of package.json? This
resolved the issue for me.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/okonet/lint-staged/issues/682#issuecomment-622613035,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEAMCHLM66L75DJ2ZDZZ63RPNNH7ANCNFSM4IJNQIZA
.

I guess there must be serious technical constraints in order for lint-staged to not work on multi-packages monorepos without Lerna at subrepo level and not root. In our case we cannot have all linters at root level because each subrepo has a different configuration and versioning.
It's a totally pity because lint-staged is a must-package on any npm project.

@AntonioRedondo that cd project1 doesn't do anything related to the following commands, especially when you're not using the --shell option. I don't recommend using shell either, since it's potentially unsafe. In your config it will run eslint and prettier from the repo root, but pass different files to each. Maybe combine that with explicit config files like eslint -c project1/.eslintrc.json --version.

Oops, my example contained a bug, it's eslint --fix, not eslint --version.

So what i did was i removed.
"husky": {
"hooks": {
"pre-commit": "lint-staged && npm run precommit"
}
from package.json and now i can commit my project to bitbucket.

It worked for me when I downgraded "lint-staged" package from "10.5.1" to  "10.2.11" and removed node_modules and reinstalled it.

I guess there must be serious technical constraints in order for lint-staged to not work on multi-packages monorepos without Lerna at subrepo level and not root. In our case we cannot have all linters at root level because each subrepo has a different configuration and versioning.

EDIT: actually I got it working with only one Husky and lint-staged configuration on the root package.json:
// package.json:

  "husky": {
    "hooks": {
      "pre-commit": "lint-staged"
    }
  },
  "lint-staged": {
    "project1/**/*.{js,jsx,ts,tsx}": [
      "cd project1",
      "eslint --fix",
      "prettier --write"
    ],
    "project2/**/*.{js,jsx,ts,tsx}": [
      "cd project2",
      "eslint --fix",
      "prettier --write"
    ]
  }

actually assuming your lerna packages are scoped with @AntonioRedondo you can just do this:

  "husky": {
    "hooks": {
      "pre-commit": "lint-staged"
    }
  },
  "lint-staged": {
    "project1/**/*.{js,jsx,ts,tsx}": [
      "yarn workspace @AntonioRedondo/project1 lint",
    ],
    "project2/**/*.{js,jsx,ts,tsx}": [
      "yarn workspace @AntonioRedondo/project2 lint",
    ]
  }

then in your project1 and project2 package.json have this script:
"name" "@AntonioRedondo/project2",
...
"scripts": {
  "lint": "npm-run-all lint:eslint lint:prettier",
  "lint:eslint": "eslint --fix",
  "lint:prettier": "prettier --write",
}
...

removing the node_modules folder and running npm install solved this for me.

for me, reomve xxxx.lock file too

I'll +1 @uzikilon's solution. I am in a tradition repo with all code under src/. I'd been playing with .eslintignore files at the root level and subdirectories. Eventually scrubbed that approach and removed all .eslintignores from the repo. At that point I was unable to commit anything. Recreating node_modules fixed it.  Using<EMAIL_ADDRESS>


imgscalr is not available from a Maven repository

Reported By<EMAIL_ADDRESS>The library is not hosted in any Maven repository. I'd suggest that
you either create your own Maven repository (not that difficult) or
get yourself listed in one of the public repos.

http://maven.thebuzzmedia.com  is down


What are the options?

This statement:
The Microsoft Graph Security connector is deprecated as of May 2021 and has been removed from the "Get Data" experience in Power BI Desktop. We recommend that you replace existing connections that use this connector, and refrain from using this connector for new connections. For more information, see the Microsoft Graph Security documentation.
Clearly recommends moving away from the Microsoft Graph Security Connector.  The for more information link provides LESS information than this page does.  Surely it would be useful to readers to give them an idea of what the alternatives are that they should migrate to?

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

ID: 3e7110b1-ad2a-f73a-3ada-27fbc387075c
Version Independent ID: 1f8fafd5-ddfa-76c1-1ee0-3a015b87c84c
Content: Connect to the Microsoft Graph Security API in Power BI Desktop - Power BI
Content Source: powerbi-docs/connect-data/desktop-connect-graph-security.md
Service: powerbi
Sub-service: pbi-data-sources
GitHub Login: @paulinbar
Microsoft Alias: painbar

Hi @tasranson, yes the connector has long been deprecated and the Graph Security connector team has provided no information about possible alternatives. I left our page up just in case someone needed the documentation there, but it seems to be only causing confusion so I am deleting it now and just redirecting to theirs, even though I know there is no useful information there. The only thing I can suggest is to open an issue from that page ( https://learn.microsoft.com/power-query/connectors/microsoft-graph-security ) and see whether you have better luck.
In any event, thanks for taking the time to reach out!


Implement / use / improve heading styles in coreUI

Discussed in 6/17/2022 EDA UX meeting.
Font sizes currently have no rationality.
For example:

(i) “Select a visualization” text seems to be larger than the tab labels;
(ii) the Study name is larger than “ClinEpiDB” in the header

Implement / use / improve heading styles in coreUI, for example:

Header 1: Study name
Header 2: Analysis name
Header 2: Tab labels (make bigger)
Header 3: Select a visualization

Styling to be finalized in styling WG

I moved this to web-eda, but there may be some additional work to be done in coreui.


Creating a new Resource has empty subject

I think a relatively recent refactor broke the UX for creating a new (non-controlled) Resource (e.g. new Restaurant). There is no subject generated.

[ ] Fix bug
[ ] Add test for creating new resources (no manual subject should be needed)

Closing in favor of #299


Adjust code to use Basics API rather than TSC

SwiftPM has its versions of AbsolutePath and withTemporaryDirectory, causing ambiguous call errors.

Ran tests locally for Swift 5.7, 5.8, 5.9, and nightly.


First stab at rescaling costs for ledger use

Pre-submit checklist:

Branch

[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Relevant tickets are mentioned in commit messages
[ ] Formatting, materialized Nix files, PNG optimization, etc. are updated

PR

[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reviewer requested

Pre-merge checklist:

[ ] Someone approved it
[ ] Commits have useful messages
[ ] Review clarifications made it into the code
[ ] History is moderately tidy; or going to squash-merge

Wrong button!