Failing to implement POST macro

Hi,
I am failing to implement a 'post' macro. Please can you guide me as to what I am doing wrong? And there is very little documentation around testing a post.
defmodule Skynet.Person.API do
  use Maru.Router
  alias Skynet.Person
  alias Skynet.Repo
  version "v1"

  resource :person do
    # desc "description"
    get ":id" do
      entry = Repo.get!(Person, params[:id])
      present entry, with: Person.Entity
    end

    # desc "description"
    params do
      requires :name, type: String
      requires :phone, type: String, regexp: ~r/^27[0-9]{9}$/
    end
    post do
      %Person{name: params[:name], phone: params[:phone]}
      |> Repo.insert!
      status 200
    end
  end
end

defmodule Skynet.API do
  use Maru.Router

  plug Plug.RequestId
  plug Plug.Logger
  resources do
    mount Skynet.Person.API
  end

  rescue_from Unauthorized, as: e do
    IO.inspect e
    status 401
    "Unauthorized"
  end

  rescue_from :all, as: e do
    IO.inspect e
    status 500
    "Server Error"
  end
end

@ anthony-mac> curl -H "Accept-Version:v1" -H "Content-Type: application/json" --request POST --data '{"name": "anthony", "phone": "27745765000"}' http://<IP_ADDRESS>:8000/person
Server Error

2015-10-23 13:32:48.495 [info] POST /person
%Maru.Exceptions.NotFound{method: "POST", path_info: ["person"]}

iex(2)> Skynet.Person.API.__endpoints__
[%Maru.Router.Endpoint{block: {:__block__, [line: 23],
   [{:|>, [line: 37],
     [{:%, [line: 36],
       [{:__aliases__, [counter: 0, line: 36], [:Person]},
        {:%{}, [line: 36],
         [name: {{:., [line: 36], [Access, :get]}, [line: 36],
           [{:params, [line: 36], nil}, :name]},
          phone: {{:., [line: 36], [Access, :get]}, [line: 36],
           [{:params, [line: 36], nil}, :phone]}]}]},
      {{:., [line: 37],
        [{:__aliases__, [counter: 0, line: 37], [:Repo]}, :insert!]},
       [line: 37], []}]}, {:status, [line: 38], [200]}]}, desc: nil,
  helpers: [], method: "POST",
  param_context: [%Maru.Router.Param{attr_name: :name, default: nil, desc: nil,
    group: [], nested: false, parser: Maru.ParamType.String, required: true,
    validators: []},
   %Maru.Router.Param{attr_name: :phone, default: nil, desc: nil, group: [],
    nested: false, parser: Maru.ParamType.String, required: true,
    validators: [regexp: ~r/^27[0-9]{9}$/]}], path: ["person"], version: "v1"},
 %Maru.Router.Endpoint{block: {:__block__, [line: 23],
   [{:=, [line: 26],
     [{:entry, [line: 26], nil},
      {{:., [line: 26],
        [{:__aliases__, [counter: 0, line: 26], [:Repo]}, :get!]}, [line: 26],
       [{:__aliases__, [counter: 0, line: 26], [:Person]},
        {{:., [line: 26], [Access, :get]}, [line: 26],
         [{:params, [line: 26], nil}, :id]}]}]},
    {:present, [line: 27],
     [{:entry, [line: 27], nil},
      [with: {:__aliases__, [line: 27, counter: 189], [:Person, :Entity]}]]}]},
  desc: nil, helpers: [], method: "GET", param_context: [],
  path: ["person", :id], version: "v1"}]

Note: I am using Maru version 0.7.1, because anything above results in the following error during tests:
  1) test v1 post /person (Skynet.Person.Test)
     test/person_test.exs:29
     ** (UndefinedFunctionError) undefined function: :urlencoded.parse/5 (module :urlencoded is not available)
     stacktrace:
       :urlencoded.parse(%Plug.Conn{adapter: {Plug.Adapters.Test.Conn, :...}, assigns: %{}, before_send: [], body_params: %Plug.Conn.Unfetched{aspect: :body_params}, cookies: %Plug.Conn.Unfetched{aspect: :cookies}, halted: false, host: "www.example.com", method: "POST", owner: #PID<0.643.0>, params: %{}, path_info: ["person"], peer: {{127, 0, 0, 1}, 111317}, port: 80, private: %{}, query_params: %{}, query_string: "", remote_ip: {127, 0, 0, 1}, req_cookies: %Plug.Conn.Unfetched{aspect: :cookies}, req_headers: [{"content-type", "application/json"}, {"accept-version", "v1"}], request_path: "/person", resp_body: nil, resp_cookies: %{}, resp_headers: [{"cache-control", "max-age=0, private, must-revalidate"}], scheme: :http, script_name: [], secret_key_base: nil, state: :unset, status: nil}, "application", "json", %{}, [parsers: [:urlencoded, :multipart, :json], pass: ["*/*"], json_decoder: Poison])
       (plug) lib/plug/parsers.ex:186: Plug.Parsers.reduce/6
       test/person_test.exs:3: Skynet.Person.Test.make_response/2
       test/person_test.exs:37

Could you please show me your config.exs file ? Maybe inconsistent versioning strategy was used.
And you can wirte UnitTest like this:
defmodule Skynet.Person.APITest do
  use Maru.Test, for: Skynet.Person.API

  test "post /person" do
    assert %Plug.Conn{} = conn(:post, "/person", %{"name" => "anthony", "phone" => "27745765000"}) |> make_response
  end
end

The conn function comes from Plug.Test.
And for the 2nd question:
I'll check and fix it as soon as possible, recommend using the latest release.

I have found the issue. In the post I had to include a body to the post.
So my implementation is now:
    post do
      entry = %Person{name: params[:name], phone: params[:phone]}
              |> Repo.insert!
      status 200
      present entry, with: Skynet.Repo.Entity
    end

The error was very misleading.
I can not use the latest version, as I run into the second issue. If you review plug parser you will see that in the init :urlencoded gets mapped to Plug.Parsers.URLENCODED. See: https://github.com/elixir-lang/plug/blob/master/lib/plug/parsers.ex

Perfect this works! Thank you


bug: Logic for checking for directory is incorrect

The condition should be entry.stat().filetype === wasi.FILETYPE_DIRECTORY to check if the entry filetype is actually a directory.
https://github.com/bjorn3/browser_wasi_shim/blob/51f5c2988e6fa13c0a8643f8bf2c36e31bf326fe/src/fs_fd.ts#L283

The logic here is if we want to open a directory (oflags contains OFLAGS_DIRECTORY), but the target is not directory (filetype is not FILETYPE_DIRECTORY), then return ERRNO_ISDIR as error.

Looks like the comment and error number are wrong though. It should be ERRNO_NOTDIR: https://github.com/bytecodealliance/wasmtime/blob/6aaf682894cf0ed974e2aa86fddf5d949af54add/crates/wasi-common/cap-std-sync/src/dir.rs#L95-L96


Default value using raw()

I've been trying to create a default value for an uuid column in mysql but I couldn't arrive to make it work. Is it doable? Is it a bug or am I doing something wrong?
I couldn't fin a reference of how to use a raw value as a default.
What I did is to add this to my model:
  static get jsonSchema() {
    return {
      type: 'object',
      properties: {
        uuid: {
          type: 'uuid',
          default: raw('UUID()'),
        },
      },
    };
  }

And I get this error:
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings:
   [ 1,
     '{"companies":["Samsung"],"from":"2016-09-26","to":"2017-09-26"}',
     'BrandPerception',
     { _sql: 'UUID()', _args: null } ],
  __knexQueryUid: 'f0d2936c-fa78-4033-90b5-f0ca2e2d50d9',
  sql: 'insert into `analyses` (`alien_project_id`, `intent`, `type`, `uuid`) values (?, ?, ?, ?)' }
Error: ER_BAD_FIELD_ERROR: Unknown column '_sql' in 'field list'

You cannot set non-json values to JSON schema. The best way to do this would be to set a default value in the database.
If you cannot set the default value in the database, you can use $beforeInsert hook:
class BaseModel extends Model {
  $beforeInsert() {
    this.uuid = uuid.v4(); // or something like that
  }
}

And then instead of inheriting your models from Model inherit them from BaseModel.

http://knexjs.org/#Schema-defaultTo

Thank you!
PS:
Set the default value in the database in this case is not an option,
because the default value has to be a static literal value and I need to
put a function call "UUID()". Some people recommend using a trigger, but we
prefer not to add triggers to the database.
2017-11-04 6:03 GMT-03:00 Sami Koskimäki<EMAIL_ADDRESS>
http://knexjs.org/#Schema-defaultTo
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Vincit/objection.js/issues/594#issuecomment-341882342,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEa1OVf9Vlx_Jbk_Z8QzdLeJ7NaJYNHiks5szChmgaJpZM4QR20V
.

@npasserini Nope, the default value can be a function call among other things. This should work:
table.dateTime('uuid').notNullable().defaultTo('UUID()');

This is from the knex documentation:
table.timestamp('created_at').defaultTo(knex.fn.now());

I was wrong about the first example, but this works:
table.string('uuid').defaultTo(knex.raw('UUID()'));

Are you sure?
Does not work for me:
Knex:warning - migrations failed with error: create table `test` (`uuid`
varchar(255) default UUID()) - ER_PARSE_ERROR: You have an error in your
SQL syntax; check the manual that corresponds to your MySQL server version
for the right syntax to use near 'UUID())' at line 1
Error: ER_PARSE_ERROR: You have an error in your SQL syntax; check the
manual that corresponds to your MySQL server version for the right syntax
to use near 'UUID())' at line 1

I tried it with mysql:5.6.33 and
    "knex": "^0.13.0",
    "objection": "^0.8.9"

2017-11-06 12:21 GMT-03:00 Sami Koskimäki<EMAIL_ADDRESS>
I was wrong about the first example, but this works:
table.string('uuid').defaultTo(knex.raw('UUID()'));
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Vincit/objection.js/issues/594#issuecomment-342180931,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEa1OeL5xN7naeFijK3yQlmTFtcX91q3ks5szyP8gaJpZM4QR20V
.

Are you sure?
Does not work for me:
Knex:warning - migrations failed with error: create table `test` (`uuid` varchar(255) default UUID()) - ER_PARSE_ERROR: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'UUID())' at line 1
Error: ER_PARSE_ERROR: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'UUID())' at line 1

I tried it with mysql:5.6.33 and
    "knex": "^0.13.0",
    "objection": "^0.8.9"

I do this with PG, so I'm certain passing a raw to defaultTo works with knex.
For reference: https://github.com/tgriesser/knex/issues/1488
I'm not sure how UUID() works in MySQL, can you do a simple SELECT UUID(); without issues?

@npasserini adding default problem you should be able to reproduce with plain knex. It shouldn't have anything to do with objection.


Automate releases with knope

This makes PRs slightly more annoying because they will need to have either a documented change set or conventional commits, but this will automate the release process and improve documentation. This is mostly the same setup as renterd and hostd with the addition of tagging the release when the preview PR closes.
To add a changeset to a PR:

install Knope https://knope.tech/installation/
Run knope document-change

I have to install some new tool, and this is gonna stick a bunch of files in a .changesets directory in the repo?
How has the experience been with knope in renterd and hostd so far? I looked at those repos and it seems like every .changesets file is just a PR title with no further information. Have we used it to cut an actual release yet? As you can probably tell, I'm skeptical of stuff like this, but if it's been a clear win for us I'll change my tune.

I have to install some new tool, and this is gonna stick a bunch of files in a .changesets directory in the repo?
How has the experience been with knope in renterd and hostd so far? I looked at those repos and it seems like every .changesets file is just a PR title with no further information. Have we used it to cut an actual release yet? As you can probably tell, I'm skeptical of stuff like this, but if it's been a clear win for us I'll change my tune.

I find the automation to be very useful. renterd currently uses GitHub's automatic change log tool, which is terrible for human readability. hostd was manually writing the log for every release. Considering the current "release" workflow for core and coreutils is "tag whatever version feels appropriate" and we completely skip documenting changes, I'm of the opinion that the slight inconvenience is worth it.
You don't have to install the tool. You can manually create the change set file 🤷🏻

If there is another tool that gets us automated documentation and versioning with less overhead, I'm happy to use it, but this is the one I am familiar with.

Is this a knope changelog? https://github.com/SiaFoundation/desktop/pull/98 or some other tool?
AFAICT, in practice these changelogs will just be a list of the PRs that were merged since the last release. Which, sure, is better than nothing. But you don't need a tool for that.
 I did that for siad for years, and I'll happily do it again.

That is GitHub's automatically generated change log 🤢
Here are some examples of Knope ones:

https://github.com/knope-dev/knope/releases/tag/knope%2Fv0.18.0
https://github.com/SiaFoundation/sia-sdk-rs/blob/eef9425b58b11bb228b9360dc7f052b2d2a18e62/sia/CHANGELOG.md
https://github.com/SiaFoundation/hostd/blob/57eca595e18c2325a4a6dcd54ac5cd71169836fc/CHANGELOG.md

wait. You can just press a button in the GitHub UI to create the .changeset file? ok, that's much more tolerable lol


python3 Type error

On the master branch, when formatting a python file, I get the following error:

I am using Python 3.5 on Arch Linux
When I switch to you dev branch, it works, but it adds an endline at the end of the file.  I remember seeing the endline problem come up and get fixed, but does not seem to be working for me.
It does work fine from command line usage of autopep8.

Thank you for the report. I was aware of the problem. The patch for windows line endings introduced this bug for vim with +python3 support. Unfortunately I don't have python3 support and I lack the time to get my hands on it at the moment.
It's probably a very simple fix, where you have to convert some str objects to byte objects in the TryFormatterPython3 functions, before assigning it to vims buffer.
Would you care to give it a try?

I'll take a look sometime this weekend and see what I find.

just kidding - it was bugging me so I thought I'd check into it sooner

:+1:

I'm getting:
Error detected while processing function <SNR>68_TryAllFormatters..<SNR>68_TryFormatterPython3:
line   32:
Traceback (most recent call last):
  File "<string>", line 27, in <module>
TypeError: 'str' does not support the buffer interface

Not sure if that is a similar/same error, but when I made the change using the #77 PR I get the following error:
Error detected while processing function <SNR>68_TryAllFormatters..<SNR>68_TryFormatterPython3:
line   33:
Traceback (most recent call last):
  File "<string>", line 28, in <module>
TypeError: 'str' does not support the buffer interface

I'm trying to format a JavaScript file, if that helps.
If this is a different issue, please let me know and I can create a new issue for it.
Thanks!

I'm not 100% familiar with how it works for JavaScript formatting.
It might be related to #77 but it sounds like you were having this issue before you tried it?

@gblock0 I just tested it with a JS snippet and got this error (which is what I was getting in python before #77):
Error detected while processing function <SNR>47_TryAllFormatters[41]..    <SNR>47_TryFormatterPython3:
line   33:
Traceback (most recent call last):
  File "<string>", line 28, in <module>
TypeError: a bytes-like object is required, not 'str'

But I do NOT get it when formatting a python file anymore.

@Chiel92 @gblock0  I figured it out .. I inadvertently did not test the case where the line ending was not there and it converted funky ... creating a new pull request for this

@gblock0 can you test this on your end?

Works like a charm! THANKS!

That is great to hear, thanks for letting me know!

Thank you @chrcoe for the help!


Do not depend on rspec meta gem

The rspec gem is a shortcut to depend on rspec-core, rspec-expectations and rspec-mocks, but depending on it directly clashes with the rspec-rails gem that is also a bag of sub RSpec dependencies.
This helps with scenarios where one meta gem locks the update of another one, like in the following:
bundle update rspec-rails
Fetching gem metadata from https://rubygems.org/.......
Fetching gem metadata from https://rubygems.org/.
Resolving dependencies.......
Bundler could not find compatible versions for gem "rspec-mocks":
  In Gemfile:
    mongoid-rspec was resolved to 4.0.1, which depends on
      rspec (~> 3.3) was resolved to 3.7.0, which depends on
        rspec-mocks (~> 3.7.0)

    rspec-rails (= 3.8.0) was resolved to 3.8.0, which depends on
      rspec-mocks (~> 3.8.0)

Seems reasonable if you can fix the build ;)

@dblock all green now :)


