Ask for confirmation when running apply without diff or dryrun mode

Related to https://github.com/slok/kahoy/issues/56
Not sure if this is what you have in mind @slok. I'm still missing the --yes parameter to skip the confirmation but first i want to know if this approach seems right.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #72   +/-   ##
=======================================
  Coverage   77.63%   77.63%
=======================================
  Files          23       23
  Lines        1055     1055
=======================================
  Hits          819      819
  Misses        177      177
  Partials       59       59

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 aec3726...ae451dc. Read the comment docs.

@slok could you take a look at this one? I tried it out and I believe its working as expected.


make the competition environment a package

make the competition environment a package

If @qianyi-sun implements the scenarios in #1418 by following the examples/multi_intersection.py example in commit b3098027e5d7fd04c12dd8d51fab4a6c9fc24a5a, then the users can simply install the SMARTS package and instantiate the competition environment, for example env = gym.make("smarts.env:merge-v0").

We can install SMARTS package as$ pip install "smarts[camera-obs] @ git+https://github.com/huawei-noah/SMARTS.git@comp-1"

The NeurIPS2022 competition environment can be instantiated as env=gym.make("smarts.env:multi-scenario-v0") from the comp-1 branch.


Can't find version 0.9.41 in the Maven repository

Describe the bug
Can't find version 0.9.41 in the Maven repository
To Reproduce
https://repo1.maven.org/maven2/org/roaringbitmap/RoaringBitmap/
RoaringBitmap version:
Please report bugs after testing the latest version of the library. If you are using an older version of the library, please explain.
Make sure to compare your version against the latest available version at https://mvnrepository.com/artifact/org.roaringbitmap/RoaringBitmap
Java version:
Please tell us which Java version you are using.

/cc @lemire thanks~

This is not a bug on our part. The release was done. We simply cannot create a release without uploading the artefacts, if we fail to do so, the release process abort. The problem is with the maven repository. It is shoddy software.
I will issue a new release. That's all I can do.
I recommend you do not depend on the maven repository. Create your own.

A release was issued.
Please note that we have no control over the maven repository. It routinely loses artefacts. It is a slow/lossy system.


[bug] assure format is a symbol

I ran in to an issue when using Panoramic with Rails6 and Ruby 2.7.0. The error is caused by Mime[record.format] returning an object instead of to_sym. I have not investigated what version permutation introduced the change.
I've created an example project to reproduce the error: https://github.com/vandux/panoramic_error

Hi,
thank you for the great gem :+1:
Could you make a new release including this fix and publish it on rubygems?
Best regards
Malte

Spent quite a while before finding this PR :)
@andreapavoni, would be great if you could update the gem, or at least mention the issue in the README if something else is blocking you?
Thanks!


Add PHPUnit Migration rulesets and fixers

Closes #2501
I started working on it ages ago, yet it was already reviewed internally.
If you find sth that could be improved, please open a PR, I'm glad to see it!
If you figure out that sth is missing, please raise a feature request.
This PR doesn't contain replacement for getMock call with multiple arguments, as there is no 1-1 replacing method

:+1:  :100:   @keradus  thanks!


Mise fails to update k9s due to master vs main branch name

