Maven problems on a clean room environment

[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[ERROR] Resolving expression: '${final.name}': Detected the following recursive expression cycle in 'final.name': [final.name] @
[ERROR] Resolving expression: '${final.name}': Detected the following recursive expression cycle in 'final.name': [final.name] @
[ERROR] 'dependencies.dependency.version' for io.quarkus:quarkus-vertx-web:jar is missing. @ line 49, column 21
 @
[ERROR] The build could not read 2 projects -> [Help 1]
[ERROR]
[ERROR]   The project org.graalvm.tests.integration:quarkus-json:1.0.0-SNAPSHOT (/home/tester/karm/mandrel-integration-tests/apps/quarkus-json/pom.xml) has 2 errors
[ERROR]     Resolving expression: '${final.name}': Detected the following recursive expression cycle in 'final.name': [final.name] -> [Help 2]
[ERROR]     Resolving expression: '${final.name}': Detected the following recursive expression cycle in 'final.name': [final.name] -> [Help 2]
[ERROR]
[ERROR]   The project org.acme:quarkus-vertx:1.0.0-SNAPSHOT (/home/tester/karm/mandrel-integration-tests/apps/quarkus-vertx/pom.xml) has 1 error
[ERROR]     'dependencies.dependency.version' for io.quarkus:quarkus-vertx-web:jar is missing. @ line 49, column 21

Apache Maven 3.6.3 (cecedd343002696d0abb50b32b541b8a6ba2883f)
Maven home: /opt/apache-maven-3.6.3
Java version: <IP_ADDRESS>, vendor: Eclipse Adoptium, runtime: /opt/jdk-<IP_ADDRESS>+1
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "5.14.0-319.el9.aarch64", arch: "aarch64", family: "unix"

Fixed with maven update for newer quarkus.

@Karm where is the fix?
I needed these:
commit 304fd64a03d4c64920ef3729de745b3304cfd194
Author: Severin Gehwolf<EMAIL_ADDRESS>Date:   Wed Sep 6 19:35:56 2023 +0200

    Fix final name

diff --git a/apps/quarkus-json/pom.xml b/apps/quarkus-json/pom.xml
index 57254d0..cfb1aa3 100644
--- a/apps/quarkus-json/pom.xml
+++ b/apps/quarkus-json/pom.xml
@@ -15,7 +15,7 @@

     <properties>
         <quarkus.version>${global.quarkus.version}</quarkus.version>
-        <final.name>${final.name}</final.name>
+        <final.name>quarkus-json-test</final.name>
     </properties>

     <dependencyManagement>

and
commit 52608c50db7ae8b43e77da331e0f37249fe7a47c (HEAD -> master)
Author: Severin Gehwolf<EMAIL_ADDRESS>Date:   Wed Sep 6 19:46:58 2023 +0200

    Fix vertex-web (now reactive-routes)

diff --git a/apps/quarkus-vertx/pom.xml b/apps/quarkus-vertx/pom.xml
index 4bbe1d3..7d47c9b 100644
--- a/apps/quarkus-vertx/pom.xml
+++ b/apps/quarkus-vertx/pom.xml
@@ -48,7 +48,7 @@
         </dependency>
         <dependency>
             <groupId>io.quarkus</groupId>
-            <artifactId>quarkus-vertx-web</artifactId>
+            <artifactId>quarkus-reactive-routes</artifactId>
         </dependency>
         <dependency>
             <groupId>io.quarkus</groupId>

@jerboaa The thing is that:

new Quarkus requires Maven 3.8.8, so Mandrel Integration Test suite too, that is what I meant by fixed. Maven update in test infra.
<final.name>${final.name}</final.name> is because of places like https://github.com/Karm/mandrel-integration-tests/blob/master/testsuite/src/it/java/org/graalvm/tests/integration/utils/BuildAndRunCmds.java#L97      I guess there could be some default though.  If you execute it without anything, it's invalid circular reference.
Not sure about this one, why is it needed globally and not as a patch? (e.g. Q 2.13.8 vs. 2.16.10 vs 3.2.5)

-            <artifactId>quarkus-vertx-web</artifactId>
+            <artifactId>quarkus-reactive-routes</artifactId>

@Karm Responses in-line below.

new Quarkus requires Maven 3.8.8, so Mandrel Integration Test suite too, that is what I meant by fixed. Maven update in test infra.

Not sure why a maven upgrade would fix this. It seems more likely that a not-clean ~/.m2 fixes it.

<final.name>${final.name}</final.name> is because of places like https://github.com/Karm/mandrel-integration-tests/blob/master/testsuite/src/it/java/org/graalvm/tests/integration/utils/BuildAndRunCmds.java#L97 I guess there could be some default though. If you execute it without anything, it's invalid circular reference.

OK. This sounds like we need something like default.final.name in a profile which activates if default.final.name is not provided and the tests in BuildAndRunCmds.java updated accordingly. I don't see how quarkus-json can build with only mvn clean package otherwise.

quarkus-vertx-web

It's quarkus-reactive-routes since 2.13.7.Final?:
$ cat ~/.m2/repository/io/quarkus/quarkus-vertx-web/2.13.7.Final/quarkus-vertx-web-2.13.7.Final.pom
<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0"
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
    <parent>
        <artifactId>quarkus-relocations-parent</artifactId>
        <groupId>io.quarkus</groupId>
        <version>2.13.7.Final</version>
    </parent>
    <modelVersion>4.0.0</modelVersion>

    <artifactId>quarkus-vertx-web</artifactId>

    <distributionManagement>
        <relocation>
            <artifactId>quarkus-reactive-routes</artifactId>
            <message>The artifact ${project.artifactId} is now named quarkus-reactive-routes. Please update your dependency.</message>
        </relocation>
    </distributionManagement>

PR is updated to allow for the final name customization. Please review! Thanks.


for some stocks, the get_bars method returns wrong data?

when i use get_bars to fetch minute level historical data using the following code:
self.wb.get_bars(stock='DAL', interval='m1', count=50, extendTrading=0)
this will return wrong data. the timestamp as well as open, high, low etc. are not correct:
                         open    high     low   close  volume    vwap

timestamp
2020-06-08 11:21:00-04:00  0.0221  0.0221  0.0221  0.0221     0.0  0.0221
2020-06-18 10:11:00-04:00  0.0212  0.0212  0.0212  0.0212     0.0  0.0212
2020-06-19 09:31:00-04:00  0.0200  0.0200  0.0200  0.0200     0.0  0.0200
2020-07-27 09:50:00-04:00  0.0250  0.0250  0.0250  0.0250     0.0  0.0250
2020-07-28 12:33:00-04:00  0.0250  0.0250  0.0250  0.0250     0.0  0.0250
if i replace 'DAL' by 'AAPL', it will return the following results and this data should be the right data:
                         open    high     low   close     volume    vwap

timestamp
2020-08-14 15:56:00-04:00  459.88  460.00  459.55  459.62   175152.0  456.52
2020-08-14 15:57:00-04:00  459.61  459.94  459.59  459.81   126124.0  456.53
2020-08-14 15:58:00-04:00  459.83  459.84  459.50  459.60   151860.0  456.54
2020-08-14 15:59:00-04:00  459.61  459.80  459.50  459.54   162152.0  456.56
2020-08-14 16:00:00-04:00  459.55  459.81  459.40  459.63  1283310.0  456.75
this condition will occurs to many other symbols such as 'FB'.
is this a bug?
how could i get the right data?
please

Just tested this here and it returns the expected data for DAL and FB.  Make sure you are on the latest release and test again.  pip install -U git+https://github.com/tedchou12/webull.git

wb.get_bars(stock='DAL', interval='m1', count=5, extendTrading=0)
open   high    low  close    volume   vwap
timestamp
2020-08-14 15:56:00-04:00  28.91  28.94  28.90  28.94   79094.0  28.85
2020-08-14 15:57:00-04:00  28.94  28.97  28.92  28.96   96272.0  28.85
2020-08-14 15:58:00-04:00  28.95  28.97  28.94  28.97   63626.0  28.85
2020-08-14 15:59:00-04:00  28.97  28.98  28.96  28.98   90622.0  28.85
2020-08-14 16:00:00-04:00  28.98  28.98  28.94  28.95  173490.0  28.85
wb.get_bars(stock='FB', interval='m1', count=5, extendTrading=0)
open    high     low   close     volume    vwap
timestamp
2020-08-14 15:56:00-04:00  261.02  261.15  260.93  260.98    50569.0  260.67
2020-08-14 15:57:00-04:00  260.99  261.18  260.99  261.03    46090.0  260.67
2020-08-14 15:58:00-04:00  261.02  261.06  260.80  260.87    62713.0  260.67
2020-08-14 15:59:00-04:00  260.88  260.96  260.81  260.92    68727.0  260.67
2020-08-14 16:00:00-04:00  260.90  261.24  260.87  261.24  1147457.0  260.75

A bit off-topic to this issue but I had a question. @alexonab I noticed you were the one who initially added the get_bars endpoint. How did you figure that row[7] was vwap and did you figure what row[5] was?

Hi, @alexonab! For some reason the wrong date data is outputted for HUT.
Code:
`ticker = 'HUT'
interval_ = 'd1'
n = 1
end_date_with_time = datetime(
year=2020,
month=7,
day=27,
).timestamp()
hist = wb.get_bars(ticker, interval=interval_, count=n+1, timeStamp=int(end_date_with_time.timestamp())).iloc[:, :4]
print (hist)`
Output:
open  high   low  close
timestamp
2019-12-16 00:00:00-05:00  2.24  2.24  2.24   2.24
2019-12-17 00:00:00-05:00  2.24  2.24  2.24   2.24
It works perfectly fine with other tickers

Looks like HUT is delisted on the Webul App, are you sure you are trading that stock?

The current Webull api by default does not return the ticker with exact matched symbol at first. For example, this query https://quotes-gw.webullbroker.com/api/search/pc/tickers?keyword=GS&regionId=6&pageIndex=1&pageSize=10 does not return stock with symbol "GS" first. I recommend examine the symbol in the get_ticker function.

This happens with many tickers. I go through them while the symbol and exchange match:
goodExchange = "NYSE,NYSEARCA,NASDAQ,AMEX,OTCMKTS,BATS"
For item in result['list']:
If item['symbol'] == stock.upper() AND goodExchange.find(item['disExchangeCode'] + ",") > -1:
return item['tickerId']

This fix should fix the problem I believe.
It will loop through the symbols to find an exact match and if none were found, taking the first.
The issue is that Webull doesn't give an exact match, so it will always attempt to search for similar stocks with symbols. Currently the results are hardcoded to 20 matches (I guess it would almost always be within the first 20 results). We can take the first 100 or few hundred results to ensure the accuracy, but it would slow down the process and this would be a compromise between the two.

Looks like HUT is delisted on the Webul App, are you sure you are trading that stock?

I'm not trading anything for now, just need the accurate data/


Use more optimistic openssl version constraint.

The openssl cookbook was previously pinned to version 4.0.x when they reverted a breaking change in release 4.0.0. Its newest version is 4.3.2 and brings some cool features. Therefore I set a more optimistic version constraint.
This fixes #278.

LGTM. Would be nice to see this merged.

Is this PR going to be included in a new release to the chef supermarket?

Yes. That is the plan. Hopefully soon.

@slyness, sounds good. Was just reviewing tasks today and curious when to slot in the upgrade.
Thanks!


Why package-lock is not in source control?

https://github.com/expressjs/express/blob/10c7756764fbe969b307b15a72fd074479c00f8d/.gitignore#L18
According to NPM docs:

This file is intended to be committed into source repositories.

Which doesn't mean that it should be published to npm:

One key detail about package-lock.json is that it cannot be published.

Is there any reason for lock file to be ignored I'm missing?
Thanks, and keep up with the great work.

Since npm ignores a package-lock.json file that is in a library, adding one here will have no effect on consumers and will make the CI no longer represent the dependencies a consumer would get by running npm install express.

Thanks @dougwilson for the great explanation :+1:


[Improvement][Master] Password stored in system table without encryption

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Password stored in system table t_ds_datasource without encryption .
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

There is no plan to do these unless you want to submit a DSIP to support.

I tried reslove him in dsip-#16473


With XCODE 10.1 AND POD INSTALLATION WITH COCOAPOD 1.6.0 BETA.2 any class of SideMenu called in the ViewDidLoad doesn't appear

New Issue Checklist

I have read the guidelines for contributing and I understand:

[x] My issue is happening in the latest version of SideMenu.
[x] My issue was not solved in the README.
[ ] My issue can not be answered on stackoverflow.com.
[ ] My issue is not a request for new functionality that I am unwilling to build and contribute with a pull request.
[ ] My issue is reproducible in the demo project.

Issue Description

I am Using latest XCODE and cocoapod 1.6.0 beta.2 and installed SideMenu but after import SideMenu  there was not error message . But when I tried to call The SideMenuManager or any of ther classes the xcode does not recognize that Class and auto completion does't show any referrence of those classes...
So I checked The Linked Framework and Libreries

So I added SideMenu.framework to the linked framework and Libreries and then that problem was solved .

Now the if i call any class from sidemenu it works.. I guess it's a minor bug of xcode or cocoapod...
But for learners like me It will be very helpfull if you mention to add SideMenu.framework to the Linked
Framework and Libreries   before importing SideMenu to any project class...
Thanks for this pod and continue your good work...

You forgot to import SideMenu.

As I had said... I had imported SideMenu to my ViewController Class.. but before linking the SideMenu.framework the the xcode IDE did not show any suggestion or even recognize any subclass call to my view controller


Small optimisation of the graph traversal for freeze

When traversing the closure graph of a commit during a freeze we don't need to remember the visited objects: those are already copied in their destination layer, and the skip function will skip them anyway. It is only a small optimisation (it saves ~4min out of 30min for the first freeze in bench_layers) and it saves a bit of memory allocations as well.
This PR:

vs master:

Rebased + resolved conflicts, but otherwise no changes made.

Rebased again but a test is failing:
> [FAIL]        LAYERED_PACK            16   Branches with squash.
...
FAIL copy v2 to dst

   Expected: `{"some":"X2"}'
   Received: `null'

Another test fails, but I can't reproduce it on my computer.
2020-12-09 20:02.22: Failed: LAYERED_PACK Test commits and graphs.

Are you ok to use #1188 instead of this PR? I think they share the same goal (removing a global hashtable for Repo.iter) but I believe the API is a bit simpler in #1188.

Are you ok to use #1188 instead of this PR? I think they share the same goal (removing a global hashtable for Repo.iter) but I believe the API is a bit simpler in #1188.

Sure. The API here is more flexible (i.e. you can add any tracking mechanism you want thanks to skip, it's not hardcoded in the iter function) but we don't need this flexibility right now, so let's go with #1188.

(happy to resurect this one later on if we need that extra flexibilty)


Area of a trapezoid

Title of article
Write a C# program that to compute area of trapezoid
Area of a trapezoid = ( base 1 + base 2 ) * height / 2
Example:
Base 1 = 5, Base 2 = 10, Height = 7
Area of the trapezoid = (5+10)*7/2 = 52.5
Additional information
Base 1 : 5
Base 2 : 10
Height  : 7

Area : 52.5

Code of Conduct

[X] I have read and agree to the Codinasion project's Code of Conduct

I can do this!


add some borders, background to the hex code in the webpage.

I will work on this one!

Issue fixed.


MOB 9328 add JWT in response

JIRA Ticket(s) if any

MOB-9328

Description

I have added JWT token to the response.

This will add token to the response.

@mprew97 Could you clarify what needs to be done for the MOB-9328 ticket? Based on your comment, it seems like no modifications are needed, but I want to confirm.

@mprew97 Thank you. I have revert back server code.


When a beforeEach fails, the subsequent tests are not reported as failed.

If I write a test like the following:
describe('suite', function () {

  beforeEach(function () {
    throw new Error('Something happened');
  });

  it('should report as failed', function () {

  });

  it('should also report as failed', function () {

  });
});

The output is:

I expected the report to show that two tests failed, and list the names of each of the tests. This is important to me, because I would like to see a consistent list of all of the tests listed in the report and a reason why they failed.

I guess I could write a custom reporter that would do this.


[Feature] Consider renaming the package

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
A contributor wrote this code:
select * from {{ metric.calculate(
    metric=metric('items_sold'),
    grain='week',
    dimensions=[]
) }}

and when compiling got this error:
dbt.exceptions.RuntimeException: Runtime Error
  Compilation Error in analysis test_metrics (analyses/test_metrics.sql)
    'dbt.context.providers.RuntimeMetricResolver object' has no attribute 'calculate'. This can happen when calling a macro that does not exist. Check for typos and/or install package dependencies with "dbt deps".
01:09:55  Encountered an error:
Runtime Error
  Compilation Error in analysis test_metrics (analyses/test_metrics.sql)

The reason is that they invoked metric.calculate() not metricS.calculate(). metric is a dbt Core construct, like ref, and metrics is the name of the package.
It would be less possible to make this mistake if we renamed the package, e.g. to dbt_metrics. This would also probably help with googling in the future because dbt_metrics isn't as polluted a namespace. (see also the poor sods who are trying to search how to do something using R)
Describe alternatives you've considered

Doing nothing.
Creating a package called ref to maximise chaos in the world.

Who will this benefit?
People who are trying to write code at 6pm and can't remember where the S's go
This will explicitly cause pain for people integrating against the package, so if we're going to do it we should do it ASAP.
Are you interested in contributing this feature?
No response
Anything else?
No response

Directionally I'm on board - I think it makes sense for it to better match our other packages (dbt_utils) and prevent the namespacing issue. However, I'm worried that changing the syntax again after just recently changing the macro name to calculate would be a large ask of all the partners who are working with us.
Would it make more sense for us to table this one and then combo it with a future syntax change?

I agree that it is inconvenient - it would have been better to get both changes in at the same time. But if we're going to change it, infinitely better to do so before GA. Right now it's annoying - once people are using it in production it's probably impossible without jumping through backward compatibility hoops.
Sooo if you're anticipating any other changes in the next 2 months that we could bundle with, sure! But if not then I say we bite the bullet

@drewbanin what are your thoughts here - it would be another change that partners would have to implement but @joellabes has the very good point of better now than much later once the syntax is generally available in the semantic layer.
The only changes we have for the package before Coalesce are the current issues and none of those strike me as being a full version jump from 0.3.0 to 0.4.0.

We have decided against this for the time being in order to ensure a consistent querying experience for our integration partners. Closing for now.


No solution found though it exists, using synced intervals + cumulative constraints

I am facing a problem using or-tools for scheduling in the presence of synchronized intervals and cumulative constraints with variable demands. In my example (attached as python file: cumulativeBug.zip) there are 2 tasks that are allowed to start between time 0 and 5. To each task, a "post task" is attached which starts at the respective task's end.
There are 2 resources available; each post task has a demand of 1 and can use any of the resources. I use binary indicator variables to model if task i uses resource j, and two cumulative constraints to model the "capacities" of 1 per resource. The objective minimizes the makespan.
Now, when I run this example, no solution is found, though both tasks could obviously start at 0 if their respective post tasks use different resources. However, when I change the upper bound on the starting time of both tasks from 5 to, say, 100, this solution is found. This sounds like a bug (because the same solution wolud also be feasible for the first model), or am I missing something fundamental here?

Update: the bud seems to be located in the cumulative edge finder. If I compile the library with edge finding disabled, the solver works as expected.

Thanks. I will have a look.

Hi,
I am still debugging.
In the meantime, I rewrote the code in examples/tests/issue128.py (along
the original code)
Thanks
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2015-12-15 15:34 GMT+01:00 Michael Helmling<EMAIL_ADDRESS>
Update: the bud seems to be located in the cumulative edge finder. If I
compile the library with edge finding disabled, the solver works as
expected.
—
Reply to this email directly or view it on GitHub
https://github.com/google/or-tools/issues/128#issuecomment-164781965.

Okay, thanks a lot for taking care of this issue! BTW I think it would be good to turn the flags in resource.cc into solver parameters. Because I am using C# and/or Python, I needed to recompile the .NET assembly in order to turn off edge finding (the flags also didn't show up in python-gflas though some of the ortools-flags do ... is it because these aren't specified in any header files?).
Michael

I pushed the fix to github.
Thanks for the bug report.

Thanks - however, I don't see the commit. :-) Did you perhaps forget to push?

Indeed :-(
Done
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2015-12-17 17:44 GMT+01:00 Michael Helmling<EMAIL_ADDRESS>
Thanks - however, I don't see the commit. :-) Did you perhaps forget to
push?
—
Reply to this email directly or view it on GitHub
https://github.com/google/or-tools/issues/128#issuecomment-165505523.

:-) Thank you! By the way, I noticed that, in the test file, you have used an alternative modelling approach with optional interval copies (in test_v1). Is this a "better" (in terms of performance) way to model resources needed by tasks?

I prefer this approach, as it allows different copies to have different
start time, especially if you do not decide first hand to decide which
resource to use.
And the disjunctive constraint is much faster. So disjunctive with optional
is much better than cumulative with 0-1 demands.
--Laurent
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2015-12-18 9:44 GMT+01:00 Michael Helmling<EMAIL_ADDRESS>
:-) Thank you! By the way, I noticed that, in the test file, you have used
an alternative modelling approach with optional interval copies (in
test_v1). Is this a "better" (in terms of performance) way to model
resources needed by tasks?
—
Reply to this email directly or view it on GitHub
https://github.com/google/or-tools/issues/128#issuecomment-165715545.

Thanks for that hint - works like a charm! :)


