gridtracker: 1.23.1207 -> 1.24.0104

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gridtracker is: An amateur radio companion to WSJT-X or JTDX
meta.homepage for gridtracker is: https://gridtracker.org
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.23.1226 with grep in /nix/store/6z1arqsksp4r81kabwacgb9bvqk5z5qk-gridtracker-1.23.1226
found 1.23.1226 in filename of file in /nix/store/6z1arqsksp4r81kabwacgb9bvqk5z5qk-gridtracker-1.23.1226

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
gridtracker

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6z1arqsksp4r81kabwacgb9bvqk5z5qk-gridtracker-1.23.1226 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gridtracker https://github.com/r-ryantm/nixpkgs/archive/cb184f58ec4b0901014ef376533b25a66e5dc490.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/cb184f58ec4b0901014ef376533b25a66e5dc490#gridtracker

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6z1arqsksp4r81kabwacgb9bvqk5z5qk-gridtracker-1.23.1226
ls -la /nix/store/6z1arqsksp4r81kabwacgb9bvqk5z5qk-gridtracker-1.23.1226/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

gridtracker

Maintainer pings
cc @mattmelling for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gridtracker is: An amateur radio companion to WSJT-X or JTDX
meta.homepage for gridtracker is: https://gridtracker.org
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.24.0104 with grep in /nix/store/d1rrizm0cm2hbl7xk1s55n29q20g55ii-gridtracker-1.24.0104
found 1.24.0104 in filename of file in /nix/store/d1rrizm0cm2hbl7xk1s55n29q20g55ii-gridtracker-1.24.0104

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
gridtracker

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/d1rrizm0cm2hbl7xk1s55n29q20g55ii-gridtracker-1.24.0104 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gridtracker https://github.com/r-ryantm/nixpkgs/archive/07fd92be44117a74813cb25261ae6cb9ce0befa6.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/07fd92be44117a74813cb25261ae6cb9ce0befa6#gridtracker

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/d1rrizm0cm2hbl7xk1s55n29q20g55ii-gridtracker-1.24.0104
ls -la /nix/store/d1rrizm0cm2hbl7xk1s55n29q20g55ii-gridtracker-1.24.0104/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

gridtracker

Maintainer pings
cc @mattmelling for testing.

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

1 package built:

gridtracker


Pod name fixed

This change is

@zburgermeiszter pls sign the CLA.

@tengqm Done.
How to get the bot to recheck it?


add model description

#56 default value is None, so it will not have conflict on the previous code calling model related interface.

Shall we display the description on the web page when we list the model details?

Shall we display the description on the web page when we list the model details?

The response contains an item called model_description. If you like, you can desplay it on the web page after developing related frontend code.


The trained network.pth file.

Hello, wanted to ask if you can attach the network.pth file from the siamese training, as my training is not working, giving bad features (Not equal to the extracted features in the .mat files)
Thanks in advance. @lood339

Hi,
Thanks for interesting in our work.
The code was modified from a Matlab version about two years ago. If I remember correctly, I do not train the model using this python version because I do not have a GPU after graduated.
You can have a look this issue: https://github.com/lood339/SCCvSD/issues/6 to get some idea.
Hope that helps.

Thanks a lot Dr.Chen for the prompt response.


OpenComputer can't boot with Screen Controller

I don't know if it is a Issue in RFTools or OC but if I start a Computer from OC with a Screen Controller it says "Unrecoerable Error machine:1146: no such method" on the Computer Screen.
If I first start the Computer and then connect the Controller it works fine.
RFTools. 3.10
OC 1.5.13

Did you also report this to the OC guys? If so you can cross-link the ticket by adding the ticket link here.

https://github.com/MightyPirates/OpenComputers/issues/1255


Listing all public repositories of a github user/organization

I really enjoy using hub, this is such a great tool.
I was wondering if it's possible to use hub to list all the github repos associated with a particular user - either through a command or autocompletion at the command line. I use bash on Linux if that makes a difference.
Cheers,
C

Hi, thanks for writing in with a feature idea.
Listing repositories isn't something that most git users will perform on the command-line. It seems that this would be more suited for a generic "data dump" command that could output raw data from the API for script consumption: https://github.com/github/hub/issues/1407
Once that is done, we could tweak our shell completion scripts to autocomplete hub clone github/hu to github/hubot (or offer choices if there are multiple matches), but this is a long way down the road from now, and I'm not sure when we could get to that.
I'm going to close this as a duplicate. This will be possible when we have a generic API command for fetching arbitrary data.

Think this would still be a really good feature to have! Feels out-of-place to be able to create and delete repositories from the command line, but not view them. Doing this using the generic API command becomes less trivial when you want to view private repositories.
Still hope to see this feature one day!

Until then, fetching all your repositories can be done with hub api:
repos() {
  local user="${1?}"
  shift 1
  paginate hub api -t graphql -f user="$user" "$@" -f query='
    query($user: String!, $per_page: Int = 100, $after: String) {
      user(login: $user) {
        repositories(first: $per_page, after: $after) {
          nodes {
            nameWithOwner
          }
          pageInfo {
            hasNextPage
            endCursor
          }
        }
      }
    }
  '
}

paginate() {
  local output cursor
  output="$("$@")"
  cursor="$(awk '/\.hasNextPage/ { has_next=$2 } /\.endCursor/ { if (has_next=="true") print $2 }' <<<"$output")"
  printf "%s\n" "$output"
  [ -z "$cursor" ] || paginate "$@" -f after="$cursor"
}

repos "tfcat"

In case you want to show repositories of local user(that was logged in via hub):
repos() {
  local user="${1?}"
  shift 1
  paginate hub api -t graphql -f user="$user" "$@" -f query='
    query($user: String!, $per_page: Int = 100, $after: String) {
      user(login: $user) {
        repositories(first: $per_page, after: $after) {
          nodes {
            nameWithOwner
          }
          pageInfo {
            hasNextPage
            endCursor
          }
        }
      }
    }
  '
}

paginate() {
  local output cursor
  output="$("$@")"
  cursor="$(awk '/\.hasNextPage/ { has_next=$2 } /\.endCursor/ { if (has_next=="true") print $2 }' <<<"$output")"
  printf "%s\n" "$output"
  [ -z "$cursor" ] || paginate "$@" -f after="$cursor"
}

user() {
  hub api --flat user | grep .login | awk '/.login/ {print $2}'
}

repos "$(user)" | awk '/\.nameWithOwner\t/ { print $2 }'

I use this to list my repositories hub api -X GET /user/repos | jq -r '.[] .name'

Great find @cesarnorena - thanks!

A more efficient way to list all repositories owned by you:
 hub api --paginate -XGET user/repos -f affiliation=owner -f per_page=100 | jq -r '.[].name'

Explanation:

uses--paginate to fetch all pages of results
uses per_page=100 to ensure the minimum amount of HTTP requests are necessary
uses the affiliation filter to exclude repositories you have access to but are not the owner of


pygeoapi realm is not loaded in keycloak

The exported configuration is currently not loaded when the docker composition bootstrapping

Closed by 2095797


[AMORO-3263] Iceberg table format displays primary key information

Why are the changes needed?
Close #3263 .
Brief change log

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[x] Add screenshots for manual tests if appropriate

[x] Run test locally before making a pull request

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

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

Project coverage is 29.83%. Comparing base (39a5aa1) to head (d77873b).
Report is 3 commits behind head on master.

Files with missing lines
Patch %
Lines

...rver/dashboard/MixedAndIcebergTableDescriptor.java
0.00%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #3264   +/-   ##
=========================================
  Coverage     29.83%   29.83%
  Complexity     3785     3785
=========================================
  Files           580      580
  Lines         47932    47934    +2
  Branches       6210     6209    -1
=========================================
+ Hits          14299    14300    +1
- Misses        32648    32649    +1
  Partials        985      985

Flag
Coverage Δ

core
29.83% <0.00%> (+<0.01%)
:arrow_up:

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

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


Saving and Loading models with S3

Closing this PR so as to open loading and saving as two separate PRs


"Try It" response preview caches previous response data

When I click Send API Request the Response section appears and displays the response preview. You can search the page for a substring contained in the response data.
If the response size is large enough, the returned JSON in the preview window is not colorized.
If in this state you click Send API Request again, the non-colorized preview is not updated, even if the API request returned different data.
Search in page will not find the updated data in the Response section.
Context
Requires additional carefulness when checking the API responses and causes frustration.
The Copy button, however, copies the correct response data and if you paste it anywhere else, you will see the changes.
Web inspector shows the right API request and response from the server. It's just the uncolorized preview window that doesn't get updated.
Current Behavior
The Response preview contains the last long response result.
Expected Behavior
The Response preview updates the response window upon receiving the response data from the server.
Possible Workaround/Solution
Workaround: put the page in a state where there is no Response section.

Refresh the page and send the request again.
Click another endpoint and then click this endpoint to reload this section
Change the query parameters so that the response is small and colorized and then back

Steps to Reproduce
See general description.

Send an API request that returns approximately >10000 lines of JSON so that it's not colorized in the preview.
Copy the response somewhere.
Modify the server data so that the same API request returns >10000 lines of JSON but different data.
Click Send API request again
Search the response window in the browser for the updated data.

Environment

Version used: @stoplight/elements 7.10.0 and the latest 8.0.2
Environment name and version: MS Edge/120.0.2210.144, Firefox 121.0.1, NodeJS v18.18.0
Operating System and version (desktop or mobile): Windows 10 22H2, desktop.
Link to your environment/workspace/project: private.

@Lexy2  Any chance you can attach a problematic 10000 line JSON file?

long.json
It's a bit more than 10k, but should work too :-)

@weyert how's the reproduction going?

@Lexy2 I haven't had a chance to look at it. I don't work for the creator of this library I am just a consumer. I do have other weird issue where the Try it's request pane isn't always showing up. Think there is some sort of state issue. Not sure, if it's my use of the elements-core-library


Pascal architecture is much slower than Maxwell...