Possible memory leak / OOM when closing over large constant in jitted loop

Description
I'm getting consistent out of memory errors when jitting functions that close over large constant arrays. It seems similar to #11448 but only occurs when compiling the closed over function.
MWE:
import numpy as np
import jax
import jax.numpy as jnp

ns = 900
ne = 1000
nq = 800

bigmat = jnp.array(np.random.random((ne,nq,ns)))

x = jnp.empty(ns)

def foo(x, bigmat):

    def body(i, f):
        gx = bigmat[i]@x
        # do some other stuff that isn't important
        f = f.at[i].set(gx.sum())
        return f

    f = jnp.zeros(x.shape[0])
    f = jax.lax.fori_loop(0, 1, body, f)
    return f

# this runs fine, no OOM issue
f = foo(x, bigmat)

# this also runs fine, basically the same runtime as expected since the
# fori_loop is already compiled and thats most of the cost
f = jax.jit(foo)(x, bigmat).block_until_ready()

# this causes OOM
f = jax.jit(lambda y: foo(y, bigmat))(x)

Also curiously, if bigmat is replaced with zeros or ones, it seems to work fine, suggesting there's some lower level compiler optimization going on.
What jax/jaxlib version are you using?
jax=0.4.2, jaxlib=0.4.2
Which accelerator(s) are you using?
tried on intel/AMD cpus, nvidia v100 and a100 gpus
Additional system info
No response
NVIDIA GPU info
No response

Try upgrading your jax and jaxlib version to 0.4.11 and see if it still happens?

I tried a few different versions of jax/jaxlib with similar behavior. The exact max size of the big array varies slightly with different versions but in all cases the array should fit into memory many times over, but runs out when compiling.
Also, in the cases where the compilation does succeed in the closed over case, the compilation time is 10-50x longer than in the non-closed over case, which also seems excessive.


very basic implementation to get a "connect" LED status display

Hope this time it fits.

Thanks again, and sorry for the branch trouble!


Problem with Firefox and Chrome at first load

I have used jquery panzoom for image panzoom on weathermap dot ovh dot net.
At first load, I have a strange thing which I cannot reproduce when loading another map
When mousewheel event is first triggered (only at first load), the zoom is not centered on the mouse position but seems on the top left.
Problem is not seen on MSIE, only on Firefox and Chromium/Chrome.

When trying to debug the problem, on Firefox, I found the matrix is not the same when mousescroll.wheel is triggered :
On Firefox
["0.0873684", "0", "0", "0.0873684", "-116.681", "-148.866"]
["0.137368", "0", "0", "0.137368", "-163.14", "714.795"]
["0.187368", "0", "0", "0.187368", "-209.599", "1578.46"]
On MSIE :
["0.107895", "0", "0", "0.107895", "-144.093", "-179.645"]
["0.157895", "0", "0", "0.157895", "-156.648", "-301.358"]
["0.207895", "0", "0", "0.207895", "-169.203", "-423.071"]
Seems that Firefox is not working as IE when executing this code :
$panzoom.parent().on('mousewheel.focal', function( e ) {
e.preventDefault();
var delta = e.delta || e.originalEvent.wheelDelta;
var zoomOut = delta ? delta < 0 : e.originalEvent.deltaY > 0;
$panzoom.panzoom('zoom', zoomOut, {
increment: inc,
animate: false,
focal: e
});
console.log($panzoom.panzoom("getMatrix"));
});
Is anything that is not working on Firefox and Chrome regarding mousewheel.focal event ? e.delta is  always undefined and zoomOut is true|false.
Seems problem is located on focal: e usage ?

Problem seems to be located on  _checkDims: function()
height is the height of real image, not panzoomed image ... Don't know why and how to fix, because function is only a return this.dimensions .....

Find the problem, needed $(window).load(function() {}); before launching panzoom .... now this.dimensions height and width are returning good values.


Remove blocksize_distributed split on y

Remove blocking on the Y axis from CPU and GPU schedules, in an attempt to remove constraints on local parallelism.
CPU performance/scalability seems about the same, in my testing.  (See below)
GPU schedule was simplified a bit more, to remove blocking on the Y axis, and remove my misuse of the gpu_lanes directive.  Performance is about 40% slower.  There are further improvements possible here, but it's not a priority.
Distribution pattern should be identical between CPU and GPU.
Here are the CPU runtimes I got from testing with the Sandia dataset, 4 threads per node.
before patch:
2 nodes:
Halide backprojection returned 0 in 42765 ms
Halide backprojection returned 0 in 43119 ms
4 nodes:
Halide backprojection returned 0 in 21077 ms
Halide backprojection returned 0 in 21176 ms
Halide backprojection returned 0 in 24639 ms
Halide backprojection returned 0 in 24984 ms
8 nodes:
Halide backprojection returned 0 in 11091 ms
Halide backprojection returned 0 in 11414 ms
Halide backprojection returned 0 in 11608 ms
Halide backprojection returned 0 in 12119 ms
Halide backprojection returned 0 in 12187 ms
Halide backprojection returned 0 in 12281 ms
Halide backprojection returned 0 in 14136 ms
Halide backprojection returned 0 in 14310 ms

