Make Logo pixel perfect

What does this change?
Make the jsr logo pixel-perfect.
Use a much smaller canvas size, where each unit result in one square and ensure the view box has a 2:1 ratio to ensure perfect scaling in relation to font-size and UI elements
See interactive demo for how this was generated

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

@mxdvl The original version of the animated SVG included two "squares" worth of white space all around the logo inside the viewbox, in order to allow space for the squares to be up to 200% out of position before sliding into place with the animation. (You'll notice with this version, some of the squares initially render out of bounds of the SVG and are invisible until they slide into the viewbox.)

Are you talking about this specific PR? I'm only modifying the static logo here and not touching the animated one. I think you might be referring to a follow-up branch I've got in my fork, that's not ready yet for PR.
I do have a question about static SVG assets… are they used anywhere? I could not find any reference in the code.

Are you talking about this specific PR? I'm only modifying the static logo here and not touching the animated one. I think you might be referring to a follow-up branch I've got in my fork, that's not ready yet for PR.
I do have a question about static SVG assets… are they used anywhere? I could not find any reference in the code.

Yeah, I must have seen the follow-up. The homepage animated logo was definitely affected in the branch I looked at.
The logo-square.svg file is used in _app.tsx. The main logo.svg file is used on the homepage and package score page, in addition to a couple other places as images.

The logo-square.svg file is used in _app.tsx. The main logo.svg file is used on the homepage and package score page, in addition to a couple other places as images.

Would you like me to update these assets as part of this PR?

@josh-collinsworth does this still require changes?

@mxdvl Sorry for letting this sit for so long.
My main hesitation here is in making the logo use a stroke to render, and relying on the stroke as a part of the design. Maybe I'm being overly cautious, but I've come across some edge cases where strokes can vary when SVGs are ported between platforms or softwares (the thickness, the cap, etc.)
Again, maybe that's excessive caution. But could we not achieve this desired outcome with the original SVG (or, at least, an SVG that is fully expanded and which doesn't use strokes?)
Here's an example of what I mean; it's essentially applying this PR's ideas to the original SVG. https://github.com/jsr-io/jsr/pull/402

I just like to keep SVGs as "expanded" as possible, to borrow the Illustrator terminology.

I think that's probably overly cautious but definitely not my call to make in this case. I'll expand the SVG while keeping the smaller viewBox where each unit maps to one “block” for slightly reduced filesize.


Replaced Depricated tf.contrib.deprecated.scalar_summary with tf.compat.v1.summary.scalar

This is slight change in the Summary ProtoBuf. Initially the project writers have used deprecated method scalar_summary() of tf.contrib.deprecated. But since, It may be excluded in upcomming version of tensorflow. @roadjiang sir, please review this PR.
Thank you for this awesome project. It helped me very much in my work. This also tends to work with Conventional Deep Neural Nets with structured data.

CLA check

cla check

CLA check2

Try again


Unexpected behavior when asserting an array of hashes with eq and hash_inlcuding

Subject of the issue
I have an array of hashes and am trying to assert that it has a certain number of hashes with containing a certain key with certain value. Using eq with hash_including seems like the right way to do it, but it fails.
fruits = [
  { name: 'apple', count: 3 },
  { name: 'banana', count: 7 },
  { name: 'orange', count: 52 },
]

expect(fruits).to eq([
  hash_including(name: 'apple'),
  hash_including(name: 'banana'),
  hash_including(name: 'orange'),
])

contain_exactly and include work but I care about the size of the array and the order of its elements.
Your environment

Ruby version: 2.7.4
rspec-expectations version: 3.11.0

Steps to reproduce
# frozen_string_literal: true

begin
  require "bundler/inline"
rescue LoadError => e
  $stderr.puts "Bundler version 1.10 or later is required. Please update your Bundler"
  raise e
end

gemfile(true) do
  source "https://rubygems.org/"

  gem "rspec", "3.7.0"
end

puts "Ruby version is: #{RUBY_VERSION}"
require 'rspec/autorun'

RSpec.describe 'array of hashes' do
  it 'returns 2' do
    fruits = [
      { name: 'apple', count: 3 },
      { name: 'banana', count: 7 },
      { name: 'orange', count: 52 },
    ]

    expect(fruits).to eq([
      hash_including(name: 'apple'),
      hash_including(name: 'banana'),
      hash_including(name: 'orange'),
    ])
  end
end

Expected behavior
I expect the assertion to pass.
Actual behavior
The assertion fails.

:wave: Use match rather than eq.


Running the demo shows navigation as Disabled

Just downloaded the src code for v 2.5.2 and tried to run the demo with npm start - it shows the navigation as disabled

Isn't this supposed to show the complete navigation?
There are also tons of errors when the local server starts for the first time:
 ERROR  in ./src/Breadcrumb2.js
Module not found: Error: Can't resolve 'classnames' in '/Users/toddysm/Documents/Development/Projects/Local/coreui-v2.5.2/src'
 @ ./src/Breadcrumb2.js 45:0-36 151:18-28
 @ ./src/index.js
 @ ./demo/src/containers/DefaultLayout/DefaultLayout.js
 @ ./demo/src/App.js
 @ ./demo/src/index.js
 @ multi ./node_modules/nwb/polyfills.js ./node_modules/nwb/node_modules/webpack-dev-server/client?/ (webpack)/hot/only-dev-server.js ./demo/src/index.js

 ERROR  in ./src/SidebarToggler.js
Module not found: Error: Can't resolve 'classnames' in '/Users/toddysm/Documents/Development/Projects/Local/coreui-v2.5.2/src'
 @ ./src/SidebarToggler.js 44:0-36 105:18-28
 @ ./src/index.js
 @ ./demo/src/containers/DefaultLayout/DefaultLayout.js
 @ ./demo/src/App.js
 @ ./demo/src/index.js
 @ multi ./node_modules/nwb/polyfills.js ./node_modules/nwb/node_modules/webpack-dev-server/client?/ (webpack)/hot/only-dev-server.js ./demo/src/index.js

 ERROR  in ./src/NavbarBrand.js
Module not found: Error: Can't resolve 'classnames' in '/Users/toddysm/Documents/Development/Projects/Local/coreui-v2.5.2/src'
 @ ./src/NavbarBrand.js 44:0-36 102:18-28
 @ ./src/index.js
 @ ./demo/src/containers/DefaultLayout/DefaultLayout.js
 @ ./demo/src/App.js
 @ ./demo/src/index.js
 @ multi ./node_modules/nwb/polyfills.js ./node_modules/nwb/node_modules/webpack-dev-server/client?/ (webpack)/hot/only-dev-server.js ./demo/src/index.js

 ERROR  in ./src/AsideToggler.js
Module not found: Error: Can't resolve 'classnames' in '/Users/toddysm/Documents/Development/Projects/Local/coreui-v2.5.2/src'
 @ ./src/AsideToggler.js 44:0-36 115:18-28
 @ ./src/index.js
 @ ./demo/src/containers/DefaultLayout/DefaultLayout.js
 @ ./demo/src/App.js
 @ ./demo/src/index.js
 @ multi ./node_modules/nwb/polyfills.js ./node_modules/nwb/node_modules/webpack-dev-server/client?/ (webpack)/hot/only-dev-server.js ./demo/src/index.js

Hi @toddysm
This local demo is just a testing app for @coreui/react library development.
To get rid of these errors, run:

npm install
npm run build
npm run start

You can find full blown working template/demo/source in CoreUI Free React Admin Template
or online demo in CoreUI Pro React.js Admin Template


feat(anvil): add --no-rate-limits flag

Motivation
Closes #3668. Implements the --no-rate-limits flag.
Solution
sets the CUPS for the anvil node provider to u64::MAX if the --no-rate-limits flag is present.
The one thing I'd like to discuss is being able to use --cups and --no-rate-limits together—I think it's a bit better to gracefully exit with an error saying that this is not possible, but doing so probably requires changing the into_node_config function signature.

haha same, i actually thought i called it --no-rate-limiting at first 😅
yeah i think --no-rate-limit is better as it's shorter/cleaner

tysm,
would appreciate a book PR for this flag and the reason why this is useful (non-localhost, non rate limited node)


[Bug]: Error with Lora

Is there an existing issue for this?

[X] I have searched the existing issues and checked the recent builds/commits

Are you using the latest version of the colab notebook?

[X] Yes, i am using the latest version from the repo

Which colab notebook did you use?
https://colab.research.google.com/github/camenduru/stable-diffusion-webui-colab/blob/v2.0/chillout_mix_webui_colab.ipynb
Which repo branch did the colab you use comes from ?
v2.0
Which tunnel did you use ?
--cloudflared
What happened?

I saw it created the image but Lora reported an error

Console logs
activating extra network lora with arguments [<modules.extra_networks.ExtraNetworkParams object at 0x7f6371c1cd00>, <modules.extra_networks.ExtraNetworkParams object at 0x7f6375c25910>, <modules.extra_networks.ExtraNetworkParams object at 0x7f637209cb20>]: RuntimeError
Traceback (most recent call last):
  File "/content/stable-diffusion-webui/modules/extra_networks.py", line 75, in activate
    extra_network.activate(p, extra_network_args)
  File "/content/stable-diffusion-webui/extensions-builtin/Lora/extra_networks_lora.py", line 23, in activate
    lora.load_loras(names, multipliers)
  File "/content/stable-diffusion-webui/extensions-builtin/Lora/lora.py", line 151, in load_loras
    lora = load_lora(name, lora_on_disk.filename)
  File "/content/stable-diffusion-webui/extensions-builtin/Lora/lora.py", line 83, in load_lora
    sd = sd_models.read_state_dict(filename)
  File "/content/stable-diffusion-webui/modules/sd_models.py", line 217, in read_state_dict
    pl_sd = safetensors.torch.load_file(checkpoint_file, device=device)
  File "/usr/local/lib/python3.8/dist-packages/safetensors/torch.py", line 100, in load_file
    result[k] = f.get_tensor(k)
