'Real' Auto backup session feature...

Hi I like vscode Very much. The only downside is only 1. It can’t auto backup our current work when our computer suddenly crash. It will not resume our work if the file is gone. (for example when working with Filezilla, it creates a temp file for you to work with in your local computer. When computer crash this file is also gone, because as the name suggest, its temporary file only. When the file is gone our work is gone to, vscode cant restore it. vscode can only restore if the file is still there.) the thing i can't understand is why vscode can only restore session if the file is still in there??? So the current feature is not useful when our computer crash. its not real auto save session.
You name it, I have tried every single code editor out there. The only code editor can restore file when computer crash is only Notepad++ (the way it does its job is by auto backup the current session every 7 seconds to another file). the keyword is 'another'
Yes, vscode also has “autosave" feature. But it’s totally different level of backup when compared to NotePad++. All code editor can only restore our work when the file is still there. If the original file is gone, all of them can't restore it. including vscode
That’s why i cant move from Notepad++. No other code editor offer this kind of backup.
Please vscode add this feature. I like vscode very much only because of this.

You write about Filezilla and temporary files. Are you using an extension in VSCode and editing remote files? If so, which extension?

Idont use anything. all software is fresh install.
Its really easy to reproduce without filezilla or any ftp software:

do fresh install vscode.
create 1 file named test.txt
open it vscode and type something without saving
delete test.txt from your computer (simulate filezilla, all ftp software will delete the file you are working with when you close the ftp program or when the computer crash)
force close vscode from task manager (simulate computer crash)
open vscode, voilaa all your work is gone. nothing restored

@erikdemarco I assume you know about VSCode's existing auto save features but these don't meet your need, right?

I reopened the issue because I keep losing all my work when my computer crash
Here's the problem once again:
I'm using filezilla to modify server file. When I'm working with my file via vscode, suddenly my electricity goes down for several hours. Then I opened vscode again it just show setting page not restoring my previous work.
Notes:
-) All app I mention is in mint condition without any additional plugin
-) Fresh install newest vscode
-) Fresh install filezilla (filezilla stores the file we are working with in /tmp folder, the file will get removed if filezilla or computer restarted/crashed)
-)Vscode will successfully restore session if I force restart by pushing reset button in my cpu (simulating crash). But vscode wont restore session if the electricity goes down for several hours, maybe something to do with ram? because ram data will get lose if there's no power supply

