Drop libapparmor dependency from runc build docs

As of opencontainers/runc@db093f621f4e6d40feafbe6d7b89a9bee6f6ff85 runc
no longer depends on libapparmor thus libapparmor-dev no longer needs to
be installed to build it. Adjust the documentation accordingly.
Signed-off-by: Tobias Klauser<EMAIL_ADDRESS>

Unlikely that this has anything to do with your PR, but looks like either a flaky test or something that hasn't been seen before: the PodSandbox "stop" conformance test failed:
Mar 28 08:59:20.245: INFO: Unexpected error occurred: rpc error: code = Unknown desc = failed to stop sandbox container "ce0d26f47997a88880813e60c4ae1a514b853ac5b234b996ad54fddd40e73cbb": failed to delete sandbox container: container must be created: failed precondition

I would assume having a container is a precondition of stopping it, so strange that it seems to think the container isn't there. @Random-Liu ?

Does this need to be removed from .travis.yml, as well?

@stevvooe @estesp This shouldn't happen. Let me look into it a little bit.

Does this need to be removed from .travis.yml, as well?

Yes, forgot about that one. Will also remove it there.

Codecov Report

Merging #2238 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2238   +/-   ##
=======================================
  Coverage   41.14%   41.14%
=======================================
  Files          66       66
  Lines        7755     7755
=======================================
  Hits         3191     3191
  Misses       4060     4060
  Partials      504      504

Flag
Coverage Δ

#windows
41.14% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

One question before LGTM: what happens when apparmor is used on a system without apparmor?

LGTM

@stevvooe runc writes directly to proc now and does not bind to the C libs anymore


Fix 'Add user' button label escaping

Prevent wrong button labelling due to quotes in translated label
Fixes issue #4455

Oh... You're absolutely right. Much better idea. Thanks.

Thanks! Your PR is now merged with 9a21060


Do not group routes by host, locale

Your code assumes that all routes have a locale and branding but some routes may not have one of these attributes and, in that case, grouping routes by them breaks the natural route order. Here is an example from our code:
Route 1:  /book/{name}  {branding:siteA, locale: en}
Route 2: /libro/{name} {branding:siteB, locale: es}
Route 3: /{category}/{name}
The last route has to be the last route defined because it should only match any routes which have not been matched by prior routes. But if we group routes by branding/locale, then Route 3 is ordered before the other two and it means it is matched before them.
I did not really see any reason for grouping routes by host/locale. Your bundle works ok without this. So I propose to eliminate this grouping which can only cause problems

It was for performance reason, to avoid hitting many tests. I would prefer it to be an option, activated by default.


[Typo/Mistranslation] 05-create-dax-calculations-in-power-bi-desktop-advanced

On Practice 1, Task 3, step 1 and 2, column names need to stay in English to prevent confusion.
Current:
판매
지역
영업 전체 지역
To be:
Sales
Region
Sales All Region

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Locfile currently InLoc - need it to be at Handedback so I can make any updates


Add GCT µarray creation method

Python infrastructure code for GCT file support:

Allow creation of GCT µarray files from Python code.

Used for normalization tests.

You need update changelog and bump version to '0.12.0a1'  and also bump version in ClientVersionApplication (if you dont have access to it, leave comment, that it should be done, and person who will merge this PR will do it)

To whoever merges this: please update ClientVersionApplication to point to 0.12.0a1.


How can I use my own datasets

Thank you and your team for the excellent work. How can I use my own datasets?

Sorry for the late response. I will update it! Please wait until then.


fix: Button icon part color

This PR fixes the icon color inside of Buttons.

This PR touched a few concepts in fixing the color.  I have updated it to only fix the issue with passing the correct Icon variable.
Broader changes concerning our props vs variables API can be hashed out and handled separately.

All tests now pass.  @kuzhelov, merging since I've reverted the changes you've objected to.


How to represent the relative percentage?

I'm trying to visualize using chart.js the relative percentage of some dynamic data I'm getting from an array.
Starting from this array

[
  {
    "ratiototale": [
      [
        {
          "0": "0.4-0.5",
          "1": 308
        },
        {
          "0": "0.8-0.9",
          "1": 1
        }
      ]
    ]
  }
]

I made this function to populate my chart:
 var getData = function () {
                $.ajax({
                    url: 'http://localhost:3000/ratio/grouped',
                    success: function (data) {
                        console.log(data[0].ratiototale);
                        var somma = 0;
                        for (var key in data[0].ratiototale[0]) {
                            myChart.data.labels.push(data[0].ratiototale[0][key][0]);
                            myChart.data.datasets[0].data.push(data[0].ratiototale[0][key][1]);

                        }

                        // re-render the chart
                        myChart.update();
                    }
                });
            };
            // get new data every 3 seconds
            getData();

Then in the option section of chartjs I made this in order to have the percentage representation:
      options: {
        scales: {

            yAxes: [{
            ticks: {

                   min: 0,
                   callback: function(value){

                       return value + "%"}
                },
								scaleLabel: {
                   display: true,
                   labelString: "Percentage"
                }
            }]
        }
    }

Since I'm  using value in the callbackfunction (I think) I'm having this result:

Instead of the percentage scale from 0 to 100% it's showed the number  of the entities represented in the chart with the percentage scale.
Can I fix it at label level defining a percentage scale? or I have to fix with the percentage calculation somewhere?

I'm doing something similar and we did our percentage calculations separately.

Please ask questions on StackOverflow: https://stackoverflow.com/questions/tagged/chart.js


This page contains an obsolete Github action build

This page contains an obsolete Github action build. The build for .NET latest on linux will need the -o parameter for dotnet publish not the -publish:

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

ID: 967b3514-23c5-514d-186f-d901ac94069b
Version Independent ID: 7a8581b2-feb6-534e-c88d-a0d7c7de7b92
Content: Configure CI/CD with GitHub Actions - Azure App Service
Content Source: articles/app-service/deploy-github-actions.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Thanks for the feedback @edgiardina. We'll update the doc appropriately.

@edgiardina, I'll proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


ERROR Memory size is not numeric: 'No'

Running the latest script with -nr flags. I get the following output error

$ sudo dmidecode -t memory | grep "[Ss]ize" 	Size: 8192 MB 	Size: No Module Installed 	Size: 16384 MB 	Size: No Module Installed
Huge thanks for this script btw, it's an essential!

Thank you. I also noticed the script didn't show the current max memory setting.
$ sudo dmidecode -t memory | grep "[Ss]ize"
 Size: 8192 MB
 Size: No Module Installed
 Size: 16384 MB
 Size: No Module Installed

Since DSM 7.2 was released, different Synology models' output different information for dmidecode -t memory. I don't know why, and it's not the actual dmidecode binary that's different. I just ran the DS1621xs+ dmidecode binary on a DS720+ with an empty memory slot and it didn't return "No Module Installed" for the empty slot.
# /volume1/scripts/dmidecode_1621xs -t memory | grep "[Ss]ize"
 Size: 2048 MB

I've disabled the warning for non-numeric values - as the script skips them anyway, and will show an error if the total memory size is non-numeric.
I've fixed it in https://github.com/007revad/Synology_HDD_db/releases/tag/v3.0.56 or you can run v3.0.55 again and let it update itself to 3.0.56

Many thanks!
BTW running the script doesn't auto-update. Do I need to specify a specific flag, as I can't see an option...?

Thanks so much!


210524 performance enhancements

Actually no performance enhancements were made.
Cleaned up code.
Added a test for multiple cores

Not ready


Step 2 - Set up your IDE

Step 2: Setup your environment
We will use the CodeQL extension for Visual Studio Code. You will take advantage of IDE features like auto-complete, contextual help and jump-to-definition.
Don't worry, you'll do this setup only once, and you'll be able to use it for future CodeQL development.
Follow the instructions below.

:keyboard: Activity: Set up

This course is using GitHub actions. Please make sure that GitHub actions is enabled on this repository: Enable actions
Install the Visual Studio Code IDE.
Go to the CodeQL starter workspace repository, and follow the instructions in that repository's README. When you are done, you should have the CodeQL extension installed and the vscode-codeql-starter workspace open in Visual Studio Code.
Download and unzip this U-Boot CodeQL database, which corresponds to revision d0d07ba.
Import the database into Visual Studio Code (see documentation). This is the database that we'll be running queries on for the duration of this course.
Clone this course repository on your local machine.
Add the course repository to your Visual Studio Code starter workspace, by navigating to File -> Add Folder to Workspace.... If you open this folder you should see several files in there, including qlpack.yml and README.md.

When you're done setting up, please close this issue, then wait for the next set of instructions to appear in a comment below.


error while interpolating services.mattermost.read_only

Hello there,
I get the following error message, when I try to run sudo docker-compose -f docker-compose.yml -f docker-compose.without-nginx.yml up -d:

WARN[0000] The "POSTGRES_IMAGE_TAG" variable is not set. Defaulting to a blank string.
WARN[0000] The "RESTART_POLICY" variable is not set. Defaulting to a blank string.
WARN[0000] The "POSTGRES_DATA_PATH" variable is not set. Defaulting to a blank string.
WARN[0000] The "RESTART_POLICY" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_IMAGE" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_IMAGE_TAG" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_CONFIG_PATH" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_DATA_PATH" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_LOGS_PATH" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_PLUGINS_PATH" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_CLIENT_PLUGINS_PATH" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_BLEVE_INDEXES_PATH" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_CONTAINER_READONLY" variable is not set. Defaulting to a blank string.
error while interpolating services.mattermost.read_only: failed to cast to expected type: invalid boolean:

My .env-file contains the following code:
# Domain of service
DOMAIN=my.sub.domain.de

# Container settings
## Timezone inside the containers. The value needs to be in the form 'Europe/Berlin'.
## A list of these tz database names can be looked up at Wikipedia
## https://en.wikipedia.org/wiki/List_of_tz_database_time_zones

TZ=Europe/Berlin

RESTART_POLICY=unless-stopped

# Postgres settings
## Documentation for this image and available settings can be found on hub.docker.com
## https://hub.docker.com/_/postgres
## Please keep in mind this will create a superuser and it's recommended to use a less privileged
## user to connect to the database.
## A guide on how to change the database user to a nonsuperuser can be found in docs/creation-of-nonsuperuser.md

POSTGRES_IMAGE_TAG=13-alpine
POSTGRES_DATA_PATH=./volumes/db/var/lib/postgresql/data
POSTGRES_USER=username
POSTGRES_PASSWORD=password
POSTGRES_DB=mattermost

