Docker container not cleaning up properly

I think there is some sort of storage leak when running self-hosted runner.
We have the issue that our VM runs out of storage.
This has happens every 2-4 month maybe.
I first contributed this to microsoft releasing lots of new artifacts in a short period. This would cause a lot of artifacts accumulating but this doesn't seem to be the case this time.
We had 30 GB Artifacts and 200 GB docker files.
30 GB artifacts seems normal but 200 GB docker files a bit much.
I have a little powershell script thats purges everything older then a day.
docker image prune -a --filter "until=24h" -f

Get-ChildItem –Path "C:\bcartifacts.cache" -Recurse | Where-Object {($_.LastWriteTime -lt (Get-Date).AddDays(-1))} | Remove-Item -Force -Recurse

But this didn't solve it this time.
It seems like there were tons of exited container that where still "using" the images so they couldn't be purged.
I had to add the following in the beginning of my script to free up the space this time.
docker rm $(docker ps --filter status=exited -q)
I'm contemplating, running this script every month or so, to be sure no storage accumulates. But I thought other might run into a simular issue and running this every month doesn't solve this issue for good it just makes it less likely.
Maybe there is something that can be done in the al-go cleanup to prevent this for everybody.

Thanks
I actually just added the docker image prune with a filter to Flush-ContainerHelperCache last week.
It looks like maybe I should look into added a mechanism to remove exited containers as well - probably with a flag (and then add this flag to AL-Go).

I guess this code inside of Flush-ContainerHelperCache would work:
docker container ls --format "{{.ID}}:{{.Names}}" --no-trunc -a --filter "status=exited" | ForEach-Object {
    $containerID = $_.Split(':')[0]
    $containerName = $_.Split(':')[1]
    $inspect = docker inspect $containerID | ConvertFrom-Json
    $finishedAt = [DateTime]::Parse($inspect.state.FinishedAt)
    if ([DateTime]::Now.Subtract($finishedAt).Days -ge $keepDays) {
        Write-Host "Removing container $containerName"
        docker rm $containerID -f
    }
}

This seems good but have you investigated why containers are not removed properly in the first place? This feels like a patch on a leak. I think each run shouldn't leave any unwanted data behind.

It is definitely the intention that every workflow run cleans up the container - the code should do this.
If you have any logs from workflow runs where this isn't the case, I would love to see them?
I haven't experienced this myself.

I've just checked our runner again the last 5 days didn't collect any exited container i'll check periodically to see if they accumlate maybe this helps to narrow it down.

I've just checked our runner again the last 5 days didn't collect any exited container i'll check periodically to see if they accumlate maybe this helps to narrow it down.

Yep, if I can get the logs from a run, which doesn't cleanup the containers - I will be able to discover why.
In the meantime, we will implement the cleanup of exited containers which are more than KeepDays old in the Flush-ContainerHelperCache called from AL-Go.

I just wanted to add that we are experiencing the same issue. I can't pinpoint where it comes from either, but it mainly seems like the containers that are left over on my end are from the image creation and not the actual containers use to run the pipelines.
I am contributing this to running the preview version of AL-Go and experiencing sometimes issues here and there, which break the pipeline. And, if the entire process doesn't make it far enough, it seems that it can't clean up the container. I will certainly try to keep an eye out as well to see what actually causes this.

Leftover containers from building images should have been removed with the change I made to ContainerHelper here:
https://github.com/microsoft/navcontainerhelper/pull/3107

Thanks Freddy. I will monitor it and see, if those were leftovers from before the change or not. I cleaned up all self hosted runners yesterday again, so it should not happen again, if everything works.


aix: fix -Wmaybe-uninitialized warning

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/602/

LGTM

Landed in https://github.com/libuv/libuv/commit/9f70d39b5aaaf25fb8a7e8ce1f0c9033c45abe85.


Add web command to request usb access

This experimental command triggers a request for usb devices. After the user selects these, extensions under the extension host can access them using navigator.usb.getDevices(...)

I think contrib is a bit overkill for this, if we think as a platform web USB should be supported, I would make this an official extension command somewhere where other extension commands contribute.


Machine win-9 pickup errors - Quarantined: Had 2 consecutive BOT_DIED tasks.

Machine win-9 pickup errors - Quarantined: Had 2 consecutive BOT_DIED tasks.

Error is gone by rebooting machine.


Simplify template comments to fix bugs: allow any content and any marks of the doc schema

Overview
Template comments had some problems by trying to avoid allowing <em> marks inside its content. The problem was that lists also needed to be allowed, but lists (and their button functionality) would use normal paragraphs again.
This PR allows normal paragraphs for template nodes, removing the constraint of avoiding <em> marks. It also includes some cleanup cherry-picked from another PR.
connected issues and PRs:

cherrypicked cleanup commits from https://github.com/lblod/ember-rdfa-editor-lblod-plugins/pull/236
A discussino of the problems with the custom paragraphs and list can be found in the comments of https://github.com/lblod/ember-rdfa-editor-lblod-plugins/pull/241
because this simplifies template comments, no specific ember-rdfa-editor version is needed
related bug with lists in template comments: https://binnenland.atlassian.net/browse/GN-4445

How to test/reproduce

Challenges/uncertainties/Remarks

TODO: create a ticket to look at this issue further and see if something like fully custom content in a child block is a thing that is (easily, without hacks) possible. Potentially this can be done with an embedded-editor, if the implementation for that is fixed to work with block content and it is fixed so that a custom schema can be provided.
to partially follow the design, a base <em> mark could be added when adding a template comment, but I couldn't get this to work (yet), so that's why it isn't included here. This can always be added later.

Works really nicely :) I was wondering if it is possible to pass a list of marks when creating the template node, but I tried to test it and it did not seem to work.

Yes I tried that too, but didn't work. Documentation doesn't specify well what that parameter does.
Ticket to check it separately: https://binnenland.atlassian.net/browse/GN-4454


Associate markers with vehicles

A recent update seems to have improved the behavior of map markers associated with vehicles. I will take this opportunity to check the behavior of map markers and popups associated with vehicles and adopt it if it seems to be better than the current implementation.

The map markers associated with the vehicles did indeed work in single-player. However, in multiplayer, there was still a bug that caused the map markers to be misplaced when the player was too far away from the vehicle.
There didn't seem to be a way to associate a pop-up with a vehicle yet.


[Browser Bookmarks] Search terms are not accurately matching against bookmarks

Extension
https://www.raycast.com/raycast/browser-bookmarks
Description
Search terms are not accurately matching against bookmarks
Steps To Reproduce

Add a search term that filters bookmarks down to a list. For example:

I am ultimately trying to filter down to the bookmark with "sync" in it (see above screenshot with red arrow), so add another term to attempt to filter the bookmarks down further. For example:

Observe that all bookmarks are filtered out despite providing terms that are relevant