Hi Justin,
In fact Pascal architecture is much slower than Maxwell. Could you please look at my benchmarks down below? The point is to measure old architecture GPUs with optimized DNN libraries for that specific architecture - CUDA 7.5 cuDNN v4.0.
Measured in CNTK, Caffe under Windows 7 and Ubuntu OS.
GTX 980 Ti - CUDA 7.5 with cuDNN 4.0
Caffe Performance: 5242 imgs/s
GTX 980 Ti - CUDA 8.0 RC with cuDNN 5.1
Caffe Performance: 4183 imgs/s
GTX 1080 - CUDA 7.5 with cuDNN 4.0
Caffe Performance:  Not Applicable
GTX 1080 - CUDA 8.0 RC with cuDNN 5.1
Caffe Performance: 4628 imgs/s
Best Regards,
Ondrej

There is something wrong with these numbers - the 980 Ti should be roughly the same speed as a Maxwell Titan X, and should thus be quite a bit slower than a 1080. How did you generate these numbers?

@Justin: These numbers can be calculated from log files which are generated by Caffe and CNTK frameworks during training process. For benchmarking purposes I've used standard AlexNet model architecture with modified dimensions of input images (dim: 3 dim: 101 dim: 101). You are right that GTX 1080 should be faster than 980 Ti, but before I bought GTX 1080 I have used for a while GTX 980 Ti with Caffe and CNTK and I'm pretty familiar with its performance (Caffe was compiled with CUDA 7.5 and cuDNN 4.0). Since I bought GTX 1080 I was forced to move to CUDA 8.0RC and cuDNN 5.x and recompile Caffe, because only these libraries support Pascal architecture. To my big surprise GTX 1080 was slower than GTX 980 Ti (with CUDA 7.5 and cuDNN 4.0). BUT when I benchmarked Caffe compiled with CUDA 8RC and cuDNN 5.x on GTX 980 Ti it was actually slower than GTX 1080. I obtained similar results with CNTK framework.
So my  understanding is that CUDA 8.0RC and cuDNN 5.x libraries are currently just not properly optimized for both architectures - Maxwell and Pascal as well.
Would you be so kind and run your benchmarks again with GeForce GTX Titan X card (Maxwell architecture), but compiled with CUDA 7.5 and cuDNN 4.0? If my benchmarks are correct, you should get much better results. I can guess that old Titan X will perform better than GTX 1080.
Thank you very much.
Best regards,
Ondrej

Check out my neural-style speed benchmarks:
https://github.com/jcjohnson/neural-style#speed
The Maxwell Titan X benchmarks used CUDA 7.0 and cuDNN 4 and the Pascal Titan X benchmarks used CUDA 8.0 RC and cuDNN 5; the Pascal Titan X is almost 2x faster.


Feature: Use TypeScript, Write 'jest.config.ts' file

This allows us to TypeScript ONLY.
Expected:
// File path: project/jest.config.ts
import { Configuration } from "ts-jest";

const configure: Configuration = {
    preset: 'ts-jest',
    testEnvironment: 'jest-environment-jsdom-fourteen',
    globals: {
        'ts-jest': {
            isolatedModules: true,
        },
    },
}

export default configure

hi, this feature doesn't belong to ts-jest scope, it should be in jest repo. You should ask in jest repo for this feature

@ahnpnl https://github.com/facebook/jest/issues/9636 is done.


added bangla translation

Added Bangla translation.
Pre-launch Checklist

[x] I have updated CHANGELOG.md with my changes
[x] I have run "dart format ." on the project
[ ] I have added/updated relevant documentation

@bdlukaa why there is so many change in the change log and there is a reason for change Dart sdk version constraint ?


Set road_segment_id in news_flash of Suburban resolution.

Codecov Report
Attention: Patch coverage is 21.05263% with 15 lines in your changes are missing coverage. Please review.

Project coverage is 52.95%. Comparing base (e8db8d5) to head (d345e13).

Files
Patch %
Lines

anyway/parsers/location_extraction.py
7.14%
13 Missing :warning:

anyway/parsers/news_flash.py
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2577      +/-   ##
==========================================
- Coverage   53.04%   52.95%   -0.09%
==========================================
  Files         119      119
  Lines        9849     9865      +16
==========================================
  Hits         5224     5224
- Misses       4625     4641      +16

Flag
Coverage Δ

unittests
52.95% <21.05%> (-0.09%)
:arrow_down:

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

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

Hi @atalyaalon ,
This PR is the first part of issue #2567. It sets road_segment_id in news_flash items. After it is merged, we should update all news_flash items to set road_segment_id in them.
Note:

Running the news flash update in my local environment, I saw that there are some news flash items that previously were considered not accident, and not that are considered as accidents. (e.g. id:173140).
roads in NewsFlash where typed float in models.py, and I changed them to int.


[fix](runtime filter) append late arrival runtime filters in vfilecanner

Proposed changes
VFileScanner will try to append late arrival runtime filters in each loop of ScannerScheduler::_scanner_scan.  However, VFileScanner::_get_next_reader only generates the _push_down_conjuncts in the first loop, so the late arrival runtime filters are ignored.
Effects
When running the Q9 of tpch 300g, we should set runtime_filter_wait_time_ms=20000 to wait the runtime filter created by part table ready. Currently, using the default wait time(1000ms) also has similar filtering effects.
Before this PR, a fragment should shuffle 123M rows:
-  RowsProduced:  123.552803M  (123552803)
-  RowsRead:  154.078167M  (154078167)
-  RowsReturned:  123.552803M  (123552803)
-  FileNumber:  162

After this PR, a fragment only shuffle 10M rows:
-  RowsProduced:  10.644022M  (10644022)
-  RowsRead:  34.683162M  (34683162)
-  RowsReturned:  10.644022M  (10644022)
-  FileNumber:  162
-  HasFullyRfFileNumber:  125

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

run buildall

run buildall

run buildall

run buildall


WIP: Refactor marriage-abroad country_of_ceremony

This is a work in progress. I've opened the PR in case it's useful to see the direction I'm heading in as I continue to refactor marriage-abroad.

I've split a number of commits from this branch to a new PR - #2288. I think they can be reviewed and merged independently without me having to finish the remainder of the work in this branch.


Improve docs on backpressure when forking and re-joining streams

This issue was spawned by #40

Bump.  I spent a while trying to figure out how to get a simple example working, I'm going to leave it here in case someone finds my confusion valuable.  I wanted to create a streaming system that would take in a stream of numbers, and prepend and append new numbers to fill out a range.  Desired behavior:
fillIn(1,10)([5,6,7]) // put out a stream: [2,3,4,5,6,7,8,9], where 5,6,7 are reused (important if they are objects not integers)

My final, working code looks like this:
import _ from 'highland';

var logArray = _.curry(function(name,array) {
  console.log(`${ name } : `,array);
});

function min(a,b) { return a>b?b:a; }
function max(a,b) { return a>b?a:b; }

let numbersBetween = _.curry(function(reference,err,number,push,next) {
  if(err) { push(err); }
  else if (number === _.nil) {
    console.log("Between done, sending on end.");
    push(null, _.nil);
    return;
  } else {
    var n = min(reference,number)+1;
    var end = max(reference,number);

    while(n < end) {
      console.log(`pushing ${n}`);
      push(null,n);
      n++;
    }
  }

  next();
});

// return a stream that puts out all of the integers between earliest
// and latest, including the ones from an assumed-sorted "existing"
// array.
function getNumbers(lowest,highest,inNumbers) {
  let existing = _(inNumbers || [lowest]);

  let after = existing.observe().take(1).consume(numbersBetween(highest));
  let before = existing.observe().last(1).consume(numbersBetween(lowest));

  before.observe().toArray(logArray('before'));
  existing.observe().toArray(logArray('existing'));
  after.observe().toArray(logArray('after'));

  let res = before.concat(after,existing);

//  res.toArray(logArray('res'));

  existing.resume();
//  after.resume();
//  before.resume();

  return res;
}

getNumbers(1,10,[5,6]).toArray(logArray('FinalRES:'));

What took me a long time to figure out is that you have to NOT resume some streams - ("resuming" is a "terminating" event according to some parts of the docs?).  When I don't "resume" existing, or DO resume after or before, everything gets stuck on concat, why?  The logArray calls from the observed streams are all reporting "finished and did the right thing as expected", but the concatenated stream just doesn't.  I'm guessing that when you don't resume existing it never un-pauses, and when you do resume after and before, they go from being dependent on existing to being dead or something.  I don't actually understand at a deep level what 'resumed', 'paused', or otherwise really means.  A few concrete examples with fork/merge/concat/observe, and a more detailed explanation of why you would want to use pipline would go a long way to explaining this situation to me!

Agreed that there needs to be better docs, but I can't promise anything in soon. I can explain a bit more about your specific problem, though.
You can think of streams as being an async generator. Calling resume consumes from the stream by asking it to start producing data. It's roughly equivalent (though not exactly) to calling each with a callback that does nothing. A paused stream is not producing data, while a resumed or consumed stream is. All streams start paused. They only begin producing data when a consumption (what you call "terminating") event happens. That is, when a consumption methods or resume is called.
When you observe a stream, you create a new stream that observes the source stream and remembers what values the source stream produces. It can then replay those values when it (the observing stream) is consumed. Naturally, it can't produce any values if the source stream hasn't produced any value.
Now, concat is lazy. When you concat before and after, it won't consume from after until before completes (i.e., until before produces nil). In fact, laziness is a big part of Highland streams, and most other transforms work this way. Now, before must wait for existing to produce values since it observes existing. So then you have a situation where concat won't consume from existing until before produces a nil, but before won't produce anything until existing produces something. But nothing resumes existing! You end up with a deadlock arising from the circular dependency. In general, do not introduce circular dependencies into your stream construction and you won't have this problem [1].
As for your solution. I don't think it works. First of all, concat only takes one argument, so that call you're making isn't working as expected. It probably won't if inNumbers has three or more values. I ran getNumbers(1,10,[5,6,7]) and got [ 2, 3, 4, 5, 6, 6, 7, 8, 9 ].
Second of all, your solution is overly complicated. In general, it's rare to need to use resume directly. You can implement what you want directly with a single consume.
function pushRange(min, max, push) {
    var i = min + 1;
    while (i < max) {
        push(null, i);
        i++;
    }
}

