Allow setting a message max size

Feature Request
Motivation
See https://github.com/tensorflow/serving/issues/1382, if the message size is large, tonic currently does not support changing the max message size.

Proposal
Expose the API the change allowed message size.

@L1AN0 @LucioFranco What is the maximum message size allowed now? As far as I understand, gRPC's default max size in spec is 4MB but tonic server seems to be capable of receive larger requests.

I believe there is a cap with h2, its a bug. related issue is here for that @akiradeveloper https://github.com/hyperium/tonic/issues/352

@LucioFranco Thanks for you reply but my question is why tonic doesn't have an upper limit as official implementations do
https://github.com/grpc/grpc/releases/tag/v1.29.0

A significant change was made in #22575 where gRPC Core will check whether the decompressed message fits in the configured limit for the maximum message length that the channel can receive. (The default limit is 4MB.)

Is that your intention not to comply with the official behaviors?

@LucioFranco Thanks for you reply but my question is why tonic doesn't have an upper limit as official implementations do

I don't believe this is specified in the protocol definition so it was never added initially. Would be happy to add it if people want this behavior.

Is that your intention not to comply with the official behaviors?

Just because gRPC core does it, doesn't mean its "official" per-say. Each implemenation of gRPC has its own niche things. I would love to add this but I don't have the time currently. I would totally accept a PR though.

It seems this was fixed by https://github.com/hyperium/tonic/pull/529. Just re-open if that is not the case.

Yes, I believe the C++ implementation has a non-h2 limit as well. We could add this via Grpc types.


Include an indication to which image is referred to

Thought this would be an useful feature to pull our image source along with alt tag.

On review, I can see presenting the alts[] as a nested array of results throws the metric analysis into thinking it's correct when it's not. I'm unsure how to fix this at present.

I don't think this will be good solution as it will change the output format, so it could break some applications implementing it that way. I think it will be much better to change the hard-coded Parser to allow to be injected, so everyone can implement own Parser. Then in your case you can simply extend the existing Parser overwriting only that one method. What do you think?

@csoutham, it is now rebuilt in such a way that you can use your own parser, as mentioned above. Please check the example here: https://github.com/grgk/seo-analyzer/blob/master/examples/analyze_file_with_custom_parser.php

Nice one thanks, that's a really nice feature :)


Updating documents purges attachments.

When one .Reads a document into an object, and .Writes the document back into CouchDB with this library the _attachments get purged since one can't add _attachments to the document struct as it would not start with a capital letter and thus not be exposed.
Maybe I am mistaken, but I couldn't figure this out yet. It would be amazing if somebody could point me in some direction. Thanks!

As it usually goes, I found the solution shortly after putting the problem in words :)
One can put something like json:"_attachments,omitempty" after a struct member, which tells the json decoder to unpack the _attachments part of the json into this member.
Sorry for the issue ;)

No problem.


Error on cart checkout.

Program loads perfectly, logs in, navigates to page and locates item. When the item is in stock, program appears to close current browser and open a new one to begin check out process. On this step I've found 3 different browser scenarios so far with the same error in program.
Scenario #1: Gets to check out window with in-store/curbside pickup option and stops
Scenario #2: Hangs on checkout with pick up bullet point selected
Scenario #3: Hangs on checkout with shipping bullet point selected
CMD prompt output:
Availablility tracking has started!
Current funds: $1,000.00

0it [00:00, ?it/s]AL lib: (EE) SetChannelMap: Failed to match front-center channel (2) in channel map
1it [00:01,  1.41s/it]

18:18:53 - "Pokémon - Pokemon TC..." - https://www.bestbuy.com/site/pokemon-pokemon-tcg-first-partner-collectors-binder/6450052.p?skuId=6450052

2it [00:01,  1.30it/s]Attempting safe exit for <SpawnProcess name='SpawnPoolWorker-2' parent=18900 started daemon>
3it [00:42, 18.89s/it]

4it [00:46, 11.61s/it]
Successfully closed all processes!

Hope this helps :)

Does your Best Buy account have your credit card info already entered and your location set?

The hanging on checkout seems to be another selector issue that I can quickly fix, but getting stuck on the in-store/curbside pickup option is something that isn't accounted for. If this happens even with all account info filled out I would have to add a check for that.

Sorry for delay. Yes, card info is entered and set as default as well as default address.

Do you mind sending a screenshot and blacking out sensitive details, so I can attempt to reproduce the error on my end? Thanks!


Roadmap

[ ] try to reduce time for generating documentation from RAML file
[ ] rewrite code using ES2015 + Babel
[ ] prepare website describing how to use ramlo - simple website like API Workbench
[ ] create a logo for ramlo
[ ] prepare a series of articles describing how API First approach can improve quality of the RESTful API implementation

@kamilzasada what is the expected ETA?

@sichvoge probably around end of August

Awesome! Let me know if you want some PR. I will watch to the end of August for any updates ;)

Hi,
is this project not active anymore ?

@ulaBLUE because of personal matters I don't have time now to actively develop the project, but I plan to get back to it soon.

Thank you for the clarification.


🐛 Update vscode debugger program location

fixes #17523

Updated dev.js file location in launch.json from .github/ to .github/scripts/

🤖 Generated by Copilot at a84c5e7
Updated VS Code launch configurations to match new dev script location. This improves the developer experience and testing workflow.

This PR should not be needed because this is already fixed on the latest main: https://github.com/TryGhost/Ghost/blob/main/.vscode/launch.json


Tele 5: Unable to extract jwplatform id