Nested coordinators. Fatal error: No view builder found for type

Hi, thanks for the great library. I'm trying to implement navigation with nested coordinators. My project is divided into modules, each module has its own coordinator, and there is also a root coordinator that creates all the others. There is an example of code that causes an error: Fatal error: No view builder found for type NewPath. I provide you with a minimal example to reproduce the error
enum Path: Hashable {
    case one(Int)
}

enum NewPath: Hashable {
    case one(String)
}

class FirstCoordinator: ObservableObject {
    @Published var path: Routes<Path> = []

    func push() {
        path.push(.one(1))
    }
}

struct ContentView: View {
    @ObservedObject var coordinator: FirstCoordinator

    var body: some View {
        FlowStack($coordinator.path, withNavigation: true) {
            VStack {
                Button {
                    coordinator.push()
                } label: {
                    Text("Push")
                }
                Text("ROOT")
            }
            .flowDestination(for: Path.self) { screen in
                switch screen {
                case let .one(value):
                    TestView(coordinator: .init(), value: value)
                }
            }

        }
    }
}

class SecondCoordinator: ObservableObject {
    @Published var path: Routes<NewPath> = []

    func push() {
        path.push(.one("exception"))
    }
}

struct TestView: View {
    @ObservedObject var coordinator: SecondCoordinator