// Simplifying assumptions:
// - lowest < highest.
// - inNumbers is sorted.
// - inNumbers === [] || lowest < first(inNumbers)
// - inNumbers === [] || highest > last(inNumbers)
function getNumbers(lowest, highest, inNumbers) {
    var nextValue = lowest;
    var existing = _(inNumbers || [lowest]);
    return existing.consume(function (err, x, push, next) {
        if (err != null) {
            push(err);
            next();
        } else if (x === _.nil) {
            // At the end. Push the suffix.
            pushRange(nextValue, highest, push);
            push(null, _.nil);
        } else {
            if (nextValue === lowest) {
                // First time, push the prefix.
                pushRange(nextValue, x, push);
            }
            nextValue = x;
            push(null, x);
            next();
        }
    });
}

getNumbers(1, 10, [4, 5, 6]).toArray(_.log);
// => [ 2, 3, 4, 5, 6, 7, 8, 9 ]

[1] If you're interested in how to get circular dependencies to work, see the discussion on flow-based programming in #371.

Thank you for your response, and you were right, the code I pasted first did not work as expected in general, only for one of the inputs that I happened to use as my initial test case!  Your simplified version makes a lot of sense, and is significantly simpler.  I ended up figuring out many of these details, but if you'll bear with my bad architecture for one moment, I have one final question:
function getNumbers(lowest,highest,inNumbers) {
  let existing = _(inNumbers || [lowest]);

  let after = existing.fork().take(1).consume(numbersBetween(highest));
  let before = existing.fork().last(1).consume(numbersBetween(lowest));

  before.observe().toArray(logArray('before'));
  existing.observe().toArray(logArray('existing'));
  after.observe().toArray(logArray('after'));

  let res = _([before,existing.fork(),after]).sequence();
  existing.resume();

  return res;
}

In this (now actually working, afaik) example, if I remove the existing.resume(), and the .fork() call on the line above it, when I .toArray() the result of this function, it does not work.  My internal explanation for that is that existing never got un-paused, though I find this confusing, because shouldn't it get un-paused when sequence() begins pulling on it, when I call toArray()?

This is a feature of fork. Unlike observe (which just passively observe its source), forks directly affects their source and share backpressure. This means a few things:

No fork of a source will produce any value until all forks are consumed.
Once all forks all are consumed, the source is automatically consumed to produce values.
If any fork is slow, all forks are slowed down to match it.

Here's an example:
var source = _([1, 2, 3]);

// fork1 is fast.
var fork1 = source.fork();

// fork2 is slow. It will only consume one item per second.
var fork2 = source.fork()
    .consume(function (err, x, push, next) {
        push(err, x);
        if (x !== _.nil) {
            // Simulate slowness.
            setTimeout(next, 1000);
        }
    });

// Now we consume from fork1.
fork1.each(function (x) {
    console.log('fork1:', x);
});

// Nothing will print yet.

// Only until we consume from fork2 does values start being
// produced, one per second.
fork2.each(function (x) {
    console.log('fork2:', x);
});

before and after both fork existing. sequence will consume from the source one at a time. That means it will consume from before before it even touches after. But both before and after must be consumed before either produce anything, so sequence is stuck waiting for before to produce data, which is stuck waiting for after to be consumed, which is stuck waiting for sequence to consume from it.
If you switch before and after to observe and use let res = _([before, existing.observe(), after]).sequence(), then you can use existing.resume() and everything should work. I don't recommend doing so, however, since you're basically bypassing backpressure.


Rename CI github workflows

Signed-off-by: timflannagan<EMAIL_ADDRESS>
Description of the change:
Update the various CI workflows names that are run on a per-PR basis to
help distinguish between individual action runs when interacting with
the github UI.
Motivation for the change:
Interacting with the 'actions' tab in the Github UI can be challenging when trying to view individual action runs. This is because all of the CI-related workflows are currently named 'ci', which makes filtering out for specific workflows more challenging, requiring you to hover over each action to determine the resolved link. The following screenshot shows the current interaction:

Reviewer Checklist

[x] Implementation matches the proposed design, or proposal is updated to match implementation
[x] Sufficient unit test coverage
[x] Sufficient end-to-end test coverage
[x] Docs updated or added to /doc
[x] Commit messages sensible and descriptive

/lgtm

Manually overriding the failed e2e test as this PR only affects the naming of the workflow(s) and nothing user-facing.


RFE: Overview in Alphabetical Order

May we have an option in the settings to show the entries in the overview (and other lists) in Alphabetical order, rather than the order they were entered / retrieve from the database?

Dear pklink, I also think that the default view of the entries should be the alphabetical view.


Two-dimensional array binding type problem

I'm adding matrix type to LVGL project. I added a matrix type and its c code declaration is：
typedef struct {
     float m[3][3];
} lv_matrix_t;