When I try to download a movie from Tele 5 (e.g. https://tele5.de/mediathek/american-psycho/) I get the following error:
youtube-dl --verbose https://tele5.de/mediathek/american-psycho/
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://tele5.de/mediathek/american-psycho/']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.8.10 (CPython) - Linux-4.19.128-microsoft-standard-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (DE) as X-Forwarded-For.
[Tele5] american-psycho: Downloading webpage
ERROR: Unable to extract jwplatform id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
return func(self, *args, **kwargs)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
ie_result = self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/tele5.py", line 102, in _real_extract
jwplatform_id = extract_id(JWPLATFORM_ID_RE, 'jwplatform id')
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/tele5.py", line 91, in extract_id
return self._html_search_regex(
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1021, in _html_search_regex
res = self._search_regex(pattern, string, name, default, fatal, flags, group)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1012, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract jwplatform id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Regional lockout:

Error Code: access.denied.geoblocked

Where do you see this? Any workaround?

Duplicate of #30499, pending PR #30532.

Only for completion:
https://tele5.de/mediathek/american-psycho/
Is not protected with DRM. I could download it with
https://github.com/dirkf/youtube-dl/tree/f-tele5-discofy-Patch
.. but other films are protected with DRM.

OK, if I got it right there is a patch for this issues on a separate branch. Who decides if it will be merged to master? When does this normally happy? And what do I have to do to test the patch? I have to checkout the branch, build the application and istall it? I am running youtube-dl in Windows' WSL.

So the PR has been merged to master. When will there be a new release so 'youtub-dl -U' will update the installation?


Rancher agent logs websocket: bad handshake when Rancher is deployed/running on GKE

What kind of request is this:
bug
Steps to reproduce:

Install rancher v2 using the latest chart (2018.10.2), using secret for ingress.tls.source (my own cert-manager let's encrypt certificate)
Import a cluster

Result:
Imported cluster stuck in "Waiting for API"
Catle node agent logs:
time="2018-10-25T04:18:15Z" level=info msg="Connecting to wss://xxxxx/v3/connect with token qb8rgrcwfdr2kqj78jkbrb677sp4pctw5k95gq546z2z4jnzjvds5g"
time="2018-10-25T04:18:15Z" level=info msg="Connecting to proxy" url="wss://xxxxxx/v3/connect"
time="2018-10-25T04:18:16Z" level=error msg="Failed to connect to proxy" error="websocket: bad handshake"
time="2018-10-25T04:18:16Z" level=error msg="Failed to connect to proxy" error="websocket: bad handshake"

Firewall is open for 443
I am able to curl the CATTLE_SEVER host
I am NOT able to wget the host

 wget -O- https://xxxx
--2018-10-25 22:41:41--  https://xxxx
Resolving xxxx
Connecting to xxxxx|:443... connected.
ERROR: cannot verify xxxxx certificate, issued by 'CN=Let\'s Encrypt Authority X3,O=Let\'s Encrypt,C=US':
  Unable to locally verify the issuer's authority.
To connect to xxxxxx insecurely, use `--no-check-certificate'.

I am able to wget the host from stock ubuntu:latest image

Is the cattle-node-agent image missing some CA's for let'sencrypt ?

The CA's are present, the wget is probably failing because the SSL_CERT_DIR is set to /etc/kubernetes/ssl/certs for the retrieved cacerts (if needed, not in case of recognized CA signed certificates). Curl working also indicates the chain is valid, also Golang will print x509: certificate signed by unknown authority when the chain is not valid as well.
What cluster are you importing and what is the network path between agent and rancher?

@superseb  I've been trying to import a number of GKE's into rancher v2 also running on it's own GKE. All GKEs are in their own project and the rancher server is public accessible....

I can see that the annotations of the chart Ingress is correctly setting:
			proxy_connect_timeout                   30s;
			proxy_send_timeout                      1800s;
			proxy_read_timeout                      1800s;

According to https://cloud.google.com/load-balancing/docs/https/#timeouts_and_retries to support Websockets on GCP LBs the keepalive_timeout needs to be increased to  620s from default 72s
This is unfortunately hard-coded in the ingress controller:
https://github.com/kubernetes/ingress-nginx/blob/master/rootfs/etc/nginx/template/nginx.tmpl#L136
So I'm out of ideas on how to configure my GCP tcp lb to support websockets.

Furthermore, the nginx ingress controller logs is showing just HTTP 200
x.x.x.x - [x.x.x.x] - - [30/Oct/2018:08:31:13 +0000] "GET /v3/connect HTTP/1.1" 200 60 "-" "-" 3974 0.028 [rancher-ops-rancher-80] <IP_ADDRESS>:80 60 0.028 200
x.x.x.x - [x.x.x.x] - - [30/Oct/2018:08:31:15 +0000] "GET /v3/connect HTTP/1.1" 200 60 "-" "-" 3978 0.002 [rancher-ops-rancher-80] <IP_ADDRESS>:80 60 0.002 200
x.x.x.x - [x.x.x.x] - - [30/Oct/2018:08:31:20 +0000] "GET /v3/connect HTTP/1.1" 200 60 "-" "-" 3978 0.002 [rancher-ops-rancher-80] <IP_ADDRESS>:80 60 0.002 200
x.x.x.x - [x.x.x.x] - - [30/Oct/2018:08:31:23 +0000] "GET /v3/connect HTTP/1.1" 200 60 "-" "-" 3974 0.004 [rancher-ops-rancher-80] <IP_ADDRESS>:80 60 0.003 200
x.x.x.x - [x.x.x.x] - - [30/Oct/2018:08:31:25 +0000] "GET /v3/connect HTTP/1.1" 200 60 "-" "-" 3978 0.004 [rancher-ops-rancher-80] <IP_ADDRESS>:80 60 0.003 200
x.x.x.x - [x.x.x.x] - - [30/Oct/2018:08:31:30 +0000] "GET /v3/connect HTTP/1.1" 200 60 "-" "-" 3978 0.037 [rancher-ops-rancher-80] <IP_ADDRESS>:80 60 0.038 200
x.x.x.x - [x.x.x.x] - - [30/Oct/2018:08:31:33 +0000] "GET /v3/connect HTTP/1.1" 200 60 "-" "-" 3974 0.003 [rancher-ops-rancher-80] <IP_ADDRESS>:80 60 0.004 200
x.x.x.x - [x.x.x.x] - - [30/Oct/2018:08:31:35 +0000] "GET /v3/connect HTTP/1.1" 200 60 "-" "-" 3978 0.033 [rancher-ops-rancher-80] <IP_ADDRESS>:80 60 0.034 200
x.x.x.x - [x.x.x.x] - - [30/Oct/2018:08:31:40 +0000] "GET /v3/connect HTTP/1.1" 200 60 "-" "-" 3978 0.003 [rancher-ops-rancher-80] <IP_ADDRESS>:80 60 0.003 200

I ran the following setup:

Created 2 GKE clusters version 1.10.7-gke.6 (one for Rancher with GKE ingress disabled, one to import)
Setup Helm using https://rancher.com/docs/rancher/v2.x/en/installation/ha/helm-init/ against Rancher GKE
Installed NGINX ingress controller by using helm install --name nginx stable/nginx-ingress --set controller.service.externalTrafficPolicy=Local
Retrieved IP for created LB in front of NGINX ingress controller (kubectl get svc)
Pointed FQDN to that IP in DNS
Installed Rancher using https://rancher.com/docs/rancher/v2.x/en/installation/ha/helm-rancher/ with default self signed certificates
Login to the UI, set server-url to FQDN
Add Cluster -> Import
Run generated commands on 2nd GKE cluster

If you can mention anything that deviates from these steps, I can use it to reproduce. You can also run these steps and see if you run into the same or not.

I can confirm that the above works with fresh GKEs.
Let me do some more digging with my Shared VPC GKEs.

Fixed by recreating the Rancher GKE cluster from scratch.
@superseb I think it's because at some point I changed the IP of my rancher endpoint from private to public IP and that CATTLE_HOST may have been cached somewhere in rancher server ?


Bun section

Description
https://bun.sh

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


use remote chart in draft pack

One may want to maintain a helm chart separately from an application but have developers use that chart which is production-like in their dev workflow.
Currently, draft create copies over a chart from a draft pack for the developer to use. Instead of having the chart actually live in the draft pack, what if we could just specify where the chart lives (like the git repo url). That way, draft create could just pull down the chart from the specified git repo in the correct place.
This would help make it possible to manage the helm chart separately from the application but also allow the developer to use a chart close to production.
inspired by @intheclouddan who currently has a workaround which involves: 1. cloning the git repo with a chart in it 2. symlinking it the charts/ dir in the app directory so that draft will use it

Some things to consider:

where in the draft pack itself do we want to specify the location of the chart? maybe we want to specify a draft.toml file that has a field called chart... This can be copied over and then filled out more competely during draft create
how do we go about fetching updates to chart in a remote location? should re-running draft create fetch the latest updates to the remote chart?

Another thing to keep in mind here is the chart needs to have some specific labels and annotations in order to be picked up by Draft.
I'm not sure we want to impose people maintaining production charts to add those labels and annotations. The question is whether we can inject those key + value pairs from the Helm client.
If not, the remote chart will need to include them.

@michelleN, @InTheCloudDan - once #771 gets merged, you will be able to specify the directory of your charts, eliminating the need to symlink.
Getting back to remote charts, I think the restriction to include a label and annotation in your deployment template will stay - are we ok with having the label and annotation in the remote chart?
If so, then I might get started to implement passing the URL of a remote chart in draft.toml.


Independently Configured Main and Secondary Bar Locations

With version 1.88, a new feature was added to change the location of the symbols for the main and secondary tabs. I really like the ability to see my symbols in the secondary bar even if the tab has a longer title. However, I preferred the default appearance of the symbols being to the side of the main bar.
I was wondering if you could either add a new “hybrid” configuration option to set the side bar to default and the secondary bar to top. Or, if you could just separate the configuration options for the two altogether.
Default:

Top: (Preferred for the secondary bar)

Default: (Preferred for the main bar)

Top:

Thanks!

Similar to #210027 - but not quite the same

/duplicate #210027 closing this one in favour of other one as its main goal is the same

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


client.onError doesn't triggered when there is no connection/server

When server is not available, onError message "mostly" doesn't triggered. Sometimes it works when I build and then rebuild the project, but I couldn't find any clue or pattern about this. It is possible that other dependencies cause this or maybe built in cache of Defold.
This is the client output when there isn't any server and onError doesn't triggered :
{
  roomStates = {
  }
  roomsAvailableRequests = {
  }
  requestId = 0,
  on = function: 0x0a1b70e0,
  connection = {
    emit = function: 0x0aaf9d60,
    _enqueuedCalls = {
    }
    on = function: 0x0aaf9d80,
    is_html5 = false,
    off = function: 0x0aaf9f40,
    _on = {
      close = {
        1 = function: 0x0aafa1f0,
      }
      error = {
        1 = function: 0x0aafa140,
      }
      message = {
        1 = function: 0x0aafa180,
      }
      open = {
        1 = function: 0x0aafa0a0,
      }
    }
    state = CLOSED,
    listeners = function: 0x0aaf9d40,
    endpoint = ws://localhost:8080/?colyseusid=xmjmMr3lo,
  }
  rooms = {
  }
  hostname = ws://localhost:8080/,
  connectingRooms = {
  }
  _on = {
  }
  emit = function: 0x0a1b7210,
  listeners = function: 0x0a1b71f0,
  off = function: 0x0a1b7100,
  id = xmjmMr3lo,
}

There isn't any server and onError triggered :
{
  roomStates = {
  }
  roomsAvailableRequests = {
  }
  requestId = 0,
  on = function: 0x12f4f0e0,
  connection = {
    ws = {
      step = function: 0x13892d10,
      sock_close = function: 0x13892860,
      on_close = function: 0x13892880,
      send = function: 0x13892c30,
      on_disconnected = function: 0x13892d70,
      sock = tcp{client}: 0x13893630,
      sock_send = function: 0x138928a0,
      close = function: 0x13892cc0,
      on_connected = function: 0x13892d50,
      on_message = function: 0x13892d30,
      connect = function: 0x138929f0,
      state = CLOSED,
      sock_connect = function: 0x13892830,
      sock_receive = function: 0x138928c0,
      receive = function: 0x13892c80,
    }
    emit = function: 0x13891d60,
    _enqueuedCalls = {
    }
    on = function: 0x13891d80,
    is_html5 = false,
    off = function: 0x13891f40,
    _on = {
      close = {
        1 = function: 0x138921f0,
      }
      error = {
        1 = function: 0x13892140,
      }
      message = {
        1 = function: 0x13892180,
      }
      open = {
        1 = function: 0x138920a0,
      }
    }
    state = CONNECTING,
    listeners = function: 0x13891d40,
    endpoint = ws://localhost:8080/?colyseusid=xmjmMr3lo,
  }
  rooms = {
  }
  hostname = ws://localhost:8080/,
  connectingRooms = {
  }
  _on = {
  }
  emit = function: 0x12f4f210,
  listeners = function: 0x12f4f1f0,
  off = function: 0x12f4f100,
  id = xmjmMr3lo,
}

I just test the defold-websocket and I can confirm that it works as expected.
ws:on_connected() returns an error with 'closed'.

Also ws:on_connected returns 'closed' on colyseus (connection.lua)
self.ws:on_connected(function(ok, err)
    self.state = self.ws.state

    if err then
      pprint(err) -- <- returns closed
      self:emit('error', err)
      self:close()

Hey @selimanac,
The same thing happened to me recently (I believe). The colyseus.client tries to connect immediately upon instantiation, and it emits the "error" callback before you actually start listening to it.
To fix this, I've added a second argument not to connect immediately, and later you manually tell the client to connect. Let me know if this works for you:
client = Colyseus.new("ws://yourendpoint:1234", false) -- false as second argument

client:on("open", function()
	print("client: CONNECTED")
end)

client:on("closed", function()
	print("client: DISCONNECTED")
end)

client:on("error", function(e)
	print("ERROR: " .. tostring(e))
end)

client:connect() -- manually connect after registering the listeners


Improve scroll behavior for active cells.

In this commit, we decouple setting a cell as active and scrolling to show it. This is useful when, for example, you are making a cell active by clicking on it. Clicking indicates a point of interest in the cell (like a position in the input editor, or a particular output slider you want to interact with), and scrolling when these click events make a cell active disrupts the user’s focus on the part they want to interact with. On the other hand, programmatically setting the active cell often does involve scrolling, so we added a method to explicitly scroll to the active cell.
Fixes #478. Also fixes the same long-standing problem with the current notebook.


Don't exclude .git on the slug

Is there a reason the builder still excludes .git? https://github.com/deis/deis/blob/3cc3baf0fe7084bfe11b1dad47dc377879a14a2d/builder/rootfs/usr/local/src/slugbuilder/builder/build.sh#L155
When using a github url as a dependency on an elixir app, the .git dir is needed. The app worked fine on heroku but failed to start on deis.

I don't know about "still"; the builder has excluded .git at least since v1.0 and I think the rationale was that it wasn't needed in the slug.

worked fine on heroku but failed to start on deis.

That's not good. Is there an example app & BUILDPACK_URL to test this with?

While we don't have a public example app to test this just yet (we'll probably need to create one), but the buildpack used in question was https://github.com/HashNuke/heroku-buildpack-elixir. I created a fork so we could expose the root problem, but from the logs it didn't show anything obvious other than the .git directory being missing from the elixir's dependency, which was being fetched from github. Essentially it was similar to
gem 'foo', git: 'git://github.com/bacongobbler/foo'

Only for Elixir, which uses mix.

Here's the line that causes the error from mix:
https://github.com/elixir-lang/elixir/blob/master/lib/mix/lib/mix/scm/git.ex#L38-L41

I'll see if I can create a public repo to reproduce it. Meanwhile if you create a Phoenix app and add the following reps in the mix.exs file you'll get the error due to the github dependency:
 defp deps do
    [{:phoenix, "~> 0.13.1"},
     {:phoenix_ecto, "~> 0.5"},
     {:ecto, github: "elixir-lang/ecto", override: true},
     {:postgrex, ">= 0.0.0"},
     {:phoenix_html, "~> 1.0"},
     {:phoenix_live_reload, "~> 0.4", only: :dev},
     {:cowboy, "~> 1.0"}]
  end

thanks @fbjork for the link! This should be an easy fix :)

I created 2 phoenix apps that shows this issue. I used https://github.com/HashNuke/heroku-buildpack-elixir as the buildpack.
https://github.com/crigor/phoenix3924 deployed to http://italic-universe.4b79da4.my.ey.io/ returns 200.
https://github.com/crigor/phoenix3924-broken deployed to http://jiggly-lambskin.4b79da4.my.ey.io/ returns 502.
The only difference is using ecto from github on the broken app https://github.com/crigor/phoenix3924-broken/commit/9e3c6b0b5543628f7fc7a40d0e73039a6a6fca4d.
The error from journalctl -u jiggly-lambskin_v3.web.1 is
Jun 30 07:40:15 ip-10-165-66-107.ec2.internal sh[5026]: Unchecked dependencies for environment prod:
Jun 30 07:40:15 ip-10-165-66-107.ec2.internal sh[5026]: * ecto (git://github.com/elixir-lang/ecto.git)
Jun 30 07:40:15 ip-10-165-66-107.ec2.internal sh[5026]: the dependency is not available, run `mix deps.get`
Jun 30 07:40:15 ip-10-165-66-107.ec2.internal sh[5026]: ** (Mix) Can't continue due to errors on dependencies

which is similar to what @fbjork reported on the EY ticket.
Ping me if you want to log in to the cluster to check the containers.

@crigor thanks for setting up the test apps! I've been testing changes to the --exclude flag but the second app continues to fail. I'm going to remove the flag entirely next and test again as a sanity check. I'll report here or submit a PR when I've pinned it down.

@crigor @mboersma Any update on this? Is it fixed?

@fbjork the builder itself has been completely rewritten in Go, but I think this essential behavior is the same. I apologize for not following up--I'll try again to test the above-mentioned changes with the new builder and see if that's a reasonable change.

@mboersma I just tried this again in Deis 1.10.1 and now I'm getting a new error:
-----> Fetching app dependencies with mix
* Getting newrelic (git@github.com:wooga/newrelic-erlang.git)
Cloning into '/tmp/build/deps/newrelic'...
remote: ost key verification failed.
fatal: Could not read from remote repository.

** (Mix) Command `git clone --no-checkout --progress "git@github.com:wooga/newrelic-erlang.git" "/tmp/build/deps/newrelic"` failed
Please make sure you have the correct access rights
and the repository exists.

@mboersma actually, I take that back. I switched to https instead of ssh to fetch the git dependencies. It's back to the same issue:
2015-10-16T18:39:19UTC feed-service-fbjork[web.1]: Unchecked dependencies for environment prod:
2015-10-16T18:39:19UTC feed-service-fbjork[web.1]: * newrelic (https://github.com/wooga/newrelic-erlang.git)
2015-10-16T18:39:19UTC feed-service-fbjork[web.1]:   the dependency is not available, run `mix deps.get`
2015-10-16T18:39:19UTC feed-service-fbjork[web.1]: ** (Mix) Can't continue due to errors on dependencies
2015-10-16T18:39:30UTC feed-service-fbjork[web.1]: Unchecked dependencies for environment prod:
2015-10-16T18:39:30UTC feed-service-fbjork[web.1]: * newrelic (https://github.com/wooga/newrelic-erlang.git)
2015-10-16T18:39:30UTC feed-service-fbjork[web.1]:   the dependency is not available, run `mix deps.get`
2015-10-16T18:39:30UTC feed-service-fbjork[web.1]: ** (Mix) Can't continue due to errors on dependencies
2015-10-16T18:39:40UTC feed-service-fbjork[web.1]: Unchecked dependencies for environment prod:
2015-10-16T18:39:40UTC feed-service-fbjork[web.1]: * newrelic (https://github.com/wooga/newrelic-erlang.git)
2015-10-16T18:39:40UTC feed-service-fbjork[web.1]:   the dependency is not available, run `mix deps.get`
2015-10-16T18:39:40UTC feed-service-fbjork[web.1]: ** (Mix) Can't continue due to errors on dependencies

@fbjork I tried this with the broken example app that @crigor had worked up. With Deis master (basically the same as v1.11.1), the git push goes fine but at runtime I see the error in the app unit:
$ deis create -b  https://github.com/HashNuke/heroku-buildpack-elixir
Creating Application... done, created molten-sailfish
$ git push deis master
Counting objects: 78, done.
...
-----> Fetching custom buildpack
-----> elixir app detected
-----> Checking Erlang and Elixir versions
       WARNING: elixir_buildpack.config wasn't found in the app
       Using default config from Elixir buildpack
       Will use the following versions:
       * Stack cedar-14
       * Erlang 17.5
       * Elixir 1.0.4
       Will export the following config vars:
       * Config vars DATABASE_URL
       * MIX_ENV=prod
-----> Stack changed, will rebuild
-----> Fetching Erlang 17.5
...
-----> Fetching app dependencies with mix
       * Getting ecto (git://github.com/elixir-lang/ecto.git)
       Cloning into '/tmp/build/deps/ecto'...
       remote: Counting objects: 21377, done.
remote: remote: Compressing objects: 100% (213/213), done.
remote: remote: Total 21377 (delta 103), reused 0 (delta 0), pack-reused 21160
remote: Receiving objects: 100% (21377/21377), 5.17 MiB | 3.31 MiB/s, done.
remote: Resolving deltas: 100% (12026/12026), done.
       Checking connectivity... done.
       Running dependency resolution
       * Getting phoenix (Hex package)
       Checking package (https://s3.amazonaws.com/s3.hex.pm/tarballs/phoenix-0.14.0.tar)
       Fetched package
       Unpacked package tarball (/app/.hex/packages/phoenix-0.14.0.tar)
       * Getting phoenix_ecto (Hex package)
       Checking package (https://s3.amazonaws.com/s3.hex.pm/tarballs/phoenix_ecto-0.5.0.tar)
       Fetched package
       Unpacked package tarball (/app/.hex/packages/phoenix_ecto-0.5.0.tar)
...
-----> Copying hex from /app/.mix/archives/hex.ez
-----> Compiling
       ==> poolboy (compile)
       Compiled src/poolboy_worker.erl
...
       ==> ecto
       Compiled lib/ecto.ex
       Compiled lib/ecto/adapter.ex
...
       Compiled lib/ecto/datetime.ex
       Generated ecto app
       ==> cowlib (compile)
...
       Default process types for elixir -> web
-----> Compiled slug size is 65M

-----> Building Docker image
remote: Sending build context to Docker daemon 67.87 MB
...
Successfully built 00fc84adaf10
-----> Pushing image to private registry

-----> Launching...
       done, molten-sailfish:v3 deployed to Deis

       http://molten-sailfish.local3.deisapp.com

       To learn more, use `deis help` or visit http://deis.io

To ssh://git@deis.local3.deisapp.com:2222/molten-sailfish.git
 * [new branch]      master -> master

But it doesn't actually start, and the fleet unit Deis created says this:
Oct 16 19:03:48 deis-01 sh[14958]: 0be0b2cefa6d: Download complete
Oct 16 19:03:48 deis-01 sh[14958]: Status: Image is up to date for <IP_ADDRESS>:5000/molten-sailfish:v3
Oct 16 19:03:48 deis-01 systemd[1]: Started molten-sailfish_v3.web.1.
Oct 16 19:03:50 deis-01 sh[14985]: Unchecked dependencies for environment prod:
Oct 16 19:03:50 deis-01 sh[14985]: * ecto (git://github.com/elixir-lang/ecto.git)
Oct 16 19:03:50 deis-01 sh[14985]: the dependency is not available, run `mix deps.get`
Oct 16 19:03:50 deis-01 sh[14985]: ** (Mix) Can't continue due to errors on dependencies
Oct 16 19:03:51 deis-01 systemd[1]: molten-sailfish_v3.web.1.service: Main process exited, code=exited, status=1/FAILURE
Oct 16 19:03:51 deis-01 sh[15148]: Error response from daemon: No such container: molten-sailfish_v3.web.1
Oct 16 19:03:51 deis-01 sh[15148]: time="2015-10-16T19:03:51Z" level="fatal" msg="Error: failed to stop one or more containers"
Oct 16 19:03:51 deis-01 systemd[1]: molten-sailfish_v3.web.1.service: Control process exited, code=exited status=1
Oct 16 19:03:51 deis-01 systemd[1]: molten-sailfish_v3.web.1.service: Unit entered failed state.
Oct 16 19:03:51 deis-01 systemd[1]: molten-sailfish_v3.web.1.service: Failed with result 'exit-code'.

I'll keep testing this and see what I can figure out.

After removing the --exclude flag from deis-builder's tar commands, the git push still succeeds and the actual app fails, in a different way:
Oct 16 21:17:05 deis-03 sh[8309]: 0d560c6d506f: Download complete
Oct 16 21:17:05 deis-03 sh[8309]: Status: Image is up to date for <IP_ADDRESS>:5000/ginger-traverse:v2
Oct 16 21:17:05 deis-03 systemd[1]: Started ginger-traverse_v2.web.1.
Oct 16 21:17:05 deis-03 sh[8368]: bash: mix: command not found
Oct 16 21:17:05 deis-03 systemd[1]: ginger-traverse_v2.web.1.service: Main process exited, code=exited, status=127/n/a
Oct 16 21:17:05 deis-03 docker[8458]: Error response from daemon: no such id: ginger-traverse_v2.web.1
Oct 16 21:17:05 deis-03 docker[8458]: Error: failed to stop containers: [ginger-traverse_v2.web.1]

This issue was moved to deis/builder#470


added reverse mapping for external wallets from hash

What is this PR doing?
added reverse mapping for external wallets so that the address can be looked up when setExternalWallet is called and the wallets which are already registered can be denied re-registration to maintain 1:1 mapping with BLS Key and Wallet.
How can these changes be manually tested?
Does this PR resolve or contribute to any issues?
#104
Checklist

[ ] I have manually tested these changes
[x] Post a link to the PR in the group chat

Guidelines

If your PR is not ready, mark it as a draft
The resolve conversation button is for reviewers, not authors

(But add a 'done' comment or similar)

@kautukkundan I've pushed a new failing test but it's not quite right... I'm pretty sure there's something going on here though, please bear with me while I try to sort it out 🤔.

It looks like this mechanism only protects in the case that an upgraded wallet makes a duplicate call to setExternalWallet, and doesn't cover the case of calling setExternalWallet where the wallet is native to the gateway.
However, I think you guys are across this and you can basically disregard my changes where I was getting tangled up in some other ideas. I've pushed this branch back to where it was. If you're interested in what I've removed, I created another branch here - https://github.com/web3well/bls-wallet/tree/feat/reverse-lookup--andrew-wip.

closed in favour of #212


docs: Automated Changelog Update for v5.0.0-beta6.2

copilot:all
Automated changes by create-pull-request GitHub action

🤖 Generated by lychee action
Summary

Status
Count

🔍 Total
1258

✅ Successful
377

⏳ Timeouts
0

🔀 Redirected
0

👻 Excluded
879

❓ Unknown
0

🚫 Errors
1

Errors per input
Errors in docs/blog/zh-Hans/2023/vm-sealos-create.md

[ERR] https://laogen.cc/multipass-bridge-network-on-linux/ | Failed: Network error: dns error: no record found for Query { name: Name("laogen.cc.elnf45sohryeljqptybdtvknwa.bx.internal.cloudapp.net."), query_type: AAAA, query_class: IN }

Full action output
Full Github Actions output


Add docs for filter data path

Summary
"Filter Data Path" is missing from the docs, it's used for masking out parameters in a webhook.
Why do we need this ?
So people can know about it.
What is already there? What do you see now?
Not found in the docs
What is missing? What do you want to see?
the relative doc
How do you propose to document this?
Can you do this yourself and submit a Pull Request?
No

Do you maybe have a few more info about this?


Update common-questions-discovery-assessment.md

Adjusting VMWare to VMware for consistency.

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


Ark survival ascended server: Auto update Mods

Is it possible to somehow check for mod updates? If I'm not mistaken, the server updates mods only at startup, and mod updates are not checked during operation.

The mods are on a 3rd party website and not steam.
It is technically possible, but would require a lot of changes to the code base to implement since it would be different than any other game.
So right now, it is not planned.


Missing Instructions for "Social" starter pack usage.

[Enter feedback here]
Implementing this TechnicalProfile appears to depend on starting with one of the starter packs including "Social" login. If this is case, please update the docs here. If you started with a "Local" starter pack, which the docs don't seem to preclude, there a lot of missing steps.

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

ID: bac6db2d-bff9-cbcf-d522-966ed17ea27f
Version Independent ID: 91e3eeb7-8602-9e26-21c3-1fa70d03a354
Content: Set up sign-in for an Azure AD organization - Azure AD B2C
Content Source: articles/active-directory-b2c/identity-provider-azure-ad-single-tenant.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@danlopez
Thanks for your feedback! We will investigate and update as appropriate.

@danlopez
Reading through the Set up sign-in documentation, there are two pre-reqs that should be completed, prior to walking through the custom policy steps.

Within the Get started with custom policies in Azure Active Directory B2C doc, there's a Custom policy starter pack which needs to be downloaded in order to complete the tutorial. The XML files within the start pack are also used in this article.
Is this the starter pack you're referring to?
If you have any other questions, please let me know.
Thank you for your time and patience throughout this issue.

Got it, I think I would suggest noting that you should be starting with one of the starter packs with Social, and not just the Local starter pack. While I think that you're correct in noting that if you followed the directions exactly and went through that pre-req, it would work, it wasn't obvious to me as a developer working on an actual implementation that I'd need the "Social" Starter pack. Even a disclaimer at the top that says something like "Note that you should be starting from one of the "Social" starter packs to ensure all of the resources required for Azure AD login are present" would be a big help.

@danlopez
Thank you for the feedback and for taking the time to point this out! I've created a PR to update the documentation and it's currently waiting for the author(@msmimart) to review and sign off.
I will now close out this issue. Please allow some time for the author to review and the changes to reflect.
Thank you for your time and patience!


Updating to sylius 1.11

Only updating tests. All looks good :heavy_check_mark:

Adding messenger env
Adding calendar bundle
Updating composer.json
Adding empty api_platforme required directory

Thanks @David-Crty !


handle errors with invalid component counts for attribute data

For attribute data, the lists should all have the same size to ensure each vector has the same component length.
# Potential confusing since people will try to assign [0.0, 1.0].
o.texture_coordinates[0].data # [0.0, 0.0 ,0.0 ,0.0]

# May lose data if the type is specified as Float3, for example.
o.texture_coordinates[0].data # [0.0, 0.0]

This is handled on saving when python types are converted to the corresponding strongly typed Rust data types.


feat(RewardsStreamerMP.t.sol): Make tests use calc functions and test…

… those functions
Description
Fixes #81
Checklist
Ensure you completed all of the steps below before submitting your pull request:

[ ] Added natspec comments?
[ ] Ran pnpm adorno?
[ ] Ran pnpm verify?

@3esmit this one needs some pnpm adorno love I believe :)

@3esmit FYI there's also #92 which touches the test functions as well. Might make sense to rebase on top of that. Also happy to update #92 if this one here lands first.


[T4A5][W09-B1] Jasmine Ang

Ready for review

Sorry I branched from T4A4

Some comments added. Please close the PR after reading comments.


Directpose

What do you think about anchor-free inspired approaches like https://arxiv.org/abs/1911.07451?

Thanks for sharing. Looks like a very cool approach. Solves a lot of the complexity with multiple pose estimation we discussed in the paper. We don't have any concrete plans to add multiple pose estimation, but I'll keep this open as a feature request.

As you are using Tensorflow you could be also interested in https://github.com/google-coral/project-posenet/issues/16


How can I dynamically change validator options after creating?

Environment

Ember Version: 2.9.1
Ember CLI Version: 2.9.1
Ember CP Validations Version: 3.1.2

Steps to Reproduce
I need to change 'presence' option from 'true' to 'false' depending on dynamic form state.
If formState == 'a' 'name' field is required, if == 'b' then not, and if I dynamically toggled formState, validator should change its option.
I tried do that, but it didn't help:
//form/component.js
init() {
    this._super(...arguments);

    const Validations = buildValidations({
        name: validator('presence', {
	    presence: Ember.computed('formState', function() {
	        return this.get('formState') == 'a';
	    })
	 }),
    });

    // I use ember-validated-form-buffer for buffering changes
    this.set('formValues', formBufferProperty('model', Validations));
}

Is there any way to solve it?

Try this:
//form/component.js
init() {
    this._super(...arguments);

    const Validations = buildValidations({
        name: validator('presence', {
	    presence: Ember.computed.equal('model.formState', 'a')
	 }),
    });

    // I use ember-validated-form-buffer for buffering changes
    this.set('formValues', formBufferProperty('model', Validations));
}

Thanks, it works!
But for those who use ember-validated-form-buffer: it is sensitive to imports order and should be above the ember-cp-validations, at least in current versions:
"ember-validated-form-buffer": "0.0.1",
"ember-cp-validations": "3.1.2",

How can I do so in 2022? :sweat_smile:

Try this:
const Validations = buildValidations({
  name: [
    validator('presence', {
      presence: true,
      disabled: computed('model.formState', function () {
        return this.get('model.formState') == 'a';
      })
    })
  ]
});

This also helped me:
https://github.com/poteto/ember-changeset-validations/issues/116#issuecomment-315304932


403 Forbidden + 404 page not found http errors

When called from within a function in production server, access is denied to proxy regardless of chmod settings 644 or 755 as well as .htaccess settings for DirectoryIndex and magic_quote_gpc.
Works perfectly on local server even with magic_quote_gpc on

Same behavior...

Found the cause of this.
Check your Mod_security settings. You will need to whitelist the domain that is running the script.


BREAKING CHANGE: rewrite using gesture handler and reanimated

This is a work in progress
Demo: https://exp.host/@satya164/react-native-tab-view-demos
TODO
Below is the list of tasks I need to finish to release a beta:

[ ] Polish the spring animation during swipe
[x] Use a faster animation when switching tabs on tab press
[x] Make sure the position prop is usable on first render until layout is measured
[ ] Figure out how to make the pager play well with scrollview
[x] Fix an issue on iOS where something is NaN after tab press
[ ] Fix bugs (???)

Motivation
The PR aims to implement the tab view using gesture handler and reanimated to solve long-standing performance issues and platform inconsistencies. It's been very annoying to deal with platform differences in ScrollView and buggy behavior in ViewPagerAndroid.
The rewrite uses a single implementation for both iOS and Android. This should be much more maintainable and will have consistent behavior across platforms.
I have tried my best to keep the API mostly unchanged for an easier upgrade, but breaking changes are inevitable. The good news is, if you don't use a custom tab bar or pager, the migration should require minimal changes (other than linking gesture handler and reanimated)
Breaking changes

The renderPager prop is removed. Now there's only one pager implemented with gesture handler.
The animationEnabled prop is removed. It's not straightforward to implement, and doesn't make much sense in the scope of this library.
The canJumpToTab prop is removed
Few props such as onSwipeStart, onSwipeEnd and onAnimationEnd have been removed (if you have ideas on how to implement these, happy to add them back)
The position prop received by the renderTabBar and renderScene are now reanimated nodes, so you'll need to migrate your custom tab bar to use reanimated. This should mostly be straightforward.
Any scrollable content inside the tabs needs to use ScrollView/FlatList etc. from gesture handler
No react-native-web support because both gesture handler and reanimated are native deps. I can implement the pager with pan responder for web, but can't do much for reanimated

I've renamed master to 1.0 and merged this branch to master. I also released 2.0.0-alpha.0 to npm, so you can try it in your apps.


rename fail

create a file success, but the interface lfs_rename return -2;
read the file list ,file is exist.

Hi @WanCooder, do you have more information when this happens? Can this be reproduced in a test case?
https://github.com/littlefs-project/littlefs/blob/master/tests/test_files.toml


Count Babel tests towards Babylon code coverage

As we're already running Babel's tests in CI to detect any integration issues early, we can also use them to meaningfully bump up Babylon's test coverage, by doing this with an instrumented build of Babylon and using codecov's automatic report merging feature.
In practice this currently helps us cover things like do expressions and the :: operator which are conspicuously uncovered in the current master. Obviously for new features the ideal still is to add covering tests from the get-go.
(Also, file this under "workarounds we can retire if Babylon is ever merged back into the Babel monorepo" 😉)

Current coverage is 96.14% (diff: 100%)

Merging #182 into master will increase coverage by 1.56%

@@             master       #182   diff @@
==========================================
  Files            19         19
  Lines          3189       3189
  Methods         333        333
  Messages          0          0
  Branches        839        839
==========================================
+ Hits           3016       3066    +50
+ Misses           94         60    -34
+ Partials         79         63    -16

Powered by Codecov. Last update 394c954...3c64365

Do you think it is okay now that the babel tests are only running on the latest node version? I'm not sure if it should stay like it is or run on all supported node version now that we count it towards the coverage.

@danez I think it's fine as is, assuming these assumptions hold (and I think they do):

We only care about hit/miss/partial stats - # of hits per line is irrelevant;
We run all the Babel tests at least once;
Babel's parsing needs are independent of the version of Node it runs on.


Dynamically enabling/disabling the HTTP feature

I need to enable the HTTP feature and add an auth header when the user is logged in, but disable that feature when the user isn't logged in. But it seems that the ready() method expects these on init, and setConfig() doesn't take effect until the app is restarted. So how can I enable http and later disable it at runtime? Thanks!
iOS / Android version: iOS 11.4.1, Android 8.1, Android 9.0
NativeScript version: 4.2.0 (incl. latest runtimes)
Plugin version: latest

#ready only consumes the supplied config at first install of your app.
You should always #ready the plugin, regardless of logged-in / not.
export class HelloWorldModel extends Observable {

  private _location: string;
  private _counter: number;
  private _message: string;
  private _isMoving: boolean;
  private _enabled: boolean;
  private _paceButtonIcon: string;

  constructor() {
    BackgroundGeolocation.on('location', (location) => { ... });

    BackgroundGeolocation.ready({
      url: '...',
    }, (state) => {
      console.log('[ready] BackgroundGEolocation');
      // NOTE:  we do NOT start tracking until we know user is logged in.
    });
  }

  onLogin() {
    BackgroundGeolocation.setConfig({
      headers: {"X-FOO", "bar"}
    }, (state) => {
      // Now that user is logged in, start tracking them.
      BackgroundGeolocation.start();
    });
  }

  onLogout() {
    // Presumably you no longer want to track users when they're logged out.  Just turn off the plugin.
    BackgroundGeolocation.stop();
  }
}

Super, thanks.


Secrets documentation incorrectly states that kubelet/node compromise gives access to all secrets in the cluster

This is a Bug Report

Problem:
Concepts -> Configuration -> Secrets -> Risks section (https://kubernetes.io/docs/concepts/configuration/secret/) contains an outdated claim (added in 2016 in commit a0fb30a6fbff36d8eb86e2edf5d8272fec00a74c) that compromise of a node/kubelet gives an attacker access to all secrets in the cluster. Specifically:

Currently, anyone with root permission on any node can read any secret from the API server,
by impersonating the kubelet. It is a planned feature to only send secrets to nodes that
actually require them, to restrict the impact of a root exploit on a single node.

My understanding is that modern clusters by default are configured to provide node/kubelet access only to secrets for Pods scheduled on that node/kubelet. It looks like this security security improvement is implemented via the NodeAuthorizer. Tested on Amazon EKS 1.15.
Proposed Solution:
Edit content/en/docs/concepts/configuration/secret.md to either entirely remove the above quote, or, if the security improvement is not enabled by default, provide information about how to enable it.
Page to Update:
https://kubernetes.io/docs/concepts/configuration/secret/

/language en

Thanks for reporting this @klyubin

/language en

The outdated statement appears to be there in fr as well. I suspect it might be there in more languages.

That's OK - the localization teams have their own processes for picking up changes that are fixed in the upstream English localization.

/assign

I notice also that https://kubernetes.io/docs/reference/access-authn-authz/node/ says:

In future releases, the node authorizer may add or remove permissions to ensure kubelets have the minimal set of permissions required to operate correctly.

(my emphasis)

/sig auth

/remove-lifecycle rotten

Anyone who'd like to, please feel free to open a pull request with an update.

Anyone who'd like to, please feel free to open a pull request with an update.

/kind documentation

/sig security

/triage accepted


Update backfill monorepo

This pull request primarily involves updates to the backfill monorepo and its dependencies across multiple packages. The most significant changes are the patches made to several packages and the updates to their dependencies in the package.json files. Additionally, @azure/core-http has been added to the list of inlined libraries in dts-bundle.config.js.
Monorepo updates:

change/change-d0bde26c-13d3-4b9b-a3bb-c5f7c69d02e2.json: The backfill monorepo has been updated with patches to @lage-run/cache-github-actions, @lage-run/cache, @lage-run/config, and lage.

Dependency updates:

packages/cache-github-actions/package.json: The backfill-config dependency has been updated from version 6.4.1 to 6.4.2.
packages/cache/package.json: The backfill-cache and backfill-config dependencies have been updated from versions 5.7.1 and 6.4.1 to 5.8.0 and 6.4.2, respectively.
packages/config/package.json: The backfill-config dependency has been updated from version 6.4.1 to 6.4.2.

Inlined libraries:

packages/lage/dts-bundle.config.js: The @azure/core-http library has been added to the list of inlined libraries.

No longer needed.


Simulator for Poisson regression

Would be nice to have poisreg_simulate() function to simulate from the standard Poisson regression:
y | x ~ Pois(exp(X%*%betas))
since it is very easy to implement. This would be useful for SDA2 and SDA3.
I would even like to have a negative binomial regression simulator. Can be used to show that Poisson regression is restrictive (mean = variance).

Added poisreg_simulate and nbinomreg_simulate. Closing now.


Improve detection of change in statefulset

I'm already using another library see https://github.com/Orange-OpenSource/cassandra-k8s-operator/pull/99/files#diff-37aff102a57d3d7b797f152915a6dc16R22
Originally posted by @cscetbon in https://github.com/Orange-OpenSource/cassandra-k8s-operator/pull/112

duplicate of #69


SharePoint deployment files

Hi!
I'm triying to test this wonderfull sample but I can't find the .aspx and .xml files from SharePoint Deployment step.
Any way to get them?

Thanks, we have included these files as part of deployment tool.
https://github.com/OfficeDev/Virtual-Health-Templates/blob/master/VirtualHealthDeploymentTool.zip
Regards,
Ritesh

thanks!!!!!


VmReconfigureRequest / VmMigrateRequest should be sent to the zone where the VM is located.

instead of the zone where the request is made.
https://bugzilla.redhat.com/show_bug.cgi?id=1201383

Can you add a test for that?

@gmcculloug Please review.

@gmcculloug @tenderlove @Fryguy
Added test cases. Please review.

@lfu This change will not work for the reconfigure and migration requests.  They allow multiple sources to be selected at the request level so the source relationship is not available there, you will need to look at options[:src_ids] to determine the selected sources at the request level.  The task level would have the source relationship set and we need to make sure we are passing the zone name there as well.

@gmcculloug Updated.

Validated fix.  Looks good.


Feature Request: On powering down VM, try to shutdown first

In movirt, we got a red arrow for bringing down a VM.
It will be nice  if movirt check if the VM has an agent installed and invoke graceful shutdown in such cases instead of pulling the plug.
I guess a timeout can be defined , which will trigger ungraceful 'power off'.

I am sorry. but I don't much time for this now. I will have a chance to look at it in one month or so.
We could add a bit simpler solution - just adding a graceful shutdown action.
I like your idea much better though.

You can ping me , once you add this feature and I can test it on a test VM.

Sorry it took me so long to get back to this. I added the shutdown action. There is a PR for the new version: https://github.com/oVirt/moVirt/pull/311 - I will try to release it next week.
Let me know if it works for you (apk link), if you can spare some time to check it out.

Great Job!
It shutdown both my Win10 and CentOS7 VMs without issues.
Also,my openBSD 6.6 (no guest agent is available)  is also powered off ungracefully  (that's expected - no guest tools) sl the feature is working.

Good to hear that :) thanks for testing it

released version 2.1 which should be available on Google Play soon


Release

Closes #
📝 Description

Add a brief description

⛳ Current behavior (updates)

Please describe the current behavior that you are modifying

🚀 New behavior

Please describe the behavior or changes this PR adds

💣 Is this a breaking change (Yes/No):

📝 Additional Information

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.97%. Comparing base (669bf5d) to head (41457bc).
Report is 24 commits behind head on release.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           release     #105      +/-   ##
===========================================
+ Coverage    98.95%   98.97%   +0.01%
===========================================
  Files           20       20
  Lines          383      389       +6
  Branches       111      114       +3
===========================================
+ Hits           379      385       +6
  Misses           4        4

Flag
Coverage Δ

Autosizer
100.00% <ø> (ø)

PhoneInput
98.44% <100.00%> (+0.04%)
:arrow_up:

useBreakpoint
98.80% <ø> (ø)

useClickOutside
100.00% <ø> (ø)

usePreserveInputCaretPosition
100.00% <ø> (ø)

usePrevious
100.00% <ø> (ø)

useSelectionRange
100.00% <ø> (ø)

useToggle
100.00% <ø> (ø)

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

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


An Online Education Portal made using HTML CSS and JS

Responsiveness is included too

@Vanshiii203 please assign me this issue

A student service portal is already present in the repo. You can create an issue if you have any ideas or changes for the existing project


Refs 145: walking on broken line fix

Following solution fixes problem of character playing once oriented animation from  the very beginning up to the end of animation, ignoring geometry of broken line path.
In order to fix that _play_walk had to be called in order to have _get_valid_oriented_animation called.
Reservations:
_play_walk is private function. In order to make this solution clean, it should be made public or should be wrapped into some public function eg.:
func take_turn(target_pos: Vector2) -> void:
    _play_walk(target_pos)

@Whyshchuck LGTM, thanks a lot! :+1:
@mapedorr I leave this to you to merge so you can rebase when you prefer


Hello, in the android gesture recognition, I want to use your own video YUV data is passed to the mediapipe, instead of using the data of mobile phone camera, I need how to modify com.google.mediapipe.com ponents package of files. Please help me, thank you.

What is the source of the YUV data?

I am also looking for similar solution, Its looking they added support for YUV frames in flow_limiter_calculator.cc calculator but there is no method available in AndroidPacketCreator or in PacketCreator classes to create YUVBuffer. So I convert YUV2RGB using shader and render to FrameBuffer. I used GL_TEXTURE_2D instead of GL_TEXTURE_EXTERNAL_OES in TextureRenderer, Now I can pass this texture to FrameProcessor.onNewFrame(final TextureFrame frame) method.
Other  things will be similar now.

I am also looking for similar solution, Its looking they added support for YUV frames in image_transformation_calculator.cc calculator but there is no method available in AndroidPacketCreator or in PacketCreator classes to create YUVBuffer. So I convert YUV2RGB using shader and render to a texture of FrameBuffer. I used GL_TEXTURE_2D instead of GL_TEXTURE_EXTERNAL_OES in TextureRenderer, Now I can pass this texture to FrameProcessor.onNewFrame(final TextureFrame frame) method.
Other things will be similar now.

Thanks for your help. I am busy with other things these days. I will try the way you said.

What is the source of the YUV data? Is it from a different camera or a video file?

Hello, I used my own camera to transmit the data, which is currently in YUV format


vspk-javascript: Include minLength, maxLength, minValue, maxValue in attribute if provided

Include minLength, maxLength, minValue, maxValue in attribute if provided in api-specification.
Sample generated code NUCommand.js:
import NUAttribute from 'service/NUAttribute';
import ServiceClassRegistry from 'service/ServiceClassRegistry';
import NUEntity from 'service/NUEntity';
import { NUCommandCommandEnum, NUCommandStatusEnum, NUCommandOverrideEnum } from './enums';

/* Represents Command entity
   A Command represents an operation that needs to be executed on an entity (NSG, Gateway, ...)
   which requires little processing by VSD, but may result in a long activity on the external
   entity.  An example would be to trigger an action on VSD so that a Gateway download a new image.
   VSDs handling of the request is limited to generating a message to be sent to the device on which
   the download process is expected.  The device then acts on the request and proceeds with the
   download...  That may be a long process.  The commands API is similar to the Jobs API with
   regards to triggering operations on objects.
*/
export default class NUCommand extends NUEntity {
    constructor(...args) {
        super(...args);
        this.defineProperties({
            detail: undefined,
            detailedStatus: undefined,
            detailedStatusCode: undefined,
            command: NUCommandCommandEnum.UNKNOWN.name,
            commandInformation: undefined,
            progress: undefined,
            assocEntityType: undefined,
            associatedParam: undefined,
            associatedParamType: undefined,
            status: undefined,
            fullCommand: undefined,
            summary: undefined,
            override: NUCommandOverrideEnum.UNSPECIFIED.name,
        });
    }

    static entityDescriptor = {
        description: `A Command represents an operation that needs to be executed on an entity (NSG, Gateway, ...) which requires little processing by VSD, but may result in a long activity on the external entity.  An example would be to trigger an action on VSD so that a Gateway download a new image.  VSDs handling of the request is limited to generating a message to be sent to the device on which the download process is expected.  The device then acts on the request and proceeds with the download...  That may be a long process.  The commands API is similar to the Jobs API with regards to triggering operations on objects.`,
        userlabel: `Command`,
    }

    static attributeDescriptors = {
        ...NUEntity.attributeDescriptors,
        detail: new NUAttribute({
            localName: 'detail',
            attributeType: NUAttribute.ATTR_TYPE_STRING,
            description: `Details about the command execution as reported directly from the NSG independent of status codes.`,
            isReadOnly: true,
            userlabel: `Detail`,
            minLength: 0,
            maxLength: 1023,
        }),
        detailedStatus: new NUAttribute({
            localName: 'detailedStatus',
            attributeType: NUAttribute.ATTR_TYPE_STRING,
            description: `A string representing the detailed status of the operation that was triggered by the execution of the Command instance.`,
            isReadOnly: true,
            userlabel: `Detailed Status`,
            minLength: 0,
            maxLength: 255,
        }),
        detailedStatusCode: new NUAttribute({
            localName: 'detailedStatusCode',
            attributeType: NUAttribute.ATTR_TYPE_INTEGER,
            description: `A numerical code mapping to a list of detailed statuses that can apply to a Command instance.`,
            isReadOnly: true,
            canOrder: true,
            canSearch: true,
            userlabel: `Detailed Status Code`,
            minValue: 0,
            maxValue: 65535,
        }),
        command: new NUAttribute({
            localName: 'command',
            attributeType: NUAttribute.ATTR_TYPE_ENUM,
            description: `Specifies the type of command that is stated for execution on the system receiving the operation request.  A request for download, a request for upgrade, a request for revocation, ...`,
            isRequired: true,
            isCreateOnly: true,
            canOrder: true,
            canSearch: true,
            choices: [NUCommandCommandEnum.NSG_APPLY_PATCH, NUCommandCommandEnum.NSG_DELETE_PATCH,
                NUCommandCommandEnum.NSG_DOWNLOAD_OS_IMAGE,
                NUCommandCommandEnum.NSG_UPGRADE_TO_IMAGE, NUCommandCommandEnum.UNKNOWN],
            userlabel: `Command Requested`,
        }),
        commandInformation: new NUAttribute({
            localName: 'commandInformation',
            attributeType: NUAttribute.ATTR_TYPE_STRING,
            description: `Informative details on what command is to be executed.  It complements the commandType attribute.  An example of a value could be a URL, a version number, a UUID of another object, ...`,
            isReadOnly: true,
            userlabel: `Command Information`,
            minLength: 0,
            maxLength: 1023,
        }),
        progress: new NUAttribute({
            localName: 'progress',
            attributeType: NUAttribute.ATTR_TYPE_STRING,
            description: `JSON string detailing the progress of the command execution on Gateway.`,
            isReadOnly: true,
            userlabel: `Progress`,
            minLength: 0,
            maxLength: 1023,
        }),
        assocEntityType: new NUAttribute({
            localName: 'assocEntityType',
            attributeType: NUAttribute.ATTR_TYPE_STRING,
            description: `Managed Object Type of the entity on which this Command is associated.`,
            isReadOnly: true,
            userlabel: `Associated Entity Type`,
        }),
        associatedParam: new NUAttribute({
            localName: 'associatedParam',
            attributeType: NUAttribute.ATTR_TYPE_STRING,
            description: `Parameters to be supplied for execution of this command. This should be the ID of the object supplying parameters.`,
            canSearch: true,
            userlabel: `Associated Parameter`,
            minLength: 0,
            maxLength: 1023,
        }),
        associatedParamType: new NUAttribute({
            localName: 'associatedParamType',
            attributeType: NUAttribute.ATTR_TYPE_STRING,
            description: `Type of the object which supplies parameters for this command. For NSG_APPLY_PATCH command this should be NSG_PATCH_PROFILE. For NSG_DELETE_PATCH it should be PATCH`,
            canSearch: true,
            userlabel: `Associated Parameter Type`,
            minLength: 0,
            maxLength: 255,
        }),
        status: new NUAttribute({
            localName: 'status',
            attributeType: NUAttribute.ATTR_TYPE_ENUM,
            description: `The status of the Command from a VSD perspective.`,
            isReadOnly: true,
            canOrder: true,
            canSearch: true,
            choices: [NUCommandStatusEnum.ABANDONED, NUCommandStatusEnum.COMPLETED,
                NUCommandStatusEnum.FAILED, NUCommandStatusEnum.RUNNING,
                NUCommandStatusEnum.SKIPPED, NUCommandStatusEnum.STARTED,
                NUCommandStatusEnum.UNKNOWN],
            userlabel: `Status`,
        }),
        fullCommand: new NUAttribute({
            localName: 'fullCommand',
            attributeType: NUAttribute.ATTR_TYPE_STRING,
            description: `Full command including parameters that is to be executed.`,
            isReadOnly: true,
            userlabel: `Full Command`,
            minLength: 0,
            maxLength: 1023,
        }),
        summary: new NUAttribute({
            localName: 'summary',
            attributeType: NUAttribute.ATTR_TYPE_STRING,
            description: `A generated summary for the action giving some general context on the command executed.`,
            isReadOnly: true,
            userlabel: `Summary`,
            minLength: 1,
            maxLength: 255,
        }),
        override: new NUAttribute({
            localName: 'override',
            attributeType: NUAttribute.ATTR_TYPE_ENUM,
            description: `Operator specified action which overrides the normal life cycle of a command.`,
            canOrder: true,
            canSearch: true,
            choices: [NUCommandOverrideEnum.ABANDON, NUCommandOverrideEnum.UNSPECIFIED],
            userlabel: `Override`,
        }),
    }

    static getClassName() {
        return 'NUCommand';
    }

    static getAllowedJobCommands() {
        return [];
    }

    static supportsAlarms() {
        return false;
    }

    static getInstanceFromID(ID) {
        const instance = new NUCommand();
        instance.ID = ID;
        return instance;
    }

    get RESTName() {
        return 'command';
    }

    get resourceName() {
        return 'commands';
    }

    getClassName() {
        return NUCommand.getClassName();
    }

    getAllowedJobCommands() {
        return NUCommand.getAllowedJobCommands();
    }
}

ServiceClassRegistry.register(NUCommand);

@natabal can you pls review?


addVars not working for spatial output of climr downscale

if select out_spatial=TRUE on downscale call in climr, addVars function will not work because input is not a dataframe. How to fix this? @kdaust @cmahony

@cour10eygrace i recommend you work with out_spatial=FALSE.

workaround with rasterize from vector then pull info out to data frame and addvars


Allow work agent to create subjectaccessreviews

Signed-off-by: zhujian<EMAIL_ADDRESS>issue reference: https://github.com/open-cluster-management-io/work/issues/153

/hold for https://github.com/open-cluster-management-io/work/pull/152 to merge

/cc @qiujian16 @zhiweiyin318

/approve
/lgtm

/unhold


fix(render-error): Toggle error text based on devTools theme

Resolves #833

@riavalon We've previously managed this just based off CSS classes (.dark as compared to none). Any chance we could align this one with that? Just for consistency.

:shipit:


cannot clone large repo.

Gitea version (or commit ref): 1.13.0+dev-664-g2dbca92a0
Operating system: DSM 6.2.3-25426 Update 2 with docker
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (https://try.gitea.io/test006/webkit)
[ ] No
[ ] Not relevant

Log gist:

Description
When I try to clone https://github.com/WebKit/webkit, I just direct add it form migrate, and it'll start clone, after a while, it'll be 404 when I try to open the url of the webkit on my own website. and there is no log with docker just have visit log. And I tried to edit the app.ini for section [timeout], not working.
Screenshots

You can find some logs on admin panel about task log

Got it, it says  Migrate repository from https://github.com/WebKit/webkit failed: Clone: context deadline exceeded - fatal: early EOF, and I search this error again with the exist issue, it's problem with git.timeout section, and I will extend this number to bigger and try again.

OK, It has been done.
This is my custom section with git.timeout
[git.timeout]
DEFAULT = 720
MIGRATE = 30000
MIRROR = 72000
CLONE = 30000
PULL = 30000
GC = 60


changes from example branch

merge request. readme.md changes

:+1:

earlier closed the request but now want to merge the changes.


[feature request] support passing -H / --host to docker sandbox

I'd like to be able to specify that some tasks / samples should run on a remote machine by passing -H to the docker commands.  Right now, I am using os.environ["DOCKER_HOST"] = docker_host, but it seems like this might interfere with other tasks if I try to run them in parallel.  Could support be added to bind -H on a per-task basis?

This is a good suggestion -- right our now SandboxSpec type is a simple tuple and I think it needs to become a richer data structure.


New release with helm namespace feature

Hi, can you do a new release with helm namespace feature please ?
At the moment (March 22nd 2022) main doesn't work so I'll wait for the new release ;)

I tried with main and sha 6d27e77ac46a5c7fe2af0b876591a280fd137de7 but both give the same kind of result:
Run azure/k8s-bake@6d27e77ac46a5c7fe2af0b876591a280fd137de7
Error: File not found: '/home/runner/work/_actions/azure/k8s-bake/6d27e77ac46a5c7fe2af0b876591a280fd137de7/lib/run.js'

I created a new release. v2.2 will work.


Improvement in SignIn SignUp Pages

Describe the feature
Hey,
From the design point of view, There are some issues with the Sign in and Sign up pages in the current version.
Issues:

There are too many colours when you first open either page (grey, white, back, navy blue, green buttons, gradient box) and I feel they are not going too well somehow.
Continue button should look distinct with respect to the background (it is blending in).
Size of the continue button is not the same on both pages.
When you hover over the continue button, it changes to a muted orange colour (not in the colour palette or branding) and scales in size which is going out of the boundaries of the signup box.
Font size of the Headings "sign in to your account" and "create an account" are not uniform.
Size of sign-in/sign-up box changes when you switch between the two.
Input field of sign up menu is greater in height as compared to the fields of sign in menu
Overall look and feel is too bold.

Add ScreenShots
Improvements -

Simplifying the UI of the page as shown below
Uniforming the headings, box size, input field size, button size, font size
Overall More formal, designed look in accordance with the branding and color palette.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'23 contributor
[X] I want to work on this issue

Hey @MonalikaPatnaik @TechSpiritSS ,
Hope, you guys are doing great. Please assign this issue to me under GSSOC'23 Label.
Thanks.

Making it level 3, but ensure to fix everything in this sign in

Also, you could add relevant images instead of the text in green part of the form!

Also, you could add relevant images instead of the text in green part of the form!

Hello @MonalikaPatnaik ,
I can surely do that. I have raised a PR solving all the issues I addressed. I would request you to kindly review that first and give me feedback on the same.
I will raise more issues with fixes to more such design problems we have in this project.
Thanks,
Yogit

Hii, left a comment on your PR


firefoxPackages.tor-browser: 8.5.4 -> 8.5.6

git log

firefoxPackages.tor-browser: 8.5.4 -> 8.5.6

nix-instantiate environment

Host OS: Linux 4.9, SLNOS 19.09
Nix: nix-env (Nix) 2.2.2
Multi-user: yes
Sandbox: yes
NIXPKGS_CONFIG:

{
  checkMeta = true;
  doCheckByDefault = true;
}

nix-env -qaP diffs

On x86_64-linux:

Updated (2):

firefoxPackages.tor-browser
tor-browser-bundle

On aarch64-linux: ditto
On x86_64-darwin:

Updated (1):

firefoxPackages.tor-browser

/cc @joachifm

Thank you. This has been picked to release branches as well.


Add API to fetch particular report and reporter details

Added to fetch a particular report and added API to fetch the reporter details.
Also added User Schema and routes to add new user

@saranrajshri I went through the code for dengue-stop-server. I wanted to suggest a few suggestions:

For instance, if you see here reportedBy and reporterID are both being sent. Similarly, locationName, country and coordinates are being fetched. Is it possible to reduce adding so many keys? For instance, if you only share the reporterID instance you may be able to fetch the reportedBy value. Similarly, I feel that the coordinates key would be self sufficient than the country and locationName keys. That would reduce the data being sent over the request and make it more efficient.


fixes deprecation warning on linter.rb

With ruby 2.7 we start seeing the following warnings when running tests
/factory_bot/lib/factory_bot.rb:74: warning: Using the last argument as keyword parameters is deprecated; maybe ** should be added to the call
/factory_bot/lib/factory_bot/linter.rb:3: warning: The called method `initialize' is defined here

I'm proposing the following changes so that the message is supressed.

Looks good. Thanks!


Make Public API smaller

I made parts private somewhat arbitrarily. I split this into multiple commits so feel free to take as much or as little as you want. Since master is v1.0.0-dev it's an important conversation to have, what's public and what's private.
Thanks for creating and maintaining this library over the years!

Thank you.

There are conflicts, but I did at @moduledoc false to a number of modules. I haven't gone over the specs changes, but I think they might have conflicted with another PR that was merged first. If there are spec changes you think should be merged, a new PR or rebasing this one to fix conflicts is welcomed.

@lau what's on master looks good to me, closing this.

The only minor change I'd consider is replacing a custom type:
Tzdata.time_zone_name()
with the one from stdlib:
Calendar.time_zone()
WDYT? Happy to send a PR.

with the one from stdlib:
Calendar.time_zone()
WDYT? Happy to send a PR.

That sounds good


Update radio_getradiosasync_548754145.md

The Radio class functions have some operational defects.  GetRadiosAsync only works if your environment is static.  If radios are added or removed, the function does not reflect the changes when you call it again.  This is different than the behavior seen in Device Manager, which instantly reflects such a change.
In connection with that, the StateChanged function also fails to fire when you remove the radio from the system, as is common with USB Bluetooth dongles on laptops, or desktops with intermittent failures in a USB port, USB cable, or USB Bluetooth dongle on that cable.  Such failures are more common than would be desired in industrial settings, and software must diagnose the problem and assist in its correction.

@laurenhughes : PR passed review checklist. Please evaluate the proposed changes further. Thanks!


[Question] Will 10.0.0-beta.3 be unavailable in future after some stable updates?

Description
We're trying to use 10.0.0-beta.3 in our project. This version have some fixes which I was looking for Tha's why we are using this version.
But we have concern about the further stable releases.

Could you please let us know what is the release cycle?
After further stable release, will this version be unavailable from npm repository?
When is the nearest stable release coming?

Thanks

Hello,
Currently at 10.0.0-beta.4, the beta branch will eventually make it into master with a 10.0.0 release.
I don’t think any versions are ever removed from npm. You can test this by running npm info lint-staged versions.
There are not really any release cycles planned, but I guess if 10.0.0 has some bugs, those will be fixed with 10.0.X patch versions.
What do you think, @okonet?

We use semantic-release to release versions and now we adopted beta channel to allow our users test upcoming releases. If you just install the lib from npm you’ll get the current stable version which is v9.x.

There is no official release cycle since people are working in their free time. If you want to support us, please use “support” button.


Programming Question: Ref and anonymous function

Hey @PeturDarri
Thank you for this tool!
I have a programming question regarding subscription to the event:
in the start function, I call:
GenericEventBus.Instance.SubscribeTo(Handler);
then another method is subscribed:
    private void Handler(ref RequestTryChangeStateEvent eventdata)
    {
        throw new NotImplementedException();
    }

But I'd like to do that:
        GenericEventBus<IEvent>.Instance.SubscribeTo<RequestTryChangeStateEvent>(ref args =>
        {
            var physicsState = args.RequestedPhysicsState;
            TryChangePhysicsState(physicsState);
        });

but I get an error on the args parameter, it says: "Parameter 'args' must be declared as 'ref'
I understand it's not possible to use the ref parameter in the anonymous function since this function manipulates the variable (which cannot be modified if is 'ref'). Can I use a similar syntax as I'd like, in this case?
Thank you!

When defining a ref argument in a lambda, you must specify the type and wrap it in parenthesis:
GenericEventBus<IEvent>.Instance.SubscribeTo<RequestTryChangeStateEvent>((ref RequestTryChangeStateEvent args) =>
{
    var physicsState = args.RequestedPhysicsState;
    TryChangePhysicsState(physicsState);
});

Since the type is now specified in the argument, it no longer needs to be specified in the method call, so this can be simplified to:
GenericEventBus<IEvent>.Instance.SubscribeTo((ref RequestTryChangeStateEvent args) =>
{
    var physicsState = args.RequestedPhysicsState;
    TryChangePhysicsState(physicsState);
});

However, I would not recommend subscribing with anonymous functions, because it makes it impossible to unsubscribe unless you save and reuse the same instance in both calls. The only valid use case is if you know this event handler doesn't need to be unsubscribed.

Thank you so much for this explanation.


Update to use beta4  release of Api, Engine.Core and Agent.Core

:tada: This issue has been resolved in version 2.3.0 :tada:
The release is available on:

GitHub Release
NuGet Package
Chocolatey Package


How to handle route params on child routes and interactions between sagas and selectors

Issue Type

Question

Description
Hello, here I am striking again!
Now I am asking help to clarify which probably is a misconception about route params, child routes selectors and sagas. I don't have problems understanding each piece on it's own: react-router is easy, reselect has a small surface API, sagas are not a challenge anymore to me, but when I try to put all those things together I get lost.
My scenario is as follows:
I have a route which is /explore that display a collection of items. Each item is complete, this means that I don't need to ask the server for extra information.  However, the explore route only shows a small set of the information each item has. I want to create a child route (or another route, not sure if it should be a child route) like this: /explore/:id.
Such route should display a detailed view of the item with id and provide edit functionalities.
My first thought of what should be done was:

Navigate to the route (by link or dispatch)
Get the id from the route params
Select the portion of the state that has all the items and select the one which idmatches

The first step was easy, however the second is not obvious. Yes, the rendered container has the params as part of their props, but I don't know what to do with it.  However, I don't understand when selectors are executed or if I'm able to re-execute them or how to pass any parameter to them.
Another approach I tried is triggering an action on route load:
importModules.then(([actions, reducer, sagas, component]) => {
                injectReducer('licenseDetail', reducer.default);
                injectSagas(sagas.default);
                renderRoute(component);
                store.dispatch(actions.displayLicenseAction(nextState.params.id))
            });

However, this will require sagas to manage the action, and it seems more like a task for selectors. I don't need to make any request to the server, I only need to make the id available to selectors so they can select the appropriate part of the state. Should I dispatch an action, go through reducers, make the selected task id part of the redux state and then use reselect to select the appropriate item? Seems like a way, but it also seems very complex for such a simple task.
Many thanks

I have something similar. However I decided to use a new route for /explore/:id
I make an ajax call in /explore (by a searchText in my case), so I get a list of items that can be clicked (detailed view for each item in /explore/:id). Detailed view should get data from store, as you say.
However maybe someone goes to /explore/:id directly so then I want to make an ajax request just for that item (only if item is not in the store, because I cache items).
I think your case should be something like this:
const mapStateToProps = (state, ownProps) => {
  const items = state.getIn(['explore, 'search', ownProps.params.searchText]) || []
    return ({
       items
  })
}

render() {
    const { items } = this.props
    const gallery = items.length > 0 ? React.cloneElement(children, { items }) : null
    return (
        {gallery}
      </View>
    )
  }

 childRoutes: [
        {
          path: '/explore/:id',
          name: 'gallery',
          getComponent(nextState, cb) {
            const importModules = Promise.all([
              import('components/exploreItem')
            ])
            const renderRoute = loadModule(cb)
            importModules.then(([component]) => {
              renderRoute(component)
            })
            importModules.catch(errorLoading)
          }
        }
      ]

Just now I've changed my mind, as proptypes aren't checked, see: https://github.com/ReactTraining/react-router/issues/1857
Maybe better than using cloneElement and a child route, use just one route with an optional parameter
path: '/explore(/:id)',

I've got the same issue as @danielo515 and I also didn't expect this to be this hard.
For my case, the route is something like /posts/:id, and I can get a hold of the ID in the container. I then dispatch an action to make the AJAX request and from there I don't know how to proceed.
Here's the action that is dispatched from the container. It is called with the current post id in the componentDidMount function.
import { GET_POST_DETAILS } from './constants';

export function getPostAction(id) {
  return {
    type: GET_POST_DETAILS,
    id,
  };
}

export function postLoaded(post) {
  return {
    type: GET_POST_DETAILS_SUCCESS,
    post,
  };
}

Here's the corresponding reducer, that updates the state with the post_id and the post details, once they are loaded:
import { fromJS } from 'immutable';
import {
  GET_POST_DETAILS,
  GET_POST_DETAILS_SUCCESS,
} from './constants';

const initialState = fromJS({
  post_id: null,
  post: [],
});

function postReducer(state = initialState, action) {
  switch (action.type) {
    case GET_POST_DETAILS:
      return state.set('post_id', action.id);
    case GET_POST_DETAILS_SUCCESS:
      return state.set('post', action.post);
    default:
      return state;
  }
}

export default postReducer;

My sagas look like this:
export function* getPostDetails() {
  const state = yield select(makeSelectPost());
  const post = yield call(getPostDetails, state.post_id);
  yield put(postLoaded(post));
}

export function* watchPostDetailsSaga() {
  yield takeLatest(GET_POST_DETAILS, getPostDetails);
}

However, when the getPostDetails saga is executed, the selector is only able to get the initial state, even though I have already dispatched the action that should update the state with the post_id.
Is there anything obvious I'm missing here to make it work?

@kevgathuku You "catch" GET_POST_DETAILS action, both from the saga and from the reducer. I'm not sure about state.post_id value being there or empty...
I prefer to get the value from the action (in the saga, the same as in the reducer), something like this:
export function* getPostDetails(action) {
const post = yield call(getPostDetails, action.id);
yield put(postLoaded(post));
}
export function* watchPostDetailsSaga() {
yield takeLatest(GET_POST_DETAILS, getPostDetails);
}

Thanks @juanda99 for the suggestion. I eventually figured it out.
I got rid of the selector because value from the action is more straightforward
The problem was that I wasn't getting any data in the action parameter and I found out it was because I was exporting the wrong saga
I eventually changed
export default [
  getPostDetails,
];

to
export default [
  watchPostDetailsSaga,
];

and it worked.
I can now access the action inside the getPostDetails saga

@juanda99 thanks for your suggestion. I ended using a MAIN route with the parameter and another main route without the parameter.
What I do is trigger an action on route change, that action includes the id, which is processed by the reducer to put it on the current's view-portion of the state. Then the saga gets the item to display through a selector. If the selector returns nothing, then an AJAX call is dispatched and the item is retrieved. To be honest, this is so damn complicated that it was hard to remember/explain less than one week after I coded it, I can't imagine how hard will be in two months.
Another problem of my approach is that both components ( the one at /explore and the other one at /explore/:id) are being triggered (reducers and selectors executed ) when I navigate from /explore to /explore/:id. Only one gets rendered, but it's a waste IMO.
I tried using a child route, but then only the component at the main route gets rendered, which is something I cant' understand.
Probably ONE single route with one optional parameter is the simplest solution. Then you can decide which component to render. But then you are loading two components when you may need only one.

Closing due to inactivity


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
b273cc7 -> b52801c

github.com/IBM/platform-services-go-sdk
require
minor
v0.54.1 -> v0.55.0

github.com/gruntwork-io/terratest
require
patch
v0.46.7 -> v0.46.8

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.25.3 -> v1.25.6

Release Notes

IBM/platform-services-go-sdk (github.com/IBM/platform-services-go-sdk)
v0.55.0
Compare Source
Features

Usage Reports: add billing snapshot config validate service to usage reports (#291) (da51703)

gruntwork-io/terratest (github.com/gruntwork-io/terratest)
v0.46.8
Compare Source
Modules affected

k8s

Description

Updated k8s.io/apimachinery from v0.27.2 to v0.28.4.
Updated k8s.io/client-go from v0.27.2 to v0.28.4.

Special thanks

Special thanks to the following users for their contribution!

@marcofranssen

Related links

https://github.com/gruntwork-io/terratest/pull/1375

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.25.6
Compare Source
Bug Fixes

deps: update module github.com/go-git/go-git/v5 to v5.11.0 (#714) (021df86)

v1.25.5
Compare Source
Bug Fixes

deps: update gomod (#711) (9a2a01d)

v1.25.4
Compare Source
Bug Fixes

deps: update module github.com/go-openapi/strfmt to v0.21.8 (#708) (46329a3)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline


CSCETSIN-504: Fixed broken download implementation on frontend.

This should fix the broken download feature on Etsin side. The current implementation is using axios.get which is meant for rest requests, not for binary downloads.

It also looks that the current implementation does trigger downloads twice. Once without saving response to anywhere and second time it will download the file with "save as" dialog.

This fix basically reverts it to how it was before. The reason why the axios get request to the backend was added was so that IF any errors occur, they would get caught and handled. The problem with the previous implementation and for the looks of it, this as well is that if an error happens in the download component it will not get handled. It might be redundant that the download is triggered twice but at least it was a way to catch errors. One of the original problems was that we did not handle the errors in etsin and this caused the browser to handle it, that is why Firefox gave the corrupted content error when it got confused by the headers.

This is now replaced by https://github.com/CSCfi/etsin-finder/pull/348.
To note:
"It might be redundant that the download is triggered twice but at least it was a way to catch errors"
This is not a good reason to consume twice the bandwidth and to cause full browser crash on Chrome.


what other spacer/standoffs sizes are compatible

Hi,
My issue is i am not getting m2 spacer nearby and nor on online at least in my country at good price, can i use m3 spacer or m2.5 spacer, or what are the other options regarding sizes for spacers if possible?

Hello!
The size of the hole in the board to accommodate the standoffs is 3.5mm. Only M2 of a standoffs will fit into such a hole.

You can save money by using regular M2 bolts and nuts. Just calculate the height of the bolts so that they are not too long. This method is not as convenient compared to racks.
You can also use spacers made of different materials. For example, print on a 3D printer or wrap some duct tape))

Be careful, the very first version of the board had 3.2mm holes; if you ordered the board more than a year ago, you may have this version.

i eventually get the m2 spacer/standoff so order it,
but here i am with a new issue and that is, i am try to test the working of some keys before soldering all of them, thus i perform the following tasks:

solder all the diodes.
flash the pro micros with kapl_v1.hex file given in the link of the repo( i do test my pro connecting two of its pins on opposite sides and they are printing something on my device terminal app).
fit all the keys to the top plate and set it with pcb.
solder one key among all the keys to test whether it is working or not.

but sadly i am not getting any character when pressing that key, i also tried the reset button and that also seems to be not working( what i am doing is, why reflashing my pro micro rather than shorting the GND and RST pins i am pressing the reset button in the hope that it will start the flash in the qmk toolbox app like it did if shorted the pins directly but nothing is happening there).
i don't know what i am doing wrong so here is the picture of my setup

the circled key is the one i have souldered to test the pcb(i order the pcb for jlpcb so i am expecting it of good quality).
so once again can you guide toward any pointer which can help me figure out.

This pin look bad soldered. If the contact is poor, the switches will operate very unstable.
You missed one diode.
Did you remember to solder the diode under the controller?

For just testing pcb you can use tweeters. This is not very convenient, but it is much easier than soldering each switch for testing. https://www.youtube.com/shorts/-FCV9InV3x8
It’s better to write to me in telegram or email, it will be more productive. Contacts are in the profile.

thanks again, yes i soldered the diode(it is not visible though because of the pro micro), but yes i do missed the second diode that you pointed out(thanks again), and also thanks for the telegram info it will be much helpfull for me.


Support Google auth as user to enable DSS data loading from EC2 instance

Identify how to do programmatic Google auth as/for user account, so DSS data loading can be run on an EC2 instance
┆Issue is synchronized with this JIRA Story
┆Project Name: CGP-Data Loader
┆Issue Number: LOAD-11
┆Epic: Loader Improvements

➤ Michael Baumann commented:
This can probably be addressed working out and documenting the steps for the appropriate use of the “gcloud auth” command and associated credential files.

➤ Michael Baumann commented:
This will also be facilitated by current work on TLP-720.

➤ Michael Baumann commented:
This issue will be resolved by the work Lon is doing for TLP-720.

Closed by #49.


python312Packages.cirq-core: 1.4.0 -> 1.4.1

Diff: https://github.com/quantumlib/cirq/compare/refs/tags/v1.4.0...v1.4.1
Changelog: https://github.com/quantumlib/Cirq/releases/tag/v1.4.1
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 338568 run on x86_64-linux 1

40 packages built:

python311Packages.cirq
python311Packages.cirq-aqt
python311Packages.cirq-aqt.dist
python311Packages.cirq-core
python311Packages.cirq-core.dist
python311Packages.cirq-google
python311Packages.cirq-google.dist
python311Packages.cirq-ionq
python311Packages.cirq-ionq.dist
python311Packages.cirq-pasqal
python311Packages.cirq-pasqal.dist
python311Packages.cirq-rigetti
python311Packages.cirq-rigetti.dist
python311Packages.cirq-web
python311Packages.cirq-web.dist
python311Packages.cirq.dist
python311Packages.duet
python311Packages.duet.dist
python311Packages.stim
python311Packages.stim.dist
python312Packages.cirq
python312Packages.cirq-aqt
python312Packages.cirq-aqt.dist
python312Packages.cirq-core
python312Packages.cirq-core.dist
python312Packages.cirq-google
python312Packages.cirq-google.dist
python312Packages.cirq-ionq
python312Packages.cirq-ionq.dist
python312Packages.cirq-pasqal
python312Packages.cirq-pasqal.dist
python312Packages.cirq-rigetti
python312Packages.cirq-rigetti.dist
python312Packages.cirq-web
python312Packages.cirq-web.dist
python312Packages.cirq.dist
python312Packages.duet
python312Packages.duet.dist
python312Packages.stim
python312Packages.stim.dist

@ofborg eval


chore(#4,#5): Manual deploy test

chore(#4,#5): Manual deploy test

Code Coverage

Total Project Coverage
100.00%
:white_check_mark:


thunderbird, thunderbird-bin: 60.3.0 -> 60.3.1

Motivation for this change

Bug fixes

https://www.thunderbird.net/en-US/thunderbird/60.3.1/releasenotes/
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Fits CONTRIBUTING.md.

@Mic92 It would make sense. I can do the porting to 18.03 so we get that updated there as well for those few folks that might still be stock on there.