after patch:
2 nodes:
Halide backprojection returned 0 in 43549 ms
Halide backprojection returned 0 in 45614 ms
4 nodes:
Halide backprojection returned 0 in 20503 ms
Halide backprojection returned 0 in 20947 ms
Halide backprojection returned 0 in 24490 ms
Halide backprojection returned 0 in 25900 ms
8 nodes:
Halide backprojection returned 0 in 11230 ms
Halide backprojection returned 0 in 11355 ms
Halide backprojection returned 0 in 11506 ms
Halide backprojection returned 0 in 11506 ms
Halide backprojection returned 0 in 12184 ms
Halide backprojection returned 0 in 12421 ms
Halide backprojection returned 0 in 14449 ms
Halide backprojection returned 0 in 16613 ms

I ran this on Summit for 32kx32k at 256 nodes with the distributed CPU schedule.  There is some improvement in thread utilization, but not as much as I'd hoped.  It was about 31-32 threads for img and now is about 32-34.  Total runtime improved from 1526 s to 1417 s.
I might merge this later, but I think we should further discuss/experiment.
Note also: since blocksize_distributed is not longer used here, it should be removed entirely.

Thanks for trying it.  Could you paste some profile output from one of the nodes?

SARBP-Sandia-256.746334.log
SARBP-Sandia-256-no_blocksize_distributed.749331.log

Looking at the last few nodes in the log, I see that the thread parallelism hovering between 31-38 for all Funcs, whether it's distributed or not.  In fact, the only "threads: 42.0" I see are for really tiny things, <5ms.
Do you observe the same pattern for other apps?

This is the only app I'm running.

Ok.  Here's how it looks to me:
We had a practical limit on parallelism of 32 threads.  The profiles reflect this:
1: 167:   norm_rr0:              27424.931ms(5%)   threads: 31.999 peak: 33554432 num: 1048576   avg: 1048576
1: 167:   img:                   445488.750ms(89%) threads: 32.000 stack: 64
1: 224:   norm_rr0:              56829.039ms(11%)  threads: 30.149 peak: 33554432 num: 1048576   avg: 1048576
1: 224:   img:                   425729.906ms(84%) threads: 30.000 stack: 64
1: 174:   norm_rr0:              37198.226ms(7%)   threads: 31.169 peak: 33554432 num: 1048576   avg: 1048576
1: 174:   img:                   422619.937ms(88%) threads: 31.175 stack: 64
1:   0:   norm_rr0:              27218.693ms(5%)   threads: 31.999 peak: 33554432 num: 1048576   avg: 1048576
1:   0:   img:                   446599.906ms(90%) threads: 31.999 stack: 64

Close to 32, but never above 32.
This patch lifts that restriction, but it didn't help that much.
1:  84:   norm_rr0:              65782.867ms(15%)  threads: 35.729 peak: 44040192 num: 1048576   avg: 1048576
1:  84:   img:                   352502.406ms(80%) threads: 33.031 stack: 64
1: 249:   norm_rr0:              37135.921ms(8%)   threads: 33.327 peak: 44040192 num: 1048576   avg: 1048576
1: 249:   img:                   389488.031ms(86%) threads: 32.499 stack: 64
1: 117:   norm_rr0:              38079.222ms(7%)   threads: 32.351 peak: 44040192 num: 1048576   avg: 1048576
1: 117:   img:                   419290.718ms(87%) threads: 34.343 stack: 64
1:   0:   norm_rr0:              27763.125ms(6%)   threads: 32.092 peak: 44040192 num: 1048576   avg: 1048576
1:   0:   img:                   409558.718ms(89%) threads: 32.293 stack: 64

Generally greater than 32, but not by much.
So it seems like something else is going on.  Do you get good thread utilization for single node jobs?


x/pkgsite: viewing std at master is broken

For instance, https://pkg.go.dev/bytes@master is a 404, and https://pkg.go.dev/reflect@master gives an empty package.
I'm fairly sure these used to work. The latter is even linked form https://tip.golang.org/doc/go1.18 under the "reflect" section.
If the changelog will be shown to users in this form, then this bug is a release blocker for 1.18, I think.
cc @golang/pkgsite @golang/release

Indeed. This is #49229.

Ah, apologies, my search did not show those.

Not a problem!


Fix and Add more screenshots for Boring Cyborg App

Checklist

[ ] If you're adding a listing for your app, be sure your app is in the /_apps/ folder before opening this Pull Request.
[x] All comments in frontmatter need to be removed.
[x] Performs a useful action through the GitHub API that solves an existing problem for developers: {explain the action here}

View rendered _apps/boring-cyborg.md

cc @gr2m


[security] do not cache all environment variables in configuration cache

Expected Behavior
only explicitly requested environment variables are put in the cache. Meaning only if I do something like this would CI's result be cached
providers.environmentVariable("CI")

Current Behavior (optional)
from what I know all environment variables are put in the cache
Context
I'm writing this as a result of my great annoyance with this github vulnerability, which is arguably their fault for putting secure credentials in environment variables in the first place (they always put a TOKEN in).
additionally, I think it would be good to provide a way to define a secure process.

Please follow the policy when reporting security issues.

Thank you for your interest in Gradle!
This issue needs a decision from the team responsible for that area. They have been informed. Response time may vary.

@gradle/bt-configuration can you confirm it already works this way? That is the configuration cache would only write environment variables to the cache which are used as task inputs?
Should we consider clarifying this in the documentation?

gradle/bt-configuration can you confirm it already works this way? That is the configuration cache would only write environment variables to the cache which are used as task inputs?

That is indeed the case. There are plugins, though, which are not prepared for the configuration cache and grab the whole environment. The configuration cache report can help to pinpoint these plugins. There are APIs that can help to reduce the number of environment variable read.

@mlopatkin so what's going on in #27797 that's invalidating the cache based on an environment variable that's not requested by gradle?


add Couple-quest

https://github.com/Clonewayx/Couple-quest GPL-3.0
Simple "board-like" game for couples.

The new projects home seems to be https://github.com/Couple-quest/Couple-quest accordig to your PR #62.
I'm not sure this should be on the list for three reasons:

The project doesn't seem to be mature enough (see UI)
The code files are only partly written in english language so the projects way of implementing its functionalities can't be understand by most of the people
The filestructure of the project is flat

This project could be added to the Godots Showcase but I'm not a fan of adding it to an awesome list.

Closing per @pwab's comment.


Hub 2.0 dev

Initial Structure of Hub 2.0

@imshashank can we utilize release/2.0 branch?

why did you create a hubV2 folder instead of just deleting all other files


Issue with firefox

There is some issue with this directive on Firefox browser, I have tested the directive in Firfox version 31.0 and its not working, and it doesn't even showing any error messages in console.
I think its the issue with "e.innerText" wchich doesn't support in Firfox. Either that should be replaced with innerHTML (supports in all browsers) or with textContent (which won't work in IE8).

indeed, simply replacing innerText with innerHTML solved the issue for me

I have the same problem... Solution of Freundschaft doesn't work for me... Firefox 38.0.5

I fixed it by replacing innerText to textContent

innerText is no longer supported by firefox as of 2015-03-13.
Firefox uses the W3C-compliant textContent property.

So it appears that the unminified file in the src/count-to.js is using .textContent() but the minified file in build/angular-count-to.min.js uses .innerText() which causes the Firefox breakage.
I was able to use the plugin w/o forking by adding this to my bower.json file to just use the src/count-to.js instead of referencing the minified version.
"overrides": {
"angular-count-to": {
"main": "src/count-to.js"
},
}
I end up minifying the code down the line anyhow.


Add logo header to ReadMe

I have been trying to move away from ln-vortex and just have it be named vortex anyway you can update that here?

@benthecarman np updated to just vortex


Problem using "-" in the string

Thank you a lot for this great compount splitter. It works really good. Unfortunately it does have some problems with hyphens.
char_split.split_compound("Kraftfahrzeug-Haftpflichtversicherung")
returns [[1.0, 'Kraftfahrzeug-Haftpflichtversicherung', 'Haftpflichtversicherung']]
The whole word is returned in the first part of the result. Do you know how to fix this issue?
thanks in advance :)

Break hyphenated words yourself and pass them one part at a time.

I would consider this a bug and I have fixed the issue.
[[1.0, 'Kraftfahrzeug', 'Haftpflichtversicherung']]
is now returned.


Upgrade vscode + deep learning image

Once this issue is resolved, we can upgrade vscode and include it in the onepanel/dl Docker image to get consistent environment across the built-in vscode, JupyterLab and training environments.

Confirmed the latest versions of ms-python and vscode work. Docker image is updated and we now need to add migrations to close this issue.


Route files are not being compiled with babel polyfill

It looks like route files are not being compiled with babel polyfill. The use of async throws an Unexpected token function error.

I'll take a look! Mind posting your .babelrc or config flags and the command you use to run? I'll try to repro and get a fix in!

I don't have a .babelrc file. My understanding is that micro is taking care of babel support for async/await functionality. A quick fix has been to use node 7 and harmony flags. This is what my npm start script looks like node --harmony-async-await ./node_modules/.bin/micro But I feel this is more of a last resort.

For sure! in node v7.6+, things should run natively, and below that micro uses async-to-gen to add async/await support. If you're transpiling, what is your deployment target and build process?

I will be deploying to zeit. I'm not sure whether I need a build process as such, my understanding is that micro should take care of it. Am I wrong?

Mainly what I'm after is a way to reproduce your issue! I'm using this lib on zeit right now without any issues, so I figure there must be something in your setup preventing the routes from being transpiled properly.

Node async/await is LTS now, and this issue is likely irrelevant. Please re-open if I closed in error!


Refactor runners to have more control over representations

This PR refactors Webtest.Utils and Webtest_js.Runner to be able to run tests using node.js and display it in familiar way:
screenshot.

Just to give a little bit more info: the idea is to use ocaml-webtest to test applications, libraries for node.js infrastructure, or smth like http://github.com/little-arhat/ppx_jsobject_conv — something, that's impossible to test in pure OCaml, since interop is required, and at the same time something that is unnecessary to test inside browser.

Got it, thanks.

OK, I'm going to merge this and have a play with it. Thanks for the patches :-)


Load hardcode Genesis notes in Tendermint Docker container