But A type error occurred when generating the binding code of micropython. An error similar to the following:
build-standard/lvgl/lv_mpy.c:25340:28: error: expected identifier or ‘(’ before ‘[’ token
25340 | GENMPY_UNUSED STATIC float [3] *mp_arr_to_float___3____3__(mp_obj_t mp_arr)
      |                            ^
build-standard/lvgl/lv_mpy.c:25356:70: error: expected ‘;’, ‘,’ or ‘)’ before ‘*’ token
25356 | GENMPY_UNUSED STATIC mp_obj_t mp_arr_from_float___3____3__(float [3] *arr)
      |                                                                      ^
build-standard/lvgl/lv_mpy.c: In function ‘mp_lv_matrix_t_attr’:
build-standard/lvgl/lv_mpy.c:25397:39: error: implicit declaration of function ‘mp_arr_from_float___3____3__’; did you mean ‘mp_arr_from_float___3__’? [-Werror=implicit-function-declaration]
25397 |             case MP_QSTR_m: dest[0] = mp_arr_from_float___3____3__(data->m); break; // converting from float [3][3];
      |                                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                                       mp_arr_from_float___3__
build-standard/lvgl/lv_mpy.c:25397:37: error: assignment to ‘mp_obj_t’ {aka ‘void *’} from ‘int’ makes pointer from integer without a cast [-Werror=int-conversion]
25397 |             case MP_QSTR_m: dest[0] = mp_arr_from_float___3____3__(data->m); break; // converting from float [3][3];
      |

Its type becomes float[3] * , Is this a bug?

I ran into this same issue today when trying to enable LV_USE_VECTOR_GRAPHIC in lv_conf.h.
The python bindings generator doesn't currently support multi-dim arrays at all. I started trying to add support in https://github.com/andrewleech/lv_binding_micropython/commit/ac16d02550c2b4094ae8caae5b67cde246a7b946 but haven't had time to finish it yet.


ARTEMIS-1679 Stopping consumer no need to wait until all queued tasks…

Sping's SimpleMessageListenerContainer shutdown will stop consumers and then close them. It will take lots of time(each consumer costs 10s at most) if there are lots of consumers bcs stop process will wait until all queued tasks done. We assume that waiting for the current task is enough and this will accelerate spring shutting down process.

I will merge this.. but if there's any failures from the full testsuite.. I may have to revert this.


Feat/lesq 430/promote search api [LESQ-430]

Description
Music year: 1961

Removes the feature flag and uses 2023 api on all searches

How to test

Go to https://deploy-preview-2153--oak-web-application.netlify.thenational.academy
Search for 'macbeth' for example
You should see 'new' results first

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


Is there a way to configure environment variables?

Is there a way to set FC environment variables via TF?
https://www.alibabacloud.com/help/doc-detail/69777.html

HI @Omermiz I am sorry for late reply. The provider 1.36.0 has supported this feature: https://www.terraform.io/docs/providers/alicloud/r/fc_function.html#environment_variables. Please check it.
Please let me know if you have other problems.

Closed by no reply.


For large decimals, I think the calculation is not correct

For example this is what I got from the documentation dev console:
var b =<PHONE_NUMBER>1341235532441.12412342352345234234123
undefined
var d =<PHONE_NUMBER>1341235532440.12412342352345234234123
undefined
var B = new Big(b)
undefined
var D = new Big(d)
undefined
b < d
false
b > d
false
B.lt(D)
false
B.gt(D)
false
0.3 - 0.2
0.09999999999999998
0.3 - 0.2 === 0.1
false
new Big(0.3).minus(0.2).eq(0.1)
true
B.toString()
"2.3423523421341234e+22"
b.toString()
"2.3423523421341234e+22"
B.toFixed(3)
"23423523421341234000000.000"
b.toFixed(3)
"2.3423523421341234e+22"

Sorry just realise this is actually working. I just need to manually convert b and d into string first. i.e.,
var b = '23423523421341235532441.12412342352345234234123'
var d = '23423523421341235532440.12412342352345234234123'


buy/sell operation, Error: Place order error: Insufficient funds.

Hi.
Periodically for SELL/BUY operations I see this error:
ExchangeError ('cex {"error":"Error: Place order error: Insufficient funds."}',)
Very strange because balanse is enough for operation.
If I restart script, error not repeat.
P.S. CEX.IO market

Hi, @evgen-y2k ! Can you paste a minimal code snippet to reproduce the problem?

I can not reproduce this yet. I'm trying to determine the cause of the error. Today 9 hours the script worked without errors. During the last hour the error appeared 3 times.

@evgen-y2k do you have funds on open orders? And how do you check the balance? In order to determine the free amounts, you need to look at 'free' balances. Unfortunately, it's impossible to help until we see some code...

Yes, I fully understand how it work! I describe situation:

My script place 5-7 BUY orders. Work correct
If price go up, my script make cancellation of all orders and place new 5-7 orders with new price. At this time can be error for BUY operation. For 5-7 orders script use only 20% of my deposit.  There is always a supply of money.

I think that the possible cause of the error is the limitation on the number of CEX.IO requests (600 requests for 10 minutes maximum). Now I inserted more delay between all operations with orders and started script again. I wait errors ...

@evgen-y2k currently, it's impossible to tell from your text description where the error is. Unfortunately, we can't debug your trading logic for you, so you need to make a minimalistic test to reproduce the bug. Most likely, you have an error with evaluating free and used balances at some stage. I am closing this until you're able to reproduce it. Feel free to reopen it or just continue asking your questions. We will be happy to help, where possible.
Also, if you are having a rate limiting issues, please read this:
https://github.com/ccxt-dev/ccxt/wiki/Manual#rate-limit
https://github.com/kroitor/ccxt/wiki/Manual#authentication
https://github.com/kroitor/ccxt/wiki/Manual#overriding-the-nonce
https://github.com/kroitor/ccxt/wiki/Manual#troubleshooting

P.S. and this: https://github.com/ccxt-dev/ccxt/issues/319

Ok. When I found cause of this error and will can reproduce, I will open new ticket with full description and code example. About your links abowe: I checked this possible cause before opening ticket and all right. I use override nonce in my script (work fine), rate limit my script take from market and use it for check (MIN_LOT_SIZE = float(cex.load_markets()['%s/%s' % (CUR1,CUR2)]['info']['minLotSize']) ..... if float(long_calc_min_order_volume()) / BID < MIN_LOT_SIZE:),
without correct auth script cen't not work e.t.c....

@evgen-y2k

When I found cause of this error and will can reproduce, I will open new ticket with full description and code example.

You can simply reopen this issue, so to keep it simple. We'll stand by for more info from you to reproduce it.

Ok, I found mistake! It is problem with rounding on CEX.IO. In the Rounds Policy described:

14.1. For all financial calculations, CEX.IO uses a rounding policy in the favour of the Platform. CEX.IO rounds the fiat currencies to the 2nd digit after the separator. Rounding policy for cryptocurrencies varies depending on cryptocurrency.

14.4. Cryptocurrencies with rounding to 8th digit after separator are BTC, LTC, NMC.

Nothing about ETH, but for ETH/USD pair it is not correct. For this pair CEX rounds to 6 digit after separator and to down! Not know why. ETH/EUR pair have normal rounds to 8 digits.

@evgen-y2k ok, so, do we need to change something in ccxt?

I think no. It is problem of CEX.

@evgen-y2k ok, let me know if you have any other difficulties with the library. Thx!


">upscale" ignores the index

When using the manual ">upscale" command the bot ignores the index and uses 0. However when you use "/upscale" or the buttons it works fine.

Addendum. This also applies to ">riff" (defaults to 0, "/riff" works fine


Can't build via Nix with configured GitHub token

I configured GitHub PAT (all access to private repos) and placed it into /etc/nix/nix.conf:
access-tokens = github.com=ghp_8J...
...

And when building via Nix, it asks for GitHub username/password. So it seems like it doesn't see my GitHub token because of nil_cluster input type (git instead of github).

nil repo is no more a dependency, consider closing this @akokoshn


Theme examples?

Not a bug exactly but I just read over all the docs and I can't seem to find an example of what theme's look like. I see that there are 10 presets but not what they look like. Any chance this is planned?

I will add a demo of all theme on the theme page with a dropdown so the user can see which colors are in the theme

Here is the codesandbox demo
And, here is the updated website page
https://apexcharts.com/docs/themes/

Just to make sure that i understood the "themes" part correct: once that it is set up, the colors should appear automaticaly or do we need to manualy use the "colors" attribute for the series?


Loss function for LSTM based model

Among the RSGAN, standard GAN, and hinge loss which one did you find to be the best for the LSTM based model?

Actually I didn’t systematically compare these loss functions, in particular for LSTM based model. Experimentally, I found they seem to have similar performance in RelGAN.


Add @dhau/relay-compiler-webpack-plugin

Looks like @dhau has a fork of the relay-compiler-webpack-plugin that supports relay-compiler 1.1.0 and also doesn't require watchman optionally. This PR brings in the namespaced package (upstream has stopped responding).

Any ideas what caused all of these failures @ticky?

Well, there’re some lint errors in the webpack config. I’m not sure what changed about Badge, though.
Also looks like the new plugin is expecting a relay executable in the PATH, which we don’t appear to be installing?

Totally missed those linting errors under the warnings!

I assume the extra output in the json profile analysis is a relay-compiler-webpack-plugin error, have filed an issue: https://github.com/danielholmes/relay-compiler-webpack-plugin/issues/5


Add information about howto modify default route

Can we please add information about changing the default route? This can´t be done in the portal for cross-tenant connections. It can either be done during the creation of the peering or afterwards.
During the creation:
New-AzVirtualHubVnetConnection -ParentResourceId "/subscriptions/9076d78f-8a52-4f8d-8567-11b42296bf31/resourceGroups/RG-VWAN-GER/providers/Microsoft.Network/virtualHubs/WANHUBGER" -Name "crossTConn" `-RemoteVirtualNetworkId "/subscriptions/58e27c9d-67fe-4406-9274-2569368cd48f/resourceGroups/RG-VWANVNET/providers/Microsoft.Network/virtualNetworks/VNETSAP101" -EnableInternetSecurityflag $false
After:
Update-AzVirtualHubVnetConnection -ResourceGroupName RG-VWAN-GER -ParentResourceName WANHUBGER -Name CrossTConn -EnableInternetSecurity $true

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

ID: dcdbb009-ef13-5e6c-d206-6d86418cd270
Version Independent ID: 8684030a-20a7-33ba-9b94-2acf37d613c6
Content: Connect cross-tenant VNets to a hub:PowerShell - Azure Virtual WAN
Content Source: articles/virtual-wan/cross-tenant-vnet.md
Service: virtual-wan
GitHub Login: @wtnlee
Microsoft Alias: wellee

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

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

Thank you for you dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Hardening systemd unit for additional security

Add a couple of options to restrict what prometheus process can see and do.
Mostly taken from:

https://github.com/konstruktoid/hardening/blob/master/systemd.adoc#unit-configuration
https://prog2017.rmll.info/IMG/pdf/2017-07-05_rmll2017_durcissement_avec_systemd.pdf
https://gist.github.com/ageis/f5595e59b1cddb1513d1b425a323db04
https://www.freedesktop.org/software/systemd/man/systemd.exec.html

PR needs some extensive testing to ensure everything works.

Why add the extra config option?  All of these changes look sane enough that users shouldn't need to disable it.

I worry that on older systems it might cause problems. That's why I added an extra config option which by default is set to True.
For example ProtectSystem=strict is available since systemd 232.

Ahh, yes, it would be good to keep compatibility with at least Debian/Jessie and Ubuntu/16.04.  I think that specific feature may need to stay out of the default since 16.04 only has 229.

I did some research:

OS
Systemd version

Debian Jessie
215

Centos 7
219

Ubuntu 16.04
229

Debian Stretch
232

Ubuntu 18.04
237

Fedora 27
238

Feature used in PR with related systemd version (first occurance):

Feature
Version
More info

PrivateTmp
pre 183

ProtectSystem
214
strict option since 232

ProtectHome
214

NoNewPrivileges
?

ReadWritePaths
231
Previously ReadWriteDirectories since pre 199

ProtectControlGroups
232

Fun fact, systemd ignores options it doesn't understand.
I ran this PR against debian jessie (systemd 215) with all hardening options enabled and prometheus works. Journalctl displays some warnings and an error:

May 16 08:16:18 debian-s-1vcpu-1gb-fra1-01 systemd[1]: [/etc/systemd/system/prometheus.service:18] Failed to parse protect system value, ignoring: strict
May 16 08:16:18 debian-s-1vcpu-1gb-fra1-01 systemd[1]: [/etc/systemd/system/prometheus.service:19] Unknown lvalue 'ProtectControlGroups' in section 'Service'
May 16 08:16:18 debian-s-1vcpu-1gb-fra1-01 systemd[1]: [/etc/systemd/system/prometheus.service:20] Unknown lvalue 'ProtectKernelModules' in section 'Service'
May 16 08:16:18 debian-s-1vcpu-1gb-fra1-01 systemd[1]: [/etc/systemd/system/prometheus.service:21] Unknown lvalue 'ProtectKernelTunables' in section 'Service'

"Unknown lvalue" are just ignored, and error can be fixed by lowering security a little and applying ProtectSystem=full instead of ProtectSystem=strict.

May 16 08:16:20 debian-s-1vcpu-1gb-fra1-01 prometheus[1379]: level=info ts=2018-05-16T08:16:20.667936858Z caller=main.go:588 msg="Loading configuration file" filename=
May 16 08:18:01 debian-s-1vcpu-1gb-fra1-01 systemd[1]: [/etc/systemd/system/prometheus.service:19] Unknown lvalue 'ProtectControlGroups' in section 'Service'
May 16 08:18:01 debian-s-1vcpu-1gb-fra1-01 systemd[1]: [/etc/systemd/system/prometheus.service:20] Unknown lvalue 'ProtectKernelModules' in section 'Service'
May 16 08:18:01 debian-s-1vcpu-1gb-fra1-01 systemd[1]: [/etc/systemd/system/prometheus.service:21] Unknown lvalue 'ProtectKernelTunables' in section 'Service'

TL;DR; @SuperQ you were right, we shouldn't have a flag to disable security hardening :smile:

We could have a variable for prometheus_systemd_protect_level: full, so users can flip to strict if they want.


Add a platform_group_assignment  resource

Is your feature request related to a problem? Please describe.
The documentation for JFrog Platform REST API shows that there is a Add or Remove a Group Member method.  This would enable us to have a resource type that allows a 1-2-1 mapping between  users and groups.
Using this new resource allows for independence of the creation of users and groups and treats the assignments separately.   This would result in more items in state, but it allows for fine and granular control, and introduces a single level of responsiblity.
Describe the solution you'd like

I would like terraform resource that allows an assignment of one user to one group.
When the resource is created, it would add the group membership.
When the resource is deleted, it would delete the group membership.
The resource should detect if the resource is deleted.

Describe alternatives you've considered
Adding users to groups and adding groups to users.   This is not ideal.
Additional context

@SKY-Mark-Rhoades-Brown Thanks for the suggestion. We have similar concept in the Project provider (project resource with project_repository, project_user, etc.) to solve the same catch-22 issue, so this will be consistent and make lots of sense.


Masquerade

https://github.com/agrc/masquerade/issues

closing in favor of more specific tasks/features, such as https://github.com/agrc/planning-queue/issues/250


New API

New API for components:

Allocations moved out to be managed by runners;
io.UnexpectedEOF is not handled anymore, EOF should be returned;
Processors encouraged to implement in-place processing algorithms;
Sinks discouraged to write data into buffers.

Buffers pooling.

Resolves #85 #86 #84


Segfault when using VK_KHR_get_physical_device_properties2 on 1.0 instance

Description
Calling vkCreateInstance with VK_KHR_get_physical_device_properties2 in the extensions array and VkApplicationInfo apiVersion set to 1.0.0 segfaults while running in RenderDoc. Either running the application without RenderDoc, removing VK_KHR_get_physical_device_properties2 from the extensions array or setting the apiVersion to 1.1.0 resolves this issue.
Steps to reproduce
I don't think it is necessary to supply a example as this is very basic code.
It may be possible that this bug only occurs on certain drivers/hardware/operating systems.
Environment

RenderDoc version: 1.9 (https://aur.archlinux.org/packages/renderdoc-bin/)
Operating System: Arch Linux latest
Graphics API: Vulkan
GPU: RX 580
Drivers: Mesa 20.1.5

Usually I try hard to be polite, as it's easy for someone reading along to get the wrong impression from a rude reply that may not be seen in context.
I hope though that anyone can see the sheer arrogance required for someone to decide that they are too important to bother providing the reproducible test case explicitly asked for when filing a bug asking for free support on a free program. Closing this issue.

I am very sorry about this, i thought i would be less of a hassle for reproducers to setup a quick test program than trying to set up my program.


oninit is not running for nested components

When using the rewrite branch mq is not running component oninit function for nested components.
My simple guess is that in https://github.com/StephanHoyer/mithril-query/blob/rewrite/index.js#L106
states[treePath] is already defined and therefore oninit is never triggered.
test('component', function(t) {
    var result = {
        oninit: 0,
        view: 0,
    };

    var component = {
        oninit: function() {
            result.oninit++;
        },
        view: function(vnode) {
            result.view++;
            return m("div.test", vnode.children);
        },
    };

    mq(
        m(component, [
            m(component, [
                m(component), // <-- oninit is not running for this component.
            ]),
        ])
    );

    t.equal(result.oninit, result.view,
        'oninit should run before every view');

    t.end();
});

I currently haven't the time to fix this. PR would be highly appreciated.

fixed with #65


ld-linux-x86-64.so.2 missing on alpine linux

added CGO_ENABLED=0 to Makefile to force statically linked binaries


How to move left and right sidebar further apart and expand width of body?

Hi @mmistakes ,
Thanks for the work again.
I am trying to give more space to the body of text so that two sidebars are pushed further to left and right. I am kindly wondering which variable I should change without damaging other functionality.
Thanks for your help.

Duplicate of #1373, #1265, #623, #384, #74

@mmistakes Thanks for pointing out. I can successfully extend the right sidebar to further right. However, I am wondering if I can extend left sidebar to further left as well. Is there a simple change to that?

No simple way. You write some custom CSS just like you would for the right sidebar.
You'll have to reduce the with of the main content to make room for the left increase.

@mmistakes Thanks for the reply.
Also, another question is that did you have padding-left as well? I feel like you also "fake" some empty space on the left as well. Is it what you did?

Your browser's web inspector is what you'll want to use here. Inspect elements and it'll tell you the exact CSS I'm using.


Fix locally registering part of manual

The manual for locally registering the component was incomplete.
Using the :code prop breaks the syntax highlighting, if you use v-model, it works.

thanks, fixed 🙏


Update sbt-scalajs, scalajs-compiler, ... to 1.11.0

Updates

org.scala-js:sbt-scalajs
org.scala-js:scalajs-compiler
org.scala-js:scalajs-junit-test-plugin
org.scala-js:scalajs-junit-test-runtime
org.scala-js:scalajs-junit-test-runtime_2.13
org.scala-js:scalajs-library
org.scala-js:scalajs-library_2.13
org.scala-js:scalajs-test-bridge
org.scala-js:scalajs-test-bridge_2.13

from 1.5.1 to 1.11.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.5.1).
You might want to review and update them manually.
README.md

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scala-js" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, old-version-remains, commit-count:1

Superseded by #51.


Speed up feature extraction by using numpy feature extractions

Hey @dirko, what do you think about this approach. It's still pretty clear what's going on and it's much much faster.

Giving the choice of how to vectorise the feature generation to the user makes a lot of sense to me. So as I understand it each binary feature function must return a slice through the final three dimensional ndarray. Can you maybe just document this new API here - then I'll merge. Thanks!

Sorry. Where do you want the documentation?

Ah it seems that the line number disappeared - Line 43 of feature_extraction.py.

How does that look?

Great!


Not clear how to run set the kernel for a notebook to C++

Hi,
I was trying to use a C++ kernel with nbterm. I have already installed the xeus-cling kernel and have it working with my jupyter notebook environment.
I see the "--kernel-cwd" option but nothing that would allow me to specify the kernel name.
Looking through the code I see that kernel_name is a parameter to the KernelDriver initialization but I can't seem to figure out how to specify the C++ kernel.
Thanks!

Hi @armaninspace,
There is currently no way to choose a kernel, if you create a new kernel it will use a Python kernel. But that will be a nice improvement, do you want to open a PR?

Hello,
I added some basic kernel selection to the nbtermix fork. Feel free to reuse the idea.
https://github.com/mtatton/nbtermix
Best wishes


Write tree generator benchmark

something like a custom generator in Haskell’s QuickCheck. E.g. generate a 100,000-node AST of algebraic expressions using mutually recursive calls (kind of opposite of parsing).

Originally Huang and Weixin worked on it, but it was closed on Bitbucket, since it wasn't a pressing issue


RazorComponents - Mix of IsFixed Cascading values causes Exception

Mix of IsFixed==true and IsFixed==false cascading values leads to exception in the unsubscribe method.
Error: System.NullReferenceException: Object reference not set to an instance of an object.
   at Microsoft.AspNetCore.Components.CascadingValue`1.Microsoft.AspNetCore.Components.ICascadingValueComponent.Unsubscribe(ComponentState subscriber)
   at Microsoft.AspNetCore.Components.Rendering.ComponentState.RemoveCascadingParameterSubscriptions()
   at Microsoft.AspNetCore.Components.Rendering.ComponentState.DisposeInBatch(RenderBatchBuilder batchBuilder)
   at Microsoft.AspNetCore.Components.Rendering.Renderer.RenderInExistingBatch(RenderQueueEntry renderQueueEntry)
   at Microsoft.AspNetCore.Components.Rendering.Renderer.ProcessRenderQueue()

Behaviour of ComponentState.AddCascadingParameterSubscriptions and ComponentState.RemoveCascadingParameterSubscriptions is suspicious. Subscribe is called only when IsFixed is false. CascadingValue with IsFixed = true is not subscribed. CascadingValue.Subscribe initialize _subscribers property. CascadingValue.Unsubscribe calls _subscribers.Remove.  ComponentState.RemoveCascadingParameterSubscriptions calls Unsubscribe for each cascading value (doesn't care if it is fixed or not) and it leads to call Unsubscribe on nonsubscribed value and crash.

Consider following change in the CascadingValue class:
void ICascadingValueComponent.Unsubscribe(ComponentState subscriber)
{
if (_subscribers != null)
_subscribers.Remove(subscriber);
}

Thanks for contacting us, @ladislavherman.
Will you be interested in sending us a PR, we'll happily consider it.
//cc @SteveSandersonMS

Fixed in https://github.com/aspnet/AspNetCore/pull/11854


Trim white spaces before and after the equal sign

This fixes #7888
Also cleans up the code for trimming and squeezing spaces.

The original code is terrible, you should take this opportunity to fix it. Foremost, don't use stringstream.  Why not something like
std::string out;
out.reserve(in.size());
size_t n = 0;
for ( char c : in ) {
  if (!isspace(c)) { out << c;  ++n; }
}
out.resize(n);
return out;

[approve ci]

Cherry-picked to v9.2.x, merged both of these fixes. @duke8253 let me know if this is not desirable for 9.2.x.


PowerDnsProvider does call to PowerDNS server for version information

As part of the new mode_of_operation octodns-powerdns needs to validate wether the underlying PowerDNS supports the mode.
https://github.com/octodns/octodns-powerdns/blob/42f3de5fcf908dda73ff8e22da63be42f97aa24d/octodns_powerdns/__init__.py#L96
octodns-powerdns does that via calling to PowerDNS and asking for its version:
https://github.com/octodns/octodns-powerdns/blob/42f3de5fcf908dda73ff8e22da63be42f97aa24d/octodns_powerdns/__init__.py#L283-L303
This creates issues in CI, when octodns-validate doesn't have the production credentials, which obviously it shouldn't.
We went around the issue, by using our own PowerDnsProvider which instantiates octodns-powerdns and then we override the mode_of_operation setter.
https://github.com/octodns/octodns-powerdns/blob/42f3de5fcf908dda73ff8e22da63be42f97aa24d/octodns_powerdns/__init__.py#L335-L345
Its not the nicest solution and maybe there is a better way out, by either providing PowerDnsProvider with the version number, so it doesn't have to call PowerDNS or the ability to disable the call for CI.
A gist of where it failed in CI:
https://gist.github.com/fjaeckel/6500b24b822492916cf2b58aea1bedc9#file-gistfile1-txt

by either providing PowerDnsProvider with the version number, so it doesn't have to call PowerDNS or the ability to disable the call for CI.

Either of those may work. The other option that came to mind would be to defer/delay the call until the if/when the mode_of_operation was specifically required. Presumably/hopefully octodns-validate could run without needing that answered.


[FLINK-13331][table-planner-blink] Add CachedMiniCluster to share cluster between ITCases

What is the purpose of the change
Add CachedMiniClusterWithClientResource to share cluster between ITCases.
This should reduce a lot of test time.
Verifying this change
This change is already covered by existing tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? JavaDocs

We need do some work to override false reuseForks from https://github.com/apache/flink/pull/2003

I find a potential problem of this approach. After running blink planner IT cases, the cluster resources
are still being hold. It may leaks resources (threads, memory) and affect following tests?

This approach will always leak resources as @wuchong pointed out.
I have doubts whether this will improve the test times in a significant way, since we're only skipping the creation of ~60 mini clusters.
Nevertheless please also provide actual numbers when doing optimizations such as this; i.e., how long did the tests take beforehand, how long do they take now, so we can gauge how valuable this change is.

@wuchong @zentol Thanks for your attention.
About leak resources, there is a clean thread to invoke MiniClusterWithClientResource.after to clean resources. Compared with the past, it only delayed cleaning up resources by 15 seconds.

I have doubts whether this will improve the test times in a significant way, since we're only skipping the creation of ~60 mini clusters.

Yeah, we should gauge how valuable this change is.
1.I test on my macbook pro, blink-planner will use 17min before, and use 11min now for mvn clean verify. If we just run mvn verify, the gap will be more pronounced.
2.But the more accurate test time should be travis triggering, I will update later.

Now blink-planner test time is 31-32min: https://travis-ci.org/JingsongLi/flink/jobs/562395265
Without this PR, test time is almost 50min: https://travis-ci.com/flink-ci/flink/jobs/218419037
@zentol @wuchong FYI.

Based on the test run in #9306 it appears that the speed-up is mostly caused by the reuse of forks, so we may just go with that without the caching of clusters.

Based on the test run in #9306 it appears that the speed-up is mostly caused by the reuse of forks, so we may just go with that without the caching of clusters.

Do you know any reason why we disabled reuse of forks before?

The testing time of blink planner is 30min if we enable reuse forks. So I'm +1 with @zentol 's opinion.

Not re-using forks is literally the only difference between unit tests and IT cases, and I would presume we did that to ensure they are fully isolated from another.
In other words, instead of messing with the reuseForks setting, we could just label them all as unit tests.

However, IMO, integration-test category is helpful. For example, the *ITCase suffix is helpful sometimes because they are easy to distinguish with unit tests. And we are still planning to reduce ITCase time in blink planner in some other ways.
If isolation is the purpose, I think we can enable it for blink planner tests, because we have an individual stage for blink planner ?

Well in any case we would be enabling it for the blink planner; be it by enabling fork reuse of marking them as unit tests.
So yes, we can just set it to true in the blink planner.

Thanks @zentol .
@JingsongLi could you prepare another pull request with only enabling reuse forks for blink planner?

Maybe there are some stories why start cluster and shutdown cluster so quick in travis when enable reuse forks. In my local IDE, it still very slow.
But I am +1 to speed up testing to blink-planner by simple code modify.
Maybe we can just use #9306

#9306 has been closed. I created a PR #9326 to continue the work.
Closing this PR.


Added HCL formatter

summary
Introduced HCL Formatter for *.hcl filetype.
References

https://github.com/gruntwork-io/terragrunt/issues/1037#issuecomment-583057827
https://terragrunt.gruntwork.io/docs/getting-started/configuration/#formatting-hcl-files

Thanks! Had no idea that this was even a thing 😄


docs: add Hera workflows URL to the client library doc

Don't bother creating a PR until you've done this:

[ ] Run make pre-commit -B to fix codegen, lint, and commit message problems.

Create your PR as a draft.

Your PR needs to pass the required checks before it can be approved. If the check is not required (e.g. E2E tests) it
does not need to pass.
Once required tests have passed, you can make it "Ready for review".
Say how how you tested your changes. If you changed the UI, attach screenshots.

Tips:

If changes were requested, and you've made them, then dismiss the review to get it looked at again.
Add you organization to USERS.md if you like.
You can ask for help!

Could you fix the checks?

Done. Thank you, Terry!


change to MIT license

kindly review
deleted the previous license- it doesn't allow for script reusing
changed to MIT license for infrastructure/scripting as per the issue #421

kindly review deleted the previous license- it doesn't allow for script reusing changed to MIT license for infrastructure/scripting as per the issue #421

my name is Oluwatobi Adelakun, an outreachy applicant, please kindly review the pull request and await your feedback

Thanks @oluwatobiBolu for this contribution-
It is usually better to have a Pull Request for 1 issue. Could you update this Pull Request to focus only on the License?
Did you make sure that CC-BY-SA 4.0 (for content) is mentioned on the README, and on the website?

Thanks @oluwatobiBolu for this contribution- It is usually better to have a Pull Request for 1 issue. Could you update this Pull Request to focus only on the License? Did you make sure that CC-BY-SA 4.0 (for content) is mentioned on the README, and on the website?

i have reverted the last commit so the pull request focuses on just the license issue
thanks for the feedback

i think this is appropriate.

what do you think?
i didn't necessarily have to write all the conditions.
does the summary cover it?

There is a conflict there. Can you resolve it?

GitHub still says there is a conflict on the README file. Could you try to resolve it? Thanks

Thanks @oluwatobiBolu for fixing the conflict.
I am thinking about something: could you also change the footer in the website (in the _includes folder) to add a sentence similar to the one in the README file about the infrastructure License? That would really clarify what people can do. Thanks

It is a Jekyll plugin: https://jekyll.github.io/github-metadata/

It is a Jekyll plugin: https://jekyll.github.io/github-metadata/

thanks for the resource

@bebatut  I think there are no more changes to be made here..and hopefully can be merged

We just need the approval from @malvikasharan there. But she is currently in holidays

Hi @malvikasharan just tagging you here for your approval :)

So late to this! Apologies all. I had not been previously oinged until recently @bebatut brought this to my attention. I have approved this and merging this.


How to add the plugin markdown-it-mermaid？

https://github.com/tylingsoft/markdown-it-mermaid
↑this one
how should I edit my _config.yml file?

markdown-it-mermaid plugin depends on yarn. What you just should do is to install markdown-it-mermaid by using yarn --- yarn install markdown-it-mermaid --save, and build it if should. Then edit your _config.yml like this:
markdown:
    plugins:
        markdown-it-mermaid:   # params is not necessary here!

It should work now. I have not given it a try because I use npm. Sorry!


Tooltips for exported variables

Backward compatible solution to add custom tooltips for exported variables.
Uses comments syntax, so it won't throw errors on previous versions. Tooltips are parsed only in editor. Supports BBCode.
## [b]Tooltips[/b] for exported variables. [br][img]icon.png[/img] [br]With [color=green]BBCode[/color] support. [br]Enjoy!
export var with_tooltip = 2

Related issue https://github.com/godotengine/godot/issues/6204

Could this be expanded to be a general system for inline documentation, similar to the /// <summary> XML comments in C#? There's an issue somewhere about generating in-editor documentation for user-created scripts.

@aaronfranke The proposal you're referring to is https://github.com/godotengine/godot-proposals/issues/177.

Is this something that @vnen should be considering as another type of GDScript annotation?

Yeah, this should be added as an annotation.
However, @ThakeeNathees is working on showing GDScript documentation

Eagerly waiting for this to get merged and usable!

@ShivamMukherjee This pull request will most likely conflict with https://github.com/godotengine/godot/pull/39359 and the GDScript rewrite, so I doubt it'll be merged.

@Calinou should we consider export variable doc comments as tool tips ?

I'll be Waiting for Godot then! As long as I get to use tooltips like a sane person again.

@ThakeeNathees only if the export variable's doc comments are also rendered as editor tooltips. Has that been confirmed by @vnen?

@willnationsdev No, I'm just suggesting that to use it as a tooltip yet no decision has made.

@willnationsdev , @vnen agreed to use the documentation of export variable to use it as tooltip,
here is a working example

This feature is now available in master (what will become 4.0 in the future) thanks to https://github.com/godotengine/godot/pull/41095.
Should we aim to support this in 3.2? We should aim to use a compatible syntax so that projects can be upgraded smoothly without having to update comments.
cc @vnen

This feature is now available in master (what will become 4.0 in the future) thanks to https://github.com/godotengine/godot/pull/41095.
Should we aim to support this in 3.2? We should aim to use a compatible syntax so that projects can be upgraded smoothly without having to update comments.
cc @vnen

@Calinou I fixed auto-rebase issues and the comment with trailing comma in the same line. I've also added the solution for multiline tooltip comments, which you asked about at the very beginning and I missed this earlier.

man i would really like this feature. i'm designing an interface for our writers to use without touching much code and tooltips would be massively helpful.

Apparently this implementation works with built-in scripts, unlike the 4.0 one. However, it breaks compatibility, due to the token issue mentioned before. I tend to put ## in comments for stuff I want to check layer, so it's easier to find. If this was merged, all scripts using ## "incorrectly" will break.

@KoBeWi could you please test those corner cases on newer version of my PR? I found some issues in the release build too, so it should be covered now. Right now it will generate token only if ## comment is just before export variable.

How difficult is it to make a markdown or even standard XML comment parser?

How difficult is it to make a markdown or even standard XML comment parser?

I wouldn't add Markdown syntax here, as BBCode is what we went for in 4.0.

Does anyone know if tooltips from documentation (e.g. XMLDoc) are available for C# [Export] members in 4.0? I tried a simple test and it doesn't seem like it's supported (yet?).
If not, where is an issue/PR I can watch to track this? Sorry if this is the wrong place to ask. I tried to find a specific issue for C#, but I only found this. Should I open a new issue?
For example, something like this:
///<summary>Tooltip blah blah </summary>
[Export] string myProp;

Or even something like this, if you want to be more flexible?
///<tooltip><summary>Tooltip blah blah </summary></tooltip>
///<remarks>Not in tooltip.</remarks>
[Export] string myProp;

@geekley It's only supported in GDScript in 4.0. This is not the place to discuss C# support though; please open a proposal for that.

Thanks Calinou! I was going to, but turns out there is a proposal already: https://github.com/godotengine/godot-proposals/discussions/5675
Linking here in case people want to upvote it.

There's also properties exported via _get_property_list().
Looks like something that should be tightly related, but I'm not familiar with the code.
I wonder if implementing this for those should better be done in another PR,
or could just be done in one go.


Local queue service will not load

Preflight Checklist

[x] I have installed the latest version of Storage Explorer.
[x] I have checked existing resources, including the troubleshooting guide and the release notes.
[x] I have searched for similar issues.

Storage Explorer Version
1.36.2
Regression From
No response
Architecture
arm64
Storage Explorer Build Number
20241029.2
Platform
macOS
OS Version
macOS 14.6.1 (23G93)
Bug Description
Queue Service times out when attempting to connect to local azurite container instance.
Steps to Reproduce

Create docker compose file with latest azurite instance (referenced below)
$> docker compose up
Open Storage Explorer
Select Emulator & Attached > Storage Accounts > docker emulator instance (or Emulator - Default Ports) > open tree for "queues" > timeout after 15 minutes.

Actual Experience
The "queue" service does not successfully perform a GET request to the azurite instance to retrieve a list of queues available. It gets stuck loading. After 15 minutes, a timeout error is produced, referenced below:
Unable to retrieve child resources

ProducerError:{
  "name": "Node Fetch Error",
  "message": "Headers Timeout Error",
  "stack": "HeadersTimeoutError: Headers Timeout Error\n    at Ev.onParserTimeout [as callback] (/private/var/folders/j4/fsxxqtb925v8607vkld7fdhr0000gn/T/AppTranslocation/640BC08B-45C8-496D-A003-E3CC92B1C318/d/Microsoft Azure Storage Explorer.app/Contents/Resources/app/out/app/index.js:73:11116)\n    at Timeout.onTimeout [as _onTimeout] (/private/var/folders/j4/fsxxqtb925v8607vkld7fdhr0000gn/T/AppTranslocation/640BC08B-45C8-496D-A003-E3CC92B1C318/d/Microsoft Azure Storage Explorer.app/Contents/Resources/app/out/app/index.js:62:38070)\n    at listOnTimeout (node:internal/timers:581:17)\n    at process.processTimers (node:internal/timers:519:7)",
  "cause": {},
  "code": "UND_ERR_HEADERS_TIMEOUT"
}

Blob and Table services work as expected. I can create, read, update, and delete containers, tables, rows, etc.
Other services, such as a custom web app, can CRUD from queue service hosted in container instance. Only the storage explorer times out attempting to connect.
Expected Experience
When selecting the "Queues" tree, the expectation is that a list of available queues will be populated. In my example provided in additional context, the expectation is that the storage explorer will determine that there are no queues available and provide me with the option to create a new queue.
Additional Context
No settings changes to Azure Storage Explorer (I didn't even know it had a dark mode now lol)
Docker instance is version 4.36 (latest as of today)
Azurite instance is latest (3.33), determined by opening container -> terminal -> "$> azurite --version"
docker-compose.yaml:
  services:
    azurite-serve:
      image: mcr.microsoft.com/azure-storage/azurite
      container_name: azurite-internalapp
      ports:
        - "10000:10000"
        - "10001:10001"
        - "10002:10002"
      command: 'azurite --location /data --debug /data/azurite_debug.log --blobHost <IP_ADDRESS>  --queueHost <IP_ADDRESS> --tableHost <IP_ADDRESS>'
      volumes:
        - ./storage/azurite:/data
      networks:
        - internalapp
  networks:
    internalapp:

volume ./storage is empty and created when containers are spun up. Also fails when connecting to directory with previous local azure storage.
Messages produced by the following:
$> docker compose up
[+] Running 1/0
 ✔ Container azurite-internalapp  Created                                                                                0.0s
Attaching to azurite-internalapp
azurite-internalapp  | Azurite Blob service is starting at http://<IP_ADDRESS>:10000
azurite-internalapp  | Azurite Blob service is successfully listening at http://<IP_ADDRESS>:10000
azurite-internalapp  | Azurite Queue service is starting at http://<IP_ADDRESS>:10001
azurite-internalapp  | Azurite Queue service is successfully listening at http://<IP_ADDRESS>:10001
azurite-internalapp  | Azurite Table service is starting at http://<IP_ADDRESS>:10002
azurite-internalapp  | Azurite Table service is successfully listening at http://<IP_ADDRESS>:10002

After selecting tree for "azurite-internalapp", then opening tree for blob followed by table:
[+] Running 1/0
 ✔ Container azurite-internalapp  Created                                                                                0.0s
Attaching to azurite-internalapp
azurite-internalapp  | Azurite Blob service is starting at http://<IP_ADDRESS>:10000
azurite-internalapp  | Azurite Blob service is successfully listening at http://<IP_ADDRESS>:10000
azurite-internalapp  | Azurite Queue service is starting at http://<IP_ADDRESS>:10001
azurite-internalapp  | Azurite Queue service is successfully listening at http://<IP_ADDRESS>:10001
azurite-internalapp  | Azurite Table service is starting at http://<IP_ADDRESS>:10002
azurite-internalapp  | Azurite Table service is successfully listening at http://<IP_ADDRESS>:10002
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:19 +0000] "GET /devstoreaccount1?comp=properties&restype=account HTTP/1.1" 200 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:19 +0000] "GET /devstoreaccount1?comp=properties&restype=account HTTP/1.1" 200 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:22 +0000] "GET /devstoreaccount1?comp=list&include=metadata HTTP/1.1" 200 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:22 +0000] "GET /devstoreaccount1?comp=list&include=metadata&timeout=30 HTTP/1.1" 200 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:22 +0000] "GET /devstoreaccount1/%24logs?restype=container HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:22 +0000] "GET /devstoreaccount1/%24blobchangefeed?restype=container HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:25 +0000] "GET /devstoreaccount1?comp=list&include=metadata HTTP/1.1" 200 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:25 +0000] "GET /devstoreaccount1/%24logs?restype=container HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:25 +0000] "GET /devstoreaccount1/%24blobchangefeed?restype=container HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:25 +0000] "GET /devstoreaccount1?restype=service&comp=properties HTTP/1.1" 200 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/Tables HTTP/1.1" 200 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsCapacityBlob()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsHourPrimaryTransactionsFile()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsHourPrimaryTransactionsBlob()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsHourPrimaryTransactionsQueue()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsHourSecondaryTransactionsBlob()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsHourPrimaryTransactionsTable()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsHourSecondaryTransactionsFile()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsHourSecondaryTransactionsQueue()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsMinutePrimaryTransactionsBlob()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsHourSecondaryTransactionsTable()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsMinutePrimaryTransactionsQueue()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsMinutePrimaryTransactionsFile()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsMinutePrimaryTransactionsTable()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsMinuteSecondaryTransactionsFile()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsMinuteSecondaryTransactionsQueue()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsMinuteSecondaryTransactionsBlob()?$top=1 HTTP/1.1" 404 -
azurite-internalapp  | <IP_ADDRESS> - - [18/Dec/2024:14:32:26 +0000] "GET /devstoreaccount1/%24MetricsMinuteSecondaryTransactionsTable()?$top=1 HTTP/1.1" 404 -