    let value: Int

    var body: some View {
        FlowStack($coordinator.path) {
            VStack {
                Button {
                    coordinator.push()
                } label: {
                    Text("push")
                }
                Text(value.description)
            }
            .flowDestination(for: NewPath.self) { screen in
                switch screen {
                case let .one(value):
                    Test2View(value: value)
                }
            }
        }
    }
}

struct Test2View: View {
    let value: String
    var body: some View {
        Text(value)
    }
}

Is this a library bug or a bug in my implementation?

Thanks @lisindima - this was indeed a bug in the library code. I have made a change in v0.6.2 that should resolve the issue.
Side note: there is another issue with the example code you posted:
.flowDestination(for: Path.self) { screen in
   switch screen {
   case let .one(value):
     TestView(coordinator: .init(), value: value)
   }
}

Because you're initialising a new coordinator each time this closure is called, the child coordinator will soon get replaced by one with an empty path, popping the screen. You could prevent this by including the coordinator in the parent's path (e.g. case one(Int, SecondCoordinator)) or by keeping hold of it in the first coordinator, so that it remains stable. It seems like NavigationStack behaves differently though (see https://github.com/johnpatrickmorgan/NavigationBackport/issues/47), so there should maybe be a change in this library to not call the closure multiple times.

Thank you very much for the quick fix!


Update start_server.ps1

This script return two errors. Multiline character in powershell is "`".  And execution commands with $(command). With this changes script run correctly.

Thank you very much @rhigueras! I don't actually use Windows or PowerShell that often, so it's nice to see that someone who knows what they're doing has fixed this. Cheers!


Feature/resend details

Does not pass 2 A/C:

No confirmation before sending
No DSCC number in notification email

Only tests that clicking the "resend details" btn displays a flash message.
Does not test that an email is sent


Add Mnesia native Statestore

Related with https://github.com/eigr/spawn/pull/88

Closed by https://github.com/eigr/spawn/pull/296


Turn --verify-hashes on by default

Fixes #9164
Using clap's default_value_t makes the flag function unhappy, so just set the default when we unwrap. Tested with no flags, --verify-hashes, --no-verify-hashes and setting in uv.toml

Note that UV_VERIFY_HASHES=0 won't really do anything because flag will receive false, false, so we'll continue to verify hashes. We need something that does the opposite of BoolishValueParser

What if we change HashCheckingMode::from_args to take two Option? Then we can differentiate between UV_VERIFY_HASHES being 0, 1, or unset.
It would also be somewhat conventional for us to make --verify-hashes hidden, and add UV_NO_VERIFY_HASHES.

I assume you meant flag, HashCheckingMode::from_args. Right now it's too late by then, since clap will already have given you bools. The naive thing of turning verify_hashes into an Option<bool> in the CLI struct seemed to mess with clap's logic for flags and setting default_missing_value didn't seem to fix. I'm probably missing some dumb clap trick, feel free to take over this PR if it's obvious to you!

Sounds good, I'll see what I can do...

Okay, the issue is that default_missing_value just silently doesn't work if you don't also specify require_equals and num_args. I'll push the change, but if you see a better way feel free to!

Okay I modified things a bit such that instead of UV_VERIFY_HASHES=0, we require UV_NO_VERIFY_HASHES=1, which is consistent with the rest of the CLI. We also now document --no-verify-hashes instead of --verify-hashes.


doesn't recognise jpg.

Hi Hiroalchem,
thanks for this interesting tool. I am testing it for some jpg images, but it doesn't load them.
is it because
imgs = sorted(list(Path(self.target_dir.value).glob('./**/*png')) + list(Path(self.target_dir.value).glob('./**/*tif')))
in
load_dir(self)
doesn't check the jpg?
Regards
Chen

Hi @cloudsforest ,
Yes you are right, .jpg is not currently supported because of that part of the code. I will update it to support .jpg later.


Update face recognition

Motivation for this change

Bumped pythonPackages.face_recognition to the latest version to get several important fixes (see the commit message for further reference)
Reverted the latest pythonPackages.dlib bump as it broke the build

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

I think you should

raise an upstream issue about the dlib incompatibility
override the dlib version in the face_recognition expression instead of reverting the update

@dotlambda that's not the problem. When running nix-build -A pythonPackages.dlib the build fails right now locally and on hydra: https://hydra.nixos.org/build/72222721

@Ma27 That's bad. Do you have an idea what could be causing the segfault? /cc @globin
@GrahamcOfBorg build python2.pkgs.dlib python3.pkgs.dlib

Don't have a spontaneous idea unfortunately. I can have a look at the diff of dlib between 19.9 and 19.10 though %)

Success on x86_64-linux (full log)
Attempted: python2.pkgs.dlib, python3.pkgs.dlib
Partial log (click to expand)
.
.
.
.
.
                      [100%]

========================== 51 passed in 0.59 seconds ===========================

I'm currently running git bisect and will keep you updated.

Success on aarch64-linux (full log)
Attempted: python2.pkgs.dlib, python3.pkgs.dlib
Partial log (click to expand)
tools/python/test/test_matrix.py ......                                  [ 39%]
tools/python/test/test_point.py ...                                      [ 45%]
tools/python/test/test_range.py .....                                    [ 54%]
tools/python/test/test_rgb_pixel.py .                                    [ 56%]
tools/python/test/test_sparse_vector.py ....                             [ 64%]
tools/python/test/test_vector.py ..................                      [100%]

========================== 51 passed in 3.23 seconds ===========================
/nix/store/7dp4adw4r2ah9mpy5bb71bcxyznj1z3k-python2.7-dlib-19.9
/nix/store/saxjq6rnc6wspnqwvpyr1djcpx6crcp1-python3.6-dlib-19.9

awesome, thanks! I updated the description as it is indeed somehow misleading regarding which build was broken by the last dlib bump ;)

It seems like https://github.com/davisking/dlib/commit/14cbb804a9a69ba5ea928519f01c6c2875da2050 is causing this. However, I don't get a segfault but
  tools/python/test/test_svm_c_trainer.py ....../nix/store/6kz2vbh98s2r1pfshidkzhiy2s2qdw0a-stdenv-linux/setup: line 1223:  2914 Aborted                 /nix/store/azw9ys2m2fpfzf730xjcxja890gpyp58-python3-3.6.4/bin/python3.6m nix_run_setup test


[ts] problems have to be toggled with javascript.validate.enable

These seem to be errors relating to TypeScript but you toggle them off not with the typescript.validate.enable setting but javascript.validate.enable.
I now lose JavaScript diagnostics just because I want to use a different type system.
Expected: that I disable the typescript.validate.enable checkbox to lose the typescript warnings, not the JavaScript one. Also the warning is a bit disingenuous because there are other type systems 😄

@jamesgpearce from looking at the screenshot I see that the language used for the file is 'JavaScript.' Therefore the error message is correct. It would be more clear when the error message is reported with the prefix [js] and not [ts]. @mattbierner and idea why this happens?
I also see that you have the flow extension installed, but this extension does not contribute a language. I would have expected that the flow extension contributes a new language e.g. 'JS-Flow' that takes overs the .js file suffix. An external contribution should win over a built-in extension. A consequence for this is that the language would also need to define a grammar for colorization and some configuration for smart editing (bracket matching etc), but this could reuse the JS/TS artifacts. I haven´t tried this, but this would be better answered by the author of the flow extension.


Two Columns within a Card

I need to create a card with two columns that looks like this for the hospital marketing kit download page https://main--mammotome--hlxsites.hlx.page/us/en/hospital-marketing-kit-download-center

The image uses about 20% of the card and the text uses about 80% of the card. There are several download links in the card, so the entire card should not be clickable. I do not need the divider line under the header.
I'm not sure if the solution is a card variation or a column variation. There is already a "image cards" style for columns that displays the image like a card, but not the text.

@kelseywilliams751 is this something you need before the US-en go-live? If no, we could push it for right after go-live and you could use an alternate arrangement until then.

I can work with the columns layout temporarily. But will want to resolve this soon after go-live.

@dzehnder since go-live moves by a week, would you have time to implement this? should not be dramatic :)


store: only update CurrentIndex in CompareAndDelete if Remove succeeds

/cc @xiang90

Is there a reported issue for this?

No, noticed it when looking over the store.go code. Seems broken when compared to Delete, though.

@heyitsanthony I am aware of this. The problem is that if we change this then etcd might have inconsistency issue when replaying the log with a new version.

ack. closing


Enable clippy::missing_const_for_fn

While we're here, remove defensive programming against bug in Layout::from_size_align which is no longer needed on our new MSRV.