This PR loads hardcoded Genesis notes into the Tendermint Docker container, so that running docker-compose up --build -d will result in an end-to-end working setup.
NOTE: broken right now, some reason the app hash is mismatching when the node tries to start :(
I think the issue has to deal with a mismatch between DB state and what's in the tendermint Docker volume data directory, I tried doing an unsafe-reset-all but it's still giving the app hash mismatch...

The commit history is here, I can rebase or squash, but ready for review


"AndroidAlarmManager.periodic" Not working. only work for first time but not interval also not showing any error.

``void printHello() {
final DateTime now = DateTime.now();
final int isolateId = Isolate.current.hashCode;
print("[$now] Hello, world! isolate=$isolateId");
NotificationService().showScheduledNotification();
}
void main() async {
WidgetsFlutterBinding.ensureInitialized();
await AndroidAlarmManager.cancel(0);
await AndroidAlarmManager.initialize();
runApp(const MyApp());
await AndroidAlarmManager.periodic(const Duration(minutes: 1), 0, printHello);
}

Please provide info according to suggested issue template. Without full information and description we can't help.

i have the same problem, reported here https://github.com/fluttercommunity/plus_plugins/issues/686


[zh] Localize 'infrastructure_as_code'

[ ] Localize the content/en/infrastructure_as_code.md to content/zh/ in dev-zh.

acknowledge


Make the pod deployment explicit

Make the pod deployment explicit to ensure difference between managed identity & service principal mode is clear

@hemantksingh : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@hemantksingh : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

#sign-off


⚠ AGEG website (prod) has degraded performance

In d8c082f, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 1397 ms

Resolved: AGEG website (prod) performance has improved in e646923.


Cannot edit file after running ONLYOFFICE Docs using HTTPS.

Do you want to request a feature or report a bug?
bug
What is the current behavior?
Cannot edit file after running ONLYOFFICE Docs using HTTPS while everything is ok in http.
I open this file in community server(https also), error happened when i try to edit.

Exception detail:
{
"Detail": "File editing start error",
"inner": {
"message": "File editing start error",
"source": "mscorlib",
"stack": " at ASC.Web.Files.Services.WCFService.FileStorageServiceController.StartEdit (System.String fileId, System.Boolean editingAlone, System.String doc) [0x00141] in :0 \n at (wrapper dynamic-method) System.Object.lambda_method(System.Runtime.CompilerServices.Closure,object,object[])\n at System.Web.Http.Controllers.ReflectedHttpActionDescriptor+ActionExecutor+<>c__DisplayClass10.b__9 (System.Object instance, System.Object[] methodParameters) [0x00000] in :0 \n at System.Web.Http.Controllers.ReflectedHttpActionDescriptor+ActionExecutor.Execute (System.Object instance, System.Object[] arguments) [0x00000] in :0 \n at System.Web.Http.Controllers.ReflectedHttpActionDescriptor.ExecuteAsync (System.Web.Http.Controllers.HttpControllerContext controllerContext, System.Collections.Generic.IDictionary2[TKey,TValue] arguments, System.Threading.CancellationToken cancellationToken) [0x00045] in :0 \n--- End of stack trace from previous location where exception was thrown ---\n\n at System.Web.Http.Controllers.ApiControllerActionInvoker.InvokeActionAsyncCore (System.Web.Http.Controllers.HttpActionContext actionContext, System.Threading.CancellationToken cancellationToken) [0x000c7] in :0 \n at System.Web.Http.Controllers.ActionFilterResult.ExecuteAsync (System.Threading.CancellationToken cancellationToken) [0x00137] in :0 \n at System.Web.Http.Filters.AuthorizationFilterAttribute.ExecuteAuthorizationFilterAsyncCore (System.Web.Http.Controllers.HttpActionContext actionContext, System.Threading.CancellationToken cancellationToken, System.Func1[TResult] continuation) [0x00109] in :0 \n at System.Web.Http.Controllers.ExceptionFilterResult.ExecuteAsync (System.Threading.CancellationToken cancellationToken) [0x00097] in :0 ",
"type": "System.InvalidOperationException"
},
"message": "Bad request."
}
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
I followed this link https://helpcenter.onlyoffice.com/installation/docs-community-install-docker.aspx to run ONLYOFFICCE Docs in https.
step 1.
openssl genrsa -out onlyoffice.key 2048
openssl req -new -key onlyoffice.key -out onlyoffice.csr
openssl x509 -req -days 365 -in onlyoffice.csr -signkey onlyoffice.key -out onlyoffice.crt
openssl dhparam -out dhparam.pem 2048
mkdir -p /app/onlyoffice/DocumentServer/data/certs
cp onlyoffice.key /app/onlyoffice/DocumentServer/data/certs/
cp onlyoffice.crt /app/onlyoffice/DocumentServer/data/certs/
cp dhparam.pem /app/onlyoffice/DocumentServer/data/certs/
chmod 400 /app/onlyoffice/DocumentServer/data/certs/onlyoffice.key
step 2.
Run documentServer:
sudo docker run --net onlyoffice -i -t -d -p 443:443 --restart=always --name onlyoffice-document-server -v /app/onlyoffice/DocumentServer/logs:/var/log/onlyoffice -v /app/onlyoffice/DocumentServer/data:/var/www/onlyoffice/Data -v /app/onlyoffice/DocumentServer/lib:/var/lib/onlyoffice -v /app/onlyoffice/DocumentServer/db:/var/lib/postgresql onlyoffice/documentserver
step 3.

Can edit file.
DocumentServer version:
5.6.4
Operating System:
centos 7
Browser version:
Edge/Chrome lastest version.

Hello @xunmeng2015 If you run workspace over https - you need to generate a certificates for communityserver, not for documentserver. Documentserver and communityserver works each other on a local docker network. You can see instructions in this repo.
If after that problem not solved - please look at web.files.log and web.log for the errors.

Hello @xunmeng2015 If you run workspace over https - you need to generate a certificates for communityserver, not for documentserver. Documentserver and communityserver works each other on a local docker network. You can see instructions in this repo.
If after that problem not solved - please look at web.files.log and web.log for the errors.

Thanks for your help. It worked.

Thanks for your help. It worked.


Selection of matter adapter in Visu App on Android is empty

I tried to to add matter devices via the Android app "Visu". But after selecting "Nein, das Gerät ist neu", the "Gerät hinzufügen" dialog is just empty. It shows an empty selection field when I try to set the drop-down for the "Matter-Instanz".
Otherwise the "Visu" app works fine for me.
Maybe this is a bug of the app and not of this adapter. Please move the ticket if this is the case.

Sanme issue I have ... we need to find the info

This is how my settings looks like
Android
App version 1.3.0

I see a matter instanze.
When I started with visu app I did not see the matter instanz as well. I changed the ip address to the server where my active matter controller is running and I entered username and passwort for the server. After this  it worked.
(Of course there is a valid mail address for the pro account, I just only deleted it here for the screenshot)

I tried "Neue Verbindungszertifikate anfordern" and then restarted the iot adapter. Still not working. But I saw in the logs that the visu app is using the cloud connection, despite me being in local network. I got no password for my admin account, can this be the reason for the cloud fallback?

@CommanderXXL which version of the visu app do you use? On Android version 1.3.1 is available.

@Feuer-sturm it is working now. The update came yesterday evening to my phone. Now "matter.0" is written in the instance field.
For some reason the connection always uses the cloud connection instead of the local one. But it is still working. I now try to add a new thread device to my OpenThread border router to check if everything works.
Thanks for your help.

@CommanderXXL as far as I know from @Apollon77 it works as designed that the cloud connection is used for commisioning the new devies and not the local network


[BACKLOG-7111] Switch to Linux-style EOL because woodstox library encodes CRLF incorrectly

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/kettle-xml-plugin/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f plugins/kettle-xml-plugin/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  plugins/kettle-xml-plugin/src/org/pentaho/di/trans/steps/xmloutput/XMLOutput.java


Ollie: Watch bscscan.com

Ollie requests the watch of the watch_keyword bscscan\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

bscscan\.com has been seen in 2 true positives, 12 false positives, and 1 NAA.

Rejected by Spevacus in Charcoal HQ. Reason: 'Quite a few FP, and only a couple of TP. If we see more of this, we should look into rather specific expressions rather than just watching the site itself.'


Update nl.coffee

Spelling fixes + some new translations.

Coverage remained the same at 56.244% when pulling dcfeda6fcb6bcccfff1f23275c138e697c85315d on JurianLock:patch-20 into 8d2c5837c1552824ffb2cb22e9f47f296b834eb3 on codecombat:master.

Thanks!


Logging model events Causer

Hi!
I am using ActivityLog for logging user activity on models, I used the LogsActivity trait on a model (say Article) and put CausesActivity on the users model (that in my case is called Person).
Logging works but I don't get any Causer int he db, is there a way I can do this?
Thanks!
C.

Is there a way to set the causer for seeders? Or other automated jobs?  I guess technically they're not logged in so the causer isn't known from the Guard?


tooltips custom animations

Please remove the dist files

@wolfy1339 I need them for demo.
I will remove generated files before this PR is good to go.

Punting on this for v4, thanks though!


NPS Extension Advanced Configuration page is outdated

This page dates back to July of 2018.  There are several iterations of the NPS Extension since then and at least one more registry key added to the functionality.  This key "OVERRIDE_VOICE_OTP" was noted in the release notes of <IP_ADDRESS> but is not documented anywhere else that I can find.  Here's the note from the release notes:
Version <IP_ADDRESS> of the Azure MFA NPS Extension adds the following additional functionality:

Added support for OVERRIDE_VOICE_OTP registry entry to allow NPS Extension to use Voice
Approval instead of Voice OTP for installations that don't support access/challenge.

I cannot find any documentation on this functionality.  Please advise as we have customers asking what this key does and how to use it.  I am on the Azure Identity team and can help if needed.

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

ID: 17f663ca-635a-10ab-d78a-f60424a5484c
Version Independent ID: cd178cad-c4c4-16a0-ca02-ac336165cad7
Content: Configure the Azure AD MFA NPS extension - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-mfa-nps-extension-advanced.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@natharatms
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@natharatms @JamesTran-MSFT sorry for delay. This registry setting is related to future work and we don't have any documentation to share about it at this time. When the work reaches preview we will provide docs.
#please-close


Test with localstack

Hello @paguilera I want to send now this merge without create the new version because with this new changes we can test the library locally... so will be much easy for everyone who wants to collaborate.
My idea is create the local tests for dynamo and cloudwatch, then create and release the version 1.0.0

@samuelpcabral Can we have these tests running in Gtihub actions? That would be great

@MummanaSubramanya Sure.. I have been working on this today.. I am still making some improvements.
Right after this PR is approved I will send the GitHub actions code 😁

Hi @paguilera The error seems strange to me, it seems like you have some things updated and others no, because I made two different pushes to the branch... let me do the push with github actions and make sure everything is up to date... I will send another message when is done..

PR moved to #59


Add badge for first onboarding visitors

Add badge for first Onboarding visitors (removed once the onboarding wizard has been opened):

This PR includes as well:

Fix hover in Selectors card
Fix icon path in fake Dataset card of Builder tutorial

Nice. Thanks @csubira!

Usage of Onboarding is going to skyrocket once we merge this 😉!

Acceptance done! Tested with different users. 👍

retest this please


How to get the estimated geometry?

Hi，thank you for the great work!
I noticed that your data includes ground truth geometry. How can I use monosdf to estimate the geometry of your data? I found https://github.com/Jerrypiglet/monosdf, but I couldn't find some of the files mentioned in the readme, such as convert_mitsubaScene3D_to_monosdf.py and kitchen_hdr_est_mlp.conf.

Hey there, please check here for downloading scene files, and the iccv23 branch: updated README for full instructions on getting DPT depth/normals and training models. Let us know if you have further questions.


Temperature Controller Logic Execution Installs with a 'Untitled 1.vi'

File should be deleted on disk.

Do you have the complete file path for this file? I cant find it in the temperature controller logic folder or project.


Is there a way to add a prefix to the module pathnames?

Search terms
typedoc export directory prefix
Question
My project as a src directory for TypeScript which outputs to lib (pretty standard). When my package is imported, files other than index.js are imported like this:
import {...} from '@my-workplace/my-package/lib/other-stuff/abc';
My typedoc config looks like this
{
  "entryPoints": [
    "src/index.ts",
    "src/other-stuff",
  ],
  "out": "build/typedoc",
  "exclude": [
    "src/**/__tests__/**/*",
    "src/**/__mocks__/**/*"
  ]
}

At the top of the Typedoc page, it says

@my-workplace/package-name / other-stuff/abc /
Module other-stuff/abc

However, I want it to say this

@my-workplace/package-name / lib/other-stuff/abc /
Module lib/other-stuff/abc

with lib/ at the beginning of the top line (the bottom line I could take or leave). I can't find anything in the configuration document that lets me set what path is displayed in the documentation, but is there anyway to do this?

There isn't any option to add this on directly, unless you count --name "@my-workplace/package-name/lib", but TypeDoc does let you override the names of modules individually with the @module tag, so you could put:
/**
 * @module lib/other-stuff/abc
 */

at the top of your entry point.

@Gerrit0 Thanks for the quick reply and sorry for the delay on my end 🙏
I see, I was hoping that I could put something on the page that the person reading the doc could just copy and paste to get the module imported. I just had the idea that it might be possible with a plugin or a custom theme, so I will take a look at the documentation for those.

Another option that might work for you is to include that information in the module comment:
// other-stuff/abc.ts
/**
 * To import this module:
 * ```ts
 * import * as Abc from "@my-workplsace/other-stuff/lib/abc"
 * ```
 * @module
 */


[multistage] support SEMI/ANTI join

optimize for SEMI and ANTI join.

Codecov Report

Merging #10294 (ead72d6) into master (2187a9b) will decrease coverage by 6.25%.
The diff coverage is 90.32%.

@@             Coverage Diff              @@
##             master   #10294      +/-   ##
============================================
- Coverage     70.44%   64.19%   -6.25%
+ Complexity     5824     5806      -18
============================================
  Files          2016     1962      -54
  Lines        109636   107187    -2449
  Branches      16681    16392     -289
============================================
- Hits          77237    68814    -8423
- Misses        26979    33358    +6379
+ Partials       5420     5015     -405

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
67.67% <90.32%> (-0.01%)
:arrow_down:

unittests2
13.68% <0.00%> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

...org/apache/pinot/query/planner/stage/JoinNode.java
87.50% <50.00%> (-7.50%)
:arrow_down:

...pinot/query/runtime/operator/HashJoinOperator.java
95.04% <96.00%> (+0.21%)
:arrow_up:

...not/query/planner/logical/RelToStageConverter.java
85.52% <100.00%> (+0.19%)
:arrow_up:

...va/org/apache/pinot/common/config/NettyConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

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

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

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

...ache/pinot/server/access/AccessControlFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ache/pinot/common/messages/ForceCommitMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/messages/SegmentReloadMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 435 more

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


Compiler produce large output model

Hi!
We have model that cause size problem:

$ edgetpu_compiler -s -a yolo_no_fake_input.tflite
Edge TPU Compiler version 15.0.340273435
Model compiled successfully in 1487 ms.
Input model: yolo_no_fake_input.tflite
Input size: 2.77MiB
Output model: yolo_no_fake_input_edgetpu.tflite
Output size: 31.90MiB
On-chip memory used for caching model parameters: 2.71MiB
On-chip memory remaining for caching model parameters: 5.02MiB
Off-chip memory used for streaming uncached model parameters: 29.07MiB
Number of Edge TPU subgraphs: 1
Total number of operations: 68
Operation log: yolo_no_fake_input_edgetpu.log
Operator                       Count      Status
DEPTHWISE_CONV_2D              17         Mapped to Edge TPU
ADD                            10         Mapped to Edge TPU
RESHAPE                        1          Mapped to Edge TPU
CONV_2D                        36         Mapped to Edge TPU
PAD                            4          Mapped to Edge TPU
I see in hex editor that there is lots of zeros.
The problem still occurs even if we split the model to multiple segments:

$ edgetpu_compiler -s -n 5 yolo_no_fake_input.tflite
Edge TPU Compiler version 15.0.340273435
Warning: For the given model, you're creating more segments than is necessary, which can increase inference latency.We instead recommend num_segments = 1
Model compiled successfully in 372 ms.
Input model: yolo_no_fake_input_segment_0_of_5.tflite
Input size: 664.39KiB
Output model: yolo_no_fake_input_segment_0_of_5_edgetpu.tflite
Output size: 692.59KiB
On-chip memory used for caching model parameters: 629.50KiB
On-chip memory remaining for caching model parameters: 7.11MiB
Off-chip memory used for streaming uncached model parameters: 0.00B
Number of Edge TPU subgraphs: 1
Total number of operations: 46
Operation log: yolo_no_fake_input_segment_0_of_5_edgetpu.log
Operator                       Count      Status
PAD                            3          Mapped to Edge TPU
DEPTHWISE_CONV_2D              12         Mapped to Edge TPU
CONV_2D                        24         Mapped to Edge TPU
ADD                            7          Mapped to Edge TPU
Model compiled successfully in 136 ms.
...
Input model: yolo_no_fake_input_segment_4_of_5.tflite
Input size: 828.06KiB
Output model: yolo_no_fake_input_segment_4_of_5_edgetpu.tflite
Output size: 29.91MiB
On-chip memory used for caching model parameters: 799.50KiB
On-chip memory remaining for caching model parameters: 7.04MiB
Off-chip memory used for streaming uncached model parameters: 29.07MiB
Number of Edge TPU subgraphs: 1
Total number of operations: 4
Operation log: yolo_no_fake_input_segment_4_of_5_edgetpu.log
Operator                       Count      Status
CONV_2D                        3          Mapped to Edge TPU
RESHAPE                        1          Mapped to Edge TPU

The last output segment is always much larger so we decided to remove RESHAPE op just to check if it is causing this problem:

$ edgetpu_compiler yolo_no_reshape.tflite -o 15.0/
Edge TPU Compiler version 15.0.340273435
Model compiled successfully in 795 ms.
Input model: yolo_no_reshape.tflite
Input size: 2.77MiB
Output model: 15.0/yolo_no_reshape_edgetpu.tflite
Output size: 2.81MiB
On-chip memory used for caching model parameters: 2.71MiB
On-chip memory remaining for caching model parameters: 5.02MiB
Off-chip memory used for streaming uncached model parameters: 0.00B
Number of Edge TPU subgraphs: 1
Total number of operations: 67
Operation log: 15.0/yolo_no_reshape_edgetpu.log
See the operation log file for individual operation details.

The problem can be reproduced with this file

@marcin-kamionowski can you give me access to that file?
It seems like it could be related to this bug: https://github.com/google-coral/edgetpu/issues/169
which should be fixed...

@marcin-kamionowski it seems like you commented on that issue too, did you try compiling with 14.x?
if 14.x produces a small model, then it is likely that the bug still exist in some form :/

Just got your model, also tried with 14.x and the model is still abnormally large, running in in our unreleased compiler got the model size down to 4MB. If you attach your whole model, I'll compile it for you, apologies for the issue

@marcin-kamionowski  can you please attach your full model.

@hjonnala:
yolo_no_fake_input.zip

@marcin-kamionowski Thanks for sharing the full tflite model. I don't know the exact date of next release. This issue would be fixed in next release which is very soon.

@marcin-kamionowski producing large output model got resolved with the new release of edgetpu compiler


Multiline keyword arguments indentation is inconsistent between method definitions and calls

I would be happy to work on this as a PR if you're happy with this proposed solution
Is your feature request related to a problem? Please describe.
Two things relating to keyword arguments:

There is inconsistency between method calls and method definitions. Layout/AlignHash is in charge of keyword arguments in method calls, and Layout/AlignParameters is in charge of keyword arguments in method definitions.

To me keyword arguments are more a type of parameter more than a type of hash. I'd like it to be possible to always align them using the rules for Layout/AlignParameters

with default config, rubocop currently thinks this is correct:
def method(position_1, keyword: true,
           keyword_2: true)
end

method(position_1, keyword:   true,
                   keyword_2: true)

Describe the solution you'd like
When Layout/AlignHash: EnforcedLastArgumentHashStyle is ignored, i'd like keyword arguments indentation to instead be controlled by Layout/AlignParameters, and when EnforcedLastArgumentHashStyle is not ignored, I'd like keyword arguments to be controlled by Layout/AlignHash
For example, with the following config
Layout/AlignHash:
  EnforcedLastArgumentHashStyle: always_ignored
Layout/AlignParameters:
  EnforcedStyle: with_fixed_indentation

def method(position_1, keyword: true,
  keyword_2: true)
end
method(position_1, keyword: true,
  keyword_2: true)

Layout/AlignHash:
  EnforcedLastArgumentHashStyle: always_ignored
Layout/AlignParameters:
  EnforcedStyle: with_first_parameter

def method(position_1, keyword: true,
           keyword_2: true)
end
method(position_1, keyword: true,
       keyword_2: true)

with default config
def method(position_1, keyword:   true,
                       keyword_2: true)
end
method(position_1, keyword:   true,
                   keyword_2: true)

I would be happy to work on this as a PR if you're happy with this proposed solution

Thanks for the detailed report. I think it seems sensible. PRs always welcome. 🙇

I'm with the suggestion overall, but it seems to me there should be zero interplay between those two cops. Likely we should just stop AlignHash from operating on implicit hashes.

Likely we should just stop AlignHash from operating on implicit hashes.

I think that might be more intuitive to users as well. 🤔 WDYT @robotdana?

I agree.

change the options for Layout/AlignHash: EnforcedLastArgumentHashStyle.
to: 'ignore' and 'enforce', and only operate on explicit hashes
treat implicit hashes in method calls as just more parameters for Layout/AlignParameters
Layout/AlignParmeters dosen't see inside explicit last argument hashes (it will still align the whole hash according to its rules but not the keys & values)

is it ok to just change the options, with instructions in the changelog, or should I make some effort to migrate the old keys to a new way?
Have i understood what you'd prefer?


Added missing link to ISemanticZoomInformation

@svick,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Feature/3879 add update app banner

Description
From now on, a banner will be displayed whenever the cashed version of the application is outdated. The check is done based on a version file that is generated at each build and will contain the commitHash, the date and the version number from the package.json file. This results in a json file that is being retrieved from the server through an API call. If the local bundled commitHash is not the same, or if the file doesn't exist yet, the banner will be displayed, inviting the user to hard reload his browser.

closes #3879
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Code is clear and sufficiently documented
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ]End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request