Selecting "queue" tree provides no logging, error message, etc. The timeout that is produced after ~15 minutes is pasted in section "Actual Experience"

Did not think about images.
Where queue service gets stuck loading:

After about 5 minutes:

Production resources have no issues connecting to queue services.

@gallowasusc Not able to reproduce on my end. I'm able to list and create queues as expected.
Click on the Docker node in the tree view and check the endpoints. Do they look right to you?

Also try creating a manual connection to the docker container to make sure that the queue service can receive requests.

Your comment about manual connections inspired me to make the troubleshooting even more vanilla. I removed the docker container from the equation and attempted to spin up azurite on the host. The queue service would not start. I ran the following:
sudo lsof -i -P | grep LISTEN | grep :$PORT
and discovered a vscode process listening on port 10001. It persisted through restarts and an uninstall of the vscode extension. Ran a kill command on that process, retried the docker container and now queues work as intended.

I'd been troubleshooting this one for days, but I'm glad it was just a user error. Thank you for your time!


feat: batch support for owlv2

closing in favor of https://github.com/landing-ai/vision-agent-tools/pull/72


[Doc] Action.Id property missing in schema explorer

Problem Description
Even though all our SDKs support/expose the Action.Id property, it is missing from our doc. We need to add it.
Schema Version
All schema versions.