Codecov Report
Attention: Patch coverage is 89.47368% with 2 lines in your changes missing coverage. Please review.

Project coverage is 87.36%. Comparing base (8342c71) to head (ab3a087).

Files with missing lines
Patch %
Lines

src/error.rs
66.66%
1 Missing :warning:

src/pointer/ptr.rs
92.85%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##              0.9    #1883      +/-   ##
==========================================
- Coverage   87.40%   87.36%   -0.05%
==========================================
  Files          16       16
  Lines        5996     5974      -22
==========================================
- Hits         5241     5219      -22
  Misses        755      755

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


add flex:1 to SwipeViewsHeader class in css for proper tab height

I tried the library on chrome. Used dev tools for openeing mobile view. The height of SwipeViewsHeader is almost half the screen as shown in the picture below.

I solved it by adding flex:1 to SwipeViewsHeader class in SwipeViews.css
After making this fix, it looks like

Please advice,

Also, you can see that the tab indicator is coming at the bottom. This is another issue that needs to be fixed. I will create an issue for that and hopefully a PR as well. Thanks

Hi, sorry for the delayed answer. I'm not sure what's going on here.
Does the demo work ok for you ?
Actually, adding flex:1 to SwipeViewsHeader allows it to grow whereas it should have a fixed height, and only the SwipeViews should have a 100% (remaining) height.

Hey, thanks for getting back. The demo works fine however if I use the project after fetching it from npm it shows up like the image I have shown in my previous comment. Adding flex: 1 solves it for me. I think SwipeViewHeader and SwipeViews are in different div tags, that's why


feat: leverage backend more

What

Leverages backend more, removes need for this bot to have database access 🎉
Updates dependencies
updates bot to run every 5 mins between 17:00 GMT and 24:00 GMT to attempt to solve handling when Bungie's API is down

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


Resource parameter names cannot be set using label

I am following the documentation here to create a resource parameter. The parameter needs to hold log analytics workspaces for use in workbook queries. The workspace name isn't that useful as it's been auto generated so I'd like to replace it with something more useful.
Trying the documented approach
Renaming sounds possible by using a column named label in the result set. Based on the documentation I would expect the test query below to give me a list with everything named x.

But in the drop down, the resources names are still shown unmodified.

Note on parameter order
If I modify the query to change the position, it now seems to rename as expected but the parameter doesn't work in queries. Adding some debug text, I can see that both the value and label have been changed to just x.

It doesn't seem to matter what the first column is called, this is always used as the value and then it seems like the label is derived from it (using the resource name if the first column is a resource ID).

For now, this as designed: resource parameters get their names from the resources themselves, you can't override them with custom labels.  this is for security purposes to avoid people spoofing the names of resources.
as the warning says:

When using a Resource picker, the query should return the first column of Resource picker Id values, which are expected to be Azure Resource Id values. Additional columns 'selected' and 'group' are supported. Any other data will be ignored.

if you return any other invalid data in column 0, the value is still treated as an azure resource id, and if it isn't valid, it is passed along as is.  so in this case, your 'x' values are all not valid resource ids so the values are passed along as is.
there's a current active feature request to allow whatever is in the label field to be ADDED to the resource picker, like
resourcename1 (whatever was in label column)
resourcename2 (whatever was in label column)
resourcename3 (whatever was in label column)
but it hasn't been approved or prioritized yet.
As a workaround it is possible to return custom values in the "group" field after selected column, we've suggested that to some teams that wanted to add region/location or another tag like prod/test/dev to resource names.


Add Wander Terminal

Add this feature:
https://github.com/pxsocs/warden_terminal
Maybe for v0.9.2-3

Happy to help. The data.py file contains most of the data gathering methods. For example:
python3 data.py data_btc_price

Will output the Bitcoin Price Data.

Happy to help. The data.py file contains most of the data gathering methods. For example:
python3 data.py data_btc_price

Will output the Bitcoin Price Data.

On what line of data.py can I call directly from the code? Don't want to call it from shell.

Happy to help. The data.py file contains most of the data gathering methods. For example:
python3 data.py data_btc_price

Will output the Bitcoin Price Data.

On what line of data.py can I call directly from the code? Don't want to call it from shell.

Will add your project into mine in another folder if you agree, of course?

Depends how you'd like to make the call.
But you could, for example:

This returns the ANSI table

Absolutely fine to include, modify, delete...

Depends how you'd like to make the call.
But you could, for example:

This returns the ANSI table

Looks awesome!! Great job! Will do this.

Absolutely fine to include, modify, delete...

Do you have an ANSI logo or do you want I make one for you? Can do lot of desings with python-cfonts.

You can use the one in the repository (welcome.py)

Need to test this in other terminals, BTW. Only tested on Mac but will soon test on some of my nodes running different os / screens.

Cool! Will use that logo!

Testing

def callGitWardenTerminal():
    if os.path.isdir('warden_terminal'):
        os.system("cd warden_terminal && python3 node_warden.py")
    else:
        git = "git clone https://github.com/pxsocs/warden_terminal.git"
        os.system(git)
        os.system("cd warden_terminal && python3 node_warden.py")

Simple solution and the user can explore the hole program.

def callGitWardenTerminal():
    if os.path.isdir('warden_terminal'):
        os.system("cd warden_terminal && python3 node_warden.py")
    else:
        git = "git clone https://github.com/pxsocs/warden_terminal.git"
        os.system(git)
        os.system("cd warden_terminal && python3 node_warden.py")

Simple solution and the user can explore the hole program.

def callGitWardenTerminal():
    if not os.path.isdir('warden_terminal'):
        git = "git clone https://github.com/pxsocs/warden_terminal.git"
        os.system(git)
    os.system("cd warden_terminal && python3 node_warden.py")

Better

The Easter Egg in @PyBLOCK:
“The WARden Terminal.”
By @AlphaaZeta.
Now need 1 new module.
Install it:
a@A:~> sudo pip3 install pyfiglet


Error to send email to hotmail address

Hi !
I'm currently using node-sendmail to send emails which don't need any answer. I have tried with my personal address but I have some troubles. So, I hope you can help me :) (it's not necessarily a bug, maybe I'm doing something wrong)
Expected Behavior
Normally, the email should be sent and receive in my personnal mailbox.
Current Behavior
Actually, I have the following error:
mx resolved:  [ { exchange: 'mx1.hotmail.com', priority: 5 },
  { exchange: 'mx4.hotmail.com', priority: 5 },
  { exchange: 'mx2.hotmail.com', priority: 5 },
  { exchange: 'mx3.hotmail.com', priority: 5 } ]
Error on connectMx for:  { exchange: 'mx1.hotmail.com', priority: 5 } { Error: connect ETIMEDOUT <IP_ADDRESS>:25
    at Object.exports._errnoException (util.js:1023:11)
    at exports._exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 25 }
Error on connectMx for:  { exchange: 'mx4.hotmail.com', priority: 5 } { Error: connect ETIMEDOUT <IP_ADDRESS>:25
    at Object.exports._errnoException (util.js:1023:11)
    at exports._exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 25 }
Error on connectMx for:  { exchange: 'mx2.hotmail.com', priority: 5 } { Error: connect ETIMEDOUT <IP_ADDRESS>:25
    at Object.exports._errnoException (util.js:1023:11)
    at exports._exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 25 }
Error on connectMx for:  { exchange: 'mx3.hotmail.com', priority: 5 } { Error: connect ETIMEDOUT <IP_ADDRESS>:25
    at Object.exports._errnoException (util.js:1023:11)
    at exports._exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 25 }
error on connectMx Error: can not connect to any SMTP server
    at tryConnect (E:\Cours année 2015 - 2017\Mémoire\mookup\node_modules\sendmail\sendmail.js:86:49)
    at Socket.<anonymous> (E:\Cours année 2015 - 2017\Mémoire\mookup\node_modules\sendmail\sendmail.js:92:13)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:189:7)
    at emitErrorNT (net.js:1280:8)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)
Error: can not connect to any SMTP server
    at tryConnect (E:\Cours année 2015 - 2017\Mémoire\mookup\node_modules\sendmail\sendmail.js:86:49)
    at Socket.<anonymous> (E:\Cours année 2015 - 2017\Mémoire\mookup\node_modules\sendmail\sendmail.js:92:13)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:189:7)
    at emitErrorNT (net.js:1280:8)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)

Steps to Reproduce (for bugs)

I use this piece of code

const sendmail = require("sendmail")();

exports.sendEmail = function(destinataire){
    return new Promise(function(resolve, reject){
        sendmail({
            from<EMAIL_ADDRESS>            to: destinataire,
            subject: 'validation de votre inscription',
            html: 'Mail of test sendmail ',
        }, function(err, reply) {
            if(err){
                console.log(err);
                reject(err);
            }
            resolve(reply);
        })
    })
};

destinataire is an email address. In my case, this is my personal address with the following form<EMAIL_ADDRESS>
Context
I'm trying to send an email to check the validity of an inscription on my website. Now, I'm just trying to send email and receive it. Later, I will provide a special link in the email to validate the inscription.
Your Environment

Version used: node-sendmail 1.1.1
Environment name and version (e.g. Chrome 39, node.js 5.4): Chrome Version 56.0.2924.87; Node.JS 7.5.0
Operating System and version (desktop or mobile): Windows 7 integral edition (64 bits)

Where are you trying to send it from? Ex. Google Cloud, Heroku or localhost?
It sounds like it did work before? If so where was the last known work server or localhost

I work with localhost and this is my first time with node-sendmail. Do you think this is a problem with the firewall ?

Try sending to a Gmail address if you have one.  It looks like you were trying to send it to a mx2.hotmail.com live account or a Hotmail. Let me know what happens

I have the same problem. I have tried with my personal gamil address and this is the log:
mx resolved:  [ { exchange: 'alt4.gmail-smtp-in.l.google.com', priority: 40 },
  { exchange: 'alt3.gmail-smtp-in.l.google.com', priority: 30 },
  { exchange: 'alt2.gmail-smtp-in.l.google.com', priority: 20 },
  { exchange: 'alt1.gmail-smtp-in.l.google.com', priority: 10 },
  { exchange: 'gmail-smtp-in.l.google.com', priority: 5 } ]
Error on connectMx for:  { exchange: 'alt4.gmail-smtp-in.l.google.com', priority: 40 } { Error: connect ETIMEDOUT <IP_ADDRESS>:25
    at Object.exports._errnoException (util.js:1023:11)
    at exports._exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 25 }