Set default service offering over api and ui

Description
This adds the possibility to add/change the defaultUse/unique name of system service offerings over the ui and api.
The create and update command of the system service offerings now have an optional parameter for the default use.
The default use parameter is a boolean value.
When setting a a new service offering or when editing an existing one to be a default one, the unique name and default use value of the offering which currently had it gets set to NULL.
The unique name depends on the system vm type, localstorage and defaultuse values.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots
In the list it gets shown if the service offering is a default one.

You can now set the offering to a default one when creating a new system service offering.

You can set/unset the service offering as a default one in the edit view.

I also added the possibility to create a elastic lb here because it was missing.

How Has This Been Tested?
Deployed this to our test environment and tested functionality.

I opened a new PR because I had some conflicts when trying to push my changes to the old one.
https://github.com/apache/cloudstack/pull/3198

@svenvogel code looks alright but please explain how uniquename and defaultuse are connected.

@svenvogel code looks alright but please explain how uniquename and defaultuse are connected.

@DaanHoogland The unique name is a combination of the "default use", "use local storage" and system vm type of the offering.

@DaanHoogland can you approve?

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2728

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

I would prefer to add a global setting to specify the default service offering for routers.
The service offering will be determined in findServiceOfferingId() in RouterDeploymentDefinition.java and VpcRouterDeploymentDefinition.java
If you do not mind, I can create a PR for it.