This was fixed in PR #5813. The website just needs to be deployed.


Local installation isn't documented

When running the game locally, the instructions are currently incorrect:

Both instructions and install.sh assume that python and pip refer to Python 3. Usually that will be python3 and pip3 however.
You need to run sudo pip3 install gsutil before you can run install.sh
The tensorflow module is an additional requirement not listed in requirements.txt.

Also, should be tensorflow==1.15 since version 2 won't work.

Strange, pip installed tensorflow 1.14.0 for me yesterday. No idea why...

Let alone trying to install it on Windows 10, referring to a problem in grpcio ( https://github.com/grpc/grpc/issues/20831 ) you currently cannot install tensorflow after all using python 3.8.
Will try it with older versions and supply a full install for windows10 if needed.

@nEOtechnican I just removed the termios related lines (import and 1 code line) and it works fine on Windows

@nEOtechnican I just removed the termios related lines (import and 1 code line) and it works fine on Windows

already did and on to the next, lots of thanks.

I will share how I made it work on Ubuntu 18 running from pendrive on my 12GB notebook:

apt install python3-pip
pip3 install -r requirements.txt
cd generator/gpt2
pip3 install -r requirements.txt
pip3 install pyyaml
pip3 install tensorflow==1.13.1
cd ../..
(note that 1.12 as stated on the original GPT-2 won't work as it will throw an error on play.py)

Then to run it:

python3 play.py

Seems to be working just fine in the terminal:

Could we just put this in a docker image? Would simplify a lot and I don't think there's any size limits?

Not quite fixed - I listed three points in the issue description, and only the last one was addressed.

Would need to be an Nvidia docker image though to take advantage of cuda

Creating a docker image from tensorflow/tensorflow:latest-gpu-py3 with embedded the model_v5 folder is indeed very simple and also there is no limit to images size on docker hub. I can create a Dockerfile for you if you want.


fix(server): fix graceful_shutdown on freshly opened connections

When opening a http1 connection, the keep_alive state is set to Busy,
preventing a graceful_shutdown in that case. With this patch the
keep_alive disabling and closing of idle connections is decoupled and
should fix this issue.
Closes #1885
Although all tests go through (afaict), i am not entirely sure this is desired behaviour, though
we run into an issue where we use the graceful_shutdown for a daemon reload and an
open connection (with no data) can block our reload... (this issue fixes that)
Signed-off-by: Dominik Csapak<EMAIL_ADDRESS>

seems most of the ci tests time out, and the one test on macos that fails i cannot reproduce here on linux (i have no access to a mac)

Thanks for the PR!
I'm not sure that accepting a connection should immediately put in an idle state, ready to shutdown. We'd usually assume that a new connection means a request is coming soon. It also affects what a parsing error means.
I think the issue this is linking to should instead just be dealt with by a user selecting on a timeout after triggering a graceful shutdown. It may be that the current graceful shutdown API doesn't easily allow for that, I haven't thought too hard about that part.

Thanks for the PR!
I'm not sure that accepting a connection should immediately put in an idle state, ready to shutdown. We'd usually assume that a new connection means a request is coming soon.

ok no problem

It also affects what a parsing error means.

in what way exactly? (just to understand)

I think the issue this is linking to should instead just be dealt with by a user selecting on a timeout after triggering a graceful shutdown. It may be that the current graceful shutdown API doesn't easily allow for that, I haven't thought too hard about that part.

we solved a part of the issue another way, but is there any mechanism to have a timeout on an incoming connection until the service is called? i did not really find an example of this

I'm not sure that accepting a connection should immediately put in an idle state, ready to shutdown. We'd usually assume that a new connection means a request is coming soon.

@seanmonstar It was already a long time ago when I was thinking about this but it is not true that a request can be coming soon on any already idle keep-alive connection? Why would it be ok to close such a connection but not a freshly opened connection on which still did not arrive first request?

I'm not sure that accepting a connection should immediately put in an idle state, ready to shutdown. We'd usually assume that a new connection means a request is coming soon.

@seanmonstar It was already a long time ago when I was thinking about this but it is not true that a request can be coming soon on any already idle keep-alive connection? Why would it be ok to close such a connection but not a freshly opened connection on which still did not arrive first request?


Fatal error: Current target does not support the following CPU features that are required by the image: [FP, ASIMD]

Dear, when trying to run the apk, it launches the message, and that I have tried with a new project, I use wsl, clean the distro in case it was something in the configuration, but it continues to give the same message, please help, I need to do a Program.

I have started a new Gluon project, and when creating the Apk, everything is fine, but I use Android studio to test the project, and I get the message, I have installed Ubuntu from Zero, and still not. I hope your help, I use Graalvm according to the documentation, and jdk 11, everything is from scratch, I have not modified the project that is created from gluon.start at all, nor added any library.

Are you using the Android emulator? That is not supported (the APK doesn't include libraries for the x86_46 architecture), you need to test on a real device.
It might work if you use an emulator of ARMv8.


implemented velocity dispersion function

@ajshajib it seems like, again, that the black formatter is not pushing the format changes. Did this just change again?

@nkhadka21 do you happen to know why the pre-commit PR does not work?

@ajshajib it still fails, probably due to different black versions. Also, how do I run these locally? It really should not happen and I wonder whether black adds more troubles than it solves

I don't exactly know why this is failing. Currently, it's failing due to docformatter, which is probably the main cause for the previous failures as well.
To run locally, just pip install black and pip install docformatter, then do:
black .
docformatter ./* -r --black --in-place

I only ran black in my last commit, so please try running docformatter on your local machine to check if that solves the problem for now.

Pre-commit CI gives this message:

pre-commit.ci was unable to push due to branch protection rules

Did anything change in the branch protection rules recently?

Pre-commit CI gives this message:

pre-commit.ci was unable to push due to branch protection rules

Did anything change in the branch protection rules recently?

hmm, this could have been the issue on my side

Thank you @ajshajib! I could manually solve it and added the commands in the make file as well in this PR, should be good to go once the tests pass

A point on pre-commit is that the originally intended workflow for pre-commit is to have it run locally. The name pre-commit comes from the fact that if it is installed locally, then the hooks (for black, docformatter, etc.) would automatically run locally at every git commit command ("pre" of every attempted commit). The way I added the pre-commit system to this repo is a workaround as more like a "post-commit" enforced at the main repo level on GitHub.
Preferably, pre-commit needs to be installed by every developer with this:
pip install pre-commit
cd /path/to/sim-pipeline/directory
pre-commit install

In that case, pre-commit should be added to the requirements.txt.


Error running the filter

Thank you very much for making this filter. I'm getting an error message that I am not sure how to read and fix it. Any thoughts? Many thanks.
Traceback (most recent call last):
File "/usr/local/bin/pandoc-codeblock-include", line 9, in
load_entry_point('pandoc-codeblock-include==0.0.2', 'console_scripts', 'pandoc-codeblock-include')()
File "/Library/Python/2.7/site-packages/pandoc_codeblock_include.py", line 40, in main
return run_filter(include, doc = doc)
File "/Library/Python/2.7/site-packages/panflute/io.py", line 265, in run_filter
return run_filters([action], *args, **kwargs)
File "/Library/Python/2.7/site-packages/panflute/io.py", line 252, in run_filters
dump(doc, output_stream=output_stream)
File "/Library/Python/2.7/site-packages/panflute/io.py", line 161, in dump
ensure_ascii=False  # For Pandoc compat
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/init.py", line 250, in dumps
sort_keys=sort_keys, **kw).encode(obj)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/encoder.py", line 210, in encode
return ''.join(chunks)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 3339: ordinal not in range(128)
Error running filter pandoc-codeblock-include:
Filter returned error status 1

Can you please post your files somewhere for testing?


pull request for e2e test at 12/02/2016 23:45:03

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

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


Fix Cirrus/Packer VM image building

Fix some encrypted values blocking image building.
Add feature to only build images on PR merge if PR description has magic words (so we don't collect up a billion useless images).
Centralize definitions used to name newly produced VM images.
Add some documentation for the cirrus setup.

CIRRUS: REBUILD IMAGES

@baude mind take a quick peek when you get a moment?
Summary:

Fix some encrypted values blocking image building.
Add feature to only build images on PR merge if PR description has magic words (so we don't collect up a billion useless images).
Add some documentation for the cirrus setup.

@baude We need you to review this?

@rhatdan ya, @baude knows the cirrus stuff pretty well.  Though most of its just bash, and this PR adds docs to describe the work-flow.  So (I would hope) just about anyone could wrap their head around it :smiley:

rebased and --force pushed.

rebased and --force pushed.

LGTM

@baude PTAL

/retest

/retest

Hrmmm, an actual integration-test failure.  Rebased again, and --force pushed, hopefully that picked up a needed fix.

Drat, failed twice in the same way.  It's the dreaded 'cgroups: cgroup mountpoint does not exist' in both:

test.test_pods_ctnrs.TestPodsCtnrs
test.test_pods_no_ctnrs.TestPodsNoCtnrs

@jwhonce I thought Matt/Brent fixed the underlying issue, but maybe I'm misunderstanding things.  If it would help, I can put that IRC nagging back in so you can poke around inside the VM.  Let me know what I can do.
In the meanwhile, I'll smash the re-run button since this problem seems to be intermittent (other PR testing is passing).

@jwhonce nevermind, chatted with Matt about this.

@mheon @baude This PR is medium-important IMHO, it adds some docs, a fix, and a mechanism so we don't go crazy building new VM images on every merge.  PTAL when you have a mo (esp. at the docs).

LGTM

LGTM

Thanks @TomSweeneyRedHat

/lgtm

Rebased against latest master and --force pushed.

@rhatdan @TomSweeneyRedHat @mheon @baude rebased, and green here, PTAL when you have a moment.

/approve

/lgtm


All first column should always have a copy button

What happened?
In the Dataplane proxies list for example it's missing.
Let's go over all our lists and make sure we have copy buttons.

Couple things we said if we are ever looking for the reasoning in the future:

We should always copy the thing the user can see, so if we see name-of-thing we should copy name-of-thing not name-of-thing.namespace.
Because we wrap the "name of the thing" in a link, its slightly more difficult to copy by just selecting and hitting ctrl-c, therefore there is a good reason to use a copy button on these things rather than just assume the user can copy these normally.


deps: update eslint to version 3.9.1

https://greenkeeper.io/

Current coverage is 88.21% (diff: 100%)

Merging #501 into master will decrease coverage by 0.05%

@@             master       #501   diff @@
==========================================
  Files           170        170
  Lines          1825       1825
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           1611       1610     -1
- Misses          214        215     +1
  Partials          0          0

Powered by Codecov. Last update e12eb40...b966e0d


fix(Composer): Safe bounding box and 3D cursor fix

Overview
Push out for the 3D cursor was causing spinning and movement due to it being executed in useFrame. This is not actually necessary for the cursor due to it's visibility through objects
Helpers, most notably the Camera Helper make the scene too large and trying to focus on a camera as well as the whole scene while active will zoom incorrectly when finding the best viewing angle
Keeps the system cursor visible with 3D cursor as per UX decision for visibility at any scale and distance. The 3D cursor is a nice helper that we can improve upon later. Adding the cursor makes this non-blocking
Legal
This project is available under the Apache 2.0 License.

Approving as risk is minimal. Will revisit after upcoming deadline.


[3.x] Add logic to upload the Godot Android library to MavenCentral

Address step 3 in https://github.com/godotengine/godot-proposals/issues/4230
main version

@akien-mga This should also be included in the 3.5 branch. Should I make a separate PR for it?

No need, both branches are still in sync currently.

@akien-mga Per our discussion, I've updated the MavenCentral logic to separate the upload and release steps.

I updated this PR with necessary changes to make it work, after doing some testing on the build server:

We need to source config.sh from build-android/build.sh to have the relevant values, and make the keystore visible to the build container. I opting for copying them to a dedicated folder that's exposed to the build container (so it doesn't tamper with the original files). See changes in the root build.sh.
I've been struggling with some signing steps due to a special character that was being expanded in a double quotes " enclosed string. I changed the format to single quotes ' to prevent this.
Added the .jks keystore to .gitignore.

Updated further, as to publish to MavenCentral in build-release.sh, we need to use podman to have access to the Android SDK, so we move the config of the podman binary and images version to config.sh, where they belong anyway.

I made a further change to fully separate the MavenCentral logic to its own script, so we no longer upload things from build-android/build.sh. I wasn't too happy with uploading content automatically to an official repository any time I do a test build, I prefer that this stays a manual step once things are ready for release.
So now we copy the godot source folder with compiled objects outside the container, so we can reuse it from build-android/upload-mavencentral.sh (which no longer needs passing a version string as it's not used anymore).

Thanks!


Forgot Password form fails without clear message if two accounts have the same email address

If two accounts have the same email address, the form gives an unclear message.
Ideally, no two accounts should have the same email address, but this message makes the issue harder to debug.

@scottbommarito , if two accounts can have the same e-mail address we have a serious problem. This seems like a theoretical/local debugging scenario. Reopen if you disagree.


Use meaningful processTitle for better monitoring

At the moment when we run several gulp tasks separately, they have the same process title gulp which is very hard to differentiate and monitor them. This PR is to solve that issue.

The CLI embedded in 3.x is frozen.  Can you submit this at https://github.com/gulpjs/gulp-cli (which supports the 3.x and 4 version lines)?