@gjsjohnmurray any update about this? :(

VS Code does exactly what you're asking for already with hot exit. The problem seems to be this:

Fresh install ftp software (all ftp software stores the file we are working with in /tmp folder, the file will get removed if computer restarted/crashed)

If the files and more importantly the folder is gone from the system, the backup system will not know how to restore them. I strongly suggest you to change your workflow as it sounds horrible, especially if your power goes out frequently. For example, instead of working on files on some server via FTP, why don't you persist them in source control (such as git) and then connect to your server and pull them down from there. Or perhaps even better we just released the Remote - SSH to allow easy editing of files on a server via SSH.
There is also auto save available as @gjsjohnmurray called out and of course saving more frequently.


wrong entry into built importMap.server.json

After running deno task build there should be an entry for the root  file entry point server.tsx but instead of it having the .tsx extension it is .js:
{
  "imports": {
    "./server.js": "./server.js",

Manually changing it to this fixes it:
{
  "imports": {
    "./server.tsx": "./server.js",

This will be fixed in the next beta release! Apologies for the delay

This is fixed in the latest beta release


v1.15 Backports 2024-05-06

[ ] #29803 (@julianwiedmann)
[ ] #32225 (@giorio94)
[ ] #32199 (@aanm) :warning: resolved conflicts by ignoring the changes to values.schema.json not present in v1.15
[ ] #32284 (@sayboras)
[ ] #30548 (@squeed)
[ ] #32240 (@pchaigno)
[ ] #32303 (@marseel)
[ ] #32244 (@learnitall)
[ ] #32333 (@jrajahalme)
[ ] #32155 (@julianwiedmann)
[ ] #32340 (@squeed)
[ ] #32347 (@giorio94) :warning: resolved conflicts moving the wait duration bump up to step Wait for cluster mesh status to be ready after the first Cilium rollout in cluster2
[ ] #32325 (@nathanjsweet)
[ ] #32267 (@qmonnet)

Once this PR is merged, a GitHub action will update the labels of these PRs:
 29803 32225 32199 32284 30548 32240 32303 32244 32333 32155 32340 32347 32325 32267

/test-backport-1.15


Stop passing around reliable message managers when we don't need to

Problem
What is being fixed?  We are passing around and storing reliable message manager pointers when we don't really need to.
Change overview
Get the reliable message manager from the reliable message context in the one place where we need it.
Testing
There should be no behavior changes here, just simpler and smaller code, so nothing to test other than that nothing breaks.  I expect out CI to do that.

@saurabhst @msandstedt @Damian-Nordic @andy31415 ?


[Enhancement] Add ConnectorTblMetaInfoMgr to manage metadata which can not get from hms

Signed-off-by: Youngwb<EMAIL_ADDRESS>What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

#17453
Problem Summary(Required) ：

For external table, it do not persist it's meta info, it will generate a new table from hms after refresh external table stmt, but sometimes the exteral table need some meta info which can not get from hms (such like related mv on this table), so we add ConnectorTblMetaInfoMgr to manage these meta info.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 85 / 93 (91.40%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/server/GlobalStateMgr.java
2
7
28.57%
[1415, 1416, 1417, 1418, 1419]

:large_blue_circle:
com/starrocks/connector/ConnectorTableInfo.java
21
22
95.45%
[34]

:large_blue_circle:
com/starrocks/connector/ConnectorTblMetaInfoMgr.java
45
47
95.74%
[80, 85]

:large_blue_circle:
com/starrocks/catalog/MaterializedView.java
6
6
100.00%
[]

:large_blue_circle:
com/starrocks/server/MetadataMgr.java
4
4
100.00%
[]

:large_blue_circle:
com/starrocks/server/LocalMetastore.java
7
7
100.00%
[]

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 83 / 97 (85.57%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/server/GlobalStateMgr.java
3
8
37.50%
[1416, 1417, 1418, 1419, 1420]

:large_blue_circle:
com/starrocks/connector/ConnectorTableInfo.java
16
22
72.73%
[34, 54, 55, 56, 57, 59]

:large_blue_circle:
com/starrocks/connector/ConnectorTblMetaInfoMgr.java
44
47
93.62%
[80, 85, 103]

:large_blue_circle:
com/starrocks/catalog/MaterializedView.java
6
6
100.00%
[]

:large_blue_circle:
com/starrocks/server/MetadataMgr.java
6
6
100.00%
[]

:large_blue_circle:
com/starrocks/server/LocalMetastore.java
8
8
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

@Mergifyio backport branch-2.5


propEq documentation error

Apparently, the propEq function temporarily had its argument order reversed from prop -> value to value -> prop before it was reverted.  However, the example on the ramdajs documentation for propEq shows the wrong order.

It's not been reverted? Currently value -> prop. https://ramdajs.com/repl/?v=0.29.0#?let x %3D {a%3A1}
propEq(1%2C 'a'%2C x)%3B
let x = {a:1}
propEq(1, 'a', x); // => true

It's not been reverted? Currently value -> prop. https://ramdajs.com/repl/?v=0.29.0#?let x %3D {a%3A1}
propEq(1%2C 'a'%2C x)%3B
let x = {a:1}
propEq(1, 'a', x); // => true

#2938

Sorry I'm missing something. For me, the current documentation matches the current release. The propEq example from the docs is const hasBrownHair = R.propEq('brown', 'hair');, which is "value -> prop", which matches the current release.

I am using 27.0.2, and I'm going to stay on that then.


fix: governance tests work for governor and non-governor chains

Motivation
If we run fork tests on other chains, the governance router tests need to function for both the governor domain and the non-governor domain

Solution
for non-governor domains,

for handle tests, ensure the remote domain is setUp to the governor domain. the GovernanceRouter tests assume that the remoteDomain is the governor domain; on the actual governor domain, the tests just transfer to the remote domain; but for non-governor domains, this is not possible
for TransferGovernor tests, don't assume homeDomain is the current governor domain.

PR Checklist

[ ] Added Tests
[ ] Updated Documentation
[ ] Updated CHANGELOG.md for the appropriate package

Merge https://github.com/nomad-xyz/monorepo/pull/559 and https://github.com/nomad-xyz/monorepo/pull/560 first


Fix some waitUntil error "expects its completion closure to be only called once"

I don't understand. After Paddy fixes I can have easily have a green in my machine 🤷♂


Seed peers should be lower friction to set

The issue is we have different testnets and the names of seed nodes may change.
We can set default list of seed peers which we ship with the binary or in a separate with the binary, or on a server somewhere which @bkase believes in.

we want to put seed peers in a daemon.json that we ship with the binary. We need:

To load /etc/coda.json or something
Peers in the json at all


resetOnBlur doesn't work as expected with createStackNavigator

Current Behavior

Add 2 random screens inside the createBottomTabNavigator
Place createStackNavigator inside the createBottomTabNavigator and switch resetOnBlur to 'true'

When you leave stack navigator and switch to another createBottomTabNavigator screen your  first screen inside the createBottomTabNavigator renders in the background (componentDidMount triggers).
Expected Behavior
createBottomTabNavigator screen is not rendering in the background
Your Environment
"react-navigation": "^4.0.10",
"react-navigation-animated-switch": "^0.4.0",
"react-navigation-drawer": "~1.4.0",
"react-navigation-props-mapper": "^1.0.1",
"react-navigation-stack": "^1.10.3",
"react-navigation-tabs": "^2.6.0",
"react-navigation-transitions": "^1.0.12",

react-navigation v4 is not stable, and they're working on v5...
I guess I will keep using 3.11.1.  Even 3.11.3 is throwing a dependency error


[bug] HTTP/HTTPS to WS/WSS Protocol string change cause the passed commands with URLs in it to fail

Currently KubernetesCluster::makeWsRequest() replaces too much when transforming from https:// -> wss:// and http:// -> ws://. For example, when calling exec on a pod with any URL in the command, these URLs are also transformed causing the command to fail.

@bobab12 The latest patch for 2.9 should work right now.


Refactoring and logging related changes

This open issue will keep track of the refactoring or any other minor changes required.

Investigate why the connector prints partition details after Finished streaming log:
2022-12-08 06:15:25,755 INFO   YugabyteDB|test_server|streaming  Finished streaming   [io.debezium.pipeline.ChangeEventSourceCoordinator]
2022-12-08 06:15:25,755 INFO   YugabyteDB|test_server|streaming  YBPartition is YBPartition{tabletId='9d8d21450edf4454aad0b3da4e42389c'} and YugabyteDBOffsetContext while streaming is null   [io.debezium.connector.yugabytedb.YugabyteDBChangeEventSourceCoordinator]

Add task ID in the logging context.

Currently, we handle tablet split in the order:

Remove parent tablet from polling list
Get child tablets and add them to the list
In case step 2 failed and we went for a retry, we will not be left with the parent tablet in the list since it has been removed.

Now what we need to do is:

Get child tablets for the parent
Remove the parent tablet from polling list

This will ensure that if the API to fetch child tablet fails for some reasons, we are not left with no option to get the parent further.

Reduce the frequency of logging:
Requesting schema for tablet xyz

Goal: Make this a debug log


Add fallback for click handler emitter

All our apps use the root #main element. In storybook environments we don't have that element so the fallback is needed.

This change fixes @goodhood/components slider story


set bins="auto" for axis histograms in pages/mutations

added option of log-y for mutations panel
removed unused functions in plot_helpers
fixes  #112

https://github.com/tskit-dev/tsqc/assets/25925005/302052cd-12ad-483e-835b-39bc5fdd629c

Ok to use the same checkbox to log all Y axes here? One axis is time and the others are mutation counts.

Love it, great stuff


fails crawl the correct dirs without reloading page after failed export

issue seems to have been the exportTargets JS var having left-overs after a failed export. Running again would have the clean up calls clash with the export...


bug(MatTabHeader): Rouding problem when zooming will show PaginationControls when not needed

Reproduction
Use StackBlitz to reproduce your issue:
https://stackblitz.com/edit/angular-1v9muy
see also issue mat-tab-header-pagination displaying incorrectl
Steps to reproduce:

open in fullscreen and try to zoom in and out until there are PaginationControls visible

Expected Behavior
No PaginationControls  when not needed
Actual Behavior
PaginationControls are visible. Sometimes they are always but are not easy to reproduce in stackblitz
Environment

Angular:
CDK/Material:
Browser(s): chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows

Can be fixed by adding 1
see also SO
// original code from paginated-tab-header
function _checkPaginationEnabled(this: MatTabHeader) {
  if (this.disablePagination) {
    this._showPaginationControls = false;
  } else {
    // const isEnabled = this._tabList.nativeElement.scrollWidth > this._elementRef.nativeElement.offsetWidth;
    const isEnabled = this._tabList.nativeElement.scrollWidth > this._elementRef.nativeElement.offsetWidth + 1;
    if (!isEnabled) {
      this.scrollDistance = 0;
    }

    if (isEnabled !== this._showPaginationControls) {
      this._changeDetectorRef.markForCheck();
    }

    this._showPaginationControls = isEnabled;
  }
}

Log of scrollwidth,offsetwidth when zooming in and out. The scrollwidth is not constant with a specefic zooming level but offset width is.

@Anderman at which zoom level do you run into this?


Device doesn't find modem Huawei MS2131i-8

relevant flowdock conv

Fixed by https://github.com/resin-os/resin-intel/pull/164


Moved Redisson into a provider to enable reuse

Pull Request Test Coverage Report for Build 3452

2 of 28 (7.14%)  changed or added relevant lines in 2 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 69.983%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

redis-lock/src/main/java/com/netflix/conductor/locking/redis/config/RedisLockModule.java
0
26
0.0%

Files with Coverage Reduction
New Missed Lines
%

redis-lock/src/main/java/com/netflix/conductor/locking/redis/config/RedisLockConfiguration.java
4
0.0%

Totals

Change from base Build 3439:
-0.1%

Covered Lines:
11121

Relevant Lines:
15891

💛  - Coveralls


Mistake in installation documentation

In the README.md file gives ] instantiate ., but should be just ]instantiate

Closed by d4bd268


feat: operator-sdk upgrade

Closing for now since we just did this, but won't delete the branch for now in case there are some other changes @gee4vee wants to pull in at some point. Will delete the branch later if not used.

Closing for now since we just did this, but won't delete the branch for now in case there are some other changes @gee4vee wants to pull in at some point. Will delete the branch later if not used.


NimaActor doesnt play on specific phone

Actually, i don't have any logs but my NimaActor just working in an Android Emulator but when I install the same app to my Mi 5 phone it doesn't play. I'm using this btw https://github.com/2d-inc/HistoryOfEverything/tree/master/app/assets/Recorded_history
It just has a candle burning effect and this works in an Android Emulator but on my phone, it shows up but without animation that burning animation...
I just wonder if you have any idea why this not works on my phone. I didn't check the logs but i cant check if you need too..
NimaActor("assets/nima/Recorded_history.nma",
            fit: BoxFit.scaleDown, animation: "Untitled")

Okay i get the logs
I/flutter (27383): ══╡ EXCEPTION CAUGHT BY SCHEDULER LIBRARY ╞═════════════════════════════════════════════════════════
I/flutter (27383): The following NoSuchMethodError was thrown during a scheduler callback:
I/flutter (27383): The method 'advance' was called on null.
I/flutter (27383): Receiver: null
I/flutter (27383): Tried calling: advance()
I/flutter (27383):
I/flutter (27383): When the exception was thrown, this was the stack:
I/flutter (27383): #0      NimaActorRenderObject._advance (package:nima/nima_actor.dart:334)
I/flutter (27383): #1      NimaActorRenderObject._beginFrame (package:nima/nima_actor.dart:277)
I/flutter (27383): #2      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1012)
I/flutter (27383): #3      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleBeginFrame.<anonymous closure> (package:flutter/src/scheduler/binding.dart:928)
I/flutter (27383): #4      __InternalLinkedHashMap&_HashVMBase&MapMixin&_LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:367)
I/flutter (27383): #5      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleBeginFrame (package:flutter/src/scheduler/binding.dart:926)
I/flutter (27383): #6      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._handleBeginFrame (package:flutter/src/scheduler/binding.dart:856)
I/flutter (27383): #10     _invoke1 (dart:ui/hooks.dart:233)
I/flutter (27383): #11     _beginFrame (dart:ui/hooks.dart:172)
I/flutter (27383): (elided 3 frames from package dart:async)

i'm facing the same problem but mine the animation work sometimes and other doesn't, when it stopped i restart the app many times till it start playing again, tried release and it didn't work at all

@luigi-rosso Do you still maintain nima repo ?

I am facing the same issue. Please help

I am also having this issue, on my emulator and the physical device. NimaFlutter is basically unusable like this


On Access Protection

This is very unclear, and in InTune, the option is Block or Not Configured. Could someone please spend some time clarifying this?
Allows or disallows Windows Defender On Access Protection functionality.
ADMX Info:
GP English name: Monitor file and program activity on your computer
GP name: RealtimeProtection_DisableOnAccessProtection
GP path: Windows Components/Windows Defender Antivirus/Real-time Protection
GP ADMX file name: WindowsDefender.admx

The following list shows the supported values:
0 – Not allowed.
1 (default) – Allowed.

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

ID: 66b1ab2e-1964-5580-7c99-4bd4cd480075
Version Independent ID: f7b5da46-04cb-3501-65fa-870a3c6f09b8
Content: Policy CSP - Defender - Windows Client Management
Content Source: windows/client-management/mdm/policy-csp-defender.md
Product: w10
Technology: windows
GitHub Login: @ManikaDhiman
Microsoft Alias: dansimp

Hello @nathanmcnulty,
could you please clarify which intune option you are talking about?
as far as I can see "Monitor file and program activity on your computer" has values "yes" or "no"
Thank you

Hey @MaratMussabekov , there are two settings in InTune:

On the docs, I see this for the On Access Protection section:

Are the two settings in the images combined into the one On Access Protection CSP, or is the ADMX Info - English name information incorrect and the actual "Monitor file and program activity on your computer" CSP is missing from the page?
Also, in the On Access Protection image, I have an option of Block or Not Configured... What exactly am I blocking? And to make it more confusing, the docs for the defender security baselines says it sets it to Enabled with a default of Yes...

Hello @nathanmcnulty,
this naming is kind of confusing )
"Monitor file and program activity" feature from your first screenshot is RealTimeScanDirection from the original article, and is named "Configure monitoring for incoming and outgoing file and program activity" in GP
"On Access Protection" feature from your second screenshot is AllowOnAccessProtection from the original article, and is named "Monitor file and program activity on your computer" in GP
So yes, second screen shows it correctly. According to this article:
On Access Protection: Block prevents scanning files that have been accessed or downloaded. Users can't turn it on.
When set to Not configured (default), Intune doesn't change or update this setting. If you block the setting and then change it back to Not configured, Intune leaves the setting in its previously OS-configured state. By default, the OS enables this feature and allows users to change it.
Intune doesn't turn on this feature. To enable it, use a custom URI.
According to this, using intune configuration policy you can only turn this feature off or leave it as it was pre-configured.
Thank you

@nathanmcnulty  - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by MaratMussabekov so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!

I am confused how [On Access Protection] relates to [Real-time monitoring] ? Isn't the act of accessing a file (and protecting against it) part of real-time monitoring work Windows Defender must perform?


Show updated wiki buy/sell prices on slots tab items

Currently the slots interface leaves much to be desired.
If you have it open on the side, you constantly have to switch to the flipping tab to make sure your prices are relevant.
In this commit I copied the wiki buy/sell price lines from the flipping tab to the slots tab.
This will probably require additional changes like user customization of whats shown, and perhaps some refactor to consolidate doubled code between the slots and flipping tab files.

Done on master


JSON issue with run() parameters

Some kind of JSON issue when passing the following params to run() with agents that have integers for "name":
results_baseline = jobs.run(n = 1, disable_remote_inference = True, progress_bar = True, cache = False)
Issue does NOT occur when no params are used or when agents have string "name".
Compare runs:
https://www.expectedparrot.com/content/df8ccbf7-340c-4032-a83b-cc72d8ecca8d
--> 180     raise TypeError(f'Object of type {o.__class__.__name__} '
    181                     f'is not JSON serializable')

TypeError: Object of type int64 is not JSON serializable

The issue also goes away when this param is removed: disable_remote_inference=True


Set Kanban columns and cards

Hi,

link of project: https://github.com/VuDej/Capstone-2-JavaScript/projects/3
team members 2

Hi Team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


chore(deps): update etcd-druid to v0.25.0 (minor)

This PR contains the following updates:

Package
Type
Update
Change

gardener/etcd-druid

minor
v0.24.1 -> v0.25.0

github.com/gardener/etcd-druid
require
minor
v0.24.1 -> v0.25.0

Release Notes

gardener/etcd-druid (gardener/etcd-druid)
v0.25.0
Compare Source
[gardener/etcd-druid]
📰 Noteworthy

[OPERATOR] Etcd ConfigMap Naming Update: The naming convention has changed to {etcd.Name}-config for consistency, replacing etcd-bootstrap-. Unused old ConfigMaps will be removed in v0.27.0 by @anveshreddy18 [#812]
[USER] The UseEtcdWrapper feature gate has been promoted to GA and locked to true. by @shreyas-s-rao [#936]

🏃 Others

[DEVELOPER] Enhanced the check make target to ensure the Go version adheres to x.y.0. by @renormalize [#930]
[OPERATOR] Upgrade the alpine image version to 3.20.3 by @renormalize [#916]
[OPERATOR] Enhances Etcd configuration by organizing ConfigMap naming convention, enabling snapshot-count configuration, and rectifying URL issues for improved functionality and consistency by @anveshreddy18 [#812]

[gardener/etcd-backup-restore]
⚠ Breaking Changes

[USER] The etcd configuration parameters initial-advertise-peer-urls and advertise-client-urls now accept a structured YAML format where URLs are listed per etcd instance. This replaces the previous unstructured string format with @ as a separator. Existing configurations must be updated to match this new format. To know the new format in detail, check the example config file at pkg/miscellaneous/testdata/valid_config.yaml by @anveshreddy18 [gardener/etcd-backup-restore#715]

📰 Noteworthy

[OPERATOR] Snapshots garbage collection performed by etcd-backup-restore (if enabled) for ABS is performed only when the objects' immutability period expires. by @renormalize [gardener/etcd-backup-restore#781]
[OPERATOR] etcd-backup-restore now supports immutable objects for storage provider: Azure Blob Storage, provided by the Container Level WORM feature. by @renormalize [gardener/etcd-backup-restore#781]
[OPERATOR] Support for Container level WORM (ABS) in etcd-backup-restore is backward compatible. For more info please refer to this doc: https://github.com/gardener/etcd-backup-restore/blob/master/docs/usage/immutable_snapshots.md by @renormalize [gardener/etcd-backup-restore#781]

🏃 Others

[DEVELOPER] Enhanced the check make target to ensure the Go version adheres to x.y.0. by @renormalize [gardener/etcd-backup-restore#803]
[OPERATOR] Don't skip full snapshot, always trigger a full snapshot independent of new updates to etcd, preventing prometheus alerts of not taking a scheduled full snapshot. by @Shreyas-s14 [gardener/etcd-backup-restore#804]

Docker Images

etcd-druid: europe-docker.pkg.dev/gardener-project/releases/gardener/etcd-druid:v0.25.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

Release note:
The `gardener/etcd-druid` image has been updated to `v0.25.0`. [Release Notes](https://redirect.github.com/gardener/etcd-druid/releases/tag/v0.25.0)

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: undefined
Post-upgrade command 'make generate MODE=sequential' has not been added to the allowed list in allowedPostUpgradeCommands


Import new officers to the officers table in PROD

Business Context:
Use the information in the attached officer CSV below to perform an update on the officer table in prod.
Additional context
Some time ago last year, there was an issue with the officers table where the data had a lot of duplication. In order to create a new case, there needed to be a lot of workarounds in order to find the correct officer. This card ensures that the correct data is uploaded to the table.
In Scope

Make a backup of the officers table before importing the CSV
SKIP THIS STEP * Set the start date of the search officers functionality
SKIP THIS STEP  * Start date should be 1 day before we are doing the upload
SKIP THIS STEP  * environment variable name PRODUCTION_OFFICER_ROSTER_LATEST_DATE
SKIP THIS STEP   * The time is in epoch format (seconds) https://www.epochconverter.com/
Update officers table in production for NOIPM with the attached CSV
Test it works (log into prod or meet w/ Christian)

Link for file: https://github.com/PublicDataWorks/complaint-manager/files/13746563/Officers.CSV.-.Sheet1.csv
Notes

[x] make copy of CSV from NO Prod
[x] Set Date to 1/30/2024 in epoch time
[ ] Update officers table in production for NOIPM with the attached CSV
[ ] Make sure append new officers into existing table with old info

Checklist
Kickoff

[ ] We have we indicated which organizations and user permissions require this
[ ] We have indicated whether we have any dependancies to other cards/work
[ ] We have indicated which environment this can be tested in
[ ] We have included a mockup if this is design related
[ ] We have included clear ACs to test it
[ ] We have estimated the story

Deskcheck

[ ] This work has been demoed in staging (or higher) and led by the dev pair who worked on this card
[ ] We have run through and passed all the ACs
[ ] Our documentation has been updated
[ ] Our onboarding checklist has been updated

Download the officers table from prod and import it into local
Figure out the Update SQL command
Create a new table with the new_officer_roster.csv
Update officers using new_officer_roster (?)
Drop new_officer_roster
Run it
Fix errors
Do it in prod!

Update: Upon further investigation, I saw NOIPM included a new value for employee_type, "Civilian". In the past, employee_type was an enum that only accepted the values "Commissioned", "Non-Commissioned", or "Recruit".
I'll need to write a migration to allow employee type to accept the value "Civilian". I'll write this migration to update employee_type in both the officers tables and cases_officers table.

In order to update the prod database with the new roster, we need to drop the uniqueOfficerNumber constraint from the officers table.
We've got to do this because in our new roster windows username was just copied and pasted into the officer number field, and coincidentally some people's windows usernames is the same as other people's officer number, thus violating the constraint.
We're going to delete the constraint, which shouldn't be a big deal because we're dropping the entire officer number column in future cards.


Create dedicated SecondaryNav component

Disentangle expandables from our standard secondary nav and create a dedicated component.
Additions

Create dedicated SecondaryNav component.

Changes

Shorten secondary-navigation name to secondary-nav.
Merge nav-link component macro into secondary nav templates.
Remove top border from secondary nav

How to test this PR

Secondary navs on the following should still function the same as prod:

http://localhost:8000/data-research/small-business-lending/filing-instructions-guide/2024-guide/
http://localhost:8000/rules-policy/regulations/1002/
http://localhost:8000/paying-for-college/your-financial-path-to-graduation/
http://localhost:8000/enforcement/payments-harmed-consumers/

👍  This does not appear to break any of Paying for College's Financial Path to Graduation tool!

@anselmbradford I brought over the failing regression tests from https://github.com/cfpb/consumerfinance.gov/pull/7738. I'll see if I can get them green.

🫗 for @brandon-rusk's last PR approval


Indigo release items

The following is a quick list of the items that must be addressed before transitioning to Indigo:

[ ] Update yaml parsing to new yaml library API
[ ] CP 1616 driver does not compile under Ubuntu 14.04

Regarding the yaml-cpp changes: if there are no significant functional differences in the rest of the package between the Hydro and the Indigo version, it would perhaps make sense to localise the impact of the changes (ie: move yaml parsing to some helper functions) and use something like:
if(NOT ${YAML_CPP_VERSION} VERSION_LESS "0.5")
add_definitions(-DHAVE_NEW_YAMLCPP)
endif(NOT ${YAML_CPP_VERSION} VERSION_LESS "0.5")

in the CMakeLists.txt together with some conditional inclusion to keep the package compatible with both Hydro and Indigo.


Use tabs when saving user configs

Make it nicer, IMO. Also makes it consistent (it saved with single space in one place).

👍 and merging.


Upgrade keycloak to v18

Align upstream keycloak release with RHSSO 7.6 (based on v18)
Targeted for 2.1.2

References in RHSSO 7.6 announcement :
RHSSO 7.6

@fbladilo thanks for the heads up. In the UI we are using the keycloak-js client npm package published by keycloak, so hopefully all we'll need to do is upgrade that dependency to match. I think the actual redirect_uri param is abstracted away.

@fbladilo thanks for the heads up. In the UI we are using the keycloak-js client npm package published by keycloak, so hopefully all we'll need to do is upgrade that dependency to match. I think the actual redirect_uri param is abstracted away.

No problem, hopefully is straightforward , it is easy to reproduce by just attempting to log off a user and should see the error.

@fbladilo, @mturley, the redirect_uri is not used by tackle2-ui and neither by tackle 1 at least when we took over.


How can i change the header row?

Hello! By default the header row it's the first.
But in the document i have, the rows in the header are 7 and the row where i have the values is the 7. SO how can i change it? With sheet.setHeaderRow() i set the value so how can i change it?
Thank you

closing to combine with #407


syncthing-inotify is deprecated

Issue description
Apparently syncthing-intofy is now deprecated[1] and its functionality included in the main syncthing package. I'd do a PR to "fix" this in nixpkgs myself, but I'm not sure how to deal with this kind of situation. For now syncthing-inotify still works, but its probably a good idea to switch to the "blessed" solution before 18.03.
@peterhoeg
[1] https://github.com/syncthing/syncthing-inotify

Indeed it is. The nice way to handle this would be something like:
a) drop the -inotify package
b) add an "ExecStartPre" script that runs sed against the config file if useInotify is enabled and replaces fsWatcherEnabled="false" with fsWatcherEnabled="true".
c) update the description for useInotify option to reflect the situation
If you take a stab at a PR, please ping me on it and I'll help out.

a) drop the -inotify package

Won't just dropping thep ackage lead to errors while updating?

b) add an "ExecStartPre" script that runs sed against the config file if useInotify is enabled and replaces fsWatcherEnabled="false" with fsWatcherEnabled="true".

Its a bit scary to just overwrite those options.
If somebody has enabled this for some folders and disabled it for others, his config will be irrevocably lost once he triggers useInotify.
But maybe its still the best option.

As a user of that package I rather have that option deprecated with having to enable fsWatcherEnabled myself (and an explanation how to do that).
I personally would find it more annoying to have my config touched by NixOS when I don't expect it, than to have the inotify feature turned off after switching to a new channel.
I think it would be very confusing if only this one option was controlled by a NixOS option.

Synthing apparently plans to offer an option that does basically the same: Simply set the fsWatcherEnabled Option true for all folders.
Since its also an option that you'd generally set for all folders anyways (at least I'd think so), maybe its a good idea in this case. It would probably be less confusing than just having useInotify be a noop. Then people would think inotify is used when it is not anymore.
What do you think @Moredread ?

A noop would be worse, you're right.

Please keep in mind that the nixos module only kicks in for the "run syncthing as a system service" and not for any normal users so we are only going to modify user config files if:
a. services.syncthing.enable and services.syncthing.useInotify are both true, and
b. A user runs another copy of syncthing in parallel to the system-wide instance and uses the systemd unit
For the syncthing as a system service which is what we configure using the nixos module, it makes sense that useInotify actually does enable inotify.

I just tried to implement this, but ran into difficulties. I'm running the syncthing module as a user, configured like this:
services.syncthing = {
    enable = true;
    useInotify = true;
    user = "timo";
};

Which means that the config file from my home directory is used. So we can't count on the config file being global. I think to correctly locate the config we'd pretty much have to implement the whole xdg logic.

Unfortunately I don't use syncthing anymore and maybe I'm forgetting something, but couldn't we just remove useInotify with mkRemovedOptionModule and add a notice there how to enable inotify? This will print a warning on evaluation that useInotify won't have an effect anymore. This should then also be mentioned in the release notes.

I didn't know about mkRemovedOptionModule, that does seem like the best option.
But if we go that route we should wait for the next synthing update because I think the new "native" way is not yet accessible in the user interface.

I think the new "native" way is not yet accessible in the user interface.

I'm not sure if this is the final place for turning on/of inotify, but it is available now under Settings -> Advanced -> [Your folder] -> FS Watcher Enabled (checkbox)

Which means that the config file from my home directory is used. So we can't count on the config file being global. I think to correctly locate the config we'd pretty much have to implement the whole xdg logic.

Referring to %h/.config/syncthing/config.xml in the context of a systemd unit should refer to the right file. But I'm learning towards @fpletz's suggestion here - this way there are no surprises.

Referring to %h/.config/syncthing/config.xml in the context of a systemd unit should refer to the right file.

Really? Wouldn't the config be in /etc/xdg or something like that if it runs under root?

But I'm learning towards @fpletz's suggestion here - this way there are no surprises.

And how should we deal with the synthing-inotify package then? Is there somthing similar to mkRemovedOptionModule for packages?

Wouldn't the config be in /etc/xdg or something like that if it runs under root?

Possible, but nobody should be running this as root.

And how should we deal with the synthing-inotify package then?

We cannot deal gracefully with someone adding it to environment.systemPackages but if they doservices.syncthing.useNotify it's pretty straight forward.

I've deprecated useInotify  and removed syncthing-inotify in #35432.


Add PORT option for specify webserver and jupyter notebook ports

This PR allows PORT to be specified in the Makefile.
This is useful for jupinx to specify port 8890 for launching webserver and jupyter notebooks servers so there is no inadvertent collision with a users other running instances.

@AakashGfude once you review this PR -- can you propagate to quickstart template in addition to lecture-source-jl?


Syncing viewport from server

This is not a bug, more of a question. I have the following code:
var connectOptions = new ConnectOptions()
{
      BrowserWSEndpoint = "ws://..."
};

var browser = await Puppeteer.ConnectAsync(connectOptions);

var pages = await browser.PagesAsync();

foreach (var page in pages)
{
       JObject response = await page.Client.SendAsync("Performance.getMetrics");
       Console.WriteLine("Response is " + response.ToString());
}

To my surprise, as soon as the command ran, it resized the viewport to the default (800,600). My expectation would be that if a viewport was already set on the server, it shouldn't modify it. If there is already a viewport set on the server, the PuppeteerSharp state should be synchronized to match it. Thoughts?

You could try setting the DefaultViewport to null in the ConnectOptions.

I completely forgot to mention that this is what I did and it did work, but the problem becomes that if I try to do page.Viewport, it is null. Is there a way to "sync" that so it fetches the server value?

@chitoiur you could try to get that using a javascript code and calling EvaluateFunctionAsync

Thanks! Closing this out.

this was helpful to me, thanks! hopefully this becomes better in the future


sched: Properly account for timeslicing in tickless mode (take two)

The first version of this patch caused a bunch of failures on ARM hardware platforms because, apparently, of the change of _ready_thread() from an inline to a proper function.  I don't have an answer for why that happens and am continuing to investigate.
But this patch seems to work (it really should!  It's a proper noop on builds without CONFIG_TICKLESS_KERNEL) and fixes the original bug.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #9697   +/-   ##
=======================================
  Coverage   52.31%   52.31%
=======================================
  Files         212      212
  Lines       25937    25937
  Branches     5589     5589
=======================================
  Hits        13569    13569
  Misses      10122    10122
  Partials     2246     2246

Impacted Files
Coverage Δ

kernel/include/ksched.h
94.25% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Standard merge

Enabled git_merge (NodeGit.Merge() and NodeGit.Merge.merge) and added a convenience method.
git_merge is the standard merge that merges into HEAD and writes the MERGE_* to the git repo which is the indicator that the repo is currently in a merge state. (git_repository_state looks for those files to determine if it's in a merge state)
Other merge methods, such as Merge.commits don't actually leave the repo in a merge state

"hey, tests passed"


Searching In Activities With ElasticSearch

Here is my activities controller where I list all the activities in an admin panel:
admin/activities_controller.rb =>
class Admin::ActivitiesController < ApplicationController
  def index
    @activities = PublicActivity::Activity.includes(:trackable, owner: [:contact, :profile]).paginate(page: params[:page], per_page: 10).order(created_at: :desc)
  end
end

I would like to add searching support for activities in the same page. ElasticSearch is installed through my application. In a typical model I include the elasticsearch like this:
class SomeModel < ActiveRecord::Base
  include Elasticsearch::Model
  include Elasticsearch::Model::Callbacks
end

The problem is, publicactivity doesn't have a model file. How can I include elasticsearch to publicactivity?

You can extend the PublicActivity::Activity model by using class_eval. Showed here: https://github.com/chaps-io/public_activity/wiki/[How-to]-Use-custom-fields-on-Activity#setup


Getting "An exception occurred while trying to process the frame: list index out of range" error

Getting "An exception occurred while trying to process the frame: list index out of range" error

i've got the same error


Track invalid usage around record

Closes #18

@Delgan if you have a time, would you mind checking if that goes in right direction that aligns, well, perfectly with runtime behavior?

Hi @kornicameister, I'm once again very impressed by your work and the static checks that can be generated!
I noticed a false positive:
logger.opt(record=True).info("Record: {record[time]}")

The key does not need to be surrounded by ' nor ". Doing so actually generates a KeyError while running the Python program.
Another false positive happens when using nested keys (although as I said, it also raises an exception due to the surrounding '):
logger = logger.bind(ip="<IP_ADDRESS>")
logger.opt(record=True).info("Current ip: {record['extra']['ip']}")

Also I noticed an internal fatal error while running the mypy plugin against this code:
from loguru import logger

opts = {"record": True}
logger.opt(**opts).info("...")

test.py:5: error: INTERNAL ERROR -- Please try using mypy master on Github:
https://mypy.rtfd.io/en/latest/common_issues.html#using-a-development-mypy-build
Please report a bug at https://github.com/python/mypy/issues
version: 0.782
Traceback (most recent call last):
  File "mypy/checkexpr.py", line 3761, in accept
  File "mypy/nodes.py", line 1544, in accept
  File "mypy/checkexpr.py", line 263, in visit_call_expr
  File "mypy/checkexpr.py", line 340, in visit_call_expr_inner
  File "mypy/checkexpr.py", line 817, in check_call_expr_with_callee_type
  File "mypy/checkexpr.py", line 876, in check_call
  File "mypy/checkexpr.py", line 988, in check_callable_call
  File "mypy/checkexpr.py", line 725, in apply_function_plugin
  File "/home/delgan/Programmation/loguru-mypy/loguru_mypy/__init__.py", line 250, in _loguru_opt_call_handler
    lazy=NAME_TO_BOOL[lazy_expr.name] if isinstance(
KeyError: 'opts'

Another false positive happens when using nested keys (although as I said, it also raises an exception due to the surrounding '):
logger = logger.bind(ip="<IP_ADDRESS>")
logger.opt(record=True).info("Current ip: {record['extra']['ip']}")

@Delgan so it makes sense to also track bind calls. Hmm...on its own that's an interesting problem to solve. How to know if logger was first configured with bind, later with record and even later with lazy. Somehow, I'd have to accumulate those configurations leading to final logger that is being tracked.

That seems very ambitious (read, crazy) to consider monitoring all those parameters! :stuck_out_tongue_winking_eye:
Surely that would be a good way to learn and experiment even more with mypy plugins, though.

ha...that dict unpacking is sort of crazy but valid thing :D nice catch !

Closing this down for now. This codebase is far away from current master and will be better to revisit entire concept later on.

Closing this down for now. This codebase is far away from current master and will be better to revisit entire concept later on.


Question: Does the CfCT Pipeline trigger upon account creation/updates made in Control Tower?

Describe the bug
Currently, I have implemented CfCT based on the most current implementation hosted in this GitHub repo as of 07/20/2023. After completing the implementation, I tested an account update within a demo account that is targeted within the manifest file, and was hoping to see the CodePipeline rerun, but did not see any activity. Although, after manually creating a rule in EventBridge to invoke the Pipeline after seeing an account update and/or creation, I was presented with expected behavior. So my question is, are account updates and creations meant to trigger CfCT, applying all customizations to the new/updated accounts? If so, is this by default or are there best practices to implement this? Is adding the EventBridge Rule in the customizations-for-aws-control-tower.template resources section a viable way to do this?
To Reproduce
Deploy CfCT, add customizations to manifest file, deploy account update/creation.
Expected behavior
CfCT would run against newly updated account by default.
Please complete the following information about the solution:

[ ] Version: v2.6.0

To get the version of the solution, you can look at the description of the created CloudFormation stack. For example, "(SO0089) - customizations-for-aws-control-tower Solution. Version: v1.0.0". You can also find the version from releases

[ ] Region: us-west-2
[ ] Was the solution modified from the version published on this repository? No
[ ] If the answer to the previous question was yes, are the changes available on GitHub?
[ ] Have you checked your service quotas for the sevices this solution uses?
[ ] Were there any errors in the CloudWatch Logs? No

Screenshots
If applicable, add screenshots to help explain your problem (please DO NOT include sensitive information).
Additional context
Add any other context about the problem here.

@eambriz27 thank you for reaching out.
Please may you elaborate on what updates are performed on the targeted account?

The only update that was made was the update to Landing Zone version 3.1 @snebhu3

CFCT triggers the customization pipeline when:

A manifest update is made (either in CodeCommit or S3)
When a Control Tower CreateManagedAccount event occurs.

See: https://github.com/aws-solutions/aws-control-tower-customizations/blob/main/customizations-for-aws-control-tower.template#L3134
Note that the pipeline does not trigger on UpdateManagedAccount events - that feature request is tracked via https://github.com/aws-solutions/aws-control-tower-customizations/issues/173


typo

Sorry for creating a bug report on this but I'm sure you want this corrected eventually.
The Github description of this repo has a typo:
"Multitiered file storage API built on Filecon and IPFS "

@LukeWheeldon , thanks for pointing that out!


Create transaction table and remove earning and spending

Like earning and spending are similary. It will be good thing to merge the two one on only one table : transaction

Négative value will be spending
Positive value will be earning
Like that, the spending can have tag too
That will simplify the logic of index controller for transactions

Maybe first make an overview how to handle this merge-operation by answering the questions down below.

How do cooperate with existing earnings/spendings
Do we want to merge the Controllers/Repositories for earnings/spendings
How do we recognise earnings vs spendings in the Transaction-table? By using a field (preferably an enumerator) type or by determining it via the amount-field?

Thanks for your feed back, in my opinion :

We can create a new table named "transactions" with the same columns as spendings (the two one are simily, just tag_id is missing in earning) and make a copy of earnings and spendings into it. After that, remove the earnings/spendings table. All of that on a migration.
I think it will be good to merge the two one yes, or rework on it for simplify. On place where we try to find earnings only or spendings only, we have just to add filter on amount negative or positive
I have spoiled on last point, i think the more simple is to determining it via amount no ? It's explicit, you can't earn negative amount ? i dont see case when type column will be necessary, but i can miss a usecase maybe you have an example ?

What is your opinion ?

Your answers seem okay to me.
Regarding the last question, I haven't checked how the system corporates when filling in negative amounts right now, but for code readability I thought that using a type-column makes it easier to filter instead of determining it via the amount. You store the type explicitly.

We can make like that if it's more explicit
And like you suggest, we can use a enumeration. Or... maybe tinyint ? 0 for earning, 1 for spending and constant in code explain that ?
class TransactionRepository
{
  const TYPE_EARNING = 0;
  const TYPE_SPENDING = 1;

  public function fakeMethodForFetchSpendings()
  {
    //it's an example
    return $query->where("type", static::TYPE_SPENDING)
  }
}

Enum is more readble for users in database. But if i'm not talking nonsense i thing it's better for performance to use tinyint no ?

Tinyint looks okay to me. I don't know what the performance penalty will be when using enum over tinyint, but logically thinking, enum will definitely use more memory so it might as well be slower

Exactly, me too i don't know where it's better to use tinyint but for memory it's sure tinyint will be better


QuantumObjective function: Pass a symbol or function for Loss

Currently, the QuantumObjective function in objectives.jl takes a loss::Symbol which is called internally using eval(loss). This makes it hard for the user to define a local loss function, which might be a common task. Can the quantum objectives function take a loss function instead of symbol? In this alternative, the user passes an existing loss function fromlosses.jl instead of a symbol.
What are the tradeoffs here?

the reason for passing a symbol and not a function (currently) is to allow problems to be saved with the objective.  julia does not allow saving of functions so the way around this was to make the argument a symbol that evals to a function defined in losses.jl that can be saved and used to reconstruct the objective during a load.  modifying the code to include a user-defined loss function should be pretty straight forward -- basically we just need to define a new Objective constructor that accepts a function (or potentially a Loss <: AbstractLoss defined on line 61 of losses.jl; though this was primarily meant for quantum state (or unitary) losses though). saving and loading should be modified to handle this case.  unfortunately tests don't yet exist for this part of the code.


Applicant can view list of their past applications

And then view, delete, and "takeout" their data.
See tiny.cc/cvf-ux-tracker for more details.

whoops, nope, just noticed this is ring 4.


Let's just deprecate SI-1.2. It's the only Peppol BIS V2 based (Invoice) format left on the earth.

Originally posted by @ri4a in https://github.com/phax/phive-rules/issues/9#issuecomment-1000532587
And yes, I know the NPa is working on this.

Finally it was deprecated and marked as removed in OpenPeppol:
https://docs.peppol.eu/edelivery/codelists/v8.7/Peppol Code Lists - Document types v8.7.html#75
Must not be used by 2024-01-01

AFAICT after 2024-01-01 its use will be considered non-compliant by the NPA.


[Question] OAuth card customization

I have two questions for web chat OAuth card

When clicking to sign in, user will redirected to a new browser tab. Can this be changed to small pop-up window instead?
After user clicks ‘accept’, user will be asked to enter a validation code in the chat window to continue. Is this also a necessary step? Any way to make it more concise?

This will need a window.open ponyfill, which is not available yet. Please track this in #368. You will need to adjust how Web Chat handles URLs differently with the ponyfill.
Please refer to the comments on https://github.com/Microsoft/BotBuilder/issues/4632#issuecomment-441957719 to create a smoother experience without the validation code.


Add RTFD badge

Fixes #5 :)

Coverage remained the same at 100.0% when pulling 2c636ce1170628a7ecfd6af0ea1ba2e6ed24c9c4 on docs/rtfd into f4b4e7543468cb99b8c82378e3386c1a0ddc2486 on master.

Coverage remained the same at 100.0% when pulling 2c636ce1170628a7ecfd6af0ea1ba2e6ed24c9c4 on docs/rtfd into f4b4e7543468cb99b8c82378e3386c1a0ddc2486 on master.

Coverage remained the same at 100.0% when pulling 2c636ce1170628a7ecfd6af0ea1ba2e6ed24c9c4 on docs/rtfd into f4b4e7543468cb99b8c82378e3386c1a0ddc2486 on master.

Coverage remained the same at 100.0% when pulling 2c636ce1170628a7ecfd6af0ea1ba2e6ed24c9c4 on docs/rtfd into f4b4e7543468cb99b8c82378e3386c1a0ddc2486 on master.

Coverage remained the same at 100.0% when pulling 2c636ce1170628a7ecfd6af0ea1ba2e6ed24c9c4 on docs/rtfd into f4b4e7543468cb99b8c82378e3386c1a0ddc2486 on master.


i got that error

Could not build the precompiled application for the device.
Error (Xcode): ../../.pub-cache/hosted/pub.dev/photo_manager-2.7.2/lib/src/internal/image_provider.dart:72:28: Error: The parameter 'decode' of the method 'AssetEntityImageProvider.load' has type 'Future Function(ImmutableBuffer, {TargetImageSize Function(int, int)? getTargetSize})', which does not match the corresponding type, 'Future Function(Uint8List, {bool allowUpscaling, int? cacheHeight, int? cacheWidth})', in the overridden method, 'ImageProvider.load'.
Error launching application on iPhone 14 Pro.

Please try the new 2.2.0 version


Contexts in reports are missing with relative_files=True

Describe the bug
When I enable relative_files the json and html reports no longer contain context information. I still see context info in the sqlite DB though.
To Reproduce

coverage.py 5.0, py3.7.5
Set relative_files=True
python3 -m pytest --cov --cov-context=test tests/test_foo.py
python3 -m coverage html --show-contexts
HTML report doesn't include any context infos

Expected behavior
HTML report contains context infos

I'll mark this a dup of #900.

This is now available in coverage==5.0.1


Add a test case to the zookeeper app that actually connects to the ensem...

...ble and initiates a session.

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/184/
Test FAILed.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/185/
Test FAILed.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/186/
Test PASSed.

Merged.


Use EmbeddedActiveMQ in messaging and jta integration tests

Since we're using an out-of-date & deprecated image, we should use the EmbeddedActiveMQ in ActiveMQTestResource and could inherit artemis-server from the quarkus-bom and that way we don't have to worry about additional maintenance on our side.

The quarkus bom does not include org.apache.activemq:artemis-mqtt-protocol, so I just raise https://github.com/quarkusio/quarkus/issues/17720 to add it. All the tests should be OK.

https://github.com/quarkusio/quarkus/pull/17721 was merged.

Great - and is it possible to take these changes to quakrus-main branch first to see if it works ? But I find the "Daily Build Quarkus main" is still failing.

Great - and is it possible to take these changes to quakrus-main branch first to see if it works ?

Yes, please feel free to send a pull request against quarkus-main

But I find the "Daily Build Quarkus main" is still failing.

Sorry about that, it is was my fault. I have fixed the branch now.

https://github.com/apache/camel-quarkus/pull/2751 was cherry-picked to quarkus-main


Adobe Reader gives an error after using fix-qdf on files with removed objects

I'm not sure if this is a bug in fix-qdf or if the manual is missing something. When I try to remove a reference to an object in a QDF file, I can create a PDF file and it looks fine in other tools. In Adobe Reader I get a warning that the file is broken.
The manual says: "It is not generally practical to remove objects from QDF files without messing up object numbering, but if you remove all references to an object, you can run qpdf on the file (after running fix-qdf), and qpdf will omit the now-orphaned object."
So, I attach an example PDF file that I've converted to QDF.
test-qdf.pdf
In this file, I can remove the red image (object 15) by removing the reference /REI0 15 0 R. After that, I run fix-qdf and can send the file through qpdf without warnings.
When I open the resulting file in Adobe Reader, it says that there's an error in the file. The same file can be opened in other PDF readers without errors.
The same thing happens if I remove the contents from object 15 as well or if I just remove the stream.
QPDF version: 10.0.1

Removing the reference to the object in the page dictionary is in fact sufficient to make qpdf drop object 15, but /REI0 is pointing to an image, and that image is being drawn in the content stream for the page. In line 129 of the file, which is the 18th line of object 6, the contents for page 1, you will see
/REI0 Do

which is drawing the image. If that line remains but the object and reference to the object in the resource dictionary are removed, the file is broken because the content is referencing an unknown object. qpdf does not validate that case as it generally doesn't know how to interpret content streams. Other readers are probably silently ignoring the non-existent resource reference.
You could either remove just that one line, or you could remove the block that draws the image, which starts at the preceding q and ends at the following Q. In this case, that would be
q
0.00000 0.00000 m
70.00000 0.00000 l
70.00000 -71.00000 l
0.00000 -71.00000 l
0.00000 0.00000 l
h W* n
72.00000 0 0 72.00000 0.00000 -72.00000 cm
/REI0 Do
Q

That code is just creating a clipping region and rendering the image within the clipping region. You could remove that entire block as well as the reference /REI0 15 0 R, then run fix-qdf, then run qpdf, and the resulting image should work with no errors in Adobe Reader.

By the way, thanks for including a test file, detailed information about what you were doing, and the version of qpdf you are using. :-)

Ok, thank you. I'll have to see how this relates to other PDF files, and if I can recognize the parts to remove.
Would it be possible to make qpdf or fix-qdf automatically remove these unused resources in a future version? It might be helpful enough to just get a warning when running the commands.

It wouldn't be appropriate for fix-qdf to remove them -- fix-qdf is explicitly designed only to update cross reference tables and lengths. In fact, I use qdf all the time to create erroneous files for testing. However having qpdf --check detect and warn about this seems like a good idea. I've opened #466 to track this. I don't have much time to work on qpdf, so I doubt this will happen anytime soon, but it's worth tracking as a possible enhancement. Thanks for the suggestion.

Ok, thanks. I guess you can close this bug report now.


add svc cidr in ovs LB for optimization

As an automated optimisation patch for #1166
kubeovn >= 1.9

The commits need to be squashed


[feat(model)]: Support MoE layer

for issue2250

please follow the instructions to format the code.


Small updates to UI (search type), update README.md and minor version…

Hi Girish,
Updated after review.
Jpg was already removed since it was updated in README.md.
Sure, tests would be best to have for each new feature. (But I am not so sure how a test for search would work, to actually prove something meaningful. Do you ?) I will think about adding some in the future.

Hi Girish,
Updated after review.
Jpg was already removed since it was updated in README.md.
Sure, tests would be best to have for each new feature. (But I am not so sure how a test for search would work, to actually prove something meaningful. Do you ?) I will think about adding some in the future.

Hi Adi, All seems to be okay except for the seachnodes set in each of the search criteria.
Also, we should increment the version in setup.py. Could you please do that also.

Hi Adi, All seems to be okay except for the seachnodes set in each of the search criteria.
Also, we should increment the version in setup.py. Could you please do that also.

Updates performed

Updates performed


Bug: All heading styles aren't being applied

Describe the bug
All of our headings aren't retaining their font-weights and font-sizes. I'm not entirely sure, but it might be due to a dependency update to the site. Possibly one relating to PostCSS or Tailwind. I think I saw an email about one of these.

@fvcproductions This is now fixed. It seems we never labeled font-sizes and etc explicitly to the headings.
The reason they all got small was because a CDN'd version of Tailwind was included in the head of the site somewhere in the past. This forced an automatic V.1 update onto our site and included an updated version of Normalize.css via the Tailwind CDN that added "display: block" to images by default, normalized all our headings to the same size, and etc. The CDN was removed as it shouldn't have been included in the first place.

fixed in #134


Store cosmetic attributes

Description
One way of doing #917, not sure it's the best approach for all use cases
Checklist

[x] Add tests
[x] Lint
[ ] Update docstrings

@SimonBoothroyd Just tagging you for visibility - Matt will merge in 5 days, so please take a glance and comment if you have time before then!


Update helpers.ts for Lattice1 Hardware Wallet Support

Background
The Lattice1 is GridPlus' next generation hardware wallet. It is an always-online device with physically separated compute environments which provide unmatched security. For more information, see https://gridplus.io/lattice
About this PR
This PR adds the Lattice1 as a wallet option for bnc-notify. The Lattice1 is supported in v1.14.0 which currently used by Bancor Swap.
Note that a (nearly) identical integration was added to Curve. That pull request has several photos documenting what it is like to connect to and use an app with the Lattice1 and bnc-notify.
Thanks and please let us know if you have any questions!

@H34D can you ensure this is using the right RPC configured and nurse it through?

@Velua Thanks for taking a look at this - I went ahead and updated the RPC URL after reading your comment and double checking. Please let us know if you need any other info - thanks!

@H34D can you ensure this is using the right RPC configured and nurse it through?

i will take a look today

merged, thanks for contributing!


How to perform JOIN across many to many relation?

Describe the bug or question
I have 2 models, A and B, in M2M relationship linked via an ABAssociation table. The documentation explains a way to perform left and outer joins using JoinConfig. It lacks details on how to proceed with M2M relationships. Is that possible?
To Reproduce
Please provide a self-contained, minimal, and reproducible example of your use case
class A(SQLModel, table=True):
    id: int | None = Field(primary_key=True, index=True)

class B(SQLModel, table=True):
    id: int | None = Field(primary_key=True, index=True)

class VideoSearchAssociation(SQLModel, table=True):
    a_id: int | None = Field(primary_key=True, foreign_key="a.id")
    b_id: int | None = Field(primary_key=True, foreign_key="b.id")

Description
Screenshots
Additional context

Hey, @JakNowy, it works if you do this:
join_condition_1 = A.id == ABAssociation.a_id
join_condition_2 = ABAssociation.b_id == B.id

joins_config = [
    JoinConfig(
        model= VideoSearchAssociation,
        join_on=join_condition_1,
        join_type="inner",
        join_prefix="association_"
    ),
    JoinConfig(
        model=B,
        join_on=join_condition_2,
        join_type="inner",
        join_prefix="b_"
    )
]

records = await crud_a.get_multi_joined(
        db=async_session,
        joins_config=joins_config,
)

Naturally, It should not work for get_joined, it's a get_multi_joined situation by default. I'll add this in the docs.

I noticed that the count is wrong in get_multi_joined for Many-to-Many relationships (the data is fetched correctly, but the total_count is wrong), but I wrote a fix already. I'm finishing the documentation on Many-to-Many and writing the tests and I'll update it later today.

Docs added here


CSV with tab-chars in a cell: Whole row is parsed as single cell

When parsing a CSV-file that contains a tab-character (in this case within a cell), the whole row is used as value for the first cell in the row, even though comas and quotes exist.
I have attached the example file ('test.csv') as test.csv.txt.
In order to reproduce this, please rename the file to 'test.csv' and then run the following snippet
var xlsx = require('xlsx')
console.log(JSON.stringify(xlsx.readFile('test.csv'),0,2))

In my case (with version 0.10.8) the (incorrect) output is
{
  "SheetNames": [
    "Sheet1"
  ],
  "Sheets": {
    "Sheet1": {
      "A1": {
        "t": "s",
        "v": "cellA1\",\"celA2"
      },
      "A2": {
        "t": "s",
        "v": "value with\ttab\",\"value without tab"
      },
      "!ref": "A1:A2"
    }
  }
}

We have a (rather hacky) workaround for this:

Call read once
Check if it is CSV (only Sheets and SheetNames appear as keys in the result)
Check if the data contains a tab character
Remove the tabs and call read again

This works short-term, but it is pretty ugly. I just wanted to say that there is no hard pressure since we have a temporary solution to the problem.

Looking at the code, it seems the dsv_to_sheet_str function does a check for determining the field separator - and it checks for the existence of any tab (\t) character before checking for a comma (,). As such, it finds your embedded tab and assumes a tab-delimited file.
A quick search for other methods for determining the delimiter seem to settle on interrogating the first X lines of the file and using which ever delimiter occurs the most frequently across all lines. In your file's case, it would be a comma (,).
I will see about putting together a prototype solution and post it as a pull request.

@SlyW @nknapp the issue was correctly diagnosed -- the current approach scans the leading characters and assumes it is tab-separated if the tab character appears in the first 1024 chars.  That probably should be changed.
@SlyW if you are willing to explore this further, the "delimiter determiner" routine would need to disregard characters within quotes:
a,b\tc",,,,,\t\t\t\t\t"\td,e
      |-- disregard --|

Note: If you can control the origin, the best approach is to force the delimiter by prepending with a sep=, header.

@SheetJSDev - concur that sep= at the top of the file is the best method. Unfortunately, complete control over the file's origin is elusive at best.
I have a terribly naive approach which I will adjust to account for the quoted delimiters. Is it safe to (naively) split the incoming file in \n\r for the purpose of identifying "rows" of data to scan? Technically, a delimited file could support embedded CRLF values (quoted of course) ... however, it seems there are other places where the naive row splitting approach was used.

DSV is read character-by-character -- there's a boolean instr which keeps track of whether the current character is in a quoted string.
Other formats use the naive approach because its simpler.  DIF doesn't support newline characters (it dates back to visicalc and the format never envisioned the freedoms of modern utilities.  SYLK explicitly uses newline to separate records, encoding the newline character in a specific way.  PRN is ambiguous.

Thank you @SheetJSDev. It seems, then, it would be prudent to spin through the whole of the file [i]twice[/i]. The first time to ascertain the delimiters, the second to actually process the file. I can still use a limiting row count to only scan the first "n" rows...

We changed the resolution in lines with this discussion.  Based on the first 1024 bytes, the parser counts the number of unquoted commas and tabs and assumes the more-frequent character is the delimiter.

Thanks for the fix. I'll try it out next week. By "prepending sep=," you mean just literally prepending it to the to the file contents?
This might be feasible for us as well, but I have to check.

It's an Excel feature not covered in RFC4180.  The hint tells Excel to use the specified delimiter, and Excel knows to skip that first "row".  It even works with other delimiters!  Here's an example with a pipe character:
sep=|
1|2|3
4|5|6

If you have control over the origin of the data, the sep metadata is really useful when dealing with international users.


kubelet log full of errors

Error on reading termination-log /var/lib/kubelet/pods/fa438d63-f986-11e4-9c82-42010af0ae3d/containers/skydns/933ddf81dbaf586e425240432429aa57bec939383e19bfb25d35d478597203f7: open /var/lib/kubelet/pods/fa438d63-f986-11e4-9c82-42010af0ae3d/containers/skydns/933ddf81dbaf586e425240432429aa57bec939383e19bfb25d35d478597203f7: no such file or directory

I think there is a regression lately breaking termination-log.

#8436 might be related to this issue.

Was this fixed by #8436?

No, I still observed such error logs once a while, but so far those errors are red herring, without any breakage. Haven't get down to look into the detail yet.

If a pod gets deleted from the apiserver, kubelet (more specifically, the SyncPods function) would stop the container, and remove the orphaned volumes and pod directories. In the meantime, the per-pod workers (goroutines) may not have terminated yet. Therefore, the workers could be inspecting the containers to generate the status, causing the errors. Note that this does not happen if containers terminated by themselves while the pod is not yet deleted.
The error messages are harmless, since the pods have been deleted and no one would examine the termination logs anyway. However, the fact that kubelet.log is full of such errors is misleading and confusing to users. I think this would be solved by graceful termination...


terraform download filename suggestions

Currently all the terraform download links have the same lattice.tf.example name; and it ends with .example which means my laptop doesn't know what type of file it is (compared to ending with .tf).
Proposal for filenames in the release notes:

+1 - will prioritize this story.

we changed around the terraform directory structure to put the example .tf file in a separate folder from the other .tf files.  this should be more clear now and you won't run into that issue again.
https://github.com/cloudfoundry-incubator/lattice/commit/de342bd07b37a2a9aa91328a3823182a184dc064
#39 #43 #45 should all be addressed with this change


Training details of jasper10x5_LibriSpeech_nvgrad

Could you please show me more details of jasper10x5_LibriSpeech_nvgrad pre-train model? Such as hardware, training time, whether to use external data or not, whether to use synthetic data or not. Thanks.

@blisc could you please comment on this

The pre-trained model can be achieved as is with the jasper10x5_LibriSpeech_nvgrad.py example config.

No data external to LibriSpeech is used (no synthetic data was used).
We used 3-fold speed perturbation (0.9, 1.0, 1.1) as data augmentation
It was done on 8xV100

In additional, compared to older Jasper models:

We switched from python_speech_features to librosa for pre-processing
We switched from SGD to Novograd
We added dithering
Input spectrogram is now normalized per frequency bin as opposed to over the whole spectrogram

Thank you for your valuable information. @blisc


remove in-cluster components from UI

This PR largely follows https://github.com/helm/monocular/pull/524/files.
Note that ConfirmDialog is kept as it is used in the Hub (by the comment component). Also the components related to Repositories was removed in #67.
Based on #67 and should be rebased on master once #67 is merged.

Rebased onto master


bug(storage): incorrect use of retention_second

Describe the bug

Epoch consists of time || seq, and both part need to be considered in filter.
Read result of merged imm is not checked when retention_second is set.

To Reproduce
No response
Expected behavior
No response
Additional context
No response

finished by #9309


Catalog : New Entity type and specs for defining workflow/businessprocess

New Entity type and specs for defining workflow/businessprocess
Feature Suggestion
Define new entity type (Workflow/Process) to define/model business processes which can reference the involved systems.
The intentions is to provide broader ecosystem view for the courser-grain business processes, most probably under Explore plugin.
Possible Implementation
The specs for the Workflow can be similar to AWS steps funtions/steps in scaffolder templates.
Workflow steps would reference the available domains:system to create the representation and provide context into business processes.
Reference:
The workflow visualization can be something like this :

This visualization allows to overlay many variants of similar process using minimum space and clutter.
Library for Tube map :  https://observablehq.com/@johnwalley/d3-tube-map
Alternate approach considered :
Domains and Systems (or even components) can be tagged with XYZ business processes. However, this approach does not seem to be managible as

It may get clutter with N number of high level business processes.
It relies on multiple multiple teams to apply consistent tags for e2e business process view, which is practically not easy to achieve due to cross-team dependencies.

Context
Current Ecosystem covers views for domains and systems. These information can be further leveraged to define business proces context in which they are used by refering them into Workflow kind, which is also stored as catalog entity.

Hey! I just wanted to acknowledge that I saw this ticket so it doesn't seem to go unnoticed into the aether.
We're sort of careful with adding more kinds to the core offering. However, adding custom private extensions to the model is quite fine. Especially adding a completely new kind, which would not interfere with existing things.
I'll also add that in a lot of cases, documentation is good holder for information like this, and it might be fine to reference that documentation as metadata.links for example.
Closing this for now. Do feel free to continue the discussion here or on Discord if you feel for it!


Add builtin: cpid

This adds the cpid builtin which makes the pid of the spawned child
command available to programs:
bpftrace -e '
tracepoint:syscalls:sys_enter_openat /pid == cpid/
{
  printf("%d :: %s\n", cpid, str(args->filename));
}' -c ls
22804 :: /etc/ld.so.cache
22804 :: /lib/x86_64-linux-gnu/libselinux.so.1
22804 :: /lib/x86_64-linux-gnu/libc.so.6

Fixes: #298

Generally looks good, but why is child_pids_ definted as a vector? It looks like your new code is assuming there'll only be one child, and as far I know that's always going to be true.

It's a left over from the original -c implementation, I'm not sure why it was done that way. I'll take it out as it will probably only add confusion.

Nice, LGTM.


Allow writes with no expected response

Most of the SCPI based equipment I use (Agilent DMM, DAQ, etc.) if the command is write-only, will not allow you to do a read afterwards. The mocks must be queried (write+read) in order to flush the newline response. There should be some way to make the writes work without having to do a read.
Current workaround is overwriting my write() function with query() for the unittests, but I'd like a way to do this more cleanly.

Yes. This is an unintended behavior. Would you like to submit a patch? It could be along the following lines.
If there is not r field in the dialogue, store a particular sentinel object (maybe called MISSING). You will need to change at least the parser and to_bytes)
Then in Device.write, if the matched response is this MISSING value then do not write the eom.

I can look into it because I'm running into it quite a bit. Thanks for the pointers

I think this can be closed now.


[The D Bug Tracker] "File does not exist" FileException on c:\hiberfil.sys

vital.fadeev reported this on 2020-10-04T06:53:46Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=21289
Description
Created attachment 1805
file

"File does not exist" FileException on existent file: c:\hiberfil.sys

Source code:

import std.stdio;

void main()
{
    import std.file : DirEntry;

    writeln( DirEntry( "c:\\hiberfil.sys" ).isDir() );
}

I expected "false".

Not FileException.

Because "c:\\hiberfil.sys" is exist.

!!!There are attachements in the bugzilla issue that have not been copied over!!!

vital.fadeev commented on 2020-10-04T15:20:06Z
I fix bug.
Check the patch.
I will be happy when it will be committed in next version.

std-file-dirEnrty-windows.patch

( see attached .patch in ticket )

vital.fadeev commented on 2020-10-04T07:30:13Z
I look into the std.file... function "existsImpl()"...

...
    version (Windows)
    {
        // http://msdn.microsoft.com/library/default.asp?url=/library/en-us/
        // fileio/base/getfileattributes.asp
        return GetFileAttributesW(namez) != 0xFFFFFFFF; // INVALID_FILE_ATTRIBUTES
    }
...

existsImpl() functions return "false" on file "c:\\hiberfil.sys".
Because INVALID_FILE_ATTRIBUTES.
It not correct? because file exists.

I tried several ways: GetFileAttributesW(), GetFileAttributesExW(), FindFirstFileW().

Working solution :

    WIN32_FIND_DATAW fd;
    auto hFind = FindFirstFile ( "c:\\hiberfil.sys", &fd );

    if ( hFind != INVALID_HANDLE_VALUE )
    {
        writeln( "[ OK ]" );
        FindClose( hFind );
    }

Of course solution above is a little slower than GetFileAttributes(), but work correct with .sys files.

I will suggest using "FindFirstFileW()" in the "std.file: existsImpl()"  there is no better solution yet.

vital.fadeev commented on 2020-10-04T15:19:55Z
Created attachment 1806
std-file-dirEnrty-windows.patch

vital.fadeev commented on 2020-10-04T15:28:53Z
Created attachment 1807
fixed std.file.exists()

Using FindFirstFile() for check file existens under Windows.
Because correct work with system locked files: like a c:\hiberfil.sys


Improve inventory table UI

Description

Removed the Attribute Summary column.
Modified the text to drilldown in the actions menu
Improved calculation for the menu height for the drilldown.
Made sure the Deploy Progress column had enough width, made a switch case if we need to finetune the width of other columns.
Updated E2E tests for flake in Scenario 3.

Closes #5280  and #5278 and #4934
https://github.com/inmanta/web-console/assets/44098050/152672e9-3f42-4644-aed2-96fb09a50a75

Processing this pull request

Merged into branches master, iso6 in 31dfb1243671c35e7ae10e6c13d71a9aa313e084


verb command with multiple options in version 2.0

 Issue by zou-mono
Saturday Oct 08, 2016 at 09:55 GMT
Originally opened as https://github.com/gsscoder/commandline/issues/360

Hello,
In version 2, is there any way to implement the verb command with multiple options? e.g:
git tag -a tagname -m 'Tag Description'
I have tried many ways, but it can support only single option, like git tag -a tagname or git tag -m 'Tag Description'. If a class is labeled VerbAttribute, and contains two options at least, then it will meet errors during argument parsing as follows:
Options: 'X' is not compatible with : 'XX'. Options: 'XX' is not compatible with : 'X'.
where X and XX are custom options.

 Comment by nemec
Saturday Nov 05, 2016 at 07:09 GMT

Hi, I cannot reproduce this issue. Does the example below work for you?
[Verb("tag")]
public class Verb
{
    [Option('a')]
    public string Name { get; set; }

    [Option('m')]
    public string Description { get; set; }
}

void Main()
{
    var result = CommandLine.Parser.Default.ParseArguments<Verb>("tag -a tagname -m TagDescription".Split());
    result.WithParsed<Verb>(opts => opts.Dump());
    result.WithNotParsed(err => err.Dump());
}

Maybe the answer below would help. Try set multiple options with the same SetName or just leave their SetName's undefined.
https://github.com/commandlineparser/commandline/issues/246#issuecomment-376872194


[BUG] Documentation shows unreleased features

Description
The current setup has documentation updates that are deployed instantly even for features that have not been released.
Since everything is merged in main when it's ready, we get to a state where documentation is out of sync with features in the current version.
Deploying the doc site for tags only would fix this versus deploying from the main branch.
Steps to Reproduce
Visiting Running Your Project shows crewai run command that is not available in v0.41,1
Expected behavior
Only released features should be on the documentation site
Possible Solution
Either deploy the docs from the latest tag or we have versioned docs

Thanks for raising @zinyando,
This has been addressed in #1212
New version dropping soon so Docs will stay as they are for now


[roles-0.1.0] Determine Top Hat status

If no Hat Tree is set for the given Safe, build a workflow to determine if the Safe is the wearer of any Top Hats

Don't care about top hats any longer


Any option to change background color of the dates?

Hi! First of all, I want to say thank you for taking the time to develop and maintain this library. Your implementation of the calendar is, without a doubt, unique and pleasant to look at.
I've been developing an app that shall have DayNight theme as one of its features. Using your Datepicker Timeline, this is what I see whenever my app get into the Night (Dark) theme:

Please note that the rest of my app has been properly adapted to use dark backgrounds (as can be seen from the window and CardView background in the screencap above).
Looking into the sources, I can tell that it's currently not possible to change the background to another color. Could you please provide a way to do so?
Thanks in advance!

I've made a PR to address this issue. When you got time, please check it out and tell me if something's missing. :)

With the changes introduced within #18, it is now possible to change the background color of the dates. I'm marking this issue as resolved. :)


404 on https://player-ui.github.io/latest/#/storybook-demo

Describe the bug
404 error

To Reproduce
go to https://player-ui.github.io/latest/#/tools/storybook
select the "here" link in "Reference Assets"

Expected behavior

Screenshots

Environment:

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context

This should be fixed with the doc site rewrite merged with #506 and is available on the next version of the doc site at https://player-ui.github.io/next/. Will close this once this available on the mainline doc site


Fixes URL for download

The FileZilla URL provider no longer works to provide a valid tarball.

I have an alternative fix in https://github.com/autopkg/keeleysam-recipes/pull/100

did #100 instead


docs: 安装通道文档优化，增加页面填写指引(closed #2051)

PR
CheckList（PR Assigners）
确认已完成以下操作：

[x] 已将分支与当前合入分支同步
[x] 已在关联 Issue 中补充「功能自测」部分内容
[x] 已更新关联 Issue 的状态 backlog -> todo -> for test -> tested（需通过测试同学测试） -> done
[x] 将 PR 添加上 pr/reviewable

CheckList（PR Reviewers）
确认已完成以下操作：

[ ] 检查关联 Issue 「功能自测」部分内容
[ ] 必要时打回 pr/reviewable，要求重新修改
[ ] 及时 Approve

 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.

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

Project coverage is 74.71%. Comparing base (f22e93e) to head (93d81b7).
Report is 66 commits behind head on v2.4.4-dev.

Files
Patch %
Lines

apps/node_man/handlers/debug.py
8.69%
147 Missing :warning:

apps/node_man/periodic_tasks/sync_cmdb_host.py
58.62%
12 Missing :warning:

apps/backend/subscription/steps/adapter.py
84.90%
8 Missing :warning:

apps/core/concurrent/retry.py
71.42%
8 Missing :warning:

apps/core/gray/handlers.py
83.33%
5 Missing :warning:

apps/node_man/views/install_channel.py
63.63%
4 Missing :warning:

apps/node_man/handlers/cmdb.py
66.66%
3 Missing :warning:

...nents/collections/agent_new/install_other_agent.py
66.66%
2 Missing :warning:

...ckend/management/commands/init_official_plugins.py
91.66%
2 Missing :warning:

...ckend/subscription/steps/agent_adapter/handlers.py
91.66%
2 Missing :warning:

... and 16 more

Additional details and impacted files
@@              Coverage Diff               @@
##           v2.4.4-dev    #2100      +/-   ##
==============================================
+ Coverage       74.58%   74.71%   +0.13%
==============================================
  Files             403      404       +1
  Lines           27750    28266     +516
==============================================
+ Hits            20697    21119     +422
- Misses           7053     7147      +94

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


Add it to microjs

Maybe add this to microjs ? It certainly falls into the "less than 5 kb" category ;)
Github: https://github.com/microjs/microjs.com
cu, w0lf.

@ginsterbusch Cool! Do you know the maintainers?

@charlike Are you one of the maintainers of Microjs? What do we need to do so that Hyperapp shows on your website?
/cc @SimonWaldherr

Yea, I'm. But the site is pretty outdated. We need much work there to update the whole site. We regularly merge some PRs, but if i remember correctly they are not shown on the site currently.
You can send PR and once we do our work it will be shown :)

Thanks @charlike.
/cc @ginsterbusch

I'd love for Hyperapp to be added to microjs, but I don't have time to deal with that, so if anyone wants to beat me to it, feel free to add it and ping me then! Thanks @ginsterbusch.

FYI @ginsterbusch, @JorgeBucaran, @charlike: done
via https://github.com/microjs/microjs.com/commit/f3fc8bc03f1eac5796055b0d8a3c0193bbc623c6


[FEATURE] Eve App 4.2 Issue with homebridge-webos-tv

@merdok
The lates Eve for HomeKit App v4.2 is causing issues with several Homebridge Plugins.
They changed something so that the serial number of a device is linked somehow in the background. This was not with previous Eve versions.
Problems causing now like described here.
homebridge/homebridge#2503
So all devices/switches from homebridge-ssh can't be sorted anymore and can't get an icon for each device separately.
Do you see a solution to fix this "Eve" Issue, by changing something in your plugin?
Thanks again for your work and the time you are investing generally.

The tv accessory already has a unique serial number, is that not enough?

It is looking good to me. I also use webos-tv Plugin and did not mention it in my main post.
Eve App 4.2 is not causing any problem with homebridge-webos-tv.
Al least not causing any issue in my scenario. Maybe @palasinio has another use case?
Thanks for your work @merdok and the time you are investing. Wouldn't want to miss your plugin.

Hi, this is funny. I opened this ticket, because in my „Home“ app every accessory of this plugin has the same serial. In the Eve app I cannot see a single accessory of this plugin.

I always write the mac address of the tv as the serial number which should be unique. So if you added the same TV twice then of course those two accessories will have the same mac address, the question just would be why would you want to add twice the same accessory?

@palasinio I still would assume this is not an Issue of the Plugin. All good.
Eve App is showing things different than Apple Home App.
In the Home App you probably have just one tile including all other switches. See Video.
You have the option to separate them if you like.

Anyway, the plugin is working with Eve App, even with the current Bug in Eve App.

Okay - so I close this. Thank you very much. I messed up that one accessory can have more components. Why I can‘t see the WebOS accessory in EVE is a different thing.


Add licence to repo

Hey Maksym,
I would like to use this plugin as a base to work of for a project, however, I would need have a licence to properly make use of this code. Could you add a licence to the repo so I know what I can and can't use this plugin for?

what kind of license is preferred?

The MIT license as it is simple and permissive https://choosealicense.com/licenses/mit/

https://github.com/max-verem/FreeDLiveLink/commit/6d1cc8a5b5880daa443148883c0e25f44cf92c28

Awesome, thanks for the quick response!


Add caffe instructions for MacOS

I guess it's simpler to follow the instructions on the caffe's own installation page instead.

@trivedigaurav, this would be a great addition. If you could squash this into a single commit (git rebase -i origin/master) and reopen this pull request, I'll merge it.
Did you have to do any other work to get DIGITS running on a mac, or was the DIGITS install pretty trivial? I'm assuming at least this command didn't work:
% sudo apt-get install graphviz gunicorn

What else would need to change in the README to have full Mac support?

@lukeyeager Yes, I was able to get it working on MacOS. Although I was not able to build models for MNIST yet. I hit the following error:
2015-04-27 19:15:58 [20150427-191554-8b5e] [DEBUG] memory required: 711 MB
2015-04-27 19:15:59 [20150427-191554-8b5e] [DEBUG] memory required: 1.08 GB
2015-04-27 19:15:59 [20150427-191554-8b5e] [ERROR] Train Caffe Model: Check failed: error == cudaSuccess (2 vs. 0)  out of memory

The main problem while using it on MacOS is the missing makefile instructions for building caffe in https://github.com/NVIDIA/caffe/blob/master/Makefile. We have it fixed in the bvlc caffe. I have created a pull request here: https://github.com/NVIDIA/caffe/pull/2
The instructions for building caffe are easy to follow from http://caffe.berkeleyvision.org/install_osx.html. Otherwise for digits we can add a couple more things for completeness (again, using homebrew):
$ brew install graphviz
$ pip install gunicorn
$ for req in $(cat requirements.txt); do pip install $req; done

I  have created a new pull request with these instructions here: https://github.com/NVIDIA/DIGITS/pull/94


NL: Support date constraints in queries

Example: [how many earthquakes since 2000?]
This is probably a subset of https://github.com/datacommonsorg/website/issues/2366 which involves identifying constants in the query.

Another example [how much more rain did california get this year vs last year]

Adding additional examples:
How many burglaries occurred in the US last year?
How many electricity consumers were there in the US last month?
Largest cities in 2020

Example query: what is the percentage of people in California that do not work in the past 2 years
Expected answer: restrict the range of time to just the last two years

Supporting time based queries as follow-up queries using context.
Example: https://autopush.datacommons.org/nl/#q=How does the birthrate of korea compare with japan;what about over time
Q1: how does the birthrate of korea compare with japan. => shows the most recent data point comparison.
Q2: what about over time  => ideally, should plot timeline plots for the two countries


Reusing configuration cache  in gradle

In my project i am using this plugin to support build cache via AWS. And before i can connect AWS i need to pass awsAccessKeyId, awsSecretKey and sessionToken values in the plugin and the values of them are different everytime when i get them from AWS. In result, I found that configuration cache cannot be reused via CI because the values are different everytime. Do we have any solution for that? 🤔

You are most likely using short term credentials, in case you use circle CI, you can use circleci/aws-cli@ orb and use CircleCI OIDC, in that case, no credentials needs to be stored in CI/CD.


[ISSUE #4432] fix dubbo invoker cache bugs.

fix #4432

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4433 (b474976) into master (f88127f) will decrease coverage by 0.19%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #4433      +/-   ##
============================================
- Coverage     68.04%   67.85%   -0.19%
+ Complexity     7500     7485      -15
============================================
  Files          1023     1023
  Lines         29005    28994      -11
  Branches       2592     2588       -4
============================================
- Hits          19736    19675      -61
- Misses         7720     7769      +49
- Partials       1549     1550       +1

Impacted Files
Coverage Δ

...in/apache/dubbo/proxy/ApacheDubboProxyService.java
61.53% <100.00%> (+3.84%)
:arrow_up:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
68.75% <0.00%> (-31.25%)
:arrow_down:

.../client/server/consul/ShenyuConsulConfigWatch.java
57.44% <0.00%> (-29.79%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
70.40% <0.00%> (-19.39%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
68.96% <0.00%> (-13.80%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
21.27% <0.00%> (-6.39%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
52.12% <0.00%> (-4.26%)
:arrow_down:

...he/shenyu/plugin/grpc/client/ShenyuGrpcClient.java
8.00% <0.00%> (-4.00%)
:arrow_down:

...yu/common/dto/convert/rule/Resilience4JHandle.java
71.60% <0.00%> (-3.71%)
:arrow_down:

...he/shenyu/loadbalancer/spi/RandomLoadBalancer.java
87.87% <0.00%> (-3.04%)
:arrow_down:

... and 13 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mahaitao617 Hi. I have tested that it works in multiple namespaces, please review.


add ncbi taxon back to CL

Was previously removed for performance reasons, but looks better now that I have more juice in my machine.
closes #712

Was previously removed for performance reasons

Why not use taxslim as seed? Or the uberon taxon imports (which is pretty rich and should cover CL)


Typos fixed in /config

Corrected misspelled words, added periods, and fixed some grammatical errors.

@chandescartes 감사합니다.