RuntimeError: self.size(-1) must be divisible by 2 to view Byte as Half (different element sizes), but got 380031

  0% 0/40 [00:00<?, ?it/s]
  2% 1/40 [00:00<00:15,  2.58it/s]
  5% 2/40 [00:00<00:12,  2.94it/s]
  8% 3/40 [00:00<00:11,  3.09it/s]
 10% 4/40 [00:01<00:11,  3.16it/s]
 12% 5/40 [00:01<00:10,  3.19it/s]
 15% 6/40 [00:01<00:10,  3.22it/s]
 18% 7/40 [00:02<00:10,  3.23it/s]
 20% 8/40 [00:02<00:09,  3.23it/s]
 22% 9/40 [00:02<00:09,  3.22it/s]
 25% 10/40 [00:03<00:09,  3.25it/s]
 28% 11/40 [00:03<00:08,  3.24it/s]
 30% 12/40 [00:03<00:08,  3.23it/s]
 32% 13/40 [00:04<00:08,  3.22it/s]
 35% 14/40 [00:04<00:08,  3.22it/s]
 38% 15/40 [00:04<00:07,  3.20it/s]
 40% 16/40 [00:05<00:07,  3.19it/s]
 42% 17/40 [00:05<00:07,  3.18it/s]
 45% 18/40 [00:05<00:06,  3.18it/s]
 48% 19/40 [00:05<00:06,  3.17it/s]
 50% 20/40 [00:06<00:06,  3.16it/s]
 52% 21/40 [00:06<00:06,  3.16it/s]
 55% 22/40 [00:06<00:05,  3.16it/s]
 57% 23/40 [00:07<00:05,  3.13it/s]
 60% 24/40 [00:07<00:05,  3.15it/s]
 62% 25/40 [00:07<00:04,  3.14it/s]
 65% 26/40 [00:08<00:04,  3.14it/s]
 68% 27/40 [00:08<00:04,  3.14it/s]
 70% 28/40 [00:08<00:03,  3.15it/s]
 72% 29/40 [00:09<00:03,  3.14it/s]
 75% 30/40 [00:09<00:03,  3.11it/s]
 78% 31/40 [00:09<00:02,  3.12it/s]
 80% 32/40 [00:10<00:02,  3.12it/s]
 82% 33/40 [00:10<00:02,  3.12it/s]
 85% 34/40 [00:10<00:01,  3.13it/s]
 88% 35/40 [00:11<00:01,  3.12it/s]
 90% 36/40 [00:11<00:01,  3.11it/s]
 92% 37/40 [00:11<00:00,  3.11it/s]
 95% 38/40 [00:12<00:00,  3.10it/s]
 98% 39/40 [00:12<00:00,  3.09it/s]
100% 40/40 [00:12<00:00,  3.15it/s]
Total progress: 100% 40/40 [00:12<00:00,  3.12it/s]

try changing LoRA name to normal text, delete Chinese/Japanese text
use 1.5 version instead of 2.0

I have same exact problem like this a week ago

Hi @tranthai2k2 👋 is this solved with @zaviier's idea ❤ ? if not please open this issue again


[W5.7][W13-4]Anthony Halim

Review made for W5.7:
Good attempt! Easy to understand and simple code.


Exhausted searching IDs in known runs

Hi,
From time to time, we would see this error "Exhausted searching IDs in known runs". See the logs below. I bet this is caused by github API not finding the relevant run. That said, is there anything we can do to mitigate this problem? Thanks!
Mon, 07 Aug 2023 16:14:00 GMT Fetching Workflow ID for smokes-cron.yml...
Mon, 07 Aug 2023 16:14:00 GMT Fetched Workflow ID: 7198060
Mon, 07 Aug 2023 16:14:01 GMT Successfully dispatched workflow:
Mon, 07 Aug 2023 16:14:01 GMT   Repository: bay1inc/api-platform
Mon, 07 Aug 2023 16:14:01 GMT   Branch: refs/heads/main
Mon, 07 Aug 2023 16:14:01 GMT   Workflow ID: smokes-cron.yml
Mon, 07 Aug 2023 16:14:01 GMT   Workflow Inputs: {"envs":"staging-test,staging-live"}
Mon, 07 Aug 2023 16:14:01 GMT   Distinct ID: 29fc4ecb-c9d5-44d8-9216-aa22fc11ff89
Mon, 07 Aug 2023 16:14:01 GMT Attempt to extract run ID from steps...
Mon, 07 Aug 2023 16:14:03 GMT Exhausted searching IDs in known runs, attempt 1...
Mon, 07 Aug 2023 16:14:11 GMT Exhausted searching IDs in known runs, attempt 2...
Mon, 07 Aug 2023 16:14:18 GMT Exhausted searching IDs in known runs, attempt 3...
Mon, 07 Aug 2023 16:14:26 GMT Exhausted searching IDs in known runs, attempt 4...
Mon, 07 Aug 2023 16:14:33 GMT Exhausted searching IDs in known runs, attempt 5...
Mon, 07 Aug 2023 16:14:41 GMT Exhausted searching IDs in known runs, attempt 6...
Mon, 07 Aug 2023 16:14:48 GMT Exhausted searching IDs in known runs, attempt 7...
Mon, 07 Aug 2023 16:14:56 GMT Exhausted searching IDs in known runs, attempt 8...
Mon, 07 Aug 2023 16:15:03 GMT Exhausted searching IDs in known runs, attempt 9...
Mon, 07 Aug 2023 16:15:11 GMT Exhausted searching IDs in known runs, attempt 10...
Mon, 07 Aug 2023 16:15:18 GMT Exhausted searching IDs in known runs, attempt 11...
Mon, 07 Aug 2023 16:15:26 GMT Exhausted searching IDs in known runs, attempt 12...
Mon, 07 Aug 2023 16:15:33 GMT Exhausted searching IDs in known runs, attempt 13...
Mon, 07 Aug 2023 16:15:41 GMT Exhausted searching IDs in known runs, attempt 14...
Mon, 07 Aug 2023 16:15:48 GMT Exhausted searching IDs in known runs, attempt 15...
Mon, 07 Aug 2023 16:15:55 GMT Exhausted searching IDs in known runs, attempt 16...
Mon, 07 Aug 2023 16:16:04 GMT Exhausted searching IDs in known runs, attempt 17...
Mon, 07 Aug 2023 16:16:12 GMT Exhausted searching IDs in known runs, attempt 18...
Mon, 07 Aug 2023 16:16:20 GMT Exhausted searching IDs in known runs, attempt 19...
Mon, 07 Aug 2023 16:16:27 GMT Exhausted searching IDs in known runs, attempt 20...
Mon, 07 Aug 2023 16:16:35 GMT Exhausted searching IDs in known runs, attempt 21...
Mon, 07 Aug 2023 16:16:42 GMT Exhausted searching IDs in known runs, attempt 22...
Mon, 07 Aug 2023 16:16:55 GMT Exhausted searching IDs in known runs, attempt 23...
Mon, 07 Aug 2023 16:16:57 GMT Exhausted searching IDs in known runs, attempt 24...
Mon, 07 Aug 2023 16:17:05 GMT Exhausted searching IDs in known runs, attempt 25...
Mon, 07 Aug 2023 16:17:12 GMT Exhausted searching IDs in known runs, attempt 26...
Mon, 07 Aug 2023 16:17:20 GMT Exhausted searching IDs in known runs, attempt 27...
Mon, 07 Aug 2023 16:17:27 GMT Exhausted searching IDs in known runs, attempt 28...
Mon, 07 Aug 2023 16:17:35 GMT Exhausted searching IDs in known runs, attempt 29...
Mon, 07 Aug 2023 16:17:42 GMT Exhausted searching IDs in known runs, attempt 30...
Mon, 07 Aug 2023 16:17:50 GMT Exhausted searching IDs in known runs, attempt 31...
Mon, 07 Aug 2023 16:17:57 GMT Exhausted searching IDs in known runs, attempt 32...
Mon, 07 Aug 2023 16:18:05 GMT Exhausted searching IDs in known runs, attempt 33...
Mon, 07 Aug 2023 16:18:13 GMT Exhausted searching IDs in known runs, attempt 34...
Mon, 07 Aug 2023 16:18:20 GMT Exhausted searching IDs in known runs, attempt 35...
Mon, 07 Aug 2023 16:18:28 GMT Exhausted searching IDs in known runs, attempt 36...
Mon, 07 Aug 2023 16:18:35 GMT Exhausted searching IDs in known runs, attempt 37...
Mon, 07 Aug 2023 16:18:43 GMT Exhausted searching IDs in known runs, attempt 38...
Mon, 07 Aug 2023 16:18:50 GMT Exhausted searching IDs in known runs, attempt 39...
Mon, 07 Aug 2023 16:19:05 GMT Exhausted searching IDs in known runs, attempt 40...
Mon, 07 Aug 2023 16:19:07 GMT Exhausted searching IDs in known runs, attempt 41...
Mon, 07 Aug 2023 16:19:12 GMT Error: Failed to complete: Timeout exceeded while attempting to get Run ID
Mon, 07 Aug 2023 16:19:12 GMT Warning: Does the token have the correct permissions?
Mon, 07 Aug 2023 16:19:12 GMT Error: Timeout exceeded while attempting to get Run ID

Apologies, I missed this issue being raised
Yes typically this would indicate that the run fails to start, do you find that the run is starting and this action is failing to get the ID?
Mitigate in which sense? Are you using this action to simply kick off another action and are not concerned for the ID? Or?

What I see often is that a run was successfully triggered (the "Distinct ID" matches the expected ID) but this action couldn't find it after many tries.

Hmmm I wonder if the rework i'm doing for v2 will help. In the interim, can you please post the full log with debug logging enabled?
You can enable this by adding a secret ACTIONS_RUNNER_DEBUG with true

Good idea. I just added that secret. Will keep you posted. Thanks!

Happy to re-open this should there still be any issues, let me know :)

So this happened today.
It failed to find a workflow run with the matching dispatch id:

However, the workflow run was dispatched successfully (the id matches)

How long is it between the job starting and the action dispatching? do you use custom runners or gh runners?

The action was dispatched at Thu, 14 Sep 2023 04:44:27 GMT and the job started running at Thu, 14 Sep 2023 04:44:35 GMT so it took 8 seconds.
We use self-hosted runners for this job. We used to use self-hosted runners and had this problem as well.

Weird, does the output for return-dispatch to catch happen before anything else?

Sorry what you mean exactly?

Ah sorry, didn't explain myself very well, does this step happen before any other steps?
      - name: echo distinct ID ${{ github.event.inputs.distinct_id }}
        run: echo ${{ github.event.inputs.distinct_id }}

No, that job only has one step like this:
  id:
    runs-on: [infra, us-central1, build-small]
    if: ${{ github.event.inputs.distinct_id != null }}
    steps:
      - name: Distinct ID ${{ github.event.inputs.distinct_id }}
        run: echo ${{ github.event.inputs.distinct_id }}

@jduan-highnote Could it be because your targeted repository triggers a lot of workflows on regular basis? I see that the API call to get the workflow runs only fetches 5 when a branch name is specified as ref, and only 10 otherwise:
https://github.com/Codex-/return-dispatch/blob/78fb303cfd7699e9c743c2c76479906f0e970b80/src/api.ts#L135-L147
If more than 5 workflows have been triggered at the same time, then your run might never be in the first page of results and therefore impossible to find. It would be great to increase the per_page value or make it configurable.

@pmrotule that's great insight. It's certainly possible that the target workflow has more than 5 concurrent runs at some point. It'd be great to make this a configurable param! @Codex-

