Refactor the folder ui/viewmodel and ELSeedValidator. Replace RegExpValidator (deprecated) with RegularExpressionValidator.

Refactor ui/viewmodel
Determine which files and classes are considered "helper" files/objects and move them into appropriate subfolders.
This requires feedback from other contributors to decide best layout.
Suggested (or to be considered) files  -- along with their .h counterparts of course

currencies
el_seed_validator
fee_helpers
qml_globals
settings_helpers
ui_helpers
window_event_filter

Refactor class ELSeedValidator

ELSeedValidator inherits from QValidator, and then seems to implement a regular expression validation manually in its validate() method. The validate method contains the allowed regular expression to be checked against.
Shouldn't this operate in the same way as every other validate method? That is, the regular expression is defined in the QML file, in this case SeedInput.qml.
So, shouldn't ELSeedValidator simply inherit from RegularExpressionValidator so it can call the parent validate() method to check the regular expression, and then do the extra code in its validate() method?

Refactor RegExpValidator references
RegExpValidator is apparently deprecated and it is recommended to use RegularExpressionValidator

@anatolse


Can you clarify if the below scenario results into deadlock?

for spark on k8s with spark operator kindly tell me how it works in this case

there are only 4 nodes and each with 4cores and 16gb

i submit a 100 jobs each taking 1 core and 1gb ram

assume 1 core 1gb per machine is given to OS. so effectively we have 3 X 4 = 12 cores

now will this scenario spin 12 drivers first? each driver wants to spin executors. but there is no room for executors. So Wouldnt all drivers wait forever? kind of deadlock. And other 88 jobs fall in queue.
Assume each driver needs 50 executors to run the program. Now what happens?
In case of yarn on k8s,

It spins only some jobs. meaning each driver will spin few executors.
It doesn't wait till 50 executor JVMs are available per driver. But schedules some of the executor JVMS to the extent available on the cluster. Meaning we have parallel spark jobs running, no resources are wasted, and all of the jobs are complete.

I expect similar behavior with spark operator on native k8s. But Need clarification. Any help?
Can you kindly let me know how this scenario behaves/ is handled?

What you're describing is a Gang scheduling problem and we've faced the same issue. It's because Spark relies on the Kubernetes default scheduler to first run drivers, and then each driver is responsible for creating executor pods. So the k8s default scheduler doesn't understand the assocation between a driver and it's executors. Only if you could somehow group them together
We're using the Yunikorn scheduler to solve this problem in our cluster. But there are a few more options out there that might work e.g. the Volcano  scheduler
I've found this blog very useful: https://blog.cloudera.com/spark-on-kubernetes-gang-scheduling-with-yunikorn/

Yes. its gang scheduling. I discovered it late. Also, From spark page I see that external scheduler is not yet available on k8s --so how are you handling this part ?


hmr

unregister component feature might be needed. https://github.com/w3c/webcomponents/issues/152

https://www.w3.org/TR/custom-elements/#custom-elements-upgrades-examples

Hi @aruntk, can you give me some insight with something?
I'm the mantainer of a similar loader but instead of using parse5, it directly uses vulcanize. I encountered the same problem with the downgrading of custom elements.
If you have resolved it (the README file points it out) could you explain me how did you done it?

@aitoroses Hot code reload works now(reload on element file change). I'vent yet figured out how to implement hot module reload. I think as of now it is not possible to implement a complete element = module re-rendering since there is not way to unregister a custom element (afaik). Instead what I'm trying to implement is a partial re-rendering (functions, inner html and property values). I hope it'll lead to a near complete re-rendering.


point at master branch for wasmtime

this is slightly ahead of the 0.15 release.
the v0.15.0 tag is not on the master branch, unfortunately.
Supercedes #482

Thank you for doing this, @pchickey !

@froydnj can i bug you for a quick approval via the github reviews feature?


Limit GPU memory usage not working in distributed training of inception

I'm using the example provided for distributed trainig of inception. I have three hosts, with one for parameter server and two for workers. All hosts have TensorFlow 1.2 installed with CUDA 8.0, cuDNN 5.1 and Titan X Pascal GPU running Ubuntu 14.04.
I followed the instruction for distributed training as provided in the readme and it runs successfully. But when I try to limit the GPU usage by making the following changes.
Change
sess_config = tf.ConfigProto(
allow_soft_placement=True,
log_device_placement=FLAGS.log_device_placement)
to
sess_config = tf.ConfigProto(
allow_soft_placement=True,
log_device_placement=FLAGS.log_device_placement,
gpu_options=tf.GPUOptions(per_process_gpu_memory_fraction=0.6))
in inception_distributed_train.py
I recompiled the example using 'bazel build //inception:imagenet_distributed_train' and run the example. But the process still occupies all GPU memory available observed using nvidia-smi.

The device options are interpreted somewhat differently in the distributed version of TensorFlow, because they are properties of the tf.train.Server, rather than an individual tf.Session. If you pass the same sess_config object as the optional config arg to the tf.train.Server constructor, you should get the desired behavior.
I'll close this issue for now, but feel free to reopen if it doesn't solve the problem.

Thank you @mrry ，that solves the problem.

Thanks for confirming!


Improve Web Components Section

Hello there,
for a recent client of mine I had to dive into Web Components and how to use them in React. Their goal is to create a UI library, based on Web Standards, which shares a common structure, style  and behavior. This way they would enable all their developer teams to pick their frameworks of choice (React, Angular, ...) to wrap these common UI components. Long story short: I had to come up with a way to use Web Components in React.
It's fairly straight forward, but I didn't find lots of documentation in the official React docs. Since I wrote extensively about Web Components and how to use them in React myself, I thought it may be helpful for people who face the same scenario.
Optionally it comes also with a custom React hook which takes care about formatting information to JSON and attaching (and detaching) event listeners. I hope it's useful for the community 🚀 🙂
Note: The original PR is from June 2019 https://github.com/reactjs/reactjs.org/pull/2094, but I applied some improvements

Closed in favor of https://github.com/reactjs/reactjs.org/pull/2633


Add PERF logging level for performance metrics

Closing because of a different approach.
The approach, delivered in https://github.com/Senzing/go-logging/issues/24, will allow adding a "detail" of time.Duration.   If this is seen in the []details, then a duration: JSON key will be added to the message with the time.Duration.Nanoseconds value.


ObjectId in query builder not working

I have a Laravel scope that adds condition to a model globally.
$builder->where(['rankedSeason' => new MongoDB\BSON\ObjectId(self::$activeSeasonId)]);

However it gets translated and casted to string when I inspect the query that runs:
rankedMatches.countDocuments(
  { "$and" :[{"player":"e908fe9e-97a6-4dc2-b1ff-55c5b9a87ce2"},{"player":{"$ne":null}},{"rankedSeason":"6555d1f82039d742bfbcfa70"}] },
  []
)

It should cast it to ObjectId instead.

What is your laravel-mongo package version?

Hi, I am using
"mongodb/laravel-mongodb": "^4.0.0-rc1"

Can you test this using 4.1 version?

@hans-thomas just tried with 4.1.x-dev 1c2d1fe same issue there. Thanks for looking into this!

I don't see any error in Builder class. I guess your rankedSeason is stored as a string in the DB and that's why you're getting empty results.
I wrote this test and it's passed.
$objectId = (string)(new ObjectId());
$user = User::create(['name' => 'Hans Thomas', 'rankedSeason' => $objectId]);

$query = User::query()->where('rankedSeason', $objectId);
self::assertSame($user->_id, $query->first()->_id);

But if the stored and queried rankedSeason data types weren't the same, it wouldn't pass.
$objectId = (string)(new ObjectId());
$user = User::create(['name' => 'Hans Thomas', 'rankedSeason' => $objectId]);

$query = User::query()->where('rankedSeason', new ObjectId($objectId));
self::assertSame($user->_id, $query->first()->_id);

If you think when you pass the rankedSeason as an ObjectId but it's a string in the query log, I recommend using toMql() to see the exact created mql.
$objectId = new ObjectId();
$user = User::create(['name' => 'Hans Thomas', 'rankedSeason' => $objectId]);
$query = User::query()->where('rankedSeason', $objectId);
dd($query->toMql());

// dd's output
array:1 [
  "find" => array:2 [
    0 => array:1 [
      "rankedSeason" => MongoDB\BSON\ObjectId {#1292
        +"oid": "655b4250ddd3884d3b0ae692"
      }
    ]
    1 => array:1 [
      "typeMap" => array:2 [
        "root" => "array"
        "document" => "array"
      ]
    ]
  ]
]

Closing as unconfirmed.


Validate if upgrade to Pi-hole requires changes to guide

Validate if upgrade to Pi-hole requires changes to guide
Changelog Pi-hole: https://pi-hole.net/2018/08/06/pi-hole-v4-0-released-with-ftldns-improved-blocking-modes-regex-docker-and-more/

No changes required. But the addition of DNS-over-HTTPS opens the door for new stuff. Created new ticket for that.


Remove manylinux CI workflow

@jimjag how do you feel about deleting the manylinux.yml workflow as well? The wheel build workflow does basically the same thing, and deleting it would get CI passing on this repo again.

I'm hesitant to remove it since it provides insight into how things will work on AWS and various PaaS environments, such as Heroku. Let's see if we can clean it up 1st.

@jimjag can you clarify what this workflow is doing that should be tested that wheels.yml isn't doing?

I'll close this one then, since @jimjag isn't interested in it.


Higher memory usage in pre-0.12 version using HTTP2 compared to h2 branch

We've been using the h2 branch since the early stages, a couple of days ago upgrading to the master pre-0.12. Something in the master branch uses more RAM than the old version, possibly the new pool. Our use case is many concurrent open connections, keeping them alive and using http2.
Here's an image showing the regression:

The deployment was 16.4.2018 around 11am local time. Also the noticeable growth happens when we hit the keep-alive timeout and get lots of requests through a new connection. So the culprit might be how we dispose the dead connections in the pool.

Is this http1 or 2 connections?

Http2. I checked what was updated after the deployment:
hyper from h2 branch to  0.12.0-pre.0 (HEAD), tokio-core 0.1.16 replaced with tokio 0.1.5. So there's quite minimal changes in the deployment, that's why the main culprit I can think of is hyper's new Pool implementation.
We'll monitor how it looks like after couple of days, will it settle into some memory usage or keep growing.

If at all possible, logs from hyper::client::pool,hyper::proto::h2 would probably also help.

Ok, so I'm having this running and logging with tee some hours with production traffic on a test machine.