Current Behaviour
Search terms ignore terms in a bookmark's name. I do not have this issue in Alfred with their default bookmark search:
First query (notice how results are basically the same as Raycast's at this point):

Second query:

Expected Behaviour
Search terms should match terms in a bookmark's name

Thanks for bringing this up. It appears that this is a duplicate of issue #8514, so I'll mark it as such.


Error when using with React

I tested with React but got error
Code:
import { pipeline } from "@xenova/transformers";
....
const classifier = await pipeline("translation");

Error:
tokenizers.js:1080 Uncaught (in promise) TypeError: text.split is not a function
    at Function._encode_text (tokenizers.js:1080:1)
    at Function.encode (tokenizers.js:1105:1)
    at tokenizers.js:970:1
    at Array.map (<anonymous>)
    at Function._call (tokenizers.js:970:1)
    at Function.closure [as tokenizer] (utils.js:372:1)
    at Function._call (pipelines.js:214:1)
    at closure (utils.js:372:1)
    at basicStateReducer (react-dom.development.js:16540:1)
    at updateReducer (react-dom.development.js:16664:1)

Hi! This is because the you are using setState on a function, which (I believe) calls the function under the hood.
I'm by no means a react expert, but you can get around this by saving
import React, { useState } from 'react';
import { pipeline } from '@xenova/transformers';

/*
Very basic sample react app for transformers.js, showing how to store the model as a promise.

NOTE: This will work for testing, but since the prediction is running in the main thread, the
UI will freeze when loading and running the model. The correct way to handle this is by using
a web or service worker, to offload those intensive operations to a separate thread.
*/
function App() {
  // Stores the model's function as a promise
  const [modelPromise, setModelPromise] = useState<Promise<any> | null>(null);

  // Stores the input text
  const [inputText, setInputText] = useState('');

  // Stores the result/preduction
  const [resultText, setResultText] = useState('');

  const handleInputChange = async (event: React.ChangeEvent<HTMLInputElement>) => {
    setInputText(event.target.value);

    let model = await modelPromise;
    if (!model) {
      // The model hasn't been loaded yet. So, we load it here
      let modelPromise = pipeline('text-classification');
      setModelPromise(modelPromise);
      model = await modelPromise;
    }

    setResultText(await model(event.target.value))
  };

  return (

      Input Text:
      <input id="input-text" type="text" value={inputText} onChange={handleInputChange} />
      {resultText && (

{JSON.stringify(resultText)}

      )}

  );
}

export default App;

@phamquyhai did the above code solve your problem?

@phamquyhai did the above code solve your problem?

Sorry to say that code does not look correct @xenova. Its instantiating a new model and inferring everytime is typing. He needs to use useEffect to create the model once and do the infer after finished typing like you would do in a rest api with setTimeout.
I would be able to help. Im quite proficient in react. But I would need to see what he wants to build.

this my hooks
// @ts-ignore
import { pipeline } from "@xenova/transformers";
import { useEffect, useState } from "react";

type TASK =
  | "text-classification"
  | "question-answering"
  | "fill-mask"
  | "summarization"
  | "translation"
  | "text2text-generation"
  | "text-generation"
  | "automatic-speech-recognition"
  | "image-to-text"
  | "image-classification"
  | "zero-shot-image-classification"
  | "embeddings"
  | "sentiment-analysis"
  | "ner"
  | "vqa";

export const useTransformers = (task: TASK, model?: string | null) => {
  const [classifier, setClassifier] = useState<any>(null);
  const [status, setStatus] = useState<"pending" | "done" | "error">("pending");
  useEffect(() => {
    (async () => {
      setStatus("pending");
      try {
        const classifier = await pipeline(task, model || null, {
          progress_callback: ({
            progress,
            ...args
          }: {
            progress: number;
            loaded: boolean;
            total: number;
          }) => {
            console.log("progress args : ",args);
          },
        });
        setClassifier(classifier);
        setStatus("done");
      } catch (error) {
        setStatus("error");
      }
    })();
  }, [model, task]);

  return { classifier, status };
};

Using this hook
const { classifier, status } = useTransformers("translation");
  console.log("status", status);

Hi @phamquyhai, any updates?

tokenizers.js:1080 Uncaught (in promise) TypeError: text.split is not a function

Are you adding the wasm files to your public folder?

Hey, thanks a lot for this amazing library!
I am using the above example code to run the whisper-base model inside an electron app. Could you tell me what I would have to change / add to make the model run in a web/service worker?

Sure! I would recommend looking at the demo web worker script here:
https://github.com/xenova/transformers.js/blob/main/assets/js/worker.js
There are many ways to go about using the library, but probably the easiest is to use a singleton object for your app. This just ensures that you only create the pipeline once and creation is done lazily (the first time it is called).

Thanks for the quick reply! Will try this tomorrow and let you know how it goes

Closing since original issue is not a problem with the library, rather it is an issue with how it was used. Also because the original poster has not responded in 3 weeks.
@lucabeetz Feel free to open a new issue if you wish to continue our discussion.


Create how-to-work-on-coding-challenges.md in german

Translate how-to-work-on-coding-challenges.md into german

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

Hi @Plingooo same as your previous pull-request can you get this PR reviewed

Thank you for reviewing @Squizzi3 !

@raisedadead
Why did the Job fail?

We are still in the process of integrating the Circle CI. You can ignore that check for now. See #37284


Cant link to room

I really need the room function cause my game is multiplayer with different matches
But i cant because the username function is required to create a room but its disabled (saw it in the logs)
Is there any other way i can make a room and why got the username function disabled

Your using server 2, that uses a custom protocol extension.

So what of the code must i install to only use server 1?

connect to server (1)

thanks for the help. it works now

i use my own server but it always seems to use the custom protocol so where can i find the source code without the this

how can i use this in my own server?

i use my own server but it always seems to use the custom protocol so where can i find the source code without this

Where are you currently getting the source code from?

rhis repository the whole code

this repository, the whole code

This repo doesn't include the custom protocol stuff that server 2 uses. Are you getting the error about setid being disabled on others servers?

yes i cloned this repository on my replit server and got the error in the console

no but i cloned this repository on my replit server and got the error in the console

Can you send me a link to the repl?

no but i cloned this repository on my replit server and got the error in the console

Can you send me a link to the repl?

cloudlink.zghghffgbfgfg.repl.co

Yeah this has custom codes that's used within server 2... this was definitely not cloned from this repo though because that code doesn't exist in this repo.

It looks like your repl was cloned from @showierdata9978's repl which is server 2. Create a new repl and choose 'Import from GitHub'. Also make sure when importing you do MikeDev101/cloudlink for the repo you want to import from.

ok, thanks it works now


ci(pre-commit): add autoupdate workflow

See https://github.com/autowarefoundation/autoware-github-actions/pull/195.

Worked:
https://github.com/autowarefoundation/autoware/pull/3051
https://github.com/autowarefoundation/autoware/pull/3052

Since these are just new CI workflows, I'll merge and test them.

FYI @xmfcx


Pin Python dependencies with pip-tools

Motivation and context
pip dependency resolver was taking extremely long due to complex version
resolution prior to this change (>6 hours on CI). This commit introduces
version pinning with pip-tools. A wrapper script to update the pinned
versions when dependencies are added, or when upgrades are needed, is
provided.
Fixes #4618
How has this been tested?
CI-nightly build phase seems to pass the error described in #4618. However, the full build still doesn't complete (>6h passed).
Checklist

[x] I submit my changes into the develop branch
[ ] I have added a description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
[x] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below)

# Copyright (C) 2022 Intel Corporation
#
# SPDX-License-Identifier: MIT

@ngaloppo, thank you for the PR! It seems that the issue linked was with a specific package, and it is resolved now. I'm not sure that we want this specific solution for the develop branch, but it can be a good way to make tagged releases and custom deployments more reproducible and stable against non-fixed dependency updates. @azhavoro, what do you think? Can we run such a script before releases?

Please also check and comment on https://github.com/opencv/cvat/issues/4270.

I quite like pip-compile, and I would support integrating it. That said, I think a few tweaks need to be made to the current approach:

We should not immediately drop all version specifiers for the requirements.in files. Rather, the initial .in files should just be renamed from the existing .txt files. This will help avoid unexpected changes when people regenerate the .txt files later. Later on, we can unpin/relax versions on a case-by-case basis.

We should use pip-compile-multi to avoid duplication of requirements between the various .txt files. pip-compile-multi also automatically finds the files to process, so it'll let us drop the automation script.

https://github.com/opencv/cvat/issues/5310


Only call QWidget::windowHandle() for Qt >= 5.0

Coverage remained the same at 94.97% when pulling 9a930e6ee37cf0665e40706a769088342a981faf on fix-qt5-error into 0843b61f26cc6cef4150e9f81925153fad19d3a6 on develop.


luaPackages.ldbus: init at scm-0

Motivation for this change
This adds ldbus, a Lua library to use dbus. Note that the last version is 3 years old, but it's used by some AwesomeWM widgets, so it might be useful to have.
I use it for a bluetooth widget on awesome and it works well.
Related issue: #90532
cc @teto
Things done

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

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

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