## Exposed ports to the host. Inside the container 80 and 443 will be used
HTTPS_PORT=443
HTTP_PORT=80

# Mattermost settings
## Inside the container the uid and gid is 2000. The folder owner can be set with
## `sudo chown -R 2000:2000 ./volumes/app/mattermost`.

MATTERMOST_CONFIG_PATH=./volumes/app/mattermost/config
MATTERMOST_DATA_PATH=./volumes/app/mattermost/data
MATTERMOST_LOGS_PATH=./volumes/app/mattermost/logs
MATTERMOST_PLUGINS_PATH=./volumes/app/mattermost/plugins
MATTERMOST_CLIENT_PLUGINS_PATH=./volumes/app/mattermost/client/plugins
MATTERMOST_BLEVE_INDEXES_PATH=./volumes/app/mattermost/bleve-indexes

## Bleve index (inside the container)
MM_BLEVESETTINGS_INDEXDIR=/mattermost/bleve-indexes

## This will be 'mattermost-enterprise-edition' or 'mattermost-team-edition' based on the version of Mattermost you're installing.
MATTERMOST_IMAGE=mattermost-team-edition
MATTERMOST_IMAGE_TAG=7.1

## Make Mattermost container readonly. This interferes with the regeneration of root.html inside the container. Only use
## it if you know what you're doing.
## See https://github.com/mattermost/docker/issues/18
MATTERMOST_CONTAINER_READONLY=false

## The app port is only relevant for using Mattermost without the nginx container as reverse proxy. This is not meant
## to be used with the internal HTTP server exposed but rather in case one wants to host several services on one host
## or for using it behind another existing reverse proxy.
APP_PORT=8065

## Configuration settings for Mattermost. Documentation on the variables and the settings itself can be found at
## https://docs.mattermost.com/administration/config-settings.html
## Keep in mind that variables set here will take precedence over the same setting in config.json. This includes
## the system console as well and settings set with env variables will be greyed out.
## Below one can find necessary settings to spin up the Mattermost container
MM_SQLSETTINGS_DRIVERNAME=postgres
MM_SQLSETTINGS_DATASOURCE=postgres://${POSTGRES_USER}:${POSTGRES_PASSWORD}@postgres:5432/${POSTGRES_DB}?sslmode=disable&connect_timeout=10
## Example settings (any additional setting added here also needs to be introduced in the docker-compose.yml)
MM_SERVICESETTINGS_SITEURL=https://${DOMAIN}

My docker-compose.yml looks like this:
version: "3.8"

services:
  postgres:
    image: postgres:${POSTGRES_IMAGE_TAG}
    restart: ${RESTART_POLICY}
    security_opt:
      - no-new-privileges:true
    pids_limit: 100
    read_only: true
    tmpfs:
      - /tmp
      - /var/run/postgresql
    volumes:
      - ${POSTGRES_DATA_PATH}:/var/lib/postgresql/data
    environment:
      # timezone inside container
      - TZ

      # necessary Postgres options/variables
      - POSTGRES_USER
      - POSTGRES_PASSWORD
      - POSTGRES_DB
    labels:
      - traefik.enable="false"
    networks:
      - default

  mattermost:
    depends_on:
      - postgres
    image: mattermost/${MATTERMOST_IMAGE}:${MATTERMOST_IMAGE_TAG}
    restart: ${RESTART_POLICY}
    security_opt:
      - no-new-privileges:true
    pids_limit: 200
    read_only: ${MATTERMOST_CONTAINER_READONLY}
    tmpfs:
      - /tmp
    volumes:
      - ${MATTERMOST_CONFIG_PATH}:/mattermost/config:rw
      - ${MATTERMOST_DATA_PATH}:/mattermost/data:rw
      - ${MATTERMOST_LOGS_PATH}:/mattermost/logs:rw
      - ${MATTERMOST_PLUGINS_PATH}:/mattermost/plugins:rw
      - ${MATTERMOST_CLIENT_PLUGINS_PATH}:/mattermost/client/plugins:rw
      - ${MATTERMOST_BLEVE_INDEXES_PATH}:/mattermost/bleve-indexes:rw
    environment:
      # timezone inside container
      - TZ

      # necessary Mattermost options/variables (see env.example)
      - MM_SQLSETTINGS_DRIVERNAME
      - MM_SQLSETTINGS_DATASOURCE

      # necessary for bleve
      - MM_BLEVESETTINGS_INDEXDIR

      # additional settings
      - MM_SERVICESETTINGS_SITEURL
    labels:
      traefik.enable: "true"
      traefik.http.routers.mattermost.entrypoints: "http"
      traefik.http.routers.mattermost.rule: "Host(`my.sub.domain.de`)"
      traefik.http.middlewares.mattermost-https-redirect.redirectscheme.scheme: "https"
      traefik.http.routers.mattermost.middlewares: "mattermost-https-redirect"
      traefik.http.routers.mattermost-secure.entrypoints: "https"
      traefik.http.routers.mattermost-secure.rule: "Host(`my.sub.domain.de`)"
      traefik.http.routers.mattermost-secure.tls: "true"
      traefik.http.routers.mattermost-secure.tls.certresolver: "http"
      traefik.http.routers.mattermost-secure.service: "mattermost"
      traefik.http.services.mattermost.loadbalancer.server.port: "80"
      traefik.docker.network: "proxy"
    networks:
      - default
      - proxy

Weirdly enough, when I run the docker-compose.yml twice, I get a slightly different error message:

WARN[0000] The "RESTART_POLICY" variable is not set. Defaulting to a blank string.
WARN[0000] The "POSTGRES_DATA_PATH" variable is not set. Defaulting to a blank string.
WARN[0000] The "POSTGRES_IMAGE_TAG" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_IMAGE" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_IMAGE_TAG" variable is not set. Defaulting to a blank string.
WARN[0000] The "MATTERMOST_CONTAINER_READONLY" variable is not set. Defaulting to a blank string.
error while interpolating services.mattermost.read_only: failed to cast to expected type: invalid boolean:

In another forum, a user suggested using Docker Compose V2, as they had a similar problem, which was then fixed by using Docker Compose V2. Anybody has some 5 cents regarding this?

Do you still face this issue? I randomly saw your post while dealing with MM.
I noticed you have two -f, why? I'd also recommend moving to v2. While LTS is preferable in all things, there's no point making your life harder than it needs to be. The only major breaking change is the lack of the -, so scripts have to be fixed.


TypeError: loss() got an unexpected keyword argument 'gt_bboxes_ignore'

while i run the command
./tools/dist_train.sh configs/ssd300_scratchdet_coco.py 2 --validate,
i got an error:
Traceback (most recent call last):
File "./tools/train.py", line 108, in
main()
File "./tools/train.py", line 104, in main
logger=logger)
File "/home/jshuzi/mmdetection/mmdet/apis/train.py", line 58, in train_detector
_dist_train(model, dataset, cfg, validate=validate)
File "/home/jshuzi/mmdetection/mmdet/apis/train.py", line 186, in _dist_train
runner.run(data_loaders, cfg.workflow, cfg.total_epochs)
File "/home/jshuzi/mmcv/mmcv/runner/runner.py", line 358, in run
epoch_runner(data_loaders[i], **kwargs)
File "/home/jshuzi/mmcv/mmcv/runner/runner.py", line 264, in train
self.model, data_batch, train_mode=True, **kwargs)
File "/home/jshuzi/mmdetection/mmdet/apis/train.py", line 38, in batch_processor
losses = model(**data)
File "/home/jshuzi/.local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/home/jshuzi/mmcv/mmcv/parallel/distributed.py", line 50, in forward
return self.module(*inputs[0], **kwargs[0])
File "/home/jshuzi/.local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/home/jshuzi/mmdetection/mmdet/core/fp16/decorators.py", line 49, in new_func
return old_func(*args, **kwargs)
File "/home/jshuzi/mmdetection/mmdet/models/detectors/base.py", line 86, in forward
return self.forward_train(img, img_meta, **kwargs)
File "/home/jshuzi/mmdetection/mmdet/models/detectors/single_stage.py", line 60, in forward_train
*loss_inputs, gt_bboxes_ignore=gt_bboxes_ignore)
TypeError: loss() got an unexpected keyword argument 'gt_bboxes_ignore'
Traceback (most recent call last):
File "./tools/train.py", line 108, in
main()
File "./tools/train.py", line 104, in main
logger=logger)
File "/home/jshuzi/mmdetection/mmdet/apis/train.py", line 58, in train_detector
_dist_train(model, dataset, cfg, validate=validate)
File "/home/jshuzi/mmdetection/mmdet/apis/train.py", line 186, in _dist_train
runner.run(data_loaders, cfg.workflow, cfg.total_epochs)
File "/home/jshuzi/mmcv/mmcv/runner/runner.py", line 358, in run
epoch_runner(data_loaders[i], **kwargs)
File "/home/jshuzi/mmcv/mmcv/runner/runner.py", line 264, in train
self.model, data_batch, train_mode=True, **kwargs)
File "/home/jshuzi/mmdetection/mmdet/apis/train.py", line 38, in batch_processor
losses = model(**data)
File "/home/jshuzi/.local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/home/jshuzi/mmcv/mmcv/parallel/distributed.py", line 50, in forward
return self.module(*inputs[0], **kwargs[0])
File "/home/jshuzi/.local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/home/jshuzi/mmdetection/mmdet/core/fp16/decorators.py", line 49, in new_func
return old_func(*args, **kwargs)
File "/home/jshuzi/mmdetection/mmdet/models/detectors/base.py", line 86, in forward
return self.forward_train(img, img_meta, **kwargs)
File "/home/jshuzi/mmdetection/mmdet/models/detectors/single_stage.py", line 60, in forward_train
*loss_inputs, gt_bboxes_ignore=gt_bboxes_ignore)
TypeError: loss() got an unexpected keyword argument 'gt_bboxes_ignore'
Traceback (most recent call last):
File "/home/jshuzi/anaconda3/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/jshuzi/anaconda3/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/jshuzi/.local/lib/python3.6/site-packages/torch/distributed/launch.py", line 235, in
main()
File "/home/jshuzi/.local/lib/python3.6/site-packages/torch/distributed/launch.py", line 231, in main
cmd=process.args)
subprocess.CalledProcessError: Command '['/home/jshuzi/anaconda3/bin/python', '-u', './tools/train.py', '--local_rank=0', 'configs/ssd300_scratchdet_coco.py', '--launcher', 'pytorch', '--validate']' returned non-zero exit status 1.
Have anyone met the problem? thank you for reply.

Hi, did you find some solutions?


Define 'SpecifLanguageText'