First logs, if you need more trace points, please let me know. I could run this longer, but right now the production instances freed some memory, about 10% drop, so at least they're not going to OOM. Just something seems to use more RAM than before...
With long keep-alive timeout, reusing connections:
TRACE 2018-04-18T18:59:05Z: hyper::client::pool: checkout waiting for idle connection: ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T18:59:05Z: hyper::client::connect: Http::connect(https://api.push.apple.com/3/device/###########)
DEBUG 2018-04-18T18:59:05Z: hyper::client::dns: resolving host="api.push.apple.com", port=443
DEBUG 2018-04-18T18:59:05Z: hyper::client::connect: connecting to <IP_ADDRESS>:443
TRACE 2018-04-18T18:59:06Z: hyper::client::pool: Pool::put ("https://api.push.apple.com", Http2)
DEBUG 2018-04-18T18:59:06Z: hyper::client::pool: pooling idle connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T18:59:06Z: hyper::proto::h2: send body chunk: 404B
TRACE 2018-04-18T18:59:06Z: hyper::proto::h2: send body eos
TRACE 2018-04-18T18:59:06Z: hyper::client::pool: Pool::put; existing idle HTTP/2 connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T18:59:42Z: hyper::client::pool: take? ("https://api.push.apple.com", Http2): expiration = Some(Duration { secs: 600, nanos: 0 })
DEBUG 2018-04-18T18:59:42Z: hyper::client::pool: reuse idle connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T18:59:42Z: hyper::proto::h2: send body chunk: 660B
TRACE 2018-04-18T18:59:42Z: hyper::proto::h2: send body eos
TRACE 2018-04-18T18:59:42Z: hyper::client::pool: Pool::put; existing idle HTTP/2 connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:00:13Z: hyper::client::pool: take? ("https://api.push.apple.com", Http2): expiration = Some(Duration { secs: 600, nanos: 0 })
DEBUG 2018-04-18T19:00:13Z: hyper::client::pool: reuse idle connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:00:13Z: hyper::proto::h2: send body chunk: 480B
TRACE 2018-04-18T19:00:13Z: hyper::proto::h2: send body eos
TRACE 2018-04-18T19:00:13Z: hyper::client::pool: Pool::put; existing idle HTTP/2 connection for ("https://api.push.apple.com", Http2)

With short keep-alive timeout, creating new connections:
TRACE 2018-04-18T19:06:44Z: hyper::client::pool: checkout waiting for idle connection: ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:06:44Z: hyper::client::connect: Http::connect(https://api.push.apple.com/3/device/############)
DEBUG 2018-04-18T19:06:44Z: hyper::client::dns: resolving host="api.push.apple.com", port=443
DEBUG 2018-04-18T19:06:44Z: hyper::client::connect: connecting to <IP_ADDRESS>:443
TRACE 2018-04-18T19:06:45Z: hyper::client::pool: Pool::put ("https://api.push.apple.com", Http2)
DEBUG 2018-04-18T19:06:45Z: hyper::client::pool: pooling idle connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:06:45Z: hyper::proto::h2: send body chunk: 440B
TRACE 2018-04-18T19:06:45Z: hyper::proto::h2: send body eos
TRACE 2018-04-18T19:06:45Z: hyper::client::pool: Pool::put; existing idle HTTP/2 connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:07:04Z: hyper::proto::h2::client: client tx dropped
TRACE 2018-04-18T19:07:24Z: hyper::client::pool: checkout waiting for idle connection: ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:07:24Z: hyper::client::connect: Http::connect(https://api.push.apple.com/3/device/##############)
DEBUG 2018-04-18T19:07:24Z: hyper::client::dns: resolving host="api.push.apple.com", port=443
DEBUG 2018-04-18T19:07:24Z: hyper::client::connect: connecting to <IP_ADDRESS>:443
TRACE 2018-04-18T19:07:25Z: hyper::client::pool: Pool::put ("https://api.push.apple.com", Http2)
DEBUG 2018-04-18T19:07:25Z: hyper::client::pool: pooling idle connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:07:25Z: hyper::proto::h2: send body chunk: 404B
TRACE 2018-04-18T19:07:25Z: hyper::proto::h2: send body eos
TRACE 2018-04-18T19:07:25Z: hyper::client::pool: Pool::put; existing idle HTTP/2 connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:07:42Z: hyper::client::pool: take? ("https://api.push.apple.com", Http2): expiration = Some(Duration { secs: 10, nanos: 0 })
TRACE 2018-04-18T19:07:42Z: hyper::client::pool: remove unacceptable pooled connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:07:42Z: hyper::client::pool: checkout waiting for idle connection: ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:07:42Z: hyper::proto::h2::client: client tx dropped
TRACE 2018-04-18T19:07:42Z: hyper::client::connect: Http::connect(https://api.push.apple.com/3/device/#######################)
DEBUG 2018-04-18T19:07:42Z: hyper::client::dns: resolving host="api.push.apple.com", port=443
DEBUG 2018-04-18T19:07:42Z: hyper::client::connect: connecting to <IP_ADDRESS>:443
TRACE 2018-04-18T19:07:42Z: hyper::client::pool: Pool::put ("https://api.push.apple.com", Http2)
DEBUG 2018-04-18T19:07:42Z: hyper::client::pool: pooling idle connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:07:42Z: hyper::proto::h2: send body chunk: 440B
TRACE 2018-04-18T19:07:42Z: hyper::proto::h2: send body eos
TRACE 2018-04-18T19:07:43Z: hyper::client::pool: Pool::put; existing idle HTTP/2 connection for ("https://api.push.apple.com", Http2)
TRACE 2018-04-18T19:07:54Z: hyper::proto::h2::client: client tx dropped
TRACE 2018-04-18T19:07:56Z: hyper::client::pool: pool closed, canceling idle interval

The only thing I can think of is that something is holding on a reference to the h2 streams type, and perhaps being stuck in a future that doesn't poll as finished. And then eventually, Apple closes the connection and that triggers the internal h2 streams type to clean itself up.
I've pushed some minor clean up to master, along with some clearer logs, if you want to take a look.

 The memory leak is about 60-100M, meaning it's much more than a couple of dead connections (should be about 6 per customer). I'd say something is not cleaned when we race the new connection in the pool, but I'm not sure.

it happens when a dead connection gets a big spike of requests

What determines it is dead? The remote has closed the connection, but the pool hadn't yet cleared out the handle? Or it's reached its expiration and was waiting for the idle interval to clean it up?

Monitoring the latest 0.12 for the weekend, the memory usage is bigger but doesn't really grow above certain limit. Also we notice drops every now and then, evening it out. The graph starts from a deployment and ends to the current timestamp.

The RAM usage was reduced by several commits before, but this being the best deal and now the usage actually even gets lower when running the system longer. I consider this fixed now.

For reference, it was noticed that Weak::new() allocates space for the thing that the related Arc<T> would hold, even though Weak::new() can never upgrade to a strong reference by design. The fix in hyper was to create a single Weak::new() per pool, and clone that instead of creating Weak::new()s for every single waiter.
I've open a PR to libstd to fix this easy to miss issue.


Can't see data list in drop down

Can you please help me out data list in drop down - Plnkr

Hi. In wrapper.component.ts line# 49 you are emptying the array. Very obvious you would not see the list. Comment this line and see, it works.
PS. Please raise only bugs related to the component here. For any help on implementations do drop a mail to me. happy coding :)


Ending cutscenes and where do I save dialogue picture and XML

What is your question?
Can I make ending cutscenes after the final song of the mod and if so, how and where do I save the dialogue png and XML

Don't mind me just making my first comment ever

Dialogue picture and XML goes inside mods/images/dialogue/, which is also the same folder as of where the .json files
If you mean a video after the song ends: https://github.com/ShadowMario/FNF-PsychEngine/issues/230


Inconsistencies in gRPC rq_5xx stats

I'm looking into how an application can leverage Envoy stats to measure an SLO in terms of the ratio of "successful" requests to "failed" requests. The service I'm currently looking at supports both HTTP and gRPC for the same logical endpoint.
The first stat I looked at was the downstream_rq_5xx one, thinking that it might also include requests with a gRPC status code that maps to an HTTP 5xx code. I found that's not the case, although there are some edge cases when Envoy sends a local reply.
For instance: when an upstream timeout occurs, the timeout_response_code (defaults to 504) gets charged to the stats, and then this is converted to a 200 response code in sendLocalReply. This means that it's counted in upstream_rq_5xx, but NOT in downstream_rq_5xx because the conn manager sees the 200.
To summarize, there are two seemingly strange inconsistencies:

a gRPC error returned by the upstream (e.g. DEADLINE_EXCEEDED) will not be counted in upstream_rq_5xx, but one returned by Envoy local reply (e.g. upstream timeout) will be
Envoy local reply errors for gRPC requests count toward upstream_rq_5xx but not downstream_rq_5xx

I imagine we can't just change the meaning of these stats, but ideally we'd decide that either we always map gRPC codes to http codes and charge them in both upstream and downstream stats, or we never do and emit separate gRPC codes stats.

I think this is a dupe of https://github.com/envoyproxy/envoy/issues/5720 actually, which I forgot I filed!

I imagine we can't just change the meaning of these stats, but ideally we'd decide that either we always map gRPC codes to http codes and charge them in both upstream and downstream stats, or we never do and emit separate gRPC codes stats.

Yeah I agree it would be nice to be consistent here, but honestly, this is very confusing so I think we will need a concrete proposal that takes into account the new gRPC stats filter.


chore: Mock API locally for e2e tests

Note that for merging annd afterwards the CI checks need to be updated (ping @hugotiburtino @eliflores )

@kulla the e2e is flaky, I'm just fixing it before sending it to the queue again


[docs] API docs missing information since flow 0.53.x upgrade

Problem description
update/fix react-docgen after https://github.com/reactjs/react-docgen/issues/207
Steps to reproduce
yarn docs:api
Temp debug script is ./docs/scripts/buildApiES5.js

Also, I thought I'd ask here before opening another issue. There are some issues getting the demos to show on the site. I see that the document structure has changed so I tried different routes only to get 404's and build errors. Is this still a WIP or should i reproduce the issue and provide more feedback?

@zabieru96 that would be a new issue. Docs should be 100% working.

@oliviertassinari this has been released so I'm working on it now.

We need to be on v3-beta or better and it has yet to be released.

@rosskevin Would rollbacking to flow 0.53.0 help?

No, same inference types are used in 53.

I released beta7. let me know if it works
https://github.com/reactjs/react-docgen/releases/tag/v3.0.0-beta7

It works, thanks @danez


⚠ Secret Website 3 has degraded performance

In 30e5a56, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 1916 ms

Resolved: Secret Website 3 performance has improved in 8fd2242.


0.2.0 on PyPi

Trying out f_it today, pretty sweet so far - I noticed the lack of the .to method before realizing that was introduced on 0.2.0.
Any chance 0.2.0 could be pushed to PyPI? Or any reason I should be cautious not to use 0.2.0?

Glad it's helpful!
There's no reason it was held back; tox was behaving weirdly, which failed the build (despite the tests and so on passing), which prevented the auto-deploy. I'll see if I can get that fixed.

@clbarnes thanks so much - I don't know much about tox but happy to help if I can!

Fixed the tox issue - PEBKAC, I was deleting the tox dir while it was running. Fix merged, CI passed, v0.2.1 deployed :)


updated with .env file

[ODS-4640] updated env file on .gitignore

changes to push on previous pull request


Added link to the adaptive designer site

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/actionable-messages/adaptive-card.md
:white_check_mark:Succeeded

docs/actionable-messages/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


#1116 provide check_java_code_style.sh and JavaCodeFormatter.xml for eclipse

@caofangkun - can you please check what @billonahill suggested and let us know?

@caofangkun any progress on this?

@caofangkun could you add comments to the top of this file, or a brief readme that explains how it should be used.


Allow Htmlable on Table actions column label

Description
Allows to render HTML content in a table action column label as would in other table column labels.
Visual changes
$table->actionsColumnLabel(new HtmlString(Blade::render('<x-heroicon-s-bolt class="w-5 h-5"/>')));

Or by default with Allow Table static default Actions Column Label
Table::$defaultActionsColumnLabel = new HtmlString(Blade::render('<x-heroicon-s-bolt class="w-5 h-5"/>'));

Functional changes

[x] Code style has been fixed by running the composer cs command.
[x] Changes have been tested to not break existing functionality.
[ ] Documentation is up-to-date.

Thanks

Actually gonna revert the static var part - you can use this https://filamentphp.com/docs/3.x/tables/advanced#global-settings


check sno

check ocp infrastructure's status ControlPlaneTopology to determine whether the cluster is sno
open-cluster-management/backlog#11675
Signed-off-by: Marco<EMAIL_ADDRESS>

hold until https://github.com/open-cluster-management/multicluster-observability-operator/pull/541 merged
/hold

/retest
/unhold

/retest

/retest

/retest
/lgtm

/retest

/retest

/retest

/retest


Support default copyright format of IntelliJ IDEA

Currently there are issues with the default copyright injection of IntelliJ IDEA, notably on XML files.
It would be great to support these formats.
The IntelliJ IDEA default is:
<!--
  ~ Copyright Lorem ipsum
  ~ Lorem ipsum
  -->

and it has to be configured to become:
<!--
  Copyright Lorem ipsum
  Lorem ipsum
  -->

otherwise a license violation is reported.

Using version license-maven-plugin:3.0 and the configuration capabilities we can redefine the format for XML files
idea-license-header-definition.xml
<?xml version="1.0" encoding="ISO-8859-1"?>
<additionalHeaders>
    <ideaxml_style>
        <firstLine><![CDATA[<!--]]></firstLine>
        <beforeEachLine>  ~ </beforeEachLine>
        <endLine><![CDATA[  -->
]]></endLine>
        <skipLine><![CDATA[^<\?xml.*>$]]></skipLine>
        <firstLineDetectionPattern><![CDATA[(\s|\t)*<!--.*$]]></firstLineDetectionPattern>
        <lastLineDetectionPattern><![CDATA[.*-->(\s|\t)*$]]></lastLineDetectionPattern>
        <allowBlankLines>true</allowBlankLines>
        <isMultiline>true</isMultiline>
        <padLines>false</padLines>
    </ideaxml_style>
</additionalHeaders>

configuration in pom
<plugin>
    <groupId>com.mycila</groupId>
    <artifactId>license-maven-plugin</artifactId>
    <version>3.0</version>
    <configuration>
        <header>com/mycila/maven/plugin/license/templates/APACHE-2.txt</header>
        <headerDefinitions>
            <headerDefinition>idea-license-header-definition.xml</headerDefinition>
        </headerDefinitions>
        <mapping>
            <xml>IDEAXML_STYLE</xml>
        </mapping>
        <properties>
            <owner>Somewhere INC.</owner>
<EMAIL_ADDRESS>        </properties>
        <excludes>
            <exclude>**/README</exclude>
            <exclude>src/test/resources/**</exclude>
            <exclude>src/main/resources/**</exclude>
            <exclude>idea-license-header-definition.xml</exclude>
        </excludes>
    </configuration>
    <executions>
        <execution>
            <phase>validate</phase>
            <goals>
                <goal>check</goal>
            </goals>
        </execution>
    </executions>
</plugin>

IDEA configuration

add the Copyright to the pom.xml file
check licenses in maven
pom.xml is checked successfully

@ljacomet @McFoggy : ahah I finally have time to look down the issue list and found this one.
Do you think it is worth adding a new style in the plugin for XML ? I.e. IDEA_XML_STYLE ?

@mathieucarbou I also use intellij.

Do you think it is worth adding a new style in the plugin for XML ? I.e. IDEA_XML_STYLE ?

Personnaly (and in my company also) on the projects where I use license-maven-plugin I do not configure Intellij to modify the headers ; it is left to the plugin.

Given that there is a configuration option, I do not think it is worth having a custom format for this. The only thing that might be worth a change would be to allow line prefixes to contain one non-whitespace character.
But I have no idea how complex this could be, and I no longer work on a project that had this issue so can't commit time to help improve the situation.


fix: use min value to check the first partition

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


obfuscated function and variable names for js/vbs?

I was thinking about this last night when looking into how koadic uses the scripts. I noticed that it opens them up and replaces some names with the proper settings.
I am wondering if there is any benefit for setting the functions and variables to random, since they should mainly be in memory only.  As I understand it no real AV supports in memory scanning, and if they do I doubt they would hit on a specific function name or combination rather.
Still tho it bothers me that they are not modified, one of the solutions I came up with while working on my own project was to add each line and substitute it out for a random name. This is painful and cumbersome especially when dealing with multiple scripts and languages (eventually I would also like to support PS).
However after looking at the issue again I think I came up with a rather elegant solution. We can replace each function and variable for our scripts in the data directory with "__function_", or "__variable_" then we can write a regex oneliner to replace all found instances with our randomly generated names.
I think this would be the best solution if it needs implementing as the scripts should still work (underscores shouldn't affect execution) and we can always add in some industrial strength obfuscation scripts at a later time.
It shouldn't take to long to modify everything, I'm guessing around a day or so for all the scripts.

This is definitely something I have wanted to look into. We should be able to just do a pass with a proper JavaScript obfuscation library instead of having to write one.  We need one that produces random output (not a "minimizer" which is deterministic).

This seems to become more pertinent now.
https://www.microsoft.com/en-us/wdsi/threats/malware-encyclopedia-description?name=Trojan%3AJS%2FKoadic.G!attk&threatid=-2147244324&enterprise=0
:+1: :100:

Doing work on this over on the 'obfuscate' branch. We've got a working xor cipher implementation that works on payloads as large as our mimikatz shellcode. Since I've got this in the works, I'll go ahead and close.


which image for upgrades?

The documentation on the main page does not make it clear which sd card image is best for upgrades from previous versions.
The new scheme (fat) (min) does not illustrate which type of image came before either.

You can choose which image you use. The important thing is the version number.
There is no upgrade in the way that that the sd-card in use gets (hot)fixes.
You should use another sd-card, flash a new version (fat or min) and replace the old sd-card.
Don't know about downgrading.

This is correct as covered in the readme. Please detail if you need more clarification.

You can upgrade with both images fat & min - but only the fat offers the WebUI for recovery/update. When in doubt use fatpack image.


M3-1378 Access Panel Fix

Purpose
Fix issue where access panel was not rendering correctly on Create Linode from StackScript form

Coverage remained the same at 26.394% when pulling 4aa49c76587408bb54a344c6c1d17888601241ed on martinmckenna:M3-1378 into 64999b211d247832e16103bf8e340720202f69fd on linode:develop.


Add next CVI - self-provisioning

I am adding this cvi doc to continue with the furyctl self-provisioning feature to add GKE / Google Cloud provisioners.

Closed in favour #29


[Question] Will you publish this extension in P9 extension downloader?

Basically the question is in the title. Thanks for the awesome extension!

Pre-requisite manifest file was added in https://github.com/konnogatto/SwitchDisplay/pull/8
PR to add to addon repo - https://github.com/JosefNemec/PlayniteAddonDatabase/compare/master...sharkusmanch:patch-3

Now available in the addon browser under generic


Implement Pushpin from Materialize.css

Pushpin provides scrolling behavior which was very useful in combination with scrollfire. This should only require a directive wrapper to get going.
http://materializecss.com/pushpin.html

Simple directive to pair with scrollspy.


Check if scanning for systemwide TCP Table is unnecessary

https://docs.microsoft.com/en-us/windows/win32/winsock/sio-tcp-info

It works with 4aaa8a7a9f2aee963c3332bb2af8ad3a1d3a524f.


Implement overloaded procedures

This will unlock classes (methods)

This isn't really blocking classes, per se.


chore: add docstrings for response types

I agree that would be great, but I think this is a good incremental step for now? We can flesh them out further as time permits?

Absolutely


WIP: Stop to use container for CI

Current GitHub Actions Runner requires 30sec to start up
each of containers.
This try to reduce its overhead.

By this change, we can reduce 1min for the total workflow.

Pros: actions/setup-node@v1 is slow to update their node.js database. So we cannot use a latest node.js.....
The same problem happens with docker container, but they would update soon in few days after releasing node.js

We retry this in https://github.com/option-t/option-t/pull/1818


AGP 7.0.3 compatibility changes

Please merge this pull request. I need to use it with AGP 7 support for using it along with Android Jetpack Compose.

Any timeline to merge the pull request?  It'd great to be able to support newer version of AGP.

Hi @vlad-eremia, looks like nobody has responded to your pull request, do you have the changes on other repo?  Thanks.

Just to say, this PR has failing tests (4/5 fail). I've resolved 2 and will continue to look at the other 2 as I get some time over the next day or so.


[BUG]:Helm install fails with Error: template: csi-unity/templates/node.yaml:142:85: executing "csi-unity/templates/node.yaml" at <.Values.images.driver>:nil pointer evaluation interface{}.driver

Installing with helm intall produces the following error:
Error: template: csi-unity/templates/node.yaml:142:85: executing "csi-unity/templates/node.yaml" at <.Values.images.driver>:nil pointer evaluation interface{}.driver
To Reproduce
Steps to reproduce the behavior:

get all dependecies and upload the to local repository
update values file, driver-image.yaml, k8s files with local repository's pull command link
helm package and upload
helm install

Expected behavior
image should be pulled as is happening with all other apps deployed from the repository
Screenshots
cannot provide
Logs
cannot provide
System Information (please complete the following information):

Windows 10
Helm v3.6.3
Kubernetes Version 1.2
Tanzu Kubernetes Cluster v1.20.7

Additional context
The Value of the image location is definetly valid, this is ot the first deployment on this cluster from this repository.
docker pull with the same link works locally as well.

@guy7711 can you share manifest file that populated / provided an input for the driver installation ?  i see you are mentioning Windows 10 as platform just an FYI we don't support windows OS yet, Please do refer support details here https://dell.github.io/csm-docs/docs/csidriver/

@guy7711 can you also share which version of CSI Driver that you are trying to install ?

im installing v2.0.0, now trying with v2.1.0 - though im unable to run docker pull for this version, it only works with "latest" tag
docker pull dellemc/csi-unity:v2.1.0
Error response from daemon: manifest for dellemc/csi-unity:v2.1.0 not found: manifest unknown: manifest unknown
about winOS, i ment that my personal workstation from which i'm running the commands is windows 10 with git bash mintty based.

@guy7711 can you share manifest file that populated / provided an input for the driver installation ? i see you are mentioning Windows 10 as platform just an FYI we don't support windows OS yet, Please do refer support details here https://dell.github.io/csm-docs/docs/csidriver/

unfortunately i cannot share any files, i'm insatlling it in an airgapped site.

im installing v2.0.0, now trying with v2.1.0 - though im unable to run docker pull for this version, it only works with "latest" tag
docker pull dellemc/csi-unity:v2.1.0 Error response from daemon: manifest for dellemc/csi-unity:v2.1.0 not found: manifest unknown: manifest unknown
about winOS, i ment that my personal workstation from which i'm running the commands is windows 10 with git bash mintty based.

@guy7711 : v2.0.0 works with latest tag and v2.1.0 works with nightly tag.

@guy7711 our suggestion to use Released build tag which v2.0.0 and v2.1.0 still under development.

@guy7711 this tag is not released yet docker pull dellemc/csi-unity:v2.1.0.  hence closing this bug.

@guy7711 this tag is not released yet docker pull dellemc/csi-unity:v2.1.0. hence closing this bug.

this is not relevant to the BUG, do not close it, i just mentioned that i'm going to try v2.1.0 as well, if it's not released yet, then im gonna stick with v2.0.0 and the BUG is still there.
please refer to the ERROR mentioned.
Error: template: csi-unity/templates/node.yaml:142:85: executing "csi-unity/templates/node.yaml" at <.Values.images.driver>:nil pointer evaluation interface{}.driver

@guy7711 - Do ensure that you are pointed to the right code in github(pull from v2.0.0 tag instead of main branch)
git clone -b v2.0.0 https://github.com/dell/csi-unity.git
Refer this section https://dell.github.io/csm-docs/docs/csidriver/installation/helm/unity/#install-csi-driver

@guy7711 sorry , we will help you on this bug, please share your observation

@guy7711 : Yes this is a latest issue, This issue has been already reported https://github.com/dell/csm/issues/99
and fixed as part of https://github.com/dell/csi-unity/pull/32, Please pull the latest code from main and let us know if you are still facing the issue.

@guy7711 i guess this due to change in version required in https://github.com/dell/csi-unity/blob/main/helm/csi-unity/driver-image.yaml, please change image as v2.0.0 then you should be able install the driver.

i will try again as suggested above and update soon, thanks for the answers

Hi,
The issue was not resolved, i ran git clone -b v2.0.0 https://github.com/dell/csi-unity.git and cleared the files,
editted all values.yaml files, including the drivers files and added the repository URL path.
still, same issue

i found the problem, there is an issue with the helm chart. when i edit the node-template.yaml file, adding the images url manually, it doesnt complain and move forward with the same error for the next image - same error different line (195:91)
apparently there is a problem with the helm chart file links

@guy7711 : Have you tried git clone https://github.com/dell/csi-unity.git the latest branch from main. Could you please try and update on the same.

Fixed the issue by doing the following:
Copied all images: pointers from the relevant k8s-1.20-values.yaml and added the image from driver-image.yaml as well to:
values.yaml.
helm install worked.

@guy7711 glad to hear that your setup is working now. hope are ok to close this issue ?

Thanks for connecting with us, happy to help as always closing the defect now.  if you need any further assistance please do connect us in SLACK https://app.smartsheet.com/b/form/e99b4d2da13e42518df4d3307c010f47


Removing Parser binding

The OpenTelemetry\SDK\Common\Dsn\Parser was removed in the release: 1.0.0beta16 https://github.com/open-telemetry/opentelemetry-php/pull/1099/files

Feel free to make a release after merge


Review accessibility

See https://www.a11yproject.com/checklist/

I'd be happy to help with this. I've written for the A11y Project a while back here -> https://www.a11yproject.com/authors/#lucia-cerchie
I think manual checks are indispensable, but the right type of automated ones can also help catch issues. I can use Lighthouse as a starting point, see my livestream with Ben Myers to learn more about: https://www.youtube.com/watch?v=4rUcwB7MuqU&t=10s

Accessibility reviewed and changes made to aria attributes, alt text, and other items. The following issues remain unfixed at time of writing as adjustments would be complex or could potentially negatively impact user experience of the story:

The videos in the video conferencing section autoplay when the user scrolls to that section
No controls to pause the video are present on the video (controls take up substantial real estate in the video and video serves role similar to a looping gif)
alt-text for photos and videos don't change when sliders changed as 1000s of variations


Updated setup_logging to use UTF-8 in FileHandler

Updated FileHandler in utils.logging setup_logging to use UTF-8. Fixes #777

Thanks for the fix @dameikle!


Wire up v1alpha2 API to API server.

This allows the API server to serve the v1alpha2 API alongside
v1alpha1.
Requirement for watcher to send requests to the v1alpha2 API server (#23).

/lgtm

/approve


Remove the function registration from Velox validation.

Reverted https://github.com/oap-project/velox/pull/42.


Add a pprof command with a trace subcommand

The pprof command will expand to include other stuff from
go's pprof package.

Yeap, it's hidden, since the pprof command has no Usage.


segmentation violation

Hi, I have modified the gcd_nontd_test.tcl test script to point to the libraries I am using and a DEF design output from QFlow. I am getting the following on rep init_replace line:
** WARNING: Your Detail Placement Step must be skipped.
              (i.e. this program will be executed as -onlyGP)
     If you want to have DP after GP, please specify -dpflag and -dploc.

[INFO] TargetDensity = 1.000000
[INFO] ExperimentIndex = 27
[INFO] DirectoryPath = /home/[user]/Desktop/usb/scratch/etc/live/experiment027
child killed: segmentation violation

Is it possible to use the flow at mature nodes? like 0.6u, 0.35u or 0.18u? I am attempting with X Fab.

To solve the error, could you give your input files and Tcl scripts for me?

Do you have access to XFab's kits on your end? They're confidential, and I am under NDA.

I am using XC06's digital cells library with 3-metal option (not thick).

Sorry, I dont' have.
Could you give me a log when you type "$ valgrind ./replace < your_script.tcl"?

Yes, sure, I was looking for a method to log the script. Thank you. I'll post back INSHAALLAH
Does RePlAce (and other tools) produce a log file when working? It'd help...

The script:
#
# Examples for Non Timing-driven RePlAce with TCL usage
#

set design live
set lib_dir /home/[user]/pdk
set design_dir /home/[user]/Desktop/usb

set lef_path ${design_dir}/scratch/pdk.lef
set def_path ${design_dir}/layout/${design}.def

replace_external rep

# Import LEF/DEF files
rep import_lef $lef_path
rep import_def $def_path
rep set_output $design_dir/scratch

puts $def_path

rep set_verbose_level 0

# Initialize RePlAce
rep init_replace

# place_cell with BiCGSTAB
#rep place_cell_init_place

# print out instances' x/y coordinates
#rep print_instances

# place_cell with Nesterov method
#rep place_cell_nesterov_place

# print out instances' x/y coordinates
#rep print_instances

# Export DEF file
#rep export_def ${design_dir}/scratch/{$design}_nontd.def
puts "Final HPWL: [rep get_hpwl]"

Previously I was typing replace in console, then replace < [script_name].tcl in the replace tcl console, which gave me previous output.
This time I typed valgrind replace < [script_name].tcl

$ valgrind replace < gcdn.tcl > log.txt
==16634== Memcheck, a memory error detector
==16634== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==16634== Using Valgrind-3.14.0 and LibVEX; rerun with -h for copyright info
==16634== Command: replace
==16634==
==16634== Invalid read of size 1
==16634==    at 0x5D2B4A: GetTokenFromStack (lef_keywords.cpp:345)
==16634==    by 0x5D2B4A: LefDefParser::GetToken(char**, int*) [clone .cold.152] (lef_keywords.cpp:405)
==16634==    by 0x95F0A5: LefDefParser::lefyylex() (lef_keywords.cpp:657)
==16634==    by 0x973D01: LefDefParser::lefyyparse() (lef.y:579)
==16634==    by 0x6396E8: Replace::Circuit::ParseLef(std::vector<std::__cxx11::basic_string<char, std::char_traits, std::allocator >, std::allocator<std::__cxx11::basic_string<char, std::char_traits, std::allocator > > >&, bool) (lefParser.cpp:3218)
==16634==    by 0x62AA81: Init (lefdefIO.h:134)
==16634==    by 0x62AA81: ParseLefDef() (lefdefIO.cpp:403)
==16634==    by 0x62ACC6: ParseInput() (lefdefIO.cpp:241)
==16634==    by 0x6C4604: replace_external::init_replace() (replace_external.cpp:325)
==16634==    by 0x6CB065: _wrap_replace_external_init_replace (replace_wrap.cpp:3240)
==16634==    by 0x6C8E1E: SWIG_Tcl_MethodCommand (replace_wrap.cpp:1329)
==16634==    by 0x5C4AEB1: ??? (in /usr/lib64/libtcl8.5.so)
==16634==    by 0x5C8F36B: ??? (in /usr/lib64/libtcl8.5.so)
==16634==    by 0x5C97646: ??? (in /usr/lib64/libtcl8.5.so)
==16634==  Address 0x0 is not stack'd, malloc'd or (recently) free'd
==16634==
==16634==
==16634== Process terminating with default action of signal 11 (SIGSEGV)
==16634==  Access not within mapped region at address 0x0
==16634==    at 0x5D2B4A: GetTokenFromStack (lef_keywords.cpp:345)
==16634==    by 0x5D2B4A: LefDefParser::GetToken(char**, int*) [clone .cold.152] (lef_keywords.cpp:405)
==16634==    by 0x95F0A5: LefDefParser::lefyylex() (lef_keywords.cpp:657)
==16634==    by 0x973D01: LefDefParser::lefyyparse() (lef.y:579)
==16634==    by 0x6396E8: Replace::Circuit::ParseLef(std::vector<std::__cxx11::basic_string<char, std::char_traits, std::allocator >, std::allocator<std::__cxx11::basic_string<char, std::char_traits, std::allocator > > >&, bool) (lefParser.cpp:3218)
==16634==    by 0x62AA81: Init (lefdefIO.h:134)
==16634==    by 0x62AA81: ParseLefDef() (lefdefIO.cpp:403)
==16634==    by 0x62ACC6: ParseInput() (lefdefIO.cpp:241)
==16634==    by 0x6C4604: replace_external::init_replace() (replace_external.cpp:325)
==16634==    by 0x6CB065: _wrap_replace_external_init_replace (replace_wrap.cpp:3240)
==16634==    by 0x6C8E1E: SWIG_Tcl_MethodCommand (replace_wrap.cpp:1329)
==16634==    by 0x5C4AEB1: ??? (in /usr/lib64/libtcl8.5.so)
==16634==    by 0x5C8F36B: ??? (in /usr/lib64/libtcl8.5.so)
==16634==    by 0x5C97646: ??? (in /usr/lib64/libtcl8.5.so)
==16634==  If you believe this happened as a result of a stack
==16634==  overflow in your program's main thread (unlikely but
==16634==  possible), you can try to increase the size of the
==16634==  main thread stack using the --main-stacksize= flag.
==16634==  The main thread stack size used in this run was 8388608.
==16634==
==16634== HEAP SUMMARY:
==16634==     in use at exit: 999,096 bytes in 896 blocks
==16634==   total heap usage: 1,084 allocs, 188 frees, 1,458,632 bytes allocated
==16634==
==16634== LEAK SUMMARY:
==16634==    definitely lost: 0 bytes in 0 blocks
==16634==    indirectly lost: 0 bytes in 0 blocks
==16634==      possibly lost: 788,777 bytes in 45 blocks
==16634==    still reachable: 210,319 bytes in 851 blocks
==16634==         suppressed: 0 bytes in 0 blocks
==16634== Rerun with --leak-check=full to see details of leaked memory
==16634==
==16634== For counts of detected and suppressed errors, rerun with: -v
==16634== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)
Segmentation fault (core dumped)
[[user]@localhost scratch]$ mv log.txt /mnt/hgfs/cad/
[[user]@localhost scratch]$ valgrind replace < gcdn.tcl > log.txt
==16894== Memcheck, a memory error detector
==16894== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==16894== Using Valgrind-3.14.0 and LibVEX; rerun with -h for copyright info
==16894== Command: replace
==16894==
==16894== Invalid read of size 1
==16894==    at 0x5D2B4A: GetTokenFromStack (lef_keywords.cpp:345)
==16894==    by 0x5D2B4A: LefDefParser::GetToken(char**, int*) [clone .cold.152] (lef_keywords.cpp:405)
==16894==    by 0x95F0A5: LefDefParser::lefyylex() (lef_keywords.cpp:657)
==16894==    by 0x973D01: LefDefParser::lefyyparse() (lef.y:579)
==16894==    by 0x6396E8: Replace::Circuit::ParseLef(std::vector<std::__cxx11::basic_string<char, std::char_traits, std::allocator >, std::allocator<std::__cxx11::basic_string<char, std::char_traits, std::allocator > > >&, bool) (lefParser.cpp:3218)
==16894==    by 0x62AA81: Init (lefdefIO.h:134)
==16894==    by 0x62AA81: ParseLefDef() (lefdefIO.cpp:403)
==16894==    by 0x62ACC6: ParseInput() (lefdefIO.cpp:241)
==16894==    by 0x6C4604: replace_external::init_replace() (replace_external.cpp:325)
==16894==    by 0x6CB065: _wrap_replace_external_init_replace (replace_wrap.cpp:3240)
==16894==    by 0x6C8E1E: SWIG_Tcl_MethodCommand (replace_wrap.cpp:1329)
==16894==    by 0x5C4AEB1: ??? (in /usr/lib64/libtcl8.5.so)
==16894==    by 0x5C8F36B: ??? (in /usr/lib64/libtcl8.5.so)
==16894==    by 0x5C97646: ??? (in /usr/lib64/libtcl8.5.so)
==16894==  Address 0x0 is not stack'd, malloc'd or (recently) free'd
==16894==
==16894==
==16894== Process terminating with default action of signal 11 (SIGSEGV)
==16894==  Access not within mapped region at address 0x0
==16894==    at 0x5D2B4A: GetTokenFromStack (lef_keywords.cpp:345)
==16894==    by 0x5D2B4A: LefDefParser::GetToken(char**, int*) [clone .cold.152] (lef_keywords.cpp:405)
==16894==    by 0x95F0A5: LefDefParser::lefyylex() (lef_keywords.cpp:657)
==16894==    by 0x973D01: LefDefParser::lefyyparse() (lef.y:579)
==16894==    by 0x6396E8: Replace::Circuit::ParseLef(std::vector<std::__cxx11::basic_string<char, std::char_traits, std::allocator >, std::allocator<std::__cxx11::basic_string<char, std::char_traits, std::allocator > > >&, bool) (lefParser.cpp:3218)
==16894==    by 0x62AA81: Init (lefdefIO.h:134)
==16894==    by 0x62AA81: ParseLefDef() (lefdefIO.cpp:403)
==16894==    by 0x62ACC6: ParseInput() (lefdefIO.cpp:241)
==16894==    by 0x6C4604: replace_external::init_replace() (replace_external.cpp:325)
==16894==    by 0x6CB065: _wrap_replace_external_init_replace (replace_wrap.cpp:3240)
==16894==    by 0x6C8E1E: SWIG_Tcl_MethodCommand (replace_wrap.cpp:1329)
==16894==    by 0x5C4AEB1: ??? (in /usr/lib64/libtcl8.5.so)
==16894==    by 0x5C8F36B: ??? (in /usr/lib64/libtcl8.5.so)
==16894==    by 0x5C97646: ??? (in /usr/lib64/libtcl8.5.so)
==16894==  If you believe this happened as a result of a stack
==16894==  overflow in your program's main thread (unlikely but
==16894==  possible), you can try to increase the size of the
==16894==  main thread stack using the --main-stacksize= flag.
==16894==  The main thread stack size used in this run was 8388608.
==16894==
==16894== HEAP SUMMARY:
==16894==     in use at exit: 999,096 bytes in 896 blocks
==16894==   total heap usage: 1,084 allocs, 188 frees, 1,458,632 bytes allocated
==16894==
==16894== LEAK SUMMARY:
==16894==    definitely lost: 0 bytes in 0 blocks
==16894==    indirectly lost: 0 bytes in 0 blocks
==16894==      possibly lost: 788,777 bytes in 45 blocks
==16894==    still reachable: 210,319 bytes in 851 blocks
==16894==         suppressed: 0 bytes in 0 blocks
==16894== Rerun with --leak-check=full to see details of leaked memory
==16894==
==16894== For counts of detected and suppressed errors, rerun with: -v
==16894== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)
Segmentation fault (core dumped)

I got the exact same error when I use CentOS 8/gcc8.

Could you use Docker environment, instead?

Yes, I do have gcc 8.3.0. Machine is CentOS 7.
What is your development platform? I am now compiling using gcc 4.8.5

If you can compile again with gcc-4.8.5/CentOS7, then the new binary would not have this problem.
I think this issue is due to the compiler's different memory handling on copy constructor/malloc...
We're going to integrate OpenDB with RePlAce, so this error would be removed in the near future.
(My modified LEF/DEF parsers seem to have OS-dependent problem...)

Here, compiled using 4.8.5:
==21032== Memcheck, a memory error detector
==21032== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==21032== Using Valgrind-3.14.0 and LibVEX; rerun with -h for copyright info
==21032== Command: replace
==21032==
RePlAce Version: 1.0.0
/home/mbk/Desktop/usb/layout/live.def

** WARNING: Your Detail Placement Step must be skipped.
              (i.e. this program will be executed as -onlyGP)
     If you want to have DP after GP, please specify -dpflag and -dploc.

[INFO] TargetDensity = 1.000000
[INFO] ExperimentIndex = 34
[INFO] DirectoryPath = /home/mbk/Desktop/usb/scratch/etc/live/experiment034
[INFO] DefUnit = 100
[INFO] LefMetal1Name = MET1
==21032== Invalid read of size 8
==21032==    at 0x8BF6E0: LefDefParser::defiRow::macro() const (defiRowTrack.cpp:364)
==21032==    by 0x5DABC1: SetParameter() (lefdefIO.cpp:352)
==21032==    by 0x5E036C: ParseLefDef() (lefdefIO.cpp:405)
==21032==    by 0x5E065D: ParseInput() (lefdefIO.cpp:241)
==21032==    by 0x673483: replace_external::init_replace() (replace_external.cpp:325)
==21032==    by 0x678B34: _wrap_replace_external_init_replace (replace_wrap.cpp:3240)
==21032==    by 0x67C8A4: SWIG_Tcl_MethodCommand (replace_wrap.cpp:1329)
==21032==    by 0x5C4AEB1: ??? (in /usr/lib64/libtcl8.5.so)
==21032==    by 0x5C8F36B: ??? (in /usr/lib64/libtcl8.5.so)
==21032==    by 0x5C97646: ??? (in /usr/lib64/libtcl8.5.so)
==21032==    by 0x5C4C6B6: TclEvalObjEx (in /usr/lib64/libtcl8.5.so)
==21032==    by 0x5C9D38B: Tcl_RecordAndEvalObj (in /usr/lib64/libtcl8.5.so)
==21032==  Address 0x18 is not stack'd, malloc'd or (recently) free'd
==21032==
==21032==
==21032== Process terminating with default action of signal 11 (SIGSEGV)
==21032==  Access not within mapped region at address 0x18
==21032==    at 0x8BF6E0: LefDefParser::defiRow::macro() const (defiRowTrack.cpp:364)
==21032==    by 0x5DABC1: SetParameter() (lefdefIO.cpp:352)
==21032==    by 0x5E036C: ParseLefDef() (lefdefIO.cpp:405)
==21032==    by 0x5E065D: ParseInput() (lefdefIO.cpp:241)
==21032==    by 0x673483: replace_external::init_replace() (replace_external.cpp:325)
==21032==    by 0x678B34: _wrap_replace_external_init_replace (replace_wrap.cpp:3240)
==21032==    by 0x67C8A4: SWIG_Tcl_MethodCommand (replace_wrap.cpp:1329)
==21032==    by 0x5C4AEB1: ??? (in /usr/lib64/libtcl8.5.so)
==21032==    by 0x5C8F36B: ??? (in /usr/lib64/libtcl8.5.so)
==21032==    by 0x5C97646: ??? (in /usr/lib64/libtcl8.5.so)
==21032==    by 0x5C4C6B6: TclEvalObjEx (in /usr/lib64/libtcl8.5.so)
==21032==    by 0x5C9D38B: Tcl_RecordAndEvalObj (in /usr/lib64/libtcl8.5.so)
==21032==  If you believe this happened as a result of a stack
==21032==  overflow in your program's main thread (unlikely but
==21032==  possible), you can try to increase the size of the
==21032==  main thread stack using the --main-stacksize= flag.
==21032==  The main thread stack size used in this run was 8388608.
==21032==
==21032== HEAP SUMMARY:
==21032==     in use at exit: 36,814,447 bytes in 1,179,547 blocks
==21032==   total heap usage: 2,760,034 allocs, 1,580,487 frees, 4,678,754,460 bytes allocated
==21032==
==21032== LEAK SUMMARY:
==21032==    definitely lost: 11,829,995 bytes in 569,819 blocks
==21032==    indirectly lost: 2,904 bytes in 352 blocks
==21032==      possibly lost: 756,049 bytes in 44 blocks
==21032==    still reachable: 24,225,499 bytes in 609,332 blocks
==21032==                       of which reachable via heuristic:
==21032==                         stdstring          : 1,366,181 bytes in 49,366 blocks
==21032==         suppressed: 0 bytes in 0 blocks
==21032== Rerun with --leak-check=full to see details of leaked memory
==21032==
==21032== For counts of detected and suppressed errors, rerun with: -v
==21032== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)
Segmentation fault (core dumped)

It means your DEF has a problem...
Could you check that your DEF contains the ROW definition?

No, it does not contain ROW definitions.

The ROW must be defined to run Global Placer ...

Ok, maybe I should just use the openflow as complete flow instead of QFlow.

I tried a make test on OpenROAD-Utilities/verilog-to-def and it resulted in a segfault. I re did make using gcc4.8.5 and it doesn't segfault.

Where can I ask questions regarding the tool usage?
I see capacitance and resistance units for RePlAce. What should I set as the units values? Isn't resistance and capacitance per micron, dependent on each metal layer? And has a different value for each layer...
Secondly, whats dploc flag? Google returned a result explaining the flags: RePlAce/doc/BinaryArguments.md, but apparently the file is no longer available.

I tried a make test on OpenROAD-Utilities/verilog-to-def and it resulted in a segfault. I re did make using gcc4.8.5 and it doesn't segfault.

OpenROAD-Utilities/verilog-to-def  is obsolete. Could you use https://github.com/The-OpenROAD-Project/Resizer instead?

Could you please point me to a document which states the flow you follow?
I am trying to follow the demo script on yosys.

Where can I ask questions regarding the tool usage?
I see capacitance and resistance units for RePlAce. What should I set as the units values? Isn't resistance and capacitance per micron, dependent on each metal layer? And has a different value for each layer...

This is a known problem for us. For now, you may use M1 values. We're going to use a Machine-learning model or Global-Router tree model in the future.

Secondly, whats dploc flag? Google returned a result explaining the flags: RePlAce/doc/BinaryArguments.md, but apparently the file is no longer available.

-dpflag is no longer supported. you can use OpenDP for a detailed placer in our flow.
https://github.com/The-OpenROAD-Project/OpenDP

-dpflag is no longer supported.
I see reference to this flag on https://github.com/The-OpenROAD-Project/yosys

Could you please point me to a document which states the flow you follow?
I am trying to follow the demo script on yosys.

Could you check the alpha-release repositories?

Dockerfile for building all of OpenROAD tools.
https://github.com/The-OpenROAD-Project/alpha-release/tree/master/build/docker

Makefile for running all of OpenROAD tools.
https://github.com/The-OpenROAD-Project/alpha-release/blob/master/flow/Makefile

-dpflag is no longer supported.

I see reference to this flag on https://github.com/The-OpenROAD-Project/yosys

You can ignore that flag now. It is only used when you're going to use physical synthesis.
The Yosys should be updated not to have -dpflag.

Hi,
I am not using docker, but I succeeded in compiling all tools ALHAMDOLILLAH.
I have also downloaded the already built release files in the alpha-release: https://github.com/The-OpenROAD-Project/alpha-release/releases/tag/v2019-09-05_20-10
However, I've noticed the files such as verilog2def and POW9 etc are not included in the release zip file, even though it contains the built binaries for the tools...

I do have verilog2def and POW9 etc files when I built the tools myself. Just wondering why they're not included in the release.

6_final.tar.gz
ALHAMDOLILLAH i managed to get the flow running. I didn't use docker, but instead ran setup.sh in released binaries, and then ran the flow from the source i downloaded from github, /alpha-release/flow
I did make in the flow directory, however, it wouldn't go to the Final/ Finish stage. I edited the Makefile statement: all: route to all: finish (I wonder, why it wasn't already...)
The final gds is attached here. Please have a look. I don't remembering altering any 'area' parameter etc in the configuration files for the gcd design, but the gds seems odd. It gave me 8% utilization. Also, the power buses seem missing and the cell seem to overlap in two consecutive rows.
Am I doing something wrong...?

Sorry for my late reply.
You can change the DIEAREA in the following Makefile settings:
https://github.com/The-OpenROAD-Project/alpha-release/blob/7c0421f6a4cb7f9c5df2fee126b2d40ed72d3d95/flow/designs/gcd_nangate45.mk#L14-L17
I will close this issue. (Your issue is out of my scope)


API healthcheck to retry if services are not OK

Resolves #3585
What is being addressed
This modified script checks if the api server endpoint /api/health is available(response code 200) and then verifies that the response contains all OK. Both verifications have a retry (4 retries, including first call makes total 5 calls)

/test-extended

/test-extended

/test-extended

/test-destroy-env

/test-extended

/test-extended


there is no "Script: Configure Script"

I want change the default Python to Python3,  I don't know how set it !  thank you very much!

Top menu bar - packages - Script

yeah ，gotta it ， thank you very much！


Signed integers

Type of change

Bug fix
New feature
Improvement (refactoring, restructuring repository, cleaning tech debt, ...)
Documentation
Other (describe below)

Changes
The following changes have been made:

Change 1
Change 2

Notes

Note 1

Related Issues

Closes #<issue number>

Just for posterity, could you fill out the PR description with the change?

@rostyslavtyshko  merge conflict. Same for #31

Most of the comments I've made for i128 apply for the others as well.
Are you able to add some documentation like a README.md and a SPECIFICATION.md so other developers know how to use this library? You can base the outline on the recently merged String Library.
I also don't see any tests being run. The library should be added to the harness in order for CI to run your tests. I would also suggest splitting up your tests to make sure you are testing individual functions rather than a single main() function. You can check out how the String Library is tested here.

CI will be a separate issue: https://github.com/FuelLabs/sway-libs/issues/44
The tests are running fine, so I propose to merge this PR as is.

Most of the comments I've made for i128 apply for the others as well.
Are you able to add some documentation like a README.md and a SPECIFICATION.md so other developers know how to use this library? You can base the outline on the recently merged String Library.
I also don't see any tests being run. The library should be added to the harness in order for CI to run your tests. I would also suggest splitting up your tests to make sure you are testing individual functions rather than a single main() function. You can check out how the String Library is tested here.

CI will be a separate issue: #44 The tests are running fine, so I propose to merge this PR as is.

It is generally bad practice to merge code that has not been run through CI. Running tests manually, I found that i16 had compile errors and both i128 and i256 seem to have some problems with bytecode size. I am unable to compile i256 due to its size and i128's test compiles to a size of 97,252 while all other tests compile to roughly 3,500 bytes in size.
I've gone ahead and added CI in https://github.com/FuelLabs/sway-libs/pull/45. I believe #44 should be used to modify the current CI to recursively check folders for a Forc.toml and compile those folders rather than the current setup of all in a single folder. This will help with organization.

After waiting 10 minutes I was able to compile i256 with a size of 911,092 bytes

So... is this ready to go ?


torch.cuda.is_available() false when running multi-gpu inference with accelerate launch

System Info
- `Accelerate` version: 1.0.1
- Platform: Linux-4.15.0-213-generic-x86_64-with-glibc2.27
- `accelerate` bash location: /[user_dir]/miniconda3/envs/gotenv/bin/accelerate
- Python version: 3.10.9
- Numpy version: 1.26.4
- PyTorch version (GPU?): 2.0.0 (True)
- PyTorch XPU available: False
- PyTorch NPU available: False
- PyTorch MLU available: False
- PyTorch MUSA available: False
- System RAM: 125.80 GB
- GPU type: NVIDIA GeForce GTX 1080 Ti
- `Accelerate` default config:
        - compute_environment: LOCAL_MACHINE
        - distributed_type: MULTI_GPU
        - mixed_precision: fp16
        - use_cpu: False
        - debug: True
        - num_processes: 4
        - machine_rank: 0
        - num_machines: 1
        - gpu_ids: '[1,2,3,4]'
        - rdzv_backend: static
        - same_network: True
        - main_training_function: main
        - enable_cpu_affinity: True
        - downcast_bf16: no
        - tpu_use_cluster: False
        - tpu_use_sudo: False
        - tpu_env: []

Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[X] My own task or dataset (give details below)

Reproduction
I am trying to do multi-gpu inference on 4 (of 7 available) NVIDIA GeForce GTX 1080 Ti. My script version without accelerate works fine when running it on a single GPU (GPU '0') with CUDA_VISIBLE_DEVICES=0 script.py.
Whenever I try to run it with accelerate launch --config_file [config_file] script.py and the following config
compute_environment: LOCAL_MACHINE
debug: true
distributed_type: MULTI_GPU
downcast_bf16: 'no'
enable_cpu_affinity: false
gpu_ids: '[1,2,3,4]'
machine_rank: 0
main_training_function: main
mixed_precision: fp16
num_machines: 1
num_processes: 4
rdzv_backend: static
same_network: true
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false

or any other (also default) for that matter torch cannot find cuda, then defaults to cpu and I get a torch.cpu has to attribute device_count error.
My code is the following, but cuda can already not be found when importing torch at the very top.
def eval_model_accelerate(inputs):
    distr_state = PartialState()
    tokenizer = AutoTokenizer.from_pretrained(
        inputs["model_name"], trust_remote_code=True
    )
    model = GOTQwenForCausalLM.from_pretrained(
        inputs["model_name"],
        device_map="cuda",
        use_safetensors=True,
        pad_token_id=151643,
    ).eval()
    model.to(distr_state.device, dtype=torch.float16)
    with distr_state.split_between_processes(inputs["files_dir"]) as ins:
        for elem in ins:
            image_list, input_ids, conv, stopping_criteria = preprocess_input(
                tokenizer, elem, -1
            )
            outfile_name = elem.split("/")[-1]
            with torch.autocast("cuda", torch.float16):
                output_ids = model.generate(
                    input_ids,
                    images=[
                        (
                            image_list.half().to(distr_state.device),
                            image_list.half().to(distr_state.device),
                        )
                    ],
                    do_sample=False,
                    num_beams=1,
                    # no_repeat_ngram_size = 20,
                    streamer=None,  # removed TextStreamer for no output
                    max_new_tokens=4096,
                    stopping_criteria=[stopping_criteria],
                )
            postprocess_output(tokenizer, output_ids, input_ids, outfile_name, -1, conv)

The same problems occur when I omit the config file, and run accelerate launch [...] by passing all the arguments directly.
accelerate launch script.py --multi_gpu --num_processes 4 --debug. In the very same conda environment the script runs without issues, torch finds cuda and so on, if I don't use accelerate.
Expected behavior
torch.distributed should launch my inference script on 4 GPUs.

Change GPU ID's to be 1,2,3,4 is something to try

Turns out this was a corrupt conda config on my end. All resolved now


trigger github action release workflow on push to main

Signed-off-by: Robbie Lankford<EMAIL_ADDRESS>

I would be open to trying it, although I'm not sure that's all it takes :-)


httpStatus of error in getUrl callback not available in Firefox

I've noticed that the callback of getUrl doesn't have httpStatus set on the error object in Firefox.
I'm particularly interested into a request that might fail with the status code 401. While I can check for this in browsers like Chrome and Safari, this doesn't work in Firefox.
Do you know what the root of this problem is?

I'm not sure I follow. getUrl does not actually execute the HTTP request at all. (Of course, if it is a traversal process with multiple steps, it will execute the intermediate requests, only excluding the last one). See https://github.com/basti1302/traverson/blob/master/api.markdown, docs for getUrl. The idea here is that you can execute the request yourself. So you should never get an 401 error from Traverson because the request has not been executed.
Are you talking about the case where an intermediate URL fails with 401?

For what it's worth, I added a test for the case where, during a getUrl traversal, an intermediate URL fails with 404 and this works fine in Firefox for me. Could you check again? If you still think that this is a Traverson issue, please feel free to reopen this bug and provide more detail, or better yet, a complete reproducible test case that fails in Firefox for you. Also, knowing the OS and Firefox version might help.

Thanks for your response! Maybe it was my apps fault then. I've worked around the problem for now, but if I revisit it in the future and still discover the problem I'll provide a reproducible test case.


feat: sign user op and send raw user op methods added to smart account client

Pull Request Checklist

[ ] Did you add new tests and confirm existing tests pass? (yarn test)
[ ] Did you update relevant docs? (docs are found in the site folder, and guidelines for updating/adding docs can be found in the contribution guide)
[ ] Do your commits follow the Conventional Commits standard?
[ ] Does your PR title also follow the Conventional Commits standard?
[ ] If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
[ ] Did you run lint (yarn lint:check) and fix any issues? (yarn lint:write)
[ ] Did you follow the contribution guidelines?

PR-Codex overview
The focus of this PR is to introduce new types and functionalities related to user operation handling.
Detailed summary

Added UserOperationCallData and UpgradeToData types
Updated SendUserOperationResult type
Modified functions related to building and sending user operations
Added functionality for dropping and replacing user operations

The following files were skipped due to too many changes: site/packages/aa-core/smart-account-client/actions/checkGasSponsorshipEligibility.md, packages/core/src/actions/smartAccount/types.ts, site/packages/aa-core/smart-account-client/actions/signUserOperation.md, site/packages/aa-core/smart-account-client/actions/sendUserOperation.md, packages/core/src/client/decorators/smartAccountClient.ts, site/packages/aa-core/smart-account-client/actions/sendTransaction.md, site/packages/aa-core/smart-account-client/actions/sendTransactions.md, site/using-smart-accounts/send-user-operations.md, site/packages/aa-core/smart-account-client/actions/buildUserOperationFromTx.md, site/packages/aa-core/smart-account-client/actions/buildUserOperation.md, site/packages/aa-core/smart-account-client/actions/buildUserOperationFromTxs.md

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@moldy530 PR is ready for review.


QuillJS 1.1.x support

It seems this ngQuill directive is locked to 0.20.1. When I used the latest QuillJS 1.1.x, it does not work, all toolbar disappeared.

please use search through closed issues.
This version only works with 0.20.1.
QuillJS 1.x has a changed api and uses other technologies.

@KillerCodeMonkey  Thanks


minor improvement for best practices section index

Codecov Report

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

@@             Coverage Diff              @@
##             master    #5434      +/-   ##
============================================
- Coverage     82.37%   82.35%   -0.03%
+ Complexity    11287    11285       -2
============================================
  Files          1359     1359
  Lines         44305    44305
  Branches       5559     5559
============================================
- Hits          36497    36488       -9
- Misses         5208     5214       +6
- Partials       2600     2603       +3

Impacted Files
Coverage Δ
Complexity Δ

.../operation/udf/DropUserDefinedFunctionRequest.java
77.14% <0%> (-17.15%)
8% <0%> (-3%)

...in/java/io/crate/data/CollectingBatchConsumer.java
69.23% <0%> (-15.39%)
4% <0%> (-1%)

...rc/main/java/io/crate/analyze/UnboundAnalyzer.java
70% <0%> (-15%)
2% <0%> (ø)

...ration/reference/doc/lucene/IpColumnReference.java
72.72% <0%> (-13.64%)
5% <0%> (ø)

...ration/collect/collectors/OrderedDocCollector.java
90% <0%> (-10%)
5% <0%> (-1%)

...src/main/java/io/crate/jobs/UpsertByIdContext.java
80% <0%> (-8.58%)
6% <0%> (ø)

...io/crate/protocols/postgres/types/VarCharType.java
91.66% <0%> (-8.34%)
10% <0%> (-1%)

.../java/io/crate/blob/TransportDeleteBlobAction.java
75% <0%> (-5%)
4% <0%> (-1%)

.../collect/collectors/LuceneOrderedDocCollector.java
90.12% <0%> (-3.71%)
25% <0%> (-1%)

...o/crate/planner/node/dql/join/NestedLoopPhase.java
77.1% <0%> (-3.62%)
21% <0%> (-2%)

... and 15 more

Continue to review full report at Codecov.

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


[Element: PlayEffect] Effects path resets to 'none'

Attempting to input an effect to a PlayEffect element causes it to reset to none once you reselect the element.

Doesn't happen


BN failing to disassemble a MIPS instruction

IDA represents the following two instructions with a single psuedo instruction:
3C 1A C1 00 67 5A 04 3C                 la      $k0, 0xC100043C
Binary Ninja successfully disassembles the first 32-bits as
lui $k0, 0xc100

But fails to disassemble the instruction represented by the last 32-bits.

This should be fixed since at least https://github.com/Vector35/arch-mips/commit/c8e2d4a813c604d04310f2d5a5e60a687b713c50
I tested by File > New Binary Data, entering 3c 1a c1 00 67 5a 04 3c and creating a function (right click > Make Function at This Address > mips64 > mips64:


feat: implement 0x44 - PREVRANDAO Opcode

SinceGroupMergeBlock InformationIndex 1 is top of the stack. See PUSH.Stack outputprevRandao: previous  block's RANDAO  mix.ExampleInputOutput10xce124dee50136f3f93f19667fb4198c6b94eecbacfa300469e5280012757be94Error casesThe state changes done by the current context are reverted in those cases:Not enough gas.Stack overflow.

Important update: we are gathering some bugs in the Kakarot v0 codebase, we need to make sure each issue and each PR in Kakarot-ssj is aware of the lists of known bugs. Look at this link everytime you take an issue and check your issue isn't targeted by a known bug.

Important update: we are gathering some bugs in the Kakarot v0 codebase, we need to make sure each issue and each PR in Kakarot-ssj is aware of the lists of known bugs. Look at this tracking issue everytime you take an issue and check your issue isn't targeted by a known bug. Will add this reminder in many places to make sure we keep track of known bugs.

https://eips.ethereum.org/EIPS/eip-4399


KEYCLOAK ISSUE - AUTHENTICATION ERROR LOGIN

Background: a supervisor/manager attempts to review/approve an employee's telework agreement via the link generated in the
employee's confirmation email. Once the link is selected, the user logins, following this, an error message appears:   "PSA-DIGITAL JOURNEY" We are sorry.... Unexpected error when authenticating with identity provider.  The supervisor is unable to have their IDIR and password authenticated.
Steps to Reproduce:   Supervisor to follow the employee's confirmation email link to login, review and approve or deny the telework agreement. Anomaly - this error message happens occasionally and it not a persistent problem with most users.
Actual Behavior:  Error message is generated after the supervisor tries to access the confirmation email located under the: After obtaining proof of ADM (or delegate) approval section of the email.
Expected Behaviour: Supervisor is taken their task list to review and approve/deny telework agreements vis the confirmation email link.
Environment: Prod env.

[Uploading... Online Telework Agreement - Access Error697.msg]https://app.zenhub.com/files/439477060/a4315147-3f59-4ae9-83a6-f80ae7f760ec/download

@MissionT it looks like we have duplicate tickets for this issue, #697 and #653

I'll close this. Thanks Chris


Extract JIT tests from test_functional to the dedicated test module

This PR is derived from #466.
It extracts torchscript consistency test from test_functional module and put them in a dedicated module/class (test_torchscript::TestFunctional).

Yay


Documentation suggestion - Add a new type screen should show 'catagorical' vs 'numeric'

Please take a look at the screen shot in the attachment.  Should the  new type screen show 'catagorical' vs 'numeric' ?
Bug032920.docx

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

ID: bd2725bb-44ba-bff1-7dee-f90139eee785
Version Independent ID: 7c5d81f8-8633-0d7e-dedc-7c45ca38ea0d
Content: Tutorial: Set up a Preview environment - Azure Time Series Insights
Content Source: articles/time-series-insights/time-series-insights-update-create-environment.md
Service: time-series-insights
GitHub Login: @deepakpalled
Microsoft Alias: dpalled

@MikeWilcoxMicrosoft Thanks for the feedback! We are currently investigating and will update you shortly.

Adding @lyrana for further review and update the doc as appropriate.

@MikeWilcoxMicrosoft @lyrana  The change has been made and merged.  It should appear within the next few days.

Great thank you @Virginia001 !

Link to PR is here
#please-close

Hello, Mike.
Once a PR is merged, the items it contained are no longer accessible via that PR.  The change will show up in the documentation within a few days.
Regards,
Virginia Dooley
You can have my Oxford comma when you pry
it out of my cold, dead, and lifeless hand. 😊
From: Mike Wilcox<EMAIL_ADDRESS>Sent: Thursday, April 2, 2020 2:02 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Virginia Dooley (Aquent<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Documentation suggestion - Add a new type screen should show 'catagorical' vs 'numeric' (#51208)
Re: Link to PR is here https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs-pr%2Fpull%2F110036&data=02|01|Mike.Wilcox%40microsoft.com|c10f8dd531814f3fa08808d7d7398f45|72f988bf86f141af91ab2d7cd011db47|1|0|637214514531553317&sdata=11SG03EggoPtsvJGK3aMHAgnd5lWwEDGp1Tj1Zeh1P4%3D&reserved=0 Hmm 404 error ?
<EMAIL_ADDRESS>Mike Wilcox
Industry Technology Strategist
Cloud Solution Architect
Mobile: +1<PHONE_NUMBER>
Mike.Wilcox@microsoft.com<mailto:Mike.Wilcox@microsoft.commailto:Mike.Wilcox<EMAIL_ADDRESS>[MSFT_logo_Gray DE sized SIG1.png]
[linked.png]http://www.linkedin.com/in/mwilcox1<EMAIL_ADDRESS>http://ieee.org/
From: lyrana <notifications<EMAIL_ADDRESS>Sent: Thursday, April 2, 2020 2:11 PM
To: MicrosoftDocs/azure-docs <azure-docs<EMAIL_ADDRESS>Cc: Michael Wilcox <Mike.Wilcox<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Documentation suggestion - Add a new type screen should show 'catagorical' vs 'numeric' (#51208)
Link to PR is here https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs-pr%2Fpull%2F110036&data=02|01|Mike.Wilcox%40microsoft.com|c10f8dd531814f3fa08808d7d7398f45|72f988bf86f141af91ab2d7cd011db47|1|0|637214514531553317&sdata=11SG03EggoPtsvJGK3aMHAgnd5lWwEDGp1Tj1Zeh1P4%3D&reserved=0
#please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F51208%23issuecomment-608050584&data=02|01|Mike.Wilcox%40microsoft.com|c10f8dd531814f3fa08808d7d7398f45|72f988bf86f141af91ab2d7cd011db47|1|0|637214514531553317&sdata=eGanZ9Ii71MCCJ9JFFRkkCwxGeFn4BU909CYUAbpFYs%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANWCYMQWZ4ATXUWOQI4U72LRKTPLVANCNFSM4LWGVMWA&data=02|01|Mike.Wilcox%40microsoft.com|c10f8dd531814f3fa08808d7d7398f45|72f988bf86f141af91ab2d7cd011db47|1|0|637214514531563309&sdata=D8lDe1TPrMSw5jhwITZbvKLe%2Bu0GMx%2FJcVAmgPMpYkU%3D&reserved=0.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F51208%23issuecomment-608082427&data=02|01|v-vidool%40microsoft.com|6b6870e827844a6175e208d7d7492531|72f988bf86f141af91ab2d7cd011db47|1|0|637214581461881107&sdata=qJ51evJZK1ydrSMVjGTSTG1C71oL7IkOZhRlp26ruoI%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMLGSHCCH2EPMM3BLKDVUE3RKT4N5ANCNFSM4LWGVMWA&data=02|01|v-vidool%40microsoft.com|6b6870e827844a6175e208d7d7492531|72f988bf86f141af91ab2d7cd011db47|1|0|637214581461891102&sdata=udsJrR95CH62mXI1XoHxG3sGzPjtN0wrkfvTTHqOIRE%3D&reserved=0.

Hi, Mike.
Actually, it’s out there now.
https://docs.microsoft.com/en-us/azure/time-series-insights/time-series-insights-update-create-environment#define-and-apply-a-model
Regards,
Virginia
From: Mike Wilcox<EMAIL_ADDRESS>Sent: Thursday, April 2, 2020 2:02 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Virginia Dooley (Aquent<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Documentation suggestion - Add a new type screen should show 'catagorical' vs 'numeric' (#51208)
Re: Link to PR is here https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs-pr%2Fpull%2F110036&data=02|01|Mike.Wilcox%40microsoft.com|c10f8dd531814f3fa08808d7d7398f45|72f988bf86f141af91ab2d7cd011db47|1|0|637214514531553317&sdata=11SG03EggoPtsvJGK3aMHAgnd5lWwEDGp1Tj1Zeh1P4%3D&reserved=0 Hmm 404 error ?
<EMAIL_ADDRESS>Mike Wilcox
Industry Technology Strategist
Cloud Solution Architect
Mobile: +1<PHONE_NUMBER>
Mike.Wilcox@microsoft.com<mailto:Mike.Wilcox@microsoft.commailto:Mike.Wilcox<EMAIL_ADDRESS>[MSFT_logo_Gray DE sized SIG1.png]
[linked.png]http://www.linkedin.com/in/mwilcox1<EMAIL_ADDRESS>http://ieee.org/
From: lyrana <notifications<EMAIL_ADDRESS>Sent: Thursday, April 2, 2020 2:11 PM
To: MicrosoftDocs/azure-docs <azure-docs<EMAIL_ADDRESS>Cc: Michael Wilcox <Mike.Wilcox<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Documentation suggestion - Add a new type screen should show 'catagorical' vs 'numeric' (#51208)
Link to PR is here https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs-pr%2Fpull%2F110036&data=02|01|Mike.Wilcox%40microsoft.com|c10f8dd531814f3fa08808d7d7398f45|72f988bf86f141af91ab2d7cd011db47|1|0|637214514531553317&sdata=11SG03EggoPtsvJGK3aMHAgnd5lWwEDGp1Tj1Zeh1P4%3D&reserved=0
#please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F51208%23issuecomment-608050584&data=02|01|Mike.Wilcox%40microsoft.com|c10f8dd531814f3fa08808d7d7398f45|72f988bf86f141af91ab2d7cd011db47|1|0|637214514531553317&sdata=eGanZ9Ii71MCCJ9JFFRkkCwxGeFn4BU909CYUAbpFYs%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANWCYMQWZ4ATXUWOQI4U72LRKTPLVANCNFSM4LWGVMWA&data=02|01|Mike.Wilcox%40microsoft.com|c10f8dd531814f3fa08808d7d7398f45|72f988bf86f141af91ab2d7cd011db47|1|0|637214514531563309&sdata=D8lDe1TPrMSw5jhwITZbvKLe%2Bu0GMx%2FJcVAmgPMpYkU%3D&reserved=0.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F51208%23issuecomment-608082427&data=02|01|v-vidool%40microsoft.com|6b6870e827844a6175e208d7d7492531|72f988bf86f141af91ab2d7cd011db47|1|0|637214581461881107&sdata=qJ51evJZK1ydrSMVjGTSTG1C71oL7IkOZhRlp26ruoI%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMLGSHCCH2EPMM3BLKDVUE3RKT4N5ANCNFSM4LWGVMWA&data=02|01|v-vidool%40microsoft.com|6b6870e827844a6175e208d7d7492531|72f988bf86f141af91ab2d7cd011db47|1|0|637214581461891102&sdata=udsJrR95CH62mXI1XoHxG3sGzPjtN0wrkfvTTHqOIRE%3D&reserved=0.


Require pytest 7.2.0 or greater

pytest 7.2.0 and beyond removes dependency on legacy py package

Let's agree on whether we should have a max or not in Alexandre's PR and then do the same thing here.

Is the py package being deprecated? If yes, I'm fine with bumping up the pytest version. Otherwise, I'm not so sure.
The 4.6.9 version of pytest was chosen because that's the version you get when you install it with apt-get on Ubuntu Focal. Requiring a higher version will force users to use pip3 to install pytest, which puts more burden on users who would like to manage all their packages by apt-get for whatever reason. In fact, I would change the min version of numpy to 1.17.4 so that all the min versions can be satisfied by apt-get without using pip3.

Updating pytest but leaving pytest-xdist capped at 2.5 won't remove the dependency on py.

@NathanielMcVicar Interesting. Thanks for pointing that out!
I did try and check that and neither 2.5.0 (https://github.com/pytest-dev/pytest-xdist/blob/v2.5.0/setup.cfg) nor 1.31.0 (https://github.com/pytest-dev/pytest-xdist/blob/v1.31.0/setup.py#L3) directly required py and only specified min versions of pytest. I didn't think about that pytest-xdist could indirectly require py under the assumption that pytest brings it in.
Based on the link you provided, it looks like pytest-xdist is dropping that indirect depedency as well. I'll hunt down what version that applies to and update this PR.

@gongsu832

Is the py package being deprecated? If yes, I'm fine with bumping up the pytest version. Otherwise, I'm not so sure.

I missed replying to this. Basically yes. From https://pypi.org/project/py/
NOTE: this library is in maintenance mode and should not be used in new code.

I hope pytest-xdist is the only secret dependency on py. It's just the one I happened to have dealt with before :) Otherwise, I'm happy with this change, but I want to make sure @gongsu832 is too, because I sympathize deeply with people trying to avoid pip. Can anyone confirm that numpy 1.17 would work? I see that onnxruntime requires numpy 1.16+ so we should be good there.

@NathanielMcVicar numpy 1.17.4 works fine. All our Jenkins CI builds actually use 1.17.4.

I hope pytest-xdist is the only secret dependency on py. It's just the one I happened to have dealt with before :)

Apparently the answer is yes and no. pytest-xdist itself no longer requires py. However we use the --forked flag which appears to have been moved out of pytest-xdist and into pytest-forked. To keep --forked we'd need add pytest-forked and that still uses py. Instead, I dropped --forked.

We now have warnings like this that we didn't have before:
=============================== warnings summary ===============================
Release/inference_backend.py:1313
  /workdir/onnx-mlir/build/test/backend/Release/inference_backend.py:1313: UserWarning: Cannot deduce desired output endianness, using native endianness by default.
    warnings.warn(

-- Docs: https://docs.pytest.org/en/latest/warnings.html

Looking a bit further, it appears that we might have had those warnings before but they were suppressed by the -q (decrease verbosity) option. The behavior of the option may have changed now. Nevertheless, it would be nice to understand the source of those warnings.

@gongsu832 In some of the constant tests there's no actual input tensors for the model. The if for the endianness check goes down the warning path when there's no input tensors for the model because there's no input tensor to check.
The lack of input tensors also created issues in https://github.com/onnx/onnx-mlir/pull/2035 for the python test client. The fix there was to create a fake input with a [0] so the test infrastructure wouldn't explode in that scenario. I think the same thing can be generalized slightly earlier to address the endianness warning.
If we create the dummy input just a few lines sooner, then there will be an input to feed into the endianness check. As a side bonus, we can then drop a couple ifs entirely and reduce the complexity a little bit.

Can we lower the min version of numpy to 1.17.4? That's the default installed by apt-get on Linux Ubuntu Focal.

@gongsu832 I'm ok with that. I'm pushing up a commit with the change. If any one feels differently, let me know and I can revert it.

Ok, changes complete and tests passing. This is ready for review.

Jenkins Linux s390x Build #10136 [push] Require pytest 7.2.0 or ... started at 13:51

Jenkins Linux amd64 Build #10125 [push] Require pytest 7.2.0 or ... started at 12:51

Jenkins Linux ppc64le Build #9189 [push] Require pytest 7.2.0 or ... started at 13:56

Jenkins Linux s390x Build #10136 [push] Require pytest 7.2.0 or ... passed after 1 hr 42 min

Jenkins Linux amd64 Build #10125 [push] Require pytest 7.2.0 or ... passed after 1 hr 54 min

Jenkins Linux ppc64le Build #9189 [push] Require pytest 7.2.0 or ... passed after 2 hr 2 min

@cjvolzka Note that we typically ask the dev to signoff the commit with the --signoff option when you do git commit.

I thought I did otherwise the DCO test would fail. Did I miss one or does it need to be pgp signed and sign off message in the commit.

I thought I did otherwise the DCO test would fail. Did I miss one or does it need to be pgp signed and sign off message in the commit.

Oops. That was me. I replied from my phone and apparently had it signed in from my personal GitHub account.

I also thought DCO would fail if you didn't do git commit --signoff. Apparently that's not the case. Maybe there is some sort of git commit hook needs to be setup. I don't know much about the DCO stuff.


Change the output path for assets when using css url processing

Hello.
As described in the title, I'm looking for a way to specify the output folder for the assets generated through css url processing. Is there an option available for this?

Use mix.setResourceRoot().


Designing Lean Operators

The page needs to be updated.
There was a breaking change in controller-runtime so that BuilderWithOptions has been removed.
The use case should now be covered by cache.Options, the ByObject field being now a map.
Controller-runtime design document: https://github.com/kubernetes-sigs/controller-runtime/blob/main/designs/cache_options.md

This is the page in question:
https://master.sdk.operatorframework.io/docs/best-practices/designing-lean-operators/

Relates: #6381

Changes introduced in v0.15.0
SDK currently using  v0.14.1.
Thanks for linking the bump issue.

/remove-lifecycle stale

More relevant than even since https://github.com/operator-framework/operator-sdk/pull/6613 has merged

/remove-lifecycle stale


Add prefix to RequestLibrary.Get & Fix PVC alert error

Add prefix to RequestLibrary.Get to avoid keyword conflicts
Update Teardown in PVC alert error to avoid keyword conflicts
Note: differences in PVC size between RHODS 1.2.0 and 1.3.0 are now handled in the test notebook itself

Signed-off-by: Jorge Garcia Oncins<EMAIL_ADDRESS>

It's being tested right now:

RHODS 1.2.0: job 240
RHODS 1.3.0: job-239

As long as clean up works and PR tests come back green, lgtm


301 redirect loop when creating a task via Mailgun webhook

I'm not absolutely sure if there's something wrong on my Apache config but Kanboard gives a 301 error when trying to create a task via Mailgun webhook. I have checked my .haccess files and there is nothing related to redirects.
From the Mailgun log:
{
  "severity": "temporary",
  "tags": [],
  "delivery-status": {
    "retry-seconds": 7200,
    "message": "<!DOCTYPE HTML PUBLIC \"-//IETF//DTD HTML 2.0//EN\">\n\n<title>301 Moved Permanently</title>\n\n
Moved Permanently
\n
The document has moved here.
\n
\n<address>Apache/2.2 Server at www.mydomain.tld Port 80</address>\n\n",
    "code": 301,
    "description": "<!DOCTYPE HTML PUBLIC \"-//IETF//DTD HTML 2.0//EN\">\n\n<title>301 Moved Permanently</title>\n\n
Moved Permanently
\n
The document has moved here.
\n
\n<address>Apache/2.2 Server at www.mydomain.tld Port 80</address>\n\n",
    "session-seconds": 0.23880386352539062
  },
  "envelope": {
    "targets": "http://www.mydomain.tld/kanban?controller=webhook&action=mailgun&token=xxx",
    "transport": "http",
    "sender"<EMAIL_ADDRESS>  },
  "log-level": "warn",
  "id": "c7Db7R_4Q6WlKSYGIfRqfw",
  "campaigns": [],
  "reason": "generic",
  "user-variables": {},
  "flags": {
    "is-routed": true,
    "is-authenticated": true,
    "is-system-test": false,
    "is-test-mode": false
  },
  "timestamp":<PHONE_NUMBER>.227647,
  "message": {
    "headers": {
      "to"<EMAIL_ADDRESS>      "message-id"<EMAIL_ADDRESS>      "from": "Kanboard User<EMAIL_ADDRESS>      "subject": "Does it function"
    },
    "attachments": [],
    "recipients": [
<EMAIL_ADDRESS>    ],
    "size": 1959
  },
  "recipient": "http://www.mydomain.tld/kanban?controller=webhook&action=mailgun&token=xxx",
  "event": "failed"
}

I just tested with my own installation and that works for me.
Just in case, are you sure that the webhook url is correct? a slash seems to be missing: http://www.mydomain.tld/kanban?controller... should be http://www.mydomain.tld/kanban/?controller...

Correcting the application URL solved the problem, thank you!  In the Kanboard application settings I had my Kanboard URL without slash in the end, so the integration webhook urls were not constructed correctly. I had blindly copied the webhook URL to Mailgun without checking its correctness.


Last Week in Pony - June 14, 2020

Audio from the Pony developers sync for June 9, 2020 is now available.

Version 0.1.7 of ponylang-mode has been released.
See the release notes for more details.

Version 0.1.8 of ponylang-mode has been released.
See the release notes for more details.

Version 0.2.0 of ponylang-mode has been released.
See the release notes for more details.

Version 0.2.1 of ponylang-mode has been released.
See the release notes for more details.

Version 0.3.0 of ponylang-mode has been released.
See the release notes for more details.

The 'Add RFC for explicit self member access.' RFC has been updated to 'new'
See the RFC for more details.

The 'Add RFC for assign param syntax.' RFC has been updated to 'new'
See the RFC for more details.

The 'The 'Add log_level argument to LogFormatter.apply()' RFC has been updated to 'new'. See the RFC for more details.


Fix broken SSE 'stream' APIs

'stream' supported operations Accounts/Ledgers/Transactions/Payments/EffectsRequestBuilder, are broken with the conversion to Android

fixed in https://github.com/kinfoundation/android-stellar-sdk/pull/8


Fix unknown table id in keyviz

Maybe we can hide the unknown regions caused by placement rules.

Codecov Report

Merging #1220 (b665ddc) into master (e79a650) will decrease coverage by 7.60%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1220      +/-   ##
==========================================
- Coverage   31.66%   24.06%   -7.61%
==========================================
  Files         291       92     -199
  Lines       17115     9314    -7801
  Branches      686        0     -686
==========================================
- Hits         5419     2241    -3178
+ Misses      11465     6933    -4532
+ Partials      231      140      -91

Flag
Coverage Δ

be_integration_test_latest
?

be_integration_test_v4.0.1
?

be_unit_test
24.06% <0.00%> (-0.02%)
:arrow_down:

e2e_test
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

The situation tends to be resolved by this pr may be caused by TiDB for some reason. https://github.com/pingcap/tidb/issues/34318
I'll close this pr first and fix another unknown table id issue caused by the keyviz region key merge.