when there is a server set, luarocks-nix shouldn't use a mirror. I don't remember if I already fixed it or not but while waiting for an update, you could fix the url manually and use http://luarocks.org/dev/ldbus-scm-0.rockspec instead of mirror:///

when there is a server set, luarocks-nix shouldn't use a mirror. I don't remember if I already fixed it or not but while waiting for an update, you could fix the url manually and use http://luarocks.org/dev/ldbus-scm-0.rockspec instead of mirror:///

Ok, a thing to notice is that the rockspec seems to be downloadable in luarocks.org and luarocks.org/dev as well. Idk why Luarocks asks to specify that server.


Need to support configure pullimagePullPolicy for deployment service

In GitLab by @luotian-github on Dec 6, 2019, 06:27
删除服务后，重新上传镜像，版本号相同，再次创建服务时还是使用的原先的镜像

In GitLab by @luotian-github on Mar 4, 2020, 08:39
assigned to @luotian-github


Individual Content Size for every  State Image/Icon

If there is more than one 'State' (Universal Widget) and there is the need to have different Icons for each 'State' please add the possibility to change the size for Image/ Icon  individually for each Icon of the State.

v0.1.7
Option added to each state. If value is zero the default size will be used


dte: Allow negative 'monto_total' when DTE type is 'liquidación-factura electrónica'

Resolves: https://github.com/fyndata/lib-cl-sii-python/issues/79

Codecov Report

Merging #80 into develop will increase coverage by 0.18%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop      #80      +/-   ##
===========================================
+ Coverage    66.75%   66.94%   +0.18%
===========================================
  Files           22       22
  Lines         1444     1446       +2
  Branches       213      214       +1
===========================================
+ Hits           964      968       +4
+ Misses         435      434       -1
+ Partials        45       44       -1

Impacted Files
Coverage Δ

cl_sii/dte/constants.py
100% <100%> (ø)
:arrow_up:

cl_sii/dte/data_models.py
66.9% <100%> (+1.9%)
: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 7d30bf1...dc8b740. Read the comment docs.


ColorVariable incorrectly reporting error

I have a color map that looks like this
$green: #9ecc3b;
$white: #fff;
$red: #f00;

$colors: (
    'green': $green,
    'white': $white,
    'red': $red
);

and I am getting errors like this
assets/sass/base.scss:50 [W] ColorVariable: Color literals like `green` should only be used in variable declarations; they should be referred to via variable everywhere else.
assets/sass/base.scss:51 [W] ColorVariable: Color literals like `white` should only be used in variable declarations; they should be referred to via variable everywhere else.
assets/sass/base.scss:52 [W] ColorVariable: Color literals like `red` should only be used in variable declarations; they should be referred to via variable everywhere else.

Which version of scss-lint are you using, @ryanmc2033?
Please upgrade to the latest version of scss-lint, as a similar issue was reported in #402 and has been fixed.


Start using jar-dependencies in the gemspec.

This change affects the Java gem and is an attempt to start using Maven artifacts for all the jar dependencies in Nokogiri.
See #1253. The commented versions can't be found on Maven Central. Until we can resolve them, they would have to remain as bare jar files in the repository and gem.
Next step for this PR would be to get the gem building using jar-dependencies and remove the properly-versioned jars from the repository.

@headius @jvshahid please see my branch
https://github.com/mkristian/nokogiri/tree/jar-deps

load all jar via jar-dependencies using it maven coordinate
add rake task to update jar using Jarfile as source
using Jarfile to declare jar deps in gemspec

a few jars are not part of maven central so I copies them to right place, i.e. they will be still part of the duplicate jar warning/detection of jar dependencies. the jing.jar will start up but produces NullPointerException in tests. Still need to look into this - as I did see the same errors when using the original jar.
running tests I used rake test or is there something else as I do see errors on master as well.

The underlying rakefiles and gemspec have evolved enough that I'm going to close this and start a new PR. I'll tag @headius and @mkristian for review and feedback on it.


[Clarification] Supported input / output combinations for VPP colorspace conversion

YUY2 output format support using VPP CSC (colorspace conversion) works on Intel media drivers (Windows and Linux) which I tested to comply with MSDK version 1.22+, but this support is not very well documented in the manual. The MSDK reference manual clearly specifies that NV12 and RGB32 output formats are only supported for color space conversion. I believe the manual needs to be updated to reflect the new YUY2 output format support, and any other supported input / output CSC combinations.
So for now, I would like to ask until such information is documented: what is the minimum MSDK version support that should be advertised by the media driver to know for sure that YUY2 output format is definitely supported for CSC?

Actually, I think I'll just use MFXVideoVPP_Query () to query the support instead of hacking with the MFX version. I'll rename this question then instead, it could help clarify things for others.

Hi @ishmael1985 ,
I found YUY2 in color conversion support declared at p11-12 of Media SDK manual. And I see it mentioned at the API versions <1.22. So, it seems like it's supported for very long time and all drivers working on officially supported platforms should have this feature.
Best wishes,
Anna

@avbelova I was referring to NV12->YUY2, P010->YUY2 output CSC, MSDK reference only gives NV12 and BGRA output CSC in  the manual.

We have a detailed format table and are working on a way to share this table in GitHub.

We are preparing the document.

Hi @FurongZhang do you have any updates?


Update API url

Septa updated their api since the last time I checked, the new api docs are @ https://www3.septa.org/ and the endpoints have changed too. Looks like the old urls still work, but I don't want to count on that for too long.
Take a look and change the base url in project. Should be a drop in replacement if I had to take a guess.
That said, it doesn't look like it has an endpoint to track a train by it's number. but the /TrainView/ does produce a list of all regional trains.

Nvm I lied about the train tracking endpoint, there's /RRSchedules/ that essentially does the same thing


Blog page doc

Fixes #84

This was an example PR from a short git workshop in Hacktoberfest 2019.


add: test script to get response from the api when the image is encoded

- It's an example to address issue #1120

Tickets
https://github.com/serengil/deepface/issues/XXX
What has been done
With this PR, ...
How to test
make lint && make test

I added a test script to address the issue regarding the sending of an image using base64 encoding

Thank you for your contribution, but I think this should not be put into the source.
API has already its unit tests - https://github.com/serengil/deepface/blob/master/tests/test_api.py

yeah. you're right.
But I think you haven't written any tests here to test the encoded images as input. Do you think it'd be suitable to add such a test ?

@rezakarbasi yes, please


Exporter/Stackdriver: Expose unregister API.

Fixes https://github.com/census-instrumentation/opencensus-java/issues/1917.

Why do you need this API?

See #1917 for background:

I'd like to be able to gracefully shutdown the application, but the lack of StackdriverStatsExporter#unregister prevents this. It's also not possible to use in a container environment as this would fail on repeat deploy.

Ahh - apologies, missed that, thanks!


chore: New Versioning Pattern

Description
Updates the versioning pattern. The new interfaces in pco-license and registry define all the external functions that the diamond will have, as well as the different versions. If a facet has multiple versions, the latest version will have its external functions defined in one interface, with each version also having their own.
Checklist:

[x] My commit message follows the Conventional Commits specification
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have tested my code
[x] My changes generate no new warnings
[x] My PR is rebased off the most recent main or appropriate feature branch
[x] My PR is opened against the main or appropriate feature branch

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

27 of 75 (36.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-10.2%) to 88.071%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

contracts/registry/facets/GeoWebParcelFacet.sol
3
4
75.0%

contracts/registry/facets/test/PCOLicenseClaimerFacet.test.sol
0
8
0.0%

contracts/beneficiary/test/BeneficiarySuperApp.test.sol
0
11
0.0%

contracts/registry/facets/test/GeoWebParcelFacet.test.sol
3
31
9.68%

Totals

Change from base Build<PHONE_NUMBER>:
-10.2%

Covered Lines:
627

Relevant Lines:
725

💛  - Coveralls


Better sorting of crafts

