this pull request is an updated this resolves the issue #48090

Checklist:

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #48090

this solves the issue typo in pull request

This looks just like your pull request that was rejected. Please don't submit the same PR twice. Please review our contribution guidelines.


Remove unnecessary metrics from whitelist

The removed metrics are either no longer produced (payload_errors)
or are or should be redundant with existing metrics (up and ALERTS
for image registry and etcd).
Depends on https://github.com/openshift/telemeter/pull/197 and
https://github.com/openshift/telemeter/pull/198

This should still be done in the Telemeter repository, right @squat? Then we can bump the dependency in this repo like we did in https://github.com/observatorium/configuration/pull/12.

Yes @metalmatze that is the correct flow. This changed depended on openshift/telemeter#197 and openshift/telemeter#197 so it didn't bump jsonnet yet. Now that those pull requests are in, we can update this PR to actually be a jsonent + generation change. I'll push to this branch directly.


Energy sensor

Description:
Sometimes a device only provides power information (measured in W or kW).
This sensor does an integration of Power and time in order to measure energy consumed in kWh (standard energy measurement unit)
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#8012
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: energy
    name: Energy Consumed
    source: sensor.measured_power

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

@dgomes any chance for this also as a water sensor? I have it with liter-counter without being able to get sum value. It would just do the job with liters instead of W/kWh :-)

@andriej liter is a volume though, what you suggest would make sense for a flowrate sensor. Or maybe I misunderstand?

@emontnemery I have flowmeter installed and based on it's reports I'd love to see this day/week/month water usage baded on sum - just like in kWh sensor :) I know it's not related to this PR, just an idea for improvement

Aha, I see. Maybe #19718 does what you want?

@emontnemery could be, yeah. Can't wait to test it