The items of 'SpecifMultiLanguageText' have not a type of its own. Change the schema so that 'SpecifMultiLanguageText' is a list of 'SpecifLanguageText'.

Has been proposed for SpecIF Schema v1.2. Any data-set according SpecIF Schema v1.1 is compatible.

Done with schema v1.2


chore: update wake and switch to nixpkgs 23.11

I did:

update wake to the latest version
switch to the latest stable nixpkgs version (poetry2nix doesn't work with 23.05 https://github.com/nix-community/poetry2nix/issues/1422)

I also need to instantiate poetry2nix.mkPoetryApplication, but I can't. I hope it close #421

But like that it works fine: https://github.com/nix-community/poetry2nix/blob/master/templates/app/flake.nix#L18

@selfuryon I have a competing unfinished commit to update to nixpkgs-23.11. Here is what I found:

lighthouse doesn't build on 23.11 because Rust is too new. That's because of lighthouse depending on an old version of libmdbx-rs in its slasher Cargo.tomb. I "solved" this by introducing a nixpkgs_2305 input and just having lighthouse stay behind. * ssvnode: My fix is exactly like yours.
I haven't fixed Nimbus yet in my commit.
Just FYI.

@selfuryon I have a competing unfinished commit to update to nixpkgs-23.11. Here is what I found:
* lighthouse doesn't build on 23.11 because Rust is too new. Lighthouse depends on an old version of libmdbx-rs in its slasher Cargo.tomb, and that old libmdbx-rs in turn points to an old bindgen library. The old bindgen dependency chokes on an LLVM dependency update like [this](https://github.com/rust-lang/rust-bindgen/issues/2312). bindgen would need to be updated inside sipg's fork of libmdbx-rs. But I am not going to touch that to be frank. I "solved" this by introducing a nixpkgs_2305 input and just having lighthouse stay behind.

* ssvnode: My fix is exactly like yours. Just my override is inside packages/default.nix.

* I haven't fixed Nimbus yet in my commit.
  Just FYI.

For the first point seems related this one: https://github.com/sigp/lighthouse/issues/4280. Maybe we can just disable slasher feature for now to mitigate that. I will try

Good, switch to slasher-lmdb (it's enabled by default) helps

The only problem that poetry2nix.mkPoetryApplication still is the derivation, idk how to fix that

The only problem that poetry2nix.mkPoetryApplication still is the derivation, idk how to fix that

My approach was to also add the poetry2nix overlay to pkgs
          pkgs = lib.extras.nix.mkNixpkgs {
            inherit system;
            inherit (inputs) nixpkgs;
+            overlays = [inputs.poetry2nix.overlays.default];
          };

This made my nix flake check pass.

The only problem that poetry2nix.mkPoetryApplication still is the derivation, idk how to fix that

My approach was to also add the poetry2nix overlay to pkgs
          pkgs = lib.extras.nix.mkNixpkgs {
            inherit system;
            inherit (inputs) nixpkgs;
+            overlays = [inputs.poetry2nix.overlays.default];
          };

This made my nix flake check pass.

Is nix flake show also works fine without --allow-import-from-derivation?

Is nix flake show also works fine without --allow-import-from-derivation?

No, but nix flake show doesn't work either on the current main branch.

Is nix flake show also works fine without --allow-import-from-derivation?

No, but nix flake show doesn't work either on the current main branch.

Yeah, but with flake-utils with the same style it works, I just wanted to fix that, but maybe it would be easy to fix a script for nix-update.
@brianmcgee @aldoborrero can you pls check this PR and approve it?


Optional requiremnt for Mandatory Access Control

Should we include one?
FDP_IFC_EXT.1.1: The OS shall have a mechanism to prohibit a information channel between two unprivileged processes without the assistance of the kernel or other processes.
Basically I'm trying to say that pick two processes, any processes. Can they be configured such that one process cannot read any object that the 2nd process can write to? The CDS team thought this would be helpful. Also MRAT is requesting a similar thing from MDF PP.

MLS or other fancy access controls should be in an EP.


Firmware cleanup

Some code deleting I did a couple of months ago.

removes code for loading pre-pic games (untested for ages as nothing that old would still run)
removes remaining parts of the "firmware in ext flash" code
replaces/cleans up some HAL GPIO code
tidies up stm32 screen mode handling a little
removes HAL layers between fatfs and the SD code

Wow! RIP obsolete exe switching!
Nice work. It's refreshing to see more and more HAL stuff just disappear :laughing:

More small cleanups/space savings. (Ran out of space and slightly forgot about these changes... so found other savings)

Ah does this now fix the ~100 bytes debug build overflow?

Awesome! Thank you.


dialog_field_refresh.js#initializeDialogSelectPicker - fix 2 ignored and broken specs

Because a stray done() was called before any expect calls, the test would "succeed" but not actually test anything. With a warning in browser console..
Spec 'dialogFieldRefresh #initializeDialogSelectPicker triggers the auto refresh when the drop down changes' has no expectations.
Spec 'dialogFieldRefresh #initializeDialogSelectPicker triggers autorefresh with "false" when triggerAutoRefresh arg is false' has no expectations.

Fixing that made both specs fail - the failing test mocks miqSelectPickerEvent in beforeEach, but calls a function that passes that triggerAutoRefresh as a callback to miqSelectPickerEvent .. and expects triggerAutoRefresh to be called.
dialogFieldRefresh #initializeDialogSelectPicker triggers the auto refresh when the drop down changes
Expected spy triggerAutoRefresh to have been called with [ 'fieldId', 'true' ] but it was never called.

dialogFieldRefresh #initializeDialogSelectPicker triggers autorefresh with "false" when triggerAutoRefresh arg is false
Expected spy triggerAutoRefresh to have been called with [ 'fieldId', 'false' ] but it was never called.

Making the mock actually call the callback fixes the tests :).
The failure is the same as in https://github.com/ManageIQ/manageiq/pull/8514#issuecomment-217492071, and indeed this PR fixes it. => Cc: @petrblaho

@petrblaho : can you confirm?

@martinpovolny solves the issue with tests for me
@himdel thank you

@martinpovolny darga?

@petrblaho : marking as garga/no, right?

@chessbyte @martinpovolny I am not sure but I am building a solution for https://bugzilla.redhat.com/show_bug.cgi?id=1332109 on top of this...


Integrate with VSCode' Polyglot Notebooks

Following up on @evertoncolling suggestion in #90 , make such that GoNB work in Polyglot Notebook.

I created a feature request with Polyglot to add support for Go (and GoNB in particular) https://github.com/dotnet/interactive/issues/3511


file-size: Adapt it to be more like file-age, plus using ranges

In GitLab by @markuslf on Oct 11, 2021, 12:20

See PR #735


feat: new ui for tipping options on publisher card

Describe the changes you have made in this PR
Users seem to confuse the "send sats" button on the publisher card, also we want to make tipping more prominent in the app if there is tipping information available for the current site.
Open tasks

[ ] Decide about icons / emojis and their respective values
[ ] Implement proper buttons (currently they are a hack)
[ ] Take users to the tipping flow with the amount already filled in
[ ] Add a custom amount option
[ ] Think about re-using them below the amount field (quick selection of amounts)

Screenshots of the changes [optional]

How has this been tested?

Checklist

[ ] My code follows the style guidelines of this project and performed a self-review of my own code
[ ] New and existing tests pass locally with my changes
[ ] I checked if I need to make corresponding changes to the documentation (and made those changes if needed)

great try
This is the original view

the view  click after send sats

After clicking coffee(1k)pizza(5k) in the new view, does the page view change?
For the current interaction, see the text introduction.
In the wallet, there is only the website introduction, not the text introduction or description related to tips.
It would be better to add some introductions and links related to the tip module

New users should see that the color of the wallet icon has changed, so they will click to open the wallet to take a look. If there is no introduction to the tip module here, they probably don’t know what the relationship between the tip module and the current page is.


proxy swallows error message when container fails to start

$ docker run --rm -it gliderlabs/alpine
Error response from daemon: No command specified
$ weave launch
$ eval $(weave env)
$ docker run --rm -it gliderlabs/alpine
Error response from daemon: ba8ea356fa43646ec9e758af06d4443695483f27152c6ff40ca192ce188a5c01 container is not running.
$ docker logs weaveproxy
...
WARN: 2015/07/30 09:54:09.488245 Attaching container ba8ea356fa43646ec9e758af06d4443695483f27152c6ff40ca192ce188a5c01 to weave network failed: ba8ea356fa43646ec9e758af06d4443695483f27152c6ff40ca192ce188a5c01 container is not running.

WARN: 2015/07/30 09:54:09.488313 Error intercepting response: ba8ea356fa43646ec9e758af06d4443695483f27152c6ff40ca192ce188a5c01 container is not running.

NB: I haven't tested this on 1.0, but I suspect the problem is present there too.

NB: I haven't tested this on 1.0, but I suspect the problem is present there too.

Indeed it is.

Fixed in #1238

The title of this bug was fixed in #1238.
The example provided was a different issue, and was fixed in #1253.


Avoid duplicate index.html files

Implemented changes from http://stackoverflow.com/questions/37157548/generate-html-files-in-sbt and http://stackoverflow.com/questions/37409540/can-sequences-be-used-in-build-sbt-to-define-multiple-similar-settings to avoid two similar index.html files.
Note: one needs to run the html from target folder now, not the source one.

I like it :+1:


Question about the transact mechanism

Hi @AugustNagro,
Thanks for this great lib 🙂
I was reading the code of the transact function and was surprised you don't restore setAutoCommit value of the connection at the end of the transaction:
Current code is:
def transact[T](dataSource: DataSource)(f: DbTx ?=> T): T =
  Using.resource(dataSource.getConnection)(con =>
    con.setAutoCommit(false)
    try
      val res = f(using DbTx(con))
      con.commit()
      res
    catch
      case t =>
        con.rollback()
        throw t
  )

Restoring the value would give something like:
def transact[T](dataSource: DataSource)(f: DbTx ?=> T): T =
  Using.resource(dataSource.getConnection)(con =>
    val previous = con.getAutoCommit
    con.setAutoCommit(false)
    try
      val res = f(using DbTx(con))
      con.commit()
      res
    catch
      case t =>
        con.rollback()
        throw t
   finally
     con.setAutoCommit(previous)
  )

That's what Quill seems to be doing, for example.
See https://github.com/zio/zio-protoquill/blob/806867078dfb0e99ed9a5a43663a55e65b9fbad6/quill-jdbc/src/main/scala/io/getquill/context/jdbc/JdbcContext.scala#L93-L104
Why don't you need to restore the previous value? 🤔

I would think that restoring the setAutoCommit previous value would be superfluous when the resource manager code (Using.resource) will always be releasing the connection at the end of the code block that it encloses. This will close the db connection or, if you're using a pooling datasource, return it to the pool. I don't know, but I would expect, that most/all connection pooling packages (e.g. HikariCP) are always going to reset connections to a default state when they're added or released to their pool, or perhaps just before they're handed-out to a client.
The above would apply to that protoquill code too - I don't know why they're bothering to save+restore the autocommit state on connections that they're allocating in transact(), unless the authors don't trust a pooling datasource to properly manage the state of pooled connections.

Would be interesting to know what HikariCP is doing indeed because if it doesn't reset the connection settings, this might be an issue for Magnum users

There are quite a few Connection settings that could be tweaked in addition to AutoCommit.  HikariCP specifically tracks whether or not some of those settings get changed by its clients, and resets them to default (configurable) values when the client returns the Connections to the pool via close(). You can see some of that code here.
AutoCommit is included in the list of tracked parameters.
One would think that the default settings for pooled connection objects would be defined by the JDBC spec but if so I haven't found it. "By default, new connections are in auto-commit mode." is all I've found. I can only speculate that the Protoquill authors might have had some adverse experiences w/various connection pooling packages and decided to play it safe by ensuring that at minimum, transact() wouldn't have any side-effects on the connection's AutoCommit state.

Shouldn't Magnum play it safe too? It's not like it's a complicated or expensive thing to do 🤔

Thanks for sharing Jules.
I'm happy to accept this change if we can figure out why Quill introduced
it (specifically which connection pool).
On Mon, Aug 19, 2024, 10:48 PM Jules Ivanic @.***>
wrote:

Shouldn't Magnum play it safe too? It's not like it's a complicated or
expensive thing to do 🤔
—
Reply to this email directly, view it on GitHub
https://github.com/AugustNagro/magnum/issues/40#issuecomment-2298016627,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQOKNRP2FHT7Z6PNSUMUKDZSLKBVAVCNFSM6AAAAABMWZ5XVWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJYGAYTMNRSG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

I didn't know it was safe to not re-initialize the Connection when using a connection pool. I guess we can leave it like this then. I'm not sure we'll be able to know why Quill did it this way.


I didn't yet try to sign up into Azure, but the information provided didn't tell anything for those who doesn't have credit cards. In the country where I live, there is no credit card service. That means, I have no credit card to register. And, if the information I will enter about credit card is not optional, what solution will you have for that?

I didn't yet try to sign up into Azure, but the information provided didn't tell anything for those who doesn't have credit cards. In the country where I live, there is no credit card service. That means, I have no credit card to register. And, if the information I will enter about credit card is not optional, what solution will you have for that?

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

ID: 0c90e134-28de-dd70-5bce-c4a829866288
Version Independent ID: 2c62c86f-4bac-9cce-f05d-ebd95cbb3d15
Content: Troubleshoot issues when you sign up for a new account in the Azure portal - Microsoft Cost Management
Content Source: articles/cost-management-billing/manage/troubleshoot-azure-sign-up.md
Service: cost-management-billing
Sub-service: billing
GitHub Login: @bandersmsft
Microsoft Alias: banders

@Tedygit Thank you for contacting us. If you do not have credit card, you can use debit card to sign up for Azure. FAQ link for reference - https://azure.microsoft.com/en-in/free/free-account-faq/ Another option would be to use Azure for students. Refer to the FAQ link - https://learn.microsoft.com/en-us/azure/education-hub/azure-dev-tools-teaching/program-faq#who-is-eligible-for-azure-for-students-

Hope this helps!

@Tedygit Checking in to see if the above answer helped. Let us know if there are still any additional issues we can help with.

@Tedygit We haven't heard back from you, so we will now close this issue. If this remains an issue, please reply and we will gladly continue the discussion.


I cannot search "iclr" "ijcai" in my powertoys run.

Microsoft PowerToys version
0.79.0
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
Search iclr, ijcai, aaai
My first computer can search them, but my second computer cannot search them.
But I can search nips, icml, acs
and also I can search with "iclr 2024" or "aaai 2022"
✔ Expected Behavior
search iclr, ijcai, aaai
find all paper lists with
[name of the paper] - ICLR 202x
[name of the paper] - IJCAI 202X
❌ Actual Behavior
Does not appear anything.
"- iclr" "- nips" retrieves same recent files, but not related to keywords.
Other Software
No response

Can you look in your search indexing settings if the location of these files are indexed?
/needinfo

Can you look in your search indexing settings if the location of these files are indexed?
/needinfo

Yeah, I checked the indexing settings several times. I can search the same file with different keywords, so it's not an indexing problem.
ex) target file: Crystalformer, Infinitely Connected Attention for Periodic Structure Encoding - ICLR 2024.pdf
failed keyword: iclr
success keyword: car 2024
This keyword worked before the update of the Powertoys version... 6.9? -> 7.9
I want to fully uninstall Powertoys and reinstall the program, but I cannot fully uninstall this program! I erased the backup and restore folder (Documents/Powertoys), but the settings remain.