Error on connectMx for:  { exchange: 'alt3.gmail-smtp-in.l.google.com', priority: 30 } { Error: connect ETIMEDOUT <IP_ADDRESS>:25
    at Object.exports._errnoException (util.js:1023:11)
    at exports._exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 25 }
Error on connectMx for:  { exchange: 'alt2.gmail-smtp-in.l.google.com', priority: 20 } { Error: connect ETIMEDOUT <IP_ADDRESS>:25
    at Object.exports._errnoException (util.js:1023:11)
    at exports._exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 25 }
Error on connectMx for:  { exchange: 'alt1.gmail-smtp-in.l.google.com', priority: 10 } { Error: connect ETIMEDOUT <IP_ADDRESS>:25
    at Object.exports._errnoException (util.js:1023:11)
    at exports._exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 25 }
Error on connectMx for:  { exchange: 'gmail-smtp-in.l.google.com', priority: 5 } { Error: connect ETIMEDOUT <IP_ADDRESS>:25
    at Object.exports._errnoException (util.js:1023:11)
    at exports._exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 25 }
error on connectMx Error: can not connect to any SMTP server
    at tryConnect (E:\Cours année 2015 - 2017\Mémoire\mookup\node_modules\sendmail\sendmail.js:86:49)
    at Socket.<anonymous> (E:\Cours année 2015 - 2017\Mémoire\mookup\node_modules\sendmail\sendmail.js:92:13)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:189:7)
    at emitErrorNT (net.js:1280:8)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)
Error: can not connect to any SMTP server
    at tryConnect (E:\Cours année 2015 - 2017\Mémoire\mookup\node_modules\sendmail\sendmail.js:86:49)
    at Socket.<anonymous> (E:\Cours année 2015 - 2017\Mémoire\mookup\node_modules\sendmail\sendmail.js:92:13)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:189:7)
    at emitErrorNT (net.js:1280:8)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)

I am too facing this issue. It sends mail to corporate address but not to personal Gmail account. Any quick resolution would be highly appreciated.

I have tried sending mails to both my personal Gmail and Hotmail addresses from the domain pointed at my server and there are no issues whatsoever.
My env:
Ubuntu server 14.04
Node version 6.1.0
Sendmail version 1.1.1

I think I have found the problem (thanks to @Geex-Renzo ). I don't use sendmail (I was thinking it was no needed because node-sendmail don't use smtp server).
Before closing this ticket, can you explain why Sendmail (or postix) is needed ?
Thank you very for your help :)


Alt+C hotkey activated even when no search panel is opened

Type: Bug

Press Alt+C or Alt+W or Alt+R

Now if area with code is in focus, broken search parameter selector will appear without search panel itself. If terminal is in focus, search panel will appear.
I think it is completely unneccesary to intercept alt+c globally when no search panel is opened. In my setup it leads to conflict with alt+c fzf's key binding in terminal.
VS Code version: Code 1.86.2 (903b1e9d8990623e3d7da1df3d33db3e42d80eda, 2024-02-13T19:41:37.860Z)
OS version: Linux x64 6.7.6-arch1-1
Modes:

System Info

Item
Value

CPUs
13th Gen Intel(R) Core(TM) i7-13700H (20 x 569)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 2, 1

Memory (System)
15.35GB (6.16GB free)

Process Argv
--unity-launch --force-device-scale-factor=2 --crash-reporter-id 9aeb40d4-24c1-4074-b918-b13c7885d80f

Screen Reader
no

VM
0%

DESKTOP_SESSION
plasmawayland

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
KDE

XDG_SESSION_TYPE
wayland

Extensions (19)

Extension
Author (truncated)
Version

vscode-nginx-conf
ahm
0.3.2

vscode-cython
ktn
0.3.10

vscode-docker
ms-
1.29.0

autopep8
ms-
2023.8.0

debugpy
ms-
2024.2.0

isort
ms-
2023.10.1

python
ms-
2024.2.1

vscode-pylance
ms-
2024.2.2

jupyter
ms-
2024.1.1

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.17

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

remote-containers
ms-
0.348.0

cmake-tools
ms-
1.17.17

cpptools
ms-
1.19.4

cpptools-extension-pack
ms-
1.3.0

rust-analyzer
rus
0.3.1860

cmake
twx
0.0.17

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscorecescf:30445987
vscod805:30301674
binariesv615:30325510
vsaa593:30376534
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
a9j8j154:30646983
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
h48ei257:30968133
pythontbext0:30879054
accentitlementsc:30887149
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
d6fi2104:30959263
d34g3935:30971562
edj9j583:30969295
fegfb526:30973843
e3gdj431:30958358
dsvsc020:30976470

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.87.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Can you share a screen recording? I'm not sure if I understand what you're talking about.

Screencast_20240417_193229.webm
This video shows the behavior of vscode after pressing alt+c

Screencast_20240417_193734.webm
This video shows what I expected after pressing alt+c in terminal
It is configured with following lines in .zshrc:
source /usr/share/fzf/key-bindings.zsh
source /usr/share/fzf/completion.zsh

I can repro when the widget is visible even if my focus isn't there
https://github.com/microsoft/vscode/assets/29464607/03d1b4c0-2786-48c9-aff2-6f0d1094205f

Never mind, the little toolbar in the editor is coming from the editor command toggleFindRegex, not our terminal one.


Fixed gem security vulnerability, added to readme, fixed static declaration

Updated some gems at version 5.1.5 which we're causing a security vulnerability to version <IP_ADDRESS>. Added a portion to the Readme to instruct users on how to build and run the project locally. Also added some brackets around a variable which was causing tests to fail

Thanks Isaac!


Update version to 1.0.21-0 on master

Update version to 1.0.21-0

🚀 [Deployed](https://github.com/Expensify/Expensify.cash
/actions/runs/765130740) 🚀 to
staging on Mon Apr 19 2021 at 22:41:56 GMT+0000 (Coordinated Universal Time)

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Commandline options

When I saw this tool (herbe) i instantly knew i wanted to try to get commandline options for it i always wanted this program on linux. I am not experienced with C programming, feel free to do and say what you want, but it would be really cool if we could get CLI options. I used the external library https://github.com/skeeto/optparse , because it looked like the easiest (and best?) implementation of a getopt function that supported both long and short options, author also wrote this: https://nullprogram.com/blog/2020/08/01/ ..
I also added more enums to "corner", which i think you should rename "anchor":
123
456
789

Anyways, big thanks for your tool.

I appreciate your contributions, but I don't think it's the best time to make pull requests right now. I still make substantial changes to herbe nearly everyday. The code changes frequently and at the time you published this pull request, it was already a commit behind resulting in conflicts that have to be resolved. I will get back to this PR when I finish all the "main" functionality. I will mark that with a 1.0 release to signal the appropriate time to make PRs and patches. I hope that herbe won't change much after that point. Thank you.

Sounds good.

The 1.0 release is out and as promised I'm coming back to this PR. I also saw your videos and now I understand why someone would benefit from command-line options in herbe. You can now finally make a patch without much trouble. I would suggest splitting the patch into "cmd options" and "more anchors".
I was also thinking about creating my own, more minimal, patch for command-line options as I'm not a big fan of "unnecessary" external libraries and long options. But that's a big maybe, so feel free to get to work if you are still interested. Thanks.

Cool, i will give it some bud this weekend. I completely understand you on the "no-external-lib" thing. And i don't think it will be difficult to replace the optionparsing functions with homebrew ones. But still, the library is quite compact and minimal and half of the headerfile could be removed i think since we are not using all of the functions here.
Do you have any thoughts about "option-names", meaning is -c good for backgroundcolor, or should it be saved for a possible future shortoption for --config-file and stuff like that?

If we are talking about a patch, I don't really want to limit you in any way. A patch should be exactly as the author envisioned it, so if you want to use the external library or you want to support long options, feel free to do so. If someone doesn't like the patch for any reason, they can create their own. That's the beauty of it. And to be honest, I don't really have much thoughts about naming the cmd arguments. Maybe I'll add a -v flag for version or -h for help in the future, but that's it, at least for now.


new: usr: support for composite command in sdk, only hive and hadoop com...

...mand covered, others will be supported post initial review

Looks good apart from a few minor nits. Also, it would be great if you can add an example in the examples/ directory.

will add example in next PR

Squash merged: https://github.com/qubole/qds-sdk-java/commit/fbcaa512fd0449cbeceda191557eda224a9713b1


Resetting view location causes rendering issue in panel

Move output and debug console to sidebar
Reset their view location
:bug: The font is displayed weirdly and the icon sometimes appears

This does not appear to affect Problems and the Terminal

duplicate


AVRO-3724 Fix C# JsonEncoder for nested array of records

What is the purpose of the change
https://issues.apache.org/jira/browse/AVRO-3724
Fix C# JsonEncoder for nested array of records.
The SpecificDatumWriter is missing a call to encoder.StartItem().  The GenericDatumWriter has this call.
Verifying this change
Added test with nested array of records that fails without the fix and passes with the fix.
Documentation
N/A

@martin-g , @KyleSchoonover , hi again!  Could I kindly get a review?  Thanks in advance!

Lgtm

Thank you, @rayokota !

Thanks for the quick review, @martin-g , @zcsizmadia !

Thank you for fixing and merging this so quickly everyone, your efforts are very much appreciated.  I've pulled the latest code and can confirm that I can now produce the Avro JSON for my schema.


Argon containers use only weak cores on systems with heterogeneous CPUs

Windows Build Number
10.0.22621.0
Processor Architecture
AMD64 (also ARM64)
Memory
64GB (also 32GB, 4GB, etc.)
Storage Type, free / capacity
Micron 3400 SSD, 929GB/1900GB + WDS400T3X0C, 1250GB/3700GB (not that it matter in any way; also various other devices with all kinds of SSD storage sizes)
Relevant apps installed
Containers feature enabled in Turn Windows features on or off.
Traces collected via Feedback Hub
Which WPR profile would you like me to record? Not that it would give you anything you can't get buy just running five PowerShell commands.
Isssue description
Argon containers use only weak cores on systems with heterogeneous CPUs.

We use Docker for our build environment. This is required for many reasons.
For example, it's quote common for our code to be incompatible with past and future Visual C++ versions because of new C++ features going in and bugfixes (or new bugs) modifying behavior of old code.
Generally, people can always build the master branch on their host machine, but they can't build an older version (required when servicing an issue with an older but still supported version). But people can't refrain from upgrading Visual Studio because then the current code won't build. It's impractical to keep all the version of the Visual C++ build tools installed.
So we have Docker images with the entire build environment, and a text file in the code repository pointing to the tag of the corresponding build tools Docker image. That way in order to build any commit from the product source repository we only need to pull the referred Docker image. This is also what our CI system does.