Trillian test result (tid-3545)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 24412 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3294-t3545-kvm-centos7.zip
Smoke tests completed. 70 look OK, 0 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

@pbankonier @svenvogel , I agree with @ustcweizhou . It would be better to reduce this PR to only enabling setting the uniquename and to let a global setting point to the default offering. The functionality like this seems clunky. Also we may be changing names against the convention that someone might want to maintain. What are your thoughts?

@DaanHoogland we dont agree. we would prefer this functionality. we dont think its somehow like "clunky". "clunky" was it implemented before. we think it should be easier so set it for all types of system vms. this feature allow to set different type of system vm images. to get this a graphical menu is more user friendly than one value for each type of system vm in global settings. at the moment its a

I would prefer to add a global setting to specify the default service offering for routers.
The service offering will be determined in findServiceOfferingId() in RouterDeploymentDefinition.java and VpcRouterDeploymentDefinition.java
If you do not mind, I can create a PR for it.

what this means its determined in this functions? is there any functionality they breaks this? why it should set this in global setting when all other things for "offering" are set there?

Also we may be changing names against the convention that someone might want to maintain.

@DaanHoogland which function? which convention? please declare it exactly what you mean. at the moment we dont think we break anything.

@DaanHoogland @ustcweizhou The uniquename gets set seperate per every system service offering. I don't understand how you want to realize this with a global setting maybe you can explain this in more detail.
Here are some screens of the affected db tables. In the first one you see the service_offering table and in the second one the disk offerings. I marked the affected columns/entries.

@DaanHoogland we dont agree. we would prefer this functionality. we dont think its somehow like "clunky". "clunky" was it implemented before. we think it should be easier so set it for all types of system vms. this feature allow to set different type of system vm images. to get this a graphical menu is more user friendly than one value for each type of system vm in global settings. at the moment its a

I would prefer to add a global setting to specify the default service offering for routers.
The service offering will be determined in findServiceOfferingId() in RouterDeploymentDefinition.java and VpcRouterDeploymentDefinition.java
If you do not mind, I can create a PR for it.

what this means its determined in this functions? is there any functionality they breaks this? why it should set this in global setting when all other things for "offering" are set there?