Can you look in your search indexing settings if the location of these files are indexed?
/needinfo

Yeah, I checked the indexing settings several times. I can search the same file with different keywords, so it's not an indexing problem.
ex) target file: Crystalformer, Infinitely Connected Attention for Periodic Structure Encoding - ICLR 2024.pdf failed keyword: iclr success keyword: iclr 2024
This keyword worked before the update of the Powertoys version... 6.9? -> 7.9
I want to fully uninstall Powertoys and reinstall the program, but I cannot fully uninstall this program! I erased the backup and restore folder (Documents/Powertoys), but the settings remain.

Settings are saved under %localappdata%/microsoft/PowerToys not in the backup location.


ts.swap_nodes throws error on latest Neovim master

I run neovim master, and have run into an issue with all of the functions that use ts.swap_nodes. Since https://github.com/neovim/neovim/pull/27614, neovim now requires that the buffer number passed to apply_text_edits (which ts.swap_nodes uses) is non-nil and non-0. Every call to ts.swap_nodes in nvim-paredit that I saw uses 0 to specify "the current buffer". The fix should be as easy as passing vim.api.nvim_get_current_buf() to all of the ts.swap_nodes calls in place of 0.

I implemented such a change locally and the tests pass with 0.9.5 but fail for other reasons with 0.10 (master). I'm not entirely sure what else has changed, but I can open a PR if you want just this fix.

Yea the tests for neovim nightly have been failing here for a while. I haven't had a proper chance to investigate it. Those tests are currently disabled for the CI tests that run here.
Thanks for the fix!

Glad to help.


aatype feature

Dear Madame/Sir,
The aatype feature (Lx21) is computed per amino acid, but it needs to be transformed to an LxLxN tensor to be concatenated with the rest of input features.
For each ij aa pair, do you include the aa type for both i and j ending up with a LxLx42 tensor or are you only using the encoding for either i or j and repeat them along rows/colums ending up with LxLx21?

You are correct, we end up with an L × L × 42 tensor for the aatype in the 2D features, i.e. the location (i, j) has the aatype for both i and j.


Add use_unified_worker to pipeline options.

use_unified_worker is also needed, otherwise the pipeline will use java runner.
r: @aaltay

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Thanks for the quick review!


Font selection should work as an input box, not a suggestion box

What should be added?
The ability to type your font and add it from a list. If you type slow, then it will select a different font for every letter. It should work as an input box, not a suggestion box:

Why should it be added?
The current implementation seems old and outdated

I don't think that this is worth changing. IMO it requires too much effort to recreate the Dialog and get all the functionality including preview, selecting all fonts properly and testing.

I don't think that this is worth changing. IMO it requires too much effort to recreate the Dialog and get all the functionality including preview, selecting all fonts properly and testing.

Can you suggest anything else? Currently I find it a bit of a pain to change my font.

Here is my suggestion:

Accept that it is a subpar experience for yourself. Even if you are used to a certain way that ui works take a moment to analyse the way another system does it. It may be a refreshing experience to realize that there are multiple ways to achieve a goal in ui design! In this case it works similar to a couple of elements in the windows operating system. The windows file explorer for example uses the same system to select items quickly. As such users of the system will already be used to the way a certain dialog operates.
Only change your font so often.  I now realize the level of zen I get  by sticking to the same font without feeling the need to change it. However, if you still feel a strong compulsion to change your font often enough to trigger a negative experience in the dialog experience my advice would be, as mentioned in 1., to try and accept that this is a minor part of the Chatterino experience we are trying to create. The important part is to allow the user to select a font in case he has trouble reading the default one. This should be done without hassle in a familiar environment that the user is already accustomed to. In our case, since qt is such a popular framework, the user may have even used the same dialog in another application! Most other font dialog also look and behave similar.
Selecting a font is only done in a single place - to select the chat font. This also means that if work put in is proportional to the  time a user uses the program this would mean that there shouldn't be a lot of development time spent on this dialog. It already achieves its goal quite well! It shows all the available fonts on the system and let's the use select one of them. This is the exact functionality wanted from this component! The important feature therefore is already provided by the framework with a level of quality that we can rely on. With the amount of development time proportional to user time spent using the feature we can't possibly justify doing our own implementation of this dialog.we can simply not reasonably allocate enough resources to fix this issue. The time would be better spent in other places. Quite upsettingly this issue does not seem to have any concept of what important for the users of this program is. It seems to be a minor annoyance posted to an issue without further thought.
Realistically, what does it take to fix this? It requires a full reimplementation of the font dialog. We can of course not base it off of qts own version because of licensing issues. Adding an external dependency is not worth it compared to the payoff in any capacity. It is simply out of scope to "fix" this. It isn't even broken.


Fix MethodError raised when defining Structs with non-Param arrays

Adds AbstractArray to ignore list.
Closes #62

Hmm, AbstractArray works for my specific use case but it seems StaticArray should not be ignored.
Is using the Array type an acceptable compromise?

Yes Array is better, thats where the problem is.


SDL fipsified - but it needs a git command installed

https://github.com/mgerhardy/fips-sdl
I gave it another try - but sdl2 is using mercurial, not git - therefore you need to install https://github.com/mgerhardy/fips-sdl/blob/master/git-remote-hg in your path. Is there a better way to deal with it? Maybe install dedicated git modules - or put fips-git directories into the path in the git.py?

btw... this might not yet work 100% - it's an ongoing effort and help is much appreciated ;)