We make every effort to run Docker containers using process isolation (mostly - make sure our images start from base images compatible to the host), as our tests have shown that running under process has no overhead compared to running directly on the host (sometimes even slightly faster¹), while running under Hyper-V isolation has a significant overhead, among other problems.
Unfortunately we have recently discovered that when running on system with heterogeneous CPUs (e.g. Intel Alder Lake and Raptor Lake CPUs and all ARM64 CPUs supported by Windows), the processes inside process-isolated containers utilize only the "weak" cores (E-core on Intel, LITTLE cores on ARM). See: https://github.com/docker/for-win/issues/13562
Using sophisticated debugging techniques we have also discovered that the issue is not with Docker/Moby but rather with Windows. See below in the reproduction section.
On all of our machines with Intel CPUs prior to Alder Lake, building inside process-isolated Docker containers runs as fast (or slightly faster¹, but on our Raptor Lake machines with Intel i9 13900H CPUs (with 6 P-cores = 12 logical cores + 8 E-core) we get the following results:

Host
Process isolation

Debug
500s
1000s

Release
1000s
2500s

(The numbers are rounded averages. For example, Debug on the host actually takes 490s-497s.)
This it pretty awful. We found out it is actually faster to build on an older model of the same computer, from 2 iterations back, with an Intel i9 11900H CPU, since it doesn't have two kinds of cores and it actually uses all of them.
On the Raptor Lake aforementioned the 12 logical P-core do nothing while the 8 E-cores do all the work. We're assuming Release is more compute-intensive due to the optimizations and there we see x2.5 factor (which is close to the 40% CPU utilization claimed by some tools seeing 100% on 8 cores and practically nothing on 12 other cores), and Debug it not so compute-bound so there the slowdown factor is "only" x2.
A workaround we've considered is using Hyper-V isolation. It kind of works but not really. A complete table with Hyper-V isolation would be:

Host
Process isolation
Hyper-V isolation

Debug
500s
1000s
700s

Release
1000s
2500s
-ICE-

The Debug build has a significant overhead compared to running directly on the host and compared to what process isolation should have been, but it's still better than what process isolation does on this machine. But the Release build just crashed MSBuild.exe for lack of memory.
You see, Hyper-V isolation has the "nice" property of causing MSBuild.exe to crash with screams about not enough memory when launching the containers with as few as --memory 64GB and refusing to run at all with, let's say, --memory 96GB:
> docker run --rm -it --mount "type=bind,src=$(Get-Location),dst=C:\whatever" -w C:\whatever --isolation=hyperv --cpu-count 20 --memory 96GB some-builder:some-tag powershell.exe -Command "build command"
docker: Error response from daemon: hcs::CreateComputeSystem a3270d2ab11086bbcaac433a660d5cbde47fea3d2f0349b9d2ed22a9839a76e9: The paging file is too small for this operation to complete.

That's a separate issue with Hyper-V isolation being less that useful but we're not here to solve that. The point is that using Hyper-V isolation isn't a valid workaround on the grounds of not working. Not that it would be a good workaround even if it had worked, on grounds of being slow and no reason for process isolation not to work properly.

¹ One element we have discovered that makes processes inside process-isolated containers to run slightly faster than directly on the host is less interference by certain security software. We assume there may be other causes. Generally we say that the performance of process-isolated docker containers is approximately equal to that of running directly on the host, except in pathological cases. Like the one we have here, unfortunately.
Steps to reproduce
Execute the following PowerShell commands:
[E:\]
> $sieve_URI = "https://github.com/kimwalisch/primesieve/releases/download/v11.1/primesieve-11.1-win-x64.zip" #or arm64
[E:\]
> Invoke-WebRequest -Uri $sieve_URI -OutFile "sieve.zip"
[E:\]
> Expand-Archive -Path ".\sieve.zip" -DestinationPath ".\sieve\"
[E:\]
> CmDiag.exe CreateContainer -Type ServerSilo -Id 11111111-1111-1111-1111-111111111111 -FriendlyName Foo
The container was successfully created. Its ID is: 11111111-1111-1111-1111-111111111111
The container will continue running until it is terminated. A new instance of cmdiag has been spun
up in the background to keep the container alive.

[E:\]
> CmDiag.exe Map 11111111-1111-1111-1111-111111111111 -ReadOnly "$PWD\sieve" "C:\sieve"
[E:\]
> CmDiag.exe Console 11111111-1111-1111-1111-111111111111 powershell.exe
Executing: powershell.exe
Windows PowerShell
Copyright (C) Microsoft Corporation. All rights reserved.

Install the latest PowerShell for new features and improvements! https://aka.ms/PSWindows

PS C:\Windows\system32> C:\sieve\primesieve.exe 1e14
Sieve size = 256 KiB
Threads = 20
0%

Take a look at your favorite CPU utilization tool. I used Sysinternals Process Explorer. You're welcome to use Task Manager, perfmon.msc or whatever rocks your boat.
Dell XPS 9530 with i9 13900H, 64GB RAM, aforementioned Micron SSD:

If you're not sure which core is which you can hover over the core. Process Explorer, unlike Task Manager tell you which logical core belongs to which physical core. Or you can use Sysinternals Coreinfo:
coreinfo.exe output:
Logical Processor to Cache Map:
**------------------  Data Cache          0, Level 1,   48 KB, Assoc  12, LineSize  64
**------------------  Instruction Cache   0, Level 1,   32 KB, Assoc   8, LineSize  64
**------------------  Unified Cache       0, Level 2,    1 MB, Assoc  10, LineSize  64
********************  Unified Cache       1, Level 3,   24 MB, Assoc  12, LineSize  64
--**----------------  Data Cache          1, Level 1,   48 KB, Assoc  12, LineSize  64
--**----------------  Instruction Cache   1, Level 1,   32 KB, Assoc   8, LineSize  64
--**----------------  Unified Cache       2, Level 2,    1 MB, Assoc  10, LineSize  64
----**--------------  Data Cache          2, Level 1,   48 KB, Assoc  12, LineSize  64
----**--------------  Instruction Cache   2, Level 1,   32 KB, Assoc   8, LineSize  64
----**--------------  Unified Cache       3, Level 2,    1 MB, Assoc  10, LineSize  64
------**------------  Data Cache          3, Level 1,   48 KB, Assoc  12, LineSize  64
------**------------  Instruction Cache   3, Level 1,   32 KB, Assoc   8, LineSize  64
------**------------  Unified Cache       4, Level 2,    1 MB, Assoc  10, LineSize  64
--------**----------  Data Cache          4, Level 1,   48 KB, Assoc  12, LineSize  64
--------**----------  Instruction Cache   4, Level 1,   32 KB, Assoc   8, LineSize  64
--------**----------  Unified Cache       5, Level 2,    1 MB, Assoc  10, LineSize  64
----------**--------  Data Cache          5, Level 1,   48 KB, Assoc  12, LineSize  64
----------**--------  Instruction Cache   5, Level 1,   32 KB, Assoc   8, LineSize  64
----------**--------  Unified Cache       6, Level 2,    1 MB, Assoc  10, LineSize  64
------------*-------  Data Cache          6, Level 1,   32 KB, Assoc   8, LineSize  64
------------*-------  Instruction Cache   6, Level 1,   64 KB, Assoc   8, LineSize  64
------------****----  Unified Cache       7, Level 2,    2 MB, Assoc  16, LineSize  64
-------------*------  Data Cache          7, Level 1,   32 KB, Assoc   8, LineSize  64
-------------*------  Instruction Cache   7, Level 1,   64 KB, Assoc   8, LineSize  64
--------------*-----  Data Cache          8, Level 1,   32 KB, Assoc   8, LineSize  64
--------------*-----  Instruction Cache   8, Level 1,   64 KB, Assoc   8, LineSize  64
---------------*----  Data Cache          9, Level 1,   32 KB, Assoc   8, LineSize  64
---------------*----  Instruction Cache   9, Level 1,   64 KB, Assoc   8, LineSize  64
----------------*---  Data Cache         10, Level 1,   32 KB, Assoc   8, LineSize  64
----------------*---  Instruction Cache  10, Level 1,   64 KB, Assoc   8, LineSize  64
----------------****  Unified Cache       8, Level 2,    2 MB, Assoc  16, LineSize  64
-----------------*--  Data Cache         11, Level 1,   32 KB, Assoc   8, LineSize  64
-----------------*--  Instruction Cache  11, Level 1,   64 KB, Assoc   8, LineSize  64
------------------*-  Data Cache         12, Level 1,   32 KB, Assoc   8, LineSize  64
------------------*-  Instruction Cache  12, Level 1,   64 KB, Assoc   8, LineSize  64
-------------------*  Data Cache         13, Level 1,   32 KB, Assoc   8, LineSize  64
-------------------*  Instruction Cache  13, Level 1,   64 KB, Assoc   8, LineSize  64

You can see that the first 6 pairs of cores share caches and have larger caches than the other 8 cores.

This also happens on ARM devices such as the Samsung Galaxy Book go (with Snapdragon 7c, 4GB RAM) and the Windows ARM Dev Kit ("Project Volterra", with a variant for the Snapdragon 8cx gen 3 and 32GB RAM):

But on the Samsung Galaxy Book Go it's not deterministic. Sometimes it does utilize all the cores:

The first six cores are the LITTLE ones and the other two are the big:
Logical Processor to Cache Map:
*-------  Instruction Cache   0, Level 1,   32 KB, Assoc   4, LineSize  64
*-------  Data Cache          0, Level 1,   32 KB, Assoc   4, LineSize  64
*-------  Unified Cache       0, Level 2,   64 KB, Assoc   4, LineSize  64
********  Unified Cache       1, Level 3,    1 MB, Assoc  16, LineSize  64
-*------  Instruction Cache   1, Level 1,   32 KB, Assoc   4, LineSize  64
-*------  Data Cache          1, Level 1,   32 KB, Assoc   4, LineSize  64
-*------  Unified Cache       2, Level 2,   64 KB, Assoc   4, LineSize  64
--*-----  Instruction Cache   2, Level 1,   32 KB, Assoc   4, LineSize  64
--*-----  Data Cache          2, Level 1,   32 KB, Assoc   4, LineSize  64
--*-----  Unified Cache       3, Level 2,   64 KB, Assoc   4, LineSize  64
---*----  Instruction Cache   3, Level 1,   32 KB, Assoc   4, LineSize  64
---*----  Data Cache          3, Level 1,   32 KB, Assoc   4, LineSize  64
---*----  Unified Cache       4, Level 2,   64 KB, Assoc   4, LineSize  64
----*---  Instruction Cache   4, Level 1,   32 KB, Assoc   4, LineSize  64
----*---  Data Cache          4, Level 1,   32 KB, Assoc   4, LineSize  64
----*---  Unified Cache       5, Level 2,   64 KB, Assoc   4, LineSize  64
-----*--  Instruction Cache   5, Level 1,   32 KB, Assoc   4, LineSize  64
-----*--  Data Cache          5, Level 1,   32 KB, Assoc   4, LineSize  64
-----*--  Unified Cache       6, Level 2,   64 KB, Assoc   4, LineSize  64
------*-  Instruction Cache   6, Level 1,   64 KB, Assoc   4, LineSize  64
------*-  Data Cache          6, Level 1,   64 KB, Assoc   4, LineSize  64
------*-  Unified Cache       7, Level 2,  256 KB, Assoc   8, LineSize  64
-------*  Instruction Cache   7, Level 1,   64 KB, Assoc   4, LineSize  64
-------*  Data Cache          7, Level 1,   64 KB, Assoc   4, LineSize  64
-------*  Unified Cache       8, Level 2,  256 KB, Assoc   8, LineSize  64

Expected Behavior
All cores utilized.
Actual Behavior
Only "weak" cores are utilized.
This is the same behavior described in https://github.com/docker/for-win/issues/13562, but without Docker. Inbox CmDiag.exe is enough.

Temporary workaround would be to disable E-cores in bios. Then i think it would be faster with process isolation with P-cores, than Hyper-V with all cores.

@SauliusZ87: I've tried that and saw that when all the E-cores are disabled in the firmware settings indeed the container is scheduled to run on the P-cores, the only ones that are available. I didn't mention it in the issue because we don't consider it a viable alternative. For one, disabling the E-cores presumably disabled TXT.
Even worse, it significantly degrades the performance on the machine and its battery life, and running these build tasks inside the container isn't the only thing we do on them.
Until the issue is property resolved we prefer the trade-off in which we suffer the overhead of Hyper-V isolation during said builds but the rest of the time we have all 14 cores rather than speed up these builds but give up the 8 E-cores completely. Perhaps on a build server under constant load we would have gone the other way.

Since opening the issue we have made the following findings, both obviously related to scheduling:

If we modify the process affinity only to the P-core (and no E-core) the process is scheduled to the P-core, but only to half of them, i.e. only one of every two hyper-threaded cores. It sometimes even switch the logical hyperthreaded core the same physical core, but never runs on both logical cores of the single physical core.

If we set the process priority to above normal all the process is scheduled to all cores.

I hope that helps.
@SauliusZ87: That's far from a perfect workaround, but if you combine setting the processes to above normal priority and set a CPU limit (so it won't hog down all the CPU power and prevent other processes from running), it might be better than disabling the E-cores completely.
Process priority can be set via IFEO and it works for the Registry within the container and not only on the host.