Describe the bug
On Aug 23 2024, the k9s repo used by mise changed their "master" branch name to "main".  As a result, when running mise plugins update with the latest version of mise, k9s fails to update.  In my mise doctor output below, you can see I have v0.27.3 of k9s configured.  This is a bit old.  So I tried setting this to 1.0.0 (as spec'd in the version.txt in the k9s repo) but this fails at install time (can report in a different issue if that is the path forward).  Note that when installing k9s v0.27.3 for the first time via mise, this works as expected -- it is only the mise plugins update that fails.
To Reproduce
run mise plugins update with the k9s plugin configured for mise.
Expected behavior
Expect update to work by pulling k9s from the main branch, but it tries to pull from master instead, and fails.
mise doctor output
~ $ mise doctor
version: 2024.8.12 linux-x64 (778f806 2024-08-20)
activated: yes
shims_on_path: yes

build_info:
  Target: x86_64-unknown-linux-gnu
  Features: DEFAULT, NATIVE_TLS, OPENSSL
  Built: Tue, 20 Aug 2024 22:22:19 +0000
  Rust Version: rustc 1.80.1 (3f5fd8dd4 2024-08-06)
  Profile: release

shell:
  bash
  GNU bash, version 5.1.16(1)-release (x86_64-pc-linux-gnu)
  Copyright (C) 2020 Free Software Foundation, Inc.
  License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

  This is free software; you are free to change and redistribute it.
  There is NO WARRANTY, to the extent permitted by law.

dirs:
  data: ~/.local/share/mise
  config: ~/.config/mise
  cache: ~/.cache/mise
  state: ~/.local/state/mise
  shims: ~/.local/share/mise/shims

config_files:
  ~/.config/mise/config.toml

backends:
  cargo
  core
  go
  npm
  pipx
  spm
  ubi
  vfox

plugins:
  awscli      https://github.com/MetricMike/asdf-awscli#ac99537
  bat         https://gitlab.com/wt0f/asdf-bat#64022b1
  choose      https://github.com/carbonteq/asdf-choose#0291009
  ctop        https://github.com/neohsu/asdf-ctop#77c1040
  delta       https://github.com/andweeb/asdf-delta#501318b
  duf         https://github.com/NeoHsu/asdf-duf#61e14e2
  dust        https://github.com/looztra/asdf-dust#11bdb8a
  exa         https://github.com/nyrst/asdf-exa#3f525b7
  fd          https://gitlab.com/wt0f/asdf-fd#17d56e0
  fzf         https://github.com/kompiro/asdf-fzf#d19eb67
  github-cli  https://github.com/bartlomiejdanek/asdf-github-cli#e0605b7
  helm        https://github.com/Antiarchitect/asdf-helm#f4b7f38
  hyperfine   https://github.com/volf52/asdf-hyperfine#6e64a91
  jq          https://github.com/ossareh/asdf-jq#66fb2be
  k9s         https://github.com/looztra/asdf-k9s#2b3af87
  krew        https://github.com/bjw-s/asdf-krew.git#6354415
  kubectl     https://github.com/asdf-community/asdf-kubectl#cbe6df4
  kustomize   https://github.com/Banno/asdf-kustomize.git#8e929af
  neovim      https://github.com/richin13/asdf-neovim#d6118ad
  pulumi      https://github.com/canha/asdf-pulumi.git#99d675b
  ripgrep     https://gitlab.com/wt0f/asdf-ripgrep#e836665
  steampipe   https://github.com/carnei-ro/asdf-steampipe.git#261baed
  terraform   https://github.com/asdf-community/asdf-hashicorp.git#f1602b6
  tmux        https://github.com/aphecetche/asdf-tmux#eb8deee
  viddy       https://github.com/ryodocx/asdf-viddy#e35bcd3
  zoxide      https://github.com/nyrst/asdf-zoxide#8ed95c9

toolset:
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
  python@mambaforge
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>

env_vars:
  MISE_SHELL=bash

settings:
  activate_aggressive = false
  all_compile = false
  always_keep_download = false
  always_keep_install = false
  asdf = true
  asdf_compat = false
  cargo_binstall = true
  color = true
  disable_default_shorthands = false
  disable_tools = []
  experimental = false
  go_default_packages_file = "~/.default-go-packages"
  go_download_mirror = "https://dl.google.com/go"
  go_repo = "https://github.com/golang/go"
  go_set_gopath = false
  go_set_goroot = true
  go_skip_checksum = false
  http_timeout = 30
  jobs = 4
  legacy_version_file = true
  legacy_version_file_disable_tools = []
  libgit2 = true
  node_compile = false
  not_found_auto_install = true
  paranoid = false
  plugin_autoupdate_last_check_duration = "7d"
  python_default_packages_file = "~/.default-python-packages"
  python_pyenv_repo = "https://github.com/pyenv/pyenv.git"
  raw = false
  trusted_config_paths = []
  quiet = false
  verbose = false
  vfox = false
  yes = false
  ci = false
  debug = false
  trace = false
  log_level = "info"
  python_venv_auto_create = false

  [status]
  missing_tools = "if_other_versions_installed"
  show_env = false
  show_tools = false

No warnings found
1 problem found:

1. unused shims are present, run mise reshim to remove them
   Unused shims: kubectl-krew

Additional context
Out from mise --verbose plugins update:
Error:
   0: [k9s] plugin update:
         0: git failed: Cmd(["git", "-C", "/home/coder/.local/share/mise/plugins/k9s", "-c", "safe.directory=/home/coder/.local/share/mise/plugins/k9s", "fetch", "--prune", "--update-head-ok", "origin", "master:master"]) fatal: couldn't find remote ref master
         0:

      Location:
         src/git.rs:72

      Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
      Run with RUST_BACKTRACE=full to include source snippets.

Location:
   src/cli/plugins/update.rs:59

Version:
   2024.8.12 linux-x64 (778f806 2024-08-20)

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.

I don't think mise is doing anything wrong here. It's pointing to a branch that was deleted and failing appropriately.

I guess the issue here is that the repository changed the main branch from master to main (and deleted the old master branch?). There's probably no straightforward way mise can know that the tracked upstream changed and automatically adjust that to whatever.
I assume the easiest remedy in this case is to simply re-install the plugin.

The plugin installs fine the first time.  It is only the mise plugins update that is failing, even after re-installing fresh.
The issue is the version referenced -- presumably this older version of k9s (0.27.3) causes the reference to the master branch to be maintained because that was the branch that existed at that time, and that branch doesn't exist anymore.
I fixed this by specifying a newer version of k9s (0.32.5, which is the latest).


[Event Request] table 37 "Sales Line" - GetVATBaseDiscountPercentage

In https://github.com/microsoft/ALAppExtensions/issues/21662 , we requested a new function on SalesLine : GetVATBaseDiscountPercentage
Can you please integrate this new function and replace SalesHeader."VAT Base Discount %" in the OnValidate triggers of the following fields on table Sales Line?

Amount
Amount Including VAT

This would allow us to overrule the SalesHeader."VAT Base Discount %" value with zero in case a certain sales line is to be excluded from any discounts.
As-is
field(29; Amount; Decimal)
        {
            AutoFormatExpression = "Currency Code";
            AutoFormatType = 1;
            Caption = 'Amount';
            Editable = false;

            trigger OnValidate()
            begin
                Amount := Round(Amount, Currency."Amount Rounding Precision");
                case "VAT Calculation Type" of
                    "VAT Calculation Type"::"Normal VAT",
                    "VAT Calculation Type"::"Reverse Charge VAT":
                        begin
                            "VAT Base Amount" :=
                              Round(Amount * (1 - SalesHeader."VAT Base Discount %" / 100), Currency."Amount Rounding Precision");
                            "Amount Including VAT" :=
                              Round(Amount + "VAT Base Amount" * "VAT %" / 100, Currency."Amount Rounding Precision");
                        end;

field(30; "Amount Including VAT"; Decimal)
        {
            AutoFormatExpression = "Currency Code";
            AutoFormatType = 1;
            Caption = 'Amount Including VAT';
            Editable = false;

            trigger OnValidate()
            begin
                "Amount Including VAT" := Round("Amount Including VAT", Currency."Amount Rounding Precision");
                case "VAT Calculation Type" of
                    "VAT Calculation Type"::"Normal VAT",
                    "VAT Calculation Type"::"Reverse Charge VAT":
                        begin
                            Amount :=
                              Round(
                                "Amount Including VAT" /
                                (1 + (1 - SalesHeader."VAT Base Discount %" / 100) * "VAT %" / 100),
                                Currency."Amount Rounding Precision");
                            "VAT Base Amount" :=
                              Round(Amount * (1 - SalesHeader."VAT Base Discount %" / 100), Currency."Amount Rounding Precision");
                        end;

To-be
field(29; Amount; Decimal)
        {
            AutoFormatExpression = "Currency Code";
            AutoFormatType = 1;
            Caption = 'Amount';
            Editable = false;

            trigger OnValidate()
            begin
                Amount := Round(Amount, Currency."Amount Rounding Precision");
                case "VAT Calculation Type" of
                    "VAT Calculation Type"::"Normal VAT",
                    "VAT Calculation Type"::"Reverse Charge VAT":
                        begin
                            "VAT Base Amount" :=
                              Round(Amount * (1 - GetVATBaseDiscountPercentage(SalesHeader) / 100), Currency."Amount Rounding Precision");
                            "Amount Including VAT" :=
                              Round(Amount + "VAT Base Amount" * "VAT %" / 100, Currency."Amount Rounding Precision");
                        end;

field(30; "Amount Including VAT"; Decimal)
        {
            AutoFormatExpression = "Currency Code";
            AutoFormatType = 1;
            Caption = 'Amount Including VAT';
            Editable = false;

            trigger OnValidate()
            begin
                "Amount Including VAT" := Round("Amount Including VAT", Currency."Amount Rounding Precision");
                case "VAT Calculation Type" of
                    "VAT Calculation Type"::"Normal VAT",
                    "VAT Calculation Type"::"Reverse Charge VAT":
                        begin
                            Amount :=
                              Round(
                                "Amount Including VAT" /
                                (1 + (1 - GetVATBaseDiscountPercentage(SalesHeader) / 100) * "VAT %" / 100),
                                Currency."Amount Rounding Precision");
                            "VAT Base Amount" :=
                              Round(Amount * (1 - GetVATBaseDiscountPercentage(SalesHeader) / 100), Currency."Amount Rounding Precision");
                        end;

@ZepposBE

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52774.


Arduino Nano Every / ATmega4809 support

Implement #114 and add a barebones PoC that we can target ATmega4809.  As outlined in https://github.com/Rahix/avr-hal/issues/97#issuecomment-778763531, this is currently blocked on us still relying on avr-libc...
Don't consider this anything usable yet.

Is this still being worked on?

This is still blocked on the avr-libc problem which is linked above.  I'm not aware of any workaround we can take so unfortunately I think it is still going to take a bit longer...

I see, are you still thinking of rewriting the compiler in rust?

I just found this PR, after implementing most of 49fc191 myself xD
Could this be added behind an "unstable" feature flag? It should be safe for anyone willing to use the latest git version of avr-libc.

As a sidenote, and revisiting #94, there are a few older atxmega chips that don't use the new style IO cells.

I just found this PR, after implementing most of 49fc191 myself xD
Could this be added behind an "unstable" feature flag? It should be safe for anyone willing to use the latest git version of avr-libc.

Does this work?

Could this be added behind an "unstable" feature flag? It should be safe for anyone willing to use the latest git version of avr-libc.

@rmsc: Is it actually working for you?  Back when I wrote this changeset, it was all very fragile.  Even something as simple as a blinky was having problems because the delay timing was completely off IIRC.
If we can get this to work reasonably well (= people can use this more or less productively), I'd have no objections to adding it in this form.  Let's not block people from using their MCUs until the work in avr-device finally gets done.  If we can get it to work well enough, that is...

To be honest I still don't have the hardware to test it on.
The GPIO code is simple enough and does seem to be correct, but I haven't looked into the rest of the code. I did get the git version of avr-libc and merged this PR into main. With it I was able to build my own blinky for the attiny202 without errors.
On the other hand I didn't realize this was so fragile. I understand why you might not want to merge it yet. My suggestion was also in part motivated by my own frustration.
I'd be glad to test this as soon as I get suitable hardware.

I can test it if told what to do

I went ahead and rebased this branch on the latest main.  I have not done any tests besides trying to compile the examples...
@PiemanAU, this is very generous of you.  First of all, try checking out this branch and building the example:
cd examples/arduino-nano-every/
cargo build --release

Then to flash the program, there is the following comment from back when I originally worked on this:
https://github.com/Rahix/avr-hal/blob/6e47364d41331f2846e4c7a2a9f9250e23b5a3c3/examples/arduino-nano-every/src/bin/every-blink.rs#L8-L16
I have no clue how up-to-date this is.  At least it must have worked back then, I guess...
I hope this helps.  I know this is all in very bad shape right now :/

I went ahead and rebased this branch on the latest main. I have not done any tests besides trying to compile the examples...
@PiemanAU, this is very generous of you. First of all, try checking out this branch and building the example:
cd examples/arduino-nano-every/
cargo build --release

Then to flash the program, there is the following comment from back when I originally worked on this:
https://github.com/Rahix/avr-hal/blob/6e47364d41331f2846e4c7a2a9f9250e23b5a3c3/examples/arduino-nano-every/src/bin/every-blink.rs#L8-L16
I have no clue how up-to-date this is. At least it must have worked back then, I guess...
I hope this helps. I know this is all in very bad shape right now :/

Just saw this, I will check it in a couple of hours

When I run that command I get

stty: illegal option -- -F
usage: stty [-a | -e | -g] [-f file] [arguments]

So I tried changing to lowcase then it just did nothing
Just so you know this the is board that I have

The stty + echo command should reset the arduino such that it is ready for receiving a new firmware.  I may have some time on the weekend to test it myself again.

Is this being worked on still? I have a Arduino Uno wifi rev 2

Hey, is there any way I could help to further the progress on this PR?

I just got this working on an atmega4808. I want to help with the development here, I'm using the atmega as a co-processor for the ESP32, and would love it if I could share some DTO code between the two. Essentially, I need to get I2C(TWI) and input pins working and that would make this PR do everything I need. Will see how it's done for the others and try to get something going.


fix: mobile fixes

Description

Remove trade widget on mobile
Fix border between transactions and widget on trade page
Fix recent transaction block running together on mobile

Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
n/a
Risk
all visual stuff, should be no functional risk
Testing
n/a
Engineering
n/a
Operations
On mobile the trade widget should no longer be seen, and to trade you go to the actual trade page.
Transaction history should be nice and tidy both on the recent transaction block and the page.
Screenshots (if applicable)

@0xdef1cafe here is the fix for the recent tx block running together.


Throw an exception when having tasks within the same task-schedule with the same name

Task-names are unique strings within the Task-Schedules. This is used for scheduling across different devices.
Currently, there is no check to guarantee that tasks should have different names.
TaskSchedule ts = new TaskSchedule("foo")
                              .task("bar", ...)
                              .task("bar", ...);       <<- Throw an error when having this type of code

PR #85 solves this issue


Allow user overrides to always take priority

Alternative to #1218
Allows user to override most options,
Will help to minimize the code base and wont need to check each time a badge sets something that it uses the users default choice first.

I appreciate this a lot. In principle this is a much better way to go. It would avoid tedious calls to helper functions, increase our confidence that the overrides are always applied, and remove the need to try to catch these issues during code review.
However, post #963 we'll begin to phase out all of this. The new service handlers are shaping up to be async functions which simply return the badge data. The query data won't even be passed in. (Well, unless a service uses some of its own query parameters. Though that hasn't been implemented yet.)
After the handler returns, the server processes the rest of the query parameters and overrides the values returned by the handler. There won't be a need for getBadgeData, getLabel, etc.
Given the big gaps in test coverage across server.js, and total absence of tests on colors, I'd prefer to make incremental bug fixes than to refactor code we're simply going to rewrite soon. Does that make sense?

Yeah that makes sense, and #963 definitely sounds promising!


Data Splitting List

Hi, may I have the list for 10-cross validation splitting for your LUAD 5-gene mutation dataset? If possible, may I have the splitting list for other tasks as well?
Thanks in advance!

Hi Jingwei, we're checking to see if it's possible to release these splits. Please stay tuned!

Thanks! Looking forward to hearing from you.


Can I use the modal with Tensorflow Object Detection API?

Can I use the modal with Tensorflow Object Detection API?
https://github.com/tensorflow/models/tree/master/research/object_detection
Thanks!

I think so as long as the usage comply with the term of Apache 2.0 license and widerface dataset license.


Replaced myapp.yml with config.yml

Replaced myapp.yml and stale.yml with config.yml as it is a recommended naming-convention applied in many Probot applications.

Thanks @aps120797!


Twitter Downloader Weird Issue

There is a weird error when the user is trying to download this tweet.
https://x.com/rakuten_pacific/status/1845374551762534712?s=46&t=yRbmW3LBSHPc4JceoqUgFA

Please investigate and fix this issue

I just found the problem, and its very weird 🤔
{
    "chat_id": "939376599",
    "photo": "https://pbs.twimg.com/amplify_video_thumb/1845374508246630404/img/3-ICXTNr463mirF6.jpg",
    "parse_mode": "HTML",
    "disable_web_page_preview": "True",
    "reply_markup": {
        "inline_keyboard": [
            [
                {
                    "text": "Download",
                    "web_app": {
                        "url": "https://api.animemoe.us/twitter-downloader/safelink/?key=78093f27-ff7f-4e42-a26f-62b4a8ffa5b7"
                    }
                }
            ]
        ]
    }
}

We will got the too many request error when the photo url is https://pbs.twimg.com/amplify_video_thumb/1845374508246630404/img/3-ICXTNr463mirF6.jpg. lol

But another url just working fine

Post Deployment Test

✅


Replace basic-card with BobBucks in paymentrequests samples

Since the "basic-card" payment method has been deprecated, this PR is to update the samples using "basic-card" to use BobBucks service-worker-based payment handler instead.

@rsolomakhin , PTAL. Thanks!


[Feature Request] classes for Check-In only and Check-out only days

I'd love to see classes appended to dates that are check-in or check-out only.  Would make formatting them a breeze.
Thanks!

Well, I think you can use these classesis-start-date and is-end-date.

@wakirin - I'm digging into this and though I'm not quite seeing how I can set these dates.  Is there a method I can use to set the dates with these css classes?


Fix/1079 bug ops s3 pug stc execution failed with error code 139 for products sy 2 vg1

Included:

PUG_SY_2_VG1.03.xml: Remove pduLength & MtdPDUFrameNumbers to fix RS-1079
PUG_SR_2_LAN.03.xml: Totally Removed (not used on RS)
PUG_SR_1_SRA.03.xml: Totally Removed (not used on RS)

I guess I made a mistake in this PR.
The TaskTable.PUG_SY_2_SYN.03.xml was removed but I dont find any reason to explain this removing.

Nothing about it in the PR conversation.
No comment on original issue #1079.
Not removed in the twin PR: #12

Moreover, the TaskTable.PUG_SY_2_SYN.03.xml is just after TaskTable.PUG_SR_2_LAN.03.xml which was removed. I guess I included TaskTable.PUG_SY_2_SYN.03.xml in the TaskTable.PUG_SR_2_LAN.03.xml removing...
@w-jka : What's your opinion on PUG_SY_2_SYN.03.xml removing ?


Merge with KopyKate Big?

@misses-robot

Hi,
I am actually going to modify zeroTube so it would work like mixtape. It will use BigFile feature and merger site feature. I am still experimenting in Mixtape. @misses-robot is doing great with KopyKate and maintain it. However I am open to collaborating on this so mergerSite could both work in KopyKate and zeroTube (but I am still not sure how).


Add response decorator capability, and add the ability for REST methods to return a HttpResponse object via callback.

Could I have an example?

I am a hundred feet under water, but remind me.


plantuml-code-generator can't handle comments

Following plant file fails:
  @startuml
  ' this is a comment
  class A
  class B
  A --|> B
  @enduml

with message:
Error parsing input file:
SyntaxError: Expected "@enduml", "abstract ", "class ", "hide empty members", "namespace ", "package ", "setpropname.*", "skinparam", "title ", [ \t], [#], [+], [-], [A-Za-z_], [\n] or [\r\n] but "'" found.

Tested with upstream code

This is probably due to the comment tag, I can fix this within the week. Will close this issue when completed.

This should be fixed by
https://github.com/bafolts/plantuml-code-generator/commit/d085f96fd760ebfc4be5d2f73cab5754ec069e6d

I added an integration test with the sample file you provided. You should be good to go. Thanks for the help.


fix: dont use virtual key for UPDATE … where (<key>) in <subquery>

For a draft enabled model:
namespace my;

entity Books {
  key ID : Integer;
  title  : String;
  stock  : Integer;
  author : Association to Authors;
}

entity Authors {
  key ID : Integer;
  name   : String;
  alive  : Boolean;
}

service CatalogService {
    @odata.draft.enabled
    @readonly entity Books as projection on my.Books;

    @readonly entity Authors as projection on my.Authors;
}

my.Books will have a field:
        "IsActiveEntity": {
          "type": "cds.Boolean",
          "key": true,
          "default": {
            "val": true
          },
<EMAIL_ADDRESS>true
        },

which is marked as virtual by the lean draft implementation. This virtual key must not be part of the primary key matching which is done for a path expression in a UPDATE.where:
https://github.com/cap-js/cds-dbs/blob/982b8b796da4b577c5641039d2036816209c0437/db-service/test/cqn4sql/UPDATE.test.js#L73-L97
TODO:

[ ] test this

also check if UPDATEs on tables w/o primary keys cause issues


Default last date for a new jobs

Description
Default last date added in the job creation form. By default are 90 days, defined in a setting's constance.
Code formating
The code has to be well formatted following the formatting rules. For example in Python the code has to follow the PEP8.
Before create your PR, please make sure your code is well formatted and styled doing:
$ >> pre-commit run --all

I close it because I've moved this functionality to #158


Nil pointer at Eventually Fill

Given
Eventually(page.Find("#username").Fill("demo")).Should(Succeed())

(This looks almost the same as the demo code at http://agouti.org/#adding-acceptance-tests )
results in:
Full Stack Trace
  /home/etiennebruines/workspaces/go-all/src/github.com/onsi/gomega/internal/asyncassertion/async_assertion.go:30 (0x58f6ce)
        New: if actualType.Kind() == reflect.Func {
  /home/etiennebruines/workspaces/go-all/src/github.com/onsi/gomega/gomega_dsl.go:199 (0x508ac2)
        EventuallyWithOffset: return asyncassertion.New(asyncassertion.AsyncAssertionTypeEventually, actual, globalFailHandler, timeoutInterval, pollingInterval, offset)
  /home/etiennebruines/workspaces/go-all/src/github.com/onsi/gomega/gomega_dsl.go:181 (0x50892f)
        Eventually: return EventuallyWithOffset(0, actual, intervals...)
  /home/etiennebruines/workspaces/go-all/src/git.huperwebs.nl/nu-1-website/client-server/controllers/login_test.go:37 (0x48dfa9)
        func.007: Eventually(page.Find("#username").Fill("demo")).Should(Succeed())

but why?
It's a nil-pointer exception, because actualType (async_assertion.go line 30) is nil.
actualType is nil because reflect.TypeOf(actualInput) returns nil when actualInput is nil.
actualInput is the second argument (type interface{}) which has the variable name actual at the caller at line 199 at gomega_dsl.go.
Nothing is being done to that variable in EventuallyWithOffset and Eventually, it's just being passed along.
At the end, that actualInput (or actual) variable is the result of
page.Find("#username").Fill("demo")

However, .Fill returns an error object. This is nil...
Question
I don't really understand any of it (I don't know the gomega/agouti framework). It's identical to the demo code, but results in an error.
Background info:
Go version 1.3.3
Ginkgo version 1.1.0
ChromeDriver 2.14.313457

Perhaps it's something I did, but perhaps it is indeed a bug?
I managed to fix this by doing:
File async_assertion.go
Line 30 becomes:
if actualType != nil && actualType.Kind() == reflect.Func {

(added actualType != nil)
Line 65: becomes
return actualType != nil && actualType.Kind() == reflect.Func && actualType.NumIn() == 0 && actualType.NumOut() > 0

(added actualType != nil)

Hi @EtienneBruines,
Thanks for opening an issue! The example on agouti.org is indeed incorrect. The Eventually Gomega function may only be used directly with zero-argument *Selection methods, like so:
Eventually(page.FindByLabel("Gebruikersnaam").Click).Should(Succeed())

For other *Selection or *Page methods, you need to wrap them in a zero-argument function that returns the error, as you suggested:
Eventually(func() error {return page.FindByLabel("Gebruikersnaam").Fill("demo")}).Should(Succeed())

This is also why I provided a matchers package, as this:
Eventually(page.FindByLabel("Gebruikersnaam").Attribute("name")).Should(Equal("bob"))

suffers the same issue, but this:
Eventually(page.FindByLabel("Gebruikersnaam")).Should(HaveAttribute("name", "bob"))

is fine.
I'm leaning towards getting rid of the dsl package (which does not provide any asynchronous functionality), and adding matchers for all *Selection and *Page methods. Example:
Eventually(page.FindByLabel("Gebruikersnaam")).Should(BeFilledWith("demo"))

Would that be an improvement?
I've updated the gh-pages branch to fix the incorrect examples: 87c921fbfb54437c53be4fd19dafe95dce4663ad
Please don't hesitate to open more issues in the future!
-Stephen

After some thought, I realized that
Eventually(page.FindByLabel("Gebruikersnaam")).Should(BeFilledWith("demo"))

implies that Agouti should wait for that field to be filled by something else, which is not the intention. What would you think about matchers that start with Accept? Some examples:
Eventually(page.FindByButton("click me")).Should(AcceptClick())
Eventually(page.FindByLabel("username")).Should(AcceptFillWith("demo"))
Eventually(page.FindByLabel("pick one")).Should(AcceptSelectWith("some option"))

Not totally sure I like it though. Maybe @onsi would have some opinions?
-Stephen

@EtienneBruines, did this answer your question?
Seems like @onsi hasn't had a chance to look at this.

Tricksy.
It sounds like there are really two assertions being made here:
Eventually(page).Should(HaveElement("#username"))
Expect(page.Find("#username").Fill("demo")).To(Succeed())

I know it's more convenient to turn this into a one-liner but I think having a plethora of matchers to support setting things asynchronously may be more confusing than it's worth.
I prefer the explicitness of the two-liner (and it shouldn't be necessary too often?)

This makes the most sense to me. I've updated agouti.org accordingly. I worry a little bit about situations where the appearance of an element does not correlate to an action succeeding, but these are probably edge cases that can be solved be wrapping the action in a function.
@EtienneBruines, to clarify, your use case would look like:
Eventually(page.Find("#username")).Should(BeFound())
Expect(page.Find("#username").Fill("demo")).To(Succeed())

I'm closing this issue for now, but feel free to re-open it.


Gopass exits with stack trace while processing config.yml after upgrade to 1.10

Summary

After upgrading gopass from 1.9.x to 1.10.x, gopass was broken in the commandline but continues to work fine with gopass bridge
PS > gopass
Error: Failed to initialize store: failed to initialized stores: failed to initialize the root store at '/C:\Users\leogott\.password-store': failed to init storage backend: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.: failed to init storage backend: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.: failed to initialize the root store at '/C:\Users\leogott\.password-store': failed to init storage backend: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.: failed to init storage backend: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.

modifying path in the config located at ~.config\gopass\config.yml does not appear to change the behavior at all. Neither does gopass config --store path different/path
Renaming the store makes gopass open a seperate window for part of a second, but not return anything in the original window.
I am not aware of a method to learn if gopass is actually aware of the location of the config.
I tried providing the location via environment variables but was unsuccessful. Though it's possible i missed something.
Steps To Reproduce

upgrade an existing gopass installation on windows from 1.9 to 1.10
Expected behavior

gopass lists the contents of the store
Environment

OS: Windows 10 Pro
OS version: 19041.508
gopass Version: gopass 1.10.1+377d2ab8507ed6d0025a1d18acf390d38680d0aa (377d2ab8507ed6d0025a1d18acf390d38680d0aa) go1.14.2 windows amd64
Installation method: scoop

Additional context

(Also "failed to initialized stores" looks like a typo to me)

I may be seeing the same thing - just upgraded to 1.10.1 on mac, and lost everything.

I was able to repair gopass locally by renaming another config.yml I didn't know about yet.
By using $Env:GOPASS_DEBUG_LOG="path/to/logfile" I was able to determine that a config.yml existed at ~\AppData\Roaming\gopass\ and that there was an issue with it.
Most likely the only problem with it is the "protocol" part of the path entry, that was previously accepted(?) and no longer is.
root:
...
  path: gpgcli-gitcli-fs+file:///C:\...

After removing the broken config, gopass worked as expected. I created a new config with the output of gopass config.
Click for logfile ...
2020/11/22 19:59:42.520554 pwrules/aliases.go:52	pwrules.loadCustomAliases	no custom aliases found at C:\Users\leogott\AppData\Roaming\gopass\domain-aliases.json
2020/11/22 19:59:42.706555 config/io.go:37	config.loadConfig	Trying to load config from C:\Users\leogott\AppData\Roaming\gopass\config.yml
2020/11/22 19:59:42.714555 config/io.go:114	config.decode	Trying to unmarshal config into *config.Config
2020/11/22 19:59:42.716554 config/io.go:120	config.decode	Extra elements in config: unknown fields: [root]
2020/11/22 19:59:42.716554 config/io.go:114	config.decode	Trying to unmarshal config into *config.Pre193
2020/11/22 19:59:42.716554 config/io.go:126	config.decode	Loaded config: *config.Pre193: &{Path: Root:0xc00016df00 Mounts:map[] XXX:map[]}
2020/11/22 19:59:42.716554 config/io.go:129	config.decode	Loaded legacy config. Should rewrite config.
2020/11/22 19:59:42.716554 config/io.go:45	config.loadConfig	Loaded config from C:\Users\leogott\AppData\Roaming\gopass\config.yml: &config.Config{AutoClip:true, AutoImport:true, ClipTimeout:45, ExportKeys:false, MIME:true, NoColor:false, NoPager:false, Notifications:true, Path:"/C:\\Users\\leogott\\.password-store", SafeContent:false, Mounts:map[string]string{}, configPath:"C:\\Users\\leogott\\AppData\\Roaming\\gopass\\config.yml", XXX:map[string]interface {}(nil)}
2020/11/22 19:59:42.720555 root/init.go:17	root.(*Store).Initialized	initializing store and possible sub-stores
2020/11/22 19:59:42.720555 root/init.go:65	root.(*Store).initialize	initialize - /C:\Users\leogott\.password-store
2020/11/22 19:59:42.721554 leaf/store.go:51	leaf.New	Instantiating  at /C:\Users\leogott\.password-store
2020/11/22 19:59:42.722555 backend/storage.go:74	backend.DetectStorage	Trying gitfs for /C:\Users\leogott\.password-store
2020/11/22 19:59:42.724555 fsutil/fsutil.go:49	fsutil.IsDir	failed to check dir \C:\Users\leogott\.password-store\.git: CreateFile \C:\Users\leogott\.password-store\.git: The filename, directory name, or volume label syntax is incorrect.
2020/11/22 19:59:42.724555 backend/storage.go:76	backend.DetectStorage	failed to use gitfs for /C:\Users\leogott\.password-store: no .git
2020/11/22 19:59:42.724555 backend/storage.go:74	backend.DetectStorage	Trying ondisk for /C:\Users\leogott\.password-store
2020/11/22 19:59:42.724555 fsutil/fsutil.go:64	fsutil.IsFile	failed to check file \C:\Users\leogott\.password-store\index.gp1: CreateFile \C:\Users\leogott\.password-store\index.gp1: The filename, directory name, or volume label syntax is incorrect.
2020/11/22 19:59:42.724555 backend/storage.go:76	backend.DetectStorage	failed to use ondisk for /C:\Users\leogott\.password-store: not supported
2020/11/22 19:59:42.724555 backend/storage.go:74	backend.DetectStorage	Trying fs for /C:\Users\leogott\.password-store
2020/11/22 19:59:42.724555 fsutil/fsutil.go:49	fsutil.IsDir	failed to check dir /C:\Users\leogott\.password-store: CreateFile /C:\Users\leogott\.password-store: The filename, directory name, or volume label syntax is incorrect.
2020/11/22 19:59:42.724555 backend/storage.go:76	backend.DetectStorage	failed to use fs for /C:\Users\leogott\.password-store: dir not found
2020/11/22 19:59:42.725573 action/errors.go:58	action.ExitError	Stacktrace: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.
unknown storage backend
github.com/gopasspw/gopass/internal/store/leaf.(*Store).initStorageBackend
	src/github.com/gopasspw/gopass/internal/store/leaf/storage.go:16
github.com/gopasspw/gopass/internal/store/leaf.New
	src/github.com/gopasspw/gopass/internal/store/leaf/store.go:59
github.com/gopasspw/gopass/internal/store/root.(*Store).initialize
	src/github.com/gopasspw/gopass/internal/store/root/init.go:66
github.com/gopasspw/gopass/internal/store/root.(*Store).Initialized
	src/github.com/gopasspw/gopass/internal/store/root/init.go:18
github.com/gopasspw/gopass/internal/action.(*Action).Initialized
	src/github.com/gopasspw/gopass/internal/action/init.go:29
main.setupApp.func2
	src/github.com/gopasspw/gopass/main.go:144
github.com/urfave/cli/v2.(*App).RunContext
	pkg/mod/github.com/urfave/cli/v2@v2.2.0/app.go:315
main.main
	src/github.com/gopasspw/gopass/main.go:95
runtime.main
	go/src/runtime/proc.go:203
runtime.goexit
	go/src/runtime/asm_amd64.s:1373
failed to init storage backend: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.
github.com/gopasspw/gopass/internal/store/leaf.New
	src/github.com/gopasspw/gopass/internal/store/leaf/store.go:60
github.com/gopasspw/gopass/internal/store/root.(*Store).initialize
	src/github.com/gopasspw/gopass/internal/store/root/init.go:66
github.com/gopasspw/gopass/internal/store/root.(*Store).Initialized
	src/github.com/gopasspw/gopass/internal/store/root/init.go:18
github.com/gopasspw/gopass/internal/action.(*Action).Initialized
	src/github.com/gopasspw/gopass/internal/action/init.go:29
main.setupApp.func2
	src/github.com/gopasspw/gopass/main.go:144
github.com/urfave/cli/v2.(*App).RunContext
	pkg/mod/github.com/urfave/cli/v2@v2.2.0/app.go:315
main.main
	src/github.com/gopasspw/gopass/main.go:95
runtime.main
	go/src/runtime/proc.go:203
runtime.goexit
	go/src/runtime/asm_amd64.s:1373
failed to initialize the root store at '/C:\Users\leogott\.password-store': failed to init storage backend: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.
github.com/gopasspw/gopass/internal/store/root.(*Store).initialize
	src/github.com/gopasspw/gopass/internal/store/root/init.go:68
github.com/gopasspw/gopass/internal/store/root.(*Store).Initialized
	src/github.com/gopasspw/gopass/internal/store/root/init.go:18
github.com/gopasspw/gopass/internal/action.(*Action).Initialized
	src/github.com/gopasspw/gopass/internal/action/init.go:29
main.setupApp.func2
	src/github.com/gopasspw/gopass/main.go:144
github.com/urfave/cli/v2.(*App).RunContext
	pkg/mod/github.com/urfave/cli/v2@v2.2.0/app.go:315
main.main
	src/github.com/gopasspw/gopass/main.go:95
runtime.main
	go/src/runtime/proc.go:203
runtime.goexit
	go/src/runtime/asm_amd64.s:1373
failed to initialized stores: failed to initialize the root store at '/C:\Users\leogott\.password-store': failed to init storage backend: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.: failed to init storage backend: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.: unknown storage backend: mkdir /C:: The filename, directory name, or volume label syntax is incorrect.
github.com/gopasspw/gopass/internal/store/root.(*Store).Initialized
	src/github.com/gopasspw/gopass/internal/store/root/init.go:19
github.com/gopasspw/gopass/internal/action.(*Action).Initialized
	src/github.com/gopasspw/gopass/internal/action/init.go:29
main.setupApp.func2
	src/github.com/gopasspw/gopass/main.go:144
github.com/urfave/cli/v2.(*App).RunContext
	pkg/mod/github.com/urfave/cli/v2@v2.2.0/app.go:315
main.main
	src/github.com/gopasspw/gopass/main.go:95
runtime.main
	go/src/runtime/proc.go:203
runtime.goexit
	go/src/runtime/asm_amd64.s:1373

Windows users should make sure their path config entry looks like this:
path: c:\path\to\your\password-store
without gpgcli-gitcli-fs+file:/// prepended.
I guess the migration path from previous config might have failed on Windows at some point in the past.
When a broken config file is causing issues, the prompt should now tell what settings are the issue and where the config file is located, thus helping with the resolution of such issues.
All tests passes on Windows since #1680 so hopefully Gopass works better on Windows now than it used to.
Closing since it seems solved to me. Let us know if it's failing again with the future releases, and we'll dig into it.

Windows users should make sure their path config entry looks like this:
path: c:\path\to\your\password-store
without gpgcli-gitcli-fs+file:/// prepended.
I guess the migration path from previous config might have failed on Windows at some point in the past.
When a broken config file is causing issues, the prompt should now tell what settings are the issue and where the config file is located, thus helping with the resolution of such issues.
All tests passes on Windows since #1680 so hopefully Gopass works better on Windows now than it used to.
Closing since it seems solved to me. Let us know if it's failing again with the future releases, and we'll dig into it.


PoC Network - Lag in creating products using UI

Release- v1.4.0
Env. - nvs.pla.health
It is taking lot of time to create/update product and batches even on a fast network and without VPN. Creating this issue to track performance improvements.
As per @salboaie -

To create a new product from UI will take around 40 seconds. We will work on the next releases to make it better.  It is not really dependent on the cloud resources, it is more on how many interactions with the blockchain network we have. The network speed also plays a role.
We are working to reduce this timing to something la 10-15 seconds on a network in Europe (we have concrete plans and open tasks to achieve this in the next release).

Fixed by enabling OBA blockchain anchoring


Feature request: Link to specific view

The easiest way to explain is an example:
https://www.google.com/maps/@51.9816396,5.9123597,15z
Google maps allows users to copy/paste the URL to save or share their view of the map. It would be great if the pointclouds viewer had a similar feature. This would make communication about particular sections of the map/pointcloud much easier.

I realise it's a bit late, but there's good news on this. The feature has been implemented in the new develop branch, we will update the production version of the viewer soon(tm).


[BUG] can not upload files

i want to upload a  image file,  here is my code

i has try to upload it in swagger and post man ,both got a 422 error: There was an error parsing the body

please give me help!

I assume you were able to solve your problem. Thanks for closing the issue.
Feel free to add more comments or new issues.

How was the issue solved? I'm experiencing the same problem.

How was the issue solved? I'm experiencing the same problem.

you can get the UploadFile object

How was the issue solved? I'm experiencing the same problem.

https://github.com/tiangolo/fastapi/issues/405#issuecomment-610369383 i had write a demo , maybe can make some help for you

How was the issue solved? I'm experiencing the same problem.

i had write a demo application: https://github.com/tiangolo/fastapi/issues/405#issuecomment-610369383  hope can give some help to you

More information for the fresh project, you have to install python-multipart. Refer to this and this links.


fix: set configuring writers from options not tables

Set configuring writers from options, not tables, because tables will be always more (since they are from spec object) than tables in option.selection

Fixed in other PR


Create schema.org person transform

Given a VIVO Person object (https://wiki.lyrasis.org/display/VIVODOC110x/Person+Model), the transform should spit out a schema.org Person object (https://schema.org/Person).  This should be isomorphic, ie run in both Node and browser JS lands.

Looking at Bryan Jenkins, I see jobTitle represented as
"jobTitle": [
    "AGRON AES-B/E/E",
    "PROF-FY-B/E/E",
    "DEPARTMENT CHAIR",
    "Chair and Professor"
  ],

In the Schema.org example, there's this example:
"jobTitle": {
        "@type": "DefinedTerm",
        "inDefinedTermSet": "https://targetjobs.co.uk/careers-advice/job-descriptions",
        "termCode": "277133-aid-workerhumanitarian-worker-job-description",
        "name": "Aid worker/humanitarian worker",
        "url": "https://targetjobs.co.uk/careers-advice/job-descriptions/277133-aid-workerhumanitarian-worker-job-description"
    }

Perhaps our jobTitle should be changed to an array of name objects, e.g.,
 "jobTitle": [
    {
      "@type": "Text",
      "name": "AGRON AES-B/E/E"
    },
    {
      "@type": "Text",
      "name": "PROF-FY-B/E/E""
    },
    {
      "@type": "Text",
      "name": "DEPARTMENT CHAIR""
    },
    {
      "@type": "Text",
      "name": "Chair and Professor"
    }
  ]

Re: "knowsAbout," the application of DefinedTerms to people appears to be an AE (JM) innovation, as  this page doesn't appear to depict that but rather limits terms to describing a work's aboutness.


Strategies for variant prioritization

[ ]  check the MAF's of variants (shouldn't be frequent) - can be done with VEP
[ ]  filter on expression
[ ]  filter/sort on the delta
[ ] Check if the top variants are known cancer variants

Use ClinVar
vcf from ICGC

Heres a list of cancer variant databases:
https://github.com/seandavi/awesome-cancer-variant-databases
Cheers!
Ben
On Wed, Nov 9, 2016 at 9:22 AM, Friederike Dündar<EMAIL_ADDRESS>wrote:

check the MAF's of variants (shouldn't be frequent) - can be done
with VEP
filter on expression
filter/sort on the delta
Check if the top variants are known cancer variants

Use ClinVar
vcf from ICGC

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NCBI-Hackathons/Cancer_Epitopes_CSHL/issues/8, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFePtS8Cp5UeK5iPQ9Ci6sHe2Xze2gqPks5q8dcggaJpZM4KtlNY
.

--
What have you done today to make the world a better place?


Support for private properties

I have been browsing around the issus for a while and found some related things (#246, #2319) but nothing that matches exatctly. I am trying to make EF map a private property.
Steps to reproduce
Consider the following Code:
namespace ConsoleApplication
{
     public class Something
    {
        public Something()
        {
            SecretInfo = DateTime.Now.ToString();
        }

        public int SomethingId { get; set; }
        public string Name { get; set; }
        private string SecretInfo { get; set; }

    }

    public class SomethingContext : DbContext
    {
        public DbSet<Something> Somethings { get; set; }

        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        {
            optionsBuilder.UseSqlServer(@"Server=(localdb)\mssqllocaldb;Database=EFCoreTest;Trusted_Connection=True;");
        }

        protected override void OnModelCreating(ModelBuilder modelBuilder)
        {
            modelBuilder.Entity<Something>()
                .Property(b => b.SomethingId).UseSqlServerIdentityColumn();

            modelBuilder.Entity<Something>()
                .Property(typeof(string), "SecretInfo").HasColumnName("SecretInfo");
        }

    }

    public class Program
    {
        public static void Main(string[] args)
        {
            using(var blogDb = new SomethingContext())
            {
                var something = new Something()
                {
                    Name = "My awesome Something"
                };

                blogDb.Somethings.Add(something);
                blogDb.SaveChanges();
            }
        }
    }
}

The issue
In the database appears a table Somethings with columns SomethingId, Name and SecretInfo. The first two columns are filled properly, while the third column remains null. The expected outcome would have been that it contains the value of DateTime.Now.ToString() as this is the value of the field set in the constructor of Something.
Further technical details
EF Core version: "Microsoft.EntityFrameworkCore.SqlServer": "1.0.0-rc2-final",
Operating system: Windows 10 Enterprise Visual Studio version: VS Code 1.2.0

The code sample in #2319 finally lead me twoards a solution for my problem. I added an override for SaveChanges in SomethingContext:
 public override int SaveChanges()
        {
            foreach (var entry in ChangeTracker.Entries())
            {
                foreach (var pi in entry.Entity.GetType().GetProperties(BindingFlags.Instance | BindingFlags.NonPublic))
                {
                    entry.Property(pi.Name).CurrentValue = pi.GetValue(entry.Entity);
                }
            }
            return base.SaveChanges();
        }

This sets the values of all private properties to the corresponding change-tracker entries.This way they end up in the database during insert / update.
If this is the correct way to go this issue can be closed - if there is an easier route to success I would still be intereseted.

@trichling I ran your repro code with the current bits and the SecretInfo property was correctly persisted to the database. I think what you are seeing is a consequence of issue #5446, which was fixed post RC2.


Use 'webapp' alias for RP template

Fixes #6692

My approval indicates that this looks good once the suggested changes are made.


Merge visits from several files

The ability to merge line visits if the same lines are found across other files.
This could be useful if you have Unit tests and Integration tests and need to see the overall coverage of your library or application.
This will require me to create an internal model to track these things and rethink the caching methods too.

Solved with
https://github.com/ppumkin/PrestoCoverage/pull/7


Suggestion ( Banks )

Right, here is what i been thinkin'
There will be a new button added to the bot saying - Bank Logging - ( Or something like it )
So, what this button does, it searches through ( Put desired amount here ) of your bank ips ( Already Hacked bank IPS ) And if they have any money ( Put desired money here. If money over 1000$, Steal, if below ( Desired Amount ) then leave. )
It would be a cool concept to do, but im curious about security.
If this is possible, i'd like the devs to have a go at it.
Best Regards,
Wolf.

Nice! I would like to see this feature working.
I'll try to code something.


Master

Updated class names from .box to .card and .box-body to .card-body
Updated Glyphicons to FontAwesome
Added additional classes to .table

@aniqakhan755 we do no longer use master branch and also package name is changed in this PR. so it can not be merged.


causes error when running "npm i"

cross-env NODE_ENV=production rollup -c

events.js:160
throw er; // Unhandled 'error' event
^
Error: spawn rollup ENOENT
at exports._errnoException (util.js:1036:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
at onErrorNT (internal/child_process.js:359:16)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)
at Module.runMain (module.js:592:11)
at run (bootstrap_node.js:394:7)
at startup (bootstrap_node.js:149:9)
at bootstrap_node.js:509:3
npm ERR! Darwin 16.5.0
npm ERR! argv "/Users/qj/.nvm/versions/node/v6.7.0/bin/node" "/Users/qj/.nvm/versions/node/v6.7.0/bin/npm" "run" "build"
npm ERR! node v6.7.0
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: cross-env NODE_ENV=production rollup -c
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'cross-env NODE_ENV=production rollup -c'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the vue-echarts package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     cross-env NODE_ENV=production rollup -c
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs vue-echarts
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls vue-echarts
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/qj/Work/FE/vue-echarts/npm-debug.log
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of vue@* but none was installed.
npm WARN<EMAIL_ADDRESS>No repository field.
npm ERR! Darwin 16.5.0
npm ERR! argv "/Users/qj/.nvm/versions/node/v6.7.0/bin/node" "/Users/qj/.nvm/versions/node/v6.7.0/bin/npm" "i"
npm ERR! node v6.7.0
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>prepublish: npm run demo && npm run build && cp -r ./src/* . && rm index.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prepublish script 'npm run demo && npm run build && cp -r ./src/* . && rm index.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the vue-echarts package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run demo && npm run build && cp -r ./src/* . && rm index.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs vue-echarts
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls vue-echarts
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:

Hi, the latest version should solve this problem. Welcome to try and reopen this issue if the problem still exists.


dmg files need to be tested

I'd be surprised if dmg (end-512) works rn. It might work, but needs to be tested.

dmg files work rn, but they also return true as bz2 ... which is also correct.


React Tooltips

Description
Convert our Sage Tooltips to a React flavored version. These tooltips don't have a wrapping DOM element and pass mouse events to components that it wraps.
Thanks to @pixelflips for working out the original tooltip positioning math. 🤘
Screenshots

Test notes
See the storybook for example:
http://localhost:6006/?path=/story/sage-tooltip--default

Tooltip should Portal outside of the trigger DOM location
Check to ensure React Component is perfomant

Related
BUILD-367

A11y consideration: the tooltip doesn't open if it is tabbed into, it's working correctly with a mouse interaction


fix ordering of alpha and beta spins in example notebook

In the LUCJ circuit, alpha qubits come before beta qubits. In Qiskit, qubits are ordered from right to left. Therefore, the alpha qubits are on the right part of the bitstring, and the beta qubits are on the left. In this particular case the numbers of alpha and beta electrons are the same, which is why there is no error, but the distinction is material in the open-shell case.

Thanks for bringing this up @kevinsung . This is a bit deeper of a problem with the API. I have opened #11 to bring more consistency to the API and a bit more clarity in the docs.

I believe we will merge #11 and do a quick release to improve the workflow. Thanks again 🚀


⚠ 广电VRBT has degraded performance

In eed5f1b, 广电VRBT (https://vrbt.10099.com.cn/#/home) experienced degraded performance:

HTTP code: 502
Response time: 7722 ms

Resolved: 广电VRBT performance has improved in 6f80585 after 1 hour, 17 minutes.


[ServerClient] Creating and Updating attachments returns 415

Hello!
First of all thanks for this awesome lib! kudos to you 🥳
I think this is related to #23 , according to the [server documentation][https://docs.atlassian.com/ConfluenceServer/rest/7.18.3/#content/{id}/child/attachment-updateData] in order to update an attachment data the content-type header of the request needs to be multipart/form-data but it's currently being set to application/json
Please let me know if you need code examples and files, I can provide those, I'm reading a bit the code, trying to understand things, I will also try to submit a fix for this later🤞
Best regards
Gonzalo G

Hi @ggarcia24! Thank you! I will wait your PR :)

Released in v1.5.2. Thanks for your work


use rapidfuzz instead of fuzzywuzzy

FuzzyWuzzy is GPLv2 licensed which would force you to licence the whole project under GPLv2. I had the same problem on one of my projects and so I wrote rapidfuzz which is implementing the same algorithm but is based on a version of fuzzywuzzy that was MIT Licensed and is therefor MIT Licensed aswell, so it can be used in here without forcing a License change. As a nice bonus it is fully implemented in C++ and comes with a few Algorithmic improvements making it between 5 and 100 times faster than FuzzyWuzzy. (actually even faster since python-Levenshtein was not used)

I do not want to impose a C++ build system on my users. Especially since installing C/C++ tool chain on windows is a huge task.

@vn-ki there are prebuild wheels for Windows https://pypi.org/project/rapidfuzz/0.2.1/#files

One of the reason I do not want to swap out a dependency because I do not maintain the brew formula and it would cause more headaches than the fear of someone trying to sue me for misusing the GPL license.
As for the breach of license, this is a tool which downloads anime illegally from websites. I think a breach of license is the least illegal thing we're doing. (I'm not proud of either btw)
I will discuss with the one who maintains the formula and see what can be done.


deleting more pycache files and adding ci/cd

I grabbed the CI/CD we have for indico-operator and made some changes to it. Note that the integration tests workflow is deleted for now because we have no integration tests so far. I still left the build and publish oci images untouched so we can use them in the near future.

The images are not being built. This is normally done in the integration_test.yaml
Consider copying it and adding an empty test for future integration tests.

The images are not being built. This is normally done in the integration_test.yaml Consider copying it and adding an empty test for future integration tests.

This is a good idea. Please add a very basic integration test as part of this. Something that just checks the application deploys and reaches active state would be fine.

Looks good. Have you tested it? You can trigger the "release" wf by pushing a track/* branch

We'll have to wait until I get my token for charmhub to test that workflow, the rest of the workflows and the integration test are working properly.

@gtrkiller I changed the repository settings to be in-line with the contributing guide, specifically related to the number of reviewers and to require new reviews after commits


The impact of valid on the results

Sorry to bother you again. I wonder whether train and valid can be the same?Or could valid be part of the train?I want to reproduce the result of the paper, but I haven't got a good result.
What I am trying to do now is to copy 100 pieces from each block generated by each picture in the train to the valid.I use the Linux platform and the environment is the same

Hi. Sorry for the slight delay in responding. You can get your validation set from your training set. I.e you can randomly select a proportion of your training images for validation and then extract patches accordingly.

What results are you getting? You should be able to obtain results similar to what we report in the paper.

Thank you for your reply.
Paper: 0.635 0.631 0.566 0.426
my: 0.53310 0.63373 0.51674 0.27748
Sometimes (after 50 rounds, and often in the last few rounds) the classification accuracy of the last class is 0

Take a look at #61 for this

Thank you for your reply
I took a closer look at the links you gave me, but I didn't get into how the training and validation sets I mentioned were split, and my environment is exactly the same as yours.It's just that the results are quite different, as I said above.

@zerodohero ，hi, do you solove the problem what you said about :  Sometimes (after 50 rounds, and often in the last few rounds) the classification accuracy of the last class is 0.   Now I meet same question with you. And I wonder know whether the dataset I use in valid is error(I use test as valid).

@zerodohero ，hi, do you solove the problem what you said about : Sometimes (after 50 rounds, and often in the last few rounds) the classification accuracy of the last class is 0. Now I meet same question with you. And I wonder know whether the dataset I use in valid is error(I use test as valid).

I have solved it myself, you should separate Val from train instead of using test, or maybe there is something wrong with your environment, is it Linux system?

@zerodohero ，Yes, it's Linux system. And I just get the vaild set from train set. and it get the valid_dice_Miscellaneous=0 after about 70 epochs(but when I get the valid set from test set it doesn't appear.). By the way , I get valid set from train set but not delet them in train set. should I delete them? Or what should I do.

@zerodohero ，I have try some methods to get valid set but all have some problems. And I really don't know how to divide the valid set now. Can you tell the detail how you divide? I'd appreciate if you teach me. Please

@zerodohero @haiqinzhong i am facing the same issue for the Miscellaneous class, how did u guys solve it, i would be really grateful if you could share?? Thankyou


[flutter_tools] Add a space between URL and period

Use case
I am developing flutter application. and I face this warning as follows:
$ flutter run

Launching lib/main.dart on Pixel 4 in debug mode...
[!] Your app isn't using AndroidX.
    To avoid potential build failures, you can quickly migrate your app by following the steps on
    https://goo.gl/CP92wY.
Running Gradle task 'assembleDebug'...
(This is taking an unexpectedly long time.)       ⡿

But I think this URL is invalid because it contains ..
Proposal
How about adding a space between URL and period?
https://github.com/flutter/flutter/blob/e092dcfa22cef503cfdceea67e45c49c0aa8062d/packages/flutter_tools/lib/src/android/gradle.dart#L250
https://github.com/flutter/flutter/blob/03c566768b6baf32b5631e212e48cea2f20b0d0d/packages/flutter_tools/test/general.shard/android/gradle_errors_test.dart#L326
and more.

Before

https://goo.gl/CP92wY.

After

https://goo.gl/CP92wY .

Regards

Hi @MizukiSonoko
Does it cause issues when selecting the link?

Yes
In VSCode,  we can not access this URL by clicking.
I understand  VS Code is an official editor tool on Flutter, so this error is not good

Thanks for the bug report @MizukiSonoko , sorry for the delay in getting it fixed


Bug 1840539: [release 4.5] Remove the host from the crush map

This change will remove the host associated with the failed osd from the
crush map. When node removal process is done and all OSDs have been
removed from that host, the empty host will be removed from crush map.
The user will be notified that the error while removing host should be
ignored if there are other OSDs present on the same host
Signed-off-by: Servesha Dudhgaonkar<EMAIL_ADDRESS>

/retest


Use private field visibility in generated source.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/bdg-formats-prb/198/
Test PASSed.

Closing as WontFix


Modify image names to match the one in dockerhub.

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Base Tests Mac failures: #11223
Portability Tests Linux: #9057


passing a tsconfig.json path to tsb.create no longer works for typescript 1.8.x

In this case tsb.create(path) returns a nop compilation function that returns null.

@weinand are you experiencing it under OS X? Using it under Ubuntu works fine. However, it seems that npm install gulp-tsb installs different content for each platform and the problem is that in line 11 of the OS X version fs_1.readFileSync(path, undefined); should be fs_1.readFileSync(path, 'utf-8'); instead. @jrieken what could be the reason that the same version of the package is delivering different content for different platforms?

This seems to be the same issue as #36
Btw I checked the latest v1.10.2 on Ubuntu, the problem is there as well so not only OS X is affected.

it seems like a dup of https://github.com/jrieken/gulp-tsb/pull/33

fixed with either #33 or https://github.com/jrieken/gulp-tsb/commit/e28375af4f7375fb79bbc4d5e0d4f9d94ddb56ab


robustness: Validate $ZSH_HIGHLIGHT_STYLES keys

If the $ZSH_HIGHLIGHT_STYLES association contains an unknown key, warn about it at startup. (But for compatibility reasons, don't make this a fatal error.)
For example:
ZSH_HIGHLIGHT_STYLES[zzz]='fg=red'
source zsh-syntax-highlighting.zsh

should trigger a warning, because zzz is not a recognized key.

This isn't quite straightforward since the $ZSH_HIGHLIGHT_STYLES parameter is shared by all highlighters; keys that one highlighter doesn't recognize might be meaningful for another highlighter.
This is probably not solvable without an API change (either the user-facing API, or the modules API).


Allow showing and hiding links with filters

From Jonathan:

I want an option to add a link to both global and my private repo, or failing that, a view of the public repo that shows only my contributions (and maybe my topic contributions with any content under them)

One first step towards this I've been thinking about is the use of filters along the righthand side.  One filter shows links in your private repo, and another filter shows all the stuff in wiki, etc.

^ @tensiondriven fyi


Update Hyperband Tutorial to new API

[ ] Updates Tuning.ipynb tutorial to new API
[ ] Recover tuning results from @agnusmaximus initial notebook

@jdunnmon @agnusmaximus this is great, lmk when passing tests!  Initial comments:

There's an argument for doing random search for the tutorial, and then just checking hyperband separately... but either way works
Keep in mind that this is getting run by Travis, so please make sure it doesn't take too long to run (also note it's currently failing due to what looks like a Queue error- maybe run single threaded for tutorial?)
For the tutorial, perhaps run in a less verbose mode?  Otherwise this generates an annoyingly large amount of diffs

Otherwise looks great so far, thanks!


Issue with k8s.io/docs/setup/certificates/

This is a Bug Report

There is an inconsistency in the nomenclature between the table in the portion "All Certificates" and the table in the portion "Certificate Paths".  kube-apiserver-etcd-client and kube-apiserver-kubelet-client in "All Certificates" apparently are named apiserver-etcd-client and apiserver-kubelet-client in "Certificate Paths".
Proposed Solution:
Change the nomenclature to one or the other.
Page to Update:
https://kubernetes.io/docs/setup/certificates/

/language en

/priority backlog


Should be able to define target_overrides in libraries

I have a library that depends on mbed Client. In the past I'd just reference mbed Client as a dependency and be done with it. Now I need to declare the CLIENT feature in mbed_app.json. I added an mbed_app.json file to my library but it's not being picked up by mbed-cli.
That's highly annoying because now I need to tell the user to not only reference my app but also add stuff to this config file, where in the past this would go automatically when he includes my library.
@sg- @screamerbg

I think you need to use mbed_lib.json instead?
It looks like the config system docs are still in a PR: https://github.com/mbedmicro/mbed/pull/1984/files

Yep, that worked. Thanks.


3.0.4

Added

feat: make testsuite and run testsuite
feat: testcase/testsuite config support getting variables by function
feat: har2case with request cookies
feat: log request/response headers and body with indent

Fixed

fix: extract response cookies
fix: handle errors when no valid testcases generated

Changed

change: har2case do not ignore request headers, except for header startswith :

Codecov Report

Merging #902 into master will decrease coverage by 0.21%.
The diff coverage is 74.43%.

@@            Coverage Diff             @@
##           master     #902      +/-   ##
==========================================
- Coverage   80.31%   80.10%   -0.22%
==========================================
  Files          34       34
  Lines        2063     2166     +103
==========================================
+ Hits         1657     1735      +78
- Misses        406      431      +25

Flag
Coverage Δ

#unittests
80.10% <74.43%> (-0.22%)
:arrow_down:

Impacted Files
Coverage Δ

httprunner/response.py
62.74% <ø> (ø)

httprunner/cli.py
35.16% <6.66%> (-0.89%)
:arrow_down:

httprunner/loader.py
79.67% <66.66%> (-1.76%)
:arrow_down:

httprunner/ext/make/__init__.py
82.40% <76.54%> (-4.56%)
:arrow_down:

httprunner/__init__.py
100.00% <100.00%> (ø)

httprunner/client.py
79.54% <100.00%> (+0.72%)
:arrow_up:

httprunner/exceptions.py
100.00% <100.00%> (ø)

httprunner/ext/har2case/core.py
90.00% <100.00%> (-0.30%)
:arrow_down:

httprunner/ext/make/make_test.py
100.00% <100.00%> (ø)

httprunner/loader_test.py
100.00% <100.00%> (ø)

... and 5 more

Continue to review full report at Codecov.

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


VCPU Shutdown Request on Windows 10 Pro, Intel i5 8th Gen, 8GB RAM

Describe the Bug

Summary:
Host Environment

HAXM version: Tried with 7.3.2 and 7.4.1

Host OS version: Windows 10 Pro version

Host OS architecture: x86_64

Host CPU model:  Core i5-8250U

Host RAM size: 8GB

(Optional) Host computer model: HP Probook G5

Guest Environment

Android Emulator or QEMU version:

Guest OS version: Tried with Lolipop, Nougut, Oreo

Guest OS architecture: tried with both x86 and x86_64

To Reproduce

Steps to reproduce the behavior:

emulator -avd Nexus_5X_API_27 -netspeed full -netdelay none

Expected Behavior

Expectation: Should up & run the emulator
Reproducibility

100%
Diagnostic Information

Host crash dump: VCPU Shutdown Request (Repeatedly)

HAXM log:

Android Emulator or QEMU log:

Screenshots:
Additional context

None

Would you get log on Windows, https://github.com/intel/haxm/blob/master/docs/manual-windows.md#Capturing driver logs

Would you get log on Windows, https://github.com/intel/haxm/blob/master/docs/manual-windows.md#capturing-driver-logs , and what is your emulator version? You could get the version in Android Studio. About steps, do you mean the emulator crash just after you launch it, by emulator -avd Nexus_5X_API_27? Guest(Emulator) Crash, not Host(Windows) Crash?

thanks for the reply. only thing in the powershell is showing as "VCPU Shutdown request" repeatedly. Yes the emulator crashed. not windows.
emulator is working fine if i used ARM based architecture which is very slow.

Would you get log on Windows, https://github.com/intel/haxm/blob/master/docs/manual-windows.md#capturing-driver-logs , and what is your emulator version? You could get the version in Android Studio.

Issue has been resolved with https://github.com/intel/haxm/releases/tag/v7.5.1 HAXM v 7.5.1 release

The issue is still open for me with 7.5.1

The issue is still open for me with 7.5.1

Which version of emulator you are using?
Does the issue still happen if only set only one vCPU?

Have the same issue

Hi @freelancerliketodev , could you provide the details configuration when met this issue? Eg. HAXM version, host OS version, HW config, Android image version.

Hi, I also have the same problem (100% reproducable):
Host:
CPU i5
RAM: 16 GB
Windows 7
HAXM 7.6.5
Android emulator version <IP_ADDRESS> (build_id 7033400)
Guest environment:
Android 11.0 - tested with both "Emulated performance" options: software and hardware
CPUs: tested 1 and 2
Command to reproduce:
emulator -avd Pixel_3a_API_30_x86


Utilizing a virtual environment for against global dependencies, and usage of sudo

Greetings, I noticed the usage of sudo in pip in the instructions. This is usually a security issue, and a frowned-upon practice. Allow me to introduce a better method : usage of python's virtual environment, so all the package management can be localized within the repository itself. That way, you do not need to use utilize super user privileges to get the script working.
Best,
Nobel

Thanks for the fix, and sorry for late response. :)


Postgres Enhancements on GlusterFS

As we were running into this issue a few times running postgres on GlusterFS

Thereby OP indicated that turning off open-behind and write-behind mitigated the issue.
via https://bugzilla.redhat.com/show_bug.cgi?id=1518710

So we should make sure we can turn off the open-behind and write-behind values for the Postgres volumes that run on Openshift

Together with #297 that should fix our issues we see with postgres at the moment

Closing as we don't have demand for postgres at the moment


route var name as parameter in ng g module --routing =true

Command
generate
Description
We would like to be able to choose a specifc variable other than "routes" in the routing module when we generate modules and components that are to be routed we have a branch for it pulling from main however it seems we can push our branch for merge consideration
Describe the solution you'd like
Here is the solution we have so far
from the respective index.ts,schema.json we would like an additional option
export function addRouteDeclarationToModule(
  source: ts.SourceFile,
  fileToAdd: string,
  routeLiteral: string,
  routesVarNameParam?:string
): Change {
...
if (ts.isArrayLiteralExpression(routesArg)) {
    routesArr = routesArg;
  } else {
    const routesVarName = routesVarNameParam ?? routesArg.getText();
    let routesVar;
    if (routesArg.kind === ts.SyntaxKind.Identifier) {
      routesVar = source.statements.filter(ts.isVariableStatement).find((v) => {
        return v.declarationList.declarations[0].name.getText() === routesVarName;
      });
    }
...
}

Describe alternatives you've considered
pulling down the source code and making our own schematic which is working well

Hi @windmillcode0,
Thanks for the issue but providing an option to change the variable name is not something that we'd like to add as a public API.
In this case, extending the module schematic is the right approach.
Thanks.


Caching in circleci doesn't work

error computing cache key: template: cacheKey:1:23: executing "cacheKey" at <checksum "Pipfile.lo...>: error calling checksum: open /home/circleci/repo/Pipfile.lock: no such file or directory

More work on https://github.com/dekespo/AI_ToolKit/pull/32 , but cannot make it work. For some reason it cannot save the file Pipfile.lock

See the forum and suggestion here: https://discuss.circleci.com/t/cannot-find-pipfile-lock-file-while-restoring-cache-in-python-3-7-1-image/33084/2


Add method to created conditional writer that takes no config.

In AccumuloClient a method for creating conditional writers that takes no config could be added.  Doing this would probably require also adding client properties for the conditional writer.

Are you suggesting having a method that accepts a table name as its sole parameter?

Doing this would probably require also adding client properties for the conditional writer.

What do you think would need to be added?

In AccumuloClient a method for creating conditional writers that takes no config could be added.

We could add an overloaded createConditionalWriter(String tableName) that just calls createConditionalWriter(tableName, new ConditionalWriterConfig()), but conditional writers seem used infrequently enough that it doesn't really warrant the overload, since this is already pretty easy to do.

Doing this would probably require also adding client properties for the conditional writer.

It wouldn't require it, but we could make it so new ConditionalWriterConfig() gets default values from client properties. How common is it to have an application need to create many different ConditionalWriters with the same ConditionalWriterConfig, though? And, it seems convenient enough to be able to reuse a ConditionalWriterConfig instance and pass it as the second arg in those cases. I'm not sure the advantage to growing client properties over simply reusing the object.

Are you suggesting having a method that accepts a table name as its sole parameter?

Yeah that is what I was thinking, like createBatchWriter(String), createBatchScanner(String), createScanner(String).  All of those method were added for 2.0. I had wanted to add one for the conditionalWriter before 2.0 to make it consistent, but I suppose it slipped through the cracks.

Making it consistent is a good idea. I'm still not sure about changing the defaults via client properties though. That may or may not be worth it.

It does seem like it could clean up ConditionalWriterIT a bit seeing that createConditionalWriter(tableName, new ConditionalWriterConfig())) is used a good number of times.

I'm still not sure about changing the defaults via client properties though.

I was thinking the client prop behavior should be consistent w/ the other methods. Looking into it, the method w/o a config object is completely unrelated to client config override for createBatchWriter() methods.  For example the following will merge in client config for batch writer for either of the createBatchWriter methods.
https://github.com/apache/accumulo/blob/af2d2d630094fc06e0ac1d6702a9a49d30ec1e94/core/src/main/java/org/apache/accumulo/core/clientImpl/ClientContext.java#L535
I think when I write the ticket I was thinking they are related.  They are not related from an impl standpoint.  I think the only relation is more from a usability standpoint, in that the methods w/ no config obj exists to support only using client props.

I'm not sure I understand what is being proposed in terms of using client properties. Is it being suggested that properties be added to the client in order to create a ConditionalWriterConfig?

I would like to be assigned to this issue please.

I created #2116 in response to this ticket. @keith-turner @ctubbsii


Missing flags to run cordinator readme example

I tried to run the readme example without docker, inside a virtuaenv configured with python3 and i had this error:
cli.py: error: the following arguments are required: --storage-endpoint, --storage-bucket, --storage-key-id, --storage-secret-access-key

There is something missing in the readme? thank you!!

I'm closing this issue because the issue/solution relates to an old version (v0.6.0) of the python coordinator that is no longer maintained.
If there is a similar issue in the upcoming version, please open a new ticket instead of opening it again.
Thank you!


Don't negotiate TLSv1.3 with the ossl_shim

The ossl_shim doesn't know about TLSv1.3 so we should disable that
protocol version for all tests for now.
This fixes the current Travis failures.
[extended tests]

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

CIs passing, so pushed!


Missing Vertical Separator Style

In a very old VisTable it was possible to set custom style between horizontal and vertical Separator: L46
Is it possible to add this feature again? Maybe this line could be edit in this way:
Cell<Separator> cell = add(new Separator(vertical ? "vertical" : "default")).padTop(2).padBottom(2);

Is there any particular reason why it was removed?
Thanks

According to the change log this was removed in VisUI 1.0.0. The reason was "it's not used". If you look at the Separator implementation at that time the vertical property of the SeparatorStyle is not used anywhere. The vertical property set to true was the only difference between the "vertical" and "default" styles. How separator looks in a table (vertical or horizontal) depends on the cell sizing and not on this vertical property.
What's your intent of using this? Do you want to use different style (background, thickness) for the vertical/horizontal separator?

Yes, the idea is to control different thickness and background between vertical and horizontal Separator, as it happens for PopupMenu.
com.kotcrab.vis.ui.widget.Separator$SeparatorStyle: {
    default: {
      background: separator,
      thickness: 2
    },
    vertical: {
      background: separator-vertical,
      thickness: 3,
    },
    menu: {
      background: separator-menu,
      thickness: 2
    }
  }

I've made it in a custom VisTable class that override addSeparator, anyway it might be useful to include by default.
    @Override
    public Cell<Separator> addSeparator (boolean vertical) {
        Cell<Separator> cell = add(new Separator(vertical ? "vertical" : "default")).padTop(2).padBottom(2);

        if (vertical)
            cell.fillY().expandY();
        else {
            cell.fillX().expandX();
            row();
        }

        return cell;
    }

I guess it can be added, this would also require adding back the vertical style to avoid breaking change.


[Bug]: SAML certificate has suddenly changed

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[ ] I have joined the ZITADEL chat

Version
7.15.0
App version
v2.51.3
Describe the problem caused by this bug
We use Kimai2 (latest helm chart) with Zitadel and SAML Auth.
Today one of our colleagues noticed that the login is no longer possible.
After some research, I realized that the SAML certificate has changed. I have not changed this or anything.
Is this renewed after a certain time or did it come with an update and was renewed in the process?
Is it possible to deactivate this automatic renewal, if there is one, or something similar? Because we also have instances with customers and it is annoying if they suddenly can no longer log in.
Unfortunately, I cannot provide any further information or data. I had compared the certificates and noticed it.
To reproduce
You can't reproduce that.
Logs
none
Expected behavior
I would expect the certificate to always remain the same and not change randomly.
Relevant Configuration
none
Additional Context
No response

I am not aware that we change the cert automatically but @stebenz will be able to confirm here.

@rhizoet
The certificate has a lifetime of 1 year in the default configuration, but can be changed.
@fforootd
https://github.com/zitadel/zitadel/blob/d92717a1c60e3125eeb5227f3659c24cce130f2f/cmd/defaults.yaml#L529
As the keys are handled also with a lifetime in OIDC, do we need to have a look that it's also possible with no expiry?

@stebenz thank you... I completely forgot about that.

I'll close this issue, since there's no bug.


Feat: Adding animations to the Cursor

Describe the feature
I want to add some animations while moving the cursor over the website, and clicking a button. This feature will surely enhance the website, and ensure an improved User experience.
Add ScreenShots
I don't have screenshots right now, because I've not implemented it yet, but if assigned, I can assure not to disappoint you by any chance.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I think you forgot to add label @Anushkabh @RitiChandak

@harshmishra19 Don't worry about that we will add it at the time of pr after seeing your work

Oh ok, I have made the PR

Okay let me check it


Add inventory loading in worker mode

This is useful in a use case where EDA is running isolated with the inventory provided locally.

Should we not add tests for the inventory here? https://github.com/ansible/ansible-rulebook/blob/main/tests/test_websocket.py


packfile: cache undeltified objects to improve decode performance

Simple object cache that keep in memory the last undeltified objects. When no more objects can be keept into memory, the oldest one is deleted.
Benchmarks:

master branch:

clone: ~4m2s
count: ~3m40s count: 207956

this PR:

clone: ~19s
count: ~14.2s count: 207956

Feel free to add any comment or ways to improve this.

Current coverage is 76.40% (diff: 87.71%)

Merging #218 into master will decrease coverage by 0.51%

@@             master       #218   diff @@
==========================================
  Files            96         97     +1
  Lines          6299       6336    +37
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           4845       4841     -4
- Misses          922        971    +49
+ Partials        532        524     -8

Powered by Codecov. Last update 51e9a9c...0348fc6

The improvement is great, but why? Will be great know the average of the re-used objects in a repository, maybe we can have a more sophisticated algorithm. How is done this at jgit and libgit?

JGit implementation: https://github.com/eclipse/jgit/blob/57a263f1823d164142235a72072154f0568cb61c/org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/file/DeltaBaseCache.java

I wrote from scratch the implementation, taking into account all your comments. Basically, is the same implementation as JGit.

Code used to check times:
package main

import (
	"time"
	"os"

	"gopkg.in/src-d/go-git.v4"
	"gopkg.in/src-d/go-git.v4/plumbing"
	"gopkg.in/src-d/go-git.v4/storage/filesystem"

	. "gopkg.in/src-d/go-git.v4/examples"
	osfs "srcd.works/go-billy.v1/os"
)

func main() {
	for i := 0; i < 3; i++ {
		directory := "/tmp/git/"
		os.RemoveAll(directory)
		s, err := filesystem.NewStorage(osfs.New(directory))
		CheckIfError(err)

		r, err := git.NewRepository(s)
		CheckIfError(err)
		Info("cloning")
		now := time.Now()
		err = r.Clone(
			&git.CloneOptions{
				URL: "file:///path/to/git/repository",
			},
		)
		CheckIfError(err)
		spent := time.Since(now)
		Info("clone: %s", spent)

		Info("starting to iterate")
		iter, err := s.ObjectStorage.IterEncodedObjects(plumbing.AnyObject)
		CheckIfError(err)

		count := 0
		now = time.Now()
		err = iter.ForEach(func(o plumbing.EncodedObject) error {
			count++
			return nil
		})
		CheckIfError(err)

		spent = time.Since(now)
		Info("count: %s count: %v", spent, count)
	}
}

We should probably improve this in the future with an LRU cache with maximum memory size, but so far it looks great. LGTM.


Add --group-directories-first

This PR adds the "--group-directories-first" option.  See #27

Hi, since this repository is abandoned I opened an issue in order to discuss about the feasibility of starting a fork in order to accept and merge contributions and keep the software maintained as far as possibile. Would you please checkout the issue and let me know what you think? I have unmerged stuff too and I would like to keep using logo-ls. Thanks!


typechecked(string) should have more consistent return type

Currently if you call typechecked on a string module name, it has case-by-case behavior to sometimes return the original string and sometimes return the resolved module:
typechecked('requests') → <module 'requests'>
typechecked('requests') → 'requests'

Could we simplify to make it unconditionally return the value that was passed in, str→str, module→module, class→class, etc?

I agree to that solution. Accepting a string in place of the module was meant for convenience, not to complicate static typechecking. I think nothing depends on this particular current behavior. Please go ahead and change it as you proposed.


How to use fs

How do I use fs in my main vue app, I need to store data in the users temp folder...?

Hey @borovez, So it is pretty simple. I have done exact same thing with one of my client's projects. So the method, I use was that I made an express API in the electron.js and used routes to make the electron.js less messy, and then I set the endpoint as any other database API would do, you can use Axios then to make post, get requests to add, update, delete data.  As you told, you want to use fs, then you can just do the same thing instead of using any database just use fs to manipulate the database, as I used sqlite3 in my case. Anyways if you didn't understand what I said, then you can ask me for some examples, I will be happy to help you.

@RajvirSingh1313 Hey, thank you for the fast response!
Okay I understand now that I should be building the API using express, and I notice there is an electron.js file in the root project of the boilerplate code, however, how should I run this electron.js? Do I need to run this seperately from the npm run serve using my main.js?
Also, unrelated, but the npm run build, the dist that is generated is just the compiled website files, how do I then package this into an .exe for windows 10? (I am new to electron.js)
Thanks!

So, let me start with your first question.
Solution 1
You need to run this command to make it work, and if you just want to like develop the API and use postman to test it. Then you can make a separate file like server.js, and copy and past the express related things from electron.js. After that, you can install nodemon for like reloading of the API every time you make a change in serever.js. And at last, make a script command in package.json like this ```json
scrip
![image](https://user-images.githubusercontent.com/63385587/144775681-3093f45a-cb6b-4264-921a-0fc0f353840a.png)

So, let me start with your first question.
Solution 1
You need to run this command to make it work

And if you just want to like develop the API and use postman to test it. Then you can make a separate file like server.js, and copy and past the express related things from electron.js. After that, you can install nodemon for like reloading of the API every time you make a change in serever.js. And at last, make a script command in package.json like this
"scripts":{
      "server":"nodemon server.js"
}

And when you will be finished making the API then you can just copy the content of the server.js and past it into electron.js
Solution 3
So yeah you would be sending requests to 5001 or the port mentioned in the electron.js.
There is one little thing that you need to know is that this method is only good if your project is just a side project or a small project in which you are not transferring sensitive data, as users will also be aware of the express API.
I am working on this right now, about every secure way to contact with electron.js, I tried IPC but isn't working right and needs a lot of boilerplate. So I am researching some other ways which can be easier to set up and easier to use. I hope I can make it someday.
I think these two were the only questions, as you solved the others yourself. So have a good day!

@borovez Should I close this issue if your doubts are cleared?


Fixed typo in swiper-events.d.ts

Update swiper-events.d.ts
Also fixed a typo in CHANGELOG.md

Thanks!


Support for unknown properties

Hey, just wondering if there is a way to check for unknown properties when parsing a JSON object.
So this library throws an error when there is one property inside the JSON object which is not specified in the corresponding JsonObject class.
Any ideas on this?

This could be done, of course. Is there a specific reason why you would want it?


Issue with quotes inside of strings

First of all I really like the extension.
I is really handy when you want to add an expression to an existing string.
Unfortunately I came across the following issue:
Say you have a string like this
const test = '"This is" "a test"';

and you add ${} between the inner quotes like this
const test = '"This is" ${} "a test"';

I would expect, that the outer single quotes get converted to back-ticks, but unfortunately the inner ones get converted.
// Expected
const test = `"This is" ${} "a test"`;
// Actual
const test = '"This is` ${} `a test"';

The same happens if you add ${} in a template string which contains quotes.

Have you tried changing the setting convert outermost quotes to true?

works as you expect for me with that setting


Error: Error building site: failed to render pages: render of "home" failed:

Error: Error building site: failed to render pages: render of "home" failed:
"/codebuild/output/src039271864/src/github.com/varunchandak/XXXXXXXXX/layouts/index.html:19:17": execute of
template failed: template: index.html:19:17: executing "index.html" at <partial "gallerydeluxe/init.html" (dict "sourcePath"
"images")>: error calling partial: partial "gallerydeluxe/init.html" timed out after 30s. This is most likely due to infinite
recursion. If this is just a slow template, you can try to increase the 'timeout' config setting.

all my files are *.jpg
current photos count is 239 (more coming soon)

my config.toml
baseURL      = "https://XXXXXXX/"
title        = "XXXXXXX"
disableKinds = ["section", "taxonomy", "term"]

[params]
    # Set to enable Plausible Analytics.
    plausible_domain = ""

    [params.gallerydeluxe]
        # Shuffle the images in the gallery to give the impression of a new gallery each page load.
        shuffle = false

        # Reverse the order of the images in the gallery.
        reverse = false

        # Enable Exif data in the gallery.
        # See https://gohugo.io/content-management/image-processing/#exif-data for how to filter tags.
        enable_exif = false

[caches]
    [caches.images]
        dir    = ':cacheDir/gallerydeluxe'
        maxAge = "4320h"                   # 6 months.

[imaging]
    resampleFilter = "CatmullRom"
    quality        = 71
    anchor         = "smart"
    [imaging.exif]
        disableDate    = false
        disableLatLong = true
        includeFields  = 'Artist|LensModel|FNumber|ExposureTime'

[server]
    [[server.headers]]
        for = '/**'
        [server.headers.values]
            Referrer-Policy = 'strict-origin-when-cross-origin'

[module]
    [[module.mounts]]
        source = "assets"
        target = "assets"
    [[module.mounts]]
        source = "layouts"
        target = "layouts"
    [[module.mounts]]
        source = "content"
        target = "content"
    [[module.mounts]]
        source = "static"
        target = "static"
    [[module.imports]]
        path = "github.com/bep/gallerydeluxe"

my go.mod file
module github.com/varunchandak/XXXXXX

go 1.19

require github.com/bep/gallerydeluxe v0.9.3 // indirect