Previously, the "best" crafts would display the best crafts by net profit per station level. That doesn't make a lot of sense, so now it displays the two best crafts per station.
Also improved it so that  sorting the table by profit/hour will show the best crafts by profit/hour. The table can also be sorted by the other columns to see the best crafts according to other measures.
https://preview.tarkov-dev.pages.dev/

.deploy to development

.deploy

.deploy


[feature] #1827: Parametrize hashes

TBD

Description of the Change

Issue

Benefits

Possible Drawbacks

Usage Examples or Tests [optional]

Alternate Designs [optional]

Closed temporarily as issue is difficult to resolve directly.


Extension function alter

This PR introduces an extension function alter(..) for TransactionalEntity-s.
alter(..), just like her sister update(..) allows the user to change properties of the entity's builder without the hassle of calling Java-style setter methods.
Unlike update(..), alter does not return any value, allowing it to be used as an expression body in handler methods. Example:
internal class DailyReport : Projection<..> {

    @Subscribe
    internal fun on(e: WorkLogged) = alter {
        workMinutes += e.minutes
    }
}

It's recommended to keep the call to alter(..) at the same line with the method signature, so that the whole construction resembles a regular method.

@alexander-yevsyukov, PTAL.


WIP: ENH: Add Experimental::SquareImageNeighborhoodShape

Added the function GenerateSquareImageNeighborhoodOffsets(size_t radius)
and the class SquareImageNeighborhoodShape to the Experimental
namespace, to allow specifying more explicitly and more efficiently that
the neighborhood shape should be square (or n-dimensional hypercubic).
Replaced the more general GenerateRectangularImageNeighborhoodOffsets
by the newly added GenerateSquareImageNeighborhoodOffsets in
MeanImageFunction, SumOfSquaresImageFunction, and
ConfidenceConnectedImageFilter

Thanks for your feedback, @dzenanz
First of all, I don't feel too attached to this pull request, I wouldn't feel too bad about a rejection. However...

It appears that the fast majority of neighborhood shapes are in practice square-shaped (or cubic-shaped in 3-D). I would like to have it as easy as possible for the majority.
It seems clearer to me, when you want a square shape, that you can say so explicitly (instead of by saying you want a rectangular shape of size N x N).
The proposed SquareImageNeighborhoodShape might serve as a simple example for others, of how to write an ITK neighborhood shape class.

What do you think? 👍 or 👎   😃

OK, I can give it another 24 hours, but tomorrow I think I'll just close and abandon this pull request. As it seems that nobody really likes the little square (or cubic) neighborhood shape utility that I'm proposing here... 😢

OK: @jhlegarreta like the idea (thanks Jon  😄 ) but @dzenanz does not, and the pull request does not seem to have a priority to anyone else, so I give it up for now...  Close without merge!


[PDI-14606] Salesforce Input default field format (yyyy-MM-dd'T'HH:mm:ss'.000'Z) fails for Date/Time fields

SimpleDateFormat class did't support ISO-8601 fully in java-6, but in java 7 new date-patterns were added, and ISO-8601 support became possible.
The purpouse of this PR was to provide the ISO-8601 support, and the code for helping SimpleDateFormat (in java 6) to provide ISO-8601 support was to removed.
Since it is removed, we need to change the pattern to java-7 style.
Here is the test  for parsing this date pattern with SimpleDateFormat.
@akhayrutdinov, @brosander, review it please.

Build Commands:
Build:

rm -r ~/.kettle; rm -r ~/.pentaho; echo removed hidden folders
ant -Dpentaho.coding.standards.version=1.0.4 -Djunit.jvmargs=-XX:PermSize=256m -f plugins/salesforce/build.xml clean-all resolve jacoco jacoco-integration checkstyle publish-local

Cleanup:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
plugins/salesforce/src/org/pentaho/di/ui/trans/steps/salesforceinput/SalesforceInputDialog.java

Build Commands:
Build:

rm -r ~/.kettle; rm -r ~/.pentaho; echo removed hidden folders
ant -Dpentaho.coding.standards.version=1.0.4 -Djunit.jvmargs=-XX:PermSize=256m -f plugins/salesforce/build.xml clean-all resolve jacoco jacoco-integration checkstyle publish-local

Cleanup:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
plugins/salesforce/src/org/pentaho/di/ui/trans/steps/salesforceinput/SalesforceInputDialog.java

good to go here. @akhayrutdinov can you review, please

@mtseu, @kolinus, review it please.