@andriej as @emontnemery pointed out, you want #19718 :)
These two PR are related, but separate. You will need to have energy (this PR) in order to use the Utility Meter (#19718). Since many devices already provide energy readings I've split the two and made the utility meter more general (supporting water, gas, heating, etc)

We should create a generic sensor, measure_calc or so they have preset for volume or kw/h. I don't like to see too many different platforms they do the same at the end.

Volume could be a function of time if integrating a flow rate sensor (volume/time) as discussed yesterday. It was mentioned the sensor reports liters though which is volume, not flow rate.

@emontnemery ok I see the case for a flow meter providing L/h.
Are you aware of any flow meter sensor that provides L/h and not L? And I'm not referring to DIY stuff, because in that case both energy and volume would be better handled in the Arduino code.
I could rename the sensor "Integrator Sensor" and remove the energy specific stuff, but would like to avoid that if no practical use case exists as the name would be less obvious to users.

Are you aware of any flow meter sensor that provides L/h and not L

No, I'm not.


Per-module opt-ins

This https://github.com/bswck/lazy_importing/blob/d0051596efc638ee6abc9df6f2da038cc800784e/tests/tests.py#L30-L35
won't be necessary if we collect info about modules that enabled lazy importing and continue default behavior for other modules.

Instead of tracking module names, we can track names and check if they're in the stack.

Instead of tracking module names, we can track frames and check if they're in the stack.


Option fields in struct

This PR should fix issues with Option fields in struct.
If struct has Option and field is empty, or missing, it should be None.
If struct has Option and field is set it should be Some(value).

Thank you!

Pushed v0.1.1 to crates.io


Actions job to check whether Cargo.lock is up to date

In #1320 we found that Cargo.lock was outdated although this PR had nothing to do with Cargo.lock or even Rust code.
We should add a job to our actions workflow to check a PR for up-to-date Cargo.lock so that these things do not happen anymore (and we don't need PRs such as #1321).

One issue I see: If crates get updated but the dependabot PR has not landed yet, this might cause Cargo.lock to be outdated in a PR, although really it is not:
gitGraph
    commit

    branch feature-abc
    branch dependabot-update-xyz
    checkout dependabot-update-xyz

    commit id:"Update xyz"

    checkout feature-abc
    commit id:"Implement abc"

In above diagram, the actions job for "Implement abc" might fail because it does not yet include the "Update xyz" patch. Rebasing fixes this easily, of course. Is this acceptable or too much of a hassle?
I think it is acceptable. The "Implement abc" patch should not include dependency updates and rebasing is most of the time a no-brainer in this scenario. So once "Update xyz" has landed on "main", the author of "Implement abc" just needs to rebase and update their PR and that's it.

The implementation of such a action job might be error prone. Not sure how to do it properly.
Please discuss.

Maybe it is an option to run cargo build with the --locked flag?

After some testing and discussion it seems that cargo check --locked exhibits the wanted behaviour:

It does not touch existing fulfilled dependencies
If a dep. does not exist, it would try adding it (and fails due to --locked)
If a dep. does not have the required version in the .lock file it also fails

This should be enough to have it run whenever a Cargo.toml file changes in the repo.


fix(frontend) unable to retrieve config map error when trying to manage newly created resource pool (ce#180)

Closes https://github.com/portainer/portainer/issues/4629

It's good for me. I have committed it.

/azp run


add touch down mouse event, add responsive text, add vertical center'

thanks @JaxGeller

@jhil stop taking credit


[Feature]: JSONForm options for AddRow (blank form) or UpdateRow (populate from sourcedata), so blank forms can detect field types from source data

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
The JSONForm widget can automatically detect data types from the source data, and then set the form input types accordingly. This only works when the source data includes values though. If you want a blank form and provide an object with blank values, the widget assumes every field should be a string. Then every column has to be manually updated to set other types (number, date), which defeats the purpose of auto-generated forms.
This request is for a new property on the JSONForm, with options for AddRow or UpdateRow.

UpdateRow
Same as current behavior. The form is populated from the source data.

AddRow
Source data is used to determine field types only. Values are not populated in the form.

JS
expression which returns AddRow or UpdateRow

Why should this be worked on?
The JSONForm widget can already detect the type, but users aren't able to take advantage of this feature when they want a blank form. This would be especially useful for large tables with lots of columns that would otherwise need configured manually.
Additionally, this would allow using the same JSONForm for adding and for updating rows, if the property has a dynamic JS binding. This would simplify UI building, and could also be used to correct the current issues with the Generate a page from data feature, which is populating all AddRow forms with values from a previous row (Slack thread).

@GreenFlux can you please elaborate on how the above properties solve the problem, user will still need to define the types

@somangshu , the user would continue to define the data types as they do now, by providing JSON with values. The request here is that we add a new property in the JSONForm widget which tells it to use the source data for determining field types only, and not to use it to populate the form.

AddRow | Source data is used to determine field types only. Values are not populated in the form.

@GreenFlux if I am getting this right, this will also the dev to use the same form for add new & edit operation.
cc @dilippitchika @ashit-rath what do you say? Can you shed some light on the use case if thought of in the initial phase?

@somangshu , yes! The same form could easily be used for addRow or updateRow. Then you could use a JS condition to switch the form mode on the fly. Both versions would use the same source data to get the field types, but one would be blank.

I think add / update might be a little specific for the use case but we should definitely just add a property to fillDefaultValues: true/false

@Nikhil-Nandagopal , that works for me! The fillDefaultValues label would probably be more obvious and need less explanation as well. Either way, it solves the issue and allows devs to use the same form for adds and updates, as long as we also give it a JS binding to switch states programatically.

Stats

Stat
Values

Reach
1669

Effort (months)
0.75

Are you thinking there will be a switch or some indication in the Form Label to indicate whether user is Inserting or Updating a Row?

@pchambless , yes, a switch with a JS option, so that you can change modes programmatically and use the same form for adds and updates.

Nice!  I've had to build some goofy code that can be error prone to re-use the same form for inserts and updates.  This will be nice.

From: GreenFlux @.>
Sent: Monday, January 29, 2024 1:25 PM
To: appsmithorg/appsmith @.>
Cc: pchambless @.>; Mention @.>
Subject: Re: [appsmithorg/appsmith] [Feature]-[1112.67]:JSONForm options for AddRow (blank form) or UpdateRow (populate from sourcedata), so blank forms can detect field types from source data (Issue #21995)
@pchamblesshttps://github.com/pchambless , yes, a switch with a JS option, so that you can change modes programmatically and use the same form for adds and updates.
—
Reply to this email directly, view it on GitHubhttps://github.com/appsmithorg/appsmith/issues/21995#issuecomment-1915408581, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKWYZBI4C467IXUZR5DTQNDYQ7ZRJAVCNFSM6AAAAAAWQJYJMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJVGQYDQNJYGE.
You are receiving this because you were mentioned.Message ID: @.***>


ambiguous withCurrentDirectory

[ 9 of 15] Compiling System.Directory.Extra ( src/System/Directory/Extra.hs, dist/dist-sandbox-febc0bd9/build/System/Directory/Extra.o )

src/System/Directory/Extra.hs:7:5:
    Ambiguous occurrence ‘withCurrentDirectory’
    It could refer to either ‘System.Directory.Extra.withCurrentDirectory’,
                             defined at src/System/Directory/Extra.hs:28:1
                          or ‘System.Directory.withCurrentDirectory’,
                             imported from ‘System.Directory’ at src/System/Directory/Extra.hs:11:1-23

src/System/Directory/Extra.hs:7:5:
    Conflicting exports for ‘withCurrentDirectory’:
       ‘module System.Directory’ exports ‘System.Directory.withCurrentDirectory’
         imported from ‘System.Directory’ at src/System/Directory/Extra.hs:11:1-23
       ‘withCurrentDirectory’ exports ‘System.Directory.Extra.withCurrentDirectory’
         defined at src/System/Directory/Extra.hs:28:1

Due to System.Directory new version:
-- | Run an 'IO' action with the given working directory and restore the
-- original working directory afterwards, even if the given action fails due
-- to an exception.
--
-- The operation may fail with the same exceptions as 'getCurrentDirectory'
-- and 'setCurrentDirectory'.
--
-- @since <IP_ADDRESS>
--
withCurrentDirectory :: FilePath  -- ^ Directory to execute in
                     -> IO a      -- ^ Action to be executed
                     -> IO a
withCurrentDirectory dir action =
  bracket getCurrentDirectory setCurrentDirectory $ \ _ -> do
    setCurrentDirectory dir
    action

Thanks, needs an ifdef. I'm on holiday for 4 days but will sort it out on my return.

+1

I've just released version 1.3.1, which I think fixes the issue. Can someone who was encountering a problem please check it fixes it for them?

Fixed. Thanks.

Thanks for the confirmation!


Add prerelease build number to kbfsfuse

Whoops, I forgot Linux, apologies
@cjb @strib @maxtaco

cc: @oconnor663 !!

Could you give me a little more context for what's happening here? Were our ldflags for kbfs always broken, or was the decision to use a different variable for the prerelease build a recent change?

On kbfs, the variable is:
github.com/keybase/kbfs/libkbfs.PrereleaseBuild
In the client, it is:
github.com/keybase/client/go/libkb.PrereleaseBuild
I don't know what the history is, whether they use to be the same, but they aren't now.  Do you know why the KBFS code doesn't just reference PrereleaseBuild from the libkb code that's compiled into it? Have we just been doing the wrong thing forever in the Linux build?

@zanderz LGTM with the small comments above. Just for my understanding, what prompted you to fix this?

I believe it's so that the git hash can be added to the prerelease build number, though this could also be done with go generate or something.

Oh hmm. If KBFS wants to be reporting its own commit number, than the code here needs to be changed. Right now the build_number variable is going to refer to client's commit hash.

Yeah, should be the kbfs git commit hash.

@taruti first noticed that keybase status gave the kbfs version without the prerelease build number on Windows. I don't know who realized Linux was the same.
There, now I got this:
-ldflags_kbfs '-X github.com/keybase/kbfs/libkbfs.PrereleaseBuild=20160407211319+a245428'

LGTM with those nits. Thanks Steve!

Oops, my fault, this might've broken the Arch build, because of the way Arch tries to hack kbfs-beta in place of kbfs. I'll fix it.

Done. What have I become?


Fetchival not working with Safari

I suspect something inside a Promise is not working inside fetchival. Its is not working with Version 9.1.3, and no error messages are shown. The promise never gest fulfilled.
Code example bellow (work on Chrome and Firefox).
console.log('Console log works here');

fetchival(module.settings.endpoint(['user', apiKey])).get().then(function(response) {
    console.log("Iam here, but never works on safari");
    console.log(response);

});

Just to confirm what I'v said, i have replaced the fetchival Ajax code by a random library (https://github.com/yanatan16/nanoajax), and it works in Safari.

fetchival is base on window.fetch
// Expose fetch so that other polyfills can be used
  // Bind fetch to window to avoid TypeError: Illegal invocation
  fetchival.fetch = typeof fetch !== 'undefined' ? fetch.bind(window) : null

Safari is not support fetch

Hi @adadgio,
fetchival doesn't come with fetch polyfill, but you can use the following command and you should have everything you need ;)
npm install es6-promise whatwg-fetch --save # polyfills
npm install fetchival --save


Error with optional Arguments

Trying to run the new sampleMetrics API endpoint (endpoint number 34 in the CLI). I get the following error:
I entered 3 for the translation (OTUCode 2) and 300 for the fixed count. When I typed 1 in preparation for typing some sample IDs, I got the following error:

I'll include this in the next push.


ERROR : Failed to execute createObjectURL on chrome

Hi Chapin666,
This is the error we are facing when inspect the chrome browser for errors with  your nodejs group call server.
Uncaught TypeError: Failed to execute 'createObjectURL' on 'URL': No function was found that matched the signature provided.
at WebRtcPeerSendonly.WebRtcPeer.showLocalVideo (WebRtcPeer.js:304)
Please do let me know if you have any suggestions.
Thanks & Regards,
Sudhish
<EMAIL_ADDRESS>

This issue is related to the kurentu-utils library. It seems has been deprecated createObjectURL.
The bug is making this application not to work.

Go to server/static/bower.json, change the version of kurento-utils-js to 6.9.0, run bower install and you're set.


Fix FNV bug on 64-bit Windows (hashes being cut off to 32-bit)

I observed an issue in my app where many input values (of a nested record type) were being hashed to the same value, and eventually determined it was only happening on Windows. After tracing the problem back, I noticed Text values were always being hashed to 32-bit values, and that when I combined the hashes of (record, text), the input record hash (used as salt) was also being cut off to 32-bit.
That led me to these issues with fnv.c. It is using long which is 32 bits on 64-bit Windows. However Haskell Int, as well as WORD_SIZE_IN_BITS, are 64. So hashes in general are 64-bit, but then getting improperly cut off when they interact with this code, such as through Text and other byte array types. I also noticed the FNV_PRIME constant added recently was not actually being used; I assume it was supposed to replace the numeric constant in the function below.
I tested that this fixes the hash behavior in my app, but I have not tested on a 32-bit platform yet.

Merged  in #213


Store scan metadata to warehouse/cloud/scan result: soda scan --store-metadata

Is your feature request related to a problem? Please describe.
Store scan metadata in the warehouse. Like which queries/tests/metrics are executed and how long they took.
I would like to use this table to monitor Soda scans. (Yes, very meta, monitoring the monitor system.) We could provide a default scan definition that goes with this table. The use case I for see:

Get insights on how long certain tests take.
Get warnings if certain tests execute a lot longer than they did before.
Get insights about which tests fail most
Get warnings if there is an increase in test failures.

Describe the solution you'd like
When the user adds the flag --store-metadata (or something similar) to a soda scan, then we store the metadata of that particular run in a table in the database. The table would look something like:

run_timestamp
dataset
test name
execution time
result

2021-26-10T15:55:50
my_table
row count == 0
00:00:10
FAILED

Additional context

See dbt's `store_failures

part of this is handled by #543 which adds open telemetry to the scans.

@vijaykiran how do we proceed? Maybe describe what #543 does exactly, which metrics it yields, and adapt this issue accordingly?

@fakirAyoub good question, I've updated #543 to add a bit more info on how things are derived which should help answering your question and @JCZuurmond 's a bit better


Rename fixed timestep state and add a test

Objective
fixes #3234
Solution

rename bevy::core::State to LocalFixedTimestepState
add a test for FixedTimestep since I am already there

bors r+


UI: Fix action buttons view when public IP address is selected, in Network -> Public IP addresses page

Description
This PR fixes action buttons view when public IP address is selected, in Network -> Public IP addresses page
Fixes #8106

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI
[ ] test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
Before Fix =>

After Fix =>

How Has This Been Tested?
Accessed Network -> Public IP addresses page in UI, and selected public IP address & checked the action buttons.

How did you try to break this feature and the system with this change?

@blueorangutan ui

@sureshanaparti a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9288 (QA-JID-385)

@blueorangutan ui

@sureshanaparti a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9288 (QA-JID-387)

While the Public IP view looks okay now, other views may not be looking the same

Got it, thanks for testing @shwstppr


runtime error: invalid memory address or nil pointer dereference [signal 0xc0000005 code=0x0 addr=0x8 pc=0x1424390]

azqr version 0.31.1
Fails scan on different resources each time, so there is no consistency. In examples below, the scan  failed on Azure Event Hubs and App Services:
INF Scanning Container Registries in Resource Group rg_usea_prod
INFINF Scanning CosmosDB Databases in Resource Group rg_usea_prod
Scanning Virtual Machines in Resource Group rg_usea_prod
INF Scanning Postgre in Resource Group rg_usea_prod
Scanning Front Doors in Resource Group rg_usea_prod
Scanning MySQL in Resource Group rg_usea_prod
Scanning SignalR in Resource Group rg_usea_prod
Scanning Event Hubs in Resource Group rg_usea_prod
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x8 pc=0x1424390]
goroutine 608 [running]:
github.com/Azure/azqr/internal/scanners/aks.(*AKSScanner).GetRules.func16({0x1c9cea0?, 0xc000282050?}, 0xf14f05?)
D:/a/azqr/azqr/internal/scanners/aks/rules.go:268 +0x50
github.com/Azure/azqr/internal/scanners.(*RuleEngine).EvaluateRule(_, {{0x1fbb0fb, 0x7}, {0x1fd8c0a, 0x16}, {0x1fb6e03, 0x4}, {0x1ff1dea, 0x27}, {0x1fb5fdc, ...}, ...}, ...)
D:/a/azqr/azqr/internal/scanners/scanner.go:100 +0x6c
github.com/Azure/azqr/internal/scanners.(*RuleEngine).EvaluateRules(0xc0002d6950?, 0xc000324420?, {0x1c9cea0, 0xc000282050}, 0x1fb97ec?)
D:/a/azqr/azqr/internal/scanners/scanner.go:119 +0x158
github.com/Azure/azqr/internal/scanners/aks.(*AKSScanner).Scan(0xc0002d6950, {0xc000324420, 0xc}, 0x1d5?)
D:/a/azqr/azqr/internal/scanners/aks/aks.go:40 +0x1c5
github.com/Azure/azqr/cmd/azqr.retry(0x3, 0xc000710360?, {0x21889d0, 0xc0002d6950}, {0xc000324420, 0xc}, 0x0?)
D:/a/azqr/azqr/cmd/azqr/scan.go:344 +0x125
github.com/Azure/azqr/cmd/azqr.scan.func2({0xc000324420?, 0x1264cc5?}, {0x21889d0?, 0xc0002d6950?})
D:/a/azqr/azqr/cmd/azqr/scan.go:260 +0x99
created by github.com/Azure/azqr/cmd/azqr.scan in goroutine 1
D:/a/azqr/azqr/cmd/azqr/scan.go:257 +0x1cce
INF Scanning Virtual Network in Resource Group rg_usea_prod
Scanning Virtual Machines in Resource Group rg_usea_prod
Scanning AKS Clusters in Resource Group rg_usea_prod
Scanning Postgre in Resource Group rg_usea_prod
INFINF Scanning App Service Plans in Resource Group rg_usea_prod
Scanning Application Insights in Resource Group rg_usea_prod
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x8 pc=0x1424390]
goroutine 778 [running]:
github.com/Azure/azqr/internal/scanners/aks.(*AKSScanner).GetRules.func16({0x1c9cea0?, 0xc00010e1e0?}, 0xc000468600?)
D:/a/azqr/azqr/internal/scanners/aks/rules.go:268 +0x50
github.com/Azure/azqr/internal/scanners.(*RuleEngine).EvaluateRule(_, {{0x1fbb0fb, 0x7}, {0x1fd8c0a, 0x16}, {0x1fb6e03, 0x4}, {0x1ff1dea, 0x27}, {0x1fb5fdc, ...}, ...}, ...)
D:/a/azqr/azqr/internal/scanners/scanner.go:100 +0x6c
github.com/Azure/azqr/internal/scanners.(*RuleEngine).EvaluateRules(0xc0002b8980?, 0xc000592370?, {0x1c9cea0, 0xc00010e1e0}, 0x0?)
D:/a/azqr/azqr/internal/scanners/scanner.go:119 +0x158
github.com/Azure/azqr/internal/scanners/aks.(*AKSScanner).Scan(0xc0002b8980, {0xc000592370, 0xc}, 0x0?)
D:/a/azqr/azqr/internal/scanners/aks/aks.go:40 +0x1c5
github.com/Azure/azqr/cmd/azqr.retry(0x3, 0x0?, {0x21889d0, 0xc0002b8980}, {0xc000592370, 0xc}, 0x0?)
D:/a/azqr/azqr/cmd/azqr/scan.go:344 +0x125
github.com/Azure/azqr/cmd/azqr.scan.func2({0xc000592370?, 0x0?}, {0x21889d0?, 0xc0002b8980?})
D:/a/azqr/azqr/cmd/azqr/scan.go:260 +0x99
created by github.com/Azure/azqr/cmd/azqr.scan in goroutine 1
D:/a/azqr/azqr/cmd/azqr/scan.go:257 +0x1cce

@Arek-itect thanks for reporting. This one is fixed in version v.0.31.2

Thanks for the prompt response and addressing the issue. Excuse my ignorance though: how do I update installation to 0.31.2 on Windows?


User Column

use FastColumnRenderableSeries draw Chart
this is code

The following problem will appear:

How to solve the problem

@andyb1979 @klishevich @jinoralen @ShadyEM please help me solve the problem

@andyb1979 Can you look at this problem and solve it as soon as possible，device resolution is 1920*1080 60dpi

@andyb1979 Can you look at this problem and solve it as soon as possible，device resolution is 6.88inch 1920*1080 320ppi 60fps

Hi there, we have logged this in our task board for SciChart.android. our team will get to it when they can!
Best regards
Andrew

We look forward to your team solving this problem as soon as possible! If it is solved, can you tell me?


Update config module name

the upstream name has changed in most recently docs theme package

waiting for crate/crate-docs-theme#162


Only run Github Actions for relevant branches

The current Github workflows run for all branches. Since the workflows are for ROS1 melodic and noetic, they fail when running with the new humble branch.
I propose to change the existing workflows to only run on relevant branches (probably master).
When the ROS2 port on humble nears its completion, we should also look into adding a workflow for it.

ROS1 CI only runs on noetic branch now.


Unable to run Upgrade assistant latest version 0.5.261 through automated pipeline

Unable to run Upgrade assistant latest version 0.5.261 through automated pipeline, it's failing with below error message
Initializing and loading extensions...
Telemetry
Upgrade Assistant collects usage data in order to help us improve your
experience. The data is collected by Microsoft and shared with the community.
You can opt-out of telemetry by setting the
DOTNET_UPGRADEASSISTANT_TELEMETRY_OPTOUT environment variable to '1' or 'true'
using your favorite shell.
Read more about Upgrade Assistant telemetry:
https://aka.ms/upgrade-assistant-telemetry
Read more about .NET CLI Tools telemetry: https://aka.ms/dotnet-cli-telemetry
Press any key to continue...
System.InvalidOperationException: Cannot read keys when either application does
not have a console or when console input has been redirected. Try Console.Read.
at System.ConsolePal.ReadKey(Boolean intercept)
at
Microsoft.UpgradeAssistant.Cli.Startup.FirstUseStartup.StartupAsync(Cancellation
Token cancellationToken) in
D:\a_work\1\s\src\Experiments\UpgradeAssistant\cli\Startup\FirstUseStartup.cs:l
ine 37
I have also tried setting environment variable as suggested in logs but still failing with same message.

@niteshcse009  did you set DOTNET_UPGRADEASSISTANT_TELEMETRY_OPTOUT=1 or true? It does work on our CI machine with "true". There seem to be a bug when value is 1. Please try to set value to "true" to get unblocked while issue is being fixed.

Also DOTNET_UPGRADEASSISTANT_SKIP_FIRST_TIME_EXPERIENCE is the correct variable to avoid this message to appear in the console. DOTNET_UPGRADEASSISTANT_TELEMETRY_OPTOUT is the variable to opt out from telemetry when upgrade is actually running.

Hi Anton,
I have tried both way 1 and true but it still doesn't work.
Regards, Nitesh
On Thu, 28 Sept 2023, 19:46 Anton Piskunov, @.***>
wrote:

@niteshcse009 https://github.com/niteshcse009 did you set
DOTNET_UPGRADEASSISTANT_TELEMETRY_OPTOUT=1 or true? It does work on our CI
machine with "true". There seem to be a bug when value is 1. Please try to
set value to "true" to get unblocked while issue is being fixed.
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/upgrade-assistant/issues/1518#issuecomment-1739838535,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCNNFE5IX5WPHHO4ZJDGSDX4XAYTANCNFSM6AAAAAA4YU5XF4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Just to clarify: you set DOTNET_UPGRADEASSISTANT_SKIP_FIRST_TIME_EXPERIENCE=true ?

Good day,
I also tested all possible settings using set as 1 and true. But always the same result. We need the result in the file.
Example:
upgrade-assistant upgrade -o sdkstyle --non-interactive c:\test\test.csproj > test.txt
output:
Initializing and loading extensions...
Determining supported upgrades...
System.IO.IOException: Neplatný popisovaƒ.
at System.ConsolePal.Clear()
at Spectre.Console.Flow.FlowRunner.RunAsync(CancellationToken cancellationToken) in
D:\a_work\1\s\src\Experiments\UpgradeAssistant\spectre.flow\Flow\FlowRunner.cs:line 96
Version:
C:>upgrade-assistant -v
0.5.261.51392

@janbalcarik this is different issue and thanks for pointing it out. We are working on the fix for this for the next release. Original issue was about opting out from telemetry and first time question via env variables and it does work for us (we have our tests using them for every build).

We have integration tests setting environment variables like below running every day on CI machines. Could you please double check that you do have them set? Print out all variables?         EnvironmentVariables["DOTNET_UPGRADEASSISTANT_SKIP_FIRST_TIME_EXPERIENCE"] = "true";
EnvironmentVariables["DOTNET_UPGRADEASSISTANT_TELEMETRY_OPTOUT"] = "true";

I just tried this way and aparrently it works:

No errors so far


Simultaneous Python 2 and Python 3 support.

In one of my projects I have a need to deploy to both Python 2 and Python 3 targets, making the 2to3 conversion solution rather painful. Would there be any objections to creating a polyfill (backfill?) for Python 2 for unicode or something similar?
I couldn't find the full reasoning when this was discussed previously. Thanks for bearing with me.

Is this PR still needed?

I personally don't need this anymore.
I ended up co-authoring (some pairing and speed fixes) noahmorrison/chevron. Also, since then I have left the company with this requirement, and they may have fully converted to python3 anyway.
That being said, I'm personally of the opinion to provide backwards compatibility to python3 thorough writing compatible code, rather than forwards compatibility using the converter.


⚠ Mobitopup has degraded performance

In b82b6c8, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 4359 ms

Resolved: Mobitopup performance has improved in 97c4101 after 7 minutes.


uir file is damaged

I meet the issue uir file is damaged when i copy a project to other computer

Hi,
I am not sure what this means. Could you specify what the "uir" file is? And how exactly did you copy your project to the other computer?
regards,
Jörg


Go to Definition should go to the implementation, not .d.ts when using project references

TypeScript Version:  3.5.2

Search Terms:
"go to definition" declaration project references
Code
Running "Go to Definition" command on Disposable (via project reference) at this line will go to the .d.ts file, not the implementation.


Fix for [box2d][gwt] World.getFixtures() missing from sources #5383

Fix for [box2d][gwt] World.getFixtures() missing from sources #5383

Added missing getFixtures method
Added a test in Box2DTest

Ready to merge


OpenPose not working

I've tested every function of this and they all seem to work well except for pose and inpaint. I've attached a screenshot showing the input image and the result next to each other.

Please check whether you have replace the draw pose function here:  https://github.com/xinsir6/ControlNetPlus?tab=readme-ov-file#inference-scripts.

I searched for these two but neither of the util.py files have draw_bodypose anywhere in them when I open them in notepad++


Unresponsive Widget

Expected behavior
When you load the widget on your screen you are able to play/pause or perform other functions.
Actual behavior
The widget is unresponsive and does not work after it is loaded onto the screen.
How to reproduce

Navigate to the coffee music player from the list of widgets present and load it.
Click on any of the buttons provided but they appear unresponsive.

Device: Lenovo Z2 Plus
Operating system: Android OS 6.0.1
Rom: Stock Zui <IP_ADDRESS>ST
App version: 2.0.6

Recording Of The Bug
https://youtu.be/NkCUNoaon0I
Posted on Utopian.io -  Rewarding Open Source Contributors

Thanks for showing the issue. I'll check it and fix it in the next update.


Update MSDN links, Part 4

@NextTurn : Thanks for your contribution! The author, @vladvino, has been notified to review your proposed change.

The changes in this submission touch multiple articles. Requesting review from multiple authors.
Reviewers, please verify the changes for the file(s) assigned to you:

@mikebudzynski - api-management-howto-disaster-recovery-backup-restore.md
@darrelmiller - api-management-log-to-eventhub-sample.md
@vladvino - api-management folder (eight articles)

Thank you.

Please assign @darrelmiller task to @miaojiang.

#sign-off

@vladvino - Acknowledged. Changed assignment to @miaojiang.
PR reviewers: Two additional authors need to sign off before this request is ready for merge. Thanks

@NextTurn : Thanks for your contribution! The author, @vladvino, has been notified to review your proposed change.

#sign-off

@mikebudzynski: I'm sorry - only the author of this article, @vladvino, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@vladvino - Acknowledged approval.
@mikebudzynski - Acknowledged approval.
PR reviewers: @miaojiang also needs to sign off before this request is ready for merge. Thanks
#hold-off

#sign-off

@miaojiang: I'm sorry - only the author of this article, @vladvino, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off

Thanks, all. Merging the PR.


docs: update README

Update the README to simplify it and reflect the new changes on the project.

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

v2.0.0
GitHub release

Your semantic-release bot :package::rocket:


Add colon-style emoji support

This allows :emoji:s to be placed in the text of chores and notes. Colon-style emojis are often used with Slack, and so long as the emoji is standard (i.e. not a custom-added emoji to a Slack workspace), it should be converted to Unicode and show up as appropriate.

Consider switching to PNG or otherwise image emojis rather than Unicode ones

Can be reopened with fresh code if the feature is still desired.


There is no text for electrum random node - Linux

Instead of 'settings-random-node-text', the text should be "Random node address and port will be displayed when connection is established'.
Ubuntu 18.04.4

Checked on build: 5.1.9590.3200


Set up Docker-based testing container and update testing deps

Replace NPM with Yarn
Replace Bower with Yarn
Migrated Bower components to Yarn
Updated NodeJS to v8 LTS

Travis CI is in transition to new GitHub Apps integration. Private repos are currently broken, so must force merge for now.


[feat] Rename Cluster Diff to Schema Diff

Cluster Diff shows the schema differences, not the cluster configuration.
This needs to say CQL Schema Diff. I would prefer it if it shows the history of changes, and being able to diff between the changes.

Also "New" -> "Current"

@hshimizu we will rename it to Schema Diff


build: Add build target for vkcube-display

Add a new CMake target for vkcube-display, which the variant using VK_KHR_display.
This is a variant which already exists, but is currently hard to build as only one variant can be selected per build using CUBE_WSI_SELECTION. With this option, it is possible to build this variant in addition to other ones, similar to what is done for vkcube-wayland.
As of now this only takes effect in Linux (same as the wayland one below it) as this is the only platform I'm able to test on.

Author enunes not on autobuild list. Waiting for curator authorization before starting CI build.

Author enunes not on autobuild list. Waiting for curator authorization before starting CI build.

 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.

I want to state that while this PR is welcome, the solution is more of a band-aid than a long term fix. The root cause is vkcube being unable to choose the WSI platform at runtime.
I am already working on a PR which enables selection of xcb, xlib, wayland, directfb, and this PR's VK_KHR_display. I'd appreciate your review when I publish it in the coming days. I have the loading all happening, the last bit of work is polishing the selection logic to handle automatically choose an available WSI platform.

I agree that a refactor is better either by reworking the CMake logic or choosing the WSI at runtime. If you're already working on it then that will be better.
So I can help review the new PR when it arrives and maybe we can drop this one.
In the end all I'm looking for is a way to ship this cleanly in distributions so I don't have to build it over and over for a quick check for development on new installs. If the new PR would still take a long time, then I would still be interested in having this as a way to build this for distributions in the meanwhile.

Closing in favor of #1029

In the end all I'm looking for is a way to ship this cleanly in distributions so I don't have to build it over and over for a quick check for development on new installs. If the new PR would still take a long time, then I would still be interested in having this as a way to build this for distributions in the meanwhile.

That is exactly the reason that vkcube should have supported multiple WSI platforms at the same time, but alas it was easier to just copy-paste the cmake code and tweak the build than it was to actually make it work.
It has irked me that vkcube didn't handle that, so thank you for making this PR in order to encourage me to fix the underlying problem :D

@charles-lunarg Thanks! I'll test the new PR and post some review feedback there shortly.


Add ability to check whether PVC storage requirements are fulfilled by the node

Describe the problem/challenge you have
It seems that a PVC can claim and be provisioned more storage than the disk size of a particular node.
For example, the following PVC requesting 5G should be denied if the available capacity of nodes in cluster is 1G
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: local-hostpath-pvc
  namespace: openebs
spec:
  accessModes:
  - ReadWriteOnce
  resources:
    requests:
      storage: 5G

As it currently stands, the above PVC will be provisioned and bound to a PV .
Describe the solution you'd like
We would like  to limit the storage policy to existing free storage in node.
Environment:

OpenEBS version:

$ kubectl get po -n openebs --show-labels
NAME                                           READY   STATUS    RESTARTS   AGE    LABELS
openebs-localpv-provisioner-b559bc6c4-k87hl    1/1     Running   0          4h4m   name=openebs-localpv-provisioner,openebs.io/component-name=openebs-localpv-provisioner,openebs.io/version=3.0.0,pod-template-hash=b559bc6c4
openebs-ndm-cluster-exporter-b48f4c59d-f24d9   1/1     Running   0          4h4m   name=openebs-ndm-cluster-exporter,openebs.io/component-name=ndm-cluster-exporter,openebs.io/version=3.0.0,pod-template-hash=b48f4c59d
openebs-ndm-node-exporter-22mv4                1/1     Running   1          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-node-exporter-2g78z                1/1     Running   1          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-node-exporter-4h526                1/1     Running   1          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-node-exporter-9p2xv                1/1     Running   3          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-node-exporter-c999w                1/1     Running   1          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-node-exporter-d5xsq                1/1     Running   3          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-node-exporter-fnl8m                1/1     Running   1          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-node-exporter-h2q88                1/1     Running   4          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-node-exporter-kcrs7                1/1     Running   1          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-node-exporter-vclcz                1/1     Running   2          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-node-exporter-x6ww5                1/1     Running   1          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-node-exporter-xczq2                1/1     Running   1          14d    controller-revision-hash=5fc6c5df65,name=openebs-ndm-node-exporter,openebs.io/component-name=ndm-node-exporter,openebs.io/version=3.0.0,pod-template-generation=1
openebs-ndm-operator-77d54c5c69-pt85n          1/1     Running   0          89m    name=openebs-ndm-operator,openebs.io/component-name=ndm-operator,openebs.io/version=3.0.0,pod-template-hash=77d54c5c69

Kubernetes version :

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.2", GitCommit:"092fbfbf53427de67cac1e9fa54aaa09a28371d7", GitTreeState:"clean", BuildDate:"2021-06-16T12:59:11Z", GoVersion:"go1.16.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"19+", GitVersion:"v1.19.6-4+2076e5554ee7d2", GitCommit:"2076e5554ee7d2e0ac57857d161db7fd6fad915a", GitTreeState:"clean", BuildDate:"2021-01-05T22:46:00Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}

OS (e.g: cat /etc/os-release):

~# cat /etc/os-release
NAME="Ubuntu"
VERSION="18.04.5 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.5 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

kernel:
linux

As mentioned in a Slack #openebs thread:
There are a few dimensions to the problem:

(any filesystem) Current available space ← naïve solution to this would be a great start!
(XFS) Quotas & their usage (and maybe an overcommit percentage)
(LVM) Space available in VG

Affinity-by-IOPS might be another way to schedule, and as I understand it affinities can have some ranking metric so I can say 'Allocate this PV where:' ... 'it just fits' or 'there is the most space' or 'where other local PVs are generating fewer IOPS'.

The hostpath provisioner volumes are thin-provisioned (basically how the filesystem handles it).
Taking an example...
Let's say there's a filesystem with 10 GiB storage capacity. A hostpath volume with 7 GiB capacity request is created on this filesystem. It succeeds. The application process consumes a small amount of space, say 1 GiB.
Going forward, if another volume request for say, 5 GiB is created, it will succeed. As the available capacity for the filesystem will be observed to be (10-1) GiB = 9 GiB!
The hostpath provisioner does not pre-allocate the requested capacity.
@maunavarro @trathborne -- Does the above  hostpath provisioner behaviour suffice your use-case? Thick provisioning is beyond the scope of the hostpath provisioner's design limitations.

This is working fine for me because I am using XFS quotas and I am ok with overcommitting.
I am more concerned about balance. Consider this case: I have 3 nodes, each with 10GiB of storage. 4 PVCs for 6GiB of storage come in. 2 nodes get 1 PVC, one node gets 2 PVCs. Now another PVC for 6GiB comes in. What is to guaranteed that it does not end up on the node that is already overcommitted?

@trathborne -- The LVM-based LocalPV CSI-driver is better suited for capacity-aware scheduling use-cases. Here's link: github.com/openebs/lvm-localpv

@niladrih I was going to say that LVM doesn't let me overcommit, but then I found https://github.com/openebs/lvm-localpv/blob/2d5196996e97edb82d1840900416835b0b729328/design/monitoring/capacity_monitor.md#thin-logical-volume ... thanks!

Hi @maunavarro, this issue has been open for a while. Are you are still facing this issue?
This feature is one which is typically present in a CSI driver based storage plugin (e.g. LocalPV ZFS, or LocalPV LVM). This is unfortunately beyond the scope of the out-of-tree provisioner library that this hostpath provisioner uses.
That being said, my viewpoint might be pigeonholed. Please feel free to open a design proposal or continue the discussion here. Marking this as 'wont-fix' for now.

@niladrih if there is no possible way to achieve this in the external provisioner, should this be closed?


Release @google-cloud/bigquery v2.1.0

This pull request was generated using releasetool.

Release is at https://github.com/googleapis/nodejs-bigquery/releases/tag/v2.1.0

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


reopen #500

Please reopen #500. It looks like that I can't do this myself.

Do you have a concrete suggestion for how to improve the performance?  If so, we can try to implement it; as I commented on 500 though, using seeds for large table ingest is not an intended use case.

I guess a upload and a call of spark.read.option("delimiter", ",").csv(path) could be a fast option. But would probably require to first upload the file to a temp storage in databricks.

I found an alternative solution for myself so I do not need anymore a higher csv performance


Manual page is needed

There are couple of special instructions possible in rpmautospec commits, but they are kind of hard to find on the installed system. rpmautospec package does not contain even README file, nor any other %doc document.
I think usable manual is needed for common operations. I do not think one should have to open browser with html pages describing special tags like:

https://fedora-infra.github.io/rpmautospec-docs/autochangelog.html#skipping-changelog-entries
https://fedora-infra.github.io/rpmautospec-docs/autorelease.html#one-off-bumping-release-numbers

Or terse description what goes into changelog and what does not, like described on:

https://fedora-infra.github.io/rpmautospec-docs/autochangelog.html#changelog-entries-generated-from-commit-messages

I think web version is somehow too chatty for people who already know the feature, but looks for special tags. Ideally I would expect just man rpmautospec to provide such information right on the system where I do those commits.

Hi,
in the Fedora and EPEL packages from version 0.6.4, a man page is built and distributed with the command line tool.


improve GLSL classification

added extensions .gs, .comp and .glslf.
See: https://stackoverflow.com/q/6432838

add disambiguation rule for .gs files:
if it starts with #version <number>, it's GLSL.
This seems to work for most GLSL files with few exceptions
that are catched by the classifier.

added a sample extrude_normal.gs from kyle-piddington/ShaderTool
(licensed under MIT):
https://github.com/kyle-piddington/ShaderTool/blob/c753a53bde6eab942da617adab9483c945f27f51/assets/shaders/extrude_normals.gs

See usage in GitHub: .gs, .comp, .glslf.

It would be good if someone with JavaScript insight could comment on those JavaScript .gs files.

Friendly nudge.

@lildude @pchaigno I have updated the branch with the requested changes.

@smola If you sent a new commit, I don't see it...


Update Room to Support M1 Macbooks

The project does not compile on an M1 Mac, since the Room version used is old and doesn't have native sqlite library needed
Steps to reproduce

Clone the repo on an M1 Mac
Run the app on an emulator or a physical device

Expected behavior
I expect the app to be installed on my device or the emulator
Actual behavior
Android Studio throws this error
Caused by: java.lang.Exception: No native library is found for os.name=Mac and os.arch=aarch64. path=/org/sqlite/native/Mac/aarch64

Environment

Android Version: Any version
raywenderlich.com App Version: Latest development branch
Device: Any device

This issue is already fixed. Going ahead to close this.


Fix splitting devices with hyphenated filesystem names

Fixes #1093
Signed-off-by: Nathaniel Clark<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1121   +/-   ##
=======================================
  Coverage   95.42%   95.42%
=======================================
  Files           2        2
  Lines         153      153
=======================================
  Hits          146      146
  Misses          7        7

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 dbe8064...9a9e685. Read the comment docs.


Can't use buffers with the encrypt/decrypt functions

The encrypt and decrypt functions both convert the input plaintext to and from a Buffer to perform the encryption/decryption.
The functions should allow a user to pass in a Buffer directly, and if so, the functions should return an encrypted/decrypted Buffer accordingly.

Fixed by PR #8


add a type definition for the ctest output

I also cleaned up the ctest run function to handle a promise rejection.
When you await an async function, if that function rejects, then the
rejected promis throws. So you either need to put the await in a
try/catch or use then/catch methods

Thanks for the contribution. I manually merged and fixed the merge conflicts in f7c2fd1763d984c60c184e0e7874e3f5e86834a5.


xxd.exe is detected by an awful lot of virus scanners

Steps to reproduce
Maybe a false positive, but:

Run Windows
Download installer from: https://github.com/vim/vim-win32-installer/releases/download/v9.1.0512/gvim_9.1.0512_x64.exe
C:\Program Files\Vim\vim91\xxd.exe is probably detected by your virusscanner (e.g. Windows Defender does).

See on VirusTotal:

xxd.exe (17/74) https://www.virustotal.com/gui/file/b6660cf54ac304336411b9a6ee9f2eaf7f6c5fe124efab42a3cc0e62d20b7b13/community
Intezer scan of xxd.exe doesn't see anything in particular: https://analyze.intezer.com/analyses/dbee94ef-d430-4081-93c5-460da2e4bf96/genetic-analysis
Vim installer (3/73): https://www.virustotal.com/gui/file/e76d8747e5cb7f651b9157d0ca2ecb0104b9e13ee191c9b357347c94bda77d7f/community
GitHub download (0/95): https://www.virustotal.com/gui/url/69b17b49fc38dae354935273140f99d6c4ae0efe15b5094bb687c759b652042f

Expected behaviour
No virus detection.
Version of Vim
Windows gvim 9.1.0512
Environment
Windows 11 23H2
Logs and stack traces
No response

Should I (re)post this at vim/vim-win32-installer? E.g. https://github.com/vim/vim-win32-installer/issues

Please see this FAQ: https://github.com/vim/vim-win32-installer/wiki
There is not much we can do about it. If you don't mind helping us, please inform it to each anti-virus vendor.

Yeah, but it's not really detecting the Nullsoft installer (very much, only by some obscure scanners).
I'll try contacting Microsoft, Bitdefender and TrendMicro. I think the most legitimate from the bunch.

Just for reference, increase in detections on VirusTotal seems to have started with gvim 9.1.0466 from 5 June 2024.

I've submitted a false positive review to Bitdefender and Microsoft.
Locally Bitdefender no longer detects and removes xxd.exe 9.1.0514
On VirusTotal those both still detect xxd.exe 🤷♂. I suppose most of the other scanners just "went along with the big guys".


New design gamecard

Summary
Issue Number:
Resolves #{Enter Issue Number here}
Please include a summary of changes

Changed styling to better match new design.
Programmatically shorten summary strings in GameCard's.

Type of change
Select all that apply (add an x inside brackets)

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Scripts
[ ] Documentation

Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

@mayashley I made the image a bit wider because it wasn't far from showing all of the cover image.


carousel.getCurrentItem into an ImageView?

I am wondering how to generate a new ImageView from the current item stored in my carousel?

This is not working - OnClick is calling only of there was click on image. But if caorusel changes there is now click.


Colin McRae Rally 1 (1998) doesn't launch

Game closes instantly on startup.
GPU: Intel HD 530 + GTX 960M (Optimus laptop)
OS: Windows 10 Home 22H2 (19045.3086)
DDrawCompat: v0.4.0
Debug log:
DDrawCompat-Rally.log

Indeed, there is a bug in DDrawCompat which causes the last GetAttachedSurface method call to fail, which leads to a crash. This quick fix should solve it for now:
ddraw.zip (diff.txt compared to v0.4.0)


Remove get_space_secrets endpoint

For security the GET /secrets endpoint is not exposed (see discussion -internal link-).
Let's remove the get_space_secrets(...) method for now since it's not a requested feature and not so important for now.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Development: Update docker-based monitoring setup.

Checklist
General

[x] I chose a title conforming to the naming conventions for pull requests.

Motivation and Context
We currently have a monitoring-stack consisting of, among others, Grafana and Prometheus.
This PR updates to provided monitoring setup to one that is more similar to the production one.
Description

Update versions of Grafana and Prometheus
Include Dashboards

Steps for Testing

Note: Testing is only possible locally

Start Artemis locally, e.g. from your IDE
Start the monitoring stack by running docker-compose -f src/main/docker/monitoring.yml up
Grafana should be reachable at http://localhost:3000
Login using the username admin and password admin (You will be asked to change your password now, you can also set it to admin again.
See that the provided metrics (e.g. Artemis User count work as expected) - You have to open Artemis in your browser so that your connection is counted.

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Class/File
Branch
Line

Screenshots

#removeStale


v2.1.2 does not work.

Had porblems with previous version cuz it crashed every time i hit RUN button. Todays release of 2.1.2 doen't crash when hitting RUN button but also nothing happens at all. So i doesn't even switch to "Connected" or something.Just feels like its not a button but a picture.Any way to fix that?
P.s. Tried both x86 and x64 versions in Admin mode and simple exe launch

Hello @layserk1,
Can you provide more information and a screenshot? Did you use proxy mode or TUN mode?

Hello @layserk1, Can you provide more information and a screenshot? Did you use proxy mode or TUN mode? Also, make sure you extract all the files in the zip.

Hello and thx for fast response.Will try my best with feedback
1.Only proxy mode
2.Exctracted all files for sure
3.Here is the link of x64 version behaviour . x86 does the same. https://youtu.be/Cf5qnINLx4o

Have the same problem on new installation of Windows 10. On my old PC with Win 10 all working fine

Thanks, @layserk1 for the video and description.
@layserk1, @ilhass Can you click on the Check button to see if the config is giving any ping or not? Also, if you can, try another config file to ensure the app doesn't work with any config in your computers.

@layserk1 @ilhass I'm not sure but maybe it's because of your directory name (some folders with non-English names). Please move the app to a full English path and try again.

@layserk1 @ilhass I'm not sure but maybe it's because of your directory name (some folders with non-English names). Please move the app to a full English path and try again.

 Changing directory fixed button problem but no there is another one.
Not app works only in TUN mode. Using proxy mode shows "Connected" but doest nothing at all. TUN mode seems to work fine.

@layserk1 Make sure you checked Use system proxy option and try it with Http protocol to see the problem exists or not.

@layserk1 @ilhass I'm not sure but maybe it's because of your directory name (some folders with non-English names). Please move the app to a full English path and try again.

 Changing directory fixed button problem but now there is another one. Now app works only in TUN mode. Using proxy mode shows "Connected" but does nothing at all. TUN mode seems to work fine.

UPDATE: PROXY MODE doesnt work at all, shows CONNECTED but nothing happens. TUNNEL MODE shows correct VPS ip but doesnt access any websites i have access to with FoxRay app and same config on ios at the same time.
Was using exact setup as u showed.

@layserk1 I think it depends on your config and maybe Invisible Man XRay doesn't support it. Try using another config and give me feedback. Thanks

@layserk1 Can you check it with another config? Seems your config doesn't support with Invisible Man XRay.

@layserk1 I think it depends on your config and maybe Invisible Man XRay doesn't support it. Try using another config and give me feedback. Thanks

Got same config on laptop with previous InvisibleMan version and all works great so the problem is 100% not with config

@layserk1 In this version, I upgraded the xray core. Let me check that. If you find more details, please let me know.

@layserk1 In this version, I upgraded the xray core. Let me check that. If you find more details, please let me know.

The problem is that i can't launch previous version on my PC. Every time i hit RUN button it crashes. So i installed new release and everything next you know. The previous release with same config on my laptop works greatly.

Now tried another my config from absolutely different VPS. Same situation with TUN and PROXY modes.

Thanks, @layserk1 for the video and description. @layserk1, @ilhass Can you click on the Check button to see if the config is giving any ping or not? Also, if you can, try another config file to ensure the app doesn't work with any config in your computers.

Downloaded 1.4.4 version and it's working fine...So i don't know why new version is not working..

Thanks, @layserk1 for the video and description. @layserk1, @ilhass Can you click on the Check button to see if the config is giving any ping or not? Also, if you can, try another config file to ensure the app doesn't work with any config in your computers.

Downloaded 1.4.4 version and it's working fine...So i don't know why new version is not working..

Pity... No foggiest idea what to do with it

Found another issue.
First of all Proxy mode doesnt work for me at all.
After full close of app and opening again and turning on TUN mode it works fine until u hit DISCONNECT. Afterwards u can hit connect as many times as u want - nothing will happen. U have to fully close app again and open it once more. Then u have one more chance to use it and it will work fine again until u disconnect at least once...
Adding video proof
https://youtu.be/05oStFpW3Ys

UPDATE:
Checked on my laptop and office pc . Absolutely same issues as i written before.
Also checked my config with v2rayN -everything works perfectly.
Video linked
https://youtu.be/Oy2gyCxVKNo

@layserk1 Based on the video you sent, you check the internet connection with your terminal. In fact, Invisible Man XRay doesn't proxy your terminal by default, in the proxy mode. But when you switch to the TUN mode, it tunnels all the traffic so you can see the changes in the terminal, too.
The proxy mode just works for something that supports system proxy (like your internet browser and some apps like Telegram if you set the connection type to Use system proxy settings)

@layserk1 Based on the video you sent, you check the internet connection with your terminal. In fact, Invisible Man XRay doesn't proxy your terminal by default, in the proxy mode. But when you switch to the TUN mode, it tunnels all the traffic so you can see the changes in the terminal, too. The proxy mode just works for something that supports system proxy (like your internet browser and some apps like Telegram if you set the connection type to Use system proxy settings)

Ok, but it still doesn’t answer why it doesn’t reconnect until I reboot app and why on proxy mode v2rayN works correctly on same config.
its not on video but I also tried entering blocked websites in proxy mode with InvisibleMan and it doesn’t work , when the same works on v2rayN so its not just terminal ping command.If needed I can provide video later

Ok, but it still doesn’t answer why it doesn’t reconnect until I reboot app...

Hello @layserk1!
About this question, there is a bug in the TUN service that I fixed it now (see #126)
In the next release, this problem will be solved.


Let's revert this change until we address #23760.
Signed-off-by: Michi Mutsuzaki<EMAIL_ADDRESS>

/ci-multicluster-1.13


Placeholder text color too similar to real text

Hello everyone,
since I first used h2o wave, it really startled me, how similar text and placeholder font color are.

For two text boxes so close to each other it is visible, but if you have more Textboxes more spread out it's hard to figure out what's going on.
For all other apps I know, the difference is much more striking.
Unfortunately I haven't found an easy personal fix.
So, if you could consider change for the placeholder style or have an easy personal fix, I'm all ears :)
All the best
Hannes

Update. Just saw
Docs: Add page about Custom CSS and how to write CSS selectors.
So basically thanks for the solution:
# make the placeholders lighter
style = '''
input[type=text]::placeholder {
/* Chrome, Firefox, Opera, Safari 10.1+ */
color: #D3D3D3;
opacity: 1; /* Firefox */
}
'''
q.page['meta'] = ui.meta_card(box='', stylesheet=ui.inline_stylesheet(style))

Cheers
Hannes


Mtsre 1713

What type of PR is this?
Creates new make/mage targets to only run PKO tests. The default target invoked from the CI now skips PKO tests.
What this PR does / why we need it?
PKO tests are flaky hence this split.
Which Jira/Github issue(s) this PR fixes?
https://issues.redhat.com/browse/MTSRE-1713
Special notes for your reviewer:
Pre-checks (if applicable):

[x] Tested latest changes against a cluster
[x] Ran make go-test command locally to run all the unit tests and mock tests locally.
[ ] Included documentation changes with PR

/retest-required

/override ci/prow/precommit-check

/retitle Decouple PKO integration tests
/label tide/merge-method-squash


How to change model url ??

How to change https://s3.ap-northeast-2.amazonaws.com/open-mmlab  to https://open-mmlab.oss-cn-beijing.aliyuncs.com
Which file should we change? Thank you.

From https://s3.ap-northeast-2.amazonaws.com/open-mmlab/xxxxxx to https://open-mmlab.oss-cn-beijing.aliyuncs.com/xxxxxxx, where https://s3.ap-northeast-2.amazonaws.com/open-mmlab/xxxxxx is the model url you would like to download.

From https://s3.ap-northeast-2.amazonaws.com/open-mmlab/xxxxxx to https://open-mmlab.oss-cn-beijing.aliyuncs.com/xxxxxxx, where https://s3.ap-northeast-2.amazonaws.com/open-mmlab/xxxxxx is the model url you would like to download.

Thank you！ But，Which python file should we modify?  I don't know which file to modify.


Bug: PEI Test Data displays an invalid Validation/Error

Issue:
When a user is adding Test Data for Primary Element Inspection (PEI), it displays a Span Scale Required validation which does not apply to PEI data
Steps to Reproduce:

Login to ECMPS
Navigate to Test Data and Open and Check-out a Configuration
Select PEI from the Test Type Code dropdown
Click Add for Test Data, enter data for all fields and click on Save and Close
System displays the following error:

Span Scale is not an applicable field for PEI hence this error should never be displayed

Bug fixed


Handle username setting and dodging initial tutorial flow

Handle the ability to use a completely fresh account that has not yet set the username, and skip tutorial flow

No, unfortunately this won't be implemented as this bot is an account leveling bot, not an account creation bot.

Sorry I didn't mean to create the account. But if a level 1 account is already created, but hasn't set its username.

Yea you have to set the username yourself, but you can skip the tutorials and the bot will run like normal from level 1.


[react-app] Add TraceID to QueryStatsViewer

Fixes #5651

[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Extract traceID from response headers and display it in the QueryStatsViewer component
If no trace ID, the field is suppressed (only the response time, resolution, and number of series are shown)

Verification

Built locally and manually changed props with React Dev Tools
Updated the test but one is failing - for the life of me I cannot workout why this is (yet)

Here's how it looks with a Trace ID set:

Without, it looks the same as now.
@bwplotka I don't think any changes are needed on the backend, as the trace ID is taken from the query response headers here.
The only problem I can see right now if the failing test. No idea why this fails, if I replace the .text() with .render() then I can see the correct output. I think using dangerouslySetInnerHTML causes this change, but I haven't yet found the "correct" way to render and check that string.

We probably need to extract the header value and add it to component props. Then we can render it as a regular DOM node instead of overwriting html.


Fix hadoop-run build error

Related issue: #2021 #2293
docker daemon volume is a host path. We should map it to devbox, then our code could retrive it.

Coverage remained the same at 52.637% when pulling 40632c3a1b41fa83010a2f80a790e63aa7777e3e on zimiao/fix_hadoop_build into c8856373a24ed06c8e5616af0b70872b12ee41a1 on master.


Rename integration tests package.

#1436  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alonh5 and the rest of your teammates on  Graphite


Implement a content hash for remapped build scripts

This commit adds a getContentHash method on classes which have been rewritten by our remapping build script compiler. This
content hash gives access to a hash of the original class, before it has been remapped. The hash is provided as a lazy HashValue.
The value of the hash is stored into the class as a static field, in hex format (to simplify the implementation) and is returned
lazily as a HashValue, but the generated HashValue is not cached (so subsequent calls to getContentHash are going to return
a new HashValue). This means that if you don't look at the hash value, there's no overhead to adding the content hash, but there
is at reading it.
See https://github.com/gradle/task-output-cache/issues/644

I originalByteCodeHash would be a better name than contentHash
can we use byte[] instead of String to store the hash in the generated class?

No pb for using originalByteCodeHash, but it seems specific to this use case. Maybe that interface could be used somewhere else where "original bytecode" doesn't make sense.
We can use byte[] instead of String for the generated hash but is has drawbacks:

we cannot initialize the field using a constant pool String, meaning that the class initialization will take longer (byte[] isn't a constant type)
generation of bytecode is going to be more complex (we need a field + a static initializer, what if there is already a static initializer in the original class?)
generated bytecode is going to be longer (because we need to store a byte[], and in short, it's about LOAD 0, LOAD 128, put it in the array, LOAD 1, LOAD 67, put it in the array, LOAD 2, LOAD 99, put it in the array, ...). As there's no bytecode instruction for array init.

This has been merged manually.


STORM-1203. worker metadata file creation doesn't use storm.log.dir config

@harshach great find.
+1

+1

Great, +1

Btw, seems like LOG-DIR is not used anywhere. Could you check it?


SNOW-897742: snow snowpark package produces empty zip

Python version
3.8
Platform
mac os M1
What happened
Current version (1.0.1):
I run :
snow snowpark package create faker-bilogy
and get the following output in terminal:
The package is not in Anaconda. Do you want to try to see if it's supported as a custom package (requires pip)? [Y/n]: Y
WARNING: You are using pip version 22.0.4; however, version 23.2.1 is available.
You should consider upgrading via the '/Users/jhollan/.pyenv/versions/3.8.16/bin/python3.8 -m pip install --upgrade pip' command.

I get a zip file, but when I open it's a completely empty zip.

Pervious version (0.2.9):
I run:
snow package create faker-biology
I get a much more verbose output (by default - related to #271 ):

Click to view
"faker-biology" not found in Snowflake anaconda channel...
The package is not in Anaconda. Do you want to try to see if it's supported as a custom package (requires pip)? [Y/n]:
Collecting faker-biology
Using cached faker_biology-0.6.0-py3-none-any.whl (110 kB)
Collecting Faker<13.0.0,>=12.3.0
Using cached Faker-12.3.3-py3-none-any.whl (1.5 MB)
Collecting faker-python<0.0.2,>=0.0.1
Using cached faker_python-0.0.1-py3-none-any.whl (2.7 kB)
Collecting python-dateutil>=2.4
Using cached python_dateutil-2.8.2-py2.py3-none-any.whl (247 kB)
Collecting six>=1.5
Using cached six-1.16.0-py2.py3-none-any.whl (11 kB)
Installing collected packages: six, faker-python, python-dateutil, Faker, faker-biology
Successfully installed Faker-12.3.3 faker-biology-0.6.0 faker-python-0.0.1 python-dateutil-2.8.2 six-1.16.0
WARNING: You are using pip version 22.0.4; however, version 23.2.1 is available.
You should consider upgrading via the '/Users/jhollan/Development/Temp/faker_biology/.venv/bin/python -m pip install --upgrade pip' command.
Checking for dependencies available in Anaconda...
Downloaded packages: dict_values([['dateutil', 'python_dateutil-2.8.2.dist-info'], ['__pycache__', 'six.py', 'six-1.16.0.dist-info'], ['faker-python', 'faker_python-0.0.1.dist-info'], ['faker_biology', 'faker_biology-0.6.0.dist-info'], ['faker', 'Faker-12.3.3.dist-info']])
"faker-python" not found in Snowflake anaconda channel...
"faker-biology" not found in Snowflake anaconda channel...
Good news! The following package dependencies can be
                imported directly from Anaconda, and will be excluded from
                the zip: ['python-dateutil', 'six', 'Faker']
Package python-dateutil: deleting ['dateutil', 'python_dateutil-2.8.2.dist-info']
Package six: deleting ['__pycache__', 'six.py', 'six-1.16.0.dist-info']
Package Faker: deleting ['faker', 'Faker-12.3.3.dist-info']
Checking to see if packages have native libaries...

No non-supported native libraries found in packages (Good news!)...

The package faker-biology is supported, but does depend on the following Snowflake supported native libraries you should include the following in your packages: ['python-dateutil', 'six', 'Faker']

Package faker-biology.zip created. You can now upload it to a stage (`snow package upload -f faker-biology.zip -s packages`) and reference it in your procedure or function.

The package faker-biology is supported, but does depend on the following Snowflake supported native libraries you should include the following in your packages: ['python-dateutil', 'six', 'Faker']

And I get a zip file that has the content in it. I don't know what all changed but seems this command has a few hiccups
Console output
No response
How to reproduce
No response

Empty zip issue was fixed in https://github.com/Snowflake-Labs/snowcli/pull/304
The fix should become a part of next release.
I'm closing this issue as it is for empty zip issue.
For command output there is already other issue: https://github.com/Snowflake-Labs/snowcli/issues/271


Update Windows Readme and Scripts

Update the readme design documentation to the latest architecture. Also updates the Appveyor build scripts to remove failing test and build artifacts a bit differently.

Current coverage is 70.63% (diff: 100%)

Merging #1216 into master will increase coverage by 0.54%

@@             master      #1216   diff @@
==========================================
  Files           127        129     +2
  Lines         18953      19059   +106
  Methods        2644       2671    +27
  Messages          0          0
  Branches       2307       2308     +1
==========================================
+ Hits          13285      13463   +178
+ Misses         4788       4726    -62
+ Partials        880        870    -10

Powered by Codecov. Last update 827a480...df53935


Docker is running but no status with "docker ps"

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?
Yes
Are you sure you're not using someone else's docker image?
No
Have you searched for similar issues (both open and closed)?
Yes

Describe the bug
Works perfect ...... but  with  "docker ps"  it's up for more than 20 minutes and working, but no status like other containers  "health".
Nginx Proxy Manager Version

V.2.9.19
To Reproduce
Steps to reproduce the behavior:
docker ps
no status
Expected behavior

Screenshots

Operating System

PRETTY_NAME="Raspbian GNU/Linux 11 (bullseye)"
NAME="Raspbian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
Additional context

What you want is a docker healthcheck.
https://nginxproxymanager.com/advanced-config/#docker-healthcheck

thnx !!

you're welcome :D don't forget to close the issue when your problem is solved


chore: add missing walletconnect exports to RN wallet exports

Problem solved
Short description of the bug fixed or feature added
Changes made

[ ] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[ ] Manual tests: step by step instructions on how to test

Contributor NFT
Paste in your wallet address below and we will airdrop you a special NFT when your pull request is merged.
Address:

PR-Codex overview
This PR adds WalletConnect functionality to the wallets.native.ts file.
Detailed summary

Added WalletConnect client creation functions
Added WalletConnect session management functions
Exported WalletConnect request handlers
Exported WalletConnect types

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#3444  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gregfromstl and the rest of your teammates on  Graphite


fix: thread-safety issue in proclist savefile

Since the proclist is not thread-safe, we had some corrupted captures
because the length was pre-computed and then when the actual item was
written it was different, hence the total was off.
Reduce the likelihood of this by using a secondary dumper, summing up
the lengths in a single loop and then writing everything at once.
Signed-off-by: Angelo Puglisi<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-udig

/area libscap
/area libsinsp

/area tests

/area proposals

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: deepskyblue86
To complete the pull request process, please assign ldegio after the PR has been reviewed.
You can assign the PR to them by writing /assign @ldegio in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Updated the PR after discussing with @gnosek.

/ok-to-test

@deepskyblue86: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs-arm
1b065af5464e40ffd32422b8169c20e6afc5ae11
link
true
/test build-libs-arm

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@deepskyblue86: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs-arm
1b065af5464e40ffd32422b8169c20e6afc5ae11
link
true
/test build-libs-arm

build-libs
1b065af5464e40ffd32422b8169c20e6afc5ae11
link
true
/test build-libs

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@deepskyblue86: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs-arm
1b065af5464e40ffd32422b8169c20e6afc5ae11
link
true
/test build-libs-arm

build-libs
1b065af5464e40ffd32422b8169c20e6afc5ae11
link
true
/test build-libs

build-libs-minimal
1b065af5464e40ffd32422b8169c20e6afc5ae11
link
true
/test build-libs-minimal

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Changes LGTM! Thank you very much!

LGTM label has been added.  Git tree hash: d824d7c96fbaf5934fe44d136553ab253023e486

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: deepskyblue86, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


repository sponsor button broken

Describe the bug
When clicking on the repository's sponsor button a YML parsing error occurs.

It shows the error because my GitHub Sponsors application has not been approved yet.
Nothing to worry about.


Location information in generated messages.xlf points to wrong location

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see use [gitter](https://gitter.im/mgechev/angular2-seed) or [stackoverflow](https://stackoverflow.com/questions/tagged/angular2)

Current behavior
When I run npm run i18n to generate the messages.xlf each trans-unit-entry contains location information which point to the wrong file.
Based on the master branch I get the following file:
<?xml version="1.0" encoding="UTF-8" ?>
<xliff version="1.2" xmlns="urn:oasis:names:tc:xliff:document:1.2">
  <file source-language="en" datatype="plaintext" original="ng2.template">
    <body>
      <trans-unit id="6cdb1fea93d77c07950c0c76c6e0ad79ebbef084" datatype="html">
        <source>Features</source>
        <target/>
        <context-group purpose="location">
          <context context-type="sourcefile">C:/Users/hirse/git/angular-seed/dist/tmp/app/about/about.component.ts</context>
          <context context-type="linenumber">3</context>
        </context-group>
      </trans-unit>
    </body>
  </file>
</xliff>

The sourcefile (C:/Users/hirse/git/angular-seed/dist/tmp/app/about/about.component.ts) is wrong in multiple ways:

It points to the absolute location on my device, not the location in the project.
It points to a file in dist/tmp which doesn't even exist anymore by the time I see the messages.xlf.
It points to the TypeScript file, but the translated text is in the HTML-template.
On Windows, it is displaying a Windows-style path, but using the wrong slashes.

Expected behavior
I would expect the sourcefile to be src/client/app/about/about.component.html.
Minimal reproduction of the problem with instructions
Run npm run i18n and look at the messages.xlf.
What is the motivation / use case for changing the behavior?
The location information in the messages.xlf should be useful (or removed).
Please tell us about your environment:
Windows, although I expect equivalent problems to be present on other OS as well.

Angular Seed Version: 649257d
Node: node --version = v8.0.0

Hi Hirse,
Please check this PR https://github.com/mgechev/angular-seed/pull/1987


Bridge request for zenn.dev

Bridge request

Zenn is a Japanese developer blogsite. It has RSS feeds, but only a small chunk of the post is featured.
f'ex: This article's generated RSS entry looks like this in my feed reader:

Formatting is also stripped from the little bit of the article that is featured, leading it to just be displayed as a jumble of text (above post should have a header and a list)
If possible, it would be nice to be able to have a bridge for Zenn that serves the entirety of the post in the feed, with formatting, images, et cetera preserved, maybe either without that big annoying preview card at the bottom, or with an option to shut it off in case someone does want it. Being able to pick multiple tags etc to include in a feed would also be nice, but is of course not a necessity since RSS-Bridge has its own feed merge feature!
General information

Host URI for the bridge (i.e. https://github.com): https://zenn.dev/

Which information would you like to see?

Post content contained in the RSS feed in its entirety.

How should the information be displayed/formatted?

As close to exactly as it is on the website as possible

Which of the following parameters do you expect?

[X] Title
[X] URI (link to the original article)
[X] Author
[ ] Timestamp
[X] Content (the content of the article)
[X] Enclosures (pictures, videos, etc...)
[ ] Categories (categories, tags, etc...)

Options

[ ] Limit number of returned items

Default limit: 5

[X] Load full articles

Cache articles (articles are stored in a local cache on first request): yes
Cache timeout (max = 24 hours): 24 hours

[X] Balance requests (RSS-Bridge uses cached versions to reduce bandwith usage)

Timeout (default = 5 minutes, max = 24 hours): 5 minutes

Thanks!

You don't need a bridge. I am not sure which feedreader you are using but if you switch over to miniflux or FressRSS then both of them support expanding the feed out of the box and your problem will be resolved.
For miniflux - see https://miniflux.app/docs/ui.html
FreshRSS - https://freshrss.github.io/FreshRSS/en/users/04_Subscriptions.html (see truncated feed section)


Disable More Videos

Is it possible to remove the More Videos tab when the video is paused?

I could not find any solution and as it is a feature from youtube, it might not be possible, but maybe someone here has encountered something similar.
EDIT: I found out that it doesn't appear when the app is in portrait mode, so I might be doing something in that area.

I don't think it possible, those tabs come from YouTube (I assume it is based on the player size).
Even it there was a way to remove them it would violate their TOS.

Hi @phcs971,
Thanks for opening up an issue.
Sadly the YouTube Player iFrame API doesn't provide a parameter to hide the more videos tab.


Create Pull Request template

@yochannah @NPDebs as we discussed

@NPDebs  - I'm merging this, but please add anything you think might be missing in a PR! 🙏

Thanks, @bebatut & @yochannah. 🙌


Uncaught TypeError: Cannot read property 'length' of undefined

[Enter steps to reproduce:]

...
...

Atom: 1.36.0 x64
Electron: 2.0.18
OS: Mac OS X 10.14.1
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot read property 'length' of undefined
At /Applications/Atom.app/Contents/Resources/app/static/<embedded>:14

TypeError: Cannot read property 'length' of undefined
    at ScreenLineBuilder.emitText (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:1103691)
    at ScreenLineBuilder.buildScreenLines (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:1099397)
    at DisplayLayer.getScreenLines (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:89305)
    at TextEditorComponent.queryScreenLinesToRender (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:79324)
    at TextEditorComponent.updateSyncBeforeMeasuringContent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:69977)
    at TextEditorComponent.updateSync (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:67605)
    at visible.suppressUpdates.updatedSynchronously.updateScheduled.i.getScheduler.updateDocument (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:66633)
    at ViewRegistry.performDocumentUpdate (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:326703)

Commands

Non-Core Packages
atom-beautify 0.33.4
atom-ide-debugger-python 0.7.3
autocomplete-clang 0.11.5
autocomplete-python 1.12.0
autosave-onchange 1.5.1
build 0.70.0
busy-signal 2.0.0
dbg 1.6.3
dbg-gdb 1.7.8
dbg-gdb-custom-server 0.3.0
docblock-python 0.12.0
file-icons 2.1.29
find-in-directory 1.3.1
goto-definition 1.3.4
gpp-compiler 3.0.7
intentions 1.1.5
jekyll 2.1.0
jekyll-snippets 0.3.2
jekyll-syntax-highlighting 0.1.0
kite 0.156.0
language-assembly 0.7.2
language-ini 1.20.0
language-latex 1.2.0
language-linkerscript 0.2.0
language-matlab 0.2.1
language-matlab-octave 0.3.0
language-systemverilog 0.9.0
latex 0.50.2
linter 2.3.0
linter-gcc 0.7.1
linter-ui-default 1.7.1
monokai 0.24.0
output-panel 0.3.4
pdf-view 0.71.0
python-debugger 0.2.0
stlink 0.1.1
sublime-block-comment 0.5.1
swd-debugger 0.1.1
sync-settings 0.8.6
teletype 0.13.3
tool-bar 1.1.11
typewriter 0.7.1
vk-terminal 3.0.1

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/atom/issues/19108 so you can subscribe there for updates if you'd like.  If you have specific steps to reproduce, please let us know in https://github.com/atom/atom/issues/19108.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Add class manipulation in prop

It will be good to be able to manipulate some style properties through :props.
Actually, i would like to change the current z-index value of .vue-html-5-editor and .tools.

Vue support it native.
<!--custom-class will add to component root element-->
<editor class="custom-class"></editor>

Yes but you can't manipulate inner class of the component from the parent. Props helps with that.


[nmstate-1.3] nm ovs: Fix race problem when attaching bond to OVS bridge

Got failure when attaching linux bond to ovs bridge:
libnmstate.error.NmstateLibnmError: Activate profile
uuid:cf81a923-5999-4a21-bd7e-dc96c7977451 iface:eth1 type: ethernet
failed: reason=<enum
NM_ACTIVE_CONNECTION_STATE_REASON_DEVICE_DISCONNECTED of type
NM.ActiveConnectionStateReason><enum
NM_DEVICE_STATE_REASON_DEPENDENCY_FAILED of type NM.DeviceStateReason>

This is caused by race between we activating OVS port profile manually
and NettworkManager autoconnect-slaves function.
To fix the problem, for OVS bridge and OVS port, the IP_CONFIG state
is not considered as activated as they are not allowed to have IP.
Integration test case included.

Any options we can backport this to nmstate 1.2?


Write protected error

Hello in a Raspberry Pi 3 with amibian 1,4 and latest hstwb 1.5.1
Upon finalizing the installation of AmigaOS 3.2
I get a message in box saying:
System Request
Volume
KickstartDir
is write protected
I only have two options in the box retry or cancel
Retry does nothing
Is thia a bug ??
Or i did anything wrong?
Im installing AmigaOS 3.2 from adf disks
But i also have the AmigaOS 3.2 iso should i try that?

Update: if i press cancel it continues the installation,i get this message again a couple of times i press cancel and continues,finishes and the amigaOS 3.2 loads fine
Must be a bug e?
Strange coz i dont do it from windows to give me write protection errors
Do you know what's wrong and how to fix it??
Thanks

This error has been fixed with latest release v1.6.3

I get this error  when running the hst installer via FS-UAE in Unbuntu 22.04 LTS


Modified About Section and added content

Description
I have modified the About page , made some changes in UI and added content.
Fixes #166
Screenshots

Checklist

[x] Tests have been added or updated to cover the changes
[x] Documentation has been updated to reflect the changes
[x] Code follows the established coding style guidelines
[x] All tests are passing


Transfering downloaded data to a new profile

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I had to delete my old profile and create a new one to link to my personal account. In this process, I downloaded my profile data and am unable to upload the progress/XP to my new profile.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Transfer all of my old XP/progress to my new profile.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@surajb2 Can you provide some more information about the issue you're having?

Hello,
In order to attach my work profile to my personal profile I had to delete
my existing work profile and relink. Before deleting my old work profile, I
downloaded my old work profile data (json file).
Wondering if it's possible to import that data into my new work profile.
On Fri., Feb. 12, 2021, 4:42 p.m. Genevieve Warren, <
<EMAIL_ADDRESS>wrote:

@surajb2 https://github.com/surajb2 Can you provide some more
information about the issue you're having?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/feedback/issues/3470#issuecomment-778470517,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS2CB4J3ZEYXDTTZ7BVTC43S6WOC5ANCNFSM4XO2TE2A
.


Replaced isolate in validation panel

Replaced isolate which was producing 2-Enteritidis in the MOST update to another isolate which is producing 3-Enteritidis.

Thanks Arlsan, though It looks like there were changes for a number of isolates (red - before to green - now) and not just a single Enteritidis?

I think that might just be a quirk of how github handles an entire row being changed in a spreadsheet for some reason. If you look closely, it shows those isolates being replaced by themselves.


Update Announcement “2020-05-new-dds-director-to-expand-high-impact-technology-initiatives”

Automatically generated by Netlify CMS

Deploy preview for dds-mil ready!
Built with commit 31f00a21ce1cf55b86ddb1382ff96eba0d9c6670
https://deploy-preview-266--dds-mil.netlify.app