I have a couple of ideas, but they require work on the low-level parts of fips...
First: I think it makes sense to add direct support for more version control systems besides git, I want to start using fips also for our Bigpoint C++ projects in the not-too-distant future, which would fetching dependencies from svn repos (another nice side effect would be that we could then start mixing git and svn projects, and move some of our projects over to git).
Second: provide an alternative to git submodules for 'fipsified projects' to include the original project. git submodules have a couple of problems when actually working with these fipsified projects (e.g. working on pull requests for upstream etc...)... I'm currently thinking that it would be useful to allow fetching non-fips projects as dependencies (I think I even already implemented this) so that the fips-wrapper would not "contain" the original project, but instead fetch this as a normal dependency...
Both of these combined (direct support for hg, and not including the original project as git submodule) would solve the problem of SDL not using git...

Ok, then I will stop my work on fips-sdl (again) and wait for direct mercurial support (or provide a pull request once subversion is in and the basic support for more vcs is done)
btw. I still think that looking at MXE might help - we only build stable releases there. Which also has a lot of advantages. At BigPoint we could do the same with the artifactory. Generate a release archive and just download it via http might be better than supporting all the vcs's out there. And it also has the advantage to not break compilation if someone commits something for some reason.


ESLint fixes for unrelated files in pythonEnvironments/discovery

For #13142
Added directives to disable ESLint checks for for ... of + await in loop, these will be revisited once we add async array.find and array.filter utility functions.

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #13857 into main will decrease coverage by 59.93%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main   #13857       +/-   ##
===========================================
- Coverage   59.93%    0.00%   -59.94%
===========================================
  Files         684       14      -670
  Lines       37953       38    -37915
  Branches     5463        3     -5460
===========================================
- Hits        22748        0    -22748
+ Misses      14046       38    -14008
+ Partials     1159        0     -1159

Impacted Files
Coverage Δ

src/client/debugger/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/common/process/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/common/platform/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/datascience/notebook/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/interpreter/autoSelection/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...t/datascience/jupyter/jupyterDataRateLimitError.ts
0.00% <0.00%> (-50.00%)
:arrow_down:

src/client/common/serviceRegistry.ts

src/client/common/installer/productService.ts

...c/client/activation/languageServer/platformData.ts

src/client/testing/nosetest/runner.ts

... and 664 more

Continue to review full report at Codecov.

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


Can't sign in

I get a error when I'm redirected from Steam OAuth page to steamdb:
"There’s an issue with the database.
Code 42000. Please try again later."
Tested on Firefox and Chrome, I will test on different PC tomorrow.

Should be fixed now, thanks!

That was fast, thanks !

I've got:
"This challenge page was accidentally cached and is no longer available.
If you are having issues, make sure that you are using a fully up to date browser.
RAY: 6d8f43125efb82ea"
What does this mean?

It's all good, I just logged in on my old laptop


xen: align ram bank 0 configuration with libxl

Align GUEST_RAM0_SIZE with value in libxl.
Ensure that the first extended region does not overlap with devices memory addresses.
Changes have been synced with the 'xen-4.19-xt0.1' branch on the https://github.com/xen-troops/xen.git repository.

added acks + note to commit message


Fix txt prefix bug, should be lowercased because when writing to dns …

…it is lowercased

/assign @njuettner

Hi @p53, can you write a better explanation fo what is the problem that you are trying to solve?

@Raffo i explained it here https://github.com/kubernetes-incubator/external-dns/issues/1012

Should we add a test for this?

+1 for tests.

I added tests and removed some debug statements. TXT prefix is case-insensitive with this change.
This will break ownership for people who currently rely on case-sensitivity in their TXT prefixes, but I think we can safely ignore that case.

/lgtm

/approve

/lgtm
/approve


Add blocks support to mix

Up to v1.0.0 this code worked as expected:
var b1 = block('b1');
var b2 = block('b2');
b2().mix(b1('element')); // 'b2 b1__element'

#8 broke this functionality:
b2().mix(b1('element')); // 'b2 toStringfunction () { [native code] } splitfunction () { [native code] } mixfunction () { [native code] } statefunction () { [native code] }'

This PR restores it.

Your implementation and examples for README isn't correctly enough.
I'll fix this issue soon. Wait version v1.1.1.

Done.
Please use version v1.1.1.


sc-876 - Feature - Switch Component

Short summary

Reafactored original switch to use bootstrap switch
added documentation
still working on outstanding types that may be missing in this release
Not yet styled to suit the theme, using default bootstrap functionality

I've merged the main branch to this and refactored the component to use the post CSS prefixes plugin so we don't need to handle the classes prefix in BS utility classes.
Putting this on the side while waiting on the class issue fix. Once we get it, in case it doesn't require any refactoring, it is good to go.


TimeoutError: Deadline exceeded on activities taking greating than 120 seconds

Here's the stack trace.
2021-05-13 13:22:37,579 | ERROR | retry.py:retry_loop:29 | run failed: Deadline exceeded, retrying in 3 seconds
Traceback (most recent call last):
  File "/Users/chad.greenburg/.pyenv/versions/3.8.5/lib/python3.8/site-packages/grpclib/client.py", line 360, in recv_initial_metadata
    headers = await self._stream.recv_headers()
  File "/Users/chad.greenburg/.pyenv/versions/3.8.5/lib/python3.8/site-packages/grpclib/protocol.py", line 349, in recv_headers
    await self.headers_received.wait()
  File "/Users/chad.greenburg/.pyenv/versions/3.8.5/lib/python3.8/asyncio/locks.py", line 309, in wait
    await fut