For tracking:
Please, pay attention: transformations with old date - pattern (in java 6 style), will be broken. Newly created transformations (with this pattern) will work as exptected (it's the consequence of this commit)
According to Pedro:
Well, if there are transformations that use that date pattern, they are not working right now, right ?
So, I think we should go ahead and fix the default date pattern as this is breaking not only existing transformations as new transformations.

@mtseu , could you please review it?


[Question] Is disk image and iOS version have to be the same

In readme.md
"Go here and download the ZIP corresponding to the closest iOS version to the target device. Unzip the download and you should get DeveloperDiskImage.dmg and DeveloperDiskImage.dmg.signature."
I installed Jitterbug app on iPhone with iOS 14.6 and try to start the UTM app on iPad with iOS 14.6. An error was reported: Unable to mount
The latest disk image version available is iOS 14.5

I do not think the version mismatch is the cause of your problem. I was able to mount 14.5 DeveloperDiskImage on iPadOS 14.6. If you want to be sure, though, you can get 14.6 disk image from here.

In readme.md
"Go here and download the ZIP corresponding to the closest iOS version to the target device. Unzip the download and you should get DeveloperDiskImage.dmg and DeveloperDiskImage.dmg.signature."
I installed Jitterbug app on iPhone with iOS 14.6 and try to start the UTM app on iPad with iOS 14.6. An error was reported: Unable to mount
The latest disk image version available is iOS 14.5

In most cases it has to match but not every version of ios updates the DMG so that's why for example 14.5 dmg works on 14.6. there are cases like 14.4 dmg works on the increment changes also.

I have the same issue. But I can open the app in this way:
1.Close utm and jitterbug.
2.Open jitterbug on iphone, launch utm via jitterbug.
3.Utm will automatically start on ipad, but vm will crash if you open it.
4.Reopen utm on ipad, it will show warning message, just ignore.
5.launch utm via jitterbug on iphone.
6.Start vm via utm on ipad, it works!

Problem solved


Issue with profiles.yml, need to make the project profile independent

profile: 'google_ads_houseware' should be commented out. We have custom profiles we use for different use-cases

Resolved


Separate code of each plugin into separated repositories

Hi @robbyrussell,
I would like to propose to put the code of each plugin inside separated repositories. By doing so we can have a more fine issue reporting and make it clear where to find them. Also, the main reason for me is that some tools like Antibody don't support importing plugins from this repo, so by splitting the plugin's code into repositories we start to support Antibody.
I gave it a try by splitting each plugin into a single repository, keeping the history, and it worked. Now I just want to align with your expectations and if I can do such thing, if yes, I'm planning to contact either @nicoulaj (@zsh-users) or @desyncr (@zsh-plugins) to use their organizations as a place for the plugins.
What do you think about that?

I think it is a great idea, because you can easily create pull requests/issues for each plugin, but on the other hand I can imagine its kind of hard to get this working all together.

I don't think it gonna be hard because what oh-my-zsh does is reading the $plugins array and loading each one of them from the plugins folder. With the current approach, we get all the plugins and load only what we want. In this new approach, you just need to clone the plugin's repo and load it by adding to the plugins array as before, making each plugin optional.
One possible difficulty maybe comes when trying to have the same setup as now by installing all the plugins with the latest code, but for that, we can create a script for that. Do you think is really necessary to install all of them in a new setup?

Yeah sounds good an easy. And for the Plugins you need some kind of mapping table like pluginA="git@github.com:username/pluginA.git"
So zsh knows which plugins to clone and how to update.
So the initial setup will take some time, but the updates, will be easy and faster.

Maybe the use of submodules is even a better approach. I'm not so familiar with these submodules, but it sounds like it is quite suitable.

AFAIK submodules are used to keep subprojects in a manner more towards customization. I've played with them and one limitation is that it will lock the module at a specific commit, making the updating process a little bit more bureaucratic. We would have to update the submodule and push it to the upstream, on the other hand, this makes the updates management more controllable because we could release updates with fixed versions of the plugins.
Although I don't think this would bring many advantages since oh-my-zsh  is mainly a configuration management tool and not a preset of plugins. To overcome the difficulties of updating, we could assume the folders in plugins are git repositories, so for updating it, we pull and for checking we fetch. This way we don't have the need to create a mapping table since all of them are repositories, the mapping would only be needed if we pretend to have the current plugins for a raw installation. That is the simplest approach that I can think of.

Just a note that it is now possible to load a built-in oh-my-zsh plugin with antibody by using folder annotation. For example:
robbyrussell/oh-my-zsh folder:plugins/sudo

This is not something we're going to be working on in the near future.


Quotation marks in subject lines are entity-encoded

If you try to use a subject line that contains a quotation mark it is entity-encoded.
E.g. $mail->setSubject("Amy O'Neil"); becomes
Subject: Amy O'Neil in the resulting email message.
setSubject() runs the subject string through filterOther().
filterOther() uses FILTER_SANITIZE_STRING which will encode quotes by default (as it's meant for sanitizing HTML, not plain text).
A quick fix would be to add the FILTER_FLAG_NO_ENCODE_QUOTES flag as you've done in filterName()
That doesn't necessarily fix the underlying problem of filtering subject lines as if they are HTML though. E.g. you still can't have as a subject line anything that looks like an HTML tag as it will be stripped.

Fixed on November 15th, 2015


Memory leak

OIDNDenoiser VRAM Leak Issue
The OIDNDenoiser implementation consistently results in a major VRAM leak due to improper resource management during the destruction of the OIDNDenoiser object.
EDIT: This issue is specific to OIDN; OptiX doesn't show a VRAM leak.
Observation
Use the test the code below and observe the VRAM leak.
using UnityEngine;

public class OIDNVramLeak : MonoBehaviour {

    private UnityDenoiserPlugin.DenoiserPluginWrapper OIDNDenoiser;

    private void Start() {
        CreateDenoiser();
    }

    private void Update() {
        if (Input.GetKeyDown(KeyCode.Space)) {
            DestroyDenoiser();
            CreateDenoiser();
        }
    }

    private void CreateDenoiser() {
        UnityDenoiserPlugin.DenoiserConfig cfg = new UnityDenoiserPlugin.DenoiserConfig() {
            imageWidth = 2048,
            imageHeight = 2048,
            guideAlbedo = 1,
            guideNormal = 1,
            temporalMode = 0,
            cleanAux = 1,
            prefilterAux = 0
        };
        OIDNDenoiser = new UnityDenoiserPlugin.DenoiserPluginWrapper(UnityDenoiserPlugin.DenoiserType.OIDN, cfg);
        Debug.Log("OIDNDenoiser created.");
    }

    private void DestroyDenoiser() {
        if (OIDNDenoiser != null) {
            OIDNDenoiser.Dispose();
            OIDNDenoiser = null;
            Debug.Log("OIDNDenoiser destroyed.");
        }
    }
}

Hi, thank you for reporting the memory leak issue.  There's a known memory leak with the OIDN denoiser that was fixed in commit 718fe52. In the meantime, I will add more code to detect memory leaks.


Android 10 - requestLocationPermission always rejected (Permission denied), even when granted

Hi, thanks for your work - first of all!
I'm encountering an issue on Android 10 (OxygenOS 10.0.9). Calling 'requestLocationPermission' causes the permission dialog to open initially, but regardless of the user's choice, the Promise is always being rejected and returns "Permission denied".
If the user denies location permission, calling 'requestLocationPermission' again will display the popup a second time, offering the "Deny & don't ask again" option - however, granting permission either on the first or the second time will cause any subsequent attempts of calling 'requestLocationPermission' to be immediately rejected without the dialog being shown.
  Don't say that. This solved my problem, so thanks.


Add support for stable and beta Chrome releases

Kicked off with: @gary-rafferty
Description
We are now running performance tests against Chrome's stable and beta releases.
This PR differentiates between the two at initialization, and handles accordingly.
Also adds some convenience rake tasks rather than relying on rake arguments.
Deliverables

[ ] Gem can handle Chrome beta and Chrome release

👍


feat: #11 Add typeorm as a peer dependency

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remove -cli postfix from the cli binary name

I've brought this up a few times but only now got around to submitting a PR. This might be a bit controversial but imo this makes the cli more usable. Opinions welcome :)

Changelog check is checking the main changelog only, what should I do?

+1000 to this.
However, keeping the probe-rs-cli crate name means the user will still have to run cargo install probe-rs-cli I think? It might be confusing.

However, keeping the probe-rs-cli crate name means the user will still have to run cargo install probe-rs-cli I think? It might be confusing.

I think the only way to avoid that would be to fold the cli into probe-rs itself. I went with the least intrusive option as a start but happy to go the other way if we think that's better.

I went with the least intrusive option as a start but happy to go the other way if we think that's better.

Yeah, this solution was discussed a while back, everyone's fine with it.

Hmm, so I just gave this a try, not sure it's going to be possible without some major refactoring. The cli depended on probe-rs-cli-util which depends on probe-rs. Moving the cli into probe-rs creates a circular dep issue. This will also be the case for the gdb server too. I probably don't have time to do all that (lot of work involved in moving across tests, CI, readme, deps etc).
What about this PR in its current state, and then later we can revisit folding the cli stuff into probe-rs itself?

ouch, that's unfortunate :man_facepalming:
I'm going to give the full refactor a try, let's see...

Closing in favor of #1637


feat(vercel): ISR

Description (required)

Adds configuration reference for a new option being introduced in the next minor version.
Brief; detailed instructions would likely go on vercel docs instead.

Related issues & labels (optional)

https://github.com/withastro/astro/pull/9714

Thanks 😊

Quick q so I can prioritize, @lilnasy - I see you made the implementation PR into a draft last week. Would anything there that you might rework change this user-facing documentation, or is this content safe to review?  Thanks!

It is safe to review. The reason for making it draft was resolved soon after.

This is going to need all that good stuff from the other PR's changeset!

This is going to need all that good stuff from the other PR's changeset!

From what I've seen with other frameworks, vercel documents ISR in more depth on their site. SvelteKit, for example. We have an opportunity to keep the changes here minimal and link to Astro on Vercel when it updates.
How much should we exercise this opportunity?

We need to document the configuration options! our stuff: expiration etc.

Should VercelImageConfig have been documented as well?

My general rule is: anything that we built, literally anything you can configure or use that only we control, we should document.
If, for example (as I think was once the case with something in an adapter or middleware or something), we use/support someone else's entire options (we exactly mirror them), then a link to them would be sufficient.
I don't know what VercelImageConfig is, but if it is something we created, even if only in order to use their image service, then yes we should document it. BUT, for example, if all the config options are just mirrored from their service, that's where it's fine to link to them.
Does that help?

That helps, thanks!
I've documented VercelISRConfig.
I don't have the full context for VercelImageConfig but it is our config, and not present on docs. It can be a future PR, possibly by someone else.

@sarah11918 Thanks, this is much clearer now!


boards: arm: add support for NUCLEO-L433RC-P

Support the ST STM32 Nucleo-64 development board with
STM32L433RC SoC.
Tested samples: hello_world, blinky and button.
Signed-off-by: Matija Tudan<EMAIL_ADDRESS>

is this PR completing  #26309 ?

is this PR completing  #26309 ?

is this PR completing #26309 ?
I didn't notice that PR before... But yes, it seems those are the same boards.

is this PR completing #26309 ?
I didn't notice that PR before... But yes, it seems those are the same boards.


Bug when using none buttons

In Skjemadetaljer I can choose that the form should have no buttons. But forms with this option only display the header and  no content in the form.
To reproduce:

Create a new form in Skjemabyggeren
In Skjemadetaljer, select "Ingen" in "Visning av knapperad"
Add some elements, e.g. a Beskjed
Upload to test environment and open it in your brower
Only the title of the form is displayed

Bug in Helsenorge skjemautfyller


Lables now can't have spaces

From the first whitespace to the end of line the text must be ignored.

So I miss the colon in <token: LabelIdentifier> [^\n\s]*
I'm going to merge this, but please, could you add the colon?
Thank you!


Android: launch by URI now accepts intent flags, which for example ca…

…n be used to open the target in its own context with FLAG_ACTIVITY_NEW_TASK. Otherwise the activity starts within the calling app.

Very nice, we needed this exactly. Is there a timeline for merge? Documentation is missing, we can add some if necessary.

I'm adding this in today.


Whois results in double info listing in new query

I am running 9b065608990f5673cfb82d45d1ffb44189500cd0 plus #1631 merged into a temporary branch. Clicking on someone's name, or using /whois on them, results in a fresh query with two listings of the whois info. I traced the server<->znc and client<->server traffic:
S-Z: WHOIS MiniDigger MiniDigger
Z-S: :orwell.freenode.net 311 kode54 MiniDigger ~MiniDigge some.random.host.cause.default.is.boring.minidigger.me * :Martin
Z-S: :orwell.freenode.net 319 kode54 MiniDigger :#thelounge
Z-S: :orwell.freenode.net 312 kode54 MiniDigger orwell.freenode.net :Amsterdam, NL, EU
Z-S: :orwell.freenode.net 330 kode54 MiniDigger MiniDigger :is logged in as
Z-S: :orwell.freenode.net 318 kode54 MiniDigger :End of /WHOIS list.
S-C: ["join",{"shouldOpen":true,"network":9,"chan":{"type":"query","name":"MiniDigger","id":229,"messages":[],"key":"","topic":"","firstUnread":0,"unread":0,"highlight":0,"users":[]}}]
S-C: ["msg",{"chan":229,"msg":{"type":"whois","whois":{"nick":"MiniDigger","user":"~MiniDigge","host":"some.random.host.cause.default.is.boring.minidigger.me","real_name":"Martin","channels":"#thelounge","server":"orwell.freenode.net","server_info":"Amsterdam, NL, EU","idle":"2995","logon":"1506486847","account":"MiniDigger","idleTime":1508627362816},"from":"","id":359548,"previews":[],"text":"","self":false,"time":"2017-10-21T23:59:17.816Z"}}]
S-C: ["open",229]
S-C: ["more",{"chan":229,"messages":[{"type":"whois","whois":{"nick":"MiniDigger","user":"~MiniDigge","host":"some.random.host.cause.default.is.boring.minidigger.me","real_name":"Martin","channels":"#thelounge","server":"orwell.freenode.net","server_info":"Amsterdam, NL, EU","idle":"2995","logon":"1506486847","account":"MiniDigger","idleTime":1508627362816},"from":"","id":359548,"previews":[],"text":"","self":false,"time":"2017-10-21T23:59:17.816Z"}]}]
Somehow it's receiving both the initial msg of the query contents, as well as triggering a "more" data request?

That's #1631 causing it, comment there instead.


Domoticz show loading

Issue description
I have 2 domoticz in the same version V2020.2
However I notice a difference between the 2 versions.
For one the version is written V2020.2 in large characters and the other 2020.2 (without the V) and in very small characters.
My problem is that for version V2020.2, the plugin does not work with Chrome (only firefox).
The version 2020.2 works well with Chrome.
Here are the differences :
-Domoticz1 version 2020.2    Rpi2<EMAIL_ADDRESS>     Works well
-Domoticz2 version V2020.2  Rpi3<EMAIL_ADDRESS>   Works only with firefox (Other show Loading)
Additional information
Zigbee2MQTT version:
<EMAIL_ADDRESS> and<EMAIL_ADDRESS>Python version:  Python 3.7.3 for the both
Domoticz version:  2020.2  and V2020.2
Plugin version: ???

Rectification,
my domoticz2 appears with the same size of character with firefox.
Here are the differences :
-Domoticz1 version 2020.2 Rpi2<EMAIL_ADDRESS>     Works well
-Domoticz2 version 2020.2 Rpi3<EMAIL_ADDRESS>👍
- With Firefox works well (same presentation as domoticz1)
- With Chrome plugin does not work well (the version is written V2020.2 in large characters)

Do you see any errors in browser console?

There is no error in browser, however the plugin interface does not work. (Just Loading...)
I just noticed that the problem is related to one of my Rpi3 b +.
When I move the SD card from one Rpi that works on the 2nd, then it's not works.
I don't understand why one Rpi works and the other doesn't.
And why does it work with Firefox and not chrome ?
And it's the same SD card (with Domoticz and Zigbee plugin).
Maybe it is because I did the complete installation from the first (however the device works on the 2nd).

I was wrong.
I just noticed that my 1st Rpi3B+ is an Rpi3B.
It therefore seems that the plugin interface does not work on the RPI3B.
Or may be you have to do the complete installation from the Rpi3B because I did it from the Rpi3B +

What Domoticz build are you using and how did you install it?


Allow installing newer versions of python-jose

Coverage remained the same at 92.544% when pulling 3aba3c6fe78587e4e7299a6dc52a608e34ef3e87 on illia-v:python-jose<3.1.0 into 71a054af92a45e2a8b5d7c4fb435841949e1895c on spulec:master.

Closed in favor of https://github.com/spulec/moto/pull/1927


TODO: Do not merge yet:  Add Dockerfile + CognitiveServices and some doc fixes

These commits were made from the Bot Hackathon on the same day of the commits.
There some changes to this repo that may/may not be part of your goal for the repo, but I wanted to use this as a starting point to discuss what you'd like to merge.
Are all/any of these relevant to you and this repo?
Cheers,
-Tommy.

After discussions, I'm closing and will continue to keep the changes localized to my personal repo.


CeskaTelevize

Site has changed, for example this video is not working now: http://www.ceskatelevize.cz/porady/11914971217-mrnouskove/218384604930073/

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.04.25. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.04.25

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
"youtube-dl(2).exe" -v http://www.ceskatelevize.cz/porady/11914971217-mrnouskove/218384604930073/
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'http://www.ceskatelevize.cz/porady/11914971217-mrnouskove/218384604930073/']
[debug] Encodings: locale cp1250, fs mbcs, out cp852, pref cp1250
[debug] youtube-dl version 2018.04.25
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.16299
[debug] exe versions: ffmpeg 3.3.2, ffprobe 3.3.2
[debug] Proxy map: {}
[CeskaTelevizePorady]<PHONE_NUMBER>30073: Downloading webpage
ERROR: Unable to extract iframe player url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\build\youtube_dl\YoutubeDL.py", line 789, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\build\youtube_dl\extractor\common.py", line 440, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\build\youtube_dl\extractor\ceskatelevize.py", line 283, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\build\youtube_dl\extractor\common.py", line 808, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract iframe player url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description of your issue, suggested solution and other information
Site has changed, for example this video is not working now: http://www.ceskatelevize.cz/porady/11914971217-mrnouskove/218384604930073/

This URL doesn’t work anymore, and the situation has probably changed meanwhile so much, that it doesn’t make much sense to work on this issue (I cannot reproduce THIS issue anymore).
This ticket should be closed IMHO.


DO NOT MERGE just kicking CI

Our CI is very broken right now. :-)
PRs are welcome to improve it, but please keep them small so it's easy to understand the changes.