This issue has been redirected to the appropriate team:
https://github.com/microsoft/Windows-Containers/issues/397


Prepare release to crates.io

What does this PR do?
Prepare lading crates for publishing to crates.io.
Motivation
This will get lading docs on docs.rs and will let us take a versioned dependency on lading-capture.
Steps before merging

[x] Set lading-capture and lading versions to 0.1.0 and 0.17.3 respectively.
[x] Release lading-capture and lading crates manually to create them on crates.io. The CI token intentionally lacks permission to publish new crates.
[x] Add SMP team to lading owners: cargo owner --add github:DataDog:single-machine-performance.
[x] Undo the manual version changes.
[x] Tag 0.17.4 and verify that it is released correctly.

This release is going to be a little ungainly due to the error fixed in f4a3b48. Rather than do something hacky, I'm going to add a note to use 0.17.3 in the github release notes.


Resnet50 e2e WH no-skips

Run resnet50 e2e on WH

Initial repro commands from working GS resnet50
git checkout shwetankTT/resnet
pytest -ssv /proj_sw/user_dev/tpatel/tt-metal-workspace/tt-metal-gs/tt-metal/tests/ttnn/integration_tests/resnet/resnet50.py::test_resnet

First failure occurs on shallow convs in tt_eager/tt_dnn/op_library/sliding_window_op_infra/tt_py_composite_conv.py:245
if padded_input_channels == 16 and (act_block_h_ntiles // out_subblock_h_ntiles % 2 != 0):
            assert is_1d_systolic
            # TODO: fix this temporary hack for shallow conv
>           assert act_block_h_ntiles % 2 == 0
E           AssertionError

Disabling this optimization for now

Next error: no L1 bank exists for core (x=8, y=0)
reader_indices_sharded_tensor = reader_indices_tt_tensor.to(self.device, mem_config)
E           RuntimeError: TT_THROW @ tt_metal/impl/allocator/allocator.cpp:235: tt::exception
E           info:
E           No L1 bank exists for core {}
E           (x=8,y=0)

Thanks Taps! Can you file issues for these, tag them w/ Resnet50, op_cat: sliding window, CNNs and assign to me

Actually, first do a bit of investigation on the second one, double check there isn't a shard spec override that using WH grid OOB

def determine_parallel_config(swo_params: SlidingWindowOpParams):
This function in file tt_eager/tt_dnn/op_library/sliding_window_op_infra/tt_py_max_pool.py has hardcoded values for GS. It needs to be changed for wormhole grid size (8x8 for n150 and 7x8 for n300). That is the cause of the above error where it's setting the wrong grid size and tries accessing a core that is not Tensix (so doesn't have an L1)

I selected a grid size of (8,8). Num of worker cores changed from 98 to using full 64 since 50176 / 64 = 784 (clean whole number). 25088 was determined based on input tensor size. [N, C, H, W] for input is [1, 3, 224, 224]. Convert this to [1, 1, NHW, C] is [1, 1, 1 * 224 * 224, 3] = [1, 1, 50176, 3]. This passed the l1 bank exists stage.

This function uint32_t get_num_cores(CoreCoord grid_size, uint32_t out_nhw, uint32_t nbatch) { in tt_eager/tt_dnn/op_library/pool/max_pool_multi_core.cpp needs to be updated with to account for WH grid sizes. It gets called in this line uint32_t ncores = max_pool_helpers::get_num_cores(input.device()->compute_with_storage_grid_size(), out_nhw, nbatch); when trying to create output tensors in tt_eager/tt_dnn/op_library/pool/max_pool.cpp. It returns a value of 98 which is incorrect and leads to assertion error down the read in this function num_cores_to_corerange_set because it tries setting output tensor core size to 98 but total core count is only 64.
Hardcoding this to 64 passes

After making these changes, resnet50 runs end to end on WH.

How's the perf looking?

This can be in CI on N150s (8x8) ?

performance: 59fps
Working on getting this into WH CI after refactoring and some cleanups

Updating here too, we are now at 625fps

Closing issue as test on WH has been merged in

Reopening as there are still certain features to enable (eg bfp8_b)

Task is done, follow on work tracked via CNNs on Metal project


Mesh metrics don't work with just generate mesh

The notebook for mesh metrics show it with a mesh that it loads from file. Now I try to compute them from a just generated mesh without saving and loading it. The basic approach from the script is
# Image processing happening before this
mesher = Mesher2D(segmentation)
mesher.generate_contour(max_contour_dist=3)
# Some padding and setting region markers left out
mesh = mesher.triangulate(opts='pAq30a')
metrics.histogram(mesh, metric='max_angle')
# ax = metrics.plot2d(mesh, metric='max_angle')

As result it errors on making the histogram
Warning: Appending zeros to replace the missing geometrical tag data.

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
/var/folders/yp/w25rz7hs47n4mhxgqmbl8xch0000gn/T/ipykernel_64069/1301693132.py in <module>
      6 fig.set_size_inches(24,4)
      7
----> 8 metrics.histogram(mesh_local_thresh_wave, metric='max_angle')
      9 ax = metrics.plot2d(mesh_local_thresh_wave, metric='max_angle')

/usr/local/Caskroom/miniconda/base/envs/nanomesh/lib/python3.8/site-packages/nanomesh/metrics.py in histogram(mesh, metric, ax, **kwargs)
    296
    297     descriptor = _metric_dispatch[metric]
--> 298     quality = descriptor.func(mesh)  # type: ignore
    299
    300     fig, ax = plt.subplots()

/usr/local/Caskroom/miniconda/base/envs/nanomesh/lib/python3.8/site-packages/nanomesh/metrics.py in __call__(self, mesh)
     23
     24     def __call__(self, mesh: BaseMesh) -> np.ndarray:
---> 25         grid = mesh.to_pyvista_unstructured_grid()
     26         ret = grid.compute_cell_quality(self.metric)
     27         quality = ret.cell_data['CellQuality']

AttributeError: 'MeshContainer' object has no attribute 'to_pyvista_unstructured_grid'

This is working as intendend. You must specify which cell type you want to run the metrics on.
The triangulation always returns a MeshContainer, which includes line and triangle cells.
Try this:
triangle_mesh = mesh.get('triangle')
metrics.histogram(triangle_mesh, metric='max_angle')

That indeed works. Could it be useful to highlight/document this difference?
As idea, the metrics example starts with:
mesh = TriangleMesh.from_meshio(meshio.read('out.msh'))
mesh.prune_z_0()
mesh.plot()
....
metrics.histogram(mesh, metric='min_angle')

Which might not make it directly clear that TriangleMesh is quite different from the MeshContainer. The latter does contain multiple meshes, but in practice I do think of it like the mesh for the highest dimension.

Yeah, good point. That notebook needs some attention anyway. I don't think I updated since I first made it 😅

I updated the notebook in #238, so I hope that addresses the issue. I will add some sort of workflow chart to the documentation (#241) to make the distinction between the different types.
Closed by #238


Ashlands update bug, blood sea everywhere

I think the blood sea starting to generate from a specific distant from spawn and increasing the world size wont change it

Unfortunately this might be hard coded. I haven't found a way to change it.


favicon & mobile nav

Hello,
Aqua is a great product, and I have two questions.
Every time I restart the server the favicon gets replaced with the aqua one. Also, how can I set a responsive nav for mobile phones on the home screen?
Thank you,
Kyle.

Every time I restart the server the favicon gets replaced

I'm not sure how you're editing the favicon but in the latest version, the favicon can be updated in the server side page templates:

https://github.com/jedireza/aqua/blob/4e7cb75253a04e870b73e61a73842f896ecf983e/server/web/admin/index.jsx#L15
https://github.com/jedireza/aqua/blob/4e7cb75253a04e870b73e61a73842f896ecf983e/server/web/account/index.jsx#L15
https://github.com/jedireza/aqua/blob/4e7cb75253a04e870b73e61a73842f896ecf983e/server/web/main/index.jsx#L24

Also, how can I set a responsive nav for mobile phones on the home screen?

I just merged a new universal front-end (like 30 mins ago). This works just like the admin now. So all should be good.

Reza,
Thank you!


Vp9 unit

use base's setFormat and ensureProfile, and implement decode API unittest for VP9.

it's done, @xuguangxin, thanks for your suggestion.

add jpeg unittest relative patches, please help to review. Thanks.


Drone 0.8 Compatibility

Drone 0.8 receives its config via environment variables, so is therefore not compatible with the current version of this plugin.

Hi, what is the status of pyton client?

Since Drone lost a lot of the magic which requires the custom JSON parsing this lib hasn't been updated anymore. You just need to read environment variables, so IMHO you won't need this lib anymore (at least for plugin development). It just needs to be updated in case of the Drone API client.


Using Popover component in Dropdown Button

Dropdown component should use Popover component in it to show action list

There is a slight difference in behavior between the current dropdown popover and the popover component.
If we don't pass the "fit-size" attribute, the popover will size itself according to the content size. The current implementation has a minimum width of the button size.
However, when we do pass the "fit-size" attribute, the popover sets its width to be the same as its target, and the bl-dropdown-item will cut off the text with an ellipsis.
Short content without fit-size:

Long content with fit-size:

Long content without fit-size:

Long content and short text with fit-size:

I have a couple of approaches listed, feel free to suggest new ones:

We may change the implementation of the popover component by changing width to min-width. I am wary of this one since I personally haven't used it before.
We may extend the implementation of "fit-size" prop by changing it to a string with options: width, min-width, max-width. And defaulting to width.
We may add a "fit-size" prop to the dropdown component, allowing users to make the decision themselves.
We may ignore the issue and leave it as is; functionally, nothing will change, but the behavior will be obviously different.

Draft

Good catch. Since our expected behaviour is as in dropdown component (it’s also same in Select component) popover fit-size should work as min-width, I think. Adding a custom value for this attribute can be considered in the future, once we need it (as your 2nd suggestion). I’m not sure if its possible to use an attribute as both boolean and as a string at the same time though. If so, <bl-popover fit-size> will use min-width, <bl-popover fit-size=“width”> will use width. I didn’t check but I assume this would be possible.


docs: backport compatibility info changes to 9.x

This is a backport of #470.
Idea is to make clear when/why to use v7, v8, v9 or v10. Backporting it here in case somebody gets onto this branch via a tag or some release notes.

lgtm


Centralize script python window

hi all,
i would like to see with you a topic
i want to centralize the window of a script in python
he is in usr/bin
when i use
Group
Name-jwmsession-/Name
Option-centered-/Option
/Group
in file groups does not work!
any idea?
vlw fwi, Holmes :)