asyncio.exceptions.CancelledError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/chad.greenburg/.pyenv/versions/3.8.5/lib/python3.8/site-packages/temporal/retry.py", line 17, in retry_loop
    await fp(*args, **kwargs)
  File "/Users/chad.greenburg/.pyenv/versions/3.8.5/lib/python3.8/site-packages/temporal/decision_loop.py", line 1083, in run
    decision_task: PollWorkflowTaskQueueResponse = await self.poll()
  File "/Users/chad.greenburg/.pyenv/versions/3.8.5/lib/python3.8/site-packages/temporal/decision_loop.py", line 1165, in poll
    task = await self.service.poll_workflow_task_queue(
  File "/Users/chad.greenburg/.pyenv/versions/3.8.5/lib/python3.8/site-packages/temporal/api/workflowservice/v1.py", line 828, in poll_workflow_task_queue
    return await self._unary_unary(
  File "/Users/chad.greenburg/.pyenv/versions/3.8.5/lib/python3.8/site-packages/betterproto/__init__.py", line 1133, in _unary_unary
    response = await stream.recv_message()
  File "/Users/chad.greenburg/.pyenv/versions/3.8.5/lib/python3.8/site-packages/grpclib/client.py", line 408, in recv_message
    await self.recv_initial_metadata()
  File "/Users/chad.greenburg/.pyenv/versions/3.8.5/lib/python3.8/site-packages/grpclib/client.py", line 380, in recv_initial_metadata
    self.initial_metadata = im
  File "/Users/chad.greenburg/.pyenv/versions/3.8.5/lib/python3.8/site-packages/grpclib/utils.py", line 70, in __exit__
    raise self._error
asyncio.exceptions.TimeoutError: Deadline exceeded

Here's what I've uncovered:
The python-sdk starts up two parallel threads:

One to handle workflows (runs workflow logic and determines which activity needs to run)
One to handle activities (runs activity code)

Both threads use the same grpc channel to communicate with the temporal server with a timeout set to 120s. Concurrent RPC calls are supported according to the grpclib docs: https://grpclib.readthedocs.io/en/latest/client.html
The workflow thread polls the workflow task queue and the activity thread polls the activity task queue. Both take 60 seconds before continuing the while loop to poll again if nothing is returned. When the activity thread receives something on the activity task queue, it starts running the activity code. Meanwhile, the workflow thread is in the middle of polling the workflow task queue.
What I'm noticing is that the workflow poll request is "blocked" and doesn't return like it usually would after 60 seconds. The workflow poll request doesn't complete until the activity in the other thread is finished. If an activity takes long enough to complete, the workflow poll request can take more than 120 seconds (note the timeout mentioned earlier) causing a deadline exceeded error.
Solution:

Figure out why the workflow poll request is "stuck" and can't complete while an activity is running in the other thread.

A couple of temporary workaround:

Set the timeout to the max time that we expect activities could take. I don't know if there are consequences to this.

OR

Not worry about the deadline exceeded error because when it happens, it immediately continues the while loop and polls the workflow task queue again.

The downside to this is having this error clutter the worker logs

Hi @CGreenburg, I'm not able to reproduce this with the code below. Could you share some sample code that produces that error on your side.
import asyncio
import logging
from datetime import timedelta

from temporal.activity_method import activity_method
from temporal.workerfactory import WorkerFactory
from temporal.workflow import workflow_method, Workflow, WorkflowClient

logging.basicConfig(level=logging.INFO)

TASK_QUEUE = "HelloWorld"
NAMESPACE = "default"

# Activities Interface
class GreetingActivities:
    @activity_method(task_queue=TASK_QUEUE, schedule_to_close_timeout=timedelta(seconds=500))
    async def compose_greeting(self, greeting: str, name: str) -> str:
        raise NotImplementedError

# Activities Implementation
class GreetingActivitiesImpl:
    async def compose_greeting(self, greeting: str, name: str) -> str:
        await asyncio.sleep(200)
        return greeting + " " + name

# Workflow Interface
class GreetingWorkflow:
    @workflow_method(task_queue=TASK_QUEUE)
    async def get_greeting(self, name: str) -> str:
        raise NotImplementedError

# Workflow Implementation
class GreetingWorkflowImpl(GreetingWorkflow):

    def __init__(self):
        self.greeting_activities: GreetingActivities = Workflow.new_activity_stub(GreetingActivities)

    async def get_greeting(self, name):
        return await self.greeting_activities.compose_greeting("Hello!", name)

async def worker_main():
    client = WorkflowClient.new_client(namespace=NAMESPACE)
    factory = WorkerFactory(client, NAMESPACE)
    worker = factory.new_worker(TASK_QUEUE)
    worker.register_activities_implementation(GreetingActivitiesImpl(), "GreetingActivities")
    worker.register_workflow_implementation_type(GreetingWorkflowImpl)
    factory.start()
    print("Worker started")

if __name__ == '__main__':
    loop = asyncio.get_event_loop()
    asyncio.ensure_future(worker_main())
    loop.run_forever()

We migrated a pyspark/pandas application to a workflow activity and it takes longer than 2 minutes. That's where we've seen the deadline exceeded exception. I've been replicating it using time.sleep(150).

You might want to look into https://docs.python.org/3/library/asyncio-eventloop.html#asyncio.loop.run_in_executor if your activity is CPU bound or if the APIs that you're using aren't async friendly.

There was a thread about this in the past where I proposed creating an event loop per worker (running in its own thread) so that the activity code is free to block as it pleases but they feedback I got perviously was that the library should do the bare minimum and allow the user to decide how to handle blocking calls.
https://community.temporal.io/t/timeline-for-python-client-support/223/26

If you're able to, I would run my workflow and activities in separate workers.

You can also try something like this:
import asyncio
import logging
from datetime import timedelta
from threading import Thread

from temporal.activity_method import activity_method
from temporal.workerfactory import WorkerFactory
from temporal.workflow import workflow_method, Workflow, WorkflowClient
import time

logging.basicConfig(level=logging.INFO)

TASK_QUEUE = "HelloWorld"
NAMESPACE = "default"

# Activities Interface
class GreetingActivities:
    @activity_method(task_queue=TASK_QUEUE, schedule_to_close_timeout=timedelta(seconds=500))
    async def compose_greeting(self, greeting: str, name: str) -> str:
        raise NotImplementedError

# Activities Implementation
class GreetingActivitiesImpl:
    async def compose_greeting(self, greeting: str, name: str) -> str:
        time.sleep(150)
        return greeting + " " + name

# Workflow Interface
class GreetingWorkflow:
    @workflow_method(task_queue=TASK_QUEUE)
    async def get_greeting(self, name: str) -> str:
        raise NotImplementedError

# Workflow Implementation
class GreetingWorkflowImpl(GreetingWorkflow):

    def __init__(self):
        self.greeting_activities: GreetingActivities = Workflow.new_activity_stub(GreetingActivities)

    async def get_greeting(self, name):
        return await self.greeting_activities.compose_greeting("Hello!", name)

async def worker_main(activities=False, workflows=False):
    client = WorkflowClient.new_client(namespace=NAMESPACE)
    factory = WorkerFactory(client, NAMESPACE)
    worker = factory.new_worker(TASK_QUEUE)
    if activities:
        worker.register_activities_implementation(GreetingActivitiesImpl(), "GreetingActivities")
    if workflows:
        worker.register_workflow_implementation_type(GreetingWorkflowImpl)
    factory.start()
    print("Worker started")

def thread1():
    asyncio.set_event_loop(asyncio.new_event_loop())
    loop = asyncio.get_event_loop()
    asyncio.ensure_future(worker_main(activities=True, workflows=False))
    loop.run_forever()

def thread2():
    asyncio.set_event_loop(asyncio.new_event_loop())
    loop = asyncio.get_event_loop()
    asyncio.ensure_future(worker_main(activities=False, workflows=True))
    loop.run_forever()

if __name__ == '__main__':
    t1  = Thread(target=thread1)
    t2  = Thread(target=thread2)
    t1.start()
    t2.start()
    t1.join()
    t2.join()

Thanks for the suggestions @firdaus!

It's fine because there are separate GRPC calls behind the scenes for polling for workflow tasks and activity tasks so a workflow worker will never get the work of an activity worker.


Initial clk gen pearl

This PR has the initial clock generator pearl. A few discussion points:

Should bsg_clk_gen_lfsr_div30.v be generalized / go into basejump?
bsg_clk_gen_monitor_buf is basically just a buffer wrapper, but for hard-swapping. Should we have a clk_gen_buf in basejump for this purpose?
bsg_clk_gen_pkg should go in basejump. I just included it here for the purpose of not having two PRs necessary
I added a synchronous reset output. We almost always have clock-reset together, and the times when it's not necessary it's just an extra tag client. But I could be convinced to remove it

Perhaps the suffix for modules in this repo should be bsg_pearl_XXXX ?

Add README.md

@gaozihou you should be able to run your test bench against this?

@gaozihou you should be able to run your test bench against this?

Yeah it works with simple test after fixing the typo. Running more tests on it.

Superseded by #5


update with a single-node nodeport

Added an example with a single-node using nodeport instead of using some VIP address

it's tested on the lab and it's working fine:

systemd removed so it's finalise correctly
all pods running
nodeport configured correctly:

mgmt-cluster-network:/etc/systemd/system # k get svc -n metal3-system
NAME                                                    TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)                                        AGE
baremetal-operator-controller-manager-metrics-service   ClusterIP   <IP_ADDRESS>    <none>        8443/TCP                                       6m2s
baremetal-operator-webhook-service                      ClusterIP   <IP_ADDRESS>   <none>        443/TCP                                        6m2s
metal3-mariadb                                          ClusterIP   <IP_ADDRESS>   <none>        3306/TCP                                       6m2s
metal3-metal3-ironic                                    NodePort    <IP_ADDRESS>   <none>        6180:32534/TCP,5050:32087/TCP,6385:32494/TCP   6m2s

@e-minguez also added the change about clusterctl.


Infinite loop on driver open fail

fix for bug where we were going into an infinite loop if the open database failed (saw this on BB10 cordova app)
I don't think we should call _initStorage again since it just failed
fixed in pull request: #510

Perhaps we could remove this fallback and just reject the promise. The mechanism in the main localforage.js module can handle the fallback to the next preferred driver. I had seen that earlier, before the es6 refactor and started a discussion about it in an other issue, but can't find it right now. This was probably used before the fallback-on-fail mechanism was introduced, to prevent some missconfigurations.

@thgreasi so your saying we might not have to do self.setDriver(self.LOCALSTORAGE), it looks like indexdb does it this way (just resolves it):
        function ignoreErrors() {
            // Don't handle errors here,
            // just makes sure related localForages aren't pending.
            return Promise.resolve();
        }

        for (var j = 0; j < dbContext.forages.length; j++) {
            var forage = dbContext.forages[j];
            if (forage !== self) { // Don't wait for itself...
                initPromises.push(forage._initReady().catch(ignoreErrors));
            }
        }

Exactly. I think that catch should just do a reject().
Any chance you can provide a test case or a code example with the infinite
loop?

--
Thodoris Greasidis
Computer, Networking &
Communications Engineer

yes, it is easy to repro:
in websql.js replace:
dbInfo.db = openDatabase(dbInfo.name, String(dbInfo.version), dbInfo.description, dbInfo.size);
``
with:

throw new Error('quota exceded');

results in this:

Running "mocha:unit" (mocha) task
Testing: http://localhost:9999/test/test.component.html
․
Running PhantomJS...ERROR

PhantomJS has crashed. Please read the crash reporting guide at https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug report at https://github.com/ariya/phantomjs/issues/new with the crash dump file attached: /tmp/4966558E-41D1-407A-8444-C4363882E5FC.dmp 0 [ 'PhantomJS has crashed. Please read the crash reporting guide at https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug report at https://github.com/ariya/phantomjs/issues/new with the crash dump file attached: /tmp/4966558E-41D1-407A-8444-C4363882E5FC.dmp' ]
Warning: PhantomJS exited unexpectedly with exit code 2. Use --force to continue.

Aborted due to warnings.
localForage mike$

I guess we will have to add a test with since monkey patching...
On Tue, Jan 26, 2016, 21:13 Mike<EMAIL_ADDRESS>wrote:

yes, it is easy to repro:
in websql.js replace:
dbInfo.db = openDatabase(dbInfo.name, String(dbInfo.version), dbInfo.description, dbInfo.size);
``
with:
throw new Error('quota exceded');
results in this:
Running "mocha:unit" (mocha) task
Testing: http://localhost:9999/test/test.component.html
․
Running PhantomJS...ERROR
PhantomJS has crashed. Please read the crash reporting guide at
https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug
report at https://github.com/ariya/phantomjs/issues/new with the crash
dump file attached: /tmp/4966558E-41D1-407A-8444-C4363882E5FC.dmp 0 [
'PhantomJS has crashed. Please read the crash reporting guide at
https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug
report at https://github.com/ariya/phantomjs/issues/new with the crash
dump file attached: /tmp/4966558E-41D1-407A-8444-C4363882E5FC.dmp' ]
Warning: PhantomJS exited unexpectedly with exit code 2. Use --force to
continue.
Aborted due to warnings.
localForage mike$
—
Reply to this email directly or view it on GitHub
https://github.com/mozilla/localForage/issues/511#issuecomment-175185756
.

--
Thodoris Greasidis
Computer, Networking &
Communications Engineer

We might be able to redefine window.openDatabase to ‘undefined’ so that it throws an exception during a test, not sure I will have to play around with it. I’ll be pretty busy today, maybe tomorrow I can look into it
--
Mike Marcucio
marcucio.com http://marcucio.com/
<EMAIL_ADDRESS><EMAIL_ADDRESS>blog http://blog.getitdoneapp.com/ | twitter https://twitter.com/GetItDoneBlog | facebook https://www.facebook.com/GetItDoneTasks | google+ https://plus.google.com/u/0/b/116698933997097774721/+Marcucio/posts

On Jan 27, 2016, at 1:28 AM, Thodoris Greasidis<EMAIL_ADDRESS>wrote:
I guess we will have to add a test with since monkey patching...
On Tue, Jan 26, 2016, 21:13 Mike<EMAIL_ADDRESS>wrote:

yes, it is easy to repro:
in websql.js replace:
dbInfo.db = openDatabase(dbInfo.name, String(dbInfo.version), dbInfo.description, dbInfo.size);
``
with:
throw new Error('quota exceded');
results in this:
Running "mocha:unit" (mocha) task
Testing: http://localhost:9999/test/test.component.html
․
Running PhantomJS...ERROR
PhantomJS has crashed. Please read the crash reporting guide at
https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug
report at https://github.com/ariya/phantomjs/issues/new with the crash
dump file attached: /tmp/4966558E-41D1-407A-8444-C4363882E5FC.dmp 0 [
'PhantomJS has crashed. Please read the crash reporting guide at
https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug
report at https://github.com/ariya/phantomjs/issues/new with the crash
dump file attached: /tmp/4966558E-41D1-407A-8444-C4363882E5FC.dmp' ]
Warning: PhantomJS exited unexpectedly with exit code 2. Use --force to
continue.
Aborted due to warnings.
localForage mike$
—
Reply to this email directly or view it on GitHub
https://github.com/mozilla/localForage/issues/511#issuecomment-175185756
.

--
Thodoris Greasidis
Computer, Networking &
Communications Engineer
—
Reply to this email directly or view it on GitHub https://github.com/mozilla/localForage/issues/511#issuecomment-175427876.

Or even better to a function that throws an error.
Something similar is done in test.nodriver.html.
How about placing the new tests in a new file named test.faultydrivers.html or test.drivererrors.html?

Merged in #517 with some tidying.


Forked snapshot

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 79.164%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/amifamily/ami.go
1
90.32%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
5665

Relevant Lines:
7156

💛  - Coveralls


Search bar to search monitoring URLs.

If a user configures a number of URLs to keep track of then its really difficult to scroll down or use pagination to find a particular URL. In that case, a search bar can help them to find details of desired in no time.

Will this be done in the frontend or should we add this to backend ?

It can be done in frontend for sure. Cool, let's do it in frontend initially.

@Dineshs91 As we are doing polling every 5 seconds for getting latest status for each URLs. Implementing search in the client side is a bit complicated.
Can we add search query in GET /monitoring-url and backend will return the filtered results?


[NETBEANS-1288] Add trait support for Groovy

I am not an expert in this area, which specific parts need to be reviewed, how confident are you about this PR, what are the possible side effects, should we merge it?

Maybe, @brunoflavio-com should be able to review this area.

No big deal. Low risk change with large benefit - opening a file with a trait will not longer kill all your groovy editing.


Placeholder usage should be consistent among single- and multi-selects

As per the documentation, the usage of placeholders is different among single- and multi-selects:

In order for a string placeholder value to appear, you must have a blank <option> as the first option in your <select> control. This is because the browser tries to select the first option by default. If your first option were non-empty, the browser would display this instead of the placeholder. Note that this issue does not apply to multiple selects.

So in single-select, placeholders won't work without the <option></option>.
However in multi-select mode, having an empty placeholder option actually causes an issue where this empty value gets added as a selection whenever another item is selected.  See #4742, with an example here.
What's more, this appears to be further complicated by the use of allowClear.  See #4367 and #4880.
I propose that Select2 should handle creating the empty <option> element internally when needed (or some other workaround), providing a consistent interface for placeholder usage in both single- and multi-select scenarios.

I just want to note that this was proposed way back when in https://github.com/select2/select2/issues/3553.
The write up on there is 2 years old, so things very well may have changed, but it addresses the difficulties around creating the empty <option> element internally.


Build Names are suffixed with -*s2i regardless of build strategy

Description
While building a simple project with Docker build strategy on Openshift. I can see that the build object created during the build process is wrongly named. It is always suffixed with *-s2i regardless of whether build strategy is S2I or Docker. Here are some logs which can offer more clarity of issue:
~/work/repos/fabric8-maven-plugin/samples/webapp : $ mvn fabric8:build -Dfabric8.build.strategy=docker
[INFO] Scanning for projects...
[INFO]
[INFO] ---------------< io.fabric8:fabric8-maven-sample-webapp >---------------
[INFO] Building fabric8-maven-sample-webapp 4.4-SNAPSHOT
[INFO] --------------------------------[ war ]---------------------------------
[INFO]
[INFO] --- fabric8-maven-plugin:4.4-SNAPSHOT:build (default-cli) @ fabric8-maven-sample-webapp ---
[INFO] F8: Running in OpenShift mode
[INFO] F8: Using OpenShift build with strategy Docker
[INFO] F8: Running generator webapp
[INFO] F8: webapp: Using fabric8/tomcat-9:1.2.0 as base image for webapp
[INFO] Copying files to /home/rohaan/work/repos/fabric8-maven-plugin/samples/webapp/target/docker/fabric8/fabric8-maven-sample-webapp/latest/build/maven
[INFO] Building tar: /home/rohaan/work/repos/fabric8-maven-plugin/samples/webapp/target/docker/fabric8/fabric8-maven-sample-webapp/latest/tmp/docker-build.tar
[INFO] F8: [fabric8/fabric8-maven-sample-webapp:latest] "webapp": Created docker source tar /home/rohaan/work/repos/fabric8-maven-plugin/samples/webapp/target/docker/fabric8/fabric8-maven-sample-webapp/latest/tmp/docker-build.tar
[INFO] F8: Creating BuildServiceConfig fabric8-maven-sample-webapp-s2i for Docker build
[INFO] F8: Creating ImageStream fabric8-maven-sample-webapp
[INFO] F8: Starting Build fabric8-maven-sample-webapp-s2i
[INFO] F8: Waiting for build fabric8-maven-sample-webapp-s2i-1 to complete...
[INFO] F8:
[INFO] F8: Pulling image fabric8/tomcat-9:1.2.0 ...

 It would always be suffixed with *-s2i

The issue is not as trivial as it seems.
The suffix -s2i comes from the configuration property fabric8.s2i.buildNameSuffix (BuildMojo) which by default takes this value.
Given this fact we can consider 3 different scenarios each of one having a different expected behavior:

OpenShift build, S2I build strategy, no buildNameSuffix configuration: Build name gets suffixed with -s2i.
OpenShift build, Docker build strategy, no buildNameSuffix configuration: Build name doesn't get any suffix.
OpenShift build, Any build strategy, buildNameSuffix configuration exists: Build name gets suffixed with the provided buildNameSuffix.

hmm, I see. But from what I see build name is getting *-s2i suffix in the second option.

Yes, I described the expected behavior after the implementation. Just want to confirm this is what we want i.e. if s2i suffix is configured, keep it. Else if it's a s2i build append -2si by default.


pre-commit has changed from hooks.yaml -> .pre-commit-hooks.yaml

Hello pre-commit hook implementer!
In version 0.12.0 pre-commit has changed the default location for the file formerly known as hooks.yaml to make it more convincing for others to add more hooks.
As such, a migration has to (unfortunately) occur.
For maximum compatibility it is suggested to cp hooks.yaml .pre-commit-hooks.yaml (at least for the migration period).  A copy is suggested over a symlink unless you do not care for windows compatibility (and I wouldn't blame you!).
Once the migration period is over (or you no longer care to support old versions of pre-commit), the hooks.yaml file is no longer necessary and may be deleted.
See https://github.com/pre-commit/pre-commit/pull/470 for more details
Thanks again for contributing to the pre-commit ecosystem, we couldn't do it without you :)
Anthony

This repository has been temporarily removed from pre-commit.com as the latest version of pre-commit (0.16.0) has removed support for legacy hooks.yaml.  Without providing .pre-commit-hooks.yaml, users will be unable to use this hook repository with pre-commit>=0.16.0.

We use these plugins at our company all day long, so I've forked the repo to digitalpulp/pre-commit-php.  I'll pull @djun-kim PR into that repo and push it up in a moment.

@FatherShawn sweet, I'll add that repo for the time being as this one seems to be no longer maintained

@asottile Thanks!  We are super grateful to Yelp for pre-commit and to Hootsuite for contributing th e PHP hooks.  And we are happy to take them over since this issue has been open for so long.

See https://github.com/pre-commit/pre-commit.github.io/pull/123

👍 there's probably one or more of these PRs you'll want to incorporate as well: https://github.com/hootsuite/pre-commit-php/pulls

closing due to inactivity


Cannot query field "edges" on type "ProductImage"

What I'm trying to achieve
I get the error message Cannot query field "edges" on type "ProductImage" when I run the following query.
query sampleQuery {
  products(first: 10){
    edges{
      node{
        images{
          edges{
            node{
              alt
            }
          }
        }
      }
    }
  }
}

edges is highlighted by the Playground editor. This query ran fine on previous versions of saleor. And the GraphQL schema itself also suggests that images should be a list.
Screenshots

System information
Operating system: Windows 10 (64 bit)
Browser: Firefox 63

That's right, images should be a list, as that's not a connection field anymore. We've changed that recently

Our API is in beta and we're still doing improvements like this. We decided to change some of the connections to simple lists as there is no need to do the pagination where number of related object isn't expected to be high (e.g. product images or variations). See PR #3307 for details.


Use the rust-specific config

Per the TODO, we could leverage Nixpkgs to do this better still, but it
isn't terribly convenient at this time. I rather do it after the other
cross PRs are merged.

I'm not longer worried about testing this so much, because now that it uses a function and isn't conditional, the ball is more in Nixpkgs court (there is a single code path on the create2nix side).
So merely ensuring that existing functionality still works should be good.


feat: print component id when spin up

The available routes will be printed out in the following format:
Component Id: URL (wildcard)
 (wildcard) will be printed out iff applicable.
Examples:

hello: http://<IP_ADDRESS>:3000/hello
static: http://<IP_ADDRESS>:3000/static (wildcard)

Signed-off-by: Hanzhen Liang<EMAIL_ADDRESS>

Fix first point of #412

Could you also add a reference to the new field in the component configuration here, please?
https://github.com/fermyon/spin/blob/main/docs/content/configuration.md
Thanks!


Remove feature flags for bypassing the toolcache

We can now use the default bundle version feature flags to remediate a bad bundle update.
Controlled switchover ensures that a repo consistently gets the same bundle version, so we no longer have alert churn concerns with Kotlin and Swift.

Merge / deployment checklist

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.
[x] Confirm the changelog has been updated if necessary.

🔥


Listing doesn't continue from the last post.

I'm trying to get all the posts from a subreddit/user page.
Notice that, for instance, listing #94 is the same in all four iterations of listing posts and also with each new iteration the listing gets decreased by one post.
I think my issue is with the infinite loop but I'm not sure.
 In each iteration, you’re asking for all posts chronologically after the chronologically oldest post in the set, so I would expect your results.
You can use ListingWithParams to specify Reddit API arguments directly. To Reddit directly, “after” will mean “chronologically before”, and you will observe different behavior.

Thank you very much. Fixed.


Generating ASP.NET Core Controllers

SwaggerToCSharpWebApiControllerGenerator currently generates Web API 2 controllers. Would you be willing to accept a PR to generate ASP.NET Core controllers?
I'm thinking this could be handled by passing a parameter, similar to #884.
What I'm open-minded about is whether to have separate T4 templates for ASP.NET core or to just add conditional formatting based on the model.

I think this is a good idea. We should use the same template as it will be much more time consuming to always update both templates with new features... What do we have to change? Just change the namespaces of the base class and attributes?

That's pretty much it. It's minor enough.

This would be a great feature. I use it for client generation right now but it would be excellent to also use it as a generator for the controllers.  Then the producers and consumers could agree on one OpenAPI spec file and generate code on both sides. The perfect setup.

I created a powershell script and generated the controller. In the end of the script i just ran:
`
$replacements = @{
"System.Web.Http.ApiController" = "Microsoft.AspNetCore.Mvc.Controller";
"System.Web.Http." = "Microsoft.AspNetCore.Mvc.";
}
$content = [System.IO.File]::ReadAllText($sourceFile)
foreach ($replacement in $replacements.GetEnumerator()) {
Write-Host "replacing $($replacement.Name) with $($replacement.Value)"
$content = [System.Text.RegularExpressions.Regex]::Replace($content, $replacement.Name, $replacement.Value)
}
[System.IO.File]::WriteAllText($sourceFile, $content)
`
And then it worked fine for ASP.NET Core 2.0 at least

Looks like we need two new settings:

[ ] AspNetNamespace (default: "System.Web.Http"), use for all attributes, etc. OR

Or just IsAspNetCore and hardcode the used namespace in the template

[ ] ControllerBaseClass (default: "System.Web.Http.ApiController"). Does this already exist?

Hi @RSuter , wondering if you have an ETA for resolution of this issue? I can see there has been a lot of recent activity on NSwag, so I'm sure you're quite busy with other issues.  This one happens to be high-priority for one of our clients. Any updates appreciated.  Thanks!

@tedepstein I'm currently concentrating on other issues which are relevant for a bigger audience and my paying clients

OK @RSuter . We may take a look at this and consider submitting a pull request if it's something we can address.

See https://github.com/RSuter/NSwag/wiki/SwaggerToCSharpControllerGenerator

It seems something changed, see https://github.com/RSuter/NSwag/issues/1564


Build failure: rubiks

Steps To Reproduce
Steps to reproduce the behavior:

build rubiks

Build log
building '/nix/store/bb6ai7mn37881im72lr2xhgk576f1yaa-dietz-cu2-Makefile.patch?id=07d6c37d18811e2b377a9689790a7c5e24da16ba.drv'...

trying https://git.sagemath.org/sage.git/plain/build/pkgs/rubiks/patches/dietz-cu2-Makefile.patch?id=07d6c37d18811e2b377a9689790a7c5e24da16ba
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0  9115    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404
error: cannot download dietz-cu2-Makefile.patch?id=07d6c37d18811e2b377a9689790a7c5e24da16ba from any mirror

Additional context
Since all these URLs on git.sagemath.org seem to fail, I changed them to corresponding raw.githubusercontent.com URLs: https://github.com/zeuner/nixpkgs/tree/rubiks-download-fix . However, this leads to a hash mismatch:
trying https://raw.githubusercontent.com/sagemath/sage/develop/build/pkgs/rubiks/patches/dietz-cu2-Makefile.patch?id=07d6c37d18811e2b377a9689790a7c5e24da16ba
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1059  100  1059    0     0   3845      0 --:--:-- --:--:-- --:--:--  3850
error: hash mismatch in fixed-output derivation '/nix/store/a6xxf5q9bh1xfhq2jk8izd14f5xw0mam-dietz-cu2-Makefile.patch?id=07d6c37d18811e2b377a9689790a7c5e24da16ba.drv':
         specified: sha256-uOKUWM6P/krhalRJdyFplN4Y2wpK0R9Sw5LgNGvgw+c=
            got:    sha256-bRU7MJ/6BgCp2PUqZOragJhm38Q3E8ShStXQIYwIjvw=

Unfortunately, I don't have the previous patch available, so I can't check what change leads to the hash mismatch. This one particular patch doesn't look unreasonable. I don't know how to update the hash entry, though, because it does not follow the sha-256- notation: https://github.com/zeuner/nixpkgs/commit/0ca86d970642a81cfeb9b3d19e944861c4abb8f5#diff-fd1b8a8025457c6f8c2fc29431ea1e4fc177a47d5ab85fc5c63bd198ff04a430L32
Notify maintainers
@timokau @omasanori @raskin @collares
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.0-78-generic, Ubuntu, 22.04.3 LTS (Jammy Jellyfish), nobuild`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.6.0`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/root/.nix-defexpr/channels/nixpkgs`

On it! Thanks for the report.


Adjust color contrast for github-hover

Fixes Issue
#315

Changes proposed

Adjusted color contrast for github-hover for both the dark mode as well as for the light mode

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Dark Mode

Light Mode

Note to reviewers

We can also go for another option, the light tint of grey, let me know what looks good(attached some screenshots for your reference)
Dark Mode

Light Mode

Let's go with darker tone,it matches the other boxes contrast tone


Cannot get the path of a dropped directory from Finder

This probably also applies to other platforms, as the limitation is due to an inaccessible private interface which can actually provide the directory path. You can either include the following private files:
import 'package:super_clipboard/src/reader_value.dart';
import 'package:super_native_extensions/src/native/virtual_file.dart';

final adapter = (reader as DataReaderVirtualFileAdapter);
final virtualFile = (adapter.file as VirtualFileFromFile);
final directoryPath = virtualFile.file.path;

Or you can dispense with the private imports:
      try {
        final path = (reader as dynamic).file.file.path;
        if (!kIsWeb && FileSystemEntity.isDirectorySync(path)) {
          ... // can perform folder operations
        } else {
          ... // can get file bytes
        }
      } catch (_) {}

Have you tried getting the fileUri value from the Reader? This seems like doing lot of unnecessary work just to get the URI.

@knopp I cannot see how to do that. Do you have any code example? I am using:
super_clipboard: ^0.8.19
super_drag_and_drop: ^0.8.19
I cannot see any fileUri function or property for example on Reader.


improvement(policies): make policies easier to navigate

User Story
As a: contributor to the defsec project,
I would like to: have a simplified project structure for policies*,
So that: it's easier to contribute new ones and navigating existing policies
*: In the context of this improvement, policies, rules and signatures are used interchangeably
What exists today

Rules are scattered in different places through the project. This makes it a little harder for new folks to discover and understand the project structure. For instance:

a) Some rego rules: https://github.com/aquasecurity/defsec/tree/master/internal/rules/policies
b) Golang rules: https://github.com/aquasecurity/defsec/tree/master/internal/rules
c) Some rego rules are places alongside with helper policies: https://github.com/aquasecurity/defsec/tree/master/internal/rules/policies/docker and https://github.com/aquasecurity/defsec/tree/master/internal/rules/policies/kubernetes
d) There's a symlink to policies dir that exists on the top level of the project: https://github.com/aquasecurity/defsec/blob/master/policies
Further more as shown by point a) polices are part of the internal directory, something that makes them a little obscure to the outer world. IOW, policies can be reused and aren't necessarily only limited to be part of the internals of the defsec project.
What is desired
In an ideal world (all signatures are Rego based), the directory structure could look like the following:
$ defsec > tree .
./rules
├── cloud
│   └── policies
├── docker
│   ├── README.md
│   ├── lib
│   ├── policies
│   └── test
└── kubernetes
    ├── README.md
    ├── lib
    └── policies

Where furthermore each of the services could look like the following:
$ defsec > tree ./rules/cloud/policies
rules/cloud/policies
└── aws
    └── rds
        ├── disable_public_access.rego
        ├── disable_public_access_test.rego

Acceptance Criteria

Policies should be easier to discover and navigate.

Signed-off-by: Simar<EMAIL_ADDRESS>

great! another piece of content we need to consider in this structure is compliance reports

do we need the policies level? seems redundant?

after we have a stable hierarchy, we can consider applying in to the package name. This could serve us in the future for in the rego level. For example, to apply an input schema at an aws service level. or to query results for a specific service

Closed via https://github.com/aquasecurity/defsec/pull/1161


Experimental example for LiDAR and IMU extrinsics

Work in progress - will land once we confirm the transforms make sense.

Current dependencies on/for this PR:

main

PR #73   👈

This comment was auto-generated by Graphite.


fix: prettify warning

This is soo much easier on the eyes...

@tlivings bump. What do you think?

A month later... done!

@tlivings bump!


Temp pref test

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@0edc5c9). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop      #10   +/-   ##
==========================================
  Coverage           ?   77.72%
==========================================
  Files              ?       12
  Lines              ?     2352
  Branches           ?        0
==========================================
  Hits               ?     1828
  Misses             ?      524
  Partials           ?        0

Flag
Coverage Δ

unittests
77.72% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Cxf-cdi archetype has a dependency issue

Get the following error during the integration test phase:
java.lang.NoSuchMethodError: com.fasterxml.jackson.databind.ObjectMapper.readerFor(Ljava/lang/Class;)Lcom/fasterxml/jackson/databind/ObjectReader;

I have not seen that error, are you able to reproduce this with latest release?

Cannot reproduce


"az rest": allow the HTTP request body to be taken from a file (similar to curl's "--data @filename")

Is your feature request related to a problem? Please describe.
When using az rest under Windows, if you need to pass a large body you quickly hit the length limit for a single command.
Describe the solution you'd like
I'd like to be able to put the body in a file and pass a reference to that file instead of the the actual body to the az rest command, e.g.
az rest -m PUT -u https://management.azure.com/bla --body @filename
Describe alternatives you've considered
The alternative is to use curl instead but it requires obtaining the subscription ID and the authentication token beforehand:
az account show --query id
az account get-access-token --query accessToken
curl -X PUT "https://management.azure.com/subscriptions/%SUBSCRIPTION_ID%/..." -H "Authorization: Bearer %TOKEN%" -H "Content-Type: application/json" --data @filename

Additional context
According to the doc, headers and query params can already be loaded from a file using the @ notation, so it would seem natural to extend this to the body.

--body actually does support @ notation. @ notation applies to all CLI args. https://github.com/Azure/azure-cli/blob/516d297a9f1276bd9a9208e4a43281238b0c496e/src/azure-cli-core/azure/cli/core/commands/init.py#L83
I have created PR #10632 to fix the help message.


Adds lxd-agent option to apply name and MTU to nic devices

Adds information about expected nic devices to the VM config share. The lxd-agent reads and applies the configuration on startup.
Closes #7233

Marking as draft as I think this change may be causing lxc list to be missing some ip addresses. Investigating now.

Also as there are a fair few moving parts to this PR I think it would be good idea to update https://github.com/lxc/lxc-ci/blob/master/bin/test-lxd-network with tests for mtu and name renaming.

I've added an issue in the CI repo for this and assigned myself.

Left some comments, on top of that, I think the device name validation logic ought to be its own PR.
And on the lxd-agent feature front, you need an API extension as you're introduce a new config key, so will need a api: agent_rename_interfaces commit with changes to doc/api-extensions.md and shared/version/api.go

@markylaing please can yiu et me know when done and I'll do another scan through. Thanks

jenkins: test this please

@markylaing lets wait until https://github.com/lxc/lxd/pull/9960 is merged and we can test this against it.


README.md: add example for pulling 5.6

Emphasize that images for 5.5 and 5.6 reside in a different namespaces.
PTAL @bparees @mfojtik

LGTM