@Timer I know, working on it. :) I could've closed this PR myself, sorry about taking your time.


Downgrade metadata-json-lint for Ruby 1.9

Without this change, tests fail. metadata-json-lint is not supported on
Ruby 1.9, but versions of the gem prior to 1.2.0 happen to work for our
purposes. This downgrades metadata-json-lint when using Ruby 1.9.

I'm not familiar with modulesync, so please let me know if I've goofed this up.
(Really, I'd like to ditch Ruby 1.9 "support").

Can someone merge this? It seems to be making all PRs fail in Travis. :(


SDIT-1136 Attendance reconciliation endpoint

Codecov Report
Attention: 16 lines in your changes are missing coverage. Please review.

Files
Coverage Δ

...esmanagementapi/repository/AttendanceRepository.kt
100.00% <ø> (ø)

...anagementapi/resource/SynchronisationController.kt
33.33% <0.00%> (-6.67%)
:arrow_down:

...iesmanagementapi/service/SynchronisationService.kt
25.00% <12.50%> (-8.34%)
:arrow_down:

...ementapi/model/AttendanceReconciliationResponse.kt
0.00% <0.00%> (ø)

:loudspeaker: Thoughts on this report? Let us know!.


Update gotomeeting to <IP_ADDRESS>56

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

If anyone can figure out a cleaner way of installing this, I’m all for it. It was previously using a version-specific URL that was nevertheless labelled latest, which is obviously not ideal. The download URL seems pretty much arbitrary, and the process of downloading it normally is… bizarre to say the least.

I found http://support.citrixonline.com/en_US/Meeting/all_files/G2M010014 - that gives me a link of http://support.citrixonline.com/en_US/Meeting/downloaddocument/G2MD00250 for this latest version.

@adidalal That’s not a download link, unfortunately, just a redirect to a similarly messy one.

Probably only the ka338000000CjPXAA0 part needs to change.

@reitermarkus I just tried removing that parameter. Unfortunately, curl still downloads a useless webpage without it.

Probably only the ka338000000CjPXAA0 part needs to change

@Saklad5 This works. Unfortunately https://support.citrixonline.com/en_US/Meeting/all_files/G2M010014 is not stable, so it can't be used as an appcast.
cask 'gotomeeting' do
  version '<IP_ADDRESS>56,ka338000000CjPXAA0'
  sha256 '5722fb63a00c320a20d071f8cb00d2fe93a1c4f73cbf8b80602021a8f8fb82de'

  # support.citrixonline.com was verified as official when first introduced to the cask
  url "https://support.citrixonline.com/servlet/fileField?entityId=#{version.after_comma}&field=Content__Body__s"
  name 'GoToMeeting'
  homepage 'https://www.gotomeeting.com/'

  app 'GoToMeeting.app'
end


Rename log to debug.

To make space for using log for math.

LGTM

Why have a debug global at all rather than just see?

Debug is parallel to console.debug - it's a more descriptive word than
see...
On Aug 8, 2015 7:02 AM, "Weihang Fan"<EMAIL_ADDRESS>wrote:

Why have a debug global at all rather than just see?
—
Reply to this email directly or view it on GitHub
https://github.com/PencilCode/jquery-turtle/pull/95#issuecomment-128961469
.

See is still there. When the question of “see” came up, David said:
I really don't like the name "see" - it's been a source of confusion, and that's why it's currently "log”.
So we settled on “debug”. But feel free to use “see”.

On Aug 8, 2015, at 7:02 AM, Weihang Fan<EMAIL_ADDRESS>wrote:
Why have a debug global at all rather than just see?
—
Reply to this email directly or view it on GitHub https://github.com/PencilCode/jquery-turtle/pull/95#issuecomment-128961469.


Feature: Add native notifications support

Hello, awesome project 👏👏👏
I was able to add native OS notifications using https://github.com/mikaelbr/node-notifier

I'm thinking of that we can add this lib to project to have built in notification support simply by bridging.
I can make a PR for this if everything is okay with adding support for notifications, along side this, I have a lot of other ideas and things missing in this project like lack of support for printing for example...

Proton-native is based on libui, there is an issue for notifications there (https://github.com/andlabs/libui/issues/338), but it's not going to happen very soon (if ever): "I still don't know if this is in scope for libui."

printing for example

How should that work? What would you like to print?  (PDFs, images?)

@mischnic
I taught that libui is only for UI stuff and nothing more so we should consider adding other extra stuff ourselves.
please let me know if you decided to do it in proton native I have ideas for performance of this (electron app with 3000 daily active users).
for printing there are some several functionalities needed. like a method to return OS printers, a method to print with options like which printer, page size, adjusting margins and...
what we needed was printing an html document in electron which we could make a browser window put html into that with whatever css we wanted then print it, along side support for image printing, I can make a separate issue for printing.

I taught that libui is only for UI stuff and nothing more so we should consider adding other extra stuff ourselves.

I would say that notifications are part of the ui.

please let me know if you decided to do it in proton native I have ideas for performance of this (electron app with 3000 daily active users).

With my previous comment I wanted to say: it should be done this way because it won't happen in libui.
What keeps you from using that library together with proton-native as it is?

If there's no special thing that needs to be done on Proton Native to make it work, it's best if it's not merged. This leads to additional and additional libraries being merged just bloating the project. I can mention it in the documentation, but I'd rather not add another library.


Support cluster default pod security standards.

Is your feature request related to a problem? Please describe.
Some Kubernetes clusters are configured to apply a default pod security standards for any namespaces created. This can be a problem when using Educates to install contour as depending on configuration it can fail as a result.
 message: '(combined from similar events): Error creating: pods "envoy-qqlnh" is forbidden:
   violates PodSecurity "baseline:latest": hostPort (container "envoy" uses hostPorts
   443, 80)'

To deal with this the projectcontour namespace would need to have labels:
     pod-security.kubernetes.io/audit: privileged
     pod-security.kubernetes.io/enforce: privileged
     pod-security.kubernetes.io/warn: privileged

Describe the solution you'd like
Look at way of applying these labels automatically, either always, or as an option.
Right now no choice but to disable default pod security standards for the cluster as a whole.
Describe alternatives you've considered
No response
Additional information
No response

Note that may need to also look at how such a default pod security standard may apply to workshop sessions when Kyverno is used as cluster security policy engine. Although, if there is a default cluster pod security standard applied, may be forced to deploy Educates using pod security standards for cluster security instead of Kyverno, with Kyverno only used to workshop/session security.

For Contour case, need to see if hostPorts was needed in this case as was using LoadBalancer, so may not have.


fix: error thrown on large files

Codeium.nvim throws error on large files because curl fails to do the request when using --data-raw with:
E2BIG: argument list too long
This fixes the problem by writing the data to a temporary file and passing it to curl as a binary file with --data-binary.
Fixes #20

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


Gradient as pattern's colour

Hey,
I'd like to know whether it is possible to give a gradient as the pattern's colour?

@m4tty-d This is definitely possible but will require a few changes to how the background colour is applied to the pattern. Are you interested/able to raise a PR for this?


product icons from theme extensions

Can you add support product icons from theme extensions?
Your latest experiment with product icons will raise another wave of dissatisfied (#78752).
But restore old icons we do not have the opportunity :/
thank you.

Marking as duplicate of #78752 then.

@octref How feature suggestion can be duplicate of decision complaint?

@SailorMax see https://github.com/microsoft/vscode/issues/26130 for theming product icons


Failed to scrape content with empty command match and no metrics output

Hello team,
Did anyone run into the same issue? could anyone share some insight how to troubleshoot this?
Thank you all in advance!
Error Message / Output:
Only aruba_up gauge is scraped after 15 minutes
aruba_exporter metrics output.txt
Many empty command match:, ran commands every 5 minutes or so, and finally timeout.
aruba_exporter service status.txt
Config file
./aruba_exporter -config.file=config.yaml

level: debug
timeout: 180
batch_size: 20000
username: operator
password: P@ssw0rd!
#key_file: /path/to/key
devices:

host: <IP_ADDRESS>

features:
system: true
environment: false
interfaces: true
optics: false
routes: false
wireless: false

Hardware-wise

Vendor files are used for my offline system using cmd go mod vendor & go build

aruba-exporter-vendor.zip

Tested on aos-cx OVA and Aruba 8100 switch, same issues

Update

Tried re-creating vendor mod after mod clean -cache and -modcache, no luck

Tried Go version 1.22.3 & 1.19, no luck

The go mod module.txt looks good to me.

Tried copying $GOPATH/pkg/mod (mod cache) to offline server and run go run main.go -mod=mod/vendor, no luck with same undefined error message

Tried built aruba_exporter with internet access, got the same file size and same behaviour

Tried go run -mod=vendor ./main.go -config.file=config.yml on both online & offline environment, with or without -mod flag. Here's the error message/output:

# command-line-arguments
./main.go:75:17: undefined: defvicesForConfig
./main.go:156:7: undefined: newArubaCollector

Is it related to the issues?

Tried re-creating vendor mod after mod clean -cache and -modcache, no luck

Was your main.go modified?  In the source, line 75 is whitespace and line 76 is devices, err = devicesForConfig(c) but it looks like there is a typo in yours, defvicesForConfig.
newArubaCollector comes from aruba_collector.go

Thank you so much for the response.
I revisited the issue today, the main.go is not modified, sorry for the typo:
# command-line-arguments
./main.go:75:17: undefined: devicesForConfig
./main.go:156:7: undefined: newArubaCollector

I downloaded and re-built the aruba_exporter again.
It has the same issue with empty command match.
From the Aruba 8100 log, I can see the operator user from my config.yml logged in and out when I go to http://:9909/metrics


SplitSpace simplification results in futhark-opencl error (Cannot allocate memory in kernel.)

When turning on both my new segmented redomap, and inlining SplitSpace, some of my test programs result in errors when compiled with futhark-opencl
segsum-comm.fut and real-nocomm.fut both work fine, but there are problems with for example ex1-comm.fut and ex1-nocomm.fut. So it seems like there is only problems when there is output from the "map part" not only used by the reduce.
Error reported is Cannot allocate memory in kernel.

Rasmus Wriedt Larsen<EMAIL_ADDRESS>writes:

When turning on both my new segmented redomap, and inlining
SplitSpace, some of my test programs result in errors when compiled
with futhark-opencl
segsum-comm.fut and real-nocomm.fut both work fine, but there are
problems with for example ex1-comm.fut and ex1-nocomm.fut. So it
seems like there is only problems when there is output from the "map
part" not only used by the reduce.
Error reported is Cannot allocate memory in kernel.

Yes, that is why I sent you an email that we were going to do something
else.  That simplification rule is going anyway.  Continue using
SplitSpace, but do not use SplitArray.
--
\  Troels
/\ Henriksen


GE/Jasco ZW3008 and ZW4006 support double taps but configuration doesn't support

Checklist:

[ ] I am not using HomeAssistant. Or: a developer has told me to come here.
[ ] I am not using ZWaveJS2MQTT. Or: a developer has told me to come here.
[ ] I have checked the troubleshooting section and my problem is not described there.
[ ] I have read the changelog problem was not mentioned there.

Describe the bug
Double taps are supported in the hardware for ZW3008 and ZW4006, but the zwavejs json configuration is not set up to handle it.
Device information
Which device(s) is/are affected (make/model)?  GE/Jasco ZW3008 and ZW4006
What are the node IDs?
Did you change anything?

[ ] Yes: (please describe)
[X ] No

Did this use to work before?

[ ] Don't know, this is a new device
[ X] No, it never worked anywhere
[ ] Yes, in: (specify application with versions)

How are you using node-zwave-js

[ X] zwavejs2mqtt (latest) docker image
[ ] zwavejs2mqtt (dev) docker image
[ ] zwavejs2mqtt Manual Docker build

node-zwave-js branch:
zwavejs2mqtt branch:

[ ] ioBroker.zwave2 adapter
[ ] HomeAssistant version XYZ
[ ] Pkg
[ ] Manual Docker build

node-zwave-js branch:
zwavejs2mqtt branch:

[ ] Manually built (as described in the docs)
[ ] Other:

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '...'
Scroll down to '...'
See error

Additional context
Add any other context about the problem here.
Logfile:

@blhoward2 your turf :)

Not sure if this helps, but someone developed a configuration for smartthings for a similar model ZW3010 here:
https://github.com/mwav3/smartthingscode/blob/ccfd245160e67d04897dfc33219a33696507702a/devicetypes/mwav3/ge-46203-dimmer-switch.src/ge-46203-dimmer-switch.groovy

We have a compat flag to enable the double tap support. It just needs to be put in there.

Yeah, this is a 30 second change. We already support it on 25+ GE models. I'll handle it tonight.

Actually, the problem is just that we don't have files for these yet that I see. What specific models are you referring to? The zw designation is usually the base hardware. I need the product type and product id.

26932 / 26933 / ZW3008
and
26931 / ZW4006
Is what I see in zwavejs dashboard

Ok. They just haven't been updated yet to add the compat flag and metadata.

More info...
26932 / 26933 / ZW3008
product type: 18765
product ID: 12340
and
26931 / ZW4006
product type: 18765
product ID: 12338

@blhoward2 got any insight here?

That's how these work. It doesn't add central scene support. Please post the zwave log itself so we can see what's going through.

That does look like the event event gets passed through:

command_class_name": "Basic", "label": null, "property": "event", "property_name": "event"

Confirmed with the integration developers. As of now at least you need to watch for the event in an automation.

I am indeed using an automation to watch for the event, but the problem is that a single press and double press both produce the same "Basic" event.  Maybe my firmware is out-of-date?

That shouldn't happen. It should be sending a single press as a binary switch or a multi-level switch. Please open an issue and fill out the template. We'll need to see the zwave log of the interview, the switch event, and export of the node so we can see what CCs are setup here.

https://github.com/zwave-js/node-zwave-js/issues/2042


Why not replacing bash.ico?

Why do you need to use archlinux.ico instead of replacing the existing bash.ico?
The thing is other programs, such as ConEmu, uses that as icon for bash.

Didn't think about that. That's changing in .6, I've assigned it to the project tracker.
Off topic:
About ConEmu: My experience with ConEmu + the WSL "bash.exe" wrapper was awful so far. It constantly misplaces the cursor and occasionally it doesn't even register arrow keys (for navigating through the history). The bash.exe actually has a routine (use IDA to look for it), that checks if it's hosted by another console. If it is, it spits out a few null-bytes and terminates. MS actually confirmed that in the BoW tracker. Dunno if that causes these glitches in ConEmu, but they have to do some work anyway. I've since switched to using the standard command prompt and tmux, or, on machines where I have Xming installed, terminator.

Thank you!
I was also using cmd + tmux, but switched to "have everything in one place". However I can confirm your feedbacks about ConEnu :-(

Done in private dev branch.

This wasn't working in the fresh install I just tried. After install I was still seeing the Ubuntu icon.


Add queried attribute to improve system printing

Add a queried attribute to System, which will keep track of whether
a particular system has been queried during the run. This attribute will
later be used to provide a more informative output in the publisher.
This mechanism overlaps with the printing mode. I decided to leave them
both for the moment in case we later find another use for the simple
printing mode, otherwise we can remove it.
Solves OSPCRE-393

Everything looks good.


Handle self-signed SSL server certificates

Currently ddm-tool stumbles when trying to connect to a server which uses SSL but has only a self-signed ("dummy") certificate installed. This is because node.js doesn't automatically accept self-signed certificates but throws a DEPTH_ZERO_SELF_SIGNED_CERT error in this case.
For development environments however it is kind of typical to use "dummy" certificates for DEV or QA machines being available only inside the local corporate network. Thus ddm-tool should be able to handle this situation gracefully.

Just pushed version 0.8.6 which adds the setting allowSelfSignedCerts (bool).
Any new created projects will have this setting in its settings file (default set to false). To add the setting to a previously created projects, edit the json file of the project in ~/.ddmtool/config/projects and just add the line allowSelfSignedCerts: true.
Now the million dollar question: Does it work?

Works fine.  Thanks for the patch!

By the way....I spent over 6 hours trying to get a CA certificate loaded in NPM in the .npmrc file before I read the fine README.md file wish indicated this property settings.  My CA is self-signed, not the web site certificate...so...the self-sign was "in the chain.  Argh...teach me to read...lesson learned....