@jduan-highnote I've cut a new release with some enhancements courtesy of @pmrotule
Let me know how it goes (v1 tag has been updated so if you're using the v1 tag you shouldn't have to change anything)

@Codex- can you given a summary of the recent enhancements? We do use v1 across the board so we will pick up the enhancements automatically.

An octokit iterator is now being used to check for runs across multiple API response pages

Nice! I'll keep you posted.

An octokit iterator is now being used to check for runs across multiple API response pages

@Codex- The iterator is only used to get the workflow id within getWorkflowId, but getWorkflowRunIds is still only looking at the first 5 runs:
https://github.com/Codex-/return-dispatch/blob/dbbbe1d543b3d42aee5613cd7eb4dea947dc54a5/src/api.ts#L143-L160
So I think that my assumption from last month is still valid:

If more than 5 workflows have been triggered at the same time, then your run might never be in the first page of results and therefore impossible to find.

A nice way to solve this would be to use an iterator within getWorkflowRunIds as well, but with a limit since going through all workflow runs in the history of a repository could take a very long time. So something like this:
const workflowRunIterator = octokit.paginate.iterator(
  octokit.rest.actions.listWorkflowRuns,
  {
    owner: config.owner,
    repo: config.repo,
    workflow_id: workflowId,
    per_page: 20,
  },
);
let runIds: number[] = [];
let page = 1;
const maxPage = 5;

for await (const response of workflowRunIterator) {
  if (response.status !== 200) {
    throw new Error(
      `Failed to get Workflow runs, expected 200 but received ${response.status}`,
    );
  }
  // wrong type definition
  const runs: typeof response.data.workflow_runs = response.data;
  runIds = runs.map((workflowRun) => workflowRun.id);

  if (runIds.length || page === maxPage) {
    break;
  }
  page++;
}

I am also running into this issue:

Successfully dispatched workflow:
Repository: ***
Branch: ***
Workflow ID: ***.yml
Distinct ID: 348a1190-04f9-4c90-b5e5-ef3608dd4265
Attempt to extract run ID from steps...
Exhausted searching IDs in known runs, attempt 1...
Exhausted searching IDs in known runs, attempt 2...
Exhausted searching IDs in known runs, attempt 3...
Exhausted searching IDs in known runs, attempt 4...
Exhausted searching IDs in known runs, attempt 5...
Exhausted searching IDs in known runs, attempt 6...
Exhausted searching IDs in known runs, attempt 7...
Exhausted searching IDs in known runs, attempt 8...
Exhausted searching IDs in known runs, attempt 9...
Exhausted searching IDs in known runs, attempt 10...
Error: getWorkflowRunJobs: An unexpected error has occurred: Server Error
Error: Failed to complete: Server Error
Warning: Does the token have the correct permissions?
Error: Server Error

The weirdest thing is that it will keep attempting to find the ID while the successfully triggered pipeline is running and then hits the Server Error or timeout error once the pipeline is done.  If there's anything I can add to help with troubleshooting, please let me know.  Thank you for working on this, this functionality should be included in Github Actions by default.

When the search is being exhausted, if you do an API request yourself, is the run present?

Thank you for working on this, this functionality should be included in Github Actions by default.

Totally agree, really hoping we see it one day

@ChrisPage-AT  When the search is being exhausted, if you do an API request yourself, is the run present?

Thank you for working on this, this functionality should be included in Github Actions by default.

Totally agree, really hoping we see it one day

@Codex- Using the api calls listed in your readme, I can find the triggered pipeline with List Workflow Runs and use the id to see the jobs with List Jobs for a workflow run.  However I am noticing that neither of those calls seem to list the distinct_id that I'm passing in.  Are you doing a different call to obtain that?

I started seeing this error recently. You can see that the workflow was dispatched successfully and the ID matches.

Server error? Unusual, can you enable debug logging?
https://docs.github.com/en/actions/monitoring-and-troubleshooting-workflows/enabling-debug-logging

Interesting. I wasn't aware of enabling debug logs for runners. Should I enable both runner & step debug logs?

I think step but I'd do both to get a full picture

I enabled debug log a few weeks ago and today the problem happened for the first time. Please see the attached logs.

Thanks for that @jduan-highnote  , I'll have a dig when I get a chance. Really odd that the ID being returned is so short, almost looks truncated.
I've deleted your post so as to not reveal anything about your infra

Here's another occurrence if it helps:
Fri, 26 Apr 2024 19:25:45 GMT
Run codex-/return-dispatch@v1
Fri, 26 Apr 2024 19:25:45 GMT Fetching Workflow ID for prbox_update_routing.yaml...
Fri, 26 Apr 2024 19:25:45 GMT Fetched Workflow ID: 88382506
Fri, 26 Apr 2024 19:25:46 GMT Successfully dispatched workflow:
Fri, 26 Apr 2024 19:25:46 GMT   Repository: bay1inc/devops
Fri, 26 Apr 2024 19:25:46 GMT   Branch: main
Fri, 26 Apr 2024 19:25:46 GMT   Workflow ID: prbox_update_routing.yaml
Fri, 26 Apr 2024 19:25:46 GMT   Workflow Inputs: {"services":"non-pci","tenants":"test"}
Fri, 26 Apr 2024 19:25:46 GMT   Distinct ID: f9c198f0-84b8-4943-a7cf-31be5bcec42b
Fri, 26 Apr 2024 19:25:46 GMT Attempt to extract run ID from steps...
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]Attempting to fetch Run IDs for Workflow ID 88382506
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]failed to get branch for ref: main, please raise an issue with this git ref.
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]Fetched Workflow Runs:
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]  Repository: bay1inc/devops
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]  Branch: undefined
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]  Workflow ID: 88382506
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]  Runs Fetched: [8852916369,8852872434,8852351822,8852314544,8852302751,8852270907,8851776517,8851726747,8851694157,8851681964]
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]Attempting to get step names for Run IDs: [8852916369,8852872434,8852351822,8852314544,8852302751,8852270907,8851776517,8851726747,8851694157,8851681964]
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]Fetched Workflow Run Job Steps:
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]  Repository: bay1inc/devops
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]  Workflow Run ID:<PHONE_NUMBER>
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]  Jobs Fetched: [24312583511,24312583707,24312583931,24312591506,24312619999,24312642954]  Steps Fetched: [Set up job,Print Input Values,Clone Devops,Generate Matrix JSON,Post Clone Devops,Complete job,Distinct ID 61897684-539d-4539-b241-54ec6159841e,Set up runner,Setup K8s auth,Deploying Apps,Complete runner]
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]Fetched Workflow Run Job Steps:
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]  Repository: bay1inc/devops
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]  Workflow Run ID:<PHONE_NUMBER>
Fri, 26 Apr 2024 19:25:46 GMT ##[debug]  Jobs Fetched: [24312437185,24312437385,24312437629,24312444058,24312464693,24312481469]  Steps Fetched: [Set up job,Print Input Values,Clone Devops,Generate Matrix JSON,Post Clone Devops,Complete job,Distinct ID fcc9476c-2cae-43a1-8850-ae2df9728507,Set up runner,Setup K8s auth,Deploying Apps,Complete runner]
Fri, 26 Apr 2024 19:25:47 GMT ##[debug]Fetched Workflow Run Job Steps:
Fri, 26 Apr 2024 19:25:47 GMT ##[debug]  Repository: bay1inc/devops
Fri, 26 Apr 2024 19:25:47 GMT ##[debug]  Workflow Run ID:<PHONE_NUMBER>
Fri, 26 Apr 2024 19:25:47 GMT ##[debug]  Jobs Fetched: [24310882803,24310883124,24310883576,24310892691,24310930047,24310957296]  Steps Fetched: [Set up job,Print Input Values,Clone Devops,Generate Matrix JSON,Post Clone Devops,Complete job,Distinct ID 0fec8571-ebc2-49f5-9364-747a1b6d3649,Set up runner,Setup K8s auth,Deploying Apps,Complete runner]
Fri, 26 Apr 2024 19:25:47 GMT ##[debug]Fetched Workflow Run Job Steps:
Fri, 26 Apr 2024 19:25:47 GMT ##[debug]  Repository: bay1inc/devops
Fri, 26 Apr 2024 19:25:47 GMT ##[debug]  Workflow Run ID:<PHONE_NUMBER>
Fri, 26 Apr 2024 19:25:47 GMT ##[debug]  Jobs Fetched: [24310757841,24310757992,24310758179,24310764698,24310791043,24310814079]  Steps Fetched: [Set up job,Print Input Values,Clone Devops,Generate Matrix JSON,Post Clone Devops,Complete job,Distinct ID 6822826a-513e-4f60-8535-fe7573de521e,Set up runner,Setup K8s auth,Deploying Apps,Complete runner]
Fri, 26 Apr 2024 19:25:57 GMT Error: getWorkflowRunJobs: An unexpected error has occurred: Server Error
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]HttpError: Server Error
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]    at file:///runner/_work/_actions/codex-/return-dispatch/v1/dist/index.mjs:19832:26
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]    at async getWorkflowRunJobSteps (file:///runner/_work/_actions/codex-/return-dispatch/v1/dist/index.mjs:23186:22)
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]    at async run (file:///runner/_work/_actions/codex-/return-dispatch/v1/dist/index.mjs:23269:25)
Fri, 26 Apr 2024 19:25:57 GMT Error: Failed to complete: Server Error
Fri, 26 Apr 2024 19:25:57 GMT Warning: Does the token have the correct permissions?
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]HttpError: Server Error
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]    at file:///runner/_work/_actions/codex-/return-dispatch/v1/dist/index.mjs:19832:26
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]    at async getWorkflowRunJobSteps (file:///runner/_work/_actions/codex-/return-dispatch/v1/dist/index.mjs:23186:22)
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]    at async run (file:///runner/_work/_actions/codex-/return-dispatch/v1/dist/index.mjs:23269:25)
Fri, 26 Apr 2024 19:25:57 GMT Error: Server Error
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]Node Action run completed with exit code 1
Fri, 26 Apr 2024 19:25:57 GMT ##[debug]Finishing: Trigger routing updates

@Codex- do you have enough logs for debugging? I'd like to turn off debug logs for our github actions.

@jduan-highnote Are you sure your PAT has the required permissions : https://github.com/Codex-/return-dispatch?tab=readme-ov-file#permissions-required (as it's stated in the logs)
Fri, 26 Apr 2024 19:25:57 GMT Warning: Does the token have the correct permissions?

Yes it does. I only see this failure < 1% of the time.
On Sat, May 4, 2024 at 8:50 AM Stephdotnet @.***> wrote:

@jduan-highnote https://github.com/jduan-highnote Are you sure your PAT
has the required permissions :
https://github.com/Codex-/return-dispatch?tab=readme-ov-file#permissions-required
(as it's stated in the logs)
Fri, 26 Apr 2024 19:25:57 GMT Warning: Does the token have the correct
permissions?
—
Reply to this email directly, view it on GitHub
https://github.com/Codex-/return-dispatch/issues/183#issuecomment-2094271028,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV6IUATQIFEC2VIJRHDZAZTZAT7T7AVCNFSM6AAAAAA3HIFJAGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJUGI3TCMBSHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Codex- did you get a chance to look into this? I got another failure and I confirmed the dispatch actually worked (the ID matched)

I've been looking on and off but it seems as though there are just bad results from the API itself, looking to add some additional debug logging to see what the API is returning because something isn't right and it would either always work or never work, not just somehow end up with the truncated ID 🤔

An unexpected error has occurred: Server Error

When this error happens, can the code retries it a few times?

@Codex- did you get a chance to look more into this? Thanks!
I found another occurrence of this problem yesterday:
Attempt to extract run ID from steps...
Error: getWorkflowRunJobs: An unexpected error has occurred: Server Error
Error: Failed to complete: Server Error
Warning: Does the token have the correct permissions?
Error: Server Error

@Codex- did you get a chance to look more into this? Thanks!
I found another occurrence of this problem yesterday:
Attempt to extract run ID from steps...
Error: getWorkflowRunJobs: An unexpected error has occurred: Server Error
Error: Failed to complete: Server Error
Warning: Does the token have the correct permissions?
Error: Server Error

@jduan-highnote server errors have been more common recently with some github issues that have been happening off and on.
I've made some improvements to how I handle this, they have not yet been released but can be tested as the bundle has been generated and pushed for the changes:

Branch: https://github.com/Codex-/return-dispatch/tree/feat/improve_retry_strategy
Draft PR: https://github.com/Codex-/return-dispatch/pull/241/files

Yes, the number of incidents github.com has is crazy to say the least. How do I try out this branch in our github action workflows?

You should be able to update your workflow uses field for that step to point to the branch: uses: codex-/return-dispatch@feat/improve_retry_strategy

Got it. BTW, we have metrics on how often this error happens. This is the past data. It doesn't happen often. I'll switch to @feat/improve_retry_strategy but it may take a while before I can see how effective it is.

Server Errors are now captured and retried for n times so it'll be interested to see if that'll be enough to keep it going with more stability

@jduan-highnote how's it looking so far? :)

I merged the change to try this out on 8/19 and so far there's been no issues. That said, I think we need to wait a bit longer since the problem was happening about once a month before.

@Codex- created here: https://github.com/Codex-/return-dispatch/issues/253

@jduan-highnote can you please move to use v2, as v2 includes the improvements for server error retries
Once you've moved I'll be removing the experimental branch

@Codex- I've upgraded to v2. You can remove the experimental branch now. Thanks!

@jduan-highnote how have things been looking since the experimental branch and the release of v2?

So far so good! Free feel to close this ticket. I'll let you know if I see problems again.


Add CI to this project

This changeset provides building and unit testing on Ubuntu (latest version of Swift as of Jan 30th 2020) and MacOS (latest version of Swift).

Hey there,
Could this be merged into the Silica project?


It should be possible to import several files or files in a folder

At this moment, it is only possible to important 1 file from the file explorer.
A "Solution" can be composed of several objects and thus, several XML files.
These files can be in a single folder.
To ease the deployments of "artifacts", it should be possible to import several files.

There are a few ways to import files:

Import one current file
Import all uncommitted changes
Import all open files in the workspace
Build and import the whole SSB set

On Thu, Mar 16, 2023 at 9:18 AM yannick-beot-sp @.***>
wrote:

At this moment, it is only possible to important 1 file from the file
explorer.
A "Solution" can be composed of several objects and thus, several XML
files.
These files can be in a single folder.
To ease the deployments of "artifacts", it should be possible to import
several files.
—
Reply to this email directly, view it on GitHub
https://github.com/lispercat/sailpoint-iiq-dev-accelerator/issues/40,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACCZB5KDS6A2WVWRLNAU3TW4MHJZANCNFSM6AAAAAAV5GOHKE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Ctrl-Shift-p and type "iiq deploy open files"
On Thu, Mar 16, 2023 at 9:41 AM yannick-beot-sp @.***>
wrote:

How do you "Import all open files in the workspace"?
—
Reply to this email directly, view it on GitHub
https://github.com/lispercat/sailpoint-iiq-dev-accelerator/issues/40#issuecomment-1471973614,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACCZB4OF2A7BMYIDACIZFDW4MJ6XANCNFSM6AAAAAAV5GOHKE
.
You are receiving this because you commented.Message ID:
@.***>

Merged


Viewing actions you don't have permission to do

Describe the Bug
This might be in other places but this bug is specifically for images.
If a user is allowed to VIEW pictures but NOT DELETE them the user will still have access to the delete button in the image select menu.

If I click on the confirm button, I will get a bugged view of the image select screen that is only temporary.

The image doesn't delete (good) but the button still shouldn't be visible. Just a minor QoL thing.
Steps to Reproduce

Create a role that can view images but not delete them.
Have a book where that role can view/create/edit but not delete.
Upload an image to a page is said book.
Save page.
Open page and try to delete the image from the image select menu.
Observe.

Expected Behaviour
Delete button to be not visible or at least not-clickable.
Screenshots or Additional Context
No response
Browser Details
No response
Exact BookStack Version
v22.07.3
PHP Version
No response
Hosting Environment
Ubuntu 20.04 LTS in a LXC container.

Thanks for reporting @Mailstorm-ctrl.
Can confirm that permission-based visibility of this button was not active.
Have added within commit fbef0d06f2d61030183d5b85a49620f4d23717b1, to be part of the next feature release.


not working with ng-class

I used ui-scroll for li elements, which are also annotated with ng-class. Then the ng-class is not correctly evaluated.

If the ng-class is used in a sub element, everything works as expected

@dhilt is this the same issue you opened a bug report with angular team?

@Repalogic Here is the demo with ng-class for ui-scroll: http://jsfiddle.net/dhilt/ypdbmrt8/
So I can't reproduce the issue. Could you share your demo?
@mfeingold It was another story https://github.com/angular/angular.js/issues/12813, but some correlations are possible...


chore: gateway-cli info cleanup

Cleans up a bunch of stuff about gateway-cli info that has been bugging me for awhile.

Removes ClientConfig from the info command. ClientConfig can still be retrieved using gateway-cli config command.
Fixes gateway-cli config command
Removes route hints and routing fees from info. Routing fees are already included in each individual federation. Route hints can be acquired by generating an invoice.
Adds the gateway's API URL
Adds the gateway's Lightning Mode for determining what type of node it is.

Not currently backwards compatible.

Confirmed https://github.com/fedimint/fedimint/pull/5804 fixes backwards compatibility tests for this PR

A couple of questions about test coverage, but otherwise LGTM!

Backport failed for releases/v0.4, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin releases/v0.4
git worktree add -d .worktree/backport-5803-to-releases/v0.4 origin/releases/v0.4
cd .worktree/backport-5803-to-releases/v0.4
git switch --create backport-5803-to-releases/v0.4
git cherry-pick -x ba85693faf5851c1ba3f9f3920220a3b8fc8a3e9


Fix Question Filtering

Overview
StackedChart items ('agree or disagree') weren't being matched by the
question search, because for these items multiple survey responses are
nested under one question code. We therefore need to compare the
query to multiple responses for these questions, using the question code
instead of the response code.
Connects #78
Demo

Testing Instructions

On staging, select some countries and click 'next'.
Enter 'expense' into the search bar - you will see three questions found.
In the Netlify preview for this PR, select some countries and click 'next'. (You can find the Netlify preview link near the merge button below - where it says, 'deploy/netlify' click the 'details' link).
Enter 'expense' into the search bar - you will see four questions found.
Select any item and enter a new search that doesn't match that item. The selected item should remain visible, and appropriate items should be found matching your query.

Thank you for the review!


Problem with device's filter

Hello, i run genie learn or pyats learn and use --device filter so that learning runs for several devices. When i do it an error occurs. If i run lerning for one device it works. For example, i run:
genie learn all --testbed-file ../pod/testbed.yml --output ../pod/network_state/running/ --devices "device1,device2"
And i see an error:
local variable 'dev' referenced before assignment
But command below works:
genie learn all --testbed-file ../pod/testbed.yml --output ../pod/network_state/running/ --devices "device1"

Hi @nazarko-es ,
As the documentation describes, please use a list of devices instead.
Thanks

I can't open this link, fix it please

Hi @nazarko-es ,
Please try this one: https://developer.cisco.com/docs/genie-docs/

Thank you, this link is work, but i don't see in documentation about list format. How must i give list-type to genie learn? I tried to use follow formats:
--devices device1,device2
--devices "device1,device2"
--devices [device1,device2]
--devices "device1","device2"
All of them don't work

Hi @nazarko-es ,
Sorry about that, it is a typo in our documentation. I will fix it today.
Please try --devices device1 device2

Thanks, it is work. Also genie learn --help have says the same thing(List of devices, comma separated)


String literal highlighting goes haywire after seeing \"

This code:
https://github.com/dart-lang/pub/blob/master/lib/src/package.dart#L256
duplicated here:
  files = files.map((file) {
    if (Platform.operatingSystem != 'windows') return "$dir/$file";
    return "$dir\\${file.replaceAll("/", "\\")}";

confuses the string literal syntax highlighter - it thinks the quote here continues onto the following lines.

It looks like it's breaking at \\".


Upload /themes/nav-community-v1/static/images/uploads/screen-shot-2018-06-06-at-7.29.26-pm.png

Automatically generated by Netlify CMS

The CMS created this as a separate PR - but it's just an image for the weekly report

@zanuka @IfaqYurmama can 1 of you approve this image


autocomplete widget

new widget for select_one minimal with appearance="autocomplete" (and alias "search")

https://docs.google.com/spreadsheets/d/1MFcIVduKyWeoaWnJaBL82M9vOiubOGhAil0CE_VXKnU/edit#gid=0

http://selectize.github.io/selectize.js/

[ ] multiple languages
[ ] grid theme
[ ] printing
[ ] mobile (touchscreen)

f213566cf5e1eb0c7d536566d9e5417b59bac042


Updating CHS Translation [2]

to update & keep the same with current translation patch for main game

Consider adding translations for Act. 2 yotogi classes and noon works (added in version 1.33).


Extended error codes (MS-WEBDAVE)

The documentation can be downloaded from Microsoft: MS-WEBDAVE.
The extended error codes are returned using the X-MSDAVEXT_ERROR header. A description of the header format can be found in the MS-WDV documentation, chapter 2.2.3.

Is it supported???

No, not yet. It only implements the standard as defined by the RFCs for WebDAV class 1 and 2.


[Bug]: Installation steps - no matching manifest for linux/arm/v7 in the manifest list entries

Is there an existing issue for this?

[X] I have searched the existing issues

Example public repository
coolify
Description
I'm using a Raspberry Pi 3 with a fresh standard Debian distribution (Just flashed it with Raspberry Pi Imager v1.7.3)

When I launch install.sh everything goes fine until we try to docker pull
When running the script, the error message is the following
Pulling Coolify latest image (3.12.31) from ghcr.io.
Cannot find any Coolify image. Please check your internet connection. Exiting.
./install.sh exited unexpectedly with status: 1

So I tried to sudo docker pull ghcr.io/coollabsio/coolify:3.12.30, but then I get
3.12.30: Pulling from coollabsio/coolify
no matching manifest for linux/arm/v7 in the manifest list entries

Is there some other versions I can try to install coolify on my Raspberry?
Thanks and keep up the work,
Regards
Steps To Reproduce
For the first error message, run sudo bash ./install.sh on a fresh Debian distribution on a Raspberry Pi 3
For the second error message, run sudo docker pull ghcr.io/coollabsio/coolify:3.12.30 or 3.12.31
Version
3.12.31

Can you post the output of uname -m?

Hi @codesalatdev and thanks for your answer, output is armv7l

Hi @codesalatdev and thanks for your answer, output is armv7l
In addition, the model of the Pi is Raspberry Pi 3 Model B Plus Rev 1.3

You seem to have installed a 32-bit OS on your 64-bit Raspberry Pi. Coolify is not built for armv7l. A Raspberry Pi 3 is ARMv8 tho, which supports 64-bit (aarch64).
See https://www.raspberrypi.com/news/raspberry-pi-os-64-bit/

Thanks and apologies for the false alert.


[QUESTION] This package is compatible with which Unity version?

I wasn't able to make it work on 2021.2.17f1 without adding an CodeEditor.Register(new NeovimCodeEditor()) at the beginning of the NeovimCodeEditor static constructor.
I believe that the API has changed since the version stated on the package.json.
Besides that, the Sync Solution method is giving errors too, I believe that's for the same reason (API change).

Hi @niscolas
This project currently supports unity 2019.x.
Just last week I migrated a project from unity 2019.x to 2020.x and realized that the unity api used in this project was removed.
Unity 202x support is currently work in progress.


Documentation for lighthouse CI action. But not working

I am following Shopify documentation for lighthouse CI action. But not working

It fails for me too after
Getting a new page...
Getting preview cookie...
ProtocolError: Protocol error (Page.navigate): Cannot navigate to invalid URL

Same here


Fix maximum frequency for SRS860 - it is 500 kHz

Cherrypicked from #1160

Appveyer failure is due to a timeout not related and fixed by https://github.com/QCoDeS/Qcodes/pull/1166


Request for new release

@neoeinstein Hey, would it be possible to get a new release on crates.io that includes https://github.com/neoeinstein/protoc-gen-prost/pull/17 ?

@neoeinstein

I've contacted @neoeinstein by email.

Release for this is in progress

This is now complete. Sorry for the delay, I was on vacation for most of July and forgot to get this out prior to leaving. Thanks for the email to ping my attention, @diceride.


[compiler][flow] Move cast, reshape and bitcast after transfer op

We got incoming IR of the form
  %cast = tensor.cast %0 : tensor<2xf32> to tensor<?xf32>
  %2 = flow.tensor.transfer %cast : tensor<?xf32>{%c2} to
    #hal.device.affinity<@__device_0>
  %cast_2 = tensor.cast %2 : tensor<?xf32> to tensor<2xf32>

We would like to allow for the 2 casts to get folded. We would also like to reduce the dynamism of the transfer op. To operate on a tensor with fewer dynamic dimensions.

I've been trying to fix a similar problem with tensor.collapse_shape/tensor.expand_shape here https://github.com/iree-org/iree/pull/18729

@IanWood1 This seems similar. I am not sure what is more appropriate, to fold the reshape or to move it.
We may need some more general approach to constructing the new op with the folded shape. Some ops in the Flow dialect for example change their arguments depending on the dynamic dimensions of the tensor, where we need to provide the dynamic dimensions as arguments as well. Injection of a constructor into the pattern will probably solve it.

Folding is best whenever possible - the reshape/cast ops are only there to carry metadata and should be aggressively removed when the metadata they carry is not useful.
For example,
%0 = flow.tensor.reshape %arg : tensor<1x2xf32> -> tensor<2x1xf32>
%1 = flow.tensor.transfer %0 : tensor<2x1xf32> to "target"
%2 = flow.tensor.reshape %1 : tensor<2x1xf32> -> tensor<1x2xf32>

should never exist - the canonical form would be:
%2 = flow.tensor.transfer %arg : tensor<1x2xf32> to "target"

If propagating the reshapes/casts/etc allows those to fold by having an intermediate state like:
%0 = flow.tensor.transfer %arg : tensor<1x2xf32> to "target"
%1 = flow.tensor.reshape %0 : tensor<1x2xf32> -> tensor<2x1xf32>
%2 = flow.tensor.reshape %1 : tensor<2x1xf32> -> tensor<1x2xf32>

and then the existing reshape-reshape canonicalizer kicks in that's a good thing.


Update RDF calculation sample.py

Fixed RDF calculation: the results were giving the wrong order of magnitude and the units of the equation were not correct.
Added the lacking "2dr" for 2D-Simulations.
Note: "len(system.distances)/(system.number_of_particles - 1)" is the same as "system.number_of_particles*0.5" but it was not clear to me where that came from.

Hi, I am currently working on the package. Just wondering if you can walk me through how the current method is wrong, and vice-versa how your's is right.


Adding additional AES-GCM encryption as v2

This adds the AES-GCM encryption method which is needed for some devices on newer firmware versions (e.g. gree model 32776, v1.23). The discovery happens automatically during the bind process with a retry on v2 if v1 fails. Key and encryption handling was put into connection.js with little override possibilities to ensure automatic handling is working. Additional logging was added.
A bind-scan socked was added to split normal communication from binding.
This should fix #28.

Thanks for verifying this.
If you need any more input, I am happy to discuss or test on the ticket.

Thank you too! Your changes already are available on version 1.1.0

That's great, will pull and test that later today!


Prevent triggering open/close events while dragging vertically

During touchmove events, vertical dragging should be detected to eliminate any cases of unwanted Slidemenu open/close events.

+1

+1

+1, this is a constant complaint in our implementation of slideout.js, and having this addressed would complete its functionality for us :smile:

+1

+1 It would be nice to have horizontal threshold option.

Still no response after almost 2 years. Do people already have alternatives?

More than two years. This is a major pain too.

You can disable touch event initially and enable it only when slideout is openned.
var slideout = new Slideout({
    ...
});

slideout.once('open', slideout._initTouchEvents);
slideout.on('open', slideout.enableTouch);
slideout.on('close', slideout.disableTouch);

I'd also be interested in hearing if anyone had a better solution for this.

This is not a perfect solution, but to fix some unintended opening of slideout, you can increase the pixel value required for the swipe to trigger slideout.
Line 271 in the current distribution has a default of 20 pixels movement to start opening slideout:
if (Math.abs(dif_x) > 20) {
Currently I am experimenting with setting this number to 40 so that slideout doesn't get triggered as often with small horizontal swipes or when scrolling vertically with a diagonal swipe.
Keep in mind that slideout will immediately snap the panel to this position. So, if you were to set a high value here, not only would it require a large swipe for the user to see indication of the slideout feature, but it would also mean that you would lose some of the smoothness of the opening effect.


[Blog] Revise blog "Spotlight on Kubernetes Upstream Training in Japan"

following up #48039
Revise blog article based on feedbacks
/assign sftim
for approval
Deploy preview: https://deploy-preview-48566--kubernetes-io-main-staging.netlify.app/blog/2024/10/28/k8s-upstream-training-japan-spotlight/

If you're willing to update the https://k8s.dev/ site too: even better.


test: Introduce some functionality for inspecting emitted events in tests

Problem to Solve
At the moment, there is no option to inspect emitted events in regular UTs for service as it just leads to NoOp.
Possible Solution
Consider registering a handler using self address
Notes
No response

Duplicates #325 . This one is closed as the referenced one contains more impl details


Preload Attribute not working correct in afterglow

Hello
html5 preload attribute not working in afterglow player
here is my code:
<video class="afterglow" id="myvideo2" data-skin="dark" width="1280" height="720" preload="metadata">       <source type="video/mp4" src="480p.mp4" />       <source type="video/mp4" src="1080p.mp4" data-quality="hd"/> </video>

Preload is not a very reliable attribute, browsers tend to do what they want with it - in my experience, the preload attribute is ignored most of the times.
This is an html5 video tag thing and has nothing directly to do with afterglow. Read more on this here: https://www.w3schools.com/tags/att_video_preload.asp

Preload is not a very reliable attribute, browsers tend to do what they want with it - in my experience, the preload attribute is ignored most of the times.
This is an html5 video tag thing and has nothing directly to do with afterglow. Read more on this here: https://www.w3schools.com/tags/att_video_preload.asp

All of desktop browsers default support preload attribute, and mobiles too (Until the optimal data saver mode on mobile is active)
Anyway
If this feature is supported on your player, that's great
VideoJS, Plyr, FluidPlayer, JWplayer and more support as well this feature (preload attribute)
Accept this as a suggestion from me
Thank you

Again: you are applying the attribute to a regular video tag. Afterglow doesn't change that, so if the attribute is ignored, you'll most likely want to look for other possible causes. To name just a few:

used browser
number of windows that are open at the same time
available bandwidth
proper cors or other headers
media encoding

There are many more possible causes for the attribute being ignored - afterglow should not be one of them.


Items in "More" dropdown aren't visible in dark mode

What happens?

In dark mode, items in the "More" dropdown are not visible
What did you expect to happen?
For items in the dropdown to be visible
How can we reproduce the issue?
Go to home page of reference docs, and under "CLIENT LIBRARIES" click "More"
OpenAPI Document
No response

Thanks for the report! We’ve had this issue multiple times already. I wonder what we’re breaking there. 🤔
Any ideas? @antlio @hwkr?

This also looks like it might be a different browser / OS. @SynthLuvr do you mind adding your browser and operating system to the bug report?

Windows + Chrome


Update cssnano from 4.1.10 to 4.1.11

Updates cssnano from 4.1.10 to 4.1.11:

Release notes
Commits

Changes look fine – bumps cssnano-preset-default which in turns bumps postcss-svgo from 4.0.2 to 4.0.3 to remove a dependency on is-svg (https://diff.intrinsic.com/postcss-svgo/4.0.2/4.0.3)

Seeing a lot of seemingly unrelated changes in the lock file diff – is that expected?

Seeing a lot of seemingly unrelated changes in the lock file diff – is that expected?

Hmm, good catch. I did this with npm update cssnano, I wonder if doing it "by hand" would result in fewer changes.


Detect _MSVC_LANG in addition to __cplusplus

Addressing issue https://github.com/Tessil/hat-trie/issues/34 ...
With this change, a stock Visual Studio C++17 project will register support for string_view as expected.

Looks good, thank you for your contribution.


Added Kiri's Bubble

I just added my server, Kiri's Bubble to the list! I hope it works and everyone loves it!

I messed up the server link for Kiri's Bubble and didn't have a permanent
one but now I do. I updated it now so it will be good to go. I'm sorry for
the inconvenience. Thank you in advance!
On Mon, Mar 28, 2016 at 2:12 AM, Jiiks<EMAIL_ADDRESS>wrote:

Merged #237 https://github.com/Jiiks/BetterDiscordApp/pull/237.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/Jiiks/BetterDiscordApp/pull/237#event-605006254


SR-hypotheses compliant datasets

Our datasets should contain information about (the list is not exhaustive):

releases
pulls
release_workflow (true or false)
ddirs
w (check this)

Let's make it less configurable than its now. Quiet hard to remember all the parameters we need to pass, and it still won't be compatible to all the steps

@rultor release, tag is 0.0.30

@rultor release, tag is 0.0.30

@h1alexbel OK, I will release it now. Please check the progress here.

@rultor release, tag is 0.0.30

@h1alexbel Done! FYI, the full log is here (took me 18min).

@rultor release, tag is 0.0.32

@rultor release, tag is 0.0.32

@h1alexbel OK, I will release it now. Please check the progress here.

@rultor release, tag is 0.0.32

@h1alexbel Done! FYI, the full log is here (took me 19min).


postgresql version

Why not update postgresql version to 9.4-3 like docker-gitlab?
Is there any concern about it?
Because I am using these two compose file at the same time.
But it download different version of postgresql. I think this is not necessary.

its just that is was not updated.


How to get the origin string of trigger phrase ?

Is there a way we can get string of trigger phrase as the environment variable ?
like those:
ghprbActualCommit
ghprbActualCommitAuthor
ghprbActualCommitAuthorEmail
ghprbPullDescription
ghprbPullId
ghprbPullLink
ghprbPullTitle
ghprbSourceBranch
ghprbTargetBranch
sha1

in my case, we need use this phrase control several build flow for the next steps.

Found ghprbCommentBody from https://github.com/jenkinsci/ghprb-plugin/blob/master/src/main/java/org/jenkinsci/plugins/ghprb/GhprbAdditionalParameterEnvironmentContributor.java#L33-L33
It works for me!

better to add this to doc.


Getting tx confirmation using Provider requires manual creation tx ID - passing tx confirmation doesn't work

Description
I'm using the signer/provider provided by HashPack's HashConnect in my application. Version 2.13.1 of the hashgraph SDK and 0.1.6 of HashConnect.
HashConnect uses the signer/provider in the SDK (see imports in https://github.com/Hashpack/hashconnect/blob/main/lib/src/provider/signer.ts AND https://github.com/Hashpack/hashconnect/blob/main/lib/src/provider/provider.ts)
I freeze a transaction with .freezeWithSigner(signer) and execute it with .executeWithSigner(signer). That works as expected.
However, getting a tx confirmation (receipt or record) is difficult. I should be able to just get and use the transaction id from the execution object, like:
const tokenCreateSubmit = await tokenCreateTx.executeWithSigner(signer);
const txId = tokenCreateSubmit.transactionId
const tokenCreateRx = await provider.getTransactionReceipt(txId)

But that doesn't work. I had to create the txId manually as follows:
const tokenCreateSubmit = await tokenCreateTx.executeWithSigner(signer);
const sec = tokenCreateSubmit.transactionId.validStart.seconds.low;
const nano = tokenCreateSubmit.transactionId.validStart.nanos.low;
const txId =<EMAIL_ADDRESS>const tokenCreateRx = await provider.getTransactionReceipt(txId);

I also tried generating the tx id before freezing and executing the transaction with the signer using TransactionId.generate() (https://docs.hedera.com/guides/docs/sdks/transactions/transaction-id), but that approach doesn't work either to get the tx Id that enables retrieving a confirmation.
A user reported the exact same behavior and issue via Discord. This observation was shared with the internal team.
Steps to reproduce
tokenCreate.txt
The file attached is a component in a larger application but can be used to reproduce/analyze the issue.
Can reproduce the whole application and see the code here; https://gitpod.io/#https://github.com/ed-marquez/hedera-dapp-days/tree/testing
Additional context
N/A
Hedera network
testnet
Version
v0.25
Operating system
Windows

I'm going to close this issue as it doesn't seem to be an SDK issue. However, I did review hashconnects implementation and the odd thing I found is https://github.dev/Hashpack/hashconnect/blob/6e449504f89d0d1f421b457f3e7d6c7dcb2f0964/lib/src/message/message-handler.ts#L62-L63 which seems to simply parse the response from string into a JSON object. It doesn't construct a TransactionResponse so https://github.dev/Hashpack/hashconnect/blob/3175f7a7b1cc7ad18198dc0c347e05aaa402128a/lib/src/provider/provider.ts#L82-L83 doesn't seem right to me. The res.response is not a TransactionResponse, it's only an object that matches the look of the response. All the methods that exist on TransactionResponse and hence the .transactionId field inside are not  there.


Revive fsdoc overnight run

Last run a month ago.

@sishtiaq ! Nice ! The doc just got generated, thanks ;)
FWI, I just noticed the following "Unidentified product" in the generated documentation...
val uint8_to_uint64:Unidentified product: [a:U8.t] (Tot (b:U64.t:{=(U64.v b, U8.v a)}))

I did that by hand. Well, I ran the script manually. It's still not running on the build-machine. Investigating...
The "Unidentified product" is #831. I plan to rewrite fsdoc to use the new, proper pretty-printer. It'll go away then.

Ok. please feel free to tell me if you need to beta-test or anything...

Thanks! Could you do that, please? Just run .ci/fsdoc.sh. You'll need to provide a value for PAT (generate a throw-away PAT from your GitHub settings). At least it'll prove that it works for not-just-me. The problem is probably with the Linux container the script runs in.

bbeurdouche@bmpi:/opt/fstar/.ci$ ./fsdoc.sh
Running fsdoc in /opt/fstar/.ci
./fsdoc.sh: line 21: pushd: ulib: No such file or directory```
There is probably already a problem of rights on linux, looking at line 21...

Ok, pushd and popd don't run on my machine, what are those ?

It's meant to be run from the repo top-level:
FStar> chmod u+x .ci/fsdoc.sh
FStar> .ci/fsdoc.sh

Ok, my bad, I mistakenly interpreted pushd popd as valid command outside a bash script
sed: 1: "index.md": command i expects \ followed by text

Ok this was the classic GNU sed vs BSD sed on OSX. (on OSX gsed works instead)

Well, seems to work for me, the only thing was to use gsed instead of make. I can update the script to check for the first on OSX as we do for F* if you want

You can use sed -i.bak, which works both in GNU and BSD sed.

Currently stuck 'cos, on the docker container, these files:
root@355bf985d524#: ls /LinuxAgent/_work/5/s/fstarlang.github.io/.git/objects/pack
cifsc19  cifsc1e

can't be rm-ed (not even sudo rm -f!) by the nightly build job.

Can we close this one @sishtiaq ?

Closed.


Added support for scripts in file system

Changes:

Updated DbUp dependency to latest available version
Added support to read script files from the file system. #2
Ability to inject logger for better IoC

Breaking Change:

DbUpdater is an abstract class now. To use specific script provider, there are factory methods: DbUpdater.UsingEmbeddedScripts and DbUpdater.UsingScriptsInFileSystem. UsingEmbeddedScripts factory method requires same parameters as previously. Hence, for clients it would be a quick change.

Looks like you never got anywhere with merging this in.  Are you using this code currently? I'm definitely interested in the file system source for hashed scripts.

@piwaf We are not actively using it for other reasons, but we tested it briefly and it got us going with scripts in file system. Let me know what you think about the changes and we can move it forward.


fix: webpack script

This change is necessary because there is no ./config/webpack file.

@igoroctaviano it would be good to have it fixed. But it didn't webpack-base didn't work for me in the past for local dev with the package linked. So if we're changing it to that it's not needed from my perspective.


Drop Java 8 compatibility

Hi Everyone!
I understand that a big chunk of the Java community is still stuck on Java 8, but I wonder if there are any other reason why this project should still have compatibility to that version. As I understand it is not a supported version anymore and Java 11, the current LTS version has already been around for almost 2 years now. Wouldn't it be a good idea to default to Java 11 for this project?

The main reason is 80% developers are using java 8)

While I understand the appeal, we want to ensure we follow Spring conventions here. Not following the conventions puts this project in a hard place if we want to merge the code into Spring Security at some point. Right now JDK 8 is the minimum JDK for every other Spring project and we should remain consistent.


[iOS] ListView with UIKitView (NativeAds) Lags

Hi,
If you use NativeAds in a list, you will notice slight janks when scrolling.
This is harder to see on the video than in real life.
Now there are several ways to improve this:

Make UIKitView cheaper to create
Create a UIKitView pool which can be used by the ads in the list (I don't know if this is possible here)
You can see that there are jerks when initializing the ad, but as soon as the ad is loaded it runs smoothly.

I hope there is a solution for this, because this makes the experience with an app monetized with AdMob very bad.
Target Platform: iOS
**Devices: iPhone X **
Reproducible example
MaterialApp(
      showPerformanceOverlay: true,
      home: ListView.builder(
        itemBuilder: (context, index) {
          var showAd = index % 5 == 0;

          if (showAd) {
            return ListTileNativeAd();
          } else {
            return Container(
              height: 80,
              decoration: BoxDecoration(
                color: Colors.blue,
                border: Border.all(),
              ),
            );
          }
        },
      ),
    );

class ListTileNativeAd extends StatefulWidget {
  @override
  State<ListTileNativeAd> createState() => _ListTileNativeAdState();
}

class _ListTileNativeAdState extends State<ListTileNativeAd> {
  NativeAd _ad;

  bool _isAdLoaded = false;

  @override
  void initState() {
    super.initState();

    _ad = NativeAd(
      adUnitId:
          'ca-app-pub-3940256099942544/3986624511', //AdHelper.nativeAdUnitId,
      factoryId: 'listTile',
      request: AdRequest(),
      listener: NativeAdListener(
        onAdLoaded: (_) {
          setState(() {
            _isAdLoaded = true;
          });
        },
        onAdFailedToLoad: (ad, error) {
          // Releases an ad resource when it fails to load
          ad.dispose();

          print('Ad load failed (code=${error.code} message=${error.message})');
        },
      ),
    );

    _ad.load();
  }

  @override
  Widget build(BuildContext context) {
    return Container(
      height: 100,
       alignment: Alignment.center,
      child: _isAdLoaded ? AdWidget(ad: _ad) : Container(),
    );
  }
}

Logs

Flutter (Channel master, 2.13.0-0.0.pre.880, on macOS 12.2.1 21D62 darwin-arm)

https://user-images.githubusercontent.com/18357332/166615299-6f1ec534-68b5-437e-95ab-e5df34a01f2c.mov

Hi @b3nni97, Thanks for filing the issue. Is the jank related to listview scroll or loading of the ads while scrolling? Can you share a minimal and complete sample project?

hi @maheshmnj, I used the documentation here so you can run the example I showed above. https://codelabs.developers.google.com/codelabs/admob-inline-ads-in-flutter#0
(you have to change some files in the iOS/Android folder)
So it doesn't jerk yet when you just call _ad.load() but only when the ad has loaded and you create the UIKitView object. At this moment it jerks slightly (so I think it's because of initializing the UIKitView).
Once the ad is completely loaded and displayed, the jerking stops. (see video).
You can see in the performance overlay that the app sometimes reaches over 16ms in the raster thread (17.7ms can be seen at the end).

Thanks for the report. Using the code sample provided, running it on iPhone 6s (OS 15.3.1) shows that scrolling with native ads (coming from google_mobile_ads plugin), shows lag in the UI as below:
https://user-images.githubusercontent.com/67046386/166894715-6e36fb48-93e2-4773-b26e-96aaf75a4882.MP4
But I am not 100% sure if this is an issue with platform-views on iOS or with google_mobile_ads issue integration with platform-views.

 stable, master flutter doctor -v

[✓] Flutter (Channel stable, 2.10.5, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 2.10.5 at /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5464c5bac7 (12 hours ago), 2022-04-18 09:55:37 -0700
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 2.13.0-0.0.pre.907, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 2.13.0-0.0.pre.907 at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f7010e00e4 (20 minutes ago), 2022-05-05 00:39:11 -0400
    • Engine revision 912da8da47
    • Dart version 2.18.0 (build 2.18.0-84.0.dev)
    • DevTools version 2.12.2

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

It looks like the ads took sometime to load? We can try the same app, replacing ads with a different platform view (Maybe google map?) to compare the result.

I found performance issues with Flutter 3.0.1, with Dart 2.17.1, using the google_mobile_ads on iOS too
(Android works pretty well).

It looks like the ads took sometime to load? We can try the same app, replacing ads with a different platform view (Maybe google map?) to compare the result.

I'm experiencing this issue too. I replaced the native ad view with an embedded native map view and the behaviour was the same (noticeable jank /lag on the scroll view). Behaviour only affects iOS it seems.

This is a plateform view issue.
If we place native view instead of ads in list we get the same lag result as I explained and recorded in my related issue #107486
This is not a time-to-load problem for me, if we wait for loading and do scrolling or animating the screen is lagging.
All plug-in using platform-views I tried is freezing the screen while a scrolling / or animating something.
Without the Google ads sdk :
I also tried to implement native ads in full swift SDK, as I did with a native IOS app, and create my own platform view to display it in flutter and the result is EXACTLY the same.

A solution would be to stop using platform views in the Google ads and display flutter widget instead as all others widgets like video_player which is working great in list etc...

Or fix/improve the platform view render which is very heavy for flutter

The problem is the same for Android for us but may be a little less lagging.

I am experiencing the same problem. I am using the platform view to wrap a third party native ad plugin. I'm adding native ads at 5 widgets intervals in the ListView.
When the native ad appears in the visible area during the scrolling animation, the scrolling animation freezes for a while and then the animation continues. Android doesn't seem to have this problem. :stuck_out_tongue_closed_eyes:

I think this is a dup with https://github.com/flutter/flutter/issues/107486, I'm going to close this one because I left my performance analysis on the other issue.


Readonly mode

Also: if a database is opened against a read-only database file (e.g. the user does not have write permissisons) we should automatically open in read-only mode.

Added in d4c2e3c6b029419caf6fc39e006a72b5af54727e


still looks shitty as always

Sorry just cant lie :/

then make it better my g

blud ur name is "I-Skid", and literally there are better ones out there, this one is not heavily worked on and is a small project

Sorry just cant lie :/

nobody gives af

blud ur name is "I-Skid", and literally there are better ones out there, this one is not heavily worked on and is a small project

i woulnd't say it's a small porject and we worked heavily on it but yea devs are pretty inactive but we tryna work more on it but yuh i still wouldn't say our readme is ass it does it's job

you didn't even roll out full of my pull req 💀

i explained in my comment on it that i couldn't because restyled said that there are bugs and i fixed em and did it myself


latest node is 6.5.0. ">= 4" resolves to 5.11.1

https://semver.io/node/resolve/>=4
engines.node of >= 4 should resolve to the latest version of node, which is 6.5.0, not 5.11. yes? I think 5.x series is nearly EOL already.
If I am wrong, then I'd appreciate any insight on how to write "the largest version number and also greater than 4.0". thanks

It looks like the node module is scraping the https://nodejs.org/dist listing instead of parsing the more appropriate https://nodejs.org/dist/index.json or https://nodejs.org/dist/index.tab

@mscdex That's true (I adapted the existing code when I was adding support for newer Nodes) but it shouldn't affect this issue, all the files are still there.
The stable version is updated manually so one of the maintainers needs to bump it; see https://github.com/heroku/semver.io/issues/44#issuecomment-138689865.

@hunterloftis could you have a look? Node 5 has been EOL'd for some time so defaulting to it is dangerous.

Is there any news regarding this issue?

This has been fixed. @tjwebb you can close the issue.

resolved by https://github.com/heroku/semver.io/pull/63


Retry init cursor and improve loggging when intersection isn't found

Issue Number

None yet
Overview

1e50d2aef597937884ab9a7e6e4ff579f92c6749
retry fetching intersection and increase timeout delay

3a7a6ffd110a2afc2932eaa219505ce2e638e608
Add explicit warning log line when retrying init cursor

Comments

Investigating -->
[31m[cardano-wallet.wallet-engine:Error:253][0m [2020-03-19 15:53:20.14 UTC] Worker has exited unexpectedly: user error (initCursor: intersection not found? This can't happen because we always give at least the genesis point...)

bors try (why isn't hydra starting?)

bors try

Damn. We actually give a nullTracer to the network layer :man_facepalming:  ..fixing


In HTTPChallengeHandler, allow pass nil as h http.Handler to ignored unhandled HTTP requests

When use certmagic with fasthttprouter we will route all "/.well-known/acme-challenge/*token" requires to certmagic HTTPChallengeHandler and want to ignore  invalid challenge requests which may not handled by certmaigc.HandleHTTPChallenge.
Actually, it is for https://github.com/wheelcomplex/magicgate :XD
Many thanks.

Can you help me understand why this is useful? Why not just use HandleHTTPChallenge()?

we will route all "/.well-known/acme-challenge/*token"

If you still want to do this, you might want to use https://pkg.go.dev/github.com/caddyserver/certmagic#LooksLikeHTTPChallenge instead -- rather than repeating that logic.

LooksLikeHTTPChallenge

Thank you for the reply.
For "Why not just use HandleHTTPChallenge()?" , certmagic is using net/http to process http requests, but there is not a HandleFastHTTPChallenge() for the one who use fasthttp.
Currently a wrap func from fasthttp can wrap HandleHTTPChallenge() into a fasthttp handler. Also   another wrap func from fasthttp can wrap HTTPChallengeHandler. And a wrap func always has extra cost.
After review my idea I believe you are right, it's no reason to repeating that logic. A front dispatcher and a wrapped  HandleHTTPChallenge is the best way currently.
I hope I can implement a HandleFastHTTPChallenge for fasthttp later.
Thanks again!

Okay, thanks for considering!
What is the actual cost of wrapping a function? I'm willing to bet we're talking nanoseconds, if that, which is microscopic on the scale of network requests.


No height property generated for building:levels

What did you see?
building:levels tag in OSM did not result in a height property being generated in tile.

What did you expect to see?
My understanding was that we were synthesizing a height property in tiles (where one doesn't already exist) from other data including building:levels.

What map location are you having problems with? City and country are helpful, as well as tile coordinates or latitude / longitude.

http://tangrams.github.io/bubble-wrap/#17/37.25730/127.05379
https://www.openstreetmap.org/way/161390790#map=18/37.25823/127.05604
https://vector.dev.mapzen.com/osm/all/16/55897/25449.topojson?api_key=vector-tiles-HqUVidw
buildings: {
   type: "GeometryCollection",
   geometries: [
      {
         type: "Polygon",
         properties: {
            building_levels: "25",
            kind: "Samsung tower 3",
            name: "R5",
            area: 46363,
            sort_key: 475,
            landuse_kind: "industrial",
            source: "openstreetmap.org",
            min_zoom: 13,
            id: 161390790
         },
      ...

For example see https://github.com/mapzen/vector-datasource/blob/650353bc0bca603fd3d5f356eb3b134d9a5c84d0/data/functions.sql#L489-L514

Ooops, thanks for pointing that out. Looks like we dropped the : earlier and didn't update downstream transforms?

Ooops, this is a regression over v0.9 tiles. Moving to new v0.10.2 bug fix release.


Application can be used without accepting terms and conditions #946

Oh, I didn't care about  for legal and marketing team, will make change to be support with external link.

There were additional exception that users can navigate to the some other pages like 'namespaces', 'mosaic' and etc.
The terms & conditions accept modal was restricting user only on home page that shows transactions history.
So, some suggestion was moving the terms & conditions accept logic to the create/import profile logic, and I integrated this on finalize of create/import profile page to make sure users must accept terms and conditions before their use.


Add Waterfox to the Portal

Waterfox is a fork of Firefox ESR with an easier to customise UI vs stock Firefox w/o diving into bare CSS files and the profile settings. Whilst its in-app customization isn't as fully-featured as a CSS theme, it's still nice for folk newer to customizing Firefox! It also has a tab sidebar which is more functional than tree style tab (in my opinion), and it can use Chromium extensions, too.

https://github.com/BrowserWorks/Waterfox-Classic/wiki/Unpatched-Security-Advisories


Flyway Logging New Enhancement

Which version and edition of Flyway are you using?
flyway:9.16.1
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
yes
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line docker image : redgate/flyway:9.16.1
Which database are you using? (Type & version)
Yugabyte (PostgreSQL 11.2)
Which operating system are you using?
MacOS
What did you do?
To do logging for audit purpose and sanity check I want to append logs against each script
For this directory structure

I want to see logs something like :

What did you see instead?
Currently I can see all the logs on the console. That I can append only in Single file.

Thank you for the request ucguy4u
To help qualify the request I had an initial question please. As you're capturing the whole output, there will be some initial flyway output that's not script specific, would you envision that captured separately or just included in the first scripts output ideally?


Custom Fields: Improve how long input (messages) is shown in order details

Description
Easy to miss the entire content of long strings in custom fields on order details.
Current Behavior
Long messages/strings in custom fields are cut in the order details – requiring hover to show the full contents.
Expected Behavior
Experiment with the more user friendly option & design:

Truncate string after X characters followed by ... (Show more)
Show in <textarea disabled> with a somewhat decent default size, but ability to scroll within textarea

Screenshots

Just an idea: when this is customer facing you should also remove the label "Custom Fields". I believe it's confusing (or just not relevant in the best case). "Custom Fields" as a label is of no interest for them - they don't care how things are called inside of Polar. IMHO you should just remove it. Keep the single label + value and that's fine. Nothing else needed.

To my defence, this was designed to be displayed in the table, not in a modal with plenty of space 😅

Just an idea: when this is customer facing you should also remove the label "Custom Fields". I believe it's confusing (or just not relevant in the best case). "Custom Fields" as a label is of no interest for them - they don't care how things are called inside of Polar. IMHO you should just remove it. Keep the single label + value and that's fine. Nothing else needed.

Just to clarify the screenshot is from the dashboard and what the developer/seller sees and not the end customer. They only see the field on checkout as an input like the others.
I still hear you though, but I think it's less obvious to not distinguish them since the feature is Custom Fields that the developer can setup themselves. That said, they'll recognize that and perhaps we can distinguish it more with an icon / link to edit the custom field or equivalent vs. the title.

Hi @birkjernstrom, it seems the described functionality is already implemented - unless I'm misunderstanding the issue?
https://github.com/user-attachments/assets/d203aa42-7f82-4f18-8bf3-f030b796a0d0
The dummy data used in the above recording can be found in this commit.


fix: include base path in path to start script

fixes https://github.com/sveltejs/kit/issues/2958

wtf. I hate turbo so much

Is there a test we can add for this?

Is there a test we can add for this?

This one is tricky to add a test for because it will pretty much work as it is today. The only reason it's broken in the linked issue is because the users are accessing the site through a proxy, but I don't want to setup a proxy just for the test. We could write a test that looks for the string import { start } from "/path-base in the rendered source, but it feels a bit brittle. Would you like me to add that?

Nah, that's probably not a good test. The reason I think we ought to have a test is that it honestly feels like a Vite bug that<EMAIL_ADDRESS>and<EMAIL_ADDRESS>work interchangeably, and it would be good to be able to guard against changes one way or the other. Unless that's by design?

(I feel like<EMAIL_ADDRESS>should probably 404 if base is configured...)

I agree with that. I think the test would be best as part of a PR to change the behavior in Vite. It'd be hard to change the behavior at the moment as users might be relying on it via their Vite plugins and it'd be a breaking change. This is highly related to https://github.com/vitejs/vite/issues/7363 which I just linked to in the code comment in https://github.com/sveltejs/kit/pull/8683. It's probably my top request for Vite 5 so with any luck I'll get to fixing it then now that I cleaned up all the other base path stuff in Vite 4

I added a comment so that no one unknowingly breaks things by changing the code because they're unaware the base is needed


Fixed typo in BaseMigrateController.php

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
-

Thank you

:thumbsup:


how to delete table with limit ?

the sql e.g:
delete from table_name  order by order_name limit 100;

Hey @baoyachi, problem solved?

Umm if your problem is solved making a QnA will also be appreciated


Parse words dsv files w custom delimiters

The webpage driver needs to use the words_delim config file attribute and have a corresponding control in the config form to use words files with custom delimiters.


Added ru_RU (Russian) translation for the "pull.js".

I have added ru_RU (Russian) translation for the "pull.js" lesson. With approval of project owner I added my account name into list of contributors.

With approval of project owner I added my account name into list of contributors.

very well deserved! haha. :clap: :tada: thanks for the PR -- sorry for the delay but all the interns are showing up at Facebook so things are getting hectic :D
merging now

:thumbsup:


Add link to the active fork & transfer gem ownership?

Hey @krobertson,
Looks like there's an active fork at https://github.com/deb-s3/deb-s3
Hoping you can:

Simply add the link to the readme?
Transfer gem ownership to the new repo maintainers?

Thanks!

@krobertson if you have 2 mins, this would be super helpful to lots of folks!


Migrate from LlamaKit to antitypical/Result

As noted in the LlamaKit README, it "has been merged with and superceded by Rob Rix's Result µframework.".

Good call, +1. I'll try to work on this ASAP, pull requests also welcome! :sandal:

I'm trying to get a pull request together. :smile:


[Change Request] Codeunit 80 "Sales-Post" – Changed Event “OnAfterCheckTrackingAndWarehouseForReceive”

Could we please have the existing parameter “TempSalesLine” of the existing publisher “OnAfterCheckTrackingAndWarehouseForReceive” be transferred by reference (var) ?
    [IntegrationEvent(false, false)]
*** start old code ***
    //local procedure OnAfterCheckTrackingAndWarehouseForReceive(var SalesHeader: Record "Sales Header"; var Receive:
         Boolean; CommitIsSuppressed: Boolean; var TempWhseShptHeader: Record "Warehouse Shipment Header" temporary;
         var TempWhseRcptHeader: Record "Warehouse Receipt Header" temporary; TempSalesLine: record "Sales Line"
         temporary)
*** stop old code ***
*** start new code ***
    local procedure OnAfterCheckTrackingAndWarehouseForReceive(var SalesHeader: Record "Sales Header"; var Receive:
        Boolean; CommitIsSuppressed: Boolean; var TempWhseShptHeader: Record "Warehouse Shipment Header" temporary;
        var TempWhseRcptHeader: Record "Warehouse Receipt Header" temporary; var TempSalesLine: record "Sales Line"
        temporary)
*** stop new code ***
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


More OpenDatabase() Testing

Fixes up TestOpenDatabase:

there is now a broader variety of testInputs and testBuckets
testInputs are now hardcoded rather than randomly generated
testInputs can thus now be defined via anonymous struct, so type testInput can be removed
filepath argument to OpenDatabase is now the full filepath of the database (previously just the file name, dbFilename, was passed in, which meant that the database was created in the wrong directory)

Introduces TestBadFileModeOpenDatabase, which checks that a file with the wrong filemode (less than 0600) will cause OpenDatabase to return a permissions error.

New changes:

Rename TestBadFilemodeOpenDatabase to TestErrPermissionOpenDatabase.

Introduce several new functions:

TestErrTxNotWritable (tests that updateMetadata returns an error when called from a read-only transaction)
`TestErrDatabaseNotOpen (tests that checkMetadata returns an error when called on a closed BoltDatabase)
TestErrCheckMetadata (tests that checkMetadata(md) returns current error when called on a database whose metadata does not match md)
TestErrIntegratedCheckMetadata (tests that checkMetadata returns correct errors when called within OpenDatabase on a database whose metadata does not match md)

Make global testInputs and testFilenames structs, since tests require the same sorts of inputs

Latest changes address Luke's comments:

Fix error handling in OpenDatabase
Use View() instead of Begin() and Commit() in TestErrTxNotWritable

Latest update removes extraneous new lines and that empty comment David spotted

db.View now takes updateMetadata directly as an argument


Implement JDBC PreparedStatement.getParameterMetaData

This should be implemented using DESCRIBE INPUT.

I am interested in helping out, thank you!


Pasting cells from Excel misbehaving

Original conversation here: https://discord.com/channels/253897423439659008/459650160658481152/1054159287087276062
Original post:

Excel has... weird... copy + paste shenanigans it does to try and maintain cell information. When I try to copy from Excel to Chatterino the paste just doesn't show up at all. My workaround for this is to use office.com which has a browser-based version of Excel that works fine. Google Sheets works fine as well. Also, copying from Excel to other stuff (say, Notepad) works fine. But I was curious if anyone knew of a way that I could use Excel and paste into Chatterino?

Single-cell & multi-cell copying fails with no error message.
Potentially due to us trying to upload it as an image but image uploading is disabled
OS: Windows
Application: Excel Version: Microsoft® Excel® for Microsoft 365 MSO (Version 2211 Build 16.0.15831.20098) 64-bit
Example file: TestBook1.xlsx

Confirmed the user had no image uploader configured - with an image uploader configured, pasting a single cell resulted in an image being uploaded

Libreoffice (on linux x11) seems to exhibit the same behavior as MS Excel.
$ xclip -o -t TARGETS -selection clipboard
TIMESTAMP
TARGETS
MULTIPLE
SAVE_TARGETS
application/x-openoffice-embed-source-xml;windows_formatname="Star Embed Source (XML)"
application/x-openoffice-objectdescriptor-xml;windows_formatname="Star Object Descriptor (XML)";classname="47BBB4CB-CE4C-4E80-a591-42d9ae74950f";typename="LibreOffice 7.4 Arkusz kalkulacyjny";displayname="file:///[snip]/TestBook1.xlsx";viewaspect="1";width="26385";height="1589";posx="0";posy="0"
application/x-openoffice-gdimetafile;windows_formatname="GDIMetaFile"
application/x-openoffice-emf;windows_formatname="Image EMF"
application/x-openoffice-wmf;windows_formatname="Image WMF"
image/png
application/x-openoffice-bitmap;windows_formatname="Bitmap"
image/bmp
text/html
application/x-openoffice-sylk;windows_formatname="Sylk"
application/x-openoffice-link;windows_formatname="Link"
application/x-openoffice-dif;windows_formatname="DIF"
text/plain;charset=utf-16
application/x-libreoffice-tsvc
text/rtf
text/richtext
text/plain;charset=utf-8
UTF8_STRING
STRING
application/x-libreoffice-internal-id-2765303


Change Github to GitHub

It's minor, but but the H in GitHub is capitalized. :)

Thanks for the PR :)


portlint logs aren't easily located

I have USE_PORTLINT=yes in my config, but the testport logs do not contain any portlint output. Looking at the code, I see that the portlint output is logged to a separate file, but that file is not copied to the latest-per-pkg directory or anywhere easily located.
I wonder, is there any particular reason portlint output cannot be part of the testport log? If it's not desirable, it would be nice to have the portlint log copied together with the testport log.
I'm using poudriere 3.1.9 on FreeBSD 10.2-RELEASE, fwiw.

No good reason, it should just be inlined.

It looks like the output is included inline now as well as being written to a file, according to https://github.com/freebsd/poudriere/blob/3.1.19/src/share/poudriere/testport.sh#L217-L219:
	cd ${MASTERMNT}/usr/ports/${ORIGIN} &&
		PORTSDIR="${MASTERMNT}/usr/ports" portlint -C | \
		tee ${log}/logs/${PKGNAME}.portlint.log

It shouldn't be, the port log file tee starts at the log_start call further down at https://github.com/freebsd/poudriere/blob/3.1.19/src/share/poudriere/testport.sh#L245