@svenvogel
Currently we have global setting consoleproxy.service.offering for CPVM and secstorage.service.offering for SSVM, no code change is needed.
Of course it would be better to change it from global setting to zone setting, so zones can have different offering for systemvms f there are multiple zones created.
We do not have setting for router vms. I prefer to add an account setting (eg router.service.offering" so we can set offering of router globally and also for individual account. it is a simple change similar as consoleproxy.service.offering and secstorage.service.offering
I understand that you have spent some time on this PR and it might be running on your production already. I agree that it is user friendly. Considering It will not break any functionality, this PR is ok for me.

@ustcweizhou can you approve it?

@ustcweizhou @svenvogel @DaanHoogland The motivation behind this was to make it possible to set a default system offering over the ui instead of the way described at the bottom of this page http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.8/service_offerings.html.

@svenvogel @pbankonier - I can see that you've put a lot of work into this, but unless I've missed something, couldn't this be done a bit simpler by populating the service_offering_id for all of the default network offerings as: 9 or 10 (shared/local storage offerings for domainrouter).
Then the network offering would be linked to an actual service offering rather than the weird global settings business.
Migrating from the global settings if people have updated them wouldn't be difficult, the service offerings 9 and 10 would just have to be updated with the values in the global settings.
Editing the material properties of a offering has always been a no-no (with very good reasons) but maybe we can make an exception for the system vm offerings to allow change CPU and RAM values.
// @ustcweizhou

@PaulAngus thanks for info i will discuss this with @pbankonier and come back

@PaulAngus can you make it more clear what you mean with "service_offering_id" 9 and 10?

In a base installation there are domainrouter offerings (which i don't think are actually used)
i believe that they always have IDs 8 and 9

@svenvogel @pbankonier - I can see that you've put a lot of work into this, but unless I've missed something, couldn't this be done a bit simpler by populating the service_offering_id for all of the default network offerings as: 9 or 10 (shared/local storage offerings for domainrouter).

i try to understand this... maybe we dont have them anymore. you mean populating the service_offering_id. what do you mean practically with this? after some changes you will have different ids because you have different ram and cpu sizes each linked to a domainrouter. the only difference is the default use flag.

Migrating from the global settings if people have updated them wouldn't be difficult, the service offerings 9 and 10 would just have to be updated with the values in the global settings.

you mean that these values are always fix? whats with different sizes? whats the difference between globalsettings vs http://docs.cloudstack.apache.org/projects/archived-cloudstack-administration/en/latest/service_offerings.html database hack?

If we actually use the system service offerings 'properly' the global settings 'default' becomes redundant.  We should probably make specifying a service offering for a network offering mandatory.
I guess my idea is to do one 'hack' to end them all and connect network offerings to service offerings properly and then update the API to allow operators to change the service offering of a network offering.
To be honest, i'm also struggling to see why we separate out the offerings wrt vm_type - CPVM, SSVM, VR etc.  Sure maybe name them differently, there should probably be VPC independently too,  but 2G RAM 2vCPU is the same in a CPVM, SSVM or VR.

Yes you are right at the moment we have two way to set default values

global settings
this PR

First of all we introduced this to make it possible to use the UI to change the service offering for the routers not to use this ugly database hack. before you could set it in the global settings and the hack. now you can set in in the UI. we focused on not to break the functionality. there are different tables.
"disk offering" and "service offering" they are connected. there are a "unique_name" and "default_use" ... they seems to be connected.
first we thought to rewrite them but there is a lot of work to do and it would break the existing functionality.
okay here is the plan,

we implement this PR
we and our team will work on all our issues and we will improve and change them step by step accordingly. we remove in a second PR the global settings

it can be under discussion to change/adapt the "network offering" "but" i have to admit  "network offering" the present network offerings meet all recent requirements. if we change them we would create a huge and complex beast. these mean, change the database and the UI. its a lot of work.
at the moment i think the System Offering is the best location to set up values.

the main problem here is the awful connection between the "disk offering" "service offering" because in "Service Offering" there will be saved CPU, RAM and so one...
lets try to work out best solution...

well, since you ask....
My view on that is that a service offering should specify and disk offering. And that disk size should be decoupled from disk offering.  Disk offering performance will probably have to offer performance as either 'per GB' or absolute values ie: 1000 IOPS or 100 IOPS/GB
Maybe an operator would want to specify some preset sizes, which could then optionally be included in a service offering.

@PaulAngus just so I understand it correctly...

add the disk offering from for domain router, CPVM, SSVM ... to the service offering table. right?
remove them, after unused from disk offering?

is that correct?
i need to add that if you have a managed storage the VPC, SSVM, CPVM will be created with...

i dont know if this is hard coded...

@pbankonier and @svenvogel thanks for the PR and your valuable contributions. I have read the discussions, but still, have a question here.
What does this change mean for the current method we have to override/change the default service offerings for system VMs? I mean, currently we have the following configurations available:

consoleproxy.service.offering
internallbvm.service.offering
secstorage.service.offering

I do understand that you are introduced a method to change these default for other types of VMs, such as VRs, but still, how will this play out for people that are already used to use the current global settings parameters to configure the service offering for console proxy, ssvm, and internal LB VM?
I do like the UI change you are proposing, but I see that it can be implemented with the same backend we have. Of course, you are now supporting the override for other types of system VMs as well, but I guess that we could just use the "standard" we have currently in place.
Now, if the proposal is to change this "standard", then we need to create/propose a migration path for people.

@rafaelweingartner thanks for the comment.
Yes, the proposal is to change this "standard" and add a to create/propose a migration path for people who use the default settings.

@svenvogel can you describe how this migration path would work for people that are currently using the "configurations" table when they start to use the code introduced by this PR?

@svenvogel @DennisKonrad @rafaelweingartner @PaulAngus looked into the code for the global default offering settings and found that the only place where this global config variables are used is in the ManagerImpls for the different offering types.
The global settings are just used to check if there is already an offering with the given id existing in the database. There is no code which sets an offering to a default one with this configs. So maybe this was implemented to prevent that a new default offering was created at the start of the application before the uniquenames were introduced.
There is no need for a migration path with this PR. There will be no changes which will break existing functionality.
After my analysis the global default settings do nothing in the current state of the software and we can remove them.

@rafaelweingartner ?

Ping @rafaelweingartner @ustcweizhou @PaulAngus - any comments?

@pbankonier what happens if I changed the value of ConsoleProxyServiceOffering configuration? How will the new service offering I can set there work with your proposal? Will it be used? Will this configuration still be available in the configuration view?

Ping?

@blueorangutan package

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Looks like there are still on-going discussions and disagreement around the approach. I've moved this to the next major milestone <IP_ADDRESS> now.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-113

@rafaelweingartner As far as I saw the configuration in the global settings can and should be removed because it does not affect anything. Currently the uniquename gets used to determine which offering should be used as the default one.

@rhtyd closed in favor of cloudstack primate development https://github.com/apache/cloudstack-primate/


Upgraded to microprofile-openapi 1.1.1 (along with the SmallRye impl)

Signed-off-by: Eric Wittmann<EMAIL_ADDRESS>This PR addresses the following issue:  https://issues.jboss.org/browse/THORN-2260

[x] Have you followed the guidelines in our Contributing document?
[x] [v2] Have you created a JIRA and used it in the commit message?
[x] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[ ] Have you built the project locally prior to submission with mvn clean install?

I have been able to build locally, but only with -DskipTests.  When I try to run the MicroProfile OpenAPI TCK, I get a number of stack traces related to Arquillian failing to delete temp files (during undeployment).  The first few TCK tests pass, but fairly quickly I start getting the following error:
org.jboss.msc.service.StartException in service org.wildfly.undertow.listener.default: Address already in use: bind /<IP_ADDRESS>:8080

I have never been able to successfully run the TCK tests locally, so I'm hoping that the CI build is OK.  If not, I'll have to address any issues found in CI as separate commits.

The build on semaphoreci is failing, but it doesn't seem related to my changes?  It's hard for me to tell:
[ERROR] Plugin io.thorntail:thorntail-maven-plugin:2.3.1.Final-SNAPSHOT or one of its dependencies could not be resolved: Could not find artifact io.thorntail:thorntail-maven-plugin:jar:2.3.1.Final-SNAPSHOT -> [Help 1]
```

You can ignore SemaphoreCI, that's only for the v4 branch.

I don't know why, but there's one test in the MP OpenAPI TCK which didn't shutdown the running Thorntail instance, and all the subsequent ones failed with "Address already in use".

I tried running the microprofile-tck/openapi locally for this PR and got some failures: https://gist.github.com/michalszynkiewicz/835043b5d4d5e3c17d1bce285dd2b7d1

The full output log for this can be found here:
https://gist.github.com/EricWittmann/1766ff930d0fba3ad0ab82dd7d3f24c6
I honestly don't know how to debug this.  Anyone have any ideas?

retest this please

Hi, will have a quick look at this PR as well, thanks

Thanks for looking at this!  I don't know why the thorntail instance doesn't get properly shut down in one of the tests.

Hi @EricWittmann, np at all.
OASConfigWebInfTest causes the problems. It is the only test which extends another test class OASConfigScanClassTest with its own Deployment.
If I change OASConfigWebInfTest to extend from AppTestBase and copy a test from OASConfigScanClassTest then OASConfigWebInfTest causes no problems.
Can you please confirm as well ?
It look like Thorntail can not handle a test suite which has 2 Deployments. @Ladicek, @mkouba, is it correct ?
If yes then the fix is to update OASConfigWebInfTest in the MP OpenApi and may be as well have  OASConfigWebInfTest copied to the Thorntail microprofile test suite and updated as suggested above.

I don't recall seeing any test that would have two @Deployments. Isn't Arquillian supposed to ignore the @Deployment from ancestor class, if the @Deployment in subclass is on a method with the same name? IDK, but I think I've read something like that somewhere...

The 2nd Deployment has an 'order' property, may be this is why the one in the super class is not ignored

I don't really know. But I can imagine two deployments confuse our Arquillian adapter a lot.

I am going to try to get this fixed on the Microprofile-OpenAPI side by updating the TCK.  Hopefully I can get that fixed there within the week.  So please hold off on any other changes until we see if I can get that done. :)

For tracking/reference purposes:  https://github.com/eclipse/microprofile-open-api/pull/335

Update: I have fixed the upstream TCK tests to no longer have multiple deployments.  And I've rebuilt that (and smallrye) locally and then run the thorntail MP+OpenAPI TCK suite.  All greeeeeeen!

I'm submitting a SmallRye PR to update versions there.  Then we can do a SmallRye OpenAPI release and then update this PR and everything should work.
Thanks @sberyozkin for figuring out what was wrong!

Hi @EricWittmann, that is a super fast fix, np at all, thanks

retest this please

This scared me a little, but no, it's actually the usual disk space issue :-) But that scares me anyway, because I did a disk space cleanup of all Windows build agents just last week...

retest this please

So this can be merged, right?

Yes, merging now


Way to minify all bundle js files?

Hi,
I’m looking for a way to minify all bunfle js files on build. I couldn't figure what I’m missing here.
Here’s an example of what I tried (here with babili):
build: {
	extend (config, { isDev, isClient }) {
		if (!isDev) {
			config.devtool = false
			config.plugins.push(
				new BabiliPlugin({
					// options here
				}, {
					comments: false
				})
			)
		}
	}
}

The build is correctly made, but has no effect on the bundles files. Another try with uglifyjs:
build: {
	extend (config, { isDev, isClient }) {
		if (!isDev) {
			config.devtool = false
			config.plugins.push(
				new webpack.optimize.UglifyJsPlugin({
					parse: {
						strict: true
					},
					compress: {
						unsafe: true,
						warnings: false,
						drop_console:true
					},
					output: {
						comments: false,
						ascii_only: true
					},
					mangle: true
				})
			)
		}
	}
}

This is globally the last thing I need to get a good grade on PageSpeed: https://gtmetrix.com/reports/dev.emmanuelbeziat.com/OpP40maF
If anybody has a working example, it would surely help :) Thanks!

@EmmanuelBeziat  HI , I don't change any setting with nuxt.js default
and seems like my js already minify ...

Hi @ausir0726
Strange thing… Is this because you’re using SSR and I use the static build?

this link is I am use nuxt generate static site ...

I'm sure default when nuxt production have UglifyJs ...
why you don't ?

Well, I removed node_modules and reinstalled everything, and updated to nuxt 10.6, and it works now…
Thanks! :)

double A
so cool .
but , how can let google speed test get 100 point ?! @@ I never can't touch that ...

I only hit 100 when I had a good old WordPress. Now I'm more around 96-99.
Most of time, it's about server configuration; I have a dedicated server that allow me to make the customizations I need.


Extra 'File Shares/Queues' nodes display for one 'Name & Key' attached blob storage account

Storage Explorer Version: 1.14.0-dev
Build: 20200527.5
Branch: master
Platform/OS: Windows 10/ macOS Mojave
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Select one Blob storage account -> Attach it via name & key.
Expand the attached storage account -> View the result.

Expect Experience:
Only display 'Blob Containers & Tables' nodes.
Actual Experience:
'File Shares & Queues' nodes also display.

More Info:
This issue also reproduces for one premium account.

Won't fix. We cannot call ARM if all we have is a key, therefor we don't know the attached account is a blob account.


Discovery fails when data is None

Fix is_graphql function to better handle anomalies in responses

https://github.com/dolevf/graphql-cop/pull/14


Incorrect default value for markdown.sanitize

In the library itself, markdown.sanitize defaults to true, whereas the documentation indicates that it defaults to false.
I wasn't sure whether the implementation in the code is wrong, or whether it was the documentation so I've not submitted a PR.
Thanks!

Thanks will be checked/updated with next minor release.


MB-5734 | formalize already existing dev comments in annotation

Description
This linter finding was already sort of annotated in code, so I added the rest of the expected format. :)
Reviewer Notes
How is the phrasing? Do we want any more detail here? Is the referenced PR actually indicated instead?
References

PR referred to in comments
Jira story

Warnings

:warning:
It looks like you are attempting to bypass a linter rule, which is not a sustainable solution to meet
      security compliance rules. Please remove the bypass code and address the underlying issue. cc: @transcom/Truss-Pamplemoose

Messages

:book:
:link: MB-5734

  Generated by :no_entry_sign: dangerJS against a060d77d433be309f4400d808c70b85231f9394b

from conversation in slack with @jneuner-mitre:
annotation is not sufficient for this case. it leaves us open to multiple attack vectors. because of this, we will prioritize working with relevant folks to get the reverse of https://github.com/transcom/mymove/pull/5039 merged instead.


Problem: test ackermann33 fails

We'll need to have a closer look into this test in file vmPerformanceTest.json.
The test does not contain "out", so gaslighter expects it to fail. However, spunikvm runs it succeeded without any problem. The full code for this test is:
0x60e060020a6000350480632839e92814601e57806361047ff414603457005b602a6004356024356047565b8060005260206000f35b603d6004356099565b8060005260206000f35b600082600014605457605e565b8160010190506093565b81600014606957607b565b60756001840360016047565b90506093565b609060018403608c85600186036047565b6047565b90505b92915050565b6000816000148060a95750816001145b60b05760b7565b81905060cf565b60c1600283036099565b60cb600184036099565b0190505b91905056

This program might be better debugged by using EDB's fire debug. The current return value is:
[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 61]

We'll need to figure out what this program actually do and then check whether this return value is correct or not. Or whether it should actually return an error.

The tests arith and boolean are of the same problem -- they don't contain "out", while runs successfully.

out is for chaining vm executions together sub-vm calls, so to say. If the objective is to just test whether the expected output of storage is equal to the actual output of storage then that should be enough information qualify. Surely there's no need for return_values (aka out) in the situation of arith and boolean tests?

But most of the tests, even if the return value is 0x0, would contain it ("out": "0x"). Tests meant to fail indeed do not contain them.
These three are the exceptions I found that a) execute on the VM successfully, and b) no "out" column.

This problem seems to be fixed after integrating gas cost check.


Switch: Add loading state

Switch: Add loading state

Above: Example shows the loading state, which is resolved after 1 second. The parent component is handling the state switching for the Switch.
This update adds a new "loading" state for the Switch. The loading state
can be triggered by passing isLoading = true as a prop. The UI adjusts
to a loading animation. During this state, clicks/changes are blocked, until
isLoading is set back to false.

Coverage remained the same at 100.0% when pulling bfe4f4fc1a10f7877cd940cf9b6ef2b6353ed661 on switch-loading into a7054e94cbdb8706f1555f91b8cbfa456c0aac71 on master.


sPRM

Program a sPRM that will guide the robot to its goal.

[x] Use uniform sampling of the configuration space.
[x] For finding nearest neighbors use brute force.
[x] For the metric use the Euclidean metric.
[x] For finding the shortest path use a Djkstra (Use a lib, do not program it on your own)

Graph Doku:
https://pypi.org/project/Dijkstar/

[ ] add collision check before adding edge to graph.

[ ] find appropriate number of nodes and minimal distance.


Update README.md

Fix some typos

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


Removed docs for unsupported versions of azps

PR Summary
Removed docs for unsupported versions of AzPS.
PR Checklist

[x] Descriptive Title: This PR's title is a synopsis of the changes it proposes.
[x] Summary: This PR's summary describes the scope and intent of the change.
[x] Contributor's Guide: I have read the contributors guide.
[x] Style: This PR adheres to the style guide.

Learn Build status updates of commit 2c3463b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azps-8.2.0/Az.Accounts/Add-AzEnvironment.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Az.Accounts.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Clear-AzConfig.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Clear-AzContext.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Clear-AzDefault.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Connect-AzAccount.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Disable-AzContextAutosave.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Disable-AzDataCollection.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Disable-AzureRmAlias.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Disconnect-AzAccount.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Enable-AzContextAutosave.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Enable-AzDataCollection.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Enable-AzureRmAlias.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Export-AzConfig.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Get-AzAccessToken.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Get-AzConfig.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Get-AzContext.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Get-AzContextAutosaveSetting.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Get-AzDefault.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Get-AzEnvironment.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Get-AzSubscription.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Get-AzTenant.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Import-AzConfig.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Import-AzContext.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

azps-8.2.0/Az.Accounts/Invoke-AzRestMethod.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Does not build on latest SBCL

I get this:
;   (COERCE ORG.MIDDLEANGLE.FOREIGN-NUMERIC-VECTOR::VAL '(SIGNED-BYTE 32))
;
; caught WARNING:
;   Cannot coerce (OR (NOT INTEGER) (INTEGER * -2147483649) (INTEGER<PHONE_NUMBER>)) to (SIGNED-BYTE 32)

Longer log is here: http://report.quicklisp.org/2019-03-19/failure-report/fnv.html#org.middleangle.foreign-numeric-vector
This breaks lisp-matrix, too.

Oh no.  Time to crawl out from under a log and hack... Thanks.
On Wed, 20 Mar 2019, 00:29 Zach Beane<EMAIL_ADDRESS>wrote:

I get this:
;   (COERCE ORG.MIDDLEANGLE.FOREIGN-NUMERIC-VECTOR::VAL '(SIGNED-BYTE 32))
;
; caught WARNING:
;   Cannot coerce (OR (NOT INTEGER) (INTEGER * -2147483649) (INTEGER<PHONE_NUMBER>)) to (SIGNED-BYTE 32)
Longer log is here:
http://report.quicklisp.org/2019-03-19/failure-report/fnv.html#org.middleangle.foreign-numeric-vector
This breaks lisp-matrix, too.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/blindglobe/fnv/issues/4, or mute the thread
https://github.com/notifications/unsubscribe-auth/AABb3rs_wiEVWeeA4NEBkhrzue1No1Ecks5vYXLwgaJpZM4b9Uj0
.

Any update on this?
On Wed, Mar 20, 2019 at 12:06 AM Tony Rossini<EMAIL_ADDRESS>wrote:

Oh no. Time to crawl out from under a log and hack... Thanks.
On Wed, 20 Mar 2019, 00:29 Zach Beane<EMAIL_ADDRESS>wrote:

I get this:
; (COERCE ORG.MIDDLEANGLE.FOREIGN-NUMERIC-VECTOR::VAL '(SIGNED-BYTE 32))
;
; caught WARNING:
; Cannot coerce (OR (NOT INTEGER) (INTEGER * -2147483649) (INTEGER
2147483648)) to (SIGNED-BYTE 32)
Longer log is here:

http://report.quicklisp.org/2019-03-19/failure-report/fnv.html#org.middleangle.foreign-numeric-vector

This breaks lisp-matrix, too.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/blindglobe/fnv/issues/4, or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AABb3rs_wiEVWeeA4NEBkhrzue1No1Ecks5vYXLwgaJpZM4b9Uj0
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/blindglobe/fnv/issues/4#issuecomment-474676557, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAT2rR1jMCuDlHguUASxYCKEchsbXXUCks5vYbOngaJpZM4b9Uj0
.


v12.14.10 is breaking existing projects

There seems to be problem with TypeScript & @nuxt/types.
Versions

nuxt: v12.14.10
node: v12 / v14

Reproduction
https://github.com/jankal/jankal.dev/pull/20
See different failing checks (building application) in the pull-request.
Steps to reproduce
Upgrade TypeScript project from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>What is Expected?
Application builds like usual.
What is actually happening?
TypeScript issues a type-error concerning nuxt's Context not having properties merged in via declaration merging.
I've not yet finished investigation. But I think that the Context type broke somehow.
Ping @pi0: This might be a problem with some upstream dependency upgraded by renovate.

TS errors:
TS2339: Property '$content' does not exist on type 'Context'.

Type 'import("/app/node_modules/@nuxt/content/node_modules/@nuxt/types/app/index").Context' is not assignable to type 'import("/app/node_modules/@nuxt/types/app/index").Context'.

Type 'Context' is missing the following properties from type 'Context': $colorMode, $axios

I don't know why... but my investigations showed that renovate did build a corrupted dependency tree.

Hi. It was probably NPM cdn propagation.


Document that the white channel is not used

I'm fairly sure that the line of code which sets the signal for any LED is here: https://github.com/awawa-dev/HyperSerialWLED/blob/24d825fef57e7d6a8337eca5d3e4a9967b0c7cd6/wled00/wled_serial.cpp#L105
If I understood correctly, this means that the white channel is always unused on RGBW LED strips, which is in contrast to e.g. https://github.com/awawa-dev/HyperSerialEsp8266.
As far as I'm aware, even though both WLED and HyperHDR (and forks) individually have support for RGBW, it's not possible to use the white channel when using both. See https://github.com/hyperion-project/hyperion.ng/issues/446. I guess this itself isn't an issue per say, as HyperSerialEsp8266 is preferred over HyperSerialWLED anyway, but it should probably be documented.

Hi
It's implemented the same as in WLED:
https://github.com/Aircoookie/WLED/blob/f922268af7fba7b035d5e8c1ca707ecce13ffd3f/wled00/wled_serial.cpp#L174
If they change it then I will consider to implemented it also. As you noticed I consider WLED useful only as a very helpful tool for testing the setup, not necessarily as the final driver so won't extend the documentation about how it's implemented in WLED (but if you were you i would first make sure setRealtimePixel somehow doesn't update it to RGBW as this offers a WLED interface if enabled...you can ask its developers as it is beyond the scope of this fork)
Regards
Awawa


go: Better handle solidity bindings

We would like to address the following issues:

Ensure that the compiler version is deterministic and locked

We do this with solidity pragmas for version 0.8.10, but should be better

We want a CI check that ensures that if a contract changes, the go bindings also change
We may want to restructure where contract bindings are located / how they are built

The output oracle contract bindings are split from the opnode contract bindings requiring two makefile invocations
The opnode contracts are built in a specific folder. That makefile should probably be lifted to the main makefile (or we should add recursive make)

The bindings have a separate go module now, with a single makefile, which is checked in CI to be equal to the bindings that are generated from the latest contracts: https://github.com/ethereum-optimism/optimistic-specs/pull/446
Aside from compiler-version locking I think this issue can be closed.

The compiler version is locked by foundry, it's the version of abigen that is not locked. Currently we use latest, as the check in ci uses the master branch of geth to install abigen


How to get default YouTube but disable annotations and auto-play?

I just want 2 features from this extension - disable annotations and autoplay. Is there a way to set all of the settings to the YouTube defaults so that I can just have those 2 settings?

As I mentioned in #2055, I have been using Magic Actions for YouTube since it's got a lot less default customization and it's pretty simple to turn off everything except for disable annotations and disable autoplay. I tried using YouTube Center again today, but there are still so many default customizations - I am not sure how to toggle them all so that I just get the 2 features.

Necroposting to the extreme.
I recommend you check out Youtube+.
It's similar to YTC(which is kind of dead) but it's actually being updated regularly and is alot better than Magic Actions imho.


create shellcheck github action

fix #855

Current issue with this PR:
ludeeus/action-shellcheck@master is not allowed to be used in helidon-io/helidon-build-tools.  Actions in this workflow must be: within a repository that belongs to your Enterprise account, created by GitHub, or matching the following: docker/*, gradle/wrapper-validation-action*<EMAIL_ADDRESS>ruby/setup-ruby*, graalvm/*, ossf/scorecard-action*, slsa-framework/slsa-github-generator*.


Outside color wheel does not trigger color selection when on white/black or any shade of grey

A bit of a strange issue - clicking on the color wheel does not trigger the correct color selection (onColorChanged) when the currently selected color is black/white or any shade of grey. The center square box does change accordingly though.
If any other color is already selected, then clicking the color wheel does trigger color selection correctly.
This can be reproduced on the web version:
https://rydmike.com/flexcolorpicker/

switch on color wheel
choose a any shade of grey or black or white
click around on the color circle only
notice the "start", "change" and "end" stay the same white/black/grey color

Here is a recording of my own app. The "selected color preview" is set by setState with the onColorChanged:
https://github.com/rydmike/flex_color_picker/assets/869261/71b013fb-36a0-4ee7-b316-40554aa8f915

Hi @adamkoch, what you are seeing is correct and expected behavior.
If you set color to a pure grey scale value (anything on the left side side), or white (upper left corner), or black, entire bottom side, your color selection has no chroma (no color in it). Thus when you change the hue (color on the color wheel), it has no impact on the produced RGB color value, since there is no color in the resulting RGB color value and the RGB color value is the same regardless of what Hue you use. As soon as you move away a bit from these edges, the RGB value will change.