hi @Israel-
see the result of the xwininfo -all
https://gist.github.com/holmeslinux/8a07c33fc0d1781662020227a2212a2e
how to interpret?
vlw fwi, Holmes :)

@Israel-
if you want to test the script
https://gist.github.com/holmeslinux/3d01840aecb43ef9a1eab0ca9ec11b0c
the idea is to centralize the window!
vlw fwi, Holmes :)

@holmeslinux
according to your output
xwininfo: Window id: 0x2000002 "JWMSession"
You need to use
<Group>
  <Name>JWMSession</Name>
  <Option>centered</Option>
</Group>

Note the capital JWM

hi,
i already did that and it did not work...
vlw fwi, Holmes :)

@holmeslinux I am so sorry I gave you the wrong command... you need to run
xprop
:smile:
And after running it I get:
WM_NAME(STRING) = "JWMSession"
Are you sure you tried:
JWMSession
But really... you should do that in your application..
See here:
http://stackoverflow.com/questions/20243637/pyqt4-center-window-on-active-screen#20244839

RESOLVED
with self.move(QtGui.QApplication.desktop().screen().rect().center()- self.rect().center())
thank you @Israel-
vlw fwi, Holmes :)

@Israel-
although the window is centered
now I'm trying to hide in the toolbar
in the groups file use the nolist option, but does not work
any idea?
vlw fwi, Holmes :)

@holmeslinux I don't program in Python, so I don't know off the top of my head how to do it in that language... jwm-settings-manager has something similar to what you are suggesting already but it is FLTK and C++. The image on the README.md
Did you specify JWMSession (case sensitive) AFAIK JWM reads the actual name of the window, so it should work to do this, but again, you should make your program do this rather than JWM, you should look in the documentation for PyQt4 to learn how to do this.

@joewing
perhaps it would be interesting to add something in groups file
Name-exec:path/file_name-/Name
vlw fwi, Holmes :)

Resolved (nolist)
with self.setWindowFlags(QtCore.Qt.Tool)
thank you @Israel-
vlw fwi, Holmes :)

@holmeslinux Great!  glad to hear it!


add torchtuples feedstock

Add feedstock recipe for torchtuples, a small python package for
training PyTorch models.
See https://github.com/havakv/torchtuples

Did you mean to include build.out in the PR?

Definitely didn't mean to include build.out :) Removed.


Support for Nested objects

Do you have any plans for support of Nested Objects in a form?
For example:
{
    "$schema": "http://json-schema.org/draft-04/schema#",
    "type": "object",
    "properties": {
        "nested": {
            "title": "nested",
            "type": "object",
            "properties": {
                "builds": {
                    "title": "builds",
                    "type": "array",
                    "items": {
                        "type": "object",
                        "properties": {
                            "builds": {
                                "title": "builds",
                                "type": "string"
                            },
                            "time": {
                                "title": "time",
                                "type": "number"
                            },
                        }
                    }
                }
            }
        }
    }
}

I can contribute some time if necessary, would be glad to help!

Hello @icosamuel, thank you for your feedback. An implementation of the nested objects would be great for the project and any contributions are welcome :-)

@demsking Great! I will try provide a working example in the upcoming weeks.

Ok. We can exchange on Gitter if you want: FormSchema/native room

Anyone has any updates regarding this feature? It would be nice if root objects could be categorized as tabs or menu and their children in a form.

Adding a PR that relates to this
https://github.com/formschema/native/pull/130

This project has been moved to https://gitlab.com/formschema/native


Retry failed job does not work or is not implemented

Cannot retry failed jobs because PATCH http://localhost:4567/api/queue/application/docgen/job/3 fails with 404.

Fixed in release v1.4.0, thank you for the report.


Table using ServerData and Group with InnerGroup display duplicate data

Bug type
Component
Component name
MudTable
What happened?
When table is using server side data and inner grouping was defined, duplicate/incorrect data is displayed when navigate to next page.
Code-Snippet: https://try.mudblazor.com/snippet/GawPOXcwxFDtWOgi - this code is based from https://mudblazor.com/wasm/components/table#server-side-filtering,-sorting-and-pagination but with inner group definition. When Navigate to next page, it display the first page items (Nr 1, 2, 3) instead of displaying page 2 items (Nr 11, 12).
When run without InnerGroup definition, behavior work as expected.
Expected behavior
Page 2 should display Sodium and Magnesium
Reproduction link
https://try.mudblazor.com/snippet/GawPOXcwxFDtWOgi
Reproduction steps

Run the code
Remember the first 3 names
Click next page
Next page showing 1st page items

Relevant log output
No response
Version (bug)
5.1.4
Version (working)
No response
What browsers are you seeing the problem on?
Chrome, Safari
On what operating system are you experiencing the issue?
Mac OSX
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks for the capture @wilchn. I'm investigating how to resolve this.
Feel free to suggest any correction or join the work to resolve it, if you wish.

After some tests, I figured that the problem occurs with Pagination + Grouping (with inner level(s) ). Inner groups are caching the first page items only. It's not related to Server Data.


Unable to disable cache on a per-database basis.

I currently have a project with two database connections; a PostgreSQL database for all the usual Django things and an external MySQL data we're using to pull data from.  If objects get deleted from the external MySQL database using that provider's tools (not though Django) django-cachalot (obviously) doesn't know that's happened and continues to return data which no longer exists (and messes up some of the Django logic).  What would be great is if we could disable cachalot on the MySQL database, but keep it on the Django one - is this possible?

You can do this by customizing CACHALOT_QUERY_KEYGEN and raising an UncachableQuery exception if the query is using MySQL.
Suppose the aliases you use are 'default' for PostgreSQL and 'mysql' for MySQL, you can write this:
from cachalot.utils import UncachableQuery, get_query_cache_key as original_query_keygen

def get_query_cache_key(compiler):
    if compiler.using == 'mysql':
        raise UncachableQuery
    return original_query_keygen(compiler)

Then define CACHALOT_QUERY_KEYGEN = 'path.to.where.you.put.get_query_cache_key' in your settings.py.
It should be that easy. Could you confirm?

That worked well, thank you. I do still think it would be nice to have an option, perhaps in the DATABASES setting which gives you this control without adding utils. It's a nice workaround though. Thanks again.

I prefer providing simple workarounds for edge cases like this instead of adding a ton of settings. Adding settings has a performance cost since they are dynamic in cachalot.
With a flexible function setting like this, a lot of edge cases can be handled without having to wait for the next release to change its behaviour :)


Selectively disable modules for build

System information

**OS Platform and Distribution Linux Ubuntu 16.04)
Apollo installed from (source or binary):
**Apollo version 5.0:

I would like to disable some of the modules and only have select modules to be built and debugged to isolate effect from them on the module I am working on. For example: I only need drivers module on and all others turned off without impacting the full function of drivers.
How can I do this in Apollo? What do I need to modify? Please let me know
thank you

Take the camera module as an example.
build the module
You can compile the camera module separately
bazel build modules/drivers/camera:libcamera_component.so

start the module
You can use cyber_launch to start the module. here is a link
cd /apollo && cyber_launch start modules/drivers/camera/launch/camera.launch

Hi @jpcloud1577 , module-level build is only available in the r6.0.0  and master branch. You can build single module with command like,   ./apollo.sh build cyber, ./apollo.sh build_opt_gpu drivers, etc.
Please consider switching to the two branches I mentioned above and have a try.

Thank you @storypku and @daohu527 . I will give it a try

Hi @jpcloud1577 , hopes you have already had this issue resolved, then we will close this issue now. Please feel free to create another issue if you have other questions, or reopen this one if the problem persists.
Thank you.

Yes. thank you